Is there someone has met the issue when you use  AQL queries in Spark, the query is normal after a while will report 404

My Environment

ArangoDB Version:        3.3.16
Storage Engine:             RocksDB
Deployment Mode:       Cluster
Deployment Strategy:    ArangoDB Starter
Configuration:
Infrastructure:
Operating System:
Total RAM in your machine:
Disks in use:
Used Package:

Component, Query & Data
Affected feature:

AQL query (if applicable):
1.
WITH Company, Person
FOR v, e, p IN 1..3 ANY  'Company/${md5_comp}' invest,tradable_share,suspect_same_company OPTIONS {uniqueVertices: 'path'}
FILTER p.vertices[-1].is_belong == "true" or p.vertices[-1].is_belong == true or ${ignoreIsBelong.toString}
RETURN p
2.
WITH Company, Person
FOR v, e, p IN 4 ANY  'Company/${md5_comp}' invest,tradable_share,officer,person_merge,suspect_same_company OPTIONS {uniqueVertices: 'path'}
FILTER split(p.vertices[2]._id, "/")[0] == "Person"
FILTER p.vertices[-1].is_belong == "true" or p.vertices[-1].is_belong == true or ${ignoreIsBelong.toString}
RETURN p
AQL explain (if applicable):
I want to query a company for all its affiliates in 3 degrees and wtih second vertex was people all its affiliates in 4 degrees.
Problem:
INFO org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) - ShuffleMapStage 5 (distinct at BlackListQueryFromArangoV1.scala:215) failed in 325.108 s due to Job aborted due to stage failure: Task 150 in stage 5.0 failed 1 times, most recent failure: Lost task 150.0 in stage 5.0 (TID 1150, localhost, executor driver): com.arangodb.ArangoDBException: Response: 404, Error: 1600 - cursor not found
at com.arangodb.internal.velocystream.VstCommunication.checkError(VstCommunication.java:106)
at com.arangodb.internal.velocystream.VstCommunicationSync.execute(VstCommunicationSync.java:134)
at com.arangodb.internal.velocystream.VstCommunicationSync.execute(VstCommunicationSync.java:46)
at com.arangodb.internal.velocystream.VstCommunication.execute(VstCommunication.java:96)
at com.arangodb.internal.velocystream.VstProtocol.execute(VstProtocol.java:46)
at com.arangodb.internal.ArangoExecutorSync.execute(ArangoExecutorSync.java:58)
at com.arangodb.internal.ArangoExecutorSync.execute(ArangoExecutorSync.java:47)
at com.arangodb.ArangoDatabase$1.next(ArangoDatabase.java:326)
at com.arangodb.internal.ArangoCursorIterator.next(ArangoCursorIterator.java:66)
at com.arangodb.ArangoCursor.next(ArangoCursor.java:117)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$.expandKLevelPath(BlackListQueryFromArangoV1.scala:134)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$$anonfun$8$$anonfun$9.apply(BlackListQueryFromArangoV1.scala:199)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$$anonfun$8$$anonfun$9.apply(BlackListQueryFromArangoV1.scala:199)
at scala.collection.Iterator$$anon$11.next(Iterator.scala:409)
at scala.collection.Iterator$$anon$12.nextCur(Iterator.scala:434)
at scala.collection.Iterator$$anon$12.hasNext(Iterator.scala:440)
at scala.collection.Iterator$$anon$11.hasNext(Iterator.scala:408)
at org.apache.spark.util.collection.ExternalSorter.insertAll(ExternalSorter.scala:191)
at org.apache.spark.shuffle.sort.SortShuffleWriter.write(SortShuffleWriter.scala:63)
at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:96)
at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:53)
at org.apache.spark.scheduler.Task.run(Task.scala:99)
at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:325)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
Driver stacktrace:
this error appearance after program run about 10 minutes.
[2019-04-22 16:44:47,394] INFO org.apache.spark.internal.Logging$class.logInfo(Logging.scala:54) - Job 2 failed: collectAsMap at BlackListQueryFromArangoV1.scala:217, took 325.633962 s
Exception in thread "main" org.apache.spark.SparkException: Job aborted due to stage failure: Task 150 in stage 5.0 failed 1 times, most recent failure: Lost task 150.0 in stage 5.0 (TID 1150, localhost, executor driver): com.arangodb.ArangoDBException: Response: 404, Error: 1600 - cursor not found
at com.arangodb.internal.velocystream.VstCommunication.checkError(VstCommunication.java:106)
at com.arangodb.internal.velocystream.VstCommunicationSync.execute(VstCommunicationSync.java:134)
at com.arangodb.internal.velocystream.VstCommunicationSync.execute(VstCommunicationSync.java:46)
at com.arangodb.internal.velocystream.VstCommunication.execute(VstCommunication.java:96)
at com.arangodb.internal.velocystream.VstProtocol.execute(VstProtocol.java:46)
at com.arangodb.internal.ArangoExecutorSync.execute(ArangoExecutorSync.java:58)
at com.arangodb.internal.ArangoExecutorSync.execute(ArangoExecutorSync.java:47)
at com.arangodb.ArangoDatabase$1.next(ArangoDatabase.java:326)
at com.arangodb.internal.ArangoCursorIterator.next(ArangoCursorIterator.java:66)
at com.arangodb.ArangoCursor.next(ArangoCursor.java:117)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$.expandKLevelPath(BlackListQueryFromArangoV1.scala:134)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$$anonfun$8$$anonfun$9.apply(BlackListQueryFromArangoV1.scala:199)
at com.haizhi.blacklist.BlackListQueryFromArangoV1$$anonfun$8$$anonfun$9.apply(BlackListQueryFromArangoV1.scala:199)
at scala.collection.Iterator$$anon$11.next(Iterator.scala:409)
at scala.collection.Iterator$$anon$12.nextCur(Iterator.scala:434)
at scala.collection.Iterator$$anon$12.hasNext(Iterator.scala:440)
at scala.collection.Iterator$$anon$11.hasNext(Iterator.scala:408)
at org.apache.spark.util.collection.ExternalSorter.insertAll(ExternalSorter.scala:191)
at org.apache.spark.shuffle.sort.SortShuffleWriter.write(SortShuffleWriter.scala:63)
at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:96)
at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:53)
at org.apache.spark.scheduler.Task.run(Task.scala:99)
at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:325)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
Expected result:
company with its all associated companys or people.

Please edit your post and fill out the template. We need to know what your environment is, what you use in which version and how this problem can be reproduced.

@Simran-B I have update my arango cluster  enviroment and the error information.thank you for answer my question.

@zhuhan66 please add the driver version

I'm sorry about that I reply you guys too slowly.@OmarAyo @BCaxelbecker @Simran-B
driver version is below:

com.arangodb
arangodb-java-driver
4.2.2

And there is another problem I met recently.I use the cluster mode of arango with 3 computer,but when I delete data ,only the data of the primary node can be deleted, and the data of the two duplicate nodes cannot be deleted.please help me.

Hi,
sorry for the late reply. There are two reasons for Cursor not found:

You are talking to the wrong coordinator, the cursor lives on the other. Meanwhile later Arango-Releases coordinators will forward the cursor request to the proper coordinator.
the cursor has timeouted. You need to specify a longer TTL so it lasts until you reach out for it again.

Anyways, ArangoDB 3.3 has reached EOL meanwhile; please upgrade your environment.
Please note that ArangoDB 3.7 has been released - closing this.
If you still encounter similar issues after an upgrade, please open a new issue.


fix(mqtt): fix epsagon-id as strong id

:tada: This PR is included in version 1.57.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use of PWM pins

The GPIO register commands you use don't seem to work to well on the PWM pins (GPIOs 12, 13, 15).
If you replace:
WRITE_PERI_REG( PERIPHS_GPIO_BASEADDR + GPIO_ID_PIN(WSGPIO), 1 );
with
gpio_output_set(GPIO_BIT_ID, 0, GPIO_BIT_ID, 0);
where GPIO_BIT_ID = BIT13 for pin13
Now these pins seem to work fine.
Great lib by the way.

Thanks.  Do you mind just making this change yourself and pushing it to my repo?

It doesn't seem to work for WSGPIO=4 or WSGPIO=5 either ( for GPIO4 and GPIO5)?

Ah! I believe that is because of the multiple functions on the pins.  You have to configure pins to be GPIO in some cases.
This is the code used in esp8266ws2812i2s
	static const uint32_t AFMapper[16] = {
		0, PERIPHS_IO_MUX_U0TXD_U, 0, PERIPHS_IO_MUX_U0RXD_U,
		0, 0, 1, 1,
		1, 1, 1, 1,
		PERIPHS_IO_MUX_MTDI_U, PERIPHS_IO_MUX_MTCK_U, PERIPHS_IO_MUX_MTMS_U, PERIPHS_IO_MUX_MTDO_U };

	int nr = my_atoi( &pusrdata[2] );

	if( AFMapper[nr] == 1 )
	{
		buffend += ets_sprintf( buffend, "!G%c%d\n", pusrdata[1], nr );
		return buffend - buffer;
	}
	else if( AFMapper[nr] )
	{
		PIN_FUNC_SELECT( AFMapper[nr], 3);  //Select AF pin to be GPIO.
	}


Add build files to build project with Gradle

Looks like this PR addresses #124

Thanks for this, but there was some other cleanup that I wanted to get done so I covered it elsewhere.


updated eslint and rules, inspired by new internal rules

✌
On Thu., Dec. 12, 2019, 16:41 Pablo Rincon<EMAIL_ADDRESS>wrote:

Merged #1 https://github.com/SSENSE/eslint-config-ui-core/pull/1 into
master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SSENSE/eslint-config-ui-core/pull/1?email_source=notifications&email_token=AAEOZXLHYVDMZ7XEIDYH33TQYKVYNA5CNFSM4FBYVSOKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVOQVNOI#event-2879477433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEOZXOXTGZRLKPM4UUZ343QYKVYNANCNFSM4FBYVSOA
.


IDs in some methods should be optional

I noticed that methods related to MenuButton have an optional ID field in Telegram API, but televerse uses a non-nullable ID type. Telegram only accepts either a valid ID or absence of the field, so a hack like ID.create("") doesn't work. Absence of the field is used to set default state for the property.
There might be other fields like this one with a similar omission.

As a side question while I'm on it, is it possible to send a custom request? I didn't find a method like it and RawApi seems to be non-extensible. but it would help in blocker cases like this one.

Hey, thanks for reporting! This must be a mistake from my side. As per the documentation the setChatMenuButton method only requires the MenuButton and not the chat_id.
I'll fix this within a couple of hours.
And for the second question, as of now, we don't have a method to make custom request. But it sounds like a good idea as well! Will look into this in the next releases!

Hey @Sominemo, just released v1.13.3 to pub.dev. Thanks again for raising the issue. Please feel free to raise more, or discuss in our Telegram chat, I guess you're already in there! ✨


NSLocalizedString creates layout problem

Hi,
I am using latest version of Active label library
It was working fine with English language, but when started to localized words like following it start showing layout problem ..
in Localisation file
"test writing text for active label" = "تجربة كتابة عربية للاكتف لابيل";
in code:
label.customize{ label in
                    label.text = NSLocalizedString("test writing text for active label", comment: "")

                label.sizeToFit()
                label.adjustsFontForContentSizeCategory = true

                label.font = UIFont.preferredFont(forTextStyle: .footnote)

                label.textColor = UIColor.contentText2
                label.handleHashtagTap { hashtag in
                    print("Success. You just tapped the \(hashtag) hashtag")
                }

}
but result showing one word missing لابيل

can you help please

I am not sure if it helps, but you should always use fonts who support RTL languages. We've had some weird stuff happening with RTL languages and the solution was to use a font that supports RTL languages and not just a default font.

👍👍
Thanks a lot maziyarpanahi for your advice.
It works, when i used this font
label.font = UIFont.init(name: "Arial", size: 20)
many thanks to you

I am glad it helped 👍


Document + add sample in readme for the default configuration

We should explain how to use the default configurations for the factories.
I'm thinking a section in the README.

@MatFillion Closing this issue, I think it's fixed. See https://github.com/nventive/Chinook.DataLoader#define-default-behaviors for more details.


[pgpu-render] Fixes to PgpuRenderer::render

Fixed the is_color check to look for MTLPixelFormat::R8Unorm instead
of MTLPixelFormat::A8Unorm since the backing texture format always
uses the former.
Added a call to metal::CommandBufferRef::finish() to clean up the blit
command encoder to free the command buffer for new encoders.

The version of the pipeline this is targetting no longer exists. I suggest we close this

Closing as it's pretty far out of date. I am hoping to pick up the integration work again, but it would be based on the new wgsl pipeline.


New version: Kubernetes.minikube version 1.23.2

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

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

I got late by 20 seconds ☹

😁 Filled in some more Metadata, can accept yours if you add the metadata

Done 👍


Two Enhancements

It would be nice to be able to hide apps (for example I have language-packs that show up) that I don't need to see in the list.
Another nice enhancement would be choose sort order; package name or app name.

Sorting already has an issue #61
Please make a new issue for hiding apps.


Odd Suppress Behavior

System: Windows 11
Sharphook version: 5.3.2
.NET Version: 8.0
Hello! I'd like to "convert" a real mouse button 4 click into a simulated mouse button 2 click. My set up is that I listen for mouse button 4 press and release events, suppress that event, and simulate mouse button 2 press and release events. However, I cannot get this to work when I simulate a mouse event. Both the original mouse click and simulated mouse go through. I can suppress the original mouse click when I simulate a keyboard event though.
Here is a video to show what I'm writing about: https://www.youtube.com/watch?v=TDHwdC9fakI
In the video I show two scenarios:

Suppress mouse button 4 and simulate w key press (mouse button 4 suppressed)
Suppress mouse button 4 and simulate mouse button 2 (mouse button 4 not suppressed)

We can see that scenario two does not work because the tabs get flipped through in notepad++ because of mouse button 4.
Here is the code in the video:
using SharpHook;
using SharpHook.Native;

namespace rmbtest;

class Program
{
    static void Main(string[] args)
    {
        var hook = new SimpleGlobalHook();
        var simulator = new EventSimulator();
        hook.MousePressed += (sender, eventArgs) =>
        {
            switch (eventArgs.Data.Button)
            {
                case MouseButton.Button4:
                    Console.WriteLine(eventArgs.IsEventSimulated);
                    eventArgs.SuppressEvent = true;
                    // simulator.SimulateKeyPress(KeyCode.VcW);
                    simulator.SimulateMousePress(MouseButton.Button2);
                    break;
                case MouseButton.Button5:
                    hook.Dispose();
                    break;
            }
        };
        hook.MouseReleased += (sender, eventArgs) =>
        {
            switch (eventArgs.Data.Button)
            {
                case MouseButton.Button4:
                    Console.WriteLine(eventArgs.IsEventSimulated);
                    eventArgs.SuppressEvent = true;
                    // simulator.SimulateKeyRelease(KeyCode.VcW);
                    simulator.SimulateMouseRelease(MouseButton.Button2);
                    break;
            }
        };
        var t = new Thread(() =>
        {
            hook.Run();
        });
        t.Start();
        t.Join();
    }
}

Am I doing this incorrectly?
Thank you!

Hello! Thanks for posting this issue! I'm not sure, seems like it should work the way you assume. I will look into it.

Well, I can reproduce this behavior, but I don't know why that happens, and searching for that particular behavior in Windows API didn't yield any results, though I do think it's some weird quirk of the Windows API. I will try looking into it later, but for now all I can do is just shrug.
As a workaround, you can simulate events in a different thread. For example, wrapping calls to eventSimulator in Task.Run made this work for me - but you shouldn't use Task.Run as you need to guarantee the correct sequence of simulated events, and Task.Run won't have such a guarantee.

I appreciate you looking into this. I recreated the issue with direct win32 calls using the cswin32 nupkg. Comment lines 32-42 to not send rmb clicks and properly suppress xb1:
using System.Runtime.InteropServices;
using Windows.Win32;
using Windows.Win32.Foundation;
using Windows.Win32.UI.Input.KeyboardAndMouse;
using Windows.Win32.UI.WindowsAndMessaging;

namespace newtest;

class Program
{
    static unsafe void Main(string[] args)
    {
        var handle = PInvoke.GetModuleHandle((string?)null);
        var hhook = PInvoke.SetWindowsHookEx(
            WINDOWS_HOOK_ID.WH_MOUSE_LL,
            (code, param, lParam) =>
            {
                if (code < 0)
                {
                    return PInvoke.CallNextHookEx(null, code, param, lParam);
                }
                if (param == PInvoke.WM_XBUTTONDOWN || param == PInvoke.WM_XBUTTONUP)
                {
                    var msll = Marshal.PtrToStructure<MSLLHOOKSTRUCT>(lParam);
                    // switch on high-order word
                    switch (((ushort*)&msll.mouseData)[1])
                    {
                        case PInvoke.XBUTTON1:
                            Console.WriteLine("XB1");

                            // send rmb
                            var input = new INPUT { type = INPUT_TYPE.INPUT_MOUSE };
                            input.Anonymous.mi.dx = msll.pt.X;
                            input.Anonymous.mi.dy = msll.pt.Y;
                            input.Anonymous.mi.dwFlags =
                                MOUSE_EVENT_FLAGS.MOUSEEVENTF_ABSOLUTE
                                | MOUSE_EVENT_FLAGS.MOUSEEVENTF_RIGHTDOWN
                                | MOUSE_EVENT_FLAGS.MOUSEEVENTF_RIGHTUP;
                            input.Anonymous.mi.mouseData = 0;
                            input.Anonymous.mi.dwExtraInfo = 0;
                            input.Anonymous.mi.time = 0;
                            PInvoke.SendInput(1, &input, sizeof(INPUT));

                            // suppress xb1
                            return (LRESULT)(-1);
                    }
                }
                return PInvoke.CallNextHookEx(null, code, param, lParam);
            },
            handle,
            0
        );
        while (PInvoke.GetMessage(out var lpMsg, HWND.Null, 0, 0) != -1)
        {
            PInvoke.TranslateMessage(lpMsg);
            PInvoke.DispatchMessage(lpMsg);
        }
    }
}

To be able to run that code, it needs a project with cswin32 nupkg and a NativeMethods.txt designating which functions/structs to generate. I've created that here: https://github.com/FaithBeam/win32-suppress-issue

Well, in that case I don't think I can really do anything to fix this behavior as this is a quirk of the Windows API itself. I think the best way to do what you want is to simulate events on a different thread.
I'm going to close this issue as a won't-fix, but we can continue the discussion.


Error: "Something went wrong. The container schema was not created. Network error: Response not successful: Received status code 401"

Ran into the below error - "Something went wrong. The container schema was not created. Network error: Response not successful: Received status code 401", while creating the custom object.
PROJECT_KEY and AUTH_TOKEN were set as per the instructions
$ export PROJECT_KEY={project_key}
$ export AUTH_TOKEN={access_token}
Please let me know if am missing anything here.

