benchmark pending by unknown reason

What version of Go are you using (go version)?

$ go version
go version go1.12 darwin/amd64

Does this issue reproduce with the latest release?
yes,I use go1.12 already
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/XXX/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/XXX/.gvm/pkgsets/go1.12/global"
GOPROXY=""
GORACE=""
GOROOT="/Users/XXX/.gvm/gos/go1.12"
GOTMPDIR=""
GOTOOLDIR="/Users/XXX/.gvm/gos/go1.12/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/90/hr1_x8993q9fyw7gf7g_85v80000gp/T/go-build592076382=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

runnining benchmark like https://github.com/FelixSeptem/sizepool/blob/master/sizepool_test.go#L111-L119 but pending forever ,running result like https://www.travis-ci.org/FelixSeptem/sizepool
What did you expect to see?
running benchmark like usual
What did you see instead?
the benchmark pending forever

There are 2 main things wrong with your code.
func (p *sizePoolChan) BGet(timeout time.Duration) (interface{}, error) {
	ticker := time.NewTicker(timeout)
        ...
}

You need to add a defer ticker.Stop(), also a time.Timer is better for one time events.
func BenchmarkSizePoolChan_BGet(b *testing.B) {
	p := NewChanPool(65535, newConn, resetConn)
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		if _, err := p.BGet(time.Millisecond); err != nil {
			_ = err
		}
	}
}

b.N is a very large number, e.g. 1000000. worst case it will take ~1000000 ms = 16.67 minutes
If you must use time.Millisecond then you need to tell test to run it less times. e.g. go test -benchtime=100x 100x = run 100 times.

There are 2 main things wrong with your code.
func (p *sizePoolChan) BGet(timeout time.Duration) (interface{}, error) {
	ticker := time.NewTicker(timeout)
        ...
}

You need to add a defer ticker.Stop(), also a time.Timer is better for one time events.
func BenchmarkSizePoolChan_BGet(b *testing.B) {
	p := NewChanPool(65535, newConn, resetConn)
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		if _, err := p.BGet(time.Millisecond); err != nil {
			_ = err
		}
	}
}

b.N is a very large number, e.g. 1000000. worst case it will take ~1000000 ms = 16.67 minutes
If you must use time.Millisecond then you need to tell test to run it less times. e.g. go test -benchtime=100x 100x = run 100 times.

I don't specify execute time but I give a running time here by
go test -run none -bench . -benchtime 1s -benchmem, but it still pending more than 10minutes

also, I see the sample code snippet as below:
`
func BenchmarkBigLen(b *testing.B) {
   big := NewBig()
   b.ResetTimer()
   for i := 0; i < b.N; i++ {
       big.Len()
   }
}

The thing with using 1s over 100x is that 1s means if this b.N took over 1s then stop.It runs the benchmark multiple times with increasing b.N. So at the start b.N will be small and it thinks it's a good idea to do a large maybe 1000000 run and then it doesn't stop until it finishes.
So 100x means literally run 100 times and don't try to be smart about it.

I can tell the difference between 100x and 1s.But I can't understand why it will pending a lot of time by 1s

Because 1s does not mean stop running immediately after 1s. The check for 1s is after the function returns. So if it takes a very long time it will not stop.

I try to fix it, the pending version code:
func BenchmarkSizePoolChan_BGet(b *testing.B) {
	p := NewChanPool(65535, newConn, resetConn)
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		if _, err := p.BGet(time.Millisecond); err != nil {
			_ = err
		}
	}
}

and the new pass version:
func BenchmarkSizePoolChan_BGet(b *testing.B) {
	p := NewChanPool(65535, newConn, resetConn)
	fc := new(fakeConn)
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		p.Put(fc)
		if _, err := p.BGet(time.Nanosecond); err != nil {
			_ = err
		}
	}
}

the difference is the previous one will trigger a lot of times of Ticker.C,and the latter one won't.I suspect that a big amount of Ticker.C may cause some problem in benchmark situation.

I reproduce it with https://play.golang.org/p/gAXjYIdJb2K  no clear reason to pending with benchmark

As I've already said in my very first reply, you need to have a defer ticker.Stop()
tickers keep running even after the function returns, forever. That is why you need to tell it to stop or use time.Timer which only trigger once.

As I've already said in my very first reply, you need to have a defer ticker.Stop()
tickers keep running even after the function returns, forever. That is why you need to tell it to stop or use time.Timer which only trigger once.

I have tried to manually call defer ticker.Stop,no help for the pending problem

I have tried to manually call defer ticker.Stop,no help for the pending problem

With which code? With BenchmarkSizePoolChan_BGet you need to do that and your code gets very slow with big b.N so you need to figure out why it gets slow or as I've already said, tell test to don't use a big b.N by using -benchtime=1000x or some other number.
There is no bug here, just the code you have run slow when calling in a long loop. You will find better help asking in the places listed in https://github.com/golang/go/wiki/Questions

I have tried to manually call defer ticker.Stop,no help for the pending problem

With which code? With BenchmarkSizePoolChan_BGet you need to do that and your code gets very slow with big b.N so you need to figure out why it gets slow or as I've already said, tell test to don't use a big b.N by using -benchtime=1000x or some other number.
There is no bug here, just the code you have run slow when calling in a long loop. You will find better help asking in the places listed in https://github.com/golang/go/wiki/Questions

I can't agree your explanation. As I post in https://github.com/golang/go/issues/33383#issuecomment-517089766  I do more operation in the second code snippet,and it doesn't pending. Obvious, it's not due the long loop for the first code to pending.

ticker := time.NewTicker(timeout)
	select {
	case i := <-p.pool:
		return i, nil
	case <-ticker.C:
		return nil, ErrNoEnoughItem
	}

case i := <-p.pool: You drain from the pool, after 65535 times the pool is empty, so this statement blocks forever. 65535 because p := NewChanPool(65535, newConn, resetConn)
case <-ticker.C: this statement fires after 1ms.
So now every single iteration of the loop after 65535 times will be 1ms long. If b.N is 1000000, that is a very very long time.
p.Put(fc) works because you are adding to the pool so the pool is never empty, so case <-ticker.C: will not fire.

ticker := time.NewTicker(timeout)
	select {
	case i := <-p.pool:
		return i, nil
	case <-ticker.C:
		return nil, ErrNoEnoughItem
	}

case i := <-p.pool: You drain from the pool, after 65535 times the pool is empty, so this statement blocks forever. 65535 because p := NewChanPool(65535, newConn, resetConn)
case <-ticker.C: this statement fires after 1ms.
So now every single iteration of the loop after 65535 times will be 1ms long. If b.N is 1000000, that is a very very long time.
p.Put(fc) works because you are adding to the pool so the pool is never empty, so case <-ticker.C: will not fire.

I got it.Thanks for your reply.The benchmark function must run the target code b.N times. During benchmark execution, b.N is adjusted until the benchmark function lasts long enough to be timed reliably. when I give the params with 65535, the function run time will not be stable for a long range of time,I shall give it 0 or 1000000(or other number big enough) to let it run stable. Thanks for your help.I will close this issue.


axe:test target individual stories

Summary
I suggest the modification of e2e/scripts/playwright/axe-clean.spec.ts which used to load links like this 👇
http://localhost:6006/?path=/story/components-accordion--default
This means that it was loading the whole Storybook interface with the story sidebar. This was causing some of the stories to fail AXE tests, here is an example.
The fix for it should be to load the stories via links like this 👇
http://localhost:6006/iframe.html?id=components-accordion--default&viewMode=story
But then we will need to add an ignore list for certain AXE violations, because not all stories contain certain page elements, like all necessary landmarks and headings, cause we are testing components, not the whole pages.
@rezrah do tell me what you think 🙏  Keep in mind that this is just a quick sketch! ☺
Contributor checklist:

[ ] All new and existing CI checks pass
[ ] Tests prove that the feature works and covers both happy and unhappy paths
[ ] Any drop in coverage, breaking changes or regressions have been documented above
[ ] New visual snapshots have been generated / updated for any UI changes
[ ] All developer debugging and non-functional logging has been removed
[ ] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

@rezrah developing the ignore exclude list (you can also exclude via a part of the ID, meaning the whole groups) in order to test this a bit complex excluding feature I needed to test it, so I also updated the script to print violations so we can now know what failed directly from the CI 👇

Hope that's ok? I'm also going to speed up the test a bit. Seems like the collector of all violations is within the for loop, and launch the browser outside of the loop, and close it after the loop if possible. So we don't launch it every time.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4246

Questions ?
Please refer to the Backport tool documentation


Implement model I/O via JSON using json-fortran

....trying to make I/O consistent between the fortran and python models using JSON files. This will make reading in initial conditions and parameters easier for restarts, as well as make exchanging output simpler and more transparent.

LGTM 👾


Embedding recordings as gif or png

This would allow people to embed their asciicasts in README files as static "screenshot"-like PNGs or animated GIFs.
Some helpful resources:
https://gist.github.com/lmjabreu/922530
http://blog.animatron.com/post/73949058475/rendering

+1

PNG support is already there. You can now embed asciicast in your READMEs, issues, comments here on Github. See: https://twitter.com/asciinema/status/581517854887002113
@tav thanks for building this tool.
I'm closing this issue now. If you want to discuss animated GIF support feel free to open another issue.

FYI there's "image link" embedding as well as support for oEmbed/Open Graph/Twitter Card: https://asciinema.org/docs/embedding

Please try out https://github.com/asciinema/a2gif (I implemented the idea from https://github.com/asciinema/asciinema-player/issues/24#issuecomment-217708694).


Added nice, setpriority and getpriority along with constants and type…

… definitions

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Looks like the CI failures may be legit?

Ok will try to sort this out tomorrow.
On 19 May 2016 8:06 p.m., "Alex Crichton"<EMAIL_ADDRESS>wrote:

Looks like the CI failures may be legit?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

OK, think I've got a handle on this. The Linux failures are because glibc's signatures use an enum for which, rather than an int. Yet more gratuitous breakage which deviates from the man pages. (It seems they've fiddled here more than once). I'll try to work something out for them. Fingers crossed musl eventually conquers Debian and RedHat...
The Mac OS X failure is on i686. I don't fully understand what causes the 'bad function pointer' test failure. Given that Darwin on i686 is effectively dead, if I can't fix this expediently I'll just not add nice() for this platform.

Right, the Mac OS i686 issue seems solved. FreeBSD failures are occurring because they don't use id_t. Fix applied. GNU glibc issues still happen as I think they're enum definition uses unsigned ints. Trying again... then I'll squash the commits.

OK, just Android left; have adjusted getpriority / setpriority signature to use int and not id_t... Let's see what happens.

@bors: r+ bed46f4819c543e143245f1fae24faec7c324e01

:hourglass: Testing commit bed46f4 with merge 2ebd434...

:boom: Test timed out

Whoops, pushed commits I shouldn't have...

Looks like some lint errors in CI?

:umbrella: The latest upstream changes (presumably #298) made this pull request unmergeable. Please resolve the merge conflicts.

@bors: r+ cbc88ca0d565146d0f759b6c3060225df6e74535

:hourglass: Testing commit cbc88ca with merge 263eab1...

:broken_heart: Test failed - travis

@bors: reetry
On Wed, May 25, 2016 at 10:48 AM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - travis
https://travis-ci.org/rust-lang/libc/builds/132896300
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/libc/pull/294#issuecomment-221650925

@bors: retry
On Wed, May 25, 2016 at 10:53 AM, Alex Crichton<EMAIL_ADDRESS>wrote:

@bors: reetry
On Wed, May 25, 2016 at 10:48 AM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - travis
https://travis-ci.org/rust-lang/libc/builds/132896300
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/libc/pull/294#issuecomment-221650925

:hourglass: Testing commit cbc88ca with merge a200a04...

:sunny: Test successful - status-appveyor, travis


Creating a rule with month: "*" fails

Specifying the month in a CronTimer rule results in an Internal Server Error. The below rule code works fine without the month: "*" line.
---
name: "remote_test"
pack: "default"
description: ""
enabled: true

trigger:
    type: "core.st2.CronTimer"
    parameters:
        minute: "3"
        hour: "15"
        day: "*"
        month: "*"
        day_of_week: "*"

action:
    ref: "core.remote_sudo"
    parameters:
        cmd: "whoami"
        hosts: "myhost"

Try to load it
$ st2 rule create remote_test.yaml
ERROR: 500 Server Error: Internal Server Error
MESSAGE: Internal Server Error for url: http://<IP_ADDRESS>:9101/v1/rules

We can see what happens in the logs here:
==> /plogs/st2/st2api.log <==
2016-06-03 15:26:55,881<PHONE_NUMBER>16784 ERROR hooks [-] API call failed: 'NoneType' object has no attribute 'update'
Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 631, in __call__
    self.invoke_controller(controller, args, kwargs, state)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/pecan/core.py", line 531, in invoke_controller
    result = controller(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/models/api/base.py", line 230, in callfunction
    result = f(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/rbac/decorators.py", line 85, in func_wrapper
    return func(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2api/controllers/v1/rules.py", line 96, in post
    increment_trigger_ref_count(rule_api=rule)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/services/triggers.py", line 202, in increment_trigger_ref_count
    Trigger.update(trigger_db, inc__ref_count=1)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/persistence/base.py", line 200, in update
    cls._get_impl().update(model_object, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2common/models/db/__init__.py", line 178, in update
    return instance.update(**kwargs)
AttributeError: 'NoneType' object has no attribute 'update' (_exception_data={},_exception_class='AttributeError',_exception_message="'NoneType' object has no attribute 'update'")

<snip>

2016-06-03 15:26:55,887<PHONE_NUMBER>16784 INFO hooks [-] 972bc83e-e8ab-4e6c-bbc7-7145d8deed68 - POST /v1/rules result={
    "faultstring": "Internal Server Error"
} (result='{\n    "faultstring": "Internal Server Error"\n}',request_id='972bc83e-e8ab-4e6c-bbc7-7145d8deed68',status_code='500 Internal Server Error',remote_addr='<IP_ADDRESS>',path='/v1/rules',method='POST')

Thanks for reporting this, I will look into it tomorrow.
From the error it actually seems related to a bad state in DB and not "*" value for month per say, but I could be wrong and I will see when I look into it and try to reproduce it,

Just a heads up - I was able to reproduce it and I'm looking into it.

Looks like it's some weird issue when querying and Python types are unicode (and it doesn't always happen :/)
{u'month': u'*', u'day': u'*', u'minute': u'3', u'hour': u'15'}
ipdb> Trigger.query(parameters=parameters)
[]
ipdb> parameters={'month': '*', 'day': '*', 'minute': '3', 'hour': '15'}
ipdb> Trigger.query(parameters=parameters)
[<TriggerDB: TriggerDB(description=None, id=575559840640fd0df24cdac2, name="9f95dcf2-d125-45f6-b2e1-fcaed2667778", pack="core", parameters={u'hour': u'15', u'day': u'*', u'minute': u'3', u'month': u'*'}, ref_count=0, type="core.st2.CronTimer", uid="trigger:core:9f95dcf2-d125-45f6-b2e1-fcaed2667778:e24eb4b23e53d023769d26b74010318a")>]

Thanks!

After more digging it looks like a bug lies inside pymongo. I'm trying to come up with a work-around but it turns out it's quite hard - there also appears to be a bug with query set cache so even if I disable it I get an exception back :/

@codyaray I just merged #2732 (as noted in that PR, the work-around / hack is terrible, but that's the only option we have right now).
I did confirm it myself, but can you please also confirm that this change fixes the issue for you as well (you will also need to delete "bad rules" aka ones without corresponding TriggerDB objects from the database, you might need to do that use mongo shell / repl)?
Once you confirm it, I will go ahead and close this issue.

This was discovered on my prod cluster, so I'm unlikely to update to an unstable version since I have a workaround (i.e., my rules don't actually have to specify the month). However, I'll try to test/reproduce in my dev environment.

@Kami Most of my cron triggers didn't fire again this morning. Is this related? From the logs:
StackStormDBObjectNotFoundError: ('Trigger not found for %s', {'uid': u'trigger:core:219ce49f-ae70-4491-8d23-9d2889be8dc8:7b2e3f06c16a3583662b5b67b3d0046f', 'parameters': {u'minute': 50, u'hour': 8, u'day_of_week': u'1-5'}, 'description': None, 'ref_count': 1, 'name': u'219ce49f-ae70-4491-8d23-9d2889be8dc8', 'type': u'core.st2.CronTimer', 'id': '574755ec59e5f11c4198bc0f', 'pack': u'core'})

@codyaray Yeah, I forgot to say that this bug is (as far as I know) causing this issue as well since it follows the same code-path and utilizes the same function.
In short - it should fix this issue as well.

Well. In that case, I think I might just upgrade. Can't have failing jobs in my prod cluster. :)

Upgraded to 1.5dev-92 and confirmed that this fixed the reported issue with month: "*". Can I just delete all the rules using the CLI and re-sync them to effectively removed the bad ones from the DB?

If it works, you can, yes. If it doesn't you will need to resort to Mongo shell / repl.

@Kami deleting and re-adding apparently did not fix. Got "Trigger not found" again this morning.
Just ran db.trigger_d_b.remove() on mongo master, sudo st2ctl restart on each node, and then ran st2ctl reload --register-all (and cleaned up errors until it finished successfully; still need to finish #2701]). I guess we'll see how tomorrow morning goes. :)

Closing this issue and moving back to #2715 where the existing problem may or may not still exist.


Close/Exit a browser

I want to close/exit the browser after all processing. Can someone help me how to do this in tagui??

Please reply fast....

Hi Hassan, by default, TagUI closes the browser session after running. For example when you do tagui samples\1_yahoo chrome it will launch Chrome to run and closes it after running.
If you are using visual automation to visually control an existing browser session, then you can add a step to click on the close button of the browser.

Ok. Thanks for the help. I have one more issue: I want to login to a specific site with  my credentials. If I have the credentials stored in a specific file & I want to extract those & type in username & password fields, how can I do this with tagui?? Please all suggestions are appreciated.

Credentials are stored in a text file. Although there's a  'load' command that can store filename in a variable, but I want to extract the credentials & type in the respective fields.

Create a new issue next time since different question, so other users can find answer easily.
Created #393


fix(selection tile): add ability to make selections using keyboard

Closes #880

It looks like the "selected" event on the tile-group is not getting triggered when the selection is being made using the keyboard but "selected" event is getting triggered only when the selection is made using the mouse.
I am using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I have the title-group and selection-tile's defined as follows:
      <ibm-tile-group id="dataProviderTileGroup"
                      (selected)="onDataProviderTypeChange($event)"
                      name="dataproviderTile"
                      [multiple]="false">
        <ibm-selection-tile *ngFor="let dataprovider of allDataproviders"
                            [selected]="selectedDataProviderType === dataprovider.type"
                            [value]="dataprovider.type">
        </ibm-selection-tile>
      </ibm-tile-group>

Could you please look into it and get back to me ? Thanks!

Could you please look into it and get back to me ? Thanks!

I'll look into this, thanks for bringing it up!


Add Jammy stack to Yarn dependency

Summary

https://github.com/paketo-buildpacks/yarn/issues/242
Use Cases

Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

bump @brayanhenao


add option setting: disable promethues metrics tls

What problem does this PR solve?
allow TiFlash turn off the promethues metric tls when TiFlash's tls is on. It will be used when collect metrics from multiple different tiflash tenants.
Issue Number: close #7500
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge

/merge

/merge

@bestwoody need to open an issue, and link to it.

/run-all-tests


Clarify & provide example of the validation return

In order to spare other developers from spending hours hunting this down, I've clarified that the validation token is sent in the querystring.  I've also provided an example of the return code for reference.
Category

[x] Content fix
[ ] New article

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

What's in this Pull Request?
The original article was not clear with regard to the validation token.  This change clarifies that the validation token is sent in the querystring.  It also provides an example of the return code for reference.
Submission guidelines

Docs Build status updates of commit d9ac3ed:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/apis/webhooks/lists/create-subscription.md
:warning:Warning
View
Details

docs/apis/webhooks/lists/create-subscription.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hey Vesa,
I don't know how to fix that.  Its referring to line 2 which I did not touch.
Thoughts?

Learn Build status updates of commit 12127ed:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: CannotMergeCommit] Cannot merge commit 12127ed2ca01b7150b6710059d9ce00a4c6f3a2c in branch patch-3 of repository https://github.com/cjvandyk/sp-dev-docs into branch main (commit 09f6f74e1f3916f649eda745b6ef7b79fda66432). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit cd21f50:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/8422/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


No projects load by default when clicking on the 'All' tab in /projects

Summary

Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

What is the current bug behavior?
What is the expected correct behavior?
Relevant logs and/or screenshots
Environment

Device/OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Hardware wallet: [e.g. Ledger w/ Metamask]

Hey @jmilldotdev - do you have some steps to repeat for this one? Currently, when I click all on https://juicebox.money/#/projects I see:
https://user-images.githubusercontent.com/104132113/168271933-65ec0392-d654-493a-9e1f-c542f9375c3c.mov

yeah i think it's a problem with the subgraph URL i'm using.


Build a MinIO terraform module with newest version of template

Goal: document all issues discovered along the way.
Will use template as the backbone for a new repo that should do the following:

Build a MinIO docker image
Use and test this image within the hashistack eco-system
Be possible to import this repo as its own terraform-module in use for other modules (for example be able to use this module together with a presto module)

Started this process here: https://github.com/pdmthorsrud/TF_MinIO/

Another example: https://github.com/zhenik/terraform-nomad-minio1

Update from me: I've gone through most steps of creating a terraform-module, but not the last part of making the terraform module generic and easy to use as an imported module in other modules. Will put this task on hold while doing some other stuff.
Nikita will make a new MinIO module as he linked above!

We keep work on it here: https://github.com/fredrikhgrelland/terraform-nomad-minio


auto blur nsfw photos

Description of the new feature / enhancement
auto blur nsfw photo but only for 18-
Scenario when this would be used?
at browsers and all microsoft apps
Supporting information
mail<EMAIL_ADDRESS>discord : death.#7138

No plans for powertoys to do this. Outside our core goals


FermiGBMBurstCatalog() stopped working

Hi,
This morning when I do
`%matplotlib inline
import os
import seaborn as sns
from matplotlib import cm
from matplotlib import pylab
import matplotlib.pyplot as plt
import numpy as np
import warnings
os.environ["MKL_NUM_THREADS"] = '1'
os.environ["OMP_NUM_THREADS"] = '1'
os.environ["NUMEXPR_NUM_THREADS"] = '1'
warnings.simplefilter('ignore')
from threeML import *
from threeML.io.package_data import get_path_of_data_file
gbmCatalog = FermiGBMBurstCatalog()`
I got
`---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
 in ()
----> 1 gbmCatalog = FermiGBMBurstCatalog()
/usr/local/lib/python2.7/site-packages/threeML/catalogs/Fermi.pyc in init(self, update)
55         super(FermiGBMBurstCatalog, self).init('fermigbrst',
56                                                    threeML_config['catalogs']['Fermi']['GBM burst catalog'],
---> 57                                                    'Fermi-LAT/GBM burst catalog')
58
59         self._gbm_detector_lookup = np.array(['n0', 'n1', 'n2', 'n3', 'n4', 'n5', 'n6',
/usr/local/lib/python2.7/site-packages/threeML/catalogs/VirtualObservatoryCatalog.pyc in init(self, name, url, description)
24         self.catalog = VOSCatalog.create(name, url, description=description)
25
---> 26         self._get_vo_table_from_source()
27
28         self._last_query_results = None
/usr/local/lib/python2.7/site-packages/threeML/catalogs/Fermi.pyc in _get_vo_table_from_source(self)
66         self._vo_dataframe = get_heasarc_table_as_pandas('fermigbrst',
67                                                          update=self._update,
---> 68                                                          cache_time_days=1.)
69
70     def apply_format(self, table):
/usr/local/lib/python2.7/site-packages/threeML/io/get_heasarc_table_as_pandas.pyc in get_heasarc_table_as_pandas(heasarc_table_name, update, cache_time_days)
136     with warnings.catch_warnings():
137         warnings.simplefilter("ignore")
--> 138         vo_table = votable.parse(file_name_sanatized)
139
140     table = vo_table.get_first_table().to_table(use_names_over_ids=True)
/usr/local/lib/python2.7/site-packages/astropy/io/votable/table.pyc in parse(source, columns, invalid, pedantic, chunk_size, table_number, table_id, filename, unit_format, datatype_mapping, _debug_python_based_parser)
140         _debug_python_based_parser=_debug_python_based_parser) as iterator:
141         return tree.VOTableFile(
--> 142             config=config, pos=(1, 1)).parse(iterator, config)
143
144
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in parse(self, iterator, config)
3403             if start:
3404                 tag_mapping.get(tag, self._add_unknown_tag)(
-> 3405                     iterator, tag, data, config, pos)
3406             elif tag == 'DESCRIPTION':
3407                 if self.description is not None:
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in _add_resource(self, iterator, tag, data, config, pos)
3332         resource = Resource(config=config, pos=pos, **data)
3333         self.resources.append(resource)
-> 3334         resource.parse(self, iterator, config)
3335
3336     def _add_coosys(self, iterator, tag, data, config, pos):
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in parse(self, votable, iterator, config)
3154             if start:
3155                 tag_mapping.get(tag, self._add_unknown_tag)(
-> 3156                     iterator, tag, data, config, pos)
3157             elif tag == 'DESCRIPTION':
3158                 if self.description is not None:
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in _add_table(self, iterator, tag, data, config, pos)
3111         table = Table(self._votable, config=config, pos=pos, **data)
3112         self.tables.append(table)
-> 3113         table.parse(iterator, config)
3114
3115     def _add_info(self, iterator, tag, data, config, pos):
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in parse(self, iterator, config)
2403                             'TABLEDATA', six.iterkeys(data), config, pos)
2404                         self.array = self._parse_tabledata(
-> 2405                             iterator, colnumbers, fields, config)
2406                         break
2407                     elif tag == 'BINARY':
/usr/local/lib/python2.7/site-packages/astropy/io/votable/tree.pyc in _parse_tabledata(self, iterator, colnumbers, fields, config)
2480                 row_mask = mask_default[:]
2481                 i = 0
-> 2482                 for start, tag, data, pos in iterator:
2483                     if start:
2484                         binary = (data.get('encoding', None) == 'base64')
ValueError: 2677:5876: not well-formed (invalid token)`
I am wondering whether this is an issue for my laptop or on the NASA server side?
Thanks,
David

Hi David. I just checked on my computer and cannot reproduce the result. Are you using the latest version? Is it possible you just have a slow connection?

Hi Michael, I just uninstalled 3ML and then re-installed. I checked that the internet speed is fast enough too, but the problem still persist.

Hi,
So the latest build of threeml on the test server worked and passed the test. There must be something else going on.
/J. Michael

On 13 Mar 2019, at 10:19, Hoi-Fung David Yu<EMAIL_ADDRESS>wrote:
Hi Michael, I just uninstalled 3ML and then re-installed. I checked that the internet speed is fast enough too, but the problem still persist.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Michael, I did nothing but it is suddenly working again this afternoon. Strange.
David

Great! It’s likely some local internet configuration gone awry.
/J. Michael

On 15 Mar 2019, at 08:28, Hoi-Fung David Yu<EMAIL_ADDRESS>wrote:
Hi Michael, I did nothing but it is suddenly working again this afternoon. Strange.
David
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.


add MemcacheStats unit test

remove cache pool dependency from MemcacheStats; instead, NewMemcacheStats func takes
a func param to return memcache stats. This change makes MemcacheStats more lightweight
and easy to unit test.
replace bool params in original NewMemcacheStats func by flags: enableMain, enableSlabs
and enableItems.
Pre-allocate memory for MemcacheStats.main, slabs and items. This removes several null checks.
expose refresh freq to let caller decide how frequent to refresh the stats.
replace one log error place by panic in updateSlabsStats func. Since slabsSingleMetrics and
MemcacheStats.slabs should always be in sync (keys presents in slabsSingleMetrics should also
presents in MemcacheStats.slabs, this is guaranteed by publishSlabsStats func), always panic
if a key is found presents in slabsSingleMetrics but not in MemcacheStats.slabs.

LGTM


History bundle entry fullUrl not produced correctly when using ClientIdStrategyEnum.ANY

Describe the bug:
When using client supplied resource ids, the fullUrl of the resources within a history bundle are not correctly generated. For example, the fullUrl produced when doing a GET {baseUrl}/Patient/clientSuppliedId1234/_history is:
{
  "resourceType": "Bundle",
  "id": "03dd2e54-b1c3-470f-8ef8-19a02e358560",
  "meta": {
    "lastUpdated": "2023-07-13T11:18:10.199-04:00"
  },
  "type": "history",
  "total": 1,
  "link": [
    {
      "relation": "self",
      "url": "http://localhost:8080/fhir/Patient/clientSuppliedId1234/_history"
    }
  ],
  "entry": [
    {
      "fullUrl": "Patient/Patient/clientSuppliedId1234",
      "resource": {
         ...
      }
    }
  ]
}

To Reproduce:
Steps to reproduce the behavior:

Clone the hapi-fhir-jpaserver-starter project.
Enable and update the hapi.fhir.client_id_strategy property to ANY.
Start the local server (i.e. mvn clean spring-boot:run -Pboot).
Using a synthetic patient resource produced from synthea, issue a PUT command using the "id" contained in the synthetic patient resource: PUT {baseUrl}/Patient/clientSuppliedId1234
Access the patient resource history: GET {baseUrl}/Patient/clientSuppliedId1234/_history
Note the incorrectly produced fullUrl attribute for the entry within the bundle.

Expected behavior:
The fullUrl should contain a legitimate resource URL which contains the base FHIR URL.
i.e. "fullUrl": "http://localhost:8080/fhir/Patient/clientSuppliedId1234"
Environment:

HAPI FHIR Version: 6.6.0
OS: macOS
Browser: Chrome

The method fetchEntities in the HistoryBuilder class, creates a PersistentIdToForcedIdMap object that results in a map of values like:
1 -> Patient/clientSuppliedId1234
The forced id is qualified with the resource type, however, the remaining logic in fetchEntities doesn't seem to account for that, and thus, the attribute myTransientForcedId is set to this value. That value is later used to create the resource id, and ultimately, the fullUrl seen in the history output.
I'm a little unclear whether there should be logic in fetchEntities to account for that, or whether the method translatePidsToForcedIds in the class IdHelperService should be altered to omit the resource type prefix? Anyone have thoughts on this?


Add date and datetime conversions

Fulfil requires dates and datetimes to be formatted in an object that contains a iso8601 datetime.  This PR adds a conversion class and integrates it into the model to update the domains automatically.

I'll fix the reeks.

We'll need this in query, too: https://github.com/knowndecimal/fulfil/blob/main/lib/fulfil/query.rb#L49-L118

Actually, let's talk through this. We're already doing some of this in another spot.

Figuring out how to fix these codeclimate reeks.  I've got dates in the queries.  Do those look right @cdmwebs?


Cherry pick for HUDI-5007 and HUDI-5228

Change Logs
Cherry pick for HUDI-5007 and HUDI-5228
Impact
Cherry pick for HUDI-5007 and HUDI-5228
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

CI is green


is Piwik a required component to run Wherehows?

I saw errors in the frontend service log:
2018-02-14 23:54:33 ERROR application:177 - Piwik site ID must be an integer
It seems the root cause was that I didn't set values for:
https://github.com/linkedin/WhereHows/blob/master/wherehows-frontend/conf/application.conf#L92-L93
is Piwik a required component to run Wherehows?

It's not required. The UI will function just fine without it.


Add distribution-based forecast types for torch, output layers, tests

cc @kashif
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

awesome! Sorry I couldnt find the time this last week.

awesome! Sorry I couldnt find the time this last week.

awesome! Sorry I couldnt find the time this last week.

@kashif no worries at all! let me know if you can spot anything, I did some minor changes (like extending DistributionOutput instead of IndependentOutput)

awesome! Sorry I couldnt find the time this last week.

@kashif no worries at all! let me know if you can spot anything, I did some minor changes (like extending DistributionOutput instead of IndependentOutput)

Right... the IndependentOutput was something on our side which helped in code reuse when specifying independent multivariate distributions... So all looks good!

Right... the IndependentOutput was something on our side which helped in code reuse when specifying independent multivariate distributions... So all looks good!

Makes sense. I think it's nice to have something like that, but maybe one could wrap a DistributionOutput class (rather than having an additional type) in a similar fashion to how pytorch distributions handle turning event dimensions into batch ones

Makes sense. I think it's nice to have something like that, but maybe one could wrap a DistributionOutput class (rather than having an additional type) in a similar fashion to how pytorch distributions handle turning event dimensions into batch ones

BTW the tests use the script in the workflows to install the torch stuff... perhaps you can change that to use the requirements?

BTW the tests use the script in the workflows to install the torch stuff... perhaps you can change that to use the requirements?

@lostella  ah i remember the issue was that the default pytorch pip will pull in the gpu version on Linux etc. which is ok when using the library but not ok when running the tests CI/CD machines...

@lostella  ah i remember the issue was that the default pytorch pip will pull in the gpu version on Linux etc. which is ok when using the library but not ok when running the tests CI/CD machines...

@mbohlkeschneider @kashif I've consolidated things quite a bit, now the DistributionForecastGenerator is implemented once, and the specific type of DistributionForecast which it generates (from gluonts.mx or gluonts.torch) depends on the type of Distribution it's given (either from gluonts or torch).
Plus I added some tests which I forgot to include earlier today.
@kashif note my comment on the output transformation: I think it doesn't really make sense to apply it, please confirm. I believe the __call__ method of ForecastGenerator types is getting way too much information, and maybe the class could be refactored so that many of those details (e.g. the network) are constructor arguments. But this suggests that a bigger redesign of how Predictor/ForecastGenerator interact is required, so that can go to another time.

@mbohlkeschneider @kashif I've consolidated things quite a bit, now the DistributionForecastGenerator is implemented once, and the specific type of DistributionForecast which it generates (from gluonts.mx or gluonts.torch) depends on the type of Distribution it's given (either from gluonts or torch).
Plus I added some tests which I forgot to include earlier today.
@kashif note my comment on the output transformation: I think it doesn't really make sense to apply it, please confirm. I believe the __call__ method of ForecastGenerator types is getting way too much information, and maybe the class could be refactored so that many of those details (e.g. the network) are constructor arguments. But this suggests that a bigger redesign of how Predictor/ForecastGenerator interact is required, so that can go to another time.

awesome! next we can do a jax backend too 😃

awesome! next we can do a jax backend too 😃


menu: support for external links

Hello, thank you for this fork!
I have a question, how do I create a (reference) link as an item in the left side menu?
I want that item to reflink to a different part of the documentation.
I have tried something like:
+++
menutitle = "Link to other page"
url = "/other-page/"
+++

But it seems that depending on hugo's mood it actually works, while other times "hijacks" the other pages route.
But probably I am not doing it the way it is supposed to? Is there even an easy or non-hacky way to do it?
If not then I would kindly request such feature to be in the theme :)

Without seeing the full example my answer is based on assumptions:
Yout want to have a (probably nested) entry int the site's menu to a page that already exists in a different area of the menu?
That's not possible and I doubt that Hugo is capable of this.
The clostest you can get is, to create a shortcut link. This is always displayed as a flat list below the menu and needs to be defined for each language (if you have a multilang setup).

Hi thank you for the response.
Yes I think you understood me correctly, I would like my documentation to behave like this:
1 Overview
 - Basics
 - Advanced
 - More Advanced
 - Reference (should be just a reflink to actual reference guide ->)

2 Configuration
 - Paragraph 1
 - Paragraph 2
 - Paragraph 3

3 Reference Guide (-> link to this chapter)
 - [...]

So the "1 Overview -> Reference" should just be a reference to the actual chapter further below, because it belongs to thematically  to the Overview/Introduction but is itself a bigger topic that deservers it's own full chapter.
As I said this actually works half-way with the url= param, but unfortunately hugo randomly prioritizes which route is the correct one, each time it generates the pages.

That's not possible and I doubt that Hugo is capable of this.

Hm damn there is no way? I thought maybe an attribute could be added to the pages meta tags, that it is only a reflink with no content?
(Another creative way I came up with, is that the file I create in the folder which should only be a reflink, would be a symlink poiting to the actual file on the file system. This works half way but it does not correct the route unfortunately. Just mentioning it here for the fun though^^)

The clostest you can get is, to create a shortcut link. This is always displayed as a flat list below the menu and needs to be defined for each language (if you have a multilang setup).

I know, I have looked at this, but this would mean the link is under the whole menu? That is unfortunately not what I am looking for, since it should be part of a nested menu.
Any other ideas or is this actually impossible with hugo? :/

@McShelby Apparently I was not the first one to miss such fucntionality, see https://github.com/google/docsy/issues/449
Following the docsy bugreport it seems it was actually implemented with https://github.com/google/docsy/pull/511
I am not that deep into the code, but it seems this is what I am looking for?

I saw you've changed the title. Just be exact, my original bug report was about support for reflinks, while the docsy issue I have linked in my previous post was mainly about external links.
But it seems that in the final PR they've added support for both, in which case it obviously fits the bill^^

Yeah. I just quickly scanned over the changesets and think, if we add this featrue, we should add it this way to satisfy every use case.

Just talking to myself as a reminder:
The basic idea here would be, to add menu entries to the parents frontmatter and later merge those into the list of pages. This could be done in page-meta.hugo and stored in a scratch to be later been used.
https://discourse.gohugo.io/t/adding-a-child-to-a-menu-item/42007
https://gohugo.io/templates/menu-templates/#menu-entries-from-the-pages-front-matter

The Docsy way is reasonable. https://www.docsy.dev/docs/adding-content/navigation/#add-manual-links-to-section-menu
Have to think how arrow nav, print and the children shortcode should behave, but this will be part of a next feature release.

Now available using menuPageRef or menuUrl front matter values.


Add error handling to cads_date_field

@marianayovcheva Yeah, that's the idea. Date error messages are tricky, because if you leave one of the day/month/year fields blank it's no longer generating a valid date.


Kotlinter - configuring excluded files

This is a follow up to the previously raised issue here - https://github.com/jeremymailen/kotlinter-gradle/issues/208
The issue is it is very hard to ignore files baed on the package name. There are 2 problems with this approach:

The package name being ignored could be used in the src code as well. So you might inadvertently ignore code you wrote instead of ignoring just the ones that have been generated.
At times when you are generating code for something such as protobufs, we pull in packages based on what the proto files depend on. Whenever a proto on a new package is pulled in, we need to edit the exclusions list again.

It would be great to do a blanket exclusion of all files in the generated folder.
Would be grateful to have the below config working:
tasks.lintKotlinMain {
  exclude("**/generated/**")
}

Huh, I still haven't understood the root cause (the "the way SourceTask works is matching the package part of the path" part), but I can think of a workaround other than filtering by package:
tasks.named("lintKotlinMain") {
    source = source - fileTree("$buildDir/generated")
}
tasks.named("formatKotlinMain") {
    source = source - fileTree("$buildDir/generated")
}

It worked for me with apollo generated files, maybe it'll work for your case as well? 🤞 This doesn't rely on the exclude mechanism, but removes files under /generated a step earlier

@jeremymailen any updates on this? the above solution did not work for me.

Im facing the same issue
I have my source project and outside my source poject in the build folder, im importing a lot of classed auto generated by a framework.
I cant find any way to ignore those files, in ktlint gradle I do this and it works, but here it doesnt:
filter {
exclude { element ->
element.file.path.contains("generated/")
}
}
Remember this classes and folder is in the build directory, and not in the main source path

Yes the FileTreeElement is probably what you want we do:
it.exclude { fte -> fte.file.absolutePath.contains("/build/") }

Ok, one style is now documented in the readme and it seems like people have ways to target the files they want. Closing for now.

Exclude in Kotlin Gradle:
tasks.withType<LintTask> {
    exclude("**/KickAPI.kt")
}

tasks.withType<FormatTask> {
    exclude("**/KickAPI.kt")
}

Even more DRY:
import org.jmailen.gradle.kotlinter.tasks.ConfigurableKtLintTask

withType<ConfigurableKtLintTask> {
    exclude("generated/")
}


nodePackages.teck-programmer: fix build

Motivation for this change
Was broken recently by an update
Things done

Built on platform(s)

[ ] 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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] 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

[ ] Fits CONTRIBUTING.md.

fixes a build, so I'm going to merge this:
https://github.com/NixOS/nixpkgs/pull/146166

1 package built:
teck-programmer

@GrahamcOfBorg eval


Update JetNews CI

Runs JetNews only :+1: Now let's trigger for another project

Runs both projects correctly :+1:


Add FileMeta for previous timestamp to current timestamp

Marked as PoC to prevent preemptive merging. This is complete working and tested code.
This creates a verifiable history chain. Still waiting for feedback on this idea from various advisors.
Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

I think there is a rebase problem with the number of arguments for EmptyRepo, but otherwise this LGTM.

LGTM after clarifying comments

closing for now, while we decide what to do for append-only history.


Taipy Studio is almost non-existent in the doc

Description
Taipy Studio is not mentioned until the Scenario Management tutorial which comes quite late in the learning journey of Taipy. We should probably add a comment with an image at the beginning of the GUI tutorial so that new users get at least the completion capabilities of Taipy Studio.
Expected change
In the beginning of the GUI tutorial. Add a sentence like "If you have VSCode, you can install Taipy Studio, our VSCode extension that provides completions for Taipy code and allows you to edit data pipelines in a graph based editor"
With an image like:

Why not! Having a GIF is more visual. Taipy Studio is useful for the Taipy GUI when using Markdown (not the Python API). It may be better not to explain this.

A pull request attached to the issue is currently in review. I am removing the "waiting for contributor" label.


Error while running vanilla CRA template

When running the current CRA template from scratch the app doesn't load and I keep getting the error below in the Safe-React app:
ThirdPartyApp: A message was received without message id.
Any idea what's wrong?

Which version of the cra template is used?

The current one 2.0.
I assume the problem comes from running a custom Safe-React app( version 2.15.1) which was not upgraded to the latest 2.17?

what version of the interface are you using?

I have just tested it by running
npx create-react-app my-safe-app-v1 --template @gnosis.pm/cra-template-safe-app

and then the app worked perfectly fine (Loaded, an address is shown with a submit button for dumb tx) on safe-react development branch
I assume you're using the old version of the interface (I guess you're running it locally), the new SDK is not backward compatible with older versions of the interface, so just do
git pull origin master

and it should work :)

Yes the latest version of the sdk is only compatible with the latest version of the interface. The older version of tge sdk will also work with the new interface version.
We recommend using the new sdk as we will not backport features (e.g. being able to route reading rpc calls through the interface, therefore removing the requirment of your own rpc endpoint)

Upgrading the Safe-React app to the latest version fixed the problem! Thanks for the quick responses and awesome work on the sdk.


Delete deprecated pages from redesign

[ ] Download package page (previously reached with the "how-to" link on the package details page)
[ ] Verify package page (merged with upload package)

I'm going to a full pass of the gallery code checking for dead files.


Add logging.properties for travis build

This file is to be used by travis build scripts.

This PR is required by #1108. These 2 PR in combination reduce the amount of logging produced by Travis build.
/cc @lesv

LGTM


wrong type argument commandp, lsp-rename

What did you expect to happen?
lsp-rename should rename the variable name
What actually happened?
received an error: wrong type argument commandp, lsp-rename
Additional details:
config: doom default
keybinding: SPC c r
language: python

This means the :tools lsp module isn't enabled. How to enable Doom modules.

Thanks :lang (python +lsp) is the one that worked!


Allow filtering for targets when building spectral library from existing blib files

I wish you could force diff to work at the function block level. :) It seems we're missing a test for this case. You're wanting to run BlibBuild with a blib input and the -u or -U option to filter for only certain peptides? The test we have uses msms.txt input. Can you add one for blib input? Presumably that one would fail without this change?


Spiderfy at all zoom levels

Is there a way to not cluster the pins, but to have the pins spider and zoom to the max zoom level when clicked? I like having all of the pins visible, but can't click on all of them since there are a few that overlap. Thanks!

I'd like to be able to programmatically disable clustering to fall to spiderfying like in the spiderfier plugin.
Would be great to be able to do this with only your plugin.

Same here. Unfortunately that plugin hasn't been maintained in more than 2.5 years now too.


Fix NumPy float and int

np.float and np.int does not exist in NumPy anymore.
Code changed to use np.float64 and np.int64.

Closing PR, because I failed to read the contribute guidelines about starting from the branch devel


prevent submitUserMessage from showing up in deep-chat

I want to submit a User Message in order to trigger a first response from the chatmodel, I am currently using:
chatElementRef.value.submitUserMessage({
          text: 'the user has entered the session, please greet him with a welcome message',
        });

Is there any way that I can prevent this initial UserMessage to show up in deep-chat, but only display the first response that comes back?

Hi @FranzHell.
The quickest way to not show the initial user message would be to trigger the clearMessages method after you trigger the submitUserMessage method.
Example:
chatElementRef.value.submitUserMessage({
  text: 'the user has entered the session, please greet him with a welcome message',
});
setTimeout(() => {
  chatElementRef.value.clearMessages();
});

An alternative to this would be to trigger the initial call outside of the Deep Chat component and add the AI's response via the addMessage method.
Let me know if this helps. Thanks!

To note, I am also currently working on the ability to edit previous messages styling, which may come in handy in your use-case (but the release date for this will be some time later in the future).
You can track the progress in this issue.


Spring boot 2.0 milestone

Hi, I've started here the support for incoming Spring Boot 2 (current M6 milestone). There are two things to be changed in Redisson to make it compatible. First is RedissonSessionRepository and RedissonSession, because ExpiringSession is removed from Spring Boot 2 - this is currently done in my repo. The second thing is RedissonCacheStatisticsProvider. It looks CacheStatisticsProvider is no longer supported and Spring Boot 2 uses now Micrometer and MeterBinder class for statistics. More info is here. This is what I didn't need for my current project so it's currently just commented out.
Cheers :)

Spring Boot updated to 2.0.0 version in https://github.com/redisson/redisson/issues/1323


Difference Between This Add-on vs Meross LAN?

I've been using the Meross LAN HA add-on along with you custom pairer Android app. Recently I've having a lot of issues with my devices going unavailable. I need to find a more reliable solution and wonder what the difference is with your add-on? It seems like the one I've been using has been highlighted more but if I can't use my devices reliably I don't care which add-on is the popular choice.

Hi @timnolte ,
the Meross LAN integration is an integration explicitly designed for running locally without the need to interact with Meross APIs. This ha-meross-local-broker, instead, is a component that enables another integration - initially designed to work only with Meross Cloud API - to work locally.
For beginners, I'd suggest to start with the Meross LAN integration as it requires less experience to work. The other option, is to use the Meross Homeassistant component together with this addon.
Note: this addon is still in early alpha and is updated less frequently.


feat: implement azure pushsecret ifnotexist updatepolicy via secretEx…

Implementation of the "SecretExists" function for the Azure provider.
Problem Statement
The function was created but not implemented yet.
Related Issue
I did not find a specific issue for this provider, related to #2643
Follows first implementation: #3117
Proposed Changes
I implemented the function for the azure provider.
Checklist

[X] I have read the contribution guidelines
[X] All commits are signed with git commit --signoff
[X] My changes have reasonable test coverage
[X] All tests pass with make test
[X] I ensured my PR is ready for review with make reviewable

/ok-to-test sha=011dc47

/ok-to-test sha=3bd41a7


add require line to the readme

There's not really any reason to change this in our documentation. I'd rather refer to the karma documentation here


Fix home page broken links

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Reviewed only the home-page.html file because the rest of the files are already reviewed under https://github.com/wso2/docs-choreo-dev/pull/712


Use debconf.set state instead of cmd.run

r? @ms2ger

@bors-servo r+

:pushpin: Commit 5f1942d has been approved by larsbergstrom

:hourglass: Testing commit 5f1942d with merge 3c256ee...

:sunny: Test successful - travis


added maven wrapper files to root project

@salemraffa Can you please respond to my review?

will do !

in my project they are updated but it seems I can't push them. Maybe because im on another branch ?


Type use throttle fn

Description
My proposal for Issue https://github.com/streamich/react-use/issues/764.
Side note:  The returned value is T | null, T | undefined would probably be more accurate and would avoid workaround on line 5 (useState<T>() instead of useState<T | null>(null)), but I did not implemented it as it would have been a breaking change.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as before)
[x] Improve typing

Checklist

[ ] Read the Contributing Guide
[ ] Perform a code self-review
[ ] Comment the code, particularly in hard-to-understand areas
[ ] Add documentation
[ ] Add hook's story at Storybook
[ ] Cover changes with tests
[ ] Ensure the test suite passes (yarn test)
[ ] Provide 100% tests coverage
[ ] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[ ] Make sure types are fine (yarn lint:types).

:tada: This PR is included in version 13.6.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to handle cancelled window/showMessageRequest

We have sentry enabled on our language-server code and its capturing an unhandled error:
Request window/showMessageRequest failed with message: Canceled

Stack Trace

There are a lot of instances of this unhandled exception (as its almost always being thrown on initialisation) on sentry and we would like to handle and resolve this properly.
I have traced the request to a notification message we send from the server on connection.onDidChangeConfiguration (which also gets called on connection initialisation), using connection.window.showInformationMessage('some_message')
This is the trace on output console:
[Trace - 17:42:52] Received request 'window/showMessageRequest - (8)'.
Params: {
    "type": 3,
    "message": "DDN CLI path configured successfully.",
    "actions": []
}

I could not see any other trace with a cancelled window/showMessageRequest request. This is the only showMessageRequest that is visible in the VSCode Output channel.
I've tried to wrap the request in a try...catch but with no success, the error still bubbles up on sentry as unhandled :
     try {
        connection.window.showInformationMessage('some_message')
      } catch (err) {
        console.error(err);
      }

From stacktrace it seems like coming from the vscode-jsonrpc package, which is rejecting the promise with error. I guess some handler needs to be installed on the connection object to handle the cancelled request, but could find any resources where something similar is done. Also could not understand from the error message that who is cancelling the request, is it the client or the server.
Any help would be greatly appreciated 🙏.  Thanks.
Version: "vscode-languageserver": "7.0.0"

"vscode-languageserver": "7.0.0" is a fairly old version. Are you seeing the same behavior on the latest released version 9.0.1 together with the latest client.
And does this happen with VS Code as the client or another editor?

VS Code is the client editor we support currently (with vscode-languageclient: 7.0.0).
I'll post the results here after upgrading our client and server versions to the latest

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

Sorry for being late, this got deprioritised internally, leaving my observation if someone faces this in future.
We've upgraded our client ("vscode-languageclient": 9.0.1) and server ("vscode-languageserver": "9.0.1") to the latest versions. Sentry is still capturing the exception:
Request window/showMessageRequest failed with message: Canceled

Sentry seems to capture the error when the user reloads VSCode while a notification message is still visible on the editor.


I can obtain the schema for MySQL, but not for PostgreSQL

Describe the bug
I successfully started the sqlchat project locally, and everything was fine when I configured the MySQL instance connection. However, when I configured the PostgreSQL instance and tried to query data, I found that the prompt on the project interface was empty. It couldn’t retrieve any table structures under the PostgreSQL instance. Strangely, it could retrieve the table structures under the MySQL instance. Does anyone know why this is happening? Thanks!
To reproduce
nothing
Additional context

The problem has been resolved, possibly because my Postgres instance did not set a login password. After setting the password, it can be used now.


Refresh property needs to be update to enum

I've noticed that the following 2 classes have a boolean Refresh property instead of using the Refresh enum:
DeleteByQueryRequest
UpdateByQueryRequest

thanks for reporting @ejsmith. They're both bool in the rest specs

Update by query
-Delete by query

checking if these should be updated.

https://github.com/elastic/elasticsearch/blob/master/modules/reindex/src/main/java/org/elasticsearch/index/reindex/AbstractBaseReindexRestHandler.java#L94
The spec is correct here, thanks for reporting though @ejsmith 👍

Ok, thanks for checking and sorry for the trouble.
"Sending the refresh will update all shards in the index being updated when the request completes. This is different than the Index API’s refresh parameter which causes just the shard that received the new data to be indexed."


Local pinning of chunk

Refer https://github.com/ethersphere/bee/issues/182 for task details

TODO in future PRs

list pagination
exclude pinned chunks in GC
Add file support
Add collection support

@pradovic Thanks for your comments.


Add scale application

Project Abstract

This is a SCALE encoding/decoding library written in C. It has only 1 dependency for UTF8 which is included. (standalone header file). It serializes and deserializes the types of Fixed Ints, Compact Ints, Bools, Options, Enums, Vectors (and Strings), Tuples, and User-Defined Data Structures.

For which grant level are you applying?

[x] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $50,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for > $100k Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied, renamed ( project_name.md) and updated.
[x] A BTC or Ethereum (DAI/USDT) address for the payment of the milestones is provided inside the application.
[x] I have read and acknowledged the terms and conditions.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted.

 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.

Thank you for the application @MatthewDarnell. Could you please:

Merge both milestones into one, as we generally treat testing as being an integral part of a milestone and not a standalone deliverable.
Denominate the cost in USD. The amount in BTC will be determined at the payout date according to the exchange rate. It should also be below $10k for a level 1 grant.
Also commit to inline documentation, which helps us understand your code when evaluating the delivery?

Why is the C++ implementation not suitable as standalone? Could you point out what's missing/incompatible with their approach?

Why is the C++ implementation not suitable as standalone? Could you point out what's missing/incompatible with their approach?

https://github.com/soramitsu/kagome/issues/906
I had issues building it and created an issue, it was explicitly stated that it is not intended for standalone use. Also, the example provided depends on Hunter. In addition, nothing is extern C so there will be name mangling issues.

Ok, understood. That's a shame,  but I guess you will be able to re-use parts of their code? I will be happy to support your application as long as you also provide a cross-implementation test that ensures the cohesiveness with parity's SCALE implementation - something along the lines of the SCALE codec comparison RFP for your C implementation.

Hi @MatthewDarnell
I just wanted to check how the development of M2 is going for you.
Is there any way we can support you?

Hi @MatthewDarnell I just wanted to check how the development of M2 is going for you. Is there any way we can support you?

I don't have the bandwidth to do it at the moment but hope to get to it soon!

Thanks for the update @MatthewDarnell
Can you give me an approximate estimate on when you plan to work on it again and potentially also when you plan to deliver?

@MatthewDarnell please note that we'll need an amendment to update the timeline unless you're planning to submit within the next 2 weeks, since the milestone is now delayed by over 2 months. Otherwise, we'll terminate the grant and you can reapply when you're ready to work on it again.

@MatthewDarnell please note that we'll need an amendment to update the timeline unless you're planning to submit within the next 2 weeks, since the milestone is now delayed by over 2 months. Otherwise, we'll terminate the grant and you can reapply when you're ready to work on it again.

Hi @alxs, thanks for your reply. Unfortunately I can't get to it atm, but I certainly may reapply when I have the ability to work on it. I haven't done much with FFI and am not sure how to programatically capture the output of the Rust parity-scale-codec tests in order to match with cScale, so that will be something I'd need to investigate as well.

Sounds good. I'll open a PR to terminate the grant in that case. From the little I know about Rust's FFI, I think you'd have an easier time doing it the other way around i.e. calling your library from within a Rust project. As I said, feel free to reapply at any time.


Provide RHEL 8 RPM packages

RHEL 8 is supported and the README is updated for how to use the packages we provide.


SW-4390 Batches by Species -> Add / view batch modal updates

Modify 'batch modal' so it can be used with either Species or Nursery as origin
Move accessions, nurseries, species, and sublocations logic to their own hooks and dropdowns
Implement the updated modal in both InventorySeedlingsTable and InventorySeedlingsForNurseryTable

Nick I was testing this in Vercel and when adding a nursery filter, and then removing it, I don't get all the results back, it keeps filtered, but maybe it's because latest changes are not merged?

Hmm yea I am seeing that in Vercel, but unable to reproduce locally. I am going to push another commit to see if the updated version works as expected

@constanzauanini it seems like the latest push to Vercel fixed the bug we were seeing.


Buffer.alloc / Buffer.from

Issue: https://github.com/dominictarr/varstruct/issues/12
@dcousens please check when you will have free time

Why movement to lib/?
I always found this confusing,  and often only to be seen in libraries that are using a transpiler for distribution...

ACK all but https://github.com/dominictarr/varstruct/pull/13/commits/87f6e3deea2396fb6edc57ff846eac44a42c1aba,  and nit on https://github.com/dominictarr/varstruct/pull/13/commits/3d5ee1bcdd7984f5bc88d31d3e7b41ff1b877ce9

Why movement to lib/?
I always found this confusing,  and often only to be seen in libraries that are using a transpiler for distribution...

What you prefer? src? What about libraries where we have c++ bindings and js?

@fanatid I actually just prefer the flat layout where possible.
In this case I suppose src/ would be my preferred... but now i'm not sure either way.
Does standard define anything here? Haha

ACK on lib/

Published as 6.0.0


igraph: 0.9.8 -> 0.9.9

Description of changes
https://github.com/igraph/igraph/releases/tag/0.9.9

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 176261 run on x86_64-linux 1

17 packages built:

hal-hardware-analyzer
igraph
python310Packages.explorerscript
python310Packages.igraph
python310Packages.kmapper
python310Packages.scikit-tda
python310Packages.skytemple-dtef
python310Packages.skytemple-files
python310Packages.skytemple-ssb-debugger
python39Packages.explorerscript
python39Packages.igraph
python39Packages.kmapper
python39Packages.scikit-tda
python39Packages.skytemple-dtef
python39Packages.skytemple-files
python39Packages.skytemple-ssb-debugger
skytemple


feat(*): Bumps our wasmcloud wit to 1.0

This bumps our main wasmcloud:bus package and the package in the runtime crate to 1.0.0 in preparation for release. Also removes the use of the wasmcloud interface in tests where it wasn't needed

Please note that this is stacked on top of #1809 to avoid merge conflicts. I will rebase once the other is merged

This will be ready to go once #1815 is merged. I'll rebase on that just to be sure before merging this


Fix example for postalCode and streetNumber

What type of PR is this?

correction

What this PR does / why we need it:
Fix example for postalCode and streetNumber
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

Hi @GillesInnov35 ,
Is there an issue related to this PR?  Sorry, I judt don't understand why we need to do this changes?
Thanks,
Toshi

hi @ToshiWakayama-KDDI, no there's no issue related to this PR. streetNumber and postalCode are string like all request body attributes. But Swagger Editor tool (or any code editor solution) show those 2 attributes in the example as integer. This is a good remark from one french operator currently implementing the API. I think this will just improve the readability of the yaml file without breaking compatibility with the previous version. Just to add '' in the example.
Thanks
BR
Gilles

Got it.  Thanks very much, @GillesInnov35 !
BR
Toshi


How to add table footer?

Thanks for creating the vuetable-2 component!
Is there any way to add <tfoot> to hold sub totals for the fields?  Or is it possible to put VuetablePagination and VuetablePaginationInfo inside <tfoot>?
Thanks!
Zhu Ming

@blackpuppy Not at the moment. You will have to mimic that using <div> and CSS to achieve that look. The reason is that it is hard to achieve the same look and behavior with tfoot in different CSS frameworks.

No problem.  I will use <div> for now.  Thanks!


Hidden tilemaps are sometimes visible

Godot version
v4.0.alpha.custom_build [e6f00a0f7]
System information
Windows 10, Vulkan Mobile, NVIDIA TI 1200
Issue description
I sometimes need multiple tilemaps and hide some of them. In the latest godot 4 build, hidden tilemaps are shown in-game. Sometimes even in the editor:

A bisect of the recent commits shows, that this started to happen after this merged pull request: #58413
The pull request is not directly related to tilemaps it seems. It's possible, that other things are affected, too.
Steps to reproduce

Open the repro project

Hide all tilemaps except one

Start the game

Only one tilemap should be visible, but all are visible

Save the scene and restart the editor.

Only one tilemap should be visible, but all are visible in the editor.

Minimal reproduction project
godot4-tilemap-visibility-problem.zip

cc @Sauermann

Fixed by https://github.com/godotengine/godot/pull/63248


Safari + Typescript Branch: Panner no sound

It seems that there's been a regression where a Panner node does not work in the typescript branch on safari.
I hear a small pop sound, but the same code works in @next
Example typescript
Example @next

thanks for all your testing of the typescript branch!
From what i can tell, this might be an issue with standardized-audio-context. And it seems like the issue might be fixed in the latest release. Here's a minimal example of just<EMAIL_ADDRESS>(which is what Tone.js is currently using).
If you set it to 23.0.0, it seems to work ok. I'll update the dependency and see if the issue persists in Tone.js

Just switched my local fork to use 23.0.0 and it does indeed seem to work :+1:
Could make a PR but assuming you want to check if there are any other issues

feel free to make a PR if all the tests pass. that’d be helpful to save me a little time.

closed by #576


Regression in date of birth formatting (EXPOSUREAPP-9297)

Addresses EXPOSUREAPP-9297
The dob formatting now is 03/20/1981 instead of 1981/03/20.

@marcauberer / @mtwalli
It seems that PR https://github.com/corona-warn-app/cwa-app-android/pull/3988 "Booster Vaccination (EXPOSUREAPP-8718)" which is a major feature commit from feature/8718-booster-vaccination has broken PersonDetailsFragmentTest.kt in release/2.10.x and (since this is also merged into fix/dob-regression) in https://github.com/corona-warn-app/cwa-app-android/pull/4036 as well.
Test results attached:
Test Results - PersonDetailsFragmen___.zip

@marcauberer / @mtwalli
It seems that PR #3988 "Booster Vaccination (EXPOSUREAPP-8718)" which is a major feature commit from feature/8718-booster-vaccination has broken PersonDetailsFragmentTest.kt in release/2.10.x and (since this is also merged into fix/dob-regression) in #4036 as well.
Test results attached:
Test Results - PersonDetailsFragmen___.zip

@MikeMcC399  this issue should be fixed by: https://github.com/corona-warn-app/cwa-app-android/pull/4041

@mtwalli

It seems that PR #3988 "Booster Vaccination (EXPOSUREAPP-8718)" which is a major feature commit from feature/8718-booster-vaccination has broken PersonDetailsFragmentTest.kt in release/2.10.x and (since this is also merged into fix/dob-regression) in #4036 as well.
Test results attached:
Test Results - PersonDetailsFragmen___.zip

@MikeMcC399 this issue should be fixed by: #4041

#4041 does fix the issue with PersonDetailsFragmentTest.kt I mentioned, however when trying to run the full androidTest set to generate a complete set of screenshots, the tests now fail at RecoveryCertificateDetailFragmentTest.kt instead. There may be other tests which would also fail, so fixing test modules should continue until all tests run successfully.

@mtwalli

It seems that PR #3988 "Booster Vaccination (EXPOSUREAPP-8718)" which is a major feature commit from feature/8718-booster-vaccination has broken PersonDetailsFragmentTest.kt in release/2.10.x and (since this is also merged into fix/dob-regression) in #4036 as well.
Test results attached:
Test Results - PersonDetailsFragmen___.zip

@MikeMcC399 this issue should be fixed by: #4041

#4041 does fix the issue with PersonDetailsFragmentTest.kt I mentioned, however when trying to run the full androidTest set to generate a complete set of screenshots, the tests now fail at RecoveryCertificateDetailFragmentTest.kt instead. There may be other tests which would also fail, so fixing test modules should continue until all tests run successfully.

@MikeMcC399  I will check them, but it does not prevent you from reviewing this issue here ,right?
Can you verify the fix here is covering the issue of formatting the date ?

@mtwalli

I will check them, but it does not prevent you from reviewing this issue here ,right?
Can you verify the fix here is covering the issue of formatting the date ?

I was intending to verify the fix by generating a complete set of screenshots and then checking the dates in all the screenshots.
I can't do that unless screenshot generation works properly. The failing RecoveryCertificateDetailFragmentTest.kt for instance generates a screenshot which includes a date of birth, which I can't therefore check.

@mtwalli
Would you prefer me to open a separate issue about the failure to generate screenshots in the current release/2.10.x branch so that it can be handled separately?

Would you prefer me to open a separate issue about the failure to generate screenshots in the current release/2.10.x branch so that it can be handled separately?

@MikeMcC399  No need for issue , here is a PR https://github.com/corona-warn-app/cwa-app-android/pull/4043


add typescript typings

Hey there. Thanks for the awesome library. It's saving us a lot of time and guesswork. I'm currently using it with my own declarations in my project. I thought it'd be a good idea to send you PR to add a declaration file into this package so you can publish types alongside the code.

Coverage remained the same at 96.104% when pulling 7ae68114e86e213b6572ce611cbddedd75999440 on disintegrator:typescript-declarations into 4630d1961dbf65772fc957bb36a38cbfcb072318 on fridays:master.

Awesome! I never used typescript but several people asked for this, thank you!

yay! thank you @fridays :)
Please feel free to ping me if you make changes to the API. I can try to keep the typings up to date.

Great, thanks for the support!

@disintegrator Considering that the default server.js needs to use require, not import, statements and therefore routes.ts also would not be allowed to use import statements, are you using type assertions for routes.js imports in your client code?
An example project would be helpful. Thanks!

@smkhalsa  server.js doesn't need to use require. there's an example in the next repo that uses a separate tsconfig.json to build server.ts, then run the app
@disintegrator  is the default export typing correct? the default export line  is module.exports = opts => new Routes(opts), but the default export type is a export default class Routes implements Registry {...
I'm still new w/ TS, but only asking b/c my IDE is yelling at me about this code:
import makeRoutes from 'next-routes';
const routes = makeRoutes({ Link });

yields:
Value of type 'typeof Routes' is not callable. Did you mean to include 'new'?

☝ like @brandonmp said, the typing file doesn't appear to be right here.  This library deafult export is returning a new instance of the class... not the class itself.

@KevinBusch a pull request has been created (https://github.com/fridays/next-routes/pull/333).

@disintegrator How should i implement your changes them in my project??? using next-routes and next v8.1 .


ghrepo-stats: 0.5.3 -> 0.5.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mrbean-bremen/ghrepo-stats/releases.
meta.description for ghrepo-stats is: Python module and CLI tool for GitHub repo statistics
meta.homepage for ghrepo-stats is: https://github.com/mrbean-bremen/ghrepo-stats
meta.changelog for ghrepo-stats is: https://github.com/mrbean-bremen/ghrepo-stats/blob/v0.5.4/CHANGES.md
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.4 with grep in /nix/store/xa539vs0zwnd0j2zajg9n0w18438nzcm-python3.10-ghrepo-stats-0.5.4
found 0.5.4 in filename of file in /nix/store/xa539vs0zwnd0j2zajg9n0w18438nzcm-python3.10-ghrepo-stats-0.5.4

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
ghrepo-stats
python310Packages.ghrepo-stats
python311Packages.ghrepo-stats

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xa539vs0zwnd0j2zajg9n0w18438nzcm-python3.10-ghrepo-stats-0.5.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ghrepo-stats https://github.com/r-ryantm/nixpkgs/archive/997353a46fab3ec69dfd58f94e4f81f48658e092.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xa539vs0zwnd0j2zajg9n0w18438nzcm-python3.10-ghrepo-stats-0.5.4
ls -la /nix/store/xa539vs0zwnd0j2zajg9n0w18438nzcm-python3.10-ghrepo-stats-0.5.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

ghrepo-stats (python310Packages.ghrepo-stats)
python311Packages.ghrepo-stats

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 221309 run on aarch64-darwin 1

2 packages built:

ghrepo-stats (python310Packages.ghrepo-stats)
python311Packages.ghrepo-stats

Result of nixpkgs-review pr 221309 run on aarch64-darwin 1

2 packages built:

ghrepo-stats (python310Packages.ghrepo-stats)
python311Packages.ghrepo-stats


marimo edit hangs

Describe the bug
I exec marimo edit --headless on the Ubuntu server, but nothing was printed.
it seems to hang. I can not connect it in browse or vscode.

marimo tutorial and marimo run can exec right.
I tried to install 0.6.26 0.7.0 and 0.7.1, got the same problem.
I tried to enable the DEBUG logger but still nothing was printed.

Environment
{
"marimo": "0.6.26",
"OS": "Linux",
"OS Version": "5.4.0-186-generic",
"Processor": "x86_64",
"Python Version": "3.11.0",
"Binaries": {
"Browser": "--",
"Node": "--"
},
"Requirements": {
"click": "8.1.7",
"importlib-resources": "missing",
"jedi": "0.19.1",
"markdown": "3.6",
"pymdown-extensions": "10.8.1",
"pygments": "2.18.0",
"tomlkit": "0.12.5",
"uvicorn": "0.30.1",
"starlette": "0.37.2",
"websockets": "12.0",
"typing-extensions": "4.12.2",
"black": "24.4.2"
}
}
Code to reproduce
No response

Thanks for reporting. I tried to reproduce but wasn't able to yet.
Does marimo edit main.py without headless work?

marimo edit main.py

It does not work.

When I finished eating (about 2 hours later), the terminal showed localhost contacted. waiting for reply...
After I press command + c, the program works as shown:

It's working

Now marimo edit is working correctly.
That's weird.

That is odd - @FrostMiKu is this good to close?

That is odd - @FrostMiKu is this good to close?

yep


More definition for text boxes in dark theme

Is your feature request related to a problem? Please describe.
When I use the dark theme, I find it hard to see where there is text I can edit or fields that need to be filled in. It's easy to miss a field.
In this example, I find it difficult to see how to change the name of the file or the timestamp format.

In this one, it's easy to miss the port

I didn't see anything about this when searching online and I reset my colors in case there was something I did on my end.
Describe the solution you'd like
Can you make these fields have a different color or have a border that makes it easier to discern that the field is editable?

Hello @rjake,
Sorry for the late response.
Unfortunately, I believe that it's the issue of Eclipse.
Have you tried the Darkest Dark theme? https://marketplace.eclipse.org/content/darkest-dark-theme-devstyle

There have been no updates on the issue for a long time.
If it is still actual - feel free to reopen it with additional info


Does not load custom snippets

I cloned the lazy starter template from lazyvim.org with this cmd:
git clone https://github.com/LazyVim/starter ~/.config/nvim
in plugins folder, I created a file called snippets.lua, inside the file I copied the config:
return {
  "L3MON4D3/LuaSnip",
  -- follow latest release.
  version = "v2.*", -- Replace <CurrentMajor> by the latest released major (first number of latest release)
  -- install jsregexp (optional!).
  build = "make install_jsregexp",
}

then, I create a file ~/.config/nvim/my_snippets/snippets/all.json and in init.lua, I loaded with the following cmd:
require("luasnip.loaders.from_vscode").lazy_load({ paths = { "~/.config/nvim/lua/custom/snippets/" } })
but this doesn't work, when I use  LuaSnipListAvailable  in neovim, it shows like
{
all = {},
lua = {}
}

below is my full configuration:
https://github.com/1057437122/luasnipnotwork.git

You need a ~/.config/nvim/my_snippets/snippets/package.json which lists the snippet-file all.json.
{
	"name": "test-snippets",
	"contributes": {
		"snippets": [
			{
				"language": [
					"all"
				],
				"path": "./all.json"
			}
		]
	}
}

Should work I think :)

Thank you so much! Now, when I use LuaSnipListXXX cmd, it lists all the data, look ,there's a cbt snip

but it doesn't work when type in file, I typed c , what I expected is cbt but it doesn't show:

Do you have cmp_luasnip? The snippets are not automatically listed by nvim-cmp (if you use nvim-cmp), you need a source and have to activate it

now it works ! Thanks a lot. But I wanna say, I used this plugin maybe a year ago since lazyvim 3, the config before was really easy, the new version is a little complicated and there is not a clear document says what I should do when do the upgrading, however, thanks for your patience and help.


HTTP header name verification should be case insensitive

Context
Use yaks HttpClientSteps to assert HTTP headers and body in response on different test environments.
Problem
On some of our test environments HTTP headers are modified by an intermediate system.
In our case we use OpenShift Cluster with ingress, which converts all HTTP header names to lower case.
According to RFC 2616 - "Hypertext Transfer Protocol -- HTTP/1.1", Section 4.2, "Message Headers", the HTTP header names are case insensitive. But Yaks while verifying the headers does not consider the case insensivity. This makes it hard to write environment independent tests.
Another problem is, that the the step And expect HTTP response body is always ignored if the step Then expect HTTP response header Content-Type="..." is not defined.
See HttpClientSteps.java#L191
So we can also not verify response body at all because header verification always fails.
Example:
The following scenario fails on some environment in step Then expect HTTP response header Content-Type="application/xml;charset=UTF-8".

Header expected: Content-Type="application/xml;charset=UTF-8"
Header received: content-type="application/xml;charset=UTF-8"

  Scenario: HTTP response headers are case sensitive

    Given URL: http://server
    And HTTP request header Content-Type="application/json"
    And HTTP request body
    """
    {"hello": "world"}
    """
    When send POST /api/test
    Then expect HTTP response header Content-Type="application/xml;charset=UTF-8"
    And expect HTTP response body
    """
    <errorResponse>
      <errorCode>123</errorCode>
    </errorResponse>
    """
    And receive HTTP 400 BAD_REQUEST

What I wish
It would be great to have the following features in Yaks / Citrusframework

Verify received Header where header names are case insensitive
Verify received Body without verifying the header Content-Type

Citrus already provides a setting to enable case insensitive header validation. We just need to make this settable via  Cucumber step in YAKS.
I am not aware of any mandatory Content-Type header validation. There must be something else going wrong in your scenario.


Not Supported 10.0.19042.1526

After Running windows updates.....not supported

https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini

Your version is supported. https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini

thank you.I will try

Your version is supported. https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini

it works well on V19042.1526. Thanks

Duplicate of https://github.com/stascorp/rdpwrap/issues/1697