Verify If It's Possible to Deploy Pachyderm Server and Created Teardown

For Teardown I have created this keyword. And in similar fashion of Aikit test file. And for Teardown it's necessary.

Jenkins Job : https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-ci-pr-test/609/robot/report/files/log.html
Result : Passed

lgtm, other than some small comments


Phase Tracker Problem

Tested out the module today - and ran into an issue that made the module unusable for us.
When the Gamemaster changed phases, the phase tracker would not update unless each player refreshed the page. Otherwise, seems to work.

Very sorry about that. It seems I broke this by accident in v2.0.0. Thank you so much for the report. It should be fixed in the next release :)


Wrong Event machine dependencies

A lot of Ruby applications never work within Event Machine.
This dependency should be totally extracted into other classes/mixins.

A little later.
And this is mistake for a lot of ruby applications :)

:)

"Little late" is now! Please, try it.


Paragraph + list (stability), variation 2

Another edge case of #63, almost the same but a little different.
Before:
/**
 * Some title
 *
 * <p>1. Test
 * 2. Test
 */
fun foo() = Unit

After:
/**
 * Some title
 *
 * 1. Test
 * 2. Test
 */
fun foo() = Unit

After again:
/**
 * Some title
 * 1. Test
 * 2. Test
 */
fun foo() = Unit

Using v1.5.4 with default options.

Fixed in 1.5.5


Failed to load properly

I get a Playnite notification saying "SuccessStory failed to load properly" using last update (2.0). I also see that text when checking the Settings menu:

I'm using Playnite's last update and I've got no other plugging installed.

I can't reproducte this issue.
Can you post your playnite.log?

Here you are. Thanks for the help!
playnite.log

Can you try with the new version (v2.1)?

Do you use Playnite 8?
Beacause the versions 2.x are Playnite 9 only. It's in changelog.

Aagh, that's it, my bad, I was focused on working around issue #142 and I didn't see the changelog, sorry.


Purge Version Hints

Add a compatibility version attribute to actions which can be used to purge the local client state (instead of the constant version flag in the sourcecode which purges the local state even if no incompatible actions have been submitted to the stream)

Naming:

App-Version: Compatibility of the complete state on the client (including credentials and stuff). Every user would have to log in again (constant of the client code)

Major Version(formerly known as Reducer Version): All actions from the beginning of time have to be reduced again because the handling of old actions has changed (e.g. 200 instead of 100 recent items are saved in the state) (constant of the client code)

Minor Version or Compatibility Version: All actions from the beginning of time have to be reduced again but only if there already are some of the new actions in the stream (constant property of an action, and the max of the properties as global constant of the client code)

client: If action provides apiVersion property and compatibilityVersion property in Higher Order Reducer is undefined or smaller than apiVersion property, update the compatibilityVersion property in the state

client: Send the compatibilityVersion if available along to the server with each request

server: If a compatibilityVersion is part of an API payload and the compatibilityVersion of the stream is undefined or smaller than the compatibilityVersion property of the API payload, update the compatibilityVersion of the stream

server: If a version is part of an API payload and the version of the stream is undefined or smaller than the version property of the API payload, update the version of the stream and reset the compatibilityVersion

server: Send the compatibilityVersion as part of each API response

client: If the compatibilityVersion in an API response is bigger than the local global compatibilityVersion, set property in state "purgeOnCompatibilityVersion": compatibilityVersion of the API response

client: if purgeOnCompatibilityVersion is smaller or equal the local global compatibilityVersion, purge local state and resync

Solved much easier together with snapshots (clientside rocks)


Remove deprecated runner on README

Removed pronto-bundler_audit from runners list on Readme. It is a deprecated runner (as said here: https://github.com/pdobb/pronto-bundler_audit#maintainer-needed) that isn't working.

Thanks for the contribution, @MatheusPoliCamilo! In case @pdobb or any maintainer does make it compatible, I'd be fine with adding it back.


Not able to produce Jmeter HTML Report in Release Pipeline with this extension & failed with error

I am using Jmeter for performance load testing in Azure Release pipelines, I am using this task (PublishHTMLReport) but somehow it always get failed at this task with below error:
2021-03-26T01:26:37.5513930Z ##[error]Unable to process command '##vso[task.addattachment type=firstscriptname;name=content;]$(Build.ArtifactStagingDirectory)/HTMLReports/content/js/dashboard-commons.js' successfully. Please reference documentation (http://go.microsoft.com/fwlink/?LinkId=817296)
2021-03-26T01:26:37.5526204Z ##[error]Value cannot be null. (Parameter 'Cannot upload task attachment file, attachment file location is not specified or attachment file not exist on disk'.
I did echo on this directory $(Build.ArtifactStagingDirectory) and $(system.DefaultWorkignDirectory) Directory of D:\a\r1\a\HTMLReports  .  ..  content 9,670 index.html  sbadmin2-1.0.7 2,998 statistics.json 2 File(s) 12,668 bytes 4 Dir(s) 14,701,731,840 bytes free.
Index.html is getting generated under the directory but somehow this HTML report extension does not work at all. I have tried few different path values in input section such as:
htmlType: 'Jmeter'
JmeterReportsPath: '$(Build.SourcesDirectory)/LoadReports'
nothing is working.
Somebody know any alternate method of producing Jmeter HTML report as tab within Azure DevOps.
Many Thanks

Can you let me know:

The task how you have used it.
Jmeter version you are using
ls -lrt output on LoadReports folder.

The following tasks in Release pipeline are as below:

Jmeter task version: 5.2.1

Command Line Script:
echo 'Jmeter............'
jmeter -n -t _JmeterTest-AG\JmeterWebApp.jmx -l _JmeterTest-AG\Summary.jtl -e -o HTMLReports

Publish HTML Report Task (your extension)
htmlType: Jmeter
JmeterReportsPath: $(System.DefaultWorkingDirectory)/HTMLReports/index.html

That is all I have.
I did not get you when you said -Irt output on LoadReports folder. I have tried many different path values, nothing is working

Come on gitter. https://gitter.im/PublishHTMLReports/community?utm_source=share-link&utm_medium=link&utm_campaign=share-link

Release pipeline is not supported yet, this is a duplicate issue. Closing this


Fix for incorrect type and size calculated in ctstring_test on s390x

The OffsetType test fails on s390x as it returns 0 for type/size. This is happening because TF_TString_ToInternalSizeT returns 8 bytes(size_t is 8 bytes on s390x), however offset type has "size" which is uint32. Thus the data containing 'type' information is lost. This further causes wrong 'size' calculation here.
The mark and other calculations work for Large type strings as the 'size' is 8bytes.
Fixes #56046. More details in that issue.

@gharibian could you please take a look?

@gbaned @gharibian Could you please check?

Looks good, but can you squash this into a single commit?

Looks good, but can you squash this into a single commit?

Done @gharibian , Apologies I missed it earlier.

Sorry for the inconvenience, but could please update the commit description to reflect what it currently is, i.e. this is now a fix for TF_TString_GetDataPointer method for the OffsetType, and fix for TEST(TF_CTStringTest, OffsetType)?

Sorry for the inconvenience, but could please update the commit description to reflect what it currently is, i.e. this is now a fix for TF_TString_GetDataPointer method for the OffsetType, and fix for TEST(TF_CTStringTest, OffsetType)?

Updated the description. Thank you.


Update Front End UI Year Validation for Facility Attributes

[ ]Need to update the Front End Validation to allow for facility attributes to accept the current year.
[ ]

tlageman (2/14/2024) - I believe this issue relates to the current facility attributes year filter which doesn't accept the current year as valid input if the date on which it is inputted is before the first quarter. For example, if a user before end of first quarter of 2024, tries to input the year 2024 into the facility attributes, an error will be thrown. This hasn't been a huge issue since people most likely won't need 2024 facility attributes data until 1st quarter emissions is reported, but the system should allow any year (XXXX) to be inputted into the year filter beginning January 1st, XXXX. See ticket #4656

Changed this priority to low because this is a seasonal issue and there are available work arounds.


first PublishData always results in an error

Hi!
Thx for your great work with livekit. It's a really awesome framework and very easy to use!
However I'm getting an error when sending some data
room.Callback.OnParticipantConnected = func(p *lksdk.RemoteParticipant) {
  if err := room.LocalParticipant.PublishData([]byte("hello"), livekit.DataPacket_RELIABLE, []string{p.SID()}); err != nil {
    log.Printf("could not publish data to participant '%s', err: '%v'", p.SID(), err)
  }
}

After a few SECONDS this call results in the error
could not publish data to participant 'PA_kCS2E6r2YV3c', err: 'could not connect after timeout'

Immediately after that log entry, the following log entry occurs:
"level"=0 "msg"="successfully set publisher answer"

After that log entry I'm able to successfully send data messages. It seems to me that the first call to PublishData blocks some logic.
If I try to send with a go routine like this
	room.Callback.OnParticipantConnected = func(p *lksdk.RemoteParticipant) {
		log.Printf("OnParticipantConnected '%s'", p.Identity())
		go func() {
			time.Sleep(time.Second * 1)
			sendData(p)
			time.Sleep(time.Second * 1)
			sendData(p)
		}()
	}

the first send results in the error
could not publish data to participant 'PA_73nWKR3avB7X', err: 'io: read/write on closed pipe'

the second send succeeds.
In conclusion: the first PublishData always fails. Afterwards it works.
I'm not sure if it matters but the other participant is using the js-sdk
Any help would be greatly appreciated  :smile:

since publisher is not establish until you publish something(mediatrack or data), so you publish data in OnParticipantConnected will cause publisher pc to connect and block the signal msg process loop, that's why you get timeout.
If you publish data at goroutine, the publisher pc will connected then open the data channel, the publishdata method just wait publish pc connected but data channel may still opening, so your first pub might get closed pipe.
There are two points to improve:

we might need make callback in goroutine to avoid user do something block our signal loop @davidzhao
create datachannel after publish pc connected, that might solve your first pub failed problem

create datachannel after publish pc connected, that might solve your first pub failed problem

I'm not sure if I understand this correctly. Can you give me some additional hints how I might create datachannel after publish pc connected. What Callback is getting triggered for pc connected?

we might need make callback in goroutine to avoid user do something block our signal loop @davidzhao
create datachannel after publish pc connected, that might solve your first pub failed problem

Agreed with both points! Our other client SDKs will block publishData until publish PC is established.

@qwertzui11  made pr #50 to fix this, now you can pub data in OnParticipantConnected callback. If you verify all things ok, I'll close this issue

Hi! thx for the patch!
Sadly, no, it does not work. There is no error any more, however the first package is not getting transmitted. The package seems to silently get dropped.
So it does NOT work inside room.Callback.OnParticipantConnected. There is no error, however the message gets silently dropped.
The other part is using the javascript sdk. according to chrome://webrtc-internals/ no message is getting transmitted to the browser. So I assume the go-sdk is at fault.

I can receive the first datapacket by use livekit/client-sdk-js's sample app as receiver. send code is like
	room.Callback.OnParticipantConnected = func(rp *lksdk.RemoteParticipant) {
		fmt.Println("conn", rp.Identity())
		err := room.LocalParticipant.PublishData([]byte("123"), livekit.DataPacket_RELIABLE, []string{})
		if err != nil {
			fmt.Println(err)
			panic(err)
		} else {
			fmt.Println("published data")
		}
	}

Hi, thx for getting back to me!
I debugged my code and actually, everything seems to work correctly. The Message is getting successfully send on the reliableDC
My code looks like yours, however I'm sending the message to the newly connected participant ONLY.
err := room.LocalParticipant.PublishData([]byte("123"), livekit.DataPacket_RELIABLE, []string{rp.SID()})

When I broadcast the message to all participants, it actually works! ([]string{}). But when I send it to the newly connected participant the message gets dropped ([]string{rp.SID()}).
Due to the lack of errors, I assume it's getting dropped on the livekit-server?
I checked the debug error logs of the livekit-server, but there seems to be nothing note worthy.
Thx for your time!
Cheers

@qwertzui11 which version of livekit-server are you running?

@davidzhao
root@sfu ~/s/livekit (master)# docker run --rm livekit/livekit-server --version
livekit-server version 0.15.4

I see, version 0.15.5 is available, I'll re-test within the next 12 hours.

I have test with 0.15.4,0.15.5 and master @livekit-server, all of those release can work correctly with send data to special participant
room.LocalParticipant.PublishData([]byte("123"), livekit.DataPacket_RELIABLE, []string{rp.SID()})

I narrowed the error down to a small project available at https://gitlab.com/qwertzui11/livekit-server-sdk-go-issue-49
the error seems to occur when the livekit-server is run without redis
whenever a client leaves the room and then joins again, the first data message is getting dropped. however when everyone leaves the room, and join again, the messages are getting transmitted successfully.
i guess this issue succeeds this issue?
Thanks for your time

@qwertzui11 thank you for the repro
I'm able to reproduce this as well.. It's not clear to me why it's happening though. there seems to be some leftover state here.
It seems unrelated to #83.

Hi, upgraded livekit-server to master and server-sdk-go to main. Then re-tested, and success!!!! :heart_eyes:
Awesome work! Thanks a lot!


Update mobile-appconfig.rst

@jasonblais Made tweaks to it, this is ready for final review.

@amyblais Thanks! Looks good, just wondering if the Adding Mattermost mobile apps to your EMM provider section is also mentioned in the opening paragraphs.

@jasonblais Done


add an importFromMSE button to midas

fixed here : https://github.com/moosetechnology/Midas/commit/b5b58a21f937bb3b70ec7dcb749660dfacf454b5


[PE-D][Tester B] Edit command description is misleading

edit command here is misleading as it suggest that "This does not deal with user appointments"

However, after editing "Alex Yeoh" sample data to "Alexy" and running the findappt command, the "findappt Alexy", there is a match for an appointment with the name "Alexy".

Labels: type.FeatureFlaw severity.Low
original: Zack-Tay/ped#10

This is the intended behavior. It is clear that appointments are linked to their person. We will update the UG accordingly.


Build fails on esp-idf v4.0-dev-512 on Mac OSX (IDFGH-1144)

Environment

Development Kit:    (n/a for this issue, but typically ESP32-DevKitC-v4)
IDF version: v4.0-dev-512-g6fd535c
Build System:         Make
Compiler version: xtensa-esp32-elf-gcc (crosstool-NG crosstool-ng-1.22.0-80-g6c4433a) 5.2.0
Operating System:     macOSX v10.11.6 El Capitan
Power Supply:        (n/a for this issue, but using USB)

Problem Description
"make flash" fails using IDF versions v4.0-dev-512-g6fd535c and esp-idf-v4.0-dev-459-gba1ff16.  Everything works fine with v4.0-dev-141-g106dc05-dirty, but after updating to esp-idf-v4.0-dev-459, and then to v4.0-dev-512, the build process fails during compilation of even simple programs like the hello_world example.
Steps to Reproduce

Moved old version of esp-idf to backup directory and downloaded latest version of esp-idf using $ git clone --recursive https://github.com/espressif/esp-idf.git

Attempted to make a simple example.

$ cd esp-idf
$ git describe --tags --dirty
v4.0-dev-512-g6fd535c-dirty

$ cd examples/get-started/hello_world/
$ make menuconfig

Python requirements from /Users/username/esp/esp-idf/requirements.txt are satisfied.
DEFCONFIG
MENUCONFIG
configuration written to /Users/username/esp/esp-idf/examples/get-started/hello_world/sdkconfig

*** End of the configuration.
*** Execute 'make' to start the build or try 'make help'.

GENCONFIG
App "hello-world" version: v4.0-dev-512-g6fd535c-dirty

$ make flash monitor
Toolchain path: /Users/username/esp/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: crosstool-ng-1.22.0-80-g6c4433a
Compiler version: 5.2.0
Python requirements from /Users/username/esp/esp-idf/requirements.txt are satisfied.
  Stop.
make[1]: *** [component-soc-build] Error 2
make: *** [/Users/username/esp/esp-idf/examples/get-started/hello_world/build/bootloader/bootloader.bin] Error 2

try make clean

Thank you for your suggestion, yene.  Unfortunately, the problem still occurs after running make clean.  I was finally able to get through the compilation process by ignoring the errors and running make flash three times in succession. See below for abbreviated output.  I re-downloaded the latest esp-idf and started over from within the new examples directory, but it didn't help.
There was some discussion about needing to run make clean in issue 712 , but it's not clear to me that those solutions apply to this case.  I've updated esp-idf versions before and did not have any problems.
$ cd /Users/user/esp/esp-idf/examples/get-started/hello_world
$ make clean       # works fine
$ make menuconfig  # works fine

$ make flash       # fails at CC build/bootloader/soc/esp32/gpio_periph.o
Python requirements from /Users/user/esp/esp-idf/requirements.txt are satisfied.
GENCONFIG
App "hello-world" version: v4.0-dev-512-g6fd535c
...
make[2]: *** No rule to make target `esp32/gpio_periph.o', needed by `libsoc.a'.  Stop.
make[1]: *** [component-soc-build] Error 2
make: *** [/Users/user/esp/esp-idf/examples/get-started/hello_world/build/bootloader/bootloader.bin] Error 2

$ make flash       # now fails at CC build/soc/esp32/gpio_periph.o
...
make[1]: *** No rule to make target `esp32/gpio_periph.o', needed by `libsoc.a'.  Stop.
make: *** [component-soc-build] Error 2

$ make flash       # finally succeeds and successfully flashes

master seems broken, try latest release tag

cd ~/esp/esp-idf
git checkout tags/v3.1.4
git clean -df
git submodule update --recursive

As noted in my original post, everything up through v4.0-dev-141-g106dc05-dirty works fine.  Just the last couple development versions are not working.  So, yes, v3.1.4 works as well.  I can develop on earlier versions of 4.0, but I wanted to note this problem now so that (hopefully) it gets fixed and I can develop using the latest version.  There are some new features that I would like to use.  Again, thank you for your suggestions and help on this issue.

I am also seeing this same issue. It appears to be a missing dependency in the make files of some sort.

Hi @0cca
Thanks for reporting this. I can reproduce this issue on mac with make3.81. To resolve this you can use CMake build system or update you GNU make (4.0 works fine).
Interestingly removing the trailing / in https://github.com/espressif/esp-idf/blob/master/components/soc/component.mk#L3 seems to solve the issue with make 3.81 for me.

Since there is an identified fix that does not involve changing the build system or make version, can that be pushed to the repo please?

@david-cermak, thank you.  I confirmed that it works using CMake.  I'll check GNU make at a later time, but I'm up and running now.

@bpietsch of course it will. Just keep this issue open so it can be tracked and closed upon merging the fix.
Have just noted CMake and new version of GNU make to encourage users to employ up to date tools where chances to fail are lower:)


[MB-5079] Add db migrations for webhook feature

Description
Adding some new values to the enum and new columns - you can read more about the use of the columns here:
Design document - Retry Mechanism section
Reviewer Notes
I dropped and recreated an index. I had to move it to its own migration to avoid transaction issues.
@monfresh mentioned I should add "concurrently" but again that needs to be done outside a transaction and our migrations are setup to be done in a transaction. So it is not concurrent but since there is minimal activity on production right now, I hope this is ok.
Relevant slack conversation.
Setup
Two tests updated in the models folder:
go test ./pkg/models --testify.m TestWebhookSubscription_Instantiation -v
go test ./pkg/models --testify.m TestWebhookNotification -v

Code Review Verification Steps

Any new migrations/schema changes:

[x] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[x] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs - Not Applicable

References

[ MB-3627 Push Enable: Notifications framework should have retry mechanism - Jira](https://dp3.atlassian.net/browse/MB-3627)
Design document - Retry Mechanism section

Messages

:book:
:link: MB-5079

  Generated by :no_entry_sign: dangerJS against 5efae171fd5b9d7be89df75ddd60d493421a030b


Clang Format: remove unused values.

None of the options are effective, as can be observed by the lack of any changes after removal.

[approve ci autest]

No - if you change them, it has no effect. It seems more like a trap than useful.

No - if you change them, it has no effect. It seems more like a trap than useful.

Oh, that's interesting. I had wondered that when I first saw this, but I confirmed that those configurations exist here:
https://clang.llvm.org/docs/ClangFormatStyleOptions.html#bracewrapping
Now that you mention that the section isn't doing anything, I wonder whether this has to do with this:

BraceWrapping (BraceWrappingFlags) clang-format 3.8
Control of individual brace wrapping cases.
If BreakBeforeBraces is set to BS_Custom, use this to specify how each individual brace case should be handled. Otherwise, this is ignored.

It could be you have discovered a bug of sorts in our clang format file. I bet we need to add this to the configuration:
BreakBeforeBraces: Custom  # <----
BraceWrapping:
  AfterCaseLabel:  false
  ...


feat: add open graph package

Task: Add social tags [developer.aiven.io]
developer.aiven.io pages are lacking OpenGraph Tags and Twitter Cards.
OpenGraph Tags
Make sure your pages have Open Graph tags if you want them to look good in social feeds when shared.
Please note that URLs inside OG tags must be absolute and utilize the http:// or https:// protocols.
You can find more information on the Open Graph protocol here.
Twitter Cards
If you want your pages to look good in the Twitter feed when shared, you should implement the Twitter card tags available.
Please note that URLs inside Twitter cards must be absolute and utilize the http:// or https:// protocols.
You can find more information about Twitter cards here.
Solution
Add opengraph Sphix package (sphinxext-opengraph) and Twitter card metadata to the base.html file.
Note
For the Twitter metadata, I wasn't about to get the dynamic description for each page. I looked at the base template and I thought adding {{ description }} to the Twitter metadata tag would solve the issue. However, the content returned empty on the web page. The current description in place is from the home page: "What can we help you with today? (Looking for the previous documentation? Visit Aiven Help or use the search above.) Explore: Learn about the Aiven platform icon-postgres PostgreSQL Powerful relation...".
It's a general description that, I think, could apply to all page links. The only thing not dynamic is the page description.
If you have any insights into having a more dynamic page description, feel free to share them.
Preview URL: https://deploy-preview-196--developer-aiven-io-preview.netlify.app/

How does this relate to #191 ? Perhaps we could combine them, they seem to overlap.

How does this relate to #191 ? Perhaps we could combine them, they seem to overlap.

Ah, I didn't know this. I was given the assignment directly from Olga in Slack. I'll look into it on Monday.

The PR has been updated. Since Heikki already has a solution in place for the OpenGraph issue (PR 191), I removed the sphinxext-opengraph package and kept the Twitter tags.

Thanks! I'm sorry about the crossed wires, I'm trying to sort out our internal process so we don't cause this again.
Would it be possible to include an image with the twitter card?

Ohhh, I think the twitter card will pick up the og images, I'm going to rebase this branch onto main now so that it picks up the changes already merged.

The og image may have worked, but it was no problem to go ahead and add that image path to the Twitter image meta tag. I used the same image path as Heikki for his og solution.

I tried previewing the twitter card on the preview of this PR and it doesn't seem to pick up the image. Does twitter:image need a full URL rather than a relative path, do you know?

I have just updated the image path to include the URL. I've also found this handy Tweet preview tool from Twitter. According to this, the image is now present. https://cards-dev.twitter.com/validator
Thanks for your help and patience, Lorna!


supabase db remote commit errors with Postgis

Bug report
Describe the bug
supabase db remote commit doesn't work.
Returns:
pg_dump: error: query failed: ERROR:  permission denied for table direction_lookup
pg_dump: detail: Query was: LOCK TABLE "tiger"."direction_lookup" IN ACCESS SHARE MODE

To Reproduce

Supabase instance.
Install postgis
Run the command.

Expected behavior
It would write the schema.
System information

OS: macOS. Homebrew supabase.

Thanks for the quick turn around. Just did brew reinstall supabase and it appears this has not been released.

OK, currently on 1.40.1.

brew upgrade supabase

supabase/tap/supabase 1.40.1 already installed

supabase db remote commit

Dumping schemas from remote database...
pg_dump: error: query failed: ERROR:  permission denied for table direction_lookup
pg_dump: detail: Query was: LOCK TABLE "tiger"."direction_lookup" IN ACCESS SHARE MODE
Error: Error running pg_dump on remote database: error running container: exit 1
Try rerunning the command with --debug to troubleshoot the error.

Could you try brew update first before upgrade?
The tap seems to be updated successfully to v1.40.3 https://github.com/supabase/homebrew-tap

brew upgrade supabase

==> Upgrading supabase/tap/supabase
  1.40.1 -> 1.40.3

supabase db remote commit

Dumping schemas from remote database...
pg_dump: error: query failed: ERROR:  permission denied for table direction_lookup
pg_dump: detail: Query was: LOCK TABLE "tiger"."direction_lookup" IN ACCESS SHARE MODE
Error: Error running pg_dump on remote database: error running container: exit 1
Try rerunning the command with --debug to troubleshoot the error.

supabase -v

1.40.3

OK, thanks. I saw the same issues with the flags as well.
Any options to get local dev working? I'm working on a major new push and without local dev I'm going to be lost.
Advice here? I'm stuck.

Could you email<EMAIL_ADDRESS>with your project ref? We can apply this fix for you manually to unblock.

We have completed the rollout to all existing projects. Could you let me know if remote commit is still stuck?
I will close this ticket as resolved but feel free to reopen.


fix: notebook execution for apptainer

At https://github.com/snakemake/snakemake/blob/afa7bad1a72118dc609f1499560c0847edc36413/snakemake/deployment/singularity.py#L118-L125
snakemake searchpaths are only bound into apptainer when is_python_script is True. For notebook directive, is_python_script will be False and snakemake searchpaths are not bound into apptainer, causing ModuleNotFoundError: No module named 'snakemake' error in the preamble. This PR sets is_python_script to True for notebook directive.
Also, the name, is_python_script, could then be somewhat confusing. Changing it to something like is_script or has_preamble might be better.
QC

[ ] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[ ] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Summary by CodeRabbit

New Features

Introduced a new parameter to enhance script execution for Python scripts in Jupyter notebooks.
Added functionality to determine whether a notebook should be drafted or executed based on user input.

Bug Fixes

Improved validation for local notebook paths to ensure editing is only allowed for local files.

Also, it may be better to add some tests for notebook execution in apptainer, to prevent trivial bugs like this one here.

Also, it may be better to add some tests for notebook execution in apptainer, to prevent trivial bugs like this one here.

Indeed! I won't be able to do that now, but if you want to do it in this PR please feel free to do so. If not, it would be ok to merge this without or while adding a TODO.

Also, it may be better to add some tests for notebook execution in apptainer, to prevent trivial bugs like this one here.

Indeed! I won't be able to do that now, but if you want to do it in this PR please feel free to do so. If not, it would be ok to merge this without or while adding a TODO.

Thanks for the quick reply! I would recommend merging this first, as this bug makes notebook in apptainer unusable. I can open an issue to track on adding tests and do it later.


update doc reference links

The links to the reference pages are broken. This PR updates these links.

duplicate


Group by on orders was causing some problems

No DB or CFG updates on this one.

Merged to stable. Ready for pull to QA.

Pulled to QA, ready for testing.


Update solana.tokenlist.json (Solark)

updated with meta data for the Sol.Ark token.

[x] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/lib/.json and in the PR
At minimum each entry should have

Token Address: AhRozpV8CDLJ5z9k8CJWF4P12MVvxdnnU2y2qUhUuNS5
Token Name: Solark
Token Symbol: ARK
Logo: https://i.ibb.co/JyJ9kf0/Logo-1.png
Link to the official homepage of token: www.solark.xyz
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Invalid json, please run yarn test before submitting PR.


How can i set the retry count?

Universal image loader is too many try to connecting when image is not found.
I wish control the retry connection count.
Is it possible?
many thanks.

I didn't see retry happen! Can you tell me how can see the retry happened?


Add error handling when view is not an object in render()

I got stuck with this error for a long while "Uncaught TypeError: this.tail.search is not a function" and in the end somebody helped by pointing out I was passing render() an array rather than object for view.
I know a lot of people won't have issues with this, but as I explained in my post  http://morenafiore.com/2-things-i-did-not-understand-about-mustache/ it might help people who approach Mustache.js for the first time to not give up by having more specific errors displaying.
Other people had this error, but it was down to template not being passed as plain text  http://stackoverflow.com/questions/10872615/what-is-the-difference-between-mustache-render-and-mustache-to-html
I hope you might be able to help :)

Hi @morena,
IMO we should absolutely write APIs which helps you as a developer, rather than you having to delve into the source code. I'm all for throwing a TypeError with an informative error message if we get anything else than an object provided for mustache.render().
Thanks for writing a tutorial on using mustache.js! :+1:

Great post @morena, thanks for the thoughts!
I agree with @phillipj; the library should be more dev friendly, especially for newcomers.

Thanks! I would like to help but I think I might not have the right skills yet

No worries, reporting this issue was great already! :)

I've been looking at this issue now and just realized I was probably too quick when answering the first time. An array as a view should not be a problem. Although I reproduced the error you mentioned when providing Mustache.render() with anything else than a string as the template. Could that be your case too?
Mustache.render('{{#.}}{{.}} {{/.}}', ['hello', 'world']);
// -> hello world

Though wrapping the first argument into e.g. an array throws the tail.search error.


Unexpected behavior when using Angle.from_three_points

Expected behavior
When i running those code below and export

It's expected to show 4 full interior angle and I had tried all the possibilities of quadrant and just doesn't help.
How to reproduce the issue

Code for reproducing the problem
class Example(Scene):
    def construct(self):
        p1 = np.array([-2, 0, 0])
        p2 = np.array([-3, 1.5, 0])
        p3 = np.array([0, 3, 0])
        p4 = np.array([3, 1.5, 0])
        p5 = np.array([2, 0, 0])
        p6 = np.array([-2.5, -1.5, 0])
        p7 = np.array([2.5, -1.5, 0])
        dot = VGroup(Dot(p1), Dot(p2), Dot(p3), Dot(p4), Dot(p5), Dot(p6), Dot(p7))
        pentagon = Polygon(p1, p2, p3, p4, p5)
        a2 = Angle.from_three_points(p1, p2, p3, quadrant=(1, 1))
        a1 = Angle.from_three_points(p5, p1, p2, quadrant=(1, 1))
        a3 = Angle.from_three_points(p2, p3, p4, quadrant=(1, 1))
        a4 = Angle.from_three_points(p3, p4, p5, quadrant=(1, 1))
        a5 = Angle.from_three_points(p4, p5, p1, quadrant=(1, 1))
        a_p = VGroup(a1, a2, a3, a4, a5)
        a6 = Angle.from_three_points(p5, p1, p7, other_angle=True)
        a7 = Angle.from_three_points(p1, p5, p6, other_angle=False)
        a8 = Angle.from_three_points(p5, p6, p7, other_angle=True)
        a9 = Angle.from_three_points(p6, p7, p1, other_angle=True)
        a_q = VGroup(a6, a7, a8, a9)
        a_a = VGroup(a_p, a_q)
        quadrileteral = Polygon(p1, p6, p7, p5)
        self.play(Create(pentagon), Create(quadrileteral), Create(dot))
        self.play(Create(a_a))

FFMPEG
Output of ffmpeg -version:
ffmpeg version 6.0-essentials_build-www.gyan.dev Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)

I assume you got confused by the naming/numbering of your points...
class Example(Scene):
    def construct(self):
        p1 = np.array([-2, 0, 0])
        p2 = np.array([-3, 1.5, 0])
        p3 = np.array([0, 3, 0])
        p4 = np.array([3, 1.5, 0])
        p5 = np.array([2, 0, 0])
        p6 = np.array([-2.5, -1.5, 0])
        p7 = np.array([2.5, -1.5, 0])
        dot = VGroup(Dot(p1), Dot(p2), Dot(p3), Dot(p4), Dot(p5), Dot(p6), Dot(p7))
        self.add(Text("1").move_to(p1))
        self.add(Text("2").move_to(p2))
        self.add(Text("3").move_to(p3))
        self.add(Text("4").move_to(p4))
        self.add(Text("5").move_to(p5))
        self.add(Text("6").move_to(p6))
        self.add(Text("7").move_to(p7))
        pentagon = Polygon(p1, p2, p3, p4, p5)
        a2 = Angle.from_three_points(p1, p2, p3, quadrant=(1, 1))
        a1 = Angle.from_three_points(p5, p1, p2, quadrant=(1, 1))
        a3 = Angle.from_three_points(p2, p3, p4, quadrant=(1, 1))
        a4 = Angle.from_three_points(p3, p4, p5, quadrant=(1, 1))
        a5 = Angle.from_three_points(p4, p5, p1, quadrant=(1, 1))
        a_p = VGroup(a1, a2, a3, a4, a5)
        a6 = Angle.from_three_points(p6, p1, p5, quadrant=(1, 1))
        a7 = Angle.from_three_points(p1, p5, p7, quadrant=(1, 1))
        a8 = Angle.from_three_points(p7, p6, p1, quadrant=(1, 1))
        a9 = Angle.from_three_points(p5, p7, p6, quadrant=(1, 1))
        a_q = VGroup(a6, a7, a8, a9)
        a_a = VGroup(a_p, a_q)
        quadrileteral = Polygon(p1, p6, p7, p5)
        self.play(Create(pentagon), Create(quadrileteral), Create(dot))
        self.play(Create(a_a))
        self.wait()

omg i didn't realize that tysm for your help

what should i do when i want to make another obtuse angle at point 7

what should i do when i want to make another obtuse angle at point 7

I don't understand exactly what you want, and I don't think this is the correct place to discuss this. Come over to Discord:
Where can I find more resources for learning Manim?


Add js events

Create the following list of events:

beforeCreate
afterCreate
beforeDelete
afterDelete

Hi, please update your package to  dev-master version

Now you can use events as follows https://github.com/yii2mod/yii2-comments#using-comment-plugin-events

Hi, thanks for the update, gonna work on it!

That basic example should be in the view file that uses the Comment::widget or in the comments view like index or _form? I tried that basic example in the one with the widget and its not working right now, gonna keep trying to find out whats going on.

Yes, you're right. I placed this code in the view file, for example:
// your-view.php
<h1>Some page </h1>

<?php echo \yii2mod\comments\widgets\Comment::widget([
      'model' => $model,
]); ?>

<?php
$this->registerJs(
    "
    $(document).on('beforeCreate', '#comment-form', function (e) {
        console.log(e);
    });

    $(document).on('afterCreate', '#comment-form', function (e) {
        console.log(e);
    });
     ",
    \yii\web\View::POS_READY,
    'custom-js'
);
?>

I finally get it working, the assets were not updating the changes. But I'm seeing a problem, are you sure the 'afterCreate' can add events to the comments like, for example:
$(document).on('afterCreate', '#comment-form', function (e) {
        console.log('Hi');
        $('.comment-author-avatar').on('click', function() {
            console.log('Bye');
        }
});

This code doesnt add the event on click on the image when the comment is created, it says 'Hi' but the event is not working. Maybe because this code is running when the nodes arent still created?

I think in this situation you need to use click event on avatar image separate from afterCreate event. afterCreate event runs after pjax has been updated comments form with comments tree.

I think you can use the following code for handle the click on avatar image
        $(document).on('click', '.comment-author-avatar', function() {
            console.log('Bye');
        }

Yes, after a few tries I have finally achieved what I wanted, had a few problems with trying to fill the textarea with the username like '@user213 ' when it is a reply and delete it when you cancel it, but I've found the way. Thank you for your answers and the help.


Update frontend

Every error will be warning attached with response
Warning color will be yellow

Tested-by: zlq

/gcbrun

/gcbrun


[FEAT] Remove need for target_rows arg from estimate_u_using_random_sampling, use convergence instead

Is your proposal related to a problem?
It would be nice to not have to supply this hyperparameter.
Describe the solution you'd like
Similar to EM training, which iterates to convergence, we could run the random sampling on iteratively larger batches until the u params converge
perhaps steps of

10e5
5e6
10e6
5e7
10e6
5e7
10e7
etc

If I understand correctly, the runtime of the initial runs will be dwarfed by the runtime of the final run, similar to iterative deepening, so it shouldn't take much longer than if you had divined the ideal number of samples from the beginning.
IDK if you would still want some sort of convergence criteria, so that during exploratory runs you would stop iterating sooner, and during final training you would train longer. But this criteria of "percentage change since last run" might be more intuitive than "number of rows", because I would want to know "ehh during my exporatory phase I just care about being within 5% of the ideal" so I could set the convergence criteria to ~.03 or something.

A few thoughts on this:

To some extent, it is possible to 'directly estimate' what target rows needs to be from the cardinality of the input columns.  For example, if the input column with the highest cardinality has just 100 distinct values, target rows could be relatively small.
Or, another way of saying the same thing, is that many of the u values can be estimated fairly precisely from a relatively small target rows.  When we're setting target rows high, that's usually because we need a precise estimate of a very low probability event (e.g. two people having the same full name).
That does suggest that, at the cost of additional code complexity, you could probably speed the code up by running estimates separately for different comparison columns (high target rows for full name, low target rows for month of birth)

Overall I'm not sure.  The strategy you suggest definitely sounds like a sensible way of working out what target rows should be.  It might be faster to run some analysis on the cardinality of the columns to go 'straight to' the answer.  The exact algorithm to estimating this would take a bit of thought but should be possible.
A final consideration is that in practice a lot of users are constrained by their compute.   So they end up using a the highest value target rows which they can tolerate from a compute-time perspective, rather than the ideal value from a precision-of-parameter-estimation point of view.
Does that make sense?  Curious if you have any further thoughts in light of the above

Those are great thoughts, thanks for the explanation.
I think I have a vague sense of how cardinality could be used to estimate how many rows we need, but I'm nervous that any algorithm we come up with isn't going to be principled enough, and there is going to be some edge case that doesn't work for someone and then they get a bad estimation.

A final consideration is that in practice a lot of users are constrained by their compute.

Hmm, that's good to know. I don't see any way around that though. If we are trying to capture a very rare event, I think you just have to try a lot of samples. We can't somehow "stack the deck" so that the event happens more frequently, because we can't correct the odds for "stacking the deck" without knowing the base probability, which is what we're trying to find! IDK maybe I'm not thinking of that correctly
I think the best we can do is reduce number of samples/compute time for the comparisons which don't need many samples (like month of birth, as you say). I think the additional code complexity would be worth it so we can find u values for each Comparison separately. Some comparisons, like levenshtein, will have a much larger performance penalty than == checking, so say someone is using levenshtein on month-of-birth (OK that's a bad example, but ...), then reducing the number of samples for that column will have a really big effect.
Convergence based on uncertainty
I'm thinking more on convergence criteria, and it might be principled to treat it like a counting experiment:
If you take S samples, and T of them are true, then the rate is of course r=T/S. But the interesting thing is that the uncertainty of T is sqrt(T), so the uncertainty in r is sqrt(T)/S (source: page 245 of this book)
For our scenario, for one comparison level: say we sample P pairs, and our level is true T times, then our estimate for u is u=T/P and uncertainty in u is sigma_u=sqrt(T)/P. But, since we are using u as a multiplier, what we really care about is not the absolute uncertainty in u, but the fractional uncertainty, E=sigma_u/u. This is merely E=sqrt(T)/T=1/sqrt(T). Solving for T we get T=1/E^2. So for example if we wanted our fraction uncertainty in u to be less than .1 ("68% (one sigma) of the time, our prediction is within 10% of its true value"), then we would need T=1/.1^2=100 true samples.
So we could keep random sampling in batches of say 10k, with either of two stopping criteria: Either we get 100 True samples, or in the case of a really rare event, we may never see 100, so there is also a max_pairs parameter

The counting idea makes sense and I think has the potential to make the calculation quite a bit faster.
One challenge here is how the algorithm would work in Spark (or other distributed systems).
In Spark, if a single large query parallelises well, then this is often faster than a ostensibly more efficient algorithm that requires several queries.  Often it just requires quite a bit of testing to work on what in practice works better.
Another aspect to this that's worth keeping in mind is that there's (arguably) less need to optimise the methods in Splink that are used for training.  At least in our use, models are trained once and used many times, so the performance of predict() is more important than the performance of the various estimate...() methods.


nerdctl-full: remove stargz-store (unused by stargz-snapshotter) and unused CNI plugins

The nerdctl-full does not need to contain the following files:
-rwxr-xr-x. 1 root root  36M Sep 29 07:05 bin/stargz-store
-rwxr-xr-x. 1 root root 3.9M Sep  7 19:48 libexec/cni/bandwidth
-rwxr-xr-x. 1 root root 9.4M Sep  7 19:49 libexec/cni/dhcp
-rwxr-xr-x. 1 root root 3.9M Sep  7 19:48 libexec/cni/host-device
-rwxr-xr-x. 1 root root 3.3M Sep  7 19:49 libexec/cni/host-local
-rwxr-xr-x. 1 root root 4.0M Sep  7 19:48 libexec/cni/ipvlan
-rwxr-xr-x. 1 root root 3.4M Sep  7 19:48 libexec/cni/loopback
-rwxr-xr-x. 1 root root 4.1M Sep  7 19:48 libexec/cni/macvlan
-rwxr-xr-x. 1 root root 4.2M Sep  7 19:48 libexec/cni/ptp
-rwxr-xr-x. 1 root root 3.6M Sep  7 19:48 libexec/cni/sbr
-rwxr-xr-x. 1 root root 2.9M Sep  7 19:49 libexec/cni/static
-rwxr-xr-x. 1 root root 3.5M Sep  7 19:48 libexec/cni/tuning
-rwxr-xr-x. 1 root root 4.0M Sep  7 19:48 libexec/cni/vlan
-rwxr-xr-x. 1 root root 3.6M Sep  7 19:48 libexec/cni/vrf

cc @ktock Are we safe to remove stargz-store?

Yes, it is. It's not used from containerd.

I think since https://github.com/containerd/nerdctl/pull/848 at least the macvlan and dhcp CNI plugins are supported/used. So why remove those from nerdctl-full? I find it very convenient for ubernerd that nerdctl-full contains everything to run nerdctl. Perhaps this issue can be closed?

Sure


Report section and python API reference

I worked on the build section of the report system, added more context variables and a more complex example.
Hopefully this translates fine to readthedocs this time.
There is also a starting point for the python API reference included.

@SergeoLacruz the additions to the build report docs look great, thanks for that.
However the "api_reference" page feels unfinished, is not linked from anywhere, and seems to be somewhat duplicate of the existing examples page
I'd suggest removing the "api_reference" page from this PR, and perhaps (in a separate PR) extending the "examples" page


boards: nxp: Add missing RW board feature tables

Add missing items to RW board feature tables.

We need a fix like #78415 for the FRDM board as well it seems. Sorry, should have realized this when I made my earlier suggestion


FancyZones has phantom position when using Win+Arrow and 'Restore the original size...' setting is on

I have a FancyZone layout of 6 zones, 3h x 2v.  I use Win+Arrow to move the windows about a lot.  I also usually have the 'Restore the original size of windows when unsnapping' feature enabled.
I have just upgraded from v27.1 to v29.0, and when using the Win+Arrow feature, a 7th phantom position appears, either before position 1, or after position 6, depending if I am cycling forwards or backwards, using the left or right arrows.  The appearance of this 7th zone is that the header of the app stays in the same position, but then the window resizes to its original pre-snapping size and shape.
If you turn off the 'Restore the original size of windows when unsnapping' feature, then the phantom zone disappears.  if you turn it on again, it reappears.

This is currently by design, it's similar to Windows Snap where one of the position is the original window position.

Ahh, ok, well, test it out and see what you think, because I think it fails in a few areas - 1, it's giving me a 7th zone that I haven't asked for, 2, when it restores it to its original size in Zone 7 it doesn't also restore it to its original location, so half the window is off screen, and 3, it's an undocumented feature that is attached to a different settings option (i.e. one that says that when you UNsnap it, restore it to its original size/position - I haven't UNsnapped it, I'm just cycling through my zones).
Thanks,
Russ
Sent from my phone, and either typed or dictated, so please forgive any typos :)
-------- Original message --------
From: Enrico Giordani<EMAIL_ADDRESS>Date: 30/12/2020 11:17 (GMT+00:00)
To: microsoft/PowerToys<EMAIL_ADDRESS>Cc: RussellStainer<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/PowerToys] FancyZones has phantom position when using Win+Arrow and 'Restore the original size...' setting is on (#8819)
This is currently by design, it's similar to Windows Snap where one of the position is the original window position.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/8819#issuecomment-752423064, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOHXXGXJQZPF32AK6U6VPCDSXMD6FANCNFSM4VOGF3FA.

We already know it's not ideal in all scenarios https://github.com/microsoft/PowerToys/issues/5895

when it restores it to its original size in Zone 7 it doesn't also restore it to its original location, so half the window is off screen

That doesn't sound as a FZ bug, but more as an application bug, does it happen with all application? Try notepad just to verify it.

it's an undocumented feature that is attached to a different settings option

We may improve the documentation but it's not undocumented https://docs.microsoft.com/en-us/windows/powertoys/fancyzones#settings and it also behaves like Windows Snap.
Let's continue the discussion in https://github.com/microsoft/PowerToys/issues/5895


ChromeOS support for UltraBox offline

I'm no coder, so this unfortunately doesn't help me with anything.


Move to Swiftz

The Basis is no longer a viable platform to build out these kinds of libraries.

We can handle this via Carthage, now that that's (somewhat) working.


Quality Assurance: Fix inconsistent spacing in actions drop-down menu.

Fixes: #3010

Automated message from Dropbox CLA bot
@mahimg, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

@mahimg can you sign the Dropbox CLA?

Also, you should set your GIT_AUTHOR_EMAIL to your email, not your name.

@timabbott I have signed the Dropbox CLA, I had a recent username change and hence it was rejected, I have signed again.

Automated message from Dropbox CLA bot
@mahimg, thanks for signing the CLA!

Merged as 7f6a616c3024e361e4e69162162392ac145cb6cf after fixing the author email on the commit.  Please fix your repo so that your future commits don't have this problem.

(and thanks for the contribution!


feat: add compat flag to treat BinarySwitchSet and ThermostatModeSet command as a report

Hello, i've try to do the job for BinarySwitchSet and ThermostatModeSet command as a report.
(exemple for SRT321 HRT4-ZW)
for this : https://github.com/zwave-js/node-zwave-js/issues/5937
I've tested : it's ok (value updated)
just one thing I don't understand: The commands arrive during their first update (I don't know why)

if you have un idea ?
and if you can push this update please.
Thx you.

just one thing I don't understand: The commands arrive during their first update (I don't know why)

This is because the devices don't support the CCs (allow being controlled), but they only control them in other devices. Unless specifically handled by a compat flag, Z-Wave JS only exposes values for supported CCs.

I'd prefer to have one compat flat, probably treatSetAsReport, which is an array of command names (e.g. ["BinarySwitchCCSet", "ThermostatModeCCSet"]), instead of potentially having to add one or more compat flags per command class to map.
You don't have to implement the mapping for all of them, but laying the foundation for this once would be good.

Yes, I also thought about this (use array or end with "Set"), but the problem is that BinarySwitchCCSet returns "currentValue", and ThermostatModeCCSet retun "mode". How can I know (in code) the command name to use (currentValue or mode) ?
And how to use on this code for example ? this._valueDB.setValue(BinarySwitchCCValues.currentValue.endpoint(command.endpointIndex), command.targetValue);

but the problem is that BinarySwitchCCSet returns "currentValue", and ThermostatModeCCSet retun "mode"

Yeah we still need CC-specific code to handle this like you did in Node.ts. That shouldn't prevent us from unifying the compat flag into one though. We'll stick with the two you already added to Node.ts and add others on demand.


AWS S3 Binding - Add Content-Type in the metadata request

Description
With this PR is possible add the Content-Type header when creates the object in the storage
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Hi @ItalyPaleAle, i've just updated the code from your suggestions and signed-off the commit. Let me know if it's okey

@luigirende the "sign-off" is not applied to all commits.
To add your Signed-off-by line to every commit in this branch:
Ensure you have a local copy of your branch by checking out the pull request locally via command line.
In your local branch, run: git rebase HEAD~3 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin main

@luigirende the linter is reporting some issues

@luigirende the linter is reporting some issues

@ItalyPaleAle in the task I see there is this issue "Error: File is not gofmt-ed with -s (gofmt)" I doubt the issue is caused by my changes.

@ItalyPaleAle there are 2 issue on test task, I don't know how to fix them.

@ItalyPaleAle there are 2 issues on test task, I don't know how to fix them.

Thanks for fixing the linter errors! Those are unrelated - some tests are flaky

Thanks again for this.
In the future please open an issue for discussion before making a PR and allow for maintainers or other Dapr users to leave comments. Or make a PR for an issue which has already been open for some time (and then link to that issue in your PR).


alias slugs & getTranslatedLink

Hi,
thanks again for this awesome tool.
I had the need to create localized slugs for some languages
so for instance
static page
en --> /contact
it   --> /contatti
dynamic route
en --> /products/[id].js
it   --> /prodotti/[id].js
I created a new field in i18n.json called alias which can be used for this purpose
If you want to translate the slug of a page just add a new key corresponding to the page name in the pages_ folder and a list of translations in various languages. You don't have to include the slug translation for every language or for all pages.
...
   "alias": {
          "/contact": {
                "it": "/contatti"
          },
          "/products/[id]": {
                "it": "/prodotti/[id]"
          }
          ...
   }
...
}

I have also changed the Link and Router components (and also the fixHref and fixAs files).
In addition, I created a new function
getTranslatedLink
which returns the correct translation in another language given an alias and the target language. This is useful when for instance you want to include a link to the translate versions of the current page in the header component passing the router route and path without caring about aliases
It works without problems on the website I shared with you in issue 116
Tests (sorry, I'm not used to them) and documentation are missing, I can write the latter if you think this could be a feature to merge
Hope it helps

First of all, thank you for your first contribution @tonxxd !! 🎉
I like the idea to translate the slugs. It's a feature that brings a lot of possibilities. I'm going to review the code carefully during the next days! Sorry if I can't provide you fast feedback this time.
Nevertheless, there is an open RFC of Next.js with a similar feature (rewrites), that nowadays it's experimental. Take a look:

https://github.com/zeit/next.js/issues/9081

I'm not sure if it is possible (or will be possible) to do the same with Next.js.
I would appreciate it if you can add some tests to cover some cases; ex: Not overwriting /_next or /api folders... What happens in nested alias, or alias with a different dimension, ex:
"/contact": { "it": "/another/path/contatti" }

I see that there are a lot of cases to cover. Maybe, in the case that this feature is not covered by the Next.js RFC, we can provide a simple use case just to rename the page and not the full path. 🤔
However, if this feature is part of the Next.js RFC I prefer to don't bring this responsibility to this lib. And maybe I prefer to write some examples and good documentation about how to do it correctly with the Next.js way.
So I recommend you before writing some tests or improvements to this PR, to try the experimental rewrites of Next.js https://github.com/zeit/next.js/issues/9081
I'm also going to experiment with it in the next days to have a better idea of how to achieve this.

Oh wow didn't know about it, seems exactly what I need. Yes probably is nonsense to implement a customized "rewrite" system when it's already (or will be) handled by next.js
I'll take a look
Thanks ;)

Indeed I managed to accomplish the slug localization with next.js rewrites, awesome feature!
I am still using next-translate to generate the locale folders and manage the translations. To better handle routes I am using next-routes so if I have to generate a route for products I simply pass the route name to the next-routes Link component and I get back the localized slug with the populated params.(even if this feature could be probably integrated into next-translate Link component).
I think you can reject this pull request since it adds custom logic not needed (and it doesn't handle different alias dimension as you pointed out)

@tonxxd Thank you so much for your effort 👏! I'm going to close it, but what do you think about doing a new PR with an example under /examples folder? This would be very helpful for some people!


Set up Prettier and Eslint for the project and run it through the whole code-base (Pair or groups work)

I'm submitting a ...

feature request/bug report (a bit both I would say)

What is the current behavior?
Prettier and Eslint were installed but never set up properly. That's why we had to run Prettier manually all the time. I tried to configure them in the following PR - it did run well locally however it would need to format the whole codebase on deployment to pass the vercel check which was risky to do at that time:

Ideally we would need to have a group session to go through config (to customise Eslint as per our preferences) then run it through the whole codebase and fix issues along the way. Sometimes solution offered by Eslint is not the best solution - we we might need to come up with the walk-around for some cases.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
You can load the eslint branch from the following PR to check the results of it's run.
What is the expected behaviour?
No longer running Prettier for each page. Being prompted to fix format issues on the go.
Solution

[ ] Group (or pair) session to configure the Eslint - discussion which features we need, how we format the code, etc;
[ ] Running the confit through code base and fixing existing issues;

Link to Trello card: here

 [DEV]: Set up Prettier and Eslint for the project and run it through the whole code-base


DCC incorrectly enabled for certain format lists

It seems AMDVLK and AMDGPU-PRO enable DCC for images if a compatible integer format with a different sign than VkImageCreateInfo::format appears in VkImageFormatListCreateInfo::pViewFormats before an incompatible format. Reordering the format list so that the incompatible format is before the compatible format disables DCC.
The issue can be observed with this program: https://gist.github.com/pendingchaos/53f88567390a7df936a46dc7c518eb08
AMDVLK 2021.Q1.4:
driverVersion: 8388785
deviceName: Unknown AMD GPU
driverName: AMD open-source driver
       UINT,SINT: dcc
UINT,SFLOAT,SINT: nodcc
UINT,SINT,SFLOAT: dcc

driverVersion: 8388785
deviceName: Radeon RX 580 Series
driverName: AMD open-source driver
       UINT,SINT: dcc
UINT,SFLOAT,SINT: nodcc
UINT,SINT,SFLOAT: dcc

AMDGPU-PRO 20.45:
driverVersion: 8388766
deviceName: Unknown AMD GPU
driverName: AMD proprietary driver
       UINT,SINT: dcc
UINT,SFLOAT,SINT: nodcc
UINT,SINT,SFLOAT: dcc

driverVersion: 8388766
deviceName: Radeon RX 580 Series
driverName: AMD proprietary driver
       UINT,SINT: dcc
UINT,SFLOAT,SINT: nodcc
UINT,SINT,SFLOAT: dcc

"Unknown AMD GPU" is a Radeon RX 6800.
It seems the cause of this issue is that PAL's Device::ComputeDccFormatEncoding exits the loop as soon as a compatible integer format with a different sign is found, before it can check the compatibility of the rest of the formats.
This issue occurs when a Direct3D application under DXVK uses certain typeless formats such as DXGI_FORMAT_R32_TYPELESS. Trine 4: The Nightmare Prince is an example of this, though it appears enabling DCC for these formats does not cause obvious issues for this game.

Fixed in 2021-Q2.2


Github Code Spaces Template

If anyone if familiar with GitHub code spaces it would be awesome if we have a template with Pynecone preloaded so new users can experiment as fast as possible. We can also put a link to the template in the readme.

@Alek99 could you please assign this to me, I am looking to contribute to this project. thanks!

@Alek99 I have the template repository and the readme instructions ready. Here is the template repository - https://github.com/rahulbagai/pynecone-template
and here is the README instructions
You can also access this project in a fully configured development environment using Github Codespaces. Click the following link to create a new Codespace:

[![Open in Codespaces](https://img.shields.io/badge/Open%20in-Codespaces-blue?logo=github&style=for-the-badge)](https://github.com/codespaces/new?template_repository=rahulbagai/pynecone-template)

Could you please help review and allow me to make the contributions to pynecone repos. thanks!

Awesome thanks! I'll test this, after I confirm it works the next steps have a couple options:
a) We make a subfolder in the main Pynecone repo and add this code there.
b) I make another public repo under the Pynecone organization just to have this code.

I tried this with the counter app and it seems that state is not working with code spaces. Based on a discord discussion I had with a user I think it may require some special configuration for code spaces to support web sockets?
Is the counter example working with your code spaces instance?

@Alek99 Ah my bad! I was working on local devcontainer and it worked fine there. I will test it out in codespaces as well, make necessary changes and send the change request.

@Alek99, I've been investigating a challenge I've encountered and would appreciate your guidance.
I've been running a Pynecone application (counter app) in Github Codespaces. However, I've noticed that while the app runs locally on localhost:3000, there might be some necessary adjustments or configurations required to make it fully compatible with Codespaces, which provides a cloud-based development environment, especially regarding the support for WebSockets.
I've confirmed this necessity based on the following browser logs:
[blocked] The page at https://rahulbagai-miniature-broccoli-4qvxg4pxxwcqvv9-3000.preview.app.github.dev/ was not allowed to run insecure content from ws://localhost:8000/event/?EIO=4&transport=websocket.

The issue seems to be mixed content, which occurs when a webpage loaded over a secure HTTPS connection attempts to load resources via an insecure HTTP or WebSocket connection.
To mitigate this, I believe we should switch to a secure WebSocket connection (wss://) rather than an insecure one (ws://). However, I'm unsure whether Pynecone currently supports this functionality.
Upon inspecting the constants.py file for any possible configurations, I noted a segment in the Endpoint class related to URL replacement and WebSocket protocol switching. I suspect this might be contributing to the insecure content issue.
Here are a few potential solutions I've considered:

Secure Server Usage: Hosting the application on a server that uses https:// and supports wss:// for WebSocket connections could prevent mixed content issues by ensuring all connections are secure. Does Pynecone currently support any such configurations in the pcconfig.py file?

Source Code Modification: Theoretically, we could modify the Pynecone source code to always use wss:// instead of ws://. However, this would require an intimate understanding of the library's internals, which I might not be equipped for at the moment.

Proxy Server Implementation: Using a proxy server that accepts wss:// connections and forwards them as ws:// connections to the Pynecone server could also solve the mixed content problem. The client would only ever see the secure wss:// connections. However, this method would require additional setup and could potentially introduce additional latency.

Could you please advise on the best way to proceed? I would greatly appreciate your insights.

@Alek99
I hope you're doing well. I am following up on the issue related to the integration of Pynecone. I'd like to know if there have been any updates or progress.
Specifically, I would appreciate any information on the following:

Compatibility with GitHub Codespaces: Have you encountered any guidelines or suggestions from the GitHub Codespaces environment that could help resolve the WebSocket compatibility issue? Understanding any platform-specific considerations would be beneficial in determining the best solution.

Community Input: Has any feedback or insights from other contributors or maintainers within the Pynecone community regarding WebSocket support in GitHub Codespaces? Their experiences or suggestions could provide valuable guidance in addressing this issue.

Next Steps: Based on our previous discussions, I would like to know how to resolve this issue. Are there any specific steps or recommendations you have in mind? I'm eager to contribute and make progress on this task.

I appreciate your time and support throughout this process. Any updates or guidance you can provide would be highly appreciated.
Thank you!

@Alek99 I have submitted a pull request for this feature. Would you kindly review it?


Unique identfier is on wrong field of GElement

Looked at the source.
Code test uniqueness on element.id but, as is, 'id' in tags is not mandatory.
I think that is way text and interaction tags are not handled.
But I see that image tag in nifty-editor terminal log have id="".
 <image id="" filename="Interface/start-background.png"/>

So, if code test IDgenerator.isUnique() there can be just one tag with id="". Correct ?
If it is correct my proposal is new transient field with generated key. That key is unique. If it is needed.
Code can then test for id uniqueness but not require that 'id' is not empty or null.

Hello, first of all thank you for digging into the code! you are brave :smile: !!! I wrote the code at least one year ago so I don't have a very clear view of the system, sorry. As far as I can tell the id uniqueness is mandatory for the right binding between GElement model and Nifty-gui internal Element instances. So no, the id field shouldn't be optional and should be present both on the Element ( the image tag ) and the GElement itself. I think that the default behavior in the case there's no id attribute is to generate one using the IDgenerator.
Now the real problem the text and interaction tag are skipped in the loading level, have a look at GUIReader. The unknown elements are not added to the GUI instance, so when you save again the file they disappear.


Huge Dupe Bug

Multiplayer in version <IP_ADDRESS>
Chests allow duplication of items when using the loot all button. I'm pretty sure this is being caused by the server. The way it works is you put a bunch of separate stacks of items in a chest, hit loot all, you get all the items but when you close the chest and reopen it all but the first slot of items are in the chest AND in your inventory. This can be replicated numerous times with natural and player placed chests. Seems like a sync issue, server doesn't realize you pressed loot all or something. Not exactly sure.

Now fixed, thanks.


xplat: DETACH_PROCESS code missing

UnInitialization code originally in DllMain(DETACH_PROCESS) isn't available on xplat.

static lib: missing. ATTACH_PROCESS initialization is simulated in Jsrt entry, but DETACH_PROCESS uninitialization is missing.
shared lib: essentially missing as well. Manually called by CH (used to be called during PAL FreeLibrary, changed to manually calling DllMain by jit PR).

One job done in DETACH_PROCESS uninitialization is emitting dynamic profile cache file. After jit PR, "static lib" builds fail to emit dynamic profile files in "interpreter" runs due to this issue, causing "dynapogo" test variant failing to load dynamic profile inputs. This results in 2 seconds for each unit test (10+ minutes total) as the code repeatedly retry loading the dynamic profile input file.

@obastemur


[CI/Build] Update torch to 2.4

This PR updates pytorch to version 2.4. All dependencies that depend on pytorch have also been updated.

FYI: torch 2.4 will install triton 3.0.0, and the _sample_triton kernel will encounter compilation failure issues with triton 3.0.0,  tests/kernels/test_sampler.py can be used to reproduce the issues.

torch is still at 2.3.1. in pyproject.toml, requirements-build.txt

FYI: torch 2.4 will install triton 3.0.0, and the _sample_triton kernel will encounter compilation failure issues with triton 3.0.0, tests/kernels/test_sampler.py can be used to reproduce this issue

@jeejeelee is there a workaround for this? We would like to proceed with this upgrade for a few reasons, if possible

@jeejeelee good news. Looks like we just need to mark a variable as constexpr.

One 2.3.1 reference left here: https://github.com/vllm-project/vllm/blob/main/.github/workflows/publish.yml

Another one here: https://github.com/vllm-project/vllm/blob/052b6f8ca4041f90a1d6825342a4836befbcf478/CMakeLists.txt#L35

@WoosukKwon It looks like the only tests failing in CI are somewhat flaky. I'm hesitant to resolve the merge conflict because I don't want to fire off another full build. What's the best way to proceed here? Are you all comfortable just merging as is or should we do another run after the merge conflict has been resolved?
Also, please let me know if there are any actual issues showing up in CI. I'm always hesitant to write off failing tests as "not related".

@SageMoore Could you resolve the merge conflict and re-enable the CI tests? While I do understand that waiting for the tests to finish is quite painful, I think we should make sure this upgrade does not break anything in vLLM. In our experience, upgrading PyTorch sometimes led to subtle bugs or performance impacts. We should catch these potential issues as much as possible in this PR.

Looks like the entrypoint test consistently fails: https://buildkite.com/vllm/ci-aws/builds/5911#0191064c-b6a0-4ef7-b122-92a481cfeb2b Not sure what the problem is.

I'll see if I can get a local repro

Looks like there's a bit of a hint at the bottom of the test output
[2024-07-31T01:29:46Z]   File "/usr/local/lib/python3.10/dist-packages/vllm/worker/worker.py", line 188, in determine_num_available_blocks
[2024-07-31T01:29:46Z]     assert peak_memory > 0, (
[2024-07-31T01:29:46Z] AssertionError: Error in memory profiling. This happens when the GPU memory was not properly cleaned up before initializing the vLLM instance

I've seen this same error with the tensorizer tests. It looks like there could be some kind of GPU memory leak that's causing these tests to spuriously fail. This could also just be a symptom of an earlier failure. The logs don't give a clear answer AFAICT.
FWIW I'm not able to reproduce these failures on an H100. I'm happy to try other environments if we think that would be helpful.
I'd like to run the entrypoints test group in CI again and see if it consistently fails.

Without this it seems like the PEP517 style cpu builds are broken since for the build the requirement is torch==2.3.1 while requirements-cpu.txt has 2.4.0.
The CPU dockerfile is not affected since it only installs requirements-cpu.txt  and the build requirements are installed manually:
https://github.com/opendatahub-io/vllm/blob/main/Dockerfile.cpu#L19-L20

@SageMoore Now the PR passes the entrypoint test!

While I do understand that waiting for the tests to finish is quite painful, I think we should make sure this upgrade does not break anything in vLLM. In our experience, upgrading PyTorch sometimes led to subtle bugs or performance impacts. We should catch these potential issues as much as possible in this PR.

I'd like to echo this. vLLM, as a prod-ready repo with significant impact, should be very careful here. Unless there is significant benefits & urgency, I'd suggest not dialing up to the latest release immediately, especially the x.x.0 version. It can become very risky, if any hidden bug is there.


I have got no idea what Im doing wrong (commands dont work)

I went through everysingle step in the guide, hitting brickwalls here and their but managed to make it in the end, getting the bot to enter the designated channel, also tried it with the auto playlist that comes with the bot, eventually turning it off. But for some odd reason the bot doesnt respond to any commands what so ever, i honestly dont know what im doing wrong, whether it would be im typing in the commands in the wrong place. Because I have tried every method that i could atleast thing of, i typed in the commands in the discord chat, no respond, i typed in cmd, pythoncmd, and also the cmd scripts and bach's in the music bot but those cant have anything typed into them. :(
thanks in advance for any help ^^

nvm its resolved, i just went through the whole process again i think i messed up something on the notepad++ section of the guide


Organize developmental Jupyter notebooks.

As you are going to be doing a lot of development, it's good to pro-actively have a strategy for organizing your development notebooks. On each branch, create a sub-directory (with a name like development or notebooks or something similiar) to store all .ipynb files. It's also a good idea to name these with names starting with the date e.g. 04_25_20_preprocessing_functions.ipynb.

Created an in_progress folder within the src folder to store .ipynb notebooks that are still in progress. This folder for demonstration purposes has a file that is a work in progress


Buildpack compilation step failed

I try to push a golfing application, but I get the following errors:
Starting app s3pics in org s3pics / space development as admin...
-----> Downloaded app package (163M)
Cloning into '/tmp/buildpacks/go-buildpack'...
Submodule 'compile-extensions' (https://github.com/cloudfoundry-incubator/compile-extensions.git) registered for path 'compile-extensions'
Cloning into 'compile-extensions'...
Submodule path 'compile-extensions': checked out 'f752ecf4b27d2f31bb082dfe7a47c76fefc769d7'
-------> Buildpack version 1.4.0
-----> Resource https://storage.googleapis.com/golang/go1.4.2.linux-amd64.tar.gz does not exist.
-----> Installing go1.4.2...Staging failed: Buildpack compilation step failed
I've tried with CF_TRACE=true, but don't get more useful information.
It looks it can't download the go1.4.2 archive and it could be due to our firewall.
I generally need to install our certificates on clients, so I've tried to install them on several CF VMs, like the controller, the DAE, ... But, I still have the same issue.
Any idea ? Could I install it manually ?

Hi @djannot,
Here's the line in the go-buildpack's compile script that's likely failing:
https://github.com/cloudfoundry/go-buildpack/blob/v1.4.0/bin/compile#L111
Unfortunately the log information is a bit opaque, and it's not entirely clear exactly what URL it's trying to download, whether the curl failed and if so why, or whether the tar failed and if so why.  One thing we could try, if you're able to get a different app running, is to get into another app container and manually try what the buildpack is trying, to see what the cause of the error is.  We can try to figure out if it's networking or something else.
/cc @flavorjones @mhoran (go-buildpack-1.4.0 is failing to fetch golang 1.4.2, not clear why from the error message)

I was able to grab the go tarball linked in the issue without issue. My guess is that the compile is failing due to the firewall, as @djannot suspected. It looks like the -b parameter was provided to cf push, which will use an online buildpack. We'd recommend trying to use one of the built-in buildpacks, or uploading an admin buildpack via the command line. In that case, the buildpack distributed at https://github.com/cloudfoundry/go-buildpack/releases can be used, which is built for offline.

@mhoran I noticed the log line says:
-----> Resource https://storage.googleapis.com/golang/go1.4.2.linux-amd64.tar.gz does not exist.

That URL is the $url variable in the compile script.  However, the actual curl command is this:
(curl `translate_dependency_url $ver` -s --fail -o - | tar xzf - -C $cache/$ver 2>/dev/null)

So we don't know whether translate_dependency_url $ver is the same as $url.
That said, @djannot can you try @mhoran's suggestion of using the offline buildpack, which will include a version of go bundled in, and won't need to fetch it from storage.googleapis.com.

@Amit-PivotalLabs @mhoran Thanks for your help.
I found a workaround.
When I push my app using cf push -b compile-extensions -b go_buildpack s3pics, it's using the local buildpacks, so it works. But it would be nice to allow adding trusted SSL certificates to avoid this kind of issues.

Hi @djannot,
Glad you got your push to work!
It's not clear that the SSL cert is the issue.  It may be, but I don't see conclusive evidence of that, it looks like a number of things could have gone wrong in that curl ... | tar ... command.  Again, if you SSH'd onto the DEA and then start a warden shell session into a currently-working app container (we can give you more details on how to do this) then you can manually run some of the curl and tar commands and see exactly why it's failing.
The root file-system (aka stack or rootfs) for the containers and the base image for the DEA VM (aka stemcell) are designed to be essentially the same for any installation of Cloud Foundry, so if other installations are able to trust the cert, then yours should be to.  It's possible that the problem is upstream, outside of Cloud Foundry, if traffic is going through a network gateway/firewall.
I'll close this issue out for now, if you have further issues feel free to open a new one.
Thanks,
Amit

@Amit-PivotalLabs: While the go-buildpack does utilize translate_dependency_url, our manifest specifies storage.google.com as the location for the buildpack. You'll notice the source and translated URL are identical in the manifest.
Most other buildpacks reference the pivotal-buildpacks S3 bucket, but we left the go-buildpack at its original source location. translate_dependency_url is used in this case to allow offline use -- but that is not enabled when the -b option is provided.


Add config for Giles

This will need to be merged and this GitHub app will need installing: https://github.com/apps/giles
Repo for the app is here: https://github.com/Cadair/giles which is based on https://github.com/astrofrog/baldrick. If anything happens to my hosted version it should be trivial for you to spin up your own.

Woo! Thanks :D


Throughput/Inference Throughput issue in benchmark.py

When running benchmark.py values of throughput printed in the terminal during program execution does not correspond to inference throughput saved in the result file <model_name>_.csv (example patchcore_gpu.csv or patchcore_cpu.csv).
Values printed in the terminal seems to be OK. Values saved in the file however differs quite significantly from them, especially when running testing/inference on GPUs
My setup machine consist of 4GPUs Nvidia A100-SXM4-40GB.

@brm738 thanks for noticing this discrepancy! The models use a callback to calculate inference time and throughput. That's what gets printed on the terminal. The benchmarking script computes time taken without using the callback. Also, The models report batch throughput while the benchmarking reports throughput on sequential images. This is because currently the torch and openvino inferencers are used on streaming/individual images. I'll update the benchmarking script to remove this ambiguity.

Closing as it is addressed in PR #221


New crash on linux when logging meshes

As of https://github.com/rerun-io/rerun/commit/b07149139450f37c584d5fedfd7022f9724461e4 there is a new crash on linux:
This only occurs when we log meshes via the log_mesh API.
Can repro by running;
python examples/python/raw_mesh/main.py

Stack
$ 2023-03-01T22:51:09.178829Z ERROR wgpu::backend::direct: Handling wgpu errors as fatal by default
thread '<unnamed>' panicked at 'wgpu error: Validation Error

Caused by:
    In CommandEncoder::copy_buffer_to_buffer
    Copy error
    copy of 0..23952 would end up overrunning the bounds of the Destination buffer of size 23940

', /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/wgpu-0.15.0/src/backend/direct.rs:3024:5
stack backtrace:
   0: rust_begin_unwind
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/std/src/panicking.rs:575:5
   1: core::panicking::panic_fmt
             at /rustc/d5a82bbd26e1ad8b7401f6a718a9c57c96905483/library/core/src/panicking.rs:64:14
   2: core::ops::function::Fn::call
   3: wgpu::backend::direct::Context::handle_error_nolabel
   4: <wgpu::backend::direct::Context as wgpu::context::Context>::command_encoder_copy_buffer_to_buffer
   5: <T as wgpu::context::DynContext>::command_encoder_copy_buffer_to_buffer
   6: wgpu::CommandEncoder::copy_buffer_to_buffer
   7: re_renderer::allocator::cpu_write_gpu_read_belt::CpuWriteGpuReadBuffer<T>::copy_to_buffer
   8: re_renderer::mesh::GpuMesh::new
   9: re_renderer::resource_managers::mesh_manager::MeshManager::create
  10: re_viewer::misc::mesh_loader::LoadedMesh::load
  11: re_viewer::misc::caches::mesh_cache::MeshCache::load
  12: <core::iter::adapters::map::Map<I,F> as core::iter::traits::iterator::Iterator>::fold
  13: re_query::visit::<impl re_query::entity_view::EntityView<Primary>>::visit2
  14: <re_viewer::ui::view_spatial::scene::scene_part::meshes::MeshPart as re_viewer::ui::view_spatial::scene::scene_part::ScenePart>::load
  15: re_viewer::ui::view_spatial::scene::SceneSpatial::load
  16: re_viewer::ui::space_view::SpaceView::scene_ui
  17: <re_viewer::ui::viewport::TabViewer as egui_dock::TabViewer>::ui
  18: egui::containers::frame::Frame::show_dyn
  19: core::ops::function::FnOnce::call_once{{vtable.shim}}
  20: egui::ui::Ui::scope_dyn
  21: egui_dock::DockArea<Tab>::show_inside
  22: core::ops::function::FnOnce::call_once{{vtable.shim}}
  23: egui::ui::Ui::scope_dyn
  24: re_viewer::ui::viewport::Viewport::viewport_ui

Seems to happen on both wayland and X11.
I went back to the last successful build when we were still getting wheels: https://github.com/rerun-io/rerun/commit/96b30357a4e12dddd12f063b8af6c0905f181707 and it works just fine there.
The issue is somewhere in this stack: https://github.com/rerun-io/rerun/compare/96b30357a4e12dddd12f063b8af6c0905f181707...b07149139450f37c584d5fedfd7022f9724461e4
https://github.com/rerun-io/rerun/pull/1416 seems to be a likely candidate.

likely caused by #1416 since buffer size was adjusted there


Add Account Merge UI to NPSP Contact Merge

Brian Pickett (North Peak)
@Katie McFadden (Swift River Consulting) and @Anne Crawford (KELL Partners) thanks for spelling out your process. This is definitely a challenging process in the Household account model.
I like how in your process when doing a Contact merge with DemandTools users are always looking for contacts within the same account. This should prevent orphaned accounts.
The main scenario that creates the lion's share of challenges using DemandTools is merging accounts when the duplicate is a secondary contact. In these cases its very difficult to find the right accounts to merge first. In your process presumably this is done within Wave 3 (i.e. - eyeball it).
@David Habib the main remaining challenge (not an issue) with the native Contact Merge is the user has no access to select how the Households should be merged. For example, if we're storing important profile/donor-qualification information on the Household record, when merging contacts you can't tell the system which household fields should "win" and you risk losing important information. It seems to cover this the Merge Contacts process would need an interim step where the Households of the selected contacts are displayed in the merge-wizard. So in essence Merge Contacts becomes a Merge-Accounts-then-Merge-Contacts wizard.
Like
·   Today at 8:37 AM   ·
David Habib
David Habib
In the scenario where the household has interesting data that needs merging and control by the user to decide what to do, the solution is to have them use the standard Salesforce Account Merge tool first.
The trigger work I've put into the NPSP to handle this scenario should then do the right thing about removing duplicate addresses and affiliations, as well as moving Opportunities to the winning household account, recalculating rollups, etc.
After merging the household accounts, the user can then either use the standard Salesforce Contact Merge (within an account) functionality, or use NPSP's Contact Merge. Having the NPSP also provide the functionality of Account Merge while in our Contact Merge seems like too heavy a lift given there is a solution provided by Salesforce.
Like
·   Today at 8:44 AM   ·
Brian Pickett (North Peak)
Brian Pickett (North Peak)
Yes, that is do-able but the workflow is ...ahem... "not ideal" in that scenario. So if I find Sally Smith needs merging, I have to click through somewhere else to merge her Account first. Generally account names are generic so finding the right "Smith Household" takes extra steps and is risky (since virtually no identifiable information is provided).
Then I have to repeat the steps using the Contact Merge.
Do-able yes. Tight and simple ~ no.

**lurch: attach W-007294

Tracking W-007294

Related Idea on Hub here.

This has been marked as "Never" internally.


ensure install struct includes proper yaml labels

Signed-off-by: David Cassany<EMAIL_ADDRESS>

Codecov Report

Merging #57 (f18b86c) into main (2381ebc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   27.06%   27.06%
=======================================
  Files           5        5
  Lines         351      351
=======================================
  Hits           95       95
  Misses        250      250
  Partials        6        6

Impacted Files
Coverage Δ

pkg/config/config.go
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


User cannot export the private keys of their btc wallet

Fix the bug where the user cannot export the private keys of their BTC or any non-EVM chain wallet.

Linked PR #71 was closed due to changes being reflected in another PR #73 ,


JBPM-4851: Highlight the 'all' filter by default

The fix + test of the following JIRA:
https://issues.jboss.org/browse/JBPM-4851

Hey @dgutierr ,
I have check it, no compilation neither test issues, but i was expecting to have the "All" filter enabled by default , and the default one is "Active"... was this the fix? If so, it's not working....

note: I used the same jbpm-console-ng-showcase to test it in SDM

@romartin  It seems that you've been checking at the wrong place. The was not about Tasks perspective (where the 'Active'' filter tab activation by default is desirable), but about Dashboards > Process & Task dashboards perspective (the metric displayer 'All' for selection of all visible processes).

+1

master: d42b02a32a0d82010944b70de9090bed5da24517


feat: add clawback vesting account

Description
Closes: #1092
This PR includes the integration for clawback vesting account: an implementation of BaseVestingAccount, but with the clawback feature added.
Current tests are run by calling specific methods directly, I would prefer adding new tests that would test via msg_server if reviewers agree with the need for it as well, or we might as well address it in a separate issue.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

cc @p0mvn for test snapshot with pruning failing here, it seems likely to be unrelated to this PR

cc @p0mvn for test snapshot with pruning failing here, it seems likely to be unrelated to this PR

Created: https://github.com/osmosis-labs/cosmos-sdk/issues/179
Will look into that

Final version of clawback vesting account, which includes all change including proto, messages, cli, tests etc

@p0mvn  Thanks for the detailed in-depth review! Finished going through all the other reviews, would get back w/ you for another round of review after writing tests for DisjuntPeriods, ConjunctPeriods!

@p0mvn Going to merge this decision made on call. Most of your comments were fixed, todos are listed in the newly created issue. Is there anything you think should be fixed upon merging? Or would it be good to merge?

Hi @mattverse ! Thanks for the update and for creating the issue.
I have a question about the following comment: https://github.com/osmosis-labs/cosmos-sdk/pull/178#discussion_r862520887
Do you mind either creating an issue for it or fixing it here before merging? Other than that should be good to go!

I don't see why we should block merge on that

Not suggesting to block merge but let's create issues for unresolved discussions, please

@p0mvn Added in https://github.com/osmosis-labs/cosmos-sdk/issues/219, going ahead to merge this!


[Linux]Fail to debug a function project with JavaScript

Repro Steps:

Create a new project using JavaScript.
Create a new function and add Httptrigger.
Set a breakpoint in the opened index.js file.
Debug.

Expect:
Succeed to debug.
Actual:
Fail to debug the functin project with the below error.

log_message.txt

Yeah so your error in log_message is very similar to the error here: https://github.com/Azure/azure-functions-core-tools/issues/562
Closing as this was fixed a month or two ago in the func cli. Please re-open if it still repros after updating the func cli (I think you need at least version 2.0.1-beta.34)


layers.LocallyConnected2D takes a lot of time to get initialized

Tensorflow version: 2.7.0
layers.LocallyConnected2D takes an awful amount of time to build. The time taken to build increases dramatically with increasing size of input.

LocallyConnected2D in mode 1 takes 11.0303 seconds which is 43.0397 times Normal Conv2D
LocallyConnected2D in mode 2 takes 6.6656 seconds which is 26.0089 times Normal Conv2D
LocallyConnected2D in mode 3 takes 4.1878 seconds which is 16.3407 times Normal Conv2D

Here mode means implementation.
Sample code to generate the problem can be found in this notebook
Moreover the sparse tensor generated in implementation mode 2 uses sparse_mat_mul which is deprecated.

@sachinprasadhs Was able to replicate the issue on colab using TF v2.7.0 ,please find the gist here for reference.Thanks!

It may be due to the different approach as mentioned here.
The LocallyConnected2D layer works similarly to the Conv2D layer, except that weights are unshared, that is, a different set of filters is applied at each different patch of the input.


[QTC] Make compliant with --future from #49

As --future returns QSR dictionaries, the only QSR not complying is QTC as there were some things not clear to me when I try to edit it.
Either @yianni to have another look or @cdondrup
Not high priority at the moment.

I can do that.

@cdondrup Just to ask how are you thinking of doing it? My feeling would be something as follows: qtcb {"qtcb": <relation>}, qtcc {"qtcc": <relation>} and qtcbc {"qtcb": <relation>, "qtcc": <relation>}.
How does this sound? Let me know when you think about it.

This is not really the purpose of qtcbc. If I wanted that I could just get qtcb and qtcc. Deciding when to transition is quite important here. So it will be {"qtcbc": <relation>}.

Hmmm. Are these QTCs here different that what is described in section 3 of a later paper from Van de Weghe 2004, which provides a concise and clear description of QTC2d link.

qtcbc is because that was our invention. For qtcb and qtcc I currently only use two points in time instead of three which is a more simplified version of it.

This is what I am doing: http://www.mdpi.com/2218-6581/4/1/63

This is what I am doing: http://www.mdpi.com/2218-6581/4/1/63

OK, I will have a look. Just the name implies that qtcbc = join(b, c). Anyway, do as you see fit.

Which is what it does ;) But with a bit of logic behind it.


Torrentio Links do not seem to work

Me and a few other people on Reddit are having trouble with Torrentio in the past couple of days.
Could you maybe check your hosting to see what may be going on? If not I'd be willing to put up a seedbox of my own to balance between.

Hi, it seems it's related to proxy used for realdebrid requests. Some of the proxy hosts seem to freeze up. I'll try to investigate and for now I've added a timeout so that not all requests would fail and remove identified slow proxy hosts.

Thanks for your response. That's good to hear you identified the issue. One thing has me wondering, should real debrid being down have any effect on the torrent links I was receiving? Maybe they were just bad torrents?

Links without the [RD+] prefix are not affected by real debrid. Just the torrent's infohash is returned, which is then used by stremio's local server to stream that torrent.


Add shell: true to TSP install command.

This is a breakage due to a breaking change in node's latest incremental version. (See https://github.com/nodejs/node/issues/52554 for background). Addint shell: true fixes this.
Fixes #3242.

Note that I don't have a mac to test in, but I tested this in WSL as well, and it works (although it works without the fix too).

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

Show changes


adding documentation of the usage Hashicorp Vault for both verify and sign sub-commands, used clean sub-command in USAGE.md instead of using both delete and triangulate commands

Signed-off-by: Batuhan Apaydın<EMAIL_ADDRESS>TLDR;

used clean sub-command instead of using both delete and triangulate commands in USAGE.md
added usage of Hashicorp Vault KMS support to the description parts of both verify and sign sub-commands

NOTE: I did the test to make sure everything is working properly, and it worked.

Thanks!


OneDrive  having trouble with folder names..

What is your rclone version (eg output from rclone -V)
1.36

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Debina Linux  (buster/sid)

Which cloud storage system are you using? (eg Google Drive)
Onedrive

The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone -W copy /home/verwijs/Dropbox/Public/Windows Onedrive:Openbaar/Windows

A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)
there' s no -W  flag ....

instead I run:
rclone --log-file rclone.log --log-level DEBUG copy /home/verwijs/Dropbox/Public/Windows Onedrive:Openbaar/Windows
output rclone.log: (see attachments with screenshot)
Issue:
whe copy folder with white space, rclone adds duplicate folders and folder with a "+" character instead of a white space (using a "\" character)
screenshot:

added log file....
rclone.log.txt

This is fixed already in the latest beta which I'm going to release as 1.37 this weekend hopefully.

ok :) thank you..

The fix is now in 1.37 - just released.  I'm going to close this now - please re-open if you are still having a problem.

ok :) thank you..

Thank you very much for this!