Tracking down mysterious linking errors in gb runs

Starting a brand new Go (1.6.2) project using gb for the first time. We have very simple code and tests in our project structure just to get our basic build environment operational. gb build and gb test both execute to completion and with expected results on an OS X machine and an Ubuntu machine. However, when we run gb on our Continuous Integration machine (Ubuntu) we see the following link errors in the build and test runs.
gb info output looks fine. It appears we're missing something essential in the CI machine's environment. We're at a loss as to what it is we're missing. Any help appreciated.
# utilities/testmain
_testmain.go:6: can't find import: "os"
/tmp/gb930956108/runtime/cgo/_cgo/gcc_libinit.o: In function `_cgo_wait_runtime_init_done':
/teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_libinit.c:30: multiple definition of `_cgo_wait_runtime_init_done'
/tmp/gb930956108/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb930956108/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb930956108/runtime/cgo/_cgo/gcc_linux_amd64.o: In function `threadentry':
/teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_linux_amd64.c:101: undefined reference to `crosscall_amd64'
collect2: error: ld returned 1 exit status
# runtime/cgo
# clock/testmain
_testmain.go:6: can't find import: "os"
FATAL: command "test" failed: exit status 1

unicode
/tmp/gb710488638/runtime/cgo/_cgo/gcc_libinit.o: In function `_cgo_wait_runtime_init_done':
/teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_libinit.c:30: multiple definition of `_cgo_wait_runtime_init_done'
/tmp/gb710488638/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb710488638/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb710488638/runtime/cgo/_cgo/gcc_linux_amd64.o: In function `threadentry':
/teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_linux_amd64.c:101: undefined reference to `crosscall_amd64'
collect2: error: ld returned 1 exit status
# runtime/cgo
FATAL: command "build" failed: exit status 1

This error could be caused by a mismatch between the version of go that is
on the machine and the version of Go that built gb.
On Thu, 28 Apr 2016, 08:01 Mike Karlesky<EMAIL_ADDRESS>wrote:

Starting a brand new Go (1.6.2) project using gb for the first time. We
have very simple code and tests in our project structure just to get our
basic build environment operational. gb build and gb test both execute
fine on an OS X machine and an Ubuntu machine. However, when we run gb on
our Continuous Integration machine (Ubuntu) we see the following link
errors in the build and test runs. gb info output looks fine. It appears
we're missing something essential in the CI machine's environment. We're at
a loss as to what it is we're missing. Any help appreciated.
utilities/testmain
_testmain.go:6: can't find import: "os"
/tmp/gb930956108/runtime/cgo/_cgo/gcc_libinit.o: In function _cgo_wait_runtime_init_done': /teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_libinit.c:30: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb930956108/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb930956108/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb930956108/runtime/cgo/_cgo/gcc_linux_amd64.o: In function threadentry': /teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_linux_amd64.c:101: undefined reference to crosscall_amd64'
collect2: error: ld returned 1 exit status
runtime/cgo
clock/testmain
_testmain.go:6: can't find import: "os"
FATAL: command "test" failed: exit status 1
unicode
/tmp/gb710488638/runtime/cgo/_cgo/gcc_libinit.o: In function _cgo_wait_runtime_init_done': /teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_libinit.c:30: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb710488638/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb710488638/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb710488638/runtime/cgo/_cgo/gcc_linux_amd64.o: In function threadentry': /teamcity/TeamCity/buildAgent/work/634a865bbd200e73/tools/linux/go-1.6.2/src/runtime/cgo/gcc_linux_amd64.c:101: undefined reference to crosscall_amd64'
collect2: error: ld returned 1 exit status
runtime/cgo
FATAL: command "build" failed: exit status 1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/591

Thanks for the quick response. We've verified that everything is Go 1.6.2 all the way around. From your comment we decided to try an experiment. When gb was built on machine A (a dev laptop) our build environment ran to completion only on machine A. We just rebuilt gb on machine B (the CI server). Now our build environment runs successfully on both machine A and machine B. Most curious, but it would seem we're on our way. Let's hope the environment is replicable. We can close this issue for now. Thank you again.

I'm glad you found the answer. The underlying reason is gb needs to know
the path of the go installation it was built by so it can find the stdlib.
This makes it complicated to distribute a binary version of gb as we'd also
have to tell people to set something like GOROOT, which is a step in the
opposite direction, so for the moment, when you change your Go install, you
need to rebuild gb with that new install.
At a guess, your Ci server had an older version of Go in something like
/usr/local/go and that didn't match the newer version, also in
/usr/local/go that you built gb with locally.
On Thu, Apr 28, 2016 at 9:48 AM, Mike Karlesky<EMAIL_ADDRESS>wrote:

Thanks for the quick response. We've verified that everything is Go 1.6.2
all the way around. From your comment we decided to try an experiment. When
gb was built on machine A (a dev laptop) our build environment ran to
completion only on machine A. We just rebuilt gb on machine B (the CI
server). Now our build environment runs successfully on both machine A and
machine B. Most curious, but it would seem we're on our way. Let's hope the
environment is replicable. We can close this issue for now. Thank you again.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/591#issuecomment-215266256

Assuming the same Go version but located in different places on different machines, can you sketch out a procedure or recommendations on how to best build gb to work as a binary distribution across multiple dev machines and a CI server? The current build of gb we have is working for us, but I can't say that we fully understand how we arrived at success. I'd like to try some further experiments / document this as we will soon be adding one or more devs to our project team. Any thoughts appreciated.


Add index view

Create index view providing the central dashboard of the application

Will create a Django app fseco to contain all model and view code


Duplicating ROIs ( like LengthTool)

Hi All,
I am developing an application, where I need to show images in different layout ( Viewport Size ). Suppose I have a layout of 1X2 and loaded the same image to Port 1 and Port 2.
If I have placed an ROI in the Port 1. It will automatically Placed in the Port 2.
I have changed the imageID of the of the element in both port.
Port1 image have ImageID dicomImage_1 and port 2 image have image ID dicomImage_2.
And also I can see the same Data for LengthTool in the toolState  (toolStateManger.toolState) of each element .
Please help me to resolve this issue.
Thanks,
Biju Balan

@BIJUBALAN83, to make sure I understand, you have drawn a length annotation on one image, and would like to "copy" its state and have it drawn on the second image as well?
Do you have any requirements around what would happen if either viewport updates the annotation, or if either viewport is scrolled or loads a new stack/displaySet?


Support for NATS as a subscriptions backplane

NATS GraphQL Subscriptions backplane for HotChocolate
What is NATS?
NATS is a simple, secure and high performance open source messaging system for cloud native applications, IoT messaging, and microservices architectures. It offers services for both asynchronous communication and streaming data.
Why do I need NATS for my GraphQL Subscriptions?
If you scale your HotChocolate servers beyond a single node, some of your GraphQL Subscriptions will be handled by different servers. The subscriber may receive the subscription results on a different server than the publisher. In order to make this work, you need a backplane. Another example of a backplane is the Redis backplane that is included in HotChocolate. The Redis backplane is a good choice if you want to reuse some of your existing infrastructure. However, if you want to avoid the overhead of a Redis server, you can use the NATS backplane.
The NATS backplane is a simple and lightweight alternative to the Redis backplane. It is also typically faster than the Redis backplane, and cheaper, as you can run it on lightweight hardware, virtual or not, and scale it horizontally. The Publish/Subscribe pattern does not require a lot of memory, and Redis typically scales in a vertical fashion, adding more memory to a single server. NATS scales horizontally, adding more servers to the cluster.
How do I use the NATS backplane?
You can start with a single node of NATS and see where you need to go from there. You can also start with a cluster of NATS servers and scale it. The NATS client uses a technique called client-side load balancing to distribute the load across the cluster. The NATS client will automatically discover the cluster and choose the correct server to publish to or subscribe from, based on a hash of the subject. The NATS client will also automatically reconnect to the cluster if a server goes down.
You do not need to enable persistence in the NATS server (JetStream) for Publish/Subscribe to function.
using AlterNats;
using HotChocolate.Execution;
using HotChocolate.Subscriptions;

var builder = WebApplication.CreateBuilder(args);

builder.Services
    .AddNats(poolSize: 1, opts => opts with
    {
        Url = "nats://localhost:4222",
        // Optional serializer (defaults to System.Text.Json)
        Serializer = new MessagePackNatsSerializer()
    })
    .AddNatsSubscriptions()
    .AddGraphQLServer()
    .AddMutationConventions()
    .AddQueryType<Query>()
    .AddMutationType<Mutation>()
    .AddSubscriptionType<Subscription>();

var app = builder.Build();

app.UseWebSockets();
app.UseRouting();
app.UseEndpoints(ep =>
{
    ep.MapGraphQL();
});

await app.RunAsync();

NOTE:
You need to target net6.0 or higher to use this provider, as the NATS client dependency is not available for net5.0 or lower.

Multiple Environment Support
If you wish to share a NATS instance/cluster with multiple, distinct GraphQL servers, you must provide a unique subject prefix for this instance, e.g. an environment string like "dev01". This prefix is passed in the initial setup, like so: AddNatsSubscription(prefix: "dev01"). All servers with the same prefix will be part of the same publish/subscribe group, and will receive all messages. The prefix will be prepended to all NATS subjects with the "." subject token separator, and as such can be monitored with the NATS CLI using the > operator:
./nats -s localhost:4222 subscribe "dev01.>"

If you do not provide a prefix, the server will use the default prefix of "graphql". Only a-z, A-Z and 0-9 characters are permitted. The prefix is case sensitive.
Technical Information about NATS
NATS has a nice command line interface which you can use to monitor the cluster. You can also use the NATS dashboard to monitor the cluster.
NATS PubSub overview using NATS CLI
https://www.youtube.com/watch?v=jLTVhP08Tq0

Is there a docker image with this available?

https://hub.docker.com/_/nats

We need to create a squadron package for this so we can test this thing.

We need to create a squadron package for this so we can test this thing.

Yeah, I looked to see what Squadron was, and it seems there is no OOTB NATS package - if this is something that is expected to be written by us* (i.e. could be me?) then let me know where I could start.

Waiting on https://github.com/SwissLife-OSS/squadron/pull/123

