Binding generator fails since godot 4.2 beta

Since 4.2 beta2 the generator crashes because it stores int sized enum values but in json file there is now pretty big values like this one.
"name": "ARRAY_FLAG_FORMAT_VERSION_2",
"value":<PHONE_NUMBER>8

edit:
While the generator itself is a one-liner fix, it is now pops up everywhere as we use ints (godot uses long and double by default), e.g.
classes\godot\renderingserver.d(677,29): Error: cannot implicitly convert expression `34359738368L` of type `long` to `int`
classes\godot\renderingserver.d(679,35): Error: cannot implicitly convert expression `34359738368L` of type `long` to `int`

edit2:
Another issue is that we have some fancy naming and that now conflicts with type names
classes\godot\editorplugin.d(110,38): Error: template instance `GodotMethod!(EditorInterface)` does not match template declaration `GodotMethod(Return, Args...)`
classes\godot\editorplugin.d(1158,18): Error: function `godot.editorinterface.EditorInterface` is used as a type

EditorInterface is a getter for that singleton, that type name is actually EditorInterfaceSingleton

Fixed with #143


Code release

Hi,
Congratulations for your paper. It presents interesting insights.
Are you planning to release the code for noise modelling, and if yes, could you let us know a tentative time/date for the same?
Thanks!

Hi, the code will be released at the time of official hold of CVPR at the latest.

Hi, CVPR was already hold. Could you please release the code?

Hi, CVPR was already hold. Could you please release the code?

I'm cleaning the code and uploading the dataset. It will be released soon


CI: Gemfile complains about platforms

This kind of message

Your bundle only supports platforms ["arm64-darwin-22"] but your local platform
is x86_64-linux. Add the current platform to the lockfile with
bundle lock --add-platform x86_64-linux and try again.

Out of a fuller message in https://github.com/camping/camping/actions/runs/7482002838/job/20364696736
/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle config --local path /home/runner/work/camping/camping/vendor/bundle
/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle config --local deployment true
Cache key: setup-ruby-bundler-cache-v6-ubuntu-22.04-x64-ruby-3.2.2-wd-/home/runner/work/camping/camping-with--without--only--Gemfile.lock-58733e68547864def4455dfff7ed7e384a66cec2df61c03c7e0092270eac62f9
/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle install --jobs 4
Your bundle only supports platforms ["arm64-darwin-22"] but your local platform
is x86_64-linux. Add the current platform to the lockfile with
`bundle lock --add-platform x86_64-linux` and try again.
Error: The process '/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle' failed with exit code 16

Fixed in #165.


Milestone-1: Data Architecture

Dear TSE,

We created our data plan on lucid diagrams. :heavy_check_mark:
Saved the files in docs folder. :heavy_check_mark:

With all our respects...

Hello @kubilaycaglayan and @mariosknl
You have done great so far  However, you have a few changes to work on before your application can be approved.

[ ]  In the user records, add image column

NOTE: You have done more than asked for in this milestone. It is always good to follow instructions
Make the changes and submit for another code review. Good luck!

Happy coding

By Raymond
slack @IAmRaymond

Hey guys, I have just realized, I made a mistake. Sorry, the project is built on top of the existing one.  Change the image column in users record and submit for another review.

Hello guys, I'm Sarah, your reviewer for this PR 👩🚀

Good job!

Your project is approved! 🎊 🥅 💛
You may close this Pull Request and merge the changes

Keep being an awesome dev and good luck with your next project!

Cheers and Happy coding!👏👏👏
Code review by Sarah Uryelah Silva Chamorro
(Feel free to contact me on slack by the handle of  sarah if you have any additional question)

Hey guys, the file you need to change is actually he ERD diagram saved at docs/KubilayMarios.pdf
Please add the user_image property to the diagram, not just the txt file.


ci: pull request template

✨ Context
As discussed in @RobinM-code in #510, we would benefit from having a PR template so that contributors understand what are the expectations when opening new PRs.
🛠 What does this PR implement
A Github PR template.
🙈 Missing
Templates are a very subjective topic. I welcome any feedback you might have cc @RobinM-code @DSuveges @ireneisdoomed
🚦 Checks

[x] covers one single feature (one change at a time)
[x] documentation added/updated
[x] there is no commented out code in this PR
[x] follows conventional commits in PR title and commits.
[x] is up-to-date with dev branch
[x] tests have been added to verify that the new code works
[x] passes local tests (make test)
[x] passes pre-commits (e.g poetry run pre-commit run --all-files)

Some nice work @d0choa, this will help for sure!
Also:

Currently our contribution guidelines are "polluted" with instructions about environment set up, we could improve this.

Agree with this :)

I think these are fine, if changes will be necessary later, it can always be updated.


allow drag&drop of folder with pxt.json to import it

This is likely needed for testing

Not sure about this after all - drag&drop of folders is a bit tricky. @pelikhan @tballmsft comments?

Feature cut.


Update to dataset-go v0.14.0

After the PR - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/26094 - with upgrade to dataset-go was merged, we have to as well update our own configuration files for the builder.

Codecov Report

Merging #13 (ef4af94) into main (14a2262) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   97.01%   97.01%
=======================================
  Files           5        5
  Lines         469      469
=======================================
  Hits          455      455
  Misses         11       11
  Partials        3        3

Just to clarify, for reference sake - this PR updates all the submodules, including otel contrib one, not just dataset-go.


Fix V22ManifestListTemplate cast when pulling an oci index manifest

Fixes #3958
Since there's no more downcast in PullBaseImageStep of manifest lists actual implementation, use the parent class in the unit tests.
Also update jib-core gradle example to compile with 0.23.0.
I was able to reproduce via a gradle build using jib-core (from: gcr.io/distroless/java17-debian11:nonroot@sha256:5154596f58d81687d66c8302face284198caf65033cde6bf6667c488b21745e6)  and verify the fix works. Though I'm not sure why the issue wasn't caught by the new integration test cases added in #3965 ?
Before filing a pull request, make sure to do the following:

[x] Create a new issue at https://github.com/GoogleContainerTools/jib/issues/new/choose.
[x] Ensure that your implementation plan is approved by the team.
[x] Verify that integration tests and unit tests are passing after the change.
[x] Address all checkstyle issues. Refer to the style guide.

This helps to reduce the chance of having a pull request rejected.

Regarding the integration test I've realized ManifestPullerIntegrationTest only tests the RegistryClient, not the actual Jib build steps.
I think it's missing a case in JibIntegrationTest that would pull an oci index.
Today it's targeting only docker.io busybox image which is a V22 Manifest list.
I'll have a look later in the week.

I've added a test case to JibIntegrationTest, which was failing before the fix.

Awesome, thank you @rquinio for the fix and improvement to test coverage!


Get Releases and download latest for the selected channel

Use Releases in GitHub API to get the releases.
https://docs.github.com/en/rest/releases/releases?apiVersion=2022-11-28
Need some way to identify between channels.

Consider two channels:

Pre-release- using GitHub pre-release functionality for Releases
Production- where not marked as a pre-release in GitHub

Consider a service that gets update status from GitHub according to configured channel.
If no configuration, no check is performed.

Investigate Xamarin Essentials:
https://stackoverflow.com/questions/70599331/where-to-save-net-maui-user-settings

https://api.github.com/repos/ProgramX-NPledger/Clock/releases
[
    {
        "url": "https://api.github.com/repos/ProgramX-NPledger/Clock/releases/127770291",
        "assets_url": "https://api.github.com/repos/ProgramX-NPledger/Clock/releases/127770291/assets",
        "upload_url": "https://uploads.github.com/repos/ProgramX-NPledger/Clock/releases/127770291/assets{?name,label}",
        "html_url": "https://github.com/ProgramX-NPledger/Clock/releases/tag/v0.0.1-alpha.1",
        "id": 127770291,
        "author": {
            "login": "ProgramX-NPledger",
            "id": 145849772,
            "node_id": "U_kgDOCLF9rA",
            "avatar_url": "https://avatars.githubusercontent.com/u/145849772?v=4",
            "gravatar_id": "",
            "url": "https://api.github.com/users/ProgramX-NPledger",
            "html_url": "https://github.com/ProgramX-NPledger",
            "followers_url": "https://api.github.com/users/ProgramX-NPledger/followers",
            "following_url": "https://api.github.com/users/ProgramX-NPledger/following{/other_user}",
            "gists_url": "https://api.github.com/users/ProgramX-NPledger/gists{/gist_id}",
            "starred_url": "https://api.github.com/users/ProgramX-NPledger/starred{/owner}{/repo}",
            "subscriptions_url": "https://api.github.com/users/ProgramX-NPledger/subscriptions",
            "organizations_url": "https://api.github.com/users/ProgramX-NPledger/orgs",
            "repos_url": "https://api.github.com/users/ProgramX-NPledger/repos",
            "events_url": "https://api.github.com/users/ProgramX-NPledger/events{/privacy}",
            "received_events_url": "https://api.github.com/users/ProgramX-NPledger/received_events",
            "type": "User",
            "site_admin": false
        },
        "node_id": "RE_kwDOKhKfaM4HnZ6z",
        "tag_name": "v0.0.1-alpha.1",
        "target_commitish": "release/alpha",
        "name": "Alpha",
        "draft": false,
        "prerelease": true,
        "created_at": "2023-11-02T22:30:50Z",
        "published_at": "2023-11-02T22:50:44Z",
        "assets": [
            {
                "url": "https://api.github.com/repos/ProgramX-NPledger/Clock/releases/assets/133673859",
                "id": 133673859,
                "node_id": "RA_kwDOKhKfaM4H97OD",
                "name": "Clock.Maui-windows-alpha-ci-build.zip",
                "label": null,
                "uploader": {
                    "login": "ProgramX-NPledger",
                    "id": 145849772,
                    "node_id": "U_kgDOCLF9rA",
                    "avatar_url": "https://avatars.githubusercontent.com/u/145849772?v=4",
                    "gravatar_id": "",
                    "url": "https://api.github.com/users/ProgramX-NPledger",
                    "html_url": "https://github.com/ProgramX-NPledger",
                    "followers_url": "https://api.github.com/users/ProgramX-NPledger/followers",
                    "following_url": "https://api.github.com/users/ProgramX-NPledger/following{/other_user}",
                    "gists_url": "https://api.github.com/users/ProgramX-NPledger/gists{/gist_id}",
                    "starred_url": "https://api.github.com/users/ProgramX-NPledger/starred{/owner}{/repo}",
                    "subscriptions_url": "https://api.github.com/users/ProgramX-NPledger/subscriptions",
                    "organizations_url": "https://api.github.com/users/ProgramX-NPledger/orgs",
                    "repos_url": "https://api.github.com/users/ProgramX-NPledger/repos",
                    "events_url": "https://api.github.com/users/ProgramX-NPledger/events{/privacy}",
                    "received_events_url": "https://api.github.com/users/ProgramX-NPledger/received_events",
                    "type": "User",
                    "site_admin": false
                },
                "content_type": "application/x-zip-compressed",
                "state": "uploaded",
                "size": 44941667,
                "download_count": 0,
                "created_at": "2023-11-02T22:48:02Z",
                "updated_at": "2023-11-02T22:50:38Z",
                "browser_download_url": "https://github.com/ProgramX-NPledger/Clock/releases/download/v0.0.1-alpha.1/Clock.Maui-windows-alpha-ci-build.zip"
            }
        ],
        "tarball_url": "https://api.github.com/repos/ProgramX-NPledger/Clock/tarball/v0.0.1-alpha.1",
        "zipball_url": "https://api.github.com/repos/ProgramX-NPledger/Clock/zipball/v0.0.1-alpha.1",
        "body": ""
    }
]

