Can't move widget down

Trying to move item onclick of Up, Down, Left and Right buttons. In all case it is working fine but not with move down. On clicking Down button get current gs-y (row) value, add +1 and call move method.  It doesn't move at all.
If I increase row number by 2 then it works but not getting expected result.
Not sure what is wrong. Could you point it out what is wrong or suggest me work around?
		switch (action) {
			case 'moveup':
				newRow = self.activeitemdata.row - 1; // self.activeitemdata.row is current row
                self.grid.move(podEl, self.activeitemdata.col, newRow);
				break;
			case 'movedown':
				newRow = self.activeitemdata.row + 1; // self.activeitemdata.row is current row
                self.grid.move(podEl, self.activeitemdata.col, newRow);
				break;
			case 'moveleft':
				newCol = self.activeitemdata.col - 1; // self.activeitemdata.col is current col
                self.grid.move(podEl, newCol, self.activeitemdata.row);
				break;
			case 'moveright':
				newCol = self.activeitemdata.col + 1; // self.activeitemdata.col is current col
                self.grid.move(podEl, newCol, self.activeitemdata.row);
				break;
		}

I don't really understand what you trying to do. Could you record a screencast or so?
By default gridstack widgets cannot float. You can try to use float http://troolee.github.io/gridstack.js/demo/float.html though

Basically I am using Gridstack to layout pod on page and allow user to rearrange pods. Now to provide keyboard support I need to create interface with buttons (Up, Down, Left and Right) button. So user can tab to button to move pod up, down, left and right in the grid.
Please refer to this image: http://www.dushpatel.ws/data/Gridstack.png
I am able to make it working for Up, Left and Right button with code provided below. When user try to move pod up using 'Up' button, I decrease current X value by -1  and use grid.move method to move it up. And so for Right button increase current Y value  +1 and for left button decrease current Y value of -1. For down button I  increase current X value +1. But it is not working. But surprisingly if increase by +2 then it works.
I was trying to debug and I assume something is wrong in fix_collision method. Could you please look into it and suggest me?
switch (action) {
            case 'moveup':
                newRow = self.activeitemdata.row - 1; // self.activeitemdata.row is current row
                self.grid.move(podEl, self.activeitemdata.col, newRow);
                break;
            case 'movedown':
                newRow = self.activeitemdata.row + 1; // self.activeitemdata.row is current row
                self.grid.move(podEl, self.activeitemdata.col, newRow);
                break;
            case 'moveleft':
                newCol = self.activeitemdata.col - 1; // self.activeitemdata.col is current col
                self.grid.move(podEl, newCol, self.activeitemdata.row);
                break;
            case 'moveright':
                newCol = self.activeitemdata.col + 1; // self.activeitemdata.col is current col
                self.grid.move(podEl, newCol, self.activeitemdata.row);
                break;
        }

I don't think gridstack will for work for your task. If you play a little bit with a demo http://troolee.github.io/gridstack.js/ you can see a logic which gridstack use:  when you move tile no. 3 one block down tile 5 will move on block down as well to prevent collision. Then if you don't use float option both blocks will move up to prevent empty space.

Thank you for you response.
I understand. When I move pod 3 down I want pod 5 to take place of it. With drag-n-drop it works. I will take a look at logic of drag-n-drop and will try to do something.

What's height of your blocks? Is it constant? You need to move it down at least on bottom block height.

height is constant 100px. I didn't follow what you suggested. What is that?

How many grid rows does each block take?

Each block takes one grid row.

All pods have same constant height and one grid row all time.

If you some ideas that you think might work then please share with me. Thank you :)

The logic executed should be the same. Could you provide a jsfiddle for us? We'll be able to get a better understanding of what you're trying to accomplish.

Closing due to inactivity. Please feel free to reopen the ticket if you can provide more information.

I'm getting this behaviour too. I want to add a keyboard interface to allow simple move up/down/left/right.
Using move works for right x+1, left x-1 and up y-1, but if I try to move an element down y+1 it does not work.

@lordelph
Try with +2 like below. It worked for me.
case 'movedown':
newRow = self.activeitemdata.row + 2; // self.activeitemdata.row is current row
self.grid.move(podEl, self.activeitemdata.col, newRow);
break;


New version ?

Are you going to publish a new version soon ? (last 3 versions were "pre" versions)

Yes, good point. Sorry for the delay. I usually test the pre-releases in unreleased software of my own. But it looks good now for a stable release.

Ok @tranb3r  there was a problem with the build pipeline. Latest version (as by now) is 2.0.20011.1: https://www.nuget.org/packages/ValueConverters/2.0.20011.1


variable "changed" to "updated"

We should replace "changed" with "updated", as the variable-changed event really triggers any time the variable is assigned to, regardless of whether or not the variable's value changed.

understood


Support docker-compose via localhost server

Is your feature request related to a problem? Please describe.
Right now, the Swagger editor, http://editor.swagger.io, doesn't allow for a local host when trying a URI
Describe the solution you'd like
Add servers: to allow use on localhost.

As a work around, a user can modify the YAML to point to their server.  Example:
From :
servers:
  - url: http://localhost:{port}
    variables:
      port:
        default: '8250'
  - url: http://senzing-api.local

To:
servers:
  - url: http://<IP_ADDRESS>:{port}
    variables:
      port:
        default: '8250'
  - url: http://senzing-api.local

Information on how to do this is at https://swagger.io/docs/specification/api-host-and-base-path/

Pull requests:

https://github.com/Senzing/senzing-rest-api/pull/35
https://github.com/Senzing/senzing-rest-api/pull/45


update version to 0.8.0

The version used in the master branch should be 0.8.0 to properly resolve dependencies, e.g. jsonapi-utils.

@mfenner Thanks. I bumped the version to 0.9.0.pre to distinguish the master branch from the 0.8.x release. When new features on master gel a bit we'll create a beta branch for 0.9.0.


Ability to vote on active proposals

For Proposals that are active (defined as in the stage where voting is possible)
Show buttons for voting YES NO or Abstain.
These should call the vote method on the StakedCelo Vote Contract. which requires setting the
proposal Id
index
yes no and abstain vote counts.
Note the contract allows voting both yes and no and obstain however UI will require going all in on one.

visually show title that links to github page


Add Test Cases

for GitHubUtil

Codecov Report

Merging #263 (c594a05) into master (b4e043d) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #263      +/-   ##
============================================
+ Coverage     67.48%   67.54%   +0.05%
- Complexity      790      791       +1
============================================
  Files           111      111
  Lines          3580     3580
  Branches        340      340
============================================
+ Hits           2416     2418       +2
+ Misses         1041     1040       -1
+ Partials        123      122       -1

Impacted Files
Coverage Δ

...in/java/seedu/address/commons/util/GitHubUtil.java
74.62% <0.00%> (+1.49%)
: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 b4e043d...c594a05. Read the comment docs.


.NET April 2021 Update - .NET 5.0.5, .NET Core 3.1.14 and .NET Core 2.1.27

Release Notes
5.0.5
3.1.14
2.1.27
Please report any issues you find with .NET 5.0.5 either responding to this issue, creating a new issue or creating a new issue in one of the following repos:

dotnet/aspnetcore - for ASP.NET Core questions
dotnet/efcore - for Entity Framework Core questions
dotnet/sdk - for CLI tools and questions
dotnet/runtime - for runtime, API, or installer issues and questions
dotnet/winforms - for WinForms issues
dotnet/wpf - for WPF issues
nuget/home - for NuGet questions and issues

Status of availability of SDK on Microsoft's Linux feeds:

Distro
5.0.5
3.1.14
2.1.27

Ubuntu 16.04
✅
✅
✅

Ubuntu 18.04
✅
✅
✅

Ubuntu 20.04
✅
✅
✅

Ubuntu 20.10
✅
✅
✅

Centos 7       ✅
✅
✅

Debian 9      ✅
✅
✅

Debian 10      ✅
✅
✅

Fedora 32     ✅
✅
✅

Fedora 33     ✅
✅
✅

OpenSUSE 15      ✅
✅
✅

SLES 12        ✅
✅
✅

SLES15     ✅
✅
✅

Note: This list refers to the Microsoft-provisioned feeds (packages.microsoft.com) and does not in any way represent direct availability in distros (eg RHEL, Fedora).
Known Issues
If there are any issues with this release we will track them here and check issues off as they're resolved. See the linked issues for details on progress and resolution details.

[x] .NET SDK 5.0.202 fails to compile netcoreapp3.1 projects https://github.com/dotnet/sdk/issues/16758 https://github.com/dotnet/sdk/issues/16765 - We have pushed NuGet packages for 3.1.14 and 2.1.27 to resolve this issue.
[ ] There is known issue https://github.com/dotnet/core/issues/6161 with snaps while running a .NET Core 3.1 applications on Ubuntu 20.04 and Ubuntu 20.10. We recommend to follow the install steps using package manager for these Distros

Can't get ".NET Desktop Runtime 5.0.5"  by windows update and My Visual Studio 16.9 still install ".NET SDK 5.0.201"

I am getting below unit test execution error both in Visual Studio IDE 2019 (16.9.3) and command line through vstest.console.exe (16.9.1). This happens even after installing the latest .NET 5.0.5 (5.0.202 SDK). Any idea?
I even tried to debug into the Newtonsoft 13.0.1 but only to find the constructor just quits and throws the exception. The module runs fine during runtime.
The failing unit test project is using the following:

xunit (2.4.1)
xunit.runner.visualstudio (2.4.3)
NSubstitute (4.2.2)
mstest.testframework (2.2.3)
Mircrosoft.NET.Test.Sdk (16.9.4)
Microsoft.QualityTools.Testing.Fakes (16.7.4-beta.20330.2)

It is testing components/apps that are .NET Core 2.1 targeting .NET FW 4.6.2. Below is the stack trace from the unit test command line execution
Error Message:
System.InvalidProgramException : Common Language Runtime detected an invalid program.
Stack Trace:
at Newtonsoft.Json.Linq.JContainer..ctor(JContainer other)
at Newtonsoft.Json.Linq.JObject..ctor(JObject other)
at Newtonsoft.Json.Linq.JObject.CloneToken()
at Newtonsoft.Json.Linq.JContainer.EnsureParentToken(JToken item, Boolean skipParentCheck)
at Newtonsoft.Json.Linq.JContainer.InsertItem(Int32 index, JToken item, Boolean skipParentCheck)
at Newtonsoft.Json.Linq.JContainer.TryAddInternal(Int32 index, Object content, Boolean skipParentCheck)
at Newtonsoft.Json.Linq.JContainer.Add(Object content)
at Newtonsoft.Json.Linq.JArray.Add(JToken item)
at MyUnitTests.<MyTest_HasNeededData_ReturnsData>d__45.MoveNext() in XXXXXXXXXXXXXXXXXXXXXXX:line 643
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.GetResult()

Version 3.1.14 is missing in the release json: [https://dotnetcli.blob.core.windows.net/dotnet/release-metadata/3.1/releases.json] .

Hello @XieJJ99 releases.json is updated for 3.1


JSON keys enclosed in double quotes

I don't know if this is possible, but I'd like to know if there's an option to force JSON keys into double quotes.
From this:
[ key1: "valueA", key2: "valueB", key3: "valueC" ]
to this:
[ "key1": "valueA", "key2": "valueB", "key3": "valueC"]
I need it using Pyhon version from code.
Thank you.
EDIT:
For the moment I've solved using regex (see https://gist.github.com/gsscoder/e212d1630bc4e7f036c63411b1262116). But I'm interested if js-beautify can do this.

The beautifier only changes whitespace formatting not text.


Python Script not listed in Data Source's "Other"

The documentation lists Python Script as a data source for Power BI: https://docs.microsoft.com/en-us/power-bi/desktop-data-sources

I do not see this as a potential source in my version of Power BI (2.62/Sept 2018)
My Get Data menu options:

My Version:

In recent versions of Power BI Desktop, it's there - directly below R script in the Other category.


Added optional AnimatedInsertedItemBuilder to AnimatedScrollView.insert/insertAll

This PR adds an optional builder parameter to AnimatedScrollView that is preferred over the itemBuilder that was specified when the ScrollView was created.
closes #100418
There is no migration necessary because this change will not break anything. However I will still add tests to ensure the functionality of the optional builder.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Test is weirdly timing out, either a flake (we can retry another day), or something with the PR. Weird.

Tests are good, just make sure you don't modify something else by mistake (I don't remember what you did in your tests), but ideally you should create a test that "fixes" this issue.

Ok, thanks, I will do it in a different commit. I will just extend the current tests.

Following https://github.com/flutter/flutter/pull/138376#discussion_r1439041807 I also made the observation that there are severe inconsistencies regarding the redundancy of the index while using the builder.
For example for the function removeAllItems:
void removeAllItems(AnimatedRemovedItemBuilder builder, { Duration duration = _kDuration }) {
  for (int i = _itemsCount - 1 ; i >= 0; i--) {
    removeItem(i, builder, duration: duration);
  }
}

There is no way to access the index used to remove the specific item. So when using this function I must keep track of the build widgets myself and need to increment through the list when there could just be an index exposed in the builder?
It seems way more intuitive as its done in insertAllItems which uses the default builder:
void insertAllItems(int index, int length, { Duration duration = _kDuration }) {
  for (int i = 0; i < length; i++) {
    insertItem(index + i, duration: duration);
  }
}

Summary:
I think this section should be refactored as a whole. Yes, it will be a breaking change, but there are many inconsistencies that complicate the API. E.g. the requirement of a builder when removing an item from the list but not even providing an option for a different insert animation (as this PR tries to solve).
My suggestion would be the following:

the itemBuilder provided when the widget is created should be treated as the default builder
removeItem and insertItem should both provide the option for a different builder just for this insertion/removal, the builders should have the same signatures as the default builder

Summary: I think this section should be refactored as a whole. Yes, it will be a breaking change, but there are many inconsistencies that complicate the API. E.g. the requirement of a builder when removing an item from the list but not even providing an option for a different insert animation (as this PR tries to solve).
My suggestion would be the following:

the itemBuilder provided when the widget is created should be treated as the default builder
removeItem and insertItem should both provide the option for a different builder just for this insertion/removal, the builders should have the same signatures as the default builder

This sounds like a larger change that would benefit from wider feedback. Typically we create a design document that explains the motivation and details of such a change so others can understand the full scope and leave comments. There is a guide in the wiki that contians a design document template here: https://github.com/flutter/flutter/wiki/Design-Documents

Sorry about the large confusion. I had a mistake in thinking about the removal process (at the time we are animating the item does not exist in the list anymore therefore we need a specific builder to animate).
However this still raises a question about the AnimatedListState.removeAllItems. I think this function is only appropriate to use if all items are build the same? In my opinion this does really confuse.

However this still raises a question about the AnimatedListState.removeAllItems. I think this function is only appropriate to use if all items are build the same? In my opinion this does really confuse.

What is the question? I think the API can be used multiple ways, an intentionally so it can support multiple kinds of effects.

Not a question at all just a lot of confusion tbh :)
I will revert the tests and introduce new ones and I will change the new type to the default item builder type.

Some vcs issues sorry


Updating to the public release of netty.

Also updating the source code to reflect the fact that Channel is now an abstract class rather than an interface.

LGTM


Little fix for PR #6

Rebased to master, resolving conflicts
Fixed a little issue with missing quotes
edited json.swim

See PR #6

@perlpunk

Issues running tests on OSX
https://gist.github.com/anonymous/c2193a6d201c56085118aabbbd90ad32


1.1.0 can't start again when I use SQLite

Gitea version (or commit ref):1.1.0
Git version:
Operating system: Windows 7 pro / Windows Server 2012 R2
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
A database file of SQLite not exists:
The file of the default is generated, and Gitea starts.
A database file of SQLite exists:
The service of Gitea does not start. (system error 1067)
System error 1067　is MySQL error.
I think that Gitea choose a database unlike setting.

@andreynering maybe you can confirm this one.

Not happening to me

@papiorasu please give more detail since @geek1011 and I cannot reproduce it.

@papiorasu could you show your config file?

I see.
I am using gitea at the company so I will present it on Monday.

APP_NAME = Gitea: Git with a cup of tea
RUN_USER = PAPIORASU$
RUN_MODE = prod

[database]
DB_TYPE  = sqlite3
HOST     = <IP_ADDRESS>:3306
NAME     = gitea
USER     = gitea
PASSWD   =
SSL_MODE = disable
PATH     = C:/gitea/data/gitea.sqlite3

[repository]
ROOT = C:/gitea/gitea-repositories

[server]
SSH_DOMAIN       = localhost
HTTP_PORT        = 3000
ROOT_URL         = http://localhost:3000/
DISABLE_SSH      = false
SSH_PORT         = 22
LFS_START_SERVER = false
OFFLINE_MODE     = false

[mailer]
ENABLED = false

[service]
REGISTER_EMAIL_CONFIRM     = false
ENABLE_NOTIFY_MAIL         = false
DISABLE_REGISTRATION       = false
ENABLE_CAPTCHA             = false
REQUIRE_SIGNIN_VIEW        = false
DEFAULT_KEEP_EMAIL_PRIVATE = false
NO_REPLY_ADDRESS           = noreply.example.org

[picture]
DISABLE_GRAVATAR        = false
ENABLE_FEDERATED_AVATAR = false

[session]
PROVIDER = file

[log]
MODE      = file
LEVEL     = Info
ROOT_PATH = C:/gitea/log