Uncovered a race condition in my NatsPubSub implementation -- marking as draft until squadron tests can prove otherwise

@PascalSenn @michaelstaib - ready to review now, but there's still an issue in core with net7-rc1

Let me check

Added multiple environment support:

updated readme.md
updated top comment above

Multiple Environment Support
If you wish to share a NATS instance/cluster with multiple, distinct GraphQL servers, you must provide a unique subject prefix for this instance, e.g. and environment string like "dev01". All servers with the same prefix will be part of the same publish/subscribe group, and will receive all messages. The prefix will be prepended to all NATS subjects with the "." subject token separator, and as such can be monitored with the NATS CLI using the > operator:
./nats -s localhost:4222 subscribe "dev01.>"

If you do not provide a prefix, the server will use the default prefix of "graphql". Only a-z, A-Z and 0-9 characters are permitted. The prefix is case sensitive.

Before we can move forward on this we have to solve the connection issue that we have on the local dev system. I have started investigating.

Pushed new updates to deal with topics that are complex objects; previously I was assuming topics would only be strings. Topics are now converted into a 128-bit hash of a MessagePack serialized byte stream. This also happens for plain strings, as they may contain characters that are not valid for a NATS subject string. Main changes:

All messages are now wrapped in a NatsMessageEnvelope, which has a field to convey the type of message: a fully body/event, or a terminator message to signal to end of the stream. This is more reliable than a magic string.
The hashing algorithm being used is SpookyHash, an updated version of the Jenkins Hash algorithm; which is a non-cryptographic has designed for speed and extreme collision avoidance. Crypto-based hashing is slow, and has a different threat model.

We will be waiting until https://github.com/Cysharp/AlterNats/issues/23 is resolved.

@PascalSenn @michaelstaib This should be unblocked now, but you'll need to publish a new squadron package with my fix and then we can update this PR to use it
https://github.com/SwissLife-OSS/squadron/pull/126

@PascalSenn @michaelstaib This should be unblocked now, but you'll need to publish a new squadron package with my fix and then we can update this PR to use it
SwissLife-OSS/squadron#126

Squadron published this morning 18-preview.4 with the osx fix. I've just merged into this PR. We're officially ready to go.


Update Operational Layer Content

In an AGO Webmap with a large number of layers, updating Field order for popups and popup content is cumbersome through the AGO interface. We have gotten around this by customizing webmaps in a WebMap JSON file. However, then we can no longer use the AGO webmap which is now out of date.
It would be great to be able to edit the JSON manually, or be able to update the content for multiple similar layers.

