Bug fix in displaying public IP address of shared networks

Description
Public IP address dedicated to one domain should not be accessed
by other domains. Also root admin should be able to display all
public ip address in system.
Currently following issues exist

Public IP address assigned to one domain can be accessed by
other sibling domains

If use.system.public.ip is false then child domains should not
see public ip of ROOT domain
Before fix
(test1) mgt01 > list publicipaddresses listall=true fordisplay=true allocatedonly=false forvirtualnetwork=true filter=ipaddress,
{
  "count": 59,
  "publicipaddress": [

After fix
(test) mgt01 > list publicipaddresses listall=true fordisplay=true allocatedonly=false forvirtualnetwork=true filter=ipaddress,
{
  "count": 10,

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[X] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@ravening conflict here. Should this go in <IP_ADDRESS> as it is a bug fix?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2717

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3556)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 34467 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4675-t3556-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Smoke tests completed. 85 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_public_ip_admin_account
Failure
32.31
test_network.py

test_public_ip_user_account
Failure
2.29
test_network.py

test_releaseIP
Failure
663.16
test_network.py

@ravening can you look at those failures ^^?

@ravening can you look at those failures ^^?

@DaanHoogland will look into it

There's also a conflict @ravening

@shwstppr can you help resolve conflict and then kick tests. Thanks.

@rhtyd I've fixed the conflicts but test failures from previous run doesn't seem intermittent and would need investigation/changes cc @ravening
2021-02-16 21:39:22,806 - CRITICAL - FAILED: test_public_ip_admin_account: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 236, in test_public_ip_admin_account\n    self.fail("list public ip response is not empty")\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 450, in fail\n    raise self.failureException(msg)\n', 'AssertionError: list public ip response is not empty\n']
2021-02-16 21:39:25,098 - CRITICAL - FAILED: test_public_ip_user_account: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 285, in test_public_ip_user_account\n    "Check if disassociated IP Address is no longer available"\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 553, in assertEqual\n    assertion_func(first, second, msg=msg)\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 546, in _baseAssertEqual\n    raise self.failureException(msg)\n', 'AssertionError: Check if disassociated IP Address is no longer available\n']
2021-02-16 21:56:14,340 - CRITICAL - FAILED: test_releaseIP: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 894, in test_releaseIP\n    "Failed to disassociate IP address")\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 462, in assertTrue\n    raise self.failureException(msg)\n', 'AssertionError: Failed to disassociate IP address\n']

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: :heavy_check_mark: centos7 :heavy_multiplication_x: centos8 :heavy_check_mark: debian. SL-JID 87

@rhtyd I've fixed the conflicts but test failures from previous run doesn't seem intermittent and would need investigation/changes cc @ravening
2021-02-16 21:39:22,806 - CRITICAL - FAILED: test_public_ip_admin_account: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 236, in test_public_ip_admin_account\n    self.fail("list public ip response is not empty")\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 450, in fail\n    raise self.failureException(msg)\n', 'AssertionError: list public ip response is not empty\n']
2021-02-16 21:39:25,098 - CRITICAL - FAILED: test_public_ip_user_account: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 285, in test_public_ip_user_account\n    "Check if disassociated IP Address is no longer available"\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 553, in assertEqual\n    assertion_func(first, second, msg=msg)\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 546, in _baseAssertEqual\n    raise self.failureException(msg)\n', 'AssertionError: Check if disassociated IP Address is no longer available\n']
2021-02-16 21:56:14,340 - CRITICAL - FAILED: test_releaseIP: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 369, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_network.py", line 894, in test_releaseIP\n    "Failed to disassociate IP address")\n', '  File "/usr/lib64/python2.7/unittest/case.py", line 462, in assertTrue\n    raise self.failureException(msg)\n', 'AssertionError: Failed to disassociate IP address\n']

@shwstppr I have pushed a commit. can you retrigger the tests to see if that fixes?

okay @ravening
@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 90

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests [S]

[S] Trillian test result (tid-90)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37102 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4675-t90-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_network.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_releaseIP
Failure
668.16
test_network.py

test_01_migrate_VM_and_root_volume
Error
76.34
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
51.23
test_vm_life_cycle.py

@Spaceman1984 can you help manually test/verify this? cc @shwstppr Thanks.

@ravening still some issue

@ravening still some issue

@shwstppr but I dont think the issue is with ip management.

@Spaceman1984 can you help manually test/verify this? cc @shwstppr Thanks.

I'll test @rhtyd

@ravening still some issue

@shwstppr found the issue.. without the change, if ip address is in free state, its not displayed by list publicipaddress command but with this change, it will be displayed. so need to pass an extra param state=Allocated. Then its not displayed

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Packaging result: :heavy_multiplication_x: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 168

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_multiplication_x: centos7 :heavy_multiplication_x: centos8 :heavy_multiplication_x: debian. SL-JID 222

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_multiplication_x: centos7 :heavy_multiplication_x: centos8 :heavy_multiplication_x: debian. SL-JID 230

@blueorangutan package

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 233

@blueorangutan test

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-253)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40948 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4675-t253-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_deploy_kubernetes_cluster
Failure
3602.73
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
71.83
test_kubernetes_clusters.py

test_01_migrate_VM_and_root_volume
Error
67.20
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
51.12
test_vm_life_cycle.py

Trillian test result (tid-259)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35585 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4675-t259-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 84 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestKubernetesCluster>:teardown
Error
70.88
test_kubernetes_clusters.py

test_01_migrate_VM_and_root_volume
Error
67.17
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
50.06
test_vm_life_cycle.py

Ping @ravening @shwstppr @Spaceman1984 what's the update on this? Thanks

Manually tested, fixes #4723


No wheel for 0.3.0 release

Just noticed that 0.3.0 is built from source, PyPI doesn't have the associated wheel present whilst 0.2.0 does, any reason for this? Thanks!

Nice catch! I'm not sure why that happened. Version 0.4.0 should be coming in the couple of weeks.

With version 0.4.2 being released, manylinux are now being built for Python 3.6 and Python 3.7 :)


The wallets does not show up in the bottom sheet and show grey screen instead

Describe the bug
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: RangeError (end): Invalid value: Not in inclusive range 0..2: 4. Error thrown building Widget.
       at ListBase.getRange(dart:collection)
       at _WalletsListShortPageState.build.<fn>(wallets_list_short_page.dart:58)
       at _ExplorerServiceItemsListenerState.build.<fn>.<fn>(explorer_service_items_listener.dart:43)
       at _ValueListenableBuilderState.build(value_listenable_builder.dart:186)
       at StatefulElement.build(framework.dart:5409)
       at ComponentElement.performRebuild(framework.dart:5297)
       at StatefulElement.performRebuild(framework.dart:5462)
       at Element.rebuild(framework.dart:5016)
       at BuildOwner.buildScope(framework.dart:2779)
       at WidgetsBinding.drawFrame(binding.dart:916)
       at RendererBinding._handlePersistentFrameCallback(binding.dart:360)
       at SchedulerBinding._invokeFrameCallback(binding.dart:1297)
       at SchedulerBinding.handleDrawFrame(binding.dart:1227)
       at SchedulerBinding._handleDrawFrame(binding.dart:1085)

To Reproduce
Steps to reproduce the behavior:

Initialize WalletConnect SDK as Getit service'
Launch the app
See error

Expected behavior
No error to be seen of such kind
Screenshots

Smartphone (please complete the following information):

Device: [OnePluse 8 Pro, Oppo F9, Xiaomi Mi A1]
OS: [ Android 13, Android 10, Android 11]
Browser [e.g. stock browser, safari]

Additional context
Add any other context about the problem here.

This issue is the same as includedWalletIds option causing an exception

Hello @shahji-mufin
That issue has been solved here https://github.com/WalletConnect/Web3ModalFlutter/pull/58
Are you using the latest version of the SDK?
Can I have a min reproducible code?

Hi @quetool, thanks for the quick response.
Yes, I saw the issue #58 but it is still coming up.
I am using web3modal_flutter: ^3.0.0-beta16 and now shifted to web3modal_flutter: ^3.0.0
I have made some changes and tested it further, if the issue still persists, I will comment here otherwise I will close the ticket.
The funny thing is no error came on Android 14 😅

Please switch to web3modal_flutter: ^3.0.0 version as it's stable now. Thanks!

The issue does not come up on version v3.0.0. The funny thing is no error came on Android 14 😅


Remove rush update

This PR
Updates the build and pack script, we shouldn't run "rush update" in CI/CD builds, rush update, updates/creates the shrinkwrap  lock file, rush install will install package dependancies based on the shrinkwrap, rush install will fail if the shrinkwrap is out of date, so we don't need to run rush update in CI/CD builds.

Looks like the rush update step still exists in the github workflow build, should that be removed?

Looks like the rush update step still exists in the github workflow build, should that be removed? Just wanted to make sure it's been tested without that step

@amerjusupovic , thanks for pointing this out, Yes update shouldn't be used in CI/CD builds, this has been updated thanks


[Posenet] Call PosenetDecoder in python

Hi Team,
I wonder that how i can call Posenet Decoder (https://github.com/google-coral/edgetpu/tree/master/src/cpp/posenet) in Python when Edge TPU API dont open any API related Posenet?

The decoder code wasn't intended to be called directly but only as part of a custom op. If you want to run it standalone you can simply use this tflite file:
https://github.com/google-coral/project-posenet/blob/master/models/customop_decoder_stub.tflite
And run it on CPU using the standard TFLite Python API (no edgetpu required).

I have try to run (https://github.com/google-coral/project-posenet/blob/master/models/customop_decoder_stub.tflite) using the standard TFLite Python API on CPU but it failed to allocated tensor.

Can you share your code and error message ? In theory this should work.

i share code first
`class PoseNet2:
""" Posenet  class"""
def __init__(self, recv_queue, send_queue, model_path, decoder_path, device, name):
    """ Construct function
        Args:
        Raises:
        ValueError: An error occurred when
     """
    #self._output_queue = queue.Queue(maxsize=20)
    #self._vs_inst = vs_inst
    self._recv_queue = recv_queue
    self._send_queue = send_queue
    self.engine = PoseEngine(model_path, device)
    self._thread = None
    self._process = None
    self._input_img_size = self.engine.get_input_size()
    self._name = name
    self._is_active = False

    self.interpreter = tflite.Interpreter(model_path=model_path, experimental_delegates=[
        load_delegate('libedgetpu.so.1.0', {"device": self.get_device_enumerated_from_path(device)})])
    self.interpreter.allocate_tensors()
    _, self.height, self.width, _ = self.interpreter.get_input_details()[0]['shape']

    # Get de-quantization factor
    self.float_heatmaps_dequantization = self.interpreter.get_output_details()[0]['quantization']
    self.float_short_offsets_dequantization = self.interpreter.get_output_details()[1]['quantization']
    self.float_mid_offsets_dequantization = self.interpreter.get_output_details()[2]['quantization']

    # Run on CPU
    self.interpreter_decoder = tflite.Interpreter(model_path=decoder_path)
    self.interpreter_decoder.allocate_tensors()

def start(self):
    """
        Args:
        Raises:
        ValueError: An error occurred when
    """
    self._thread = threading.Thread(target=self.job, args=())
    self._thread.start()
    #self._process = Process(target=self.job, args=())
    #self._process.start()

@staticmethod
def get_device_enumerated_from_path(device_path):
    if re.search('/sys/bus/usb/', device_path):
        return "usb"
    elif re.search('/dev/apex_0', device_path):
        return "pci"

@staticmethod
def preprocess(size, frame):
    """
        Args:
        Raises:
        ValueError: An error occurred when
    """
    preprocessed_frame = cv.resize(frame, size).astype(np.uint8)
    return preprocessed_frame

@staticmethod
def postprocess(frame, outputs, inf_time, threshold=0.4):
    """ Draw pose
        Args:
        Raises:
        ValueError: An error occurred when
    """

    inf_time_text = "inf_time=%.2f(ms)" % (inf_time)
    cv.putText(frame, inf_time_text, (10, 20), cv.FONT_HERSHEY_SIMPLEX, 0.5,
               (0, 0, 255), 1)

    for pose in outputs:
        if (pose.score < threshold):
            continue
        #print('\nPose Score: ', pose.score)
        for label, keypoint in pose.keypoints.items():
            # print(' %-20s x=%-4d y=%-4d score=%.1f' %
            #      (label, keypoint.yx[1], keypoint.yx[0], keypoint.score))
            if keypoint.score > threshold:
                x = keypoint.yx[1]
                y = keypoint.yx[0]
                #cv.putText(frame, "{}".format(label), (int(x), int(y)), cv.FONT_HERSHEY_SIMPLEX, fontScale=0.5,
                #           color=(0, 0, 255), thickness=2,
                #fa            lineType=cv.LINE_AA)
                cv.circle(frame, (int(x), int(y)), radius=2, color=(0, 255, 255), thickness=-1, lineType=cv.FILLED)

def ParseOutput(self):

    keypoints = output_tensor(self.interpreter_decoder, 0)
    keypoint_scores = output_tensor(self.interpreter_decoder, 1)
    pose_scores = output_tensor(self.interpreter_decoder, 2)
    nposes = output_tensor(self.interpreter_decoder, 3)

    assert nposes < keypoints.shape[0]

    # Convert the poses to a friendlier format of keypoints with associated
    # scores.
    poses = []
    for pose_i in range(nposes):
        keypoint_dict = {}
        for point_i, point in enumerate(keypoints[pose_i]):
            keypoint = Keypoint(KEYPOINTS[point_i], point,
                                keypoint_scores[pose_i, point_i])
            keypoint_dict[KEYPOINTS[point_i]] = keypoint
        poses.append(Pose(keypoint_dict, pose_scores[pose_i]))

    return poses

def decoder(self):
    """
        Args:
        Raises:
        ValueError: An error occurred when
    """

    # Dequantization
    float_heatmaps= self.float_heatmaps_dequantization[0] * (
            output_tensor(self.interpreter, 0).astype(np.float64) - self.float_heatmaps_dequantization[1])
    float_short_offsets = self.float_short_offsets_dequantization[0] * (
            output_tensor(self.interpreter, 1).astype(np.float64) - self.float_short_offsets_dequantization[1])
    float_mid_offsets_ = self.float_mid_offsets_dequantization[0] * (
            output_tensor(self.interpreter, 2).astype(np.float64) - self.float_mid_offsets_dequantization[1])

    # Decoder

    self.interpreter_decoder.set_tensor(self.interpreter.get_input_details()[0]["index"], float_heatmaps)
    self.interpreter_decoder.set_tensor(self.interpreter.get_input_details()[1]["index"], float_short_offsets)
    self.interpreter_decoder.set_tensor(self.interpreter.get_input_details()[2]["index"], float_mid_offsets_)

    self.interpreter_decoder.invoke()

    return self.ParseOutput()

def job(self):
    """
        Args:
        Raises:
        ValueError: An error occurred when
    """
    start_time = time.monotonic()
    self._is_active = True
    file_name_exection = 'process_pose.txt'
    if os.path.exists(file_name_exection):
        os.remove(file_name_exection)
    f_pose_exection = open(file_name_exection, "a")
    while self._is_active:
        if not self._recv_queue.is_empty():

                start_get = time.monotonic()
                frame = self._recv_queue.get()
                get_elapsed = time.monotonic() - start_get

                #poses, inference_time = self.engine.DetectPosesInImage(np.uint8(frame))
                input_tensor = np.reshape(frame, newshape=[1, self.height, self.width, 3], order='C')
                # self._lock.release()

                self.interpreter.set_tensor(self.interpreter.get_input_details()[0]["index"], input_tensor)
                # self.interpreter.set_tensor(self.interpreter.get_input_details()[0]["index"], frame)

                # self._lock.acquire()
                start = time.monotonic()
                self.interpreter.invoke()
                inference_time = time.monotonic() - start

                poses = self.decoder()

                start_anotate = time.monotonic()
                self.postprocess(frame, poses, inference_time)
                anotate_elapsed = time.monotonic() - start_anotate

                start_put = time.monotonic()
                self._send_queue.put(frame)
                put_elapsed = time.monotonic() - start_put

                end_time = time.monotonic()
                total_elapsed = end_time - start_time
                start_time = end_time

                line = '%.2f,%.2f,%.2f,%.2f,%.2f' % (
                get_elapsed * 1000, inference_time, anotate_elapsed * 1000, put_elapsed * 1000,total_elapsed*1000)
                f_pose_exection.writelines(line)
                f_pose_exection.write('\n')
                f_pose_exection.flush()

        else:
            time.sleep(0.001)

    print('Finished %s inferencing'%self._name)

def close(self):
    """
        Args:
        Raises:
        ValueError: An error occurred when
    """
    #self._is_active = False
    #self._vs_inst._is_active = False
    #self._vs_inst.close()

`

Hi @AlexTraan , can you try changing this:
self.interpreter = tflite.Interpreter(model_path=model_path, experimental_delegates=[
        load_delegate('libedgetpu.so.1.0', {"device": self.get_device_enumerated_from_path(device)})])

to this:
self.interpreter = tflite.Interpreter(model_path=model_path)

Hi Nam,
That part should run on Edge TPU.

Let me try this and see if i can reproduce the issue

@mtyka @Namburger I cannot reproduce error message right now, but i appear like below:
RuntimeError: Encountered unresolved custom op: xxx Node number 0  failed to prepare.

@AlexTraan Oh, you're right, sorry chiming in halfway, plus delegate should automatically detects cpu models. Monday morning...
On another note, that message:
RuntimeError: Encountered unresolved custom op: xxx Node number 0 failed to prepare.

usually means you have mismatching libedgetpu.so version.
The current pair should be libedgetpu.so version 13 and tflite_runtime-2.1.0.post1.

Ah, yes, you'll have to register the decoder custom op (this has nothing to do with the libedgetpu). Let me look into how to do that best.

Ok, so in https://github.com/google-coral/edgetpu/blob/master/src/cpp/posenet/BUILD
you build a posenet_decoder.so
Then you have to do something like:
self.interpreter_decoder = tflite.Interpreter(model_path=decoder_path,
experimental_delegates=[load_delegate('posenet_decoder.so')])
I think that should register the custom op.

@mtyka I follow up your guidance.
Then i faced other error: "RuntimeError: src/cpp/posenet/posenet_decoder_op.cc:106 heatmaps->type != kTfLiteUInt8 (1 != 3)Node number 1 (PosenetDelegateForCustomOp) failed to prepare."
This  (https://github.com/google-coral/project-posenet/blob/master/models/customop_decoder_stub.tflite)  require input tensor is float type, however seem this (https://github.com/google-coral/edgetpu/tree/master/src/cpp/posenet) require the input is uint8.
Can i need to de-quantize the out put from main posenet model (without decoder)?

The outputs of the plain, decoder-free posenet network are floats (i.e. dequantized) and you just have to feed them to the stand-alone decoder tflite which takes floats as inputs.

@mtyka
customop_decoder_stub.tflite input tensor shapre mismatch with decoder-free posenet shape, for example:

Decoder-free posenet output float_heatmaps  tensor shape = [1x31x41x17]
Decoder input float_heatmaps tensor shape = [1x16x21x17]

Therefore, i cannot run two part of network with each other.
Can you help?

@mtyka Input tensor for decoder operator is match, but output tensor of it not match as below:

(https://github.com/google-coral/project-posenet/blob/master/models/decoders/posenet_mobilenet_v1_075_481_641_decoder.tflite) decoder output tensor shape is:
id: poses:3
type: float32[1,20,17,2]
id: poses:1:4
type: float32[1,20,17]
id: poses:2:5
type: float32[1,20]
id: poses:3:6
type: float32

(https://github.com/google-coral/project-posenet/blob/master/models/posenet_mobilenet_v1_075_481_641_quant_decoder.tflite) posenet with decoder that have output tensor:
id: poses:36
type: float32[1,10,17,2]
quantization: -10 ≤ q ≤ 10
id: poses:1:37
type: float32[1,10,17]
quantization: -10 ≤ q ≤ 10
id: poses:2:38
type: float32[1,10]
quantization: -10 ≤ q ≤ 10
id: poses:3:39
type: float32
quantization: -10 ≤ q ≤ 10

As you see, it be mismatched. Can you continue to help me?

HI @mtyka ,
Do you have any comment on above point?

This should be fine. The dimension that's different is just the max number of poses that can be detected, which is now 20 instead of 10. The outputs dimensions of the decoder are:
tensor0: 1,max_poses,keypoints,2   (coordinates)
tensor1: 1,max_poses,keypoints  (keypoint_scores)
tensor2: 1,max_poses  (pose scores)
tensor3: 1.  (n_poses detected)

Hi @mtyka,
I can run base-model and decoder part but accuracy is bad (compared with your original model).
Currently, in my implementation, i dequantize output tensor get from base-model, then feed it up to decoder part. However, in (https://github.com/google-coral/edgetpu/blob/master/src/cpp/posenet/posenet_decoder_op.cc), it seem already included dequantize function. Can you help to check? How about Decoder with uint8 input?

@mtyka In my case, i think decoder part model should be uint8 tensor because this (https://github.com/google-coral/edgetpu/blob/master/src/cpp/posenet/posenet_decoder_op.cc) use uint8 as input for decoder. do you have any comments?

What is the range of values you see for each of the output tensors of the
base model after dequantization ?
On Wed, May 20, 2020 at 6:47 PM AlexTran<EMAIL_ADDRESS>wrote:

@mtyka https://github.com/mtyka In my case, i think decoder part model
should be uint8 tensor because this (
https://github.com/google-coral/edgetpu/blob/master/src/cpp/posenet/posenet_decoder_op.cc)
use uint8 as input for decoder. do you have any comments?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google-coral/edgetpu/issues/86#issuecomment-631829674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI4ZJNDENCGORQU5HJWDITRSSB4XANCNFSM4LWV44BA
.

--
[ www.miketyka.com  Twitter: @mtyka http://www.twitter.com/mtyka Instagram:
@miketyka https://www.instagram.com/miketyka/]

@mtyka
I use this file (https://raw.githubusercontent.com/google-coral/edgetpu/master/test_data/posenet/test_image.bmp)  as input.
I write the out (already flatten) of base model after dequantize to file in attachment.
You can change file format name from .txt to .csv in order to see in Office.
heatmaps.txtY
[Uploading mid_offsets.txt…]
short_offsets.txt
Can you help to check based on above log?

Something is wrong with either your dequantization or your image
preprocessing.
Your heatmap has min/max -10.0392160415649, -5.25490236282349
But it should be -10.039216* 8.313725*
Those are log likelihoods so if -5.25 is your most confident spot on the
heat map that explains why your pose results are bad.
Simple code to check:
import numpy as np
from PIL import Image
from edgetpu.basic.basic_engine import BasicEngine
pil_image = Image.open('couple.jpg')
pil_image = pil_image.resize((641, 481), Image.NEAREST)
model_path = 'models/posenet_mobilenet_v1_075_481_641_quant.tflite'
engine = BasicEngine(model_path)
print(model_path)
input_tensor_shape = engine.get_input_tensor_shape()
print("InputSizes: ", input_tensor_shape)
osize = engine.get_all_output_tensors_sizes()
print("OutputSizes:", osize)
inp = np.uint8(pil_image)
results = engine.run_inference(inp.flatten())
tensor0 = results[1][0:osize[0]]
tensor1 = results[1][osize[0]:osize[0]+osize[1]]
tensor2 = results[1][osize[0]+osize[1]:osize[0]+osize[1]+osize[2]]
print(np.min(tensor0), np.max(tensor0))
print(np.min(tensor1), np.max(tensor1))
print(np.min(tensor2), np.max(tensor2))
On Wed, May 20, 2020 at 9:18 PM AlexTran<EMAIL_ADDRESS>wrote:

@mtyka https://github.com/mtyka
I use this file (
https://raw.githubusercontent.com/google-coral/edgetpu/master/test_data/posenet/test_image.bmp)
as input.
I write the out (already flatten) of base model after dequantize to file
in attachment.
You can change file format name from .txt to .csv in order to see in
Office.
heatmaps.txt
https://github.com/google-coral/edgetpu/files/4660560/heatmaps.txtY
[Uploading mid_offsets.txt…]
short_offsets.txt
https://github.com/google-coral/edgetpu/files/4660557/short_offsets.txt
Can you help to check based on above log?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google-coral/edgetpu/issues/86#issuecomment-631869473,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI4ZJIRJRVKH5ICTQKVZRTRSSTRRANCNFSM4LWV44BA
.

--
[ www.miketyka.com  Twitter: @mtyka http://www.twitter.com/mtyka Instagram:
@miketyka https://www.instagram.com/miketyka/]

@mtyka It works now. The causes is  image preprocessing part.
Thanks you for your strong support.
BTW, i see you just updated Posenet based Resnet. I guess (not tried yet) the accuracy will increase, but speed will reduce. I wonder why you don't chose backbond net is
EfficientNet-EdgeTPU, it seem good tradeoff between accuracy and speed, it made from Google as well.


Bibliography data specific to most of the Google datasets.

This should be resolving PR #3: The bibitems are added verbatim to the corresponding about.html index pages.

Thanks!  @jtrmal please note!  Will make you owner.

amazing, thanks a lot, Alexander
On Tue, Jun 16, 2020 at 6:22 AM Daniel Povey<EMAIL_ADDRESS>wrote:

Thanks! @jtrmal https://github.com/jtrmal please note! Will make you
owner.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/danpovey/openslr/pull/4#issuecomment-644522538, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUKYX73OSCZ2IKGV5IOIMDRW3XRLANCNFSM4N6XPDXA
.


Fix flatten tiles and control points not refreshing

Two main fixes in this one:

The flatten tile wouldn't work with a negative selection width or height. A lot of other methods had if logic to handle, so I fixed at the source (where width and height are set) and then removed the logic in all of the other places. No need to remember to cut and paste that logic any more.
When selecting a new tile it would not always clear the control points from the old tiles. I added a control point clear on selection of a new tile, so now the control point will move to the newly selected tile right away (no need to re-click or toggle vertex mode or anything).

Added another fix and a feature.

Fixed entities being moved (to floor or ceiling) were still showing up in their old location until the next refresh.
Added a new feature that will center the entity within their current tile. Useful for those entities that are 1 tile wide that you want to get lined up just right.


Can w write the byte array directly into file target without converting it in to string

Hi!
If you have a question that the logging isn't working, check the troubleshooting guide first before posting.
Almost every case is solveble with the internal log
Are you searching for a layout or layout renderer? You could find them here
StackOverflow.com is recommended for questions. You could expect a faster response there (but do include all the info below) and the search is better.
Important details when asking question on StackOverflow.com:

Use the tag "nlog"
Include versions of NLog/NLog.AspNetCore etc.
Include the platform (.NET 4.5, ASP.NET Core 3, Xamarin etc)
Include the current config (xml or code)
Explain current result
Explain desired result
Please post full exception details (message, stacktrace, inner exceptions) in case of an error

I guess you could make your own custom FileTarget that inherits from NLog FileTarget, and then override this method:
        protected override void RenderFormattedMessageToStream(LogEventInfo logEvent, StringBuilder formatBuilder, char[] transformBuffer, MemoryStream streamTarget)
        {
            if (logEvent.Parameters?.Length == 1 && logEvent.Parameters[0] is byte[] logArray)
               streamTarget.Write(logArray, 0, logArray.Length);
            else
               base.RenderFormattedMessageToStream(logEvent, formatBuilder, transformBuffer, streamTarget);
        }

Then it will check if the NLog LogEventInfo has been initialized using a single byte[]-array.

Thanks for the prompt reply, the solution you suggested worked for me..


Issue Link

I am creating this pull request because I am unable to send an issue link and request a code review.
The issue link is this: https://github.com/fbclh/js-capstone/issues/1#issue-1161706906
This is a team of two people.

We have recreated the project inside the repository as per the last code reviewer request.
https://github.com/fbclh/js-capstone/projects/1
Thank you.

@iKuartz and @fbclh Could you fix the linter errors so I review your work, I have dropped you messages on Slack but you are not responding to them.


Looping the script

This is really not a issue since the program is working fine but I dont know other way to ask you this.
I have no clue how to program with node or anything .js related, but i was wondering if you could tell me how could i automate the script. For example "run node Index.js" everytime the script ends its execution, from what I saw I believe its just a few lines of code but could you point me to the right direction ?

If anyone would like to do the same just use Task-Scheduler that comes with windows. If you're on unix system just use crontab


feat: enhancing vcap parsing & removing configureService reference from BaseService ctor

ref: https://github.ibm.com/arf/planning-sdk-squad/issues/1098
The vcap changes were based on @padamstx initial implementation in the service-factory branch. Thanks Phil!
Changes:

removed the call to the configureService() method out of the constructor.
updated the unit tests that were dependent on configureService() being called when instantiating the BaseService.
updated how external configuration is loaded from a vcap file based on this design. Specifically, the parsing now checks for a matching service name within an entry, if it doesn't find any then it will return the first entry with a matching outer key to the service name.
other general cleanup of unused code

@padamstx - I've added some more unit tests that test an unconfigured and a configured service as you requested.

@jorge-ibm before you merge this PR in, could you please make the BaseService.configureService() method public instead of protected?   After our discussion yesterday afternoon, I think we're going to want to give the SDK user the ability to call it directly.

If this is going to be a major release, could we release a new minor version of the core before bringing this change in? There are some things that have been added since the 7.0.0 release that I'd like to bring into the Watson SDK.

If this is going to be a major release, could we release a new minor version of the core before bringing this change in? There are some things that have been added since the 7.0.0 release that I'd like to bring into the Watson SDK.

Yes, we could do that, but the next version of the generator (to be used for the watson sdk delivery next week) will need to be paired with this next major release of the java core.   So assuming that you use the to-be-delivered-soon new release of the generator to produce your next version of the watson java sdk, you'd need to use the new major version (8.0.0) of the java core for that.

I was going to make a patch release around the dependency cleanup PR I submitted and a bug fix in the SDK I'm going to be looking into today, so I'd rather get that out when it's ready as opposed to waiting for the next minor release.
Also just for clarity: I'm slightly confused by this comment of yours, but if I'm understanding correctly, it won't cause any issues for users of the Watson SDK? Trying to make sure that upgrading to the major version of the core for that release isn't problematic.

@jorge-ibm before you merge this PR in, could you please make the BaseService.configureService() method public instead of protected? After our discussion yesterday afternoon, I think we're going to want to give the SDK user the ability to call it directly.

sure. This makes sense. I've gone ahead and made that change

I was going to make a patch release around the dependency cleanup PR I submitted and a bug fix in the SDK I'm going to be looking into today, so I'd rather get that out when it's ready as opposed to waiting for the next minor release.
Also just for clarity: I'm slightly confused by this comment of yours, but if I'm understanding correctly, it won't cause any issues for users of the Watson SDK? Trying to make sure that upgrading to the major version of the core for that release isn't problematic.

Ok, yeah I can see why you're confused :)
This PR will in fact constitute a "breaking change" of sorts for an SDK user if they just simply updated their dependency to point to this new version while still using the existing generated code.  The reason for that is that we're removing the call to configureService() from the BaseService ctor.   That means the BaseService ctor will no longer trigger the loading and use of external config for service-level properties (URL, disable_ssl, etc.).   However, Jorge will be making a corresponding change the Java generator such that the generated service ctor will invoke configureService() instead of relying on the call to "super()" to do that.   This is part of our implementation of the service factory feature.  The design is here.
So, we don't want a user of the current Java SDK to unilaterally upgrade to this new version of the java core unless that is also coupled with new generated code from the about-to-be-delivered Java generator.   When we deliver the new generator release, we'll update the compatibility matrix to reflect the new major version of the java core.
Due to what you need, I'll go ahead and tag a new minor version (or patch... I'll look at the new commits there to determine which is needed), then update the compatibility chart for the current (2.1.0) generator to reflect this new version.

Okay, that all makes sense now @padamstx. Thanks for the clarification!


How to add matchedGeometryEffect to the components

Hello,
Firstly, I want to send special Thanks to the author :D unbelievable package 👯
I have a question, how to add matchedGeometryEffect to the components.
Thanks alots :D

@tsapeta please help. thanks


Set X-Miq-Group header for every request

Break down Session.create into two distinct setters because the auth
token and miq group values come from two separate HTTP requests.
The group value is returned as part of the user object when the
authorizations request is made.
https://www.pivotaltracker.com/story/show/140246987
https://bugzilla.redhat.com/show_bug.cgi?id=1422551
@miq-bot add_label euwe/yes, bug
/cc @chalettu
Separate follow-up PR in progress for backport.

Backported to Euwe via https://github.com/ManageIQ/manageiq-ui-service/pull/528


Hello. I would like to know if you are going to release a version for Swift 4.2

Hello. I would like to know if you are going to release a version for Swift 4.2

Yes, there’s a plan to support Swift 4.2

Great @adamcichy This is the last repository that I have in my project with missing support for Swift 4.2
glad to hear that

@adamcichy I was trying to convert to Swift 4.2. Can you explain what's the deal with extension AVAudioSession: Session Can you tell more how are you using that? Thanks! If you can help here I'm halfway to make a PR.

I think it's not possible because of https://forums.swift.org/t/using-methods-marked-unavailable-in-swift-4-2/14949
I don't think this library should introduce some wiggling with Objective-C.

@fassko That is the issue. I'd rather not introduce any Objective-C workarounds. There is also an option of using performSelector:... from Swift but that's also not perfect

@adamcichy I agree with you. That’s why I like his library as it’s Objective-C free. Let’s hope Swift team will fix it.
Sent with GitHawk

+1

Swift 4.2
#if os(iOS) || os(tvOS)
/// SoundCategory is a convenient wrapper for AVAudioSessions category constants.
public enum SoundCategory {
    /// Equivalent of AVAudioSessionCategoryAmbient.
    case ambient
    /// Equivalent of AVAudioSessionCategorySoloAmbient.
    case soloAmbient
    /// Equivalent of AVAudioSessionCategoryPlayback.
    case playback
    /// Equivalent of AVAudioSessionCategoryRecord.
    case record
    /// Equivalent of AVAudioSessionCategoryPlayAndRecord.
    case playAndRecord

    fileprivate var avFoundationCategory: String {
        get {
            switch self {
            case .ambient:
                return convertFromAVAudioSessionCategory(AVAudioSession.Category.ambient)
            case .soloAmbient:
                return convertFromAVAudioSessionCategory(AVAudioSession.Category.soloAmbient)
            case .playback:
                return convertFromAVAudioSessionCategory(AVAudioSession.Category.playback)
            case .record:
                return convertFromAVAudioSessionCategory(AVAudioSession.Category.record)
            case .playAndRecord:
                return convertFromAVAudioSessionCategory(AVAudioSession.Category.playAndRecord)
            }
        }
    }
}

#endif
fileprivate func convertFromAVAudioSessionCategory(_ input: AVAudioSession.Category) -> String {
return input.rawValue
}

SwiftySound has been updated to Swift 5 now

Some of us won't update to Mojave anytime soon. So it means we can't use this library

For Swift 4.2 pin carthage at version 1.0.0


Update docker compose files for tendermint

Description
After syncing with master, docker-compose with Tendermint integration was broken

BigchainDB does not use --init flag anymore
Either we use bigchaindb init explicitly or bigchaindb start --no-init
other wise bigchaindb start implicitly does an init

Issues This PR Fixes
Fixes #1923

Codecov Report

Merging #1924 into tendermint will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           tendermint    #1924   +/-   ##
===========================================
  Coverage       48.09%   48.09%
===========================================
  Files              70       70
  Lines            3352     3352
===========================================
  Hits             1612     1612
  Misses           1740     1740


The formatting is major off

It is a bit hard to play when it is like this...

Hello!
Can you please tell me what terminal you're using and also what font you have set up in it?

Hello!
Can you please tell me what terminal you're using and also what font you have set up in it?

I am using gotty and i dont know the font

Sounds like a font issue to me. Can you also tell me what operating system you're using and what's the exact version of your browser?

Sounds like a font issue to me. Can you also tell me what operating system you're using and what's the exact version of your browser?

I am using debian as the linux server
Chrome OS as the client
Browser is chrome 100.0.4896.133

This definitely looks like the default monospaced font from Chrome OS (Cousine) acts weirdly in Chrome. I was able to reproduce this in Chrome on both Windows and Chrome OS, but not with other monospaced fonts (like Fira Code or DejaVu Sans).
Weirdly enough, Cousine looks fine in my terminal.
This is not a bug within wordle-cli, and as such, there is nothing I can really do about it. What I can suggest, however, is try installing a different monospaced font (like the ones I mentioned) and change your default monospaced font to that in your Chrome settings.


Add support for Jarvis through Unav and make appID of notifications configurable

With recent release, Jarvis is not under help component in Unav. Added jarvis as a child component.
Make appID of unav's notifications component configurable
Unav has a isSectionDividerRequired feature which gnav needs to support. Global navigation component will take it in the config and pass it to unav config

Resolves: MWPW-159217 and MWPW-159143
Test URLs:

Before: https://main--milo--adobecom.hlx.page/?martech=off
After: https://unav-jarvis--milo--adobecom.hlx.page/?martech=off

Qa: https://adobecom.github.io/nav-consumer/navigation.html?env=stage&navbranch=unav-jarvis&showUnavSectionDivider=true

Validation done on the below test url to see divider
Note: The consumer team can check the Jarvis functionality as the test url is not whitelisted and cannot check Jarvis.
Case 1: The section divider is displayed when the divider flag is set to true.
https://adobecom.github.io/nav-consumer/navigation.html?env=stage&navbranch=unav-jarvis&showUnavSectionDivider=true&authoringpath=/federal/dev/gnavtest/locgnav

Case 2: No section divider is displayed when the divider flag is set to false.
https://adobecom.github.io/nav-consumer/navigation.html?env=stage&navbranch=unav-jarvis&showUnavSectionDivider=false&authoringpath=/federal/dev/gnavtest/locgnav

Case 3: Sign in flow - Notification check
https://adobecom.github.io/nav-consumer/navigation.html?env=stage&navbranch=unav-jarvis&showUnavSectionDivider=true&authoringpath=/federal/dev/gnavtest/locgnav&customlinks=home,learn,apps,discover,files


Apply --no-install options when constructing resolution

Summary
We need to apply the --no-install filters earlier, such that we don't error if we only have a source distribution for a given package when --no-build is provided but that package is omitted.
Closes #7247.

Thanks a lot for fixing this so quickly !


Fixing random performance bug in CMS due to catalyst issue

CMS sometimes gets randomly stuck in catalyst when dropping missing values if a specific sequence of estimators is run before it.  Caching the data resolves the issues with catalyst.  This looks like a bug in spark's catalyst which might have been already resolved in future versions of spark.
This resolves issue #186

Thank you for finding a solution to the timing problem so that the more meaningful/interesting version of the notebook can be run.

done - updated based on comments, thanks!


Author active status

fuel_authors table now has an is_active column, which can be changed through admin/authors form

@rkillackey looks good! 👍


CDC changefeed is slow recently

What did you do?

Run automation test case redo_apply_multiple_workload_node_restart_sync,

What did you expect to see?
Case should pass
What did you see instead?
Case failed since 11/25, and from cdc metrics, we can see the changefeed is much slower than that that of previous versions
== checkpoint lag metric 11/29==

== checkpint lag metrics on 11/25

Versions of the cluster
Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
master

Upstream TiKV version (execute tikv-server --version):
master

TiCDC version (execute cdc version):
[\"Welcome to Change Data Capture (CDC)\"] [release-version=v6.5.0-alpha] [git-hash=5dc3c5030992364db67d6e1ada9accc4409e9d35] [git-branch=heads/refs/tags/v6.5.0-alpha] [utc-build-time=\"2022-11-27 11:06:30\"] [go-version=\"go version go1.19.3 linux/amd64\"]

/severity major

/found automation

/assign @hicqu

Run the case redo_apply_multiple_workload_node_restart_sync severral times when debug.enable-pull-based-sink = false.
The changefeed lag( 5min+) is better than that in original bug (~50min),  but it is still about 2 times of previous lag, which is less than 3 minutes.
bash-5.1# /cdc version
Release Version: v6.5.0-alpha
Git Commit Hash: 519c6f08dc5ca000d8c92101e2987daf071d716e
Git Branch: heads/refs/tags/v6.5.0-alpha
UTC Build Time: 2022-11-30 11:07:02
Go Version: go version go1.19.3 linux/amd64
Failpoint Build: false

/remove-severity major

Retest with latest CDC version
For case redo_apply_multiple_workload_node_restart_sync

latest result

11/25 result

bash-5.1# /cdc version
Release Version: v6.5.0-alpha
Git Commit Hash: 5b3158b1f13fea049be88004275f12cc0739d0ea
Git Branch: heads/refs/tags/v6.5.0-alpha
UTC Build Time: 2022-12-03 11:43:23
Go Version: go version go1.19.3 linux/amd64
Failpoint Build: false

verified and close it

https://github.com/pingcap/tiflow/issues/7780

/remove label affects-6.5

/remove-label affects-6.5


Prepare for Kubernetes deployment

As a [Developer]
I need [a proper ./k8s folder in the repository add all necessary Kubernetes manifest files for our application's deployment ]
So that [we can properly deploy to kubernetes]
Details and Assumptions

[document what you know]
part of rec 3

Acceptance Criteria
Given the repository contains a ./k8s folder
When all necessary Kubernetes manifest files are added for the application's deployment
Then the app is ready to deploy to kubernetes

Just adding the details:
• Add manifest files for deployment: deployment.yaml, service.yaml, secrets.yaml, ingress.yaml, etc.
• Create postgres.yaml manifest
• Add a /health endpoint that returns {"status":"OK"} and 200_OK return code


[PG] Improve enums type-safety

Addresses #332.

Add support to "native-like" TS enums via the enum getter on PgEnum. Native TS enums can't be created at runtime, but we can create an object that has essentially the same functionality, aside from the fact that you can't use numbers to index the object.

const roles = pgEnum('roles', ['admin', 'basic']);
const Roles = roles.enum;

await db.insert(users).values({ role: Roles.basic });

Add $inferValues property to PgEnum to return the enum's values as a union type. InferEnumValues type was also introduced with the same purpose.

const roles = pgEnum('roles', ['admin', 'basic']);
type Roles = typeof roles.$inferValues; // "admin" | "basic"
// Or, if you prefer:
InferEnumValues<typeof roles>; // "admin" | "basic"

@AndriiSherman How can we help to get this merged soon? Thanks.


eredis and pooler - pair not made in heaven

This has occurred too many times for me and I am not sure what could be the cause. Hoping someone else has experienced the issue and can provide some workarounds.
I manage 5 riak connections and 1 redis via pooler on my application. After N weeks all of a sudden eredis connections start reporting the below while riak connections are fine:
2016-04-04 19:17:53 =SUPERVISOR REPORT====
     Supervisor: {local,pooler_redis01_member_sup}
     Context:    child_terminated
     Reason:     killed
     Offender:   [{pid,<0.4356.64>},{name,eredis},{mfargs,{eredis,start_link,undefined}},{restart_type,temporary},{shutdown,brutal_kill},{child_type,worker}]

The kills continues after the first error and doesn't really seem to be able to recover.
My config is as follows:
[
  {pooler, [
    {pools, [
      [{name, riak01},
       {group, riak},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {riakc_pb_socket, start_link, ["riak-01.prod.domain.com", 8087]}}],

      [{name, riak02},
       {group, riak},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {riakc_pb_socket, start_link, ["riak-02.prod.domain.com", 8087]}}],

      [{name, riak03},
       {group, riak},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {riakc_pb_socket, start_link, ["riak-03.prod.domain.com", 8087]}}],

      [{name, riak04},
       {group, riak},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {riakc_pb_socket, start_link, ["riak-04.prod.domain.com", 8087]}}],

      [{name, riak05},
       {group, riak},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {riakc_pb_socket, start_link, ["riak-05.prod.domain.com", 8087]}}],

      [{name, redis01},
       {group, redis},
       {max_count, 5000},
       {init_count, 10},
       {start_mfa, {eredis, start_link, ["redis.prod.domain.com", 6379]}}]
     ]}
  ]},

Riak and redis are both operating and being used successfully by other services. Open file descriptors for the beam.smp process is about 106 and limited to 65536. Unfortunately not much else in the log. Any help appreciated.

@sdebnath: could this be due to culling of stale members? i.e number of processes grows from init_count to a higher number and are culled after 1 minute[1]. The way pooler kills workers matches the brutal_kill response, at least the default[2] and [3]. You can use the custom stop MFA added in #45.
The 'correct' way of terminating a simple_one_for_one child is though by going through the supervisor:
supervisor:terminate_child(Sup, Id) rather than using exit/2. Though with the custom stop mfa you can't make this call since you only got a pid, which requires you to handle it in the client code. Shouldn't be too difficult to figure out which member_sup it is by the record when calling terminate_pid. I'll have a look in the weekend.
@seth: the use of exit/2, is it due to avoid putting any tension on the member_sup supervisor?
[1] https://github.com/seth/pooler/#culling-stale-members
[2] https://github.com/seth/pooler/blob/549aad444e07f0edd911b055f37af5640b4c4b6e/src/pooler.erl#L923
[3] https://github.com/seth/pooler/blob/549aad444e07f0edd911b055f37af5640b4c4b6e/src/pooler.hrl#L9

Done, #66 should sort this out for you.

With #66 merged, I'm going to close this. If still an issue, please re-open.


Updating README/setup.py for new location.

Based on https://github.com/GoogleCloudPlatform/google-auth-library-python/issues/167
Fixed "and and" typo
Fixed URL for project
Updated the package description to explain it was for httplib2 transport
Added httplib2 as a dependency

LGTM and thanks for doing this, @dhermes. You just gotta add that final comma and Travis should be happy.

Just --amend-ed and did a force push. (I don't have merge privileges, or any sort of privileges in this repo.)

@dhermes you do now. :)


Change how flags are represented

In the recent re-architecture of front we have this solution for "flags":
# CONTAINER FLAGS
CONTAINER_APP = 0x1000000000000000
CONTAINER_VIEW = 0x2000000000000000

# COMPONENT FLAGS
COMPONENT_TEXT = 0x0000000000000001
COMPONENT_TEXT_AREA = 0x0000000000000002

COMPONENT_INT = 0x0000000000000010
COMPONENT_SLIDER = 0x0000000000000020

COMPONENT_FLOAT = 0x0000000000000100
COMPONENT_FLOAT_SLIDER = 0x0000000000000200

Before that, Enum with power-of-2 integers was used, to be interpreted as bitmaps: The idea being that we can then use the disjunction of these integers to get another integer that would represent the union of the elements (and/or sets of elements).
my_union = 1 | 4
assert my_union == 5

# and we can use conjunction to interrogate the "contents" (1 and 4, meaning first item (binary `1`) and third item (binary `100`))
contains = lambda superset, subset_or_element: superset & subset_or_element == subset_or_element

# demo:
assert contains(my_union, 1) == True
assert contains(my_union, 2) == False
assert contains(my_union 3) == False
assert contains(my_union, 4) == True
assert contains(my_union, my_union) == True

Cute. Knuth would congratulate you for knowing how binary works. Bob should frown at you though.
The interface is obscure. The tracebacks would be even more so. To extend you need to manage collision, and these collisions would be easy to run into (most people would just take the next available power of two, right?). No, no, no. This just smells.
One step back: What are we actually looking for?
The ability to define boolean flags in an extendible way? Why not just sets of strings?
my_union = {'COMPONENT_INT', 'COMPONENT_FLOAT'}
assert 'COMPONENT_INT' in my_union
assert 'COMPONENT_FLOAT' in my_union
assert not ('SOMETHING_NOT_IN_MY_UNION' in my_union)
assert my_union.issuperset(my_union)

If we want something a bit more controlled -- endowed with and application specific validations, conveniences etc. -- then we can wrap this set in a facade. For example, consider the class below, that exposes a few operators of the set, validates that you can only use sets of strings, and exposes these strings as attributes of your instance (allowing you to use tab-completion):
# And you can access the flags via attributes (and use TAB-complete on them!)
# The value of these attributes are the strings themselves.
assert flags.apple == 'apple'
assert flags.orange == 'orange'

# You can "add" ("or", really) a string or a set of strings
extended_flags = flags | 'blue'
assert extended_flags == Flags({'orange', 'apple', 'blue'})

# Note that flags didn't change in the above operation:
assert flags == Flags({'orange', 'apple'})

# But you can also use the `|=` operator
flags |= {'red', 'riding'}  # you can also extend this way (with a single string or )
assert flags == Flags({'orange', 'red', 'apple', 'riding'})

# Note though, that this does NOT mean flags is mutable. x |= y is just a shorthand for x = x | y (the old x is overridden by the new)
# Proof:
flags = Flags(['apple', 'orange'])
pointer_to_flags = flags

assert pointer_to_flags == Flags({'orange', 'apple'})
flags |= {'red', 'riding'}
assert flags == Flags({'orange', 'red', 'apple', 'riding'})
assert pointer_to_flags == Flags({'orange', 'apple'})

But there's more
Note that I purposely ignored the fact that:
COMPONENT_TEXT = 0x0000000000000001
COMPONENT_TEXT_AREA = 0x0000000000000002
COMPONENT_INT = 0x0000000000000010
COMPONENT_SLIDER = 0x0000000000000020

Is doing more than using powers of 2 -- it's organizing things in hex buckets.
It seems the intent is to be able to classify categories further.
Again, nice hack; but do we want extenders to read Knuth to be able to operator (maybe the answer should be yes...).
Anyways, whatever the intent is here, I think we can provide a safer and more intuitive interface to it by creating a custom object for it instead of harnessing low level bit operations.

Note that whenever we have these dict-heavy config interfaces, something like this can come in handy.
We can always (and should) validate the config dicts, but even better would be not needing validation at all (or reducing the need) via the use of objects that make you get it right in advance.
For example, instead of:
{
    'this': 'that',
    ...
}

Having a helper object h such that
h = ConfigHelper(...)

{
    h.this: h.this.that,
    ...
}

so that validation happens through tab completion, linting, and static analysis.


Bidirectional LSTM?

I'm in the process of porting some Tensorflow models over to Candle and I have a bidirectional LSTM layer in my model. Is this something that Candle can/will support in the future?
tf.keras.layers.Bidirectional(
    tf.keras.layers.LSTM(
        self.parameters.lstm_units,
        return_sequences=True)
)

We could add support for this but it seems pretty easy to do it in the user code with the current version, e.g. with something like:
    let inp_sequence = [3f32, 1., 4., 1., 5., 9., 2.];

    // Create one lstm for the forward pass and one for the backward pass
    let lstm = candle_nn::lstm(2, 3, Default::default(), vb.pp("forward"))?;
    let lstm_rev = candle_nn::lstm(2, 3, Default::default(), vb.pp("backward"))?;

    // Apply the forward lstm and collect the results in states
    let mut states = vec![lstm.zero_state(1)?];
    for &inp in inp_sequence.iter() {
        let inp = Tensor::new(&[[inp, inp * 0.5]], cpu)?;
        let state = lstm.step(&inp, &states.last().unwrap())?;
        states.push(state)
    }

    // Apply the backward lstm and collect the results in states_rv
    let mut states_rev = vec![lstm.zero_state(1)?];
    for &inp in inp_sequence.iter().rev() {
        let inp = Tensor::new(&[[inp, inp * 0.5]], cpu)?;
        let state = lstm_rev.step(&inp, &states_rev.last().unwrap())?;
        states_rev.push(state)
    }

    // Merge the results together
    let states = states
        .into_iter()
        .zip(states_rev.into_iter().rev())
        .collect::<Vec<_>>();

Thanks, I can follow your example for now! I mainly suggested it because I don't know enough about the under-the-hood mechanics to re-implement it on my own.


Update storage-how-to-mount-container-linux.md

@v-owendeng : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add client ctor with credential support if swagger provide security definitions

For most auto-generated LLC packages, they have clear security definitions to indicate how to get credential when using the service (some examples: #1, #2, #3).
In order to provide a more friendly client and reduce manual work for DPG, I'd like to auto generate the ctor with credential according to the swagger.
Regarding there will be multiple security definitions for one client, the client ctor will use security definitions name as suffix, such as NewFeatureStateClientWithAADTokenCredential, NewFeatureStateClientWithSharedKeyCredential, NewFeatureStateClientWithSasTokenCredential.
Also, a new KeyCredential and KeyCredentialPolicy will be added to azcore.
This is somehow a DPG feature and also will resolve this issue.
Any concern? @jhendrixMSFT  @JeffreyRichter

I do not think you can ultimately be successful with this, which is why we have not done it already.
First, we are assuming that all Azure services will support TokenCredential now and so we do want to auto-generate ctors for this and because we (Azure) want customers to use this cred primarily, we name this ctor NewFeatureStateClient; we do NOT add the WithAADTokenCredential suffix but this ctor does take a TokenCredentail.
Second, most new Azure services will use TokenCred as the only auth mechanism and so we'll get pretty far with code-gen'ing this ctor.
Third, you can't auto-gen SharedKey or SAS creds because what services do with these varies and it's not in the swagger. For example, Storage requires both of these creds to create strings to sign and how to construct the string is not defined in the swagger so it must be manually written. And because the algorithm is service-specific, it must live in the service's package and not in the core package.

At present, the only ctor we generate for data-plane is one that takes a pipeline.  Clearly, we don't want to ship like this.
In swagger we currently support token and shared key credentials.  We can certainly change data-plane codegen to emit the correct token credential constructor (with the "good" name) when specified.
For shared key, if we end up with a NamedKeyCredential type in azcore we can also emit this constructor.  However, as Jeff pointed out, the underlying implementation varies by service so there will still be some amount of hand-written code necessary to make it work.  What would be ideal in this situation is to emit code in such a way that the plumbing is hooked up but it won't actually compile until the appropriate named key policy is implemented.
Finally, how do we want to handle the case when no authentication type is specified in the swagger?  My preference would be to stop doing what we do now (i.e. emit ctor that takes a pipeline) and just emit no ctor.  This would require that implementors provide one.

Yes, I agree: emit no ctor and write it by hand.

I agree with Joel's suggestions. If swagger specifics the auth, then we could generate the ctor (e.g., purview, cognitive service), if not, we do nothing (app configuration, key vault, storage). And if the key credential is not a simple get set, we still need to add some hand-written code.
It will be useful for DPG as all the releases will be handled by service team and we only do the tooling support.
I think the only problem is the ctor naming. It is easy when only one auth defined. But one spec could have multi security definitions. We could set first as default and add suffix for the followings. But it will again depend on the swagger author to not involve any breaks with the sequence.

A ctor without a credential (NewClient, for example) always takes a TokenCredential - our code generator can always produce this ctor.
Other ctors always have the credential name (NewClientWithSharedKeyCredential, for example) - I do not believe we can code generate these other ctors because there is usually not enough info in the swagger to do it right.

Only if it's specified in the spec though, yes?  Put another way, do all services support AAD authentication?

How are credentials specified in the spec?

@JeffreyRichter Here are some examples: #1, #2, #3

Only if it's specified in the spec though, yes? Put another way, do all services support AAD authentication?

I think so. No credential specified, no ctor. For arm spec, m4 always add AAD auth because of the flag --azure-arm.

I see the securityDefinitions in the spec (thanks). However, I still do not think you have enough information to actually construct the pipeline with the right policy to make it work correctly (at least in some cases). But, if you think you can, give it a try and we can see if it works.

The only authentication types supported at present are OAuth2 and API key (see docs).  For the former we can reliably support this for codegen.  For API keys you are correct that we don't have enough info.  We can emit the ctor but obviously we would be missing the correct pipeline policy to make it work.  So it might not be worth emitting it at all.
Regarding ctor naming, we want to reserve the "good name" for the ctor that takes a token credential.  How do we want to handle the case where a service doesn't support AAD auth at present but adds it later?  i.e. if a service only supports shared key auth, should we still name the ctor NewClientWithSharedKey(), reserving the "good name" for later?  I think the answer is yes.

Yes, the good name is always for AAD whether the service supports it today or not.

Reopen because we need to rethink about the implementation.

For keyvault, I reviewed the related code with JS and Python.
It seems they all generate the original CTOR of the client and replace them with customized client.
For Go, I think we could also always do the CTOR generation. And for the package that need  existed export-clients flag to determine whether we

Change will be in preview.46.


fix: logging nil errors in slo native workers

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Pull request type
Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
workers logging even nil errors (no error)

Codecov Report
Patch coverage has no change and project coverage change: -0.05 :warning:

Comparison is base (656cad4) 64.79% compared to head (a9d8651) 64.74%.

: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     #701      +/-   ##
==========================================
- Coverage   64.79%   64.74%   -0.05%
==========================================
  Files         228      228
  Lines       22274    22274
==========================================
- Hits        14432    14421      -11
- Misses       7071     7080       +9
- Partials      771      773       +2

Flag
Coverage Δ

51.44% <ø> (-0.03%)
:arrow_down:

1.17.x
64.59% <ø> (+0.22%)
:arrow_up:

1.19.x
64.48% <ø> (+<0.01%)
:arrow_up:

1.20.x
46.38% <ø> (-18.15%)
:arrow_down:

integration
51.44% <ø> (-0.03%)
:arrow_down:

macOS
46.29% <ø> (-0.02%)
:arrow_down:

secure
51.44% <ø> (-0.03%)
:arrow_down:

ubuntu
46.33% <ø> (+0.01%)
:arrow_up:

unit
46.39% <ø> (+0.01%)
:arrow_up:

windows
46.12% <ø> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 10 files with indirect coverage changes
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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[MINOR][DOC] Update the condition description of serialized shuffle

What changes were proposed in this pull request?
1. The shuffle dependency specifies no aggregation or output ordering.
If the shuffle dependency specifies aggregation, but it only aggregates at the reduce-side, serialized shuffle can still be used.
3. The shuffle produces fewer than 16777216 output partitions.
If the number of output partitions is 16777216 , we can use serialized shuffle.
We can see this mothod: canUseSerializedShuffle
How was this patch tested?
N/A

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5750/
Test PASSed.

Test build #99703 has started for PR 23228 at commit d5dadbf.

Merged build finished. Test FAILed.

Test build #4453 has started for PR 23228 at commit d5dadbf.

Test build #4453 has finished for PR 23228 at commit d5dadbf.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

cc @JoshRosen  @cloud-fan

Please update the title [MINOR][DOC] Update the condition description of serialized shuffle

retest this please

Test build #99892 has started for PR 23228 at commit d5dadbf.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5908/
Test PASSed.

Test build #99892 has finished for PR 23228 at commit d5dadbf.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99892/
Test FAILed.

retest this please

Test build #99902 has started for PR 23228 at commit d5dadbf.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5915/
Test PASSed.

Test build #99902 has finished for PR 23228 at commit d5dadbf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99902/
Test PASSed.

thanks, merging to master!


Dependency conflicts

Only in keqingw branch:
The Pipfile in this branch is a bit broken.

Clustergrammer only works with pandas up to 0.25 since it uses .ix, which is deprecated in later versions.
mofax needs pandas >= 1.0.0

Solution would be to update clustergrammer codes to not use .ix.
Until we do that, the heatmap using clustergrammer isn't going to work I think

False alarm. Seems like it's working now?

Temporary fix in this commit: https://github.com/glasgowcompbio/GraphOmics/commit/e729dae1125d8712a8ec8c7f5681ff8ba13f0eb1.
Best to switch to clustergrammer2 which doesn't have this problem. See the discussion here: https://github.com/MaayanLab/clustergrammer-py/issues/16

Can temporarily closed for now


Uncaught Error: Assertion failed: Need to call node.removeChild() with a Node.

In friction, highlight to macro book by way of keyboard nav, then click on the keyboardHelpDialog. This should repeat the error. What I'm finding is that the Book is focusHighlightLayerable, but for some reason doesn't have accessible content when it gets to FocusOverlay's deactivateHighlight method, so it goes into the else clause:

        // If focusHighlightLayerable, then the focusHighlight is just a node in the scene graph, so set it invisible
        if ( this.node.accessibleContent && this.node.accessibleContent.focusHighlightLayerable ) {
          this.node.accessibleContent.focusHighlight.visible = false;
        }
        else {
          this.highlightNode.removeChild( this.nodeHighlight );
          this.nodeHighlight = null;
        }

assert.js:22 Uncaught Error: Assertion failed: Need to call node.removeChild() with a Node.
    at window.assertions.assertFunction (assert.js:22)
    at Node.removeChild (Node.js?bust=1508360797944:639)
    at FocusOverlay.deactivateHighlight (FocusOverlay.js?bust=1508360797944:163)
    at FocusOverlay.onFocusChange (FocusOverlay.js?bust=1508360797944:203)
    at Emitter.emit2 (Emitter.js?bust=1508360797944:205)
    at Property._notifyListeners (Property.js?bust=1508360797944:190)
    at Property._setAndNotifyListeners (Property.js?bust=1508360797944:178)
    at Property.set (Property.js?bust=1508360797944:125)
    at Property.set value [as value] (Property.js?bust=1508360797944:219)
    at Function.set focus [as focus] (Display.js?bust=1508360797944:1815)

Maybe this would be something good to look at in the a11y pair this afternoon.

@zepumph @mbarlow12 and I worked on this today, the problem was that FocusOverlay was referencing accessibleContent when it was undefined because a Dialog had opened. The fix was to reference the focusHighlight through the Node getter directly,  which wasn't possible when FocusOverlay was first written. Closing!


Connect to tensorboard behind jupyterhub

I've put my tensorboard behind an apache proxy so that calls to the tensorboard are
http://localhost/tensorboard/
Would be interested to see how to modify jupyter_tensorboard to connect to this type of call.

The problem may be that it's going through jupyterhub.  The url that is going into the system is
http://big-apple.local/jupyterhub/user/user/api/tensorboard?1549244001578

I found the problem.  The auto enable for jupyter_tensorboard installs the server extension in the user context.  When I'm using jupyterhub, I naturally want the server extension in the system context, so I have to add them manually.
Not sure what is the best way of dealing with this other than to add information into the error message or documentation.

@joequant any luck ?

I have this issue (similar) into a Kubernetes cluster, running JupyterHub... Any solution?


A special target for testing local nuget builds

It is sometimes needed to quickly test local changes against an app that consumes nuget packages.
This PR allows to quickly update 9999.0.0-localbuild version in nuget cache from the current source.
(incremental build)
═══════════════════════════════════════════════════════
Target                             Status      Duration
───────────────────────────────────────────────────────
GenerateCppHeaders                 Skipped                // SkipReason: skipping CompileNative
Clean                              Skipped                // SkipReason: via --skip parameter
CompileNative                      Skipped                // SkipReason: EnvironmentInfo.IsOsx
CompileHtmlPreviewer               Skipped                // SkipReason: via --skip parameter
Compile                            Skipped                // SkipReason: via --skip parameter
CreateIntermediateNugetPackages    Succeeded       0:31
CreateNugetPackages                Succeeded       0:09
BuildToNuGetCache                  Succeeded     < 1sec
───────────────────────────────────────────────────────
Total                                              0:41
═══════════════════════════════════════════════════════

@kekekeks:

does this allow to locally debug into Avalonia code?
how would I run this locally to have the version in the local nuget cache?
The app itself - I guess - I would then target to 9999.0.0-localbuild, right?

@llfab
You need to run nuke BuildToNuGetCache in the repository root to get these packages.
After that - just include Avalonia as you usually do from nuget, but with 9999.0.0-localbuild version.
The main point of this feature is to be able to quickly debug local Avalonia code changes, without waiting for Nightly NuGet builds. Debugging experience shouldn't be really different from typical NuGet (though, not sure how SourceLink symbols would work here).

Updated build instructions to include this command https://github.com/AvaloniaUI/Avalonia/blob/master/Documentation/build.md

Indeed I am looking for an Option to debug into Avalonia code and set breakpoints there. Seems this would not be possible with this option right?
I also thought about .props magic to easily switch between nuget packs and Avalonia repo csproj. That would be easy but then the Avalonia csproj would need to be "injected" into the solution. Not sure if that can be done.
So: would the above local packs option allow for setting breakpoints into Av code?

You can debug your app directly from Avalonia solution that way.

open avalonia desktop slnf
build to nuget cache
add your executable like this:

remove "build solution":

run

Breakpoints should work.

I came up with this here which works nicely for me:
https://github.com/AvaloniaUI/Avalonia/discussions/14819


Specify path to index.js

Hi I hope this isn't something I've missed in the documentation but I've been trying for a day and a half now so forgive me if so.
My directory layout is as follows -
DMForge
----app
--------index.js
----assets
----lib
----node_modules
----partials
----.bowerrc
----package.json etc.
package.json scripts -
"start": "electron ./app",
"build": "electron-packager . DMForge --out=dist --ignore='^/dist$' --all --version=0.34.1"

npm start works fine with the electron prebuilt
When I run "npm run build" the generated application looks for my index.js in resources\app\index.js when it should be in resources\app\app\index.js

If I try putting ./app into the electron-packager command I'm missing all of the dependencies that are a level up the tree.
Am I missing a swtich that would let me achieve this build? I know it would probably all work if I threw everything in the one directory but I like the directory layout I'm working with.
My project is at https://github.com/Liam-Ryan/DMForge, at this point it's really nothing more than an alteration of the yeoman generator as a proof of concept.
Thank you for your time

Update - I've moved index.html and index.js to the root of my project and now the packaged app is running fine, I'd like to keep this issue open to see if it's feasible to have an option for the path to the entry point of the electron app

Isn't this normally solved by specifying the main property in package.json?

Sorry I haven't had time to test this, real life interference the last few days. I'll close if it works for me when I get a chance

Tested this.
Setting package.json main` prop as entry file solves this problem :)


[dns] obscure test failure in absence of network connection

--- FAIL: TestSimpleQuery (0.00s)
	util.go:87: Unexpected error: listen udp <IP_ADDRESS>: setsockopt: no such device
		goroutine 13 [running]:
		github.com/zettio/weave/testing.stackTrace(0x2b3375f79b00, 0x0, 0x0)
			/home/matthias/go/src/github.com/zettio/weave/testing/util.go:96 +0x88
		github.com/zettio/weave/testing.StackTrace(0x0, 0x0)
			/home/matthias/go/src/github.com/zettio/weave/testing/util.go:91 +0x35
		github.com/zettio/weave/testing.Fatalf(0xc20806c630, 0x883610, 0x14, 0xc20807bd48, 0x1, 0x1)
			/home/matthias/go/src/github.com/zettio/weave/testing/util.go:87 +0x27
		github.com/zettio/weave/testing.AssertNoErr(0xc20806c630, 0x2b3375f79b00, 0xc208033000)
			/home/matthias/go/src/github.com/zettio/weave/testing/util.go:24 +0xea
		github.com/zettio/weave/nameserver.setup(0xc20806c630, 0x1, 0x1, 0x0, 0x0)
			/home/matthias/go/src/github.com/zettio/weave/nameserver/mdns_client_test.go:79 +0xbd
		github.com/zettio/weave/nameserver.TestSimpleQuery(0xc20806c630)
			/home/matthias/go/src/github.com/zettio/weave/nameserver/mdns_client_test.go:111 +0x11a
		testing.tRunner(0xc20806c630, 0xa4c5f8)
			/usr/local/go/src/testing/testing.go:447 +0xbf
		created by testing.RunTests
			/usr/local/go/src/testing/testing.go:555 +0xa8b

The root cause appears to be the call to net.ListenMulticastUDP("udp", ifi, ipv4Addr), which supplies a nil interface, which according to the docs means "uses default multicast interface". I'm guessing that my machine only gets one of these when connected to a network.
This may not be easily fixable, but perhaps the tests relying on such an interface should be skipped when it is absent, or at least a more informative error should be produced.

I get the same error when you do not have connectivity. In that case, we could abort and print more information on the reason (WeaveDNS tests would be useless though).


Random Battle: Add missing move enforcement for Surge abilities

The criteria of requiring Electric moves with Electric Surge and Psychic
moves with Psychic Surge is present in the random doubles code, but not in
random singles.

I have this in my next update. Also, the Electric Surge check is redundant as Electric STAB is already enforced on its own. So I'm just gonna close this.


Reciee JSON

Hello,
is there a way that I can make a receiving method which receive json. That json contains an array

You can send/pass strings. But one of the ideas behind SignalR is to create a corresponding type and send/receive instances of this type. For instance instead of sending/receiving a string that is a JSON array and then parse it yourself why not send/receive an array itself?

Considering the question answered.

hello, can you give some example about send and get json? im stuck in this problem, as in client i have:
let typee = INVOKE_TYPE(Typee: 4000)
        self.connection?.invoke(method: "TestReturnJson", typee, resultType: INVOKE_TYPE.self, invocationDidComplete: { (result, error) in
              print("SignalRService TestPingPong \(result) - \(error)")
        })

struct INVOKE_TYPE : Encodable, Decodable{
   var Typee : Int = 4000
}

But im only get error SignalRService TestPingPong nil - Optional(SwiftSignalRClient.SignalRError.hubInvocationError(message: "Failed to invoke 'TestReturnJson' due to an error on the server."))

@fukemy - this issue has been closed. Please open a new issue and attach logs. From what you posted it seems to me that there is a failure on the server side. Here is where the error you are seeing is being thrown:
https://github.com/dotnet/aspnetcore/blob/2ad8121efbdddd46920918bddb9ee67e1c4144a9/src/SignalR/server/Core/src/Internal/DefaultHubDispatcher.cs#L212-L213
Also check #78 - it shows some steps that can help troubleshoot.


On a german keyboard the 'ü' key is interpreted wrong.

With a german keyboard layout (setxkbmap de on Linux) LightTable ignores the ü key and inserts [ instead. Both are on the same key, but differ depending on the layout.
I don't think this is intentional, but I'm not sure. The key on the right of  the ü or [ results in the correct character (+ and ] respectively).
I'm using 0.4.11, but I'm not sure whether this is a new problem or I noticed it only now.

on japanese keyboard on mac book pro
all the keys are swapped. caps key is interpreted as control and so forth.
this way I would have to have different shortcuts for different layouts, even though I wanna keep same layout.


CortanaVoiceCommand: InstallCommandDefinitionsFromStorageFileAsync fails to return

Hello,
I am having issues running the example projects. I have tried the C# and Javascript version. Both of them fail at the same point of installing the VCD.
In Javascript, it fails at js/default.js:52  return voiceCommandManager.installCommandDefinitionsFromStorageFileAsync(file); with the error

'WWAHost.exe' (Script): Loaded 'Script Code (MSAppHost/3.0)'.
The program '[13004] WWAHost.exe' has exited with code 1 (0x1).

In C#, it fails at App.xaml.cd:102 await Windows.ApplicationModel.VoiceCommands.VoiceCommandDefinitionManager.InstallCommandDefinitionsFromStorageFileAsync(vcdStorageFile);. In this case it never returns with any kind of exception.
In both cases AdventureWorks does not show up under the Cortana help menu.
I am running Microsoft Windows Version 10.0.10586 (ProfessionalN edition) with Cortana linked to an account, and language set to en-US. I will try the same example on a non-N environment to see if that makes a difference.
I believe this should be working to some extent on this machine because it appears the Netflix application was able to install their Cortana bindings.
Thanks,
Kevin

I was able to resolve this by creating an app package (Project > Store > Create app packages...) rather than running it within Visual Studio.


request: command to generate shell completion scripts

per title, is it possible for the completions scripts to be generated on demand instead of manually linking them from $CARGO_HOME?
many thanks.

that sounds right for me

Sorry for taking a while but let me know if that is what you were looking for. Using code on main you can now do
tgrep --completions <shell>

To generate completions to stdout.


Run Recent Command from REPL doesn't list Python commands

From a Python file, use shift+enter to run a command in the Python REPL
From the blue circle decoration for that command, select Run Recent Command
:bug: the commands listed do not include the Python commands, only terminal commands

This is as expected, the plan to add this is in my next iteration plan

tracking here: #22647


[panic] No available pixel format

Hello,
I'm using Windows 10 on a VMware Horizon virtual desktop, that I access through VMware Horizon Client.
I downloaded the Lapce text editor 0.2.0 (portable), which uses a fork of Druid for its UI.
That repo doesn't have an Issues tab, so I'd thought I'd open an issue here, in case it might be relevant to other projects that use Druid.
Lapce panics at start with the following error:

thread 'main' panicked at 'called Result::unwrap() on an Err value: Other(no available pixel format)'

And mentions druid-shell/src/backend/windows/window.rs:843:26
Screenshot:

Based on discussion in the Lapce issues this seems to be related to their modified GPU code. Thus I'm going to close this as not applicable. If you can reproduce the crash with any of the Druid examples we can reopen it.


Allow range accept only one argument

Feature Request
Is your feature request related to a problem? Please describe.
I'm never able to remember whether RxJS range has a variant that accepts only one argument. For example in lodash I can use range(10) and it'll generate an array of 10 items. However, in RxJS I always have to use two parameters range(0, 10).
import { range } from 'rxjs';
import _ from 'lodash';

// will work
range(0, 10).subscribe(console.log)

// won't work
range(10).subscribe(console.log)

// will work
console.log(_.range(10))

https://stackblitz.com/edit/rxjs6-demo-n1wwfd?file=index.ts
This isn't consistent across different Rx implementations:
RxPY does allow only one param https://github.com/ReactiveX/RxPY/blob/master/rx/operators/observable/range.py#L6.
RxJava doesn't https://github.com/ReactiveX/RxJava/blob/2.x/src/main/java/io/reactivex/Observable.java#L3540
RxSwift does ... something https://github.com/ReactiveX/RxSwift/blob/master/RxSwift/Observables/Range.swift#L30
Lodash does https://lodash.com/docs/4.17.11#range
Describe the solution you'd like
Ideally, if I use just one argument it'll be used as count parameter  and start will be set to 0.
So using range(10) and range(0, 10) will produce the same result.
Describe alternatives you've considered
I can keep on using range(0, 10) but if I could use just range(10) it would be easier to remember and probably shouldn't break any existing functionality.

I can have a look into it and create a PR...


Super shield weapons are affected by Multishot

I feel like this isn't intentional? Not sure.

iirc I change the code specifically so they would work together, the super shield weapons seem kinda bad by defualt so becoming kinda OP with multishot is probably fine.


Any interest in standardized commit message formatting and/or CHANGELOG generation?

This could be as simple as making the repo commitizen friendly or as deep as integrating semantic-release, or pretty much anywhere in-between. If you're interested, I'll take it on.

I'm very -1 on semantic-release - automating change log generation is nice - but enforced commit message conventions are pretty annoying for contributors.

I think there's a lot more to win by introducing a formal message syntax. Commitizen, and it's use of Inquirer, make it very simple to get the formatting right without changing your typical workflow (git cz instead of git commit). We're talking about something that could automate the workflow from commit to version bump to CHANGELOG generation to release, which to me vastly outweighs the potential annoyance it may bring to authoring a commit message. Outside of the improvements to the CI workflow that semantic-release could bring, simply adding some focus on the message itself naturally tends to nudge collaborators toward making meaningful, atomic commits. So while I hear you that it could be initially annoying to some users, I can't see that as a reason to disregard all the other benefits it would bring.

Just sharing the experience of trying to make PRs to projects that have CI-enforced commit message conventions - it's pretty frustrating, and lots of users don't know git well enough to know how to fix it easily.

Yeah, I hear ya.
If that's a big concern, we should be able to mitigate it with a simple client side hook. This would prevent the bad messages from ever making it in, so people won't need to worry about fixing a bad message.

Clientside hooks can't be installed except with explicit client cooperation; they don't work on Github's UI; and they're just as frustrating for new contributors, which are a far more important audience to cater to than previous contributors (due to the much higher risk of attrition)

Doh, forgot about that after living in a non-GitHub world for quite some time...
Regardless, I still wouldn't kill the whole idea due to the inconvenience of formatting a message. Big picture there are a lot more wins here..

As a collaborator on many projects, I find it's usually easier to manually correct commit messages upon merging (or directly on the branch, now that github allows that), and not to make it fail tests.

Manually correcting them is also an option, although it feels like a much bigger pain.
I think we're deviating from the core discussion though. To me the big question is around the focus of the project team -- do you prefer release automation, at the (IMO slight) cost of additional contribution guidelines, or do you want to keep the guidelines very loose to attract contributors?

Other collaborators should of course weigh in; my preference is to gently encourage git hygiene (according to whatever rubric we all want), but not to hard-require it via automation, and to manually correct it on the PR when needed. "Managing releases" and "writing a changelog" imo are not the most pressing concerns of open source development, even if they seem seductively easy to automate.

Other collaborators should of course weigh in; my preference is to gently encourage git hygiene (according to whatever rubric we all want), but not to hard-require it via automation, and to manually correct it on the PR when needed. "Managing releases" and "writing a changelog" imo are not the most pressing concerns of open source development, even if they seem seductively easy to automate.

I agree. I love structure that reduces thinking and friction overtime; structure with a constant coefficient of friction is less desirable. When pain exceeds that friction coefficient, structure makes sense. In our case, I think we're still small enough to calibrate organically.
Soft-closing for now. Feel free to re-open if you think there is more to discuss.


chore) add ReadAll method

approve

Codecov Report

Merging #12 into master will increase coverage by 30.15%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #12       +/-   ##
=========================================
+ Coverage   69.84%   100%   +30.15%
=========================================
  Files           1      1
  Lines          63     10       -53
=========================================
- Hits           44     10       -34
+ Misses         14      0       -14
+ Partials        5      0        -5

Impacted Files
Coverage Δ

iioutil.go
100% <100%> (+30.15%)
:arrow_up:

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 282a781...df6af01. Read the comment docs.

Codecov Report

Merging #12 into master will increase coverage by 30.15%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #12       +/-   ##
=========================================
+ Coverage   69.84%   100%   +30.15%
=========================================
  Files           1      1
  Lines          63     10       -53
=========================================
- Hits           44     10       -34
+ Misses         14      0       -14
+ Partials        5      0        -5

Impacted Files
Coverage Δ

iioutil.go
100% <100%> (+30.15%)
:arrow_up:

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 282a781...df6af01. Read the comment docs.


OpenAPI agent does not support PATCH and PUT and DELETE, only GET and POST

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
agent = planner.create_openapi_agent(
api_spec=api_spec,
requests_wrapper=requests_wrapper,
llm=llm,
allow_dangerous_requests=True,
)
Error Message and Stack Trace (if applicable)
No response
Description
i want the tool to have full HTTP capabilities and not only GET and POST
System Info
latest langchain python installation which is 0.1.16

it seems like can just add the requests to _create_api_controller_agent in planner.py at line 251 in langchain_community/agent_toolkits/openapi/planner.py

Feel free to enhance the functionality -- it should be opt-in if you do so

@yair-the-super-wise Please check this out pull request.
I don't know why this issue wasn't added to be automatically closed on merge despite the fact that I referenced it in the PR description.