[Event hub][Error]New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected

Package Name: @azure/event-hubs
Package Version: 5.5.1
Operating system:
[ ] nodejs

version: v14.15.4

[ ] browser

name/version:

[ ] typescript

version: 3.7.3

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Currently we run 8 instances of our application.We are regularly seeing below errors in processError method. We know the reason of it being ownershipLost  as we have the same number of messages of ownershipLost and  MessagingError at the same point of time.
MessagingError: New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:cb4ef8780002f9180003aaad60d145bb_G25_B17, SystemTracker:iot-prd01-evhns:eventhub:iot-prd01-evh~20479|iot-prd01-devices-events-configuration-service-cg, Timestamp:2021-06-22T02:07:34}\"}

We have some questions:

Are the above errors due to load balancing done by Azure Event hub sdk? If yes, is load balancing an ongoing process even if none of our services restarted i.e. is the ownership of partitions keep changing even if every application instance is running 24*7.
As per, this line
it seems load balancing is an ongoing process and the ownership keeps on changing from time to time and our logs also suggest that seeing the amount of error[Ownership] logs we get frequently. Please confirm.

Is  processInitialize method called each time the ownership is transferred from one application instance to other? As per our logs, we never received the processInitialize call after the MessagingError or OwnershipLost error log received. As per this line it should have been called.

To Reproduce
Steps to reproduce the behavior:

Run multiple instance of your application with Event hub consumer.

Expected behavior
processInitialize should be called if ownership is changed between instance of application.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@amit12cool

We are regularly seeing below errors in processError method.

Do you notice any patterns (e.g. they get called every 30 minutes? All stolen from the same consumer?) Just to confirm, are you using the BlobCheckpointStore?

Are the above errors due to load balancing done by Azure Event hub sdk?

The OwnershipLost error is expected when load balancing occurs. Load balancing is indeed an ongoing process. When using the BlobCheckpointStore with your consumer, every ~10 seconds (by default) the consumer will check to see which consumers have a claim for each partition, then run some logic to determine whether it needs to 'steal' a partition from another consumer.

If yes, is load balancing an ongoing process even if none of our services restarted i.e. is the ownership of partitions keep changing even if every application instance is running 24*7.

If a consumer doesn't reassert its claim over a partition within 60 seconds, then other consumers will assume that partition is no longer being read from and will attempt to claim it. So it is possible to see this occur if a consumer was unable to update Storage Blob within 60 seconds (this is separate from the calls to updateCheckpoint you make.) There are 3 situations I can think of that could lead to this happening:

A temporary network issue prevented the consumer from calling Storage Blob to reassert its claims.
Some processing took up a lot of time on the thread preventing the Storage Blob call to take too long to occur. (Since the calls occur every ~10 seconds, and it takes ~60 seconds to consider a claim as invalid, that would have to be a long blocking task)
One of the hosts a consumer is running on is experiencing clock skew and thus thinks that partition claims are stale.

If it's #3, then I'd expect a single consumer to be stealing partitions from other consumers periodically, so that may be something to consider if you see this as the pattern.
Aside from these 3, if a new EventHubConsumerClient was created and started reading, then as you say, you'd expect to see OwnershipLost as well.

Is processInitialize method called each time the ownership is transferred from one application instance to other?

processInitialize should be called by whichever consumer is 'stealing' the partition, and it should occur prior to OwnershipLost on the consumer being stolen from. Do you never see processInitialize called before OwnershipLost on any of the other consumers? Do you see processInitialize when you start your application? Can you tell which consumer is reading from the partition after you see an OwnershipLost?

Do you notice any patterns (e.g. they get called every 30 minutes? All stolen from the same consumer?)
I have attached screenshot of 2 instances with my observations

-> Partition 23 claim was lost on June 15,15.20
-> Each day the same pod has lost claim randomly to partition numbers
-> For each day partition claim lost is almost around the same time with in 1 minute interval
-> For each day instance is loosing claim for at most 9 partition for example June 14th

-> Somewhat same observations for this instance too.

Just to confirm, are you using the BlobCheckpointStore?
yes, we use BlobCheckpointStore

A temporary network issue prevented the consumer from calling Storage Blob to reassert its claims.
We don't see any network issue in our service as are services are functioning functioning rightly.

Some processing took up a lot of time on the thread preventing the Storage Blob call to take too long to occur. (Since the calls occur every ~10 seconds, and it takes ~60 seconds to consider a claim as invalid, that would have to be a long blocking task)
For this, as soon as the instance get an event we process it using some business logic and make some MongoDb calls for read write plus get/update twin on IoT hub and for some events we make an REST api call to other services. Each of this step may or may not run for specific event type received. So the processing time of event in our service is varying.

Question(s):- At what point in time the storage call is made from SDK to blobstorage for claim reassertion? Will the claim be lost in the scenario if I'm getting an empty array on any partition for more than 60 seconds, since as per screenshot I see randomly the claim is lost for any partition can this be the case of claim lost?

One of the hosts a consumer is running on is experiencing clock skew and thus thinks that partition claims are stale.
Since we don't get the processInitialize method I could not tell it's been claimed by same consumer.

Question:- Even if one consumer is having clock skew, until other consumer(s) don't loose the claim the clock skew consumer cannot claim. Is it correct understanding?

Do you never see processInitialize called before OwnershipLost on any of the other consumers?
processInitialize is called, I can see the logs now, somehow the filter was not working for on the logs. Changed the filter query resulted in the processInitialize logs appear on Kibana.

Do you see processInitialize when you start your application?
Yes, they are there.

Can you tell which consumer is reading from the partition after you see an OwnershipLost?
Yes. Below are the logs captured for claim lost[partition 22] and first events received on that partition on another consumer.

-> Event was received on another consumer 13f75a21-ca99-11eb-bad6-020335e365a1 and then the Ownership lost came on the consumer 05baebd9-ca99-11eb-bad6-020335e365a1 which had previously claimed partiton 22.
So in totality, the flow seems to working correctly. Now the only point to clear is the reason for frequent claim loss by consumers.
I hope I was able to answer your questions.

@chradek Any update on this?

At what point in time the storage call is made from SDK to blobstorage for claim reassertion? Will the claim be lost in the scenario if I'm getting an empty array on any partition for more than 60 seconds, since as per screenshot I see randomly the claim is lost for any partition can this be the case of claim lost?

By default, the SDK will attempt to reassert its claims every 10 seconds. This is separate from checkpointing. Even if you receive 0 events for 60 seconds, the claim reassertion will continue to happen in the background. It takes ~60 seconds of not being reclaimed before a partition is considered not claimed. Now, each partition that is claimed/reclaimed is 1 call to storage-blob, so if for some reason it took more than 60 seconds to reclaim all the partitions, then another consumer could come in and steal one away.

Even if one consumer is having clock skew, until other consumer(s) don't loose the claim the clock skew consumer cannot claim. Is it correct understanding?

The answer is a little complicated. If a consumer had clock skew such that it was falling behind the real time, then it would see the expiry time of the existing claims as being further in the future than they really are. However, if may still try to steal a partition anyway in order to balance the load. (e.g. If you have 32 partitions and 4 consumers, each one would try to get to a steady state of 8 partitions, even if all partitions are claimed)
If a consumer had clock skew such that it was ahead of the real time, then it would see the expiry time of the existing claims as being further in the past than they really are. So, it may think that the other consumers shutdown for some reason as their claims appear to be expired, and attempt to steal. In this scenario, you'd see one consumer repeatedly stealing partitions away randomly from the other consumers.
You just have 1 consumer it looks like that keeps getting stolen from, so I don't think it matches these scenarios.
At this point I can't really say why one pod keeps losing some partitions throughout the day. Can you tell when the pod steals back some partitions? (You'd see this via the processInitialize calls)
Turning on debugging may help us determine if there's an issue with claiming the partitions within a certain amount of time. You can turn that on by setting the AZURE_LOG_LEVEL environment variable to verbose. This will generate a lot of logs to stderr though.

You just have 1 consumer it looks like that keeps getting stolen from, so I don't think it matches these scenarios.
We have 8 consumers and all 8 consumers are randomly loosing claims to random partitions. I shared screenshot above from one of the consumer.

Can you tell when the pod steals back some partitions? (You'd see this via the processInitialize calls)
Below is screen shot for consumer id 05baebd9-ca99-11eb-bad6-020335e365a1 for processInitialize  calls.

My observation:-
-> This consumer lost claim for partition 22[Screen shot shared in my last reply] and at around same time this consumer claimed partition 10. Other that that on that same day this consumer didn't claimed any other partition.
-> On the next day this consumer claimed 3 partitions 23,25,27.
Another screenshot for the same consumer loosing partition claim on next day.

My observation:-
-> The consumer lost claim for partition 10 on 23rd June which it had claimed on 22nd June[As per above observations].

Turning on debugging may help us determine if there's an issue with claiming the partitions within a certain amount of time. You can turn that on by setting the AZURE_LOG_LEVEL environment variable to verbose. This will generate a lot of logs to stderr though.
Yes, I will set this soon and share the logs.

Meanwhile, Is the above info any help to find a pattern/issue?

@amit12cool
Thanks for answering my questions. Do all 8 consumers lose/steal partitions at the same time? If so, that could indicate that your pods are having a temporary issue talking to storage blob, as I'm not sure what else would explain that kind of coincidence. Otherwise, I think we need logs to figure out what's going on further.

@chradek  Thanks for your response.

Do all 8 consumers lose/steal partitions at the same time?
No, I see the pattern is like 2-3 consumers[ any consumer ] is loosing partition daily that too 2-3 or max 7 partitions. And for past 3 days, I did not see these errors too.

Otherwise, I think we need logs to figure out what's going on further.
I have added the AZURE_LOG_LEVEL env in our service. Logs are coming in too. Please let me know where can I share those logs.

@chradek Any update on this?

@amit12cool
If there's no sensitive information, you can share logs here. Otherwise (or if uncomfortable sharing logs here), please open a support ticket and we can take a look at your logs through there. If you open a support ticket, feel free to mention me (chradek) so I get notified more quickly.

@amit12cool
I wanted to summarize some of what we found/discussed from our call here.
When subscribing to events, there is a loop that periodically attempts to claim ownership of partitions. The bulk of that loop is this:
https://github.com/Azure/azure-sdk-for-js/blob/fb755e31805106a016e42da053ac596a1b0f4a72/sdk/eventhub/event-hubs/src/eventProcessor.ts#L433-L456
So there are a few non-blocking operations that could cause the loop to appear to hang if they never returned:

get list of partition ids
calculate which partitions to claim and claims them (when using the @azure/eventhubs-checkpointstore-blob package, this leads to calls to Storage Blob)
wait for processError to yield in the event an error occurs
sleep for X seconds (~10 seconds minus the time since the last iteration started)

What we were able to see in logs is that at some point in time, this event hubs loop seemed to hang. We looked at storage blob calls and didn't see any non-200 results. To be sure this isn't where we are hanging, we could look for the logs:

Attempting to claim ownership of partition

This will include the partition id being claimed. Then we should see a matching log that follows it:

Successfully claimed ownership of partition

One theory I had was that the processError handler may be hanging. If this is the case, then we should see

An error occured within the EventProcessor loop

without the following appearing after it:

Pausing the EventProcessor loop

So, we're still not sure where the hang was occurring. What we can do on our side is add timeouts to the claimOwnership calls so that there's no possibility of them hanging indefinitely. We can also add a timeout around the processError handler to ensure it can't hang the loop as well.

Hi @amit12cool I don't have a clear timeline at this time. I'll discuss with the team and report back here with an update.

We are using Azure cloud stream binder for event hubs and getting the same error however we are getting it when consumer instances are started.
One question: Does these error can cause events to get lost in any way as when we see these error there are few events missing in the output files that we are writing?

Hello! @amit12cool , could you inform us if you solved the issue? If yes, could you please, share your workaround?
Hello, @chradek , could you please tell us if the issue was fixed or something we can do to overcome the problem?
I just faced the same error using your Java SDK:
2022-05-25 12:00:32.869  WARN c.p.s.c.e.CdcEventErrorHandler   : Error occurred while consuming a message from event-hub 'user', partition '4'.

com.azure.core.amqp.exception.AmqpException: New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:83d396390001018800008c2f628d9b0a_G14_B9, SystemTracker:***************:eventhub:user~27305|user_consumer_group, Timestamp:2022-05-25T03:00:32, errorContext[NAMESPACE: ***************. ERROR CONTEXT: N/A, PATH: user/ConsumerGroups/user_consumer_group/Partitions/4, REFERENCE_ID: 4_4ad124_1653447434342, LINK_CREDIT: 500]
at com.azure.core.amqp.implementation.ExceptionUtil.toException(ExceptionUtil.java:85)
at com.azure.core.amqp.implementation.handler.LinkHandler.handleRemoteLinkClosed(LinkHandler.java:111)
at com.azure.core.amqp.implementation.handler.LinkHandler.onLinkRemoteCle(LinkHandler.java:59)
at com.azure.core.amqp.implementation.handler.ReceiveLinkHandler.onLinkRemoteClose(ReceiveLinkHandler.java:219)
at org.apache.qpid.proton.engine.BaseHandler.handle(BaseHandler.java:176)
at org.apache.qpid.proton.engine.impl.EventImpl.dispatch(EventImpl.java:108)
at org.apache.qpid.proton.reactor.impl.ReactorImpl.dispatch(ReactorImpl.java:324)
at org.apache.qpid.proton.reactor.impl.ReactorImpl.process(ReactorImpl.java:291)
at com.azure.core.amqp.implementation.ReactorExecutor.run(ReactorExecutor.java:91)
at org.springframework.cloud.sleuth.instrument.reactor.ReactorSleuth.lambda$null$6(ReactorSleuth.java:312)
at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)

2022-05-25 12:00:41.959  WARN  c.p.s.c.e.CdcEventErrorHandler   : Error occurred while consuming a message from event-hub 'store', partition '6'.

com.azure.core.amqp.exception.AmqpException: New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:dd28387a0001018800008c28628d9b09_G5_B9, Timestamp:2022-05-25T03:00:41, errorContext[NAMESPACE: ***************. ERROR CONTEXT: N/A, PATH: store/ConsumerGroups/store_consumer_group/Partitions/6, REFERENCE_ID: 6_c73820_1653447433334, LINK_CREDIT: 500]

Hi Team,
Is there any updates for this issue? I was getting this issue in latest SDK.
"@azure/event-hubs": "^5.8.0",
"@azure/eventhubs-checkpointstore-blob": "^1.0.1",

Hi @tony0918, could you please share more details about the issue you're facing and how your environment is setup? What are the recent changes performed by the load balancer right up to the occurrence of the issue?
Please see the [troubleshooting guide] (https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/eventhub/event-hubs/TROUBLESHOOTING.md#current-receiver-receiver_name-with-epoch-0-is-getting-disconnected) for more details on it.

Hi @tony0918, could you please share more details about the issue you're facing and how your environment is setup? What are the recent changes performed by the load balancer right up to the occurrence of the issue?
Please see the troubleshooting guide for more details on it.

@deyaaeldeen , I was running my Nodejs application with Azure Event SDK on my local laptop. The application is connecting to an Event Hub via connection string and save checkpoints on Storage account by SAS connection string. The Event Hub has 32 partitions but only one application was running. When application just started, no error log but after ~30 seconds, it repeated appears.

@tony0918 Thanks for the context, could you share a small repro code that I can use for testing on my end?

@chradek We got the same error when using checkpointstore SDK to consume message from EventHub. And we found that the entity which occur the same issue is an empty entity.  We didn't send any messages to this entity during the past one month, could you please help clarify if the empty entity would cause this issue? Thanks so much!

Hi @chradek , we encounter this same issue, we have one namespace with 2 empty entity(except for those 2 empty entity, we also has some entity in use). So we are afraid those 2 empty can cause claim eventhub partition timeout? And we also saw lots of timeout errors in logs. Any comments are welcome, thanks in advance!

Hi @deyaaeldeen，we encounter this same issue，and i do not make sure if I lose some log when clock skew. For more, how do I found how How many partitions are there we have. thanks

@godt21 Please follow the instructions in the [troubleshooting guide] to enable the logs. For your second question, the number of partitions in one event hub can be found either in the Azure portal or in the logs.


Add placeholders or skeletons to components

Steps To Reproduce

Start a fresh Evidence project and wait for the browser to open
Observe the browser

Expected Behavior
There is some static image or placeholder of a chart visible when the page is loading
Actual Behaviour
Some components are not visible while the page loads.
This includes the Charts and the Query Viewer, but not the DataTable, Value or BigValue

I don't think this is a bug, we just don't have skeleton states in most (all) of the components, given how fast they are already. @ItsMeBrianD has done some work on this on next.

I guess I don't understand why the DataTable load instantly but the QueryViewer does not (for example)

I think that's because we default to not showing QueryViewer, so it isn't SSR'd

That's right -- you could set it to be included by default during SSR but then it would jank off the page for most folks who have queries hidden.

Queryviewer is a bit of a special case, but SSR'ing the charts would definitely be a good improvement

ECharts SSR'd charts are currently more of an embedded thing (i.e. something you would return from an endpoint and you link to it as a .png or such) but apache/echarts#18334 could be a very interesting feature to keep an eye on


[flutter.js] Wait for reg.update, then activate sw (if not active yet).

In this PR we handle the update case of Flutter Web service worker slightly better, by awaiting on the promise returned by it, instead of using the instance upon which update was called.
Once that's done, we fetch the service worker in the state in which it might be, and pass it to the method that waits for its activation (if needed!).
Fixes #101459
Tests: couldn't test this, because AFAIK we don't have the ability to swap a Flutter Web service worker on the fly in our testing environment :/
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Oh, wait, I missed your comment:

Tests: couldn't test this, because AFAIK we don't have the ability to swap a Flutter Web service worker on the fly in our testing environment :/

What causes the service worker to change to trigger this code path? This test is attempting to do something like that:
https://github.com/flutter/flutter/blob/master/dev/bots/service_worker_test.dart

What causes the service worker to change to trigger this code path? This test is attempting to do something like that:

I think what we need to do is to have:

An app running with a given serviceWorkerVersion, so it registers that serviceWorker (v1)
Rebuild the app with a different serviceWorkerVersion (v2)
Refresh the app. That should trigger the "serviceWorkerVersion is different, I need to update the SW" branch of the code

(I think, not entirely sure)

I've added a new test to service_worker_test.dart that implements what I described above:

Load app v1
Reload app v1
Change app to v2
Load app v2

That does exercise the code path where the serviceworker IS updated, but it doesn't trigger the issue on my machine :S
Also expanded the service worker test to have 3 different initializations: without flutter js, with flutter js and all the steps, and with flutterJS "short" (calling autoStart directly).


256 block limit?

You said in #93 that you can register 512 blocks, but here:
https://github.com/andyhall/noa/blob/65e6c88ad64c05055190ba42255e2b530455f221/src/lib/registry.js#L35
it's limited to 255. Is there a reason for that or is it an oversight?
Thank you

Ah, nice catch! That line is wrong, the logic that stores voxel data uses nine bits for the voxel ID. I'll fix this next release.

Thank you!

Should be fixed in #develop, so #master will be fixed in next update.


Update hugegraph-loader.md

输入源更新为4个，增加kafka

Thanks for your contribution, could u merge the 2 PRs into one? https://github.com/apache/incubator-hugegraph-doc/pull/301/files

ok，I have resubmitted it

ok，I have resubmitted it

U could just cherry-pick it easily (no need to submit a PR request, could just reuse it)
u could use command line or GitHub desktop:


Explicit attributes should create a buffer for me.

When creating explicit attributes, this works:
    position: {
      buffer: regl.buffer([1, 2, 3, ...]),
      offset: 0,
      normalized: false,
      stride: 24,
      size: 3
    }

but this doesn't work:
    position: {
      buffer: [1, 2, 3, ...],
      offset: 0,
      normalized: false,
      stride: 24,
      size: 3
    }

but why should I not be able to do the latter? Can't it just infer that it should create a  regl.buffer from the array? Do I really have to explicitly create the buffer?
My suggestion is that both the versions should work.

Yeah, this is a bug.  I'll take a shot at fixing it.


Allow Addenda Type Code 99 for BatchPOS

What were you trying to do?
Attempting to load an ACH return file, containing entry details from an original POS file.
What did you expect to see?
I expected to see a valid ach.File with the correct contents.
What did you see?
*ach.BatchError: BatchNumber 1 Addendum *ach.Addenda99 found where Addenda02 is required for SEC code POS

How can we reproduce the problem?
// TestBatchPOSAddendum99 validates Addenda99 returns an error
func TestBatchPOSAddendum99(t *testing.T) {
	mockBatch := NewBatchPOS(mockBatchSHRHeader())
	mockBatch.AddEntry(mockPOSEntryDetail())
	mockBatch.GetEntries()[0].AddAddenda(mockAddenda99())
	if err := mockBatch.Create(); err != nil {
		t.Errorf("%T: %s", err, err)
	}
}

Will be resolved with issue 323


Running VIO with own camera and IMU data + acquiring the pose

Hello,
This questions might seem silly, but i've been banging my head against the wall trying to figure it out. Basically, I want to create a VIO stereo pipline that takes camera images + IMU data acquired in realtime, and not from a dataset, then get the 3d poses from the resulting calculation. I'm not using ROS and not intending to use it.

Do I need custom data provider in that case?
After we spin the VIO pipline with our own (synchronized) data, how do we acquire the latest 3d pose from the VIO pipline? I'm having hard time understanding the interaction between frontend and backend.

I've been straggling too. For simplicity - I also prepared a sequence with Carla simulator, where I have a full control over everything. I put the camera and IMU at the exact same location with the same orientation, yet - I can't even execute it on this synthetic sequence.

Any luck on this for either of you so far? I'm having troubles with the ROS portion producing garbage results, and am hoping to do something similar to what you're talking about. I think I will need a custom data provider, I'm just not sure how to start on getting it to loop properly each time an image/batch of imu measurements is put into the queue rather than expecting a static dataset.

Hi,
No. We stopped using it for now.

From: zippyzoo77 @.>
Sent: Wednesday, 3 July 2024 21:51
To: MIT-SPARK/Kimera-VIO @.>
Cc: Tali Brayer @.>; Comment @.>
Subject: Re: [MIT-SPARK/Kimera-VIO] Running VIO with own camera and IMU data + acquiring the pose (Issue #227)
Any luck on this for either of you so far? I'm having troubles with the ROS portion producing garbage results, and am hoping to do something similar to what you're talking about. I think I will need a custom data provider, I'm just not sure how to start on getting it to loop properly each time an image/batch of imu measurements is put into the queue rather than expecting a static dataset.
—
Reply to this email directly, view it on GitHubhttps://github.com/MIT-SPARK/Kimera-VIO/issues/227#issuecomment-2206983029, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFFJBRLL2CZJAZUWQJYZPC3ZKRBZTAVCNFSM6AAAAABDE54ZMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBWHE4DGMBSHE.
You are receiving this because you commented.Message ID: @.***>


Roll engine to 84e3b137a16d3c673e89b340fbd2b097a4b5a3ba

Picks up changes that unblock the Fuchsia roll https://github.com/flutter/engine/pull/6949.

This manually rolls the engine similar to the failed attempt by the autoroller. The Cirrus job for generating the docs was apparently successful but was marked as failed. This unblocks Fuchsia.
cc @liyuqian, @jonahwilliams


Run code without CUDA

First of all, great paper and great that you guys make your code publish available. :-)
For both I need CUDA: train_self_distill.sh and train_mutual_distill.sh. Does the repo/training also works without CUDA available?

Hey, thanks for your interest :)
Unfortunately, I didn't explicitly implement the situation when CUDA/GPU is unavailable. I imagine one needs to tweak the code a bit to make this possible.


Add a link to the diff page in the notification

Proposed MR to add a link to the diff page in the notifications.
Example of the before and after:

I've been using this for some weeks now and it's great, but I found it fustrating to click the link to the live server and scroll through to find and click the diff. This should help especially if you have lots of things being monitored.
I saw in-line comments saying this notification should be customisable (maybe I can help out with this if you're open to it) but this should help in the meantime.
All tests are passing it's just a simple body message change and passing in the UUID to n_object in the update worker.

closing in favour of implementing custom notifications


Weird off-color in terminal and editor

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.69.0
OS Version: MacOS 12.4

Steps to Reproduce:

Open VS Code

This started to happen lately, most of the themes are doing this, does not matter if 3rd party or pre-installed.

\duplicate https://github.com/microsoft/vscode/issues/152816


runners: jlink: add option to erase flash range

Introduce a new option that allows the user to erase the flash range of the selected device.
Fixes: https://github.com/zephyrproject-rtos/zephyr/issues/40201
Signed-off-by: Bartosz Bilas<EMAIL_ADDRESS>

Hi, I am a bit confused about this PR. The issue it is fixing has a discussion where the agreed-upon approach is to add sector erase. Usually I interpret that to mean that we will just erase the areas that are affected by the image, but this PR seems to be erasing manually specified ranges instead. That seems different than the approach in the issue and like more work for the user. Was a regular sector erase not practical for some reason?

It seems that your approach sounds much better than the proposed one. Is there any easy way to obtain the information about the flash range used by the image in the runner script?

@bbilas at least for Nordic SoCs and according to this case report, JLinkExe should do sector erase by default when --erase is not given: https://devzone.nordicsemi.com/f/nordic-q-a/52596/jlinkexe-equivalent-of-nrfjprog---sectorerase
I see that this patch is for an STM32 part only, though, so perhaps that does not apply.
However, the JLink runner will flash a .hex file preferentially if one exists (see code near the comment "Get the build artifact to flash" in jlink.py), so I wonder if we can fix this issue for that particular board by:

not using --erase
enabling CONFIG_BUILD_OUTPUT_HEX=y in the board defconfig

@Zzzwierzak since you were the original reporter of https://github.com/zephyrproject-rtos/zephyr/issues/40201, can you please test the above approach?


How can admins enable parallel processing for qiime2 tools

Hi @ebolyen and @colinvwood
In https://github.com/qiime2/q2galaxy/pull/47 parameters setting the number of cores have been removed from the XML (which was the right thing to do.
I'm wondering how can the number of cores now be set (by admins). Typically Galaxy tools use the GALAXY_SLOTS environment variable (e.g. here) and pass it via CLI parameter. Alternatively qiime2 tools could of course directly access the variable.

Found the docs here:
https://planemo.readthedocs.io/en/master/writing_advanced.html#developing-for-clusters-galaxy-slots-galaxy-memory-mb-and-galaxy-memory-mb-per-slot
Looks straight-forward. I think we can update action_kwargs to add $GALAXY_SLOTS as the argument to a parameter when it has the Thread primitive type.
I think we would not do anything for our Job type, as that is better handled by Galaxy running the same action over a collection.

Also, out of curiosity, is there any mechanism to submit a new job from inside a galaxy job and retain some kind of reference/future to it? @Oddant1 is refactoring some stuff with our parallel processing and there's an outside chance we could make this happen if such an API existed and server admins were amenable to the concept of it.

This sounds good to me. I think it would be good to add resource requirements to the tools. Since otherwise admins (or dynamic job rules) have no means to judge which tools support parallelism.
<requirements>
   ...
   <resource type="min_cores">X</resource>
   <resource type="max_cores">Y</resource>
</requirements>

For completeness, there are more resource types .. see here.

Also, out of curiosity, is there any mechanism to submit a new job from inside a galaxy job and retain some kind of reference/future to it? @Oddant1 is refactoring some stuff with our parallel processing and there's an outside chance we could make this happen if such an API existed and server admins were amenable to the concept of it.

There is an API that allows to execute Galaxy tools that are installed on a Galaxy instance. But doing this from inside tools seems to be a bad idea, because it will be difficult   for users to trace what has been executed. You could only run existing Galaxy tools and I think this would be much better implemented as a workflow. Also I do not know if we can assume that the Galaxy instance can be reached from the executing host.
One also needs to keep in mind the diversity of Galaxy job runners (local, SLURM, AWS, pulsar, ...). So I do not think that there can be a single mechanism and intuitively I would think that subprocesses / threads are the way to go. Might it be an option to tweak the granularity of the tools in case you need parallelism beyond a single compute node? E.g. by splitting inputs / making the jobs that are subprocesses separate tools?

Edit: For instance, you could add

<resource type="max_cores">1</resource> for tools not supporting parallelism
<resource type="min_cores">1</resource> for tools supporting parallelism (<resource type="max_cores">X</resource> if more than X cores would be inefficient).

Perfect, that should map to our Thread % Range(...) predicate, so we can populate these for all tools and provide min_cores+max_cores whenever an action indicates such on a Thread type.


main.c:66:13: warning: incompatible pointer types passing 'char *[20]' to parameter of type 'char *'

https://github.com/foxweb/pico/blob/master/main.c#L65
When I use definition char *index_html; — this pointer has undefined value and causes HTTP-response failure.
Renaming doesn't help.
When I specify the length of the string — char *index_html[20]; — HTTP-response works fine, but produces warnings at compile time:

I think changing the type is a bad way to fix this because other functions are reporting about incompatible pointer types passing.
OS: macOS Catalina 10.15.7
gcc:
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 11.0.3 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

Try this:
char index_html[20]

Also here:
  ROUTE_GET(uri) {
    char file_name[20];

With * you create an array of pointers to symbols, and every one of them is not initialized. So there are pointers to potentially random memory addresses that can be already used for any other purposes inside this process.
So basically file_name is linked to random memory and you are writing something to that space.

Thanks a lot! Fixed.


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add ^3.5.1 remove ^3.5.0 (devDependencies.ava)

Codecov Report

Merging #797 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #797   +/-   ##
=======================================
  Coverage   96.07%   96.07%
=======================================
  Files           2        2
  Lines          51       51
  Branches        2        2
=======================================
  Hits           49       49
  Misses          2        2

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 a3c3861...e17bd53. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adding a second Linear.app server invalidates API for the first one (after restart)

I filed an issue on PyCharm's issue tracker by mistake, not realizing I had this plugin installed.
Steps to reproduce:

Tools -> Tasks & Contexts -> Configure servers -> Add server -> Linear
Configure Team ID and API key
Click Test button and confirm the connection is successful
Click OK (closing the modal)
Repeat steps 1 thrugh 4
Exit PyCharm
Start PyCharm
Tools -> Tasks & Contexts -> Configure servers
Confirm that the first server now has empty API key field and the Test button is grayed out.

Yes, it is very inconvenient when working across multiple teams 🙁


minCurrent depending on # Phases

Is it possible to define different minCurrent/maxCurrent Settings depending on the amount of phases that are used (minCurrent1P/minCurrent3P)?
I do have a Zoe and it charges well with 6A/1P. However, if 3 phases are used the minCurrent is 11A. I have an Amperfied Wallbox now which can do 1P/3P but if I have minCurrent 6A it will not charge if its is using 3Phases.
Or is there any other way to force the WB to use 1P only through UI even if 3P are connected?

Is a know problem. Is not easy to solve, see https://github.com/evcc-io/evcc/issues/10330#issuecomment-1849028639


Implement the bubble sort algorithm

Pull Request Type
What kind of change does this PR introduce?
[X] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?
The bubble sort function is empty.
Issue Number: #1523
What is the new behavior?
The algorithm is now implemented and works as expected.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Sample output order inconsistency with 1.1.0 SeqLib development version

We're running in bcbio with the lighterweight SeqLib PR from @walaj (PR #338), version 1.1.0-9 and running into an issue with sample ordering consistency for multi-BAM inputs. We split up calling into regions, call identically, then concatenate the outputs together. @phu5ion identified an issue where the output sample ordering in the VCF is inconsistent for the same version and command line inputs:
https://github.com/chapmanb/bcbio-nextgen/issues/1848#issuecomment-285222225
We have a workaround using bcftools view -s to force a consistent sample order, but thought this was an issue that might cause problems for other users as well. It would be helpful if the output was consistently sorted for the same BAM inputs.

@ekg - Can you direct me to where the order of the VCFs is being formed in the code? It sounds like this may be related to some differences between BamTools vs SeqLib in perhaps the order that reads are read from a "multi" reader? Or something different in the order of concatenating the headers?
I looked through a bit and saw the sampleList structure, but thought you might be able to quickly point me where to focus, or if this is the right structure to consider for this issue. I can then see how to address with SeqLib.

@ekg Any update on this? It'd be nice to be able to pass in some list or something to force the order to be consistent. I'm probably going to implement a workaround similar to @chapmanb using bcftools view -s.


[master] Upgrade to latest dependencies

Produced via:
./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

Cron. -knative-prow-robot
/cc knative-sandbox/source-wg-leads lberk
/assign knative-sandbox/source-wg-leads lberk

/lgtm
/approve

/lgtm
/approve

@lberk can you merge this one?

@lberk can you merge this one?

/lgtm
/approve

/lgtm
/approve


Replace deprecated use of Sphinx add_stylesheet()

Fixes #512

Open/closing to retrigger tests

@lesteve I don't know if this is even still necessary, maybe you know better?

Thanks for the PR!
This was about something in the doc that was weird with sphinx_rtd_theme, I don't really know whether that's still necessary. See https://github.com/dask/dask-jobqueue/pull/301.
Maybe you can build the docs locally to see whether the problem has disappeared with the change in this PR?
We don't build the doc in the PR, we should probably do it with ReadTheDocs ...

It looks like this is no longer an issue, I built the docs with sphinx-4.1.1 having commented out the entire block (that version does throw the same error this PR was fixing). The build looks like

so I think the issue in #301 is resolved. I'll update to remove that whole section.

OK after following the doc and trying many different things it looks like ReadTheDocs is enabled in PRs now (it was enabled but something was misconfigured somehow maybe at the webhook level not sure) ...

@ocaisa I completely trust you but just in case I'll double-check the build on ReadTheDocs: https://readthedocs.org/projects/dask-jobqueue/builds/14300647/ ...

OK it looks good, thanks a lot @ocaisa, merging!

By the way, just curious, do you see too long lines too in your local build:

@lesteve Yes I see the same thing locally, lines are too long


moving ref3339 formatting and parsing classes from storage to google_cloud_cpp_common

This closes #2721

This change is

@coryan
PTAL

@coryan
PTAL


cannot find circleStrokeSpin

'circleStrokeSpin' cannot be found on the list

how to include it, please ?

You can use my fork. If you use Carthage, you can do it with: github "alickbass/NVActivityIndicatorView" "fix-build-circle-stroke-spin"

Thank you


codec: remove EditMode, split AcceptHeader in 2 separate func

Simplify the codec API by removing EditMode and splitting AcceptHeader in two funcs.

could you elaborate?


Example:
https://api.hiro.so/ordinals/v1/inscriptions/23198d6656ae23a9313eb3c2fffc8e6179e95905b92a042813feb705b6aeb1c5i0
https://ordinals.com/inscription/23198d6656ae23a9313eb3c2fffc8e6179e95905b92a042813feb705b6aeb1c5i0
A lot more ordinals like this are in this wallet: https://api.hiro.so/ordinals/v1/inscriptions?address=bc1ppmhxm8myvy55547ee4gyjv5ewzms4hrnhvpfg2z4el68as5f65vsskhg0y
It seems that the issue impacts only ordinals that haven't been transferred yet, i.e. tx_id == genesis_tx_id

hi @kuba-neoswap thanks for the report this is a known issue that we're currently resolving cc @lgalabru

Hey, just to let you know: the issue still exists.
Same example as before:
https://api.hiro.so/ordinals/v1/inscriptions/23198d6656ae23a9313eb3c2fffc8e6179e95905b92a042813feb705b6aeb1c5i0
https://ordinals.com/inscription/23198d6656ae23a9313eb3c2fffc8e6179e95905b92a042813feb705b6aeb1c5i0
A lot more ordinals like this are in this wallet: https://api.hiro.so/ordinals/v1/inscriptions?address=bc1ppmhxm8myvy55547ee4gyjv5ewzms4hrnhvpfg2z4el68as5f65vsskhg0y

The data is looking good but there could be an issue in the "presentation" layer. Looking at the parseDbInscriptions function: we're using the offset (coming from ordinal_offset, chainhook side) to the location, which is incorrect - @rafaelcr.

Noticing that this is causing issue for Hiro Wallet users. We prohibit sending non-zero sat index offsets currently, so this causes false positives.

Perhaps this will help track down the issue.
An example, freshly minted:
Inscription number 6992438
In output: d0d8ecbdb0fbd03fd90d1c0e42756f236b2165466f655f7379f1ab000eb71fc8:0
In offset somewhere in the 129 million is an impossible offset when the output only has 546 sats in it (max offset is 545).

Another example, where explorer.hiro.so displays the right offset after the ordinal is transferred (one or twice?).
Inscription number  6861317
https://ordinals.hiro.so/inscription/8c4b0980f30b1a5faec0f252d38d3f68e4a97019655367d1c16b9867ddb66617i0 This was displayed a similarly high offset (in the millions) earlier today, inconsistent with the 10k output it is in.
Then it was transferred (twice) and displays the offset correctly at 0, just as this other explorer was already doing before: https://ordinalswallet.com/inscription/8c4b0980f30b1a5faec0f252d38d3f68e4a97019655367d1c16b9867ddb66617i0

:tada: This issue has been resolved in version 0.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

This issue is now resolved! https://api.hiro.so/ordinals/v1/inscriptions/23198d6656ae23a9313eb3c2fffc8e6179e95905b92a042813feb705b6aeb1c5i0 returns the correct 0 offset.
(If you still see the old offset, though, you should clear your local browser cache)


Order of icons should match the design.

Expected behavior and actual behavior:
When all actionable icons are present, their order should be switched to match the design.
Environment
CB: redesign branch/latest pull
Browser: chrome59
Screenshots:
Design

Implemented

/cc @davidrleonard

Fixed in 6110c34e9135d770e9c7200e8e18865ed5a4d247


C++ code highlighting

Code highlighting for C++ language is not supported.
Even though It is supported for C, but doesn't highlight C++ codes

it will be fixed after https://github.com/lapce/lapce/pull/957 as we are adding new scopes requires for many languages. here is how it gonna look


Governance and organization for this repo and for the spec development process

Now that we have a shared place to work on this spec, we also need a process by which to evolve it. If we want to move forward without the official OGC SWG process, we will have to answer some basic questions about how we will operate.

How will we make decisions about how to evolve the spec? We should strive for consensus. But what if we can't reach it? Will we vote? Who gets to vote?
Who will have commit rights on this repo? I see this more as a technical question rather than a governance question. If we can answer the governance question above, then merging PRs is just a technical step--but someone will still need to do it. For the time being, I have created the @zarr-developers/geozarr group, which contains myself, @christophenoel, and @briannapagan.
How will we involve the community? There has been a lot of discussion on twitter. But we need to move this to the issue tracker and work to enumerate the specific suggestions that have been made about evolving the spec?

There are many other questions, but these are the main ones that come to mind.

For the decision making method , I recommend applying principles similar to OGC SWG. Creating a SWG (soon or later) is the final goal that initiated all these discussions.
It would also be beneficial to have a repository to store and collaborate on documents (e.g. a report on yesterday's discussion).
Finally, the involvement of xArray is essential, as this is nearly the only tool available to demonstrate the applications of Zarr in the spatial context.

I am about to send some follow up emails from last week's discussion, but want to do this as openly as possible, and it touches on some of your questions @rabernat. I am going to send a doodle poll to those who attended and others mentioned at the end of this meeting: https://hackmd.io/@MSBYE-SmSS-O706S4WXH0Q/geozarr-spec-swg-20230119. From this I will select a standing bi-weekly time and post information on how to join these meetings.
I would like to better understand how the SWGs work when it comes to voting/consensus. This can be a topic of discussion for our first biweekly meeting.

Brianna, has this doodle gone out? I didn't see it in my email.

@rabernat With so many people I just selected a day/time, you should have received an invitation last week for bi-weekly meeting Wednesdays at 11AM EST. It was sent to your Columbia email.

Information for how to join and notes from meetings will be kept here:
https://hackmd.io/@MSBYE-SmSS-O706S4WXH0Q/geozarr-spec-swg

It would be great to collect the brainstorming ideas and be able to work on that during the week.

Given what I think is consensus on pursuing an OGC standard along the "full standard" track, which I've summarized in the open PR #13 -- the outstanding issue (for this issue) is whether we will continue to work in this space or take up development of an OGC specification under the opengeospatial github organization? Perhaps we develop a SWG charter using this repository / issue tracker and move to the OGC github for SWG activities?

For your information, I'm still waiting for confirmation from Scott Simmons regarding whether our activities are suitable for a SWG.

Scott is able to join for the first 30 minutes tomorrow at our biweekly meeting. I would encourage folks to come with any open questions.

Just to share comments from Scott:

Yes, the objectives are definitely in line with OGC intent and the ZEP-4 document includes information that is suitable for an OGC Standard. BUT, OGC would still need a formal Standard document to reference that looks quite different from the ZEP-4 document.
Here is how I can see this working. The content of a ZEP would need to be described as clearly-defined requirements that are testable. The OGC template for a Standard [1] would then be populated from the ZEP text and requirement(s). The Standard could be very short - no need to write hundreds of pages if very few are needed. Finally, the enhancement (which I suppose would be extended, but optional functionality for Zarr) could also be described as is best for the Zarr community as an included Annex in the Standard so that Zarr users see what they are used to and OGC Standard readers also see what they expect.
OGC could publish the documentation in more developer-friendly ways than just the ugly Standard document itself, see OGC API - Features [2].

I suppose that adding a few more requirements to the ZEP-4 convention is acceptable.

We should develop a CONTRIBUTING.md that outlines the answers to @rabernat's OP.
I think we are going to move forward with this as an opt-in consensus (objection to unanimous consent / vote and debate) process akin to the OGC. How we identify voting members is not clear, but doesn't need to be complicated if we don't want it to be. Does the ZARR community governance offer any guidance?
We need to answer the "who has commit rights" and who is moderating question. This is a coalition of the willing / able and we should move ahead with a light weight and transparent approach.
IMHO, we should notify the ZARR/xarray community, the CF community, and the OGC community of this effort but keep development here with a target deadline to have a ZARR convention formatted as an OGC community standard.

Following the recent progress and discussions from our bi-monthly meeting, I would like to respectfully share my thoughts and insights on two key subjects that have been frequently discussed and debated within the project. I acknowledge that not everyone may share my opinions, and I appreciate the diversity of thoughts that contribute to the success of the project.
GeoZarr: Scope
First, I would like to address the focus areas for GeoZarr. It is crucial that we first establish a common understanding of our goals before delving into highly specific technical details, such as encoding new coordinate types. From our perspective, the primary objectives (which are way beyond CF conventions) include:

standardizing the encoding of geospatial data in Zarr to facilitate seamless data interpretation and processing by any client through a unified set of metadata and encoding conventions (including coordinate encoding as the purpose is that anybody can read the data in standard way).
fully support the range of functionalities offered by cloud-native formats like COG, we aim to encompass map visualization with the necessary provisions for multiple projections and resolutions, accommodate various geospatial data types such as hyperspectral, multispectral, and 4D+ ARD data, and leverage Zarr's multidimensional array format by enabling support for multiple chunking versions.
drawing from established conventions like CF but potentially others, where relevant, but overall identifying and prioritizing the most crucial properties and levels for our project.

Refining the CF conventions data model, although advantageous to numerous users, is not the primary focus of the GeoZarr project. While we recognize the potential benefits of such enhancements, they are not critical to the quality or success of GeoZarr. Furthermore, it is important to recognize that the acceptance and adoption of changes to the CF conventions may involve a more complex and extensive process across various stakeholders.
Standardization Approach
Secondly, I would like to address the working framework debate between the OGC SWG approach and an independent community-based approach (with the intention of proposing the result as an OGC Community Standard).
I suggest that developing conventions in isolation may not be advantageous, as it could lead to a less collaborative and less widely accepted outcome. This approach poses challenges, as it tends to enforce a finalized product rather than fostering deeper consensus among a more extensive group of stakeholders. Additionally, such a framework may be characterized by a lack of structure, and efficiency (and thus far, our progress has been limited)
Instead, an OGC SWG framework fosters a more structured and collaborative environment, enabling a broad range of stakeholders to provide valuable feedback and contribute to the development of a high-quality specification. Furthermore, the OGC SWG approach offers the added benefits of hardware infrastructure, organizational support, and increased visibility.

"if we want to go with SWG we have to write a charter, get it approved and only then we can start working, so could also delay things"

As we are already actively working on the project, I don't see how developing a clear charter would impede our progress; instead, it could provide a solid framework that aligns our efforts and enhances our overall efficiency.
In conclusion, we appreciate the diverse perspectives, and I hope that we can engage in a constructive dialogue to address these key topics. The initial intention of Spacebel and ESA was to support the OGC SWG framework, which promotes collaboration and a structured approach, ultimately leading to a successful and widely adopted GeoZarr standard.
Thank you for considering my thoughts and for your continued dedication to the GeoZarr project.

I can offer a second to what you are proposing, @christophenoel. If you would like USGS to be a co-convener of the SWG, please feel free to add my name to the charter and add me as a reviewer when you are ready for review. If and when the SWG kicks off, I will join as a member and take part in the process.
I would suggest modifying the README here to indicate that this decision has been taken and that activity will move to the opengeospatial venue in the future so it is clear to others that this will not be developed in the zarr-developers organization.

Make sense to close because of creation of the SWG.


CORTX-31595 : Exclude CODE_OF_CONDUCT file from Alex Scan

Exclude CODE_OF_CONDUCT file from Alex Scan
Signed-off-by: subhalaxmisa<EMAIL_ADDRESS>Problem Statement

Exclude CODE_OF_CONDUCT file from Alex Scan

Design

For Bug, Describe the fix here.
For Feature, Post the link for design

Coding
Checklist for Author

[x] Coding conventions are followed and code is consistent

Testing
Checklist for Author

[x] Unit and System Tests are added
[x] Test Cases cover Happy Path, Non-Happy Path and Scalability
[x] Testing was performed with RPM

Impact Analysis
Checklist for Author/Reviewer/GateKeeper

[x] Interface change (if any) are documented
[x] Side effects on other features (deployment/upgrade)
[x] Dependencies on other component(s)

Review Checklist
Checklist for Author

[x] JIRA number/GitHub Issue added to PR
[x] PR is self reviewed
[x] Jira and state/status is updated and JIRA is updated with PR link
[x] Check if the description is clear and explained

Documentation
Checklist for Author

[x] Changes done to WIKI / Confluence page / Quick Start Guide

Closing as an obsolete


Correction for Quick Start Guide

I installed ionic material using bower, and the quick start guide does not work.
This is my correction (adding /dist in ionic.material.min.css and ionic.material.min.js).
Step 3
<link href="lib/ionic/css/ionic.css" rel="stylesheet">
<link href="lib/ionic-material/dist/ionic.material.min.css" rel="stylesheet">
<link href="css/style.css" rel="stylesheet">

<script src="lib/ionic/js/ionic.bundle.js"></script>
<script src="lib/ionic-material/dist/ionic.material.min.js"></script>

Agreed. I needed to add the /dist/ in there as well.

Still not done, had trouble with this too

Agreed

+1


Action fails if repository is unshallow

If the workflow has already checked out an unshallow repository in an earlier step, this action will fail in the first step when calling running git fetch --prune --unshallow --tags.
I've made a small pull request to fix this issue here: https://github.com/GeoWerkstatt/create-jira-release/pull/11

hey guys, could you please merge @adrianimajion code?


fix: pending error not repackaged

Description
Fixes FL-1678
How has this been tested?

Checklist

[ ] changelog was updated with notable changes
[ ] documentation was updated

/e2e --tests sdk,substrafl,camelyon,doc,mnist

End to end tests: :hourglass_flowing_sand: STARTING


servcie:create service from yaml，error info is not friendly [REPLACEMENT ISSUE]

The original issue
Id: 1142
Title: servcie:create service from yaml，error info is not friendly

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @liyanq528 on Nov 8, 2019, 07:48
fixed by pull request #1496

In GitLab by @sokril on Nov 8, 2019, 08:20
closed

In GitLab by @sokril on Nov 8, 2019, 08:20
assigned to @cuiaq123


Updated FCU installation instructions to work with INT lab FCU

Needs to be tested with OS2 and PROD before merging

Verified for all stages.


I will update some information.

Hello Sir,
I am your subscriber of Kavyansh Pandey youtube channel.
Please accept my pull request. Please Sir..

Hi, Can you please try to re fork and re -raised PR again, because we are getting conflict here..

Wait sir mai apko aek aur pr bhejta hun, but ye wala cancel mat kar dijiyega


Suggest strings.Clone

string([]byte(s))
=>
strings.Clone

https://cs.opensource.google/go/go/+/master:src/strings/clone.go;l=21

We already have this checker.


ip fast field broken without null index

As a temp workaround 0 is used to encode "null"
Depends on https://github.com/quickwit-oss/tantivy/issues/1575

fixed with columnar


Open in desktop

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I added a line describing my change to CHANGELOG.md
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #6659
This is based on the spotify web implentation of "Open in desktop app" feature.
I added a simple landing / loading page. Feel free to change it.

Todo

[ ] Better message ?

I'm concerned about this causing a lot of confusion among Odysee users that don't know about LBRY desktop. But I think you can use /user/me to know if the user has ever installed the desktop app -- I'd be fine with showing this to those people.

I'm concerned about this causing a lot of confusion among Odysee users that don't know about LBRY desktop.
I added a landing page for this. The message is just a simple example. It could be better:


Restrict tag to:div for Blankslate

Part of #491
What
Don't allow Blankslate tag to be changed. Currently we have no restrictions but we only ever instantiate this component as a div. We already don't mention the tag attribute in the docs here so no need to change anything here.
Notes
I verified that there's no instance of Blankslate with tag argument by grepping the following regex in dotcom:
Primer::BlankslateComponent.new(((?!%>).)|\n)*(tag:)+(((?!%>).)|\n)*%>

This change looks good to me. ✨
Can we document that it is invalid to pass the tag system argument to this component?

hey @koddsson!  For a bit of context, we recently deprecated tag: from the System argument docs ( https://github.com/primer/view_components/pull/506) as the following statement wasn't true with regards to tag:
All Primer ViewComponents accept a standard set of options called system arguments,

With this update, we're trying to eliminate the notion that primer view components accept a tag unless it's explicitly documented in the Arguments section of the component page.
We're moving towards the following plan, so for now we're thinking that we don't need to mention tag for components that have locked tags like this.


Update thumbnails/avatars/etc. when screen scale changes

Maybe just dump caches?

I did some brainstorming with @Wevah and these are some things that came up.
The current cache is global and only takes into consideration the screen with keyboard focus when the app is launched.  It doesn't handle moving to another screen with higher or lower density or opening additional windows on other density screens.
To have a 1 to 1 match between the icons and the screen scale, we would need to implement 2 or 3 caches or have per window caches instead of the global one.  We would also need to refresh the icons (reload cells) to pick up the new backing scale when the screen changes and also do something like change a per window cache (if we made one).
We think the better solution is to always cache and use the icons at 2x for Macs and 3x for iOS and then render them smaller on the less dense displays.  That way there is no UI reloading happening or caches that need to be dumped and rebuilt.  This will have a small performance impact on lower density displays as they will be working with larger images than they need to.


Debian bullseye: Unable to locate package libdispatch-dev

E: Unable to locate package libdispatch-dev

This package isn't available in future versions of Debian, I guess?

Ow forgot:
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux bullseye/sid
Release:	testing
Codename:	bullseye

Hi, thanks for the info, i just tested it until debian 10, i will take a look soon and fix it!
thanks again!

the libdispatch-dev package is in the deb-multimedia repo, you can find it here: https://deb-multimedia.org/


How to reconnect rabbitmq when heartbeat timeout?

How to reconnect rabbitmq when heartbeat timeout?

Hey @ganruip. Currently we provide access to the underlying AMQP connection and channel in our connection abstraction: https://github.com/WonderPanda/nestjs-plus/blob/master/packages/rabbitmq/src/amqp/connection.ts
This allows you to attach appropriate event handlers to the connection, such as the close event where you could put in logic to attempt to reconnect in the case of a heartbeat timeout. If I have some free time I will look at providing an easier way to do this through our library directly but no promises on a timeline right now.


Make changes more visible on ClusterMode

When making updates to a YAML file on Cluster Mode it's not clear enough that there are changes made or that they even need to be deployed.
In the next screenshot there's a configuration that's been updated manually on ClusterMode, but you can't recognize that it was in fact updated

Could we add a banner of sorts showing that the changes are not applied yet and perhaps suggesting to click on Deploy?

@julioramosest Can you add this to the design for 2.0? So we can close it

@julioramosest did you do any work on this? was it implemented?

@olensmar @chargio We should be having the same global element for changes that we have already on place for Cloud:

Find design here: https://www.figma.com/file/V9FLU3sgj5oIwDhAIRNyIb/Policy?type=design&node-id=22%3A116&t=pJhsx5WDf1ZiBzQC-1


Xbox buttons

The Xbox buttons are not in the correct order. Currently the only way to get proper button controls is to use button id's from the driver station instead of the TitanUtil.

Please make sure you are using the correct Joystick class and that the button order in Titan.java are updated. Check to see if it the index isn't shifted, and if it is, let Liav know so that he can update it.
public enum Button implements ButtonZone {
            // ordered correctly, so ordinal reflects real mapping
            A, B, X, Y, BUMPER_L, BUMPER_R, BACK, START
        }

@littlekishi is right, TitanUtil somehow broke. While Java's .ordinal() returns a 0-indexed value, WPILib's .getRawButton() requires a 1-indexed value. I fixed this by simply adding a + 1 to the getRawButton() in TitanJoystick


Regualr Sync: Simpler getBestPeer and handle empty range

Current:

get genesisTime from headState
getBestPeer with headSlot >= current_slot() - 1
keep retrying empty range

Change:

get genesisTime from chain
getBestPeer without caring about its headSlot
based on the best peer's head we know when to get next range, when to retry

With this change we aren't stuck at this range anymore Regular Sync: Requesting blocks from slot 73488 to slot 73507


Some misc bug fixes

Fix three bugs:

subliminal -l en  treats the file as a language.
Crash with guessit 0.7.1.
Searching for show IDs in the addic7ed provider.

Please reopen an issue if you still have problems.


mcce version installed via conda

Hi,
I successfully installed mcce using conda as suggested here:
https://gunnerlab.github.io/Stable-MCCE/quick/
which gives me mcce version 1.1.0 (at least it tells me so when I run the command with the --version flag)
Now there is a paper from 2009 called "MCCE2: Improving protein pKa calculations with extensive
side chain rotamer sampling".
also on https://sites.google.com/site/mccewiki it says:
"Current development focuses are on the following 4 aspects: Integrating the new version
of DelPhi into MCCE 3.5..."
and on https://github.com/GunnerLab  there is a repository called:
Zoom-Link-for-meeting-MCCE4
so it looks as if there were a number of versions around (at least 1, 2, 3, and 4) and conda installation
leaves me with the seemingly oldest version There is also a repository called Develop-MCCE, but here
the last update appears to be four years old, and documentation is rather sparse.
Is there a way to obtain a newer/the most recent version?
thanks!
Michael

The most recent public version is Stable-MCCE compiled from source. The
compiling procedure is here:
https://mccewiki.levich.net/books/install-mcce/page/download-and-install-mcce
A newer MCCE version is under development with additional interfaces to
APBS and NextGenPB (the main developer is visiting Marilyn's lab this
summer).
On Tue, Jun 11, 2024 at 6:41 AM Michael Brunsteiner <
@.***> wrote:

Hi,
I successfully installed mcce using conda as suggested here:
https://gunnerlab.github.io/Stable-MCCE/quick/
which gives me mcce version 1.1.0 (at least it tells me so when I run the
command with the --version flag)
Now there is a paper from 2009 called "MCCE2: Improving protein pKa
calculations with extensive
side chain rotamer sampling".
also on https://sites.google.com/site/mccewiki it says:
"Current development focuses are on the following 4 aspects: Integrating
the new version
of DelPhi into MCCE 3.5..."
and on https://github.com/GunnerLab there is a repository called:
Zoom-Link-for-meeting-MCCE4
so it looks as if there were a number of versions around (at least 1, 2,
3, and 4) and conda installation
leaves me with the seemingly oldest version There is also a repository
called Develop-MCCE, but here
the last update appears to be four years old, and documentation is rather
sparse.
Is there a way to obtain a newer/the most recent version?
thanks!
Michael
—
Reply to this email directly, view it on GitHub
https://github.com/GunnerLab/Stable-MCCE/issues/300, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPRCWSC3BQER66NS7RS723ZG3H4FAVCNFSM6AAAAABJEB2KYWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2DMMBYGAYTINI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

thanks for the link to the compilation instructions, that looks useful!
I assume what I get compiling the source from Stable-MCCE is version 2, correct?
Is there anywhere something like "release notes" with some details about which algorithms and features are available in this current version?

Yes, same algorithm as MCCE2, with more tools for analysis and some bug
fixes. The version under development will have regular release time and
notes. We shifted focus to the new version which will add new parameters,
alternative PBE solvers, and new algorithms. At this moment I can't say the
exact beta release date, but it is reasonable to expect within the end of
this year.
On Tue, Jun 11, 2024 at 8:19 AM Michael Brunsteiner <
@.***> wrote:

thanks for the link to the compilation instructions, that looks useful!
I assume what I get compiling the source from Stable-MCCE is version 2,
correct?
Is there anywhere something like "release notes" with some details about
which algorithms and features are available in this current version?
—
Reply to this email directly, view it on GitHub
https://github.com/GunnerLab/Stable-MCCE/issues/300#issuecomment-2160619118,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPRCWV3BN4TMUCDQBNPLGLZG3TN5AVCNFSM6AAAAABJEB2KYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRQGYYTSMJRHA
.
You are receiving this because you commented.Message ID:
@.***>


unset RUSTC_WRAPPER for xargo invocation

Fixes https://github.com/rust-lang/miri/issues/1421
@bjorn3 @oli-obk do you think that is a reasonable way to solve this issue?

do you think that is a reasonable way to solve this issue?

Yes, in case of rustc both RUSTC_WRAPPER and RUSTFLAGS are ignored for the sysroot too, because you use a pre-compiled version. Also there isn't much benefit from using sccache for the sysroot, as it is cached across multiple projects anyway.

All right, I added a warning for when RUSTC_WRAPPER is set so people get some indication that their env var is being ignored. Thanks for the feedback everyone!
@bors r+

:pushpin: Commit cf7d88fd91046f1f851a3d0687188d9cd283a162 has been approved by RalfJung

:hourglass: Testing commit cf7d88fd91046f1f851a3d0687188d9cd283a162 with merge a6c28f08458e15cead0e80f3b5b7009786bce4a4...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing a6c28f08458e15cead0e80f3b5b7009786bce4a4 to master...


Add createSubscriber

Add createSubscriber function that passes down the value of for the channel as prop to the passed in based component.
In certain use cases having a createSubscriber function could be more handy than creating an extra Subscriber component for each channel.
Currently it basically returns a Subscriber component that looks almost the same as the Subscriber component. Happy for any ideas to combine the code somehow if you would like to go forward with it.

Thanks for the PR, @maicki. For now, I think we'll stick with the component-based approach instead of the HOC. HOCs introduce another component to the hierarchy and come with a bunch of baggage (like hoisting statics) that we don't need to impose on people.


Test pretrained models

Dear digits community.
Is there a possibility to test a pretrained model inside digits? I love the feature to test images on trained models, but is it possible to do the same on pretrained models? I would like to compare the performance on different trained models but also on pretrained models, do I have to fiddle around with some of the scripts?
Thank you so much,
Bernhard

Hello, have you seen https://github.com/NVIDIA/DIGITS/tree/master/examples/classification ?

Hey,
Thanks for your quick answer. Looks like I have to adapt this for object-detection models. I'll take a look into it.
Thx, Bernhard

In practice, it's difficult to use just any pre-trained model in DIGITS because when the model was trained outside of DIGITS we don't know what kind of pre-processing needs to be applied to the data (e.g. resize, crop, etc.). For this reason we have decide not to allow this in DIGITS.

Hey Greg.
In my case I want to use the pre trained BVLC-GoogLeNet model. What I already did was to replace one of the epoch models of the refinement training with it and then ran tests on it through DIGITS. Is pretty hacky but should be a valid test, isn't it? Nothing was detected anyway on the KITTI val data.
Thx 4 your help


Add ManifestEvaluator, used to filter manifests in table scans

This PR has been broken out of https://github.com/apache/iceberg-rust/pull/241 as that PR was getting too large.
It introduces ManfestEvaluator, which is used to apply the filter predicate from a table scan to a ManifestFile to see if it should be filtered out of the scan.
(Update: refactored based on changes to BoundPredicateVisitor recently merged to main)

@Fokko @liurenjie1024 @marvinlanhenke: I've rebased this on top of main now that the InclusiveProjection has been merged and it would be good to get some initial feedback.

There are intentionally a lot of todo!s in here. The aim is to get this PR merged so that more people can contribute implementations for the different visitor methods that need implementing, in subsequent PRs.

Thanks for the reviews, @marvinlanhenke and @liurenjie1024! All comments addressed and ready for re-review 😄

Let's wait to see if others have comments.

cc @Xuanwo @Fokko @marvinlanhenke PTAL

@sdd Thanks a lot. LGTM

There are intentionally a lot of todo!s in here. The aim is to get this PR merged so that more people can contribute implementations for the different visitor methods that need implementing, in subsequent PRs.

cc @Fokko I think @sdd is planning to add implementations in following smaller prs.


mod compatibility

DynamicTrees mod is incompatible with
Pam's HarvestCraft, errors occur with trees, or rather, tree textures disappear, and trees from the Pam's HarvestCraft mod are not replaced and remain normal, as from Vanilla

I run Dynamic trees with Pam's harvestcraft and the Dynamic trees add-on for harvestcraft.  It works great with none of the problems you're mentioning.  Existing vanilla style trees that have already generated don't get replaced with Dynamic versions, that's not how it works.


/request command

Acceptance Criteria

[ ] A user can call /request and the bot replies with a default response requesting the username of the requestee
[ ] A user can call /request @username and the bot sends a checkout page to the corresponding user. The requestee should already have a Rapyd e-wallet, otherwise the bot replies with a username does not have a Rapyd e-wallet yet message
[ ] Requesting is handled by the wallet mcs?
[ ] A user can call /request @username <amount> and sends a checkout page URL with the amount to the requestee. The bot replies back with a "request sent!" message


Draft: Replace simple token replacement with templating function

This PR replaces the replace-tokens Kpt function with render-template which provides full Go templating functionality. The other main change is that the function config for render-template now specifies one or more Kptfiles that the function reads and parses out the setters. This means that all key/value pairs use in interpolation need to be specified as setters, which is more in keeping with Kpt's design. Hopefully, this will mean that if Kpt addresses the need to apply setters/substitutions within large text blocks (e.g., ConfigMaps) then we can just delete this function and the setters will already be in place.

Looks good, other than the design issues we discussed (whether the templates should be simple kpt setters, or whether they should be explicitly defined in the function configuration).


Python Programming Puzzles

Adding a Dataset

Name: Python Programming Puzzles
Description: Programming challenge called programming puzzles, as an objective and comprehensive evaluation of program synthesis
Paper: https://arxiv.org/pdf/2106.05784.pdf
Data: https://github.com/microsoft/PythonProgrammingPuzzles (Scrolling through the data)
Motivation: Spans a large range of difficulty, problems, and domains. A useful resource for evaluation as we don't have a clear understanding of the abilities and skills of extremely large LMs.

Note: it's a growing dataset (contributions are welcome), so we'll need careful versioning for this dataset.
Instructions to add a new dataset can be found here.

👀 @TalSchuster

Thanks @VictorSanh!
There's also a notebook and demo available now to try out some of the puzzles


When BeforeMethod throw exception, the Test(With Retry Listener and driven by dataProvider) does not skip

TestNG Version
v7.4.0

Note: only the latest version is supported

Expected behavior
skip test When BeforeMethod throw exception
Actual behavior
When BeforeMethod throw exception, the Test(With Retry Listener and driven by dataProvider) does not skip
Is the issue reproductible on runner?

[*] Maven
[*] IntelliJ

Test case sample

Please, share the test case (as small as possible) which shows the issue

Test Scripts:
`
package testScripts.sample.dataProvider;
import listener.DefaultRetryAnalyzer;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.DataProvider;
import org.testng.annotations.Test;

public class DataProviderFailTest {
    @DataProvider(name = "arrayProvider")
    public Object[][] dataArray() {
        return new Object[][]{
              {"Scenario_pass", 1.0d},
              {"Scenario_fail", 2.0d},
              {"Scenario_skip", 3.0d}
      };
    }

@BeforeMethod()
public void preCondition(Object[] params) {
    String scenarioName = (String) params[0];
    System.out.println("preCondition scenarioName is " + scenarioName);
    if (scenarioName.equals("Scenario_skip")) {
        throw new IllegalStateException("Config failed.", new UnsupportedOperationException());
    }
}

@Test(dataProvider = "arrayProvider", retryAnalyzer = DefaultRetryAnalyzer.class)
public void testProvider(String scenarioName, double data) {
    System.out.println("scenarioName: " + scenarioName);
    System.out.println("data: " + data);
    if (scenarioName == "Scenario_fail") {
        throw new IllegalStateException("This Scenario should be failed.", new UnsupportedOperationException());
    }
}
}

`
RetryAnalyzer.java
`
package listener;
import org.testng.IRetryAnalyzer;
import org.testng.ITestResult;

public class DefaultRetryAnalyzer implements IRetryAnalyzer {
    private int retryCount = 1;
    private static final int maxRetryCount = 3; // max retry times

@Override
synchronized public boolean retry(ITestResult result) {
    if (retryCount < maxRetryCount) {
        retryCount++;
        return true;
    }
    return false;
}
}

`
TestXML   set configfailurepolicy="continue"

Run Results:

If I remove retryAnalyzer = DefaultRetryAnalyzer.class in @Test()
the result is ok.

@Li-Vincent - Here's what the documentation states in terms of configfailurepolicy from here

Whether TestNG should continue to execute the remaining tests in the suite or skip them if an @Before* method fails. Default behavior is skip.

Since your suite file is configured to continue, TestNG will ignore the @Beforex method results and continue to execute the Tests.
TestNG is working as designed here.

There is a question here. Skip will be ignored only when RetryAnalyzer.class is added, but skip will not be ignored even if  configfailurepolicy = continue when RetryAnalyzer is not used.

@krmahadevan It looks there is an undocumented side-effect on skipped status (or configfailurepolicy) when a retry analyzer is used.
I think using a retry analyzer should not have impacts on the result (except some retrials)
Thoughts?


Add 2.x version to CHANGELOG

Signed-off-by: Kunal Kotwani<EMAIL_ADDRESS>Description

Add 2.x version to changelog for tracking unreleased 2.x changes.

Issues Resolved

N/A

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Once https://github.com/opensearch-project/OpenSearch/pull/4298 has merged in, I will backport this to 2.x
That should bring everything wrt changelog verification for 2.x and main in order.


fix: connection no goal code

Summary of Changes
Fix an edge case where we need to process the state where we have a connection but not a valid goal code. In this case the user should be taken to the default Chat screen.
Related Issues
n/a
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

I also notice in some cases the navigation is using getParent and in some cases it isn't. Should be probably be consistent and verify that it works with both scan and deeplink

Codecov Report
Attention: Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 62.47%. Comparing base (d6da297) to head (50e3fd5).
Report is 3 commits behind head on main.

Files
Patch %
Lines

packages/legacy/core/App/screens/Connection.tsx
79.16%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1199      +/-   ##
==========================================
+ Coverage   62.39%   62.47%   +0.08%
==========================================
  Files         184      184
  Lines        6098     6090       -8
  Branches     1746     1739       -7
==========================================
  Hits         3805     3805
+ Misses       2272     2263       -9
- Partials       21       22       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I also notice in some cases the navigation is using getParent and in some cases it isn't. Should be probably be consistent and verify that it works with both scan and deeplink

We use getParent() when navigating to "Chat" because its on a different navigation stack. Others are not so we don't need to do that.


Enable batch/v2alpha1

Tectonic Version
1.5.7
Environment
bare metal
Other Information
Is it easily possible to integrate batch/v2alpha1 into apiserver by passing --runtime-config=batch/v2alpha1=true to apiserver daemon set or will it be overwritten by an automatic update?
I really need the CronJob feature of kubernetes.

You may add that to your manifest, so in your version (1.5.7), that flag would look as follows.
- --runtime-config=rbac.authorization.k8s.io/v1alpha1,batch/v2alpha1=true

Updates will only impact the container version and not overwrite this customization.
Do note that after doing this modification, assuming the cluster is already running, you'll need to:

Cycle your API server pods (kill each manually since daemonsets don't support rolling updates in 1.5.x).
Restart all kube-controller-manager pods. Otherwise, cron jobs will never run.

Lastly, do note that generally only beta+ features are enabled to ensure stability across auto updates. While CronJobs may cause you no problem, a massive change to it [while in alpha] could cause issues.
Let us know if this helps.

Ok thanks, I will try that out and will give feedback.

@kersten, I failed to emphasized the following in my advice above.
The suggestions I made are only meant for experimentation in your cluster(s).
While it is true that today we only alter the image field, we make no guarantee that adding in an alpha feature such as this or, more importantly, altering the API server manifest won't break in future updates if we were to change the internals of updates in Tectonic.
Hopefully CronJobs will make it to beta in 1.7 and this problem will go away.
Apologies that this wasn't clear.

@joshrosso Ok understood.
Working nearly like expected. Only these two items does not work:
failedJobsHistoryLimit: 0
successfulJobsHistoryLimit: 0

--runtime-config=rbac.authorization.k8s.io/v1alpha1,batch/v2alpha1=true

Is this a typo?  Should it be - --runtime-config=rbac.authorization.k8s.io/v2alpha1,batch/v2alpha1=true


RadioButton click event not triggered

Hi,
[I'm not sure if I should log this here or in Appium or in Selenium.  Please advise!]
I'm trying to create regression tests for my UWP app.  I've had success clicking buttons, but when I click a RadioButton the Click event is not being triggered.  It does select the item (ie the circle next to the text becomes solid), but why doesn't the click event occur?
Xaml
Download New
Test Code
AimSession = new IOSDriver(new Uri(WindowsApplicationDriverUrl), appCapabilities);
AimSession.FindElementByAccessibilityId("downloadNew").Click();

We'll take a look.

Which version of WinAppDriver are you using? And also how are you testing whether the click event is triggered or not? I tried to repro and am able to see the radio button being selected but can't tell if the click event is being triggered.

Hi alakshmi1030!
I have version 0.4.  I'm testing against my app (trying to write a test harness for it).  When I click the radio button with the cursor manually the app reacts and proceeds to the next page.  When I "click" through my test app (which works great for other controls) the radio button shows as selected, but the event never triggered.
I can quickly create a tiny UWP app that would show this.  If I do, are you able to Sideload it onto a Win10 device?  Let me know if that would help.  I would love to use this product, but this issue is definitely a showstopper for me!
Looks like my xaml didn't make it into the original post so I'll try posting again.  It is:
           <StackPanel x:Name="selectActionPanel" Orientation="Vertical" Padding="20,0,0,80">                 <TextBlock Style ="{ThemeResource NewTextBlockStyle}" Margin="0,4" FontWeight="Bold">I want to...</TextBlock>                 <RadioButton x:Name="downloadNew" Style="{ThemeResource NewRadioButtonSize}"  Click="actionRadio_Click" >download a new inspection to work on</RadioButton>             </StackPanel>

^Sorry, didn't mean to close this.

Hi @ddelapasse2
Yes, if you can share a repro project, it will be easier for us to investigate.

Hey @ddelapasse2 - could you try modifying the Click property in the Radio button to "Checked" See here for more info on the Checked property for RadioButtons. Let us know if that works for you.

@alakshmi1030 , Yes!  I will try and let you know.

I added a Checked handler, but the code in the test harness stayed as:
AimSession.FindElementByAccessibilityId("overwriteCheckbox").Click();
Is that correct?  No change in behavior.  The screen didn't receive anything.
I've been away working on another project, but hope to return to this one later this week.  As soon as I do I'll make you a little test UWP app with a test harness so you can try it.

I recreated a small sample and the click is working there.  My app has several nested controls so I wonder if that is throwing it off?  I will debug further and let you know!

Hey @ddelapasse2 - are you still experiencing this issue? Please let us know.

OnCheck is working for me, so that is enough.  Thanks!

Same issue i am also facing...how i can check checkBox present inside Stack panel?

<ListBox.ItemTemplate>

</ListBox.ItemTemplate>


Update README.md

Clarifies that Grunt is used to obfuscate and minify the files regardless of build tool selected.

Thank you for pointing that out.


uninitialized uvector returns garbage size()

This is the size computation in the uvector static_cast<size_type>(p_end - p_begin), however, the pointers are not initialized with nullptr in the default initializer, which results in garbage value for size().
I think it should return 0?

Hmm, it's actually quite weird. I debugged a bit, and the constructors look correct. I added print statements in xcontainer.hpp size() function to figure out why the compute_size(shape()) fix fails and that's when I came across this.
Not sure what the cause is.

I'm not sure what exactly was happening ... but it's gone :)


Displaying metrics onto the sink

Hi,
I am trying to run the https://github.com/groupon/spark-metrics/blob/master/src/main/scala/org/apache/spark/groupon/metrics/example/MetricsExampleApp.scala locally with a metrics.properties file mapped with a ConsoleSink.  I see the default metrics printed on the console but not the "MapTimer" timer .  Is my expectation that custom metrics should show up  on sink reasonable or should we explicitly parse the metrics on the driver program post job completion ?
Sorry to raise this as an issue as I am not sure if there is a google group to post this.
Appreciate any help!

Hi @mravi, thanks for bringing this up. What you're doing should work - custom metrics should be reported automatically, as this is already happening at the metrics/json HTTP endpoint on the driver. That endpoint is handled by the MetricsServlet class, which implements the Sink trait like the ConsoleSink does.
Could you paste the metrics.properties file you're using? I was able to have the metrics printed to the console by setting the metrics config in the driver code, like this:
import org.apache.spark.groupon.metrics.UserMetricsSystem
import org.apache.spark.{SparkContext, SparkConf}

import scala.math.random

/**
 * Spark's SparkPi example instrumented with metrics
 */
object MetricsExampleApp {
  def main(args: Array[String]) {
    val conf = new SparkConf().setMaster("local[2]").setAppName("Spark Pi")

    // Setting up ConsoleSink here
    conf.set("spark.metrics.conf.*.sink.console.class", "org.apache.spark.metrics.sink.ConsoleSink")

    val spark = new SparkContext(conf)
    UserMetricsSystem.initialize(spark, "SparkPiMetricNamespace")

    val slices = if (args.length > 0) args(0).toInt else 2
    val n = math.min(100000L * slices, Int.MaxValue).toInt // avoid overflow

    val count = spark.parallelize(1 until n, slices).map( i => {
      UserMetricsSystem.timer("MapTimer").time({
        val x = random * 2 - 1
        val y = random * 2 - 1
        UserMetricsSystem.gauge("x").set(x)
        if (x * x + y * y < 1) 1 else 0
      })
    }).reduce(_ + _)

    println("Pi is roughly " + 4.0 * count / n)
    spark.stop()
  }
}

And the ConsoleSink output for the above:
-- Gauges ----------------------------------------------------------------------
DAGScheduler.job.activeJobs
             value = 0
DAGScheduler.job.allJobs
             value = 1
DAGScheduler.stage.failedStages
             value = 0
DAGScheduler.stage.runningStages
             value = 0
DAGScheduler.stage.waitingStages
             value = 0
local-1477419564926.driver.BlockManager.disk.diskSpaceUsed_MB
             value = 0
local-1477419564926.driver.BlockManager.memory.maxMem_MB
             value = 1966
local-1477419564926.driver.BlockManager.memory.memUsed_MB
             value = 0
local-1477419564926.driver.BlockManager.memory.remainingMem_MB
             value = 1966
local-1477419564926.driver.Spark Pi.SparkPiMetricNamespace.x
             value = -0.9501188929455202

-- Timers ----------------------------------------------------------------------
DAGScheduler.messageProcessingTime
             count = 6
         mean rate = 4.54 calls/second
     1-minute rate = 0.00 calls/second
     5-minute rate = 0.00 calls/second
    15-minute rate = 0.00 calls/second
               min = 0.05 milliseconds
               max = 120.64 milliseconds
              mean = 34.74 milliseconds
            stddev = 44.55 milliseconds
            median = 20.41 milliseconds
              75% <= 64.83 milliseconds
              95% <= 120.64 milliseconds
              98% <= 120.64 milliseconds
              99% <= 120.64 milliseconds
            99.9% <= 120.64 milliseconds
local-1477419564926.driver.Spark Pi.SparkPiMetricNamespace.MapTimer
             count = 199999
         mean rate = 289914.03 calls/second
     1-minute rate = 0.00 calls/second
     5-minute rate = 0.00 calls/second
    15-minute rate = 0.00 calls/second
               min = 0.00 milliseconds
               max = 0.02 milliseconds
              mean = 0.00 milliseconds
            stddev = 0.00 milliseconds
            median = 0.00 milliseconds
              75% <= 0.00 milliseconds
              95% <= 0.00 milliseconds
              98% <= 0.00 milliseconds
              99% <= 0.00 milliseconds
            99.9% <= 0.01 milliseconds

Thanks @akakitani ! Sorry, my bad. I see it.
Thanks again for coming up with a wonderful library!