Configuration in Controller

I can see a handy HttpContext attribute in the Controller class, can we have a Configuration attribute in the Controller?
I'm doing it this way now
 public class BaseController : Controller
    {
        private readonly IConfiguration _configuration;
        public BaseController(IConfiguration configuration)
        {
            _configuration = configuration;
        }

        public IConfiguration Configuration
        {
            get
            {
                return _configuration;
            }
        }
    }

No, we can't have a Configuration property on Controller. That would require MVC to couple to the configuration system. What you're doing looks fine if you want a property on the controller itself. If C# had extension properties, it would be even easier :smile:

Hi David,
Thank you for the response. It makes sense try to keep MVC less dependent on other classes. What's the best practice to retrieve Configuration inside a controller? By the way I got the following error when trying my solution.
Unable to resolve service for type 'Microsoft.Framework.Configuration.IConfiguration' while attempting to activate 'PrimeObjects.Platform.xxxxxController'.
Thanks
Gary

Also in another controller,
public IEnumerable Get(IConfiguration configuration)
{
}
I got
Cannot create an instance of an interface.
at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean noCheck, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& bNeedSecurityCheck)
at System.RuntimeType.CreateInstanceSlow(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
at System.Activator.CreateInstance(Type type, Boolean nonPublic)
at System.Activator.CreateInstance(Type type)

You need to add the IConfiguration instance to config first. The DI system doesn't know anything about a random type.

Sorry I'm new to ASP.NET, could you tell me how to do it? Or share me a link.

In startup.cs do something like this: https://github.com/aspnet/MusicStore/blob/master/src/MusicStore/Startup.cs#L109
Example:
services.AddInstance<IConfiguration>(Configuration);

I think that ought to do the trick.

Thank you Eilon, but it did not work. same error "Cannot create an instance of an interface."
public void ConfigureServices(IServiceCollection services)
{
// Add Application Insights data collection services to the services container.
services.AddApplicationInsightsTelemetry(Configuration);
services.AddMvc();
services.AddSingleton(_ => Configuration);
//Is this the right place?
services.AddInstance(Configuration);
    }

Get rid of the call to services.AddSingleton(_ => Configuration);

You bet, I thought about the same and tried, but failed
public void ConfigureServices(IServiceCollection services)
{
// Add Application Insights data collection services to the services container.
services.AddApplicationInsightsTelemetry(Configuration);
        //services.AddSingleton(_ => Configuration);
        services.AddInstance< IConfiguration >(Configuration);
        services.AddMvc();

    }

I just tried this with beta5 and it worked fine.
I added this code to Startup.cs:
        public void ConfigureServices(IServiceCollection services)
        {
            ...

            services.AddInstance<IConfiguration>(Configuration);
        }

And this is my controller:
    public class HomeController : Controller
    {
        public HomeController(IConfiguration config)
        {
            ...
        }
    }

And I got the config values in the controller just fine.

Hi Eilon, thank you so much for the help. I did the same, but no luck. I'm on beta6
My startup.cs below
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.AspNet.Builder;
using Microsoft.AspNet.Hosting;
using Microsoft.AspNet.Http;
using Microsoft.AspNet.Routing;
using Microsoft.Framework.Configuration;
using Microsoft.Framework.DependencyInjection;
using Microsoft.Framework.Runtime;
namespace PrimeObjects.Platform.Web
{
public class Startup
{
public Startup(IHostingEnvironment env, IApplicationEnvironment appEnv)
{
// Setup configuration sources.
var builder = new ConfigurationBuilder(appEnv.ApplicationBasePath)
.AddJsonFile("config.json");
        if (env.IsEnvironment("Development"))
        {
            // This will push telemetry data through Application Insights pipeline faster, allowing you to view results immediately.
            builder.AddApplicationInsightsSettings(developerMode: true);
        }

        builder.AddEnvironmentVariables();
        Configuration = builder.Build();
    }

    public IConfiguration Configuration { get; set; }

    // This method gets called by a runtime.
    // Use this method to add services to the container
    public void ConfigureServices(IServiceCollection services)
    {
        // Add Application Insights data collection services to the services container.
        services.AddApplicationInsightsTelemetry(Configuration);

        //services.AddSingleton(_ => Configuration);
        services.AddMvc();

        services.AddInstance<IConfiguration>(Configuration);

        // Uncomment the following line to add Web API services which makes it easier to port Web API 2 controllers.
        // You will also need to add the Microsoft.AspNet.Mvc.WebApiCompatShim package to the 'dependencies' section of project.json.
        // services.AddWebApiConventions();
    }

    // Configure is called after ConfigureServices is called.
    public void Configure(IApplicationBuilder app, IHostingEnvironment env)
    {

        //app.Run(async context => await context.Response.WriteAsync("ASD"));

        // Add Application Insights to the request pipeline to track HTTP request telemetry data.
        app.UseApplicationInsightsRequestTelemetry();

        // Track data about exceptions from the application. Should be configured after all error handling middleware in the request pipeline.
        app.UseApplicationInsightsExceptionTelemetry();

        // Configure the HTTP request pipeline.
        app.UseStaticFiles();

        // Add cookie-based authentication to the request pipeline
        //app.UseIdentity();

        //app.UseFacebookAuthentication();

        //app.UseGoogleAuthentication();

        //app.UseTwitterAuthentication();

        //app.UseWelcomePage();

        // Add MVC to the request pipeline.
        app.UseMvc();
        // Add the following route for porting Web API 2 controllers.
        // routes.MapWebApiRoute("DefaultApi", "api/{controller}/{id?}");
    }
}

}
My Controller cs
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.Framework.Configuration;
using Microsoft.AspNet.Mvc;
namespace PrimeObjects.Platform.Web.Controllers
{
[Route("api/value")]
public class ValuesController : Controller
{
// GET: api/values
[HttpGet]
public IEnumerable Get(IConfiguration configuration)
{
return new string[] { "value1", "value2" };
}
    // GET api/values/5
    [HttpGet("{id}")]
    public string Get(int id)
    {
        return "value";
    }

    // POST api/values
    [HttpPost]
    public void Post([FromBody]string value)
    {
    }

    // PUT api/values/5
    [HttpPut("{id}")]
    public void Put(int id, [FromBody]string value)
    {
    }

    // DELETE api/values/5
    [HttpDelete("{id}")]
    public void Delete(int id)
    {
    }
}

}

You need to take the IConfiguration in the constructor, not in a method. Method parameters do not get their values resolved form dependency injection. Please see my earlier code sample.

you are so right. It works fine now. thank you so much.
Gary


F# and Visual F# Tools announcement for June

Hey folks,
Sorry for no announcement within the actual month of June.  There still wasn't a lot of clarity on our end about what exactly would ship in each concrete asset (.NET Core, and VS tools) and when that would be.  There's more clarity now.  Here's a high level overview.
What's new?
Quite a few things are new.  The major focus area for the team has been .NET Core 2.0.
Barring any unforeseen, super critical issues, F# will ship alongside C# on .NET Core 2.0 on day 1.
.NET Core 2.0 support is quite stable now.  The bulk of the work was in moving F# in-band with the .NET Core SDK so that we would be built and tested directly alongside C# and VB.  There were also some bugs fixed, and we do daily scenario validation with each new build of the SDK and CLI.
There are still remaining issues to work out - Type Providers, Code Quotations, and FSI support - but F# support is stable enough now to the point where these can be looked and delivered after the initial release of .NET Core 2.0 and .NET Standard 2.0.
We also are completing work to enable F# to be built from source on .NET Core.  This works on macOS today, and is being validated on Linux distributions.  When you download the .NET Core sources at release, you will be able to build them all from source, including F#.
We also worked with the community in establishing a plan for publishing FSharp.Core under the FSharp.Core NuGet package.  You'll notice that the owners of this package are "F# Software Foundation, Microsoft".  This is intentional.  This is a first step in longer-tail efforts to work more with the community and make sure that both Microsoft and the FSSF/F# community are seen as delivering F#.  We still haven't figured out what all of that means.  But we want to avoid a world where Microsoft and the FSSF/F# community deliver different things to accomplish the same goals.  We think .NET Core is a good place to kick that off.
Additionally, the VS 2017 Updates milestone has moved from 5% complete to 74% complete over the course of ~4 months.  And this is with many, many more issues added to that milestone in that time frame.  These are a mix of bugs, feature requests, and augmentations.  The majority of this movement has come from the F# community.  To those who made this happen, thank you.  You're an incredible group of people.
Finally, VS 2017 Update 3 will feature the first bits enabling F# support for .NET Core in Visual Studio.  There is one remaining work item for us to do in our language service.  We are doing it this week.  Once it's finished, we believe that it will be a good experience overall.  In addition to .NET Core support, this will be using the new project system that the .NET IDE team is building.  They have built out almost all of the support, and are dedicated to continuing work there to support F#.  You can track issues related to this work here.  We're very, very excited about this work.  This project system is an upgrade to the F# project system in nearly every conceivable way: project load times, reliability, maintenance cost, and much more.  Our long-term goal is to move F# entirely over to that project system so that we can finally be rid of the current F#-only project system.  We don't have a timeline for this yet, but this is a top-of-mind item for us.
Here are some additional things accomplished:

Numerous bug fixes in the F# compiler.  Some perf improvements as well.

These will be shipped alongside VS 2017 Update 3.

Removed the generation of PCLs from our build

PCL is now deprecated technology with .NET Standard 2.0 being adopted by Xamarin
The PCL templates will still exist in VS, but under a legacy folder
We will not perform updates to PCLs anymore aside from super critical bug fixes

Made significant progress towards moving our build entirely off of project.json

Still some remaining work to do, but we're close
This work will allow us to free up resources on internal builds
We're on track to finish this soon

Fixed bugs on .NET Core
Fixed numerous internal bugs related to crashes, our build, and bugs
Established a nightly release channel for Preview builds of VS

More on this later.  There will be a blog post explaining this and how to onboard

Fixed multiple bugs related to support on the new .NET project system
Currently working with the good folks at Jet.com to pin down issues with project load performance, IntelliSense performance, and more
Added F# support for the 1.x SDK shipping inside of VS 2017
Inserted templates for .NET Core and .NET Standard projects, which will "light up" at release

What work is remaining?
As mentioned above, there is still remaining work.  Here's a re-stated overview.

F# Interactive support on .NET Core
Type Provider support for the F# compiler in the .NET SDK (even for .NET Framework programming with new project file, see this issue for a workaround)
Type Provider support for  .NET Core and .NET Standard programming (see this issue for a workaround)
Full Code Quotations on .NET Core
Remaining work to move the F# build onto the .NET Core SDK
Infrastructure work for VS tooling

If you wish to try out some preview bits and report issues, here's how:

Install VS 2017 preview with F# and .NET Core support if you're interested in using VS.
Install the CLI and SDK from this link: https://github.com/dotnet/cli/tree/release/2.0.0#installers-and-binaries

The second step is critically important, as there are numerous issues fixed in that branch.
And just to clarify, there is currently no affinity between .NET Core versions and VS Preview or VS 2017 versions.  .NET Core 2.0 is currently entirely out of band with VS 2017.  There are no current plans to make .NET Core ship in the same "package" as VS 2017.  As .NET Core 2.0 updates (and F# support for it also updates), you can simply update .NET Core on your machine.  There is no need to wait for a VS update to see bug fixes or improvements in this area.
Additionally, there are three major long-tail efforts that we've planned.  We have not "costed" this work yet, as we're head's down for .NET Core 2.0 support, but these are top-of-mind things that we want to see accomplished.

Infrastructure improvements so that we can remove our internal copy of this repository for localization purposes.  We'll move to a newer localization toolkit, and eventually build F# tooling in VS as a part of the same build that Roslyn does.  This will free up a nontrivial amount of our time.
Moving F# entirely off of its own project system and onto the .NET Project System.  Our current project system isn't very pretty.  It's a bit of a hornet's nest when you want to make any changes, with small improvements in one area causing a catastrophic failure in others.  It has performance issues that many people suffer from.  And finally, it's awfully difficult code to comprehend.  Being able to rid ourselves of this baggage to move to a modern project system owned by experts in this space is going to be a huge improvement for F#.  We're very, very motivated to make this change.
All the remaining work in this issue.  This unification of assets will allow us to make cross-platform work easier across all the different places F# exists in an editor.  It will also allow us to work on infrastructure to the point where we could consider moving the F# compiler itself out of this repository.  This is something we'd like to see happen, but there is currently no good infrastructure or testing in place to enable this in a safe way.  The linked issue tracks work needed to move F# in that direction.

So, how about some dates?
No dates at this time.  Sorry!  Things are very, very tight right now for F#, .NET, and VS 2017.  You can try out preview builds to assess F# and .NET Core today.  Much of you have been doing this already, including reporting critical bugs that have since been fixed.  Thank you for this.  As stated earlier, we do daily validation of scenarios, but the more eyes that can look at stuff, the better things will be.  Thanks!

Thanks for the clarifications @cartermp & @dsyme
I will implement the workaround to get the type provider working with the dotnet cli as an interim step to FSharp.Data being released on .netstandard2.0
Thanks for all the efforts they are very appreciated this is an exciting time for the platform and I cannot wait for F# to be a full blown first class citizen across the board.

@buvinghausen  Thanks for the encouragement Brian!
There will be a lot of work to do to keep improving out .NET Core/.NET SDK  story and help bring community projects across to the SDK, the community would love to have your help where you can :)

Closing old discussion.


draft: Introduce protobuf schemas for seqvar query output

Still very WIP, just opening to track progress ^^
Will solve #375

I think this is ready for a final round of review then I can properly split this up into the two PRs (if that's still wanted).


Bitrix Website scenario

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru
Description of changes:
Added New Bitrix Website instruction to docs/ru/solutions/web/bitrix-website.md
Also i have added Bitrix website installation process printscreens to the following directory: docs/ru/solutions/_assets/bitrix-website/

I have made changes at  Bitrix Website scenario according to change requests at pull request #34 (yandex-cloud/docs).  I am requesting you a merge.

I have made changes at Bitrix Website scenario according to change requests at pull request #34 (yandex-cloud/docs). I am requesting you a merge.

Good evening!
I have finished changes at scenarion according to pull request #34 requested changes. Now the document is completed. Please see the changes and if everything is good, please merge the document to yandex-cloud/docs/ru/solutions/web/ directory.
Thank you in advance.

Good evening.
I have updated scenario according to requested updates on 09.09.2019. Scenario has been completed. Please proceed with corrections if needed or please merge the instruction to web directory at yandex-cloud/docs.
Thank you in advance.


[Issue]: erorr using nvm install panic: runtime error

What happened?
Using version 1.1.11, I ran the command nvm install latest and nvm install 21.1.0 then received an error
What did you expect to happen?
it installs the latest version of node!
Version
1.1.11 or newer (Default)
Which version of Windows?
Windows 11+
Which locale?
English (Default)
Which shell are you running NVM4W in?
Command Prompt, PowerShell
User Permissions?
Administrative Privileges, Elevated
Is Developer Mode enabled?
No (Default)
Relevant log/console output
panic: runtime error: index out of range [3] with length 3

goroutine 1 [running]:
main.checkVersionExceedsLatest({0x120100b0, 0x6})
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:790 +0x220
main.install({0x120100b0, 0x6}, {0xf9fa65, 0x2})
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:302 +0x35d
main.main()
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:87 +0xa69

Debug Output
panic: runtime error: index out of range [3] with length 3

goroutine 1 [running]:
main.checkVersionExceedsLatest({0x120100b0, 0x6})
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:790 +0x220
main.install({0x120100b0, 0x6}, {0xf9fa65, 0x2})
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:302 +0x35d
main.main()
        C:/Users/corey/OneDrive/Documents/workspace/oss/coreybutler/nvm-windows/src/nvm.go:87 +0xa69

Anything else?
No response

See https://github.com/coreybutler/nvm-windows/wiki/Common-Issues#cant-download-latest-nodejs. This is the "eager beaver" problem.
While it would be nice to add a better error message for this edge case, I just don't have time right now. Runtime takes priority over all non-blocking issues.


chore: remove unnecessary insert-css

🤔 This is a ...

[ ] New feature
[ ] Bug fix
[ ] Component style update
[ ] TypeScript definition update
[ ] Refactoring
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[x] Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language
Changelog

🇺🇸 English

🇨🇳 Chinese

☑ Self Check before Merge

[x] All tests pass and/or benchmarks are included
[x] Commit message follows commit guidelines
[x] TypeScript definition is updated/provided or not needed
[x] Changelog is provided or not needed

不需要加依赖，主题内建。


Unhelpful error messsage

https://github.com/KhronosGroup/SPIRV-Tools/blob/d0a827a9f313954935e3d9edb0c063fd8625a22e/source/opt/instrument_pass.cpp#L977-L990
This message complains that Mixed stage shader module not supported, it could be nice to report what the shader stages actually are.
Edit: Additionally we're hitting this message in a raytracing PSO, so it may be time to address the todo?

Are you calling this pass directly or indirectly through GPU-assisted validation? In general, this pass is not intended to be used by the general public.

Nonetheless, such a shader module is possible and should be supported.
I do believe that neither of the high-level shader compilers (glslangValidator or dxc) will create such a module, which is why I did not support it initially.
So I would be curious how this shader module was created.

This functionality has been moved in to Vulkan-ValidationLayers so I've opened an issue over there: https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/8716


unsolicited console interaction "Install mc auto-completion in your shell ?"

(I rename execitable to mcli because I use midnight-commander a lot)
REPRO:
curl -sSLo /usr/bin/mcli https://dl.minio.io/client/mc/release/linux-amd64/mc
chmod +x /usr/bin/mcli
EXPECTED:
can use mcli in scripts
CURENT:
it always goes interactive
mcli
mcli: Configuration written to /root/.mc/config.json. Please update your access credentials.
mcli: Successfully created /root/.mc/share.
mcli: Initialized share uploads /root/.mc/share/uploads.json file.
mcli: Initialized share downloads /root/.mc/share/downloads.json file.
Install mc auto-completion in your shell ? (y/n):

workaround - do "echo y | mcli" once


Add Copying Existing Flux Solutions to Cross Section Control Settings

Description

Adding new functionality to the cross-section settings to copy existing flux solution files during lattice physics evaluations. This is useful for lattice physics codes that allow for an external solver solution file to be provided.
This breaks the crossSectionControl setting for existing applications with the use of pregenerated cross sections. The name of the setting was changed from fileLocation to xsFileLocation to allow the addition of the fluxFileLocation setting.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in setup.py.

I think I like this solution a little more than mine, but we should definitely add a migration from fileLocation to xsFileLocation, otherwise this is going to be a pesky break in backwards compatibility.

Opened up https://github.com/terrapower/armi/issues/1166 for further clean-ups. Note to @mgjarrett - The crossSectionControl setting with fluxFileLocation option implies that the name of the file should contain rzmflx (case sensitive) for this to function on our end. The issue that is linked will aim to clean up these assumptions more broadly in the code base. There is no settings validation for rzmflx in the name because I don't think it's appropriate to mandate this since it should in the future be more general.

The unit tests worked when I ran locally, but they don't seem to be working here. I think the necessary ISOXA and rzmflxYA files are not being copied into the test directory.

The unit tests worked when I ran locally, but they don't seem to be working here. I think the necessary ISOXA and rzmflxYA files are not being copied into the test directory.

That reminds me -- they also need to be added to the MANIFEST.in file. I don't think that solves this problem, though

Looks great, nice job!


How to use @RequestModule()

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
I was trying to import a service into the command by Dependency Injection but it gives me an error:
ERROR [CommandRunnerService] A service tried to call a property of "undefined" in the EcomProductEventsHandleCommand class. Did you use a request scoped provider without the @RequestModule() decorator?
I have checked your documentation and there is no example of it!
Minimum reproduction code
import { Command, CommandRunner, Option } from 'nest-commander';
import { Logger } from '../../common/logger/logger.service';
import { PriceRmqHandlerService } from '../services/price-rmq-handler.service';

interface CommandOptions {
  data?: string;
}

@Command({ name: 'ecom-product-events-handle', description: 'Handling product events from shared bus' })
export class EcomProductEventsHandleCommand extends CommandRunner {
  constructor(
    private readonly logger: Logger,
    private readonly priceRmqHandlerService: PriceRmqHandlerService,
    ) {
    super()
  }

  async run(inputs: string[], options?: CommandOptions): Promise<void> {
    if (options?.data) {
      this.runWithData(inputs, options?.data);
    } else {
      this.runWithNone()
    }
    process.exit(0)
  }

  @Option({
    flags: '-d, --data json',
    description: 'A JSON data parser',
  })
  parseJson(val: string): Object {
    return JSON.parse(val);
  }

  runWithData(param: string[], option: string): void {
    console.log("Running with data...")
  }

  runWithNone(): void {
    console.log("Running...")
  }
}

Expected behavior
because we are importing the main module in main-cli.ts like this:
async function bootstrap() {
  await CommandFactory.run(AppModule, ['warn', 'error']);
}

should do the DI automatically!
Package

[X] nest-commander
[ ] nest-commander-schematics
[ ] nest-commander-testing

Package version
3.11.0
Node.js version
16
In which operating systems have you tested?

[X] macOS
[ ] Windows
[ ] Linux

Other
No response

There is documentation for it. And here's the API docs too

There is documentation for it. And here's the API docs too

I created the module for the command and imported it into the AppModule like this:
@RequestModule({
    imports: [SharedModule],
    providers: [
        EcomProductEventsHandleCommand,
        Logger,
        PriceRmqHandlerService
    ],
    requestObject: { headers: { Authorization: 'Bearer token' } }
  })
  export class PriceCommandModule {}

but still, it's giving me the same error


add deberta v1 support

Add DeBERTa V1 support since https://github.com/huggingface/transformers/pull/17617 now supports both

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @sam-h-bean, the PRs only concern the ORTOptimizer but not the ORTQuantizer, so if you want to try quantization features, pls go ahead.
We are working on a bug in the doc builder, unfortunately, we can't merge this before all checks pass.

Closed this as the supports for DeBERTa and DeBERTa-V2 have been added and merged in my previous PR. Thanks again for the contribution, excellent job!


MoveToRegion not working

Hi,
I use the last version of XFGMaps.
I try to initialize the position with:
 MapValue.MoveToRegion(               MapSpan.FromCenterAndRadius(                   new Position(SearchInputValue.Place.Latitude, SearchInputValue.Place.Longitude)                   , new Distance(SearchInputValue.Radius)));
But nothing happens.
Then I tried to use as explained in other issues
MapValue.InitialCameraUpdate = CameraUpdateFactory.NewPosition(new Position(SearchInputValue.Place.Latitude, SearchInputValue.Place.Longitude));
It works well but I would like to show the map with a radius.
How can I set the zoom ratio to radius or to make working MoveToRegion?
Thanks,

OK I finally found the solution by checking the source code
`var ms = MapSpan.FromCenterAndRadius(
new Position(SearchInputValue.Place.Latitude, SearchInputValue.Place.Longitude)
, new Distance(SearchInputValue.Radius));
        MapValue.InitialCameraUpdate = CameraUpdateFactory.NewBounds(ms.ToBounds(), 0);`

What does the padding mean for the method CameraUpdateFactory.NewBounds?

hi dalton5, im stuck at the same point. I have to zoom in deeper automatically, when a marker is selected.
could u find any solution?


UnicodeDecodeError: 'ascii' codec can't decode byte 0xd0

Using latest version.
Trying this url to create new spider - http://www.foxtrot.com.ua/
2016-06-03 20:16:44.367206 [-] "<IP_ADDRESS>" - - [03/Jun/2016:20:16:42 +0000] "GET /proxy?url=http%3A%2F%2Fwww.foxtrot.com.ua%2FContent%2Fthemes%2Fbase%2Fall.css&tabid=140482191453968&referer=www.foxtrot.com.ua HTTP/1.1" 200 515 "http://<IP_ADDRESS>:9001/static/empty-frame.html?30e6-2486-dc14" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.79 Safari/537.36"
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/splash/browser_tab.py", line 977, in _on_request_finished
    callback()
  File "/usr/local/lib/python2.7/dist-packages/splash/browser_tab.py", line 1000, in _return_reply
    callback(reply)
  File "/home/portia/slyd/slyd/splash/proxy.py", line 107, in end_response
    content = process_css(content, tabid, original_url)
  File "/home/portia/slyd/slyd/splash/css_utils.py", line 53, in process_css
    css_source = CSS_URL.sub(_absolutize_css_url, css_source)
  File "/home/portia/slyd/slyd/splash/css_utils.py", line 50, in _absolutize_css_url
    base_uri).replace('"', '%22'))
  File "/home/portia/slyd/slyd/splash/css_utils.py", line 17, in wrap_url
    url = urljoin(base, url).encode('utf-8')
