Add signal for process_tasks start

I'm suggesting addition of a new signal, such as process_tasks_started. The signal would be emitted by the process_tasks management command, and would serve as a signal to initialize repeated tasks.
The issue raises from multiple instances of Django running in parallel (multiple web workers, MQTT client etc). When a repeated task is started from AppConfig.ready(), it's done multiple times with the current setup. With a custom signal, the initialization of process_tasks_started the initialization could be tied to initialization of the process queue, at least reducing the possibility of triggering the task multiple times.
I'm planning on implementing this on my own fork, so if this is something to be considered, I'm willing to make a pull request.

This is now implemented in my own fork: https://github.com/kipe/django-background-tasks/commit/32109a87a60249aaf2785cdbbb06b97b4de26bea
As an example usage, the signals.py for one of my apps:
from django.dispatch import receiver
from background_task.signals import process_tasks_started

@receiver(process_tasks_started)
def start_nordpool_fetching(**kwargs):
    from background_task.models import Task
    from raspautomation.plugin.nordpool.tasks import fetch_nordpool

    fetch_nordpool(repeat=Task.HOURLY)

I had the same issue, but I am also running multiple instances of workers, and if I understand correctly that would mean that the issue would still remain, it would just be triggered by the workers and not Django.
In the end I have implemented a custom management command to trigger these repeated tasks and added it to my deployment pipeline.

Yeah, this doesn't solve that problem directly. For my own use I just added remove_existing_tasks=True to the background decorator, which seemed to fix the issue. Needs further testing though.
@background(remove_existing_tasks=True)
def fetch_nordpool():
    logger.info('Task started')


Add key synchronization with a JWKS endpoint.

This change introduces a new routine that runs periodically to update
the HealthAuthority keys with a JWKS endpoint. The daemon uses the key
values to determine if new keys have been introduced or retired. The URI
of the JWKS endpoint is recorded in the HealthAuthority in the database.
The periodicity is controlled with a new environment variable;
JWKS_REFRESH_RATE, which specifies (in seconds) the period the keys are
refreshed.
Fixes #911

Proposed Changes

Introduces a new mechanism to periodically refresh a HealthAuthority's keys from a JWKS endpoint.
HealthAuthority now includes an optional JWKS URI that will be queried to obtain new keys.
The key refresh rate is controlled by a new configuration parameter: JWKS_REFRESH_RATE, which specifies (in seconds) the refresh rate for all HealthAuthorities with specified JWKS endpoints.

Release Note

* Introduces a new mechanism to periodically refresh a HealthAuthority's keys from a JWKS endpoint.
* HealthAuthority now includes an optional JWKS URI that will be queried to obtain new keys.
* The key refresh rate is controlled by a new configuration parameter: JWKS_REFRESH_RATE, which specifies (in seconds) the refresh rate for all HealthAuthorities with specified JWKS endpoints.

/hold
Seems like polling's a mistake. I'll move this to a cached approach. As I read Mike's Issue, with my limited understanding of this server's architecture, I went with a simple periodic polling. My apologies. Most of this code stays the same -- just what drives the update changes.

Seems like polling's a mistake. I'll move this to a cached approach. As I read Mike's Issue, with my limited understanding of this server's architecture, I went with a simple periodic polling. My apologies. Most of this code stays the same -- just what drives the update changes.

See the pattern we use for things like the cleanup and export jobs.
polling is fine, but controlled rather than distributed

Cool, I'll dig around, find it, and move to that approach. Please hold reviews for a little bit.

/unhold
I think this is ready, I still need to test all the terraform and build changes, so there might be some churn there, but otherwise, I believe this can re-enter review.

/lgtm

I don't have OWNERS here, so I need a second reviewer. @mikehelmick

@jeremyfaller it actually looks like merging is failing due to some tests

Because of a spelling mistake in the html, I need a new LGTM, please.

/lgtm


[MinkowskiEngine 0.5.4][ME.RegionType.HYBRID vs ME.RegionType.CUSTOM] ME.RegionType.CUSTOM giving AttributeError: 'NoneType' object has no attribute 'numel'

