TimePicker error

Description
I use TimePicker and optimize js file with webpack and UglifyJsPlugin. After this actions I always see TimePicker with 24 hours format. Even if i set 12 hourse format. Before minify all work right. Help pleace!
Versions

Material-UI: 0.16.2
React: 15.3.*
Browser: chrome, firefox

@Razzwan: can you please provide a repository reproducing this issue, please?

I realized my mistake. I was guided by the class name with mappings for properties, and the class name when minification changes to the shortest. Therefore, the property is simply not applied.

Alright, thanks for answering back, @Razzwan!


Should the seafloor be a planetmap? (discussion thread)

I've heard of issues with planetmaps, but it COULD be interesting.
For one, the diving bell could actually dock on top of tiles and stop the flooding caused by opening the diving bell hatch
also it will probably make procedural wrecks/areas easier to implement
And i believe i can still make it so sweetwater and the currently mapped surrounding area exist, it'll just procedurally generate everything past the current map. Maybe just a flatter expanse with some icy rocks and stuff spaced out. Like the crater's edge of subnautica, sweetwater is built near some sort of hydrothermal/biological hotspot that allows much more life to thrive
but i've heard that planetmaps are painful sometimes, i'm not entirely sure why but many people say that

I have decided that this is not required due to another way to complete #80. Closing this.

That leaves the potential of submarines in the future, albeit the chances are low


Warnings about unknown URLs still produced when using allow_unknown_urls

I have this configuration:
Capybara::Webkit.configure do |config|
  config.allow_unknown_urls
  config.ignore_ssl_errors
end

But I still get a warning when running my tests:
Request to unknown URL: https://sub.localdomain.lan:449/api//pastry_kind/cake_color
To block requests to unknown URLs:
  Capybara::Webkit.configure do |config|
    config.block_unknown_urls
  end
To allow just this URL:
  Capybara::Webkit.configure do |config|
    config.allow_url("https://sub.localdomain.lan:449/api//pastry_kind/cake_color")
  end
To allow requests to URLs from this host:
  Capybara::Webkit.configure do |config|
    config.allow_url("sub.localdomain.lan")
  end

Looking at the implementation of allow_unknown_urls, it seems pretty straight-forward, so I must be doing something silly somewhere. Is there any reason that the global allow wouldn't work?
Other URLs that I access (there are multiple), all seem to work fine.
Versions:
Capybara-Webkit 1.7.0
Capybara 2.4.4
Ruby 2.2.3p173
Qt 4.8.6

Have you registered your own driver? It's possible that you're not providing the configuration object to the driver, which would result in the options being ignored. The driver must be registered with Capybara::Configuration.to_hash as the second argument. It's likely that you can eliminate any custom driver you were using as all the options were rolled in to the configuration object. See #806, where we intend to issue a warning for the issue.

Have you registered your own driver?

Nope; I'm using whatever happens by requiring the gem.

It's possible that you're not providing the configuration object to the driver, which would result in the options being ignored.

Unlikely, as I can set the debug option here and it definitely takes effect. I also see certain blocked URL warnings disappear when I use the confit in my original comment. That's why it's such a strange occurrence.

I am also experiencing this.

Interesting. Could you extract a failing example that reproduces the issue? A complete example, including spec_helper.rb would be much appreciated. I haven't been able to reproduce this myself, but it's possible there's something I've missed.

I'm still working on narrowing it down, but this does not reproduce when I run locally on OS X. However, when I run in an Ubuntu Docker image, it does reproduce. I'm seeing about making a smaller example.

I've tracked it down a bit. Putting debug prints in UnknownUrlHandler::reset, I see that it's being called.
I only see this being called from WebPageManager::reset. I do note that if I throw a binding.pry in my step I don't see this occurring.
My current hypothesis is this:

Access a page that performs AJAX calls
Do whatever capybara check we want.
Start to shut down the test, which presumably calls the reset command, which ultimately resets the allowed URLs.
The page still has active JS which triggers an AJAX request at this point.

Any thoughts?

@shepmaster that's possible. Maybe we need to reorder our setup function to delete pending requests before changing other settings?

Just confirming that I'm seeing the same issue; I have explicitly allowed a URL which is requested via JS but I frequently receive a warning about blocking it. When I allow all unknown URLs the behaviour is unchanged, I still see a warning.

I use this workaround:
# spec/spec_helper.rb
Before('@javascript,@webkit') do
  page.driver.allow_url('*')
end

This gives me only one deprecation warning instead of many unknown url messages.

I'm experiencing the same issue, seems it's not just showing these warnings, but also resets all custom webkit driver settings for requests on point 4. It caused hangs for my case, because my webkit settings were reseted, so maybe related to #623. Also #816 looks like the same issue.
As a temporary workaround I disabled the reset method on capybara, similar to this  http://stackoverflow.com/a/30809650/600638

Same issue here.  Capybara 2.4.4, Capybara Webkit 1.7.1.

Same issue here. Capybara 2.4.4, Capybara Webkit 1.7.1.

Any progress on this?

I haven't been able to work on this yet.

@jferris any ETA?

@lucasklaassen I can't provide an ETA, since I work on capybara-webkit during spare time unless I'm encountering a bug I have to fix for my own work. I think this is likely the highest priority issue, so I'll work on this next time I have available for capybara-webkit.
If somebody look at some of the earlier suggestions and put together a pull request, that would take me less time than writing it myself, so it could potentially get merged sooner.

@jferris just pinging you about this again for any updates? Your work on this project is much appreciated :)

I submitted #895 which I believe will resolve the race condition.

@shepmaster @lucasklaassen could one of you test out that branch to see if it resolves your issue? I still haven't been able to reproduce this locally.

@jferris Sorry to disappoint, but I've moved on from the job where I had those tests, so I doubt I'll be of much help in reproducing. The diff appears sane, for what that's worth. :smile_cat:

@jferris I unpacked the gem in a 'workspace' folder in my project and referenced the path it in my gemfile. I made the change from the PR you put up, ran bundle install and then ran bundle exec rspec. The alert messages for "Request to unknown URL" still appear even though I have the following code in my spec_helper.rb
Capybara::Webkit.configure do |config|
  config.allow_unknown_urls
end

@lucasklaassen could you include the relevant parts of your configuration? Things like spec_helper.rb, rails_helper.rb, and things from spec/support may be useful. Also, a copy of your debug log (config.debug = true) may be useful.

spec_helper.rb:
require 'capybara/rspec'
require 'pry'
require 'selenium-webdriver'
require 'byebug'
require 'capybara/webkit'
require 'yaml'
require 'helpers/session_helpers'
require 'helpers/web_helpers'
require 'helpers/wait_for_ajax'
require 'faker'

SPEC_ROOT=File.expand_path(File.dirname(__FILE__) + '/../')

Capybara.default_driver = :webkit
Capybara.javascript_driver = :webkit
options = {js_errors: false, debug: false, skip_image_loading: true, ignore_ssl_errors: true, allow_unknown_urls: true}
Capybara.register_driver :webkit do |app|
  Capybara::Webkit::Driver.new(app, options)
end

Capybara::Webkit.configure do |config|
  config.allow_unknown_urls
end

Capybara.default_max_wait_time = 10

RSpec.configure do |config|

  config.after(:all) do
    # Destroy fixtures
  end
  config.around(:each) do |example|
    begin
      example.run
    rescue Exception => ex
      save_and_open_page
      raise ex
    end
  end

  config.include WaitForAjax, type: :feature
end

I am not using rails for this project so I don't have a rails_helper and I do not have a spec/support folder.

If you register your own Capybara::Webkit::Driver, it won't pick up options from Capybara::Webkit.configure. You can get the configuration Hash from Capybara::Webkit::Configuration.to_hash, which you can merge with other options if you'd like.
Note that this option is being deprecated shortly. You should be able to configure any options using configure block without registering your own driver.
I'm going to close this, as it seems like the race condition leading to this bug has been fixed. Please continue to comment or ask questions if you have any.


Delete VPC if 'x' port is open

Hi,
After going through the Cloud Custodian Docs I can't seem to find a way to delete VPC's with a certain port is open, eg: search all vpc's if any have port 80 open then delete the VPC.
I believe it should be something like:
policies:
  - name: delete-vpc-with-port-80-open
    resource: vpc
    filters:
      - type: ingress
        OnlyPorts: [80]
    actions:
      - delete

But this doesn't work...
Any Ideas?

that doesn't quite make sense, vpcs are networks, not network controls, you use security groups and nacls to enforce network segmentation controls.. ie. do you really you want to delete a vpc if any resource in that vpc has a security group that allows port 80? deleting a vpc means deleting all resources attached to that vpc first.

So it would be better to delete security groups with port 80 open?
And when I finished the rule I would make it work with cloud trail events to only trigger when a new VPC is made.

there's been a few requests on gitter for delete vpc/delete subnet type of functionality revolving around sandbox/test environments and automating cleanup.

in this context we'll also need to delete all the resources that may potentially be using them, which can be quite time consuming (rds/elasticache, etc).


Update Readme and "Welcome" notebook

Codecov Report

Merging #4837 (8e6609d) into main (68b67c2) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #4837   +/-   ##
=====================================
  Coverage     72%     72%
=====================================
  Files        410     410
  Lines      26940   26940
  Branches    3902    3902
=====================================
+ Hits       19540   19544    +4
- Misses      5877    5878    +1
+ Partials    1523    1518    -5

Impacted Files
Coverage Δ

...t/common/application/webviewPanels/webviewPanel.ts
72% <0%> (-8%)
:arrow_down:

src/client/common/process/pythonDaemonPool.ts
78% <0%> (-5%)
:arrow_down:

src/client/common/process/baseDaemon.ts
54% <0%> (-3%)
:arrow_down:

src/client/datascience/jupyter/kernelVariables.ts
74% <0%> (+<1%)
:arrow_up:

...lient/datascience/kernel-launcher/kernelProcess.ts
80% <0%> (+1%)
:arrow_up:

src/client/datascience/jupyter/jupyterNotebook.ts
78% <0%> (+1%)
:arrow_up:


When the input stops, the line is wrapped

Vscode automatically wraps lines after each input. It feels like as soon as the input stops or is slow, it will automatically wrap, like debounce.

Please use the Start Extension Bisect command to investigate whether the problem is being caused by one of your extensions.
If this doesn't identify the cause please provide more details, and perhaps a screen recording.
/needsMoreInfo
/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

https://github.com/microsoft/vscode/assets/118976717/8df8ef83-eedd-4d8a-837c-dfefa2c49059

Excuse me, this is not a bug. The editorconfig file defines insert_final_newline = true

Hey @alexdima, this issue might need further attention.
@d3wangyi, you can help us out by closing this issue if the problem no longer exists, or adding more information.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


runtime: unexpected return pc for runtime.

When installing minio which, I guess, is dependent on Go environment, with the command
$minio.exe server /data
it throws out the exception.
By the way, I tried to uninstall minio, and type
$minio.exe uninstall
it gives me the the exception too.
The exception is as following
Exception 0xc0000005 0x8 0x0 0x0
PC=0x0
runtime: unexpected return pc for runtime.asmstdcall called from 0x11
stack: frame={sp:0x379fd20, fp:0x379fd30} stack=[0x0,0x379ff30)
000000000379fc20:<PHONE_NUMBER>1effff<PHONE_NUMBER>000400
000000000379fc30:<PHONE_NUMBER>1d0200<PHONE_NUMBER>000000
000000000379fc40:<PHONE_NUMBER>1effff<PHONE_NUMBER>000400
000000000379fc50:<PHONE_NUMBER>000000<PHONE_NUMBER>0001e0
000000000379fc60:<PHONE_NUMBER>000400<PHONE_NUMBER>000400
000000000379fc70:<PHONE_NUMBER>1d0200<PHONE_NUMBER>1effff
000000000379fc80:<PHONE_NUMBER>79fcc0<PHONE_NUMBER>418f3b <runtime.(*mcentral).grow+347>
000000000379fc90:<PHONE_NUMBER>1d0100<PHONE_NUMBER>000000
000000000379fca0:<PHONE_NUMBER>1effff<PHONE_NUMBER>7a9090
000000000379fcb0:  01e0010000000400<PHONE_NUMBER>000000
000000000379fcc0:<PHONE_NUMBER>79fd20<PHONE_NUMBER>030000
000000000379fcd0:<PHONE_NUMBER>7a9090<PHONE_NUMBER>000000
000000000379fce0:<PHONE_NUMBER>000008<PHONE_NUMBER>1effff
000000000379fcf0:<PHONE_NUMBER>7a9008<PHONE_NUMBER>000005
000000000379fd00:<PHONE_NUMBER>030000<PHONE_NUMBER>79fd68
000000000379fd10:<PHONE_NUMBER>7a9090<PHONE_NUMBER>46383e <runtime.asmstdcall+94>
000000000379fd20: <000000000379fd40 !0000000000000011
000000000379fd30: >0000000002030000<PHONE_NUMBER>030000
000000000379fd40:<PHONE_NUMBER>000020<PHONE_NUMBER>1015a0
000000000379fd50:<PHONE_NUMBER>7a9090<PHONE_NUMBER>000000
000000000379fd60:<PHONE_NUMBER>56de40<PHONE_NUMBER>79fd88
000000000379fd70:<PHONE_NUMBER>7a9090<PHONE_NUMBER>79fe18
000000000379fd80:<PHONE_NUMBER>40ce30 <runtime.mallocgc+832>  000000c000002000
000000000379fd90:<PHONE_NUMBER>0001e0<PHONE_NUMBER>0001e0
000000000379fda0:<PHONE_NUMBER>56e0c8<PHONE_NUMBER>4616d7 <runtime.asmcgocall+183>
000000000379fdb0:  0001fffffffffff8<PHONE_NUMBER>0001d8
000000000379fdc0:<PHONE_NUMBER>0001e0  000000c000002000
000000000379fdd0:<PHONE_NUMBER>7a0008<PHONE_NUMBER>79fdf8
000000000379fde0:<PHONE_NUMBER>000000<PHONE_NUMBER>7a0008
000000000379fdf0:<PHONE_NUMBER>000000<PHONE_NUMBER>430993 <runtime.stdcall+147>
000000000379fe00:<PHONE_NUMBER>4637e0 <runtime.asmstdcall+0><PHONE_NUMBER>56e0c8
000000000379fe10:  000000c000000000<PHONE_NUMBER>00000f
000000000379fe20:<PHONE_NUMBER>56de40<PHONE_NUMBER>79fe48
runtime.asmstdcall(0x2030000, 0x2030000, 0x20, 0x21015a0, 0x37a9090, 0x0, 0x356de40, 0x379fd88, 0x37a9090, 0x379fe18, ...)
runtime/sys_windows_amd64.s:60 +0x5e fp=0x379fd30 sp=0x379fd20 pc=0x46383e                                      rax     0x0
rbx     0x356e0c8
rcx     0x358ff80
rdi     0x32a000
rsi     0x379fe60
rbp     0x379fe28
rsp     0x379fd18
r8      0x4
r9      0x379fea0
r10     0x8
r11     0x1d010e
r12     0x21015a0
r13     0x0
r14     0x0
r15     0x2030000
rip     0x0
rflags  0x10293
cs      0x33
fs      0x53
gs      0x2b
What's wrong with this? And if you guys know how to install minio from source (GO111MODULE=on go get ........) without internet connection instead, please help. Thank you so much.
What version of Go are you using (go version)?
go1.14.6

$ go version
go version go1.14.6 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Window version: Windows Server 2016 Datacenter
Cpu: Intel(R) Xeon(R) Bronze 3106CPU @ 1.70GHz 1.70GHz
RAM: 128 GB
architecture: 64-bit
go env Output
$ go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\ecust\AppData\Local\go-build
set GOENV=C:\Users\ecust\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\ecust\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=NUL
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\ecust\AppData\Local\Temp\go-build525125030=/tmp/go-build -gno-record-gcc-switches

What did you do?
turn off  anti-virus software

What did you expect to see?
Install minio successfully.
What did you see instead?
unexpected runtime return.

Is there a way that we can reproduce the problem?
CC @aclements @mknyszek

Is there a way that we can reproduce the problem?
CC @aclements @mknyszek

After installation on different desktops, with/without internet connection, I guess the problem neither comes from go nor minio, it might be caused by operation system (some files are destroed). Now I try to repair the system. If you guys know how to repair without DVD, please help. The problem is killing me. Thank you.

Not sure if this will help but we saw the same error message on ppc64le back in go1.14 described here. https://github.com/golang/go/issues/37216.

I have tested on an 128g ram server and I successfully reproduced the problem. I tried to compile a 32bit version and it succeed. I think the reason for this problem may related to wrong CPU settings in bios like Virtualization settings, etc. I hope this can help you!

Possible dup of #21060 & #40469
@gopherbot add OS-Windows

I have tested on an 128g ram server and I successfully reproduced the problem. I tried to compile a 32bit version and it succeed. I think the reason for this problem may related to wrong CPU settings in bios like Virtualization settings, etc. I hope this can help you!

Virtualization settings, what was that?  I don't know how to compile a 32bit version of minio. Would you please provide a download link for minio(32bit), I see they only give a 64bit version for windows. Many thanks.

I think the minio project closed the issue you opened there too quickly. They should help find a solution, e.g. a 32-bit build.
https://github.com/minio/minio/issues/10109

@networkimprov @chalkwu
What do you mean "32 bit version"? The crash is a 64 bit binary.
Is the problem the combination of Windows 2016 with 128GB RAM?
As a sidenote MinIO will not have a 32 bit version, but that seems beside the point here.

@mt-mithril reported above (and at minio) that a 32-bit build did not crash on this configuration, so minio could provide that (or instructions on how to build it) as a workaround.
It's not clear what the problem is; possibly something to do with OS or BIOS config and the Go runtime.


How to revoke a valid token?

I want to revoke a valid token when user is logout or send a request to revoke an old valid token? Kindly to help me how to do that?

You can't - you must maintain your own blacklist.


fix the logger if there are only energies

If the training is on only energies the logger was broken because it assumes there are also the forces, at least it broke the metatrain training

I do not know how to add test in this repo


Update functions-bindings-service-bus.md

Correct the URL for 'host.json'

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

@yogister : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

#sign-off


Schema: fssbodysignals/1

We've got some work on this already in develop, and now beta, see #185 #186
Aymeric45 has already identified that we have SystemName cited as a required field in the schema, and yet this isn't in the original Journal events.  It's not at this time mentioned as a mandatory augmentation in the README.
However, given we have SystemAddress in the source Journal events I feel that should be sufficient, and would rather just remove SystemName from this schema.
We've seen recent issues with, assumed to be related to the game intermittently not writing journal events to the file, with an EDMC augmentation adding the wrong StarPos to a message because there was no Location / CarrierJump / FSDJump event to tell EDMC the system had changed.  Thus going forwards I'd rather not have augmentations in new schemas unless they're 100% necessary for functionality.

If we're going to continue having augmentations, and certainly adding them to this, and future, new schemas, there is merit to Syleo's idea to use a different keyname, i.e. aug_StarPos, not StarPos, so that it's crystal clear (without constantly checking the documentation) which data is possibly not to be trusted.

Why not put these fields under a single key ?
```json
{
  "timestamp": "2022-05-24T13:47:35Z",
  "event": "FSSBodySignals",
  "BodyName": "Eock Prau YY-S d3-1 C 1",
  "BodyID": 20,
  "SystemAddress":<PHONE_NUMBER>1,
  "Signals": [
    {
      "Type": "$SAA_SignalType_Geological;",
      "Type_Localised": "Geological",
      "Count": 2
    }
  ],
  "augmentations": {
    "SystemName": "Eock Prau YY-S d3-1",
    "StarPos": [26250.0, -1275.15625, 19813.40625],
    "horizons": true,
    "odyssey": true
  }
}

It would probably simplify the senders code by having to do only one mutation on the entry.
In a near future, we could imagine a shared shape of this augmentations object across several shemas.

Changing what we have done on other schemas is just plain work. We can't force client updates, and we just make the EDDN receivers handle two types of the same message.

I've decided, rather than change even how a new schema is going to structure and name things, that we'll just document the need for location cross-checks (and any other such I come across whilst reviewing the current schemas and EDMC code).
There's some work on this already merged into the develop branch.

This has been live for a while now.


Fix lock/unlocked account status

Accounts with a single signer that have set its own weight to 0 are marked as unlocked if the thresholds are left at 0 (the default). Shouldn't such accounts be marked as Locked instead?

Fixed in v0.8.32
Thanks @gpip!

Awesome! Thank you


Update to Angular 2 RC7

Upgraded just the dependencies to Angular 2 RC7.
Tested in a project using Angular 2 RC7 with:
npm install --save "jussikinnula/angular2-google-maps-build#0.0.3"
Note! At the moment it seems that the master (without Angular 2 RC7 package updates) does not test properly (e.g. gulp build works, but gulp test gives Promise-related fatal errors). Fixing the testing related errors should be done prior getting this pull request in (testing should work always in master with gulp test).

Cancelling PR (there's already Angular 2.0.0 final available).

@jussikinnula thanks for your work, I will release a version with 2.0.0 support today.


Add REQUEST_FILENAME to 942120 to catch path SQLi

PR adds REQUEST_FILENAME to rule 942120 at PL 2: SQL Injection Attack: SQL Operator Detected. A new test is also added.
This change allows us to detect path-based SQLi attempts like
localhost/rest/products/2499999||this.product/reviews
by matching against SQL operators in paths.
This PR should fully resolve bug bounty issue ULYKOFYK.
Associated issue: #2977

@fzipi Can you help out here? I don't understand why the linter is unhappy. I copy-pasted the "template/perfect" test from the documentation, but the linter is unhappy still?

@RedXanadu You need to quote when using */*. Accept: "*/*" should do the trick.

Thanks. Is that new? I've definitely used the templates from the documentation before, and they worked okay. Do we need to amend them?


[Feature Request] Random Name Generator

Hello, what would you think about adding a fantasy name generator to the character creator?
Often people don't know how they should name a character, a quick generator could help them.
There are opensource generator, for example https://github.com/felladrin/fantasy-name-generator

A name generator is an interesting idea. But I think a way to configure this would be very helpful to tweek it in your style of names.
I will integrate a simple version in the next release.

Implemented with 5685305a4eef73b83ca01602e083ae18cfd6d786


Exclude tweets from Twitter searches by username without login

What website(s) would this filter apply to?
twitter.com
What content should be blocked?
There are no ways to block or mute users without twitter login. However when you search Twitter, many spam bots prevent you from finding valuable information. I regularly search Twitter for certain words  and currently use about 10 filters that excludes spam accounts for best results.
Search operator -from:$USERNAME also works for the same purpose, but uBO filters has the advantage that once you add filters, exclusions work permanently for all searches.
What options should be offered?
The users must enter a list of usernames to block.
There are several web services that offer Twitter search, for example, nitter.net. I don't think it's necessary, but you can consider the option of enabling cosmetic filters for them.
Did you write a custom uBlock filter already?
Yes.
twitter.com##a[href="/$USERNAME"]:upward(article)

Additional notes
No response

Hello @hirorpt, thanks for your contribution! I'll add filter for this.
While we are at it, do you feel like a filter on hashtags would be useful?

While we are at it, do you feel like a filter on hashtags would be useful?

I am sorry for late reply, and very impressed with the prompt response.
For some reason I forgot about it when submitting this issue, but I'm also using that hashtag filters.


Scrollbar doesn't appear on main tab of connection settings

Description
Main tab of the connection settings can take a lot of space in some cases (e.g. kerberos extra settings) and lower part of the wizard becomes inacessible on smaller displays.
DBeaver Version
23.1.0
Operating System
Any
Database and driver
Any
Steps to reproduce

Open connection settings
Select Kerberos authorization
Expand extra settings
Resize the window so everything won't fit there(or do the same on a smaller display)

Expected: Scrollbar appears to access lower part of the page
Actual: Lower part becomes inacessible
Additional context
https://github.com/dbeaver/dbeaver/assets/82543000/20fa567d-0523-404b-a4d4-64bcd3d0cd25

We want to add a scroll bar to all dialog pages.


Number of infected fish vs cumulated infection rate

The infection code only computes the number of infection events, not the number of infected fish. The latter isn't simply a cumulated sum as more lice can attach to the same fish.
The reason why we care about this is in the fish mortality computation. In get_fish_growth the fish mortality due to sea lice is computed as follows:
adlicepg = 1/self.fish_growth_rate(days)
prob_lice_death = 1/(1+math.exp(-self.cfg.fish_mortality_k*(adlicepg-self.cfg.fish_mortality_center)))
elf_death = self.num_infected_fish*step_size*prob_lice_death

Right now I can think of two possibilities ahead of us:

find an expectation on self.num_infected_fish (as suggested by @magicicada , adopt a hypergeometric distribution and find its mean)
directly compute the proportion of dead fish from the average lice-per-fish rate (not sure how this should be modelled)

Luckily someone seemed to solve this issue for us: https://stats.stackexchange.com/a/296053
If we wish we can easily use the expected number for the infections, which is exponentially proportional to the number of trials but clearly bounded to the fish population. I'm quite happy with that, what do you think?


feature request: php8 support

i cant build docker image because on host i need to run "composer install"


Sync eng/common directory with azure-sdk-tools for PR 1081

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1081
See eng/common workflow

/check-enforcer override


another time try to fix linux and osx

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Power cycling for missing CI (macOS and Linux).

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I rerendered the feedstock and it seems to be already up-to-date.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Would try the things in the "Otherwise (e.g. in a PR to staged-recipes)..." below.
ref: https://conda-forge.org/docs/ciservices.html#enabling-circle-on-your-fork

@jakirkham  I guess the problem is that I previously tried to setup a freecaed-feedstock to build packages for another channel.

Ah ok. That makes sense. Glad things are working now.


Fix TSLint warnings in botbuilder-core

Proposed Changes
Fix the following TSLint warnings in botbuilder-core:

eofline
member-ordering
newline-before-return
no-consecutive-blank-lines
no-empty-interface
prefer-const
quotemark

Testing
Travis will no longer report these warnings

Pull Request Test Coverage Report for Build 21

4 of 4 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.006%) to 69.892%

Totals

Change from base Build 14:
-0.006%

Covered Lines:
2231

Relevant Lines:
3048

💛  - Coveralls

The commits were rebased.


Better code profiling

Is your feature request related to a problem? Please describe.
Currently, all the code profiling is custom based on internal functions from psutil and/or pytorch. It would be great to have a more uniform solution.
Describe the solution you'd like
I found a few options:

Torch Profiler: Pros: extensive pytorch-specific debugging information; cons: requires changes to the training/inference apis of GaNDLF
Scalene: Pros: straightforward start and stop for pieces of code that are to be profiled; cons: generic hardware-level information
octoml: Pros: potential acceleration of prediction; cons: requires modification of model code, PyTorch-2.0 needed
DLProf: Pros: CLI-based; cons: doesn't seem to give pytorch-specific information
HolisticTraceAnalysis: Pros: tight integration with PyTorch; cons: very new library so might need to see how this develops

We should have a detailed discussion on any potential requirements and how we can evaluate these libraries.
Describe alternatives you've considered
N.A.
Additional context
N.A.

@pranayasinghcsmpl you are still working on this, right?

@pranayasinghcsmpl you are still working on this, right?

I believe this is still being worked on, right?


Update to the latest version of hydrosdk

Update to the latest version of hydrosdk to fix automatic NumPy conversion
After that PR we would be able to return dictionaries with Numpy objects and TensorProtos from an executable in func_main.py
Example of func_main.py:
import pandas as pd
from joblib import load

clf = load('/model/files/classification_model.joblib')

def predict(**kwargs):
    x = pd.DataFrame.from_dict({"request": kwargs}).T
    predicted = clf.predict(x)
    return {"income": int(predicted)}

or you can return TensorProto
    return {"income": TensorProto(int_val=[int(predicted)],
                                  dtype=DT_INT32,
                                  tensor_shape=TensorShapeProto())}

@KineticCookie I think it'd be great to specify those examples and conversion capabilities in README.md, wdyt?


one epoch killed

@leftthomas
Hi , I tried to change batchsize to 8, it didn't work.
Any ideas?

I solved it ,Thanks!

no sloved.


Prepare Analytical Dataset

Dear @AaronGullickson,
Thank you for your help. This assignment is done and all work has been committed and pushed to git and is ready for review.
Thank you,
Kevin Gable

Looks good but don't you need age to get birth cohort?

Good morning Professor Gullickson,
I guess I will grab that too. I assume I need to go to GSS and grab that.
Are we having class today?
Thank you,
Kevin Gable

I tried to download the age data in a new extract. When I unzipped it, I got an empty file. Is it possible to do a Zoom meeting this week for about 5 minutes? I ran into this before and I can't remember what I did wrong.
Thank you,
Kevin Gable

Good morning @AaronGullickson,
I hope all is well. I do appreciate your help. I am still trying to unzip my tar.gz file on my Mac. I completed another extract with age included. I can't believe I forgot it. When I double click it to unzip, the folder is empty. I am not sure how to fix it. I have looked on line and haven't found useful suggestions yet. My plan is to unzip this file, replace the data in my raw data folder, then code age in groups on ten years, 0-18 = "under 18", 18-28 = "18 -28 year-olds", "...", 78 or above = "Over 78". Does this makes sense to do? I am thinking about the analysis stage and want to be sure I look at birth cohorts correctly. I will keep the age as a numeric as well just in case.
I just need to unzip this file. It is the same issue I had last time. I know the data is there, 194Mb of data is shown to be there. Do you remember how we fixed this last time?
Thank you,
Kevin Gable

I would code based on birth cohort rather than age in ten year increments. You can derive birth cohort by taking year-age and then you can use the cut command in R to create 10-year slices, something like:
gss$bcohort <- gss$year-gss$age
gss$bcohort10 <- cut(gss$bcohort, seq(from=1900, to=2010, by=10), include.lowest=TRUE)

Its possible you might need to collapse the earliest birth cohorts into a larger group if they are very small, which you can do by adjusting that second argument a bit.
On downloading the data from GSS - I am guessing its just a problem with the horrible GSS Explorer system and you are not DLing the right thing or setting up the extract right. You have to be sure to click on the extract you want in the extract tab and then use Action > Download.

Good afternoon @AaronGullickson
I hope all is well. I was able to get the file downloaded and unzipped. You were correct! I reloaded my data into my raw folder and read in the data. So far, all my other variables are working fine. When looking at age, I read it in and coded for missing variables. When I run the table command to check, table(gss$AGE, gss$age, exclude=NULL), I get a strange response. I get, [ reached getOption("max.print") -- omitted 62 rows ]. My table result stops giving data after age 27. This surprised me because the missing variables are coded the same on GSS, anything less than zero is NA. Am I doing something wrong? It works for the other variables.
I put in the code for age cohorts. I am considering how to adjust the second argument. My lowest values are at the age of 18 and ages 78 and older. I still need to run this code so I can see what I might need to collapse. I am predicting older cohorts. That will be fun. I am so close.
Thank you,
Kevin Gable

R just truncates output beyond a certain point . Age s quant so don't table it. Just summary it.


Can't select use case step without opening up step dialog

Expected behaviour
Clicking on a use case step should highlight the row and show the exception table.
Actual behaviour
Clicking on a use case step both highlights the row, and opens the use case step dialog.
Steps to reproduce the behaviour

Open ACME Water model on live demo
Open Modify Telemetry Software use case.
Click any step in the Day or Night environment

Version of CAIRIS
All
Operating Systems / Hardware
All

Fixed in previous update


addPointRadius does not accept doubles

and similar functions like addPointHitRadius should also accept doubles, use Number instead of Integer.

Made attributes into numbers in commit d9cb935e071b31124fb26f6b994fe010c2b835e1


Invalid dbf file causes incorrect data to be linked to GeoJSON output

First, I don't necessarily think this is an issue with this library, but it would be great if this library would throw an exception in the situation in which there are more dbf entries than shape features. Yes, this is a bad/invalid shapefile but we don't have control over the source. There might actually be shape features but they have no coordinates/points.
Attached is a very small shapefile to reproduce the issue. badshp.zip
If you take that file and drop it here, http://leaflet.calvinmetcalf.com/#6/-33.596/138.076, you will see it cover Kangaroo Island and this is the correct location. If you click on the shape to display the info window you will see it has the incorrect data and the name is "No usual address" which is coming from the first row in the dbf file.

Interestingly, if I load this in Qgis and select the feature, it shows the correct information.

If this library was able to work like Qgis, great, but at the very least it should probably throw an exception as this can result in very corrupt data outputs.

Hm I guess there might be id's I can check instead of just going row by row
On Thu, Aug 13, 2020, 1:32 AM Jason Venable<EMAIL_ADDRESS>wrote:

First, I don't necessarily think this is an issue with this library, but
it would be great if this library would throw an exception in the situation
in which there are more dbf entries than shape features. Yes, this is a
bad/invalid shapefile but we don't have control over the source
<EMAIL_ADDRESS>2013?OpenDocument
.
Attached is a very small shapefile to reproduce the issue. badshp.zip
https://github.com/calvinmetcalf/shapefile-js/files/5067255/badshp.zip
If you take that file and drop it here,
http://leaflet.calvinmetcalf.com/#6/-33.596/138.076, you will see it
cover Kangaroo Island and this is the correct location. If you click on the
shape to display the info window you will see it has the incorrect data and
the name is "No usual address" which is coming from the first row in the
dbf file.
[image: Screen Shot 2020-08-12 at 10 26 49 PM]
https://user-images.githubusercontent.com/1007085/90097530-fccd6a80-dcea-11ea-90fc-886215d24467.png
Interestingly, if I load this in Qgis and select the feature, it shows the
correct information.
[image: Screen Shot 2020-08-12 at 10 28 16 PM]
https://user-images.githubusercontent.com/1007085/90097634-41590600-dceb-11ea-8bd2-4fefc2bf87fa.png
If this was able to work like Qgis, great, but at the very least it should
probably throw an exception as this can result in very correct data outputs.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/calvinmetcalf/shapefile-js/issues/134, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAITRHYXISJI4E44RV7K74DSAN3FZANCNFSM4P52J2YQ
.

Row by row should work, as that is the spec, but looks like some files don't follow the sec. I may try to get a PR for this if I can squeeze in the time but at least wanted to get the issue logged in case anyone else came across the issue or felt ambitious about doing a PR.

ok cool, If I have some time at some point I can take a look

@yakomaru @calvinmetcalf thanks for merging this fix so quickly. It definitely helps us with some data corruption issues we were experiencing.


[dv] Use macro UVM to distinguash UVM TB and the other environments

Xcelium doesn't compile UVM at the beginning. UVM_PKG_SV macro isn't set
before compile prim_assert. Hence, assert error isn't reported as
uvm_error. Use +define+UVM to solve this issue
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

LGTM. Please check if there are other places where UVM_PKG_SV is being used.
grep -nir UVM_PKG_SV *
dv/sv/sw_msg_monitor_if/sw_msg_monitor_if.sv:413:`ifdef UVM_PKG_SV
ip/prim/rtl/prim_assert.sv:15:`ifdef UVM_PKG_SV
ip/prim/rtl/prim_assert.sv:37:`ifdef UVM_PKG_SV

LGTM. Please check if there are other places where UVM_PKG_SV is being used.
grep -nir UVM_PKG_SV *
dv/sv/sw_msg_monitor_if/sw_msg_monitor_if.sv:413:`ifdef UVM_PKG_SV
ip/prim/rtl/prim_assert.sv:15:`ifdef UVM_PKG_SV
ip/prim/rtl/prim_assert.sv:37:`ifdef UVM_PKG_SV

Thanks. Updated sw_msg_monitor_if


Add image support

Allow a markdown file contain image links.  The images themselves should be in the respective post/page's directory rather than a static/images/ type solution.

Done.  Need to add logic to differentiate between local images and links to web hosted images.  Separate issue created.


Tool Issues Failure for GHG Tracking when Active/Stored Labels Don't Match Lifetime

Tool issues a failure for the labels for active/stored not being a subset of lifetime. This doesn't align with ARB's requirements. I suggest that it is changed to check if stored/lifetime are a subset of active.
16:45:00.5481 GHG Tracking Arrays from Engine #2 (1)
|-------------------------+-------------+-------------+-------------|
|                         |    Active   |    Stored   |             |
|                         |   100 Hour  |   100 Hour  |   Lifetime  |
|-------------------------+-------------+-------------+-------------|
| Engine Run Time, s      |           0 |         N/A |          17 |
| Vehicle Dist., km       |           0 |           0 |           0 |
| Vehicle Fuel, l         |           0 |           0 |           1 |
| Engine Fuel, l          |           0 |           0 |           2 |
| Eng.Out.Energy, kW-hr   |           0 |           0 |           0 |
| PKE Numerator           |           0 |           0 |           0 |
| Urban Speed Run Time, s |           0 |           0 |           0 |
| Idle Run Time, s        |           0 |           0 |         N/A |
| Engine Idle Fuel, l     |           0 |           0 |           0 |
| PTO Run Time, s         |           0 |           0 |           0 |
| PTO Fuel Consumption, l |           0 |           0 |           0 |
| AES Shutdown Count      |           0 |           0 |         N/A |
| Stop-Start Run Time, s  |           0 |           0 |         N/A |
|-------------------------+-------------+-------------+-------------|

16:45:00.5551 Destination Specific GHGTTL Request to Engine #2 (1)
16:45:00.5571 18EA01F9 [3] 01 FB 00 (TX)
16:45:00.6451 1CFB0101 [28] FA 01 00 00 00 00 00 00 00 F9 00 00 00 00 00 00 00 00 F8 C4 02 00 00 00 00 00 00 00
Green House Gas Lifetime Active Technology Tracking from Engine #2 (1):
  Active Technology:  Mfg Defined Active Technology 1 (250), Time = 1.000 s, Vehicle Distance = 0.000 m
  Active Technology:  Mfg Defined Active Technology 2 (249), Time = 0.000 s, Vehicle Distance = 0.000 m
  Active Technology:  Mfg Defined Active Technology 3 (248), Time = 708.000 s, Vehicle Distance = 0.000 m

16:45:00.6536 Destination Specific GHGTTA Request to Engine #2 (1)
16:45:00.6556 18EA01F9 [3] 00 FB 00 (TX)
16:45:00.7251 1CFB0001 [16] FA 00 00 00 00 F9 00 00 00 00 F8 00 00 00 00 00
Green House Gas Active 100 Hour Active Technology Tracking from Engine #2 (1):
  Active Technology:  Mfg Defined Active Technology 1 (250), Time = 0.000 s, Vehicle Distance = 0.000 km
  Active Technology:  Mfg Defined Active Technology 2 (249), Time = 0.000 s, Vehicle Distance = 0.000 km
  Active Technology:  Mfg Defined Active Technology 3 (248), Time = 0.000 s, Vehicle Distance = 0.000 km

16:45:00.7261 Destination Specific GHGTTS Request to Engine #2 (1)
16:45:00.7286 18EA01F9 [3] FF FA 00 (TX)
16:45:00.8051 1CFAFF01 [16] FA 00 00 00 00 F9 00 00 00 00 F8 00 00 00 00 00
Green House Gas Stored 100 Hour Active Technology Tracking from Engine #2 (1):
  Active Technology:  Mfg Defined Active Technology 1 (250), Time = 0.000 s, Vehicle Distance = 0.000 km
  Active Technology:  Mfg Defined Active Technology 2 (249), Time = 0.000 s, Vehicle Distance = 0.000 km
  Active Technology:  Mfg Defined Active Technology 3 (248), Time = 0.000 s, Vehicle Distance = 0.000 km

16:45:00.8066 GHG Active Technology Arrays from Engine #2 (1)
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
|                                     |    Active   |    Active   |    Stored   |    Stored   |             |             |
| Index                               |   100 Hour  |   100 Hour  |   100 Hour  |   100 Hour  |   Lifetime  |   Lifetime  |
| Description                         |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
| SAE/ISO Reserved                    |         N/A |         N/A |         N/A |         N/A |  69,625,446 |           0 |
| Cylinder Deactivation               |         N/A |         N/A |         N/A |         N/A |           0 |           0 |
| Mfg Defined Active Technology 3     |           0 |           0 |           0 |           0 |         N/A |         N/A |
| Mfg Defined Active Technology 2     |           0 |           0 |           0 |           0 |         N/A |         N/A |
| Mfg Defined Active Technology 1     |           0 |           0 |           0 |           0 |           0 |           0 |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|

FAIL: <IP_ADDRESS>.g - Active 100 hr array value received was greater than zero from Engine #2 (1) for SPN SPN 12694, GHG Tracking Active 100 Hour Active Technology Index: Mfg Defined Active Technology 1
FAIL: <IP_ADDRESS>.f - Active labels received is not an equivalent set of lifetime labels
FAIL: <IP_ADDRESS>.f - Stored labels received is not an equivalent set of lifetime labels

There are similar complaints … this may be a duplicate of one of them.

Set of labels matching bugs in progress.  Excess categories complete.  Steps 1.26, 2.17 and 11.9 are all being addressed.

Users will recheck with 3.1.13

@ericthomasswenson This is still an issue in 3.1.19:
15:24:27.7941 Destination Specific GHGTTS Request to Engine #2 (1)
15:24:27.7981 18EA01F9 [3] FF FA 00 (TX)
15:24:27.8721 1CFAFF01 [16] FA 00 00 00 00 F9 00 00 00 00 F8 00 00 00 00 00
Green House Gas Stored 100 Hour Active Technology Tracking from Engine #2 (1):
Active Technology:  Mfg Defined Active Technology 1 (250), Time = 0.000 s, Vehicle Distance = 0.000 km
Active Technology:  Mfg Defined Active Technology 2 (249), Time = 0.000 s, Vehicle Distance = 0.000 km
Active Technology:  Mfg Defined Active Technology 3 (248), Time = 0.000 s, Vehicle Distance = 0.000 km
15:24:27.8751 GHG Active Technology Arrays from Engine #2 (1)
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
|                                     |    Active   |    Active   |    Stored   |    Stored   |             |             |
| Index                               |   100 Hour  |   100 Hour  |   100 Hour  |   100 Hour  |   Lifetime  |   Lifetime  |
| Description                         |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
| SAE/ISO Reserved                    |         N/A |         N/A |         N/A |         N/A |  69,625,446 |           0 |
| Waste Heat Recovery                 |         N/A |         N/A |         N/A |         N/A |           0 |           0 |
| Mfg Defined Active Technology 3     |           5 |           0 |           0 |           0 |         N/A |         N/A |
| Mfg Defined Active Technology 2     |           0 |           0 |           0 |           0 |         N/A |         N/A |
| Mfg Defined Active Technology 1     |           0 |           0 |           0 |           0 |           0 |           0 |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
FAIL: <IP_ADDRESS>.e - Number of active labels received differs from the number of lifetime labels
FAIL: <IP_ADDRESS>.e - Number of stored labels received differs from the number of lifetime labels
FAIL: <IP_ADDRESS>.f - Active labels received is not a subset of lifetime labels
FAIL: <IP_ADDRESS>.f - Stored labels received is not a subset of lifetime labels

@ericthomasswenson The above makes more sense to be under #1233

Using data from this bug report I created a unit test and found the bug.  I assume the correct output should be:
10:15:30.0000 GHG Active Technology Arrays from Engine #1 (0)
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
|                                     |    Active   |    Active   |    Stored   |    Stored   |             |             |
| Index                               |   100 Hour  |   100 Hour  |   100 Hour  |   100 Hour  |   Lifetime  |   Lifetime  |
| Description                         |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |    Time, m  |   Dist, km  |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|
| Mfg Defined Active Technology 3     |           0 |           0 |           0 |           0 |          12 |           0 |
| Mfg Defined Active Technology 2     |           0 |           0 |           0 |           0 |           0 |           0 |
| Mfg Defined Active Technology 1     |           0 |           0 |           0 |           0 |           0 |           0 |
|-------------------------------------+-------------+-------------+-------------+-------------+-------------+-------------|

The bug was a continuation of the "interpreting the lifetime packet as a 100 hour packet" bug that was discovered a while back.


design: Part title pages, mini change

Following my design mockup in #97 the "Teil 1" should also be in 9pt.
Is that possible?

Of course, done in d85b2f6 ;)
However, the part prefix looks much smaller compared to the actual part than in your design. Can you please check that? (If everything is fine, just close the issue.)


feat: added sepolia testnet

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.


VEGA-1914 - Add paymentDate field and update tests

VEGA-1914
Checklist

[ ] I have updated the end-to-end tests to work with my changes
[ ] I have added styling for Dark Mode
[ ] I have checked that my UI changes meet accessibility standards

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e1c1c64) 73.70% compared to head (2f08560) 73.71%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #438      +/-   ##
==========================================
+ Coverage   73.70%   73.71%   +0.01%
==========================================
  Files         101      101
  Lines        4118     4120       +2
==========================================
+ Hits         3035     3037       +2
  Misses        777      777
  Partials      306      306

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


df: disable failing tests on Windows

This PR disables the four tests mentioned in https://github.com/uutils/coreutils/issues/5015 on Windows.

I would like to know a bit more why we regressed :/

Yes, that would be nice, though I have no more ideas how to figure it out. In my experiments (running cargo update; testing v0.0.19; using windows 2019 in the workflow file) the four df tests still failed on Windows.

@cakebaker https://github.com/uutils/coreutils/pull/5026 some improvements here
once it is merged, we can adjust this PR


How about adding hypothesizer that supports gating

In my case(mechanical scanning radar), approximately 50 or more targets are in the region of surveillance simultaneously, which rendered GNN association impossible because of the combination explosion problem.
I think association algorithm speeds up by dividing surveillance region into several sections and only considering hypothesis which associates detections that are in nearby sections.

Some improved approaches to association would be useful.
I've found much better performance from using #126, but whilst making GNN faster, doesn't solve the issue of gating to avoid unnecessary hypotheses being generated.
I've just created (#145), a hypothesiser that uses a kd-tree for detections, and used to wrap existing hypothesisers to provide gating. There's some overhead in constructing the tree, but performance reward soon outweighs this. I'd be interested if you find this improves things in your scenario @Huang-Chuan?
It may also make sense to have a tree that is track based instead, but think this will be a limit more awkward to implement due to current layout of the framework.

@Huang-Chuan Have you tried the 2d assignment GNN in #126? This is orders of magnitude faster than the default GNN.

Not yet. But I did read the paper by Dr. Crouse where he suggested clustering when problem size is over 10. Is there any way to do this?

Hopefully you'll find the 2D assignment GNN may be fast enough for 50 targets, but it is also dependent on number of detections.
Do you have the title or link to their paper?

Link: https://ieeexplore.ieee.org/document/7842561
Title: Computation of Target-Measurement Association Probabilities Using the Matrix Permanent
I think 2D assignment GNN uses exactly the algorithm proposed in this paper where the author gives performance evaluation on different number of targets. The time it takes increases from 1.51ms for 1 target to 2.6s for 10 targets.

Thanks @Huang-Chuan. So the 2D assignment that is used in #126 is outlined in the paper On implementing 2D rectangular assignment algorithms by @DavidFCrouse, which David kindly converted into Python for us from the Tracker Component Library.
You'll see from the paper, the performance expected is significantly better.

Forgive me for carelessly referring to an irrelevant paper. I'll read that paper and see if it works.


Custom function to collapse all whitespaces (incl newlines) into a single space

Is your feature request related to a problem? Please describe.
I write details git commit messages. So they often have newlines and indentations. But when git-cliff is parsing those, I would like to see a 1-liner commit message truncated to 90 characters. I would like to do something like s/\s+/ /g as one would do in sed (i.e. collapse multiple spaces into a single space). But I cannot find a way to do regex based replacement using Tera.
Describe the solution you'd like
Custom git-cliff function called ws_collapse or something that reduces consecutive newlines, spaces, tabs into a single space.
Describe alternatives you've considered
So I am doing this:
https://github.com/kaushalmodi/ox-hugo/blob/7af7e72e139655132cfa3ca5b9152690b8eaf1fe/doc/cliff.toml#L31-L35

If a generated regex based replacement function can be added then I would like to replace this:
replace(from="https://github.com/kaushalmodi/ox-hugo/issues/", to="Issue ")

https://github.com/kaushalmodi/ox-hugo/blob/7af7e72e139655132cfa3ca5b9152690b8eaf1fe/doc/cliff.toml#L36
with something like
s|https://github.com/.*/issues/([0-9]+)|[Issue # \1](\0)|g

Hello!
I recently pushed 64317f21168a9e9aa088befbd2841632f72e13a3 which makes it possible to manipulate commit messages before actually parsing/templating. Maybe it can also be utilized for your use-case like this:
[git]
commit_preprocessors = [
	{ pattern = "  +", replace = " "}, # remove multiple whitespaces
    { pattern = "https://github.com/.*/issues/([0-9]+)", replace = "[Issue #${1}]"} # turn issue links into numbers
 ]

Can you try this out?
See commit_preprocessors for more information.

Thank you! bow . You can see me using this new feature in kaushalmodi/ox-hugo@cb2fad1 .

Oh nice! You seem to use it very extensively, feel free to add new examples to README.md 😄


[MetricsAdvisor] Cannot overwrite serialization method in Patch models

Method
void IUtf8JsonSerializable.Write(Utf8JsonWriter writer)

cannot be overwritten. CodeGen still generates the original one, which prevent us from changing the default serialization behavior for patch models.

Fixed by https://github.com/Azure/azure-sdk-for-net/pull/22457, https://github.com/Azure/azure-sdk-for-net/pull/22463, https://github.com/Azure/azure-sdk-for-net/pull/22464, and https://github.com/Azure/azure-sdk-for-net/pull/22465.


migrate beefy relayer into own worker

all seems to be working, with initial and complete transactions all being sent :)
e2e tests all pass
it's based off of the parachainCommitmentRelayer branch for convenience and I expect that will be merged first
@denalimarsh plz review thoroughly

@denalimarsh all fixed in last commit

@vgeddes will do fixes in follow up PR - issue here: https://github.com/Snowfork/polkadot-ethereum/issues/359


Monitor and Manage Event-Driven Messaging

Hi,
Istio Excel in managing and Monitoring Microservices using direct messaging (a.k.a (a)synchronous service-to-service communication)
Is there any plan to support Event-Driven Messaging in the future when a Messaging Broker is introduced in the architecture (Kafka, Pub/Sub...) as a medium of communication between services
Like monitor ans control messages sent to the broker, messages consumed from the Broker, ...
Thank you in advance

What features are you interested in specifically? You can use any TCP based services, including Kafka with Istio. See https://banzaicloud.com/blog/kafka-on-istio-benefits/ for some info about this

Thank you @howardjohn, I will check that out


Badge Request: GitHub repo size for git-lfs

:clipboard: Description

Is it possible to display GitHub repo size including the git-lfs files? The current badge of GitHub repo size only reports the size without the git-lfs files.

Hmm, interesting. I'm not sure if GitHub provides a way to access that information except in aggregate.

https://help.github.com/en/articles/viewing-your-git-large-file-storage-usage
https://help.github.com/en/articles/about-billing-for-git-large-file-storage

Thanks for the reply. I have just posted on GitHub Community Forum requesting GitHub to provide this piece of information.
https://github.community/t5/GitHub-API-Development-and/Feature-request-can-GitHub-API-show-repo-size-including-git-lfs/m-p/23555#M1688

It looks like there's not much we can do about this, so closing for now. Feel free to comment on the issue if there's new information and one of the maintainers can reopen it.


feat(sdk): Allow user to name the models given in wandb.watch

Fixes #5135
Description
This pull request solves the issue by allowing the user to pass models as a dictionary. In addition, an additional parameter has been added to allow users to specify names when models are passed as list/tuple.
The visibility in the UI would greatly benefit from this feature: https://community.wandb.ai/t/nested-log-structure-which-is-visible-in-the-ui/3938
So, the collapsible would look like:
parameters/
├── model_1/
│   ├── hist_1
│   ├── hist_2
│   └── ...
├── model_2/
│   ├── hist_1
│   ├── hist_2
│   └── ...
gradients/
├── model_1/
│   ├── hist_1
│   ├── hist_2
│   └── ...
├── model_2/
│   ├── hist_1
│   ├── hist_2
│   └── ...
Testing
Tested manually
Checklist

[ x] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ x] Ensure PR title compliance with the conventional commits standards

Thanks.. we are going to look at this for the next months release.

@jacr13 thanks for the PR. we are going to revamp our watch function in the coming months and we will address the request to be able to specify names for the models passed.


Style 'flag drink (revision)' modal

@entrity I can't at the moment successfully approve my own revision so I can see/modify the flagging features.  Can you show me a way of doing this?

@sirtastic : Set up another user account, then run rails r 'User.update_all confirmed_at:1.day.ago'.
Let me know if that doesn't work.

@entrity I never could create an account-- from what I can recall it errors out when you click submit.
If you want to take a look I'll be free every evening Monday through Friday (will be working on tipsy every evening).


Differences between *_rigid.pdbqt and *.pdbqt files of receptors

Hi,
I prepared *_rigid.pdbqt and _*_flex.pdbqt files using prepare_flexreceptor.py for flexible docking with vina scoring function.
Beforehand, I executed ordinary dockings (only receptors rigid) and on that time I prepared usual *.pdbqt files of receptors using the ADFRsuite script.
I'm not sure of differences between usual *.pdbqt and *_rigid.pdbqt files.
Can I use *.pdbqt files for flexible docking instead of using *_rigid.pdbqt if they are regarded as actually same ones(because *.pdbqt files are supposed to be rigid)?
if *_rigid.pdbqt files have differences from *.pdbqt ones, I'd like to know what is different from.
Regards,

hi @eunos-1128
The two files are different. The PDBQT with “rigid” in its name doesn’t contain the sidechain of the modified residue. If you use a file comparison tool you can get a better look how they differ.
For grid map computing (autogrid) and for docking, it makes more sense to use the “rigid” PDBQT as the receptor. Doing this excludes the sidechain from being doubly counted in receptor and flexible residue (which is essentially part of the mobile “ligand” in flexible docking)

@rwxayheee
Thank you! I understand :)


Use WordPress-Coding-Standard with PHP_CodeSniffer correctly "today"

Hello everybody, mine is a question more than an issue (being only a question I thought it was more correct to consider it as an "Issue" instead of a "Dependency Change" or a "Feature request"). In the transition from Atom (which recently closed its doors) to Visual Studio Code, I decided to get my hands on my personal "Developer Kit" and therefore the tools used and the related configuration files. These also include PHP_CodeSniffer and WordPress-Coding-Standard.
Clearly it is good to clarify that I use macOS. These are the operations carried out to set up these instruments:
1. Check that PHP is installed on the machine.
To do this just use the php -v code in the Terminal.
2. Installing the PHP_CodeSniffer
I personally install PHP_CodeSniffer via Homebrew. The code used for the installation is as follows brew install php-code-sniffer.
3. Installing the WordPress-Coding-Standards
The team that created this standard is currently working on the new version 3.0.0. The latest version 2.3.0 does not support PHP 8.0, 8.1, etc… therefore it does not allow the correct functioning of PHP_CodeSniffer. The problem was solved in the development version, for this reason we will use it for now by downloading it in a folder of our choice (as suggested by @jrfnl, one of the contributors of the WordPress-Coding-Standards project, in this issue https://github.com/WordPress/WordPress-Coding-Standards/issues/2035).
4. Installing PHPCSUtils and PHPCExtra dependencies
From what I was able to understand by reading the link https://github.com/squizlabs/PHP_CodeSniffer/issues/3110, PCSUtils and PHPCSExtra are two new dependencies that will be used in WordPress-Coding-Standards version 3.0.0, as confirmed @jrfnl, one of the contributors to the PHP_CodeSniffer project. For this reason we will download it in a folder of our choice.
5. Adding WordPress-Coding-Standards and the PHPCSUtils and PHPCSExtra dependencies to the PHP_CodeSniffer standards
At this point we are going to add WordPress-Coding-Standards, and dependencies, to the standards supported by PHP_CodeSniffer via the command phpcs --config-set installed_paths ~/namefolder/WordPress-Coding-Standards-develop,~/namefolder/PHPCSUtils-1.0.1,~/namefolder/PHPCSExtra-1.0.1 (clearly we're going to replace the ~/namefolder folder with the one we used for the download).
This method is not the only one that can be followed, but I wanted to ask you:

I carried out all the steps correctly and "awarely"? The "explanations" that I "gave myself" are correct? Did I miss any other important steps?
Will the dependencies be integrated into WordPress-Coding-Standard in the future, or not and therefore this method to install the tools will also be useful for me in the future?

Thanks in advance for your attention and for your work.

@vizzale As @dingo-d already suggested: using Composer (global/local) to install is the preferred method and as of WPCS 3.0.0 will be the only supported method of installation.
You can still install using the above steps, just don't expect any support from the team here.

Did I miss any other important steps?

Yes, you missed checking if there were any standards already installed using phpcs --config-show as --config-set will overwrite whatever is there, which can break existing installs of other standards. (which is a prefect example of why we will only support Composer based installs, which come with a Composer PHPCS plugin which will handle the installation of external standards correctly)

Yes, you missed checking if there were any standards already installed using phpcs --config-show as --config-set will overwrite whatever is there, which can break existing installs of other standards.

@jrfnl First of all thanks for the reply and sorry for the delay.
I had done a standards check but took it for granted.

which is a prefect example of why we will only support Composer based installs, which come with a Composer PHPCS plugin which will handle the installation of external standards correctly

So do you suggest that I also install PHP_CodeSniffer with Composer? Because currently PHP_CodeSniffer on my machine is installed via Homebrew. Do you have any well explained guide for beginners (maybe for macOS) that explains all (or almost) the steps:

Composer installation (preferably local version).
Installing PHP_CodeSniffer (only if necessary).
...

Thanks in advance for your availability.

The guide is quite simple. In your project you can just do:
composer require wp-coding-standards/wpcs --dev
composer require dealerdirect/phpcodesniffer-composer-installer --dev

The second package will register the custom rulesets you decide to add to your project automatically.
Then just create a custom ruleset for your project according to the annotated ruleset described in the phpcs repo, where you can customize the sniffs as you wish.
I recommend using composer scripts (add them to your composer.json) for easier script runs from your terminal
"scripts": {
	"standards:check": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcs",
	"standards:fix": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcbf"
}

No need to add a dependency on phpcs, as that is already the required dependency of wpcs, so the composer will know which version to pull.
So in the end, in your project, you'll have something like
{
    "name": "your-project",
    "description": "My cool project!",
    "license": "MIT",
    "require-dev": {
        "wp-coding-standards/wpcs": "^2.3",
        "dealerdirect/phpcodesniffer-composer-installer": "^1.0"
    },
    "config": {
        "allow-plugins": {
            "dealerdirect/phpcodesniffer-composer-installer": true
        }
    },
    "scripts": {
        "standards:check": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcs",
        "standards:fix": "@php ./vendor/squizlabs/php_codesniffer/bin/phpcbf"
    }
}

You can add all sorts of cool things to your composer, but that's not the point of your question.
With the above composer, you'll be able to run
composer standards:check

And get the output of the violations (or no if there are none 😄).
Hope this helps.

What @dingo-d said 👍🏻
One caveat for when WPCS 3.0.0 gets released: as of WPCS 3.0.0, you will no longer need to require-dev the dealerdirect/phpcodesniffer-composer-installer package. That will be included with WPCS as of WPCS 3.0.0. (You will still need to give permission for that plugin to run though)

To avoid that too much time passes this time, I thank both of you.
At this point to use this workflow I will wait for WPCS 3.0.0 to be ready.
But I still want to start getting familiar with Composer, so I'll do some tests on a virtual machine and if it's okay with you I'll give you some feedback on the matter. At the end of these tests I will close the issue myself.

I installed Composer locally and then, as @dingo-d suggested, I installed:

WordPress-Coding-Standard
composer require wp-coding-standards/wpcs --dev
PHP_CodeSniffer Standards Composer Installer Plugin
composer require dealerdirect/phpcodesniffer-composer-installer --dev

as of WPCS 3.0.0, you will no longer need to require-dev the dealerdirect/phpcodesniffer-composer-installer package

@jrfnl By this you mean that when WPCS 3.0.0 is ready there will be no need to install the plugin PHP_CodeSniffer Standards Composer Installer Plugin because it will be automatically installed as a dependency?
However from the tests I've done while installing these tools with Composer you still find yourself having to use the develop version of WPCS, so you might as well wait for version 3.0.0 to switch to Composer.

By this you mean that when WPCS 3.0.0 is ready there will be no need to install the plugin PHP_CodeSniffer Standards Composer Installer Plugin because it will be automatically installed as a dependency?

Correct. WPCS 3.0 will use the PHPCSUtils package, and that has require dependency on the Composer plugin.


Xtensa xcc compile build fails with SOF application on latest Zephyr main

Describe the bug
Build fails with recent Zephyr main with XCC compiler. Works ok still with gcc.
To Reproduce
install Xtensa toolchain for Intel cAVS target (RG-2017.8-linux)
git clone https://github.com/thesofproject/sof.git
cd sof
scripts/xtensa-build-zephyr.sh tgl-h
Above checks out "main" Zephyr directly and builds SOF application on top of it. See scripts/xtensa-build-zephyr.sh to clone a specific Zephyr commit.
Expected behavior
Build succeed.
Impact
Unable to build SOF. All SOF CI cases testing Zephyr fail.
Logs and console output
-c /mnt/sof/zephyrproject/zephyr/arch/xtensa/core/offsets/offsets.c
In file included from /mnt/sof/zephyrproject/zephyr/include/init.h:12,
                 from /mnt/sof/zephyrproject/zephyr/include/device.h:30,
                 from /mnt/sof/zephyrproject/zephyr/kernel/include/kernel_offsets.h:7,
                 from /mnt/sof/zephyrproject/zephyr/arch/xtensa/core/offsets/offsets.c:8:
/mnt/sof/zephyrproject/zephyr/include/kernel.h:114: error: redefinition of typedef ‘k_thread_user_cb_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:89: error: previous declaration of ‘k_thread_user_cb_t’ was here
/mnt/sof/zephyrproject/zephyr/include/kernel.h:1347: error: redefinition of typedef ‘k_timer_expiry_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:90: error: previous declaration of ‘k_timer_expiry_t’ was here
/mnt/sof/zephyrproject/zephyr/include/kernel.h:1365: error: redefinition of typedef ‘k_timer_stop_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:91: error: previous declaration of ‘k_timer_stop_t’ was here
/mnt/sof/zephyrproject/zephyr/include/kernel.h:2561: error: redefinition of typedef ‘stack_data_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:92: error: previous declaration of ‘stack_data_t’ was here
/mnt/sof/zephyrproject/zephyr/include/kernel.h:3066: error: redefinition of typedef ‘k_work_handler_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:93: error: previous declaration of ‘k_work_handler_t’ was here
/mnt/sof/zephyrproject/zephyr/include/kernel.h:4005: error: redefinition of typedef ‘k_work_user_handler_t’
/mnt/sof/zephyrproject/zephyr/include/kernel.h:94: error: previous declaration of ‘k_work_user_handler_t’ was here

Environment (please complete the following information):

OS: Ubuntu
Toolchain: See above
Commit SHA: 022bb25691fc44541e6fadf02ab669d150b073f1

Additional context
Bisect the issue to commit f527a81dc
commit f527a81dcf35e0f111f2c892ab69873b932c43a9
Author: Torbjörn Leksell<EMAIL_ADDRESS>Date:   Tue Nov 9 09:15:37 2021 +0100

    Tracing: Tracing/Kernel dependency issue fix

    Added forward declarations of types declared
    throughout kernel.h at the start of
    kernel.h. With this change it is possible to
    include tracing/tracing.h early in kernel.h
    and use the tracing functionality in
    kernel.h functions without compile errors
    which would result if tracing/tracing.h
    was included at the end of kernel.h
    (type dependencies).

    Signed-off-by: Torbjörn Leksell<EMAIL_ADDRESS>

FYI @tleksell-pe @nashif @andyross .

Confirmed, and it is indeed an error specific to the Xtensa compiler.  That patch did indeed duplicate various callback typedefs (for header dependency reasons), so they exist twice.  But GNU is OK with that and Cadence is not: (compiler flags pasted in from a hand-built Zephyr SOF)
andy@gatera:~$ cat test.c
struct a { int i; };
typedef struct a a_t;
typedef struct a a_t;

int foo(struct a *s) { return s->i; }
andy@gatera:~$ /home/andy/xcc/install/tools/RG-2017.8-linux/XtensaTools/bin/xt-xcc -Os -ffreestanding -fno-common -g -fms-extensions -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -fgnu89-inline -Werror=implicit-int -fno-pic -fno-strict-overflow -ffunction-sections -fdata-sections -mlongcalls -std=c99 -c test.c
test.c:3: error: redefinition of typedef ‘a_t’
test.c:2: error: previous declaration of ‘a_t’ was here

I'm not expert enough to know which compiler is correct per which language standard, and that's probably not helpful anyway.  What we want is a way to make xcc act like gcc here, which must surely be possible (this version of xcc is just a gcc 4.2 front end with a custom code generator).
We could put the second/repeated definitions in a XCC #ifndef I guess (we can't remove them, because those are where the documentation lives).

Some fiddling with compiler flags shows:

The same compiler emits the error with no arguments too: xt-xcc -c test.c behaves identically.
GCC 9.3.0 will emit a similar error, but only when -pedantic is passed and only for -std=c99 and lower (c11 is apparently where this entered the language?)
Older xcc's like this don't support -std=c11 (that only arrives with their clang toolchain)

Oh, and:

xcc seems to hardwire -pedantic?   Or at least I can't find a way to disable this error even though its progenitor compiler doesn't do it by default.

Hacky fix up in the linked PR.  Not really what we want to be maintaining, but this will work to unbreak the build for now.

According to #30105 C11 is not supposed to be allowed yet. However none of this is documented is it?

What we want is a way to make xcc act like gcc here, which must surely be possible (this version of xcc is just a gcc 4.2 front end with a custom code generator).

Zephyr has very elaborate toolchain abstractions in its CMake code so I was under the impression that it had much higher portability ambitions. Do these ambitions exclude C11 still?

This problem does not really surprise me, as I pointed out in #40279 (and related discussion in #40112 that spawned #40279), redefining typedefs would become a problem with compilers that are strict C99. I'm still in favor of removing the typedef duplication (unless C11 is standard) and going for a solution where we move the typedefs earlier in kernel.h (without duplication) or break them out in separate headers if needed.
@andyross You have my blessing to revert #40279 and #40112 (depends on #40279) until we agree on a better solution for the typedefs. While #40279 and #40112 are nice changes, they aren't critical for Zephyr tracing and they aren't worth breaking the build system on some platforms.

It seems revert is safer. C99 does explicitly forbid this, so you could have other compilers that hit this.


Support for Isolated Plan

Is Azure App Service on Linux supported on the isolated plan (i.e. on ASE)? The Limitations section says that it is supported for  Free, Basic, Standard, and Premium plans but not the shared tier. It doesn't make any mention of the Isolated plan.

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

ID: ef9002ed-c817-c615-fe32-05ad4cc78b61
Version Independent ID: 72295255-8d90-a2c6-0995-0e1208fcdcab
Content: Run code on default Linux containers - Azure App Service
Content Source: articles/app-service/containers/app-service-linux-intro.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@pnarsi, Thanks for the question! We are taking a look into this and will get back to you soon.

@pnarsi, Your feedback has been shared with the content owner for further review. Thanks for taking time to share your valuable feedback.

@pnarsi, We have pushed the requested changes. The document reflects the changes. Thanks again for taking time to share your feedback.


Settings theme gets saved before applying and remains until current PowerToys window is closed

Steps to reproduce

Open PowerToys settings and change the Settings color to Dark.
Switch to another settings page (example FancyZones) and click discard the changes when prompted.

Expected behavior

After discarding the settings color should revert to whatever it was originally.
Actual behavior

The settings page remains in Dark until the window is closed and then reopened or the setting is changed. The actual saved setting in General settings remains as the original setting throughout this.
Screenshots

Hi @arjunbalgovind
we will fix this in Settings v2.

Lets close this out for Settings v2, which is #889


Functions which return tuples show duplicated return types

Start with:

Which is correct-ish, ignoring the missing types (which would be better shown as Any, but that's a different issue). Now, add a call, which (in the current implementation) propagates types to the function:

The function does in fact return a tuple with three ints, but it's being shown as tuple[int, int, int][int, int, int] instead of just tuple[int, int, int].
Note that if I go and annotate the parameters myself with int (restricting the possible types from Any), this goes away:

Also:

The first one shows ? for unknown types, while the second just shows an empty string. Might help indicate where the extra info is coming from.

Let me look at it as it is related heavily to #50


overriding word length bits in UART?

I read through the article (which is very nice, thank you!) and saw that you write the register 3 here without keeping the word length bits written into the register at the start of the function. Below, when deactivating the DLAB bit, you do a modify with read and write.

To prevent commits like merge branch 'master' of ..., when you do git fetch, you can run git rebase origin/master so your local commits get applied on master

When changing from lcr.write_volatile(1 << 7) to lcr.write_volatile(lcr.read(volatile) & (1 << 7)), I now get a "P" that gets printed out to the screen.
cargo build
    Finished dev [unoptimized + debuginfo] target(s) in 0.00s
riscv64-unknown-linux-gnu-gcc -Wall -Wextra -pedantic -Wextra -O0 -g -static -ffreestanding -nostdlib -fno-rtti -fno-exceptions -march=rv64gc -mabi=lp64 -Tsrc/lds/virt.lds  -o os.elf src/asm/trap.S src/asm/boot.S -L./target/riscv64gc-unknown-none-elf/debug -lmyos -lgcc
qemu-system-riscv64 -machine virt -cpu rv64 -smp 4 -m 128M  -nographic -serial mon:stdio -bios none -kernel os.elf -drive if=none,format=raw,file=hdd.dsk,id=foo -device virtio-blk-device,scsi=off,drive=foo
PMyOS
QEMU: Terminated

Before, when the WLS0 and WLS1 were wiped, then did not occur. I do not have a P printing anywhere in my code, and if I remove the line println!("MyOS");, it still prints a P out to the screen.
Any reason as to why this would happen?

Are you writing lcr.write_volatile(lcr.read_volatile() & !(1 << 7))? Also, is your lcr variable a u8?
(Notice the invert operation for 1 << 7). When I test on my end, I can't duplicate your issue.

To prevent commits like merge branch 'master' of ..., when you do git fetch, you can run git rebase origin/master so your local commits get applied on master

I usually do this, but I rushed to get the patch out and did git pull instead of rebasing. Thanks for the info!

Figured it out; turns out, 1 & 0 is not 1. Instead of setting the DLAB, I was just putting the higher 8 bits of the divisor 592 (which turns out to be 80) to the buffer when writing to the DLM.


Add Codecov for all repositories

Codecov is already used in MeiliSearch, it helps us to know if a PR add non-tested code. If it's the case, we don't accept the PR. It's easy to install, easy to configure. Don't hesitate to ping me if you need help.

I love that!

@bidoubiwa? We need your approval so that I can try it on one repo

Great!

Great!

I would love to see this implemented in all the SDKs, but unfortunately, we will not have time to work on this for the next months. I'll appreciate it if people from the community do it :)

All the issues were created in the correspondent repositories!


Recurrent DQN 170801

In testing your recurrent branch I found the following changes helpful:

Parameterized training a bit more
Created flag to turn off progbar logging every step
Trained 1.75m steps with train_interval = 100*
Used Breakout-v0 as you had in your branch

*With the original setting of train_interval = 10, training was taking roughly 2 hours / 10,000 steps after the initial nb_steps_warmup=50000, so set to 100 to get through training.
Here was the observed reward difference for this branch Recurrent DQN 170801:
recurrent_dqn_atari: Breakout-v0, train_steps = 1750000, train_interval = 100
Testing for 10 episodes ...
Episode 1: reward: 16.000, steps: 819
Episode 2: reward: 16.000, steps: 840
Episode 3: reward: 8.000, steps: 759
Episode 4: reward: 8.000, steps: 577
Episode 5: reward: 7.000, steps: 330
Episode 6: reward: 10.000, steps: 467
Episode 7: reward: 8.000, steps: 408
Episode 8: reward: 12.000, steps: 650
Episode 9: reward: 14.000, steps: 641
Episode 10: reward: 5.000, steps: 355

vs. the Master Branch:
dqn_atari: Breakout-v0, train_steps = 1700000, train_interval=4
Testing for 10 episodes ...
Episode 1: reward: 0.000, steps: 10000
Episode 2: reward: 38.000, steps: 1833
Episode 3: reward: 25.000, steps: 10000
Episode 4: reward: 20.000, steps: 10000
Episode 5: reward: 27.000, steps: 1452
Episode 6: reward: 20.000, steps: 10000
Episode 7: reward: 27.000, steps: 1192
Episode 8: reward: 21.000, steps: 10000
Episode 9: reward: 16.000, steps: 10000
Episode 10: reward: 30.000, steps: 1590

Any further suggested changes that might prove the efficacy of recurrent vs. what you have in the master branch?

I am not sure it is fair to compare the results with different train intervals if you are going to stop before convergence.

Fair enough, @RyanHope .  Any particular train interval/steps combo you would suggest we try for each for apples to apples comparison?  Right now recurrent_dqn_atari  10 / 1750000 = 250 hours on my machine.  I guess I could just test 100 / 1750000 on dqn_atari, making for a fair comparison to the recurrent_dqn_atari results above.  Just kicked that off, which should run in just a few hours.

@RyanHope it does seem as if the same setting of train_interval = 100 on the original dqn_atari.py (below) does not perform as well as the new recurrent_dqn_atari.py (previous post above):
# Aug 2, 2017 13732 seconds, train_interval = 100
# Episode 1: reward: 0.000, steps: 10000
# Episode 2: reward: 1.000, steps: 10000
# Episode 3: reward: 2.000, steps: 10000
# Episode 4: reward: 4.000, steps: 10000
# Episode 5: reward: 1.000, steps: 10000
# Episode 6: reward: 2.000, steps: 10000
# Episode 7: reward: 1.000, steps: 10000
# Episode 8: reward: 2.000, steps: 10000
# Episode 9: reward: 2.000, steps: 10000
# Episode 10: reward: 6.000, steps: 10000

I do wonder if there is a better nb_steps / train_interval middle ground to test both and compare again, as train_interval = 100 seems to make dqn_atari.py act inconsistent.
Visualization on recurrent_dqn_atari.py, Breakout-v0, 1,750,000 steps train_interval = 100:

Visualization on dqn_atari.py, Breakout-v0,  1,750,000 steps train_interval = 100:

After 5 days of running, here is a train_interval = 50 comparison of dqn_atari vs. recurrent_dqn_atari:
Recurrent_dqn_atari, Breakout-v, train_interval = 50, 1,750,000 steps
Episode 1: reward: 11.000, steps: 1000
Episode 2: reward: 16.000, steps: 778
Episode 3: reward: 15.000, steps: 1069
Episode 4: reward: 11.000, steps: 684
Episode 5: reward: 20.000, steps: 1073
Episode 6: reward: 21.000, steps: 1009
Episode 7: reward: 15.000, steps: 805
Episode 8: reward: 6.000, steps: 535
Episode 9: reward: 6.000, steps: 470
Episode 10: reward: 5.000, steps: 518
Average Reward: 12.6
Dqn_atari, Breakout-v0, train_interval = 50, 1,750,000 steps
Episode 1: reward: 8.000, steps: 542
Episode 2: reward: 4.000, steps: 334
Episode 3: reward: 9.000, steps: 581
Episode 4: reward: 11.000, steps: 741
Episode 5: reward: 13.000, steps: 773
Episode 6: reward: 7.000, steps: 531
Episode 7: reward: 16.000, steps: 890
Episode 8: reward: 9.000, steps: 577
Episode 9: reward: 12.000, steps: 730
Episode 10: reward: 8.000, steps: 600
Average Reward: 9.7
So, that seems like good news for recurrent_dqn_atari, though sure wish we knew how to speed it up with it taking 5 days to train 1,750,000 steps vs dqn_atari taking just an hour or two.
Here is a vis for both:
recurrent_dqn_atari (left)  ------------------------------- dqn_atari (right)

Trained weights:
recurrent_dqn_atari: https://www.dropbox.com/sh/i3vnudfes5owo06/AAACGvOv0erh4XhQUKQRyn9ya?dl=0
dqn_atari: https://www.dropbox.com/sh/1o1aj3xtwgevp21/AAC3_cPGTEYxa94utJ5CSABEa?dl=0
Any interpretations anyone have to offer?

Nice to see this pull request!
I'm trying to use DRQN and extend it with augmented memory structures, using Keras-RL.
I'd be really grateful for any help you all can give in this regard.
Thanks