Build of Ubuntu 18.04 container fails

Setup proxy in config file.
ran: ipdk install ubuntu1804
then: ipdk build --no-cache --use-proxy
Errored out after a while:
Successfully installed pip-21.3.1
Collecting grpcio
Downloading grpcio-1.48.2-cp36-cp36m-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (4.6 MB)
Requirement already satisfied: six>=1.5.2 in /usr/lib/python3/dist-packages (from grpcio) (1.11.0)
Installing collected packages: grpcio
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
Successfully installed grpcio-1.48.2
ERROR: Command errored out with exit status 1:
command: /usr/bin/python3 -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-f0dsnyt0/ovspy_7384b2183870492abca21ff9cda021ea/setup.py'"'"'; file='"'"'/tmp/pip-install-f0dsnyt0/ovspy_7384b2183870492abca21ff9cda021ea/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-mz_v88la
cwd: /tmp/pip-install-f0dsnyt0/ovspy_7384b2183870492abca21ff9cda021ea/
Complete output (7 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-f0dsnyt0/ovspy_7384b2183870492abca21ff9cda021ea/setup.py", line 9, in
long_description=open('README.md').read(),
File "/usr/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 57: ordinal not in range(128)
WARNING: Discarding https://files.pythonhosted.org/packages/3e/0c/34a7a123526bcd205a61996302fd7eb2b496572818d05a6e09772b640b94/ovspy-0.1.0b2.tar.gz#sha256=38a01de9ab122973af2c9e9b7eea70c9bf74bbb118951da14f0c25b3ceda4017 (from https://pypi.org/simple/ovspy/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Command errored out with exit status 1:
command: /usr/bin/python3 -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-f0dsnyt0/ovspy_e576f7d372d54847a548f4d8d6cb9a86/setup.py'"'"'; file='"'"'/tmp/pip-install-f0dsnyt0/ovspy_e576f7d372d54847a548f4d8d6cb9a86/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-rtgy0wuy
cwd: /tmp/pip-install-f0dsnyt0/ovspy_e576f7d372d54847a548f4d8d6cb9a86/
Complete output (7 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-f0dsnyt0/ovspy_e576f7d372d54847a548f4d8d6cb9a86/setup.py", line 9, in
long_description=open('README.md').read(),
File "/usr/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 57: ordinal not in range(128)
WARNING: Discarding https://files.pythonhosted.org/packages/d5/07/e53edfc1411dc8d10525aee1f9bf8a911ae1fb8c9e2ab20493553d0d09a2/ovspy-0.1.0b1.tar.gz#sha256=872000c8a2e699d1f9403e0a1252bf58f81301445d6e1032ad7fd277b0e3159c (from https://pypi.org/simple/ovspy/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement protobuf==3.20.1 (from versions: 2.0.0b0, 2.0.3, 2.3.0, 2.4.1, 2.5.0, 2.6.0, 2.6.1, 3.0.0a2, 3.0.0a3, 3.0.0b1, 3.0.0b1.post1, 3.0.0b1.post2, 3.0.0b2, 3.0.0b2.post1, 3.0.0b2.post2, 3.0.0b3, 3.0.0b4, 3.0.0, 3.1.0, 3.1.0.post1, 3.2.0rc1, 3.2.0rc1.post1, 3.2.0rc2, 3.2.0, 3.3.0, 3.4.0, 3.5.0.post1, 3.5.1, 3.5.2, 3.5.2.post1, 3.6.0, 3.6.1, 3.7.0rc2, 3.7.0rc3, 3.7.0, 3.7.1, 3.8.0rc1, 3.8.0, 3.9.0rc1, 3.9.0, 3.9.1, 3.9.2, 3.10.0rc1, 3.10.0, 3.11.0rc1, 3.11.0rc2, 3.11.0, 3.11.1, 3.11.2, 3.11.3, 3.12.0rc1, 3.12.0rc2, 3.12.0, 3.12.1, 3.12.2, 3.12.4, 3.13.0rc3, 3.13.0, 3.14.0rc1, 3.14.0rc2, 3.14.0rc3, 3.14.0, 3.15.0rc1, 3.15.0rc2, 3.15.0, 3.15.1, 3.15.2, 3.15.3, 3.15.4, 3.15.5, 3.15.6, 3.15.7, 3.15.8, 3.16.0rc1, 3.16.0rc2, 3.16.0, 3.17.0rc1, 3.17.0rc2, 3.17.0, 3.17.1, 3.17.2, 3.17.3, 3.18.0rc1, 3.18.0rc2, 3.18.0, 3.18.1, 3.18.3, 3.19.0rc1, 3.19.0rc2, 3.19.0, 3.19.1, 3.19.2, 3.19.3, 3.19.4, 3.19.5, 3.19.6, 4.0.0rc1, 4.0.0rc2, 4.21.0rc1, 4.21.0rc2, 4.21.0)
ERROR: No matching distribution found for protobuf==3.20.1
The command '/bin/bash -c /root/scripts/distro_pkg_install.sh -b &&     /root/start_p4ovs.sh /root &&     /root/scripts/run_cleanup.sh /root ${KEEP_SOURCE_CODE} ${DEPLOYMENT_IMAGE}' returned a non-zero code: 1

Hello Patrick, Ubuntu 18.04 is not officially supported version for ipdk container(https://github.com/ipdk-io/ipdk/blob/main/RELEASENOTES.md) as officially Ubuntu Linux 18.04 is supported only till end of April 2023.
Supported versions are Ubuntu 20.04 and Fedora 33 for ipdk container.
If you are interested in Ubuntu18.04 ipdk container, please look at the PR  https://github.com/ipdk-io/ipdk/pull/224 to resolve protobuf related build issues.

I'm not really interested, but it is listed as something that should work.  I'd ask that the documentation be updated with this information and a link to the method to resolve.

Addressed Ubuntu 18.04 build issues with PR #293
Please pull the latest code and use it for building Ubuntu 18.04 image.


Neuroglancer not working, showing "no zarr metadata found" error

When clicking on View Tomogram, neuroglancer seems to be having trouble loading dataset 10007

@uermel @manasaV3 @jgadling any idea if this is an issue with NG or something caused by the backend data?

I'm a little confused by this one.
For that run, "Voxelspacing13.48" should not exist, which is why that zarr-link would not work. With the proper value "Voxelspacing7.840" it loads fine for me (because that file actually exists). When I click on 'View Tomogram' here the correct voxelspacing is used.
Potentially this bug is outdated after the reimport?

nevermind, I rechecked and wasn't able to reproduce either! will close this bug


app will crash when I quit activity

11-01 17:18:20.613  1335 28986 I Camera2ClientBase: Closed Camera 0. Client was: io.github.sceneview.sample.armodelviewer (PID 12868, UID 10289)
11-01 17:18:20.614 12868 12868 I Choreographer: Skipped 38 frames!  The application may be doing too much work on its main thread.
11-01 17:18:20.615 12868 12868 W Looper  : PerfMonitor doFrame : time=1ms vsyncFrame=19573743 latency=635ms procState=2 historyMsgCount=3 (msgIndex=3 wall=609ms seq=239 running=5ms runnable=1ms binder=47ms late=33ms h=android.app.ActivityThread$H w=159)
11-01 17:18:21.165 12868 12868 I native  : I0000 00:00:1667294301.165340   12868 session_lite_c_api.cc:37] Deleting ArSession...
11-01 17:18:21.181 12868 12868 I native  : I0000 00:00:1667294301.181094   12868 android_camera.cc:762] [Camera=1; State=CLOSED] Reset cleanly got to CLOSED state.
11-01 17:18:21.181 12868 12868 I native  : I0000 00:00:1667294301.181312   12868 android_camera.cc:762] [Camera=0; State=CLOSED] Reset cleanly got to CLOSED state.
11-01 17:18:21.181 12868 12868 I native  : I0000 00:00:1667294301.181779   12868 session_lite_c_api.cc:39] Deleted ArSession.
11-01 17:18:21.183 12868 12868 F libc    : Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x6d0 in tid 12868 (e.armodelviewer), pid 12868 (e.armodelviewer)
11-01 17:18:21.236   778   778 I tombstoned: received crash request for pid 12868
11-01 17:18:21.237 13097 13097 I crash_dump64: performing dump of process 12868 (target tid = 12868)
11-01 17:18:21.243 13097 13097 F DEBUG   : pid: 12868, tid: 12868, name: e.armodelviewer  >>> io.github.sceneview.sample.armodelviewer <<<
11-01 17:18:21.641  1754  3956 I ActivityManager: Process io.github.sceneview.sample.armodelviewer (pid 12868) has died: fg  TOP
11-01 17:18:21.641  1754  1807 I Process : PerfMonitor : current process killing process group. PID: 12868
11-01 17:18:21.643   788   788 I Zygote  : Process 12868 exited due to signal 11 (Segmentation fault)
11-01 17:18:21.647  1754  1807 I libprocessgroup: Successfully killed process cgroup uid 10289 pid 12868 in 5ms

Your app is running too much in man thread , try using background thread
Use coroutines scope for handling your app

Your app is running too much in man thread , try using background thread Use coroutines scope for handling your app

I only new a activity and start MainActivity  in samples  ar-model-viewer。it will crash in MainActivity when BackPressed.

class TestActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_test)
        findViewById<Button>(R.id.btn_go_ar).setOnClickListener {
            startActivity(Intent(this,MainActivity::class.java))
        }
    }
}

can you upload your codes for better understanding

can you upload your codes for better understanding

TestActivity.java
package io.github.sceneview.sample.armodelviewer

import android.content.Intent
import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.widget.Button

class TestActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_test)
        findViewById<Button>(R.id.btn_go_ar).setOnClickListener {
            startActivity(Intent(this,MainActivity::class.java))
        }
    }
}

MainActivity.java
package io.github.sceneview.sample.armodelviewer

import android.os.Bundle
import android.util.Log
import android.view.Menu
import android.view.MenuItem
import android.view.View
import android.view.ViewGroup
import androidx.appcompat.app.AppCompatActivity
import androidx.appcompat.widget.Toolbar
import androidx.core.view.isGone
import androidx.core.view.isVisible
import com.google.android.material.floatingactionbutton.ExtendedFloatingActionButton
import io.github.sceneview.ar.ArSceneView
import io.github.sceneview.ar.arcore.LightEstimationMode
import io.github.sceneview.ar.arcore.LightEstimator
import io.github.sceneview.ar.node.ArModelNode
import io.github.sceneview.ar.node.PlacementMode
import io.github.sceneview.light.Light
import io.github.sceneview.light.LightInstance
import io.github.sceneview.math.Position
import io.github.sceneview.utils.doOnApplyWindowInsets
import io.github.sceneview.utils.setFullScreen

