Install fails

Hi, I have been trying to install the tool (tested 2 devices, 2 browsers on each - Edge and Chrome) but it always fails.
Error is always the same in the log file.

Activation of https://spmtreleasescus.blob.core.windows.net/build-firstrelease/build-firstrelease.application resulted in exception. Following failure messages were detected:
+ Downloading http://spmtreleasescus.blob.core.windows.net/build-firstrelease/migrationtoolinstall/libcef.dll.deploy did not succeed.
+ Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host.
+ An existing connection was forcibly closed by the remote host

P.S. Even though I did not select 'Install first release', the URL seems to point to the first release build.

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

ID: 193d7cc4-4270-f69e-262b-6384a273f835
Version Independent ID: 6625a0f7-e4d0-2352-d78a-0a82273b1f07
Content: Introducing the SharePoint Migration Tool
Content Source: migration/introducing-the-sharepoint-migration-tool.md
Product: sharepoint-server-itpro
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Hi Steve:  Investigating this now.

Hi Joanne, seems like 'Abenold' is experiencing something similar. Same dll is showing in his log.

@JoanneHendrickson any updates on this? Thanks!

@JoanneHendrickson Any updated on this, still not able to install the software.

Please see https://docs.microsoft.com/en-us/sharepointmigration/spmt-install-issues for help -- specifically this section:
Download the tool instead
If you still are experiencing unexplained errors, try downloading the SPMT tool to your local computer.

Download zip package to your local computer: Download SPMT.zip package.
Right-click the zip file, and click properties.
On the General tab, select Unblock and then click OK.
Open the zip file and extract all files.
Run the build.application file.

We have found downloading the tool first resolves the issue for people experiencing this. Hope this helps.


Re-enable code analysis and fix some CppCoreGuidelines checks

The /analyze flag had been disabled for a while because of a bug in VS, but that's been fixed so we can re-enable it now.
This also brings in the new CppCoreGuidelines checker. We're fairly conformant on the default rule set, I only had to disable ES.103 Don't Overflow because we trip that everywhere with our float->double promotions.
I made some fixes to get us conformant on:

Enum.3: Prefer class enums over "plain" enums
ES.84: Don't try to declare a local variable with no name, which also fires when you ignore a non-trivial return value.
C.66 Make move operations noexcept, and others -- make constructors noexcept.
Don't std::move when you don't need to
Always initialize fields

Once we establish this baseline then we can look to enable some more of the Cpp Core Guideline checks.

I don't see the build fail with out-of-heap-space locally but the pipeline is. I filed this bug report to see if we can get a workaround from VS.

Closing until we get traction with the /analyze issues.

Hi all! I've re-opened this PR. I've enabled a couple more CppCoreGuidelines checks and also turned on code analysis by default in Debug. From what I see, the code analysis doesn't add that much extra time to the build so I wanted to reduce frustration by having the checks essentially only happen during PR builds.
Because this still causes out-of-memory issues on the Azure Pipelines builds, I had to change the Debug build configuration to disable multiproc builds to work around it. It's not great but Debug builds aren't the long pole so this doesn't increase the overall build times.

Ok actually I found that this does slow down the local build substantially so I changed it to only run locally in intellisense builds. This seems to give the best balance of impact but also catching the issues early in the authoring process.

The acrylic test failures look like a test app crash, at least the screen shots don't have a running MUX controls test app.

The acrylic test failures look like a test app crash, at least the screen shots don't have a running MUX controls test app.

There is an Acrylic test that tries to hide the window and then re-show it. Sometimes the re-show fails and then the rest of the acrylic tests all fail. This has been happening a lot and we need to fix the test. :(

I believe this is ready for review now. Can someone take a look? Thanks!

Good news! With the latest C++/WinRT's performance improvements it looks like code analysis doesn't run out of memory anymore so I'm removing the part where MultiProc is disabled and the run time of the Debug leg is back to about where it was before.

Good news! With the latest C++/WinRT's performance improvements it looks like code analysis doesn't run out of memory anymore so I'm removing the part where MultiProc is disabled and the run time of the Debug leg is back to about where it was before.

Oh awesome. That's great news.

sweet !


automatic updates

On application lunch, system checks if app version is same with latest release on github,
If app is not up to date, then notification is displayed asking customer to Automatically Update, manually download and update or Skip Version
Changelog should be displayed in the notification
During automatic updates,

system downloads and unzip latest release from github then,
backup old files and database.
fetch github files changed list to overwrite the old files (mainly systems/ ../controllers/ ../languages )

Add check update automatically
Display changelog
Improve tastyigntier download update API


Apply (relaxed) checkstyle rules to test and fix existing violations …

Similar to lang.

Now complete. If I knew before how much work this would be, I would not have started.

Coverage remained the same at 98.238% when pulling ef79bfbd6a8198a45028676b970109287ad24b79 on PascalSchumacher:checkstyle_for_tests into 8f7d0494d19a54b1606d0d7779ff4754c3d66b23 on apache:master.

Coverage remained the same at 98.298% when pulling 1c8470a8361d9dfb17a0ec2b3836f583e896de71 on PascalSchumacher:checkstyle_for_tests into a200b34e20a25478332fd3dc2a3553654e5c286a on apache:master.

But thanks for working on this :-) :clap:

Agreed.


Travis CI jobs failing intermittently

test_batch.test_submit_job fails intermittently across all Python versions and regardless of whether TEST_SERVER_MODE=false or TEST_SERVER_MODE=true.

Nose Failure

======================================================================
FAIL: test_batch.test_submit_job
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/home/travis/build/spulec/moto/moto/core/models.py", line 99, in wrapper
    result = func(*args, **kwargs)
  File "/home/travis/build/spulec/moto/moto/core/models.py", line 99, in wrapper
    result = func(*args, **kwargs)
  File "/home/travis/build/spulec/moto/moto/core/models.py", line 99, in wrapper
    result = func(*args, **kwargs)
  [Previous line repeated 2 more times]
  File "/home/travis/build/spulec/moto/tests/test_batch/test_batch.py", line 750, in test_submit_job
    [event["message"] for event in resp["events"]].should.equal(["hello"])
  File "/home/travis/virtualenv/python3.7.6/lib/python3.7/site-packages/sure/__init__.py", line 387, in wrapper
    raise AssertionError(e)
AssertionError: given
X = []
    and
Y = ['hello']
Y has 1 items whereas X has only 0
>>  raise AssertionError(e)

The test is still failing occasionally after the recent switch from nose to pytest.

        It primarily assures that the compared objects are absolute equal ``==``.

        :param what: the expected value
        :param epsilon: a delta to leverage upper-bound floating point permissiveness
        """

        try:
            comparison = DeepComparison(self.obj, what, epsilon).compare()
            error = False
        except AssertionError as e:
            error = e
            comparison = None

        if isinstance(comparison, DeepExplanation):
            error = comparison.get_assertion(self.obj, what)

        if self.negative:
            if error:
                return True

            msg = '%s should differ from %s, but is the same thing'
            raise AssertionError(msg % (safe_repr(self.obj), safe_repr(what)))

        else:
            if not error:
                return True
>           raise error
E           AssertionError: given
E           X = []
E               and
E           Y = ['hello']
E           Y has 1 items whereas X has only 0

../../../virtualenv/python3.6.10/lib/python3.6/site-packages/sure/__init__.py:673: AssertionError

This "flipper test" was mentioned and acknowledged as a problem in #3161, but I wasn't able to find an existing issue tracking it.  I will investigate.

Hey @bpandola, just as an FYI: this was my attempt at fixing it: https://github.com/spulec/moto/pull/3146
That obviously wasn't the solution, hopefully you have better luck/investigator skills ;)


How to add .marko files

Hi!
I'm trying use intellisense in a .marko file, trying add it  as  a 'html-css-class-completion.HTMLLanguages' but it doesn't works
 "git.autofetch": true,     "explorer.compactFolders": false,     "cSpell.enableFiletypes": [         "marko"     ],     "window.zoomLevel": 1,     "html-css-class-completion.includeGlobPattern": "**/*.{css,html,marko}" }
Can you help me?
Thanks!

It doesn't seem to work for me either. Tried adding Jinja, no matter how I change the setting, it doesn't work :/


"Finder" should be capitalized

Small nitpick but "finder" is not correct. It should be "Finder":
https://github.com/NomarCub/obsidian-copy-url-in-preview/blob/5cf5211f695d438d6a3b7328eb1b2a3e370780de/src/main.ts#L305

I'll make a new release with this fix soon, cheers!


False positives on no-internal-module

/// <reference path="../../../../../typings/tsd.d.ts" />

module MyModule {}

export default MyModule;

is triggering no-internal-module.

The intent of the no-internal-module rule, if I understand right, is simply about favoring the namespace keyword instead of the module keyword for internal modules. So if you change your code to read namespace MyModule {} the error will go away.

I'm exporting the module, and importing it in the style of external modules from other files.

Can this be re-opened until the issue is clarified?

We're probably not going to make a special case for this. Is there a reason you're using modules/namespaces inside an external module? one of the TypeScript devs recommends against this unless you are exporting many things from your external module.
Either way, you can change module to namespace and everything will work identically, except the error will go away.

@jiaweihli do you still want to keep this open? namespace is a direct replacement for module, and the spirit of the no-internal-module rule is to encourage usage of the newer, more semantically correct keyword. Perhaps it should've been called no-module-keyword, but I don't see much benefit in changing that now.

Modules will continue to exist for the purpose of external modules: https://github.com/Microsoft/TypeScript-Handbook/blob/master/pages/Namespaces and Modules.md#a-note-about-terminology
namespace only replaces internal modules, but not external ones.
Re: http://stackoverflow.com/a/30357635/759714, my use case is the last one - I export several things, and I want to group them under an umbrella name to prevent top-level conflicts with other imports.
I don't have much code that conflicts with this, so I could probably refactor it into an abstract class with static methods (though that makes less sense).  Renaming the keyword would make things much clearer, though I understand that probably won't happen until v3.0 at least.

I have this warning with definition files (.d.ts) I think this files should be excluded from this inspection.
Basically I follow the same conventions of DefinitelyTyped

Upon reconsideration, I don't think no-module-keyword is the right name either, because the rule permits external modules with that keyword (export module "tslint" { ... }). I won't be renaming this for 3.0.


[Feature]: data source for Directus / NocoDB / Grist

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
To build an integration of Directus / NocoDB / Grist with their OpenAPI Specification
https://docs.directus.io/reference/system/server/#get-openapi-specification
Why should this be worked on?
To make it smoother like on n8n when we need to bind the datasource with the rest api .
And also I was browsing in a subreddit "selfhosted" and I saw Budibase asking to build data source for Grist too.

I agree that these integrations would make a difference among these tools in terms of productivity and ux. See tooljet with appwrite for instance.


Homepage tweaks to support the global banner

What
Make changes to the homepage to improve it now that the global banner is in place. Trello
Why

The header bar looks too deep as the blue border on the global banner butts up against it.
There's not enough space between the bottom of the global banner and the GOV.UK main title text.

BEFORE/AFTER

I've reviewed the design. Looks good on multiple viewport sizes 👍🏽


Add **kwargs to RedisChatMessageHistory

Add **kwargs argument to class RedisChatMessageHistory. I have used this in my own project so I can pass a password along with the url to connect to Redis.
Also added add a ping function in the RedisChatMessageHistory class. I have used this in my own project so I can check if I have successfully authenticated, before doing anything.

Slightly tangential / broader in scope than this issue, but what if we pass redis client in directly in init? as syntactic sugar we can add a from_url classmethod that constructs client for you (and that can take in kwargs).

Something like my recent commit? Also I was looking around and I noticed in vectorstores/redis.py in the Redis(VectorStore) class there is **kwargs passed into the redis.from_url in init, like my first commit.


[Feature Suggestion] Enhance Telegram function with sendphoto

Hi @kensoh ,
Was working on building a monitoring platform using the library and wanted to send myself telegram notifications based on the RPA automated task.
I think it would be a useful enhancement (or secondary addition) to the telegram method to allow it to send images using the telegram api. Eg. the sendPhoto method

Not sure if there are costing or privacy issues associated with your instance to send the image- so feel free to close this if it's not feasible from a cost, privacy or other perspective

Thanks @ignatiustanjh for raising this! It is exactly like you said, my VPS has a limited data bandwidth every month, so I could risk causing issues for other users using normal text Telegram messages, if I enable the photo sending function.
However, you can use the bin() function to auto-upload your image to a temporary URL, then send that URL as part of your Telegram message - https://github.com/tebelorg/RPA-Python?tab=readme-ov-file#secure-temporary-storage
Closing the issue for now but I will look out for your reply!


Beam Sync: pivot to latest block if the lag gets too high

What is wrong?
If Beam Sync lags behind far enough, then it stops being able to sync. Sync fails because peers no longer have the necessary trie nodes.
How can it be fixed
Unfortunately it's difficult to strictly define that this scenario has occurred, because peers are permitted to return empty results when requesting trie nodes. It could be a peer that is overloaded, or just recently started syncing, or that the local peer has fallen too far behind.
Likely, we'll need to measure the lag from the tip, or the time lag, and use that to decide that we have fallen too far behind. At that point, we would simply skip over importing the remaining blocks in between the last imported block and the tip, and re-start beam sync from the tip again. This should all happen without intervention.

I'm not certain this belongs in the MVP, just adding it for visibility, and maybe we will remove it again.

I think we could get reasonably reliable results if we define the need to pivot as:

no forward progress of syncing state after some timeout

all peers fail at returning needed nodes
maybe confirm by checking the actual block state-root node is also not available

state root of newer blocks is available

at least one peer does return non-empty state requests for "new blocks"

This allows us to a) detect the stall condition which I think can be done semi-reliably now and b) confirm that peers will return data for newer blocks (as defined as them being willing to return the node data for a fresh state root.

(this is probably more complex than is absolutely necessary but I want to document it).
If we define an integer BEAM_WINDOW which is the number of blocks from the known chain head, we can calculate the probability that we will fail to import or preview a given block.  It seems like this might be a good metric to determine if we should pivot.  This would allow us to pivot before starting a preview or import rather than timing out in the middle which might be uglier.

BLOCK_TIME is the average block time measured over recent history.
BLOCK_STDEV is the standard deviation of the block time over recent history
IMPORT_AVG is the average time it takes to import a block
IMPORT_STDEV is the standard deviation
PREV_AVG is the average time it takes to import a block
PREV_STDEV is the standard deviation.

We can compute an upper bound on block times with:

MAX_EXPECTED_BLOCK_TIME = BLOCK_TIME + 3 * BLOCK_STDEV

We can determine an upper bound on import or preview time via with 99.9% likelyhood as:

MAX_EXPECTED_IMPORT_TIME_IN_BLOCKS = (IMPORT_AVG + 3 * IMPORT_STDEV) / MAX_EXPECTED_BLOCK_TIME for import
MAX_EXPECTED_PREV_TIME_IN_BLOCKS = (PREV_AVG + 3 * PREV_STDEV) / MAX_EXPECTED_BLOCK_TIME for preview

Compute PIVOT_POINT which is the observed chain head minus BEAM_WINDOW.
For a given block number that we are considering previewing or importing, we define BLOCKS_FROM_PIVOT as the number of blocks between the block number and the PIVOT_POINT.
According to these numbers, we can know with 99.9% probability that a preview or import block will not fall outside the BEAM_WINDOW if we only choose to import it under the case that.

BLOCK_NUMBER - MAX_EXPECTED_IMPORT_TIME_IN_BLOCKS > PIVOT_POINT
BLOCK_NUMBER - MAX_EXPECTED_PREV_TIME_IN_BLOCKS > PIVOT_POINT

In the case where this condition is not satisfied we should instead choose to pivot our beam sync.

I think we could get reasonably reliable results if we define the need to pivot as:

no forward progress of syncing state after some timeout

all peers fail at returning needed nodes
maybe confirm by checking the actual block state-root node is also not available

state root of newer blocks is available

at least one peer does return non-empty state requests for "new blocks"

Interestingly, a couple of times, I seem to have connected to some kind of archive node. It will keep feeding me very old state (slowly). So we can't count on 0 results being a reliable trigger. Checking the new state root is interesting, but if we're already stuck behind, seems like we don't lose much by pivoting without checking, which will simplify.
I think we might need to just look at the timestamp diff between the active beam import header and the tip. The downside is that we won't be able to quickly adapt, if the window of trie history shrinks/grows.
Maybe we could expose some kind of command line variable like --allowed-beam-lag 600 to permit 10 minutes of lag from the tip. Then if other clients change, we could communicate a new flag without requiring a new release.

I think we might need to just look at the timestamp diff between the active beam import header and the tip.

Went with block number instead of timestamp. The main reason is that other clients define the window by block number, so if the timestamp gets screwed up (or advances at a different rate, on say a testnet), then the pivot window is still the right size.


blazed.quest

Blazed Quest - Project Management platform for the Blazed One Solution.
Requirements:

Use Firebase to store/load current active Projects.
Allow users to easily reach the contact form (on blazed.contact).
Allow the user to reach each organizational GitHub (@blazed-labs, @blazed-space, @blazed-xyz, @ruff-org).
Briefly explain Blazed Labs' development rationale.
Display all active projects' names, images, url, repo, and latest release version.
Display "get involved" CTA; including: contribution guidelines, code of conduct, link to Blazed City.

Tasks for Blazed Quest

Integration Tasks

[ ] Integrate Firebase

Development Tasks

[ ] Develop Basic Prototype
[ ] Create About Page
[ ] Create Projects Page
[ ] Create Get Involved Page

UX Testing Tasks

[ ] Alpha Testing
[ ] Beta Testing
[ ] Stage Testing

Deployment Tasks

[ ] Deploy Production Project


build: add Cypress linter via ESLint

#662  👈
main

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

Join @davidlj95 and the rest of your teammates on  Graphite


AccessRules return

I'm writing a AccessRules manager for our application. When creating an item, it just returns true instead of the ID for storage.
Would it be possible to get the id on return instead of having to make a second API to call to search for it.

I'm writing a AccessRules manager for our application. When creating an item, it just returns true instead of the ID for storage.
Would it be possible to get the id on return instead of having to make a second API to call to search for it.

Hello. Did you manage to figure it out?
I need something similar, but for the DNS record creation ID.
Thank you in advance.


Modify dependencies info to include the full dependency path instead of just the version

Modify the main build.gradle and related build files to reference the full dependency data instead of just the version. This change attempts to make our dependency info more friendly with dependabot, which is currently unable to parse through our dependencies.
For #2602

Not sure why the travis builds aren't coming through? I'll re-trigger them


In TextContentHelper: better constants names; used constants where applicable

LF -> NEWLINE
CR -> CARRIAGE_RETURN
Use NEWLINE in joinLines() method.

ok with me


[dashboard v2] tests!

This PR adds a ton of tests for the new dashboard app. I have a few more todo (enumerated below), but wanted to make a PR to start getting an idea of coverage.
@graceguo-supercat
TODO

[x] fix old Dashboard tests
[x]  add reducer tests
[x] add action tests

ping @graceguo-supercat

javascript tests are passing 🎉 we might need to rebase the dashboard-builder branch on master to fix the py36-sqlite + py36-postgres tests

I only see filters related tests for componentDidUpdate. We should also test dashboard's default_filters are pre-populated.

abandoning because of a weird git state. see #5066


Idg/logging sync

Checking to see if merging master into this branch still builds

Closing because this PR was just to check the the build works before offering it back to @alastairs

Closing because this PR was just to check the the build works before offering it back to @alastairs


Linux issue:     collect2: error: ld returned 1 exit status

When I tried to use this package for linux, it always show the error.
    collect2: error: ld returned 1 exit status
How can I fix this issue?
#linux #collect2

Share output of flutter run --verbose.

@alexmercerind  Same issue,
I'm new to this package,  i was using the example project (media_kit_test) and i was getting error a Cmake error due to miss installation of libmpv-dev. Then after successfullyu installation using sudo apt install libmpv-dev i faced this error:     collect2: error: ld returned 1 exit status
This is flutter run --verbose:

Hi @kekko7072 & @oniangel12933,
Most likely you guys are using snap installation of Flutter. Please use the manual installation:
https://docs.flutter.dev/get-started/install/linux#method-2-manual-installation

Unlike other platforms, media_kit on GNU/Linux uses user installation of libmpv (how most GNU/Linux distributions work; dependent on packages from distribution). On other platforms, you don't need any additional steps other than editing pubspec.yaml like usual.

@alexmercerind  i reinstalled using manual installation but i'm still facing issue.

Try to do a flutter clean & check if flutter doctor reports any issues. I don't see anything media_kit specific in the logs.
Thanks!

I run 'flutter clean' and 'flutter doctor' with no problems.
flutter run --verbose
`
[  +39 ms] executing: uname -m
[   +4 ms] Exit code 0 from: uname -m
[        ] x86_64
[  +95 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required,
skipping update.
[   +3 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping
update.
[        ] Artifact Instance of 'LegacyCanvasKitRemover' is not required,
skipping update.
[   +7 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required,
skipping update.
[  +69 ms] executing: /home/francesco/Android/Sdk/platform-tools/adb devices -l
[  +82 ms] List of devices attached
[   +7 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping
update.
[   +8 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping
update.
[   +1 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required,
skipping update.
[ +113 ms] Connected devices:
[  +58 ms] Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS
6.2.0-35-generic
[   +2 ms] Chrome (web)    • chrome • web-javascript • Google Chrome
118.0.5993.88
[   +1 ms] [1]: Linux (linux)
[        ] [2]: Chrome (chrome)
[   +1 ms] Please choose one (or "q" to quit)
[        ] :
[+1585 ms] 1
[  +25 ms] Skipping pub get: version match.
[  +84 ms] Found plugin file_picker at
/home/francesco/.pub-cache/hosted/pub.dev/file_picker-5.5.0/
[  +12 ms] Found plugin flutter_plugin_android_lifecycle at
/home/francesco/.pub-cache/hosted/pub.dev/flutter_plugin_android_lifecycle-2.0.1
6/
[  +21 ms] Found plugin package_info_plus at
/home/francesco/.pub-cache/hosted/pub.dev/package_info_plus-4.2.0/
[  +15 ms] Found plugin path_provider at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider-2.1.1/
[   +6 ms] Found plugin path_provider_android at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_android-2.2.0/
[   +5 ms] Found plugin path_provider_foundation at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_foundation-2.3.1/
[   +4 ms] Found plugin path_provider_linux at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_linux-2.2.1/
[   +5 ms] Found plugin path_provider_windows at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_windows-2.2.1/
[  +12 ms] Found plugin screen_brightness at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness-0.2.2+1/
[   +4 ms] Found plugin screen_brightness_android at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_android-0.1.0+2/
[   +5 ms] Found plugin screen_brightness_ios at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_ios-0.1.0/
[   +3 ms] Found plugin screen_brightness_macos at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_macos-0.1.0+1/
[   +5 ms] Found plugin screen_brightness_windows at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_windows-0.1.3/
[  +26 ms] Found plugin volume_controller at
/home/francesco/.pub-cache/hosted/pub.dev/volume_controller-2.0.7/
[  +12 ms] Found plugin wakelock_plus at
/home/francesco/.pub-cache/hosted/pub.dev/wakelock_plus-1.1.3/
[  +12 ms] Found plugin media_kit_libs_android_video at
/home/francesco/Desktop/media-kit-main/libs/android/media_kit_libs_android_video
/
[   +3 ms] Found plugin media_kit_libs_ios_video at
/home/francesco/Desktop/media-kit-main/libs/ios/media_kit_libs_ios_video/
[   +3 ms] Found plugin media_kit_libs_linux at
/home/francesco/Desktop/media-kit-main/libs/linux/media_kit_libs_linux/
[   +3 ms] Found plugin media_kit_libs_macos_video at
/home/francesco/Desktop/media-kit-main/libs/macos/media_kit_libs_macos_video/
[   +4 ms] Found plugin media_kit_libs_windows_video at
/home/francesco/Desktop/media-kit-main/libs/windows/media_kit_libs_windows_video
/
[   +7 ms] Found plugin media_kit_native_event_loop at
/home/francesco/Desktop/media-kit-main/media_kit_native_event_loop/
[   +8 ms] Found plugin media_kit_video at
/home/francesco/Desktop/media-kit-main/media_kit_video/
[  +90 ms] Found plugin file_picker at
/home/francesco/.pub-cache/hosted/pub.dev/file_picker-5.5.0/
[   +4 ms] Found plugin flutter_plugin_android_lifecycle at
/home/francesco/.pub-cache/hosted/pub.dev/flutter_plugin_android_lifecycle-2.0.1
6/
[  +16 ms] Found plugin package_info_plus at
/home/francesco/.pub-cache/hosted/pub.dev/package_info_plus-4.2.0/
[   +6 ms] Found plugin path_provider at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider-2.1.1/
[   +2 ms] Found plugin path_provider_android at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_android-2.2.0/
[   +2 ms] Found plugin path_provider_foundation at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_foundation-2.3.1/
[   +2 ms] Found plugin path_provider_linux at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_linux-2.2.1/
[   +4 ms] Found plugin path_provider_windows at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_windows-2.2.1/
[   +8 ms] Found plugin screen_brightness at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness-0.2.2+1/
[   +3 ms] Found plugin screen_brightness_android at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_android-0.1.0+2/
[   +2 ms] Found plugin screen_brightness_ios at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_ios-0.1.0/
[   +1 ms] Found plugin screen_brightness_macos at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_macos-0.1.0+1/
[   +3 ms] Found plugin screen_brightness_windows at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_windows-0.1.3/
[  +13 ms] Found plugin volume_controller at
/home/francesco/.pub-cache/hosted/pub.dev/volume_controller-2.0.7/
[   +3 ms] Found plugin wakelock_plus at
/home/francesco/.pub-cache/hosted/pub.dev/wakelock_plus-1.1.3/
[   +7 ms] Found plugin media_kit_libs_android_video at
/home/francesco/Desktop/media-kit-main/libs/android/media_kit_libs_android_video
/
[   +2 ms] Found plugin media_kit_libs_ios_video at
/home/francesco/Desktop/media-kit-main/libs/ios/media_kit_libs_ios_video/
[   +2 ms] Found plugin media_kit_libs_linux at
/home/francesco/Desktop/media-kit-main/libs/linux/media_kit_libs_linux/
[   +2 ms] Found plugin media_kit_libs_macos_video at
/home/francesco/Desktop/media-kit-main/libs/macos/media_kit_libs_macos_video/
[   +3 ms] Found plugin media_kit_libs_windows_video at
/home/francesco/Desktop/media-kit-main/libs/windows/media_kit_libs_windows_video
/
[   +4 ms] Found plugin media_kit_native_event_loop at
/home/francesco/Desktop/media-kit-main/media_kit_native_event_loop/
[   +4 ms] Found plugin media_kit_video at
/home/francesco/Desktop/media-kit-main/media_kit_video/
[  +56 ms] Found plugin file_picker at
/home/francesco/.pub-cache/hosted/pub.dev/file_picker-5.5.0/
[   +1 ms] Found plugin flutter_plugin_android_lifecycle at
/home/francesco/.pub-cache/hosted/pub.dev/flutter_plugin_android_lifecycle-2.0.1
6/
[   +4 ms] Found plugin package_info_plus at
/home/francesco/.pub-cache/hosted/pub.dev/package_info_plus-4.2.0/
[   +2 ms] Found plugin path_provider at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider-2.1.1/
[   +2 ms] Found plugin path_provider_android at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_android-2.2.0/
[   +1 ms] Found plugin path_provider_foundation at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_foundation-2.3.1/
[   +1 ms] Found plugin path_provider_linux at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_linux-2.2.1/
[   +2 ms] Found plugin path_provider_windows at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_windows-2.2.1/
[   +4 ms] Found plugin screen_brightness at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness-0.2.2+1/
[   +1 ms] Found plugin screen_brightness_android at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_android-0.1.0+2/
[   +1 ms] Found plugin screen_brightness_ios at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_ios-0.1.0/
[   +1 ms] Found plugin screen_brightness_macos at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_macos-0.1.0+1/
[   +3 ms] Found plugin screen_brightness_windows at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_windows-0.1.3/
[   +7 ms] Found plugin volume_controller at
/home/francesco/.pub-cache/hosted/pub.dev/volume_controller-2.0.7/
[   +1 ms] Found plugin wakelock_plus at
/home/francesco/.pub-cache/hosted/pub.dev/wakelock_plus-1.1.3/
[   +3 ms] Found plugin media_kit_libs_android_video at
/home/francesco/Desktop/media-kit-main/libs/android/media_kit_libs_android_video
/
[        ] Found plugin media_kit_libs_ios_video at
/home/francesco/Desktop/media-kit-main/libs/ios/media_kit_libs_ios_video/
[        ] Found plugin media_kit_libs_linux at
/home/francesco/Desktop/media-kit-main/libs/linux/media_kit_libs_linux/
[   +1 ms] Found plugin media_kit_libs_macos_video at
/home/francesco/Desktop/media-kit-main/libs/macos/media_kit_libs_macos_video/
[   +1 ms] Found plugin media_kit_libs_windows_video at
/home/francesco/Desktop/media-kit-main/libs/windows/media_kit_libs_windows_video
/
[   +1 ms] Found plugin media_kit_native_event_loop at
/home/francesco/Desktop/media-kit-main/media_kit_native_event_loop/
[   +3 ms] Found plugin media_kit_video at
/home/francesco/Desktop/media-kit-main/media_kit_video/
[ +105 ms] Generating
/home/francesco/Desktop/media-kit-main/media_kit_test/android/app/src/main/java/
io/flutter/plugins/GeneratedPluginRegistrant.java
[ +187 ms] Initializing file store
[  +15 ms] Skipping target: gen_localizations
[   +6 ms] gen_dart_plugin_registrant: Starting due to
{InvalidatedReasonKind.inputChanged: The following inputs have updated contents:
/home/francesco/Desktop/media-kit-main/media_kit_test/.dart_tool/package_config_
subset,/home/francesco/Desktop/media-kit-main/media_kit_test/.dart_tool/flutter_
build/dart_plugin_registrant.dart}
[  +22 ms] Found plugin file_picker at
/home/francesco/.pub-cache/hosted/pub.dev/file_picker-5.5.0/
[   +4 ms] Found plugin flutter_plugin_android_lifecycle at
/home/francesco/.pub-cache/hosted/pub.dev/flutter_plugin_android_lifecycle-2.0.1
6/
[   +5 ms] Found plugin package_info_plus at
/home/francesco/.pub-cache/hosted/pub.dev/package_info_plus-4.2.0/
[   +2 ms] Found plugin path_provider at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider-2.1.1/
[   +1 ms] Found plugin path_provider_android at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_android-2.2.0/
[   +1 ms] Found plugin path_provider_foundation at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_foundation-2.3.1/
[   +1 ms] Found plugin path_provider_linux at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_linux-2.2.1/
[   +2 ms] Found plugin path_provider_windows at
/home/francesco/.pub-cache/hosted/pub.dev/path_provider_windows-2.2.1/
[   +5 ms] Found plugin screen_brightness at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness-0.2.2+1/
[        ] Found plugin screen_brightness_android at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_android-0.1.0+2/
[   +1 ms] Found plugin screen_brightness_ios at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_ios-0.1.0/
[   +1 ms] Found plugin screen_brightness_macos at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_macos-0.1.0+1/
[   +2 ms] Found plugin screen_brightness_windows at
/home/francesco/.pub-cache/hosted/pub.dev/screen_brightness_windows-0.1.3/
[   +7 ms] Found plugin volume_controller at
/home/francesco/.pub-cache/hosted/pub.dev/volume_controller-2.0.7/
[   +1 ms] Found plugin wakelock_plus at
/home/francesco/.pub-cache/hosted/pub.dev/wakelock_plus-1.1.3/
[   +3 ms] Found plugin media_kit_libs_android_video at
/home/francesco/Desktop/media-kit-main/libs/android/media_kit_libs_android_video
/
[   +1 ms] Found plugin media_kit_libs_ios_video at
/home/francesco/Desktop/media-kit-main/libs/ios/media_kit_libs_ios_video/
[   +1 ms] Found plugin media_kit_libs_linux at
/home/francesco/Desktop/media-kit-main/libs/linux/media_kit_libs_linux/
[        ] Found plugin media_kit_libs_macos_video at
/home/francesco/Desktop/media-kit-main/libs/macos/media_kit_libs_macos_video/
[   +1 ms] Found plugin media_kit_libs_windows_video at
/home/francesco/Desktop/media-kit-main/libs/windows/media_kit_libs_windows_video
/
[   +1 ms] Found plugin media_kit_native_event_loop at
/home/francesco/Desktop/media-kit-main/media_kit_native_event_loop/
[   +1 ms] Found plugin media_kit_video at
/home/francesco/Desktop/media-kit-main/media_kit_video/
[  +15 ms] gen_dart_plugin_registrant: Complete
[   +1 ms] Skipping target: composite
[   +3 ms] complete
[   +7 ms] Launching lib/main.dart on Linux in debug mode...
[   +6 ms] /home/francesco/Developer/flutter/bin/cache/dart-sdk/bin/dart
--disable-dart-dev
/home/francesco/Developer/flutter/bin/cache/dart-sdk/bin/snapshots/frontend_serv
er.dart.snapshot --sdk-root
/home/francesco/Developer/flutter/bin/cache/artifacts/engine/common/flutter_patc
hed_sdk/ --incremental --target=flutter --experimental-emit-debug-metadata
-DFLUTTER_WEB_AUTO_DETECT=true
-DFLUTTER_WEB_CANVASKIT_URL=https://www.gstatic.com/flutter-canvaskit/767d8c75e8
98091b925519803830fc2721658d07/ --output-dill
/tmp/flutter_tools.UELCWI/flutter_tool.ASXBAE/app.dill --packages
/home/francesco/Desktop/media-kit-main/media_kit_test/.dart_tool/package_config.
json -Ddart.vm.profile=false -Ddart.vm.product=false --enable-asserts
--track-widget-creation --filesystem-scheme org-dartlang-root
--initialize-from-dill
build/9849d3fde14f8238bc82d80453267a41.cache.dill.track.dill --source
file:///home/francesco/Desktop/media-kit-main/media_kit_test/.dart_tool/flutter
build/dart_plugin_registrant.dart --source
package:flutter/src/dart_plugin_registrant.dart
-Dflutter.dart_plugin_registrant=file:///home/francesco/Desktop/media-kit-main/m
edia_kit_test/.dart_tool/flutter_build/dart_plugin_registrant.dart
--verbosity=error --enable-experiment=alternative-invalidation-strategy
[  +33 ms] Building Linux application...
[  +26 ms] <- compile package:media_kit_test/main.dart
[   +5 ms] executing: [build/linux/x64/debug/] cmake -G Ninja
-DCMAKE_BUILD_TYPE=Debug -DFLUTTER_TARGET_PLATFORM=linux-x64
/home/francesco/Desktop/media-kit-main/media_kit_test/linux
[ +258 ms] -- The CXX compiler identification is Clang 14.0.0
[  +12 ms] -- Detecting CXX compiler ABI info
[ +140 ms] -- Detecting CXX compiler ABI info - failed
[        ] -- Check for working CXX compiler: /usr/bin/clang++
[ +143 ms] -- Check for working CXX compiler: /usr/bin/clang++ - broken
[        ] -- Configuring incomplete, errors occurred!
[        ] See also
"/home/francesco/Desktop/media-kit-main/media_kit_test/build/linux/x64/debug/CMa
keFiles/CMakeOutput.log".
[        ] See also
"/home/francesco/Desktop/media-kit-main/media_kit_test/build/linux/x64/debug/CMa
keFiles/CMakeError.log".
[   +2 ms] CMake Error at
/usr/share/cmake-3.22/Modules/CMakeTestCXXCompiler.cmake:62 (message):
[   +4 ms]   The C++ compiler
[        ]     "/usr/bin/clang++"
[        ]   is not able to compile a simple test program.
[        ]   It fails with the following output:
[        ]     Change Dir:
/home/francesco/Desktop/media-kit-main/media_kit_test/build/linux/x64/debug/CMak
eFiles/CMakeTmp
[        ]     Run Build Command(s):/usr/bin/ninja cmTC_ae792 && [1/2] Building
CXX object CMakeFiles/cmTC_ae792.dir/testCXXCompiler.cxx.o
[        ]     [2/2] Linking CXX executable cmTC_ae792
[        ]     FAILED: cmTC_ae792
[        ]     : && /usr/bin/clang++
CMakeFiles/cmTC_ae792.dir/testCXXCompiler.cxx.o -o cmTC_ae792   && :
[        ]     /usr/bin/ld: cannot find -lstdc++: No such file or directory
[        ]     clang: error: linker command failed with exit code 1 (use -v to
see invocation)
[        ]     ninja: build stopped: subcommand failed.
[        ]   CMake will not be able to correctly generate this project.
[        ] Call Stack (most recent call first):
[        ]   CMakeLists.txt:3 (project)
[  +38 ms] Building Linux application... (completed in 609ms)
[+27926 ms] Exception: Unable to generate build files
[   +1 ms] "flutter run" took 31,617ms.
[   +5 ms]
#0      throwToolExit
(package:flutter_tools/src/base/common.dart:10:3)
#1      RunCommand.runCommand
(package:flutter_tools/src/commands/run.dart:760:9)

#2      FlutterCommand.run.
(package:flutter_tools/src/runner/flutter_command.dart:1297:27)

#3      AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#4      CommandRunner.runCommand
(package:args/command_runner.dart:212:13)

#5      FlutterCommandRunner.runCommand.
(package:flutter_tools/src/runner/flutter_command_runner.dart:339:9)

#6      AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#7      FlutterCommandRunner.runCommand
(package:flutter_tools/src/runner/flutter_command_runner.dart:285:5)

#8      run..
(package:flutter_tools/runner.dart:115:9)

#9      AppContext.run.
(package:flutter_tools/src/base/context.dart:150:19)

#10     main (package:flutter_tools/executable.dart:90:3)

[  +92 ms] ensureAnalyticsSent: 85ms
[        ] Running 1 shutdown hook
[  +20 ms] Shutdown hooks complete
[        ] exiting with code 1
`


[Edit] Save and Discard

Right now I have these as buttons at the bottom of each card, but probably makes sense to have them more visible. Maybe the top of each card like the previous version? Or floating on the right side like where we were going to have the feedback button?
I also added a save button unique for each card/form which works ok. If you have a simpler way of switching the Update functions but using the same button that would be cool.

Also Prev/Next necessary again?

commit 01a06828ac58db2a28a6b7419e71bddace9704d


ordered_load does not error on duplicate keys (can impact pre-commit usage)

After reading this lib to see if what I'm looking for is reasonable, I think it is, but admit to not being certain. I don't have enough background with pyyaml to be sure.
ordered_load would ideally have an option to error when loading a duplicate key. This lets calling libraries (most notably pre-commit) treat these as errors or at least warnings.
Consider the following malformed .pre-commit-config.yaml file:
repos:
- repo: https://github.com/python/black
  rev: 19.3b0
  hooks:
  - id: black
    language_version: python3
repos:  # DUPLICATE KEY!
- repo: https://gitlab.com/pycqa/flake8
  rev: 3.7.8
  hooks:
  - id: flake8
    language_version: python3
    additional_dependencies: ['flake8-bugbear==19.3.0']

when loaded by pre-commit via ordered_load, it would be nice if this either failed, or at least produced a warning of some kind.
The current behavior is "last one wins" so you would see the flake8 hook above, but not the black one.

this library ~unfortunately just wraps pyyaml and therefore inherits its behaviour:
$ python
Python 3.6.8 (default, Jan 14 2019, 11:02:34)
[GCC 8.0.1 20180414 (experimental) [trunk revision 259383]] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import yaml
>>> yaml.safe_load('{a: 1, a: 2}')
{'a': 2}

(that's the main motivation for https://github.com/pre-commit/pre-commit/issues/897)

Here's the upstream issue: https://github.com/yaml/pyyaml/issues/165

Oh my! I didn't realize that there were relevant issues. The didn't come up in my searching.
Seems like you're calling out exactly this case in pre-commit/pre-commit#897 .
Sorry for the extra issue / noise. I'm closing this as a duplicate -- you can reopen it if it's useful for some purpose, but it seems like this is already tracked / on the radar.


Add pluggable String-based query parser

queries may be stored (improves #113)
queries may be used in configuration (#115)

The proposed solution has the following flaws

hard to read non compacted json
querybuilder grouped arrays/jsonobjects to represent and/or clauses is not readable.
hard to test and verify
brittle/prone to breaking
awful implementation..

Create a simple query language to replace the borked solution.

sample:
query: {'name' startswith 'q' or 'level' between 5 to 25 and 'skill' in [one, two, three] or 'lastname' like '.*'}
projections: {'unique':'field', 'name':'count', level:'sum', 'skill':'high', 'limit': {'seconds':'x', 'count':50, 'require': 'field'}}
options: {'orderby':'field', 'page':0, 'offset':0, 'order':'ascending'}

add support for projections to logging/statistics api
add a ProjectionBuilder for logging/statistics api

The interface is there. No implementation provided by default. Postpone.

Unpostponed.

New class Query can be used to build string based queries.
No parser implemented yet, need to figure out how to handle data types.
Json could be used to handle data types for a while.

Allow looking up queries by name in some kind of query register.

Implemented parser, range query and options supports integers, other queries supports strings only.
More data types needs to be supported.

Added support for integer, boolean and double for EQ and IN queries using the QueryParser.


Rest-assured and Spring boot dependency management

We encountered weird problem while upgrading the Spring boot version due to some vulnerabilities in the versions defined by the earlier Spring boot version. After the update one of our tests using REST-assured stopped to work.
The problem is that if the URL path given to REST-assured contains some URI reserved characters (";" in our case), the rest-assured is not encoding them to URL query anymore.
Or actually rest-assured is encoding them, but underlying Apache's httpclient is not. The root cause seems to be actually in Apache's httpclient's latest changes from version 4.5.6 -> 4.5.8.
It appears that HttpClient is doing some rewriting & encoding to the URL where it wrongly formats the resulting URL query. While this "URL normalization", as it is called in httpclient, is possible to disable in newer HttpClient -implementation, it seems to be hardcoded in the AbstractHttpClient that is used in the rest-assured at the moment.
The test project demonstrating the problem and downgrade -workaround are available in here:
https://github.com/VaiTuomoMakela/RestAssuredApacheHttpClient
I noticed that there exists a discussion about upgrading from the AbstractHttpClient and it isn't a easy feat to do: https://github.com/rest-assured/rest-assured/issues/497

Looks like a really painful debug session :( Given that it's configurable, would it perhaps make sense to add support for configuring the behavior in the HttpClientConfig? If so perhaps you could tell me how this is configured in HttpClient and I could add a setting for it?

Unfortunately it seems that "URL normalization" isn't configurable for the older AbstractHttpClient or the implementation that we are using, DefaultHttpClient, but only for the newer HttpClientBuilder -based HttpClient. Or at least I couldn't find any relevant configuration parameter for it from the older AbstractHttpClient.
I was able to find some background information form Apache HttpClient's issue repository
from which I understood that the configuration possibility is available only for the newer HttpClientBuilder based HttpClient where RequestConfig could have that behavior disabled.
For the HttpClient I would use something like
HttpClientBuilder.create()
            .setDefaultRequestConfig(
                RequestConfig.custom().setNormalizeUri(false).build())
            .build()

but that would require RestAssured to use HttpClient instead of AbstractHttpClient which seems quite big effort as well, when I browsed through the source code while debugging the issue and as it is explained in the issue #497.


Expect().toEqual gives false equivalencies for Immutable Records

Recently upgraded to Jest 21.0.2, and some equality tests relating to comparing Immutable Records started failing.
I was able to trace the problem back to a utility called iterableEquality.
It turns out that iterable equality falsely identifies Immutable Records as iterables (likely because of this line):
  if (a.size !== undefined) { // Immutable Records also have a size prop

since IRs (Immutable Records) have a size property.
It then constructs a new object out of the properties of the compared objects (by combining them in an array and passing that array to the constructor of the a property), but merely compares the size of the newly constructed Record to the old a record and nothing else. It seems that when constructing a new Record out of the merged properties, Immutable uses the first reference listed in the array, not the second, to build the Record.
That means that if you have two Immutable Records of any kind that have the same properties, (nevermind the contents of those properties), this test will erroneously pass and not bother to look deeper in the records.
The real concern here is that the duck-typing utilized in iterableEquality is brittle, and subject to whether or not two constructed objects have a size param, and how they construct themselves.
Workaround: Convert Immutable Records to plain JSON objects and compare those instead, or stringify them and do the same.
Jest: 21.0.2
Node: 8.4.0
npm: 5.3.0

See #4404.


Private methods being reported as undocumented

For some reason Inch CI is reporting some private methods of a class as undocumented. It doesn't do this most of the time, so I'm unsure why this class's private methods are being treated differently.

Re: [U] IGMarkets::Streaming::Subscription#on_raw_data in rviney/ig_markets

I have a hunch that Inch thinks this is an important method to document (albeit private) because it takes 4 parameters.
That is probably not what we want 😁

Some people might be interested, but you're right it's not all that relevant for assessing the quality of the public API documentation.
For what it's worth the repository does have a .yardopts file which contains --no-private.


Unflatten analyze results

Add optional uuid parameter to return uuid and issues as object
Fix tests

Pull Request Test Coverage Report for Build 321

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+5.5%) to 81.754%

Totals

Change from base Build 319:
5.5%

Covered Lines:
244

Relevant Lines:
292

💛  - Coveralls

I am waiting on getting answers from @daniyarchambylov


Add new_mask and into_inner

Summary of the PR
Please summarize here why the changes in this PR are needed.
It helps simplifying some written code inside and outside this crate.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[ ] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

Refusing to touch the Changelog, since:

You don't need to edit the changelog it is automatically generated from commit messages.


Support ignoring unsupported pydantic field types. Fixes #32

LGTM! In your use case, do you update those unsupported fields manually inside the command, or do you relie on default values?

All the fields have default values so I just rely on them.
Also it just occurred to me I probably should have named this exclude_unsupported instead of ignore_unsupported for consistency with the existing exclude parameter. I can change it an do another PR if you care.

ignore is fine, don't worry.


[Script]: OS interaction script

Aim
I want to add operating system interaction script using python.
Details
Monitors the CPU usage using the psutil library and alerts the user if the usage exceeds a certain threshold.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

/assign


Docs: add installation details for Claudia.js

I could not find any documents for Claudia JS. It's very similar to AWS Lambda deployment since it's actually also a Lambda deployment. Took me a while to set it up and do the deployment inside the docker.

Coverage remained the same at 98.978% when pulling 908a43d676682109f113eb9e97645e4488d51ddd on aliustaoglu:master into dcd68303a43fd3080d51714bdd92614f649cf96b on lovell:master.

This initialisation basically prevents any issues like module initialization error when working with Claudia.js. With Claudia, you cannot create your own zip file and upload it. Everything is done by Claudia.JS therefore all has to be done inside the docker.

Hello, thanks for this PR for the docs.
I haven't used claudia but notice it can control npm install via its ---npm-options flag. Do you think the following might work instead?
claudia create --npm-options="--arch=x64 --platform=linux --target=8.10.0" ...

Hi, I did not know about this npm-options parameter. Not sure if works but will give it a try when I get home. Docker method definitely works but have to admit that it's pretty low level and could be hard to setup. And it's slower.
I'll let you know about the results later.

I get an error 'linux-x64' binaries cannot be used on the 'darwin-x64' and it seems that is the main reason for Docker.
Full error message is as below:
$ claudia update --no-optional-dependencies --profile claudia --npm-options="--arch=x64 --platform=linux --target=8.10.0"
packaging files	npm install -q --no-audit --production --arch=x64 --platform=linux --target=8.10.0

><EMAIL_ADDRESS>install /private/var/folders/x_/18qnbsm16r94s68crds61jxm0000gp/T/cmkeZ5/s3resizer-1.0.0-BPs4dR/package/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

info sharp Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.7.0/libvips-8.7.0-linux-x64.tar.gz
validating package
Error: 'linux-x64' binaries cannot be used on the 'darwin-x64' platform. Please remove the 'node_modules/sharp/vendor' directory and run 'npm install'.
    at Object.hasVendoredLibvips (/private/var/folders/x_/18qnbsm16r94s68crds61jxm0000gp/T/cmkeZ5/s3resizer-1.0.0-BPs4dR/package/node_modules/sharp/lib/libvips.js:61:13)
    at Object.<anonymous> (/private/var/folders/x_/18qnbsm16r94s68crds61jxm0000gp/T/cmkeZ5/s3resizer-1.0.0-BPs4dR/package/node_modules/sharp/lib/constructor.js:9:22)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Module.require (module.js:579:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/private/var/folders/x_/18qnbsm16r94s68crds61jxm0000gp/T/cmkeZ5/s3resizer-1.0.0-BPs4dR/package/node_modules/sharp/lib/index.js:3:15)
cannot require ./index after clean installation. Check your dependencies.

Thanks for checking. The "validating package" message and subsequent error is from validatePackage inside claudia, which appears to always require your code and its dependencies using the build platform rather than the target platform.
The advice in the Claudia docs at https://claudiajs.com/tutorials/packaging.html#local-modules is to  lazy-load native modules such as sharp:

"For example, if you use binaries for a different operating system, lazy-load those modules when you need them, so they do not prevent package validation."

Perhaps claudia could be modified to reduce its validatePackage check from an error to a warning when --npm-options includes a --platform=linux override? That feels like a more user-friendly approach than pushing documentation onto all the native modules that might be used with it.


Simple Query...

Hi Guy,
Your plugin is very popular and looks to be well written. Its clear you put a lot of work in it, so it obviously deserves a look.
The problem is, I cannot figure out why it is useful.  :)  Could you explain when someone would want to use it?  In other words, what are the use cases?
This is not to insult you or just be a troll.  My guess is that I'm not the only one who doesn't "get it".
Thanks for listening.
-T

Hello,
It's a really simple plugin, and it doesn't really do much so I get why you think it's useless/shit.
I personally found it a ball ache having to open my browser and search around my remote repository for the block of code I'm looking at in PhpStorm, finding the right branch and line number etc each time I want to discuss/share that block of code with someone. Past providing a shortcut to do this for you so you can share that link with someone or just get to it in the remote repository it doesn't do anything.
Other than that I just felt like giving Java a go and experimenting with Jetbrains plugins so in short it's a really trivial plugin.

Thanks for the reply.   :)  Now that you pointed out what it fixes for you, it makes perfect sense.  Pretty cool, actually.


Ignore elements that belong to other/nested forms

Note to self: Need to fix the case described in #2035 as this PR only ignores the elements when submitting the form or when validating it directly using either $(form).valid(), validator.form() or validator.checkForm().

Fixed the issue mentionned in my previous comment.
This PR is good to go now.


Notifications stop working after Asyncstorage Removeitem

Hey,
I am having a issue that my alerts stop working after I remove the authtoken using Asyncstorage.removeitem. Can you please suggest me a solution to that?

I don't know if I understood your problem correctly, but you can create a context that encloses your project containing your notification logic , or a serviceEnvoyé de mon iPhoneLe 11 janv. 2023 à 19:57, Waqar Ali @.***> a écrit :
Hey,
I am having a issue that my alerts stop working after I remove the authtoken using Asyncstorage.removeitem. Can you please suggest me a solution to that?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Disable the BackgroundLayerChooser

This suggest to disable the BackgroundLayerChooser as it seems to be broken right now (see #485) and:

There is no way to define the required property isBackgroundLayer in shogun nor the shogun-admin.
There seems to be a conflict by setting it to a layer that is also visible in the tree (the layer must be visible in the tree to be catched up by the chooser).
There seems to be a request loop once a layer is activated in it.

Please review @terrestris/devs.

Thanks for the review!


feat(swap): add cross chain swap fees display

Description

Test plan

Related issues

Fixes RET-1066

Backwards compatibility

Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[ ] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

Closing for #5637 and #5636


Dates calculations

Add functionality to get milliseconds from fictional date

Limited application of this will be done in #2. This will focus on full datetime conversion.


Forcing api meme type ?

Hi @kocsismate,
jsonapi specs define the mime type application/vnd.api+json.
It could be interesting to force consumer to use this mime type, I mean return an HTTP 406 if not.
It let the developer use an other response formalism, for example msgpack ...
What do you think ?
Regards,

Hi @waghanza,
Yes, Yin has this feature: https://github.com/woohoolabs/yin#content-negotiation.
But if you'd like to "hack" the spec, and use msgpack or any other format besides JSON, you are also equipped with necessary tools: https://github.com/woohoolabs/yin#custom-serialization


[cli] Rancher 2.6 Project context switching support

Hello It seems the CLI doesn't support project context switching within 2.6 clusters just as of yet.
running latest stable Rancher CLI version: v2.4.12
It looks like the cli doesn't like the new c-m-* as opposed to the older c-* project naming convention
$ rancher login -t <token> --context 'c-m-qvdxgtc2:p-bgvwr' https://<rancher cluster dns>

FATA[0002] Unable to parse context (c-m-qvdxgtc2:p-bgvwr). Please provide context as local:p-xxxxx, c-xxxxx:p-xxxxx, or c-xxxxx:project-xxxxx

$ rancher context switch
NUMBER    CLUSTER NAME      PROJECT ID             PROJECT NAME   PROJECT DESCRIPTION
1         testing-context   c-m-qvdxgtc2:p-bgvwr   System         System project created for the cluster
2         testing-context   c-m-qvdxgtc2:p-lwwf9   Default        Default project created for the cluster
3         local             local:p-f8s9m          Default        Default project created for the cluster
4         local             local:p-phx6w          System         System project created for the cluster

The regex needs to be updated to include c-m to basically match how project works. https://github.com/rancher/cli/blob/master/cmd/common.go#L536

Reproduced this issue on v2.6.0

Works for rke1 clusters and local cluster without any errors but not for rke2 clusters which have the cluster name that start with c-m.
For rke2 clusters, the following error is thrown:

FATA[0002] Unable to parse context (c-m-jlmjq882:p-hp9hn). Please provide context as local:p-xxxxx, c-xxxxx:p-xxxxx, or c-xxxxx:project-xxxx
To reproduce:

Create an rke1 node driver and rke2 node driver cluster or custom clusters.
Download rancher cli and login to the rancher with the command
./rancher login https:// --token token- --context 'c-m-ztq8qwcf:p-xb8gw'
Error is thrown as mentioned above.
As a workaround, if we try logging in and then select the context it works:
./rancher login https://<redacted> --token token-<redacted>
./rancher context switch --> select the project number to switch to

INFO[0003] Setting new context to project System
INFO[0003] Saving config to /Users/anupamaupadhyayula/.rancher/cli2.json

Should be available to test when v2.4.13-rc1 is ready. https://github.com/rancher/cli/releases/tag/v2.4.13-rc1

Verified with the rancher cli version: v2.4.13-rc1, rancher server: v2.6-head 1a57856

Created a downstream rke2 and rke1 node driver clusters, custom clusters
from the rancher cli gave the command
./rancher login -t token-redacted --context 'c-m-z6thzkwx:p-24p5m' https://redacted
Works as expected.
Verified for rke1 clusters and local clusters - works as expected


Minor interface cleanup

Before offering this as a public app, we should clean up a few minor interface issues. This ticket can serve as a log as those come up:

[x] - The typo "successfull" appears in a few places (https://github.com/geobtaa/geomg/search?q=successfull)
[ ] - On the Bulk Action page, a list of Identifiers for selected items will appear. These still link to local host (i.e. http://localhost:3001/catalog/1f3cf343-bad9-427b-8416-6552954aa301)
[ ] Remove all footer links except one for "OpenGeoMetadata" (https://opengeometadata.org)

Adding buttons to view the Bulk Action item in GEOMG or the Geoportal:


Dockerizing tests

@alexanderbez

@mvid mind giving a quick instruction set on how to test this? I can obviously start the Docker network via docker-compose, but how then do I test completeness? Are there commands or tests to run? Or is just starting the docker compose network sufficient?

@alexanderbez I am still working on completeness for the network, but right now I have a script in the root called ./start-testnet.sh which should handle genesis, key signing, etc. After that, there will (probably) be another container that runs tests. Or maybe just a script.

I see, so if I execute ./start-testnet.sh with a zero exit-code, I can deem this PR acceptable?

To run ./start-testnet.sh on my Mac I needed to
cd module && make install
brew install sponge
docker-compose logs orchestrator0 shows the timeout of the orachestor trying to communicate with http://gravity0:1317
A debug message shows the url is correct up to being passed to the Contact library
orchestrator0_1      | [peggy_utils/src/connection_prep.rs:109] &cosmos_legacy_url = "http://gravity0:1317"

This is the only thing I've found in the dependencies for the orchestrator that might be an explanation
https://github.com/actix/actix-web/issues/599

@jkilpatr could you take a look? Does this meet your testing needs?

@mvid can you fix the conflicts?

@zmanian conflicts fixed

Seeing this error in all but the first orchestrator:
Valset cost estimate for Nonce 1 failed with Err(EthereumRestError(JsonRPCError { code: 3, message: "execution reverted: Supplied current validators and powers do not match checkpoint
Any ideas @jkilpatr ?

This looks like it is passing checks. Going to merge into the gentx branch


webhooks: use separate service and port for validating webhook

Description:
For some reason the validating and mutating webhooks served by
osm-controller don't work as expected when both refer to the
osm-controller service. This change creates a separate service
for the config validator webhook endpoint.
Also fixes the http server mux to use separate instances, otherwise
the server configurations for the two webhooks can conflict and
affect each other.

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2273 (b709529) into main (ef33665) will decrease coverage by 0.29%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #2273      +/-   ##
==========================================
- Coverage   56.98%   56.69%   -0.30%
==========================================
  Files         147      147
  Lines        6568     6560       -8
==========================================
- Hits         3743     3719      -24
- Misses       2820     2836      +16
  Partials        5        5

Impacted Files
Coverage Δ

cmd/osm-controller/osm-controller.go
22.75% <ø> (-0.25%)
:arrow_down:

pkg/configurator/validating_webhook.go
68.84% <50.00%> (-10.15%)
:arrow_down:

pkg/injector/webhook.go
74.09% <100.00%> (ø)

pkg/metricsstore/metricstore.go

pkg/metricsstore/metricsstore.go
100.00% <0.00%> (ø)

pkg/kubernetes/event_handlers.go
63.88% <0.00%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd65933...b709529. Read the comment docs.

Codecov Report

Merging #2273 (b709529) into main (ef33665) will decrease coverage by 0.29%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #2273      +/-   ##
==========================================
- Coverage   56.98%   56.69%   -0.30%
==========================================
  Files         147      147
  Lines        6568     6560       -8
==========================================
- Hits         3743     3719      -24
- Misses       2820     2836      +16
  Partials        5        5

Impacted Files
Coverage Δ

cmd/osm-controller/osm-controller.go
22.75% <ø> (-0.25%)
:arrow_down:

pkg/configurator/validating_webhook.go
68.84% <50.00%> (-10.15%)
:arrow_down:

pkg/injector/webhook.go
74.09% <100.00%> (ø)

pkg/metricsstore/metricstore.go

pkg/metricsstore/metricsstore.go
100.00% <0.00%> (ø)

pkg/kubernetes/event_handlers.go
63.88% <0.00%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd65933...b709529. Read the comment docs.

This is a fix for https://github.com/openservicemesh/osm/issues/2264
Further discussion needed for if we want to separate the validating webhook from osm-controller.


Request: switch license

Rationale:
The current MIT license in use is generally only for software, and the main part of this project is documentation, so a more appropriate license is needed.
Suggestion:
Use CC or GFDL license.
But in my personal opinion, since this project is about documenting facts rather than creating new things, it could be done under a more open license that makes it easy for people to use, like public domain. so when people copy parts of a document, they don't need to credit who the author is or include a license in context.
This issue is a fork of a previous thread (#41), you may be interested in reading the previous discussion history.

https://choosealicense.com/non-software/

https://choosealicense.com/non-software/

Personally, I would recommend CC0. This releases the document into the public domain or disclaims copyright to the fullest extent permitted by law.
The reason is that this will be a dictionary where most of the content is consensus and not original, CC0 is good for documenting facts and is easy to use, licenses like CC-BY require attribution to the author or citation to the original source, this makes it difficult to freely use the documentation.
But CC0 is not compatible with Wikipedia, because Wikipedia uses the stricter CC-BY-SA 3.0, which will prevent us from directly importing the list. But since this is a dictionary, no one can claim copyright on a word, so maybe we can just copy it? idk.

But in my personal opinion, since this project is about documenting facts rather than creating new things

I'm not sure what "fact" means in this context. For example "sw" is a recommended abbreviation of "software". Is it fact? I'd rather say it's an opinion.

The reason is that this will be a dictionary where most of the content is consensus and not original [...]

I sounds better to me.

I know only the basic open source software licenses so I can't really decide which option would be the best, however CC0 really sounds like what we need.
If you all agree with it, I'll change the license.

I'm not sure what "fact" means in this context. For example "sw" is a recommended abbreviation of "software". Is it fact? I'd rather say it's an opinion.

When there is a consensus, I would call it a fact, and dictionaries are not for coining new words, but for documenting the consensus.

however CC0 really sounds like what we need.

The benefit of CC0 is that other projects or anyone in the world can use this project completely freely, but the challenge for us is that we cannot copy content from Wikipedia, Wiktionary or other copyrights, unless different content is given different license.
CC-BY is the documentation license equivalent to the current MIT license, but this is still too broad for Wikipedia's CC-BY-SA, so even with CC-BY, we don't allow importing content from Wikipedia.
If you plan to import content from Wikipedia, it must be licensed under the same license as Wikipedia, which is CC-BY-SA 3.0.

CC-BY is the documentation license equivalent to the current MIT license, but this is still too broad for Wikipedia's CC-BY-SA, so even with CC-BY, we don't allow importing content from Wikipedia.

Clear now, thank you. Let's see what will be conclusion in #47 and make decision accordingly.

Clear now, thank you. Let's see what will be conclusion in #47 and make decision accordingly.

Also, if other members of the project previously contributed parts licensed under MIT, switching licenses requires their approval, or rewriting that part.
But since this is a dictionary, I don't know how to rewrite a word. 🤣

Hey guys.
We are talking about switching the license of this project, since you are a contributor to this project, I hope you can join our discussion.
Because your contribution is licensed under the MIT license, switching licenses requires your consent, or we need to rewrite the part you contributed.
In addition, we are making a series of major changes to the project, here are some topics:

Initial discussion #41
Request: switch license #45
Request: make this project a dictionary #46
Request: Make the project more inclusive and accessible #47
Request: translate this project #48
Request: move project under an organization #49

Let's build this project together.
@yan-kuan @h-h-h-h @cfjello @philipeachille @stevencychuang @Firemanarg @chriscalo @Ramesh-X @emareg @SharifElfouly

Consent from me is granted to change the license to whatever you feel is best for the project. Let me know if you need my consent phrased in any particular way or recorded anywhere specific (other than here).

Consent from me is granted to change the license to whatever you feel is
best for the project. Let me know if you need my consent phrased in any
particular way or recorded anywhere specific (other than here). On Sun, 5
Mar 2023 at 00.47, oood @.***> wrote:
Hey guys.

We are talking about switching the license of this project, since you are
a contributor to this project, I hope you can join our discussion.
Because your contribution is licensed under the MIT license, switching
licenses requires your consent, or we need to rewrite the part you
contributed.
In addition, we are making a series of major changes to the project, here
are some topics:
Initial discussion #41
https://github.com/kisvegabor/abbreviations-in-code/issues/41
Request: switch license #45
https://github.com/kisvegabor/abbreviations-in-code/issues/45
Request: make this project a dictionary #46
https://github.com/kisvegabor/abbreviations-in-code/issues/46
Request: Make the project more inclusive and accessible #47
https://github.com/kisvegabor/abbreviations-in-code/issues/47
Request: translate this project #48
https://github.com/kisvegabor/abbreviations-in-code/issues/48
Request: move project under an organization #49
https://github.com/kisvegabor/abbreviations-in-code/issues/49
Let's build this project together.
@yan-kuan https://github.com/yan-kuan @h-h-h-h
https://github.com/h-h-h-h @cfjello https://github.com/cfjello
@philipeachille https://github.com/philipeachille @stevencychuang
https://github.com/stevencychuang @Firemanarg
https://github.com/Firemanarg @chriscalo https://github.com/chriscalo
@Ramesh-X https://github.com/Ramesh-X @emareg
https://github.com/emareg @SharifElfouly
https://github.com/SharifElfouly
—
Reply to this email directly, view it on GitHub
https://github.com/kisvegabor/abbreviations-in-code/issues/45#issuecomment-1454925373,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD273J2BBLUK2YGTRFYWC6LW2PIB7ANCNFSM6AAAAAAVPAWSPU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi, everyone, we are still discussing the license that the project will adopt in the future.
A more permissive license than MIT is expected to be adopted, CC0 is my recommendation as the proponent of changing the license, this will put the project directly under the public domain, so everyone can use the project completely freely.
But there are some other options out there too, like CC-BY, CC-BY-SA, GFDL licenses.
It would be very helpful if you would like to join the discussion. You can reply to this thread with the license you want to use, and we will choose a new license in the future based on the preferences of everyone in this thread.
I may @ you twice in the future, the first time is about consensus generation. The second time is to ask you to confirm the new license.
We look forward to your joining us to build a better project, thank you for your contribution.
@yan-kuan @h-h-h-h @cfjello @philipeachille @stevencychuang @Firemanarg @chriscalo @Ramesh-X @emareg @SharifElfouly

CC0 is fine with me.

CC0 is fine with me.

It is worth pointing out that
Some differences between CC0 and the current MIT license are:
MIT requires attribution to the project author(s) and includes a copy of the license each time the project is used.
CC0 has no requirements.
MIT licensees can change the license and even close source it.
CC0 is irrevocable, which means it will be shared forever as the shared knowledge of humanity.
In addition, the CC-BY license is almost a document license equivalent to the MIT license, which requires attribution by the user like MIT, except that it is also irrevocable. It is worth noting that bylined authors can have a positive impact on promoting a project.

CC0 is fine with me.

Same. Choose whatever. I thought this would be possible with MIT anyway - taking the work and relicensing it.

maybe one can license the planned JSON file as CC0 and any other works derived from that with CC-BY-SA

sounds cool. Since that's CC0, anyone can use it in any way, including a more restrictive license.

I didn't follow this issue very well, so what was the final decision?

The list is open source and as such MIT is the best


DiagnosticsAsRows(): panic: runtime error: index out of range

I'm running into this issue when running off the latest build from master. Since running with the fix I proposed in #2366 I now start seeing this. I don't think it's related as sometimes this comes up even without my int64 fix. This requires monitoring=true to trigger. I believe the problem occurs in DiagnosticsAsRows() when we append the path value at line 3310 to the slice of interfaces.
panic: runtime error: index out of range

goroutine 278 [running]:
github.com/influxdb/influxdb.(*Server).convertRowToPoints(0xc20816c000, 0x9d4c10, 0xb, 0xc208041ce0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/jari/go/src/github.com/influxdb/influxdb/server.go:3978 +0x7b7
github.com/influxdb/influxdb.func·011()
	/Users/jari/go/src/github.com/influxdb/influxdb/server.go:399 +0x5c0
created by github.com/influxdb/influxdb.(*Server).StartSelfMonitoring
	/Users/jari/go/src/github.com/influxdb/influxdb/server.go:412 +0x241

Fixed by https://github.com/influxdb/influxdb/pull/2376

All fixed. Thanks to @jtakkala for the bug report.


Add zeroes to barcodes in mapping files

Not sure when/why they were removed, but this adds them back in. I used the
code at the following gist:
https://gist.github.com/adamrp/cc337b63e7ac33160f4b
Fixes #130

:+1:

In running an analysis that was using AG.txt, PGP.txt and others, I ran into a similar issue, tried applying this script but it didn't fix my problem.
Actually looking at the mapping file + the OTU table, there seems to be a mismatch:

In [1]: from biom import load_table

In [2]: from md import load_template_to_dataframe

In [9]: mf = load_template_to_dataframe('AG.txt')
/Users/yoshikivazquezbaeza/.virtualenvs/qiime-dev/lib/python2.7/site-packages/pandas/io/parsers.py:1159: DtypeWarning: Columns (34,200) have mixed types. Specify dtype option on import or set low_memory=False.
  data = self._reader.read(nrows)

In [10]: bt = load_table('AG.biom')

In [11]: mfids = set(mf.index.tolist())

In [18]: btids = set(bt.ids('sample'))

In [19]: len(mfids)
Out[19]: 4545

In [21]: len(btids)
Out[21]: 4545

In [23]: len(mfids - btids)
Out[23]: 3459

I gotta go back to the processing directory and checkout whats going on
On Fri, Mar 6, 2015 at 5:26 PM, Yoshiki Vázquez Baeza <
<EMAIL_ADDRESS>wrote:

In running an analysis that was using AG.txt, PGP.txt and others, I ran
into a similar issue, tried applying this script but it didn't fix my
problem.
Actually looking at the mapping file + the OTU table, there seems to be a
mismatch:
In [1]: from biom import load_table
In [2]: from md import load_template_to_dataframe
In [9]: mf = load_template_to_dataframe('AG.txt')/Users/yoshikivazquezbaeza/.virtualenvs/qiime-dev/lib/python2.7/site-packages/pandas/io/parsers.py:1159: DtypeWarning: Columns (34,200) have mixed types. Specify dtype option on import or set low_memory=False.
data = self._reader.read(nrows)
In [10]: bt = load_table('AG.biom')
In [11]: mfids = set(mf.index.tolist())
In [18]: btids = set(bt.ids('sample'))
In [19]: len(mfids)
Out[19]: 4545
In [21]: len(btids)
Out[21]: 4545
In [23]: len(mfids - btids)
Out[23]: 3459
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/American-Gut/pull/131#issuecomment-77661060.

Were these zeroes added back in by some other fashion, or are we still investigating? If this PR is not needed any more, please close.

Regenerated, this should be safe to close
On Wed, Mar 25, 2015 at 11:02 AM, adamrp<EMAIL_ADDRESS>wrote:

Were these zeroes added back in by some other fashion, or are we still
investigating? If this PR is not needed any more, please close.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/American-Gut/pull/131#issuecomment-86122206.


added cs to code markdown

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


about waymo result

Hi, you mentioned you use adabin trained on waymo. So how you do that, since waymo don't provide the gt depth map. Another question is did you train the total model, depthcompletion, 2d detection and 3d detection in an end-to-end manner?

We trained the AdaBins[https://github.com/shariqfarooq123/AdaBins] using sparse depth maps generated by point clouds as ground truth. Since we focus on the 3d detection in our paper, the depth estimation and 2d detection are completed offline.

We trained the AdaBins[https://github.com/shariqfarooq123/AdaBins] using sparse depth maps generated by point clouds as ground truth. Since we focus on the 3d detection in our paper, the depth estimation and 2d detection are completed offline.

thankyou


Adds rebase-client-sdk to Rust portion of project

allows for usage creating server-side witness issuers and clients
adds js package for WASM compiled version of Rebase lib
updates witness and dapp in demo folder to use these libs
adds example of live post checking

Updated with improved ergonomics!


Defs block for extracting gradients.

With Symbols is it possible to configure svg-sprite to extract all the gradients <linearGradient> (etc) into a <defs> block -- or will I need to use Cheerio for that?

Hey @jaywolters,
it's not really within svg-sprite's scope to perform SVG manipulations like that. You could, however, use a custom transform to operate on each single SVG shape before it's compiled into the sprite. See here for a recent example of how to re-color a shape (used with grunt-svg-sprite, but it'll work with svg-sprite just the same way).
svg-sprite doesn't use Cheerio but operates on a raw DOM implementation instead (xmldom). Please see here for some info about custom callback transformations.
Finally, I am considering adding a global post-transformation on the final sprite, but that has to be done yet.
Hope this helps!?
Cheers,
Joschi

Using the Symbol method and when working with pre-colored Icons that use gradients-- moving the gradients to a <Defs> block is essential for FF and IE to render the SVG correctly. This problem has been brought up in a few SO posts and grunt/gulp-svgstore made the change just recently. Thanks for your response.

Good to know, thanks for this info. You don't happen to have some resources handy elaborating on the topic? I might consider hardwiring it then ...

Refer to David Bushell's blog post on SVG and MDN using gradients in SVG you will see where it shows the SVG markup and how all the Gradients are placed inside of the <defs> block.
I experimented with this problem before figuring it out myself as I was transitioning from using <defs>  to <symbols>. I noticed that gradients worked fine when I was using <defs>  alone but once I started using <symbols>  I lost the gradient fills in FireFox. After reading the SVG 1.1 spec and reading SO posts like this one it turns out that gradient fills are resources that should be defined in the <defs> block. Firefox is trying to address this issue at bugzilla.
 Firefox will render the gradients and icons just fine while Chrome and Safari will do different things either filling with currentColor or simply not filling the space at all. David Bushell's article covers all this and says everything I learned from trial and error. In-order for Symbols with Gradients to display correctly and predictably across all browsers the svg-sprite must be inline (in the page/Dom) and the gradients have to be in a <defs> block.
The svgstore people talk about it here. I hope this helps.

Thanks a lot for these useful resources! You convinced me that this should be a core feature of svg-sprite, as it will greatly improve the overall usability of the sprites. I'll try to implement the gradient extraction, but please give me some time for that. Unfortunately, I'm extremely overloaded these days an am having a hard time getting all my things done. Again, thanks a lot!

Necromancy!
Since nobody is working on this issue I decided to quickly create a solution using transforms in config.
Here is my solution, in case somebody else needs to support gradients on Firefox.
@jkphl in the next few weeks I could create PR with this solution (cleaned up, of course). Though I am not sure where to place this kind of functionality. Any ideas?
const defs = new DOMParser().parseFromString('<defs></defs>');
let count = 0;

const config = {
  dest: 'public/svg',
  mode: {
    symbol: {
      dest: '.',
      sprite: 'sprite.svg',
      inline: true,
    },
  },
  shape: {
    transform: [
      gradientsExtraction,
      'svgo',
    ],
  },
  svg: {
    transform: [
      /**
        * Adds defs tag at the top of svg with all extracted gradients.
        * @param {string} svg
        * @return {string} svg
        */
      function(svg) {
        return svg.replace(
          '<symbol ',
          new XMLSerializer().serializeToString(defs) + '<symbol '
        );
      },
    ],
  },
};

/**
 * Extracts gradient from the sprite and replaces their ids to prevent duplicates.
 * @param {SVGShape} shape
 * @param {SVGSpriter} spriter
 * @param {Function} callback
 */
function gradientsExtraction(shape, spriter, callback) {
  const idsToReplace = [].concat(
    extractGradients(shape, 'linearGradient'),
    extractGradients(shape, 'radialGradient')
  );

  shape.setSVG(updateUrls(shape.getSVG(), idsToReplace));

  callback(null);
}

/**
 * Extracts specific gradient defined by tag from given shape.
 * @param {SVGShape} shape
 * @param {string} tag
 * @return {Array}
 */
function extractGradients(shape, tag) {
  const idsToReplace = [];

  const linearGradients = shape.dom.getElementsByTagName(tag);
  for (let i = 0; i < linearGradients.length; i++) {
    // Add gradient to defs block
    defs.documentElement.appendChild(linearGradients[i]);

    // Give gradient new ID
    const id = linearGradients[i].getAttribute('id');
    const newId = 'g' + (++count);
    linearGradients[i].setAttribute('id', newId);

    idsToReplace.push([id, newId]);
  }

  return idsToReplace;
}

/**
 * Updates urls in given SVG from array of [oldId, newId].
 * @param {string} svg
 * @param {Array} idsToReplace
 * @return {string}
 */
function updateUrls(svg, idsToReplace) {
  for (let i = 0; i < idsToReplace.length; i++) {
    const str = 'url(#' + idsToReplace[i][0] + ')';
    svg = svg.replace(
      new RegExp(regexEscape(str), 'g'),
      'url(#' + idsToReplace[i][1] + ')'
    );
  }

  return svg;
}

/**
 * Escape regex characters in given string
 * @param {string} str
 * @return {string}
 */
function regexEscape(str) {
  return str.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&');
}

@Laruxo
Nice script.
But I have no DomParser in node.js. So how did this work for you?

Any updates on this?

Sharing this in case it helps anyone. Snippet will add your defs trimming whitespace and new lines:
transform: [
  function (svg) {
    const defs = `
      <defs>
        <linearGradient id="gradient" x1="56.0849658%" y1="50%" x2="105.749837%" y2="128.905203%">
          <stop stop-color="#FFAE6C" offset="0%"></stop>
          <stop stop-color="#FF4E51" offset="100%"></stop>
        </linearGradient>
      </defs>
    `

    return svg
      .replace('<symbol', `${defs.split(/\n/).map((s) => s.trim()).join('')}<symbol`)
      .replace(/<symbol/gi, '<symbol fill="url(#gradient)"')
  },
],

Any updates on this? Any alternative library?


Hard coded colors

Bug Report
Theme colors can't overwrite hard coded colors in certain places.
Example here:
https://github.com/OHIF/Viewers/blob/master/platform/ui/src/components/studyBrowser/StudyBrowser.styl#L6
Describe the Bug
When trying to overwrite the default colors some components wont take the color from the :root css variables.
How to find all hardcoded colors:
search for
#([a-f]|[A-F]|[0-9]){3}(([a-f]|[A-F]|[0-9]){3})?
and exclude:
*.svg, *.lock, *.md
Solution suggestion:
Replace them with css variables.
e.g. change background-color: black to background-color:  var(--primary-background-color)

You should modify the tailwind configuration. Since we moved to tailwindcss

Is it possible to ask for an example? For example, how to change the top bar color? The one with the tool icones?

@sedghi that was a great decision. @nicoesiea i haven't tried, but from my experience with tailwind: Change this file https://github.com/OHIF/Viewers/blob/master/platform/ui/tailwind.config.js  and build the viewer.

Just be careful, there are two tailwind.config.js files (the first in the APP folder and the second in UI).
I do not understand the reason for these two files.
NOTE I created a new issue for splitButton hard-coded color (https://github.com/OHIF/Viewers/issues/3767)


Improve CI

Codecov Report

Merging #9 (ff1d213) into master (857b7b2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        18
  Lines          211       211
  Branches        45        45
=========================================
  Hits           211       211

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 857b7b2...ff1d213. Read the comment docs.


Ctrl+Alt+ doesn't work in the Shortcut Editor on international keyboards

Problem Description
This is a follow up of PR #7929.
As mentioned in https://github.com/spyder-ide/spyder/pull/7929#issuecomment-425202097, Some Ctrl+Alt+<key> combinations are not interpreted correctly by the Shortcut Editor. For example, Ctrl+Alt+S is registered as Ctrl+Alt+ß with my United States - International keyboard.
PR #7929 fixed the problem for Ctrl+Alt+Shift+<key>, but it was not possible to do the same with Ctrl+Alt+<key> combinations due to a bug in Qt: https://bugreports.qt.io/browse/QTBUG-60768
I could try to submit a patch to fix this in Qt directly, but I do not think I am up to this task, and the fix won't be available until a very long time... Alternatively, I could implement a workaround I have developed in the Shortcut Editor. This is not pretty, but this works.
Note that this issue happens in Spyder 3 and 4, but I intend to fix it only for Spyder 4.

Alternatively, I could implement a workaround I have developed in the Shortcut Editor. This is not pretty, but this works

I think this is the best alternative.

Note that this issue happens in Spyder 3 and 4, but I intend to fix it only for Spyder 4.

I'm ok with that.


trust wallet connection

What is the purpose of the change:
Added trust wallet connection
Linear Task
Linear Task URL
Brief Changelog

Add new module for trust wallet

Testing and Verifying
Test on local is successfully

@forbesus can you rebase stage?

Yes @MaxMillington
And should I re-create PR?

@MaxMillington
Are you okay with this?


Basic text input widget

Description
text input widget, uses material-ui TextField and takes callbacks for keypress, blur/deselect and focus/select events.
Testing instructions

Review code
Check Travis build
Review changes to test coverage
npm run storybook
examine the widget

Agile board tracking
connect to #26

Codecov Report

Merging #140 into version1 will decrease coverage by 0.07%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##           version1     #140      +/-   ##
============================================
- Coverage     93.35%   93.28%   -0.08%
============================================
  Files            41       42       +1
  Lines           632      640       +8
  Branches         89       89
============================================
+ Hits            590      597       +7
- Misses           39       40       +1
  Partials          3        3

Impacted Files
Coverage Δ

...colmWidgets/textInput/WidgetTextInput.component.js
87.5% <87.5%> (ø)

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 5094e11...a61751e. Read the comment docs.

Looks good, just one naming issue


Upgrade to 2.14

Per https://www.gerritcodereview.com/releases/2.14.md -- Upgrading to 2.14

Hi @ShakataGaNai
Thanks for the PR. There are still some work need to be done before the PR can be merged.

Would you please rebase you patch onto the master branch because PR of upgrading to 2.13.8 has been merged into the master branch.

According to the release note of 2.14 that the Bouncy Castle Crypto API libraries are now distributed in the Gerrit .war. So I think these lines should be removed from Dockerfile.

2.14 has been released. Let me know if it works for you, please.


App Crash with maxSize: 1080

Current behavior
For Video compression on Android when using manual mode with maxSize: 1080 the library crashes.
Expected behavior
Ideally the library shouldn't have crashed
Platform

[X] Android
[ ] iOS

React Native Version
0.73.0
React Native Compressor Version
1.8.24
Reproducible Steps And Demo
You can use any video with 1080x1920 aspect ratio with maxSize:1080 and see the app crash

Thank you so much for building such an incredible library.


Bump dependencies

earmark 0.2.1 -> 1.3.1
ex_doc 0.11.5 -> 0.19.3
idna 5.1.0 -> 6.0.0
Judging by the idna changelog, there aren't any actual breaking changes
in 6.0.0:
https://github.com/benoitc/erlang-idna/blob/3eb54ccbfa6fb917c0f4ca9197da337ad888ffe0/CHANGELOG
Turns out that idna 6.0.0 has some breaking changes so adjust how
punycode conversions are done
References:

https://github.com/benoitc/erlang-idna/pull/28
https://github.com/benoitc/erlang-idna/issues/26#issuecomment-466939431

Note: the test "compiles using a newly fetched copy of the rules file if so configured" seems to use a version of publicsuffix-elixir from the deps directory so I couldn't get it to pass locally.

Should probably test this new code with previous idna versions, but not sure if there's a way to do that in exunit. Maybe by specifying an extra mix.lock file for testing?

@axelson @seomoz anything I can do to help get this merged? the current version of idna conflicts with hackney.

@r3l4t1v you can just use my fork: https://github.com/axelson/publicsuffix-elixir
Seomoz hasn't been responsive for quite a while. I may push my fork to hex at some point.


Typo correcting 3.0.7 -> 0.3.7 for vint --no-color

Since there is no vint v3.0.7, I assumed (and confirmed) that the change was to target v0.3.7 of vint instead.
Comparing 0.3.7 with 0.3.6, it looks like the intended target version was 0.3.7 after all.
Did not add myself as a contributor since this only corrects a minor typo.

Yep, that's just a simple mistake. Cheers! :beers: