chore: shutdown when we see non retryable udf errors

Client side
Sink
2024/11/05 06:26:32 ServerInfo: &{uds go 1.3.1-z v0.8.1 map[]}
{"level":"info","ts":"2024-11-05T06:26:32.140803178Z","logger":"numaflow.Sink-processor","caller":"jetstream/kv_store.go:264","msg":"Successfully created watcher","pipeline":"simple-pipeline","vertex":"out","kvName":"default-simple-pipeline-cat-out_PROCESSORS","watcher":"default-simple-pipeline-cat-out_PROCESSORS"}
{"level":"info","ts":"2024-11-05T06:26:32.140827511Z","logger":"numaflow.Sink-processor","caller":"jetstream/kv_store.go:199","msg":"Watcher initialization and subscription got nil value","pipeline":"simple-pipeline","vertex":"out","kvName":"default-simple-pipeline-cat-out_PROCESSORS"}
{"level":"info","ts":"2024-11-05T06:26:32.140840261Z","logger":"numaflow.Sink-processor","caller":"fetch/processor_manager.go:212","msg":"Successfully added a new fromProcessor","pipeline":"simple-pipeline","vertex":"out","fromProcessor":"simple-pipeline-cat-0"}
{"level":"info","ts":"2024-11-05T06:26:32.14197322Z","logger":"numaflow.Sink-processor","caller":"metrics/metrics_server.go:236","msg":"Generating self-signed certificate","pipeline":"simple-pipeline","vertex":"out"}
{"level":"info","ts":"2024-11-05T06:26:32.14228647Z","logger":"numaflow.Sink-processor","caller":"metrics/metrics_server.go:276","msg":"Not enabling pprof debug endpoints","pipeline":"simple-pipeline","vertex":"out"}
{"level":"info","ts":"2024-11-05T06:26:32.142302886Z","logger":"numaflow.Sink-processor","caller":"metrics/metrics_server.go:289","msg":"Starting metrics HTTPS server","pipeline":"simple-pipeline","vertex":"out"}
{"level":"info","ts":"2024-11-05T06:26:32.142405845Z","logger":"numaflow.Sink-processor","caller":"sinks/sink.go:257","msg":"Start processing sink messages ","pipeline":"simple-pipeline","vertex":"out","isbsvc":"jetstream","fromPartition ":"default-simple-pipeline-out-0"}
{"level":"info","ts":"2024-11-05T06:26:32.142431345Z","logger":"numaflow.Sink-processor","caller":"forward/forward.go:120","msg":"Starting sink forwarder...","pipeline":"simple-pipeline","vertex":"out"}
{"level":"error","ts":"2024-11-05T06:26:32.147017303Z","logger":"numaflow.Sink-processor","caller":"forward/forward.go:271","msg":"failed to write to sink","pipeline":"simple-pipeline","vertex":"out","error":"gRPC client.SinkFn failed, failed to receive sink response: rpc error: code = Unknown desc = panic inside sink handler: Sink is not valid","stacktrace":"github.com/numaproj/numaflow/pkg/sinks/forward.(*DataForward).forwardAChunk\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/sinks/forward/forward.go:271\ngithub.com/numaproj/numaflow/pkg/sinks/forward.(*DataForward).Start.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/sinks/forward/forward.go:140"}
{"level":"error","ts":"2024-11-05T06:26:32.148659386Z","logger":"numaflow.Sink-processor","caller":"forward/forward.go:141","msg":"Failed to forward a chunk","pipeline":"simple-pipeline","vertex":"out","error":"gRPC client.SinkFn failed, failed to receive sink response: rpc error: code = Unknown desc = panic inside sink handler: Sink is not valid","stacktrace":"github.com/numaproj/numaflow/pkg/sinks/forward.(*DataForward).Start.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/sinks/forward/forward.go:141"}
{"level":"info","ts":"2024-11-05T06:26:32.148691011Z","logger":"numaflow.Sink-processor","caller":"forward/forward.go:154","msg":"Closed buffer reader","pipeline":"simple-pipeline","vertex":"out","bufferFrom":"default-simple-pipeline-out-0"}
{"level":"info","ts":"2024-11-05T06:26:32.148695345Z","logger":"numaflow.Sink-processor","caller":"forward/forward.go:161","msg":"Closed sink writer","pipeline":"simple-pipeline","vertex":"out","sink":"out"}
{"level":"error","ts":"2024-11-05T06:26:32.148701928Z","logger":"numaflow.Sink-processor","caller":"sinks/sink.go:269","msg":"Sink forwarder stopped with error","pipeline":"simple-pipeline","vertex":"out","fromPartition":"default-simple-pipeline-out-0","error":"gRPC client.SinkFn failed, failed to receive sink response: rpc error: code = Unknown desc = panic inside sink handler: Sink is not valid","stacktrace":"github.com/numaproj/numaflow/pkg/sinks.(*SinkProcessor).Start.func3\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/sinks/sink.go:269"}
{"level":"info","ts":"2024-11-05T06:26:32.148717803Z","logger":"numaflow.Sink-processor","caller":"sinks/sink.go:306","msg":"Exited...","pipeline":"simple-pipeline","vertex":"out"}

Map
{"level":"info","ts":"2024-11-05T07:07:29.203806724Z","logger":"numaflow.MapUDF-processor","caller":"udf/map_udf.go:293","msg":"Start processing udf messages","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","isbsvc":"jetstream","from":"default-simple-pipeline-cat-0","to":["default-simple-pipeline-out-0"]}
{"level":"info","ts":"2024-11-05T07:07:29.203829099Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:121","msg":"Starting forwarder...","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf"}
{"level":"info","ts":"2024-11-05T07:07:29.204130808Z","logger":"numaflow.MapUDF-processor","caller":"metrics/metrics_server.go:276","msg":"Not enabling pprof debug endpoints","pipeline":"simple-pipeline","vertex":"cat"}
{"level":"info","ts":"2024-11-05T07:07:29.204152058Z","logger":"numaflow.MapUDF-processor","caller":"metrics/metrics_server.go:289","msg":"Starting metrics HTTPS server","pipeline":"simple-pipeline","vertex":"cat"}
2024/11/05 07:07:29 failed c.grpcClt.MapFn stream.Recv: NonRetryable: error processing requests: rpc error: code = Internal desc = panic inside map handler: Invalid map operation
{"level":"error","ts":"2024-11-05T07:07:29.214500974Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:629","msg":"mapUDF.Apply error","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","error":"gRPC client.MapFn failed, NonRetryable: error processing requests: rpc error: code = Internal desc = panic inside map handler: Invalid map operation","stacktrace":"github.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).applyUDF\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:629\ngithub.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).forwardAChunk\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:273\ngithub.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).Start.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:141"}
{"level":"error","ts":"2024-11-05T07:07:29.214536683Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:275","msg":"failed to applyUDF","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","error":"gRPC client.MapFn failed, NonRetryable: error processing requests: rpc error: code = Internal desc = panic inside map handler: Invalid map operation","stacktrace":"github.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).forwardAChunk\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:275\ngithub.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).Start.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:141"}
{"level":"error","ts":"2024-11-05T07:07:29.215329183Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:142","msg":"Failed to forward a chunk","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","error":"gRPC client.MapFn failed, NonRetryable: error processing requests: rpc error: code = Internal desc = panic inside map handler: Invalid map operation","stacktrace":"github.com/numaproj/numaflow/pkg/udf/forward.(*InterStepDataForward).Start.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/forward/forward.go:142"}
{"level":"info","ts":"2024-11-05T07:07:29.215351474Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:155","msg":"Closed buffer reader","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","bufferFrom":"default-simple-pipeline-cat-0"}
{"level":"info","ts":"2024-11-05T07:07:29.215356683Z","logger":"numaflow.MapUDF-processor","caller":"forward/forward.go:162","msg":"Closed partition writer","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","bufferTo":"default-simple-pipeline-out-0"}
{"level":"error","ts":"2024-11-05T07:07:29.215352849Z","logger":"numaflow.MapUDF-processor","caller":"udf/map_udf.go:305","msg":"Map forwarder stopped with error","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf","fromPartition":"default-simple-pipeline-cat-0","error":"gRPC client.MapFn failed, NonRetryable: error processing requests: rpc error: code = Internal desc = panic inside map handler: Invalid map operation","stacktrace":"github.com/numaproj/numaflow/pkg/udf.(*MapUDFProcessor).Start.func2\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/udf/map_udf.go:305"}
{"level":"info","ts":"2024-11-05T07:07:29.215479183Z","logger":"numaflow.MapUDF-processor","caller":"jetstream/kv_store.go:166","msg":"stopping WatchAll","pipeline":"simple-pipeline","vertex":"cat","kvName":"default-simple-pipeline-in-cat_OT","watcher":"default-simple-pipeline-in-cat_OT"}
{"level":"info","ts":"2024-11-05T07:07:29.215466558Z","logger":"numaflow.MapUDF-processor","caller":"publish/publisher.go:286","msg":"Closing watermark publisher","pipeline":"simple-pipeline","vertex":"cat","entityID":"simple-pipeline-cat-0","otStore":"default-simple-pipeline-cat-out_OT","hbStore":"default-simple-pipeline-cat-out_PROCESSORS"}
{"level":"info","ts":"2024-11-05T07:07:29.215537099Z","logger":"numaflow.MapUDF-processor","caller":"jetstream/kv_store.go:166","msg":"stopping WatchAll","pipeline":"simple-pipeline","vertex":"cat","kvName":"default-simple-pipeline-in-cat_PROCESSORS","watcher":"default-simple-pipeline-in-cat_PROCESSORS"}
{"level":"error","ts":"2024-11-05T07:07:29.215631683Z","logger":"numaflow.MapUDF-processor","caller":"jetstream/kv_store.go:170","msg":"Failed to stop","pipeline":"simple-pipeline","vertex":"cat","kvName":"default-simple-pipeline-in-cat_PROCESSORS","watcher":"default-simple-pipeline-in-cat_PROCESSORS","error":"nats: invalid subscription","stacktrace":"github.com/numaproj/numaflow/pkg/shared/kvs/jetstream.(*jetStreamStore).Watch.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/shared/kvs/jetstream/kv_store.go:170"}
{"level":"info","ts":"2024-11-05T07:07:29.216648849Z","logger":"numaflow.MapUDF-processor","caller":"jetstream/kv_store.go:172","msg":"WatchAll successfully stopped","pipeline":"simple-pipeline","vertex":"cat","kvName":"default-simple-pipeline-in-cat_OT","watcher":"default-simple-pipeline-in-cat_OT"}
{"level":"info","ts":"2024-11-05T07:07:29.218063308Z","logger":"numaflow.MapUDF-processor","caller":"udf/map_udf.go:363","msg":"All udf data processors exited...","pipeline":"simple-pipeline","vertex":"cat","protocol":"uds-grpc-map-udf"}

Source
{"level":"info","ts":"2024-11-05T15:08:54.948115542Z","logger":"numaflow.Source-processor","caller":"jetstream/kv_store.go:199","msg":"Watcher initialization and subscription got nil value","pipeline":"simple-pipeline","vertex":"in","kvName":"default-simple-pipeline-in_SOURCE_PROCESSORS"}
2024/11/05 15:08:54 ServerInfo: &{uds go 1.3.1-z v0.8.1 map[]}
{"level":"info","ts":"2024-11-05T15:08:54.951542376Z","logger":"numaflow.Source-processor","caller":"metrics/metrics_server.go:236","msg":"Generating self-signed certificate","pipeline":"simple-pipeline","vertex":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.951920501Z","logger":"numaflow.Source-processor","caller":"metrics/metrics_server.go:276","msg":"Not enabling pprof debug endpoints","pipeline":"simple-pipeline","vertex":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.951939667Z","logger":"numaflow.Source-processor","caller":"sources/source.go:305","msg":"Start processing source messages","pipeline":"simple-pipeline","vertex":"in","isbs":"jetstream","to":["default-simple-pipeline-cat-0"]}
{"level":"info","ts":"2024-11-05T15:08:54.951967876Z","logger":"numaflow.Source-processor","caller":"metrics/metrics_server.go:289","msg":"Starting metrics HTTPS server","pipeline":"simple-pipeline","vertex":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.951977917Z","logger":"numaflow","caller":"forward/data_forward.go:129","msg":"Starting forwarder..."}
{"level":"warn","ts":"2024-11-05T15:08:54.952330001Z","logger":"numaflow","caller":"forward/data_forward.go:203","msg":"failed to read from source","error":"failed to read messages from udsource: failed to receive read response: rpc error: code = Unknown desc = panic inside source handler: Source not valid"}
{"level":"info","ts":"2024-11-05T15:08:54.952369376Z","logger":"numaflow.Source-processor","caller":"udsource/user_defined_source.go:120","msg":"Shutting down user-defined source...","pipeline":"simple-pipeline","vertex":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.952395292Z","logger":"numaflow","caller":"forward/data_forward.go:162","msg":"Closed source reader","sourceFrom":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.952404959Z","logger":"numaflow","caller":"forward/data_forward.go:172","msg":"Closed partition writer","bufferTo":"default-simple-pipeline-cat-0"}
{"level":"error","ts":"2024-11-05T15:08:54.952411709Z","logger":"numaflow.Source-processor","caller":"sources/source.go:317","msg":"Source forwarder stopped with error","pipeline":"simple-pipeline","vertex":"in","error":"failed to read messages from udsource: failed to receive read response: rpc error: code = Unknown desc = panic inside source handler: Source not valid","stacktrace":"github.com/numaproj/numaflow/pkg/sources.(*SourceProcessor).Start\n\t/Users/yhl01/Documents/numaproj/numaflow/pkg/sources/source.go:317\ngithub.com/numaproj/numaflow/cmd/commands.NewProcessorCommand.func1\n\t/Users/yhl01/Documents/numaproj/numaflow/cmd/commands/processor.go:86\ngithub.com/spf13/cobra.(*Command).execute\n\t/Users/yhl01/go/pkg/mod/github.com/spf13/cobra@v1.8.1/command.go:985\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/Users/yhl01/go/pkg/mod/github.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\t/Users/yhl01/go/pkg/mod/github.com/spf13/cobra@v1.8.1/command.go:1041\ngithub.com/numaproj/numaflow/cmd/commands.Execute\n\t/Users/yhl01/Documents/numaproj/numaflow/cmd/commands/root.go:32\nmain.main\n\t/Users/yhl01/Documents/numaproj/numaflow/cmd/main.go:24\nruntime.main\n\t/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/proc.go:272"}
{"level":"info","ts":"2024-11-05T15:08:54.952440667Z","logger":"numaflow.Source-processor","caller":"sources/source.go:335","msg":"Exited...","pipeline":"simple-pipeline","vertex":"in"}
{"level":"info","ts":"2024-11-05T15:08:54.952462001Z","logger":"numaflow.Source-processor","caller":"metrics/metrics_server.go:293","msg":"Metrics server shutdown","pipeline":"simple-pipeline","vertex":"in"}
{"level":"warn","ts":"2024-11-05T15:08:54.952471292Z","logger":"numaflow.Source-processor","caller":"sources/source.go:221","msg":"Failed to close gRPC client conn","pipeline":"simple-pipeline","vertex":"in","error":"rpc error: code = Canceled desc = grpc: the client connection is closing"}

Server side
Sink
User Defined Sink: {"Data":{"value":1730780477445230311},"Createdts":1730780477445230311}
2024/11/05 06:26:51 panic inside sink handler: Sink is not valid goroutine 35 [running]:
runtime/debug.Stack()
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/debug/stack.go:26 +0x64
github.com/numaproj/numaflow-go/pkg/sinker.(*Service).processData.func1()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sinker/service.go:181 +0x40
panic({0x419040?, 0x5846d8?})
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/panic.go:785 +0x124
main.(*logSink).Sink(0x0?, {0x0?, 0x0?}, 0x0?)
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sinker/examples/log/main.go:21 +0xec
github.com/numaproj/numaflow-go/pkg/sinker.(*Service).processData(0x0?, {0x58a980, 0x40001a2550}, {0x58da18, 0x40001920e0}, 0x0?)
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sinker/service.go:185 +0x74
github.com/numaproj/numaflow-go/pkg/sinker.(*Service).SinkFn.func2()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sinker/service.go:93 +0x34
golang.org/x/sync/errgroup.(*Group).Go.func1()
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:78 +0x54
created by golang.org/x/sync/errgroup.(*Group).Go in goroutine 33
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:75 +0x98
2024/11/05 06:26:51 Stopping the SinkFn with err, panic inside sink handler: Sink is not valid
2024/11/05 06:26:51 shutdown signal received
2024/11/05 06:26:51 gracefully stopping grpc server
2024/11/05 06:26:51 grpc server stopped

Map
2024/11/05 07:08:49 panic inside map handler: Invalid map operation goroutine 129 [running]:
runtime/debug.Stack()
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/debug/stack.go:26 +0x64
github.com/numaproj/numaflow-go/pkg/mapper.(*Service).handleRequest.func1()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/mapper/service.go:137 +0x40
panic({0x418ea0?, 0x5844b0?})
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/panic.go:785 +0x124
main.(*Forward).Map(0x0?, {0x0?, 0x0?}, {0x0?, 0x0?, 0x0?}, {0x58a858, 0x40002f2960})
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/mapper/examples/forward_message/main.go:21 +0x50
github.com/numaproj/numaflow-go/pkg/mapper.(*Service).handleRequest(0x40000ac750, {0x58a740, 0x400007cb90}, 0x40002f24b0, 0x4000228770)
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/mapper/service.go:144 +0x1fc
github.com/numaproj/numaflow-go/pkg/mapper.(*Service).MapFn.func2()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/mapper/service.go:94 +0x30
golang.org/x/sync/errgroup.(*Group).Go.func1()
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:78 +0x54
created by golang.org/x/sync/errgroup.(*Group).Go in goroutine 9
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:75 +0x98
2024/11/05 07:08:49 Stopping the MapFn with err, rpc error: code = Internal desc = panic inside map handler: Invalid map operation
2024/11/05 07:08:49 shutdown signal received
2024/11/05 07:08:49 gracefully stopping grpc server
2024/11/05 07:08:49 grpc server stopped

Source
2024/11/05 15:09:26 panic inside source handler: Source not valid goroutine 11 [running]:
runtime/debug.Stack()
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/debug/stack.go:26 +0x64
github.com/numaproj/numaflow-go/pkg/sourcer.(*Service).receiveReadRequests.func1.1()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sourcer/service.go:122 +0x48
panic({0x4194c0?, 0x5874f0?})
	/Users/yhl01/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.23.1.darwin-arm64/src/runtime/panic.go:785 +0x124
simple_source/impl.(*SimpleSource).Read(0x0?, {0x0?, 0x0?}, {0x58b698?, 0x400009b0f0?}, 0x0?)
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sourcer/examples/simple_source/impl/simple_source.go:48 +0x6c
github.com/numaproj/numaflow-go/pkg/sourcer.(*Service).receiveReadRequests.func1()
	/Users/yhl01/Documents/numaproj/numaflow-go/pkg/sourcer/service.go:132 +0xf4
golang.org/x/sync/errgroup.(*Group).Go.func1()
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:78 +0x54
created by golang.org/x/sync/errgroup.(*Group).Go in goroutine 9
	/Users/yhl01/go/pkg/mod/golang.org/x/sync@v0.8.0/errgroup/errgroup.go:75 +0x98
2024/11/05 15:09:26 error processing requests: panic inside source handler: Source not valid
2024/11/05 15:09:26 shutdown signal received
2024/11/05 15:09:26 gracefully stopping grpc server
2024/11/05 15:09:26 error receiving from ack stream: context canceled
2024/11/05 15:09:26 grpc server stopped

https://github.com/numaproj/numaflow-go/pull/165 should be merged first.

I reran the entire suite again and it is failing with
Watching POD: flatmap-stream-java-udsink-0-bu3ey
    map_test.go:117: Expected vertex ["java-udsink"] pod log to contain ["hello"] but didn't.
    panic.go:629: Deleting Pipeline flatmap-stream

I am not sure why the previous run was successful, @KeranYang do you think the java SDK PR will solve this?


Parameter is not found when using module outputs in KeyVault deployment

Bicep version
az bicep - 0.14.46
Describe the bug
Situation: Using modules to deploy a simple app service plan, app service and keyvault which has the app service's managed identity as a some allowed values for access policies.
Issue: running a what-if deployment through hits an error of "The template parameter 'appServiceName' is not found.
Expected: Should be able to use the output of a module as an input for KeyVault deployment.
To Reproduce
main.bicep:
param location string = resourceGroup().location
param appServiceName string = 'test-application-name'
param keyVaultName string = 'testkeyvault'

module appServicePlanModule 'ModuleTemplates/appServiceplan.bicep' = {
  name: 'appServicePlanDeploy'
  params: {
    location: location
    appServicePlanName: 'test-application-plan'
    appServicePlanSize: 'S1'
  }
}

module appServiceModule 'ModuleTemplates/appservice.bicep' = {
  name: 'appServiceDeploy'
  params: {
    location: location
    appServiceName: appServiceName
    appServicePlanId: appServicePlanModule.outputs.appServicePlanId
  }
}

module keyVaultModule 'ModuleTemplates/keyvault.bicep' = {
  name: 'keyVaultDeploy'
  params: {
    location: location
    keyVaultName: keyVaultName
    principalIds: [
      appServiceModule.outputs.appServicePrincipalId
    ]
  }
}

appserviceplan.bicep:
param location string
param appServicePlanName string
param appServicePlanSize string

resource appServicePlan 'Microsoft.Web/serverfarms@2020-06-01' = {
	name: appServicePlanName
	location: location
	sku: {
		name: appServicePlanSize
		capacity: 1
	}
}

output appServicePlanId string = appServicePlan.id

appservice.bicep:
param location string
param appServiceName string
param appServicePlanId string

resource appService 'Microsoft.Web/sites@2020-06-01' = {
	name: appServiceName
	location: location
	properties: {
		serverFarmId: appServicePlanId
	}
	identity: {
		type: 'SystemAssigned'
	}
}

output appServiceId string = appService.id
output appServicePrincipalId string = appService.identity.principalId

keyvault.bicep:
param location string
param keyVaultName string
param principalIds array

var principalsAccessPolicies = [ for principalId in principalIds: {
	tenantId: subscription().tenantId
	objectId: principalId
	permissions: {
		secrets: [
			'Get'
			'List'
		]
	}
}]

resource keyVault 'Microsoft.KeyVault/vaults@2021-04-01-preview' = {
	name: keyVaultName
	location: location
	properties: {
		tenantId: subscription().tenantId
		sku: {
			family: 'A'
			name: 'standard'
		}
		accessPolicies: principalsAccessPolicies
	}
}

Run the command az deployment group what-if --resource-group <resource-group> --template-file .\main.bicep and the error occurs.
Additional context
This was working for us at end of last week and has randomly stopped working this week.
If you edit the main.bicep instead have an existing resource of the app service, and then use existingAppService.identity.principalId instead of appServiceModule.outputs.appServicePrincipalId it works.

@woodylaurence -- are you only seeing this happen during what-if (vs the actual deployment)?

Apologies, I'd not tried that @alex-frankel
I've just checked and it does seem like az deployment group create... works fine with the same bicep that fails doing a what-if run.

No apology needed! We suspect we accidentally shipped a regression specifically in what-if, so this helps us confirm that. Will update this thread as we know more. cc @jeskew

I have been trying to track down a similar issue all day. Regular deployment works, but what-if fails. Running what-if on an earlier revision that I know have been through our CI-CD (which fires off a what-if as its first step) also fails.

A fix for this issue is being deployed as part of the ARM w07 release.

A fix for this issue is being deployed as part of the ARM w07 release.

@jeskew, when is ARM w07 release being deployed?

I just checked it it is almost done rolling out. I believe it will be done by EOD today.

I'm going to close this as w07 is rolled out with just a handful of exceptions due to other issues. For most/if not all, you should not be running into this issue anymore.


Unofficial repo?

Hi,
I found this Pytorch implementation of RTM3D: https://github.com/maudzung/RTM3D. Is this an unofficial implementation or if the official implementation is moved to that repo?

Hi @wufeim
It is my unofficial implementation with several modifications. I have asked the author some questions here.

Hi @wufeim
It is my unofficial implementation with several modifications. I have asked the author some questions here.

Thanks for the clarification. And thank you for your work! That is a great implementation and the code is clean and organized.


Added jokes and latency checker.

Jokes - gets a random joke from discord-jokes.
Latency - measures response times.

Removed jokes since you don't like to rely on random libs


Replaced const with let

Fixes Issue
**My PR closes https://github.com/Dun-sin/Code-Magic/issues/423
👨💻 Changes proposed(What did you do ?)
✔ Check List (Check all the applicable boxes)
My code follows the code style of this project.
This PR does not contain plagiarized content.
The title and description of the PR is clear and explains the approach.

PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project.  The results are listed in the  following section.
Issue Reference In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide. This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.
Protected Branch
In order to be considered for merging, the pull request changes must  not be implemented on the "main" branch. This is described in our Contributing Guide.  We are closing this pull request and we would suggest that you implement your changes as  described in our Contributing Guide and open a new pull request.
Conventional Commit PR Title
In order to be considered for merging, the pull request title must match the specification in  conventional commits. You can edit the title in order for this check to pass.
Most often, our PR titles are something like one of these:

docs: correct typo in README
feat: implement dark mode"
fix: correct remove button behavior

Linting Errors

Found type "null", must be one of "feat","fix","docs","style","refactor","perf","test","build","ci","chore","revert"
No subject found

@Bashamega I told you to read the rules, and you replied thanks but ended up not reading the rules. If it feels like a waste of time to you why bother contributing, if you unintentionally forgot about it, please read it and do the right thing. Else this PR will be closed without being merged


[DO NOT MERGE] Start snapshot build for release process

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Gradle Publish

Run Gradle Publish


Update README.md

@hasanzeynal please, consider my updates, thanks in advance.

Thank you for your attention, @mammadyahyayev !


Mobile Navigation Bar

One thing I'm missing in the UI is a mobile menu showing the navigation bar.

it is by design.
having a navigation bar and the content on a mobile(phone) form-factor is something our UX think  isn't a good experience.
There are other ways to do things in that area, like overlays and slide-in
but at present our primary target is for tablet and above form-factor as our data shows thats where is our 98% user base is.
I wont mind taking an PR on this if done without creating a performance issue.


Add getter to TFLite tensor dims

Add a public getter function of tensor dims in tflite interpreter. It is useful for users to fully aware the shapes of tensors, for processing dynamic shape outputs and for debugging.

Nagging Assignee @aselle: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @aselle: It has been 31 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @aselle: It has been 46 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Describe what COMMUNITY supported device types are

Should note that these are boards are:

not maintained by our team, but contributed by the community
don't get full testing on hardware for every release but rather rely on the community to test
can be sponsored and brought in house if a customer is interested.

closed by https://github.com/balena-io/docs/pull/1071


Reveal operations should not incur fees

Currently, reveals incur fees, and since reveals are now bundled with transactions, delegations, and originations, there can be unexpected doubling of fees. We want reveals to not insure any fees, even when bundled with other operations.

It seems @ba205 fixed it, already

This is no longer the case with Protocol 003.


Poc microcrate account

This is an experiment.  I don't have a great idea for what the tradeoffs might be, but all other things being equal, I think more encapsulated is better.

I like it that there are no use statements.   I like it that it can be plugged into zingolib as a "completely" contained component.    It should be easy to integration test..

I think we should probably have a crate for many "primitive" wallet objects like account. I think one crate for just this one struct will end up with a silly amount of crates. Or we just have the struct in zingolib.


Some private variables are not displayed in the debugger window

VSCode Version: 1.11.1
OS Version: Windows 10

Steps to Reproduce:

Have a similar code in place (can't put the original one here)

namespace A1 {
    export abstract class A extends B {
        public abstract el1: Object;
        private _el2: string;
        public get el2(): string { return this._el2; };
        private _el3: number;
        public get el3(): number { return this._el3; };
    }
}
namespace A2 {
   export class APlus extends class A1.A {
        public el1: Object = { foo:bar }
        public baz: X;
   }
}

Start to debug and instantiate a new object APlus.
On the debug window you'll see: el1, _el2, el2, el3, baz but you won't see _el3.

My debug configuration is the following:
   {
            "type": "node",
            "protocol": "inspector",
            "request": "launch",
            "name": "Local",
            "program": "${workspaceRoot}/node_modules/protractor/bin/protractor",
            "args": ["${workspaceRoot}/protractor.conf.js", "--troubleshoot"],
            "cwd": "${workspaceRoot}",
            "sourceMaps": true,
            "runtimeArgs": [
                "--harmony", "--expose_debug_as=v8debug"//, "--trace-warnings"
            ],
            "skipFiles": [
                "<node_internals>/**/*.js",
                "node_modules/selenium-webdriver/**/webdriver.js",
                "node_modules/protractor/**/browser.js"
            ]
        },

PS: I need to explicitly state the protocol as inspector (even though I use Node 7.8.0) because since 1.11.0 VS Code will launch the tests in legacy if I set it as protocol:auto
PS2: Don't know if this will help, but in the case where this is implemented if I check the proto element, then I can only see el2 and el3 (and both state TypeError: Cannot convert undefined or null to object)

What version of TS are you using? Output to which ES version? Maybe just post the transpiled JS? B is missing from this snippet, is it important? Also, export class APlus extends class A1.A { is not valid, I think you mean export class APlus extends A1.A {.
When I try something like this, I don't see _el2 or _el3, because TS actually doesn't output anything with those names, so they don't exist on the object at that point.


Handle missing doc in npm change stream

We assume all change stream events will include a doc, but Sentry suggests that is a bad assumption.
https://sentry.io/gratipay/gratipay-com/issues/284250550/

Hopefully #4574 should resolve this.

Done in #4574. !m @kaguillera @dmk246


[docs] Overriding nested component styles on custom components.

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/customization/how-to-customize/#overriding-nested-component-styles
Kind of issue
Unclear explanations
Issue description
Docs on Overriding nested component styles needs clarification.
When creating a component using the styled function, i expect to be able to target the class based name and slot of these components in the same way as components from @mui/material. By providing name and slot we are able to generate a class following the naming pattern described in the documentation. And logically, we should be able to target them.
So i would like some clarifications on

How to target custom styled components.
If this is a know limitation.

Interactive example: https://codesandbox.io/p/sandbox/nifty-bush-3x34xf
const CustomBox = styled(Box, { name: "Custom", slot: "root" })(
  ({ theme }) => ({
    height: "100px",
    width: "100px",
    background: theme.palette.background.paper,
  }),
);

// ...
<>
 <Box
        sx={{
          "& .Custom-root": {
            background: "red",
          },
        }}
      >
        <CustomBox />
      </Box>
</>

Context 🔦
We have an internal library of common components based on the material components. I need to select a layout component and change its styling. Its possible to to with "& .MuiBox-root" but i want to be some specific, and  aply the style to & .Layout-root instead.

When you create a component using the styled API, it generates class names with hash prefixes. This behavior is inherited from the styled API of emotion JS, which adds these hash prefixes. However, we also provide class names that you can use for custom styling, such as MuiBox-root as recommended in the documentation.
There's no need to specify the name and slot. You can use the provided class names. Attempting to target a class like & .Custom-root won't work because the actual class name is something like css-76zpby-Custom-root (or another hash prefix in your case). In a production build, it resolves to just css-76zpby. Therefore, it's better to rely on the class names we provide (e.g., MuiBox-root where Box is the component and root is the slot).
If you need to change the class name, you can use the experimental API for class name customization, as described in this link: https://mui.com/material-ui/experimental-api/classname-generator/#setup.
To target custom-styled components (I assume which you import from your internal library), you can create another wrapper using styled or the sx prop, like this:
const StyledCustomBox = styled(CustomBox,
...
...


Add status badges in Readme

as title

/test ?

/test metal3-ubuntu-e2e-integration-test-main

/close
This is not needed or relevant right now, discussed offline.


Code: 2 Message Invalid Key Type

I encountered an issue while initializing  RecaptchaEnterprise.initClient('siteKey',timeout: 10000)
Could not proceed further with after this issue?
Testing on IOS simulator

You are using an android or web key. You need to use an iOS key.


Task AzureDevTestLabsCreateVM can only allow 1 parameter being overrided

I'm trying to add task AzureDevTestLabsCreateVM to my yaml pipeline. The code piece is like below:

task: AzureDevTestLabsCreateVM@3
displayName: 'Create virtual machine'
inputs:
azureSubscription: $(mySubscription)
LabId: $(myLabId)
VirtualMachineName: $(myVMName)
TemplateFile: 'MyTemplate.json'
ParametersFile: 'MyParameters.json'
ParameterOverrides: |
-password '$(my_password)'
-sasToken '$(my_token)'

Let's say I have 2 parameters (password and sasToken) in the template file, and I'm trying to override them with the yaml variables. I notice that the first parameter (password) doesn't actually get overriden.
After looking at your source code, I think the problem is in function addParameterOverrides() in deployutil.ts. Here is the piece of code in this function:
function addParameterOverrides() {
// Skip some code here
let newParameter: DevTestLabsModels.ArmTemplateParameterProperties = Object.create(DevTestLabsMappers.ArmTemplateParameterProperties);
// Skip some code here
parsedParameterOverrides.forEach(parameterOverride => {
    // Code to find a newParameter in the overrides.
});

existingParameters = checkParamArray(newParameter, existingParameters);

return existingParameters;

}
I think there is a bug here. Currently implementation only updates existingParameters with the last parameter in parsedParameterOverrides. That's exactly the problem I'm having now. As a solution, the following line of code should be put into the forEach loop:
existingParameters = checkParamArray(newParameter, existingParameters);

Thank you for the report. We'll take a look and report back.

Is there any update, or plan? As a workaround, we can hard-code the parameters in the arm template. But we don't want to do that for sensitive data such as sas token for security concern. So we definitely need this fix.

@simen-wu, we have a story in our backlog to finalize these preview tasks. No ETA.

is this problem still exists?
I am trying to use the same azure dev test labs task and trying to override 2 parameters.
-LogonPassword $(VMPassword)
-Azure_Pipelines_Agent_vstsPassword $(vstspassword)
but once the vms are created I am not able to login to the vm with actual value of my secret $(VMPassword) and even the agent download is also failed with unauthorized error.
I have 2 doubts here:

is the syntax of parameters override is correct? I have not enclosed the variables with single quotes, is that the reason for failure of agent download?
is there any other way to use secrets like directly giving the secret reference in the ARM template?

PS: I am using Azure Key Valut task to dowload the secrets and make them available for whole job


Add suggestions enum and input from internal schema

Issues

Changelog

[ ] List your changes here
[ ] Make a changie entry

Tophatting

👎 - we cannot use "internal" things in opslevel-go. We need to get the ruby teams to make the API public first and my understanding is they are not happy with the API as is so it needs significant work that is not on the short term roadmap

Yep, this was just me experimenting. closing


fix(engine) Make sub-parts of Quote visited by visitors

Fixes #1196
The investigation for #1196 revealed that subparts of Quote variant in the AST were ignored by visitors. The easiest way to change that is to use a quote_contents ADT inside instead of an inlined polymorphic variants. This involves quite a few changes here and there.

I succeeded in making a version that fixes the bug for F*. But I am not sure that it does the right thing for the generic printer (at least it builds correctly and doesn't break tests).


Merging per instructions (2)

Your merge request is backward. You probably want to merge from my repo to yours. Instead, this request attempts to merge your content into my repo.


buildstream/utils.py: Fix regex Deprecation Warning

Part of https://github.com/apache/buildstream/issues/1758, backport of https://github.com/apache/buildstream/commit/e8055a56d265d558248557782fa3ed0e6419caa4

The former warning is now an error on latest Python release.


Dynamic Trees Spawning Issues

Describe the bug
In a newly generated world, I am getting both "vanilla" style thick full block style trees, mixed in with the Dynamic style trees. I have both BOP and the BOP Dynamic Trees addons installed. Below is a screen shot of an example as well as a screen shot of the mods that I have installed. I cannot think of anything here that would be causing this issue, but I'm not a developer, so what do I know?!
This is for minecraft 1.12.2.
Screenshots

Rendering Glitches
I am not using Optifine

I believe this is being caused by Recurrent Complex. Should be a nature option in its config or something along those lines.

Going to close this as I believe the above should solve your problem. If it doesn't feel free to re-open the discussion.


Cleanup docs for app developers

This PR removes outdated instructions for writing custom apps and instead adds pointers for the developer documentation.


Timetable creation fails with feed using stop code

@rpedraza01 I think this is related to issue 6474
I was working on Placer County which uses stop codes. Running locally I was able to get this report.
Source: https://data.trilliumtransit.com/gtfs/placercounty-ca-us/placercounty-ca-us--ttable.zip
Warnings:
No stop found found for stop_id=1006 in timetable_id=4975
No stop found found for stop_id=1001 in timetable_id=4977
No stop found found for stop_id=1006 in timetable_id=4978
No stop found found for stop_id=1001 in timetable_id=4979
No stop found found for stop_id=7001 in timetable_id=4980
No stop found found for stop_id=1002 in timetable_id=4981
No stop found found for stop_id=7001 in timetable_id=4982
No stop found found for stop_id=1002 in timetable_id=4983
No stop found found for stop_id=3017 in timetable_id=4984
No stop found found for stop_id=1001 in timetable_id=4985
No stop found found for stop_id=3020 in timetable_id=4986
No stop found found for stop_id=3023 in timetable_id=4987
No stop found found for stop_id=4002 in timetable_id=4988
No stop found found for stop_id=1001 in timetable_id=4989
No stop found found for stop_id=1002 in timetable_id=4992
No stop found found for stop_id=1001 in timetable_id=4993
No stop found found for stop_id=1002 in timetable_id=4994
No stop found found for stop_id=1001 in timetable_id=4995
No stop found found for stop_id=7001 in timetable_id=4998
No stop found found for stop_id=4001 in timetable_id=5004
No stop found found for stop_id=6006 in timetable_id=5005
No stop found found for stop_id=7001 in timetable_id=5571

I tried reproducing that issue through the G2H website and it didn't make any timetables. I don't have it setup on my machine to run locally though.

@MarkTrillium Are you still recieving that error? I just ran the files locally without an issue.

@MarkTrillium, Estelle reminded me that we can now create timetables through GTFSM on the Exports page. - we don't have to use the G2H website or even run them locally. Would you please try that going forward? You can ask Estelle for a refresher on how to do that.

Just to clarify! Still use local for any special sites like WRTA since their template isn't in g2h-as-a-service in Manager. Adding those to this repo would be nice however in the future (https://github.com/Trillium-Solutions/timetables) But I'm guessing @MarkTrillium You ran the files locally since Manager was down yesterday? Either way, let me know if you're still having issues locally.

Placer is an example of where this issue is occurring, but I've made due and it is not an immediate crisis.
@ekeaton This morning I produced the timetables locally after posting an older -ttable export. It will fail again if I post the current export to Transmit.
@rpedraza01 +1 Estelle's comment. Typically Support runs the timetables through GTFSM, reviews and posts the timetables in the zendesk ticket, but there are some sites that are a little special.

@MarkTrillium Based on this closed issue https://github.com/Trillium-Solutions/GTFSManager/issues/6474
Can you try running g2h locally again for Placer with the current --table export (not the older export you previously used)


Fixed #25465 -- Returned line breaks addition to plain readonly fields in admin

As you can see, this broke some tests. I think the correct resolution would be to apply that filter somewhere in Python code. Also, we need a regression test for this fix.

@timgraham ok done.

@timgraham ok done.

If this is a regression in 1.8, it qualifies for a backport, in which case, please add release notes for 1.9.1 and 1.8.8.

Ok done.

merged in 69208a5a1c55d42ca0eaffa900be643d9f801089, thanks!


contrast limits slider broken with very small max value

🐛 Bug
I was working with an image today with a very small max value, 2.6e-17, but a lot of richness around 10-20 etc, and found that the contrast limits slider is not adjustable, i.e. when i click on it I can't drag it around.
To Reproduce
v = napari.Viewer()
v.add_image(np.random.random((10, 10))*2.6e-17)

Expected behavior
I should be able to adjust contrast limits value
Environment
Current main, and latest superqt

If you can produce this with only superqt, can you open an issue there instead?

PR fix created on the napari side #3895


404 Page

Description
Design a 404 Not Found Component

Be creative and make it look geeky and codey :p

Screenshots/testimonials

I am interested in making this component . Should I start working on it

That's awesome @sandesh-08! 😄 Looking forward for your contribution.. 🙌🏻

closing against #30


Incorrect rendering of titlebar in full screen mode

Description
The demo app, when switched to Full Screen Mode, displays an empty titlebar. See the screenshot attached.
Steps To Reproduce

Compile the demo.
Launch the demo.
Enter the full screen mode.

Expected behavior
The upper part of the sidebar and the upper part of the main pane should still be visible.
Screenshots

Yes, seeing that too. It happens even when ToolBar or Titlebar are missing, so probably an issue with the Swift window layout code?

Did some testing, and it indeed comes from MainFlutterWindow.swift. The full screen settings here are not used at all currently. Simply changing this: https://github.com/GroovinChip/macos_ui/blob/e4c37d6699d4f6b8b1ce98ded9a7d250d7d64341/example/macos/Runner/MainFlutterWindow.swift#L37
to also extend NSWindowDelegate:
class MainFlutterWindow: NSWindow, NSWindowDelegate {
fixes the issue, and makes the app use the required full screen settings.
However, the opaque empty toolbar still shows up when hovering the mouse at the top of the screen. We could try to listen to the full screen event and toggle it accordingly.


chore: Fonts and general styles

Adds customer fonts and a little bit of general styling.
Test URLs:

Before: https://main--walgreens--hlxsites.hlx.live/personal-care-deals
After: https://general-styles--walgreens--hlxsites.hlx.live/personal-care-deals

Brought A11Y back to 100 in https://github.com/hlxsites/walgreens/pull/22 , sorry for the noise :)


Fix MDArray generated Get and Address methods

We need to push something on the stack for these after ThrowHelper
returns (even though it really doesn't return).

Hi @MichalStrehovsky, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Bug: Google Drive overlay icons are blurry

Description
How they look in windows explorer:

How they look in Files:

Steps To Reproduce

Set files and windows explorer to similar preview size.

Steps to fix
Make icons have the same size and smoothness as in windows explorer.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.819
Relevant Assets/Logs
debug_fulltrust.log

Is this still an issue in the preview version? https://files.community/download/preview

Is this still an issue in the preview version?

Yes.

Can you send a screenshot of how it looks in the details layout and another one for the grid layout?

Details:

Grid (Large icons):

@soredake what's your screen scale & resolution?

what's your screen scale

100%

resolution

1920x1080

Seems this is not related to google drive, files app shows shortcut icon for folders small too.


GoogleSignIn with FitnessOptions throws wierd ApiException 5000

GoogleSignIn with FitnessOptions
Step 2: Describe your environment

Device: Nexus5x, Nexus5, GalaxyS7 and several more devices
OS version: Android 5+ maybe older are affected too
SDK version:

implementation "com.google.android.gms:play-services-fitness:11.8.0"
implementation "com.google.android.gms:play-services-auth:11.8.0"

[Android] Google Play Services version: 11.9.51<PHONE_NUMBER>61)

Step 3: Describe the problem:
Fitness.getHistoryClient(context, GoogleSignIn.getLastSignedAccount(context))
        .readData(dataReadRequest)

end with ApiException with FitnessStatusCode 5000 (https://developers.google.com/android/reference/com/google/android/gms/fitness/FitnessStatusCodes.html#API_EXCEPTION)
even though I have had requested all needed permissions and user has approved my request
and status.startResolutionForResult(..) cannot be called because resolution == null
Steps to reproduce:

download my library with example https://github.com/thefuntasty/RxGoogleFit.git (master branch)
create new google account (gmail)
with that new account signed in run example app(fittester) from repo in step 1
click on GET DATA in app
select newly created google account and approve all permissions
app should print Statuses for all read requests and some insert requests,
instead it throws ApiException

Observed Results:

What happened?

in ResolutionActivity.onActivityResult after all granted perms I try to get data with tryGetData()
which is successful but same request in HistoryReadSingle with same Account results in ApiException 5000
this happens only on newly created google accounts or on account where no fit permissions have been requested yet
the most strange thing is, when I close app from last used apps and run it again and click GET DATA everything works
Expected Results:
when I check GoogleSignIn.hasPermisions(account, signInOptionsExt) and it returns TRUE, I would expect that I can request data via Fitness.getHistoryClient(context, account).readData(..)
without any ApiException just like it behaves when I close app and start it again
or when there is statusCode 5000 there should be resolution too as documentation says
Relevant Code:
// code here to reproduce the problem
https://github.com/thefuntasty/RxGoogleFit.git (master branch)

@RudolfHladik Did you get fix this?

same problem. Do u solve it?

same problem. @Tooto @pawnjester @RudolfHladik have you guys got any solution? thanks

I have the same problem, now my app completely broken

I have the same problem, now my app completely broken

The problem is with Permission. YOu should have matching permission.
My isseu was, even though I did requested for Fitness Read and Write application, Google was not considering my write permission and was throwing error, hence I removed my write request from app and used only read request, and app started to work.
Note here- on Google Play console I allowed both scope Activity.Fitness.Read and activity.fitness.write
but somehow, in development of app, Google was only allowing me to use read scope.
might be a bug in current sdk

I have the same problem, now my app completely broken

The problem is with Permission. YOu should have matching permission.
My isseu was, even though I did requested for Fitness Read and Write application, Google was not considering my write permission and was throwing error, hence I removed my write request from app and used only read request, and app started to work.
Note here- on Google Play console I allowed both scope Activity.Fitness.Read and activity.fitness.write
but somehow, in development of app, Google was only allowing me to use read scope.
might be a bug in current sdk

I have fix the problem: update to SDK 20.0.0 and request both read and write permission. But I think that there is a bug, on the consent screen and docs they say FitnessOptions.ACCESS_WRITE permission can write and read app own data, but not could not read itself data, unless request READ permission.
More info https://developers.google.com/fit/improvements

I just Comment out  this line addDataType(DataType.TYPE_STEP_COUNT_DELTA, FitnessOptions.ACCESS_WRITE)
and update both the library
implementation 'com.google.android.gms:play-services-fitness:20.0.0'
implementation 'com.google.android.gms:play-services-auth:19.0.0'
uninstall the app and again install works fine


enhancement: only one button to publish/project election results

create an election
project the election
start it and enter election results
mark one person as elected

result: The elected person is immediately displayed as such in the projector.

now press the "Publish" button.

result: nothing happens neither in the projector view nor inside OS.

Damn.
OS is so unintuitive in this respect. Why do one have to press 2 buttons to display the results? The "Publish" button should be sufficient. There is no need for an additional projection button (or for the publish button).

@donovaly Right the current ui concept is not very intuitive...
Suggestion: We remove the "publish" button and use the "project" button only for publish and project because very often you will publish all results in one step with project the result slide.
Important for me: Don't use hidden project function behind a button which does not say clear that clicking on it changes the projector.
@donovaly Would this suggestion ok for your?

I do not understand how this publish/unpublish concept should work. What if I want to pubulish results in the backend but do not want to project anything? How can I revert publishing results (if it was a mistake)?

What if I want to pubulish results in the backend but do not want to project anything? How can I revert publishing results (if it was a mistake)?

Right, both would be not possible with this "easy publish concept" (one project&publish button). Maybe we could add a new "unpublish" button for revert. Publish for backend only is not really important I think.

What if I want to publish results in the backend but do not want to project anything?

Just don't press the project button.

How can I revert publishing results (if it was a mistake)?

Unpress the project button.
In my opinion after entering the election results they are already published. You can change the results any time by re-entering the results. So I don'*t see any reason why one need the Publish button.

we like the posibility to publish election results, should be undo. this requires an extra button. So we don't know a better concept - so we wont't change it at the moment. Please reopen if you have a nice idea.


Can i use TT-Mars for bacterial genomes?

HI, I study bacterial genomes.
I want to use TT-Mars to assess mapping quality in structure  variants of bacterial short reads to a reference  genome.
Can I use this for the purpose?

Hi!
TT-Mars is designed for any kind of structural variants validation. It can assess the accuracy of your variant calls as long as the genome assembly is available.
Could you provide more details about your project?
Thanks,
--Quentin

Dear Quentin,
Thank you very much for your reply.
I want to create input files for drawing trees by using Gubbins, BratNextgen, etc. that can detect and remove recombination regions. I'm planning to conduct time-divergence analysis after removing recombination regions.
I have short reads of test isolates (they are haploid) that belong to the same clone and I plan to map the reads to a reference genome. Then, after variant calling for both SNV and indels, I want to obtain aligned complete genomes of the test isolates using vcf files and vcf-consensus. I mask the indel regions including structure variant regions with "N". Through the process, I want to use TT-Mars to evaluate the quality of the called SNPs and indels. As you know, mapping and variant calling for bacterial genomes are still challenging; there is a problem of the balancing between sentisitivity and faulse positve. In addition, the detection of large indels (i.e. structure variants) is very difficult and there are few softwares that can detect large indels with a good sensitivity. Actually, I tried to use Freebayes, Deepvariant, GATK, Snippy, Pilon and Octopus, only Pilon could detect large variants; other softwares often identified the regions as the accumulating SNP and/or short indel regions.
I want to build a in-house pipeline to do these process automatically (and accurately) due to thoudands of isolates. So, I want to take TT-Mars into the pipeline for the purpose of the judgement of the qulity of the variant calling.
Satoshi
Mapping.pdf

Hi Satoshi,
Thanks for your clarification. Here are my suggestions and notes about using TT-Mars in your project:

TT-Mars is designed for validation of structural variations. It should work for indels > 20bp, but not for SNPs.
TT-Mars is designed to validate on human genome with pre-processed files (which can be downloaded here: https://github.com/ChaissonLab/TT-Mars/blob/main/download_files.sh and https://github.com/ChaissonLab/TT-Mars/blob/main/download_asm.sh). It requires human reference genome and sample genome assembly. In your case to use TT-Mars, a reference genome and bacteria assembly is required.
You can refer to this file about how to use TT-Mars with your own reference and sample genome: https://github.com/ChaissonLab/TT-Mars/blob/main/liftover.sh
Basically, it follows these steps:

align your assemblies to a reference
TT-Mars will trim overlapping contig in the alignment
Generate liftover files
Compress
Generate regions not covered by the assemblies
So it needs additional efforts as mentioned above.

Hope this helps. Please let me know if you are not clear with any of the above steps and I will be more than happy to help.
--Quentin


Session Window, last blue window in diagram should start with at event

The Diagram of "Session Window" is not precise enough, left side of third blue window should start with dashed line, which points to the event instead to the timeline.

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

ID: 63c64692-7533-cc84-251d-1994aab6a8fb
Version Independent ID: 10d6faea-cf4a-1e28-2dd6-76146128e1ce
Content: Introduction to Azure Stream Analytics windowing functions
Content Source: articles/stream-analytics/stream-analytics-window-functions.md
Service: stream-analytics
GitHub Login: @JSeb225
Microsoft Alias: jeanb

@mathias-nickl
Thanks for the feedback! We have assigned the issue to the content author to investigate this and update the document as appropriate.

Thanks, we submitted a PR to edit the illustration. It should be updated in the public doc shortly
#please-close


Add dependency line to README

It seems common practice to show the Cargo.toml line in rust repos. Adding that, so the lazy of us don't have to go an extra round trip to find the latest version.

Thanks for the quick merge and the improvement on my pull-request!

Thank you


constant generation: generates an OpConstant for the alignment value on OpCopyMemory when it doesn't have to

The internal intrinsic spirv.copy_memory intrinsic will generate an Aligned literal operand with the alignment value taken from one of its arguments.
E.g.
  call void @spirv.copy_memory(%struct.a addrspace(1)* %struct_out, %struct.a* nonnull %local_a, i32 4, i32 0) #1

Will result in OpCopyMemory %57 %56 Aligned 4  but also  %uint_4 = OpConstant %uint 4   , where %uint_4 could be dead.
The compiler should be able to sweep away dead constants like this, or avoid generating them in the first place.

copy_memory does not exist anymore. It has been removed by #1119 .


Scoreboard enhancements

[ ] Competition scoreboard ordered by highest scoring team (ideally even various orders available)
[ ] Series over time for all scoreboards (i.e. a time series for scores within a task group)

I assume this is now superseded by #194, right?

Closing this as it got superseded by #194 , as correctly pointed out


Travis Fixes

Fixes artifacts indentation, as well as download stats filename.

https://adafruit-circuit-python.s3.amazonaws.com/index.html?prefix=adabot/
Fixed this lister too.


MSID: 88510 Version: 2 DOI: 10.1101/2<IP_ADDRESS>6306

MSID: 88510
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6306
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=88510
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6306

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/88510v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6306
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/88510/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

elocations missing from references.


ENSWBSITES-967: Enable download of cDNA sequences for non-coding tran…

…scripts

Type

[ ] Bug fix
[ ] New feature
[x] Improvement to existing feature

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-967
Importance
N/A
Description
Enable download of cDNA sequences for non-coding transcripts
Deployment URL
http://enable-cdnadownload.review.ensembl.org
Views affected
Entity viewer (Instant download)
Genome browser ZMenu
Other effects
None that i know of

[ ] I have checked any requirements for Google Analytics
[ ] I have created any required tests
[ ] The PR has as its final commit a WASM/JS update commit if any Rust files were updated.

Possible complications
N/A

@imransl @azangru can you please review again, i have put in a fix for the checkboxes issue reported by Steve in the ticket during QA. The download itself is still broken, i have reported it to kamal who will hopefully fix it this afternoon.

Still looks good 😄

@ens-ma7 Both the ncRNAs that @ens-st3 highlighted as broken now download the cDNA properly


[Bug] Update Stmt Fail use in predicate query

Proposed changes
Issue Number: close #11413
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[x] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

please update title to pass PR title Check the valid title is:
\[([a-zA-Z0-9 \-_])+\]\(([a-zA-Z0-9 \-_])+\)(.*)


Anghami doesn't scrobble

Song playing in Anghami shows up in Last.fm now playing but it doesn't really scrobble it. The extension icon also keeps blinking.

Duplicate of #1452.


Where to put Tag ID for Withdrawing XRP

I dont know where to put tag ID for XRP when using witdraw API

Hi @noteshamrock , there's now an addressTag parameter availabe in the withdraw method.


Consider extending subdiv arg to accept multiple values

Add ability to request holidays for more than one entity's subdivision. Keep it backward compatible.

This sounds helpful. I was looking into this for my organisation that is spread across multiple Australian states. I think to work around the lack of this I'll probably have to manually combine the holidays for each state into a set.

I think I once started implementing this for v0 but I'm not sure if it'll make it there. Will definitely take another look for v1 support.

I managed to work around it as described above, but definitely a 'nice-to-have' feature.


Signing in for opportunity lands you on home page, not opportunity page

Current behaviour:
Given a user is not signed in
And they are on an Opportunity page
Then the user will see a message sign-in to submit a proposal
When the user clicks sign in they will go to the sign page
And when they click sign
Then the user is redirected to the homepage
Expected behaviour:
Then the user is redirected back to the opportunity page

Tested in Dev branch today (July 24, 2018).  The application's behaviour is now consistent with the "expected behaviour" set out above.  Closing this issue.


Added butterfly pattern

I hope this helps your repository

@all-contributors please add @dinesh21o9 for his code


QDM 5.0 Model Info

Questions about the following file: https://github.com/cqframework/clinical_quality_language/tree/modelinfo-profiles/Src/java/qdm/src/main/resources/gov/healthit/qdm/qdm-modelinfo-5.0.xml
First: The file contains the following label values for the patient characteristic datatypes:

Patient Characteristic: Birthdate
Patient Characteristic: Clinical Trial Participant
Patient Characteristic: Ethnicity
Patient Characteristic: Expired
Patient Characteristic: Payer
Patient Characteristic: Race
Patient Characteristic: Sex

Based on the draft QDM 5.0 document (https://ecqi.healthit.gov/system/files/qdm_5_0_draft_.pdf), none of these datatypes contain colons.
Second: It appears that the generic "Patient Characteristic" datatype is missing from the model info file. Since this datatype is part of QDM 5.0, how is this datatype being handled?
Third: There is a typo for the DeviceApplied element anatomialLocationSite - should be anatomicalLocationSite

Agreed on all counts, thank you for the detailed review. These changes have been made to the 5.0 and 5.0.1 xsd and model info files.


Update sql-data-warehouse-load-sample-databases.md

Hi @lodipalm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (lodipalm). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@lodipalm : Thanks for your contribution to the Azure documentation! The author, @lodipalm, has been notified to review your proposed change.

@lodipalm - Drew, please resubmit via the private repository. For more info, see https://microsoft.sharepoint.com/teams/azurecontentguidance/article-author-working-in-the-public-repo


Fixedunittests

Fixed a bug in the unit tests that led to false positive test results, and fixed those tests too.
Added two convenience functions to create handles more easily: register_handle_kv(..) and register_handle_json(...).
Added a NotImplemented exception for an argument that is not treated.
Some simplification of code in the REST client.

Added some developments I had made back in 2020 but never merged:

Removed logging-handler, which should not be present in a library, and which made all log statements be printed double to stdout.
Convenience: Import the relevant submodules in init.py.

The unit tests pass.


possible names for the package

SAMOA
Simulated Agent-based Mental-health Outcome Analysis
COMMA
COvid Mental-health Model with Agents

LIMES
Lockdown Induced Mental-health Effects Simulator

Comma wins! I close this issue


COBOL compiler does not require SYSUT16 and SYSUT17.

cobol.groovy is allocating SYSUT01 to SYSUT17. However the compiler is only using SYSUT01 to SYSUT15.
https://www.ibm.com/docs/en/cobol-zos/6.4?topic=output-data-sets-used-by-compiler-under-zos
The language script should be changed to avoid the unnecessary allocations.

My system has a storage group which seems to prevent the allocation of temporary data.
So I have made a test to build a Cobol program with values 17 and 16 in the Cobol.groovy script, it didn't work.
But test OK with value 15:
(1..15).toList().each { num ->
compile.dd(new DDStatement().name("SYSUT$num").options(props.cobol_tempOptions))
}

This has been delivered with zAppBuild Release 3.6.0


fix documentation on tryCatch method

In regards #2519.
However I can see that the it was fixed on the source with another example
R.tryCatch(JSON.parse, R.F)({});      //=> false
But hasn't been built to reflect it on the documentation

Changed the examples (I squashed the commits to avoid pollution)
for a simple scenario:
R.tryCatch(() => { throw 'foo'}, R.always('catched'))('bar') // => 'catched'

and a little more advanced one:
R.tryCatch(R.times(R.identity), R.always([]))('s') // => []

Thank you!

Is this issue still open?

This was merged some time ago; it is closed.


interested in levenberg-marquart ?

Hi, is the developer interested to add the the levenberg-marquart algorithm to the package? I have implemented it recently in Torch. Practice has shown that it is one of the best optimization algorithms for common (not deep) neural nets.

Sure, if you have an implementation please make a pull request.


parser: add nowait keyword, add for update no wait grammar, and related error…

… code/msg

What problem does this PR solve?
support select for update nowait
# Session 1:

mysql> CREATE TABLE t (i INT, PRIMARY KEY (i)) ENGINE = InnoDB;

mysql> INSERT INTO t (i) VALUES(1),(2),(3);

mysql> START TRANSACTION;

mysql> SELECT * FROM t WHERE i = 2 FOR UPDATE;
+---+
| i |
+---+
| 2 |
+---+

# Session 2:

mysql> START TRANSACTION;

mysql> SELECT * FROM t WHERE i = 2 FOR UPDATE NOWAIT;
ERROR 3572 (HY000): Statement aborted because lock(s) could not be acquired immediately and NOW

What is changed and how it works?
Check List
Tests

Unit test

Code changes

Has exported variable/fields change

Side effects
Related changes

Need to update the documentation

NOWAIT is an unreserved keyword, we need to add it to UnReservedKeyword: section in the parser.y.

From the description in https://dev.mysql.com/doc/refman/8.0/en/select.html#idm140006656103232, it's more appropriate to make NOWAIT its own modifier rather than grouping FOR UPDATE NOWAIT as a whole thing.
    [FOR {UPDATE | SHARE} [OF tbl_name [, tbl_name] ...] [NOWAIT | SKIP LOCKED]
      | LOCK IN SHARE MODE]]

we're currently groupint "FOR UPDATE" as a whole, and "SelectLockType" is defined as single int type,   maybe we need to some refactorations on this for 8.0 compatibility ?  There will be some other related work, do this refactoration in the future?
@coocood @kennytm

LGTM

@jackysp @tiancaiamao PTAL

we're currently groupint "FOR UPDATE" as a whole, and "SelectLockType" is defined as single int type, maybe we need to some refactorations on this for 8.0 compatibility ? There will be some other related work, do this refactoration in the future?

@cfzjywxk IMO it's better to turn SelectLockType into a struct now, or add a new member to SelectStmt indicating whether it is NOWAIT (and SKIP LOCKED in the future).
To support SelectLockForUpdateNoWait you'll need to modify TiDB once now, and later when you refactor for 8.0 compatibility you'll need to modify the code again. So I think it's better to choose a solution which we know is going to last.

@kennytm
We can modify the code again later when we need to.
It's not certain if we need to support those features.


Possibly Error

There is an error possibly

Please elaborate on why you think there might be an error, closing for now.


Correct wording of the WSL recommendation

Fixes  #104901

Sorry but you have a conflict becasue of my version bump! Just make sure you do not overwrite it and you use the 1.48.1


Parse config settings as integers

For some reason, my environment variables and .env files had the integers coming through as strings and I was seeing this error:
TypeError [ERR_INVALID_ARG_TYPE]: The "msecs" argument must be of type number. Received type string ('30000')

LGTM, thanks!


feat(spanner): add samples for PG

Adds samples and integration tests for PG features.

@codyoss Please help approve this

This is a lot of region tags to add in a single PR. In the future, please break it up into multiple smaller PRs. That makes code review both faster & more efficient!

@codyoss @telpirion Thanks for taking time to review this PR, I have updated the PR as per suggestions, please review again.
Sorry about the huge PR, will keep in mind to open smaller PR.


Feature/ios/location hashing

Description:
Adds the native iOS component to support storing and retrieving scrypt hashes based on geohash and timestamp. Includes Realm migration with the addition of a hashes column, which is a List of Strings containing all possible hashes for the given timestamp. No migration code needs to go inside the block but the schema version must increase and there must exist a migrationBlock. https://realm.io/docs/swift/latest/#migrations
This PR brings in and slightly modifies a geohash struct found here: https://github.com/nh7a/Geohash/blob/master/Sources/Geohash/Geohash.swift
It also brings in CryptoSwift for the Scrypt requirement: https://github.com/krzyzanowskim/CryptoSwift
Currently when running unoptimized tests in Debug mode, the Scrypt hashes take a long time to generate (290+ seconds for 8 hashes).  We might need to look into a different Scrypt implementation which may need to be bridged from a lower level.
Linked issues:
Android PR with corrected comments: https://github.com/Path-Check/covid-safe-paths/pull/907
Previous iOS PR: https://github.com/Path-Check/covid-safe-paths/pull/898
Previously the thought was to bridge a saveLocation method from the react side but that brought up the issue that the RN dependency actually saves it's own SQLite DB of location data.  This required us to provide a transform the the library and implement the hashing natively on iOS and Android.
iOS component for:
#807
#806
How to test:
Run  ios/COVIDSafePathsTests/

Before merging I would like to try to use a c library https://github.com/technion/libscrypt/tree/a402f4116245ce8677b3d9f4f87096b5ccbe26e9, which will require a little setup work from the swift side, but should prove beneficial for performance.

Closing with new PR, much tidier with all tests working https://github.com/Path-Check/covid-safe-paths/pull/928


Lazy loading doesn't work with Angular 2 spa.

We use Angular 2 template but it is not possible to use lazy loading with this template.
What ist wrong?
            { test: /\.ts$/, include: /ClientApp/, use: ['awesome-typescript-loader', 'angular2-template-loader', 'angular-router-loader'] },

Call:
path: 'login', loadChildren: './+login/login.module#LoginModule',

see here https://github.com/aspnet/JavaScriptServices/issues/792

Sorry #792 doesn't help. We have all search.  Nothing runs. Debugging breaks with lazy loading modules.  So we can't work. We search one bug after next.
Is there a final Version which run really with lazy-loading and Debugging?

@j-nord I don't know what's not working in your case, but I just tried adding lazy loading using the exact set of changes in the example mentioned from #792, and it worked perfectly.
I've just pushed an update to this example so that it's now rebased onto the latest template: https://github.com/aspnet/JavaScriptServices/commit/df5d27d0d8fb82570cb5d72ce7b763601ab3b64b. This definitely does lazy loading. So, I'd recommend you use that as a basis for comparing against your application. Hope that's OK!

Yes, that's ok. Thank you for your help.


[Low] Message about deletion of a locator is displayed when selecting element type in the plugin filter

Summary: the bug is reproducible with Vuetify library only, and under the following conditions: after deletion of a locator (in the plugin window or in the website one), all checkboxes in the plugin filter are unchecked and then any of them is checked (see attached screencast)
Steps to reproduce:

Generate locators via JDN plugin using Vuetify library
Delete any locator
Click filter button in the plugin window and uncheck all checkboxes
Check any of the checkboxes in the filter that does not coincide with the deleted element type (e.g. if "Alert" has been deleted, check any other that "Alert" checkbox)

Expected result: checked type of locators is highlighted in the website window, locators are displayed in the JDN plugin window, no message about deletion is displayed
Actual result: checked type of locators is highlighted in the website window, locators are displayed in the JDN plugin window, the message about previous deletion is displayed
https://user-images.githubusercontent.com/106632366/224316532-2400e73b-57e0-44dd-a99a-aeb7c1eb5b9d.mp4
JDN v 3.9.411 Back-end v 0.2.27
Windows 10 (64 bit) / Chrome Version 111.0.5563.65
The bug was found during testing of the https://github.com/jdi-testing/jdn-ai/issues/1062

JDN v 3.9.425 Back-end v 0.2.27
Chrome Version 111.0.5563.111 (Official Build) (64-bit)
The bug is fixed.
https://user-images.githubusercontent.com/106632366/227188639-34c4ba53-f229-4996-9d34-c2ec5857aefc.mp4


Reader cannot match writer error

Occasionally we get the following error when running ROS 2 tests repeatedly:
Writer 1.f.<IP_ADDRESS>.<IP_ADDRESS>.0.0|<IP_ADDRESS> cannot match reader 1.f.<IP_ADDRESS>.<IP_ADDRESS>.0.0|<IP_ADDRESS>

or
Reader 1.f.0.2.a1.6f.<IP_ADDRESS>.0.0|<IP_ADDRESS> cannot match writer 1.f.0.2.a0.6f.<IP_ADDRESS>.0.0|<IP_ADDRESS>

Example 1: http://ci.ros2.org/view/nightly/job/nightly_linux_repeated/736/testReport/junit/(root)/projectroot/test_showimage_cam2image__rmw_fastrtps_cpp/
Example 2: http://ci.ros2.org/view/nightly/job/nightly_osx_repeated/772/testReport/(root)/projectroot/test_tutorial_add_two_ints_server_add_two_ints_client_async__rmw_fastrtps_cpp/
Example 3: http://ci.ros2.org/view/nightly/job/nightly_linux_repeated/725/testReport/(root)/projectroot/test_tutorial_add_two_ints_server_add_two_ints_client__rmw_fastrtps_cpp/
Is there anything we can do to prevent this from occurring?

We are still interested in knowing if it's safe to ignore this error or if we should be making changes to our system to prevent it from occurring

Commit 398e0b68592d2ee8fd54fef19667f174f54674a1 removes those messages. There is no error in your case. It shows the reader/writer has been already matched.

thanks @richiware


Add --//some/path:incompatible_some_flag  as a valid incompatible flag format

fixes #118

Does that mean we have actual flags that start with -- and then go on with a label syntax in Bazel?
Like this?
bazel build --//some/path:incompatible_user_defined_flag --spawn_strategy=foobar //my:app

On Tue, Mar 10, 2020 at 8:56 AM Philipp Wollermann<EMAIL_ADDRESS>wrote:

Does that mean we have actual flags that start with -- and then go on
with a label syntax in Bazel?
Like this?
bazel build --//some/path:incompatible_user_defined_flag --spawn_strategy=foobar //my:app

Yes.  https://github.com/bazelbuild/bazel/issues/8857
It is removal of starlark build_defs. As we get incompatible changes in
rules we are going to see more of this.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazelisk/pull/125?email_source=notifications&email_token=AAXHHHCLEKFKHXH4CQIBSMTRGY2IFA5CNFSM4LEWW4L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOLJHOA#issuecomment-597070776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXHHHEUNNYZP5EG2WAU2NTRGY2IFANCNFSM4LEWW4LQ
.

friendly ping


Support for automatically generating backreferences for SDOs

In pursuit of https://github.com/tc39/ecma262/issues/1950, specifically https://github.com/tc39/ecma262/issues/1950#issuecomment-636945295.
Marked as draft until I've tried it out on the full spec, with https://github.com/tc39/ecma262/issues/1950 done.

I don't see sdoMap.js in this diff.

Updated. I uploaded a build of ecma262 made using this PR here so you can see how it looks.

Updated. I uploaded a build of ecma262 made using this PR here so you can see how it looks.

I changed the styling of the productions in the menu header. I think it's acceptable this way, but I'd prefer the styling to be the same as in the main body (family, weight, italics). But other than subscripting the optionality mark as I mentioned above, I think that can wait until a follow-up PR.

I changed the styling of the productions in the menu header. I think it's acceptable this way, but I'd prefer the styling to be the same as in the main body (family, weight, italics). But other than subscripting the optionality mark as I mentioned above, I think that can wait until a follow-up PR.

I'd prefer the styling to be the same as in the main body

Eh, easy enough. I tightened the spacing slightly (vs the main document) and dropped the yellow coloring from the opt subscript, but otherwise it'll match.
Looks like this:

Or when hovering the production:

I'd prefer the styling to be the same as in the main body

Eh, easy enough. I tightened the spacing slightly (vs the main document) and dropped the yellow coloring from the opt subscript, but otherwise it'll match.
Looks like this:

Or when hovering the production:

Fixed the italics for terminals:

Fixed the italics for terminals:


Disable hide all menu if all tabs hidden

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

Hide all flows tab menu is enabled even when all tabs are hidden.
This PR tries to disable the menu when no tabs are shown.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage: 68.241%. Remained the same when pulling 66f9686e48e108e35d466092779e797b2b58336e on node-red-hitachi:fix-hide-flow-menu into a607ee90e0b0edd59c83adebaf7d9589af1e8dd2 on node-red:dev.


How to generate an indoor room without creating the whole apartment floor plan?

Steps to Reproduce
If this relates to running the codebase, please provide steps to reproduce:
python -m infinigen_examples.generate_indoors --seed 0 --task coarse --output_folder outputs/indoors/coarse -g fast_solve.gin singleroom.gin -p compose_indoors.terrain_enabled=False restrict_solving.restrict_parent_rooms=\[\"Bathroom\"\]

What version of the code were you using?
85db7b5e2c3815097847009e8a622fd7b2e89d9e
What command did you run?
python -m infinigen_examples.generate_indoors --seed 0 --task coarse --output_folder outputs/indoors/coarse -g fast_solve.gin singleroom.gin -p compose_indoors.terrain_enabled=False restrict_solving.restrict_parent_rooms=\[\"Bathroom\"\]

Additional context
I would like to know how to create only a single room without creating the complete floor plan? which options to disable? the above command populates only one room as intended but still it contains a full apartment (with the empty rooms).

There is no commandline config to achieve this currently. Generating the whole floorplan adds very little extra runtime, so if I were to add a commandline config I would just make it so that the other rooms get deleted after generating.
If you want to try implementing it yourself for the timebeing you would modify generate_indoors.py:
# existing code
solved_rooms = [
        state.objs[assignment[cu.variable_room]].obj
        for assignment in greedy.iterate_assignments(
            stages["on_floor_freestanding"], state, [cu.variable_room], limits
        )
    ]

# new code
delete_rooms = [os.obj for os in state.objs.values() if t.Semantics.Room in os.tags and os.obj not in solved_rooms]
butil.delete(delete_rooms)

Thank you very much!

The suggested change above is incomplete. I made a slight modification but the code still breaks later.
    # Delete unsolved rooms from the floor plan.
    delete_rooms = [os.obj for os in state.objs.values() if t.Semantics.Room in os.tags and os.obj not in solved_rooms]
    butil.delete(delete_rooms)
    ....

    # Find house bounds using only solved rooms.
    house_bbox = np.concatenate(
        [
            butil.bounds(obj)
            for obj in solver.get_bpy_objects(r.Domain({t.Semantics.Room}))
            if obj in solved_rooms
        ]
    )

This is where it breaks..
    p.run_stage("solve_medium", solve_medium, use_chance=False, default=state)
  File "/home/silkyar/Work/repos/infinigen/infinigen/core/util/pipeline.py", line 89, in run_stage
    ret = fn(*args, **kwargs)
  File "/home/silkyar/Work/repos/infinigen/infinigen_examples/generate_indoors.py", line 289, in solve_medium
    solve_stage_name("on_wall", "medium")
  File "/home/silkyar/Work/repos/infinigen/infinigen_examples/generate_indoors.py", line 196, in solve_stage_name
    solver.solve_objects(
  File "/home/silkyar/Work/repos/infinigen/blender/3.6/python/lib/python3.10/site-packages/gin/config.py", line 1605, in gin_wrapper
    utils.augment_exception_message_and_reraise(e, err_str)
  File "/home/silkyar/Work/repos/infinigen/blender/3.6/python/lib/python3.10/site-packages/gin/utils.py", line 41, in augment_exception_message_and_reraise
    raise proxy.with_traceback(exception.__traceback__) from None
  File "/home/silkyar/Work/repos/infinigen/blender/3.6/python/lib/python3.10/site-packages/gin/config.py", line 1582, in gin_wrapper
    return fn(*new_args, **new_kwargs)
  File "/home/silkyar/Work/repos/infinigen/infinigen/core/constraints/example_solver/solve.py", line 179, in solve_objects
    active_count = greedy.update_active_flags(self.state, var_assignments)
  File "/home/silkyar/Work/repos/infinigen/infinigen/core/constraints/example_solver/greedy/active_for_stage.py", line 106, in update_active_flags
    set_active(state, objkey, active)
  File "/home/silkyar/Work/repos/infinigen/infinigen/core/constraints/example_solver/greedy/active_for_stage.py", line 99, in set_active
    child.hide_viewport = not active
ReferenceError: StructRNA of type Object has been removed

cc @araistrick, any ideas how to resolve this?


Tests of various nxp boards fail after #78429

Describe the bug
Tests of various boards featuring NXP soc´s fail after #78429 was merged. I tested mm_feather, teensy41, mimxrt1060_evk, frdm_mcxc242 but there should be even more.
Inserting CONFIG_PLATFORM_SPECIFIC_INIT=y in the boards defconfig-file fixes it, but seems to be a deprecated option.
To Reproduce
./scripts/twister -c -b -p mm_feather -T samples/subsys/shell/shell_module
delivers at least 1 error.
Expected behavior
Tests must succeed.
Impact
PRs cannot be merged due to failing tests, e.g. #78417

Did you west update to get the latest NXP HAL

I think this has been fixed on main already: #79558

I think this has been fixed on main already: #79558

Thanks, #79558 solves the issue


Add support for hierarchical configuration management in db

Description
Rather than specifying the complete configuration for every specific combination of slack app and context property, it would be a lot easier to administer if we can specify configuration values and a "selector" to apply the value to, similar to CSS.
The available scopes include:
app: app_id, bot_name
slack message context: channel, team
Additional Information
No response

Testing completed.


Error: invalid option: --bundle-exec

I am getting this error Error: invalid option: --bundle-exec when I try to run the task. Here is my grunt task:
compass: {
  dist: {
    options: {
      config: 'config.rb',
      bundleExec: true
    }
  }
},

I am using bundler and have run bundle install. Compass 0.12.2 is specified and installed. Running Yosemite. The exact same setup seems to work perfectly fine on a number of other folks machines in our office. Any thoughts on where the error could be coming from?

Ripped everything out and started with fresh installs of node, bundler and this library and got it back.


Could you please add onestopny.com as a merchant on your site

Looking to add our services to monero. We do development and production in fashion for high end women's and men's clothing. Any designers looking to create their own line or even people wanting to custom make their own clothing can contact us.
Thanks!

Couldn't find any reference to Monero in your website. Do you already accept it as form of payment or are you going to do it in the future?
+merchant

Oh i don't have any reference was expecting any interested parties to call in. Is there some kind of standard seal or something that other merchants use to show they accept monero?

Something like this? https://www.themonera.art/2017/09/22/monero-promotional-graphics-badges-and-stickers-for-websites/

Thanks added it to the footer of our all our webpages.

Thank you. I added your website in #602
+in progress


⚠ Bofrost.at has degraded performance

In a4fac58, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 890 ms

Resolved: Bofrost.at performance has improved in 2a3381a.


Fixes for quicklogic_testsuite designs

This pull requests fixes some issues with the designs:

cavlc_top
Some IOs were not constrained which caused Symbiflow not to use free SDIOMUX pads
iir
Similar situation as with cavlc_top
sha256_fullfit
This design contains two files sha1.v and sha256_fullfit.v. Both files define a single top-level module and do not reference each other. This caused Yosys to choose sha1.v as the top which presumably is not the one that should be used (It has 64 IO ports). I removed that file from the design.

With this pull request the iir design should successfully pass. The other two still won't fit in the device.

Maciej, Shouldn't the io_place auto assign pads to SDIOMUX even if there is no pcf file provided.

@tpagarani So currently the decision whether an IO cell is BIDIR or SDIOMUX is made in Yosys when the quicklogic_iob plugin is run. Only constrained IOs are considered so only those can be transformed to SDIOMUX. That off course can be changed with some amount of work.


pull request for e2e test at 12/21/2016 09:56:25

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[CT-290] [Bug] logger not outputing colors to stdout when stdout is piped to another process

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
dbt run will output color as expected on Windows, MacOS and Linux.
dbt run | cat will not output color to stdout (on Windows, dbt run | out-host -paging, is closest equivalent to | cat afaik).
The log file in <dbt project>/logs/dbt.log does contain ANSI color codes.
Expected Behavior
Piping stdout to another process should not stop color output to stdout
Steps To Reproduce
Ensure you've got dbt profiles setup and make a new project if you don't already have one. This can be reproduced on a minimal, dbt init project as long as you've got a profile setup to use.
dbt run to see good and expected behavior.
dbt run | cat to see incorrect behavior. dbt run | out-host -paging on Windows.
Relevant log output

Environment
- OS: MacOS, Windows, Linux
- Python: Python 3.7, 3.9
- dbt: 1.0.3

What database are you using dbt with?
snowflake
Additional Context
I believe this is related to the changes done in https://github.com/dbt-labs/dbt-core/pull/4474

@varun-dc  Thanks for submitting this! Let's keep following up on this in your PR!


Uninstalling  gatekeeper-operato chart does not remove gatekeeper-postinstall-job

Describe the bug: (a clear and concise description of what the bug is)
Uninstalling  gatekeeper chart does not remove gatekeeper-postinstall-job
To Reproduce
Steps to reproduce the behavior:

Install gatekeeper-operator chart
helm uninstall gatekeeper-operator
Install gatekeeper-operator chart
Observe error:

│   Error: release gatekeeper-operator failed, and has been uninstalled due to atomic being set: failed post-install: warning: Hook post-install gatekeeper/templates/post-job.yaml failed: jobs.batch "gatekeeper-postinstall-job" already  │
│ exists

Workaround:
 k delete job gatekeeper-postinstall-job
job.batch "gatekeeper-postinstall-job" deleted

Expected behavior: (a clear and concise description of what you expected to happen)
Some post uninstall routine remove that job

@j-zimnowoda can this also be reported upstream please? and create a PR if you think this has prio (I don't think so). Otherwise put the issue in "On hold" and report what is being waited for.
@redkubes/devops Can we please always do this when the bug is in a 3rd party app/chart? Don't just only report a bug here in core.


MAINT: backports from 1.0.x

Backports of:

gh-7929
gh-7918
gh-7905
gh-7902
gh-7900
gh-7891

And skip a test that gave warnings from within Fortran code, see gh-7888.

Okay, all green so in they go.


Update existing datasets by adding more data

I think adding an endpoint in the API to batch update datasets would really help and this would also be useful for retraining classifiers without posting a new dataset each time.

The request is to enable updating of a dataset by adding more images without having to rebuild the whole database. There are some outstanding questions to be answered first:

Can you update a dataset which is being used by a model for training? Does LMDB even allow for it? Does Caffe?
How do you keep a history of which data was used to train a model if the dataset may change?

Maybe simplest solution is to use just .txt file with list of images? it also will save HDD space. Some Caffe forks allow this (not sure about official branch).
For example https://github.com/alexgkendall/caffe-segnet , but it used for semantic segmentation, not classification.
Here is input layer of type DenseImageData accepting .txt
https://github.com/alexgkendall/SegNet-Tutorial/blob/master/Models/segnet_basic_train.prototxt

I am trying to use segnet with DIGITS but when I train the model it throws
Traceback (most recent call last):
  File "digits/scheduler.py", line 512, in run_task
    task.run(resources)
  File "digits/task.py", line 189, in run
    self.before_run()
  File "digits/model/tasks/caffe_train.py", line 220, in before_run
    self.save_files_generic()
  File "digits/model/tasks/caffe_train.py", line 632, in save_files_generic
    network = cleanedUpGenericNetwork(self.network)
  File "digits/model/tasks/caffe_train.py", line 1701, in cleanedUpGenericNetwork
    'Unsupported data layer type %s' % layer.type
AssertionError: Unsupported data layer type DenseImageData

Have you experienced it @mrgloom ?


i posted data sampler error, but that is not solved but closed...!

Bug description
https://github.com/Lightning-AI/lightning/issues/18391
plz check
What version are you seeing the problem on?
v2.0
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response

We don't need to open a duplicate issue. Let's continue in the original one.


Move to guidelines.readthedocs.[org|com]

Current URL is sooo ugly ;)

Hahaha, thanks for the note. We can get this done :)

This is now done ✨

Ironically, we have an ops bug that disallows us from hosting *.rtd.[org|com] URLs on the same domain as the actual application for historical reasons. So I used a .org domain for a project on .com infra, just make sure we still have some weird brokenness 👍

The .org redirects me to https://guidelines.readthedocs.io/en/latest/

@humitos works for me !?

▶ curl -I guidelines.readthedocs.org
HTTP/1.1 301 Moved Permanently
Date: Mon, 13 Feb 2023 10:14:35 GMT
Content-Type: text/html
Content-Length: 162
Connection: keep-alive
Location: http://guidelines.readthedocs.io/
X-Backend: web-i-06a0860f87c49d481
X-Served: Nginx-Domain-Redirect
CF-Cache-Status: DYNAMIC
Set-Cookie: __cf_bm=Ay1_RwLqvgUoPYjCvr424BIS1YE4Na0hFudE6N9x8V8-1676283275-0-AXgt/15F67wa1tdxTZzXo4qhBgHdiStSLWQeto6XaMmli74IFHyyts2duPDcIMiPkWmruH0Opon1Z7VDyodCt3I=; path=/; expires=Mon, 13-Feb-23 10:44:35 GMT; domain=.readthedocs.org; HttpOnly; SameSite=None
Server: cloudflare
CF-RAY: 798ccc447b87194c-BCN

@ericholscher is this OK?

@humitos That seems correct. The URL is https://brand-guidelines.readthedocs.org/

I'd probably remove this exceptional use-case and use https://brand-guidelines.readthedocs.io/ directly. This would avoid having exceptions in our infra.

It's hosted on readthedocs.com, so it's gonna be an exception regardless :)

It's a public repository, we can host it in readthedocs.org 😉

That's true, but I think it's fine for now, unless there's something else we need?