class MainActivity : AppCompatActivity(R.layout.activity_main) {

    lateinit var sceneView: ArSceneView
    lateinit var loadingView: View
    lateinit var placeModelButton: ExtendedFloatingActionButton
    lateinit var newModelButton: ExtendedFloatingActionButton

    private val TAG = "MainActivity"

    data class Model(
        val fileLocation: String,
        val scaleUnits: Float? = null,
        val placementMode: PlacementMode = PlacementMode.BEST_AVAILABLE,
        val applyPoseRotation: Boolean = true
    )

    val models = listOf(
        Model("models/spiderbot.glb"),
        Model(
            fileLocation = "models/zoe.glb",
            // Display the Tiger with a size of 3 m long
            scaleUnits = 0.5f,
            placementMode = PlacementMode.PLANE_HORIZONTAL,
            applyPoseRotation = false
        ),
        Model(
            fileLocation = "https://sceneview.github.io/assets/models/DamagedHelmet.glb",
            placementMode = PlacementMode.INSTANT,
            scaleUnits = 0.5f
        ),
        Model(
            fileLocation = "https://storage.googleapis.com/ar-answers-in-search-models/static/GiantPanda/model.glb",
            placementMode = PlacementMode.PLANE_HORIZONTAL,
            // Display the Tiger with a size of 1.5 m height
            scaleUnits = 1.5f
        ),
        Model(
            fileLocation = "https://sceneview.github.io/assets/models/Spoons.glb",
            placementMode = PlacementMode.PLANE_HORIZONTAL_AND_VERTICAL,
            // Keep original model size
            scaleUnits = null
        ),
        Model(
            fileLocation = "https://sceneview.github.io/assets/models/Halloween.glb",
            placementMode = PlacementMode.PLANE_HORIZONTAL,
            scaleUnits = 2.5f
        ),
    )
    var modelIndex = 0
    var modelNode: ArModelNode? = null

    var isLoading = false
        set(value) {
            field = value
            loadingView.isGone = !value
        }

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)

        setFullScreen(
            findViewById(R.id.rootView),
            fullScreen = true,
            hideSystemBars = false,
            fitsSystemWindows = false
        )

        setSupportActionBar(findViewById<Toolbar>(R.id.toolbar)?.apply {
            doOnApplyWindowInsets { systemBarsInsets ->
                (layoutParams as ViewGroup.MarginLayoutParams).topMargin = systemBarsInsets.top
            }
            title = ""
        })
        sceneView = findViewById(R.id.sceneView)

        sceneView.lightEstimationMode = LightEstimationMode.DISABLED
        loadingView = findViewById(R.id.loadingView)
        newModelButton = findViewById<ExtendedFloatingActionButton>(R.id.newModelButton).apply {
            // Add system bar margins
            val bottomMargin = (layoutParams as ViewGroup.MarginLayoutParams).bottomMargin
            doOnApplyWindowInsets { systemBarsInsets ->
                (layoutParams as ViewGroup.MarginLayoutParams).bottomMargin =
                    systemBarsInsets.bottom + bottomMargin
            }
            setOnClickListener { newModelNode() }
        }
        placeModelButton = findViewById<ExtendedFloatingActionButton>(R.id.placeModelButton).apply {
            setOnClickListener { placeModelNode() }
        }

        newModelNode()
    }

    override fun onCreateOptionsMenu(menu: Menu?): Boolean {
        menuInflater.inflate(R.menu.activity_main, menu)
        return super.onCreateOptionsMenu(menu)
    }

    override fun onOptionsItemSelected(item: MenuItem): Boolean {
        item.isChecked = !item.isChecked
        modelNode?.detachAnchor()
        modelNode?.placementMode = when (item.itemId) {
            R.id.menuPlanePlacement -> PlacementMode.PLANE_HORIZONTAL_AND_VERTICAL
            R.id.menuInstantPlacement -> PlacementMode.INSTANT
            R.id.menuDepthPlacement -> PlacementMode.DEPTH
            R.id.menuBestPlacement -> PlacementMode.BEST_AVAILABLE
            else -> PlacementMode.DISABLED
        }
        return super.onOptionsItemSelected(item)
    }

    fun placeModelNode() {
        modelNode?.anchor()
        placeModelButton.isVisible = false
        sceneView.planeRenderer.isVisible = false
    }

    fun newModelNode() {
        isLoading = true
        modelNode?.takeIf { !it.isAnchored }?.let {
            sceneView.removeChild(it)
            it.destroy()
        }
        val model = models[modelIndex]
        modelIndex = (modelIndex + 1) % models.size
        modelNode = ArModelNode(model.placementMode).apply {
            applyPoseRotation = model.applyPoseRotation
            loadModelAsync(
                context = this@MainActivity,
                lifecycle = lifecycle,
                glbFileLocation = model.fileLocation,
                autoAnimate = true,
                scaleToUnits = model.scaleUnits,
                // Place the model origin at the bottom center
                centerOrigin = Position(y = -1.0f)
            ) {
                sceneView.planeRenderer.isVisible = true
                isLoading = false
            }
            onAnchorChanged = { node, _ ->
                placeModelButton.isGone = node.isAnchored
            }
            onHitResult = { node, _ ->
                placeModelButton.isGone = !node.isTracking
            }
        }
        sceneView.addChild(modelNode!!)
        // Select the model node by default (the model node is also selected on tap)
        sceneView.selectedNode = modelNode
    }
}

@ThomasGorisse @xuanxuandaoren @vigit3

2022-11-28 12:01:51.809 13175-13175/io.github.sceneview.sample.armodelviewer E/native: E0000 00:00:1669633311.809612   13175 scheduler.cc:278] INTERNAL: RET_CHECK failure (third_party/mediapipe/framework/scheduler.cc:278) state_ != STATE_NOT_STARTED (0 vs. 0)
Stack trace:
2022-11-28 12:01:51.809 13175-13175/io.github.sceneview.sample.armodelviewer E/native: E0000 00:00:1669633311.809660   13175 normal_detector_cpu.cc:231] Error graph_->WaitUntilIdle():INTERNAL: RET_CHECK failure (third_party/mediapipe/framework/scheduler.cc:278) state_ != STATE_NOT_STARTED (0 vs. 0)
=== Source Location Trace: ===
third_party/mediapipe/framework/scheduler.cc:278
third_party/mediapipe/framework/calculator_graph.cc:845
2022-11-28 12:01:51.817 1806-3335/? E/SecHAL: nfc_data_callback: [Cmd]Received a response packet : data_sending_flag = false!!
2022-11-28 12:01:51.817 1690-3304/? E/libnfc_nci: [ERROR:nfc_ncif.cc(1457)] nfc_ncif_proc_deactivate - evt_data.deactivate.reason 0, nfc_cb.deact_reason = 0
2022-11-28 12:01:51.817 1806-1806/? E/SecHAL: nfc_hal_write: [Cmd]Send EVT data_sending_flag = true, return size=18!!
2022-11-28 12:01:51.826 1806-3335/? E/SecHAL: nfc_data_callback: [Cmd]Received a response packet : data_sending_flag = false!!
2022-11-28 12:01:52.060 430-430/? E/vold: getUsedF2fsFileNode -> Cannot Get Used FileNode Number (errno=25 Inappropriate ioctl for device)
2022-11-28 12:01:52.251 13175-13175/io.github.sceneview.sample.armodelviewer A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x6d0 in tid 13175 (e.armodelviewer), pid 13175 (e.armodelviewer)
2022-11-28 12:01:52.591 13391-13391/? A/DEBUG: *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***

I'm running into the same issue with SEGV_MAPERR.  I have a simple way to repo though which is simply enabling layout inspector when running samples/model-viewer. Crash every time with 0.9.0.
With the latest, running samples/model-viewer directly in the repo, it also crashes when enabling layout inspector this time with ending with this.
at io.github.sceneview.nodes.Node.destroy(Node.kt:262)
at io.github.sceneview.nodes.CameraNode.destroy(CameraNode.kt:53)
at io.github.sceneview.SceneView.destroy(SceneView.kt:693)
at io.github.sceneview.SceneView$setLifecycle$3.invoke(SceneView.kt:718)
at io.github.sceneview.SceneView$setLifecycle$3.invoke(SceneView.kt:718)
E/AndroidRuntime:     at com.gorisse.thomas.lifecycle.LifecycleKt$observe$1.onDestroy(Lifecycle.kt:164)
at androidx.lifecycle.FullLifecycleObserverAdapter.onStateChanged(FullLifecycleObserverAdapter.java:51)
at androidx.lifecycle.LifecycleRegistry$ObserverWithState.dispatchEvent(LifecycleRegistry.java:360)
at androidx.lifecycle.LifecycleRegistry.backwardPass(LifecycleRegistry.java:290)
at androidx.lifecycle.LifecycleRegistry.sync(LifecycleRegistry.java:308)
at androidx.lifecycle.LifecycleRegistry.moveToState(LifecycleRegistry.java:151)
at androidx.lifecycle.LifecycleRegistry.handleLifecycleEvent(LifecycleRegistry.java:134)
at androidx.fragment.app.Fragment.performDestroy(Fragment.java:3352)
at androidx.fragment.app.FragmentStateManager.destroy(FragmentStateManager.java:781)
at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:335)
at androidx.fragment.app.SpecialEffectsController$FragmentStateManagerOperation.complete(SpecialEffectsController.java:771)
at androidx.fragment.app.SpecialEffectsController$Operation.cancel(SpecialEffectsController.java:615)
at androidx.fragment.app.SpecialEffectsController.forceCompleteAllOperations(SpecialEffectsController.java:350)
at androidx.fragment.app.FragmentManager.dispatchStateChange(FragmentManager.java:2980)
at androidx.fragment.app.FragmentManager.dispatchDestroy(FragmentManager.java:2931)
at androidx.fragment.app.FragmentController.dispatchDestroy(FragmentController.java:346)
at androidx.fragment.app.FragmentActivity.onDestroy(FragmentActivity.java:259)
at androidx.appcompat.app.AppCompatActivity.onDestroy(AppCompatActivity.java:280)
at android.app.Activity.performDestroy(Activity.java:8319)
at android.app.Instrumentation.callActivityOnDestroy(Instrumentation.java:1376)
at android.app.ActivityThread.performDestroyActivity(ActivityThread.java:5413)
at android.app.ActivityThread.handleDestroyActivity(ActivityThread.java:5459)
at android.app.ActivityThread.handleRelaunchActivityInner(ActivityThread.java:5759)
at android.app.ActivityThread.handleRelaunchActivity(ActivityThread.java:5675)
at android.app.servertransaction.ActivityRelaunchItem.execute(ActivityRelaunchItem.java:71)
at android.app.servertransaction.ActivityTransactionItem.execute(ActivityTransactionItem.java:45)
at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)
at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)
at android.app.ClientTransactionHandler.executeTransaction(ClientTransactionHandler.java:62)
at android.app.ActivityThread.handleRelaunchActivityLocally(ActivityThread.java:5742)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2262)
at android.os.Handler.dispatchMessage(Handler.java:106)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7842)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)
Please help with a fix (even if I can just do it locally that would be much appreciated).
I adding logging in MainFragment onDestroyView() override and it dies After super.onDestroyView()

I got some more logs that were a bit more helpful. I was able to reproduce on the sample app sample-ar-cursor-placement with an introduced additional dummy launcher activity with a button that launches the provided Activity
20:06:42.977  E  E0000 00:00:1670893602.977739   28618 scheduler.cc:278] INTERNAL: RET_CHECK failure (third_party/mediapipe/framework/scheduler.cc:278) state_ != STATE_NOT_STARTED (0 vs. 0)
                 Stack trace:
20:06:42.977  E  E0000 00:00:1670893602.977776   28618 normal_detector_cpu.cc:233] Error graph_->WaitUntilIdle():INTERNAL: RET_CHECK failure (third_party/mediapipe/framework/scheduler.cc:278) state_ != STATE_NOT_STARTED (0 vs. 0)
                 === Source Location Trace: ===
                 third_party/mediapipe/framework/scheduler.cc:278
                 third_party/mediapipe/framework/calculator_graph.cc:845
20:06:43.556  E  in destroy:959
                 reason: destroying material "base_lit_opaque" but 8 instances still alive
20:06:43.556  E
20:06:43.669  A  Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x0 in tid 28618 (y.direct_relief), pid 28618 (y.direct_relief)
20:06:44.284  A  Cmdline: com.app.balcony.direct_relief
20:06:44.284  A  pid: 28618, tid: 28618, name: y.direct_relief  >>> com.app.balcony.direct_relief <<<
20:06:44.284  A        #00 pc<PHONE_NUMBER>0cbca0  /data/app/~~n6dZR_hsLeFIyUwdXVKWTQ==/com.app.balcony.direct_relief-mMLFF8DeW3UYAwuxMHnHQA==/base.apk!libfilament-jni.so (BuildId: 566f8cb854d0326e1c1af97cea1dd2f3831c599b)

I'm experiencing the same Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR) error when exiting back to a parent activity. The GLB is only 15k in size.

heads up that very possible this will be fixed by https://github.com/SceneView/sceneview-android/pull/150. Not sure though, just heads up in case anyone is in dire need.

Also facing the similar problem.
I'm using the sceneView in something like bottom navigation Fragment. When I switch the fragment from the one with sceneView to another one. The app will crash and seems there's something wrong with the destroy process.

We also had this problem on back navigation. We got it working by  replacing the lifecycle = lifecycle with :
 loadModelAsync(
                context = requireContext(),
                lifecycle = lifecycle,
                lifecycle = requireActivity().lifecycle,
                glbFileLocation = "models/spiderbot.glb"
            )

But this will only work if you are running a single activity type in your application, I would see this crashing when you would navigate back towards a activity that hasn't been initialized.

None of the above has helped with the crashes. Has anyone gotten this to work doing an exit of AppCompatActivity with a SceneView in it?

#150

This will "mute" your crash because your app will be already in the background when the crash happens, so you won't see it :) It's definitely an issue with the destroy process, I can't imagine any workaround for it.

Any more progress on this issue?

For the moment, the only way I found to don't have this issue is to have only one activity and several fragment and to don't destroy the fragment containing the arSceneView when I want to use an other fragment.
So so fragment with the arSceneview continue to consume some ressources in the background but at least the app doesn't crash

I've done something similar by putting the ArSceneView in a dialog that I hide instead of dismiss/cancel, but the camera stays on after returning back to the parent activity. This can look suspicious to the user when the camera indicator shows it's active. How did you handle that?

I'm using this function when I press the back button:
`private fun backToMainMenu(){
val fragment : Fragment = MainMenuFragment()
    val manager : FragmentManager = supportFragmentManager

    // Begin the fragment transition using support fragment manager
    val transaction: FragmentTransaction = manager.beginTransaction()

    // Replace the fragment on container
    transaction.replace(R.id.containerFragment, fragment)

    // Finishing the transition
    transaction.commit()
}`

With this code, the fragment where I have my arSceneView is Stopped but not Destroyed. So the camera is closed (normally) and the app doesn't crash.
But if this parent activity is closed, the fragment is also closed and the app crash. So the issue is not solved but only pushed back

Fixed in v0.9.2

This still seems to be an issue in v0.9.2.

@m-paunovic-vegait Are you sure it's the same issue? Can you send your logs?

@m-paunovic-vegait Can you make a test with v0.9.3

I still get the "Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR)" error when exiting a fragment with ArSceneView back to the calling activity. Both 0.9.2 and 0.9.3 fail.
Is there a test case that was used to valid the error as resolved? Maybe looking at the code I could do something similar.

It's quite hard for me to debug since not all your logs show the same thing.
Since it might depends on your usage (Flutter, multiple activities, self destroy calls), could you please all help debugging the destroy call that is throwing the crash?

Ctrl+Click on observe() here https://github.com/SceneView/sceneview-android/blob/f08c2689d4397c2177958b37d48cf299127ed927/sceneview/src/main/java/io/github/sceneview/model/GLBLoader.kt#L69
Add a breakpoint on the destroy call

Add a breakpoint here: https://github.com/SceneView/sceneview-android/blob/f08c2689d4397c2177958b37d48cf299127ed927/sceneview/src/main/java/io/github/sceneview/SceneView.kt#L658
Add a breakpoint here: https://github.com/SceneView/sceneview-android/blob/c00654b5348dbe27e03a1304965fa87ab452b1ef/sceneview/src/main/java/io/github/sceneview/Filament.kt#L105

The goal is to F7 on Filament destroys so we find the one that is throwing a runtime exception (Double destroy, Instance not released before class,...)
Thanks

Please check v0.9.4

v0.9.4 no longer crashes for me.

0.9.4 also works for me. Thanks for getting this issue resolved!

@ThomasGorisse The crashes are back in 0.9.5 version whenever I go back. I had to search for this issue in order to find the reason, but reverting back to 0.9.4 fixed it for me.
Here's the error -
`Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x110 in tid 7304 (myappid), pid 7304 (myappid)
pid: 7304, tid: 7304, name: myappid  >>> myappid <<<
2023-02-02 18:47:21.334  7544-7544  DEBUG                   crash_dump64                         A        #00 pc<PHONE_NUMBER>1ce494  /data/app/~~ndZOfziN33j0aLA9-J7bcw==/myappid-r_D_W4qeV5gz75r50UHj0Q==/base.apk!libgltfio-jni.so (filament::gltfio::FilamentAsset::getAssetInstanceCount() const+0) (BuildId: 1f80989e9281f45e9e1be04a9226cfdef6f2338b)
`

Sorry about that, 0.9.5 uses the latest Filament version that completely renew the model asset and instances usage.
Could you please fix this destroy issue @vedprakashwagh ?

@ThomasGorisse I'm sorry but I actually have zero experience about this library or AR in general since I just started exploring this yesterday. I'm afraid I might inadvertently make things worse, haha.

Hi @ThomasGorisse, I'm quite out of my wheelhouse on this one but trying my best to help fix this issue on 0.9.5.
Essentially, the destroy that crashes the app is the first call to asset.destroy()  after filament.destroy() has already been called. The _assetLoader is set to null, so it is reinstantiated from  the call to Filament.assetLoader in asset.destroy(), that works fine, but the subsequent call to AssetLoader.nDestroyAsset(mNativeObject, asset.getNativeObject()); seems to crash the app.
https://github.com/sameerjj/sceneview-android/tree/sameer-debug-destroy-crash
I have a forked branch here that has a MainActivity in sample/ar-cursor-placement which allows for easy reproduction of the issue.
Hoping this provides some help!

Hi @sameerjj
Nice investigation !!!
Could you make a Pull Request with it?
You should be able to directly create from your fork.
Thanks

Thanks @ThomasGorisse, done :)

Hi everyone !
Is there any news regarding this issue as this crash makes this library unusable for me :(


Merge pull request #198 from MicrosoftDocs/main

Main to Live

Docs Build status updates of commit b33a388:
:warning: Validation status: warnings
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Foxit.FoxitReader version <IP_ADDRESS>94

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

looks like the build is getting stuck somehow. will take a look tommorow


✨ feat: Update Algolia configuration for improved search functionality

feat: update Algolia configuration for improved search functionality
Change the Algolia appId, apiKey, and indexName in the Docusaurus configuration to enhance the search experience and align it with the new project structure.

A very nice fix, makes HAMi website smarter
Thanks


Compatibility with Crystal 0.30

fix for abstract method change in https://github.com/crystal-lang/crystal/pull/7956

Thanks


"Valea Lungă" is listed in the drop-down menu

"Valea Lungă" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Valea Lungă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Valea Lungă, DÂMBOVIȚA".


Feat(seed/models/routes): created seed file, defined models, created some api routes

Please review changes

Closes #8, #9, #10


Improve Searcher core class

Have a look at https://github.com/krzysztof-gzocha/searcher/blob/master/src/KGzocha/Searcher/Searcher.php#L60
This can be improved. Result of it can be calculated outside of foreach and passed to internal function. There is no need to call it on each foreach

Implemented in #86


Fix windows

When building v7 with cuda 8 in windows:
 fatal error C1128: number of
sections exceeded object file format limit: compile with /bigobj

This just adds bigobj to all the builds in windows for v7, the only drawback for this option seems to be that older linkers can't consume the generated objects.
https://stackoverflow.com/questions/15110580/penalty-of-the-msvs-compiler-flag-bigobj

pfnCI, test this please.

Jenkins CI test (for commit 6483d9042e5a3966a540bcf1d5f73043f0e26493, target branch v7) succeeded!

Jenkins CI test (for commit 6483d9042e5a3966a540bcf1d5f73043f0e26493, target branch v7) failed with status FAILURE.

？


[TypeError: Cannot read property 'start' of undefined] on Mac

After I run npm install and npm run hot-server, npm run start-hot fails with the below; what am I doing wrong? Thank you
ericat ~/electron-react-boilerplate $ npm run start-hot

><EMAIL_ADDRESS>start-hot /Users/ericat/electron-react-boilerplate
> cross-env HOT=1 NODE_ENV=development electron ./

App threw an error when running [TypeError: Cannot read property 'start' of undefined]

This is the stack trace:
Uncaught Exception:
TypeError: Cannot read property 'start' of undefined
    at Object.<anonymous> (/Users/ericat/electron-react-boilerplate/main.js:15:14)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Object.<anonymous> (/Users/ericat/electron-react-boilerplate/node_modules/electron-prebuilt/dist/Electron.app/Contents/Resources/default_app/main.js:258:23)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)

Did you try removing electron's crashReporter?

Yes, in fact I replaced the whole thing with:
crashReporter = require('crash-reporter');
crashReporter.start({
  productName: 'YourName',
  companyName: 'YourCompany',
  submitUrl: 'https://your-domain.com/url-to-submit',
  autoSubmit: true
});

like the docs say; that works just fine, next error down the line:
Uncaught Exception:
TypeError: Cannot read property 'on' of undefined
    at Object.<anonymous> (/Users/ericat/electron-react-boilerplate/node_modules/electron-localshortcut/index.js:118:4)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (/Users/ericat/electron-react-boilerplate/node_modules/electron-debug/index.js:3:23)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)

Sounds like there something fundamentally wrong with your setup. Make sure you're using a recent version of node (4+) and do a fresh npm install.

well, I am using Node v5.1.1 and that's the first thing I have tried. Will close this, giving up :p Thanks though, it's appreciated.

I also had the same problem. I removed the package (https://www.npmjs.com/package/electron) that was set globally.
This issue is discussed in detail:
https://github.com/atom/electron/issues/3708
https://github.com/atom/electron/issues/4017
https://github.com/atom/electron-quick-start/issues/30

I came across same issue when I tryed using branch/tag v0.10.0.
v0.9.0 seems to work
master as  another error I have not been able to figure out...

I came across same issue when I tryed using branch/tag v0.10.0.
v0.9.0 seems to work
master as  another error I have not been able to figure out...

I got same error as @Ericat ,it worked after I just run npm run dev in iterm instead of atom terminal plugin.
https://github.com/electron/electron-quick-start/issues/113

In my case the solution appears after check the python-shell official documentation. 'electron' is correct and be sure to pass run function all the parameters that the function needs (following some tutorials, I was not writting arguments: options) also be sure to write correctly PythonShell
const { app, BrowserWindow } = require('electron')

let { PythonShell } = require('python-shell')

PythonShell.run('hello.py', null, function (err, results) {
    if (err) throw err;
    console.log('hello.py finished.');
    console.log('results', results);
});

Hope be useful!


bullet-train doesn't support 12 hour time format in prompt

For those who prefer a 12 hour timestamp (am/pm style) it would be nice to support it and have a flag to enable/disable.
I'm going to submit a PR for this in a second but just wanted to open a proper issue for it.

Hi.  Thanks for adding 12hr time support and it's a fantastic theme.  The %D{%r} switch used in the zsh- theme file however displays a format which is unusual for users of that kind of time, ie it shows as HH:MM:SS PM eg 07:48:34 PM.  Better is to use the switch %D{%l:%M%P} which would display that same time as the much more readable format H:MMpm or 7:48pm.  I changed this for myself so no problem, but maybe worth changing in the official Git file?  Cheers.


fix: Refactor the func runConfigMapWatcher to use Informers. Fixes #11657

Fixes #11657
Fixes #11463
Conflit #11799
Motivation
After some time running, when the connection to k8s API is broken (timeout for example), the watcher cannot handle the exception increases the CPU usage, and floods the log with the message level=error msg="invalid config map object received in config watcher. Ignored processing". After reaching the pod limit, it'll be restarted. See #11657  for details.
Modifications
I replaced the function with a new one using informers (many thanks @agilgur5 ).
I tried to use the code pattern used by other functions, but please let me know if something can be improved.
I also changed the namespace used to "managed-namespace" instead of "namespace" (This change will solve #11463 )
Verification
Manual changes in the ConfigMap are being watched and applied;
An instance with this change is running for a while;
We also did some integration and performance tests;

Fixes #11463

I also changed the namespace used to "managed-namespace" instead of "namespace" (This change will solve #11463 )

I don't see how this fixes #11463? The RetryWatcher was already listening to managedNamespace.
#11463 also points out the opposite -- the Controller's ConfigMap is not in the managedNamespace, it's in the Controller's own namespace. Using managedNamespace is the bug that #11463 points out.
Per my review comments in https://github.com/argoproj/argo-workflows/pull/11799#issuecomment-1714735706, there's also an issue with the semaphore notification that will have to be moved, as semaphores are in the managedNamespace

is this ok to be merged?

is this ok to be merged?

No, my previous review has yet to be addressed

Hey @juranir do you have any time to continue working on this next week? If not, I'd like to take over the work as #11657 has gotten to be one of the top issues (seemingly due to k8s 1.27 upgrades?).
I'll ofc credit you with the existing work and leave you as co-author etc 🙂

@agilgur5 any updates on this it's becoming very problematic.

The proposed suggestions by @agilgur5 and @juliev0  were implemented

While I have the context, and given the priority & impact of this, I'm just going to go ahead and make the changes requested above myself and push them to the PR

While I have the context, and given the priority & impact of this, I'm just going to go ahead and make the changes requested above myself and push them to the PR to make things faster (and reduce "telephone game" interpretations). In this review cycle I now have permissions to do that.

No worries and many thanks for that.
@agilgur5  Please let me know if I can help with something.

I'm also going to build a test image and push to a test registry for folks to try and test out

Pushed agilgur5/argo-workflow-controller:fix-configmap-informer for testing

Unfortunately, it looks like there are some file conflicts now.
Another question: which of the different configmaps have been effectively tested (either manually, unit testing, or e2e)?

I'm going through old PRs now, looking for stale ones to add "stale" label to. What's the status of this one @agilgur5 @juranir ?

It's still very necessary, although the upstream issue may have been resolved in k8s 1.30 if our hypotheses are correct, but even if that is accurate it may break again as it was due to a breaking bugfix in etcd that is now feature flagged due to the unexpected breakage. This change is still good to have and fixes other issues either way as well.
But I never got around to writing the tests, and, as far as I could tell, there were no tests for this previously, which would explain why it wasn't caught during k8s upgrades.


A race to see who will do it first (1.19.4 port)

No need for quilt port since quilt supports fabric mods (Illegal to do it the other way around. I tried that once and got consequences.)

Also, just to note, this PR makes your Mixin more efficient and improves compatibility with other mods, by simplifying the Mixin down to what it should've been, no unnecessary injections, just a plain and simple Mixin.

Thanks for your contributions. I appreciate the simplifying of the Mixin. This mod is designed to support as many Minecraft versions backwards as possible in one jar, and for fabric that appears to go all the way back to 1.16.5. So, changing the supported Minecraft version in fabric.mod.json would limit that. Also, I'm pretty sure changing the supported Java version in fast-portals.mixins.json and build.gradle would make it incompatible with 1.17.1 and earlier (or at least force players using those versions to use java 17+), but I might be wrong. If you want to change those things back, I would be happy to merge this PR.

Thanks for your contributions. I appreciate the simplifying of the Mixin. This mod is designed to support as many Minecraft versions backwards as possible in one jar, and for fabric that appears to go all the way back to 1.16.5. So, changing the supported Minecraft version in fabric.mod.json would limit that. Also, I'm pretty sure changing the supported Java version in fast-portals.mixins.json and build.gradle would make it incompatible with 1.17.1 and earlier (or at least force players using those versions to use java 17+), but I might be wrong. If you want to change those things back, I would be happy to merge this PR.

Is it ok if the minimum version usable is 1.17.x?

Don't merge this yet, as I have to test it first.

Okay it works now, everything gone green, you can merge this now.

Nevermind, it still takes 4 seconds to get in and out, completely no effect, even after I changed it from a 40 to a 20

Okay so I found out the problem; I ran this mod on a fabric server and it reported an error saying that it could not find the Player Entity class that your Mixin needed.

I'm confused as to what's happening. Did you still want to merge the optimization part? I think all that needs to be changed is the code in FastPortalsMixin.java. For me, the mod seems to work fine on both client and server, from 1.16 to 1.19.4.

I'm confused as to what's happening. Did you still want to merge the optimization part? I think all that needs to be changed is the code in FastPortalsMixin.java. For me, the mod seems to work fine on both client and server, from 1.16 to 1.19.4.

No, it's fine


docs: correctly document result output parameter

Motivation
HTTP templates set output.result to the response body, but this wasn't documented: https://github.com/argoproj/argo-workflows/blob/1f304ba6780ee043ea5ecd74e06787bcba298f8f/workflow/executor/agent.go#L260
This has confused users (e.g. https://github.com/argoproj/argo-workflows/discussions/13950#discussioncomment-11404302). Additionally, the API docs for result only mention script templates, but container templates also set it.
Modifications
Update relevant docs
Verification
Ran make docs-serve

Thanks for updating this


[UX] Target Specific Posts

implementing an argument in prompt for the user to enter the starting/ending numbers or range of posts?, ensuring they are valid before proceeding.
This will help when the user doesn't need to scrape the whole profile!

This is not off the table, but will require a substantial amount of change in the usage and I would want to do it right.
I would want to change the usage to be flag based instead of positional first. Then flags for range of posts can be implemented, either by post index or post date.
This would also be a good opportunity to to add support for downloading a single post instead of a profile.
ETA... much later. Unfortunately, life is not allowing me to focus on a coomer project for some time. I'm open to reviewing any pull requests that implement this.

Further testing has not shown any bugs with the solution to this issue.


Submitting a community blog for Pulsar's 10k+ Github stars milestone

Please answer the following questions before submitting your doc issue. Thanks!
What issue do you find in Pulsar docs?
I would like to post a community blog for Pulsar's 10k+ Github stars milestone.
What is your suggestion?
I would like to post a blog to the apache-pulsar.org site
Do you have any references?
Apache Pulsar hits 10,000 GitHub stars. .md
https://streamnative.io/blog/community/2021-12-07-pulsar-hits-10000-github-stars-milestone/
(For example, website links, etc)?
Would you like to fix this issue?
Yes
(Welcome to contribute docs! Pulsar grows from a tiny seed to a big tree with every little effort by us!

Closed as stale. Feel free to just submit a patch to https://github.com/apache/pulsar-site/tree/main/site2/website-next/blog


Restrucure and installation

If zppy is to be included into the next E3SM Unified, we probably need:

zppy executable instead of python post.py.
Installation code.
Some code restructure to follow Python convention?

Hey @xylar. zppy is a new package we would like to be included in next version of e3sm-unified 1.5.0. We are aiming for a v1 release for zppy end of May. To be able to include it in e3sm-unified, some restructure is needed, would you advice on best way to package it and make it installable through conda? We will assume users call zppy from a computer platform with e3sm-unified installed. Zppy's dependency list should be simple. Thank you!

I could help with that.  Some basics of what would need to happen to make this a python package would be:

move all the code (everything except the README.md, LICENSE and .gitignore) into a zppy directory
rename post.py to __main__.py
put the contents of post.py in a main() function that will become the entry point (python -m zppy in test mode, zppy in the conda environment)
add a setup.py in the root of the repo
add a zppy entry point that points to zppy.__main__.main()
add a recipe for the conda package

Does someone else want to take this on or should I make a PR after EGU (i.e. early May)?

Thank you, Xylar. This is great!
@forsyth2 I know you have a list of features to be worked on in May. Not sure about the best timing to do the conversion though,  are you up to do the python packaging following Xylar's instruction?

One more thing.  I'm not sure if this is an issue but it's important the zppy not try to load E3SM-Unified as part of setting up jobs, since zppy itself will live within E3SM-Unified. (It's okay to load E3SM-Unified in job scripts that it creates and launches, thought.)

Yes all our tools can/should assume the user has loaded the E3SM-unified conda package.

@chengzhuzhang The first 3 of Xylar's bullet points will be easy to implement. The last 3 I would need some more direction on.

@forsyth2, here are some tips:
This includes an example setup.py:
https://packaging.python.org/tutorials/packaging-projects/
There are lots of other examples you can find in our own repos and elsewhere on GitHub.
Here is an explanation on entry points:
https://amir.rachum.com/blog/2017/07/28/python-entry-points/
The entry point for zppy would need to be defined both in setup.py and the conda recipe.
Here is the documentation on conda recipes:
https://docs.conda.io/projects/conda-build/en/latest/concepts/recipe.html
Again, there are numerous examples in our repos and also each feedstock on conda-forge is essentially a recipe.
I will, of course, be happy to answer questions as well as review and test what you come up with.


Misc fixes for EntitySnapshot

Fix EntitySnapshot.builder().build().toContainer(); NPE
Fix EntitySnapshot.builder().build(EntitySnapshot.builder().build().toContainer()); Optional#empty()
Fix tests
Fix EntitySnapshot hash-based collections (missing hashcode/equals)

EntitySnapshots are not supposed to be treated like EntityArchetypes in that they need AT LEAST valid information from an Entity to be created. Much like BlockSnapshots in a sense for TileEntity information.
The issue with changing this is that you now make it acceptable to abuse snapshots to restore or create them as necessary, without really differing them from EntityArchetypes.

EntitySnapshots are not supposed to be treated like EntityArchetypes in that they need AT LEAST valid information from an Entity to be created

That's not the case at all. There's nothing stopping you from doing EntitySnapshot.builder().build()
This means the current system is broken. Working on it.


Navigation seems to slow down auto complete

When I have a certain file open (internally), and run a CPU profile while trying to autocomplete, I get that we're spending > 500 ms in Directory.exists:

It seems like _DartNavigationComputerVisitor calls resourceProvider.getFile for every doc comment visited in visitComment. This was introduced in https://github.com/dart-lang/sdk/commit/ae7e3517c80532eaac28c8d718e5caaf06722b46#diff-31f9a5232da9a94e12badc4bff0d6a6212b70984059e0dcab30c60e94243c510. It seems bad to me.
@scheglov can you say why _DartNavigationComputerVisitor would be doing work at this moment? Here are the steps I take to reproduce (with dummied down text):

File contains something like class C { final _x = {}; void f() { _x.entries.^ } } with cursor at the caret.
I start a CPU profile, in DevTools
I type <backspace>
I type .
I wait several seconds.
completions appear for instance members on List.

CC @gspencergoog who wrote the referenced change and @bwilkerson who might be familiar as well.

Yes, doing IO, is a very bad idea. Even if there is a need to get some additional data, this data should be cached and not recomputed every time.
I suspect that _DartNavigationComputerVisitor does this work because we wanted to implement some feature. I'm not sure if you had a more specific question that I fail to recognize.

That could easily be optimized. We don't need nearly as many existence checks, and we could probably cache the result temporarily while processing a single compilation unit (or maybe even longer term).
I have no idea why _DartNavigationComputerVisitor is running during code completion (or whether it is). Can you expand the call stack and find out where it's being run from?

_DartNavigationComputerVisitor is doing the work so that we can navigate from the link to an example to the source for the example in Flutter-style doc comments.

@scheglov

I'm not sure if you had a more specific question that I fail to recognize.

I was wondering why _DartNavigationComputerVisitor is entered during the steps I mentioned.

The reason _DartNavigationComputerVisitor is running during completion might be that completion is an async operation, so it can be interleaved with other operations running in parallel.

I'll ask a different question: What IDE events trigger entering _DartNavigationComputerVisitor? Clicking the mouse around? Typing a character into a file?

Even if it isn't interleaved sometimes, this could sometimes account for the latency we're seeing. If server has started computing navigation before the request for code completion is seen, and it's taking a long time for navigation to return, then it's might be preventing code completion from getting started.

What IDE events trigger entering _DartNavigationComputerVisitor?

Modifying a file.

Here is the rest of (a different repro of) the call to DartNavigationComputerVisitor:

Typing a character. This changes the file overlay, which triggers file analysis, and when the resolved unit is received by the server, it starts computing notifications, such as the navigation and semantic highlighting.

Sounds like we need a way to delay computing the notifications when a code completion request is already on the queue so that these tasks won't interfere with completions. Kind of sounds like a job for a scheduler.

Ah that's a good point. Since I'm typing two things: <backspace> and then ., the requests might go something like:

user types <backspace>
IDE requests completions at _x.entries^
IDE requests navigations
etc.
user types .
IDE requests completions at _x.entries.^

so one or more navigation requests are in the pipe. And may be in progress, so they can't be cancelled.

OK we will revert https://github.com/dart-lang/sdk/commit/ae7e3517c80532eaac28c8d718e5caaf06722b46 unless someone jumps on a fix forward, to dramatically reduce the introduced file system I/O.

Looking at the code more, we call out to the file system for every import directive and export directive, in _addUriDirectiveRegion, during navigation-computation (on every edit to a file).

Which probably wouldn't be a problem if we were cancelling obsolete navigation requests and scheduling them to happen after code completion requests.

The lifetime of _DartNavigationComputerVisitor  is the duration of computeDartNavigation, so you don't need to clear the cache.
Yes, a string search is much faster than file I/O and will probably take care of the performance issue.


Request: enhancements to electrode layout view

The mapping to electrode number is difficult to determine for the electrodes layout at the bottom. I would like a few more labels:

It would be great to have the electrode numbers on the circles. This could be hover-over if that makes the design smoother.
It would also be nice to have a scale bar to show what distance is in units of mm
A colorbar would be nice too

This is great! The color bar is the least important so this can be closed


Possible Missing Release Image in quay.io Image Registry

Describe the bug
The install of 4.13.0-0.okd-2023-09-03-082426 is failing for me. The error reports that the bootstrap machine cannot download a container image from quay.io. The image appears to be missing. I can confirm that I am able to login to the bootstrap machine via ssh and manually pull an existing image from the quay.io image registry. So, I've ruled out a networking or proxy problem. Please correct me if I'm wrong, but this image appears to be missing in the quay.io image registry:
...
reading manifest sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7
in quay.io/openshift/okd: manifest unknown: manifest unknown
...

Version
I'm installing OKD release: 4.13.0-0.okd-2023-09-03-082426 via the following command openshift-install create cluster --dir="/builds/devops/provision-openshift/roles/clusters/test".
NOTE: I'm allowing the openshift-install command to install my infrastructure for me in AWS. (I believe this is called the IPI installer)
How reproducible
I've run the install command above many times and am receiving the same error 100% of the time.
Log bundle
level=info msg=Credentials loaded from default AWS environment variables
level=info msg=Consuming Install Config from target directory
level=info msg=Creating infrastructure resources...
level=info msg=Waiting up to 20m0s (until 2:36PM) for the Kubernetes API at https://api.test.redacted.net:6443/...
level=info msg=Pulling VM console logs
level=info msg=Pulling debug logs from the bootstrap machine
level=error msg=Attempted to gather ClusterOperator status after installation failure: listing ClusterOperator objects: Get \"https://api.test.redacted.net:6443/apis/config.openshift.io/v1/clusteroperators\": dial tcp <IP_ADDRESS>:6443: connect: connection refused
level=error msg=Bootstrap failed to complete: Get \"https://api.test.redacted.net:6443/version\": dial tcp <IP_ADDRESS>:6443: connect: connection refused
level=error msg=Failed waiting for Kubernetes API. This error usually happens when there is a problem on the bootstrap host that prevents creating a temporary control plane.
level=error msg=The bootstrap machine failed to download the release image
level=info msg=Error: initializing source docker://quay.io/openshift/okd@sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7: reading manifest sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7 in quay.io/openshift/okd: manifest unknown: manifest unknown
level=info msg=Pull failed. Retrying quay.io/openshift/okd@sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7...
level=info msg=Error: initializing source docker://quay.io/openshift/okd@sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7: reading manifest sha256:59b1e486ed132fa084f654ca930d84a751c7956870f659ef522a9928bc200ec7 in quay.io/openshift/okd: manifest unknown: manifest unknown
level=info msg=Bootstrap gather logs captured here \"/builds/devops/provision-openshift/clusters/test/log-bundle-20230914143606.tar.gz

I suppose I can rollback to an earlier stable release, but I thought I would let you know about this issue. I appreciate your help and thank you for your time and efforts on this.

Right, there is no sha256:59b1e486ed... image listed in the payload - so perhaps uploaded installer is invalid?
Could you check if installer extracted from the payload works? oc adm release extract --command=openshift-install --from=quay.io/openshift/okd:4.13.0-0.okd-2023-09-03-082426

Hmm, I'll need to reupload the files attached to the release then

Files reuploaded, installer now points to existing release


Operations on different resultsets using similar-nested query

Feature Request
Proposal: [Description of the feature]
It should be nice to perform mathematical operations (+,-, ...) on results of different queries.
Example queries:
query_E = SELECT sum(value) AS E FROM trevano_CPT_roof WHERE item='Pyranometer' AND signal='IrrEst' and time>='2016-05-27T00:00:00Z' and time<='2016-05-27T23:59:59Z'
name: trevano_CPT_roof
query_H = SELECT sum(value) AS H FROM trevano_CPT_techroom WHERE item='Inverter1' AND signal='kW_tot' and time>='2016-05-27T00:00:00Z' and time<='2016-05-27T23:59:59Z'
name: trevano_CPT_techroom
Current behavior: [What you would like to happen]
query_E
time			E
1464307200000000000	2.749669275633156e+07
query_H
time			H
1464307200000000000<PHONE_NUMBER>0008276
Desired behavior:  [Why is this important (helps with prioritizing requests)]
SELECT E/H AS ratio FROM query_E, query_H

time			H
1464307200000000000<PHONE_NUMBER>0008276/2.749669275633156e+07
Use case:
Sometimes I need to perform simple calculations between results directly on Grafana. Typically this happens when I have to calculate and show ratios as in the example above.

+1

Duplicate of #3552.


docs(aboutmodal): remove content component dependency in content area

fixes https://github.com/patternfly/patternfly-next/issues/1754

Deploy preview for pf-next ready!
Built with commit bbc20a2a27723a9706fba1c49a1fc8553ee7d296
https://deploy-preview-1768--pf-next.netlify.com

:tada: This PR is included in version 2.6.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


This simplifies the workflow (and maybe it can also be the solution for #28: vuepress-plugin-git-log injects each page with the $page.git.updated and $page.git.commits variables).

Thanks a lot for the contribution!
Unfortunately, this is a solution I tried and then excluded because it's tied to Vuepress.
One of the principles (see the RFC) of this project is keeping Pandocs in a agnostic format (markdown) and use tools able to consume this format to produce the final documents. Currently we only have "2 tools":

Vuepress produces an HTML version,
the Pandoc toolchain produces a PDF build

vuepress-plugin-git-log would mean to adopt an additional solution to add the same data but on the other tool (because pandoc takes the plain MD file), thus duplicating the effort.
Currently, the metadata gets added directly in the MD, so it's available to whatever tool makes use of the markdown file.
However, this could be useful when/if we decide to render every single page also in a different HTML page (so an additional version to the "merged" one).
Let me know if I explained myself, @bfabio

@avivace Thanks for the project!

One of the principles (see the RFC) of this project is keeping Pandocs in a agnostic format (markdown) and use tools able to consume this format to produce the final documents.

That makes a lot of sense.
My reasoning behind this PR was to use just one environment (vuepress) and absorb in it the features handled manually (merge.sh and markdown-pp). After all, a documentation system like vuepress should cover all our bases.
I felt we were kinda fighting vuepress and in the long run I think it can end up biting us (especially if we can't use its advanced features), negating all the goodies we can get from it and forcing us into a limbo of "almost standard vuepress, but not quite".
Also, merge.sh is kinda awkward and does that unexpected trick with About.md which makes me uncomfortable.
So, I think ideally a single tool supporting what we need out of the box from the get go would be great, but that tool wouldn't be vuepress.
Have you considered readthedocs?
It's powerful, themeable, extensible (templating the index shouldn't be a problem), supports reStructuredText (which can be a even finer format for documentation) and a ton of conversion formats via Sphinx.
I think it can check all our boxes.

@bfabio Sorry for the late reply, busy time.

Also, merge.sh is kinda awkward and does that unexpected trick with About.md which makes me uncomfortable.

What are you referring to? The way I'm injecting the revision data?
I think everything you mentioned makes a lot of sense. My main concern here is burying the project (if we change the infrastructure) under more content-unrelated work that could slow things down now that we are able to accept contributions and actually push the research forward (one of the main priorities).
I don't actually have time to take a look at the alternative (but I know readthedocs and I really like it), although it seems promising, especially considering that the only requisite I want from the frontend build is being able to use our CSS.
Feel free to work on proof of concepts and ask for feedback here (or on our Discord server where you can get more instantenous feedback).

What are you referring to? The way I'm injecting the revision data?

Yes, in particular changing a source file, even for a brief amount of time, was unexpected to me. It's even more visible if markdown-pp fails for some reason and you are left wondering who changed it.
It's also awkward if we wanted to Dockerize the whole thing, because it forces us to keep track of the file UID to restore it after the change.
It would be cleaner to just add
This document version was produced from git commit ["$shortcommithash"](https://github.com/gbdev/pandocs/tree/"$commithash") ("$timestamp")."
to index.mdpp and commit the change, as  if$shortcommithash and the others were magic variables the doc system provides. WDYT?

Feel free to work on proof of concepts and ask for feedback here (or on our Discord server where you can get more instantaneous feedback).

I'll be happy to give it a go, I just joined the Discord server :)


[youtube:tab] recommended: Unable to recognize tab page

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.02.17 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Reading home page works, but reading home page with cookies does not.
This works
yt-dlp -vU https://www.youtube.com --flat-playlist --lazy-playlist --print '%(id)s'

And this doesn't
yt-dlp -vU https://www.youtube.com --flat-playlist --lazy-playlist --print '%(id)s' --cookies-from-browser chrome

Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://www.youtube.com', '--flat-playlist', '--lazy-playlist', '--print', '%(id)s', '--cookies-from-browser', 'chrome']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2023.02.17 [a0a7c01] (pip)
[debug] Python 3.11.2 (CPython x86_64 64bit) - macOS-12.6.2-x86_64-i386-64bit (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[Cookies] Extracting cookies from chrome
[debug] Extracting cookies from: "/Users/brahim/Library/Application Support/Google/Chrome/Default/Cookies"
[debug] using find-generic-password to obtain password from OSX keychain
[Cookies] Extracted 3189 cookies from chrome
[debug] cookie version breakdown: {'v10': 3168, 'other': 0, 'unencrypted': 21}
[debug] Proxy map: {}
[debug] Loaded 1782 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2023.02.17, Current version: 2023.02.17
yt-dlp is up to date (2023.02.17)
[youtube:recommended] Extracting URL: https://www.youtube.com
[youtube:tab] Extracting URL: https://www.youtube.com/feed/recommended
[youtube:tab] recommended: Downloading webpage
WARNING: [youtube:tab] Incomplete yt initial data received. Retrying (1/3)...
[youtube:tab] recommended: Downloading webpage
WARNING: [youtube:tab] Incomplete yt initial data received. Retrying (2/3)...
[youtube:tab] recommended: Downloading webpage
WARNING: [youtube:tab] Incomplete yt initial data received. Retrying (3/3)...
[youtube:tab] recommended: Downloading webpage
WARNING: [youtube:tab] Incomplete yt initial data received; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U. Giving up after 3 retries
ERROR: [youtube:tab] recommended: Unable to recognize tab page; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/usr/local/Cellar/yt-dlp/2023.2.17/libexec/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 693, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/yt-dlp/2023.2.17/libexec/lib/python3.11/site-packages/yt_dlp/extractor/youtube.py", line 4454, in wrapper
    info_dict = func(self, url, smuggled_data)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/yt-dlp/2023.2.17/libexec/lib/python3.11/site-packages/yt_dlp/extractor/youtube.py", line 6305, in _real_extract
    raise ExtractorError('Unable to recognize tab page')

I can't repro. Could you add --dump-pages and attach the resulting .dump files?

yt-dlp -vU https://www.youtube.com --flat-playlist --lazy-playlist --print '%(id)s' --cookies-from-browser chrome --write-pages --dump-pages 2> page-dumps

Archive.zip

Pretty sure this is supposed to fallback to the API.
Seems like YouTube is not giving the full browse response as part of the webpage and getting it from the API after the fact.  We've seen it before but taking we never implemented support for it.
window['ytCommand'] = {"clickTrackingParams":"XXX","commandMetadata":{"webCommandMetadata":{"url":"/","webPageType":"WEB_PAGE_TYPE_BROWSE","rootVe":3854,"apiUrl":"/youtubei/v1/browse"}},"browseEndpoint":{"browseId":"FEwhat_to_watch"}}

var ytInitialData = {"responseContext":{"webResponseContextExtensionData":{"ytConfigData":{"visitorData":"XXX%3D%3D","sessionIndex":0,"rootVisualElementType":3854}}}}

Here's another scenario where we get incomplete data received, but then it returns a few video ids. Can we safely ignore, or does it need a patch?
yt-dlp --skip-download https://www.youtube.com/channel/UCI197iM3KpFWEqef07Owfbw/streams --get-id --flat-playlist

See how it keeps loading infinitely on the webpage.

That is what causes the "incomplete data" error. It is issue on youtube's side and we can't do anything. The video ids yt-dlp returns are whatever is above that point.


Can't get osc authorization

I'm getting this error
HTTPSConnectionPool(host='api.openstreetcam.org', port=443): Max retries exceeded with url: /auth/openstreetmap/client_auth (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1123)')))
Python 3.9.1
macOS 11.2.1 (20D74)

Should no longer be a problem. This was addressed a long time ago.


Clarify that config files require unique keys

From https://github.com/atom/atom/issues/7552#issuecomment-116735581
A lot of people try to use the config files as they would a stylesheet, trying to override keybindings for instance by repeating the same selector multiple times in the file, which causes issues because CSON keys must be unique. This makes it a bit more explicit that keys need to be unique.

Looks good to me! :boat:

:zap: @mnquintana for this -- helping users not make these mistakes would be great. I left two comments  -- wondering what your :thought_balloon: are.

@izuzak Just opened up https://github.com/atom/docs/pull/100 - I'll link to that new section from within these comments.

:sunglasses: Thanks again for this, @mnquintana


How to filter results using azure-spring-data-cosmos-core-query Criteria and CriteriaType "ARRAY_CONTAINS"

Is your feature request related to a problem? Please describe.
I want to filter the result set using azure-spring-data-cosmos-core-query Criteria and CriteriaType "ARRAY_CONTAINS".
I have the Result set structure similar to this:
{
"id": "UUID",
"name": "…",
"emailAddress": "…",
"version",
"arrayField": [
{
"id": "UUID",
"status": "active",
"name": "abc"
},
{
"id": "UUID",
"status": "draft",
"name": "xyz"
},
{
"id": "UUID",
"status": "active",
"name": "qwe"
},
…
],
"country": "",
…
}
I would like to query on arrayField.id which is a unique field.
The implementation for the ARRAY_CONTAINS is ARRAY_CONTAINS(object.arrayField, { "id": "UUID_value" }, boolean), where the boolean value "true" implies to match the partial array and "false" implies otherwise.
Since I'm using the Criteria to build my query, there is no supported method of getInstance in the Criteria class to set the boolean value for ARRAY_CONTAINS condition.
So, the boolean value is "false" by default and I'm not able to filter the results for partial array match.
Every time the result set is empty because it is matching the entire array not just one field.
Describe the solution you'd like
I would like to have an additional boolean field in the Criteria class to set the value for ARRAY_CONTAINS condition.
Also an additional method of getInstance which will accept that boolean value.
This would also need some changes in the azure-spring-data-cosmos-core-generator AbstractQueryGenerator for generating the ARRAY_CONTAINS query.
Describe alternatives you've considered
This could be done using the JOINS but then I cannot use the Criteria implementation and also I cannot use SELECT * from root.
I also found a similar issue at https://github.com/Azure/azure-sdk-for-java/issues/16676. But that is restricted to sql query implementation and not using the Criteria class.
Additional context
Add any other context or screenshots about the feature request here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

@backwind1233 @yiliuTo could you please follow up with @rajesh-melige on this question?

@mbhaskar  - can you please help on this issue ?

@mbhaskar @kushagraThapar May I know if you have any update on this issue ? We are blocked on this implementation.

@mbhaskar @kushagraThapar can you write if you have any updates on this issue?

@shitikovegor - one of the customers is using this as a solution - https://github.com/Azure/azure-sdk-for-java/issues/25628#issuecomment-992101766
Let me know if that helps.

@kushagraThapar  everything work with @Query annotation, but I mean this issue is about 'ARRAY_CONTAINS' criteria in method name as keyword

@trande4884  - can you please take a look at this, thanks!

@shitikovegor are you able to post the code snippet of the Criteria you are trying to create with Criteria.getInstance()? This will greatly assist me in debugging this issue.

@shitikovegor per the documents here, the boolean flag for partial matching only pertains to ARRAY_CONTAINS with objects. Our SDK does not currently support this. Criteria only supports ARRAY_CONTAINS with single property values.


💄 [LLM][LLD]: Flex assets

✅ Checklist

[ ] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
Flex asstets integration in LLM/LLD. Rollback CI workflow modifications

❓ Context

JIRA or GitHub link:

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/generate-screenshots

/generate-screenshots

I'm not sure that this is necessary to add changesets for this PR, as those changes are already in prod


Badge for compliant projects?

Hey, thanks for this, and for the talk at SOSS Fusion!
I have a question/suggestion - should there be an official badge, which projects can put on their readme, so signify that their project provides attestations?
I'm interested in this for Narwhals where I think I've done things correctly on the publishing side:
In [7]: packages = [{'name': 'narwhals'}]

In [8]: annotate_wheels(packages)
Getting wheel data...
1 1 narwhals

In [9]: packages
Out[9]:
[{'name': 'narwhals',
  'wheel': True,
  'value': 1,
  'css_class': 'success',
  'icon': '🔏',
  'title': 'This package provides attestations.'}]

and wanted to know what the best way (if any) would be to communicate PEP740 compliance on the README

Thanks for filing an issue @MarcoGorelli!
We don't have any kind of official badge for this, but you could make one for yourself using https://shields.io.
I played around with shields' dynamic XML support, and came up with this:
https://img.shields.io/badge/dynamic/xml?url=https%3A%2F%2Fpypi.org%2Fsimple%2Fsampleproject%2F&query=%2F%2Fa%5B%40data-provenance%5D%5Blast()%5D&logo=pypi&label=PEP%20740

which renders as:

This isn't quite perfect though, since shields doesn't seem to support XPath functions like booleans, so it returns the mostly recently attested file (or none) instead of a bool.
You can also experiment with it a bit yourself here: https://shields.io/badges/dynamic-xml-badge
(You might also be able to use PyPI's JSON API + dynamic JSON in shields for this; I didn't try!)

thanks @woodruffw ! sure, i've done that


Refactor: Replace Remaining '__x' Instances with '__id' #944

Description:
This PR addresses issue #944 by refactoring the codebase to replace all remaining instances of '__x' with the more descriptive '__id'. The changes aim to improve code readability and maintain consistency throughout the project.
Changes Made

Replaced '__x' with '__id' across the codebase, ensuring consistency in variable and function naming conventions.
Checked and updated affected files in compliance with the project's style guide.
Conducted comprehensive testing to verify that the changes do not introduce any new issue.

Related Issue
Closes #944
Please review and merge this PR. Thank you!

Thank you @ericniebler for the detailed explanation. Really sorry about the poor changes pushed earlier, I didn't go through the console, that was a blunder from my end.
I went through the code base and realized that the changes requested for class templates already implemented in some places. I'd really appreciate if you can share the list of files where these changes are required to be added.

You've already identified the places that need to change. Any place a class template was being instantiated with an argument wrapped in __x, that class template needs to be refactored.

Closing. Feel free to reopen if and when you to come back to this.


multiple endpoints will be created if has path variable in URL

Please answer these questions before submitting your issue.

Why do you submit this issue?
[X] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
Hi, we have an api which url is /find/{userId}/{companyId}. When this api is called with different id, a new endpoint instance will be persisted in endpoint inventory. Is there any solution to avoid this problem? We just want to treat it as one endpoint.

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

Is this your private framework? SpringMVC has dealt with this case.

It's a spring boot project.

SpringMVC will fetch the path automatically, if you have set the parameter({param}) in the controller path.

SpringMVC will fetch the path automatically, if you have set the parameter({param}) in the controller path.

I also have the same problem. the url is /api/userCrowd/{adSystem}/{userId}, while the result has two kind of endpoints. /api/userCrowd/{adSystem}/{userId} and /api/userCrowd/YT/001, the throughput of the service is double of the requests. Is there any properties to set to fix this problem?

Is there any properties to set to fix this problem?

Maybe you are using webflux as web container? If so, remove the plugin.

Is there any properties to set to fix this problem?

Maybe you are using webflux as web container? If so, remove the plugin.

Thanks for reply. In my project I use undertow as the embed container in springboot. when I switch to tomcat, the problem is fixed.

Then simply remove the undertow plugin, because of the async http framework, two entry span detected, that is why the CPM is not accurate.


Change to use the pgx:develop branch

For now, lets have plrust use pgx' develop branch, so we can move forward with the "trusted-pgx" crate.
This PR makes that change and updates the code (mostly around Spi) so that it compiles.
Also note that the user function dependency on pallocator has been commented out -- I believe that needs some TLC against pgx:develop as well.

I approve this PR.

I approve this PR.


Fuction 'ToInt', which defined in an anonymous namespace, is never used

https://github.com/google/REAPER/blob/a3926d3c256bcdc500e7b524ebed56c05a5fbc3a/core/track.cc#L31

track.cc is a generic library used across different tools, not only reaper. I'll leave it there since it doesn't seem to be problematic.


[QoI] Don't try to lookup members on incorrect type while diagnosing keypath components

While trying to diagnose problems related to keypath components
don't assume that type of the component is always correct, check
before trying to see if it's bridged type or has members.

Resolves SR-5370.
Resolves: rdar://problem/33135487

@swift-ci please smoke test


Finish needed tests for MarketContractOraclize

Basic tests have been written but need to be extended to cover the following scenarios

attempt to overfill (fill a maker order with more qty than the order is for acting as a taker) an order
should fail only allowing remaining qty to be filled
attempt to over cancel (cancel more qty than order has available to fill)
should fail only allowing remaining qty to be cancelled
attempt to fill expired order
should fail, no trade should occur
attempt to trade zero qty
should fail, no trade should occur
attempt to create maker order and fill as taker from same account
should fail, no trade should occur
attempt to fill and order with the price changed
should fail, no trade should occur
attempt to cancel an already expired order
should fail

I would like to jump on this if no one else is already working on it.

Cool, I would also like to dive in. @perfectmak feel free to start and finish. I will have a look in the evening to set up dev environment anyway.

Thanks for the interest in the project and bounty @perfectmak and @cryptomental!  Please let me know if you have any questions getting the tests running.

Hi @pelsasser it took me about an hour to go through the initial setup as I had colliding nodejs's. Finally I ended up dockerizing the ethereum bridge thing (partially for now) and hurray
  TestMathLib
    ✓ testSign (170ms)
    ✓ testIsSameSign (131ms)
    ✓ testAbs (117ms)
    ✓ testAbsMin (276ms)
    ✓ testAbsMax (230ms)
    ✓ testCalculateNeededCollateral (98ms)

  TestOrderLib
    ✓ testCreateOrder (431ms)

  Contract: CollateralToken
    ✓ Initial supply should be 1e+22 (38ms)
    ✓ Main account should have entire balance
    ✓ Main account should be able to transfer balance (96ms)

  Contract: MarketCollateralPool
    ✓ Both accounts should be able to deposit to collateral pool contract (727ms)
    ✓ Both accounts should be able to withdraw from collateral pool contract (321ms)

  Contract: MarketContractOraclize
    ✓ Trade occurs, cancel occurs, balances transferred, positions updated (1832ms)
    ✓ Trade is unwound, correct collateral amount returns to user balances (1178ms)

  Contract: OrderLib
    ✓ Orders are signed correctly (399ms)

  15 passing (11s)

Once I finish with this task I will try to clean up the dockerfiles and submit a separate PR. Having dockerized test harness would allow everyone to have a clean test environment. The idea would be that with a single docker-compose up you would get test results on any Linux/MacOS workstation. I am not sure about Windows, have not tried docker there yet. I myself am working on a few machines with Ubuntu 14.04 LTS 64 bit and 16.04 LTS 64bit.
So, finally, I can start ;)

@cryptomental - thank you, this is great.  I have not personally used docker, so will be excited to get it set up and try it out.  Sorry that the set up was a pain.  I am also running this on Ubuntu 16.04.

I am unfamiliar wth gitcoin and the process of how this works,  I jumped in and 'claimed', I am happy to share with others.  I have already coded 4 of the 7 tests.

@pelsasser I just submitted a PR for the tests on this issue. Kindly take a look and revert. I'll be looking forward to your feedback.

@cryptomental  Great idea with adding Docker. It would make it easier to setup and test.

Thanks so much for the help everyone, I am closing this issue - there are many more tests to be written and work to be done.  I will try and create some more issues to get the ball rolling on them.

@cryptomental - I opened a new issues for the docker container you mentioned #7


feat: Share files

I've re-run the tests, but one of your tests is still failing. Please check what's happening.

@MiguelMRojas You can implement the un-share tests now.


LoRA Dreambooth Dependency Error

Error on installing dependencies for current Collab version of LoRA Dreambooth:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts. tf-keras 2.15.1 requires tensorflow<2.16,>=2.15, but you have tensorflow 2.16.1 which is incompatible. torchaudio 2.2.1+cu121 requires torch==2.2.1, but you have torch 2.1.0 which is incompatible. torchtext 0.17.1 requires torch==2.2.1, but you have torch 2.1.0 which is incompatible.

try change xformers to !pip install xformers==0.0.24 --no-deps

try change xformers to !pip install xformers==0.0.24 --no-deps

where should I put it? I tried both in requirements.txt and the main but it doesn't work

main or requirements should work, did you replaced the old one?
or if still can't i have fork that worked, you might can try it : https://github.com/DEX-1101/kohya-trainer/blob/main/kohya_LoRA_trainer_XL_v1.ipynb


Issues W/ useEffect

Hello! Awesome Repo! I filled the other missing useEffect Dependencies, but when doing the getMessagesAndSubscribe Dependency, It Says:
Compiled with warnings.
src/context/appContext.js
Line 18:19:  'setIsGuest' is assigned a value but never used                                                                                no-unused-vars
Line 73:16:  'getMessagesAndSubscribe' was used before it was defined                                                                                no-use-before-define
Line 108:9:  The 'getMessagesAndSubscribe' function makes the dependencies of useEffect Hook (at line 73) change on every render. To fix this, wrap the definition of 'getMessagesAndSubscribe' in its own useCallback() Hook  react-hooks/exhaustive-deps
Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.
but when i remove that dependency it says:
Compiled with warnings.
src/context/appContext.js
Line 18:19:  'setIsGuest' is assigned a value but never used                                                                             no-unused-vars
Line 73:6:   React Hook useEffect has a missing dependency: 'getMessagesAndSubscribe'. Either include it or remove the dependency array  react-hooks/exhaustive-deps
Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.
Any Help? @shwosner @winnekes @AsherCarneiro

Have these same errors when running the code

@talmax1124 @arpitjacob
Are these warnings stopping your code from running?

Yes! @shwosner

@talmax1124
Seems to be strict es-lint rules that are blocking you, no?
Is it possible to share a codesandbox link?

@shwosner Could we hop on a discord call and work on it together from scratch to see if I'm doing something wrong?

@shwosner Could we hop on a discord call and work on it together from scratch to see if I'm doing something wrong?

Not at the moment, I'll contact you later by email.

Ok, what time zone are you in? So we could co-ordinate a time @shwosner

Ok, what time zone are you in? So we could co-ordinate a time @shwosner
New York

ok, I'm Est. Florida. So we have the same time. 5pm today works? @shwosner

ok, I'm Est. Florida. So we have the same time. 5pm today works? @shwosner

Yes, please email me at<EMAIL_ADDRESS>to continue the conversation.

In case anyone else runs into this same issue.
Try adding this to the first line in src/context/appContext.jsx
/* eslint-disable react-hooks/exhaustive-deps */


iOS video trimming/cropping facility not available in landscape mode

The video trimming facility (along the top of the screen) is not available after taking a video on iOS devices in landscape mode.  It is available if taking a video in portrait mode or if the device is rotated to portrait mode after creating the video in landscape mode.
If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Bug Information
Version Number of Plugin: 3.01
Device Tested On: iPad Pro 10.5
Simulator Tested On: None
Version of VS: 10.3.3
Version of Xamarin: Forms <IP_ADDRESS>
Versions of other things you are using: As pulled down by MediaPlugin
Steps to reproduce the Behavior
Take a video in portrait mode.  After recording, but before using the video, a bar appears across the top of the video allowing the video to be trimmed.  Do the same in landscape mode and the "trim" bar doesn't appear.  Rotate the device to portrait mode and the "trim" bar appears.
Expected Behavior
The "trim" bar should appear in both landscape and portrait mode.
Actual Behavior
See Steps to reproduce above
Code snippet
var storeVideoOptions = new StoreVideoOptions
{
DefaultCamera = CameraDevice.Rear,
DesiredLength = new TimeSpan(0, 0, 10),
Quality = tempVideoQuality,
};
storeVideoOptions.AllowCropping = true;
var file = await CrossMedia.Current.TakeVideoAsync(storeVideoOptions);
Screenshots
Landscape (without trim)

Portrait (with trim)

Unable to reproduce on iOS 10/11 iPhone6 and iPad Pro.
This is the default controls so out of my control anyways.

Thanks for investigating James.  Much appreciated.


WIP implementing metrics caching

Related issue: https://issues.redhat.com/browse/APPSRE-6250

Hey @janboll !
Are you planning to do something about https://github.com/app-sre/aws-resource-exporter/blob/master/proxy.go ? @cblecker  mentioned yesterday that the caching functionality had already been implemented as part of the rds exporter in the past. i think it'd made sense to combine these two into one solution so we don't end up with two things implementing the same functionality

I ran the current code locally and having different labels will lead to only the latest metric being stored in the cache, because the key-hash does not have the actual values for the labels:
func getMetricHash(metric prometheus.Metric) string {
	fmt.Printf("Generating hash for %v\n", metric.)
	checksum := sha256.Sum256([]byte(metric.Desc().String()))
	return fmt.Sprintf("%x", checksum[:])
}

This outputs (as an excerpt):
Generating hash for Desc{fqName: "aws_resources_exporter_vpc_interfacevpcendpointspervpc_usage", help: "The usage of interface vpc endpoints per vpc", constLabels: {}, variableLabels: [aws_region vpcid]}
Generating hash for Desc{fqName: "aws_resources_exporter_vpc_interfacevpcendpointspervpc_usage", help: "The usage of interface vpc endpoints per vpc", constLabels: {}, variableLabels: [aws_region vpcid]}
Generating hash for Desc{fqName: "aws_resources_exporter_vpc_interfacevpcendpointspervpc_usage", help: "The usage of interface vpc endpoints per vpc", constLabels: {}, variableLabels: [aws_region vpcid]}
Generating hash for Desc{fqName: "aws_resources_exporter_vpc_interfacevpcendpointspervpc_usage", help: "The usage of interface vpc endpoints per vpc", constLabels: {}, variableLabels: [aws_region vpcid]}
Generating hash for Desc{fqName: "aws_resources_exporter_vpc_interfacevpcendpointspervpc_usage", help: "The usage of interface vpc endpoints per vpc", constLabels: {}, variableLabels: [aws_region vpcid]}

Which means it will loose metrics with different labels right now.

Compared the metrics from running the exporter using the main branch and this branch on one of our APP-SRE AWS accounts and did see no diff in the results (except for metric values, which is expected)

Done, I changed timeout to be optinal and added the default values to the Readme

Bypassing branch protection since @mrWinston is listed as approver https://github.com/app-sre/aws-resource-exporter/blob/master/OWNERS#L15


Issue with k8s.io/docs/setup/production-environment/container-runtimes/

This is a Bug Report

Problem: CRIO installation part: in the CLI, the crio version is outdated and not available anymore.
Proposed Solution: In the section "CRI-O", in the tab "Ubuntu 16.04", change the line "apt-get install cri-o-1.11" by "apt-get install cri-o-1.13"
Page to Update:
https://kubernetes.io/...

/assign

/language en


Unable to parse request for presentation openID4VP

Hi team
I am trying to implement OpenID4VP in Bifold after we successfully implemented the credential issuance and acceptance flow.
I am using such blockchain issuer now as means of testing issuance and verification and Paradym wallet as my resource for implementation.
I am getting this error when trying to resolve presentation request
Invalid token specified: invalid base64 for part #1 (Property 'atob' doesn't exist)
That seems coming from
agent.modules.openId4VcHolder.resolveSiopAuthorizationRequest(requestUri)
=> const verifiedAuthorizationRequest = await openidProvider.verifyAuthorizationRequest(requestJwtOrUri)
I tried both scenarios, passing the uri => failed with same error
extracting certificate from auth request => adding to agent trusted certs => passing Jwt : also fails with same error
In paradym wallet, it works fine
I made sure those sphereon libraries are installed though
    "@sphereon/did-auth-siop": "0.16.1-next.66",
    "@sphereon/oid4vc-common": "0.16.1-next.66",
    "@sphereon/oid4vci-client": "0.16.1-next.66",
    "@sphereon/oid4vci-common": "0.16.1-next.66",
    "@sphereon/oid4vci-issuer": "0.16.1-next.66",
    "@sphereon/ssi-types": "0.29.1-unstable.121",

It is used in @sphereon/did-auth-siop specifically to get the authorization from uri or JWT, the expectation comes from this function in credo-oidc
const verifiedAuthorizationRequest = await openidProvider.verifyAuthorizationRequest(requestJwtOrUri)
I believe RN doesn't have a built in decoding for base64 but its working in paradigm with no issues
Am I missing a polypfill or something
Bifold is at RN 0.72.5
And we are using latest credo 0.5.11

Okay and are you using hermes? Might be the react native version / hermes.
Is the sphereon lib using atob method directly? Or through a dependency?

I can see that its using
import { jwtDecode } from 'jwt-decode';
so through a dependency I believe
And yes we are using hermes

@TimoGlastra  I solved it :D
I added base64 polypill and defined it to be global in index.js and tested. The presentation request came in and was decoded successfully


[Question] Tensorflow 2 wrapper: init already called before, but still complaining

I am following tutorial here,  code:
hugectr_tf_ops.init(visiable_gpus=[0], seed=seed, key_type='int64', value_type='float',
                            batch_size=self.batch_size, batch_size_eval=self.batch_size)
...
emb = PluginEmbedding(vocabulary_size=vocabulary_size, slot_num=slot_num,
                                           embedding_vec_size=dims,
                                           embedding_type=embedding_type,
                                           gpu_count=gpu_count, initializer=initializer)

However, the error still occurs,
  File "<string>", line 273, in hugectr_create_embedding
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 750, in _apply_op_helper
    attrs=attr_protos, op_def=op_def)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 3540, in _create_op_internal
    op_def=op_def)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1994, in __init__
    self._traceback = tf_stack.extract_stack()

I1228 02:45:49.556892<PHONE_NUMBER>45056 monitored_session.py:1293] An error was raised. This may be due to a preemption in a connected worker or parameter server. The current session will be closed and a new session will be created. This error may also occur due to a gRPC failure caused by high memory or network bandwidth usage in the parameter servers. If this error occurs repeatedly, try increasing the number of parameter servers assigned to the job. Error: 2 root error(s) found.
  (0) Unavailable: /HugeCTR/tools/embedding_plugin/cc/kernels/embedding_plugin_create_embedding.cc:/HugeCTR/tools/embedding_plugin/cc/kernels/embedding_plugin_create_embedding.cc There is no wrapper instance, you should call hugectr.init() first.
	 [[node inputs/ctr_SynCate1Cos/HugectrCreateEmbedding (defined at <string>:273) ]]
	 [[lambda_loss/lambda_loss_7/Greater/_925]]
  (1) Unavailable: /HugeCTR/tools/embedding_plugin/cc/kernels/embedding_plugin_create_embedding.cc:/HugeCTR/tools/embedding_plugin/cc/kernels/embedding_plugin_create_embedding.cc There is no wrapper instance, you should call hugectr.init() first.
	 [[node inputs/ctr_SynCate1Cos/HugectrCreateEmbedding (defined at <string>:273) ]]
0 successful operations.
0 derived errors ignored.

Could you please help?

Hi @serser , You are using TensorFlow 2.x, right?
Could you please provide some code snippet about how you used those APIs, so that we can try to reproduce this issue. Thanks.

Hi @serser , You are using TensorFlow 2.x, right?
Could you please provide some code snippet about how you used those APIs, so that we can try to reproduce this issue. Thanks.

Hi Jianbing,  I am with the official docker image, so yes tf2.5-dev. It turns out that I've put hugectr_tf_ops.init in a func call that constructs the embedding layer. Instead when I moved init to model definition, it is initiated all right. Thanks again. But I've got new issues, please see https://github.com/NVIDIA/HugeCTR/issues/198.