This error indicates an issue with authentication.
Are you trying to create a new GraphQL schema file (https://github.com/commercetools/commercetools-mc-custom-objects-starter#generating-graphql-schema) where you need to export those values?
Are you trying to run it locally? In which case you need to ensure the environment variables are set correctly (https://github.com/commercetools/commercetools-mc-custom-objects-starter/blob/master/env.local.json) and you have a valid login to the commercetools Merchant Center.  Please be sure you are connecting to the correct region for your account.

Thanks @jluterek , I am running the application locally, the issue is resolved now.


Added npm files filter

For smaller package

Don't forget to make patch release please.

Done.
On 12/08/2015 13:21, Bogdan Chadkin wrote:

Don't forget to make patch release please.
—
Reply to this email directly or view it on GitHub
https://github.com/hiddentao/fast-levenshtein/pull/8#issuecomment-130171476.

--
Ramesh Nair
Freelance Web Developer
www.hiddentao.com
www.github.com/hiddentao

Thanks!


Infer model was not loaded normally on the Jetson TX2 development board

2022-05-24 10:48:17,167   INFO  ==> Loading parameters from checkpoint ./checkpoint_epoch_1.pth to GPU
2022-05-24 10:48:21,515   INFO  ==> Checkpoint trained from version: pcdet+0.5.2+c533261+py48dbe0c
2022-05-24 10:48:21,631   INFO  Not updated weight backbone_3d.conv_input.0.weight: torch.Size([3, 3, 3, 4, 16])
2022-05-24 10:48:21,631   INFO  Not updated weight backbone_3d.conv1.0.0.weight: torch.Size([3, 3, 3, 16, 16])
2022-05-24 10:48:21,632   INFO  Not updated weight backbone_3d.conv2.0.0.weight: torch.Size([3, 3, 3, 16, 32])
2022-05-24 10:48:21,632   INFO  Not updated weight backbone_3d.conv2.1.0.weight: torch.Size([3, 3, 3, 32, 32])
2022-05-24 10:48:21,632   INFO  Not updated weight backbone_3d.conv2.2.0.weight: torch.Size([3, 3, 3, 32, 32])
2022-05-24 10:48:21,632   INFO  Not updated weight backbone_3d.conv3.0.0.weight: torch.Size([3, 3, 3, 32, 64])
2022-05-24 10:48:21,632   INFO  Not updated weight backbone_3d.conv3.1.0.weight: torch.Size([3, 3, 3, 64, 64])
2022-05-24 10:48:21,633   INFO  Not updated weight backbone_3d.conv3.2.0.weight: torch.Size([3, 3, 3, 64, 64])
2022-05-24 10:48:21,633   INFO  Not updated weight backbone_3d.conv4.0.0.weight: torch.Size([3, 3, 3, 64, 64])
2022-05-24 10:48:21,633   INFO  Not updated weight backbone_3d.conv4.1.0.weight: torch.Size([3, 3, 3, 64, 64])
2022-05-24 10:48:21,633   INFO  Not updated weight backbone_3d.conv4.2.0.weight: torch.Size([3, 3, 3, 64, 64])
2022-05-24 10:48:21,633   INFO  Not updated weight backbone_3d.conv_out.0.weight: torch.Size([3, 1, 1, 64, 128])
2022-05-24 10:48:21,634   INFO  ==> Done (loaded 245/257)
test.pcd
Infer model was not loaded normally on the Jetson TX2 development board
how to solve?

his happens to all of second,pointpillar,voxel_rcnn_car...

Could you make sure that this also happen in pointpillar?


"No Refresh Token found in the cache. " exception when running BackgroundWorker

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [x] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Running SPA and API, logged in with some valid users and call Web API.
After that, able to see the correct records in SQL Azure for MsalAccountActivities and TestCache, which share the same Id and CacheKey.
But when trying to run BackgroundWorker, it raised exception saying "No Refresh Token found in the cache. "
More detail callstack:
at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__18.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.<RefreshRtOrFailAsync>d__14.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.<ExecuteAsync>d__5.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.<ExecuteAsync>d__5.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Microsoft.Identity.Client.Internal.Requests.RequestBase.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.ApiConfig.Executors.ClientApplicationBaseExecutor.<ExecuteAsync>d__2.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at BackgroundWorker.Program.d__4.MoveNext() in C:\git\ms-identity-dotnet-advanced-token-cache\1-Integrated-Cache\1-2-WebAPI-BgWorker\BackgroundWorker\Program.cs:line 122
Any log messages given by the failure

at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__18.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.<RefreshRtOrFailAsync>d__14.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Identity.Client.Internal.Requests.Silent.CacheSilentStrategy.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.<ExecuteAsync>d__5.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at Microsoft.Identity.Client.Internal.Requests.Silent.SilentRequest.<ExecuteAsync>d__5.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at Microsoft.Identity.Client.Internal.Requests.RequestBase.d__12.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at Microsoft.Identity.Client.ApiConfig.Executors.ClientApplicationBaseExecutor.<ExecuteAsync>d__2.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at BackgroundWorker.Program.d__4.MoveNext() in C:\git\ms-identity-dotnet-advanced-token-cache\1-Integrated-Cache\1-2-WebAPI-BgWorker\BackgroundWorker\Program.cs:line 122
Expected/desired behavior

BackgroundWorker should able to retrieve the accessToken correctly with cached refresh token.
OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I'm also having the same issue, albeit with the sample implemented into my code. Would be great to get some help with it!

@mywhere is your application configured to use SQL or Redis?

@mywhere @TanmayDharmaraj
Would you guys be able to share the DefaultSlidingExpiration setting for your background workers?
If this value is not set it will default to 20 minutes

Checking in.
Did the above help diagnose or resolve your issue?

Also running into this issue, if anyone can point me in the right direction, it would be much appreciated!

I reran the sample and found a bug within the BackgroundWorkerTokenCacheAdapter class.
There is a pull request here.
@cmgilles if you need the sample to work adding the following methods in the BackgroundWorkerTokenCacheAdapter.cs file should resolve your issues.
        protected override async Task RemoveKeyAsync(string cacheKey, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogInformation($"RemoveKeyAsync::cacheKey-'{cacheKey}'");
            await _distributedCache.RemoveAsync(_cacheKey).ConfigureAwait(false);
        }

        protected override async Task<byte[]> ReadCacheBytesAsync(string cacheKey, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogDebug($"ReadCacheBytesAsync::cacheKey-'{cacheKey}'");
            return await _distributedCache.GetAsync(_cacheKey).ConfigureAwait(false);
        }

        protected override async Task WriteCacheBytesAsync(string cacheKey, byte[] bytes, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogInformation($"WriteCacheBytesAsync::cacheKey-'{cacheKey}'");
            await _distributedCache.SetAsync(_cacheKey, bytes, _cacheOptions).ConfigureAwait(false);
        }

Be sure to add using Microsoft.Identity.Web.TokenCacheProviders.Distributed; at the top of the file to include the CacheSerializerHints.
From a high level, the issues seems to be how the tokens are stored in SQL. In sample 1-1 the default identifiers are used for each token which the Microsoft.Identity.Client uses to query for tokens. In sample 1-2, however, the tokens are serialized with a different ID which is why the adapter is used to ensure the core library is querying for the correct tokens. The original code only overrode one of the Remove, Read and Write methods that did not accept CacheSerializerHints as a parameter which resulted in cache misses.
Let me know if this resolves your issues.

Reran the sample and found a bug within the BackgroundWorkerTokenCacheAdapter class.
There is a pull request here.
@cmgilles if you need the sample to work adding the following methods in the BackgroundWorkerTokenCacheAdapter.cs file should resolve your issues.
        protected override async Task RemoveKeyAsync(string cacheKey, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogInformation($"RemoveKeyAsync::cacheKey-'{cacheKey}'");
            await _distributedCache.RemoveAsync(_cacheKey).ConfigureAwait(false);
        }

        protected override async Task<byte[]> ReadCacheBytesAsync(string cacheKey, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogDebug($"ReadCacheBytesAsync::cacheKey-'{cacheKey}'");
            return await _distributedCache.GetAsync(_cacheKey).ConfigureAwait(false);
        }

        protected override async Task WriteCacheBytesAsync(string cacheKey, byte[] bytes, CacheSerializerHints cacheSerializerHints)
        {
            _logger.LogInformation($"WriteCacheBytesAsync::cacheKey-'{cacheKey}'");
            await _distributedCache.SetAsync(_cacheKey, bytes, _cacheOptions).ConfigureAwait(false);
        }

Be sure to add using Microsoft.Identity.Web.TokenCacheProviders.Distributed; at the top of the file to include the CacheSerializerHints.
From a high level, the issues seems to be how the tokens are stored in SQL. In sample 1-1 the default identifiers are used for each token which the Microsoft.Identity.Client uses to query for tokens. In sample 1-2, however, the tokens are serialized with a different ID which is why the adapter is used to ensure the core library is querying for the correct tokens. The original code only overrode one of the Remove, Read and Write methods that did not accept CacheSerializerHints as a parameter which resulted in cache misses.
Let me know if this resolves your issues.

Worked.  Thanks!

Please feel free to re-open if issue persists.

@luismanez
Thank you for your patience.
These issues have been resolved with the merge of pull request 18
You should now be able to run this sample using the latest libraries without issue.
You mention a viable solution already here.
For anyone else looking at this issue I'd recommend you look at the issue referenced above and also this documentation on OBO flow.
The changes with the sample now include updated libraries and removal of the BackgroundWorkerTokenCacheAdapter class and leverage the AcquireTokenInLongRunningProcess  method.
// BackgroundWorker/Program.cs

var result = await ((ILongRunningWebApi)app)
                .AcquireTokenInLongRunningProcess(
                      scopes,
                      account.AccountCacheKey)
                 .ExecuteAsync();

Works for both Redis and SQL.

Thanks @v-michaelmi !! Glad to see the sample update, so it´s still relevant. Worth to mention that if you try this in an Azure Function as background process, you won´t be able to use MS Identity Web library as the Builders, like MicrosoftIdentityAppCallsWebApiAuthenticationBuilder, will change Az Function Auth scheme breaking the Function Default Auth mode (Function key, etc). As I mention in the other thread, the solution is to use MSAL with the AcquireTokenInLongRunningProcess (as you show here). Also worth to mention that the MS Identity Web team is working in a new version that will allow to use the library in this scenarios.
Thanks again!


More browser zoom options

The zoom levels options are not precise enough.
50% is way too small, while 100% is too big for a small tablet (I am displaying the Home Assistant UI)
Ideally the zoom level should be picked freely from a wider range (eg. 50-200%) or at least from more precise steps (e.g. 50-55-60....)

Same issue here, a free number input would be great to fine tune

I have added more zoom options, but just so you know, this feature does not work as expected.  It does not just zoom in.  This is because it has more to do with the initial screen dimensions of the device.   It will not work like a magnifying glass which is what most people assume.   From the documentation:

public void setInitialScale (int scaleInPercent)
Sets the initial scale for this WebView. 0 means default. The behavior for the default scale depends on the state of WebSettings#getUseWideViewPort() and WebSettings#getLoadWithOverviewMode(). If the content fits into the WebView control by width, then the zoom is set to 100%. For wide content, the behavior depends on the state of WebSettings#getLoadWithOverviewMode(). If its value is true, the content will be zoomed out to be fit by width into the WebView control, otherwise not. If initial scale is greater than 0, WebView starts with this value as initial scale. Please note that unlike the scale properties in the viewport meta tag, this method doesn't take the screen density into account.

I'm  not sure if I understand.
I am not expecting a magnifying glass effect (what you would imagine zooming into a picture would be) but the same zoom behavior you get on a desktop browser with CTRL - or CTRL +.
I'll try the new options as soon as they hit the Play Store.

The WebView is not the same as the browser on the device, so no you will not get the same zoom behavior as CTRL +/- .  This feature works differently as I described and its the only zoom type feature we have available in WebView.

It might still be good enough (although since the lovelace UI in home assistant has the collapsable sidebar it's less of an issue for me).
I look forward to try it!


Update dependencies version

Currently, some of the jax libraries are pinned to these versions:
jax==0.4.1
jaxlib==0.4.1
flax==0.6.3

I think last time I check there were some conflicts between newer versions of package, especially distrax is not synced to newer version of Jax. Might worth checking whether it is possible to sync flowMC to version newer than 0.4.8.

@charlyandral Great to know! RQSpline uses the bijector from Distrax, whereas the RealNVP is custom-built. So this confirms indeed distrax is the problem here.


speed up texture loading - minimize vector alloc

We're spending a lot of time loading textures in wasm debug builds. It's a bit of a false alarm in that non-wasm builds and non-debug builds don't seem to be much affected by it, but it can't hurt.
before: 2.0s

after 0.36s

It's significantly faster, but still surprisingly slow (360ms in debug). Further improvements could be done by operating on the bytes directly, but the windowing logic we get from GenericImageView is annoying enough to implement that it didn't seem worth it for now.

Thanks for the fix! Working well for me.
I actually didn't realize browsers had such nice profiling tools built in. I just tried the one in Firefox and am excited to dig into all sorts of slowness with initialization. Are you using Safari there? The UI seems a bit more understandable.

Are you using Safari there? The UI seems a bit more understandable.

Those screenshots are from Chrome. I sometimes browse with Safari, but I think Chrome is probably the gold standard for browser developer tools.


Serialize for SearchResult<T> broken

This is a follow up to #29. Thanks to the fixes, it now works fine for releases.
However, if I serialize a SearchResult<Release> and then deserialize it again, I\m getting the following error:
JSON error: unknown field `entities`, expected one of `created`, `count`, `offset`, `artists` at line 1 column 74

Use Case: I'm building a cache implementation for an application I'm working on to reduce the load on MusicBrainz servers.

Search results have a custom deserializer so yeah it's going to break. I'll see what I can do as I never tried custom de/serializers before.
But good luck on your cache! I tried to make one myself using cacache + bit serialising and it was so painful still that I'm just going SQLite...

Alright, I tried to do it, but I have not enough knowledge in serializers to actually do it.
What I could suggest is maybe create a SearchResultDe similar to the original that can deserialize from the badly serialized json. Then add a From to convert to a standard search result.
This one is definitely a "need help"

Fixed in https://github.com/RustyNova016/musicbrainz_rs_nova/pull/40/commits/568789bbfa8f38106d7f3cc51c3ef08eb5c2c34b


Handle the case of multiple IP addresses for publicDNSIP

We have noticed that sometimes on AWS platform the ingress.hostname of LB service is resolved to multiple IP addresses, and each time their order may change.
This PR updates PublicDNSIP to support multiple IP addresses.
Signed-off-by: Yossi Boaron<EMAIL_ADDRESS>

Looks good @yboaron. Do you want to add a unit-test case for this use-case?

Looks good @yboaron. Do you want to add a unit-test case for this use-case?

Any idea how to simulate multi IP addresses DNS Resolution ?
IIUC, multi IP address is not supported in nip.io

@yboaron how critical this fix is? should we backport it?

@yboaron how critical this fix is? should we backport it?

@nyechiel well, LB service is relevant only for ROSA and ARO, I couldn't even reproduce the issue on AWS, in my tests LB DNS hostname resolved to more than one IP only for a short time about 30-60 seconds,
I don't think we should backport this PR

Personally I have noticed this behaviour only recently and I'm not sure if AWS is experimenting this feature.
Either way, as long as the public IPs associated with ELB falls back to a single IP (hopefully in a minute or two as Yossi observed) which points to the zone in which submariner-gateway pod is scheduled, we should be good.

Looks good @yboaron. Do you want to add a unit-test case for this use-case?

@sridhargaddam Any idea how to simulate multi IP addresses DNS Resolution ? IIUC, multi IP address is not supported in nip.io

Correct Yossi, that's what I'm seeing too. Thanks for checking.


add setting of prefix parameter to None or value in json config file.

With this change following is achieved;
The json credentials are read from the json file.

If the file contains a key 'prefix' it is set to the prefix.
if the file contains NO key 'prefix'  it is set to None.

You can than use the function get_prefix() to get the value.

Can one of the admins verify this patch?

Hi @cookie33. Could you please re-create the pull request against the devel branch?

PR has been re-created against the devel branch (see #10)


refreshRegistersAsync called in tight loop and killing perf

Looks like we are calling this in a tight loop.

fixed


Docker and .net

https://gist.github.com/endofcake/8af9f7debbd3e8784a15db48c37b5b42/

See also https://docs.docker.com/engine/examples/dotnetcore/

https://devblogs.microsoft.com/dotnet/collecting-net-core-linux-container-cpu-traces-from-a-sidecar-container/ looks interesting


if statements in parsers

Personnel

[x] Owner: @mbudiu-vmw
[ ] Supporters:

Design

[ ] Document: https://github.com/p4lang/p4c/issues/2127

Implementation

[x] p4-spec: #808
[x] p4c: https://github.com/p4lang/p4c/pull/2386

Process

[x] LDWG discussed: https://docs.google.com/document/d/1XSgdXeG1UuF1FM_XAqxDrHeN4dHZWBnJPKVS6SnGNwM/edit#, July 2020 meeting
[ ] LDWG approved:
[ ] Merged into p4-spec:
[ ] Merged into p4c:

start: { bit<32> x; if (x) { .x. } else { .x. } }

To consider: multiple transitions in a state.


readme: Use SVG image for build status badge

More accessible, better scaling.

Could you explain it more? Why do we need to make the change?

Cheap to make
Some screen-readers may read SVG text to the user
SVGs are, well, vector graphics, and render with increased readability, especially if the page is zoomed-in.
Rendering text in an image when you could use vector graphics is silly

FWIW the other badge next to it is already an SVG. If you zoom in the page the difference is obvious.

I see. LGTM.
Thanks for the contribution!


Construct exact POSIXTimeRange given to TxInfo

There are cases where we need to be able to obtain the exact POSIXTimeRange that gets passed to scripts with TxInfo - for example if the there are time-dependent values on Datums.
The problem is that there is a round-trip conversion from

the POSIXTimeRange provided to MustValidateIn
Conversion to Slot
conversion to TransactionValiditySlot
conversion back to POSIXTimeRange, when the ledger constructs TxInfo

See pg 35 of the Alonzo spec for part of the conversion. This may also require some reverse-engineering of the cardano-ledger

I did some digging and cardano-ledger implements slotToPOSIXTime [1], using epochInfoSlotToUTCTime
https://github.com/input-output-hk/cardano-ledger/blob/2acff66e84d63a81de904e1c0de70208ff1819ea/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/TxInfo.hs#L186-L203
That function in turn is provided by cardano-slotting, with addtional parameters about the protocol and epoch:
https://github.com/input-output-hk/cardano-base/blob/8fe904d629194b1fbaaf2d0a4e0ccd17052e9103/slotting/src/Cardano/Slotting/EpochInfo/API.hs#L80-L85
Most things can be directly queried from the node, and the EpochInfo can be calculated from EraHistory which can also be calculated.
https://github.com/input-output-hk/cardano-node/blob/91a11ba26707d2de1c72abfe4a15f59f84e8e349/cardano-cli/src/Cardano/CLI/Shelley/Run/Query.hs#L1313-L1316

Ultimately it comes down to properly using the outputs of the following slotToWallclock query. In that module we also have wallclockToSlot which would be useful in the other direction.
https://github.com/input-output-hk/ouroboros-network/blob/bd9e5653647c3489567e02789b0ec5b75c726db2/ouroboros-consensus/src/Ouroboros/Consensus/HardFork/History/Qry.hs#L461-L481


POST response status code is not acted on for external fact requests

The following function is used to perform external fact POST requests:
https://github.com/Comcast/rulio/blob/54befe498aa13ff976ca65c7404e4aa8df4dcbb5/core/http.go#L387-L395
Since the result status code is not used anywhere within this function or the functions it calls, and since it is not propagated to the calling function, there is no information available to event processing about unexpected/unwanted status codes.
I propose defining a fixed set of status codes for external fact requests, and handling them either in the above function or in the calling function:
https://github.com/Comcast/rulio/blob/54befe498aa13ff976ca65c7404e4aa8df4dcbb5/core/query.go#L376-L404
This could be as simple as (pseudocode) status == 200 ? 👍  : 👎

FYI I would be happy to create a PR for this.  But of course I want to run this by the maintainers first.

Good point. The very simple approach you suggest sounds good. As it stands, we are somewhat limited in our ability to handle bad fact responses, so happy/sad analysis might be sufficient. A PR would be great.


update location of esprima.js in package.json "main" field

Current coverage is 100.00%

Merging #1384 into master will not affect coverage as of e2931b2

@@            master   #1384   diff @@
======================================
  Files            1       1
  Stmts         3060    3060
  Branches       585     585
  Methods          0       0
======================================
  Hit           3060    3060
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of e2931b2

Powered by Codecov. Updated on successful CI builds.

:+1:

We should probably release a patch update after this lands.

We should probably release a patch update after this lands.

Why is that? There is no release yet based on master.

Ah, I forgot.


add ignore .log for hs_err_pid.log/replay_pid*.log

Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

This pr conflicts with master. And where are these logs from?

This pr conflicts with master. And where are these logs from?

hs_err_pid*.log/replay_pid*.log generated when jvm crash on maven build.

That is unexpected, why we need to ignore those? We don't have tests for JVM crashing. The ignore file is not for all caees, it just ignore for tool config or test generated files.

That is unexpected, why we need to ignore those? We don't have tests for JVM crashing. The ignore file is not for all caees, it just ignore for tool config or test generated files.

当构建时内存不足就会产生这些临时文件, we should ignore it just like target dir.

You could have many files, such as dump file. It is not reasonable to add ignore for all.


For fun - Steam comments that are pretty trait_dumb3

Compiling a selection of 'edge of the bell curve' comments, from people that clearly haven't tried the mod, or don't read before complaining or asking, or just say straight up rubbish. Because why get annoyed when you can have a good laugh about it?

The ominvore trait, giving bonus growth speed, appears to have no effect in game.

There is no omnivore trait

whats up with the lack of adaptability traits?

does this mod loer the music volume?

Interesting collection of traits, but can we please have a version without the silly and cheaty stuff? I dread what the computer could do with them selecting randomly.

You need to remove the Masohist Trait from AI's and possibly hive minds anything that Has no need for ethics attraction. its just rediculusly OP.

I try to answer any questions if I can when I get on because some are actual valid bug reports and very rarely they have actual decent suggestions

I know and I surely appreciate it! I deliberately evade Steam for its comment section.

Could you maybe edit the description on the Steam pages? I think the two versions could use cross-links to each other, so people are aware there is an expanded version (Heavy version is significantly lagging in subscribers on steam, whereas on the Nexus pretty much everyone downloads both).
A clear link to the subreddit wouldn't hurt either, so people can get additional information and discussion and stuff.

The 2 versions have cross links now as well as one to the subreddit

the addon traits for robots are not working, I think because they have the same name as the vanilla robot traits. For example, this mods mass produced trait and the one in vanilla have the same name but it doesnt replace the vanilla one, so when you select it the game resets it to the vanilla version rather than this mods version and you cant select it. Please fix this.

Mod conflict.

The traits show up fine in the empire creation menu, but when starting the game not a single AI empire will have any traits. Doesn't seem to be an issue on my end since I wiped everything, so I'll probably just give this mod a couple weeks before trying again >.>

This guy probably checked the AI races at the first day, when they indeed have no traits. Or mod conflict.

I think the Tiny mod is bugged for AI empires. It applies the tiny modifier twice for mineral gains. -75% and -80%, guessing it's been forgotten after a tweek?

Nope it doesn't. Proooobably a mod conflict.


test.failed branch statement ignored in report.html

In the main template file  report.html two different syntax are used to asses if a test case has failures
line 29:
{% if test.failed() %}
lines83, 91:
{% if test.failed %}
It seems second variant will return true even if no failures, or the branch statement be disregarded, because for me this caused elements associated with failures to be printed into the output file, despite no failures in the test. Adding the brackets fixed it.

Moved to https://gitlab.com/inorton/junit2html/-/issues/28


React Top-Level API Spanish Translation

Work in progress
This is the translation for the React Top-Level API page, assigned to me on issue #4

Hi, @karolincastano! Check the translation guide. You shouldn't translate heading IDs.

Hi @alejandronanez, @carburo, @dmoralesm I've updated this PR with the latest changes. Also, I restored the heading IDs to their original form.


Update old incompatible python 3 code in statemanager, roverprocess and main

Only goal is the code needs to not crash when you do a pull request!

[x] main
[x] satemanager
[x] roverprocess
[ ] Brag that we have updated to the most cutting edge standards!

May want to put this on the same branch as the server updating branch.

I got the core rover running in python 3. Just fixed the print statements, a few changes with the semantics of including modules, and stopping the ReceiverThread in the RoverProcess.
Is this ready for a pull request, or should we wait until things like the CAN server are ported as well?

I did it last night too with the standard 2to3 script in tools. Does yours crash on closing?

Also I'd keep this branch open until the end of the milestone

No it doesn't crash; it looks like python 3 depreciated thread._Thread__stop() so I changed the cleanup method to use join() instead.

Great! We'll hold onto this branch and see if we can port the servers or rewrite them. Would you be able to do a brief writeup on what you did for the wiki for future reference?

Sure. Should I just make a new page called Porting to Python 3 or something like that?

Yeah whatever you like - you may want to create some sections. (This is more so for issue #9)


Contrast Loss did not go down

When training on S3DIS using PointTransformer, I find that the contrast loss did not go down, but the CrossEntropy Loss go down normaly.
Can you provide the detailed trainnig log for me?

Hi,
Thanks for your interest and sorry for my late reply.
We have provided the logs along with the pre-trained model here.
Best,
Liyao


MacOS builds are not published as snapshot releases anymore

Going to https://github.com/OpenRefine/OpenRefine/actions/runs/8355021909
I see
mac_test_and_deploy
No files were found with the provided path: ./packaging/target/openrefine-mac-3.9-SNAPSHOT.dmg. No artifacts will be uploaded.

and I don't see any DMG in the artifacts.
We should make it possible for Mac users to easily test our master branch by downloading a snapshot release.

This seems to have been a transitory problem, as the DMG build is available again here:
https://github.com/OpenRefine/OpenRefine/actions/runs/8545103408


Dependency review workflow can't write to pull requests

There is a warning that review-dependencies workflow can't write to the pull request.
I suppose it means ilee can't see whatever advice it provides.
This could block from merging pull requests.

Note @JamieSlome

See https://github.com/Citi/gradle-helm-plugin/blob/master/.github/workflows/dependency-review.yml#L8

@magner669 - the GitHub Action will still work as intended, it just means that the bot will not automatically comment on the pull request but rather provide the feedback in the GitHub Actions terminal only.

@magner669 - perhaps adjust the workflow to use the exact YAML:
https://github.com/finos/git-proxy/blob/main/.github/workflows/dependency-review.yml#L2-L6


CI: use Composer stable release for AppVeyor

@keradus @SpacePossum @julienfalque should we use Composer stable version and go with this PR or wait for Composer snapshot to be fixed?

I see no reason to prefer snapshot release over stable, let's keep stable.

Thank you @kubawerlos.


Minor improvements to the wrapper.sh script

I added bashisms and tightened up the code a bit.

Thanks!


Analyze Image 4.0 API does not provide angle?

I've used Computer Vision 3.2 Read API before, and it offers the angle.

Are there any methods to get the text orientation from the API itself if using Analyze Image 4.0 API?

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

ID: 36aef639-5b98-c8b6-0710-ae10bab78d87
Version Independent ID: dfd0c4fb-b85e-973d-bce4-fbc493738242
Content: OCR for images - Azure AI Vision - Azure AI services
Content Source: articles/ai-services/computer-vision/concept-ocr.md
Service: azure-ai-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@Huirouuu
Thanks for your feedback! We will investigate and update as appropriate.

@Huirouuu
Yes, the Analyze Image API in version 4.0 preview offers the ability to detect the orientation of the text in an image. You can include the Read feature in the features query parameter to extract the text from the image, and the response will include the "textAngle" property, which represents the angle, in degrees, of the detected text orientation. Here's an example of what the response might look like:
{
    "language": "en",
    "textAngle": 0.0,
    "orientation": "Up",
    "regions": [
        {
            "boundingBox": "0,0,1024,768",
            "lines": [
                {
                    "boundingBox": "71,71,882,71",
                    "words": [
                        {
                            "boundingBox": "71,71,882,71",
                            "text": "Hello"
                        }
                    ]
                }
            ]
        }
    ]
}

In this example, the "textAngle" property is 0.0, indicating that the text is oriented horizontally.

@Huirouuu Yes, the Analyze Image API in version 4.0 preview offers the ability to detect the orientation of the text in an image. You can include the Read feature in the features query parameter to extract the text from the image, and the response will include the "textAngle" property, which represents the angle, in degrees, of the detected text orientation. Here's an example of what the response might look like:
{
    "language": "en",
    "textAngle": 0.0,
    "orientation": "Up",
    "regions": [
        {
            "boundingBox": "0,0,1024,768",
            "lines": [
                {
                    "boundingBox": "71,71,882,71",
                    "words": [
                        {
                            "boundingBox": "71,71,882,71",
                            "text": "Hello"
                        }
                    ]
                }
            ]
        }
    ]
}

In this example, the "textAngle" property is 0.0, indicating that the text is oriented horizontally.

Is this property available in the latest version of SDK?
I'm following the quick start article for Image Analysis 4.0 (https://learn.microsoft.com/en-us/azure/ai-services/computer-vision/quickstarts-sdk/image-analysis-client-library-40?pivots=programming-language-python&tabs=visual-studio%2Cwindows), and I've included the Text feature to my analyzer.

The JSON response seems have different format from the sample above, and there's no attribute related.

@Huirouuu
I've delegated this to @PatrickFarley, a content author, to review and share their valuable insights.

Hi @Huirouuu , text angle is not available in the latest SDK, and that was by design. If I may ask, what is your use case that requires the text angle?

@Huirouuu it is also no longer available in the latest REST API. The OCR (Read) result was simplified. See "readResult" in the schema tab: https://eastus.dev.cognitive.microsoft.com/docs/services/Cognitive_Services_Unified_Vision_API_2023-10-01/operations/61d65934cd35050c20f73ab6

@Huirouuu  Feel free to reopen this issue if you'd like us to explore your use case
#please-close

While processing the rotated samples, Azure returns the ROI for the rotated texts, with the use of the angle, the actual ROI of the texts can be retrieved. Is there anyway in the latest SDK to know if the sample is rotated or skewed ?

In my case, the text rotation angle is crucial information because my application needs to rotate the detected images to leave them at a 0-degree angle, which significantly improves readability for end users. This functionality is particularly important when images contain text that isn't horizontally oriented, as it ensures that the content is displayed in a readable format.
In previous versions, the API provided this information, and it was very useful for this purpose. I understand that there may be technical reasons behind the decision to remove it, but if it doesn't involve a high processing or performance cost, I believe removing this data is a significant mistake. Its absence complicates image processing and affects the end-user experience, so it would be ideal to continue having access to it.

@JoseMancebo Image Analysis Read API is optimized for general, non-document images. We removed global textAngle from Image Analysis OCR because it is not well-defined for non-document images. You can calculate the text angle for each returned text line from the bounding_polygon using sample code below:
polygon = line.bounding_polygon
left_center_x = (polygon[0].x + polygon[3].x) / 2
left_center_y = (polygon[0].y + polygon[3].y) / 2
right_center_x = (polygon[1].x + polygon[2].x) / 2
right_center_y = (polygon[1].y + polygon[2].y) / 2
text_angle = math.atan2(-(right_center_y - left_center_y), right_center_x - left_center_x)

If you are processing documents or images of documents we encourage you to use Document Intelligence Read API which still returns textAngle in the API response.

@JoseMancebo Image Analysis Read API is optimized for general, non-document images. We removed global textAngle from Image Analysis OCR because it is not well-defined for non-document images. You can calculate the text angle for each returned text line from the bounding_polygon using sample code below:
polygon = line.bounding_polygon
left_center_x = (polygon[0].x + polygon[3].x) / 2
left_center_y = (polygon[0].y + polygon[3].y) / 2
right_center_x = (polygon[1].x + polygon[2].x) / 2
right_center_y = (polygon[1].y + polygon[2].y) / 2
text_angle = math.atan2(-(right_center_y - left_center_y), right_center_x - left_center_x)

If you are processing documents or images of documents we encourage you to use Document Intelligence Read API which still returns textAngle in the API response.

Thank you for the info and the code example! This clarifies why the global textAngle was removed. I'll use the bounding polygon approach as suggested. Thanks also for the tip about the Document Intelligence Read API for documents.


Unable to use multiple profiles on CI

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (388c3c0 2023-04-10T00:12:13.189631296Z)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
I need to compile contracts with multiple solc versions (0.6.x and 0.8.x).
I managed to do that successfully in my local environment like this:
# foundry.toml
[profile.default]
src = 'src'
out = 'out'
libs = ['lib']
compiler_version = '0.8.19'
optimizer = false

# See more config options https://github.com/foundry-rs/foundry/tree/master/config
[profile.0_6_x]
# `src` must be different fot it to work.
# We also recommend putting all Solidity test files and scripts inside `src*/`.
src = 'src-0_6_x'
out = 'out'
# The order matters! When using `forge install` with FOUNDRY_PROFILE=0_6_x,
# it will use the first directory as the installation path.
# If the library is compatible with all versions,
# you can install it with the default profile.
libs = ['lib-0_6_x', 'lib']
optimizer = false
compiler_version = '0.6.12'

So my project root directory has a structure like this:
├── foundry.toml
├── lib
├── lib-0_6_x
├── src
└── src-0_6_x

My .gitmodules contains:
[submodule "lib/forge-std"]
	path = lib/forge-std
	url = https://github.com/foundry-rs/forge-std
[submodule "lib/dss-test"]
	path = lib/dss-test
	url = https://github.com/makerdao/dss-test
[submodule "lib-0_6_x/dss-vest"]
	path = lib-0_6_x/dss-vest
	url = https://github.com/makerdao/dss-vest.git

And I have a src-0_6_x/Imports.sol file that imports the required files:
// src-0_6_x/Imports.sol
pragma solidity =0.6.12;

import {DssVest, DssVestMintable} from "dss-vest/DssVest.sol";

When I run the following sequence of commands on my machine, it works flawlessly:
# build the 0.6.x contracts
FOUNDRY_PROFILE=0_6_x forge build
# build the 0.8.x contracts
forge build
# run the tests
forge test -vvv

I need this to run on CI through Github Actions, so I modified the sample test.yml file to look like this:
name: test

on:
  push:
    branches:
      - master
  pull_request:
    branches:
      - master

env:
  FOUNDRY_PROFILE: ci

jobs:
  check:
    name: Foundry project
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
        with:
          submodules: recursive

      - name: Install Foundry
        uses: foundry-rs/foundry-toolchain@v1
        with:
          version: nightly

      ###################################
      # Additinal step added
      ###################################

      - name: Run Forge build (solc 0.6.x)
        env:
          FOUNDRY_PROFILE: 0_6_x
        run: |
          forge --version
          export FOUNDRY_PROFILE=0_6_x
          FOUNDRY_PROFILE=0_6_x forge build --sizes
        id: build_0_6_x

      ###################################

      - name: Run Forge build
        run: |
          forge --version
          forge build --sizes
        id: build

      - name: Run Forge tests
        run: |
          forge test -vvv
        id: test

The issue is that on CI the 0_6_x build step fails:
> Error:
> Failed to resolve file: "/home/runner/work/endgame-toolkit/endgame-toolkit/lib-0_6_x/dss-vest/DssVest.sol": No such file or directory (os error 2).
> Check configured remappings..

From the file above, you can see I tried to set FOUNDRY_PROFILE in 3 different ways. I tried them individually though and the result is the same. It seems to be picking the src-0_6_x directory as the source directory, but cannot find dependencies in lib-0_6_x.

Interesting... I deleted the repository from my local machine and cloned it again.
At first, git clone --recurse-submodules didn't work properly and didn't install lib-0_6_x/dss-vest.
I ran:
git submodule update --init --recursive

And the dependency was installed.
Now if I try to run:
FOUNDRY_PROFILE=0_6_x forge build

I get the same error as CI:
Failed to resolve file: "PROJECT_ROOT/lib-0_6_x/dss-vest/DssVest.sol": No such file or directory (os error 2).
 Check configured remappings..
    --> "PROJECT_ROOT/src-0_6_x/Imports.sol"
        "dss-vest/DssVest.sol"

Apparently the path is wrong: "PROJECT_ROOT/lib-0_6_x/dss-vest/DssVest.sol" should be "PROJECT_ROOT/lib-0_6_x/dss-vest/src/DssVest.sol".

Apparently there's something wrong with the auto-remappings:
FOUNDRY_PROFILE=0_6_x forge remappings gives me:
ds-test/=lib/forge-std/lib/ds-test/src/
dss-interfaces/=lib/dss-test/lib/dss-interfaces/src/
dss-test/=lib/dss-test/src/
dss-vest/=lib-0_6_x/dss-vest/
forge-std/=lib/forge-std/src/

Notice the src/ suffix is missing from dss-vest.

@mattsse I have noticed similar issues where sometimes the src/ suffix is missing, but I'm not sure why it happens. I know the auto-remappings feature is bit complex, so maybe best to just manually fix remappings in these cases?

libs remappings are using compilers Remapping::find_many fn
    /// Attempts to autodetect all remappings given a certain root path.
    ///
    /// This will recursively scan all subdirectories of the root path, if a subdirectory contains a
    /// solidity file then this a candidate for a remapping. The name of the remapping will be the
    /// folder name.

https://github.com/foundry-rs/compilers/blob/21617d05ae9bab42a509c8c2021977d31b93809a/crates/artifacts/solc/src/remappings.rs#L176-L180


Getting started snippets issue

The getting started got me a bit confused.
I assumed the two snippets below represent two separate files (e.g. tables.dart and database.dart in my case).

URL: https://drift.simonbinder.eu/docs/getting-started/

MyDatabase is defined in both snippets as highlighted in the image above:
In this setup, the generator will complain about missing database.g.dart import directive.
Long story short, to get everything working,  I had to:

Remove the table definition from the top snippet.
Include part 'filename.dart' (part 'database.g.dart' in my case) in the bottom snippet.
Remove the part 'filename.dart' (i.e. part 'tables.dart' in my case) from the top snippet (for some reason build_runner generates everything in database.g.dart so I removed it -- unless I'm missing something).

Sorry if there was a misunderstanding on my part..
Cheers and thanks for your work (I've been using sqlite3 before and trying to test drive drift now).

Hi, and thanks for the report! I agree that the two snippets may not be too helpful if you want to split your tables and the database file into two separate files. The snippets are meant to first show how to define tables followed by how to actually write the database class, but in a single file. They're split into two snippets because there's a lot going on :) I will rephrase parts of that page to make it clearer that the two snippets belong into the same file.
That said, you are completely right about where to place part statements when using multiple files. At the moment, only the main database class needs a part file as drift generates everything into that file. If tables are defined in other files, those don't need a part declaration.
And thanks for checking out drift. If you're running into more issues, feel free to reach out!


Error(i.e. fatal error C1189: #error:  STL1001: Unexpected compiler version, expected MSVC 19.20 or newer) while building Tensorflow with MSVC 2019 on windows

System information
OS Platform : Windows 10 Pro
TensorFlow version: r1.3
Python version: Python 3.6.8 :: Anaconda, Inc.
Installed using virtualenv: conda
Bazel version : 0.26.1
MSVC 19
configuration :
Please specify the location of python. [Default is C:\Users\zen2_microsoft\Anaconda3\envs\tensorflow-1.13.1-without-mkl\python.exe]:
Found possible Python library paths:
C:\Users\zen2_microsoft\Anaconda3\envs\tensorflow-1.13.1-without-mkl\lib\site-packages
Please input the desired Python library path to use. Default is [C:\Users\zen2_microsoft\Anaconda3\envs\tensorflow-1.13.1-without-mkl\lib\site-packages]
Do you wish to build TensorFlow with XLA JIT support? [y/N]: N
No XLA JIT support will be enabled for TensorFlow.
Do you wish to build TensorFlow with ROCm support? [y/N]: N
No ROCm support will be enabled for TensorFlow.
Do you wish to build TensorFlow with CUDA support? [y/N]: N
No CUDA support will be enabled for TensorFlow.
Please specify optimization flags to use during compilation when bazel option "--config=opt" is specified [Default is /arch:AVX]:
Would you like to override eigen strong inline for some C++ compilation to reduce the compilation time? [Y/n]: N
Error Description :
C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.16.27023/bin/HostX64/x64/cl.exe /nologo /DCOMPILER_MSVC /DNOMINMAX /D_WIN32_WINNT=0x0601 /D_CRT_SECURE_NO_DEPRECATE /D_CRT_SECURE_NO_WARNINGS /bigobj /Zm500 /EHsc /wd4351 /wd4291 /wd4250 /wd4996 /I. /Ibazel-out/x64_windows-opt/bin /Iexternal/fft2d /Ibazel-out/x64_windows-opt/bin/external/fft2d /showIncludes /MD /O2 /Oy- /DNDEBUG /wd4117 -D__DATE__="redacted" -D__TIMESTAMP__="redacted" -D__TIME__="redacted" /Gy /Gw -w -DWIN32_LEAN_AND_MEAN -DNOGDI /arch:AVX /Fobazel-out/x64_windows-opt/bin/tensorflow/lite/kernels/internal/_objs/audio_utils/spectrogram.obj /c tensorflow/lite/kernels/internal/spectrogram.cc
Execution platform: @bazel_tools//platforms:host_platform
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.21.27702\include\yvals_core.h(352): fatal error C1189: #error:  STL1001: Unexpected compiler version, expected MSVC 19.20 or newer.
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 1719.835s, Critical Path: 17.80s
INFO: 18 processes: 18 local.
FAILED: Build did NOT complete successfully

I meet the problem too,do you have sloved this problem??

@ChaoyiXie
I didn't find solution . facing different issue after VS update 16.2 #31608 . Let me know in case you find something .

I ran into this problem when there were multiple versions of visual studio installed. When I  removed the extra visual studio installations (or the command line tools for 2017 or 2015 that you can install with vs 2019) it worked OK for me.
@laszlocsomor @meteorcloudy this looks like a bug in bazel VS detection, maybe?

@gunan , yes, this is a bug in Bazel, which is already fixed from Bazel 0.27.0
See https://github.com/bazelbuild/bazel/pull/7686#issuecomment-492796469

@gunan , the command looks legit.
I have VC 2019 Community (14.20.27508) installed, and my yvals_core.h looks like:
   337  #if _MSC_VER < 1920 // Coarse-grained, not inspecting _MSC_FULL_VER
   338  #error STL1001: Unexpected compiler version, expected MSVC 19.20 or newer.
   339  #endif // ^^^ old MSVC ^^^

For that particular installation, _MSC_VER is 1921.
It looks like @mayadav1 is getting a bad _MSC_VER.
@mayadav1 : Could you please run Bazel with --verbose_failures and paste the envvars here too? It will print the failing command, but also all the envvars Bazel set for it.

@mayadav1 : Could you please try something else as well?

Create a new directory, for example c:\foo
Create an empty WORKSPACE file there (if you create it with Notepad then make sure to remove the .txt extension)
Create a BUILD file (again, watch that there's no .txt extension) with this content:cc_binary(
    name = "msvc_ver",
    srcs = ["msvc_ver.cc"],
)

Create a msvc_ver.cc file with this content:#include <stdio.h>
#define _T(x) #x
#define T(x) _T(x)
int main() {
  printf("_MSC_VER = (%s)\n", T(_MSC_VER));
  return 0;
}

Open cmd.exe, then:cd c:\foo

bazel run //:msvc_ver

What's the output?

You can now use more recent bazel versions with TF, therefore, this issue is resolved.

You can now use more recent bazel versions with TF, therefore, this issue is resolved.

As if we didn't have strict version limitations of Bazel to use for every single TensorFlow version. Upgrade to Bazel 0.27.0 rules out TensorFlow 1.15 (which needs <= 0.26.1) and older.


[bug]when a jobset type workload created using Kueue is suspended

refer to: https://github.com/kubernetes-sigs/kueue/issues/2096
Maybe a patch version should be released to solve this problem?

/assign @danielvegamyhre @kannon92 @mimowo

/kind bug

Yes, I'll do the patch release today.

ok, i closed this issue


Proposed 2.3.0-rc1

High Level Overview of Change
This is a release candidate for the 2.3.0 release.
Highlights:

#5103
#5143
#5142
#5176
#5122
#5169
#4959

The base branch is release. All releases (including betas) go in release. This PR will be merged with --ff-only (not squashed or rebased, and not using the GitHub UI) to both release and develop.
Context of Change
This introduces

the Credentials amendment to support on-ledger credentials.
the MPTokensV1 amendment to support Multi-purpose tokens.
the AMMClawback  amendment to add support for clawing back tokens held in an AMM.
the fixAMMv1_2 amendment to fix some issues in AMM functionality.

Type of Change

[x] Release

API Impact

[X] Public API: New feature (new methods and/or new fields)
[X] libxrpl change (any change that may affect libxrpl or dependents of libxrpl)

@godexsoft can you check "libXRPL compatibility with Clio / Publish libXRPL" and verify if there is a Clio compatibility issue?

@godexsoft can you check "libXRPL compatibility with Clio / Publish libXRPL" and verify if there is a Clio compatibility issue?

Rerunning "Check libXRPL compatibility with Clio / Publish libXRP" succeeded

Superseded by #5183


Add support for excluding globs instead of only filenames

Can you please rebase this into one commit.

Done

Sorry, but I actually had to revert this change, because it was a breaking change and I didn't realize it. I added a new test. Could you try to get your PR working with this new test?

I'll have a look, but one question: the test says "it accepts a string" but you are passing an array? What is this test actually about? 8-)

Sorry, the current format of exclude is an array of strings

@rickharrison  The test reads like a simple string is also supported (instead of the whole array, e.g. if only one exclude is needed), though. I could PR this as well, if you like 8-)

sure go for it.

If you just mean renaming the test that is. We don't need to support a simple string. An array of strings is fine.

done


Signup/weight goals page

Description
Created a signup / 'weight goals' component with a slider for rate of weight change
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[x] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] 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
[x] There are no merge conflicts

 Weight Goals Component for Sign Up


Fixed SA1508 warning in SA1303UnitTests

Current coverage is 79.44%

Merging #1998 into master will increase coverage by +0.01% as of c919f86

@@            master   #1998   diff @@
======================================
  Files          559     559
  Stmts        35237   35237
  Branches      9751    9751
  Methods
======================================
+ Hit          27992   27993     +1
+ Partial       5694    5693     -1
  Missed        1551    1551

Review entire Coverage Diff as of c919f86

Powered by Codecov. Updated on successful CI builds.


#WIP, add local cache for the deployed openwhisk entities.

which can  help user query the deployed entities, then may help to avoid deploy contents repeatedly.

So the goal here is to serialize the deployment plan as a JSON file, then be able to restore it w/o parsing?
One concern is that someone may modify their deployments out-of-band from wskdeploy, e.g. use the CLI to delete an action.  This will invalidate the cache.  Perhaps we need a new command to validate or inspect a deployment - we could use the "get" calls in the REST API to download information about the installed actions/trigger/rules and match this to the cache/manifest.

@paulcastro , thank you for your review and constructive comments. Yes, I agree with your concern, I realized perhaps rename it to log instead of local cache maybe more appropriate. So the feature will try to log all the deployments in the past, not rewriting the deployments yaml to json.
The purpose is try to provide a function to help user log the deployments items in the past, perhaps we could use the util function added by you, and change the wskdeploy behavior to let users decide whether to redeploy the duplicated item again instead of just exit if duplication found by default.
I also agree with you we could add "get" to let users get all remote deployments from OpenWhisk REST API and "get" past deployments by local json file, so users could have a rough idea of deployment history even without a openwhisk API connection.

Close it after discussion.


fix incorrect variable when checking id

Noticed that this checks for the presence of a user id rather than a channel id. :)

Thank you!


WRONG COLOR WHEN CONVERT PARQUET TO PLY

I got wrong color between taichi viewer and PLY which converted from parquet. It got right color when opened by viewer but got strange color with PLY which is more contrast.
conda activate my_environment

cd /media/ichsan/DISK/taichi_3d_gaussian_splatting/

python3 visualizer.py --parquet_path_list OUTPUT/truck/best_scene.parquet

#########
I convert to PLY using this code. The PLY can be opened by Antimatter15 WebGL Splat viewer
conda activate my_environment

cd /media/ichsan/KERJAAN/Linux_World/taichi_3d_gaussian_splatting/

python3 parquet_to_ply.py --parquet_path OUTPUT/truck/best_scene.parquet --ply_path OUTPUT/truck/point_cloud.ply

########
Python version==3.10.10
CUDA version==12.1.105

taichi==1.7.0
taichi-3d-gaussian-splatting==0.0.1
torch==2.1.2
torchvision==0.16.2
plyfile==1.0.2

I wrote this ply conversion function. But since the taichi is using a different color parameterization, the exported ply is not consistent with the official ply. See https://github.com/wanmeihuali/taichi_3d_gaussian_splatting/issues/150 for more details.

I wrote this ply conversion function. But since the taichi-gs is using a different color parameterization, the exported ply is not consistent with the official ply. See #150 for more details.

Sorry I am not advance in coding, but the problem seems come from function to_ply instead of max(0, <SH, dir> + 0.5). I guess that because it has right color when opened with taichi-viewer.
This is originial implementation of save_ply : https://github.com/graphdeco-inria/gaussian-splatting/blob/2eee0e26d2d5fd00ec462df47752223952f6bf4e/scene/gaussian_model.py#L177
There is multiply calculation of f_dc & f_rest. But, I don't see the multiplication on this implementation. Is that cause problem? @jb-ye
def construct_list_of_attributes(self):
        l = ['x', 'y', 'z', 'nx', 'ny', 'nz']
        # All channels except the 3 DC
        for i in range(self._features_dc.shape[1]*self._features_dc.shape[2]):
            l.append('f_dc_{}'.format(i))
        for i in range(self._features_rest.shape[1]*self._features_rest.shape[2]):
            l.append('f_rest_{}'.format(i))

@ichsan2895 No. What you observe is caused by not using max(0, <SH, dir> + 0.5) in color parameterization in taichi training. The taichi-viewer is consistent as it uses the same color parameterization sigmoid(<SH, dir>) as done in taichi training. There are a few other difference regarding how tiny gaussians are treated between official GS and taichi GS. The expectation is the exported ply shouldn't be used interchangeably at the moment.

@ichsan2895 No. What you observe is caused by not using max(0, <SH, dir> + 0.5) in color parameterization in taichi training. The taichi-viewer is consistent as it uses the same color parameterization sigmoid(<SH, dir>) as done in taichi training. There are a few other difference regarding how tiny gaussians are treated between official GS and taichi GS. The expectation is the exported ply shouldn't be used interchangeably at the moment.

I see, hopefully someone can fix the problem too. Taichi3D-GSplat is a great tool. It outperform nerfstudio's gsplat implementation and rivaling with your gsplat implementation.

Well, technically speaking, both the improved performance of taichi-gs and nerfstudio-gs are contributed by me. LOL

Well, technically speaking, both the improved performance of taichi-gs and nerfstudio-gs are contributed by me. LOL

Yes, I also thanks to the maker of this repo @wanmeihuali

Thank you all! Sorry for not replying for a long time. Due to some reason, I cannot contribute to this repo anymore(majorly because of my new company's open-source policy...). My friends @yanzhoupan and @Alexma3312 are now maintaining this repo, and you can ask for their help if needed.

Thank you all! Sorry for not replying for a long time. Due to some reason, I cannot contribute to this repo anymore(majorly because of my new company's open-source policy...). My friends @yanzhoupan and @Alexma3312 are now maintaining this repo, and you can ask for their help if needed.

Glad to hear this repo is still maintaned by your friends (@yanzhoupan @Alexma3312 ) :ok:


Repo organization question: Why have an src directory at all?

Are there plans to have directories except openfermion under it in the future?

I've often wondered the same thing. I think we should get rid of it and just have openfermion at the top level. @babbush can jump in if he objects.

I too have wondered this...


The last line of a multiline comment appears as an empty range from column 0 to 0

Testing microsoft/vscode#148048
Follow the steps of microsoft/vscode-pull-request-github#3496

duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/3496


fix(ui): decouple the MappingTableView from the business logic functions

Fixes #2137

It's all about decoupling, which leads to ease of testing (and writing stories in Storybook).
The fact that the core is a singleton that can be pulled in and be used everywhere shouldn't be a shortcut to avoid wiring the components the expected way, which is by passing properties to the components.
The hierarchy is:
Atlasmap
  Views
	Layout
  	UI

Each level can depend on any component at the same level or a lower level (eg. Layout can depend on UI components, but can't depend on Views components).

Okay, as I mentioned initially, I completely get the desire to have decoupling, but it's the rules behind that decoupling that I feel we should document, basically your last statement about what the expectation is as to which level can depend on what. Maybe it's just me, but I'm not used to seeing that level of decoupling in Java projects. Not saying there shouldn't be, but just that it's not necessarily an obvious expectation.


✨Allow Video Stream widget to play any video file type.

This allows the video player to play any video source, including m3u8 live streams, mp4, gifs, etc.
Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category

One of: Bugfix or Feature (Depending on how you view it)

Overview

This change allows the video on a Video Stream widget to be any type of video file. Including gif, mp4, or even m3u8 live streams

Screenshot (if applicable)

Here is a before and after video showing the change not just working with new video types, but also does not break past functionality.

Before:
https://user-images.githubusercontent.com/39219859/232852017-28107601-2e3f-4930-a280-80e12b8fda67.mp4
After:
https://user-images.githubusercontent.com/39219859/232852084-5bb0afe1-961f-46ef-ac59-d57afc0f7314.mp4

Thanks a lot for your contribution.
We'll review this as soon as possible


chore: add dependabot config

Codecov Report
Base: 81.22% // Head: 81.22% // No change to project coverage :thumbsup:

Coverage data is based on head (b8f8709) compared to base (92c4fd4).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   81.22%   81.22%
=======================================
  Files          59       59
  Lines        2344     2344
  Branches      516      516
=======================================
  Hits         1904     1904
  Misses        435      435
  Partials        5        5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Full support for React 16.3 ref objects as target property

According to #947, there should be support for using the React 16.3 createRef api as the target element in components such as Tooltip. If this is the case, the type definitions should be updated to reflect this.
I am currently trying to use this api to use tooltips and I'm using Typescript, and if this is implemented, the type definitions do not reflect this. I'd be happy to submit a PR to fix this if necessary.

I believe your usage may not be right. Tooltip's target accepts a DOMElement which can be obtained React 16.3 "ref objects" via myref.current. You should be passing in the current reference from the object and not the object itself.
this.myRef = React.createRef();
//...

// this would not return the reference
<Tooltip target={this.myRef}>
// This is how you get the reference
<Tooltip target={this.myRef.current}>

Just as before the ref object people used functions to obtain the ref, those functions could not be used as target, but the ref obtained from those functions could be used:
getRef = ref => {
  this.myRef = ref;
}
//...

// this would be the "old way" equivalent of what you are trying to do
<Tooltip target={this.getRef}> {/* This never worked */}
// this is how the "old way" worked
<Tooltip target={this.myRef}> {/* This works */}

Note: the target must be provided, so you need to ensure you have the ref before you attempt to render Tooltip

@virgofx Cool I'll start in on it.
@TheSharpieOne I feel like this adds unnecessary boilerplate to the user of the library. It's not a hard thing to add a check in the rendering logic to allow for passing in the full Ref object. In fact I would argue that it's a better practice because it keeps it more declarative, and the intent is very clear.
This way, I can write
class MyComponent extends React.Component {
  private target = React.createRef<HTMLAnchorElement>();

  private container = React.createRef<HTMLDivElement>();

  public render() {
      return (

Somewhere in here is a tooltip.

              <UncontrolledTooltip container={this.container} target={this.target}>
                  Hello world!
              </UncontrolledTooltip>

      );
  }
}

And now I as the implementer don't have to worry about the null checks, and I can be confident that all of my references are correct, and I don't have to concern myself with the ordering of when the target is rendered, but I know that when it is rendered the tooltip should work as expected.
The alternative would be something like
class MyComponent extends React.Component {
  private target = React.createRef<HTMLAnchorElement>();

  private container = React.createRef<HTMLDivElement>();

  render() {
      return (

Somewhere in here is a tooltip.

              {
                this.container.current && this.target.current
                &&
                <UncontrolledTooltip container={this.container.current} target={this.target.current}>
                     Hello world!
                </UncontrolledTooltip>
              }

      );
  }
}

Which seems overly verbose and adds unnecessary mental overhead to the writer. I would be okay with the above if the library would handle null current values instead, but I would prefer the first way I've outlined here as it seems more idiomatic to me.


Add Subscription RP commands

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[X] Each command and parameter has a meaningful description.
[X] Each new command has a test
(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5453
This is an experimental preview for @microsoft users.

Replacing this PR with https://github.com/Azure/azure-cli-extensions/pull/49.


Update connector-x to latest commit of original repo

This updates connextor-x to the latest commit of the original repo to get the build working again.
sfu-db/connector-x has updated to to r2d2_mysql 23, the roapi fork was an update to version 22, so I think the fork is not needed anymore.
https://github.com/sfu-db/connector-x/blob/962b396857979c813486ad842e91a0c21ce55f72/connectorx/Cargo.toml#L46
https://github.com/roapi/connector-x/commit/31d2730845d4bc97f1b29a5516ed6f1c11d58594
Closes #271

Can we make a patch release for this?
We found some regression build failure with rust 1.70 build, https://github.com/Homebrew/homebrew-core/pull/132584


CMake Error at third_party/CMakeLists.txt:170 (find_package): By not providing "Findgflags.cmake" in CMAKE_MODULE_PATH this project has asked CMake to find a package configuration file provided by "gflags", but CMake did not find one.

Describe the bug
CMake fails to find gflags.
To Reproduce
Clone the repository, pull down deps with git submodule, and run CMake with or without any flags.
Expected behavior
I expected CMake to configure the project correctly.
Environment

OS: Windows 10 21H2 and up-to-date no-testing-repos Arch Linux
Compiler version: MSVC 2022 17.1.0 Build Tools or Community, with Windows SDK 10.0.19041.685 or clang 11.0.0 on same Windows or gcc 11.2.0 on same Windows or clang 13.0.1 on said Arch Linux or gcc 11.2.0 on same Arch Linux. CMake 3.22.2 in all cases.
CPU type: x86_64
cjxl/djxl version string: Ain't one cuz it didn't build but it was from libjxl commit 89875cb and gflags commit 986e8ee

Do you have a quick fix that we can do on our side to get a build out of docker? I m having the same issue

I am facing the very same problem. Currently unable to build on both Win and Arch. Both CMake and GCC are up to date.

related: https://github.com/libjxl/libjxl/pull/1234

Is this still happening, or was it fixed by #1234

Sorry for the tardy response, yes I'm still getting the same error on both targets.

Should be fixed by https://github.com/libjxl/libjxl/pull/1588, please reopen if there is still a problem!


How do I connect my mac to a Virtual Machine

You can connect to servers running in the machine via localhost, as if they were running on the host. https://docs.orbstack.dev/machines/network#running-servers
Support for connecting by IP will be added in the next feature release. See #89 for more info.


feat: verify commitments in groth16 recursion verifier

Description

This PR adds commitment verification to groth16 recursion AssertProof method. This implementation only supports at most one commitment.
TODO:

[x] Depends on https://github.com/Consensys/gnark-crypto/pull/488. Need to update go.mod after that PR merged

Type of change

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

How has this been tested?

[ ] Added test case TestBN254InBN254Commitment to test inner circuit with commitment

How has this been benchmarked?

Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[x] golangci-lint does not output errors locally
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Thanks for the contribution! I started reviewing the PR, but it takes a bit time. I'm proposing some refactors related to this PR and some which I wanted to include as part of a cleanup of Groth16 verifier (simpler init of groth16 verifier instance for example).
I'll have a longer response with specific items/suggestions soon.

I run our tests with the refactored version. Everything seems fine. I left a comment about subgroupCheck, everything else LGTM. Thanks for the review and refactoring @ivokub


Remove the extra calculation of image orientation for ImageIO coder

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: #2271
Pull Request Description
This PR is cherry-picked from #2271, which already merged in 5.x branch. This fix the extra decoding cost to check EXIF orientation.
Tested with iOS 7 device as well, now +[UIImage imageWithData:] can parse the EXIF orientation to image orientation, no need to manually check and using +[UIImage imageWithCGImage:scale:orientation:]

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2313      +/-   ##
==========================================
- Coverage   75.81%   75.48%   -0.34%
==========================================
  Files          36       36
  Lines        3751     3716      -35
==========================================
- Hits         2844     2805      -39
- Misses        907      911       +4

Impacted Files
Coverage Δ

Tests/Tests/SDWebImageDecoderTests.m
100% <ø> (ø)
:arrow_up:

SDWebImage/SDWebImageImageIOCoder.m
81.43% <ø> (+0.43%)
:arrow_up:

SDWebImage/SDWebImageDownloaderOperation.m
84.46% <0%> (-2.92%)
:arrow_down:

SDWebImage/SDWebImageManager.m
73.45% <0%> (-1.43%)
:arrow_down:

SDWebImage/SDImageCache.m
65.57% <0%> (+0.43%)
:arrow_up:

Continue to review full report at Codecov.

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

Just merged. Since iOS 7 is still works... 😴


Fix grammar/punctuation in phpdoc annotations

:+1:

Actually the grammar of that comment is totally wrong anyway.  It should be "The key to delete."  Can you update the PR?

@Crell Updated, correcting other similar ones.

This needs mirror'ing over to php-fig/fig-standards. Issue created.

Thanks @chalasr