Editing JSON is now available (added in #64). Let me know if it works out for your use case.


getKeys

Hello,
Is it possible to get keys ?
Has example of use, I have a list of
And for each li I have a key so I'd like loop on it. But for that I need access keys of json translation files.

It's easy:
translate.getTranslations('en').subscribe((translations: any) => {
    console.log(Object.keys(translations));
});


OpenDAL Cpp Binding

OpenDAL already has a C binding that can be used to integrate Cpp projects. However, its user experience falls short of my expectations. I want Cpp projects like Clickhouse, Doris, DuckDB can integrate OpenDAL with native Cpp binding (for namespace, morden async, modules and so on).
As discussed in https://github.com/apache/incubator-opendal/discussions/1897, we can use cxx for this issue.

Is there any progress on it? If possible, I'd like to have a try.

Is there any progress on it? If possible, I'd like to have a try.

No progress so far, welcome to take a try!

#2980 introduced basic cpp binding. Now the development tracking issue is on #2981

Let's close this idea issue.


[Chore] Avoid unwanted imports from users

Some components of the library can be still imported by users even if they are not exported from index.ts file.
This has to deal with the bundle creation process and it's something that needs to be fixed.

Closing as no longer required


check that unrolled polybench benchmarks work with icarus-verilog

These benchmarks: https://github.com/cucapra/calyx-evaluation/tree/master/benchmarks/unrolled
Use the runt file in the in the directory above

These simulate correctly with iverilog.


Specify in the documentation that the FQDN that is assigned when you create a private endpoint is globally unique

Hi there,
Are we able to specify in the documentation that the FQDN that is assigned when you create a private endpoint is globally unique? I contacted MS support and they advised to me that it is globally unique per private endpoint. However I could not find this documented anywhere, and he failed to show me where this is documented anywhere.
I know that it is easily assumed, however we need to make decisions based on documentation at our company.
More information:
We are trying to implement the Private Link and DNS integration at scale Cloud Adoption Framework solution and we need to ensure that the application owners can not overwrite each others DNS records. We can assume that each FQDN is globally unique however without documentation we cannot know this for sure.
Kind regards,
Tully

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

ID: a88af182-32aa-a8f3-359d-b92ae1cfb8c7
Version Independent ID: 01ce07fc-4bc2-def5-9ecd-a80330ad9488
Content: Azure Private Endpoint DNS configuration
Content Source: articles/private-link/private-endpoint-dns.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@tullydwyer , thank you for your feedback. We'll review this and get back to you shortly!

@tullydwyer , could you please provide the support request number for your conversation to validate the information on our end?

Hi @GitaraniSharma-MSFT, I'm not comfortable posting that number in a public GitHub issue (company tenant).
If there is any private way I can send it I will

@tullydwyer , please send an email to<EMAIL_ADDRESS>referencing this Github issue and the support request number. Mention "ATTN gishar" in the subject field.

@tullydwyer , I'm following up on my above comment. Could you please send the requested information over email?

@tullydwyer , Since we have not heard back from you we will now proceed to close this thread. Please do let us know once you email us the support request number and we will reopen the issue for further investigation.

@GitaraniSharma-MSFT, sorry I was slow! but I have just emailed the information with the following subject: "ATTN gishar - Github Issue #78380 tullydwyer"
Thanks,
Tully


fix: Remove throws from TrapManager init

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Files
Coverage Δ

Sources/Trap/Manager/Manager.swift
82.80% <100.00%> (ø)

:loudspeaker: Thoughts on this report? Let us know!


Glsl occlusion

implement occlusion removal on gpu with glsl

Great job @nohayassin @gwen2018
I'm testing GPU version and see that the algorithm is too aggressive - invalidates also points from  foreground. Not sure why yet...

@gwen2018 - found the issue:
if (uv[i].x > uvmax)\n" needs to be if (uv[i].x > uvmax && xyz[i].z > 0.0)\n"


Update sbt to 1.3.12

Updates org.scala-sbt:sbt from 1.3.10 to 1.3.12.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Pull Request Test Coverage Report for Build 69

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 87.302%

Totals

Change from base Build 66:
-0.4%

Covered Lines:
55

Relevant Lines:
63

💛  - Coveralls


Allow custom data injection for master and agent VMs

We do have a few requirements in addition to vanilla DC/OS (mostly around CA injection and some management tooling) that we would like to automate into the provisioning cycle.
I have seen the /parts folder having the dcoscustomdata{n}.t files which I think could be a great point to inject things.
Is there any plan to make this available via acs-engine and template-ise this further?
If so, here would be my suggestion:

add {{ .customRunCommand }} into the runcmd section
add {{ .customFiles }} into the write_files section
add a -customRunCommand and -customFiles arg to acs-engine expecting correctly formatted files
merge the files together on generation
profit

Thoughts? I need this soon so might be able to get a PR going for this

customData/cloud-init does not reexecute either though...

oh. I only used cloud-config via CoreOS where that's actually the case.
Well in that case extensions are the way to handle this. Thank you! 🥇


Hosting docker container from r-dev-env repo?

The development environment currently downloads an image hosted in another github repository. I wonder if it is possible to host the dockerfile from the r-dev-env repo instead. This could use a conditional workflow where the commit message includes 'build docker image'.
Something for me to look at in the future.

Exploration showed this to be possible. The dev-container branch of this repo now build the development container from a dockerfile. The image can be downloaded via docker.
docker pull ghcr.io/r-devel/r-dev-env:dev-container
See https://github.com/r-devel/r-dev-env/pkgs/container/r-dev-env


Add Overseas AO Providers

Trello card
N/A
Context
We've had a request from international colleagues to include a new set of AO providers that are overseas providers of Assessment Only. Adding these in.

Hi Pete can you please add last 2 paragraphs into the grey box for international assessment only providers. Thanks

I've had to adjust the assessment only provider structure a bit to accommodate a description key that can point to any partial. This allows us to add a bit of extra descriptive text for groups should we need to, for example:

Mobile
Desktop

Review apps appear to have ceased. Reopening.


TextField readonly does not work when referencing a variable defined by a checkbox

TextField readonly does not work when referencing a variable defined by a checkbox.
Example:
Expanded(
  child: FormBuilderTextField(
    attribute: 'textField',
    controller: textFieldController,
    readonly: _disableTextField,  // This does not work
    decoration: InputDecoration(
      labelText: _disableTextField ? 'Disabled' : 'Enabled', // This works
    ),
  )
),
Expanded(
  child: FormBuilderCheckbox(
    attribute: 'checkBox',
    initialValue: false,
    onChanged: (val) {
      setState(() {
        if (val) {
          textFieldController.clear();
          _disableTextField= true;
        } else {
          _disableTextField= false;
        }
      });
    },
  )
)
In the example above, label switching works, but deactivation of the field does not work
Need help. Thanks in advance

Same applies for FormBuilderRadio

I being trying to debug the problem, and moving the line:
    _readonly = (_formState?.readonly == true) ? true : widget.readonly;
from method initState() to the first line of build fix the issue

Hi @Tibasa.
Try out version 3.2.3 of the plugin which works as expected thanks to the good work by @dhaalves

Thanks very much!!!


[core] Several functions allocate memory inside raylib but do not provide functions to deallocate it

Functions like GetImageData allocate memory inside Raylib but do not provide any functions to deallocate the memory.
Currently the only options are to leak the memory, or free it from within the client code.
On Windows when using Raylib as a dll calling free from the client code will cause a debug assert because the memory was allocated from the DLL module and must be deallocated in the same module. There needs to be a function that causes raylib to free the memory from within the module itself.
This is only a problem on Windows (Visual Studio 2019) when using raylib as a DLL. Static linking does not have this problem, but static linking is not viable in a real windows subssytem application because of name conflicts with some raylib functions (CloseWindow, ShowCursor, etc...)
The specific assert is
**
File: minkernel\crts\ucrt\src\appcrt\heap\debug_heap.cpp
Line: 996
Expression: __acrt_first_block == header
**
Code Example
The simplest way to see the problem is to build an application that links raylib as Debug.DLL and call code similar to this.
Color* imageData = GetImageData(imagePath);
// read image data or whatever.
free(imageData)

I reviewed all raylib functions and here it is a list with the functions allocating memory:

Function with MEM_ALLOC
Function to MEM_FREE
Possible solutions

LoadFileData()
❗
IDEA: UnloadFileData() / MemFree()

LoadFileText()
❗
IDEA: UnloadFileData() / MemFree()

GetDirectoryFiles()
✔ ClearDirectoryFiles()
Using Get/Clear convention

GetDroppedFiles()
✔ ClearDroppedFiles()
Using Get/Clear convention

CompressData()
❗
USE: MemFree()?

DecompressData()
❗
USE: MemFree()?

GetImageData()
❗
IDEA: LoadImageColors() / UnloadImageColors()?

GetImagePalette()
❗
IDEA: LoadImagePalette() / UnloadImagePalette()?

GetImageDataNormalized()
not any more
REMOVED: https://github.com/raysan5/raylib/commit/d6f4f3ee88a1eaa77acba4ae9aab7ed2e52049bf

LoadFontData()
✔ UnloadFontData()
ADDED: https://github.com/raysan5/raylib/commit/14c1ee268107b2de727c2820e56a333b85ebc500

TextReplace()
❗
IDEA: TextFree() / MemFree()

TextInsert()
❗
IDEA: TextFree() / MemFree()

TextToUtf8()
❗
IDEA: TextFree() / MemFree()

GetWaveData()
❗
IDEA: LoadWaveSamples()/UnloadWaveSamples()?

It seems for most cases providing a MemFree() function should be enough but in that case it's up to the user to do a right use of the function over the raylib internal-allocated memory. Adding some TRACELOG info message could be useful in some cases...
Alternatively, providing a Load*()/Unload*() functions pairs could seem more intuitive to novel users.

In my option keeping parity with load/unload would be best, it is excessive, but consistency in the API is better for users in this case.

Two additional functions have been exposed (MemAlloc() / MemFree()) for advance users requiring access to the library internal allocation mechanisms.


datovka, libisds: init at 4.12.0 and 0.10.8; added maintainer cptMikky

Motivation for this change
Add new package Datovka (https://www.datovka.cz) useful to anyone who has to communicate with Czech government via Databoxes.
Add new pacakge libisds (http://xpisar.wz.cz/libisds/) which Datovka depends on.
Add new maintainer, @cptMikky .
Things done

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

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

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

Just a preview, Datovka doesn't build.
g++ -c -g -O0 -std=c++11 -Wall -Wextra -pedantic -Wdate-time -Wformat -Werror=format-security -O2 -Wall -W -D_REENTRANT -fPIC -DDEBUG=1 -DVERSION=\"4.12.0\" -DDATADIR=\"/nix/store/7gc3dryzi5xw41b42gvi5sp9d8285ndz-datovka-4.12.0/share\" -DPKGDATADIR=\"\" -DLOCALE_INST_DIR="\"/nix/store/7gc3dryzi5xw41b42gvi5sp9d8285ndz-datovka-4.12.0/share/datovka/localisations\"" -DTEXT_FILES_INST_DIR="\"/nix/store/7gc3dryzi5xw41b42gvi5sp9d8285ndz-datovka-4.12.0/share/doc/datovka\"" -DQT_NO_DEBUG -DQT_SVG_LIB -DQT_PRINTSUPPORT_LIB -DQT_WIDGETS_LIB -DQT_GUI_LIB -DQT_NETWORK_LIB -DQT_SQL_LIB -DQT_CORE_LIB -I. -I/usr/include/libxml2 -isystem /nix/store/ywqn919y4lnmfy2dpk5zx1bd6pkkv9dg-qtsvg-5.12.0-dev/include/QtSvg -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtPrintSupport -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtWidgets -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtGui -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtNetwork -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtSql -isystem /nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/include/QtCore -Igen_moc -I/nix/store/vl9ma114n5xh3q7p9vjhl1p7vzg3vp1r-mesa-noglu-18.3.4-dev/include -Igen_ui -I/nix/store/9ghnbhx3qwdawci3p24yc4axxblwjhbw-qtbase-5.12.0-dev/mkspecs/linux-g++ -o gen_objects/src/cli/cli_parser.o src/cli/cli_parser.cpp
In file included from src/about.cpp:32:0:
/nix/store/cvx6vfnl6wc89l5cigvz3rlvh85rbb90-libisds-0.10.8/include/isds.h:10:10: fatal error: libxml/tree.h: No such file or directory
 #include <libxml/tree.h>    /* For xmlDoc and xmlNodePtr */
          ^~~~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:2225: gen_objects/src/about.o] Error 1
make: *** Waiting for unfinished jobs....
builder for '/nix/store/gkvm2g4hv9svvi901va7gvhins2r2p88-datovka-4.12.0.drv' failed with exit code 2

Updated, now builds and runs.

Should I squash the changes? Or maybe separate them to three commits (add maintainer, add libisds, add datovka)? Or should I just leave the PR as it is?

Three commits would be good. Also, please format them to match https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md

@cptMikky any update on this pull request?

Closing due to inactivity, feel free to re-open this pull request.


apr_jwe_decrypt_content_aesgcm() fails to null terminate string

The test test_jwt_decrypt_gcm() in test.c would sometimes fail. The
failure was caused by extra garbage appearing in the decrypted string
after the expected string. This is due to the failure of
apr_jwe_decrypt_content_aesgcm() to null terminate the string as is
similarity done in apr_jwe_decrypt_content_aescbc().
There is a patch and pull request to fix this: https://github.com/pingidentity/mod_auth_openidc/pull/127

thanks for reporting and fixing; I will review and merge asap


crash  at main.c:230

Starting program: /tmp/c-jwt-cracker/jwtcrack eyJhbGciOiJuIiwiZCI6IkpEZ0xjUnNaRDNFSUt5NG1KUWtMY0JzWmNYdz0ifQ
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
0x000055555555567f in main (argc=<optimised out>, argv=<optimised out>) at main.c:230
230		g_payload_b64_len = strlen(g_payload_b64);
(gdb) bt
#0  0x000055555555567f in main (argc=<optimised out>, argv=<optimised out>) at main.c:230
(gdb)

I meet it too. This is a null pointer reference segment fault. I think jwtcracker need more exception handle code. But this is just a cmd line tool, so it is ok to me.


[1.3] TlsCredentialsOption not getting enabled

With the updates to protobuf (to 3.14) and grpc-cpp (to 1.36.4) on the main branches, the TlsCredentialsOptions in arrow-cpp are not getting enabled:
Build log:
-- Checking support for TlsCredentialsOptions...
-- TlsCredentialsOptions not found in grpc::experimental.
-- TlsCredentialsOptions not found in grpc_impl::experimental.

Investigate why. Things have likely moved around.
Note there is a build option in arrow called HAS_GRPC_132 that we may need.

I created PR 18 with the fix. More details added to the PR.

Fixed in #18


cargo-lipo doesn't work with workspaces

I have a project which defines a cargo workspace as per http://doc.crates.io/manifest.html#the-workspace-section
I'd like to invoke cargo lipo on one of the members of my workspace (it's a sub-crate exposing C bindings for the main crate), but both cargo lipo -p MEMBER or cargo lipo --all are missing.
Is there any plan to make cargo-lipo support workspaces?

Hi @poros,
there haven’t been any plans until now ;)
I can look into it over the next 1 - 2 weeks, however if you want to start an implementation, I’ll gladly accept a pull request (just comment here when you start work, so we don’t duplicate work).
Please ping me if you haven’t heard back in ~2 weeks, that means this issue has probably dropped off my radar (I’m not currently actively using cargo-lipo).

Ping ;)

Just ran into this problem myself. :)

I'm getting this when trying to install the latest beta (which strangely enough is listed on crates.io) via cargo install:
cargo install cargo-lipo "2.0.0-beta-3"
    Updating registry `https://github.com/rust-lang/crates.io-index`
  Installing cargo-lipo v1.0.3
   <snip>
   Compiling cargo-lipo v1.0.3
    Finished release [optimized] target(s) in 22.96s
  Installing <snip>/.cargo/bin/cargo-lipo
error: could not find `2.0.0-beta-3` in registry `https://github.com/rust-lang/crates.io-index`
     Summary Successfully installed cargo-lipo! Failed to install 2.0.0-beta-3 (see error(s) above).

Any idea what's going wrong here? 🤔

