Tests fail on i386 because time_t overflows

When the tests are run on i386 they fail. Here's where it fails.
        activities = activities_from_pubnub_message(
            lock,
>           datetime.datetime.fromtimestamp(16844292526891571 / 1000000),
            {
                "status": "unlatched",
                "callingUserID": "manualunlatch",
                "doorState": "open",
            },
        )
E       OverflowError: timestamp out of range for platform time_t

tests/test_pubnub_activity.py:320: OverflowError

Full build log at https://ci.debian.net/packages/p/python-yalexs/testing/i386/50435663/

fixed via #156
releasing now


Window Specials

Like Line Specials in Doom
Basically interactivity for windows
Basic effects include:

Progressbar
%tage
Window name
Basic text

Advanced ones include:

Inventory slots
Selectors (Vertical and Horizontal)

Fancier ones include:

Gradient progressbar
Textboxes
Scrolling text (Overflow and cosmetic scrolling)

Why was this the first issue i opened
5 months have passed and I haven't even made the Window system yet


OpenAI API Key Exposed in HTML on /dashboard/ai-assistant

Hello,
I have noticed a critical security issue in the Horizon UI Chakra project. On the /dashboard/ai-assistant page, the OpenAI API key is exposed in the HTML as the value for the key apiKeyApp. This exposure can lead to unauthorized use of the API key and potential security breaches.
Steps to reproduce:

Navigate to the /dashboard/ai-assistant page.
Inspect the HTML source code.
Find the apiKeyApp key, which contains the OpenAI API key as its value.

This issue needs immediate attention to secure the API key and protect sensitive information.
Thank you.

Hello, @ImmanEb ! It's fixed, I will ping you when it is up, and I will also close the issue then.
Vlad.

To problem has been fixed on every version of our product that presented the issue. @ImmanEb


Update playground travelocity passivests web apps to IS550

Purpose
Update web apps

Updated to suit with IS 55x dependencies


Make the error logged in the event.go file be more informative

Currently stuff like error dispatching internal event, member not found doesn't really give us enough info for debugging the culprit of the error.

I think this error should just be reduced in severity. It's harmless.
On Feb 23, 2017 12:34 PM, "alex"<EMAIL_ADDRESS>wrote:

Currently stuff like error dispatching internal event, member not found
doesn't really give us enough info for debugging the culprit of the error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bwmarrin/discordgo/issues/326, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdVggGzAVeGSGoVdOp8_a_roCkZFwRsks5rfe1igaJpZM4MKd3z
.

Yeah that's true, maybe we should log it as a warning instead?

Sounds good to me
On Feb 23, 2017 12:37 PM, "alex"<EMAIL_ADDRESS>wrote:

Yeah that's true, maybe we should log it as a warning instead?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bwmarrin/discordgo/issues/326#issuecomment-282113595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdVgjJtlUPRnmSwIYSZeGabTUBiTb_iks5rfe4agaJpZM4MKd3z
.

This issue can be closed. Log level was changed in https://github.com/bwmarrin/discordgo/commit/c34affbc570dee6050ae15e42b08d9c061a0a3aa

Thanks @tsdtsdtsd  :)


Variables: Add Date Type

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
The addition of a Date type variable.
Describe alternatives you've considered
I have considered using ISO formatted strings, however future calculations on this would be more troublesome.
Additional context
I expect there will also be the need to be able to specify the output format of the date variable when used in presentation strings as a future requirement.  For example "dd/mm", "dd/mm/yyyy", or "mm/dd/yyyy".

this feature is available using Cottle and the custom getdatetime function.
Example:
{getdatetime("yyyyMMdd")}
Note: MM is for month, mm is for minutes.
For formatting options instead of "yyyyMMdd", here are some links from the dotnet docs:

Standard format strings
Custom format strings


About the URI given from the tb.start

Is there anyway to get an external url that allow me to monitor the experiment on the fly? In an Azure ML Experiment:
run = Run.get_context()
tb = Tensorboard([run])
tb.start()
run.flush()

Output:
http://localhost:6006/
I still can't not monitor the training using this uri. Is there anyway to get a uri to visit the tb from my pc?

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

ID: abae00a4-5326-f074-4270-619f151d3fbe
Version Independent ID: e6b2893d-f10e-c069-9eb6-4c5dd4b0f2bc
Content: Visualize experiments with TensorBoard - Azure Machine Learning
Content Source: articles/machine-learning/service/how-to-monitor-tensorboard.md
Service: machine-learning
Sub-service: core
GitHub Login: @maxluk
Microsoft Alias: maxluk

@Jiaming-Liu Thanks for the feedback! We are currently investigating and will update you shortly.

#assign:@maxluk

For need to launch Tensorboard from outside of the run to monitor it. For example you can submit experiment from Notebook VM notebook and launch tensorboard from there. It will give you URL hosted on that notebook vm which will monitor the run as it progresses. See example here: https://github.com/Azure/MachineLearningNotebooks/blob/4a3f8e7025334ea8c0de0bada69b031ce54c24a0/how-to-use-azureml/track-and-monitor-experiments/tensorboard/tensorboard.ipynb

Thanks for your report. We hope this answers your questions, we shall proceed to close this issue, #please-close.
If you feel that your issue has not been resolved, please reopen the documentation issue and provide additional context. For product issues, contact technical support.


Handle both SwiftArgParser 0.3.x and 0.4.x output.

Cherry-picking this change in release/5.4 to unblock Apple Silicon bot.

@swift-ci test

@shahmishal is this ready to be merged? also note we would need a corresponding PR for release/5.5 branch if this needs to make it into 5.5

@tomerd I thought this was merged before we branched for release/5.5

@tomerd I thought this was merged before we branched for release/5.5
oh sorry, you are correct, I misread the target branch


How to run in SPV mode?

I see there's so support for bitcoinj, but there are no instructions. I'm confused.

This document is build and run for SPV mode.
https://github.com/nayutaco/ptarmigan/blob/master/docs/build_for_spv.md#build
Ubuntu 18.04 setup

make clean
edit options.mak

set NODE_TYPE=BITCOIND
    ↓↓↓
set NODE_TYPE=BITCOINJ

make
set LD_LIBRARY_PATH for JVM

cd install
source jdk.sh

# output libjvm.so if success

create node folder and run

same as bitcoind version
or you can use node folder created for bitcoind version

After run ptarmd

get address from bitcoinj for fund

../ptarmcli --getnewaddress

send bitcoin to address

wait balance available

../ptarmcli --getbalance

You can use this balance for channel funding.


Sandbox support: we need to support easy-checks for users

We need to create a new VIEW, where users will be able to check their configs
It should be like diktat demo
Has 2 input windows (config, test) and 1 output window with debug info

We also were talking about improved support in save-cli:

Better (clear and readable) step-by-step logging: which test files have matched, why, which options have been used to extract warnings
This information could be additionally written into a separate structured file (JSON?) that could be easily parsed for displaying in save-cloud-frontend

Another idea - we can base the new view on the existing one with public tests under contests. Then before submitting execution for a contest the user will be able to test their tool on a public test.

The idea is changed a little bit:

[ ] https://github.com/saveourtool/save-cloud/issues/1228


Separate branch for localizations

Currently it's almost impossible to find commits with changes to the program, due to the huge amount of localization entries. Would it be possible to create a separate branch for localizations (which can then only be merged occasionally)?

There is separate branch for localization entries - l10n. I do not have a direct choice how pull requests are handled, as that part relies on Crowdin without many options, basically GitHub integration or not.
And GitHub integration is very important for me, as it allows me to develop program, add new strings and ensure they all land in strings to translate, without my helpful hand or manually updating them every once in a while. I didn't go crazy from all of that yet thanks to that fact and how it's all done - I develop ASF as usual, and I only merge ongoing pull request when I'm about to make a release or just feel like it. It's perfect.
So why I can't really do much with l10n branch and PR itself, as that would mean abandoning GitHub integration and making more hassle for me, there is still something I can do - picking merge & squash option instead of merging only in PR, which will result in a history like this.
I must check how exactly squashing affects integration, but if it doesn't cause any problems, then it solves your issue in 100%. I'll close this issue once I'm sure this can work.

Squashing won't do, it breaks entire history later on, and causes me more trouble, which is what I want to avoid in the first place.
Hey!

I'm working with you translation platform and it's great, I'm enjoying it very much and the fact that you support open-source projects really makes me happy!

I'm currently using GitHub integration for syncing my project, and it works really great for syncing strings, but I have some issues with automatically generated pull requests. In general every single crowdin action results in a commit - if I keep this PR open for several days, it's entirely cluttered with commits, and merging it later to the target branch clutters it as well. I could squash that PR, but then l10n branch that is being used for Crowdin commits is out of the sync with main branch (has conflicts), and dealing with entire situation is even worse.

Could you please consider adding a setting that would allow to "bundle" Crowdin changes together, in a single commit, in given time interval? For example, I could pick "1 day" option, that would limit Crowdin into 1 commit per day, bundling all changes done in the meantime automatically. It would delay some changes, but thanks to that keeping history clean, as merging such PR later on won't result in merging 300+ commits made across 2-3 days.

Alternatively, if above is too troublesome, maybe you could add an option of disabling creating/maintaining localization branch + sending PRs, so I could still use GitHub integration for synchronization of strings, but update outcome localizations myself in a single commit. Currently it's either "all" or "nothing", and I don't want to drop it entirely only because Crowdin is too "noisy" when it comes to sending PRs :). If it's possible for you to implement bundling suggestion, that would be great, but I'll be also very happy if I can simply disable localization branch and maintain localizations myself, while still using strings syncing provided by you for translation on the platform.

Sincerely,
~Łukasz

But I sent an e-mail, so we'll see if it will be possible to save me hassle in the future.

Hello Łukasz,
I will forward your suggestion to our devs team and ask whether it is possible at least to add an option of disabling creating/maintaining localization branch + sending PRs as you mentioned.
Hopefully, I will update you today on the matter, thank you!
Have a great day :)

Hello Łukasz,
I just want to update you regarding our conversation about branch localization.
I forwarded your suggestion to our devs and they really consider to be a great idea although it is not planned to be implemented in the near future since it's a kind of time-consuming procedure. It is included in our list of future improvements, but unfortunately, I can't guarantee it is implemented in the short run.
Thank you again for a suggestion, it does make sense, so hopefully we can put it into life someday!
Have a lovely weekend!

This link can be used to filter out translations. It's not as clear as the normal view but it's better than with 10000 translation messages in between.

Let's hope Crowdin adds that feature in the future, I think that would make everyone (regardless of which project) very happy.

I'm not against this suggestion and I also hope that they add it sooner than later, but you should also understand my reasoning - I want to minimize maintenance work I have in ASF, not add myself extra one. There is little difference between merging translations from other project and maintaining strings fully manually. The way it's right now makes it much easier for me to manage them without much hassle, which doesn't automatically mean that I can't see a problem, as I'd appreciate less clutter in commits history as well. Therefore, we're waiting until something better happens.

Eventually I got annoyed enough myself to drop GitHub integration until it'll have an option to spam less. I'll update translations periodically myself until then.

I even managed to get best of both worlds 😀


Added whitespaces after ###

Problem
There are 15 headings which are not parsed correctly by GitHub site, because in each, the last "#" is followed immediately by text.
For example:
###Lexical Structure

Solution
I added whitespaces so that these headings are parsed as expected.
For example:
### Lexical Structure

thanks


CLI is not working with project referencing Autofac

Repro steps:

Add swashbuckle nuget packages to the project:

SwashBuckle.AspNetCore 4.0.1
SwashBuckle.AspNetCore.Annotations 4.0.1

Configure swashbuckle:

In ConfigureServices:

services.AddSwaggerGen(options =>
{
    var projectName = GetType().Assembly.GetName().Name;
    var path = Path.Combine(AppContext.BaseDirectory, $"{projectName}.xml");

    options.SwaggerDoc("v1", new Info
    {
        Title = $"{projectName} API Reference",
        Version = "v1"
    });

    options.IncludeXmlComments(path, true);
    options.EnableAnnotations();
});

In Configure:

app.UseSwagger();
app.UseSwaggerUI(options => options.SwaggerEndpoint("/swagger/v1/swagger.json", "v1"));

Swashbuckle & Swagger UI now works fine

Add DotNetCliToolReference to .csproj file:

  <ItemGroup>
    <DotNetCliToolReference Include="Swashbuckle.AspNetCore.Cli" Version="4.0.1" />
  </ItemGroup>

Run

dotnet swagger tofile --output swaggerexport.json path\to\project.dll v1

Got an error

Unhandled Exception: System.InvalidOperationException: No service for type 'Microsoft.Extensions.DependencyInjection.IServiceProviderFactory`1[Autofac.ContainerBuilder]' has been registered.
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)
   at Microsoft.AspNetCore.Hosting.Internal.StartupLoader.ConfigureServicesDelegateBuilder`1.<>c__DisplayClass14_0.<ConfigureServices>g__ConfigureServicesWithContainerConfiguration|0(IServiceCollection services)
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.ConfigureServices(IServiceCollection services)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Initialize()
   at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
   at Swashbuckle.AspNetCore.Cli.Program.<>c.<Main>b__0_3(IDictionary`2 namedArgs)
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args)
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args)
   at Swashbuckle.AspNetCore.Cli.Program.Main(String[] args)

You'll need something like this #887 to create your WebHost.

I have the exact same issue. Were you able to resolve this?

Is there a work around for this isse?

@patriksvensson It's not pretty but we had success cloning the Swashbuckle.AspNetCore.Cli project and applying the PR #887 mentioned above.
In the end, we actually simplified it further by pulling in the specific project we're creating swagger typings for and replacing this block with class specific types. That does make for a pretty gnarly set of dependencies, plus the added hassle of keeping in step with patches, but it is a work around of sorts if you're desperate!

@andrewbridge Thanks for the reply! Will probably just fork Swashbuckle until this PR is getting merged (or fixed another way).

It's been a while, so I'll need a bit of time to regain the context. I'm working on another area right now, but reviewing the PR, resolving conflicts etc. is next on the list. So, stay tuned ...

I just merged #887 to fix this issue. However, I've sinced discovered a subsequent problem with the approach. The PR introduces the new interface ISwaggerWebHostFactory, and if your application includes an implementation of that interface, then the CLI tool will use it to create an IWebHost to run the CLI through.
However, the Swashbuckle.AspNet.Cli package is a .NET Global Tool, and therefore can't be added to your project as a regular dependency. If you can't add the dependency, then you won't have access to the ISwaggerWebHostFactory type, and will therefore be blocked from actually using the new capability.

There's two immediate workarounds that come to mind:

Use a naming convention rather than a strongly-typed inteface for the application hook (e.g. a static class named "SwaggerWebHostFactory" with a method called "CreateWebHost")
Introduce an additional package for the sole purpose of defining the ISwaggerWebHostFactory interface.

Thoughts?

@domaindrivendev Thanks for taking the time to resolve this issue! I'm excited to get spec generation restarted in my project.
As for your solutions, my vote would be a naming convention, just because I'm generally in favor of keeping dependencies to a minimum, but certainly either could work. However, either way, I think it would be great if the CLI could fail fast and specify exactly what consumers need to do if they haven't properly configured their project, whether it's defining this static class or installing the additional package.

Is there a resolution for this? With upgrading our project from .Net core 2.2 to .Net core 3.1 and also from StructureMap (IoC) to Lamar, I am facing this issue:
Unhandled exception. System.InvalidOperationException: No service for type 'Microsoft.Extensions.DependencyInjection.IServiceProviderFactory1[Lamar.ServiceRegistry]' has been registered.    at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)    at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)    at Microsoft.AspNetCore.Hosting.StartupLoader.ConfigureServicesDelegateBuilder1.<>c__DisplayClass14_0.g__ConfigureServicesWithContainerConfiguration|0(IServiceCollection services)
at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.ConfigureServices(IServiceCollection services)
at Microsoft.AspNetCore.Hosting.WebHost.EnsureApplicationServices()
at Microsoft.AspNetCore.Hosting.WebHost.Initialize()
at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
at Swashbuckle.AspNetCore.Cli.Program.BuildWebHost(Assembly startupAssembly) in C:\projects\ahoy\src\Swashbuckle.AspNetCore.Cli\Program.cs:line 142
at Swashbuckle.AspNetCore.Cli.Program.<>c.b__0_3(IDictionary2 namedArgs) in C:\projects\ahoy\src\Swashbuckle.AspNetCore.Cli\Program.cs:line 68    at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable1 args) in C:\projects\ahoy\src\Swashbuckle.AspNetCore.Cli\CommandRunner.cs:line 68
at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args) in C:\projects\ahoy\src\Swashbuckle.AspNetCore.Cli\CommandRunner.cs:line 68
at Swashbuckle.AspNetCore.Cli.Program.Main(String[] args) in C:\projects\ahoy\src\Swashbuckle.AspNetCore.Cli\Program.cs:line 107
Swashbuckle.AspNet.Core: 5.2.1
"tools": {
"swashbuckle.aspnetcore.cli": {
"version": "5.2.1",
"commands": [
"swagger"
]
}
}

@shivanaru We ran into a very similar error with Autofac trying to generate the Open API document, though using Swasbuckle/CLI version 5.3.1:
No service for type 'IServiceProviderFactory[Autofac.ContainerBuilder]' has been registered

We worked around this by using the SwaggerWebHostFactory instead of SwaggerHostFactory. Our main app is still built & run using a generic host configuration, but we exposed a one-off static method to create our app using the .NET Core 2 style WebHost just for static spec generation:
public static IWebHost BuildWebHost(string[] args) =>
    WebHost.CreateDefaultBuilder(args)
        .ConfigureServices(s => s.AddAutofac())
        .UseStartup<Startup>()
        .Build();

I don't know if this applicable to your particular situation, but it helped us.

Hi all,
Doesn't look like i can add Swashbuckle.AspNetCore.Cli to 3.1 projects, therefore i can't reference ISwaggerWebHostFactory. Do I have another option?
Thanks!

@MarkLeMerise Do you have a 2nd app that references the 1st app so you can reference the controllers?

Hi all,
Doesn't look like i can add Swashbuckle.AspNetCore.Cli to 3.1 projects, therefore i can't reference ISwaggerWebHostFactory. Do I have another option?
Thanks!

I moved to using NSwag and it works well with Open API/Swagger v3 spec. We also needed to generate the TypeScript interfaces and NSwag does that as well. Was easy to configure as well.
Hope that helps!

@MattHartz

Doesn't look like i can add Swashbuckle.AspNetCore.Cli to 3.1 projects, therefore i can't reference ISwaggerWebHostFactory. Do I have another option?

Yeah, just use NSwag 😛 Judging from your following comments, it sounds like you did get the tool installed. As a test, I switched my project to .NET 3.1 (from 3.0) and didn't have any issues restoring the CLI tool and generating the Open API document.

Do you have a 2nd app that references the 1st app so you can reference the controllers?

No second app. My Web project contains all the controllers and SwaggerWebHostFactory. The dotnet swagger command runs as a post-build step on this Web project (per the Swashbuckle docs).

...and it now can successfully build as long as i commented out my "ConfigureContainer" method specified by Autofac.

Per my previous comment, I'm using the SwaggerWebHostFactory because trying to build a generic IHost (via SwaggerHostFactory) with an Autofac container didn't seem to play nicely with the Swashbuckle CLI.

@MarkLeMerise Yeah, that's what I ultimately ended up doing. You can kinda tell in that little code section I posted. I had to read to code to see what I literally was looking for IHost or whatever.
Thanks for responding! :)

Had the same issue and I got kinda confused by the replies here.
What fixed it for me was adding this:
public class SwaggerHostFactory
{
    public static IHost CreateHost()
    {
        return Program.CreateHostBuilder(new string[0]).Build();
    }
}

Had the same issue and I got kinda confused by the replies here.
What fixed it for me was adding this:
public class SwaggerHostFactory
{
    public static IHost CreateHost()
    {
        return Program.CreateHostBuilder(new string[0]).Build();
    }
}


Adjusted reconnect.py for ignoring zombie session id's during deploym…

…ent. Works with bonsai-cli=1.0.5, but not 1.0.4 so upgraded environment.yml

ignore session id's if the bsuc command doesn't work (blocked list)
if too many ignores happen with retries, reverse the list of session id's in hopes of connecting valid session id's
upgrading 1.0.5 for bonsai-cli

Pasting PR into ICM: https://portal.microsofticm.com/imp/v3/incidents/details/237840669/home


Using Object Oriented Programming with AsyncClient

Greetings,
My apologies if this is not the best way to seek help on this issue. I tried Python Discord and Stack Overflow but could not find a solution to my problem.
I am trying to write this piece of code in an object oriented way by creating the client object within my __init__() but this doesn't seem to work.
class BinanceInterface:
    def __init__(self) -> None:
        pass
    async def make_client(self):
        self.client = await AsyncClient.create(
            BINANCE_SPOT_API_KEY, BINANCE_SPOT_API_SECRET, testnet=True
        )

I wish to make an internal interface for the this library to use within my project.

You can't achieve what you seek at present, since python hasn't async __init__().
As you can see, this project is using a common practice, by constructing the instance in async create() method. You can do the same thing in BinanceInterface.

Thank you for your response. My the time of raising this issue, my understanding of asynchronous programming in Python was greatly lacking. I understand this now. Thank you for responding.


Bypass the two audit warning for dev deps temporarily in CI build

@Mergifyio backport release/2.16.x


Add epel9 and epel9-next

Signed-off-by: Troy Dawson<EMAIL_ADDRESS>

Thanks for the contribution! At a first glance, I think you found all the places where new releases need to be added in both code and tests.
Since I haven't yet had time to set up a CI pipleline for this project (this is also the first PR I've got!), I'll pull your PR locally and run the tests manually. ⏱

There's some minor problems with the stuff under /tests (since EPEL-9 is not yet active in Fedora, generating tests and downloading data for that release will fail), but I'll fix that with a follow-up commit.

Changes published with version 1.1.1: https://github.com/ironthree/bodhi-rs/releases/tag/1.1.1


Behavior of Calc "join" function with arrays or objects is weird

For example: a=[1,2,3]; join a -> "132", since each of the values is treated as if it were listed individually (as in "join 1,2,3"), so the 3rd value is the "glue" value...
It would be better if with one parameter, which is an array or object, the values are simply concatenated, so that join a -> "123". An optional second parameter would be the "glue" value, so that join a, ',' -> "1,2,3" as you would kind of expect.
The other tricky thing is that the order of the values in an object (map) is undefined (based on a hash function), but it would be better if they were listed in alphabetical order by the key names (maybe??) or in the order of definition (how to figure this out?), but in some predictable order.

Both the code and the help page have been updated.


re-render view without "show" animation?

First off — thanks for your awesome work on this! Loving it.
I'm doing a simple user login form in my modal. If auth fails, I want to shake the modal, and show some error messages at the top of the form. I was thinking of doing this in the data passed into the modal's view template, but when I call .render() on the modal, it does the fade-in animation again, so it feels like a whole new modal.
Is there any way to refresh the contents of the modal without any animation?
I've noticed the tab example, where I can define an area inside my modal view for other view(s) — that feels a little more complex to me for my use-case (where I only really need one view), but is that the answer I'm looking for?

I ended up making this work quite nicely based on the tab example, but with only one "subview".
Assuming you created the modal with one or more backbone views via the Modal views object/array, I discovered you can access the view shown inside the modal and re-render it as follows:
modalView.currentView.render()

This re-renders the modal's content, without redrawing the entire modal, thus no animation or flashing.

@jeff-h from what I can see, child views do not have a render method. At least not in current version.


Modularization: Egress Gateway

The Egress Gateway consumes a bunch of events from k8s, creates a desired current state from that and finally updates some BPF maps to effect changes in the datapath. It's a good example of the kind of work the various subsystems do (so I've been told!)
Based on discussion with @jibi we've settled on:

[x] Move EGW specific bpf maps into pkg/egressgateway and provide a cell for them
[x] Wrap egressgateway into a cell with minimal changes
[ ] Write a simple test that feeds some fake events into egressgateway and ensures that the correct action is taken
[ ] Split egressgateway into datapath (responsible for updating maps) and control plane part (which integrates events)

Splitting the EGW is delayed until excludedCIDR lands to avoid conflicts. Writing tests will be harder, since it's not trivial to get the correct BPF program to feed packets to.
Related: https://github.com/cilium/cilium/issues/23425

Few additional thoughts:

Move option.Config.EgressGateway into EgressGatewayConfig (cell.Config)
Once feasible, consider moving the datapath parts (the egressgw map and the wrapper around it) to e.g. pkg/datapath/linux/egressgateway. Though this should probably wait until the datapath cell has been moved there from daemon/cmd so we have some scaffolding into which to hook it.
For BPF tests take a look at bpf/tests/tc_egressgw.c. Default "test" build of bpf_host.o might actually work.
For the event handler, we'll need Resource[CiliumEndpoint], Resource[CiliumNode] and Resource[CiliumEgressGatewayPolicy] added to pkg/k8s/shared_resources.go. The last one is easy since the existing K8sWatcher is not tied with anything else, but the first two are more involved. CiliumEndpoint watcher for example gets stopped when kvstore connects (watchers/cilium_endpoint.go:102). BTW, does this mean egressgateway is not compatible with kvstore?

@dylandreimerink did some initial modularization work for egress gateway: https://github.com/cilium/cilium/pull/23046.

Didn't end up moving maps to pkg/egressgateway for now.

With feature freeze being one week away, I assume we aren't going to finish the last remaining items for 1.14. Will we finish this for 1.15?

Sorry, I didn't see your comment. I wrote the last two bullet points before working on the egw code base. "Injecting events" will start happening when we start using Resource[T] and the dataplane / controlplane split is a bit fuzzy as a concept, I think. EGW already has the split between the maps used to communicate between DP and the manager, so in a way this already exists. We can revisit whether the interface should change once the dust has settled on porting to Resource[T].
So my preference would be to close this ticket and track Resource[T] elsewhere.


Prevent SRG SSR livestreams from being played in the past

Pull request
Description
This PR ensures that player livestream items delivered by CoreBusiness cannot be played in the past.
Changes made

Implement short buffering strategy with live edge anchoring.
Update the demo with an example (Showcase tab).
Update player item creation API.

Checklist

[x] Your branch has been rebased onto the main branch.
[x] APIs have been properly documented (if relevant).
[x] The documentation has been updated (if relevant).
[x] New unit tests have been written (if relevant).
[x] The demo has been updated (if relevant).

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
-

tvOS
1.0.0 (104)


custom stub not working

I want to use a custom stub for example adding some functionality (like decrypting, ...) to the stub. but the default stub is base64 binary and obviously can't edit it. there is a folder named stub in the repo that contains the C source of a simple stub. I compiled it and gave it to amber but the generated file doesn't work. so I appreciate it if you give me the source of the b64 default stub or tell me why the C stub doesn't work.
thank you

contact me on discord @silentsniper im selling what u need


Which platform are you using? I've not since "apt install" before, only "apt-get install".

Debian 8, in new Ubuntu it's also working.
It can be apt-get, no difference for me, just less typing. :smile:

Yes, Debian now has apt, apt-get and aptitude. Not confusing at all.
Isn't apt install gnome-tweak-tool also proposed ? (using the arrow keys)

Which one is the most common? I only use apt-get. So in my mind that is the one that should be used.

@mcarton: isn't.
@NDevox: apt-get should be preferred, because apt and apt-get works the same and apt-get works also in older systems. apt is preferred by me only because it's 4 keys to type less, and if I don't must write it at all it's no difference for me. :smiley:

"it also works also on older systems" is a terrible reason.
We would never use anything new if we'd blindly follow that.

@mcarton but apt is nothing really new, it's only shortcut wrapper for apt-get...

You should install python-commandnotfound / python3-commandnotfound for apt-get rule.

@nvbn the install script could do that on Debian/ubuntu.

@mcarton yep =)

That was fast :)


Regression: links inside html are now link nodes

don't know if this is really a bug, but at least a regression.
with 0.5.4, a link inside html was parsed as a text node:
markdown:

| links |
|-------|
|
https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
|

mdast:
root[1] (1:1-5:1, 0-176)
└─0 table[2] (2:1-4:155, 1-175)
    │ align: [null]
    ├─0 tableRow[1] (2:1-2:10, 1-10)
    │   └─0 tableCell[1] (2:1-2:10, 1-10)
    │       └─0 text "links" (2:3-2:8, 3-8)
    └─1 tableRow[1] (4:1-4:155, 21-175)
        └─0 tableCell[5] (4:1-4:155, 21-175)
            ├─0 html "
" (4:2-4:5, 22-25)
            ├─1 html "" (4:5-4:81, 25-101)
            ├─2 link[1] (4:81-4:146, 101-166)
            │   │ title: null
            │   │ url: "https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be"
            │   └─0 text "https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be" (4:81-4:146, 101-166)
            ├─3 html "" (4:146-4:150, 166-170)
            └─4 html "
" (4:150-4:154, 170-174)

but with version 0.5.5, it is parsed into a link node:
markdown:

| links |
|-------|
|
https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
|

mdast:
root[1] (1:1-5:1, 0-176)
└─0 table[2] (2:1-4:155, 1-175)
    │ align: [null]
    ├─0 tableRow[1] (2:1-2:10, 1-10)
    │   └─0 tableCell[1] (2:1-2:10, 1-10)
    │       └─0 text "links" (2:3-2:8, 3-8)
    └─1 tableRow[1] (4:1-4:155, 21-175)
        └─0 tableCell[5] (4:1-4:155, 21-175)
            ├─0 html "
" (4:2-4:5, 22-25)
            ├─1 html "" (4:5-4:81, 25-101)
            ├─2 link[1] (4:81-4:146, 101-166)
            │   │ title: null
            │   │ url: "https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be"
            │   └─0 text "https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be" (4:81-4:146, 101-166)
            ├─3 html "" (4:146-4:150, 166-170)
            └─4 html "
" (4:150-4:154, 170-174)

As a consumer of the mdast, it is very difficult to remove the link inside the .... I don't know what the correct behaviour should be...

I just realized that the same happens in 0.5.4 when the inner text is sourrounded with spaces:
const doc = `
| links |
|-------|
|
 https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
|
`;

I just realized that the same happens in 0.5.4 when the inner text is sourrounded with spaces:
const doc = `
| links |
|-------|
|
 https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
|
`;

I believe actual and expected in your issue are the same thing?

I believe actual and expected in your issue are the same thing?

I believe actual and expected in your issue are the same thing?

oh, I think I copy pasted the wrong snipped.... one sec :-)

I believe actual and expected in your issue are the same thing?

oh, I think I copy pasted the wrong snipped.... one sec :-)

Btw, GH doesn’t render it nicely either:
Take this, to disambiguate which link “wins”:
| links |
|-------|
|
https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
|

Gives:

links

https://www.youtube.com/watch?v=kd2i50J9MZI&#x26;feature=youtu.be

Note that the autolink literal is still used.
Rendered here:

links

https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be

The same is true for gists btw

So, what would the expected output be for you? I know that this is a change, but gh also has the two links, and exactly matching how gh sanitizes their html isn't really something for this project either 🤔

In my example, the link href and the link text are the same. so this can be considered as an already autolinked HTML result. in this case, I think the gfm should not autolink it again.

so this can be considered as an already autolinked HTML result. in this case, I think the gfm should not autolink it again.

But this is not how GFM handles it: the autolink literal is linked. They do have two s. So I don’t see why this extension should do that?
For the actual GFM output: given that they sanitize the output, which is done by parsing the output of markdown as HTML and then cleaning it, and given that HTML parsing closes an open anchor element when another opening anchor tag is opened:
d = document.createElement('div')
d.innerHTML = ''
console.log(d.innerHTML)

Yields:

I think the current behavior is correct: if micromark/remark outputs '' and GH outputs , both are seen as the same thing in browsers.

but when add the HTML from above:

https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be

https://www.youtube.com/watch?v=kd2i50J9MZI&feature=youtu.be
you can see that github doesn't add an additional

That’s a different case, and micromark/remark (most probably) handle that correctly, see: https://gist.github.com/wooorm/41e4ba5779fc89b78ece9d6a67faaa41.
Because p (or div) is a known “block” tag (ref: https://spec.commonmark.org/0.29/#html-blocks), the rest of the line is seen as raw HTML: emphasis and such won‘t work there either. Whereas in a table cell, or by using a non-block tag such as a (or span), the parser works differently, and “inline” things such as autolink literals or emphasis do work.

hmm... you're correct.

Btw, there’s definitely a bug with autolink literals, which is very weird, and I’m working on here: https://github.com/micromark/micromark-extension-gfm-autolink-literal/pull/6#issuecomment-770957354.
But for this specific case, mircomark/mdast/remark are as close we can get to how GH parses, I think

Closing this. Meanwhile, there are new releases for this project and the corresponding mdast utility to match GH!


Retrieve chat history from database

Thanks to the team for creating this wonderful chatbot. Can you please share your inputs on following queries?

In my chatbot, i want to show choices like a/b/c , based on the user inputs it should respond based on the selection.

I want to save the chat conversation in database (I have read the documentation) and pull out the chat histroy for analytics?

I want to host it inside Apache web server and got the this link for reference https://docs.djangoproject.com/en/1.11/howto/deployment/wsgi/modwsgi/

Thanks,
Krishna

Hi @vkosuri any inputs?

I don't understand what you are looking exactly. could you please elaborate.
By looking points those things are doable using chatterbot.

Feedback example will guide you how to do this https://github.com/gunthercox/ChatterBot/blob/master/examples/learning_feedback_example.py
chatterbot default storage is sqllite https://chatterbot.readthedocs.io/en/stable/storage/index.html#sql-storage-adapter
I haven't tried on apache, I know few other things
a. Django https://chatterbot.readthedocs.io/en/stable/django/index.html
b. Deploy it on heroku https://github.com/vkosuri/chatterbot-live-example
c. Using flask https://github.com/chamkank/flask-chatterbot

Thanks @vkosuri . I want to retrieve the chat conversations between bot and user, with that we can identify which are most asked.

Hi @vkosuri , I want to host it in open source web server like Apache. The one you suggested Heroku involve pricing. Please suggest open source project can host the django or WSGI?


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name? (First, Last)
Pawel Haracz
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/PawelHaracz/25daysofserverless/tree/master/Day11
What Challenge is this for? (Challenge number between 1 and 25)
11
(Optional) Anything else we should know? e.g., language used, location, blog post?
Cosmos Db, slack api - incoming webhooks, c#

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless

THANK YOU FOR YOUR #25DaysOfServerless SUBMISSION
We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


add README to docs

This should add the README to the root of the gh-pages branch after it's merged into master and the doc site is published to gh-pages. It will also be viewable at https://www.vinyldns.io/readme at that time although we aren't linking to it in the documentation site itself. closes #61

Codecov Report

Merging #122 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #122   +/-   ##
=======================================
  Coverage   94.22%   94.22%
=======================================
  Files         103      103
  Lines        4780     4780
  Branches       92       92
=======================================
  Hits         4504     4504
  Misses        276      276

Continue to review full report at Codecov.

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


Tjmadonna/add vitessce config

@tjmadonna Please update the api-spec.yaml to support this new endpoint

@maxsibilla I updated the spec file with the new endpoint


Prevent List from scrollling

Is there a way to prevent list from scrolling?
node, npm, react-native, react and native-base version
Node: 10.4.0
NPM: 6.1.0
react-native: 0.55.4
react: 16.3.1
native-base: most recent
Expected behaviour
List does not scroll
Actual behaviour
List view is scrolling
Steps to reproduce should include code snippet and screenshot

Is the bug present in both iOS and Android or in any one of them?
I assume both but only testing on iOS
Any other additional info which would help us debug the issue quicker.

Please follow the issue template guidelines


Fix formatting on copyright page

The content on the copyright page isn't inside a container and looks a bit awkward.
Before

After

The copyright page has been replaced with the full Legal page which has all Copyright, Privacy, and Terms of Use on it. It would be better for the Copyright page to be removed entirely. I usually try to leave 'deleted' pages on the site for a couple months just in case there is great outcry and the changes need to be reverted.

Sorry, I didn't realise it was meant to be deleted. I followed a link to it from the forums.
It's safe to delete pages because they'll all be in the Git history if you ever want them back.


don't know why this programme is not executing properly

@prabal-007 can you provide more information?

It’s a Programme to make a calculater in c.
The Programme is correct according to me but when I execute it
It executes but skip one statement idk why so. 🤷🏻♂🤷🏻♂


Current auto created bot role does not have access to committee channels by default

CSS' setup is slightly fucked that we have given the bot a completely separate role to the default one it creates itself which has full admin permissions

come again

The bot creates it's own "TeX" role when it's invited to a server, intended to give it all the necessary permissions to Exist:tm:
This default role it creates doesn't have access to committee channels.
In CSS TeX does have access to committee channels because we've(not me) seemingly given it a new role (created manually) which has the Administrator permission


Different cursor over handle hover?

It would be nice to see a hold cursor while hovering over the draggable header.

I added a different cursor for the dropdown and also to tell that you can drag by the header.


use defaults from DefaultTransport for our custom transport (ie timeouts)

fixes #132
to test using k8s service catalog, do a svcat sync broker ups-broker repeatedly.  Without the fix, you see the number of established connections grow as reported by sudo nsenter  -t $BROKERPID --net netstat -n -p
With this fix, the number of established connections remains constant which is a small number related to the number of connections used within the last 90 seconds (IdleConnTimeout).

Coverage decreased (-1.1%) to 91.547% when pulling 4e8d1a3a83eb294a2a6a6bbead24894deedbde6c on jboyd01:use-defaulttransport-defaults into 9cc214e88d00504888af633376c9462e0ab2bd8b on pmorie:master.

Best repro test ever! 💯

Thanks to @carolynvs & @shawn-hurley for the review, I'll merge it.


Adds universal websocket manager any extension can use

Connect to the ws://<your-lnbits>/api/v1/ws/{item_id} websocket
POST data tohttps://<your-lnbits>/api/v1/ws/{item_id} or use the updater(item_id, data) function

Codecov Report

Merging #1146 (4b707b5) into main (c8e18ac) will decrease coverage by 2.92%.
The diff coverage is 48.14%.

@@            Coverage Diff             @@
##             main    #1146      +/-   ##
==========================================
- Coverage   51.64%   48.71%   -2.93%
==========================================
  Files         239      239
  Lines       13519    13545      +26
==========================================
- Hits         6982     6599     -383
- Misses       6537     6946     +409

Impacted Files
Coverage Δ

lnbits/core/views/api.py
41.19% <48.14%> (+0.37%)
:arrow_up:

lnbits/extensions/boltz/mempool.py
32.75% <0.00%> (-50.00%)
:arrow_down:

lnbits/extensions/boltz/crud.py
29.23% <0.00%> (-49.24%)
:arrow_down:

lnbits/extensions/boltz/utils.py
23.52% <0.00%> (-41.18%)
:arrow_down:

lnbits/extensions/boltz/boltz.py
17.91% <0.00%> (-39.31%)
:arrow_down:

lnbits/wallets/cln.py
19.00% <0.00%> (-29.76%)
:arrow_down:

lnbits/wallets/lndrest.py
17.09% <0.00%> (-29.06%)
:arrow_down:

lnbits/wallets/lndgrpc.py
22.72% <0.00%> (-27.93%)
:arrow_down:

lnbits/extensions/boltz/views_api.py
31.40% <0.00%> (-18.19%)
:arrow_down:

...nbits/wallets/lnd_grpc_files/lightning_pb2_grpc.py
38.22% <0.00%> (-12.25%)
:arrow_down:

... and 6 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Final 1.0 schema and samples

@matthidinger,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[docs] removing an extra character

Problem
There is an extra character on the page: ]
Summary of Changes
An extra character has been removed: ]

Thanks


dockerBuild fails with an exception

Hi,
I'm trying to use your plugin with a quite old version of the plugin. It's java8 + Gradle 5.6.4.
$ java -version
openjdk version "1.8.0_332"
OpenJDK Runtime Environment (Temurin)(build 1.8.0_332-b09)
OpenJDK 64-Bit Server VM (Temurin)(build 25.332-b09, mixed mode)

Gradle build task is finalized by dockerBuild task. Error is following:
$ ./gradlew build
Welcome to Gradle 5.6.4!

Here are the highlights of this release:
 - Incremental Groovy compilation
 - Groovy compile avoidance
 - Test fixtures for Java projects
 - Manage plugin versions via settings script

For more details see https://docs.gradle.org/5.6.4/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

> Configure project :
激活了插件

> Task :build
done:<PHONE_NUMBER>

> Task :dockerBuild FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':dockerBuild'.
> com/spotify/docker/client/shaded/com/google/common/base/Optional

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 33s
5 actionable tasks: 5 executed

update version to 1.2.3

It works fine now, thanks


remove/add entry sequence for zip archives created with macOS Finder

Summary
When performing a remove/addEntry sequence of operations on an Archive, the new entry does not appear in the archive if the original .zip file was created with the macOS finder.
Steps to Reproduce
command line program below reproduces the issue when used with attached .zip files:

import Foundation
import ZIPFoundation

testArchive(fileURL: URL(fileURLWithPath: "ZIPFoundationArchive.zip"))
testArchive(fileURL: URL(fileURLWithPath: "Archive.zip"))

func testArchive(fileURL: URL) {

    print("\n -----Testing remove/add for: \(fileURL.path) ------")

    let archive = Archive(url: fileURL, accessMode: .update)!

    let entryName = "myFile.txt"

    if let entry = archive[entryName] {
        try! archive.remove(entry)
        print("\(entry.path) removed from the archive")
    } else {
        print("could not find: \(entryName)")
    }

    let newData = "my data string".data(using: .utf8)!
    try! archive.addEntry(with: entryName, type: .file, uncompressedSize: Int64(newData.count), provider: {position, size in
        return newData.subdata(in: Int(position)..<Int(position)+size)
    })
    print("added entry: \(entryName)")

    if let entry = archive[entryName] {
        print("\(entry.path) is found in archive")
    } else {
        print("Error!  could not find: \(entryName) despite adding it to archive")
    }

}

Note:  Archive.zip was created in the macOS 12.0.1 Finder using the 'Compress' contextmenu.  ZIPFoundationArchive.zip was created by programmatically using ZIPFoundation itself.
Archive.zip
ZIPFoundationArchive.zip
Expected Results
myFile.txt is found in archive for both ZIPFoundation.zip and Archive.zip
Actual Results
myFile.txt is not found in archive Archive.zip
Regression & Version
Xcode 13.1
macOS 12.0.1
ZIPFoundation 0.9.13
Related Link

Thanks for the report - I can reproduce the problem with ZIPFoundation 0.9.13.
Looks like a regression with 0.9.13 - the remove/add sequence works properly with 0.9.12.
I haven't investigated in detail yet but the reason why only the Finder-generated archive is affected could be due to the entry count being > 1 (there's a hidden .DS_Store directory in there).
Maybe we are mis-calculating some offsets in remove(entry: ...)? /cc @Ckitakishi

I think I found the problem. With the latest release, the "Extra Field Data" (which is optional and can contain all sorts of  things such as time zones for modifications dates etc.) was not preserved properly for the remaining entries when an entry got removed.
While this doesn't cause any data integrity issues, it can lead to wrong offsets in some ZIP headers. Pedantic programs or libraries (like ZIPFoundation 🙈) might return nil when encountering mismatching offsets during entry lookup.
The issue should be fixed in the above PR. Will test a bit more and then tag a new release.

Fixed in https://github.com/weichsel/ZIPFoundation/releases/tag/0.9.14

Nice.  Works exactly as expected.  Thank you so much for ZIPFoundation.


General live query support

Background: https://groups.google.com/forum/m/#!topic/pipelinedb/DLINDuae1w4
Given an arbitrary, possibly non-aggregate query:
select m.body, u.name from rooms r
join messages m using(room_id)
join users u using(user_id)
where r.name = 'name';

One would both want all old records, and the feed of all new records matching the query. I suggested this in the mailing list:

I'm not sure I understand how the referenced issue would enable live queries, though. I can only think of something like:
create stream chat_messages;
create continuous view chat_log as select * from chat_messages;
Then:
copy (select * from chat_log) to stdout;
Or perhaps:
copy (select * from chat_log union all select * from chat_messages) to stdout;

But as I suspected, continuous views are not built for granular data.
So is there another way PipelineDB could support live queries?

@bopjesvla thanks for the writeup! The main reason continuous views have historically not been conducive to storing granular data is that we've never wanted PipelineDB to be a data warehouse. That being said, as our core high-throughput aggregation infrastructure has matured, I do see value in broadening the scope slightly to handle the types of continuous queries you've outlined, because the output of such a query isn't necessarily "granular" data: it is the output of some kind of processing.
The main technical reason that continuous views aren't well-suited for granular storage is that currently they're sharded by aggregation grouping so that combiner processes can continuously update disjoint subsets of the CV's rows without contention. So if a grouping doesn't exist, only one process can ever do any work for that CV.
It would be trivial to shard ungrouped CVs though (e.g. just make the shard key random since there would only ever be inserts). That in conjunction with #1582 would provide all the necessary infrastructure to enable "live queries".


Thumbnails

Instead of trying to intuit what the thumbnail should be automatically, for now we can let the user set the thumbnails manually. A button will appear in the render controls bar if only one object is selected for rendering.
If the layers shown only includes "Original", the thumbnail captured will be applied to the Dataset object. Otherwise it will be applied to the Project object.

The cards on the selection page will show thumbnails if they exist. Additionally, dataset cards will have a short summary of the objects they include (number of meshes, number of segmentations).

@manthey Good catch. Changed in 488ef81

When I change an existing project thumbnail and then navigate back to the project cards, I see a javascript console error:
Uncaught TypeError: Cannot read properties of undefined (reading '0')
   at publicAPI.getViewportSize (index.js?f8e9:50:1)
   at publicAPI.computeViewport (index.js?efce:37:1)
   at publicAPI.updateViewport (index.js?efce:67:1)
   at ResizeObserver.eval (index.js?efce:23:1)

@manthey that error seems to be coming from getViewportSize from the vtk render. I have a fix for that in the other PR: 3525cd1

@manthey that error seems to be coming from getViewportSize from the vtk render. I have a fix for that in the other PR: 3525cd1

Hmm... I tried merging the branches together and now I get a different console error.  Either I merged them badly or there is a lingering issue.

Is the error that you're getting similar to this.orientationCube is not defined? If so, this is just a merge caveat; in this PR I moved the orientation cube to be stored inside this.vtk, so the reference to that object should be changed from this.orientationCube to this.vtk.orientationCube. I suggest we merge one so I can reconcile this change in the other.

@manthey could you check that you no longer get the console error?

@manthey could you check that you no longer get the console error?

Yes, it now is error free in my test.


SDXL support

It doesn't look like it's running for SDXL.  Any chance it could support that as well?

Also, thanks for doing this so quickly! :)

Here's a quick and dirty patch.  It duplicates a ton of code, but it works:
https://pastebin.com/7E57HEZp

Thanks, working on it. Should push a fix shortly.

I'm not happy with all that copy pasted code but I guess it's not really practical to monkey patch it either. There may be a better solution but we'll go with that for now


Support for ARIA label

For the web accessibility support ARIA label to be given in props

Hello,
This component is being used in our project but can't add an aria-label to pass the accessibility insights.

Sorry for a late reply. Please give me an example of what it should look like. Feel free to make a PR if you want to.


Add option to color schemes to limit the scale range to selection

especially for atom/residue/chain/modelindex color schemes

residueindex colors now per chain, chainname per model
more flexibility is always enabled by custom schemes


[desktop] sogou input window can't track textfield cursor position

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @yinzhou2018
Can you please provide your flutter doctor -v
Thank you

------------------ 原始邮件 ------------------
发件人:                                                                                "flutter/flutter"                                                                                @.>;
发送时间: 2021年4月7日(星期三) 晚上7:33
@.>;
<EMAIL_ADDRESS>主题: Re: [flutter/flutter] 【Desktop for windows】sogou input window can't track textfield cursor position (#79933)
Hi @yinzhou2018
Can you please provide your flutter doctor -v
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @yinzhou2018
In order to reproduce the issue, we need more details about the issue, clear steps to replicate the issue and your flutter doctor -v
Here I tried to test textfield cursor pisition issue but there is nothing wrong when testing on the latest stable
Preview

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.19042.870], locale en-US)
    • Flutter version 2.0.4 at C:\Users\Taha\Code\flutter_stable
    • Framework revision b1395592de (6 days ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\Taha\Code\SDK
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.9.31112.23
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3
    • Flutter plugin version 55.0.5
    • Dart plugin version 211.6693.108

[✓] VS Code (version 1.55.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68

! Doctor found issues in 1 category.

Input popup window position don't track textfield input cursor position in your screen snapshot
------------------ 原始邮件 ------------------
发件人:                                                                                "flutter/flutter"                                                                                @.>;
发送时间: 2021年4月8日(星期四) 下午3:20
@.>;
<EMAIL_ADDRESS>主题: Re: [flutter/flutter] 【Desktop for windows】sogou input window can't track textfield cursor position (#79933)
Hi @yinzhou2018
In order to reproduce the issue, we need more details about the issue, clear steps to replicate the issue and your flutter doctor -v
Here I tried to test textfield cursor pisition issue but there is nothing wrong when testing on the latest stable
Preview
flutter doctor -v  [✓] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.19042.870], locale en-US)     • Flutter version 2.0.4 at C:\Users\Taha\Code\flutter_stable     • Framework revision b1395592de (6 days ago), 2021-04-01 14:25:01 -0700     • Engine revision 2dce47073a     • Dart version 2.12.2 [!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)     • Android SDK at C:\Users\Taha\Code\SDK     • Platform android-30, build-tools 30.0.3     • ANDROID_HOME = C:\Users\Taha\Code\SDK     • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)     ✗ Android license status unknown.       Run flutter doctor --android-licenses to accept the SDK licenses.       See https://flutter.dev/docs/get-started/install/windows#android-setup for more details. [✓] Chrome - develop for the web     • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe [✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)     • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community     • Visual Studio Community 2019 version 16.9.31112.23     • Windows 10 SDK version 10.0.19041.0 [✓] Android Studio (version 4.1.0)     • Android Studio at C:\Users\Taha\Code\android-studio     • Flutter plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/9212-flutter     • Dart plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/6351-dart     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01) [✓] IntelliJ IDEA Community Edition (version 2021.1)     • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3     • Flutter plugin version 55.0.5     • Dart plugin version 211.6693.108 [✓] VS Code (version 1.55.0)     • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code     • Flutter extension version 3.21.0 [✓] Connected device (3 available)     • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]     • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114     • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68 ! Doctor found issues in 1 category.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @yinzhou2018
I know what you mean now, I can see the difference when using other apps vs Flutter
Input window doesn't move with the cursor in Flutter
Flutter

Chrome

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Material App Bar'),
        ),
        body: Center(
          child: Container(
            child: TextField(),
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.19042.870], locale en-US)
    • Flutter version 2.0.4 at C:\Users\Taha\Code\flutter_stable
    • Framework revision b1395592de (7 days ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\Taha\Code\SDK
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.9.31112.23
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3
    • Flutter plugin version 55.0.5
    • Dart plugin version 211.6693.108

[✓] VS Code (version 1.55.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.1.0-13.0.pre.556, on Microsoft Windows [Version 10.0.19042.870], locale en-US)
    • Flutter version 2.1.0-13.0.pre.556 at C:\Users\Taha\Code\flutter_master
    • Framework revision 1a36c18497 (2 hours ago), 2021-04-08 23:33:00 -0700
    • Engine revision a9f4883dc8
    • Dart version 2.13.0 (build 2.13.0-219.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\Taha\Code\SDK
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.9.31112.23
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3
    • Flutter plugin version 55.0.5
    • Dart plugin version 211.6693.108

[✓] VS Code (version 1.55.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68

! Doctor found issues in 1 category.

Yes, there is the same issue in macos platform,  maybe need a system test in all desktop platforms including linux.
------------------ 原始邮件 ------------------
发件人:                                                                                "flutter/flutter"                                                                                @.>;
发送时间: 2021年4月9日(星期五) 下午4:55
@.>;
<EMAIL_ADDRESS>主题: Re: [flutter/flutter] 【Desktop for windows】sogou input window can't track textfield cursor position (#79933)
Hi @yinzhou2018
I know what you mean now, I can see the difference when using other apps vs Flutter
Input window doesn't move with the cursor in Flutter
Flutter
Chrome
code sample  import 'package:flutter/material.dart'; void main() => runApp(MyApp()); class MyApp extends StatelessWidget {   @override   Widget build(BuildContext context) {     return MaterialApp(       title: 'Material App',       home: Scaffold(         appBar: AppBar(           title: Text('Material App Bar'),         ),         body: Center(           child: Container(             child: TextField(),           ),         ),       ),     );   } }
flutter doctor -v  [✓] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.19042.870], locale en-US)     • Flutter version 2.0.4 at C:\Users\Taha\Code\flutter_stable     • Framework revision b1395592de (7 days ago), 2021-04-01 14:25:01 -0700     • Engine revision 2dce47073a     • Dart version 2.12.2 [!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)     • Android SDK at C:\Users\Taha\Code\SDK     • Platform android-30, build-tools 30.0.3     • ANDROID_HOME = C:\Users\Taha\Code\SDK     • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)     ✗ Android license status unknown.       Run flutter doctor --android-licenses to accept the SDK licenses.       See https://flutter.dev/docs/get-started/install/windows#android-setup for more details. [✓] Chrome - develop for the web     • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe [✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)     • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community     • Visual Studio Community 2019 version 16.9.31112.23     • Windows 10 SDK version 10.0.19041.0 [✓] Android Studio (version 4.1.0)     • Android Studio at C:\Users\Taha\Code\android-studio     • Flutter plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/9212-flutter     • Dart plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/6351-dart     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01) [✓] IntelliJ IDEA Community Edition (version 2021.1)     • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3     • Flutter plugin version 55.0.5     • Dart plugin version 211.6693.108 [✓] VS Code (version 1.55.0)     • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code     • Flutter extension version 3.21.0 [✓] Connected device (3 available)     • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]     • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114     • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68 ! Doctor found issues in 1 category.
[✓] Flutter (Channel master, 2.1.0-13.0.pre.556, on Microsoft Windows [Version 10.0.19042.870], locale en-US)     • Flutter version 2.1.0-13.0.pre.556 at C:\Users\Taha\Code\flutter_master     • Framework revision 1a36c18497 (2 hours ago), 2021-04-08 23:33:00 -0700     • Engine revision a9f4883dc8     • Dart version 2.13.0 (build 2.13.0-219.0.dev) [!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)     • Android SDK at C:\Users\Taha\Code\SDK     • Platform android-30, build-tools 30.0.3     • ANDROID_HOME = C:\Users\Taha\Code\SDK     • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)     ✗ Android license status unknown.       Run flutter doctor --android-licenses to accept the SDK licenses.       See https://flutter.dev/docs/get-started/install/windows#android-setup for more details. [✓] Chrome - develop for the web     • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe [✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.2)     • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community     • Visual Studio Community 2019 version 16.9.31112.23     • Windows 10 SDK version 10.0.19041.0 [✓] Android Studio (version 4.1.0)     • Android Studio at C:\Users\Taha\Code\android-studio     • Flutter plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/9212-flutter     • Dart plugin can be installed from:       🔨 https://plugins.jetbrains.com/plugin/6351-dart     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01) [✓] IntelliJ IDEA Community Edition (version 2021.1)     • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2020.3.3     • Flutter plugin version 55.0.5     • Dart plugin version 211.6693.108 [✓] VS Code (version 1.55.0)     • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code     • Flutter extension version 3.21.0 [✓] Connected device (3 available)     • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.870]     • Chrome (web)      • chrome  • web-javascript • Google Chrome 89.0.4389.114     • Edge (web)        • edge    • web-javascript • Microsoft Edge 89.0.774.68 ! Doctor found issues in 1 category.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I was only able to verify on windows since this required following Chinese instructions when installing this keyboard, i got help for someone to install it on windows a:desktop should all desktop environments

cc @cbracken

Yes, there is the same issue in macos platform,  maybe need a system test in all desktop platforms including linux.

@TahaTesser 的视频中，输入框是在textfield 的下方，而我的情况是输入框固定在屏幕左下角位置。
In @TahaTesser 's video, the input box is below the textfield, and in my case, the input box is fixed at the lower left corner of the screen.
by google translate
OS:  macops
flutter:  2.2.0-11.0.pre.152


Add default annotation to heatmap.

The default annotation is format(float(value), '.2g') where value is the value in the value_map passed to Heatmap.__init__. It should work if the value is always SupportsFloat specified by the ValueMap type.
To remove the annotation, the user can call unset_annotation() method afterwards.
cirq.Heatmap(value_map).unset_annotation().plot(ax)

Demo:

Automerge cancelled: A status check is failing.


Explicitly specify required X11 components

The simplification to our CMake scripts about finding X11 in #2392 went a little bit too far as we noticed. This PR brings back the explicit list of X11 components we link to.
This makes the project fail to configure if a component is not found instead of failing later at compile time.

SFMLBuildMaster: Build this please.


Infinite load when stake/unstake in the following cases

Description the bug

[x] Case 1: Infinite load when staking for validator that have Commission = 100 %

To reproduce the bug
Steps:

Open the extension
Perform staking for validators that have Commission = 100 %
Input valid data for the all field then observe staked result

Actual: Infinite load after user select Confirm transaction
Expected: Show warning message when user click on Start staking button

[x] Case 2: Infinite load when staking for validator that have max "Nominators count/ staked count"

To reproduce the bug
Steps:

Open the extension
Perform staking for validators that have max "Nominators count/ staked count"
Input valid data for the all field then observe staked result

Actual: Infinite load after user select Confirm transaction
Expected: Show warning message when user click on Start staking button

[x] Case 3: Infinite load when staking for validators that have minimum stake = 0 and user stake 0

To reproduce the bug
Steps:

Open the extension
Perform staking for validators that have minimum stake = 0
Input Amount = 0 and input valid data for the remain fields then observe staked result

Actual: Infinite load after user select Confirm transaction
Expected: Add validate: Amount must be greater than 0

version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./761-merge-build-0.6.7-1666930712960.zip
Vaildation passed!


Please provide GPU requirements!

Amazing work guys, but why most projects totally skip the hardware requirements needed to run? Isn't it the most important information for a project that can be run locally?

Thank you for your advice! In my tests on an A100 GPU, the VRAM usage is approximately 26.38GB. However, you can reduce it to around 17GB by using the offloading method with the following code after loading the pipeline:
pipe.enable_model_cpu_offload()

Thanks a lot for posting it! Will wait to test till it can run on poor GPU (12GB) ... Best wishes guys!

You can also test it using the Hugging Face online demo, available here.

Sadly too much traffic; wait time it over 2 hours :(

Then use my template on Runpod.io.
https://runpod.io/console/deploy?template=8pqb3dg7lq&ref=2pdhmpu1
1 video generate time: 8 min

Testing on RTx4080s, half an hour has passed and progress has only reached 10%. I have given up

Testing on RTx4080s, half an hour has passed and progress has only reached 10%. I have given up

Use A40 . 1 video 8 min

I'm able to run it on a 3090 and get about 8-10 minutes per video


Migrate dependencies to external core package

Many parts of azure.cli.core have been moved to an external package.  Move dependencies over to that package and remove the duplicate code from the CLI.

cc/ @derekbekoe, @mayurid

See also https://github.com/Azure/azure-cli/issues/838

Moving this forward to Sprint 21 since knack work will likely need to be done before I can return to this for the third phase of changes.

Continuing work into sprint 22 (and likely into 24 and beyond!)

Closing this as we now have a project with each of these tracked as individual issues.


chore(*): warning box about Lean2

@Kha  Thanks. This will help eliminate confusion. I made one small change, "Lean2" to "Lean 2".


Is it Possible To Change the Google Maps Language to English Only Rather Than Device's System Language?

Environment details
iOS 18
Using XCode 16.0
Google Maps iOS SDK 9.1.1
Expectation:
We need Google Maps iOS SDK to show places in English language only, even the device language has set to something else.
Description:
The expectation is that setting the language parameter when fetching the place details from Google Maps APIs should return results in that given language. However, it appears that, when working with Turkish locations for instance, the API may return addresses in Turkish, even when English is specified.
Example URL for fetching place details:
https://maps.googleapis.com/maps/api/place/details/json?place_id=YOUR_PLACE_ID&key=YOUR_API_KEY&language=en
How can I force Google Maps iOS SDK to show places in English language only even the device language is something else? Is it possible? If it's not possible, is it possible to change the language of the address details using placeId after user selected a location?
func viewController(_ viewController: GMSAutocompleteViewController, didAutocompleteWith place: GMSPlace) {
     // TODO: when user selects a place, I want to change it to English language before using this info.
        For instance, if the selected place is
        "Eskişehir, Türkiye" I want to use it like => "Eskisehir, Turkey"
        including English formatted address details.
}

Thank you in advance.

To ensure that the selected place information is returned in English, you can set the language preference for GMSAutocompleteViewController or GMSPlacesClient.
You can specify the desired language by configuring GMSPlacesClient or GMSAutocompleteViewController to use English.
GMSPlacesClient.shared().autocompleteSessionToken().sessionLanguageCode = "en"


Feature/edit file

OPH-138

updated my comments as it wasn't making sense 😬

You're right. For now, I've made the pr draft. I will now first continue the other ticket in this branch (replacing bpmn file) and later (probably not this sprint anymore, so i'll create a ticket) finish the pr with the correct graph/roles separation


Set a higher timeout for Travis apt-get

The Travis builds are failing due to apt-get running into a timeout. This seems to be a common problem which can be fixed by setting a higher timeout. For this, the APT addon must be changed in favor of three manual before_install apt-get calls.

Looks like Travis is working fine now.

Thanks for this improvement.
Let's hope Travis builds get more stable now.


Kotlin Langaue server perfectly working in windows

just we need to change the binary path of
let binary_path = format!("{version_dir}/server/bin/kotlin-language-server"); to
let binary_path = format!("{version_dir}/server/bin/kotlin-language-server.bat");
@maxdeviant  @cholwell  @Angelk90  @osiewicz

As this issue has been resolved closing the issue.


Feature Request: Additional "details" row for album grid items (i.e "Year" repositioning, "Release type", etc..)

Hi CDrummond,
With release types gaining more feature support, I was wondering if it might be a useful option to display an "additional info" row under album grid items -- something that aligns release type with release year and potentially other information. (Could even be clickable?):
Proposed functionality:

Perhaps useful for displaying other potential items like technical info, format/resolution badges, genre, etc...
Of note, one thing that always bothered me was how the (Year) would often get cut off when aligned with the album name. At least having it on a new row, similar to browsing under an Artist, would allow (Year) to be visible and visually consistent:
Current functionality (Under Artists):

It would inflate the grid items a bit, hence it would be best as a Interface toggle. Text line-height or font size might also be tweaked? Just some thoughts.

I got the idea from a few other apps, which we could take inspiration from:
Spotify:

Jplay:

If release type is important you can configure Material (via LMS's settings) to always group by release type - and this will affect 'New Releases', 'Releases', etc.
Sorry, but not too keen on adding a 3rd row of text just for release year.


add Adobe's package paths to default path list

This PR simply adds the paths from Adobe's official packages (that can be downloaded at get.adobe.com/flashplayer/otherversions/). I think it could be a very common path for libpepflashplayer.so as it's used in the RPM package distributed by Adobe.

Looks good to me.
Thanks!


ServiceConnectionCallback.onServiceDisconnected never called

I am looking for a solution to check if the ChromTab is closed.
For this reason I try the Custom-Tabs-exampe-app with "Service Connection Activity"
Looking to the  ServiceConnectionCallback interface I assume the method onServiceDisconnected is called when the Chrom tab is closed.
Smartphone (please complete the following information):

Device: [Pixel 3a]
OS: [Android 11]
Browsers Installed [Chrome]
Browser Versions [92.0.4515.131]
android-browser-helper library version [androidx.browser:browser:1.3.0 and 1.1.0]

A better way should be to get an onServiceDisconnected callback.

OK I try the CustomTabsCallback


Added failing test for strange Promise.reject issue

I think this is a "wont fix" for now. It looks like a bug. I was able to create a minimal repro case for this problem.
I think that this is what happens:

TS grabs the function type from this service property
it gets its return type and provides the contextual type (the provided string), this expands the union from like Union<T> to Union<string>
it recognized the function as being async so it wraps that return type in a Promise, so the type becomes Promise<Union<string>>
it checks if the type is a union and if it is then it would try to narrow it down, this is probably what happens for my test4, However, at this time this type is not a union so this stage is being skipped

I think that, potentially, this could even be an easy fix in TS itself -> just move the narrowing logic before wrapping the type into a Promise type. However, promises have this "auto-flattening" behavior that is recursive and maybe this is why this (unconfirmed) order of operation has been chosen. Although, it feels that this wouldn't change that much as the final "flattening"/unwrapping should happen at the end - hard to tell without examining more situations and edge cases.

This is an upstream issue in TS (see here) and the fix for it awaits review here. It's not actionable on out side here.


Regression tests for bitshift

Disabled right now because it will be fixed by  #4821

Can you also add a test for when bitshift isn't triggered? (arg not in set)

Added

#4821 was merged

#4821 was merged

Enabled tests

I'm fine with merging this as is, but I'm very supportive of end to end tests that apply the pattern matcher rules and confirm the output is the same. This could be fuzzed as well.

@Qazalin There are tests that specifically check the correctness of the calulations https://github.com/tinygrad/tinygrad/blob/5ac30c29d88fe4ae2240a9f1ac4b7b9c642a8427/test/test_uops.py#L125
In terms of test_ops introducing an error in shift left makes 107 tests fail and an error in shift right makes 22 tests fail.
Do you feel like there are some more explicit test needed?

Those tests don't rely on the pattern matcher, though I could verify that if there are errors in the pattern matcher, some test_ops fail. I think this is sufficient, thanks!


UI Docstring improvements

Moved config.py back inside the app folder to correct an import error (sorry about that!)
Removed the migrations folder (otherwise flask db upgrade fails). User should follow the typical method of flask-migrate:

flask db init
flask db migrate
flask db upgrade
flask init (to create the admin user)

Added code to inject ReDoc's Authentication legend to the openapi.yaml endpoint only (so it doesn't show up as an empty section in the Swagger UI)
Added summaries and descriptions to resource docstring for better appearance in both Swagger and ReDoc UIs

I also have a couple general questions regarding your choice of structure:

Why'd you opt to separate the "auth" endpoints from "api"? I'm guessing it's because it's unlikely for auth to change across API versions?

I curious how api versioning could be implemented with this structure. The "api" folder is where you instantiation flask-restful's Api. If we were to create an "api_v2" folder next to the "api" folder, would we instantiate another instance of flask-restful's Api or wouldnt just make more sense to instantiation Api with the rest of the flask extensions?

Hello @caffeinatedMike and thanks for this PR, also sorry for the delay on this answer.
This PR looks great to me, but small nitpicking: since migrations folder has been removed a users need to follow a new process for init, could you also update the README and Makefile to make it easier and avoid confusion ?
Now for your questions:

Why'd you opt to separate the "auth" endpoints from "api"? I'm guessing it's because it's unlikely for auth to change across API versions?

You're guessing right, and in addition I like to think of those two parts as two different APIs that serves different purposes, so in that case I use Flask's blueprint mechanism to induce some separation of concerns between them, marking the difference and keeping the code in dedicated places (at least for views). In addition, using blueprints with application factory make your life easier if you want to have 2 separate deployments for the API and the Auth part for example.

I'm curious how api versioning could be implemented with this structure. The "api" folder is where you instantiation flask-restful's Api. If we were to create an "api_v2" folder next to the "api" folder, would we instantiate another instance of flask-restful's Api? Wouldn't it just make more sense to instantiation Api with the rest of the flask extensions?

This is related to my previous answer but also to how flask-restful's documentation explain using it with blueprints here. So yes, my approach would be to create a new folder, with a new blueprint and associated Api object for a new version and separating the 2 versions as much as possible to avoid confusion, both in the code and in the documentation.

The security access and refresh token workflow seems a bit confusing. Why would someone revoke an access token instead of simply revoking the refresh token (to prevent creation of any further access tokens)?

When implementing a logout mechanism (for example on a frontend app using your API), you want to revoke both tokens. Sure, revoking the refresh token prevent creation of new access tokens, but meanwhile your current access token remains usable and you want to avoid this, that's why this API provide a way to revoke it, not allowing anyone that should be logged out to still use the API if they have an access token.

Related: In the Swagger UI, the POST /auth/refresh route doesn't appear to respect the value placed in the Try It Out header field and only takes from the blanket security applied via the Authorize button. This is due to the OpenAPI spec

Thanks for pointing this out, I'll try my best to check this and see how we could fix that.
I hope I answered your questions, don't hesitate if you want further details.
Once again thanks for this PR but also your questions, I haven't been able to put as much time as I wanted recently on this project so it's always nice to have people contributing to it and asking questions to challenge its current state.

@karec No worries on taking time to reply! I get it, life gets busy! Thank you for openly engaging in a discussion. I also really enjoy when I can civilly discuss implementation and pick other developers' brains to see why and how they came to the make the decisions they have in their structure/logic. As for my PRs, I'm happy to be able to contribute to open-source! It makes me feel great, knowing I can assist in improving something that is used by others.
As for the requested additional tweaks to the PR, I have updated the Readme and Makefile accordingly. Forgive me if I didn't quite get the docker commands right (I never used docker or docker-compose). And pertaining to the /auth/refresh route, there wasn't really anything to fix. The easiest remedy is removing the explicit Authorization header in the swagger docstring. I've taken care of that, plus updated that endpoint's description to more-explicitly indicate that the user should pass the refresh_token in the Authorization header, opposed to the usual access_token.


Remove Test targets filter

There is no obvious reason why amimono would not work for test targets.
On the other side, iOS 9 simulator works better when we don't have tons of frameworks, including in Test targets. Especially for UI tests.

We had EXACTLY the same issue. iOS 9 dylib is extremely laggy, dependent on number of linked frameworks, so we also enabled working with test target.
Highly required change!

@beefon nice!!!!

This will help me too. @Ruenzuo please merge this PR

Nice))

Looks great

Thanks a lot for the contribution! I haven't been able to verify this myself but plenty of folks here already did.
I can't prepare a release on RubyGems this week because I'm on holidays but will do as soon as I can!


relayd not adding pf anchors

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
relayd isn't creating the "relayd" anchors in the pf ruleset without rebooting the system; configctl relayd restart/reload do not cause them to be added
To Reproduce
Steps to reproduce the behavior:

Install os-relayd plugin
Configure plugin appropriately
Enable & start plugin
Observe that configctl relayd status and configctl relayd summary show the plugin ought to be working as expected
Observe that the "relayd/*" anchors do not exist in pfctl -s all | grep anchor
Reboot
Observe that the "relayd/*" anchors now exists in pfctl -s all | grep anchor

Expected behavior
relayd would create appropriate pf anchors, without a reboot
Describe alternatives you considered

Checking for errors in configuration
Reloading/restarting relayd plugin

Relevant log files
Before reboot:
Edit (replaced "-s rules" with "-s all"):
root@(redacted):/home/(me) # pfctl -s all | grep anchor
nat-anchor "acme-client/*" all
rdr-anchor "acme-client/*" all
anchor "acme-client/*" all
root@(redacted):/home/(me) #

After reboot:
Edit (replaced "-s rules" with "-s all"):
root@(redacted):/home/(me) # pfctl -s all | grep anchor
nat-anchor "acme-client/*" all
rdr-anchor "acme-client/*" all
rdr-anchor "relayd/*" all
anchor "acme-client/*" all
anchor "relayd/*" all
root@(redacted):/home/(me) #

root@(redacted):/home/(me) # configctl relayd status
osrelayd is running as pid 96097.

root@(redacted):/home/(me) # configctl relayd summary
Id      Type            Name                            Avlblty Status
1       redirect        <IP_ADDRESS> port 80                    active
1       table           haproxy_servers:80                      active (2 hosts)
1       host            <IP_ADDRESS>                   100.00% up
2       host            <IP_ADDRESS>                   100.00% up
2       redirect        <IP_ADDRESS> port 443                   active
2       table           haproxy_servers:443                     active (2 hosts)
3       host            <IP_ADDRESS>                   100.00% up
4       host            <IP_ADDRESS>                   100.00% up

root@(redacted):/home/(me) # cat /usr/local/etc/relayd.conf
# DO NOT EDIT THIS FILE -- OPNsense auto-generated file

interval 5
prefork 3
timeout 200

table <haproxy_servers> {
<IP_ADDRESS>
<IP_ADDRESS>
}

redirect "<IP_ADDRESS> port 80" {
listen on <IP_ADDRESS> tcp port 80
forward to <haproxy_servers> mode least-states  check tcp
session timeout 600
}
redirect "<IP_ADDRESS> port 443" {
listen on <IP_ADDRESS> tcp port 443
forward to <haproxy_servers> mode least-states  check tcp
session timeout 600
}

Additional context
The hardware I'm running is underpowered for ssl proxies with haproxy, so I'm moving the reverse-proxy functionality I currently use to other machines, and trying to forward connections to them using relayd.
Environment
PCEngines APU2, ROM v<IP_ADDRESS>
root@(redacted):/home/(me) # uname -a
FreeBSD (redacted).(domain) 13.1-RELEASE-p3 FreeBSD 13.1-RELEASE-p3 stable/22.7-n250262-83840459d88 SMP amd64

Edit:

os-relayd v2.7
OPNsense 22.7.9

I expect the registration is there, but needs a firewall restart after installation to be used.
https://github.com/opnsense/plugins/blob/16f3522d08d30919b17e66bdec38352ef4c75208/net/relayd/src/etc/inc/plugins.inc.d/relayd.inc#L44-L45
Maybe we could hook this in after plugin restart, I expect a firewall restart (apply in the alias section or firewall) would fix the issue after installation.

So it seems my familiarity with pf on the command line isn't as thorough as I'd thought: I'd incorrectly assumed that "pfctl -s all" would show, well, "all" the things, including rdr rules within relayd's anchors.  It does not.
relayd is redirecting the packets, once the anchor is created - though that still requires a reboot.
I've updated the issue.  Mea culpa.
(On a related note: can someone please tell me what command would list the redirect rules within the relayd anchor and rdr-anchors?)

So it seems my familiarity with pf on the command line isn't as thorough as I'd thought: I'd incorrectly assumed that "pfctl -s all" would show, well, "all" the things, including rdr rules within relayd's anchors. It does not.

The documentation (https://www.freebsd.org/cgi/man.cgi?query=relayd) suggests it does list in pf, but as far as I know it only registers a hook after which relayd injects its own rules not visible to pfctl (table names aren't the same ones either).

(On a related note: can someone please tell me what command would list the redirect rules within the relayd anchor and rdr-anchors?)

relayctl ?

relayctl is useful for showing redirect/relay/session info from relayd, but not the actual pf rules injected into the pf anchors - which I'm beginning to realize isn't really that important to know.


Serving the admin assets separately

Hey,
Is it possible to serve the ember-admin assets separately from the main app - so that regular users don't need to download all the admin specific code?
I'm looking at ways to add admin to my own ember-cli app, I need to share the models between both my app and the admin app, but not sure what the best way is to do it.

The solution is #63