i3dm tiles cannot be loaded into Cesium

Hello,
I am trying to use the tiles generated by i3dm in Cesium JS, but it seems that the tileset.json pyramid is not correct, it is somewhere in the ocean around india while my dataset is in France.
Here is a test in the Cesium Sandcastle:
var viewer = new Cesium.Viewer("cesiumContainer");
var tileset = new Cesium.Cesium3DTileset ({
	url: "https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/tileset.json",
});
viewer.scene.primitives.add(tileset);
viewer.zoomTo(tileset);

tileset.readyPromise.then (function (tileset) {
	console.log("DATASET i3DM loaded");

});

Link for live demo: https://sandcastle.cesium.com/#c=bZHBTgIxEIZfpdnTmkAbww0XIgFvGk3YeOql2w7LaLfddLqLYHx3y4IJKr20M/N/07/TXgXWI+wgsBlzsGNLIOwa/jrkcpnpIV56FxU6CDK7uZOuT1RECwTxN3baJqvyXMw/pZOxC3bKZLaNsaWpEAZ6XnWU2hGNa/B1UBvUXPtGVOZjPDFjo6ISODGNUKEKQOJ8GX8j72Q2ku5rsDGY5KTBAW8DNhixB+LKmPxMXMgO3jelvyxI99M3gDL7l+AbJOBxC47lm87piD6dfgg2PEZ7R94Ct75O41ktysX6oWQ4WT0x65UBcxqRjGeT2SgrKO4tzKVjw7rHpvUhsjSWnHMRoWmtiumVVaffkxlNdASP0kJcooXBnqGZXfkVpq0iSpVNZ+0aDyCzeSGS/h96NImufu4hWLU/yra388dTknNeiBReJ6P3tlLhT+dv
When i generated the tiles, I only had this log:
Exporting i3dm's from public.arbres_instances...
Tile extent: 1000
Set extent: 10000
Potential tiles: 4000 in 40 sets.

The postgis dataset is projected in 4326:
                 st_astext
-------------------------------------------
 POINT(-0.580255753718069 44.8414808634925)
 POINT(-0.58024728846137 44.8413548361828)
 POINT(-0.580249814770417 44.8412793588877)
 POINT(-0.580250249559187 44.8412109737063)
...

Do you know what could be the issue here ?
Thanks!
Fabien

If I compare your sample:
https://bertt.github.io/mapbox_3dtiles_samples/samples/instanced/tileset.json
and what i generated:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/tileset.json
it seems that I don't have a "transform" node in the root node, while you have:
"transform": [
      1.0,
      -0.0,
      0.0,
      0.0,
      0.0,
      1.0,
      0.0,
      0.0,
      0.0,
      0.0,
      1.0,
      0.0,
      518020.625,
      6881447.0,
      0.0,
      1.0
    ]

And 518020.625,  6881447.0 are the EPSG:3857 coordinates for north of Harlem, which is the correct location for your dataset

If I compare your sample:
https://bertt.github.io/mapbox_3dtiles_samples/samples/instanced/tileset.json
and what i generated:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/tileset.json
it seems that I don't have a "transform" node in the root node, while you have:
"transform": [
      1.0,
      -0.0,
      0.0,
      0.0,
      0.0,
      1.0,
      0.0,
      0.0,
      0.0,
      0.0,
      1.0,
      0.0,
      518020.625,
      6881447.0,
      0.0,
      1.0
    ]

And 518020.625,  6881447.0 are the EPSG:3857 coordinates for north of Harlem, which is the correct location for your dataset

Tried it in mapbox-gl-js with your sample page, I get an error:
Mapbox3DTiles.js:53932 ReferenceError: TileSet is not defined
    at ThreeDeeTile.load (Mapbox3DTiles.js:53915)
    at ThreeDeeTile.checkLoad (Mapbox3DTiles.js:54138)

It seems to crash while trying to load a "tileset json child" (Class "TileSet" does not exist which might be another issue)
Thanks,

Tried it in mapbox-gl-js with your sample page, I get an error:
Mapbox3DTiles.js:53932 ReferenceError: TileSet is not defined
    at ThreeDeeTile.load (Mapbox3DTiles.js:53915)
    at ThreeDeeTile.checkLoad (Mapbox3DTiles.js:54138)

It seems to crash while trying to load a "tileset json child" (Class "TileSet" does not exist which might be another issue)
Thanks,

Note: cesium as client is not supported

Note: cesium as client is not supported

I think this tool could be of great interest for the geospatial community since to my knowledge there is currently no open tool available to create 3D Tiles tilesets with i3dm tiles. However, 3D Tiles is a standard format that is client-agnostic, therefore could you elaborate on why the produced tilesets are constrained to mapbox-gl-js ?
In addition, the tileset generated by Fabien cannot be loaded in mapbox-gl-js. Do you have any idea why ?
Thanks.

I think this tool could be of great interest for the geospatial community since to my knowledge there is currently no open tool available to create 3D Tiles tilesets with i3dm tiles. However, 3D Tiles is a standard format that is client-agnostic, therefore could you elaborate on why the produced tilesets are constrained to mapbox-gl-js ?
In addition, the tileset generated by Fabien cannot be loaded in mapbox-gl-js. Do you have any idea why ?
Thanks.

We have build this tool specifically for using i3dm tiles in MapBox GL JS. Technically it should be possible to add Cesium support, but it's not trivial because of mathematics involved.

We have build this tool specifically for using i3dm tiles in MapBox GL JS. Technically it should be possible to add Cesium support, but it's not trivial because of mathematics involved.

Did you follow the getting started guide? https://github.com/Geodan/i3dm.export/blob/main/docs/getting_started.md
Roughly the same procedure should be needed for your table public.arbres_instances

Did you follow the getting started guide? https://github.com/Geodan/i3dm.export/blob/main/docs/getting_started.md
Roughly the same procedure should be needed for your table public.arbres_instances

Yes, I followed the getting started step-by-step to generate public.arbres_instance. The only difference was that I didn't use this line:
delete from traffic_signs_instances where st_x(geom) < 4.5 or st_x(geom)>5.0;

Which must be only for your dataset I guess?
And i serve the file with a simple Apache Web server, not a python server, but that shouldn't change anything, it is just serving static files
Fabien

Yes, I followed the getting started step-by-step to generate public.arbres_instance. The only difference was that I didn't use this line:
delete from traffic_signs_instances where st_x(geom) < 4.5 or st_x(geom)>5.0;

Which must be only for your dataset I guess?
And i serve the file with a simple Apache Web server, not a python server, but that shouldn't change anything, it is just serving static files
Fabien

yes that line is to delete some outliers in the sample data.
Is the generated mapbox tileset available online somewhere?

yes that line is to delete some outliers in the sample data.
Is the generated mapbox tileset available online somewhere?

You can load the data via this URL, if that's what you mean:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/tileset.json
The URL and the files are all on a public server, so you can test locally

You can load the data via this URL, if that's what you mean:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/tileset.json
The URL and the files are all on a public server, so you can test locally

ok, and the mapbox viewer (with the error) is there as well?

ok, and the mapbox viewer (with the error) is there as well?

ah no, sorry, i'll upload it if that helps you, just a minute

ah no, sorry, i'll upload it if that helps you, just a minute

Here is the page with the mapbox-gl viewer and your Mapbox3DTiles.js file from the demo page, to reproduce the issue:
http://dev.business-geografic.com/github-i3dm/index-i3dm.html

Here is the page with the mapbox-gl viewer and your Mapbox3DTiles.js file from the demo page, to reproduce the issue:
http://dev.business-geografic.com/github-i3dm/index-i3dm.html

what happens if you change the reference to tileset.json to relative url (like url: 'tileset.json')

what happens if you change the reference to tileset.json to relative url (like url: 'tileset.json')

moved the files next to tileset.json and changed to a relative URL, no change:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/index-i3dm.html

moved the files next to tileset.json and changed to a relative URL, no change:
https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/index-i3dm.html

looks like the client code is a bit older to support the sub tileset.json... first thing I would try is update the client code to latest version of https://github.com/Geodan/mapbox-3dtiles

looks like the client code is a bit older to support the sub tileset.json... first thing I would try is update the client code to latest version of https://github.com/Geodan/mapbox-3dtiles

