Support large vrt from ArduPilot SRTM1 terrain server

background
This PR adds support for loading terrain from the ArduPilot terrain server. I've manually generated the VRT dataset that refers to all the tiles. It's a massive dataset, and if you try loading it on ros2 branch, it will cause a runtime memory allocation failure crash from trying to read 4TB from the GDAL dataset.
To fix it, I added ROS-configurable limits on the grid map size, and set them to reasonable defaults such that none of the other tif files in your terrain-models/models/*.tif have problems.
Next, in order for the user to set the map origin instead of assuming it's at the center of the dataset, I've added ROS params to set the origin in WGS-84 lat-lon. The vrt launch file
runs the ArduPilot vrt dataset with an origin at CMAC, a popular flying field for ArduPilot SITL.
Due to the increased flexibility of loading terrain, additional protections needed to be added. Through some forward and reverse geo transforms, the user-configured origin is validated to be inside of the raster dataset bounds.
To preserve existing behavior, the configured origin defaults to NaN, and falls back to the assumption that the origin will be at the center of the dataset.
other improvements
I've also done some drive-by improvements such as

Switch from bind to lambda https://github.com/ethz-asl/grid_map_geo/pull/43
Use smart pointers for GDAL dataset to fix memory issue https://github.com/ethz-asl/grid_map_geo/pull/44
Replace tif specific naming to gdal_dataset since that's whats more accurately supported
Improve some error messages
Add a few assertions to catch errors when run in debug mode
Add instructions and launch support for running under GDB
Add const in some places it was missing
Add ability to just set the path to the DEM file, and then ignore color dataset. Previously, the default was ".", but the validity check was looking for empty string, so no data would be published.
Timestamp the grid_map message; previously the timestamp was not populated https://github.com/ethz-asl/grid_map_geo/pull/46
Move GDAL headers to the implementation (hide them from public API) https://github.com/ethz-asl/grid_map_geo/pull/42
Rename test_tif_loader to map_publisher
Check Wkt ProjectionRef is valid before continuing to initialize the grid_map. If the projection is unknown, it should not be used. https://github.com/ethz-asl/grid_map_geo/pull/47
Fix some unused variable compile warnings
Switch to using std::array for the transforms to increase compile time safety with double arrays

related
This relates to #35, because it allows using a large DEM. It doesn't yet support dynamically moving the grid. This needs to be exposed via params or a ROS service.
follow up

How to determine espg from the dataset at runtime instead of assuming WGS-84
whether to use gdal's transform functions
What's the plan for color data on srtm? Perhaps a good test would be to load the sertig elevation data from SRTM, but then overlay the sertig color file and check it loads?

assumptions

Ignore reprojection error at large distances
Ignore home position changes in the autopilot
split new MapPublisher from original test_tif_load, MapPublisher will "work with any dataset and do everything for you"

I attempted to try loading the sargans color tif file at the same time as the SRTM1 dataset for the same area. The pixel resolutions are different, and the data raster sizes are different. The maps don't align, and it causes the grid_map iterator to crash with a floating point exception. I haven't really been able to dig into it yet, but perhaps this would be OK to merge as-is without the capability to handle layers with different sizes/resolutions/datums?

I saw your open PR to align multiple maps may be related...

Exactly, you would need https://github.com/ethz-asl/grid_map_geo/pull/36 to align multiple DEMs

Thanks for the review

Sorry that it took me a while to review this.
Thanks for the amazing effort. I think the fact that we can load VRT into grid_map_geo is fascinating!
I have some comments that might be worth discussing.

I would prefer if we could utilize tf as the primary mechanism of aligning maps.(at least for now) This way we do not need to worry about alignments or storing the origin position of the tif map. It would just automatically align in rviz.

Yea, this sounds like a good idea. Any recommendations?

The maximum map size needs to be always defined? Could we only use it when specified?

I adjusted the defaults to allow infinite size, but the params for max map size can still be overridden at launch time, which is necessary for the SRTM1 VRT.

I am a bit worried that the map_publisher will become more and more complicated as we add features. Could we have a simple tif_loader that demonstrates how you can use the API, and maybe a standard map publisher that is actually useful for loading VRT .etc

Yes, I can split it up if you like. There will be some code duplication; hope that's ok.

I am not sure if I understand the VRT loading, since it seems like it is mounted locally onto your system. (under resources?). What would be the path to creating a demonstration of dynamically loading small map tiles as an entity(vehicle) navigates through terrain? Is there a way to directly communicate with the server?

Fixed, all the examples use the file on the server now.

Sorry that it took me a while to review this.
Thanks for the amazing effort. I think the fact that we can load VRT into grid_map_geo is fascinating!
I have some comments that might be worth discussing.
I would prefer if we could utilize tf as the primary mechanism of aligning maps.(at least for now) This way we do not need to worry about alignments or storing the origin position of the tif map. It would just automatically align in rviz.
Yea, this sounds like a good idea. Any recommendations?

In principle, you should not need to do anything, as long as the tf is published with the correct datum: e.g. https://github.com/ethz-asl/grid_map_geo/blob/02189d61cdc30c384e59a76ba1d044e002c39ba9/src/test_tif_loader.cpp#L76-L90
The only problem is that the SRTM DEMs are represented in WGS84, but ROS coordiantes are orthograpthic(e.g. ENU). We can probably ignore the distortion for now and represent everything in WGS 84(e.g. x=46.0 means 46.0 degrees in longitude). Long term we should probably reproject this into UTM to be useful for actual fusion with other map representations.

@Ryanf55 Any updates regarding this direction?

@Ryanf55 Any updates regarding this direction?

@Ryanf55 Any updates regarding this direction?

Sorry for the delay. Busy time at work and little time for FOSS.
I've done a quick rebase and squash. Let me know what you would like with the launch files, and I'll get those cleaned up. The map publisher and tif loader are now separate.

Remaining tasks I'd like before merge if you aren't in a rush

[ ] Write tests for all the launch files, disable RVIZ, and make sure the nodes don't crash, and make sure they publish a non-empty map
[ ] Fix any bugs from testing (I think the original tif loader doesn't work because we need to set the origin to the raster origin.

@Ryanf55 I have started looking into implementing a rolling query depending on vehicle position.
What I learned in between though is that ROS 2 is not really usable in the field over LTE with large messages (elevation maps/images). It seems to be a known issue without a clear solution. Something we need to keep in mind before people use this package in the field

Since elevation unit is in m, and the x, y in degrees for WGS84 this is what it looks like when visualized in Rviz:


Port download helper from CI3

Needed for Compression module.
CI3 had skipped unit tests, reproduced here... will need fleshing out at some point.

The unit tests now show a number of risky tests - output buffers not closed!?
Another PR has caused these - I have made no changes to those tests :(

@ytetsuro That does solve my problem, thanks.


cant install plugin

Error: Failed to fetch plugin git+https://github.com/mapsplugin/cordova-plugin-googlemaps.git#multiple_maps via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Failed to get absolute path to installed module

I'm submitting a ...  (check one with "x")
[ ] question
[. ] any problem or bug report
[ ] feature request
plugin version:  (check one with "x")
[ ] 1.4.x
[ ] 2.0.0-beta2
cordova information:  (run $> cordova plugin list)
insert the output from the command here

Current behavior:

Expected behavior:

Steps to reproduce:

Screen capture or video record:

Related code, data or error log (please format your code or data):
insert any relevant code, data, or error log here

If your problem is solved, please consider small amount donation to this project.
Appreciate for your kindness.
https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/README.md#buy-me-a-beer

Please search the past issues before asking.
https://github.com/mapsplugin/cordova-plugin-googlemaps/search?q=install&type=Issues&utf8=✓

@wf9a5m75  please provide a working answer instead, there are different issues referenced on that search query.

Try this.......It doesnt make sense but works
Delete ios platform
$> cordova platform add ios  //install the platform first for just in case
// If you get the below error, just ignore. And try again.
Error: Failed to fetch platform<EMAIL_ADDRESS>Probably this is either a connection problem, or platform spec is incorrect.
Check your connection and platform name/version/URL.
Failed to get absolute path to installed module
$> cordova platform add ios
// Add the SDK plugin at first with --nofetch option
$> cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps-sdk --nofetch
$> cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps#multiple_maps --variable API_KEY_FOR_IOS="(API key)"

i re-added the platform and it worked. it didn't 1 hour ago. the hell is going on  😕

Thanks @chukwu and @tryadelion this solved my problem, and I didn't have to downgrade Cordova to 6.5 as mentioned in other articles.
I deleted my iOS platform and then added it back.
I then executed:
ionic cordova plugin add cordova-plugin-googlemaps
--variable API_KEY_FOR_ANDROID="(API Key"
--variable API_KEY_FOR_IOS="API Key"
Worked after that.

I had the same problem ...
I already had a project on Android developed on Windows 10 and I was now working on Mac for iOS. When I wanted to add the platform (sudo ionic cordova platform add ios) I had this problem ... I tried many things but the problem was that I added "sudo" in the command line. Removing "sudo" gave me permission errors in the platforms / ios directory, so I gave it read / write permissions and ran the command again ...
This was what worked for me, I hope it serves the one who has this problem


Consider speaker notes

Supported for reveal.js ; may need to be removed for the Jupyter notebook (? need to think about it).

Speaker notes (tagged with the notes class in a div) are note properly handled in the current conda environment : they are note filtered out of the slides. Need to investigate if an old version of pandoc is responsible for this bug.

There is apparently a specific issue here (with the build system?) since with the same conda environment, the reveal speaker notes "work". However, the mathjax stuff doesn't render well there (duplicated) and maybe the speaker notes slot is too tiny to contain the amount of content I want to put there.

Tested with a recent pandoc (2.14.2) and the mathjax redering is gone and there is a slider for long speaker notes which is fine.
So the remaining issues are:

[ ] why does the stuff not work (speaker notes detection) on my documents,

[ ] can I upgrade the setting to use a more recent version of pandoc

[ ] I have embedded an old version of reveal.js stuff ; the recent one had a different layout, not sure it works even with the recent pandoc. Need to test this.

OK, recent pandoc (2.14.2, maybe before?) have adopted the new (3.8.0) file layout of reveal.js.

OK, I see why speaker notes won't work with my process : there is a div removal / transformation scheme that removes all top-level divs, including those tagged as notes. Need to be more specific ...


Display players lost to in Stats Leaderboard

Please check the following

[x] Do the tests still pass? (see Run the Tests)
[x] Is the code formatted properly? (see Linting (Formatting))
For New Features:

[x] Have tests been added to cover any new features or fixes?
[x] Has the documentation been updated accordingly?

Please describe additional details for testing this change
As requested in #182 , this adapts the "Players Beaten" dialog to include a list of Losses in addition to the list of Wins.
In the "Total" column, a counter is also shown next to the player's name, depicting the number of times each player appeared in winning/losing matches over the weeks.

All feedback is appreciated! :)

Please don't hesitate to ask any questions you might have here, or if you'd like to chat in real-time (about the code, the game, or anything else), you can join us on discord
Thanks so much for your help! We just wrapped up our 3rd season championship last night and have kicked off the new competitive season, so this is a timely QoL change :)

@itsalaidbacklife thank you for the kind words and thorough feedback!! :smile:
I've updated the selectors and the tests (I left one for the outer v-card since it makes it easier to get to the intended close button), and also made sure to include the special case for the Total column.
Really excited about the thriving competitive scene! I haven't delved too deep into strategic play just yet, though played a few matches against AI and liked what I saw quite a bit, as a fellow digital card game enjoyer. :)
Thanks for the discord invite as well! Hopped aboard and will check it out.

Resolves #182


[Failing Test]: SpannerReadIT.testReadWithDataBoost failing PERMISSION_DENIED

What happened?
Affecting beam_PreCommit_Java_GCP_IO_Direct, e.g.: https://ci-beam.apache.org/view/PostCommit/job/beam_PreCommit_Java_GCP_IO_Direct_Cron/1121/testReport/junit/org.apache.beam.sdk.io.gcp.spanner/SpannerReadIT/testReadWithDataBoost/
Error message:
org.apache.beam.sdk.Pipeline$PipelineExecutionException: com.google.cloud.spanner.SpannerException: PERMISSION_DENIED: io.grpc.StatusRuntimeException: PERMISSION_DENIED: This instance (projects/apache-beam-testing/instances/beam-test) is not allowed to use DataBoost at this time. Please contact<EMAIL_ADDRESS>for more information.

The test was added in Apr 4th in #26007 but test start failing on Apr 7th. Possibly due to recent roll out of gcp service that included permission changes? CC: @darshan-sj
If needed we should add the permission to the service account used for Jenkins
Issue Failure
Failure: Test is continually failing
Issue Priority
Priority: 2 (backlog / disabled test but we think the product is healthy)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

CC: @jrmccluskey just FYI another failing test affecting also release branch. Though the issue is not in Beam side.

Working on this.

The feature is now enabled for the test project and instance and the error has stopped occurring now.

Recent successful CI build -- https://ci-beam.apache.org/view/PostCommit/job/beam_PreCommit_Java_GCP_IO_Direct_Cron/1166/testReport/org.apache.beam.sdk.io.gcp.spanner/SpannerReadIT/

The test now failing again:
09:47:30 org.apache.beam.sdk.io.gcp.spanner.SpannerReadIT > testReadWithDataBoost FAILED
09:47:30     java.lang.RuntimeException at SpannerReadIT.java:214

It stucks for hours and causing the test suite timeout.

@darshan-sj can you take a look at this again?

@darshan-sj
as of Apr 2024 the test is still disabled. Could you please take a look?


First realease for : AC light leak  +  Nop  & NopSF clear

This is the initial release for :

Nop & NopSF sequencer  to improve clear after saturated image
AC light leak masking

Thank you Pierre for doing this. The new sequencer file section looks great. I wanted to separate the FCS part out from this technote since it could be another whole technote. But the text you have already put in is probably good for this technote since you are talking about "dark". Let's focus on the EO testing result and not dive into much for other hardware stuff.


Add integration tests against crates in the rust-lang-nursery

This commit adds integration tests against some crates in the
nursery.
Each integration test is added as a separate build-bot, where
the rust-lang-nursery/${CRATE} is first downloaded and its tests run.
Afterwards, cargo fmt --all is applied to the crate, and the tests
are re-run. If the tests fail after formatting, the integration test
fails.
The crates that currently fail are added as allowed-to-fail, but the
intent is that either these crates or rustfmt should be fixed such
that the tests pass.
Closes #2692 .

Isn't rustfmt itself is covered by https://github.com/rust-lang-nursery/rustfmt/blob/659d0c97412f2b010003df445f4c9cd84aa4bc75/src/test/mod.rs#L207?

@gnzlbg Thank you for the PR! I have been thinking of a similar idea for a while, so it is great to see a PR.
In this PR each repo is formatted by rustfmt that is installed via cargo install. However, the source code of rustfmt is formatted by the master version, so self_tests will almost always fail unless the two versions match.
We should just remove rustfmt repo from this integration test.

@topecongiro I've removed rustfmt from the integration ci.
I've also opened PRs to clippy (https://github.com/rust-lang-nursery/rust-clippy/pull/2739) and mdBook( https://github.com/rust-lang-nursery/mdBook/pull/689) to remove removed options that trigger warnings because those warnings are real.
However, many repos trigger internal error and Warning/warnings .  IMO either rustfmt should return EXIT_FAILURE when that happens, or it should swallow these by default. Reporting a warning or an internal error that the user cannot do anything about, and still returning EXIT_SUCCESS is very weird.

So this PR is ready. All the allow_failure builds are red, and the other ones are green. The allow_failures should be fixed such that they can be turned on by default, preventing future PRs from breaking code for users.

@nrc what do you think?

LGTM, but will wait for @topecongiro since he started to review. Thanks for doing this! I think it is a great addition to our test suite.

Sorry for the late review. Thanks for the change! LGTM.


How to get the coverage plan for a particular polygonal area on the global map, instead of the whole global map.

What params to specify so as to get coverage of only some particular zone on the map.
Thank you

Unfortunately there is no such feature yet. Hence the full_ in the name :wink:
One option would be to enclose this area in the costmap so that the robot has no way out. In that case it should stay within the bounds.

@Timple Thanks for the help.
Trying to interpret your suggestion, I Understand that I need to have a script/package which will enclose the particular zone on the map from /map topic and publish it on another costmap2d topic. Then I need to specify this costmap2d topic to the full_coverage_path_planner package.
I would really appreciate if you can correct me or point me to a resource where I can try figure it out myself.

I meant enclosing a room in your costmap. Something like this but then with the 'door' closed:

If the robot cannot leave the room, the coverage path will stay within that room.

hi @Timple
I tried your recommendation. It worked great for most part, but there was one anomaly I couldn't understand.
As seen the picture below, the global path is not always same distance from the border (see the red arrows).
Thank you so much for your responses

In the current implementation the complete area is divided into a grid. The above artifacts are an unfortunate result of that.
A better implementation would be a finer grained grid where multiple rows are covered so these edge-cases would be taken into account.
Unfortunately that is not there yet..


[EPIC] Add stream type catalog support for Gravitino

Describe the proposal
Gravitino currently has already supported relational and fieset type catalog, with the increased requirements of streaming data system(message queue), we should add the stream type catalog support in Gravitino.
Task list

[ ] #2370
[ ] #2371
[ ] Add the code skeleton for stream catalog
[ ] #1297

@SteNicholas @zhoukangcn Can you take some time to join this discussion? thx.


add WES import_vcf.py script

Adding our WES import_vcf.py script to a common repo.

Thanks for the comments!

@ch-kr Sorry, could you approve again, looks like the suggested changes require new approval.


Static body scaling disable all collision in escene

Godot version:
3.2.3
OS/device including version:
Ubuntu studio 20.04
Issue description:
I created a scene with a tree, the tree contains a static body with collision. The idea is that the tree is scaled according to its growth.
Use that same mechanic to create more trees
I created several instances of these different trees in the main scene,
The problem is that once the static body is scaled, the collisions in the whole scene stop working, even the collisions of platforms and other nodes that have nothing to do with the tree.
The error only occurs if there are several trees of different types, with a single type of tree nothing happens.
Try creating trees one by one without duplicating anything (scenes, mesh, statc body). But nothing worked
My solution was to scale only the collision shape and not the static body, for this I had to separate the static body from the mesh in order to scale the mesh and the collision shape without touching the static body.
Steps to reproduce:

Create a scene for the tree
create a mesh with the tree model (I used a tree object from kenney.nl)
create a static body either using the mesh button in the menu or doing it manually, the error occurs in both cases.
Create another scene with another tree and repeat the process.
create main scene, with player (kinematic body, with collisions)
Add trees of different types to the main scene and create a script where the trees are scaled by manipulating the mesh and the static body, not the whole scene. (I used a timer to call the scaling function)
Remember to add a platform to pose the player and the trees.

Minimal reproduction project:
Just start the main scene called world.

minimo_error.zip

cc @madmiraal

This sounds like the scaling to 0 problem (#32405, #41031, #40952) which was fixed with #41426, which was cherry-picked for 3.2.4.
@CHIESOFTWARE please test with 3.2.4.beta1 and confirm that the problem has been resolved.

@madmiraal is fixed in the beta of godot 3.2.4, is working fine. Thanks


Bugfix - Broken AQL request when build is not defined in FilesGroup

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.

This PR is to fix a bug we've encountered on a group of Jenkins pipelines using the jenkins-artifactory-plugin.  The gist is that a call to download artifacts through the jenkins-artifactory-plugin results in a call to AqlHelper with a FilesGroup that doesn't have a build set.
This causes fetchBuildArtifactsSha1 to send a broken AQL request to Artifactory, that ultimately times out (in our case, causing a 504).  It used to work when the call went through PatternAqlHelper, which did the search but didn't do the additional filtering by build.
I've posted an example input, the rough callstack to get to this point, and the broken AQL that was sent from fetchBuildArtifactsSha1 below:
artifactoryServer.download(
  failNoOp: false,
  spec: """{
    "files": [
      {
        "pattern": "generic-local/subdir/latest/generic-file.txt",
        "target": "out/subdir/",
        "flat": "true",
        "recursive": "false"
      }
    ]
  }"""
)

// in jenkins-artifactory-plugin
org.jfrog.hudson.pipeline.scripted.steps.DownloadStep.Execution::runStep
org.jfrog.hudson.pipeline.common.executors.GenericDownloadExecutor::execute
org.jfrog.hudson.generic.FilesResolverCallable::invoke
// in build-info-extractor
org.jfrog.build.extractor.clientConfiguration.util.spec.SpecsHelper::downloadArtifactsBySpec
org.jfrog.build.extractor.clientConfiguration.util.DependenciesDownloadHelper::downloadDependencies
org.jfrog.build.extractor.clientConfiguration.util.ArtifactorySearcher::SearchByFileSpec
org.jfrog.build.extractor.clientConfiguration.util.AqlHelper::run
org.jfrog.build.extractor.clientConfiguration.util.AqlHelper::fetchBuildArtifactsSha1

POST /artifactory/api/search/aql HTTP/1.1
Content-Type: text/plain; charset=ISO-8859-1
User-Agent: ArtifactoryBuildClient/2.32.3
Accept-Encoding: gzip,deflate

items.find({"artifact.module.build.name": "null","artifact.module.build.number": ""}).include("name","repo","path","actual_sha1")

I can confirm that this fix results in the pipelines working, based on a build of jenkins-artifactory-plugin with this updated dependency.  It ultimately skips the call to fetchBuildArtifactsSha1 if the build name or number isn't provided, since there won't be a useful return result if either isn't present.

I have read the CLA Document and I hereby sign the CLA


feat: access client side logging from Test Tools Modal

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/38456
PROPOSAL:
Tests

Important note: Tests should be performed only on native Android, iOS, Android Web and iOS web
iOS/Android:
Open dev menu and select "Open Test Preferences"
iOS Web/Android Web
Use physical device and use four-finger tap or switch isVisible={!!isTestToolsModalOpen} to isVisible in src/components/TestToolsModal.tsx

Enable "Client side logging"
Wait a while and disable "Client side logging"

iOS/Android
3. Verify path of the file is visible
4. Press "Share" and verify sharing modal appeared
5. Go to the Files on the device and verify it was saved
iOS Web/Android Web
3. Verify file was downloaded on the device
Downloading file after disabling Client side logging in troubleshoot page

Go to Profile -> About -> Troubleshoot
Enable Client Side Logging
Disable Client Side Logging
Verify the file is available to share/save

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Important note: Tests should be performed only on native Android, iOS, Android Web and iOS web

Open the app
Tap any area of the app with four fingers simultaneously.
Verify "Client side logging" is available
Enable Client Side logging
Disable Client Side logging

iOS/Android
6. Verify path of the file is visible
7. Press "Share" and verify sharing modal appeared
8. Go to the Files on the device and verify it was saved
iOS Web/Android Web
6. Verify file was downloaded on the device
Downloading file after disabling Client side logging in troubleshoot page

Go to Profile -> About -> Troubleshoot
Enable Client Side Logging
Disable Client Side Logging
Verify the file is available to share/save

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/13985840/5f103f8b-74a4-41ea-9d38-09bd673232d6

Android: mWeb Chrome
https://github.com/Expensify/App/assets/13985840/4856963b-7f8e-41fb-aa73-016fafe11d04

iOS: Native
https://github.com/Expensify/App/assets/13985840/4ee76ce6-ef92-42ed-a508-42aa30ac56a2

iOS: mWeb Safari
https://github.com/Expensify/App/assets/13985840/37c81180-ac77-4f1a-a483-9ecba34ef483

MacOS: Chrome / Safari
n/a

MacOS: Desktop
n/a

@s77rt thanks for your review, I have addressed all of your comments

On toggle on I think we need to hide the previous path and share buttons
https://github.com/Expensify/App/assets/16493223/3b56df3e-0113-4e6a-92b1-6edc043a5a65

@shawnborton The path (in muted text) and the Logs menu item

Got it - yeah, I think it works.

Who is this waiting on -- @s77rt ?  what is the next step and when will it happen?

Not sure I can test on Android. It does not seem to be working (file not being saved?)

I think it might be an emulator thing, here's the video from Android device:
https://github.com/Expensify/App/assets/13985840/eb715be6-f3d0-4db6-88b4-c9ee4cdd3925
On the emulator I have the same issue, but i think we might assume the file is accessible by having a file name here:
https://github.com/Expensify/App/assets/13985840/1dd81cec-e9d2-46eb-94a9-bbdb7d11a1aa

Pressing Share causes the illustration to disappear

Seems like it's related to the #38093. When Share menu is displayed, the app state is changed to backgrounded. I couldn't find the way to determine if share menu is open or not. Do you think it is a blocker or we could proceed without it and try to find a solution later?

I think it might be an emulator thing, here's the video from Android device

Can you confirm if the file is actually there? e.g. was able to read the content or share it

Do you think it is a blocker or we could proceed without it and try to find a solution later?

Thanks for checking this one. It's not a blocker, we can proceed and fix it later

@TMisiukiewicz Can you please address this too #38803 (comment)

@s77rt could you elaborate more on that? because I'm not sure I understand what exactly needs to be done, do you want to use both file prop and pass file through onShareLog param?

@TMisiukiewicz Can you please unify the testing steps? I think this is testable on all platforms

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I verified that this PR follows the guidelines as stated in the Review Guidelines
[X] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/16493223/c8a57133-8280-433a-9372-bb1f04280c03

Android: mWeb Chrome

https://github.com/Expensify/App/assets/16493223/3ede9486-47b6-48e4-873b-c88a2afb7b5d

iOS: Native

https://github.com/Expensify/App/assets/16493223/97b30227-fb64-4efd-bf4e-5270170ed976

iOS: mWeb Safari

https://github.com/Expensify/App/assets/16493223/6775a980-79d2-4947-ad3a-38603954ec7f

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/16493223/1ac3453d-8a7a-46d7-b0f2-07a78d1da52c

MacOS: Desktop

https://github.com/Expensify/App/assets/16493223/c358bbc0-4f00-4701-bd05-44b6de827184

I think it might be an emulator thing, here's the video from Android device

Can you confirm if the file is actually there? e.g. was able to read the content or share it

I just tested it by sending it with an email, seems like a file is visible as an attachment and once sent, it's possible to open it and read the content:
https://github.com/Expensify/App/assets/13985840/7f13b44a-b36c-4c27-a7e6-cb3297e78b71
https://github.com/Expensify/App/assets/13985840/ab1bde3e-0d00-444c-b32a-5d87ac226cea
I've updated test steps for Web & desktop for all platforms together with missing recordings for web & desktop based on Troubleshoot section

@TMisiukiewicz Thank you!

@amyevans thanks for you comments! I have updated the translations, I think it makes sense to use translation for Client side logging as well, would be good to remove all existing hard-coded phrases in the future

@amyevans thank you, updated!

🚀 Deployed to production by https://github.com/Julesssss in version: 1.4.60-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Hi 👋  @TMisiukiewicz @s77rt  ,For Client side logging, Why are we showing the share option for Android and iOS only? I am reviewing a PR that captures profiling data for web/desktop, and there we are showing the share button, but it downloads the file on the web. Moving forward with that PR, I need to understand why we hide the Share button for web/desktop for Client side logging.

@jayeshmangwani On web there is no share functionality. The log file is downloaded automatically after you stop capturing the logs.


How to switch to stable release?

I keep getting this error when on  Flutter 2.10.0-0.2.pre • channel beta and i can't figure out how to switch to the stable version.
I've set the environment variable to the latest stable version and it keeps going back to 2.10.0-0.2.pre.
remote: Target dart2js failed: Exception: /app/.pub-cache/hosted/pub.dartlang.org/platform-3.0.2/lib/src/interface/local_platform.dart:46:19: remote: Error: Member not found: 'packageRoot'. remote:       io.Platform.packageRoot; // ignore: deprecated_member_use remote:                   ^^^^^^^^^^^ remote: Error: Compilation failed.
Any help would be really appreciated.
@diezep

This is happening to me too


JVM static methods

In this PR we make some new Kotlin API more compatible with Java by introducing @JvmStaticMethod annotations.
We also update Spine dependencies, logging in particular.

@alexander-yevsyukov, FYI.

@dmdashenkov, the ”mystery” was implementation of the logging context, which previously was our own. Recently, core-java got dependency on gRPC-based logging context. Having two implementations of the logging context implementation (which are AutoService-es) cased Logging to fallback to default (no-op) implementation. That's why LogLevelMap` stopped working.
I alo disabled the test which assumes that WARNING is the default level. When we have JUL as the logging backend (which is our case), the default level is INFO, and it takes some tricks to make it stricter than INFO. If this is really needed, it can be done.
@yevhenii-nadtochii, @armiol, FYI.

@armiol, @alexander-yevsyukov, PTAL.

@alexander-yevsyukov, I removed the test you mentioned completely. It was added previously in this branch while trying to find the root cause of the logging issue.


fix: Normalize default constrain fn name

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Table created with the following SQL cannot run with SHOW CREATE
CREATE TABLE IF NOT EXISTS cpu_metrics3 (
    hostname STRING,
    environment STRING,
    usage_user DOUBLE,
    usage_system DOUBLE,
    usage_idle DOUBLE,
    ts TIMESTAMP DEFAULT CURRENT_TIMESTAMP,
    TIME INDEX(ts),
    PRIMARY KEY(hostname, environment)
);

Error:
pg:j2qY4ErSyTkM93HyanOBl400@i9ulpzu4mdgf/q6dj4up5mmriweak_flowers-public=> show create table "cpu_metrics3";
error: pq: XX000: sql parser error: Expected (, found: EOF
pg:j2qY4ErSyTkM93HyanOBl400@i9ulpzu4mdgf/q6dj4up5mmriweak_flowers-public=> show create table cpu_metrics3;
error: pq: XX000: sql parser error: Expected (, found: EOF
pg:j2qY4ErSyTkM93HyanOBl400@i9ulpzu4mdgf/q6dj4up5mmriweak_flowers-public=>

Server error:
2023-11-13T12:59:56.952175Z ERROR frontend::instance: Failed to execute query: show create table cpu_metrics3 trace_id=7129814621363507161 err=0: Table operation error, at src/frontend/src/instance.rs:419:14
1: Failed to execute statement, at /home/wayne/repo/greptimedb/src/operator/src/statement/show.rs:68:14
2: General SQL error, at src/query/src/sql/show_create_table.rs:110:76
3: , at src/sql/src/parser.rs:75:14
4: ParserError("Expected (, found: EOF")

Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

There are some clippy warnings.

Fixed. PTAL @killme2008


phpdoc_to_comment should be allowed above extract function

Bug report
phpdoc_to_comment fixer should not be triggered if the docblock comes before a call to extract(). Using a docblock here allows IDE to autocomplete variables that can be used after extraction.

PHP version: 7.4
PHP CS Fixer version: 2.18.2
php-cs-fixer fix --dry-run --diff --verbose

Code snippet that reproduces the problem

<?php

/**
 * @var int $test
 * @var string $another
 */
extract($args);

can you quote a PSR part when PHPDoc is allowed for this scenario, @coreyworrell ?
I find, myself, extract method to be harmful, almost in same way as eval. Unless PSR officially mention this case, I don't think we should support it, as it's not spreading good practices

@keradus I don't know of a PSR, and I do agree with it being harmful. WordPress uses it in several places in their templating functions. Working in VSCode autocomplete will only work if phpdoc format is used and not as just a comment... Is there an option or anything to maybe allow this? If not I can just disable the rule or leave out IDE integration for these WP projects.

I would suggest to take exactly what's needed - like $test = $args["test"] instead. if you can't, I would suggest to ignore given PHP file in PHP CS Fixer configuration.
I'm closing this thread, as we are not gonna add support for extract function. If you wish, feel free to continue the discussion.


[wip] Refactor and simplify

Rewriting the sample to simplify.

r? @tmarek-stripe mind taking a quick pass at the react changes to make sure I'm not doing anything wild? :D

r? @tmarek-stripe mind taking a quick pass at the react changes to make sure I'm not doing anything wild? :D


couple of dialogs not mentioned in the readme

I got these 2 dialogs during installation. They're not listed in the installation part of the readme. They are not unusual dialogs, but installation should probably mention about allowing incoming network connections

Thanks! I've updated the README: https://github.com/nyu-mlab/iot-inspector-client/releases/tag/v2.0.0-alpha


Elements list is not sorted consistently on elements tab of behavior form

Describe the bug
Sometimes when navigating to the elements list on the behavior form, the selected items are grouped together. Othertimes they are not grouped.
To Reproduce
Steps to reproduce the behavior:

Go to the "Demo environments" story
Open "Eco Lab Water Plant"
Click on Behaviors tab
Click on Tank levels behavior
See the selected elements are NOT grouped together
Click Cancel
Click the "Tank levels" behavior again
See the selected elements ARE grouped together

Expected behavior
The sorting and grouping should be consistent when navigating to and from the same behavior.
Screenshots
First load

Second load

I'm also seeing some weird behavior here when you switch scenes a few times and then come back to the Behavior form. The list showed me elements for a different behavior/scene entirely until I switched tabs and came back, then the list had the right items and was sorted properly.


Increase robustness of icon file loading

Search for icon in script dir to prevent a pic subdir in system script dir
Check if file exists before trying to load
Catch load exceptions and continue without wm iconphoto if failed

thanks for the improvement.


Is it possible to upload to Microsoft Store as a UWP application?

Do you mean as a 'project centennial' app? If so.... Yes, but I can't do that without an authorization from Microsoft.

Wait, is the program in the Microsoft Store in any format?  Not just a "universal" app or whatever UWP is?

No, not yet.

not yet

I do not like Microsoft Store and most App Store setups in general, but I acknowledge that some locked-down computers will only download from Microsoft-approved download servers.  As a result, it's useful for some users who otherwise won't be able to use your program and it might help with virus false-positive issues in new releases (by over-paranoid AV tools).  Still, it's far from critical.


knife supermarket didn't use chefignore file

I've added a standard chefignore file to our cookbooks. It includes a lot of default entries like this:
.vagrant
Vagrantfile

But when I call knife supermarket share ... the Vagrantfile is added to the cookbook tar.gz file.

This is a bug https://github.com/chef/chef/issues/2994. We work around the issue by copying the chefignore file up one directory to our cookbook_path.

Closing, since we are aware and tracking in chef/chef#2994


Hygiene: Enabled new linters and increased timeout.

Changes
Enabled new linters and increased timeouts; no functional changes in this PR.
Context: https://github.com/tektoncd/pipeline/issues/5899
/kind misc
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[N/A] Has Docs included if any changes are user facing
[N/A] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[N/A] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/assign wlynch


GH-38063: [C++] Use absolute path for external project's ar/ranlib

Rationale for this change
If we use relative path, CMake try using relative path from a build directory.
What changes are included in this PR?
Ensure using absolute path.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

Closes: #38063

+1


Image gets cropped

Subject: Image gets cropped
Description:
tried to upload an image on the events page.
Results:
Image gets cropped, only a part of the image is visible.
what I did expect is to display the full image (although smaller maybe) or before confirmation a rectangle to see what part of the image will be displayed and able to move / adjust that part.
Backend version: develop-71fd7f09
Frontend version: develop-71fd7f09
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:96.0) Gecko/20100101 Firefox/96.0
Page: https://app.couchers.org/event/new
User: richee (2740)

As intended for now.


Enabling IAM and SCRAM authentication at the same time fails

Describe the Bug
When enabling both client_sasl_iam_enabled and client_sasl_scram_enabled an Error: Too many sasl blocks is being produced.
Expected Behavior
Having multiple authentication is supported by AWS MSK and also by the AWS Provider, however the terraform-aws-msk-apache-kafka-cluster module is not working when enabling multiple SASL authentication at the same time.
Steps to Reproduce

Create a simple module consuming the terraform-aws-msk-apache-kafka-cluster module and set  client_sasl_iam_enabled and client_sasl_scram_enabled to true, proper providing an AWS SecretManager Secret ARN on the client_sasl_scram_secret_association_arns variable.
Run terraform init
Run terraform plan
The mentioned Error: Too many sasl blocks is going to be produced.

Screenshots

Environment (please complete the following information):

OS: OSX
Module Version: 0.7.3
Terraform Version: 1.0.2
AWS Provider Version: 3.63.0

Hey Guys, I am still facing this error in the latest version of the module. Could this be happening because the MSK cluster is already running?
I would appreciate any help in this matter. Cheers!


[New Port Request] facebook/igl

Library name
Intermediate Graphics Library (IGL)
Library description
No response
Source repository URL
https://github.com/facebook/igl
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response
### Tasks

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Update README.md

add b7 to links and move ipad mini 4/air 2 to supported

No need, readme has already been updated.

@gzimbric I added this PR before #321. Closing anyway.


Delete invalid assertion

Fixes #4694.

Pull Request Test Coverage Report for Build 2059

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 59.101%

Totals

