W-15361184 for branch 0624-acb-ascyncapi-duke

closed bc the branch name was wrong,
new PR with 0624-acb-asyncapi-duke will replace it


expose flex property to YogaKit

This exposes yoga property to YogaKit.
The reason why I need the property is demonstrated in https://github.com/axl411/TestYogaKit

why closed?
need it!


Best Actions not consistently pointing to terminal states when using multiple terminal states

Non-terminal state values are going above the values of the terminal states.

Issue is in policy improvement.

Issue was in terminal state rewards being 1.


⚠ Fortnite has degraded performance

In 190cbad, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1609 ms

Resolved: Fortnite performance has improved in a0c6196.


does this work with parsedown packages or is composer required?

See title, I am looking to use markdown on our mediawiki instance. But I don't want to install unnecessary libraries which come with composer. So is it possible to install this extension with the use of php-parsedown package e.g. for debian?

I've never tried using anything other than composer.  You are welcome to try using alternative methods of installing the dependencies needed for this extension, but I can't guarantee they will work.

Hm thats sad to hear, esp. since under requirements on this page its nowhere mentioned that composer is required. Also its mentioned that you could install the parsedown "manually" so I assumed that would be possible.
Also the install instructions here https://www.mediawiki.org/wiki/Extension:WikiMarkdown#Configuration are different from whats shown on this github
I am not sure where the discrepancy stems from.

I'm sure that as long as you have parsedown installed and the extension is able to access it, you will be able to use it, but I've not personally tried that as composer is the preferred method.

Still I am not sure the installation instructions on this github are leading anywhere? By following them I would simple install the parsedown package from my repos and add wfLoadExtension( 'WikiMarkdown' ); to LocalSettings.php
Which probably isn't enough to make this extension work. The previously linked wiki page at least mentions cloning this repo...

You need to download the files for this repo and put them into a folder in your extensions folder for your server just like any other extension install.  The instructions on the wiki page are just more complete.

So for debian there is only one php-parsedown package, I think extended and extra are part of backports, so we will not use them for now. I get this

content drop down when editing pages but when entering any markdown syntax its not showing up. When using  tags on the source code it throws this error when trying to save.

I assume you are referring to the source editor? That is the one I used in the previous comment, where I received the Internal error.

Okay, then can you add $wgShowExceptionDetails = true; to your LocalSettings.php and see if that provides a better error message to diagnose the issue?

`[0cdca3b43efdd41cbe61ea28] /index.php?title=IT:Abteilungsbesprechung_20230427&action=submit Error from line 330 of /var/www/wiki.intern.weizenbaum-institut.de/extensions/WikiMarkdown/includes/WikiMarkdown.php: Class 'Parsedown' not found
Backtrace:
#0 /var/www/wiki.intern.weizenbaum-institut.de/extensions/WikiMarkdown/includes/WikiMarkdown.php(215): WikiMarkdown::getParsedown()
#1 /var/www/wiki.intern.weizenbaum-institut.de/extensions/WikiMarkdown/includes/WikiMarkdown.php(46): WikiMarkdown::parseMarkdown()
#2 /var/www/wiki.intern.weizenbaum-institut.de/includes/parser/Parser.php(3922): WikiMarkdown::parserHook()
#3 /var/www/wiki.intern.weizenbaum-institut.de/includes/parser/PPFrame_Hash.php(330): Parser->extensionSubstitution()
#4 /var/www/wiki.intern.weizenbaum-institut.de/includes/parser/Parser.php(2887): PPFrame_Hash->expand()
#5 /var/www/wiki.intern.weizenbaum-institut.de/includes/parser/Parser.php(1556): Parser->replaceVariables()
#6 /var/www/wiki.intern.weizenbaum-institut.de/includes/parser/Parser.php(651): Parser->internalParse()
#7 /var/www/wiki.intern.weizenbaum-institut.de/includes/content/WikitextContent.php(374): Parser->parse()
#8 /var/www/wiki.intern.weizenbaum-institut.de/includes/content/AbstractContent.php(590): WikitextContent->fillParserOutput()
#9 /var/www/wiki.intern.weizenbaum-institut.de/includes/EditPage.php(4282): AbstractContent->getParserOutput()
#10 /var/www/wiki.intern.weizenbaum-institut.de/includes/EditPage.php(4187): EditPage->doPreviewParse()
#11 /var/www/wiki.intern.weizenbaum-institut.de/includes/EditPage.php(2965): EditPage->getPreviewText()
#12 /var/www/wiki.intern.weizenbaum-institut.de/includes/EditPage.php(701): EditPage->showEditForm()
#13 /var/www/wiki.intern.weizenbaum-institut.de/includes/actions/EditAction.php(71): EditPage->edit()
#14 /var/www/wiki.intern.weizenbaum-institut.de/includes/actions/SubmitAction.php(38): EditAction->show()
#15 /var/www/wiki.intern.weizenbaum-institut.de/includes/MediaWiki.php(527): SubmitAction->show()
#16 /var/www/wiki.intern.weizenbaum-institut.de/includes/MediaWiki.php(313): MediaWiki->performAction()
#17 /var/www/wiki.intern.weizenbaum-institut.de/includes/MediaWiki.php(940): MediaWiki->performRequest()
#18 /var/www/wiki.intern.weizenbaum-institut.de/includes/MediaWiki.php(543): MediaWiki->main()
#19 /var/www/wiki.intern.weizenbaum-institut.de/index.php(53): MediaWiki->run()
#20 /var/www/wiki.intern.weizenbaum-institut.de/index.php(46): wfIndexMain()
#21 {main}
`
I guess the php-parsedown package is not working?

Yes that is indeed the case.  I'm not certain what it would take to get MediaWiki to be able to access Parsedown via php-parsedown.   Composer works well, but if you don't want to use that then I'm not certain what you need to do.

btw, this is kind of a kludge but does work for me to get things running without composer:
in localsettings.php
require_once("<parsedown path>/Parsedown.php");

kludge

Can you elaborate why you describe this solution as a such? I haven't tried it yet, but really would like to get markdown running for our mediawiki without composer.


Windows RecentFileCache parser job

Given that Plaso does not parse RecentFileCache ref, create a job to parse RecentFileCache on Windows machines. https://github.com/EricZimmerman/RecentFileCacheParser
Also need to find a Linux compatible version since link shows it to be compiled as executable/C#: https://github.com/EricZimmerman/RecentFileCacheParser

hi @wajihyassine, This issue seems to be a good initiation of my journey towards contribution to FOSS projects.
Can you please assign this to me?


EF.CompileQuery is not working with predefined static query failing

I have query part defined as static query, to reuse in multiple places of code. When I running static query directly all is working properly, however when I'm trying to use it in EF.CompileQuery it throws exception. Putting the same part directly to compilation part work as expected. But then DRY principle is lost. And I have to repeat myself many times.
Steps to reproduce
Following query:
private static IQueryable<Address> GetBaseQuery(Context db, string searchSql)
            => from address in db.Address
               join search in db.AddressSearch on address.Id equals search.Id
               where EF.Functions.Contains(search.Address, searchSql)
               select address;

Following code works:
int total = await GetBaseQuery(this.Db, searchSql).CountAsync(this.CancellationToken)

This not:
private static Func<Context, string, Task<int>> queryFullSearchCount
   = EF.CompileAsyncQuery((Context db, string searchSql)
   => GetBaseQuery(db, searchSql).Count());

int total = await queryFullSearchCount(this.Db, searchSql);

It throws following exception:
InvalidOperationException: Processing of the LINQ expression 'Count<Address>(GetBaseQuery( db: Context, searchSql: (Unhandled parameter: __searchSql)))' by 'NavigationExpandingExpressionVisitor' failed. This may indicate either a bug or a limitation in EF Core. See https://go.microsoft.com/fwlink/?linkid=2101433 for more detailed information.

    Microsoft.EntityFrameworkCore.Query.Internal.NavigationExpandingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
    System.Linq.Expressions.MethodCallExpression.Accept(ExpressionVisitor visitor)
    System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
    Microsoft.EntityFrameworkCore.Query.Internal.NavigationExpandingExpressionVisitor.ExpandAndReduce(Expression query, bool applyInclude)
    Microsoft.EntityFrameworkCore.Query.Internal.NavigationExpandingExpressionVisitor.Expand(Expression query)
    Microsoft.EntityFrameworkCore.Query.QueryTranslationPreprocessor.Process(Expression query)
    Microsoft.EntityFrameworkCore.Query.QueryCompilationContext.CreateQueryExecutor<TResult>(Expression query)
    Microsoft.EntityFrameworkCore.Storage.Database.CompileQuery<TResult>(Expression query, bool async)
    Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CompileQueryCore<TResult>(IDatabase database, Expression query, IModel model, bool async)
    Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CreateCompiledAsyncQuery<TResult>(Expression query)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledAsyncTaskQuery<TContext, TResult>.CreateCompiledQuery(IQueryCompiler queryCompiler, Expression expression)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryBase<TContext, TResult>+<>c__DisplayClass6_0.<EnsureExecutor>b__0(TContext c, LambdaExpression q)
    Microsoft.EntityFrameworkCore.Internal.NonCapturingLazyInitializer.EnsureInitialized<TParam1, TParam2, TValue>(ref TValue target, TParam1 param1, TParam2 param2, Func<TParam1, TParam2, TValue> valueFactory)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryBase<TContext, TResult>.EnsureExecutor(TContext context)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryBase<TContext, TResult>.ExecuteCore(TContext context, CancellationToken cancellationToken, object[] parameters)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryBase<TContext, TResult>.ExecuteCore(TContext context, object[] parameters)
    Microsoft.EntityFrameworkCore.Query.Internal.CompiledAsyncTaskQuery<TContext, TResult>.ExecuteAsync<TParam1>(TContext context, TParam1 param1)

Further technical details
EF Core version: 3.0
Database provider: (e.g. Microsoft.EntityFrameworkCore.SqlServer)
Target framework: (.NET Core 3.0)
Operating system: Windows 10 x64
IDE: (e.g. Visual Studio 2019 16.3.2)

Notes from triage: currently composition in this way (where a function returning an IQueryable is present in the expression tree rather than everything being part of the same tree) is not supported. We could potentially make this work (for compiled queries and in other places) but it's not something we are likely to do without more feedback. One reason for this is that explicitly compiled queries often don't provide noticeably faster performance than normal, auto-compiled queries.
@msmolka Are you using compiled queries because of a significant performance improvement? If so, can you provide details, including your performance numbers?

Hello,
I'm trying different approach to avoid lag during startup (first run). Sometimes 5 second delay for first entity access is annoying. So I'm trying to find the way to improve first experience. I'm already using pooling, but still first one is longer.

@msmolka You might be experiencing the delay caused by model building. You can access context.Model before performing a query to force the model to build and measure it separately.


Create edu.txt

website of the center
https://www.fernandowirtz.com/publicar.php
IES Fernando Wirtz  881 960 260<EMAIL_ADDRESS><EMAIL_ADDRESS>

Location and list of emails
https://www.fernandowirtz.com/publicar.php?tag=Localización

@cesarbouzas Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Fix build of release notes

Release note build failed in CI, but I think it may work once merged. Reno looks at the actual branch, e.g. stackhpc/xena, instead of the topic branch. It works locally when I commit to stackhpc/xena.

Release note build failed in CI, but I think it may work once merged. Reno looks at the actual branch, e.g. stackhpc/xena, instead of the topic branch. It works locally when I commit to stackhpc/xena.

Seems like a bug - shouldn't it use the PR branch if we want to test added renos?

Release note build failed in CI, but I think it may work once merged. Reno looks at the actual branch, e.g. stackhpc/xena, instead of the topic branch. It works locally when I commit to stackhpc/xena.

Seems like a bug - shouldn't it use the PR branch if we want to test added renos?

Possibly Zuul does something different to allow generating release notes during CI jobs?
Also we may be hitting limitations of Reno's design, since we don't seem to follow its expected workflow: https://docs.openstack.org/reno/latest/user/design.html#assumptions


'Forbidden' error when creating Immutable secrets

Describe the bug
I'm trying to create a an ExternalSecret with data coming from a password generator and using a template. As we don't want this password to be refreshed, we set the spec.target.immutable attribute to true. However, whenever I create it, the controller reports the following error
Status:
  Binding:
    Name:  es-5-test
  Conditions:
    Last Transition Time:   2024-06-19T05:40:30Z
    Message:                could not update Secret
    Reason:                 SecretSyncedError
    Status:                 False
    Type:                   Ready
  Refresh Time:             2024-06-19T05:40:30Z
  Synced Resource Version:  1-afdbc4873f1d1bdfd2885d1d7b0e682b
Events:
  Type     Reason        Age                From              Message
  ----     ------        ----               ----              -------
  Normal   Updated       28s                external-secrets  Updated Secret
  Warning  UpdateFailed  1s (x11 over 28s)  external-secrets  Secret "es-5-test" is invalid: data: Forbidden: field is immutable when `immutable` is set

If we set creationPolicy: Orphan then we don't have this issue but the secret is not recreated whenever it is deleted by accident.
To Reproduce
Kubernetes version: 1.27.13
ExternalSecrets version: v0.9.16
This is the ExternalSecret manifest
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: es-5-test
spec:
  dataFrom:
  - sourceRef:
      generatorRef:
        apiVersion: generators.external-secrets.io/v1alpha1
        kind: Password
        name: password-generator
  refreshInterval: 0m
  target:
    immutable: true
    name: es-5-test
    template:
      data:
        password: '{{ .password }}'
        username: someone

For the sake of completeness, here's the Password's manifest:
apiVersion: generators.external-secrets.io/v1alpha1
kind: Password
metadata:
  name: password-generator
spec:
  allowRepeat: true
  length: 16
  noUpper: false
  symbolCharacters: ~!%^&*()_+-={}|[]\<>?,./

Expected behavior
Secret is created with immutable: true and ExternalSecret conditions is set to SecretSynced.
Screenshots
not applicable
Additional context
Add any other context about the problem here.
Many thanks in advance for your support!

Not a bug, this is intentional behaviour.
Remove the immutable field and set refreshInterval to 0, that should do.

Hi @moolen,
Thanks for your prompt reply!
If we only set the RefreshInterval to 0, there's still a risk that the secret is refreshed if, for example, labels or annotations are updated on the source ExternalSecrets object. In our case, since we intend to use ArgoCD and Helm to deploy those secrets, it could happen that labels or annotations are updated while we don't expect the secret to be refreshed.
We wanted to use the immutable flag to have the guarantee that the secret is not updated, the shouldReconcile function from the external secrets controller would then skip reconciliation.
I don't know if it would make more sense to look at the shouldRefresh function from the external secrets controller and return false if only non "critical" attributes from the ExternalSecret object are updated. But then we need to find another way to manually trigger a secret refresh.

I was able to reproduce the issue, though unfortunately it appears to be a race condition. The .status.condition is empty for some reason and thus shouldReconcile() does not return false to stop reconciliation. It does work 4/5 times on my machine :tm:, so if we resolve that race everything should work as you expect.

Weird stuff happening there :raised_eyebrow:. I've found a workaround. Not pretty, but it seems to work reliably.
See #3608. Can you verify if that works for you?
There's a image available from the PR:
ghcr.io/external-secrets/external-secrets:v0.9.19-38.ga963796b@sha256:1b5c9bc38fc0ae1557772d395a9e2bb0a0a660d7fd84f7817575b3d0044151aa

Waw, that was fast, Thanks much!
I will check with my SRE team to deploy the PR image on our test environment and I'll get back to you with the outcome of my tests.

Hi,
I forgot to come back and close this issue. Sorry about that! Thanks again for the support


regression-test temporarily comment out unstable stats p0

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

closed because the same pr #19729


updated rules_mapping to include spike_aggregation

Was getting message "elastalert.util.EAException: Error loading file rules/apache_busy.yaml: Could not import module spike_aggregation: not enough values to unpack (expected 2, got 1)". Adding rule to mapping fixed issue.

I know you are probably busy, but could you look into this fix :
@pdscopes @Qmando @danielpops


TAS: Respect node taints if the lowest level is node

What type of PR is this?
/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
Part of #3658
Special notes for your reviewer:
Preparatory PR: https://github.com/kubernetes-sigs/kueue/pull/3676
Does this PR introduce a user-facing change?

Respect node taints in Topology-Aware Scheduling when the lowest topology level is kubernetes.io/hostname.

rebased after https://github.com/kubernetes-sigs/kueue/pull/3676

Thank you
/lgtm
/hold to address nits


Update content.cljs

Christina: I have received actual figures from NICOR so have updated the 2015 and 2014 tables. I have also added in the 2010-13 data as well.

If we want the year to be selectable, then we'll need to decide where is a sensible place to put the selection control on the web. The current site has no options controls in the footer - it was there because I didn't want people finding it and playing. If we do want people to choose the year they are looking at, then we will need a new menu somewhere global and noticeable. We'll also need to make it very obvious what year the site is pointed at.

Need to:

identify areas of the site that is year dependent
create slots for possible future texts - e.g. to cope with out-of-range instances
define a current year default


Do not run publish step every time

Description:
CI runs publish step everytime and it fails everytime.

What do you mean by "everytime"?  Do you mean on every push?  I'm not seeing it on the latest push.

On every push to develop: https://circleci.com/workflow-run/c01fa5c9-8b35-495f-a049-36a053396188

Still doing it: https://circleci.com/gh/cosmos/voyager/6844


[Bug report]: error occurred after update or something.

Platform
Desktop
Version
1.7.3
Description
App Version: 1.7.3
platform: macOS
platformVersion: 10.15.7
platformArchitecture: x64
cpuCount: 4
totalMem: 8192.0 MB
freeMem: 527.2 MB
Error Type: [Preload Context] Error
Error: node-loader:
Error: dlopen(/Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node, 1): Symbol not found: __ZNSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE7seekoffExNS_8ios_base7seekdirEj
Referenced from: /Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node (which was built for Mac OS X 13.0)
Expected in: /usr/lib/libc++.1.dylib
in /Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node
Expected behaviour
be able to access the application after update.
Actual behaviour
received the attached error message after upgrade
Can the issue reliably be reproduced?
No
Steps to reproduce the issue
No response
Operating System
Mac
Diagnostics
No response
Developer Profile
No
Profile Type
Software (Stronghold)
Staking
Actively staking
Crash Status
None of the above
Error Messages
App Version: 1.7.3
platform: macOS
platformVersion: 10.15.7
platformArchitecture: x64
cpuCount: 4
totalMem: 8192.0 MB
freeMem: 527.2 MB
Error Type: [Preload Context] Error
Error: node-loader:
Error: dlopen(/Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node, 1): Symbol not found: __ZNSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE7seekoffExNS_8ios_base7seekdirEj
Referenced from: /Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node (which was built for Mac OS X 13.0)
Expected in: /usr/lib/libc++.1.dylib
in /Applications/Firefly.app/Contents/Resources/app.asar.unpacked/public/build/index.node
Error Logs
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closed by https://github.com/iotaledger/firefly/pull/7332, we will release a new version soon

Hi, we just released the new version 1.7.4 which contains the fix
Firefly IOTA 1.7.4 announcement message
We just released Firefly IOTA 1.7.4. Among the changes, it contains the fixes for macOS crash at startup and the slow stronghold export.
:warning: Important: macOS 10.13 will not be supported anymore in Firefly IOTA, so if you are running macOS 10.13 we highly recommend upgrading your macOS version to a newer one.
:pencil: Changelog:

fix slow stronghold export
fix macOS support (official drop support macOS <= 10.13)
add official asmb staking event 6 results

:arrow_down:  Download:
https://firefly.iota.org/
:bug:  Bug Reporting:
https://github.com/iotaledger/firefly/issues/new?assignees=&labels=bug+report&template=bug_report.yml&title=[Bug+report]%3A

@DanMClark yes we just released the new firefly iota app (version 1.7.4)


support more complex accessors for fields

currently by default a simple: row[column] is used. It should support more advanced accessors, e.g. column='arr[5]' or column='ueber.x'

part of latest release_1 branch


One kind of Gaussian field mock

For Gaussian field code testing, what I'd really like is something that simply takes a random catalog and multiplies weights by 1+delta. I'm sure I could dig into code and figure out how to do it, but if you can provide an example...

I have just added an example on top of this notebook (a2251d0b8a003ec2015ac0788b9ff1aa4cb2a396)


test: Add elm-format job to travis ci configuration

This runs elm-format inside a docker container serving the Elm
environment. Afterwards we check with git if a file changed and fail
accordingly.

Ready to merge if you don't have objections.

Maybe we should try to compile the project as well.

I commented that we should run elm make and see if that passes, but actually the make command was already there. I'm not sure if it calls elm make though.

@stuartnelson3 added --validate flag
@w0rm added elm-make job
@stuartnelson3 @w0rm Please rereview.
Just as a heads up, make sure you update your local elm-format version to the newest 0.6.0-alpha version. Otherwise the CI will fail on your changes.


Problem with Side Navigation when switching View without using Nav Items from SideMenu

Hi,
at first I want to thank you for your great work. It saved me a lot of time :) ..
I've got following setup:

A "source" View Controller embedded in the NavigationController, 2 MenuItems --> 2 View Controllers
A Button, OnTouch: Shows a third independent View Controller (not related to the Menu Items / Navigation Controller)
On the third View I've put on a button redirecting to the "source" View Controller

I've got following issue: Whenever I am returning to the "source" view form the independent (3.) View Controller, the side menu items don't work anymore. The side bar still works well but whenever I click on a menu item it won't show up the corresponding View.
The return Segue (from the 3. View Controller to the "source" View Controller) is connected to the Navigation Controller.
What am I missing?
Thank you very much

I have the same problem, you managed to solve?

@jdoes unfortunately not :( ... I managed to create a sidemenu with following tutorial: http://www.raywenderlich.com/78568/create-slide-out-navigation-panel-swift
At the bottom of the page there are the project source files to start right away.
Regards

Hi, thanks for this great work but I also have this problem...

Hi @fsad , sorry for long response.
What kind of segue do you use to return to the first VC? It probably creates a new instance of navigation controller. I would suggest don't use segue and make it programmatically.

Hi Evnaz,
I personally use a modal segue as the return segue

Hi guys,
same problem here.
@evnaz could you provide an example of doing it programmatically or maybe do a sample project to show the right way to do it?
I am using as the others segues and unwind segues and the menu disappears.

Hi @robomotic ,
Unfortunately i cant reproduce the issue, can you please post basic project with view controller's logic causing the problem.

Hi @evnaz ,
will do give me a couple of hours, I am cloning yours on mine and adding the example.
Cheers.

Here we go the demo project is here:
https://github.com/robomotic/ENSwiftSideMenu
You can see how the Menu disappears after the segue transition from ViewController to ViewController3.
I also added an unwind segue with the Back button (I couldn't do with the left swipe because that of course should trigger the menu).
Also it will be nice to have a navigation control in this case.
Let me know what you think.
Cheers!

I checked your example and noted that ViewController3 is a modal view controller, it is outside of the navigation view controller's stack, as you see in the RightSegue.swift it inserts as a subview of current window:
window?.insertSubview(secondVCView, aboveSubview: firstVCView)

The side menu can't handle gestures over the inserted view. So, side menu only work if you use a push(show) segue. To prevent back action on swipe you can disable back swipe gesture by:
override func viewWillAppear(animated: Bool) {
        navigationController?.interactivePopGestureRecognizer.enabled = false
}

At the moment i don't see any way how to manage modal view controllers by side menu.

So I made a commit in the RightSeque.swift to use the push instead of mode:
-        let window = UIApplication.sharedApplication().keyWindow
-        window?.insertSubview(secondVCView, aboveSubview: firstVCView)
+        //let window = UIApplication.sharedApplication().keyWindow
+        //window?.insertSubview(secondVCView, aboveSubview: firstVCView)
+
+        let navigat = UINavigationController()
+        navigat.pushViewController(self.destinationViewController as! UIViewController, animated: true)

But I am still unable to see the Menu,is it to do with the UINavigationController that doesn't get initialised with the one of the Menu?

Hi @robomotic ,
You don't need to use custom segue to push a view controller, just use show(push) segue.
And, yes, you create a new instance of navigation controller and then just present the destination controller as a modal view, in that case just created nav controller is not used.
Also remove this function or modify the return value in the ViewController3:
func sideMenuShouldOpenSideMenu() -> Bool {
        println("sideMenuShouldOpenSideMenu")
        return false
}


Troubleshooting

It would be useful to add some troubleshooting tips in the event that this tutorial isn't working (it isn't for me)! For example mentioning running 'iotedge list' on the edge device to confirm that the modules are deployed and their status, then 'iotedge logs <module>' to try and help determine the nature of the error.
Tutorial modules are deploying OK but don't run successfully. If you've got any ideas why the tutorial modules are failing with 'AuthenticationException: The remote certificate is invalid according to the validation procedure.'  (Ubuntu 16.04). I would be pleased to hear them!

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

ID: a7852968-3c54-5584-df91-b01521efae43
Version Independent ID: c7340985-f6d2-2bf8-71e5-ee62dadb25e3
Content: Deploy Azure functions with Azure IoT Edge
Content Source: articles/iot-edge/tutorial-deploy-function.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@tsorfs Thanks for the feedback. Are you aware of the following documentation: Common issues and resolutions for Azure IoT Edge. I want to make you aware that such a document does exist.
This is also good to know: Azure IoT Edge Support
It terms of your specific issue, it sounds as if the security module did not get deployed correctly. Have you attempted a simple stop and start of the services?

Thanks Mike. I hadn't spotted the Troubleshooting doc, but probably should have considering it's further down the 'How To' tree.
'Security module was fine, had already tried restarts. Scrapping the first cut and building a new Hub device seems to have improved the situation.


Request for deeper linking to some pages

Tabs pages for example generate #tab links as you navigate through tabs and could be used to directly open a tab...
Requested by Brane Šmitek on [Xerte] mailing list...

Added in https://github.com/thexerteproject/xerteonlinetoolkits/commit/54b3192db1682a470d836d86fce3959d89210c05


Update hijacker.py

So that we don't need to set the HIJACKER_EMAIL_BACKEND if we are using the default backend.

Wow, sorry James, I just saw this. If you're still interested, that'd be good to add EMAIL_BACKEND = settings.EMAIL_BACKEND in defaults.py after importing the django settings


Removals and Updates

Removals:

Rig Veda / 404
Vedic Society / 404
Fun Fact / Redirects to domain parking
ZipRecruiter / Appears to no longer have an API (if you are from US, please try to confirm)

Updates (I assume this is okay, the contributing.md is not entirely clear):

Wandbox / corrected link to API docs, was 404
Transport for Berlin / updated from v3 to v5
Transport for Germany / corrected link

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not have more than 100 characters
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Thank you!


regenerate boto client when credentials are None

this prevents flaky metadata service from permanently poisoning the Table and its Connection

:+1: thank god

+1 :smile:

cherry picked into https://github.com/lyft/PynamoDB/releases/tag/<IP_ADDRESS>


Object reference not set to an instance of an object on Facebook/Twilio channels

A bot created in composer which only replies to an unknown intent and greets a user runs just fine via facebook and Twilio.  However if you add any complexity to it such as prompting for a question or adding QnA, it reponds once with Object reference not set to an instance of an object and then never responds again.  I'm using the azure web app option for publishing.  Even publishing the default echo bot creates this issue over Facebook and Twilio.  The bot still works just fine doing a web chat test as well as locally testing.
I asked on this issue: https://github.com/microsoft/BotFramework-Composer/issues/4073 whether it was related and it just effects channels other than the direct line channel.
I'm using composer 1.1.1 which is using 4.10.2 SDK on macOS

@luhan2017 can you please take a look at this?

@crebstock，could you please check if the locale is set in the facebook and Twilio channels? it could relate to this issue #4073, and we already have a fix on the sdk side. You can try to set the locale to mitigate it for now.

@crebstock，could you please check if the locale is set in the facebook and Twilio channels? it could relate to this issue #4073, and we already have a fix on the sdk side. You can try to set the locale to mitigate it for now.

Sorry @crebstock, if this is the case I was incorrect on https://github.com/microsoft/BotFramework-Composer/issues/4073

Where would you edit locale specific to the channel?  I don't see those settings either in the channel registration?  Would this be done in composer or is it a setting on facebook/twilio?

Just an head up as we're experiencing the exact same issue.

Today we tried Telegram channel as a fallback. The same issue arise.
To repro the issue you can try to deploy the template "Todo with LUIS".

Where would you edit locale specific to the channel? I don't see those settings in the channel registration. Would this be done in composer or is it a setting on facebook/twilio?

It should be a setting on facebook/twilio.

Telegram

Did you set the locale for Telegram channel? you can do this to mitigate this issue. We will have a patch release for this soon.

@luhan2017 thank you for your feedback. Can you please elaborate how to set the locale and on which entity?
I was unable to find any setting that reseamble what you indicate on Bot Composer or in the Azure Portal. Thank you!

Seeing this as a stale issue, is there something we (the community) can actionate to progress this issue?
We we'll be keen to arrange a PR if any hints or insights are provided.

Hi @nitroin - @luhan2017  has been on holiday and returning tomorrow. Thank you for your patience.


Add show or hide Labels

great good idea thx for providing this PR

I have renamed option to 'limits'

Ок. no problem! )))))

options was there but not yet implemented...;)

i modify locale code
          scope.options.limits = scope.options.limits || true;

to
if (angular.isDefined(scope.options.limits)) {
   scope.options.limits = true;
} else
{
   scope.options.limits = false;
}

because scope.options.limits always true

how can it be always true ?

If I'm not mistaken,
scope.options.limits = scope.options.limits || true;  =
bool(scope.options.limits) === false;
But I could be wrong!
I always true

my mistake..


[sw/silicon_creator] Defining manifests in assembly may cause issues if boot stages access their own manifests

#6727 replaces the MMIO-based manifest parser with a struct and adds manifest.S for defining the manifest sections of boot stage images stored in flash. The main reason to use an .S file was to ensure that the sizes of the manifests of boot stage images and the size of the corresponding struct defined in C match. @mcy raised a concern that

this generates a TU that LLVM could potentially look at, and perhaps optimize all manifest accesses out to zero.

While boot stages don't access their own manifests right now, we should be aware of this potential issue if this changes in the future.

Closed by #11990


Add a tippy tooltip to inline message imaes, as well as the lightbox title.

Commit 1:
This commit adds a tippy tooltip for inline image previews in messages.
There exists some (reasonable) logic in static/js/util.js which
overrides all title attributes for links to user-uploaded content to
ensure they always display "Download ". This doesn't make
sense for inline images specifically because they will be opened in a
ligthbox, so we prevent that.
There is an additional tippy instance created in static/js/tippyjs.js
to add tippy tooltips to inline images, which takes advantage of the now
preserved title attribute of the parent link.
Commit 2:
This commit adds a tippy tooltip to the lightbox title which enables the
user to view the filename of an image if the filename is different than
the image title.
Fixes: https://github.com/zulip/zulip/issues/21333

Nice, looks pretty good!
For the message feed tooltip when the name is blank, I wonder if it would be better to say "View or download <file name>" rather than "View or download attachment"? It would be more consistent with the lightbox as well.

No other feedback on my end.

For the message feed tooltip when the name is blank, I wonder if it would be better to say "View or download <file name>" rather than "View or download attachment"? It would be more consistent with the lightbox as well.

The only case I can find that this occurs is with youtube videos, and possibly some other embedding objects - so there is no filename to use. If we wanted to get fancy we could probably do some plumbing so it could say "view or download youtube video" or something like that. Might be more complexity than we need though.

@timabbott I've implemented your requested changes. Let me know if this looks cleaner to you.

The only case I can find that this occurs is with youtube videos, and possibly some other embedding objects - so there is no filename to use. If we wanted to get fancy we could probably do some plumbing so it could say "view or download youtube video" or something like that. Might be more complexity than we need though.

Hm, it certainly happens if you manually delete the name for the file you uploaded (the [] part)-- that's what I encountered in manual testing, and where I was proposing using the name of the file instead.
"attachment" seems fine in the situation you describe, where there is no name at all. Am I remembering right that it was bold when I tested it? If so, I would remove that formatting.

Hm, it certainly happens if you manually delete the name for the file you uploaded (the [] part)-- that's what I encountered in manual testing, and where I was proposing using the name of the file instead.

Ah! I see, I did not test what would happen if you manually remove the title. I'll see what I can do for that case.

"View or download attachment" feels off to me. One option is this, where we show the URL where the word "attachment" appears; that feels acceptable to me for both the Youtube video case and also the "generic link to image" case.
diff --git a/static/js/tippyjs.js b/static/js/tippyjs.js
index 73d1fab727..093d0419b6 100644
--- a/static/js/tippyjs.js
+++ b/static/js/tippyjs.js
@@ -258,7 +258,7 @@ export function initialize() {
         onShow(instance) {
             // Some message_inline_images aren't actually images with a title,
             // for example youtube videos, so we default to a generic title
-            const title = $(instance.reference).parent().attr("aria-label") || "attachment";
+            const title = $(instance.reference).parent().attr("aria-label") || $(instance.reference).parent().attr("href");
             instance.setContent(parse_html(render_message_inline_image_tooltip({title})));
         },
         onHidden(instance) {

@timabbott I've implemented the change you suggested for the multi-lime tooltip, I think it looks really good.
@alya You can see at the end of the recording an example of an image where the title was completely erased.
https://user-images.githubusercontent.com/3046397/159365084-755113c9-6237-4aee-b16c-cde72f1e0f78.mp4

The new tooltip design looks great to me, as does the implementation.
I think my only outstanding question is whether we want Click to view or download to end with a . or not.

Yeah, looks great in the video!
For a full sentence like "Click to...", having a period makes sense to me. If it were just "View or download", I think not using a period would be consistent with our other tooltips.

I added a period after "Click or view to download".

Great, marked to merge once CI finishes. Thanks for making this improvement happen!


S610: false positive on unrelated method named extra

main.py:
class A:
    def extra(self, foo): ...

A().extra(1)

> ruff --version
ruff 0.5.4

> ruff check main.py
main.py:5:10: S610 Use of Django `extra` can lead to SQL injection vulnerabilities
  |
5 | A().extra(1)
  |          ^^^ S610

See #1646 and #10316

Thanks for reporting. Unfortunately, this is hard for ruff to get right at the moment without having type inference. But we're working on it.

I agree with @MichaReiser. I'd also note that erring on the side of false positives is probably the correct tradeoff when it comes to a security-related rule. I think you'd much rather have something be incorrectly flagged as a security issue than have some dangerous code accidentally slip through the net.


useCookies is not a function or its return value is not iterable

Describe the bug
useCookies is not a function or its return value is not iterable
To Reproduce
Steps to reproduce the behavior:
fresh pull from 'master' branch and yarn install.
Device (if applicable):

OS: macOS 10.15.6
Browser Chrome
Version 89.0.4389.82 (Official Build) (x86_64)

Additional context
Error while server-rendering. date: Mon Mar 15 2021 18:38:19 GMT+0530 (India Standard Time) url: [object Object]
TypeError: useCookies is not a function or its return value is not iterable
at useLocaleData (packages/vulcan:core/lib/modules/containers/localeData.js:29:21)
at withLocaleData (packages/vulcan:core/lib/modules/containers/localeData.js:43:22)
at processChild (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/react-dom/cjs/react-dom-server.node.development.js:3204:14)
at resolve (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/react-dom/cjs/react-dom-server.node.development.js:3124:5)
at ReactDOMServerRenderer.render (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/react-dom/cjs/react-dom-server.node.development.js:3598:22)
at ReactDOMServerRenderer.read (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/react-dom/cjs/react-dom-server.node.development.js:3536:29)
at renderToStaticMarkup (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/react-dom/cjs/react-dom-server.node.development.js:4261:27)
at /Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/@apollo/client/react/ssr/ssr.cjs.js:81:21
at new Promise ()
at process (/Users/vipin.lahoti/Documents/meteor/Vulcan/node_modules/@apollo/client/react/ssr/ssr.cjs.js:79:16)
at /Users/vipin.lahoti/.meteor/packages/promise/.<IP_ADDRESS>xavwef.9sbp++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/meteor-promise/fiber_pool.js:43:40
=> awaited here:
at Function.Promise.await (/Users/vipin.lahoti/.meteor/packages/promise/.<IP_ADDRESS>xavwef.9sbp++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/meteor-promise/promise_server.js:56:12)
at packages/vulcan:lib/lib/server/apollo-ssr/renderPage.js:61:7
at /Users/vipin.lahoti/.meteor/packages/promise/.<IP_ADDRESS>xavwef.9sbp++os+web.browser+web.browser.legacy+web.cordova/npm/node_modules/meteor-promise/fiber_pool.js:43:40

It sound like you need to add the react-cookie NPM package to your project

Yeah the starter has been updated with this, it's always the reference (there is no known way to keep them in sync sorry, and there were no libraries to do that last time I checked)


ci: combined build and deployment steps

Signed-off-by: Jan<EMAIL_ADDRESS>

We should integrate the build option for workflow dispatch:
  workflow_dispatch:
    inputs:
      build:
        default: true
        type: boolean
        required: false
        description: Build the website before deploying

and
        if: inputs.build == true

Is this the correct syntax?
if: inputs.build == true && github.ref == 'refs/heads/main'`?

And will this only check the needs if build is true?
if: inputs.build == true && github.ref == 'refs/heads/main'
needs: [build-and-push-image-server, build-and-push-image-client]

And will this only check the needs if build is true?

Good question.  I think it will work ?? ❓


Failed to load resource: the server responded with a status of 503

Precondition
The "Home" page is displayed
Steps to reproduced:

Open the "Statistics" page
Pay attention to the  "Application error"/check the error in the console

Actual result: The "Statistics" page isn't displayed and the "Failed to load resource: the server responded with a status of 503" error message is displayed in the console
Expected result: The "Statistics" page should be displayed
Checked on QA V0.24.1-3
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.101
Desktop: Windows 10 // Google Chrome  83.0.4103.106

Just the GUN is down again on QA. I couldn't restart it as still have no accesses to its dyno in the Heroku

@AnastasiiaOdnoshevna @serdiukov-o-nordwhale
the gooddashboard for qa heroku server was with old configuration. restarted it.

STATISTICS is displayed
Checked on QA V0.24.1-3
Desktop: Windows 10 // Google Chrome  83.0.4103.106


FATAL EXCEPTION: Timer-0

After update to Flutter, I am getting this error when I am trying to run a function in my application.
E/AndroidRuntime( 9793): FATAL EXCEPTION: Timer-0
E/AndroidRuntime( 9793): Process: org.heartchurch.heartland, PID: 9793
E/AndroidRuntime( 9793): java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: Timer-0
E/AndroidRuntime( 9793): 	at io.flutter.embedding.engine.FlutterJNI.ensureRunningOnMainThread(FlutterJNI.java:605)
E/AndroidRuntime( 9793): 	at io.flutter.embedding.engine.FlutterJNI.dispatchPlatformMessage(FlutterJNI.java:515)
E/AndroidRuntime( 9793): 	at io.flutter.embedding.engine.dart.DartMessenger.send(DartMessenger.java:76)
E/AndroidRuntime( 9793): 	at io.flutter.embedding.engine.dart.DartExecutor.send(DartExecutor.java:166)
E/AndroidRuntime( 9793): 	at io.flutter.view.FlutterNativeView.send(FlutterNativeView.java:155)
E/AndroidRuntime( 9793): 	at io.flutter.plugin.common.MethodChannel.invokeMethod(MethodChannel.java:98)
E/AndroidRuntime( 9793): 	at io.flutter.plugin.common.MethodChannel.invokeMethod(MethodChannel.java:84)
E/AndroidRuntime( 9793): 	at live.exit.musicplayer.MusicPlayer$play$1.run(MusicPlayer.kt:49)
E/AndroidRuntime( 9793): 	at java.util.TimerThread.mainLoop(Timer.java:562)
E/AndroidRuntime( 9793): 	at java.util.TimerThread.run(Timer.java:512)

 info • This class (or a class which this class inherits from) is marked as '@immutable', but one or more of its instance fields are not
          final: PrayerSubmit.things • lib/pages/prayerSubmit.dart:5:7 • must_be_immutable
   info • Avoid using braces in interpolation when not needed • lib/pages/prayerSubmit.dart:30:151 • unnecessary_brace_in_string_interps
   info • The value of the local variable 'sendReports' isn't used • lib/pages/prayerSubmit.dart:32:11 • unused_local_variable

[✓] Flutter (Channel beta, v1.6.3, on Mac OS X 10.14.4 18E226, locale en-US)
    • Flutter version 1.6.3 at /Users/Brandon/Desktop/Programming/DEV/flutter
    • Framework revision bc7bc94083 (2 weeks ago), 2019-05-23 10:29:07 -0700
    • Engine revision 8dc3a4cde2
    • Dart version 2.3.2 (build 2.3.2-dev.0.0 e3edfd36b2)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/Brandon/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Users/Brandon/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/183.5522156/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2, Build version 10E125
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.4)
    • Android Studio at /Users/Brandon/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/183.5522156/Android
      Studio.app/Contents
    • Flutter plugin version 35.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.1.3)
    • IntelliJ at /Users/Brandon/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 35.3.3
    • Dart plugin version 191.7479.14

[✓] VS Code (version 1.33.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.0.1

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

after flutter upgrade, I am also getting the same error
E/AndroidRuntime: FATAL EXCEPTION: Timer-0
Process: x.x.x.x.x, PID: 9178
java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: Timer-0
at io.flutter.embedding.engine.FlutterJNI.ensureRunningOnMainThread(FlutterJNI.java:794)
at io.flutter.embedding.engine.FlutterJNI.dispatchPlatformMessage(FlutterJNI.java:684)
at io.flutter.embedding.engine.dart.DartMessenger.send(DartMessenger.java:80)
at io.flutter.embedding.engine.dart.DartExecutor.send(DartExecutor.java:189)
at io.flutter.view.FlutterNativeView.send(FlutterNativeView.java:155)
at io.flutter.view.FlutterView.send(FlutterView.java:750)
at io.flutter.plugin.common.MethodChannel.invokeMethod(MethodChannel.java:98)
at io.flutter.plugin.common.MethodChannel.invokeMethod(MethodChannel.java:84)

Hello @BND10706 !
Does the problem still persist in the latest stable version?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


gpu: Blocking for GPU offloading

This PR aims to enable tiling in GPUs codes both with OpenMP and OpenACC.

in light of recent discussions on slack, this needs some work before being considered ready for merge


[eventhub] Failing build:samples scripts

Sample Quality Effort
This issue was created because one or more packages in the eventhub service directory had failing build:samples NPM scripts, and I determined that the existing script was broken and could not work, or would take extra effort to repair. #12715 (Build Samples in PR CI) will replace the broken script with a placeholder that simply executes echo Skipped..
The packages in question are:

@azure/eventhubs-checkpointstore-blob
@azure/event-processor-host

Usually, these errors were caused by faulty assumptions, such as:

a script that executes cd samples && tsc -p ., but with no tsconfig.json or other required files in the samples folder
use of old samples scripts that no longer exist
use of new samples scripts (dev-tool) without a correct samples folder layout (e.g. missing typescript subfolder)

What do I do about this?

If the package is not customer-facing, and there are not intended to be any samples in the package (such as internal code-sharing packages): leave a comment and close this issue.
If there are currently no samples, but the package is customer facing and you intend to add samples eventually: create a tracking issue for samples if one does not exist already and close this issue.
If there are samples, and they should build, but the current scripts are not correct, either:
a. Create an issue to fix the errors and wait until #12715 is merged, then incorporate your changes.
b. Work with me to incorporate the necessary changes into #12715.
If you believe this is an error, that the package is correctly configured, and that the script should work: leave a review with a request for changes on #12715 describing how to run the samples, so that we can reconcile the build scripts with the packages' sample programs.

@azure/event-processor-host fixed by https://github.com/willmtemple/azure-sdk-for-js/pull/4
@azure/eventhubs-checkpointstore-blob fixed by https://github.com/willmtemple/azure-sdk-for-js/pull/3
Both should be fixed in master as part of #12715

Both PRs were merged into the larger PR #12715


Add migration from delegated to "owned" mode

In dashboard in delegated mode show big red "Migrate to own your site" button:

shows a modal with explanation that "sign up with OTP makes your site signed by our own keys, you need to migrate to use your own keys to fully own your site". Next we should explain how to use nsec.app to store your keys and login (screenshots?), url /admin/siteId/own
then user clicks "continue" we go to /admin/siteId/own/ready and show "Please log in with your keys" text and show nostr-login modal with "log in" screen, after user relogins the screen auto-reloads and the text on this page changes to "Migrate your site XXX to your own keys", and "Migrate" button
click on "Migrate" does:
copy site event, change pubkey to user's, sign, publish
call /migrate?from=oldSiteId&to=newSiteId with "to" site id of newly signed event - it will re-render the new site to the same cdn directory
call DELETE /site?site=oldSiteId to make server delete the old site event
when all done we show "Success" text and "Back to dashboard" button which should open /admin/newSiteId/dashboard page

/migrate is TBD in nostr-site-cli

Should be fixed by https://github.com/nostrband/nostr-site-manager/pull/127

tested on feature/otp-migrate. Fixed.


Merge PR #2 and #3 and add compatibility with Theano and CNTK backend

This PR is a merge of my last two PR (#2 and #3)
add compatibility with Theano and CNTK

I ran a small test overnight on the ptb dataset with the small configuration, --aug and --tying options. Results after 15 epochs are :

val_acc: 0.2045
val_perplexity: 193.1504

Thank you for your pull request! Could you let me some check?
About TimeDistributed
You remove the TimeDistributed in the one_hot_model.py but it is necessary for many-to-many(2) type network.

Ordinary implementation of the language model is as follows (Chainer/TensorFlow/PyTorch etc). To implements this, we have to use stateful LSTM, and it is difficult (I tried to do this and you can see its result).

About cross_entropy order
In the K.backend, the argument order is pred, true. You can confirm it at here.
But it is so confusable so the fix is added recently. But it is not deployed to PyPI (will be included 2.0.7).
I want to fix this order so I'll merge your request after above fix is available.
PerplexityLogger is a good idea! I want to use it in another project.

About TimeDistributed
I thought that TimeDistributed(Dense()) was needed when keras Dense layer could only be applied to rank 2 tensor. So that to my understanding when applying to a rank 3 tensor :

Dense == Conv1D(kernel_size=1) = TimeDistrbuted(Dense())

And that's what the said [here]](https://stackoverflow.com/questions/42755820/how-to-use-return-sequences-option-and-timedistributed-layer-in-keras#comment72635994_42758532). However I don't understand the interaction with statefulness of LSTM.
About cross_entropy order
Good catch! I was using the master version and didn't pay enough attention...
About stateful LSTMM
Because you used sequence size of 20, it means that you are doing BPTT up to 20 timesteps which seems enough. But this help to better initialize the LSTM cell state.
Some research actually use Conv1D to do language modeling, which is faster because easily parallelizable. And it is compatible with weight tying and augmented loss, so I want to test it when I have the time.


Add hpx::components::tuple_space

Implement a new (distributed) component hpx::components::tuple_space with the following characteristics:

[x] Expose an interface similar to Linda tuple spaces
[ ] Expose additional interface to allow for registering of event handlers
[ ] Distributed data store
[x] Connect (optionally) to persistent distributed storage (OrangeFS)

Moving this to the next milestone to have more time for integration with the distributed unordered_map.


Null check operator used on a null value

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Hi, everyone! I'm using firebase crashlytics to record errors. And I often getting "Null check operator used on a null value" in setstate.

In the image above, crashlytics saying on line 52, causing  "Null check operator used on a null value". I can't understand. Why it's happening
Expected results
There should not be any logs like that. Because there any null operator
Actual results
Causing "Null check operator used on a null value" issue
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.3.1 22E772610a darwin-x64, locale en-KG)
    • Flutter version 3.7.12 on channel stable at /Users/tologonkudaiberdiuulu/tools/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (3 weeks ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc1)
    • Android SDK at /Users/tologonkudaiberdiuulu/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.3.1 22E772610a darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 111.0.5563.146

[✓] HTTP Host Availability
    • All required HTTP hosts are available

Probably the Future completing setState being called after the widget is already removed from the tree. Check if (mounted) before doing anything after something async in a stateful widget's state.

Thank you, will try it and monitor crashlytics if there will be problem anymore


Update datatypes in documentation

Need to add type-specific information in the documentation.

Ideally this information will go in docstrings for the relevant files and get pulled in with the existing automodule directives.


[app/widbt_notify] screen not properly cleared after a pop-up

Affected hardware version
Bangle 2
Your firmware version
2v13
The bug
Screen is not properly cleared after widbt_notify popup.
Steps to reproduce

Install widbt_notify.
Run a dummy clock app.

function draw() {
  g.reset();
  g.clearRect(50,50,100,120);
  g.drawString("01:01", 50, 50);
}

g.clear();
draw();
var secondInterval = setInterval(draw, 1000);

Disconnect your watch from bluetooth.

Expected results: pop up appears, and the screen is showing 01:01 in a couple of seconds again.
Actual result: pop up appears, but the screen is not properly cleared.
I suspect the reason for that is how widbt_notify cleans up the screen:
    redrawCurrentApp: function(){
        if(typeof(draw)=='function'){
            draw();
        }else{
            load(); // fallback. This might reset some variables
        }
    },

Do I understand correctly, that widbt_notify is incorrect in assuming that if the app has a draw() function, it will correctly redraw the whole screen?
If this is true, the trivial fix would be just to call load() regardless of the draw() method.
            E.showMessage(/*LANG*/'Connection\nlost.', 'Bluetooth');
            setInterval(()=>{load();}, 3000);

CC @KungPhoo

I can make a PR, if that helps. But I would like to first make sure that I understand the problem correctly.

You're right. Also, I have problems, that pressing the button will not take me to the launcher. So, "load()" is the way to go.

Yes, I think load() is best as there's no global way to ensure the screen is redrawn.


Open Prs - Asmita Mishra

Added all the open PRs.
Total points - 90
I will make a 2nd PR later to balance the points.

you need to raise one more pr


Bug fix/internal issue #566

New behavior for TOKENS function.
Added ability to tokenize numbers, nulls and booleans with corresponding analyzers (terms returned as Base64Encoded)
Analyzer name (second parameter) is now optional (identity used by default)
Added ability to process arrays (possible nested)
Added tests for new functionality (js and GTEST)
https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5262/

Is there documentation for the new behaviours / functionality? Anything that needs to be mentioned in UpgradingChanges?

Implementation is backward compatible, e.g. all existing queries utilizing TOKENS functions will behave the same way as before. So, I assume there is nothing to mention in UpgradeChanges? Documentation will be updated with separate PR.

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5323/


RT-3.1: Policy based VRF selection

Add test RT-3.1.
This is a continuation of #521 with 2d83ab4 changes & rebased to main.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.588%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1602

Relevant Lines:
2177

💛  - Coveralls

Most of these changes are in #712.


New CSG Polygon bugs introduced by "Fix multiple issues with CSGPolygon #49314"

Godot version
3.4 beta 4
System information
Windows 10, GLES3
Issue description
Regarding https://github.com/godotengine/godot/pull/49314 I have run into a few issues.

Interval is now a fraction of the entire length of the curve (???)* rather than a distance, meaning long curves behave different from short curves with the same settings as far as polygon density goes.  This worked better as a distance in meters and was consistent with the curve bake interval. *Not sure if it's actually a fraction, because .1 seems like it would be broken into 10ths, but there are more than 10 segments, and sometimes a value of 1 has more than 1 segment.

Can't get the UV's to behave the same as before.  The UV behavior is probably better... mostly, but still has oddities.
Before:

After:

On the plus side, it tries to wrap the texture around.  On the minus side, it seems to only use 1/2 of the texture:

I would expect this to wrap around entirely so I could have a tiled pipe texture or something.  Maybe it's because the other half of the texture is reserved for the caps?  Wonder if that should be an option.
Also, the texture coverage doesn't take the size of the polygon into account, so a really small face uses the same amount of texture as a really large one:

Also, it looks like the UV's reset to 0 for each segment, which works OK for static textures, but not for custom shaders (ex: a pulse that flows across a cable).  And, since the segment length is a fraction of the total curve, the overall texture coverage, and thus timing of pulses and other shader effects, don't cover a set distance using the UV's like it used to.

Limit of .1 for the interval is insufficient for long curves.

Pretty self-explanatory.  I need more detail for my long cables!  I actually had an engine modification to simplify based on the curve angle which I need to make a pull request for (after I update it to match the new code).

Steps to reproduce
Make a really long path and try to make the CSG geometry follow it and look good. :)
Minimal reproduction project
test_csg_path_3.4b4.zip
test_csg_path_3.3.3.zip

cc @madmiraal

As mentioned in #49314, there are some compatibility breaking changes that need to be mentioned in the release notes. The changes are by design and were explained and discussed in the original master PR #49313.

Interval is now a fraction of the entire length of the curve (???)* rather than a distance

Maybe I'm missing something, but this wasn't changed. It does and always did use fixed steps along the length of the curve using interpolate_baked() method.

Can't get the UV's to behave the same as before

In MODE_PATH, the path_continuous_u parameter now defaults to true to make it consistent with the other extrusion modes. It can be set to false to create a repeating pattern along the length of the path as before.
To prevent the UV mapping from simply repeating on each end and each extrusion (#23257), the UV mapping for the entire CSGPolygon was defined as: the top half is rolled around the extrusions (u along the the length of the extrusions and v around the outline of the 2D polygon shape) and the bottom half left and right sides are used for the front and end caps respectively.

Limit of .1 for the interval is insufficient for long curves

A minimum of 0.1 and a step size of 0.05 seemed reasonable for making adjustments in the editor, while keeping the number of polygons at a reasonable number. Since path_interval simply sub-divides each extrusion without adding fidelity, if a user wants to increase the number of extrusions, I expected the preferred approach to be to increase the number of points on the curve and leave path_interval at 1. That been said, this is just a limitation in the editor. The user can always set the path interval to any value between (0:1] in code using set_path_interval() or the property path_interval. If .1 is insufficient for long curves, what would be a reasonable value?

Maybe I'm missing something, but this wasn't changed. It does and always did use fixed steps along the length of the curve using interpolate_baked() method.

Old behavior:

This is with interval set to 1.  I have a 1m cube for reference.  Note that the curve is broken into 1m segments.  This is consistent with the bake interval.
New behavior:

Interval is set to 1, but the segments are not 1m in length.  There's also more than 1 segment.  It seems dependent on the number of control points added to the curve.
The problem with this behavior is that, if I want to have a consistent spacing (ex: every .3m) of my geometry, I have to do a bunch of calculations instead of just setting the interval once.  Also, any time I tweak the curve, I have to update these calculations, and even when calculated, it won't be exact,

To prevent the UV mapping from simply repeating on each end and each extrusion (#23257), the UV mapping for the entire CSGPolygon was defined as: the top half is rolled around the extrusions (u along the the length of the extrusions and v around the outline of the 2D polygon shape) and the bottom half left and right sides are used for the front and end caps respectively.

Yeah, that's what I figured.  It's probably a good change.  The only down side is that you can end up with a seam since the texture won't tile in the middle.  Also, could waste texture memory if you don't have the caps visible (ex: pipes).  Might be worth adding an option for.

A minimum of 0.1 and a step size of 0.05 seemed reasonable for making adjustments in the editor, while keeping the number of polygons at a reasonable number.  Since path_interval simply sub-divides each extrusion without adding fidelity, if a user wants to increase the number of extrusions, I expected the preferred approach to be to increase the number of points on the curve and leave path_interval at 1. That been said, this is just a limitation in the editor. The user can always set the path interval to any value between (0:1] in code using set_path_interval() or the property path_interval. If .1 is insufficient for long curves, what would be a reasonable value?

Since it's based on a subdivision of the curve, it's difficult to say what a good value is.  If I have a 100m curve, I might need 0.001, but that would be really small for shorter curves.  I now understand why you have the interval set up the way you do (since some people probably want a 1:1 polygon with the number of points added).  Perhaps the solution here is to have an interval type: distance vs subdivisions, that way those of us who need to do distance-based things like cables with UVs that repeat at a certain length can use the distance-based one, and people using CSG for other stuff can use that.
Also, I'm using this all in-editor for environment work, so setting those values via code doesn't work well.

I need to change the interval to minimum of 0.01 and a step of 0.01 for my new pull request of that:
#41499
In this small video, I show the difference between minimum of 0.1 and a step size of 0.05 VS minimum of 0.01 and a step of 0.01
With my pull request the actual setting without code, allow only a low poly style of sculpt.
https://user-images.githubusercontent.com/26193340/131589887-bf7fcf09-a375-4bcb-add8-d9925790579c.mp4
Without my new feature, I prefer give us more option with the Editor. 0.1 as minimum is not appropriate when you want modify the angle in the path curve.
https://user-images.githubusercontent.com/26193340/131591947-8ddb3f3d-19ea-43f1-8634-eb4f9098b867.mp4

Interval is set to 1, but the segments are not 1m in length. There's also more than 1 segment. It seems dependent on the number of control points added to the curve.
The problem with this behavior is that, if I want to have a consistent spacing (ex: every .3m) of my geometry, I have to do a bunch of calculations instead of just setting the interval once. Also, any time I tweak the curve, I have to update these calculations, and even when calculated, it won't be exact,

Yes, the calculation for the number of extrusions required was changed to depend on the number of points on the curve and not the length of the curve, which didn't make sense. The number of polygons shouldn't be dependent on the distance. It should be dependent on the number of points on the curve. Increasing the number of polygons without increasing the fidelity of the shape creates unnecessary polygons and vice versa. I notice, however, that the formula is undercounting by 1, so that needs to be fixed.
Ideally, the size of each of the polygons should be dependent on the distance between the curve's points (and path_interval should be inverted to be sub-divisions between the points, so that it's the same as spin_sides in MODE_SPIN). However, the problem is that the UV mapping (with path_continuous set to false) currently also depends on the size of the polygons, and to keep the repeating pattern consistent the polygons need to be of equal length. Hence the compromise of keeping the fixed steps along the length of the curve.
The challenge you're facing is wanting the UV mapping to repeat by distance, which is a different issue. I don't even know how you would do this in Blender. It would, as you suggest, require calculating the repeating interval based on the length of the curve.

If I have a 100m curve, I might need 0.001, but that would be really small for shorter curves.

and

In this small video, I show the difference between minimum of 0.1 and a step size of 0.05 VS minimum of 0.01 and a step of 0.01
With my pull request the actual setting without code, allow only a low poly style of sculpt.

In games you want low polys to reduce the rendering time. Instead smooth shading is used to hide the corners. Note: a setting or 0.01 would create 100 x 2 x polygon.size() faces per extrusion, and a setting of 0.001 would create 1000 x 2 x polygon.size() faces per extrusion! That been said, I accept that 0.1 is too low for curves with large bezier deviations. MODE_SPIN has 64 as a maximum number, which is equivalent to a path_interval of 1/64 ≈ 0.015. So, having a minimum value and step size of 0.01 makes sense.

I'm working on a change to address several of the issues I have as well as adding some more flexibility:

Option for the interval type (subdivide, like you have, or distance, like it used to use).
UV tile distance, so you can set it to 1m, like it used to behave, 0 to cover the whole mesh, like you have, or any custom value you'd like if you want to tile textures at a different distance.
Mesh simplification based on customizable angle, so you can have detailed curves, but long near-straight sections won't have tons of subdivisions.
Tweaked limits.

I'm trying to keep the changes minimal, but the PoolVector is problematic, because the Write needs to go out of scope before the array can be resized (needs to be resized to reduce the poly count), but that would mean putting a large scope around everything... I don't suppose there's any other way?  I know 4.x is going away from the PoolVector.  Should we even be using a PoolVector here?

I've opened 2 PR's for this in both main and 3.x (since the class name changes, removal of pool vectors, etc. make merging ugly):
3.x: https://github.com/godotengine/godot/pull/52509
main: https://github.com/godotengine/godot/pull/52512

Ok, just to give some more info on this, I know i reacted on madmiraals post awhile ago and just didn't have time to revisit it. I've used CSGPolygon a lot over the past few years and I know various other people who make a lot of use of it as well who will run into trouble with the "wanted breaking changes".
This is a track in 3.3 using path extrusion and most importantly, continious U to create a predictable distance based tiling of the texture over a long distance:

This is that same track in 3.4 (without Jitspoe's fix):

Two things spring out here:

Path interval has changed, I need to set this to a much lower value to get the track to render properly again, the end result seems the same though I'm not entirely sure what the unit now represents.
But more importantly, continuous U no longer tiles by distance, it seems to stretch the texture over the total track. Yes I can scale U but that isn't a very good solution especially when we are dealing with generated geometry or where use cases like these introduce separate segments with variable lengths (like the pitlane). This is not an acceptable breakage. This use case was the use case for which continuous U was introduced, to create a tileable U over a set and predictable distance and it no longer behaves that way.

@jitspoe I just tried out your PR but with that the texture is still stretched over the entire distance instead of re-introducing the original desired behavior:

(I did change the path interval so we have corners again (forgot to update the fence so that is still wrong).

What's your UV distance set to?  I thought that defaulted to 1, which should have the same behavior as the old CSG (tiles at 1m). (Note: Will most likely be changing this to path_u_distance instead of uv_distance).

@jitspoe testing now with the proper build, path interval is working beautifully with interval type set to distance.
UV distance is working but it is now working for both U and V, and that is a problem in my use case because it's nearly impossibly to get the road texture to line up properly with the road. We almost need separate options for U and V.
Love the angle simplify btw, I think I applied a similar approach in my procmesh library, which I'm really starting to feel I should revive.

Ok, I don't understand it but purely by trial and error I found out that if I scale my U by 8, the width of my track is exactly right. There probably is a good explanation for this:

Separate options for U and V should be a thing either way - I bet there's a ton of use cases that need them.

Separate options for U and V should be a thing either way - I bet there's a ton of use cases that need them.

The thing is, you're only extruding along one, so it might just lead to overkill to have too many choices. I think the current state with the changes Jitspoe added will cover the needs of most if not all. I would be against making further changes until someone comes up with a valid use case.

UV distance is working but it is now working for both U and V

That's strange.  It should only apply along the length of the curve (U).  I renamed it to path_u_distance in master since it only applies to the U.

UV distance is working but it is now working for both U and V

That's strange. It should only apply along the length of the curve (U). I renamed it to path_u_distance in master since it only applies to the U.

Yup, I drew the wrong conclusion, what you said I think in your reaction on your PR was spot on. I've got 4 sections in my polygon and it's using half of the texture width to make room for the end caps, so each section is 0.125 in length. * 8 makes the U go from 0 to 1.0, it's unaffected by the uv scale.


⚠ Samsung has degraded performance

In ea365fd, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1993 ms

Resolved: Samsung performance has improved in 121525d.


[WIP] Testing Ruby 2.4.0

New Ruby version 2.4.0 just seeing how we run against it.

@miq-bot add_label wip

Replica of this PR #13104


Initial implementation for IVar variants and consensus

nf-core/viralrecon pull request
Many thanks for contributing to nf-core/viralrecon!
Please fill in the appropriate checklist below (delete whatever is not relevant).
These are the most common things requested on pull requests (PRs).
PR checklist

[ ] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/viralrecon branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[ ] CHANGELOG.md is updated
[ ] README.md is updated

Learn more about contributing: CONTRIBUTING.md

@saramonzon Looks like you need to change your review to Approve to merge

Thank you, @drpatelh @saramonzon @JoseEspinosa


RPKI-SIGNED-OBJECTS and RPKI_SIGNED-OBJECTS

Two identical items, apparently? https://dev.bibxml.org/search/?bypass_cache=1&query=IANA+rpki+signed+objects&query_format=websearch

Are these two items being duplicated because of old data?

I don’t know where the data comes from (@andrew2net knows perhaps?)
This one seems to have a bit more data: https://dev.bibxml.org/get-one/by-docid/?docid=IANA+rpki%2Fsigned-objects&doctype=IANA&query=IANA rpki signed objects&query_format=websearch&page=1

The data comes from https://github.com/ietf-ribose/iana-registries
The RPKI-SIGNED-OBJECTS converts from https://github.com/ietf-ribose/iana-registries/blob/main/rpki-signed-objects/rpki-signed-objects.xml
The RPKI_SIGNED-OBJECTS is included document from https://github.com/ietf-ribose/iana-registries/blob/main/rpki/rpki.xml
The included documents treated as discussed here https://github.com/relaton/relaton-iana/issues/2#issuecomment-981653026
We use / as a separator when making IDs for included documents and replace the separator with _ when making filename.


Add documentation page

The documentation page is missing, please complete the documentation.

Thanks.
Fixed in https://github.com/stape-io/facebook-leads-tag/commit/4d0860b4dc785b8b848da1e893da06d0c1f64df9
Hope this tag will be added soon to the Gallery.


Cost based optimization

Related to #183.
This PR introduces real cost-based optimization into dask-sql using the Calcite volcano planner.
For this, I did three steps:

refactored the RelationalAlgebraGenerator class into smaller sub-classes to make the handing and go away from the "default" Framework program
Put a volcano after the hep-planner, which is allowed to choose rules based on the costs
give the user the possibility to add custom statistics to a table - currently, it is only the row count but we might want to add more later.

The create_table function has now an additional parameter statistics, where you can give a dask_sql.Statistics object. In my very first small tests, I could not find a relational algebra which is now optimized differently - but that does not mean there is none. Maybe we can find some benchmark use-cases.

Hi @rajagurunath! I know, this is quite a large PR (and I am sorry for that) and I do not expect you to read through all of it - but maybe you find the time to have a quick look? I am more or less confident that I did not screw it up completely (because I only changed minor things in the tests and they still work), but I would feel a lot better of you also have a rough look :-) Thanks!

Hi @nils-braun,
Really Amazing work, kudos to you !!! 👏
Not able to understand all the things, but able to correlate ~ 60% of things, And I have tested in my local,  as you mentioned it doesn't break anything and works like a charm. Dask Nodes and their respective Dask Rules are looking clean and elegant from Java side.
I compared the plan of SQL, with and without cost-based optimization able to see effect of the row count in the plan, definitely it will be a great value add for the dask-sql users.
With Cost Based optimization plan
DaskProject(sepal_length=[$0], sepal_width=[$1], petal_length=[$2], petal_width=[$3], species=[$4]): rowcount = 22.5, cumulative cost = {256.25 rows, 314.5 cpu, 0.0 io}, id = 273
  DaskJoin(condition=[AND(=($4, $5), =($0, $6))], joinType=[inner]): rowcount = 22.5, cumulative cost = {233.75 rows, 202.0 cpu, 0.0 io}, id = 272
    DaskTableScan(table=[[root, iris]]): rowcount = 100.0, cumulative cost = {100.0 rows, 101.0 cpu, 0.0 io}, id = 249
    DaskAggregate(group=[{4}], sepal_length=[MAX($0)]): rowcount = 10.0, cumulative cost = {111.25 rows, 101.0 cpu, 0.0 io}, id = 271
      DaskTableScan(table=[[root, iris]]): rowcount = 100.0, cumulative cost = {100.0 rows, 101.0 cpu, 0.0 io}, id = 249

And time to complete the above join :)
119 ms ± 2.14 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)
Without Cost Based optimization plan
LogicalProject(sepal_length=[$0], sepal_width=[$1], petal_length=[$2], petal_width=[$3], species=[$4])
  LogicalJoin(condition=[AND(=($4, $5), =($0, $6))], joinType=[inner])
    LogicalTableScan(table=[[root, iris]])
    LogicalAggregate(group=[{4}], sepal_length=[MAX($0)])
      LogicalTableScan(table=[[root, iris]])

136 ms ± 12.9 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)
Not able to do benchmark completely but with help of dask-sql Feature-Overview.ipynb, I am able to see some improvement and I am super exicited about that. (even the default row count was set as 100, but able to see some improvement in the above join query)
So As you have mentioned we will be adding more statistics (min, max, avg ) to the plan right? And how about the ANALYZE statement to add those statistics to each table. (maybe we will create a new sub-issue  to track this )


CLI support fully qualified name with 3 parts with no leading slash.

Right now  wsk action invoke /whisk.system/utils/echo works and wsk action invoke whisk.system/utils/echo does not.
I propose we make it work from the CLI.   Specifically, if the FQN has 3 parts (2 slashes), infer that it is namespace/package/name.
In the documentation this will entail filling in the blank cell for Alias in the first row of the FQN examples.

Fully qualified name
Alias
Namespace
Package
Name

/whisk.system/cloudant/read

/whisk.system
cloudant
read

/myOrg/video/transcode
video/transcode
/myOrg
video
transcode

/myOrg/filter
filter
/myOrg

filter

This would be consistent with the API which can tolerate a missing leading slash on a fully qualified name.

If we don't require the leading / then it will fix the problem we have with git-bash on Windows https://github.com/apache/incubator-openwhisk/issues/2313


Whenever someone just clicks, not swipes on any card, it just pops to the next one without any callback.

Hi,
have you correctly handled the recover event?

How do I have to handle it?

You have to check which orientation has been chosen for the card in swipeCompleteCallback.
if (orientation != CardSwipeOrientation.RECOVER) {   // handling left or right orientation } else {   // handling if user haven't swiped or just tapped on a card }

Yes but it just skips a card if you press it but not swipe it

I had the same issue but after checking the card orientation as above, all
troubles are gone, so I'm saying that from my own experience.
On Mon, Nov 23, 2020 at 10:25 PM PhiJoTo<EMAIL_ADDRESS>wrote:

Yes but it just skips a card if you press it but not swipe it
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ShaunRain/flutter_tindercard/issues/57#issuecomment-732406197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTP2KWWG4PAQ6LFRDKKHITSRLAKPANCNFSM4SOQ7S3A
.

--
Kind regards, Eugene

this is my code.. i dont know whats missing
**  swipeCompleteCallback:
(CardSwipeOrientation orientation,
int index) {
if (AppBrain().publicSelectedMode == 1) {
stopAudio();
}
setState(() {
AppBrain().nextCard();
});
if (orientation == CardSwipeOrientation.LEFT) {
if (selectedCurrentCard() ==
selectedListLength()) {
setCurrent0();
Navigator.of(context).push(toSwipePage());
}
playHandler();
}
if (orientation == CardSwipeOrientation.RIGHT) {
stopAudio();
Navigator.of(context)
.push(toInformationPage());
if (selectedCurrentCard() ==
selectedListLength()) {
setCurrent0();
Navigator.of(context).push(toSwipePage());
}
}
},**

This should work.
** swipeCompleteCallback:
(CardSwipeOrientation orientation,
int index) {
if(orientation != CardSwipeOrientation.RECOVER{
if (AppBrain().publicSelectedMode == 1) {
stopAudio();
}
setState(() {
AppBrain().nextCard();
});
if (orientation == CardSwipeOrientation.LEFT) {
if (selectedCurrentCard() ==
selectedListLength()) {
setCurrent0();
Navigator.of(context).push(toSwipePage());
}
playHandler();
}
if (orientation == CardSwipeOrientation.RIGHT) {
stopAudio();
Navigator.of(context)
.push(toInformationPage());
if (selectedCurrentCard() ==
selectedListLength()) {
setCurrent0();
Navigator.of(context).push(toSwipePage());
}
}}
},**

thanks ill try later

It works, thank you so much 👍🏻


Added throttle skeleton and tests

This includes one additional skeleton method with accompanying tests.
This does not close any existing Issue.

[x] Running yarn lint does not trigger any linter errors
[x] You have written a new skeleton method for someone else to work on!
[x] You have written tests to accompany your skeleton method

Could you open up a new Issue for the throttle func? Thanks!

Yep!
see #652


To get XLS support XLRD should always be installed

despite tablib support for and bundling of xlrd, I could not get XLS import working without installing XLRD separately, which is weird ... but that is likely because tablib in Pypi is so outdated

I've submitted a pull request upstream to de-vendorize xlrd https://github.com/kennethreitz/tablib/pull/185


Mulitmodel LLMs (esp. images): how to use them?

I would like to use dspy.AzureOpenAI with a dspy.functional.TypedPredictor on images.

Is this possible?
If so, how?

I'm not seeing a guide on this linked in the README, sorry if I overlooked something.

This is not currently supported, unfortunately.
See https://github.com/stanfordnlp/dspy/issues/624

check here for update https://github.com/stanfordnlp/dspy/pull/675

It seems the PR is closed, but the PR isn't merged?
Should we keep this issue open until we land it?

is image support on the roadmap? gpt-4o is motivating...

here are some breadcrumbs: https://github.com/stanfordnlp/dspy/blob/56a0949ad285e0a3dd5649de58a6f5fb6f734a60/dsp/modules/gpt4vision.py#L106C1-L147C20


Follow the Rule Of Zero in VBReader and VBWriter classes

Member std::unique_ptr already takes care of resource management.
https://en.cppreference.com/w/cpp/language/rule_of_three#Rule_of_zero

Remember that this is using the pImpl idiom. The private class Impl is completely opaque to client code, so the compiler can't create a default move ctor or move operator in the client code. The only place that has that information is inside the implementation .cpp.
I suspect this code builds find in the library, but have you tried consuming it from just the public header?  The test suite can be found here.

You're right, I missed that. The tests don't check movability of the objects, therefore they failed to compile only because of destructor using an incomplete type. Anyway, those special members seem to be fine being defaulted in .cpp, so I will open another pull request implementing that.


How to hot swap the streaming?

Hi sir. I would like to know if there is a way to change streaming orientation in hot swap.
I mean, there is a method to change streaming orientation but only when preparing streaming:
rtspCamera1.prepareVideo(width, height, fps, bitrate, hardwareRotation, rotation)) {
with this whe can define the streming orientation as Landscape or portrait but, is there a way to do it meanwhile streaming?

You cna try with rotation filter:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/java/com/pedro/rtpstreamer/openglexample/OpenGlRtmpActivity.java#L234


[WIP] Adds Linear, Ridge and Lasso regressions

see #7

Thanks for working on it @Nimpruda!
I left some comments here and there to give you pointers on the next step to make this PR ready to be merged - it's already half-way there 😁
I see we have some issues in CI due to ndarray-linalg requiring a BLAS implementation to build correctly - I'll work in a separate PR to get this issue solved so that your branch can compile correctly on Travis 👍

I'll work on it whenever I have time! Thanks for all the feedback

superseded by #20


Add SQL support for time-ordered scans

Motivation
See https://github.com/apache/incubator-druid/issues/6088 for original idea.  PR #7133 is close to completion and the next step is to add SQL support for time-ordered scans.  This would eliminate the need for using select queries in SQL planning since the only thing select is good for is time-ordering results.  Updating to use scan would improve memory performance.
Proposed changes
The SQL planning in DruidQuery will be changed so that Scan is used if ordering by __time is specified.  After that, Select will be essentially obsolete and will be removed from SQL planning altogether.
The user interface won't change.
Rationale
I think removing select queries from the SQL planner completely is the best choice since its design isn't great memory-wise.  Although this means that time-ordered SELECT queries that fall outside of the configurable scan time-ordering limits (default 100K rows or 30 segments per time chunk) will fail, these limits can be tuned based on machine specs to a point where the query will succeed.  Furthermore, if the query is big enough to cause memory issues with scan, using a select will be even worse.
Operational impact
No impact to overall cluster operation.  Existing select queries might start failing if they're outside of the configurable row or segments per time chunk limits.

Scan is used if ordering by __time is specified

Scan is already used if ordering by __time is not specified so I suppose this proposed change means that Select will never be used by Druid SQL. That sounds ok to me, because in situations you describe (lots of segments or high thresholds), Select queries have resource usage issues anyway as described in #6088. Make sure to update the Druid SQL documentation to talk about how the queries are planned now.
👍


[1.16 deployment] Resource level temporal coverage display in edit mode is not correct

For composite resource the display of resource level coverage is correct in view mode. But when displayed in edit mode, it is not correct - shows always the current date.  Most likely a bug in JavaScript code.

Closing.  Duplicate of #2949