feat: add  support

The inverse of <!-- panvimdoc-ignore-start/end -->, where the content is only visible in the vimdoc output. Because we have no control on the mardown renderer, all content must be embedded in the comment, compared to using a start and end marker.
The test is written expecting #47 to be merged.

That is a neat feature! Can we change the name to panvimdoc-include-comment-content?

I was thinking of making another PR to augment the existing "include" function, since as written, the {.include} block ends up in the markdown, eg as:
other-doc.md

Where as if we used a comment it would be invisible, you could have something like  <!-- panvimdoc-include other-doc.md arg arg -->.
Obviously I have not written that yet, or looked at how complex it would be to do -- though I guess it should be the same as the current include, just matching on another block type?
Would that influence what name you'd want to give this and the other theoretical PR?
Happy to make whatever change you prefer of course, panvimdoc-include-comment-content and panvimdoc-include-file-content?

Interesting PR idea! That PR would be welcome. I believe you can reuse some of the machinery for the existing include functionality.

panvimdoc-include-comment-content and panvimdoc-include-file-content

These seem good to me. I'm open to suggestions as well!

I think I like panvimdoc-include-comment because it is shorter. I made the change corresponding to that and merged it! Thanks for the PR!


UserJoined/UserLeft events not working

I've already seen other issues talking about that but I don't know why my code didn't work.
My start function with the config of the socket (I already tried a lot of configurations)
Async Sub Start()
        Dim client_config As New Discord.WebSocket.DiscordSocketConfig
        client_config.GatewayIntents = Discord.GatewayIntents.All
        Client = New DiscordSocketClient()
        AddHandler Client.MessageReceived, AddressOf MessageReceived.MessageReceived
        AddHandler Client.UserJoined, AddressOf UserJoined.UserJoined
        AddHandler Client.UserLeft, AddressOf UserLeft.UserLeft
        AddHandler Client.ReactionAdded, AddressOf ReactionAdded.ReactionAdded
        AddHandler Client.ReactionRemoved, AddressOf ReactionRemoved.ReactionRemoved
        AddHandler Client.InteractionCreated, AddressOf InteractionCreated.InteractionCreated
        AddHandler Client.Ready, AddressOf Ready.Ready

        Await Client.LoginAsync(Discord.TokenType.Bot, Config.Token)
        Await Client.StartAsync()
    End Sub

My UserJoined function
Shared Async Function UserJoined(ByVal _user As SocketGuildUser) As Task
        CLI.Log("User joined : " & _user.Username)
        Dim user_image_brush As Drawing.Brush = New TextureBrush(New ImageConverter().ConvertFrom(New Net.WebClient().DownloadData(_user.GetAvatarUrl())))

        Dim _Width = CInt(Math.Max(380, 136 + TextRenderer.MeasureText(_user.Username, Fonts(Font.HugeSemiBold)).Width + TextRenderer.MeasureText(_user.ToString.Replace(_user.Username, ""), Fonts(Font.BigSemiBold)).Width))
        Dim flag As New Bitmap(_Width + 64, 128)
        Dim g As Graphics = Graphics.FromImage(flag)

        [Draw functions (I removed)]

        flag.Save("renders/JoinRender_" & _user.Id & ".png")

        Await _user.Guild.GetTextChannel(ServersManager.GetOrCreateServer(_user.Guild.Id).JoinLeftChannel).SendFileAsync("renders/JoinRender_" & _user.Id & ".png", LanguageManager.getText(_user.Guild.Id, "UserJoined_Card_Welcome"),)

    End Function

I enabled all Privileged Gateway Intents

But I haven't any log or message when a user join or left.

You create a DiscordSocketConfig but you never actually use it in your DiscordSocketClient. Your client_config should be passed to the client constructor.

Thanks, I'm just stupid...


[BUG] Cannot communicate with actors after stopping all instances

Describe the bug
When the last instance of an actor is stopped, the actor's RPC supervisor is terminated. There's a bug in the logic to start actor RPC supervisors that prevents starting one that used to exist
To Reproduce
Steps to reproduce the behavior:

Start one or more copies of the echo actor
Notice you can successfully invoke the actor
Terminate all instances of the actor
Notice you can't invoke the actor (because it's not running)
Start the echo actor again
Notice you still can't invoke the actor

Expected behavior
RPC supervisors should gracefully restart when an actor is restarted

Closed with #429


Adjusted README to actually link to github pages

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   96.87%   96.87%
=======================================
  Files           1        1
  Lines          32       32
=======================================
  Hits           31       31
  Misses          1        1

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 4318f94...192b470. Read the comment docs.


[API DESIGN REVIEW] Dynamic Batch Size

This proposal means to add the capability to dynamically change the batch size while training a model. As this paper presented in the ICLR 2018 conference shows, many tasks can benefit from this kind of methodology.
It proposes that as training goes on, a method that speeds up training without losing accuracy is to periodically increase the batch size of the training set. It proves that not only  it reaches loss values almost identical to training without it, but the increase of batch size speeds up the training significantly.
This is a really useful feature both to people that are starting to learn about machine learning and want to experiment on diverse datasets to more professional applications with uneven datasets.

Here is the full proposal:
https://docs.google.com/document/d/1XPQ0cz_2pN7A7z2WydUWXhkF5B2foJhYGM6_Jy1RZW4/edit?usp=sharing

Could this functionality be implemented by creating a descendent of keras.util.Sequence which implements the desired behavior rather than adding a new parameter to fit ?
It may make the implementation clearer vs adding yet another parameter.

Certainly, I just saw an easy and fast way to do it.
This method also will force to use Sequence (maybe there are people against
it?).
Absolutely a more convenient way to do this as it will fit nicely for
multiprocessing. However it also increases difficulty of implementing, not
by much though.
Cheers,
Alex
El lun., 25 feb. 2019 a las 11:01, Pedro Marques<EMAIL_ADDRESS>escribió:

Could this functionality be implemented by creating a descendent of
keras.util.Sequence which implements the desired behavior rather than
adding a new parameter to fit ?
It may make the implementation clearer vs adding yet another parameter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keras-team/keras/issues/12314#issuecomment-467067556,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Al7CWSntmb2oxwOyr9dJi8LsRYzhx6LSks5vRAjggaJpZM4bGqgV
.

Certainly, I just saw an easy and fast way to do it. This method also will force to use Sequence (maybe there are people against it?). Absolutely a more convenient way to do this as it will fit nicely for multiprocessing. However it also increases difficulty of implementing, not by much though. Cheers, Alex El lun., 25 feb. 2019 a las 11:01, Pedro Marques<EMAIL_ADDRESS>escribió:
…
Could this functionality be implemented by creating a descendent of keras.util.Sequence which implements the desired behavior rather than adding a new parameter to fit ? It may make the implementation clearer vs adding yet another parameter. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#12314 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/Al7CWSntmb2oxwOyr9dJi8LsRYzhx6LSks5vRAjggaJpZM4bGqgV .

When I mean "increase difficulty of implementing I mean to the user.

Could this functionality be implemented by creating a descendent of keras.util.Sequence which implements the desired behavior rather than adding a new parameter to fit ?
It may make the implementation clearer vs adding yet another parameter.

Also technically its not a new argument just that the parameter batch_size will accept a 2nd type of argument.

I believe that this kind of use case is better fitted for a sequence or a generator. We are going to make the API too bloated if we try to take care of all use cases. Since users can make a sequence and a generator easily, I would consider that there is no need for a change in the API. If we see that this type of training becomes a best practice and resist the test of time, then we'll enable it in model.fit.


docs: update code example setRequestInterceptionEnabled

Good catch! Thank you so much for the PR.

I signed it!


Improved presentation flexibility

This pull requests adds logic to handle a presentation definition more diligently, but this unfortunately leads to some changes in semantics expected from the "createPresentation" function before that. I hope the changes are reasonable, I tried following the guide from PresentationExchange at:

https://identity.foundation/presentation-exchange/spec/v2.0.0/#input-evaluation

I tried to stay conform with openId4vp, but I hope that we don't need to support other presentation specifications?
Semantic changes to be approved:
Presentation Submissions now only contain matches from input descriptors.
Previously a verifiable presentation contained all the verifiable credentials with the required credential scheme, but not anymore.
Each input descriptor matches to exactly one verifiable credential, and that verifiable credential must satisfy all constraint fields.
This may actually break some things, as it was previously possible to present multiple AtomicCredentials at once, but after this change each AtomicCredential attribute needs to be requested separately (in separate input descriptors).
-> result.vp.verifiableCredentials now always has size 1!
Impacted Tests:

AgentTest."valid presentation is valid": change last "shouldHaveSize" to "1"
ValidatorVpTest."valid parsed presentation should separate revoked and valid credentials": Change last "shouldHaveSize" to "issuedCredential.successful.size"

Presentation Submissions where no fields are requested don't return any fields
Previously it seems like not requesting any claims should return all available claims, but not anymore.
Impacted Tests:

OidcSiopIsoProtocolTest."test with Fragment for mDL": request an attribute to satisfy the "document.validItems.shouldNotBeEmpty()" assertion
OidcSiopIsoProtocolTest."test with Fragment for custom attributes": request an attribute to satisfy the "document.validItems.shouldNotBeEmpty()" assertion

Scope value semantics have somewhat changed
Previously the scope values were used to deduce the requested credential schemes, but not anymore.
As per openid4vp the scope value has the following semantics:

Such a scope value MUST be an alias for a well-defined Presentation Definition that will be referred to in the presentation_submission response parameter.
https://openid.net/specs/openid-4-verifiable-presentations-1_0.html#section-5.3-2

scope values are therefore only taken into consideration now in case neither presentationDefinition nor presentationDefinitionUri return a presentation definition.

Closed due to further work on JSONParser


Include client-ip header on event object

Hey there 👋
- Do you want to request a feature or report a bug?
Bug
- What is the current behavior?
It appears that although it exists on a deployed function, the client-ip header is not set when running netlify dev locally.
- If the current behavior is a bug, please provide the steps to reproduce.
I'm using a "dump" route that just returns the entire event and context objects as JSON, a la:
exports.handler = async (ev, ctx) => {
  return {
    statusCode: 200,
    body: JSON.stringify({
      ev,
      ctx
    })
  }
})

- What is the expected behavior?
It should be populated locally, if available, like it is in production.
- Please mention your node.js, and operating system version.
Node: v10.15.3
OS: macOS Mojave 10.14.5

P.S. Originally posted this on the Netlify Community.

Fixed in latest merge https://github.com/netlify/cli/pull/396/commits


Modules in DSC_Resorces cant be used to manage machine running invoke-DSCBuild.

when I try to update my tools and it brakes I alwase fall back to a clean seutp and the sample build.
when it Trys to run I get some errors I think is related to the recent change of folders, having the resorces in both the DSC_Resorce folder and C:\Program Files\WindowsPowerShell\Modules (required for ISE to validate SampleConfiguration)
VERBOSE: Performing the operation "Test-DscResourceIsValid" on target "modules from H:\DSC\DSC_Tooling\Examples
DSC_Resources".
ImportCimAndScriptKeywordsFromModule : A second CIM class definition for 'PSHOrg_cSmbShare' was found while
processing the schema file 'C:\Program
Files\WindowsPowerShell\Modules\cSmbShare\DSCResources\PSHOrg_cSmbShare\PSHOrg_cSmbShare.Schema.mof'. This
class was already defined in the file 'H:\DSC\DSC_Tooling\Examples\DSC_Resources\cSmbShare\DSCResources\PSHOrg_
cSmbShare\PSHOrg_cSmbShare.Schema.mof'. Remove the redundant definition, and then try again.
At C:\windows\system32\windowspowershell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.p
sm1:2426 char:29

                $null = ImportCimAndScriptKeywordsFromModule -Module $mod -R ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], PSInvalidOperationException
  + FullyQualifiedErrorId : System.Management.Automation.PSInvalidOperationException,ImportCimAndScriptKeywo
 rdsFromModule

....

it seems to be that line 11 of  Test-DscResourceIsValid.ps1 is the point of failure
$AllResources = Get-DscResource | Where-Object {$_.ImplementedAs -like 'PowerShell'}
This is due to the path containing both SourceResourceDirectory and the Program Files location.
changeing the line to
$AllResources = Get-DscResource -ErrorAction SilentlyContinue | Where-Object {$_.ImplementedAs -like 'PowerShell'}
gets passed the error but it still fails to create the .mof files.

Hmm... the way I was using this in testing was to clear out everything in the PSModulePath that wasn't needed prior to running Invoke-DscBuild.  Something like this:
$dependenciesDirectory = Join-Path $PSScriptRoot Dependencies

$env:PSModulePath = "$dependenciesDirectory;$pshome\Modules"

Invoke-DscBuild # etc

It looks like maybe we should make this a parameter to Invoke-DscBuild instead.  It would automatically include $pshome\Modules and the DSC_Resources directories on the path when running the build, and you could optionally pass in other paths to Invoke-DscBuild as well (where you have things like Pester, cDscResourceDesigner, etc.)  That way you don't run into this problem if the build server happens to have duplicate modules on its normal module path.

Both Resource build and Configuration build will fail if any resource is both in program files and dsc_Resources.
while Resource build gives useful errors, Configuration does not.
I also found one of my resources was faulty and was braking configuration build, removing it solved it so now I can build scripts but only if I have my build.ps1 compare dsc_Resources to Program Files and removes resources, then puts them back at the end. This is not a good fix but it's got me working with the latest version.

I've just pushed a change to the Development branch to try to address this.  Here's the new behavior:

By default, Invoke-DscBuild will now set the PSModulePath during its run so that only the $SourceResourceDirectory and $pshome\Modules folders are on the path.
There's a new parameter to Invoke-DscBuild called -ModulePath, which takes an optional array of strings for additional folders that should be on the module path.  You might need to set this to make Invoke-DscBuild load $ConfigurationModuleName, for example.

The idea here is that everything DscBuild needs in order to do its job should be located somewhere other than the default PSModulePath on the build server.  This way, it can run a build and test new versions of these modules without affecting the production state of the build server (which may also be managed by DSC.)
Let me know if this makes sense, and if it helps with the problems you were having.

Thank you, that worked. I had to add the configuration location, in this case dsc_tooling
so my settings look like this

This is no my day, it initially worked when I just updated the one file. but I thin copied all the Tools back into the module folder just to make sure my poking around was reversed, and now it's broken.
And when it fails to load the configuration it does not give any useful errors.
I'm going to try to pinpoint the failure to see if it's my config script or Invoke-DscConfiguration.

You may be hitting a bug with caching of resources.  Is the LCM in debug mode? Or have you killed the wmi process hosting the lcm?

I haven't been using the DSC_Tooling functionality of DscBuild at all yet (and am not really sure what the use case is, there).  Do you think $SourceToolDirectory should also be on the default module path when Invoke-DscBuild is running?

$BaseSettings = @{
WorkingDirectory         = (Get-TempDirectory).FullName
SourceResourceDirectory  = "$PSScriptRoot\DSC_Resources"
SourceToolDirectory      = "$PSScriptRoot\DSC_Tooling"
DestinationRootDirectory = "$outBasePath\DSC_Out"
DestinationToolDirectory = "$outBasePath\DSC_Tools"
ModulePath               = "$PSScriptRoot\DSC_ConfigScript"
ExcludedModules          = 'Example', 'MySConfig', 'SampleConfiguration'
ConfigurationModuleName  = 'SampleConfiguration'
}
$TestInvokeConfig = @{
ConfigurationName = 'SampleConfiguration'
Configuration     = $true
Resource          = $flase
Tools             = $false
}
Invoke-DscBuild @BaseSettings @TestInvokeConfig -ConfigurationData (Get-DscConfigurationData -Path  "H:\DSC\DSC_Tooling\Examples\DSC_Configuration" -Force  -Verbose) -Verbose
...
VERBOSE: Is a Configuration Build - True
VERBOSE:
VERBOSE: Performing the operation "Invoke-DscConfiguration" on target "Configuration module SampleConfiguration and co
nfiguration SampleConfiguration".
VERBOSE:
VERBOSE: Importing configuration module: SampleConfiguration
VERBOSE: Loading module from path 'H:\DSC\DSC_ConfigScript\SampleConfiguration\SampleConfiguration.psm1'.
VERBOSE: Loading module from path 'H:\DSC\DSC_ConfigScript\SampleConfiguration\SampleConfiguration.psm1'.

Import-Module : The specified module 'DscConfiguration' was not loaded because no valid module file was found in any
module directory.
At H:\DSC\DSC_ConfigScript\SampleConfiguration\SampleConfiguration.psm1:2 char:1

Import-Module DscConfiguration -ErrorAction Stop

  + CategoryInfo          : ResourceUnavailable: (DscConfiguration:String) [Import-Module], FileNotFoundException
  + FullyQualifiedErrorId : Modules_ModuleNotFound,Microsoft.PowerShell.Commands.ImportModuleCommand

Import-Module : The specified module 'SampleConfiguration' was not loaded because no valid module file was found in
any module directory.
At line:1 char:1

Import-Module -Name SampleConfiguration -Verbose

  + CategoryInfo          : ResourceUnavailable: (SampleConfiguration:String) [Import-Module], FileNotFoundExcepti
 on
  + FullyQualifiedErrorId : Modules_ModuleNotFound,Microsoft.PowerShell.Commands.ImportModuleCommand

PS H:> $env:PSModulePath = $pathold

think I may have it.
ModulePath               = "$PSScriptRoot\DSC_ConfigScript", "$PSScriptRoot\DSC_Tooling"

Looks like i'm good. all my configurations build, and I got all my configs and updated tools checked into the onprem git.


[Bug] . and ) don't work in page name

When creating / renaming a page inside an Appsmith application, the characters . and ) are not allowed. Ideally, we should allow all characters to be typed inside the page name without restriction.

I would like to worn on this if i may get the proper path to the code file of this section .

@riodeuno Can you help @Jain-Sanchit with the relevant files that he can use to fix this issue?

@Jain-Sanchit The page entity names are resolved using the function resolveAsSpaceChar in app/client/src/utils/helpers.tsx. This function takes the user entered string and returns a new value to allow only spaces (currently), by replacing all consecutive non-alphanumeric characters (regex \W) with one space character.
Hope this helps.

@riodeuno Thanks for the info . I will get onto this and try solving the issue.

@satbir121  @riodeuno Can you please let me know where is being this function actually used , in which component , as there may be scenarios that I update the function but function was needed somewhere else . So is there any way to know where is this function is being used in the code ? So that before raising a pull request I will be sure about its correctness .
Thanks

@Jain-Sanchit If you're using an IDE like VSCode, you can right click on the function name and choose "Find all references". A quick google search for your IDE will tell you the flow (if you aren't using VSCode).


Fixed setup image version, Enterprise installation support

Enterprise installation wasn't working using the 7.0 image.
The setup script included in that image doesn't support the M2SETUP_USE_COMPOSER_ENTERPRISE variable.
Looking at the image used for phpfpm, it seems like the setup one should have been bumped too?

@careys7 thank you sir for your updates

@markoshust - apologies - I was just about to cancel this PR as it needs more work:
http://devdocs.magento.com/guides/v2.0/install-gde/system-requirements-tech.html#php

Not supported:
PHP 7.1
PHP 5.4

@careys7 i find it hard to believe that 7.0 is supported... but 7.1 is not? 🙃

Yes I know 👎 It looks like there is a few small incompatibles: https://github.com/magento/magento2/issues/7663
Composer now fails on the PHP requirement during installation for Enterprise, and I assume Community is the same. I should have waited for this to complete before opening PR.
Possibly the best option is to add the M2SETUP_USE_COMPOSER_ENTERPRISE flag into a PHP70 image for the time being?


Syntax highlighting feature [not an issue More like a feature request]

i wonder if it is possible to color the words like the rust addon

This extension by itself won't support that, but I was able to achieve something similar using a patched version of zls.


Chart is not drawn on Edge (Windows)

Lightweight Charts Version: 1.0.2
Steps/code to reproduce:
See jsfiddle link below
Actual behavior:
Bug founded on latest edge version on Windows (44.18362.267)
When setted rgb
chart.addAreaSeries({
    bottomColor: 'rgb(0, 120, 255, 0.0)',
});

Instead of rgba
chart.addAreaSeries({
    bottomColor: 'rgba(0, 120, 255, 0.0)',
});

Expected behavior:
Chart is visible?
Screenshots
Any other browser:

Edge:

CodeSandbox/JSFiddle/etc link:
https://jsfiddle.net/syllik/6mt7vn3e/3/

in this fiddle,,   i changed line #8
to rgba()     you had rgb()
bottomColor: 'rgba(0, 120, 255, 0.0)',

im guessing this is an EDGE issue,  because  the other browsers dont care if rgb has 4 arguments

Thanks for answer, I wrote here just in case anyone has a similar problem.
I think we can close this issue


Disable Accessibility Access override for OSX 10.12

Duplicate of https://github.com/Homebrew/brew/issues/905.

TCC.db is (rightfully) protected by SIP in Sierra.

We should prevent enable_accessibility_access  if MacOS.release >= :sierra (throw up a caveat instead?)

Moving it here since we have other core bugs here.
@reitermarkus @jawshooah What’s your take on issues related to HBC? Here or on HB’s repo?

Not sure where it goes but if no one picks it up, I can probably add in the conditional this weekend. (Interestingly, I think fixing accessibility for 10.11 was my very first core HBC contribution)

What’s your take on issues related to HBC? Here or on HB’s repo?

For now, I'd leave them here as there are still quite a few open, and when they are down to a smaller number, migrate them to Homebrew.

Also, we could add minor issues which are low priority but can be fixed relatively easily to the newly introduced Projects tab.

Was also thinking of adding issues with ready to implement, there. And perhaps even cask requests and documentation requirements.
Something to close https://github.com/caskroom/homebrew-cask/issues/17337, basically. Something we can point users to help where they can pick what to help with depending on their level of expertise.

Which component of SIP needs to be disabled for this to work? Should TCC.db access be disabled entirely for macOS >= Sierra, or should it check the SIP config first and handle appropriately? (Also note disable_accessibility_access should be prevented as well.)

Disabling SIP is not something we should either be trying to do or encourage users to do.

Closing in favour of https://github.com/Homebrew/brew/pull/1029.


Unable to load prisoner's dilemma

Sorry if there's a more appropriate avenue for asking questions (didn't see a google group discussion).
I would like to play the prisoner's dilemma game, and I can see code for it exists.
However, I can't get the game playing using the below code:
import pyspiel
game = pyspiel.load_game("prisoners_dilemma")
game = pyspiel.load_game("python_iterated_prisoners_dilemma")
game = pyspiel.load_game("iterated_prisoners_dilemma")
OpenSpiel exception: Unknown game 'iterated_prisoners_dilemma'. Available games are:
2048
add_noise
amazons
backgammon
bargaining
battleship
blackjack
blotto
breakthrough
bridge
bridge_uncontested_bidding
catch
checkers
chess
cliff_walking
clobber
coin_game
colored_trails
connect_four
coop_box_pushing
coop_to_1p
coordinated_mp
crazy_eights
cursor_go
dark_chess
dark_hex
dark_hex_ir
deep_sea
dots_and_boxes
dou_dizhu
efg_game
euchre
first_sealed_auction
gin_rummy
go
goofspiel
hanabi
havannah
hearts
hex
kriegspiel
kuhn_poker
laser_tag
leduc_poker
lewis_signaling
liars_dice
liars_dice_ir
maedn
mancala
markov_soccer
matching_pennies_3p
matrix_bos
matrix_brps
matrix_cd
matrix_coordination
matrix_mp
matrix_pd
matrix_rps
matrix_rpsw
matrix_sh
matrix_shapleys_game
mfg_crowd_modelling
mfg_crowd_modelling_2d
mfg_dynamic_routing
mfg_garnet
misere
morpion_solitaire
negotiation
nfg_game
nim
nine_mens_morris
normal_form_extensive_game
oh_hell
oshi_zumo
othello
oware
pathfinding
pentago
phantom_go
phantom_ttt
phantom_ttt_ir
pig
quoridor
rbc
repeated_game
restricted_nash_response
sheriff
skat
solitaire
spades
start_at
stones_and_gems
tarok
tic_tac_toe
tiny_bridge_2p
tiny_bridge_4p
tiny_hanabi
trade_comm
turn_based_simultaneous_game
twixt
ultimate_tic_tac_toe
universal_poker
y
zerosum

I am running the presumably latest version of open_spiel
$ pip show open_spiel
Name: open-spiel
Version: 1.5
Summary: A Framework for Reinforcement Learning in Games
Home-page: https://github.com/deepmind/open_spiel
Author: The OpenSpiel authors
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/shaoyu/.local/lib/python3.10/site-packages
Requires: absl-py, attrs, ml-collections, numpy, pip, scipy
Required-by:

Thanks in advance for the guidance.

Ah, yes: it's because it's called matrix_pd.  (To emphasize that it's a matrix game.)

Thanks for the pointer!
Got everything working now. By the way, thanks for this library which I think is an important contribution to the entire scientific community.

Great, you're welcome :)


#1152: Fixed header for small menu and empty page puts footer below t…

…he scroll. https://github.com/almasaeed2010/AdminLTE/issues/1152

Now the height of the content wrapper is correct.


Talk about local helper macros

This probably should get its own page in the minutiae category?
See https://doc.rust-lang.org/edition-guide/rust-2018/macros/macro-changes.html#local-helper-macros

What do you mean by local helper macros? When I follow that link, it doesn't mention "local" or "helper".

Looks like they removed it from the book which makes sense, it's not really necessary nowadays. So I don't think we need this section anymore, for the curious, it's still somewhat explained in the reference https://doc.rust-lang.org/reference/macros-by-example.html#hygiene


Lost 0 (I could be wrong)

https://github.com/A3Antistasi/A3-Antistasi/blob/f58f7e40e1fc860bef0f1c177d639219737e3158/A3-Antistasi/defines.hpp#L107
must be:
colorText[] = {1,1,1,0.5};

Same as the other: no, as long as you may see the statistics bar.

why don't you fix it? or don't you want to make it nice?


ddnet: 15.5.4 -> 15.6

Motivation for this change
Upgrades ddnet from 15.5.4 to 15.6
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

I apologize for the vscode commit. I forgot to switch to my previous branch.

I am getting a build error, could you please double-check the hash?:
$ nix run 'github:ElXreno/nixpkgs/package-ddnet-15.6#ddnet'
error: hash mismatch in fixed-output derivation '/nix/store/sx8xxsk9vhrh2xs6qf4vz610avas347h-source.drv':
         specified: sha256-81xW82LPeqb+O56NC0q0ThkiRfzc/X9xT52b9XxWqV0=
            got:    sha256-5nAdmPUAvmFVcdupggNPfct1dVRKy7hJDJ8jQys/VNM=
error: 1 dependencies of derivation '/nix/store/3zs9f26zfc4m8f0gq51jwkil73v3kpa2-ddnet-15.6.drv' failed to build

@endgame It seems that the hash was wrong because I used nix-prefetch-github without the --no-fetch-submodules parameter. Thanks for the check!

@SirSeruju you don't need to re-approve if ofborg takes a bit to eval and requests it after you already approved the latest push.


Reaction MAR05998 should be listed as reversible

Description of the issue:
Lactate transport reaction MAR05998 should be marked as reversible. Sources: Uniprot, KEGG.
Expected feature/value/output:
H+ [s] + L-lactate [s] ⇔ H+ [c] + L-lactate [c]
Current feature/value/output:
H+ [s] + L-lactate [s] ⇒ H+ [c] + L-lactate [c]
I hereby confirm that I have:

[X] Tested my code on my own computer for running the model
[X] Done this analysis in the main branch of the repository
[X] Checked that a similar issue does not exist already

Thanks for reporting this! How do you feel about implementing the expected fix? We have created Contributing guidelines as guidance.
@all-contributors please add @stairs for the bug report.

@stairs thanks for reporting this.
With the discussion in gitter, please go ahead to make a fixing PR.

I'd be happy to make a PR later in the evening (unless somebody decides to fix it earlier of course).


Test all cases of VXRM.

Currently only vxrm=0 is tested, I think

Added vxrm traversal for vaadd*, vasub*, vnclip*, vsmul*, vssra* and vssrl* instructions. I believe that's all the instructions that use vxrm.

Thanks! This is much appreciated


Add pandoc autoformat rule to makefile.

Reviewed 2 of 3 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


[zh] merge dev-zh into main

Follow #1090
Checklist to make localization live.

[x] cherry-picks commits from #1094 into dev-zh
[x] update cncf:dev-zh with cncf:main #1024

AND, Thx very much for @jihoon-seo guidance and @Petrie assistance!

It would be better to merge this PR after the PR #1104 gets merged into dev-zh.
For who merges this PR: please feel free to unhold (or just ignore the hold label) and merge if the PR #1104 has been merged!


Add algorithms course by Harvard

What does this PR do?
This PR adds a free course to teach algorithms made by Harvard
For resources
Description
This is a free course for algorithms by Harvard.
Why is this valuable (or not)?
This is a great course that teaches algorithms foundations with an incredible professor.
How do we know it's really free?
At the 'CS50 Certificate' section you can read that you only need to pay for the certificate but the course is Free.
For book lists, is it a book? For course lists, is it a course? etc.
It's a course.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

whay do you link to week 3?

whay do you link to week 3?

It was a mistake I already updated.
@eshellman


Reconciliation should happen, if there is secret change for CSI driver

Describe the bug
Reconciliation should happen, if there is secret change happen for CSI driver
How to Reproduce?
Please list the steps to help development teams reproduce the behavior

Deployed the driver and operator with gooperator dev image

[root@csiscale-master yamls]# oc get pods -w
NAME                                              READY   STATUS    RESTARTS   AGE
ibm-spectrum-scale-csi-2v6xp                      3/3     Running   0          139m
ibm-spectrum-scale-csi-attacher-0                 1/1     Running   3          34h
ibm-spectrum-scale-csi-j67nd                      3/3     Running   0          138m
ibm-spectrum-scale-csi-operator-996fdcb6c-k4f48   1/1     Running   0          34h
ibm-spectrum-scale-csi-provisioner-0              1/1     Running   3          34h
ibm-spectrum-scale-csi-resizer-0                  1/1     Running   3          34h
ibm-spectrum-scale-csi-snapshotter-0              1/1     Running   3          34h

Deleted CSI Spectrum Scale Rest API secret

[root@csiscale-master yamls]# oc delete secret secretlocal
secret "secretlocal" deleted
Note : There was no impact ( reconciliation) on the csidriver pods, they were still running

Create secret using below file ( with wrong password)

[root@csiscale-master yamls]# cat secret1.yaml
apiVersion: v1
data:
  password: YWRtaW4wMDE=
  username: Y3NpYWRtaW4=
kind: Secret
metadata:
  labels:
    app.kubernetes.io/name: ibm-spectrum-scale-csi-operator
    product: ibm-spectrum-scale-csi
  name: secretlocal
  namespace: ibm-spectrum-scale-csi-driver
type: Opaque

Note: Again there was no impact ( reconciliation) on the csidriver pods, they were still running
Expected behavior
Reconciliation should happen, if there is secret change happen for CSI driver

@nitishkumar4 Please work on this issue. I am not able to assign it to you.

Reconciliation should happen, if there is secret change happen for CSI driver
Controller reconciles when there is change in secret and also when secret is created and deleted.

@Jainbrt  I assume the ask here is that the driver pod should be restarted when secret is created and deleted.

@nitishkumar4 Please check this issue for CSI 2.6.0

Verified this issue on CSI build 280 using below steps and found that it is working as expected i.e the Reconcilation happens on deletion and creation of secrets. Hence closing the defect.

All the CSi Pods are running for version 2.8.0
[root@multigui20-master ~]#  kubectl get pods -n ibm-spectrum-scale-csi-driver

NAME                                                  READY   STATUS    RESTARTS   AGE
ibm-spectrum-scale-csi-attacher-5c5f879f9b-f28d4      1/1     Running   0          24h
ibm-spectrum-scale-csi-attacher-5c5f879f9b-khstc      1/1     Running   0          24h
ibm-spectrum-scale-csi-nlsfn                          3/3     Running   0          95s
ibm-spectrum-scale-csi-operator-647d488b4f-fvkbj      1/1     Running   0          24h
ibm-spectrum-scale-csi-provisioner-6956678877-fxr5h   1/1     Running   0          24h
ibm-spectrum-scale-csi-resizer-bfb8f6df-cjdmc         1/1     Running   0          24h
ibm-spectrum-scale-csi-snapshotter-6cf649bb46-8xqr4   1/1     Running   0          24h
ibm-spectrum-scale-csi-vg7qg                          3/3     Running   0          2m53s

Deleted CSI Spectrum Scale Rest API secret
[root@multigui20-master ~]# kubectl get secret -n ibm-spectrum-scale-csi-driver
NAME        TYPE     DATA   AGE
guisecret   Opaque   2      5m16s

[root@multigui20-master ~]# kubectl delete  secret guisecret -n ibm-spectrum-scale-csi-driver
secret "guisecret" deleted
[root@multigui20-master ~]# kubectl get secret -n ibm-spectrum-scale-csi-driver
No resources found in ibm-spectrum-scale-csi-driver namespace.

Observe the CSi pods, the driver pod goes into Container creating state.
[root@multigui20-master ~]# kubectl get pods -n ibm-spectrum-scale-csi-driver

NAME                                                  READY   STATUS              RESTARTS   AGE
ibm-spectrum-scale-csi-attacher-5c5f879f9b-f28d4      1/1     Running             0          24h
ibm-spectrum-scale-csi-attacher-5c5f879f9b-khstc      1/1     Running             0          24h
ibm-spectrum-scale-csi-operator-647d488b4f-fvkbj      1/1     Running             0          24h
ibm-spectrum-scale-csi-provisioner-6956678877-fxr5h   1/1     Running             0          24h
ibm-spectrum-scale-csi-resizer-bfb8f6df-cjdmc         1/1     Running             0          24h
ibm-spectrum-scale-csi-snapshotter-6cf649bb46-8xqr4   1/1     Running             0          24h
ibm-spectrum-scale-csi-tbtvh                          0/3     ContainerCreating   0          26s
ibm-spectrum-scale-csi-vg7qg                          3/3     Running             0          6m24s

Again create the secret.
kubectl create secret generic guisecret --from-literal=username=csiadmin1 --from-literal=password=adminuser1 -n ibm-spectrum-scale-csi-driver

kubectl label secret guisecret product=ibm-spectrum-scale-csi -n ibm-spectrum-scale-csi-driver
5. Observe the CSI pods, the driver pods restarted and comes into running state.
[root@multigui20-master ~]#  kubectl get pods -n ibm-spectrum-scale-csi-driver
NAME                                                  READY   STATUS    RESTARTS   AGE
ibm-spectrum-scale-csi-attacher-5c5f879f9b-f28d4      1/1     Running   0          24h
ibm-spectrum-scale-csi-attacher-5c5f879f9b-khstc      1/1     Running   0          24h
ibm-spectrum-scale-csi-h4txz                          3/3     Running   0          32s
ibm-spectrum-scale-csi-operator-647d488b4f-fvkbj      1/1     Running   0          24h
ibm-spectrum-scale-csi-provisioner-6956678877-fxr5h   1/1     Running   0          24h
ibm-spectrum-scale-csi-r768q                          3/3     Running   0          4s
ibm-spectrum-scale-csi-resizer-bfb8f6df-cjdmc         1/1     Running   0          24h
ibm-spectrum-scale-csi-snapshotter-6cf649bb46-8xqr4   1/1     Running   0          24h

as monica has verified the same,closing.


Week 4 Maps pt2

Styles the map to my liking, addresses @shaquille-hall comment on last PR.

I could do more with it, but I think I'll call it good there, so that I can move on to tdd.


Feat prototype

There seems to be a problem in the app bar of learning overview where I get the following Alignment :Alignment(Infinity, 1.00)

There seems to be a problem in the app bar of learning overview where I get the following Alignment :Alignment(Infinity, 1.00)

#19

There seems to be a problem in the app bar of learning overview where I get the following Alignment :Alignment(Infinity, 1.00)

#19


Changed the behavior of 'bundle clean --dry-run' to output the list regardless of path set or force option

Changed the behavior of 'bundle clean --dry-run' to output the list of gems bundle without having the local path set or providing the '--force' option. This change does not affect the actual behavior of 'bundle clean' which requires either the path being set or use of '--force'.

Closes #5027.

@homu r+

:pushpin: Commit fca76d3 has been approved by segiddins

:hourglass: Testing commit fca76d3 with merge efc37d8...

:broken_heart: Test failed - status

@homu retry

:hourglass: Testing commit fca76d3 with merge 37ec605...

:sunny: Test successful - status


Multiple config files

THis will allow you to have global confi file for a project and specific ones for specific services

i guess i'm +0.
regarding implementation i guess it might now be worth factoring out

Eagerly waiting for this feature.

Hello,
Any update for this feature ?


help me understand and fix 'ERROR in Cannot read property 'map' of undefined' error am getting where is run/serve or build my app

hey guys please help me understand and fix 'ERROR in Cannot read property 'map' of undefined' error am getting where is run/serve or build my app (ionic4 angular project ) and doesnt say where & in which file the error is occuring.
this is my environment version after running ng versrion command

Hi @aminemshady2080, can you please the log file contents / trace please?
Also, can you kindly update to version 9 and see it the problem persists?
Thanks.

i ended up deleting the whole project and over cause i couldn't find solution due to incatibilities of angular packages once i run npm update  many packages werent compatible with new angular version now i moved back to 8.3.25 atleast things were going well for a while until now am getting
this error
ERROR in ../nodes/rxjs/src/internal/scheduled/scheduled.ts:5:60 - error TS2307: Cannot find module 'rxjs'. [ng] 5 import { ObservableInput, SchedulerLike, Observable } from 'rxjs';

Kindly see: https://github.com/angular/angular-cli/issues/17326#issuecomment-605297902
If the problem persists kindly file a new issue with a minimal reproduction.
Thanks.


Add land parameters

Adds land parameters

Adds some structural organization

Still needs doc updates / tested

This has gotten a bit stale, @gagnelandmanna let me know if we need to restart this PR but with the the new toml entries

Hi Charlie,
All good, we don't need to restart it. Hope all is well!
Anna
On Mon, 31 Oct 2022 at 15:37, Charles Kawczynski @.***>
wrote:

Closed #33 https://github.com/CliMA/CLIMAParameters.jl/pull/33.
—
Reply to this email directly, view it on GitHub
https://github.com/CliMA/CLIMAParameters.jl/pull/33#event-7708615355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOX4JOD4B7BIJJAWRXPVMKTWGBC25ANCNFSM4N3YDGSQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Incorrect blending when using a render target.

Drawing a texture with transparency will overwrite the color instead of blending it.

There doesn't seem to be any obvious mistake, this is very suspicious. Any ideas @skejeton?

The blending properties seem to be the same as in the canvas pipeline.

I think I see what you mean:

Here I added transparency to both placeholder and render target.

I think I might know why. The background in the render target is 0x00000000, e.g. completely transparent, but also black. It'd explain why it blends in this gray color.

I mean this (the text is rendered over the sign in a render target mode):

Hmm, how do you draw the entity frame?
        e.tex.img.draw({
                s: { 1, 1 }
        })

I assume this, but is e.tex.img initialized to be the frame before that?

@marekmaskarinec I found one prominent issue is that framebuffers are inherently pre-multiplied in the end, inconsistent with the straight alpha blending mode we do.

@marekmaskarinec I added pre-multiplied alpha to render targets, test it out!

@marekmaskarinec I think I figured out the problem, in the most recent commit you accidentally used pass_action instead of offscreen_pass_action.

Fix confirmed.


Feature request

Thanks for this card. It looks great.I would really like that the color on the card changes with the current theme automatically like it does for many other cards.

Good idea, I thought it did already. Perhaps I messed up something, I'll check.

My card is blue regardless of what theme I use.

Hmm, that's strange. Did a quick test changing my theme color to yellow, and it seems to work on my side.

But I only changed the "primary color", not really a theme. Are you trying with a theme?

Im using themes yes

Hmm, ok. I'll look into it.

@Hellowlol; really sorry for the delay here, but it seems my understanding of themes was not too good since I don't use it myself. Hopefully your request is covered in v0.0.9, so if you're still using the card - please let me know if it works for you. See also #5 for details.


Fix data race on publish images

If you uncomment the copyRefs code and just return the shared is.refs you will get:
"Race detected during the execution of the test" while executing the added test.
go test -race -count=5 ./pkg/environment/...
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>
Changes

copy image store map for read-only access

/kind bug

Fixes #410

Release Note

Fix data race on publish images.

/cherry-pick release-1.8

/cc @matzew @cardil

Downstream test failures are unrelated


Question about multiple SQLite databases

Hi there,
I'm working on creating an SQLite database that has CEMS data. I already have a different SQLite database with EIA 860 and 923 data. Is there anyway to direct my python scripts to a different "pudl_db". It looks like it currently just pulls the default one which is set from running the ETL process. At least that's my understanding. Thanks for the help!
All the Best,
Bri

Hi brianna! Most of our system defaults to using one sqlite db, so we haven't built in functionality to interact with multiple dbs, but depending on your use case, you can certainly use multiple db's. Are you looking to generate multiple db's or use the PudlTabl output functionality w/ multiple db's?
The latter isn't too hard. You just give pudl.output.pudltabl.PudlTabl a pudl_engine using a non-standard path.
pudl_engine = sa.create_engine(pudl_settings["pudl_db"])

PUDL is set up to operate within a particular "workspace" -- a hierarchy of directories that includes sqlite, data, datapkg, parquet, settings etc. If you want to work with a new and different set of data, you can create a whole new workspace, by running pudl_setup in a new directory somewhere else on your computer.  In practice, if I'm playing with more than one set of things (like, testing ETL outputs) I'll often just rename the sqlite/pudl.sqlite file to something like pudl-2020-03-04.sqlite so I know when it came from, and then all the same automatically generated paths still work.
Or like Christina said, depending on what you're wanting to do with the new database you can potentially just pass its new path in to the appropriate function.
How much CEMS data are you looking to work with? If it's a substantial amount, you may want to use the parquet file output instead (see the epacems_to_parquet script).  I tried to load all ~1 billion rows of EPA CEMS into SQLite once, and it took 30+ hours (at which point I killed the process).  If you're loading a modest amount of CEMS data, you'll still need to also re-load the EIA 860/923 data (which will be generated in the same datapackage output).
Generally at this point we're recommending folks use the archived data packages from Zenodo if possible -- they include the full collection of possible outputs.

For now I was just using one year of CEMS data so I will go ahead and just combine it into a single SQLite database with the EIA860 and EIA 923. I could eventually be using more which at that time I'll look into the parquet file output.
I am curious though how I would direct to a different database in my script. Currently, these are the two lines in my script hooking up the the "pudl_db":
pudl_settings = pudl.workspace.setup.get_defaults()
pudl_engine = sa.create_engine(pudl_settings["pudl_db"])

What path would need to change?
Thank you both for the help!

I am set for now and will close this. Much appreciated on the quick responses.


Issue with getURL() not a function

When I'm using the plugin in my website following error occured. could you tell me what has happened?
       Uncaught TypeError: this._baseLayer.getURL is not a function (leaflet.timedimension.min.js:17)
          at e._createLayerForTime (leaflet.timedimension.min.js:17)
          at e._getLayerForTime (leaflet.timedimension.min.js:17)
          at e._update (leaflet.timedimension.min.js:17)
          at e.onAdd (leaflet.timedimension.min.js:17)
          at e.onAdd (leaflet.timedimension.min.js:17)
          at e._layerAdd (leaflet.js:5)
          at e.whenReady (leaflet.js:5)
          at e.addLayer (leaflet.js:5)
          at e.addTo (leaflet.timedimension.min.js:17)
          at app.html:169

it was fixed after relevant js files were called


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2774 ms

Resolved: Samsung performance has improved in 6d42c13.


Add Tensor methods .ndim , .numel, .element_size, .nbytes and also__len__

Fixes #861
@nima10khodaveisi Sorry did not see your #869, but also .numel should be a @peoperty .

Tests failing. (btw, install the pre-commit hooks to run the linter on commit)

@geohot , my bad, I pushed a branch that was not rebased. Should be good now.

I've never seen anyone use is_floating_point

I'm using is_floating_point because you can't calculate mean/std on a non-float tensor.  If the criteria for adding stuff is anyone is using it, then yes, I'm using it. If you hate having it, I'll drop it.
My bad re __len__, I guess I never use len(). I'll drop it.
I'll drop .nbytes because I agree the semantics is confusing - should it return the real bytes, or just numel * sizeof. Non-obvious methods are not ideal. It's easy enough to multiply 2 numbers explicitly if one needs to.

@nima10khodaveisi , don't think there was a duplication, except for my earlier commits. Do you see them?


Feature/geode 3780

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[na] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>@WireBaron @kohlmu-pivotal @PivotalSarge @galen-pivotal @hiteshk25

Looks like your protobuf changes ended up getting caught up in this review.

I'm closing this to submit a new PR without the protobuf changes


Implement basic multi-tqubit gate

Let's start by implementing a multi qubit gate for surface code based qubits.
These initial gates will be implemented between surface codes of the same size and type.
Later, we can generalize with more advanced concepts from Ref [1].
[1] https://arxiv.org/pdf/1709.02318.pdf

@aaronhouseholder  implement a manual CX gate between two surface code qubits of fixed size by Jun 16.

@aaronhouseholder  reading https://arxiv.org/pdf/1709.02318.pdf
@Phionx  helping with implementation


System.import with a default export

When you import a module with only a default export, is it expected that you'd have to call .default to access it?
System.import('module').then(function(m) {
    m.default;
});

I ask because I noticed that when Babel compiles to AMD - if the module only has a default export, that default export gets returned directly. Only after you've added a named export to the module do you need to use .default.
I wasn't able to tell what the expected behavior is from the current spec.

When importing AMD or CommonJS, SystemJS adds a helper to ensure you don't have to do this, but strictly you should. That code is here - https://github.com/systemjs/systemjs/blob/master/lib/extension-core.js#L25.
The other behaviours are all interop solutions that allow for consistent behaviours with what users would expect.
The spec doesn't define how to load other formats, that's something that's through these interops in transpilers and loaders.

I'm mainly curious about what the behavior of the native Loader (with a native ES6 module) will be. Is it too early to say for sure?

You would need to directly access the default property.

Alright, thanks!


DatePickerModal jump wildly when landscape in IOS device

Current behaviour
The DatePickerModal component jumps wildly when the device is in landscape mode on iOS using react-native-paper-dates.
Expected behaviour
I expect the DatePickerModal component to remain stable and not jump when the device is switched to landscape mode on iOS.
How to reproduce?

Open the app on an iOS device.
Navigate to a screen where the DatePickerModal component is used.
Rotate the device to landscape mode.
Observe the DatePickerModal jumping wildly

Preview
https://github.com/web-ridge/react-native-paper-dates/assets/67258104/8da4c8ac-6fe3-45f7-90c0-5c14447f2745
What have you tried so far?

Install version 0.22.9
wrap in to View have width = deviceWidth

Your Environment

software
version

ios
17.4

android
x.x.x

react-native
0.73.1

react-native-paper
5.11.6

node
21.7.2

npm or yarn
yarn

This appears to be linked to this pull request feel free to try it out and fix the PR if it is a proper solution!

Thank you, you saved my time. I am very happy to have fixed it in the node module with the following comment. I hope this bug will be fixed in the next version.

Thank you, you saved my time. I am very happy to have fixed it in the node module with the following comment. I hope this bug will be fixed in the next version.

I'll test it and try to get it merged in the next version. Thanks!

@UIT19521334 Did the proposed fix, make the calendar look like this? It seems to fix the jump, but doesn't appear to be fully responsive, so it needs some work still apparently.


LevenbergMarquardt do not seems to work with non-flat input.

GaussNewton is working as intended with Pytree. I would expect the same for LM. Instead, I had to flatten the array to make it properly works.

The errors appear at line 445. The error comes from the fact that the pytree of params and vec do not match.

@amir-saadat I was going to report this as well. I wrote a super simple test case to demonstrate, though I'm not sure its what you're looking for.
config.update("jax_enable_x64", True)
import jax.numpy as jnp
import jaxopt

M = 5
params = jnp.zeros((2,M))
params = params.at[0].set(jnp.arange(M) * 1.0)
params = params.at[1].set(jnp.arange(M)**2 * 1.0)
params_dict = {'A': jnp.arange(M) * 1.0, 'B': jnp.arange(M)**2 * 1.0}

def F(params):
	return jnp.asarray([jnp.sum(params[0]), jnp.sum(params[0] * params[1]**2)])

def F_dict(params):
	return jnp.asarray([jnp.sum(params['A']), jnp.sum(params['A'] * params['B']**2)])

def optimize_F_gn(params, F):
	gn = jaxopt.GaussNewton(residual_fun=F)
	return gn.run(params).params

def optimize_F_lm(params, F):
	gn = jaxopt.LevenbergMarquardt(residual_fun=F)
	return gn.run(params).params

print(params)
print(optimize_F_gn(params, F))
print(optimize_F_gn(params_dict, F_dict))
print(optimize_F_lm(params, F)) # fails
print(optimize_F_lm(params_dict, F_dict)) # fails


Should this project be archived or change ownership?

I'm not sure this project is actively maintained. Personally I'm no longer using it and do not have the time to commit to maintenance. There are a few outstanding bugs that would probably involve significant rework. Additionally, 11ty v2 has better support for processing asset files so there is less need for projects like this.
@gregives perhaps its time to archive?
If anyone relies on this project, or wants to carry it forward please reach out.
Thanks,
John.

Hi @johnhunter, first and foremost I want to say thank you for the time you've spent maintaining this project, I really appreciate it!
I no longer use 11ty or have any time to devote to this library, so I will archive it. If anyone wants to take ownership of the library, please do let me know.
Thanks,
Greg


[telegram:embed] unable to extract video URL

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Provide a description that is worded well enough to be understood
yt-dlp https://t.me/leakedgameplay/376 returns a warning that the video URL could not be retrieved.
If you open the link in a browser, you can see that the page says that the video is too large and is not displayed in the browser.
You can handle this somehow, and not write that it failed and ask to report the error.

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://t.me/leakedgameplay/376']
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (pip)
[debug] Python 3.11.5 (CPython AMD64 64bit) - Windows-10-10.0.22631-SP0 (OpenSSL 3.0.9 30 May 2023)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, requests-2.31.0, sqlite3-3.42.0, urllib3-2.0.4, websockets-12.0
[debug] Proxy map: {'no': '<IP_ADDRESS>'}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[telegram:embed] Extracting URL: https://t.me/leakedgameplay/376
[telegram:embed] 376: Downloading embed frame
WARNING: [telegram:embed] unable to extract video URL; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: Extractor telegram:embed returned nothing; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

Duplicate of #3606


Ports and IP's Burried

Expected Behavior
https://developers.cloudflare.com/cloudflare-one/connections/connect-apps/install-and-setup/installation
Links the FAQ page for ports
https://developers.cloudflare.com/cloudflare-one/faq/cloudflare-tunnels-faq#what-are-the-ports-and-ips-used-by-cloudflared
Actual Behavior
This page does not, and if you're installing i feel its missing info that is crucial to success.
Section that requires update
https://developers.cloudflare.com/cloudflare-one/connections/connect-apps/install-and-setup/installation
Additional information
Any other details or screenshots you think are relevant.

We have made several updates to this page since the issue was opened. Closing, although feel free to open another issue should there be any specific requests to clarify content.


fix the bug when org_id is empty we will receive an error

change the condition that only when orgID and account
are empty we will receive an error
related to issue -THEEDGE-2298
fix the bug when trying to add device to group
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I run make pre-commit to check fmt/vet/lint/test-no-fdo

I push by mistake to this pr, I missed that it already merged


Tables don't render

Copy the table as noted in this tutorial and you will see it not render. Perhaps table rendering is not implemented yet.
https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet

Hi @javajon
Yes, it is known ^^ Please refer to https://github.com/nea/MarkdownViewerPlusPlus/issues/24 regarding the details.
I am still working con changing the backend converter to allow tables in the future :)
Cheers