I think the syntax you want is cargo install --version 2.0.0-beta-3 cargo-lipo, now that you already installed the older version probably with --force as well. (Your command is attempting to install a crate called "2.0.0-beta-3", I believe.

That fixed it, thanks!

I've released a 2.0.0 which includes the workspace changes, so I think this issue can be closed.


AoCF support

List of issues that needs to be solved for full AoCF support

[ ] #70 Subtitles patching
[ ] #71 Move and resize balloons
[ ] #72 Skipgame patch
[ ] #73 Missing translations in config screen
[ ] #74 Left-align the menu settings

Projects are better suited for this - moved to https://github.com/thpatch/thcrap/projects/2


[Candidate 3v3 Nocky Mountains ZH v2] Tractor on right side is car bomb

Observed behaviour
Tractor on right side is car bomb. On left side there is no tractor.
Expected behavior
Tractor is removed.
Screenshots

fixed its not passible to enter anymore

I suggest to remove it or put same tractor on the mirrored field on the other side, because it can still affect gameplay and be a rare hazard to infantry units as is seen here:
https://github.com/TheSuperHackers/GeneralsRankedMaps/assets/4720891/b67a7fd2-aec4-4589-add4-cb4d5adf5b5e

lol ok


No TouchBar Support in High Sierra

Running MacOS High Sierra currently, public download. I have lost the playing music toggle with the playhead and pause, play, skip icons etc.

Macbook Pro 2017 13' on 10.13.0

To replicate:

Play any song and enter any other app
The tiny playing music icon does not show in the Touch Bar, thus shortcuts can't be accessed.

@Colecperrine I haven't updated my mac to High Sierra yet so I can't debug this for a while.  (I normally wait at least a month after public release for things to stabilize)

Just tested this on a friends macbook with High Sierra and the touch bar stuff works fine.  I'd suggest giving your mac a quick reboot to see if that fixes it.

I am having the same problem with version 4.4.1 running on macOS High Sierra 10.13. Rebooting does not fix the issue. I also tried uninstalling and reinstalling the app, but that didn't help either.

Same here, I got it to work for a few minutes but it went away immediately. The "Now Playing" item in notification center works for play, pause, and skip, but it doesn't show a timestamp and also says "undefined" for any metadata. Rebooting doesn't help.

Update: The issue doesn't appear to happen on 10.13.1, it may have just been an error with 10.13.0 on the Macbook 13", any similar results from anyone else?

It's still not working for me on 10.13.1, unfortunately. ☹

Lost it again, seems to only work on the first try after rebooting.

When I play music in the app and open Notification Center, the Now Playing widget says "Undefined" as seen in this screenshot:

I suspect the touch bar issue is a side effect of a bug in the Now Playing feature. Should I create a new issue for this @MarshallOfSound?

Not trying to re-open a semi-oldish issue here, but I have a comment on this. Unfocused, seems to be working fine. Focused, it has no controls on the touch bar. I can use spacebar and arrow keys, but maybe it would be nice to have that bar present when the app is focused?
Thanks!


Bubble chart filtering

Allow filtering within the Bubble chart

what kind of filtering do you want? we have filtering using the checkbox next to the app in the datatable list to the left of the bubble chart. that list is also orderable, so you can push the high business criticality apps to the top before checking the boxes, or any other field that the datatable shows.

I think what we have is now OK as we have changed this since writing this requirement


added new parameters in options as payload which can be helpful to pa…

ability to pass extra data as payload in options so that can be useful for developers to get more detail about the vue-dialog

@javisperez

Hey @msonowal! Sorry for the delay didn't notice there was an open PR, my bad 😅 gonna check it later today and merge asap. Thank you a lot for the help!

@javisperez  sure


burr gives wrong mean() and expect()

My issue is about ...
The burr distribution does not produce the correct mean() and expect() values.
Reproducing code example:
>>> from scipy.stats import burr
>>> f_burr = burr(0.94839, 4.382028)
>>> f_burr.mean()
-90.8450580569245
>>> f_burr.expect()
/Users/rlucas/anaconda3/envs/scipy-dev/lib/python3.6/site-packages/scipy-1.2.0.dev0+6683b5f-py3.6-macosx-10.7-x86_64.egg/scipy/integrate/quadpack.py:385: IntegrationWarning: The integral is probably divergent, or slowly convergent.
  warnings.warn(msg, IntegrationWarning)
-90.8450580500504

Error message:
In my dev environment, the first time the .expect() is called I get
IntegrationWarning: ...

Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.2.0.dev0+6683b5f 1.15.4 sys.version_info(major=3, minor=6, micro=7, releaselevel='final', serial=0)

This may be a known issue given that these parameter values are in the source code.
at tests/test_continuous_basic.py LIC 45. Does anyone know or remember anything about this issue?

the var() method also gives non-sensical results
f_burr.var()
-7997.5681215257

cdf appears to be fine:

f_burr.cdf(0)
0.0
import numpy as np
f_burr.cdf(np.inf)
1.0
f_burr.cdf(10)
0.6264828708111116
f_burr.cdf(1)
0.04795988467974732
f_burr.cdf(3)
0.2660479841928383
f_burr.cdf(50)
0.8994637244282145
f_burr.cdf(500)
0.9880109529403922
f_burr.cdf(50000)
0.9998468284412179
f_burr.cdf(5000000)
0.9999980571535964
f_burr.cdf(5000000000)
0.9999999972249616

For c <= 1, the mean of the Burr Type III distribution doesn't exist.  The distribution is also known as the Dagum distribution, and you can see the formula for the mean on the Dagum distribution wikipedia page.  (The parameter a on the wikipedia page is the parameter c in the SciPy implementation.)
This leads to a natural enhancement request for burr: use the explicit formula for the mean when computing the default value of expect().  (And as long as someone is editing that part of the code: make a note in the burr docstring that the distribution is also known as the Dagum distribution.)

that is is a special case of Dagum (w/b=1

b is the scale, and burr has a scale parameter, so technically, the Dagum distribution on the wikipedia page is a special case of burr with loc=0. 😃

expect() is numerical approximation whereas mean() is exact.

But when func is not given to expect(), it computes the mean.  Since we know we are computing the mean in that case, and we have an explicit formula for it, it is silly to not use it.  I don't see any problem with adding a note explaining this in the expect() docstring, and implementing it wherever we can.  (I'm not saying you have to do this if you create a pull request to enhance the implementation of burr, but it would be nice if eventually such an enhancement to expect is made to burr and other distributions where appropriate.)

just in general about parameter values like burr(0.94839, 4.382028).
The parameters in the unit tests where initially created by random numbers until I found some that "worked". There was not enough time and often no (accessible) references to check each individual distribution specifically.
It would be better if those are replaced by "good" parameters, if they are still the old random parameters, and have, for example, infinite or non-existent moments.

Thanks for the info @josef-pkt super helpful. I'm working on a PR, should be out within another day or 2, I'll add you and Warren to the PR.

https://github.com/scipy/scipy/pull/10381 was merged and that incorporated a changeset that addressed the issue(s) mentioned here, therefore closing. Thanks @pvanmulbregt @josef-pkt @WarrenWeckesser @ev-br


Allow  +  chords

Since I use Vim extension, would like to use the  +  chord as an alias for Esc, (as I did with with the help of the vimscript extension Arpeggio).
Currently, chords have to be in the form of:  +

Formatter ate my French bracketing, meant:
<Key> + <Key>
Only this seems to be allowed:
<Modifier> + <Key>


no suitable screen available

Hi,
running win7 64-bit, con-emu cmder (http://cmder.net)
thanks!

I don't know about win7.... but under win10, this works fine provided that I clear the $TERM variable.
The problem with $TERM being set is that in this case the emulation attempts to use XTerm style UNIX handling, instead of windows console handling.  Quite likely I should change the code to attempt to detect Windows first, and use the console API in preference.  The XTerm API doesn't work at all on Windows at present since the termios support I need is missing there.

that worked. thanks! tscreen.go set empty string @ LookupTerminfo line 37.
long term fix would be good but not sure how you want to detect windows.

Yeah, I have to be careful here.  Apparently there are some terminal emulators for Windows that are not true consoles... but use the xterm API.  (I think MSYS is one such.)

Adding a significant wrinkle into all this is WSL.  I need to figure how to detect when the console API is available, because that is always superior.  termios on Windows (absent WSL) do not exist, and so we can't have the getchar stuff that we need to do non-blocking reads.

This was fixed in 2fca0432ba2d854410ab0a63d7d6f3f9ce3a5e48  (back in March '19)


[AutoPR sql/resource-manager] Sql auditing swagger - add queueDelayMs property

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7759

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/6321


Get Documenter deployment of documentation working

Striking out trying to follow the documenter docs and trying to just let juliahub build and host. Ugghhh.

Sorry, but why are you using juliahub in the first instance ? You don't need it, just use the github pages.. you were very close, you had the documentation correctly built on  the gh-branch....

Yes, why? Because I've had no-muss success with other packages using JuliaHub (though not here) and for the life of me couldn't get gh-pages properly setup, despite following the instructions faithfully (though likely I skipped some step along the way). Maybe I'll give it another go, but yesterday I got impatient.

Have a look at my Tutorial and Dummy implementation
Adding the deploydoc call  in make.jl you were able to get the documentation built, as I saw it.
The last step was to go to the settings of your package and set the source to the gh-pages branch (see the GitHub own documentation on pages)
End at the next push the documentation will (well.. should..) be available to https://jverzani.github.io//QuizQuestions

Thanks again! I think this can get closed in a few minutes....

Super :-)
You'll get the "stable" version automatically the first time you publish a release on github...


No more tests

Just wondering why you've removed all the tests ? I'm kind of missing them.
Cheers

Removed to simply the codebase — they didn't actually accomplish much as they were, and were causing some problems.

thanks for the precision :)


Hide days without any events

Would like to suggest an option to hide days without anything planned.

Today should always be shown even without events.
"days" options should still be respected to still only show events x days in the future.

Can you check if https://github.com/FamousWolf/week-planner-card/pull/54 does what you had in mind?

Yep description and logic looks like it should display events within the next X days if there are any and always display "today" even if empty.
cool, can't wait to try it out.

@FamousWolf , How to i hide days without events? i dont know how to see #54 xD noob here on github


Consider development and publication process update

develop - master branch pair is an option:

each PR when merged goes to develop
when develop accumulates changes (in particular just one PR) that are ready to be released publicly, then develop is merged into master (or just master is FFed to match develop);
each merge to master triggers an automatic release (in our case, plugin publish)
also: the versioning scheme would likely need to be adapted, TBD

Originally posted by @PawelLipski  on Slack
This will possibly resolve #193  as well

Wrt. new versioning scheme: pre-release suffix like -1, -2 etc. can be used (even without specifying -RC, -M, -alpha, -beta etc., just a number to introduce a clear ordering of the pre-release versions)

Also, the following scenario is likely that after a release e.g. 1.0.3, we'll have the following sequence of pre-release versions:

1.0.4-1
1.0.4-2
1.0.4-3
1.1.0-1 - coz we've just adding new feature, the new release won't be a PATCH-level anymore, but MINOR-level one
1.1.0-2
1.1.0-3
1.1.0 - finally releasing as minor release; as a consequence, 1.0.4 never actually gets released

TBD how develop->master release should be done, preferably a separate CI job that:

can only ever be triggered manually from CI (and not automatically after anything is pushed)
commits the pre-release suffix removal from the version on master
pushes the updated master, hence triggering a regular master build

Each successful master build, in turn, should publish the plugin artifact to Marketplace as its final step.

Whoops, this part:

can only ever be triggered manually from CI

seems pretty hard to achieve on Circle CI (https://discuss.circleci.com/t/manually-triggered-deployment-stages/5515/28), the closest we can get is:
https://circleci.com/docs/2.0/workflows/#holding-a-workflow-for-a-manual-approval

So likely we should instead provide a script that would run the steps 1-4 locally (TBD how to handle master branch protection then?), and master builds can publish the plugin - but only after getting a manual approval in Circle CI.


CocoaPods 1.0 support.

CocoaPods 1.0 was recently released and introduces some changes to podspec requirements. This PR addresses those and makes the version to be based on package.json so there's less work to do releases.

Thanks!

Great! Can I ask for an npm bump? :-)

Done:)

NPM still says 1.5.6?

Oops, forgot to actually publish after bumping the version. Just published 1.5.7 on npm


feat: try to extract .png icon

This tries to extract <ExecName>.png if .DirIcon doesn't exist in the appimage.
I added this mainly because the appimage from osu did not contain a .DirIcon, thus failing the installation with pho.
Though .DirIcon is the spec, I thought there might be some appimages that also has this problem,
so maybe this can be added to make app installation easier.

LGTM!


DO NOT USE IT!!!!!!

This lib cannot handle relationships like OneToOne, etc.

sure, i have the same problem, here is an example for you:
class Project(SoftDeleteModel):
current_stage = models.OneToOneField("Stage", on_delete=models.CASCADE, related_name="current_stage", null=True)
class Stage(SoftDeleteModel):
project = models.ForeignKey(Project, on_delete=models.CASCADE)

Thanks a lot @droomo for finding the critical bug and for a very eloquent heading )
The fixes are already made it the next package version will include them.

@droomo amf @no-ask-star are you using the latest version? 1.0.15? Can you try it so we can close this issue?


do not gc vm pod lsp when vm still exists

What type of this PR

Bug fixes

1、do not gc vm pod lsp when vm still exists (keep-vm-ip flag is set in kube-ovn-controller)
2、The same problem exists for sts when sts pods do not use ip_pool to assign static ips
Which issue(s) this PR fixes:
Fixes #1502

release-1.9 和 release-1.10 已经cherry-pick回合
master分支，通过 #1561 合入


But the help does not show how to configure thefuck for windows cmd, only for powershell.
The latest commit shows that there is support for windows cmd
https://github.com/nvbn/thefuck/commit/3cd187a3bb47351890ac7308464e1a2780507220
But I guess since the latest release is from January and the commit is from July a version that supports windows cmd wasn't released yet.

But the help does not show how to configure thefuck for windows cmd, only for powershell.

Terminal (CMD)

The terminal does not store the session in memory  globally without making environment variables where as the shell does store the session data as objects globally. The  terminal (CMD) only outputs data in characters.. Therefore the terminal cant store the instance of the commands result after execution.

Shell (PowerShell)

The Shell (PowerShell) the state in memory will be stored throughout the session as objects instead of characters; until the session is closed.

BASH (Bourne-Again SHell) [GitBash]

So also a concept that should be considered is BASH. Being as a shell can store memory throughout the session BASH (Bourne-Again SHell) which is similar to treating data as text but instead of plain text it uses structures.

I am familiar with these concepts but I just wanted to address your Git Issue with some support. Hopefully it helps. Also Below I will provide a resource i used to fact check my understanding of explaining to your the reason why you could prompt the Git Issue.

References

What is ______ ?

| 1. Terminal | 2. CMD | 3. Shell | 4. PowerShell | 5. Bash |

Comparison Chart

CMD v. PowerShell v. Bash Table

Sorry but I am not sure how your posts helps my issue.

Thanks for the info!


missing module: mapbox_earcut

