PokerStars ClubHouse Games

I've been trying to parse some ClubHouse games to keep track of how a bunch of friends are doing (PokerStars statistics don't seem to be loading).
A club House game seems to have a slightly different format to the other game types I've seen.
PokerStars Home Game Hand #214296101572: {Club #32872232} Tournament #2905961235, $5.00+$1.25+$0.50 USD Hold'em No Limit - Level III (25/50) - 2020/05/22 21:56:50 WET [2020/05/22 16:56:50 ET] Table '2905961235 1' 9-max Seat #7 is the button
So I've amended the REGEX so accept the new pieces of information (Home Game, {Club #3940394}, factoring in the original timezone date, and a 3rd +$1.50 in the prize values (due to playing a knockout variant)
` _header_re = re.compile( r"""
                    ^PokerStars.+?\s+                             # Poker Room
                    Hand\s+\#(?P<ident>\d+):\s+                   # Hand history id
                    (\{Club\s+\#(?P<club_ident>\d+)\}\s+)?        # Club id
                    (Tournament\s+\#(?P<tournament_ident>\d+),\s+ # Tournament Number
                     ((?P<freeroll>Freeroll)|(                    # buyin is Freeroll
                      \$?(?P<buyin>\d+(\.\d+)?)                   # or buyin
                      (\+\$?(?P<knockrake>\d+(\.\d+)?))?          # knockout or rake value depends on how you're playing
                      (\+\$?(?P<rake>\d+(\.\d+)?))?               # and rake if this is a knockout game
                      (\s+(?P<currency>[A-Z]+))?                  # and currency
                     ))\s+
                    )?
                    (?P<game>.+?)\s+                              # game
                    (?P<limit>(?:Pot\s+|No\s+|)Limit)\s+          # limit
                    (-\s+Level\s+(?P<tournament_level>\S+)\s+)?   # Level (optional)
                    \(
                     (((?P<sb>\d+)/(?P<bb>\d+))|(                 # tournament blinds
                      \$(?P<cash_sb>\d+(\.\d+)?)/                 # cash small blind
                      \$(?P<cash_bb>\d+(\.\d+)?)                  # cash big blind
                      (\s+(?P<cash_currency>\S+))?                # cash currency
                     ))
                    \)\s+
                    -\s+.+?\s+                                    # localized date
                    \[(?P<date>.+?)\]                             # ET date
                    """,
    re.VERBOSE,
)`

This got rid of the error I was getting due to match(ident), but now I'm being told that the game type is incorrect but it's evaluating:
"[CLUB #3287223] TOURNAMENT #2905961235, $5.00+$1.25+$0.50 USD HOLD'EM"
rather than just the HOLD'EM piece.

unsubscribehttps://github.com/notifications/unsubscribe-auth/AA74VUV25MMYNFFMXY6KBHDRTKFATANCNFSM4NJTO7JQ

From: Rob McGhee<EMAIL_ADDRESS>Sent: Monday, May 25, 2020 3:28 PM
To: pokerregion/poker<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [pokerregion/poker] PokerStars ClubHouse Games (#42)
I've been trying to parse some ClubHouse games to keep track of how a bunch of friends are doing (PokerStars statistics don't seem to be loading).
A club House game seems to have a slightly different format to the other game types I've seen.
PokerStars Home Game Hand #214296101572: {Club #32872232} Tournament #2905961235, $5.00+$1.25+$0.50 USD Hold'em No Limit - Level III (25/50) - 2020/05/22 21:56:50 WET [2020/05/22 16:56:50 ET] Table '2905961235 1' 9-max Seat #7 is the button
So I've amended the REGEX so accept the new pieces of information (Home Game, {Club #3940394}, factoring in the original timezone date, and a 3rd +$1.50 in the prize values (due to playing a knockout variant)
_header_re = re.compile( r""" ^PokerStars.+?\s+ # Poker Room Hand\s+#(?P\d+):\s+ # Hand history id ({Club\s+#(?P<club_ident>\d+)}\s+)? # Club id (Tournament\s+#(?P<tournament_ident>\d+),\s+ # Tournament Number ((?PFreeroll)|( # buyin is Freeroll $?(?P\d+(.\d+)?) # or buyin (+$?(?P\d+(.\d+)?))? # knockout or rake value depends on how you're playing (+$?(?P\d+(.\d+)?))? # and rake if this is a knockout game (\s+(?P[A-Z]+))? # and currency ))\s+ )? (?P.+?)\s+ # game (?P(?:Pot\s+|No\s+|)Limit)\s+ # limit (-\s+Level\s+(?P<tournament_level>\S+)\s+)? # Level (optional) ( (((?P\d+)/(?P\d+))|( # tournament blinds $(?P<cash_sb>\d+(.\d+)?)/ # cash small blind $(?P<cash_bb>\d+(.\d+)?) # cash big blind (\s+(?P<cash_currency>\S+))? # cash currency )) )\s+ -\s+.+?\s+ # localized date [(?P.+?)] # ET date """, re.VERBOSE, )
This got rid of the error I was getting due to match(ident), but now I'm being told that the game type is incorrect but it's evaluating:
"[CLUB #3287223] TOURNAMENT #2905961235, $5.00+$1.25+$0.50 USD HOLD'EM"
rather than just the HOLD'EM piece.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/pokerregion/poker/issues/42, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA74VUV25MMYNFFMXY6KBHDRTKFATANCNFSM4NJTO7JQ.


Mirror pro repo

I'm imagining lots of reasons why this might be inadvisable, but is there a way (or reasons why not to) mirror the Anaconda "pro" repository?  Seems the "free" repository is the only one to mirror when referring to the 'defaults' or 'anaconda' repo, but when I look at anaconda.org and search for a package under "pro" (e.g. accelerate) it appears as being under the "anaconda" owner.  Recommendations or thoughts to work around this?

@dmarkwat It is not a technical challenge to update conda-mirror to do this.
I have no information regarding the legality of mirroring the pro repo (i am not a lawyer).
I am going to start having to mirror the r channel internally as we are starting to get some requests for r packages. The proposed update (PR incoming shortly) will allow you to pass in the fully qualified channel --upstream-channel https://repo.continuum.io/pkgs/r instead of just a channel (like --upstream-channel conda-forge.  Incidentally this will allow you to mirror the pro channel if you so choose.

Sounds great!  That would work perfectly for the automation I'm setting up around this.  Looking forward to the PR.

closed by #25


Umpleonline readOnly argument setting for embedding allows certain changes such as tab adding

The following page shows some features of UmpleOnline allowing embedding. This is a general feature and the following shows examples only
http://cruise.eecs.uottawa.ca/umpleonline/iframeexample.html
However, certain changes are needed;

It should not be possible to add or delete tabs in embedded mode
The visit count at the bottom should not be displayed if noChrome is set

Also in read only mode it should not be possible to load examples from the menu

Increased priority, since with tasks we want to ensure people looking at task results cannot modify them


Update 5.4

Contact:
Sergey L. Napelenok, U.S. EPA
Type of code change:
Bug fix
Description of changes:
Inadvertently, DDM-3D and ISAM code were not functioning with the CMAQ 5.4 release when the potential vorticity option was enabled during compilation. This pull request corrects both models.
Summary of Impact:
This update does not change the base model outputs in any way.  However, it enables correct output of sensitivity and apportionment fields for simulations where potential vorticity calculations are enabled.
Tests conducted:
Testing was done on the NE benchmark domain, where reasonable outputs for both sensitivities and apportionment outputs were observed.  Additionally, test were performed on the hemispheric domain where output was compared between ISAM, DDM-3D, and brute force simulations.
This PR passes the following build and run checks:

Compilers: Intel, GNU, Portland Group
Compile Mode: Optimized, Debug
Model Scenario: 2016 Southeast U.S. Benchmark

Sorry... This was a mistake.  Closing.


Error message while switching projects

Hello,
first of all, THANK YOU VERY MUCH for your work, your .emacs.d is one of the bests I have ever seen and I'm already using it (with minor modifications) as my main emacs configuration.
 In fact, the project is switched to the selected one, but when the scratch buffer is being openned the message below pops into the Messages buffer.
doom/wg-projectile-switch-project: Symbol’s function definition is void: doom-reload-scratch-buffer
For my setup, I commented the line 16 at core/defuns/defuns-workgroup.el, and that solved the problem for me.

Sorry for the late reply!

first of all, THANK YOU VERY MUCH for your work, your .emacs.d is one of the bests I have ever seen and I'm already using it (with minor modifications) as my main emacs configuration.

Thank you! And you're very welcome.

Symbol’s function definition is void: doom-reload-scratch-buffer

The correct function name is doom-scratch-reload. If you correct that on the line you commented out, it should work fine. I renamed it a while back and forgot to update defuns-workgroup.el, my bad!
I'm doing a huge rewrite of my config at the moment (should be done this weekend), so I may not push a fix for the current version.

I'll close this issue since it's resolved in v2, which I've just merged to master.


Reward votes sync fix

Ignore one vote but process others if one vote already validated

prnet cleaned-up successfully


modalDescription method doesn't update view for Create action requiresConfirmation overlay

Package
filament/forms
Package Version
v3.2.72
Laravel Version
v11.5.0
Livewire Version
v3.4.11
PHP Version
v8.3.6
Problem description
When adding a confirmation modal to the create action using the requireConfirmation() method, I can set a modalTitle as expected, however setting modalDescription does not overwrite the description.
Looking at the code, I think the issue is in:
/filament/actions/src/Concerns/CanRequireConfirmation.php
If I overwrite the $this->modalDescription line to be:
        $this->modalDescription ??= fn (MountableAction $action): ?string => $action->evaluate($condition) ? __('filament-actions::modal.confirmation') : null;
it works for my use case. However I have no clue if this breaks other things.
Expected behavior
I expect the description of the confirmation modal to be overwritten from the default "Are you sure you would like to do this?" to the value I provided,
Steps to reproduce
On the Create page for any resource (e.g. CreateSheet), add the following code into the class to turn on the confirmation modal, the title will be correctly set to 'TEST', but the description won't get set to 'TEST2'

    protected function getCreateFormAction(): Action
    {
        return Action::make('create')
            ->modalHeading('TEST')
            ->modalDescription('TEST2')
            ->requiresConfirmation()
            ->action(fn () => $this->create());
    }

To replicate on the reproduction repository, create a user (in the db) and then try and create a sheet record, when you click 'create' and see the modal you won't get TEST2 as the modal description.
Reproduction repository
https://github.com/MikeCodesSomething/filament-create-modal-description-replication/
Relevant log output
No response

@MikeCodesSomething Try rearranging your code to this:
protected function getCreateFormAction(): Action
{
    return Action::make('create')
        ->modalHeading('TEST')
        ->requiresConfirmation()
        ->modalDescription('TEST2')
        ->action(fn () => $this->create());
}

I suppose the order of the methods defined matters in this case. I would still constitute this as a bug though.

Check out the content of requiresConfirmation(), it is a shortcut for setting the description, hence this behaviour

Thanks both!


Add Travis Config File

@schickling Would you be able to add this repo to Travis? I would do so, but since this repository belongs to you, I don't have access to Travis for this repo.

This has some issues. Resubmitting.


[azservicebus] nil pointer dereference in GetRecoveryKind()

pkg: github.com/Azure/azure-sdk-for-go/sdk/messaging/azservicebus
version: v0.3.4
go version: go 1.17

Description
In testing I'm seeing a nil pointer dereference error generated by this line within errors.go.
Even though other functions calling GetRecoveryKind() have checked to see if err == nil, they have not accounted for the fact that an error interface containing a nil pointer to an amqp.Error type is non-nil. So there is still a possibility of a nil pointer error here.
The following code may make the problem clearer:
        var detachErr *amqp.Error
	var err error
	err = detachErr

	// err is not nil, even though it contains a nil pointer
	if err == nil {
		return
	}

	var amqpErr *amqp.Error
	if errors.As(err, &amqpErr) {
		// trying to access amqpErr.Condition will cause a panic
		t.Log(amqpErr.Condition)
	}

It would be great if you could address this problem.

I think I'm running into a similar "non-nil interface pointing at nil" issue, seeing a lot of these in my logs:

link detached, reason: *Error(nil)

This might be related to https://github.com/Azure/go-amqp/pull/123

Thanks for the quick response @richardpark-msft !

No problem, thanks to both of you for reporting it!
@arussellf5, @ryepup: This fix has been released in v0.3.5 of azservicebus: link


Cannot set the connection pool to Sandbox

Environment

Elixir version (elixir -v): 1.2.3
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): Postgres 9.5
Ecto version (mix deps): ecto 2.0.0-beta.2
Database adapter and version (mix deps): poolboy 1.5.1, postgrex 0.11.1, db_connection 0.2.5
Operating system: Windows 10 64bit

Current behavior
config/test.exs
use Mix.Config

config :myapp, Myapp.Repo,
  pool: Ecto.Adapters.SQL.Sandbox

config/config.exs
use Mix.Config

config :myapp, Myapp.Repo,
  adapter: Ecto.Adapters.Postgres,
  database: "myapp",
  username: "postgres",
  password: "password"

test_helper.exs
ExUnit.start()

# Set the pool mode to manual for explicitly checkouts
Ecto.Adapters.SQL.Sandbox.mode(Myapp.Repo, :manual)

myapp_test.exs
defmodule Myapp.MyappTest do
  use ExUnit.Case
  import Logger

  setup do
      :ok = Ecto.Adapters.SQL.Sandbox.checkout(Repo)
    end

    test "create valid user", %{uapi: uapi} do
      user = %{username: "username", password: "password", email<EMAIL_ADDRESS>      Myapp.Repo.insert(User.changeset(%User{}, user))
    end

end

Result
$ mix test
** (RuntimeError) cannot configure sandbox with pool DBConnection.Poolboy.
To use the SQL Sandbox, configure your repository pool as:

      pool: Ecto.Adapters.SQL.Sandbox

    (ecto) lib/ecto/adapters/sql/sandbox.ex:269: Ecto.Adapters.SQL.Sandbox.mode/2
    (elixir) lib/code.ex:363: Code.require_file/2
    (elixir) lib/enum.ex:604: Enum."-each/2-lists^foreach/1-0-"/2
    (elixir) lib/enum.ex:604: Enum.each/2
    (mix) lib/mix/tasks/test.ex:186: Mix.Tasks.Test.run/1

Expected behavior
I'm expecting the pool to be set to Sandbox and for the test to run.

You are using different names, afaik. In your config you use:
config :myapp, Myapp.Repo,
  pool: Ecto.Adapters.SQL.Sandbox

But in your test helper you use:
Ecto.Adapters.SQL.Sandbox.mode(Persistence.Repo, :manual)

which one is it? Persistence.Repo or Myapp.Repo? :)

My bad - missed some references when renaming the app - I've updated the original post to all say Myapp.

Is that your complete config/config.exs file? You will need to import the config/test.exs to use it, e.g.
import_config "#{Mix.env}.exs"

I'll try and post a complete example project when I get home.
I've included the test.exs and even tried setting pool directly in the
config.exs.
On 5 Apr 2016 7:27 pm, "James Fish"<EMAIL_ADDRESS>wrote:

Is that your complete config/config.exs file? You will need to import the
config/test.exs to use it, e.g.
import_config "#{Mix.env}.exs"
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/elixir-lang/ecto/issues/1355#issuecomment-205932784

Tried to replicate this in a fresh project and failed.
I deleted the _build folder in my project and then re-ran the test and it passed :-S
So solved by rebuilding the project.

I experienced I the same problem recently, and it turned out I had  MIX_ENV set to prod in the environment. So MIX_ENV=test mix test solved this issue for me in this particular instance.

@jebbench's solution didn't work for me.

mine was because my docker-compose.yml file had explicitly set MIX_ENV  to dev. so when running test, i had to set that to test like this: MIX_ENV=test mix test


Daemon fails to start because Keyring requires extra password

Hi!
First of all thanks for the development of this useful piece of software. :-)
But I do have a problem, when I try to start the daemon with
onedrived start
A second after starting I get:
Starting onedrived ... FAILED ERROR: Child exited immediately with exit code 127
When I start it with
onedrived start --debug
it's asking me for the keyring password. So, I assume, that when I try to start the daemon without --debug it's trying to get the password too?
Are there any additional logfiles to check, what's really going on?
Thanks for any support.
Regards,
Andreas

Daemon mode and debug mode basically do the same except that daemon mode redirects output to log while debug mode prints to stdout. If keyring asks for password in debug mode it should do the same in daemon mode.
My reference distro is Xubuntu 16.04 and haven't encountered this situation yet. What desktop environment are you using? Is it prompting for password in cmdline (For now I don't have any idea how to automatically and securely feed the process this important password) or does a new window pop up?

I just noticed that on my Kubuntu machine kWallet will ask for password if it's different from my login password, but it pops up a new window.

Thanks for your reply. I have it running in a LXC-container with Debian Jessie on a Turris Omnia router, so a very specific setup.
I tought about it again, and then I remembered, that I couldn't install the keyring package, but I had to install the keyring.alt package. That might be the problem.
Thanks so far. If anything else comes into your mind, please let me know.

OK, I accidentally deleted my last comment. So, once again, (but different):
I have it running within a LXC-container with Debian Jessie on a Omnia Turris router with ARM architecture. So I have a very specific setup.
Still everything works fine, when I start it with --debug. The linux login password and the keyring password match.
When I start it, it looks like that:
root@file:~# onedrived start --debug
Starting onedrived ... OK
Please enter password for encrypted keyring:

And, to make it clear: I don't use any desktop environment at all on that machine. I have onedrived running on my fileserver.

Hi there
I have the same issues but get error... Try everything but doesnt work...
echo 'password' | /usr/bin/gnome-keyring-daemon --login
GNOME_KEYRING_CONTROL=/root/.cache/keyring-W3I9ZZ
pve01:~# onedrived start
Starting onedrived ... FAILED
ERROR: Child exited immediately with exit code 127
pve01:~# onedrived start --debug
Starting onedrived ... OK
Please enter password for encrypted keyring:
[2019-04-12 12:40:09,971] INFO: MainThread: Shutting down.  Parent is exiting...


fix typo

Coverage decreased (-0.6%) to 87.268% when pulling 1fc53b1ce77a3a922e97e2f03ffaf95042f3faf6 on JackLangerman:patch-1 into 069e97b27cabce08845b451726faaa81f5690a6a on quantopian:master.

Thanks!


Configure theme-section-feed-content-node to display appropriate date configuration

PRODUCTION:

DEV:

This corrects the ordering of the date conditionals to go in the order of:

Multi-date with start or end date (Webinar/Event)
With start or end date (Webinar/Event)
Multi-date with published
Published

 Incorrect webinar date


Typing of $set and $get

Is there a specific reason why these methods do not make use of keyof?
For example,
$set<R extends Model<R>>(propertyKey: string, instances: R | R[] | string[] | string | number[] | number, options?: IAssociationActionOptions): Promise<this>;
could instead be defined as:
$set<T extends keyof this>(propertyKey: T, instances: this[T], options?: IAssociationActionOptions): Promise<this>;
Thus rendering it type-safe.

Hey @iRath96, I thought about it, too. But sequelize supports singular and plural values, which wouldn't be covered by keyof unfortunately.

Hey @RobinBuschmann, I don't see the issue with this right now. Can you elaborate a bit more on this, please?
In my codebase, I've got a situation like this:
@BelongsTo(() => Group)
parent: Group;

@HasMany(() => User)
users: User[];

and for both cases (singular + plural), using the keyof approach works fine here.

You could write someInstance.$add('user', ...) which would cause an compiler error, since user is not a known property of your model

Ah, I get your point. I see why it's hard to make $add (as well as $del etc.) type-safe, but this does not affect $set and $get, right?

I‘ll have to check that but I think you are right - we should use keyof for get and set

@nickbclifford Thanks for bringing this up and sorry for the late reply. Restricting the type to arrays only, won't work for has-one relations. Since there the get/set methods are used as well.


Add LOTS+connected=>locally connected

Per issue #727 and the request under the already closed PR #729 .

@Jianing-Song With this new theorem, are there any trait files that are now redundant and could be removed?

@Jianing-Song With this new theorem, are there any trait files that are now redundant and could be removed?

Yes. There are S39/P41, S41/P41, S25/P43, and S38/P43 to be removed. Will do after #733 is settled.


Jared/readme updates

@ldanzinger Ready for review. Thanks!
This is the PR that updates the readme's and related samples that were caught with our diff tool.

@ldanzinger I can't see to find any information about the ss6 layer being used so I just removed the about the data section for the time being. Otherwise it should be good to go now.


⚠ Bofrost.de has degraded performance

In 69f0600, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1433 ms

Resolved: Bofrost.de performance has improved in d922511 after 6 minutes.


About ProgramVQVAE

Thanks for your great work!
I'm very interested in the  ''program <=> point cloud '' and seen the scripts of ''point cloud => program'',  but I can't find the encoder of ProgramVQVAE. Is there scripts about encoding program input?

And another question: when I run generate_pgm_condpts.py. It results
` from core.nn import SharedMLP
ModuleNotFoundError: No module named 'core' `.
The scripts are not complete or I miss some information?

Hi @hmax233 , thank you for your quick attempts.
I've updated the codebase along with more detailed installation instructions. The current version is successfully testified on my end. Could you please take a re-try? Please feel free to let me know if anything errors.

Currently, Program ⇒ Point Clouds is a deterministic process as described in the paper. We would first execute the program and obtain voxels, and then further extract point clouds thereon.
I've incorporated Program ⇒ Point Clouds codes in the script of Point Clouds ⇒ Program.
Hope the above solves your issues.

Thanks for your reply and work sincerely! I ran the code successfully and understand the program=>point cloud process.


Refactor highlight handling in FocusManager

Description
This refactors the highlight mode management in the FocusManager to separate it out into a separate private class. The public API hasn't changed, but this separates the responsibilities so it's easier to tell what part of the code is focus management and which part is highlight management. They're linked in that they both operate off of the same key event stream, but they probably should have been separate from the beginning.
Tests

This is a refactor without any semantic change, so no test changes needed.

test-exempt: code refactor with no semantic change


Setting canvas height

Hi again! I cannot seem to find a way to limit the height of where the user can draw on the view. I've set the height of the drawable view to be wrap content and match parent both allow drawing height to be the height of the device. What I need to do is allowing drawing only over the height of a picture that I have loaded onto the same view and I have the height of that picture and I've set it to canvasHeight of the config but I am still able to draw all along the height of the whole device. I have also tried manually setting the canvasHeight to be 200 for example of 50, still able to draw all over the height.
Check this code please, this is just inside an Activity that has an imageView and a drawableView inside it, nothing else. setting canvasWidth works just fine by the way, I am not able to move the canvas to the right because the width is as just as the width of the device

        DisplayMetrics displaymetrics = new DisplayMetrics();
        getWindowManager().getDefaultDisplay().getMetrics(displaymetrics);
        int width = displaymetrics.widthPixels;

        final DrawableViewConfig config = new DrawableViewConfig();
        config.setStrokeColor(getResources().getColor(android.R.color.white));
        config.setStrokeWidth(20.0f);
        config.setMinZoom(1.0f);
        config.setMaxZoom(1.0f);
        config.setCanvasWidth(width);

bookmarkIMG.getViewTreeObserver().addOnPreDrawListener(
                new ViewTreeObserver.OnPreDrawListener() {
                    public boolean onPreDraw() {

                        int height = bookmarkIMG.getMeasuredHeight();

                        config.setCanvasHeight(height);

                        Log.d("SIZE", "Canvas size set to " + height);

                        return true;
                    }
                });

        drawableView.setConfig(config);

Thanks so much for your help so far!

The pre draw listener you are using is async, not sync. So you are changing the configuration height after you setted the config. if you move the "drawable view.setConfig" inside the pre draw I think you achieve what you want. Check the code, the setConfig configures some things inside the drawableview.

I get what you mean by I also tried to set the height to 200 or 50 manually without calculation or anything. I will try again and let you know

@Odaym So the problem is that the canvas w/h you are setting is minor than the view w/h?

Well no the problem is that regardless of what number I give I can still draw on all the height of the device

I see, to reproduce it I setted canvas height to 100, my drawable view uses all the height and I can paint in all the height wich is not correct, the correct is to limit the view to the canvas height. I got it?

Yep, that's it

@Odaym I just uploaded a fix in the branch "limit_canvas_size" can you test it and tell me what do you think?

You'll have to excuse me but I don't know how to do what you're suggesting. I can see the branch of course but I don't know what to do with a library that I usually include by specifying that gradle line that I'm used to. How do I test limit_canvas_size branch?

Alright, I cloned the repo to my computer but I couldn't get it to work as a library under my project

Grab the "library" folder from the repo and place it under my project/libs and then ask gradle to compile it in build.gradle and settings.gradle

What I tried and did get to work was:

Import the whole project DrawableView as a new project in AS and run it.

And I saw that it's perfectly working to limit the height of the canvas, it's great :) Thank you
Are you going to include this into master and upgrade the maven number so that people can include it still in 1 line as before?

Hi, I'm still not happy with the solution and is not working good if you are using zoom, so I re-think and implement in a better way, if this is quite enough for you, see the build.gradle in the sample project how to import a dependency and you can work with this branch. I will fix it asap but I want to do some refactor before.

Done, works fine when included into the project. I can see it running on my own app from that branch. But I will still wait for when its on master, for a much cleaner include process :P Thanks

I'm still improving this, but in the branch "canvas_drawer" you will find a fixed version with scale/scrolling support.


[WIP] DW: Implement "Requested resources" bullet charts on Project Metrics page (formerly "Resource usage" charts)

WIP - Source is ready for review, but I need to finish unit tests and write up the description below. I'm opening this now just to get CI to build me an image.
NOTE: This PR  builds on #2650, #2658 and #2659 and contains commits from all of them. Do not review this diff directly. This will be marked as hold until it is rebased after those PRs merge.
If you want to review these changes, either wait until those PRs are merged and we rebase this, or review only the last commit.

Closes RHOAIENG-2844.
Description

How Has This Been Tested?

Test Impact

Request review criteria:

Self checklist (all need to be checked):

[ ] The developer has manually tested the changes and verified that the changes work
[ ] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/retest


4.0.13 not on cdnjs

Version 4.0.13 does not appear on https://cdnjs.com/libraries/select2 . Do we have any idea why or when it will appear?
Thanks!

Yeah, cdnjs is a mess which is why we now recommend using jsdelivr. It took them months to get recent versions online, while jsdelivr gets them online within minutes.
Raise a ticket over at cdnjs and I'm sure they will manually pull in the new versions again.


npm installation fails

npm installation fails below versions.
Platform : Windows 10
Node version : 8.4.0
Npm version : 5.3.0
Python version : 2.7.13
C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter>npm install

><EMAIL_ADDRESS>install C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli
> node-pre-gyp install --fallback-to-build

node-pre-gyp ERR! Tried to download(403): https://node-zopfli.s3.amazonaws.com/Release/zopfli-v1.4.0-node-v57-win32-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI) (falling back to source compile with node-gyp)
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140\Platforms\x64\PlatformToolsets\v140\Toolset.targets(36,5): error MSB8036: The Windows SDK version 8.1 was not found. Install
 the required version of Windows SDK or change the SDK version in the project property pages or by right-clicking the solution and selecting "Retarget solution".  System Windows_NT 10.0.14393
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Development\\Workspaces\\TESTS\\Universal\\fusebox-angular-universal-starter\\node_modules\\node-gyp\\bin\\node-gyp.js" "build" "--fallback-to-build" "--module=C:\\Development\\Workspaces\\TESTS\\Universal\\fusebox-angular-universal-starter\\node_modules\\node-zopfli\\lib\\binding\\node-v57-win32-x64\\zopfli.node" "--module_name=zopfli" "--module_path=C:\\Development\\Workspaces\\TESTS\\Universal\\fusebox-angular-universal-starter\\node_modules\\node-zopfli\\lib\\binding\\node-v57-win32-x64"
gyp ERR! cwd C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli
gyp ERR! node -v v8.4.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute 'C:\Program Files\nodejs\node.exe C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-gyp\bin\node-gyp.js build --fallback-to-build --module=C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli\lib\binding\node-v57-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli\lib\binding\node-v57-win32-x64' (1)
node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-pre-gyp\lib\util\compile.js:83:29)
node-pre-gyp ERR! stack     at emitTwo (events.js:125:13)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:927:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
node-pre-gyp ERR! System Windows_NT 10.0.14393
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Development\\Workspaces\\TESTS\\Universal\\fusebox-angular-universal-starter\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli
node-pre-gyp ERR! node -v v8.4.0
node-pre-gyp ERR! node-pre-gyp -v v0.6.36
node-pre-gyp ERR! not ok
Failed to execute 'C:\Program Files\nodejs\node.exe C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-gyp\bin\node-gyp.js build --fallback-to-build --module=C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli\lib\binding\node-v57-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter\node_modules\node-zopfli\lib\binding\node-v57-win32-x64' (1)
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >=4.0.0-beta <5.0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >=4.0.0-beta <5.0.0 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\merih\AppData\Roaming\npm-cache\_logs\2017-09-01T14_59_37_299Z-debug.log

C:\Development\Workspaces\TESTS\Universal\fusebox-angular-universal-starter>

Thanks for reporting. I have yet to test this on a Windows machine. Will try sometime this week.

Hi, I'm having the same issue.
Seems to be related to compression features. It's impossible to npm install shrink-ray on windows as it requires node-zopfli. Only way I could install node-zopfli AT ALL was by uninstalling Python 3, installing windows tools, downgrading npm to LTS 6.11 from 8... then shrink-ray still doesn't install so I gave up.
Maybe we could get a version without compression in the meantime for Windows users (50%+ of people trying to download it) while a fix is worked on?
Thanks in advance - really looking forwards to trying this once we can run it.

To follow up, the fix is simply to remove shrink-ray from package.json dependencies, and then comment out or remove the following 2 lines in server.ts:

line 21: const shrinkRay = require('shrink-ray');
line 54: app.use(shrinkRay);

This most likely needs to be fixed in master by replacing shrink-ray with compression or something which builds on windows.

I am able to successfully run npm install in windows 10. I don't think its a  windows issue.
may be it is related to global node modules.

Oh wow great to know! Maybe we can make it so windows uses don't use shrink-ray, but just compression instead? Miss out on the Brotli? Would this help everyone?

I confirm that npm installation successfully done using @bluelightning2k solution.

@merihs can you try npm rebuild node-sass?

@bluelightning2k Yes :) It worked after rebuilding node-sass.. Thank you so much
Thank you guys

After the latest update I get "Cannot find module 'node-zopfli'" error.
I think it is related with this issue.
[Browsersync] Proxying: http://localhost:8001
[Browsersync] Access URLs:
   Local: http://localhost:8000
External: http://<IP_ADDRESS>:8000

      UI: http://localhost:3001

UI External: http://<IP_ADDRESS>:3001
module.js:491
throw err;
^
Error: Cannot find module 'node-zopfli'
at Function.Module._resolveFilename (module.js:489:15)
at Function.Module._load (module.js:439:25)
at Module.require (module.js:517:17)
at require (internal/module.js:11:18)
at Object. (C:\Development\Workspaces\TESTS\Universal\patrickmichalina\fusebox-angular-universal-starter\node_modules\shrink-ray\index.js:31:14)
at Module._compile (module.js:573:30)
at Object.Module._extensions..js (module.js:584:10)
at Module.load (module.js:507:32)
at tryModuleLoad (module.js:470:12)
at Function.Module._load (module.js:462:3)
Typechecker plugin(async) :App typechecker.
Time:Sun Sep 03 2017 23:58:43 GMT+0300 (Russia TZ 2 Standard Time)
All good, no errors :-)
Typechecking time: 15738ms
Quiting typechecker
killing worker


std::unordered_map not "updating" from Lua's side.

I am adding an std::unordered_map into a Sol2 state, and afterwards emplacing something within it. But on the Lua side it does not seem to "update".
Code to reproduce error:
#include"sol.hpp"
#include<unordered_map>

struct thing {
	std::string s;
};

int main(int, char**) {
    sol::state stat;
    stat.open_libraries();

    std::unordered_map<void*, thing> map;
        stat["t"] = stat.create_table_with("cppmap", map);

    map.emplace((void*) 0x29AAFF, thing{"Sol2 bug"});

    stat.script(R"""(
	    print(#(t.cppmap));
    )""");

    return 0;
}

Compile settings: g++ -std=c++14 -I/usr/include/lua5.3 solbug.cpp -llua5.3
Expected output should be 1, however I get 0.

Very sorry for the ignorance :P, thank you.


Improve Dup Checking Solution for Projects and Namespaces

Currently the system does a very simple, to simple, brute force search for duplicate terms in the kraken configuration files.  If a duplicate is detected, the action is rejected.

Need to improve the communication of the status on a duplicate instance to the client

Need to improve the fidelity of checking for duplicate names to understand the object and it's context before rejecting the action.

Example:
Attempting to add the project/namespace acme is rejected for a completely unnecessary reason (a textual match), on string acme due to the following stanza in the configuration file.
        - name: kube-lego
          registry: quay.io
          chart: samsung_cnct/kube-lego
          version: '0.0.5-0'
          namespace: ingress
          values:
            image: "jetstack/kube-lego:0.1.4"
            email<EMAIL_ADDRESS>            domain: launchdock.io
            # api: "https://acme-colonybead.api.letsencrypt.org/directory"
            api: "https://acme-v01.api.letsencrypt.org/directory"


Best Practices: Deploy to IIS

Hello,
I am looking to publish my solution to IIS in my sandbox environment.  I have a universal certificate that I currently use to host all of my sandbox projects.  My question is about the configuring the validation and signing certificate, and I do apologize if this is a rookie issue.  Do I simply create a self signed certificate on the web server and configure the application to use that certificate, or do I need to use my universal certificate for these settings?  In either case, do I use just the thumbprints or do I need to use the path and password?
Regards,

i am also interested in this feedback.

I think I have this figured out.  It took some digging, which I was hoping to avoid, hence posting here.
It doesn't appear that wildcard certificates are supported at this time, see: https://github.com/dotnet/aspnetcore/issues/21184
So I simply created a self-signed certificate for each of the two operations (Signing and Validation).  As an aside, I generated them on my Windows 10 workstation using power-shell as IIS is running on windows server 2012 r2 and the New-SelfSignedCertificate function is limited.
I used this link to get me started on generating self-signed certificates using powershell: https://mcguirev10.com/2018/01/04/localhost-ssl-identityserver-certificates.html.
I store my certs in a common location (e.g. c:\certs\sandbox), so I copied them from my Windows 10 workstation to here and imported them into the certificate store.
I needed to do some setup on IIS to enable all this to work.

Create a user account to run my application pool with.  (e.g. ./wp_IS4Admin)
Create an application pool for IS4 to run in.
a. Disable 32bit processing (General -> Enable 32bit application = false)
b. Configure to run with another user account (Process Model -> Identity = ./wp_IS4Admin)
c. Configure the pool to load the user profile (Process Model -> Load user profile = true)
see: https://stackoverflow.com/questions/17840825/cryptographicexception-was-unhandled-system-cannot-find-the-specified-file
Create a website to publish IS4 to and have it use the configured application pool
Publish the application to IS4 website

My publish profile looks like this (one day I'll figure out how to configure Azure DevOps for CI/CD).
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">   <PropertyGroup>     <WebPublishMethod>MSDeploy</WebPublishMethod>     <LastUsedBuildConfiguration>Release</LastUsedBuildConfiguration>     <LastUsedPlatform>Any CPU</LastUsedPlatform>     <SiteUrlToLaunchAfterPublish>https://IS4Admin.mysandboxdomin.com</SiteUrlToLaunchAfterPublish>     <LaunchSiteAfterPublish>True</LaunchSiteAfterPublish>     <ExcludeApp_Data>False</ExcludeApp_Data>     <TargetFramework>netcoreapp3.1</TargetFramework>     <SelfContained>false</SelfContained>     <MSDeployServiceURL>https://www.mysandboxdomin.com:8172/msdeploy.axd</MSDeployServiceURL>     <DeployIisAppPath>sts.identity</DeployIisAppPath>     <RemoteSitePhysicalPath />     <SkipExtraFilesOnServer>True</SkipExtraFilesOnServer>     <MSDeployPublishMethod>WMSVC</MSDeployPublishMethod>     <EnableMSDeployBackup>True</EnableMSDeployBackup>     <RuntimeIdentifier>win-x64</RuntimeIdentifier>     <PublishReadyToRun>True</PublishReadyToRun>   </PropertyGroup> </Project>
Once that was done, I configured appsettings.production.json as follows:
###NOTE: The paths used for certs needed to be escaped (e.g. c:\ vs c:)
{   "exclude": [     "**/bin",     "**/bower_components",     "**/jspm_packages",     "**/node_modules",     "**/obj",     "**/platforms"   ],   "ConnectionStrings": {     "ConfigurationDbConnection": "Server=qa_sql_farm;Database=IS4Admin;Trusted_Connection=false;uid=IS4Admin;pwd=ItsAPassword;MultipleActiveResultSets=true",     "PersistedGrantDbConnection": "Server=qa_sql_farm;Database=IS4Admin;Trusted_Connection=false;uid=IS4Admin;pwd=ItsAPassword;MultipleActiveResultSets=true",     "IdentityDbConnection": "Server=qa_sql_farm;Database=IS4Admin;Trusted_Connection=false;uid=IS4Admin;pwd=ItsAPassword;MultipleActiveResultSets=true"   },   "DatabaseProviderConfiguration": {     "ProviderType": "SqlServer"   },   "CertificateConfiguration": {     "UseTemporarySigningKeyForDevelopment": false,     "CertificateStoreLocation": "LocalMachine",     "CertificateValidOnly": true,     "UseSigningCertificateThumbprint": false,     "SigningCertificateThumbprint": "",     "UseSigningCertificatePfxFile": true,     "SigningCertificatePfxFilePath": "c:\\certs\\sandbox\\token_signing.pfx",     "SigningCertificatePfxFilePassword": "secretPassword",     "UseValidationCertificatePfxFile": true,     "ValidationCertificatePfxFilePath": "c:\\certs\\sandbox\\token_validation.pfx",     "ValidationCertificatePfxFilePassword": "anotherSecretPassword",     "UseValidationCertificateThumbprint": false,     "ValidationCertificateThumbprint": ""   },   "RegisterConfiguration": {     "Enabled": true   },   "ExternalProvidersConfiguration": {     "UseGitHubProvider": false,     "GitHubClientId": "",     "GitHubClientSecret": ""   },   "SmtpConfiguration": {     "Host": "",     "Login": "",     "Password": ""   },   "SendgridConfiguration": {     "ApiKey": "",     "SourceEmail": "",     "SourceName": ""   },   "LoginConfiguration": {     "ResolutionPolicy": "Username"   },   "AdminConfiguration": {     "PageTitle": "Skoruba IdentityServer4",     "HomePageLogoUri": "/images/skoruba-icon.png",     "FaviconUri": "/favicon.ico",     "IdentityAdminBaseUrl": "http://IS4Admin.mysandboxdomin.com",     "AdministrationRole": "Administrators"   },   "CultureConfiguration": {     "Cultures": [],     "DefaultCulture": null   } }

Thanks @kroleigh for sharing your experience with IIS deployment.

Create a user account to run my application pool with. (e.g. ./wp_IS4Admin)
Create an application pool for IS4 to run in.
a. Disable 32bit processing (General -> Enable 32bit application = false)
b. Configure to run with another user account (Process Model -> Identity = ./wp_IS4Admin)
c. Configure the pool to load the user profile (Process Model -> Load user profile = true)

SqlException: 用户   $' 登录失败。
when  i use a  new account ; the error message: user login failed. in fact the account  is match password;
Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, uint waitForMultipleObjectsTimeout, bool allowCreate, bool onlyOneCheckConnection, DbConnectionOptions userOptions, out DbConnectionInternal connection)

Can anyone share any link to deploy this solution to IIS windows ?

I have been trying to deploy this app to IIS on VPS for a month now with no success. Hopefully somebody comes along and bails me out

i am also interested in this feedback.

Have you been able to deply successfully to IIS?
I have been trying to deply this to IIS on a VPS and its been a month now. I tried following the guide provided but I just cant seem to figure things out


Incorrect offset for reading 1 byte file

A bug on this line vertx-java.io/src/main/java/io/cloudonix/vertx/javaio/WriteToInputStream.java at a5de4c13e6d121814260482ef98bc213f62e3ec2 · cloudonix/vertx-java.io
Basically, when read(b, off + 1, len - 1) reaches end of stream, it returns -1. Here 1 + (-1) would be 0, but we actually want offset to be 1 because the byte from read() could be successful. It’s the offset for the buffer set improperly. This caused the 1 byte to be lost
This failure only happens when:

the file has only one byte
the read from the buffer in WriteToInpusStream happens before the write of the first byte.

Fixed in 1.4.0


How To Install?

Hi,
I'm really looking forward to playing around with your grid-sensor! However, I'm new to Unity and I'm not sure how to install your component extension for it to work with ml-agents? I looked through your documentation but there doesn't seem to be an installation guide 😅 Could you give me a few pointers please?
Thank you!

Hi, thanks for the quick reply! I've already installed the python backend and trained my agent. I mean how do I use your sensor in my project? Do I just manually add the manifest.json file from your Packages folder? Or how would I do it? Thank you! :D

You only need to add the line
com.dbrizov.naughtyattributes": "https://github.com/dbrizov/NaughtyAttributes.git#upm",
to the dependencies in your manifest.json
Other than that, just copy the Scripts folder from my repo.

Great, thank you so much! :D


Upgrade Markdown

Upgrade Markdown

Upgrade to Python Markdown 2.6.2 via https://github.com/EnTeQuAk/python-markdown-py26-support  fork (makes hand merging way easier as this fork as re-added in PY26 support).
Upgrade 3rd party plugins to latest version and modify to work with
https://github.com/EnTeQuAk/python-markdown-py26-support fork and the Sublime Text environment.
Update readme with proper doc links which satisfies #281.

Notes

2.6 Python Markdown branch appears to moving in a bug fix only state as Python Markdown 3 will have many backwards compatibility breaks.  I probably won't try to update Markdown Preview past 2.6 releases moving forward as I suspect there will be much refactoring that will have to be done for 3.0.  But 2.6 moving to a bug fix state means that most of the 2.6 releases should be pretty easy to merge in from now on.
Python Markdown has deprecated headerid as functionality is now pushed into toc.  This is why we remove headerid from the default extensions in the settings.
enable_pygments option now only has an effect if using enable_highlight.  This is just like how guess_language works.  If defining codehilite manually, you should use the use_pygments options or guess_language option that is built into the codehilite plugin officially.  Personally, I feel enable_pygments, guess_language, and enable_highlight should be deprecated in favor of the manual configuration method as it would reduce code and be consistent with the way all other plugins are configured, but I feel this should be discussed and/or completed at another time.

great, thanks !


Update compilerplugin, scalapb-runtime, ... to 0.11.12

Updates

com.thesamet.scalapb:compilerplugin
com.thesamet.scalapb:scalapb-runtime
com.thesamet.scalapb:scalapb-runtime-grpc

from 0.11.9 to 0.11.12.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thesamet.scalapb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.thesamet.scalapb" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #132.


[Feat]:- Password Hide and Show button

Is your feature request related to a problem? Please describe.

It has no show and Hide password input on the login and register pages.
Describe the solution you'd like

we need to add a show and hide button in the password input for user-friendly
Demo Video/Screenshot (Mandatory )

which one you like
Are you contributing under any open-source program ?

I am GSSOC24 Contributor
Please assign me, I would like to work on it

Can you pls assign me this issue?

@SrinivasDevolper u are assigned . There are recent changes in the login page .So kindly update your branch before adding this feature . All the best

I would like to work on this issue , please assign this to me.

Hello @apu52  iam a contributor @GSsoc 2024 extnd  so i have worked on this project isuue . please assign me this issue under gssoc level 1 .
iam providing the screen shots of the resolved issue please look into this and assign me this issue .
thank you,

Hey @apu52 please assign me this task I  have much  experience in front end development


Resampling transform files

Hi, I need to register a bunch of different modalities into a template space, with various resolutions for a group analysis. By example, I have some SWI images (0.6mm isotropic) and fMRI files (3mm iso) with their transform files to put them in their T1 referential (1mm). I also have the transform files for the T1 (1mm) to OASIS (1mm). If I want to apply these transforms to my SWI files, I'll end with a registered SWI of 1mm isotropic (loss of details). One idea was to upsample the T1 and OASIS template and redo the registration the SWI on them, but I wondered if there was a method to apply this resampling to the affine and warp files.

You can use ConvertImage ... 10 to separate the warp files into their component scalar images.  You can then resample those scalar images, using ResampleImage and then put them back together using ConvertImage ... 9.  The affine files are not based on image resolution so you wouldn't need to changes those files.

in antsApplyTransforms , the -r option controls the resolution.  so you can
just resample your reference image to whatever resolution you prefer - may
not need to change the transformations.
brian
On Fri, Feb 24, 2017 at 2:13 PM, Nick Tustison<EMAIL_ADDRESS>wrote:

You can use ConvertImage ... 10 to separate the warp files into their
component scalar images. You can then resample those scalar images, using
ResampleImage and then put them back together using ConvertImage ... 9.
The affine files are not based on image resolution so you wouldn't need to
changes those files.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/415#issuecomment-282378334, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATyfgj7tV3E9GXYd9xh9yaIpAdh_2o4ks5rfyvVgaJpZM4MLiyX
.

I did something similar in ANTsR recently. All it takes is resample the
refrenece image to the desired resolution and apply transforms using the
newly resampled image as reference.
Dorian
On Fri, Feb 24, 2017, 14:31 stnava<EMAIL_ADDRESS>wrote:

in antsApplyTransforms , the -r option controls the resolution. so you can
just resample your reference image to whatever resolution you prefer - may
not need to change the transformations.
brian
On Fri, Feb 24, 2017 at 2:13 PM, Nick Tustison<EMAIL_ADDRESS>wrote:

You can use ConvertImage ... 10 to separate the warp files into their
component scalar images. You can then resample those scalar images, using
ResampleImage and then put them back together using ConvertImage ... 9.
The affine files are not based on image resolution so you wouldn't need
to
changes those files.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/415#issuecomment-282378334, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AATyfgj7tV3E9GXYd9xh9yaIpAdh_2o4ks5rfyvVgaJpZM4MLiyX
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/415#issuecomment-282382990, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIqafcJrTAvVCVuJ-PjnMzJKR_XLxyNrks5rfzAngaJpZM4MLiyX
.

Thanks guys, '-r option' indeed makes the job easy. I thought that I needed to upsample the warp files, but it's automagically done. Good job!


[Android] Support additional dependency declaration format

Thanks for submitting a PR! Please read these instructions carefully:

[x] Explain the motivation for making this change.
[x] Provide a test plan demonstrating that the code is solid.
[x] Match the code formatting of the rest of the codebase.
[x] Target the master branch, NOT a "stable" branch.

Motivation (required)
Previously, isInstalled was somewhat naively checking for the presence
of a string in the build.gradle file to determine whether or not that
dependency was already linked. I.e.:
    compile project(':${name}')\n

…where name is replaced with the name of the dependency being checked.
This was inflexible as it only supported that particular format of
compile definition. Another, valid compile definition follows:
    compile(project(':example') { … }

However, this failed the check because it didn't exactly match the
format for which the check was searching the build.gradle contents. As
a result, running react-native link would incorrectly duplicate the
dependency definition and thus cause a crash upon launching the app.
This change adds an installPattern to the object returned from
makeBuildPatch, which includes the particular dependency name and is
valid for both compile definition formats.
Test Plan (required)
This commit adds an additional compile definition in the associated fixture,
an additional test case in isInstalled.spec.js to check for this additional
format, and an additional test in makeBuildPatch.spec.js to ensure the
object returned includes the aforementioned installPattern Regex pattern.
Next Steps
Sign the CLA, if you haven't already. ✅
Small pull requests are much easier to review and more likely to get merged. Make sure the PR does only one thing, otherwise please split it. ✅
Make sure all tests pass on both Travis and Circle CI. PRs that break tests are unlikely to be merged.
For more info, see the "Pull Requests" section of our "Contributing" guidelines.

I can't seem to figure out why these Travis CI builds are failing. Any insight?

bump
Anyone available to take a look and (hopefully) help me get this merged? Thanks!


Enforce ordering of bus and branch numbers in PowerSystem construction.

This should fix issue #145.

Codecov Report

Merging #172 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #172     +/-   ##
=========================================
+ Coverage   57.76%   57.87%   +0.1%
=========================================
  Files          44       44
  Lines        3708     3715      +7
=========================================
+ Hits         2142     2150      +8
+ Misses       1566     1565      -1

Impacted Files
Coverage Δ

src/base.jl
92.3% <100%> (ø)
:arrow_up:

src/utils/IO/system_checks.jl
80.76% <100%> (+1.54%)
:arrow_up:

src/utils/IO/base_checks.jl
96% <100%> (+0.76%)
:arrow_up:

src/utils/IO/branchdata_checks.jl
97.36% <100%> (+0.14%)
:arrow_up:

Continue to review full report at Codecov.

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


enhance the function getProperty() to deal with quotations

@norbline Azah, I think your changes look good. Now there's two things to check:

Check with @sophia-guo if the intention was also to fix the test.properties? It's debateable whether that's needed, at least in this PR anyway.
You need to test your changes. The easiest way to do that is to submit what we call a Grinder using your branch of aqa-tests. So launch this job: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/build?delay=0sec
specify your fork and branch for ADOPTOPENJDK_REPO and ADOPTOPENJDK_BRANCH, then specify the test build list that will invoke the code you changed, I believe that's probably any external test suite, so BUILD_LIST=external, TARGET=sanity

@andrew-m-leonard i recieved this message when trying to test my changes " Access Denied
norbline is missing the Job/Build permission"

@andrew-m-leonard - The reviewers are responsible for testing Outreachy PRs.  Outreachy interns during their term will be given access, but for contribution period, it is still restricted access.
Testing via: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1719

@ShelleyLambert Hello mam, is like the test failed "Failed>console output" what does that mean and what next step am i supposed to take


feat:select nearest diagnostic

Hi - thanks for the pull request. I don't really understand exactly what it's doing though - can you explain please?

It selects diagnostic under the cursor and if  the cursor isn't under any diagnostic selects the closest one

Hmm - so this is bidirectional?
Can you give me a usecase for this? Is this something you think folks will use?
I do already want to take a look if it's possible to simplify or refactor the current next_diag_inclusive() function. I'll look at this in that context.
Thanks!

Yes its bidirectional.
use case: you're on the same line as diagnostic and you don't know wheatear your cursor is before, under or after diagnostic.with this function you can select it without using any other motion.

I tried to change the next_diag_inclusive()
M.next_diag_inclusive = function(local_opts)
	local cursor = vim.api.nvim_win_get_cursor(0)
	local current_line = cursor[1] - 1
	local current_column = cursor[2]

	local next = vim.diagnostic.get_next({ cursor_position = { current_line, current_column } })
	local prev = vim.diagnostic.get_prev({ cursor_position = { current_line, current_column } })
	vim.pretty_print(current_line, current_column, prev, next)
	local closest = next
	if
		(current_line >= prev.lnum and current_line <= prev.end_lnum)
		and (current_column >= prev.col and current_column <= prev.end_col)
	then
		closest = prev
	end
	if closest ~= nil then
		select_diagnostic(closest)
	end
end

but it doesn't select the next diagnostic when cursor is on a same line with a diagnostic.

So thinking about this - I don't think I want to change the behaviour of next_diag_inclusive() (although simplifying/refactoring the implementation would be welcome if possible). But most folks I believe are wanting text objects which look ahead, even if there is a closer one behind (see all the inbuilt NeoVim text objects for an example).
I am open to including this as a separate function which is not keymapped by default but can be bound by the user if they wish. However, currently this is failing some automated tests which I have written for it. I'll take a look later to see if I can find out what's wrong.
    it("can change nearest diagnostic - row 1", function()
        vim.api.nvim_win_set_cursor(0, { 1, 0 })
        vim.cmd("normal cnghello")
        check_lines({
            "test1",
            "hello",
            "test3",
            "test4",
            "test5",
        })
    end)

    it("can change nearest diagnostic - row 2", function()
        vim.api.nvim_win_set_cursor(0, { 2, 0 })
        vim.cmd("normal cnghello")
        check_lines({
            "test1",
            "hello",
            "test3",
            "test4",
            "test5",
        })
    end)

    it("can change nearest diagnostic - row 3", function()
        vim.api.nvim_win_set_cursor(0, { 2, 0 })
        vim.cmd("normal cnghello")
        check_lines({
            "test1",
            "test2",
            "hello",
            "test4",
            "test5",
        })
    end)

Sorry, I messed up the git history.
The third test you mentioned just had a typo. I added some test for the function and they're passing.
I added the mapping just for the tests, if you don't want to map it by default; I can remove it.

okay I will test them.

Both functions now are passing the tests.

Thanks for this! For now, I have decided to stick with the previous implementation of next_diag_inclusive(), as I find it clearer to understand/maintain - and theoretically it should be faster in performance too, since it only has to iterate the diagnostic list once, although this isn't the main consideration.
But I've merged nearest_diag(), and made that something the user can set up their own keybinding for. It seems to work well, thank you for the contribution!


Improve the documentation

Update and expand the documentation to cover all aspects of the application and provide user guides.

Hello, can you please tell what kind of documentation expansion do you want. Is there any specific aspect which needs to be covered in detail?


ssl: allow use of cipher strings.

Allow use of "ALL", "HIGH", "EECDH+AES128" and other cipher strings,
which evaluate to more than one cipher suite.
As a side-effect, this allows use of ChaCha20+Poly1305 cipher strings,
which evaluate to two variants of the cipher suite (draft and RFC7539)
in currently linked version of BoringSSL.
While there, account for equal-preference groups when using BoringSSL.

While I think use of cipher strings is valid in general, the workaround for ChaCha20+Poly1305 cipher suites is temporary, because the draft variant is already removed from the next chromium-stable branch of BoringSSL.

We should probably document this?
https://lyft.github.io/envoy/docs/configuration/listeners/ssl.html
https://lyft.github.io/envoy/docs/configuration/cluster_manager/cluster_ssl.html


Using @now/node with transpiled code doesn't work as expected.

I have included a build step in my deployment by adding a now-build script to my package.json as described at: https://zeit.co/docs/v2/deployments/official-builders/node-js-now-node/#build-step
My now-build script uses babel to transpile my javascript code and output it to a new directory called dist. It is this transpiled code that I want to deploy, so I have specified this in my now.json by adding the following to the builds array:
 { "src": "api/hello/dist/index.js", "use": "@now/node" },

Unfortunately, @now/node checks for the existence of api/hello/dist/index.js BEFORE executing the now-build script. This means that if I want the deployment to work I need to keep a dist folder with an index.js file lying around. This is not ideal, but not terrible when deploying from a local machine.
When adding GitHub integration, things get messy. Because now I have to check-in a dist/index.js to make sure that the automated build works. I don't want to have to checkin auto-generated, transpiled code.
In my mind it should work the other way around. The build should run my now-build script BEFORE looking for the build's src file. Am I missing something?

Thank you for creating this issue!
Please note that – in order for us to be able to handle each feature request with care – we have to consume all of them through a unified pipeline that makes it easier for us to prioritize, track and progress on the features our user base is interested in.
In turn, the Issues tab on this repository is best only be used for reporting bugs, which we can immediately act on.
For well-explained feature requests (like the one you just posted), please contact us at<EMAIL_ADDRESS>so that we can consider adding it to our roadmap.


Allow things like stacking and density measurements to ignore sources in a sample with no temperature measurements

So that the whole analysis doesn't get derailed by one annoying object.

What they do is give it a default temperature of 3keV

What they do is give it a default temperature of 3keV


Install R and dependencies in the configure block rather than setup

This should allow using language: r on the container based infrastructure.
From looking at the csharp builder it looks like sudo commands can be used in the configure() block for language implementations. This PR simply moves the R setup() commands to configure() and puts them in a travis fold so the output log is easier to browse.
@BanzaiMan if I am mistaken about this or if there is a more preferred method please let me know.
cc @craigcitro @hadley

In general, this approach should work.

Ok great, I will get the tests working and ping you.

Tests now pass https://github.com/jimhester/travis-build/commit/83926b6d949f2ab012a2123d092bed0937f73fd5

LGTM, just a few questions inline

LGTM -- ship it!

This has been deployed to https://staging.travis-ci.org/. Please test it.

@BanzaiMan could you deploy https://github.com/jimhester/travis-build/commit/8c0a9b395feadb286b396a70f9f374dcb3449576 to staging, it contains a small tweak to fallback to R source installs if not using sudo: true.

@jimhester It's been deployed to staging.

@BanzaiMan could you deploy https://github.com/travis-ci/travis-build/commit/058bdecd523051c364711443721bb8383a125b92 to staging. It contains a bugfix for checking sudo: false (https://github.com/jimhester/travis-build/commit/d5a5da07e5bde79bf909d80d20f73f4f2935f558) and switching to installing pandoc from the official releases rather than rstudio's mirrors.

@jimhester It's been deployed to stating.

https://github.com/jimhester/travis-build/commit/4af3cd0dd2996d4785e19488b028dab1009ab769 Adds support for dependency caching and lays some groundwork for supporting multiple R versions (to be done later). If it could be deployed to staging that would be great!

@BanzaiMan Could you deploy https://github.com/jimhester/travis-build/commit/4af3cd0dd2996d4785e19488b028dab1009ab769 to staging when you have time. Thanks!

Sorry for the delay. This has been deployed to staging.

@hadley, @craigcitro https://github.com/jimhester/travis-build/commit/4af3cd0dd2996d4785e19488b028dab1009ab769 supports caching, but I am not sure what the best syntax we should use is. Currently you turn it on by using a .travis.yml file like
language: r
sudo: false
cache: R

See jimhester/travis-test/builds/475863 for example build output. (There is network congestion today on travis staging so the build time is much longer for that build than it would be normally)
Other languages use syntax like cache: bundler or cache: ccache or cache: pip (see https://docs.travis-ci.com/user/caching/ for more examples). Any thoughts on what to call Rs version? If we stick with using cache: R should we make it

Upper case like the R executable
Lower case like language: r
Just make it case insensitive

We could use cache: cran if we want a name that suggests which dependencies we're caching?
I'm also totally fine with r -- it would feel weird to have them with different case, like R CMD BUILD vs. R CMD check. :grin: Could we make the language key case-insensitive, too?

I just verified language: is case insensitive. Hadley suggested cache: packages which is what I am going with. It makes it explicit what is being cached and allows for extensibility later if we want to cache something else, the knitr cache for knitr documents for example...

I think https://github.com/jimhester/travis-build/commit/61c8b815bf3bb47d64da9a1889e8af6963aeddd4 is probably ready to merge unless you see anything else that needs to be changed.

@BanzaiMan this can be merged unless you have any further review comments.


Feature/Schema builder check if column has index and if has foreign key

This PR will let users check if the table has been indexed in a specific column
\Illuminate\Support\Facades\Schema::hasIndex('table_name','column_name','index_name')
Or
will search for default index name table_name_column_name_index
\Illuminate\Support\Facades\Schema::hasIndex('table_name','column_name')
$this->schemaBuilder()->create('pandemic_table', function (Blueprint $table) {
            $table->id();
            $table->string('stay_home')->index();
            $table->string('covid19');
            $table->string('wear_mask');
            $table->unique(['wear_mask','covid19']);
        });
$this->assertTrue($this->schemaBuilder()->hasIndex('pandemic_table','id','primary'));
$this->assertTrue($this->schemaBuilder()->hasIndex('pandemic_table','stay_home'));
$this->assertTrue($this->schemaBuilder()->hasIndex('pandemic_table',['wear_mask','covid19'],'pandemic_table_wear_mask_covid19_unique'));
$this->assertFalse($this->schemaBuilder()->hasIndex('pandemic_table',['wear_mask']));
$this->assertFalse($this->schemaBuilder()->hasIndex('pandemic_table',['wear_mask'],'primary'));

Previous code which I used to handle this case
$keys = collect(DB::select(DB::raw('SHOW KEYS from table_name')));
if (!$keys->where('Key_name', 'Index_name')->first()) {
                        $table->string('column_name')->index()->change();
}

if If I want to check foreign keys
$foreignKeys = collect(
                    DB::select(
                        "SELECT *
FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE
WHERE TABLE_SCHEMA = 'da_name'
      AND REFERENCED_COLUMN_NAME IS NOT NULL;"
                    )
                );

again for loop and  if and else in the migration file
Why do we need this,
Real case
If we have multiple apps working with a shared ( single) database every app didn't know what others do in the database so we need to check if

[x] If the table exists (which laravel have this feature)
[x] if a column exists (which also laravel have this feature)
[x] if column(s) have specific index (done in this PR)
[ ] If column linked to another table foreign key (In progress)

Please rebase with master which should normally fix the tests and mark as ready for review.

Looks like something went wrong with branching here. Please submit anew.

Thanks, @driesvints
So which branch I should target with the new Pr

I'm sorry, I told you to rebase with master while it should have been 8.x, sorry about that..

Never mind


Update Mintaka to 0.5.28

Proposed changes
This PR updates Mintaka to 0.5.28.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Update docker image version of FIWARE GE.
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[X] I have updated the change log (CHANGELOG.md)
[X] I send this pull request to the v0.19.0-next branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (8bd4edb) compared to base (626be36).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff               @@
##           v0.19.0-next      #216   +/-   ##
==============================================
  Coverage        100.00%   100.00%
==============================================
  Files                 2         2
  Lines              1746      1746
==============================================
  Hits               1746      1746

Impacted Files
Coverage Δ

config.sh
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[T4A2][W11-B3]Wong Kang Fei

@kfwong
Some comments added. Please ack & close the PR after reading comments.


Hello, I'd like to ask you how you process datasets. Would it be possible to provide a portion of your processed dataset?

Hi! The data is available here. I also added the link to the data to the README. Thanks for pointing this out!
Let me know if there are any further issues!

Your prompt reply and willingness to share your data with me are greatly appreciated. Thank you very much for your kindness and generosity.

Hi,I downloaded your dataset and noticed that its shape is [N, 500, 129]. I understand that the frequency  is 500Hz, but I'm confused about why there are 129 channels when your paper mentions using a 128-channel EEG Geodesic Hydrocel system.

That's a good questions that needs clarification: during preprocessing, we compute the average of the signal at all 128 physical EEG electrodes and subtract it from the EEG signal at every electrode for every time point. This average of the 128 channels is then added as 129-th channel.
Let me know if you have any further questions!

Hi @Onlyknight ! Can you specify in more detail what your question is? I understand it as how we handle the case that during training, there might be no object in a sample.
If this is indeed your question: for the segmentation tasks we consider (eye events and sleep stages), this case does not occur, since there is always some eye event (fixation, saccade, blink) present (similar in the domain of sleep staging). The architecture can be generalized easily to have a "no object" class (like the original DETR for vision) in case you are processing time series data where "no object" is a valid case.

Hi,@lu-wo ,thank you for your answer.
Yet,when I try to use the DETRTime for my training task on my dataset, I found that the train set must have object in order to finish the HungarianMatcher.
I hope to train the DETRTime on my dataset in order to detect the sleep apnea event.
But most samples of my train samples without any object ,  I mean that they are  negative samples.
Now, I try to solve the problem with a method that I regard the normal time as a object (like the fixation in your eye event).But, I want to know how to set the biased_weights and empty_weight, can you give some guiding opinions?
Thanks.

Hi @Onlyknight! It seems like you want to train a segmentation task with binary labels (sleep apnea event vs. none). The easiest way to do this, is simply assigning "no event" class 0, and "sleep apnea" class 1.
Let me know if you need further advice!

Hello, I would like to ask, what is the basis for setting the loss weights and sampling weights in your network?

Loss weights: The loss weights are taken from the original DETR implementation and can be adapted depending on the convergence of a particular loss component during training.
Sampling weights: You can set them to upsample minority classes. For example, in our ocular event datasets, fixation is the (temporal) majority class, while saccades and especially blinks occur less often. By setting the sampling weights, one can guarantee that the model is being updated oftentimes on the minority classes as well.
Let me know if you have further questions.


Fix creating valid workspace file

This will fix the issue described in #35, where the workspace will not open. I double checked the workspace format by one created with CocoaPods.

Thank you for the fix, much appreciated 👍

With Xcode 9.4.1 this seems to be a problem again.


How to dispose Chewie Video Controller

Hello, I'm using the video player for the first time to create a movies app and every time I navigate away from the movie detail page back to the list of movies. The Chewie player keeps playing, infact it keeps starting over overlapping the previously playing one and you know the same video is playing more than twice. How do I dispose the playerController after I'm done on the page. I've tried calling the dispose function in WillPopScope but it's not working. What's the proper way to dispose the video player?

Hey. You should dispose of both videoPlayerController and chewieController in dispose method. I'm assuming you navigate to "list of movies" page with some button and not with Back button. If that is the case your Chewie player page stays in Navigator stack and dispose is not called. This is also why your onWillPop function isn't called because the page was not popped from the Navigator stack.
I managed to fix Chewie playing in the background with RouteObserver by overriding didPushNext() function which is fired when you leave the page that has RouteAware. I just paused the video and then sought to the start.
As for overlapping videos, you should check your Chewie page build function for loop caused by calling SetState() in it.


[feedback] Command Palette experiment

This issue is meant to collect feedback about the Edge Command Palette experiment.
Edge Command Palette is a new experiment that makes it possible to run useful productivity and developer commands in Edge by pressing Ctrl+Q. This includes running DevTools commands without having to open DevTools first. Learn more about this experiment and how to use it from the documentation.
If you used Edge Command Palette and have any feedback about it, please leave a comment in this issue.
Shortcut note: the experiment started with the Ctrl+Shift+Space shortcut and that shortcut works in Microsoft Edge 105 to 107. Based on feedback received in this issue, it was changed to Ctrl+Q and this shortcut can be used starting with Microsoft Edge 108.

One of the things I am missing is direct interaction. You can open DevTools from it and use some of the functionaliity there but often I don't want to do that. I'd love to see some immediate toggles being available through the command palette:

Turn JavaScript on/off
Switch dark/light mode of the current page (this is part of the rendering tool)
Take full page screenshot (this is available as cmd+shift+s but people may not know it)
Show/hide keyboard order (this is part of the accessibility panel)

Saw and tried out, found some issues.
Machine     SurfaceBook 2
Edition        Windows 11 Enterprise
OS build	   22000.739
Edge (beta) Version 104.0.1293.14

(with dark mode) Loading the palette each time, palette flashes in light mode then loads in the correct theme
(with dark mode) Similar to the above, when typing out queries, when suggestions enter/leave, they sometimes can cause individual suggestions or the entire suggestions panel to re-render in what looks like "light mode" then back to the theme.
Suggestions and or input field get cut off in certain cases (see screenshot below)

Second screenshot is upon navigating into the suggestions with arrow keys

Typing can feel a bit sluggish and un-responsive, at times, it's minimal, but I would expect perhaps a slight delay on the suggestions instead of typing 🙂

@raekul the work to make this UI accessible to Screen Readers is planned for us. Thank you for trying out the feature and providing your feedback.

The shortcut Ctrl+Shift+Space is conflicted with macOS's change input method shortcut (Ctrl+Space), and won't work until I change the input method shortcut.

The shortcut Ctrl+Shift+Space is conflicted with macOS's change input method shortcut (Ctrl+Space), and won't work until I change the input method shortcut.

How does that work? I am a MacOS user and never triggered that. Do you need an external keyboard?

The shortcut Ctrl+Shift+Space is conflicted with macOS's change input method shortcut (Ctrl+Space), and won't work until I change the input method shortcut.

How does that work? I am a MacOS user and never triggered that. Do you need an external keyboard?

Yes, I'm using an external keyboard, but I also tried the macbook built-in keyboard, doesn't work either.
Here are the versions:
macOS 12.4
edge 105.0.1321.0

The shortcut  Ctrl+Shift+Space is not working on my desktop (Windows 11) either in Edge dev or release. Not sure if another app is capturing it. I do have Power Toys installed but it doesn't seem to trigger anything when I press that combination. Is there a way to alter the key combo or see what is capturing that combo?

The shortcut Ctrl+Shift+Space is not working on my desktop (Windows 11) either in Edge dev or release. Not sure if another app is capturing it. I do have Power Toys installed but it doesn't seem to trigger anything when I press that combination. Is there a way to alter the key combo or see what is capturing that combo?

This is currently only enabled in Canary, not in Dev yet.

I’d like to suggest the shortcut being changed to Ctrl + Shift + P so that it matches tooling such as VS Code and Windows Terminal.
I feel like it would just make sense to keep them similar, especially since it's mainly power users / developers who will use the feature and they're already used to that shortcut.

Just enabled it and it works great, the only thing I noticed is that when first set up, it conflicts with the standard 1Password Quick Actions shortcut so I had to disable that first before I could use it.
Also adding the ability to take a full page screenshot would be great.

Just enabled it and it works great, the only thing I noticed is that when first set up, it conflicts with the standard 1Password Quick Actions shortcut so I had to disable that first before I could use it.
Also adding the ability to take a full page screenshot would be great.

Thanks so much for your feedback. Good catch with the 1Password extension - I guess it is tough to find any keyboard shortcut that isn't used yet.
As for the full page screenshot, I suppose you mean one that also scrolls the whole document, right?
This is already in device mode, so it should be easy to add.

I am having the same issue with the onepass extension. Is there a way to remap the edge keyboard shortcut to a different shortcut of the users choice?

speaking of screenshots... size (height) is limited to 16384px (also in Chrome)
i.e. try full page screenshot of https://css3test.com/
works in FF though

@Nova38 Is there a way to remap the edge keyboard shortcut to a different shortcut of the users choice?

try edge://extensions/shortcuts

speaking of screenshots... size (height) is limited to 16384px (also in Chrome)
i.e. try full page screenshot of https://css3test.com/ works in FF though

Interesting, I will file a bug and see why this is.

@Nova38 Is there a way to remap the edge keyboard shortcut to a different shortcut of the users choice?

try edge://extensions/shortcuts

That is only for extensions not for edge shortcuts.

That is only for extensions not for edge shortcuts.

yes, I thought you might try to change the shortcutss of the onepass extension

That is only for extensions not for edge shortcuts.

yes, I thought you might try to change the shortcutss of the onepass extension

Ah okay, that shortcut is actually also for the system wide onepass program that integrates with the extension. I would really prefer not to change that as i have been using it for years and it is very convenient. It registers the hotkey globally.

@WebMechanic @Nova38 @pixiekat we will be updating the keyboard shortcut, thank you for your feedback!

The description at the top of this issue, and the documentation were both updated with the new Ctrl+Q shortcut that can be used starting with Microsoft Edge 108.

I love this feature in a browser. Vivaldi has a really nice implementation of this sort of feature that I would love to see expanded in Microsoft Edge.
One option I use constantly is the ability to use the command palette to close tabs to the right of my active tab. This seems like a small thing, but it's a huge quality of life feature for me. Here's an example in Vivaldi:
https://user-images.githubusercontent.com/2912202/198965713-b273a0e7-a28d-4b89-a8bb-2efbf5e6b7f4.mp4
Even if the Edge version doesn't explicitly support this option, I'd love to be able to define custom commands and select options like closing tabs to the right of an active tab.

Excited about this, lots of potential.
I am keen to add a command palette to my own product, but if we're heading towards a browser standard palette with a global shortcut (a la GitHub and Slack) then I'd rather be able to re-use the native one rather than implement my own in-app one, and register commands from my code (via the manifest.json maybe?) that can appear in the palette. Maybe the > prefix or similar could quick filter to app-specific commands.

One suggestion is to have capability to "Search Tabs" through Command Palette itself instead of having another shortcut for it. Result can be displayed similar to how historical tab search result is displayed.

One suggestion is to have capability to "Search Tabs" through Command Palette itself instead of having another shortcut for it. Result can be displayed similar to how historical tab search result is displayed.

I agree, having this directly in command palette, instead of having to switch the tab search window, would be a nicer experience.

VM66

If the mouse cursor hovers over the command palette while entering text, it will focus.
I would prefer that text input be the preferred method.

The ability to change the shortcut would be great.
VsCode has ctrl + p for files and ctrl + shift + p for tools. something like this would be great.

I wouldn't want ctrl + p in the browser window taken over by a feature I probably wouldn't use that often (no matter how useful it seems) as it's been assigned to "Print" since ever and in virtually every application on all systems that access printers.
There are likely as many people that like to "print" (to pdf) and thus want "ctrl+p" to do just that as there are nerds and geeks and self acclaimed power users that like to use it to open a devtools/vscode-like Command Palette and go from there (which always requires two or more steps to perform the most simple task/command).
Different users have different needs, and nobody here is THE ultimate reference user for any app whatsover, so by all means Microsoft: make this shortcut customizable so users can pick whatever they want according to their priority and let's not get into a futile discussion which shortcut is good or not or preferred by anyone.
Once a Command Palette is available I'm confident it can be stuffed with a sorts of features bit by bit 'cos unlike menu items and toolbar buttons it doesn't take any more screen estate to do so.
These features sound interesting to have, their proposed shortcut: not so much :)

showed the all the tabs in order of recently used
close all tab

Have a nice time.

Hello, the CTRL + Q shortcut interferes with Outlook web app. Previously you could toggle emails being read/unread and now that's unusable in Edge.

Thanks @martinjanocko, for reporting this. Does Ctrl+U works to mark emails as unread? And, what does Ctrl+Q do? Nothing? Open the Command Palette?
I'm asking because it seems to be working for me still. In any tab other than Outlook, Ctrl+Q opens the Command Palette, and in Outlook Ctrl+U and Ctrl+Q mark emails as unread/read without Command Palette interference.
Any extra detail you can share would help us understand the issue and therefore investigate it.

Since yesterday I had the command palette feature disabled in edge://flags/, but now when I wanted to test it again to be able to reply and enabled it, the CTRL + Q works as expected in OWA. So all good on my side, probably a temporary issue after MS Edge update not really sure what has happened. Sorry for the troubles.

Closing this issue. Thank you all for the feedback you have shared here while the issue was open. It helped us a lot work on this experiment.

Is this feature gone? Ctrl+Q doesn't do anything for me on 123.0.2420.51 beta macOS and I can't find the flag in edge://flags anymore either

Correct, we're removing this experimental feature starting with Microsoft Edge 123. A release note, which mentions this, will be published later today.

Aw, I'm sad to hear this but I know something better will come along with the feedback gathered!

This was really one of the top-notch features that made Edge my main browser for a long time. I really hate that it's gone, and I can't see a reason for its demise.

@Relin404 this experiment had never made it out of experimental mode, and our team doesn't have capacity to maintain it over time.
I'm curious though, which commands did you used most with it? If what you used most were DevTools commands, you can access them from within DevTools directly by using the Command Menu (Ctrl+Shift+P).


integrate external benefits (dash dividend, faucets)

Members of Encointer communities will soon enjoy more benefits than just their community-issued income:

DASH dividend: A first example of an external token that will be distributed to Encointer reputables
Faucets

I suggest to integrate these benefits on the Account overview page:

Dash dividend

user taps info button to learn more about dash and the dividend and how to install a dash wallet (and importing their account using the same 12-word mnemonic as for encointer! This simplifies things)
user taps "claim". the app will then create a dash wallet from the same mnemonic that is stored for the currently displayed account.
the app signs a message with the Encointer account. the message includes the new dash account address. This will say"I'm a reputable and this is my dash wallet". This signed message should be sent to a backend and collected for the dash community to fetch and see how many people are claiming and to be able to verify their reputation
the claim button disappears and is replaced by

a copy-paste button to copy paste the dash address
if simple: display balance of dash account

Faucets
The first few faucets will only support the KSM token. Still, there might be multiple

grouped by token: for each token (i.e. KSM) display an info button and the current balance of the displayed account in that token
3 for each faucet, app displays faucet name and claim button
user taps "claim". the app will then call faucet drip extrinsic, claiming with all the reputation(s) available.
claim button disappears until there is fresh unused reputation

question:

Should claiming should always be lazy or does the app remember the claim and re-claim whenever there is fresh reputation?

Questions answered: Only active claiming for the moment.


Watch i18n service for locale change and update labels

When switching locale, label_text was not recomputed, because it was not aware of the locale. I added i18n.locale to the watch list.

Could you please add some tests?


Updated appengine.py to conforms with values in redirect_uris

In downloaded client_secrets.json redirect_uris is a list with URIs. This change add redirect_uri_index parameter in oauth2decorator_from_clientsecrets and OAuth2DecoratorFromClientSecrets. It defines which item from redirect_uris to be used.
P.S. Can someone suggest better variable name than redirect_uri_index ?

(1) Please update your commit message to conform to chris.beams.io/posts/git-commit/#seven-rules.
(2) What hope is there for test coverage for this change?

Closing this out. This "feature" can be achieved by someone just editing the contents of their clientsecrets.json files.


Target Sensor calculation

Describe the bug
It looks like there may be a bug in its calculation, see images.
Current rates sensor has data untill -

start: '2023-12-20T22:30:00+00:00'
end: '2023-12-20T23:00:00+00:00'
value_inc_vat: 0.02772

this can also be seen in the UI on the dashboard.
with target sensor set 09:00 - 22:30 for 0.5 I would assume the 22:30 slot to be the one chosen but its missed.

at the moment I cant get into vs code so i cant remove the error for agile times and test 23:30..
Reproduction steps
as above
Expected behaviour
as above
Tariff Code
agile
Integration Version
9.2.1
Home Assistant Version
latest
Fresh Install?
Not specified
Home Assistant Logs
n/a
Confirmation

[X] I confirm that I cannot find my solution within the documentation
[X] I confirm that I cannot find my solution within the FAQ

Hello. 22:30 has been set as the end time, whereas the slot you're referring to starts at 22:30 so wouldn't be picked.


Pylance 2021.1 builds can't interpret multiple dispatch

Environment data

Language Server version: 2021.1.0 and 2021.1.1
OS and version: Windows 10 Build 19042
Python version (& distribution if applicable, e.g. Anaconda): 3.6.8

Expected behavior
Semantic highlighting and code navigation functional
Actual behavior
All code after multiple dispatch ( https://pypi.org/project/multipledispatch/ ) invocation reads as unreachable

This library is untyped, and does something pretty interesting that can't really be figured out statically.
from multipledispatch import dispatch

@dispatch(int, int)
def add(x, y):
    return x + y

@dispatch(object, object)
def add(x, y):
    return "%s + %s" % (x, y)

add(1, 2)        # 3
add(1, 'hello')  # '1 + hello'

I'd think this would need a stub, but I don't quite know how this library can be described within the typing spec. The library also doesn't seem to be getting maintained, so stubs would have to be provided a different way.

This library is untyped, and does something pretty interesting that can't really be figured out statically.
from multipledispatch import dispatch

@dispatch(int, int)
def add(x, y):
    return x + y

@dispatch(object, object)
def add(x, y):
    return "%s + %s" % (x, y)

add(1, 2)        # 3
add(1, 'hello')  # '1 + hello'

I'd think this would need a stub, but I don't quite know how this library can be described within the typing spec. The library also doesn't seem to be getting maintained, so stubs would have to be provided a different way.


Make option to disable module visible in GUI

As stated in #32 with this change you can enable/disable inputstreamhelper via GUI at
"kodi settings" > "system" (change from standard to expert there) > "addons" > "manage dependencies" > "Inputstream Helper" > "configure".

I guess breaking compatibility with Kodi versions before Leia is not wanted (yet)? If you want to I can revert the second commit, doesn't change the functionality anyway.

I'd like to postpone this for the v0.5.0 release as this has an impact on how the plugin behaves.
One concern I have is that users do not have an friendly way to access this plugin's settings.
So this would then only be useful if (consumer) add-on provide a settings entry to access this one's.
And that will only be useful if the settings has a wider use.
PS Can you please rebase your PR?

Done.

OK, the commits have been really fucked up by the rebase, but the end result is fine. I think there's no harm in merging this and building a full-fledged setting page based on it.

I agree, this doesn't require any API calls. Let's get you out of this misery ;-)

Thanks

While reworking the interface for the future, I think we should redesign this interface.
Interfaces should list functionality, and functionality can be enabled or disabled.
The problem here is that "disabled" is actually a negative imposition. The value "true" should be enabled, and a value "false" should be disabled. And by default this functionality is enabled. This makes it a much better interface for users.
The problem with changing this is two-fold, either we support people that have used this manually and make it more complex and error-prone, or we don't, and those advanced users would need to reapply. IMHO I don't think we should be adding complexity, and expect that users do not have it disabled. (Especially with the recent Widevine CDM breakage, users have it enabled again ;-))
I will propose this in a subsequent PR.