AppState feature request: scoped Blaze global helpers

I would prefer to use global helpers like this:
<template name='VideoAuthor'>
  Author name is {{AppState.videoAuthor.name}}.
</template>

Otherwise I need to prefix my root path in order to avoid mix up with template helpers...

Interesting. We can add it to the initialisation options:
AppState = new MeteorFlux.AppState({ prefix: 'state' });

Would that be ok?

That looks great.

Global helpers make sense in the case of a singleton object. So, to me, global helpers should be part of AppState, not ReactiveState.

Totally true, they should be in a different package

Just spent 1 hour because of this. Suppose you use AppState.set('doc', object) in your code, then the following code has very unexpected results:
<template name=myTemplate1>
  {{>myTemplate2 doc=whatever}}
</template>

<template name=myTemplate2>

doc.name

</template>

That's not because of AppState, but because of how Meteor global Template helpers work.
If you do:
Template.registerHelper('dod', function() {
  return object;
});

it will happen the same.
If you make a PR with a namespace option for ReactiveState I will accept it.
For example:
AppState = new ReactiveState({
  namespace: 'state'
});

<template name=myTemplate2>

doc.name

state.doc

</template>


compatibility with react-native-sqlite

how hard would this be to do? react-native-sqlite

Begin the scenes it just used knex so that would be something to look into
there
On Sat, Nov 21, 2015, 3:38 PM Mark Vayngrib<EMAIL_ADDRESS>wrote:

how hard would this be to do? react-native-sqlite
https://github.com/almost/react-native-sqlite
—
Reply to this email directly or view it on GitHub
https://github.com/calvinmetcalf/SQLdown/issues/32.

how easy it is will entirely depend on how much of a spatial snowflake react-native-sqlite is vs if it reuses the api from websql or node-sqlite3

@calvinmetcalf ic, i'll give it a try one of these days


Readme update proposal

New Readme structure based on current draft proposal

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

Do not approve! More changes are coming....


[FLINK-17432][docs-training] Rename Tutorials to Training for better SEO

What is the purpose of the change
This pull request renames the new Tutorials section of the documentation.
Brief change log

Renamed the tutorials directory to training
Changed all links
Reworded a few sentences to talk about training
Added a redirect from /tutorials to /training

@nicok @sjwiesman Perhaps one of you can review this.

I was thinking of this as a training comprised of a set of tutorials and exercises, but I've gone ahead and completely eliminated the usage "tutorial".


Gymnasium v1.0.0 alphas

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Conda-forge doesn't seem to have created an auto PR for the 1.0.0 alpha 1
Installed packages
.

Environment info
.

Indeed probably this is a bug in the version autotick bot, or somehow it detects it is an alpha version.
Anyhow, note that creating pre-release on conda-forge requires a bit more modifications beside bumping the version in the recipe: see https://conda-forge.org/docs/maintainer/knowledge_base/#pre-release-builds for more details.

The bot does not open PRs for pre-release versions, you'll need to open a PR manually. If you want to also release those builds (not just run the CI through its paces), the linked docs are the right place, but it's not very complicated. We'd just need to add:
channel_targets:
  - conda-forge gymnasium_dev  # or gymnasium_alpha or ...

to recipe/conda_build_config.yaml and then rerender. I can create a branch for this, so we don't do have to do these things on main, which is supposed to have the latest GA version.

@h-vetinari With v0.29.1 merged, could we make a PR for v1.0.0a1.
I was intending this as a testing ground for alpha 1 as conda tests on windows (and mac) which we don't have on the main repo so I don't care if we don't end up merging until v1.0.0 is fully done

@h-vetinari With v0.29.1 merged, could we make a PR for v1.0.0a1.
I was intending this as a testing ground for alpha 1 as conda tests on windows (and mac) which we don't have on the main repo so I don't care if we don't end up merging until v1.0.0 is fully done

https://github.com/conda-forge/gymnasium-feedstock/pull/36


[RFC]webscraping shaders that aren't on API

This was considered in #25 however I am not sure we should include this.
It's really useful for debugging and testing, so I wanted to share this. Perhaps we can make this part of the optional dev dependencies?  Basically looking for comments on this idea.
It works by spawning a browser, waiting for the shader to load and then calling a javascript function that produces almost exactly the same json as the api returns.
Pros:

50k additional shaders accessible (public and unlisted, instead of just public+api). API gives access to slightly more than 35k
no API key required
less hassle than copy & paste

Cons:

maybe unethical, shader authors chose not to allow API access - and we just ignore that. (it's always possible to fork a shader and then download it, or make your fork available on api)
additional dependencies: selenium, some webdriver
additional setup required: webdriver available
more maintaince: different browsers, website might change...

Perhaps there is also better implementation for this

If you're able to get the info with just an HTTP request that could reduce the required dependencies.
You can also try to disable JS and see if the page is rendered differently.


Fix for Sketch 46.2

.array() is no longer needed in Sketch >= 46, causing the script to fail.

Thank you


Accessibility

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
JP
FGA build number
1951
Describe the bug
Not sure is it a bug or not, but I could not turn on the Accessibility for this app. It says something related to security reason. I don't have any idea where to do the settings, maybe other users with the same device can solve this for me?
Video
https://cdn.discordapp.com/attachments/1054022573769359381/1081936950937788476/Screenshot_20230305-215021_Settings.jpg
https://cdn.discordapp.com/attachments/1054022573769359381/1081936951223009380/Screenshot_20230305-215023_Settings.jpg
Device model
ROG phone 6D
Android version
7
Screen size
1080 x 1920
RAM
16

See https://community.gamepress.gg/t/fate-grand-automata-discussion-and-help-thread-round-2/107040/1264
If you can make a screenshot of that setting, I'll add it to the FAQ.

OK, I kind of solved the problem. I have to give permission to the app which I never had to do on other phones.
It is on the top right corner on the screenshot.
https://cdn.discordapp.com/attachments/1054022573769359381/1081941626773061783/Screenshot_20230305-220727_Permission_controller.jpg


WIP: remove peers that have disappeared from kubernetes

Fixes #2797
Using Kubernetes annotations as a way to ensure only one peer at a time runs rmpeer

I have squashed some of the commits and written an implementation of the pseudo-code previously discussed.
Next steps: try it out, figure out how to test it properly.

Also still to do, as noted earlier:
We need to distinguish between "stopped but coming back" and "stopped and never coming back".
Possibly we could extend the use of the peerList annotation to say "If I'm not currently listed as a peer then delete any old IPAM persistence data"?

We need to distinguish between "stopped but coming back" and "stopped and never coming back".

I'm not very familiar with weave, but, why? If a node disappears and we release its IPs and etc, and if it come back later, can't we just "re-setup" it (with new IPs and all)?

Weave Net has no central point of control: its native data structure is a CRDT. The "release its IPs" breaks the rules of the CRDT so we need some extra mechanism to avoid going mad.

hmm, interesting. Thanks for explaining @bboreham  :)

Replaced by #3149


feat: User friendly command line report for single scan

Pull Request
Description
Current PR contains an updated, user-friendly version of the command line report for single scans (PR or local files).
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Code cleanup/refactoring
[ ] Documentation update
[ ] This change requires a documentation update
[ ] CI system update
[X] Test Coverage update

Testing
Tested locally for PR and local file scan.

Checklist:

[X] My code follows the style guidelines of this project
[X] My code meets the required code coverage for lines (90% and above)
[X] My code meets the required code coverage for branches (80% and above)
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

Codecov Report
Attention: Patch coverage is 94.44444% with 5 lines in your changes missing coverage. Please review.

Project coverage is 93.57%. Comparing base (1b69e16) to head (c427c0c).

Files
Patch %
Lines

src/main/java/com/lpvs/entity/LPVSFile.java
87.87%
1 Missing and 3 partials :warning:

...java/com/lpvs/entity/report/LPVSReportBuilder.java
98.18%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #577      +/-   ##
============================================
+ Coverage     93.28%   93.57%   +0.28%
- Complexity      579      583       +4
============================================
  Files            49       49
  Lines          1982     2009      +27
  Branches        232      234       +2
============================================
+ Hits           1849     1880      +31
+ Misses           63       59       -4
  Partials         70       70

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


UEFI Script "sources.sh" not work

Hello,
this "Script" are don´t work...
https://github.com/MrChromebox/scripts/blob/master/sources.sh
the links in there are death
In you tutorial https://wiki.mrchromebox.tech/Unbricking are also many death links
Thanks for fix it

none of those are full links, they are base URLs onto which additional paths and/or filenames are concatenated.  The directories themselves are not browsable by design.

Okay, and how can i use this Script?

use in what way? this file is a component of the firmware utility script, you run that (per the Readme)

i found this "Script" out of your unbrick Wiki.... to find the correct UEFI rom file name to flash the Chip

exactly, the unbricking wiki tells you how to get the correct filename for your device from that file - and even gives an example. it doesn't tell you to run the script.

Okay, then I do not understand the wiki... i Know my Chromebox are an Haswell CPU...

you need to know the device name for your chromebox.  Look it up https://mrchromebox.tech/#devices.  Then use that name to find the correct firmware file.  For example, if you have an Asus CN60, then the device name is panther, the UEFI file name is coreboot_tiano-panther-mrchromebox_20190104.rom, and so the full URL would be https://www.mrchromebox.tech/files/firmware/full_rom/coreboot_tiano-panther-mrchromebox_20190104.rom.  Make sense?


Provide the Age search as Age numbers range instead of in between Birth Dates

T.13686404
In this thread https://una.io/page/view-discussion?id=670 asked a request to make search of persons with "age numbers range".
Similar to #2135 - this should be another variant of field for date for field.

This feature is already available


No Link For DownLoading The Wallet Or Server Files

Hi i like To Get The Carft Coin Wallet + Bukkit Server File For This Coin.
But All Links http://craftcoin.net are Not Working Can You Fix Them & Tell us What is New In the UpDate if any ?
Will It Work With Bukkit & mcpc-plus? 1.7.9

Hi these coins still have value of over a hundred satoshi each. Is there not a way to get a  wallet at least to try and trade them?

They shouldn't have any value, they're completely abandoned, their original intention isn't even viable anymore, the network is broken as fuck, they are literally garbage. I have no idea how or why anyone's even trying to trade them anymore.

Www.cryptsy.com look up craft coin there lol not a bad price!
On Dec 9, 2015 7:53 PM, "Marshal Walker"<EMAIL_ADDRESS>wrote:

They shouldn't have any value, they're completely abandoned, their
original intention isn't even viable anymore, the network is broken as
fuck, they are literally garbage. I have no idea how or why anyone's even
trying to trade them anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/craftcoin/craftcoin/issues/18#issuecomment-163461393.


HydroDyn

Dear guys,
I want to set 2ND-ORDER WAVES in HydroDyn, and Openfast it works until it's setting.
My 2ND-ORDER WAVES settings are as follows.

The following picture is the error generated after setting.

The error is MD_UpdateStates:NaN state detected.
Does anyone else have the same problem or know how to solve it?
Thanks!

Dear @LeoHongLiZuo,
It doesn't look like enabling second-order wave kinematics triggered an immediate error.  The error appears to happen well along (48 minutes of simulation time) into the simulation.  Can you clarify if the floating wind turbine response is reasonable up until the error?  Is the solution slowly diverging over time or does it diverge immediately at 48 minutes in?
Best regards,

Dear Jonkman,
48 minutes of simulation time  part of results are shown below.

After checking, I thought it might not be because the results diverged that the error occurred.
As illustrated in following figure, I think the reason for the error may be that the initial document(Platform) given did not contain the second-order wave kinematics.

I'm going to try adding the second-order wave kinematics and run it again and see if I get an error
Best regards!


Fix new Rust 1.67 clippy errors

Fixes new Clippy warnings that came with Rust 1.67.
Warnings:

uninlined_format_args
unchecked_duration_subtraction

Also fixes a formatting issue: comments around undocumented_unsafe_blocks: //false positive was being considered part of the same comment block with the following // SAFETY: ... comment, so the latter was indented to be on the same level. Like
let x = 0; // first value
           // will be used later

Rust-analyzer and cargo fmt also seemingly disagreed on the correct way to format time/src/formatting/mod.rs#L128.

I am making the changes myself so that I can move ahead with other work without CI failing. Closing for that reason.

Absolutely, sorry I didn't have time! I'm glad if the PR was of any help.

Not an issue! I ended up pushing changes similar to what would have been the end result of this PR.


[JOSS] fix continuum figure plot

Hi @bootphon this is PR is part of my JOSS review https://github.com/openjournals/joss-reviews/issues/2989
While reading the paper, I realized that the figures for the continuum vs best alignment are identical. I guess you just copied the wrong file twice.  Good thing that you noticed it !
Thanks a lot for the PR, merging it right away.


[Bug]: Merging the sign in and signup button as a single button

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
It is odd to have website have both buttons for sign in and sign up. furthermore, it can be on the right side to emphasize users attention towards it.
Expected behavior
The merged button will lead to login and sign up
Add ScreenShots

On which device are you experiencing this bug?
Chrome, Windows
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@VaibhavArora314 please do assign this issue to me

For keeping it on right there exists another issue #512, you can merge this in one


Allow mapping of log levels in logging configuration

I keep track of all errors from all micro-services in tools like Kibana. I had defined queries and dashboards that show me various metrics including logged error statements. Unfortunately some libraries that I use e.g. IdentityServer4 log e.g. failed logging attempts as errors. For me this makes no sense and I think that they should be logged instead as warnings. Currently I have no option to override log level in my logging configuration, I can only turn a particular logger on or off (which for security reasons I do not want).
It would be nice to have an option to redirect log levels from certain loggers to different log levels e.g. to make an error logged by IdentityServer4.Endpoints.AuthorizeEndpoint logger to appear as Info in log output.
{
  "Logging": {
    "Console": {
      "IncludeScopes": false,
      "LogLevel": {
        "Microsoft.AspNetCore.Mvc.Razor.Internal": "Warning",
        "Microsoft.AspNetCore.Mvc.Razor.Razor": "Debug",
        "Microsoft.AspNetCore.Mvc.Razor": "Error",
        "Default": "Information"
      },

	  // Proposed syntax
	  "MapLogLevels": {
		"IdentityServer4.Endpoints.AuthorizeEndpoint": {
			// All log statements with level Error will be logged with level  Debug
			"Error": "Information",
		}
	  }
    }
  }
}

UPDATE: Actually something like NLog filtering can also solve my problem: https://github.com/NLog/NLog/wiki/Filtering-log-messages.

:+1:


enhancement: Reduce image size in user popups.

This not a final decision yet, so before working on this, you'll want to get consensus.
The user popup menu features a large image of the person, which makes it kind of large for what's essentially a context menu.
We may want to use a design like this, at least at smaller resolutions, and possibly always:

@zulipbot claim

@vinitS101 Some tips for you if this design gets approved. The design was basically 2-3 design patch while I was doing experimentation. Some changes in user_info_popover_content.handlebars and some in .scss file. Also see render_user_info_popover() function . But this is just a rough estimate

Yeah, I was just adding the css to style tags in the user_info_popover_content.handlebars file for testing. Basically just centering the image, proper padding between the divs, etc. I can move them over to the scss files when making the final changes.

@zulipbot abandon

We open the user popup when clicking on a profile picture, so we definitely want to display a larger version of the profile picture in the user popover. This is also the way Slack, Mattermost and RocketChat do it. Our current user popup fits fine on mobile screens and since I don't think there is anything to be gained from reducing the image size, I'm closing this.


Tricky interactions between Suspense, Navigate, and createResource()

Describe the bug
So, I'm trying to make a landing page that will fetch some data, then automatically send the user to the correct route based on that data.
My attempt looks like this:
export default async function Landing() {
  const [firstVersion] = createResource(async () => {
    const versions = await fetch(`${RunEnv.apiUrl}v1/versions`)
      .then((response) => response.json())
      .then((versions) => versions.map((version) => version.replace(".", "_")));
    return versions[0];
  });
  return (

      <Suspense fallback="Please hold. Your call is important to us...">
        <Navigate href={`/${firstVersion()}`} />
      </Suspense>

  );
}

However, this creates a kind of race condition. Based on how I think this works, the Navigate element needs to be constructed before Solid sees the signal get fired, so that Navigate knows what to render. But, because of how Navigate works, it redirects the user as soon as it's created.
Is there a better way to handle this?
Your Example Website or App
http://donthaveone.com
Steps to Reproduce the Bug or Issue
I'm pretty sure this component will have the same affect/result. It's the same as my example above, just dropped in JsonPlaceholder for data
export default async function Landing() {
  const [firstId] = createResource(async () => {
    const versions = await fetch("https://jsonplaceholder.typicode.com/todos")
      .then((response) => response.json())
    return versions[0].userId;
  });
  return (

      <Suspense fallback="Please hold. Your call is important to us...">
        <Navigate href={`/${firstId()}`} />
      </Suspense>

  );
}

Expected behavior
As described above, I'd like to fetch data before the Navigate component kicks on
Screenshots or Videos
No response
Platform

OS: Linux
Browser: Firefox
Version:

SolidJS 1.5.1
@solidjs/router 0.5.1

Additional context
No response

Could you do something like...
export default async function Landing() {
  const navigate = useNavigate()
  await fetch(`${RunEnv.apiUrl}v1/versions`)
    .then((response) => response.json())
    .then((versions) => navigate(`/${versions[0].replace(".", "_")}`));

  return (

      Please hold. Your call is important to us...

  );
}

Unsure if you are server-side rendering or if you want to do the fetch server-side.  If so may be able to still do similar within the createResource fetcher, or create an effect that waits for the resource result to come in and then calls navigate.
Probably need to handle the error case too (like if fetch fails).

Hmm, on paper it looks like it should work. I'm unable to test immediately, but when I do, I'll come back and report how it went.

Well, the code as written results in a console warning of:
"Unrecognized value. Skipped inserting
Promise { : "pending" }
dev.js:433:17"
But, that seems like a silly fix I can deal with. Beyond that it is working as intended! Thanks a bunch!

DId you spot my chance to the code sample above yesterday?  I change the function definition to not be async and removed the await.  That should resolve your warning.


yaml Parsing - middleware does not exist - upgrade from 2.2 to 2.3

Do you want to request a feature or report a bug?

Bug

What did you do?

Hi, im upgrading from 2.2.x to 2.3.5 and my config broke. The root cause is that I'm using duplicate keys in my yml. See the config below.
What did you expect to see?
I expected:

my config works like before
error messages should describe my problem: "there was an yaml-Parsing error: duplicate key"
It would be nice if the yaml parser merges the dict. (I think I'm violating the yaml spec here)
the migrating docs should be more explanatory about the changes in the yaml-Parser

What did you see instead?

Nearly all pages showed "404 page not found".
I got the message "middleware does not exist".

Output of traefik version: (What version of Traefik are you using?)

Im using the yaml file provider on Ubuntu.
Working in 2.2, not working in 2.3
---
http:
  routers:
    errorpages_example_net-https:
      rule: "Host(`errorpages.example.net`)"

      entryPoints:
        - https
      service: errorpages_example_net
      middlewares:
        - error-middleware
        - add-catcodes
      tls:
        certResolver: letsencrypt-dns

  services:
    errorpages_example_net:
      loadBalancer:
        servers:
          - url: http://<IP_ADDRESS>/
        passHostHeader: false

http:
  middlewares:
    error-middleware:
      errors:
        status:
          - "400-599"
        service: errorpages_example_net
        query: "/catcodes/{status}.html"

    add-catcodes:
      addPrefix:
        prefix: "/catcodes"

I had to  comment out the http to make it work for 3.3. Im using the duplicate keys for example when the config is longer than one page or when I configure multiple servers/routers that belong to one application:
---
http:
  routers:
    errorpages_example_net-https:
      rule: "Host(`errorpages.example.net`)"

      entryPoints:
        - https
      service: errorpages_example_net
      middlewares:
        - error-middleware
        - add-catcodes
      tls:
        certResolver: letsencrypt-dns

  services:
    errorpages_example_net:
      loadBalancer:
        servers:
          - url: http://<IP_ADDRESS>/
        passHostHeader: false

# http:
  middlewares:
    error-middleware:
      errors:
        status:
          - "400-599"
        service: errorpages_example_net
        query: "/catcodes/{status}.html"

    add-catcodes:
      addPrefix:
        prefix: "/catcodes"

Hello @FloMiau,
Thanks for your interest in Traefik!
Since the yaml format must be JSON "valid", duplicates key are illegal.
Yet, you are right that in v2.2 duplicate keys were allowed in the dynamic configuration, thus, this was not intended. It was too permissive.
In the v2.3 version of Traefik, the parsing of the dynamic configuration has evolved and relies now on the paerser library, which not allows duplicate keys. In fact, only the last key applies.
As you already guessed, you need to remove the duplicate keys to make your configuration work again.
I'll close this issue as a question since it's an intended change.

Hey @rtribotte,
it would be nice if paerser would log duplicate keys so it is easier to find such errors.


on basics/slices  T[]

maybe is to early for templates :) If slice and dynamic arrays are the same. what I should use?
(reported by @claudiug)

This issue was moved to dlang-tour/english#39


Fixed JSON issue with Multiline API Description

Describe the PR
Fixed an issue where the JSON breaks when programmatically setting a Multiline API Description.
Relation issue
#480

Codecov Report

Merging #481 into master will decrease coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #481      +/-   ##
==========================================
- Coverage   86.58%   86.45%   -0.13%
==========================================
  Files           7        7
  Lines        1558     1558
==========================================
- Hits         1349     1347       -2
- Misses        120      121       +1
- Partials       89       90       +1

Impacted Files
Coverage Δ

gen/gen.go
89.65% <ø> (ø)
:arrow_up:

parser.go
80.85% <0%> (-0.22%)
:arrow_down:

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 c19280e...1c38154. Read the comment docs.


Version bump.

Ready for bugfix release.

The variant annotation search fix would be good to include as well. https://github.com/ga4gh/server/pull/984

This is a critical bugfix release (the package is currently broken). We can look at releasing the VA code over the next couple of days as 0.2.2.


Add SPDX Matching Guidelines

Edit per the ClearlyDefined community call today
Cc: @royaljust , @capfei , @nellshamrell

@ariel11 Thank you! 🎉

Thank you, @ariel11! Merged!


Smooth data errors

Pre-edge fitting of the data fails with the error: Warning: Cannot smooth data: must be strictly increasing. Have attempted to tweak E0 and the pre-edge range, but this was not successful (may be with further effort, e.g. extracting exact params used from Athena). Possibly due to a rise in the pre-edge region for the data? Is there any handling we can do to mitigate this (or make the error more informative)? Note that Abraham's ipynb analysis did not start from the plaintext, but the prj file.

Error is caused by the tool auto-selecting the first energy column, which is actually static at 6800 (offset?) for I20 data. Assuming correct column is Ef instead as this increases monotonically and appears to match the scan energy range (based on header info). Since this appears as the second column, it will need to be manually selected.


docs: Clarity about screenshot

Type of feature
🍕 Feature
Current behavior
Contributors have been skipping the screenshot section when filling out their PR form. It’s mostly likely due to assuming that’s it’s not required inspite of the course mentioning that it’s needed.
Suggested solution

Go the .github folder
Go to the Pull Request file
In the Screenshot section, write the following:
“ Please upload the screenshot of your profile being generated in the Markdown Live Preview”

Additional context
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

@CBID2 did you mean for this to be in our .github repo or in the guestbook repo?

@CBID2 did you mean for this to be in our .github repo or in the guestbook repo?

The guestbook one @BekahHW


Make possible to set up intent for Lua script

Hi!
First of all, it's just for the record for people who met with
ERR Error running script: -READONLY You can't write against a read only replica.

as a result of some Lua script execution. That happens because you, probably, set up readFrom settings for your Redis connection as non UPSTREAM or null value. That leads to making "intent" for your operation as "read" (see these lines of code).  So, regarding https://redis.io/commands/readonly
When the connection is in readonly mode, the cluster will send a redirection to the client only if the operation involves keys not served by the replica's master node

and regarding the fact, our beloved lettuce library is smart enough and can place your request to the right node according to the keys you provided, you end up with a read-only connection that is being used for a write operation.
Please note, in this case, there will be no MOVED results, because you're already talking to the right node and it tries to serve your request.
Questions to the lettuce developers
Ok, now when we all know an existed issue please, let me ask few questions and raise some concerns:

First of all, seems like your change with renaming MASTER to UPSTREAM broke the default behaviour for MASTER. The same lines of code show us that even if we set up readFrom = MASTER all your Lua scripts will have read intention. That, probably, might lead to having requests on a replica node, instead of an upstream one (but correct me if I'm wrong)
It would be super nice if a user could provide intention for eval or evalsha calls. For example, in my application, we have 2 different Lua scripts: read-only and with write operations. Right now I don't see any solutions other than keeping 2 different StatefulRedisClusterConnection: with readFrom = UPSTREAM and readFrom = REPLICA_PREFERRED (or something similar), which is, IMHO, a complete resource-wasting, but without it, I have a risk either to be deprived of  REPLICA_PREFERRED benefits for the read queries or having issues as I described above.

Would be glad to discuss this case and please, feel free to correct me if I'm mistaken at any point.
Sorry for a long read, thank you very much for your time.
I bet someone's brought up this issue already, but I hadn't been able to find one, feel free to redirect me.

Thanks for reaching out. We should fix the issue with the UPSTREAM/MASTER constants. The intent is actually used to determine the right type of connection. You can route commands yourself by obtaining a node-connection through connection.getConnection(Intent.WRITE, slot).async().eval(…). Only the slot remains to calculate, you can use the SlotHash utility for that.
Commands are defined on group interfaces (RedisScriptingCommands) that aren't tied to the Redis operation mode at all and introducing that concept would break the existing design.

oh, wow, thanks @mp911de this approach would really help!
but at the moment I kinda struggle to understand how can I access this getConnection method. We're talking about the method
io.lettuce.core.cluster.PooledClusterConnectionProvider#getConnection(io.lettuce.core.cluster.ClusterConnectionProvider.Intent, int)

right? Could you help with the exact lines?
RedisClusterClient client = RedisClusterClient.create(Collections.singleton(RedisURI.create("<IP_ADDRESS>", port)));
StatefulRedisClusterConnection<String, String> connection = client.connect();
// ???
// I found this way, but it won't compile
// ((StatefulRedisClusterConnectionImpl) externalRedisConnection).getClusterDistributionChannelWriter().getClusterConnectionProvider().getConnection(ClusterConnectionProvider.Intent.WRITE, 42);

But even if I could access this method, the io.lettuce.core.cluster.ClusterConnectionProvider.Intent enum is package private. So users outside the library can't "officially" use it.
Sorry, for maybe a dumb question, still getting familiar with the library.
thank you for the help once again

Sorry, my bad. I wanted to refer to StatefulRedisClusterConnection that exposes a getConnection() method returning a master connection.

ah, ok, so, imagine we have a code like
RedisFuture<?> redisEval =
        connection()
            .async()
            .eval(
                script,
                ScriptOutputType.MULTI,
                keys,
                values);

you're saying it could be rewritten with
StatefulRedisConnection<String, String> nodeConnection =
    connection()
        .getConnection(
            connection()
                .getPartitions()
                .getPartitionBySlot(SlotHash.getSlot(keys[0]))
                .getNodeId());
RedisFuture<?> redisEval = nodeConnection.async()
    .eval(
        script,
        ScriptOutputType.MULTI,
        keys,
        values);

am I right?

Yeah, the code from above is exactly what you need.

I see you have changed the src/main/java/io/lettuce/core/masterreplica/ReadOnlyCommands.Com mandName read-only command definition, want to ask, this change can take effect in which version?


ENH: Support partial pathlength factors for each wavelength in Beer-Lambert law

This pull request modifies the beer_lambert_law function to support partial pathlength factors per wavelength, as requested here by myself. The ppf's are now specified in a tuple and converted to a 2x1 array which than can be used in the equation.

Thank you for the input @larsoner

Thanks @scholzri !


Removing Tutorials from the Nav bar

Hey, I am using the Tutorials feature and I like it but I do not want the tutorials to appear on the nav bar, is there an option I can pass in to get this behavior ?
btw I looked at http://usejsdoc.org/about-tutorials.html and there is no info on removing them from the nav bar.

You'll need to modify the JSDoc template to remove the tutorials from the navbar. There's no option to control this behavior.
(In the future, please post "how do I do this" questions to the jsdoc-users mailing list. Thanks!)

awesome, thank you.


CHECK-808 Add Fact Check Hindi

@caiosba and @sonoransun this case is still failing for the ElasticSearch problem I raised in other tickets - the index is for some reason not initialized - would love a pair of eyes here. Sometimes, the tests, fail, other times they hang indefinitely where they used to take about 10 minutes max. For one, it's trying to say that the table claim_reviews doesn't exist, but for these tests it's supposed to be using claim_reviews_test and never attempt anything on an index named claim_reviews in the first place - could this be some misconfig? I don't know how else to explain it - particularly since it doesn't seem to be prompted by any of the underlying code...


⚠ PrivateHD has degraded performance

In 2b0dedc, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2575 ms

Resolved: PrivateHD performance has improved in 8a7541f.


existing evidence item that uses PMID warning stays even after changing PMID

(taken from live site)
If you try to add an evidence item using a PMID that has been used for a previous evidence item for the same variant, we display a warning saying that someone has already referenced this study. But if you stay on this add evidence page and try to change the PMID, the warning does not disappear, even for a new PMID. Changing the gene or variant also does not make this go away. This bug could get you if you were on a curation spree and saw this warning, then decided to just move on to adding the next evidence item on your list with the same view, only to see this warning and be confused (or worse, potentially skip over the rest of your evidence items).

Latest commits to PR #713 should fix this. Just refreshes vm.duplicates every time the search is run - before, no response would be provided to the callback, so vm.duplicates would remain the same as before.

Merged!


Blades in the Dark v1.6

I hope I did the translations correctly. Please check that I did.

This didn't give me any errors, so it appears you did everything right!


no menu visible in mobile view

if the width of the browser window is lower than 1000 the menu disappears. at some point before there were the three lines stacked on each other but no more. (tested on mobile, chrome, firefox and safari)

fixed in deployment settings