Seems a standard way of doing it, but would nee Github to package it. (And throw away existing code)
https://learn.microsoft.com/en-us/windows/msix/app-installer/create-appinstallerfile-vs


Intel GPU doesn't work

I switched my Ubuntu system to Intel graphics. Shadertoy seems to work, so do the glslViewer examples.
3D Curv programs work. Many 2D programs crash or freeze the GPU (requiring a reboot), including 'square 1' and 'circle 1'. But smoke.curv works.
Okay, this looks like an issue with glslViewer. Here's the shadertoy.com "hello world" program:
void mainImage( out vec4 fragColor, in vec2 fragCoord )
{
vec2 uv = fragCoord.xy / iResolution.xy;
fragColor = vec4(uv,0.5+0.5*sin(iGlobalTime),1.0);
}
This program runs fine on the shadertoy.com website, inside Firefox. But give the same program to glslViewer and we need to reboot. The problem, apparently, is that glslViewer runs shader programs at max frame rate, with no throttling. If the shader program is too simple and fast, then BOOM.

Reported as https://github.com/patriciogonzalezvivo/glslViewer/issues/73


ios sound classtification using task library

@st-tuanmai Please also add a run script to automatically download the TFLite model at build time.
See this example and see how to trigger the run script in Xcode's project settings.


test: client needs auth token

Following up from #11 TestDataRoundTrip requires the client to pass a auth token, which is only available from the docker bridge container keyring. Need a way to either create a static keyring or query the container for the token.

Latest testcontainers have a way to query the container using ForExec:
https://golang.testcontainers.org/features/wait/exec/#match-an-exit-code-and-a-response-matcher
But unforunately their latest version has a lot of dependencies and needs several replace statements in go.mod
Another alternative is to pass a static keystore as a volume to the bridge container, however it requires adding test keystore data to the repo. Thoughts? @tzdybal

We should use local-celestia-devnet and then we can use ory/dockertest (not much deps), and exec works fine. I'm working on full unit-test-flow right now.

Already resolved on main.


Documentation has bad URL

In manifest.json, the documention url is:
"documentation": "https://github.com/sbabcock23/hass-pytryfi"
it should be:
"documentation": "https://github.com/sbabcock23/hass-tryfi"

Thank you for identifying the issue. This has now been fixed and pushed to the master branch.


Note that "time" is a reserved word and should not be used as tag or field key

The keyword "time" will be dropped on any incoming points, which should be noted in the documentation.

@rossmcdonald - time seems to be treated differently from other key words: I can use it as a measurement name without a problem but when I write it as a field or tag key, its data are silently dropped (like you said).
# Measurement treatment: time vs. all

> insert time value=1
> select * from time
name: time
time                value
----                -----
1486403076637310944 1

> insert all value=1
> select * from all
ERR: error parsing query: found ALL, expected identifier at line 1, char 15

# Field key treatment: time vs. all

> INSERT whatever all=1
> INSERT whatever time=1
> SELECT * FROM whatever
name: whatever
time                all
----                ---
1486403430399284559 1

Is this more of a bug with the database?

@rkuchan The "time" drops should be logged server-side, so they are only silently dropped from the client's perspective. From my perspective the database is operating as intended, as a "time" field/tag key would be indistinguishable from the "time" timestamp when querying (and the database is purposefully stripping the "time" component out before writing the points to disk).
We can also open a feature request to have a message sent back to the client, if you think that would help.

No, that's fine. I was just curious. I'll get this added!


Rotate object dynamically.

Hi there, is there any way to Rotate Object dynamically?
Here are some step which I want to achieve

Initially my model loads with below position

but I want to rotate it like the below screenshot programmatically (left facing instead of right-facing).

I used, manipulator, to achieve the above functionality
like below
manipulator = Manipulator.Builder()
.targetPosition(0.0f, 0.0f, -4.0f)
.orbitHomePosition(0f, 0.0f, -1.25f)
.viewport(mySurfaceView.width, mySurfaceView.height)
.build(Manipulator.Mode.ORBIT)
modelViewer = ModelViewer(mySurfaceView, engine, manipulator)
but seems like it's not that much properly positioned.
also, I want to reset it to 2nd step position(left facing), Is that possible?
Any help will be highly appreciated. Thank you.

I tried using transform Manager as below.
modelViewer.transformToUnitCube(Float3(0f, 0.of, -4f))
but it makes no effect.

That is not transform manager. This is only to scale the object. Check out the samples to see how to use transform manager.

Okay, but when i try to set transform like below code. my model disappear from screen,
engine.transformManager.setTransform(engine.transformManager.getInstance(mesh.renderable),
floatArrayOf(
1.0f,  0.0f, 0.0f, 0.0f,
0.0f,  1.0f, 0.0f, 0.0f,
0.0f,  0.0f, 1.0f, 0.0f,
0.0f, -1.2f, 0.0f, 1.0f
))

Also, I do not want to move it, as the current transform does.

My model will be steady at one place. user can rotate 180 degree. but instead of right facing, it will front facing.

You create a translation transform, you'll need to create a rotation transform to achieve what you want. You can use the Mat4 class from the filament-utils library to help you create correct transform matrices.

Okay, I did checked samples but did not found any relevant code for this, it will be very helpful if you post sample code for rotation transform

https://github.com/google/filament/blob/main/android/samples/sample-hello-triangle/src/main/java/com/google/android/filament/hellotriangle/MainActivity.kt#L236

Okay, so far, I tried the following code. but still no luck with it.
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
       super.onViewCreated(view, savedInstanceState)
       setupViewModel()

       setupFilament()
       initFilament()
   }

private fun setupFilament() {
       choreographer = Choreographer.getInstance()
       engine = Engine.create()
   }

 /**
    * Initialize filament and Start loading Model
    */
   private fun initFilament() {

       //To change the height and width of object
       manipulator = Manipulator.Builder()
           .targetPosition(0f, 0.0f, -4.0f)
           .orbitHomePosition(0f, 0f, -1.25f)
           .viewport(mySurfaceView.width, mySurfaceView.height)
           .build(Manipulator.Mode.ORBIT)

       modelViewer = ModelViewer(mySurfaceView, engine)

       createRenderables()
       createIndirectLight()

       mySurfaceView.setOnTouchListener { _, event ->
           modelViewer.onTouchEvent(event)
           true
       }
   }

private fun createIndirectLight() {
       val scene = modelViewer.scene
       val ibl = "venetian_crossroads_2k"

       viewLifecycleOwner.lifecycleScope.launch {
           val byteBuffer = readCompressedAsset("envs/$ibl/${ibl}_ibl.ktx")
           scene.indirectLight = KtxLoader.createIndirectLight(engine, byteBuffer)
           scene.indirectLight!!.intensity = 30_000.0f
       }

       viewLifecycleOwner.lifecycleScope.launch {
           val byteBuffer = readCompressedAsset("envs/$ibl/${ibl}_skybox.ktx")
           scene.skybox = KtxLoader.createSkybox(engine, byteBuffer)
       }

       scene.skybox = Skybox.Builder().color(01f, 01f, 01f, 01f).build(engine)

       modelViewer.transformToUnitCube(Float3(0f, 0f, -4f))

        //Trying this code to rotate the object
        val transformMatrix = FloatArray(16)
        android.opengl.Matrix.setRotateM(transformMatrix, 0, 0.55f, 0.0f, 0.0f, 1.0f)
        val tcm = engine.transformManager
        tcm.setTransform(tcm.getInstance(modelViewer.asset?.root!!), transformMatrix)
     }

    private fun createRenderables() {
    viewLifecycleOwner.lifecycleScope.launch {

           val  assetsName = "models/CUSTOMFX_FX870_ANDROID_ASSET.gltf"

           val byteBuffer = readCompressedAsset(assetsName)

           modelViewer.loadModelGltfAsync(byteBuffer) { uri -> readCompressedAsset("models/$uri") }
     }

private fun readCompressedAsset(assetName: String): ByteBuffer {
       val input = activity?.assets?.open(assetName)
       val bytes = ByteArray(input?.available()!!)
       input.read(bytes)
       return ByteBuffer.wrap(bytes)
   }

inner class FrameCallback : Choreographer.FrameCallback {
       private val startTime = System.nanoTime()
       override fun doFrame(frameTimeNanos: Long) {
           choreographer.postFrameCallback(this)
           modelViewer.animator?.apply {
               if (animationCount > 0) {
                   val elapsedTimeSeconds =
                       (frameTimeNanos - startTime).toDouble() / 1_000_000_00000000000
                   applyAnimation(0, elapsedTimeSeconds.toFloat())
               }
               updateBoneMatrices()
           }

           modelViewer.render(frameTimeNanos)
       }
   }
}

You are rotating by 0.5 degree. Please check the documentation:
https://developer.android.com/reference/android/opengl/Matrix#setRotateM(float[], int, float, float, float, float)

okay but if i use any of above values. my model starts rotating inside whole screen, like user drag model by its touch.
but I want it on specific position on screen. and just rotate

It is possible to make a model positioned at a specific position? and just rotate from right to left ?

This is what i achieved after
android.opengl.Matrix.setRotateM(transformMatrix, 0, 0.55f, 0.0f, 0.0f, 1.0f)

By using setRotateM you are overriding the translation you specified just before with modelViewer.transformToUnitCube(Float3(0f, 0f, -4f)). You can use the other Matrix APIs to do both a rotation and translation. And again your rotation is only 0.5 degree, I don't expect you'll see much on screen.

So how can i get only rotation, not translation

This is how it behaves currently

You can get both, see the Matrix APIs, you can combine transforms in a single matrix.

