UWP for IoT Core Not Working

I've created a Windows 10 IoT Core Background Application using the Visual Studio Template for IoT Core. I'm trying to create an Asp.Net Core web Api using Mvc and I'm getting an exception before the service is even started.
When initializing MVC in my configureservices method (services.AddMvc or services.AddMvcCore) I get a FileNotFoundException in System.Private.CoreLib.

Hi @JeffMarqMetrix, can you please provide more details about how exactly are you trying to configure ASP.NET Core ? Can you send us a repro?

Yes. So the end goal is to run an Asp.Net Core web application with Web Api functionality and several pages. I'm attempting to run this on Windows 10 IoT Core. The examples for Raspberry Pi that I've seen are all running Asp.Net Core as a .Net Core Exe running on the device. I'm attempting to run the app as a Windows IoT Core Background Application via the templates available here. This is necessary for me to access the serial ports, since I cannot access them from a .Net Core application.
Also, the device I'm running IoT Core on is not an ARM device. It's x64 Windows IoT Core, running an Intel processor.
Side note: I did have to create my own server based on HttpListener. But aside from that, Mvc crashes the application before the server can ever get started.
When using the Visual Studio templates, the only code that needs to be added to the project to repro the issue, is the following:
StartupTask.cs:
using Windows.ApplicationModel.Background;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Builder;

namespace TestBackgroundApp
{
    public sealed class StartupTask : IBackgroundTask
    {
        public void Run(IBackgroundTaskInstance taskInstance)
        {
            IWebHost webHost = new WebHostBuilder()
                .UseKestrel()
                .UseUrls("http://*:80/")
                .UseStartup<Startup>()
                .Build();
            webHost.Run();
        }
    }
}

Startup.cs
using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.Http;
using Microsoft.Extensions.DependencyInjection;
using System;

namespace TestBackgroundApp
{
    class Startup
    {

        public void ConfigureServices(IServiceCollection services)
        {
            services
                .AddMvc();
        }

        public void Configure(IApplicationBuilder app, IHostingEnvironment env)
        {
            app
                .UseMvcWithDefaultRoute();

            app.Run(async context =>
            {
                await context.Response.WriteAsync($"Hello world! {DateTime.Now}");
            });
        }

    }
}

Again, I had to create an HttpListenerServer in order to successfully receive and process requests from my IoT Background App. And that works fine if I remove the Mvc-related code. But since Kestrel doesn't work in IoT Background Apps, you will need to use the HttpListener I've created in order to fully test.
Also, I think you're only able to test this remotely on a device that's running Windows 10 IoT Core. These types of applications cannot be started by Visual Studio on a normal Windows machine.
TestIoTCoreAspNetCore.zip

@JeffMarqMetrix, can you also share the exception details you're getting? Stack trace?

System.IO.FileNotFoundException
HResult=0x80070002
Message=Could not load file or assembly 'TestBackgroundApp, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.
Source=System.Private.CoreLib
StackTrace:
at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)
at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)
at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.DiscoverAssemblyParts(String entryPointAssemblyName)
at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.GetApplicationPartManager(IServiceCollection services)
at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.AddMvcCore(IServiceCollection services)
at Microsoft.Extensions.DependencyInjection.MvcServiceCollectionExtensions.AddMvc(IServiceCollection services)
at TestBackgroundApp.Startup.ConfigureServices(IServiceCollection services) in C:\Users\jscherrer\Documents\Visual Studio 2017\Projects\TestIoTCoreAspNetCore\TestBackgroundApp\Startup.cs:line 14

