Allow subscribing to table changes.

This was a feature request that's been in limbo for about three months.  I rebased and I'll open this after some clean up.  It's a fully backward compatible change, so would be a minor bump. We can take it or leave it, but seems like we are in a good spot to decide with the start of this season.

nice, cool to see this landed!

i'll let @carsonfarmer do the review

i'll let @carsonfarmer do the review

Sweet, sounds good!
I have another PR incoming that has more tests, some clean up, and the addition of an alternative async/await api


[wct-mocha] Doesn't work with the latest major version of mocha (^6.0.0)

Description
When using wct-mocha with mocha version ^6.0.0, the tests start to run but then the following error is thrown at https://github.com/mochajs/mocha/blob/master/lib/reporters/html.js#L236:
TypeError: Cannot read property 'tests' of undefined

I believe this is happening because of the following breaking change in version 6:
#3458: Instantiating a Base-extending reporter without a Runner parameter will throw an exception

As part of this, statistics were refactored out into their own helper: https://github.com/mochajs/mocha/pull/3458/files
If I pull lib/stats-collector.js into wct-mocha, and then add createStatsCollector(runner); before these two lines, the tests appear to start working again:
https://github.com/Polymer/tools/blob/master/packages/wct-mocha/src/reporters/console.ts#L84
https://github.com/Polymer/tools/blob/master/packages/wct-mocha/src/reporters/title.ts#L24
Versions

wct-mocha ^1.0.0
mocha ^6.0.0 (mocha ^5.0.0 is fine)

@rictic @usergenic @azakus @justinfagnani @aomarks @FredKSchott Would it be possible to get an update on this?


Feature: Add auto log triggers for game start, and game end.

Add auto log triggers for game start, and game end. This should allow removal of the rest button, and removal of the issue that causes the debug.log to stop writing because reset was clicked.

This is the log entry for the game start event. It looks like gameId can be extracted at the same time.  Play name won't be useful as it does not include the player id.
00:00:16.803: Log]: (gameID: 'bbf1751c-0345-43d7-7705-bfcfb346f78d' realm: '2' player 0 name: 'TCG | Birdgineer', player 1 name: '<#FFD700>Trey5')
Player.OnGameSetup:337

This log entry might be the best one for finding the end of the game, you should be able to also tell who won the game by checking who's turn it was.
[00:08:28.379: Log]: Player: PlayerHuman(Clone) isMyTurn: False @ OnEndGame()
Player.OnEndGame:414

There is also this log entry for end of game. However, the above seems like a better approach.
[00:08:34.115: Log]: GameNetworkManager.StopClient: LocalPlayer won
GameNetworkManager.StopClient:28

Finally, this line also works, but does not offer who won the game like the other two lines.
[00:08:35.183: Log]: Ending the game
OnLeftGameLoading.Start:34


Do not load the deprecated view_helpers template code

fixes #130
Simple fix: Just removed the incriminated load statements.

@hatsat32: OK, I just saw that you fixed that issue already in #129 ... so I'll close this one.


fix: Skip zone_awareness_config if zone_awareness is not enabled

Description
zone_awareness_config should be considered only if zone_awareness is enabled.
Motivation and Context
When zone_awareness_enabled is false and zone_awareness_config is set (whatever the value, even null or {}), the module keeps trying to modify aws opensearch cluster and apply this zone_awareness_config. Obvisouly AWS doesn't consider it since awareness_config is only relevant when awareness is enabled. So this provokes a continuous terraform drift.
Note that we can't just remove the zone_awareness_config from our TF code since we actually uses it for some environements.
How Has This Been Tested?

I modified the module locally and there is no more a terraform drift. Module won't apply any zone_awareness_config if zone_awareness_enabled is false

This PR is included in version 1.4.1 :tada:


How to specify client key and client secret of multiple facebook and google clients  in settings.py file

Hi!  Suppose I am building and API which is consumed by an Android app(let's say MyApp) and a web application(let's say MyWeb) and I am using django-rest-framework-social-oauth2 for user login via facebook and google.
Let's suppose we have two users A and B. A logging via MyApp and B logging via MyWeb and MyApp and MyWeb both have different client id and client secret for Facebook or Google then here is problem arises
How can I specify client key and client secret of both application(MyWeb and myApp) in my settings.py file to allow both applications to use my API?
SOCIAL_AUTH_FACEBOOK_KEY and SOCIAL_AUTH_FACEBOOK_SECRET are list type or a string variables? How can I specify facbook_key and facebook_secret of both application in settings.py?

This is an issue with python-social-auth. You should check over there first and come back here if you have a solution you can implement with my package.


Use portable apps

Hi,
I've WinPython in my USB key to use it at work, and I would like to use textract on .docs.
I've read that it requires Antiword, and I've it in the key.
Is there a way to use portable apps, like Antiword in this case, with textract?

I solved it in an ugly way: i changed the doc_parser to adapt it to my environment:
    def extract(self, filename, **kwargs):' '	import os' '        dir=os.getcwd()' '        os.putenv('ANTIWORDHOME', dir[0]+':/WinPython/tools/antiword/')' '        stdout, stderr = self.run(dir[0]+':/WinPython/tools/antiword/antiword "%(filename)s" -m 8859-15' % locals())
Is there a better way?

The issue is that antiword must not be on your PATH, or at least that's what the issue would be in a UNIX environment. I'm not sure what the equivalent is in a WinPython environment. Basically, your shell interpreter can't find antiword.
The documentation for running textract in a windows environment is woefully inadequate. If you come up with a reliable way of installing antiword and the other textract dependencies, it would be really awesome if you could contribute your advice and guidance as a pull request. There are basic instructions for adding new operating system installation instructions, but even throwing something in this or another github issue would be great so other people can use textract in windows environments as well.
I'm going to close this issue and #106 in the meantime, but I'll try and keep an eye on these things and help out where I can if you have follow-up questions to either of these issues.


Documentation on modules

Is it possible to get any documentation on this https://d3-node-editor.readthedocs.io/en/latest/Extras/#modules
Also with modules is it possible to make one module say I named it test then in another block say use the module test rather than need to recreate all the blocks each time

I dont have enought time for this. Bsides it will be better to update the documentation after the implemention of the new plugin-based architecture. (Modules will also be a plugin)

Also with modules is it possible to make one module say I named it test then in another block say use the module test rather than need to recreate all the blocks each time

You can just copy Module data and pass them to new Module node

For now there are no examples for the new architecture for the modules, but I prepared an example where the main things are done inside the plug-in and you do not need to worry about the details of their work.
https://github.com/retejs/examples/tree/master/Module
The main difference is that now the data of the modules are stored in a list, rather than nested in the node data, as it was before

https://rete.readthedocs.io/en/latest/Plugins/


strange difference - please review carefully

@NilsPur - can you explain how this difference could occur?
can we safely merge this - i.e. is this the desired code in my fork:main-branch?

most changes are already in develop - closing


Current default implementation of getstate and setstate could be made safer

This is a known "sharp edge" of pickle, but attrs could make this a bit safer by slightly modifying the default implementation of __getstate__ and __setstate__. The problem is that due to returning a tuple in the default implementation of __getstate__, removing a member, and then unpickling from a previous version can be very unsafe. This is not a hypothetical situation - it commonly happens when an object is pickled, and then stored in a data-store of some sort, and unpickled some time later by a changed version of the code.
Here's a simple reproducer that demonstrates it (python 3.10, attrs 22.1.0):
import pickle

import attr

@attr.s(slots=True, hash=False, auto_attribs=True)
class Test:
    count: int
    enable_minor_feature: bool
    should_launch_missiles: bool

t = Test(count=1, should_launch_missiles=False, enable_minor_feature=True)
print(t)  # Test(count=1, enable_minor_feature=True, should_launch_missiles=False)
tp = pickle.dumps(t)

@attr.s(slots=True, hash=False, auto_attribs=True)
class Test:
    count: int
    should_launch_missiles: bool

tl = pickle.loads(tp)
print(tl)  # Test(count=1, should_launch_missiles=True)  <== an attribute is assigned a dangerously wrong value

While there's certainly an argument to be made that pickle should not be used for such things (and I'd agree) - the default implementation could, I believe, be made safer. Raise if things don't match, or even ignore unknown attributes in setstate,  by potentially returning a dictionary instead of a tuple here would help.

Have you tried replacing the tuple by the dict and see how much it breaks? Performance difference should be benign, since we save ourselves zipping.

I wanted to get some thoughts on the feasibility of making the change at all first. Let me put together a patch and see how much it breaks, and we can take it from there.

I don't think our implementation of pickling is a public API. So if you can make it compatible, that should  be OK.

I think this should now have the right coverage, and all tests seem to pass so \o/
Let me know what you think :)
Thanks!


UTM using incorrect zone identifier

If you expand the coordinate parts for a UTM coordinate the Zone is using the UTM Hemisphere Band (N or S) not the UTM Band:

We need to provide clarity on this issue.

This needs to be removed from the known issues, http://solutions.arcgis.com/defense/help/coordinate-conversion/resources/release-notes.htm before the closing label can be applied. Removing the E-verified label until this is addressed in the documentation.

NGA has a caution:
"...

http://earth-info.nga.mil/GandG/coordsys/grids/utm.html
That indicates to me that NGA has a preference for Hemisphere Zone, not a Latitudinal Band method.

At first I was thinking that we should go off of what NGA states, but their documentation shows that they support both, but with a warning from one. Checking how ArcGIS Pro does it I found that they use bands and we are using bands in our addin. I feel like this issue should be closed and we can readdresss this if a customer comes back to share this same issue.

There are no documentation tasks that I feel like need to be done on this issue.

This was list as a known issue so I added it to the issues addressed for the June 2018 release


Update hybrid-azuread-join-managed-domains.md

@deosandeep : Thanks for your contribution! The author, @MarkusVi, has been notified to review your proposed change.


Add a release series maintenance branch for v0.2.x

Add a v0.2.x release series maintenance branch that keeps v0.2.x-specific documentation and demo project. Link to it from the corresponding releases (plugin, CLI).
This should avoid confusion with the stuff on the main branch.

This makes things too complicated. Just point to the release tag's documentation or demo project.
Won't fix.


Add a private secret manager example and test

Description
A private only secrets manager example needs to be created and used in a test in the pr_test.go.
New or affected modules

By submitting this issue, you agree to follow our Code of Conduct

PR has been merged. Closing issue.


Reminder to pass at least gain and exposure from raw to radiance

Passing now possible with bde1757ef9776f6f7de331f522c5b78b6f2a6083, closing.


Update 2023-12-22-meshery-v07-release-announcement.md

Description
Fixed the internal linking and some typographical errors.
This PR fixes #There is no number to this. I found the issue and fixed it.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Way to be, @JeremiahObikoya! 👏 💯


"Getting started" documentation is missing

Hi guys, thank you for your great work.
I think that this gem need a "Getting started" section in the README to be more mainstream and to speed up adoption.
More details in the config parameters

lets_encrypt_domains must be a string separating domains by a space (it's not written anywhere);
pretty much all parameters are mandatory;
at least one server that match the parameter lets_encrypt_roles must be set as primary: true, otherwise it exits without notice (#6).

How to add to the workflow?
What Capistrano task I have to call and when?
In my humble first attempts I matched capistrano-lets-encrypt with capistrano3-nginx. Maybe it's too specific but if you find it useful, I can write a wiki page for that.

lol, I realized later that even capistrano3-nginx is yours... :-)

could you please make a PR with the changes you consider would help you??

yes, we also use capistrano3-nginx and we pass variables from one to another...
something like this:
# deploy/production.rb

set :nginx_ssl_domains, 'foo.mydomain.com'
set :nginx_domains, 'foo.mydomain.com'

set :nginx_ssl_certificate_path, -> {
  File.join(fetch(:lets_encrypt_output_path), fetch(:nginx_ssl_domains))
}
set :nginx_ssl_certificate_key_path, -> {
  File.join(fetch(:lets_encrypt_output_path), fetch(:nginx_ssl_domains))
}

set :nginx_ssl_certificate, 'fullchain.pem'
set :nginx_ssl_certificate_key, 'key.pem'

set :lets_encrypt_email<EMAIL_ADDRESS>set :lets_encrypt_domains, -> { fetch(:nginx_ssl_domains) }

Note: this work only for 1 ssl domain... but would be easy to tweak it

I will close this, please open if needed


bootstrap4 is incompatible with bootstrap-table

Neither the pagination bar nor the label that displays the number of rows will display properly

You should report this there.


Buy a Trash Can

Purchase or make a trash can that can support the hardware and function according to the project.


Windows Notifications

This is now ready for review.

My :european_castle: for anyone who can tell me how we can link WindowsCreateStringReference (and its friends).
FAILED: C:\Python27\python.exe gyp-win-tool link-with-manifests environment.x86 True electron.exe "C:\Python27\python.exe gyp-win-tool link-wrapper environment.x86 False link.exe /nologo /OUT:electron.exe @electron.exe.rsp" 1 mt.exe rc.exe "obj\electron.electron.exe.intermediate.manifest" obj\electron.electron.exe.generated.manifest ..\..\atom\browser\resources\win\atom.manifest
brightray.lib(brightray.notification_presenter_win.obj) : error LNK2019: unresolved external symbol __imp__RoGetActivationFactory@12 referenced in function "long __cdecl Windows::Foundation::GetActivationFactory<struct ABI::Windows::UI::Notifications::IToastNotificationFactory>(struct HSTRING__ *,struct ABI::Windows::UI::Notifications::IToastNotificationFactory * *)" (??$GetActivationFactory@UIToastNotificationFactory@Notifications@UI@Windows@ABI@@@Foundation@Windows@@YAJPAUHSTRING__@@PAPAUIToastNotificationFactory@Notifications@UI@1ABI@@@Z)
brightray.lib(brightray.notification_presenter_win.obj) : error LNK2019: unresolved external symbol _WindowsCreateStringReference@16 referenced in function "public: __thiscall StringReferenceWrapper::StringReferenceWrapper<4>(wchar_t const (&)[4])" (??$?0$03@StringReferenceWrapper@@QAE@AAY03$$CB_W@Z)
brightray.lib(brightray.notification_presenter_win.obj) : error LNK2019: unresolved external symbol _WindowsDeleteString@4 referenced in function "public: __thiscall StringReferenceWrapper::~StringReferenceWrapper(void)" (??1StringReferenceWrapper@@QAE@XZ)
electron.exe : fatal error LNK1120: 3 unresolved externals
Traceback (most recent call last):
  File "gyp-win-tool", line 316, in <module>
    sys.exit(main(sys.argv[1:]))
  File "gyp-win-tool", line 29, in main
    exit_code = executor.Dispatch(args)
  File "gyp-win-tool", line 71, in Dispatch
    return getattr(self, method)(*args[1:])
  File "gyp-win-tool", line 169, in ExecLinkWithManifests
    subprocess.check_call(ldcmd + add_to_ld)
  File "C:\Python27\lib\subprocess.py", line 511, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'C:\Python27\python.exe gyp-win-tool link-wrapper environment.x86 False link.exe /nologo /OUT:electron.exe @electron.exe.rsp electron.exe.manifest.res' returned non-zero exit status 1120
ninja: build stopped: subcommand failed.

Felix and I added the lib that you had found to brightray.gyp (line 196):

       'VCLinkerTool': {

                'AdditionalDependencies': [
                  'advapi32.lib',
                  'dbghelp.lib',
                  'delayimp.lib',
                  'dwmapi.lib',
                  'gdi32.lib',
                  'netapi32.lib',
                  'oleacc.lib',
                  'powrprof.lib',
                  'user32.lib',
                  'usp10.lib',
                  'version.lib',
                  'winspool.lib',
   ---->          'windowsapp.lib',
                ],

But still the same linking errors. Any way to get better console out from gyp/ninja?

@jpoon Good question - maybe we're linking against an older SDK?

For the new added libraries, you probably need to add them as global link settings:
        ['OS=="win"', {
          'msvs_settings': {
            'VCLinkerTool': {
              'AdditionalDependencies': [
                'runtimeobject.lib',
              ],
            },
          },
          'conditions':  [
            ['libchromiumcontent_component', {

Otherwise they will only be linked in Release build.

Quick update for everyone following along: It's building, just not working like we want it to. But we're getting closer.

We have native notifications! :fireworks:  This is what I see on Windows 10:

Still some stuff left to do:

[ ] Set the body/text of the toast
[ ] Handle callbacks
[ ] Verify behaviour on Win8, Win7

Rebased and added some stuff back - we now have basic notifications in.

@paulcbetts @zcbenz @kevinsawicki :stars: :fireworks: We think that we're done here. I know there's a big merge button right on this page, but I'd never merge it without y'all taking a look and giving me expressed consent.
If you want to try this puppy out, run the following:
var windowsNotification = new Notification("Test Title", {
 	body: "Hi, I'm an example body. How are you?",
 	icon: "file:///C:/Path/To/Your/Image.png"
});

windowsNotification.onshow  = function () { console.log("Notification shown") };
windowsNotification.onclick = function () { console.log("Notification clicked") };
windowsNotification.onclose = function () { console.log("Notification dismissed") };

The obvious next step is some documentation, which we'll also gladly cover.

Need to add a runtime check to see if you're on Windows 7 or else you'll segfault

Thanks for the comments @paulcbetts. Pushed an update to address the concerns you brought up.

Nice! Thanks everyone (especially @paulcbetts and @jpoon). Code review fixes are in, commits are squashed. Ready for merge?

:+1:

@felixrieseberg This is still going to crash hard on Windows 7 if anyone uses it. Can you submit another PR to make sure that we no-op this?

You can use WRL::RuntimeClass:
class ToastEventHandler : public RuntimeClass<WRL::RuntimeClassFlags<WRL::ClassicCom>, DesktopToastActivatedEventHandler, DesktopToastDismissedEventHandler>
go the the implementation of AddRef, Release and QueryInterface generated automatically.

https://github.com/atom/brightray/pull/166

I also don't think it's a good idea to keep a raw pointer to a COM object, which is probably gets AddRef-ed by IToastNotification when you pass the ToastEventHandler instance to the add_Activated and add_Dismissed methods. It should be stored in a WRL::ComPtr instead.

It looks like the implementation is based on https://code.msdn.microsoft.com/windowsdesktop/sending-toast-notifications-71e230a2, in that case I also wonder why you didn't keep StringReferenceWrapper. Why do you keep calling WindowsDeleteString manually every time?

Actually, according to MSDN:

You don't need to call the WindowsDeleteString function to de-allocate a fast-pass HSTRINGcreated by the WindowsCreateStringReference function.

To make sure that it does not break Windows 7, you can do something like this:
typedef decltype(&RoGetActivationFactory) RoGetActivationFactoryPtr;
typedef decltype(&RoInitialize) RoInitializePtr;
typedef decltype(&RoUninitialize) RoUninitializePtr;
typedef decltype(&WindowsCreateStringReference) WindowsCreateStringReferencePtr;

HRESULT WINAPI RoGetActivationFactory(HSTRING activatableClassId, REFIID iid, void ** factory)
{
    static auto hLib = LoadLibrary(L"api-ms-win-core-winrt-l1-1-0.dll");
    static auto func = reinterpret_cast<RoGetActivationFactoryPtr>(GetProcAddress(hLib, "RoGetActivationFactory"));

    return func ? func(activatableClassId, iid, factory) : E_FAIL;
}

HRESULT WINAPI RoInitialize(RO_INIT_TYPE initType)
{
    static auto hLib = LoadLibrary(L"api-ms-win-core-winrt-l1-1-0.dll");
    static auto func = reinterpret_cast<RoInitializePtr>(GetProcAddress(hLib, "RoInitialize"));

    return func ? func(initType) : E_FAIL;
}

void WINAPI RoUninitialize()
{
    static auto hLib = LoadLibrary(L"api-ms-win-core-winrt-l1-1-0.dll");
    static auto func = reinterpret_cast<RoUninitializePtr>(GetProcAddress(hLib, "RoUninitialize"));

    return func ? func() : (void)0;
}

STDAPI WindowsCreateStringReference(PCWSTR sourceString, UINT32 length, HSTRING_HEADER * hstringHeader, HSTRING * string)
{
    static auto hLib = LoadLibrary(L"api-ms-win-core-winrt-string-l1-1-0.dll");
    static auto func = reinterpret_cast<WindowsCreateStringReferencePtr>(GetProcAddress(hLib, "WindowsCreateStringReference"));

    return func ? func(sourceString, length, hstringHeader, string) : E_FAIL;
}

To fix inconsistent DLL linkage errors you have to #define _ROAPI_ before including wrl.h
roapi.h:

...
#ifdef _ROAPI_
#define ROAPI
#else
#define ROAPI DECLSPEC_IMPORT
#endif // _ROAPI_
...

Oh, I see. You are using WindowsCreateString instead of WindowsCreateStringReference, which would have been more efficient.


Date Picker in Figma

Consider adding hour format to both XD and figma for localization of most common.


Added compression, better emitter /w tests

Pull Request Template
Please try to complete the below as best as possible. Some of the fields
may not be necessary so feel free to add or edit as you see fit.
Checklist
Confirm you have completed the following actions prior to submitting this PR.

[X] There is an existing issue report for this PR. (Existing PR)
[X] I have forked this project.
[X] I have created a feature branch.
[X] My changes have been committed.
[X] I have pushed my changes to the branch.

Title
Added compression as an option, created a fully documented, better emitter, and created tests for said emitter.
Description
Decompression was changed so that this time we create a decompression object, and use it repeatedly to decompress information. A new emitter was added that supports priority, a bit more efficient and is documented. Lastly created tests for the the created emitter.
Issue Resolution
Doesn't fix, just enhances
This Pull Request fixes #3
New or Changed Features

Added decompression support
New event emitter
event emitter tests

NO NOT MERGE: Still a bug with the gateway reconnect test that needs to be resolved.

Ok, thanks for your effort. Just hit me up on Discord when this is ready to merge


Introduce epoch-millis timestamp format

What is the purpose of the change
This PR introduces the epoch-millis timestamp format to represent the number of milliseconds since epoch.
Brief change log

Introduce UnixTimestampMillis and FromUnixTimestampMillis.
Support epoch-millis timestamp format

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API: yes

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? Python docstring

@lindong28 Could you review this PR?

@lindong28 Thanks for your review! I updated the PR accordingly. Could you please take another look?

Thanks for the updater. LGTM.


Decision Tree Process Documentation

The decision tree section may benefit from adding a markdown paragraph explaining why the method was chosen as an effective classifier

Fixed, the documentation was added


juliaup fails to build on Alpine Linux

I am trying to build juliaup from source under Alpine Linux (edge) in Docker on a Fedora host as discussed at https://github.com/JuliaLang/julia/issues/52279
Steps taken:
docker run -it --rm alpine:edge
apk add rust cargo git
git clone https://github.com/JuliaLang/juliaup
cd juliaup
RUST_BACKTRACE=1 cargo build

Result:
error: failed to run custom build command for `juliaup v1.12.5 (/juliaup)`
note: To improve backtraces for build dependencies, set the CARGO_PROFILE_DEV_BUILD_OVERRIDE_DEBUG=true environment variable to enable debug information generation.

Caused by:
  process didn't exit successfully: `/juliaup/target/debug/build/juliaup-3d1a5dc242035853/build-script-build` (exit status: 101)
  --- stderr
  thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', build.rs:27:47
  stack backtrace:
     0: rust_begin_unwind
     1: core::panicking::panic_fmt
     2: core::result::unwrap_failed
     3: core::result::Result<T,E>::unwrap
     4: build_script_build::main
     5: core::ops::function::FnOnce::call_once
  note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
warning: build failed, waiting for other jobs to finish...

The file system on the Fedora host is btrfs. I have seen similar issues open here, but none identical, feel free to close this issue if it is a duplicate.

The issue here was using apk add rust to install Rust, which resulted in a vendor string being set in the host triple instead of vendor unknown. Installing Rust with apk add rustup and then running rustup-init resulted in a working installation of Rust that built a working juliaup binary, further good evidence that Julia should probably use the same approach for installation.


terminal-typeracer: 2.0.2 -> 2.0.4

Motivation for this change
Update to latest release.
https://gitlab.com/ttyperacer/terminal-typeracer/-/releases/v2.0.4
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 108061 run on x86_64-darwin 1

1 package built:

terminal-typeracer

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108061 run on x86_64-linux 1

1 package built:

terminal-typeracer


Do not use SO_REUSEPORT on WSL

In PR https://github.com/IBM-Swift/BlueSocket/pull/61 (https://github.com/IBM-Swift/BlueSocket/pull/61/commits/e639a3fd5e15445f32319f26936f0daf5fea0ca3), use of the SO_REUSEPORT socket option was added. This however causes issues on Windows Subsystem for Linux (WSL), which does not (and will likely never) support SO_REUSEPORT.
Strace shows the following:
setsockopt(15, SOL_SOCKET, SO_REUSEPORT, [1], 4) = -1 ENOPROTOOPT (Protocol not available)

By the way, SO_REUSEADDR works without issue on WSL:
setsockopt(15, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0

Please consider adding a way to disable the use of SO_REUSEPORT, or do not fail with an exception (but rather silently) when the call to setsockopt fails with ENOPROTOOPT (at least on Linux). Unfortunately it is not possible to detect WSL at compile-time, as the userland is actually Ubuntu (or other Linux). A third option would be to perform detection at runtime (or perform some other check to see whether SO_REUSEPORT is available) but then again, that is probably more complicated than simply ignoring an ENOPROTOOPT error.

Fixed in the latest release, 0.12.57.  It now will fail silently should the errno be set to ENOPROTOOPT.  Thanks.


find elements using :lang(...) pseudo-class selector throws exception

I'm trying to find elements using a :lang(...) pseudo-class selector syntax, but my program throws an exception, maybe the selector has not been supported in jsoup?
Exception in thread "main" org.jsoup.select.Selector$SelectorParseException: Could not parse query ':lang(jv-Java-ID)': unexpected token at ':lang(jv-Java-ID)'
        at org.jsoup.select.QueryParser.findElements(QueryParser.java:209)
        at org.jsoup.select.QueryParser.parse(QueryParser.java:62)
        at org.jsoup.select.QueryParser.parse(QueryParser.java:45)
        at org.jsoup.select.Selector.select(Selector.java:92)
        at org.jsoup.nodes.Element.select(Element.java:396)
        at CariKarakterTerpakai$DaftarKarakter.<init>(CariKarakterTerpakai.java:67)
        at CariKarakterTerpakai.main(CariKarakterTerpakai.java:25)

I along with my group will be fixing this issue in this semester.

Maybe :lang() is not support in Jsoup, because it is not included in the cookbook.

I along with my group will be fixing this issue in this semester.

@QAQGaeBolg : I am reaching out to you here because you have no contact information in your profile. Sorry, I don't know who you or your group are, and the commits you've been sending are not production ready. I am I am not in the place to do class tutoring for you if that is your expectation.
Please stop commenting on issues in a way that is appearing to either be officially confirming a bug or are assigning the issue to yourself and perhaps making others hesitant to contribute.
I welcome production ready commits or those approaching it, but you and your group (who, specifically?) have not been making those, and so far it's just been clutter.
On this issue: I would just recommend selecting off the attribute value vs using the pseudo-selector. It's not currently supported and I haven't seen much demand to add it. I'm not against the feature if there's a good quality PR, though.


fix README for mpas-bundle

TYPE: bug fix
KEYWORDS: mpas-bundle, README
Jamie noticed that the README instructions are out of date, asking users to  "cd mpasjedi" before running ctest.  This replaces "mpasjedi" with "mpas-jedi" in two places.
LIST OF MODIFIED FILES: README

Closing and will open another PR with content like that of JCSDA/fv3-bundle.

Yes, we should decide on what we want to do here before we merge because after we merge, we may want to define a new tag of 1.0.1.
I see another problem - the default build type is not release; it is RelWithDebInfo.
As we see here, the disadvantage of putting build instructions in a README is that one has to remember to keep them up to date.   I would recommend linking to the generic build instructions for a bundle and the build tutorial for general build instructions.  But, it still may be good to mention mpas-specific things here, like the use of the env-setup scripts.

@mmiesch, I didn't see your comment before closing this PR and opening another with a README.md like that for fv3-bundle.
In that PR, I originally had a link to the JEDI-MPAS build tutorial, as you suggest.  But I had a confidence crisis about how to link to specific pages in the documentation.  Is it sufficient to just navigate to the desired page, then cut and paste the address from the browser bar?  (Like: https://jointcenterforsatellitedataassimilation-jedi-docs.readthedocs-hosted.com/en/latest/learning/tutorials/level2/dev-container.html)

@mmiesch, I have several questions.

I see another problem - the default build type is not release; it is RelWithDebInfo.

In the CMakeLists.txt file of master branch, the default build type is set as Release. https://github.com/JCSDA/mpas-bundle/blob/7aa2bd3c19e677131189858c5cf78450216848a9/CMakeLists.txt#L16
Do you mean that this should be RelWithDebInfo ?
Also, in the JEDI-MPAS build tutorial, the command line instruction uses ecbuild --build=Release ../mpas-bundle.
If we want to define a new tag, I'd like to remove following two lines (rttov-related) because they are not available to public users.
https://github.com/JCSDA/mpas-bundle/blob/7aa2bd3c19e677131189858c5cf78450216848a9/CMakeLists.txt#L35-L36

@byoung-joo - you are correct - I did not look closely.  The default build type for ecbuild is RelWithDebInfo but I see this is overridden by that line in mpas-bundle.  So, disregard my comment.  The default build type is release and that is fine.
I'm ok with removing rttov.  I think we kept it in because external users might want to use it but even if they have the code they don't have the JCSDA repo and there are no instructions on how to implement it.  So, I think it's best to leave it out.  For advanced users who want to use it, we can guide them on the forum.

@mmiesch, I didn't see your comment before closing this PR and opening another with a README.md like that for fv3-bundle.
In that PR, I originally had a link to the JEDI-MPAS build tutorial, as you suggest. But I had a confidence crisis about how to link to specific pages in the documentation. Is it sufficient to just navigate to the desired page, then cut and paste the address from the browser bar? (Like: https://jointcenterforsatellitedataassimilation-jedi-docs.readthedocs-hosted.com/en/latest/learning/tutorials/level2/dev-container.html)

Either way - a minimal readme like for fv3-bundle works too.  And yes, you can link to specific pages as you say.   For the master branch it might be better to replace "latest" with the "1.2.0" because the former is always changing, eg:  https://jointcenterforsatellitedataassimilation-jedi-docs.readthedocs-hosted.com/en/1.2.0/learning/tutorials/level2/dev-container.html.   And, as I have said before, I think it generally better to link to the top-level tutorials page than the specific tutorial.  But, in some contexts, like here where you're talking specifically about how to build mpas-bundle, then it might be appropriate to link the specific bundle.
All of this is a judgement call and we can accommodate whatever you want to do - as long at it is accurate!


📝 Add a section on "format on save" to README

Hi there,
I was wondering how to enable format on save and I found how. So I updated the README :)

Thanks!


crypto/x509: Certificate.Verify does not consistently return UnknownAuthorityError

What version of Go are you using (go version)?

$ go version
go version go1.19.6 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/rittneje/.cache/go-build"
GOENV="/home/rittneje/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/rittneje/go-workspace/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/rittneje/go-workspace"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/jrittner/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/rittneje/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.19.6"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/rittneje/go-workspace/src/certtest/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3903595383=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I made three certificates - root, intermediate, and leaf. The intermediate is already expired. I then ran Certificate.Verify on the leaf certificate.
https://go.dev/play/p/WIWEm7L0Bel
What did you expect to see?
I expected to get back an x509.CertificateInvalidError wrapped inside an x509.UnknownAuthorityError.
What did you see instead?
I got back an unwrapped x509.CertificateInvalidError. Consequently, the resulting error message has insufficient context.
This is due to a scoping/shadowing bug in Certificate.buildChains.
https://github.com/golang/go/blob/dd16258f48bc9e328ef2086a49da0432fe46f413/src/crypto/x509/verify.go#L926-L933
That err is the named return value from the outer scope. So at the end of the function, because it is non-nil, it is never wrapped.
https://github.com/golang/go/blob/dd16258f48bc9e328ef2086a49da0432fe46f413/src/crypto/x509/verify.go#L955-L957

Probably we should (a) always return a UnknownAuthorityError from Verify, wrapping errors discovered during chain building and (b) implement Unwrap on UnknownAuthorityError so that the hint can be accessed, since it is essentially a wrapped error.
There are currently cases where we expect to return a different error type though (as indicated by the fact we explicitly test for those cases). Perhaps it's reasonable to break those, especially if we implement Unwrap, but it is worth consideration.


Implement upstream echo response

Hallo,
I am trying to implement auto reply echo request from the ISO host. When the ISO host send echo request I got this upstream error:
2018-03-26 16:47:50 - info: Upstream error: ISO host sent data with no client connected! Ignoring...

ISO host says
================================================================================================

     [Echo Request]

     Message Type Indicator [0].......................0800
     Bitmap [1].......................................82200000000000000400000000000000
     Transmission Date and Time [7]...................0326094750
     System Trace Audit Number [11]...................018386
     Network Management Information Code [70].........301

================================================================================================

It is not the same.

Done.

Hello,
I have an error like that picture. Im trying to echo test

Hi!
Please refer to this line of code: https://github.com/juks/SocketQueue/blob/ddbab6578559cd2ff2191c03615ea873f95fe9d3/lib/iso8583/lib/packager/smartVista.js#L21
There are some basic sets of fields that SQ expects for every known message type.

Thank you i have change that code.
But when i run my code in port 80, the server listen in tcp6 . and when i send the data i got error like this

What happens is the upstream host closes the connection after it receives the message. This might be due to the fact it expects different message format or due to some other reason else. Checking the upstream log might help.

How to Checking the upstream log ?

But when i run the server, host to host connection refused to my server. But can access my ip server in their browser.
Host to host say they dont receive the message. The format that I made was in accordance with what the host to host needed and in accordance with the documentation

What you might check:

Do they have a connection even in their log;
Is there any firewall in between;
What happens if you send the payload using netcat like it is mentioned here https://github.com/juks/SocketQueue/blob/master/sample_payloads/readme.md;

Hi,

How to check connection in their log?
Yes, there is a firewall in between
To test the payload i just need to run that command line?

Thank you

The Host To Host run aplikasi visual basic, they dont have port? so how can i connect?
example if my port 80 listend as apache2 they can connect. but if i stop my apache and run the socket queue they can;t connet

You should not use privileged port numbers (under 1024).
I am sorry, but I can't help with your specific setup. You need to engineer it yourself.

why i can't use privileged port numbers?


Add VSS Processor Plugin

Added a new composite module which provides the new vss-processor-plugin. It is mainly independent of the whole project. It has its own settings.gradle (not needed yet / default is used) etc. Which means some gradle configurations will be redundant.
Documentation:
Gradle Plugins: https://docs.gradle.org/current/userguide/custom_plugins.html
Disable signing for publishToMavenLocal: https://docs.gradle.org/current/userguide/signing_plugin.html
Task dependencies with composite builds: https://docs.gradle.org/current/userguide/composite_builds.html
Test:
Deploy: https://github.com/eclipse-kuksa/kuksa-android-sdk/actions/runs/7726264784
See documentation for how to test it with mavenLocal + Companion app. Same setup as with the SDK TestApp.
The following should now work and the workaround inside the KSP module was removed.
./gradlew clean kspDebugKotlin
close #45

Scenario: Add a VSS Property to VSS Spec file
-> ./gradlew clean build -> OK: Property is correctly generated -> ./gradlew build -> NOK: Property is not correctly generated because task is up-to-date. That's probably the best we can do for now, because there is no clear dependency between our Specification and the ksp*Kotlin task. Could be solved when the files are generated inside the Plugin?
Scenario: Changing Version
-> ./gradlew setSnapshotVersion -> OK -> ./gradlew setReleaseVersion -> OK
Scenario: vssProcessor.searchPath does not exist
-> Android Studio Gradle Synch -> OK -> ./gradlew clean build -> OK -> minor improvements with the message to be done, see corresponding finding
Scenario: Create Dash Report
-> ./gradlew clean mergeDashFiles -> OK
=> Function-wise OK!

For the NOK I also think this can only properly solved when we move the generation code aways from KSP. Known issue for now.

Added tests for the vss processor plugin. Used the following guide:
https://docs.gradle.org/current/userguide/test_kit.html#sub:test-kit-build-cache
Use the following command to test or execute the Android Studio run config:
./gradlew vss-processor-plugin:test
Note:

Debugging into the tests does not work here.
Sometimes tests errors are obscured so I recommend opening the test results file.
Everything was implemented in kotlin so the recommended "spock" framework was not necessary.

./gradlew clean publishToMavenLocal-> NOK (Unable to make progress running work. There are items queued for execution but none of them can be started)
./gradlew :vss-processor-plugin:clean :vss-processor-plugin:publishToMavenLocal-> NOK (Execution failed for task ':vss-processor-plugin:generateMetadataFileForPluginMavenPublication'.

java.io.FileNotFoundException: /Users/wba2hi/git/kuksa-android-sdk/vss-processor-plugin/build/libs/vss-processor-plugin-0.1.3-SNAPSHOT.jar (No such file or directory))

> cd vss-processor-plugin
> gradle wrapper # add gradle wrapper; requires gradle being installed
> ./gradlew wrapper --gradle-version=8.6 # use same version as in sdk rootProject
> ./gradlew clean publishToMavenLocal

-> OK
The composite linking of clean seems to break something? Maybe the composite links are not adequately "planned into the dependency graph"?
Without clean everything works fine. In this particular case I would be ok with not executing clean, but the minimum I expect is a follow-up ticket. Also it mean that if more of these issues appear we have to remove the composite links again and handle the project as a "real and independent rootProject" (which I also would be fine with, because I don't expect many changes in this project over time)

Created an issue for the first problem: https://github.com/eclipse-kuksa/kuksa-android-sdk/issues/80
Fixed the java.io.FileNotFoundException


Update the Upgrading Minikube topic

Related to #19861
This PR update the Upgrading Minikube topic.

/assign @tengqm

/lgtm
/approve


Removed config-demo|hello-world applications

Removed config-demo & hello-world resources from values-hub.yaml
Removed config-demo & hello-world charts from git path

PR to wrong location


Issue loading Object Color

Hi,
I've started using it, Played around with the github examples, files with .vrx seems working fine.
Faced an issue loading .obj files but this answer helped me to render obj files.
But i bumped into a small issue regarding the color of the object which is rendered.
Sent an email with .mlt, .obj and textures files to<EMAIL_ADDRESS>This is the screenshot where the color of chair is red rather than grey(according to the  textures provided).

This is the code(AR placing objects sample in github except that its in Kotlin).
        mScene?.setListener(object: ARSceneListener(this, viroViewARCore){
            override fun onARInitialized() {
                trackingText.visibility = View.VISIBLE
                trackingText.setText(R.string.ar_is_intialised)
                addObject.visibility = View.VISIBLE
            }
        })
        // Add a light to the scene so our models show up
        mScene?.rootNode?.addLight(AmbientLight(Color.WHITE.toLong(), 1000f))

        viroViewARCore?.setScene(mScene)

And here is the 3DDraggableObject.kt
fun addModelToPosition(position: Vector,arScene: ARScene,type:Object3D.Type) {
        val object3D = Object3D()
        object3D.setPosition(position)
        // Shrink the objects as the original size is too large.
        object3D.setScale(Vector(.2f, .2f, .2f))
        object3D.gestureRotateListener = GestureRotateListener { i, node, rotation, rotateState ->
            if (rotateState == RotateState.ROTATE_START) {
                rotateStart = object3D.rotationEulerRealtime.y
            }
            val totalRotationY = rotateStart + rotation
            object3D.setRotation(Vector(0f, totalRotationY, 0f))
        }

        object3D.gesturePinchListener = GesturePinchListener { i, node, scale, pinchState ->
            if (pinchState == PinchState.PINCH_START) {
                scaleStart = object3D.scaleRealtime.x
            } else {
                object3D.setScale(Vector(scaleStart * scale, scaleStart * scale, scaleStart * scale))
            }
        }

        object3D.dragListener = DragListener { i, node, vector, vector1 -> }

        // Load the Android model asynchronously.
        object3D.loadModel(Uri.parse(mFileName), type, object : AsyncObject3DListener {
            override fun onObject3DLoaded(`object`: Object3D, type: Object3D.Type) {

            }

            override fun onObject3DFailed(s: String) {
                Toast.makeText(context, "An error occured when loading the 3D Object!",
                        Toast.LENGTH_LONG).show()
            }
        })

        // Make the object draggable.
        object3D.dragType = Node.DragType.FIXED_TO_WORLD
        arScene.rootNode.addChildNode(object3D)
    }

I'm calling the placeObject this way:
placeObject("file:///android_asset/{myobject}.obj",Object3D.Type.OBJ)
Am i missing anything here?

Hi @manikantagarikipati, we took a look at the OBJ and the couch is actually red. Material color and illumination are controlled by the mtl file and the color was set as red. We modified the chair to grey (Kd 0.5000 0.5000 0.5000) but left the pillow as red (Kd 0.7804 0.0000 0.0000) by modifying the mtl. I will email that to you.

Also, to see more details of the couch you might want to add a Directional or Omni light to your scene in addition to the Ambient light you already have.

The edited .mtl file( with colors changed ) works!!. Thanks @dam00n


Improvements

This PRs:

Adds a feature similar to the one we will need to create in starter.
Use   "base": "micronaut-data-jdbc-repository" to generate a sample project with the starting point we tell the user.

I have moved parts of the contents of this PR to this other:
https://github.com/micronaut-projects/micronaut-guides/pull/1103

sorry, I have accidentally merged this. Let me know if you have any concerns.
I have changed AZURE_CLIENT_ID to AZURE-CLIENT-ID. However, it was written as AZURE_CLIENT_ID in your original commits.

I have no additional concerns.


Adding PIP package exports

PIP package is missing exports. Code examples fails with errors like:
 AttributeError: module 'hplib' has no attribute 'load_database'

@l-kotzur is this project still being maintained?


Converting icon pack to React Native compatible package

Hi, we need the icons to be able to be reused in React native projects.
The icons are available to be used in HTML by using CDN and using npm package.

can you please assign me this?

Hi @AshutoshDash1999, I would like to work on this issue. I know HTML, CSS and javascript, and react js. Plz, assign it to me.

Hi @rafiya2003, thanks for commenting. Please go ahead. Make sure to create a new branch from dev and submit the PR against it.

hi @rafiya2003 any update on this issue?

I am working on this

@rafiya2003 Can you please give any update on this?

I am still working

Unassigning current assignee due to inactivity and failing to give proper update.

@abhi03ruchi can you work on this?


RuntimeError: CUDA out of memory. happend in train.py#cal_loss#

there is a memory problem in train.py#cal_loss# with batce_size is 128.
code:one_hot = one_hot * (1 - eps) + (1 - one_hot) * eps / (n_class - 1)
please help me,thanks.

I have met CUDA error: out of memory too, even though I decreased the batch_size to 32. I think the training process with cuda have some little bugs and I cannot figure it out.

@huangxianliang @Elenore1997
Please take a look on the newest code.
I tried to reduce all the contiguous operations and temporary variables as possible.
But for me, batch size 256 is the limit. (the paper used 2048, which is crazy.)
Hope it helps.


SVMTransactionAdapter

Problem

Need to comply with legacy interfaces for geyser and poh

Summary of Changes

Trait to abstract transition to the legacy types

Fixes #

Converted to draft - removed review requests.
This PR will be on hold until after RuntimeTransaction is in place and we can actually start using the SVMTransactionAdapter trait where it is necessary.

Got folded into #3041


Need for clarification

Hello,
I must say your representation is a bit hard to follow for a novice :D
When you look at this page : https://storage.googleapis.com/demos.webmproject.org/webp/cmp/2024_01_25/index.html?load=jpegxl_vs_avif.json#ref=AVIF+speed+6
avif seems to have the lead over jpegxl at every efforts
When you look at this one : https://storage.googleapis.com/demos.webmproject.org/webp/cmp/2024_01_25/index.html?load=all.json#WebP+method+4-quality=95..95&matcher_ssim=off
The tendency is flipped, with jpegxl taking the lead ?
Could you explain how to use your site to have a good idea on the relative performance of both codec ?
Also, it is difficult to get an idea of the quality selected as reference, perhaps a link to an image at this quality could help showing the level of distortion obtained (I guess < 0.1bpp can not seriously be used for display)

Thank you for the feedback!

I must say your representation is a bit hard to follow for a novice :D

I agree and I heard that multiple times already. I will try to improve that side of the framework at some point.

When you look at this page : https://storage.googleapis.com/demos.webmproject.org/webp/cmp/2024_01_25/index.html?load=jpegxl_vs_avif.json#ref=AVIF+speed+6
avif seems to have the lead over jpegxl at every efforts
When you look at this one : https://storage.googleapis.com/demos.webmproject.org/webp/cmp/2024_01_25/index.html?load=all.json#WebP+method+4-quality=95..95&matcher_ssim=off
The tendency is flipped, with jpegxl taking the lead ?

You are correct. As for any data visualization tool, the results can greatly vary depending on the constraints applied to the comparison. One can easily skew the findings in one or another way, intentionally or inadvertently.
I noticed that your second plot only has 2521 comparisons, which may signal low confidence in the results due to the lack of data points.

Could you explain how to use your site to have a good idea on the relative performance of both codec ?

Sure, if you specify relative to what.
The goal of this framework is not to prove that "codec X is better". The purpose is showing that "codec X is A% faster to encode, B% slower to decode, generates files C% smaller than codec Y, for the images in this data set with the same metric value, on this quality range etc."
Having stronger corpus or environment assumptions could be misleading.
I tried to minimize any bias in the default settings:

Two modern codecs AVIF and JPEG XL, both compared to WebP which has been stable in both format and implementation for years now.
The effort/speed/method encoding settings (meaning how much time/CPU the user would like to spend for encoding an image) is the same as the default in the binaries of the main codec implementations (cwebp, avifenc, cjxl).
The full input quality setting ranges are used.
Comparing bytes and timings is easier to reason with: "twice faster" and "20% smaller" are more accessible than "better looking by 2dB". That is why comparisons are made for the same visual quality, according to some objective metric.
There is no consensus on the objective metric to rely on. SSIM favors AVIF and SSIMULACRA2 favors JPEG XL. To avoid picking a side, I enabled both by default. Unfortunately this approach reduces the total number of comparisons and probably skews the plot in its own way, but this is somewhat compensated by the fact that the interactive tool allows for easy tuning of matchers and metrics to fit one's needs or convictions.
Subjective visual quality evaluation (aka user surveys) could settle the debate for a given corpus and settings. Unfortunately this is rather involved.

Also, it is difficult to get an idea of the quality selected as reference, perhaps a link to an image at this quality could help showing the level of distortion obtained

This heavily depends on the image unfortunately. I agree that showing one image at a given quality setting or bpp would give a sense of the subjective visual quality loss, but it would also be misleading for other images that may require a different amount of quantization to achieve the same "loss" (if such a definition exists).
In codec-compare one can see the images by clicking on a small dot, scrolling down and clicking on the image below. I was thinking about making this more visible.

I guess < 0.1bpp can not seriously be used for display

From the <0.1bpp plot:
Top left point: bpp < 0.03
Right-most point: bpp < 0.03
Not super crisp, but can be displayed as a background for example.

Which SSIM code did you use ?

It is written here:

libwebp2 quality metrics command
Command line to get libwebp2 quality metrics PSNR and SSIM:
git clone https://chromium.googlesource.com/codecs/libwebp2
cd libwebp2
git checkout e0c6533
cd ..
cmake -S libwebp2 -B libwebp2/build -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
cmake --build libwebp2/build --parallel
libwebp2/build/get_disto -ssim ${encoded_name}.png ${original_name}

So it would be https://chromium.googlesource.com/codecs/libwebp2/+/e0c6533107649063c236b5666f2cc4c10b9b7591/src/enc/distortion_enc.cc#418

I agree there is no consensus about the metric, yet ssimulacra2 is the only one which was fitted on subjective data (which does not make it perfect everywhere but still)

The issue with picking any single metric by default is that it is unfair to all codecs that do not optimize for that in their Rate-Distortion optimization.
libavif uses aom's SSIM by default but allows for PSNR to be used.
libwebp uses PSNR for RD-opt.
Do you know if libjxl supports tuning the metric used for RD-opt? It uses SSIMULACRA2.1 by default right?

I'm not sure but I believe it uses the butteraugli metric as an internal target (hence a clear focus on high quality compression).
If you want to know more I invite you to jump on the discord :) ( link here: https://jpegxl.info/ )
Many core developers are there, even some from google (Zürich office) and even one which worked on the webp lossless codec :) and they all enjoy technical discussions

The 2024_03_25 batch uses SSIM and Butteraugli as discussed above. Feel free to reopen this thread if necessary.


Getting intent uri data in root destination's viewmodel

In Activity I get the as part of bundle in intent.
Then from there I launch a screen, which is root destination.
@RootNavGraph(start = true)
@Destination
@Composable
fun Screen(
    navigator: DestinationsNavigator,
    vm: VM= hiltViewModel()
)

Here I want to get that intent data in VM
How can I get that in as part of savedStateHandle in VM

Hi @UKMIITB !
I would simply pass it as non-navigation argument.
Check both ways to do it here:
https://composedestinations.rafaelcosta.xyz/destination-arguments/navhost-level-parameters
Or, using the manual composable alternative, you can get a hold of your hiltViewModel() and call some setter method directly there instead of passing it to the Screen composable.
Something like:

DestinationsNavHost(
    navGraph = NavGraphs.root
) {
    composable(SomeScreenDestination) { //this: DestinationScope<SomeScreenDestination.NavArgs>
        val vm = hiltViewModel<YourViewModelName>()
        vm.setData([YOUR DATA])

        SomeScreen(
            navigator = destinationsNavigator, // destinationsNavigator is a `DestinationsNavigator` (also lazily evaluated)
        )
    }
}


Support for health checks using CMD rather than CMD-SHELL

Currently only CMD-SHELL health checks are supported which require a shell on the docker container to run. distroless docker images like mockserver don't have a shell so that mechanism doesn't work, see https://github.com/GoogleContainerTools/distroless/issues/183.
To support docker containers without a shell the health check needs to be in the format
test: ['CMD', '/path/to/program', 'arg1', 'arg2', 'arg3']

Here is a PR to support CMD health checks https://github.com/testcontainers/testcontainers-node/pull/343

@cristianrgreco is there an expected time frame for this to get merged?

@richjharris sorry for the delay, haven't had much time to work on this lately. I'm hoping to get this done in the next couple of weeks

Released in 9.0.0


"future cannot be sent between threads safely" when returning ctx.read_csv from async_trait function

Describe the bug
A compilation error is raised in a situation that I believe is a valid use case of DataFusion
To Reproduce
Define an #[async_trait] with an async function signature that returns a DataFrame. Then implement the trait with a function that creates a context (ctx) and returns the result of ctx.read_csv.
#[async_trait]
trait CallReadTrait {
    async fn call(&self) -> Arc<dyn DataFrame>;
}

struct CallRead {}

#[async_trait]
impl CallReadTrait for CallRead {
    async fn call(&self) -> Arc<dyn DataFrame> {
        let mut ctx = ExecutionContext::new();
        ctx.read_csv("dummy.csv", CsvReadOptions::new()).await.unwrap()
    }
}

When compiling with the stable 1.55.0 toolchain, the following error is raised
error: future cannot be sent between threads safely
    --> datafusion/src/execution/context.rs:4151:52
     |
4151 |           async fn call(&self) -> Arc<dyn DataFrame> {
     |  ____________________________________________________^
4152 | |             let mut ctx = ExecutionContext::new();
4153 | |             ctx.read_csv("dummy.csv", CsvReadOptions::new()).await.unwrap()
4154 | |         }
     | |_________^ future created by async block is not `Send`
     |
     = help: within `impl futures::Future`, the trait `std::marker::Send` is not implemented for `std::sync::MutexGuard<'_, context::ExecutionContextState>`
note: future is not `Send` as this value is used across an await
    --> datafusion/src/execution/context.rs:341:14
     |
341  |               &LogicalPlanBuilder::scan_csv(
     |  ______________^
342  | |                 object_store,
343  | |                 path,
344  | |                 options,
...    |
347  | |             )
348  | |             .await?
     | |__________________^ first, await occurs here, with `self.state.lock().unwrap()` maybe used later...
note: `self.state.lock().unwrap()` is later dropped here
    --> datafusion/src/execution/context.rs:351:5
     |
346  |                 self.state.lock().unwrap().config.target_partitions,
     |                 -------------------------- has type `std::sync::MutexGuard<'_, context::ExecutionContextState>` which is not `Send`
...
351  |     }
     |     ^
     = note: required for the cast to the object type `dyn futures::Future<Output = std::sync::Arc<(dyn dataframe::DataFrame + 'static)>> + std::marker::Send`

Expected behavior
I expect this code compile and run successfully
Additional context
This error is not raised when calling the code in an async function that is not defined in an #[async_trait] implementation.
Here is an excerpt of the current ctx.read_csv function
        Ok(Arc::new(DataFrameImpl::new(
            self.state.clone(),
            &LogicalPlanBuilder::scan_csv(
                object_store,
                path,
                options,
                None,
                self.state.lock().unwrap().config.target_partitions,
            )
            .await?
            .build()?,
        )))

I don't fully understand the root cause here, but the compilation error goes away if the self.state.lock().unwrap().config.target_partitions expression is lifted to a local variable like this:
        let target_partitions = self.state.lock().unwrap().config.target_partitions;
        Ok(Arc::new(DataFrameImpl::new(
            self.state.clone(),
            &LogicalPlanBuilder::scan_csv(
                object_store,
                path,
                options,
                None,
                target_partitions,
            )
            .await?
            .build()?,
        )))

The situation is similar for the read_parquet and read_avro functions as well.
If this change makes sense, I'm happy to open a PR.

Interesting! From https://github.com/dtolnay/async-trait, does it work if you use following:
#[async_trait(?Send)]
instead of #[async_trait]?


chore(alerts) remove outlier type from nrql_conditions

Reference: https://newrelic.atlassian.net/browse/AINTER-8230
As part of the EOL process, remove the outlier type from nrql_conditions.
I'm marking this as a DRAFT PR until the EOL process happens, tentatively scheduled for Feb 1, 2022.
More info here: https://docs.newrelic.com/docs/alerts-applied-intelligence/transition-guide/

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jay McGrath seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closed in favor of #892


Error: Could not find or load main class launch.json

Hi @DonJayamanne ,
I am trying to use your plugin to debug my java code on vs.
I have made changes to java path and tasks.json as you indicate
When I try to run the debugger, Iam now getting the error
**Error: Could not find or load main class launch.json**.
How do I fix this? Please help.
Thanks

In case you haven't solved it by now, that's usually the case when you run any vscode debugger in a wrong file - your selected file was launch.json, and the debugger could not figure out how to debug it.
Just select (open) the .java file that you want to debug and start debug.

@kmuthukrishnan would you like me to add you as a contributor to this repo?

I had the same problem.
The solution was to edit the lauch.json
something like this:
 {             "name": "Java",             "type": "java",             "request": "launch",              "sourcePath": [                "${workspaceRoot}/src/"              ],             "jdkPath": "C:\\Program Files\\Java\\jdk1.8.0_121\\bin",             "stopOnEntry": true,             "cwd": "${fileDirname}",             "startupClass": "main.Main",             "preLaunchTask": "build",             "options": [                 "-classpath",                 "${workspaceRoot}/target/classes"             ]         },
the startupClass:"${fileBasename}" do not work if the class is inside a package.
Also, set sourcePath, Or may it won´t find your .java files.
As I am using maven, I have to set "options": [                 "-classpath",                 "${workspaceRoot}/target/classes"             ]
otherwise it won´t find my .class files.

Thanks @Vinicius CR. Will try and revert.
On Apr 25, 2017 7:42 PM, "Vinicius CR"<EMAIL_ADDRESS>wrote:
I had the same problem.
The solution was to edit the lauch.json
something like this:
{ "name": "Java", "type": "java", "request": "launch", "sourcePath": [
"${workspaceRoot}/src/" ], "jdkPath": "C:\Program
Files\Java\jdk1.8.0_121\bin", "stopOnEntry": true, "cwd":
"${fileDirname}", "startupClass": "main.Main", "preLaunchTask": "build",
"options": [ "-classpath", "${workspaceRoot}/target/classes" ] },
the startupClass:"${fileBasename}" do not work if the class is inside a
package.
Also, set sourcePath, Or may it won´t find your .java files.
As I am using maven, I have to set "options": [ "-classpath",
"${workspaceRoot}/target/classes" ]
otherwise it won´t find my .class files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DonJayamanne/javaVSCode/issues/33#issuecomment-297043139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2PvpsrO4A2nUqeSqn9x9mzBVB7B6qSks5rzf9fgaJpZM4M2mXw
.

README has been update, please take a look


Can Huginn achieve changes monitoring?

Such as this feature in Inoreader.
Monitor Web Pages For Changes Using Web Feeds - Inoreader blog
https://www.inoreader.com/blog/2021/04/monitor-web-pages-for-changes-with-web-feeds.html
When I need to check the updates and changelogs of an app, it will be very handy.
Such as this changelog
Release notes — Pine Script™ v5 User Manual v5 documentation
https://www.tradingview.com/pine-script-docs/en/v5/Release_notes.html#august-2022
It will return uneven number of entries, like 1 title and multiple entries.

Yes, this is very easy.

Website Agent
Data Output Agent (RSS)

Setup

Results:

@EaglePPP did this answer your question? Perhaps we can close this issue?


Add controller builders

Create branch: issue-4-add-controller-builders from develop.

Create branch: issue-4-add-controller-builders from develop.


chore: upgrade go and sarama libs

Sarama 1.37.2 had moved to Go 1.17 making an update a necessity.  It seemed to make sense to move straight to the latest stable Go release,  Go 1.19.

I ran the new code for a few hours.  It is functional and the cpu/mem look good (green trace is old, yellow is this PR).


update links (cherry-pick #786 into gh-pages branch)

Description from pull request #786:

The stable tag does not exist anymore. latest shows always to the latest stable version

Hopefully I targeted the correct branch. It looks like the PR was merged into master but not deployed to the gh-pages branch, therefore not addressing the 404 error (i.e., in index.md the links to the latest stable versions still pointed at the stable tag).

@Murmele Sorry i wasn't sure what you meant. My master branch is now up-to-date. Can you run CI again?


Added Contributing.md

Intro to contributing

Can you elaborate on this? Not sure it is really necessary without an actual code contribution.

You've got cd sysmon - assume that's from when you copy and pasted the same for sysmon.

Doesn't it also need a code of conduct?

Can you elaborate on this? Not sure it is really necessary without an actual code contribution.

I thought you meant the CONTRIBUTORS.md which is generally meant as formal intro to the method of contributing on github , but it seems i misunderstood aplogoies for it @readikus .
Are you looking for contributing.md file which includes the explanation of working of the internals and how one can specifically contribute to it ?

You've got cd sysmon - assume that's from when you copy and pasted the same for sysmon.
I was working on something earlier this morning it is my fault to have not double checked the template and misssing that single point

Doesn't it also need a code of conduct?
if its not sarcastic , it depends on you i would be glad to help you out needing a bit info about licenses and stuff


403 Forbidden

I keep getting 403 Forbidden with this. I tried using the default github token and creating a new one and using it as a secret.
Here is some further error output:
Error: Invalid status code: 128     at ChildProcess.<anonymous> (/home/runner/work/_actions/actions-js/push/v1.4/start.js:9:[19](https://github.com/**************/actions/runs/5336922590/jobs/9672288669#step:5:20))     at ChildProcess.emit (node:events:5[27](https://github.com/*********************/runs/5336922590/jobs/9672288669#step:5:28):28)     at maybeClose (node:internal/child_process:1092:16)     at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5) {   code: 128 } Error: Invalid status code: 128     at ChildProcess.<anonymous> (/home/runner/work/_actions/actions-js/push/v1.4/start.js:9:19)     at ChildProcess.emit (node:events:527:28)     at maybeClose (node:internal/child_process:1092:16)     at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

same here

You must use PAT because GITHUB_TOKEN repo permission is only read.

I resolved it by adding the token to the checkout at the beginning of the action as well.

Having the same issue and adding that token to checkout step didn't solve this...

@tarlepp Had the same problem, but I seems to solved it. In your repo, have you set Settings > Actions > General > Workflow permissions to Read and write permissions?


Issues running purethermal1 on pi zero 2w

Hello,
We've been using your thermal board for some time now so just wanted to thank you for the continued support that's been given over using this particular product in our line of robots.
I'm experiencing a stressful issue at the moment trying to stream the purethermal1 over wifi with flask. (flir lepton 3.5)
If I start up the program I get this error (over and over)
[<EMAIL_ADDRESS>global /tmp/pip-wheel-nz49_ez6/opencv-python-headless_488dea44cb7d406baf80df0bfc388ff9/opencv/modules/videoio/src/cap_v4l.cpp (1013) tryIoctl VIDEOIO(V4L2:/dev/lepton_camera): select() timeout.
I may get a single frame to display every minute.
The board is just running one purethermal device.
We've tried using the product, same program, on a pi3b, also a pi zero, and a pi 4.
We only have access to one pi zero 2, so are looking for an alternative solution to determine how to fix than simply replacing pi zero 2  (as you are aware limited numbers world wide)
What should we be using and is there an alternative way to stream the device over wifi?
Regards
Nathan

I've done a little digging. Can I get some help with what could possibly be happening?
This is the dmesg log when running the opencv program.
https://pastebin.com/FeypS204
I've tried using libuvc version of getting frames and found out some more helpful information.
using this program:
https://pastebin.com/RUAkVxEJ
it appears that after 30 seconds it can't find any frames from the callback and closes.
Tried camera on a pi zero, Pi3b, and Pi4b. Tried the camera on the same program on Pi3b and Pi4b, works.
It appears to be something specific to the pi zero 2.
It appears to be an issue with the start_streaming function.
Should note that the opencv program DOES receive frames every once in a while. but it is largely the select timeout issue trying to read frames from the camera.

It doesn't appear to be related to hardware... We tried plugging in a keyboard to the USB port and it works fine.
Also if we run the camera through a USB hub it actually starts getting data, but after quickly after starting the program it closes because it doesn't read any frames.

Any updates on this issue? I am having the same problem with PT Mini + Lepton 3.5 and Raspberry Pi 4B.

Sorry for the late reply, but we've found that the Purethermal Breakout board a good replacement for the Purethermal 2. It connects the thermal camera over I2C and SPI. Hope this still helps


Dockerize the client backend app

Folks who aren't familiar with python environments struggle to set up the client_backend.py code, set this up with a Dockerfile

done: https://docs.vocode.dev/react-quickstart#self-hosted


When using a reloadable deployment, the build after entering continuous build reports the wrong total build time.

This issue has been extensively discussed in this forum question.
Expected Behavior

Current Behavior

When we start the Play application, we expect it to load just once before starting to listen for requests. However, currently it loads twice.
Per @bigdaz from Gradle Inc in the forum question

The fact that we execute the build twice on initial startup is a bug.

Context

Server start up becomes slower, which impacts developer productivity.
Steps to Reproduce (for bugs)

This can be easily reproduced in sample Play apps that are inside Gradle.
[info] p.c.s.AkkaHttpServer - Listening for HTTP on /0:0:0:0:0:0:0:0:9000

> Task :runPlayBinary
Running Play App (:runPlayBinary) at http://localhost:9000/

BUILD SUCCESSFUL in 54s
7 actionable tasks: 7 executed

Reloadable deployment detected. Entering continuous build.

BUILD SUCCESSFUL in 54s
7 actionable tasks: 1 executed, 6 up-to-date

Waiting for changes to input files of tasks... (ctrl-d to exit)

Your Environment

Build scan URL:
It happens both on Mac and Linux.

I think the bug here might be with the output.
If you watch the console output, is there a ~minute between the first BUILD SUCCESSFUL and the second? Locally, I see the same behavior, but the second time is definitely wrong and the build is completely up-to-date. It looks like the time isn't reset for the second build.

Thanks @big-guy
This is the info log. We ran on play-2.4 sample project with Gradle 4.9.
ymwang play-2.4 $ ./gradlew runPlayBinary --info
Initialized native services in: /home/ymwang/.gradle/native
The client will now receive all logging from the daemon (pid: 36046). The daemon log file: /home/ymwang/.gradle/daemon/4.9/daemon-36046.out.log
Starting 4th build in daemon [uptime: 1 mins 24.739 secs, performance: 100%, no major garbage collections]
Using 16 worker leases.
Starting Build
Settings evaluated using settings file '/home/ymwang/Downloads/play-2.4/settings.gradle'.
Projects loaded. Root project using build file '/home/ymwang/Downloads/play-2.4/build.gradle'.
Included projects: [root project 'play-2.4']

> Configure project :
Evaluating root project 'play-2.4' using build file '/home/ymwang/Downloads/play-2.4/build.gradle'.
All projects evaluated.
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
file or directory '/home/ymwang/Downloads/play-2.4/app/assets', not found
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
Selected primary task 'runPlayBinary' from project :
Tasks to be executed: [task ':createPlayBinaryAssetsJar', task ':compilePlayBinaryPlayRoutes', task ':compilePlayBinaryPlayTwirlTemplates', task ':compilePlayBinaryScala', task ':processPlayBinaryPlayResources', task ':createPlayBinaryJar', task ':playBinary', task ':runPlayBinary']
:createPlayBinaryAssetsJar (Thread[Task worker for ':',5,main]) started.

> Task :createPlayBinaryAssetsJar
Task ':createPlayBinaryAssetsJar' is not up-to-date because:
  Output property 'archivePath' file /home/ymwang/Downloads/play-2.4/build/playBinary/lib/play-2.4-assets.jar has been removed.
:createPlayBinaryAssetsJar (Thread[Task worker for ':',5,main]) completed. Took 0.012 secs.
:compilePlayBinaryPlayRoutes (Thread[Task worker for ':',5,main]) started.
This JVM does not support getting OS memory, so no OS memory status updates will be broadcast
Initialized native services in: /home/ymwang/.gradle/native

> Task :compilePlayBinaryPlayRoutes
Task ':compilePlayBinaryPlayRoutes' is not up-to-date because:
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/routesScalaSources has been removed.
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/routesScalaSources/controllers has been removed.
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/routesScalaSources/controllers/javascript has been removed.
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
Starting process 'Gradle Worker Daemon 5'. Working directory: /home/ymwang/.gradle/workers Command: /export/apps/jdk/JDK-1_8_0_121/bin/java -Djava.security.manager=worker.org.gradle.process.internal.worker.child.BootstrapSecurityManager -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -cp /home/ymwang/.gradle/caches/4.9/workerMain/gradle-worker.jar worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Worker Daemon 5'
Successfully started process 'Gradle Worker Daemon 5'
Started Gradle worker daemon (0.353 secs) with fork options DaemonForkOptions{executable=/export/apps/jdk/JDK-1_8_0_121/bin/java, minHeapSize=null, maxHeapSize=null, jvmArgs=[], classpath=[/home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.play/routes-compiler_2.10/2.4.0/12b1375ab1c872c0c3b2aa8217881ad8ef8aea88/routes-compiler_2.10-2.4.0.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.play/twirl-api_2.10/1.1.1/b11c77752bfbb3d454a54a3675a5f9ecbe564dba/twirl-api_2.10-1.1.1.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-library/2.10.5/57ac67a6cf6fd591e235c62f8893438e8d10431d/scala-library-2.10.5.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/commons-io/commons-io/2.4/b1b6ea3b7e4aa4f492509a4952029cd8e48019ad/commons-io-2.4.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.apache.commons/commons-lang3/3.4/5fe28b9518e58819180a43a850fbc0dd24b7c050/commons-lang3-3.4.jar], keepAliveMode=SESSION}.
:compilePlayBinaryPlayRoutes (Thread[Task worker for ':',5,main]) completed. Took 0.834 secs.
:compilePlayBinaryPlayTwirlTemplates (Thread[Task worker for ':',5,main]) started.
This JVM does not support getting OS memory, so no OS memory status updates will be broadcast
Initialized native services in: /home/ymwang/.gradle/native

> Task :compilePlayBinaryPlayTwirlTemplates
Task ':compilePlayBinaryPlayTwirlTemplates' is not up-to-date because:
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/twirlTemplatesScalaSources has been removed.
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/twirlTemplatesScalaSources/views has been removed.
  Output property 'outputDirectory' file /home/ymwang/Downloads/play-2.4/build/src/play/binary/twirlTemplatesScalaSources/views/html has been removed.
All input files are considered out-of-date for incremental task ':compilePlayBinaryPlayTwirlTemplates'.
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
Starting process 'Gradle Worker Daemon 6'. Working directory: /home/ymwang/.gradle/workers Command: /export/apps/jdk/JDK-1_8_0_121/bin/java -Djava.security.manager=worker.org.gradle.process.internal.worker.child.BootstrapSecurityManager -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -cp /home/ymwang/.gradle/caches/4.9/workerMain/gradle-worker.jar worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Worker Daemon 6'
Successfully started process 'Gradle Worker Daemon 6'
Started Gradle worker daemon (0.395 secs) with fork options DaemonForkOptions{executable=/export/apps/jdk/JDK-1_8_0_121/bin/java, minHeapSize=null, maxHeapSize=null, jvmArgs=[], classpath=[/home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.play/twirl-compiler_2.11/1.1.1/c52e000cfe1895cd1dc9fb5c0d4ac5bbd73713cc/twirl-compiler_2.11-1.1.1.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-compiler/2.11.6/fa7d8a52d53c6571bce334b2c1d3d9546314dde3/scala-compiler-2.11.6.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.play/twirl-api_2.11/1.1.1/b029d9500caec7fe30f9a485fc654ee82d40d404/twirl-api_2.11-1.1.1.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.play/twirl-parser_2.11/1.1.1/8af782249d76c70d411305f94eff19c9906c43af/twirl-parser_2.11-1.1.1.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-reflect/2.11.6/f539c9c9cf312472b3a7771ac85ecda859657a1d/scala-reflect-2.11.6.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang.modules/scala-xml_2.11/1.0.3/8f295fc7620bf5a71eb7e34e71d97207aa534abf/scala-xml_2.11-1.0.3.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang.modules/scala-parser-combinators_2.11/1.0.3/b85f2ff7cbdb5540a4c293a947cc936892040fc9/scala-parser-combinators_2.11-1.0.3.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-library/2.11.6/be3457b4b748df35bffba675d8cddf44e9df4f7b/scala-library-2.11.6.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.apache.commons/commons-lang3/3.4/5fe28b9518e58819180a43a850fbc0dd24b7c050/commons-lang3-3.4.jar], keepAliveMode=SESSION}.
:compilePlayBinaryPlayTwirlTemplates (Thread[Task worker for ':',5,main]) completed. Took 1.965 secs.
:compilePlayBinaryScala (Thread[Task worker for ':',5,main]) started.
This JVM does not support getting OS memory, so no OS memory status updates will be broadcast
Initialized native services in: /home/ymwang/.gradle/native

> Task :compilePlayBinaryScala
Task ':compilePlayBinaryScala' is not up-to-date because:
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/classes has been removed.
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/classes/controllers has been removed.
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/classes/controllers/ReverseApplication.class has been removed.
Starting process 'Gradle Worker Daemon 7'. Working directory: /home/ymwang/.gradle/workers Command: /export/apps/jdk/JDK-1_8_0_121/bin/java -Djava.security.manager=worker.org.gradle.process.internal.worker.child.BootstrapSecurityManager -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -cp /home/ymwang/.gradle/caches/4.9/workerMain/gradle-worker.jar worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Worker Daemon 7'
Successfully started process 'Gradle Worker Daemon 7'
Started Gradle worker daemon (0.372 secs) with fork options DaemonForkOptions{executable=/export/apps/jdk/JDK-1_8_0_121/bin/java, minHeapSize=null, maxHeapSize=null, jvmArgs=[], classpath=[/home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.zinc/zinc/0.3.15/12e1f782684f2702e847faa0994eed4711270687/zinc-0.3.15.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.sbt/incremental-compiler/0.13.15/95e20d00b25a7aae19838009c11578b7e6b258ad/incremental-compiler-0.13.15.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.sbt/compiler-interface/0.13.15/bad996ed4fc3e83b872525e9cd7b80d81b98a324/compiler-interface-0.13.15-sources.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-compiler/2.10.6/9b15174852f5b6bb1edbf303d5722286a0a54011/scala-compiler-2.10.6.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/com.typesafe.sbt/sbt-interface/0.13.15/93fe450d5f5efb111397a34bc1fba0d50368a265/sbt-interface-0.13.15.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-reflect/2.10.6/3259f3df0f166f017ef5b2d385445808398c316c/scala-reflect-2.10.6.jar, /home/ymwang/.gradle/caches/modules-2/files-2.1/org.scala-lang/scala-library/2.10.6/421989aa8f95a05a4f894630aad96b8c7b828732/scala-library-2.10.6.jar], keepAliveMode=SESSION}.
Compiling with Zinc Scala compiler.
Initialized Zinc Scala compiler: 0.554 secs
Prepared Zinc Scala inputs: 0.606 secs
Pruning sources from previous analysis, due to incompatible CompileSetup.
Compiling 7 Scala sources and 1 Java source to /home/ymwang/Downloads/play-2.4/build/playBinary/classes...
Completed Scala compilation: 6.544 secs
:compilePlayBinaryScala (Thread[Task worker for ':',5,main]) completed. Took 7.114 secs.
:processPlayBinaryPlayResources (Thread[Task worker for ':',5,main]) started.

> Task :processPlayBinaryPlayResources
Task ':processPlayBinaryPlayResources' is not up-to-date because:
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/resources has been removed.
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/resources/routes has been removed.
  Output property 'destinationDir' file /home/ymwang/Downloads/play-2.4/build/playBinary/resources/application.conf has been removed.
:processPlayBinaryPlayResources (Thread[Task worker for ':',5,main]) completed. Took 0.002 secs.
:createPlayBinaryJar (Thread[Task worker for ':',5,main]) started.

> Task :createPlayBinaryJar
Task ':createPlayBinaryJar' is not up-to-date because:
  Output property 'archivePath' file /home/ymwang/Downloads/play-2.4/build/playBinary/lib/play-2.4.jar has been removed.
:createPlayBinaryJar (Thread[Task worker for ':',5,main]) completed. Took 0.016 secs.
:playBinary (Thread[Task worker for ':',5,main]) started.

> Task :playBinary
Skipping task ':playBinary' as it has no actions.
:playBinary (Thread[Task worker for ':',5,main]) completed. Took 0.0 secs.
:runPlayBinary (Thread[Task worker for ':',5,main]) started.
Initialized native services in: /home/ymwang/.gradle/native
--- (Running the application, auto-reloading is enabled) ---

[info] p.a.l.c.ActorSystemProvider - Starting application default Akka system: application
[info] p.c.s.NettyServer$ - Listening for HTTP on /0:0:0:0:0:0:0:0:9000

> Task :runPlayBinary
Task ':runPlayBinary' is not up-to-date because:
  Task has not declared any outputs despite executing actions.
Starting process 'Gradle Play Worker 8'. Working directory: /home/ymwang/Downloads/play-2.4 Command: /export/apps/jdk/JDK-1_8_0_121/bin/java -Djava.security.manager=worker.org.gradle.process.internal.worker.child.BootstrapSecurityManager -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -cp /home/ymwang/.gradle/caches/4.9/workerMain/gradle-worker.jar worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Play Worker 8'
Successfully started process 'Gradle Play Worker 8'
Running Play App (:runPlayBinary) at http://localhost:9000/
:runPlayBinary (Thread[Task worker for ':',5,main]) completed. Took 1.253 secs.

BUILD SUCCESSFUL in 11s
7 actionable tasks: 7 executed

Reloadable deployment detected. Entering continuous build.
Starting Build
Settings evaluated using settings file '/home/ymwang/Downloads/play-2.4/settings.gradle'.
Projects loaded. Root project using build file '/home/ymwang/Downloads/play-2.4/build.gradle'.
Included projects: [root project 'play-2.4']

> Configure project :
Evaluating root project 'play-2.4' using build file '/home/ymwang/Downloads/play-2.4/build.gradle'.
All projects evaluated.
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
file or directory '/home/ymwang/Downloads/play-2.4/app/assets', not found
Asked to use scala version 2.11 on play < 2.4.6. Will have to use the 2.10 routes compiler
Selected primary task 'runPlayBinary' from project :
Tasks to be executed: [task ':createPlayBinaryAssetsJar', task ':compilePlayBinaryPlayRoutes', task ':compilePlayBinaryPlayTwirlTemplates', task ':compilePlayBinaryScala', task ':processPlayBinaryPlayResources', task ':createPlayBinaryJar', task ':playBinary', task ':runPlayBinary']
:createPlayBinaryAssetsJar (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :createPlayBinaryAssetsJar UP-TO-DATE
Skipping task ':createPlayBinaryAssetsJar' as it is up-to-date.
:createPlayBinaryAssetsJar (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.006 secs.
:compilePlayBinaryPlayRoutes (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :compilePlayBinaryPlayRoutes UP-TO-DATE
Skipping task ':compilePlayBinaryPlayRoutes' as it is up-to-date.
:compilePlayBinaryPlayRoutes (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.002 secs.
:compilePlayBinaryPlayTwirlTemplates (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :compilePlayBinaryPlayTwirlTemplates UP-TO-DATE
Skipping task ':compilePlayBinaryPlayTwirlTemplates' as it is up-to-date.
:compilePlayBinaryPlayTwirlTemplates (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.002 secs.
:compilePlayBinaryScala (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :compilePlayBinaryScala UP-TO-DATE
Skipping task ':compilePlayBinaryScala' as it is up-to-date.
:compilePlayBinaryScala (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.024 secs.
:processPlayBinaryPlayResources (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :processPlayBinaryPlayResources UP-TO-DATE
Skipping task ':processPlayBinaryPlayResources' as it is up-to-date.
:processPlayBinaryPlayResources (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.001 secs.
:createPlayBinaryJar (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :createPlayBinaryJar UP-TO-DATE
Skipping task ':createPlayBinaryJar' as it is up-to-date.
:createPlayBinaryJar (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.003 secs.
:playBinary (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :playBinary UP-TO-DATE
Skipping task ':playBinary' as it has no actions.
:playBinary (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.0 secs.
:runPlayBinary (Thread[Task worker for ':' Thread 2,5,main]) started.

> Task :runPlayBinary
Task ':runPlayBinary' is not up-to-date because:
  Task has not declared any outputs despite executing actions.
:runPlayBinary (Thread[Task worker for ':' Thread 2,5,main]) completed. Took 0.001 secs.

BUILD SUCCESSFUL in 11s
7 actionable tasks: 1 executed, 6 up-to-date

Waiting for changes to input files of tasks... (ctrl-d to exit)
<=============> 100% EXECUTING [17s]
> IDLE

Yep. That's what I was seeing locally too.  The second build completes <1 second, but it reports that it's build time is much longer.
I renamed the issue.

Thanks @big-guy
Is there an ETA for Gradle 5.1?

This will be fixed in 5.3.

@lacasseio  Thanks, is there a nightly that we can check?

The RC that should be coming tomorrow will have the fix. However, I think the fix was merged back to master last week. You can use the latest nightly.

@ymwang1984 Just so we can close this issue, the error within the reported time was fixed. However, regarding the duplicated run when the build transition from non-continuous to continuous because of the deployment being generated won't be fixed. If this causes significant pain, you can open another issue explaining how it's affecting you so we can prioritize work around this work.

@lacasseio  Thank you for the reply. The reported time is not the issue.
The issue is truly about the duplicate runs because that's the actual time that we can save for developers. Every day, "runPlayBinary" is invoked more than 4000 times in the company. Removing duplicate runs will have a huge impact.


Explain AF cluster differences.

Added more details on the differences between AF and cluster architecture.

@nerpaula we would need this also in all other versions.

@nerpaula we would need this also in all other versions.

@joerg84 added this to the other versions as well.


Import PartialTypedDataConfig is failing

Hi,
I am currently trying to verify an offchain attestation by following the provided example from the readme.
Unfortunately the code snippet seems irrelevant (or am I missing something?).
Thank you for the clarification/fix.
Edit: the same example from the documentation seems to exhibit even more issues.

Hi @zeGzD,
We need to update the documentation to support the newer version. We'll do it soon 🙏

Resolved


Java SDK 9+ XML Issue

If using the latest JDK (9+) then the required XML libs are no longer available and require that the code be recompiled to manually add the XML libs.

You are right, since JDK 9, Modules shared with Java EE are not resolved by default. I'm planning a bugfix release soon. Binaries with Java 10 will fix this issue

fixed by 642649f

Ok so yes its normal :) I'm planning to release a new version by the weekend. You can track new versions on the release page.

Thanks for working on this for us!  Cheers!


Blame current file concerning HEAD

3161
2.17.0.windows.1
win10x64

Blame current file gives the choice of commits in which the file was affected, but after merging the branches it is unclear which commit was taken as the basis. It is necessary to look git blame concerning HEAD.

I thought the first commit is always the head, isn’t it?

If there was a merge with the conflicts, it may turn out that the last commit of the offered menu does not reflect the current state of the file, because the file state was selected earlier in the conflict, and the merge commit in the menu is not displayed.

Perhaps you misunderstood me, the difficulties of translation. Conflicts have already been resolved. Git blame shows who wrote this or that line. When you select gitsavvy git blame current file, it shows a list of commits for this file, but changes in this commit could be dropped when merging with another branch, that is, changes were made from the older commit during fusion. Thus, the first commit on top of the git blame current file command will not match HEAD.

So passing -m to git log should fix it.

Could you share your repo with us? I was trying to reproduce your result but it correctly shows the merge commit as HEAD.

https://github.com/mercurykd/test

I got this with git 2.16.2
(randyimac)-test$ git log -- 1.txt (master)
commit 8848b731c0b960337f5c55fd1473f54a4566d835 (HEAD -> master, origin/master, origin/HEAD)
Merge: eb640ff 462289f
Author: mercury<EMAIL_ADDRESS>Date:   Sun Apr 15 00:33:19 2018 +0400

    Merge branch 'old-branch'

     Conflicts:
            1.txt

commit eb640ffbf748cfa4fb531b8e18f2fe75bf64515b
Author: mercury<EMAIL_ADDRESS>Date:   Sun Apr 15 00:31:57 2018 +0400

    3 commit in master

commit 462289f08c60b83893752f26dda71ce97f5d35ca
Author: mercury<EMAIL_ADDRESS>Date:   Sun Apr 15 00:31:16 2018 +0400

    2 commit in old-branch

commit f4117b9f332b493f5b4422844e37334191c38f43
Author: mercury<EMAIL_ADDRESS>Date:   Sun Apr 15 00:30:03 2018 +0400

    init

It seems that it is an issue of git, I cannot reproduce it with the most updated version of git.

Yes, could you add the git blame option without specifying a commit?

It should work by just defining the following (untested) in your user Default.sublime-commands
    {
        "caption": "git: blame current file at HEAD",
        "command": "gs_blame",
        "args": {"commit_hash": "HEAD"}
    }

What version of git are you using @mercurykd ?

2.17.0.windows.1

I still cannot reproduce with git 2.17.0 on macOS.

@mercurykd maybe WSLGit could be a nice workaround. Could you try downloading it and then defining the path to the binary in the settings as in:
https://github.com/divmain/GitSavvy/blob/1196f7819ebbc2b8de71222481c23ea58b4553ec/GitSavvy.sublime-settings#L15-L23

The problem with WSLGit is file paths. When we want to read it form disk.

Hmm good point @stoivo ... I'm not even sure where to start mapping it out

Open an issue on Windows on git?

The problem with WSLGit is file paths. When we want to read it form disk.

I can be wrong, but in my opinion there is only a difference in the name of the drive letter, for example, instead of C:\ => /mnt/c/, and reverse the backslashes to direct ones.

Reply to: https://github.com/divmain/GitSavvy/issues/929#issuecomment-382561367
There is an interesting development with regards to windows/linux paths over here:
https://github.com/andy-5/wslgit/issues/12
Reply to: https://github.com/divmain/GitSavvy/issues/929#issuecomment-382569439
I'm guessing we could try to "quote" the arguments to --format (in either single or double quotes).  E.g this sort of patch:
 core/git_mixins/branches.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/git_mixins/branches.py b/core/git_mixins/branches.py
index 1b8f45e..67ca703 100644
--- a/core/git_mixins/branches.py
+++ b/core/git_mixins/branches.py
@@ -23,7 +23,7 @@ class BranchesMixin():
         """
         stdout = self.git(
             "for-each-ref",
-            "--format=%(HEAD)%00%(refname)%00%(upstream)%00%(upstream:track)%00%(objectname)%00%(contents:subject)",
+            "--format='%(HEAD)%00%(refname)%00%(upstream)%00%(upstream:track)%00%(objectname)%00%(contents:subject)'",
             "--sort=-committerdate" if sort_by_recent else None,
             "refs/heads",
             "refs/remotes")

If someone with ready access to a windows machine can try it out it'll be great.


updated to remove dep on 1.18 methods

it fixes 1.18 calls that where failing and falls back to 1.16 logic when needed.

LGTM