I'm doing it like this and it worked
` modelViewer.transformToUnitCube()
    val localMatrix = FloatArray(16)
    modelViewer.engine.transformManager.getTransform(modelViewer.asset?.root!!, localMatrix)

    val rotateMatrix = FloatArray(16)
    Matrix.setRotateM(rotateMatrix, 0, 0F, 0.0f, 0.0f, 1.0f)

    val resultMatrix = FloatArray(16)
    Matrix.multiplyMM(resultMatrix, 0, rotateMatrix, 0, localMatrix, 0)

    val tcm = modelViewer.engine.transformManager
    tcm.setTransform(tcm.getInstance(modelViewer.asset?.root!!), rotateMatrix)

`

Hi @romainguy I am also working on android filament. I have a glb model. I need to rotate the model without moving the position. The center axis of the model should be filed. I need to change the rotation alone of the entire model programatically. What I need to do? Please suggest me some suggestions on this. There is a lack of examples for filament rotation. So Its hard to understand the rotation.


Overlapping Marker Spiderfier in Xamarin.Forms.GoogleMaps

FEATURE REQUEST / SUGGESTION
I was in need of a feature that Google Earth has when there are multiple markers near the same location you are able to click on that marker and it expands.
SUMMARY
Basically, this feature would make us able to click on a master pin that would expand if it had children inside him.
IDEA TAKEN FROM
https://github.com/jawj/OverlappingMarkerSpiderfier
https://github.com/leaflet/Leaflet.markercluster/
PLATFORMS
which platform do you want?

[x] Android
[x] iOS
[ ] UWP

+1

We called it "Marker Clustering".
please vote to #123


Loading as ES module crashes Firefox 54

I use Firefox 54 (64 bit) with ES modules enabled and successfully use them throughout my own code but when I use import euler from '../lib/cytoscape-euler.js'; (Firefox doesn't seem to accept module names without path and extension), the browser crashes without console statements.

ES modules can only work if you build a bundle using browserify or webpack.  It's not possible to publish ES module libs that are backwards compatible with non-ES-module browsers.

But Firefox 54 supports ES modules, at least when the experimental flag is enabled. Shouldn't it work in that case?

No, it's not possible to support ES6 export/import directly and also support CommonJS.
You can't try-catch for syntax errors like missing export.  Try running this on node@6:
try {
  export function foo(){
    console.log('export');
  }
} catch(err){
  module.exports = function foo(){
    console.log('module.exports');
  }
}

/private/tmp/exp.js:2
  export function foo(){
  ^^^^^^

SyntaxError: Unexpected token export
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:389:7)
    at startup (bootstrap_node.js:149:9)


[Feature]: Video Player play/pause during incoming calling

Description
We need functionality where the video player pauses when a call is incoming and resumes when the call ends. This should not happen automatically but should be controlled via a prop value so that our custom functionality can handle the play/pause actions.
There is an old npm package named react-native-call-detection which is outdated (last updated 4 years ago) and incompatible with the newer Android/iOS versions. This package detects different call states like Incoming, Disconnected, Dialing, and Connected for iOS. For Android, it provides states like Offhook, Incoming, Disconnected, and Missed.
Our goal is to handle at least the Incoming and Disconnected events to implement custom controls for playing and pausing the video.
Why it is needed ?
Every ott platform like Amazon Prime, Netflix, Disney plus HotStar has this functionality where any video is playing and call is coming in between then automatically the player pause and when its disconnected then it automatically starts playing. Atleast need those two event listener (Incoming and Disconnected) and then video player would be completed. Its a much needed functionality for any video player. If possible please release it as early as possible as we have made an ott platform completed with this package and react-native and only this functionality and android pip support is missing.
Possible implementation
Refer this package for idea " react-native-call-detection".
Code sample
Code sample

@Sumeeth-24 Bro I have tried this approach after lots of tries using the library you provided and finally this approach works for me,
onAudioFocusChanged={event => {
if (!event.hasAudioFocus) {
setPaused(true);
setActiveCall(true);
} else {
setPaused(false);
setActiveCall(false);
}
}}

The player works already as you describe in Android @Sumeeth-24 . See the example/basic. Run it by:
yarn install
cd examples/basic
yarn install
yarn run android
yarn run start a

For iOS, there is started work on refactoring so that iOS audio category can be set more easily so that interruptions are handled well. https://github.com/TheWidlarzGroup/react-native-video/pull/3850 For now, see https://github.com/TheWidlarzGroup/react-native-video/issues/3652 for a patch I made.

@Sumeeth-24
On Android, you can simply follow their examples/basic's implementation:
  onAudioBecomingNoisy = () => {
    this.setState({paused: true});
  };

  onAudioFocusChanged = (event: OnAudioFocusChangedData) => {
    this.setState({paused: !event.hasAudioFocus});
  };
  ...
  <Video
    onAudioBecomingNoisy={this.onAudioBecomingNoisy}
    onAudioFocusChanged={this.onAudioFocusChanged}
    .../>

https://github.com/TheWidlarzGroup/react-native-video/blob/master/examples/basic/src/VideoPlayer.tsx#L387

This would be a great addition.

One way that this can be implemented is by breaking other functionality fyi @Sumeeth-24. You can let exoplayer manage its own audioFocus:
Edit node_modules/react-native-video/android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerView.java
@@ -1256,10 +1273,10 @@ public class ReactExoplayerView extends FrameLayout implements
         }

         if (playWhenReady) {
-            this.hasAudioFocus = requestAudioFocus();
-            if (this.hasAudioFocus) {
-                player.setPlayWhenReady(true);
-            }
+            // this.hasAudioFocus = requestAudioFocus();
+            // if (this.hasAudioFocus) {
+            player.setPlayWhenReady(true);
+            // }
         } else {
             // ensure playback is not ENDED, else it will trigger another ended event
             if (player.getPlaybackState() != Player.STATE_ENDED) {
@@ -2134,7 +2151,7 @@ public class ReactExoplayerView extends FrameLayout implements
             AudioAttributes audioAttributes = new AudioAttributes.Builder().setUsage(usage)
                     .setContentType(contentType)
                     .build();
-            player.setAudioAttributes(audioAttributes, false);
+            player.setAudioAttributes(audioAttributes, true);
             AudioManager audioManager = (AudioManager) themedReactContext.getSystemService(Context.AUDIO_SERVICE);
             boolean isSpeakerOutput = output == AudioOutput.SPEAKER;
             audioManager.setMode(


custom fields: how to connect with a vocabulary

[x] Showcase how we can connect the CF with a small vocabulary i.e small generic vocabulary (e.g resource type)
[x] Ensure that there is a recipe on how to add a new vocabulary e.g start with a small generic. Will be done in https://github.com/inveniosoftware/docs-invenio-rdm/issues/384

PR has not been merged to master but it is available in custom-fields. Works will take over form that branch, closing as achived.


Missing full-store in cache results in "opening file: not found" error

This happens when the backend cache is missing a full-kv file in its sequence (ex: 10, 20, 40).
It is not a scenario that should happen during normal operations, but if someone is moving kv files around, this case is not handled well.
Here's an example log entry:
An error occurred while streaming blocks: status: Internal, message: "error during init_stores_and_backprocess: failed run_parallel_process: parallel processing run: scheduler run: process job result for module \"unknown\": job ended in error: receiving stream resp: rpc error: code = Internal desc = error building pipeline: failed to setup subrequest stores: load full store: load full store erc721:store_address at<PHONE_NUMBER>-0000462663.kv: opening file: not found", details: [], metadata: MetadataMap { headers: {} }, provider: mainnet-substreams-sf, deployment:

A workaround is to delete the cache, but it is inefficient.
Wanted behavior: the substreams engine should schedule the missing jobs and inform the "squasher" that it needs to process these missing file segments.
How to reproduce locally

In one terminal run (in firehose-ethereum devel folder):

DEBUG=.* ./runtier1.sh

In another terminal run:

DEBUG=.* ./runtier2.sh

Then in another terminal run any substreams you want.

substreams run map_eth_stats --plaintext -e localhost:9000 -s 1000 -t +1

This will create 10 files on disk for store_eth_stats

Manually delete any full kv
Run again your substreams but with a different boundary (production mode or not)

substreams run map_eth_stats --plaintext -e localhost:9000 -s 400 -t +1 --production-mode

This will make the run wait and not complete as it is missing the full kv file that you delete

This branch contains work in progress, needs to be rebased and tested in real-world scenario:
https://github.com/streamingfast/substreams/tree/feature/missing-ranges

Latest changes to feature/missing-ranges makes it that if you delete, let's say, the file 0-200.kv and then run (production mode or not)
substreams run map_eth_stats --plaintext -e localhost:9000 -s 200 -t +1 --production-mode

This will work. BUT if you run it with a start block not at the exact same height of the kv file deleted then it will create this kind of output locally:
0000000100-0000000000.54e06f8b1e0a145cf525ccf966e15bda.partial.zst
0000000100-0000000000.8a8a1fbabfe985ce9adad621dfa15da0.partial.zst
0000000100-0000000000.kv.zst
0000000200-0000000000.kv.zst
0000000200-0000000100.54e06f8b1e0a145cf525ccf966e15bda.partial.zst
0000000200-0000000100.8a8a1fbabfe985ce9adad621dfa15da0.partial.zst
0000000300-0000000000.kv.zst
0000000400-0000000000.kv.zst
0000000500-0000000000.kv.zst
0000000600-0000000000.kv.zst
0000000700-0000000000.kv.zst
0000000800-0000000000.kv.zst
0000000900-0000000000.kv.zst
0000001000-0000000000.kv.zst

To fix we would need to relaunch a squasher for that specific range, basically have a concept of store squashers by module AND by range...

Will be addressed with rework: https://github.com/streamingfast/substreams/issues/226

This issue has been fixed  with #226 and deployed to our production endpoint.


rocm_bandwidth_test deb package should not use underscores.

It is somehow a bad practice in Debian packaging world to use underscores in package names and package file names.
Consider rocm-bandwidthtest

Yes, underscores in the command name are surprising. This makes ROCm's bandwidth test feel out of place on a Linux system. The underscores also break auto completion if users assume a hyphen and hit tab on rocm-. Since "bandwidthtest" is not a proper english word, I would suggest a second hyphen: rocm-bandwidth-test

@rerrabolu What is the solution to this issue? Have you changed the name or have you considered renaming, but decided against it?

I sincerely apologize for not updating the ticket. The name was changed to suggested name: rocm-bandwidth-test. The changed was made nearly year plus ago. Including here the updated names of exe, and its DEB and RPM packages
*- rwxrwxr-x 1 user1 user1 1098600 Jun  8 17:48 rocm-bandwidth-test

rw-rw-r-- 1 user1 user1  373952 Jun  8 17:48 rocm-bandwidth-test_<IP_ADDRESS>999-local_amd64.deb
rw-rw-r-- 1 user1 user1  378931 Jun  8 17:48 rocm-bandwidth-test-<IP_ADDRESS>999-local.x86_64.rpm**

Let me know if this resolves the issue or not.

@rerrabolu Never mind only updating this ticket now. Awesome that the name was changed to for the better in the past. Thanks for coming back with an explanation. I can confirm that the package and the executable name are now nicer:
$ sudo apt install rocm-bandwidth-test
[...]
$ /opt/rocm/bin/rocm-bandwidth-test
................
          RocmBandwidthTest Version: 2.5.1

          Launch Command is: /opt/rocm/bin/rocm-bandwidth-test (rocm_bandwidth -a + rocm_bandwidth -A)

          Device: 0,  AMD Ryzen Threadripper 1950X 16-Core Processor
          Device: 1,  Vega 10 XL/XT [Radeon RX Vega 56/64],  GPU-02151de3936c4944,  0b:0.0
          Device: 2,  Vega 10 XL/XT [Radeon RX Vega 56/64],  GPU-0213f260e1523984,  45:0.0

          Inter-Device Access

          D/D       0         1         2

          0         1         0         0

          1         1         1         0

          2         1         0         1

          Inter-Device Numa Distance

          D/D       0         1         2

          0         0         N/A       N/A

          1         20        0         N/A

          2         20        N/A       0

          Unidirectional copy peak bandwidth GB/s

          D/D       0           1           2

          0         N/A         9.590       9.590

          1         9.027       72.591      N/A

          2         9.026       N/A         69.220

          Bidirectional copy peak bandwidth GB/s

          D/D       0           1           2

          0         N/A         14.493      14.448

          1         14.493      N/A         N/A

          2         14.448      N/A         N/A


refactor TS on goals page

Looks good to me, just added a few comments for a few functions

We went over this so its all good. I'll close.


Enable range based looping of sf::VertexArray

Description
begin and end iterators are all that is required of a container to supported C++11 ranged-for loops. This PR also uses this feature as much as possible internally. Let me know if I missed any more places we can be using this.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

25 of 27 (92.59%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.001%) to 57.014%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/SFML/Graphics/Text.cpp
2
4
50.0%

Files with Coverage Reduction
New Missed Lines
%

src/SFML/Graphics/VertexArray.cpp
2
96.51%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.001%

Covered Lines:
12064

Relevant Lines:
20000

💛  - Coveralls

VertexArray::operator[] is not covered by tests now.

This operator is still used indirectly by some other tests.

Which one? Coveralls report does not see it.

This operator is still used indirectly by some other tests.

Which one? Coveralls report does not see it.

https://github.com/SFML/SFML/blob/3c084bf66186378dc52a53175ad396b344cb6a73/test/Graphics/VertexArray.test.cpp#L78
It’s used in a few places.

Ok, this is because the non-const variant is used by tests but the const variant is not.

Ok, this is because the non-const variant is used by tests but the const variant is not.

Ah okay this makes sense. I added tests explicitly for this case to make sure it remains covered. Thanks for catching this!


feat(root): add 'copied to clipboard' tooltip for 'copy code' button

Summary of the changes
Add 'copied to clipboard' tooltip for 'copy code' button on click using disabled hover.
To test go to any component's code page and click the copy code button, you should see a tooltip saying 'Copied to clipboard'.
Related issue
#264
Checklist

[x] I have manually accessibility tested any changes, if relevant.
[x] I have raised this pull request against the develop branch

I don't think tooltip is the best component to use in this case
According to the tooltip guidance, "tooltips provide helpful information when the cursor hovers over an element or when focus is applied."
We are currently changing the behaviour of the tooltip for this enhancement. I think that an auto-dismissible toast is best suited for this use case

The tooltip now has disable hover functionality for this use case, so maybe the guidance needs updating to reflect this?

In both circumstances, using a tooltip or an auto-dismiss toast, the same problem arises; the permanence of the message and whether it is important enough to warrant something more.
Typically, auto-dismiss toasts should only be used for information that is clear another way, as otherwise this information could easily be missed before the toast dismisses itself. To combat this, you can make a toast permanent on screen and require a dismiss action to clear it. However, we also must be cognizant of the additional interaction costs being presented in this scenario for information that is not essential to continued operation.
The use of a tooltip operates in much the same way, showing a temporary message that dismisses once focus is removed from the button. Tooltips are typically shown on hover, but can also be tied to a triggering element such as a button press. This pattern is fairly common and probably achieves the same goal, whilst associating the message closer to the component that has just been interacted with.
We also do not have toasts available to us at this time, so I would suggest keeping the tooltip approach for now, and testing this to understand if this approach is problematic or inconsistent. A guidance update, or replacement toast implementation can be carried out at a later date.

It is possible that a different type of tooltip - a pop-up tip could be created to satisfy this. Pop-up tips would differ from tooltip in that they are launched from a click trigger, and dismissed either by a dismiss button, or when focus is removed from the triggering element.

@DBD324 So is this approach okay for now and later down the line either update the guidance or replace with toast/pop-up tip?


Failed when trying add platform iOS

I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Just clone the source minutes ago, then I ran 'npm i' and 'ionic cordova platform add ios', at the end I got
--save flag or autosave detected
Saving<EMAIL_ADDRESS>into config.xml file ...
✖ Copying default image resources into ./resources/ios - failed!
Error<EMAIL_ADDRESS>not found

Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.0.0
    Cordova CLI      : 6.5.0
    Ionic CLI        : 3.0.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-cordova       : 1.1.1
    @ionic/cli-plugin-ionic-angular : 1.0.0
    Ionic Framework                 : ionic-angular 3.2.1

System:

    Node       : v6.10.0
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.2 Build version 8E2002
    ios-deploy : not installed
    ios-sim    : 5.0.13

@qiluo please try:
ionic platform add ios

@xupea, not worked
→ ionic platform add ios
The platform command has been renamed. To find out more, run:

  ionic cordova platform --help

I'm not able to reproduce this using the latest of the conference app:
--save flag or autosave detected
Saving<EMAIL_ADDRESS>into config.xml file ...
✔ Copying default image resources into ./resources/ios - done!

Please try cloning a fresh copy of the conference app and doing a fresh npm install. You may also want to update your ionic CLI to the latest release: 3.4.0.
If you are still having problems with this please create an issue on the Ionic CLI repository since it isn't specific to the conference app: https://github.com/ionic-team/ionic-cli/issues
Thank you!


All my files are gone after cut/paste-ing multiple folders

Okay, I tried to cut/paste 5 folders full of my valuable scripts and programs, and what do you know, only the folders were moved, with no files inside of them. Seriously, I just lost 500+ programs of mine and ended up with empty folders. I thought the process wasn't finished, so I left it for half an hour, came back, empty folders. I started panicking, I didn't know what to do, I quickly tried CTRL + Z, nothing. I hoped it was a visual bug, went inside the regular File Explorer, empty folders! Panic, panic, panic! I started restoring all my 6000+ files from the recycle bin, hoping the scripts were there, and what do you know, they weren't. Luckily, I got myself out of the trap by installing Recuva (not an ad lol). So yeah, I just want to warn the peeps here to never, ever move important files with this program, you might end up in this situation.

@MatejaSrejic First of all, our apologies about this issue, we are working on improving the quality of copy/paste and as we get close to releasing v1, we are going to make sure this feature works as expected. Can you share with us the version number you had this issue on?

@MatejaSrejic First of all, our apologies about this issue, we are working on improving the quality of copy/paste and as we get close to releasing v1, we are going to make sure this feature works as expected. Can you share with us the version number you had this issue on?

Don't know if it helps, but I installed the latest version three days ago. If that doesn't help, I'll check which version I used tomorrow (I don't have my computer on me)

Don't know if it helps, but I installed the latest version three days ago. If that doesn't help, I'll check which version I used tomorrow (I don't have my computer on me)

@yaichenbaum is the newest version after #2742 patch?

@yaichenbaum is the newest version after #2742 patch?

#2742 doesn't help I'm afraid
The move operation is implemented as a Copy + (permanent!) Delete, Delete should only be done if the copy operation returns success.... except after investigation it appears that Copy always returns success regardless of the actual result.
@d2dyno1 could you check as well?
Thanks @MatejaSrejic for bringing this out, we'll be sure to fix it.

#2742 doesn't help I'm afraid
The move operation is implemented as a Copy + (permanent!) Delete, Delete should only be done if the copy operation returns success.... except after investigation it appears that Copy always returns success regardless of the actual result.
@d2dyno1 could you check as well?
Thanks @MatejaSrejic for bringing this out, we'll be sure to fix it.

Thank you. I made a huge mistake but I luckily got out of it.

Thank you. I made a huge mistake but I luckily got out of it.

@gave92 indeed. We should take the result of errorCode variable.

@gave92 indeed. We should take the result of errorCode variable.

@d2dyno1 I'd be careful with that. What we do with errorCode in a lot of places is incorrect:
var returnStatus = NO_ERROR;
errorCode.ProgressChanged += (s, e) => returnStatus = e.ToStatus();`
...
errorCode?.Report(SOME_ERROR);
----> do not expect that here returnStatus == errorCode :(

@d2dyno1 I'd be careful with that. What we do with errorCode in a lot of places is incorrect:
var returnStatus = NO_ERROR;
errorCode.ProgressChanged += (s, e) => returnStatus = e.ToStatus();`
...
errorCode?.Report(SOME_ERROR);
----> do not expect that here returnStatus == errorCode :(

Hmm..
I suggest disabling the Move operation in a patch update for now to prevent further damage.
@yaichenbaum

Hmm..
I suggest disabling the Move operation in a patch update for now to prevent further damage.
@yaichenbaum

Or at least sending the stuff to recycle bin instead of permanent delete. Anyway it should not be too hard to fix.

Or at least sending the stuff to recycle bin instead of permanent delete. Anyway it should not be too hard to fix.

Hmm..
I suggest disabling the Move operation in a patch update for now to prevent further damage until the issue is resolved.
@yaichenbaum

This can be done but we are so close to releasing v1 that it might be just as fast to fix the actual bug and ship out v1 then to ship a change that disables the feature.

Hmm..
I suggest disabling the Move operation in a patch update for now to prevent further damage until the issue is resolved.
@yaichenbaum

This can be done but we are so close to releasing v1 that it might be just as fast to fix the actual bug and ship out v1 then to ship a change that disables the feature.

@MatejaSrejic This issue should be fixed with #2830, we are going to do some extensive testing first but you can expect this update to be available in around a week or two.

@MatejaSrejic This issue should be fixed with #2830, we are going to do some extensive testing first but you can expect this update to be available in around a week or two.

Cool, thanks.

Cool, thanks.

Oh no, looks like Recuva didn't recover some of the files properly. When opened in Notepad++ it only has NUL over and over. F*** me. I'm screwed... can anyone recommend what to do. 😭😭

Oh no, looks like Recuva didn't recover some of the files properly. When opened in Notepad++ it only has NUL over and over. F*** me. I'm screwed... can anyone recommend what to do. 😭😭


Publisher apps reporting "Timeout was thrown when waiting for channel to attach"

We have a customer who is reporting seeing this exception quite a lot via their Crashlytics reports:
Non-fatal Exception: s3.c: Timeout was thrown when waiting for channel to attach
       at com.ably.tracking.common.DefaultAbly.waitForChannelReconnection(Ably.kt:890)
       at com.ably.tracking.common.DefaultAbly.access$waitForChannelReconnection(Ably.kt:247)
       at com.ably.tracking.common.DefaultAbly$waitForChannelReconnection$1.invokeSuspend(Ably.kt)
       at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
       at kotlinx.coroutines.internal.ScopeCoroutine.afterResume(Scopes.kt:33)
       at kotlinx.coroutines.AbstractCoroutine.resumeWith(AbstractCoroutine.kt:102)
       at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:46)
       at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:104)
       at kotlinx.coroutines.internal.LimitedDispatcher.run(LimitedDispatcher.kt:42)
       at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:95)
       at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:570)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677)
       at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)

This is because of a 10 second timeout baked into the AAT implementation. The value of this timeout is likely arbitrary (confirmed by @KacperKluka) and could be a sign that we're fighting or otherwise overriding behaviours of the underlying, core SDK. 🤔
From this Slack thread (internal) where we've initially discussed this report...
@paddybyers said:

Attaching to a channel can fail, either for reasons that aren't going to change, or for reasons that are transient. Adding a trackable, and getting it to the state where it is working (ie also attached) is therefore also subject to failure, where a retry might succeed.
So there are two questions:

what developer experience do we want to expose here? Should we always pass failures (including failures for transient reasons) back to the caller and abort the attempted operation? Or should AAT keep retrying all the time that there's reason to believe that a retry might succeed? What's the currently implemented API, and do we think this is the correct API?
What should AAT do in respect of interacting with the underlying Ably client? If it simply times out, then it has no chance of passing a meaningful error back to the caller, or taking meaningful action itself. It should at least be waiting for the underlying library to complete the operation (successfully or otherwise) because then it can know whether or not the operation is retriable, and it can pass meaningful information back to the caller.
So IMO the question isn't whether 10s is long enough or not. Simply timing out, and not waiting to find out from the underlying library what happened, is just wrong IMO

@davyskiba added some further analysis:

I think this is not causing a crash, the provided logs start with “Non-fatal Exception”.
In the code, it seems we’re never just throwing that exception outside - it is caught and returned via the callback or wrapped in a result.
I don’t mean that this is not an issue, just wanted to point out that it looks like it is not an app crash.
The reconnect method is called every time a publisher attempts to send a location. Even twice if the connection keeps on failing.
The occurrence numbers might be quite inflated 🤔

I started looking into this whilst on some downtime from another issue, but I've moved back to that issue now so I'm going to unassign this one from me so somebody else can pick it up.
My plan was to document the following:

the current error handling, recovery, and timeout behaviour in AAT Android
the error handling, recovery, and timeout behaviour of the Ably client libraries

My intention was that we would then define an error handling / recovery behaviour for AAT that took advantage of everything provided to us by the Ably client libraries, and that we only override or preempt the client library behaviour with good cause.
So far, all I've done is to start documenting the first bullet point above (AAT behaviour). I'm posting here what I've written so far in case it's useful to somebody else.
Asset Tracking Publisher SDK (Android)
As of Android repo 63c5295.
Ably.kt
It seems like all exceptions thrown by this file are wrapped in a ConnectionException - I don’t think it lets anything leak out.

waitForChannelReconnection(channel, timeout)

this one just waits for channel to get into attached state, with specified timeout
if timeout elapses, then it throws ConnectionException

which is called by retryChannelOperationIfConnectionResumeFails

if channel is suspended it first waits for it to become attached (using the above)
then it performs the operation on the channel
and if that operation throws a message == "Connection resume failed" && code == 50000 && statusCode == 500 exception then it retries the above two steps
if the retry fails then it rethrows

which is called by disconnectChannel / private sendMessage / public updatePresenceData

Auth token issue retrying:

private enterChannelPresence

tries to enter presence on the channel
if it catches a ConnectionException with errorInformation.code == AUTH_TOKEN_CAPABILITY_ERROR_CODE (40160), then it calls its own renewAuthSuspending – which in turn calls ably.auth.renewAuth – and then attaches to the channel and tries to enter presence on the channel
if the retry fails then it rethrows

I remember that the idea of this one was for a use case where a capability for a given channel is only granted to a user once they are assigned trackable that corresponds to that channel, but that this capability might not yet be reflected in their latest auth token.
Closing channel:

private tryDisconnectChannel

calls disconnectChannel and ignores any thrown exceptions

Connecting:

private connectSuspending(timeout)

waits for connection to become CONNECTED, calling connect() if necessary
if starts in or enters FAILED state, it throws a ConnectionException
if it doesn’t enter into CONNECTED state within the timeout then it throws a ConnectionException

The rest of the SDK
What happens to the exceptions that come out of Ably.kt? Are there other retries going on?
Let's look for "catch", "timeout", "retry" in the rest of the publisher codebase.
catch:

ConnectionCreatedWorker catches a ConnectionException

timeout:

Publisher#stop(timeoutInMilliseconds:)

This gets passed to StopWorker.

retry:

TODO there seem to be a fair few of these in the codebase, not gone through them yet

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-3194

This issue might be a result of https://github.com/ably/ably-java/issues/885 where channel fails to reattach because of attach/detach race.
I'm wondering if we have any timings of these events happening and see if times matches logs described https://github.com/ably/ably-asset-tracking-android/issues/863
I think it would be good to filter that possibility

If we remove the channel attach timeout, we need to consider #912.

I'm going to look into removing this AAT arbitrary timeout.

Is this getting resolved by #965?


[Feature] Decimal to binary calculator

Description
to add a calculator for converting decimal/integer to binary format for learning for coding purpose.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I follow Contributing Guidelines of this project.
[X] I am willing to work on this issue (optional)
[X] I am a SWOC'24 contributor

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Issue and PR already exist for this. Pls work on other good issues @Komal73

okay @Rakesh9100


Unable to understand the features

I have recently started working on this project. I am unable to understand what exactly are the features (i.e. f1, f2, f3 and so on) signify? Can you name some?

Divya, the following papers, co-authored by Dr. Iman Khosravi (University of Tehran), provide the details you are looking for:
http://i-khosravi.ir/_sources/tres2019.pdf - Section 3.1
http://i-khosravi.ir/_sources/tres2018b.pdf - Section 2.2
Please let me know if this addresses you question, so that I can close this open issue. Thank you!

Hello,
Thanks a lot for your assistance. The links provided by you above are not
accessible. It's showing me "site cannot be reached" when I am clicking on
those. Is there any other way in which I can access those pdfs or if you
can provide me with the downloaded copies it would be much appreciated.
Thanks a lot.
On Tue, 21 Jul 2020 at 20:49, Paulo Breviglieri<EMAIL_ADDRESS>wrote:

Divya, the following papers, co-authored by Dr. Iman Khosravi (University
of Tehran), provide the details you are looking for:
http://i-khosravi.ir/_sources/tres2019.pdf - Section 3.1
http://i-khosravi.ir/_sources/tres2018b.pdf - Section 2.2
Please let me know if this addresses you question, so that I can close
this open issue. Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1#issuecomment-661924959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBVACQUF4HWMY6U4BZLQYDR4WWWJANCNFSM4PCM4S2A
.

Links are working just fine for me. I uploaded copies of both papers to the support folder into this repository.

Thank you so much for your help. You can close the issue.
I had a question regarding data augmentation. As crop 2 and 7 have
unbalanced distribution of data as compared to the rest of the data, why
didn't you apply data augmentation there to increase the quantity of data
for those crops?
Thanks and Regards,
Divya
On Tue, Jul 21, 2020, 11:21 PM Paulo Breviglieri<EMAIL_ADDRESS>wrote:

Links are working just fine for me. I uploaded copies of both papers to
the support
https://github.com/pcbreviglieri/data-science-cropland-mapping/tree/master/support
folder into this repository.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1#issuecomment-662012489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBVACWVUH5NB26ITS77NBTR4XITPANCNFSM4PCM4S2A
.

Hi Paulo,
Yes, I believe balance distribution of data is required to get a well
performing model.
Data augmentation for an image usually involves transformation, rotation,
scaling, etc of the input image. ImageDataGenerator from keras can be
used to multiply the data for crop 2 and 7. But here the doubt that I have
is, can we really plot these image features? Let me know if you are aware
of any technique which can help me plot the image features to form a
cropland image.
The other alternative approach can be SMOTE library which helps in
reducing data imbalance by creating data by taking a dataset as input.
One more area which I am looking forward to exploring is, can we predict
the yield of the crop with the given data? Let me know your thoughts.
Thanks and Regards,
Divya
On Wed, 22 Jul 2020 at 19:15, Paulo Breviglieri<EMAIL_ADDRESS>wrote:

Thanks, Divya. I am very much interested in understanding how data
augmentation could be implemented in this particular case to address the
dataset imbalance. Maybe we could push performance metrics up to the 99%
range also for these two less represented classes. What would you suggest?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1#issuecomment-662462008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBVACQIACOTD7MRMVXF42DR43UPPANCNFSM4PCM4S2A
.

Fully agree with your proposal, but the dataset in this case is not a set of images. Instead, the dataset is tabular - a .txt table with numerical features representing parameters resulting from previous preprocessing of images (proprietary optical and radar-based information), executed by the dataset authors. As we are not dealing with images directly, the techniques you highlight are not applicable.
Predicting crop yields would, in my opinion, require data collected in a larger set of dates (e.g. once a week, over several weeks), so that the crop growing status could be tracked. We could tentatively investigate not only the size of crops, leaves etc. but also the changes in leaf colors, for example. If such data were available my first answer would be yes, it could be possible. However, the dataset furnished by the University of Tehran team includes data collected on two specific days only (July 5 and 14, 2012), very close to each other. With the current dataset I can't see how to predict yields. But you can contact the dataset authors directly, they may have a different insight.
Hope to have answered your questions, please confirm.