UnicodeDecodeError: 'ascii' codec can't decode byte 0xd0 in position 41: ordinal not in range(128)
Aborted

Anyone ?

There is a case: http://www.centarahotelsresorts.com/exclusive-room-offers/

Got the same issue with current latest version.

Same issue latest version using docker


Video not playing in RTSP Players

Hello.
When I start the stream in VLC player I only see the first video frame.
Also i tried start stream in iOS RTSP Player. The same result.
Can you check? i can donate 100$ for fix :)
P.S: when i run rtsp stream from this project  video works fine in VLC and iOS RTSP Player.

Hello,
I need more info.

P.S: when i run rtsp stream from this project  video works fine in VLC and iOS RTSP Player.

I can assume that using this library to push and that server is working fine. Can you tell me the other server that fail?
Can you see in the logs that you are sending packets? Is audio working fine?


branch 1.3.0 The front-end log output is inconsistent

My task ran for about 20 minutes. In the last time, I found that the front-end log was inconsistent with the output log of the background log. May I ask what happened?No error was reported in the log. Finally, the program was executed successfully, but the foreground still shows that it is running.

我的任务跑了大概20分钟，在最后的时间里发现，前端日志，和后台日志输出的日志不一致， 请问这是怎么回事。日志也没报错，最后程序执行成功了，但是前台，还是显示正在运行中。

