Consider adding a way to see raw anon scores

One option would be a way to see how many of each star level you got. Similar to what amazon does.
I. E.
4 :star:  :star:  :star:  :star:  :star:
8 :star:  :star:  :star:  :star:
1 :star:  :star:  :star:
2 :star:  :star:
1 :star:
Twitter thread
https://twitter.com/jharler_dev/status/1130966000839680000?s=19

@jharler_dev
#ldjam results came in.  Mine don't seem to line up with feedback.  I noticed a lot more ratings than comments (2x).  @mikekasprzak would it be possible to see an anon list of ratings you got?  I'm interesting in seeing if there were any all 1-star ratings mean to lower scores.

@mikekasprzak
This is unfortunately the norm. Many first-time users don't realize there is actual benefit to leaving feedback. Also scores aren't a simple average.

@jharler_dev
Consider this a formal feature request for an anonymous list of ratings your entry received. I would be very interested in seeing how individual ratings look.

It would be great to see the overview of votes as suggested, but it would also be great to see a list of all the votes you received to see how people voted as a whole (anonymous of course).  This would specifically allow you to isolate ratings that were possibly intended to lower your score (all 1-star votes).

see #991 for the amazon like view


API for answering queries

A simple API to send queries and get results back would be a nice feature for automation - e.g., to automatically check results.

Introduced with a59cfbbc34c5f106d89fff75ce856b857e638236


Weird bug with the evenodd property on Text

Hello,
The default property of fill-rule is evenodd. Which should be ok.
But when I convert a text in path using the model.Text I got something really strange :

I don't want the intersection of the letter to be white. I want them to be the same color.
I tried with many different font. Same problem.
Using the fill-rule="nonzero" make the problem even worst.
Any idea how I can fix that ?
Thanks

Set the "combine" parameter to true in makerjs,models.Text(font, text, fontSize, combine, centerCharacterOrigin, bezierAccuracy)
http://microsoft.github.io/maker.js/docs/api/classes/makerjs.models.text.html#constructor
Here is an example, try checking / unchecking the "combine" checkbox:
http://microsoft.github.io/maker.js/playground/?script=Text#params=["montserratsubrayada-bold","Hello",72,true,false]

Thanks. Indeed. combine property solved the problem.
I have another weird behavior though. For most of the cursive/script fonts :
With cursive/script font i have some weird behaviors. See image below :
text : cde => GOOD

text : acde => NOT GOOD.

text : aac => NOT GOOD

First I thought It was a opentype.js problem. But I display the text using only opentype.js and it work. So it's probably in makerjs.
I think the problem came from the way the glyph are transform in model  with font.forEachGlyph(text, 0, 0, fontSize, opentypeOptions, cb);
Any idea ? else I would just rewrite another function using getPath method from Opentype. And then import the path in the model. It's not the idea of Makerjs but It should fixe the bug and convert it faster
var pathObj = font.getPath(word, 0, 0, 40);
var pathData = pathObj.toPathData(1);

Thanks for your help

This may be due to a bug in bezier.js not calculating arcs correctly, can you provide the JSON for just the "a" character?

Ok. Not all the times, but sometimes in the console I have this message "arc abstraction somehow failed..."
I attached 2 json :

one that convert only the character 'a' and this one is good (a_good.json)
one that convert "aac" and the letters are not displayed correctly. (a_not_good.json)

Archive.zip

The bezier.js library has been updated just now, can you try again? You may need to git pull.

I pull the whole project maker.js. But same problem. I've seen makerjs use bezier-js 2.1.0.
Is there a compatibility reason to not move to bezier-js 2.2.x ?

Oh, yes you are right, the node version should also be published. I updated the browser version :|

I updated bezier-js to 2.2.3. Still the problem. I've seen there is bug for more simple font as well (mosk.ttf) . For example, the middle of the "o" will be fill out.

If I generate the text outside makerjs and import it in makerjs the problem will also occur. Is there a way to import a brute raw path without convert into a makerjs model ? (letting the positioning problem aside)

Yes I am also seeing some other bezier errors. Seems worse than before. Perhaps you can downgrade for the moment.

There was a pull request that may have fixed the problem: https://github.com/deanmartinez/bezierjs/blob/4cbbc9223d5159d8796557163032aaea14bf602b/lib/bezier.js
Maybe you can try to put this into your node_modules ?

Nice it fixed both bugs 👍

Awesome! I will wait for the lib to stabilize before updating.

okay. Should I close the issue or leave it open until the update ?

Let's leave it open in case someone else has an issue, they will see it here. Thanks!

Actually I have still the problem. Just less frequently. Which is still good though.

ok try and get specific repro steps if possible.

Ok I do that asap. When i import a path with the method makerjs.importer.fromSVGPathData() the intersection between each path are not filled up. Is there a way to modify this behaviour the same way than with the argument combine of Text ?

Doing it that way is going to be tricky, because you will not know which paths make up a letter. You will be wanting to combineUnion each letter, but you will also be not wanting to combineUnion the "hole" in the middle of a letter like o. You can try to findChains but since your letters are already overlapping, there is a small chance that you may get a "figure 8" kind of chain if they have paths that share a point.

If you are willing to have a very small outline surrounding your shape, you may want to look at doing something like model.expandPaths(yourModel, .002) and then doing a model.findLoops and removing loops by their order of containment.

OK. I made a small repo to test easily opentype and makerjs text :

git clone https://github.com/microSoftware/display-font.git
npm install
node makerjs.js (or nodemon makerjs.js if you want to watch for modif)
modify the words to see which one work and don't work
Go on http://localhost:8900 to see the result

Note 1 : makerjs used is the fork I made. But I almost didn't change anything. And it's the last version of makerjs
Note 2 : when you will run npm install, you need to modify the bezierjs/lib/bezierjs with  https://github.com/deanmartinez/bezierjs/blob/4cbbc9223d5159d8796557163032aaea14bf602b/lib/bezier.js because the modifs will have disapeared.
Tell me if you have any issues testing. You can test before and after modifying bezierjs. It's not the same words that don't work. And changing the font as well.

@danmarshall Thanks for your answer. I think it's going to be a bit slow to use findLoop(). My dream solution would be to use opentype.js font.getPath. Because it's freaking fast. But I just can't figure out how to get the value of x and y based on the font and text.

Yes, boolean combination is not a fast operation ;)

I thought it was going to loop through every point of the whole path. Which would be long with a cursive font. But I'll give it a try.It's the best way to see what the result and performance looks like.

Hi @microSoftware thanks for your repo - I was able to reproduce your scenario. I have some more testing to do, but here is a preliminary patch of isBetweenPoints you may like to try:
        function isBetweenPoints(pointInQuestion, line, exclusive) {
            var oneDimension = false;
            for (var i = 2; i--;) {
                if (MakerJs.round(line.origin[i] - line.end[i], .000001) == 0) {
                    if (oneDimension)
                        return false;
                    oneDimension = true;
                    continue;
                }
                var origin_value = MakerJs.round(line.origin[i]);
                var end_value = MakerJs.round(line.end[i]);
                if (!isBetween(MakerJs.round(pointInQuestion[i]), origin_value, end_value, exclusive))
                    return false;
            }
            return true;
        }
        measure.isBetweenPoints = isBetweenPoints;

I test with node.js and got an error :
/Applications/MAMP/htdocs/display-font/node_modules/bezier-js/lib/utils.js:540
      arc.s = s;
            ^

TypeError: Cannot create property 's' on boolean 'false'
    at Object.getccenter (/Applications/MAMP/htdocs/display-font/node_modules/bezier-js/lib/utils.js:540:13)
    at Bezier._iterate (/Applications/MAMP/htdocs/display-font/node_modules/bezier-js/lib/bezier.js:797:23)
    at Bezier.arcs (/Applications/MAMP/htdocs/display-font/node_modules/bezier-js/lib/bezier.js:758:19)
    at getArcs (/Applications/MAMP/htdocs/display-font/node_modules/makerjs/index.js:6845:26)
    at new BezierCurve (/Applications/MAMP/htdocs/display-font/node_modules/makerjs/index.js:7016:32)
    at /Applications/MAMP/htdocs/display-font/node_modules/makerjs/index.js:7003:64
    at Array.forEach (native)
    at new BezierCurve (/Applications/MAMP/htdocs/display-font/node_modules/makerjs/index.js:7002:36)
    at /Applications/MAMP/htdocs/display-font/node_modules/makerjs/index.js:8077:42
    at Array.map (native)

I modified bezier.js with the code you gave me last time. Should I put the original code back ?

That happens when a Bezier is actually a straight line. Can you provide a repro?

git clone https://github.com/microSoftware/display-font.git
cd display-font
npm install
node .

The error occurred only with some words. For example : "okitlaks" with the font "galaxie_cassiopeia_bold"

Thanks, was able to repro. Will work on a fix.

Ok I believe this is fixed - can you update to 0.9.49 and verify?

Nice job !!! Thank you 😀  😀  😀


ProPresenter timers don't update frequently enough

ProPresenter timers seemingly only update on a stage display every 1-3 seconds, even though they update immediately in the clocks panel of ProPresenter immediately. Will likely move to fully native ProPresenter clocks out of necessity.

Possibly blocking issue. While we can easily start \ stop \ set clocks in ProPresenter, ProPresenter provides no functionality to jump a clock to a specific time, which will cause issues when seeking or jumping to specific times on the timeline.

ProPresenter clocks don't support jumping to time when they are running, thus it's not really possible to use native clocks. The subsequent issue that was discovered was that the stage display requires the timer to be 'reset' for it to display on the stage display, but this does not happen immediately after an API change, but requires about a ~1sec delay, otherwise the clock will be reset to the prior time, not the new time.


Fluent validation issue

