Split CI Jobs for bench and compile and test

This should speed up the whole CI workflow and make it more robust.

Codecov Report

Merging #107 (fc8a383) into master (78418ae) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head fc8a383 differs from pull request most recent head dcc9f97. Consider uploading reports for the commit dcc9f97 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   70.31%   70.31%
=======================================
  Files          10       10
  Lines        1132     1132
=======================================
  Hits          796      796
  Misses        213      213
  Partials      123      123

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


dashboard tokens are missing in the build

[x] bug report
[ ] feature request

Subject of the issue
The dashboard tokens aren't present in the build (linking to unpkg here. You can also find it on npm -> code -> /js/dashboard/lightMode/common.js (same with darkmode version, and any other target platforms)).
Expected Behaviour
The dashboard tokens should be present in the build.
Current Behaviour
The dashboard tokens are not present in the build.
Possible Solution
Not really a solution, but rather a bit of debugging I've done:
While the tokens are present in the source data set, they already disappear during the prebuild step. The generated tmp/tokens.json file still has the sections and structure for dashboard/lightMode and dashboard/darkMode, but all the values are missing.
I noticed that in the source, the dashboard sections reference variables from origin. It looks like these aren't resolved properly during the build, and silently go missing.
Steps to Reproduce
Option 1: inspect the existing published packages.
E.g. on npmjs.org in the code tab you can see that the variables for dashboard/lightMode and dashboard/darkMode are missing (for every target: css, js, android, etc.).
Option 2: build from scratch

Run the build script npm run build
Observe that none of the dashboard variables are present in the files in the dist folder.

Your environment

Can't find this value https://github.com/Sage/design-tokens/blob/f96855e1fc47d5c2fd762f7ad7b2ffd449c68fab/data/tokens.json#L290  as well.


[Story][Dockerfile Improvements] Generalize and reduce the number of Dockerfiles

Background

As an Osmosis engineer, I would like our Dockerfiles to be reusable and modular so that I don't have to make changes in many files and avoid the images going out of sync

Our Dockerfiles share a lot of the same logic. On one side of the issue, some folks update all Dockerfiles, wasting time for no reason. On the other side, other folks only update some, letting the images go out of sync.
First, we should remove images that are not used. Second, we should investigate the rest.  Some can be generalized to be used as a base, and the rest import that base.

#1684
#1685

Status

June 6 - WIP: Defining Scope

At the time of the issue we had the following Dockerimages:
❯ find . -iname "*dockerfile*"

./Dockerfile
./tests/localosmosis/Dockerfile
./tests/localosmosis/mainnet_state/Dockerfile-stateExport
./tests/e2e/initialization/init.Dockerfile
./contrib/images/osmobuilder/Dockerfile
./contrib/images/rbuilder/Dockerfile

Fixing https://github.com/osmosis-labs/osmosis/issues/2482, we removed the contrib folder and the following images:
./contrib/images/osmobuilder/Dockerfile
./contrib/images/rbuilder/Dockerfile

Removed also:
./tests/localosmosis/Dockerfile
./tests/localosmosis/mainnet_state/Dockerfile-stateExport

Seeing that theres only two dockerfiles remaining, can we close this issue?

Yes. Closing


Add more information to res.render

This adds more information to the res.render API documentation, mainly documenting what the view argument actually is and does, and calls out that the argument is to be trusted input, rather than user-controlled input.

Thanks @dougwilson, this is good information to add.  I'm going to make some minor edits to the wording before merging.


Add missing docs files to site navigation

Codecov Report

Merging #1263 (db74b7a) into master (5907d50) will increase coverage by 1.71%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1263      +/-   ##
============================================
+ Coverage     70.23%   71.94%   +1.71%
- Complexity     5063     5111      +48
============================================
  Files           485      473      -12
  Lines         22262    21909     -353
  Branches       2375     2345      -30
============================================
+ Hits          15635    15763     +128
+ Misses         5414     4943     -471
+ Partials       1213     1203      -10

Impacted Files
Coverage Δ

.../src/main/java/ai/djl/ml/xgboost/XgbNDManager.java
75.51% <0.00%> (-6.31%)
:arrow_down:

...gine/src/main/java/ai/djl/dlr/engine/DlrModel.java
51.61% <0.00%> (-3.23%)
:arrow_down:

...pi/src/main/java/ai/djl/util/ClassLoaderUtils.java
50.00% <0.00%> (-2.95%)
:arrow_down:

api/src/main/java/ai/djl/ndarray/NDList.java
65.35% <0.00%> (-2.65%)
:arrow_down:

...ain/java/ai/djl/onnxruntime/engine/OrtNDArray.java
70.37% <0.00%> (-2.36%)
:arrow_down:

api/src/main/java/ai/djl/util/Platform.java
69.81% <0.00%> (-1.89%)
:arrow_down:

...ngine/src/main/java/ai/djl/mxnet/jna/LibUtils.java
40.00% <0.00%> (-1.18%)
:arrow_down:

...in/java/ai/djl/repository/zoo/BaseModelLoader.java
77.23% <0.00%> (-1.10%)
:arrow_down:

...n/java/ai/djl/paddlepaddle/engine/PpNDManager.java
87.50% <0.00%> (-1.08%)
:arrow_down:

...n/java/ai/djl/onnxruntime/engine/OrtNDManager.java
72.22% <0.00%> (-0.76%)
:arrow_down:

... and 53 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 5907d50...db74b7a. Read the comment docs.


Does the client generation not work with schema.json?

I couldn't find any info on whether I can use the schema.json for generating my client code. All the documentation asks for .graphqls files which is hand-written only, as it seems. I would like to re-generate my client as part of the CI to keep it up to date, so it will use the schema.json that is served by the deployed application.
This is the way the Apollo plugin works.
https://github.com/aoudiamoncef/apollo-client-maven-plugin
https://www.apollographql.com/docs/android/essentials/get-started-java/

Have you tried renaming your schema.json into .graphqls and tried from there? When I generate my server, I use several
graphqls files and not a schema.json. Apollo seems to expect the graphql requests in graphqls files only.
In the end its all JSON...and graphql :)
On Mon, 2021-12-13 at 15:20 -0800, Sam Kruglov wrote:

I couldn't find any info on whether I can use the schema.json for generating my client code. All the documentation
asks for .graphqls files which is hand-written only, as it seems. I would like to re-generate my client as part of the
CI to keep it up to date, so it will use the schema.json that is served by the deployed application.

Hum,
I see the point.
The question is to take the result of an introspection query, without any change.
So no, it doesn't work (yet).
But it's an interesting point. I'll check that.
Étienne

Hi there, :)
Yes, would be great if JSON could be supported as many APIs don't make their schema file available, so all you can do is a manual introspection. And this returns JSON...

Hello,
Ok, thanks for this comment.
I'm working on the native compilation mode. I'll work on this one after.
Etienne

Done in release 2.6


Update translations pt_br and add translations pt_pt

I've updated the Portuguese (Brazil) translations and added translations for Portuguese (Portugal).
Changes made:

Updated tooltips, actions, and button titles in pt_br.
Added translations for pt_pt to tooltips, actions, button titles, and measurements.

Please review and merge these changes accordingly. Thank you!

Thank you for updating the translations


resolved merge conflict and sync my code .
added style with sass

there is something wrong with your code.

The deployment shows some extra unnecessary padding for no reason.

What I think id that you are adding complete react project whereas you just have to add your js files and css files.
Kindly go through contributing.md for more insights Mridul.


Build native apps for Windows, Mac, Linux and upload the app files to the v1.0.0 release

We can upload files to the v1.0.0 release at https://github.com/Rookies-RGSOC2016/hoodie-electron-app/releases/edit/v1.0.0

@gr2m @titay2 Hey :) To make an executable and installable native application, I used 'electron-packager'. I used 'electron-packager . --all' command line to make a cross-platform app. But it seems it only created the native app for Linux.
For creating a native app for Window and Mac, do I need to use Window and Mac OS?
If it's working, I wanted to replace the desktop icon with Hoodie icon, but I was not able to find where is the default desktop icon is. I followed this Packaging and Distributing Electron Desktop Apps tutorial which is made by Kyle Robinson Young. This is current scripts in package.json. For replacing the default icon image, we can do '&& cp   In the tutorial, the directory looks like Myapp.app/Contents/Resources/atom.icns and in the official Electron application distribution website, electron desktop seems to have this format '.app/Contents/'. But in my current code, it does not have these directories.
Currently, those features are working with scripts.
-The native app for Linux seems ready(When we click the app icon, we can open the desktop app)
-It has app.asar so that it won't expose the source code.
-It ignores node_module which can take the space.
"scripts": {
    "start": "electron .",
    "package": "asar pack Hoodie-linux-x64/resources/app Hoodie-linux-x64/resources/app.asar",
    "build": "electron-packager . Hoodie --ignore=node_module/electron_"
  }

Good job!
I checked the electron boilerplate project and saw:
"You can create Windows installer only when running on Windows, the same is true for Linux and OSX. So to generate all three installers you need all three operating systems."

@zeke can you point us to some places to learn how other projects do this? For context: the hoodie-electron-app project is our project we are working on with Hoodie’s Rails Girls Summer of Code Team, your help would be much appreciated!

You can create Windows installer only when running on Windows, the same is true for Linux and OSX.

☝ This is incorrect. If you're on MacOS, you can package apps for all three platforms. You cannot, however, code sign MacOS apps from any other platform.
https://github.com/electron-userland/electron-packager#readme
https://github.com/electron-userland/electron-builder#readme (more 🔔 and 🎺)
See also https://github.com/electron/electron/blob/master/docs/api/auto-updater.md, which highlights some popular tools for distributing updates to your users.

Thanks a ton @zeke for your help <3
electron-builder looks great! And I think this is a great example we could learn from: https://github.com/develar/onshape-desktop-shell/blob/master/package.json

@omidfi @zeke @gr2m Thank you so much for the kind helps and helpful links :D We'll read carefully and try to test/use electron-packager & electron-builder and auto-updater tools in our project with MacOS ;D

@gr2m @zeke :D If it's ok ,could I ask one question regarding electron-builder? I tried to install electron-builder by 'npm i electron-builder --save -dev' in Mac and Linux several times. But I got this error and 'electron-builder' is not in the devDependencies. I googled to solve this problem but I wasn't find the solution. How could we save the memory?:)
<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x.... <JS Object>
    1: indexOf(aka indexOf) ....
    2: formatProperty(aka formatProperty)....

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Aborted (core dumped)

--save -dev should be --save-dev with no spaces. That might be the issue.


docs: Update new-feature-tutorial.

This updates the new-feature-tutorial for the recent changes done
in org settings subsystem.
This is my first ever documentation change, so let me know where I improve and what am I still missing :crossed_fingers:

@timabbott Updated the PR.

Merged as d3627ab419608e64b038771585885eaaf482accf, thanks @shubhamdhama!  I did a decent amount of fixing grammar; it's really important to edit your text for good English when writing documentation since the quality of the writing impacts how users perceive a project :).

Yes, I understand good English is an important part of the documentation.
I try to use online grammar tools to remove typos and basic grammar problems but yeah I've to confess I never had a good relation with English grammar :smile:
I'll to do more proofread next time before submitting.

You could also get a friend who is better at English writing to help; there's no need to do it all yourself :)


If docker is not found, it tries to use podman

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
If docker is not found, the program fails.

What is the new behavior (if this is a feature change)?
If docker is not found, it tries to use podman instead. If podman is not found, the program exits.

Other information:

Thanks for the feature contribution & it looks good to me.


CLI Eval

This is a quite small thing, but most interpreters have it.
$ wren -e 'System.print("Hello, world!")'

It seems simple enough (famous last words, right?) to add in basic support for it by breaking the runFile function into the part that actually loads the file and the part that runs the VM (probably in a new function called runVM), and then looking for the flag on the commandline (maybe just seeing if argc == 3 and taking argv[2] for now and not worrying about a full commandline parser).
If this sounds like something you want I can probably do this myself and submit a pull request.

that sound nice, but I would go with solid foundations (like using getopt) and not quickly hacking a non-scalable thing.

I'm ambivalent to the commandline option solution really.
Likely, getopt (or equivalent) could be added with a simple --help and --version flags, because right now wren can't do either of those. But that's beyond the scope of this issue imho.

Yeah, this sounds reasonable to me too. Figuring out how to actually parse the command line options is a tricky one. My hunch is that getopt() is a little too primitive. getopt_long() may not be portable enough.
Right now, the command line options are simple enough that we could probably just hand code them in C like we do for --help and --version.


Respect no_proxy settings next to the proxy settings.

With #21 there is now support for proxy settings, which do not adhere to the no_proxy settings, breaking in our setup where we set both http_proxy and no_proxy.

I've been checking to see if I can contribute, and wanted to discuss first about the direction😁.
From what I can see, https-proxy-agent does not support the no_proxy settings 😞. That means we have two options:

Add a setting to ignore the https_proxy environment variable: ignore_proxy: false, which requires user input.
Write a check to see if the target url is in the no_proxy list, and if so, do not inject the https-proxy-agent here

Tagging @peter-murray for visibility.

Option 1 is the quickest to deliver in this case, but seeing that we are reading the settings from the environment ti does present Option 2 as being a more consistent approach.
If I was to do this, if you explicitly inject the proxy as an input to the action, there would be no ignoring the proxy URLs, otherwise why did you inject it?

Good point. Then it's Option 3?

Make the injection of the proxy as input required and do not load the environment settings at all.

Or would you want do do both 3 and 2 then (sorry if this is getting confusing)?

After going around the bush a few times with this, I landed on the assumption that the no_proxy would be host names, so in the case of GitHub.com would be api.github.com in the string and it works in my limited testing.
@rajbos would you mind checking that the branch no-proxy-support does what you need it to do before I publish this to main and release a new version?

Works like a charm @peter-murray!
I've tested:

With our environment settings (HTTPS_PROXY and NO_PROXY), with a github_api_base_url set
With our environment settings (HTTPS_PROXY and NO_PROXY), without a github_api_base_url set
With our environment settings (HTTPS_PROXY and NO_PROXY), with the https_proxy variable set

Things work as expected with these combinations! Thanks for turning this around so quickly 🤗.


Execution failed for task ':app:minifyReleaseWithR8'.

try build app using this command
flutter build apk --target-platform android-arm,android-arm64,android-x64 --split-per-abi
has problem to build it
* What went wrong:
Execution failed for task ':app:minifyReleaseWithR8'.
> com.android.tools.r8.CompilationFailedException: Compilation failed to complete

flutter doctor -v log
`[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Mac OS X 10.15.4 19E287, locale en)
• Flutter version 1.12.13+hotfix.9 at /Users/majidalnuumani/Documents/development/flutter
• Framework revision f139b11009 (4 weeks ago), 2020-03-30 13:57:30 -0700
• Engine revision af51afceb8
• Dart version 2.7.2
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/majidalnuumani/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• ANDROID_HOME = /Users/majidalnuumani/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_212-release-1586-b4-5784211)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.4.1, Build version 11E503a
• CocoaPods version 1.9.1
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 45.1.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[✓] VS Code (version 1.39.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.7.1
[✓] Connected device (2 available)
• Pixel 4 XL       • 9B191FFBA002R8                           • android-arm64 • Android 10 (API 29)
• No issues found!`
flutter verbose -v

If you examine the logs of flutter build apk --target-platform android-arm,android-arm64,android-x64 --split-per-abi -v you'll should see what is causing the issue.
This is most likely not an issue with flutter, but I'll keep it open to confirm

Hi @MajidAlNuumani
Without additional information, we are unfortunately not sure how to resolve this issue. We are
therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the
bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Fix typo in the doc str -> plot

Codecov Report

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

@@            Coverage Diff             @@
##           master     #465      +/-   ##
==========================================
- Coverage   93.34%   93.18%   -0.16%
==========================================
  Files          29       29
  Lines        3771     3771
==========================================
- Hits         3520     3514       -6
- Misses        251      257       +6

Impacted Files
Coverage Δ

databricks/conftest.py
96.15% <0%> (-3.85%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

databricks/koalas/namespace.py
88.46% <0%> (-1.29%)
:arrow_down:

databricks/koalas/frame.py
94.82% <0%> (-0.14%)
:arrow_down:

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 6efa969...3371086. Read the comment docs.


Add comment in readme about installing libudev

One little thing I came across after merging #321 and playing with the install process.

I just git commit --amend ed it with the proper apt-get instructions. Anything else left?


Sorting

Is it possible to specify the order of the included files? I have a situation where I need a shim file to appear before angular loads. Thanks for your help if you can provide any.

any news @wmbutler ?


Update gaze-in-unity.md

MS author clean-up

Docs Build status updates of commit 3b1ffdf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/gaze-in-unity.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


How to install in Ubuntu 19.04

Ubuntu 19.04, released on April 2019, is not in this list. May I assume this URL will work?
curl https://packages.microsoft.com/config/ubuntu/19.04/multiarch/prod.list > ./microsoft-prod.list

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

ID: a3b8296d-c923-e990-dcad-35ef2b908a2a
Version Independent ID: 5f85fd7b-f2b6-b3b1-6be2-b6054bb33507
Content: Install Azure IoT Edge on Linux
Content Source: articles/iot-edge/how-to-install-iot-edge-linux.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hi @rido-min I would recommend you to stick with what is published on this document. Did you tried running that?

In the doc there is not mention to Ubuntu 19.04, so I dont know how to install.
Should I use the url I suggested? Should I install like a Tier 2?

@rido-min Oh Okay, I thought you tried by modifying the URL. I would not advise you to do that I would recommend you to stick to what it is listed.

But 19.04 is not listed !!
This means that 19.04 is not supported (Tier 1)? May I try to install as Tier 2?
Any plans to include docs for 19.04?

@rido-min  No please stick with what it is in the list

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Introduce HTTP timeout

Context
As discussed in #868 , we should introduce timeout settings to Gradle to avoid hanging connections.
This PR cherry-picks changes from https://github.com/gradle/gradle/commit/25d87bf905d56a7d026084b34a11341cc569f6a7 and removes the parts unrelated to HTTP timeout setting so that this PR wouldn't be a huge monster.
This PR adds timeout settings to HttpClient and adds a system property http.socketTimeout and http.connectionTimeout to control the value.

@bmuschko Thanks for reminding. I've introduced some test fixture to mock SSL handshake timeout. Please let me know if there's anything inappropriate.

@blindpirate As I'm the mentioned community member: Isn't your test-case for SSL handshakes simply putting a timeout in front of everything else instead of actually testing the handshake phase? Could you explain a bit more into detail why you think this is sufficient?

@dreis2211 Thanks for pointing out this. Yes, I was simply putting a timeout before SSL handshake.
I'm not an expert on SSL, so I took a look at jetty 6's SslSocketConnector and hooked customize() method. I thought I was making the handshake timeout, but after looking into this carefully, it looks like I didn't touch the handshake phase.
I would appreciate your opinion on this, how can I mock the SSL handshake timeout effectively?

@blindpirate Arguably a tough nut to crack to hook exactly into that phase. Unfortunately, the fix on the HTTP-Client lib itself didn't add a test either. I will take a look over the weekend, if I find a method.
@bmuschko Any ideas?
In general, I would argue that your implementation misses to set setDefaultSocketConfig as described here

Where are the default timeouts documented?


Integration added: DatoCMS blog starter

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)
New or updated content
Translated content
Changes to the docs site code
Something else!