Hi @chrischoy
I am trying to run the code in presumably written with the 0.4.3 MinkowskiEngine but my version in 0.5.4
https://github.com/chrischoy/SpatioTemporalSegmentation/blob/4afee296ebe387d9a06fc1b168c4af212a2b4804/models/modules/common.py#L58
Which uses  ME.RegionType.HYBRID.
I replaced HYPERCUBE and HYPERCROSS to HYPER_CUBE and HYPER_CROSS respectively. Then I replaced  ME.RegionType.HYBRID with ME.RegionType.CUSTOM  in the above mentioned line.
I am getting the following error
Traceback (most recent call last):
  File "/home/anaconda3/envs/py3-mink/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/anaconda3/envs/py3-mink/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/SandeepMenon/SpatioTemporalSegmentation/main.py", line 156, in <module>
    main()
  File "/home/SandeepMenon/SpatioTemporalSegmentation/main.py", line 117, in main
    model = NetClass(num_in_channel, num_labels, config)
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/res16unet.py", line 339, in __init__
    super(STRes16UNetBase, self).__init__(in_channels, out_channels, config, D, **kwargs)
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/res16unet.py", line 23, in __init__
    super(Res16UNetBase, self).__init__(in_channels, out_channels, config, D)
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/resnet.py", line 25, in __init__
    self.network_initialization(in_channels, out_channels, config, D)
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/res16unet.py", line 58, in network_initialization
    self.block1 = self._make_layer(
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/resnet.py", line 106, in _make_layer
    block(
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/modules/resnet_block.py", line 23, in __init__
    self.conv1 = conv(
  File "/home/SandeepMenon/SpatioTemporalSegmentation/models/modules/common.py", line 128, in conv
    kernel_generator = ME.KernelGenerator(
  File "/home/anaconda3/envs/py3-mink/lib/python3.8/site-packages/MinkowskiEngine/MinkowskiKernelGenerator.py", line 303, in __init__
    self.kernel_volume = get_kernel_volume(
  File "/home/anaconda3/envs/py3-mink/lib/python3.8/site-packages/MinkowskiEngine/MinkowskiKernelGenerator.py", line 91, in get_kernel_volume
    region_offset.numel() > 0
AttributeError: 'NoneType' object has no attribute 'numel'

While trying to create an ME.KernelGenerator
using the following params
ME.KernelGenerator(
      3, 1, 1, region_type=ME.RegionType.CUSTOM, axis_types=[<RegionType.HYPER_CUBE: 0>, <RegionType.HYPER_CUBE: 0>, <RegionType.HYPER_CUBE: 0>, <RegionType.HYPER_CROSS: 1>], dimension=4)

It is because I am not providing region_offsets. Can you help me understand offsets to give in this case?
How to use the ME.RegionType.CUSTOM type in replacement to this?
Else, how to install MinkowskiEngine 0.4.3. I dont see an archive.
Thank you

@sandeepnmenon  hello,MinkowskiEngine 0.4.3 at here : https://github.com/NVIDIA/MinkowskiEngine/releases/tag/v0.4.3

@suyunzzz Thank you
Closing the issue.

Hey @sandeepnmenon,
did you solve the issue in the end? I'm using mink v 0.5 but the HYBRID kernel is not available yet.

@saltoricristiano
No, I could not solve it in v0.5.
I solved the issue by using v0.4.3 in my environment.

Hey @sandeepnmenon,
did you solve the issue in the end? I'm using mink v 0.5 but the HYBRID kernel is not available yet.

I found that Chris did have a branch for v0.5, that can solve the region type update issue. See this commit.


IE 11 adding files events fire twice

I have in view in flow-init the directive
flow-files-added="uploader.actionsWhenFilesAdded($files)"
flow-files-submitted="uploader.actionsWhenFilesSubmitted( $files)"
and i see that the event's fire twice resulting breaking my code that handles the files, since the second call the files parameter is an empty array.
To fix this problem i read the flow.files reference.

Have same problem with IE11, calling files-submitted twice. Any ideas how to fix that ?

@Duskfall https://github.com/flowjs/flow.js/pull/110 this helped me to resolve it, so check the files changed and do it in your library until this fix is pushed to ng-flow as well

Thanks for the reference link @f1ght4fun !

This issue is currently not fixed.


tf.math.segment_max/min/mean/sun/prod crashes(aborts) when segment_ids is large

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):2.1.0
Python version:3.7.6
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A

Describe the current behavior
tf.math.segment_max/min/mean/sun/prod crashes(aborts) when segment_ids is large
Describe the expected behavior
expect an exception message if the input is unexpected instead of crash
Standalone code to reproduce the issue
tf.math.segment_max(data=np.ones((1,10,1)), segment_ids=[1676240524292489355])
tf.math.segment_min(data=np.ones((1,10,1)), segment_ids=[1676240524292489355])
tf.math.segment_mean(data=np.ones((1,10,1)), segment_ids=[1676240524292489355])
tf.math.segment_sum(data=np.ones((1,10,1)), segment_ids=[1676240524292489355])
tf.math.segment_prod(data=np.ones((1,10,1)), segment_ids=[1676240524292489355])

Output:
2021-02-03 16:44:25.849065: F tensorflow/core/framework/tensor_shape.cc:405] Check failed: 0 <= new_num_elements (0 vs. -1684338830784658056)
Aborted (core dumped)

Related issue: #46696

Was able to reproduce the issue with TF v2.3, TF v2.4 and TF-nightly. Please find the gist of it here. Thanks!

Was able to reproduce this issue in TF 2.6.0-dev20210528,please find the gist here ..Thanks !

Added a PR #51733 for the fix.


feat: adding trailing slash to rpc url if there isn't one

I've made a similar change just now :) https://github.com/skyzh/make-a-fortune/pull/18

😢😢😢


Fixed status indicators.

changed package to com.javaslang to match the real groupid in maven
changed icon types to .png (png is more widely available).

Current coverage is 96.00%

Merging #1159 into master will not affect coverage as of e27cd6b

@@            master   #1159   diff @@
======================================
  Files           78      78
  Stmts         9117    9117
  Branches      1713    1713
  Methods          0       0
======================================
  Hit           8753    8753
  Partial         91      91
  Missed         273     273

Review entire Coverage Diff as of e27cd6b

Powered by Codecov. Updated on successful CI builds.

Thank you - right, the current readme should always provide working links.


Getting Build Failure for backend in hosting Env... in case of Backend Env it is fine

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
14.16.1
Amplify CLI Version
10.7.3
What operating system are you using?
windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
Build failed to complete successfully.
Starting phase: build
2023-02-22T11:33:20.907Z [INFO]: Amplify AppID found: d3l7ssfuo42ec5. Amplify App name is: awsamplifycognito
2023-02-22T11:33:20.953Z [INFO]: Backend environment dev found in Amplify Console app: awsamplifycognito
2023-02-22T11:33:21.569Z [WARNING]: - Fetching updates to backend environment: dev from the cloud.
2023-02-22T11:33:22.113Z [WARNING]: - Building resource api/api20eed79a
2023-02-22T11:33:22.138Z [WARNING]: - Building resource auth/awsamplifycognito8f199c27
2023-02-22T11:33:22.248Z [WARNING]: - Building resource storage/dynamof008afce
2023-02-22T11:33:22.272Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2023-02-22T11:33:22.409Z [INFO]: ✅
2023-02-22T11:33:23.933Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2023-02-22T11:33:25.344Z [WARNING]: - Initializing your environment: dev
2023-02-22T11:33:25.943Z [WARNING]: - Building resource api/api20eed79a
2023-02-22T11:33:25.965Z [WARNING]: - Building resource auth/awsamplifycognito8f199c27
2023-02-22T11:33:26.070Z [WARNING]: - Building resource storage/dynamof008afce
2023-02-22T11:33:26.082Z [WARNING]: ✔ Initialized provider successfully.
2023-02-22T11:33:26.202Z [INFO]: Unable to find Java version >=11 on the path. Download link: https://amzn.to/2UUljp9
Unable to find Java compiler version >=11 on the path. Download link: https://amzn.to/2UUljp9
Unable to find Gradle version >=5 on the path. Download link: https://bit.ly/3aGYDj6
2023-02-22T11:33:26.203Z [WARNING]: ✖ There was an error initializing your environment.
2023-02-22T11:33:26.203Z [INFO]: 🛑 Missing required dependencies to package awsamplifycognito9a16b3e8
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-02-22T11:33:26.209Z [INFO]:
2023-02-22T11:33:26.209Z [INFO]: Session Identifier: e74e9d8e-11d7-4047-934b-84431a5e84a6
2023-02-22T11:33:26.229Z [ERROR]: !!! Build failed
2023-02-22T11:33:26.229Z [ERROR]: !!! Non-Zero Exit Code detected
2023-02-22T11:33:26.230Z [INFO]: # Starting environment caching...
2023-02-22T11:33:26.230Z [INFO]: # Environment caching completed
Terminating logging...
Expected behavior
Checking for associated backend environment...
                             ## Backend environment association found
                             ## Running amplify pull to generate aws-exports.js file for frontend

2023-02-13T18:25:30.444Z [INFO]: Amplify AppID found: d2rw3952zq3mlz. Amplify App name is: demoapp
2023-02-13T18:25:30.514Z [INFO]: Backend environment dev found in Amplify Console app: demoapp
2023-02-13T18:25:31.138Z [WARNING]: - Fetching updates to backend environment: dev from the cloud.
2023-02-13T18:25:31.662Z [WARNING]: - Building resource api/demolambdaapi
2023-02-13T18:25:31.743Z [WARNING]: - Building resource storage/dynamo751808a0
2023-02-13T18:25:31.780Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2023-02-13T18:25:31.817Z [INFO]: ✅
## Checking for associated backend environment...
## Backend environment association found
## Running amplify pull to generate aws-exports.js file for frontend
2023-02-13T18:25:30.444Z [INFO]: Amplify AppID found: d2rw3952zq3mlz. Amplify App name is: demoapp
2023-02-13T18:25:30.514Z [INFO]: Backend environment dev found in Amplify Console app: demoapp
2023-02-13T18:25:31.138Z [WARNING]: - Fetching updates to backend environment: dev from the cloud.
2023-02-13T18:25:31.662Z [WARNING]: - Building resource api/demolambdaapi
2023-02-13T18:25:31.743Z [WARNING]: - Building resource storage/dynamo751808a0
2023-02-13T18:25:31.780Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2023-02-13T18:25:31.817Z [INFO]: ✅
Reproduction steps
Step 1:- npx create-react-app app-name
Step 2:- npm install aws-amplify / npm install react-bootstrap / npm install aws-amplify @aws-amplify/ui-react
Step 3:- amplify init
Step 4:- amplify add storage
Step 5:- amplify add function (amplify update function - for future use)
Step 6:- amplify add api
step 7:- amplify push
Step 8:- amplify hosting add
Project Identifier
No response
Log output
Build failed to complete successfully.
Starting phase: build
2023-02-22T11:33:20.907Z [INFO]: Amplify AppID found: d3l7ssfuo42ec5. Amplify App name is: awsamplifycognito
2023-02-22T11:33:20.953Z [INFO]: Backend environment dev found in Amplify Console app: awsamplifycognito
2023-02-22T11:33:21.569Z [WARNING]: - Fetching updates to backend environment: dev from the cloud.
2023-02-22T11:33:22.113Z [WARNING]: - Building resource api/api20eed79a
2023-02-22T11:33:22.138Z [WARNING]: - Building resource auth/awsamplifycognito8f199c27
2023-02-22T11:33:22.248Z [WARNING]: - Building resource storage/dynamof008afce
2023-02-22T11:33:22.272Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2023-02-22T11:33:22.409Z [INFO]: ✅
2023-02-22T11:33:23.933Z [INFO]: Note: It is recommended to run this command from the root of your app directory
2023-02-22T11:33:25.344Z [WARNING]: - Initializing your environment: dev
2023-02-22T11:33:25.943Z [WARNING]: - Building resource api/api20eed79a
2023-02-22T11:33:25.965Z [WARNING]: - Building resource auth/awsamplifycognito8f199c27
2023-02-22T11:33:26.070Z [WARNING]: - Building resource storage/dynamof008afce
2023-02-22T11:33:26.082Z [WARNING]: ✔ Initialized provider successfully.
2023-02-22T11:33:26.202Z [INFO]: Unable to find Java version >=11 on the path. Download link: https://amzn.to/2UUljp9
Unable to find Java compiler version >=11 on the path. Download link: https://amzn.to/2UUljp9
Unable to find Gradle version >=5 on the path. Download link: https://bit.ly/3aGYDj6
2023-02-22T11:33:26.203Z [WARNING]: ✖ There was an error initializing your environment.
2023-02-22T11:33:26.203Z [INFO]: 🛑 Missing required dependencies to package awsamplifycognito9a16b3e8
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2023-02-22T11:33:26.209Z [INFO]:
2023-02-22T11:33:26.209Z [INFO]: Session Identifier: e74e9d8e-11d7-4047-934b-84431a5e84a6
2023-02-22T11:33:26.229Z [ERROR]: !!! Build failed
2023-02-22T11:33:26.229Z [ERROR]: !!! Non-Zero Exit Code detected
2023-02-22T11:33:26.230Z [INFO]: # Starting environment caching...
2023-02-22T11:33:26.230Z [INFO]: # Environment caching completed
Terminating logging...
Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @arpan-banerjee :wave: thanks for raising this! It appears Java is not available in the Hosting build pipeline:
2023-02-22T11:33:26.202Z [INFO]: �[31mUnable to find Java version >=11 on the path. Download link: https://amzn.to/2UUljp9�[39m
�[31mUnable to find Java compiler version >=11 on the path. Download link: https://amzn.to/2UUljp9�[39m
�[31mUnable to find Gradle version >=5 on the path. Download link: https://bit.ly/3aGYDj6�[39m

This issue is similar to https://github.com/aws-amplify/amplify-hosting/issues/2776#issue-1253002245, transferring to Hosting for guidance


Added DeepLink Travel Page to the ToC and changed Filename and Title

Added the Deeplink Travel Page to table of contents.
Also changed the ending of the filename from ".md" to ".markdown" and the Title of the DeepLink Travel Page for a better consistency with other DeepLink Integration Request page

@jak231024 Two things, first the file extension really doesn't matter, but if you want to change it that's fine. Second, we don't add prerelease items to the TOC, and the doc team does that task as part of an overall release process. I'm closing this PR for now, let me know if you're ready for release.


Unit tests

For service:

Create pipeline ✔
Pass test build ✔

For cuckoo.py:

Write unit tests ✔

For cuckooresult.py:

Write unit tests ✔

For signatures.py:

Write unit tests ✔

For safelist.py:

Write unit tests ✔

best of luck folks, unit testing cuckoo is a montrous task


[Feature Request] Allow selection of categories instead of only free-form text

Currently there is no way to select categories for a recipe other than entering free-form text which makes it easy to duplicate categories like "dessert" and "desserts".

This is implemented with version 0.7.6


Fix ETag Handling

We have a few problems with ETag handling:

RFC 7232 sec. 3.1 requires us to use strong validation on If-Match, we use weak.
Even our weak validation is incorrect; see RFC 7232 sec. 2.3.2.
If we use strong validation, clients will never be able to use If-Match (since we don't generate strong ETags).

Discussion of this issue within the LDP WG is in this thread.

The restriction referred to in the email thread linked above was added in https://tools.ietf.org/html/draft-ietf-httpbis-p4-conditional-24. I'm unsure whether the LDP WG ever discussed this development.

The proposed fix is to fail If-Match (Precondition Failed) for all checks.


Add common AO instructions

go_const(t, val)

Implemented go_constant both in nicompiler_backend and card.py in nistreamer.


efcore.jet 2.2.0 database provider error.

I want to use efcore.jet 2.2.0 in my asp.netcore 2.2.0 project.
But I got a exception as below:
System.InvalidOperationException: 'No database provider has been configured for this DbContext. A provider can be configured by overriding the DbContext.OnConfiguring method or by using AddDbContext on the application service provider. If AddDbContext is used, then also ensure that your DbContext type accepts a DbContextOptions object in its constructor and passes it to the base constructor for DbContext.'
My Connection String is
"ConnectionStrings": {
    "MSAccessConnection": "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\\Users\\********\\db\\test.mdb;"
  }

"ConfigureServices" Method:
public void ConfigureServices(IServiceCollection services)
{
    services.Configure<CookiePolicyOptions>(options =>
    {
        // This lambda determines whether user consent for non-essential cookies is needed for a given request.
        options.CheckConsentNeeded = context => true;
        options.MinimumSameSitePolicy = SameSiteMode.None;
    });
    services.AddDbContext<TestDbContext>(options =>
        options.UseJet(Configuration.GetConnectionString("MSAccessConnection")));
    services.AddScoped<UserDAO, UserDAO>();

    services.AddMvc().SetCompatibilityVersion(CompatibilityVersion.Version_2_2);
}

Any one can help me? Is there any Samples?

Probably you can find an answer faster asking on stack overflow.

System.InvalidOperationException: 'No database provider has been configured for this DbContext. A provider can be configured by overriding the DbContext.OnConfiguring method or by using AddDbContext on the application service provider. If AddDbContext is used, then also ensure that your DbContext type accepts a DbContextOptions object in its constructor and passes it to the base constructor for DbContext.'

If you use ASP.NET and register your DbContext with services.AddDbContext<TestDbContext>(), then you should use dependency injection to get an instance of your context.
So do not use code like the following:
public class MyController : ControllerBase
{
    public async Task<ActionResult<int>> GetIceCreamCount()
    {
        // This will throw with your example code.
        using (var dbContext = new TestDbContext())
        {
                var iceCreams = _dbContext.IceCreams.ToListAsync();
                return iceCreams.Count;
        }
    }
}

Instead do something like the following:
public class MyController : ControllerBase
{
    private readonly TestDbContext _dbContext;

    // This contructor uses dependency injection.
    public MyController(TestDbContext dbContext)
    {
        _dbContext = dbContext;
    }

    public async Task<ActionResult<int>> GetIceCreamCount()
    {
        var iceCreams = _dbContext.IceCreams.ToListAsync();
        return iceCreams.Count;
    }
}

You would also need to make sure, that you run the application for the processor architecture of the underlying database provider, which is likely to be 32 Bit for the usual Jet database driver.
To accomplish that in Visual Studio, you would need to make sure, that VS uses the correct version of dotnet, which can be accomplished by adding the following lines to you project file:
<PropertyGroup>
  <RunCommand Condition="'$(PlatformTarget)' == 'x86'">$(MSBuildProgramFiles32)\dotnet\dotnet</RunCommand>
  <RunCommand Condition="'$(PlatformTarget)' == 'x64'">$(ProgramW6432)\dotnet\dotnet</RunCommand>
</PropertyGroup>

Then set you platform target to x86 (either in the project file or in the project properties dialog).
The following program shows, how this would work (here without a controller):

Proof of Concept Application

using System;
using System.Diagnostics;
using System.Linq;
using System.Text;
using EntityFrameworkCore.Jet;
using Microsoft.AspNetCore;
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;

namespace EfCoreJetSample
{
    public class Program
    {
        public static void Main(string[] args)
        {
            CreateWebHostBuilder(args).Build().Run();
        }

        public static IWebHostBuilder CreateWebHostBuilder(string[] args) =>
            WebHost.CreateDefaultBuilder(args)
                .UseStartup<Startup>();
    }

    public class IceCream
    {
        public int IceCreamId { get; set; }
        public string Name { get; set; }
    }

    public class TestDbContext : DbContext
    {
        public DbSet<IceCream> IceCreams { get; set; }

        public TestDbContext(DbContextOptions options)
            : base(options)
        {
        }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.Entity<IceCream>(entity =>
            {
                entity.HasKey(e => e.IceCreamId);

                entity.HasData(
                    new IceCream {IceCreamId = 1, Name = "Vanilla"},
                    new IceCream {IceCreamId = 2, Name = "Chocolate"},
                    new IceCream {IceCreamId = 3, Name = "Strawberry"}
                );
            });
        }
    }

    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Debug.Assert(!Environment.Is64BitProcess);

            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

        public void ConfigureServices(IServiceCollection services)
        {
            services.AddDbContext<TestDbContext>(options =>
                options.UseJet("Provider=Microsoft.ACE.OLEDB.12.0;Data Source=test.mdb;"));

            services.AddMvc()
                .SetCompatibilityVersion(CompatibilityVersion.Version_2_2);
        }

        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }

            app.Run(async httpContext =>
            {
                using (var scope = app.ApplicationServices.CreateScope())
                {
                    var dbContext = scope.ServiceProvider.GetService<TestDbContext>();

                    dbContext.Database.EnsureDeleted();
                    dbContext.Database.EnsureCreated();

                    var iceCreams = dbContext.IceCreams
                        .OrderBy(i => i.IceCreamId)
                        .ToList();

                    Debug.Assert(iceCreams.Count == 3);
                    Debug.Assert(iceCreams[0].Name == "Vanilla");

                    await httpContext.Response.WriteAsync(
                        new StringBuilder()
                            .AppendJoin("\r\n", iceCreams.Select(i => $"{i.Name} ({i.IceCreamId})"))
                            .ToString());
                }
            });
        }
    }
}

In theory, this should all work.

Unfortunately, the provider does not yet work with the .NET Core (or ASP.NET Core) runtime, mainly because it depends on System.Data.OleDb from .NET Framework 4.
This is also stated on the main page of this repository:

This provider [...] does not work under the .NET Core runtime (netcore and netstandard).

Because of that, the provided sample code should not work with the .NET Core runtime at all.

The planned support for EF Core 3.1.0 will make use of the new System.Data.OleDb .NET Core package and will fully work with EF Core 3.1.0.

@bubibubi @lauxjpn Thank you for your explain。For now  i am using System.Data.OleDb .NET Core package to access my MS Access.mdb, is there any release schedule of the JetEFCore3.1 now?

I plan to start working on the upgrade in February.

See #34 for further details.

I plan to start working on the upgrade in February.

See #34 for further details.

Thank you for your reply. Looking forward.


Short solution needed: "How to convert data frame to list" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to convert data frame to list (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to convert data frame to list


I used a starter from https://github.com/antonybudianto/angular2-starter  no luck when adding angularfire2

<!--

IMPORTANT! YOU MUST FOLLOW THESE INSTRUCTIONS OR YOUR ISSUE WILL BE CLOSED.

use this starter
https://github.com/antonybudianto/angular2-starter

just add the module and sug to the todolist (it doesn' thave to do anything) and errors on runtime.

my attempt ad adding a memberlist is here:
[https://github.com/foxjazz/memberbase.git](https://github.com/foxjazz/memberbase.git)

All issues have to follow the issue template provided. I understand this is asking a lot and is generally annoying when you feel your question is straightforward, but just trust that it's critical to keep things streamlined and manageable for our team. I'm closing this for now because it doesn't adhere to these requirements. Feel free to resubmit according to the template, being absolutely sure to include a working, minimal repro of the problem and version info.
Note that how-to questions are better suited for Stack Overflow or a discussion group.


Offer offline support using workbox

This PR addresses issues raised in this issue.
@julienben brought it up to my attention and I thought I'll have a go trying to resolve. (Full disclosure - Julien is a co-worker and my tech lead. 🥇).
This PR removes the current offline-plugin in favour of the workbox-webpack-plugin.
I follow the guidelines of create-react-app and chose an opt-in approach.
Open issues
[-] Missing tests - Will add once I get some feedback and see if the direction is good
[-] Documentation could be improved. Not a native english speaker so might need help
Will appreciate any feedback, comments and anything else that could be improved.
This is my first time contributing to OSS, so please be gentle with me 😄.
NOTE:
A similar branch was opened by me and closed since. This one is aligned with latest upstream dev

...did this ever get merged? Is there a branch available to use this version?

Sorry @t3db0t,
This branch was closed without merging.
The feedback given suggested a different approach.
Will take another swing at it and open a new PR


[question] is react-konva practical for creating user interfaces?

Hey there!
I'm creating a mobile-focused project with React. I want the UI animations and touch gestures to be very performant, and as near to native-like as possible, so I'm looking at implementing some sort of canvas based renderer for the app.
I've boiled down to two choices, react-canvas and react-konva.
The benefit of react-canvas to me seems to be that it's more targeted towards creating interfaces, wheras konva seems less so. That being said, Konva is still actively maintained and developed, while Flipboard seems to have let react-canvas fall behind.
So I wanted to ask, how practical is it to use react-konva for building complete app UI, comparable to react-canvas?
Thanks for any input. :)

Hello. First of all, it really depends on your interface. What exactly do you want to draw? Will it have animations, what kind of animations etc?
But my first answers is no. DO NOT USE react-konva and DO NOT USE react-canvas.
react-canvas is designed for building simple but very performant interfaces (in Flipboard UI they have just images and some text in a long list of posts). The issue here is that the project is not maintained anymore.
react-konva is not designed for building UI (here I mean classic UI - text, icons, buttons, inputs, forms, etc). It is designed for drawing on canvas in "react-style" and it is very useful when you need pixel manipulations, exporting to image, filters.
So for complete app UI I recommend to use our old friends - HTML + CSS.

Hi @lavrton,
Thanks for the reply. Ultimately I agree, good old HTML and CSS are much more suitable for building UI's. Unfortunately we're running into a similar issue that Flipboard ran into, where mobile web performance just isn't there for the kind of almost-native-app-like experience we want to create with heavy animations, dynamic content, etc.
I suppose this could boil down to refining and optimizing our react render stack, but the idea of a canvas lib for building interfaces sounds like it combines the best of both worlds ( performance and precision of canvas, flexibility of the DOM ). But I suppose canvas libs just aren't there yet ( not to mention the various other trade offs, difficulty with text / links etc ).
Thanks for your honesty and candidness!

As I mentioned before, I do not recommend to use react-konva for such purpose. Because I think it will be NOT faster than classic HTML. But I didn't make real tests, it is just my assumption. You can create small prototype and test by yourself, probably it will be good.
Another possible good solution is to create "canvas-ui-ultra-fast-library" by yourself manually (probably webgl performance will help here a lot, take a look into http://www.pixijs.com/). And then build react bridge on top of it (like react-konva is a bridge for Konva).


Fix: Add description to composer.json

This PR

[x] validates composer.json on Travis CI
[x] provides a description in composer.json

💁♂ Running
$ composer validate

on current master yields
./composer.json is valid for simple usage with composer but has
strict errors that make it unable to be published as a package:
See https://getcomposer.org/doc/04-schema.md for details on the schema
description : The property description is required
No license specified, it is recommended to do so. For closed-source software you may use "proprietary" as license.

❗Also note that currently, license information is missing:

@Jean85 What about the license ?

@Jan0707
We should choose one, I suggest MIT.
Also take a look at https://github.com/Jan0707/phpstan-prophecy/community.

Added MIT license in : https://github.com/Jan0707/phpstan-prophecy/commit/61facf7b6aa0328e800905ed1139240a5c7e4733

Thank you, @Jan0707!


importing PathParticle

I'm building a pixi videogame using es6 and babel. I've used this boilerplate https://github.com/magnonellie/pixi-boilerplate and I've added pixi-particles as dependency.
I'm importing pixi and pixi-particles as following:
import * as PIXI from "pixi.js";
import * as PIXIParticles from "pixi-particles";

I've no problem using PIXI.Container or PIXI.Particles.Emitter. But PIXI.particles.PathParticle results to be undifined. How should be included? is it not part of the PIXI.particles namespace

PathParticle is part of an optional submodule, to make the main library lighter for people that don't need it.
After importing pixi-particles, do require('pixi-particles/modules/path-particle');, which should make PIXI.particles.PathParticle available.

What if I do not want to use the CommonJS syntax (using require) but i want to stick to es6 Syntax?

Also using the CommonJS syntax is not working:
import * as PIXI from "pixi.js";
require("pixi-particles/dist/modules/path-particle");

path-particle.js:63Uncaught TypeError: Cannot read property 'prototype' of undefined(…)
This happens around line 63
var s = Particle.prototype;
where Particle is not defined.

I've published version 2.0.0 which has everything in one output file.

thank you ;)


Feature Request: Provide limit for number of regions in control markets

Is your feature request related to a problem? Please describe.
The GeoLift tool considers all regions to be available as a control outside of the forced test regions and any exclusions we set.  However, there can be a cost to tracking all US DMAs, for example, or there can be a need to run multiple tests at one time, so we want to limit the regions that need to be left alone.  There is no easy way to do this in the current implementation.
Describe the solution you'd like
A solve would be to provide a parameter giving an upper bound on the number of regions to include in the control.
Describe alternatives you've considered
An alternative is to provide helper functions that, for the best solution, provide truncated output that orders by weight and keeps only the regions within the upper bound count.  That function would then rerun, say, Market Selection to provide the new fit with the reduced set.
Additional context
The current approach assumes no cost to collecting control data, and also assumes that all regions in the set are eligible to be used as controls (except for the filtering parameters mentioned previously.   While we do want all regions considered, we may have a constraint such that we really want to know out of all the regions, what's the best, say, 10 region, best 9 region, on down  solution that also meets the needs for power, etc.  Running any test that meets our power requirements but has fewer cities involved wins on parsimony, though a small number of control regions may either be difficult to match or have higher probability that a rare event in one region has higher impact in your analysis.  Still, having that upper bound as a parameter allows more data folks to take advantage of the tool to solve resource-constrained tests: I want to consider every DMA in the US to find the best combo for my test... but I can only keep 10 as controls.

Hi @michaelwexler thank you for your time in writing this and your engagement
I'm not sure I understood your request perfectly. My understanding is that you need to find the best combination of up to X DMAs (with X being an arbitrary number) that gives you the highest expected power?
If so, with the functionGeoLiftMarketSelection you are able to define with parameter N the different number of locations to consider for selecting the DMAs that will receive the treatment. By providing a list of number from 1 to X, you will get all the possible combinations of 1, 2, 3,..., X locations and the estimated power, thus satisfying your need (if this is indeed your request)
I will close this thread for now, but if my answer doesn't solve your problem or I missunderstood the problem entirely, please, do reopen this issue.

Thank you, but this isn't quite it. You are talking about how many regions to include in the treatment group.   I am asking for the same capability to limit the control group, aka the "holdout" group.  That is, it's difficult to answer the simple question of "Out of all US DMAs, give me some options of a single treatment region vs. controls of no more than 3 regions".  Instead, GeoLiftMarketSelection() always assumes that, other than named regions to exclude, all submitted DMAs should be added to the control calculation if it adds power.
As I stated, in some cases, there is a cost to measuring a control region.  Therefore, rather than looking for the unfettered best match among all submitted DMAs, I want to see the best options of a constrained set.  I do want all regions available, so I'd keep all DMAs in the submitted data; the suggestions of "just submit fewer DMAs" assumes I know which DMAs are the best to use as tests and controls.
The holdout parameter is a good start, but it refers to the %age of conversions, not actual count of regions.  If I had to guess, I suspect the authors wanted to make an allegory to the RCT concept of 2 condition split percentage, but that's only one aspect of how folks think about wide-region holdouts. Since each region we hold out can be a large opportunity cost (a region is a big unit!), having the ability to simply cap this is helpful.
With this additional context, is there a way to workaround this through clever calls to GeoLiftMarketSelection()?
Due to time constraints, we did a brute force cycling of excluding all but 4 regions, getting best option, and then looping again, eventually permuting through all DMAs.  I suspect there is a better approach.

Ah, now I understand, @michaelwexler , thank you for the clarification.
You are right, there is currently no currently capability to apply that same limitation to the control group. The closest thing to what you described would be a hack using the holdout parameter as you mentioned, but normalizing the number of conversions so that each location/DMA appears to have the same number of conversions (but keeping the daily variance). With this, you can use the holdout to specify the share of regions to use, and if you want a absolute number, then you need to find the share where holdout = limit_number_region / distinct_regions
Still, this would mean that you will not have the benefit of having an estimate based on the number of conversions as a consequence. We will add it to the roadmap of features to add, but it will take some time until this get deployed, as we have restricted resources allocated to GeoLift for now.


git config --get-regexp url.*.insteadof doesn't seem to work in a buildCommand.

Issue description
As per the title. It just mysteriously exits, saying that there is an incorrect number of arguments.
Technical details

System: 17.09pre104410.65593e6 (Hummingbird)
Nix version: 1.11.8
Nixpkgs version: 17.09pre104410.65593e6

Is this fixed?


Update: Issue profile-data

There are no changes in this pull request, so I will close. Please feel free to create a new pull request with changes


Creation of Domain umbrella to tag assets and Glossaries to a higher level grouping

Is your feature request related to a problem? Please describe.
Currently, in the OMD explore tab, all search functionality is related to technical metadata (what's my service, what's my application, what's my table) and it is lacking a Business Context, higher level grouping.
For example, I might want to have all my Product related assets under one logical, easily findable umbrella (my Product systems, my product pipelines, my product related glossaries, my product related tables in the data platform).
Describe the solution you'd like
Creation of Domains tag, but which is more exposed to the end-users.  Provides "I know where to click straight away" to the user.
So by clicking on the domain, I am taken to the explore tab, but everything I see is already pre-filtered by the chosen domain. This would really improve user experience as they are exposed to assets that is relevant to them. Especially good for business side users of specific function.
Describe alternatives you've considered
Using tags, however tags are not very "in you face" obvious, and we have a lot of tags and using them for initial search is not convenient.
Additional context

@DovileKr please see a more comprehensive implementation of Domains that adds domain as a first-class entity instead of depending on Tags or Glossaries in #12142. Please provide feedback on that issue. I am closing this issue for now.

Feedback
Domains and Glossaries to be  provided together "For example, I might want to have all my Product related assets under one logical, easily findable umbrella (my Product systems, my product pipelines, my product related glossaries, my product related tables in the data platform)."
Domain examples that we have provided based on datahub features:  https://github.com/open-metadata/OpenMetadata/issues/10327


Background scrollable while sliding Side Menu

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[X] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[X] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
While sliding a Side Menu, it's still possible to scroll the current page. This feels and looks wiered sometimes. This behavior occured for me on devices (android/ios) only.
Expected behavior:
As soon as the Side Menu begins to slide, the current page should be looked or sth similiar.
Steps to reproduce:
Works with the simplest side menu of ionic.

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (C:\Users\SCHUL256\AppData\Roaming\npm\node_modules)

    @ionic/cli-plugin-proxy : 1.4.12
    @ionic/cli-utils        : 1.13.0
    ionic (Ionic CLI)       : 3.13.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : none
    Ionic Framework    : ionic-angular 3.7.1

System:

    Node : v6.11.3
    npm  : 3.10.10
    OS   : Windows 10

Misc:

    backend : pro

Hello! Thank you for opening an issue with us! I am closing this issue because it is a duplicate of Issue #13282. Please track that issue for further updates. Also please comment on that issue with any new information you may have to share.  Thank you for using Ionic
The only difference I see is that #13282 does not occur on Android. If you could provide a sample app via GitHub where this occurs also in Android that would be very helpful. I will also re-test #13282 on an Android device when I get a chance just to make sure.


Customer feedback |  az vm run-command invoke with RunPowerShellScript does not fail with Exit 1

This customer suggestion is being moved from MicrosoftDocs 4542. Please route to VM engineering team.
Reference command
az vm run-command invoke
Customer request summary

Show the exit code from my script returned in the json if that's possible.
Allow the command to accept --fail-on-non-zero, to propogate the exit code of az vm run-command-invoke back to me
document some best practice for error handling around az vm run-command invoke with both bash and powershell.**

Feedback
I'm using the following form to run a powershell script on a virtual machine in Azure.
az vm run-command invoke  --command-id RunPowerShellScript --name win-vm -g my-resource-group \
    --scripts @test/bad.ps1 --parameters "arg1=somefoo" "arg2=somebar"

The trouble is, the returned JSON doesn't tell me that my script has exited 1:
param(
    [string]$arg1,
    [string]$arg2
)
Write-Host "[$Env:COMPUTERNAME] This is script should fail with parameters $arg1 and $arg2"
Exit 1

Yields:
{
  "value": [
    {
      "code": "ComponentStatus/StdOut/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "[win-vm] This is script should fail with parameters somefoo and somebar",
      "time": null
    },
    {
      "code": "ComponentStatus/StdErr/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "",
      "time": null
    }
  ]
}

When I throw an exception, I do get some error information in the second value object returned by the CLI:
param(
    [string]$arg1,
    [string]$arg2
)
Write-Host "[$Env:COMPUTERNAME] This is script should fail with parameters $arg1 and $arg2"
# https://learn.microsoft.com/en-us/powershell/scripting/learn/deep-dives/everything-about-exceptions?view=powershell-5.1#working-with-exceptions
# https://powershellexplained.com/2017-04-07-all-dotnet-exception-list/#systemargumentexception
Write-Error -Exception ([System.ArgumentException]::new("[$Env:COMPUTERNAME] I am not a happy script with parameters $arg1 and $arg2")) -ErrorAction Stop
Exit 1

Yields:
{
  "value": [
    {
      "code": "ComponentStatus/StdOut/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "[win-vm] This is script should fail with parameters somefoo and somebar",
      "time": null
    },
    {
      "code": "ComponentStatus/StdErr/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "C:\\Packages\\Plugins\\Microsoft.CPlat.Core.RunCommandWindows\\1.1.18\\Downloads\\script0.ps1 : [win-vm] I am not a \nhappy script with parameters somefoo and somebar\nAt C:\\Packages\\Plugins\\Microsoft.CPlat.Core.RunCommandWindows\\1.1.18\\Downloads\\script0.ps1:8 char:1\n+ Write-Error -Exception ([System.ArgumentException]::new(\"[$Env:COMPUT ...\n+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n    + CategoryInfo          : NotSpecified: (:) [Write-Error], ArgumentException\n    + FullyQualifiedErrorId : System.ArgumentException,script0.ps1\n ",
      "time": null
    }
  ]
}

**I think I'm asking for a few things:
💡  It would be nice if I could see the exit code from my script returned in the json if that's possible.
💡  What would be super duper is if I could pass --fail-on-non-zero, to propogate the exit code of az vm run-command-invoke back to me
📚 I think it would be helpful to document some best practice for error handling around az vm run-command invoke with both bash and powershell.**
I am sure I can cobble something together like checking result.value[1].message.length === 0 && result.value[1].code ==="ComponentStatus/StdOut/succeeded" but it feels fragile.
Here's what I feel I can safely assume:
results="bad.json"
az vm run-command invoke --command-id RunPowerShellScript --name win-vm -g win-rg --scripts @test/bad.ps1 --parameters "arg1=somefoo" "arg2=somebar" > $results
#!/bin/bash
results=$1
if [ ! -f "$results" ]; then
    echo "##[error] could not find $results"
    exit 1
fi

stdout_output=$(cat $results | jq -r '.value[0].message')
stdout_result=$(cat $results | jq -r '.value[0].code')
if [ "$stdout_result" = "ComponentStatus/StdOut/failed" ]; then
    echo "##[error] could not read stdout"
elif [ "$stdout_result" = "ComponentStatus/StdOut/succeeded" ]; then
    if [[ -n $stdout_output ]]; then
        echo $stdout_output
    fi
fi

stderr_output=$(cat $results | jq -r '.value[1].message')
stderr_result=$(cat $results | jq -r '.value[1].code')
if [ "$stderr_result" = "ComponentStatus/StdErr/failed" ]; then
    echo "##[error] could not read stderr"
elif [ "$stderr_result" = "ComponentStatus/StdErr/succeeded" ]; then
    if [[ -n $stderr_output ]]; then
        echo "##[error]$stderr_output"
    fi
fi

Page URL
https://learn.microsoft.com/en-us/cli/azure/vm/run-command?view=azure-cli-latest
Content source URL
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/vm/run-command.yml

Thank you for opening this issue, we will look into it.


Update remote-patient-monitoring.md

@stversch : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; merging.


DOC: stats.mood: validity only when observations are unique

Reference issue
gh-13730
What does this implement/fix?
The scipy.stats.mood test statistic (and p-value) is not defined correctly when there are repeated observations / ties, and the fix is not trivial. In the meantime, this documents that the implementation is only valid when all observations are unique.

Yeah p-values not adjusted for ties don't bother me as much, but this test statistic needs a more general definition, too.


Update recipe for py312_patch

Preparation for py312 release.

Linter check found the following problems:
The following problems have been found:
ERROR: build.sh:31: uses_setup_py: python setup.py install is deprecated.
ERROR: clone/recipe/meta.yaml:31: cython_needs_compiler: Cython generates C code, which will need to be compiled
ERROR: clone/recipe/meta.yaml:31: cython_needs_compiler: Cython generates C code, which will need to be compiled
ERROR: clone/recipe/meta.yaml:37: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:38: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:67: missing_imports_or_run_test_py: Python packages require imports or a python test file in tests.
Errors were found

won't do


Use more strax.RUN_METADATA_PATTERN

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Following https://github.com/AxFoundation/strax/pull/893
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coverage: 90.379% (+0.02%) from 90.36%
when pulling f622d72c4094862a2d0cc91304872abbd865ff94 on run_metadata_pattern
into b12383ceec16487e7f7986afce5c093d5f1f2157 on master.


CKA Lecture Request

Port naming / named ports
Patching a deployment

Hi @jmaczura
Please raise your request in the community forum here https://community.kodekloud.com/c/kubernetes/
Thanks


Prevent deprecation warnings

Closes #48.

This has been rebased on main.

@starbelly, this need a review to be mergeable. 😄


[bug]: can not modify the height, weight and Impedance options on the GUI

me too, and I have to delete and re-add person


Fix double free heap corruption bug.

Fix double free heap corruption bug. We had code that passed a raw pointer from one unique_ptr to another unique_ptr, causing that pointer to be freed twice.
Changes filterAndDedup to filter and dedup in place, and to consume the input vector.
cc @soam-stripe
Motivation

I believe this will fix https://github.com/sorbet/sorbet/issues/3737
Test plan

idk why CI didn't catch this before.

Actually, I just realized... that should invoke the copy constructor. 🤦

Maybe this PR is worth merging anyway for the in-place filter?


Any page with search results (multiple items) loads very slowly when accessed via direct link

Unsure if the root cause is on backend (REST API) or frontend, but logging this here for further analysis.
If you access pages via direct link (paste URL in browser) which include search results, they load very slowly (anywhere from 10-15 seconds before you see anything appear).  This behavior seems worse when accessing Entities like Person or Journal (as both of these display search results on their page).
For example:

Person: https://dspace7-demo.atmire.com/items/b1b2c768-bda1-448a-a073-fc541e8b24d9
Journal: https://dspace7-demo.atmire.com/items/a23eae5a-7857-4ef9-8e52-989436ad2955

You can also see this behavior by just trying to directly access the first page of search results, e.g. https://dspace7-demo.atmire.com/search?page=1
However, a normal Item (one that does not have search results embedded) loads rather quickly (2 seconds or less), e.g. https://dspace7-demo.atmire.com/items/82b9dfd6-038e-4472-8749-0a1ce87aa869
Additionally, if you access any of the above pages by clicking on a link (after a prior search or browse) in the Angular UI, then the page loads rapidly.  This performance issue only seems to occur when accessing these pages via a direct link.
At this time, it is unclear if the performance issues are with Discovery (backend) or with the Angular server-side (pre)compilation, or a combination of both.  This behavior needs more analysis. If the performance issue is found to be in Discovery (backend), this ticket should be moved to JIRA so that it can be resolved in the REST API.

The problem remains and makes the system unusable. The initial download is very long.

Pulling this over to the 7.4 board to see if we can get some analysis done.  It's still reproducible on latest code...but seemingly only when Entities are used and have embedded search results.


test on bird dataset

i test sqlcoder model on BIRD dataset,only achieve .15 accuracy, anyone have ever test it?

The BIRD dataset is in SQLite, while SQLCoder is optimized for Postgres. You will have to fine tune it for SQLite before testing on BIRD for best results

The BIRD dataset is in SQLite, while SQLCoder is optimized for Postgres. You will have to fine tune it for SQLite before testing on BIRD for best results

thank you! and how to finetune sql coder on sqlite, is there any open source framework or project?

Hi! @steph730,
We have fine-tuned CodeS (our pre-trained LLM specifically optimized for SQL generation) on Bird, and it has shown impressive performance on Bird's development set, surpassing even GPT-4. The fine-tuned checkpoints are now available for download and immediate use!
For more details, you can check CodeS.


[typescript-operations] add description and @deprecated comments with preResolveTypes=true

Is your feature request related to a problem? Please describe.
We use preResolveTypes since I dont want the devs to accidentally use the graphql types witch happens quiet often recently (maybe an option to dont export them could also help here). I read recently that deprecations are added to the generated files via @deprecated comment. So Ive looked them up and found they are only added to the Graphql types but not to the operation types with preResolveTypes. Even more there are no comments added to the operation types.
Describe the solution you'd like
I think it would be a good Idea to also add them to the Operations or at least when preResolveTypes is true.
Describe alternatives you've considered
I have no idea. Description and deprecated could be very helpful.

Another possible option for achieving the same practical result would be a new option for generating, but not exporting, the non-operation types. Then you could use "preResolveTypes": false and "exportOperationTypesOnly": true.
Properties of "operation types" will be picked from the non-exported non-operation types and therefore "inherit" the documentation and @deprecated tags.
I think it would be overall better for comments to be propagated to pre-resolved types, but this may possibly be a quicker solution?


Allow dockerfile to build

The current dependency<EMAIL_ADDRESS>requires node >=18. Tested build and runtime.

🙏 Thanks for spotting the issue and the fix!


Feature: Preact support

In the spirit of keeping everything as light and fast as possible, we should allow for native support of Preact!

Yeah, it could be worth expanding the scope of this plugin to eleventy-plugin-jsx at some point...

@SimeonGriggs FWIW, I also published https://github.com/kaicataldo/eleventy-plugin-jsx. This is only for using JSX to generate static HTML, though, so not sure if the addition of Preact in it is a huge win.

That's cool, love it.

With a big focus of this project being to balance DX with performance, I do think moving to Preact is inline with the project goals. Might be worth exploring sooner rather than later! If we do want to do that, does it make sense to combine this and eleventy-plugin-jsx into one project? I think we'd probably drop support for React and make this a more batteries-included experience.


Add Fire Hydrant connector

Description

Provide description here

Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

Hi @willmarks, Can you help to advice for test issues:
Non-abstract class 'FirehydrantIncidents' does not implement inherited abstract member 'source' from class 'FirehydrantConverter'.
It just got issue when I updated yesternight, it worked before

Hey @huongtn, I believe the issue is related to this. It looks like all converters need to supply the source rather than the source being inferred from the class name.


Check with FE why the login windows is not being closed at the end of the login

Is it a BE problem?

@BarVolunteering is this still relevant?

No


Bug Report: help

Ok, sorry for keep asking for help but I dont know why it is not working when i build 1.6, 1.8 and the master branch, I get the same problom, It runs but I go to the file and there is no .exe file, can you guys help?

As I said it could be by an antivirus problem. When I compile the game avast alerts me of kade engine as a virus. Make an exception to Kade Engine.exe it should solve your problem.

As I said it could be by an antivirus problem. When I compile the game avast alerts me of kade engine as a virus. Make an exception to Kade Engine.exe it should solve your problem.

So what blocker do you use to allow it?

Go to your antivirus' virus chest, right click in Kade Engine.exe and select make exception.

Go to your antivirus' virus chest, right click in Kade Engine.exe and select make exception.

ok so quick question would it work with windows anti virus?

Go to your antivirus' virus chest, right click in Kade Engine.exe and select make exception.

I have the same problem it still starts up but there is no exe file it is just the other stuff it was post turn into an exe (i did this in 1.6 btw)

Go to your antivirus' virus chest, right click in Kade Engine.exe and select make exception.

do you know a fix?

The .exe from export/windows/bin disappears right?

The .exe from export/windows/bin disappears right?

uhhh... where is that? (sorry i am new to this)

The .exe from export/windows/bin disappears right?

also I never saw an exe file in the kade engine -1.6 file


[Ready to Review] API User Update - Blank but not empty full name

Purpose
Issue: https://openscience.atlassian.net/browse/OSF-4578
If you try to update a user with a blank but not empty full_name, you get an uncaught 500 ValidationValueError.
Changes
Uses a try-except, identical to how we handle Node create/update validation and raise an InvalidModelValueError(400) instead.

CR1 complete.
Requesting clarification on possibly duplicated test. No other change actions requested.
Confirmed with @pattisdr that source:null is an intended-or-acceptable behavior (because this is a very rare error caught only on model save, rather than by the DRF serializer).

CR1 complete.
Thanks for making the tests more specific wrt failure reasons! :octopus:

Looks good to me @pattisdr! CR2 complete :palm_tree:


[CSBindings] Prefer conjunctions over closure variables without bindings

If a closure doesn't have a contextual type inferred yet it should be delayed in favor of
already resolved closure conjunction because "resolving" such a closure early could
miss result builder attribute attached to a parameter the closure is passed to.
Partially resolves https://github.com/apple/swift/issues/67363

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility release

Release source compat failures are unrelated and happen on main as well - Assertion failed: (next->getType() == type && "explosion doesn't match payload schema"), function operator(), file EnumPayload.cpp, line 166.

@swift-ci please test source compatibility release


[T5A2][F11-B1] Justin Ng

Ready for review.

Some comments added. Please close the PR after reading comments.


Fixes for pandas 1.3.0

Fixes #1917 (at least works around it)

Just a heads up, it looks like Pandas 1.3.3 might break things again, was experiencing errors that I was able to resolve by downgrading. I can create a new issue if you'd like.
Error below so you can determine if this is the same issue or not:
    adata.obs['log_counts'] = np.log(adata.obs['n_counts'])
  File "/opt/conda/lib/python3.8/site-packages/pandas/core/frame.py", line 3612, in __setitem__
    self._set_item(key, value)
  File "/opt/conda/lib/python3.8/site-packages/pandas/core/frame.py", line 3784, in _set_item
    value = self._sanitize_column(value)
  File "/opt/conda/lib/python3.8/site-packages/pandas/core/frame.py", line 4509, in _sanitize_column
    com.require_length_match(value, self.index)
  File "/opt/conda/lib/python3.8/site-packages/pandas/core/common.py", line 531, in require_length_match
    raise ValueError(
ValueError: Length of values (1) does not match length of index (38978)


Resolving AttributeError when using ModelChain with multiple Arrays

[x] Closes #1759
[x] I am familiar with the contributing guidelines
[x] Tests added
[ ] Updates entries in docs/sphinx/source/reference for API changes.  -- Doesn't seem to be relevant for PVSystem
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[ ] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

@cwhanse, these all look like formatting changes that I can accept right here.  However, would like to understand the reason for these updates.  For example:

Why do we need the line to begin with a space?
Is there a setting that I need to tweak in my linter / yapf to format it according to this standard?  The only reference to spaces I see in the Contributing  page is
Set your editor to strip extra whitespace from line endings. This prevents the git commit history from becoming cluttered with whitespace changes.
Thanks

I noticed that the Contributing document has 2 instances of PVSystem instantiation, which are just defined but not executed, so no error would come up during the build.  Added the new required temperature_model_parameters argument in the 2 places of instantiation.

Why do we need the line to begin with a space?

It may be unnecessary but it makes the text file easier to read.

Why do we need the line to begin with a space?

It may be unnecessary but it makes the text file easier to read.

Those RST files are used to build our documentation, and the spaces are necessary for that multi-line text to be considered a single bullet item instead of a bullet followed by a new paragraph.  That file isn't included in our docs build yet, but you can check that currently it is not rendering correctly using GitHub's viewer: https://github.com/matsuobasho/pvlib-python/blob/main/docs/sphinx/source/whatsnew/v0.10.4.rst

In the current behavior, a PVSystem can be instantiated without these inputs. In this PR, a ValueError would be raised.

I still need to read and digest the discussion in #1759, but on the face of it, this seems not ideal to me.  Currently it's possible to instantiate a PVSystem with no arguments at all.  Seems strange to me for these temperature model inputs to be the only required inputs.

@cwhanse please confirm my interpretation that it's best to wait without making any updates to this until further word

@matsuobasho yes, we're sorting out what the desired behavior should be.


Particle System

Added ParticleSystemComponent.
Only Cone shaped right now.
Particle size, speed, emision rate and lifetime can be configured.
Size and speed can be set with a bezier curve
CoreLib updated to c++ 17

Task linked: CU-86by2420x Particle System


Error on console for Select tag element

Describe the bug
Getting error on console for Select tag element
Uncaught TypeError: Cannot read property '4' of undefined
    at eval (Select.js:33)

To Reproduce
Steps to reproduce the behavior:

Taken select code from https://github.com/resin-io-modules/rendition/blob/master/src/stories/Select.js
Code:

import React from 'react'
import { Select } from 'rendition'

const SelectComponent = props => {
	return (
    	<Select>
          <option value={1} key={1}>Option 1</option>
        </Select>
    )
}
export default SelectComponent

Reload the page
See error on devtool console tab

Expected behavior
Should render Select box, should not give error
Specifications (please complete the following information):

Browser: Chrome
OS: OSX 10.11.2
Environment: devenv

Screenshot
Accessing theme element, while theme is empty object

@Rahul-Sagore Are you wrapping your app in the Provider component?

@LucianBuzzo No, using simple React Component. The code mentioned in above description is what I have. I am able to use other components (Input, Dropdown, Table). But Select is breaking.

@Rahul-Sagore You need to wrap your entire react application in the Provider component so that rendition components can correctly inherit the theme property, see my comment here for more detail https://github.com/resin-io-modules/rendition/issues/508#issuecomment-406983002

@Rahul-Sagore did wrapping your components with a <Provider> fix the issue?
Should we close this?

Thanks, it is working now.
I had followed the documentation on https://resin-io-modules.github.io/.
From there I saw Select example on: stories/Select.js and there was no info about using Provider. Other people also might get confused. Need to change documentation/Examples if possible.

Linking to #516 for the <Provider> part in the stories and closing for now.


[L4-3] Lighthouse 4 Complaints Module

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-
Changed

Screenshots

@FleetAdmiralButter I need to close some of these old PR's - can I close this one?

I don't think this is going anywhere @dylankelly, I'll get rid of it.


App: container doesn't calibrate if container has label

overview
Bug: Protocols that load containers with a label does not save the calibration and move to the next container during calibration.
current behavior
If I use the OT logo protocol (http://protocols.opentrons.com/protocol/Opentrons_Logo) (with a p300 and tube rack) the calibration procedure is completed as normal. But if I change the container loading to utilize the label parameter then after doing the calibration for the tiprack, and clicking 'move to labware' it opens the calibration window and I can jog but when I press 'save calibration' the calibration window closes briefly but then re-opens again. The only way to get out is by hitting the 'back' button but the container isn't calibrated
steps to reproduce
Use the attached protocol in the app and complete the calibration
Opentrons_Logo_wNames.txt
expected behavior
It should complete the calibration similar to when labels aren't use on the labware

Unable to reproduce on current edge with virtual smoothie. Will continue to investigate with a real robot

@buffavaholic I'm unable to reproduce the described behavior with the protocol provided. Are you on the latest stable app + robot API + robot firmware versions?

App: 3.2.0 (More > App > Information > Software Version)
API: 3.2.0 (Robot  > {your robot's name} > Information > Server version)
Firmware: edge-6168d32 (Robot  > {your robot's name} > Information > Firmware version)

Also, what OS are you running?

@mcous I was using an edge from a couple weeks ago. I just updated and don't see the issue any more. Thanks! (Now works on App 3.2.0 API 3.2.0 and firmware edge-6168d32)


Display challenge description over diffView

Issue: The reviewer has to open two tabs, one for the submission code, and another for the challenge description.
Solution: Display the challenge description over the diffView using an accordion.

Duplicate of #1154


nextcloud: misc changes

Motivation for this change

Use php8 where possible
Run tests against all major versions

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Please note that the mysql test will be fixed in #140089.


Windows maximize

Current coverage is 63.00% (diff: 30.00%)

Merging #80 into master will decrease coverage by 0.55%

@@             master        #80   diff @@
==========================================
  Files            22         22
  Lines           999       1011    +12
  Methods           0          0
  Messages          0          0
  Branches         83         88     +5
==========================================
+ Hits            635        637     +2
- Misses          339        349    +10
  Partials         25         25

Powered by Codecov. Last update 0b7e360...0dc0744


Why the default flex-direction is CSS_FLEX_DIRECTION_COLUMN

https://developer.mozilla.org/en-US/docs/Web/CSS/flex-direction
In this mozilla specification, the initial value for flex-direction should be "row". Why the implementation in css-layout is "column" in init_css_node function?

While we try to stick to the specification in most cases, especially when it comes to the implementation of the algorithm we have chosen to change the default values to values we thought were more sensible / common when developing mobile UIs. I see a future where we might have two init_css_node functions. One with the defaults specified in the w3c specification and one with the default we have found work best for us. No timeline on this though but I would happily accept a pull request which implemented this.


Make-up of the group that doesn't have access to a browser version that is "in Baseline"

There is ongoing research and discussion around a definition for Baseline that uses marketshare as a primary metric.
These typically focus on majority market share. e.g. is 90% sufficient?
But I personally think the make-up of the remaining share needs to also be explicitly discussed and researched.
Any definition of Baseline will split the entirety of users into two groups :

uses a browser version that is "in Baseline"
uses a browser version that is "not in Baseline"

If Baseline is widely adopted it might become progressively worse to browse the internet with a browser version that is "not in Baseline".
This in itself is fine.
It is not possible to support all "legacy" versions in current or new projects that are actively being worked on.

However there is some correlation between socioeconomic factors and the browser version you are using. In short and without any nuance: if you are not elderly, not a child and wealthy you are much more likely to have access to a device that is up to date and a device that is up to date is much more likely to have a browser version that is up to date.
Any definition of Baseline should be sensitive to this reality and should take measures to prevent making access to online content harder for specific groups.

I think there are two important questions, both are very hard/impossible to answer. But I think they still need consideration :

will developers support more or fewer browser versions when adopting Baseline?
if developers would support fewer browser versions, does this have a higher impact on specific groups?

It is entirely possible that most developers only support latest Chrome today and that adopting Baseline would in fact widen support.

The minutes are not being posted/linked back here on GitHub, making it harder to follow this WG async. Maybe this was already discussed? If so, can someone link to the notes?

The minutes are not being posted/linked back here on GitHub, making it harder to follow this WG async. Maybe this was already discussed? If so, can someone link to the notes?

Meeting agenda and minutes are in a Google doc. The doc is linked from the group's home page. I try to link back meeting notes to GitHub issues, as done in https://github.com/web-platform-dx/feature-set/issues/240#issuecomment-1655361317 but there certainly is plenty of room for improvement :)

Forgot to add that the group also actively exchanges on baseline on its Matrix channel.

Thank you for those @tidoust 🙇
I am maybe a bit to accustomed to the csswg's way of recording conversations :)
If it's not on GitHub then it's harder for me to keep track of.
I will try to periodically read up on the latest additions in the Google doc.

I think this is still relevant, even with the current definition.
Finding good ways to get the perspective of vulnerable groups and the browser versions they have access to feels important to me.


Ledger API needs to support GetTransaction by TCert

The ledger API should provide the ability to retrieve transactions by TCert so that

transactions for a particular entity can be retrieved quickly
UTXO like functionality can be built into chaincode

API example
GetTransactionsByTCerts([]tcert) []transactions

@srderson is this still valid? If so, please open a jira and close this one. Thanks


Adding video capture preview function

To visualize the video online, we need to save the video data. It's better to add a preview function to view the video without saving the data.

@alexpiet Yes!


Add overrun behavior to the PopupMenu items.

Add support for overrun behavior / max. width to the PopupMenu items.

I agree, but since both use a PopupMenu internally I guess this implementation makes sense?

It can be added to the MenuButton and OptionButton (to auto set width and overrun for the underlying menu). But since the actual list is drawn by a PopupMenu, it's still necessary to have it here (no necessary to expose it, but why not).

So I think this issue is not relevant to popups, but the buttons only.

Not sure if it's relevant for popups, but I'll make a separate PR for the overrun of the main button text.

TheTileMap issue should be already solved by the other PR, and it's probably not that useful for popup itself.


ESP32 SD Card Memory Allocation Inefficient & Crash Prone, Fix Suggested (IDFGH-4794)

Hello,
I have found a small change to sdmmc_cmd.c which should lead to performance increases and decreased probability of crashes. If you look into sdmmc_read_sectors, you will notice the following:
tmp_buf_read = heap_caps_malloc(block_size, MALLOC_CAP_DMA);
followed by
free(tmp_buf)
This DMA memory is allocated / deallocated every single time a SD card sector is read from. For applications similar to video streaming, this can be many, many times per second. When an application is low on DMA memory, this constant allocation / deallocation will often end up triggering
if (tmp_buf == NULL) { return ESP_ERR_NO_MEM; }
In the vast majority of cases, higher level functions call sdmmc_read_sectors with a block_size of 512 bytes. Taking this into account, it is far faster to allocate 512 bytes of DMA memory one time globally, and then use it each time sdmmc_read_sectors is called (the same 512 DMA bytes can be used for sdmmc_write_sectors). After making this change in sdmmc_cmd.c, my issues vanished

Hello, do you still have any information on how big was the improvement? I am trying to do the same (on IDF v5.0 and ESP32-WROVER-KIT) however I don't seem to get any speed up from this.

Closing because of no improvements detected and user not responding.


Random Battle: Split Castform into Sunny and Rainy Formes

Also implement fix for Meloetta starting out in Pirouette Forme in Random
Doubles Battles.

All right, fair enough. Do the sets look good?

Blah, I'll need to do some tier things in formats-data.js with Castform as well, won't I

No, I don't think you do.
Ice Beam is better on Sunny than Thunderbolt because SolarBeam hits Water already.

I was using Tbolt as a neutral move to hit Fire with a little bit, honestly. If you think Ice Beam is better then I'll change it when I get home.

Neutral is gonna be very weak. I almost wanted to suggest Ice Beam for Rainy so that it has BoltBeam but that is extremely weak without STAB. So yeah, I'd say just switch Bolt for Beam on Sun.

Right, because Fire/Grass is ALSO walled by Dragon, which also walls Electric but is weak to Ice. All right, I'll fix it when I get home.

Also, if you could, remove https://github.com/Zarel/Pokemon-Showdown/blob/master/data/scripts.js#L1347 and add case 'sunnyday': to https://github.com/Zarel/Pokemon-Showdown/blob/master/data/scripts.js#L1331 and remove the check for hasMove['sunnyday'] in that. Rain Dance + Sunny Day was only relevant to Castform.

So, like this.

Also I was considering Fire Blast/Hydro Pump for the Castform formes, but Castform without Weather Ball seems just... wrong

I meant 08f108aefdfa487f4e182cc61061aea29bb90cbb.


allow packages and local packages to be used w/terraform

i don't think this has been integrated yet; if it has we just need to document it

I am currently troubleshooting the result image which looks like it is broken somehow
arknable@arknable-x1c:~/.../ops$ ops instance logs test-pkg-1624003633
ffff80000015fde0:   ffffffff8006e28f	(register_special_files +<PHONE_NUMBER>0006af/00000000000006ea)
ffff80000015fea0:   ffffffff80085a8f	(init_unix +<PHONE_NUMBER>00027f/00000000000004be)
ffff80000015ff10:   ffffffff80021b16	(startup +<PHONE_NUMBER>000046/00000000000009fa)
ffff80000015ffc0:   ffffffff80020877	(runloop_internal +<PHONE_NUMBER>000177/0000000000000561)
assertion program failed in /home/eyberg/go/src/github.com/nanovms/nanos/src/unix/special.c: register_special_files() on line 120; halt

This is even without any argument and using provider,BuildImageWithPackage() which currently defaulted to onprem.
I will continue after fixing https://github.com/nanovms/ops/issues/1086.

That panic is due to the manifest missing the "program" entry which has the path to the elf binary to be executed.

ahh I see, thanks @sanderssj

I'm currently troubleshooting following error
│ Error: Plugin did not respond
│
│   with ops_image_package.hello,
│   on basic.tf line 13, in resource "ops_image_package" "hello":
│   13: resource "ops_image_package" "hello" {
│
│ The plugin encountered an error, and failed to respond to the plugin.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.

test function works though and no details in the log file.

probably the plugin crashed
2021-06-20T16:46:13.828+0700 [ERROR] plugin.(*GRPCProvider).ApplyResourceChange: error="rpc error: code = Unavailable desc = transport is closing"
2021-06-20T16:53:24.431+0700 [ERROR] plugin.(*GRPCProvider).ApplyResourceChange: error="rpc error: code = Unavailable desc = transport is closing"
2021-06-20T17:05:25.041+0700 [ERROR] plugin.(*GRPCProvider).ApplyResourceChange: error="rpc error: code = Unavailable desc = transport is closing"
2021-06-20T17:10:32.146+0700 [ERROR] plugin.(*GRPCProvider).ApplyResourceChange: error="rpc error: code = Unavailable desc = transport is closing

Confirmed, it crashed when calling flags.MergeToConfig().

Confirmed, it crashed when calling flags.MergeToConfig().

Fixed


Reserve Motorcycle - Backend

To reserve an motorcycle, the user has to select a date and city (username and selected item are autofilled).
Use the design based on the "Book a vespa test-ride" and add all necessary inputs.
The user can also access the "Reserve" page from the navigation panel. In that case only username is autofilled.

This taks is assigned to Omachi


Japanese translations from #67

This extracts the Japanese translations from #67.

I've reviewed 22 of 112 files; will resume later.


Add Everett Community College

@Archiamycin I am sorry, the domain everettcc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Pack at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Avoid deleting PV-backing vsphere volumes on instance delete

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #394
Fixes #356
Special notes for your reviewer:
NONE

/assign @mrIncompetent

/approve
/lgtm

/hold

/hold cancel


Is it published on crates?

I have error:
error: no matching package named `btree` found

and
https://docs.rs/btree/
is empty

I'm almost sure it is not, considering it is still WIP... If you would like to use it though, i would recommend using the git repository as a crate.

@gustawdaniel Correct - I did not publish it. If you would like - I can publish it as a crate. However, use at you're own risk - this code hasn't been used in production settings.
Alternatively, use the suggestion of @maekoos .

@gustawdaniel Published in crates.io under nimrodshn-btree v0.1.0. Use with care under MIT license :)


When setting "type": "module" in package.json, cypress throws an error

Current behavior

When trying to run cypress-cucumber-preprocessor with the package.json file including "type": "module" I get an error stating that "exports is not defined in ES module scope"
Desired behavior

I should be able to run cypress with cypress-cucumber-preprocessor with the package.json file including "type": "module" without any errors
Test code to reproduce

Clone the repository: https://github.com/goldeneye5671/ccp-issue.git and run npm install
Launch cypress with npx cypress open
Cypress will then throw an error stating that "exports is not defined in ES module scope"

This is the only log I was able to get from cypress:
Your configFile is invalid: ./ccp-issue/cypress.config.ts

It threw an error when required, check the stack trace below:

ReferenceError: exports is not defined in ES module scope
    at file://ccp-issue/cypress.config.ts:14:23
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:527:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:438:15)
    at async loadFile (/Users/user1/Library/Caches/Cypress/10.8.0/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js:106:14)
    at async EventEmitter.<anonymous> (/Users/user1/Library/Caches/Cypress/10.8.0/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js:116:32)

notice that if "type": "module" is removed that cypress no longer throws the error
Versions

Cypress version: 10.8.0
Preprocessor version: 13.0.1
Node version: 18.7.0

Checklist

[x] I've read the FAQ.
[x] I've read Instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Let me know if there are any problems with replication or for more information

This isn't related to preprocessing, you have this issue whether you use cypress-cucumber-preprocessor. You need to instruct TS to output the expected type.
{
  "compilerOptions": {
    "module": "es6"
  }
}

I'm not so sure about that, as even setting that as the compilerOptions, I am still unable to get past errors either stating that it can't find the plugin:
file:///Users/user/Projects/ccp-issue/cypress.config.ts:38
import { addCucumberPreprocessorPlugin } from "@badeball/cypress-cucumber-preprocessor";
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Named export 'addCucumberPreprocessorPlugin' not found. The requested module '@badeball/cypress-cucumber-preprocessor' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@badeball/cypress-cucumber-preprocessor';
const { addCucumberPreprocessorPlugin } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:189:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:527:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:438:15)
    at async loadFile (/Users/user/Library/Caches/Cypress/10.8.0/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js:106:14)
    at async EventEmitter.<anonymous> (/Users/user/Library/Caches/Cypress/10.8.0/Cypress.app/Contents/Resources/app/packages/server/lib/plugins/child/run_require_async_child.js:116:32)

or when that change is made
the error was thrown while executing your e2e.setupNodeEvents() function:

ReferenceError: require is not defined
    at file:///Users/user/Projects/ccp-issue/cypress.config.ts:19:19
    at step (file:///Users/user/Projects/ccp-issue/cypress.config.ts:32:23)
    at Object.next (file:///Users/user/Projects/ccp-issue/cypress.config.ts:13:53)
    at fulfilled (file:///Users/user/Projects/ccp-issue/cypress.config.ts:4:58)

Again, not an issue with the preprocessor. If you want node in module-mode, then require.resolve (in your cypress.config.ts) won't be available. See eg. https://stackoverflow.com/questions/54977743/do-require-resolve-for-es-modules.


Decouple the core functionality

Decouple the core functionality from project.

Create new organization called yournewauthentication.js (make up better name)
Create repo under there called core, which will contain the core functionality.
Create repo called serverless, which will contain functionality specific to serverless utilizing the core
Create repo called express, which will contain functionality specific to express utilizing the core
Create frontend repo

New organization already created. Instead of creating many repos, I think I'm gonna go with a monorepo design.


Feat/25 create organisation endpoint

Description

Develop a backend API for creating organisations, including authentication and authorization checks, input validation, and error handling.

Related Issue (Link to Github issue)

Issue #25

Motivation and Context

Provides a backend service that allows users to create new organisations, ensuring proper authentication, authorization, and validation.
How Has This Been Tested?

Tested on Postman
Endpoint
[POST] api/v1/organisations
Payload
{
  "name": "String",
  "description": "String",
  "email": "string",
  "industry": "string",
  "type": "string",
  "country": "string",
  "address": "string",
  "state": "string",
}

Screenshots (if appropriate - Postman, etc):
Unauthenticated request

Field validation response

Success response

Test

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

This endpoint doesn't follow the recommended response. it should look like this.
{   "message": "Activity logs retrieved successfully",   "status_code": 200,   "data": [     {       "activity": "User logged in",       "timestamp": "Timestamp"     },     {       "activity": "User updated profile",       "timestamp": "Timestamp"     }   ] }

I have fixed it, please review


Support JSON Schema

Would you consider supporting JSON Schema?
I assume that the best way to do that is by converting JSON Schema into a proper data format, right?
here is an example https://raw.githubusercontent.com/fge/sample-json-schemas/master/json-home/json-home.json

@kuncevic it's already possible to load and serve JSON data, so not exactly sure what supporting JSON schema is about
https://stackblitz.com/edit/js-36myfl?file=index.html


[Fantom] Fix deployer and clean up

  203 passing (21m)
  7 pending
  36 failing

Known issues:

BOBA API Tests (4 failure tests) - need a new gas price oracle key
BOBA Fee Payment Tests (27 failure tests)
eth_sendRawTransaction{tag:rpc} should reject a transaction with a too low gas limit or too low:

invalid transaction: insufficient funds for gas * price + value\

Use secondary Fee Token (2 failure tests)
{tag:other} should delete testInt slot storage:

AssertionError: Expected "5001999701070000000000" to be equal<PHONE_NUMBER>516000000000

After updating GasPriceOracle

235 passing (28m)
  8 pending
  5 failing


HvMessage optimisations

Request to change HvMessage.c
case HV_MSG_BANG: len[i] = hv_snprintf(NULL, 0, "%s", "bang") + 1; break;
to
case HV_MSG_BANG: len[i] = 5; break;

case HV_MSG_SYMBOL: len[i] = hv_snprintf(NULL, 0, "%s", msg_getSymbol(m, i)) + 1; break
to
case HV_MSG_SYMBOL: len[i] = hv_strlen(msg_getSymbol(m, i)) + 1; break

and suitable changes to use strncpy() instead of snprintf() in
case HV_MSG_BANG: hv_snprintf(finalString+pos, len[i], "%s", "bang"); break;
and
case HV_MSG_SYMBOL: hv_snprintf(finalString+pos, len[i], "%s", msg_getSymbol(m, i)); break;

@pingdynasty does the OWL not like hv_snprintf?

printf-family are quite weighty, would be good to loose a bit of code
size and ram: about 3k.
I might substitute for my own ftoa() and itoa() functions, try it out
and send a diff.
Am I right in thinking msg_toString() is only called when the print
hook is used?
On 03/03/16 01:31, Martin Roth wrote:

@pingdynasty https://github.com/pingdynasty does the OWL not like
|hv_snprintf|?
—
Reply to this email directly or view it on GitHub
https://github.com/enzienaudio/heavy/issues/72#issuecomment-191528742.

Yes, that's correct. Heavy only calls msg_toString for the print object. The function is exposed in the top-level API as well, so potentially others are using it too. I will make your changes though, they're a good idea.

Now I think there's a typo in HvMessage.c:
./Build/Source/HvMessage.c:250:25: error: expected ')' before '!' token
   hv_assert(finalString ! = NULL);

Here's my HvMessage.c:
https://github.com/pingdynasty/OwlProgram/blob/memmathperf/HeavySource/HvMessage.c
And the ftoa/itoa implementations:
https://github.com/pingdynasty/OwlProgram/blob/memmathperf/Source/message.cpp

!! hv_assert(finalString ! = NULL); is fixed now.


Disable connection prompt message when link unfurling is disabled globally

This issue refers to https://jira.atlassian.com/browse/API-383.
As a Jira admin, I'd like to disable the connection prompt message sent by the bot to the user when the link unfurling was disabled globally.

This issue was tackled in #149 that was merged to dev. It is pending next release now.

The fix was included into Jira Server for Slack 2.0.23.


Awww it's so fuzzy!

Re: https://github.com/LeopoldArkham/Molten/pull/40#issuecomment-349012966
This PR

adds some fuzzer documentation/introduction,
runs fuzzer with debug assertions (they should really hold),
and finally adds a Travis build that does nothing but run the fuzzer for 20s. I think it's a good idea to make this fail the build: It's easy to overlook otherwise and the fuzzer seems pretty stable right now, so it'll probably find actual bugs in the future. We can pin the nightly to a specific version to make it even more stable.

Ha, and it fails. No idea what this means specifically, especially since the base64 it outputs is empty.
==7231==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1

I guess I'm gonna do that then.
Also, 20s seems to be on 152,096 runs and a coverage of 1753 code paths. With some demo files, it goes up to 2467 on my machine. I'll increase it to 60 seconds for now, and maybe we can copy the toml files already used in tests as corpus as well.

 What dark magic did you use? Does using GCC5 make the difference? Is it because I forgot to do this last week when there as a super moon?

nooooo idea

My thoughts are with you at this difficult time. Summon the eldritch powers, Pascal!


Make overriding ("inheritance") of properties more explicit

After some discussions with @stain at an IBISBA meeting:
The only explicit mentioning that I could find about overriding properties is the attribution:

In ROLite, if a file does not list a creator, and is within the Research Object's folders, it's creator can reasonably be assumed to be the creator of the containing research object. However, where appropriate, the Research Object manifest allows overriding with more precise attribution per resource.

It is unclear if this also applies to e.g. license as well or only attribution
Does an @id referring to a folder propagate the properties to all the files that are contained in the referred to folder (unless a more specific thing (sorry, lacking the word) is available)?

If we agree we allow some implicit properties like creator and license to apply to a research object folder "root folder" (?) or on a directory, then yes, we should agree:

Which provenance/license properties re-apply recursively?
Do these properties re-apply also to sub-sub-folders?
What is the order of overriding? (intuitively the most specific path should win)
What kind of tooling should do this inference to "complete" the RO?
Does such properties also apply to "external" resources outside the "root"?
Do we need to define the research object "root folder"? (e.g. fix the location of manifest.jsonld)

A bit of added complexity here. The Root RO-Crate Dataset should use creator etc as properties pointing to well described contextual entities with a URI (as per DataCrate but for indivdual files creator is going to be better expressed using provenance via CreateEvent. But yes we should make some statements about what is inherited. But I think machine reasoning about RO-Crate of un-trusted provenance would be risky.

See approach to inheritance from the Psych-DS spec

dc350427c4d612f47d41d105d283a8196705e987 expands a bit on license recursiveness, but not for folders.