[WIP][SPARK-32357][INFRA] Publish failed and succeeded test reports in GitHub Actions

What changes were proposed in this pull request?
This PR proposes to report the failed and succeeded tests in GitHub Actions by using ScaCap/action-surefire-report plugin.
See this example:
Scala: https://github.com/HyukjinKwon/spark/pull/16/checks?check_run_id=939449186
PySpark: https://github.com/HyukjinKwon/spark/runs/939798970

and

Why are the changes needed?
Currently, it's difficult to check the failed tests. You should scroll down long logs from GitHub Actions logs.
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
GitHub Actions will test. It was also partially manually tested at https://github.com/HyukjinKwon/spark/pull/16

I am running other tests to make sure it includes PySpark test reports as well in this PR. I will add a comment here once all testing is finished. Currently, I only did a basic test as described in the PR description.
cc @viirya, @dongjoon-hyun, @gengliangwang, @dbtsai FYI

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31593/
Test PASSed.

Test build #126969 has started for PR 29333 at commit 95bc43a.

Test build #126969 has finished for PR 29333 at commit 95bc43a.

This patch fails some tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126969/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31595/
Test PASSed.

Test build #126971 has started for PR 29333 at commit f866821.

Test build #126971 has finished for PR 29333 at commit f866821.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126971/
Test FAILed.

Test build #126981 has started for PR 29333 at commit 7197436.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31605/
Test PASSed.

Test build #126986 has started for PR 29333 at commit 36259f1.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31610/
Test PASSed.

@HyukjinKwon the report looks great. Thanks for the work!

Test build #126981 has finished for PR 29333 at commit 7197436.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126981/
Test FAILed.

Test build #126986 has finished for PR 29333 at commit 36259f1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126986/
Test FAILed.

It's great, @HyukjinKwon ! :)

Thanks @HyukjinKwon This will speed up our development velocity dramatically.

Just to share the current status,
In ScaCap/action-surefire-report plugin (and all other similar plugins), it leverages GITHUB_TOKEN that is set by default in GitHub Actions. It uses GitHub API to create status checks via here - it requires write permission to the repo. However, the permissions of GITHUB_TOKEN does not cover the case when a PR was raised based on the fork.
There are many similar issues and questions, for example, in codecov or GitHub community. In case of Codecov, they managed to remove the requirement of GITHUB_TOKEN at here. Basically they used existing GitHub Actions environment to verify in their service. This is not feasible in our case because the plugin is dependent of GitHub API to create the status checks directly.
I investigated this issue yesterday and concluded there's no clean workaround to make this working out of the box.
I am currently investigating the feasibility of potential alternatives. I am not sure if all of them work or not:

Use one environment variable, for example, TEST_REPORT_GITHUB_TOKEN as a GitHub secret. And then, guide committers to set TEST_REPORT_GITHUB_TOKEN as a GitHub secret in their forks. Note that the contributors would be able to report the test results as their tokens don't have the write access to the repo.

Just don't run this test report for now but wait until GitHub provides an alternative to work around this. There looks many requests such as this.

Just generate a token that only has the permission to change the status checks, and hardcode it in the repo.

BTW, let me know if you guys have a preference in one of the options. I can investigate fewer options in this case.

adding @srowen, @gatorsmile, @cloud-fan as well to collect more feedback.

If there's not a clean way to do it, and this isn't essential but nice-to-have, then option 1 seems the most viable. It would be opt-in and only for people who want to take the time to set this up, so not sure it would cover much. But probably more useful than doing it only after the fact, or exposing a token.

Sure, let me take a closer look for that approach. In worst case, we might have to drop this and go back to the original @viirya's approach at #29169.

I am making some changes to demonstrate the problem in terms of the fork and PRs. Please ignore the changes made from now on. I will switch back from the draft later when this is ready for a review.

Test build #127073 has started for PR 29333 at commit 2688f21.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31695/
Test PASSed.

The latest commit above (https://github.com/apache/spark/pull/29333/commits/2688f21f1852b3e6a577fa9292985b346b9bdf6d) contains the problem in terms of forked repos and PRs at this workload.
The current changes in this PR are exactly same as https://github.com/HyukjinKwon/spark/pull/16 at my own fork and PR in terms of how the ScaCap/action-surefire-report works.

Test build #127073 has finished for PR 29333 at commit 2688f21.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127073/
Test FAILed.

Okay, GitHub team suggested a scenario that it could work out of the box. It's a bit complicated then I thought but I will try. Yes, it is a nice-to-have but I think it is very worthwhile as it saves time of all Spark developers in their daily development and review at Apache Spark.

Test build #127407 has started for PR 29333 at commit 2888457.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32028/
Test PASSed.

I elaborated it in the PR description how it works and why I had to fork two plugins.

Test build #127410 has started for PR 29333 at commit 1e4fa43.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32031/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127410/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127407/
Test FAILed.

Test build #127411 has started for PR 29333 at commit 96b7dda.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32032/
Test PASSed.

https://github.com/HyukjinKwon/spark/runs/980576471 should show the example of this PR. Before this PR is merged, the test results would not be reported because the workflow B (see PR description) should run in main repo, not in the PR.

https://github.com/HyukjinKwon/spark/runs/980576471 should show the example of this PR. Once this example shows the results correctly, this PR should be good to go.
Before this PR is merged, the test results would not be reported because the workflow B (see PR description) should run in main repo, not in the PR.

Test build #127413 has started for PR 29333 at commit c36ec7a.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127411/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32034/
Test PASSed.

Test build #127413 has finished for PR 29333 at commit c36ec7a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127413/
Test PASSed.

Okay the example looks working fine. This PR should be ready for a review and merged. cc @srowen, @gengliangwang, @dongjoon-hyun, @dbtsai, @viirya can you take a look please?

Thank you all !!

I opened a PR to verify if this works well in the main commit (https://github.com/apache/spark/commit/5debde94019d46d4ab66f7927d9e5e8c4d16a7ec) and the PR (https://github.com/apache/spark/pull/29429).

Also, I would like to thank @cpintado from GitHub. He virtually guided me here a lot on this.

A late LGTM. Thanks for the great work!

Thanks, @HyukjinKwon @cpintado ! Great work!

Okay .. I checked that it works in both PR and commit. There look a bit of delay roughly 2 ~ 5 minutes to have the test report after the main tests are finished but I think this is good enough.


Errors when building on Debian Bookworm

When compiling Lagrange on bullseye, this procedure works fine with the latest commit on the release branch. However, it fails with the error below when using the newer version of Debian on ARM64. I've checked that all of the dependencies listed in the README are installed.
cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr/local
-- Compiling bincat for merging resource files...
-- Configuring done
-- Generating done
-- Build files have been written to: /home/cmcgee/git/lagrange/build/res
Consolidate compiler generated dependencies of target bincat
[100%] Built target bincat
-- CPU does not support SSE 4.1
-- Checking for module 'eopenssl11'
--   No package 'eopenssl11' found
-- Checking for module 'openssl'
--   Found openssl, version 1.1.1l
-- Checking for module 'libwebp'
--   No package 'libwebp' found
-- Preparing resources...
--   res/default.fontpack
usage: zip [-vD] [-1-9] [-f zipfile] file ...
CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

CMake Error at res/Embed.cmake:133 (list):
  list GET given empty list
Call Stack (most recent call first):
  CMakeLists.txt:102 (embed_make)

-- Configuring incomplete, errors occurred!
See also "/home/cmcgee/git/lagrange/build/CMakeFiles/CMakeOutput.log".
See also "/home/cmcgee/git/lagrange/build/CMakeFiles/CMakeError.log".

usage: zip [-vD] [-1-9] [-f zipfile] file ...

This seems to indicate that the -0 option is not supported. That is used for creating default.fontpack without compression.
If you replace the -0 on Fontpack.cmake line 15 with -1, does it work then?
Does the version of zip on the system have any option that creates uncompressed ZIPs? If not, can you suggest any available alternatives for creating uncompressed ZIPs that would work for you?

It turns out that this was the plan9 version of zip because I had not install the zip package on this system. Closing this issue. Thank you.


Can't show small value when axis has large number

Check  4th column has value but can't show

When I face that issue I change the value programmatically so it becomes readable:
let series = dataArrays.map((dataArray) => {
return {
data: dataArray.map((val) => val === 0 // If value is 0, it is set at 0.1 so something appears on the chart
? 0.1
: val
)
}
}

Value is 1 not zero

You just need to change the condition:
data: dataArray.map((val) => val < 2
? 2
: val
In that case for example the value would always be at least 2


Adjust resources for MongoDB Stateful Set

Currently, MongoDB container crashed because of resource constaints i.e.
out of memory exception. This change updates the resources and provides
data on how the configure/calculate them(if not following the guide).
Also, add the ability to specify the storage engine(WiredTiger) cache
size for MongoDB, this configuration also helps with keeping the resources
constrained for MongoDB containers.
Minor changes in some other documents as well.

Let me update the patch. Seems like a mistake.

Codecov Report

Merging #1757 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1757   +/-   ##
=======================================
  Coverage   98.58%   98.58%
=======================================
  Files          59       59
  Lines        2896     2896
=======================================
  Hits         2855     2855
  Misses         41       41


Validate tsp config for modular in SDK Automation pipeline

Background: https://github.com/Azure/azure-sdk-for-js/pull/30102#discussion_r1661728106
Depend on: https://github.com/Azure/azure-sdk-tools/issues/8307

@wanlwanl can you confirm if this work has been done?

no, it's P1, has a blocker https://github.com/Azure/autorest.typescript/issues/2636


[CMT-859-1215] translate(article): Diving into Yearn Metrics - DE

Description
This pull request translates diving-into-yearn-metrics article into GERMAN
Fixed issue
Fixes #(issue)
Type of change

[x] Add a new translation

Resources

Any screenshots, links or relevant resources to explain better the changes.

@qosmonot I have seen a tag "needs review by translator" on my PR but without comment. Can you please let me know what I need to fix exactly? Thank you

@Delace19  it was a small thing, you can see in the conversation above, and on the file changed tab.

maybe keep this in english, or both (just a thought.
let me know

@qosmonot changes have been made. thanks

checked and looks good.


Implementing SNMP Trap-PDU

First of all, thanks for the effort in implementing SNMP in JavaScript. I have done the same in PHP to get rid of the PHP snmp extension.
Could it be possible to implement all PDU types, especially Trap-PDU types, so that the server could handle alerts sent from devices (socket listening on udp/162).

0xa0 (160)	SNMP GetRequest-PDU
0xa1 (161)	SNMP GetNextRequest-PDU
0xa2 (162)	SNMP GetResponse-PDU
0xa3 (163)	SNMP SetRequest-PDU
0xa4 (164)	SNMPv1 Trap-PDU (replaced by 0xa7 in SNMPv2)
0xa5 (165)	SNMPv2 GetBulkRequest-PDU
0xa6 (166)	SNMPv2 InformRequest-PDU
0xa7 (167)	SNMPv2 SNMPv2-Trap-PDU

Certainly it's possible. :) I don't currently have the time or inclination, but patches are always welcome.


add Spiral Abyss Class

New feature name

Spiral Abyss
Monster

New feature description
By adding Spiral Abyss information, it can be checked regularly, and it would be useful to link it with Monster as an originality of this library.
It can also be used effectively on the owner's Discord server.
Versions to be added
0.2.0～
Issue priority
High

21104 is an Id that cannot get monsterId in this way, so convert it to 22110403.
To add a Test Action because there are other concerns that may arise in the future besides this. #11
/**
   * Get all monster ids
   * @returns All monster ids
   */
  public static findMonsterIdByDescribeId(describeId: number): number {
    const convertId = describeId.toString().padStart(5, '0')
    return describeId == 21104 ? 22110403 : Number(`2${convertId}01`)
  }


Update aws-core, http-client-spi, ... to 2.17.115

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.104 to 2.17.115.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #32.


Unique errors for CF

['(2.3): Invalid variable name']
['(2.3): Invalid variable name', '(3.1): Invalid units: -']
['(2.6.3): Variable areacella named as an external variable must not be present in this file']
['(2.6.3): Variable volcello named as an external variable must not be present in this file']
['(3.1): Invalid units: -']
['(3.1): Units are not consistent with those given in the standard_name table.']
['(3.3): Invalid region name: a', '(3.3): Invalid region name: i', '(3.3): Invalid region name: g']
['(3.3): Invalid region name: atlantic_arctic_extended_ocean']
['(3.3): Invalid region name: carribean_windward_passage', '(3.3): Invalid region name: taiwan_and_luzon_straits', '(3.3): Invalid region name: agulhas_section']
['(3.3): Invalid region name: g', '(3.3): Invalid region name: a']
['(3.3): Invalid standard_name: air_pressure_at_top_of_atmosphere']
['(3.3): Invalid standard_name: alevel']
['(3.3): Invalid standard_name: atmosphere_optical_thickness_due_to_sulfate_ambient_aerosol']
['(3.3): Invalid standard_name: bounds']
['(3.3): Invalid standard_name: conductive_heat_flux_at_sea_ice_surface']
['(3.3): Invalid standard_name: depth_of_soil_moisture_saturation']
['(3.3): Invalid standard_name: heat_content_of_ocean_layer']
['(3.3): Invalid standard_name: Latitude', '(3.3): Invalid standard_name modifier: axis']
['(3.3): Invalid standard_name: ocean_sigma_z', '(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): No formula defined for standard name: ocean_sigma_z']
['(3.3): Invalid standard_name: ocean_sigma_z', '(4.3.3): No formula defined for standard name: ocean_sigma_z']
['(3.3): Invalid standard_name: olevel']
['(3.3): Invalid standard_name: sea_ice_basal_net_downward_sensible_heat_flux']
['(3.3): Invalid standard_name: sea_ice_salt_content']
['(3.3): Invalid standard_name: sea_ice_time_fraction']
['(3.3): Invalid standard_name: seaice_transect']
['(3.3): Invalid standard_name: tendency_of_sea_ice_amount_due_to_dynamics']
['(3.3): Invalid standard_name: tendency_of_sea_ice_amount_due_to_thermodynamics']
['(3.3): Invalid standard_name: tendency_of_snow_mass_due_to_sea_ice_dynamics']
['(3.3): Invalid standard_name: tendency_of_surface_snow_and_ice_amount_due_to_sublimation']
['(3.3): Invalid standard_name: total_water_storage']
['(3.3): Invalid standard_name: transect']
['(3.3): Invalid standard_name: Vertical', '(3.3): Invalid standard_name modifier: levels']
['(3.3): Invalid standard_name: vertical_integral_eastward_wind_by_dry_static_energy']
['(3.3): Invalid standard_name: vertical_integral_eastward_wind_by_total_water']
['(3.3): Invalid standard_name: vertical_integral_northward_wind_by_dry_static_energy']
['(3.3): Invalid standard_name: vertical_integral_northward_wind_by_total_water']
['(3.3): Invalid standard_name: water_flux_into_sea_water_from_sea_ice']
['(3.3): Invalid standard_name: water_flux_to_downstream']
["(3.3): Invalid syntax for 'standard_name' attribute: 'number of layer interfaces'"]
["(3.3): Invalid syntax for 'standard_name' attribute: 'number of layers'"]
['(3.3): Variable basin of invalid type. Region variable should be of type char.']
['(4.3.3): Formula term nsigma not present in formula for ocean_sigma_z_coordinate']
['(4.3.3): formula_terms attribute only allowed on coordinate variables']
['(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): ap_bnds is not declared as a variable', '(4.3.3): b_bnds is not declared as a variable', '(4.3.3): ps is not declared as a variable']
['(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): Formula term nsigma not present in formula for ocean_sigma_z_coordinate']
['(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): ps is not declared as a variable']
['(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): Standard name of variable ptop inconsistent with that of lev_bnds', '(4.3.3): Standard name of variable ps inconsistent with that of lev_bnds']
['(4.3.3): ps is not declared as a variable']
['(4.3.3): Standard name of variable ptop inconsistent with that of lev', '(4.3.3): Standard name of variable ps inconsistent with that of lev']
['(4.4.1): Attribute calendar may only be attached to time coordinate variable', '(3.1): Invalid units: days since', '(4.4): Invalid units and/or reference time']
['(5): coordinates attribute referencing non-existent variable']
['(5): co-ordinate variable not monotonic']
['(5): Dimensions must be a subset of dimensions of siarean']
['(5): Dimensions must be a subset of dimensions of siareas']
['(5): Dimensions must be a subset of dimensions of sivoln']
['(5): Dimensions must be a subset of dimensions of sivols']
['(5): Dimensions must be a subset of dimensions of thetaoga']
['(5): Dimensions must be a subset of dimensions of volo']
['(7.1): Boundary var lev_bnds has inconsistent standard_name to lev', '(3.1): Units are not consistent with those given in the standard_name table.', '(4.3.3): formula_terms attribute only allowed on coordinate variables']
['(7.1): Boundary var lev_bnds has inconsistent standard_name to lev', '(3.1): Units are not consistent with those given in the standard_name table.', '(4.3.3): formula_terms attribute only allowed on coordinate variables', '(4.3.3): p0 is not declared as a variable', '(4.3.3): a_bnds is not declared as a variable', '(4.3.3): b_bnds is not declared as a variable', '(4.3.3): ps is not declared as a variable']
['(7.1): Boundary var lev_bnds has inconsistent units to lev']
['(7.1): Boundary var time_bnds has inconsistent calendar to time']
['(7.1): Boundary var time_bnds has inconsistent units to time']
['(7.1): bounds attribute referencing non-existent variable alevel_bnds']
['(7.1): bounds attribute referencing non-existent variable lat_bnds']
['(7.1): bounds attribute referencing non-existent variable lon_bnds']
['(7.1): bounds attribute referencing non-existent variable sdepth_bnds']
['(7.1): Incorrect number of dimensions for boundary variable: time_bounds']
['(7.2): cell_measures variable areacella must either exist in this netCDF file or be named by the external_variables attribute']
['(7.2): cell_measures variable areacellg must either exist in this netCDF file or be named by the external_variables attribute']
['(7.2): cell_measures variable areacello must either exist in this netCDF file or be named by the external_variables attribute']
['(7.2): cell_measures variable areacello must either exist in this netCDF file or be named by the external_variables attribute', '(7.2): cell_measures variable volcello must either exist in this netCDF file or be named by the external_variables attribute']
['(7.2): Invalid cell_measures syntax']
['(7.3): Invalid cell_method: mask', "(7.3): Invalid 'name' in cell_methods attribute: (comment"]
["(7.3): Invalid 'name' in cell_methods attribute: month", "(7.3): Invalid 'name' in cell_methods attribute: year"]
["(7.3): Invalid 'name' in cell_methods attribute: scalar_axis"]
["(7.3): Invalid 'name' in cell_methods attribute: y", "(7.3): Invalid 'name' in cell_methods attribute: x"]
['(7.3): Invalid syntax for cell_methods attribute']
['(7.3): Invalid type1: landuse - must be a variable name or valid area_type']
['(7.3): Invalid type1: sector - must be a variable name or valid area_type']
['(7.3): Invalid unit hours, in cell_methods comment']
["Attribute missing_value of incorrect type (expecting 'Data Variable' type, got 'Numeric' type)"]
["Attribute missing_value of incorrect type (expecting 'Data Variable' type, got 'Numeric' type)", '(3.1): Units are not consistent with those given in the standard_name table.']
["Attribute missing_value of incorrect type (expecting 'Data Variable' type, got 'Numeric' type)", '(7.2): Invalid cell_measures syntax']
["Attribute missing_value of incorrect type (expecting 'Data Variable' type, got 'Numeric' type)", '(7.3): Invalid type1: sector - must be a variable name or valid area_type']
['Invalid attribute name: _CoordinateAxisType']

This is the list of unique errors. I haven't got counts, but when I load up the raw files into python, I do some aggregations and some counts, this was just a quick grep but it gives an idea of what we have.

Thanks .. that is quite a list. Makes me wonder why there aren't any errors found for section 6 of CF. To make progress it would be useful to have the count of files and also perhaps a list of variable and models for associated with each error (both truncated if length > 20) and a few sample files.
(7.2): cell_measures ......  is one that I think we can flag as "dont care": this is a new CF rule brought in for CMIP6 to tidy up some issues which troubles the purists .. it shouldn't break anyone's software.

PS: probably best to provide the above as json or csv.

missing_value errors discussed here: #3
external_variables: see #4

50 : (2.3): Invalid variable name
147 : (2.6.3): Variable areacella named as an external variable must not be present in this file
1203 : (2.6.3): Variable volcello named as an external variable must not be present in this file
27 : (3.1): Invalid units: -
11049 : (3.1): Units are not consistent with those given in the standard_name table.
573 : (3.3): Invalid region name: a
489 : (3.3): Invalid region name: atlantic_arctic_extended_ocean
14 : (3.3): Invalid region name: carribean_windward_passage
70 : (3.3): Invalid region name: g
172 : (3.3): Invalid standard_name: air_pressure_at_top_of_atmosphere
2739 : (3.3): Invalid standard_name: alevel
52 : (3.3): Invalid standard_name: atmosphere_optical_thickness_due_to_sulfate_ambient_aerosol
2065 : (3.3): Invalid standard_name: bounds
101 : (3.3): Invalid standard_name: conductive_heat_flux_at_sea_ice_surface
10 : (3.3): Invalid standard_name: depth_of_soil_moisture_saturation
10 : (3.3): Invalid standard_name: heat_content_of_ocean_layer
114 : (3.3): Invalid standard_name: Latitude
3462 : (3.3): Invalid standard_name: ocean_sigma_z
1590 : (3.3): Invalid standard_name: olevel
101 : (3.3): Invalid standard_name: sea_ice_basal_net_downward_sensible_heat_flux
101 : (3.3): Invalid standard_name: sea_ice_salt_content
137 : (3.3): Invalid standard_name: sea_ice_time_fraction
101 : (3.3): Invalid standard_name: seaice_transect
101 : (3.3): Invalid standard_name: tendency_of_sea_ice_amount_due_to_dynamics
101 : (3.3): Invalid standard_name: tendency_of_sea_ice_amount_due_to_thermodynamics
101 : (3.3): Invalid standard_name: tendency_of_snow_mass_due_to_sea_ice_dynamics
244 : (3.3): Invalid standard_name: tendency_of_surface_snow_and_ice_amount_due_to_sublimation
33 : (3.3): Invalid standard_name: total_water_storage
101 : (3.3): Invalid standard_name: transect
2065 : (3.3): Invalid standard_name: Vertical
10 : (3.3): Invalid standard_name: vertical_integral_eastward_wind_by_dry_static_energy
10 : (3.3): Invalid standard_name: vertical_integral_eastward_wind_by_total_water
10 : (3.3): Invalid standard_name: vertical_integral_northward_wind_by_dry_static_energy
10 : (3.3): Invalid standard_name: vertical_integral_northward_wind_by_total_water
101 : (3.3): Invalid standard_name: water_flux_into_sea_water_from_sea_ice
3 : (3.3): Invalid standard_name: water_flux_to_downstream
51 : (3.3): Variable basin of invalid type. Region variable should be of type char.
764 : (4.3.3): Formula term nsigma not present in formula for ocean_sigma_z_coordinate
73020 : (4.3.3): formula_terms attribute only allowed on coordinate variables
238 : (4.3.3): ps is not declared as a variable
344 : (4.3.3): Standard name of variable ptop inconsistent with that of lev
10 : (4.4.1): Attribute calendar may only be attached to time coordinate variable
76 : (5): coordinates attribute referencing non-existent variable
15 : (5): co-ordinate variable not monotonic
126 : (5): Dimensions must be a subset of dimensions of siarean
126 : (5): Dimensions must be a subset of dimensions of siareas
126 : (5): Dimensions must be a subset of dimensions of sivoln
126 : (5): Dimensions must be a subset of dimensions of sivols
506 : (5): Dimensions must be a subset of dimensions of thetaoga
204 : (5): Dimensions must be a subset of dimensions of volo
2739 : (7.1): Boundary var lev_bnds has inconsistent standard_name to lev
914 : (7.1): Boundary var lev_bnds has inconsistent units to lev
8267 : (7.1): Boundary var time_bnds has inconsistent calendar to time
172 : (7.1): Boundary var time_bnds has inconsistent units to time
18 : (7.1): bounds attribute referencing non-existent variable alevel_bnds
4 : (7.1): bounds attribute referencing non-existent variable lat_bnds
4 : (7.1): bounds attribute referencing non-existent variable lon_bnds
20 : (7.1): bounds attribute referencing non-existent variable sdepth_bnds
12 : (7.1): Incorrect number of dimensions for boundary variable: time_bounds
10368 : (7.2): cell_measures variable areacella must either exist in this netCDF file or be named by the external_variables attribute
22 : (7.2): cell_measures variable areacellg must either exist in this netCDF file or be named by the external_variables attribute
2182 : (7.2): cell_measures variable areacello must either exist in this netCDF file or be named by the external_variables attribute
2997 : (7.2): Invalid cell_measures syntax
134 : (7.3): Invalid cell_method: mask
566 : (7.3): Invalid syntax for cell_methods attribute
1 : (7.3): Invalid type1: landuse - must be a variable name or valid area_type
644 : (7.3): Invalid type1: sector - must be a variable name or valid area_type
219 : (7.3): Invalid unit hours, in cell_methods comment
51893 : Data Variable
13492 : Invalid attribute name: _CoordinateAxisType

Just some initial numbers I'll get more details when I do proper analysis of the results.


docs:add NginxCommunityVersion

Previously offered documents only provides tutorials for using Casdoor and Nginx Plus together. Based on my own usage experience, I have written a guide for using Casdoor with the Nginx community open source version. In this article, the human readable language I use is Chinese (Simplified). If there is anything wrong or inappropriate, please do not hesitate to advise.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@GZYZhy hi, contribution like this PR is welcomed! One thing needs to be noted is that all docs in this repo should be in English, including all texts and the UI in screenshots. The translations into other languages like zh will be handled by another process named Crowdin (which is not included in this PR). So can you please change the PR into English only? ChatGPT will be a good help for this work.

Okay, I have changed the language used in the article to English (using ChatGPT as an auxiliary tool), and the prompts in the pictures used in the article have been used in English, too. Thank you for your inspection. Wish you a happy New Year.

@GZYZhy plz fix CI errors:

I have reviewed the error message, and it indicates that the error occurred in the following file: docs/provider/oauth/overview.md.

Strangely, this file is completely unrelated to my current pull request. According to my memory and the Git commit history, I haven't made any modifications to that file.
I have thoroughly reviewed all 7 files related to my pull request and checked them one by one using yarn and VSCode to ensure there are no errors.
To verify if my changes caused the error in that file, I cloned your official repository (i.e., casdoor/casdoor-website.git) again on my computer, which contains all the content before I submitted the pull request. Then, I opened the file using VSCode with the lint plugin installed, and I found that it also showed Markdown syntax errors.

Based on this, I believe that the error did not originate from my end. I'm not very familiar with the content described in the docs/provider/oauth/overview.md file, and I'm unsure about what modifications should be made to make it syntactically correct without altering the original content. Therefore, I may not be able to fix this error.
Thank you for your review, and wish you a happy new year. If there's anything else you need, please let me know.


Why did I configure {cache: false} in vite-eslint-plugin and still fail to hot update successfully?

Why did I configure {cache: false} in vite-eslint-plugin and still fail to hot update successfully?
import eslintPlugin from 'vite-plugin-eslint' eslintPlugin({    include: ['src/**/*.js', 'src/**/*.vue', 'src/*.js', 'src/*.vue'],   cache: false }),

I'm glad to solve this problem quickly！ In the vue3 project created based on vite, it is not necessary to install the vite-plugin-elint plug-in. If it is installed, it may cause an error after the hot update.....


Please support .net standard

Please guys...

A beta version is released with .net standard support

Full version just released with .net standard support.

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Evaluate whether the Web.config is required in the \proxy folder

The Web.config is present in the proxy folder, but we need to confirm whether it is still needed, particularly now that we have updated to the 1.1 version of the proxy.

I had many issues with the Proxy file and Tokens when I was setting mine up. We ended up using another Web.config from Git and it worked but getting to that point took a lot of trial and error.. mostly error. What made it worse was it would work sometimes then it would stop working for no reason. since updating the Web.config file we have had no issues with obtaining a token or using the GeoEnrichment service as we did prior.


unresolved import std::os::fd

I take it that you mean that those imports are extraneous. I've removed them.


Disable marian test until openvino next release

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Tests failing in new astropy

As the title says. Might be related to new treatment of units

Fixed in 648b43e


Host settings in GUI

I've started converting application settings to GUI. So far I'm doing it in a unified fashion, by creating a base schema for common attributes and then schema per host. It's quite a quick process and should allow adding extra attributes where needed.
The main question is, what exactly do we need to specify apart from an executable for each platform? Most of the launchers are identical and I think we'll still need to have some base launcher for each host due to minor difference between them. We should be then dynamically just replacing the version and executable location taken from the settings.

There are some hosts, that act more as variations, (nukex vs nuke) sharing identical setup with the main app, so I think we could treat those just with extra choices in the main host. For instance nuke has nukeX checkbox. When on, we should also generate  a launcher for that. That however would increase code complexity substantially probably, so up for discussion.

Unified the settings with templates and added all default values


add note to migration guide about changes to AnyUrl etc

Change Summary
As per the title - add a short comment informing users searching the migration guide about the changes to Url.
Related issue number
Ref #6593, which is an example of user experiencing friction due to the change.
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


adding null primitive type

After the discussion on issue #26 I felt it would be a good idea to add the primitive type null.

Hey @obihann ! Appreciate your effort but I would like to wait with this PR until the discussion in https://github.com/apiaryio/mson/issues/26#issuecomment-133029966 is closed. Please lets discuss there, what is the best possible solution and why.
Thanks!

@zdne Thanks for the reply. I understand wanting to wait till the discussion is over however what I take from that discussion is two things 1) their is no implementation of null as a primitive type, which is critical because null is a valid JSON type, and 2) how will we allow fields to be optionally null.
The far larger issue is 2, and I purposely avoided addressing that as to allow the conversation to solve that for me.
Either way, let me know if I can help.

@obihann
Thanks! As discussed in #26 I am hesitant to introduce null as a type. With that, however, I totally see the need to address the following:

allow a value of any type to be null (not set)
allow a value of any type to be a default empty value ("", [], {}, 0)

Let's discuss it in #26 ...

@zdne Just to keep this thread up to speed with my other comments in #26 I do agree with you on 1 and 2. Well carry this on in their now.


Support TestRule along with MethodRule

MethodRule has been replaced by TestRule in JUnit 4.9. See JavaDoc for MethodRule.
ArquillianRules now supports usage of both, even mixed.

thanks for your effort, we will check it

thanks! I have just released it with your changes included. 0.3.0 of Arquillian Rule should hit Maven Central in a day or two.

You're welcome and thanks for the release


meow, 1.20 support

blah blah blah, i hate modern optifine, will come back eventually to bugfix optifine not working, but CIT Resewn works perfectly fine for 1.20.1 users

i hate optifine, added old readme, prob should add porky what should and shouldn't be there

also looks good


Optical Ports of PCell

I find that the structure normally has default port name. Like a straight waveguide, the default name of two optical ports is 'o1' and 'o2'. However, sometimes, I want to use the port convention by my own. Like 'in' and 'out'.
Is that possible for user to rename the port name by their own. I found that, if I create a PCell without default ports, then I can add ports with name defined by myself. But if the PCell already has ports, then I cannot rename them to what I want. Like from 'o1' to 'in'.
Another stange thing is that if a structure already has default port at a position. Although I cannot change the name of ports, but I can add ports in the same position. Should this be foribidden?
Hope to get information from all of you. GDSFactory is really a cool tool. I like it very much!!!

Ok, thank you for your explantion, I will have a try by following your instructions. Hope this can be enhanced soon and to make GDSFactory more concrete.


Results of NeuralNetwork is unexpected

What is wrong?
Thank you for this wonderful project. I am attempting to create a program that can tell if a person is a child or not with provided height and weight. For some reason, it is telling me the opposite results of what I would expect.
Where does it happen?
In the a NeuralNetwork when trying to the program in node.js on my mac.
How do we replicate the issue?
const brain = require("brain.js");
const net = new brain.NeuralNetwork();

net.train([
  { input: { height: 6, weight: 180 }, output: { notAChild: 1 } },
  { input: { height: 6, weight: 190 }, output: { notAChild: 1 } },
  { input: { height: 6, weight: 193 }, output: { notAChild: 1 } },
  { input: { height: 5, weight: 150 }, output: { notAChild: 1 } },
  { input: { height: 5, weight: 160 }, output: { notAChild: 1 } },
  { input: { height: 5, weight: 164 }, output: { notAChild: 1 } },
  { input: { height: 3, weight: 60 }, output: { child: 1 } },
  { input: { height: 3, weight: 68 }, output: { child: 1 } },
  { input: { height: 3, weight: 69 }, output: { child: 1 } },
])

const output = net.run({ height: 3, weight: 65 });
// { notAChild: 0.6541248559951782, child: 0.34587541222572327 }

console.log(output);

How important is this (1-5)?
1 - I maybe doing something wrong here.
Expected behavior (i.e. solution)
I would have expected that when given { height: 3, weight: 65 } it would have given a high likelihood that it IS a child. But it's actually saying its more likely to be a not a child. What am I doing wrong here?
Thanks again!

You have to normalize your input data. Watch this tutorial -> https://scrimba.com/p/pVZJQfg/cpyevhz

Hmm that guy is dividing by the lowest value in the data.

Not really sure how it applies to my use case, thanks any way. I'll close this now.


Scope UpdateSourceOnBuild target

Scope UpdateSourceOnBuild target to prevent cyclic run that causes files to be locked by other process
This fixes https://github.com/Azure/azure-sdk-for-net/issues/10997

This might not be reqired once this https://github.com/Azure/azure-sdk-for-net/pull/22783 is merged.


CollectionWrapper: initially not requesting more items to fulfill request threshold

https://github.com/rdkcentral/Lightning-ui/blob/67472fff84fa630b367d4a57c829018af7e25717/src/helpers/CollectionWrapper.js#L67-L80
Probably the previous < target the condition needs to be dropped, because initially it should be triggering request when there are not enough items to reach the request threshold.
In my use-case, I have a grid that has 6 columns and the requests are made with 10 items, which means that if I do the first request and add the first 10 items, it doesn't trigger more requests until I navigate to the second line of the grid.
This change would also let CollectionWrapper manage the requests exclusively. As of now, we need to add the first items to the CollectionWrapper. If this condition doesn't exist, we only need to provide a onRequestItems callback and CollectionWrapper does all it needs to do.

Fix now part of dev branch, will release a new version when #48 has been resolved


Inconsistent datatype from train_loader.dataset.train_data

I've noticed that the type of data contained in the data loaders are different from one dataset to another. For example
data_loader.dataset.train_data with CIFAR10 is a np.array, while with MNIST it is a torch.Tensor
Is this for some reason?
I think it should be uniformed because I often write code which requires knowing something from the training data and if I want to write it for multiple dataset this is a bit inconvenient.

hi @iacolippo - I don't theres a specific reason as to why the data is stored this way inside the datasets. AFAIK both formats are fine for the default collate function of the DataLoader.
If you want to access data in a consistent format, one option is to pass the transform=ToTensor() to the dataset and access the elements via the __getitem__ function (via []). This will ensure you get a tensor version of the images back, as both datasets convert the internal representation to a PIL Image before passing it to the transforms.

Ok, that's reasonable, even though this asymmetry annoys my OCD 😄 Thank you. Closing.


how to extend marked

I would like to do like github has with @person. How do I extend marked to look for the @person? Thanks for all the help and awesome module.

+10000

You could override all renderer methods. Then inside each, parse the mentions, then call the original renderer methods and return its result.

@activey use @markdown-it or jonschlinkert/remarkable or bevacqua/megamark (is awesome!), ForbesLindesay/supermarked and etc
marked is almost dead. Markdown-it is upstream from half year or so. It is highly customizeable, and have plugins in npm https://www.npmjs.com/browse/keyword/markdown-it

Oh, thanks for the info :D

Closing as stale, resolved, and out of scope at present. See #956


remove hour limit rate from docs

review and feel free to adjust
Fixes #1478

@markusthoemmes thanks for the review, I submitted a second commit 4ec150fb880c944463eb3615287a9ec8de6d5674 with the changes.

replaced by https://github.com/openwhisk/openwhisk/pull/1532


getPixelFromCoordinate breaks after map scroll wraps

If you scroll a map in one direction all the way round the globe, getPixelFromCoordinate starts to give invalid coordinates.
As an example (screenshot from Chrome console below):

Load a map, centred roughly on [0, 0]
map.getPixelFromCoordinate([0, 0]) returns a reasonable answer
Scroll east, going all the way around the world to cross the Atlantic and return to [0, 0]
map.getPixelFromCoordinate([0, 0]) now gives a negative x pixel value, even though the same point is visible on the screen.

I've tried this on a few maps from the examples page, and I get the same result, so I don't think I've done anything silly.
I haven't dug deeply into this, but it seems to be an issue with the frameState.coordinateToPixelTransform not taking the wraparound into account.
I'll look into either a fix or workaround and add my findings...

The behaviour you are seeing may be unexpected, but it is intentional. Because when you went around the world once, the pixel for [0, 0] is way beyond the visible viewport. However, map.getPixelFromCoordinate([180, 0]) should return the same pixel that you had with [0, 0] before going around the world.

map.getPixelFromCoordinate([180, 0]) doesn't work - it always gives the same result as map.getPixelFromCoordinate([0, 0]) or any multiplications of 180.

Sorry, I meant multiplications of 360. Here is what you can do to see how it works, e.g. in https://openlayers.org/en/latest/examples/simple.html:
map.getPixelFromCoordinate(ol.proj.fromLonLat([0, 0])); // [543, 200]
map.getView().setCenter(ol.proj.fromLonLat([360, 0]));
map.getPixelFromCoordinate(ol.proj.fromLonLat([0, 0])); // [-481, 200] - outside viewport
map.getPixelFromCoordinate(ol.proj.fromLonLat([360, 0])); // [543, 200]

Ah, that makes more sense. (Still getting the hang of projecting between co-ordinates and pixels!) I can make a bit of logic that works out how many times I've been round the world to offset the pixels.


Crash "pointer being freed was not allocated" v2.6.0

When we use the mPulse 2.6.0 we get a crash very often with the following output:
OurApp(52271,0x70000cb11000) malloc: *** error for object 0x600001746c60: pointer being freed was not allocated
OurApp(52271,0x70000cb11000) malloc: *** set a breakpoint in malloc_error_break to debug
The issue occurs at start up, when we initialise Soasta, set the view group, start a timer and set 3-4 times a custom dimension. All these commands run on the main thread very quickly. It seems however that the loading of a config file causes this crash
More information (including output from a run with the address sanitizer on) can be seen at: https://gist.github.com/KoxarasA/867cf0c833f5eb64619011ea6d7161ca
The issue does not happen when we downgrade to 2.3.6

Hi @KoxarasA!
We've fixed this issue in mPulse iOS SDK 2.6.1.  Can you verify if that release works?

Hi @nicjansma
The issue is resolved! Thank you!


Place edge labels to avoid overlap

This places edge labels containing mutations at the bottom of the edge, and changes the format for recombination positions to put the numbers on 2 lines, with "≥" and "<" in front of the numbers to distinguish from mutations.

I've staked a load of extra changes to the sample_subgraph() function to this PR. I hope that's OK.They are require to produce the graph we are using for XAG in the paper.

@a-ignatieva , want to take a look?

Looks great @hyanwong! Just one thought, might it be better to have the intervals on the same line particularly if there are multiple breakpoints on the edge?

Yeah, it would be better, but there's not enough room between the edges, even when we push the labels up towards the parents.
Any suggestions for alternative approaches?

I think there are no non-hacky ways to adjust the positions, but maybe hacky is fine? Eg to move them slightly to the left and right
    pos = nx.nx_agraph.graphviz_layout(G, prog="dot")

    x_max = 0
    for k, v in pos.items():
        x_max = max(x_max, v[0])
    x_adj = x_max / 15
    flip = -1
    for k, v in pos.items():
        if k in recombinant_parents:
            pos[k] = (v[0] + x_adj * flip, v[1])
            flip *= -1

where recombinant_parents is a set of those nodes. I can see this breaking when the graph is complicated but should work fine in a lot of cases?

And yes I like the idea of recombination nodes being black! Good to distinguish them from lineage based colours.

Eg to move them slightly to the left and right

Can we do that outside of sc2ts? It would be best to be able to do this after calling sc2ts.sample_subgraph, I think, so that we don't hard-code the behaviour into the layout function. Perhaps we should pass in a dict of "tweak_x" positions to sc2ts.sample_subgraph, which is what I did in the ARG paper layout stuff?

Alternatively (and a bit of a hack), we could allow the positions to be (optionally) passed in to sc2ts.sample_subgraph, and simply call it twice, discarding the plot on the first time around.

we could allow the positions to be (optionally) passed in to sc2ts.sample_subgraph,

Implemented in https://github.com/jeromekelleher/sc2ts/pull/143/commits/1f53ec1d9112de6143e0a8a6dd2b720ffff074a3 along with intervals-on-a-line (separated by "..."). We can now make plots like:

Let's merge this now, I think it's ready, otherwise more viz stuff will keep on piling up.


libvirt: session-mode libvirt can't always find /run/libvirt/nix-helpers/qemu-bridge-helper

Describe the bug
I sometimes create a qemu VM in libvirt in "session mode", meaning that the libvirt daemon is running as my non-root user, and the system-wide libvirtd does not need to be running (docs). This VM has a network bridge, which libvirt apparently creates by calling /run/wrappers/bin/qemu-bridge-helper, which in turn uses /run/libvirt/nix-helpers/qemu-bridge-helper.
The problem is that /run/libvirt doesn't get fully populated until the system libvirtd gets started. Previously, in nixpkgs 7f9b6e2ba (from February), this wasn't noticeable because libvirtd.service was WantedBy libvirt-guests.service (which is WantedBy multi-user.target), so libvirtd would always be started on boot to populate that directory. Now, with 1d08ea2b, this is no longer the case, and libvirtd.service does not get started (presumably, if libvirt-guests.service actually needed to start a VM, then libvirtd.service would be socket-activated, but for me this is not the case). As a result, /run/libvirt doesn't get populated and my session-mode libvirt can't find /run/libvirt/nix-helpers/qemu-bridge-helper.
Steps To Reproduce

In python:conn = libvirt.open('qemu:///session')
dom = conn.createXML(Path('path/to/vm.xml').read_text())

The VM's XML file defines a network bridge, which is why qemu-bridge-helper gets involved.
Get the following output:libvirt: QEMU Driver error : internal error: /run/wrappers/bin/qemu-bridge-helper --use-vnet --br=virbr0 --fd=29: failed to communicate with bridge helper: Transport endpoint is not connected
stderr=/run/wrappers/bin/qemu-bridge-helper: cannot run `/run/libvirt/nix-helpers/qemu-bridge-helper': No such file or directory
...
libvirt.libvirtError: internal error: /run/wrappers/bin/qemu-bridge-helper --use-vnet --br=virbr0 --fd=29: failed to communicate with bridge helper: Transport endpoint is not connected
stderr=/run/wrappers/bin/qemu-bridge-helper: cannot run `/run/libvirt/nix-helpers/qemu-bridge-helper': No such file or directory

Expected behavior
The libvirt user session should be usable without first starting the system-wide libvirtd.
Additional context
If you start (and optionally stop) the system libvirtd.service before following the steps to reproduce, everything works fine, because the /run/libvirt directory gets populated.
Probably, /run/libvirt/nix-helpers/qemu-bridge-helper should be put somewhere else, where session-mode daemons can use it without needing to have previously started system-mode libvirtd. (The same may be true of other stuff in /run/libvirt too.)
Notify maintainers
libvirt maintainers: @lovesegfault, @fpletz, @globin
Metadata

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

Okay, but I'm using flakes and that channel looks old. See relevant commit hashes above.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.17.0, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0pre20220322_d5d4d98`
 - channels(root): `"nixos-21.11pre333218.263ef4cc414, nixos-hardware"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Oof, happy to find this, it would have taken me a long time to figure out I needed to run systemctl start libvirtd first.
Can't the sudo wrapper point to the file in the nix store?


Changing edges in the model

Hello,
I want to solve my task using the library. In my problem, the source graph and target graph has the same nodes but some edges are different.  And I want to predict the changes. As I understood, the only manipulation with edges, nodes and global features is supported. Am I right?
Thank you in advance!

I guess the wider question here is what kind of learning algorithm do you plan to use?
You could for instance fully connect your initial graph, adding a marker for the edges in the source graph, and then try to predict the edges that are different in the target graph ?

@vbapst
Thank you so much for your answer. I've had the same idea as yours (using fully connected graphs), but I haven't checked the approach.

Closing assuming the last answer solved the problem due to inactivity. Please feel free to reopen if necessary.


Settings for controlling if outputs should be saved to disk

Originally filed here by @rebornix https://github.com/microsoft/vscode-jupyter/issues/4670
I think we should consider adding this feature into VSCode. I've implemented this same feature in Kusto notebooks.
Feels like something generic for VS Code.
Similar to having a cell toolbar position per viewtype, we can have Save Notebook Outputs per viewtype. This can be achieved easily from VS Code by passing an empty array into the serializer, when saving notebooks.
I personally think this would be useful for rest notebooks as well, one might want the output from the REST & GithubIssues Notebook to be saved in the notebook, just like kusto notebooks (some might want the charts/data & some might not).
Thus all notebooks can be considered serializable (including output for sharing) & user choses whether the output is to be persisted on disc or not.
E.g. with .NET team building their notebook format (*.dib) this would have to be impelemented yet again in their extension.
@rebornix /cc

E.g. with .NET team building their notebook format (*.dib) this would have to be impelemented yet again in their extension.

I doubt so, dib is a concatenation of cell inputs so it doesn't save outputs.

if outputs should be saved to disk

if we are talking about serializing the content and saving to disk, then it's the responsibility of the content serializer. VS Code has no idea how the outputs will be used to generate the final file format. If we ever implement something in the core, there are many open questions:

should the content serializer completely drop outputs info in the file, or save them as outputs: []? how would it know it's empty because users disable it or it's simply empty?
should the content serializer save execution count? Same question as above, how would it know?
is there any other metadata that can be linked with outputs and should have the same persistence story as outputs?

In Kusto extension, IIRC if users set not to save outputs, then we also make the executionCount transient. Maybe we do the same in Jupyter, but we don't know yet if it's generic enough.
cc @jrieken @tanhakabir @roblourens

Yeah, I don't believe that it can be done. The closest request we have is support for transient per output-instance, not per notebook type: https://github.com/microsoft/vscode/issues/120600

I don't have this case where I'm not saving outputs yet (no request yet) but this case might be possible from core by not giving output cell items in NotebookData when the serialize function is called?
So far though I don't have such a feature planned

+1
This would be also super useful in managing version control of .ipynb, in order to avoid committing large outputs/sesntive output which may be generated by the notebook.

I came looking for this. Sad to see it hasn't got much enthusiasm. I also want it

Preparing notebooks for teaching without accidentally giving students any pre-run outputs
Hiding sensitive output and metadata when committing to git repositories

Chiming in to say I'd also like this feature

It would be best to expose an ability to filter what get's saved, NBDEV that facilitate building python libraries in notebooks in literate programming, has special comment markup to remove only some outputs.  I would love to be able to run the same hook that is already implemented for Jupyter in vscode.

Does anybody know if this can be made possible by using the new Notebook: Code Actions On Save setting?

Workaround: add a git filter so that in the middle of committing, the notebook is intercepted and the outputs are removed before they are saved by git. The outputs are still saved to disk locally, so you can still see them, but when you git push the outputs won't be included, or if you git checkout a different commit, you won't see the outputs there.
See this gist (PS, run git add --renormalize . as a 4th step before your first commit, so that all of your existing notebooks get scrubbed, or you will get heinous confusing merge conflicts later.)

This is a big issue when you working on remote SSH with Jupyter Notebook files. During the VSCode's "save" action, a notebook that contains variables/data frames with a large amount of data in the output is written to the remote file. Sometimes this can cause VSCode to crash or wait a few minutes to finish the save action. It would be helpful If you could add an option to disable saving the notebook output to the file and keep it only in the memory during the session.


how to build armv7 arch ios

we user filament to render GLB Mode，Make a SDK for APP, but only arm64，not have armv7, HOW TO BUILD ARMV7 AND ARM64
I  TRY TO FIX BUILD.SH

THIS ERROR
-- Build files have been written to: /Users/botao/Desktop/aliyun/aidemo/filament/out/cmake-ios-release-armv7
[  0%] Building CXX object third_party/libgtest/tnt/CMakeFiles/gtest.dir//googletest/src/gtest-all.cc.o
clang: error: invalid iOS deployment version '-mios-version-min=11.0', iOS 10 is the maximum deployment target for 32-bit targets [-Winvalid-ios-deployment-target]
clang: warning: optimization flag '-fomit-frame-pointer' is not supported for target 'armv7' [-Wignored-optimization-argument]
make[2]: *** [third_party/libgtest/tnt/CMakeFiles/gtest.dir//googletest/src/gtest-all.cc.o] Error 1
make[1]: *** [third_party/libgtest/tnt/CMakeFiles/gtest.dir/all] Error 2
make: *** [all] Error 2

Do you need armv7 for iOS? This is not something we support nor intend to support since Apple doesn't support armv7 anymore.

The last armv7 iPhone was iPhone 5c, which only supports up to iOS 10. Filament requires iOS 11 support, so it doesn't make sense for us to support armv7 for iOS.

@bejado But. if not support armv7 ,How To Add exits APP ,the App IS Support ARMV7 And ARM64

Can I.  modify Shell. to. Support ARMV7 and arm64. to build filament.a for exit APP

others framework is support armv7 and arm64 ,only filament is arm64 , can not  compile and send appstore,can you help to solution this problem

Filament requires iOS 11 minimum and iOS 11 does not support armv7. We will therefore not provide support for it since we can't compile Filament for armv7.

hi. @romainguy  thouth filament support ios11,but. some exist app is support armv7 and arm64, and want to use filament , yet can not do this? how do you think about this case?

we are only to match exit app ,not only to support ios10, just so

You need to configure your app's build to only include Filament when targeting arm64 then.

@bejado  thiank you , can i. build source file to  support armv7 ,need to modify which file ,can you help tell me?  our app must support armv7,  if. you have not plan to support

This SO question may help you: Xcode Conditional Build Settings based on architecture.


Added system theme detection and logic to remove duplicate entries

Disclaimer: I am very new to programming and don't know what I am doing!
added system theme detection with the dark-light crate should work on all platforms. Default theme if system theme doesn't get detected is still Dark.
Added logic to remove duplicates. with the current version if you click a previous entry from the clipboard you get duplicate entries. which clutters the program and makes it harder to access the other entries. I added a logic that simply check for same entry and deletes it if it finds one before added the new one. Which results in the entry you clicked just going to start of the list instead getting duplicated.

You probably have problems with the format and clippy, check the result of the ci so I can pass it, anyway to fix the problem with the format, you can run this command:
cargo fmt --all

And to check the clippy and all the verifications you can run this other one

Note: You need to have cargo-make installed, just run cargo install cargo-make

cargo make

Or if you don't want to install anything
cargo clippy -- -D warnings
```

Voy a aprobarlo porque se ve bien, luego subiré una actualización acomodando ese error en clippy, no te preocupes, se ve muy bien, muchas gracias por tu ayuda :D


Only first Dimension saves to DB

Go to Dimensions
Edit an entry
Save
Try to edit another entry
An error in the console is displayed and you can't save your changes

Cannot reproduce it @teveritt  Can you share more details about the error?


Merge the metatheory repository into the main repository

The idea is that James will keep working on a branch, with periodic merges to master. This will make it easier to set up CI that tests the two against each other.
Everything is in a metatheory subdir, the history is preserved including a final move into the subdir. Usual caveats for git history and renames apply, but it should be no worse than if we'd moved it within the repository.

Also, there's no build of the Agda yet, we'll do that later.


Align audio player with img better

The audio player isn't very well aligned with the image.
Example 1 - common image size:

Example 2 - larger image size

Example 3 - default image

Not sure what a good solution is, given that images will be different sized. I think ideally, it should be centered with respect to the image, with a small amount of spacing between the bottom of the image and the top of the player.
I notice that image, player, and transcript are all in the same bootstrap column. Maybe separating into different columns would work?
Done when the audio player is better aligned with the image, either as suggested above or otherwise.

I think ideally, it should be centered with respect to the image, with a small amount of spacing between the bottom of the image and the top of the player.
@mccalluc: I agree with your recommendation above.


fix: apply severity and all breached thresholds to threshold behavior

Overview
Addresses https://github.com/awslabs/synchro-charts/issues/153
Basically, thresholds were originally designed to only consider a breach IF the data point is between 2 intersecting thresholds. This happens when there is a lower 'greater than' comparator and a higher 'less than' comparator.

The issue here is that a user could reasonably expect that the data points beyond this intersected area would trigger a breached threshold. Even though this use case was intentionally designed, it leads to a confusing user experience.
To address this issue, this PR:

Removes the logic to only consider a breached threshold within the intersected area.

Add an operator to the threshold label to make it more clear what the threshold is actually doing.

Fixes severity and add tests.

Updates documentation to reflect these updates.

Tests
[Include a link to the passing GitHub action running the test suite here.]
Legal
This project is available under the Apache 2.0 License.

@diehbria
Severity should now be addressed. After we find the breached thresholds, we look for the highest severity among those breached thresholds: https://github.com/awslabs/synchro-charts/pull/183/files#diff-d4d358a6fe4cbd74e77b0128e58965a48b750e6619f4f3b15f6bccd5c594908eR338.
I added a couple helper function highestPriorityThresholds, which returns the thresholds with highest priority, and isHigherPriorityThresholds, which returns an array of the higher priority thresholds.


Get typed value from ng-select component

Is there a way to get just the typed in value for ng-select ?
I know using a change listeneer, you can get selected items - but how do I get the typed value ?

I have the same problem.


Docs: remember to update the version number in conf.py

Hi.
On https://aqtinstall.readthedocs.io/en/latest/ , I got "aqtinstall 1.0.0b2 documentation" from its title; in the PDF file downloaded from https://aqtinstall.readthedocs.io/_/downloads/en/latest/pdf/ , I got "Release 1.0.0b2" from its cover and header. The version number of "1.0.0b2" does not match the version I selected.
This version number is defined in /docs/conf.py. It seems that it has not been updated for a long time:
https://github.com/miurahr/aqtinstall/blob/4f0d92dc977ca9255f7ac912e1fd4fc303e953c1/docs/conf.py#L59-L67
I think we need to update it before releasing each version, otherwise it may confuse the user.
The docs of Sphinx: https://www.sphinx-doc.org/en/master/usage/configuration.html#project-information

Good point.
Would it be possible to pull that information in automatically in some way? The file aqt/version.py contains a version number, and it appears to be generated automatically, by the root-level setup.py; maybe we could leverage the code that generates that version number to update the number in docs/conf.py. Another possibility would be a CI script that checks the version numbers in docs/conf.py, and maybe even the latest changelog entry.
It's hard to remember to update those numbers without a tool to remind you.

It's hard to remember to update those numbers without a tool to remind you.

I agree with you.

Would it be possible to pull that information in automatically in some way? The file aqt/version.py contains a version number, and it appears to be generated automatically, by the root-level setup.py;

Yes, it is possible.
In this repository, I cannot find any CI scripts for publishing documents, so I guess @miurahr has configured a webhook to let GitHub notify Read The Docs to fetch the latest content. To use the version information defined in the "/aqt/version.py" file, I think we need to do the following things:

Remove the webhook.
Use the generated version number. Add from aqt.version import __version__ to "/docs/conf.py" and replace the values of version and release fields with __version__ .
Create a series CI steps in "/.github/workflows/publish-to-test-pypi.yml" for building and publishing documents. These steps must be placed after the step used to build aqtinstall.

It is also feasible to overwrite the value defined in "/docs/conf.py" by passing the -D setting=value parameter when building documents, but we need to manually obtain the version number defined in "/aqt/version.py" first.

... Another possibility would be a CI script that checks the version numbers in docs/conf.py, and maybe even the latest changelog entry.

Hmm, I think it might be harder.

I guess @miurahr has configured a webhook to let GitHub notify Read The Docs to fetch the latest content.
Yes the webhook is configured.

Document is built on readthedocs for each master branch and release tags.
https://readthedocs.org/projects/aqtinstall/

I guess miurahr has configured a webhook to let GitHub notify Read The Docs to fetch the latest content.

Yes the webhook is configured.

Yes. Due to the existence of the webhook, Read The Docs will automatically fetch and build the documents when we update the "master" branch of the GitHub repository.
To use the version number that is generated by setuptools_scm, we have to build the documents, and then upload it to Read The Docs by ourselves; to keep the documents up to date all the time, we need to configure some CI steps to perform these operations (as I said in the previous comment).
It seems that you didn't fully understand what I said; please let me know if anything is unclear.

Thank you for suggestions.  Now latest document has version and release number as same as version.py version.__version__  by provisioning readthedocs build configuration in #389.


Enable tap for layer "below"

I am not sure if there is a way to change this, but currently if I create two layers of clusters, then only the last added (or topmost) reacts to gestures.
That happens also if they do not overlap. Is there a way to let the event sink through the layer if nothing was touched?
I am adding gesture like:
new Container(
              child: GestureDetector(
            onTap: () {
               .....
            },
            child: Icon(
              ......
            ),
          )),

Thank you

hi! any suggestions on how to address that?

You might try looking at the docs for gestures or reproducing this in a simple (non-flutter_map based) example to help pin down the issue.


Support compatible prometheus service

To check prometheus server availabiilty, IsOnline method is checking /api/v1/status/flags.
For our platform prometheus itself cannot handle our size and we use some compatible querier services in front of prometheus that can handle queries but do not have this api.
I'm wondering if it's possible to change the availability with a simple query (vector(1) ?) so it will support any prometheus compatible querier.
for info we are using thanos and cortex in addition to prom to do queries.

I'm wondering if it's possible to change the availability with a simple query (vector(1) ?) so it will support any prometheus compatible querier.

Yes that should be a harmless change. Are you willing to implement it?

FYI, IsOnline is called only for prometheus and during the startup time of Flagger's main goroutine. It is never be called in canary's control loop.
I think this is a little bit confusing since metric analysis phases do not invoke IsOnline and not check the availability of metric server.
@stefanprodan I think we'd better refactor IsOnline's behavior so that this is not prometheus specific, or just remove IsOnline method. Any thoughts?

The IsOnline method is intended for debugging Flagger's install. Before this, we got many issues from users that misconfigured the Prometheus URL, having a sanity check at startup helps users detect that.
What I think we should do is call IsOnline on canary initialisation for all metric providers used in the analysis metrics section and log a warning since Flagger startup check doesn't cover the metrics templates.


refactor(Operators): use isScheduler utility function

Use isScheduler where applicable to reduce code duplication. It wasn't being used for now.

:+1:

I'll check in myself around tomorrow morning, to see if any other comments / opinion on this.

Merged with 8bca656.


Add pod yaml file with injected istio

Signed-off-by: Peter Gubka<EMAIL_ADDRESS>

Coverage decreased (-0.2%) to 47.298% when pulling ece7d306b7b1555f942482ca6830e2b237c7b2cd on pgubka:master into 4e7c7cc66a85d7b47d5e0513c66cf0283be1f3b3 on contiv:master.


UID and GID for the main process are not optional

The spec requires UID and GID to be specified, so we shouldn't ignore if they are not specified.
Signed-off-by: Amit Saha<EMAIL_ADDRESS>

815ce4e2 looks good to me.

LGTM

LGTM


stream_output parameter of execute.execute_command does not work in multi-threaded context

[ ] main
[ ] 4-2-stable

https://github.com/irods/irods_testing_environment/blob/6e26e9269d3933538fae837b3ec9cdc593c3be1d/irods_testing_environment/execute.py#L18-L20
These lines are not thread-safe. Furthermore, even if they were, it wouldn't necessarily result in the desired outcome as they are not coordinating the setting of the log level. This should be fixed.

I think it would be better to just adjust stream_output based on the configured log level rather than forcing the global log level. In this way, the user can know for sure that the log level is being honored or, if they don't care, the configured log level is being used appropriately to determine if the output should be streamed.
See solution offered in #72


[card-92] concurrent ruby for parallel uploads

you can see the trend for successful uploads here:
http://build.codemancers.com:8080/job/integration-diff-engineering-blog/buildTimeTrend
earlier: 5 min 40 sec
now: 1 min 20 sec

require some validity checks, otherwise all good i suppose.


Create PnP + Windows.Devices.Enumeration support, including USB Group Terminal Blocks for USB MIDI Devices

Group Terminal Blocks for both USB MIDI 2.0 and creation for USB MIDI 1.0 is complete as of https://github.com/microsoft/MIDI/pull/159. This is now in testing and debug from driver point of view.

I've tested this with USB MIDI 2.0 and USB MIDI 1.0 devices and haven't had any problems. Group Terminal blocks are coming through in both cases as expected. Closing as complete/fixed.


[ltsmaster] std.datetime should ignore leapseconds file.

Backport from master, see here: https://github.com/ldc-developers/phobos/blob/ldc/std/datetime/timezone.d#L2410

This fixes the unittest failure on the ARMv7 builder.


Is it possible to have multiple dashboards?

Since Dashing runs on Thin, I know it's possible to start multiple server processes with dashing start --servers 2, but the problem is that config.ru has only 1 default dashboard set, so both Thin processes serve the same dashboard. Is there any way to specify two different dashboards?

How do you get config.ru to specify a different dashboard?


Extend spec to allow fetching of assets

Public-Facing Changes

Extend spec to allow fetching of assets

Description
Allows clients to request assets (files) from the server.
Example use case: Client requests mesh files which are referenced by a URDF

Left a few comments - I'd like to work through some different scenarios that we'd experience in studio i.e. local data w/layout, then using layout for live, and also data platform access so we are clear on what the various asset names can be in those cases. I don't feel like I have a good intuition that we've covered a cohesive story around these workflows. It might turn out that there is no single story so we'll see.

I still find the use of assetId in the request somewhat "awkward" if the intent is to load an asset by URI/URL.
I'm inclined to say that imposing the requirement that you fetch assets by URI sane given that this protocol is to power interfacing with Studio and this seems like a sane way to describe references to assets in Studio.

Yes, I would propose to do exactly that.

Does fetch asset return a single asset? What happens if there are multiple for a given "uri"? Is one picked randomly? Error?

When fetching by an identifier (such as URI), only a single asset can be returned (or error if the asset is not found). If we decide to later allow fetching assets by name, this may be different.

Maybe for websocket/live connections it is more obvious that there should be one asset for a uri but for mcap we allow the same name to appear for multiple attachments. If you imagine that the name would be the uri (i.e. package://foo.urdf) then there's an open question on what the Studio APIs for fetching an asset would be like. Maybe that's a problem for studio but worth considering whether that flexibility needs to find its way to the ws-protocol.
All of that is to say maybe the opcodes could be specific like fetchAssetById fetchAssetByUri. Another common term used from ORM lands is findOneBy vs find. Not necessarily saying we should adopt these terms here but instead offering them as other kinds of prior art for "finding stuff".

I don't think we should be influenced here too much how assets (attachments) are stored & retrieved in MCAP. At least I don't see why this should prevent us from enforcing that the fetchAsset (by URI) operation only returns a single asset. If we realize that this operation is not sufficient, we can easily add another fetchAssetsByName operation as you have noted.


Update docs so that they no longer use the "inputs" pipeline field

which has been removed as part of the 1.6.0 release

LGTM


Swift Compiler Error (Xcode): Method does not override any method from its superclass /Users/sony/.pub-cache/hosted/pub.dev/flutter_inappwebview-5.8.0/ios/Classes/InAppWebView/InAppWebView.swift:1225:25

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected Behavior
The error in text format (screenshot above):

Launching lib/main.dart on iPhone 16 Plus in debug mode...
Running Xcode build...
Xcode build done.                                           19.3s
Failed to build iOS app
Swift Compiler Error (Xcode): Method does not override any method from its superclass
/Users/sony/.pub-cache/hosted/pub.dev/flutter_inappwebview-5.8.0/ios/Classes/InAppWebView/InAppWebView.swift:1225:25

Could not build the application for the simulator.
Error launching application on iPhone 16 Plus.

-- After running the flutter project it shows the error though it previously worked very well in my machine on previous version of flutter and xcode.
What causes the issue ?
What to do ?
Steps with code example to reproduce

Pubspec.yaml file
name: abc
description: A new Flutter project.
publish_to: 'none'
version: 1.0.0+1

# Code Structure :- GetX

environment:
  sdk: ">=3.0.0 <4.0.0"

dependencies:
  path: any
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.2
  flutter_lints: ^3.0.1
  get: ^4.6.5
  sizer: ^2.0.15
  get_storage: ^2.1.1
  fluttertoast: ^8.2.1
  intl: ^0.17.0
  auto_size_text: ^3.0.0
  badges: ^3.1.1
  scrollable_positioned_list: ^0.3.6
  syncfusion_flutter_gauges: ^21.1.41
  charts_flutter_new: ^0.12.0
  align_positioned: ^3.0.0
  date_picker_timeline: ^1.2.3
  flutter_slidable: ^3.0.0
  youtube_player_flutter: ^8.1.2
  table_calendar: ^3.0.8
  sqflite: ^2.2.8+1
  firebase_core: ^2.10.0
  firebase_auth: ^4.4.2
  google_sign_in: ^6.1.0
  cloud_firestore: ^4.5.3
  pin_code_fields: ^8.0.1
  flutter_rounded_date_picker: ^3.0.2
  flutter_local_notifications: ^16.2.0
  flutter_native_timezone: ^2.0.0
  rxdart: ^0.27.7
  flutter_tts: ^3.6.3
  audioplayers: ^5.2.1
  share_plus: ^7.2.1
  url_launcher: ^6.1.10
  open_settings: ^2.0.2
  android_intent_plus: ^4.0.3
  rate_my_app: ^2.0.0
  date_format: ^2.0.7
  connectivity_plus: ^5.0.2
  in_app_purchase: ^3.1.11
  dio: ^5.1.1
  in_app_purchase_android: ^0.3.0
  in_app_purchase_storekit: ^0.3.6+3
  timezone: ^0.9.2
  google_mobile_ads: ^3.0.0
  app_tracking_transparency: ^2.0.4

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:

  uses-material-design: true

  assets:
    - assets/database/
    - assets/icons/
    - assets/images/
    - assets/img_ex/
    - assets/img_ex/ABDOMINALCRUNCHES/
    - assets/img_ex/ADDUCTORSTRETCHINSTANDING/
    - assets/img_ex/AIRCYCLING/
    - assets/img_ex/ALTERNATINGHOOKS/
    - assets/img_ex/ALTVUP/

  fonts:
    - family: SFProDisplay
      fonts:
        - asset: fonts/roboto-regular.ttf #Reguler
          weight: 400

Stacktrace/Logs

Stacktrace/Logs
Swift Compiler Error (Xcode): Method does not override any method from its superclass
/Users/sony/.pub-cache/hosted/pub.dev/flutter_inappwebview-5.8.0/ios/Classes/InAppWebView/InAppWebView.swift:1225:25

Could not build the application for the simulator.
Error launching application on iPhone 16 Plus.

Flutter version
v3.24.4
Operating System, Device-specific and/or Tool
iOS 18.1
Plugin version
My flutter project doesn't have the dependency but it shows the error
Additional information
No response
Self grab

[X] I'm ready to work on this issue!

You are using 5.8.0 plugin version. You should use the latest 6 version otherwise it won't compile.

I don’t know which dependency (probably youtube_player_flutter) may use it, but it is using an outdated version of the plugin.
Check for dependency version upgrade


Unattented error occur : list index out of range

20.11.2017 10:51:052017-11-20 09:51:05,030 - INFO - Start to dump databases...
20.11.2017 10:51:052017-11-20 09:51:05,589 - ERROR - Unattented error occur : list index out of range
20.11.2017 10:51:052017-11-20 09:51:05,590 - ERROR - Traceback (most recent call last):
20.11.2017 10:51:05  File "/opt/backup/backup.py", line 222, in <module>
20.11.2017 10:51:05    listServices = rancherService.getServices()
20.11.2017 10:51:05  File "/opt/backup/fr/webcenter/backup/Rancher.py", line 60, in getServices
20.11.2017 10:51:05    instance['host'] = self._client._get(instance['links']['hosts'])[0]
20.11.2017 10:51:05IndexError: list index out of range
20.11.2017 10:51:05
20.11.2017 10:51:05[cont-init.d] 04-run-backup.sh: exited 1.
20.11.2017 10:51:05[cont-finish.d] executing container finish scripts...
20.11.2017 10:51:05[cont-finish.d] done.
20.11.2017 10:51:05[s6-finish] syncing disks.

Image: webcenter/rancher-backup:2.0.1-1

Data with DEBUG=true:
20.11.2017 11:19:402017-11-20 10:19:40,946 - DEBUG - http://<IP_ADDRESS>:8080 "GET /v2-beta/services/1s98/instances HTTP/1.1" 200 None
20.11.2017 11:19:402017-11-20 10:19:40,956 - DEBUG - http://<IP_ADDRESS>:8080 "GET /v2-beta/containers/1i176/hosts HTTP/1.1" 200 539
20.11.2017 11:19:402017-11-20 10:19:40,958 - ERROR - Unattented error occur : list index out of range
20.11.2017 11:19:402017-11-20 10:19:40,958 - ERROR - Traceback (most recent call last):
20.11.2017 11:19:40  File "/opt/backup/backup.py", line 222, in <module>
20.11.2017 11:19:40    listServices = rancherService.getServices()
20.11.2017 11:19:40  File "/opt/backup/fr/webcenter/backup/Rancher.py", line 60, in getServices
20.11.2017 11:19:40    instance['host'] = self._client._get(instance['links']['hosts'])[0]
20.11.2017 11:19:40IndexError: list index out of range
20.11.2017 11:19:40
20.11.2017 11:19:40[cont-init.d] 04-run-backup.sh: exited 1.
20.11.2017 11:19:40[cont-finish.d] executing container finish scripts...
20.11.2017 11:19:40[cont-finish.d] done.

/v2-beta/containers/1i176/hosts:
{
"type": "collection",
"resourceType": "host",
"links": {
"self": "…/v2-beta/containers/1i176/hosts"
},
"createTypes": { },
"actions": { },
"data": [ ],
"sortLinks": {
"accountId": "…/v2-beta/containers/1i176/hosts?sort=accountId",
"agentId": "…/v2-beta/containers/1i176/hosts?sort=agentId",
"agentState": "…/v2-beta/containers/1i176/hosts?sort=agentState",
"computeFree": "…/v2-beta/containers/1i176/hosts?sort=computeFree",
"computeTotal": "…/v2-beta/containers/1i176/hosts?sort=computeTotal",
"created": "…/v2-beta/containers/1i176/hosts?sort=created",
"description": "…/v2-beta/containers/1i176/hosts?sort=description",
"hostTemplateId": "…/v2-beta/containers/1i176/hosts?sort=hostTemplateId",
"id": "…/v2-beta/containers/1i176/hosts?sort=id",
"isPublic": "…/v2-beta/containers/1i176/hosts?sort=isPublic",
"kind": "…/v2-beta/containers/1i176/hosts?sort=kind",
"localStorageMb": "…/v2-beta/containers/1i176/hosts?sort=localStorageMb",
"memory": "…/v2-beta/containers/1i176/hosts?sort=memory",
"milliCpu": "…/v2-beta/containers/1i176/hosts?sort=milliCpu",
"name": "…/v2-beta/containers/1i176/hosts?sort=name",
"physicalHostId": "…/v2-beta/containers/1i176/hosts?sort=physicalHostId",
"removeAfter": "…/v2-beta/containers/1i176/hosts?sort=removeAfter",
"removeTime": "…/v2-beta/containers/1i176/hosts?sort=removeTime",
"removed": "…/v2-beta/containers/1i176/hosts?sort=removed",
"stackId": "…/v2-beta/containers/1i176/hosts?sort=stackId",
"state": "…/v2-beta/containers/1i176/hosts?sort=state",
"uri": "…/v2-beta/containers/1i176/hosts?sort=uri",
"uuid": "…/v2-beta/containers/1i176/hosts?sort=uuid"
},
"pagination": {
"first": null,
"previous": null,
"next": null,
"limit": 100,
"total": null,
"partial": false
},
"sort": null,
"filters": {
"accountId": null,
"agentId": null,
"agentState": null,
"computeFree": null,
"computeTotal": null,
"created": null,
"description": null,
"hostTemplateId": null,
"id": null,
"isPublic": null,
"kind": null,
"localStorageMb": null,
"memory": null,
"milliCpu": null,
"name": null,
"physicalHostId": null,
"removeAfter": null,
"removeTime": null,
"removed": null,
"stackId": null,
"state": null,
"uri": null,
"uuid": null
},
"createDefaults": { }
}```

Rancher:  v1.6.10

This is our temporary solution (Monkey patch):
                if 'instances' in service['links']:
                    service['instances'] = self._client._get(service['links']['instances'])
                    for instance in service['instances']:
                        hosts = self._client._get(instance['links']['hosts'])
                        if isinstance(hosts, list):
                            instance['host'] = hosts[0]
                        else:
                            instance['host'] = hosts

                    logger.debug("Service %s have %d intances", service["name"], len(service['instances']))

I am not sure of this solution but it works.

I have the same error
`webcenter/rancher-backup:2.0.1-1``
Rancher: 1.6.10
This line https://github.com/disaster37/rancher-backup/blob/3e589e6e8948309dbb9fce8423707a68db8e98cd/backup/src/fr/webcenter/backup/Rancher.py#L60
There are 2 instances, but the first has the state: "purged" and don't have data

Same problem. @disaster37 Could be this fix released in stable version? I tried to use develop, but it did not work at all for me.


Adding extensions and separating entity configuration for better code d…

Hi Eduardo,
How Entity Framework Core don't implement the class EntityTypeConfiguration and the possibility of add configuration by modelBuilder. I implemented a solution for this problem using extension methods.
I hope you enjoy!

Adding extensions for entity mapping (Fluent API)


No Sources in Multiplatform

Describe the bug
No Sources in Multiplatform, IDE can'r find them
Affected platforms
Select one of the platforms below:

All

If the bug is Android-only, report it in the Jetpack Compose tracker
Versions

Kotlin version*: 1.9.21
Compose Multiplatform version*: 1.5.11
OS version(s)* (required for Desktop and iOS issues): Arch Linux
OS architecture (x86 or arm64): x86
JDK (for desktop issues): 17

To Reproduce
Steps and/or the code snippet to reproduce the behavior:
Look sources
Expected behavior
See sources
Screenshots

Hello!
I can't reproduce that. Could you please provide information about reproducing the issue, step by step?

I have the same problem after apply org.jetbrains.compose plugin in shared project. Add a dependency e.g. implementation(compose.material3) to commonMain. Then open MyApplicationTheme.kt in androidApp, from there open MaterialTheme composable on line 49 with Command+Click. That will reproduce this bug.
Here is the demo project https://github.com/daoenpan/kmp-bugreport

I'm facing the same issue. I created a project through the Kotlin Multiplatform Wizard web page. When I add, for example,
@Composable
fun Foobar() {
    Text("Foo")
}

to a file under androidMain and click go to definition on the Text function, it will show the sources of Text composable. But if I add Foobar under commonMain and go to definition of Text from there, I get IntelliJ API Decompiler stub source generated from a class file.
Is it possible to access the sources from commonMain somehow?

I get the source on Android Studio Iguana | 2023.2.1

Hello! I can't reproduce that. Could you please provide information about reproducing the issue, step by step?

Now it's working in commonMain, but it doesn't work for specific target for example desktopMain (aka jvmMain)

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Implement Hi-C

Arises from liaison activity with Stanford center, as tracked in Jira: https://sagebionetworks.jira.com/browse/HSD-82
RFC doc: https://docs.google.com/document/d/1dJ7NUoVCtewdtny8bITwtWnzItB4IibL5kJO3ZNh0go/edit?usp=sharing
Implementation checklist

[ ] Branch created
[ ] HTAN.model.csv editied and pushed
[ ] HTAN.model.jsonld validated and created by Github action
[ ] PR opened
[ ] PR approved
[ ] PR merged into main
[ ] New Component added to ncihtan/HTAN-data-curator (if required)
[ ] DCA update requested
[ ] DCA updated

if no clarity on next steps by 1/13, push to next sprint.

Agree to proceed in 23-01 sprint and at least get this into staging

@aclayton555 Yep i'll get on it

Hey @PozhidayevaDarya I spotted some issues with Hi-C as implemented. Could you take a look the issue #169 and my review comments in the linked PR #170 .
Currently it isn't in the data model as we were missing the DependsOn Component entries.
I've added this, but if you could review the comments in my review on #170 and make any fixes in the add-missing-descriptions branch that would be amazing.

@adamjtaylor Yep, will do - thanks!

Needs #170 to be merged before will progress to staging


npx shadcn-svelte@next init 'tailwind.config.mjs' error "tailwindcssAnimate is not defined"

Describe the bug
When running the shadcn-svelte init command for the "next" version, (npx shadcn-svelte@next init), it adds the following inside the tailwind config file:
plugins: [tailwindcssAnimate]

This causes a runtime error as "tailwindcssAnimate" is not defined.

Proposed fix - update TAILWIND_WITH_VARIABLES inside https://github.com/huntabyte/shadcn-svelte/blob/next/packages/cli/src/utils/templates.ts from:
//...
plugins: [tailwindcssAnimate] // @line 117

TO
//...
plugins: [require('tailwindcss-animate')] // @line 117

Reproduction
Install latest version of shadcn-svelte for astro via https://next.shadcn-svelte.com/docs/installation/astro
Logs
No response
System Info
System:
    OS: macOS 15.0.1
    CPU: (8) arm64 Apple M2
    Memory: 59.80 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.15.1 - ~/.nvm/versions/node/v20.15.1/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v20.15.1/bin/npm
  Browsers:
    Chrome: 130.0.6723.92
    Chrome Canary: 132.0.6823.1
    Edge: 130.0.2849.68
    Safari: 18.0.1
  npmPackages:
    bits-ui: ^1.0.0-next.46 => 1.0.0-next.46
    svelte: ^5.1.11 => 5.1.11
    vaul-svelte: ^1.0.0-next.2 => 1.0.0-next.2

Severity
blocking an upgrade

Generally you get linting errors when you use require in that way. All you need to do is npm install. Please confirm that the tailwindcss-animate package was added to your package.json. If it wasn't added on init I can get a PR working for that.

Oh yeah good point and thanks for your reply.
It looks like it is getting added to the package.json on running init however it's not getting imported into tailwind.config.mjs and hence the runtime error.

Last I checked it completely overwrote the config. Let me look into this in a bit.

Just ran it it seems to overwrite this config without any issues make sure you are running
npx shadcn-svelte@next init

For me, it's adding import tailwindcssAnimate from "tailwindcss-animate"; to the top of the file no issue, not sure why it wouldn't be there.
Instead, I'm getting:
TS1259: Module ... can only be default-imported using the esModuleInterop flag

index.d.ts(3, 1): This module is declared with export =, and can only be used with a default import when using the esModuleInterop flag```

Suppressing this with a `@ts-expect-error` right now, but does seem that library isn't maintained.

For me, it's adding import tailwindcssAnimate from "tailwindcss-animate"; to the top of the file no issue, not sure why it wouldn't be there.
Instead, I'm getting:
TS1259: Module ... can only be default-imported using the esModuleInterop flag

index.d.ts(3, 1): This module is declared with export =, and can only be used with a default import when using the esModuleInterop flag

Everything works fine apart from the red squiggly, so just suppressing this with a @ts-expect-error right now, but does seem that library isn't maintained.
Mainly putting this here not to report an issue (I don't feel that it warrants that), but if anyone else with default SK setup sees a red squiggly when searching.

This sounds like an issue with tsconfig but if you can create a repro I can take a look!

In all honesty I've just checked the tsconfig and esModuleInterop is true.

In all honesty I've just checked the tsconfig and esModuleInterop is true.

Maybe need to restart ts server?

Just ran it it seems to overwrite this config without any issues make sure you are running
npx shadcn-svelte@next init

I've run this again and all seems okay. I'm not sure what I did the first time around my mistake (I'm just new haha)
Thank you for looking into it anyway :)