Thanks a lot for your help. We're currently exploring how different 2&7 are
from others using KNN to determine if we need to resample.
On Thu, 23 Jul 2020 at 17:53, Paulo Breviglieri<EMAIL_ADDRESS>wrote:

Fully agree with your proposal, but the dataset in this case is not a
set of images. Instead, the dataset is tabular - a .txt table with
numerical features representing parameters resulting from previous
preprocessing of images (proprietary optical and radar-based information),
executed by the dataset authors. As we are not dealing with images
directly, the techniques you highlight are not applicable.
Predicting crop yields would, in my opinion, require data collected in a
larger set of dates (e.g. once a week, over several weeks), so that the
crop growing status could be tracked. We could tentatively investigate not
only the size of crops, leaves etc. but also the changes in leaf colors,
for example. If such data were available my first answer would be yes, it
could be possible. However, the dataset furnished by the University of
Tehran team includes data collected on two specific days only (July 5 and
14, 2012), very close to each other. With the current dataset I can't see
how to predict yields. But you can contact the dataset authors directly,
they may have a different insight.
Hope to have answered your questions, please confirm.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1#issuecomment-662976051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBVACVH2YERQNWCJE34N5TR5ATS5ANCNFSM4PCM4S2A
.

Closing this issue. Feel free to open a new one should you have further questions. Thanks.

Closing this issue. Feel free to open a new one should you have further questions. Thanks.

yes sure thanks.
On Sat, 25 Jul 2020 at 19:13, Paulo Breviglieri<EMAIL_ADDRESS>wrote:

Closed #1
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pcbreviglieri/data-science-cropland-mapping/issues/1#event-3586282199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBVACXOHYTQKQSLLYEQJ7DR5LOQ5ANCNFSM4PCM4S2A
.


Fixes #879. UI of 'Add to Calendar' Button

Previous :

Now :

I'd suggest to add it into the toolbar instead.
The separate button for this looks out of place and context.
On Thursday, December 22, 2016, Piyush Ajmani<EMAIL_ADDRESS>wrote:

Previous :
[image: screen shot 2016-12-22 at 9 54 12 pm]
https://cloud.githubusercontent.com/assets/20954567/21434222/46f0d5da-c899-11e6-96fb-6ad167bf8296.png
Now :
[image: screen shot 2016-12-22 at 9 57 11 pm]
https://cloud.githubusercontent.com/assets/20954567/21434230/51e63ee4-c899-11e6-8eb3-349df3cde233.png
You can view, comment on, or merge this pull request online at:
https://github.com/fossasia/open-event-android/pull/892
Commit Summary

Changed the UI of button

File Changes

A android/app/src/main/res/drawable-hdpi/ic_calendar_white_24dp.png
https://github.com/fossasia/open-event-android/pull/892/files#diff-0
(0)
A android/app/src/main/res/drawable-mdpi/ic_calendar_white_24dp.png
https://github.com/fossasia/open-event-android/pull/892/files#diff-1
(0)
A android/app/src/main/res/drawable-xhdpi/ic_calendar_
white_24dp.png
https://github.com/fossasia/open-event-android/pull/892/files#diff-2
(0)
A android/app/src/main/res/drawable-xxhdpi/ic_calendar_
white_24dp.png
https://github.com/fossasia/open-event-android/pull/892/files#diff-3
(0)
A android/app/src/main/res/drawable-xxxhdpi/ic_calendar_
white_24dp.png
https://github.com/fossasia/open-event-android/pull/892/files#diff-4
(0)
M android/app/src/main/res/layout/activity_sessions_detail.xml
https://github.com/fossasia/open-event-android/pull/892/files#diff-5
(14)

Patch Links:

https://github.com/fossasia/open-event-android/pull/892.patch
https://github.com/fossasia/open-event-android/pull/892.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/pull/892, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AGQtlugWStoz2pbCNxmNktm-CFihKcNSks5rKrH-gaJpZM4LUMcZ
.

--
Best,
Harshit Dwivedi

Also for these actions, set the attributes as follows.
For adding to calender : showintoolbar = always
For bookmarking : showintoolbar = always
For sharing : showintoolbar = ifSpace


feat: Grpc Server 支持匿名函数的路由定义

[This message is created by hyperf-bot]
[APPROVAL NOTIFIER] This pull-request is APPROVED
This pull-request has been approved by: huangzhhui


[NFT-874] fix: updated chart colors

@cnasc This looks great, although that orange does look weird...  I see it's set to #ff6333 but it looks so unsaturated compared to how it renders in Figma. Anything weird going on? Or just environments/screenshots?


Manually setting the range is broken

The following command used to work for me:
time ck run cmdgen:benchmark.image-classification.qaic-loadgen --model=resnet50 \
--sdk=<IP_ADDRESS> --sut=eb6 --mode=performance --scenario=offline --target_qps=699 \
--power --power_server_port=4949 --power_server_ip=<IP_ADDRESS> \
--power_max_amps=0.234 --power_max_volts=300 --timestamp --verbose

but now it returns:
ptd-server 2023-02-07 08:07:40,119 [INFO] Sending reply to client 'Error setting current range: None'
ptd-server 2023-02-07 08:07:40,120 [INFO] Connection closed
ptd-server 2023-02-07 08:07:40,120 [WARNING] Client connection closed unexpectedly
ptd-server 2023-02-07 08:07:40,123 [INFO] Done processing

ptd-server 2023-02-07 07:06:55,599 [INFO] Connected to PTDaemon
ptd-server 2023-02-07 07:06:55,599 [INFO] Sending to ptd: 'RR'
02-07-2023 07:06:55.599: Response to client sent: Ranges,1,0.200000,1,300.000000
ptd-server 2023-02-07 07:06:55,599 [INFO] Reply from ptd: 'Ranges,1,0.200000,1,300.000000'
ptd-server 2023-02-07 07:06:55,599 [INFO] Initial range for Amps is Auto for Volts is Auto
ptd-server 2023-02-07 07:06:55,599 [INFO] Sending to ptd: 'SR,V,300.0'
02-07-2023 07:06:55.600: Volt range set to 300.000000V
02-07-2023 07:06:57.101: Response to client sent: Range V changed
ptd-server 2023-02-07 07:06:57,101 [INFO] Reply from ptd: 'Range V changed'
ptd-server 2023-02-07 07:06:57,101 [INFO] Sending to ptd: 'SR,A,None'
02-07-2023 07:06:57.101: ERROR: invalid range 0.000000A requested

I don't know if hastily resolving conflicts ahead of the code freeze has something to do with it.
@arjunsuresh @araghun @s-idgunji

Hi @psyhtest yes, it indeed happened due to the merge. This PR should fix it.

@psyhtest  -  Dejan worked with Arjun to undo the fix that led to this issue. Can you check if that solution works ?

@psyhtest I have made a new PR removing other changes not related to this issue. Please review it.


chore: bdd-spring-1584723904 to 0.0.1

chore: Promote bdd-spring-1584723904 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Increase Performance of Test-IsNanoServer

The cmdlet Test-IsNanoServer is used by many of the resources. This cmdlet uses Get-ComputerInfo to determine if the machine is Nano server:
function Test-IsNanoServer
{
    [OutputType([System.Boolean])]
    [CmdletBinding()]
    param ()

    $isNanoServer = $false

    if (Test-CommandExists -Name 'Get-ComputerInfo')
    {
        $computerInfo = Get-ComputerInfo

        $computerIsServer = 'Server' -ieq $computerInfo.OsProductType

        if ($computerIsServer)
        {
            $isNanoServer = 'NanoServer' -ieq $computerInfo.OsServerLevel
        }
    }

    return $isNanoServer
}

However, Get-ComputerInfo is very slow, taking over 2 seconds on average to execute.
This causes the resources to run slow and increases overall test time significantly.
We should find a better way to detect if a machine is Nano Server.

One possible way of increasing performance is to instead use a CIM command to pull the Win32_OperatingSystem class (https://docs.microsoft.com/en-us/windows/win32/cimwin32prov/win32-operatingsystem) and use the OperatingSystemSKU property:
$productDatacenterNanoServer = 143
$productStandardNanoServer = 144

function Test-IsNanoServer
{
    [OutputType([System.Boolean])]
    [CmdletBinding()]
    param ()

    $operatingSystemSKU = (Get-CimInstance -ClassName Win32_OperatingSystem).OperatingSystemSKU

    return ($operatingSystemSKU -in ($productDatacenterNanoServer,$productStandardNanoServer))
}

Is there anyone who knows a reason why this would not be a viable alternative? @gaelcolas , @johlju

If the class exist in a Nano server then I don't see an issue using it.

The class exists on Nanaoserver. Ref: https://docs.microsoft.com/en-us/previous-versions/windows/desktop/legacy/mt588480(v=vs.85)
Any reason not to go ahead and implement this as in "Add Test-IsNanoServer dsccommunity/DscResource.Common#9"?

No reason at all :) Please go ahead @HansOMartinsen !

Implementing #685 will resolve this issue. I'll be planning this over the next week.


[CXF-6998] Search for ApplicationPath in super classes since its not …

…inherited.
See also: https://java.net/jira/browse/JAX_RS_SPEC-534

Hi Andriy
That spec statement is many years old, coming form 1.1 or earlier, before ApplicationPath was introduced (AFAIK) thus one may say well that statement does not apply to ApplicationPath as it is really about the resource classes and methods.
(FYI, CXF checks class/interface level Path itself and never had any issues with various TCKs.)
I'd say it should def be valid to have a custom Application extending another custom Application with ApplicationPath. So IMHO the patch is safe enough :-)
Sergey

👍 , sounds good to me, thanks @sberyozkin and @johnament !


[Python BQ] Substitute final destination schema when no input schema is specified

When users perform BQ file load writes, we depend on them to provide an input schema. This is a limitation because for larger loads we create temp tables. Although we don't need a schema to write to an existing table, we do need it when creating a new table. Without an input schema, this use case fails.
To fix this, we should fetch the final destination's schema when necessary.

R: @shunping

Thanks! LGTM with some suggestion about adding comments in the code


ExtendedSliverAppbar centerTitle

Seem like the title always center, is that so?
Could yo please provide an option to align the title, and hide the back button in the next release? Thanks

ExtendedSliverAppbar(leading:Contianer(),mainAxisAlignment : MainAxisAlignment.start)
https://github.com/fluttercandies/extended_sliver/commit/51c850aa058cf4fff4385a04b877e3a73a8bdb9f

Thanks


Update dependencies

Codecov Report
Base: 59.55% // Head: 60.66% // Increases project coverage by +1.11% :tada:

Coverage data is based on head (5a0a294) compared to base (6885b86).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   59.55%   60.66%   +1.11%
==========================================
  Files           2        2
  Lines         450      450
==========================================
+ Hits          268      273       +5
+ Misses        106      103       -3
+ Partials       76       74       -2

Flag
Coverage Δ

unittests
60.66% <100.00%> (+1.11%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

external_server.go
65.38% <ø> (ø)

local_client.go
58.16% <100.00%> (+1.70%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


late return types should be invariant locations

As it turns out, for the same reasons variable return types must be invariant locations, late return types must too.
Here is an example of unsound code that exploits the fact that they aren't:
class C<out T>()
{
    shared late T t;
}
value c = C<String>();
(c of C<Object>).t = 123;
print(c.t.size);

Thanks @Zambonifofex, fixed!


docs(team): Add to team

Information

Type
Breaking change

Doc
No

Description
I want to support and improve tsed :).

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

v6.39.1
GitHub release

Your semantic-release bot :package::rocket:


Headers in Configuration.DefaultHeader are not added to the HttpRequestMessage in ApiClient.CreateRequestMessage

This seems like an accidental oversight. We need to add headers to the outbound http request logging purposes.
We're adding them to the Configuration object with AddDefaultHeader(key, value). The DefaultHeader dictionary does not appear to be read from at all.

Hey @joshsacks!
Thanks for reporting this issue, I'll look into it.

@TheUnderScorer It would also be super helpful if Configuration could have an optional property for HttpClient so we can pass in a client instance instead of ApiClient instantiating a new HttpClient() each time. (Of course if one not set, then ApiClient should instantiate a new one as it does now).

I'm happy to make this change (and resolve the original issue re DefaultHeader) as well if you'll entertain pull requests from the outside community.

Sadly, for now we don't accept external contributions. However, I like the suggestion of having extra configuration option for HttpClient so I'll include it in the next release 🙂


Prepare next Dev Cycle v1.99.0-dev

How to categorize this PR?

/area delivery
/kind enhancement
What this PR does / why we need it:
Prepare next Dev Cycle v1.99.0-dev.
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
Release note:

/hold

/retest-required

/hold cancel


Logging to a terminal and to a file

I've already asked about your journaling. I really like it. Question: is it possible to use LOG(INFO) and so on to output simultaneously to a file and to a terminal?

Yes. You need to set up https://github.com/tdlib/td/blob/master/tdutils/td/utils/CombinedLog.h as a logger.

Thx)


Input box for ui-select multiple ALWAYS forcing text entry at new line

Bug description:
Input box for ui-select multiple forcing text entry at new line, even if values do not take up entire first line
There is already closed issue for this #1556 undesired behavior.. but for older version. Since I got no help there, opening new issue.
Link to minimally-working plunker that reproduces the issue:
plunker
All selects forces new line when there is at least 1 item selected.. this was not happening before..
Notice also that only like 40% of the line is clickable in order to start typing in it:

Version of Angular, UI-Select, and Bootstrap/Select2/Selectize CSS
Angular: 1.6.4
UI-Select: ^0.19.8
Bootstrap/Select2/Selectize CSS (if applicable): ^3.3.7

Hi,
Thanks for the issue, maybe you could have a look where the problem is coming from and make a PR?
Happy to review for you.

I didn't find the actual cause yet, but it got introduced in 0.19.7. Reverting back to 0.19.5 fixes the issue.

+1

Oh I haven't seen it was a recent issue, I commented the older one. Here is the fix I found.
In the code of the updateIfVisible function, it tries to make the input fit in the line using var inputWidth = containerWidth - input.offsetLeft.
updateIfVisible = function(containerWidth) {
  if (containerWidth === 0) {
    return false;
  }
  var inputWidth = containerWidth - input.offsetLeft;
  if (inputWidth < 50) inputWidth = containerWidth;
  ctrl.searchInput.css('width', inputWidth+'px');
  return true;
}

However it cannot be done since the input is already on the second line. By reducing the input first and enlarging it to the rest of the block, it works:
updateIfVisible = function(containerWidth) {
  if (containerWidth === 0) {
    return false;
  }
  ctrl.searchInput.css('width', '50px');
  setTimeout(function(){
    var inputWidth = Math.max(50, containerWidth - input.offsetLeft - 10);
    ctrl.searchInput.css('width', inputWidth+'px');
  }, 0);
  return true;
}

also, there is a height problem for multiple. the input's height is shorter than it should be, at least for bootstrap theme. you can compare multiple input's height with non-multiple

+1, same problem with chrome

Changing this line https://github.com/angular-ui/ui-select/blob/master/dist/select.js#L839
from
var inputWidth = containerWidth - input.offsetLeft;
into
var inputWidth = containerWidth - angular.element(input).offset().left;
fixes it for me.

As far as I can tell the root issue is that ui-select is relying on the element's clientWidth property when calculating the available area for the input. clientWidth includes the element's padding, so it overestimates the available area.
Previously, ui-select compensated by subtracting an extra 10px from the container width:
var inputWidth = containerWidth - input.offsetLeft - 10;
But that was removed in 7ad4ef13273bf0900de0735fc5d6e04003c0bfc3
Even if that was change was undone, I believe it would still break for anyone who wanted to set a larger custom padding on their element. I think the best solution is probably to take into account the actual computed padding values from window.getComputedStyle(element) when determining available area
I can probably work on a pr on the weekend but probably not before then

Finally got a clue why the "-10" was removed. In not multiple case, this -10 will shrink from 10 pixels the input (so visually, the whole select shrinks from 10 pixels).
I'm now using:
var inputWidth = containerWidth - input.offsetLeft - (ctrl.multiple ? 10 : 0);

None of solutions actually fix the problem. I think the problem is much deeper since it does not provide enough flexibility. User should be able to access drop-down list by clicking ANY space on input box. If you decrease the input width as suggested, generally ends up with less interaction area even if it may not force text entry in new line.

I don't know what the browser support requirements are but this is an easy fix for flexbox (IE10+) the current structure for multi-select is (not exact names)

        first selection
        ....

    <input class="ui-select-search">
</wrapper div>

change it to

    first selection
    ....
    <input class="ui-select-search">
</wrapper div>

and give it the following css
.wrapper{
    display: flex;
    flex-wrap: wrap;
}
.ui-select-search{
    flex-grow: 1;
}

this will allow you to let css handle it automagically and not have js setting the width at all

I would also expect to be able to click anywhere in the box to search.
We can achieve this easily with basic HTML, without any complicated calculations. Just wrap ui-select-match and ui-select-search in a :

    ...

  <input type="text" class="ui-select-search" id="ui-select-search">

And add the following CSS to make it look like the whole thing is an input:
label {
  display: block;
  cursor: text;
}

+1

Can we get a fix for this merged? This really breaks the functionality of this library.

yes https://github.com/angular-ui/ui-select/pull/2125 is working for my case thanks @sunilkandakatla

Can this be closed? it has a merged pr and is very old.

AFAICT the bug is still there, at least for 0.19.8. I don't think PR #2125
have been merged yet.
On Fri, Aug 17, 2018, 11:10 Stephen Dahl<EMAIL_ADDRESS>wrote:

Can this be closed? it has a merged pr and is very old.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/angular-ui/ui-select/issues/1980#issuecomment-413878081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEKqyqJuzkkVGWQdmKw_-bp2YMmEPdwEks5uRs69gaJpZM4NDwag
.

@stephen-dahl I just successfully tested PR #2125 on Chrome 68, Firefox 61 and IE 11 and it does fix the problem in my projects, but the PR hasn't been merged into master yet.


Throws exception on restart

Getting the following exception at startup:
2024-11-26 09:48:58.737 WARNING (MainThread) [custom_components.zigbee_lock_manager.zha_manager] automation.reload service not available at this time
2024-11-26 09:48:59.738 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Zigbee Lock Manager - front_door_lock for zigbee_lock_manager
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 635, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/zigbee_lock_manager/init.py", line 23, in async_setup_entry
await hass.services.async_call("automation", "reload")
File "/usr/src/homeassistant/homeassistant/core.py", line 2735, in async_call
raise ServiceNotFound(domain, service) from None
homeassistant.exceptions.ServiceNotFound: Action automation.reload not found

What version of HA are your running?
Is this preventing the integration from starting?  What if you try reloading from the integration page after restart? (Curious if that works)
On my test HA instance (Core 2024.12.0), I get a warning that automation.reload isn't availably during startup, but this doesn't impact HA ability to start and the integration doesn't need it during startup so it can be ignored.


Add API debugging support with enhanced logging

This commit adds a logger to the provider to help with debugging API calls. The logger is disabled by default and can be enabled by setting the environment variable MIST_API_DEBUG=true. Test cases have been added to verify that the provider is functioning as expected and the refactoring that has been made does not break any existing functionality

Hi @tmunzer
I opened this in hopes it would make troubleshooting and debugging easier for the provider. Feel free to take a look at the branch and do some tests on it and let me know what you think. There is likely some room for improvement, but let me know if you have any questions on the new feature or the tests.

Thanks @zbuchheit

No problem! @tmunzer


Update sbt to 1.5.0

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-minor

Closing in favor of : https://github.com/Banno/vault4s/pull/186 to go forward.


my gcc-make-run complile error

i just install atom for windows as good platform for ide and i want to do c projects when i compile and run the program throws an error as compile error i added atom in my path environmnet variable plz help me in this problem

it's a bit late, but there is not enough information to judge the cause of your error.


Can't get file open options in CLI mode without running the mdserver

Related to #1053
In CLI mode, I need to do this:
OpenMDServer()
plainTextOpenOptions = GetDefaultFileOpenOptions("PlainText")

Otherwise I see this
Traceback (most recent call last):
  File "script_a.py", line 4, in <module>
    plainTextOpenOptions = GetDefaultFileOpenOptions("PlainText")
visit.VisItException: "PlainText" is not a valid plugin type.  Make sure the Metadata Server is running.

There is a brief comment in the Read options via cli section, and in the example for the GetDefaultFileOpenOptions function.

Why must it be that way?

If we don't change this, then we should fix the example here to reflect this as well.

GetDefaultFileOpenOptions() should open MDServer if its not already available (just like https://github.com/visit-dav/visit/issues/1053)


Incorrect instructions for "Migrating from an IP firewall rule to a virtual network ACL" for Cosmos DB

The instructions for Cosmos DB "Migrating from an IP firewall rule to a virtual network ACL" is incorrect.
Based on its explanation:

After an Azure Cosmos DB account is configured for a service endpoint for a subnet, requests from that subnet are sent to Azure Cosmos DB with virtual network and subnet source information instead of a source public IP address. These requests will no longer match an IP filter configured on the Azure Cosmos DB account, which is why the following steps are necessary to avoid downtime.

That means the suggested way is to add the VNet ACLs first before the service endpoint is turned on using the "IgnoreMissingVnetServiceEndpoint" and then turn on the service endpoint.  That is also explained in virtual network service endpoints FAQ.

Certain services (such as SQL and CosmosDB) allow exceptions to the above sequence through the IgnoreMissingVnetServiceEndpoint flag. Once the flag is set to True, VNet ACLs can be set on the Azure service side prior to setting up the service endpoints on the network side. Azure services provide this flag to help customers in cases where the specific IP firewalls are configured on Azure services and turning on the service endpoints on the network side can lead to a connectivity drop since the source IP changes from a public IPv4 address to a private address. Setting up VNet ACLs on the Azure service side before setting service endpoints on the network side can help avoid a connectivity drop.

However the provided PowerShell commands in this doc did not reflect this and the line above it seems to suggest the readers to enable the service endpoint first, which will actually recur downtime.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 801eeaa5-d78f-0dd9-e605-c27571ccc37b
Version Independent ID: 0b4a6e97-7a60-6013-7a5b-66f4e01a81d9
Content: Configure virtual network based access for an Azure Cosmos account
Content Source: articles/cosmos-db/how-to-configure-vnet-service-endpoint.md
Service: cosmos-db
GitHub Login: @ThomasWeiss
Microsoft Alias: thweiss

@t-bzhan Thanks for your findings, I'm assigning this to the author for the feedback review.

#reassign:seesharprun

@t-bzhan, you are absolutely right.
We have a change queued in our pipeline to re-order the steps in this section to make this clearer.
Thank  you for your feedback.
This issue will auto-close once that change is queued for publishing.


fix for flakey-dns.rb example

Ruby noob. Made a simple edit/copy to fix issue #63. Given that each block is written for a distinct dns query, the otherwise block is a discrete entity separate from the microsoft regex.

Coverage remained the same at 83.75% when pulling 543ba2e1ee26929f74a01fa7cc65a4b142de3745 on kaleforsale:master into b596c60f5deb5dc7305a439dbfbb57cb366c09a5 on ioquatix:master.

Thanks so much for your contribution and I hope you'll forgive me for taking so long to attend to it.


Implement 3GPP Releases for citation and linking with 3GPP specifications

3GPP has the concept of a "release":
https://www.3gpp.org/specifications/releases
This page provides all releases:
https://portal.3gpp.org/#/55934-releases

The "start date" is when the release is opened, and the "end date" is when the release is officially released. "closure date" is when the release is retired.
This information is fully provided in the 3GPP MDB file, under the Releases table:

Notice that there are duplicate rows in this table, e.g. Release 17:

The real entry to obtain is under these conditions:

PROJECT_ID must not be NULL
sort_order the highest integer among the rows of the Release

We will need to add entries to relaton-data-3gpp for every Release
We will need to link all 3GPP specs to every Release ("partOf"?)

@ronaldtse we have moved to use CSV source. Is this issue still actual?


Add "Deep learning papers"

I'd like to add a curated list of deep learning papers. Thank you.

Hey. As I see it, your list is already a sublist of the awesome-deep-learning list. I'm not the maintainer of the list but imho it is good that the main awesome repo doesn't contain all the  niche lists, just the more broad topic lists, and then those awesome lists are collections of more niche lists themselves.
By the way seems liek a really comprehensive list, good job :smile:
(Follow the PR guidelines)

Looks good. Interesting content.
Only one minor nitpick: Table of Contents => Contents.


Program received signal SIGTRAP

Hi, I am using Ubuntu 20.04 on WSL2 to develop Mynewt on Nordic 52840 DK. I am trying to build a simple NimBLE application following the doc.
Building and flashing worked fine, but gdb received signal SIGTRAP when debugging, where it is supposed to the idle loop in main:
Reading symbols from /root/nimble-dev/miniBLE/bin/targets/ble_tgt/app/apps/ble_app/ble_app.elf...
--Type <RET> for more, q to quit, c to continue without paging--c
0x0002011a in strsep (stringp=stringp@entry=0xfffffff9,
    delim=delim@entry=0x941 "(=\321\001\230\203hZx*t\331x\232xB\352\001\"j\202\032yjt\031z\332yB\352\001\"\252\202Zy\233yB\352\003#\353\202\032\360\213\370!F\001\252A\362\002")
    at repos/apache-mynewt-core/libc/baselibc/src/strsep.c:9
9               char *s = *stringp;
Resetting target
0x00020b82 in __preinit_array_start ()
(gdb) c
Continuing.

Program received signal SIGTRAP, Trace/breakpoint trap.
0x0002011a in strsep (stringp=stringp@entry=0xfffffff9,
    delim=delim@entry=0x941 "(=\321\001\230\203hZx*t\331x\232xB\352\001\"j\202\032yjt\031z\332yB\352\001\"\252\202Zy\233yB\352\003#\353\202\032\360\213\370!F\001\252A\362\002")
    at repos/apache-mynewt-core/libc/baselibc/src/strsep.c:9
9               char *s = *stringp;
(gdb) c
Continuing.

Program received signal SIGTRAP, Trace/breakpoint trap.
0x0002011a in strsep (stringp=stringp@entry=0xfffffff9,
    delim=delim@entry=0x941 "(=\321\001\230\203hZx*t\331x\232xB\352\001\"j\202\032yjt\031z\332yB\352\001\"\252\202Zy\233yB\352\003#\353\202\032\360\213\370!F\001\252A\362\002")
    at repos/apache-mynewt-core/libc/baselibc/src/strsep.c:9
9               char *s = *stringp;

I am not sure whether this is the result that the tutorial indicates. I am not running other debugging sessions and there is no breakpoint added by me in the program. I wonder what cause the signal SIGTRAP. Thank you.

Sorry, I figure it out. Since the BLE doc presumes that you have come through the previous chapters, it omits the steps to load the bootloader. However, I happened to load another stack into my board yesterday, so the bootloader was messed up.


Url-based pagination

For Spotify assets where we lazy-load extra items, create this as a URL parameter. This will allow us to hit 'back' and return to the scroll point and items that we last had.
Would be worth investigating a better way to do this prior.

Kind of remedied by using <body> as the scrolling element. Browser remembers previous scroll position.


RetroArch UI missing icons / looks ugly

Describe the bug
RetroArch from Nixpkgs lacks icons / assets (see screenshots below) and looks ugly.
I believe it's been like this for years, but I thought it'd be good to document. AFAIR, upstream mixes user data and program data in their config (e.g. ~/.config/retroarch/retroarch.cfg), which could mean that we might need fix some of this upstream. (IMHO, assets is part of the program, not user data.)
Steps To Reproduce
Steps to reproduce the behavior:

nix-build -A retroarch (or nix-build -A retroarchFull)
./result/bin/retroarch
Look at the ugly UI without icons.

Expected behavior
I expect the UI to have icons and look OK, like it does with the flatpak package.
Screenshots
RetroArch from Nixpkgs (bad):

RetroArch from flathub (good):

Notify maintainers
@thiagokokada @matthewbauer @kolbycrouch @aanderse @edwtjo @MP2E
Metadata
Nixpkgs master and release-22.11. (And probably all the way back to the initial Nix expression.)

This is expected, we don't ship assets at all since RetroArch itself can download the assets. Just run Online Updater -> Update Assets and the issue should be fixed.

Does the flatpak also require manual download? I don't remember doing that.
IMHO it's strange (and annoying) to have to manually complete the program install like that though.

Does the flatpak also require manual download? I don't remember doing that.

Not sure, but it is not like shipping without assets is unexpected behavior. AFAIR Android builds are like this, or used to be at least.
There is even a prompt at the first run saying you do to this, so upstream fully expects that some releases will not include assets.

IMHO it's strange (and annoying) to have to manually complete the program install like that though.

It is not like I am against shipping assets though. The refactor in https://github.com/NixOS/nixpkgs/pull/198271 was originally done so we could support adding more files inside the wrapper (now done with symlinkJoin), and this could support the assets case.
However we still need to patch the retroarch derivation so it points inside symlinkJoin path, and this gets pretty messy (the reason why retroarchBare itself still depends on libretro-core-info instead being done inside the wrapper).
If you can think a way to get out of this mess, I would be happy to review/merge your solution.

Started a PR in https://github.com/NixOS/nixpkgs/pull/206707, however as I said I got in the ugly issue where retroarch itself needs a patch to point out the assets directory to the symlinkJoin path, however I don't have a good way to get the path from symlinkJoin instead of retroarch itself.
Helps are welcome to fix this issue.

@thiagokokada: Thanks for the input! (And for the PR.)
I didn't know (or had completely forgotten?) upstreams attitude towards the UI assets (rely on installing at runtime). If it's difficult to workaround in nixpkgs, then I'm fine with waiting for that to be fixed upstream (if ever). (I cannot imagine their reasoning for the current behaviour.)

Closed by #206707.

Upstreamed a few changes: https://github.com/libretro/RetroArch/pull/14756
I still don't know how to properly hardcode the core info files, so for now we will still need out-of-tree patches. But with this PR above we can practically drop the assets patch (except for the part where we change it to always load from the defaults, but this is trivial).

Upstreamed core info patch: https://github.com/libretro/RetroArch/pull/14761.
With the changes now the patches are trivial: https://github.com/thiagokokada/nixpkgs/commit/7cc47607376b7afa1bbce5adf49d55f545cbd467

I think that the default asset directory installed by the package is nonexistant, also root-owned, so the "download assets" option doesn't work by default? Asset.zip extraction also doesn't report any errors when trying to update them without changing the directory to user-owned, but I guess this part is upstream's fault. After changing the directory to user-owned, asset downloading succeeded and the problem went away.

@thiagokokada, sorry for asking but can I now pass retroarch-assets and retroarch-joypad-autoconfig to retroarch? I don't want to download them through Online Updater because they are in the repos. Can I override something to use repo's versions?


xine-ui: remove a unnecessary ; in desktop file

Motivation for this change
Closes #128958
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137128 at cb1aa865 run on x86_64-linux 1

2 packages built successfully:

dvdstyler
xine-ui


tf.nn.xw_plus_b not in API docs

The method tensorflow.nn.xw_plus_b() exists but is not documented in the online API documentation.

I think this is intentional: it's better to use tf.matmul(x, w) + b directly.  We are working to lock down our symbol exports so that only documented symbols are exported, but we're a bit lax at the moment.

tf.nn.xw_plus_b()  has option to name the operation while tf.matmul(x, w) + b has not. Is there any way around that?

@dParadiz you can just use tf.add(tf.matmul(x, w), b, name='')