On Mac OSX M1 processor, I got as far as the visualization step, then received the following error
mw.visualize(structs, scale=(1, 1, 0.2))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/meow/visualize.py", line 95, in visualize
    return _visualize_structures(objs, **kwargs)
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/meow/structures.py", line 48, in _visualize_structures
    geometry=[s._trimesh(scale=scale) for s in _sort_structures(structures)]
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/meow/structures.py", line 48, in <listcomp>
    geometry=[s._trimesh(scale=scale) for s in _sort_structures(structures)]
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/meow/structures.py", line 33, in _trimesh
    return self.geometry._trimesh(
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/meow/geometries.py", line 256, in _trimesh
    prism = extrude_polygon(poly, self.h_max - self.h_min)
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/trimesh/creation.py", line 194, in extrude_polygon
    vertices, faces = triangulate_polygon(
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/trimesh/creation.py", line 458, in triangulate_polygon
    faces = _tri_earcut(vertices, rings).reshape(
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/trimesh/exceptions.py", line 48, in failed
    raise exc
  File "/Users/lukasc/gdsfactory/lib/python3.9/site-packages/trimesh/creation.py", line 34, in <module>
    from mapbox_earcut import triangulate_float64 as _tri_earcut
ModuleNotFoundError: No module named 'mapbox_earcut'

easy fix:
pip install mapbox_earcut
perhaps you can add it as a dependancy.

Interesting, mapbox_earcut seems to be a dependency for trimesh, which is in the requirements. I guess I'll have to specify trimesh[full] in stead. Thanks for letting me know!

Fixed in the commit above.


Rephrase ProjFS error messaging

On Windows RS1/Server ProjFS is not inboxed. On those PCs the current gvfs upgrade pre-check error messaging is confusing. Updated the messaging.
Issue #769

This looks good. Please also cherry-pick this against milestones/M147.


[Windows] Do not force redraw window background on mouse pass-through region change.

Disables forced window redraw on region change (which can cause default bg color to show if Vulkan context is not updating fast enough).
Remove unnecessary region deletion.

After a successful call to SetWindowRgn, the system owns the region specified by the region handle hRgn. The system does not make a copy of the region. Thus, you should not make any further function calls with this region handle. In particular, do not delete this region handle. The system deletes the region handle when it no longer needed.

Fixes https://github.com/godotengine/godot/issues/80098

Thanks!

Cherry-picked for 4.1.3.


Add tune run and refactor CLI

Context
Once again, I fail to limit the scope of a PR. As I was adding the tune run CLI command which we would use to launch all our recipes, I noticed how clunky our current CLI code was and needed to take on the refactor. Please take a look at all the code as there are some changes to the UX of the CLI, as well.  This should help us in the long run as it makes our code more debuggable, adds better error handling, adds more tests, and makes the CLI more extensible.
Look through the changelog and the code for all the information and some FAQs right below.
Why is the CLI so slow?
Startup time for the tune CLI is slowwww:
(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ time tune
usage: tune [-h] {download,ls,cp,run,validate} ...

...

real    0m5.995s
user    0m5.557s
sys     0m3.480s

Compare this to HuggingFace's accelerate:
(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ time accelerate
usage: accelerate <command> [<args>]

...

real    0m4.474s
user    0m4.176s
sys     0m3.298s

And the classic git:
(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ time git
usage: git [-v | --version] [-h | --help] [-C <path>] [-c <name>=<value>]

...

real    0m0.024s
user    0m0.006s
sys     0m0.018s

Okay, we have no chance of being as fast as git, which is written in C and has been optimized over years of development. Python itself is slow and argparse, as part of the stdlib, is no exception. We are roughly on par with accelerate, which also uses argparse under the hood, with the added overhead likely from incorporating all the commands from torchrun, which also uses argparse. There are some small optimizations like copying over all the torchrun commands rather than loading them from torchrun's argparser, but the best way to speed up our CLI would be to take a dependency on another library like click orrrrrrrr write our CLI in a language like Go. Neither of those options seems likely in the short term, so really I'm just calling this out as a known issue in case users are concerned.
Why did you add new dataclasses to represent recipes and configs? Isn't that overkill?
Attributes of our recipes are getting more and more complex as the days progress so I think it's natural to start loosely formalizing this information. Specifically for this PR, tune run needed to know when a recipe was able to be run in a distributed-fashion. Now, I could check the string for "distributed", but that's a week assertion. Now, with recipes having an attribute called supports_distributed, there's no guesswork. This adds very little to the overhead needed to add a new "core" recipe and gives us stronger reasoning about the recipes we're running.
Changelog

Remove convert_checkpoint command. cc @kartikayk  and @ebsmothers  for the approval here
Add better error handling for download and cp
Add dataclasses for recipes and configs
Add tune run command
Add tune run command tests
Update README
Update tests

Testing

CI green
Unit tests
Run all single recipes (minus alpaca generate)

(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ tune run lora_finetune_single_device --config llama2/7B_lora_single_device
Running recipe_main with parameters {'model': {'_component_': 'torchtune.models.llama2.lora_llama2_7b', 'lora_attn_modules': ['q_proj', 'v_proj'], 'apply_lora_to_mlp': False, 'apply_lora_to_output': False, 'lora_rank': 8, 'lora_alpha': 16}, 'checkpointer': {'_component_': 'torchtune.utils.FullModelMetaCheckpointer', 'checkpoint_dir': './model', 'checkpoint_files': ['consolidated.00.pth'], 'adapter_checkpoint': None, 'recipe_checkpoint': None, 'output_dir': './', 'model_type': 'LLAMA2'}, 'resume_from_checkpoint': False, 'tokenizer': {'_component_': 'torchtune.models.llama2.llama2_tokenizer', 'path': './model/tokenizer.model'}, 'dataset': {'_component_': 'torchtune.datasets.alpaca_dataset', 'train_on_input': True, 'use_clean': True}, 'seed': None, 'shuffle': True, 'batch_size': 2, 'optimizer': {'_component_': 'torch.optim.AdamW', 'weight_decay': 0.01, 'lr': 0.0003}, 'lr_scheduler': {'_component_': 'torchtune.modules.get_cosine_schedule_with_warmup', 'num_warmup_steps': 100}, 'loss': {'_component_': 'torch.nn.CrossEntropyLoss'}, 'epochs': 1, 'max_steps_per_epoch': None, 'gradient_accumulation_steps': 1, 'output_dir': '/tmp/lora_finetune_output', 'metric_logger': {'_component_': 'torchtune.utils.metric_logging.DiskLogger', 'log_dir': '${output_dir}'}, 'log_every_n_steps': None, 'device': 'cuda', 'dtype': 'bf16', 'enable_activation_checkpointing': True}
Setting manual seed to local seed<PHONE_NUMBER>. Local seed is seed + rank =<PHONE_NUMBER> + 0
Writing logs to /tmp/lora_finetune_output/log_1711657264.txt
Model is initialized with precision torch.bfloat16.
Memory Stats after model init::
GPU peak memory allocation: 13.96 GB
GPU peak memory reserved: 13.98 GB
GPU peak memory active: 13.96 GB

Tokenizer is initialized from file.
Optimizer and loss are initialized.
Loss is initialized.
Dataset and Sampler are initialized.
Learning rate scheduler is initialized.
1|11|Loss: 1.597842812538147:   0%|

(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ tune run full_finetune_single_device --config llama2/7B_full_single_device
Running recipe_main with parameters {'tokenizer': {'_component_': 'torchtune.models.llama2.llama2_tokenizer', 'path': './tokenizer.model'}, 'dataset': {'_component_': 'torchtune.datasets.alpaca_dataset', 'train_on_input': True}, 'seed': None, 'shuffle': True, 'model': {'_component_': 'torchtune.models.llama2.llama2_7b'}, 'checkpointer': {'_component_': 'torchtune.utils.FullModelMetaCheckpointer', 'checkpoint_dir': './model', 'checkpoint_files': ['consolidated.00.pth'], 'recipe_checkpoint': None, 'output_dir': './', 'model_type': 'LLAMA2'}, 'resume_from_checkpoint': False, 'batch_size': 2, 'epochs': 3, 'optimizer': {'_component_': 'torch.optim.SGD', 'lr': 2e-05}, 'loss': {'_component_': 'torch.nn.CrossEntropyLoss'}, 'max_steps_per_epoch': None, 'gradient_accumulation_steps': 1, 'device': 'cuda', 'enable_activation_checkpointing': True, 'dtype': 'bf16', 'metric_logger': {'_component_': 'torchtune.utils.metric_logging.DiskLogger', 'log_dir': '${output_dir}'}, 'output_dir': '/tmp/alpaca-llama2-finetune', 'log_every_n_steps': None}
Setting manual seed to local seed<PHONE_NUMBER>. Local seed is seed + rank =<PHONE_NUMBER> + 0
Writing logs to /tmp/alpaca-llama2-finetune/log_1711657490.txt
Model is initialized with precision torch.bfloat16.
Memory Stats after model init::
GPU peak memory allocation: 13.95 GB
GPU peak memory reserved: 13.97 GB
GPU peak memory active: 13.95 GB

Tokenizer is initialized from file.
Optimizer is initialized.
Loss is initialized.

(torchtune-2<EMAIL_ADDRESS>~/projects/torchtune (fixup-cli)]$ tune run eleuther_eval --config eleuther_eval
Running recipe_main with parameters {'model': {'_component_': 'torchtune.models.llama2.llama2_7b'}, 'checkpointer': {'_component_': 'torchtune.utils.FullModelTorchTuneCheckpointer', 'checkpoint_dir': './', 'checkpoint_files': ['qlora_trained.pt'], 'output_dir': './', 'model_type': 'LLAMA2'}, 'tokenizer': {'_component_': 'torchtune.models.llama2.llama2_tokenizer', 'path': './tokenizer.model'}, 'device': 'cuda', 'dtype': 'bf16', 'seed': 217, 'tasks': ['truthfulqa_mc2'], 'limit': None, 'max_seq_length': 4096}
2024-03-28:13:27:18,513 INFO     [_parse.py:52] Running recipe_main with parameters {'model': {'_component_': 'torchtune.models.llama2.llama2_7b'}, 'checkpointer': {'_component_': 'torchtune.utils.FullModelTorchTuneCheckpointer', 'checkpoint_dir': './', 'checkpoint_files': ['qlora_trained.pt'], 'output_dir': './', 'model_type': 'LLAMA2'}, 'tokenizer': {'_component_': 'torchtune.models.llama2.llama2_tokenizer', 'path': './tokenizer.model'}, 'device': 'cuda', 'dtype': 'bf16', 'seed': 217, 'tasks': ['truthfulqa_mc2'], 'limit': None, 'max_seq_length': 4096}
Setting manual seed to local seed 217. Local seed is seed + rank = 217 + 0
2024-03-28:13:27:19,173 DEBUG    [seed.py:59] Setting manual seed to local seed 217. Local seed is seed + rank = 217 + 0
Model is initialized with precision torch.bfloat16.
2024-03-28:13:27:40,285 INFO     [eleuther_eval.py:160] Model is initialized with precision torch.bfloat16.
Tokenizer is initialized from file.
2024-03-28:13:27:40,314 INFO     [eleuther_eval.py:146] Tokenizer is initialized from file.
2024-03-28:13:27:41,786 INFO     [huggingface.py:148] Using device 'cuda:0'
2024-03-28:13:27:48,865 WARNING  [__init__.py:194] Some tasks could not be loaded due to missing dependencies. Run with `--verbosity DEBUG` for full details.
2024-03-28:13:27:52,458 WARNING  [__init__.py:194] Some tasks could not be loaded due to missing dependencies. Run with `--verbosity DEBUG` for full details.
Running evaluation on ['truthfulqa_mc2'] tasks.
2024-03-28:13:28:08,859 INFO     [eleuther_eval.py:181] Running evaluation on ['truthfulqa_mc2'] tasks.
2024-03-28:13:28:08,861 INFO     [task.py:363] Building contexts for task on rank 0...
2024-03-28:13:28:10,287 INFO     [evaluator.py:324] Running loglikelihood requests
  2%|███▍

@joecummings I didn't follow the comment around validate. What are you removing?

@joecummings I didn't follow the comment around validate. What are you removing?

Typo on my part. Meant to call out the removal of convert_checkpoint. I updated the PR description.

Typo on my part.


Bounties

Rather than adding a command for bounties they can be implemented socially by people offering to /give bbucks for the completion of a task.


Nightly fixes

More changes for this bug. This is an extension of this PR.
Note: Commit 4a8b05c removes the unused_results lint because it apparently conflicts with #[derive(Debug)]. This commit should be reverted as soon as this is fixed upstream.

r? @vinipsmaker
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Implement tabular benchmarks in Python and C++

The benchmark will be receiving a directory of Parquet, CSV, Avro or Orca files and exporting them simultaneously into graph, document, and, potentially, path collections. The latter can also be added into the twitter benchmark.

Efficient Python implementation would require finishing #41

The construction part may be just a direct call to ukv_docs_import and ukv_graph_import_t from #154.
After the construction, sampling (#153) should be performed to randomly selected items to be queried in the analytical part of the benchmarks. Over time that benchmark will replace the twitter benchmark.


Maintenance of Filer

I'm not able to continue maintaining Filer.  It was a critical piece of various projects I did with Mozilla, but I'm not using it anymore.  I'm wondering whether to archive the repo, hand it off to someone else, or do nothing?
At the very least, I'm going to stop work on it personally.

Thank you so much for this amazing library.
Are there any things you wish someone else might have the time and motivation to work on?
Couple of things perhaps come to mind outside of working on issues:

TypeScript (rewrite or merely creating @types/filer type definitions module)
ES Modules and a newer build process (outside of UMD)


CryoSPARC Reconstruction Parameters and Preprocessing Details

Hello，
I am currently using CryoSegNet segmentation results to perform cryoSPARC reconstruction, but I am unable to achieve the results reported in the paper. I would appreciate it if you could clarify a few details about the reconstruction process:

Did you use the default parameters in cryoSPARC for the reconstruction process, or were any parameters adjusted specifically for your dataset?
Was there any preprocessing done on the segmentation results before reconstruction in cryoSPARC? If so, could you provide some details on the preprocessing steps?

Thank you very much for your help and for sharing this valuable resource!

Hi,
The default parameters are used. The extraction box size in 'Extract Micrographs job' is different for different proteins. No further preprocessing is required.
Thank you. Feel free to ask me if you have further questions


Fix: Fix the Dockerfile and the README.md

The versions used in the Dockerfile as well as the README.md didnt seems to be up to date.
I also add the build part for the Dockerfile. This would help to be independant from any previous java/maven installation on the the build host.
I also would like to state that i am not really a java developer and that i have changed the build process as it was failing probably due to a double goal .
Feel free to comment.

@oferrigni @kickroot @belosh59 Is that okay to ping you in that way ?

Thanks for the contribution!


add support for virtual service in delta clusters

Please provide a description of this PR:

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test unit-tests-arm64

lint_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test unit-tests-arm64

lint_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test lint

unit-tests_istio
af00ae5b85711c6db83f8dcd8d5cd86d95a5ef3e
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn @hzxuzhonghu can you PTAL when you get chance?

@hzxuzhonghu @howardjohn PTAL when you get a chance - not urgent

@hzxuzhonghu @howardjohn gentle ping

What other fields do you think will need to be added here? We already have DR and this PR is adding VS. I do not think we care any CRD much for Delta.
Can you please explain more by what do you mean by generic way? I get your point but trying to understand whether we can do it in the current architecture

Only DR, VS is far from delta

But we already have DR and this PR is only adding VS now

I know, but it's time to revisit that implement too

Do you have any ideas on how that can be solved?

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
170d0262a3af42873ef07fa82970b082396e5f82
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
170d0262a3af42873ef07fa82970b082396e5f82
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
31e587538e473cfa40e436f4112f861b01d931e0
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Writing a DMP

As a researcher I would like to learn to write better Data Management Plans so that my grant proposals are more competitive.

Relevant Personas
Sherman Bernhard
Patience Hansen
Acceptance Criteria

[ ] Covers parts of DMP

[ ] Covers what gets good reviews

[ ] Covers what gets bad reviews

[ ] Introduce DMPTool

[ ] How to read funder requirements and suggestions

Replaced by more formal solutions.


[WIP] Update evaluate_periodically() when eval interval is of type Duration

Description

User can now pass the '<*>dur' or Time(<value>, TimeUnit.DURATION) as eval_interval during training.
For example, eval_interval as 0.1dur with max_duration as 20ep (20 Epochs) means that evaluation would get executed at every 2 epochs with total of 10 evaluations during the whole training process.
There are multiple ways where user can define a combination of eval_interval and max_duration.

No.
eval_interval
max_duration
Description

1
'0.1dur'
'50ep'
Perform evaluation at every 5 epochs

2
'0.05dur'
'1000ba'
Perform evaluation at every 50 batches

3
'0.2dur'
'6000sp'
Perform evaluation at every 1200 samples

4
'0.1dur'
'10000tok'
Perform evaluation at every 1000 tokens

5
Time(0.25, TimeUnit.DURATION)
'40ep'
Perform evaluation at every 10 epochs

Testing
Unit test

Added a unit test to test the functionality for the above usecase except the 4th.
[TODO] Add a unittest for NLP workloads (when max_duration is '<val>tok')

Integration Test

Ran the MNIST example to different max_duration usecase such as '<value>ep', '<value>ba', and '<value>sp'

max_duration: 10ep,  eval_interval: 0.2dur, and train_batch_size: 2048

Evaluation has been called 5 times as expected which is at epoch 1, 3, 5, 7, 9 (epoch number starts with 0)

[epoch=1][batch=29/10]: metrics/eval/CrossEntropy: 0.1552
[epoch=1][batch=29/10]: metrics/eval/Accuracy: 0.9490
....
[epoch=3][batch=29/10]: metrics/eval/CrossEntropy: 0.0755
[epoch=3][batch=29/10]: metrics/eval/Accuracy: 0.9770
....
[epoch=5][batch=29/10]: metrics/eval/CrossEntropy: 0.0604
[epoch=5][batch=29/10]: metrics/eval/Accuracy: 0.9806
....
[epoch=7][batch=29/10]: metrics/eval/CrossEntropy: 0.0540
[epoch=7][batch=29/10]: metrics/eval/Accuracy: 0.9828
....
[epoch=9][batch=29/10]: metrics/eval/CrossEntropy: 0.0531
[epoch=9][batch=29/10]: metrics/eval/Accuracy: 0.9831

max_duration: 100ba,  eval_interval: 0.1dur, and train_batch_size: 2048

Evaluation has been called 10 times as expected which is at batch 10, 20, 30, 40, 50, 60, 70, 80, 90, 100

[batch=10/100]: metrics/eval/CrossEntropy: 2.1561
[batch=10/100]: metrics/eval/Accuracy: 0.3948
....
[batch=20/100]: metrics/eval/CrossEntropy: 1.0348
[batch=20/100]: metrics/eval/Accuracy: 0.7864
....
[batch=30/100]: metrics/eval/CrossEntropy: 0.5125
[batch=30/100]: metrics/eval/Accuracy: 0.9109
....
[batch=40/100]: metrics/eval/CrossEntropy: 0.2912
[batch=40/100]: metrics/eval/Accuracy: 0.9279
....
<output curtailed>
....
[batch=100/100]: metrics/eval/CrossEntropy: 0.1161
[batch=100/100]: metrics/eval/Accuracy: 0.9644

max_duration: 60000sp,  eval_interval: 0.10dur, and train_batch_size: 2048

Evaluation has been called 10 times as expected which is at samples 6144, 12288, 18432, 24576, 30720, 36864, 43008, 49152, 55296, 61440.

[sample=6144/60000]: metrics/eval/CrossEntropy: 2.3010
[sample=6144/60000]: metrics/eval/Accuracy: 0.1135
....
[sample=12288/60000]: metrics/eval/CrossEntropy: 2.2809
[sample=12288/60000]: metrics/eval/Accuracy: 0.1451
....
[sample=18432/60000]: metrics/eval/CrossEntropy: 2.2158
[sample=18432/60000]: metrics/eval/Accuracy: 0.2579
....
[sample=24576/60000]: metrics/eval/CrossEntropy: 2.0613
[sample=24576/60000]: metrics/eval/Accuracy: 0.5095
....
<output curtailed>
....
[sample=61440/60000]: metrics/eval/CrossEntropy: 1.0423
[sample=61440/60000]: metrics/eval/Accuracy: 0.8067

[TODO] Test NLP workload when max_duration is `'tok

This looks like the right approach to me, but I think we can make this a little more general while we're already refactoring it. It looks like you're basically casing duration into the 4 other types and then (approximately) converting into those units.
Can we instead rewrite this logic to say if the time unit is a duration, convert it into the type of max duration at the top of the function, and then follow the codepath for the respective type we've converted it to? This gives us the added benefit of being able to use the same logic you've added for tokens and samples to allow users to submit eval_interval in those units as well.

If the eval_interval.unit is of type TimeUnit.SAMPLE or TimeUnit.TOKEN, then the calculation of evaluation interval is different than the current implementation of eval_interval.unit as TimeUnit.DURATION and max_duration.unit as TimeUnit.SAMPLE.
For example, batch_size = 2, max_duration = 5000sp, and eval_interval = 100sp

then we perform evaluation at every 100 samples.
However, the calculation is different than the current approach in this PR where we don't have to do math.ceil(state.max_duration.value * eval_interval) instead we can just use eval_interval

Looks we have to handle all the scenarios case by case basis.
Please let me know if I didn't understand your point clearly.

Ideally, we'd just convert eval_interval from duration unit to whatever unit type max_duration at top of function for simplicity. More broadly, we want a util function set that lets us easily convert between different units. I don't think its necessary for getting this in before the next release though.
After this goes in, can you please make a new JIRA and track it for adding NLP support with tokens and unit tests for that scenario?

Sounds good. Thanks for the explanation!


Improved signalSorter memory handling and modelModifyMovies non-Miji support

Improvements to signalSorter memory handling, esp. during async calls along with other improvements.
modelModifyMovies now supports cropping of movies after pre-processing using Matlab figures instead of Miji (Fiji/ImageJ in Matlab) for users who can't or don't want to use Miji.
Various other code fixes and speed-ups.

@omidm7,
Check updated modelModifyMovies in this pull request, added feature from https://github.com/bahanonu/calciumImagingAnalysis/issues/21#issuecomment-498497331.

Thank you so much, modelModifyMovies  is working though matlab now. Will definitely make life easier. :)
Cheers
Omid


Refine fgAccent (light mode)

Quickly turns nearly black — good for readability, but perhaps a bit more chroma can be preserved for non-achromatic seeds.

Hey team! Please add your planning poker estimate with Zenhub @ichik @jsartisan @KelvinOm


feat: Add audit log output

Adds the audit_log output parameter that captures audit log information in a list of JSON objects. Fixes #110
An audit log object contains the following properties:

repo: string
target: string
action: string
secret_name: string
secret_hash: string
environment: string
dry_run: boolean

The secret_hash contains the hashed password. This can be used to track whether a secret actually changes.
Optionally, a custom salt can be provided via the audit_log_hashing_salt to make it more difficult to reverse engineer the secret.
This functionality might be a bit specific for our project's needs. If it is useful for the greater public, feel free to merge. It does not introduce any breaking changes.

I'm concerned about the handling of the case where a salt has not been provided. As far as I can tell this would mean the hash(value, "") would show up in the logs.

Following up on this to see what your thoughts are?

I'm concerned about the handling of the case where a salt has not been provided. As far as I can tell this would mean the hash(value, "") would show up in the logs.

Following up on this to see what your thoughts are?

You are right, the unsalted hashed value is printed in the logs.
As a mitigation, I now use the GITHUB_REPOSITORY_ID  as the default salt value. What do you think?


Add new ViewContainerLocation for dialogs

Fixes https://github.com/microsoft/azuredatastudio/issues/11283
It might be worth it to investigate this further. We have other instances of dialog components using ViewPanes although it looks like those might actually be unused or broken :
Options dialog - unused? https://github.com/microsoft/azuredatastudio/blob/main/src/sql/workbench/browser/modal/optionsDialog.ts#L41
Insights dialog - seems to be broken, possibly related to the view changes that we had to fix for the other views : https://github.com/microsoft/azuredatastudio/blob/main/src/sql/workbench/services/insights/browser/insightsDialogView.ts#L124

Coverage remained the same at 35.857% when pulling 02b9320d768e754070020117eb39c4d7bbb4cddc on chgagnon/fixAccountViewInPanel into 399bac33f41255b00916918581dbbbfbc5879465 on main.

I doubt this change specifically will cause any future issues, but like you said because we aren't following the assumption vscode is using for viewpanes it seems likley they will break again and we should either:

Move away from them and use a more general base class (preferably something in vs/base), in which case we would remove this code
Wholefully buy into their concept of viewpanes and make our code work correctly with those assumptions, in which case this would be apart of that change.


Poor documentation for Unity HTTP Request Configuration

The PlayFab SDK for Unity has a "Request Type" setting which specifies the underlying web request library to use. The options vary depending on the version of Unity; for example, in Unity 2021, there are three options:

Unity Web Request
Http Web Request
Custom Http

These options aren't mentioned in the online PlayFab documentation. The only decent description of them that I can find anywhere is in the README for the PlayFab Unity SDK, but this section of the README hasn't been updated since 2017 and presumably is grossly outdated.
We can't make an informed decision of which web request library to use if we don't have up-to-date and accurate information on the different options. For example, the README says

There is currently a bug in this option where performance will degrade for long running GameServers, or clients that run for more than 24 hours.

Does that bug still exist 5.5 years later?

Thanks for your feedback. We have queued this up for a review and update.

Page has been updated. Please let us know if you have any further feedback. Thanks!


uORB pub/sub regression

@mcharleb It is possible that we have a regression in muORB. We have user reports on NuttX and I have severe stability issues on Mac OS.
This branch will help to reproduce:
https://github.com/PX4/Firmware/tree/cmake-2-dbg
It is important that you have a complete setup including gamepad - the main symptom is that sensor values stop updating and the vehicle will drop out of the sky after 1-10 seconds. There is a also this troubling user report on NuttX:
https://groups.google.com/forum/#!topic/px4users/tP0kz9XPSjE

The crash on Mac OS happens because _pollset[i] has an invalid entry, leading to a crash when trying to post the semaphore: https://github.com/PX4/Firmware/blob/cmake-2/src/drivers/device/vdev.cpp#L416

The 2nd symptom is that without the usleeps on startup the system can get into an infinite loop where it stops updating the sensors.

Just to clarify, muorb will not be active for nuttx or macos builds as it is specific to snapdragon based platform.  I am suspecting this is more specific for uorb stack.  Also there are some unit tests for muorb related to changes to uorb stack.

Fixed.


[NOT FOR REVIEW] Test Harbor on Github Actions

Codecov Report

Merging #1559 (6e7ad07) into master (9d3d10b) will decrease coverage by 22.63%.
The diff coverage is 10.60%.

@@             Coverage Diff             @@
##           master    #1559       +/-   ##
===========================================
- Coverage   63.31%   40.68%   -22.64%
===========================================
  Files         170      106       -64
  Lines       14250    12747     -1503
===========================================
- Hits         9023     5186     -3837
- Misses       4292     7093     +2801
+ Partials      935      468      -467

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
40.68% <10.60%> (-0.60%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/antrea-agent/agent.go
0.00% <0.00%> (ø)

cmd/antrea-agent/options.go
21.69% <0.00%> (+0.97%)
:arrow_up:

pkg/agent/agent.go
12.05% <0.00%> (-36.67%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <ø> (-100.00%)
:arrow_down:

.../agent/flowexporter/connections/conntrack_linux.go
28.37% <0.00%> (-0.79%)
:arrow_down:

pkg/agent/openflow/pipeline.go
28.17% <0.00%> (-41.61%)
:arrow_down:

pkg/agent/stats/collector.go
51.13% <ø> (-46.60%)
:arrow_down:

pkg/apiserver/certificate/cacert_controller.go
10.81% <0.00%> (-45.58%)
:arrow_down:

pkg/controller/networkpolicy/mutate.go
0.00% <0.00%> (ø)

... and 153 more


ClearAll functionality

Hi, I read you use most of the code from angular-local-storage, but this method clearAll there has an optional parameter, the regex.
On angular-2-local-storage I'm obliged to inform the regex. My question is, if I do something like this this.localStorageService.clearAll(''); will it work the same way or you validate/require a regex for removal? If that's the case, do you plan on implementing that functionality?
Thanks!

Hi @eestein, good spotting, this is an issue with our TypeScript interface.
I have fixed it (made the parameter optional as it should be) and published the new version @0.0.23


Installation on x86_64

Hi I am trying to install the software on a x86_64 machine. But I encountered compilation errors
--- stderr: isaac_ros_image_proc
In file included from /home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:33:
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp: In function ‘cv::Mat {anonymous}::GetConvertedMat(VPIImageImpl*&, VPIImageImpl*&, VPIStreamImpl*&, const cv::Mat&, uint32_t, std::string, std::string)’:
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:68:5: error: ‘vpiImageCreateOpenCVMatWrapper’ was not declared in this scope; did you mean ‘vpiImageCreateWrapper’?
   68 |     vpiImageCreateOpenCVMatWrapper(
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/ubuntu/isaac_perception_ws/install/isaac_ros_common/include/isaac_ros_common/vpi_utilities.hpp:20:25: note: in definition of macro ‘CHECK_STATUS’
   20 |     VPIStatus status = (STMT); \
      |                         ^~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:81:61: error: too many arguments to function ‘VPIStatus vpiImageLock(VPIImage, VPILockMode)’
   81 |   CHECK_STATUS(vpiImageLock(output, VPI_LOCK_READ, &out_data));
      |                                                             ^
/home/ubuntu/isaac_perception_ws/install/isaac_ros_common/include/isaac_ros_common/vpi_utilities.hpp:20:25: note: in definition of macro ‘CHECK_STATUS’
   20 |     VPIStatus status = (STMT); \
      |                         ^~~~
In file included from /home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:28:
/opt/nvidia/vpi2/include/vpi/Image.h:556:22: note: declared here
  556 | VPI_PUBLIC VPIStatus vpiImageLock(VPIImage img, VPILockMode mode);
      |                      ^~~~~~~~~~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:82:31: error: ‘VPIImageData’ {aka ‘struct VPIImageDataRec’} has no member named ‘planes’
   82 |   cv::Mat output_mat{out_data.planes[0].height, out_data.planes[0].width,
      |                               ^~~~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:82:58: error: ‘VPIImageData’ {aka ‘struct VPIImageDataRec’} has no member named ‘planes’
   82 |   cv::Mat output_mat{out_data.planes[0].height, out_data.planes[0].width,
      |                                                          ^~~~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:83:54: error: ‘VPIImageData’ {aka ‘struct VPIImageDataRec’} has no member named ‘planes’
   83 |     g_str_to_channels.at(encoding_desired), out_data.planes[0].data,
      |                                                      ^~~~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:84:34: error: ‘VPIImageData’ {aka ‘struct VPIImageDataRec’} has no member named ‘planes’
   84 |     static_cast<size_t>(out_data.planes[0].pitchBytes)};
      |                                  ^~~~~~
/home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:84:55: error: no matching function for call to ‘cv::Mat::Mat(<brace-enclosed initializer list>)’
   84 |     static_cast<size_t>(out_data.planes[0].pitchBytes)};
      |                                                       ^
In file included from /usr/include/opencv4/opencv2/core/mat.hpp:3724,
                 from /usr/include/opencv4/opencv2/core.hpp:59,
                 from /usr/include/opencv4/opencv2/core/core.hpp:48,
                 from /opt/ros/foxy/include/cv_bridge/cv_bridge.h:43,
                 from /home/ubuntu/isaac_perception_ws/src/isaac_ros_image_pipeline/isaac_ros_image_proc/src/image_format_converter_node.cpp:20:
/usr/include/opencv4/opencv2/core/mat.inl.hpp:1380:1: note: candidate: ‘cv::Mat::Mat(cv::Mat&&)’
 1380 | Mat::Mat(Mat&& m)
      | ^~~
/usr/include/opencv4/opencv2/core/mat.inl.hpp:1380:1: note:   candidate expects 1 argument, 5 provided

It seems this software depends on a specific VPI version. Could you tell me what is a proper way to setup installation environment?
Currently I am using a Dockerfile based on nvidia/cuda:11.4.0-devel-ubuntu20.04 and install VPI libraries following https://docs.nvidia.com/vpi/installation.html
RUN apt-get update && apt-get install -y software-properties-common gnupg  && \
    apt-key adv --fetch-key https://repo.download.nvidia.com/jetson/jetson-ota-public.asc
RUN add-apt-repository 'deb https://repo.download.nvidia.com/jetson/x86_64/focal r34.1 main' && \
    apt update && \
    apt-get install -y libnvvpi2 vpi2-dev vpi2-samples git-lfs

My Nvidia driver version is 510 but I think this issue is not related to GPU driver.

The error messages with VPI indicate you've install VPI2 and not VPI1.1 (the signatures for vpiImageCreateOpenCVMatWrapper has changes, etc.). You'll want to install the older VPI 1.1 which is what Isaac ROS packages are compatible with as of Early Access 3 (EA3).

In the Dockerfile.x86_64.base you'll find the correct vpi installation commands which use the VPI Debians packaged with isaac_ros_common itself.


add media

Docs Build status updates of commit e90d2e5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/payroll/media/DEDCHILDSUP.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDCHILDSUP1.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDCHILDSUP2.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDCHILDSUP3.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDCHILDSUP4.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDLEVY01.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDLEVY02.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDLEVY03.JPG
:white_check_mark:Succeeded
View

dynamics-gp/payroll/media/DEDLEVY04.JPG
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


feat: enable creating dispatcher models

Adds new feature allowing users to select multiple models to create a dispatching model
The model is currently generated using overloaded text actions, but will do more work to introduce new type in future
The model will be viewable in the UI but require an updated SDK to function.

In training you can only view the single model (The dispatching models direct output of which model to dispatch to)
In logs you can see the full result all the way to leaf model.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update dg-readiness-tool.md

https://devblogs.microsoft.com/scripting/enforce-better-script-practices-by-using-set-strictmode/

Why

Closes #[Issue Number]

Changes

please update https://www.microsoft.com/en-us/download/details.aspx?id=53337 :-)

I'll share your feedback with the feature PM


Fix OSX build w/CUDA=ON

connect to https://github.com/caffe2/caffe2/issues/1249
With this change, build, install, and smoke test pass on OSX with CUDA=ON.
$ cmake -DUSE_CUDA=ON ..
$ sudo make install
$ python -c 'from caffe2.python import core' 2>/dev/null && echo "Success" || echo "Failure"
Success

Thanks for the fix!


feature：support h2 in AT mode #4133

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
support h2 DataBase
Ⅱ. Does this pull request fix one issue?

fixes https://github.com/seata/seata/issues/4122
Ⅲ. Why don't you add test cases (unit test/integration test)?
Test cases have been added
Ⅳ. Describe how to verify it
The SQL file is the same as mysql
As follows:
CREATE TABLE undo_log (
id bigint(20) NOT NULL AUTO_INCREMENT,
branch_id bigint(20) NOT NULL,
xid varchar(100) NOT NULL,
context varchar(128) NOT NULL,
rollback_info longblob NOT NULL,
log_status int(11) NOT NULL,
log_created datetime NOT NULL,
log_modified datetime NOT NULL,
ext varchar(100) DEFAULT NULL,
PRIMARY KEY (id),
UNIQUE KEY ux_undo_log (xid,branch_id)
) ;
Ⅴ. Special notes for reviews
See for details ：
H2DeleteExecutorTest.java
H2InsertExecutorTest.java
H2DeleteRecognizerTest.java
H2InsertRecognizerTest.java
H2UpdateRecognizerTest.java

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 19 committers have signed the CLA.:x: github-ganyu:x: JavaLionLi:x: miaoxueyu:x: whxxxxx:x: renliangyu857:x: a364176773:x: neillee95:x: Bughue:x: tuwenlin:x: robynron:x: wingchi-leung:x: zhangzq7:x: crazy-sheep:x: odidev:x: zhuyoufeng:x: l81893521:x: doubleDimple:x: zw201913:x: xingfudeshiYou have signed the CLA already but the status is still pending? Let us recheck it.

adapt v.1.6.0


Changed _internalCache access level to protected

Changed amazon\Bucket::$_internalCache access level to protected like in the parent's class.

Hi, Paul!
Met this error after calling $bucket = \Yii::$app->fileStorage->getBucket('some_bucket_name');

Forgot to mention that I'm using PHP7.
Regards.

Resolved by commit f01dc8efb1787a35ad50ab954277e67cf208fbbc