Description

Closes #
What does this PR change? Give us a brief description.
Did you change something visual? A before/after screenshot can be helpful.

Thank you for this contribution to the Astro Docs, @dospolov ! And welcome to the Face Pile!   🥳


Convert year dimensions to integers (#218)

bring back special handling for year dim
extend to any dim containing year reference
simplify df creation by using again from_dict

Note: it turns out from_dict has same/better performance in comparison with alternative aproach passing index to empty df upon creation.
It also allows to have int dtype (which empty df doesn't support because of NaNs)

@khaeru one of the test is failing because I changed backend method to let using url for hsqldb. I needed that to use in-memory db in test. What was your idea behind limiting hsqldb to use only file-based Dbs?

@khaeru thank you very much for reviewing and commenting on PR. Sorry for not clarifying it in first place - I should not probably request "normal" review, but rather ask for suggestions as I consider this changes as draft.

Remove type conversion for years...

I commented on that in linked ticket https://github.com/iiasa/message_ix/issues/268#issuecomment-557879877. I am ok with both variants mentioned there. The change made here as appropriate ticket #218 exists in this repo. I will remove it from code for the time being, but let's discuss it quickly next week.

Place the methods save_par and read_par inside their corresponding test functions. Then you can even move lines line scen = ixmp.Scenario(… out of save_par into the surrounding function test_read_par_10000—since that line is not what is being benchmarked.

Under normal circumstances your suggestion totally makes sense. But in current setup benchmark does execute the method multiple times (calculating min/max/mean etc), therefore it needs every time new version of scenario (or new database, or other aproach). Creating new scenario every time add constant time and should not have significant effect on large data volumes. Can you suggest how to implement it better with pytest-benchmark?
I'll put test to subfolder, but I would put it to separate file as non-functional, performance test.

Consider the how in-memory hsqldb...

My intention was not to expose this "feature" to external usage, but rather make it possible to use for tests. Should we document features which solely used for testing?
My comment regarding "limiting" was referring to the fact that new implementation is restrictive in this particular aspect (not being said it is good or bad). Sorry for any possible misinterpretation.
If we are going to "advertise" this feature to be used by package users - let's make a separate issue/PR adding documentation/tests/etc.
Originally I didn't want to introduce a lot of changes, but since @OFR-IIASA experienced issues with performance reading parameters elements I got an impression ticket #218 is critical to fix rather quickly.

it needs every time new version of scenario (or new database

Okay, that's a good point—I would then suggest to add a comment like # Create a new, empty Scenario for the test above that line.

I'll put test to subfolder, but I would put it to separate file as non-functional, performance test.

Sure, maybe something like tests/backend/test_jdbc_performance.py. Then other backends could have a similar split.

Should we document features which solely used for testing?

Yes, absolutely. Our test suite is already large and complex and I'm not sure anyone fully understands it. To give people in the future any hope of maintaining it (or allow other people, e.g. colleagues of @gidden, to contribute easily) we need to make it easy to read. I think that should include not making use of 'hidden' features. So if the feature is documented well enough to read the test suite, it is not much additional work to make it a public feature. (Our new RAs are a resource here: if they can't understand the feature used in testing, that means we can do better.)
I understand that the message_data performance issue @OFR-IIASA encountered was urgent to fix; however, in our haste to respond, #213 was merged before we had time to think about it carefully. That in turn spawned #218, this PR, iiasa/message_ix#268, and iiasa/message_ix#269.
Since the performance of #213 was ruled “acceptable”, I think the additional performance improvement from this PR (BTW, see above where I said “leave a comment here with the benchmark values prior to this PR”) does not need to be similarly rushed.

Add comments inline indicating...

Can we discuss it?

Sample output of jdbc backend performance test.
-------------------------------------------------------------------------------------- benchmark: 2 tests -------------------------------------------------------------------------------------
Name (time in ms)             Min                 Max                Mean             StdDev              Median                 IQR            Outliers      OPS            Rounds  Iterations
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
test_save_par_1000        74.7618 (1.0)      102.1507 (1.0)       87.6907 (1.0)       8.6063 (1.0)       89.6419 (1.0)       12.8233 (1.0)           5;0  11.4037 (1.0)          13           1
test_read_par_100000     690.2758 (9.23)     875.4846 (8.57)     786.7553 (8.97)     82.7296 (9.61)     756.9678 (8.44)     144.6853 (11.28)         3;0   1.2710 (0.11)          5           1
-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

It indicated roughly:

saving 1000-element parameter takes 87 milliseconds
reading 100000-element parameter takes 786 milliseconds
The results are highly environment dependent and should not be used in combination with results taken from another environment.

As discussed with @khaeru we close this PR for the time being as:

moving conversion to non-string dtypes to ixmp core is delayed until figuring out all the requirements (possibly for another typed of backends)
in-memory hsqldb support would be addressed as separate ticket (#224)
performance tests will be added separately (#215)


nextcloud-client 3.0.3 crashes

Describe the bug
Since I last updated, the Nextcloud client consistently crashes. I've installed it via home-manager.
To Reproduce
Steps to reproduce the behavior:
$ systemctl --user restart nextcloud-client.service

$ systemctl --user status nextcloud-client.service
● nextcloud-client.service - Nextcloud Client
     Loaded: loaded (/nix/store/pnlqj1ax06nqj62pw6awjq3kf58r205b-home-manager-files/.config/systemd/user/nextcloud-client.service; enabled; vendor preset: enabled)
     Active: failed (Result: signal) since Fri 2020-11-13 11:20:26 CET; 1s ago
    Process: 5329 ExecStart=/nix/store/4zp0f109q5z515nfah3nskkfk901i8rn-nextcloud-client-3.0.3/bin/nextcloud (code=killed, signal=SEGV)
   Main PID: 5329 (code=killed, signal=SEGV)

Nov 13 11:20:24 nixos systemd[2107]: Started Nextcloud Client.
Nov 13 11:20:26 nixos systemd[2107]: nextcloud-client.service: Main process exited, code=killed, status=11/SEGV
Nov 13 11:20:26 nixos systemd[2107]: nextcloud-client.service: Failed with result 'signal'.
Nov 13 11:20:26 nixos systemd-coredump[5342]: Process 5329 (.nextcloud-wrap) of user 1000 dumped core.

Notify maintainers
@caugner
Metadata

system: "x86_64-linux"
host os: Linux 5.4.70, NixOS, 21.03pre246543.24c9b05ac53 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.7
channels(root): "musnix, nixos-21.03pre251123.b839d4a8557, nixos-hardware"
channels(turion): "home-manager, nixpkgs-21.03pre251123.b839d4a8557, nixos-19.03-19.03.173691.34c7eb7545d"
nixpkgs: /home/turion/.nix-defexpr/channels/nixpkgs

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
nextcloud-client

@turion Unfortunately I'm not familiar with home-manager, but let's try and figure this out.

Can you find out the exact channel version of the last configuration before the crash?
Is it normal that you have nixpkgs-21.03pre and nixos-19.03 channels enabled at the same time?

This also happens for me when using nextcloud-client on nixos without home-manager.

@bachp Do you have any interesting error messages apart from the segfault when running the client with debug enabled:
nextcloud --logdebug --logfile nextcloud-debug.log

No but I made an odd observation.
I wanted to check if 3.0.2 also crashes so I reverted https://github.com/NixOS/nixpkgs/commit/ea161a63070ee425a1937a2d49d2557ca0f88d4f in my nixpkgs. As I expected version 3.0.2 did not crash.
So "un-reverted" the above commita and went back to 3.0.3. And to my suprise it is working now too.
What I don't know know is if the problem was that I went from an older version (3.0.0 or 3.0.1, unfortunatly I ran garbage collect so I don't know) directly to 3.0.3 and there was some incompatiblity in the config that was fixed by having 3.0.2 in between?

I was able to reproduce the issue locally after switching temporarily from 3.0.2 @ nixos-20.10 to 3.0.3 @ nixos-unstable.
$ nextcloud --logdebug --logfile nextcloud-debug.log
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
[1]    11972 segmentation fault (core dumped)  nextcloud --logdebug --logfile nextcloud-debug.log

$ tail nextcloud-debug.log
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database ]	[ OCC::SyncJournalDb::commitInternal ]:	Transaction commit  "update database structure: add e2eMangledName index" and starting new transaction
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::exec ]:	SQL exec "PRAGMA table_info('blacklist');"
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database ]	[ OCC::SyncJournalDb::tableColumns ]:	Columns in the current journal:  QVector("path", "lastTryEtag", "lastTryModtime", "retrycount", "errorstring", "lastTryTime", "ignoreDuration", "renameTarget", "errorCategory")
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::exec ]:	SQL exec "CREATE INDEX IF NOT EXISTS blacklist_index ON blacklist(path collate nocase);"
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::exec ]:	Last exec affected 0 rows.
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database ]	[ OCC::SyncJournalDb::commitInternal ]:	Transaction commit  "checkConnect End"
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::exec ]:	SQL exec "SELECT COUNT(*) FROM metadata"
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::bindValue ]:	SQL bind 1 QVariant(int, 3)
2020-11-13 18:10:24:498 [ debug nextcloud.sync.database.sql ]	[ OCC::SqlQuery::exec ]:	SQL exec "SELECT path FROM selectivesync WHERE type=?1"
2020-11-13 18:10:24:499 [ info nextcloud.sync.clientproxy ]:	Set proxy configuration to use the preferred system proxy for http tcp connections

I would guess it's an upstream bug.

@bachp Would you be able to checkout nixpkgs at 57883eea4a4d8fad22261c11820df6b8abc53f16 and cherry-pick ea161a63070ee425a1937a2d49d2557ca0f88d4f? If it also crashes, we know it's an upstream bug.

Would you be able to checkout nixpkgs at 57883ee and cherry-pick ea161a6? If it also crashes, we know it's an upstream bug.

Just did this myself (after figuring out how that works, hadn't done it in a while) and indeed 3.0.3 crashes, so it's almost certainly an upstream bug.

Note to myself: https://github.com/nextcloud/desktop/commit/3427dadaeb3527e6fe395b5aa5a0e6b299afc98b breaks the Nix build for 3.1.0-pre.

Now I arrived at the same point as @bachp in https://github.com/NixOS/nixpkgs/issues/103680#issuecomment-726875811 where the crash is suddenly no longer reproducible.

It would probably help to gdb the nextcloud-client, but I have no clue how this works in Nix.
(nix-build -E 'with import <nixpkgs> {}; enableDebugging nextcloud-client', as suggested in the Forum, doesn't work.)

If anybody finds out how the package can be built with enableDebugging (or keepDebugInfos), it would be possible to have a look at the coredump (identified via coredumpctl and displayed via coredumpctl gdb <PID>), which hopefully has a stacktrace to narrow down the culprit.
PS: Other things I tried that didn't work:

Adding "-DCMAKE_BUILD_TYPE=Debug" to the cmakeFlags.
Running nix-build -I nixpkgs=. -E 'import <nixpkgs> {overlays = [(self: super: { stdenv = super.keepDebugInfo super.stdenv; } )];}' -A nextcloud-client (caused everything to be built with keepDebugInfo, not just our nextcloud-client).

If anybody just wants a quickfix (using 3.0.2 instead of 3.0.3) you can use this overlay
self: super:
{
  nextcloud-client = super.nextcloud-client.overrideAttrs (old: {
    version = "3.0.2";

    src = super.fetchFromGitHub {
      owner = "nextcloud";
      repo = "desktop";
      rev = "v3.0.2";
      sha256 = "0qzriiaj6sjdkgh61iihvpsvhi0kvfwjz1vvjnwdlfdx2s4xmv24";
    };
  });
}

@turion @bachp @hoppla20 Would any of you be able to provide a backtrace?

@caugner Unfortunalty I'm not able to reproduce the issue anymore.

Same here. Just rebuild my system with nextcloud from the channel, seems to work perfectly...

Same :/ very strange. Close? Or write a nixos test and bisect?

In my case the segfault yields:
$ nextcloud
2020-11-20 13:53:58:001 [ debug default ]	[ unknown ]:	static bool LibSecretKeyring::findPassword(const QString&, const QString&, QKeychain::JobPrivate*)
[1]    3985 segmentation fault (core dumped)  nextcloud

So apparenty, at least for me, the segfault is related to keychain.
Guys, what's your working 3.0.3 package hash?
I've got a chance to test these two and both segfault in the same way:
/nix/store/4zp0f109q5z515nfah3nskkfk901i8rn-nextcloud-client-3.0.3
/nix/store/cgbzjarf3z4b8yq35mv25qbcki51kfdl-nextcloud-client-3.0.3

@cptMikky Thanks! I have forwarded to the nextcloud/desktop repository.
Out of interest, how did you manage to retrieve that backtrace ?

It's not, it's just what it spews to the stdout :) I was kind of surprised I didn't see that information anywhere above, that's why I posted it. I can try to do the actual backtrace if it may be beneficial.

I have:
$ ls /nix/store/*nextcloud-client-3.0.3/bin/nextcloud
/nix/store/4zp0f109q5z515nfah3nskkfk901i8rn-nextcloud-client-3.0.3/bin/nextcloud  /nix/store/cgbzjarf3z4b8yq35mv25qbcki51kfdl-nextcloud-client-3.0.3/bin/nextcloud

So probably exactly the same executables. But I didn't observe any segfaults since a few days. Weird.

If you want, we can check whether the build was reproducible:
$ md5sum /nix/store/4zp0f109q5z515nfah3nskkfk901i8rn-nextcloud-client-3.0.3/bin/nextcloud
f0691038eae95fc27d9290449bf7b17e  /nix/store/4zp0f109q5z515nfah3nskkfk901i8rn-nextcloud-client-3.0.3/bin/nextcloud

$ md5sum /nix/store/cgbzjarf3z4b8yq35mv25qbcki51kfdl-nextcloud-client-3.0.3/bin/nextcloud
570c7425d0ae9d4264375991b6062769  /nix/store/cgbzjarf3z4b8yq35mv25qbcki51kfdl-nextcloud-client-3.0.3/bin/nextcloud

My binaries have the same checksum so I guess the problem may not be in the nextcloud client itself.

I can try to do the actual backtrace if it may be beneficial.

@cptMikky Yes, that would help a lot. I didn't manage to create a debug build of the package.

I also have the same issue. Meanwhile I'm using an older version installed from 20.03 https://github.com/NixOS/nixpkgs/archive/f05c380a51da.tar.gz, but I would much prefer have a new version. So far I can still reproduce the bug:
$ ls -al $(which nextcloud)
lrwxrwxrwx 1 root root 80 janv.  1  1970 /run/current-system/sw/bin/nextcloud -> /nix/store/87lxd5fn78hwgylpbcklppni0hd0xl81-nextcloud-client-3.0.3/bin/nextcloud

$ md5sum /nix/store/87lxd5fn78hwgylpbcklppni0hd0xl81-nextcloud-client-3.0.3/bin/nextcloud
197980327a2e25a1142c116ef759ba71  /nix/store/87lxd5fn78hwgylpbcklppni0hd0xl81-nextcloud-client-3.0.3/bin/nextcloud

$ nextcloud
Segmentation fault (core dumped)

However, I'm not sure to know how to get a backtrace, let me know how I could provide more information.

Guys just to update, I started using the Nextcloud client from the master branch, currently everything is working fine, but I'm working on a version bump for it.

I already opened the PR, if anyone can test I appreciate  :D


IGNITE-18732 Remove blocking calls from SchemaManager

https://issues.apache.org/jira/browse/IGNITE-18732

regarding to PR seems you need to change issue title and PR description.


[PRIO 1] Fixed IOS build

@LucasBrazi06 you can merge this one :) Then I can resolve the conflicts on the others :)

@AlixH  done :)


[T4A3][T11-B1]Paul Tan Han Kiat

Ready for review.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.

@pyokagan Some comments added. Please close the PR after reading comments.


fix(curriculum): pass tests with x and y overflows

It's not necessary, but may help the test pass on iPads
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This is a bit of a wild hunch but it may close
Close https://github.com/freeCodeCamp/freeCodeCamp/issues/47513

@moT01 can you take a look at this one on Safari?
__
  Summary: :question: I have a question

I'd be fine swapping this for overflow: hidden - clip has the feature of preventing programmatic scrolling, but this is a penguin do we need that

@ojeytonwilliams are you fine with closing this in favour of https://github.com/freeCodeCamp/freeCodeCamp/pull/48163?

Absolutely. I don't think this PR does anything.  Thanks, @naomi-lgbt!


Where to find "log console"

Been looking around in my project + the issues here and ive seen something about logback. Is there a way i can have the stacktrace sent in the console too, as i would rather have it there while testing? where can i find it at all, since i havent found it yet

https://jda.wiki/setup/logging/

Alright, I've followed that. I also copied your config from https://github.com/Kaktushose/jda-commands/issues/130, and unfortunately theres no documentation in your wiki that tells me how to enable console logging.

Well, everyone is responsible for their own logging implementation and setting up logging has nothing to do with my Framework, so no need to mention it in my docs.
However, I don't know what you mean by console logging. If you followed the JDA guide, you should have a console output

No, for some reason I don't. I still get the same warning as usual in console when the bot starts.

Well, everyone is responsible for their own logging implementation and setting up logging has nothing to do with my Framework, so no need to mention it in my docs. However, I don't know what you mean by console logging. If you followed the JDA guide, you should have a console output

Also for this, I was mainly wondering where the "logs" mentioned in the built-in error handler embed, and if I'm able to disable it or edit it at all. If not, might be a good feature to add

Well, it depends on your logging setup where your log files are stored. If you need further help you need to share your logging configuration in order for me to help you

closed due to inactivity


Assignments 6 and 7

Almost everything is working as expected.
The only problem is that when I press the Load From XML button in the MainMenu and the press List all the animals are are listed and their class is OK, but the values for all the properties(name, nrOfLegs etc.) are taken from the contructor without parameters for the given class and not from the XML file.
In the AnimalRepository class there is a huge switch in which at first an animal is created then its properties are read with the decodeFromXml.
Example:
case Constants.Animals.Insects.Cockroach:
Animal cockroach = new Cockroach();
cockroach.decodeFromXml(element);
return cockroach;
As in the list all animals have their default properties defined for example in the Cockroach() constructor I suppose that the problem is somewhere in the decodeFromXml function, but I don!t know what it is.
I hope I was able to explain the problem.
I will add the part with generating random animals and addig/listing employees at a later date. Now I would really like to start learning Android development, so I'll be doing that. :)

Don't worry about this issue, it's not that important. Good luck with the Android part! Ask away if you have any questions


feat: add l1 to l2 message witness getter to cli and pxe

https://aztecprotocol.slack.com/archives/C04PUD9AA4W/p1723568232864909?thread_ts=1723566026.754609&cid=C04PUD9AA4W
Problem:

I'm stuck bridging devcoin to L2: The bridging cmd right now only bridges publicly (easy to add a private bridge) but in order to access my bridged funds I need to know at which index the message landed at in the tree which I can not query just using the CLI

Solution:
Adding this API to the PXE and cli, and doing a little bit of reorganization as we now use this code twice.

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (Token:transfer_public): 44.9 (+102%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,761
1,556
706
727 (-4%)
771 (+1%)

proof_construction_time_sha256_30_ms
11,812
3,174
1,410
1,441
1,479 (+1%)

proof_construction_time_sha256_100_ms
45,518
11,935 (-1%)
5,470 (+1%)
5,375 (-1%)
5,388

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
59.0
87.0 (-2%)

proof_construction_time_poseidon_hash_30_ms
1,530
421
203
230 (+1%)
269

proof_construction_time_poseidon_hash_100_ms
5,638
1,515
675
720 (-2%)
746

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
4,324
7,844
14,852

l1_rollup_calldata_gas
49,708
92,462
177,536

l1_rollup_execution_gas
1,294,371
2,042,093
3,869,206

l2_block_processing_time_in_ms
254 (-1%)
449 (+3%)
799 (-1%)

l2_block_building_time_in_ms
9,307
18,290 (+1%)
36,154 (-1%)

l2_block_rollup_simulation_time_in_ms
9,306
18,289 (+1%)
36,154 (-1%)

l2_block_public_tx_process_time_in_ms
7,844 (-1%)
16,722 (+1%)
34,582 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
2,890 (+3%)
3,911 (+3%)

node_database_size_in_bytes
12,628,048
16,777,296

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Circuit
simulation_time_in_ms
witness_generation_time_in_ms
input_size_in_bytes
output_size_in_bytes
proving_time_in_ms

private-kernel-init
101 (-1%)
390
21,846
44,858
N/A

private-kernel-inner
169 (+1%)
702
72,545
45,005
N/A

private-kernel-reset-tiny
507
1,192
69,614
44,844
N/A

private-kernel-tail
272
196 (+1%)
50,760
52,256
N/A

base-parity
5.53
N/A
160
96.0
N/A

root-parity
32.9
N/A
69,084
96.0
N/A

base-rollup
2,872
N/A
187,817
664
N/A

root-rollup
38.6
N/A
54,525
716
N/A

public-kernel-setup
100 (+1%)
N/A
103,760
71,222
N/A

public-kernel-app-logic
106
N/A
103,599
71,222
N/A

public-kernel-tail
572
N/A
409,190
16,414
N/A

private-kernel-reset-small
481
N/A
66,533
45,629
N/A

private-kernel-tail-to-public
1,469
705
507,554
1,697
N/A

public-kernel-teardown
93.7 (-1%)
N/A
104,005
71,222
N/A

merge-rollup
19.0
N/A
35,742
664
N/A

undefined
N/A
N/A
N/A
N/A
67,829

Stats on running time collected for app circuits

Function
input_size_in_bytes
output_size_in_bytes
witness_generation_time_in_ms

ContractClassRegisterer:register
1,344
11,731
347

ContractInstanceDeployer:deploy
1,408
11,731
18.2

MultiCallEntrypoint:entrypoint
1,920
11,731
428 (+1%)

FeeJuice:deploy
1,376
11,731
390

SchnorrAccount:constructor
1,312
11,731
106 (+1%)

SchnorrAccount:entrypoint
2,304
11,731
435

Token:privately_mint_private_note
1,280
11,731
138

FPC:fee_entrypoint_public
1,344
11,731
26.2 (+3%)

Token:transfer
1,312
11,731
284 (-1%)

Benchmarking:create_note
1,344
11,731
102 (-2%)

SchnorrAccount:verify_private_authwit
1,280
11,731
27.9 (+1%)

Token:unshield
1,376
11,731
562

FPC:fee_entrypoint_private
1,376
11,731
752

AVM Simulation
Time to simulate various public functions in the AVM.

Function
time_ms
bytecode_size_in_bytes

FeeJuice:_increase_public_balance
52.4
8,139

FeeJuice:set_portal
9.70 (-3%)
2,362

Token:constructor
80.9 (-2%)
31,107

FPC:constructor
53.6 (-3%)
22,380

FeeJuice:mint_public
37.5 (-1%)
6,150

Token:mint_public
43.0 (+2%)
11,720

Token:assert_minter_and_mint
68.2 (+1%)
8,028

AuthRegistry:set_authorized
44.5 (+17%)
4,537

FPC:prepare_fee
241 (+3%)
8,812

Token:transfer_public
:warning: 44.9 (+102%)
47,374

FPC:pay_refund
67.0 (+26%)
12,114

Benchmarking:increment_balance
978
7,450

Token:_increase_public_balance
38.2 (-5%)
8,960

FPC:pay_refund_with_shielded_rebate
63.8 (-3%)
12,663

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.165 (+2%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
2.17 (+1%)
3.84 (+1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.112 (+1%)
0.107 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
11.3 (-2%)
17.6 (-1%)
30.7
60.5 (+3%)
111

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.108 (-2%)
0.102 (-1%)
0.0996 (-1%)
0.104 (+3%)
0.0997 (-1%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
14.2 (-2%)
25.8
43.0 (-2%)
83.6 (+2%)
159 (-2%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.108 (-2%)
0.104
0.104 (-2%)
0.103 (+2%)
0.100 (-3%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
16.4 (+1%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
132
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.105 (+1%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
64,779
668,997

Transaction size based on fee payment method
| Metric |  |
| - |  |


⚠ Bofrost.lu has degraded performance

In 8868d30, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1684 ms

Resolved: Bofrost.lu performance has improved in 74a5b80 after 10 minutes.


⚠ Bofrost.gr has degraded performance

In 3f2c836, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2771 ms

Resolved: Bofrost.gr performance has improved in a7d94e4 after 2 minutes.


show the owner name in admin UI

This PR fixes #21

@KimchaC this PR solves your admin UI issue.
Existing devices will still show the Subject instead of the actual user's name or email. You'll have to remove/add the devices again for this info to show.
If you can't ask your users to remove/add their devices you can update the stored device JSON files in <storage-directory>/*/<device>.json manually. The new fields are ownerName: "", ownerEmail: "" and ownerProvider: "". The ownerProvider is the name of the auth provider you're using (any string or empty will be fine).


Error while installing jfrog/platform v1.8.2: checksum list has no SHA-256

Describe the bug
When trying to upgrade the jfrog/platform provider from 1.8.1 to 1.8.2 I get the error: Error while installing jfrog/platform v1.8.2: checksum list has no SHA-256.  The Artifactory version is 7.77.11 and also fails on the Jfrog cloud platform.
I am using Terraform version 1.9.2. The bug can be reproduced by running terraform init with the following configuration:
terraform {
    required_providers {
        platform = {
            # https://registry.terraform.io/providers/jfrog/platform/latest
            source  = "jfrog/platform"
            version = "<2.0.0"
            }
    }
}

Requirements for and issue

[x] A description of the bug
[x] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue). If this is not supplied, this issue will likely be closed without any effort expended.
[x] Your version of artifactory (you can curl it at $host/artifactory/api/system/version
[x] Your version of terraform
[x] Your version of terraform provider

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

I just ran into the same issue. The SHA256SUM listed in the GitHub release is
6f5d31c7e23c3c279bdc092da3e6f548e09e51d28252e73c92088ca1f3d55264  terraform-provider-platform_1.8.2_darwin_arm64.zip

But
❯ curl  'https://github.com/jfrog/terraform-provider-platform/releases/download/v1.8.2/terraform-provider-platform_1.8.2_darwin_arm64.zip' | shasum -a 256
e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855

@mgr110 @NiklasRosenstein We recently switched to the Terraform Registry Org beta program and I suspect it may be the issue. 11.2.0 is the first version to be published after migrating to the beta program. Nothing on our side with regards to the packaging or signing of the provider has changed.
Looks like TF registry has not updated their copy of the checksum file due to file name parsing issue on their side.

@mgr110 @NiklasRosenstein HashiCorp is aware of this issue. They've manually fixed the issue for Artifactory provider and will be working on the same for this provider.
They have also identified the issue on their side and are working on a fix.

HashiCorp has resolved the issue with this release. I've verified I can install 1.8.2 now.
They have also identified the issue on their side and are working on a fix.

The TF registry had not updated their checksums correctly, as mentioned in this issue. 1.8.2 can now be installed.


Release v1.10.0

This is a draft of changelog for the v1.10.0 release.
Here you can find all related translations changes: v1.9.3...master.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release. You can read more about the process on forum.

Show release notes preview
v1.10.0

Added support for new extensions:

marchccc/flarum-ext-mentions-group-users
xsoft-ict/mason-tag

Updated translations for extensions:

acpl/flarum-lscache
antoinefr/flarum-ext-money
askvortsov/flarum-pwa
clarkwinkelmann/flarum-ext-mailing
club-1/flarum-ext-cross-references
datlechin/flarum-silent-edit
fof/links
hehongyuanlove/flarum-auth-qq
ziiven/flarum-money-leaderboard
ziiven/flarum-raffle-ticket

All changes: v1.9.3...v1.10.0.

Success! Now you can announce new release on forum:
## Version [`v1.10.0`](https://github.com/flarum-lang/chinese-traditional/releases/tag/v1.10.0)

**Added support for new extensions**:

* [`marchccc/flarum-ext-mentions-group-users`](https://github.com/Marchccc/flarum-ext-mentions-group-users)
* [`xsoft-ict/mason-tag`](https://github.com/xsoft-ict/fof-mason-tag)

**Updated translations for extensions**:

* [`acpl/flarum-lscache`](https://github.com/android-com-pl/flarum-lscache)
* [`antoinefr/flarum-ext-money`](https://github.com/AntoineFr/flarum-ext-money)
* [`askvortsov/flarum-pwa`](https://github.com/askvortsov1/flarum-pwa)
* [`clarkwinkelmann/flarum-ext-mailing`](https://github.com/clarkwinkelmann/flarum-ext-mailing)
* [`club-1/flarum-ext-cross-references`](https://github.com/club-1/flarum-ext-cross-references)
* [`datlechin/flarum-silent-edit`](https://github.com/datlechin/flarum-silent-edit)
* [`fof/links`](https://github.com/FriendsOfFlarum/links)
* [`hehongyuanlove/flarum-auth-qq`](https://github.com/Hehongyuanlove/flarum-auth-qq)
* [`ziiven/flarum-money-leaderboard`](https://github.com/Ziiven/flarum-money-leaderboard)
* [`ziiven/flarum-raffle-ticket`](https://extiverse.com/extension/ziiven/flarum-raffle-ticket)

All changes: [v1.9.3...v1.10.0](https://github.com/flarum-lang/chinese-traditional/compare/v1.9.3...v1.10.0).

In order to update run these commands:

```console
composer update flarum-lang/chinese-traditional
php flarum cache:clear
```


Reporting of device attribute using AWSIotDevice creates a lot of traffic.

While extending AWSIotDevice and using annotation class AWSIotDeviceProperty, the SDK calls getter methods every X milliseconds to get the latest value of the attribute. If the attribute does not change on the device, it looks the SDK is still updating the shadow with same JSON object. Using a 0 for report interval will never update the shadow.
Is there a method already in the SDK which allows to send updates to IoT shadow only when attribute changes? If not, is there a plan to support such thing for AWSIotDevice class in future?

There is not currently a way to report only those attributes that have changed. By default, everything is reported at a fixed interval.
If you want to manually control when updates are sent, set the reporting interval to 0 to disable automatic reporting. When you wish to publish the latest shadow values, send them yourself. You can see some examples in the AWSIotDevice docs. Something like:
String jsonDocument = awsIotDevice.get();
...
client.update(jsonDocument);

Got it, thanks.


put  and  into edittrans

From<EMAIL_ADDRESS>on June 28, 2012 09:54:49
currently, both elements are defined in shared. All comparable elements are defined in edittrans instead. Should be moved for consistency…
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=81

From<EMAIL_ADDRESS>on June 28, 2012 12:43:55
Status: Fixed


window_with_time_or_count doesn't seem to cap by count

Hi!
Am I wrong in assuming that in window with time or count the first window cap wins? I'd expect the windows to be capped at 100k, but they're clearly limited via the time. Or, maybe a scheduler is missing?
In the quick tryout of the window_with_time_or_count operator I'd expect the windows to be capped at 100k:
#include <rx.hpp>
#include <chrono>
#include <string>
#include <memory>
#include <fmt/ostream.h>

int main() {
    int counter = 0, count = 0;
    auto values = rxcpp::observable<>::range(1, 1000000)
        .window_with_time_or_count(std::chrono::seconds(1), 100000);
    values.
        subscribe(
            [&counter, &count](rxcpp::observable<int> window) {
        int id = counter++;
        printf("[window %d] Create window\n", id);
        window.count()
            .subscribe([](int c) {printf("Count in window: %d\n", c); });
        //...
        window.subscribe(
            [id, &count](int v) {count++;},
            [id, &count]() {printf("[window %d] OnCompleted: %d\n", id, count); });
    });
}

output:
[window 0] Create window
Count in window: 170306
Len: 910731<PHONE_NUMBER>01112131415161718192021222324252...)
[window 0] OnCompleted: 170306
[window 1] Create window
Count in window: 172957
Len: 1037742<PHONE_NUMBER>08170309170310170311170312170313...)
[window 1] OnCompleted: 343263
[window 2] Create window
Count in window: 171106
Len: 1026636<PHONE_NUMBER>65343266343267343268343269343270...)
[window 2] OnCompleted: 514369
[window 3] Create window
Count in window: 173769
Len: 1042614<PHONE_NUMBER>71514372514373514374514375514376...)
[window 3] OnCompleted: 688138
[window 4] Create window
Count in window: 175673
Len: 1054038<PHONE_NUMBER>40688141688142688143688144688145...)
[window 4] OnCompleted: 863811
[window 5] Create window
Count in window: 136189
Len: 817135<PHONE_NUMBER>13863814863815863816863817863818...)
[window 5] OnCompleted: 1000000

good catch! there is a bug in that operator.
the releasewindow() result, when the count is reached, is not being scheduled.
I will get a fix out soon.

superb! will wait for a nuget update :)

I pushed through a github release v3.0.0 and a nuget release v3.0.1  (I made a mistake with v3.0.0)
Try it out and see if it works!

Will do. Thanks!

the window is now capped correctly, but getting a crash at the end in Release (not in Debug).

the window is now capped correctly, but getting a crash at the end if the window size is 0. Either the I'm using the operators wrong, or there's another bug, analyzing a bit

Yes :)
I use start_with with the reduce operators that require at least one value.


Fix indentation in README

De-indent lists so that they aren't quoted lists.

Coverage remained the same at 68.326% when pulling 7059eb6e7f869b1159eb03f6a527654e7ecec526 on jwhitlock:fix_readme into 308237ee05f19e21433aa2ff2f755d2860358a09 on django-nose:master.


Update README.md

the height of the accordion need to be specified to make the this very simple example working. this thing has be documented, because can cause so much confusion, people will think that the Accordion not working properly.

For prove, try this example, the content of the accordion wont showing up.
import React, { Component } from 'react';
import { View, Text } from 'react-native';
import Accordion from 'react-native-collapsible/Accordion';

export default class CurrencyDetail extends Component {
    constructor(props) {
        super(props);
    }

    renderHeader(section, index, isActive) {
        return (
            <View>
                <Text>header: {section}</Text>
            </View>
        )
    }

    renderContent(section, i, isActive) {
        return (
            <View>
                <Text>content: {section}</Text>
            </View>
        )
    }

    render() {
        return (
            <View style={{flex: 1}>
                <Accordion
                    sections={['a', 'b', 'c', 'd', 'e']}
                    renderHeader={this.renderHeader}
                    renderContent={this.renderContent}
                />
            </View>
        )
    }
}

Try to specify the height, it'll works
<Accordion
    style={{ height: 300 }}
    sections={['a', 'b', 'c', 'd', 'e']}
    renderHeader={this.renderHeader}
    renderContent={this.renderContent}
/>

Hmm, example project works without an explicit height? Can you try removing your wrapping <View>?

Hey, this issue has been inactive for long time and will be closed. If the issue still persists feel free to tag me to reopen.


Fix test code coverage reporting

Fixes #828 by updating the nyc configuration.
This builds on the work of @IlyaRucavitcyn's fork. Turns out the issue was the npm test command is setting the BABEL_ENV environment variable to commonjs, which doesn't have the required istanbul plugin.
To fix this, I just added the commonjs env plugin to the babel test environment and then switched npm test to use BABEL_ENV=test.
It now seems to produce an accurate coverage report and coverage.lcov is no longer empty after running npm run coverage.

Sorry, I forgot to look at this! Thanks!


Re-reverse order of scaffolded db fields.

Fixes #3802

@tractorcow can you weigh in on this post 3.1.9 release? :) Wonder if we need a test in place for this to avoid any future regressions?

@sheadawson 85b4ba15fcfa83ef11569121020aa3b18840c70d reversed the order that classes (but not fields) were looped over, so I’m not sure whether this will entirely fix the issue. Take the following example:
class MyObject extends DataObject {
  private static $db = array(
    'Title' => 'Varchar',
    'Description' => 'Text'
  );
}

class MyChildObject extends MyObject {
  private static $db = array(
    'Location' => 'Varchar',
    'Price' => 'Varchar'
  );
}

The expected order is: Title, Description, Location, Price. I think this change would result in Description, Title, Price, Location.
In the short term, I think we could change DataObject::db() to only reverse the classes array if $fieldName is present. That solves both this issues and the bug I originally was trying to fix.
This has pointed out another issue: if you don’t provide an argument to DataObject::db(), you can’t overload the field type (basically the same bug as #3620) - but that bug already exists independent of my suggested change.

Hi @kinglozzer , there's also a bug I can see with your pull request.
Because you reverse the order you check classes, checkse for db('FieldName') will now correctly inherit, but db() will give the incorrect field types. E.g. if you override 'FieldName' with a new type, checking that field by itself will return your new type, but in the db() list the old type will remain.
My suggestion is to fix both issues in db method, rather than trying to fix it everywhere the method is called.
I would expect the following:

Requests for a single field name should return the type specified by the highest class which defines it
Requests for all DB fields should return the most highly defined specification for each field
Requests for all DB fields should return the fields in the same order as per 3.1.8, regardless of the field type.

I think the original commit is fine, but a new array_merge will be needed to merge fields together to respect the above logic.

See my comment on https://github.com/silverstripe/silverstripe-framework/pull/3623/files#r23183303

@tractorcow yes, that’s the bug I was talking about in my above comment. We need to remove the array_reverse from DataObject::db() if no argument is given, or calls to it will return child class db fields before their parent classes (see my above example). I’ll put together a PR

That would work. I wonder though, at what point do you have a separate method for each behaviour? A single method should have a single purpose, IMO.

Closing in favour of #3804


Validate Hive location path includes nameservice name

Check Hive table location path is valid in HDFS.
We had a cluster that at one time had HDFS HA enabled and tables were defined with a location that included the nameservice name.  When HDFS HA was disable  the hive table location was not updated to reflect the correct HDFS path.
HSMM failed during migration due to some of the hive tables had incorrect hdfs location defined.
It appears that the hive-sre tool did not report that these tables had invalid hdfs locations.

We ran into another client that had issues with incorrect location missing hdfs://nameservice-example/data/
SDS table in metastore had a location for the table so it did not get flagged as missing but it was incorrect and caused failure in hive metadata schema validation.
Location should be looks like this:
hdfs://nameservice-example/data/...
Support Case 911529

For the missing nameservice id, besides validating the SDS entries (table and partition locations) the DBS table also needs to be validated.
SELECT * from SDS where LOCATION NOT LIKE 'hdfs://%' and LOCATION NOT LIKE 'har://%' ;
and
SELECT DB_ID,NAME,DB_LOCATION_URI from DBS where DB_LOCATION_URI NOT LIKE 'hdfs://%';
See also case 1057442.
Could you help with this @dstreev ?


Process of AggregationStrategy implementation of pollEnrich

After calling pollEnrich, I want to set a new header from the field of the previous body via oldExchange. But, I can't implement it. I tried this approach but failed. Can anyone guide me to implement AggregationStrategy in Karavan?
This is my flow in Karavan:
apiVersion: camel.apache.org/v1
kind: Integration
metadata:
  name: testflow
spec:
  flows:
    - route:
        from:
          uri: timer:test
          steps:
            - setBody:
                expression:
                  constant:
                    expression: Sample message
            - log:
                message: ${body}
            - pollEnrich:
                expression:
                  simple:
                    expression: file:file_src?noop=true
            - log:
                message: 'polled: ${body}'
          parameters:
            delay: '1000'
            period: '3000'

I want to implement this Java part of AggregationStrategy through Karavan.
.....
.pollEnrich()
.simple("file:file_src?noop=true")
.timeout(0)
.cacheSize(-1)
.aggregationStrategy((oldExchange, newExchange) -> {
    newExchange.getMessage().setHeader("toEmail", oldExchange.getMessage().getBody(Map.class).get("email"));
    return newExchange;
})
.....

Java Version: 11
Camel Version: 3.18.1
Karavan Version: 3.18.2
Run command: jbang -Dcamel.jbang.version=3.18.1 camel@apache/camel run testflow.yaml
Please help.

To include java classes from other files in folder, run the following command
jbang -Dcamel.jbang.version=3.18.2 camel@apache/camel run *

Working example of aggregator https://github.com/apache/camel-karavan/tree/main/karavan-demo/aggregator

It's working nicely. Thanks. @mgubaidullin


Refactoring Ping()

Currently the ping function pings a static address of "<IP_ADDRESS>" and it needs to be refactored to do the following:

Ping the internal address selected
Ping the external address selected
Pnig the external url selected

Implemented in version 0.0.12


0.12

Updated the code to work with Bevy 0.12.
Will be testing it, just FYI.

Thanks! should I merge it already or wait for you to test it?

Well, I dd test it in my project and... all the meshes disappeared for me,
so it might need some more work...
Den sön 26 nov. 2023 kl 16:44 skrev Jan Hohenheim @.***

:

Thanks! should I merge it already or wait for you to test it?
—
Reply to this email directly, view it on GitHub
https://github.com/janhohenheim/pixelate_mesh/pull/8#issuecomment-1826817402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADJQZR42YSMPGAUE3BRJZTYGNPWNAVCNFSM6AAAAAA7YZJJD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWHAYTONBQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Mvh,
Tommie Nygren
0704-38 27 81
Bealo AB

Is it better now with 12.1?

I did actually try it out with 12.1 and it worked fine - however, what
about shadows? Haven't checked the examples for that, but the shadows seem
to be unaffected by the pixelation effect...
I'll see if I have done any other changes and then update my PR if you want
to give it a try...
Den sön 7 jan. 2024 kl 07:02 skrev Jan Hohenheim @.***>:

Is it better now with 12.1?
—
Reply to this email directly, view it on GitHub
https://github.com/janhohenheim/pixelate_mesh/pull/8#issuecomment-1879965593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADJQZWF6JNSPAT2YVWTMQTYNI25TAVCNFSM6AAAAAA7YZJJD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZHE3DKNJZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Mvh,
Tommie Nygren
0704-38 27 81
Bealo AB

Yep, I never managed to get the shadows working, hehe.
Thanks!


Integration PR for #71

Serves as my testbed for #71, including travis stuff.

Works locally, merge once travis turns green.

All platforms except Linux pass CI. The error on Linux is:
mdbook: command not found


separate combo pipelines into build & deploy pipelines

existing pipelines contain both build & deploy components, this change separate them into individual pipelines

Benefits:

a more accurate usage of AzDO built-in deploy success metrics (mix of build&deploy means we were incorrectly counting failed PR builds)
easier to find deploys without the mix of pr builds
showing accurate pipeline badge status in markdown

LGTM - I am just curious about the pipeline trigger order to ensure the deploy pipeline would always pick the latest successful build after the code had been merged. Not sure if this would help or not, but plz check it out if you have time https://docs.microsoft.com/en-us/azure/devops/pipelines/build/triggers?view=azure-devops&tabs=yaml#pipeline-triggers

@alkofahi
not sure what you're suggesting here.  There are already trigger in place.

What do you mean by "latest successful build"?

The deploy pipeline always rebuild the artifacts for deploy.  The "build" pipeline (PR trigger) is for running tests, we do not publish any artifact from these builds to the deploy pipeline


doc: remove mention of X11 license variant

The Nixpkgs license is pure MIT.
Cf. https://en.wikipedia.org/wiki/MIT_License#X11_License
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@FRidh it seems you introduced the "permissive MIT/X11 license" formulation in 92179785b60d9fd9e67ac5301e78b892b1b7a0c7, but the X11 clause is not present in the license (it had a different addition, but that doesn't seem related to the X11 clause). Was this an oversight, and do you agree with removing reference to X11?
/cc @alyssais as per @roberth's suggestion

Also /cc @edolstra as the one who added the COPYING file way back in 2006: besides the notice that was removed in this commit, the intention was for Nixpkgs to be pure-MIT, no X11 variant, right?

@FRidh it seems you introduced the "permissive MIT/X11 license" formulation in 9217978, but the X11 clause is not present in the license (it had a different addition, but that doesn't seem related to the X11 clause). Was this an oversight, and do you agree with removing reference to X11?
/cc @alyssais as per @roberth's suggestion

Probably, and I suppose yes.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2023-09-14-documentation-team-meeting-notes-80/33033/1


Dropped * spans because maxQueueSize reached.

Hello! We are seeing this error message from our otel-instrumented app with varying span amounts roughly every 5 seconds.

Dropped 2228 spans because maxQueueSize reached

I've tried increasing the maxQueueSize to 6144 (the default is 2048).
const sdk = new NodeSDK({
  autoDetectResources: true,
  traceExporter: exporter,
  spanProcessor: new BatchSpanProcessor(exporter, {
    maxQueueSize: 6144,
  }),

but this did not seem to affect the behavior at all. Is someone able to recommend a way to look into diagnosing/fixing this issue?

[ ] This only affects the JavaScript OpenTelemetry library
[x] This may affect other libraries, but I would like to get opinions here first

Hi @jholm117, thanks for reaching out.
I'm closing this as a duplicate of #3094. The BatchSpanProcessor does not eager-export at the moment, which means that if the queue size is reached, it will not attempt to export but actually drop the spans. There are several PRs open for #3094, so you can expect this to be addressed in an upcoming release.

Hello, unfortunately I'm still seeing this on version 1.26.0 of @opentelemetry/sdk-trace-base. Any ideas?


Refresh websecurity client

This does remove the path formatting methods. Those methods are soon-to-be deprecated, and their functionality is already covered by the better generated resource name classes. This is a non-GA client.

PTAL


fix: grep command error with unknown args

https://github.com/asinkxcoswt/shell-script-template/issues/1

Hi, thank you for the PR, but I have tested your change and still find the issue. It seems removing the quote doesn't help (I am testing on macOs). So let's try using the double dash (--) instead.

Yes, you are right 👍  I  just made a mistake


Couldn't run this project

Hi, can anyone please help me to run it on my computer maybe provide with a detailed readme.

Hi,
I have the same issue, I am trying to run the project but it fails with the following logs
/De/groceries-shopping-flutter-app master !1 > flutter run         10:16:33 PM
Running "flutter pub get" in groceries-shopping-flutter-app...         0.5s
Launching lib/main.dart on GM1901 in debug mode...
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:67:23: Error: Type 'ActionFactory' not found.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:96:21: Error: Type 'ActionFactory' not found.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:152:21: Error: Type 'ActionFactory' not found.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:219:23: Error: Type 'ActionFactory' not found.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:88:26: Error: Type 'DiagnosticableMixin' not found.
class PictureStream with DiagnosticableMixin {
                         ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:192:44: Error: Type 'DiagnosticableMixin' not found.
abstract class PictureStreamCompleter with DiagnosticableMixin {
                                           ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:88:7: Error: The type 'DiagnosticableMixin' can't be mixed in.
class PictureStream with DiagnosticableMixin {
      ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:192:16: Error: The type 'DiagnosticableMixin' can't be mixed in.
abstract class PictureStreamCompleter with DiagnosticableMixin {
               ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:67:23: Error: 'ActionFactory' isn't a type.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:96:21: Error: 'ActionFactory' isn't a type.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:152:21: Error: 'ActionFactory' isn't a type.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:219:23: Error: 'ActionFactory' isn't a type.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:312:30: Error: The argument type 'Map<LocalKey, invalid-type>' can't be assigned to the parameter type 'Map<Type, Action<Intent>>'.
 - 'Map' is from 'dart:core'.
 - 'LocalKey' is from 'package:flutter/src/foundation/key.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/foundation/key.dart').
 - 'Type' is from 'dart:core'.
 - 'Action' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
 - 'Intent' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
      actions: data?.actions ?? actions,
                             ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:363:30: Error: The argument type 'Map<LocalKey, invalid-type>' can't be assigned to the parameter type 'Map<Type, Action<Intent>>'.
 - 'Map' is from 'dart:core'.
 - 'LocalKey' is from 'package:flutter/src/foundation/key.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/foundation/key.dart').
 - 'Type' is from 'dart:core'.
 - 'Action' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
 - 'Intent' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
      actions: data?.actions ?? actions,
                             ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:167:11: Error: Superclass has no method named 'debugFillProperties'.
    super.debugFillProperties(properties);
          ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:171:30: Error: The method 'toStringShort' isn't defined for the class 'PictureStreamCompleter'.
 - 'PictureStreamCompleter' is from 'package:flutter_svg/src/picture_stream.dart' ('../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart').
Try correcting the name to the name of an existing method, or defining a method named 'toStringShort'.
      ifPresent: _completer?.toStringShort(),
                             ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:266:11: Error: Superclass has no method named 'debugFillProperties'.
    super.debugFillProperties(description);
          ^^^^^^^^^^^^^^^^^^^

FAILURE: Build failed with an exception.

* Where:
Script '/home/hackerx/snap/flutter/common/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 904

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/home/hackerx/snap/flutter/common/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 12s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        12.7s
Exception: Gradle task assembleDebug failed with exit code 1

@AhmedAbouelkher

Hi,
I have the same issue, I am trying to run the project but it fails with the following logs
/De/groceries-shopping-flutter-app master !1 > flutter run         10:16:33 PM
Running "flutter pub get" in groceries-shopping-flutter-app...         0.5s
Launching lib/main.dart on GM1901 in debug mode...
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:67:23: Error: Type 'ActionFactory' not found.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:96:21: Error: Type 'ActionFactory' not found.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:152:21: Error: Type 'ActionFactory' not found.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:219:23: Error: Type 'ActionFactory' not found.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:88:26: Error: Type 'DiagnosticableMixin' not found.
class PictureStream with DiagnosticableMixin {
                         ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:192:44: Error: Type 'DiagnosticableMixin' not found.
abstract class PictureStreamCompleter with DiagnosticableMixin {
                                           ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:88:7: Error: The type 'DiagnosticableMixin' can't be mixed in.
class PictureStream with DiagnosticableMixin {
      ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:192:16: Error: The type 'DiagnosticableMixin' can't be mixed in.
abstract class PictureStreamCompleter with DiagnosticableMixin {
               ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:67:23: Error: 'ActionFactory' isn't a type.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:96:21: Error: 'ActionFactory' isn't a type.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:152:21: Error: 'ActionFactory' isn't a type.
      Map<LocalKey, ActionFactory> actions,
                    ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:219:23: Error: 'ActionFactory' isn't a type.
  final Map<LocalKey, ActionFactory> actions;
                      ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:312:30: Error: The argument type 'Map<LocalKey, invalid-type>' can't be assigned to the parameter type 'Map<Type, Action<Intent>>'.
 - 'Map' is from 'dart:core'.
 - 'LocalKey' is from 'package:flutter/src/foundation/key.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/foundation/key.dart').
 - 'Type' is from 'dart:core'.
 - 'Action' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
 - 'Intent' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
      actions: data?.actions ?? actions,
                             ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_platform_widgets-0.51.0/lib/src/platform_app.dart:363:30: Error: The argument type 'Map<LocalKey, invalid-type>' can't be assigned to the parameter type 'Map<Type, Action<Intent>>'.
 - 'Map' is from 'dart:core'.
 - 'LocalKey' is from 'package:flutter/src/foundation/key.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/foundation/key.dart').
 - 'Type' is from 'dart:core'.
 - 'Action' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
 - 'Intent' is from 'package:flutter/src/widgets/actions.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/widgets/actions.dart').
      actions: data?.actions ?? actions,
                             ^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:167:11: Error: Superclass has no method named 'debugFillProperties'.
    super.debugFillProperties(properties);
          ^^^^^^^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:171:30: Error: The method 'toStringShort' isn't defined for the class 'PictureStreamCompleter'.
 - 'PictureStreamCompleter' is from 'package:flutter_svg/src/picture_stream.dart' ('../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart').
Try correcting the name to the name of an existing method, or defining a method named 'toStringShort'.
      ifPresent: _completer?.toStringShort(),
                             ^^^^^^^^^^^^^
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.17.4/lib/src/picture_stream.dart:266:11: Error: Superclass has no method named 'debugFillProperties'.
    super.debugFillProperties(description);
          ^^^^^^^^^^^^^^^^^^^

FAILURE: Build failed with an exception.

* Where:
Script '/home/hackerx/snap/flutter/common/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 904

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/home/hackerx/snap/flutter/common/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 12s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        12.7s
Exception: Gradle task assembleDebug failed with exit code 1

@AhmedAbouelkher

Hi,
you should change the flutter_platform_widgets package version, I will do that and push a fix.
Sorry for answering so late  @rvsingh011 @winstonjaysa

Hi,
you should change the flutter_platform_widgets package version, I will do that and push a fix.
Sorry for answering so late  @rvsingh011 @winstonjaysa

Hi @AhmedAbouelkher,
Thanks for a very quick reply. I am still getting the error. Although your release apk workes fine.
Flutter --version output
~/Desktop/groceries-shopping-flutter-app master > flutter --version                                                10:22:29 PM
Flutter 1.22.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision<PHONE_NUMBER> (5 weeks ago) • 2020-12-10 11:54:40 -0800
Engine • revision ae90085a84
Tools • Dart 2.10.4

Logs after running flutter run
~/Desktop/groceries-shopping-flutter-app master > flutter run                                                      10:21:02 PM
Launching lib/main.dart on GM1901 in debug mode...
lib/main.dart:5:8: Error: Error when reading '../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/response-0.1.1/lib/Response.dart': No such file or directory
import 'package:response/Response.dart';
       ^
lib/screens/home.dart:8:8: Error: Error when reading '../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/response-0.1.1/lib/Response.dart': No such file or directory
import 'package:response/Response.dart';
       ^
lib/screens/home.dart:11:16: Error: Method not found: 'ResponseUI'.
var response = ResponseUI();
               ^^^^^^^^^^
lib/main.dart:18:14: Error: The method 'Response' isn't defined for the class 'MyApp'.
 - 'MyApp' is from 'package:groceries_shopping_app/main.dart' ('lib/main.dart').
Try correcting the name to the name of an existing method, or defining a method named 'Response'.
      child: Response(
             ^^^^^^^^

FAILURE: Build failed with an exception.

* Where:
Script '/home/hackerx/snap/flutter/common/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 904

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/home/hackerx/snap/flutter/common/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 10s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        11.2s
Exception: Gradle task assembleDebug failed with exit code 1

Hi @AhmedAbouelkher,
Thanks for a very quick reply. I am still getting the error. Although your release apk workes fine.
Flutter --version output
~/Desktop/groceries-shopping-flutter-app master > flutter --version                                                10:22:29 PM
Flutter 1.22.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision<PHONE_NUMBER> (5 weeks ago) • 2020-12-10 11:54:40 -0800
Engine • revision ae90085a84
Tools • Dart 2.10.4

Logs after running flutter run
~/Desktop/groceries-shopping-flutter-app master > flutter run                                                      10:21:02 PM
Launching lib/main.dart on GM1901 in debug mode...
lib/main.dart:5:8: Error: Error when reading '../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/response-0.1.1/lib/Response.dart': No such file or directory
import 'package:response/Response.dart';
       ^
lib/screens/home.dart:8:8: Error: Error when reading '../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/response-0.1.1/lib/Response.dart': No such file or directory
import 'package:response/Response.dart';
       ^
lib/screens/home.dart:11:16: Error: Method not found: 'ResponseUI'.
var response = ResponseUI();
               ^^^^^^^^^^
lib/main.dart:18:14: Error: The method 'Response' isn't defined for the class 'MyApp'.
 - 'MyApp' is from 'package:groceries_shopping_app/main.dart' ('lib/main.dart').
Try correcting the name to the name of an existing method, or defining a method named 'Response'.
      child: Response(
             ^^^^^^^^

FAILURE: Build failed with an exception.

* Where:
Script '/home/hackerx/snap/flutter/common/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 904

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/home/hackerx/snap/flutter/common/flutter/bin/flutter'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 10s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                        11.2s
Exception: Gradle task assembleDebug failed with exit code 1


Prepare for 0.10.0 release.

Coverage remained the same at 94.038% when pulling b799fb6aa85a887573c23d1a0c69a7be65aaf306 on 0.10.0 into 341e9e30df257a2f1919093b1bc64994ae78359d on master.

Coverage remained the same at 94.038% when pulling aa94a76c1cd0cfdbfc1afac9cb3a6b303b287f40 on 0.10.0 into 341e9e30df257a2f1919093b1bc64994ae78359d on master.

#109 is listed as a bug fix in the changelog which may lead people to believe that it is semver-patch. I don't think it is semver-patch, correct?

It is a bug fix, but I believe the nature of the change could be breaking for people who were coding around the problem before, so I bumped the minor version, which indicates a breaking change since we're pre-1.0.0.
I rephrased that heading to Breaking bug fixes

Coverage remained the same at 94.038% when pulling 32edf31bb63a086e06744243358ba8f83751bb40 on 0.10.0 into 341e9e30df257a2f1919093b1bc64994ae78359d on master.

Coverage remained the same at 94.038% when pulling ae1c1d681e6bff526fa8850dd368407ea420375b on 0.10.0 into 341e9e30df257a2f1919093b1bc64994ae78359d on master.

Coverage remained the same at 94.038% when pulling 0b6874cf7db67956118a5ae430ccd40cb18492c4 on 0.10.0 into 341e9e30df257a2f1919093b1bc64994ae78359d on master.

LGTM.