In case of an error, the services return an error code. But this is not a flexible and comfortable decision. It was decided to replace it with fluent validation. Create your own issue, specific to each service in the project and change the old error codes with fluent validation (example you can find in EventService).  Fluent validation should check only the main problems. For example, id cannot be null and should be more than zero. The fluent validator shouldn't make any calls to the database - if an entity doesn't exist the repository should just throw NotFoundException and then ExceptionHandleMiddleware must catch it. Also, add unit tests to your pull request to check if you didn't break some important validation logic.

Already done in https://github.com/ita-social-projects/WhatBackend/pull/507, https://github.com/ita-social-projects/WhatBackend/pull/519, https://github.com/ita-social-projects/WhatBackend/pull/528, https://github.com/ita-social-projects/WhatBackend/pull/529, https://github.com/ita-social-projects/WhatBackend/pull/530, https://github.com/ita-social-projects/WhatBackend/pull/532, https://github.com/ita-social-projects/WhatBackend/pull/535.


Link on install IPFS page which leads to https://docs.ipfs.io/introduction/install/ missing

Do you want to request a feature or report a bug?
report a bug
What is the current behavior?
leads to a page with this line:
ipfs resolve -r /ipns/docs.ipfs.io/introduction/install/: no link named "install" under QmebR8zvqzuUFKKLGAjccXaJRmSkkiwYEjaUcoknpaHsX3
**If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal From the README page on github tried to enter IPFS install explaination and got that error.
What is the expected behavior?
See installations instructions.

Install docker and run IPFS as container
$ docker run -d --name ipfs_host -e IPFS_PROFILE=server -p 4001:4001 -p <IP_ADDRESS>:8080:8080 -p <IP_ADDRESS>:5001:5001 ipfs/go-ipfs:latest

Fixed


change_torrent

client.change_torrent(torrent.id, location=new_path)
hi, does this move the save location of the torrent, i tried and it failed,I don't know how to solve it, looking forward to your reply, thank you
Environment
tr version:3.0
RPC:16

try https://transmission-rpc.readthedocs.io/en/v3.3.0/client.html#transmission_rpc.client.Client.move_torrent_data

Thank you, maybe my needs are special, because I replaced a new machine, so I want to restore all the state before tr, I have backed up the resume and torrent files, but since the save address has changed, I have to change the address.Now it's even worse. When I restart transmission, some torrent addresses have become the default download addresses. I may have to update the save address of each torrent by modifying the resume file, because the original address is / A/B, the new address needs to be changed to /C/B, do you know which way to open the resume file to meet my needs, thanks


Inquiry Regarding Batch Size and GPU Memory Usage on A800 vs. A100 GPUs

Dear CogAct Team,
Thank you so much for open-sourcing your code and providing detailed documentation.
I’ve been trying to replicate your experimental setup as described in your paper and code. According to your reported settings, you were able to train with a batch size of 32 using an 80GB A100 GPU. However, when I attempt to run the same code and model on my 80GB A800 GPU, I encounter out-of-memory issues even when I significantly reduce the batch size to 2.

We use Fully Sharded Data Parallel (FSDP) to distribute the model's parameters, gradients, and optimizer states across multiple GPUs. Therefore, the code must be run on a server equipped with multiple GPUs. If the server has 8 GPUs with 80GB of memory each, you can set the batch size to 32. With fewer GPUs on the server, the batch size should be reduced accordingly. Note that it may not be possible to run the code with a single GPU.

thank you for your reply, now I close the comment


Kubernetes logs should work for any pod

The current implementation of the kubernetes logs command will attempt to use the current working directory's name as a label selector when fetching pods to view logs from. This can fail if there is no pod that shares the same label as the vscode directory name with the following strack trace:
TypeError: Cannot read property 'metadata' of null
extension.js:492
	at selectPodForApp (/Users/bhargav/code/vscode-kubernetes-tools/out/src/extension.js:708:21)
	at findPodsForApp (/Users/bhargav/code/vscode-kubernetes-tools/out/src/extension.js:715:13)
	at kubectl.invoke (/Users/bhargav/code/vscode-kubernetes-tools/out/src/extension.js:489:13)
	at context.shell.exec.then (/Users/bhargav/code/vscode-kubernetes-tools/out/src/kubectl.js:77:72)
	at <anonymous>

Would be happy to take this on… :) I think I had something noticed this issue on the older extension as well.


Allow setting package name of ruby augeas bindings

Required when using a non-standard ruby runtime, for example SCL's on Redhat.

thanks @tdevelioglu


git checkout new_branch,the resource manager list content is delayed

In a git project, when the branch is switched, the resource manager list content is delayed and the refresh is sometimes invalid

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests so if you can open a new issue with the template filled out, that would be helpful for us.   In particular, detailed steps to reproduce the behavior you're seeing, expected and actual behavior, and knowing if you can reproduce in safe mode (atom --safe) would be very helpful. Also, if your issue is specific to the GitHub package, please open your issue in the package repository: https://github.com/atom/github/.
We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


feat(obstacle_avoidance_planner): not only yaw thresh but also dist thresh for find nearest index

Description
Use distance threshold for findNearestIndex in obstacle_avoidance_planner

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@takayuki5168 How Can I check this PR?

In my environment, I confirmed that the vehicle does not stop in the middle.
Before
https://user-images.githubusercontent.com/9547070/172114836-cdb1df08-4bd0-477d-816d-77a897168fd6.mp4
After
https://user-images.githubusercontent.com/9547070/172114855-b0758f20-7c0b-4d90-9a32-2aa644f6c7c7.mp4


the example in the readme might be broken

https://github.com/choojs/hyperx#hyperscript-node-example does not support the custom y attribute and does not render the same output.

@DanielRuf could you create a repro? In bel I've definitely created all sorts of custom attributes — I wonder what scenario is preventing you from using the example. Thanks!

Well, the y attribute isn't output at all in the pure Node version on Node 8 with npm 5.

Will provide an example in about 2 hours because I'm sure this specific demo did just return a h1 element without the y attribute (on Windows 10).

const h = require('hyperscript')
const hyperx = require('hyperx')
const hx = hyperx(h)

var title = 'world'
var wow = [1,2,3]
var tree = hx`

hello ${title}!

`
console.log(tree.outerHTML)

outputs

hello world!

It seems that custom tags and attributes are not fully supported.
My dependencies:
"hyperscript": "^2.0.2",
"hyperx": "^2.3.1",
"virtual-dom": "^2.1.1"

yeah looks like hyperscript doesn't support custom attributes like that in the browser. it does on the server, which is why the test z_hyperscript that runs that snippet does work.
bel does support custom attributes, maybe we could switch the example to use that. or patch hyperscript so it works both on the server and the browser.

@goto-bus-stop
This was the node (server) example on my Windows command line, not in the browser.

oh sorry. huh. could you check if npm test in this repo fail too then on windows? that uses the same snippet.

All 53 tests pass.
Daniel MINGW64 ~/.../component-library3
$ cat index.js
const h = require('hyperscript')
const hyperx = require('hyperx')
const hx = hyperx(h)

var title = 'world'
var wow = [1,2,3]
var tree = hx`

hello ${title}!

`
console.log(tree.outerHTML)
Daniel MINGW64 ~/.../component-library3
$ node index.js

hello world!

$ npm --version
5.3.0

$ node --version
v8.4.0


Fix .safeArea constraints, apply to Example code

Turns out .safeArea auto layout constraints didn't work. This is now fixed, which allows cleaning up Example layout code.
Also, there's no easy way to implement topLayoutGuide and bottomLayoutGuide support for old versions of iOS that don't have safeAreaLayoutGuide. It's much easier to just drop iOS 10 or older ¯_(ツ)_/¯

Codecov Report

Merging #42 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files           5        5
  Lines         336      336
=======================================
  Hits          320      320
  Misses         16       16

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 ee2af79...fd0a2b1. Read the comment docs.


[Defect] Writing or Drawing on Canvas without scrolling on phone and tablet interfaces

Fixed with touch based event inclusion

Fixed with latest code update.


@superset-ui/legacy-plugin-chart-table Typescript Support

Hi,
I was trying to export component in typescript but failing, not sure whether exporting correctly in the d.ts file.
try 1:
declare module "@superset-ui/legacy-plugin-chart-table" {
export { default as TableChartPlugin } from "./lib";
}
try 2:
declare module "@superset-ui/legacy-plugin-chart-table" {
export { default as TableChartPlugin } from "./lib/Table";
}
failed with error
Attempted import error: 'TableChartPlugin' is not exported from '@superset-ui/legacy-plugin-chart-table'.
Please suggest, as my tryouts failed.
Thanks,
Rathish

Closing import from esm folder worked
declare module<EMAIL_ADDRESS>{
export default TableChartPlugin;
}


Set proper GOPATH in Makefile

fixes #74
By executing go env GOPATH it makes it compatible with all the version of go.

Amazing. I'm going to test and merge it.


Send out an email when a new approval arrives

An email should be sent out to the person managing admin when a new approval is being created. So that it's easier to quickly act on new approvals.

That got implemented a while back so I'm closing this issue now.


having some trouble when linking to MySQL

Hi, I'm having some trouble when linking to MySQL. Does it only support connecting to sqlite? There seems to be no introduction to connecting to MySQL in the document

Maybe you need to install mysqlclient before using it?

