way to create optional parameter in url?

I fiddled with some regex, but this is beyond me.
I need the last parameter in my state to be optional. In other words: capture it if it is there, but otherwise ignore it.
Something like:
$stateProvider
.state \user.page, {
  url: '/:username/:page?'
  template-url: '...'
  controller: [
  \$stateParams
  ($stateParams) ->
    # The page parameter should be optional.
    # This should activate on both '/legomind/dashboard' as well as '/legomind'
  ]
}

+1

+1 I need support for an optional trailing slash

Same here

+1

Hey guys,
I just found the amazing squash parameter for the params optionnal object in ui-router.
At the bottom of this doc: https://angular-ui.github.io/ui-router/site/#/api/ui.router.state.$stateProvider
So if we take the @AdamBuczynski example :
$stateProvider.state('tutorials.view', {
        reloadOnSearch: false,
        url:            '/:tutorialAlias/:lessonAlias/:lessonSlide',
        controller:     'TutorialsViewCtrl',
        templateUrl:    'tutorials/view/view.html',
        params: {
            lessonAlias: {value: null},
            lessonSlide: {value: null, squash: true} //The parameter's default value is omitted from the URL
        }
    });

So it will produce url like if we only have lessonAlias:
some/url/tutorial-alias/lesson-alias
Another exemple here with no lessonAlias and a custom lessonSlide value
$stateProvider.state('tutorials.view', {
        reloadOnSearch: false,
        url:            '/:tutorialAlias/:lessonAlias/:lessonSlide',
        controller:     'TutorialsViewCtrl',
        templateUrl:    'tutorials/view/view.html',
        params: {
            lessonAlias: {value: null, squash: true},
            lessonSlide: {value: null, squash: 'foo'}
        }
    });

So it will produce url like:
some/url/tutorial-alias/foo
I think the squash param is a way to avoid double slashes when parameters are missing.

@juli3773 Thanks! I'll try that out tonight

@rwwagner90 tell me if it's working ^^

For everyone :+1: this issue, this is already released—you don't need to vote for it anymore. I'm using it in one of my projects and it's working.

@jshado1 could you link the docs for this feature. I couldn't find it. Which one of the notations is implemented?

Link is mentioned in the comment above.

@juli3773 sorry for the late response, but it does work. Thanks!

So, now I have a new problem. How can I make a parameter required?

What behavior do you want on /profile ?

/profile and /profile/ should both just not be valid. They should both be ignored and just go to a 404 page or whatever we decide to show when a route is not valid. Currently, without using squash: true if I go to /profile or /profile/ it ignores those because there is no id, but once I started using squash, it made /profile and /profile/ valid.
Essentially, what I could not do before was:
/profile/id without a trailing slash.
id is a required param and slug is not. I need the following to all be valid:
/profile/id
/profile/id/
/profile/id/slug
/profile/id/slug/

Hey guys, I am stuck on this dynamic parameter passing to my controller,
template:

my state us is look like this,
.state("single", {
url: "/single/:id",
controller: "single",
templateUrl: "partials/single.html"
})
controller:
.controller("single", ["ngCart", "$log", "$scope","$stateParams", function(a, b, c, d) {
c.id = d.id;
alert(c.id);
}])
But I am not getting proper output on this. Please any one give me solution..!!

@muthu07
That ui-sref does not look valid. I think it should be ui-sref="single({id : item.id})"
A ui-sref should contain the (full) state name, not a url. If there are parent states, be sure to prepend them with periods.
NB, www.stackoverflow.com might be a better place for those kinds of questions, seeing as it does not seem to relate to this issue.

@JohannesFerner as well as anyone else.
You can check the commit comments for some sort of interim documentation
https://github.com/angular-ui/ui-router/pull/1032
Then there's the comments from the squash pull, where the syntax seems to have changed ever so slightly https://github.com/angular-ui/ui-router/issues/1501
from  params: { foo: { value: null } }
to  params: { foo: null }
But I'm not quite getting all of this to work.
It would be really neat if the official wiki could be updated - by someone in the know - with the current implementation as per the latest release 0.2.13

Please disregard my previous comment, the documentation is here
http://angular-ui.github.io/ui-router/site/#/api/ui.router.state.$stateProvider
(just browse to the params section at the bottom.
the params: { foo: null } format is a actually a shorthand when you only need to specify the value of the param.
And I just don't seem the get optional params to work as expected, which is also unrelated to the otherwise clause that

+1

Alright, closing this out. @rwwagner90 if you have a different problem, please post that in a new issue so I don't have to read a small novel to figure the status of it. :wink: :beers:

@juli3773 thanks a lot

+1

I want to have optional integer-type parameter like this: url: '/sign-up/{partnerId:int}', is it possible?

@slavafomin Yes it's possible! So you'll have:
url: '/sign-up/{partnerId:int}',
params: {
    partnerId: {value: null, squash: true}
}

Thank you @juli3773, I've finally managed to find a similar solution. Is squash: true is really necessary? I have the following code:
params: {
    partnerId: null
}

And it seems to work great )

@slavafomin the   squash: true is necessary only if you want your parameter to be optional!

I tried /s/{param1}/{param2}/{param3} and so forth including
params : { param1: {value : 'default'}, ... }
but it won't match /s/ and then auto populate the rest of the params with default value. It only works if I populate all params but the last one, then it will work but now show the default value in the URL. Any ideas on why @okendoken ?


Text behaviour has regressed

When running the typewriter example I noticed that the behaviour of the ext/text/Text type has regressed from the original pixel repository. It from looking at the git blame it seems most that this introduced by the changes @roipoussiere made here:
https://github.com/gopxl/pixel/blob/2298e16d4b77704193c2704a98678e4081a63cc3/ext/text/text.go#L261-L262
Because of this regression it is no longer possible to accurately position text by setting the txt.Dot vector to the desired position

Thank you for the report. I do see the issue you're describing, the typewriter seems to slowly drift away from the text the more you type.
Reverting this change  does seem to fix the typewriter, but it's not yet clear to me what that was addressing in the first place.

Original PR for reference: https://github.com/faiface/pixel/pull/252


[FEATURE]

Description
I want to create a new signin/signup box
Screenshots
No response
Additional information
No response

@narayan954

this issue was already concluded in #172 , how would your's be different? Also, a login and signup page is in development so it's not needed.

ok


inline code string values rendering as unicode python, eg u'some string' instead of

In a .pymd this is the mark up
My name is `python my_name`

renders as
My name is u'Chris'
instead of
My name is Chris

I'm not sure that there is an easy fix here :-( The code behind this is basically the same as writing this in a cell in the notebook:
In [5]: u"üä"
Out[5]: u'\xfc\xe4'

So, if the above inline code should return the string without quotes (and as a normal "readable" unicode instead of encoded ones) but in code blocks this should show quotes and u"..." signs... this needs a switch in the plain text formatter before/after each submitted code (or at least when switching from inline to block and reverse :-/
from IPython.core.interactiveshell import InteractiveShell
shell = InteractiveShell.instance()
tp = shell.display_formatter.formatters[u'text/plain']
# then set the following to some functions which does the same as `print`
tp.type_printers[unicode] # and str

Not sure if there are any better alternatives...

Or the easier way: add a small wrapper in the engine code which does this for you:
So instead of submitting my_name it's submitting p.text(my_name) to be executed...


Enhancement - swift3 server side sample code via swift package manager

It's now possible to package objective-c and c++ into swift package manager.
<EMAIL_ADDRESS>While it's possible to spit out a simple swift implementation of messages suitable for libraries on ios using Apple's handy work
https://github.com/apple/swift-protobuf
generating the corresponding ProtoService is something else
https://github.com/apple/swift-protobuf/issues/118
In other words - given the sample swift code in this repo-
https://github.com/grpc/grpc/tree/master/src/objective-c/examples/SwiftSample
can we package this up into SPM (decoupling ios stuff) to provide a hello world for server side swift?
To explore this further (for anyone wanting to attempt this) - I've patched a Dockerfile for swift3 local development https://gist.github.com/johndpope/01a74ffb93921df48e334097117be1b3
This was to interface to https://github.com/johndpope/DockerParseyMcParsefaceAPI
that runs parsey mcparseface using gprc + protobuffers.
I will attempt to port this over and update this ticket if I make any progress.

CC @muxi @timburks

I found this https://github.com/Huawei-PTLab/SwiftGRPC
https://github.com/Huawei-PTLab/CGRPC
but couldn't get grpc patched. https://github.com/Huawei-PTLab/CGRPC/issues/2

Yielding to google dedicated swift grpc repo. 🙌
toodles https://github.com/grpc/grpc-swift


[Question] Constructor-Injection with Variable Key Possible?

Hello,
I'm relatively new to proper MVVM and greatly enjoyed this framework so far, the documentation is excellent in getting you started in no time.  I've come across one point, where I'm not sure if it's not possible or if I understand the concept wrong. Here goes...
My application has a dialog MyDialog who displays items in a list on the left side.  Selecting an item in the list opens a detail editor on the right side of the dialog.
I want to empower the dialog to work with different types of data in order to reuse it throughout the application and would therefore like to inject functionality (a data source) into the dialog.
I'm using an interface to abstract that behaviour:
public interface IMyDataSource { ... }
public class SourceOne : IMyDataSource { ... }
public class SourceTwo : IMyDataSource { ... }

The constructor of the dialog's ViewModel looks like this.
public MyDialogViewModel(IWindowManager windowManager, IMyDataSource source) { ... }

Instances of the dialog, with different data sources, would be created by the parent window in different Actions:
public class MainViewModel : Screen
{
    ...
    public void EditSourceOneAction()
    {
        // ???
    }

    public void EditSourceTwoAction()
    {
        // ???
    }
}

How can the dialog's parent (the "Main" window) get Stylet to instantiate appropriately?
I've tried an abstract factory and registering that with the IoC container as follows, but it didn't work, I guess the key does not get (or cannot be) propagated:
public interface IMyDialogFactory
{
    MyDialogViewModel CreateDialog(string key);
}

public class Strappy : Bootstrapper<MainViewModel>
{
    protected override void ConfigureIoC(IStyletIoCBuilder builder)
    {
        ...
        builder.Bind<IMyDialogFactory>().ToAbstractFactory();
        builder.Bind<IMyDataSource>().To<SourceOne>().WithKey("one");
        builder.Bind<IMyDataSource>().To<SourceTwo>().WithKey("two");
    }
}

public class MyViewModel : Screen
{
    [Inject]
    public IMyDialogFactory DF { get; set; }

    public void EditSourceOneAction()
    {
        this.DF.CreateDialog("one");
    }
}

My workaround (or maybe this is the solution?) is to implement a factory myself, but it just doesn't feel as clean/loosely coupled because now I have ViewModels that are not instantiated through the IoC container and also I have to touch the factory for every new implementation of IMyDataSource that the dialog should support:
public MyRealDialogFactory()
{
    private IWindowManager wman;

    [Inject]
    public MyRealDialogFactory(IWindowManager wman)
    {
        this.wman = wman;
    }

    public MyDialogViewModel CreateWithSourceOne()
    {
        return new MyDialogViewModel(this.wman, new SourceOne());
    }

    public MyDialogViewModel CreateWithSourceTwo()
    {
        return new MyDialogViewModel(this.wman, new SourceTwo());
    }
}

Is what I want just not achievable, or not intended by StyletIoC, or am I missing something?

The built-in ioc container is pretty simple, and doesn't support what you want out-of-the-box.  You could use a more powerful ioc container, but my person preference would be to use hand-written factories (I don't like too much magic in my ioc container setup).
In more complex applications where types need to be created with factories, I tend to end up with a single factory per type that needs one. Where lots of decoupling is a priority, those factories are hidden behind interfaces and registered with the ioc container, and themselves only depend on abstractions: the only concrete type they know about is the type they're responsible for instantiating.
For your example, I might have MyRealDialogFactory's constructor take either (SourceOne sourceOne, SourceTwo sourceTwo) or (IDataSource sourceOne, IDataSource sourceTwo) or (Func<SourceOne> sourceOneFactory, Func<SourceTwo> sourceTwoFactory) or (Func<IDataSource> sourceOneFactory, Func<IDataSource> sourceTwoFactory), depending on needs (whether to depend on concrete types or abstractions, and whether you want factories or instances).
Does that help?