肯定是执行刷新了， 发现跟后端日志差的太多了。

I see that there are hadoop errors reported on the front end

但是我发现这个前端日志打印最多1000行日志， 我有的日志比这个多 打印不出来， 在哪设置这个1000行限制参数

You can click to download the log

我后来是找到可以下载来看， 但是能在线看不是更好么。 就是显示的不全

He displays a maximum of 1,000 entries per page. There is a split-screen loading. You can see the next 1,000 entries by pulling down

在哪点选的，我怎么没找到呢

Does your mouse have a scroll wheel? Place the cursor in the middle of the screen, and then scroll the mouse wheel down

怎么往下分页 日志  这块  有人回复我 用鼠标滚轮滑动，我都滑动到最底下了。 怎么分页 这个最多展示1000行


SOLVED: Screensaver not working

Hi there,
I am running the latest beta. Unfortunately I haven't been able to get the screensaver to work. Everything is fine in settings. I turned on the screensaver and cached all the videos. But if I leave my computer without touching it, the screen just goes black and no screensaver comes up. I am not running Little Snitch (your recommendation from the troubleshooting section). Any thoughts?
Thanks

What was the solution?


Two state-node-network_name directories after building node

After building node with:
nix-build -A scripts.mainnet.node -o mainnet-node-local
I have state-node-mainnet inside another state-node-mainnet  directory:

state-node-mainnet$ ll
total 12
drwxr-xr-x  3 artur artur 4096 Mar  3 16:38 ./
drwxr-xr-x 14 artur artur 4096 Mar  3 16:38 ../
drwxr-xr-x  3 artur artur 4096 Mar  3 16:38 state-node-mainnet/

This was fixed by: https://github.com/input-output-hk/cardano-node/pull/626
Closing issue.


Remove jhub-client from pyproject.toml

Reference Issues or PRs
The Jupyter Notebook tests that used jhub-client were removed in #2232. This PR removes jhub-client as a dependency, because I don't believe we use it anywhere else.

What does this implement/fix?
Put a x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

Good catch @pavithraes! I did not even think about this when removing the Notebook tests. Thank you!


Fine grain MaxAge

Is it possible to have a MaxAge smaller than 1 day? Say 1 hour or even in minutes?

This is a time.Duration in v3 (which is not tagged yet but exists as a branch)


Document: Explicitly check that pageContents is not an empty string.

This PR is related to a discussion in #2807, the current behaviour of the Document constructor field.pageContent when given an empty string is to set pageContent as undefined. This broke HNSWLib.addDocuments() as it attempts a string.replace() on undefined rather than the typed string expected.
I suggested allowing pageContents to be set as an empty string. @jacoblee93 suggested that a Document with an empty pageContents should not be created at all.
This implementation requires pageContents to be a string that is not empty, otherwise it throws Error("pageContents must not be empty").

Ideally we shouldn't need this with TypeScript, but an empty string should be permissible, just not a hard undefined, null, or non-string value - can you change?

Sure I change this out to allow it to record an empty string.
On Mon, 9 Oct 2023, 18:41 Jacob Lee, @.***> wrote:

Ideally we shouldn't need this with TypeScript, but an empty string should
be permissible, just not a hard undefined, null, or non-string value -
can you change?
—
Reply to this email directly, view it on GitHub
https://github.com/langchain-ai/langchainjs/pull/2836#issuecomment-1753408078,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAES7ZPDXG2I724JOIJLWXLX6QZMBAVCNFSM6AAAAAA5XQOJCWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJTGQYDQMBXHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@jacoblee93 I added the proposed change and threw in a couple of unit tests for good measure.


With product location disabled nothing works

You can disable location tracking in the grocy addon with stock_location_tracking: false.
If that is configured I noticed pygrocy will not be able to handle a shopping list call with items without location in it. Location should perhaps be optional.

I'm having the same problem!

Do you have any stacktrace or error log available, maybe by increasing the log level?

I get this: Update of Shopping_list failed with 'NoneType' object has no attribute 'get'


int64 value() turns to Scientific notation

func TestValue(t *testing.T) {
	val := `{"owner_id":106134567558}`
	fmt.Println(gjson.Get(val, "owner_id").Value())
}

go test -timeout 30s -run ^(TestValue)$ -v
=== RUN   TestValue
1.06134567558e+11
--- PASS: TestValue (0.00s)
PASS
ok      git.byted.org/aweme/gopkg/ajson/test    0.006s

is there a gjson.UseNumber option to resolve this problem?

I recommend the Int() function, which returns the number as an int64.
fmt.Println(gjson.Get(val, "owner_id").Int())

I recommend the Int() function, which returns the number as an int64.
fmt.Println(gjson.Get(val, "owner_id").Int())

The case we meet is special. We don't know the actual type(string or int) sometimes so we want to get the Value() and then switch type of it.

Perhaps I misunderstood. It appears in your case that the Value() function is converting the gjson result into a proper Go type and storing that result in an interface{}.
Here's the conversion table from JSON >> Go:
boolean >> bool
number  >> float64
string  >> string
null    >> nil
array   >> []interface{}
object  >> map[string]interface{}

The Value() call will never return a Go type other than the one above. In your case the number from {"owner_id":106134567558} is being converted into a float64. Most JSON parsers, including standard Go and Javascript, default json numbers to floating points.

The output using the fmt.Println(gjson.Get(val, "owner_id").Value()) is printing a string representation of the floating point. The way Go handles floating point strings by default is a little weird, but unless you provide a string format using fmt.Printf, Go will default the output format to %v, which for a floating point is equivalent to %g. The %g will output as %e (scientific notation) for large exponents, otherwise %f.
Thus the number 1.06134567558e+11 is equivalent to<PHONE_NUMBER>58.
The Go documentation dives into this in depth on their fmt page.

Yeah I understand why I see the unexpected results because of the %v. Thanks so much.
On this test It print the value that I want
func TestValue(t *testing.T) {
	val := `{"owner_id":106134567558}`
	fmt.Printf("%f\n", gjson.Get(val, "owner_id").Value())
}
go test -timeout 30s git.byted.org/aweme/gopkg/ajson/test -run ^(TestValue)$ -v
=== RUN   TestValue
106134567558.000000
--- PASS: TestValue (0.00s)

By the way, Golang Encoding json provide us the Usenumber option, witch seems more convinent in my case
func TestValueForEncodingJson(t *testing.T) {
	val := []byte(`{"owner_id":106134567558}`)
	var target map[string]interface{}
	d := json.NewDecoder(bytes.NewReader(val))
	d.UseNumber()
	d.Decode(&target)
	fmt.Println(target)
}

go test -timeout 30s git.byted.org/aweme/gopkg/ajson/test -run ^(TestValueForEncodingJson)$ -v
=== RUN   TestValueForEncodingJson
map[owner_id:106134567558]
--- PASS: TestValueForEncodingJson (0.00s)
PASS
ok      git.byted.org/aweme/gopkg/ajson/test    0.006s

then I can just convert the string to Int64

I'm glad to hear that you found a solution.
Regarding the UseNumber() option (and the resulting Number) type, this seems really useful for the stdlib json package. Though I'm not sure if it would be a good solution for gjson. Mainly because a gjson.Result already has all of the conversation function included, such as Int(), Uint(), Bool(), Float(), etc.
I'm closing this issue, but please feel free to reopen if needed.


File:"sketchup-stl.rb" refers to MIT License in file header

ALL Ruby files refer to MIT License in file header.
Extension copyright attribute ("sketchup-stl.rb") states plugin is released under Apache 2.0 License.
HTML, CSS, and Js files have no copyright or license information.

Good catch. I thought all of this has been replaced already. Doing a full search now.


fix pvc reclaimspace & keyrotation annotation filter

This commit adds a check in StorageClass event handler to filter out PVC not needed for Reconcile.

If the StorageClass has only ReclaimSpace annotation,
PVCs without ReclaimSpace annotations will be enqueued.
If the StorageClass has only KeyRotation annotation,
PVCs without the KeyRotation annotation will be enqueued.
If the StorageClass has both the ReclaimSpace and KeyRotation annotations,
PVCs without either annotation or having any one of them will be enqueued.

@iPraveenParihar please add more details why not to requeue in this specific case which we are addressing here?

@iPraveenParihar please add more details why not to requeue in this specific case which we are addressing here?

Updated the PR description.

This commit adds a check in StorageClass event handler to filter out PVC not needed for Reconcile.

If the StorageClass has ReclaimSpace annotation,
PVCs without ReclaimSpace annotations will be enqueued.
If the StorageClass has KeyRotation annotation,
PVCs without the KeyRotation annotation will be enqueued.

what will happen if i remove the annotation on the storageclass? i would expect that the annotations on the PVC's will be removed and reclaimspace/keyRotation CR's will get deleted, isn't that the case? have we documented what manual step user need to do in that case?

This commit adds a check in StorageClass event handler to filter out PVC not needed for Reconcile.

If the StorageClass has ReclaimSpace annotation,
PVCs without ReclaimSpace annotations will be enqueued.
If the StorageClass has KeyRotation annotation,
PVCs without the KeyRotation annotation will be enqueued.

what will happen if i remove the annotation on the storageclass? i would expect that the annotations on the PVC's will be removed and reclaimspace/keyRotation CR's will get deleted, isn't that the case? have we documented what manual step user need to do in that case?

i see that its documented for now.


Does Is readonly still have a meaning in Fractal 2.0?

@tcompa Is the Is readonly still used for anything? It still shows up e.g. in dataset edit (=> likely in the db entry of the V2 datasets?), but I'm not sure what it would do.

Thanks for the explanation @tcompa ! I could see the potential relevance when we had input & output datasets and wants to avoid that the input dataset is used to write data to. But with the new single dataset logic, that doesn't apply anymore.
Using it to "lock" datasets is not a feature we've advertised and given that Fractal server doesn't manage the underlying image data, I'm not sure if it's a particularly useful feature.
Conclusion: Let's remove the read_only properties from datasets & projects.


something wrong？

I run the code from github
python model_server.py ../data/models/urls/
and get:
Traceback (most recent call last):
File "model_server.py", line 28, in
'model_weights.h5',
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 2538, in load_weights
load_weights_from_hdf5_group(f, self.layers)
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 2970, in load_weights_from_hdf5_group
K.batch_set_value(weight_value_tuples)
File "/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py", line 2148, in batch_set_value
assign_op = x.assign(assign_placeholder)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 551, in assign
return state_ops.assign(self._variable, value, use_locking=use_locking)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 47, in assign
use_locking=use_locking, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2397, in create_op
set_shapes_for_outputs(ret)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1757, in set_shapes_for_outputs
shapes = shape_func(op)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1707, in call_with_requiring
return call_cpp_shape_fn(op, require_shape_fn=True)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 610, in call_cpp_shape_fn
debug_python_shape_fn, require_shape_fn)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 675, in _call_cpp_shape_fn_impl
raise ValueError(err.message)
ValueError: Shapes must be equal rank, but are 3 and 4 for 'Assign_1' (op: 'Assign') with input shapes: [2,32,256], [256,32,2,1].
anybody known what happend?