System.IO.FileNotFoundException
  HResult=0x80070002
  Message=Could not load file or assembly 'TestBackgroundApp, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.
  Source=System.Private.CoreLib
  StackTrace:
   at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)
   at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)
   at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
   at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.DiscoverAssemblyParts(String entryPointAssemblyName)
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.GetApplicationPartManager(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.AddMvcCore(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcServiceCollectionExtensions.AddMvc(IServiceCollection services)
   at TestBackgroundApp.Startup.ConfigureServices(IServiceCollection services) in C:\Users\jscherrer\Documents\Visual Studio 2017\Projects\TestIoTCoreAspNetCore\TestBackgroundApp\Startup.cs:line 14

Here are the exceptions from my output window:
Exception thrown: 'System.IO.FileNotFoundException' in System.Private.CoreLib.dll
An exception of type 'System.IO.FileNotFoundException' occurred in System.Private.CoreLib.dll but was not handled in user code
Could not load file or assembly 'TestBackgroundApp, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

Maybe it's because the IoT Core Background Application underlying project type is actually a Windows Runtime Component? It just seems to me like it has something to do with reflection.

@glennc, do you have any contacts who can help us with this?

@JeffMarqMetrix Even though this is unsupported it is something I also need for one of my projects and I'm also not satisfied with the workaround of splitting the project and deploying two separate binaries with some type of service bus to bridge the gap.  FWIW, I started debugging this and found the cause of the FileNotFound exception is in ApplicationPartManager.PopulateDefaultParts.  It tries to Load the assembly from the name of the application which doesn't appear it would work in the case of UWP since we don't end up with an exe or dll of the applications name, instead it appears there's a wrapper around it.  I don't know much about UWP at this point though, learning as I dig through this issue and attempt to hack it to make it work :)

So as it turns out, after hacking the whole assembly loading it just ends up in a PlatformNotSupported exception now, time to work through that one

Looks like the last hurdle is with the host builder, it does some reflection to get the assembly when you call Build() on it, internally that calls BuildCommonServices() and it looks like that's where the last issue lies.  Hacking around the AddMvc issue wasn't so bad, just had to manually initialize some things.
For anyone else trying to get this going, here is what I did in startup....
public void ConfigureServices(IServiceCollection services)
{
var manager = new ApplicationPartManager();
        var entryAssembly = Assembly.GetExecutingAssembly();

        var assembliesProvider = new ApplicationAssembliesProvider();
        var applicationAssemblies = assembliesProvider.ResolveAssemblies(entryAssembly);

        foreach (var assembly in applicationAssemblies)
        {
            var partFactory = ApplicationPartFactory.GetApplicationPartFactory(assembly);
            foreach (var part in partFactory.GetApplicationParts(assembly))
            {
                manager.ApplicationParts.Add(part);
            }
        }

        services.AddSingleton(manager);
        services.AddMvc();
    }

I need to doublecheck, but there was one method that had to be publicly exposed, though I assume you could hack down to it with reflection to get at it just the same without modifying the actual code for Mvc the way I did.  I'll do some more work on this and update with a workaround that doesn't require a code change.

Here is a reflection version of the little startup hack so AddMvc() will work...
    public void ConfigureServices(IServiceCollection services)
    {
        var manager = new ApplicationPartManager();

        // Do some reflection to get at the internal stuff we need to override
        var assembliesProviderType = Assembly.Load("Microsoft.AspNetCore.Mvc.Core").GetType("Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationAssembliesProvider");
        var resolveAssembliesMethod = assembliesProviderType.GetMethod("ResolveAssemblies");

        // Now implement enough of the internal functionality for things to pass
        var assembliesProvider = Activator.CreateInstance(assembliesProviderType);
        var applicationAssemblies = (IEnumerable<Assembly>)resolveAssembliesMethod.Invoke(assembliesProvider, new object[] { Assembly.GetExecutingAssembly() });

        foreach (var assembly in applicationAssemblies)
        {
            var partFactory = ApplicationPartFactory.GetApplicationPartFactory(assembly);
            foreach (var part in partFactory.GetApplicationParts(assembly))
            {
                manager.ApplicationParts.Add(part);
            }
        }

        services.AddSingleton(manager);
        services.AddMvc();
    }

Still trying to sort this out and was able to use the built in server working with the following....
public sealed class StartupTask : IBackgroundTask
{
    BackgroundTaskDeferral _deferral;

    public void Run(IBackgroundTaskInstance taskInstance)
    {
        _deferral = taskInstance.GetDeferral();

        var host = WebHost
            .CreateDefaultBuilder()
            .UseSetting("preventHostingStartup", "true")
            .UseStartup<Startup>()
            .UseUrls("http://*:80")
            .Build();

        host.Run();
    }
}

At this point things are pretty close to working, I can see 404's coming back from the UWP app from both localhost and remote requests.  I think it's just some configuration missing at this point.

If anyone can help me understand the flow of things from the request hitting IServer through to Mvc for routing that would be helpful

Good news and bad news, the good news is I tracked down why routing is failing, the bad news is I am not sure how to work around it yet.  UWP is not happy about a declaration like public class HomeController : Controller and complains about not deriving from System.Object.  I had ignored this previously and just removed public from the declaration, but that breaks routing's ability to locate the controller in my assembly since it does not add the class unless it is public.  I suppose a workaround here may be to implement my own ControllerFeatureProvider that can locate the controller class, but hoping someone else may have another idea.

Hey @los93sol
I really appreciate all of the effort you've put into trying to solve this.
One idea might be to try defining your controllers in a class library, and referencing that class library from your main UWP project.

@JeffMushmo Thank you, I had to get a little hacky with it but that did the trick!  Routing is now working and my breakpoint in the controller is being hit as well.  Looks like there's some stuff in Razor now that I need to deal with

Stuck again, this time it's Razor, it tries to reach back into my assembly to locate the views after it compiles them, and again, since we get a .winmd file it fails there.  I was thinking I could just disable view compilation in the solution file, but that doesn't appear to be doing the trick.  I also wasn't able to just drop the views in the class lib either, hmmmm

Bit of an update, I'm having trouble getting view compilation to work, I think views will have to be precompiled, but the normal way of doing this via csproj doesn't seem to be working for UWP, any thoughts?

I'm stuck so posting my repo, hopefully someone can pick this up and help finish it https://github.com/los93sol/UwpAspNetCore/

I got it working, check my repo for a demo

Wow. That's great news. I will be attempting this myself in the near future.


Capture email when running in Test profile?

Hi.
Thanks for this really helpful extension.
I note the docs say: for testing or running in dev mode, which certainly works really well.
I wanted to use it to capture emails generated by my tests.
I have the quarkus-mailpit-testing in my pom.xml (v1.2.2).
I tried it (with %test.quarkus.mailer.mock=true, explicitly, or using the default) and mock=false (just in case), and no email shows up in MailPit either way.
Am I right in saying that if my tests generate emails (i.e. when running tests using the Quarkus integrated test runner), the extension wont capture them to MailPit?
I am using the io.quarkus.mailer.reactive.ReactiveMailer.
I have annotated my tests as per the example in case it needed that to capture the email in the first place.
@WithMailbox
...
    @InjectMailbox
    Mailbox mailbox;

Hmmm.... in DEV mode, the emails arrive. In TEST mode, running the same code, there are no errors and no email. I am not sure what to expect here.
Thanks,
Murray

@flowt-au check out our Integration Tests project...  https://github.com/quarkiverse/quarkus-mailpit/tree/main/integration-tests
You will see the tests run and capture the emails and verify them.
https://github.com/quarkiverse/quarkus-mailpit/blob/main/integration-tests/src/test/java/io/quarkiverse/mailpit/it/MailpitResourceTest.java

Let me know what you find out comparing how are tests are running vs why yours are not.  A note ours is not use Reactive so maybe that is the difference but I wouldn't think so.

Ok, I will run up a standalone example then compare to mine.
Also, my use-case is to actually see the email in the MailPit UI. The reason is I use the "red-green" pattern and start development with the tests. The email generation is just one step in a long pipeline. So, to get the email looking the way I want I use the test of the pipeline as the "development environment" to generate then visually check the layout and content. It would be tricky and time-consuming to write a test the check the layout!
It occurs to me that since the tests run on a different port than the DEV profile, maybe that is why I cannot see them in MailPit on my dev port. I will write a test to check for presence like the sample you describe, and get back to you.
Thanks,
Murray

Ok. I see the issue. In my sleepy state I was expecting the email to be visually present in the MailPit UI. Doh!
By adding these lines to my test
Message message = mailbox.findFirst("mytest@test.com.au");
System.out.println("message: " + message.toString());

I can see the email in the terminal.
And, when I generate the email I add a line with the LaunchProfile string so I know if it was TEST or DEV that created it.
For my DEV run it says App is running under DEVELOPMENT profile. and for the test it says App is running under TEST profile.. Of course.
All good. Now I have the pipeline working I can just use my dev runs to fine tune the layout of the email.
BTW: The docs example here has the following in the test:
List<Mail> sent = mailbox.getMessagesSentTo(TO);

but that throws The method getMessagesSentTo(String) is undefined for the type Mailbox (and I am using io.quarkiverse.mailpit.test.Mailbox)
Thanks again,
Murray

That is Mock Mailbox docs.  you are NO LONGER using MockMail once you use Mailpit.  Follow my Testing Docs.


docs(website): fix lefthook pre-push recipe

Summary
fix wrong variable name pushed_files to push_files

Test Plan

Thanks!


Enrich Iter.mo with the usual filter, mapFilter, drop etc combinators, taking a cuefrom List.mo

@rossberg, @kritzcreek sound sensible?

SGTM. We can also take cues from OCaml's seq type perhaps.


fix(setTimeout): allow clearTimeout to be called in setTimeout callback

This is a fix for #290 - "More tasks executed than were scheduled" error.
The change is to set task.cancelFn  = null before invoking a task for non-periodic tasks. As it is no longer possible to cancel the task which is just about to be invoked, I think this should be ok. I also updated the setTimeout tests to create new zones to run the tests in, as initially the test was incorrectly passing due to the task counts not being 0 to begin with.

I have signed the CLA


Question: how can I use this library to dismiss a few on the pull down?

Hey, firstly, thank you for creating this library and also writing up an informative blog post! I found it very interesting. I'm a beginner in swiftui so apologies if this is something that's easy to do.
I'm trying to dismiss a view when you reach the top of the scrollview (similar to how the netflix app does it, where you can scroll the list and then you can reach to the top without it dismissing, but if you then pull down again, it will dismiss the view). Would really appreciate any pointers and advice you can give!

Hi @missphyrgames
SwiftUI has built-in support for this. Just present your screen in a sheet, and you'll get that behavior.
Netflix are smart to use dark mode to make it less obvious that the movie screen is in a sheet.

@danielsaidi thank you for the quick reply! I had a follow up question on this. How did they do the animation to open the clicked on movie with a sheet? I didn't think that was possible with matched geometry with a sheet?


model - code/colour streams above 'POTENTIAL' PSCIS barriers as potentially inaccessible (orange)

Seems worth differentiating between the various barrier status codes, especially if we start adding partial barriers.

I think this could also apply to a lot of the crossings in the barrier result fixes.
For examples see crossing_id 61617 on Coal Creek and various fords in ELKR.
https://github.com/smnorris/bcfishpass/blob/main/01_prep/02_pscis/data/pscis_barrier_result_fixes.csv

duplicate of #16


When the keyboard pops up, the default scroll bar options do not expand, but are compressed by the keyboard

Hi @fengwu913 can you write your issue in English? Otherwise it is hard for us to understand you.

hello @quoc-huynh-cosee   ok, my issue is
When the keyboard pops up, the default scroll bar options do not expand, but are compressed by the keyboard

@fengwu913 We'll have a look into that.

I will close the issue. If you encounter any problems, feel free to reopen it.

Hello @fengwu913,
this is the expected behavior. The of gets resized to avoid the keyboard. The is simply another widget that resizes depending on its parent.body``Scaffold``AlphabetListView
This behavior is the same as in Apples Contacts-App.
If you wish to alter this behavior you could set s to . would then be displayed behind the keyboard. However, this would hide the entries at the bottom of the list behind the keyboard.Scaffold``resizeToAvoidBottomInset``false``AlphabetListView

Okay, I'll give it a try. Thank you


Fix Issue Where _benthos_test.yaml files get included in glob -r './resources/*.yaml'

Benthos test files in the form _benthos_test.yaml files get included in glob -r './resources/*.yaml'.
We should either explicitly exclude the pattern when processing the glob, or skip files that only contain a test section when processing resources, streams, and templates
This could also fix the case where a resource has an embedded tests section and the error field tests not recognised
Original thread in Discord: https://discord.com/channels/746368194196799589/754099840354615296/938502114920001597

Thanks for reporting @kennyp, should be fixed in https://github.com/Jeffail/benthos/commit/a257e8726a0529bd951e54c04233525ff5e26333


Treeview vertical scrollbar not visible, when horizontal scrollbar enabled

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
When in treeview there are wide filenames and horizontal scrollbar is at its left-most position, vertical scrollbar is not visible. User needs to scroll treeview pane horizontally to get vertical scrollbar visible.
Steps to Reproduce

Connect to any server and unfold folder tree.
Reduce width of treeview pane, to initiate horizontal scrollbar.
Scroll treeview pane horizontally to the right.

Expected behavior: Horizontal scrollbar should be always visible to the right of treeview pane, when horizontal scrollbar is enabled.
Actual behavior: Vertical scrollbar is visible always when content height of treeview pane is greater than the pane height and horizontal scrollbar is not enabled, but is not visible when horizontal scrollbar is enabled. User needs to scroll the pane horizontally to see it.
Reproduces how often: 100%
Versions
Atom    : 1.26.1
Electron: 1.7.11
Chrome  : 58.0.3029.110
Node    : 7.9.0
Where problem is: Selector .remote-ftp-view .ftptree-view.list-tree, turning off overflow:auto solves the issue.

Fixed in updated version


corp: Implement regexp & regexp_utf8

Signed-off-by: wangggong<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: #9016
Problem Summary:
What is changed and how it works?
What's Changed:
Add regexp & regexp_utf8
Related changes
Check List
Tests

Unit test

Side effects
Release note
No release note.

@andylokandy PTAL
Maybe these two methods need to use Collation like https://github.com/tikv/tikv/issues/8986 ?

Please considering use the title corp: Implement regexp & regexp_utf8?

/label sig/coprocessor

@wangggong Yes, I think we should use Collation in regexp_utf8

Yes, I think we should use Collation in regexp_utf8

But the TiDB did not make regex_utf8 collation-aware...

But the TiDB did not make regex_utf8 collation-aware...

So we should implement regex_utf8 without collation until TiDB supported?

But the TiDB did not make regex_utf8 collation-aware...

So we should implement regex_utf8 without collation until TiDB supported?

@Xuanwo @andylokandy Or I can support it in TiDB myself before this PR; if it's not in a hurry, maybe I can handle it this weekend?

But the TiDB did not make regex_utf8 collation-aware...

So we should implement regex_utf8 without collation until TiDB supported?

@Xuanwo @andylokandy Or I can support it in TiDB myself before this PR; if it's not in a hurry, maybe I can handle it this weekend?

Since TiDB and TiKV's regex depends on 3rd-party-library, it becomes quite difficult to make it collation-aware. That's why TiDB ignores the collation. Do you have any suggestion?

But the TiDB did not make regex_utf8 collation-aware...

So we should implement regex_utf8 without collation until TiDB supported?

@Xuanwo @andylokandy Or I can support it in TiDB myself before this PR; if it's not in a hurry, maybe I can handle it this weekend?

Since TiDB and TiKV's regex depends on 3rd-party-library, it becomes quite difficult to make it collation-aware. That's why TiDB ignores the collation. Do you have any idea to implement it?

Maybe I'll read how MySQL implements it? Or just implement regex_utf8 without collation like now?

But the TiDB did not make regex_utf8 collation-aware...

So we should implement regex_utf8 without collation until TiDB supported?

@Xuanwo @andylokandy Or I can support it in TiDB myself before this PR; if it's not in a hurry, maybe I can handle it this weekend?

Since TiDB and TiKV's regex depends on 3rd-party-library, it becomes quite difficult to make it collation-aware. That's why TiDB ignores the collation. Do you have any idea to implement it?

Maybe I'll read how MySQL implements it? Or just implement regex_utf8 without collation like now?

I think you can implement it without collation first and to investigate the collation solution later on.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@Xuanwo @zhongzc PTAL

@Xuanwo @zhongzc PTAL

/test

/test

rest LGTM

rest LGTM

title should be copr instead of corp.

corp

Got it.
done~

/merge


traffic is going through the proxy but showing all POST requests to http://ocsp.digicert.com/

I am also facing the issue in java. Configured the browser proxy by selenium Webdriver, and after the website is access by Webdriver, the traffic is going through the proxy, but it is not showing as expected. Instead it is showing as all POST requests to http://ocsp.digicert.com/
Please help me to solve this issue.

After setting the ssl and sslport, the traffic is recorded as normal.


Request for Training Code for Human-Robot Handovers in handover-sim Environment

I am impressed by the capabilities and features of the handover-sim environment.
I kindly request your assistance in accessing the training code for human-robot handovers in the handover-sim environment. If it is possible, I would greatly appreciate it if you could share the code with me or provide any guidance on how I can access it. Any additional documentation, instructions, or insights you can provide would be immensely helpful.

Thank you for your response. I appreciate your great efforts in preparing the training code for the CVPR'23 paper. I will eagerly await the release of the code and any updates you provide.
Thank you again for your assistance!

We have released the code of our CVPR'23 paper here.


Spanner queries run from inside transaction callback via non-txn object silently fail

If I'm inside a transaction callback (runTransaction()) and I try to execute some query on the database outside the transaction, things silently fail.
They should probably either say "you can't do that!" or ... work. Here's a snippet that shows what's happening...
var gcloud = require('google-cloud')({ projectId: 'jjg-cloud-research' });
var spanner = gcloud.spanner();
var database = spanner.instance('test-instance').database('test-database');

database.runTransaction().then(function (data) {
  var txn = data[0];

  var query = 'SELECT * FROM Singers';

  // This works just fine.
  txn.run(query).then(function (results) {
    console.log('txn.run() row count:', results[0].length);
  }).catch(console.log);

  // This should either work or throw an error...
  database.run(query).then(function (results) {
    console.log('database.run() row count:', results[0].length);
  }).catch(console.log);

  // This works just fine.
  var testdb = spanner.instance('test-instance').database('test-database');
  testdb.run(query).then(function (results) {
    console.log('testdb.run() row count:', results[0].length);
  }).catch(console.log);

});

I'm not sure what's going on, but the database.run() callback never runs:
$ node test.js
txn.run() row count: 3
testdb.run() row count: 3

Which is super duper weird. Can we either make it throw an error saying "you can't do that outside of a transaction" or make it work as expected? I'd prefer it just work rather than throw an error, but if there's a reason it can't, then I guess we'll survive...
/cc @callmehiphop

So I believe this is actually a result of the session pooling. By default the pooling library sets the max number of sessions in the pool to 1 and you're hitting a race condition of sorts where you're waiting for a session to free up before it can run your second query, hence why it's hanging.
As a work around you can increase the number of sessions via
var database = spanner.instance('test-instance').database('test-database', { max: 2 });

Alternatively if you want an error you should be able to specify the fail option
var database = spanner.instance('test-instance').database('test-database', { fail: true });

But I think in general we're going to bump up the number of sessions available in the pool by default, is there a sensible default recommendation out there?
/cc @vkedia @stephenplusplus

Wait a minute. The session pool won't create new ones? What's the point of a session pool that locks up at 1? :(
Are those the guidelines that the Spanner team suggested? A session pool maxed out 1 ?

The session pool won't create new ones?

Well, you specify a max and it'll create new sessions up until that number is hit.

What's the point of a session pool that locks up at 1?

This is my bad, I don't ever remember seeing anything official about the default number of sessions and the pooling lib we used sets the default to 1 for whatever reason.

Are those the guidelines that the Spanner team suggested? A session pool maxed out 1 ?

No! I don't think the default max was specified - or if it was, I must have missed it.
This is a pretty easy change, just looking for a little direction on what to set the default max at :)

Just to clarify, adding a {max: ...} directive when I create the database fixes the problem. But nothing is hanging. Things just run immediately and then finish. Which seems totally crazy.
Some thoughts:

The session pool max should be somewhere in the range of 5... 1 is a silly number for a session pool...
The session pool configuration should not be called max on a "database" constructor. maxSessions, maxSessionPoolSize, etc are all reasonable names. max is way to vague unless I'm creating the session pool itself (but I'm only creating a database).
If I turn on fail, I get the error I'd expect. But at the very least I'd expect it to hang... Not to silently fail (which is what it does). I recognize that increasing the session pool max size will fix some of this, but can we make this so that it doesn't ever silently fail and exit as this snippet does?

I don't think what you're seeing is it silently fail, but rather waiting for a session to free up. We can add an option for a timeout, but the original specification was to either wait indefinitely for a session to free up or to throw an error if the user desires it.

All I can tell you is what I'm seeing. I have a snippet of code where if I run it without a different max size, it exists cleanly and the output is:
$ node test.js
txn.run() row count: 3
testdb.run() row count: 3

If I increase the max session pool size, it also exits cleanly, but the output is:
$ node test2.js
txn.run() row count: 3
database.run() row count: 3
testdb.run() row count: 3

If that's not "silently failing" then I'm not sure what is...

Ah ok, I think I misunderstood, I'll take a peek at that then.

As another example, check this out (I did some hand editing to refer to Singers, even though that's not the table I'm using, but hopefully that doesn't cause any problems)
var gcloud = require('google-cloud')({ projectId: 'jjg-cloud-research' });
var spanner = gcloud.spanner();
var instance = spanner.instance('test-instance');
var database = instance.database('test-database');

database.runTransaction({readOnly: true}, function (err, txn) {
  var query = 'SELECT * FROM Singers';

  database.run(query).then(function (results) {
    console.log('Outside transaction row count:', results[0].length);
  });
  txn.run(query).then(function (results) {
    console.log('Inside transaction row count:', results[0].length);
  });

  var table = database.table('Singers');
  table.insert( /* row entry */ ).then(function () {
    console.log('Added a new entry!');

    database.run(query).then(function (results) {
      console.log('Outside transaction row count:', results[0].length);
    });
    txn.run(query).then(function (results) {
      console.log('Inside transaction row count:', results[0].length);
    });
  });
});

And run log:
$ node test2.js
Inside transaction row count: 15
$ echo $?
0
$ # Add {max: 10} to the database() constructor and run again.
$ node test2.js
Inside transaction row count: 15
Added a new entry!
Inside transaction row count: 15
Outside transaction row count: 16
Outside transaction row count: 16
$ echo $?
0

(Yea the order is off because I don't have the dependency relationship set up properly... that's my bad, but the point still stands.) Looks like that with not enough sessions, things just "give up" on the code and act like there's nothing wrong.

Why not just make it unlimited? The maximum number of sessions you would end up creating is anyways bounded by maximum concurrency of your app since a session would only be created if there is no free session.

Also, what the heck is up with that guy with bizarre hair?

@lukesneeringer I think I accidentally engaged @jgeewax in a meme battle


Error: $config is not defined

Everything was working fine, but I did have a build error on a StaticSite, which I rolled back. After that I get the 'Error: $config is not defined' message. I suspect something in the .sst folder might have gotten corrupted?

npx sst dev

Error: $config is not defined
Trace: ReferenceError: $config is not defined
at file:///home/rupert/sc/gitlab/elm-realtime/.sst.config.1733227067471.mjs:4:26
at ModuleJob.run (node:internal/modules/esm/module_job:194:25)
at process. (file:///home/rupert/.npm/_npx/35e81a5509205fcc/node_modules/sst/cli/sst.js:58:21)
at process.emit (node:events:525:35)
at process.emit (node:domain:489:12)
at process._fatalException (node:internal/process/execution:149:25)
at processPromiseRejections (node:internal/process/promises:288:13)
at process.processTicksAndRejections (node:internal/process/task_queues:96:32)
===
Possibly related to: https://github.com/sst/sst/issues/4458

Weirdly removing .sst folder did not cure the problem.
I had to remove node_module and reinstall them to fix it??

It seems like you were somehow running sst v2. Closing for now.

Could be a versioning issue, somehow npm installed v2 perhaps. Certianly removing node_modules and re-running npm install fixed it.


ClassPool functionality

Main Functionality

Support for generating components at runtime
Support for lazily generating some components that are installed at configuration time.
Support for generating containers that are being lazily created.

? Are there always a ClassPool installed in a Container
A Why not?
Must integrate well with ClassLoader specified for on the container configuration

One of the main problems is
ClassPool.Builder vs ClassPool.
Since ClassPool is immutable.
When do we create the actual ClassPool?


Parse x-error head from Pocket 403 response

Currently we only attempt to parse the x-error head if the response is in 400...403. This to should be changed to 400..403 (i.e. only two dots).

Fixed in #81


Unsupported tests for foreign items

We have two tests in:
rust-tests/cbmc-reg/ForeignItems/fixme_main.rs
rust-tests/cbmc-reg/ForeignItems/fixme_varadic.rs

fixme_varadic.rs works if called as kani fixme_varadic.rs --c-lib lib.c (I think the kani-flags are wrong)
fixme_main.rs does not work because of a single assertion. I think this is either a bug in the implementation or the test itself, and needs to be reviewed.

C-lib support is still experimental. Removing "MLP" label.


WiP: Communication error handler added

error handler callback added

see https://github.com/canoo/dolphin-platform/issues/510


Using PgUp in Powershell w/ integrated terminal inserts weird text

Issue Type: Bug
I found an issue with integrated terminal using Powershell: when there are many lines of output, using PgUp key a dozen or so times inserts strange lines of text into the output, with one character per line looking like this:
S
*
Y
*
N
*
C
*
0
0
0
0
0
0
2
d

If I scroll using the mousewheel or dragging the scrollbar this odd text does not get inserted.
I tried with VS Code Insiders and get the same results.
Here is a basic python script I use to make some output to scroll through:
test.py
from __future__ import print_function

for i in range(1, 10000):
	print(i)

From integrated terminal:  c:\Python27\python.exe test.py
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770K CPU @ 3.50GHz (8 x 3498)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.94GB (8.99GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

gitlens
eam
9.5.1

lex-flex-yacc-bison
fau
0.0.3

cpptools
ms-
0.21.0

vscode-docker
Pet
0.5.2

cmake
twx
0.0.17

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693


Cannot get into the wbe page" http://www.xrce.xerox.com/Our-Research/Computer-Vision/Learning-Visual-Representations/Deep-Image-Retrieval"

Hi,I have tried to open the wbe page you supplied, but failed. Can you give us details of how to run the source code in README file? For example,how to set the parameters such as dataset path? I really hope you can list them clearly one step by step,thank you very much!

Does this work for you? It is a README from the original code-base from
Gordo et. al
On Wed, Jun 20, 2018 at 10:18 AM, wangcx2018<EMAIL_ADDRESS>wrote:

Hi,I have tried to open the wbe page you supplied, but failed. Can you
give us details of how to run the source code in README file? For
example,how to set the parameters such as dataset path? I really hope you
can list them clearly one step by step,thank you very much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AaltoVision/Object-Retrieval/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ANi_SznGsJcJErMP51HEWX_iHxhJhzbWks5t-fdDgaJpZM4Uuu1F
.

This package contains the pretrained ResNet101 model and evaluation script for the method proposed in the following papers:
Deep Image Retrieval: Learning global representations for image search. A. Gordo, J. Almazan, J. Revaud, and D. Larlus. In ECCV, 2016
End-to-end Learning of Deep Visual Representations for Image Retrieval. A. Gordo, J. Almazan, J. Revaud, and D. Larlus. CoRR abs/1610.07940, 2016
Dependencies:

Caffe
Region of Interest pooling layer (ROIPooling). This is the same layer used by fast RCNN and faster RCNN. A C++ implementation can be found in https://github.com/BVLC/caffe/pull/4163
L2-normalization layer (Normalize). Implemented in C++ in https://github.com/happynear/caffe-windows. As an alternative, we provide a python implementation of this layer that produces the same results, but is less efficient and does not implement backpropagation.

Datasets:
The evaluation script is prepared to work on the Oxford 5k and Paris 6k datasets. To set up the datasets:
mkdir datasets
cd datasets
Evaluation script
mkdir evaluation
cd evaluation
wget http://www.robots.ox.ac.uk/~vgg/data/oxbuildings/compute_ap.cpp
sed -i '6i#include ' compute_ap.cpp # Add cstdlib, as some compilers will produce an error otherwise
g++ -o compute_ap compute_ap.cpp
cd ..
Oxford:
mkdir -p Oxford
cd Oxford
mkdir jpg lab
wget http://www.robots.ox.ac.uk/~vgg/data/oxbuildings/oxbuild_images.tgz
tar -xzf oxbuild_images.tgz -C jpg
wget http://www.robots.ox.ac.uk/~vgg/data/oxbuildings/gt_files_170407.tgz
tar -xzf gt_files_170407.tgz -C lab
cd ..
Paris:
mkdir -p Paris
cd Paris
mkdir jpg lab tmp
Images are in a different folder structure, need to move them around
wget http://www.robots.ox.ac.uk/~vgg/data/parisbuildings/paris_1.tgz
wget http://www.robots.ox.ac.uk/~vgg/data/parisbuildings/paris_2.tgz
tar -xzf paris_1.tgz -C tmp
tar -xzf paris_2.tgz -C tmp
find tmp -type f -exec mv {} jpg/ ;
rm -rf tmp
wget http://www.robots.ox.ac.uk/~vgg/data/parisbuildings/paris_120310.tgz
tar -xzf paris_120310.tgz -C lab
cd ..
cd ..
Usage:
(0)[9336]$ python test.py
usage: test.py [-h] --gpu GPU --S S --L L --proto PROTO --weights WEIGHTS
--dataset DATASET --dataset_name DATASET_NAME --eval_binary
EVAL_BINARY --temp_dir TEMP_DIR [--multires] [--aqe AQE]
[--dbe DBE]
G: gpu id
S: size to resize the largest side of the images to. The model is trained with S=800, but different values may work better depending on the task.
L: number of levels of the rigid grid. Model was trained with L=2, but different levels (e.g. L=1 or L=3) may work better on other tasks.
PROTO: path to the prototxt. There are two prototxts included.
deploy_resnet101.prototxt relies on caffe being compiled with the normalization layer.
deploy_resnet101_normpython.prototxt does not have that requirement as it relies on the python implementation, but it may be slower as it is done on the cpu and does not implement backpropagation.
WEIGHTS: path to the caffemodel
DATASET: path to the dataset, for Oxford and Paris it is the directory that contains the jpg and lab folders.
DATASET_NAME: either Oxford or Paris
EVAL_BINARY: path to the compute_ap binary provided with Oxford and Paris used to compute the ap scores
TEMP_DIR: a temporary directory to store features and scores
Note that this model does not implement the region proposal network.
Examples (adjust paths as necessary):

Rigid grid, no multiresolution, no query expansion or database side feature augmentation:
python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Oxford --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Oxford
Expected accuracy: 84.09

python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Paris --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Paris
Expected accuracy: 93.57

Rigid grid, multiresolution, no query expansion or database side feature augmentation:
python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Oxford --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Oxford --multires
Expected accuracy: 86.07

python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Paris --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Paris --multires
Expected accuracy: 94.53

Rigid grid, multiresolution, query expansion (k=1) and database side feature augmentation (k=20):
python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Oxford --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Oxford –multires --aqe 1 --dbe 20
Expected accuracy: 94.68

python test.py --gpu 0 --S 800 --L 2 --proto deploy_resnet101_normpython.prototxt --weights model.caffemodel --dataset datasets/Paris --eval_binary datasets/evaluation/compute_ap --temp_dir tmp --dataset_name Paris –multires --aqe 1 --dbe 20
Expected accuracy: 96.58
If you use this model please cite:
@inproceedings{Gordo2016a,
title={Deep Image Retrieval: Learning global representations for image search},
author={Albert Gordo and Jon Almazan and Jerome Revaud and Diane Larlus},
booktitke={ECCV},
year={2016}
}
@article{Gordo2016b,
title={End-to-end Learning of Deep Visual Representations for Image Retrieval}
author={Albert Gordo and Jon Almazan and Jerome Revaud and Diane Larlus},
journal={CoRR abs/1610.07940},
year={2016}
}
Please see LICENSE.txt for the license information.


Output path linking is broken if Learn Authoring Pack is installed.

1. Describe the bug
When AL compiler warnings appear in the OUTPUT tab of VS Code, then they are not linked anymore.
2. To Reproduce

Change AL code to provoke any kind of "OUTPUT", and run AL: Package: At least one file will be listed, and the path to the file is linked.
Install the Microsoft Learn Authoring Pack extension for VS Code. The pack includes multiple other extensions, whereas the learn-validation extension will cause the issue.

3. Expected behavior
The file paths should still be linked.
4. Actual behavior
File paths are not linked.
5. Versions:

AL Language: 12.6
Visual Studio Code: 1.86.1
Learn Authoring Pack: 1.0.1
learn-validation: 0.4.2

This looks like learn-validation is adding markdown to output which is breaking the automatic linking of filepaths. We're not doing anything special to link the filepath from the output. I'm going to transfer this over to their repo to see if there's anything they can do about it (but I'm not too hopeful since the repo hasn't been active in two years).

Ah, I can't transfer this issue there. You could try re-raising it with them: https://github.com/docascode/vscode-docs-build
Sorry that we can't do much more on our end. If it's really blocking, I could try pinging internally but I expect the turnaround time for that to also be slow.


Bot not updating the patches. Might be due to changes with patcher

From yesterday, bot is not updating the patches. I checked revanced and anddea patches.

The Anddea repository just started to use API v20 (the .rvp file type instead of .jar) which is currently not supported by the ReVanced Compose Manager. Since this repository is specifically for the Compose Manager, there is no point in supporting API v20 patch bundles. Also there is no set .json file format for API v20 like there is for v19 (what this repository supports).
There are a couple other patch bundles that updated to API v20 (like inotia00 and a couple smaller repos) which are not being updated here either. Until the ReVanced team updates the Compose Manager to support API v20, this repository will stop updating patch bundles when they switch to API v20. Once support is added I will update everything to support v20.
If you want to use patch bundles that updated to API v20 (like Anddea's or inotia00's), trying using the official ReVanced Flutter Manager. Keep in mind that the flutter manager does not work the same when it comes to using an alternative patch source.

Now compose starts supporting .rvp patches. You have to make changes in the links for inotia and anddea.

Thanks for letting me know. You are partially correct. The [latest commit[(9dc716b1c80dd9105ca9e8ee9de9dd6e60d52aae) on the Compose Manager branch seems to have added support for .rvp. Also, there is a finalized format for the new API's JSON files. Because of this, I did update this repo to support the .rvp files, in addition old the old system too (.apk & .jar). There is currently no updated release with this change, but there should be one soon. When there is, the download link to it will be atomically linked/updated here.

Is there something you left to add? It is showing error on adding patch bundles.

As I stated in the previous message:

There is currently no updated release for the Compose Manager with this change, but there should be one soon.

This means that there is no release yet with API v4 update. As I stated here:

The latest commit on the Compose Manager branch seems to have added support for .rvp.

There is a release in Dev branch. I asked about that. Check this

There is a commit. You are correct. But there is no release. A release (a compiled/built APK file of the Compose Manager) would be seen here, and would be for the compose-dev branch, not the compose/downloader-system branch like you see in the latest. If you can't wait for a compiled release through the ReVanced Managers repository, you can alternatively compile/build it yourself. There is documentation on how to do that here.

Just wanted to let you know that the latest build of the Compose Manager supports API v4 now. This means you can use the API v4 patch bundles in this repository, and can no longer use any API v3 bundles (.apk/.jar). Due to this, under the latest download URL for the Compose Manager, I also provided the latest download URL that supports API v3.


EventHubBufferedProducerClient: Infrastructure and Functionality

Summary
Publishing events using the EventHubProducerClient is optimized for high and consistent throughput scenarios where an application would like to assert control over collecting events into batches and the decision of when those batches are sent to Event Hubs. Developers are expected to build and manage batches according to the needs of their application, while taking into account the constraints that Event Hubs places on which events may be batched together. This allows developers to prioritize trade-offs between ensuring batch density, enforcing strict ordering of events, and publishing on a consistent and predictable schedule.
Feedback from developers has indicated that this approach quickly becomes complicated in real-world applications and demands applications bear the burden of that complexity. A frequent ask is for the Event Hubs client library to include functionality to abstract that complexity away from applications and manage batching, concurrency, and sending implicitly. Developers have let us know that they would like a higher-level type similar to the event processor, but for publishing events rather than consuming them.  This is the goal for the EventHubBufferedProducerClient.
Scope of Work

Implement the design for the EventHubBufferedProducerClient,  enabling basic functionality for:

The client can be created
Events can be enqueued
Enqueued events are published

Out of Scope

Partition assignment for round-robin and partition key hashing;  this is covered under a dedicated issue

Flush and clear functionality;  this is covered under a dedicated issue

Success Criteria

The design has been implemented; developers can use the producer in limited form, enqueuing events that are published with handlers being signaled.

Events are assigned random partitions, including those with partition keys.

The tests necessary for its validation have been created or adjusted and pass reliably.

The existing test suite continues to produce deterministic results and pass reliably.

Related Issues and References

Design Proposal for the EventHubBufferedProducerClient
EventHubBufferedProducerClient: Partition Assignment (#23480)
EventHubBufferedProducerClient: Flush and Clear (#23482)

Adjusting estimate with push to new milestone.    Original LOE:  7


Fix decoupled gpu output error handling

Previous PR: https://github.com/triton-inference-server/python_backend/pull/361
Related PR: https://github.com/triton-inference-server/server/pull/7258
Fix two things:

The decoupled GPU tensors output does not check if the GPU output buffers are successfully allocated. This adds the check.
If there is an error in buffer, it will throw an exception instead of logging the exception. The stub/parent turn flag will be updated correctly if an exception is raised or completed successfully.

Enhance one thing:

The error message returned to the request will match the full one printed to the log, instead of only returning the basic message from the model.

Next PRs:

https://github.com/triton-inference-server/server/pull/7292
https://github.com/triton-inference-server/python_backend/pull/363

Rebase onto https://github.com/triton-inference-server/python_backend/tree/jacky-res-sender-main for merging.


Add addition details to runWithRetry error message

Description
We've seen a few cases where a host bug manifests as triggering an error with message "runWithRetry was Aborted". This message doesn't give any hint as to which API was being called when it was aborted, or the reason for the cancellation.
This PR provides a new-and-improved version of runWithRetry that returns a result (failed, or aborted) instead of throwing when something goes wrong, allowing us to move the error instrumentation to the caller (Container.attach() in this case), where we can put the appropriate level of detail in the error message.
Fixes AB#669
Reviewer Guidance
I put brackets around the variable info to highlight that, hopefully making it easier to navigate between code and logs, and to provide an standard way to aggregate (via regex in Kusto) across these values if so desired.
Any relevant logs or outputs
The error message for this error changes.
Before:
runWithRetry was Aborted

After (for example):
runWithRetry for [createFile] was aborted with cancelDescription [ContainerClosed]

Follow-ups

Replace the old one with the new on in the next branch.
~Open an issue to use reason on AbortController.abort and AbortSignal and plumb it through the "aborted" result.~ See existing AB#674

Very interesting dicussion/direction, but ballooned out of scope for the small time window I had during OCE for a "quick fix" to the error message.  Hopefully this PR/discussion will be a good reference in the future.


[0.8] Polymer.dom() should expose innerHTML and textContent

Now that we have to use the Polymer.dom() to cover both Shadow and Shady DOM, we're finding it hard to programmatically get and set element content. We'd like to be able to get and set both Polymer.dom(element).innerHTML and Polymer.dom(element).textContent. From what I can tell, neither of those are defined by the result of Polymer.dom().
While we can try to roll our own Shadow/Shady implementations of innerHTML and textContent by looking at Polymer.dom()'s .children or .childNodes properties, it seems much better to share reliable, platform-supported helpers for that.

+1
innerHTML and textContent are really missing. We made some custom code which should lead to the same result - but we're not sure if it's the best/fastest approach. So we would appreciate an official solution as well.

I believe these are on the way. @sorvell knows more.

Yup, these should be coming soon.
On Tue, Apr 21, 2015 at 10:27 AM, Justin Fagnani<EMAIL_ADDRESS>wrote:

I believe these are on the way. @sorvell https://github.com/sorvell
knows more.
—
Reply to this email directly or view it on GitHub
https://github.com/Polymer/polymer/issues/1429#issuecomment-94879791.

This was completed in https://github.com/Polymer/polymer/pull/1462.

Fantastic, thanks!


added prioritize display of successful resource searches

Hello!
I've made some simple changes to the code to prioritize the display of successful resource searches. Some of my colleagues and I believe that this would be useful.

Thanks you @n1Z3R


Feature Request: Bookmarks and Line Numbers

I have a longish chapter where I need to refer to different parts of the chapter as I am writing.
Is it possible to implement bookmarks to allow one to quickly move from one section to another? Failing that, is it possible to show line numbers in the code view to make scrolling from one section to another easier?
Thanks

I notice that this request has been tagged for gitbook.com. I should have mentioned that it is a feature request for GitBook Editor.

Also note that when gitbook plugins fail, the line number is normally key backtracing the error. Painful when I need to open my gitbook in Atom.io to trace issues.

I love the improvements with Editor lately. Is there any sign of the line number feature being added?

Version 7.0.4 now. Any sign of either feature being implemented?

@manngo Maybe an easy to implement solution would be to show a small TOC of the title hierarchy local to that file/page, screen space permitting. Where you click on a title to scroll your edit view there.
That would avoid users having to manually create bookmarks, yet still providing a structured way to navigate long pages.
Maybe this is also a sign that the page should be broken out into subchapters.


absolute_url vs relative_url

[ ] This is a question about using the theme.
[ ] This is a feature request or enhancement.
[X] I believe this to be a bug with the theme.

[X] I am using the latest version of the theme.
[X] I have updated all gems with bundle update.
[X] I have tested locally with bundle exec jekyll build.

Environment information

**Minimal Mistakes version:latest
**Jekyll version:3.7.3

Expected behaviour
Seems like this theme uses absolute_url, which will yield urls like <IP_ADDRESS>:PORT when you use it with docker. I've noticed that your other theme "so-simple" works well with docker as it uses relative_url. I was wondering if there's a reason for that. I'd be happy to update it all and send you a PR, but wanted to check with you first if there's a reason. Should all work the same after the update.

Are you correctly assigning JEKYLL_ENV=production at build? Jekyll defaults development, which overrides url with localhost... or in your case <IP_ADDRESS>:PORT.
Give production a try first. absolute_url could likely be changed to relative_url, but I think your issue is the environment flag. You might want to search through the closed issues as I know I've seen others who had the same problem and solved it without messing with the theme.

Here's how to solve it. Closing as this is a duplicate.
https://github.com/mmistakes/minimal-mistakes/issues/1162#issuecomment-320642249

Thanks @mmistakes. I'd already tried that, but it doesn't quite solve the problem. When you set the build for production, you're basically saying "use the url as it is in my _config.xml". In the example you gave me, the author changes the url to be what they want locally. However, deploying this to production would mean changing that again, which is prone to errors.
Ideally, if you use a relative path instead, the url stops getting on the way.

This is where multiple config files can come into place. You can daisy chain them in your build command.

Specify config files instead of using _config.yml automatically. Settings in later files override settings in earlier files.
ref: https://jekyllrb.com/docs/configuration/

# _config.yml
# contains all your normal configs

# _config.dev.yml (or whatever you want to name it)
# add any variables you want to override for development purposes

url: http://localhost:4000

Then your Jekyll build command would change to jekyll serve --config _config.yml,_config.dev.yml

Right on! Doing that solved it.
Still think it would be easier to just use relative urls to avoid the "hackery" around how you start Jekyll, but this completely solved it.
Thank you for the responsiveness!


Curious about choices

I'm comparing your implementation to this one: https://gist.github.com/daliwali/09ca19032ab192524dc6
For instance:

Error.call(this, message) vs super(message)
Object.defineProperty(this, 'name' ... vs get name()
Lack of if (Error.hasOwnProperty('captureStackTrace'))
Lack of fallback to Object.defineProperty(this, 'stack', {

I see in your commit history that you had some of this and then changed it. I'd love to learn what problems you had or reasons for the changes.
Thanks!

@tauren I turned my gist into a module on its own.
https://github.com/0x8890/error-class
I tried to closely emulate the native Error class, but there will still be some discrepancies (name exists as a property on the object).

Basically, I started by following this Stack Overflow answer and tweaking to get it working with ES6 syntax.
To be honest, I don't elderly remember the reason for the changes, but I believe it had to do with errors that extend this class would show up as ExtendableError when logged. I had other issues getting these errors to work with Bluebird's .catch(predicate, handler) syntax (which was my main purpose in wanting something like this).
I'll take a look at the links suggested and see if I can get this working a bit better.

@bjyoungblood  if (Error.hasOwnProperty('captureStackTrace')) is needed to support not v8 browsers


🐛 Bug Report: yarn backstage-cli versions:bump --release X.Y.Z installs latest instead for many plugins

📜 Description
if we try to use backstage-cli to target an intermediate non-latest release version, e.g.
yarn backstage-cli versions:bump --release 1.7.2,
the versions added/installed to package.json use ^ which for A.B.C version format would increase B to latest available, but for most/all Backstage plugins that is the effectively major version, i.e. can have breaking changes when increased.
👍 Expected behavior
It seems like it would make more sense to hard pin the version (no ^, ~ etc)
👎 Actual Behavior with Screenshots
yarn backstage-cli versions:bump --release 1.7.2 produces installed plugins with many much more recent versions, eg we had to implement fixes only mentioned in release notes for 1.15.1.
👟 Reproduction steps
On an older installation of Backstage, eg 1.5.1, run:
yarn backstage-cli versions:bump --release 1.7.2
and compare installed version in yarn.lock against package.json. Many plugins will have a version that is the latest Backstage release instead of the targeted version in the command
📃 Provide the context for the Bug.
As we were behind on versions recently we wanted to target an intermediate version to avoid too many breaking changes in one upgrade task, but hit on the above problem, causing other complications, as we get versions different from upgrade-helper, we see items to fix covered in release notes of versions far beyond our target version, etc.
Also different developers working on their plugins running the same command ended up with different plugin versions in yarn.lock, depending on when they ran it and what was the latest released for the plugin, instead of a pinned target version.
🖥 Your Environment
Should occur on any environment, as this is due to the ^ included in package.json versions.
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, I don't have time to work on this right now

@Rugvip I would like to work on this. Could you please assign me this?

Assigned!
I'm not sure what the exact best behavior is here - semver to "equal or best compatible higher" should ideally be safe. Maybe it could be an option to use exact versions or not? @Rugvip FYI.

Note also that ^ for 0.x versions only is meant to bump the patch version, because for zero-major packages. This is what npm and yarn do. Did you see a different behavior than that?

@freben I saw that mostly ^ has been used which is automatically updating to major versions.

Are you sure? Do you have an example of a package name, and the version range that it started with, and the version you got resolved to in the end?

Note the separate treatment of 0.x versions in the NPM docs

@freben
https://github.com/backstage/backstage/blob/5623a818b957b0ef3d44434cd4b8afbf9ae43f7d/packages/integration-react/package.json
Can we replace ^ with either ~ or just version and then having a script to bump in the latest version of packages like backstage/core-components?

@aakankshabhende we prefer using ^ because it minimizes package duplication and in general results in the fewest headaches for users. This generally shouldn't be a problem for projects on old versions either because Yarn will install dependencies in a way which satisfies all ranges.
An important think to note here is that the Backstage version does not adhere to semver, meaning that a bump from for example 1.5.0 to 1.7.0 will most likely contain breaking changes. That has nothing to do with Yarn or dependency ranges in our packages, it's simply how we version things. The k8s version policy is what inspired us to go for that particular system.
Now we could discuss whether it's worth locking down out package dependency ranges to either ~ or exact version. We do have the tooling that still makes it convenient to stay on top of the latest releases anyway. Though I don't like it for two reasons, one is that it will make patching harder, it's more straightforward to have automated dependency bumps when you only need to unlock yarn.lock entries for patches. The other is that the @backstage/ packages would become "special" in the backstage ecosystem, because a fixed version would not make sense for any other packages since you want that range to be there to avoid duplication and receive patches faster. I think it's better if we just have a single preferred way to handle dependencies in the ecosystem.
As a final note I'd also say that we optimize for the best experience for users that keep their projects up to date. If we can improve the experience for users on older versions that's fine, but it can not come at the cost of a worse experience for the aforementioned users.
I'm going to close this issue as the original bug reported is intended behavior that we do not intend to change. At least not without a broader discussion that would require an RFC.


Resourcepacks loading is slow (and 'incomplete')

I've got many Resourcepacks (~39 valid ones). Now if i start MCEdit, it takes quiet long to show me the start screen. I can shorten this by rename the Resourcepacks-Folder (so MCEdit dont find them).
This is annoying with users with many packs.
I timed it: 1 minute 10 seconds with all the 39 packs, About one second without the packs.
An idea would be only do the process in the background so you dont have to wait very long.
In addition, the list of the packs is actually 'incomplete': (Out of 39 vaild ones)

Also, "mctextures" is NOT a Resourcepack (it lacks of pack.mcmeta) but MCEdit just says its a valid pack.
I am using MCEdit <IP_ADDRESS> (64-bits). This also happend with older versions (1.2?).
Most packs are for MC 1.8, some of them are made for 1.7. There are 2 "duplicates".
MCEdit runs on Windows 7 HE 64-Bits.
mcedit.log doesnt include any information about the resourcepacks.

I suggest using zipfile.open instead of zipfile.extract. You don't need to extract all the textures to disk.

I'll modify the code to use that, but it might now improve loading times significantly.

https://github.com/Khroki/MCEdit-Unified/commit/c928b622290a540c78b47b557a1cb55e6c257610 Zipped Resource Packs no longer extract to disk, as per @codewarrior0's suggestion

I finally can run MCEdit from source, now with your little change it take about 4 seconds to get the window to appear.

Is that a good enough improvement for you, or would you like me to do some more fine tuning with it?

Does your fine tuning involve more Podshot patented bugs :P

Heh, no, 4 seconds is acceptable. Better than waiting a minute.


Make the viewstamped replication implementation into a reusable library

It'd be great to make the viewstamped replication implementation into a reusable library, just like hashicorp's raft implementation.
https://github.com/hashicorp/raft

That's the plan, but not the medium-term plan.


Vulnerability in axios

A vulnerability CVE-2020-28168 detected in package axios<0.21.1 is referenced by sp-react-native-in-app-updates via<EMAIL_ADDRESS>It has been fixed in the latest release.
Can we please use the latest version of react-native-siren in the next release?

Fixed in 1.1.3


Error after running jasper.py

hi,
when i run jsper, its not running. below are logs:
pi@raspberrypi:~ $ /home/pi/jasper/jasper.py

        JASPER - THE TALKING COMPUTER           *

(c) 2015 Shubhro Saha, Charlie Marsh & Jan Holthuis *

WARNING:root:tts_engine not specified in profile, defaulting to 'espeak-tts'
ALSA lib pcm_dmix.c:1052:(snd_pcm_dmix_open) unable to open slave
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bluealsa.device'
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:4996:(snd_config_expand) Args evaluate error: No such file or directory
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM bluealsa
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bluealsa.device'
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:4996:(snd_config_expand) Args evaluate error: No such file or directory
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM bluealsa
ALSA lib pcm_dmix.c:1052:(snd_pcm_dmix_open) unable to open slave
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
Expression 'paInvalidSampleRate' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2048
Expression 'PaAlsaStreamComponent_InitialConfigure( &self->capture, inParams, self->primeBuffers, hwParamsCapture, &realSr )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2719
Expression 'PaAlsaStream_Configure( stream, inputParameters, outputParameters, sampleRate, framesPerBuffer, &inputLatency, &outputLatency, &hostBufferSizeMode )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2843
Traceback (most recent call last):
File "/home/pi/jasper/jasper.py", line 151, in
app.run()
File "/home/pi/jasper/jasper.py", line 121, in run
conversation.handleForever()
File "/home/pi/jasper/client/conversation.py", line 31, in handleForever
threshold, transcribed = self.mic.passiveListen(self.persona)
File "/home/pi/jasper/client/mic.py", line 110, in passiveListen
frames_per_buffer=CHUNK)
File "/usr/lib/python2.7/dist-packages/pyaudio.py", line 750, in open
stream = Stream(self, *args, **kwargs)
File "/usr/lib/python2.7/dist-packages/pyaudio.py", line 441, in init
self._stream = pa.open(**arguments)
IOError: [Errno -9997] Invalid sample rate
below is my .asoudrc file (whenever i make changes to it, after reboot it goes back to same as below)
pcm.!default {
	type hw
	card 1
}

ctl.!default {
	type hw
	card 1
}

below my sound devices:
pi@raspberrypi:~ $ arecord -l
**** List of CAPTURE Hardware Devices ****
card 0: Device [USB PnP Sound Device], device 0: USB Audio [USB Audio]
Subdevices: 1/1
Subdevice #0: subdevice #0
pi@raspberrypi:~ $ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 1: ALSA [bcm2835 ALSA], device 0: bcm2835 ALSA [bcm2835 ALSA]
Subdevices: 8/8
Subdevice #0: subdevice #0
Subdevice #1: subdevice #1
Subdevice #2: subdevice #2
Subdevice #3: subdevice #3
Subdevice #4: subdevice #4
Subdevice #5: subdevice #5
Subdevice #6: subdevice #6
Subdevice #7: subdevice #7
card 1: ALSA [bcm2835 ALSA], device 1: bcm2835 ALSA [bcm2835 IEC958/HDMI]
Subdevices: 1/1
Subdevice #0: subdevice #0
below my /etc/asound.conf:
pcm.!default {
type asym
capture.pcm "mic"
playback.pcm "speaker"
}
pcm.mic {
type plug
slave {
pcm "hw:0,0"
}
}
pcm.speaker {
type plug
slave {
pcm "hw:1,0"
}
}

Did you find a solution?
I've the same issue. I had set it up 8-9 months ago. Used to work. Restarted the pi today after a long time and face this issue when trying to use Jasper.

Nops. I heard the project stopped and no updates coming now
Get Outlook for Androidhttps://aka.ms/ghei36

From: Nikhil Gupta<EMAIL_ADDRESS>Sent: Thursday, April 16, 2020 10:10:26 AM
To: jasperproject/jasper-client<EMAIL_ADDRESS>Cc: zeesh1979<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jasperproject/jasper-client] Error after running jasper.py (#722)
Did you find a solution?
I've the same issue. I had set it up 8-9 months ago. Used to work. Restarted the pi today after a long time and face this issue when trying to use Jasper.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jasperproject/jasper-client/issues/722#issuecomment-614457698, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE5TO4GED4NMZ56VEMPVMDTRM2VOFANCNFSM4K36VCTA.

The work is being continued at https://github.com/aplawson/jasper-client, specifically an update to Python3 and a tutorial on how to build it or deploy it with a custom Raspbian ISO image.  //adam

Thanks. i have put the image from sourceforge to my RPi3b but its not booting (red led is continuous on). what could be wrong? my sd card card and rpi both are fine coz i used a working set for this

From: aplawson<EMAIL_ADDRESS>Sent: Sunday, April 26, 2020 10:12 PM
To: jasperproject/jasper-client<EMAIL_ADDRESS>Cc: zeesh1979<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jasperproject/jasper-client] Error after running jasper.py (#722)
The work is being continued at https://github.com/aplawson/jasper-client, specifically an update to Python3 and a tutorial on how to build it or deploy it with a custom Raspbian ISO image. //adam
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jasperproject/jasper-client/issues/722#issuecomment-619608261, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE5TO4BD3NYPAUWNOFEBNWTROSBRJANCNFSM4K36VCTA.

I would attach a monitor and monitor the output while the rPI is booting. That will disclose many issues (and when they occur) during the boot-up process. //adam

I would attach a monitor and monitor the output while the rPI is booting.
That will disclose many issues (and when they occur) during the boot-up
process. //adam
Adam Peacock
Principal Architect
Office: +1-916-794-5706
On Tue, Apr 28, 2020 at 5:37 AM zeesh1979<EMAIL_ADDRESS>wrote:

Thanks. i have put the image from sourceforge to my RPi3b but its not
booting (red led is continuous on). what could be wrong? my sd card card
and rpi both are fine coz i used a working set for this

From: aplawson<EMAIL_ADDRESS>Sent: Sunday, April 26, 2020 10:12 PM
To: jasperproject/jasper-client<EMAIL_ADDRESS>Cc: zeesh1979<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jasperproject/jasper-client] Error after running jasper.py
(#722)
The work is being continued at https://github.com/aplawson/jasper-client,
specifically an update to Python3 and a tutorial on how to build it or
deploy it with a custom Raspbian ISO image. //adam
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/jasperproject/jasper-client/issues/722#issuecomment-619608261>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AE5TO4BD3NYPAUWNOFEBNWTROSBRJANCNFSM4K36VCTA

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jasperproject/jasper-client/issues/722#issuecomment-620578703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJ34K7DAFH5DPUMAAIHGTLRO3EXRANCNFSM4K36VCTA
.

Also there are two projects with an rPI image. https://sourceforge.net/projects/jasperproject has a stale/broken image. https://sourceforge.net/projects/jasperclient has an image that was tested in February or March of this year so much more current.

Oh i had downloaded from 1st link. I will check other one. Thanks
Get Outlook for Androidhttps://aka.ms/ghei36

From: aplawson<EMAIL_ADDRESS>Sent: Tuesday, April 28, 2020 6:23:59 PM
To: jasperproject/jasper-client<EMAIL_ADDRESS>Cc: zeesh1979<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jasperproject/jasper-client] Error after running jasper.py (#722)
Also there are two projects with an rPI image. https://sourceforge.net/projects/jasperproject has a stale/broken image. https://sourceforge.net/projects/jasperclient has an image that was tested in February or March of this year so much more current.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jasperproject/jasper-client/issues/722#issuecomment-620675394, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE5TO4B5G4UO2ISHTUWSOIDRO3YI7ANCNFSM4K36VCTA.

Hi,
I have flashed it to my SD and its up now. but how to run jasper now? i cannt seem to find any python or shell file to run

From: aplawson<EMAIL_ADDRESS>Sent: Tuesday, April 28, 2020 6:23 PM
To: jasperproject/jasper-client<EMAIL_ADDRESS>Cc: zeesh1979<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jasperproject/jasper-client] Error after running jasper.py (#722)
Also there are two projects with an rPI image. https://sourceforge.net/projects/jasperproject has a stale/broken image. https://sourceforge.net/projects/jasperclient has an image that was tested in February or March of this year so much more current.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/jasperproject/jasper-client/issues/722#issuecomment-620675394, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE5TO4B5G4UO2ISHTUWSOIDRO3YI7ANCNFSM4K36VCTA.


Added doxygen documentation to collide() function

Addresses https://github.com/WSU-CEG-6110-4410/ADAD-Craft/issues/109


Unable to modify namespaced functions

I'm not able to redefine a function that was declared inside of a namespace.  I see that namespaces.phpt has tests for modifying methods inside of namespaced classes.  But since there are no tests for modifying a namespaced function I assume that this has not been implemented yet.
Are there any plans to add/fix this functionality?
Original Testme Implementation
PHP Parse error:  syntax error, unexpected '\' (T_NS_SEPARATOR), expecting '(' in /testme.php(9) : runkit created function on line 1
PHP Stack trace:
PHP   1. {main}() /testme.php:0
PHP   2. runkit_function_redefine() /testme.php:9

<?php

namespace Foo;

function testme() {
  echo "Original Testme Implementation\n";
}
testme();
runkit_function_redefine('Foo\testme','','echo "New Testme Implementation\n";');
testme();

You're using an outdated version of runkit. He fixed this in 2012.
https://github.com/zenovich/runkit/blob/master/tests/namespaces.phpt
Instructions for installing the latest release: https://github.com/zenovich/runkit#building-and-installing-latest-release-with-pecl-in-unix

Exactly :+1:

The provided example code worked as expected with the most recent runkit.
However, I noticed a bug/inconsistency when writing a test case for modifying namespaced functions (Currently, there are only tests of namespaced methods):
Runkit's function manipulation works properly for functions with no leading "", but doesn't work when there is a leading ""
Because method manipulation works properly for namespaced classes with leading ""es, I think that runkit should handle namespaced functions the same way.
https://github.com/zenovich/runkit/pull/91

Ooops! I installed runkit using homebrew.  The homebrew version is over a year out of date.  I'll take my problem there instead.  Thanks


Adapt UI to smaller resolutions

Windows 7 x64, Chromium-based browser, 1024x768.
Those scrollbars, aggrrhh!

Ah sorry, this was never tested. Will try to fix this soon.

Just wanted to add, on a 1920x1080 res while making a long note, there is a scrollbar on the right but not the bottom.

However, once the paste is saved, depending on the note, there will be scrollbars on the side and the bottom.

For example, I just wanted to expand on it a bit :)

I have fixed some of the issues, atleast with the horizontal scroll. For the vertical scroll, I have fixed for some specific browsers. Seems the most optimal solution is to somehow get rid of the ugly scrollbars and try to adopt the new ones that disappear after scrolling.


clone method returns Error: this.copy is not a constructor

Perhaps because new was used twice.
https://github.com/maissaninc/kidjs/blob/6394760d597032d1b91eafa278914fb7acb93581/src/shape/index.js#L119
https://github.com/maissaninc/kidjs/blob/6394760d597032d1b91eafa278914fb7acb93581/src/shape/index.js#L143
(Thank you Cemal for reporting!)

This has been fixed, however it's worth noting a few oddities about the clone method.

By default, the clone is placed immediately to the right on the original, unless new x and y coordinates are passed as parameters.
Unless a color property has been explicitly set, the clone with have a different color. This is because of when theme colors are applied. See issue: https://github.com/maissaninc/kidjs/issues/19


Add return-to-admin ability (turn off espionage mode)

Also:

Refactor various related pieces (forms, endpoints)
Add a keyboard shortcut for admins to navigate to the admin page

Done reviewing code.  Did not test it, so let me know if you think that's necessary.
Ship it!


Updates Siderus Orion details

This will update the details about Siderus Orion

Can somebody re-try the CI part? There was a connection error from Jenkins!

@koalalorenzo I did restart Jenkins. I think you can too: you just need to Login with your GitHub account. If you have write permissions here, you'll be able to start/restart and stop Jenkins

:O The build is failing, but I believe it's just something on Jenkins. I'd merge this.


what Unity version should I use?

Hi,
I tried opening this with the latest beta (2017.1.0b7) but the project version says 2017.2.0b1... I'm on the alpha list but the latest version there is 2017.2.0a4.
So I tried checking out the tag unity-2017.1b5, but after reimport I see most of the shaders have been auto-upgraded (replaced 'mul(UNITY_MATRIX_MVP,)' with 'UnityObjectToClipPos()').
Does it matter at all? Which version should I use?

Hey,
We regularly update the Unity C++ code for this project, mean latest github version is high likely to not work on latest public available Unity version (alpha, beta, priority list). It is highly develop and highly experimental feature.
The only release that currently work is with Unity 2017.1b5 build:
https://github.com/Unity-Technologies/ScriptableRenderLoop/releases/tag/unity-2017.1b5

but after reimport I see most of the shaders have been auto-upgraded (replaced >'mul(UNITY_MATRIX_MVP,)' with 'UnityObjectToClipPos()').
There is no use of  UNITY_MATRIX_MVP in this github project, so it shouldn't matter.

Hope it help.

OK so basically latest is for UT internal use only right?
I started looking at the 2017.1 tag which mostly works, except some of the (older?) test scenes like Big (viking village) and FPTL are broken: had to rebuild outdated lighting data, but even after that it's all black.
I'm quite interested in FPTL, any idea what needs to be fixed?

OK got it, had to assign the FPTL asset in the graphics settings.
I suppose this can be done procedurally... this demo project would be a bit friendlier if the proper pipeline was selected when opening scenes ;).

Yes, there is no automatic detection of which pipeline asset to use as it is not really possible to know (you can have several fork of the same render pipe with very slight variation.
Note: FTPL is a toy project that we use to experiment/implement tile/cluster lighting, there is currently no plan to ship it and we will remove it in the future. But feel free to use it for your own need :)

Thanks. I don't plan to use FPTL, I'm merely dissecting it to figure out how it's implemented... so far, it's a toughie ;) !

You can find the details in GPU Pro 7 article from Morten Mikkelsen, it is an improve version of the article (done by the author himself as he work at Unity).


Drop patch version from twiddle-wakka versioning syntax

Using "twiddle-wakka" syntax with a version number including the patch level essentially locks the dependancy to that version (see http://guides.rubygems.org/patterns/#pessimistic-version-constraint)
This changes the gemspec to the minor version, which should allow it to float to any minor version before the next major version.

@dotsam Looks good. Thanks! I'll merge it right away.

@dotsam New version is released on Github and on rubygems.

@dotsam By the way. Would you be interested in making another PR to update all dependencies and test it? See https://www.versioneye.com/user/projects/561c9bfda193340f2f001728?child=summary#tab-dependencies for current outdates.


Use predictive analytics to activate a pod

A very “blue sky” feature but would be amazing to have KEDA look at historic data and patterns for deployments to try to predict when events may be coming in and scale proactively

Ok let's run this in kubeflow😁
Le mer. 15 mai 2019 à 16:24, Jeff Hollan<EMAIL_ADDRESS>a
écrit :

A very “blue sky” feature but would be amazing to have KEDA look at
historic data and patterns for deployments to try to predict when events
may be coming in and scale proactively
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kedacore/keda/issues/197?email_source=notifications&email_token=AB3W6GMUBS3WEBK44PHG7K3PVQMLBA5CNFSM4HND3DV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GT6CFWQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3W6GN2BHTCZZAVDY565S3PVQMLBANCNFSM4HND3DVQ
.

A very “blue sky” feature but would be amazing to have KEDA look at historic data and patterns for deployments to try to predict when events may be coming in and scale proactively

Especially for RabbitMQ scaler, that would be great. If we can use some additional variables too such as "consumer utilization", "consumer ack" and "delivery" with the "queueLength" variable, maybe we can scale pods in a reactive way?
 And KEDA currently scaling our pods based on the "queueLength" variable. It's great. But our some RabbitMQ consumers doing I/O based operations. If KEDA keeps continuing the scaling as a linear based on the "queueLength" variable, our other I/O services, that in the consumer, are starting to get in the bottleneck.

That's a really interesting idea. A few months ago I was doing research about best practices/patterns for autoscaling application and I stumbled on a research paper covering doing autoscaling using a predictive model.

KEDA look at historic data

@jeffhollan what historic data you had in mind? Also, should it be something maintained by KEDA itself or should this be a pluggable solution so anyone can use a custom model / algo?

@jeffhollan
We were dreaming about the same thing and developed this kind of solution. It works based on our simple, but working AI model and predicts pretty well.
Maybe it's not the "blue sky" you're looking for, but you should definitely take a look.
PR: https://github.com/kedacore/keda/pull/2418

I'd love to see a generic interface that will be siting between metrics and HPA and where we can "manipulate" the metrics the way we would like to. For example adding more logic in evaluation of metrics from multiple triggers or pluging in some AI/ML model. The only option to manipulate metrics that we have today is the fallback feature, which is great. Though some generic interface would be much better.
Writing a new pod autoscaler is something I'd like to avoid 😅


Custom component header click handler not getting event

I wonder - what exactly is going on here:
https://github.com/ratiw/vuetable-2/blob/next/src/components/VuetableRowHeader.vue#L14
Compare to this
I noticed this the other day and wondered when it might bite me in the butt, and today it did when I tried to enable multi-sort. The multi-sort function attempts to acquire the multi-sort key from the event, but is not something ever passed to the event handler in the case of a custom component.
The fix I believe would be rather simple:
@click="onColumnHeaderClicked" - let vue unpack any arguments for you
Then in your custom component you just gotta @click="$emit('click', rowField, $event)"

@martixy The L14 is definitely incorrect. Sorry about that.
Thanks for the tip, it is useful. But I would prefer this to be explicit as it is easier to catch the bugs I've made. :)


Fix for selection of tab

What got fixed:
This fix determines pre selected tabs and renders the UI with that tab and respective panel selected.
If there is no pre-selected tab, then follows the previous behavior of selecting the first tab by default.

Okay, I've fixed just a couple things ... I think we're good to go!


Not accessible with the screen readers

I am a blind person. So I daily use screen readers. This is a really nice library but unfortunately it is not accessible with the screen readers. It means that any website which uses this library will be not accessible for people with disability. It is really not difficult to fix this issue. We just need to add aria-label to squares. I am ready to help.

Thank you for raising this! If you're able to raise a pull request for this then please do. Otherwise I will try to carve out some time when I am not busy but that will not be for a while


adding default_search_alias, versioned ama indices, and "group_alias" field for creating aggregated aliases.

As discussed in person

remove the "aliases" section and move all its entries into "group_aliases"
rename "group_aliases" to aliases
update the index runner to be able to load from this


[QUERY] Partial updates in azure-spring-data-cosmos

Is partial updates available in azure-spring-data-cosmos latest version ? If now any timeline on prospect release ? We are using 3.17.0 version

@backwind1233 could you please follow up with @Abhisar on this github issue?

Hi @kushagraThapar could you please help on this issue?

Hi @kushagraThapar any updates on this? partial document update seems to be not supported currently. Any timeliness?

@yiliuTo this task is still in backlog https://github.com/Azure/azure-sdk-for-java/issues/21433 please work with @kushagraThapar to expedite this.

@sachinadiga  - Patch API is not supported on spring-data-cosmos SDK right now. This is in our backlog, but we don't have any ETA for this as of now. Will update this issue once we start working on it.

Closing out this duplicate issue. Please track https://github.com/Azure/azure-sdk-for-java/issues/21433 issue for any updates.


add ssl do circle ci docker

Some tests are still failing but the reason is in the test code, thus closing this one


Add Pytorch_to_MDF example

Adding a basic example like the ABCD_pytorch to the examples/pytorch folder.

must convert to MDF format
add graphical representation to the readme

@pgleeson do add any other pointers if I missed any.

I had added a few examples which convert PyTorch to MDF https://github.com/ModECI/MDF/tree/mdf_to_onnx/examples/PyTorch/PyTorch_MDF

@pgleeson as I can see all these examples by @mraunak had a example_outputs https://github.com/ModECI/MDF/blob/mdf_to_onnx/examples/PyTorch/PyTorch_MDF/convolution.py#L65 parameter to the pytorch_to_mdf function, were these supported in the older versions of this function, as I am having trouble running them. Also there seems to be no assertion for verifying the two outputs are they needed or are we just trying to convert to mdf and export to json formats

Test cases are available here:
https://github.com/ModECI/MDF/blob/mdf_to_onnx/tests/interfaces/pytorch/test_import.py

@mraunak I am having difficulty running, some internal changes in execution_engine.py and mdf.py are the probable ones, I tried setting an env according to your branch for the dependencies and src from the current dev branch so most probably the src folder changes are the ones creating problem.
(While the examples runs fine in your branch.)
line causing the error:
eg.evaluate(initializer=params_dict) https://github.com/ModECI/MDF/blob/a4ed0f3b03e8bc52e521a9d310861c5897bcd3c7/tests/interfaces/pytorch/test_import.py#L115
also current version doesn't support https://github.com/ModECI/MDF/blob/a4ed0f3b03e8bc52e521a9d310861c5897bcd3c7/tests/interfaces/pytorch/test_import.py#L105
so I tried removing this.
however, I think I will need to find the correct versions of the mdf code(from src) using the history. What do you think?

Try running pytorch/test_import.py on the mdf_to_onnx branch on your machine without making any changes if it works then start incorporating the latest changes and debug if it gives an error. I had made some changes in the execution engine and pytorch/importer.py

@mraunak got it thanks!!!

Done at https://github.com/ModECI/MDF/blob/main/examples/PyTorch/README.md


refactor: add VSCodeEditorContext class

refactor: add VSCodeEditorContext class
Simple clean up to:

Remov vscode-context.ts which is being used by the custom-prompt recipe
Replace vscode-context.ts with VSCodeEditorContext class

All functions and features remain unchanged.
Test plan

[ ] Run /explain command
[ ] Run /smell command
[ ] Run /test command
[ ] Ask a question

Everything should work as it was before.

tested on your branch, looks good!
nit: this is somewhat messy behaviour, if you choose another command from the side panel it opens a whole new chat window

tested on your branch, looks good!
nit: this is somewhat messy behaviour, if you choose another command from the side panel it opens a whole new chat window

That's the default behavior and not due to any changes in this PR.
This is because the context and answers from your previous chat would sometimes pollute the next question, so right now we are starting all commands in a new chat

@kalanchan thanks for reviewing and testing the PR ❤

tested on your branch, looks good!
nit: this is somewhat messy behaviour, if you choose another command from the side panel it opens a whole new chat window

That's the default behavior and not due to any changes in this PR.
This is because the context and answers from your previous chat would sometimes pollute the next question, so right now we are starting all commands in a new chat

hmm, i see. is there a thread with that decision? It seems a bit messy but let's go with it if thats what everyone decided on

tested on your branch, looks good!
nit: this is somewhat messy behaviour, if you choose another command from the side panel it opens a whole new chat window

That's the default behavior and not due to any changes in this PR.
This is because the context and answers from your previous chat would sometimes pollute the next question, so right now we are starting all commands in a new chat

hmm, i see. is there a thread with that decision? It seems a bit messy but let's go with it if thats what everyone decided on

@kalanchan Yea you should be able to reproduce the same behavior in the current insider build. It's been the default behavior for the new chat panel but agree we should discuss it in slack :)


IR fix and example

This PR introduces the following fix to IR:
When using the advanced apply for the inner solver, the initial guess will be x. This way, the initial residual of the inner solver is r - Ax which will generally not be small but close to Ax as we try to get r to be zero. This causes very poor to no convergence when e.g. using CG with tolerance 1e-2 as inner solver. By kind of manually doing the advanced apply, we can use 'r' as initial guess, which gives much better results.
This PR also introduces a small example on iterative refinement.

@fritzgoebel I think this is ready to merge...

@fritzgoebel Maybe before merging, can you do an interactive rebase onto develop and clean up the history a bit? Especially the "Update examples/*" files
git fetch   # copy changes from remote
git rebase -i origin/develop


Database installation instructions

The previous instructions worked for me. I suspect the problem was a quota issue on DevZero.
I still updated the instructions a bit to make the disk usage a bit smaller.

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I had to make a few changes to the paths for taxdump and uniprot in order to get the downloads to work, but otherwise all looks good to me


Popover with AutoLayout?

Hello. I'm using this awesome library to show quick pop ups in my app where the user takes a small action. However, on rotation, the popup does not stay in place, so I would like to use AutoLayout to keep it in place.
Whenever I try though, the pop either disappears or one of its corners gets entered in the screen.
Is there a way to fully manage the popup via auto layout? Thank you very much!

Is their any progress to supporting device rotation in some way?

I don't know if the dev is looking into the problem. For the moment, I'm dismissing the popover in viewWillTransitionToSize: and showing it again when the rotation finishes like this:
`override func viewWillTransition(to size: CGSize, with coordinator: UIViewControllerTransitionCoordinator) {
super.viewWillTransition(to: size, with: coordinator)
	// Will execute before rotation
	var shouldShowNote = false
	if notePopover != nil {
		shouldShowNote = true
		notePopover?.dismiss()
	}

	coordinator.animate(alongsideTransition: {(_ context: Any) -> Void in
		// Will execute during rotation
	}, completion: {(_ context: Any) -> Void in
		// Will execute after rotation
		if shouldShowNote {
			self.noteTapped(nil)
		}
	})
}`

where noteTapped() presents my popover. Keep your popover into a local property. It looks semi-ok, with the animations, but it would be great if the library handled rotation properly...