Error when running visualization of network in DIGITS5.1-dev

Training and testing with models runs just fine. Even adapting the network with other layers works fine. However, at whatever stage, if I click the visualization button for the adapted model, I get the following error output:
500: INTERNAL SERVER ERROR
AttributeError
'int' object has no attribute '_values'

I have tried installing the graphviz library (which I guess is the backend used here) using sudo apt-get install libgraphviz-dev but that did not solve the issue.
Steps to reproduce in my case

Create a new classification model
At the standard LeNet/AlexNet network, click customize
Then hit the visualize button

I can't reproduce this. Can you run in debug mode (digits-devserver --debug) and see if you get a stacktrace?

I will do so tomorrow at work and get back to you on this!

@lukeyeager I ran in debug, the stacktrace retrieved is
Traceback (most recent call last):
  File "/home/spu/.local/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/spu/.local/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/spu/Github/DIGITS/digits/webapp.py", line 81, in decorated
    return f(*args, **kwargs)
  File "/home/spu/Github/DIGITS/digits/model/views.py", line 126, in visualize_network
    ret = fw.get_network_visualization(flask.request.form['custom_network'])
  File "/home/spu/Github/DIGITS/digits/frameworks/caffe_framework.py", line 145, in get_network_visualization
    caffe.draw.draw_net(net, 'UD').encode('base64') +
  File "/home/spu/Github/caffe/python/caffe/draw.py", line 223, in draw_net
    return get_pydot_graph(caffe_net, rankdir, phase=phase).create(format=ext)
  File "/home/spu/Github/caffe/python/caffe/draw.py", line 167, in get_pydot_graph
    node_label = get_layer_label(layer, rankdir)
  File "/home/spu/Github/caffe/python/caffe/draw.py", line 107, in get_layer_label
    layer.pooling_param.kernel_size[0] if len(layer.pooling_param.kernel_size._values) else 1,
AttributeError: 'int' object has no attribute '_values'

Okay with the stacktrace I found the issue. It seems that this is a problem in CAFFE, where someone tried to fix the drawing of the nets in this PR.

Okay so the solution is quite straightforward, simply reverting back the changes in the draw.py file.
So to solve this, replace in draw.cpp
layer.pooling_param.kernel_size[0] if len(layer.pooling_param.kernel_size._values) else 1,
seperator,
layer.pooling_param.stride[0] if len(layer.pooling_param.stride._values) else 1,
seperator,
layer.pooling_param.pad[0] if len(layer.pooling_param.pad._values) else 0)

by
layer.pooling_param.kernel_size,
seperator,
layer.pooling_param.stride,
seperator,
layer.pooling_param.pad)

And the digits visualization button works again as before!

Thanks for tracking it down! You might want to ping Evan for a fix on https://github.com/BVLC/caffe/pull/5010.

@StevenPuttemans  nice notice


Support Python3 in Ghidra, drop ghidra_bridge

Closes #73 and bumps to the next major version.

Two separate issues may have killed this attempt:
No Subclassing Java Classes

https://github.com/jpype-project/jpype/issues/420

This affects the following line(s):
https://github.com/binsync/libbs/blob/40bc932e32d8b3e904c0faf451f727bd5d98f8f8/libbs/decompilers/ghidra/hooks.py#L185
This would mean libbs can no longer be used at all to make GUI callback classes unless we can find a workaround.
Implementing Java Classes is Thread Dependent (?)

https://github.com/dod-cyber-crime-center/pyhidra/issues/41

It's possible there is a workaround for this. If not, then we no longer can do this:
https://github.com/binsync/libbs/blob/40bc932e32d8b3e904c0faf451f727bd5d98f8f8/libbs/decompilers/ghidra/hooks.py#L17
That would mean we can no longer detect when any Artifact is changed, which means no caching or special change handling. This also makes the approach dead for BinSync proper.

I'll wait for a response from the Pyhidra dev's to see if this PR is dead in the water.

Based on the current commit, the following works in UI mode:
from libbs.api import DecompilerInterface
deci = DecompilerInterface.discover()
print(deci.functions.values())

It currently takes about 2 seconds to collect all the functions in light mode!

Headless mode now works:
from libbs.api import DecompilerInterface
deci = DecompilerInterface.discover(headless=True, force_decompiler="ghidra", binary_path="./posix_syscall")
for addr, light_func in deci.functions.items():
    print(light_func)
    print(deci.functions[addr]) # decompilation!

After headless is initiated, it takes only 0.7340998649597168 seconds to iterate and decompiler every function!

Headless mode now works with project loading:
from libbs.api import DecompilerInterface
# load from a project should be 5x faster on this binary after loading once.
deci = DecompilerInterface.discover(
    headless=True, force_decompiler="ghidra", binary_path="./posix_syscall"
    project_location=".", project_name="posix_syscall_ghidra", analyze=False
)
for addr, light_func in deci.functions.items():
    print(light_func)
    print(deci.functions[addr]) # decompilation!

This is also on CI:
https://github.com/binsync/libbs/blob/104cd93f6ce63b1c42bdfbbdbca477b266df7f65/tests/test_decompilers.py#L154

Summary of PR
This pull request was closed and replaced by #84 since we were unable to fully replace ghidra_bridge backend with pyhidra. The following problems killed the full replacement:

Inconsistent thread artifact changer callbacks, which was related to https://github.com/binsync/libbs/pull/82#issuecomment-2155323490. This manifested as inconsistent events being generated when changing something like a function name.
Inability to use PySide6 inside pyhidra's interpreter. This problem took a lot of time to debug, and I never fully found the reason. It only manifested on Mac and could be generated with the following code in Pyhidra:

from PySide6.QtWidgets import QApplication
QApplication()

This caused a segfault.
Since libbs is used by other downstream plugins for decompilers, which rely on Qt being available, #2 was the real killer.
I will leave this branch alive since, in its current state, it is usable inside the UI, just without QT and those callbacks being consistent.


No extension options?

A recent update turned the "Hide Shorts Video" option on by default with a note saying that we can turn it back off in the options. My problem is now, that I can't find those options anywhere.
The dropdown menu when clicking on the extension icon does not contain the entry "Options" like it does in other extensions that have options:

The actual extension settings also don't have an options button:

Now I'm wondering if the options actually hide somewhere else or if there is a bug in version 1.4.0 that makes them not show up. Either way I would like to disable this option again, since I don't mind having shorts in my subscription feed.

Thank you for your feedback.
Please refer to the image below for disabling ;)

( I'm planning to post a guide about disabling, but I'm thinking about how to guide users to the guide... )

Ah I see, thanks. That worked no problem.


Test Failure: Assertion `num_prompts_shown == (expected_bad_prompts_shown)' failed.

I'm running the test suite with 12181ad1ac0fddf0ffec14a4da461a23ccb2e808 and it's failing - here's the test-suite.log:
=================================================
   google-authenticator 1.03: ./test-suite.log
=================================================

# TOTAL: 1
# PASS:  0
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: tests/pam_google_authenticator_unittest
=============================================

pam_google_authenticator_unittest: tests/pam_google_authenticator_unittest.c:335: main: Assertion `num_prompts_shown == (expected_bad_prompts_shown)' failed.
Testing base32 encoding
Testing base32 decoding
Testing HMAC_SHA1
Loading PAM module

Running tests, querying for verification code
Testing failed login attempt
pam_get_user() failed to get a user name when checking verification code
FAIL tests/pam_google_authenticator_unittest (exit status: 1)

Looks like dup of #7, which I have not had time to look into. "It passes for me".


Improve error message when an endpoint is called that points to an invalid URL

Is your feature request related to a problem? Please describe.
In the case of an endpoint that has changed (#59 ), the error message is
~/.local/lib/python3.8/site-packages/averbis/core/_rest_client.py in __request(self, method, endpoint, **kwargs)
    807
    808         response = raw_response.json()
--> 809         self.__handle_error(response)
    810         return response
    811

~/.local/lib/python3.8/site-packages/averbis/core/_rest_client.py in __handle_error(response)
   1268     @staticmethod
   1269     def __handle_error(response):
-> 1270         if response["errorMessages"] is None or len(response["errorMessages"]) == 0:
   1271             return
   1272

KeyError: 'errorMessages'

Describe the solution you'd like
This is not helpful at all. We should raise a specific Error, such as an EndpointDoesNotExistError.

I investigated this issue. There are three kind of possible errors:

General errors with the request that are potentially outside of the scope of our platform or where no access is granted. Sometimes raw_response.json() does not even return a json. In that scenario, we want to just print the status_code to indicate what went wrong.
Example: Exception("Client request failed with status code 401.")

Errors within the domain of our platform, but when a missing endpoint (see initial example in this issue) is called. In that case we want to return the status_code and what is given in the "message" of the json response. This is what initially caused a problem because the error message was in "message" and not in "errorMessages".

json={
          "servlet": "Platform",
          "message": "Not Found",
          "url": "my_url",
          "status": "404",
      },

Example: Exception("Client request failed with status code 404.\nError message is: Not Found")

We call an existing endpoint, but there is a problem. For this, we already had some logic, but we should additionally return the status_code as it can be informative:
Examples: Exception("Client request failed with status code 400.\nError message is: A project with name 'test' already exists.")

In case 1, we probably want to print not only the status code but also the actual response which may contain additional information. Cf. https://github.com/averbis/averbis-python-api/commit/41b5f8940141cb289de5aaa40418f5714f9fd6e0
Also, case 3 should be case 1 or 2: if we call an endpoint and there is a problem, then there should either be a JSON response with the error message or there should be a informative message in the response body. Better the first. If this is not the case, the platform needs to be changed to provide a better message
However, there is a case 4 which is we either not talking to a platform or the platform is offline - in which case we should tell the user that there is actually no platform or that the connection could not be established at all.

Thanks for the discussion point. I agree that we can reduce all kinds of errors into two kinds: Either there is information in the raw_response (in the field status_code and reason) which could by text, bytes or something different OR there is error information in the response when the response is a json object. Then we assume that it is located either in the field "message" or in the field "errorMessages". I included this logic in my latest commit. Please have a look.


GitHub Pages Documentation

This PR moves the existing README.md documentation to GitHub pages using the gh-pages branch configuration. Changes to the contents of the documentation are minimal and focused on formatting. The theme used is Just the Docs.

Closing this PR to refactor documentation using the /docs directory approach. This will allow us to avoid some of the side effects of the build process.


Make it so that refreshing a study page will keep the player on the same move, even if not on the mainline

Implemented in v.2.3.16


Fix typo in validator.py that produced imprecise output message

Prior to this change, running the validator on https://github.com/OCFL/fixtures/pull/66 produced:
[E093] OCFL Object root inventory fixity block for digest algorithm ??? has digest..

After this change, the output is now:
[E093] OCFL Object root inventory fixity block for digest algorithm md5 has digest..

Coverage increased (+0.006%) to 94.875% when pulling b5632d4f462dd88d200d2d680cf55399ab32769a on awoods:fixity-check-typo into 2f7208768205649899fa50fb8e82fcacabae670b on zimeon:main.

Also resolves: https://github.com/zimeon/ocfl-py/issues/38

Fixed in develop:
(py38) simeon@RottenApple ocfl-py> ./ocfl-validate.py fixtures/1.0/bad-objects/E093_fixity_digest_mismatch/
[E093] OCFL Object root inventory fixity block for digest algorithm ??? has digest 9eacfb9289073dd9c9a8c4cdf820ac71 for file v1/content/test.txt which doesn't match calculated digest eb1a3227cdc3fedbaec2fe38bf6c044a for that file (see https://ocfl.io/1.0/spec/#E093)
INFO:ocfl.object:OCFL object at fixtures/1.0/bad-objects/E093_fixity_digest_mismatch/ is INVALID
(py38) simeon@RottenApple ocfl-py> git pull
remote: Enumerating objects: 8, done.
remote: Counting objects: 100% (8/8), done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 8 (delta 3), reused 4 (delta 3), pack-reused 0
Unpacking objects: 100% (8/8), done.
From github.com:zimeon/ocfl-py
   a7e236f..e6e6fdc  develop    -> origin/develop
Updating a7e236f..e6e6fdc
Fast-forward
 ocfl/validator.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
(py38) simeon@RottenApple ocfl-py> ./ocfl-validate.py fixtures/1.0/bad-objects/E093_fixity_digest_mismatch/
[E093] OCFL Object root inventory fixity block for digest algorithm md5 has digest 9eacfb9289073dd9c9a8c4cdf820ac71 for file v1/content/test.txt which doesn't match calculated digest eb1a3227cdc3fedbaec2fe38bf6c044a for that file (see https://ocfl.io/1.0/spec/#E093)
INFO:ocfl.object:OCFL object at fixtures/1.0/bad-objects/E093_fixity_digest_mismatch/ is INVALID

Thanks @awoods !


Server Logging

Server writes to a .log file to help debugging in the future

This is obsolete now with new tomcat impl right?


chore: fix some comments

Description

Testing Instructions

This PR appears to be spam, along with #7908.


Move Join nodes to your cluster to the install section

File: ee/ucp/admin/configure/join-nodes/join-linux-nodes-to-cluster.md
Its currently hard to find the documentation on Joining nodes to your cluster since there are a lot of configuration steps for UCP after install. I would propose adding the join your nodes to the install section since it is technically part of the Install process: https://docs.docker.com/ee/ucp/admin/install/

cc @paigehargrave

@david-yu Hi David - Thank you for pointing this out! We will address this for the content refactor - I'm hoping to include this with the Amberjack install work.

Closing. The current structure will change once the docs are migrated to the Mirantis website.


extension.extensionKind is wrong for extensions not running on the host

Testing #148039

in a remote window use vscode.extensions.allAcrossExtensionHosts and look at the extensionKind of the extensions that are not located on the remote.
:bug: All extension have Workspace set.
Extension located on vscode-local should be UI

* vscode.bat
  * extensionUri: vscode-local:/usr/share/code-insiders/resources/app/extensions/bat
  * extensionKind: Workspace
* vscode.clojure
  * extensionUri: vscode-local:/usr/share/code-insiders/resources/app/extensions/clojure
  * extensionKind: Workspace
* vscode.coffeescript
  * extensionUri: vscode-local:/usr/share/code-insiders/resources/app/extensions/coffeescript
  * extensionKind: Workspace
* vscode.configuration-editing
  * extensionUri: file:///home/martin/.vscode-server-insiders-testresolver/bin/ab86e0229d6b4d0cb49cfd6747c92cafcd2bd4af/extensions/configuration-editing
  * extensionKind: Workspace

Extension for testing: https://github.com/aeschli/ext-all-extensions


Testing in UseInMemoryDb EFCore.

Hello and Good day! is there any work around when using BatchDelete() in UseInMemory in ef core dbcontext?

Bulk Tests can not have UseInMemoryDb(usually used for Tests), because InMemoryProvider does not support Relational-specific methods.
You could try using SQLite with In-Memory mode instead, since SQLite is also supported.
Duplicate of https://github.com/borisdj/EFCore.BulkExtensions/issues/24


Add DLL Export to Protected Methods in CommonParticipant and child Classes

This Pull Request introduces the addition of DDSPIPE_PARTICIPANTS_DllAPI to the protected methods in the CommonParticipant and SimpleParticipant classes. By exporting these methods, we ensure they are properly exposed when building shared libraries, allowing child classes of external modules to access these functionalities.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 37.86%. Comparing base (45a0234) to head (50ad5cd).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   36.53%   37.86%   +1.33%
==========================================
  Files         169      152      -17
  Lines       11542     6956    -4586
  Branches     5277     2764    -2513
==========================================
- Hits         4217     2634    -1583
+ Misses       4580     2937    -1643
+ Partials     2745     1385    -1360

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


Attributes are ignored because the component is a fragment instance

Hi there, in my vuex / vue-router spa on a fresh install with just the basic example i'm getting:
vue.common.js?e881:1014 [Vue warn]: Attributes ":selected.sync", ":options" are ignored on component <multiselect> because the component is a fragment instance: http://vuejs.org/guide/components.html#Fragment-Instance
Would really like to use it in my project any insight on this?

How do you import the component?
It should be like this:
import { Multiselect } from 'vue-multiselect'
If you omit the { } it will throw the above error. Hope it works! 😃

Yes correct ! a small misstep by me ! Works as expected with the curly braces.
Thanks for the quick replay and for the awesome project !

Happy I could help! 👍

Using this component with Laravel Spark at the moment, I'm getting also those error, with the brackets added.
import {Multiselect} from 'vue-multiselect'

Vue.component('multiselect', {
	components: { Multiselect },

	data: function() {
        return {
        	selected: null,
        	options: ['list', 'of', 'options']
        };
    }
});

Using it with this: <multiselect :options="options" :selected.sync="selected"></multiselect>
Now I really dont understand fragments instance, I just can't wrap my head around the concept, but might I get this error because I call it from within an another component (think <component-one><multiselect></multiselect></component-one>)?

I’m not sure that the above construction is actually valid. Because you try to register another component named 'multiselect' with Multiselect as a child component.
If you need to register the multiselect component globally please do it as in the Vue docs.
// Globally register the component with tag: my-component
Vue.component('my-component', MyComponent)

If you intend on wrapping vue-multiselect inside another component, please try the following code:
import {Multiselect} from 'vue-multiselect'

Vue.component('customselect', {
    components: { Multiselect },
    template: '
<multiselect :selected="selected" :options="options"></multiselect>
',
    data: function() {
        return {
            selected: null,
            options: ['list', 'of', 'options']
        };
    }
});

You could try omiting the additionall
 inside the template string. Then you could run the component with <customselect></customselect>.
Let me know if this helps! :)

I'll be damned, it works by adding the template to the component ;-)
Thank for that!
(And wow, do I don't get fragment instance! haha)

Haha :D I’m glad to hear that!
Just wondering, does it work if you remove the

 from the template?

It does!

Oh, wait, @shentao, I take back what I said. Without the div, it works, it doesn't throw fragment instances errors, but Vue-devtools still flag the component as fragment. Putting back the div will make things perfect.

May be the fault is  globally use like this:
<multiselect></multiselect>
components: {
     'multiselect': VueMultiselect
}

but the correct use is like this:
<multiselect></multiselect>
components: {
     'multiselect': VueMultiselect.Multiselect
}

@theseawolves This is true if you import it globally, otherwise
import { Multiselect } from 'vue-multiselect' is correct.

@shentao
Hello, I have the same problem and I've tried everything suggested here, still no solution. I tried registering multiselect as global component, used it as a child component to my custom component, tried to use it in .vue component file, tried to use it in custom component markup (using inline-template directive). Tried it with both of these combinations for importing:
import { Multiselect } from 'vue-multiselect' and import Multiselect from 'vue-multiselect'
This is my custom <tag-popover component (exported in tag-popover.vue file).
This is the template part:
<template>

    <p v-if="timelog.tags.length < 1" class="tags">Add tags...

        <template v-for="tag in timelog.tags | limitBy 2">
          @{{ tag.name }}
        </template>

Popover works

        <multiselect @tag="addTag" @select="selectTag" @remove="removeTag"
                     :close-on-select="false" :taggable="true"
                     :multiple="true" label="name" key="id"
                     :selected="timelog.tags" :options="tags">
        </multiselect>

</template>

This is the script part:
import PopoverMixin from './popover-mixin'
import { Multiselect } from 'vue-multiselect'

export default {
  props: ['tags', 'timelog', 'showPopup'],
  mixins: [PopoverMixin],
  components: {
    Multiselect
  },
  methods: {
    // ...some methods
  }
}

This is the console error I'm getting [Vue warn]: Attributes ":close-on-select", ":taggable", ":multiple", "label", "key", ":selected", ":options" are ignored on component <multiselect> because the component is a fragment instance: http://vuejs.org/guide/components.html#Fragment-Instance.
Here is apps hierarchy in Vue Devtools:

One more thing, project is based on Laravel Spark.
I'm really stuck with this and I would like to use vue-multiselect cause it's awesome.


It is not possible to connect to multiple devies at the same time

I want to connect to several devices at the same time. This is currently not possible. The reason is the following function of pygatt:
    def start(self, reset_on_start=True, initialization_timeout=3):
        """
        Run gatttool to prepare for sending commands and monitoring the CLI tool
        output.

        :param bool reset_on_start: Perhaps due to a bug in gatttol or pygatt,
            but if the bluez backend isn't restarted, it can sometimes lock up
            the computer when trying to make a connection to HCI device.

When this function is called with reset_on_start=True all previous connections made by all other adapters are invalidated.
It is currently called in this repository like this:
def connect(MAC):
    try:
        adapter = pygatt.GATTToolBackend()
        adapter.start()
        device = adapter.connect(MAC)

That means it is impossible to connect to two or more devices, becaus the call to connect will always invalidate the previous one.
An easy way to make multiple devies at the same time possible would be to add reset_on_start as keyword parameter to connect and pass it through to pygatt.GATTToolBackend.start.
This would allow users to create adapters and connect to devices without restarting the backend everytime.

Tested and working just fine. I did not think of it since I had only one device to test at the time.


DAGNodes stored with v1 CIDs are returned with v0 CIDs

This test fails (add to the suite in test/ipld-dag-pb.js):
it('should return a v1 CID for a DAGNode created with a v1 CID', (done) => {
  dagPB.DAGNode.create(Buffer.from([0, 1, 2, 3]), (err, node) => {
    expect(err).to.not.exist()

    const cid = new CID(node.multihash).toV1()

    resolver.put(node, {
      cid,
      format: cid.codec
    }, (error, nodeCid) => {
      expect(error).to.not.exist()
      expect(nodeCid instanceof CID).to.be.true()
      expect(nodeCid).to.deep.equal(cid)

      resolver.get(nodeCid, (error, result) => {
        expect(error).to.not.exist()
        expect(result.value).to.deep.equal(node)

        done()
      })
    })
  })
})

If you remove the toV1() conversion it passes.

This test passes for me if I add a const CID = require('cids').

Though it certainly returns a returns a CID v0. What would you expect? That js-ipld always returns a v1 CID?

If I specify a cid as an option to resolver.put(node, { cid }, cb), I would expect the returned cid to equal the cid I specify, unless node is not addressable via that cid in which case an error should be passed to the callback.
If I then call resolver.get(cid, cb) I would expect the returned node to have the same cid that I used to get it.
So if I use a v1 CID with resolver.put, the returned CID should be the same v1 CID, and I should be able to then use it with resolver.get to retrieve the node I previously passed to resolver.put.
Similarly if I used a v0 CID, I should get a v0 CID back.  I should not use a v1 CID then have it downgraded to a v0 CID by some part of the IPLD internals.
Not sure why adding the require would affect the test.

Really I'm not sure why I can pass a CID in as an option at all, it should just retrieve or calculate it from the passed node with a cidVersion, codec etc options, but that's another discussion.

Thanks a lot for the detailed reply. I think it totally makes sense and I think the IPLD layer should handle that properly.

@achingbrain I had another look. As I said the test doesn't fail for me and seems to do what I expect. I'll try to make it clearer with commenting the code:
it('should return a v1 CID for a DAGNode created with a v1 CID', (done) => {
  dagPB.DAGNode.create(Buffer.from([0, 1, 2, 3]), (err, node) => {
    // `node` is a `DagNode` object, internally it has a CIDv0.
    // That sounds fine to me.
    expect(err).to.not.exist()

    const cid = new CID(node.multihash).toV1()

    resolver.put(node, {
      cid,
      format: cid.codec
    }, (error, nodeCid) => {
      // `nodeCid` is a CIDv1. So it is as expect, you put the node with
      // a CIDv1 and result is also a CIDv1.
      // BTW, if you pass in a CIDv0, the result will be a CIDv0.
      expect(error).to.not.exist()
      expect(nodeCid instanceof CID).to.be.true()
      expect(nodeCid).to.deep.equal(cid)

      resolver.get(nodeCid, (error, result) => {
        // `result.value` contains a `DagNode` object.
        // In there the CID is a CIDv0. I think that is fine as you
        // don't have public access to the CID anyway.
        expect(error).to.not.exist()
        expect(result.value).to.deep.equal(node)

        done()
      })
    })
  })
})

Should any of the behaviour be changed?

DAGNodes don't have .cid/.multihash properties any more so this is not a problem any more.


Helm release disappears - operator unable to uninstall release - "Release not found"

Bug Report
This issue happens with a Helm operator. We have been seeing it recently, which makes us think of a regression from either operator-sdk 1.33.0 or GKE 1.27.
What did you do?

Create a custom resource as input to the operator with ArgoCD
Wait for the operator to install the Helm release
At this point, the helm release is visible and the corresponding secret present in the namespace.
Delete the resource using ArgoCD

What did you expect to see?

The Helm release should not disappear until all resources have been removed.
After the CR has been removed, the operator should be able to properly remove the resources

What did you see instead? Under which circumstances?

The Helm release disappears (helm list and kubectl get secrets both stop showing the release)
The operator displays the following:

{"level":"info","ts":"2024-01-26T13:18:58Z","logger":"helm.controller","msg":"Release not found","namespace":"namespaces","name":"zoom-s001","apiVersion":"charts.symphony.com/v1alpha1","kind":"ExtendedNamespace","release":"zoom-s001"}
{"level":"info","ts":"2024-01-26T13:18:58Z","logger":"helm.controller","msg":"Removing finalizer","namespace":"namespaces","name":"zoom-s001","apiVersion":"charts.symphony.com/v1alpha1","kind":"ExtendedNamespace","release":"zoom-s001"}

The resources deployed by the chart are still there, even though the CR has been fully deleted
At this point we need to manually remove the leftovers.

Environment
Operator type:

Kubernetes cluster type:
GKE
$ operator-sdk version
1.33.0
$ go version (if language is Go)
1.21
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.10", GitCommit:"0fa26aea1d5c21516b0d96fea95a77d8d429912e", GitTreeState:"clean", BuildDate:"2024-01-17T13:46:28Z", GoVersion:"go1.20.13", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v5.0.1
Server Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.7-gke.1121000", GitCommit:"4daab1fd78c0b9aba478a19b363ab4a25bdadd79", GitTreeState:"clean", BuildDate:"2023-11-06T09:24:38Z", GoVersion:"go1.20.10 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}

Possible Solution
As a way to mitigate the impact: don't remove the finalizer in this case.
Additional context
Upgrade to GKE 1.27 (from 1.26) was done recently

I found a workaround using the uninstall-wait annotation described here: https://sdk.operatorframework.io/docs/building-operators/helm/reference/advanced_features/annotations/#helmsdkoperatorframeworkiouninstall-wait
This helps mitigate the impact, although the issue probably will remain.

Could you post what your CRs look like, both the operand and the ArgoCD one? It it created with finalizers on the dependent resources or owner references? It's a bit hard to follow what exactly is happening from the information you've posted.

Hey we're going to close this due to inactivity. If you're still experiencing this issue please reopen it with the requested information so we can take a look at it.
Thanks!


Adding ability to publish MQTT topics and messages

Having the ability to remotely monitor cpu temps and fan speeds on my Home Assistant instance would be pretty nice.  I also use a MacBook air that runs OctoPrint for my 3d printer that i'd also like to monitor through Home assistant. Both computers are run headless.  The Home Assistant app is on my phone and which can alert me to deviations in normal operating of those computers.
MQTT is a lightweight, easy to setup messaging protocol that can even be installed on and run from small microcontrollers.  Setting it up consists of providing the client (Fanny in this case) with server/broker info such as address, port, username and password.  Periodically, Fanny could publish the stats it's received to the MQTT broker, Home Assistant in my case, which can be logged, graphed and viewed in the app.
Maybe this can be integrated into the Fanny widget as an option to turn on for those who might find it useful?

So I've run across this project written in swift.  https://github.com/emqx/CocoaMQTT
If my programming skill weren't nearly non-existent, I'd try myself.


feat: improve tooltips and i18n for frontend

Fix: https://github.com/casdoor/casdoor/issues/1566

@makorne plz review

@makorne plz review

Week ago I asked there how to use casdoor
I showed screenshots - casdoor in official docker example makes errors and does not work with any custom policy...
Nobody answered how to fix.
Nobody checked this error and resolved.
There is no docker example with custom policy!
Why waste time on tooltips for a non-working application?

@makorne do you still have issues or questions? Please raise issue for that.
Docker has nothing to do with Casbin policy. Casdoor can be deployed natively or via Docker. Casbin policy can be stored in another database, which will be accessed by Casdoor via DB client. You can view Casdoor as only a web UI to manage your original existing Casbin policy. The Casbin policy is NOT sitting inside the Docker.

My Casdoor deployed  via Docker.
But it  web UI cant manage policy because no example what  need for this.
Why not make docker example for this?

@makorne that feature is for Casbin user: https://casbin.org/ . It's not related to Docker

You want to say the casbin cant be installed in docker?
Or casdoor  Adapters web UI is useless because no example how to make it works?

@hsluoyz
So it is very easy!
Just create single  "casbin_rule" table in Mysql docker example.
But currently  casdoor Adapters web UI is useless .


feat(ha-target-switchover): add component level bdd for switchover

Exposes del_volume_shutdown_target in rest.
Adds changes in common nvme module to handle nvme connection if ana is not enabled.
Adds the component level bdd tests for target switchover.

bors try

Looks good to me

bors try

bors merge


Windows jmod exe's/dll's are not signed

Should jmod exe's/dll's be signed?
Windows JDK Temurin builds have the jdk/bin exe's/dll's signed, but not the jmod exe's/dll's.
This means jlink built JRE's have unsigned exe's/dll's

@tellison what's your thought on this please?

As a comparison, Mac jmod dylib's are signed.

@tellison @gdams So i've done a check of various vendors, and the answer is mixed as to whether sign jmod exe's:

Temurin jdk-20 : No
Red Hat jdk-17 : Yes
Oracle jdk-20 : Yes
Zulu jdk-20 : No

Fixed


Disable Validator URL?

Is there a way to disable the validatorUrl without creating a custom UI view?  I'd like to disable it because all my servers are internally routed.
Package version
1.4.1
Node.js and npm version
20.2

Please see


refactor!: return model through an object to ease evolutions

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Description
To ease future evolution, we're updating the read hooks return values to return an object systematically.
Adapters will need to be updated when published.
Checklist:

[x] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

Actually it's a really small one, but just in case I forgot something obvious I let you do a quick review 🙏

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0707fdf). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   97.35%
=======================================
  Files           ?       10
  Lines           ?     1170
  Branches        ?       47
=======================================
  Hits            ?     1139
  Misses          ?       31
  Partials        ?        0

Continue to review full report at Codecov.

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

Looks good to merge! 👍


[FEATURE] Shortcut for "\" symbol in gather environment

Description of the Problem
Gather environment doesn't have any shortcuts.
Description of the Solution
Would be great if pressing Enter inside gather environment would insert "\\" symbol and move cursor to a new line, like in align environment.

I was confused as to why this wasn't working, when I realized that I just hadn't enabled gather in the list of matrix shortcuts. Maybe it could be there by default?


Setup commit log on project page.

we want a svelte component to consume this json and render a commit log.
{
  "node": "5448b83228f3eb8d2e957d80952486a0fb5dd574",
  "changeset_count": 7,
  "changesets": [
    {
      "node": "5448b83228f3eb8d2e957d80952486a0fb5dd574",
      "date": [1680232402.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": ["tip"],
      "user": "kevin",
      "phase": "draft",
      "parents": ["da3bf544c23e97d8f02a12b914ee552ae66cfedd"]
    },
    {
      "node": "da3bf544c23e97d8f02a12b914ee552ae66cfedd",
      "date": [1678769458.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": ["6c33cafcd951d14cd3febac6afe21d8c6c4ca519"]
    },
    {
      "node": "6c33cafcd951d14cd3febac6afe21d8c6c4ca519",
      "date": [1678768459.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": ["28a55dc2fbb7965f8384bcf641001234a174e708"]
    },
    {
      "node": "28a55dc2fbb7965f8384bcf641001234a174e708",
      "date": [1678687688.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": ["e54421a3ac60ba1af03741f4521bd848a28ac4d6"]
    },
    {
      "node": "e54421a3ac60ba1af03741f4521bd848a28ac4d6",
      "date": [1678675375.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": ["fb853b6ed66fda5f0aa09d7dc500e1a723d1d6a6"]
    },
    {
      "node": "fb853b6ed66fda5f0aa09d7dc500e1a723d1d6a6",
      "date": [1677473583.0, -25200],
      "desc": "[FLExBridge: 3.8.0-beta.8+Branch.develop.Sha.47d93f0a941a523a1190f2ebae58e118556c69f1] sync",
      "branch": "7500002.7000072",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": ["1af18777e23eadd69d4f9bddc9df202c64be804c"]
    },
    {
      "node": "1af18777e23eadd69d4f9bddc9df202c64be804c",
      "date": [1677473542.0, -25200],
      "desc": " Add FLExProject.CustomProperties",
      "branch": "default",
      "bookmarks": [],
      "tags": [],
      "user": "kevin",
      "phase": "draft",
      "parents": []
    }
  ]
}

the date is with a unix timestamp and an offset in seconds. Take a look at this C# code to see how I parsed it into a date object.

Here's a first pass at a log viewer, showing a train-tracks graph on the left and a log message on the right. It's still unstyled and doesn't have dates or usernames yet, but this demo at least shows how the train-tracks graph is adjusting automatically (and correctly) to the size of the log messages on the right, despite being in separate DOM elements.


Support conversion from JSON to TOML?

Is your feature request related to a problem? Please describe.
I could get the JSON string from TOML table by json_formatter, but how can I read it again from JSON file and parse to TOML table. Please help me if i did not notice that on the document, thanks.
Describe the solution you'd like
JSON string or file should be supported by parse function?
Additional context

There's no built-in way to do this. The json_formatter exists for two reasons:

Despite being an arguably poor config format, JSON makes for a pretty good over-the-wire/storage serialization format (and indeed is used very commonly for those purposes), so going from TOML to JSON is a reasonable use-case
toml++ already has the TOML parser and data-types so implementing the json_formatter was super easy, barely an inconvenience

For me to go in the other direction (i.e. from JSON to TOML) I'd need to implement and a complete JSON parser (plus all it's weird offshoots and dialects), which isn't something I'm willing to maintain. Fortunately I don't need to, though; C++ is spoiled for choice when it comes to JSON libs:

https://github.com/nlohmann/json
https://github.com/Tencent/rapidjson/
https://github.com/open-source-parsers/jsoncpp
probably quite a few more floating around
So if you need this functionality you could pair toml++ with your JSON lib of choice and implement it yourself relatively easily.


stream_edit: Enable notification settings for muted streams.

This PR includes the following changes:

Allow the user to interact with notification settings for muted streams.
Add "In muted streams, stream notification settings apply only to unmuted topics." line of text in stream settings and decreases the space between "Notification Settings" heading and text.
Append "In muted streams, stream notification settings apply only to unmuted topics." line of text in notification settings.

Fixes: #27272

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Add "In muted streams, stream notification settings apply only to unmuted topics." line of text in stream settings along with some styling to match the theme.

We don't need special styling here. We'll be dropping the other note about muted streams in any case (in a separate issue).

Thanks for the review @sahil839 , I have updated the commit message according to the guidelines.
Currently the text is always displayed as it wasn't specified in the issue to display it only when the stream is muted. If you want it to display only when stream is muted, please let me know.

I think it's fine to always show the info about the muted streams, yeah.

Has the visual feedback from @sahil839 and me been addressed? If so, please update the text and screenshot in the PR description accordingly.

I have updated the text and screenshot in the PR as per the changes made. Kindly review it and let me know if further adjustment is required.

@Pratik2026 The commit messages do not follow the guidelines, please read them carefully and then update the messages accordingly. There should be a blank line between commit message summary and description and there should be around 68-70 characters in one line and not more than that. You can do git log in your terminal to see the existing commit messages.
Also, please fix the commit history to remove unnecessary commits.

Hi @timabbott and @sahil839 , just wanted to know if there is any additional feedback for this PR?

@sayamsamal can you also review this, since it's coupled with #27295?

@timabbott I rebased this PR on top of #27295, and during my testing it worked as intended 👍🏻

Merged, thanks @Pratik2026 and @sayamsamal and @sahil839!


CB-11232 Appium tests: fixed element tapping on iOS 9

https://issues.apache.org/jira/browse/CB-11232

Cordova CI Build has one or more failures.
Dashboard - http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera/1/
Results

Windows 8.1 Store - Console Output - Test Results - Build Artifacts
Windows 10  Store - Console Output - Test Results - Build Artifacts
Windows 8.1 Phone - Console Output - Test Results - Build Artifacts
iOS - Console Output - Test Results - Build Artifacts
Android Mac - Console Output - Test Results - Build Artifacts
Android Windows - Console Output - Test Results - Build Artifacts

Restarted the CI check. It passed this time.

Cordova CI Build has one or more failures.
Dashboard - http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera/2/
Results

Windows 8.1 Store - Console Output - Test Results - Device Logs
Windows 10  Store - Console Output - Test Results - Device Logs
Windows 8.1 Phone - Console Output - Test Results - Device Logs
iOS - Console Output - Test Results - Build Artifacts
Android Mac - Console Output - Test Results - Build Artifacts
Android Windows - Console Output - Test Results - Device Logs

Let there be tests

Cordova CI Build has completed successfully.
Dashboard - http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera/7/
Results

Windows 8.1 Store - Console Output - Test Results - Device Logs
Windows 10  Store - Console Output - Test Results - Device Logs
Windows 8.1 Phone - Console Output - Test Results - Device Logs
iOS - Console Output - Test Results - Build Artifacts
Android Mac - Console Output - Test Results - Build Artifacts
Android Windows - Console Output - Test Results - Device Logs

Let there be tests

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android Mac
Link
Link
Link

Android Windows
Link
Link
Link

LGTM


set floatingSearch menu visibility

you can set the visibility or set menu, with java and not using XML?
Thank you for the reply

As of now this is not supported.
Best,
Ari


Milestone 3. UI design & CSS

What's to do

[ ] "a set of static pages styled with CSS"
[x] the app is hosted somewhere (https://anki-webapp.vercel.app/)

CSS libraries (e.g. Bootstrap, Tailwind) are prohibited.
Note: the task is basically to recreate the Figma design of the pages in code, using HTML and CSS. The logic is not to be yet implemented, just the general appearance of the pages is. Basically the task is to implement everything up to the back-end interaction.
HTML

The semantic to be somewhat rich for learning purposes.

CSS
In general:

media queries
dark theme

Here are the animations that would be implemented:

deck "engaged" on hover and "disengaged" when hovering is stopped, to indicate interactability
"hover pagey thing" to appear and disappear when choosing a deck ("fade in" & "fade out" ??)
buttons "sinking in and out" and changing colors when activated and deactivated
general "fade ins" & "fade outs" when changing between modes
...

opened march 20th
ok, gotta wrap it up by sunday

ok, so I suppose only responsive design left to achieve here as well as a couple of minor issues to fix and we're good to go


SPStorkController is not VoiceOver-friendly

The close button has no accessibility label
The chevron isn't accessibility-enabled

Hi! Maybe you want add it and create pull request?

Ah yes, the "send a patch" OSS maintenance strategy. :-) I use it myself for most of my projects.
There's value in simply capturing issues. I'll consider working on it, but for now I just changed my app to use the default modal presentation if VoiceOver is enabled.


Add "Open Files here..." shell context menu integration

Describe the solution
Add a switch in settings that adds (and removes) shell context menu item "Open Files here..." that opens Files in current directory. It's meant for Windows File Explorer, hide it for Files.
Describe alternatives you've considered
Can't think of any
Additional comment
RX Explorer already injects its menu item into the shell context menu but there isn't a way to disable it (?)

I believe this already exists, if you don't see it it's probably a bug.

We don't plan to implement this option in Files, but as @winston-de mentioned, you can accomplish this by modifying the registry.


[Core]: Detached message handling

Describe your changes
Introduced 3 classes that will remove the the dependency on the CANNetworkManager for all classes that want to receive / send messages. namely:

CANMessagingProvider: a simple interface that provides the "send can message" function, currently this will always be the CANNetworkManager, but it also allows us to exchange it easily, e.g. if we want some unit test without the network manager.
CANMessagingConsumer: a base class to extend from. It provides the send message function and allows for handling tx/rx messages. I.e. any class that wants to interact with can messages can extend from this one and have everything ready.
CANMessageHandler: connects the CANMessagingProvider with CANMessagingConsumer's. And it's "process tx/rx message" function can be called to automatically distribute a particular message across consumers.

I think this will be a better alternative than passing the network manager around everywhere when we are refactoring away from the singleton. And it might especially be useful for unit testing, where we can isolate a class we wish to test and thereby remove interference from others. Though I'd like to hear any thoughts on this, as it does surely add an extra layer off complexity to the stack.
I see 3 ways a consumer can be set-up;

We can pas the CANMessageHandler to the consumer class and let it register itself, i.e.:

void initialize(CANMessageHandler messageHandler)
{
	// This class must extend enabled_shared_from_this, unlikely and unintuitive...
	messageHandler.add_consumer(std::shared_from_this());

	// If we decide to allow raw-pointers, but note that also
	// introduces the risk of forgetting the removal on cleanup
	messageHandler.add_consumer(this);
}

We add an extra function to the CANNetworkManager that sets up the messaging for the passed in consumer:

void CANNetworkManager::initialize_messaging(std::shared_ptr<CANMessagingConsumer> consumer)
{
	messageHandler.add_consumer(consumer);
}

Since the above is really simple, it might instead make more sense to just let the end user/application call that function of the message handler, i.e.:

auto interfaceUnderTest = std::make_shared<ShortcutButtonInterface>(internalECU, true);
CANNetworkManager::CANNetwork.get_can_message_handler().add_consumer(interfaceUnderTest);

I think I prefer the last one. To start off, I switched over the ShortcutButtonInterface and HeartbeatInterface to this messaging system so we can see what it's like.
What's next
Once we find and agree on an approach, all the other classes that send/receive messages have to be adopted to this method.
Furthermore, I'd like some unit-tests that will be run first in line to make sure the system remains correctly in place and it doesn't cause other tests to fail.

I think we do keep getting further from the ability to ever really make the stack static allocation friendly, which is maybe fine? I know some levels of functional safety forbid any dynamic memory - which would be pretty painful for us to deal with, so maybe it's not worth worrying about especially with nobody explicitly asking for it or sponsoring that kind of thing?

Yeah I know... Even though it would have been super nice to also be able to support those kind of strict rule sets, I think static allocation is out of our scope at the moment. Especially since we have always worked with the dynamic allocation and expanded on that. The shared pointers already help a lot to manage the lifecycle of those dynamically allocated objects. Converting the whole stack to be static allocation friendly I imagine will complicate so many things, it would be almost a rewrite...

Yeah... but I do think we should worry about our current stuff and releasing 1.0 before we think too hard about it the static allocation stuff I think (if ever) I was mostly just thinking out loud 😊


Read an environment variable to define --fuzz-seed default value

Component
Forge
Describe the feature you would like
Hi!
It would be handy to be able to tweak --fuzz-seed via an environment variable, for example FOUNDRY_FUZZ_SEED (matching foundry's environment variable naming convention)
Additional context
No response

this should work by default
in fact
FOUNDRY_FUZZ_SEED=100 forge config
gives
fuzz_seed = '0x64'

Checked and confirmed, sorry for the inconvenience. I think I concluded this was not working because of gas reports being non-deterministic when fuzz seed is constant (cf. #2571)


Expose SDK setExternalAudioSource methods

Is your feature request related to a problem? Please describe.
Agora SDK allows for external audio source selection (ie external mic) but the flutter package does not expose it. https://docs.agora.io/en/Interactive Broadcast/custom_audio_android?platform=Android
Describe the solution you'd like
I'd like either to have access to that part of the SDK from the flutter code, or a workaround to use an external mic.
Additional context
There was a previous PR but it's closed with no further context. https://github.com/AgoraIO/Flutter-SDK/pull/142

you should not export setExternalAudioSource method to the dart layer. you can implement it on the native layer, and just export some methods such as enableCustomAudioCapture.

Thanks @LichKing-2234 for answering!
I'm not very experienced on the native layer, that's one of the reasons we chose dart / flutter for our project.
I'm a bit confused: is there a reason to not be able to control audio sources from the dart layer? I'd say that the flutter SDK should enable all potential capabilities from Agora, ideally - it's just a matter of extending the package. I'd love to send a PR but I don't think I'm experienced enought. I'm curious what could be the reasons to think otherwise.
If that should be done natively, can you please point me to the right direction? Is there some documentation I could use?
Thanks again!

you can refer to https://docs.agora.io/en/Voice/custom_audio_android?platform=Android

Hi @LichKing-2234 this requirement popped up again in our priority list.
Our users would like to be able to use their external microphones on their mobile devices while using our App (flutter app). Re-reading your response, I don't understand the limitation of flutter on the external audio source

Is it just not implemented and you would consider accepting a PR?
Or is it something you don't want to support in flutter for some reason? In this case, would that force us to drop the flutter dependency and reimplement this part of the App natively in Android / iOS, or is there another solution to support external sources with flutter?

Thanks!

Hi @LichKing-2234 this requirement popped up again in our priority list.
Our users would like to be able to use their external microphones on their mobile devices while using our App (flutter app). Re-reading your response, I don't understand the limitation of flutter on the external audio source

Is it just not implemented and you would consider accepting a PR?
Or is it something you don't want to support in flutter for some reason? In this case, would that force us to drop the flutter dependency and reimplement this part of the App natively in Android / iOS, or is there another solution to support external sources with flutter?

Thanks!

this is a customized requirement, I think it may be different with different customers. if it can be designed for all of the customers, then I'd like to accept the PR.
custom audio/video source needs the developer to implement the Java/OC code. I suggest you fork our repository and implement your requirement on your repository which you have forked.

this is a customized requirement, I think it may be different with different customers. if it can be designed for all of the customers, then I'd like to accept the PR.
custom audio/video source needs the developer to implement the Java/OC code. I suggest you fork our repository and implement your requirement on your repository which you have forked.

Thanks! I assumed external mic should be something straight forward, I now understand it's not.
Looking at the available forks, I see @truyenjc has some strong developments in this line. Would you be so kind to share if they are production ready? Would it make sense for @LichKing-2234 to add some of those improvements to the base project in a PR?
I'd be glad to collaborate in anything I can, testing or fixing things.
Thanks again for a nice product!

Thanks! I assumed external mic should be something straight forward, I now understand it's not.
Looking at the available forks, I see @truyenjc has some strong developments in this line. Would you be so kind to share if they are production ready? Would it make sense for @LichKing-2234 to add some of those improvements to the base project in a PR?
I'd be glad to collaborate in anything I can, testing or fixing things.
Thanks again for a nice product!

@dgilperez sorry for late reply. we will discuss internally how we will support this and maybe provide roadmap. but this will not come up very soon due to our schedule.

@dgilperez sorry for late reply. we will discuss internally how we will support this and maybe provide roadmap. but this will not come up very soon due to our schedule.

Hi @plutoless! just pinging to check the status for this. Again, I'd be glad to beta test any improvement you have in this line.

Ping @plutoless ! Could you provide an update, please?

@plutoless @LichKing-2234 This feature is also very important for our project.
when could we expect it ?
thanks.

you can refer to our native example and add it on the native layer.

Support it.


[REVIEW] Portfolio

Portfolio link
https://kendalldoescoding.gq/

Closing as the user is banned due to abusive language


Require a c++ example or documents about tensorflow lite useage

because i have my own native and c++ code, if i want to use tensorflow lite feature and put it to my android project, i have to write tensorflow lite code in c++.
but it is too hard to find out how to get the input and output.
i have look up the ios example because part of the code is written in c++, but it crashed when i have:
float* output = interpreter->typed_output_tensor(0);
for(int i = 0; i < output_size; i++)
LOGD("bkzero jni: , result: %f", output[i]);
i believe it is not a bug under iOS platform.
this part of code in android example is written in java, i cannot figure out how to write an implemetation of the tensorhandle like:
long[] outputsHandles =
run(interpreterHandle, errorHandle, sizes, dataTypes, numsOfBytes, inputs);
if (outputsHandles == null || outputsHandles.length == 0) {
throw new IllegalStateException("Interpreter has no outputs.");
}
Tensor[] outputs = new Tensor[outputsHandles.length];
for (int i = 0; i < outputsHandles.length; ++i) {
outputs[i] = Tensor.fromHandle(outputsHandles[i]);
Log.i("bkzero: ", "java: outputs[0] " + outputs[i]);
}

You can get output tensor index list by calling interpreter->outputs() after calling to interpreter->Invoke().
And you can access the output tensors with the index by calling interpreter->tensor(index).


Add integration test for belongsTo update form

Issue #, if available:
Description of changes:
This PR adds a belongsTo relationship field to update form integ test
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #801 (1dbd595) into tagged-release/cms-support (f00cf16) will not change coverage.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##           tagged-release/cms-support     #801   +/-   ##
===========================================================
  Coverage                       92.26%   92.26%
===========================================================
  Files                             115      115
  Lines                            4511     4511
  Branches                         1265     1265
===========================================================
  Hits                             4162     4162
  Misses                            328      328
  Partials                           21       21

Continue to review full report at Codecov.

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


how do you enable i2c interface for GPIO?

Hi, I have latest OMV / stretch (armhf) setup on my rock64. Working great. Thanks ayufan and other contributors.
I'd like to install a RTC on my board. It attaches to the GPIO and uses i2c.
Is this possible? Any suggestions appreciated.

Yes, just install. It is already supported.

That's great. I will try it and see if I can get it working.

I don't understand. It looks like there is a RTC already on my rock64 board - I haven't installed the RTC module.
May 31 09:03:04 rock64 kernel: [    2.263456] rk808-rtc rk808-rtc: rtc core: registered rk808-rtc as rtc0
and
May 31 09:03:04 rock64 kernel: [    3.225131] rk808-rtc rk808-rtc: setting system clock to 2016-01-21 08:50:11 UTC<PHONE_NUMBER>)
I think this is telling me there is a RTC embedded on the rock64 board. Is this possible?

@jata1, yes, it's possible. Rock64 uses RK805 power management chip with built-in RTC functionality. But, unfortunately, I failed to find a pin that can be used to power RTC while the rest of the board is not powered in the datasheet


feat: deferred de/serialisation for transactions

Follow up to #1356, introduces the deferred de/serialisation of unconfirmed transactions.

@ljedrz can you please give this a look when you get a chance?

@apruden2008 since we currently don't expect transactions to be propagated in the network, this PR is not a priority for merging right now; it's definitely desirable to have a bit later, though.

@niklaslong I think this PR is ready for rebasing, as some APIs have changed w.r.t. #1447


Issue with children as props in interactive island components

I am trying to figure out how to support this, and will try to setup the scenario.
We have a shared component library with some of our companies reusable deno building blocks.  Specifically in there, we have a control to help show items or hide them behind a popup based on screen size.  We call this our ResponsiveSet:
https://github.com/fathym-deno/atomic/blob/integration/src/molecules/ResponsiveSet.tsx
Its heavy lifting is done by the MenuButton control which hides its children behind a button click:
https://github.com/fathym-deno/atomic/blob/integration/src/molecules/MenuButton.tsx
You'll see the {props.toggleChildren} is rendered inside an Action to wire up the onclick to show the menu.
In order to make this control usable we have to wrap the ResponsiveSet in an island component to make the MenuButton interactivity work. We do this in our Fresh project with the InteractiveResponsiveSet:
https://github.com/o-biotech/biotech-manager-web/blob/integration/islands/molecules/InteractiveResponsiveSet.tsx
This works and the whole control works, the only issue is that the toggleChildren won't render the icon we are trying to get inside the action:
https://github.com/o-biotech/biotech-manager-web/blob/7d075f0b62c001d219fad0d8233fe3e71b0e19e6/components/organisms/BiotechHeader.tsx
There seems to be something going on when registering it with the InteractiveResponsiveSet, if you take the BiotechHeader and instead use the ResponsiveSet itself, the MenuIcon renders (but interactivity is lost).
In our latest integration branch, you can see that if we set the triggerChildren to a simple toggleChildren="☰" value, it is passed through just fine:
https://github.com/o-biotech/biotech-manager-web/blob/integration/components/organisms/BiotechHeader.tsx
Does this have to do with issues in passing props between server and island components?  Is there a better way i should be registering Islands in the component library (i know i'd like a better way than the wrapper).
Thanks for the help, let me know how else i can help.

When I upgraded to Fresh 1.4, I am finding that children get set to undefined when rendered. I moved the same value to a property and everything worked as before.

@kitsonk do you have a link to a GitHub repo where I can reduce the problem?

This commit of the site is what should be able to reproduce it:
https://github.com/o-biotech/biotech-manager-web/blob/7d075f0b62c001d219fad0d8233fe3e71b0e19e6/
Specifically in the components/organisms/BiotechHeader.tsx

@marvinhagemeister this is what I had to do to fix it as part of the upgrade: https://github.com/ts-why/tswhy/commit/5eb95c84311e7d232d764fdbe7d76e052572f95a#diff-40d9bbe97e321d3a88ebe909c1bb3a7b3f2b383aa00dcd5192e28a2fb4a1b67e
I will try to later on to create a branch with that change backed out.

@marvinhagemeister this branch replicates the issue: https://github.com/ts-why/tswhy/tree/children_islands
If you clone, deno task start and navigate to: http://localhost:8000/edit?code=1002 and look at the console, you see the island is complaining about an object being undefined.  If you move the data to a property (revert the last commit in that branch: https://github.com/ts-why/tswhy/commit/6bfb6c9a813a8e9d32ecbe082177823a088ccade) it will work.
I never updated this code base to 1.3 so I am not sure if it was introduced there or 1.4.

Thanks for the bbranch @kitsonk. When I run your code I get:
diagnostics.ts:38 Uncaught (in promise) TypeError: Cannot destructure property 'title' of 'object null' as it is null.
    at diagnosticDataToMarkdown (diagnostics.ts:38:5)
    at T.Editor [as constructor] (Editor.tsx:42:14)
    at T.ce [as render] (preact.mjs:2:8524)
    at O (preact.mjs:2:6232)
    at X (preact.mjs:2:2148)
    at O (preact.mjs:2:6446)
    at fe (preact.mjs:2:8639)
    at _render (main.ts:258:15)

I've created an even more minimal reproduction here. When I run this, I get:
childrenProps.tsx:18 Uncaught (in promise) TypeError: Cannot destructure property 'title' of 'object null' as it is null.
    at helper (childrenProps.tsx:18:26)
    at T.Bug [as constructor] (Bug.tsx:4:16)
    at T.ce [as render] (preact.mjs:2:8524)
    at O (preact.mjs:2:6232)
    at X (preact.mjs:2:2148)
    at O (preact.mjs:2:6446)
    at fe (preact.mjs:2:8639)
    at _render (main.ts:258:15)

So I think it's the exact same case. I've just cut out as much code as possible to get to the issue.
If I have time I'll continue debugging, but I just wanted to share this here as a starting point for anyone else.

Yes, it appears that for whatever reason when the island is hydrated on the client side, it's children value is undefined, but when it is passed as a prop, it hydrates fine. Now theory could be that fresh is only dealing with vdom nodes for children elements when hydrating (though previous versions seemed to deal with it fine).

I think i've created a similar issue in how i am trying to use our new atomic_icons library in a basic fresh starter. I have created a repo that contains a reproduction of the issue here:
https://github.com/mcgear/deno-fresh-icons-in-island
To setup the repo, used the following command to setup fresh (make sure the folder is empty):
npx fathym dev deno scaffold
Choose the fresh option
Once that is setup, you have to add the atomic-icons library as shown in the docs for the fresh plugin here:
<EMAIL_ADDRESS>Then you need to configure the plugin as shown:
<EMAIL_ADDRESS>Now:
deno task start
The issue arises in that the Counter buttons no longer work and the console throws an error:
Uncaught ReferenceError: Deno is not defined
at build_id.ts:3:22
(anonymous) @ build_id.ts:3
If you go into the Counter.tsx file and comment out the ExclaimIcon at line 14, everything goes back to working just fine.
We have some other controls that seem to maybe be impacted by similar issues.

@mcgear I'm currently looking into this, but I'm unable to run the reproduction case. It seems like something is wrong with the icons. Things I tried:

Running deno task start after fresh clone of the repo gives me this

error: Uncaught (in promise) TypeError: Module not found "file:///Users/marvinh/dev/test/biotech-manager-web/build/iconset/icons/_exports.ts".
    at file:///Users/marvinh/dev/test/biotech-manager-web/routes/index.tsx:13:8
    await import(entrypoint);
    ^
    at async dev (https://deno.land/x/fresh@1.4.2/src/dev/dev_command.ts:40:5)
    at async file:///Users/marvinh/dev/test/biotech-manager-web/dev.ts:5:1

So I assumed maybe one has to run the icons task first, so I did that, but that gives me this error:

Task icons deno run -A ./scripts/icons.atomic.ts
error: Module not found "file:///Users/marvinh/dev/test/biotech-manager-web/iconset.config.ts".
    at file:///Users/marvinh/dev/test/biotech-manager-web/scripts/icons.atomic.ts:2:34

Then I assumed ok maybe the fathym-atomic-icons.config.ts is supposed to be the config, so I copied the contents to iconset.config.ts. But this lead to this error

error: Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'IconMap')
  const map = Object.keys(this.config.IconMap).map(
                                      ^
  at IconSet.ToSheet (https://deno.land/x/fathym_atomic_icons@v0.0.3-integration1/src/sprites/IconSet.tsx:194:41)
  at useIconSet (https://deno.land/x/fathym_atomic_icons@v0.0.3-integration1/src/sprites/IconSet.tsx:19:33)
  at useFileIconSet (https://deno.land/x/fathym_atomic_icons@v0.0.3-integration1/src/sprites/IconSet.tsx:28:9)
  at file:///Users/marvinh/dev/test/biotech-manager-web/scripts/icons.atomic.ts:4:7
  at eventLoopTick (ext:core/01_core.js:183:11)

What do I need to do to get the reproduction case to run?

@kitsonk passing JSON as children in JSX for islands is not supported. The children prop for islands must be a JSX element. For serializing non-JSX data passing it as a prop is the way to go. Whilst it technically works in Preact to pass random data as children as long as the receiving component can work with that, it's a bit of an anti-pattern since it breaks the compositional nature of components.
Regardless, it's something where we should show a nice error message instead of silently failing. Made https://github.com/denoland/fresh/pull/1736 to address that.

@marvinhagemeister I just re-cloned and did not run into the issue that you had, everything just worked.
Then i noticed that you were trying the repro from my initial post, which would require cloning that specific commit.  I made a simpler repro that seems to be working if you try it:
https://github.com/mcgear/deno-fresh-icons-in-island
Typically we don't checkin the build directory for our projects, and force the icons to be generated the first time.  The issue we have here on first build, is that if you are using the icons the build fails, and the generator code never fully executes.
In the latest repro (https://github.com/mcgear/deno-fresh-icons-in-island) i just checked in the build dir to ensure you didn't run into this issue.

@mcgear thanks for providing a repro. With that I was able to reproduce the issue on my end.
The build errors in https://github.com/mcgear/deno-fresh-icons-in-island are caused by the importing the src.deps.ts file. That file imports a bunch of dependencies that cannot be bundled for the browser. svgo for example requires node's os module which doesn't exist in the browser. Same is true for https://github.com/fathym-deno/atomic-icons . There is a src.deps.ts file as well which contains imports that cannot run in the browser. This includes Fresh's asset() function which currently cannot run in the browser either. But now seeing that I'm kinda wondering if we should make it work in the browser.
In either way, we should print a better error message when server-only code is being loaded in an island in Fresh. The real issue though is that the Deno registry has no tooling or guardrails in place to catch that during publishing. I'll forward that issue to the relevant folks working on the registry as this is definitely a frustrating experience. We should do better here.
The workaround for now is to have two separate deps.ts files: One for server stuff, and one that imports the dependencies for the Preact components. It might make sense to have an additional top level entry browser.ts next to mod.ts and then import only that in your Fresh islands.

awesome @marvinhagemeister, thank you for the help on this.  That all makes sense.

I should have also mentioned a +1 for asset() working in the browser too

Closing this as the original error has been resolved.


Add option to turn off Format on Paste

Along with https://github.com/dotnet/razor-tooling/issues/4336, once https://github.com/dotnet/razor-tooling/issues/6212 is done we should add an option to turn off formatting on paste.
Currently this is, as far as I know, completely done on the platform side, so not sure if we already have a facility to tell them not to format, but we should investigate and request something if not.
See related user feedback here: https://github.com/dotnet/razor-tooling/issues/6212#issuecomment-1290019797

It makes more sense (IMO) for the client to implement these settings and simply not issue the requests.  Format on Paste especially, because it is indistinguishable from a Range Formatting request.  Such is the model that existed for language services registered through pkgdef, which included being able to enable/disable completion, quickinfo (hover), and a few other features.  For LSP, it may make sense to expand the set of language features for which the client exposes an option, and the registration of those options hopefully can be modernized or streamlined for LSP...

100% agree

Format on paste is the main reason I have to hit Ctrl + Z hundreds of times per day. Doing a line copy and paste formats both the copied line and pasted line:

@olegtk @kayle @joseparrams FYI on this since platform controls the Format on Paste in LSP (but also see my comment above about modernizing all the other features previously supported via Language Service registration in pkgdef, since those are also client-initiated features in LSP).  This also relates to our internal discussion of consolidating and simplifying settings for LSP vs other VS Editor extensibility.

Tracking as internal ask with AB#1803701

So this just keeps getting kicked down the road? Format on Paste is so frustrating, it forces you to wait up to a second before the format occurs just so you can hit undo. The only workaround is to quickly type a character which prevents the automatic format.

Any update on this, over a year later? This bug is the main reason I hate Visual Studio.

Unfortunately this fix can only be done by the Visual Studio editor team, so all we can do is ask and wait. It is one of our highest priority requests from them however.

8 GB of updates today and Paste still sucks in Razor.

17.9 has come and gone, we can only hope something will be done in 17.10.

@michaelcsikos if you want to comment in a way that is visible to the team that needs to fix this issue, then https://developercommunity.visualstudio.com/t/CSHTML-File-Format-Problem/10398165

OK. So, that bug was reported 8 months after this thread and it has 2 votes. The search for the Developer Community is abysmal, but that's a different issue.

@michaelcsikos if you want to comment in a way that is visible to the team that needs to fix this issue, then https://developercommunity.visualstudio.com/t/CSHTML-File-Format-Problem/10398165 is a much better place.

And now it's Closed - Lower Priority.

Following up with the VS team. Likely got caught in some automation.


fix: deploy scripts

two fixes:

import correct deploy WITH verify
pass tokenSweepBeneficiary to staking-pool-deploy-no-etherscan staking pool factory constructor (was originally initializer)

Purpose for this PR

@CjHare


Implement EventsBySliceQuery in PersistenceTestKitReadJournal

Related to https://github.com/apache/pekko/issues/1513, reasoning same as https://github.com/apache/pekko/issues/1527 - it would be useful if testkit had this capability.
TimestampOffset could be implemented here, because only real persistence layer implementation that uses this that I am aware of - r2dbc - does use TimestampOffset, but I am not sure if it is worth the hassle.

@ptrdom would you be able to resolve the conflicts when you have time?

Yes, of course!


Share code between local and remote terminal reconnection

This code is very similar:
https://github.com/microsoft/vscode/blob/fea4ff2130f9a2dc676c7da1ce3bab1b001f7405/src/vs/workbench/contrib/terminal/browser/terminalService.ts#L397-L426
It could be shared. Also, I don't think we currently reconnect to local terminals in remote workspaces, we probably should?

This actually wouldn't save much code at all and might make it harder to read in the end.


Thumbnails with remote media

This is an implementation of the fix described here: https://groups.google.com/forum/#!msg/mezzanine-users/WGVaNhD5vRs/7Ktw7-UkosgJ
Rather than using os functions to create thumbnails on the filesystem, create the thumbnail in memory and push it to the media storage using default_storage. This fixes the thumbnail tag when using remote media storage.
I have tested this using remote storage (S3), and local storage. I would appreciate a second set of eyes on the fix.

Closing this, as I am creating a new PR for an additional feature to aid in using S3, which depends on these changes and so will also incorporate them.


add seding Pinata creds if they exist

If the user has Pinata credentials, respond to the initialization message with those credentials instead of generating a new one.

done in #14


fix: can't update dubbo plugin registry config (#1308)

// Describe your PR here; eg. Fixes #issueNo
fix #1308 and add test case
Make sure that:

[√] You have read the contribution guidelines.
[√] You submit test cases (unit or integration tests) that back your changes.
[√] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1320 (f9dfd33) into master (28e63b1) will decrease coverage by 0.76%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #1320      +/-   ##
============================================
- Coverage     69.04%   68.28%   -0.77%
+ Complexity     2085     2079       -6
============================================
  Files           391      391
  Lines          8273     8289      +16
  Branches        827      833       +6
============================================
- Hits           5712     5660      -52
- Misses         2141     2211      +70
+ Partials        420      418       -2

Impacted Files
Coverage Δ
Complexity Δ

...in/alibaba/dubbo/cache/ApplicationConfigCache.java
35.95% <60.00%> (-1.09%)
10.00 <0.00> (+1.00)
:arrow_down:

...gin/apache/dubbo/cache/ApplicationConfigCache.java
75.28% <73.33%> (-2.50%)
21.00 <0.00> (+2.00)
:arrow_down:

...soul/client/tars/TarsServiceBeanPostProcessor.java
23.17% <0.00%> (-59.76%)
5.00% <0.00%> (-9.00%)

...omara/soul/sync/data/http/HttpSyncDataService.java
63.72% <0.00%> (-7.85%)
12.00% <0.00%> (ø%)

.../soul/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
10.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


Jiahui/connect data to charts

Name of the PR
:recycle: Current situation & Problem
Describe the current situation (if possible, with an exemplary (or real) code snippet and/or where this is used)
Please link any open issue that is addressed with this PR
:bulb: Proposed solution
Describe the solution and how this affects the project and internal structure
:gear: Release Notes
Add a summary of the feature and possible migration guides if this is a breaking change so this section can be added to the release notes.
Include code snippets that provide examples of the feature implemented if it appends or changes the public interface.
:heavy_plus_sign: Additional Information
Provide some additional information if possible
Related PRs
Reference the related PRs
Testing
Are there tests included? If yes, which situations are tested, and which corner cases are missing?
Reviewer Nudging
Where should the reviewer start? Where is a good entry point?
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[ ] I agree to follow the Code of Conduct and Contributing Guidelines.

I just want to confirm that with this PR, all surveys will now have corresponding charts appear when the data card is clicked right? There seemed to be no WIQ data card before so want to make sure.


FixImproved path handling with normalization, encoding, and error management.

Path Normalization: Added normalize() to standardize the input path, removing redundant elements to enhance security and consistency.
Path Separator Handling: Addressed Windows path separators by converting them to Unix style to ensure consistent URL formatting.
URL Encoding: Implemented URL encoding for the path to safely handle special characters and avoid issues in the URL format.

run buildall


Not working for (UE55)RU8000 series

Is your feature request related to a problem? Please describe.
I can add the TV, on/off not reporting correctly (WOL works only via smarthing, not with this integration)
Describe the solution you'd like
Support for the RU8000 series. WiFi WOL works for this TV, some sort of smartthings API thing?

What does it means "WOL works only via smarthing, not with this integration"?

It means that I can wake my TV using the smartthings app (even on wifi). But it won't work with any available integration. I also don't get the allow control popup it has to show.

My 7000 series doesn't turn on from smartthings but does from this addon.
I removed HA from external device manager on the TV, deleted the token, could not get it to reconnect.
I resolved the issue by removing the integration completely from HA and adding back in.
All is working correctly that way.

My 7000 series doesn't turn on from smartthings but does from this addon.
I removed HA from external device manager on the TV, deleted the token, could not get it to reconnect.
I resolved the issue by removing the integration completely from HA and adding back in.
All is working correctly that way.

Tried that, doesn't work. It doesn't popup the allow connection screen. It shows that my TV is already on but the integration doesn't let me control it

Did you check setting related to external devices?
On TV go to "menu -> general -> external device settings -> device connection" and in "Access notification" set "Only first time"

Did you check setting related to external devices?
On TV go to "menu -> general -> external device settings -> device connection" and in "Access notification" set "Only first time"

Yeah it's enabled, still no pop-up :(

Can you provide more deatil about your HA installation? Version, type, guest OS etc...

arch
x86_64

chassis
vm

dev
false

docker
true

docker_version
19.03.8

hassio
true

host_os
HassOS 4.11

installation_type
Home Assistant OS

os_name
Linux

os_version
5.4.50

python_version
3.7.7

supervisor
228

timezone
Europe/Amsterdam

version
0.112.4

virtualenv
false

Networking:
Lan (vlan0): <IP_ADDRESS> <-- hassio is here
Wlan (vlan25): <IP_ADDRESS>  <-- tv is here
IOT (vlan50): <IP_ADDRESS> <-- shelly, hue, blabla

Docker / different subnet not work with ssl component. Check issue #5.

Docker / different subnet not work with ssl component. Check issue #5.

I made a reservation in the correct vlan to trial and error. I'll post the results tomorrow

Unfortunately it doesn't work :(

You should provide more details, as the HA log with the error and details about your configuration.

You should provide more details, as the HA log with the error and details about your configuration.

Which details of my setup do you need? I did post my system information. To add my tv I just go to the integrations menu and add: samsung tv smart (yours). It then gives me: unknown error occurred. Which log do you need? I can't find errors inside the supervisor, core, dns, etc. Logs also not in the main log under logs. Doesn't even come up.

Enable HA debug log adding this line in configuration.yaml:
logger:
  default: info
  logs:
    custom_components.samsungtv_smart: debug

Are you totally sure that HA and the TV are in the same subnet?

Enable HA debug log adding this line in configuration.yaml:
logger:
  default: info
  logs:
    custom_components.samsungtv_smart: debug

Are you totally sure that HA and the TV are in the same subnet?

not the same subnet anymore. But everything in my wlan or iot vlans can reach each other.
2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart] Try config with port: 8001 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] WS url ws://<IP_ADDRESS>:8001/api/v2/channels/samsung.remote.control?name=W0hvbWUgQXNzaXN0YW50XSB0dnNhbQ%3D%3D 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] Connection closed. 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] Connection closed. 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart] Failing config with port: 8001, error: {'event': 'ms.channel.unauthorized'} 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart] Try config with port: 8002 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] WS url wss://<IP_ADDRESS>:8002/api/v2/channels/samsung.remote.control?name=W0hvbWUgQXNzaXN0YW50XSB0dnNhbQ%3D%3D 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] Connection closed. 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart.api.samsungws] Connection closed. 2020-07-20 20:14:15 DEBUG (SyncWorker_3) [custom_components.samsungtv_smart] Failing config with port: 8002, error: {'event': 'ms.channel.timeOut'}
The problem is that I don't see that pop-up altho I did set it up in my config


URL loading for Psych

Made this alternative because our current hscript support doesn't allow haxe.Http#request();
Usage:

for loading an image, do: var spr:FlxSprite = Paths.loadGraphicFromURL(url:String);. This WILL override a sprites properties, as this returns a brand new FlxSprite with the loaded image.
for loading a Sparrow spritesheet, do: spr.frames = Paths.loadSparrowAtlasFromURL(xmlUrl:String, imageUrl:String);. This will return the Sparrow Atlas frames, ready with the image.
for loading a files text data off the web, do: var str:String = Paths.loadFileFromURL(url:String);. This will return a string of the text data found from the url.

I have also added the possibility to load lua scripts straight from strings rather than files, this allows people to load a lua script straight from the web using Paths.loadFileFromURL();

what da dog doin

what da dog doin

if i didn't do that i wouldn't be able to load the fuckin scripts from the web now could i?

what da dog doin

if i didn't do that i wouldn't be able to load the fuckin scripts from the web now could i?

yeah i just saw lmao i didnt fully read the pr yet when i was posting this comment

kinda dope, but can you list off a few use cases for this? i don't really see it being used much aside from like storing data in a db

well the obvious saving disk space in a mod folder, or using image data such as height for something like the Pokemon Mystery Gifts and other timed events, also allows people to updates sprites and code for their mods.