Error: line 0: Failed to open IFF file for reading

Hi
Thanks for Developing Plugin for pbrt
I have a problem in using PBRTForMaya Plugin
OS: Windows 10
Maya: 2018
I built PBRT and set environment variables in maya, everythings sounds good, PBRT Render Setting is there, plugin loaded successfully, PBRT Path and oiiotool path is set, but when i try to render a sample scene it give me this error
// Error: line 0: Failed to open IFF file for reading : C:/Users/[USER]/AppData/Local/Temp//maya9288000000FE7EBF5D40.iff //
maybe problem is for having two forward slash // after Temp !!!
Whis is the reason of this error?
need to set anything else? or this is not campatible with latest version of PBRT?
please let me know
thanks

Hm... I've seen issues like that when the OpenEXR plugin isn't loaded, but PBRTForMaya should take care of that for you now.
Can you start a new session of Maya, load the plugin, load your scene, hit render. If you still see the error, copy the contents of the Script Editor into this thread.

Hi @hpd
Thanks for Answer
This is my out put of Script Editor after loading plugin and hit render button on demo scene:
PBRTForMaya - Unable to load .ply format translator
ImportError('No module named numpy',)
PBRTForMaya - Loaded plugin       : objExport
PBRTForMaya - Loaded plugin       : OpenEXRLoader
Render Settings initialize!
PBRTForMaya - Registered node     : PBRTRenderSettings
PBRTForMaya - Registered textures : PBRTBilerpTexture
PBRTForMaya - Registered textures : PBRTBlackBody
PBRTForMaya - Registered textures : PBRTConstantTexture
PBRTForMaya - Registered textures : PBRTFBMTexture
PBRTForMaya - Registered textures : PBRTMarbleTexture
PBRTForMaya - Registered textures : PBRTMixTexture
PBRTForMaya - Registered textures : PBRTScaleTexture
PBRTForMaya - Registered textures : PBRTuv
PBRTForMaya - Registered textures : PBRTWindyTexture
PBRTForMaya - Registered textures : PBRTWrinkledTexture
PBRTForMaya - Registered textures : PBRTxyY
PBRTForMaya - Registered textures : PBRTxyz
PBRTForMaya - Registered lights : PBRTDiffuseAreaLight
PBRTForMaya - Registered lights : PBRTInfiniteLight
PBRTForMaya - Registered materials : PBRTDisneyMaterial
PBRTForMaya - Registered materials : PBRTFourierMaterial
PBRTForMaya - Registered materials : PBRTGlassMaterial
PBRTForMaya - Registered materials : PBRTHairMaterial
PBRTForMaya - Registered materials : PBRTKDSubSurfaceMaterial
PBRTForMaya - Registered materials : PBRTMatteMaterial
PBRTForMaya - Registered materials : PBRTMetalMaterial
PBRTForMaya - Registered materials : PBRTMirrorMaterial
PBRTForMaya - Registered materials : PBRTMixMaterial
PBRTForMaya - Registered materials : PBRTPlasticMaterial
PBRTForMaya - Registered materials : PBRTSubstrateMaterial
PBRTForMaya - Registered materials : PBRTSubSurfaceMaterial
PBRTForMaya - Registered materials : PBRTTranslucentMaterial
PBRTForMaya - Registered materials : PBRTUberMaterial
PBRTForMaya - Registered volumes : PBRTHeterogeneousMedium
PBRTForMaya - Registered volumes : PBRTHomogeneousMedium
PBRTForMaya - Registered shapes : PBRTConeGeometry
PBRTForMaya - Registered shapes : PBRTCylinderGeometry
PBRTForMaya - Registered shapes : PBRTDiskGeometry
PBRTForMaya - Registered shapes : PBRTHyperboloidGeometry
PBRTForMaya - Registered shapes : PBRTParaboloidGeometry
PBRTForMaya - Registered shapes : PBRTIncludeGeometry
PBRTForMaya - Registered shapes : PBRTPlyMeshGeometry
PBRTForMaya - Registered shapes : PBRTSphereGeometry
PBRTForMaya - Registered command  : PBRT
updateRendererUI;
PBRTForMaya - Registered renderer : PBRT

PBRT - Using existing PBRT settings node : defaultPBRTRenderGlobals
Rendering with PBRT...
PBRT - Using existing PBRT settings node : defaultPBRTRenderGlobals
Render Settings - Node            : defaultPBRTRenderGlobals
Render Settings - PBRT Path        : D:/renderers/pbrt-v3/deploy/bin/
Render Settings - Integrator       : Path Tracer
Render Settings - Sampler          : Halton
Render Settings - Pixel Samples    : 1
Render Settings - Filter           : Gaussian
Render Settings - Render Dir       : D:/renderers/PBRTForMaya/maya\renderData
Render Settings - oiiotool Path    : D:/renderers/OpenImageIO/oiiotool.exe
Render Settings - Verbose Render   : 2
Render Settings - Verbose Export   : False
Render Settings - Export Only      : False
Render Settings - Keep Temp Files  : True
Render Settings - Cmd Line Params  :
Render Settings - Animation        : False
Render Settings - Camera           : |referenceCamera1|referenceCamera1Shape
Render Settings - Camera           : |referenceCamera1|referenceCamera1Shape

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh__MeshShape.obj D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh__MeshShape_temp.pbrt

0.186183 - Converted 1 meshes (3936 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__chromeSphere__chromeSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__chromeSphere__chromeSphereShape_temp.pbrt

0.093719 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__glossySphere__glossySphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__glossySphere__glossySphereShape_temp.pbrt

0.101318 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh1__Mesh1Shape.obj D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh1__Mesh1Shape_temp.pbrt

1.611487 - Converted 1 meshes (57152 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__ground__groundShape.obj D:/renderers/PBRTForMaya/maya\renderData\__ground__groundShape_temp.pbrt

0.074844 - Converted 1 meshes (380 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__materialSphere__materialSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__materialSphere__materialSphereShape_temp.pbrt

0.099807 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__diffuseSphere__diffuseSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__diffuseSphere__diffuseSphereShape_temp.pbrt

0.128347 - Converted 1 meshes (3120 triangles, 0 mesh emitters).
Render Settings - Version          : v3 Book

process.Process : D:/renderers/pbrt-v3/deploy/bin/ --verbose --outfile pbrt_feature_demo_base.exr ..\renderData\pbrt_feature_demo_base.pbrt

Couldn't execute command : D:/renderers/pbrt-v3/deploy/bin/
# Traceback (most recent call last):
#   File "D:\renderers\pkgs\PBRTForMaya\plug-ins\util\process.py", line 502, in execute
#     universal_newlines=True)
#   File "D:\Program Files\Autodesk\Maya2018\bin\python27.zip\subprocess.py", line 710, in __init__
#     errread, errwrite)
#   File "D:\Program Files\Autodesk\Maya2018\bin\python27.zip\subprocess.py", line 958, in _execute_child
#     startupinfo)
# WindowsError: [Error 5] Access is denied
Render execution returned : None
Keeping temporary files
// Error: line 0: Failed to open IFF file for reading : C:/Users/madoodia/AppData/Local/Temp//maya166280000005D673F5930.iff //

In the Render Settings, switch the 'PBRT Version' from 'v3 Book' to 'v3 Master Branch'. It looks like the version of pbrt that you're rendering with doesn't accept the '--verbose' flag, which is a change that happened between the publishing of the book and adding new features in the master branch.

Thanks again
I did it, but i have another error
Rendering with PBRT...
PBRT - Using existing PBRT settings node : defaultPBRTRenderGlobals
Render Settings - Node            : defaultPBRTRenderGlobals
Render Settings - PBRT Path        : D:/renderers/pbrt-v3/deploy/bin/pbrt.exe
Render Settings - Integrator       : Path Tracer
Render Settings - Sampler          : Halton
Render Settings - Pixel Samples    : 1
Render Settings - Filter           : Gaussian
Render Settings - Render Dir       : D:/renderers/PBRTForMaya/maya\renderData
Render Settings - oiiotool Path    : D:/renderers/OpenImageIO/oiiotool.exe
Render Settings - Verbose Render   : 2
Render Settings - Verbose Export   : False
Render Settings - Export Only      : False
Render Settings - Keep Temp Files  : True
Render Settings - Cmd Line Params  :
Render Settings - Animation        : False
Render Settings - Camera           : |referenceCamera1|referenceCamera1Shape
Render Settings - Camera           : |referenceCamera1|referenceCamera1Shape

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh__MeshShape.obj D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh__MeshShape_temp.pbrt

0.187257 - Converted 1 meshes (3936 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__chromeSphere__chromeSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__chromeSphere__chromeSphereShape_temp.pbrt

0.122184 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__glossySphere__glossySphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__glossySphere__glossySphereShape_temp.pbrt

0.096457 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh1__Mesh1Shape.obj D:/renderers/PBRTForMaya/maya\renderData\__MitsubaSphere__Mesh1__Mesh1Shape_temp.pbrt

1.594571 - Converted 1 meshes (57152 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__ground__groundShape.obj D:/renderers/PBRTForMaya/maya\renderData\__ground__groundShape_temp.pbrt

0.092939 - Converted 1 meshes (380 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__materialSphere__materialSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__materialSphere__materialSphereShape_temp.pbrt

0.099003 - Converted 1 meshes (3120 triangles, 0 mesh emitters).

process.Process : D:/renderers/pbrt-v3/deploy/bin\obj2pbrt D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__diffuseSphere__diffuseSphereShape.obj D:/renderers/PBRTForMaya/maya\renderData\__referenceSpheres__diffuseSphere__diffuseSphereShape_temp.pbrt

0.099851 - Converted 1 meshes (3120 triangles, 0 mesh emitters).
Render Settings - Version          : v3 Master Branch

process.Process : D:/renderers/pbrt-v3/deploy/bin/pbrt.exe --v 0 --logtostderr --outfile pbrt_feature_demo_base.exr ..\renderData\pbrt_feature_demo_base.pbrt

0.089300 - pbrt version 3 (built Sep 28 2018 at 22:50:32) [Detected 8 cores]
0.001566 - Copyright (c)1998-2018 Matt Pharr, Greg Humphreys, and Wenzel Jakob.
0.001173 - The source code to pbrt (but *not* the book contents) is covered by the BSD License.
0.001227 - See the file LICENSE.txt for the conditions of the license.
0.001107 - I1001 22:35:11.609253  5016 parallel.cpp:113] Started execution in worker thread 2
0.001171 - I1001 22:35:11.609253 13552 parallel.cpp:113] Started execution in worker thread 4
0.001168 - I1001 22:35:11.609253   224 parallel.cpp:113] Started execution in worker thread 7
0.001167 - I1001 22:35:11.609253 17276 parallel.cpp:113] Started execution in worker thread 1
0.001195 - I1001 22:35:11.609253 14016 parallel.cpp:113] Started execution in worker thread 3
0.001478 - I1001 22:35:11.609253 12796 parallel.cpp:113] Started execution in worker thread 5
0.001398 - I1001 22:35:11.609253 19388 parallel.cpp:113] Started execution in worker thread 6
0.001226 - I1001 22:35:11.609253 17516 error.cpp:83] ..\renderData\pbrt_feature_demo_base.pbrt:40:57: Error: unexpected escaped character "s"
0.001743 - ..\renderData\pbrt_feature_demo_base.pbrt:40:57: Error: unexpected escaped character "s"
Render execution returned : -1073740791
Keeping temporary files
// Error: line 0: Failed to open IFF file for reading : C:/Users/madoodia/AppData/Local/Temp//maya15528000000BB033F5E20.iff //

Can you post the first 50 lines of ..\renderData\pbrt_feature_demo_base.pbrt?

 Switching from Windows \ to Unix / for the separator should fix the issue. Odd that this didn't show up before.

I don't think it relate to file path of texture
because

I set the project and it should detect the sourceimages because workspace.mel is beside sourceimages folder
it does not accept / instead of \

I tested on a sample scene without texture, like a sphere it worked fine
I think problem, as you said, is because of OpenExr
however i installed pbrt-v3 with its submodules and built it with cmake

Try selecting that image fresh from the Attribute Editor. It would be interesting to see if that still has trouble.

Exactly same error
it does not accept '/' instead of ''

and when i add ../ in the begining of string it changes it to sourceimages\..\sourceimages\mitsuba_sunsky.exr

Just a question about OpenEXR
I should have which file of it in my lib or bin dir in pbrt?

This isn't an issue with OpenEXR or .hdr. PBRT doesn't accept .hdr though.
The problem is that the "\s" in the path "..\sourceimages\mitsuba_sunsky.exr" is treated like a special character rather than just a '' and an 's'. This may actually be a bug in PBRT. I would have expected this path to work.

oh i see
so because maya's text box does not accept / or \\ we can do anything in this side
it should be solve in code
i don't know it should be handle in pbrt or PBRTForMaya plugin

Add this
.replace("\", "/")
to the end of line 143 in plug-ins/renderer/PBRTRendererIO.py
Should fix it...
The new line should be
fileTexture = os.path.relpath( fileTexture, renderDir ).replace("\", "/")

And github is treating the Windows slash as a special character. The line should be
fileTexture = os.path.relpath( fileTexture, renderDir ).replace("\\", "/")

actually should be fileTexture = os.path.relpath( fileTexture, renderDir ).replace("\\", "/")

I changed it works.
Thanks for Finding the solution.


Missing Project Reference

I opened Chrome Dev Editor on Chrome OS and found there are files missing. I have not power washed my Chromebook, so I was surprised to find the project missing. I believe it's only the reference to the project has been removed. When I created a new project with the same name, the project name was incremented.
Is there any way to see what projects exist on disk, but whose references have been removed? I'm not in dev mode on this laptop so I can't access the root file system.

I didn't realize that project files were stored on Drive automatically. Once I restored from Drive's trash bin, everything was there.


feat(target-slack): adds slack webhook target and tests

Changes made with e1b8aac. LMK if there's anything else.

Beautiful!

Can I ask why we would use a heavily-bloated module (request-promise and request) to implement this? I really don't want these dependencies in my project when all I need is a slim logger. Why not just make this a third-party module?
I really don't think the core Bristol library is the right place for this.

Hey @MarkHerhold -- Bristol pushes messages to Loggly too. The only difference in that case is that the loggly lib is left to the user to install manually. In this case, it's a little more cumbersome to say "To use the Slack target, install request" than "To use the Loggly target, install loggly".
I agree that Bristol should be lightweight-- I approved this because Bristol lazy-loads targets and formatters. While request would be downloaded for every install, it's never loaded unless you use this target.
With that said, though, I see your argument. Bristol should be just as attractive to space-conscious  users as it is to resource-conscious users. The conundrum is in determining how fully featured Bristol should be out-of-the-box versus requiring users to find and install all the external libraries that would make it fully featured. Perhaps only allow formatters and targets that are both 1) intended primarily for the local machine (within reason), and 2) using only native libs (within reason), and make everything else external? I'd love your thoughts here.
(Regarding your Edit-- Kent and I do work together, but this wasn't a rush or a compromise to Bristol for the sake of our company. I've just always envisioned Bristol as eventually having targets for every endpoint you might hope for, and Slack seemed like a fantastic one -- especially for high-severity messages that you might want to inform a team about in real time)

@TomFrost In my opinion, I think everything should be a third party 'module'. To supplement onboarding and user experience, I would add a set of featured/recommended modules to the README of Bristol. Your users already have to install and write some code to write logs, so nothing is lost here.
e.g.
Bristol is an insanely configurable logging for Node.js
Getting Started with Bristol
Installing
npm i bristol bristol-slack --save

Use Bristol
var log = require('bristol');
log.addTarget('console'); // built-in console logger
log.addTarget(require('bristol-slack'), '<slack webhook>');

log.info("We're up and running!", {port: 3000});
log.error("Hello Slack!");

The Bristol Ecosystem

Slack - Send critical messages to Slack in realtime
Loggly - Use Loggly as a log target
Palin - A colorful console formatter
JSON Safe - Safely log circular references to JSON format

I think that the Slack target would be a good 'recommended' module and should be featured on the Bristol repo, but please don't force users to install the dependencies it if they don't need them for core functionality. I would say that even loggly shouldn't be a part of the core simply because it isn't essential even though it doesn't really bloat anything.
As another benefit of making Bristol module-focused is that you solve problems brought up by #35, where users want to add a dependency for their specific use-case but you don't want the project to suffer a performance hit overall. If you message the project as being extensible only by modules, it makes it clear that the 'right way' to add a feature is to write a module. This is the approach I tried to take with palin, because not everyone wants colors. 🌈 (even though those people are wrong)
As a whole, the Node.js ecosystem has elected to make everything small, focused modules. I think Bristol should follow suit. I'm not saying that all dependencies are bad, but they should be used sensibly and when a majority of users require it.

You make a shockingly good argument, @MarkHerhold ;-) The other benefit of that setup is that we could stop dancing around requiring certain modules (like the loggly lib) and just make it a direct dep of the add-on module. Bristol will always have bundled targets and formatters, but we'll go with this approach for anything large or dependency-laden.
@dannyrscott and @Fluidbyte, would either of you be interested in maintaining a bristol-loggly and bristol-slack module (respectively)? If not, I can create repos to host them. Per #31, using an external target or formatter will be as simple as passing in a string with the name of the module to be require()d. So no matter what configuration wrapper you may already have in place, using one of these would be as simple as npm install --save bristol-loggly and then using "bristol-loggly" as the target type.


C# and VB Background Task project missing reference to .NET Core

When you download the IoT Core Project templates and create a new background task, the project won't compile. You need to open up project.json and add a reference to .NET Core.

The dependencies entry is missing NETCore 5. When you add this, the project will compile again.
"dependencies": {
  +		"Microsoft.NETCore.UniversalWindowsPlatform": "5.0.0"
},

Steps to reproduce:

Download and install the IoT Core Project Templates from the VS Gallery
Create a new C# Background Application (IoT) project.
Compile and observe the 41 build errors.
Open up project.json and observe that the dependencies section is empty
Add "Microsoft.NETCore.UniversalWindowsPlatform": "5.0.0" to the dependencies and build (everything is now good)

The dependency should be added (as it is for the other C#/VB UWP projects) when the template is instantiated.
Can you look at the versions you have installed in About Visual Studio?

Please reopen this if you continue seeing this issue!


cap-llm-plugin vs SAP Cloud SDK for AI

Could you please clarify the positioning of cap-llm-plugin versus the new  SAP Cloud SDK for AI ?
The 2 libraries look to me a bit overlapping specially when document grounding will be part of orchestration.

SAP Cloud SDK for AI is the official Software Development Kit (SDK) for SAP AI Core, SAP Generative AI Hub, and Orchestration Service.

I'll pick the summary from the Readme of SAP Cloud SDK for AI.
The SDK will support the grounding module of the orchestration soon, after the GA release of the grounding module.

Could you please clarify the positioning of cap-llm-plugin versus the new SAP Cloud SDK for AI ? The 2 libraries look to me a bit overlapping specially when document grounding will be part of orchestration.

cap-llm-plugin is relatively pertained to CAP development with the way you handle entities and columns, but you can always check both documents for more details to pick whatever fits your need.


Cannot login

I seem unable to login user either method, and I have tried several accounts. I simply get a 'Login Failed' message. Any ideas?

This issue was solved some time ago via e-mail, so I'm closing.


Dashboard link fixup

https://github.com/lowRISC/opentitan/pull/18052 with fixes for autogen.
fixes: https://github.com/lowRISC/opentitan/issues/17591

fixed CI failure and changed docstring.

Rebased onto master.

Rebased


Saving and Loading Models

As discussed with @CheukHinHoJerry, it seems that the enourmous effort of implementing (de-)serialization routines we have in ACE1 is not required at all now that we've adopted the Lux model.
Saving a potential should simply involve:

recording the model parameters
recording the means how the model was generated
recording the versions of relevant packages.

Loading a potential:

double-check version compatibility
load the parameters
generate the potential again, set the loaded parameters

@CheukHinHoJerry @wcwitt  -- what are your thoughts on this?

Sorry just realized I was tagged here. Where can I look to see how this would work?

There is no example implemented. I should post a demo somewhere.

... following up. The gist of the proposal is this:

a model consists of a script to generate it  (with the JSON interface, it would simply be the dictionary itself)
a parameter file
the manifest of the julia environment with with the model was originally generated and fitted.

Following discussion with @cortner and @iamveronika might be help to save the manifest with the potential too, possibly with this package https://julialang.github.io/TOML.jl/dev/.

SAVING A POTENTIAL
user specifies model via Dict / JSON
perform the fit
Create new dict:
    - user spec of model  (and also fitting parameters as meta-data)
    - parameters (as vector or as a named-tuple / Dict)
    - Dict containing the Manifest
Save this super-dict as JSON file (with option to zip)

LOADING A POTENTIAL
load super-dict from JSON
(maybe check compatibility with current environment, write warning if it has changed)
try to generate the model again from the model parameters
set the parameters  => model.

Further options:

instructions how to resolve the stored manifest if loading fails
add test set, e.g. a subset of the training data and check that original fitted potential and loaded potential evaluate to the same energies, forces, virials.

This was addressed in a direct commit to main (by accident)


cleanly reinstalled cluster & kargo, after performing sso login, kargo is in a loop refreshing page

Checklist

[x] I've searched the issue queue to verify this is not a duplicate bug report.
[x] I've included steps to reproduce the bug.
[x] I've pasted the output of kargo version.
[x] I've pasted logs, if applicable.

Description
I login and the page redirects to:
https://kargo.vfp-admin.k.home.net/token-renew?redirectTo=/
Then it redirects to:
https://kargo.vfp-admin.k.home.net/
Then it repeats without end
Screenshots

Steps to Reproduce

Version
helm chart 0.8.1
Logs
[kargo-api-6f69f9749c-9kt4b] time="2024-07-28T20:47:34Z" level=info msg="finished unary call" connect.duration="11.52µs" connect.method=GetPublicConfig connect.service=akuity.io.kargo.service.v1alpha1.KargoService connect.start_time="2024-07-28T20:47:34Z"
[kargo-api-6f69f9749c-9kt4b] time="2024-07-28T20:47:36Z" level=info msg="finished unary call" connect.duration="22.993µs" connect.method=GetPublicConfig connect.service=akuity.io.kargo.service.v1alpha1.KargoService connect.start_time="2024-07-28T20:47:36Z"

attempt to fix

went back to helm chart 0.7.1 and things started working, so maybe this is an oidc/login issue with 0.8.1?
i enabled log level DEBUG and didn't see anything new

+1 , same problem here

@rbreeze is this possible related to https://github.com/akuity/kargo/pull/2282?


Error selecting three day range

When you select a range of three days and select the date in between, nothing happens. The event continues to indicate that 3 days have been selected. The correct thing would be to clean the selection and select the date in between.

Feel free to submit a PR with a fix!

Gonna close this as duplicate, feel free to check the issue here: https://github.com/prolificinteractive/material-calendarview/issues/516


MongoDB ISODate limited to millisecond precision

Since we currently store the values for date_added and request_time as an ISODate we cannot store higher precision timestamps as required by these properties.
We should change it to use regular strings but it should not affect any of our current pipeline filter work.

After realizing another problem with saving timestamps as string. It was determined to use float as the underlying datatype and convert them into the proper format before sending the resource.

Present in both v1.0.0 (TAXII 2.0) and v2.0.0 (TAXII 2.1)


meshlab: 2020.07 -> 2020.12

Motivation for this change
Update Meshlab to last version.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116247 at 420b9550 run on aarch64-linux 1

1 package built successfully:

meshlab

2 suggestions:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/graphics/meshlab/default.nix:77:5:
   |
77 |     license = lib.licenses.gpl3;
   |     ^

warning: unused-argument
Unused argument: fetchpatch.
Near pkgs/applications/graphics/meshlab/default.nix:4:3:
  |
4 | , fetchpatch
  |   ^


Error while building game:

I was trying to run lime rebuild extension-webm windows and I got this error:

I have no idea how to code so this is minecraft enchanting table language to me.

did you download Visual studio ? follow the building.md for more instructions

Yeah, i dont know whats wrong

maybe i forgot about the cloning the repository part

im going to try re-installing visual and the components

im going to try re-installing visual and the components

https://download.visualstudio.microsoft.com/download/pr/67f7bfaa-2635-43d9-ba82-26564f458881/bb057808eeff7d02561cab636b32397e298c3e770ed278602c46d119fa2e4ee8/vs_BuildTools.exe

oh nice thanks ill try that

thanks a lot im pretty sure its working
either that or this error is going off the screen

It works now! Thanks a ton!

It works now! Thanks a ton!

Your welcome.


Empty column qualifiers should not be rejected

Relevant code is here for reads: https://github.com/googleapis/nodejs-bigtable/blob/13e3d769/src/chunktransformer.js#L251
It's completely valid to use a column like "fam:" which has no qualifier

@muraliQlogic, can you please take a look?

@dmmcerlean "fam":"" is fine, but "fam":[null] is not ok.  I believe the if statement checks for "fam":[null] rather than "fam":"".  We should add a unit test to confirm that.


The Discord invitation isn't working.

I'll take a look. I might've accidentally made it only for 24 hours.

Solved.


[KYUUBI #703] support hudi-0.9.0

Why are the changes needed?
Hudi has add Spark Sql Support in version 0.9.0.
This PR add Hudi-0.9.0 dependency and Unit Testing  #703
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Hi @simon824, can you take a look at the test failure?

at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.createHiveDataSourceTable(CreateHoodieTableCommand.scala:214)
	at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.createTableInCatalog(CreateHoodieTableCommand.scala:177)
	at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.run(CreateHoodieTableCommand.scala:73)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:70)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:68)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.executeCollect(commands.scala:79)
	at org.apache.spark.sql.Dataset.$anonfun$logicalPlan$1(Dataset.scala:228)
	at org.apache.spark.sql.Dataset.$anonfun$withAction$1(Dataset.scala:3687)

cc @pan3793, the data lakes are bound to hive metastore sometimes, are we good to use in-memory for them all?

at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.createHiveDataSourceTable(CreateHoodieTableCommand.scala:214)
	at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.createTableInCatalog(CreateHoodieTableCommand.scala:177)
	at org.apache.spark.sql.hudi.command.CreateHoodieTableCommand.run(CreateHoodieTableCommand.scala:73)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:70)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:68)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.executeCollect(commands.scala:79)
	at org.apache.spark.sql.Dataset.$anonfun$logicalPlan$1(Dataset.scala:228)
	at org.apache.spark.sql.Dataset.$anonfun$withAction$1(Dataset.scala:3687)

cc @pan3793, the data lakes are bound to hive metastore sometimes, are we good to use in-memory for them all?

Make sense.

*** RUN ABORTED ***
  java.lang.NoSuchMethodError: org.apache.hadoop.yarn.server.utils.BuilderUtils.newApplicationResourceUsageReport(IILorg/apache/hadoop/yarn/api/records/Resource;Lorg/apache/hadoop/yarn/api/records/Resource;Lorg/apache/hadoop/yarn/api/records/Resource;Ljava/util/Map;Ljava/util/Map;)Lorg/apache/hadoop/yarn/api/records/ApplicationResourceUsageReport;
  at org.apache.hadoop.yarn.server.resourcemanager.RMServerUtils.<clinit>(RMServerUtils.java:493)
  at org.apache.hadoop.yarn.server.resourcemanager.ResourceManager.serviceInit(ResourceManager.java:277)
  at org.apache.hadoop.service.AbstractService.init(AbstractService.java:164)
  at org.apache.hadoop.yarn.server.MiniYARNCluster.initResourceManager(MiniYARNCluster.java:348)
  at org.apache.hadoop.yarn.server.MiniYARNCluster.access$200(MiniYARNCluster.java:128)
  at org.apache.hadoop.yarn.server.MiniYARNCluster$ResourceManagerWrapper.serviceInit(MiniYARNCluster.java:497)
  at org.apache.hadoop.service.AbstractService.init(AbstractService.java:164)
  at org.apache.hadoop.service.CompositeService.serviceInit(CompositeService.java:108)
  at org.apache.hadoop.yarn.server.MiniYARNCluster.serviceInit(MiniYARNCluster.java:316)
  at org.apache.hadoop.service.AbstractService.init(AbstractService.java:164)
  ...

YARN dep conflicts?

Have a look at hudi-spark-bundle_2.12, I'm curious why hudi only relocate part of deps but left others(e.g. curator, prometheus, hbase, rocksdb) with the original package name in the fat jar? It leads to a very high probability of class conflict.
https://github.com/apache/hudi/blob/master/packaging/hudi-spark-bundle/pom.xml

It say Hudi 0.9.0 doesn't support Spark 3.1 https://github.com/apache/hudi/issues/3554

It say Hudi 0.9.0 doesn't support Spark 3.1 apache/hudi#3554

support Spark-2.4.3+ & Spark 3.x, https://hudi.apache.org/docs/quick-start-guide/
Maybe still have some bugs in some aspects.

Do we need to test spark 3.0 only for now?

Do we need to test spark 3.0 only for now?

yes

Codecov Report

Merging #994 (1313398) into master (b382ed0) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 1313398 differs from pull request most recent head ff2585c. Consider uploading reports for the commit ff2585c to get more accurate results

@@             Coverage Diff              @@
##             master     #994      +/-   ##
============================================
- Coverage     80.06%   80.06%   -0.01%
  Complexity       11       11
============================================
  Files           152      152
  Lines          5553     5552       -1
  Branches        644      644
============================================
- Hits           4446     4445       -1
  Misses          725      725
  Partials        382      382

Impacted Files
Coverage Δ

...la/org/apache/kyuubi/zookeeper/ZookeeperConf.scala
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Congratulation! The CI passed. Thanks to @simon824 for good work. But I need some time to have a deep look at the transitive deps pulled by Hudi before merging it.

thanks, @simon824, this change LGTM, leave it to @pan3793 for the last comfirmation

thanks, @simon824, this change LGTM, leave it to @pan3793 for the last comfirmation

OK, thanks a lot for your review.

Hudi use HFileBootstrapIndex by default, I think it's not required for our ut, can we overwrite it and test if it can work without hbase deps?
https://github.com/apache/hudi/blob/release-0.9.0/hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java#L144-L147

I still see hudi-spark_2.12 pulls jackson-module-scala_2.11 into classpath

Stand on the Spark side, HUDI is a data source implementation, or just call it a plugin.
Thus, for the transitive dependencies of HUDI, like Hadoop, Curator, Parquet, Avro, ORC, Hive, we must RESPECT the Spark bundled versions, as Kyuubi support multi Spark versions, we need to separate those deps version by different Spark versions using maven profiles. e.g. use parquet 1.10.0 for Spark 3.0/3.1, but use parquet 1.12.0 for Spark 3.2(coming soon).
Besides on Spark, we should also respect the Kyuubi deps version, assume you are using IDEA, please open the External Libraries, there are lots of jars which has multi versions that exist in the classpath, which means a high risk of class conflicts.

it worths another PR to solve, this PR has met its original goal. thanks, merged to master.

Hudi uses HFileBootstrapIndex by default, I don't think it's required for our ut, can we overwrite it and test if it can work without hbase deps?
https://github.com/apache/hudi/blob/release-0.9.0/hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java#L144-L147

ok，i will test and fix it.


Pass article limit as CLI argument

The article limit is currently hard-coded to 20 articles. Is there a reason for this limit?
It would be great if we could pass this limit as a CLI argument to allow longer feeds.

No reason, just thought it's a reasonable amount since those are getting scraped regularly anyways. I'll add an env var to configure it.

Thanks, the env var is even better.

Just released 1.3.0 which allows overriding the article limit.


Convert From LESS to postCSS

What does this PR do? (please provide any background)
This converts the UI toolkit from LESS to postCSS.
What tests does this PR have?
N/A
How can this be tested?
Run the docs locally and compare to http://hungrygeek.holidayextras.co.uk/ui-toolkit/
Screenshots / Screencast
What gif best describes how you feel about this work?

Developer Definition of Done/Quality Checklist (for PR author to complete BEFORE code review):

[x] I have checked the contributing document and I'm happy for this to be reviewed.

Software Engineer or Developer review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve checked all the tests are passing.

Software Engineer or project guru review:

[ ] I’ve witnessed the work behaving as expected (this could be on the authors machine or screencast).
[ ] I’ve checked for coding anti-patterns.
[ ] I’ve checked for appropriate test coverage.
[ ] I’ve checked all the tests are passing.

I'm glad someone is getting to try out PostCSS.
I have a question (which could either be massively naive or completely fundamental).
How do you propose we theme this with UI-brand-HX?
I may be wrong, but I don't think they can be compiled together, can they?

If we use the CSS before it's transpiled then we can theme it by updating the CSS vars.

Interesting! I'll need a lesson :-)

we need to get the CSS - Less Vars aligned before this goes live, as UI-Toolkit will need to work with existing less overbrands.

@jodiedoubleday ok, i'll take a look. shouldn't be too difficult, converting the variables is just @foo to --foo when declaring and var(--foo) when using.
i've setup the compiling here so should be able to copy in to it's own module like grunt-less-brading easy enough.


[FEATURE] Adjust comment on automatic textfiles

It never saved the file from being overridden anyway.
Add hint as commment about where code snipppet is comming from
Warning: There is one error in a test that I couldn't solve
solves #228, #71

Solved test issue and naming things.


Bartender not working

Please provide the environment you discovered this bug in.
I have just finished all of my updates and logged into wow for the first time in a couple of months.  I have an issue with my spartan UI, that none of my bars are visible, Even when I try and move a spell to the bar. The outline for the UI is visible but no bar. I have reinstalled both addons and still no joy.
Which area/package is the issue in?
Don't know / other
Description
I have just finished all of my updates and logged into wow for the first time in a couple of months.  I have an issue with my spartan UI, that none of my bars are visible, Even when I try and move a spell to the bar. The outline for the UI is visible but no bar. I have reinstalled both addons and still no joy.
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

Spartan is not SpartanUI 😅
Please report your issue(s) at https://github.com/Wutname1/SpartanUI/issues

@Pouchy20 - closing as I think you meant this for a different repo as mentioned by @ajitzero


Add ability control SAML assertion query profile requests types from UI

Description
SAML Assertion Query\Request Profile consists 5 types of requests

AssertionIDRequest
SubjectQuery
AuthnQuery
AttributeQuery
AuthzDecisionQuery

As of now it can only enable\disable assertion query\request profile for the given service provider. So it can enable all these 5 request types as a single unit or disable all 5 request types.
From SAML service provider configuration UI, it can add another field to restrict what are the request types it needed to be allowed for this service provider when assertion query\request profile enabled.
SSO Config Admin service already has a flag (SupportedAssertionQueryRequestTypes) to set this, but it needs to fix UI and request validators to make this flag used in the runtime.
Type
Improvement
Priority
Normal
Affected version
5.5.0, 5.8.0
Component/s
saml2-sso
Severity
Major
Estimated Complexity
Moderate
Label/s
Type/Improvement, Type/UX, Component/SAML, Affected/5.5.0,  Affected/5.8.0, Severity/Major, Complexity/Medium, Priority/Normal
Moved from https://wso2.org/jira/browse/IDENTITY-5506

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Browser excel output docs don't match types

https://github.com/javascriptdata/danfojs/blob/a8d14d45bca2a50836a6e2d6ec1a77582c42993a/src/danfojs-base/shared/types.ts#L409
https://danfo.jsdata.org/api-reference/dataframe/dataframe.to_excel#convert-dataframe-to-excel-and-download-the-file-in-a-browser
let data = {
    Abs: [20.2, 30, 47.3],
    Count: [34, 4, 5],
    "country code": ["NG", "FR", "GH"],
};

let df = new dfd.DataFrame(data);

df.toExcel({ fileName: "testOut.xlsx", download: true }); //deprecated, docs have download, but types appear not to
dfd.toExcel(df, { fileName: "testOut.xlsx", download: true}); //current, docs have download, but types appear not to

When I try to run it with  download: true, I get the following error: Uncaught Error: Module "fs" has been externalized for browser compatibility. Cannot access "fs.writeFileSync" in client code. but I'm not sure if that's a danfo issue or a vitejs issue at the moment. dfd.toCSV works as expected.

Thanks for the catch. Docs have been updated, as you do not need the download argument.
For browser downloads, the toExcel function calls xlsx's internal writeFile function, and this will detect the environment and automatically open a download file window.


Make run_jenkins script forward arguments to run_tests.py

This is to allow customizing the run_jenkins environment per-project from Jenkins itself

./tools/run_tests/run_tests.py --use_docker -t -l c -c dbg -x report.xml bootstrapped
usage: run_tests.py [-h]
[-c {all,asan,asan-noleaks,dbg,gcov,helgrind,memcheck,msan,opt,tsan,ubsan} [{all,asan,asan-noleaks,dbg,gcov,helgrind,memcheck,msan,opt,tsan,ubsan} ...]]
[-n RUNS_PER_TEST] [-r REGEX] [-j JOBS] [-s SLOWDOWN] [-f]
[-t] [--newline_on_success]
[-l {all,build,c,c++,csharp,gyp,node,objc,php,python,ruby,sanity} [{all,build,c,c++,csharp,gyp,node,objc,php,python,ruby,sanity} ...]]
[-S] [--use_docker] [--allow_flakes] [-a ANTAGONISTS]
[-x XML_REPORT]
run_tests.py: error: unrecognized arguments: bootstrapped

Blah. This may take a few rounds :)

Build was aborted

Ready for review.


Circular layout + barplot

Would be nice if metadata can also be painted on bars on the tips
This will require a circular layout though.

May have to add this as an option to the dropdown menu
Maybe replace the metadata tab with the barplot?


App Appearance Doesn’t Persist Between App Launches

If the app appearance is set in the preferences and the app is closed, it will reset back to the system default.

Changing the NSApp.appearance to SwiftUI ColorScheme to allow easier code to work with and cross platform compatibility.


project_dir interaction with cache directory [undefined]

ref https://github.com/fractal-analytics-platform/fractal-server/issues/1934
from https://github.com/fractal-analytics-platform/fractal-server/issues/1934#issuecomment-2446689635

How does it interact with the cache directory? => use project_folder as the cache directory as well

Yes
project_dir/.fractal_cache as the new cache directory
=> we can get rid of cache_directory variable
Let's do this once we've set up project_dir usage in all deployments


[MRG] Add ability to record voltages and currents from all sections

This is an enhancement I've been meaning to add for awhile. Not sure if this is going to cause a lot of problems in terms of data transfer, but we'll see!
The way things are currently written are very redundant with the somatic recording functionality. Once I know this is working, the record_vsoma and record_isoma options to simulate_dipole can use the same code as here by restricting the section names considered to only be the 'soma'.

Codecov Report

Merging #502 (f8d9727) into master (1f550b1) will increase coverage by 0.09%.
The diff coverage is 96.72%.

:exclamation: Current head f8d9727 differs from pull request most recent head 861c05e. Consider uploading reports for the commit 861c05e to get more accurate results

@@            Coverage Diff             @@
##           master     #502      +/-   ##
==========================================
+ Coverage   91.69%   91.79%   +0.09%
==========================================
  Files          21       22       +1
  Lines        4204     4239      +35
==========================================
+ Hits         3855     3891      +36
+ Misses        349      348       -1

Impacted Files
Coverage Δ

hnn_core/params_default.py
100.00% <ø> (ø)

hnn_core/dipole.py
91.56% <66.66%> (-0.94%)
:arrow_down:

hnn_core/cell.py
97.34% <100.00%> (+0.13%)
:arrow_up:

hnn_core/cell_response.py
84.53% <100.00%> (ø)

hnn_core/network_builder.py
93.89% <100.00%> (+0.12%)
:arrow_up:

hnn_core/parallel_backends.py
83.05% <100.00%> (+0.83%)
:arrow_up:

hnn_core/__init__.py
100.00% <0.00%> (ø)

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

@rythorpe @jasmainak took a little sabbatical from this PR but everything seems to be working now! Might be a nice add-on to the 3.0 :wink:

Cursorily looks fine barring the tiny comment. I trust you tested it properly :)

good to go from your end @ntolley ?

All good with me once the CI's are green! Just pushed the last commit addressing your comment

Thanks @ntolley ! great stuff 🥳 🥳 🥳


how to put a angular component into a map infowindow

hi, @tomwayson. I recently met a new problem. In some case, a map infowindow may be complex, so I want to create a ng2 component to define the infowindow content. the question is how can I put this compnent into the infowindow?
any help is much appreciated!  thanks.

@odoe @lobsteropteryx @jwasilgeo  any help is much appreciated! thanks.

My advice would be to not use an info window. If the content is that complex, something that you can't/don't want to handle w/ the ArcGIS API's own templating and/or widgets, then I would suggest handling the feature select/click events and opening the component somewhere else on the screen (along side the map, in a modal, etc). Where would depend on things like the layout of the view containing the map, what kind of mobile support you require, etc.
I'm not saying you can't do it w/ a popup. You probably can handle the map's popup shown event, and mount a component to something like an empty div in the popup's content. However, you're going to spend a lot of effort managing the widget/component lifecycles. The specifics would depend on what version of the ArcGIS API you're using.

thanks your advice, I will try it.  by the way, I google a similar question which link is http://stackoverflow.com/questions/38614305/angular-2-insert-dom-into-infowindow-element-that-was-created-by-google-maps-v3

The approach outlined in that SO answer should work for an ArcGIS info window, or at least work as well as it would for a Google Maps info window. However, that code already looks complicated, and it doesn't even deal w/ handling the map event that would call that code, or how to communicate from that component to the rest of the angular app and/or the map, nor how to destroy the component you created on the fly. That all might be trivial, or it might be a mess.
Personally, if at all possible I would still prefer to have the map component emit an event that a feature has been clicked or selected and pass the feature data to angular and let it render the "info window" content into the component tree that it already controls (outside the map).


Pickling to Code Quotations for Deserialization in a Different Runtime

I'm trying to use FsPickler to send code quotations over a named pipe from a server running inside of FSI to a client running inside of the Unity3d engine.  The client has a quotation evaluator that it uses to execute code against the engine's internal API.  The issue comes from the reflection based loading of the FSharp.Core and mscorlib at different version numbers
Here's an example of the different JSON produced
There are also some differences in the instance and id numbers, and the FSI version having a third Item = null in  a few places, but I'm not sure how relevant those are to block being deserialized effectively.
Is there a way to specify the target runtime and target version of FSharp.Core for serialization? and if not could this be incorporated as a future feature?

This is a common issue typically solved with binding redirects. Does the Unity engine provide something similar?

Closing this old question/discussion


ReplicaSyncupManager: add separate target for downward state transition

Adding a separate catch up target for downward state transition.

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (bf8aea2) 53.33% compared to head (b3dc911) 19.59%.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2620       +/-   ##
=============================================
- Coverage     53.33%   19.59%   -33.74%
+ Complexity     7773     2780     -4993
=============================================
  Files           806      806
  Lines         65227    65230        +3
  Branches       7950     7950
=============================================
- Hits          34788    12781    -22007
- Misses        27431    51265    +23834
+ Partials       3008     1184     -1824

Files
Coverage Δ

...java/com/github/ambry/config/ClusterMapConfig.java
97.01% <100.00%> (+0.09%)
:arrow_up:

...ub/ambry/clustermap/AmbryReplicaSyncUpManager.java
6.77% <0.00%> (-0.04%)
:arrow_down:

... and 384 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


TLS ECC

ToDo: Move pr to zeronet-crypto

Moved to https://github.com/ZeroNetJS/zeronet-crypto/issues/2


Device names not showing up

Sadly the device names are not showing in the list and because of that, I can't control any device. The device name just gets set to an empty string and it always controls the first hardware input device

Sure here you go. It's not that problematic because I can still use the advanced commands to control VM.

Ah...I think I see the issue.
A quick workaround you can do is to set the names for each fader (right-click where you see "Fader Gain" on each slider, type a name and press Enter).
I'll see if I can find a good way of fixing this oversight.

Oh, now everything is working as expected. Thank you!


NaN predictions for customers with 0 frequency and 0 recency

Problem
I'm using BetaGeoFitter and have observed that in some cases the model produces NaN predictions for customers with frequency=0 and recency=0. This occurs when a customer makes only one transaction (their first) and never returns.
Reason
The issue lies in the input to the logarithm function in the conditional_expected_number_of_purchases_up_to_time method. Specifically, it is this line. The term _c becomes negative if the fitted model parameters a and b sum to less than 1, as _c = a + b + x - 1 and x = 0 in this case. As a result, the output of hyp2f1(_a, _b, _c, _z) becomes negative, and taking the log of a negative number produces a RuntimeWarning and a resulting NaN prediction value.
RuntimeWarning: invalid value encountered in log
  result = getattr(ufunc, method)(*inputs, **kwargs)

Expected behavior
Ideally, the model produces predictions for those users with a frequency and recency of 0. Alternatively, it raises a warning that a + b <= 1 if these parameter values are not expected.
Context: Data details
I fitted the model to a cohort of customers with a high T value, between 1257-1287. Approximately 70% of the customers in this cohort have a frequency of 0 and recency of 0. The model is then used to predict for more recent customers whose T value may range between 0-1287 (that's when the NaN predictions occur). This seems to be in line with the implementation recommendation in the paper: "the model should be applied separately to customer cohorts defined by the time (e.g., quarter) of acquisition".

Hey @pavlo-seimskyi ,
Development on lifetimes ceased 3 years ago. Here's the link to the successor library:
https://github.com/pymc-labs/pymc-marketing
I appreciate the deep dive you've done and information provided on this matter. The NaN predictions are due to the high T value, as described by the authors of the BG/NBD model in this PDF:
http://brucehardie.com/notes/027/bgnbd_num_error.pdf
The same PDF proposes a reformulation of the likelihood function to fix this issue, which we've implemented in pymc-marketing:
https://github.com/pymc-labs/pymc-marketing/blob/main/pymc_marketing/clv/models/beta_geo.py#L143


Features set computer

As we discussed on Discord and IRL, I implemented a system that, given a set of available Features, computes the minimal set of Features needed by a WhitelistChecker to accept a given program. I also renamed the Features object into PredefFeatures, for clarity.
To use this system, we need to be able to list available Features. I can think of 3 ways of doing that:

Simply let the user manually build the list of the Features that they defined, and define one such list in PredefFeatures. This is the most obvious solution, but if the user defines a lot of Features it becomes really easy to forget to include a Feature in the list;
Create a FeaturesProvider trait in the syntactic package that should be implemented by the classes in which Features are defined (including PredefFeatures), and use reflection in FeaturesProvider to get all the Features defined in the class (i.e. the singleton objects that inherit the Feature trait) and automatically build a list with all of them;
Create a FeaturesProvider trait in the syntactic package that should be implemented by the classes in which Features are defined (including PredefFeatures), and use an annotation to mark the Features that should be included in the list (again, the list would be obtained by using reflection in the FeaturesProvider trait).

The current implementation in this PR uses the third solution. I think that the first one is very impractical and that it is worth it automatically building the list of defined Features, but this implies using reflection and I am wondering if that is a problem (as far as I have tested it works well). Also, in case that is acceptable, is it better to include in the list all the Features that are defined in the class (solution 2) or to use an annotation (solution 3)? I wrote a comment in the implementation of FeaturesProvider that shows the (small) change that would be needed to switch from solution 3 to solution 2.

Looks good, but could you integrate this component more tightly with the project (e.g., make it simple for users to get the minimal set of features to enable as part of the result returned by a checker, or something like that)?

Integrating the minimal set of features to the CheckResult would require to implement specialized versions of CheckResult.Invalid (one for the blacklist checker and another one for the whitelist checker), because when outputted by a blacklist checker a CheckResult has obviously no features to propose. So I don't know if that is a good idea... But if the CheckResult is an Invalid then an instance of FeaturesSetComputer can run the minimalFeaturesSetToResolve method on the List[Violation] contained in the Invalid. This method returns the minimal set of features needed to accept the nodes reported in the Violations, isn't that sufficient?

Integrating the minimal set of features to the CheckResult would require to implement specialized versions of CheckResult.Invalid (one for the blacklist checker and another one for the whitelist checker), because when outputted by a blacklist checker a CheckResult has obviously no features to propose. So I don't know if that is a good idea...

I agree it’s not a good idea.

. But if the CheckResult is an Invalid then an instance of FeaturesSetComputer can run the minimalFeaturesSetToResolve method on the List[Violation] contained in the Invalid. This method returns the minimal set of features needed to accept the nodes reported in the Violations, isn't that sufficient?

Yes! But from the point of view of an end user, how do I achieve all of that?

Yes! But from the point of view of an end user, how do I achieve all of that?

The end user should instantiate a FeaturesSetComputer, giving it a set containing all available features, and run minimalFeaturesSetToResolve  on the list of Violations. But yes maybe I can add a method that does this:
extension(invalid: CheckResult.Invalid){
    def missingFeaturesAmong(availableFeatures: List[Feature]): Option[Set[Feature]] = {
      new FeaturesSetComputer(availableFeatures).minimalFeaturesSetToResolve(invalid.violations)
    }
}

Is that what you mean?

Actually I realize that my comment does not really apply to this PR, but more to the eventual sbt plugin or command-line interface for the project. So, all is good regarding this PR, thanks!


Control camera remotely

Hi,
I have a Tapo C200 that configured which DDNS and Port Forwarding, so I have no problem streaming in RTSP with VLC. However when I use the domain in host parameter (e.g. mydomain.ddns.net:554), It throw an error:
Traceback (most recent call last):
  File "c:\Users\ThoaiNguyen(Intern)\Documents\tapo_controller\testt.py", line 7, in <module>
    tapo = Tapo(host, user, password)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 41, in __init__
    self.basicInfo = self.getBasicInfo()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 232, in getBasicInfo
    return self.performRequest(
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 95, in performRequest
    self.ensureAuthenticated()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 61, in ensureAuthenticated
    return self.refreshStok()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 74, in refreshStok
    res = requests.post(
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\requests\api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\requests\api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\requests\sessions.py", line 587, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\requests\sessions.py", line 701, in send
    r = adapter.send(request, **kwargs)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\requests\adapters.py", line 563, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='abcxyz.ddns.net', port=554): Max retries exceeded with url: / (Caused by SSLError(SSLError(1, '[SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:1129)')))

How can I fix it? Thank you

ALso, when I remove the port, it says: `
C:\Users\ThoaiNguyen(Intern)\Documents\tapo_controller>"C:/Program Files/Python39/python.exe" c:/Users/ThoaiNguyen(Intern)/Documents/tapo_controller/testt.py
Traceback (most recent call last):
  File "c:\Users\ThoaiNguyen(Intern)\Documents\tapo_controller\testt.py", line 7, in <module>
    tapo = Tapo(host, user, password)
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 41, in __init__
    self.basicInfo = self.getBasicInfo()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 232, in getBasicInfo
    return self.performRequest(
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 95, in performRequest
    self.ensureAuthenticated()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 61, in ensureAuthenticated
    return self.refreshStok()
  File "C:\Users\ThoaiNguyen(Intern)\AppData\Roaming\Python\Python39\site-packages\pytapo\__init__.py", line 80, in refreshStok
    raise Exception("Invalid authentication data")
Exception: Invalid authentication data

There is no port 554 on the camera. Remapping of ports is not supported in this integration.
Port forwarding your camera to the internet is also a really bad idea so this will not be considered as a feature.

Camera control communicates via port 443
554 is only for rtsp.
Opening your camera to the internet is a terrible idea.


In documentation and code help statements change animals to individuals

We often use animals, but the programme can be used for plant, humans, etc. too!

@AprilYuZhang I think you have done this - have you done it across all of the documentation? If yes, then please close the issue.


cromwell: torque qsub fix

removed invalid flags from cromwell torque job submission config

Hi Brad,
Sorry about the mistake, The changes are supposed to be in the torque section.
As far as I know, torque has no way of setting the shell from the qsub command. However, I saw the submission scripts have a correct bash shebang, I think that should be enough for torque.
as an alternative to the -l wd flag you could use -d "workdir". I suppose that's the torque equivalent.
for more flags etc, see http://docs.adaptivecomputing.com/torque/archive/4-1-1/Content/topics/commands/qsub.htm
Cheers
M

I fixed the args and restored the PBSpro settings.

Brilliant, thanks Matthias. Are we able to keep the /usr/bin/env bash invocation once we fix the -l to -d argument? I'd like to keep this in, if you can confirm it works, so that we're consistent in how we prepare these scripts between different schedulers. Thanks again for helping with this.

The -Sflag may be used the set the shell for a job, but the qsub command can't handle the whitespace in /usr/bin/env bash. If you're okay with using /bin/bash I suppose we're good to go.
M

Thanks, if the /usr/bin/env invocation doesn't work happy to have the simplified version. Thank you again for testing and the fix.


TinyFPGA seemingly accepts data even when uart_out_ready = 0

I'm not familiar with the internals of USB Serial whatsoever, and fairly new to FPGAs, so my apologies if I'm missing something obvious. I'm working on a protocol for processing blocks of data on an FPGA (load in some data, FPGA processes it, then returns the output). I'm using a TinyFPGA BX. To get something up and running, I started with your usbserial_tbx.v file (which, when compiled and uploaded, works as expected). As a first step in implementing my protocol, I separated the in and out wires, defining them like so:
// `in` is the pipeline *in* to the host (*out* of the FPGA)
reg [7:0]  uart_in_data;
reg        uart_in_valid;
wire       uart_in_ready;

// `out` is the pipeline *out* of the FPGA (*in* to the host)
wire [7:0] uart_out_data;
wire       uart_out_valid;
reg        uart_out_ready;

As I understand from your description, data is passed in the in or out direction if and only if _ready and _valid for that direction are both 1. Since I'll be setting uart_out_ready and uart_in_valid depending on the state of the device, I figured I'd try the baby step of simply setting uart_out_ready to be 0, and not implementing any further logic. I changed the reg uart_out_ready; line to reg uart_out_ready = 0;, then compiled and uploaded the program with make prog.
I then, using pySerial, attempted to write to the device:
import serial
ser = serial.Serial("/dev/serial/by-id/usb-1d50_6130-if00")
bytes_written = ser.write(b"a")
print(bytes_written)

Strangely, this returned 1. In a REPL, it seems like I can write 1 byte two times before the command hangs. I was also able to write 8, or 20 bytes, however I could only issue the .write() once in those cases. I checked the .in_waiting and .out_waiting attributes, both of which returned 0 after writing, so it doesn't seem to be a buffering issue.
Thanks for your time, and for your effort in making FPGAs more accessible!

A confession: this gateware does not do well when held up for more than a handful of clocks.  All the projects I've used it with are dataflow oriented and consume characters faster than they can be produced by the USB interface at 12Mb/s.  Any time someone tries to use this code with a relatively slow SoC, this is an issue.
This means that a test which just stops characters flowing will result in disappointment!  Sorry!
Maybe another test where you consume the chars as fast as they are sent?   Try sending a few bits out to an LED.  Very obvious when new characters arrive...
My boilerplate code for USB includes a FIFO for the rare cases when the downstream gateware might hold the stream up.  Then all is well.
When I'm next overhauling the insides of this code, I'll implement back-pressure properly.

A confession: this gateware does not do well when held up for more than a handful of clocks.  All the projects I've used it with are dataflow oriented and consume characters faster than they can be produced by the USB interface at 12Mb/s.  Any time someone tries to use this code with a relatively slow SoC, this is an issue.
This means that a test which just stops characters flowing will result in disappointment!  Sorry!
Maybe another test where you consume the chars as fast as they are sent?   Try sending a few bits out to an LED.  Very obvious when new characters arrive...
My boilerplate code for USB includes a FIFO for the rare cases when the downstream gateware might hold the stream up.  Then all is well.
When I'm next overhauling the insides of this code, I'll implement back-pressure properly.


[ISSUES #909] Support AdminBrokerProcessor get_consumer_connection_list

Which Issue(s) This PR Fixes(Closes)

Fixes #909
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


Static Field

This is very useful in update case for example where the user does not have enough permissions to update this fields or just the fields should not be edited in any way.

Hi @petyots ,
As I mentioned in the comment, I don't think this is needed, you can easily add  the "disabled" attribute to the "text" field. Trying to keep it simple here.
Sorry for not merging this PR, I really do appreciate it.
Cheers!


Poor query performance cause by too high schedule time when deployed with k8s

Steps to reproduce the behavior
sql
TPCH 1000G
Q2
select count(*) from (
select  s_acctbal, s_name,  n_name,  p_partkey,  p_mfgr,  s_address,  s_phone,  s_comment
from  part,  partsupp,  supplier,  nation,  region
where
p_partkey = ps_partkey
and s_suppkey = ps_suppkey
and p_size = 15
and p_type like '%BRASS'
and s_nationkey = n_nationkey
and n_regionkey = r_regionkey
and r_name = 'EUROPE'
and ps_supplycost = (
select  min(ps_supplycost)
from  partsupp,  supplier,  nation,  region
where
p_partkey = ps_partkey
and s_suppkey = ps_suppkey
and s_nationkey = n_nationkey
and n_regionkey = r_regionkey
and r_name = 'EUROPE'
)
order by  s_acctbal desc, n_name,  s_name,  p_partkey
) as t1;
Test enviroment
3 fe and 3 be, 84 core 350g, Linux version 5.4.119, gcc version 8.5.0 20210514 (Red Hat 8.5.0-10)
Both k8s deployment and physical cluster deployment have the same physical hardware resources.
Query performance
k8s deployment with best-effort

Physical cluster deployment

Profile
profile-k8s.txt
profile-physical-cluster.txt
k8s deployment with best-effort

Physical cluster deployment

Flame Graph
k8s deployment with best-effort

Physical cluster deployment

found that 22% cpu overhead show at poll thread to schedule pipeline_driver when deployed with k8s whereas physical cluster deployment show 1%~ cpu overhead.
StarRocks version (Required)
2.5.4-TQ-0.6 dbc1950
Can someone offer some clues or provide any better way to analyze this problem?

Can you add some pod information which be belongs to?

Can you add some pod information which be belongs to?

kubectl get pod -n <namespace> <pod-name> -o yaml
kubectl describe pod -n <namespace> <pod-name>

apiVersion: v1
kind: Pod
metadata:
annotations:
apps.kruise.io/runtime-containers-meta: '{"containers":[]}'
lifecycle.apps.kruise.io/timestamp: "2023-05-29T02:54:32Z"
prometheus.io/path: /metrics
prometheus.io/port: "8040"
prometheus.io/scheme: http
prometheus.io/scrape: "true"
creationTimestamp: "2023-05-29T02:54:32Z"
generateName: xxx
labels:
app.kubernetes.io/component: be
app.starrocks.ownerreference/name: xxx
controller-revision-hash: xxx-be-68bf545df9
lifecycle.apps.kruise.io/state: Normal
statefulset.kubernetes.io/pod-name: xxx
name: xxx
namespace: starrocks
ownerReferences:

apiVersion: apps.kruise.io/v1beta1
blockOwnerDeletion: true
controller: true
kind: StatefulSet
name: xxx
uid: 8cd14137-34ce-4a13-b71e-3ab6de2ae834
resourceVersion: "1257272330"
selfLink: /api/v1/namespaces/starrocks/pods/xxx
uid: 03a001ca-91b4-4202-8e8a-337ebb444672
spec:
affinity:
nodeAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: node.info.kubernetes.io/instance-type
operator: In
values:
- IT5.21XLARGE320
podAntiAffinity:
requiredDuringSchedulingIgnoredDuringExecution:

labelSelector:
matchExpressions:

key: app.kubernetes.io/component
operator: In
values:

fe
topologyKey: kubernetes.io/hostname
containers:

args:

$(FE_SERVICE_NAME)
command:
/opt/starrocks/be_entrypoint.sh
env:
name: POD_NAME
valueFrom:
fieldRef:
apiVersion: v1
fieldPath: metadata.name
name: POD_NAMESPACE
valueFrom:
fieldRef:
apiVersion: v1
fieldPath: metadata.namespace
name: COMPONENT_NAME
value: cn
name: FE_SERVICE_NAME
value: xxx-service
name: FE_SERVICE_CLB_IP
value: <IP_ADDRESS>
name: FE_QUERY_PORT
value: "9030"
name: HOST_TYPE
value: FQDN
name: USER
value: root
name: SUB_DOMAIN
value: xxx
name: CONFIGMAP_MOUNT_PATH
value: /opt/starrocks/be/conf
image: xxx
imagePullPolicy: IfNotPresent
lifecycle:
preStop:
exec:
command:
- /opt/starrocks/be_prestop.sh
livenessProbe:
failureThreshold: 3
periodSeconds: 5
successThreshold: 1
tcpSocket:
port: 9050
timeoutSeconds: 1
name: be
ports:
containerPort: 9060
hostPort: 9060
name: be-port
protocol: TCP
containerPort: 8040
hostPort: 8040
name: webserver-port
protocol: TCP
containerPort: 9050
hostPort: 9050
name: heartbeat-port
protocol: TCP
containerPort: 8060
hostPort: 8060
name: brpc-port
protocol: TCP
readinessProbe:
failureThreshold: 3
periodSeconds: 5
successThreshold: 1
tcpSocket:
port: 9060
timeoutSeconds: 1
resources:
limits:
cpu: "84"
memory: 350Gi
requests:
cpu: "84"
memory: 350Gi
startupProbe:
failureThreshold: 60
periodSeconds: 5
successThreshold: 1
tcpSocket:
port: 8060
timeoutSeconds: 1
terminationMessagePath: /dev/termination-log
terminationMessagePolicy: File
volumeMounts:
mountPath: /opt/starrocks/be/log
name: be-log
mountPath: /opt/starrocks/be/storage
name: be-storage
mountPath: /opt/starrocks/be/conf/be.conf
name: be-config
subPath: be.conf
mountPath: /var/run/secrets/kubernetes.io/serviceaccount
name: xxx
readOnly: true
dnsPolicy: ClusterFirstWithHostNet
enableServiceLinks: true
hostNetwork: true
hostname: xxx
nodeName: xxx
preemptionPolicy: PreemptLowerPriority
priority: 0
readinessGates:

conditionType: InPlaceUpdateReady
conditionType: KruisePodReady
restartPolicy: Always
schedulerName: batch-scheduler
securityContext:
fsGroup: 1000
fsGroupChangePolicy: OnRootMismatch
serviceAccount: default
serviceAccountName: default
subdomain: xxx
terminationGracePeriodSeconds: 120
tolerations:
effect: NoExecute
key: node.kubernetes.io/not-ready
operator: Exists
tolerationSeconds: 300
effect: NoExecute
key: node.kubernetes.io/unreachable
operator: Exists
tolerationSeconds: 300
volumes:
hostPath:
path: /data13/xxx/sr/be-log
type: DirectoryOrCreate
name: be-log
hostPath:
path: /data13/xxx/sr/be-storage
type: DirectoryOrCreate
name: be-storage
configMap:
defaultMode: 420
name: be-config
name: be-config
name: xxx
secret:
defaultMode: 420
secretName: xxx
status:
conditions:
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:32Z"
status: "True"
type: KruisePodReady
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:32Z"
status: "True"
type: InPlaceUpdateReady
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:32Z"
status: "True"
type: Initialized
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:44Z"
status: "True"
type: Ready
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:44Z"
status: "True"
type: ContainersReady
lastProbeTime: null
lastTransitionTime: "2023-05-29T02:54:32Z"
status: "True"
type: PodScheduled
containerStatuses:
containerID: xxx
image: xxx
imageID: xxx
lastState: {}
name: be
ready: true
restartCount: 0
started: true
state:
running:
startedAt: "2023-05-29T02:54:33Z"
hostIP: <IP_ADDRESS>
phase: Running
podIP: <IP_ADDRESS>
podIPs:
ip: <IP_ADDRESS>
qosClass: Guaranteed
startTime: "2023-05-29T02:54:32Z"

Please provide the svg file of the above two flame graphs

svg file upload here is auto-transformed to a static image, please modify suffix to svg
perf_physical_cluster.txt
perf_k8s.txt

Thanks for your update. And here's another question, how many nodes(physical machines) are there in your k8s cluster? does each backend be deployed to different nodes?

Thanks for update. I'll setup a similar environment to see if I can reproduce this problem and to figure out the reason.

Thanks for update. I'll setup a similar environment to see if I can reproduce this problem and to figure out the reason.

thank you for your time~

Thanks for update. I'll setup a similar environment to see if I can reproduce this problem and to figure out the reason.

any update？

I have conduct a experiment at my local environment
starrocks-version: 2.5.4
k8s-version: v1.25.0
tpch_100g

Query
starrocks-in-host-machine
starrocks-in-k8s
diff(k8s - host)

Q1

Q1

Q1

Q1

Q1

Q1

Q1

Q1

Q1

Q1

Q1


how to reposition the elements in vue2-datatable

Here is a screenshot of the table i hope to achieve. using vue2-datatable as a requirement all tables must be presented exactly the same way. this used datatable.net but unfortunately vue doesn't play well with it. so i discovered your system and now, i want to achieve this look.. but not seeing slots where i can include the elements without having to rewrite the index of this datatable component. which seems like a rather messy approach to me..
How is this expected to be handled. thanks

no other ways, you have to move https://github.com/OneWayTech/vue2-datatable/blob/master/src/index.vue#L17 to L4

i got to sleep, it late night now in China, good night bro

Cheers

Maybe using named slots.. can be a good way to intoduce this customization behavior..

pardon me for not meeting everyone's appetite
https://onewaytech.github.io/vue2-datatable/doc/#/en/DIY


Advices:  some description about Middleware orders in the docs.

in the docs, cann't find some detail about  order of middleware .

They will be executed in the order you add them. See implementation: https://github.com/encode/starlette/blob/master/starlette/applications.py


Remove duplicate msg word and condition inside the function doc

Hi Apache Team,
I'm thrilled to submit my third contribution to Apache! I've addressed the issue of duplicate word inside function comment in the code. Paying attention to such details is vital for code quality, and I'm committed to enhancing the project.
-- MAX

Thanks @BaseMax @quic-sanirudh


[CXF-7430] The method logInputStream of the LoggingInInterceptor class passes txt/plain content type to the writePayload method if the payload content was truncated.

The method logInputStream of the LoggingInInterceptor class passes txt/plain content type to the writePayload method if the payload content was truncated.

The method logInputStream of the LoggingInInterceptor class passes text/plain content type to the writePayload method if the payload content was truncated. It should avoid issues that happen in the writePayload method of the class AbstractLoggingInterceptor when the payload stream contains truncated data (and therefore invalid xml) and content type is set to text/xml. In that case the PrettyXmlStreamWriter fails to output the payload stream.

Thanks, can you please rewrite the test code with the Java 7 in mind - CXF 3.1.x should compile with Java 7

Rewrote the test code to comply Java 7 spec. Strangely enough, though, that the core project doesn't compile with default Java 7 language spec in Intellij Idea, it shows several type conversion errors in other files.


DB changes needed for moving out from Airtable

This is an aggregated issue for:
https://github.com/MrOrz/rumors-db/issues/2
https://github.com/MrOrz/rumors-db/issues/3
https://github.com/MrOrz/rumors-db/issues/4

With cofacts/rumors-db#3 and cofacts/rumors-db#4 tracked individually, this issue can be closed.


Reorganize utils.py

Right now there is a single utils file, which is fine but will quickly become a mess as we add more functions to it, so better to break it up now. One solution is to just separate it into smaller utils files that go in modules.

Splitting to tree_utils.py and utils.py will already be a lot cleaner

Split intro tree_utils.py and utils.py in #82


Endless "Slave init complete Slave loading data" console logs

Version
8.1.5
Browser and OS info
NativeScript-Vue with Electron
Steps to reproduce

install/import nativescript-vue-devtools via import VueDevtools from 'nativescript-vue-devtools';
configure vue-devtools for my ip Vue.use(VueDevtools, {host: 'xxx.xxx.x.xx'});
run tns run ios

What is expected?
I expect not to see the pictured console logs since it this issue was supposed to be solved by this commit.

What is actually happening?
I am seeing the pictured console logs

Looking at nativescript-vue-devtools, it imports the @vue/devtools package (the shell-electron folder in this repo):

The main file is index.js:

The index.js imports the built files (in the build folder):

The built files don't contain the console.logs. So the only possible explanation is that you are using an older version from before the fix was implemented.

 "@vue/devtools": "^5.3.4",
 "nativescript-vue-devtools": "^1.5.1",

I see. It looks like I may indeed be on an older version of devtools.

I installed that version while following these instructions. So maybe that's the version that nativescript-vue supports. I'll try upgrading and see if it fixes the issue.

So I reinstalled vue-devtools, but I didn't specify the version and now those console logs are gone. Thanks for your help!


No way to get access token (Oculus)

API Token is not needed for Steam version, but I have BeatSaber from Oculus store and after registering on BeatSaver page there is no subpage to generate the token. Voting is grayed out for me.

As far as I know, there is no API access token on BeatSaver anymore

So if I'm on Oculus I'm out of luck?

Sounds like it, but you should ask Kyle1413 in BSMG Discord about this

"Oculus accounts currently cannot vote from in game as beatsaver does not currently support generating read/write api tokens and there is no way to vote via the api using an oculus id"


Refreshing token with gp-auth fails

When attempting to run raccoon --gp-auth to refresh the token I get a BadAuthentication response. I believe this is because the default HTTP client is used which fails Google's TLS fingerprinting.
The command works as expected when I modified the request to use a client configured to perform authentication, i.e. the client built in LoginLogic#createLoginClient.

Fixed in 4.22


Core concepts link in microsite goes to home

Core concepts link in microsite goes to home instead of core concepts section

This reminded me of something I noticed when I was reading through the documentation. I probably should create another issue...
The order in the navigation menu does not match how the documentation flows(and is currently linked).
In the documentation, the links follow:
Algebras
Modules
Handlers
Parellelism

The navigation bar has:
Algebras
Handlers
Parellelism
Modules


Feature Request: Immediate share once connection established

It would be great when starting a share, if there was a share location on the pop-up modal.
This has been a fantastic program by the way

That's a great idea! Should be pretty easy to implement as well. I'll make sure this makes it in v1.3.1. Thanks for the suggestion :-)

Update v1.3.1 is now tagged - it should appear in F-Droid within a few days to a week.


wlr_cursor: add wlr_cursor_is_input_device_attached to wlr_cursor

What's the use-case for this function?

What's the use-case for this function?

sway#5695

@emersion Would you like to move further with this PR and its Sway counter part?

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:
https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/2403


Restart Button Is Not Working

#The restart button is hidden behind the Game Over Massage
** Sreenshot **

Please assign me for this issue

Hi @YadavAkhileshh, I would like work on and fix the issue of restart button. Please assign me.

@ysachin438 This issue has been resolved, so you can proceed with the new issue and begin working on it.


Add a Service Worker and an offline page

Based on the discussion in #149, I took another crack at this. As the aim of Minima is to show what unmodified Jekyll looks like (E.g, jekyll start), I cached files based on the contents of lib/site_template.
For a 3.0 release (#145), this seems like a reasonable thing to include.

I am not sure that this is in keeping with the spirit of having a minimal theme.

@pathawks,
It probably isn't (in a strict sense), but this seems like a lighter-weight solution compared to using a module like Workbox. Although I've used that on a small site and it does the job well, it would require people to deal with NPM. That isn't difficult, but it is a bit heavy in terms of dependencies.
As previously mentioned, this is probably something for 3.0, so I'm happy to wait on this for a while.

Thanks @chrisfinazzo I'd really much like if we could provide a better user experience and allow offline browsing for minima. Shouldn't we add a timestamp to the staticCacheName here?

@chrisfinazzo we don't want any npm dependency to stay minimal. Thanks for using the native API.

Going offline by @adactio is out today, let's read it and make sure we have a solid strategy here 😉

@DirtyF I read it this  afternoon, so this was on my mind 😃.
To your point regarding a timestamp on the CacheName, if we update the version constant, that should cause a new registration. To be honest, the section on caching is a bit dense, so I probably need to spend a bit more time thinking about what the right thing to do is.

@chrisfinazzo  I was thinking of something like const staticCacheName = "version-{{site.time | date: '%Y%m%d%H%M%S'}}";

@DirtyF Ah, I see now. Will update.

This would be a nice addition.  Thanks for working on adding it.

The offline.html is a useful template, but wouldn't be shipped with the gem-based theme.
The same for sw.js.  It would need to be located in assets/js or some such.
Maybe call the file offline.js instead of sw.js?
If the service worker is added on every page, and the sw.js exists, but the offline.html does not, the service worker fetch will fail.  This is a Javascript console error.  Loading the site while offline will fail to load the offline page ⚡, but the browser knows how to degrade gracefully to just a regular "This site can't be reached", so 👍.
It would be nice to have the offline page cache be configurable, but it seems like this should be a Jekyll plugin?

@ashawley,
Thanks for the feedback. I have a branch in my local repo that has some of the changes you mentioned, I will review there and then add those to this PR.

If the service worker is added on every page, and the sw.js exists, but the offline.html does not, the service worker fetch will fail.  This is a Javascript console error.  Loading the site while offline will fail to load the offline page ⚡, but the browser knows how to degrade gracefully to just a regular "This site can't be reached", so 👍.

In GitHub Pages environments, the custom 404.html is used. In terms of fallbacks, it's not much work to modify an .htaccess or nginx.conf to point to it - if your host doesn't do this behind the scenes already.

I had suggested:

The same for sw.js.  It would need to be located in assets/js or some such.

According to Google's primer on Service Workers that wouldn't really work.

One subtlety with the register() method is the location of the service worker file. You'll notice in this case that the service worker file is at the root of the domain. This means that the service worker's scope will be the entire origin. In other words, this service worker will receive fetch events for everything on this domain. If we register the service worker file at /example/sw.js, then the service worker would only see fetch events for pages whose URL starts with /example/ (i.e. /example/page1/, /example/page2/).

It seems it would need to live at the root directory.

Maybe call the file offline.js instead of sw.js?

This name is cryptic, but I suppose it is the standard name for this kind of thing.


Use eventemitter

Feature Request
Simple subject of my feature...
Using eventemitter.
Feature description in brief...
I have viewed the documentation and found that there are no events to know when cache is ready. But then i found this on README
const Client = new Spotify.Client("USER_TOKEN", { cacheCurrentuser: true });

function onCacheReady(){
    console.log('Client cache is ready!');
}

if(!Client.madeCache) Client.cacheOnReady = onCacheReady;
else onCacheReady();

This seems ugly. Can you use guys events package so the code looks like this:
client.on('ready', ...

Something like discord.js? It would be fine right?

I accept the fact that the code looks ugly. But we made that method just for temporary purpose. This thing probably will be updated on v8 of this package. And using events package for just 1 event is pointless and worthless. We would better use a ready method in the CacheOptions interface or somewhat. But thanks for sharing your idea!

In that case you can actually make events like debug, rateLimit and error!

Idea is good but it will not be good for a package like spotify-api.js. And spotify-api.js aims to be a bit small and fast!

Well this was a suggestion.


conette-prepare: command not found

After  python -m pip install conette[train],  conette-prepare: command not found

Hi! Sorry for not responding in time.
If your problem hasn't been solved, or if you encounter others, don't hesitate to post another issue.


Cannot draw polygons on ipad

Cannot draw polygons on ipad
Polygon or bbox based occurrence search is not working on the ipad. It is impossible to draw or edit a shape

fbitem-90ca04c707cd953f778e8789ed7ec98e67b20053
System: Mobile Safari 10.0.0 / iOS 10.3.1
Referer: https://www.gbif.org/occurrence/gallery?has_coordinate=true&has_geospatial_issue=false&taxon_key=1&advanced=1&geometry=POLYGON((8 41,8 42,10 42,10 41,8 41))
Window size: width 921 - height 656
API log
Site log

Took forever to find the issue. The issue was angular material not working with openlayers
angular/material#1406
fixed by disabling gestures in the angular material library https://github.com/gbif/portal16/blob/master/app/views/shared/layout/html/html.js#L162
Notice that the editing experience is still not ideal as you easily get to scroll the drawer instead.  See https://github.com/gbif/portal16/issues/649


Bracket pair error in c #ifdef code

Does this issue occur when all extensions are disabled?: Yes

Version: 1.89.1 (user setup)
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
Date: 2024-05-07T05:13:33.891Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Windows_NT x64 10.0.19045
Vscode is not real comment the single bracket in #ifdef in C and it will still participate in the match.
The function use the #ifdef to comment the bracket will be judged as a syntax error and will not be able to jump to it normally.
Actually, I don't care whether the bracket matches syntax, I hope it can jump normal. the C language syntax is correct here.
I use the C/C++ extension(microsoft) to jump. I close the all extensions and the bracket syntax is still wrong. The bracket will effect the jump funtion of C/C++ extension when I open this externsion.

Steps to Reproduce:
This is a mini example code:

#define __TEST_MACRO

void test_function(void) {      // This bracket become red it means bracket Matching rule detected error
    int var = 1;

#ifdef __TEST_MACRO
    if (var) {
        var = 2;
    } else {                    // This bracket has been pair to the bracket in 15 line, it's should pair to 14 line
#else
    {
#endif
        var = 3;
    }
}

/duplicate
I'm very sure almost all static parsers struggle with this. I recommend to only use ifdef blocks for complete syntactical units!

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

Do I have any way to ignore this error and jump to this function which exist bracket fault? I use the compile_commands.json to do the jump way. The ifdef blocks is in a library which i download from github, so I don't what modify this code. I just want I can jump to functions normally.
Do I have any way to ignore this error and jump to this function which exist bracket fault?

Yes, there is a setting to disable bracket pair colorization.


stopped working today

Hi,
this was working until today. I appreciate if you take a min and fix it please. Thanks. I loved it

I tested the code on multiple browsers and on incognito tab too and the code is working fine. Twitter has not updated the Ads layout structure in my region. If you are seeing the ads make sure the extension you are using to run the code is working fine. Also make sure you are using the latest code from this repo. Try clearing the browser cache.
The extension I use to run the code is "User Javascript and CSS"

You are right. It still works on Chrome, but not on Safari. It used to work for both; I used this https://getsprinkles.app
So it is possible that only for Safari they have made some changes to css for ads.

here is my contribution to one that works on Safari https://github.com/kouroshshafi/AdBlock-Twitter/tree/main
Thanks again and yours is still great for Chrome.


Adding rubric for proposal D

Signed-off-by: Brandon Mitchell<EMAIL_ADDRESS>I've updated the table with my read on proposal D. The only area that I think needs more testing is existing clients when they see a platform with the OS set to unknown, either with or without another image they find to run.

General question: do you think an explanation for why the group has accepted such a rating is useful for folks to refer back?

General question: do you think an explanation for why the group has accepted such a rating is useful for folks to refer back?

General question: do you think an explanation for why the group has accepted such a rating is useful for folks to refer back?

I'm wondering if it makes sense to fill in the requirements responses per proposal as another section of the proposal document. We could say "Yes, here's how..." in the proposal, so the rubric itself is kept simple to glance at.

I converted this to a draft while we sort out where to track the rubric details.

Closing in favor of #34


Moving to PipelineRuns with inline pipeline spec

The current design starts pipeline runs referencing a pipeline. It would also be possible to not maintain pipeline resources at all and only deal with PipelineRuns with an inlined spec. For now this issue only exists to document this idea, without the plan for any immediate action.
Downside:

The OpenShift console UI allows to navigate from Pipeline to PipelineRun. If we do not have Pipeline resources any longer, there is no good grouping mechanism anymore

Advantages:

No need to deal with Pipeline resources separately
We could also inline the task specs (e.g. by reading the spec from an installed task), allowing to give more control to the user (such as overriding compute resources and adding sidecars)

Decided today with @kuebler and @henrjk that we will proceed with this.