some bags open with webviz, the bag during severe reduction

the bag duration in bag-database is 70 secs

but the webviz only show 35 secs data

have anybody else encountered the problem?

Could you share a bag file that demonstrates this problem?

apa_2022_06_28_13_55_10.bag.tar.gz

Could you share a bag file that demonstrates this problem? Have you tried Foxglove Studio to see if it has the same problem?

~$ rosbag info apa_2022_06_28_13_55_10.bag
path:        .../apa_2022_06_28_13_55_10.bag
version:     2.0
duration:    41.3s
start:       Jun 27 2022 22:55:12.32<PHONE_NUMBER>.32)
end:         Jun 27 2022 22:55:53.60<PHONE_NUMBER>.60)
size:        91.9 MB
messages:    100647

The actual duration of the bag as reported by rosbag info is 41.3 sec. Webviz and Foxglove Studio both show 41 seconds of data. Seems like this is a problem with your bag database.


Fix casstcl linking on linux / gcc env

If speedtables is built with casstcl support on Linux, creating new
speedtables fails because the casstcl shared library is not found.
Therefore we extract the version from the generated shared library
for sysconfig and then append that to the linker command when
creating a ctable

BCK-6685

This breaks on FreeBSD (at least), where the generated casstcl shared library is named just libcasstcl.so (no versioning). The library is installed under /usr/local/lib/casstcl2.14, so the old method worked. It is not clear to me from the description of this PR what doesn't work on Linux.. perhaps casstcl is not installed under a casstcl2.14 directory?
I think a more robust approach would be to parse the output of package versions casstcl.


[Question/Feedback]: Set-AVMModule hangs

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
Recently, running Set-AVMModule on my local machine hangs when processing files. I don't see any recent commits that would have changed much about this function and I'm not sure how to troublshoot.
PS /Users/matthewschmitt/GIT/bicep-registry-modules> Set-AVMModule -ModuleFolderPath avm/ptn/network/hub-networking -ThrottleLimit 10 Building [avm-network/hub-networking/main.bicep] /Users/matthewschmitt/GIT/bicep-registry-modules/avm/ptn/network/hub-networking/main.bicep(6,7) : Warning no-unused-params: Parameter "name" is declared but never used. [https://aka.ms/bicep/linter/no-unused-params] Generating readme for [avm-network/hub-networking/main.bicep] WARNING: Resource type [hubnetworking] does not exist in the API / is custom. Falling back to it as default. WARNING: No resource type like [Microsoft.Network/hubnetworking] found in template. Falling back to it as identifier. Processed [0/1] files [0% complete                                                                              ]
and it'll just sit there.

@Azure/avm-core-team-technical-bicep, could one of you please help @hundredacres (Matt Schmitt) with this issue?

Hey @hundredacres, this is a duplication of https://github.com/Azure/bicep-registry-modules/issues/1315.
In short, the formatter of the new Bicep version breaks the tool. I already worked on a fix - yet need to finish testing and then get the PR merged.

Hello @hundredacres, just a heads-up for today we enabled legacy (v1) formatting with this PR.
That is a temporary workaround meanwhile a fix is implemented for the readme generator to support bicep v2 formatting.

I pulled the latest changes and now gettting a new error:
PS /Users/matthewschmitt/GIT/bicep-registry-modules> Set-AVMModule -ModuleFolderPath avm/ptn/network/hub-networking Building [avm-network/hub-networking/main.bicep] /Users/matthewschmitt/GIT/bicep-registry-modules/avm/ptn/network/hub-networking/main.bicep(6,7) : Warning no-unused-params: Parameter "name" is declared but never used. [https://aka.ms/bicep/linter/no-unused-params] Generating readme for [avm-network/hub-networking/main.bicep] WARNING: Resource type [hubnetworking] does not exist in the API / is custom. Falling back to it as default. WARNING: No resource type like [Microsoft.Network/hubnetworking] found in template. Falling back to it as identifier. InvalidOperation: /Users/matthewschmitt/GIT/bicep-registry-modules/avm/utilities/pipelines/sharedScripts/Set-ModuleReadMe.ps1:1327 Line | 1327 |          $moduleDeploymentPropertyIndent = ([regex]::Match($rawBicepEx …      |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      | Cannot index into a null array. Processed [0/1] files [0% complete

I pulled the latest changes and now gettting a new error:
PS /Users/matthewschmitt/GIT/bicep-registry-modules> Set-AVMModule -ModuleFolderPath avm/ptn/network/hub-networking Building [avm-network/hub-networking/main.bicep] /Users/matthewschmitt/GIT/bicep-registry-modules/avm/ptn/network/hub-networking/main.bicep(6,7) : Warning no-unused-params: Parameter "name" is declared but never used. [https://aka.ms/bicep/linter/no-unused-params] Generating readme for [avm-network/hub-networking/main.bicep] WARNING: Resource type [hubnetworking] does not exist in the API / is custom. Falling back to it as default. WARNING: No resource type like [Microsoft.Network/hubnetworking] found in template. Falling back to it as identifier. InvalidOperation: /Users/matthewschmitt/GIT/bicep-registry-modules/avm/utilities/pipelines/sharedScripts/Set-ModuleReadMe.ps1:1327 Line | 1327 | $moduleDeploymentPropertyIndent = ([regex]::Match($rawBicepEx … | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | Cannot index into a null array. Processed [0/1] files [0% complete

Hey @hundredacres,  can you share a link to your pipeline run with me / the branch you're developing in? My running assumption is that the test files are already formatted in V2. Disabling the V2 formatter as per the merged PR does not format them back. So they may still be in a not perfectly compatible state.
Note I spent my last couple nights to work on a fix for the readme script too, but until that is in, I think it would make sense that we sort this out for you first :)

@AlexanderSehr Here's my branch: https://github.com/hundredacres/bicep-registry-modules/tree/hubspoke

This issue has been closed automatically by PR #1321
If not intended please @hundredacres @AlexanderSehr feel free to reopen it


Layout setting not responsive and the top header blocks the content (Linux Mint 18 )

The layout setting not responsive in my linux mint 18 (sonya) and the top header blocks the content when the content is top.

Duplicate of #179


Update for .NET Standard 2.1, .NET Core 2.2, and .NET Core 3.0

The table in this section might want to be updated https://docs.microsoft.com/en-us/dotnet/core/versions/#relationship-to-net-standard-versions for .NET Standard 2.1, .NET Core 2.2, and .NET Core 3.0

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

ID: 4723eb4d-c4b9-f44c-7753-caa4a3c8cfdd
Version Independent ID: 17270776-9bf5-fe60-f9a7-43311e5b4654
Content: How the .NET Core Runtime and SDK are versioned - .NET Core
Content Source: docs/core/versions/index.md
Product: dotnet-core
GitHub Login: @bleroy
Microsoft Alias: mairaw

I looked at this again. This doesn't seem important. Closing issue.

Reopening since we do need to talk about 2.1 in there for the next release.

@mairaw @BillWagner i think we can close this.

Thanks @eddynaka Yes, we can close this.


Type description of nested field

Type description of a nested field doesn't get set and shows the parent description
The introspective GraphQL Docs for a Type of a nested Field doesn't get set and shows the previous/parent's description instead
To Reproduce
Given a query getFoo that returns a type Foo that has a description, and a nested field bar that doesn't have a description.
Schema
schema {
  query: RootQueryType
}

"""
A Bar
"""
type Bar {
  """
  identifier of bar
  """
  id: ID!
}

"""
A Foo
"""
type Foo {
  """
  bar of foo
  """
  bar: Bar!

  """
  identifier of foo
  """
  id: ID
}

type RootQueryType {
  getFoo(id: ID!): Foo
}

Document
query {
  getFoo(id: "foo_id_1") {
    bar {
      id
    }
  }
}

Steps to reproduce the behavior:

Open the Docs
Click on RootQueryType
Click on getFoo
Notice the TYPE section that should mention Foo and its description right below it
Click on the bar field in the FIELDS section below
Notice the TYPE section that should now mention Bar
See the error in the line right below the type name, where it still shows the type description from the previous type Foo!

Screenshots

Navigate to: Docs: RootQueryType: getFoo

Navigate to: FIELDS: bar

See the error: TYPE: Bar: description

Expected behavior
The nested FIELDs types description should get set
Desktop

OS: macOS Mojave
Browser: Altair GraphQL Client
Version: <IP_ADDRESS>70 and <IP_ADDRESS>62

Hey @esambo Thanks for reporting. This would be fixed in the next release.


Apple rejects apps because of UIWebView reference

Bug Report
If I add this plugin to my app App Store will reject the app because this plugin contains references to UIWebView API.
I have tried removing the plugins one by one, and seen that this is the plugin referencing UIWebView. There should be a way to make it reference to WKWebView
Environment, Platform, Device
iOS
Version information
<EMAIL_ADDRESS><EMAIL_ADDRESS>**
Checklist

[ x] I searched for existing GitHub issues
[ x] I updated all Cordova tooling to most recent version
[ x] I included all the necessary information above

Duplicate of #556


Remove legacy analytics

We're getting good results from #2068, so there's no reason to keep maintaining this code.
Ref #1848

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS against 87ea20aa3ccf39d5ef5fcf6d1b05bce383ba880f


Use already calculated value prefix_count

https://github.com/urt30plus/urt30t/blob/ddf44d6f3f5464687c5a1f41cfdcca1f627eecd2/urt30t/plugins/commands.py#L474

Duplicate of #64


HDDS-9483. ozone freon --server is broken by HDDS-6176.

What changes were proposed in this pull request?
Specify a default temporary directory for HttpServer2 for 'ozone freon --server' command.
Please describe your PR in detail:
Essentially copy-pasted from HDDS-8253
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9483
How was this patch tested?
Manually tested on a cluster.

Thanks @jojochuang for the patch.

Essentially copy-pasted from HDDS-8253

Instead of copying, can you please extract to some utility class (e.g. HddsServerUtil)?

You can move the code aside or alternatively create in the current working directory for the CLI and avoid having these configs defined for the CLI. It is perfectly fine to create artifacts in the CWD for a binary and cleaning it up at the end.

Thanks @jojochuang for the fix, @kerneltime, @swamirishi for the review.

Thanks Attila Ritesh and Swami!


scratch: add doc.go

I set up a custom domain so the scratch repo is "go get" able, and godocs are now browsable: https://godoc.org/goscratch.club
However this now makes clear that we should have a doc.go alongside the README at the top level of that project, even if it's just really basic and points people elsewhere.

I set up a custom domain so the scratch repo is "go get" able, and godocs are now browsable: https://godoc.org/goscratch.club

For posterity, the packages inside the scratch repo were always go gettable at their canonical source:
https://godoc.org/go.googlesource.com/scratch.git
The ".git" suffix is necessary because go.googlesource.com isn't one of the few common code hosting sites that have special syntax. The mechanism is documented at https://golang.org/cmd/go/#hdr-Remote_import_paths, starting at the "For code hosted on other servers" paragraph.


Create asana task when nightly workflows fail

Task/Issue URL: https://app.asana.com/0/488551667048375/1203673701090347/f
Description
This PR creates an Asana task whenever a nightly workflow fails
Steps to test this PR
N/A
UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2696   👈

This comment was auto-generated by Graphite.


add the DAO to check incomplete workflow in agent

For checking draining status, use this for the first check if this passes check the DBS, PhEDEx upload status.

test this please

included in #7926


Bubblewrap runner does not run as 1000 (build)

Context
The Bubblewrap pipeline is run with same uid of the user that run melange, even though in differnt user namespaces. Only that uid is mapped inside the Bubblewrap user namespace, and not 1000 as standard and as already set in passwd via apko.
$ id -u
1001
$ melange build --runner=bubblewrap .. &
$ cat /proc/$(pgrep mycommand_in_bubblewrap)/uid_map
      1001       1001          1

Considering that my uid on the root user namespace is 1001, when I run melange build --runner=bubblewrap the pipeline runs commands as 1001 in a different user namespace but mapping 1001 of that user ns to 1001 of the host user namespace of the process that executed melange.
Expected behaviour
To have the pipeline be run in a different user namespace, but with the id of the user that run melange in that process's user namespace, be mapped to a consistent uid in the Bubblewrap user namespace.
Indeed, the passwd entry is also created with name build and uid and gid 1000.
How to reproduce
Test pipeline:
$ cat <<EOF >test.yaml
package:
  name: test
  version: 0.1.0
  epoch: 0

environment:
  contents:
    packages:
      - build-base
      - busybox

pipeline:
  - runs: |
      echo "running as $(id)"
      echo "build user is $(id build)"
      echo "running sleep..."
      sleep infinity
      exit 1
EOF

$ melange build test.yaml --runner bubblewrap [...]

...
2024/10/18 19:14:58 INFO running as uid=1001 gid=1001 groups=65534(nobody),65534(nobody),1001
2024/10/18 19:14:58 INFO build user is uid=1000(build) gid=1000(build) groups=1000(build)
2024/10/18 19:14:58 INFO running sleep...

The running user in Bubblewrap is the very same user in the host considering the different user namespaces:
$ ls -l /proc/self/ns/user
lrwxrwxrwx. 1 massimiliano.giovagnoli massimiliano.giovagnoli 0 Oct 18 19:10 /proc/self/ns/user -> 'user:[4026531837]'
$ ls -l /proc/`pgrep -f "^sleep\ infinity$"`/ns/user
lrwxrwxrwx. 1 massimiliano.giovagnoli massimiliano.giovagnoli 0 Oct 18 19:20 /proc/855437/ns/user -> 'user:[4026533321]'
$ cat /proc/self/uid_map
         0          0<PHONE_NUMBER>
$ cat /proc/`pgrep -f "^sleep\ infinity$`/uid_map
      1001       1001          1
$ ps -C sleep -o pid,cmd,user
    PID CMD                         USER
 855437 sleep infinity              massimiliano.giovagnoli # indeed

Additional context
Moreover, if it's run with Docker runner, the behaviour is even different by default:
2024/10/18 19:37:40 INFO running as uid=0(root) gid=0(root) groups=0(root),1(bin),2(daemon),3(sys),4(adm),6(disk),10(wheel),11(floppy),20(dialout),26(tape),27(video)
2024/10/18 19:37:40 INFO build user is uid=1000(build) gid=1000(build) groups=1000(build)
2024/10/18 19:37:40 INFO running sleep infinity now...

$ ls -l /proc/`pgrep -f "^sleep\ infinity$"`/ns/user
lrwxrwxrwx. 1 root root 0 Oct 18 19:42 /proc/857591/ns/user -> 'user:[4026531837]'
$ ls -l /proc/self/ns/user
lrwxrwxrwx. 1 massimiliano.giovagnoli massimiliano.giovagnoli 0 Oct 18 19:45 /proc/self/ns/user -> 'user:[4026531837]'

Meaning that we can potentially have different results based on fileystem based on the runner.
Also, the CI runs Melange as sudo melange also with Bubblewrap, leading to always be uid 0 in the pipeline.

With #1572:
$ id -u
1001
$ melange build --runner bubblewrap test.yaml ...
...
2024/10/19 00:01:15 INFO running as uid=1000(build) gid=1000(build) groups=65534(nobody),65534(nobody),1000(build)
2024/10/19 00:01:15 INFO build user is uid=1000(build) gid=1000(build) groups=1000(build)
2024/10/19 00:01:15 INFO running sleep infinity now...

$ cat /proc/`pgrep -f "^sleep\ infinity$"`/{uid_map,gid_map}
      1000       1001          1
      1000       1001          1

From a little investigation, it turns out it's not the cfg.RunAs the problem, on the contrary it's when it's NOT specified
QEMU and Docker will fallback to run as root inside the sandbox
Bwrap defaults to running as "whatever user you are", hence the problem with missing users inside
The comments on the PR will fix this


Add runtime test for integration with Experience Cloud ID Service

Description
This PR adds a runtime test for the Web SDK integration with the Experience Cloud ID Service inside of Tags. To actually fix the bug, a change to Alloy is required. The runtime test uses a mock extension that sets window.Visitor, and then makes sure that is used by Web SDK.

Related Issue
https://jira.corp.adobe.com/browse/PDCL-9421

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.
[x] I've updated the schema in extension.json or no changes are necessary.
[ ] My change requires a change to the documentation.

The test is failing because I haven't included the Web SDK fix into the build.


Enable using Skypack and jsDelivr for libraries

Currently, we allow searching for swing libraries using CDNJS, which provides a fixed set of packages. We should explore using jsDelivr or Skypack instead, or offering it as an alternative, since it provides an entire reverse proxy on top of NPM, and therefore, would have many more libraries than CDNJS.

Support for adding Skypack module references is now shipped!


Bar: keep shape

Hello @nagix, thanks for this great plugin.
Is it possible to keep the shape of the bar on hover? Currently, it changes / reinitializes on every mousemove. Can we prevent that somehow?

If you don't need the tooltip/hover animations on mousemove, this option disables the update:
options: {
    events: ['click']
}

Oh great, yeah I don't, that solves it for me. Many thanks!


feat: support more vtl language like array.contains

This pull request is highly inspired by the code available in amplify-appsync-simulator (see https://github.com/aws-amplify/amplify-cli/tree/master/packages/amplify-appsync-simulator/src/velocity)
Reason why I created this pull request is that the amplify-appsync-simulator does way more than I need. I only want to create unittests on VTL templates, so we can check if our templates are correct.
Than I found this great library and it worked for a lot of our templates, but a couple where failing. For one issue I created https://github.com/skyhookadventure/appsync-template-tester/issues/17, but it seems more functions are not supported.
This PR adds this functionality, so you can use [1,2,3].contains(1) and the VTL parser will return true instead of a string like '[1,2,3].contains(1)'
Let me know if this is something that can be merged, so I can keep on using this library and otherwise I will continue on our fork.

Sorry wrong source branch


Add canceled trains

Canceled trains are indicated in the API as well (soon). Add it to the GTFS-RT

Should work here: https://github.com/iRail/brail2gtfs-rt/blob/master/src/main/java/com/opentransport/rdfmapper/nmbs/ScrapeTrip.java#L235
Now set to 9999 minutes to indicate canceled stopsequence

Eh? 9999 is the worst thing you could have come up with imo: applications will show bogus results if not adapted

Better option in mind? 99 for example?

Not at all... If the train is canceled it should be indicated as canceled?
https://developers.google.com/transit/gtfs-realtime/trip-updates

I know for every stop of the trip if its canceled or not. When all upcoming stoptimes are canceled, I can set the ScheduleRelationship of the trip to CANCELED.
But when only certain stoptimes are partially canceled, I can set its ScheduleRelationship to SKIPPED.
Right?


Working on an identical project minbpe.rs

I was working on building the Rust implementation for karpathy/minbpe and realized that there was a crate with the same name while uploading my crate.
Here's my implementation: shubham0204/minbpe.rs
It has an implementation for BasicTokenizer for now, and RegexTokenizer and GPT4Tokenizer have to be implemented. I can contribute the following to your implementation:

README.md with usage instructions and sample code
Unit tests for functions in basic.rs and base.rs

I've added these in my implementation, so I can easily contribute these to your project.

Hi! Pleased to meet you. You are right -- I didn't invest in a better README.md, yet. I did add doc comment examples to a few things though, which could be source material for README.md contents.
In any case I'd be pleased to accept PRs for adding such things (and of course fixes if I got anything wrong!).
My objective was to make something pretty much 1-1 with what Andrej did as much as possible, but using Rust idioms.
And, I wanted to be able to have each of the implementations of tokenizers be in their own feature, so that if someone wanted to use use the base trait for Tokenizer, Loadable, Saveable, Trainable, Token, etc. they could do that and experiment making their own tokenizers but leveraging that basis.

At some point I'd like it to be at least as fast as tiktoken's gpt4 tokenizer but that is not yet true. First job for this one is to be easy to understand what it is doing, and 1-1 as much as possible with Andrej's.

I see you submitted your first PR. Thank you. I'll close this now.


ARROW-12279: [Rust][DataFusion] Add test for null handling in hash join (ARROW-12266)

This PR adds a (ignored) test for https://issues.apache.org/jira/browse/ARROW-12266
SELECT id1, id2 FROM (SELECT null AS id1) t1
LEFT JOIN (SELECT 0 AS id2) t2 ON id1 = id2

current result:
NULL, NULL
(should be empty result set)
We should filter on nulls beforehand to make this result correct. Probably the best way to go here I think is to add a filter in the logical plan on non-null for inner / left and right joins.
This can make things more efficient as the non-null filter can be pushed down which can lead to efficiency gains (making data-set smaller, not having to deal with nullable data in batches, or even entire files could be skipped when they only contain nulls).

We should filter on nulls beforehand to make this result correct. Probably the best way to go here I think is to add a filter in the logical plan on non-null for inner / left and right joins.

I am not sure this works for all join types (OUTER JOIN as well as , ANTI-JOIN and SEMI-JOIN which are optimizations for subqueries)
It might make sense to check for null when building the hash table for inner join keys (as NULL will never equal NULL)

👍 for the test

We should filter on nulls beforehand to make this result correct. Probably the best way to go here I think is to add a filter in the logical plan on non-null for inner / left and right joins.

I am not sure this works for all join types (OUTER JOIN as well as , ANTI-JOIN and SEMI-JOIN which are optimizations for subqueries)
It might make sense to check for null when building the hash table for inner join keys (as NULL will never equal NULL)

You are right, not for outer join or other joins (but we don't have them yet). For those, I think the rows have to be included, but might need some changes too wrt equality and building the hashmap. The filter approach is what Spark does fwiw. I think that makes also sense in the conceptually as joins should also support other conditions and allows for greater efficiency.


[Vue warn]: Do not use built-in or reserved HTML elements as component id: Table

I followed the guide and success, but my console have this warning.

May I ask your Vue.js version?

2.5.10

ok, i will fix it in the next minor version

npm i -S<EMAIL_ADDRESS>


Initial UI Cleanup

tighten up authenticated http requests

remove dead '/login' routes and redirects

remove nodes count from dashboard

remove header from login page

format new cloud account forms w/ layout key and
hide openstack option for now

implement JSON paste for GCE service account key

Coverage remained the same at 41.627% when pulling 04c1693da166fbb324c2a2566b98af2693872538 on josephkane:ui-cleanup into da6d99773619debab8708562f9a3fbf821385ca6 on supergiant:master.


Torque map not rendering with "custom" markers within mobile Twitter browser

S/B 8411737
Context
Map looks fine on desktop and via publish link viewed in mobile Safari, but markers don't appear when viewed on mobile within Twitter browser.
Checked in iOS 9.2.1.
Process

Create a public torque category map in CartoDB that uses a png image for the marker.
Click it's Publish button and copy the 'Get the link' url.
Tweet the map.
Open Twitter app on iPhone and click the map's link from inside the tweet.

Expected result
Torque map will autoplay & you will see markers with custom image:

Current result

If you want my test is in this Tweet.

Looks like for any reason Twitter iOS in-app browser doesn't render torque maps with custom markers properly. After checking other torque maps without custom image markers, they are working properly in that described in-app browser.


fix: #2180 - Explicit color for snack bar actions

Impacted file:

smooth_theme.dart: Explicit color for snack bar actions

What

The snack bar actions were invisible with the latest theme/color change.
Now they have an explicit color, now at least the text is visible.
Unfortunately it's the same color as the snack bar text (for the moment): there is no obvious difference between both.

Screenshot

light
dark

Fixes bug(s)

Fixes: #2180

Codecov Report

Merging #2186 (589321e) into develop (2ea0da3) will decrease coverage by 0.94%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2186      +/-   ##
==========================================
- Coverage     8.86%   7.91%   -0.95%
==========================================
  Files          161     176      +15
  Lines         6623    8631    +2008
==========================================
+ Hits           587     683      +96
- Misses        6036    7948    +1912

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
54.54% <0.00%> (-28.44%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
11.11% <0.00%> (-21.20%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
19.23% <0.00%> (-14.98%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
26.47% <0.00%> (-4.96%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

packages/smooth_app/lib/main.dart
15.74% <0.00%> (-2.16%)
:arrow_down:

...s/smooth_app/lib/pages/onboarding/next_button.dart
2.22% <0.00%> (-1.78%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

... and 162 more

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 ae54d77...589321e. Read the comment docs.


Map LootableInventory

This PR
This PR adds mappings for C_ehawoasn to fix a critical bug that crashed the game when breaking chests.
Most importantly, this maps C_ehawoasn.method_10997 to LootableInventory.getWorld and C_ehawoasn.method_11016 to LootableInventory.getPos.
Associated

This fixes #518.

These changes should probably be propagated to the 23w51b branch. Should be easy as a cherry-pick. It might be a little more involved to see how far back these changes need to be brought to.

Ok, I did some looking, and it appears this issue was introduced in 1.20.3/1.20.4. So it doesn't look like any backports are necessary.


Requests: Cursor keeps getting repositioned when writing comments

Package version: 12.0.0rc2
Browser
Firefox 127.0.2 (not to be confused with localhost) on Linux
Describe the bug
When writing comments on requests, the cursor keeps getting repositioned to the beginning of the input field, which makes writing comments very painful.
It looks like this happens periodically, probably the refresh timer.
Steps to Reproduce

Open a request (e.g. community-submission)
Visit the request's timeline/details page
Write some long text
Feel the pain

Expected behavior
The cursor keeps its position

Potentially related: 5s refresh interval for requests

I didn't understand the browser part: Firefox 127.0.2 (not to be confused with localhost) on Linux. However, I tested it on both Chrome and Firefox, and it seems like I am unable to reproduce the issue. Am I missing something @max-moser ? I am testing on Linux: Pop_OS.

I didn't understand the browser part: Firefox 127.0.2 (not to be confused with localhost) on Linux.

That was a reference to his version of Firefox. The version number 127.0.2 looks like a local network IP of the 127.0.X.Y variety.
If you type out a long comment 255 chars+ (much longer than just "asdfgadfs" and no new lines). Do you get the same behaviour?

Got it now Firefox version 127.0.2, not your local IP! 😅
Okay, using Firefox 127.0 (June 11, 2024 release), I tested with a long comment (255+ characters, single line). The cursor only jumps when pressing enter, as I mentioned earlier.

Any ideas on the cause @fenekku ?

The cursor only jumps when pressing enter, as I mentioned earlier.
I've also noticed that it loses focus randomly, as seen in the SR when clicking and pressing space rapidly also not related to the timer.

Ugh... I am still thinking it has to do with the textarea being re-rendered by React. It's a known problem that the cursor position will be lost in those situations unless kept in the state or using a Ref (:stuck_out_tongue_closed_eyes: ). If it's not (only) the 5s timer causing re-renders, but perhaps other events that would explain it. I am working on something else so perhaps a React expert could chime in.


Error: lacking either libGL or libEGL so could not fallback to other

Describe the bug
Getting the following error message when cloning the starter template (https://github.com/emilk/egui_template/) and compiling it with cargo run:
    Finished dev [unoptimized + debuginfo] target(s) in 0.04s
     Running `target/debug/egui_template`
libGL error: No matching fbConfigs or visuals found
libGL error: failed to load driver: swrast
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: GlutinCreationError(CreationErrors([NotSupported("lacking either libGL or libEGL so could not fallback to other"), NoAvailablePixelFormat]))', /home/XXXXXX/.cargo/registry/src/github.com-1ecc6299db9ec823/egui_glium-0.11.0/src/backend.rs:88:85
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I'm running it on WSL 2 on Windows 10.
I followed the installation instructions and run apt install for all the required packages.
To Reproduce
Steps to reproduce the behavior:

Clone the https://github.com/emilk/egui_template/ repository
Run cargo run (same result with cargo run --release)

Expected behavior
The example should work fine.
Desktop (please complete the following information):

OS: Debian 10 on WSL 2 on Windows 10 20H2

Does the examples at https://github.com/rust-windowing/glutin run?

Does the examples at https://github.com/rust-windowing/glutin run?

No, I get the exact same error message.

I haven't tried this myself, but it seems you will need to install a WSLg driver for your GPU: https://devblogs.microsoft.com/commandline/wslg-architecture/#hardware-accelerated-opengl

I haven't tried this myself, but it seems you will need to install a WSLg driver for your GPU: https://devblogs.microsoft.com/commandline/wslg-architecture/#hardware-accelerated-opengl

Unfortunately WSLg isn't available in the stable channel for now. Besides, I think this should work correctly without it as many heavy GUI-based applications work fine with as long as there is an X server running.

Are you sure it's supposed to work on WSL2 stable? The GUI tutorial says this can only work on Windows 10 Insider Preview build 21362+ as of 2021-04-27.

Are you sure it's supposed to work on WSL2 stable? The GUI tutorial says this can only work on Windows 10 Insider Preview build 21362+ as of 2021-04-27.

As I said, it isn't available as stable for now.
But it's been possible to run GUI applications for a very long time already, by using a third-party X server like Xming.
WSLg is the next step in that direction, allowing to run applications natively on WSL with first-class support without using a third-party server, but it's not available in stable yet.

There might be some confusion.
An X server on Windows (like Xming or VcXsrv) is a remote X server. This works for the X11 protocol, but not for OpenGL. You will need either a hardware or software rasterizer for OpenGL support in WSL. If you cannot use WSLg (especially its hardware rasterizer), then you might be able to use a software rasterizer like libgl1-mesa-swx11.

I see, so in that case it will probably require to wait for WSLg in order for egui to be compatible with WSL at all.

It depends on the backend you want to use. If you are required to use the egui-glium backend, then yes, you will need OpenGL support. But hypothetically even a TUI backend could be written that will render egui into the terminal. 😉 Or a backend that uses X11; your remote X11 server should work with that.
The trouble is, to my knowledge, none of these backends currently exist.

It depends on the backend you want to use. If you are required to use the egui-glium backend, then yes, you will need OpenGL support. But hypothetically even a TUI backend could be written that will render egui into the terminal. 😉 Or a backend that uses X11; your remote X11 server should work with that.
The trouble is, to my knowledge, none of these backends currently exist.

Ok I understand, so it's more like we'll need WSLg to use the OpenGL backend provided in the examples.

The egui_glium backend has been moved out to a separate repo (https://github.com/fayalalebrun/egui_glium), so I'm gonna close this issue


Connection Timeout Causes Sever Crash

Description
There's a random crash after several hours that started happening after an update (I'll dig out the old version soon).
Reproduction Steps
Random, server sitting idle for a while.
Expected Behaviour
No crash.
Server Details
git-Paper-168 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: dcca6cb)
LuckPerms Version
LuckPerms v5.3.89
Logs and Configs
[22:00:24 WARN]: [Vault] Stable Version: 1.7.3 is out! You are still running version: 1.7.2
[22:00:24 WARN]: [Vault] Update at: https://dev.bukkit.org/projects/vault
[02:05:42 INFO]: Stopping server
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@14ce4416 ((conn=2502) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@7e2e90ab ((conn=2503) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@6057aaec ((conn=2505) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@65fbc04 ((conn=2507) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@31e327a4 ((conn=2508) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@3b4242fb ((conn=2509) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@669f8fb5 ((conn=2514) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@1d5bb0fe ((conn=2516) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@35ad7af6 ((conn=2517) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:42 WARN]: [me.lucko.luckperms.lib.hikari.pool.PoolBase] luckperms-hikari - Failed to validate connection me.lucko.luckperms.lib.mariadb.MariaDbConnection@1cd1a322 ((conn=2518) Connection.setNetworkTimeout cannot be called on a closed connection). Possibly consider using a shorter maxLifetime value.
[02:05:47 WARN]: java.sql.SQLTransientConnectionException: luckperms-hikari - Connection is not available, request timed out after 5000ms.
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.HikariPool.createTimeoutException(HikariPool.java:696)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:197)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:162)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.HikariDataSource.getConnection(HikariDataSource.java:100)
[02:05:47 WARN]:         at me.lucko.luckperms.common.storage.implementation.sql.connection.hikari.HikariConnectionFactory.getConnection(HikariConnectionFactory.java:174)
[02:05:47 WARN]:         at me.lucko.luckperms.common.messaging.sql.SqlMessenger.getConnection(SqlMessenger.java:85)
[02:05:47 WARN]:         at me.lucko.luckperms.common.messaging.sql.AbstractSqlMessenger.pollMessages(AbstractSqlMessenger.java:115)
[02:05:47 WARN]:         at me.lucko.luckperms.common.plugin.scheduler.AbstractJavaScheduler$ErrorReportingRunnable.run(AbstractJavaScheduler.java:124)
[02:05:47 WARN]:         at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[02:05:47 WARN]:         at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[02:05:47 WARN]:         at java.base/java.lang.Thread.run(Thread.java:833)
[02:05:47 WARN]: Caused by: java.sql.SQLSyntaxErrorException: (conn=2518) Connection.setNetworkTimeout cannot be called on a closed connection
[02:05:47 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.createException(ExceptionFactory.java:62)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.create(ExceptionFactory.java:171)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.mariadb.MariaDbConnection.setNetworkTimeout(MariaDbConnection.java:1655)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.PoolBase.setNetworkTimeout(PoolBase.java:566)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.PoolBase.isConnectionAlive(PoolBase.java:173)
[02:05:47 WARN]:         at me.lucko.luckperms.lib.hikari.pool.HikariPool.getConnection(HikariPool.java:186)
[02:05:47 WARN]:         ... 9 more
[02:05:48 WARN]: Caused by: java.sql.SQLNonTransientConnectionException: Could not connect to address=(host=REDACTED)(port=REDACTED)(type=master) : Socket fail to connect to host:REDACTED, port:REDACTED. Connect timed out
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.createException(ExceptionFactory.java:73)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.create(ExceptionFactory.java:192)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.protocol.AbstractConnectProtocol.connectWithoutProxy(AbstractConnectProtocol.java:1392)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.Utils.retrieveProxy(Utils.java:635)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.MariaDbConnection.newConnection(MariaDbConnection.java:150)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.MariaDbDataSource.getConnection(MariaDbDataSource.java:302)
[02:05:48 WARN]:         ... 10 more
[02:05:48 WARN]: Caused by: java.sql.SQLNonTransientConnectionException: Socket fail to connect to host:REDACTED, port:REDACTED. Connect timed out
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.createException(ExceptionFactory.java:73)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.util.exceptions.ExceptionFactory.create(ExceptionFactory.java:183)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.protocol.AbstractConnectProtocol.createSocket(AbstractConnectProtocol.java:255)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.protocol.AbstractConnectProtocol.createConnection(AbstractConnectProtocol.java:519)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.protocol.AbstractConnectProtocol.connectWithoutProxy(AbstractConnectProtocol.java:1387)
[02:05:48 WARN]:         ... 13 more
[02:05:48 WARN]: Caused by: java.net.SocketTimeoutException: Connect timed out
[02:05:48 WARN]:         at java.base/sun.nio.ch.NioSocketImpl.timedFinishConnect(NioSocketImpl.java:546)
[02:05:48 WARN]:         at java.base/sun.nio.ch.NioSocketImpl.connect(NioSocketImpl.java:597)
[02:05:48 WARN]:         at java.base/java.net.SocksSocketImpl.connect(SocksSocketImpl.java:327)
[02:05:48 WARN]:         at java.base/java.net.Socket.connect(Socket.java:633)
[02:05:48 WARN]:         at me.lucko.luckperms.lib.mariadb.internal.protocol.AbstractConnectProtocol.createSocket(AbstractConnectProtocol.java:250)
[02:05:48 WARN]:         ... 15 more
<more of these>

Extra Details
Given the timing, and the server setup, this might be related to automatic rebooting. It's possible this occurs when the database shutsdown faster than the main server.

Given the timing, and the server setup, this might be related to automatic rebooting. It's possible this occurs when the database shutsdown faster than the main server.

Sounds plausible.
Seems to be a driver, not sure what I can do in LP to fix (https://luckperms.net/wiki/Storage-system-errors)


--help is broken

% backplane --help
usage: backplane [<flags>] <command> [<args> ...]

Choria Backplane

Flags:
      --help     Show context-sensitive help (also try --help-long and
                 --help-man).
      --version  Show application version.
  -v, --verbose  Enable verbose output
  -d, --debug    Enable debug logging

Commands:
  help [<command>...]
    Show help.

  exec* [<flags>] <service> <action>
    Executes a action against a set of backplane managed services

fine, but try as I might i cant get the detailed exec help, all basically ends in
% backplane help exec
backplane: error: enum value must be one of pause,resume,flip,health,shutdown,ping,info, got 'exec', try --help

$ backplane exec --help
usage: backplane exec [<flags>] <service> <action>

Executes a action against a set of backplane managed services

Flags:
      --help             Show context-sensitive help (also try --help-long and --help-man).
      --version          Show application version.
  -v, --verbose          Enable verbose output
  -d, --debug            Enable debug logging
  -F, --wf=FACTS ...     Match services with a certain fact
  -I, --wi=IDENTITY ...  Match services with a certain Choria identity
      --timeout=TIMEOUT  How long to wait for services to respond
      --config=CONFIG    Configuration file to use
      --insecure         Disable TLS security

Args:
  <service>  The services name to manage
  <action>   Action to perform against the managed service

$ backplane help exec
usage: backplane exec [<flags>] <service> <action>

Executes a action against a set of backplane managed services

Flags:
      --help             Show context-sensitive help (also try --help-long and --help-man).
      --version          Show application version.
  -v, --verbose          Enable verbose output
  -d, --debug            Enable debug logging
  -F, --wf=FACTS ...     Match services with a certain fact
  -I, --wi=IDENTITY ...  Match services with a certain Choria identity
      --timeout=TIMEOUT  How long to wait for services to respond
      --config=CONFIG    Configuration file to use
      --insecure         Disable TLS security

Args:
  <service>  The services name to manage
  <action>   Action to perform against the managed service

seems fine now


Dartpad crashes when using draggable

Wanted to create an issue for another bug I found, but dartpad can't even get to the actual bug.
Open https://dartpad.dev/909fcfbd3bb0df0102ed322236b3ce81 and try to drag the number in "from" into "to".
After messing up the formatting of the feedback Widget when losing the theme, dartpad seems to crash once you release the mouse. There are no errors in the console.

Hi @Lootwig please report this to the dedicated dartpad repository here https://github.com/dart-lang/dart-pad/issues
I'll go ahead and close this now, if you disagree please write in the comments and I will reopen it.

@mamuseferha https://github.com/dart-lang/dart-pad/issues/2107 they don't want to have this issue either ;)

Hi @Lootwig, I ran this locally using canvaskit rendering and got the same error here https://github.com/dart-lang/dart-pad/issues/2107#issuecomment-992473119.
Running in html rendering and no error is thrown flutter run -d chrome --web-renderer html.
It looks like a duplicate of this issue https://github.com/flutter/flutter/issues/94906, you can follow up there. I'll be marking this as a duplicate and keeping it closed.
Thank you.


Unable to Move

I'm unable to move in the game using the stick. The stick works fine using the teleport locomotion, but without enabling that, Henry is stuck. All other controls seem to work fine, and changing the settings for controller-based movement and swapping the left and right hands doesn't resolve the issue. I'm not sure if I'm missing something or if this is an issue specific to my setup. Other than this, the mod works great and I'm enjoying it very much!
I'm using an HP Reverb G2 with the included controllers and I'm running the game through Steam.

So you're stuck right at the elevator at the beginning? Or did you get stuck some time after?
Try going to the SteamVR input bindings settings and check if the bindings for smooth movement look correct. The reverb G2 is supposed to use the same bindings as Oculus, and those seem to be working fine, so this is a bit strange.

This happens anywhere, whether it's in the elevator in a new game or wherever I left off in a save. But like I said, pressing the stick forward to teleport fixed camera mode works fine.
Testing the controllers shows that all inputs are working, and I tried it in your Outer Wilds mod (which, may I add, is excellent) and it works there, so it's not a hardware issue.
As far as I can tell, the bindings all look correct. I've tried modifying them as well as changing the controller type, but to no effect. I've attached a picture of my movementhand bindings, but they look fine to me. Let me know if there's anything else I can provide that may be helpful.

Is smooth rotation working correctly for you?

Yep, that works fine.

A clean install of Windows resolved this.


Reuse work when converting functions to AST

While converting Term to AST, we currently process every term in function position (using toFun), even if that function has already been processed. We should probably keep track of what we've already processed, and reuse that work.

@bts I think this got fixed so closing


[NTR] SCN4-related myopathy, autosomal recessive

Preferred term label:
SCN4A-related myopathy, autosomal recessive
Synonyms

Congenital myopathy with "corona" fibers, selective muscle atrophy, and craniosynostosis
Myopathy with ptosis and mild dystrophic pattern
Congenital Myopathy with severe fetal hypokinesia

Definition (free text, please give PubMed ID, if applicable, in format PMID:#######)
Parent term (use OLS, or your favorite ontology browser)
Congenital Myopathy, MONDO:0019952
Your nano-attribution (ORCID)
@nicolevasilevsky this is a submission from the ClinGen LGMD GCEP! Thank you.

Hi @brookepalus could you please give me a bit more information? Is there a paper (PubMed ID) that I can reference, and can you suggest a text definition to describe this term?
Thanks!

Hi Nicole,
I apologize for the lack of information. I am putting in this request on the behalf of one of our biocurators, so please let me know how else I can clarify the request beyond what I have included below.
I also made an error in the original request, and the new MonDO term should be: SCN4A-related myopathy, autosomal recessive  (where originally I left off the A). Please let me know if you can alter the original request or if I should put in a new request.
Here is a description of the term:
SCN4A related myopathy is a spectrum of autosomal recessive disorders including congenital myasthenic syndrome, fetal hypokinesia, and congenital myopathy that are related to pathogenic variants in SCN4A gene.
There are no PMIDs that specifically refer to this term. The decision to lump the above mentioned congenital myopathies was made by the ClinGen LGMD Gene Curation Expert Panel, who determined that these conditions represent a phenotypic spectrum and not separate disease entities and thus should be lumped under an umbrella term. The PMIDs that were considered in this decision include: PMID 30824560; PMID 28003497; PMID 26700687; PMID 28003497; PMID 28262468; and PMID 28003497.
Please let me know what other information I can provide to help clarify.
Many Thanks!
Brooke

Thanks Brooke, this is great!

Hi Nicole,
I just wanted to check in regarding this MonDo ID request. Do you have an estimated time line for approval?
Many thanks,
Brooke

This should be available in the next release at the end of this month/early next month. Thanks for your patience.
The new Mondo ID is: MONDO:0100121

@brookepalus sorry for the delay, this should be available in the release at the end of July

Hi @nicolevasilevsky, it appears that this Mondo ID is still not live. Does it appear to be live on your end or do you have any updates you can share?
Many thanks,
Brooke

Hi @brookepalus apologies for the delay, the release has been slightly delayed and should be ready by the end of next week.


Added port customization to liveness probe for the statefulset template

What this PR does / why we need it:
Added port customization to liveness probe for the statefulset template
Without this using a custom port will fail if ports are not 1389 and 1636
Pre-submission checklist:

[X] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you updated the readme?
[X] Is this PR backward compatible? If it is not backward compatible, please discuss open a ticket first

Hi @thekoma ,
Thanks for the PR.
did you try with the latest version of the chart because
global:
  ldapPort: 389
  sslLdapPort: 636

Set ports to something different than containerPort which are only used internally (non privileged port, replication port and probe)
Liveness and Readiness probe are working in this configuration.

I've copied the values.yaml in toto and it didn't work withotu changing
those values.
Also an ss -tnlp showed me that the listening port for the container are
389 and 636.
Best Regards,
Andrea Cervesato
Phone: +39.3<IP_ADDRESS>1
Mail: @.***

This email and its attachments might be confidential and are only for use
of intended recipient. If you are not the intended recipient, please notify
us immediately by sending back the email and delete it from your machine.
You are not allowed to disseminate nor to copy the content of the email if
you are not the intended recipient.
On Thu, 19 Jan 2023 at 10:29, jpgouin @.***> wrote:

Hi @thekoma https://github.com/thekoma ,
Thanks for the PR.
did you try with the latest version of the chart because
global:
ldapPort: 389
sslLdapPort: 636
Set ports to something different than containerPort which are only used
internally (non privileged port, replication port and probe)
Liveness and Readiness probe are working in this configuration.
—
Reply to this email directly, view it on GitHub
https://github.com/jp-gouin/helm-openldap/pull/90#issuecomment-1396678416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB4LATNKHZPS7D3FDH7X5TWTECQDANCNFSM6AAAAAAT7U2T3M
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you copy the full values your are using ?

Sure:
https://gist.github.com/thekoma/a7d3d84bb1e3e249d6b3daa2aad4be3e

Thanks , you are using the old configuration (values from 3.x.x chart version )
Now the chart is in v4.x.x and the values has changed along the openldap image
image:
      pullPolicy: Always
      repository: osixia/openldap
      tag: 1.5.0

is now
image:
  # From repository https://github.com/osixia/docker-openldap
  repository: bitnami/openldap
  tag: 2.6.3

If you want to upgrade to the latest version , I suggest you deploy the newer version, dump all your ldap tree from the previous version. Import them in the newer and check everything is fine before removing the older release


Broker should not serve requests until it is ready

See discussion on #2942 .
The broker opens port 6650 before it is actually fully initialized, which can cause clients to see weird things (broker bounces and resends all unacknowledged messages). This can happen if there are still ephemerals from the previous instance in zk, so the broker would keep bouncing until session timeout and the client would end up getting a bunch of redeliveries.

Closed as stale. Please open a new issue if it's still relevant to the maintained versions.


Update README.md

Add two Docker Open Source Documentations
Summary
Did you read the CONTRIBUTING.md?
If Yes, proceed, else read it Contributing GUidelines
Any relevant issues this PR addresses? (optional)
#

Noted


ActionBar Hidden on Android Emulator but Gap Remains to be removed

After much tinkering, the action bar is hidden but my content only goes to the bottom of where it was. In the following example, notice the space before a custom nav component displays Example of issue
Is anyone having trouble setting, styling, and removing the action bar? This seems like a tricky operation for me and think there might be some disconnect between rollup and tns build. I've hacked various style.xml files to comment out anything with toolbar or actionbar, leaving just
   <style name="AppTheme" parent="Theme.AppCompat.Light.NoActionBar">     </style>
I'm sure there's a right way to do this and pretty sure I'm not doing it. When I remove the page attributes below, the action bar doesn't reappear so I'm feeling a bit lost in what is actually doing work. I've removed and re-added the android platform, tns build, etc and nothing is indicating causality!
My template on main.js is:
  template: `
    <page ref="page" actionBarHidden="true" backgroundSpanUnderStatusBar="false" :class="pageClasses">
      <absolute-layout class="splash" top="0" bottom="0" right="0" left="0">
        <router-view></router-view>
      </absolute-layout>
    </page>
    `,

Can you try and reproduce it on https://play.nativescript.org/?template=play-vue ?
I just tried this, and I don't see any extra space above the label: https://play.nativescript.org/?template=play-vue&id=5wbEvdsl-w0Y9JNUJjiw0

Thanks. My app's too convoluted to put up there right now. I'll try a fresh installation and see if that solves it before investing the time to mirror on playground. Btw, I'm using the Nativescript-Vue Grocery example as a starter so there may be artifacts that won't apply to a fresh install. Will be back soon.

Can't reproduce with a fresh install (rollup template). Let's close this unless anyone else runs into a similar issue. Appreciate the attention!


Deployment is 9788 files

How can I make the installation more lightweight? Current deployment seems like to have 9788 files out of which are 7227 php files.
# find BookStack-0.20.3/ -type f | wc -l
9788

Closing since conversation has come to a close. Semi-convered / related to #161.


DeepDive execution plan compiler

This not-so-small PR adds a new implementation of DeepDive that literally compiles an execution plan to run the app in a much more efficient way in both human time and machine time/space.  I invite everyone to review the code and give a try on your existing app (especially @feiranwang @SenWu @ajratner @zhangce @raphaelhoffmann @alldefector @zifeishan @xiaoling @Colossus @ThomasPalomares @juhanaka)  The plan is to get feedback over the next few days while I update the documentation and carve out release v0.8.0.
An execution plan is basically a set of shell scripts that tells what to run for user-defined extractors as well as the built-in processes for grounding the factor graph and performing learning and inference, and a Makefile and that describes the complete dependencies among them.  These are compiled from the app's deepdive.conf, app.ddlog, and schema.json, mainly by a series of JSON transformations implemented as jq programs with a little bit of help from bash, which all resides under compiler/ in the source tree.
It implements most of the existing functionality provided by the current Scala implementation, except a few things, e.g., Greenplum parallel unloading/loading, which won't be difficult to add given the much much modular architecture.  On the other hand, exciting new features and improvements are added.  To highlight just a few:

Full Dependency Support with Selective Execution. It's now possible to selectively run, repeat, or skip certain parts of the app's data flow or extraction pipeline without being aware of all the dependencies between them. (fixes #431, closes #427, closes #273)  The user has full control over every step of the execution plan.  Not only that, but grounding is also broken down into smaller processes, so it's possible to just change or add one inference rule and update the grounded factor graph without having to recompute everything from scratch. (fixes #280)
Zero Footprint Extraction. tsv_extractors now have nearly zero footprint on the filesystem.  The data is streamed from the database through the UDFs and back to the database.  mkmimo is used with named pipes to make the connection between the database and the parallel UDF processes (cf. $DEEPDIVE_NUM_PROCESSES). (fixes #428)  It doesn't support other extractor styles yet, but we can probably drop them unless there's a compelling reason. (closes #384)
Compute Drivers. A new compute driver architecture for executing such extractors is now in place, so it's now clear where to extend to support remote execution or clusters with a job scheduler, such as Hadoop/YARN, SLURM, Torque/GridEngine/PBS. (#426)  The local execution driver is what implements the streaming tsv_extractor mentioned above.  Moreover, the grounding processes as well as the user-defined extractors also make use of the compute drivers, so some parts of the grounding will automatically take advantage of such extensions.
Zero Footprint Grounding. The grounding processes also minimizes footprint on the filesystem.  No data for the factor graph are duplicated.  Instead of creating a concatenated copies of factors, weights, and variables, they are merged as the sampler loads them.  Also, the binary format conversion is done on the fly as the grounded rows are unloaded from the database, so no ephemeral textual form is ever stored anywhere.  In fact, only a few line changes to the compiler can compress the binary forms and shrink the factor graphs footprint on the filesystem by an order of magnitude (not included in this PR).
More User Commands. deepdive.pipeline config is obsolete as well as the deepdive run command, although they'll work the same as before.  Now, the user can simply state what is the goal for execution to the deepdive do command, e.g., deepdive do model/calibration-plot or deepdive do data/has_spouse_features, as many times as necessary once the app is compiled with the deepdive compile command.  Supporting commands such as deepdive plan, deepdive mark, deepdive redo are there to speed up typical user workflows with DeepDive apps.  deepdive initdb and deepdive load have been improved to be more useful, and is used by a few compiled processes. (fixes #351, fixes #357)
Error Checking. Errors in the app (and of course the compiler itself) are checked at compile time, and can also be done by the user with the deepdive check command.  The checkers are modular and isolated, so many useful checks can be quickly added, such as test firing UDF commands, etc.  Only basic checks have been implemented so far. (fixes #349, fixes #1)
Simpler, Efficient Multinomial Factors. Multinomial factors are won't materialize unnecessary data and use VIEWs as much as possible, e.g., dd_*_cardinality tables and the dd_graph_weights.  Also, nearly no code has been duplicated to support it, so that's good news for developers.
Bundled Runtime Dependencies. DeepDive now builds essential runtime dependencies, such as bash, coreutils, jq, bc, graphviz, so no more Mac vs. Linux or software installation version issues will pop up. (fixes #441 as documentation also ended up in this PR)

Also some good stuffs that comes with a clean rewrite:

Closes #412 by doing a reasonable job and making the base relations clear to the user, and also closes  #421.
Closes #110 as no more SQL parsing is done.
Closes #383 as logging has been completely redone.
Closes #361, closes #20 as JDBC is no longer used.
Closes #329 as Scala implementation will be simply dropped in a future release (v0.8.1 or v0.9?).

Wow.

That's crazy

Sounds like 1.0 instead of 0.8 :)

Let's not get crazy.
On Wed, Dec 16, 2015 at 6:29 PM alldefector<EMAIL_ADDRESS>wrote:

Sounds like 1.0 instead of 0.8 :)
—
Reply to this email directly or view it on GitHub
https://github.com/HazyResearch/deepdive/pull/445#issuecomment-165317353
.

FYI Here's the SVG rendered data flow for spouse example and data flow for chunking example.
I forgot to mention that this PR requires no change to the user's app code, except maybe a few extra dependency information that were missing in the manually written deepdive.conf.  The run.sh that typically handled env setup and many other hacks should be moved to the right place such as input/.  @ThomasPalomares and I were able to get the genomics app (in DDlog) working with minimal changes.

This is awesome :)
On Thu, Dec 17, 2015 at 7:25 AM Jaeho Shin<EMAIL_ADDRESS>wrote:

FYI Here's the SVG rendered data flow for spouse example
https://gist.github.com/netj/f4abe6eff63b837c7046#file-dataflow-svg and data
flow for chunking example
https://gist.github.com/netj/a4e363b92ffeb2fa95b7#file-dataflow-svg.
I forgot to mention that this PR requires no change to the user's app
code, except maybe a few extra dependency information that were missing
in the manually written deepdive.conf. The run.sh that typically handled
env setup and many other hacks should be moved to the right place such as
input/. @ThomasPalomares https://github.com/ThomasPalomares and I were
able to get the genomics app (in DDlog) working with minimal changes.
—
Reply to this email directly or view it on GitHub
https://github.com/HazyResearch/deepdive/pull/445#issuecomment-165483060
.

The deepdive sql '[multiple SQL queries]' does not work under postgres-XL. It's probably necessary to manually segment these queries into individual ones when compiling the run scripts.

Hi @netj, this new version of deepdive is really great! Here are a few notes I took while experimenting.

When building on OSX, I got error that X11/Xlib.h couldn't be found. This error
was thrown by buildkit module for graphviz.
/usr/include/tk.h:78:23: fatal error: X11/Xlib.h: No such file or directory

Following these instructions, I fixed it by adding a symlink
ln -s /opt/X11/include/X11 /usr/local/include/X11

Afterwards, it was not sufficient to rerun 'make'. I manually cd'ed to deepdive/depends/bundled/graphviz/graphviz-2.38.0 and ran ./configure && make && make install. Then I could run "make" in deepdive

ddlog apps didn't work with the default submodule commit, but – following your
suggestions – everything worked fine after upgrading to ddlog commit ee0b359c9f018c19df5eddbf3bff60bdf076e091.

Now everything works fine, but I've encountered two different problems in a small number of runs:

Sometimes, stdin is no longer showing on the screen after a successful deepdive do
execution. That is, I can type commands and they execute, but the letters I type are not
being shown on the screen. I'm using iterm2.

Sometimes (rarely), a mkmimo process kept running at 100% cpu usage, after deepdive do
terminated with an error (possibly related to missing input data).

I'll try to investigate more carefully if I encounter these again.

Great feedback!!
On Tue, Dec 29, 2015 at 12:42 PM Raphael Hoffmann<EMAIL_ADDRESS>wrote:

Hi @netj https://github.com/netj, this new version of deepdive is
really great! Here are a few notes I took while experimenting.

When building on OSX, I got error that X11/Xlib.h couldn't be found.
This error
was thrown by buildkit module for graphviz.
/usr/include/tk.h:78:23: fatal error: X11/Xlib.h: No such file or directory
Following these
http://stackoverflow.com/questions/11465258/xlib-h-not-found-when-building-graphviz-on-mac-os-x-10-8-mountain-lion
instructions, I fixed it by adding a symlink
ln -s /opt/X11/include/X11 /usr/local/include/X11
Afterwards, it was not sufficient to rerun 'make'. I manually cd'ed to
deepdive/depends/bundled/graphviz/graphviz-2.38.0 and ran ./configure
&& make && make install. Then I could run "make" in deepdive
2.
ddlog apps didn't work with the default submodule commit, but –
following your
suggestions – everything worked fine after upgrading to ddlog commit
ee0b359c9f018c19df5eddbf3bff60bdf076e091.
Now everything works fine, but I've encountered two different problems in
a small number of runs:

Sometimes, stdin is no longer showing on the screen after a successful deepdive
do
execution. That is, I can type commands and they execute, but the
letters I type are not
being shown on the screen. I'm using iterm2.
2.
Sometimes (rarely), a mkmimo process kept running at 100% cpu usage,
after deepdive do
terminated with an error (possibly related to missing input data).
I'll try to investigate more carefully if I encounter these again.
—
Reply to this email directly or view it on GitHub
https://github.com/HazyResearch/deepdive/pull/445#issuecomment-167874340
.

@zifeishan Regarding multiple SQL statements, I'm reluctant to do a blind split on semicolons, as any string literal may contain it.  Do you know why PGXL is not supporting them or any workaround?  One safe way for DD to deal with multiple statements is to support an array of strings for the sql field for sql_extractors, although this will break backward compatibility (but who cares! : )  How does it sound?
@raphaelhoffmann Thanks for your detailed feedback!

The runtime dependency build hasn't been thoroughly tested, hence such bugs.  Specifically on graphviz, I feel the build can be simplified without pulling in X11.  Will come up with a fix.
The submodule build has been modified to honor the developer's working tree, so whatever is on disk gets built, as opposed to forcing all submodules to be the exact version recorded in deepdive's commit or the git index.  So forgetting to run git submodule update --init before a build can now result in an unexpected result.  Previously, running build without a git add path/to/submodule wiped out changes to the submodule, upsetting people who were modifying them.  The previous behavior seems to be upsetting less number of people, so I'll revert that or maybe abort the build instead of silently making surprises under the hood.
The terminal messup is due to the pv command showing progress bar, competing with the logs printed to the same terminal.  I tried to reduce that error but haven't figured out how to completely eliminate that.  As a workaround you can run reset whenever that happens by blinding typing it followed by an return/enter.  We need a tty expert here.  There should be a simple invisible escape sequence we can print to ensure the terminal is restore in a sane state.
mkmimo had a busy waiting bug on OS X where the poll(2) syscall returns too fast with slow sinks.  I added a fix for it in f2fabb16cd5b6efab237bb7bf9ec29c97971d851, but kept it separate to keep this giant PR stable for review for a while.  Since we've got some eyes and hands over the code, I can bring in the latest changes, but that'll require another round of tests to be safe.

@netj
This seems to be a bigger issue than sql_extractor for PGXL: I am trying out some other extractors but it had other problems with this line: https://github.com/HazyResearch/deepdive/blob/dataflow-compiler/runner/compute-driver/local/compute-execute#L123 since it's also calling multiple queries with deepdive sql. I guess a cleaner workaround would be splitting queries for  https://github.com/HazyResearch/deepdive/blob/dataflow-compiler/database/db-driver/postgresql-xl/db-execute?
Regarding the PGXL issue, I could not find discussions for it online. A sample query that would fail on my environment:
$ psql $DBNAME -c  "create table tmptmp(id int);
insert into tmptmp values(1);
select * from tmptmp;
drop table tmptmp;"
ERROR:  Unexpected response from the Datanodes for 'T' message, current request type 1
ERROR:  Unexpected response from the Datanodes for 'T' message, current request type 1
ERROR:  Unexpected response from the data nodes for 'D' message, current request type 0

Server version: Postgres-XL 9.2.0.

@zifeishan If this is a blocker, we can quickly apply the workaround to pgxl driver, which is actually super simple.  db-execute can call psql multiple times after splitting the query argument by semicolon and that'll handle everything.  Wanna take a stab at it?

@netj I just found an alternative and it works: instead of
deepdive sql  "create table tmptmp(id int);
insert into tmptmp values(1);
select * from tmptmp;
drop table tmptmp;"

, if I run:
echo  "create table tmptmp(id int);
insert into tmptmp values(1);
select * from tmptmp;
drop table tmptmp;" | deepdive sql

it will work for PGXL.

@zifeishan Interesting.  However, that'll take away stdin for psql.  Is there a way to tell psql a sql file to run instead?

@netj Yes. Try psql -f commands.sql. It works with pgxl as well.

@zifeishan @netj are you guys suggesting that we write $sql to a temp file and use -f to run it (so that the whole block isn't run as one transaction)? Sounds like it should work. We only need to update PGXL's db-execute driver, right?

@alldefector Sorry the conversation went private through Slack.  Yes, that's exactly our plan, except using bash processes substitution to turn $sql into a readable file.  This is so much better than dangerously splitting the SQL queries in a sloppy way.  (btw you mean the block is run as a transaction, right?)
@zifeishan Could you make the changes, confirm it working, and add the commit to this PR?

Apparently -f does not run the file in one transaction unless you also specify -1 or --single-transaction: http://postgres-xc.sourceforge.net/docs/1_0/app-psql.html
That's actually what we need because PGXL would throw up (Zifei's error messages) if we try to run a bunch of mutating / DDL statements in one transaction.

@netj I tried this on several examples. deepdive run and  deepdive do all work great. I have a small problem with deepdive do. When I type deepdive do, it gives a list of targets. In smoke example there's a process/grounding/variable/person_has_cancer/assign_id. When I tried to run with that target, it gives an error
process/grounding/variable/person_has_cancer/assign_id: Unknown target

@feiranwang Can you try putting a .done suffix to the target? Maybe I forgot to add that case. Will also take a look into this.

@netj  I added a .done suffix to the target and it gives the same error
process/grounding/variable/person_has_cancer/assign_id.done: Unknown target

@feiranwang It turns out to be a missing schema.json issue, so I converted schema.sql into app.ddlog as an easy way to generate the json file.  The compiled Makefile was missing some dependencies for the mentioned target, which gets generated from the relational schema.  I fixed things to show such errors more transparently.  Ideally, these should be caught in the deepdive compile phase.  It'd be nice if you can contribute a checker that prevents this kind of error.  Also, it seems we're missing a test for the smoke example.


Package should reflect go gettable path

If you want I can do magic to make this appear go gettable from pathspider.net.

Nope, forgetting Go again.
This is not the correct way to do this, but it turns out that this isn't even a needed import anyway, so this whole thing can just be ignored.


using the placeholder value if nothing is specified in rules on a new…

… variable fixes #564

This would be better fixed by injecting that into the template, not the controller.

So just as the value instead? or how do you see it implemented?

@KasperFranz Sorry, thought I had replied to this, but yes. I would prefer the form have an auto-filled value, rather than defaulting to something in the code. 👍

@DaneEveritt  - That is now changed :)


Support Special Buildings

Support buildings that have a specific song, like the Pokemon Center, Poke Mart, Eterna Galastic Building, etc.

Done. Will open separate issues for any other specific buildings that are not yet supported.


[Go Live][March 22]

Docs Build status updates of commit 4f862f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/aad-icon.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/azure-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/chrome-icon.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/edge-icon.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/linux-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/macos-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/microsoft-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/sidebar-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/teams-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/vsc-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/windows-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/additional-information.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/blazor/authorize-azure.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/authorize-user-info.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/azure.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-add-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-add-app1.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-add-bot-app-local.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-app-solution-explorer.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-azure-site-url.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-bot-app-local.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-bot-app-solution-explorer.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-solution-explorer.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-tab-app-azure.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-tab-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-vs-app-service.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/blazor-vs-publish.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/configure-new-project.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/blazor/create-new-project.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/blazor/hwb-app-created.png
:white_check_mark:Succeeded

msteams-platform/assets/images/teams-toolkit-v2/blazor/install-yourapp.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/local..PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/ms-graph-permission.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/provision-warning.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/resource-group.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/select-subscription.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/tab-azure.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/tab-user-info.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/upload-custom-app.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/visual-studio-installer.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-bot-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-build-deploytocloud.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-build-deploytocloud1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-build-preview.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-build-provision.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-build-publish.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-configure-bot-project.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-configure-project.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-toolkit-v2/blazor/vs-create-project.png
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fixed wishlist display when title is too long

Questions
Answers

Description?
If the wishlist title is too long, the display is broken

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/issues/29213

How to test?
See https://github.com/PrestaShop/PrestaShop/issues/29213

Possible impacts?

I also fixed the display in the popin when clicking on the "add to wishlist" button.
Thank you @NeOMakinG

Hello @florine2623.
I tried on Firefox and Chrome and I can't reproduce it. Could you please send me more details about your environment?

Hello @nicosomb ,
Sorry for the late answer.
I retested on Chrome 105.0.5195.125, Firefox and Opera and I reproduced the issue I'm having.

More info about my shop:

I have well installed the PR:

I have deleted my browser cache and my PrestaShop cache as well.
Tested on 8.0.0. and 1.7.8.x, the issue is the same on both versions.
I hope the info I share can help ^^

ping @nicosomb :)

Hello @paulnoelcholot.
I made new tries ... and I still can't reproduce the bug (firefox or chrome).

What can we do to go forward on this issue?

@nicosomb I see florine2623 and paulnoelcholot used narrower screen mobiles than yours. Maybe the bug happens with under 380px wide screen.

@paulnoelcholot @florine2623 I think that I fixed the bug.

@MhiriFaten hmmm what is the issue on your screenshot?

@nicosomb ,
Contrary, the screenshot is for approve that the issue is well fixed in the wishlist (as it is marked just above the screenshot Fixed wishlist display when title is too long :white_check_mark: ).
The problem is in the popup wishlist as shown in the screenrecord.

ok 😄 I didn't see the video on my smartphone, but I can see it on my computer. I will have a look on this bug later.

@MhiriFaten I tried to reproduce with Firefox and Chrome, but I can't. The popin is well displayed on my side.


[Bug] GIFs are not being removed from favorites after reloading the page

Description
Description:
When clicking the remove button on the favs.html page, the GIF is successfully removed from the view. However, after refreshing or reloading the page, the GIFs that were removed reappear in the favorites list. It seems like the deletion is not being persisted in localStorage.
Steps to Reproduce:

Go to the Favorites page (favs.html).
Click on the remove button next to any GIF.
Observe that the GIF is removed from the page.
Refresh or reload the page.
The GIF that was removed reappears in the favorites list.

Expected Behavior:
The GIF should be permanently removed from the favorites list after clicking the remove button, and this change should persist even after a page reload.
Actual Behavior:
After clicking the remove button, the GIF is temporarily removed from the view, but reappears after the page is refreshed or reloaded, indicating that the change is not persisted.
Suggested Fix:
Ensure that when the remove button is clicked, the corresponding GIF is properly deleted from localStorage to persist the change across reloads.
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.

Hey @lcrojano,
Please assign me this issue.


I am unable to re-authenticate alexa_media_player api

Description:
Previously, I was never able to get the HA built-in F2A to work as expected; consequently, I need to re-auth approximately weekly. I am not able to recall the previous success methods as they all had varied until initialization success. Here are the steps I am attempting:

Removed {#} alexa_media entries from configuration.yaml. Restart HA.
Removed alexa media player from HACS. Restart HA.
Enabled alexa_media entries in configuration.yaml. Restart HA.
Configure (Discovered) Alexa Media Player in integrations.
Verified prefilled fields are all correct: EMail Address, Password, duckdns.org link, Built-in F2A App Key (Seed) from Amazon F2A page for adding second S2V authenticator:

With repeated efforts, I would get one of two errors: Unknown error occurred or unable to connect.

For clarification purposes, the https address for HA in duckdns.org is verified by coping it into a browser and opening HA.

System details
Home Assistant Core 2022.6.6
Home Assistant Supervisor 2022.05.3
Home Assistant OS 8.2
Alexa Media Player v4.0.2 located in '/config/custom_components/alexa_media'
Path to configuration.yaml: /config/configuration.yaml

Amazon 2FA is enabled (yes).

Logs
home-assistant.log:
2022-06-21 09:28:24 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration alexa_media which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-06-21 09:28:24 WARNING (SyncWorker_1) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-06-21 09:28:31 ERROR (MainThread) [homeassistant.components.notify] Error setting up platform smtp
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/notify/legacy.py", line 67, in async_setup_platform
notify_service = await hass.async_add_executor_job(
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 92, in get_service
if mail_service.connection_is_valid():
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 154, in connection_is_valid
server = self.connect()
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 144, in connect
mail.starttls(context=ssl_context)
File "/usr/local/lib/python3.9/smtplib.py", line 790, in starttls
self.sock = context.wrap_socket(self.sock,
File "/usr/local/lib/python3.9/ssl.py", line 500, in wrap_socket
return self.sslsocket_class._create(
File "/usr/local/lib/python3.9/ssl.py", line 1040, in _create
self.do_handshake()
File "/usr/local/lib/python3.9/ssl.py", line 1309, in do_handshake
self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'mail.myfairpoint.net'. (_ssl.c:1129)
2022-06-21 09:28:31 ERROR (MainThread) [homeassistant.components.notify] Error setting up platform smtp
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/notify/legacy.py", line 67, in async_setup_platform
notify_service = await hass.async_add_executor_job(
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 92, in get_service
if mail_service.connection_is_valid():
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 154, in connection_is_valid
server = self.connect()
File "/usr/src/homeassistant/homeassistant/components/smtp/notify.py", line 144, in connect
mail.starttls(context=ssl_context)
File "/usr/local/lib/python3.9/smtplib.py", line 790, in starttls
self.sock = context.wrap_socket(self.sock,
File "/usr/local/lib/python3.9/ssl.py", line 500, in wrap_socket
return self.sslsocket_class._create(
File "/usr/local/lib/python3.9/ssl.py", line 1040, in _create
self.do_handshake()
File "/usr/local/lib/python3.9/ssl.py", line 1309, in do_handshake
self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'mail.myfairpoint.net'. (_ssl.c:1129)
2022-06-21 10:17:16 WARNING (SyncWorker_6) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
The Alexa Media Player integration is the backbone of my HA. Any assistance is greatly apricated. Please let me know How I can return the favor.

Ditto here.
Region Amazon.com

Same for me, but with amazon.es

It seems the '2FA Code' option box has dissapeared in this release, now we only have the 'Built-in 2FA App key' option, which is not for use with a generated key from Authenticator.
If you leave that box empty and hit 'Submit' it takes you to an amazon page where you can enter the 2FA key, and all works well, Alexa Medai Player is now authenticaed and working well, but if I restart Homeassistant, then it need re-authenticating all over again.
Anyone on here from the dev team can point us in the right direction please?

ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'mail.myfairpoint.net'. (_ssl.c:1129)

There's a SSL host mismatch. Integration is trying to test the url you provide from the HA machine. If for some reason the cert fails, it will give that error. Use the local IP as a workaround or correct your cert as exposed to your HA machine.
Everyone else, if you're not talking about a SSL cert issue, you're off topic. Please search as almost everything asked has been already discussed and closed. Please open a new issue if your issue is actually new.

@alandtse
I simply can not express how much your direction is appreciated. I wanted to share my work-around. You mentioned correcting my SSL cert (Note: Using HACS add-on NGINX Home Assistant SSL Proxy). Not knowing how, I reinstalled using my local http address. This gave me the success But could not be loaded message:

My workaround Was to delete and reload with the exact same instructions, but this time on the Amazon Login Authentication Screen, I enabled the option to not require F2P and it complete the integration.
As mentioned, This is the most important HA integration. without it, HA is nothing more than a status app.
THANK YOU!!!


Remove visualisation code from code library

sure but also on the non-Rhino side it would be better to move this functionality to artists and objects of the plotter and/or viewer...
Originally posted by @brgcode in https://github.com/BlockResearchGroup/compas_ags/pull/102#discussion_r573280577

@ricardoavelino This comment has been "resolved," as pull request #102 was merged.


pylint updates for latest version.

Summary Change Description
There was an update to pylint that remote GH superlinter runs are using and generating errors.  Updates were made to eliminate those errors.
Affected Issue Numbers

Resolves #487

Code Review Notes
non
Checklist

[x] All issue requirements satisfied (or no linked issues)
[x] Linting passes.
[x] Migrations created & committed (or no model changes)
[x] Tests implemented (or no code changes)

[x] New test classes/functions tagged with @tag("multi_working")

[x] All (ci-run) tests pass

[x] Test function tag @tag("multi_working") added and test class/function tags @tag("multi_broken"), @tag("multi_unknown"), or @tag("multi_mixed") removed or changed.

The natural language error was fixed in a separate branch/PR


[feat] Try to optimize merkle_tree::merkle_root

[x] issue #50
[x] follows contribution guide
[ ] code change includes tests

I tried to improve performance by rewriting compute_sha256_u32_array to be more specific to our use case.

Hi @Jeanmichel7  I can see you mainly copy pasted what I did in my PR
Why just not commenting for suggestion instead opening a new one?


glfwGetMonitors() returns NULL on latest Windows 10 AMD drivers

One of our developers just updated his laptop's graphics drivers which reproduced an issue that he had a few months ago that I decided not to report as it seemed like a one-time fluke which disappeared and we were unable to reproduce it again. He has an AMD Radeon R9 275M graphics card in combination with a switchable integrated Intel card, and with the current drivers glfwGetMonitors() returns NULL regardless of which card he's using. He's currently asleep, but considering that something as basic as glfwGetMonitors() isn't working correctly I don't think there's anything else we can do to diagnose this further without help from you guys, so I thought I'd go ahead and report this now to give you some extra time.

He woke up and we ran some new tests. Seems to have been some wonkiness with LWJGL as after updating we resolved some problems with the native files. Looks like this one has already been fixed, whatever it was. =P


Error when using FastAPI request state variable AND also body parameter

I have a FastAPI POST endpoint which receives a parameter in the request body. A global FastAPI Depends sets a Request state value which I want to retrieve in my method.
The following works fine:
from pydantic import BaseModel
from fastapi import APIRouter, Request

class MyPydanticValue(BaseModel):
    # some properties here

router = APIRouter()

@router.post(
  "/foo"
)
def perform_action(
  my_val: MyPydanticValue
  request: Request
):
  # do something with my_val
  ...

  # and also reference a state value that was set globally elsewhere
  print(request.state.x_some_value)

FastAPI sees the my_val parameter, recognises that it is a Pedantic model and therefore parses it from the request body.
I now want to cache the function, so I use @cache:
@router.post(
  "/foo"
)
@cache(expire=300)
def perform_action(
  my_val: MyPydanticValue
  request: Request
):
  # do something with my_val
  ...

  #and also reference a state value that was set globally elsewhere
  print(request.state.x_some_value)

Now I get an exception:
TypeError: perform_action() got multiple values for argument 'my_val'

The fastapi-cache documentation states:

The cache decorator injects dependencies for the Request and Response objects, so that it can add cache control headers to the outgoing response, and return a 304 Not Modified response when the incoming request has a matching If-Non-Match header. This only happens if the decorated endpoint doesn't already list these dependencies already.

My reading of that is that the injection shouldn't happen because I declare a parameter referring to the Request already. I think that's what is happening in the source code for the @cache decorator: https://github.com/long2ice/fastapi-cache/blob/v0.2.1/fastapi_cache/decorator.py#L41-L46. However, I think that my function is indeed actually getting a request parameter being injected by fastapi-cache and then subsequently by FastAPI. Alternatively, something that the cache wrapper is doing is messing up the parsing of the body.
What am I doing wrong - is this a bug? If so, is there a way of working around this? How do I access the state value in the endpoint function whilst also parsing the Pydantic model from the request body?

You can try install main.
pip install git+https://github.com/long2ice/fastapi-cache.git

But if the post method you use will not cache, if you want to test, you can modify it to return false.
https://github.com/long2ice/fastapi-cache/blob/8945ac7a3a7d1533388a81460afdc2b5f0b1dfc2/fastapi_cache/decorator.py#L82-L83

@john-tipper  Did you ever find a workaround for this? Running into the same thing.

Same thing, can someone answer?

hi any update? can we please open a PR and fix the relevant line?

Try changing your method to GET instead of POST. It worked for me.


feat: DIA-1410: Add support for list of str for multilabel

Hi @hakan458!
This PR was automaticaly generated via Follow Merge.
Please ensure that all linked upstream Pull Requests are merged before proceeding with this one.

https://github.com/HumanSignal/label-studio-sdk/pull/313 linked by Workflow Run

/git merge develop

/git merge develop


Failed to run automated scorecard-action@main test-organization-ls/scorecard-action-private-repo-tests

https://github.com/test-organization-ls/scorecard-action-private-repo-tests/actions/runs/2205623841

@rohankh532 looks like an error from the POST API request.

Also, it looks like we are uploading both SARIF and JSON results to Rekor? Can we only upload JSON for efficiency?

i thought we got rid of sarif entirely?

i thought we got rid of sarif entirely?

That was only in the endpoints, I just made a PR to remove the sarif from being uploaded in scorecard-action too


leave links to maps and data as Attachments in table of contents of gitbook

nice and easy to find yo

instead of
rmarkdown::render_site(output_format = 'bookdown::gitbook',
                       encoding = 'UTF-8')

we run
{
  # These files are included in the gitbook version already so we move them out of the build
  files_to_move <- list.files(pattern = ".Rmd$") %>%
    stringr::str_subset(., '2300|2400', negate = F) #move the attachments out
  files_destination <- paste0('hold/', files_to_move)

  ##move the files
  mapply(file.rename, from = files_to_move, to = files_destination)

  rmarkdown::render_site(output_format = 'bookdown::gitbook',
                         encoding = 'UTF-8')

  ##move the files from the hold file back to the main file
  mapply(file.rename, from = files_destination, to = files_to_move)
}

slightly weird because we end up with no Attachment 2 but willing to let it go for now

closed with fda969c


Allow for setting log directory by environment variable

Allow the frame log directory to be specified in either opencue.properties file or as an environment variable, with environment variable taking precedence.

Environment variable added: CUE_FRAME_LOG_DIR


Add arg to show hostname only on remote machines

Hi @statik213 - thanks for the PR! Sorry I took so long. After all this time, I don't think I'm going to merge this. This change, to me, is not something that is generally applicable. You might be the only person who needs it :)