Yes it does, thank you very much.
I'll stick with my factory implementation then, which is basically your suggestion of "one factory per type".
I, too, don't like too much IoC setup magic - in fact, I've stayed completely clear of IoC containers for a long time because I felt that it can easily get out of hand. :)

Yeah, my approach there is normally to plan everything as if there wasn't an ioc container, then use the ioc container to remove some of the repetitive boilerplate.


godot3-mono.nugetSource.meta (anf maybe others?): error: infinite recursion encountered

Describe the bug
I was trying to fetch metadata for various packages for analysis and noticed that at least godot3-mono.nugetSource.meta attribute is failing with infinite recursion.
The reproducer against nixpkgs-unstable:
$ nix-instantiate --eval --expr 'with import <nixpkgs> {}; godot3-mono.nugetSource.meta' --show-trace
error:
       … while evaluating the attribute 'nugetSource.meta'

         at /nix/store/l71dakfm3vy5akarlqxqgqi7pdq1cblg-source/pkgs/build-support/dotnet/make-nuget-source/default.nix:29:5:

           28|   } // { # We need data from `$out` for `meta`, so we have to use overrides as to not hit infinite recursion.
           29|     meta = nuget-source.meta // {
             |     ^
           30|       licenses = let

       … in the left operand of the update (//) operator

         at /nix/store/l71dakfm3vy5akarlqxqgqi7pdq1cblg-source/pkgs/build-support/dotnet/make-nuget-source/default.nix:29:30:

           28|   } // { # We need data from `$out` for `meta`, so we have to use overrides as to not hit infinite recursion.
           29|     meta = nuget-source.meta // {
             |                              ^
           30|       licenses = let

       … while evaluating the attribute 'meta'

         at /nix/store/l71dakfm3vy5akarlqxqgqi7pdq1cblg-source/pkgs/build-support/dotnet/make-nuget-source/default.nix:29:5:

           28|   } // { # We need data from `$out` for `meta`, so we have to use overrides as to not hit infinite recursion.
           29|     meta = nuget-source.meta // {
             |     ^
           30|       licenses = let

       error: infinite recursion encountered

       at /nix/store/l71dakfm3vy5akarlqxqgqi7pdq1cblg-source/pkgs/build-support/dotnet/make-nuget-source/default.nix:29:5:

           28|   } // { # We need data from `$out` for `meta`, so we have to use overrides as to not hit infinite recursion.
           29|     meta = nuget-source.meta // {
             |     ^
           30|       licenses = let

Expected behavior
Ideally metadata should evaluate successfully (or report an error with a throw function or equivalent, then builtins.tryEval could skip it).
Notify maintainers
Not sure who the maintainer is. Trying recent contributors to pkgs/build-support/dotnet/make-nuget-source/default.nix.
@IvarWithoutBones, @raphaelr and @SuperSandro2000
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.8, NixOS, 24.05 (Uakari), 24.05.20231229.6e62521`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.19.2`
 - channels(root): `""`
 - channels(slyfox): `""`
 - nixpkgs: `/run/current-system/sw/share/nixos-flakes/inputs/nixpkgs`

Add a :+1: reaction to issues you find important.

I think that could have regressed in https://github.com/NixOS/nixpkgs/commit/5ef866783f04770bb0cc088075eb090627a4f7d4

Not sure who the maintainer is.

We have the team @ NixOS/dotnet now.


Unable to build and test karma using source on Ubuntu 16.04

Expected behaviour
Build and test via npm should be successful on Ubuntu 16.04
Actual behaviour
Build showing several warnings related to peer dependency
Environment Details
Target Platform: Ubuntu 16.04
Architecture: Intel (x86_64) and ppc64le
NPM version  v9.4.0
Steps to reproduce the behaviour
Followed below steps on x86_64 ubuntu 16.04
$curl https://raw.githubusercontent.com/creationix/nvm/v0.33.0/install.sh| bash
$source ~/.nvm/nvm.sh
$nvm install stable
$nvm use stable
cloned karma repository
$ npm install
$ npm run init
$ npm test
npm install shows following warning related to peer dependencies:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of watchify@>=3 <4 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of karma@* but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm test failures are:
_488 passing (6s)
4 failing

helper
browserFullNameToShort
should parse IE7:
AssertionError: expected 'IE 7.0.0 (Windows Vista.0.0)' to equal 'IE 7.0.0 (Windows Vista 0.0.0)'

expected - actual

-IE 7.0.0 (Windows Vista.0.0)
+IE 7.0.0 (Windows Vista 0.0.0)
at Context.it (test/unit/helper.spec.js:76:16)

helper
browserFullNameToShort
should parse IE8:
AssertionError: expected 'IE 8.0.0 (Windows 7.0.0)' to equal 'IE 8.0.0 (Windows 7 0.0.0)'

expected - actual

-IE 8.0.0 (Windows 7.0.0)
+IE 8.0.0 (Windows 7 0.0.0)
at Context.it (test/unit/helper.spec.js:84:16)

helper
browserFullNameToShort
should parse IE9:
AssertionError: expected 'IE 9.0.0 (Windows 7.0.0)' to equal 'IE 9.0.0 (Windows 7 0.0.0)'

expected - actual

-IE 9.0.0 (Windows 7.0.0)
+IE 9.0.0 (Windows 7 0.0.0)
at Context.it (test/unit/helper.spec.js:92:16)

helper
browserFullNameToShort
should parse IE10:
AssertionError: expected 'IE 10.0.0 (Windows 8.0.0)' to equal 'IE 10.0.0 (Windows 8 0.0.0)'

expected - actual

-IE 10.0.0 (Windows 8.0.0)
+IE 10.0.0 (Windows 8 0.0.0)
at Context.it (test/unit/helper.spec.js:100:16)

Warning: Task "mochaTest:unit" failed. Use --force to continue.
Aborted due to warnings._
I am running this with default configurations. Please let me know if I am missing something.
Thanks !

@spurti-chopra I think it's fixed with this pr.

Please reopen if not fixed.


E211 warning appears every time vim/MacVim is focused

It looks like the patch made for #3165 causes the E211 banner to appear every time MacVim is refocused. This is fairly annoying, I only need to know the file is deleted once. I have confirmed that reverting the change to fileio.c so that buf->b_mtime = 0; is used resolves this bug. It's unclear if there's a patch that fixes my bug but doesn't regress the other issue's functionality.

This should be reported to the MacVim repo.

What would I report? The behavior stems from a change to this repo - are you suggesting MacVim may be misconfigured/responding incorrectly to this change?

You would report exactly what you reported here.  I'm suggesting that MacVim might have to take special steps to work around this.
Also it'd help to provide the MacVim repo with exact reproduction steps.  I'm not seeing the described behavior, but I'm just guessing at what steps are supposed to bring it about.

It looks like the patch made for #3165 causes the E211 banner to
appear every time MacVim is refocused. This is fairly annoying, I only
need to know the file is deleted once. I have confirmed that reverting
the change to fileio.c so that buf->b_mtime = 0; is used resolves this
bug. It's unclear if there's a patch that fixes my bug but doesn't
regress the other issue's functionality.

Looks like there is a mismatch in using the 'autoread' option: when the
buffer does not have a local value it uses -1 instead of using the
global value.  I'll fix that.  Then please check if your problem is
fixed.
--
For a moment, nothing happened.
Then, after a second or so, nothing continued to happen.
-- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Wow, thank you for fixing this so quickly! I confirmed that this fixes the issue for me. Thank you for the quick turnaround and for maintaining this awesome editor.


Minor tweaks

Added the Georgia Douglas Johnson spreadsheet to the spreadsheets page.
Added a step to the review guide to review the initial commit. This is something I have been doing for some time now and I have noticed several cases of people committing the PG license into the repo. This seems like a good step to codify.

Thanks!


[WIP] convert src/fsharp to dotnet SDK

Work in progress, do not merge.
Redux of converting src/fsharp to the dotnet SDK.
Includes the removal of the *-proto projects and simply switching on '$(Configuration)' == 'Proto' in the project files directly.

@brettfo I added some comments above - my recommendation is just to keep fsc-proto.exe please (and please, keep the name, it is a different thing to fsc.exe with different construction).
Making Proto\fsc.exe = Release\fsc.exe in terms of complexity, dependencies is not a good thing - a good, debuggable, maintainable bootstrap build uses minimal dependencies at the bootstrap phase.

Closing in favor of #4581.


Can't access custom Deployinator properties in config/base.rb

I'm defining custom attributes via attr_accessor in the local Deployinator configuration and then set it to an empty hash in config/base.rb. However it seems like for base config the Deployinator module from the vendorized gem is used. It works fine in config/development.rb and config/production.rb.
relevant sinatra stacktrace:
undefined method `cheffilez=' for Deployinator:Module

/path/deployinator/config/base.rb	46	in `'
1	/path/deployinator/vendor/bundle/ruby/1.9.1/bundler/gems/deployinator-438a82f58710/lib/deployinator/config.rb	4	in `require'
2	/path/deployinator/vendor/bundle/ruby/1.9.1/bundler/gems/deployinator-438a82f58710/lib/deployinator/config.rb	4	in `'
3	/path/deployinator/vendor/bundle/ruby/1.9.1/bundler/gems/deployinator-438a82f58710/lib/deployinator/base.rb	10	in `require'
4	/path/deployinator/vendor/bundle/ruby/1.9.1/bundler/gems/deployinator-438a82f58710/lib/deployinator/base.rb	10	in `'
5	config.ru	2	in `require'
6	config.ru	2	in `block in
'
7	/path/deployinator/vendor/bundle/ruby/1.9.1/gems/rack-1.6.0/lib/rack/builder.rb	55	in `instance_eval'
8	/path/deployinator/vendor/bundle/ruby/1.9.1/gems/rack-1.6.0/lib/rack/builder.rb	55	in `initialize'
9	config.ru	1	in `new'
10	config.ru	1	in `
'
11	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/rack/application_spawner.rb	225	in `eval'
12	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/rack/application_spawner.rb	225	in `load_rack_app'
13	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/rack/application_spawner.rb	157	in `block in initialize_server'
14	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/utils.rb	563	in `report_app_init_status'
15	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/rack/application_spawner.rb	154	in `initialize_server'
16	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server.rb	204	in `start_synchronously'
17	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server.rb	180	in `start'
18	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/rack/application_spawner.rb	129	in `start'
19	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/spawn_manager.rb	253	in `block (2 levels) in spawn_rack_application'
20	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server_collection.rb	132	in `lookup_or_add'
21	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/spawn_manager.rb	246	in `block in spawn_rack_application'
22	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server_collection.rb	82	in `block in synchronize'
23		prelude>	10:in `synchronize'
24	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server_collection.rb	79	in `synchronize'
25	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/spawn_manager.rb	244	in `spawn_rack_application'
26	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/spawn_manager.rb	137	in `spawn_application'
27	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/spawn_manager.rb	275	in `handle_spawn_application'
28	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server.rb	357	in `server_main_loop'
29	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/lib/phusion_passenger/abstract_server.rb	206	in `start_synchronously'
30	/usr/lib64/ruby/gems/1.9.1/gems/passenger-3.0.15/helper-scripts/passenger-spawn-server	99	in `
'

@mrtazz You most likely need need to require your local deployinator configuration with the accessor included before you require deployinator/base but after you require deployinator

yup this is indeed an internal bug. Thanks! :)


Update base images for core drivers to use Ubuntu 15.10

Now that we have support for it in the main Machine binary, I'd suggest that we update our core drivers to default to Ubuntu 15.04 instead of Ubuntu 14.04.
cc @docker/machine-maintainers

I'm all for it. However, I tested the google driver with a 15.04 and the time to create a VM goes from less than 2 minutes to more than 5 minutes. I will investigate tomorrow

Cool, yes, that would indeed be annoying.  If the fault is on our end, we can fix it, and if the fault lies with the Docker project install script, we'll file an issue.

I confirm that with google driver, using the 15.04 image is much longer than with 14.04 image. Mainly because aufs support is already installed in the 14.04 image.

Hm, that's really unfortunate.  I wonder how things look for the other drivers.  I'll give DigitalOcean and AWS a whirl and get back to you.

With exoscale driver :

14.04 : 2m11s
15.04 : 2m30s
15.10 : 2m22s
I tested only once each version, not sure the if differences are caused by the provisioner .

With AWS driver :

14.04 : 10m44s / 8m53s / 5m41s
15.04 : 5m5s / 4m54s / 4m38s

Thanks @trecloux! Do you mind submitting PRs to update the default images in those drivers?

Ok, yes I will.
Could we directly jump to 15.10 ?

@trecloux I guess we could. @nathanleclaire wdyt?

Jumping to 15.10 as a new default seems reasonable to me.

Do you know how to find AWS EC2 image Ids for zones us-gov-west-1 and cn-north-1 ?

@trecloux I think both should show up using https://cloud-images.ubuntu.com/locator/ec2/

I've branches with changes needed for amazon, exoscale, azure, rackspace and digital ocean.
@dgageot are you taking care of google ?
Do you want me to submit one pull request per driver or a grouped one ?
Could you also please rename the issue ?

@trecloux Grouped one will do, since they are all the same purpose.

@trecloux Please take care of Google :-)

Ok, will update the PR


[SPARK-42763][BUILD] Upgrade ZooKeeper from 3.6.3 to 3.6.4

What changes were proposed in this pull request?
The pr aims to upgrade ZooKeeper from 3.6.3 to 3.6.4.
Why are the changes needed?
Release Notes: https://zookeeper.apache.org/doc/r3.6.4/releasenotes.html
Routine upgrade, ZooKeeper 3.6.4 contains some bugfixes:

Why is 3.6.4 but not higher?
https://github.com/apache/spark/pull/37507
https://github.com/apache/spark/pull/32572
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.

Let me close this in favor of the previous PR.

Let me close this in favor of the previous PR.

Ok

To @panbingkun , you can search the JIRA or PR before making a PR in order to avoid unintentional duplication.


[5.1] Improve exists queries

As suggested by @JosephSilber, this updates the exists() query builder method to do an actual exists query, which should be much faster for big tables.
Edit: I have tested this on MySQL, Postgres, and SQLite. I don't have a SQLServer lying around to test on, but the docs show that it should work fine.

Related #9627

FWIW, this is attempting to solve the same issue as #9627, but this PR is the most efficient way to do it. In my testing this resulted in an avg ~4000% improvement on a query with ~100,000 results.

It looks like this commit breaks select functionally at least for SqlServer
$invoiceExists = DB::table('foo')
     ->where('foo_field', '=', $id)
    ->exists();

Which results in a query like this:
select exists(from [foo] where [foo_field] = '42') as [exists]

This obviously fails, as it is not a valid SQL Statement

@HipsterJazzbo @taylorotwell
Same issue here, all my exists() calls are broken, i'm using Sql Server, could this be reverted to?:
// Illuminate\Database\Query\Builder.php

    /**
     * Determine if any rows exist for the current query.
     *
     * @return bool
     */
    public function exists()
    {
        $limit = $this->limit;
        $result = $this->limit(1)->count() > 0;
        $this->limit($limit);
        return $result;
    }

Thanks!!

@uxweb limit does not limit the rows counted; only the rows returned.
select count(*) from users limit 1

still counts all users. That's why this change was made in the first place.
We have to create a proper exists query for SQL Server, not revert to a terribly inefficient query.

Possibly all that needs to happen here for SQL Server to work is to change the Grammar::compileExists method to read
/**
     * Compile an exists statement into SQL.
     *
     * @param \Illuminate\Database\Query\Builder $query
     *
     * @return string
     */
    public function compileExists(Builder $query)
    {
        $query->select(1); // <-- this has been added.
        $select = $this->compileSelect($query);

        return "select exists($select) as {$this->wrap('exists')}";
    }

As it looks like there is no select element being generated in some cases. I don't have access to an SQL server, @uxweb or @schinken can you test this patch?

@xiphiaz The issue is that Sql Server engine does not support a query with this sintax:
select exists(from [foo] where [foo_field] = '42') as [exists]

it does not parse an exists sentence in the select section, it does in the where section.

@uxweb Oh right, ok so a totally different query is needed then. I think it is worth keeping this improvement for the other sql variants as it is a huge speed improvement for them.

@xiphiaz
Yeah, if the connection's type grammar is passed to the query builder, it can be done by overriding the compileExists on each grammar i think.
Here is a query that can be used for Sql Server:
select top 1 1 as [exists] from [foo] where [foo_field] = 1

The query mentioned by @uxweb is what I would prefer. This should perform very well

same issue here. all exist queries are broken. we're using sql server 2014.
i've never seen this syntax on sql server btw.
this select works:
select case when exists (select * from [foo] where [bar] = 42) then 1 else 0 end as [exists]
but i'd prefer ...
select top 1 1 from [foo] where [bar] = 42

I totally support @mherderich opinion. This is imho how it should be done

@HipsterJazzbo what is the isset($results[0]) check, and why does it return null otherwise?

Same issue on Oracle but need to confirm on actual code first. But based on tests, oracle fails too.

Confirmed. Oracle breaks on this PR. Reverted the old version and implement it on our own OracleGrammar to temporarily fix the issue.

Are you able to make  a PR fixing it? The majority cant really check this oracle stuff.

@arrilot, this is commit I've done to revert the old exists method. https://github.com/yajra/laravel-oci8/commit/dee08461d889dd971cd57a39ac2a6c5be311c687

We should be able to do something like this: http://stackoverflow.com/questions/3434437

@JosephSilber, yeah I think it's doable and I did try and run some execution test over 200k+ records and this seems to executes faster than the current one and the other link.
array:1 [▼
  0 => array:3 [▼
    "query" => "select t2.* from ( select rownum AS "rn", t1.* from (select count(*) as aggregate from tests where id = ?) t1 ) t2 where t2."rn" between 1 and 1"
    "bindings" => array:1 [▶]
    "time" => 0.68
  ]
]
Response time range (10 executions): 0.6+ to 0.8+ (CURRENT)

array:1 [▼
  0 => array:3 [▼
    "query" => "select count(1) "exists" from tests where id = ? and rownum = ?"
    "bindings" => array:2 [▶]
    "time" => 0.79
  ]
]
Response time range (10 executions): 0.4+ to 0.8+

array:1 [▼
  0 => array:3 [▼
    "query" => "select case when exists(select * from tests where id = ?) then 1 else 0 end as "exists" from dual"
    "bindings" => array:1 [▶]
    "time" => 0.64
  ]
]
Response time range (10 executions): 0.4+ to 0.6+

I'm just testing it thru browser response using this route:
get('test', function() {
    /** @var Connection $db */
    $db = DB::connection();
    $db->enableQueryLog();
    $exists = $db->table('tests')->whereId(1)->exists();
    dd($db->getQueryLog());
});

Will think over this and maybe will implement the fastest approach. Thanks for the suggestions!

@yajra Could you please test the performance if you just do "SELECT 1 as exists FROM...."? I guess that would be the best

@schinken, the performance is almost the same. But taking account the simplicity of the query, I've implemented "SELECT 1 as exists FROM....". Thanks for suggestions!

I also made a pull request for SqlServer, fixing this statement: https://github.com/laravel/framework/pull/11158


griddata and LinearNDInterpolator - why rescale is not default option?

#precisionmatters: today I found very long-standing bug in my filtering of some data using griddata, where values differ by few orders of magnitude. The resulting values from filtering lost precision very badly, even at double-precision floating. Rescaling resolved the problems! So the question is why rescale is not default option in scipy.interpolate? Should I be aware of any known side-effects?

I wouldn't expect losing precision "badly" for "a few order of magnitude" difference, but that's all relative to ones expectations.
Why is it not default? Two reasons I can think of straight away:

It isn't needed for many common use cases and takes computation time.
rescale was introduced in a later release than griddata itself, so it couldn't have been made the default because of backwards compat.

related issue:
https://github.com/scipy/scipy/issues/2975


⚠ Processors has degraded performance

In 785df1b, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 63 ms

Resolved: Processors performance has improved in fdcd819.


Issues with [No Name] buffer.

Oni has a few issues with the [No Name] buffer, mainly:

Its modified status is only visible when that buffer is selected. If a different tab/buffer is selected, the modified state goes away. This can easily be seen in the Config editor. Selecting the DefaultConfig shows a modified state.
In buffers mode, it isn't shown correctly on the tab bar. This can be seen by editing the [No Name] buffer and then opening a new buffer. The  [No Name] buffer can still be seen in :buffers but not on the bar.

It looks like the second bullet point is fixed now, the tab isn't vanishing at least. The modified status is still only visible when selected however.


Max Heap DS

Fully Detailed code with comments for each step.

Great work @Yugalgarg2002 please give this repo a star if it really helped you to complete your PR.
Also follow me for follow back if you want to get connected with me for any such open source contribution program. Thank you 😊


fix:  resolve browser mapping using bare imports (fix #11208)

Description

When a dep to be resolved has a browser field mapping to a bare import, Vite doesn't resolve that mapping, which is not as expected.
Fixes #11208
Note that the package env-entry in the reproduction was not treated as a dependency after scan phase, which means it will be loaded from where it was during runtime, and if it contains cjs syntax, an error will occur.
I don't know if the current handling is a expected behavior or not. If it should be pre-bundled, I think we only need to set the preserveSymlink field in resolvePlugin to true during scan phase.
Additional context

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Hello 😃 Any updates regarding this PR?

Waiting for review 😃.

Hi @sapphi-red
Is there anything I can help to get this minor bug fix merged? Thanks!

@sapphi-red I'm also looking to get this merged as it's a huge blocker for a project. More than willing to help out!

This is also a huge blocker for me :)

What can be done to get this unstuck? This is a big blocker for a lot of people.

Shall we ask additional reviewers like @bluwy

/ecosystem-ci run

Forgot to run ecosystem-ci, thanks for handling it patak 😄 Looks like we're good to go, the failures shown were also failing in main. Might be good to get another review before merging.


Verticale Lines are disappearing while zooming or panning

** Verticale Lines are disappearing while zooming or panning**
ScottPlot Version:5.0.34
Vertical lines should not disappear while zooming and panning.
It should remain in the same position until moved.

Different from you, please refer to # 3836

Hi @kareem469, thanks for reporting this! However, when I pan and zoom a plot with vertical lines, things look okay. Can you include sample code, a screenshot, or video that helps demonstrate more clearly what your issue is? https://www.screentogif.com/ may help. I'll close this issue for now, but with more information we can open it up again and take a look together. Thanks!

My requirement is, the vertical line should remain in a same position even after zooming or panning.
It should move based on dragging

I understand now! When you pan you want the data and axis limits to move, but you want the vertical axis lines to appear fixed in pixel space.
I think the way to do this is to use events like OnMouseDown, OnMouseMove, etc. to determine how far the plot has been panned and place the axis line at that place. I think you can use Plot.GetPixel() and Plot.GetCoordinates() to convert between coordinate space and pixel space. Let me know if you're unable to implement this effect and we can open this issue back up and take a closer look 👍

The vertical line is fixed at the X-axis coordinate, and regardless of how the chart is scaled or dragged, its X-axis coordinate should remain unchanged. I did not find any issues on my platform. I am using Windows 11 and VS2022, and the Scottplot version is 5.0.34
In addition, I posted # 3836, which mentioned that there is no problem with the vertical line, but there is a bug in the x coordinate position of the labeltext.
At 2024-05-22 09:50:27, "kareem469" @.***> wrote:
My requirement is, the vertical line should remain in a same position even after zooming or panning.
It should move based on dragging
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I understand now! When you pan you want the data and axis limits to move, but you want the vertical axis lines to appear fixed in pixel space.
I think the way to do this is to use events like OnMouseDown, OnMouseMove, etc. to determine how far the plot has been panned and place the axis line at that place. I think you can use Plot.GetPixel() and Plot.GetCoordinates() to convert between coordinate space and pixel space. Let me know if you're unable to implement this effect and we can open this issue back up and take a closer look 👍
Resources:

https://scottplot.net/demo/#draggable-axis-lines has very similar code

I tried storing previous X, Y values in mousedown and mousemove and using those values I am updating the vertical line while zooming and panning. It’s updating vertical line at previous position buts redrawing the line everytime (when zoomed or panned) which need to avoid.

kareem469，
Hi,your description is difficult to understand.
Do you need a cursor that can be moved with a mouse and its coordinates remain unchanged when zooming or dragging the chart?
At 2024-05-22 12:45:16, "kareem469" @.***> wrote:
I understand now! When you pan you want the data and axis limits to move, but you want the vertical axis lines to appear fixed in pixel space.
I think the way to do this is to use events like OnMouseDown, OnMouseMove, etc. to determine how far the plot has been panned and place the axis line at that place. I think you can use Plot.GetPixel() and Plot.GetCoordinates() to convert between coordinate space and pixel space. Let me know if you're unable to implement this effect and we can open this issue back up and take a closer look
Resources:
https://scottplot.net/demo/#draggable-axis-lines has very similar code
I tried storing previous X, Y values in mousedown and mousemove and using those values I am updating the vertical line while zooming and panning. It’s updating vertical line at previous position buts redrawing the line everytime (when zoomed or panned) which need to avoid.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

kareem469， Hi,your description is difficult to understand. Do you need a cursor that can be moved with a mouse and its coordinates remain unchanged when zooming or dragging the chart? At 2024-05-22 12:45:16, "kareem469" @.> wrote: I understand now! When you pan you want the data and axis limits to move, but you want the vertical axis lines to appear fixed in pixel space. I think the way to do this is to use events like OnMouseDown, OnMouseMove, etc. to determine how far the plot has been panned and place the axis line at that place. I think you can use Plot.GetPixel() and Plot.GetCoordinates() to convert between coordinate space and pixel space. Let me know if you're unable to implement this effect and we can open this issue back up and take a closer look Resources: https://scottplot.net/demo/#draggable-axis-lines has very similar code I tried storing previous X, Y values in mousedown and mousemove and using those values I am updating the vertical line while zooming and panning. It’s updating vertical line at previous position buts redrawing the line everytime (when zoomed or panned) which need to avoid. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.>

The vertical line should remain in same position even after zooming & panning
The line should move only when dragging


Update readme.md

Adding llama-server to MAC permissions

Updated this direction in active branch.
Thanks!


Fix method names and member names

Check style fixes regarding naming:

Method names start with a lowercase
Underscore is removed from member names

/cc @JonathanGiles @mayurid

Sorry - I rescind my +1 - Have one or two concerns here...

Can one of the admins verify this patch?

@JonathanGiles Luckily I create small commits, I removed all the public API changes and added them to the suppressions.


how to get iris_tracking's iris depth?

iris tracking's android demo, how to get iris_tracking's iris depth(included left depth and right depth)?

got it.

@bugmany how did you manage to find the depth? Any help would be greatly appreciated.

hi @fahadhaq,
you can refer to the file:
/mediapipe/graphs/iris_tracking/calculators/iris_to_render_data_calculator.cc
IrisToRenderDataCalculator::Process(CalculatorContext* cc) {
...
}
herein, i got it.


[YYModel] usage: modelCustomTransformFromDictionary returns void instead of BOOL

From WBEmoticonGroup:
- (void)modelCustomTransformFromDictionary:(NSDictionary *)dic {
    [_emoticons enumerateObjectsUsingBlock:^(WBEmoticon *emoticon, NSUInteger idx, BOOL *stop) {
        emoticon.group = self;
    }];
}

This returns void instead of BOOL. On iOS6 the return value seems to be considered as NO, which is more reasonable in my perspective, causing crash when entering emoticon panel.
However, on iOS7 and above void seems to be treated as YES and the code works. Is this by design or something?

Updated ~
Thanks!

This fixed the question. But seriously, do you have any idea why would the behavior be different between iOS 6 & 7? I have no clue about this. I didn't even know that void would be the same as 'YES`, for god's sake...

I don't know...

This is really, really weird. Gotta admit. Perhaps something to do with compiler and runtime. Never mind. I think we can let this go since the problem is solved.


Update karma dev dependency

Resolves npm audit warnings about 1 low, 5 moderate vulnerabilities related to versions before<EMAIL_ADDRESS>Closes #384.
/cc @humphd

And we're all green!  Merged.  Great work @0xazure!


Rspec 3 support

https://github.com/Sutto/rocket_pants/issues/94

Since there was no reply re. backwards, I've merged in a follow up patch - and this will be closed. Cheers for the original PR :)


Implement AVL tree insertion method

This method uses recursion to insert and balance the tree after insertion.

Tagged as MISTAKE. DELETED


config: remove stale config

What have you changed? (mandatory)
remove stale config
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
No
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
https://github.com/pingcap/docs/pull/673
https://github.com/pingcap/docs-cn/pull/929
Does this PR affect tidb-ansible update? (mandatory)
https://github.com/pingcap/tidb-ansible/pull/568

Do we need to update documents?

@ngaut yeah, done


Change UIShiny effect color

Hey there, first of all, I wanted to thank you for this beautiful project,
been looking for something like this for a few months now, finally found it!.
I wanted to know if there's a way to change the shine effect color
(through inspector, script, doesn't really matter).
That's it basically, thank you for your time.
Environment (please complete the following information):

Version - 3.2.0
Platform: Editor(Windows), Android, iOS
Unity version:  2019.4.3f1
Build options:  IL2CPP, MONO, .Net 4

@Happy-hub
Thanks for the feature request!


“Pusthaka Maaraata” screen - to be revamped (multiple requirements)

a.	I think there should be “discount” field in either percentage or Rupees.  This is disabled by default, in case enabled, the final price will be that much.
b.	Instead of  “Grahakara hesaru”, I propose the below solution
i.	Radio button to select either member/book seller/other/Donate

If member is selected, member list screen will come where we can chose which member is buying and buyers name is populated.
If book seller is selected, sellers’ list should come (we should add a new seller maintenance screen in “beeLupatti nirvahaNe”) and we should be able to select from that
If other, we can put a free flow text in name and a remarks column to fill in may be address or anything specific.
There are many times the book is given as honor or memento which does not generate money.  While it will reduce the book stock, unless we capture it, it will put an imbalance between stock vs sale.  These can be captured in Donate category.

Currently, this lists all books. Can we have a each row where we can select the books through a drop-down instead of all books list.. This will help in receipt size reduction and will be easy to cross verify with number of books sold to number of items in the receipt.

From Doddanna
@konkodi210


Including commits in a release?

We've been integrating the sentry-cli into our deployments and trying to register our releases and commits.
But this doesn't seem to work? In our deployment we use the sentry_cli docker image as per the documentation:
VERSION="get the git hash"
echo "Create a new release (${VERSION}) in sentry";
# Create a release
${SENTRY_CLI} releases new -p ${APPLICATION_NAME} ${VERSION}
echo "Associating commits with the release (${VERSION})";
# Associate commits with the release
${SENTRY_CLI} releases set-commits --auto ${VERSION}

This outputs the following:
Create a new release (406f9f1) in sentry
Created release 406f9f1.
Associating commits with the release (406f9f1)
+----------------+--------------+
| Repository     | Revision     |
+----------------+--------------+
| [the name of our repo] | ccc905d1448c |
+----------------+--------------+

But if we check the releases for the project in https://sentry.io/[org/project]/releases/ we can see releases as they are used in the issue reporting, but for all releases the actual commits are empty ?
I'm probably doing it wrong... ? but where?

I believe that this is not the right repository to talk about this problem as it's related to the sentry-cli package and not the PHP SDK. Please open the issue in getsentry/sentry-cli or in getsentry/sentry

@ste93cry Thanks. I thought it might have been related to the presentation in sentry.io since it's a PHP project... :smile: I'll re-ask in the CLI repo.

For the sake of privacy I masked repository name with organization/application and
have similar issue. I associate commits 'by hand' with
sentry-cli releases set-commits --commit<EMAIL_ADDRESS>3.136.0 --log-level=DEBUG
The output looks as follows:
| organization/application | f5028540c840 -> 91d80fcc9935 |
Commit hashes are real, but the commits list remains empty. Any hints?

@alubina I don't think we as the PHP SDK maintainers can help you with this, sentry-cli and the commit integration inside Sentry is not powered by this SDK and is totally separated.
You can check out if getsentry/sentry-cli#445 relates to your issue possibly and/or open up a support ticket if you are on sentry.io so that someone can check with you if your git integration is running and/or what might be happening.


[DRL4AMR] Add metadata arrays and new test function

This adds a couple new features:

Adds the element id and level number metadata arrays.
Adds a new test oscillatory test function which shows a case where a zz estimator will fail.

Oh, I realized you started with JS's branch, not mine... mine is much simplified and the ID rasterization can just be added to GetImage().

Or a similar function to GetImage can be added to do ID rasterization.

Right, I am reworking this in a branch starting from the point your nice changes were merged in.

Do you think you could merge in the simple version of GetImage? The class NCM should really be forgotten...

I think it's easier to drop this branch and add my changes starting from the simplified GetImage.  Look for a new branch to replace this one, hopefully today.

I did some simplifications to use Jakub's only way of refining geometries.
We only have one image size, which should be easier for the RL side.
I added a way to use an external seed.


How can I configure this to ignore IE11?

I see this repository depends on eslint-plugin-compat which allows us to specify which browsers to target. I am seeing a lot of errors in my js components dealing with IE11 issues. When I attempt to tell eslint-plugin-compat to only target chrome, this plugin still fails on IE11 issues.

You can override the list of targeted browsers by setting the settings.targets property in your ESLint config file. This property uses the browserlist syntax.
In the following example, eslint-plugin-compat would only target the last 2 versions of Chrome:
{
    "plugins": ["@salesforce/eslint-plugin-aura"],
    "extends": [
      "plugin:@salesforce/eslint-plugin-aura/recommended",
      "plugin:@salesforce/eslint-plugin-aura/locker"
    ],

    "settings": {
        "targets":
          "last 2 Chrome versions"
    },
}


Update widgets-development.md

Fix wrong broadcast documentation. See https://github.com/thingsboard/thingsboard/blob/master/ui-ngx/src/app/core/services/broadcast.models.ts#L27

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


V5 package.json: get ready for ES6 exports/imports

Per https://github.com/twbs/bootstrap/pull/28386 and others, bootstrap5 is going to be delivered as ES6  module (apart from pre-ES6 formats).  On the master branch package.json contains "main": "dist/js/bootstrap.js", "module": "dist/js/bootstrap.esm.js".  This is not going to work is Node modules.  I use node 13.12.0 and I get
/git/bootstrap master=$ node dist/js/bootstrap.esm.js
(node:30255) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
/git/bootstrap/dist/js/bootstrap.esm.js:6
import Popper from 'popper.js';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at wrapSafe (internal/modules/cjs/loader.js:1063:16)
    at Module._compile (internal/modules/cjs/loader.js:1111:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Module.load (internal/modules/cjs/loader.js:996:32)
    at Function.Module._load (internal/modules/cjs/loader.js:896:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47

• To be able to execute the above node-line, dist/js/ must containg package.json with "type":"module"
• To make import Alert from 'bootstrap';  work under NodeJS, the (root) package must contain conditional exports, something like "exports": {"import" : "dist/js/bootstrap.esm.js", "require":…}. In particular, NodeJS ignores "module".

Thanks for making the issue, patches welcome 🙂

Bootstrap cannot work on a Node.js environment, it works only in a Browser, so your example is not relevant here

I do not pretend to understand everything.  But my idea is to create a javascript file importing something (or everything) from bootstrap es6, then do tree shaking and end with a single file, created by Node.

to do what you're trying to achieve you've to use a bundler, something like Rollup or Webpack, it'll do exactly what you want 👍


eval data collator

This PR allows users to specify a data collator that is applied only to the evaluation dataset. This allows the application of data augmentation only to training or evaluation datasets

I'm going to merge this and open another PR with my suggested change since Alex doesn't have time to implement this right now.


tests: add tests for exception policy master switch - v2

Add tests for the new master switch config for the Exception policies. Added also a check in the exception-policy test that checks the default behavior, to ensure that things work as expected with the master switch disabled and the exception policies not set.
Previous PR: https://github.com/OISF/suricata-verify/pull/1036
Changes from previous PR:

update config name
add a test for performance mode in IDS
add min-version check for 7
add a exception-policy: disabled config setting to the default exception policy test, to check behavior remains the same

Replaced by: https://github.com/OISF/suricata/pull/8279

Closed by mistake

Replaced by https://github.com/OISF/suricata-verify/pull/1039


Translate RESTFullAPI -> Rebuild API

content/en/docs/clients/restful-api/rebuild.md

https://github.com/apache/incubator-hugegraph-doc/pull/129/commits/2b8d22f8888092eea21cffb758ae6f9b648a8fd0

due to the website branch has rebased, u should better do these steps to avoid  a lot of conflicts:

commit your changed files together (like trans: restapi, and it have a commit hash code)
rename your local website branch to another name (backup it)
pull the latest website branch from github & check out it
git cherry-pick XXHashCode in step 1 to get your commit files before

and problems could comment here, backup your local files for safety is better

a clean version is here: https://github.com/apache/incubator-hugegraph-doc/pull/130


ci: make external workload only use global DNS server in test

Follow https://github.com/cilium/cilium/pull/19483
Only use the first configured DNS server (the global one) in the
test. Also add nslookup debug and retry options in case they are needed
for further debugging and troubleshooting.
This fixes external workload CI test on master that failed like this:
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53
** server can't find clustermesh-apiserver.kube-system.svc.cluster.local: NXDOMAIN
Here an incorrect nameserver is being used by nslookup. This
<IP_ADDRESS> is configured (now) in GKE as a nameserver for some
local names. Don't know why nslookup sometimes chooses it if no server
address is given as a parameter.

Successful run of the external workloads test: https://github.com/cilium/cilium-cli/runs/6470646843?check_suite_focus=true
Removing DO NOT MERGE commit.


docs: fix deploy-site zh-cn lang filename

I noticed that there are two pages with the same title, but they are in different languages. It is a file-naming error, it is fixed in this PR.

nice catch, thank you!


Allow setting NPM version in engines

Thuis buildpack ignores the npm setting in engine in package.json, causing issues for builds that break on peerDependencies (npm 3+ just generates a warning, instead of an error)

This buildpack is no longer maintained, please follow the official guidelines at https://devcenter.heroku.com/articles/node-with-grunt


How to add hash tag to static assets

Some of my components refer the assets files in a css way:
import styled from 'styled-components';
const Logo = styled.div`
     background-image:  url('/static/logo.png'),
`

I want to use CDN for all the assets in /statics folder, it is possible to give a version tag( like BUILD_ID) to these files in production?
No clue, anyone has a advice?

Same issue here.
We tried using webpack file loader but seems not working for the current version of next.js.

We also encountered this problem, if the use of file-loader, you can generate a new image, but the node will be reported when compiled:
Error: Cannot find module '../ static / test.png'
So temporarily through githooks to deal with images to modify the problem, allowing add new image, but not allowed to submit modify the image:
.git/hooks/pre-commit
#!/bin/sh
STAGED_IMAGES=$(git diff --cached --name-only --diff-filter=M | grep -E ".(jpg|jpeg|png|gif)$")
if [[ $STAGED_IMAGES ]]; then
  echo $STAGED_IMAGES
  exit 1
fi

Actually this is pretty easy using the with-universal-configuration example.
File: ./env-config.js
const { version } = require('./package.json')

module.exports = {
  'process.env.VERSION': version
}

File: ./pages/index.js
export default () => (

    Build-Version: { process.env.VERSION }

)

I've created a fork to demonstrate this: HaNdTriX/next.js/tree/with-build-id/examples/with-universal-configuration
If you would like to add this to the examples I can create a standalone example and create a PR.
If this example solves your issue please close it.

Thanks @HaNdTriX ❤

@HaNdTriX @timneutkens
Thanks for the suggestion. It seems good for some issues here but not all of them.
For our project, we are working for images hash tags with next.js. It will be better to have the hash for every single image in order to make the browser cache the right one without extra clear work.
So the build version is only working for the release build code but not for the assets stuff as images and css files.

Thanks @HaNdTriX, but I have the same question here, what I want is the production code build all the static files with hash names, such as static/avatar.png -> static/avatar.1898173978.png.
The images may change sometime, we don't to refresh CDN on these files frequently, hash name can avoid this problem.
Could you give me some suggestions?

File: ./build-static.js
const fs = require('fs')
const path = require('path')
const execSync = require('child_process').execSync
const revHash = require('rev-hash')

const srcDir = 'static'
const distDir = 'buildstatic'
const hashData = {}

execSync(`rm -r -f ${distDir}`)

const buildStatic = (src, dist) => {
  fs.access(dist, (err) => {
    if (err) {
      fs.mkdirSync(dist)
    }

    initBuild(src, dist)
  })
}

const initBuild = (src, dist) => {
  const paths = fs.readdirSync(src)

  paths.forEach((_path) => {
    const subSrc = `${src}/${_path}`
    const subDist = `${dist}/${_path}`
    const stat = fs.statSync(subSrc)

    if (stat.isFile()) {
      const fileData = fs.readFileSync(subSrc)
      const pathObj = path.parse(subSrc)
      const hash = revHash(fileData)
      const hashDir = pathObj.dir.split(`${srcDir}/`)[1] || ''
      const hashKey = `${hashDir}/${pathObj.name}${pathObj.ext}`
      const filePath = `${path.parse(subDist).dir}/${pathObj.name}.${hash}${pathObj.ext}`

      if (pathObj.ext) {
        hashData[hashKey] = hash
        fs.writeFileSync(filePath, fileData)
      }
    } else if (stat.isDirectory()) {
      buildStatic(subSrc, subDist)
    }
  })

  fs.writeFileSync('static-hash.json', JSON.stringify(hashData))
}

buildStatic(srcDir, distDir)

run: node build-static.js
File: ./demo.js
import staticHash from 'static-hash'

const isPro = true

const staticRequire = (path) => {
  const hash = staticHash[path]
  const pathSplit = path.split('.')
  const fileName = pathSplit[0]
  const fileExt = pathSplit[1]
  const filePath = !isPro ? path : `${fileName}.${hash}.${fileExt}`

  let staticDir = '/static/'

  if (isPro) {
    staticDir = 'https://mycdn.com/static/'
  }

  return `${staticDir}${filePath}`
}

console.log(staticRequire('images/test.png'))

//

Thanks @taichenglu .
Similar solution here, I decide to add a suffix to static assets.
import qs from 'querystring';

export default (url) => {
    if (process.env.BUILD_VERSION) {
        const [urlPath, urlQueryString = ''] = url.split('?');
        const queryString = qs.parse(urlQueryString);
        queryString.v = process.env.BUILD_VERSION;
        return `${urlPath}?${qs.stringify(queryString)}`;
    }
}

We use Jenkins to deploy code, the BUILD_VERSION is assigned to Jenkins Job id.

@monopieces
If you do so, each release will update all static files, is not conducive to the client cache.


ESLint : Destructured object warning

Expected Behavior
Normally, destructured object haven't any convention.
Current Behavior
I have an ESLint warning on my destructured objects.
Steps to Reproduce
import React from `react`;

interface MyComponentStates{
    isMyState: boolean;
}

function MyComponent(): JSX.Element {

    const [IS_MYSTATE, SetMyState] = React.useState<MyComponentState>({
        isMyState: false,
    });

    const HandleChangeMyState = (): void => {
        SetMyState({
            ...IS_MYSTATE,
            isMyState: !IS_MYSTATE.isMyState,
        })
    }

    return(
        <MyChildComponent
            myState={IS_MYSTATE.isMyState}
            changeMyState= {HandleChangeMyState}
        />
    )
}

import React from 'react';

interface MyChildComponentProps{
    myState: boolean;
    changeMyState: (event: unknown, reason: "backdropClick" | "escapeKeyDown") => void;
}

function MyChildComponent(props: Readonly<MyChildComponentProps>): JSX.Element {

    // ESLint warining on myState, for ESLint myState must be in UPPER_CASE and prefixed by IS_
    const {myState, changeMyState} = props;

    return(

            <input
                 type="checkbox"
                 onChange={changeMyState}
            />

{myState.toString()}

    )
}

Context
This issue has been occurring in Collaboratio development. I haven't any more information that could be relevant.
Logs
Warning from ESLint in my context :

Checks

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm reporting the issue to the correct repository (for multi-repository projects)

Hi @galyfray,
I reopened this issue because I'm facing the same issue this time for a destructured function. I have a destructured function stored in a const variable which involve a warning from ESLint to change my variable name in UPPER_CASE. Nonetheless, in the fact that it's a function, it must be in PascalCase.


sashdat data for segmentation example

Hello, is it possible to provide a link to the sashdat file used in the segmentation example, i.e "soccer_player/sgf_256.sashdat". Would like to test out the segmentation example.
Thank you!

I added a small version of the example (I didn't want to make this project giant) here: tests/datasources/sgf_256_small.sashdat). It is the same file with less number of samples.

Thanks @ethem-kinginthenorth, will it be also be possible to provide a guide on how the sashdat should be constructed / formatted ? We are interns from SAS trying to implement segmentation on other applications. Thanks!

The easiest way would be to use ImageTable.load_files() to load both the raw images and ground truth images to two sashdat, and then use dljoin() to combine them into one using a unique id (for example, file name).


Creation of the page "HTMLCollection.length", in english and french

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[x] New documentation
[ ] Correction or update

Details

This exists: https://developer.mozilla.org/en-US/docs/Web/API/HTMLCollection/length


Update Role descriptions

Update the src/data/data_roles.js file with a description of what the responsibilities are for each role.

hey ! i would like to contribute .

hey may i contribute??

Hello! I would like to contribute.


Feature request -- Working together with React Transition Group (low level API)

At it's current state, React Router v4 will not work together with React Transition Group low level API for animated transitions on page navigation. I fiddled around, asked the question on multiple places without any answers. The only workaround I found was writing a custom switch which requires allot more boilerplate. Therefor I would like to request implemented functionality and documentation so that other people their JS fatigue remains leveled avoiding lots of fiddling.
Codepen
Stackoverflow URL
timdorr: Just follow these guidelines: #4351 (comment)
@timdorr
Doesn't seem to work following those guidelines?
This also doesn't seem to work
Everytime I report this missing feature/bug it gets closed by timdorr while he points at a solution that does not work (only for the CSS transition high level api) I provided multiple pens that show these features do not work however no one responds after the issue is closed.
Please provide a working example, prefferably on your nice looking documentation website aswell. This would benefit JS animators greatly.

Well, thanks for throwing me under the bus. Really appreciate it... >_>
The issue here is that the lifecycle hooks live on the component, but the Switch is what is transitioning in and out of the DOM. TransitionGroup expects to act on that component, not the eventual route component. There is no way around this, as it's just how React and TransitionGroup work.
What you can do is wrap Switch with a component that does have transition hook functions on it. You could then proxy those through to the component returned by Switch. Probably your best bet is just keeping it at the level of that top wrapping component, though.

@timdorr sorry for throwing you under the bus and thanks for explaining. Thats allot of overhead for animating with react router. One would be better off sticking to the version 3 for code clarity with js animation in mind.


Terminal panel pops up after closing interactive window

Issue Type: Bug
When closing the Python interactive window the Terminal panel shows itself.

VS Code version: Code 1.67.0 (57fd6d0195bb9b9d1b49f6da5db789060795de47, 2022-05-04T12:06:02.889Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No
Extensions (8)

Extension
Author (truncated)
Version

remotehub
Git
0.32.0

python
ms-
2022.6.0

vscode-pylance
ms-
2022.4.3

jupyter
ms-
2022.4.1001271738

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

remote-repositories
ms-
0.6.0

markdown-all-in-one
yzh
3.4.3

@sbatten what's the recommended approach to debug this? I'm not sure who is opening the terminal (but I know it's not notebook or iw as it's the editor tab close button)

I debugged this, and determined that when the interactive window is closed, it updates a context key which hide a view container in the panel. this incorrectly triggers the panel to show.


Add Pyraminx L3E

@OWelton-Rosie I need to implement a visualizer for pyraminx and as soon as I do that, I can merge this in

@OWelton-Rosie I need to implement a visualizer for pyraminx and as soon as I do that, I can merge this in

Sweet as! No rush. It'll take me a little while to write another json for L4E anyways.

making progress, got the basic shape down :)

Awesome! Glad to see it went through. If there's something I did wrong, please let me know.


#87 Password Reset Feature

Password Reset Flow
PocketBase has a built in password reset feature that happens on the actual PB domain side (i.e. https://pockethost-central.pockethost.io), but the experience isn't very customizable. This PR adds in a custom flow that will require an update on your side @benallfree . Instructions below:
New Password Reset UI

Default Flow
At this point, you get an email prompting you to reset your password. Here are the screenshots of the default behavior:

Custom Flow
Instead, we can modify inside of our PocketBase settings to point to our site instead. You can modify this by going into:

Settings
Mail Settings
Default "Password reset" email template

And modifying the link to look like: https://pockethost.io/login/password-reset/confirm?token={TOKEN}. This way when the page is loaded, the user will see this instead:

And once they enter in their new password, it will automatically log them in and send them to the dashboard.

This looks fantastic!
I'm curious, should we cover account activation here too? It's the same idea.
Thanks for #93 and #92 as well!

Great callout about the Account Verification
Account Verification

The PocketBase admin needs the URL updated as well. To:
https://pockethost.io/login/confirm-account?token={TOKEN}

This is awesome!! Thank you!


Benediction .tmb Adjustment Help

Not an issue per say, but just a curious ask. So I noticed the Benediction animation cancels as soon as you make any sort of movement which was why it felt so underwhelming in game. No other oGCD has this problem on WHM like Tetra, Aquaveil, Divine Benison, etx.; the animation goes off and you "slide-cast"
My question is: do we know the parameters that allow us to change the arguments for when an animation cancels on the .tmb file? A lot of it is listed unknown so I don't want to brute force it if there is information on it already. If there isn't I'm willing to experiment and figure it out, but yeah I wanted to know if anyone knew the triggers/arguements for when animations cancel.

@NaesKun  what your looking for is in the TBM its literally just called Animation Canceled by Movement and every tbm should have it. If you match the value to the duration of the PAP animation (or field just labeled: Animation) you should be good. if you are not sure, a default of 90 works for most jobs/actions.
So whats going on is every action can be essentially canceled by a few things:

movement
other actions
Auto-attacks
some other misc stuff like your char reacting to certain impact effects that trigger skeletal animations (paps).

So the Animation Canceled by Movement value only effects 1 on this list, or movement. Many caster abilities have lower values for this so it doesnt look odd seeing your character slide around with moving their legs all the time. RPR skills do this too since every action includes a post-scythe whirl animation that extends way beyond the actual GCD window (a really obvious example is Guillotine, you cant even see the whole attack if you roll your GCD).

Gotcha. I'm not a 3D editor so I don't work directly with the havok files or directly on blender so bear with me.
1.) Can the TBM you're mentioning "Animation Cancelled by Movement" be found under the pap or tmb editing sections of the VFX editor? If so where would I find them?
2.) What exactly am I matching? Would I be matching the "End" .tmb Animation field with that in the .pap?

Also is TBM the same as TMB?

Can you also post screenshots of the steps?

@XCZA bump again

So I'm finding a weird scenario diving into this:
Under specific animations such as Benediction, Aetherflow, Manaward, and the nonshared HW Awareness, there is no "Animation Cancelled by Movement" category I can adjust. Usually I can see the category under the track that contains "Fly Text" as seen in this photo:

But if we look at the old PLD HW Awareness, it simply does not exist.

Simply adding the category with the correct animation time doesn't change the animation cancelling so I'm at wits end on how to make this work. I feel like its a passive parsing issue where VFXEditor is not picking up that category properly for these skills.

@0ceal0t Can you take a look at this and let me know if its an issue with VFXEditor not parsing properly OR if its SE's raw file that doesn't have an Animation Cancel by Movement and VFXeditor cannot change this?

I also feel like this something directly related to the tmb so it should be resolvable within VFXEditor, but I want to make sure before I bark up this tree.

Did some digging, and it looks like it's also specified in the ActionTimeline sheet, which can't be edited in the same way that a .tmb or .pap file can. For example, this is the "end" animation of Sentinel. I'm not sure if there's a way to get around this, tbh

@0ceal0t Oh man the cancel by movement parameter is hard coded to another file entirely that we can't access through VFXEditor.
Then if that's the case just one question: Will you be exploring a potential workaround/direct editing of the ActionTimeline sheet through VFX Editor OR is it one of those hard tech ceilings that you can't magic your way through?
I want to ask so I can set expectations for myself, either hold on to hopium that you will be my savior, or accept the loss and move forward with something else.
Thanks for looking into this btw, regardless of answer I'm super grateful for the clarity provide by your research.

Sorry NaesKun i didnt notice the pings.
for any action that doesnt have an entry inthe tmb for 'animation canceled by movement' you can simply create one and it will work.
there are a couple other factors that effect both the duration of the animation and it being canceled.
in the Pap there categories:
Pap animation - this is One of the two places the duration for an animation exists, as well as what action the animation is tied to. (ie the button press)
animation - has data for stuff like facial posture or micro body movements, (these can often effect an animation and deleting this extra data can often fix issue you might be having with animation being canceled early or playing to fast)
Going back to TMB, the category:
Animation: also has a duration element and this interacts in interesting ways with the duration set in Pap animation. generally it helps to just match the two numbers, but sometimes you can do interesting stuff like artificially increase the playback speed of by making the TMB animation duration faster than the PAP animation duration (this doesnt always work however and is finicky in general).
If your Pap Animation duration is say 90 and the duration listed in the TMB Animation entry is 30, then your not going to see the full 90 seconds of the animation. You will likely just see the first 30 frames usually. Conversly if you have 30 in the PAP's Pap animation entry and 90 in the TMB's Animation entry you will see the the 30sec animation in the pap then the character will likely freeze in place for 60 frames (or just end the animation).
To get back to your original request, if you wanted to make the entire Benediction movement play out you need to do a few things.

figure out how long the actual animation is and make sure that value is set in both the PAP file's (animation Pap entry) and TMB (animation entry).
If there isnt one already listed in the Benediction (End) TMB, create an entry for animation canceled by movement, and set the value to match the full animation (or cancel were ever you'd prefer it to stop based on trial and error.
If this doesnt work and the skill is still ending early, this can be because: 1. your pressing another action, or if movement is canceling it there might be some entries in the Pap file that are categorized as 'animation' that might be cancling it as well, just try and error by clearing out entries in the Pap to see if one of them is interacting with the animation duration.

Also you can do the trial and error process faster by temporarily putting the Benediction PAP and TMB ontop of a GCD or OGCD that has a short cooldown (like weapon skills or tank stance for example).  once you got it how you like switch back the replaced files too Benediction.

Did some digging, and it looks like it's also specified in the ActionTimeline sheet, which can't be edited in the same way that a .tmb or .pap file can. For example, this is the "end" animation of Sentinel. I'm not sure if there's a way to get around this, tbh

Im pretty certain most skills are canceled by movement, exceptions are for example the 'Hit' window of any of the LB3s, after a certain duration your just locked in place. Almost every basic skill has or can have (by adding in the TMB) an animation canceled by movement entry.
You can effectively remove any animation canceled by movement by setting the value to match the duration of the animation, and making sure the PAP file's PAP animation duration and the TMB's animation entry duration are all the same. This will not change your ability from being canceled by other factors besides movement however, so just early weave your oGCD's if you wana see more of the animation.

@XCZA Thanks for the update. Its unfortunate that there's no work around and we cannot directly edit or access that sheet.
My VFX updates are vanilla changes that attempt to fix binders, timings, and bugs. They also attempt to add some omph to skill abilities or update the aesthetic to match the current skills in EW.
With that being said, I appreciate the PAP you provided, but I'm trying to fix the problem directly which seems like its not gonna happen.
I guess my question is if @0ceal0t will explore the possibility of finding where that animation cancel data is stored and allow us to edit it through VFX editor or not. Wondering if its worth keeping hopium going or to close this ticket.


Support for book lovers social networks

http://www.anobii.com/
http://www.goodreads.com/
http://www.librarything.com/

anobii: Requires a developer and secret key to be present in requests.
goodreads: Requires a developer key
librarything: See https://github.com/christianv/jquery-lifestream/pull/89

Other than scraping HTML out of the live profiles, I don't really see how we could get the data from anobii and goodreads :(

Closing the issue for now

@simong: Thanks for the report.


Pip install fails

If you create a new conda env, you can get a python version of e.g. 3.13, for which there are no sapien versions, causing the installation of maniskill to fail, I got the error message below, as a solution downgrade to python 3.10.
Suggestion, specify the python version as 3.10 in the setup.py script.
ERROR: Ignored the following versions that require a different python version: 1.10.0 Requires-Python <3.12,>=3.8; 1.10.0rc1 Requires-Python <3.12,>=3.8; 1.10.0rc2 Requires-Python <3.12,>=3.8; 1.10.1 Requires-Python <3.12,>=3.8; 1.11.0 Requires-Python <3.13,>=3.9; 1.11.0rc1 Requires-Python <3.13,>=3.9; 1.11.0rc2 Requires-Python <3.13,>=3.9; 1.11.1 Requires-Python <3.13,>=3.9; 1.11.2 Requires-Python <3.13,>=3.9; 1.11.3 Requires-Python <3.13,>=3.9; 1.21.2 Requires-Python >=3.7,<3.11; 1.21.3 Requires-Python >=3.7,<3.11; 1.21.4 Requires-Python >=3.7,<3.11; 1.21.5 Requires-Python >=3.7,<3.11; 1.21.6 Requires-Python >=3.7,<3.11; 1.26.0 Requires-Python <3.13,>=3.9; 1.26.1 Requires-Python <3.13,>=3.9; 1.6.2 Requires-Python >=3.7,<3.10; 1.6.3 Requires-Python >=3.7,<3.10; 1.7.0 Requires-Python >=3.7,<3.10; 1.7.1 Requires-Python >=3.7,<3.10; 1.7.2 Requires-Python >=3.7,<3.11; 1.7.3 Requires-Python >=3.7,<3.11; 1.8.0 Requires-Python >=3.8,<3.11; 1.8.0rc1 Requires-Python >=3.8,<3.11; 1.8.0rc2 Requires-Python >=3.8,<3.11; 1.8.0rc3 Requires-Python >=3.8,<3.11; 1.8.0rc4 Requires-Python >=3.8,<3.11; 1.8.1 Requires-Python >=3.8,<3.11; 1.9.0 Requires-Python >=3.8,<3.12; 1.9.0rc1 Requires-Python >=3.8,<3.12; 1.9.0rc2 Requires-Python >=3.8,<3.12; 1.9.0rc3 Requires-Python >=3.8,<3.12; 1.9.1 Requires-Python >=3.8,<3.12
ERROR: Could not find a version that satisfies the requirement sapien==3.0.0.b1 (from mani-skill) (from versions: none)
ERROR: No matching distribution found for sapien==3.0.0.b1

Python 3.9 to 3.11 should all work and are tested. I will ensure we do not permit 3.13

Sorry I stand corrected

Sorry actually we have a version of sapien that is built on 3.13, can you try installing this wheel instead?
https://github.com/haosulab/SAPIEN/releases/download/nightly/sapien-3.0.0.dev20241003+c877626f-cp313-cp313-manylinux2014_x86_64.whl
It is not on pypi yet.


Support vivaldi

Hi I use vivaldi browser instead of chrome.
However, I cannot login to Google calendar because popup window never opens.
I appreciate you if you fix this problem.
Thank you.

It seems like chrome.identity.getAuthToken doesn't work on Vivaldi.
https://github.com/Leko/crx-gcal-url-opener/blob/d2234f7f14e5952feebe9ad5dfffac3e2d0c0649/src/auth.ts#L3


Expose build_transaction through AsyncConnectionWrapper

This addresses https://github.com/weiznich/diesel_async/issues/115.
I feel I've made the section in async_connection_wrapper more complex than necessary, I can try tidy it up.
The tests don't pass right now but I know the core section compiles.
Let me know what you think about this solution!

Another approach would be to add a function that lets the user call functions inner.

Thanks for opening this PR. It addresses the problem that there is no build_transaction function for the async connection wrapper type, but I don't feel that the implemented solution is particular help full for down stream crates using that type because the indirectly exposed run function is still async.  I would prefer a solution that exposes a sync run method as well. That migth require duplicating or at least modifying parts of TransactionBuilder type, by having a separate run function depending on whether the inner connection type implements diesel::Connection or diesel_async::AsyncConnection.

Hmmmm yeah good point. In my case I'm actually already in an async environment so I think I'll just try switch to using diesel-async proper, so I'm going to just close this PR for now. I'll leave the issue open though.


Equivalence with php string functions

It turns out that we already cover many of php's string functions, so we may as well try to complete the suite
https://www.php.net/manual/en/ref.strings.php

https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html

https://hush-shell.github.io/std.html

https://pkg.go.dev/strings


Windows build of pre10

Would it be possible to supply a prebuilt binary for windows for version 2.3.0-pre10 please?

Latest release now contains a Windows build https://github.com/github/hub/releases/tag/v2.3.0


Implicit with cannot be replaced for BC 17 in Page (Extensions)

Describe the bug
Rec cannot be used for Expression of Page Field Properties.
To Reproduce
Steps and to reproduce the behavior:
I added a field to a sales order subform:
field("Prefix Test"; TestValue)
{
    ApplicationArea = All;
    Caption = 'Test';
    Editable = Rec.Type = Rec.Type::Item;
    QuickEntry = Type = Type::Item;
}

The expression Editable = Rec.Type = Rec.Type::Item; does not work with BC17. Altough the Expression without Rec does work perfectly.
Expected behavior
I would expect that I can use Rec to avoid implicit with.
Screenshots

5. Versions:

AL Language: 6.0.308593

Business Central: Version: DE Business Central 17.0 (Plattform 16.0.15023.0 + Anwendung 17.0.15096.0)

Tested this on 17.0.16297.0 (Preview) and the enabled/Visible/...  Expression of Page Field Properties are working with Rec.xxx there.
Still causes pages to give an error on 16.5.15897.16301 though.
Keeping the Rec.xxx fixes for the 17 release then.
People using extensions to quickfix this issue on 16.5.15897.16301 versions and before will have to check all the fixes on page field properties and undo those changes.

You are trying to implement a new BC 17 feature to BC 16 (runtime 5.x), right?
Follow the discussion here: https://twitter.com/KarolakNatalie/status/1302858502688444416?s=20

This is only supported on the BC 17 runtime. The Quick/Warning should be (will be) disabled for runtime version below < 6.0

Hi @pri-kise, we can't reproduce this issue on our latest version. Please feel free to reopen if you think it is still present.


Changed Whitelist terms to Allow List  :D

This update is only cosmetic and doesn't update the back-end yet.

I am curious for community input on this one.  Whitelist is a common, approved, technical term for what is being done here.
I'm not against this one necessarily but curious to others opinions

I am curious for community input on this one. Whitelist is a common, approved, technical term for what is being done here.
I'm not against this one necessarily but curious to others opinions

This is along the same lines as the rename by GitHub from master to main for the default branch.

It's included on, e.g., https://www.acm.org/diversity-inclusion/words-matter

Well approved then


NotImplementedError: Differentiation rule for 'pmax' not implemented

I'm using xmap to do model parallel experiments. I updated the spmd_mnist_classifier_fromscratch.py script to use xmap as follows:
from functools import partial
import time

import numpy as np
import numpy.random as npr

from jax import jit, grad, pmap
from jax.scipy.special import logsumexp
from jax.lib import xla_bridge
from jax.tree_util import tree_map
from jax import lax
import jax.numpy as jnp
import datasets

import jax
from jax.nn import one_hot, relu
from jax.experimental.maps import xmap, mesh

def init_random_params_single(scale, layer_size, rng=npr.RandomState(0)):
  return scale * rng.randn(layer_size[0], layer_size[1])

def named_predict(w1, w2, image):
  hidden = relu(lax.pdot(image, w1, 'inputs'))
  logits = lax.pdot(hidden, w2, 'hidden')
  return logits - logsumexp(logits, 'classes')

def named_loss(w1, w2, images, labels):
  predictions = named_predict(w1, w2, images)
  num_classes = lax.psum(1, 'classes')
  targets = one_hot(labels, num_classes, axis='classes')
  losses = lax.psum(targets * predictions, 'classes')
  return -lax.pmean(losses, 'batch')

def named_update(w1, w2, images, labels):
  grads = grad(named_loss)(w1, w2, images, labels)
  grads = [lax.psum(dw, 'batch') for dw in grads]
  return [w1 - step_size * grads[0], w2 - step_size * grads[1]]

if __name__ == "__main__":
  hidden_size = 1024
  layer_sizes = [784, hidden_size, 10]
  param_scale = 0.1
  step_size = 0.001
  num_epochs = 10
  batch_size = 128

  train_images, train_labels, test_images, test_labels = datasets.mnist()
  num_train = train_images.shape[0]
  num_complete_batches, leftover = divmod(num_train, batch_size)
  num_batches = num_complete_batches + bool(leftover)

  def simple_data_stream():
    rng = npr.RandomState(0)
    while True:
      perm = rng.permutation(num_train)
      for i in range(num_batches):
        batch_idx = perm[i * batch_size:(i + 1) * batch_size]
        images, labels = train_images[batch_idx], train_labels[batch_idx]
        yield images, labels

  batches = simple_data_stream()

  w1 = init_random_params_single(param_scale, layer_size=[784, hidden_size])
  w2 = init_random_params_single(param_scale, layer_size=[hidden_size, 10])

  in_axes = [['inputs', 'hidden', ...],
           ['hidden', 'classes', ...],
           ['batch', 'inputs', ...],
           ['batch', ...]]

  update = xmap(named_update, in_axes=in_axes, out_axes=[...],
            axis_resources={'hidden': 'x'})

  devices = np.array(jax.local_devices())

  for epoch in range(num_epochs):
    start_time = time.time()
    for _ in range(num_batches):
      images, labels = next(batches)
      with mesh(devices, ('x')):
        w1, w2 = update(w1, w2, images, labels)
    epoch_time = time.time() - start_time

    print("Epoch {} in {:0.2f} sec".format(epoch, epoch_time))

I get the following exception/error (full exception stack attached as a text file) when I run the above script as python <script_name.py>:
/usr/local/lib/python3.7/dist-packages/jax/_src/numpy/lax_numpy.py in _reduction(a, name, np_fun, op, init_val, has_identity, preproc, bool_op, upcast_f16_for_computation, axis, dtype, out, keepdims, initial, where_, parallel_reduce)
   1995       raise NotImplementedError(f"Named reductions not implemented for jnp.{name}()")
-> 1996     result = parallel_reduce(a, dims)
   1997   else:

/usr/local/lib/python3.7/dist-packages/jax/_src/lax/parallel.py in pmax(x, axis_name, axis_index_groups)
    147   out_flat = pmax_p.bind(*leaves, axes=axis_name,
--> 148                          axis_index_groups=axis_index_groups)
    149   return tree_util.tree_unflatten(treedef, out_flat)

/usr/local/lib/python3.7/dist-packages/jax/core.py in bind(self, *args, **params)
    263     tracers = map(top_trace.full_raise, args)
--> 264     out = top_trace.process_primitive(self, tracers, params)
    265     return map(full_lower, out) if self.multiple_results else full_lower(out)

/usr/local/lib/python3.7/dist-packages/jax/interpreters/ad.py in process_primitive(self, primitive, tracers, params)
    272       msg = f"Differentiation rule for '{primitive}' not implemented"
--> 273       raise NotImplementedError(msg)
    274     primal_out, tangent_out = jvp(primals_in, tangents_in, **params)

UnfilteredStackTrace: NotImplementedError: Differentiation rule for 'pmax' not implemented

Is this expected? If yes, is there any current workaround for this right now?
Thanks.
Error_differential_rule_for_pmax_doesnt_exist.txt

It's much like it says on the tin: differentiation for pmax, which is used inside logsumexp is just not implemented yet.

I tried to prepare an MNIST classification example using xmap and faced the same problem.
Is there any way to overcome this? Or I do something inherently wrong, and things should be done other way?

I ran into the same issue with pmax. What's ironic is that many common uses of pmax (e.g. for softmax) are actually under a stop_gradient, so the differentiation rule isn't truly needed. However, even so, the error style fires.
For future reference, my workaround was to define stop_gradient_pmax which does jax.lax.stop_gradient(jax.lax.pmax(...)) but with a custom gradient implementation so as to sidestep the error above. Here's an implementation which I've been using:
@partial(jax.custom_jvp, nondiff_argnums=(1,))
def stop_gradient_pmax(x, axis_name):
  """Same as `jax.lax.stop_gradient(jax.lax.pmax(x, axis_name))`, but which works with autodiff.

  Autodiff for `jax.lax.pmax` is unimplemented, https://github.com/google/jax/issues/7167.
  """
  return jax.lax.pmax(x, axis_name)

@stop_gradient_pmax.defjvp
def _stop_gradient_pmax_jvp(axis_name, primals, tangents):
  x, = primals
  t, = tangents
  return stop_gradient_pmax(x, axis_name), jnp.zeros_like(x)

Thanks for pinging this.
Hmm, if you do a stop_gradient on the argument to pmax (rather than its output), I don't think its differentiation rule should fire... have you found that's not the case?

Ah, thank you! Yes that works.
(There's a second unrelated issue that follows: https://github.com/google/jax/issues/20335.)


Docs not building, with warning: html_static_path entry does not exist

Details

Project URL: https://github.com/Ewan82/sentinelSimulator
Build URL (if applicable): http://sentinelsimulator.readthedocs.io/en/latest/
Read the Docs username (if applicable): Ewanp88

Expected Result
I am trying to build my updated docs from github that have compiled fine with Sphinx locally but RTD is not updating old documentation.
Actual Result
Receive one warning when trying to build:
copying static files... WARNING: html_static_path entry '/home/docs/checkouts/readthedocs.org/readthedocs/templates/sphinx/_static' does not exist
Have done as suggested elsewhere and set html_static_path in conf.py from:
html_static_path = ['_static']
to
html_static_path = []
But still no joy. Any help is much appreciated! Thanks for your time,
Ewan

Fixed by specify path to conf.py file in advanced settings and selecting Single Version option.


When I run my code, nothing happens? Plz help

I have trained my model and all the files are where they should be (I think) but when i run my file (below) nothing happens. I get no errors, but i cant use my chatbot or anything.
import random
import json
import pickle
import numpy as np
import nltk
from nltk.stem import WordNetLemmatizer
from tensorflow import keras
from keras.models import load_model
lemmatizer = WordNetLemmatizer()
intents = json.loads(open('intents.json').read())
words = pickle.load(open('words.pkl', 'rb'))
classes = pickle.load(open('classes.pkl', 'rb'))
model = load_model('chatbot_model.h5')
def clean_sentence(sentence):
sentence_words = nltk.word_tokenize(sentence)
sentence_words = [lemmatizer.lemmatize(word) for word in sentence_words]
return(sentence_words)
def word_bag(sentence):
sentence_words = clean_sentence(sentence)
bag = [0] * len(words)
for w in sentence_words:
for i, word in enumerate(words):
if word == w:
bag[i] = 1
return np.array(bag)
def predict_class(sentence):
wb = word_bag(sentence)
result = model.predict(np.array([wb]))[0]
ERROR_THRESHOLD = 0.25
result = [[i, r] for i, r in enumerate(result) if r > ERROR_THRESHOLD]
result.sort(key=lambda x: x[1], reverse=True)
return_list = []
for r in result:
    return_list.append({'intent': classes[r[0]], 'probability': str(r[1])})
return return_list

hello did you fix this issue ? having the same one


Add model format flexibility of AutoModel.from_pretrained()

Since I cannot load models from the huggingface hub (see https://github.com/mistralai/mistral-finetune/issues/27), I am downloading models to a local directory. However, they are either in the format
generation_config.json, pytorch_model.bin.index.json, adapter, special_tokens_map.json, added_tokens.json, pytorch_model-00001-of-00003.bin, tokenizer.model, pytorch_model-00002-of-00003.bin, tokenizer_config.json,
config.json, pytorch_model-00003-of-00003.bin

or in the format
config.json, model-00004-of-00006.safetensors, model-00001-of-00006.safetensors , model-00005-of-00006.safetensors,
model-00002-of-00006.safetensors,  model-00006-of-00006.safetensors,
model-00003-of-00006.safetensors,  model.safetensors.index.json

Could you either add the flexibility of AutoModel.from_pretrained() to wrapped_model.py or explain how I can store my huggingface models locally in a format that wrapped_model.py can digest?

Same with the save_pretrained.  I did not see that functionality in the current code base and hence can not leverage open source packages.