Update to latest dist, changed the constructor to let tileslayerTree = new Mapbox3DTiles.Layer( { , added THREE.js but getting error:
Error: i3dm tiles not yet implemented

https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/index-i3dm.html#18.54/44.835528/-0.584797/0/45

Update to latest dist, changed the constructor to let tileslayerTree = new Mapbox3DTiles.Layer( { , added THREE.js but getting error:
Error: i3dm tiles not yet implemented

https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/index-i3dm.html#18.54/44.835528/-0.584797/0/45

can you try develop branch?

can you try develop branch?

The develop branch is not built, there is no dist. Can you please try it on your end?
Thanks,

The develop branch is not built, there is no dist. Can you please try it on your end?
Thanks,

I've updated https://github.com/Geodan/i3dm.export/blob/main/docs/Mapbox3DTiles.js to the code from develop branch. Error is now gone but so far I didn't see any tree show up.

I've updated https://github.com/Geodan/i3dm.export/blob/main/docs/Mapbox3DTiles.js to the code from develop branch. Error is now gone but so far I didn't see any tree show up.

There is now a CORS error (see https://geodan.github.io/i3dm.export/samples/bordeaux/) to retrieve https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/scene.bin . Is this a glTF bin file? Normally the i3dm references a GLB file (or embed the GLB).

There is now a CORS error (see https://geodan.github.io/i3dm.export/samples/bordeaux/) to retrieve https://dev.business-geografic.com/bdx-3d-data/i3dm/arbres/scene.bin . Is this a glTF bin file? Normally the i3dm references a GLB file (or embed the GLB).

Yes, i referenced a GLTF file for this processs (I didn't have a GLB file handy). I added the scene.bin file and now, the tree magically appear on your example !
It still doesn't load properly in Cesium, maybe it is a projection issue (as you read the i3dm file with your own code, I guess you read what you produced). The only i3dm tileset.json I can find to compare (which load fine in Cesium) is:
https://github.com/CesiumGS/3d-tiles-samples/blob/master/tilesets/TilesetWithTreeBillboards/tileset.json
The coordinates are wildly different, maybe you use 3857 while they use 4326 ? I would need to check the spec on that

Yes, i referenced a GLTF file for this processs (I didn't have a GLB file handy). I added the scene.bin file and now, the tree magically appear on your example !
It still doesn't load properly in Cesium, maybe it is a projection issue (as you read the i3dm file with your own code, I guess you read what you produced). The only i3dm tileset.json I can find to compare (which load fine in Cesium) is:
https://github.com/CesiumGS/3d-tiles-samples/blob/master/tilesets/TilesetWithTreeBillboards/tileset.json
The coordinates are wildly different, maybe you use 3857 while they use 4326 ? I would need to check the spec on that

Great I see now the trees too:

Note: creating tileset for Cesium is not supported in this tool. Cesium uses an other system (ECEF - earth centric earth fixed) so all the coordinates are different.

Great I see now the trees too:

Note: creating tileset for Cesium is not supported in this tool. Cesium uses an other system (ECEF - earth centric earth fixed) so all the coordinates are different.

ah thanks for the hint. Do you think it could be difficult to add as an option to your program ? If it seems feasable, I could try a PR.

ah thanks for the hint. Do you think it could be difficult to add as an option to your program ? If it seems feasable, I could try a PR.

Adding Cesium support is not on the roadmap. It's quite a challenge to get it right but feel free to send PR's.

Adding Cesium support is not on the roadmap. It's quite a challenge to get it right but feel free to send PR's.

Thanks a lot for your help, i'll close this issue for now as at the moment, generated tiles are not meant to be loaded in Cesium

Thanks a lot for your help, i'll close this issue for now as at the moment, generated tiles are not meant to be loaded in Cesium

Hi bertt,
I've tried to do the steps based on your guidance. I did the first part (creating sample data set in SQL and run the tool on the command line). The files (tilesets and i3dm files) are stored on local storage. It seems that there is a problem with the tilesets when I try to open the Html file (There is no tree on the map) and there are two errors in the console (. I even tried to run your outcome example from the output folder and saw the same error in the console.
Is this error have anything to do with storing tileset and .i3dm files on the local storage?
Could you please take a look and see if there is anything I can do?

did you start a webserver? btw: can you open new issues for these kind of questions?

Bert is right, you need a local webserver (typically a simple Apache
webserver or one in Python) to host your file, so that your webpage and
i3dm files can be accessed through a URL like http://localhost/xxxx,
otherwise, the web browser will prevent access to anything with the file://
protocol for security reasons.
On Mon, 3 May 2021, 20:21 Bert Temme, @.***> wrote:

did you start a webserver? btw: can you open new issues for these kind of
questions?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Geodan/i3dm.export/issues/15#issuecomment-831441886,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJO3IKR2EZ7FM5UAOVJALTTL3SS7ANCNFSM4WKZGCWA
.

@bertt @fnicollet
Thanks for the tip!

did you start a webserver? btw: can you open new issues for these kind of questions?

Sure! Will do it.
Regards
Morteza


(BUG) The social buttons are absent on the "Invite" page

Steps to reproduce:

Open the "Invite" page on ios/android device
Pay attention to the lack of social buttons

Actual result: The social buttons are absent on the "Invite" page
Expected result: The  social buttons are displayed on the "Invite" page
Note; only ios/android. Buttons are displayed for desktop

Checked on  QA 1.10.1-1
iPhone 8 //13.3.1
Samsung S9 // Android 10

this is not a bug.
on mobile the button says share, clicking on it should open native share
on desktop it should say copy and have the social buttons bellow


(cargo-release) version 0.4.1

Current dependencies on/for this PR:

main

PR #268   👈

PR #269

This comment was auto-generated by Graphite.

Codecov Report

Merging #268 (a26263e) into main (00d702d) will not change coverage.
The diff coverage is n/a.

: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
@@           Coverage Diff           @@
##             main     #268   +/-   ##
=======================================
  Coverage   90.30%   90.30%
=======================================
  Files          45       45
  Lines        3949     3949
=======================================
  Hits         3566     3566
  Misses        383      383

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


process can exit before jax.debug.prints, needs jax.effects_barrier()

Description
This does not print anything:
import jax
import jax.numpy as jnp

def scanner(carry, x):
  jax.debug.print("carry={carry} x={x}", carry=carry, x=x)
  return carry + x, x

out = jax.lax.scan(scanner, jnp.array(0.), jnp.arange(10.))

For comparison, this for-loop-based version gives the expected output:
carry = 0.
for x in jnp.arange(10.):
  jax.debug.print("carry={carry} x={x}", carry=carry, x=x)
  carry = carry + x

carry=0.0 x=0.0
carry=0.0 x=1.0
carry=1.0 x=2.0
carry=3.0 x=3.0
carry=6.0 x=4.0
carry=10.0 x=5.0
carry=15.0 x=6.0
carry=21.0 x=7.0
carry=28.0 x=8.0
carry=36.0 x=9.0

System info (python version, jaxlib version, accelerator, etc.)
>>> import jax; jax.print_environment_info()
jax:    0.4.23
jaxlib: 0.4.23
numpy:  1.26.1
python: 3.11.4 (main, Nov 20 2023, 15:01:59) [GCC 12.3.0]
jax.devices (2 total, 2 local): [cuda(id=0) cuda(id=1)]
process_count: 1

$ nvidia-smi
Sat Feb 17 22:57:36 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 545.23.06              Driver Version: 545.23.06    CUDA Version: 12.3     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA RTX A6000               On  | 00000000:2D:00.0 Off |                  Off |
| 30%   26C    P8              18W / 300W |  36793MiB / 49140MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+
|   1  NVIDIA RTX A6000               On  | 00000000:41:00.0 Off |                  Off |
| 30%   29C    P8              20W / 300W |    271MiB / 49140MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|    0   N/A  N/A     25870      C   ...anifest2-ozEuPuop-py3.11/bin/python    36784MiB |
|    1   N/A  N/A     25870      C   ...anifest2-ozEuPuop-py3.11/bin/python      262MiB |
+---------------------------------------------------------------------------------------+

Hey Jacob, thanks for the question!
I believe it works in side scan (see the tests, which run in CI on CPU/GPU/TPU), but perhaps the process is exiting before you see any prints due to GPU asynchrony. Can you add a call to jax.effects_barrier() at the bottom of your file and try again?

Ah, that was exactly it -- thank you!

I think this may actually be a bug though, in that if we've dispatched a computation that may print, we should block process exit until it's done.

Hi @jbuckman
Looks like this issue is resolved in JAX version 0.4.26. I executed the mentioned code on colab (T4 GPU) with JAX version 0.4.26 without adding the jax.effects_barrier() and it printed the expected output.
import jax
import jax.numpy as jnp

def scanner(carry, x):
  jax.debug.print("carry={carry} x={x}", carry=carry, x=x)
  return carry + x, x

out = jax.lax.scan(scanner, jnp.array(0.), jnp.arange(10.))

Output:
carry=0.0 x=0.0
carry=0.0 x=1.0
carry=1.0 x=2.0
carry=3.0 x=3.0
carry=6.0 x=4.0
carry=10.0 x=5.0
carry=15.0 x=6.0
carry=21.0 x=7.0
carry=28.0 x=8.0
carry=36.0 x=9.0

Please find the gist for reference.
Thank you.

Thanks @rajasekharporeddy ! I really appreciate you checking.


At last a good Twig plugin for Vscode: This is awesome!!!

THANK YOU!!

HAHAHA! Glad you like it! Are you coming from Grav development?

@mblode yep indeed. I saw you too from you theme... (this plugin development make sens now haha)


tests:dma and adc upgrade to new ztest api

config ztest_new_api and update the testcase defines

remove shell interactive test cases, twister has another plan

add sg case support for kinetis and rt10xx platfrom

related: #47002
Signed-off-by: Hake Huang<EMAIL_ADDRESS>

test  pass on frdm_k64f and mimxrt1060_evk

@nashif  I remove the 'test' postfix and add 'test' to test case name back.

@teburd can you help to review?


python3Packages.pathy: 0.6.1 -> 0.10.1

Description of changes
Update pathy to 0.10.1. Fixes python311Packages.pathy build.
textacy and spacy* are waiting on an change to blis at #207070.
Result of nixpkgs-review run on x86_64-linux 1

3 packages failed to build:

python310Packages.spacy
python310Packages.spacy-transformers
python310Packages.textacy

2 packages built:

python310Packages.pathy
python311Packages.pathy

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 213573 run on aarch64-darwin 1

3 packages marked as broken and skipped:

python310Packages.spacy
python310Packages.spacy-transformers
python310Packages.textacy

2 packages built:

python310Packages.pathy
python311Packages.pathy

@ofborg build python311Packages.pathy python310Packages.pathy


Statement delimiter problem with BEGIN; under PostgreSQL

Hi,
With DBeaver 5.0.1, I am facing the problem descripted here: https://github.com/dbeaver/dbeaver/issues/1114
The enhancement was fixed in previous version.
Is it a regression?
Thanks in advance for your help.
Regards,
Cedric

Can't reproduce that (BEGIN statement is correctly executed as a separate statement in PostgreSQL).
Could you provide a bit more details?

OK, I think it is a border effect with this issue: https://github.com/dbeaver/dbeaver/issues/1823
It is because I have quotes and $$ just before my BEGIN statement.
Sorry for the wrong issue.

Well, if you still experience some issue with pgSQL processing - please post example of problematic query.


CertificatePolicy accepts a list of certificate LifetimeActions but specifying more than one fails when creating a new cert.

Describe the bug
The CertificatePolicy type in the Microsoft.Azure.Management.KeyVault.Models namespace accepts a list of certificate LifetimeActions. I figured this implied you could set multiple actiions but doing so causes an error when trying to create the certificate.
Here I think: https://github.com/Azure/azure-sdk-for-net/blob/cdf82949f13ea1e432d2bac6e55ac98899c0ab1c/sdk/keyvault/Azure.Security.KeyVault.Certificates/src/CertificatePolicy.cs#L230
Expected behavior
Should probably not accept a list of LifeTimeActions since having more than one causes an error. A little misleading
Actual behavior (include Exception or Stack Trace)
{"error":{"code":"BadParameter","message":"At most one action is allowed"}}

   at Microsoft.Azure.KeyVault.KeyVaultClient.<CreateCertificateWithHttpMessagesAsync>d__85.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at Microsoft.Azure.KeyVault.KeyVaultClientExtensions.<CreateCertificateAsync>d__68.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Define 2 certificate lifetime actions and add them to the certificate policy
Try to create the certificate

Environment:

<PackageReference Include="Microsoft.Azure.Management.KeyVault" Version="2.4.3" />

Thanks!

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Microsoft.Azure.KeyVault is deprecated. We recommend everyone upgrade to Azure.Security.KeyVault.Certificates for a number of good reasons.
However, it's still a list of actions (and is the code you pointed to): https://docs.microsoft.com/dotnet/api/azure.security.keyvault.certificates.certificatepolicy.lifetimeactions?view=azure-dotnet#Azure_Security_KeyVault_Certificates_CertificatePolicy_LifetimeActions
This is what the service specifies, and changing it would be a breaking change. It's possible at some point multiple values may be accepted. We have guidelines to not validate service parameters, which could become a blocking issue when the service updates it's behavior.
We could, perhaps, update the documentation for the property. Would that discoverable enough and help?

Yeah, I think updating the docs here would be the best option given the breaking change constraint. I probably would have avoided the error with a warning in the docs

@KrzysztofCwalina, @schaabs from a guidelines perspective, do you see any problem with me adding a current constraint to documentation only? There's a possibility it may some day be inaccurate, but at least it wouldn't cause any blocking behavior.


various customization to buildpack steps

includes various customization on build steps and integrated upstream changes

tl;dr (too long, didn't review)


Glimpse EF6 - Support for raw SQL statements on DbContext

When I do raw SQL query in Entity Framework, the query is not listed or counted in Glimpse.
Example code:
context.Database.SqlQuery<ResultAggregate>(@"SELECT * FROM ...");

All the normal "LINQ to entities" queries show up just fine.
Is there some way how to configure Glimpse or a workaround?
I am using Glimpse.EF<IP_ADDRESS>, Glimpse.Ado.1.7.3, Glimpse.1.8.6.

I noticed that after I started using DbProviderFactory to create connections instead of creating connections by constructing SqlConnection object, the queries appeared.
For anyone experiencing the same problem, try using DbProviderFactory to construct your SqlConnection.


Second task list doesn't get rendered if including hide task count

Hello. Thank you for creating such an awesome useful plugin. It makes GTD system possible within Obsidian.
Expected Behavior
Two task lists should get rendered, i.e., checkbox appears on both lists when I create a Daily Note:

Current Behavior
Only the first task list get rendered. Second task list is rendered as editable text.

The following is in my daily note template:
### Overdue
```tasks
not done
due before {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

### Due today
```tasks
not done
due on {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

Below is what's rendered when I create the daily note using the Calendar plugin:

As you can see, there's no checkbox in the second task list. It's rendered as a generic editable text. This bug occurs in both editing mode and reading mode of live preview mode.
However, both task lists get rendered correctly if I include some texts among them:
### Overdue
```tasks
not done
due before {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

text here
text here
text here
text here

### Due today
```tasks
not done
due on {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

I need to add at least a few lines of text among them. Simply one line of text and the bug still occurs.
I found that this bug doesn't occur if I remove hide task count. So the following works:
### Overdue
```tasks
not done
due before {{date+0d:YYYY-MM-DD}}
short mode
```

### Due today
```tasks
not done
due on {{date+0d:YYYY-MM-DD}}
short mode
```

Context (Environment)

Obsidian version: v0.14.6
Tasks version: 1.51
[x] I have tried it with all other plugins disabled and the error still occurs

Thank you again creating such as awesome plugin. This bug is not urgent fix for me because I don't mind removing hide task count. But still reporting in case it's important for others.

Figured out the problem. There's a problem with how my get dataview rendered initially:
https://user-images.githubusercontent.com/57322459/166174552-80f14819-77ae-4ca6-998a-66a34a92c327.mp4
### Yesterday's Wins
```dataview
LIST WITHOUT ID Dwi
FROM "Journal"
WHERE file.name = "<%tp.date.now("YYYY-MM-DD", -1)%>"
FLATTEN Dwi
```

### Overdue
```tasks
not done
due before {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

### Due today
```tasks
not done
due on {{date+0d:YYYY-MM-DD}}
hide task count
short mode
```

Once I removed dataview above, all the tasks are rendered correctly. I guess when dataview throws an error (even though it's queried correctly later on), the task list code below has problems.

Glad you figured it out. Thank you for the follow-up.


Visual bug when scanning

The text is not left aligned with the start of the progress bar. As a result some of the text is cut-off.
Tested in Opera and Edge browser.

Removing the right: .5em seems to fix it.

Related to #119

Related to #119

I just found the same bug in the navigation bar.

I just found the same bug in the navigation bar.

It's seems the normal work of the progressbar
https://react.semantic-ui.com/modules/progress/#content-progress


Epson receipt printing regression after update 1.27.4 > 1.28.15 (Ubuntu 20.04 > 22.04) from pdftopdf resizing output

Hi, I updated Ubuntu 20.04 to 22.04 thus updating (cups-filters 1.27.4 > 1.28.15) and receipt printing has 'broken';
To accomodate long receipts, the printer (Epson TM-T20 w/epsonsimplecups driver) has the default paper size set to 80mm x 2000mm. i.e. most of the time a standard receipt is perhaps 80x56mm, but could easily be ten times longer.
The receipt is generated as a .pdf from the backend system, and passes though the first cups filter pdftopdf (1.27.4), with no options set, unchanged, remaining at this size, and prints fine on to a 80x56mm piece of receipt paper which is auto cut by the printer.
This seems logical, no options set, don't change anything.
The newer pdftopdf (1.28.15) in Ubuntu 22.04 , without any options set, resizes the ouput to the full paper size and centres - thus the output creates a 2m long page and puts the receipt in the middle, which isn't ideal.
The changes have happened in /filter/pdftopdf/pdftopdf_processor.cc , probably refactoring its behaviour around ~1.28.13.
Is it possible to 'fix' the logic to revert to the previous behaviour, or at least have an option so no resizing, scaling or cropping is done, perhaps print-scaling=none ?

On further investigation it seems to way it was recently refactored broke it, starting with these uncommented lines
from 1.27 getting re-enabled in 1.28.
    // TODO? fitplot also without xobject?
    /*
      param.nup.width=param.page.width;
      param.nup.height=param.page.height;
    */

Anywho, after trying various things the simplest fix seems to be just add a new option "print-scaling=none" to use the input files existing width/height and not resizing to the page size.
3x patches attached zipped (it wouldn't let me upload .patch)
pdftopdf_patches_receipt_print_fix.zip


[Hooks] useDrop receives a DragDropMonitorImpl, not DropTargetMonitor

According to the typings, the useDrop hook spec method receives a DropTargetMonitor instance, but it actually currently receives a DragDropMonitorImpl instance. For example, you cannot call isOver() on it but the typings suggest you can.
I don't know if this is a fault of the code or the typings?
Conversely, the useDrag hook spec methods appears to receive a DragSourceMonitor instance.

Good catch! Thanks


large difference between the trained-out pth model inference result  and onnx  inference result

encoding: utf-8
"""
@author:  sherlock
@contact<EMAIL_ADDRESS>"""
import sys
import torch
sys.path.append('../..')
sys.path.append('.')
from fastreid.config import get_cfg
from fastreid.engine import default_argument_parser, default_setup
from fastreid.modeling import build_model
#from fastreid.export.tensorflow_export import export_tf_reid_model
from fastreid.export.tf_modeling import TfMetaArch
import onnx
from torch.backends import cudnn
import torch.nn.functional as F
import io
from fastreid.utils.checkpoint import Checkpointer
import numpy as np
import onnxruntime
import cv2
cudnn.benchmark = True
def setup(args):
"""
Create configs and perform basic setups.
"""
cfg = get_cfg()
cfg.merge_from_file(args.config_file)
cfg.merge_from_list(args.opts)
cfg.freeze()
default_setup(cfg, args)
return cfg
def to_numpy(tensor):
return tensor.detach().cpu().numpy() if tensor.requires_grad else tensor.cpu().numpy()
if name == "main":
args = default_argument_parser().parse_args()
print("Command Line Args:", args)
cfg1 = setup(args)
cfg = cfg1.clone()
cfg.defrost()
cfg.MODEL.BACKBONE.PRETRAIN = False
model = build_model(cfg)
#model.load_state_dict( torch.load('logs/Vechile1M/bagtricks_R50-ibn_4gpu/model_0211199.pth'))
Checkpointer(model).load('logs/Vechile1M/bagtricks_R50-ibn_4gpu/model_0211199.pth')

model.cuda()
model.eval()
#print (model)
layer_names = {}
for name, layer in model.named_modules():
    layer_names[layer] = name
print("torch ops name:", layer_names)
dummy_inputs = torch.randn(1, 3, 256, 256).cuda()
torch.onnx.export(model,  # model being run
                  {'images': dummy_inputs},  # model input (or a tuple for multiple inputs)
                  "reid_test.onnx",  # where to save the model (can be a file or file-like object)
export_params=True,  # store the trained parameter weights inside the model file
)
onnx_model = onnx.load("reid_test.onnx")
onnx.checker.check_model(onnx_model)

original_image = cv2.imread("/hdata/nfs_share/Vehicle-1M/Vehicle-1M/image/149/497612.bmp")

# the model expects RGB inputs
original_image = original_image[:, :, ::-1]
# Apply pre-processing to image.
image = cv2.resize(original_image, (256, 256), interpolation=cv2.INTER_CUBIC)
image = torch.as_tensor(image.astype("float32").transpose(2, 0, 1))[None]
img = image.cuda()

with torch.no_grad():
    torch_out1 = model.forward({'images': img})
    torch_out = F.normalize(torch_out1)

ort_session = onnxruntime.InferenceSession("reid_test.onnx")

# compute ONNX Runtime output prediction
ort_inputs = {ort_session.get_inputs()[0].name: to_numpy(img)}
ort_outs = ort_session.run(None, ort_inputs)
img_out_y = ort_outs[0]
norm = np.linalg.norm(img_out_y, ord=2, axis=1, keepdims=True)
print ('norm:', norm)
feat = img_out_y/norm
np.savetxt('th_re.txt', to_numpy(torch_out).T)
np.savetxt('onnx_re.txt', feat.T)
# compare ONNX Runtime and PyTorch results
np.testing.assert_allclose(to_numpy(torch_out), feat, rtol=1e-03, atol=1e-05)

print("Exported model has been tested with ONNXRuntime, and the result looks good!")

We are trying to support DDP training these days, and ONNX-inference will be supported after we release v0.2 next week.
Please give us some time.
PRs are welcome!

I have solved the problem. I commented the preprocess_image function, and then no difference between onnx and pytorch inference!  So the preprocess such as subtract mean or resize should be both before onnx and pytorch. And then reference only do deep network inference, but not do preprocess

Good job! In the caffe deployment, we just put the preprocess before the network forward.
You can check in caffe deploy.

Thanks! Now I meet a new problem when deploy onnx to tvm.
It reports "TypeError: clip() missing 1 required positional argument: 'a_max'"
I guess The error in GeneralizedMeanPooling layer! I notice in the forward process
x = x.clamp(min=self.eps).pow(self.p).
My question is what the defaut max argment is?  For safety should we set max in this clamp fuction?

the default value for torch.clamp is inf, maybe I should set a number for this.

I have solved the problem. I commented the preprocess_image function, and then no difference between onnx and pytorch inference! So the preprocess such as subtract mean or resize should be both before onnx and pytorch. And then reference only do deep network inference, but not do preprocess

hello,could yu please tell me the meaning of " reference only do deep network inference, but not do preprocess",what should i do about this?

For those stumbling on upon this issue in the future, I want to explain my mistake since it was different from @HouBiaoLiu. It took me too long to notice this subtlety, so I hope my explanation saves others some time.
In the ONNX export script, the weights from my training were not being loaded since by default MODEL.WEIGHTS = '' in most of the config YAMLs. Not only that, but the export script explicitly sets MODEL.BACKBONE.PRETRAINED = False which, if you're using a torchvision model trained on ImageNet, means you won't even be exporting with those weights loaded - you'll just get a randomly initialized model! To get the same inference accuracy as you had during training, you'll need to ensure your config file specifies the path to the model_final.pth file that was saved at the end of training

the default value for torch.clamp is inf, maybe I should set a number for this.

Excuse me, if I want set a number for this, What number should I set？thank you


"Clear Completed" button not working in live demo

Works for me on latest versions of Firefox and Google Chrome.


Consistency needed in packaging

After the major refactoring, the top level packages, eg. strata, grid, are presenting differently. Some hide the sub packages (preferred) with full doc strings in the wrapper methods. Others put the emphasis on the sub packages. This difference shows up in readthedocs. Consistency across resqpy is needed.

For the Grid module, move the docstrings from the refactored functions to the top level class methods so that they are presented at the top level on readthedocs. Make the submodule names private to discourage their use outside the Grid class.


Allow multiline TextField to expand vertically to fit container

This is the only way I have been able to achieve this and its pretty hacky. Would be nice if TextFields were able to expand vertically to fit the container.
Container(
  margin: EdgeInsets.all(8.0),
  // hack textfield height
  padding: EdgeInsets.only(bottom: 40.0),
  child: TextField(
    maxLines: 99,
    decoration: InputDecoration(
        hintText: "Comment!",
        border: OutlineInputBorder(),
    ),
  ),
),

What part do you find hacky?
How would you prefer it to be?
Please add the output of flutter doctor -v.

Having to put it in a container with an arbitrary 40px padding and force it to spill by giving it a manliness of 99.
Due to the speed of flutter development my flutter doctor -v has changed considerably since I made this post. Sorry about that.

Thanks for your update!
I don't think flutter doctor output is essential for this feature request.
I just try to annoy people with requesting it until everyone adds it automatically everywhere ;-)
Actually flutter doctor -v output provides some context that is usually helpful and often essential.

The user expectation is probably that you would be able to do this:
Column(
  children: <Widget>[
    AFixedSizeWidget(),
    Expanded(
      child: TextField(),
    )
  ],
)

Possibly based on implementation, you may also need maxLines: null or some other value. The above is how you lay out every other set of widgets in a column, it's a surprise that the TextField handles its own size and doesn't expand.

@Will-W That was my expectation/hope as well.  So now sample size is you and me :).  When doing native Android development, it's rather easy to make the textfield/textarea fill the parent dynamically and auto adjust when the keyboard opens and closes and when the device orientation changes, instead of giving it a fixed maxLines.

Closing this with https://github.com/flutter/flutter/pull/27205.
The expands parameter on TextField will do what's desired in this issue and allow the TextField to be sized based on its parent.  So @Will-W's example could be achieved like this now:
Column(children: [
  Container(height: 100.0),
  Expanded(
    child: TextField(expands: true),
  )
])

To adjust the width, you could wrap your TextField with a Container widget, like so:
new Container(
width: 100.0,
child: new TextField()
)
Find Full Code At Here

Already Defined at https://github.com/flutter/flutter/issues/27838#issuecomment-541342145

To get it like a WhatsApp textbox which increases height up to a few lines then stop,
ConstrainedBox(
        constraints: BoxConstraints(
          maxHeight: 100 //put here the max height to which you need to resize the textbox
        ),
child: TextField(
maxLines: null, ),
)

return TextFormField(
      keyboardType: TextInputType.multiline,
      textInputAction: TextInputAction.newline,
      maxLines: 16,
      maxLength: 128,
      maxLengthEnforced: true,
      initialValue: _content,
      decoration: InputDecoration(
        labelText: 'Content',
        hintText: 'Enter contetn',
        alignLabelWithHint: true,
        labelStyle: TextStyle(fontSize: 18.0, fontWeight: FontWeight.w500),
      ),
      validator: (value) =>
          value.isNotEmpty ? null : 'content can\'t be empty',
      style: TextStyle(fontSize: 20.0, color: Colors.white),
      onSaved: (content) => _content = content,
    );
  }

Closing this with #27205.
The expands parameter on TextField will do what's desired in this issue and allow the TextField to be sized based on its parent. So @Will-W's example could be achieved like this now:
Column(children: [
  Container(height: 100.0),
  Expanded(
    child: TextField(expands: true),
  )
])

I try it, I got error

I try it, I got error

Please provide a code snippet for reproduction. Thanks!

Closing this with #27205.
The expands parameter on TextField will do what's desired in this issue and allow the TextField to be sized based on its parent. So @Will-W's example could be achieved like this now:
Column(children: [
  Container(height: 100.0),
  Expanded(
    child: TextField(expands: true),
  )
])

I try it, I got error

I suspect the error you have got is assert. When you use expanded = true both minLines and maxLines have to be null. All that said, the correct code snippet should be:
TextField(expands: true, minLines: null, maxLines: null)

Sample worked codes:
Container(
            child: new TextField (
              keyboardType: TextInputType.multiline,
              minLines: 1,
              maxLines: 10,
              decoration: new InputDecoration(
                  border: new OutlineInputBorder(
                    borderRadius: const BorderRadius.all(
                      const Radius.circular(10.0),
                    ),
                  ),
                  filled: true,
                  hintStyle: new TextStyle(color: Colors.grey[800]),
                  hintText: "Type in your text",
                  fillColor: Colors.white70),
            ),
            padding: new EdgeInsets.fromLTRB(16.0, 16.0, 16.0, 16.0),
          );

TextField(
    maxLines: null,
    keyboardType: TextInputType.multiline,
)

Links: 1, 2.


Property Not Required

I have found an issue with the library which is related to the property name:
 function httpSuccessCallbackGen(str) {
          return function(responseData, status, headers, config) {
            if (!status && !headers && !config) {
              responseData = responseData.data;
            }
            scope.searching = false;
            processResults(extractValue(responseFormatter(responseData), scope.remoteUrlDataField), str);
          };
        }

In the above code for the library the responseData.data property should be optional as the remote api might not have the data property.

It's expecting a angular's http promise. So, it should have data property.

If someone is using it with Restangular then he will surely have a hard time.

Sorry I can't support all possible cases.

@ghiden I understand. Thanks for the response


Improving WordPress support

The Issue
https://github.com/ddev/ddev-browsersync/issues/51
How This PR Solves The Issue
This PR aims to replicate a similar approach used by ddev-redis to modify the project's CMS config.
Manual Testing Instructions

ddev start a test project
Install the add on fork ddev get ~/folder-of-fork/ddev-browsersync
Start browsersync with ddev browsersync
Open browsersync in your browser ddev launch :3000

WordPress should recognise the URL and load the page at port 3000 correctly.
Related Issue Link(s)
https://github.com/ddev/ddev-browsersync/issues/51

@rfay Do you have any advice on how the add-on install process executes and how this relates to if wp-config-ddev.php exists?
I'm using awk to find /** WP_HOME URL */ in wp-config-ddev.php here:

Running ddev get using my fork I'm getting this error, as if it can't find the ddev wp config:

Spotted the issue, I need to add ${DDEV_APPROOT} to all the file references

Thank you for the PR. Please click "Ready for review" when you feel its good to go.

Doing some testing on a project I've never used ddev-browsersync on and it did install and "just work™".

Marking this PR as ready for review.
Thanks!

The project I tested with didn't have #ddev-generated: in the wp-config-ddev.php file (for other reasons).
wp-config-ddev.php get's reset on each ddev restart if it is being controlled by ddev.
@rfay Is there any post_start hook for add-ons? I'm guessing not based on this: https://ddev.readthedocs.io/en/stable/users/extend/additional-services/#creating-an-additional-service-for-ddev-get
The reason I put the include in wp-config-ddev.php is because that's where the WP_HOME and WP_SITEURL are loading from, so it felt most relevant to put the browsersync code close to that incase you're debugging a site URL issue.
Maybe I need to refactor things to add the include early on in the wp-config.php file.

Maybe this placement is preferable?

Thank you for the PR. I gave it a quick run through but not knowing WP that well, it made it hard to tell what works, and what doesn't.
Test

Install addon
Start browsersync.

ddev browsersync

Visit browsersync proxy page as shown in terminal (Eg. https://wp-vanilla.ddev.site:3000)
Visit login page: https://wp-vanilla.ddev.site:3000/wp-admin

This takes me to: https://wp-vanilla.ddev.site/wp-login.php?redirect_to=https%3A%2F%2Flocalhost%2Fwp-admin%2F&reauth=1
2 things of note here:

This base URL no longer contains 3000 port.
The redirect to URL does not contain 3000 port.

So WP seems to be losing the port number. Am I missing something?

Is your wp-vanilla project a repo I could see? Particularly interested in wp-config.php, wp-config-ddev.php and any other config files you may have.
Also unsure where you’re getting localhost from in the redirect_to value!

Thank you for the quick reply @graham73may .
I just just the site again and I'm not getting localhost in the redirect URl (not sure how to replicate it).
Anyway, here is the repo I was using to test. It's a WP site following DDEV quickstart guide, with your PR added browsersync branch.

Hi @tyler36 thanks for taking the time to test it!
I've been testing using the DDEV WP quickstart and it's working mostly as I'd expect.
These are the steps I followed, all working as expected focusing on the frontend:
git clone

ddev start

ddev wp core download

ddev wp core install --url='$DDEV_PRIMARY_URL' --title='wp-vanilla' --admin_user=admin<EMAIL_ADDRESS>--prompt=admin_password

ddev launch

# => I see a working WordPress website

# Now to Install ddev-browsersync, obviously use your own folder or .gz for my PR here!
ddev get ~/Repos/ddev-browsersync

# Start browsersync to activate port 3000
ddev browsersync

# Launch on port 3000 (This is where the ddev quickstart tries to take you to the admin area. I'm going to the frontend).
ddev launch :3000

Now, looking at the admin area specifically - I've not dug too deep but it seems when WordPress redirects you as an unauthenticated user from /wp-admin/ to /wp-login.php it strips the port.
This might just be my opinion, but if you're using browsersync on WordPress, you're probably developing a frontend theme... So I'm not sure how much I'm willing to go down this rabbit hole. If you're customizing the backend in WordPress I think it's fair enough to expect them to have to login and then go to port 3000...
Some more notes on testing ddev launch :3000/wp-admin:
# Strips the port
ddev launch :3000 /wp-admin

# Even if I try this it strips the port:
ddev launch https://wp-vanilla.ddev.site:3000/wp-admin

However, if you don't use the /wp-admin URL shortcut to reach the login page (which triggers a redirect) you can load the login form on port 3000 using this, going directly to the login form:
ddev launch :3000/wp-login.php

@graham73may
Thank you for the feedback and analysis. I remember people complaining aboout that type of issue with Drupal (or maybe Laravel?)  a long time ago.
Things to do:

fix resolve conflicts
confirm uninstall works

If the PR works as intended and we can resolve those last couple of things, I'm happy to merge.

1 - Conflicts resolved and pushed to PR
:3000 still working:

2 - Uninstall was removing scripts but was leaving a couple of things, I've added a remove-wordpress-settings.sh file to do a better job of tidying up. (mainly inside wp-config.php).
After installing:

After removing (puts it back how it was):

Thank you for your patience. I didn't get a chance to look at this till today.

[X] Confirm uninstall works

$ ddev get https://github.com/graham73may/ddev-browsersync/tarball/feature_wp-config
...
$ ddev get --remove https://github.com/graham73may/ddev-browsersync/tarball/feature_wp-config
Removing wp-config-ddev-browsersync.php from: /home/user13/code/framework/wp-vanilla/wp-config.php

👍 Remove browsersync settings for WordPress if applicable
Removed add-on https://github.com/graham73may/ddev-browsersync/tarball/feature_wp-config

[X] Merge conflicts resolved.

I just kicked off the test pipeline. Will merge and release when they pass.

Thank you for the contribution!


Error Resin Calculation

If user setting AR and WL in setting section with number greater than the actual current max AR and WL (60 for AR and 8 for WL) resin calculation in to do error

After setting to 1000 AR and 1000 WL

Actually i have already solving it with adding Max AR and WL, later i'll add PR after you review https://github.com/MadeBaruna/paimon-moe/pull/16


cmd/compile: link fails when function containing linknamed symbol is inlined (-l=4)

The net/http tests fail with -l=4:
$ go version
go version devel +5c9035a Fri Dec 2 19:19:03 2016 +0000 linux/amd64

$ cd $GOROOT/src/net/http
$ go test -a -gcflags '-l=4' -v .
# testmain
net/http.(*http2Transport).RoundTripOpt: relocation target net.byteIndex not defined
net/http.(*http2Transport).RoundTripOpt: relocation target net.byteIndex not defined
net/http.(*http2ClientConn).encodeHeaders: relocation target net.byteIndex not defined
net/http.(*http2ClientConn).encodeHeaders: relocation target net.byteIndex not defined
net/http.cleanHost: relocation target net.byteIndex not defined
net/http.cleanHost: relocation target net.byteIndex not defined
...
net/http.cleanHost: undefined: "net.byteIndex"
/home/david/go/pkg/tool/linux_amd64/link: too many errors
FAIL	net/http [build failed]

Let's look at the cleanHost function in net/http/request.go to understand what's happening.  The cleanHost function calls net.JoinHostPort, which is inlined by the compiler.  The body of net.JoinHostPort has a call to net.byteIndex which is linknamed in net/parse.go:
//go:linkname byteIndex strings.IndexByte
func byteIndex(s string, c byte) int

The export data used to inline net.JoinHostPort does not include linknames, so the compiler can't find net.byteIndex when it appears in the body of cleanHost after inlining.
One solution is to include linknames in export data.

This is too late for Go 1.8.


[Bug]: [help wanted] MoE + TP + custom allreduce bug

Your current environment
main branch, H100
Model Input Dumps
No response
🐛 Describe the bug
when I try this simple command vllm serve allenai/OLMoE-1B-7B-0924 -tp 2 on the main branch, it hits an error:
Failed: Cuda error /workspace/csrc/custom_all_reduce.cuh:336 'invalid argument'
removing -tp works, and disabling custom allreduce also works.
not sure what's happening. if anyone is familiar with the moe code, please help.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

@cedonley thanks for the great information! can you try to confirm which commit leads to this bug? we have all the wheels from the main branch that's ready to directly install, see https://docs.vllm.ai/en/latest/getting_started/installation.html#install-the-latest-code .

@youkaichao Sure. I was just able to confirm the following results:
commit# 3cb07a36a20f9af11346650559470d685e9dc711 (after #9588) - FAILS
commit# 8549c82660cfa59a13cccd622f8afcc29cbd4281 (just before) - SUCCESS
The invalid argument error is coming from a call to cudaIpcOpenMemHandle. Did some various internet searching to see what might have changed here between CUDA 12.1 and 12.4 (which is the other factor introduced in #9588) and coming up empty at the moment.

might be caused by the new expandable segment:
https://github.com/pytorch/pytorch/blob/18ad44e830e6f885ee81c9ccf65f67c2fa05b9eb/c10/cuda/CUDACachingAllocator.cpp#L348

it looks expandable_segments is False by default.
we need to figure out the memory allocation strategy in pytorch 2.5 .
if you have time, you can also compile yourself, and call cuda APIs to get the pointer's attributes, that should tell you more information.

Did a little more analysis last night and it's clear that the flow should be working. Here's what I see:
custom_all_reduce.py

allocates the IPC metadata using PyTorch storage (and I see it come back)
handles/offsets look fine, rank/world size is correct
it then passes this all as parameters to init_custom_ar
the error occurs before it registers the buffers and returns

custom_all_reduce.cu

init_custom_ar  - sees the correct parameters, makes a copy of the handles allocated by PyTorch in the python code, and then passes this into the CustomAllreduce function defined in custom_all_reduce.cuh

custom_all_reduce.cuh

CustomAllreduce - Immediately tries to open the handles, which results in the error we see

Not finding any changes in recent PyTorch releases related to how storage is allocated. The metadata that is passed is allocated on python with essentially (simplified):
meta = torch.zeros(ops.meta_size() + max_size,
                                dtype=torch.uint8,
                                device=self.device)
data = meta.untyped_storage()._share_cuda_()
shard_data = (
            data[1],  # ipc handle to base ptr
            data[3],  # offset of base ptr
        )
handles, offsets = self._gather_ipc_meta(shard_data)

I see both the handles and offsets come back and don't see any issues with them.
I tried changing the way processes were spawned, etc... to see if perhaps there was a change to multiprocessing that created a side effect, etc... I'm stumped... Someone with stronger CUDA debugging skills is going to need to have a look at this.

thanks for the investigation! let me ask some cuda experts.

I think https://github.com/pytorch/pytorch/pull/130888 might be the culprit .

okay, they changed the binary format of the ipc handle in https://github.com/pytorch/pytorch/pull/130890


feat: link to current song directly

Changes getArtwork to getMetadata, which uses the tools.applemediaservices.com API instead. This response contains a link to the queried song, which is meant for sharing on social media, etc.
Also changes behaviour where if metadata isn't available, the Discord buttons won't show at all, since the URL would be empty.

I also noticed that the fallback image doesn't appear?

It seems like the new AppID introduced in 908d5021d7d29f8961dda70a398ee5998394f852 doesn't have a resource named "applemusic", nor "play" for the small image.
https://github.com/caarlos0/discord-applemusic-rich-presence/blob/908d5021d7d29f8961dda70a398ee5998394f852/main.go#L294-L295

I also noticed that the fallback image doesn't appear?
It seems like the new AppID introduced in 908d502 doesn't have a resource named "applemusic", nor "play" for the small image.
https://github.com/caarlos0/discord-applemusic-rich-presence/blob/908d5021d7d29f8961dda70a398ee5998394f852/main.go#L294-L295

ah, might be, will fix!

thanks for the pr! looks good!

It seems like the new AppID introduced in 908d502 doesn't have a resource named "applemusic", nor "play" for the small image.

should be fixed

actually, it never saves the images... will revert that commit ¯_(ツ)_/¯

Hmm which images? I assumed you were talking about the album art, but then I noticed that you reverted the own App ID.
If you mean the Assets in RP, they have quite aggressive caching, but it does work. I recently updated my MPD fork to use my AppID as well, and it took a while for the images to process when deleting and uploading to rename.

Quickly following up on the topic of caching, wouldn't it be better if it used artist album instead of artist album song for the cache key?
https://github.com/caarlos0/discord-applemusic-rich-presence/blob/930eb8e649a679ac18b4fd33b17168a100094087/main.go#L218-L221

Hmm which images? I assumed you were talking about the album art, but then I noticed that you reverted the own App ID.
If you mean the Assets in RP, they have quite aggressive caching, but it does work. I recently updated my MPD fork to use my AppID as well, and it took a while for the images to process when deleting and uploading to rename.

ahh, that might be it, will try again later and revert the revert if it works

Quickly following up on the topic of caching, wouldn't it be better if it used artist album instead of artist album song for the cache key?
https://github.com/caarlos0/discord-applemusic-rich-presence/blob/930eb8e649a679ac18b4fd33b17168a100094087/main.go#L218-L221

when I tested it seemed to yell the same results, I leave the song too "just in case", as it doesn't seem to hurt


Gaia Server Errors notification

This is coming in from a user in support who has been sent a link to this issue to add to. I will add anything else in here that they send to support.

hi, i have an unsolvable issue with gaia and putting a file. i am locally playing around with blockstack and want to write a file to the storage. so far, so good. the strange thing is, that after logging out and logging in again i am getting "server errors". this happens with multiple IDs & browsers, so it is not restricted to that.

I am using that code to put the file

return this.userSession.putFile(STORAGE_FILE, JSON.stringify(timeline))

this is the request one from the first session and in this multiple calls succeed:

https://pastebin.com/UBEHF8rD with the response https://pastebin.com/Lam7azb7

and this the second one https://pastebin.com/W0zEme8P resulting in this error >https://pastebin.com/AFLg9UkC

between this two request was a signout and signin

These is a valid bug I will assign this to @reedrosenbluth


Miscellaneous Browser Tasks

Fees + Payment

[x] Switch networks (mainnet / polygon) when deposit / (send + withdraw) Priority 0 - Ze
[x] Collect fees in Matic when sending Priority 0 - Ze
[ ] Coinbase wallet support Priority 1

Storage + Security

[x] Encryption of secrets in the browser. Priority 0
[ ] Export/Reimport IndexedDB button. Priority 3
[x] Anonymity set Priority 1

Usability and QoL

[x] Decimal Handling in browser Priority 0 #539
[x] Exchange rates and dollar values Priority 1

Infrastructure

[x] Stream and store circuits and proving keys Priority 0 #469
[ ] Web workers for multi-threading. Priority 1
[ ] Off load proof computations to web worker. Priority 1
[ ] Rollback event handler. Priority 0 #523
[ ] Integration in Polygon's website Priority -1
[ ] Websocket keep-alive in wallet Priority 0

Refactor

[ ] Remove LOCAL_PROPOSER for wallet Priority 4

Closing, any open tasks will be migrated to V2


[FEATURE] Don't make registry calls when content has completed fetching

Description
SOCI doesn't use any sort of credential rotation system, so in systems such as Kubernetes, we can encounter issues where a user can attempt to run tasks after the credential has expired. stargz gets around this by only calling the registry if the fetched size is under the expected layer size: https://github.com/containerd/stargz-snapshotter/pull/1584. We can do the same here.
Ultimately, this is a workaround, and we need a better credentials mechanism, but this should be a simple fix that should help for now.
Describe the solution you'd like
No response
Describe any alternative solutions/features you've considered
No response
Any additional context or information about the feature request
There's a separate issue here which is that we don't know exactly when the background fetcher has concluded fetching all of its data. I think if we can implement that, this fix can be a bit cleaner.

Does this mean we are not checking the cache before going to the registry? If so, what is the point of the cache?

AFAIK, before this change, the cache still did its function, but the call order meant that we still made a registry call regardless. The cache is to stop us from fetching data we don't already have, and it seems to do its job


feat: DIA-1415: Add LabelConfig->ResponseModel generator

Added datamodel-code-generator dependency https://koxudaxi.github.io/ (converter from json schema to pydantic models)

Follow Merge downstream workflow has been failed.

Workflow run


Add missing changelog entries

Usually, these changelog entries should have been added with their respective PRs, but as this hasn't been the case over the last year, I have now added all missing changelog entries.
If I missed something here or if I should change something, please let me know.

I will release once this is merged. Are you okay with this PR @knothed, @StevenMagdy?
And another question: I think, the new release should be called 3.2 to stay in sync with Apple's versioning. Are you okay with that?

LGTM. just added one comment for OS availability.
I have one more PR refactoring allCases that I want to make in an attempt to reduce the package size. Hopefully, I can submit it this weekend,


iOS: add HEIF texture loader

Suggestion for https://github.com/fusetools/fuselibs-public/issues/668.
This PR contains:

[ ] Changelog
[ ] Documentation
[ ] Tests

I have not integrated this into Image, the ImageSource  code is tightly coupled with Experimental.TextureLoader.

🤔  This could be used as the textureloader on iOS. Since UIImage supports JPG, PNG and HEIF

I think we should finish up https://github.com/kusma/fuselibs-public/tree/bitmap instead.

https://github.com/kusma/fuselibs-public/commits/eeue56-feature-bitmap has the uploading-parts.
But please, no more ad-hoc features added here until we've landed the bitmap branch.

Yup, I agree. Lets discuss this on monday


Error while running the example

Hi Team,
Thanks for this wonderful project - I am really excited to see the integration with Spark 3!
I was trying out the latest 0.0.19 and ran into an import error while trying out the example
Clojure 1.10.1
(ns spark.geni.core
  (:require
    [clojure.string]
    [zero-one.geni.core :as g]
    [zero-one.geni.ml :as ml]))

Syntax error (ClassNotFoundException) compiling at (zero_one\geni\interop.clj:1:1).
org.apache.spark.ml.linalg.DenseVector

Could you please help me out a bit here?

Here's my project.clj
(defproject spark-clj "0.1.0-SNAPSHOT"
  :description "FIXME: write description"
  :url "http://example.com/FIXME"
  :license {:name "EPL-2.0 OR GPL-2.0-or-later WITH Classpath-exception-2.0"
            :url  "https://www.eclipse.org/legal/epl-2.0/"}
  :dependencies [[org.clojure/clojure "1.10.1"]
                           ;; spark
                          [zero.one/geni "0.0.19"]]
  :repl-options {:init-ns spark-clj.core})

Actually, I fixed it - the issue was because of incomplete config in project.clj. I added the spark packages and it started working fine.

Hi @abhi18av, due to the size of the Spark packages, it's not included in Geni's clojars. It's probably easiest to use the Geni lein template: lein new geni <project-name> or, say, lein new geni <project-name> +xgboost if you want to do XGBoost.

Yeah, I realized that later. I restructured the project to that template itself - works great!


Import statement from Readme/Docs doesn't work

Hello, thanks for the nice plugin.
I'm upgrading from Sanity Studio v2 to v3.
The docs say:
import { imageHotspotArray } from "sanity-plugin-hotspot-array";

export default defineConfig({
  // ...
  plugins: [
    imageHotspotArray(),
  ]
})

That throws:
Uncaught error: Importing binding name 'imageHotspotArray' is not found.
:0:0
moduleDeclarationInstantiation@[native code]
link@[native code]
link@[native code]
linkAndEvaluateModule@[native code]
@[native code]
promiseReactionJob@[native code]

If I change the name to imageHotspotArrayPlugin it works/doesn't blow up, but I don't know for sure if that's the right export.
I'm using latest version of the plugin as recommended in the docs (not the v2 version).

@DanielHirunrusme for v3 our working config ended up being:
// sanity.config.js
import { imageHotspotArrayPlugin } from "sanity-plugin-hotspot-array";

..

export default defineConfig({
	...
	plugins: [
		imageHotspotArrayPlugin(),
	],
});

I think the names in the docs are just out of date/incorrect

Fixed in #15


how to convert a movie mp4 to hls in tsduck using tsp toolkit

tsp -I hls Idiot.mp4 -O hls manifest.m3u8
how to convert for  VOD playlist
comment the full tsp mp4 to hls code
Thanking you in advance

TSDuck, as expressed in its name, manipulates transport streams (TS), not MP4 files.
Have a look at ffmpeg for conversion of container format.

@lelegard
i have another doubt please clear my doubt please help me
how to convert tsp udp to hls with multiresolution for OTT streaming in tsduck toolkit
tsp -I ip <IP_ADDRESS>:1001 -O hls --duration 3 -p master.m3u8 stream.ts
how to convert hls with multiresolution for ott streaming in tsduck
and how to delete the hls segment file after 10 segment file automatically for live streaming
tell me a full script how to delete the segment file automatically and for multiple resolution OTT streaming udp to hls

HLS with multiresolution means:

Create N distinct TS with transcoded video (distinct resolution and/or codec) with sliding segments
Create and regularly update N distinct media playlists, one per TS
Create one static master playlist, containing references to all media playlists

TSDuck is a transport stream toolkit, not an HLS system, and tsp uses one input TS and produces one output TS. For  tsp, hls  is just an input and output plugin, reading one TS, producing one TS.
Additionally, TSDuck has no audio/video transcoding capability. This is the domain of ffmpeg and other specialised tools.
Consequently, one instance of tsp cannot create multiple output streams. You may perhaps build an HLS system using several instances of tsp and ffmpeg. Open source tools such as TSDuck and FFmpeg provide the building blocks but this is your own job to design the system. Nobody will design it for you for free.
Concerning your last point, the automatic deletion of segments, please read the TSDuck user's guide, output plugin hls:
  --live value
      Specify that the output is a live stream. The specified value indicates
      the number of simultaneously available media segments. Obsolete media
      segment files are automatically deleted. By default, the output stream is
      considered as VoD and all created media segments are preserved.

@lelegard
thanking you dude

@lelegard
another one more doubt please help me dude
tsp -I ip <IP_ADDRESS>:1001 -P zap 2 -P regulate --pcr-synchronous -P pcrbitrate -P analyze -i 10 -o report.txt -O ip -l <IP_ADDRESS> <IP_ADDRESS>:1004 --enforce-burst
how to encrypt and decrypt tsp tsduck upd to udp conversion
thank you in advance

It depends on what you mean by "encrypt". Which cipher? Which key? Static or dynamic, using a fixed CW or rotating crypto-periods using a DVB SimulCrypt-compliant ECMG? First, define precisely what you want to implement and then RTFM, plugins scrambler and descrambler.

@lelegard
Thank u soo much dude

@lelegard
another one doubt dude please reply me
tsp -I ip -l <IP_ADDRESS> <IP_ADDRESS>:1001 -P zap "NEWS" -P analyze -i 30 -o report.txt
-P regulate --pcr-synchronous
-O ip --ttl 64 -l <IP_ADDRESS> <IP_ADDRESS>:1111
iam using this script to stream channels, above 20 streams i have runned and played the video the video and audio was little slow motion strucking, what is the problem please solve this issue and i have runned one streams no video was struked
thank u in advance

Same dout me also !!!

@lelegard
how take only one audio from udp
my input udp receives 2 audio and 1 video
how to take only one audio to output stream is there is any possible command
please reply me thanking you

how to take only one audio to output stream is there is any possible command using tsp

Please read the user's guide, reference section for the zap plugin, options --audio and --audio-pid.
Please note that this forum is not a replacement for reading the documentation.

@lelegard Thank you for your reply
i used --audio-pid but the audio was not played
tsp -I ip $INPUT_URL -P zap "0401 NDTV GT" --audio-pid 0x05A0 -P analyze -i 30 -o report.txt
-P regulate --pcr-synchronous -O hls --live 15 --duration 3 -p master.m3u8
udp was receiving one with and four audio i want take only one audio but i used audio pid
video was playing but the audio was not playing
my udp report i have attached
SERVICES ANALYSIS REPORT                                                   |
|=============================================================================|
|  Global PID's                                                               |

TS packets: 8, PID's: 3 (clear: 3, scrambled: 0)

PID  Usage                                     Access          Bitrate

Total  Global PID's ................................. C       12,070 b/s

Subt.  Global PSI/SI PID's (0x00-0x1F) .............. C       12,070 b/s

0x0000  PAT .......................................... C        7,544 b/s

0x0001  CAT .......................................... C        3,017 b/s

0x0011  SDT/BAT ...................................... C        1,509 b/s

=============================================================================

Service: 0x0004 (4), TS: 0x0087 (135), Original Netw: 0x0001 (1)

Service name: 0401 NDTV GT, provider: NDTV

Service type: 0x01 (Digital television service)

TS packets: 4,361, PID's: 6 (clear: 6, scrambled: 0)

PMT PID: 0x0054 (84), PCR PID: 0x05B4 (1460)

-----------------------------------------------------------------------------

PID  Usage                                     Access          Bitrate

Total  Digital television service ................... C    6,579,451 b/s

0x0054  PMT .......................................... C        7,544 b/s

0x058C  MPEG-2 Audio (eng, Audio layer II, 128 kb/s,   C      137,292 b/s

0x05A0  MPEG-2 Audio (eng, Audio layer II, 128 kb/s,   C      137,292 b/s

0x05B4  AVC video (720x576, high profile, level 3.0,   C    6,027,266 b/s

0x05BE  MPEG-2 Audio (eng, Audio layer II, 128 kb/s,   C      135,783 b/s

0x05C8  MPEG-2 Audio (eng, Audio layer II, 128 kb/s,   C      134,275 b/s

(C=Clear, S=Scrambled, +=Shared)

=========================================================================

please reply me thank you dude

Automatically closed after 159 days without update and "close pending" label set.


286 add quadbuffer usage log

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: frank-weindel:white_check_mark: elsassph:white_check_mark: wouterlucas:x: Marcel DanilewiczMarcel Danilewicz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Feature Request : Change title, message and button font sizes

Please provide a feature to change the font size of the contents in Dialog, Thank You

Added to version 2.0.9.
setTitleTextSize(sizeInSp);

setMessageTextSize(sizeInSp);

setButtonsTextSize(sizeInSp);


cut -c3- ~/.tmux.conf | sh -s _apply_configuration returned X

In relation to #149, #201, #238 and #256, and due to this happening for both me and a colleague, running .tmux.conf failed for the following line:
cut -c3- ~/.tmux.conf | sh -s _apply_configuration
I think it's related to what default shell is set - mine was set to bash, and it was stubborn even after a reboot (even after chsh - go figure!). Changing the line to the following worked for me (note sh changed to zsh):
cut -c3- ~/.tmux.conf | zsh -s _apply_configuration
Would this be worth a pull request? Or does this change have some side effects? There are other | sh -c lines that might also benefit from being changed to | zsh -c, although I haven't tested.

Hello 👋
No, cut -c3- ~/.tmux.conf | zsh -s _apply_configuration would make this configuration depend on zsh which is not needed and likely not the proper way to solve whatever is happening.
Can you please run
$ cut -c3- ~/.tmux.conf | sh -sx _apply_configuration > /tmp/log.txt 2>&1

And upload the result?

@gpakosz You are right, of course zsh would be an extra dep - my bad!
Here's some visuals too, the before (including a manually run cut -c3- ~/.tmux.conf | sh -s _apply_configuration command):

and after running the zsh-ified command (cut -c3- ~/.tmux.conf | zsh -s _apply_configuration):

Forgot to also include the /tmp/log.txt, from running cut -c3- ~/.tmux.conf | sh -sx _apply_configuration > /tmp/log.txt 2>&1

Alright, please try out the gh-254 branch.
I'll do my best to merge what's currently in flight soon.

Thank you! Consider this issue solved for me when using branch gh-254 - thanks for the incredibly fast responses too, you are a true open source hero :)

@gpakosz has te gh-254 branch been deleted? I still have this issue

@nivv The gh-254 branch has been merged. Just use most recent master branch

Thanks! It worked after a reinstall!
lör 14 dec. 2019 kl. 13:17 skrev Gregory Pakosz<EMAIL_ADDRESS>
Closed #279 https://github.com/gpakosz/.tmux/issues/279.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gpakosz/.tmux/issues/279?email_source=notifications&email_token=AA6Y3CVLHVXEDTHCX7N6OETQYTFGPA5CNFSM4IPQRTOKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVPOGS6Y#event-2883348859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6Y3CWZTJS3W3E4BQDZEALQYTFGPANCNFSM4IPQRTOA
.


Is it already possible to use Premium Sku for Container Apps Environment?

Hi,
I'd like to test some features provided in 2022-06-01-preview, namely vnetConfiguration.outboundSettings. As far as I know, in order to do that it, I need to use Premium Sku. I've tried to create a Container Apps Environment with Sku = Premium, but I get an error: PremiumManagedEnvironmentCreationNotEnabled. What do I need to do to start using Premium Sku ?

Hi @macpak thanks for the question. We were evaluating the introduction of the Premium SKU but have delayed based on optimizations we are planning that will enhance the experience on a multitude of fronts. We will keep the community informed of the new ETA as more details are available


Use the same name for pip package and repository

At the moment the repository name is sm-analytics-python but pip installs sm-annotation-utils package.

And plz then delete “sm-“ at the beginning
On Thu 21. Jun 2018 at 19:35, Vitaly Kovalev<EMAIL_ADDRESS>wrote:

At the moment the repository name is sm-analytics-python but pip package
installed is sm-annotation-utils
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/METASPACE2020/sm-analytics-python/issues/13, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADF6_ReNDvi-9zjr2zmqpFsReDqqb4w9ks5t-9lNgaJpZM4UyfDQ
.

This repository will be merged into the metaspace monorepo


Change Tunables::static_memory_bound to bytes

This commit changes the wasm-page-sized static_memory_bound field to instead being a byte-defined unit rather than a page-defined unit. To accomplish this the field is renamed to static_memory_reservation and all references are updated. This builds on the support from #8608 to remove another page-based variable from the internals of Wasmtime.

Yeah I think most of those were handled in https://github.com/bytecodealliance/wasmtime/pull/8608 although there's still memory_pages in the pooling allocator which I'd like to remove as well and is probably also a large source of overflow checks.

I think consistently using bytes is just better

Agreed, especially since it is effectively necessary to support the custom-page-sizes proposal.

when I was mucking around in the pooling allocator it was always tricky to understand whether variables were page-based or byte-based

I think we should probably start newtyping our units. Way back when I wrote some stuff like this for wee_alloc and then someone pulled it out into a crate: https://docs.rs/memory_units
Don't think we want to use exactly that, but more that kind of thing.

Don't think we want to use exactly that, but more that kind of thing.

I agree. That's inside the code base, but even from a CLI perspective I found it useful to use bytesize in the mpk.rs example to more easily just pass in strings like "4KiB" or whatever instead of having to do mental math.


CloudFormation template not getting created

$ aws cloudformation create-stack --stack-name k8s --template-body file://k8s-create-no-alb.yaml

An error occurred (ValidationError) when calling the CreateStack operation: Parameters: [KeyName, NetworkProvider] must have values

@cloudymind can you take a look?

Output tab shows:
scp -i arun-eu-central-1.pem ubuntu@<IP_ADDRESS>:.kube/config $HOME/.kube/config

It need to be updated to:
scp -i arun-eu-central-1.pem ubuntu@<IP_ADDRESS>:.kube/config $HOME/.kube/eu-central-1-kubeconfig
export KUBECONFIG=$HOME/.kube/eu-central-1-kubeconfig:$HOME/.kube/config

eu-central-1 is referred using !Ref KeyName.
FYI @cloudymind


Add before_process and transform hooks

@jondeandres It seems that the :scope is empty/missing in the before_process hooks. Repro case:
handler1 = proc do |options|
  puts "+++ in handler1"
  puts options
  puts options[:scope][:custom][:some_key]
  puts "+++"
end

Rollbar.configure do |config|
  config.access_token = 'token'
  config.before_process << handler1
end

Rollbar.scope(:custom => {:some_key => "some value"}).debug("a debug message")

When I do the above, the output is:
+++ in handler1
{:level=>"debug", :scope=>{}, :exception=>nil, :message=>"a debug message", :extra=>nil}
E, [2016-02-10T13:42:33.745977 #20753] ERROR -- : [Rollbar] Error calling the `before_process` hook: undefined method `[]' for nil:NilClass

@brianr the interface to access the scope is options[:scope].custom in that case. But using [] shouldn't give that behavior. I'll review this.

Forget it, it's a bug yes. I'm passing there configuration.payload_options instead of the scope_object. I'll fix it.

@brianr I've fixed and changed the interface in https://github.com/rollbar/rollbar-gem/commit/3428eb13352047fb6c43291e50c9cc8ddb73811c.
I think it'd be better if the users can access the scope as you wanted, scope[:request] instead of scope.request, since the scope is defined like Rollbar.scoped(request: some_data).


Violators and deadweight tables should link to jobquery page

See #310, which provides a form to query the job database and generate a listing of jobs, whose profile data can be analyzed further.  Probably the listing of violators (and maybe the listing of zombies?) should have hyperlinked job numbers, and clicking on a job number would take us to the query page with the query fields filled out, and arguably with the query button auto-pressed so that the job in question is shown in the job listing.  Then the usual profile examination can commence.

In general, for these tables:

user names should link to jobquery with user named filled in
job numbers should link to jobquery with job number filled in
in both cases, cluster and host should be filled in


Introduce ValidatorNotExistsException

The goal of this exception is to prevent user that using an older version of ronanguilloux/isocodes it is throw if the needed validator does not exists.
This will permit to allow isocodes 2.0 instead of 2.1 and will be smoother for the future.
We will have to introduce a ConstraintInterface with a getIsoCodeVersion method.
Considering that as a "patch" because it's not a "real" feature.

Lot of refactoring. Will be a minor.


Add a clickable URL for the request

I often find myself copypasting the host, path and query into a new tab to replicate the Guzzle call. This link would allow me to do this in one click.

Very nice :+1:  thank you

Great :100: thank you

@devzer01 PR tagged as stable. Available in v1.1.0


Why the price is not change when in test mode

i provide live and test api key and secret in .env file. And BINANCE_MODE = test in docker-compose.yml but the price is not tik tik.
But when mode = live the price change tik tik happen

Because the test server for Binance does not have any transactions. :)

so can we get transactions info from live then place order in test network?

No

If you want to use testnet, try with BNBUSDT in the testnet. And set candles to be 1m/5 something. BNBUSDT in the testnet moves a bit.

:) thank you for your bot.


Adding new images and button guidance for forms

Closes https://github.com/carbon-design-system/carbon-website/issues/219
Preview: https://carbon-website-git-formimages.carbon-design-system.now.sh/components/form/usage#button-placement
This brings @chrisconnors-ibm updated button guidance to the form component

Can we hold off on merging this until we finish looking at https://github.com/carbon-design-system/carbon-website/pull/311?
We're taking advantage of Chris' good graces to finally settle on some style decisions I'd like to see reflected here.

the two Wizard examples will need to be updated based on the changes for https://github.com/carbon-design-system/carbon-website/pull/311

Button section synced with the latest button updates.


Documentation tweaks to action.yml / README.md pre-publishing

@manics I added some branding aligning with our purple theme in our other actions, provided a description of the output parameter in the README, and synced the action.yml/README.md descriptions.
After this, we are ready for release/publish of 1.1.0 IMO!

@manics this LGTM! Go for merge and release of 1.1.0 now?

@consideRatio yes, 1.1.0 sounds good

@manics I saw the release was cut, :tada:!


Swift 4.2

Any plan for migration to swift 4.2?

Yea sure.
Are there any breaking changes in this lib with 4.2?

There are no compile-time errors but there are run-time constraint resolution errors. It seems like as if the layout engine is not able to break constraints. Also, the buttons are smaller in radius than in iOS 11.

I'm getting 22 compile errors using JTAppleCalendar in empty project with Swift 4.2 and Xcode 10 GM seed. All of them associated to renamed objects within UICollectionView and UIScrollView classes.

I have the Same.
@patchthecode When to expect Swift 4.2 migration ?
Thanks

Is this version available in cocoapods?
I get 7.1.5 with swift 4.0
Thx

I have the same issue with swift 4.2.

@patchthecode Can you please push a version to Cocoapods that contains the migration to Swift 4.2?

Can you please push a version to Cocoapods that contains the migration to Swift 4.2?

Can you please push a version to Cocoapods that contains the migration to Swift 4.2?

@TiagoMaiaL
done. Closing issue.
let em know of any bugs

@patchthecode my application was made in swift 4.0 and by mistake i updated the pod now there are lots error are coming. Is there any way to get older version back?

@patchthecode my application was made in swift 4.1 and by mistake i updated the pod now there are lots error are coming. Is there any way to get older version back?

Put this in your Podfile for an Swift 4.1-compatible version of JTAppleCalendar:
pod 'JTAppleCalendar', '~> 7.1.5'

Any version with Swift 4.0?
Tks


Getting started example doesn't work. Error.

I'm submitting a ...  (check one with "x")
[ X ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior

When following getting started from
https://swimlane.gitbooks.io/ngx-datatable/introduction/getting-started.html
, adding NgxDatatableModule to App module and then adding this kind of html to App component:
<div>
      <ngx-datatable
        [rows]="rows"
        [columns]="columns">
      </ngx-datatable>
</div>

, it shows up, but there is absolutely no style or shape. All the array items (columns and rows) are listed in a single row.
And when i try to add that same html to some other component than App component and those arrays to that components .ts file, it breaks the whole app, giving error like this on browser:
EXCEPTION: Uncaught (in promise): TypeError: Cannot set property 'stack' of undefined TypeError: Cannot set property 'stack' of undefined     at SyntaxError.set [as stack] (http://localhost:4200/vendor.bundle.js:88037:61)...
I have tried with ngx-datatable 5.0.0 and 4.2.0. I have tried many kinds of things like importing to component, moving style files (material.css, datatable.css), etc.  Not working, no solutions.
Expected behavior

It is said to work out of the box.
Reproduction of the problem

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Windows 10 64-bit,
Chrome Version: 55.0.2883.87 m (64-bit),
Node v: 6.9.1,
Npm v: 3.10.9
My package.json:
{
"name": "material-test",
"version": "0.0.0",
"license": "MIT",
"angular-cli": {},
"scripts": {
"start": "ng serve",
"lint": "tslint "src/**/*.ts"",
"test": "ng test",
"pree2e": "webdriver-manager update",
"e2e": "protractor"
},
"private": true,
"dependencies": {
"@angular/common": "^2.4.3",
"@angular/compiler": "2.4.3",
"@angular/core": "2.4.3",
"@angular/forms": "2.4.3",
"@angular/http": "2.4.3",
"@angular/material": "latest",
"@angular/platform-browser": "2.4.3",
"@angular/platform-browser-dynamic": "2.4.3",
"@angular/router": "3.4.3",
"@swimlane/ngx-datatable": "^5.0.0",
"angular2-google-chart": "latest",
"core-js": "^2.4.1",
"hammerjs": "latest",
"rxjs": "5.0.3",
"ts-helpers": "^1.1.1",
"zone.js": "^0.7.5"
},
"devDependencies": {
"@angular/compiler-cli": "2.4.3",
"@types/jasmine": "2.5.40",
"@types/node": "^7.0.0",
<EMAIL_ADDRESS>"^1.4.29",
"angular-cli": "1.0.0-beta.25.5",
"codelyzer": "~2.0.0-beta.1",
"jasmine-core": "2.5.2",
"jasmine-spec-reporter": "3.2.0",
"karma": "1.4.0",
"karma-chrome-launcher": "^2.0.0",
"karma-cli": "^1.0.1",
"karma-jasmine": "^1.0.2",
"karma-remap-istanbul": "^0.4.0",
"protractor": "5.0.0",
"ts-node": "2.0.0",
"tslint": "^4.0.0",
"typescript": "~2.1.5",
"webdriver-manager": "11.1.1"
}
}

Table version: ^5.0.0, 4.2.0

Angular version: 2.4.3

Browser: [ Chrome 55.0.2883.87 m (64-bit) ]

Language: [ TypeScript ]

Please make a demo plunkr

Well, ok i'll try. I have never done that.

Here is the Plunker demo I created. The first problem, styling is missing, is here too, but putting  doesn't crash the app like it does with my project. I gotta do more testing I think..
https://embed.plnkr.co/FyHyFl33dhkprx6rEVHv/

I am having the same issue. I know the documentation says that you need to include the base styling, but I am unsure on how to do that with webpack. I'll update my response if I figure it out.

@mikkoh85 I'm not seeing the datatable.css file imported anywhere in your plunkr. I created a new file datatable.css, pasted the datatable styles into it, and then imported it in the main stylesheet with @import './datatable.css'; and it's working fine.

I guess I missed the part mentioning the base styling, so I didn't include it anyhow. I am using webpack too. I already built my own datatable component, so this is not an issue for me anymore. But thanks for help.

I'm going to be including the base styles by default in the next release.

I'm having difficulties with this as well: I have just added the mentioned imports to my scss file and then specified the class of my ngx-datatable to <ngx-datatable class="material">.
I'm a newbie, had to google up on how to include a scss, did the following:

in my app.component.ts file changed a line: styleUrls: ['./app.component.scss']
created the app.component.scss file with the imports listed on the getting started themeing part.
installed some packages that I thought should enable the scss to work: npm install -save style-loader sass-loader postcss-loader node-sass css-loadernpm install -save style-loader sass-loader postcss-loader node-sass css-loader

Still have no styling, please provide more details on this to get started.
I don't get it, what am I missing or doing wrong?
My project is based on angular 2.3.1, on pluncker there is only version 2 so the structure is different, don't see an option to upload whole folder. The table loads in but without styling.

I can confirm identical behavior described by z--aszlo, any solutions?

Ya, I'm working on this still :( ... I have yet to find suitable approach.
Issue is the AoT builds errors when ran through webpack so I have to run them manually outside thus not getting the correct compilation for the css ( which is odd cuz it generates a css compilation file for AoT ).
I think the best fix might be getting webpack to run the AoT.

CSS issue should be fixed now.

I'm still having this issue with the latest release. Not sure what I'm doing incorrectly.


Release for v29.5.0 missing

The github release for v29.5.0 seems to be missing the release asset. The manifest has
  url: https://github.com/bosh-prometheus/prometheus-boshrelease/releases/download/v29.5.0/prometheus-29.5.0.tgz

But this doesnt exist...

@bg-govau -> Fixed :)

Thankyou!


An aliasified package should be auto ignored in Validate stage or alternate mechanism to be provided

An aliasified source package at the moment, should explicitly be ignored in validate stage. There is no documentation clarifying this nor any alternate approach is proposed on how an aliasified source package should be pushed to scratch org

As a workaround, the package needs to have an additional descriptor added
"ignoreOnStage": [
                "prepare",
                "validate"
                ]


SubscriptionExceptionResolver.resolveException should accept DataFetchingEnvironment

Currently the method in DataFetcherExceptionResolver is Mono<List<GraphQLError>> resolveException(Throwable exception, DataFetchingEnvironment environment) while the method in SubscriptionExceptionResolver is Mono<List<GraphQLError>> resolveException(Throwable exception). It would be nice to have DataFetchingEnvironment passed in as well to include more information in our error logs.
As a side note in DataFetcherExceptionResolver.resolveException I've noticed that for some reason SecurityContextHolder.getContext().getAuthentication() returns an instance of AnonymousAuthentication when I know the user is authenticated (and the same piece of code works correctly elsewhere) and as a work-around I'm force to use environment.getGraphQlContext().get<SecurityContextImpl>(SecurityContext::class.name).getAuthentication() to get the proper authentication instance. Same applies in SubscriptionExceptionResolver

This would have to be the original DataFetchingEnvironment at the start of the subscription. I suppose that's the one you have in mind?

for some reason SecurityContextHolder.getContext().getAuthentication() returns an instance of AnonymousAuthentication

If you could open a separate issue. A small reporducer if available would help speed it up.

This would have to be the original DataFetchingEnvironment at the start of the subscription. I suppose that's the one you have in mind?

I'm currently using getGraphQlContext(), getField() and getArguments(). Not completely sure what you mean by the start but an exception could occur at any point while emitting a result and resolving some field so I would imagine the latter two are "per-emission", not "per-subscription"

If you could open a separate issue

Sure, I will work on a minimal repo

I have to correct myself a little. I was referring to the fact the @SubscriptionMapping method is called once at the start of request execution. Then it's up to the Publisher to keep producing results. However, it is true that GraphQL Java gets involved in calling data fetchers for related entities for the graph of each result item.
SubscriptionExceptionResolver, however handles error signals from the Publisher, and only has access to the exception that propagated and the DataFetcherEnvironment for the @SubscriptionMapping method. For anything more, it would exception handling needs to be closer to the invocation of DataFetchers.
GraphQL Java is in the best position there, but it doesn't support anything at that level as far as I know. At our level, we do handle exceptions from controller methods and invoke @GraphQlExceptionHandler controller methods. Have you considered using that?
For anything else, there would h to be something closer to the invocation of DataFetchers.
at that point it has nothing else to work but the exception. Hence the current contract. To integrate more closely, the exception has to be handled closer to the invocation of the DataFetcher.
the only way to know what DataFetcher was executing is to handle the exception at the point of DataFetcher invocation.
a SubscriptionExceptionResolver

Thank you, with your information and some more digging I was able to achieve what I needed by using @GraphQlExceptionHandler.
Feel free to close this issue if you deem so. Not sure if in the case of subscriptions there's a reason to use SubscriptionExceptionResolver over @ControllerAdvice with @GraphQlExceptionHandler

SubscriptionExceptionResolver came first, and @GraphQlExceptionHandler after that. At present, I would use the latter for controller methods. The latter is at a lower level and can handle exceptions from any other DataFetcher.


Couldn't login: Forbidden: 257 - Please purchase/redeem Elite Dangerous to continue.

I'm unable to start Elite using this launcher.

Windows 10
Steam purchase 25 June 2020
Steam is running
The regular launcher works fine
min-ed-launcher v0.5.3

2021-09-25 14:28:27.134 -04:00 [INF] Elite Dangerous: Minimal Launcher - v0.5.3
2021-09-25 14:28:27.232 -04:00 [DBG]
    Args: [|"E:\7 Games\steamapps\common\Elite Dangerous\EDLaunch.exe"; "/Steam"; "/novr";
  "/autoquit"; "/frontier"; "winter"|]
    OS: Win64

2021-09-25 14:28:27.249 -04:00 [DBG] Reading settings from 'C:\Users\Chris\AppData\Local\min-ed-launcher\settings.json'
2021-09-25 14:28:27.535 -04:00 [DBG] Settings: Ok
  { Platform =
              Frontier
                { Profile = "winter"
                  Credentials =
                               Some
                                 { Username = "[snip]"
                                   Password =
                                             "[snip]" }
                  AuthToken =
                             Some
                               "[snip]" }
    DisplayMode = Pancake
    AutoRun = false
    AutoQuit = true
    WatchForCrashes = false
    ProductWhitelist = seq []
    ForceLocal = true
    Proton = None
    CbLauncherDir = "E:\7 Games\steamapps\common\Elite Dangerous"
    PreferredLanguage = None
    ApiUri = https://api.zaonce.net/
    Restart = None
    AutoUpdate = true
    MaxConcurrentDownloads = 4
    ForceUpdate = set []
    Processes = []
    FilterOverrides = seq [[FORC-FDEV-DO-1000, edo]] }
2021-09-25 14:28:27.545 -04:00 [INF] Elite Runtime
    Platform: Frontier
    CobraBay Version: 0.4.6709.0
    Products Dir: E:\7 Games\steamapps\common\Elite Dangerous\Products
2021-09-25 14:28:28.398 -04:00 [DBG] Authenticating via Frontier
2021-09-25 14:28:28.563 -04:00 [ERR] Couldn't login: Forbidden: 257 - Please purchase/redeem Elite Dangerous to continue.

Are you trying to run it via your Steam purchase or an alternate Frontier purchased account? If just the steam account, you can get rid of the /frontier winter argument so that tries to auth with your steam credentials instead.

I have two Steam accounts and I was hoping to be able to play both without having to restart Steam. Did I misunderstand the README though? Is that supported?

I think that would only work if one of your steam accounts was purchased before 15 April 2019 which would have given you Frontier store key. When authenticating via Steam, the launcher communicates with the currently running Steam instance.
It's possible there's an auth workflow that I'm unaware of, but as of right now, I think you'll be stuck with switching Steam accounts.

Ok, thank you!


Manual Initialization broken in V3 [BUG]

Behavior:
Using initControls.initialize() doesn't succesfully init the editor. The toolbars are loaded correctly, but only [object HTMLDivElement] will be displayed where the editor window should be. Destroying the editor makes the toolbars disappear, but doesn't actually destroy the editor (initControls.getEditor() != null).
Steps to reproduce the problem:
It's broken in the demo too... Sample 3.
Editor version:
Angular Froala: V3.0
WYSIWYG: V3.0
Browser:
Chrome (75)

Fixed!


Scan view doesn't support iPhone 5 screen size

It seems scan card camera view is only support iPhone 4 size, there are black spaces both on top and bottom of the screen.

@tonyli508 we use the standard 640×480 camera resolution on all devices.
For devices with a taller aspect ratio than 640x480, this implies that we must either add black space above and below the camera image, or else that we enlarge the camera image and cut off its sides. We have chosen the former approach.