Change from base Build 1037:
0.07%

Covered Lines:
12111

Relevant Lines:
20492

💛  - Coveralls


Read relay chain state

This refactors the ConsensusInherentProvider to a more general IsmpInherentProvider. This new inherent provider has been extended to support reading the relay chain state in order to respond to relay chain GET requests.
It might be possible to extend this further by fulfilling GET requests to sibling parachains, but it's unclear what the fee model for this inherent extrinsic would be for non-system requests.

Actually this blocked on https://github.com/paritytech/cumulus/pull/2794

wait for https://github.com/paritytech/cumulus/pull/2794 to make it into the next release

meh we don't need this


Removed StackedRNNCells from docs.

Hi!
From my undertanding, the keras framework should be in charge of using the class StackedRNNCells, not the user. The user just provides list of cells.
So I might be wrong but I think it's misleading to display this class in the docs.

That's a good point.


Opening logger for first time gets error

Start a new Jadeite.exe.
Login
Select Browse>>Browse Jadeite Logs
Get walkback

Note that the logger was only added in 3.0.70.
 Fixed in 3.0.71.


Adding functions into custom component scopes

This PR covers:

Adding individual environment scopes to custom components
Parsing <script> tags to build a tree of functions split by scope
Attaching these scopes to the corresponding component's environment

Note that this does not cover supporting multiple init() functions. This will come in the next iteration.

oh dang is it time to finally create a scenegraph label in this project?!

Also, please note that me merging #339 earlier this morning caused a merge conflict!  Shouldn't be too difficult to resolve though 😉

Have updated a decent amount based on feedback. I've tried to separate each commit out into individual concerns to try and help match to feedback and hopefully make re-review quicker.


Source generator help

Hi, this isn't directly related to fast endpoints, just wondering if anyone can help.
I'm trying to build my first source generator, and i've used the 'ServiceRegistrationGenerator' in this project as a starting point.
The source generator is designed to register hangfire recurring jobs and a basic version is working similar to the 'ServiceRegistrationGenerator'. You can see this on the master branch in this project
Idea is, user add an attribute too the recurring job class -
[RecurringJob("DeleteOldReportsAtMidnight", "* *678 * *")]
public class DeleteOldReports
{
    public void Execute()
    {
    }
}

The source generator automatically generates -
namespace TestSolution;

// <auto-generated/>

using Hangfire;
using Microsoft.Extensions.DependencyInjection;

public static class ServiceRegistrationExtensions
{
    public static IServiceCollection RegisterServicesFromTestSolution(this IServiceCollection sc)
    {
		RecurringJob.AddOrUpdate<TestSolution.DeleteOldReports>("DeleteOldReportsAtMidnight", x => x.Execute(), "* *678 * *");
                return sc;
    }
}

And all the user has too do is call the registration method at start up ServiceRegistrationExtensions.RegisterServicesFromTestSolution(services)

The bit im struggling with now is using more advanced attribute with optional parameters. You can see my desired attribute here on a different branch
The current implementation on the dev branch works the same as 'ServiceRegistrationGenerator' where it uses the argument list to get the properties, and uses them directly. As the more advanced attribute has more fields/ default optional field, i'm trying to pull the values from the properties, but cant find anyway of achieving this.
Any help would be appreciated

sorry mate! don't have the capacity right now to look in to your problem in detail.
does the syntax tree visualizer in VS not help? rider should also have something similar.
sidenote, we have an "off topic" chatroom on discord. maybe post there and people will respond.
cheers!


Fervo case study

Per https://github.com/NREL/GEOPHIRES-X/issues/87?title=Add+examples/case-studies+of+real-world+geothermal+systems
Papers:

https://eartharxiv.org/repository/view/5704/
Drilling cost paper https://pangea.stanford.edu/ERE/db/GeoConf/papers/SGW/2024/Elsadi.pdf?t=1706897516

Initial parameters & result merged in https://github.com/NREL/GEOPHIRES-X/pull/203; Fervo_Norbeck_Latimer_2024.txt and Fervo_Norbeck_Latimer_2024.out are still considered work-in-progress until @kfbeckers and/or @malcolm-dsider further review & refine.

Issue was automatically closed by merge of https://github.com/NREL/GEOPHIRES-X/pull/204 - re-opened because PR was not fully reviewed by myself or by geothermal SME

Thanks a lot, they are really helpful.

Documenting @kfbeckers' current version of Project Cape Case Study for posterity:
Link: https://gtp.scientificwebservices.com/geophires?shared-geophires-result-id=c0bb86de-1356-4cab-9907-faf4005ecc3d
Input:
Reservoir Model, 1
Reservoir Volume Option, 1
Reservoir Density, 2800
Reservoir Depth, 8500 feet
Reservoir Heat Capacity, 1000
Reservoir Thermal Conductivity, 2.7
Number of Fractures, 100
Fracture Shape, 4
Fracture Height, 300
Fracture Width, 2400
Fracture Separation, 15
Number of Segments, 1
Gradient 1, 73
Plant Outlet Pressure, 1000 psi
Number of Injection Wells, 12
Number of Production Wells, 12
Well Geometry Configuration, 4
Has Nonvertical Section, True
Multilaterals Cased, True
Number of Multilateral Sections, 2
Total Nonvertical Length, 5000 feet
Production Flow Rate per Well, 98
Production Well Diameter, 8
Injection Well Diameter, 8
Well Separation, 365 feet
Injection Temperature, 60 degC
Injection Wellbore Temperature Gain, 3
Reservoir Impedance, 0.01
Ramey Production Wellbore Model, 1
Utilization Factor, .9
Water Loss Fraction, 0.02
Maximum Drawdown, 1
Ambient Temperature, 10 degC
Surface Temperature, 10 degC
End-Use Option, 1
Plant Lifetime, 15
Power Plant Type, 2
Circulation Pump Efficiency, 0.80
Exploration Capital Cost, 0
Well Drilling and Completion Capital Cost, 4.8
Reservoir Stimulation Capital Cost, 10
Economic Model, 3
Starting Electricity Sale Price, 0.15
Ending Electricity Sale Price, 1.00
Electricity Escalation Rate Per Year, 0.004053223
Electricity Escalation Start Year, 1
Fraction of Investment in Bonds, .5
Combined Income Tax Rate, .3
Gross Revenue Tax Rate, 0
Inflated Bond Interest Rate, .05
Inflated Equity Interest Rate, .08
Inflation Rate, .02
Investment Tax Credit Rate, 0
Inflation Rate During Construction, 0.05
Property Tax Rate, 0
Print Output to Console, 1
Time steps per year, 10

Issue was automatically closed by merge of #204 - re-opened because PR was not fully reviewed by myself or additional geothermal SME

Resolved by https://github.com/NREL/GEOPHIRES-X/pull/220


Rename __p__→_parent, clean-up _factory and __rings__

This was partially discussed in #983 to rename a few variables. But also clean-up variables that are no longer used with shapely-2.

Rename __p__→_parent to make it "private", and generally avoid non-standard dunder names. Another consideration is to rename it to __parent to enable name mangling for sub classes, see docs Private Variable and Reserved classes of identifiers. Another consideration is with __del__, "Python guarantees that globals whose name begins with a single underscore are deleted from their module before other globals are deleted", implying that a double underscore variable would be deleted last.
Remove unused _factory, which was formerly a callable that returned instances of Shapely geometries, i.e. geom_factory
Remove unused __rings__, which was another former structure

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-2 of 10 (80.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 83.546%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shapely/geometry/base.py
4
5
80.0%

shapely/geometry/polygon.py
4
5
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
2163

Relevant Lines:
2589

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 10 (80.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 83.546%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shapely/geometry/base.py
4
5
80.0%

shapely/geometry/polygon.py
4
5
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
2163

Relevant Lines:
2589

💛  - Coveralls

I'm tempted to also clean-up _gtag and def gtag(self) in this PR too. However, the later is public:
>>> from shapely.geometry import Point
>>> g = Point(0,1).buffer(1)
>>> g.interiors.gtag()
-7868044578161086364

I'm tempted to also clean-up _gtag and def gtag(self) in this PR too. However, the later is public:

I would personally fine with just removing it in shapely 2.0 (without having first deprecated it, we can list it in the whatsnew under the breaking changes). I would be surprised if many people are using this?

Ok, I've update this to remove _gtag and gtag(). From what I see in shapely-1.8, this was used with shapely/geometry/proxy.py with the CachingGeometryProxy class, which was removed with #961. CI errors are unrelated.

Thanks!


How many epochs are needed to train on LJSpeech dataset from scratch?

I have trained the model from scratch on the  LJSpeech dataset with batch_size of 5 on a GTX 1650 GPU.
The validation loss seems converged around 0.6, after about 10 hours, 16000 iterations.
I  ran inference with inference.ipynb  replacing tacotron2_statedict.pt  with the generated  outdir/checkpoint_16000 file.
However, the generated .wav file is still awful.  It's almost pure noise.
Is it because the training time is  not enough or because the batch_size is two small?
I did change any code and hyper-parameter except the batch_size.

Is it because the training time is not enough or because the batch_size is two small?

Both actually.
Learning rate is roughly proportional to the square-root of the batch-size.

Using this paper as reference;
https://arxiv.org/pdf/1910.10288.pdf
You'll need
0.001 * (5/256)**0.5 = 0.00014 or 1.4e-4 learning rate

The paper used 256 batch size, and tacotron2 uses Location-Sensitive Attention.
The model started understanding the text between 1500 and 3000 iterations as that's when the error values in the graph start dropping.
Their batch size is 51 times larger than yours, so you need between 76,800 and 153,600 iterations for the model to start working.

I recommend using Google Colab for free faster GPUs, and using pretrained models to reduce the amount of iterations required for good results.

@CookiePPP Thanks for your kind reply.   Now I am training it on a RTX 3090 GPU, with batch_size of 80.
After 21709 iterations, with the validation loss converging around 0.35.
Now, it sounds like a human , but  still, she is like speaking a totally different language other than English.

@chenming6615 Did it end up working?

@CookiePPP Sr, can you tell me which dataset is fit to train difference language? tks :D

hour, from one or multi people, not noise?

@lutakrystal305
I do not know.

hour, from one or multi people, not noise?

1 Person. 4 Hours of data minimum. No background noise should be allowed.

@CookiePPP Thanks for your kind reply. Now I am training it on a RTX 3090 GPU, with batch_size of 80.
After 21709 iterations, with the validation loss converging around 0.35.
Now, it sounds like a human , but still, she is like speaking a totally different language other than English.

The fact you can run this repo under rtx3090 is interesting. can you please explain how you've managed to install tensorflow 1.15 while you are forced to install Driver 450 (which leads to CUDA 11.0, therefore cudnn 8.0) consequently the least version of tensorflow you could install is tf 2.5?

@ksaidin Well, I used an RTX 3050, and mine worked as well! For my experiences using my RTX 3050 to train using this repo, I used Miniconda! I used pip to force install Tensorflow 1.15 on it (pip install tensorflow==1.5), I also have to use a the requirement.txt's specific version of librosa and additionally, numbo. For the rest of the requirements.txt, I just install the latest versions on it...
To get PyTorch working on it, I must install CUDA 11.1 Update 1 specifically as firstly, it supports RTX devices with it and secondly, it's one of the only versions that you can from PyTorch's website. I install both PyTorch and Apex using conda (conda install nvidia-apex) (conda install conda install pytorch torchvision torchaudio cudatoolkit=11.1 -c pytorch -c conda-forge).
Finally, to get it training properly with coherent pronunciations of words, I've used @CookiePPP 's learning rate formula of "0.001 * (batch size/256)**0.5" so I could use the batch size of 4 (since RTX 3050 only has a 4GB VRAM, not suited I know, but I got it to work!).
Initially at my first test, I've used the preset default learning rate of 1e-3 from NVidia. It was able to train my model, but somehow the pronunciations got so messed up that it doesn't sound like the speaker at all. But after I used Cookie's learning rate formula and trained from it, my model was able to pronounce things as it should be.
So, Cookie's learning rate formula was able to help me train my model greatly and I thank them for the advice! Anyhow, that's how I was able to train using an RTX 3050.

@ksaidin Well, I used an RTX 3050, and mine worked as well! From my experiences using my RTX 3050 to train using this repo, I used Miniconda! I used pip to force install Tensorflow 1.15.2 on it (pip install tensorflow==1.15.2), I also have to use the requirement.txt's specific version of librosa and additionally, numbo. For the rest of the requirements.txt, I just install the latest versions on it...
To get PyTorch working on it, I must install CUDA 11.1 Update 1 specifically as firstly, it supports RTX devices with it and secondly, it's one of the only versions that you can install from PyTorch's website. I installed both PyTorch and Apex using conda (conda install nvidia-apex) (conda install conda install pytorch torchvision torchaudio cudatoolkit=11.1 -c pytorch -c conda-forge).
Finally, to get it training properly with coherent pronunciations of words, I've used @CookiePPP 's formula of "0.001 * (batch size/256)**0.5" to get a suitable learning rate, so I could use the batch size of 4 (since RTX 3050 only has a 4GB VRAM, not suited I know, but I got it to work!).
Initially at my first test, I used the preset default learning rate of 1e-3 from NVidia. It was able to train my model, but somehow the pronunciations got so messed up that it doesn't sound like the speaker at all. But after I used Cookie's learning rate formula and trained from it, my model was able to pronounce things as it should be, also sounding almost accurately like the speaker.
So, Cookie's learning rate formula was able to help me train my model greatly and I thank them for the advice! Anyhow, that's how I was able to train using this repo with an RTX 3050.

Thanks for your reply!  Would you please tell us how many hours it is needed to train it with just a RTX 3050 ?

Thanks for your reply! Would you please tell us how many hours it is needed to train it with just a RTX 3050 ?
@chenming6615 Well, it really depends on how much is my dataset, but since my dataset is quite small, it took me about 9-12 hours to get a decent voice out of it. So, if I would to have a bigger dataset, it would probably took me a day or two... It may sound like it's very long, but for me, it's actually quite such a small time compared to other stuff I've seen

@CookiePPP can you help me?
This model can't converge and inference will incorrect?
 // This is alignment inference from my model.

//This is my tensorboard at step 34k.

//This is validate and training loss
tks you :D

If i train a different language, should i train from scratch or pretrain tacotron2?

This model can't converge and inference will incorrect?

Yes, that model is probably never going to learn. Once the alignment gets stuck near the top (end of sentence), it normally never unlearns that.
You will need to restart training.
Using pretrained tacotron2 is still a good idea, but before you do another training run make sure;

all audio files do not have long stretches of silence
there is no silence at the start/end (trim the audio files if you haven't already)
the text doesn't have any constant characters used in the same place every time. e.g: If every sentence ends in ., the model might start thinking the . at the end of the sentence always means silence anywhere in the audio file.

If you can't get it working with these changes then look into guiding the attention.
also, I've already written a guide that might be helpful for you.
https://github.com/NVIDIA/tacotron2/issues/321#issuecomment-603894212

@CookiePPP, any recommendations for a weight_decay hparam?

@BlazJurisic No recommendations from me. I haven't experimented with that parameter before

Hi, I wanted to train tacotron 2 from scratch with 4652 sentences (Kurdish dataset) (10 hours), batch size 32. here are some plots:

does this model are going to be converge or not? please help

@sabat84
This looks good. It should converge given more time.

@CookiePPP
How many steps I have to wait?

the text doesn't have any constant characters used in the same place every time. e.g: If every text ends in ., the model might start thinking the . at the end of the sentence always means silence anywhere in the audio file.

Great Point there. Also, can i assume using , and - can help punctuation? I seem to have British voice data, and well as ya know, multiple words are broken.


Improve Playground performances

Improve Playground perf at loading.

We are good on this one right? @RaananW

I only wanted to put the  back for the search engine to read. will close this soon

I think we can close it now right?


update packages

:white_check_mark: Build TheIdServer 7.4.7-fix-dependencies1-0185 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/95f7b815a6 by @invalid-email-address)


WIP: R2 upload and download

This was part of last month's quick wins week but was not added due to back-end issues with wrangler's api token and the upload endpoint (which I'm not sure is resolved yet). This PR should help as a reference/starting point for @JacobMGEvans upcoming work.
Commands added:
wrangler r2 object get <objectPath>
wrangler r2 object put <objectPath>

Thanks! I'll iterate on it 😁


Support for k8s 122 (networking.k8s.io/v1 ingress API)

Closes #
n/a
Changelog
New

Add _helpers.tpl to auto determine supported ingress class

Changed

default value for values.global.ingress.pathType
ingress templates

Removed
n/a
Testing / Reviewing
this can be verified by installing the chart on different k8s versions - <1.14, <1.19 and after

implemented originally by @aiciobanu for ess-core

Hi @gchickma @liviu-niculescu some good helpers... similar to what we did in Flow chart manually
{{- $apiV1 := false -}}
{{- $apiVersion := "extensions/v1beta1" -}}
{{- if $.Capabilities.APIVersions.Has "networking.k8s.io/v1/Ingress" -}}
{{- $apiVersion = "networking.k8s.io/v1" -}}
{{- $apiV1 = true -}}
{{- else if $.Capabilities.APIVersions.Has "networking.k8s.io/v1beta1/Ingress" -}}
{{- $apiVersion = "networking.k8s.io/v1beta1" -}}
{{- end }}
{{- range $k, $v := $values.apps }}
apiVersion: {{ $apiVersion }}

Biggest difference is Flow was checking APIVersions instead of KubeVersion. Not sure if you want to do the same.
Alternatively, I would suggest not bothering with backwards compatability. We have removed this from the v4 of Flow as Kubernetes 1.22 is already EOL.

Hi @gchickma @liviu-niculescu some good helpers... similar to what we did in Flow chart manually
{{- $apiV1 := false -}}
{{- $apiVersion := "extensions/v1beta1" -}}
{{- if $.Capabilities.APIVersions.Has "networking.k8s.io/v1/Ingress" -}}
{{- $apiVersion = "networking.k8s.io/v1" -}}
{{- $apiV1 = true -}}
{{- else if $.Capabilities.APIVersions.Has "networking.k8s.io/v1beta1/Ingress" -}}
{{- $apiVersion = "networking.k8s.io/v1beta1" -}}
{{- end }}
{{- range $k, $v := $values.apps }}
apiVersion: {{ $apiVersion }}

Biggest difference is Flow was checking APIVersions instead of KubeVersion. Not sure if you want to do the same.
Alternatively, I would highly encourage not bothering with backwards compatibility in this case - Kubernetes 1.22 is already EOL. We have removed this from the v4 of Flow.

makes sense, was thinking that if people want to use say latest cicd chart for some other changes but are still running on older versions (as some still do), it would be a benefit
we could hardcode the API version though I suppose
thoughts @tlawrie @gchickma ?

@liviu-niculescu I agree with still support older versions of K8S, can't hurt, and as we know there are a lot of older K8S clusters still running across the broader teams we support.
Also, Boomerang CICD will not be upgraded anytime soon to use Flow v4.
With respect to checking K8S version vs API version - the API version method seems cleaner given it appears you do not need specify/check actual K8S versions per se, i.e., there's a helper method to use for checking which capability exists.

alright guys I changed it up similar to Flow and tested in a k8s 1.23 environment
have a look pls and let me know if you have any comment
cc @gchickma @tlawrie

Approving based on IBMs request.


Imperial temps show in C

Thanks for the plugin.  I just installed it and noticed that when I set it to report in imperial that it reflects the temperature with a "C" suffix instead of a "F" suffix in the status bar.  So, it is currently 44 F where I live.  Your plugin reports the temp as 44 C.

That is an easy fix, just go to the plugins settings and and edit the Weather String Format Statusbar. The default is...
' | %desc% | Current Temp: %temp%°C | Feels Like: %feels%°C | '
Just change the two 'C' to 'F'


Revit Built Elements - Support for multiple RenderMaterials

See discussion #860.
The Issue:
The Revit converter currently exports the Revit material of objects using a RenderMaterial. (this is done here)
And the current structure/usage of this means that there is one RenderMaterial per object.
However, this leads to issues with objects with multiple materials, such as windows, where the glass and the frames should be a separate material.

The solution we currently intend, would be for us to use multiple Mesh objects in the displayMesh/displayValue of Revit elements, one for each material.


removing the tracker(css) for selection of the element

when we click the button for the selection of the Element the CSS added for tracking the Element should be removed after an element is selected.

fixed sending a message in order and added the removeEventListener and even for the click(when user selects any elements )


[Odin] Fix installation of mods having a UTF-8 BOM in their manifest.json

Description
Remove possible UTF-8 BOMs from mod manifest.json files before deserialization. Serde does not support UTF-8 BOMs, actually it suggests handling them manually. Trying to install a mod that has a manifest.json file with a UTF-8 BOM fails with the following error during Odin startup:
[ODIN][INFO]  - Installing  https://valheim.thunderstore.io/package/download/OdinPlus/PlantIt/0.1.1,
thread 'main' panicked at 'Failed deserializing manifest: Error("expected value", line: 1, column: 1)', src/odin/mods/mod.rs:103:42,
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace,

Using a hex editor the UTF-8 BOM can easily be spotted:

Contributions

I changed how the manifest.json file is loaded.
I added removal of UTF-8 BOM from downloaded mod manifest.json files.

Checklist

[ ] I added one or multiple labels which best describes this PR.
[x] I have tested the changes locally.
[ ] This PR has a reviewer on it.
[ ] I have validated my changes in a docker container and on Ubuntu. (Only needed for Odin or Docker Changes)

Queued for pipeline, its getting late for me now, at first glance it looks great! :) excellent job! Tomorrow i will pull it down and run it through the usual ropes.

Thanks for the quick feedback! The build task failed due to a formatting/linting issue, should be resolved now.

@mbround18 I'm not 100% how to interpret, but looks like linter expects a trailing comma at the ZipArchive use statement?

Thanks a bunch!


Running great on MacOS, RPi has issues

I'm creating a Discord Bot, which checks our server status every minute and updates the bot's presence.
On my Mac it runs as expected. On RPi, when the update function runs for the first time, it gets a status etc. When it runs again, it kind of stucks in that first state and doesn't update. I don't get any errors, I use the promise version. It seems like it gets some kind of response.
Here's the code which runs every minute via node-schedule, I see the "Updating presence" log every time, so no errors:
var updatePrescence = () => {     ping(ip, port, (error, response) => {         if (error) {             console.log("Server Unreachable")             client.user.setPresence({ activity: { name: Offline 🔴, type: "PLAYING" }, status: 'dnd' })         } else {             console.log("Updating presence")             client.user.setPresence({ activity: { name: Online 🟢 - 👨💻 ${response.onlinePlayers}/${response.maxPlayers} - 📶 ${response.host}:${response.port}, type: "PLAYING" }, status: 'online' })         }     }); }

How frequently are you calling updatePrescence? Also what Node.js version are you using on your Mac and your RPi?

It runs every minute via node-schedule.
I've recently updated NodeJS to v12.18.3, I updated my Pi on Saturday automatically (sorry, I can't get the version until Friday).

I just published basically a complete rewrite of the library.


ENH:  add kibana trace analytics plugin and ES health check script into example setup

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merging this to share with kibana team.


Create a tool to load / extract information from a checkpoint.

Because people may need/want to see information which are available in a checkpoint, and because it might of any use for testing, we want to create  a command line tool which will allow us to read a checkpoint file.
This will need to be a subcommand which takes as input a checkpoint file, and extract data from it (e.g: load all proposals and print them on the commands line, loads all accounts and print them, etc).
Tasks

[ ] Add a new subcommand vega checkpoint extract (or similar)
[ ] It can take as input a checkpoint file and:

[ ] Create the hash for this checkpoint file
[ ] Extract all accounts
[ ] Extract all proposals
[ ] Extract all network parameters

[ ] Thins are displayed in an appropriate format which could be used for any automation / testing purpose (e.g: json).

Issue moved from /vegaprotocol/vega#3779 to this repo


chore(nvidia): update repo signing keys

What problem does this PR solve?:
Nvidia has updated and rotated their signing keys: https://developer.nvidia.com/blog/updating-the-cuda-linux-gpg-repository-key/
Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-NUMBER

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

To best ensure the security and reliability of our RPM and Debian package repositories, NVIDIA is updating and rotating the signing keys used by the apt, dnf/yum, and zypper package managers beginning April 27, 2022.
If you don’t update your repository signing keys, expect package management errors when attempting to access or install packages from CUDA repositories.
To ensure continued access to the latest NVIDIA software, complete the following steps.

😕 so they changed the keys and then released this a day after breaking everyone?!

Yea and as of right now the SLES repos still arn't updated ;(


ReactiveCocoa

Hey where would https://github.com/ReactiveCocoa/ReactiveCocoa fit on awesome-ios? cc @jspahrsummers @joshaber

Of course!


Unable to see tracking events in browser

I am self hosting analytics.js. I have a webserver on my machine on which I have analytics.js and a web page that uses analytics. However, no matter what I try, I do not see tracking events going out on page load.
Is there some magic api that needs to be called to get analytics.js to send events? I do not even see a call to segment's endpoint.
Any help appreciated.

No integrations are enabled by default, you need to initialize the tools you are using. Since you are using the self hosted version, there are no calls sent to Segment either.
I'd recommend using the hosted version first so you understand how it works https://segment.com/docs/sources/website/analytics.js/quickstart/.

Thanks for your response. However, there is no documentation I am aware of that shows how to initialize the integrations. Also, in the open source version, can the send to segment be enabled at all? or is it not supported?

Yes the documentation is pretty sparse, I'd check out the tests in analytics.js-core https://github.com/segmentio/analytics.js-core/blob/master/test/analytics.test.js for a starting point. You can still send data to any of of the tools listed here (including segment). https://github.com/segment-integrations?q=analytics.js-integration

I built analytics.js from scratch using this repo. When I try to use it, I get this exception.
analytics.js:27 Uncaught Error: cannot find module "is-empty"     at require (analytics.js:27)     at analytics.js:49     at Object.45.type (analytics.js:1470)     at call (analytics.js:47)     at require (analytics.js:26)     at analytics.js:49     at Object.39.is (analytics.js:1397)     at call (analytics.js:47)     at require (analytics.js:26)     at analytics.js:49
Any ideas how to fix it?


Custom widget for a search window to modify/add/remove items from queries.

By creating a custom widget, the window which contains a search box along with a window above for the outputs can be created easily on demand.
Modifying how it works with the application and/or database would be much easier than implementing one function for returning multiple windows.

This shall be left for another version, as it may over-complicate things for now.


Add fix command

Like the analyze command, I added a fix command that utilizes the replacement defined inside each rule to automatically replace the fixes
You can run the command on the example/lib and it will identify the 3 issues it has in the main.dart file, and fix only 1 (which has the replacement defined)

dart run dart_code_linter:metrics fix example/lib

@byshy  please pull changes

Done

As a quick note, before publishing the changes to pub get, we should run the dart formatter on the changed files (in this PR and the 3 before) since I have custom rules on my IDE to set the length of the dart line to 130, it should be 80, so we should run that then publish

Notes handled, I'm available for new notes if any 🙏

Amazing!!
Can we connect on linkedIn @santitigaga ?

yes of course, https://linkedin.com/in/santiago-garcia-gil-742b38252/

Invitation sent


Cache "Devicelist" for a certain time when configuring more devices

When configuring devices then each time "find a device" is clicked a new request is done to the Fritzbox, which can be much traffic. But the devicelist do not update that often, so it could be cached for some time (60 seoconds, longer)

The devicelist is cached (and will be updated during runtime) until the next start of this adapter. I think you are saving and closing the config dialog. Taht results in a restart of the Adapter. Try to add all desired devices and then save and Close.
If it dose not work, let me now


Allow to have specific flight scenarious

Right now objects and targets are generated at random. it would be interesting to be able to generate flight environments with fixed conditions, i.e. single obstacle placed midway between a drone and a target.

This was essentially solved while developing the demo for eyes (ray tracing based sensor) + trajectory generation mechanism.
Hence, now a flight scenario can be developed, such that an obstacle is placed somewhere on the path, which a quad seeks to follow.


Fix invalid syntax message containing commands the sender is not perm…

…itted to use

Current just a working PoC, not sure if this is the exact approach we want to take in the end


Minecraft won't launch

Running a fresh install of 1.10.2 in the newest Curse Client (no mods) and Minecraft runs fine..
I add Mekansim .298 and Minecraft won't launch:
fml-client-3.txt
I also get a "Half command" line in the Launcher Log
Launcher Log.txt

This looks like the common Nvidia problem which has been introduced in its latest driver update of January 24th. Please try forge without any mods.
If the issue persists, try installing (or rollback) of your driver.
For more information, go and take a look at the following reddit post.
https://www.reddit.com/r/feedthebeast/comments/5pydv9/dont_install_nvidia_gpu_driver_version_37849_it/

Thanks for the response. Rolling back seems to have fixed it (as was stated in the reddit post)


Rename podspec filename and update the Podfile file content

Why? what's the issue?

Why? what's the issue?

No issue, Just for the sake of consistent with other projects, such as:

https://github.com/react-native-community/react-native-webview/blob/master/react-native-webview.podspec
https://github.com/react-native-community/lottie-react-native/blob/master/lottie-react-native.podspec
https://github.com/react-native-community/react-native-maps/blob/master/react-native-maps.podspec

@Kennytian Some other projects use "RNxxxxxx" format. It's not a big deal but it is a breaking change for every user not using autolinking 😅


5010(OriginatorOptionF should be mandatory only if {5000} is not prent and {3600} is CTP)

Codecov Report

Merging #299 (42cb657) into master (cbbafa6) will decrease coverage by 0.03%.
The diff coverage is 33.33%.

:exclamation: Current head 42cb657 differs from pull request most recent head 0cd9305. Consider uploading reports for the commit 0cd9305 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #299      +/-   ##
==========================================
- Coverage   74.88%   74.86%   -0.03%
==========================================
  Files          73       73
  Lines        8622     8630       +8
==========================================
+ Hits         6457     6461       +4
- Misses       1593     1595       +2
- Partials      572      574       +2

Do you know what version of the Wire spec includes this?


Config Client Integration Documentation for Spring Cloud Kubernetes Config Server

Hi,
Can you add config client integration documentation for Spring Cloud Kubernetes Config Server.
I have 2 questions that I'm curious about this topic. Can you share code snippets related to these questions?
thank you in advance:)
1.How can I fetch property values for different profiles on Spring Cloud Kubernetes Config Server?
2. How can I define property values for different profiles on Spring Cloud Kubernetes Config Server?

Spring Cloud Kubernetes Config Server is just a Spring Cloud Config Server with an optional environment repository for Config Maps and Secrets.  Fetching property values and separating property values is done just like any other config server.
https://docs.spring.io/spring-cloud-config/docs/4.0.4/reference/html

i created sample configmap as follows for default,qa, profiles. i can access the prod or qa property values on the kubernetes config server but my app cannot access the prod and kubernetes property files on the config server.
I'm getting an error like following. database connection information is defined in the account k8s profile on the config server.
But my app cannot access any properties on the config server for k8s and prod profiles.
Can you share config client sample for Spring Cloud Kubernetes Config Server?
The reproduce example codes are shared via my Github, check here.

If you hit the /actuator/env endpoint when the k8s and prod profiles are active what is the result?

i created simple project for config client integration.
It seems that it only fetched the default profile on the config server . It did not fetch property values for prod and k8s environments.
The reproduce example codes are shared via my Github, check here.
You can test the property values by typing the following command. Only default property values are loaded
curl --location 'http://localhost:8080/api/contact-info'

Actuator env api response

default profile on Spring Cloud Kubernetes Config Server
.
k8s profile on Spring Cloud Kubernetes Config Server

prod profile on Spring Cloud Kubernetes Config Server

https://github.com/susimsek/spring-cloud-kubernetes-config-server-sample