tha whole error is
root@ubuntu:~/eXposeDeepNeuralNetwork/src# python model_server.py ../data/models/urls/
Using TensorFlow backend.
Loading models
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your InputLayer call to the Keras 2 API: InputLayer(dtype="int32", batch_input_shape=[None, 100..., name="main_input")
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: The dropout argument is no longer support in Embedding.  Use instead layers from keras.layers.merge, e.g. add, concatenate, etc.
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your Dense call to the Keras 2 API: Dense(name="dense_1", bias_regularizer=None, bias_constraint=None, activity_regularizer=None, trainable=True, kernel_constraint=None, kernel_regularizer=None, input_dim=None, units=1024, kernel_initializer="glorot_uniform", use_bias=True, activation="relu")
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your Dropout call to the Keras 2 API: Dropout(rate=0.5, trainable=True, name="dropout_5")
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your Dense call to the Keras 2 API: Dense(name="dense_2", bias_regularizer=None, bias_constraint=None, activity_regularizer=None, trainable=True, kernel_constraint=None, kernel_regularizer=None, input_dim=None, units=1024, kernel_initializer="glorot_uniform", use_bias=True, activation="relu")
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your Dropout call to the Keras 2 API: Dropout(rate=0.5, trainable=True, name="dropout_6")
return cls(**config)
/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py:1231: UserWarning: Update your Dense call to the Keras 2 API: Dense(name="dense_3", bias_regularizer=None, bias_constraint=None, activity_regularizer=None, trainable=True, kernel_constraint=None, kernel_regularizer=None, input_dim=None, units=1, kernel_initializer="glorot_uniform", use_bias=True, activation="sigmoid")
return cls(**config)
Traceback (most recent call last):
File "model_server.py", line 28, in
'model_weights.h5',
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 2538, in load_weights
load_weights_from_hdf5_group(f, self.layers)
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 2970, in load_weights_from_hdf5_group
K.batch_set_value(weight_value_tuples)
File "/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py", line 2148, in batch_set_value
assign_op = x.assign(assign_placeholder)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 551, in assign
return state_ops.assign(self._variable, value, use_locking=use_locking)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 47, in assign
use_locking=use_locking, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2397, in create_op
set_shapes_for_outputs(ret)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1757, in set_shapes_for_outputs
shapes = shape_func(op)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1707, in call_with_requiring
return call_cpp_shape_fn(op, require_shape_fn=True)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 610, in call_cpp_shape_fn
debug_python_shape_fn, require_shape_fn)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 675, in _call_cpp_shape_fn_impl
raise ValueError(err.message)
ValueError: Shapes must be equal rank, but are 3 and 4 for 'Assign_1' (op: 'Assign') with input shapes: [2,32,256], [256,32,2,1].

Same problem here.
Can you let us know the exact versions of keras and tensorflow that work with this project?


Walker service should walk schemas recursively

I may misunderstand what the walker is going to be used for or how it works, but it seems to me that currently the nested contents of a schema node is not visited (e.g. if it is an object (Properties field) or array (Items field)). I would like to implement some logic that would need to visit all nested schema nodes. Do you agree that this is desirable? If so I could submit a pull request to add this functionality.

Closing. Please feel free to send a follow-up PR or re-open if you still see any issues with the current Walker.


Enable access to existing captions and transcript for audiovisual content

A placeholder issue for this requirement.

Moved to dimes repo as issue #305. Closing it here.

Moved to dimes repo as issue #305. Closing it here.


Feedback wanted - [Mouse Jump] - Customisable appearance - borders, margins, colours, etc

Summary of the Pull Request
This is a follow-up to #32838 and #27511 to introduce some style settings to control the visual appearance of the Mouse Jump preview popup - previous work in #32838 added the style rendering code but didn't surface it in the Settings UI project and left the visual appearance as it was previously, but generated using some new rendering code.
This PR adds Settings UI options - it shows a live preview in the "Appearance and behaviour" section of the Mouse Jump settings which updates dynamically as colour and layout options are edited.
It's a Work In Progress, but I'd appreciate a review of the functionality in this PR so far, primarily in terms of whether the overall approach fits in with general PowerToys design guidelines, or whether it needs a major rethink before I spend a lot of time refining it. I'm not so worried about the code right now - mostly just the visual appearance and functionality of the new Mouse Jump Settings UI options.
Here's a (slightly rubbish) capture of it in action for reference:
https://github.com/microsoft/PowerToys/assets/1193763/7fd0885e-71d9-46b1-a6ee-0b9e8c4086ed

@mikeclayton
Regarding the live preview panel: I don't think that we need a live preview of the screen. A fake image should be enough. And I think we should only show one desktop in the preview. Reasons for my thoughts are performance and visibility.