`from langchain.agents.agent_toolkits import SQLDatabaseToolkit
from langchain.sql_database import SQLDatabase
from langchain.llms import OpenAI
from langchain.agents import AgentExecutor
from langchain.agents import create_sql_agent
import pymysql
db = SQLDatabase.from_uri("mysql+pymysql://:@/[?]"
)
toolkit = SQLDatabaseToolkit(db=db)
llm=OpenAI(temperature=0)
agent_executor = create_sql_agent(
llm=llm,
toolkit=toolkit,
verbose=True
)`
I am facing the following error, ignore the uri string
AttributeError                            Traceback (most recent call last)
Cell In[70], line 9
6 toolkit = SQLDatabaseToolkit(db=db)
7 llm=OpenAI(temperature=0)
----> 9 agent_executor = create_sql_agent(
10     llm=llm,
11     toolkit=toolkit,
12     verbose=True
13 )
14 def create_sql_agent(
15     llm: BaseLLM,
16     toolkit: SQLDatabaseToolkit,
(...)
26     **kwargs: Any,
27 ) -> AgentExecutor:
28     """Construct a sql agent from an LLM and tools."""
---> 29     tools = toolkit.get_tools()
30     prefix = prefix.format(dialect=toolkit.dialect, top_k=top_k)
31     prompt = ZeroShotAgent.create_prompt(
32         tools,
33         prefix=prefix,
(...)
...
94 Use this tool to double check if your query is correct before executing it.
95 Always use this tool before executing a query with query_sql_db!
96 """
AttributeError: type object 'QueryCheckerTool' has no attribute 'llm'

How can I use the password with some special characters like the one ends with a @ character
For example mysql+pymysql://<username>:password@@localhost/dbname


fix: Form field has padding even when no label is present

Description
This PR removes the top padding from the form field when there is no label present.

Related links, issue #, if available: AWSUI-43667
How has this been tested?
Tested manually in the dev pages by removing the label from a form field and inspecting the layout with the dev tools.

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The visual regression tests report a visual difference in the dropdown of the Date Range Picker. This difference is expected: the Date Range Picker uses an unlabelled FormField around a grid of four FormFields. The outer form field is only used to provide a constraint text for the grid, but since each inner FormField already has a label, the outer FormField should not have a label nor extra padding.


enhancement: add sse service skeleton

Description
So far, SSE has only been used to exchange content type messages between the server and the client.
In order to be able to send more content (and context) to the client, we have moved the endpoint to a separate service and are now also using it for other notifications like:

notify postprocessing state changes.
notify file locking and unlocking.
TBD

Related Issue

Requirement for: https://github.com/owncloud/web/issues/6682

Motivation and Context
First SSE POC was build as part of our christmas hacking and is used as a content messaging (no context) layer only at the moment, this is nice but does not show or use the whole potential of that endpoint.
To do so, and also make our clients more interactive (RIP F4), we start to use it for other notifications more and more.
For the moment we should not rely on SSE exclusively for business critical communications, that's why we're going on two tracks for the moment.
How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

[ ] Provide a README.md for that service in the root folder of that service.

Use CamelCase for section headers.

[ ] For images and example files used in README.md:

Create a folder named md-sources on the same level where README.md is located. Put all the images and example files referenced by README.md into this folder.
Use absolute references like https://raw.githubusercontent.com/owncloud/ocis/master/services/<service-name>/md-sources/file to make the content accessible for both README.md and owncloud.dev
bad
good

[ ] If new CLI command are introduced, that command must be described in readme.md.
[ ] If new global envvar is introduced, the name must start with OCIS_.
[ ] Add the service to the makefile in the ocis repo root.
[ ] Make the service startable for binary and individual startup:

For single binary add service to ocis/pkg/runtime
For individual startup add service to ocis/pkg/commands

[ ] Add the service to .drone.star to enable CI.
[ ] Inform doc team in an early stage to review the readme AND the environment variables created.

The description must reflect the behaviour AND usually has a positive code quality impact.

[ ] Create proper description strings for envvars - see other services for examples, especially when it comes to multiple values. This must include:

base description, set of available values, description of each value.

[ ] When suggestable commits are created for text changes and you agree, collect them to a batch and commit them. Do not forget to rebase locally to avoid overwriting the changes made.
[ ] If new envvars are introduced which serve the same purpose but in multiple services, an additional envvar must be added at the beginning of the list starting with OCIS_ (global envvar).
[ ] Ensure that a service has a debug port
[ ] If the new service introduces a new port:

the port must be added to port-ranges.md and to the readme.md file.

[ ] Make sure to have a function FullDefaultConfig() in pkg/config/defaults/defaultconfig.go of your service. It is needed to create the documentation.


Update issue system with new context name

Added new context and location.origin in view.

Linking to issue #22


Restructured Driver and DesiredCapability factory for Mobile & Web Te…

…st Session

Created WebDriverFactory and MobileDriverFactory for Web and Mobile
Removed DriverFactory and DesiredCapabilityFactory
Removed deprecated IPad and IPhone browser flavors and corresponding classes

@sundaramrajendran Can you get rid of BrowserFlavors.GENERIC too? Also, there are many changes here that increase the visibility of classes and/or methods to public. I'd like to avoid this. Visibility was at the package level before this change, by design.

@mach6 I have addressed review comments

this is merged. thank you!


Not responding to clicks when overlaying another UIView

My screen has 2 UIViews:
1.- Header when the DropDownMenu is placed
2.- Table (UITableView) where results are shown depending on DropDownMenu selection
Problem is that the DropDownMenu is not clickable in the part of the dropdown menu that overlays the UITable.
I've tried changing the layer's zPosition to 1,  this way the DropDownMenu is fully shown but still not clickable.

me, too


Write some tests and examples

There are simple tests now, having some meaningful examples would still be useful but probably requires higher-level libraries. An example around GIO streams could be added at a later time


Company: Typescript properties defined incorrectly

When setting the company of a user the TS definition defines an attributes field.
In reality the attributes should be inside of the company object one level deep not inside of this attributes field.
export interface CustomerlyCompanyProperty {
    company?: {
        company_id: string | number;
        name?: string;
        attributes?: any;
    };
}

This causes a type error when doing the custom attributes correctly.

And worse causes an error in the data structure when following the incorrect typescript.

Fix:
export interface CustomerlyCompanyProperty {
  company?: {
    company_id: string | number;
    name?: string;
    [attribute: string]: any;
  };
}


EasyCLA Contributor Console - Help and FAQ Link Updates

Summary
EasyCLA Contributor Console - Help and FAQ Link Updates
Background
The help and FAQ links are invalid. This task is to update those links.  See page reference below.  Correct links should be similar to the EasyCLA landing page (reference below):
<lfx-header product="LF CLA" id="lfx-header" userefreshtoken="true" [expanded]="hasExpanded" (toggled)="onToggled()"
supportlink="https://jira.linuxfoundation.org/plugins/servlet/theme/portal/4/create/143"
docslink="https://docs.linuxfoundation.org/lfx/easycla/v2-current"
faqlink="https://docs.linuxfoundation.org/lfx/easycla/v2-current/getting-started/easycla-faqs">
</lfx-header>

Acceptance Criteria

Help and FAQ links are valid within the contributor console.

References

Help links
EasyCLA Landing Page Links

Now The help and FAQ links are as expected & mentioned above , So making issue status QA-Done

Docs : https://docs.linuxfoundation.org/lfx/easycla/v2-current
Support : https://jira.linuxfoundation.org/plugins/servlet/desk/portal/4/create/143
FAQ : https://docs.linuxfoundation.org/lfx/easycla/v2-current/getting-started/easycla-faqs


Added global style variables from Figma design

@stanislav-ostr noticed that the font size for the H3 (58px => 40px) and H4 (22px => 24px) don't match


doboolab-welcome

why?

Why not?
This is nothing but to inform that this package belongs to dooboolab. This is what other great libraries are doing such as react-native-elements or firebase. I've just seperated package to dooboolab-welcome to reuse this in other module that's created by dooboolab's forks. If you've installed many modules that belongs to dooboolab it will just print once.
Thanks.

Thanks for the explanation. I recommend adding this explanation to README.md under FAQ. When reviewing libraries, it can be scary at times to see dependencies that seems a little bit out of place. Also, as the package is not open sourced (though you can review the source if you have install it), it make it a little suspicious. (Thus, I'm also recommending opening sourcing the small package)
I have no problem with the dependencies and respect the owner/maintainer decisions. Thanks for the hard work!

@lxcid Oh! I understand your concern now. Yeah, there was a serious attack with npm packages and sure many people will be curious about this. Also, I'll just make dooboolab-welcome public for now (I didn't know it wasn't).

@hyochan That's what I was trying to say. I was just having a bad day that day. Please excuse me for that. Also, last time I checked, the welcome package itself depends on chalk...

@Thomazella Yes. dooboolab-welcome uses chalk to color the letters. Is there any vulnerability inside?


Raspberry Pi Build

I managed to get docker cloud agent to build and run on a Raspberry Pi, I'm not sure what the plan is around different architecture support but maybe this pull request might help.
There is a prebuilt arm deb that works on raspbian jessie available in the releases section. I had to modify the Dockerfile and some of the bash scripts so merging into master as is would be a bad idea but hopefully it might be of some use.
Its very cool to be able to add such a small cheap system as a docker cloud node :)

@rukkus
It is interesting run Docker Cloud on Raspberry Pi. However, to make it work, I think we have to make other changes so that the agent downloads the ARM version of docker instead of the x64 one.
Also, we are going to change the node provisioning to use docker native tools, and this repo will be deprecated soon.
I guess the support of Raspberry Pi will not be supported in the agent.
Thank you

Does this mean Docker Cloud Agent (and BYON) will never officially work on any ARM 64-bit architecture, or just RPI?  I was really looking forward to using my odroid-c2 as BYON as a test sandbox and for light workloads at home.

@mrmachine
To support ARMs, the agent has to download a a ARM version of docker, and currently we only support downloading the x64 version of docker.
There could be a lot of code changes to support ARMs. As the agent is going to be deprecated soon, we are not going to support ARM nodes here. Instead, when we migrate to use docker native tools to provision the node, we will think about the ARM support.

@tifayuki When is docker cloud agent going away, and what is it's replacement? If it's going to be a long time (I haven't seen it announced that Docker Cloud Agent is deprecated?), and then more time again until ARM is supported, it might be worth taking a closer look at this PR which claims to have Docker Cloud Agent built and running on ARM?

@mrmachine
There is no ETA yet, it is on our roadmap.
Even though I merge this PR, the agent will not work unless we changed the server end logic, which is not going to happen, AFAIK.
Moreover, this PR simply changes amd64 to arm, which will break the build of the agent for x86-64 architecture.

This will be supported when we support 1.12+ swarm clusters without the use of this agent


odo doesn't properly re-sync all files if a new deployment is created on 'odo push'

/kind bug
/area devfile
I'm seeing an issue on master with odo push when we re-sync the files of a project after a component restart (such as from changing the env in the devfile). The push will succeed and it will say the source code synced fine, but if I exec into the container (or look at its logs), I'll see that only the devfile (and whatever files were changed locally or created as part of the build), were synced, and the rest of the source code was not synced
Files synced on first odo push (full logs: first-push.txt):
I0826 21:22:20.287844   92492 adapter.go:211] Copying files /Users/johncollier/ododev/brokesync/nodejs/.gitignore
/Users/johncollier/ododev/brokesync/nodejs/LICENSE /Users/johncollier/ododev/brokesync/nodejs/README.md
/Users/johncollier/ododev/brokesync/nodejs/devfile.yaml /Users/johncollier/ododev/brokesync/nodejs/package-lock.json
/Users/johncollier/ododev/brokesync/nodejs/package.json /Users/johncollier/ododev/brokesync/nodejs/server.js
/Users/johncollier/ododev/brokesync/nodejs/test /Users/johncollier/ododev/brokesync/nodejs/test/test.js to pod

Files synced on odo push with redeploy (full logs: push-redeploy.txt):
I0826 21:29:25.688476   93487 adapter.go:211] Copying files /Users/johncollier/ododev/brokesync/nodejs/devfile.yaml
to pod

If I use the -f flag on odo push, sync is still broken. Additionally, the issue can be seen if you do odo delete, followed by odo push again.
It seems we're not properly detecting the scenarios where we need to bypass the odo-file-index.json file and do a full sync (such as component redeploy, or force push)

Marking this as high priority since sync is pretty much broken if a container needs to be recreated, and that this is a regression from v1.2.5 (which properly detected when/when not to do a full sync)
/priority high

Closing as a dupe of https://github.com/openshift/odo/issues/3838


Chore: DB Drizzle Schema

initial schema.
some functionalities are missing on the drizzle side .. future work arounds are needed

Dont forget to pull from main and then

resolve any merge conflict other than the lockfile
run pnpm i
run git add --update
run git commit
close the opened file
push ur changes


make ID and caveatId []byte instead of string

make ID and caveatId []byte instead of string

@edsrzf please check. pretty minor change.
P.S. Planning to use it this way: https://gist.github.com/thousandsofthem/8f4a5611b1bba8390ca8


add dual res test

Description
We need to add a test for the dual res var.
I thought PR #605 was ready to go, but it turns out it breaks the dual res var in soca-science because hocn is required for the remapping, but hocn isn't allowed to be part of increment any more with that PR.

nevermind, I don't know what i'm talking about

2 free points! Nice! Gotta do more of those.


Postgres tuning for high request volume

As per conversation with @usingtechnology :
It appears that Postgres might not be able to handle swarms of connections by default: the issue was uncovered while attempting to create several tenants programatically. When hitting 20 concurrent requests (each tenant will create its db schema), Postgres throws the following error:
sorry, too many clients already

The issue was overcome by using a custom max_conns.sql with the following contents:
ALTER SYSTEM SET max_connections = 500;

In docker, the mount path is as outlined in the following snippet (might be different in k8s/OpenShift):
wallet-db:
    image: postgres:15
    environment:
      - POSTGRES_USER=DB_USER
      - POSTGRES_PASSWORD=DB_PASSWORD
    ports:
      - 5433:5432
    volumes:
      - wallet-db-data:/var/lib/pgsql/data
      - ./max_conns.sql:/docker-entrypoint-initdb.d/max_conns.sql
    networks:
      - app-network

Acceptance Criteria:

Add custom max_conns.sql configuration file/settings to the Postgres deployment used as dependency by Traction.

Should we worry about this with the Mediator @WadeBarnes ?
Also — are there ways to monitor logs and trigger a notification when a magic string like sorry, too many clients already is seen?

The better way to set the max connections on a Postgres server is to use the Postgres configuration files.  We do this with the BC Registries databases.  It was done specifically for the wallet (secure storage) in that case.  Example here https://github.com/bcgov/von-bc-registries-agent-configurations/blob/main/openshift/templates/db/config/postgresql-cfg/dbconnections.conf.  It might be an idea to adopt the configuration as standard for any wallet databases (secure storage).
I'll have to look into how we can be alerted to the issue.

@i5okie so you are aware of this - the configuration file could be applied to all our postgres deployments in theory

waiting for HELM chart to get done and we can then test it


ARROW-4518 [JS] add jsdelivr to package.json

Observable is now using jsdelivr and so it would be good to add this and make a release.

Codecov Report

Merging #3597 into master will increase coverage by 2.97%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3597      +/-   ##
==========================================
+ Coverage   87.74%   90.71%   +2.97%
==========================================
  Files         667       71     -596
  Lines       82536     4935   -77601
  Branches     1069     1069
==========================================
- Hits        72422     4477   -67945
+ Misses      10003      452    -9551
+ Partials      111        6     -105

Impacted Files
Coverage Δ

js/src/schema.ts
84.61% <0%> (-1.93%)
:arrow_down:

js/src/ipc/metadata/message.ts
90.12% <0%> (-1.28%)
:arrow_down:

python/pyarrow/ipc.pxi

cpp/src/arrow/csv/chunker-test.cc

cpp/src/parquet/column_page.h

cpp/src/parquet/bloom_filter-test.cc

cpp/src/arrow/array/builder_decimal.cc

cpp/src/plasma/client.cc

cpp/src/arrow/io/test-common.h

cpp/src/arrow/util/int-util-test.cc

... and 588 more

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 2b9155a...cb6ccc7. Read the comment docs.

@trxcllnt can you review?


Messeges are not sent

Messeges are not sent.
Error 404 in GET DraftNotificationByID function.

Any idea?

After adding GroupMember.ReadAll permission to AzureAD authors app the issue seems to be resolved but I'm not sure it's best practice.

@OritOrlanczyk glad that issue resolved, we'll close the issue.


Sample: Send user location

Bot Info

SDK Platform: .NET Core
SDK Version: Core
Active Channels: WebChat
Deployment Environment: Azure Bot Service, Azure App Service, Bot Emulator

Issue Description
I need to get information about the user's location, (longitude, latitude, browser data, etc.) either through an API call or by receiving a new message or conversationUpdate.
How can I do it?
Thanks!
Note to fixer
There are 2 approaches for this sample:

[ ] Add a button on the page to send current location
[ ] Add a custom attachment that will request the user for lat/long, for once

The custom attachment should have two buttons, one is yes, another is no
When "yes" button is clicked, we will send the lat/long (browser may popup a request dialog)

If the request dialog is denied, we will notify the bot, same as if "no" button is clicked

When "no" button is clicked, we will notify the bot about the denial

Can you help me? please

@serpino, please see the response to you question on the botbuilder repo located here.

Related to #1476.

I was able to get the latitude and longitude in Bot Framework v4 C# using the React.js web channel using DirectLine API payload and receive the data in the bot in turnContext.Activity,Values.


draw:canceled event not included in Leaflet.Draw.Event.js

I've skipped the template because I'm not reporting a bug, exactly, just a (hopefully) helpful tip. I needed specific behavior when a user canceled drawing via clicking the "Escape" key, which triggers the "draw:canceled" event. I expected to find that event listed among the events in https://github.com/Leaflet/Leaflet.draw/blob/develop/src/Leaflet.Draw.Event.js, however it wasn't included there. That module is really helpful, and I used a few other events there for other purposes (such as L.Draw.Event.DRAWSTOP, but there is no L.Draw.Event.DRAWCANCELED... so if you need to listen for the canceled event, try this:
map.mapObject.on("draw:canceled", (event) => {
        console.log(event);
});

I'd open a PR to add that event to the https://github.com/Leaflet/Leaflet.draw/blob/develop/src/Leaflet.Draw.Event.js, but it appears that this repo is no longer being maintained.

Team, Do we have any plan to add canceled event part of draw lib itself?.
I came across this issue today and provided a fix in the app to handle it.

I also need to handle draw:canceled. Could we add this, please?


Serve websockets only

Leave the static files to a separate CDN. This clarifies the fact that the websocket relay is true independent from the front-end.

Closed by #8  and reimplemented in https://github.com/danielfone/websocket-relay-ruby


When entering Japanese with an editor and breaking the line, the last typed characters are duplicated

VSCode Version:
1.8.0-insider (1.8.0-insider) 9b06d4392aeac3af2e9da58838a518bba325d23a
2016-11-18T08:57:27.676Z
code-insiders --disable-extensions

OS Version: macOS Sierra 10.12.1（16B2657）

When entering Japanese with an editor and breaking the line, the last typed characters are duplicated.
Steps to Reproduce:

Enter Japanese characters in the Editor.
make a line break.

I have same issue

#16133

@rebornix @alexandrudima fyi the recent Electron update backported a fix for this issue from Chrome, so I am not sure our workarounds are still needed. The related fix was https://github.com/electron/libchromiumcontent/pull/293


Config object missing

The Budget object https://lunchmoney.dev/#budget-object references a config object but this object is never listed in the documentation aside from being shown in an example.

Resolved in https://github.com/lunch-money/developers/pull/38


Collect all lines for mesos node logs

https://jira.mesosphere.com/browse/DCOS-57008
The collected logs would be very minimal. Only the last second. As you can see in the linked artifacts, they are all significantly larger (closer to 1 MB)
proof it works:
Open 1.10: https://teamcity.mesosphere.io/viewLog.html?buildId=2252983&buildTypeId=DcOs_Open_Test_IntegrationTest_TestLogFetcherAwsOnpremWStaticBackendGroup1&tab=artifacts
Open master: https://teamcity.mesosphere.io/viewLog.html?buildId=2252985&buildTypeId=DcOs_Open_Test_IntegrationTest_TestLogFetcherAwsOnpremWStaticBackendGroup1&tab=artifacts
EE 1.10: https://teamcity.mesosphere.io/viewLog.html?buildId=2252987&buildTypeId=DcOs_Enterprise_Test_Inte_TestLogCollectionAwsOnpremWStaticBackendAndSecurityStr&tab=artifacts
EE master : https://teamcity.mesosphere.io/viewLog.html?buildId=2252989&buildTypeId=DcOs_Enterprise_Test_Inte_TestLogCollectionAwsOnpremWStaticBackendAndSecurityStr&tab=artifacts

@mesosphere-mergebot label ready for review

@mesosphere-mergebot changelog-not-required log script change


Fix failing example specs

Fixes #1910

They still appear to be skipped:

They still appear to be skipped

Yes, these tests have been listed in tests/slow.haskell, hence are skipped.
I've confirmed locally that erc20-spec.md and storage-spec.md are passing. Tests erc721-spec.md didn't fail in the ~25min window I let it running.
I'll remove erc20-spec.md and storage-spec.md from the exclude list.


use-tabler-icons:0.2.0

I am submitting

[ ] a new package
[x] an update for a package

Description: Update tabler-icons to v3.20.0 with 18 new icons (zyf722/typst-tabler-icons#4).

Thanks!


Mobile Notification Button causes ArrayIndexOutOfBoundsException

Summary
Pressing on the button Mobile inside Settings/Notifications causes the app to crash
Environment Information

Device Name: Nexus 5x
OS Version: Android 8.0.0
Mattermost App Version: 1.4.0 (Build 63)
Mattermost Server Version: 4.3.0 (Build 4.3.0)

Steps to reproduce
Click on Settings -> Notifications -> Mobile
Expected behavior
Button click should cause another settings view.
Observed behavior
App crashes with  java.lang.ArrayIndexOutOfBoundsException: Index 0 out of bounds: node has no children

I am having the same issue in
App Version: 1.4.1 (Build 64)
Server Version: 4.2.0 (4.2.1)

Same as #1109 and #1184, I hope the team can investigate on this issue quickly as we are more and more to have this problem :)

closed via #1233


Add optional hideLink prop for offline picker use

This pr adds a prop to allow hiding unusable links for when the language picker is being used in an offline app (a la https://github.com/sillsdev/TheCombine/issues/2232#issuecomment-1701861064).

do we want to call it "offline" since we talked of eventually having it query the end points if the user is online and not query them if the user is offline?

That's forward thinking. I'll update the pr.


Node connect hotline call (text)

Kansas contacted the hotline 5/12/21 @ 3:30pm
They had just installed sensors 90 mins ago and saw 3/4 nodes sent data to the dashboard twice, at 3:00 and 2:30, but one only sent data at 2:30.  Wanted to make sure everything was good before driving 2 hrs back.  I said that sometimes that happens and data will be sent later, but to be safe I suggested they reset everything, walked them through the pattern of gateway then nodes.
After resetting, the one node didn't refresh on the dashboard so I walked through checking the node connection using the serial monitor on tablet.  They reported all nodes synched to gateway and asked if they should wait for the data to show up on dashboard.  I said it should be safe to drive home now without waiting.

That same node at that site has again quit sending data. Will drive there as soon as I can to see what I can figure out. At that same site ( yrv) I am seeing some weird data from one sensor, reading 999 over and over. I'll check it out when I am there.

Deann's node 18000294 is still not connecting/ appearing in the tech dashboard (devices status) even though when I walked her through synching it using the serial monitor last week it showed that it had connected on the serial monitor after conducting a node scan.
Do we have any other troubleshooting advice for this besides: reset, tighten antennae, wait and see?
(trying to tag Esleyther, but name isn't appearing when @)

5pm on 5/18/21
Peter from Kansas went back to the site of 18000294 and found a broken solar panel connection in the node.  He was able to repair it and reset everything.  He said his tech dashboard hadn't updated since 12:05. I waited for the 5:05 update to come in on my computer and 18000294 did not come up in the data string.  I asked him to check the lights on the node, neither were on.  It sounds like this node needs to be replaced.
Esleyther, @elhenriq is contacting Peter now


Response object type Null type replacement raises ApiTypeError

In situations where the API returns a response with Null type and null value for empty string fields (not sure about other types), the python client will raise an ApiTypeError unless the discard_unknown_keys config property is set to True.
I first noticed this when pulling transactions from the mxbank checking account member for transactions that have an empty merchant_guid.
I'm not sure whether the OpenApi spec can account for this during code gen.  If not,  it might be helpful to update the config object in the readme to include discard_unknown_keys = True with a comment as to why it's being set.
Thanks!

Hi @peter-stratton
Yeah that is something the OpenAPI spec can account for during the codegen process. I've updated our spec and generated version 0.5.1 of the library that resolves this issue.
Thanks again for reaching out and helping improve this library 👍


Testing firestore rules against emulator fails with reason: Unexpected token < in JSON at position 0

Hi, I am trying to setup firestore rules unit tests.
Since I am using typescript and jest for testing, I have edited your provided example a bit. This is what I have currently:
import { readFileSync } from 'fs-extra'
import * as firebase from '@firebase/rules-unit-testing'
import { getDoc, setDoc, setLogLevel } from 'firebase/firestore'

describe(' Firestore Rules', () => {
    let testEnv: firebase.RulesTestEnvironment;

    beforeAll(async () => {
        testEnv = await firebase.initializeTestEnvironment({
            firestore: { rules: readFileSync('../firestore.rules', 'utf8') },
            projectId: 'podsuite',
            hub: { host: 'localhost', port: 4000 },
        });
    });

    afterAll(async () => {
        await testEnv.cleanup();
    });

    beforeEach(async () => {
        await testEnv.clearFirestore();
    });

    test('noone should have access to user document', async function () {
        // Setup: Create documents in DB for testing (bypassing Security Rules).
        await testEnv.withSecurityRulesDisabled(async (context) => {
            const testdoc = context.firestore().collection('users').doc('user1234');
            await setDoc(testdoc, { test: 'test1' });
        });

        const unauthedDb = testEnv.unauthenticatedContext().firestore();

        // Then test security rules by trying to read it using the client SDK.
        const testdoc = unauthedDb.collection('users').doc('user1234');

        await firebase.assertFails(getDoc(testdoc));

    });
});

My emulator runs on http://localhost:4000/
These are my rules:
rules_version = '2';
service cloud.firestore {
  match /databases/{database}/documents {
    match /users/{userId} {
       allow read, write: if false;
    }

    match /users/{userId}/salesItems/{documentId} {
      allow read: if true;
      allow write: if false;
    }

    match /users/{userId}/overviews/{documentId} {
      allow read: if true;
      allow write: if false;
    }
  }
}

When I run npm test I am getting an error :
 FAIL  src/tests/rules.test.ts
  ●  Firestore Rules › noone should have access to user document

    FetchError: invalid json response body at http://localhost:4000/emulators reason: Unexpected token < in JSON at position 0

      at node_modules/@firebase/rules-unit-testing/node_modules/node-fetch/lib/index.js:272:32

What is wrong?

I received the same error message while writing my test suite.
When initializing the test environment you set the hub port to 4000. This is the UI port. You have to pass the port of the emulator hub (by default 4400).
Changing your code to the following should allow the tests to connect to the emulators:
 testEnv = await firebase.initializeTestEnvironment({
            firestore: { rules: readFileSync('../firestore.rules', 'utf8') },
            projectId: 'podsuite',
            hub: { host: 'localhost', port: 4400 },
        });

@andipaetzold Thanks for taking the time to comment. I will try that ASAP


Long loading time on new "To-Do" dashboard for regions with a lot of content

Describe the Bug

The dashboard of regions with a lot of pages/events/pois are loading very slow
Steps to Reproduce

Go to https://cms-test.integreat-app.de/muenchen/
Wait for 8 seconds
See dashboard

Expected Behavior

The dashboard should load considerably faster
Actual Behavior

The dashboard is very slow, so presumably the queries that gather all the needed stats are not efficient enough.
Additional Information
This is also linked to #2516 – how can we reproduce this problem on local machines without having to clone the production database which contains a lot of personal data?

Apparently, this is not a problem of the SQL queries (they only take 13ms), but of "domLoading" instead:

The same problem occurs on the linkcheck analytics page:
https://cms-test.integreat-app.de/muenchen/analytics/linkcheck/invalid/ (20s loading time)
https://admin.integreat-app.de/muenchen/analytics/linkcheck/invalid/ (10s loading time
So I think it's definitely related to the linkcheck row in the todo dashboard.


Triggering without a click

Howdy! LOVE this plugin - saved my bacon on a redesign to Bootstrap.
I need to be able to trigger the lightbox based on an AJAX response (instead of a click event). I tried $.ekkoLightbox() which didn't work, and $(document).ekkoLightbox() which worked the first time (but then subsequent normal click events didn't work).
Is there a best practice for handling this situation? I need to be able to trigger lightboxes from AJAX AND still trigger them from normal anchor clicks, more than once, on a given page.
Thanks!

Also, and this may be part of my problem above:
If I do $("#selector").ekkoLightbox(), the lightbox appears on page load.
If I do $("#selector").click(function() { $(this).ekkoLightbox()}), it works the first time I click, but not for subsequent clicks.
I need to be able to programmatically trigger it, but I'm obviously missing something.

Never mind! The page I was loading loaded conflicting javascript (didn't realize it wasn't in an iframe). I believe I'm good now. Thanks!


probe: advbench

i mean it's ancient history, may as well get it in there, you know?
https://github.com/thunlp/Advbench

dupe of #280


Add Initial API setup

This imports all necessary libraries to create the API

The code looks good to me - how many endpoints were we targeting again? Should we create a class to store each of the endpoints?

@pthomas76 as of rn, we have 8 endpoints. I don't think it is good practice to create a separate class.

Looks good to me, let's see how we progress with the end-points.


Source Control Repositories -> Close repository - always show again

I close a repository by clicking Close repository and after a while, as soon as I navigate to an open file from this repository, the repository appears again even though I closed it a while ago.

I'm too getting the same error......what to do?

Closing as duplicate of #107398.


2.0.1

2.0.1 is the source yet no actual installation of the 2.0.1 build of the engine

just press enter to update, and if you need the build itself, you can clone the repo here -> https://github.com/YoshiCrafter29/YC29Engine-Latest


ethers v5 support

I see in the docs that ethers v4 is supported in ethereum-waffle 2.x.
With ethers v5 ready to use for new projects, I thought it would be good to kick off this conversation here.

I didn't see a stable release yet:
https://www.npmjs.com/package/ethers
Do you know when it is planned for?

@marekkirejczyk Any thoughts on what timeframe would make sense for this based on what @ricmoo had to say? Thanks!

CC @sz-piotr

I think we should put high pio on it. There is Websocket support coming finally.

The biggest blocker is I don't know how to simultaneously support ethers 4 and 5.

Maybe the simplest strategy is questioning the assumption that supporting them both at once is needed. What use case are you imagining where a user might want to choose between the two?
Or do you mean from a development perspective? I think if all the associated changes will involve lots of PRs, you could make a new ethers-v5 branch off of develop now and start opening PRs against that, and wait until it’s farther along to take develop at that later time into an ethers-v4 branch and merge ethers-v5 into develop.

Yeah, I guess we should just make it a breaking change and release Waffle 3.0 along with new ethers support

Yeah, I guess we should just make it a breaking change and release Waffle 3.0 along with new ethers support

@marekkirejczyk The buidler team just had to make similar decision with ethers v5, and their changes just shipped. Their choice about how to handle this might be useful to look at?

Didn't look at that yet, but we want to ship Waffle 3.0 in May.

hey guys, I actually migrated Buidler to ethers v5 to unblock me from my work 😅
But now Im realizing that I need Waffle to migrate too. How can I help with this? I'm happy to do the same migration
I'm confused however about this monorepo? This doesn't use Lerna, is this using Yarn workspaces?

Nevermind I just realized you can just bootstrap the whole monorepo running yarn which is pretty awesome 👌 I wish they would support hoisting

Hoping to get v5 ethers out in May too. The only hold up is the new documentation, which is coming along and should be ready soon(tm).

Hey @ricmoo, what would be the best way to reach you? Keybase, Telegram, Discord?

Email prolly works the best<EMAIL_ADDRESS>:)
I do have the others, but I don’t pay much attention to them. Of those, telegram (@RicMoo) is prolly the least-worst.

Alright, I've submitted a PR where it can it succeeds to build and linting but still failling some test regarding solc which I'm not sure if it's related to my machine.
Could you review #237? @marekkirejczyk

Ha Ha, nice job!
Will try to review, this weekend.
@sz-piotr Perhaps we push it to a branch and release as beta?

I’d love that 👌 it would be unblock it in my side

After taking a deeper look, there are no integration tests failing, all the tests that are failing are from the chai helpers. I'd love your help @sz-piotr to debug it

Any updates on this? @marekkirejczyk @sz-piotr

After taking a deeper look, there are no integration tests failing, all the tests that are failing are from the @ethereum-waffle/chai matchers. I'd love your help @sz-piotr to debug it

Hi @pedrouid . Same for me. I just upgrade our buidler repo to
    "@nomiclabs/buidler-ethers": "2.0.0-alpha.0",
    "@nomiclabs/buidler-waffle": "2.0.0-alpha.0",

in order to play with "ethers": "5.0.0-beta.184".
The only tests that seem to be failing are ones that use matchers like to.emit(contract, "myEvent").
I assume this is because of the way ethers v5 handles events on the contract interface. They are now only ordered by their signature and split into their own fragments.
May I ask, did you already find a way to get this matcher to work with ethers v5 ?

It will be merged and published soon on Buidler under the beta tag
https://github.com/nomiclabs/buidler/pull/546


Responsive header

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Updating the ARM64 CallFunction according to the new CallInfo.Count behavior

Updating the ARM64 CallFunction according to the new CallInfo.Count behavior
arm64_callFunction should take the actual argCount as a parameter and not depend on the Info.Count.

    extern Var arm_CallFunction(JavascriptFunction* function, CallInfo info, Var* values, JavascriptMethod entryPoint);

this doesn't require a change?

Refers to: lib/Runtime/Library/JavascriptFunction.cpp:1311 in c8d0f55. [](commit_id = c8d0f55f81c04b739cd4b309650f99d97e2c9de0, deletion_comment = False)

I will work on the ARM change later


⚠ Bofrost.at has degraded performance

In 13feda5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1577 ms

Resolved: Bofrost.at performance has improved in 7ae91b3.


SOPS: Add support for HashiCorp Vault token-based authentication

fixes #516
If implemented, the kustomize controller will be able to retrieve a secret containing a VAULT TOKEN and use it to decrypt the sops encrypted master key. Which will then be used to decrypt the data key that will decrypt the final data.
The kustomize-controller retrieve the secret specified in the kustomization in the same namespace. We keep the behaviour set by age and pgp decryption.
In case a token does not exist for a HashiCorp vault encrypted token, we fall back to the default server to try decrypting/encrypting the data. This will preserve the behaviour for customer relying on VAULT_TOKEN env var.
Signed-off-by: Soule BA<EMAIL_ADDRESS>

@hiddeco I have checked the Azure Vault PR. We should probably make the func NewServer(prompt bool, homeDir, vaultToken string, agePrivateKeys []string) keyservice.KeyServiceServer accept options, if we are to keep adding parameters.


feat(filters): add areas to available filters

Areas are now available filter-options

going with current implementation of Users-filter, this is hard-coded as well
relies on Area service to provide the list of all areas for dropdown

Example:

Closes #1147

Nice! Just a note - Filter By Areas should not be a selectable item. See http://www.patternfly.org/pattern-library/forms-and-controls/filter/#/design for example.

Filter on the list does not work

@debloper told, he will add another patch with working filtered list, hence approving it.

@michaelkleinhenz @debloper @sanbornsen - was my comment on the 'Filter by Areas' as a selection addressed? I received no response on that comment.


fix: :bug: Llama Index imports and track costs and token counts in the class

This pull request fixes the Llama Index imports in the code and adds functionality to track costs and token counts in the TokenCostHandler class. The TokenCostHandler class now includes variables to store the prompt cost, completion cost, prompt tokens, and completion tokens. The calculate_prompt_cost and calculate_completion_cost functions have been updated to return the cost and number of tokens. Additionally, the reset_counts method has been added to reset the counts (e.g. at the end of each query of an agent, similarly to what is done with Llama Index's original token counting callback).

Thanks for raising this! I’m not able to run and test for another day or so, but can you please also run the unit tests to account for the num tokens change? I believe these are breaking changes

Thanks for raising this! I’m not able to run and test for another day or so, but can you please also run the unit tests to account for the num tokens change? I believe these are breaking changes

I've made some extra changes so that the unit tests pass again ✅

Thanks for raising this! I’m not able to run and test for another day or so, but can you please also run the unit tests to account for the num tokens change? I believe these are breaking changes

I've made some extra changes so that the unit tests pass again ✅

Thanks for this! Overall, I think the LlamaIndex stuff should work, but I'd prefer not to merge the new way of returning token counts + cost in the single function. We already have token counting functionality as separate functions, so there's no need to overload what we already have.
The main issue is that these changes will break every other users' installations of tokencost because the cost estimators will return tuples instead of decimals.
One possible alternative is creating a function called "calculate_everything" (or something like that) that returns a dict/tuple of all the information.

Thanks for raising this! I’m not able to run and test for another day or so, but can you please also run the unit tests to account for the num tokens change? I believe these are breaking changes

I've made some extra changes so that the unit tests pass again ✅

Thanks for this! Overall, I think the LlamaIndex stuff should work, but I'd prefer not to merge the new way of returning token counts + cost in the single function. We already have token counting functionality as separate functions, so there's no need to overload what we already have.
The main issue is that these changes will break every other users' installations of tokencost because the cost estimators will return tuples instead of decimals.
One possible alternative is creating a function called "calculate_everything" (or something like that) that returns a dict/tuple of all the information.

My latest commit reverts back the breaking changes and implements a calculate_all_costs_and_tokens that is used in the Llama Index callback.
Let me know if there are still any issues to resolve.

Chanmpion. Thank you so much @AndreCNF !


Works fine with the whole model but raise "NANs" on selected layers.

When I was trying to compare the same model trained on different datasets, I encountered a weird problem:
It works fine when I compare all layers:
cka = CKA(model1, model2, device='cuda', model1_name='model1', model2_name='model2')
But, when I try to compare a selected subset of layers:
cka = CKA(model1, model2, device='cuda', model1_name='model1', model2_name='model2',           model1_layers=list(model1.state_dict().keys())[:5],            model2_layers=list(model2.state_dict().keys())[:5])
It raises:

HSIC computation resulted in NANs

Do you have any idea how to fix this? Thank you very much.

In my case, the NaNs issue was raised when there was no hook created (here), in which case no feature would return here, resulting in the divided-by-zero error at L180.  It might be helpful to see if the hooks are being generated properly.

I think an issue I am having is no matter what, in line 134 ((N - 1) * (N - 2)) ends up equalling 0 because the matrix being passes is always a size 2x2 from around line 181 K = X @ X.t().
I discovered this happens when the batch size is <= 2. So if anyone else has this issue this might be why!
I set this batch size because this method is so slow and memory consuming. Are there any tricks to it without using large batch sizes or computation?

I got the same issue: https://github.com/AntixK/PyTorch-Model-Compare/issues/10. Looking for solutions. Thanks!


Creating Dungeons in a loop creates the same dungeon?

func DunTest() {
	x := 15
	for i := 0; i < 10; i++ {
		d := NewDungeon(64, x + 10)
		d.Print()
	}
}

I'm trying to create a number of dungeons and store them in a slice to make going up and down stairs easier for myself. However this function returns 10 identical dungeons, even when adding 10 to the number of room attempts. Why is this?

https://github.com/Meshiest/go-dungeon/blob/master/dungeon/dungeon.go#L87
Dungeon Generation seeds are based on Unix time. You're likely creating them within the same second.
You can generate dungeons without NewDungeon if you're interested in changing the seed!

Gotcha! Thanks for clearing that up for me.


Fix & improve "Setting Up" section to match code and fix buildsystem fails

Thanks for sharing your guide, Qt for UI and Rust for business logic is exactly what I currently want to explore a little.
Here some proposals to fix things I stumpled over in the "Setting up" section.

Sigh... commit "No need to explicitely call show() on a child widget " slipped accidentally in, was not yet meant for you, sorry, removing now. (been some time I did stuff on github, please pardon my mess)

I'm pretty happy with these changes. Like I've mentioned before, I was jumping around a bit when writing the guide because I'd find I missed something and would have to go back, so the exact build instructions probably weren't correct.
It looks like having a fresh set of eyes follow through from start to finish helps with picking up these sorts of issues :slightly_smiling_face:

I was jumping around a bit when writing the guide because I'd find I missed something and would have to go back, so the exact build instructions probably weren't correct.

Understood. Found the text and tutorial flow itself pretty good and correct, good work there from what I can tell. So had a good experience otherwise and left the existing chapters without big questionmarks but new knowledge instead, thanks again for providing the tutorial :)


Login via Twitter breaks site

When I login using Twitter I get this on the main page:

and a completely blank profile page:

LinkedIn may also have problems, but I am not able to test that as I don't have a LinkedIn account.

This was using Firefox in Private Browsing Mode on Mac.

Removing social login for now


DO Catalog link in new dataset [ch83967]

Related to https://app.clubhouse.io/cartoteam/story/83967/add-data-connect-do-link

The idea was to let this feature be enabled or disabled globally on the whole site by a config flag. What could be the best place in the config file to set it?

Oh, I see, I'm not sure if we need this kind of configuration, I would expect a () => true in the enable attribute. However, the idea to follow the same pattern could be interesting.
@jgoizueta could you review the configuration part here? Does it make sense to have a dataobservatory: enabled: true section in app_config file?


AssertionError: Path does not exist: /media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/data/VOCdevkit2007/VOC2007/JPEGImages/filename

/media/user1/storage-1/V4_DATASET/new_annotation/train_person.txt data loaded
/media/user1/storage-1/V4_DATASET/new_annotation/test_person.txt data loaded
/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/data/VOCdevkit2007/VOC2007 data path
Loaded dataset voc_2007_train for training
Set proposal method: gt
Appending horizontally-flipped training examples...
Traceback (most recent call last):
File "trainval_net.py", line 203, in
imdb, roidb, ratio_list, ratio_index = combined_roidb(args.imdb_name)
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/roi_data_layer/roidb.py", line 116, in combined_roidb
roidbs = [get_roidb(s) for s in imdb_names.split('+')]
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/roi_data_layer/roidb.py", line 116, in
roidbs = [get_roidb(s) for s in imdb_names.split('+')]
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/roi_data_layer/roidb.py", line 113, in get_roidb
roidb = get_training_roidb(imdb)
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/roi_data_layer/roidb.py", line 97, in get_training_roidb
imdb.append_flipped_images()
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/datasets/imdb.py", line 117, in append_flipped_images
widths = self._get_widths()
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/datasets/imdb.py", line 113, in _get_widths
for i in range(self.num_images)]
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/datasets/imdb.py", line 113, in
for i in range(self.num_images)]
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/datasets/own_dataset.py", line 86, in image_path_at
return self.image_path_from_index(self._image_index[i])
File "/media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/lib/datasets/own_dataset.py", line 102, in image_path_from_index
'Path does not exist: {}'.format(image_path)
AssertionError: Path does not exist: /media/user1/storage-1/Devendra/faster_rcnn_2_jwyang/faster-rcnn.pytorch/data/VOCdevkit2007/VOC2007/JPEGImages/filename
please help me to solve this error, I am try to training with my own dataset .

That is the path error , i solved that error by adding my own file path , if your using pascal_voc dataset then please check dataset path and if it is not there then arrange based upon the error.

That is the path error , i solved that error by adding my own file path , if your using pascal_voc dataset then please check dataset path and if it is not there then arrange based upon the error.

I solved that too, but cant remember the detail, like found myself creating soft link in the wrong path I suppose, seemingly the same way

I thought soft link was not require and on which data you kept the training ? custom or pascal_voc

I thought soft link was not require and on which data you kept the training ? custom or pascal_voc

pascal_voc, and I do remember when I encountered with this, I followed the tuturial in https://github.com/rbgirshick/py-faster-rcnn#beyond-the-demo-installation-for-training-and-testing-models and tried create the soft link  VOCdevkit2007 under /VOCdevkit, but failed to include the whole path so I created that in the wrong place, anyway I completed the path and this error no longer comes out again

I need a one help , when i am kept the training with large data then my system was restarted. why its happening and its works for less data.
please help me to find the solution

I need a one help , when i am kept the training with large data then my system was restarted. why its happening and its works for less data.
please help me to find the solution

I ran the whole project on a cloud server, trained with a large dataset of over 10k images while also a much smaller one containing only 270 which is made by my team, and did not encounter with such issue, so I may not able to be of help. I found some of my fellows met the same problem however, one of them checked his hardware and solve it
issue: https://blog.csdn.net/wuzuyu365/article/details/52212544
summary of related: https://blog.csdn.net/u011622208/article/details/90579252
hopefully it would help

Thank you , the blogs are suggested to increase the power supply to GPU. ok i will try and if any other solution please drop here.

File "/media/user1/storage-1/Devendra/TF_model/tf-faster-rcnn/tools/../lib/datasets/voc_eval.py", line 251, in voc_eval
if not R['det'][jmax]:
IndexError: list index out of range
please help ,Thanking you!!!


Rename service and instance labels when origin metrics are contained.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

Could you be clear what is the change? This seems a breaking change for existing cases.

MAL engine doesn't care what labels are from where.
Such as Prometheus fetcher could add target as labels in default. OpenCensus receiver and Envoy metrics service handler, also LAL generated metrics don't have the idea of sources.
Please be clear about your cases.


Document the behavior and expectations of the various caches

Currently, it's not very clear what the caches do and where they're written to/read from.

Also relevant for #2137

I am removing myself as an assignee in this issue, but happy to help if somebody takes a lead on this issue!

@konnov Honestly, I doubt that anyone but you can do this within reasonable time.

Yeah, but then it will not happen in a reasonable time frame. I just want to reflect reality here :)

As per our discussion last week, @Kukovec or @thpani are to update this issue with more specifics on what we need documented, and then we will get Igor to help fill in these identified gaps.

I think a good starting point, as mentioned by @Kukovec in the meeting, is to document

what each cache is used for
when each cache SHOULD be called
when each cache MUST be called / what items can be expected to be in-cache
when each cache is popped/cleared

Adding to the above, caches will also be an issue in follow-ups to #2547, since arena/SMT splitting would require us to reevaluate the caches, as the cached values are cells (arena), but the cache constraints are in SMT.


tab selection in URL

Right now when I hit Refresh I go back to the project root. I would prefer if it stayed on the tab (Files, Runs, Processing tools, etc.). I would also prefer if I were able to share a link to that specific tab. Would it be possible to include the selected tab in the URL?

This is done.


Error with transformers version

Hi @yangheng95 ,
The transformers version required is kept greater than 4.5 '4.5<transformers<5.0', whereas the latest version of transformers itself is
4.17 only.
Because of this we're getting the following error with ** pip install -U pyabsa**

I encounter the same issue when using python setup.py install

I encounter the same issue when using python setup.py install

@AaronHeee If you are installing from source, you can edit the line 4.5<transformers<5.0 in setup.py to 4.0<transformers<5.0 .Alterntively you can pip install transformers.

Working well with 1.8.33 version.


multifile: make protocol-conformance-member pass on Windows

PE/COFF does not include a symbol table in the generated executable
binary.  Instead, use public to expose the getter, and then use
-coff-exports from llvm-readobj to get the symbols that are
exported.  Fortunately, the same tool can be used to list the symbol
table contents for ELF and MachO binary.  This allows us to share the
test across all the targets.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


Fix markdown lint issues in SETTING_BASH.md

This commit fixes markdown lint issues in SETTING_BASH.md file under /troubleshooting as per issue #47

There are multiple changes in this PR. Please remove the README.md changes. You may want to open a new branch and pull in just the one commit and submit a new PR or just roll back one commit with git reset --hard HEAD^


version 3.0.1

Branch for the 3.0.1 release of mui-datatables. It contains a number of bug fixes:

Bug fix for header checkbox when selectToolbarPlacement = 'none' or 'above' (https://github.com/gregnb/mui-datatables/pull/1321)
Fix bug with close icon not working in View Columns popover (https://github.com/gregnb/mui-datatables/pull/1325)

This list may grow.
This release will most likely happen either Sunday night (June 14th) or Monday night (June 15th).

Coverage increased (+0.03%) to 75.146% when pulling c6c48e7161581feb48185325de336ccd343f8d9a on v3_0_1 into 8158e238c311caadd0ff54a102132a8cd5e78aac on master.


chat interface

Does this allow followup questions which takes the context and history of the former questions into account? Thanks! tweet context

Interesting, cause that's basically the question I asked in #113. Embedchain should definitely be able to handle history, but if it's supposed to keep a self-contained history, we have to worry about sessions and invalidation, so I think it should be part of the frontend.

Working on this. Raising a WIP PR.

Released in v.0.0.15. Closing this.


Profile - Switch to Expensify Classic is under Account instead of General

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.52-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/37421
Action Performed:

NewDot: Login to Account (Web/mWeb/Desktop)
Click on 'Profile' at the bottom (user's avatar)

Expected Result:
'Switch to Expensify Classic' option is under General section
Actual Result:
'Switch to Expensify Classic' option is under Account section
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub

@zanyrenney  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1
CC @trjExpensify

This is intentional now as part of HybridApp plans, so it's easier for people to find how to switch back. Going to close this out. CC: @AndrewGable @JmillsExpensify for vis


The function of partition_num

Thanks for your excellent work!
I have a question about the effect of the parameter partition_num,please tell me if you have time.
As we know that in impletation of PER with propotional  the partition_num has to be the same as the batch size,and what about the effect of partition_num in rank_based impletaion?

And how about building distributions every time experience sample is needed?

because of the use of the parameter Partion_num,the programe will unavoidable forget some experience because record_size must no bigger than partition size

I get it a little bit,so close it anyway.


chore(release): Add a postbump lifecycle method to update the version…

… numbers in the .openshiftio/application.yaml during the release process

Coverage remained the same at 96.154% when pulling 9f90884bd10ce1c2fe54fb40e7092e4c0e4b68bd on lholmquist:release-thing into 89b5ace2e98f5d85a669010cdb511a80c20d9e44 on bucharest-gold:master.


FlutterError.onError is not called when framework throws an error

flutter doctor:
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.2, on Mac OS X 10.15.5 19F101 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.54.3)
[✓] Connected device (3 available)

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

void main() {
  FlutterError.onError = (FlutterErrorDetails details) {
    print('~~~~~~~ FlutterError catch: $details');
  };
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'FlutterError.onError not called',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: Text('FlutterError.onError not called'),
        ),
        body: Row(
          children: [
            Text('A' * 1000),
          ],
        ));
  }
}

Screenshot

Question

FlutterError.onError not called when flutter framework error

I switch the Flutter version to 1.22.0, FlutterError.onError works!
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 1.22.0, on Mac OS X 10.15.5 19F101, locale zh-Hans-CN)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.54.3)
[✓] Connected device (2 available)

• No issues found!

So in Flutter 2.0.2 version, there is a problem with FlutterError.onError.

I was able to reproduce this issue. I can confirm that in Flutter 1.22.0 the FlutterError.onError was being called in debug but in Flutter 2 it is not called.

logs (Flutter 1.22.0)
MacBook-Pro-de-MacBook:flutter_example pedromassango$ flutterm run
Using hardware rendering with device sdk gphone x86 64. If you notice graphics artifacts,
consider enabling software rendering with "--enable-software-rendering".
Launching lib/main.dart on sdk gphone x86 64 in debug mode...
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        44.5s
✓ Built build/app/outputs/flutter-apk/app-debug.apk.
Installing build/app/outputs/flutter-apk/app.apk...                 1.7s
I/flutter_exampl( 5826): Not late-enabling -Xcheck:jni (already on)
I/flutter_exampl( 5826): Unquickening 12 vdex files!
W/flutter_exampl( 5826): Unexpected CPU variant for X86 using defaults: x86_64
W/flutter_exampl( 5826): Accessing hidden method Landroid/view/accessibility/AccessibilityNodeInfo;->getSourceNodeId()J (greylist,test-api, reflection, allowed)
W/flutter_exampl( 5826): Accessing hidden method Landroid/view/accessibility/AccessibilityRecord;->getSourceNodeId()J (greylist, reflection, allowed)
W/flutter_exampl( 5826): Accessing hidden field Landroid/view/accessibility/AccessibilityNodeInfo;->mChildNodeIds:Landroid/util/LongArray; (greylist, reflection, allowed)
W/flutter_exampl( 5826): Accessing hidden method Landroid/util/LongArray;->get(I)J (greylist, reflection, allowed)
Waiting for sdk gphone x86 64 to report its views...                15ms
I/OpenGLRenderer( 5826): Davey! duration=1289ms; Flags=1, IntendedVsync=440941553762, Vsync=441074887090, OldestInputEvent=9223372036854775807, NewestInputEvent=0, HandleInputStart=441086711472, AnimationStart=441086715733, PerformTraversalsStart=441086718538, DrawStart=442114499297, SyncQueued=442117729054, SyncStart=442119292708, IssueDrawCommandsStart=442119678871, SwapBuffers=442166883865, FrameCompleted=442232890228, DequeueBufferDuration=38157672, QueueBufferDuration=1739182, GpuCompleted=129415406168625,
I/Choreographer( 5826): Skipped 68 frames!  The application may be doing too much work on its main thread.
Syncing files to device sdk gphone x86 64...                     1,984ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h Repeat this help message.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).
An Observatory debugger and profiler on sdk gphone x86 64 is available at:
http://<IP_ADDRESS>:55859/H5xl2EyuBuA=/
I/flutter ( 5826): ~~~~~~~ FlutterError catch: ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞══════════════════════
I/flutter ( 5826): The following assertion was thrown during layout:
I/flutter ( 5826): A RenderFlex overflowed by 8740 pixels on the right.
I/flutter ( 5826):
I/flutter ( 5826): The relevant error-causing widget was:
I/flutter ( 5826):   Row
I/flutter ( 5826):   file:///Users/pedromassango/dev/Projects/flutter_example/lib/main.dart:27:15
I/flutter ( 5826):
I/flutter ( 5826): The overflowing RenderFlex has an orientation of Axis.horizontal.
I/flutter ( 5826): The edge of the RenderFlex that is overflowing has been marked in
I/flutter ( 5826): the rendering with a yellow and black striped pattern. This is
I/flutter ( 5826): usually caused by the contents being too big for the RenderFlex.
I/flutter ( 5826): Consider applying a flex factor (e.g. using an Expanded widget)
I/flutter ( 5826): to force the children of the RenderFlex to fit within the
I/flutter ( 5826): available space instead of being sized to their natural size.
I/flutter ( 5826): This is considered an error condition because it indicates that
I/flutter ( 5826): there is content that cannot be seen. If the content is
I/flutter ( 5826): legitimately bigger than the available space, consider clipping
I/flutter ( 5826): it with a ClipRect w

flutter doctor -v (1.22.0)
[✓] Flutter (Channel unknown, 1.22.0, on macOS 11.2.3 20D91, locale en-AO)
    • Flutter version 1.22.0 at /Users/pedromassango/Code/flutter_master
    • Framework revision d408d302e2 (6 months ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/pedromassango/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-6915495)
    • All Android licenses accepted.

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

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

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin installed
    • Dart plugin version 203.7759

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

[✓] Connected device (4 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 11
      (API 30) (emulator)
    • macOS (desktop)            • macos         • darwin-x64     • macOS 11.2.3
      20D91
    • Web Server (web)           • web-server    • web-javascript • Flutter
      Tools
    • Chrome (web)               • chrome        • web-javascript • Google
      Chrome 89.0.4389.90

! Doctor found issues in 1 category.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.2, on macOS 11.2.3 20D91 darwin-x64, locale en-AO)
    • Flutter version 2.0.2 at /Users/pedromassango/Code/flutter_stable
    • Framework revision 8962f6dc68 (6 days ago), 2021-03-11 13:22:20 -0800
    • Engine revision 5d8bf811b3
    • Dart version 2.12.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/pedromassango/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-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 54.0.3
    • Dart plugin version 203.7759

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 89.0.4389.90

! Doctor found issues in 1 category.

The FlutterError.onError handler is being replaced by a handler used by the Flutter development tools (see https://github.com/flutter/flutter/commit/5b1aa60a46bf6317b1ff1e0e10e5751e4491881c)
One way to work around this would be setting FlutterError.onError after calling WidgetsFlutterBinding.ensureInitialized()

We should document this behavior on FlutterError.onError.
/cc @kenzieschmoll

If it is a common use case to add a custom onError handler to an app, perhaps we should allow for multiple onError handlers instead of only one. Overriding the FlutterError.onError handler set by the framework could have some unintended consequences like disabling certain tooling features that depend on the structured error handler being set (structured logs, deep links to the inspector, error notifications in DevTools, etc.)

It is very useful for applications to have a global Error handler, to ensure that ALL errors will be handled by the application.
Without this, we have to put try / catch in all the onPressed () of the application's buttons.
It is very laborious and unnecessary, as they will usually only display an error message.
That global handler can decide what to do when he catches an exception that is not handled by the application.

If it is a common use case to add a custom onError handler to an app, perhaps we should allow for multiple onError handlers instead of only one. Overriding the FlutterError.onError handler set by the framework could have some unintended consequences like disabling certain tooling features that depend on the structured error handler being set (structured logs, deep links to the inspector, error notifications in DevTools, etc.)

It is very useful for applications to have a global Error handler, to ensure that ALL errors will be handled by the application.
Without this, we have to put try / catch in all the onPressed () of the application's buttons.
It is very laborious and unnecessary, as they will usually only display an error message.
That global handler can decide what to do when he catches an exception that is not handled by the application.

duplicate of https://github.com/flutter/flutter/issues/41540

Closing as duplicate of https://github.com/flutter/flutter/issues/48972


New package: Cowsay v0.2.0

Registering package: Cowsay
Repository: https://github.com/MillironX/Cowsay.jl
Created by: @MillironX
Version: v0.2.0
Commit: dd7518654720dc9efedd7a4e2feb9f879b4411c7
Reviewed by: @MillironX
Reference: https://github.com/MillironX/Cowsay.jl/commit/dd7518654720dc9efedd7a4e2feb9f879b4411c7#commitcomment-61044291
Description: :cow2: cowsay for Julia!

I don't think that the package name similarity is actually a problem. It's pretty clear (for English speakers, anyway) what the difference is between {cow,cat,dog}say. Besides, cowsay is the original anyway.
[noblock]