[security]
INSTALL_LOCK = true
SECRET_KEY   = 34T7KAqOzV
And MySQL is not installed in my environment.
Furthermore I did a more detailed test.
I tested whether it can be started with 1.0.2 and 1.1.0.

Gitea Version

1.0.2

1.1.0

Generate Database Version

1.0

1/30

30/30

1.0.2

2/30

1/30

1.1.0

2/30

2/30

I prepared a database that is running (generated by 1.0) and newly generated one (generated by 1.0.2 and 1.1.0).
This is the number of times an error occurred after starting the service 30 times.
I used the following files.
Gitea-1.0.2-windows-4.0-amd64.exe
Gitea-1.1.0-windows-4.0-amd64.exe

Are you running a windows service?

Yes. I referred to Gitea 's blog.
https://docs.gitea.io/en-us/windows-service/

I confirmed that Gitea can be started by changing the "Version" table of the used SQLite database.
I changed "Version" from 15 to 22.
In the case of a newly created database, this process was performed automatically by Gitea.
However, Gitea of Ver.1.1.0 has not executed this processing in my used database.
So I manually fixed the database. And Gitea started.
The above-mentioned problem that does not start "30/30 times" has been solved.
However, the problems caused by Windows services remain.

So what's the issue it is?

First, It is a bug that Gitea has failed to change the "Version" of the used database.
Second, Windows services often do not start up.
I think this is the same as Issue 821.
Windows 7 Fails to start as a service, Web command works https://github.com/go-gitea/gitea/issues/821
I do not know the cause of this issue.

So let's discuss there.


Importing JSON file as type only conflicts with import attribute type requirement

🔎 Search Terms
"import attributes", "import assert", "type json"
🕗 Version & Regression Information

This changed between versions 5.3.3 and 5.7.2
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about import attributes and type only imports
I was unable to test this on prior versions because _______

⏯ Playground Link
Couldn't get playground to create a separate module, so attaching a minimal example
typescript-json-repro.zip
run
npm i
npm run build

💻 Code
// tsc fails with: Importing a JSON file into an ECMAScript module requires a 'type: "json"'
import type enMessages from "./en.json";

🙁 Actual behavior
When I try to address the issue as TSC suggests I run into another:
// tsc fails with: Importing a JSON file into an ECMAScript module requires a 'type: "json"'
import type enMessages from "./en.json";

// tsc fails with: Import attributes cannot be used with type-only imports or exports. ts(2857)
import type enMessages from "./en.json" with { type: "json" };

// tsc is happy but me and eslint are not, I don't use imported module for anything other than typings in this case
import enMessages from "./en.json" with { type: "json" };

🙂 Expected behavior
Type only imports should not require specifying 'type: "json"' as they are not retained in the JS output. Alternatively import attributes should be allowed with type only imports. One of the options below should work
import type enMessages from "./en.json";

import enMessages from "./en.json" with { type: "json" };

Additional information about the issue
tsconfig.json
{
  "compilerOptions": {
    "target": "esnext",
    "esModuleInterop": true,
    "module": "NodeNext",
    "moduleResolution": "NodeNext",
    "resolveJsonModule": true,
    "declaration": true,
    "rootDir": "src",
    "outDir": "dist/types"
  },
  "include": ["./src/**/*", "./src/**/*.json"],
  "exclude": ["build", "node_modules", "dist"]
}

I think the second option in expected behaviour should be:
import type enMessages from "./en.json" with { type: "json" };

(i.e. with a type)

We've encountered the same issue when upgrading from 5.6.3 to 5.7.2 (https://github.com/PREreview/prereview.org/pull/2130). Making it a regular import leads to an import/no-extraneous-dependencies ESLint failure, as the package we're importing from is only used for its types (as so is a dev dependency).


Consider more info on dates and related products

It would be awesome if this had release dates and maybe the corresponding version of .NET and Visual Studio. I had to go to wikipedia.org for that info and that page is not fully up to date (the other .NET 6 versions are missing as well as that C# 11 is released.
Also, just saying, this is an awesome article.

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

ID: 3d252158-0367-77c8-1816-dbc385c5c40b
Version Independent ID: 575872e5-6273-f8f6-e3e2-2b911cd9d009
Content: The history of C# - C# Guide
Content Source: docs/csharp/whats-new/csharp-version-history.md
Product: dotnet-csharp
Technology: csharp-whats-new
GitHub Login: @erikdietrich
Microsoft Alias: wiwagn

I'm planning updates to this article next month. I'll handle these at the same time.


keyboard shortcuts for slideflow-studio

Hi,
My slides have a lot of letters and labels in the scanned image. Sometimes the tissue area itself is small and tile extraction followed by Gaussian + Otsu do not yield a lot of tiles. If I make a ROI, the number of tiles will increase dramatically.
I have a lot of slides to go through (hundreds). If shortcuts for "open the next slide in the project" and "Add ROI (Freehand)" exist, it'll improve the speed of making ROIs. Is it something realistic to implement? Thanks!

These shortcuts actually already exist, although I do not believe we have explicitly mentioned them anywhere in the docs. They are:

Open next slide:  +
Add free-hand ROI: A
Add polygon ROI: P
Edit ROIs: E
Save ROIs:  + S
Exit ROI editing: A, or

Thanks!


Resolve environment variables with webpack

I'm still getting the import/no-unresolved error.
I followed closely the Simple approuch tutorial on webpack's website.
Now I have the following files:
webpack.config.babel.js
webpack.common.js
webpack.dev.js
webpack.prod.js

I have this in my webpack.config.babel.js:
module.exports = function(env) {
  return require(`./webpack.${env}.js`);
};

I starting webpack with the following in package.json:
"build:dev": "webpack-dev-server --open --env=dev --progress --profile --colors",
"build:dist": "webpack --env=prod --progress --profile --colors",

The actual resolving object in webpack is in the webpack.common.js file:
resolve: {
  extensions: ['.js', '.json'],
  alias: {
    'three/GLTF2Loader': path.join(__dirname, 'node_modules/three/examples/js/loaders/GLTF2Loader.js'),
    'three/Stats': path.join(__dirname, 'node_modules/three/examples/js/libs/stats.min.js'),
    'three/OrbitControls': path.join(__dirname, 'node_modules/three/examples/js/controls/OrbitControls.js'),
  },
},

This is in my eslintrc.js file:
settings: {
  'import/resolver': {
    webpack: {
      config: 'webpack.common.js',
    },
  },
},

@DJviolin have you found a solution? I have the same problem. Aliases are not recognized.

Not yet, in fact I didn't worked with Webpack since. I'm building the project without bundling.
Just a note, tree shaking also broken with three.js, AR.js. I'm constantly getting bigger file sizes than without. Webpack is "awesome"... :(

@KrisOlszewski Have you found some solution?
I am using this config to disable the warning for now:
  "settings": {
    "import/core-modules": ["my-alias"]
  }

I think that perhaps you want a fallback, so that --env can be omitted, and this will work?
Happy to reopen if not.


CHEBI:75930, please CHECK

Hi Gareth and Steve,
Please check CHEBI:75930, a 3,4-didehydropyranoside.
It is needed for EC <IP_ADDRESS>, http://www.enzyme-database.org/query.php?ec=<IP_ADDRESS>
Cheers,
Kristian

Done as requested.
Cheers,
Steve


Skyline: Clean-up "StaticMod.PrecisionRequired" changes

These changes ended up not getting used and just add confusion to the code
Revert UniModCompiler/InputFiles/unimod.xml to what was actually used to generate UniModData.cs
Add a few changes to UniModTemplate.cs to make it match UniModData.cs

Nick, I decided to do a bit of clean-up of the "RequiredPrecision" code before applying my unimod.xml update in the hope that doing this will make the update easier to understand without all the changes to precision required for the modifications. Glad you final solution did not require this attribute. It does make a bit of a mess out of updates.


clean-venv-cache: Handle cleaning up zulip-thumbor-venv properly.

We make clean-venv-cache aware of zulip-thumbor-venv so that its cache clean up is handled properly and cache isn't deleted when it wasn't supposed to.

Merged, after undoing the changes to the if False section (which broke the production build).
Thanks @adnrs96!

Thanks for cleaning it up and merging!


Kroki request blocked by CORS policy

There is suddenly an issue with accessing the kroki API (which is used for most diagram types).
Access to fetch at 'https://kroki.io/d2/svg/eNqrUNC1U6i0UshIzcnJVyjPL8pJ4QIAP2UGXQ==' from origin 'lsp://logseq.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Tested with logseq 0.8.16 on Windows 11 and MacOS 13.1
This only happens with the marketplace version of the plugin.
When installing the local build (using 'Load unpacked plugin') everything works as expected.
As a workaround you can download and install the plugin manually:

Download the latest release (e.g. logseq-diagrams-as-code-0.0.18.zip) from https://github.com/npgrosser/logseq-diagrams-as-code/releases
Unpack it
In Menu/Settings/Advanced enable Developer mode
In Menu/Plugins click on 'Load unpacked plugin' and select the 'logseq-diagrams-as-code' folder

There was a new Kroki release yesterday that might be related. I will check it when I have time

quickfix/workaround:
https://github.com/npgrosser/logseq-diagrams-as-code/commit/0bafd20bffc1f2f4533c12309494944eeee7b6d4

Any update on this? I'm seeing 500s and 504s now

Do you still have this problem? 500 errors indicate that there was a problem with the kroki service itself.
It works for me, so the issues have probably been resolved

I figured it was something related to Kroki, but could also be malformed request; which is also unrelated to CORS.
I'm still getting errors. Strangely it looks like it might just be mermaid based diagrams.
On Fri, Feb 03, 2023 at 4:11 PM, Nicolai Großer < @.*** > wrote:

Do you still have this problem? 500 errors indicate that there was a
problem with the kroki service itself.
It works for me, so the issues have probably been resolved
—
Reply to this email directly, view it on GitHub (
https://github.com/npgrosser/logseq-diagrams-as-code/issues/16#issuecomment-1416468540
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAAPYV7IB2RTWUTTDDANQHTWVV7CFANCNFSM6AAAAAAUA44D4Q
).
You are receiving this because you commented. Message ID: <npgrosser/logseq-diagrams-as-code/issues/16/1416468540
@ github. com>

If you think that this has something to do with the plugin, feel free to post the error message and a diagram example to reproduce the error.

Still seeing it in 0.0.19

Access to fetch at 'https://kroki.io/mermaid/svg/eNpLL0osyFAIcbHmctTVtXOCUM7WXE5ACijoDKYAstUIjg==' from origin 'lsp://logseq.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

CORS error - falling back to default ImgSrcRenderer: TypeError: Failed to fetch

Failed to load resource: the server responded with a status of 504 ()

For me the CORS issue seems to be resolved. Which version of logseq do you use @tcz717 ?

For me the CORS issue seems to be resolved. Which version of logseq do you use @tcz717 ?

My logseq version is 0.8.17


[FoxNews] HTTP Error 403: Forbidden

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: ['-f', '(bestvideo+bestaudio)[ext=webm][height <=? 720]/best[height=720]/(bestvideo+bestaudio)[height <=? 720]/best', '--prefer-free-formats', '--add-metadata', '-c', '--sub-lang', 'enUS', '--prefer-ffmpeg', '--cookies=/home/username/.config/cookies.txt', '--user-agent', 'Mozilla/5.0 (X11; Linux x86_64; rv:52.9) Gecko/20100101 Goanna/4.1 Firefox/52.9 PaleMoon/28.1.0']
[debug] Custom config: []
[debug] Command-line args: ['-s', 'https://video.foxnews.com/v/5614615980001/?#sp=watch-live', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.10.05
[debug] Python version 3.6.6 (CPython) - Linux-4.9.0-8-amd64-x86_64-with-debian-9.5
[debug] exe versions: ffmpeg 3.2.12-1, ffprobe 3.2.12-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[foxnews] Downloading Akamai AMP feed
[foxnews]<PHONE_NUMBER>001: Downloading f4m manifest
WARNING: Unable to download f4m manifest: HTTP Error 403: Forbidden
[foxnews]<PHONE_NUMBER>001: Downloading m3u8 information
WARNING: Failed to download m3u8 information: HTTP Error 403: Forbidden
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 507, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/foxnews.py", line 73, in _real_extract
    'http://%s/v/feed/video/%s.js?template=fox' % (host, video_id))
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/amp.py", line 89, in _extract_feed_info
    self._sort_formats(formats)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1286, in _sort_formats
    raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://video.foxnews.com/v/5614615980001/?#sp=watch-live

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Streams under this site that require authentication continue to return a HTTP Error 403: Forbidden (even when a Netscape cookie file is used after logging in through the browser).

Fox News does not support youtube-dl at the moment, thank you. :)

I started seeing this last year some time.  Still happening:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--prefer-ffmpeg', u'-o', u'%(series)s - s%(season_number)02de%(episode_number)02d - %(title)s.%(ext)s', u'https://www.fox.com/watch/2e9b46c3bec1d71febd4ab82a929a37e/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-32-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2
[debug] Proxy map: {}
[FOX] 2e9b46c3bec1d71febd4ab82a929a37e: Downloading JSON metadata
[FOX] 2e9b46c3bec1d71febd4ab82a929a37e: Downloading webpage
[uplynk:preplay] 751cbef9e7a74795864f510a1459fb6a: Downloading JSON metadata
[uplynk:preplay] 751cbef9e7a74795864f510a1459fb6a: Downloading m3u8 information
[uplynk:preplay] 751cbef9e7a74795864f510a1459fb6a: Downloading JSON metadata
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'http://content-auso2.uplynk.com/751cbef9e7a74795864f510a1459fb6a/k.m3u8'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 636
[download] Destination: The Gifted - s02e04 - outMatched.mp4
[download]   0.2% of ~1.65GiB at Unknown speed ETA Unknown ETAERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1908, in process_info
success = dl(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1847, in dl
return fd.download(name, info)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 364, in download
return self.real_download(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/hls.py", line 168, in real_download
self._prepare_url(info_dict, decrypt_info['URI'])).read()
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2211, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 473, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
HTTPError: HTTP Error 403: Forbidden
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--prefer-ffmpeg', u'-o', u'%(series)s - s%(season_number)02de%(episode_number)02d - %(title)s.%(ext)s', u'https://www.fox.com/watch/a7980139ac6b1bed706dfa6abf8a2d0b/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-32-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2
[debug] Proxy map: {}
[FOX] a7980139ac6b1bed706dfa6abf8a2d0b: Downloading JSON metadata
[FOX] a7980139ac6b1bed706dfa6abf8a2d0b: Downloading webpage
[uplynk:preplay] cc4ee5577e85427bb8696d0e5e83b2fe: Downloading JSON metadata
[uplynk:preplay] cc4ee5577e85427bb8696d0e5e83b2fe: Downloading m3u8 information
[uplynk:preplay] cc4ee5577e85427bb8696d0e5e83b2fe: Downloading JSON metadata
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'http://content-ause5.uplynk.com/cc4ee5577e85427bb8696d0e5e83b2fe/k.m3u8'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 650
[download] Destination: Lethal Weapon - s03e04 - Leo Getz Justice.mp4
[download]   0.2% of ~1.63GiB at Unknown speed ETA Unknown ETAERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1908, in process_info
success = dl(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1847, in dl
return fd.download(name, info)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 364, in download
return self.real_download(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/hls.py", line 168, in real_download
self._prepare_url(info_dict, decrypt_info['URI'])).read()
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2211, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 473, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
HTTPError: HTTP Error 403: Forbidden


Don't delete 'headless' services

Not currently passing the new unit test, but I believe that's because I've got some sort of type mismatch on the WithSpecField arguments, but I'm not 100% sure on that.
I'm fairly confident that the code change fixes the issue, but could use some guidance on what to fix with the test case.
Fixes #168

Ah, @nrb I see the issue. Prepare(...) errors if there's no "ports" key within the spec map. Adding a .WithSpecField("ports", []interface{}{}) (see existing clusterIP (only)... test case) should get rid of that.

@skriss Ahh, ok. I had that originally but deleted it. I'll re-add. Thanks!

Here's some sample output, after finally getting a new pod with my patched binary into my cluster.
% k get service
NAME            TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes      ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d
nginx-service   ClusterIP   None         <none>        80/TCP    21h

% _output/bin/linux/amd64/ark backup create service-test
Backup "service-test" created successfully.

% k delete service nginx-service
service "nginx-service" deleted

% k get service
NAME         TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes   ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d

% _output/bin/linux/amd64/ark restore create service-test
Restore "service-test-20171101140257" created successfully.

% k get service
NAME            TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes      ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d
nginx-service   ClusterIP   None         <none>        80/TCP    1m

And since I changed the function used for comparison, another round of testing here.
x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% k get service
NAME            TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes      ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d
nginx-service   ClusterIP   None         <none>        80/TCP    50m

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% _output/bin/linux/amd64/ark backup create service-test2
An error occurred: backups.ark.heptio.com "service-test2" already exists

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% _output/bin/linux/amd64/ark backup create service-test-2
Backup "service-test-2" created successfully.

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% k delete service nginx-service
service "nginx-service" deleted

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% k get service
NAME         TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes   ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% _output/bin/linux/amd64/ark restore create service-test-2
Restore "service-test-2-20171101145538" created successfully.

x1c in /home/nrb/go/src/github.com/heptio/ark (git) 168-fix U
% k get service
NAME            TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes      ClusterIP   <IP_ADDRESS>    <none>        443/TCP   1d
nginx-service   ClusterIP   None         <none>        80/TCP    1m

LGTM.


feature: multi-file select in file picker

Feature request
When the avante file picker opens, it would be amazingly convenient to be able to multi-select files as i fuzzy search

Motivation
No response
Other
No response

There is PR about this! https://github.com/yetone/avante.nvim/pull/971
I also want this feature and am waiting for the PR to get merged.


Okay but where is the file

I want to migrate from jsparticles since is has some problems, but cant find the tsparticles.js file anywhere. If you put it somewhere visible it would be nice, since I dont know how to use webkit to install it on the client side with import or anything

Hi @alenieto97
You can download the file from jsDelivr or cdnjs.
If you cloned the repository you can run npm run build or yarn build in the repository folder and the dist folder contains the compiled files.
You can checkout the release tab here on GitHub too, the file should be there too.
If you need more help let me know


feat!: remove external-link-icon plugin

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
681

Relevant Lines:
1238

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
681

Relevant Lines:
1238

💛  - Coveralls


fixed main->develop synchronization

From now on we'll do a merge commit and not squash between develop and main, as it can result in unexpected behavior such as the Star History section duplication


Add Command to change the "debug" target

Brief Issue Summary
I want to write a launch.json so I can automatically Build and Debug specific targets.
CMake Tools provides two commands: CMake: Set Debug Target (@command:cmake.selectLaunchTarget)
and CMake: Set Build Target (@command:cmake.setDefaultTarget)
However, these prompt the user to select a target manually when used in a task.
What I suggest are two new commands:
CMake: Set Debug Target By Name and CMake: Set Build Target By Name.
The desired usage is a launch.json / task.json like this:
"tasks": [
		{
			"isBuildCommand": true,
			"taskName": "SetMyDebugTarget",
			"command": "${command:cmake.selectDebugTargetByName}", // the new command that selects the Debug Target by Name
			"args": ["MyDebugTarget"] // The CMake Debug Target I want to debug
		}
	]

"configurations": [
	{
		"name": "Debug my Debug Target",
		"type": "cppvsdbg",
		"request": "launch",
		"program": "${command:cmake.launchTargetPath}",
		"cwd": "${workspaceFolder}",
		"args": ["my Arg to MyDebugTarget"],
		"preLaunchTask": "SetMyDebugTarget" // uses SetMyDebugTarget from tasks.json
	}
]

CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

starting in 1.11, you can change the target for your build task in the following way:
       {
            "type": "cmake",
            "label": "CMake: build",
            "command": "build",
            "targets": [
                "all"
            ],
            "group": "build",
            "problemMatcher": [],
            "detail": "CMake template build task"
        }

There is not currently a way to set the "debug" target. Thank you for suggesting this feature. We may not get to it immediately, We would accept a PR from the community if someone wants to work on this issue.

Related: #1941.


Keyboard key event listener

Hey,
I want to achieve something for that i need to know if user presses spacebar on his keyboard. So how to know if user presses spacebar? I tried using keyboard listener class and its not working
Expected results: I should be able to know when user presses spacebar
Actual results: No way to listen key press

[✓] Flutter (Channel stable, v1.17.5, on Mac OS X 10.15.5 19F101, locale en-GB)
    • Flutter version 1.17.5 at /users/faiyyazkhatri/StudioProjects/flutter
    • Framework revision 8af6b2f038 (2 weeks ago), 2020-06-30 12:53:55 -0700
    • Engine revision ee76268252
    • Dart version 2.8.4

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /users/faiyyazkhatri/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • ANDROID_HOME = /users/faiyyazkhatri/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (1 available)
    • iPhone X • D3E62461-D931-44CD-842D-DF73590F8890 • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-5 (simulator)

• No issues found!

Hi @Faiyyaz
RawKeyboardListener is working fine on MacOS and Web.

Code Sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

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

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  FocusNode _focusNode = new FocusNode();

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Material App Bar'),
        ),
        body: Center(
          child: RawKeyboardListener(
              focusNode: _focusNode,
              onKey: (RawKeyEvent event) {
                // if (event.runtimeType.toString() == 'RawKeyDownEvent') {
                //   RawKeyEventDataWeb data = event.data as RawKeyEventDataWeb;
                //   String _keyCode;
                //   _keyCode =
                //       data.key.toString(); //keycode of key event (66 is return)

                //   print("$_keyCode is pressed");
                // }

                // if (event.runtimeType.toString() == 'RawKeyDownEvent') {
                //   RawKeyEventDataMacOs data =
                //       event.data as RawKeyEventDataMacOs;
                //   String _keyCode;
                //   _keyCode = data.keyCode
                //       .toString(); //keycode of key event (66 is return)

                //   print("$_keyCode is pressed");
                // }

                print('GOT AN EVENT! $event');
              },
              child: TextField()),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.20.0-7.1.pre, on Mac OS X 10.15.6 19G73, locale
    en-GB)
    • Flutter version 1.20.0-7.1.pre at /Users/taha/Code/flutter_dev
    • Framework revision 7736f3bc90 (5 days ago), 2020-07-10 16:33:05 -0700
    • Engine revision d48085141c
    • Dart version 2.9.0 (build 2.9.0-21.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/taha/Code/sdk
    • Platform android-30, build-tools 30.0.0
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (5 available)
    • SM M305F (mobile)      • 32003c30dc19668f          • android-arm64  •
      Android 10 (API 29)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      13.6
    • macOS (desktop)        • macos                     • darwin-x64     • Mac
      OS X 10.15.6 19G73
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 83.0.4103.116

• No issues found!

if you're looking iOS key events, there is an open issue for it, f you disagree, please write in the comments and I will reopen it.
Closing as duplicate of https://github.com/flutter/flutter/issues/31945
Thank you


Making a time-stamped backup of /etc/hosts before overwriting it.

The title pretty sums it all, how about making a time-stamped backup of the hosts file before overwriting it?

This is not the behavior I want, because I use this script on my phone. But I can implement the backup as an optional argument.

Fixed (0e6aeb4948b013d1f3009f9f40897582a4b23988).


ci: add required identifier fiedl for workflow_call

Issue #

Description

The same as issue https://github.com/aws-amplify/aws-sdk-ios/pull/4915, currency workflow with group name format <github.workflow>-<PRNumber or branchName> will have the same group name when triggered by workflow_call event. Adding the input field identifier to specify the workflow name.
General Checklist

[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all target using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[X] PR title conforms to conventional commit style
[ ] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #3166 (7dbf4e5) into main (b9359ad) will decrease coverage by 4.43%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #3166      +/-   ##
==========================================
- Coverage   64.39%   59.97%   -4.43%
==========================================
  Files        1068      785     -283
  Lines       36143    25683   -10460
==========================================
- Hits        23276    15403    -7873
+ Misses      12867    10280    -2587

Flag
Coverage Δ

API_plugin_unit_test
66.81% <ø> (ø)

AWSPluginsCore
69.73% <ø> (ø)

Amplify
48.01% <ø> (ø)

Analytics_plugin_unit_test
86.39% <ø> (ø)

Auth_plugin_unit_test
?

CoreMLPredictions_plugin_unit_test
59.44% <ø> (ø)

DataStore_plugin_unit_test
80.05% <ø> (-1.07%)
:arrow_down:

Geo_plugin_unit_test
53.93% <ø> (ø)

Logging_plugin_unit_test
62.57% <ø> (ø)

Predictions_plugin_unit_test
35.17% <ø> (ø)

PushNotifications_plugin_unit_test
69.43% <ø> (ø)

Storage_plugin_unit_test
54.10% <ø> (ø)

unit_tests
59.97% <ø> (-4.43%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 291 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


why is the padding added to path line?

There are two y axis ticks and i see path and y axis ticks are not in sync
This is when there are only 0s or 1s.
Any solution?
Thanks

go it working by settting axis.y.padding.top and bottom to 0


feat(protocol-kit): add short name for Zora Testnet

What it solves
Adds short names for Zora Testnet:
https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-999999999.json

Solved in https://github.com/safe-global/safe-core-sdk/pull/716


Fire/EMS

Describe the feature
When making a Fire/EMS personal on the CAD it says Deputy. Can you change it something else?
Additional Context
No response

What do you mean by "Personal"?

I mean The people who go on duty as Fire/EMS they have to create a citizen then go to create a deputy, instead of it saying deputy maybe it say something else?

You should be able to translate this. See docs: https://cad-docs.caspertheghost.me/docs/features/translating


Resolve #17 optimize use of '^' / '\A'

The problem was that a regular expression like ^x would take linear time to is_match a string aaa.... when all it needed to do was check the first character then bail out. This PR makes it so if the regex begins with ^ (without multiline) or \A, then it stops iterating over characters as soon as there are no possible matches.
Benchmarks:
dynamic:
anchored_literal_long_non_match: 7492 ns/iter => 435 ns/iter
anchored_literal_short_non_match: 917 ns/iter => 431 ns/iter

native:
anchored_literal_long_non_match: 5020 ns/iter => 50 ns/iter
anchored_literal_short_non_match: 375 ns/iter => 50 ns/iter

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Thanks!
Could I ask you to expand the commit message a little to describe what the problem is and how it is fixed? Just saying the issue number is better than nothing, but it does mean that someone looking the source and checking the git history needs to cross reference against github (which may have disappeared at that point).
(I'm happy to help if you need assistance with git or anything else.)

Sure, expanded. Is that alright?

@tmerr This looks great. I'm happy to see that the change is small. I just had one question, but otherwise this looks awesome. Thank you!

@tmerr that messsage looks much better, thanks! Could you place it into the git commit message (e.g. git commit --amend and then git push -f to update the PR by force pushing over this branch.)

@huonw Done, thanks for the help!
It's been a blast reading through this code by the way. I had been aware of NFAs but only  the "draw pretty circles on paper with lines between them" version, not as a machine with split and jump instructions. The two models of thinking about it meld together nicely

Thanks! I merged as 10edff6bb76f1c9f4423f3bb8ad373d0c7a69773 (there was a piece of trailing whitespace which I took the initiative to remove).


[Feature Request] Make the sign route demo ready

### Tasks
- [ ] Add error handling
- [ ] Improve styling for mobile
- [ ] https://github.com/near/near-fastauth-wallet/issues/3

@Patrick1904 This ticket should unblock your demo
This is required before NEARCON

Is this done?


feat: rename CHAIN_NAMESPACE.Bitcoin -> CHAIN_NAMESPACE.Utxo

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #2635   👈

PR #2636

PR #2637

This comment was auto-generated by Graphite.


Add increaseIndentPattern/decreaseIndentPattern settings

This PR adds increaseIndentPattern and decreaseIndentPattern settings to support auto indentation.
The content is taken from https://github.com/elixir-editors/elixir-tmbundle/blob/master/Preferences/Indent.tmPreferences
https://user-images.githubusercontent.com/471335/191035946-2be8fc27-64ba-480b-868a-fca415383976.mov
https://user-images.githubusercontent.com/471335/191036001-8f9f385c-0608-4c2f-9047-007c82ec7a5a.mov

Merged it into my branch. :+1:


Hydrowood pinned&grouped count

1.all localized
2.change all chrome. to chrome.||browser. fitting firefox(no test yet)
3.change window.alert to window.confirm (some URL like chrome:// and extension store, as well as switching active tabs will not show window.confirm window and close immediately tab )
4.some errors is uncaught or showing up, but don't affect the extension's function
Your version also doesn't have the ability to close about:blank tabs by extension after the tab limit is exceeded, which means that users can create unlimited about:blanks in the latest versions of browsers anyway. trying to block it in manifestV3 results in  Error: Cannot remove NTP tabs. So I don't think the inability to limit about:blank tabs is a bug.

Thanks @king0billy 😊


Post to Slack channel every time an execution is started/completed

See https://github.com/o2r-project/o2r-bouncer/issues/4

We don't use Slack anymore.


Internal load balancer support

API changes:

loadBalancerListener.add

sourcePort is optional now
new parameter - privatePort - was added.

Either one or another parameter is required; both can be specified as well.
If sourcePort only is specified, private port is defaulted to it.

loadBalancerService.add

new parameter is added to launchConfig - "expose". Every port defined in expose,  will get translated to the listener having NULL public port, and port=port from expose set, and Format for expose is: [/[protocol]]. If no protocol is specified, http listener will get created.
if both ports and expose is specified, for every port defined in the expose, private listener will get created; and for each port - public listener.

@ibuildthecloud Darren, please review. As a part of the feature, I had to enable "expose" parameter on the launchConfig (container) object. The parameter is read by ServiceDiscovery code only, to create loadBalancerListener, and not propagated to the backend for containers. Let me know if we should add backend implementation for "expose" as a part of this PR, or separate.

https://github.com/rancher/rancher/issues/1433

I'd hold off on expose in the backend

Closing this PR as the feature is going to be merged as a part of https://github.com/rancher/cattle/pull/704


Quality check: Metadata updates in AAC Content Pattern Library

The purpose of this PR is to edit the AAC templates.
Here's the ADO work item.
This is the first PR of two.

Hi @ShannonLeavitt - requesting PM review. Thanks!

I'm closing this PR temporarily. We will reopen it when we have more time to fix everything.
#please-close


EquivlentExpression-mapper should not check for null-values in IsMatch-method

When I using the the collection mapper with entities loaded from EF7 the equivlent expression mapper return that it not shoul be used for the type. The IsMatch method should only verify the type of the data, not if the data contains any values.
I was not able to get the same issue when I not was using EF7 and putting together a simple unit-test was not succeeded either.

I don't know about this change.  The idea is if the source or the destination is null, then it's impossible to do comparison against the two lists, so the array mapper in AutoMapper should be used instead.
One thing with EF6 is that in order to save I can't new up the Poco objects.  I have to do Table().CreateNew() in order to get an object that I can save back to the database.
Also can you default your lists to be not null, to fix this problem?
Can you give me an example of code that fails, with error message, or issue in more detail?

I created a test class project here: https://github.com/Tasteful/Automapper.Collection/tree/Tests
That will have 2 tests that should be run individual, otherwise the mappings will be wrong.
One test is with the original code and the other one is with the updated mapper from the #2 PR.
The exception that will be thrown is shown in this capture.

Also if we look at the other mappers that is in AutoMapper they will not either check for null-values. I have not going down in AutoMappers code but it can be that AutoMapper are caching the IObjectMapper the type->type should use and in that case if the source or destination value is null it will not store the IObjectMapper as possible mapping of the types.

Can you set ProductDto.FieldData to equal a new list in the constructor and have that fix the problem?  Or make it virtual?  I know in EF 6 you had to make it virtual so that when the proxy class around it was generated it could overwrite the list with the data from the DBContext.  That could be why it's null and failing to map over.  I don't know if EF 7 handles this the same way, but I would believe it would.
Can't build it right now to test this out, but can try over the weekend if need be.
Also names are backwards.  Dto's should not be the EF objects but the ones the EF objects transfer over too.  Not critical, but names threw me off at first glance.

I can try and see if the error still are there if I set the collection in
the ctor.
For the moment I use an private build with my fixes in that I also added
strong naming to. I would like to not have to maintain an own version.
On 16 Oct 2015 19:24, "Tyler Carlson"<EMAIL_ADDRESS>wrote:

Can you set ProductDto.FieldData to equal a new list in the constructor
and have that fix the problem? Or make it virtual? I know in EF 6 you had
to make it virtual so that when the proxy class around it was generated it
could overwrite the list with the data from the DBContext. That could be
why it's null and failing to map over. I don't know if EF 7 handles this
the same way, but I would believe it would.
Can't build it right now to test this out, but can try over the weekend
if need be.
Also names are backwards. Dto's should not be the EF objects but the ones
the EF objects transfer over too. Not critical, but names threw me off at
first glance.
—
Reply to this email directly or view it on GitHub.

I tested to initiate the collections in the ctor and then the default implementation is working. But the IsMatch should not check null values. If we check automapper-code https://github.com/AutoMapper/AutoMapper/blob/develop/src/AutoMapper/MappingEngine.cs#L404 we can see that AutoMapper will cache the fist IObjectMapper that will get hit for the source and destination type. If one of the values is null the first execution the mapping cache will contains wrong record. If the ArrayMapper should be used if some of the values was null the ArrayMapper should probably be executed as a base class for the current implementation.

I never noticed that.  Would assume it would check multiple times.  And in EF 6 I would use CreateNew instead of new T which would never have nulls and this problem never showed itself explicitly.
Will have to bring this issue up in AutoMapper, because other things might have this problem.  Maybe make IsMatch only uses TypePair.
I agree with you now this has to be changed.

Also updated tests to do auto-generated equivalency expression
https://github.com/TylerCarlson1/Automapper.Collection/tree/Tests
Can't run tests right now, but that should be the gist of what you need to not have to write an EqualityComparision for every mapping.  Thankfully they made finding the Model and Primary Keys super easy compared to EF6.

I will merge them down and test tomorrow.
How is the autofinding part working when the entity in database has an composit key over multiple colums but in the entity that it got translated to is missing one of the fields, can it map the other colums automatomatic?

If you have a composite key you must have mappings from all columns in the key in order to make a valid equivalency expression and have the collection mapper be used in the mapping.
If you look at GenerateEquivilentExpressionsBasedOnGeneratePropertyMaps the logic for this is there, but you can always make a new class and make it work out differently.  You can always override this behavior by using EqualityExpression if you have a rare or unique scenario.  You can make your own rules to do whatever you want,  I just made it for EF's primary keys and only if all have property maps.

Updated logic to not include nulls on IsMatch, but if source or destination is null on map use CollectionMapper to resolve.  Don't want to duplicate code if I don't have to.  And it was my original intent to fall back on CollectionMapper.

Did the GenerateEquivilentExpressionsBasedOnGeneratePropertyMaps work, or was of any help for EF7?

I didn't try. I only use a part of the primary key as the key for
collection mapping.
On 22 Oct 2015 22:01, "Tyler Carlson"<EMAIL_ADDRESS>wrote:

Did the GenerateEquivilentExpressionsBasedOnGeneratePropertyMaps work, or
was of any help for EF7?
—
Reply to this email directly or view it on GitHub
https://github.com/TylerCarlson1/Automapper.Collection/issues/1#issuecomment-150339582
.


Improve clarity of unread indicator in favicon

If you haven’t already, check out our contributing guidelines for onboarding!

Platform - version:  Web <IP_ADDRESS>7
Action Performed (reproducible steps): Have an unread chat
Expected Result: The unread indicator on the favicon should be noticeable
Actual Result: It is very very easy to miss the unread indicator because it blurs into the logo
Notes/Photos/Videos:
Here's how the favicon currently appears:

Zoomed in:

We need to update it to this:

Logs - JS/Android/iOS (if applicable):
Here are the new assets that can be used to update the icon:
ECASH_unread.zip
https://github.com/Expensify/Expensify/issues/149295
https://www.upwork.com/jobs/~01ebf6f50340c191af

Hi @arielgreen,
I would like to work on this issue. Here is my proposal:
I found that CONFIG.FAVICON.UNREAD is used when there is an unread message.
https://github.com/Expensify/Expensify.cash/blob/eaba9026a62fcfe355591af578f41e533496af0e/src/libs/UnreadIndicatorUpdater/updateUnread/index.website.js#L11-L15
https://github.com/Expensify/Expensify.cash/blob/257518b71ebd9687e733bcc777e0f07a10fd61c3/src/CONFIG.js#L48-L51
We just need to change this file with new icon.
https://github.com/Expensify/Expensify.cash/blob/master/web/favicon-unread.png

Proposal
The main logic to update the favicon reside here
https://github.com/Expensify/Expensify.cash/blob/88a0c5e8f137b08c273aa7926e5326a621dd8eca/src/libs/UnreadIndicatorUpdater/updateUnread/index.website.js#L11-L15
Which takes the favicon url from the CONFIG Keys.
https://github.com/Expensify/Expensify.cash/blob/88a0c5e8f137b08c273aa7926e5326a621dd8eca/src/CONFIG.js#L48-L51.
For now, we have to replace the old asset web/favicon-unread.png with the new asset image.
There are few points which I like to point out here.

We should also add a apple-touch-icon for safari browsers lIke this.

<link rel="apple-touch-icon" sizes="180x180" href="/apple-touch-icon.png">

Thanks for the proposals! @tugbadogan please proceed with your PR.


Create new html site for Release notes

It would be better to have a site with release notes and link it from the app. The onboarding-menu of the app would be overloaded with all these information.

Good idea, but need to be separate for each App (Android, iOS and Web)

That's right. I would have forgotten it.

I'm still thinking about this. We have the changelogs for the AndroidApp also here:
https://github.com/RailwayStations/RSAndroidApp/tree/master/fastlane/metadata/android/en-US/changelogs. And there is a fastlane directory in the iOS App here: https://github.com/RailwayStations/Bahnhofsfotos/tree/develop/fastlane/metadata/en-US, but without the changelog files, only an empty releasenotes.txt. But they are plaintext only, not visual appealing.
Then we still have the challenge with supporting multiple languages, see #34.


[Bazel] using multable list as a default value in many of the rules

There are a number of bzl files with functions taking default values of []. Which could be a source of unexpected behaviour, as (like Python) the default value objects are created when the function is defined, not called.
Which means if the function is called and the default value is used, then the default value object can be mutated inside the function or any function that value is then passed to. The next time the function is called, the default value will be the mutated value and not the expected empty list [].
While this is a potential issue, it's not clear if the specific use case is present or not already. However I think it's worth fixing this up to avoid hitting this issue in the future.
See https://github.com/bazelbuild/starlark/blob/master/spec.md#functions

If a function parameter's default value is a mutable expression, modifications to the value during one call may be observed by subsequent calls. Beware of this when using lists or dicts as default values. If the function becomes frozen, its parameters' default values become frozen too.
# module a.sky
def f(x, list=[]):
  list.append(x)
  return list

f(4, [1,2,3])           # [1, 2, 3, 4]
f(1)                    # [1]
f(2)                    # [1, 2], not [2]!

# module b.sky
load("a.sky", "f")
f(3)                    # error: cannot append to frozen list

Although this could in theory be an issue, the fact that the list is frozen when one imports a function from a module should make this issue unlikely because it would be quite unusual to call a macro from within its own .bzl file at the root scope. Do you know of any instance in our codebase where that's the case?

Thinking about this a bit more, one way to fix the problem is to create a new bzl file somewhere:
# empty_list.bzl`
EMPTY_LIST = []

then use it as follows:
load("empty_list.bzl", "EMPTY_LIST")

def f(x, list=EMPTY_LIST):
  list.append(x)
  print(list)
  return list

# This will not work:
f(4, [1,2,3])           # [1, 2, 3, 4]
f(1)                    # [1]
f(2)                    # [1, 2], not [2]!

For this to work, it's very important that EMPTY_LIST be in its own bazel file. Also maybe it should be called FROZEN_EMPTY_LIST.

With Python the modules are often cached, so if you define a variable in a module it's likely going to be the same object you import everywhere. There are differences between importing the module and using the variable verses importing the variable from the module... i.e import x; x.y verses from x import y. Either way it's probably not behavior you want to be relying on. Again not sure how this translates to Starlark?
So if you did this in Python you would have likely made the situation worse not better.:
from empty_list import EMPTY_LIST

def f(x, list=EMPTY_LIST):
   list.append(x)

Here when the interpreter does the first pass it would execute the top level code in empty_list and create the EMPTY_LIST object. Likely cache the empty_list module so further imports in other modules would use the same module object... and the same EMPTY_LIST object. Then back in this module above, it would asigned the now project global object EMPTY_LIST object to the default value of list in the function object.
So any function defined with this would share the same default list object. Is there something about the way Starlark works that makes this not true?
In Python you would generally do:
def f( x, list_override: Optional[List] = None):
    list = list_override if list_override is not None else []

Something like this, where you set the default to some sentinel... the standard being None, as this also works with typing (from typing import Optional). Then if your function receives the sentinel then replace it with the default value.
Not sure if there is a more idiomatic way with Starlark?

There are some important differences between python and starlark. For example, my code in the previous comment is safe because in starlark, variables are automatically frozen when they are imported from another module. Unlike Python, you cannot simply import a module, you need to explicitly import elements from a module so I don't think there is a a starlark equivalent to import x; x.y (at least I am not aware of any).
I don't know if there really is an idiomatic way to do in starlark, the pythonic way works (except is is does not exists in stark, you use list_override != None). But unless we have a linter for that, there is a strong possibility that we will miss some instances in the future. We already have a bazel linter (buildifier) but I have no idea how hard it would be to add such a lint to it.


Switch from error-chain to failure crate

Closes #373
Failure
I implemented the custom fail type: https://boats.gitlab.io/failure/custom-fail.html
This was pretty easy and felt like it removed more code than it added.

hey, you must have seen it already, your appveyor config is older than master. You should merge master back into your PR branch and see if it passes the tests
Your appveyor file has
cargo run --example simple_search

which fails with the message below
error: no example target named `simple_search`

because there is no more simple_search example
change to cargo build --examples like in current master appveyor file

have a look at my PR #380. Don't mean to take your work away from you, but it was easier to open a PR to upstream tantivy than to find your branch. Feel free to merge and close it yourself.
I only wanted to do it, because I am doing #235 and wanted to base my PR off your work

@petr-tik no worries at all, i'm sorry that I didn't see this sooner. i'll make sure this is squared away shortly.

@petr-tik i rebased on master (thank you for the heads up) and it looks like all is well now. sorry for the hassle

@drusellers glad I helped.
@fulmicoton feel free to close #380, if/when you merge this

Maybe we should add an alias from tantivy::Error to tantivy::TantivyError and mark it as deprecated to make it the transition smoother.

thanks for all this work!

Thanks for all this work @drusellers . We may want to revisit it some day to add backtrace in some strategic part, but it is probably too soon to tell whre it is needed.
Merging!


Fetching and POSTING Data to and from API

In this branch I did the following things :

Setup the project with Webpack, linters, and Gitflow.
Fetching and POSTING Data to and from API using ASYNC and AWAIT.
Added the styles to make it very attractive.

Thanks a lot.
On Thu, Jun 16, 2022, 4:22 PM Omar Muhammad @.***>
wrote:

@.**** approved this pull request.
Approved ✔ Hi @ABDULALI3468 https://github.com/ABDULALI3468,
Great job, Your project is complete! There is nothing else to say other
than... it's time to merge it [image: :shipit:]
Congratulations! 🎉 ⚠Please consider the Optional suggestions below⚠
https://user-images.githubusercontent.com/65852621/133696182-6c5b784b-0083-427a-b24d-3ec870915f79.jpg
(Highlights) Good Points: 👍

You documented your work professionally. ✔
Github flow is used correctly. ✔
All linters checks are ok. ✔

(Changes Required) Aspects to improve: ♻

N/A

[Optional] suggestions:

Great job with API-consuming modules getData.js and test.js but
professionally the best way to do it is to make all API-consuming functions
inside one JS module and you can name it apiUtils.js or apiHelper.js.

Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
**Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear, and please remember to tag me in your question so I can
receive the notification**
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have
a limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
—
Reply to this email directly, view it on GitHub
https://github.com/ABDULALI3468/Leader-board-project-using-API-s/pull/2#pullrequestreview-1008891839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWO3YBZA522PXUN3WIYCTPTVPMFAXANCNFSM5Y6NHQSA
.
You are receiving this because you were mentioned.Message ID:
<ABDULALI3468/Leader-board-project-using-API-s/pull/2/review/1008891839@
github.com>

Thanks mate.
On Thu, Jun 16, 2022, 4:19 PM Deepak @.***> wrote:

@.**** approved this pull request.
Hi <STUDENT_MENTION>,
Status: Approved ✔
Your project is complete! There is nothing else to say other than... it's
time to merge it 🚢
Congratulations! 🎉
To Highlight 💯

No linter error ✔
Website looks professional 👍

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
You can also consider:

N/A

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
—
Reply to this email directly, view it on GitHub
https://github.com/ABDULALI3468/Leader-board-project-using-API-s/pull/2#pullrequestreview-1008888632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWO3YB3M4GORDAJGJMOCS2DVPMEU5ANCNFSM5Y6NHQSA
.
You are receiving this because you authored the thread.Message ID:
<ABDULALI3468/Leader-board-project-using-API-s/pull/2/review/1008888632@
github.com>


Add restricted status info to name cli queries and events

Description
Adds restricted status info to name lookup queries and bind/unbound events.
Note: Status not added to reverse lookup because this would be a backwards incompatible change to the protobuf structures.
closes: #806

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

the buf breaking check is an odd one ... that file it is unhappy about shouldn't have that message in it because it is being added in a separate PR that has not merged yet...

the buf breaking check is an odd one ... that file it is unhappy about shouldn't have that message in it because it is being added in a separate PR that has not merged yet...

Yeah. I'm not sure what's up with that.
Error: Previously present message "MsgGrantAllowance" was deleted from file.
Error: Previously present message "MsgGrantAllowanceResponse" was deleted from file.
Error: Previously present RPC "GrantAllowance" on service "Msg" was deleted.

All three of those are in both main and your branch, and your branch doesn't alter them at all.
The file in question is third_party/proto/cosmos/feegrant/v1beta1/tx.proto. There's only one commit for that file: e36b1b30c9c41db33bb5e100b27cf5e85e14abc9 from Feb 9, 2022.
I'm going to re-run that job to see if it just messed up.

LGTM


Try to listen the horizontal scroll view

Hi, @evrencoskun first congrats for your libs.
Can you help with a problem?
I need to listen when the scrollView is running or stopped, The listen of VerticalRecyclerViewListener it is works I get to take after the emitter, but HorizontalRecyclerViewListener I dont get to take.
I used your sample too. Maybe the HorizontalListener isnt emitter the rest of the events.
      adapter.tableView.cellRecyclerView.addOnScrollListener(object : RecyclerView.OnScrollListener(){
            override fun onScrollStateChanged(recyclerView: RecyclerView, newState: Int) {
                super.onScrollStateChanged(recyclerView, newState)
                println(">>>>>>>>>>>>>>")
            }
        })

I got to take the scrollView inside from the my adapter in the onCreateCellViewHolder I took the parent that is CellRowRecyclewView
but It can be a new improve


[UI] Registry tree should be full screen height

Current Behavior
In Settings, under Registry...

Desired Behavior
Please make the registry section the full height of the window.

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@leecalcote Can I get more insights on this? It wold be helpful.

@leecalcote Hey!, I am not exactly able to recreate this issue locally. Can you give me a few more details on how to re-create this? My UI looks fine locally when I try to view Settings/Registry. Also Which view is it? Desktop or Mobile? I will attach a picture of what I'm seeing locally when I try to recreate this.

@Dananjay996 similar on my localhost also

Desktop view. Try larger resolution size. Try Safari. Try zooming out if you don't have a screen with higher resolution. If you don't or you can't reproduce, please unassigned yourself from the issue.

@vaibhavpnimkar assigned to you

ok alright!!

@Yashsharma1911 fixed right?

@sudhanshutech no, it's not fixed.


error while getting user data. Reponse 401. Try again after 30s

all sessions have this error, the sessions are fine

Everyone using scripts, cex banned them.. happy farming

Join my tg channel for more info


[Feature] fix evaluate

Codecov Report

Merging #548 into 2.0-dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           2.0-dev     #548   +/-   ##
========================================
  Coverage    93.14%   93.14%
========================================
  Files           59       59
  Lines         2043     2043
========================================
  Hits          1903     1903
  Misses         140      140

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 e7e2057...9411721. Read the comment docs.


Use image overrides when deploying

Use image overrides, based on $PRELOAD_IMAGES, for determining which
exact images to override.
This will allow us to test just the images that we want to test, and use
the default images elsewhere.
A dev can still specify specific images, e.g.
make deploy PRELOAD_IMAGES='submariner-gateway submariner-operator'
Resolves: #915
Signed-off-by: Mike Kolesnik<EMAIL_ADDRESS>

This seems to have broken the System Tests workflow in the Operator repo.
To prevent such issues in the future, maybe we should add those tests to the Consuming Projects tests here (maybe under the e2e-projects label).

This seems to have broken the System Tests workflow in the Operator repo.
To prevent such issues in the future, maybe we should add those tests to the Consuming Projects tests here (maybe under the e2e-projects label).
Deployment (globalnet)
[14:21:34.055] [dir=submariner-operator; cl=cluster1; fn=validate_equals]$ jq -r -M .spec.repository /tmp/submariner.cluster1.json
[cluster1] Expected 'localhost:5000' but got 'quay.io/submariner'

Yes that might be wise, anyhow I'll fix the tests

Fix for operator system tests - https://github.com/submariner-io/submariner-operator/pull/2222


Adding update time of order book in _depth_cache

Coverage decreased (-0.1%) to 38.589% when pulling c5b2dd92704444d3daf717f5b3d5130d5d9a6770 on mishaker:add_event_time into 78a64a67cad02ca6397b0b95486b2bc830ae5ef0 on sammchardy:master.


Livetime from data

Alternative implementation to PR #562

hdf.py: When loading an HDF5 file, automatically attach any found attrs to the returned object as obj.attrs; remove extra return_attrs argument altogether, so only one value is returned. (NOTE: will need to modify one file in fridge that used that "feature")
events_pi.py / events.py: updates metadata via whatever metadata gets loaded from the hdf file (or any metadata present in an object passed in for initializing the arrays)
distribution_maker.py:

add set_livetime_from_data (=True by default) init kwarg. If True:

look for metadata in each stage in each pipeline; if any stage's metadata has "livetime":

make sure all stages that have this metadata have the same value
interpret as seconds (see TODO's)

loop through all pipelines and set params.livetime accordingly for any pipeline that has this param; also, set params.livetime to be fixed

TODO

should we only use metadata from a loaded file, and never pre-define any metadata fields (this would make code more generically applicable and do some future-proofing)
modify file in fridge that used pisa.utils.hdf.from_hdf(return_attrs=True)
livetime metadata with units? e.g., add a "livetime_units" field, so we can default to seconds if it's not found, or use those units if it is found?

@bourdeet @ts4051 I haven't tested out the full functionality here yet, only tested that the metadata item "livetime" gets loaded from a data hdf5 file. Hoping @bourdeet can do testing on the actual setting of livetime param in MC pipeline since presumably you are set up to run this test more quickly than I.

I just pushed the changes I think will be necessary and the bugs I found in the above example. Feel free to start from there if something still is buggy.

Closing this PR as it has been included in PR #562

I guess we can close this PR now, right? PR #562 has already included this code

Yes, this PR is subsumed by PR #562