This returns a 404

i changed project visibility.it is currently public.

Thanks this helps.  There are actually 2 problems that this illustrates.
First the kubernetes config server will return property sources from the same config map with the same property source name.  For example, when the config client requests configuration data without any profiles you will get a response like
{
  "name": "account",
  "profiles": [
    "default"
  ],
  "label": null,
  "version": null,
  "state": null,
  "propertySources": [
    {
      "name": "configmap.account.default",
      "source": {
        "build.version": "3.0",
        "account.message": "Welcome to EazyBank account related local APIs ",
        "account.contactDetails.email"<EMAIL_ADDRESS>        "account.onCallSupport[1]": "(555) 523-1345",
        "account.onCallSupport[0]": "(555) 555-1234",
        "account.contactDetails.name": "John Doe - Developer"
      }
    }
  ]
}

Then the config client will make a second request with all active profiles (in you example k8s and prod) and it will return
{
  "name": "account",
  "profiles": [
    "k8s,prod"
  ],
  "label": null,
  "version": null,
  "state": null,
  "propertySources": [
    {
      "name": "configmap.account.default",
      "source": {
        "build.version": "1.0",
        "account.message": "Welcome to EazyBank account related kubernetes APIs ",
        "account.contactDetails.email"<EMAIL_ADDRESS>        "account.onCallSupport[1]": "(236) 203-0384",
        "account.onCallSupport[0]": "(453) 392-4829",
        "spring.config.activate.on-profile": "k8s",
        "account.contactDetails.name": "Reine Aishwarya - Product Owner"
      }
    }
  ]
}

Since Spring Boot is keeping track of all the property sources that we add to the environment and both property sources get added with the same name the one without any active profiles overrides the k8s and prod profile one.  We can fix this by making the property source names unique (for example by appending any active profiles).
Second problem relates the the response from the config server when there are active profiles.
What we should be returning is all the property sources for all active profiles, so in your example I would expect to see the prod and k8s and default property sources returned and then the client will resolve which one takes priority based on the order they are returned.
This is going to take some digging in order to figure out what the right solution is.

I am going to dig into this, I am already down the rabbit hole

@susimsek  in the ConfigMap why did you use cipher to encrypt the password for database connection and email. My thinking is, shouldn't this be stored in secrets. And if am right how will you load the secrets to the final response when you call the config server client profile.  eg http://localhost:8080/account/prod


cmake setup problem

Hi,
I tried all 3 ways to use QXlsx in my project. After building, I put the QXlsx folder inside the project's root directory.
1. I added 2 lines to CMakeLists.txt
`find_package(Qt6 6.5 REQUIRED COMPONENTS Quick Widgets Sql QXlsxQt6)
target_link_libraries(myApp
PRIVATE Qt6::Quick Qt6::Widgets Qt6::Sql QXlsx::QXlsx
)`
and the result is:
"/CMakeLists.txt:7: błąd: Found package configuration file: .../Qt/6.5.2/gcc_64/lib/cmake/Qt6/Qt6Config.cmake but it set Qt6_FOUND to FALSE so package "Qt6" is considered to be NOT FOUND.  Reason given by package: Failed to find required Qt component "QXlsxQt6". Expected Config file at "/Qt/6.5.2/gcc_64/lib/cmake/Qt6QXlsxQt6/Qt6QXlsxQt6Config.cmake" does NOT exist"
2. I tried also this
`add_subdirectory(QXlsx)
target_link_libraries(myApp
PRIVATE Qt6::Quick Qt6::Widgets Qt6::Sql QXlsx::QXlsx
)`
and the result
"/usr/share/cmake/Modules/CPack.cmake:685: błąd: CPack readme resource file: "/QXlsx/../README.md" could not be found. /usr/share/cmake/Modules/CPack.cmake:691 (cpack_check_file_exists) QXlsx/cmake/modules/CPackConfig.cmake:21 (include) QXlsx/CMakeLists.txt:220 (include)"
3. and the last way
`include(FetchContent)
FetchContent_Declare(
QXlsx
GIT_REPOSITORY https://github.com/QtExcel/QXlsx.git
GIT_TAG sha-of-the-commit
SOURCE_SUBDIR QXlsx
)
FetchContent_MakeAvailable(QXlsx)
target_link_libraries(myApp
PRIVATE Qt6::Quick Qt6::Widgets Qt6::Sql QXlsx::QXlsx
)`
and this time I received
"/qxlsx-subbuild/qxlsx-populate-prefix/tmp/qxlsx-populate-gitclone.cmake:49: błąd: Failed to checkout tag: 'sha-of-the-commit'"
Any help?

Did anybody manage to use QXlsx for cmake? I used it previously for qmake without any problems on the same Linux installation, but I switched to cmake as the newest versions of the Qt Creator don't use qmake.

you need to replace the 'sha-of-the-commit'

you need to replace the 'sha-of-the-commit'

Replace to what?

@dantti , I was ignorant. Thank you for your help. I replaced it with the latest tag, and it works.
Thank you 😊


Update voice-design.md

specified 1st gen as primary

Docs Build status updates of commit c89752c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/voice-design.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


REQUEST: Add support for Nextjs serverBehaviors, imageBehavior

Can there be a way to add API gateway proxy for image lamda functions?

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble