error: Uncaught (in promise) Error: bad-yaml

I accidentally typed a pwsh command in bash via code-insiders, and

rokejulianlockhart@RQN6C6:~> Get-Command qmlformat
error: Uncaught (in promise) Error: bad-yaml
    if (!isArray(node)) throw new Error("bad-yaml");
                              ^
    at Object.getProvides (file:///opt/tea.xyz/src/v0.22.1/src/hooks/usePantry.ts:133:31)
rokejulianlockhart@RQN6C6:~>

appeared.

so yeah I'll fix this, but first need to understand where you tea binary is thanks

@mxcl,
PS /home/rokejulianlockhart/RTBJK9/code/account_manager/code> which tea
/home/rokejulianlockhart/.tea/tea.xyz/v0.33.2/bin/tea
PS /home/rokejulianlockhart/RTBJK9/code/account_manager/code> command -v tea
PS /home/rokejulianlockhart/RTBJK9/code/account_manager/code> Get-Command tea

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Application     tea                                                <IP_ADDRESS>    /home/rokejulianlockhart/.tea/tea.xyz/v0.33.2/bin/tea

PS /home/rokejulianlockhart/RTBJK9/code/account_manager/code>

I believe this is now fixed, you have upgraded and now I re-read things you were just on an old tea/cli.
once we're v1 we won't break pantry back-compatability but for now this does happen.
Certainly we should advise the user to update tea/cli if it is old. We have a ticket tracking that.
Reopen if I'm wrong abotu this being fixed in new terminal sessions. Thanks!

PowerShell 7.4.0-preview.3
PS /home/rokejulianlockhart> bash
rokejulianlockhart@RQN6C6:~> Get-Command
tea: command not found: Get-Command
rokejulianlockhart@RQN6C6:~>

Indeed, fixed.


Update PPP for v1.4

Codecov Report

Merging #259 (cb6b0cd) into master (7fce4b8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #259   +/-   ##
=========================================
  Coverage     54.04%   54.04%
  Complexity      530      530
=========================================
  Files           102      102
  Lines          2383     2383
  Branches        320      320
=========================================
  Hits           1288     1288
  Misses          970      970
  Partials        125      125

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


Group member count incorrect

Describe the bug
New groups are one member off. Not sure if it's not counting the admin, but this group has 3 members, but only lists 2: https://hcommons-staging.org/groups/daves-group-2023/members/
Old groups appear to be unaffected: https://hcommons-staging.org/groups/mesh/members/
To Reproduce
Steps to reproduce the behavior:

Join any group
Count is off

Expected behavior
The member count should match the number of members listed in the membership list.
Context
Does this bug on production, development, or both?

[x] Tested on development instance
[x] Occurs on development instance
[x] Tested on production
[ ] Occurs on productions

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

BP_Groups_Group::get_total_member_count() is the function responsible for calculating the number of group members. It excludes spammed, blocked, and inactive users. It queries for active members by querying the wp_bp_activity table for each user's last activity.
In the case of these groups, the bonnie user somehow has two last_activity records:
mysql> SELECT id,user_id,component,date_recorded FROM wp_bp_activity WHERE user_id IN (1015176,1049869,1049868) AND type = 'last_activity';
+---------+---------+-----------+---------------------+
| id      | user_id | component | date_recorded       |
+---------+---------+-----------+---------------------+
| 1615479 | 1015176 | members   | 2023-01-23 16:49:40 |
| 1766734 | 1015176 | members   | 2022-08-18 15:43:22 |
| 1785017 | 1049868 | members   | 2023-01-23 16:48:02 |
| 1785020 | 1049869 | members   | 2023-01-23 20:56:09 |
+---------+---------+-----------+---------------------+

(Bonnie has user_id 1015176)
Because of how the query is constructed, this ends up "pushing out" another user from the count and ultimately reducing the group count by 1.
The most relevant code is in BP_Core_User::get_last_activity():
                 $uncached_user_ids = bp_get_non_cached_ids( $user_ids, 'bp_last_activity' );
		if ( ! empty( $uncached_user_ids ) ) {
			$bp = buddypress();

			$user_ids_sql = implode( ',', $uncached_user_ids );
			$user_count   = count( $uncached_user_ids );

			$last_activities = $wpdb->get_results( $wpdb->prepare( "SELECT id, user_id, date_recorded FROM {$bp->members->table_name_last_activity} WHERE component = %s AND type = 'last_activity' AND user_id IN ({$user_ids_sql}) LIMIT {$user_count}", $bp->members->id ) );

			foreach ( $last_activities as $last_activity ) {
				$success = wp_cache_set( $last_activity->user_id, array(
					'user_id'       => $last_activity->user_id,
					'date_recorded' => $last_activity->date_recorded,
					'activity_id'   => $last_activity->id,
				), 'bp_last_activity' );
			}
		}

		// Fetch all user data from the cache.
		$retval = array();
		foreach ( $user_ids as $user_id ) {
			$retval[ $user_id ] = wp_cache_get( $user_id, 'bp_last_activity' );

			if ( isset( $retval['user_id'] ) ) {
				$retval[ $user_id ]['user_id']     = (int) $retval[ $user_id ]['user_id'];
			}
			if ( isset( $retval['activity_id'] ) ) {
				$retval[ $user_id ]['activity_id'] = (int) $retval[ $user_id ]['activity_id'];
			}
		}

In the upper stanza, the query has a LIMIT clause that limits results to 3. Because bonnie has 2 records, there would be 4 results from this query if the LIMIT clause wasn't there.
In the bottom stanza, the count is done by looking at which users had results come back from the previous query. Only 2 do, and so the count is 2 rather than 3.
This is all pretty deep in BuddyPress code and doesn't seem to be the result of any of our code interfering with the calculation.
It might be a quirk or our production-development sync process that caused bonnie to have multiple last_activity records.
As this is not a breaking issue, I'm closing and we can check if it occurs in production. If it does, the best first course of action is probably to run a script that de-duplicates last_activity records on the assumption that there was some one-time integrity failure with the database.


MWPW-122611 - [GH Action] Old browser function detection

defines a workflow to detect unsupported functions

Resolves: MWPW-122611

What browsers/versions is this checking?  Also this is scoped to the old-browser-support branch, should it be main?  When testing are there currently unsupported functions in Milo?

This PR was a work-in-progress, not meant to be reviewed, sorry! I will close it and open a new one when ready.


Include task runner

Example of usage:
runner := NewRunner(tasks...).WithLimit(3)
err := runner.Run(ctx)

Closes https://github.com/StudioSol/async/issues/2
Closes https://github.com/StudioSol/async/issues/3

Codecov Report

Merging #7 into master will increase coverage by 9.62%.
The diff coverage is 96.49%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage    82.6%   92.23%   +9.62%
==========================================
  Files           1        2       +1
  Lines          46      103      +57
==========================================
+ Hits           38       95      +57
  Misses          5        5
  Partials        3        3

Impacted Files
Coverage Δ

runner.go
96.49% <96.49%> (ø)

async.go
86.95% <0%> (+4.34%)
:arrow_up:

Continue to review full report at Codecov.

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

@vinicio :recycle:


Accessibility: Dates unreachable via keyboard when inline set to true

Describe the bug
When the calendar is displayed inline (inline set to true), the calendar dates seem to be unreachable via keyboard. E.g. tabbing to the date and using arrow keys to navigate doesn't seem to work.
To Reproduce
This can also be seen on the examples website.
Steps to reproduce the behavior:

Go to https://reactdatepicker.com/#example-inline-version (Inline version example)
Click on one of the month arrows to visibly have the focus there
Click tab to go into dates
See the focus going out of the calendar

Expected behavior
Dates should be accessible via keyboard independent of the calendar having inline set to true.
Screenshots
Tabbing here:

Desktop (please complete the following information):

OS: iOS
Browser: Chrome

Trying to use the calendar inline in a plugin, however, the keyboard accessibility doesn't seem to work in case of inline.
.
Assuming it's a bug since it doesn't work on the demo website either, however, it would be good to know if it needs a special configuration instead.

@miina I am seeing the same behavior with inline. For me when i navigate with arrows through the days it seems to actually be moving but the class is not getting applied to the element that will show the styling that it is keyboard selected. There is a specific check for inline that will always return false for keyboard selected

I am not sure of the history of why that is there.


Fix remaining docs references to slice field

We missed some uses of slice in the docs when we made the changes in #197 .  In particular, I see several references in the "Usage Guide" page.
I'd like someone to replace any remaining references with name, per the API docs.

Can I take this?


Ionic v1.2.4, ".ng-enter-stagger" fails after $ionicModal instantiated.

I updated my ionic lib to 1.2.4, and I am using the latest animate.css with the following properties to animate ng-repeat
.chat-item.ng-enter {
    -webkit-animation: fadeInLeft 1s;
    animation: fadeInLeft 1s;
}
.chat-item.ng-enter-stagger {
  -webkit-animation-delay:200ms;
  animation-delay:200ms;

  /* override to make sure it's not inherited from other styles */
  -webkit-animation-duration:0;
  animation-duration:0;
}

the animation works fine at first, but suppose I instantiate $ionicModal, it stops working. I tested the same with ionic v1.1.1 and it works fine

Hello! Thanks for opening an issue with us! Would you be able to provide a plunkr or codepen for us to use to reproduce this issue?

As it has been a little while since there was any activity on this issue I will be closing it for now, please feel free to reply if you are still experiencing this issue though!


feat: switch to using datafusion to parse functions

My original plan was to use DF for parsing expressions (not just functions).  However, I ran into some issues:

https://github.com/apache/arrow-datafusion/issues/7249
Datafusion does not support nested nested references (e.g. it supports x.y but not x.y.z)
Array literals always appear as List and not FixedSizeList

Datafusion's array literals initially show up as "MakeArray" calls and so they are missed by our type coercion.  They turn into literals later during expression simplification.

Functions from duckdb (e.g. is_valid) are not recognized by datafusion

Long term this kind of engine-to-engine conversion is best handled by substrait
In the short term we could maybe register these functions as DF UDFs

So this PR only uses DF to parse the functions.  In addition, we keep the legacy path to handle the is_valid function for DuckDb.
Closes #1784

Closes #1784

I've updated #1129 with the incompatibilities I observed this time.

Datafusion does not support nested nested references (e.g. it supports x.y but not x.y.z)

What do you mean? can't you just nest GetIndexedField calls?

Does this fix https://github.com/lancedb/lance/issues/1115?

Does this fix #1115?

Looks like it.  I've added a regression test for array_contains

(Sorry just merged a PR that added some conflicts. Should be easy to resolve though.)


node-sass: workaround missing vendor folder

After a fresh install:
git clone https://github.com/plotly/react-chart-editor.git
cd react-chart-editor
npm install

running npm run make:lib:css fails because of  issue https://github.com/sass/node-sass/issues/1579.
This PR adds a postinstall script to rebuild node-sass and fix this issue.

Just for context: when would you want/need to run npm run make:lib:css as a consumer of this library? Or is this just for clean installs of the library itself...?


Tabs with all the nested pages with lazy loading ion-content doesn't render

When I create a tabs page with all the pages with lazy loading, the  of every page doesn't render, just renders the  just above the tabs section.
If I import the page with regular components works just fine.

Is this behaviour correct ?

same issue here


refactor(@vtmn/css): add line-heights to ctas (WIP)

Pull Request checklist
🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch. Don't request your main!
[x] Make sure you are making a pull request against the main branch (left side).
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.

Description
Add line-heights to cta in order to give them a fitting default height.
According to design-system statements :
S should be 32px
that's 2rem
that's 0.875rem (font-size) * ~1.2857 (line-height) + 0.4375rem (padding-top) + 0.4375rem (padding-bottom)
M should be 48px
that's 3rem
that's 1rem (font-size) * 1.25 (line-height) + 0.875rem (padding-top) + 0.875rem (padding-bottom)
L should be 64px
that's 4rem
that's 1.25rem (font-size) * 1.2 (line-height) + 1.25rem (padding-top) + 1.25rem (padding-bottom)

@crotru FYI, we don't have a WIP plugin. Instead, you can use the draft PR built-in feature in GitHub.


missing static option in docs

Environment
/
Reproduction
/
Describe the bug
Hello,
In the configuration of the documentation, https://nitro.unjs.io/config, there is nothing related to a static option.
But as I can see here, https://github.com/unjs/nitro/blob/451d314657f0181bf065c413039509440df39c2b/src/options.ts#L187, or in this comment, https://github.com/nuxt/nuxt/pull/22471#discussion_r1283205835, this option is mapped to the static preset, https://github.com/unjs/nitro/blob/main/src/presets/static.ts.
We are mentioning it in the Nuxt documentation so having a link in the Nitro documentation would be appreciated!
Thanks
Additional context
No response
Logs
No response

Thanks for notice. Since we introduce it as an internal flag, initially didn't document. Feel free to make a PR to add docs 👍🏼 (also there might be more of this missing API docs)

I was documenting but I found that the static option does not have the same behavior as the preset: 'static'.
esteban@DESKTOP-HHLH0C1:~/dev/unjs/nitro$ nr dev:build

><EMAIL_ADDRESS>dev:build /home/esteban/dev/unjs/nitro
> pnpm nitro build playground

><EMAIL_ADDRESS>nitro /home/esteban/dev/unjs/nitro
> jiti ./src/cli/cli.ts "build" "playground"

undefined node-server true
static
static
✔ Generated public playground/.output/public                                                                                nitro 7:35:15 AM
✔ You can preview this build using node playground/.output/server/index.mjs                                                                                nitro 7:35:15 AM
esteban@DESKTOP-HHLH0C1:~/dev/unjs/nitro$ NITRO_PRESET=static nr dev:build

><EMAIL_ADDRESS>dev:build /home/esteban/dev/unjs/nitro
> pnpm nitro build playground

><EMAIL_ADDRESS>nitro /home/esteban/dev/unjs/nitro
> jiti ./src/cli/cli.ts "build" "playground"

undefined node-server true
static
static
✔ Generated public playground/.output/public                                                                                nitro 7:35:21 AM
ℹ Initializing prerenderer                                                                                nitro 7:35:21 AM
false node-server false
nitro-prerender
nitro-prerender
ℹ Prerendering 1 initial routes with crawler                                                                                nitro 7:35:22 AM
  ├─ / (8ms)                                                                                nitro 7:35:22 AM
✔ You can preview this build using npx serve playground/.output/public

It's like the static preset was not apply on config. Maybe it's the intended behavior but it looks strange.
esteban@DESKTOP-HHLH0C1:~/dev/unjs/nitro$ nr dev:build

><EMAIL_ADDRESS>dev:build /home/esteban/dev/unjs/nitro
> pnpm nitro build playground

><EMAIL_ADDRESS>nitro /home/esteban/dev/unjs/nitro
> jiti ./src/cli/cli.ts "build" "playground"

undefined node-server true
static
static
✔ Generated public playground/.output/public                                                                                nitro 7:39:34 AM
{
  concurrency: 1,
  interval: 0,
  failOnError: false,
  crawlLinks: false,
  ignore: [],
  routes: []
}
✔ You can preview this build using node playground/.output/server/index.mjs                                                                                nitro 7:39:34 AM
esteban@DESKTOP-HHLH0C1:~/dev/unjs/nitro$ NITRO_PRESET=static nr dev:build

><EMAIL_ADDRESS>dev:build /home/esteban/dev/unjs/nitro
> pnpm nitro build playground

><EMAIL_ADDRESS>nitro /home/esteban/dev/unjs/nitro
> jiti ./src/cli/cli.ts "build" "playground"

undefined node-server true
static
static
✔ Generated public playground/.output/public                                                                                nitro 7:39:38 AM
{
  concurrency: 1,
  interval: 0,
  failOnError: false,
  crawlLinks: true,
  ignore: [],
  routes: []
}
ℹ Initializing prerenderer                                                                                nitro 7:39:38 AM
false node-server false
nitro-prerender
nitro-prerender
ℹ Prerendering 1 initial routes with crawler                                                                                nitro 7:39:39 AM
  ├─ / (8ms)                                                                                nitro 7:39:39 AM
✔ You can preview this build using npx serve playground/.output/public


Build fails for PC, Mac & Linux Standalone

Describe the bug
Building for PC, Mac & Linux Standalone fails with an error.
To reproduce

Open a project with MixedRealityToolkit
Build for PC, Mac & Linux Standalone
Observe error

Assets\MixedRealityToolkit.Examples\Demos\HandTracking\Script\SystemKeyboardExample.cs(64,41): error CS0117: 'TouchScreenKeyboard' does not contain a definition for 'visible'
Your Setup

Unity Version 2018.3.5 & 2019.1
MRTK branch mrtk_development

Fixed literally as I was typing this up.


Draw Three alignment not working properly

=======TALON VIEW===========
[V] [F]	7 of SPADES	[22]
[V] [F]	13 of DIAMONDS	[21]
[V] [F]	10 of SPADES	[20]
[=======BLANK=======]	[19]
[H][F]	11 of HEARTS	[18]
[H][F]	8 of DIAMONDS	[17]
[H][F]	11 of SPADES	[16]
[H][F]	7 of CLUBS	[15]
[H][F]	2 of CLUBS	[14]
[H][F]	6 of DIAMONDS	[13]
[H][F]	12 of HEARTS	[12]
[H][F]	1 of HEARTS	[11]
[H][F]	12 of DIAMONDS	[10]
[H][F]	3 of SPADES	[9]
[H][F]	2 of HEARTS	[8]
[H][F]	9 of SPADES	[7]
[H][F]	10 of DIAMONDS	[6]
[H][F]	4 of HEARTS	[5]
[H][F]	9 of HEARTS	[4]
[H][F]	6 of SPADES	[3]
[H][F]	12 of CLUBS	[2]
[H][F]	7 of HEARTS	[1]
[V] [F]	6 of HEARTS	[0]
========TABLEAUVIEW========
========TABLEAUVIEW========
[V] [F]	3 of DIAMONDS	[2]
[V] [F]	4 of SPADES	[1]
[V] [B]	5 of DIAMONDS	[0]
========TABLEAUVIEW========
[V] [F]	13 of CLUBS	[0]
========TABLEAUVIEW========
[V] [F]	8 of HEARTS	[6]
[V] [F]	9 of CLUBS	[5]
[V] [F]	10 of HEARTS	[4]
[V] [F]	11 of CLUBS	[3]
[V] [B]	2 of DIAMONDS	[2]
[V] [B]	8 of SPADES	[1]
[V] [B]	1 of CLUBS	[0]
========TABLEAUVIEW========
[V] [F]	4 of DIAMONDS	[5]
[V] [F]	5 of SPADES	[4]
[V] [B]	8 of CLUBS	[3]
[V] [B]	5 of CLUBS	[2]
[V] [B]	3 of CLUBS	[1]
[V] [B]	7 of DIAMONDS	[0]
========TABLEAUVIEW========
[V] [F]	4 of CLUBS	[4]
[V] [F]	5 of HEARTS	[3]
[V] [B]	12 of SPADES	[2]
[V] [B]	9 of DIAMONDS	[1]
[V] [B]	11 of DIAMONDS	[0]
========TABLEAUVIEW========
[V] [F]	10 of CLUBS	[5]
[V] [B]	13 of HEARTS	[4]
[V] [B]	3 of HEARTS	[3]
[V] [B]	6 of CLUBS	[2]
[V] [B]	1 of DIAMONDS	[1]
[V] [B]	13 of SPADES	[0]
========FOUNDATIONVIEW========
[V] [F]	2 of SPADES	[1]
[V] [F]	1 of SPADES	[0]
========FOUNDATIONVIEW========
========FOUNDATIONVIEW========
========FOUNDATIONVIEW========
=========OptionsPreferences=========
Draw Option: THREE
Scoring Option: VEGAS
Timed Game: true
Status Bar: true
Outline Dragging: false
Cumulative Score: false
Deck: DECK_1

d

The bug has to do with the fact that the [V][F] 6 of Hearts [0] is still visible, so because of this entry there is an offset that is messing up the position


Consider making it easier to detect if an event is returned, or if there's no event, or if it's malformed.

As part of:
https://github.com/knative/eventing/pull/3450
(and the discussions around how to handle empty events...)
Wanted to float by changing the behaviour of:
https://github.com/cloudevents/sdk-go/blob/master/v2/binding/to_event.go#L23
For handling empty responses differently from a malformed event, wanted to see if we could possibly change the semantics of the return to make it easier for the users to reason about the returned events.
As a user I was expecting the following semantics, and wanted to see if we could change to be so:
nil, nil => There was no event and no error.
Currently the user has to check the encoding, read the body/close it and see if any bytes were there themselves. By having the ToEvent code return a nil event and nil error to indicate there was no event there seems nice.

Currently the user has to check the encoding, read the body/close it and see if any bytes were there themselves. By having the ToEvent code return a nil event and nil error to indicate there was no event there seems nice.

This behaviour is not defined in the spec, so i don't think it should be part of the sdk. For the spec, a request/response is a cloudevent if has ce- headers or a content-type matching an event format, otherwise it's a non-event. I think it's reasonable for the sdk to mark a message non-event (marking it with binding.EncodingUnknown) as soon as both conditions are false.
If you want to do a particular distinction of no cloudevent 200 with body and no cloudevent 200 without body, then you should handle it specifically in knative. To sdk-go, what makes sense is the only distinction made by the spec: event/non-event.

nil, nil => There was no event and no error.

This is something doable but I think we need to wait v3 for that, since it could break a looot of things


example of how to receive on a channel

I see hints in the luar.go code that channels are supported, but I don't fully get how to do that in lua code.  If it isn't too much trouble, a simple example of how to send from go on a channel and receive in lua (and vice-versa) would be super helpful.
Thanks!
Jason

They are hints because it didn't fly! Naively thought  that one could drive
a Lua coroutine from a Go goroutine but Lua is not reentrant. Would need to
run a separate Lua state in a go routine only communicating with other Lua
code through channels and (probably) shared aliases to Go objects (which
would need mutexes). I'll have a look at that when I'm back at my machines

They are hints because it didn't fly! Naively thought  that one could drive
a Lua coroutine from a Go goroutine but Lua is not reentrant. Would need to
run a separate Lua state in a go routine only communicating with other Lua
code through channels and (probably) shared aliases to Go objects (which
would need mutexes). I'll have a look at that when I'm back at my machines

Ah. Perhaps Luvit https://github.com/luvit/luvit could be of help.

Ah. Perhaps Luvit https://github.com/luvit/luvit could be of help.

Luvit is pretty much a NodeJs clone so it's all about callback soup.  A
project to follow here is Lua Lanes which has independent Lua states
communicating on channels. Share-nothing is easier to think about I find.

Working on this...
@stevedonovan Can you explain what you mean by "drive a Lua coroutine from a Go goroutine"?

Lua (as you know) is fundamentally single-threaded. But coroutines can be driven by an event scheduler and an effective illusion of synchronous multi-threading is created - e.g the way COPAS works with LuaSocket. The trouble with goroutines is that they are sometimes backed by an OS thread, sometimes not  - this is also their beauty (much more scalable). You can safely have a Lua state per goroutine, but then Lua state cannot be shared and it has to move around through messaging. Then you have the marshalling problem, which Lua Lanes solves for basic Lua types.  I think it's worth really thinking about this because it would be fantastic for the Lua/Go ecosystem.

Hmmm, so if I understand correctly, your goal was to "parallelize" Lua coroutines by running each of them on individual goroutines? That is problematic indeed, considering the non-reentrant nature of Lua states.
I believe, however, that it would be as cool as it would be complicated to implement... Doesn't it go beyond the goals of Luar?

Ah yes - and not entirely sure it can be done robustly!  But separate Lua states would always be cool.
As for project scope, I tend to agree with you - it is a reflection-based Lua/Go interface, and should probably stick to doing that well.
One could imagine each Lua state having a proxy to some shared Go data, but it would probably be a mess of mutexes.

OK, then I'll scrap your channel draft (the channel_* functions and GoLua) and implement a straightforward conversion. My first test looks good and simple.

Good idea! Stuff that doesn't work well (or at all) shouldn't be in the public interface.

Merged: https://github.com/stevedonovan/luar/commit/68e746ca8777572f7e1caa6adc33adcdd64d0138


marker: "arrow-reverse" is the same as marker: "arrow" for Line mark

To reproduce open https://observablehq.com/plot/features/markers and use both "arrow" and "arrow-reverse", results look the same as of today.

It’s different, but only for the arrow at the start of the line, so it’s easy to not notice.


Replace spotify docker-client with an alternative

The client currently used is effectively EOL: https://github.com/spotify/docker-client
It also has a lot of dependencies on other libraries which cannot be upgraded (e.g Jersey) who themselves have reported vulnerabilities (which may or may not be relevant to this plugin)
Probably should move to https://github.com/docker-java/docker-java ?

Sadly the current client version is broken with Docker Engine 25.0 (api 1.44) due to dropping of the VirtualSize field which is mandatory within the client. So will need to find a suitable fork, fork it, or switch.


Links in both code and documentation still point to code.google.com

Since the closure of code.google.com both this project and some of the related projects have moved to github.  At the time of writing anybody following the existing links will get a page telling them where the code has moved to but:

this requires an unnecessary click through by the user,
depends on Google continuing to supply this information and
just plain annoys me

Closing as out dated.


Fixed #28892 -- Removed fixed height for navbar in admin

Replaced the fixed height for the navigation in the admin
with flex to allow handling of more dynamic content while
still remaining vertically centered.

@elky, care to review?

Looks good to me! Thanks @sheins


[macOS] Add xcode-14.2

Description
Add xcode-14.2
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run macos-12

/azp run macos12

@maxim-lobanov your step does not work for 14.2 on macos-12, but you approved these changes so I'm assuming you helped test it. Do you happen to have further information on why it wouldn't be working?
We're not getting a reply on #6762.


[DOC] Fully capitalize NVIDIA

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[ ] I've applied the relevant labels to this PR
[ ] I've assigned a reviewer

PR contents

[x] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[x] I've added relevant tests for my contribution
[x] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

Per the brand guidelines of the company, NVIDIA must always be fully capitalized.
Linked issues
N/A

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.1%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
4640

Relevant Lines:
5866

💛  - Coveralls


wiki should include information on how to update the charts

The wiki should have the same info found here: https://github.com/istio/istio/blob/master/manifests/UPDATING-CHARTS.md
@sdake recommended this issue be created and assigned to him.

There should be an additional blurb on how to test. Particularly, it seems it best that you build istioctl. As per @dgn make istioctl and then use the binary in out/.


@conda-forge-admin, please rerender

Additional comment:
Opening a PR to see if we even need https://github.com/conda-forge/r-rjava-feedstock/pull/32 after https://github.com/conda-forge/admin-requests/pull/498 .

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/r-rjava-feedstock#34.


Update needed for openxcom-oxce

Please update openxcom-oxce to version 7.7.6 (commit 3eb2a81). Some mods like The X-Com Files require at least version 7.7. Thank you!

@dlpierce Thanks for the issue! I've updated with this PR: https://github.com/luxtorpeda-dev/packages/commit/6b7c3ed258a90a43933fbbc9c61b7ec8aba2b36f
It should be live about 20 minutes after this comment. Let me know if you need anything.

Thank you, works great!


Document ProxyIO and implement own :io.request/1

I've made new branch for upcoming PR: https://github.com/elixir-lang/elixir/compare/master...lexmag:opportunistic-logging-pr, and cherry-picked this commit.


Create DD Mauro Training Environment

When DD testing has been done in the NHS Digital instance, training will be required to other DMDS staff.
DMDS have discussed with Data Products the set up of  training environment.
Cath and Angie will notify Data Products when this is required.

Jira ticket: https://nhsd-jira.digital.nhs.uk/browse/DPTC-3059

This is dependent on whether we own this (Data Solutions) or the supplier.

21/05/24 - Gouri and Steve are working on this with Data Solutions.

The training environment now exists and can be found at:
Mauro: https://mauro.uat.dataproducts.nhs.uk/
Orchestrator: https://mauro.uat.dataproducts.nhs.uk/orchestration/
The relevant people have been notified with sign-in details.

See my comment from #325. I need confirmation that your penetration testing has completed on your Training environment before we can progress further.

See #325, Naresh has confirmed that Pen testing gas been completed.

@NellySSJ and @GSChandel - should this now be closed?

@AngelaFaulding are you content that the training environment created is suitable for your needs as per this ticket's objectives? If yes, then close it.

The TRAINING environment is up to date and contains the "August 2024" release branch. The environment is ready to use.

@GSChandel  - we've haven't tested it, just checked that we could access it.
Depending on when we migrate there may be another dictionary release to ingest.
I'll put the ticket in backlog until we know for definite.

As per recent emails, the August 2024 release needs removing from the Training instance and reloading. Also CR1914 needs ingesting to allow the CR tests to continue.
James is unable to do this as he doesn't have access. Without this we cannot test the issues James has fixed.


fix #316 huntorrent.org

https://github.com/hufilter/hufilter-dev/issues/316

@Alex-302 Our filter has limited support for sites that require registration. I'm also not a member of this site, and I haven't tested the rules. I've closed the related issue with wontfix label, but the author, @Hulayki sent me the rules in a comment and theoretically he has tested them.

@Hulayki
please test huntorrent.org##.text-center > a[rel="nofollow"] > img? Your element hiding rule does not look safe.

@Alex-302 Not working. This is what that section looks like:

Because originally there is a Google ad:

Could you please expand this element?

@scripthunter7

@Hulayki Please test
huntorrent.org##.text-center:has(> .adsbygoogle)
huntorrent.org##a[href*="click_banner.php"]

@scripthunter7 Everything is clear! :)

We should consider this issue before merging the PR: https://github.com/hufilter/hufilter-dev/issues/357


Reoccurring errors with checkpointing

Hi!
We are currently running Arroyo version 0.11.0. After launching a pipeline and allowing it to run for an extended period, the checkpointing process appears to become stuck.

Hi @hazelnut-99 — thanks for reporting! Can you provide some more details about how you're running Arroyo? In particular:

What scheduler are you using?
Where are checkpoints being stored (a local filesystem? s3? somewhere else?)
What does the query look like?

Thanks!

Scheduler: Kubernetes Scheduler
Checkpoint storage: local filesystem
Query:
CREATE TABLE my_kafka_sink (
    field_1 TEXT,
    field_2 TEXT,
    field_3 DOUBLE,
    ts_1 BIGINT,
    ts_2 BIGINT
) WITH (
    'connector' = 'kafka',
    'avro.confluent_schema_registry' = 'true',
    'bootstrap_servers' = 'my_servers',
    'schema_registry.endpoint' = 'my_endpoint',
    'type' = 'sink',
    'topic' = 'my_topic',
    'format' = 'avro'
);

INSERT INTO my_kafka_sink
select table_1.field_1, table_1.field_2,
    (table_2.field_y - table_1.field_z) as field_3,
    table_1.ts as ts_1, table_2.ts as ts_2
from
(
    select field_1, field_2, hop(INTERVAL '5' second, INTERVAL '3' minute) as window,
    last_value(field_z) as field_z, last_value("timestampMillis") as ts
    from connection_1
    group by 1, 2, 3
) as table_1 join
(
    select field_1, field_2, hop(INTERVAL '5' second, INTERVAL '3' minute) as window,
    last_value(value_latest) as field_y, last_value("timestampMillis") as ts
    from connection_2
    group by 1, 2, 3
) as table_2
on table_1.field_1 = table_2.field_1 and
    table_1.field_2 = table_2.field_2 and
    table_1.window = table_2.window

And to double check is this a distributed kubernetes cluster or a local one (like running in minikube or similar)?
All nodes in the cluster need to be able to read from the state, so if you're running on a distributed kubernetes cluster it's necessary to use some kind of distributed state storage (typically this will be an object store like S3 but could be a shared filesystem like NFS as well). This is described in the docs (https://doc.arroyo.dev/deployment/kubernetes) but could be made clearer.

Hi!
This is a distributed k8s cluster. I am aware the checkpoint destination needs to be available to all pods and have configured volumes and volumeMounts according to the doc.
To provide more context, this error is reproducible when I'm running it locally using the default process scheduler as well. Also typically when I launch the new pipeline, the first 100 (or so) checkpoints can succeed but will be stuck due to the reported error afterwards.
Thanks so much!

Thanks so much for providing the query — I'm able to reproduce the issue.

Thanks for the great bug report and for helping me diagnose this issue! I've merged a fix that I believe should address it. Please try out the latest master (available as the docker image ghcr.io/arroyosystems/arroyo:tip) and please re-open if you're still seeing it.

Wow, that was swift!!! I'll test it out, thanks so much!


Not working with https://github.com/vbabiy/djangorestframework-camel-case

I am using the https://github.com/vbabiy/djangorestframework-camel-case in order to format the response to camel case. The problem is  when there is any Error is raised, the fields are rendered as snake case. Here are the example:
When resource is successfully created I get this response:
{
    "createdAt": "2024-07-28T13:58:50.062565Z",
    "updatedAt": "2024-07-28T13:58:50.062691Z",
    "clientName": "string",
    "clientPhone": "string",
    "clientEmail"<EMAIL_ADDRESS>    "taskTime": "2024-07-28T13:58:13.054000Z",
    "notes": "string",
    "paymentAmount": 20,
    "paymentMethod": "ONLINE",{
    "type": "validation_error",
    "errors": [
        {
            "code": "required",
            "detail": "This field is required.",
            "attr": "client_name"
        }
    ]
}
    "isCompleted": true,
    "jobType": "APPLY",
    "jobDeadline": "2024-07-28T13:58:13.054000Z",
    "applyDeadline": "2024-07-28T13:58:13.054000Z",
    "brokerage": "MY-BROKERAGE",
    "showClientPhoneNumber": true,
    "accessInformation": "string",
    "createdBy": null,
    "property": 1,
    "assignedTo": null
}

But when an error is raised, I am getting attr rendered in snake case due to which api developers are getting too much confused :
{
    "type": "validation_error",
    "errors": [
        {
            "code": "required",
            "detail": "This field is required.",
            "attr": "client_name"
        }
    ]
}

Hi @amitv9493, djangorestframework-camel-case is not supported out of the box, still check #59, it might be helpful to solve your issue


Migrate docs from kytos.io

Migration from Kytos.io (tutorials, dev and admin guides, project descript, etc), figure out where to host and if on GitHub pages, discuss overlap with https://github.com/kytos-ng/kytos-ng.github.io

Proposed discussion linked: https://github.com/kytos-ng/kytos/discussions/342

This issue has been broken down into epic_documentation_ng, I'll close this one since it was just an initial placeholder, now that we have the subtasks and epic it's no longer needed.


Support and require pytest>=3.0

Hi @tomviner!
I used your plugin to investigate pytest-dev/pytest#2131, this PR makes the plugin work with pytest >= 3.0. I didn't try to keep compatibility with earlier versions, but let me know if you prefer that; is not that hard to keep compatibility.
Cheers,
🍻

Coverage remained the same at 64.516% when pulling 9066ccd369b34305a5c6069c7b368518a9cdbb92 on nicoddemus:pytest-3 into ec32b3a34b17b1027a22beaa1a8a948de0369304 on tomviner:master.

Coverage remained the same at 64.516% when pulling 9066ccd369b34305a5c6069c7b368518a9cdbb92 on nicoddemus:pytest-3 into ec32b3a34b17b1027a22beaa1a8a948de0369304 on tomviner:master.

Coverage remained the same at 64.516% when pulling b0870b380a8c9af6822df859aa76f726051f00ae on nicoddemus:pytest-3 into ec32b3a34b17b1027a22beaa1a8a948de0369304 on tomviner:master.

Coverage remained the same at 64.516% when pulling b0870b380a8c9af6822df859aa76f726051f00ae on nicoddemus:pytest-3 into ec32b3a34b17b1027a22beaa1a8a948de0369304 on tomviner:master.

My first outside contribution, exciting! Thanks @nicoddemus :smile:
As a pytest-dev-assist tool, I think we should match the versions that pytest itself supports, for bug fixes. But I couldn't find that information on the pytest site. If only there was someone I could ask...
If bugs are found in pytest 2.x will we release further 2.x patch releases?

My first outside contribution, exciting! Thanks @nicoddemus 😄

That warrants a 🍺 then! 😄

As a pytest-dev-assist tool, I think we should match the versions that pytest itself supports, for bug fixes. But I couldn't find that information on the pytest site. If only there was someone I could ask...

As long as I've been a contributor we never did a patch release of a previous minor release. I think it would only happen in an extraordinary circumstance, because we don't have the branches necessary in place for that to happen.

Cheers :beers: :exclamation:

As long as I've been a contributor we never did a patch release of a previous minor release. I think it would only happen in an extraordinary circumstance, because we don't have the branches necessary in place for that to happen.

In that case, let's :m: :e-mail: :registered: :nine: :cyclone: and  :shipit: !

😆

How about a new pypi release?

Released https://pypi.python.org/pypi/pytest-ast-back-to-python/1.0.0
In the process of doing that, I noticed the tests break with the latest version of coverage. I've created https://github.com/tomviner/pytest-ast-back-to-python/issues/4 to cover this.

Thanks! 😁


Dependencies between Apps created with angular-cli

I am using the Angular2-cli to create two applications: MyFirstApp & MySecondApp. I whish MySecondApp to import MyFirstApp so I can reuse directive/service/component.
When I import MyFirstApp, using the documentation for 3rd parties I have the following compilation error: Cannot find module 'MyFirstApp'.
Here's how to reproduce:
    $ npm -v
    3.9.3

    $ lsb_release -a
    No LSB modules are available.
    Distributor ID: elementary OS
    Description:    elementary OS Freya
    Release:    0.3.2
    Codename:   freya

    $ node -v
    v6.2.1

    $ npm list  | grep 'angular'
    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>
    $ ng new MyFirstApp
    $ cd MyFirstApp
    $ ng g service MyFirstAppService
    $ ng build
    $ cd ..
    $ ng new MySecondApp
    $ cd MySecondApp
    $ npm install ../MyFirstApp --save

Add MyFirstApp to angular-cli-build.js & system.config.ts as shown in the doc.

//angular-cli-build.js
var Angular2App = require('angular-cli/lib/broccoli/angular2-app');
module.exports = function(defaults) {
  return new Angular2App(defaults, {
    vendorNpmFiles: [
      'systemjs/dist/system-polyfills.js',
      'systemjs/dist/system.src.js',
      'zone.js/dist/**/*.+(js|js.map)',
      'es6-shim/es6-shim.js',
      'reflect-metadata/**/*.+(js|js.map)',
      'rxjs/**/*.+(js|js.map)',
      'my-first-app/**/*.+(js|js.map)', //HERE
      '@angular/**/*.+(js|js.map)'
    ]
  });
};

//system.config.ts
const map: any = {
  'my-first-app': 'vendor/my-first-app/index.js'
  // Also tried  'my-first-app': 'node_modules/my-first-app/index.js'
};

/** User packages configuration. */
const packages: any = {
  'my-first-app': {
   format:'cjs'
  }
};

Then,
//my-second-app.component.ts
import * as firstapp from 'my-first-app'; // -> `Cannot find module
import { MyFirstAppServiceService } from 'my-first-app/my-first-app-service.service'; // -> `Cannot find module
import { MyFirstAppServiceService } from 'my-first-app/src/app/my-first-app-service.service'; //-> Works !

The last import works, but I guess it doesn't leverage angular-cli-build.js nor system-config.ts.
Note that the resulting vendors/my-first-app/ directory doesn't contain the actual code of MyFirstApp
$ # While in MySecondApp
$ tree dist/vendor/my-first-app
dist/vendor/my-first-app
├── angular-cli-build.js
└── config
    ├── environment.js
    ├── karma.conf.js
    ├── karma-test-shim.js
    └── protractor.conf.js

Thanks.

We don't really have support for making library-type packages at the moment.
I think the Cannot find module message is due to package.json not having a defined main entry, maybe you can try adding that.
There really isn't any way to recursively build subprojects, so your best bet is to try and import the ts files directly I think.

@filipesilva the Ember CLI has a command (ember addon <addon-name>) that will scaffold a new project designed to be installed via npm into other Ember CLI projects. These "addon" projects can also be built and served standalone using a built-in "dummy app", which acts as a fake consumer of the addon for testing and demonstration purposes. This is extremely useful for quickly creating an easily maintainable styleguide/component library that is shared between apps but also deployed independently.
I would really love an equivalent for my Angular CLI projects, and was hopeful the CLI would have provided a similar method for creating addons, seeing as it is based on the Ember CLI.
Since it is not possible to consume an Angular CLI app from another Angular CLI app, should my approach be to create my own "addon" using a manual (gulp/grunt) build process, in order to achieve the same functionality? Just looking for some feedback before I start down this path. Thank you!

Well we want to support creating library-type projects, that is still on the 1.0.0 goals.
Right now, you can probably make your own addon - remember that Angular-CLI is actually an Ember-CLI adding itself. But I can't promise it will be well supported since we've moved away from the addon architecture a bit while we build the core functionality.
There is actually one example of a library app created with the CLI: https://github.com/angular/material2. They are probably the people best suited to help you in doing this.

@filipesilva Now that @angular/cli is 1.0.0 can you please update on this? Was the library-type project included? Or is there any issue where progress can be tracked? Or was it dropped completely? Thank you!


I met another question during the training

Traceback (most recent call last):
File "train.py", line 168, in
main(cfg)
File "train.py", line 159, in main
debug_freq=cfg.DEBUG_FREQ,
File "/root/autodl-tmp/STAF-main/lib/core/trainer.py", line 400, in fit
self.train()   # 调用 train 方法进行模型训练
File "/root/autodl-tmp/STAF-main/lib/core/trainer.py", line 227, in train
scores=scores,
File "/root/miniconda3/envs/yzy_staf/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/root/autodl-tmp/STAF-main/lib/core/loss.py", line 116, in forward
pred_theta = pred_theta[w_smpl]
IndexError: The shape of the mask [39] at index 0 does not match the shape of the indexed tensor [0, 85] at index 0
Can you give me some advice?

It seems like there are some mistakes in data loading. Because it's wired, the shape of pred_theta is [0, 85]. You should debug to check what went wrong.

It seems like there are some mistakes in data loading. Because it's wired, the shape of pred_theta is [0, 85]. You should debug to check what went wrong.

It achieves the training successfully. Thank you very much!

Another, I met a bug during the evaluting as the following:
Traceback (most recent call last):
File "evaluate.py", line 94, in
model.load_state_dict(checkpoint['gen_state_dict'])
File "/root/miniconda3/envs/yzy_staf/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1498, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for SAFM:
Missing key(s) in state_dict: "nonlocalblock.attention.fc.weight", "nonlocalblock.attention.fc.bias", "nonlocalblock.attention.attention.0.weight", "nonlocalblock.attention.attention.0.bias", "nonlocalblock.attention.attention.2.weight", "nonlocalblock.attention.attention.2.bias", "nonlocalblock.attention.attention.4.weight", "nonlocalblock.attention.attention.4.bias".
Unexpected key(s) in state_dict: "points_grid", "deconv_layers.0.weight", "deconv_layers.1.weight", "deconv_layers.1.bias", "deconv_layers.1.running_mean", "deconv_layers.1.running_var", "deconv_layers.1.num_batches_tracked", "deconv_layers.3.weight", "deconv_layers.4.weight", "deconv_layers.4.bias", "deconv_layers.4.running_mean", "deconv_layers.4.running_var", "deconv_layers.4.num_batches_tracked", "deconv_layers.6.weight", "deconv_layers.7.weight", "deconv_layers.7.bias", "deconv_layers.7.running_mean", "deconv_layers.7.running_var", "deconv_layers.7.num_batches_tracked", "maf_extractor.0.Dmap", "maf_extractor.0.conv0.weight", "maf_extractor.0.conv0.bias", "maf_extractor.0.conv1.weight", "maf_extractor.0.conv1.bias", "maf_extractor.0.conv2.weight", "maf_extractor.0.conv2.bias", "maf_extractor.1.Dmap", "maf_extractor.1.conv0.weight", "maf_extractor.1.conv0.bias", "maf_extractor.1.conv1.weight", "maf_extractor.1.conv1.bias", "maf_extractor.1.conv2.weight", "maf_extractor.1.conv2.bias", "maf_extractor.2.Dmap", "maf_extractor.2.conv0.weight", "maf_extractor.2.conv0.bias", "maf_extractor.2.conv1.weight", "maf_extractor.2.conv1.bias", "maf_extractor.2.conv2.weight", "maf_extractor.2.conv2.bias", "regressor.0.init_pose", "regressor.0.init_shape", "regressor.0.init_cam", "regressor.0.fc1.weight", "regressor.0.fc1.bias", "regressor.0.fc2.weight", "regressor.0.fc2.bias", "regressor.0.decpose.weight", "regressor.0.decpose.bias", "regressor.0.decshape.weight", "regressor.0.decshape.bias", "regressor.0.deccam.weight", "regressor.0.deccam.bias", "regressor.0.smpl.betas", "regressor.0.smpl.global_orient", "regressor.0.smpl.body_pose", "regressor.0.smpl.faces_tensor", "regressor.0.smpl.v_template", "regressor.0.smpl.shapedirs", "regressor.0.smpl.J_regressor", "regressor.0.smpl.posedirs", "regressor.0.smpl.parents", "regressor.0.smpl.lbs_weights", "regressor.0.smpl.J_regressor_extra", "regressor.0.smpl.vertex_joint_selector.extra_joints_idxs", "regressor.1.init_pose", "regressor.1.init_shape", "regressor.1.init_cam", "regressor.1.fc1.weight", "regressor.1.fc1.bias", "regressor.1.fc2.weight", "regressor.1.fc2.bias", "regressor.1.decpose.weight", "regressor.1.decpose.bias", "regressor.1.decshape.weight", "regressor.1.decshape.bias", "regressor.1.deccam.weight", "regressor.1.deccam.bias", "regressor.1.smpl.betas", "regressor.1.smpl.global_orient", "regressor.1.smpl.body_pose", "regressor.1.smpl.faces_tensor", "regressor.1.smpl.v_template", "regressor.1.smpl.shapedirs", "regressor.1.smpl.J_regressor", "regressor.1.smpl.posedirs", "regressor.1.smpl.parents", "regressor.1.smpl.lbs_weights", "regressor.1.smpl.J_regressor_extra", "regressor.1.smpl.vertex_joint_selector.extra_joints_idxs", "regressor.2.init_pose", "regressor.2.init_shape", "regressor.2.init_cam", "regressor.2.fc1.weight", "regressor.2.fc1.bias", "regressor.2.fc2.weight", "regressor.2.fc2.bias", "regressor.2.decpose.weight", "regressor.2.decpose.bias", "regressor.2.decshape.weight", "regressor.2.decshape.bias", "regressor.2.deccam.weight", "regressor.2.deccam.bias", "regressor.2.smpl.betas", "regressor.2.smpl.global_orient", "regressor.2.smpl.body_pose", "regressor.2.smpl.faces_tensor", "regressor.2.smpl.v_template", "regressor.2.smpl.shapedirs", "regressor.2.smpl.J_regressor", "regressor.2.smpl.posedirs", "regressor.2.smpl.parents", "regressor.2.smpl.lbs_weights", "regressor.2.smpl.J_regressor_extra", "regressor.2.smpl.vertex_joint_selector.extra_joints_idxs", "safm.nonlocalblock.attention.fc.weight", "safm.nonlocalblock.attention.fc.bias", "safm.nonlocalblock.attention.attention.0.weight", "safm.nonlocalblock.attention.attention.0.bias", "safm.nonlocalblock.attention.attention.2.weight", "safm.nonlocalblock.attention.attention.2.bias", "safm.nonlocalblock.attention.attention.4.weight", "safm.nonlocalblock.attention.attention.4.bias", "tcfm.nonlocalblock.conv_phi.weight", "tcfm.nonlocalblock.conv_theta.weight", "tcfm.nonlocalblock.conv_g.weight", "tcfm.nonlocalblock.conv_mask.weight", "tcfm.nonlocalblock.conv_mask_forR.weight".

refer to issue


[Proof of Concept] Azure Monitor - Refactoring the EventSource

Creating this PoC to get feedback on this approach.
The idea behind this change is to have basic Events in the EventSource class to prevent each PR from creating new Event methods.
This should simplify and standardize the way to handle exceptions.
Changes

Refactor EventSource class. This introduces one method per severity level, plus an additional method to handle exceptions.

Comment
An earlier version of this was re-throwing the exceptions from the EventSource class, to fully encapsulate the exception handling.
I had this working, but it required extra considerations to preserve the stack trace.
I decided against this approach because it wasn't obvious to myself (and FxCop) that a try/catch block would re-throw any exception.

@xiang17 need your input when this PR is ready for review

Hi @TimothyMothra.  There hasn't been recent engagement on this PR.  Would you please be so kind as to let us know if this is still an active work stream or if the PR should be closed out?

Since there hasn't been recent engagement, I'm going to close this out.  Please feel free to reopen if you'd like to continue working on these changes.


Proper IPv6 Support

Since this is planned but no real ticket for it yet, let me mention first that #2 covers this to a degree as well as other discussions we have had. It should be capable of:

Listening on more than one host: (#25) for 6plane and rfc4193 addresses. Ideally, we should just allow a list of hosts or a CIDR to be passed.
Listening for wildcards (#24, #2) covers situations like PTRv6 records which need to match a 6plane allocation.
Finally, some kind of solution for name -> CIDR mapping that is effectively one to many, but I'm fairly certainly supplying the whole CIDR in a AAAA response would explode most computers. I need to research this part more. member ids can be mapped with a wildcard.

ok! I think we can call this done for now.


Add Credentials Skeleton and Generate Flow

This PR adds skeleton stuff to the porter cmd/ stuff to add credentials commands.
This PR only wires up generate, however, since it's already pretty large. Will handle
the others in follow-ons. The generate command is similar to Duffle in that it uses
survey, but the text and prompts are different.
Closes #225

@carolynvs any outstanding issues? or can I merge this with @vdice's approval?

Merge at will, pew pew.


Retrieving lost tracklet information

Hey, great work. I wanted some information regarding the lost tracks.
Referring to BoTSORT implementation, here you can see that lost tracklet information can also be used if required.
Coming to HybridSORT, I see here you are removing the dead tracklets. Do these two things mean the same, if so, can I use the dead tracklet information assuming it is the lost tracklet as in BOT-SORT's case?

Hello, our algorithm also uses the lost tracklet for tracking. here means that the time for the lost tracklet to disappear exceeds max_ age (30 frames) will be removed, corresponding to the lost tracklet in BoT-SORT becoming removed_ Tracks (here).

Thanks for the reply, that is what I wanted to confirm.


Improve :Telescope loogle

As discussed in #316 we would like a few improvements:

[ ] being able to type unicode in the prompt. This requires hooking into the abbreviation framework
[x] being able to use telescope for the search directly. This is a little more complicated as we don't want to spam the API with live updated input so we should probably add some debouncing to this feature.
[ ] automatically importing the declaration that is used. Once the LSP provides an "auto import" code action it should be possible to call this with the identifier in order to add the input if it is not yet present.

Another different path to think about I think is integrating with the workspace -- e..g maybe it'd be nice to have a Loogle searcher where it helps you assemble your query using current LSP workspace symbols.
(I'm purely dreaming up crude ideas here) but just like e.g. require('telescope.builtin').lsp_dynamic_workspace_symbols() will telescope over all the Lean stuff you're currently importing, perhaps it'd be nice to combine that with Loogle such that you could type a query like Re. ?a Re. and have telescope help you convert the Re to Real.sin (because you've imported that).
Obviously that starts to use some more complicated telescope functionality but I think something like that should be feasible (of course not saying it's urgent, but I like that we can collect ideas).


add non persistent notice

This will help some, like myself to avoid confusion of why the authstore isn't saving across app restarts

Thank you, but I'll consider specifying it more clearly in the README sometime later today.
In a future version I'll add a Flutter persistent store in the SDK (most likely using encrypted shared preferences) since several other users requested it.


[1.3-stable][bug] CAS_raTypeMismatchException

The phpCAS client's constructor will attempt to throw an exception of type CAS_raTypeMismatchException (which doesn't exist) if the port number isn't an integer. It should probably be CAS_TypeMismatchException, like the other exceptions that are thrown during argument validation.
https://github.com/Jasig/phpCAS/blob/1.3-stable/source/CAS/Client.php#L892

It looks like this is already fixed in master...

Closing the issues then...
Thanks for the report.


README.MD run command might be incorrect?

Summary
Just a small one. In README.MD it says to run "cargo tauri dev".
Unless there's a way that you run Tauri that's different from what I've seen, it's typically npm run tauri dev or pnpm tauri dev

Suggested change
Change to use pnpm tauri dev to run the project rather than cargo.

Hmm seems like there is a cargo command for this, closing

Yup, just realized I never installed the tauri-cli  😂, been running it off the pnpm cli the whole time


Update chart coroot to 0.10.0

This PR contains the following updates:

Package
Update
Change

coroot
minor
0.9.1 -> 0.10.0

[!WARNING]
Some dependencies could not be looked up. Check the warning logs for more information.

Release Notes

coroot/helm-charts (coroot)
v0.10.0
Compare Source
A monitoring and troubleshooting tool for microservice architectures.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Feature/IJ-170 Login Redesign

Re-designed the login page for the platform as the original view was very overcrowded:

The root route has been changed to the User login page. The new design is a lot more concise and clean looking:

The extra links on the old login forms have been extracted out into a modal, accessed through the 'Trouble Signing In?' linkl.
Before:

After:

By clicking 'Learn More', you will be redirected to the 'About' section, where you detailed descriptions of the project, Include and the Lab can be found:

The Team Member login is as follows. It can be found by clicking the link in the top right of User login. Similarly, you can navigate back to the User login by a link in the top right.

Mobile layout:

Currently, the only differences between the User login and the Team Member login are the Icons in the form title, and the fact that the User login has the big yellow brand card - making it more user friendly than the staff alternative.
If you have any thoughts or comments let me know!

@garethdandrews; I made some minor style updates so the icons aren't repeated on the login screen.

@phil-6; Should we be running this by Include UK to see if they're happy with it? It makes much more logical to sense to me now and we styled the login form on the Facebook login form so hopefully there's identifiability for Users.

@garethdandrews; I made some minor style updates so the icons aren't repeated on the login screen.

Looks good - I much prefer what you've done with the Team Member link :+1:

Added a few style updates myself.
Mobile user login now has a header:

Only the crisis modal now has the orange sides, the devise modal now matches the login form background:

The buttons also no long have text-decoration underline on hover, and the buttons in the modal now change font colour on hover too.
I'm happy with this now. I don't think we need to run it past Include, merge away.


Fix auto-completer bug when comparing ints

This fixes a bug in the auto-completer, and should address #118.
Note: this will make sure the auto-completer doesn't crash, but the behaviour is not very logical. It tries to find auto-complete values using a LIKE operator on an integer field. E.g. this causes autocomplete to find the value 22 if you auto-complete for 2. I'm not sure if this even works in every DB but the test suite well tell us.
@ohadlevy @abenari

Yup, this doesn't work in Postgres.
Failure/Error: Foo.complete_for('int > 2').should == ["int >  22"]
     ActiveRecord::StatementInvalid:
       PG::UndefinedFunction: ERROR:  operator does not exist: integer ~~ unknown
       LINE 1: ...NCT "foos"."int" FROM "foos"  WHERE ("foos"."int" LIKE '2%')...

Maybe the right behaviour is to just always return an empty array when auto-complete for a numeric value?

I changed it to return an empty list when trying to auto-complete a numerical field. Is this an acceptable solution?

@wvanbergen does it mean it would never auto complete on numerical ? AFAIU the issue is only on the second auto completer e.g. ````int > 'returns 'int > 2'butint > 2'``` should not auto complete further unless there is a space char.

Yeah, the fix I have now makes sure it never auto-completes. If we want auto-completion  back, we either need to use different SQL than the LIKE operator, or we should case the field to a string.
I am not too well-versed in the auto-completed code, so any pointers or help from @abenari would be helpful.

Closing this in favor of #121 121

I don't really care about Oracle, because testing will be an issue forever. If somebody is using Oracle and this doesn't work for them, they can fix it in https://github.com/wvanbergen/scoped_search-oracle_enhanced.


Provider butterknife.compiler.ButterKnifeProcessor could not be instantiated

错误: 服务配置文件不正确, 或构造处理程序对象javax.annotation.processing.Processor: Provider butterknife.compiler.ButterKnifeProcessor could not be instantiated: java.lang.NoClassDefFoundError: butterknife/OnCheckedChanged时抛出异常错

Can you provide a sample project that exhibits the behavior?
On Sun, Feb 5, 2017, 2:12 AM jarvan11<EMAIL_ADDRESS>wrote:

错误: 服务配置文件不正确, 或构造处理程序对象javax.annotation.processing.Processor: Provider
butterknife.compiler.ButterKnifeProcessor could not be instantiated:
java.lang.NoClassDefFoundError: butterknife/OnCheckedChanged时抛出异常错
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JakeWharton/butterknife/issues/865, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAEEEaUjXJjAzSZuJdGPJI7kaAxHR60xks5rZXZAgaJpZM4L3ats
.

My project use Small Frame.When I execute the command "gradlew buildBundle" ,the Exception throws.(ps:Small Frame : https://github.com/wequick/Small/tree/master/Android)


Allow individual test cases in a "directory suite" without sub-suites

Hello,
My project currently use directories to organize test suites, using suite initialization files, plus one file containing a single test case :

This approach have the downside of heavily burdening the generated report and log with useless levels, as the robot files are treated as test suites themselves:

This could be fixed by creating only one suite containing all tests, but this is burdensome when you have a suite containing a lot of tests cases.
Therefore, I'm asking if there is a way for a file in a test suite directory to be seen as a single test case directly and attached to the directory suite, for example by using the extension .case.robot

It would sometimes be convenient to have tests directly inside a higher level suite. Easiest solution for that is allowing *** Test Cases *** in the __init__.robot file. Is there a reason why it wouldn't work for you and having tests in a separate file would be more convenient?

We would like to have one file per test to ease visibility and navigation, a single file wouldn't be able to provide this.
We could already create one test file containing all test cases, which would create only one useless suite, but we chose not to for that reason.
It would really be preferable for the file to be handled as a test case rather than a test suite.

I don't personally consider this that useful, but if there would be clear semantics how it would work I guess adding such a feature wouldn't hurt. Is this something you'd be interested to implement? If yes, the first step would be coming up with a bit more detailed design.
One reason I don't consider this too high priority is that it's pretty easy to create a custom solution that provides the same functionality. The --PreRunModifier option makes it possible to specify a custom class that can modify the created suite structure before it is executed. For example, this would remove all suites that have a single test and add that test to the parent suite:
from robot.api import SuiteVisitor

class SuiteFlattener(SuiteVisitor):

    def start_suite(self, suite):
        if suite.parent and len(suite.tests) == 1:
            suite.parent.tests.append(suite.tests[0])
            suite.tests = []

    def end_suite(self, suite):
        suite.suites = [s for s in suite.suites if s.tests or s.suites]

I submitted #4548 about allowing tests in __init__.robot files.

Hello,
Your solution using a prerunmodifier would suite me quite well ! I'm not managing to implement it however. As a note, i'm using run_cli() to launch robot tests, and it tells me "option --prerunmodifier suiteflattener not recognized" so i think it's not available via run_cli() ? or am i missing something else ?
Here is an extract of the generated arguments when running the command:
`
Running the tests with the Robot framework, with arguments:
-d /home/.../results --exclude manual --skiponfailure wip --skiponfailure tbf -t Get logs --PreRunModifier SuiteFlattener /home/.../tests
`

Did you read the docs about using modifiers from the User Guide?

No activity lately so I consider it ok to close this one. This is such a special functionality that using pre-run modifiers ought to be a good enough solution.


Use IPAddress object instead of null pointer in IPAddressBER

This MR aims to solve #9 , wherein it's described that the way IPAddressBER is initialized with a null pointer and is not compatible with the constructors provided by the arduino-esp32 library.
Here, we create an empty IPAddress object instead.
This was tested using the Agent.ino example on the ESP32-WROVER-E to validate the changes, and an STM32F103 (bluepill + w5500 module) to check that it's not breaking compatibility.
Thanks,
Cyril

Nice, thanks.


Exclusion tags using "!" causing bash error

Describe the bug
Using ! for tag exclusion causes error due to "!" bash special character.
To Reproduce
Steps (or project) to reproduce the behavior:

Open a gauge project with tags
Run the gauge command with tag exclusion:
gauge run --tags "!tagname" specs
See error

Logs
$ gauge run -t "!admin" specs
bash: !admin: event not found

Expected behavior
Gauge should run normally with the tags parameter.

Versions:

$ gauge -v
Gauge version: 1.1.8
Commit Hash: b1501f4

Plugins
-------
flash (0.0.2)
html-report (4.0.12)
python (0.3.15)
screenshot (0.0.1)
xml-report (0.2.3)

OS
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 20.10
Release:	20.10
Codename:	groovy

Runner info
$ python3 --version
Python 3.8.6

IDE info
VS Code
Version: 1.55.2
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:37:02.931Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-48-generic snap

Additional context
Running Ubuntu VM in VirtualBox

! has a meaning in bash - to expand history. You need to escape it in your bash scripts. Try  gauge run -t "\!admin" specs

I'm not sure it actually worked. I run spec with '\!admin' and I don't get the error, but the tagged specs run just the same:

Side note, the total # scenarios seems off. Is that a known issue? The log showed correct total.

fixed the issue. Was caused by bash variable expansion.


RE-1616 Remove execution in docker container

RE tooling now runs gating/generate_release_notes/run in a docker
container, so there is no need to spawn the container locally now.
(cherry picked from commit 2ad3762d33c7fb238fb40a2be6eecbf148717161)

Please review, but do not merge until https://github.com/rcbops/rpc-gating/pull/1003 merges.
This has been tested here.


Using conditional expression in template ref value has different behaviour for "development" and "production" env

Vue version
3.4.35
Link to minimal reproduction
https://play.vuejs.org/#eNp9UstOwzAQ/JWVhZQiVakonKK0iNcBJB4Cjj40SrbBxVlHthMqVfl31gl9IFBPtmdmvTNrb8RVXcdtgyIRqcutqj049E09l6Sq2lgPG7C4HIOhR9OQxwI6WFpTQcRVkSRJuSHHVabCG0OF8soQzELRyNsGT/cS1Gc/BINbaLqHJO2ajEanMJvDRhJAUBqNsTblaMF3JHCy4SVuM91gt+DCP5rpoJkeaLq+QToZQnI8Pnisap155BNAWqgWErYyk+J3mEuIuF8ESVinkRTzd1OWGgGpBW/g5fX5FjIqeHIIUrBWClC0NZVO+GZukU52/cRYeMf0UpXxyhni6fdRpchNVSuN9rkOrZ0UyTCEwGVam6+HHguTHW/x/APzz3/wlVsHTIoXiw5ti1LsOJ/ZEv1A37094Zr3O7IyRaNZfYR8RY7WBI+D7Lqhgm0f6Hq39/0fUlS+u7u1R3LbUMFoUHa9Xgr+TDdHou/tnscXfR2/qOi+ARIB6cQ=
Steps to reproduce

Use conditional expression for  template ref attribute on any html element
Set NODE_ENV to development and try use ref.value
Set NODE_ENV to production and try use ref value

What is expected?
Result of ref.value is html element. Expected similar behaviour between development and production environment.
What is actually happening?
Working as expected for development - you can use conditional expression for ref attribute and get html element. For production you always get ref default value - you can wrap condition with computed - anyway you get default value.
System Info
No response
Any additional comments?
It may not be possible to use it this way, but the behaviour is different. If this is a bug, there is an opportunity to make a feature?

a workaround

Closing as a duplicate of #11373. Refer to Evan's comment:

As said, this is mostly a design flaw and there's no real fix for it. I am thinking of providing a dedicate useTemplateRef function in 3.5.


Multiple Search Bars

I have two searchbars in separate pages, like "home" and "other-stuff"... When I search something in one of them, the other try to search in their "data", I change the class in data-search-in="" but nothing..

What was the solution to this?


Add option of the developer can load the custom prototypes

Recently, I in the development of a framework, using the new mongodb driver. The driver uses the namespace, file name cannot extend to name, on the "zephir/prototypes" doesn't work very well. Therefore I think of a better way to provide an option to let me decide which directory is loaded. By my own and maintain these, but not all in the zephir/prototypes".
In the configuration file:
{
    "prototype-dir": {
        "mongodb": "prototypes/mongodb"
    }
}

If "mongodb extension" was not load, the file in "prototypes/mongodb" will be loaded.

Thanks :)

@steffengy Could you please explain. This is solution to reduce these warnings http://www.phalcon.io/phalconphp-and-php7 (see build log)?

This basically allows to load 3rd party prototypes on a project-basis.
So yes you could add sodium prototypes to phalcon and load them without touching zephir
and therefore fix these warnings.

Thanks

@steffengy These warnings occur even when the libsodium is enabled. Zephir  doesn't know anything about namespaced functions. And attempting to add the prototypes:
namespace Sodium
{
    function randombytes_buf($number)
    {}

    function randombytes_uniform($range)
    {}
}

throws fatal error:
PHP Fatal error:  Cannot redeclare Sodium\randombytes_buf() in /zephir/prototypes/libsodium/libsodium.php on line 6
PHP Stack trace:
PHP   1. {main}() /usr/share/zephir/compiler.php:0
PHP   2. Zephir\Bootstrap::boot() /usr/share/zephir/compiler.php:21
PHP   3. Zephir\Commands\CommandAbstract->execute() /usr/share/zephir/Library/Bootstrap.php:200
PHP   4. Zephir\Compiler->buildDev() /usr/share/zephir/Library/Commands/CommandAbstract.php:108
PHP   5. Zephir\Compiler->install() /usr/share/zephir/Library/Compiler.php:1358
PHP   6. Zephir\Compiler->compile() /usr/share/zephir/Library/Compiler.php:1267
PHP   7. Zephir\Compiler->generate() /usr/share/zephir/Library/Compiler.php:1131

Fatal error: Cannot redeclare Sodium\randombytes_buf() in /zephir/prototypes/libsodium/libsodium.php on line 6

Call Stack:
    0.0001     218024   1. {main}() /usr/share/zephir/compiler.php:0
    0.0010     271888   2. Zephir\Bootstrap::boot() /usr/share/zephir/compiler.php:21
    0.0047     459832   3. Zephir\Commands\CommandAbstract->execute() /usr/share/zephir/Library/Bootstrap.php:200
    0.0148    1636104   4. Zephir\Compiler->buildDev() /usr/share/zephir/Library/Commands/CommandAbstract.php:108
    0.0148    1636464   5. Zephir\Compiler->install() /usr/share/zephir/Library/Compiler.php:1358
    0.0150    1637648   6. Zephir\Compiler->compile() /usr/share/zephir/Library/Compiler.php:1267
    0.0150    1639576   7. Zephir\Compiler->generate() /usr/share/zephir/Library/Compiler.php:1131

any workaround to solve this?


Add a model config generator

Model config generator takes a PyTorch model as input and generates a JSON file with model layers and other propperties that define sharding on a particular hardware.

lint please


[Bug]: duplicate scrollbar when scrolling on the release notes modal

Is there an existing issue for this?

[X] I have searched the existing issues

Description
There is a ghost scrollbar on the page when scrolling on the release notes modal
https://www.loom.com/share/78a91dad82c14379b1fe514adc8b1051
Steps To Reproduce

Public Sample App
No response
Issue video log
https://www.loom.com/share/78a91dad82c14379b1fe514adc8b1051
Version
Cloud

Non issue


Extension issue

Issue Type: Bug
Extension Name: typescript-language-features
Extension Version: 1.0.0
OS Version: Darwin arm64 21.5.0
VS Code version: 1.69.0

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.69.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

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

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


Alias Groups

Hi, I'd like to group aliasses.
This could be done either manually (Create group, select aliasses, add) or automatically (Create group, add all aliasses containing information whatever (E.g. address matching 123%, Description starting with "XYZ", Organization matching "Fire", ...) or even both (❤)
Groups could be able to override alias settings and push to plugins.

This would be neat, bundle it in with #204 or something.
Would be useful for me too - I'd like to group my aliases by district as well as agency.

Oh: And filtering by groups would be cool as well

How about Auto create groups for same alias

This would be cool to do as a "User Settings" option :)  and would benefit my users as well..


Sqlite client: ConversionError(cannot parse integer from empty string)

Bug description
I'm seeing the following exception when running a query:
PrismaClientUnknownRequestError3 [PrismaClientUnknownRequestError]:
Invalid `prisma.elections.findMany()` invocation:

  Error occurred during query execution:
ConnectorError(ConnectorError { user_facing_error: None, kind: ConversionError(cannot parse integer from empty string) })
    at PrismaClientFetcher.request (/Users/bsouthga/projects/elections/node_modules/@prisma/client/runtime/index.js:78125:15)
    at processTicksAndRejections (node:internal/process/task_queues:93:5)
    at async getResults (webpack-internal:///./pages/elections/[year]/[office]/index.tsx:111:10)
    at async getStaticProps (webpack-internal:///./pages/elections/[year]/[office]/index.tsx:151:19)
    at async renderToHTML (/Users/bsouthga/projects/elections/node_modules/next/dist/next-server/server/render.js:28:109)
    at async /Users/bsouthga/projects/elections/node_modules/next/dist/next-server/server/next-server.js:98:97
    at async __wrapper (/Users/bsouthga/projects/elections/node_modules/next/dist/lib/coalesced-function.js:1:330)
    at async DevServer.renderToHTMLWithComponents (/Users/bsouthga/projects/elections/node_modules/next/dist/next-server/server/next-server.js:123:387)
    at async DevServer.renderToHTML (/Users/bsouthga/projects/elections/node_modules/next/dist/next-server/server/next-server.js:124:874)
    at async DevServer.renderToHTML (/Users/bsouthga/projects/elections/node_modules/next/dist/server/next-dev-server.js:34:578) {
  clientVersion: '2.13.1'
}

How to reproduce

Run query (in Prisma Information section).
Can provide a copy of the sqlite db for debugging to a team member if that helps.

Expected behavior
No exception, query is successfully executed.
Prisma information
Query:
const client = new PrismaClient();
return await client.elections.findMany({
  select: {
    locations: {
      select: {
        fips: true,
        name: true,
        district: true,
        state: true,
      },
    },
    results: {
      select: {
        votes: true,
        parties: {
          select: {
            name: true,
          },
        },
        candidates: {
          select: {
            name: true,
          },
        },
      },
    },
  },
  where: {
    year: 2016,
    offices: {
      name: 'president'
    },
  },
});

Schema:
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = "file:../data/elections.db"
}

model candidates {
  id      Int       @id @default(autoincrement())
  name    String    @unique
  results results[]
}

model election_totals {
  election_id Int       @unique
  total_votes Int
  elections   elections @relation(fields: [election_id], references: [id])
}

model elections {
  id              Int              @id @default(autoincrement())
  location_id     Int
  office_id       Int
  year            Int
  special         Boolean?         @default(false)
  runoff          Boolean?         @default(false)
  locations       locations        @relation(fields: [location_id], references: [id])
  offices         offices          @relation(fields: [office_id], references: [id])
  election_totals election_totals?
  results         results[]

  @@unique([year, office_id, location_id, special, runoff], name: "elections_id")
  @@index([location_id], name: "location_id_idx")
  @@index([office_id], name: "office_id_idx")
  @@index([year], name: "year_idx")
}

model locations {
  id        Int         @id @default(autoincrement())
  state     String
  LEVEL     String
  district  Int?
  fips      Int?
  name      String?
  elections elections[]

  @@unique([state, LEVEL, district, fips], name: "location_id")
  @@index([fips], name: "fips_idx")
  @@index([district], name: "district_idx")
  @@index([LEVEL], name: "level_idx")
  @@index([state], name: "state_idx")
}

model offices {
  id        Int         @id @default(autoincrement())
  name      String      @unique
  elections elections[]
}

model parties {
  id      Int       @id @default(autoincrement())
  name    String    @unique
  results results[]

  @@index([name], name: "name_idx")
}

model results {
  election_id  Int
  candidate_id Int
  party_id     Int
  votes        Int
  candidates   candidates @relation(fields: [candidate_id], references: [id])
  elections    elections  @relation(fields: [election_id], references: [id])
  parties      parties    @relation(fields: [party_id], references: [id])

  @@unique([election_id, candidate_id, party_id], name: "sqlite_autoindex_results_1")
}

Environment & setup

OS: MacOS 11.1 (20C69)
Database: SQLite
Node.js version: v15.3.0
Prisma version:

@prisma/cli          : 2.13.1
@prisma/client       : 2.13.1
Current platform     : darwin
Query Engine         : query-engine fcbc4bb2d306c86c28014f596b1e8c7980af8bd4 (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli fcbc4bb2d306c86c28014f596b1e8c7980af8bd4 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core fcbc4bb2d306c86c28014f596b1e8c7980af8bd4 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt fcbc4bb2d306c86c28014f596b1e8c7980af8bd4 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Studio               : 0.329.0

As a followup, I tried making the nullable fields in locations not null (didn't help):
diff --git a/prisma/schema.prisma b/prisma/schema.prisma
index 185d694..94e655f 100644
--- a/prisma/schema.prisma
+++ b/prisma/schema.prisma
@@ -41,9 +41,9 @@ model locations {
   id        Int         @id @default(autoincrement())
   state     String
   LEVEL     String
-  district  Int?
-  fips      Int?
-  name      String?
+  district  Int
+  fips      Int
+  name      String
   elections elections[]

   @@unique([state, LEVEL, district, fips], name: "location_id")

Hey @bsouthga
Can you please upload a sqlite db here or email it to<EMAIL_ADDRESS>if you don't want to share it publically?
Right now I can't reproduce this and I think I do need to data here.

Here is a copy of the database + the query (updated offices.name -> offices.standardized) for which I can repro this.
DB
elections.db.zip
Query
async function problemQuery() {
  const client = new PrismaClient();
  return await client.elections.findMany({
    select: {
      locations: {
        select: {
          fips: true,
          name: true,
          district: true,
          state: true,
        },
      },
      results: {
        select: {
          votes: true,
          parties: {
            select: {
              standardized: true,
            },
          },
          candidates: {
            select: {
              name: true,
            },
          },
        },
      },
    },
    where: {
      year: 2016,
      offices: {
        name: "president",
      },
    },
  });
}

Thanks for information. I can reproduce this now. We will need fix the conversion issue.
Consolidated reproduction: https://github.com/harshit-test-org/prisma-issue-4778

Hey, I've been investigating this issue today and here are my findings.
TLDR: inconsistencies in the data.
The query in the question is not the only one that fails, but let's look into what's going on with it:
We get a conversion error trying to parse an integer from a string. This is unfortunate with SQLite due to it not having almost any typing at all. The row that breaks is this:
id    fips  name               district  state
----  ----  -----------------  --------  -----
1955        Statewide writein            NA

Now looking into the schema, it says the following (unimportant parts removed):
model locations {
  id        Int         @id @default(autoincrement())
  fips      Int?
  name      String?
  district  Int?
  state     String
}

There are two empty fields that might cause this, so let's see what they actually are:
sqlite> select id, fips, name, district, state from locations where id = 1955 and district is null;
id    fips  name               district  state
----  ----  -----------------  --------  -----
1955        Statewide writein            NA

This tells us that district is null, which maps fine to our type of Int?.
select id, fips, name, district, state from locations where id = 1955 and fips is null;

And this one then returns nothing, meaning the value of fips is not null, but an empty string. If we define in the schema the field to be Int?, we can only parse integers or null values from the database, which is the case happened here.
I dug a bit deeper, and found some other inconsistencies too:
sqlite> select distinct(special) from elections;
special
-------
0
FALSE
TRUE
NA

And the data model (the interesting part):
model elections {
  special         Boolean?         @default(false)
}

This would mean we can have values of 0 (false), 1 (true) and null stored to the column, and loosening up the requirements, values of TRUE and FALSE should also work (if the column subtype is a boolean). If you'd query the row with a value of NA, we would crash again due to not being able to convert NA to a boolean.
My suggestion is to go through all the data carefully, iron out the inconsistencies to work together with the schema. It's also quite tricky to introspect the SQLite database, and sometimes we should be looking into the data too instead of the schema only. This is a case that our introspection should catch, giving an error describing problems with the existing data, or at least warnings.

Hey, I've been investigating this issue today and here are my findings.
TLDR: inconsistencies in the data.
The query in the question is not the only one that fails, but let's look into what's going on with it:
We get a conversion error trying to parse an integer from a string. This is unfortunate with SQLite due to it not having almost any typing at all. The row that breaks is this:
id    fips  name               district  state
----  ----  -----------------  --------  -----
1955        Statewide writein            NA

Now looking into the schema, it says the following (unimportant parts removed):
model locations {
  id        Int         @id @default(autoincrement())
  fips      Int?
  name      String?
  district  Int?
  state     String
}

There are two empty fields that might cause this, so let's see what they actually are:
sqlite> select id, fips, name, district, state from locations where id = 1955 and district is null;
id    fips  name               district  state
----  ----  -----------------  --------  -----
1955        Statewide writein            NA

This tells us that district is null, which maps fine to our type of Int?.
select id, fips, name, district, state from locations where id = 1955 and fips is null;

And this one then returns nothing, meaning the value of fips is not null, but an empty string. If we define in the schema the field to be Int?, we can only parse integers or null values from the database, which is the case happened here.
I dug a bit deeper, and found some other inconsistencies too:
sqlite> select distinct(special) from elections;
special
-------
0
FALSE
TRUE
NA

And the data model (the interesting part):
model elections {
  special         Boolean?         @default(false)
}

This would mean we can have values of 0 (false), 1 (true) and null stored to the column, and loosening up the requirements, values of TRUE and FALSE should also work (if the column subtype is a boolean). If you'd query the row with a value of NA, we would crash again due to not being able to convert NA to a boolean.
My suggestion is to go through all the data carefully, iron out the inconsistencies to work together with the schema. It's also quite tricky to introspect the SQLite database, and sometimes we should be looking into the data too instead of the schema only. This is a case that our introspection should catch, giving an error describing problems with the existing data, or at least warnings.

Thanks @pimeys for the deep dive on this! Sounds like a cleanup of inconsistent data is the best immediate fix.
One developer-experience request / suggestion would be to potentially dump this info (i.e. what value failed to parse as what expected type) when throwing within rust (maybe even with a pointer to this issue / the wiki).

Thanks @pimeys for the deep dive on this! Sounds like a cleanup of inconsistent data is the best immediate fix.
One developer-experience request / suggestion would be to potentially dump this info (i.e. what value failed to parse as what expected type) when throwing within rust (maybe even with a pointer to this issue / the wiki).

You are right. The errors could be better here and we could define a user-facing error with a proper code for this too. Why this hasn't been bubbling up earlier is that SQLite is so often used in a context where Prisma doesn't make much sense, and people have just not been witnessing problems what you have here. I highly believe this will change completely when we start supporting Mongo, so it makes a lot of sense to start making the errors here better, but also having our product thinking about how to prevent them much earlier than in the Prisma client...

You are right. The errors could be better here and we could define a user-facing error with a proper code for this too. Why this hasn't been bubbling up earlier is that SQLite is so often used in a context where Prisma doesn't make much sense, and people have just not been witnessing problems what you have here. I highly believe this will change completely when we start supporting Mongo, so it makes a lot of sense to start making the errors here better, but also having our product thinking about how to prevent them much earlier than in the Prisma client...

thanks again for digging in, closing as there is a workaround

thanks again for digging in, closing as there is a workaround


getAllProposals for vote contract fails to return data

After 6 or 7 proposals that have descriptions of ~250 words, the getAllProposals function of the vote contract no longer responds with data.
Here are two examples onchain:
https://goerli.basescan.org/address/0x570f497964efb9564af6fc4175809c8e8ec2263e#readContract
https://goerli.basescan.org/address/0x314bf06caee105883b5f8f5df4f6aedee9abc436#readContract
It breaks the dashboard for the proposal extension page.

I am thinking it might be related to the Base chain because Goerli (Eth) is responding past my 7th proposal
https://thirdweb.com/goerli/0xbE1127E57770f1F9E5734a973C1FFAb5E5BB1469/proposals

Hey @mykcryptodev , did it get resolved?
I believe it has something to do with gas limit for eth_call, may differ across chains / providers. Unfortunately right now we don't have a way to query this in batches / specific range. Will see if we can update this function in a future release.

Hey @mykcryptodev , did it get resolved? I believe it has something to do with gas limit for eth_call, may differ across chains / providers. Unfortunately right now we don't have a way to query this in batches / specific range. Will see if we can update this function in a future release.

Hey @kumaryash90 do you mean to say that various block explorers can have different gas limits for eth_call and that could be the reason why we get a response to getAllProposal() on ThirdWeb's UI but not on BaseScan? Cuz the testnet is common (Goerli).
In fact I checked for the balance of 0xbE1127E57770f1F9E5734a973C1FFAb5E5BB1469 contract on 3 block explorers and the BaseScan explorer doesn't even show the updated balance.
Balance on Goerli Etherscan: 0.0134 ETH
Balance on BlockScout: 0.0134 ETH
Balance on BaseScan: 0 ETH
BaseScan's data seems to be outdated or buggy.


ArrayDisplay (in GroundFrame) to be fixed

While trying to analyse some data with a zenith of 60° for Paranal, I found a wrong behaviour in the ArrayDisplay:

Few considerations:

The reconstruction works (which is good) but the ArrayDisplay needs to be fixed.
When you are in parallel mode, the plane of the camera and the TiltedGroundFrame are parallel and therefore you can use the angles obtained from the hillas parametrization to have the right plots, as shown here below.

In this screenshot from my 3D displayer you see the frame, positions of the telescopes, the lines using the angles from the hillas parametrization and the impact point (both MC & reco) for each plane:

in green (tilted frame)
in red (ground frame)

The problem is only related to the plotting in the ground frame; we can't use the same angles to draw the "hillas_line" in the GroundFrame since angles are not preserved when doing projections and therefore have to be recomputed to have a right plotting.
This was correct until now since we've always used MC with a zenith equal to 20°, for which the difference between the angles in the TiltedGroundFrame and the GroundFrame is less than a couple of degree.
I'm working now to modify the ArrayDisplay in order to have the right angles and solve this issue (and also https://github.com/cta-observatory/ctapipe/issues/737).

I'm coming with a solution for this plotting. It works for Paranal, where the array is basically flat,

But not so well for La Palma (especially at high zenith). Here is an image obtained with a custom MC production done at az=150° and zen=60° for La Palma :

In La Palma you can see that the angle is right now, but the line should be translated. The lines have the same z of the telescope whereas the impact point have z=0.
So we can't just project them on the ground giving them a z=0 but they should be translated...it's difficult to explain 3d things just by writing :sweat_smile:

So i reprojected the positions of the telescopes in the ground frame from the tilted frame in order to make the lines passing through that point. It works:

Update: everything works in the tilted frame (reconstruction and ArrayDisplay):
horizon_frame = AltAz()

array_pointing = SkyCoord(
    az=event.mc.az,
    alt=event.mc.alt,
    frame=horizon_frame
)

fig, ax = plt.subplots(figsize=(10,10))

tilted_frame = TiltedGroundFrame(pointing_direction=array_pointing)

array = ArrayDisplay(
    event.inst.subarray.select_subarray(
        f'Paranal alt={array_pointing.alt:.2f}° az={array_pointing.az:.2f}°', hillas_dict.keys()
    ),
    frame=tilted_frame,
    axes = ax)

#port the output of HillasReconstructor from Ground to Tilted
reco_core = GroundFrame(x=output.core_x, y = output.core_y, z = 0 * u.m)
reco_tilt = reco_core.transform_to(tilted_frame)
array.axes.scatter(reco_tilt.x, reco_tilt.y, c="r", label="reco", marker="+", s = 500)

#port the MC impact point from Ground to Tilted
mc_core = GroundFrame(x=event.mc.core_x, y = event.mc.core_y, z = 0 * u.m)
mc_tilt = mc_core.transform_to(tilted_frame)
array.axes.scatter(mc_tilt.x, mc_tilt.y, c="r", label="mc", marker="x", s = 500)

array.set_line_hillas(hillas_dict, range=1000)
array.add_labels()

array.axes.legend();

So that an event with 3 telescopes triggered gives this (in the tilted frame):

And this in the ground frame:

@HealthyPear This is about the psi_corr, right? So it is fixed?

Yes, I think this was fixed during my tests with protopipe.