Getting error - missing 1 required positional argument: 'jobs' while trying to execute CVAT-CLI commands

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Steps to Reproduce (for bugs)

cvat-cli --server-host [HOST] --auth [AuthData] ls

Expected Behaviour

List all tasks in terminal.
 Swarm: inactive I suppose non of those
Operating System and version (e.g. Linux, Windows, MacOS): Ubuntu 18.04.5 LTS (GNU/Linux 5.4.0-144-generic x86_64)
Other diagnostic information / logs:
TraceBack
    Traceback (most recent call last):
      File "/home/weedbot/miniconda3/envs/yolactedge/bin/cvat-cli", line 8, in
        sys.exit(main())
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_cli/__main__.py", line 67, in main
        actions[parsed_args.action](cli, **vars(action_args))
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_cli/cli.py", line 30, in tasks_list
        results = self.client.tasks.list(return_json=use_json_output, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/core/proxies/model_proxy.py", line 156, in list
        results = get_paginated_collection(endpoint=self.api.list_endpoint, return_json=return_json)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/core/helpers.py", line 29, in get_paginated_collection
        (page_contents, response) = endpoint.call_with_http_info(**kwargs, page=page)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/api_client.py", line 1075, in call_with_http_info
        collection_formats=params['collection_format'])
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/api_client.py", line 497, in call_api
        return self.__call_api(**params)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/api_client.py", line 285, in __call_api
        _check_type=_check_type
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/api_client.py", line 395, in deserialize
        configuration=self.configuration
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1598, in validate_and_convert_types
        check_type=_check_type
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1472, in attempt_convert_item
        configuration, spec_property_naming)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1384, in deserialize_model
        return model_class._new_from_openapi_data(**kw_args)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 51, in wrapped_init
        return fn(_self, *args, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 379, in _new_from_openapi_data
        return cls._from_openapi_data(*args, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 51, in wrapped_init
        return fn(_self, *args, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model/paginated_task_read_list.py", line 254, in _from_openapi_data
        setattr(self, var_name, var_value)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 197, in __setattr__
        self[attr] = value
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 518, in __setitem__
        self.set_attribute(name, value)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 171, in set_attribute
        self._check_type, configuration=self._configuration)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1646, in validate_and_convert_types
        configuration=configuration
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1598, in validate_and_convert_types
        check_type=_check_type
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1472, in attempt_convert_item
        configuration, spec_property_naming)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 1384, in deserialize_model
        return model_class._new_from_openapi_data(**kw_args)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 51, in wrapped_init
        return fn(_self, *args, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 379, in _new_from_openapi_data
        return cls._from_openapi_data(*args, **kwargs)
      File "/home/weedbot/miniconda3/envs/yolactedge/lib/python3.7/site-packages/cvat_sdk/api_client/model_utils.py", line 51, in wrapped_init
        return fn(_self, *args, **kwargs)
    TypeError: _from_openapi_data() missing 1 required positional argument: 'jobs'

Hi, please update cvat-sdk and cvat-cli up to the 2.4.0 version.

I have the same issue - which i solved by matching versions of cvat server and cvat-sdk.
cvat versions were bumped yesterday from 2.3.0 -> 2.4.0
in my case i installed cvat-sdk which pulled latest version (2.4.0) but for testing i was using older version (2.3.0)
i resolved this by downgrading cvat-sdk to 2.3.0 until i update cvat-server

Yeah... honestly yesterday was the first time I installed CLI and tried to use it. It seems that I installed it a little bit to late and I got the latest version 😄, while CVAT About page says that it is Server version 2.0.
Downgraded both SDK and CLI from 2.4.0 to 2.3.0 and error was gone.
Thanks!

This issue has appeared again in
cvat-cli - 2.5.2
cvat-sdk - 2.5.2
in 2.4.0 job issue was fixed but it was throwing an error because there was no argument for organization
in 2.5.2 there is an argument for organization but again it starts asking for job argument


October manpages

What was the end-user problem that led to this PR?
The problem was that outdated manpages.
What is your fix for the problem, implemented in this PR?
My fix is to update them.

That'd be super awesome.

@bundlerbot r=hsbt


Better popups with just images on IE and Edge

Related to https://github.com/CartoDB/cartodb/issues/13808
This includes a fallback for IE and Edge popups through carto.js (https://github.com/CartoDB/carto.js/pull/2105)

[x] Dev
[x] Frontend CR (at carto.js)
[x] Deployment to staging
[ ] Acceptance
[ ] Deployment to production

To check for the fix:

Visualize a layer in a map, having an url to an image in one of its fields
Configure a popup with style Pop-Up with header image
Find a feature and Click on it, to show its popup with image.

A carto file is attached to easily reproduce steps 1 and 2. Download, extract .carto and import bulding_with popup.carto.zip
Acceptance

[ ] From IE11 or Edge browser: image is displayed, popup has no hook and close button is correctly positioned over the image.
[ ] From another browser (eg. Chrome): popup has a hook and there is a piece of the image clipped on it.

In IE, works like a charm :)

I have also just tested it on Edge 👍


Database: Add "sessions" to prevent async/concurrency problems

With using Singleton-class, there should be Session-feature implemented too. This will fully protect against async/concurrency problems and seems to be good-practice even in synchronous application.

Sessions added to actively used private functions


Added alternate methods for authentication

Added methods with extensive comments to the samples to acquire the
authentication token via alternate methods. These methods include
authenticating using credential and also using a certificate. These
methods do not require any manual intervention and fetch the required token
completely programmatically.

@HarvestingClouds - thanks very much for your contribution. I will discuss w/some engineering folks and see if they way to include this in the sample. Either way, this PR will provide additional/useful code for others.

Any progress on this pull request? It would be very useful to have current samples on how to use the Azure Billing APIs. Thank you

Apologies for the delay on this. If I recall correctly, the preference was to bundle these into a separate sample with a separate confidential/web app AAD registration. The reason is because this is meant as a console sample which is considered a public client, and it wouldn't be good practice to demonstrate confidential/web client code (ie: exposing secrets) in this sample.

Appreciate the quick response. If there is no intention to updating the sample and keeping it up to date (=working) I would suggest that a note saying as much is made at the top of the readme file. That would help avoid confusion for people looking at the sample for a minimal effort. Just a friendly suggestion.
Regardless, thank you for responding.

Thanks @tommysundling.
Agree. As I mentioned earlier I'd like to leave this PR open for that reason, in case people are building a web client and want to see that type of client code.
Regarding the readme note, we will definitely keep it working if there is an API change that breaks it. Can you also confirm that you are NOT having problems with it? Just want to make sure we are not overlooking something that is supposed to be functioning, but isn't. I've tested multiple times and it works fine for me as the sample it was intended to be (ie: a public/console client).


When will there be support for Appium 2.0

Description
If there is the feature you want to see added to Appium .NET client so please describe necessity of this feature and the way that it should work.
I need the ability to use AppiumBY for The Image Plugins.  Currently I do not see any support for Appium 2.0 via the Dot-net-driver. nor any examples of the Image plugins? Does this already exist for DotNet Driver or is it just not built yet?

I really want to see an example of Image Plugin with C#. How to do an image based function example and a comparison. I can see them for the other languages but I can not decode them to C#. Any help would be greatly appreciated.

https://github.com/appium/appium-dotnet-driver/blob/4570d11267572b861554142d39d4452d853e6118/src/Appium.Net/Appium/AppiumDriver.cs#L197 or https://github.com/appium/appium-dotnet-driver/blob/4570d11267572b861554142d39d4452d853e6118/src/Appium.Net/Appium/AppiumDriver.cs#L597-L683 ?

Thank you this is very helpful. I do notice Execute does not seem to work
here. One other issue is the comparison actually taking a screenshot where
I am at in the app then matching it against an earlier taken screenshot or
is that a different scenario?
On Sat, Oct 1, 2022, 7:33 PM Kazuaki Matsuo @.***>
wrote:

https://github.com/appium/appium-dotnet-driver/blob/4570d11267572b861554142d39d4452d853e6118/src/Appium.Net/Appium/AppiumDriver.cs#L197
or
https://github.com/appium/appium-dotnet-driver/blob/4570d11267572b861554142d39d4452d853e6118/src/Appium.Net/Appium/AppiumDriver.cs#L597-L683
?
—
Reply to this email directly, view it on GitHub
https://github.com/appium/appium-dotnet-driver/issues/524#issuecomment-1264508342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGVOYIY4YQTE3VKSAK6CWDWBDC4PANCNFSM6AAAAAAQWCE2FE
.
You are receiving this because you authored the thread.Message ID:
@.***>

@aaronportier we basically added support for appium 2.0 in the latest beta version, can you please check and update if something is missing?

@Dor-bl @KazuCocoa
Faced with problem in matching images. Topic looks like i should comment about but i can create new issue if need
appium client 5.0 beta 2 , server also latest 2.0.
For example i try basic scenario take Two screenshot of One element and get result that it is the same and matched
So made i something wrong or is it issue?
run coomand:
node appium server -p 4444 -a <IP_ADDRESS> -pa /wd/hub --use-plugins images --relaxed-security

[Appium] Attempting to load plugin images...
[debug] [Appium] Requiring plugin at /Users/timur/appiumNode/node_modules/@appium/images-plugin
[Appium] Welcome to Appium v2.0.0-beta.46
[Appium] Non-default server args:
[Appium] {
[Appium]   address: '<IP_ADDRESS>',
[Appium]   basePath: '/wd/hub',
[Appium]   port: 4444,
[Appium]   relaxedSecurityEnabled: true,
[Appium]   usePlugins: [
[Appium]     'images'
[Appium]   ]
[Appium] }

some code example
```           var elem = Driver.FindElement(By.XPath("//android.widget.FrameLayout/android.view.ViewGroup/android.widget.LinearLayout/android.widget.TextView[1]"));
            var scrFile = Convert.ToBase64String(elem.GetScreenshot().AsByteArray);
            var scrFile2 = Convert.ToBase64String(elem.GetScreenshot().AsByteArray);

            var matchOptions = new FeaturesMatchingOptions();
            matchOptions.GoodMatchesFactor = 40;
            matchOptions.MatchFunc = "BRUTE_FORCE_HAMMING";
            matchOptions.DetectorName = "ORB";
            matchOptions.Visualize = true;
            FeaturesMatchingResult result = Driver.MatchImageFeatures(scrFile, scrFile2, matchOptions);
            string countResult =result.TotalCount.ToString();
            result.SaveVisualizationAsFile("outputFile");
            Assert.True(result.TotalCount > 0);

I get  "not present in the dictionary." for totalCount, actually outputFile also not working
System.Collections.Generic.KeyNotFoundException: The given key 'totalCount' was not present in the dictionary.

System.Collections.Generic.KeyNotFoundException
The given key 'totalCount' was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at OpenQA.Selenium.Appium.ImageComparison.FeaturesMatchingResult.get_TotalCount()

serverside log at this moment
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Calling AppiumDriver.compareImages() with args: ["matchFeatures","iVBORw0KGgoAAAANSUhEUgAABAwAAADiCAYAAADK4lWXAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAACAASURBVHic7N1neBTnuTfw/zatem+odwlRhACB6B0DxhQDLuAat8Q+cZw4cTnOm3Ji58QpJ3HsNCcuYMeODQYbjMGAaAZElZAQAhUkJISEJNTbSqvdfT8QiMTuzsyuVrsr6f+7Lj6wc88zjzTzjHbueYosOiXdACIiIiIiIiKiPuSOrgAREREREREROR8mDIiIiIiIiIjICBMGRERERERERGSECQMiIiIiIiIiMsKEAREREREREREZYcKAiIiIiIiIiIwwYUBERERERERERpgwICIiIiIiIiIjTBgQERERERERkREmDIiIiIiIiIjICBMGRERERERERGSECQMiIiIiIiIiMsKEAREREREREREZYcKAiIiIiIiIiIwwYUBERERERERERpgwICIiIiIiIiIjTBgQERERERERkRGloytARERERMOHX0A4/AIj4ebuDaXSBRpNOzSdrairKUVXZ6ujq0dERBZgwoCISEDwqEQsWvGcYMzpo5tRVHDQ5LZ5S59GRMx4s/t2azrw6XvPD6iOg2nuku8gPHqs2e3dmnZs2fiiHWtERM4oMnYCJmTchaj4dLh7+JmNq75SiH/+7Rk71oyIiAaCCQMiIgGubp4Ii0wVjPH0CjC7zT8oCqHhyWa3a7rarK6bPQQERwv+/F2dLXasDQFAWsZyjElfAv/ACMhkcrQ01eDiuQM4fXQz9Hqdo6tHI4yP3ygsXvkDxCRMlhTv7uE7yDUiIiJbYsKAiG5JGjMbHp7+Fu0TGZsGL59gwZjKSzlob2uwqNyG+gpUluVatA/RcLfivp8ieezcfp+5unkhJCwJcUlTsfn9H0Gn63VQ7WikiYgeh1UbXoWbu7fkfQx6/SDWiIiIbI0JAyK6Zers9YJvw60l9obelPNn9zBhQNTHmPTFRsmCviJ...
[AppiumDriver@ff53] Plugins which can handle cmd 'compareImages': images
[AppiumDriver@ff53] Plugin images is now handling cmd 'compareImages'
[AppiumDriver@ff53] Command 'compareImages' was *not* handled by the following behaviours or plugins, even though they were registered to handle it: ["default"]. The command *was* handled by these: ["images"].
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Responding to client with driver.compareImages() result: {"count":0}
[HTTP] <-- POST /wd/hub/session/6ed31803-f2ff-4cd0-98f7-51bdbd190642/appium/compare_images 200 647 ms - 21
[HTTP]
[HTTP] --> DELETE /wd/hub/session/6ed31803-f2ff-4cd0-98f7-51bdbd190642
[HTTP] {}
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Calling AppiumDriver.deleteSession() with args: ["6ed31803-f2ff-4cd0-98f7-51bdbd190642"]
[AppiumDriver@ff53] Plugins which can handle cmd 'deleteSession': images
[AppiumDriver@ff53] Plugin images is now handling cmd 'deleteSession'
[AppiumDriver@ff53] Executing default handling behavior for command 'deleteSession'
[debug] [AppiumDriver@ff53] Event 'quitSessionRequested' logged at<PHONE_NUMBER>154 (13:02:26 GMT+0500 (Uzbekistan Standard Time))
[AppiumDriver@ff53] Removing session 6ed31803-f2ff-4cd0-98f7-51bdbd190642 from our master session list
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Deleting UiAutomator2 session
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Deleting UiAutomator2 server session
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Matched '/' to command name 'deleteSession'
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Proxying [DELETE /] to [DELETE http://<IP_ADDRESS>:8203/session/70a69b41-f246-4633-a7da-a86b4ab1d582] with no body
[debug] [AndroidUiautomator2Driver@92d8 (6ed31803)] Got response with status 200: {"sessionId":"70a69b41-f246-4633-a7da-a86b4ab1d582","value":null}
[debug] [ADB] Running '/Users/timur/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell dumpsys activity services io.appium.settings/.recorder.RecorderService'
[debug] [ADB] Running '/Users/timur/Library/Android/sdk/platform-tools/adb -P 5037 -s emulator-5554 shell am force-stop com.infratel.mightycall'
[debug] [Logcat] Stopping logcat capture

so is it issue or i made something wrong? Should i create another issue topic?

Perhaps https://github.com/appium/appium/blob/c43acc23ee80d461747c1912cdcfb136280001a0/packages/images-plugin/lib/compare.js#L42 's behavior, it was maybe different from Appium 1.x's one.
@jlipps Do you remember why https://github.com/appium/appium/blob/c43acc23ee80d461747c1912cdcfb136280001a0/packages/images-plugin/lib/compare.js#L38-L43 returns {count: 0}; instead of raising an exception like https://github.com/appium/appium-base-driver/blob/b65c3a82863ba5804a318827ba9939dbd09af6a1/lib/basedriver/commands/images.js#L55 did.

I am seeing additional issues with the current Nugget version 4.3 When it
comes to FindElemenybyImage. I am doing a push pull and appium gives the
wrong coordinates.
On Fri, Nov 4, 2022, 4:40 AM Dor Blayzer @.***> wrote:

@aaronportier https://github.com/aaronportier we basically added
support for appium 2.0 in the latest beta version, can you please check and
update if something is missing?
—
Reply to this email directly, view it on GitHub
https://github.com/appium/appium-dotnet-driver/issues/524#issuecomment-1303122553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGVOYM5FNYEX2VZOZ7IBKLWGTDXXANCNFSM6AAAAAAQWCE2FE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@aaronportier I'm closing this since the topic of this issue is support for appium 2.x.
Which we have in the latest beta release. In case you encounter other issues with that version, please open a new issue.
Thank you


[Bug] App Crash: IllegalArgumentException

Describe the bug
[Bug] App Crash: IllegalArgumentException
Device: SM-G988W (samsung - z3q), SDK: 33 (13), App: 2.8.1 (136)
java.lang.IllegalArgumentException: red = 0.0, green = 0.0, blue = 0.0, alpha = -3.833397E-8 outside the range for sRGB IEC61966-2.1 (id=0, model=Rgb)
at androidx.compose.ui.graphics.a.F(SourceFile:3)
at androidx.compose.ui.graphics.a.b(SourceFile:484)
at r0.v.c(SourceFile:17)
at t0.c.a(SourceFile:23)
at t0.c.i0(SourceFile:44)
at I0.J.i0(SourceFile:16)
at t0.g.j(SourceFile:65)
at t.r.a(SourceFile:587)
at t.r.n(SourceFile:54)
at o0.e.a(SourceFile:3)
at I0.J.b(SourceFile:50)
at I0.h0.I0(SourceFile:55)
at I0.h0.G0(SourceFile:30)
at I0.D.Y0(SourceFile:6)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at androidx.compose.ui.node.a.i(SourceFile:5)
at I0.y.Y0(SourceFile:28)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at I0.D.Y0(SourceFile:6)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at androidx.compose.ui.node.a.i(SourceFile:5)
at I0.y.Y0(SourceFile:28)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at androidx.compose.ui.node.a.i(SourceFile:5)
at I0.y.Y0(SourceFile:28)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at I0.D.Y0(SourceFile:6)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at androidx.compose.ui.node.a.i(SourceFile:5)
at I0.y.Y0(SourceFile:28)
at I0.h0.I0(SourceFile:9)
at t.Y.b(SourceFile:257)
at t.Y.d(SourceFile:166)
at F3.Z4.e(SourceFile:114)
at i0.w.a(SourceFile:52)
at i0.x.c(SourceFile:140)
at I0.u0.a(SourceFile:3)
at Y.L0.n(SourceFile:416)
at J0.X0.A(SourceFile:22)
at J0.a1.i(SourceFile:40)
at J0.a1.l(SourceFile:14)
at I0.h0.G0(SourceFile:5)
at androidx.compose.ui.node.a.i(SourceFile:5)
at I0.y.Y0(SourceFile:28)
at I0.h0.I0(SourceFile:9)
at I0.h0.G0(SourceFile:30)
at androidx.compose.ui.node.a.i(SourceFile:5)
at J0.z.dispatchDraw(SourceFile:69)
at android.view.View.draw(View.java:24409)
at android.view.View.updateDisplayListIfDirty(View.java:23267)
at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4732)
at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4704)
at android.view.View.updateDisplayListIfDirty(View.java:23214)
at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4732)
at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4704)
at android.view.View.updateDisplayListIfDirty(View.java:23214)
at android.view.ThreadedRenderer.updateViewTreeDisplayList(ThreadedRenderer.java:777)
at android.view.ThreadedRenderer.updateRootDisplayList(ThreadedRenderer.java:783)
at android.view.ThreadedRenderer.draw(ThreadedRenderer.java:881)
at android.view.ViewRootImpl.draw(ViewRootImpl.java:5647)
at android.view.ViewRootImpl.performDraw(ViewRootImpl.java:5330)
at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:4486)
at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:3116)
at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:10885)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1301)
at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1309)
at android.view.Choreographer.doCallbacks(Choreographer.java:923)
at android.view.Choreographer.doFrame(Choreographer.java:852)
at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1283)
at android.os.Handler.handleCallback(Handler.java:942)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:226)
at android.os.Looper.loop(Looper.java:313)
at android.app.ActivityThread.main(ActivityThread.java:8762)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:604)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1067)
To Reproduce
When previewing full filters intermittently and doesn't seem to be one specific filter as the cause. I've tried it many times and the filter which caused it to crash previously ends up working when I try again but then it crashes after selecting a different filter.

Should be fixed by bumping compose to 1.7.0-alpha08 in 2.8.2-alpha01


Regression caused by #1594

Since #1594 was merged, specifically 745af5379, I'm seeing a metal assert in a very specific case. When trying to use vkCmdDrawIndexedIndirect, I get the assert
failed assertion 'Set Vertex Buffer Offset Validation index(0) must have an existing buffer.'
Tracing through the MVK code, I can see there is a setVertexBuffer: call that is correctly setting the index(0) buffer, and doing a regular draw call. Then later on with the same render command encoder, it tries to do setVertexBufferOffset: when setting up for an indirect draw, which throws that assert. There are pipeline switches that have occurred since the first draw call and the indirect draw call. Are there cases where binding a new pipeline will invalidate the vertex binding, and that's not being handled by this PR?
What could cause the vertex binding to get lost like this? Is there any Metal operations that could invalidate the index?
Any tips on how I could help debug this more? I've set a breakpoint in MVKResourcesCommandEncoderState::encodeBinding to try to catch any calls that may be overwriting the binding, but not seeing anything questionable.
One thing that is happening between the first setVertexBuffer: to the render encoder, and the setVertexBufferOffset:, is the the same buffer is getting bound to a compute encoder. I would think that should be fine though. The computer encoder will get committed before the render encoder does, but this assert is happening during the recording of the commands anyways, not the execution.
Attn: @js6i
Thanks

Adding to this, it doesn't seem limited to the binding becoming invalidated (unbound). I have another case where the binding size/offset isn't what MoltenVK think it is, so a validation error comes up there as well (not enough data bound for the shader requirement).
So far no luck finding where the core issue is, but just always treating the bindings as dirty avoids the issue, so I do think it's related to this optimization.

I have another case...

In this second case, is there also a flip to a compute encoder between two render encoders? That is...is this behavior consistent with the original problem above?

I just tried commenting out all calls to vkCmdDispatch(), and the issue still occurs (the second case). So I think it's not specifically related to mixing those.

@mbechard
I'm curious. How did you determine that 745af53 is the problematic commit?
And...

just always treating the bindings as dirty avoids the issue

What did you change in the MoltenVK code to do this...to treat the bindings as dirty?

I am having issues with PR 1594 too. Metal assertion occurs in parallel-n64 for macOS:
Compute Function(main0): argument _5227[0] from buffer(0) with offset(0) and length(16) has space for 16 bytes, but argument has a length(32).
I have pinpointed it to https://github.com/KhronosGroup/MoltenVK/blob/master/MoltenVK/MoltenVK/Commands/MVKMTLResourceBindings.h#L77-L91, replacing the condition to line 78 to true fixes the assert but I am not sure why exactly it helps.

Do these render passes involve tessellation?

I very much doubt, it is just an N64 emulator. The source code is at https://gitlab.com/aglab2/parallel-n64/-/tree/moltenvk_noupstream, unfortunately I am not very knowledged in Vulkan to understand it entirely. I also noticed that just adding a check for other.isDirty is enough to fix the issue but I think it defeats the purpose of the optimization so I doubt it is the solution.
    inline void update(const MVKMTLBufferBinding &other) {
        if (other.isDirty) {
          isDirty = true;
        }
        if (mtlBuffer != other.mtlBuffer || size != other.size || isInline != other.isInline) {
            mtlBuffer = other.mtlBuffer;
            size = other.size;
            isInline = other.isInline;
            offset = other.offset;

            justOffset = false;
            isDirty = true;
        } else if (offset != other.offset) {
            offset = other.offset;
            justOffset = !isDirty || justOffset;
            isDirty = true;
        }
    }

I determined it was the problematic commit by recompiling at different points until things started to break. The source at 745af5379c69 will generate the metal assertion, while the source at the previous commit 334e0cee69c60c will not.
I can also avoid the assertion if I ensure that MVKMTLBufferBinding::update() always takes the first if clause, which I assume pretty much neutralizes this commit.
No Tess shaders in my system either.

@mbechard @aglab2
PR #1677 possibly fixes this. I can't be sure, because I haven't been able to test your situations.
That PR is a fix to when a push constant in one pipeline used the same Metal buffer index as a descriptor in a previous pipeline, and if the descriptor was used again in a later pipeline, and only the offset has changed, the descriptor would use the MTLBuffer from the push constant.
Please retest using that PR, and close this issue if it fixes the problem you encountered.

Did not fix it for me, still the same assert. If you want to replicate my setup, you may use https://parallel-launcher.ca/ with "Parallel" selected as a graphics plugin and launch any ROM. libMoltenVK.dylib is placed in ~/Library/parallel-launcher/data/retroarch/RetroArch.app/Contents/Frameworks/libMoltenVK.dylib, app is just adhoc signed. To test it, you may launch the app in a debugger:
METAL_DEVICE_WRAPPER_TYPE=1 lldb ~/Library/parallel-launcher/data/retroarch/RetroArch.app/Contents/MacOS/RetroArch
(lldb) r -L ~/Library/parallel-launcher/data/retro-data/cores/parallel_n64_next_libretro.dylib --config ~/Library/parallel-launcher/data/retroarch.cfg "~/Downloads/test.z64"
...
validateComputeFunctionArguments:771: failed assertion `Compute Function(main0): argument _5227[0] from buffer(0) with offset(0) and length(16) has space for 16 bytes, but argument has a length(32).'

@aglab2
Thanks for providing the environment.
Where do I found the text.z64 game file? Or other game files?

Here is the test ROM you may use: test.z64.zip. It should look like this:

One more thing to check, I think the second line:
inline void update(const MVKMTLBufferBinding &other) {
    if (mtlBuffer != other.mtlBuffer || size != other.size || isInline != other.isInline) {

should instead read
    if (mtlBuffer != other.mtlBuffer || size != other.size || isInline || other.isInline) {

it doesn't matter if inline bindings have the same pointer and size, their data may be different.

it doesn't matter if inline bindings have the same pointer and size, their data may be different.

Good point. And isInline != other.isInline is effectively the same test as mtlBuffer != other.mtlBuffer, since inline and non-inline would necessarily have different mtlBuffer values.
As would isInline || other.isInline if the values of those two booleans are different. So perhaps the only thing that matters in the inline test is that we are now inline (other.isInline):
if (mtlBuffer != other.mtlBuffer || size != other.size || other.isInline) {

@mbechard @aglab2
Okay. This should be fixed now in the updated #1677. Verified with test.z64.zip.
In the previous version of that PR, for the descriptor binding that was being overridden, I was marking it as dirty too early. Fixed now.
The PR also includes my version of @js6i 's insight about tracking inline buffers.

Confirmed this fixes the issue for me, thanks!


Lifecycle Events as Observables

Ionic version:  (check one with "x")
[x] 4.x
I'm submitting a ...  (check one with "x")
[x] feature request
Please make the lifecycle events available as observables. There is a similar feature request for Angular proper to do the same (and a second FR).
The most obvious use case to unsubscribe from streams that you can't manage with the template.
constructor(
  private navCtrl: NavController,
  private lifecycleCtrl: LifecycleController
) { }

//...

ionViewDidLoad(): void {
  Observable.timer(0, 1000)
  .do(timerInt => {/* do stuff */})
  .takeUntil(this.lifecycleCtrl.ionViewWillLeave$);
  .subscribe();
}

// Or maybe it's part of the NavController?
// - this.navCtrl.lifecycle.ionViewWillLeave$ or
// - this.navCtrl.ionViewWillLeave$

But this could factor into your nav guards discussion as well.

Hello! Thank you for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#316


docs: add missing rustup dependencies to Building from source manual

REF: https://github.com/denoland/manual/blob/bdec6a0a07e76b64a1a38f39266babf2bebd58fd/contributing/building_from_source.md
Consider adding:
# Install rustup
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh

# Install clippy and rustfmt
rustup component add clippy
rustup component add rustfmt

to the contributing docs for deno. These are required to run various formatting tests locally.

Ah missed the Update or Install Rust link on the page 🤦


About locale

First of all, thank you for allowing us to use archlinux in wsl.
The default locale is 'C.UTF-8', we can easily set it to 'en-US.UTF-8' through editing locale.gen and running command locale-gen, but other language will meet an error, just like chinese:
Generating locales...
  en_US.UTF-8... done
  zh_CN.UTF-8...[error] cannot open locale definition file `zh_CN': No such file or directory

However, after reinstalling glibc, it works!
why?
Is this because the glibc that comes with the docker version of archlinux has been streamlined?
Maybe we can add a description to the readme.

seems like it is a docker image issue https://bbs.archlinux.org/viewtopic.php?id=253198

will see what I can do to fix it

That seems default docker image is heavily stripped

Hey, any progress in that topic?
I have an arch wsl installed that i have made using the docker image and have the same issue.
Was thinking about building an image myself when i found this repo.

Hey, any progress in that topic?
I have an arch wsl installed that i have made using the docker image and have the same issue. Was thinking about building an image myself when i found this repo.

just reinstall ~glibc~, everything will work, docker container a mini glibc


Simple example app with OneMobileSDK integration

@skalozubov Please briefly look. OneMobileSDK integrated through CocoaPods + PlayerControls at version 1.0.


Disaster mitigation feature of buildings or not?

We are finding buildings with columns in their base, and to determine the properties of the building we need to confirm whether the columns at the base of the building have the function of mitigating disasters such as floods or not.

From the past phases of HP project, the disaster mitigation parts of buildings are not considered in the drawn box, so, if that type of structure in Dominica have the function of flooding mitigation, the labeling would be like following.

Is it correct to consider those structures as disaster mitigation parts and not include them in the labeling box?
cc. @nualacowan, @srmsoumya, @piligab

Good Day to all. This is not a really a mitigation feature. We often build on pilllars to allow for future expansion by constructing a second residential unit on the ground floor.

Thank you for the clarification, @cw5711!
Then, if that is the case, the corresponding labeling for that type of buildings would be: "COMPLETENESS: INCOMPLETE" and the drawn box would include all the columns, as shown in the following example:

http://<IP_ADDRESS>:8080/tasks/36/jobs/36?frame=12

http://<IP_ADDRESS>:8080/tasks/40/jobs/40?frame=2

Does this make sense to you?

I'm actually not sure. These lower levels are often built long after the
main house has been built, sometimes more than 10 years later or it never
happens. Ground floor unit is essentially an optional future addition and
not part of the blueprint for the house in most cases.
Hope this helps.
On Fri, Nov 3, 2023, 5:14 AM Karito Tenorio Palomino <
@.***> wrote:

Thank you for the clarification, @cw5711 https://github.com/cw5711!
Then, if that is the case, the corresponding labeling for that type of
buildings would be: "COMPLETENESS: INCOMPLETE" and the drawn box would
include all the columns, as shown in the following example:

http://<IP_ADDRESS>:8080/tasks/36/jobs/36?frame=12

[image: Selection_051]
https://user-images.githubusercontent.com/11504548/280226334-2fc79ca6-8212-425e-8e99-d428da789aa3.png

http://<IP_ADDRESS>:8080/tasks/40/jobs/40?frame=2

[image: Selection_053]
https://user-images.githubusercontent.com/11504548/280229277-59c820e0-52b5-4658-858d-fea73c898e81.png
Does this make sense to you?
—
Reply to this email directly, view it on GitHub
https://github.com/developmentseed/housing-passports-v2/issues/10#issuecomment-1792095967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO3TKTXFLAWXZRR2UEZUXYDYCSYZHAVCNFSM6AAAAAA6W7YBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJSGA4TKOJWG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

@cw5711, then, if those type of columns do not are disaster mitigation features neither is part of the blueprint for the building, we could say that those cases should be labeled as the images below, considering only the structure above the columns and labeling them as "Complete".

@srmsoumya, will this be ok for the ML models, or maybe it could generate some confusion with incomplete buildings?

@karitotp I would go with what you are suggesting - consider the structure above the columns & label them as complete.


Update LoadErrorMessage components

Changes
Breaking:

Replaced showImg property with image. Rather than a {showImg: boolean} these are now set using {image: "none" | "virto-thinking" | "virto-shrugging" | "virto-head-scratching"}
Updated supportUrl prop name to contactUrl to align with the contactTeam property.

New:

Adds new image types for the card. These include virto-shrugging, virto-head-scratching and virto-thinking. These have also been updated to be a reasonable size rather that high-resolution.
Updated contactTeam to support a "none" value. This can be used to not show the contact message.
Fixed contact link not rendering correctly e.g. correct pointer.
Updated to latest storybook.
Exposed default props for each use case we currently have. Included is default404Props, default401Props and defaultErrorProps<LoadErrorMessage {...LoadErrorMessage.default404Props} />

Storybook:

Update the stories for this component to match the Figma provided by @Sowbhagya-ipg https://www.figma.com/file/5qkWskMGpXY38REVIDcJnW/Landing-Page?type=design&node-id=2921%3A4413&mode=dev&t=ZqF63hVNXKpQ64Yr-1

Dependencies
To be used in https://github.com/IPG-Automotive-UK/virto/pull/552
I've included some breaking changes. This is currently only used in an unreleased staging branch and so now is the right time to improve the API.
UI/UX

Testing notes

Author checklist before assigning a reviewer

[x] Reviewed my own code-diff.
[x] PR assigned to me or an appropriate delegate.
[x] Relevant labels added to the PR.
[x] Appropriate tests have been added.
[x] Lint and test workflows pass.

Looks Good! @mattcorner

I love the new VIRTO images @Sowbhagya-ipg


feat:🎸 take the issue with take 67

In this PR we will automate little bit this project. I saw this issue a lot of people work at the same time on the same issue.
@eneajaho Could you please add in secret section GITHUB_TOKEN
Closes #67
Tnx bdougie.

much needed. Thanks


How to use ansible-opendaylight in Ubuntu environment?

Hi Daniel, I am from OPNFV compass team. We are going to use this ansible-opendaylight in our compass installer which supports both centos and ubuntu os. It seems that currently only rpm package of ODL release is supported. So, will you also support deb package of ODL release?

Hello @Shelphy, thanks for asking. Yes, I talked with Compass folks about this at OPNFV Summit. As promised, I've been mentoring a Linux Foundation intern to add .deb build logic to ODL upstream. That project is done and the logic is here. Akshita is technically done with her internship, but I'm hoping she's willing to continue contributing by getting these debs hosted for downstream consumption. I'll send her this tread via email now.

Glad to hear that. Thanks a lot!

Hi @Shelphy ,
Deb support has now been added to ansible-opendaylight. It has been tested for Ubuntu 16.04 and Debian 8. Please let me know in case you face any issue.
Thanks.

Hi @AkshitaJha
Really awesome! I'll try it later. Thanks!

I'll try it later.
@Shelphy

You can test it pretty easily via @AkshitaJha's Vagrant base box examples of Ansible/Deb-based ODL deployments.


build: hitting various swiftc segfaults

These are hard to debug, but appear to be tied to exposing Objective-C types as part of the signatures of Swift interfaces.
It's maybe worth creating a swift bug report along with a condensed example, if this continues to be an issue.

@goaway is this still something that needs addressing by filing Swift bug report(s)?


feat: Add appropriate entities to showcase modal notifications

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines

Description of change

https://github.com/Jasig/NotificationPortlet/blob/master/notification-portlet-webapp/docs/modal.md
TODO

[x] Use the Notification modal Web Component (not the portlet)
[x] Apply the notification to Students only
[ ] Dismissing the modal window lasts the entire session
[x] Better text than 'Lorem ipsum'

Use the Notification modal Web Component (not the portlet)
Apply the notification to Students only
Dismissing the modal window lasts the entire session

Sure thing

Better text than 'Lorem ipsum'

What did you have in mind @drewwills ?

This is where c23c1ae puts us

Had to use NotificationPortlet-4.2.0 as NotificationPortlet-4.3.x was throwing erroring with a regenerator runtime error
Also, the modal doesn't render in Firefox

Dismissing the modal window lasts the entire session

jasig/NotificationPortlet#114 appears to be what we would need to make this happen. Is that right?

Yes, I can have a PR in a few days for Jasig/NotificationPortlet#114

@bjagg is Jasig/NotificationPortlet#114 still on the roadmap?


EventGrid: Regenerate .NET SDK for API version 2020-01-01-preview

Regenerate the .NET SDK files for API vesion 2020-01-01-preview and add new test cases for the new functionalities and its related recorded sessions.
The latest pull request for the Swagger spec review is located here: https://github.com/Azure/azure-rest-api-specs/pull/7681

Hi @ahamad-MS , could you

Add a link to the swagger spec review PR(s) in the description
Update versions if necessary
Thanks

Hello @isra-fel .. The link for the PR for the Swagger is added to the description section. However, not sure what version files you are referring to. Kindly advise if there is any additional files needs to be updated other than generated ones and test cases. thanks,.

Hello @isra-fel .. The link for the PR for the Swagger is added to the description section. However, not sure what version files you are referring to. Kindly advise if there is any additional files needs to be updated other than generated ones and test cases. thanks,.

I mean the versions in .csproj and AssemblyInfo.cs. If you're going to release the SDK you need to bump up the version. You can do it in this PR or another.

@isra-fel Thanks. Fixed.

Hi @ahamad-MS the versions in AssemblyInfo.cs should strictly follow 4-digit convention. I updated that to <IP_ADDRESS>.


Changed plot_complex to work like plt.plot

Changed plot_complex so that it functions just like plt.plot by grabbing the inputs and expanding the inputs that are arrays into the real and imaginary parts then passing all of that to plt.plot.  That way the syntax will be identical.

I closed this because I thought I had to make a new pull request because I changed one small thing.  If I'm understanding correctly, now, I just keep committing my changes and this one pull request will keep showing all my changes so that I don't have to do a pull request for each new change I make.  Is that right?

I closed this because I thought I had to make a new pull request because I changed one small thing.  If I'm understanding correctly, now, I just keep committing my changes and this one pull request will keep showing all my changes so that I don't have to do a pull request for each new change I make.  Is that right?


Cross compat with spelunkery salt?

For example ability to use their ground salt the same as your salt if it exists?
Probably should consider making a tag and using that tag for any and all your recipes also. cause I'd bet harvestcraft has it's own salt too, so that all the salts can be cross used. Since yours so far is the only one with dynamic food salting I think.

Salt already uses a forge:salts tag for recipes.
The only thing, I can think of, that's incompatible (or not configurable) right now is growing salt crystals - you can't change what block will grow.

chuck "spelunkery:salt" in there then also. theirs isn't tagged.
your own salt recipes also don't seem to actually take "any forge:salts" also.

All these things can be configured easily by datapack, KubeJS or CraftTweaker.

just trying to save others some time. so I don't have to edit all your recipes for your own tagged salt. :V

accepts properly.

doesn't accept properly.

unless I'm missing some coding shenanigans that'll just make it work anyways.

Gunpowder recipe uses tag.
I will consider changing other recipes.


build: bump all tichi plugins to v1.3.6

/retest pull-configs-validate-external-plugin-yaml

/test pull-configs-validate-external-plugin-yaml

/merge


CLI: fix script command example

While helping someone with cardano-adresses, I noticed that a new required CLI argument is missing from the examples.

Perhaps it would be beneficial to add e2e tests here similar to cardano-wallet's, that would just run through examples from a CLI...


Bump Knuu version to v0.13.2

Summary
hello team!
we have released the knuu version v0.13.2, feel free to bump it to this one, it should speed up the process + add the scope labels to simplify the way to find the metrics in Grafana, instead of using the namespace, we switch to scope.
one recommendation, if you are using the AddFile or AddFolder to mount files to the instances, I would recommend you to do it after the Commit, this will reuse previous images (using cached ones).
you can check the examples: AddFile or this one AddFolder, you'll find the Commit before.
any feedback or reported issue is welcome!
Best! 🚀
cc: @staheri14 @cmwaters @evan-forbes
Problem Definition
bump versions
Proposal
bump version to: v0.13.2

No instances of AddFile. Two instances of AddFolder which I'll rearrange to execute after Commit. I'll do that in a separate PR.


Update fabric sample configuration files

Description
As a developer I want to update the fabric sample configuration for the major operations and remove deprecated ones.
Acceptance Criteria

Update/Add Samples of major operations
Remove deprecated samples

I'll take this issue

We need to update the corresponding references for the same, in our documentation as well, as all of the code snippets from network.yaml file present in the documentation are direct references from these sample network.yaml files.

https://hyperledger-bevel.readthedocs.io/en/latest/operations/fabric_networkyaml.html


Add basic support for Read Attributes Response in chip-tool

Problem
Can't issue Read Attributes commands via chip-tool and get responses.
Summary of Changes
Add some encoder APIs for encoding Read Attributes commands and implement some very basic processing of the Read Attributes Response command in chip-tool
fixes https://github.com/project-chip/connectedhomeip/issues/1055 sort of...

Note: this is on top of https://github.com/project-chip/connectedhomeip/pull/1915 for now, hence draft.

@BroderickCarlin @andy31415 @jelderton How does this look to you?

@jelderton @saurabhst @BroderickCarlin ?

@BroderickCarlin @jelderton could you take a look please?


Empty Space at the end. Custom slick-slide width is not considered for calculating slick-track width

Problem
When giving specific width to items inside slider, even though the same width is given to the .slick-slide, the  total width of .slick-track is calculated wrong and it creates some empty space on the end of the slider.
When calculating the overall slider width, custom width given for .slick-slide is not considered.

Codesandbox URL is included

Same problem here

Same problem. I'm thinking about dynamic slidesToScroll so the last swipe will shift to correct slide

I couldn't figure out a solution. Ended up using a different library


Fix error messages and date parsing

Fix #153 (by adding on that index should be a positive number to the invalid command error message
Fix possible issue with tags error message: valid tags need to be separated by a single space, so I added that to the error message
Fix #175, #193 by making parser stricter (29-12-2021 is no longer allowed / rounded to the nearest valid date, 00-00-0000 is invalid), and made the error message more generic
Fix #168
Fix grammar and formatting of multidelete
Add a specific check for commands with command word cook to display recipe as its expected type.
Add a check for valid command words but missing type in parseGeneralCommand.
Standardise punctuation for list command success message with !
Add punctuation to invalid command error message and invalid command type error message
Add spaces to add recipe command example

Codecov Report

Merging #210 (4efe938) into master (1736d22) will decrease coverage by 0.24%.
The diff coverage is 55.17%.

@@             Coverage Diff              @@
##             master     #210      +/-   ##
============================================
- Coverage     69.97%   69.72%   -0.25%
- Complexity      767      773       +6
============================================
  Files           128      128
  Lines          2568     2590      +22
  Branches        316      322       +6
============================================
+ Hits           1797     1806       +9
- Misses          673      683      +10
- Partials         98      101       +3

Impacted Files
Coverage Δ

src/main/java/fridgy/commons/core/Messages.java
0.00% <ø> (ø)

...c/main/java/fridgy/logic/commands/ListCommand.java
100.00% <ø> (ø)

...c/main/java/fridgy/logic/commands/ViewCommand.java
100.00% <ø> (ø)

...fridgy/logic/commands/recipe/AddRecipeCommand.java
100.00% <ø> (ø)

...ridgy/logic/commands/recipe/CookRecipeCommand.java
87.50% <ø> (ø)

...ridgy/logic/commands/recipe/ListRecipeCommand.java
100.00% <ø> (ø)

...ridgy/logic/commands/recipe/ViewRecipeCommand.java
100.00% <ø> (ø)

src/main/java/fridgy/model/tag/Tag.java
76.92% <ø> (ø)

...rc/main/java/fridgy/logic/parser/FridgyParser.java
59.25% <14.28%> (-16.94%)
:arrow_down:

.../main/java/fridgy/model/ingredient/ExpiryDate.java
84.21% <88.88%> (-0.41%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Add template job config files.

Coverage remained the same at 0.0% when pulling 4efb86cfd63a40d6b311011c50bbc20b9f11a441 on mars-lan:template into 328381ae3aa7eac42af60e96d7d5870c6d6c9821 on linkedin:master.


Any option for popup editing?

Hello,
I am going to make a image upload function that can be display a dialog/popup for cropping, and then get the result back to base page.
Can this component do that?
Sample: https://imgur.com/qeQkmtK

You can set the image for the component via a value prop. It should be a DataURL.


"Role" property of conversation update activity set to null

Screenshots

Version
WebChat 4.9.1 via CDN
Describe the bug
When the conversationUpdate activity is sent by WebChat, the Role property of the activity is set to null. As a result, the ProactiveState does not get updated with new Conversation details since it looks for the role being equal to "user". This gets updated only when a skill is invoked/message is sent. As a result, if there are any proactive notifications, they are sent to the wrong conversation.
Steps to reproduce

Initialize connection to Bot through WebChat using DirectLine.
Open a connection to the Bot.
Note in the bot back-end logic that turnContext.Activity.From.Role is set to null, and not "user".
Note the error.

Expected behavior
WebChat should pass the relevant role as "user", so that the Proactive State functionalities are not broken.
Additional context
WebChat implemented in Angular application.
[Bug]

Hi @manish-95, can you post code for how you are generating a token in Web Chat as well as any relevant details (for example, is Enhanced Authentication enabled in the Direct Line channel)?
And, what API do you call to request the token? If you call an API of your own, can you also include that code?

@stevkan this schema may give you some hints.
In short, activity.from.role is something in the spec, but Direct Line service didn't send it to us. So Web Chat to have figure it ourselves.
I think @manish-95 is talking about SDK was not having the activity.from.role. So this one should either fixed in Direct Line service, or patched in SDK.
@stevkan could you follow up with bot team to see who should pick up this one?

@stevkan this schema may give you some hints.
In short, activity.from.role is something in the spec, but Direct Line service didn't send it to us. So Web Chat to have figure it ourselves.
I think @manish-95 is talking about SDK was not having the activity.from.role. So this one should either fixed in Direct Line service, or patched in SDK.
@stevkan could you follow up with bot team to see who should pick up this one?

@stevkan @compulim I can add a bit more information here which might help. The ProactiveState middleware in the latest version of the BotFramework Solutions, looks for activity.from.role, which is null, hence it fails to update the Proactive State. In previous versions, it was looking in the activity.from.properties, which had a key value pair for role

@EricDahlvang could you please look at this from an SDK perspective? We need some sdk guidance here. Thanks!

As William mentioned, 'role' is not sent by Direct Line.  Looking through the botframework-solutions repository, it appears this 'role' field has been adopted for a different purpose than originally intended.  And, there is currently an inconsistency between DefaultActivityHandler and ProactiveStateMiddleware
https://github.com/microsoft/botframework-solutions/blob/master/samples/csharp/assistants/enterprise-assistant/VirtualAssistantSample/Bots/DefaultActivityHandler.cs#L155
private void EnsureActivity(Activity activity)
        {
            if (activity != null)
            {
                if (activity.From != null)
                {
                    activity.From.Name = "User";
                    activity.From.Properties["role"] = "user";
                }

                if (activity.Recipient != null)
                {
                    activity.Recipient.Id = "1";
                    activity.Recipient.Name = "Bot";
                    activity.Recipient.Properties["role"] = "bot";
                }
            }
        }

https://github.com/microsoft/botframework-solutions/blob/master/sdk/csharp/libraries/microsoft.bot.solutions/Proactive/ProactiveStateMiddleware.cs#L31
if (!string.IsNullOrEmpty(activity.From.Role) && activity.From.Role.Equals("user", StringComparison.InvariantCultureIgnoreCase))

Lastly, the spec states here:
Channel account role
The role field indicates whether entity behind the account is a user or bot. This field is intended for use in the Transcript format [16] to distinguish between activities sent by users and activities sent by bots. The value of the role field is a string.

A7511: Senders SHOULD NOT include this field. Receivers SHOULD ignore this field.

Based on this, it seems the VA bot should adopt a different method for proactive messaging state.

Transferring to https://github.com/microsoft/botframework-solutions/


New package: WPIWnet.UniversalSilentSwitchFinder version <IP_ADDRESS>

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.2 schema?

Microsoft Reviewers: Open in CodeFlow

#31163 (reply in thread)
@jedieaston

Isn’t that the entire purpose of the pipeline and the malware scans though, is to catch when potentially malicious software links are used?
USSF has been around since 2011, and I know that myself and others who use winget are familiar enough with the package to know that it is safe. And, if it ever were to be compromised, that is why there is hash validations

USSF is great, but SoftPedia isn't trustable.

USSF is great, but SoftPedia isn't trustable.

I’m not saying that SoftPedia is trustable as a whole. I'm saying that in certain cases where we know the package is safe, there are checks in place that would prevent bad updates. In fact, no publiser website is truly "trustable", not even Microsoft's. All websites are vulnerable to compromise. Certainly SoftPedia being open to all increases the chances of the site hosting malware, but that doesn’t mean all packages are malware.
We know USSF is safe, and if the hash were ever to change then it would be blocked from installing without using the force parameter.

I can say that the publisher of that on Softpedia, wpiw.net, is blocked by MSIT as a malware site.  Doesn't make me feel a warm and fuzzy inside.
Calling for an adult.  @denelon !

I can say that the publisher of that on Softpedia, wpiw.net, is blocked by MSIT as a malware site. Doesn't make me feel a warm and fuzzy inside.
Calling for an adult. @denelon !

I will also add on to that statement that wpiw.net as a website is no longer a legitimate website (it likely was when the software came out, but alas, no longer). However, the software itself, being hosted on SoftPedia is used by community members and is known to be safe.
Perhaps changing the publisher name in the metadata to be WPIW may also help keep this package's users safe (if accepted)

Are there any other "main sites" besides WIPW and SoftPedia? I'm guessing our user agent or network is blocked by SoftPedia. I've taken a look at the URLs failing in validation and I don't have any problems with them. I also didn't see any licensing concerns, so I'm OK with a domain waiver on this package.

Looking at this more, Softpedia doesn't use a static download url (api changes it per user) so we can't use them anyways. I did find the github repo which directs users to MSFN which directs most users back to Softpedia. Most other sites I've found also direct back to Softpedia.
There is a download on SoftSea which may work
There is a download on Capstanservices which distributes it inside a Zip (Extracted the zip, hash is the same).
I'll try the SoftSea download to see if that remains static

If this isn't something we think will be highy downloaded, it could be a great .zip to add. (mentioning @ryfu-msft)
I would guess most users who are downloading this are likely to be using it for winget-pkgs submissions, so they would also be good candidates for using the experimental .zip feature as well.

If this isn't something we think will be highy downloaded, it could be a great .zip to add. (mentioning @ryfu-msft)
I would guess most users who are downloading this are likely to be using it for winget-pkgs submissions, so they would also be good candidates for using the experimental .zip feature as well.

I think it may be a good package to add the Zip installer for, but if portable works then I don't see a reason to delay

Maybe we could create a WIPWnet.UniversalSilentSwitchFinder.zip package. Users could still install the portable with 1.3 and we could use the other as a "temporary" placeholder for the .zip version. I don't necessarily want to maintain that convention, so I would expect to delete it once .zip is GA. We also need to look at the argument and filters for installer type so users can have multiple different installer types in the same manifest and be able to choose the one they want.

microsoft/winget-cli/issues/1166

Thoughts?
It might help avoid sad 1.3 users 😊.

@wingetbot run

Maybe we could create a WIPWnet.UniversalSilentSwitchFinder.zip package. Users could still install the portable with 1.3 and we could use the other as a "temporary" placeholder for the .zip version. I don't necessarily want to maintain that convention, so I would expect to delete it once .zip is GA. We also need to look at the argument and filters for installer type so users can have multiple different installer types in the same manifest and be able to choose the one they want.

Specify InstallerType winget-cli#1166

Thoughts?
It might help avoid sad 1.3 users 😊.

Might have to be just the Zip installer anyways; Seems SoftSea is returning this as the download now-
<script type="text/javascript" language="javascript">alert('Too much download times on this program, please try later.');location.href='http://www.softsea.com/review/Universal-Silent-Switch-Finder.html';</script>

@denelon - Blocking Issue on .zip please

This package will be used as a test example for the portables in zip feature.  This manifest will be checked in manually as zip manifests are not yet supported in validation. This also means that the latest stable 1.3 client will not be able to install this package as .zip is not yet supported in that version.


bl1830 data 2008 year

ROM ID:20 3C 00 01 00 07 98 69
Batt Msg: 30 26 A1 42 1F 58 00 00 C1 C1 32 61 23 42 01 73 C1 D0 8E 65 6F 43 0A 81 F1 02 1E E9 10 28 0F 23
battery is blocked, chip мс908jk3e
The battery was working, I blocked it myself, then I found out that not all batteries can be unlocked.
But it's not scary, I bought it for the purpose of studying.

The board magically unlocked after it had been unsoldered from the battery for a week.
It has changed Batt Msg: 30 26 A1 42 1F 58 00 00 94 94 32 61 23 42 01 00 C1 D0 8E 65 90 B2 00 01 02 02 0E 00 00 00 00 21
battery is currently charging, status unlocked.
but does not show the number of charges.
Maybe it will be useful.


Add support for account switching roles

Trying to use this at work we're unable to use our existing access keys and since they come from an AWS account which only has IAM users and little else. These users are then granted the ability to assume roles in different accounts where they can manage resources related to this account. The typical setup of ~/.aws/config and ~/.aws/credentials is something along the lines of...
~/.aws/credentials
[shared]
aws_access_key_id = <my-access-key-id>
aws_secret_access_key = <my-secret-access-key>
region = eu-west-1

~/.aws/config
[profile shared]
output = json
region = eu-west-1

[profile dev]
role_arn = <role from the dev account which allows me do do stuff, like launching ec2 instances>
source_profile = shared

Just this setup (which is what I'm using for everything aws cli), with aws.aws_profile = "dev" gives me a ~/.vagrant.d/gems/2.4.3/gems/vagrant-aws-0.7.2/lib/vagrant-aws/config.rb:537:in read_aws_files': undefined method []' for nil:NilClass (NoMethodError) and it appears like credentials needs to have a matching section for the profile. Putting my shared key in a [dev] section in ~/.aws/credentials, ending up with
~/.aws/credentials
[shared]
aws_access_key_id = <my-access-key-id>
aws_secret_access_key = <my-secret-access-key>
region = eu-west-1

[dev]
aws_access_key_id = <my-access-key-id>
aws_secret_access_key = <my-secret-access-key>
region = eu-west-1

~/.aws/config
[profile shared]
output = json
region = eu-west-1

[profile dev]
role_arn = <role from the dev account which allows me do do stuff, like launching ec2 instances>
source_profile = shared

gets me to...
... Subnet ID not found, suggesting that it's not trying to create the instance in the dev account, but rather in the shared account.
Haven't dug into the code and  not sure if this is a bug, feature or a lacking feature :)

This is not the same as using .aws/config and .aws/credentials, but I can use this plugin successfully with an assumed role by making sure these vars are set in my environment:
export AWS_ACCESS_KEY_ID=XXX
export AWS_SECRET_ACCESS_KEY=YYY
export AWS_SESSION_TOKEN=ZZZ
(they come from aws sts assume-role command)

I think you have accidentally mixed up the values of AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY

Would love to see this addressed.

Here's a way to do this in the Vagrantfile:
require "iniparse"

def get_role_creds(profile, session_name)
  # Run aws sts assume-role to get temporary creds for the assumed role
  data = File.read(open(ENV['HOME'] + '/.aws/config'))
  aws_config = IniParse.parse(data)
  profile_config = aws_config['profile ' + profile]
  role_arn = profile_config['role_arn']
  json = `aws sts assume-role --role-arn #{role_arn} --role-session-name #{session_name}`
  session_data = JSON.parse(json)
  creds = session_data['Credentials']
  return creds
end

Vagrant.configure("2") do |config|
  config.vm.box = "dummy"

  aws_profile = 'myprofile'
  aws_session_name = 'my_session_name_vagrant'
  creds = get_role_creds(aws_profile, aws_session_name)

  config.vm.provider :aws do |aws, override|
    aws.region = "us-west-2"
    aws.access_key_id = creds['AccessKeyId']
    aws.secret_access_key = creds['SecretAccessKey']
    aws.session_token = creds['SessionToken']

    # ...

  end

end


Installation Instructions

Using the instructions here 'https://github.com/doe300/VC4CL/wiki/How-to-get'
I'm stumbling at the first stage of installing, there doesn't appear to be a working link or on the nightly build either.
I must be missing something obvious could you point me in the right direction please.

Look again at the wiki. I had the instructions lying around for some time, just forgot to push them, sorry.
A side-note: The debian packages have not been tested (see #12) and they need (until https://github.com/doe300/VC4C/issues/29 is finished) SPIR-LLVM to be installed at the correct location (see here for instructions).

Many thanks,  can I suggest that to reduce the confusion specifying the Clang v3.9+ as the C++11 compiler. I have to say the less people have to do to install the less support issues you'll get. :)
But great work and many thanks - Pete


Broken on V2.0 canary

The plugin is broken on 2.0 canary.
V2.0 adds optional options to the addPassthroughCopy function which in turn causes this plugin to fail with the following error (even when those options are not passed in)
[11ty] The value for a path property in addPassthroughCopy options should be true or of type String. (via TypeError)
[11ty]
[11ty] Original error stack trace: TypeError: The value for a path property in addPassthroughCopy options should be true or of type String.
[11ty]     at PassthroughCopy.getOutputPaths (./11ty/node_modules/eleventy-plugin-clean/lib/passthrough-copy.js:59:15)
[11ty]     at async ./11ty/node_modules/eleventy-plugin-clean/lib/eleventy-clean.js:36:23
[11ty]     at async Promise.all (index 0)
[11ty]     at async updateFileRecordsFromPassthroughCopies (./11ty/node_modules/eleventy-plugin-clean/lib/eleventy-clean.js:32:3)
[11ty]     at async AsyncEventEmitter.<anonymous> (/./11ty/node_modules/eleventy-plugin-clean/lib/eleventy-clean.js:113:5)
[11ty]     at async Promise.all (index 2)
[11ty]     at async Eleventy.executeBuild (./11ty/node_modules/@11ty/eleventy/src/Eleventy.js:1093:7)
[11ty]     at async Eleventy.watch (./11ty/node_modules/@11ty/eleventy/src/Eleventy.js:924:18)
[11ty] Wrote 0 files in 0.02 seconds (v2.0.0-canary.12)

Thanks for filing the issue.
I fixed it and released v1.2.0.


Enable building containers from tags or branches

Currently building the container will always build from the tip of master on every repo.  The branch or tag ref needs to be passed to align with other manageiq repos that are required for components or features.

@simaishi @bazulay @fbladilo Please review.
cc @Fryguy

@Fryguy @chessbyte This should be backported to darga, but the default arg should be changed to darga, so it's not a cherry-pick.  I can make a darga PR once this is merged.

@bdunne @chessbyte  Yeah, for backport, I would suggest making a PR to darga with the cherry-pick, and then on top of that  put your changes.

@bdunne @Fryguy Please keep in mind that build args are not implemented in Docker 1.8.2 (for the build documentation), also do we want to take the oportunity and make version also a build arg with a default value of "Darga"?

@bdunne I know you are putting this in to allow for flexibility with manual builds, however ultimately I want to use docker hub's automated builds when they fix their bug.  Wouldn't this make that impossible? or would we just rely on default values on the branch?

@Fryguy My plan was to have default values on the branches to take care of the automated branch builds. (If that issue is ever fixed...)

Updated to use zip download rather than git clone for ManageIQ repos.  Leave ADD commented for Dockerfile development use.
@fbladilo @Fryguy @simaishi Please review.

Thanks @fbladilo I updated the doc as you suggested

Thanks guys from coming to a good compromise.  I'm going to merge as @bdunne and I have some nice changes on top of this for image size.

@chessbyte When this gets backported please let @bdunne know so he can change the REF to darga for the darga branch.

@bdunne backported.  Do your thing.


Support fragments in GraphQlTester

I am in the process of converting a codebase from the old GraphQL Java Kickstart library over to spring-graphql.
The test template in the testing framework for that library included a parameter that allowed you to optionally pass in fragment resource paths.
It would be helpful in the maintenance of test requests to support the loading of GraphQL fragments from disk. Currently, as far as I can tell fragments are only supported in the document method, which requires you to juxtapose your fragment with your query in a single string.
Being able to write something along the following lines:
graphQlTester
  .documentName("graphQLDocument")
  .withFragmentNames("fragmentOne", "fragmentTwo")
  .execute()

would be much nicer than having to repeat boilerplate in your test requests.

Duplicates #923
Please comment on that other issue to provide more feedback and ideas. Thanks!

The #import syntax suggested in #923 is a much broader feature as it tends to be used in schema files too. We don't necessarily need to go there to support reusable fragments, which is a much more specific goal that makes sense to support.
As mentioned in https://github.com/spring-projects/spring-graphql/issues/923#issuecomment-2014665969, we could also expose fragmentName and fragment methods on GraphQlTester, and is also what is suggested here, so let's use this issue to track the request.
It makes sense to have equivalent methods on GraphQlClient too.

In the meantime, since "documentName" handling can be customized, you could support a name like "myDocument::myFragment1::myFragment2". I haven't tested this, but a custom DocumentSource to support that could look like this:
public class ConcatenatingDocumentSource implements DocumentSource {

	private final DocumentSource delegate = new ResourceDocumentSource(
			List.of(new ClassPathResource("graphql-test/")), List.of(".graphql"));

	@Override
	public Mono<String> getDocument(String name) {
		return name.contains("::") ?
				Flux.fromArray(name.split("::")).flatMap(this.delegate::getDocument).reduce((s, s2) -> s + s2) :
				this.delegate.getDocument(name);
	}

}

You plug that in via GraphQlTester.Builder#documentSource . The delegate is what is used by default. In the custom implementation we are simply using it to load multiple documents and join them.

Thanks, this workaround is exactly what I needed to move forward without having to abandon fragment utilization in my test cases. Much appreciated!

@rstoyanchev Thank you very much! This worked great for me as well!


Podspec update -

Hi @tcoulter ,
it seems the old repository https://github.com/minsikzzang/jockeyjs got removed from github.
In the cocoapod spec repository it still points to the old version.
https://github.com/CocoaPods/Specs/blob/d0ec5a65e80656c8d78e12ff19f251df879e0bc2/Specs/0/f/6/JockeyJS/1.0.0/JockeyJS.podspec.json
I assumed just sending a pull request updating the spec would do the trick.
I see there is an update at https://github.com/tcoulter/jockeyjs/blob/master/JockeyJS.podspec
But simple updating the github reference isn't enough as it also looks for a tag 1.0.0 , and you github repo doesn't have any releases. Could you provide a 'release' and update the cocoapods spec?
I'm happy to help in case I can.

ping @tcoulter


[Event Request] Page 6057 - Contract Line Selection - OnLookupOKOnPush

We would like to request a new event called OnlookupOKPush(Var: OKButton,Rec) within the page Contract Line Selection, so we can customize accordingly please.
trigger OnQueryClosePage(CloseAction: Action): Boolean
begin

    if CloseAction = ACTION::LookupOK then

        LookupOKOnPush();

  **OnlookupOkOnpush(OKButton, Rec);**

    if OKButton then begin

        ServContract.Get(ContractType, ContractNo);

        CurrPage.SetSelectionFilter(Rec);

        ServContractLine.HideDialogBox(true);

        if Find('-') then

            repeat

                CheckServContractLine();

            until Next() = 0;

        CreateServContractLines();

        Commit();

    end;

end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52577.


Avoid using shared resource streams between class loaders when extracting the native library.

This changes makes it so that we replace Class#getResourceAsStream() with a custom method that finds the resources and creates a stream but disables caching the stream. This is necessary in environments with where the native library has to be extracted and used with multiple class loaders that get closed.
Specifically, I was observing the following JDK-8205976 in a Gradle project with concurrent tasks that use sqlite-jdbc. The exact exception I was observing was:
java.io.IOException: Stream closed
        at java.base/java.util.zip.InflaterInputStream.ensureOpen(InflaterInputStream.java:68)
        at java.base/java.util.zip.InflaterInputStream.read(InflaterInputStream.java:143)
        at java.base/java.io.FilterInputStream.read(FilterInputStream.java:133)
        at java.base/java.io.BufferedInputStream.fill(BufferedInputStream.java:252)
        at java.base/java.io.BufferedInputStream.read(BufferedInputStream.java:271)
        at org.sqlite.SQLiteJDBCLoader.contentsEquals(SQLiteJDBCLoader.java:179)
        at org.sqlite.SQLiteJDBCLoader.extractAndLoadLibraryFile(SQLiteJDBCLoader.java:243)
        at org.sqlite.SQLiteJDBCLoader.loadSQLiteNativeLibrary(SQLiteJDBCLoader.java:344)
        at org.sqlite.SQLiteJDBCLoader.initialize(SQLiteJDBCLoader.java:67)

Which lead me to JDK-8205976 after lots of logging. The test is a big complicated but sets ups the exact case for when this occurs. Multiple concurrent threads with isolated class loaders initializing the library for which the resource on all point to the same jar.

Hi @xerial - Sorry to ping you, but is there a chance you can take a look at this PR, would love for it to get merged if possible. Thanks!

ok. I'm not aware of this bug https://bugs.openjdk.java.net/browse/JDK-8205976. Let me check

@xerial, just a friendly ping on this, hope you are well. Thank you!

Hi @xerial , sorry for pinging here again, is there any chance of merging this and releasing a new version?
This became a bigger blocker for us AndroidX at this point (due to the adoption of Apple M1).
Let us know if we can help in any way.

@xerial it would be of great help, if you could merge this pull request. Those with M1 are having a tough time.

Can this be merged?

Yes. Thanks for waiting. I'll release 3.35.0 soon.


Remove redundant static variables in BR_Envelope::LaneMaxValue

"volenvrange" is never -666 (s_max never updated, and if it did it wouldn't follow changes to the volume envelope range setting until next startup...)
s_val dosen't need to have static storage since it's computed every time
A few integer comparison are plenty fast (beside most compiler generate a jumptable for that switch)

:white_check_mark: Build sws 945-master completed (commit https://github.com/reaper-oss/sws/commit/a34d30c8f8 by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-lanemaxvalue-cleanup-a34d30c8.exe

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-lanemaxvalue-cleanup-a34d30c8.exe

macOS x86 64-bit

build/sws-<IP_ADDRESS>-Darwin-x86_64-lanemaxvalue-cleanup-a34d30c8.dmg

macOS x86 32-bit

build/sws-<IP_ADDRESS>-Darwin-i386-lanemaxvalue-cleanup-a34d30c8.dmg

macOS ARM 64-bit

build/sws-<IP_ADDRESS>-Darwin-arm64-lanemaxvalue-cleanup-a34d30c8.dmg

Linux x86 64-bit

build/sws-<IP_ADDRESS>-Linux-x86_64-lanemaxvalue-cleanup-a34d30c8.tar.xz

Linux x86 32-bit

build/sws-<IP_ADDRESS>-Linux-i686-lanemaxvalue-cleanup-a34d30c8.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-lanemaxvalue-cleanup-a34d30c8.tar.xz

Linux ARM 32-bit

build/sws-<IP_ADDRESS>-Linux-armv7l-lanemaxvalue-cleanup-a34d30c8.tar.xz


Load pinpoint Flink job after Jobmanager startup

I set a way to load the job jar inside flink after JobManager startup it's may be not the best way in term of being compliant with Docker standard but it works.
Best would be to be able to use flink version >= 1.11 which allow the use of job cluster.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Update to version 1.7 of Flink see pinpoint issue #7550

Update to version 1.7 of Flink see pinpoint issue #7550

Thank you for your feedback. Understood.
I'm testing the PR.

pinpoint-web.properties or pinpoint-collector.properties are unnecessary. I can see those files are under flink.jar. But shouldn't have been there in the first place.

I am guessing what the lines in docker-entrypoint.sh do. Can you explain in detail? It will help clear things out in my head.

I've run the docker-compose, I can see the data coming in from pinpoint-agent. But I don't think flink is working. I can't see the aggregated data from flink.

data coming in

-flink part(aggregated data)

love to hear your feedbacks

In Flink version 1.11 and over there is a mode which is called job-cluster, it's a Flink cluster dedicated to run a predefined set of job so you directly add the job inside the Flink image and cluster will run those job. In your example case that's what we want to do so that what the standalone-job command will try to mimic with the pinpoint Flink job. If people want to still use stock Flink command they can and the will just start an empty Flink cluster and they will need to manually load the pinpoint job or any other job.
# In flink version < 1.11 we can't use "job cluster" because it's not yet available
if [ "$1" = "standalone-job" ]; then

The following is updating our template configuration file and updating the jar file with them, we can may be use the extra --spring.config.additional-location to do that but we need to be sure that the configuration file exist on the task-manager container running the job. If it's in the jar it's will be sent to the task-manager so no need to worry about that.
  cd /build
  sed -i -e "s/#PINPOINT_ZOOKEEPER_ADDRESS#/${PINPOINT_ZOOKEEPER_ADDRESS}/g" profiles/release/*
  zip -r pinpoint-flink-job-*.jar profiles/
  cd -

  mv /build/pinpoint-flink-job-*.jar pinpoint-flink-job.jar

  shift

Start jobmanager give it sometime to startup
  /docker-entrypoint.sh jobmanager &
  sleep 5

Check if the pinpoint job is already running on the cluster can happen if taskmanager is still running and jobmanager is restarted in that case we don't need to create the job again.
  if ! flink list | grep 'RUNNING'; then
    flink run $@
  fi
  wait

Classical Flink entrypoint:
else
  exec /docker-entrypoint.sh $@
fi

I've done the same test as yours, just docker-compose up,
Agent data:

Flink aggregated data:

If i check status of flink on http://<IP_ADDRESS>:8081, in running job and select the job, we can see received records numbers:

I've done the same test as yours, just docker-compose up,

Sorry about this.  Found the problem there was a minor typo.

I think you can remove pinpoint-collector.properties, pinpoint-web.properties.
Also please change docker-compose
image: ${PINPOINT_FLINK_NAME}-flink:dev
->
image: "pinpointdocker/pinpoint-flink:${PINPOINT_VERSION}"
this is relevant to postprocess behind. Which will automatically build and store image on dockerhub.

@marty-macfly Thanks for the explanation.

In Flink version 1.11 and over there is a mode which is called job-cluster
I've been informed that some are trying to upgrade the version of pinpoint-flink to 1.12.

Do you think we can try with v1.12 in advance?

Already tried to go higher that 1.8 and it doesn't work last version on which things work for me is version 1.8 of Flink. So we can bump to flink 1.8 but higher I will need advice to avoid error I've report in issue 7550 on pinpoint repository.

I think upgrading version to v1.12 will be completed soon. I think changing the relevant docker files to meet the version will be better after few days. Any suggestions?

I think we can do it in 2 times integrate that PR that will do the job for the moment and when version pinpoint-flink-job 1.12 is available do a new PR to integrate it. And if that PR is only useful for a week it's fine, unless you have any concern to do 2 PR ?


initial work for document requests

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

Just curious, what's this going to do? Remix already serves document requests for SSR so I suppose is another thing

All of the data loading / error handling logic is no longer going to be owned in the server-runtime, but rather by the router package. Essentially this now calls router.query(request) and that's it™

@brophdawg11 is there a decision doc for this work? Would help me follow along

@machour Yep!  https://github.com/remix-run/remix/blob/main/decisions/0007-remix-on-react-router-6-4-0.md - this PR is steps 1.iii/1.iv under Decision - steps 1.i/1.ii were already merged in #4245 and #4359 (behind the compile-time flag)

Need to add a changeset to describe the changes in error/catch boundary behaviors in Remix.  Merging this PR will enable those even without the flag enabled.

I think this is good to go now?  Let's touch base with @chaance and make sure that we're good to merge this (since he already cut the 1.7.6 release branch).  This code in dev, without the feature flag enabled, would just result in the minor fixes to catch/error boundary handling and could be part of a 1.7.7 if needed.  But the plan is to enable the feature flag for a 1.8.0-pre.0 later this week or early next week.


Auth: Login and Register for APP and API.

COLABORATORS

Samarvir Singh
Jean S Ariza

Description
Wire Frontend login and register components with the backend (API) Auth endpoints.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] APP were run in a local server. Interaction were done on the UI rendered in a browser.
[x] DevTools were used to check HTTP calls and responses.
[x] API were tested

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes THERE ARE NO TEST YET ON THIS BRANCH

To pay attention to when reviewing this code
The employers model has a little changes it one of its properties. The field companyName had changed to email.
Do not run you MongoDB and try to test the Auth endpoints befor you have done the either one of the followings:

Clean all your Employer data in the DB
Create a new DB in your Mongo DB

You can change in .env cahnge MONGODB_URI='mongodb://localhost/developer-profiles'
to MONGODB_URI='mongodb://localhost/developer-profiles2'

Also add to .env:
 JWT_SECRET="your api secret goes here"

BEFORE REVIEW
please read this previous comment: https://github.com/Lambda-School-Labs/CS10-Developer-Profiles/pull/31#issuecomment-421518889

I am still seeing blank error messages from this https://github.com/Lambda-School-Labs/CS10-Developer-Profiles/pull/31#discussion_r217778727. I'm just going to make an issue for that so this can get merged to meet the milestone.


C# support (suggestion)

This is a suggestion on how to add C# support.
While now C# assemblies will now be processed we still have the following problems:

Methods/Constructors will not be shown in the final documentation page, we need support in FSharp.Compiler.Service (maybe we should have this pull request open until the FSharp.Compiler.Service is fixed, as it is not as usefull until this is fixed, if you want to use this you can use matthid/FSharp.Compiler.Service@c5dfd4dd488f6dcd1024b0ed2b564ce9d2d414fa)
Exception when there are extension methods (#201)
Obviously line references are not working

The other commits add support for Assembly information in a type (you can add something like: Defined in Assembly "Other.dll" to you type template), see https://github.com/matthid/RazorEngine/blob/develop/doc/templates/reference/type.cshtml#L32. Now the namespaces are sorted (when you have multiple assemblies).

I've taken a look at this.  You should not really be calling ReturnParameter on a non-standard C# Event at all - this gets the type of an event if it is considereed to be like an IEvent-typed property by the F# type system.  But some events are "non-standard" and can't be given a good type as a property, and these must be called using add_XYZ and remove_XYZ.
That said, I'm adjusting FCS to still give a ReturnParameter with the delegate type of the event in this case. I'm also adding properties EventIsStandard, EventAddMethod and EventRemoveMethod to allow you to be more fine-grained if you need to be.


freshplayerplugin will not install on Linux Mint 18 again

Hi Rinat
It seems a new issue has now cropped up, it would appear that Clement Lefebvre and his team who develop and maintain Linux Mint 18 onwards, have made some minor, but very discreet changes to the o/s.
As the freshplayerplugin will now not install, it seems the o/s is blocking this somehow. I had to re-install Linux mint as I have change desktop tower PC's.
I have uploaded the screen shot for your perusal, the portion you need to look at is this "prerm called with unknown argument `install'"
I have not encountered this error before, as I had linux mint 18 on the previous machine, so what the problem is, isn't clear.
Kind Regards
Derrick W. Sampson
United Kingdom

Hi.
This error is most probably is in the binary package. As you may know, Debian packages (Mint and Ubuntu both use Debian package format) have also supplementary scripts, preinst, postinst, prerm, postrm, which are called before or after installation or removal.
Packaging manual says that prerm script can be called with actions remove, upgrade, remove in-favour, deconfigure in-favour, and failed-upgrade. So all scripts are expected to handle all those actions, but are not required to handle others. Most of script do nothing, but also most of them have error checking. As you can see, action install for prerm script is not mentioned in the allowed action lists. And it has no sense to call pre-removal script with action called install, since package is going to be removed.
I think, you're trying to install browser-plugin-freshplayer-pepperflash_0.3.6-1~webupd8~xenial5_amd64.deb from WebUpd8 PPA. If so, here is what its prerm script looks like:
#! /bin/sh

set -e

case "$1" in
    remove|deconfigure)
        update-alternatives --remove \
        mozilla-flashplugin \
        /usr/lib/browser-plugin-freshplayer-pepperflash/libfreshwrapper-flashplayer.so
        ;;

        failed-upgrade)
        ;;

        upgrade)
        ;;

        *)
        echo "prerm called with unknown argument \`$1'" >&2
        exit 1
        ;;
esac

As you can see, if called with install argument, it prints error message and terminates with error code 1, which in turn stops package manager.
The best solution would be to fix package manager in Mint. It shouldn't pass install action.
But there is also a workaround that you can use immediately. Edit file /var/lib/dpkg/info/browser-plugin-freshplayer-pepperflash.prerm, remove exit 1 line. That will prevent script from returning error. You then will need to call dpkg --configure -a as root to allow package manager to call all required scripts and mark packages as installed in its database.

Hi
I have followed your instruction on the latter first, and it would seem that Linux Mint has fouled the download up, because I typed in, in terminal "cd /var/lib/dpkg/info/" then used "gedit browser-plugin-freshplayer-pepperflash.prem, only to find that, that file does not exist.. The only file that does exist is pepperflash-plugin-nonfree.prem. And on the former we have Synaptic Package Manager, not Package Manager, if it's the Synaptic Package Manger that needs fixing, how to I do this?

Guess what? I was wrong. I installed Mint 18 into virtual machine, added PPA, tried to install browser-plugin-freshplayer-pepperflash and got the same error. Then tried to debug dpkg, only to find that it's preinst is failing, not prerm. Package manager is fine, and it even gave a pointer. It said: "pre-installation script returned error exit status 1". But I was mislead by that "prerm called with unknown argument".
The package have also preinst script probably copied from prerm:
#! /bin/sh

set -e

case "$1" in
    remove|deconfigure)
        ;;

        failed-upgrade)
        ;;

        upgrade)
        update-alternatives --remove \
        flash-mozilla.so \
        /usr/lib/browser-plugin-freshplayer-pepperflash/libfreshwrapper-flashplayer.so
        ;;

        *)
        echo "prerm called with unknown argument \`$1'" >&2
        exit 1
        ;;
esac

It's fine for preinst script to be called with install parameter. And if preinst from a package doesn't handle it, it's a packaging error.

Hello, @hotice.
There is very suspicious preinst script in browser-plugin-freshplayer-pepperflash_0.3.6-1~webupd8~xenial5_amd64.deb. Looks like there were some kind of copying from prerm, but install) case wasn't added. So package can't be installed. Could you take a look, please?

used "gedit browser-plugin-freshplayer-pepperflash.prem, only to find that, that file does not exist

My knowledge is outdated — dpkg's behaviour have changed, it now unpacks scripts into a temporary directory, which is deleted after. So that trick with editing files and continuing installation won't work anymore.

I'm working on a fix, thanks for mentioning this!

Done, the Ubuntu 16.04 / Linux Mint 18 package should be fixed. For other Ubuntu / Mint versions, the packages are still waiting to be built in my test PPA (Launchpad is laggy today). @DWSampson try to install the latest version (so run "sudo apt update" before trying to install it), it should work now.

Thanks for the heads up @hotice will let you know how I get on, have also posted this link on Webupd8, so that other uses of Linux Mint 18 know what's going.

@hotice Sorry no joy!! Please see attached Screen-shot
 , if people wish to play online games they will temporarily need to use Chromium until this is fixed.

This was a bug on my part, not in Mint. I explained everything in a reply to your comment on WebUpd8, which I'll copy/paste here for reference:
_The issue is not with Mint and not even Ubuntu. Google no longer bundles pepper flash with Google Chrome. To get Pepper Flash, you need to install the adobe-flashplugin package, which is available in the Ubuntu Partner repository (which needs to be enabled manually on Ubuntu but I think it's available directly in Linux Mint). The bug report you posted was an issue caused by my packaging, trying get the browser-plugin-freshplayer-pepperflash package to work with these changes.
I fixed the issue now.
So to get Pepper Flash to work in Firefox now, you must install browser-plugin-freshplayer-pepperflash and adobe-flashplayer (this is already in this article, as I've updated it last week)._

@DWSampson hmm the initial issue was that the package failed to install, not that it wasn't found. The package is in my PPA as you can see here: https://launchpad.net/~nilarimogard/+archive/ubuntu/webupd8/+packages?field.name_filter=freshplayerplugin&field.status_filter=published&field.series_filter=
Can you please try to update the software sources and try again? This should do it:
sudo apt update
sudo apt install browser-plugin-freshplayer-pepperflash

If it still doesn't work, please post the output of:
apt-cache policy browser-plugin-freshplayer-pepperflash

@DWSampson looking closer to your screenshot, you added a dot (".") at the end of the install command. Remove that and it should work.

@hotice will give that a go, give me a few minutes

@DWSampson  I had a typo in my earlier comment. You must install browser-plugin-freshplayer-pepperflash and adobe-flashplugin (I typed "adobe-flashplayer"). So this should install everything you need to get pepper flash to work in Firefox:
sudo apt install browser-plugin-freshplayer-pepperflash adove-flashplugin

(As a side note, the browser-plugin-freshplayer-pepperflash package has a "Recommends" field for adobe-flashplugin, which results in adobe-flashplugin being installed automatically in Ubuntu however, Linux Mint does not install recommended packages by default, so it needs to be installed manually).

@DWSampson  I had a typo in my earlier comment. You must install browser-plugin-freshplayer-pepperflash and adobe-flashplugin (I initially typed "adobe-flashplayer" so in case you're reading that in your email, you won't see the edit). So this should install everything you need to get pepper flash to work in Firefox:
sudo apt install browser-plugin-freshplayer-pepperflash adobe-flashplugin

(As a side note, the browser-plugin-freshplayer-pepperflash package has a "Recommends" field for adobe-flashplugin, which results in adobe-flashplugin being installed automatically in Ubuntu, however, Linux Mint does not install recommended packages by default, so it needs to be installed manually).

@hotice  it's installed it. please see screen shot.

@DWSampson great, thanks for the confirmation! Like I said earlier, make sure to also install adobe-flashplugin to actually get pepper flash working on Firefox.

It's worked issue resolved. @hotice have amended Synaptic Package Manager preferences accordingly to "Consider recommended packages as dependencies, in addition to this I have also set "Repositories to enable source code repositories".


simplify file name handling in ctrl_map_ini_gentim2d.F to make both TAF and OpenAD happy

What changes does this PR introduce?
Bug fix
What is the current behaviour?
After PR #751, TAF swallows some lines in tangent linear version of ctrl_map_ini_gentim2d, breaking TL-test hs94.1x64x5
What is the new behaviour
tests pass again
Does this PR introduce a breaking change?
No
Other information:
Suggested addition to tag-index
(To avoid unnecessary merge conflicts, please don't update tag-index. One of the admins will do that when merging your pull request.)

@mjlosch these changes look safe. the tapenade tests will still need to be run..

@dngoldberg I have a hard time with running tapenade, please test it if you get a chance.

@mjlosch i have run the tapenade streamice adjoint experiment, and it is fine. im not sure which experiments the TLM should be run for. running it for halfpipe_streamice gives only 1 decimal place of accuracy for the TL gradients, but the accuracies are fairly high. I never ran this experiment previously so im not sure if it ever worked, unfortunately..!

Good to hear, that it works for tapenade.
TL: this PR address an issue for TAF-generated TL-code (in particular hs.1x64x5). I am not sure if the tapenade TL code works at all (see issue #735), certainly not a subject for this PR.

@mjlosch yes, this experiment (tapenade TLM for streamice) was giving nan's before 751. so im tempted to go with shreyas's note about TLMs on this.
Im not sure how i formally approve.. or if i even can.. but LGTM

Just to clarify: with the current master branch and tapenade_3.16.v2 (I put a tar file here: https://stuff.mit.edu/~jm_c/bin_files/ ) all experiments pass the Adm and Tlm tests, see for the Adm ones:
http://mitgcm.org/testing/results/2023_09/tr_jaures_20230921_0/summary.txt
and for the Tlm one:
http://mitgcm.org/testing/results/2023_09/tr_jaures_20230921_1/summary.txt
but this is without "-devel" (just "-ieee") on Fedora 37, with gfortran.

but this is without "-devel" (just "-ieee") on Fedora 37, with gfortran.

thanks, I guess the -devel option is what the bullet point in issue #735 is referring to. Nonetheless, that's not the subject here and I don't think that my little change/suggestion affects the TLM of tapenade.
As far as I am concerned, this is ready to be merged. @jm-c, what do you think? Do you want me to write something more in the PR description? Do we need to ammend the tag-index?

@mjlosch I could merge this PR soon, like later today.
the changes are minor, I might add 1 line in tag-index, have not decided yet, but this PR description is very clear.