@htcfreek - I can change the preview to use a static image as a fake desktop - the most recent unit tests do this anyway, so I can just re-use that. It might be worth leaving two screens though - the setting currently labelled "Screen Margin" defines the spacing between the individual screens - if there was just one screen the effect of this setting wouldn't be visible in the preview.
(I think the setting names and descriptions probably all need some thought as well, but I'm keen to get the layout and functionality right first...)

@htcfreek - I can change the preview to use a static image as a fake desktop - the most recent unit tests do this anyway, so I can just re-use that. It might be worth leaving two screens though - the setting currently labelled "Screen Margin" defines the spacing between the individual screens - if there was just one screen the effect of this setting wouldn't be visible in the preview.

Sounds both good to me. Saw that point with the two screens late.

(I think the setting names and descriptions probably all need some thought as well, but I'm keen to get the layout and functionality right first...)

Sounds good.

@htcfreek - I've replaced the live preview with a static image: - see below.

The static "fake desktop" image it uses is is here: https://github.com/microsoft/PowerToys/pull/33486/files#diff-7fa63c54234718f0f78bb369b5da042242265e8487e4fa2e8cdca1107c57b5ac and it's based on my original animation at https://github.com/mikeclayton/FancyMouse. If it's not really suitable (e.g. doesn't meet UI guidelines / colours / logos / similarity to real products / etc ) would you be able to ask someone in the core team to create one we can use as that might be quicker than me trying to meet the design criteria.
I'm also having some trouble with the CI build - it's failing due to assembly versions - e.g. https://dev.azure.com/ms/PowerToys/_build/results?buildId=589043&view=logs&j=b6b44f60-477e-5b36-eeeb-589df0c177de&t=f5043f15-036d-5d3b-8695-92dff4115778
It would be good to hear if there's any other UI feedback as well...

@htcfreek - I'm also having a bit of trouble with the Azure DevOps pipeline. I've added a new project - MouseJump.Common - but I'm getting an error about the target framework version - I've had a dig, but I  can't see what's different compared to other projects that are building successfully before it (e.g. ImageResizer).
Are you able to take a look and see if you can see what the issue is?
For reference, the pipeline error is:

##[error]C:\hostedtoolcache\windows\dotnet\sdk\8.0.302\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(266,5): Error NETSDK1047: Assets file 'D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\obj\project.assets.json' doesn't have a target for 'net8.0-windows10.0.20348.0/win-x64'. Ensure that restore has run and that you have included 'net8.0-windows10.0.20348.0' in the TargetFrameworks for your project. You may also need to include 'win-x64' in your project's RuntimeIdentifiers. [D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\MouseJump.Common.csproj]

@htcfreek - I'm also having a bit of trouble with the Azure DevOps pipeline. I've added a new project - MouseJump.Common - but I'm getting an error about the target framework version - I've had a dig, but I  can't see what's different compared to other projects that are building successfully before it (e.g. ImageResizer).
Are you able to take a look and see if you can see what the issue is?
For reference, the pipeline error is:
https://dev.azure.com/ms/PowerToys/_build/results?buildId=589043&view=logs&j=b6b44f60-477e-5b36-eeeb-589df0c177de&t=f5043f15-036d-5d3b-8695-92dff4115778

##[error]C:\hostedtoolcache\windows\dotnet\sdk\8.0.302\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(266,5): Error NETSDK1047: Assets file 'D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\obj\project.assets.json' doesn't have a target for 'net8.0-windows10.0.20348.0/win-x64'. Ensure that restore has run and that you have included 'net8.0-windows10.0.20348.0' in the TargetFrameworks for your project. You may also need to include 'win-x64' in your project's RuntimeIdentifiers. [D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\MouseJump.Common.csproj]

The order/structure of the two "common" .csproject files is different. May that's the reason. 🤔🤷🏻♂ But for such problems I the wrong person to ask. 😅

@htcfreek - no worries.
I'm at a bit of a loss as to what else to try - is there someone on the core team that I could @ mention who might be able to help?

@htcfreek - no worries.
I'm at a bit of a loss as to what else to try though - is there someone on the core team that I could @ mention who might be able to help?

Jaime (jaimecbernardo) I think.

@jaimecbernardo - I'm having some problems with the Azure DevOps Pipeline that I've not been able to resolve - is this something you're able to take a look at?
In this PR I've moved some of the MouseJumpUI code into a new project - MouseJump.Common - which lets me re-use some of the code in the SettingsUI project for configuring options.
However, when the build runs I get this error:
https://dev.azure.com/ms/PowerToys/_build/results?buildId=589043&view=logs&j=b6b44f60-477e-5b36-eeeb-589df0c177de&t=f5043f15-036d-5d3b-8695-92dff4115778
##[error]C:\hostedtoolcache\windows\dotnet\sdk\8.0.302\Sdks\Microsoft.NET.Sdk\targets\Microsoft.PackageDependencyResolution.targets(266,5): Error NETSDK1047: Assets file 'D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\obj\project.assets.json' doesn't have a target for 'net8.0-windows10.0.20348.0/win-x64'. Ensure that restore has run and that you have included 'net8.0-windows10.0.20348.0' in the TargetFrameworks for your project. You may also need to include 'win-x64' in your project's RuntimeIdentifiers. [D:\a_work\1\s\src\modules\MouseUtils\MouseJump.Common\MouseJump.Common.csproj]

I've tried comparing the MouseJump.Common to other projects and I can't see any differences that might cause the problem, so I'm a bit stuck with what else to try. If you're able to shed any light it would be massively appreciated...

@htcfreek @jaimecbernardo  @Herry0092 - apologies for @'ing everyone.
I'm at a bit of a dead-end now with the verifyDepsJsonLibraryVersions.ps1 ci pipeline issue above. I've tried a few different things which don't seem to have worked, and I'm at the point where I'm just randomly changing things to see if they make any difference.
I'm not really sure what to do to unblock this - could anyone suggest where to go with it?

@mikeclayton I'll try to take a look. I'll be mostly looking at deps.json files to double check where the difference might come from. Can you please revert the changes you've made to try to fix that build so I can start from before any tries to fix it? Thank you!

Thanks @jaimecbernardo - I'll push an update later today and let you know when it's ready.

@jaimecbernardo - I've reverted the change: I made to add this to the *.csproj:
<RuntimeIdentifiers>win-x64;win-arm64</RuntimeIdentifiers>

which seemed to fix the NETSDK1047 error (Assets file 'D:\a\_work\1\s\src\modules\MouseUtils\MouseJump.Common\obj\project.assets.json' doesn't have a target for 'net8.0-windows10.0.20348.0/win-x64'), in the new MouseJumpUI.COmmon project, so that error is currently back in the build.
MouseJump.Common.csproj is basically a copy of MouseJumpUI.csproj just as a Library rather than a WinExe, and doesn't have <UseWindowsFOrms>true</UseWindowsForms>, so I'm not sure why MouseJumpUI.csproj builds but MouseJump.Common.csproj doesn't.
Once the above error is fixed, the build then fails on the verifyDepsJsonLibraryVersions.ps1 step with a mismatch between assembly version references in the various *.deps.json files:
System.Drawing.dll
	 8.0.624.26909
		 PowerToys.ImageResizer.deps.json
                 ...
		 PowerToys.MouseJumpUI.deps.json
                 ...
	 8.0.624.26715
		 PowerToys.MouseJump.Common.deps.json
Detected  4  libraries that are mentioned with different version across the dependencies.

Again, I haven't done anything special to MouseJumpUI so I'm not sure why that is different to the new MouseJump.Common project. I had some help originally with adding MouseJump to the installer, so there might be some settings outside the project files that I've never seen affect how they build...
One thing I haven't reverted is correcting the project guids in PowerToys.sln - I removed and re-added the MouseJumpUI and MouseJump.Common projects which corrected the guids in the PowerToys.sln file:
-Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MouseJump.Common", "src\modules\MouseUtils\MouseJump.Common\MouseJump.Common.csproj", "{F35E1EF3-C2AE-4407-8029-92B8F2B4A382}"
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MouseJump.Common", "src\modules\MouseUtils\MouseJump.Common\MouseJump.Common.csproj", "{923DF87C-CA99-4D1C-B1D2-959174E95BFA}"

-Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MouseJump.Common.UnitTests", "src\modules\MouseUtils\MouseJump.Common.UnitTests\MouseJump.Common.UnitTests.csproj", "{6A10C2B0-92AE-4928-996B-FCF55D2E3F39}"
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MouseJump.Common.UnitTests", "src\modules\MouseUtils\MouseJump.Common.UnitTests\MouseJump.Common.UnitTests.csproj", "{D5E42C63-57C5-4EF6-AECE-1E2FCA725B77}"

Any light you can shed would be appreciated.

Looks like a tough one 🤔 It should be picking up the right one, but for some reason it's detecting dlls with different versions. I'll start by trying to fix it locally and then I'll go from there. There's a hack we did on some of them to only look for some dlls at runtime and I think that should help in the last case. Thank you for the info.

@jaimecbernardo
This might be a red herring, but I noticed this file does something with the projects for each  app (awake, hosts, colorpicker,etc) but not mousejump.
I haven't looks in detail at what it's doing, or tried to add mousejump, but figured it was worth mentioning in case it's part of the problem...

@jaimecbernardo - have you had any luck with the dll version problem?

Sorry @mikeclayton , got diverted to other tasks in the meanwhile. Basically I think it's because we're saying this Library is self contained, when we usually really usually only do that on Apps.
The latest changes I've pushed help here. Locally they seem to do the trick. Although I think I still need to do something for the tests, since they need Microsoft.Win32.SystemEvents.

I've ended up excluding the tests from the deps.json checks, since the tests dependencies won't be pushed to production :) This should fix it.

I think that's it @mikeclayton . Hope that helps :)

@jaimecbernardo - Amazing thanks - I don't think I'd have ever got that combination of changes by myself.
There's a little bit more work I want to do with the UI for the settings - I'll put the PR in "ready for review" mode when I'm happy with it...

Latest commit is still a WIP, but adds an initial "Preview Type" setting that lets the user choose between:

Compact - the current preview style with no spacing or bezels around individual screens on the preview image
Bezelled - the default "FancyMouse" style with bezels around individual screens on the preview image
Custom - display additional settings that give fine-grained control over the appearance of the preview image

Also adds a "Copy to custom preview style" button that lets the user copy the "Compact" and "Bezelled" settings to "Custom" so they can be used as a starting point for "Custom" settings.
Compact preview style

Bezelled preview style

Custom preview style

Suggestion: Why not replacing the expander header with the buttons and text content of the "preview style" settings card and collapsing the expander ny default?

Apart from a couple of small things that I still need to clean up (see the "To Do" section in the top description of the PR ^^^), I think this is mostly ready for review now.
It would be good if this PR could be added to the review queue so I can start getting some feedback on anything that needs to be changed to get it into a releaseable state...

Notes:
The bulk of the changes (100ish files) are a wholesale move of all files the in the "MouseJumpUI\Common" folder into a separate project so it can be referenced in Settings UI to draw the preview image. The rest of the changes are updating Settings UI to expose the additional style settings.
Things that might need a proper review:

Adding a reference to the "CommunityToolkit.WinUI.Controls.Segmented" package for the "Preview style" setting's "[Compact]  [Bezelled]  [Custom]" display

Folder structure in SettingsUI project for new files - e.g "Images\MouseJump-Desktop.png" and "SettingsXaml\Panels\MouseJumpPanel.xaml" (I separated the latter out as it was getting difficult to work with the large "MouseUtilsPage.xaml" .

Apart from a couple of small things that I still need to clean up (see the "To Do" section in the top description of the PR ^^^), I think this is mostly ready for review now.
It would be good if this PR could be added to the review queue so I can start getting some feedback on anything that might still need to be changed to get it into a releaseable state...

Notes:
The bulk of the changes (100ish files) are a wholesale move of all files the in the "MouseJumpUI\Common" folder into a separate project so it can be referenced in Settings UI to draw the preview image. The rest of the changes are updating Settings UI to expose the additional style settings.
Things that might need a proper review:

Adding a reference to the "CommunityToolkit.WinUI.Controls.Segmented" package for the "Preview style" setting's "[Compact]  [Bezelled]  [Custom]" display

Folder structure in SettingsUI project for new files - e.g "Images\MouseJump-Desktop.png" and "SettingsXaml\Panels\MouseJumpPanel.xaml" (I separated the latter out as it was getting difficult to work with the large "MouseUtilsPage.xaml" .

@mikeclayton

preview image settings card - problems horizontally centering image - see Problems centering Image inside in a SettingsCard CommunityToolkit/Windows#459

Did you try to set padding to 0,0,0,0?

@mikeclayton

preview image settings card - problems horizontally centering image - see Problems centering Image inside in a SettingsCard CommunityToolkit/Windows#459

Did you try to set padding to 0,0,0,0?

I tried setting it to 8,8,8,8 and that didn't work :-(
It had an 8 pixel padding on the top, right and bottom, but 32 (=8+24) pixel padding on the left.
I think the header icon is 24 pixels, so it's adding extra spacing for that - I think I read somewhere this happens automatically if the SettingCard is inside a SettingsExpander, but I can't find the reference now...

I think I've solved the centering problem, although I'm not sure I've done it the way you're supposed to solve it...
https://github.com/microsoft/PowerToys/pull/33486/commits/185b7ab82035fa0df6ed2063785b20caae0f15f6#diff-dd14cd8373d019dc6d8a8b90c219b1904ee49da9300bfc39358b290a532a2821R27-R69
This reaches into the SettingsCard internals and applies some styling to the private controls inside it.
The result is:

Suggestion: Why not replacing the expander header with the buttons and text content of the "preview style" settings card and collapsing the expander ny default?

@htcfreek - The expander is collapsed by default anyway, with the preview image and all the style controls hidden - I've been expanding it for my screenshots :-). The "custom" controls are also only visible when the "Custom" option is selected - they get hidden when you choose "Compact" or "Bezelled".
I'm not a UI guy, but I think it kind of works ok as it is - if I moved the "Preview style" buttons into the expander row they'd be separated from the rest of the settings when it's expanded (the preview image would be in the middle of them all).

@jaimecbernardo - this PR has turned into a bit of a mess due to its age and the number of changes / reverts, and Clint's excellent solution-wide *.csproj cleanup is starting to bleed in as merge conflicts. I'm thinking of closing my PR and breaking the changes in it into two much smaller PRs based of the current main branch:

Separate the "MouseJumpUI/Common" files into a separate library so it can be referenced by Settings UI. This is quite high churn (moving about 100 files) but very little changes needed in the files themselves (just namespaces)

Once that's done and merged, re-apply the actual style setting changes as a new PR.

Any concerns with this approach?

Closing this PR - all the changes have been reapplied in https://github.com/microsoft/PowerToys/pull/35521


Use partials for organisations

Clean up the new organisations pages code...

use partials for each section
use SECTION tags for sections
wrap 'what we do' section in if logic
standardise spacing classes on sections

@sihugh two things could be added to this PR...

the logic changes we discussed for the org's status live/exempt/?
we could get rid of the _show.html.erb template as all it does is then call _show_organisation.html.erb (but I can't see how to do it)

We're looking at the statuses soon.
We've got this card to look at tidying up the views


Information regarding openssl upgrade of i2d_X509_REQ() api

Hi team,
I have upgraded from openssl 1.1.1 to opessl 3.0.5. I am getting error in the i2d_X509_REQ(X509_REQ* req, nullptr) api to get the length of the content in openssl 3.0.5. I am getting the error of length of -1. Can anyone guide me fix this compatability issue? Many thanks in advance. Also, is it possible to print the reason for error due to openssl api's? It will be used to find the exact error occurred.

There should not be any compatibility issues on i2d_X509_REQ() however it is possible that something else silently fails before that call. Could you please try to create a simple reproducer to share here?

I'm facing the same issue. Please find the sample reproducer
`#include
#include <openssl/evp.h>
#include <openssl/rsa.h>
#include <openssl/bio.h>
#include <openssl/pem.h>
#include <openssl/engine.h>
#include
#include <openssl/x509.h>
using namespace std;
int main()
{
EVP_PKEY_CTX *ctx;
EVP_PKEY pkey = NULL;
ctx = EVP_PKEY_CTX_new_id(EVP_PKEY_RSA, NULL);
if (!ctx) {
std::cout<<"ctx error"<<std::endl;
}
/ Error occurred */
if (EVP_PKEY_keygen_init(ctx) <= 0) {
std::cout<<"EVP_PKEY_CTX_new_id error"<<std::endl;
}
if (EVP_PKEY_CTX_set_rsa_keygen_bits(ctx, 4096) <= 0) {
    std::cout<<"EVP_PKEY_CTX_set_rsa_keygen_bits error"<<std::endl;
}

/* Generate key */
if (EVP_PKEY_keygen(ctx, &pkey) <= 0) {
    std::cout<<"EVP_PKEY_keygen error"<<std::endl;
}

/* Writes the generated key to file*/
FILE *fp;
fp = fopen("/home/tpb1cob/1day/openssl/public_key.txt","w");
if (0 == PEM_write_PUBKEY(fp,pkey)) {
    std::cout<<"file write fail"<<std::endl;
}
fclose(fp);

/*Encode the key and write to buffer*/
unsigned char* p;
int ret = -1;
ret = i2d_PublicKey(pkey,&p);
if(ret < 0) {
    std::cout<<"i2d_PublicKey returns negative value"<<ret<<std::endl;
}
std::cout<<"i2d_PublicKey returns value "<<ret<<std::endl;
std::cout<<p<<std::endl;

/*Decodes buffer contents and assign to EVP_Key structure*/
EVP_PKEY *pkey1 = NULL;
if(NULL == d2i_PublicKey(EVP_PKEY_RSA,&pkey1,(const unsigned char**)&p,526)) {
    std::cout<<"d2i_PrivateKey_fp returns NULL pointer"<<std::endl;
}

/* Writes the decoded key to file*/
FILE *fpq;
fpq = fopen("/home/tpb1cob/1day/openssl/public_key_decoded.txt","w");
if (0 == PEM_write_PUBKEY(fpq,pkey1)) {
    std::cout<<"file write fail"<<std::endl;
}
fclose(fpq);

/* checks the parameters of keys pkey and pkey1 for equality */
if(1 == EVP_PKEY_eq(pkey,pkey1)) {
    std::cout<<"EVP_PKEY_eq success"<<std::endl;
}

//"C=UK, O=Disorganized Organization, CN=Joe Bloggs"

X509_NAME *nm;
X509_REQ *CR;

nm = X509_NAME_new();
CR = X509_REQ_new();
if (nm == NULL) {
    std::cout<<"X509_NAME_new failed"<<std::endl;
}
    /* Some error */

int ii = -1;

 if (1 == X509_NAME_add_entry_by_txt(nm, "C", MBSTRING_ASC,(const unsigned char*)"UK", -1, -1, 0)) {
    std::cout<<"X509_NAME_add_entry_by_txt country success"<<std::endl;
}
    /* Error */
if (1 == X509_NAME_add_entry_by_txt(nm, "O", MBSTRING_ASC,(const unsigned char*)"Disorganized Organization", -1, -1, 0)) {
    std::cout<<"X509_NAME_add_entry_by_txt organization success"<<std::endl;
}

if (1 == X509_NAME_add_entry_by_txt(nm, "OU", MBSTRING_ASC,(const unsigned char*)"Organization Unit", -1, -1, 0)) {
    std::cout<<"X509_NAME_add_entry_by_txt organization unit success"<<std::endl;
}
    /* Error */
if (1 == X509_NAME_add_entry_by_txt(nm, "CN", MBSTRING_ASC,(const unsigned char*)"Joe Bloggs", -1, -1, 0)) {
    std::cout<<"X509_NAME_add_entry_by_txt common name success"<<std::endl;
}

if (1 == X509_REQ_set_subject_name(CR, nm)) {
    std::cout<<"X509_REQ_set_subject_name success"<<std::endl;
}

X509_NAME_free(nm);

if(1 == X509_REQ_set_pubkey(CR, pkey)) {
    std::cout<<"X509_REQ_set_pubkey success"<<std::endl;
}

int length = i2d_X509_REQ(CR, NULL);
const char** file;
int* line;

if (length <= 0) {
    std::cout<<"i2d_X509_REQ fail"<<length<<std::endl;
}

return 0;

}`

I'm facing the same issue. Please find the sample reproducer test.zip

You do not sign the X509_REQ, that's most probably the reason for the failure.

marking as inactive, will be closed at the end of the 3.4 dev cycle without further input


chore: setup page with some examples

This is an alternative to https://github.com/webcomponents-cg/docs-and-guides/pull/2
There are primary only md or data files as the all templates are within @rocket/lauch.
Comes with

Mobile Layout (including Mobile swipeable Drawer)
Search
Service worker that caches already visited pages so they work offline
Social Media images
...

@castastrophe please take a look and then let's have a chat on how we wanna continue 🤗
Preview: https://deploy-preview-4--web-components-wg.netlify.app/

x-post from slack, as of this comment we have

:white_check_mark: site skeleton
:white_check_mark: sample page
:white_check_mark: dark mode css
:white_check_mark: webmanifest and icons
Still to do
:no_entry_sign: Initial content
:no_entry_sign: sitemap (i.e. taxonomy and overall structure)
:no_entry_sign: demos, codelabs, interop article
I've got plans for that interop article, theoretically I could start even without merging this, but I'd be more comfortable branching from main if possible. We can always shuffle content later then do a 'soft' launch

I'd like to get @castastrophe 's approval before we merge


initial-deploy-to-heroku

as a user, I can:
access a deployed version of the website so I can browse and purchase products.
implementation:
-Heroku
-browser router (switch between navbar links)

got this issue working on another branch second-heroku-deploy


TEST PR

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9558631). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   81.35%
=======================================
  Files           ?        3
  Lines           ?       59
  Branches        ?        7
=======================================
  Hits            ?       48
  Misses          ?       11
  Partials        ?        0

Flag
Coverage Δ

unittests
81.35% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Invalid authentication with New-PhpIpamSession -useAppKeyAuth

hi, I've read several times now the documentation on https://phpipam.net/api/api_documentation/ and unfortunately I miss an example script for creating the session and a post / get call.
The docu page probably refers to an outdated Github page "To simplify deployment you can use API classes for different languages available on Github - Collection of API clients for phpipam API." -> https://github.com/yoke88/PSPHPIPAM/tree/4a62b9e7018ca0bf12f3d554e1884ae3eb611440
There are newer versions on the main page: https://github.com/yoke88/PSPHPIPAM. But this brings me the same error message after a small bugfix.
Bugfix on File "New-PhpIpamSession.ps1" in line 57
[ValidateScript ({if ($ PSVersionTable.PSEdition -eq 'Core') {throw (" CrptoAuth need Rijndael256, it's not implemented in powershell core ")})]
missing the positive condition so an else {$ true} :
[ValidateScript ({if ($ PSVersionTable.PSEdition -eq 'Core') {throw ("CrptoAuth need Rijndael256, it's not implemented in powershell core")} else {$ true}})]
My call:
Import-Module PSPHPIPAM
New-PhpIpamSession -useAppKeyAuth -PhpIpamApiUrl http://server.domain.de/api -appid LocalAPI -appkey 'vidivBkZd5RsEa3VnhlkeVoV7IXDmcdq' -Debug

and my error message about that
New-PhpIpamSession : {"code":401,"success":false,"message":"Unauthorized","time":0.001}
In C:\Users\xxx\Desktop\phpIPam.ps1:3 Zeichen:1
+ New-PhpIpamSession -useAppKeyAuth -PhpIpamApiUrl http://sys-test14.hr ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,New-PhpIpamSession

I use the following Powershell version
Name                           Value
----                           -----
PSVersion                      5.1.17134.858
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17134.858
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Can someone tell me if I control the module incorrectly or if there is an error here?

which phpipam version do you use? encrypt request will not support in new phpipam version , so the test is mainly on powershell core and you may using -UseStaticAppKeyAuth switch

we use the latest version 1.4
I'll try it again soon

Hi there, Thanks for your scripts, I'm looking to create a PR with some additional functions, in the mean time I haven't been able to get this auth method to work against 1.3.2 of phpipam (-useStaticAppKeyAuth or -useAppKeyAuth)
I get 401 as well.  should it work?  Also could you comment on how you configured the App within API management?  once I know what works I might PR an update to the README.  Do you use App security for example?

see the updated readme and check the Tests*.ps1 file to see the examples


Feat: Config option to disable automatically adding snooze:send and snooze:prune to the schedular

Hi @atymic and @thomasjohnkane
Snooze currently automatically adds the snooze:send and snooze:prune commands to the scheduler.
This is convenient for 95% of the use cases, but becomes a bit of a problem if the project requires turning that off by default.
For example in our project we are making use of the spatie multi tenant package. If we want to run commands for all our tenants, we need to prefix the command with tenants:artisan (This way the active database will be set for each tenant)
So to schedule the send command as an example, we require doing the below:
$schedule->command('tenants:artisan snooze:send')->everyMinute()->onOneServer()->runInBackground();

Not being able to turn off automatically scheduling these commands, results in there always being a snooze:prune  or snooze:sendcommand scheduled, and these will fail every time they run as the current database won't be set.
To get around this currently I need to do some ugly config juggling, and actually disable snooze so it doesn't schedule the command, and the only way to not schedule the prune command is to have pruneAge set to null.
Since this might be the situation for other users, I thought of introducing a new config option scheduleCommands, which defaults to true to keep current behavior, but can be set to false if you don't want nooze to automatically schedule the commands.
/*
* Should snooze automatically schedule the snooze:send and snooze:prune commands
*/
'scheduleCommands' => env('SCHEDULED_NOTIFICATIONS_SCHEDULE_COMMANDS', true),

Note that the commands are always registered, this config option just determines if it should be added to the scheduler automatically or not.
There is no breaking change here as it defaults to true for the config option.
I've also added a note to the readme to indicate this option.

Codecov Report

Merging #92 (2016be6) into master (97e70d1) will increase coverage by 0.01%.
The diff coverage is 69.23%.

:exclamation: Current head 2016be6 differs from pull request most recent head 217acb9. Consider uploading reports for the commit 217acb9 to get more accurate results

@@             Coverage Diff              @@
##             master      #92      +/-   ##
============================================
+ Coverage     96.33%   96.35%   +0.01%
- Complexity       86       87       +1
============================================
  Files            10       10
  Lines           273      274       +1
============================================
+ Hits            263      264       +1
  Misses           10       10

Impacted Files
Coverage Δ

src/ServiceProvider.php
81.48% <69.23%> (+0.71%)
:arrow_up:

Continue to review full report at Codecov.

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

@atymic Could you please have a look at this PR when you get a second. Its the last item we need to be able to deploy this package to production (In the current state the snooze:prune command will get scheduled and cause errors when it run)
To summarize, this PR only adds the option to turn off automatically scheduling the commands, the default behavior remains unchanged

Tagged as 2.1.0

Thanks @atymic - much appreciated


Resolve missing index in dot notation.

This patch fixes issue #1 by including the index in the dot notation.

This change is

  Thanks!
Previously, mention-bot wrote…
> By analyzing the blame information on this pull request, we identified @imlucas to be a potential reviewer

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

published in<EMAIL_ADDRESS>


Update pion/transport to v2.2.2 and noRepalyDetector

Update pion/transport to v2.2.2 and noRepalyDetector

LGTM 👍🏻


Extracting 3D model

Hi Jeff & HybrIK User Group,
Can you kindly suggest the best way to export the animated 3D mesh after HybrIK has completed?
Currently, I'm only able to export a 3D mesh per each frame. But I want a single model which animates the motion found across all frames. Thank you.

Hi @miguelvarandas22

Currently, we are running Hybrik on an instance of google colab. We ran into issues running it locally which we have yet to debug.
We modified their cloned files to make it run on our videos.
HybrIK now outputs a video (with 3d mesh rendered) and a .pkl file containing SMPL joints and other data required to create animations.
We made a few modifications to the Blender addon code to make it run without blender and instead convert the pkl file to 3d mesh with animations.
we have not yet tried building our own AI models

(Many thanks to the HybrIK team for this code!)


Return regularization losses in stateless_call.

Previously, those were not returned with "return_losses=True". In the same line, we now get the regularizer object from the respective module.

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

Comparison is base (6a6e4f8) 79.59% compared to head (5f546cb) 55.97%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18960       +/-   ##
===========================================
- Coverage   79.59%   55.97%   -23.63%
===========================================
  Files         337      337
  Lines       35116    35121        +5
  Branches     6879     6879
===========================================
- Hits        27951    19658     -8293
- Misses       5585    14000     +8415
+ Partials     1580     1463      -117

Flag
Coverage Δ

keras
55.97% <100.00%> (-23.49%)
:arrow_down:

keras-jax
?

keras-numpy
55.97% <100.00%> (-0.02%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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


Is anything being worked on to expand the radius of scans?

Is anything being worked on to expand the radius of scans?

If the scan area isn't increased/fixed then this program is severely crippled. =\

The scan radius is not set by any program. It is set by Niantic. If the scan speed is too slow, create multiple accounts and open them 3-5 minutes apart from each other. This will start multiple threads of the map to scan the area faster.

It's not s scan speed thing at least the way I understand it.  The radius has greatly changed since about 3 days ago.

Did Niantic return the 100m scan circle? I have read on reddit similar reports though I have not yet heard from someone whom knows for sure

pill0ws, it would appear so to me.  :(

One report suggested 140m circle is new update range but I am skeptical to believe that. They may have underlying mechanics to the circle, perhaps updates occur but queue previously covered areas near the 10s mark? I have certainly seen things in game reveal themselves much further away than usual

They have certainly drastically reduced spawned pokemon. Perhaps an increase to scan radius was permitable after reducing the amount of events that can occur for any one update

Why is there a "Scanning Options" if Niantic appears to control this anyway?  I'm just confused about the option's purpose.

That is for the radius of the scanning pattern, not the radius of an individual update. That is how large of an area that the scanner will spiral out to

pill0ws, thanks or the explanation and suggestions.

how is it crippled? better than the in game options or anything out there.


ability to toggle gizmo while program is running

It would be nice if it was possible to control whether the gizmo is shown while the program is running, e.g. for an in-program 'editor' like bevy_editor_pls.

Would a simple state resource work for your use case?

Yes, that would work

Follow-up since I think this is quite important.
We are currently working through some changes to the command system. This will be solved as part of that work.


[darwin] Fix build failure for ios simulator (#9559).

Problem
What is being fixed?  Examples:

Fix build failure for ios simulator (#9559).

Currently, when build src/darwin/Framework, build will fail when target is ios simulator (x64). The reason being build/toolchain/ios/BUILD.gn only defined ios_x86_64.
Change overview

Fix build issue described previously

Testing
How was this tested? (at least one bullet point required)

ios simulator build succeeded with this patch.
target mac os works
target ios arm64 works

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Feng LI seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@woody-apple @sagar-apple can you take a look?

@fengli this fails the Darwin CI; see https://github.com/project-chip/connectedhomeip/runs/4099202530?check_suite_focus=true

Hi @fengli, looks like you'll need to rewrite your commit history to come from your GitHub account fengli vs your name

(Note, this does also really fail to build)

@woody-apple build failure fixed, also i used the email corresponds to my github account, CLA isn't an issue anymore.

Fast tracking, given a platform change that a platform maintainer approves (me for Darwin).


umbra-js security improvements

Contingent on https://github.com/ScopeLift/umbra-protocol/pull/176 being merged first, since this builds off that PR (hence all the extra commits)
Random Number Fixes

RandomNumber class ignores payload extension input (for now) and always generates 32 byte random numbers
Memo field was removed from the send form

Point validation

Adds a new utility method in umbra-js called assertValidPoint() which verifies that a point is on the secp256k1 curve

Other

I noticed wrongly named function inputs in DomainService.ts, so fixed that also

Both should be fixed now. There were some remnants of payload extension logic which was the reason for (1), and removing that seems to have resolved (2) as a I couldn't reproduce it—this makes sense since that error message is thrown by the strings package of ethers.js

Replaced by https://github.com/ScopeLift/umbra-protocol/pull/179


Fix CI

Follow up to https://github.com/huggingface/huggingface.js/pull/687

is the failing test related ?


Making requests when getting updates

hi. telegram recently add an ability to perform a request to the telegram Bot API while sending an answer to the webhook.
can you do a simple implement of this feature in your bot?

Afaic this is about making requests in the answer to the webhook request. twx.botapi is only for long polling updates and not for webhooks.

@9px twx.botapi can be used for webhooks (I use it always with webhooks) but it doesn't implement the webserver itself. twx.botapi already implements the set_webhook

so you setup your bot instance
start your WSGI app (with at least one route defined - the one that receives updates)

Normally, it can look something like this (in Bottle):
app = Bottle()
bot = TelegramBot(...)

@app.route('/update/<token>', method='POST')
def update(token):
    if token != bot.token:
        abort(404, 'Not found: \'/update/%s\'' % token)

    u = Update.from_dict(request.json)
    .... (do something with update) ...
    bot.send_message(u.message.chat.id, '123')
    return None

To use the webhook response, twx.botapi is not very friendly but can you can still achieve it changing the route as such:
    ...
    u = Update.from_dict(request.json)
    .... (do something with update) ...
    # CHANGE STARTS HERE, using botapi.send_message instead of TelegramBot().send_message to avoid implicit .run()
    m = send_message(u.message.chat.id, '123', token=bot.token)
    m.params['method'] = m.api_method
    # return the dict() instead of None. Bottle automatically encodes dict()s as JSON and sets proper content-type
    return m

I'm actually working on implementing this on my tgbotplug library which uses twx.botapi for API calls (though having some issues with design trying to keep it friendly :))