Test id no longer being reported for current version of Apex Plugin

Summary
Note originally reported in slack.
Hi Team,
When we run the test class (sfdx force:apex:test:run -l RunLocalTests -u<EMAIL_ADDRESS>-w 60 --json) with the CLI version 7.114.0 and earlier, we are getting the result as below while the test execution is happening at back-end:
{
"status":0,
"result": {
"testRunId": "thisIsTheTestId"
}
}
Here we are getting the Job ID and we can later find the report using the JOb ID (sfdx force:apex:test:report --testrunid thisIsTheTestId )
But in the CLI version 7.172.0 and latest this same command is not returning any Job ID/testRunId, and it seems that it is returning once the test execution is completed (which is not feasible if the customer's test classes ran for hours, since the customer will not be able to do anything else until test execution  is completed)
Is there anything changed on this command with the latest CLI versions?
Any insight on this will be really helpful !
plugins info for working and non-working versions
SFDX Version 7.114.0 (WORKING) output:
% sfdx plugins --core
› Warning: sfdx-cli update available from 7.114.0 to 7.172.0.
@oclif/plugin-autocomplete 0.3.0 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.3.1 (core)
@oclif/plugin-not-found 1.2.6 (core)
@oclif/plugin-plugins 1.10.11 (core)
@oclif/plugin-update 1.4.0-3 (core)
@oclif/plugin-warn-if-update-available 1.7.3 (core)
@oclif/plugin-which 1.0.4 (core)
[@Salesforce](https://salesforce-internal.slack.com/team/WPB0BC065)
/sfdx-plugin-lwc-test 0.1.7 (core)
[@Salesforce](https://salesforce-internal.slack.com/team/WPB0BC065)
/sfdx-trust 3.6.0 (core)
alias 1.1.10 (core)
apex 0.2.7 (core)
auth 1.7.1 (core)
config 1.2.23 (core)
custom-metadata 1.0.12 (core)
data 0.6.0 (core)
generator 1.2.2 (core)
limits 1.2.1 (core)
org 1.6.9 (core)
salesforce-alm 52.2.5 (core)
schema 1.0.8 (core)
sfdx-cli 7.114.0 (core)
source 1.0.9 (core)
telemetry 1.2.3 (core)
templates 52.1.0 (core)
user 1.4.0 (core)
************************************************************************************************
sfdx version 7.174.0 (Non Working )Output
@oclif/plugin-autocomplete 1.3.1 (core)
@oclif/plugin-commands 2.2.0 (core)
@oclif/plugin-help 5.1.14 (core)
@oclif/plugin-not-found 2.3.3 (core)
@oclif/plugin-plugins 2.1.1 (core)
@oclif/plugin-update 3.0.2 (core)
@oclif/plugin-version 1.1.2 (core)
@oclif/plugin-warn-if-update-available 2.0.6 (core)
@oclif/plugin-which 2.1.0 (core)
[@Salesforce](https://salesforce-internal.slack.com/team/WPB0BC065)
/sfdx-plugin-lwc-test 1.0.1 (core)
alias 2.1.5 (core)
apex 1.3.0 (core)
auth 2.2.15 (core)
community 2.0.10 (core)
config 1.4.21 (core)
custom-metadata 2.0.2 (core)
data 2.1.4 (core)
generator 2.0.7 (core)
info 2.2.0 (core)
limits 2.0.5 (core)
org 2.2.9 (core)
packaging 1.9.4 (core)
salesforce-alm 54.8.3 (core)
schema 2.1.10 (core)
sfdx-cli 7.174.0 (core)
signups 1.2.8 (core)
source 2.0.25 (core)
telemetry 2.0.2 (core)
templates 55.1.0 (core)
trust 2.0.4 (core)
user 2.1.13 (core) (edited)

Expected result
Should return test id for querying the test run.
Actual result
No Test id is returned.

This is a result of the -w wait flag being passed in, and with #270 the wait flag isn't performing as expected anyways.


Don't automatically set instantiate to True when constant is True

Fixes https://github.com/holoviz/param/issues/287
Pushing this WIP PR to get some feedback on the approach. The idea is that when:

instantiate is True -> deepcopy default and assign it to the local cache (pseudo: parameterized_obj._X_param_value = deepcopy(parameter.default))
constant is True -> don't deepcopy, just assign a reference to default to the local cache (pseudo: parameterized_obj._X_param_value = parameter.default)

With 1. taking precedence over 2.
The test added in https://github.com/holoviz/param/commit/7635d55e265b432417bad4ed5571fe4c84371037 and that is based on the one given in the issue passes with this change.
Of course this means that if the default object is mutable, any change made from a reference will be seen by all the other references, all pointing to the same updated object.
import param

class A(param.Parameterized):
    x = param.Number(0)

a = A(name="AAA")

class B(param.Parameterized):
    y = param.ObjectSelector(a, constant=True)

b = B(name="BBB")

B.y.x = 1
print(b.y.x)
# 0 before this PR
# 1 with this PR

b.y.x = 2
print(B.y.x)
# 1 before this PR
# 2 with this PR

If reviewers are OK with this approach, I'll need to clean up the code and do some refactoring, it's a bit hacky right now.

The unit test suites of Panel, HoloViews and Lumen passed with this branch checked out. Merging.


migrate structure tests over to structure test framework

also, add container build artifacts. The current Jenkins build job will need to be changed to use the new jenkins_build.sh script.
dependent on GoogleCloudPlatform/runtimes-common#49.

@dlorenc @broady

$ ./ext_run.sh -i gcr.io/google_appengine/golang -v --config test_config.json
/home/travis/build.sh: line 47: ./ext_run.sh: Permission denied

cc @zombiezen

ah, this is the first time i've actually tried running these in travis. just added a chmod +x to the travis yaml, let's see if this run passes

I think there's something very wrong with the test script.
The command "chmod +x ext_run.sh" exited with 0.
19.82s$ ./ext_run.sh -i gcr.io/google_appengine/golang -v --config test/test_config.json
Using default tag: latest
latest: Pulling from gcp-runtimes/structure_test
Status: Downloaded newer image for gcr.io/gcp-runtimes/structure_test:latest
Unable to find image 'gcr.io/google_appengine/golang:latest' locally
latest: Pulling from google_appengine/golang
Status: Downloaded newer image for gcr.io/google_appengine/golang:latest
docker: Error response from daemon: oci runtime error: exec: "./test/structure_test": stat ./test/structure_test: no such file or directory.

Two things:

The image should be local. Testing the image that's on gcr.io shouldn't be what's happening.
The script appears to have failed but didn't return a non-zero exit code.

/cc @dlorenc

the script will fail with that error until https://github.com/GoogleCloudPlatform/runtimes-common/pull/49 is merged and deployed; in the current version of the image there is no volume to mount in the image under test.
the image needs to get on the local machine somehow, so it can either be pulled from gcr.io or built by the user. building the image manually (by cloning the https://github.com/GoogleCloudPlatform/runtimes-common repository and doing a docker build) and then passing the --no-pull flag to the script will accomplish this.
i'll look into why the script exited with 0 after it failed.

@nkubala there's no set -e in run_ext
also, yes, the testing image needs to be pulled, then golang-docker image should not be pulled.
We should never see this in the logs:
Unable to find image 'gcr.io/google_appengine/golang:latest' locally
latest: Pulling from google_appengine/golang

alright, this should be fixed now. there were a few issues here, the biggest of which being that the entrypoint in the ext_run.sh script was failing if the container was started in any directory other than root (didn't catch this because golang is the first image i've seen to start somewhere else). fixed this and added failfast to the script here: https://github.com/GoogleCloudPlatform/runtimes-common/pull/54.
additionally, because of the way the travis yaml was set up, the tests were being run on the latest image pushed to the repository, instead of the image built off of the newly staged code. fixed the travis script to first build the image locally, then run the script on top of this image.
this PR does still depend on GoogleCloudPlatform/runtimes-common#49, but for a different reason than I had originally mentioned: this PR adds support for environment variable substitution in the tests, which is necessary for these tests to pass.
sorry for the confusion. cc @broady @dlorenc

This looks very close now. You can change the test to check the output of GOPATH in its entirety.
It kind of unfortunate the tests can't have comments because they're written in JSON. Any though to using YAML? (@dlorenc)

It kind of unfortunate the tests can't have comments because they're written in JSON. Any though to using YAML? (@dlorenc)

Yeah it already supports YAML too. Nick, could you switch this PR over to the YAML version?

Nick, could you switch this PR over to the YAML version?
done.

You can change the test to check the output of GOPATH in its entirety.
done.

:+1: LGTM. Thanks so much for working on this, Nick!

2016/11/09 21:19:03 Error parsing config file: invalid character 's' looking for beginning of value

2016/11/09 21:19:03 Error parsing config file: invalid character 's' looking for beginning

https://github.com/GoogleCloudPlatform/runtimes-common/pull/56


Fix error message (add proper spacing)

Minor string fix, as I've got the exception with two inconsistencies:

BigQueryIO.Write needs a GCS temp location to store temp files.This can be set by withCustomGcsTempLocation() in the Builderor through the fallback pipeline option --tempLocation.

Run Java_GCP_IO_Direct PreCommit

Thanks!


remove some line breaks in the license file

so github detect the license.

Description

If merged this pull request will change some line breaks in the license file so Github detects it properly
close #649

I will merge this once the CI passes


About third-party date and time widgets

Now the officially supported third-party date widget is bootstrap datepicker, but bootstrap datepicker cannot set the time part. Is there a way to support bootstrap datetimepicker, so that a widget date, time, datetime can be supported.

Yes, you can implement a custom date time picker widget using the https://github.com/surveyjs/widgets/blob/master/src/bootstrapdatepicker.js widget code as a basis.

@tsv2013 Thank you very much, I have implemented bootstrap datatimepicker.
After the implementation, there is a small problem that needs to be asked to you: When the scroll bar appears in the survey container, scroll the scroll bar of the survey container, the datatimepicker window is fixed on the current page, and does not move with the scroll bar, but when the survey container does not appear the scroll bar When scrolling the scroll bar of the survey container, the datatimepicker window moves with the scroll bar. I think this is not reasonable. Is there a solution?
This is an example：https://plnkr.co/edit/lPfGBzQpEa98Gbch?preview

Just now is the bootstrap datepicker example, which is also true. The following is the case of bootstrap datatimepicker.

@fengzhihenxs for some reason I think it’s caused by bootstrap... propable popup container is misplaced

@tsv2013  Hi,
I am trying to add one of the custom widget to survey creator toolbox but getting failed...plz help.
i m referring this example https://codesandbox.io/s/crazy-rain-3wqj7?file=/src/customwidget.js

Answered here - https://surveyjs.answerdesk.io/ticket/details/t5959/unable-to-add-custom-widget-to-creator-toolbox

Hello @fengzhihenxs ,
Were you able to impliment a custom bootstrap datatime picker. (not datepicker).
If Yes, Could you share the widget.js code

Hello @fengzhihenxs ,
Could you share the widget.js code for bootstrap datatimepicker .


WIP: Fix oneshot on windows

Work in progress - do not merge
Oneshot on windows does not currently behave correctly when the socket being reregistered has unread data.

lgtm

This is going to take some more (somewhat significant work).
The issue right now is that reregister unconditionally queues the token into the list of pending events, even if it as already been queued. A token should only be added to the queued list a single time.
In order to avoid scanning the current list of queued tokens before each push, a somewhat significant internal refactor will have to happen....

Hm, could you describe a little more in detail what's going on here? I'm afraid much of this is out of cache for me at least

Closing in favor of #356


Support L Variant SAMD[2|A]1E Parts

As far as I can tell - SAMD[2|A]1ExL parts are not supported by this crate - is that correct? Specifically, this is the variant with a pinout optimized for ADC/PWM usage. I am using PB02 and PB03 in a design, but am getting
error[E0609]: no field `pb03` on type `atsamd_hal::gpio::v2::Pins`
  --> src/bsp.rs:10:1
   |
10 | / hal::bsp_pins!(
11 | |     // GPIO
12 | |     PA07 { name: rf1_stat_led },
13 | |     PA06 { name: rf2_stat_led },
...  |
43 | |     }, //ALTC_SERCOM0[3]
44 | | );
   | |_^ help: a field with a similar name exists: `pa03`

when trying to use bsp_pins! on a SAMD21E17L part. I'm not sure if this is as simple as expanding the features in https://github.com/atsamd-rs/atsamd/blob/master/hal/src/gpio/v2/pin.rs#L1124 or if there are more implications.

Here is the app note that talks about the differences

I'm working on a PR here #583

I noticed these chips have separate SVDs in newer atpack files from https://packs.download.microchip.com/ , but whether we need to generate entire PACs, I am not sure.

@sajattack, we could probably just diff the SVDs. Isn't that how we decided to get rid of the memory-specifier fields of the part number?

I believe @kiranshila diffed them and there wasn't anything important. Is that right?

Closing now that #583 merged


Simplify use of Daily / Monthly Albedo in trackerdict

User requested monthly/daily (or even at timestamp) albedo input.

This can currently be done with an external script.  It would be nice to have a function that does it automatically.

This is now possible by either passing time series albedo into RadianceObj.setground(metdata.albedo) or just calling RadianceObj.setground() after you call readWeatherFile().  Fixed in 0.3.4 release


wait_for_update should be at least 100ms

https://github.com/pieeyeinc/pieeye-gtm-template/blob/1b5fbeaa0b261c3b2f4dc18c7cb2f1b6811a0f39/template.tpl#L219
When reviewing your template we've noticed that wait_for_update was set to a default less than 500 milliseconds. When specifying this it's a best practice to give some buffer time to attempt to allow for the updateConsent command to be fired. This is very important when loading the consent management script dynamically.

I have added wait_for_update at least 200 ms
Please review


Make cases-table downloadable

Describe the solution you'd like
Make cases table downloadable for the end-user, by use of existing download button.
Keeping the .csv format

Code functionality has been added and pushed to master. Created a download script to support this functionality.

Attached is the output of the csv downloaded file in excel:


raidboss: Some initial ex6 triggers

Miasmic Blast needs more testing, not sure it's 100% correct yet.

Miasmic Blast needs more testing, not sure it's 100% correct yet.

Given the fixed placement of the 9 possible blast positions, you could use some of the existing Directions stuff instead of hard-coding the safe spots.  Here's what I was playing with earlier - tested on a few pulls in game, and it also worked fine when comparing some emulator logs against log replays.
      infoText: (data, _matches, output) => {
        if (data.miasmicBlastPositions === undefined || data.miasmicBlastPositions.length !== 3) {
          return;
        }
        const centerX = 100;
        const centerY = 100;
        const cardinal16DirNums = [0, 4, 8, 12];
        const unsafe16DirModifiers = {
          center: [2, 6, 10, 14],
          cardinal: [-1, 0, 1, 4, 5, 11, 12],
          intercard: [-2, 0, 2, 3, 8, 13],
        };

        let possibleSafeSpots = Directions.output16Dir;

        for (const blast of data.miasmicBlastPositions) {
          // special case for center - don't need to find relative directions, just remove intercards
          if (Math.round(blast.x) === 100 && Math.round(blast.y) === 100)
            unsafe16DirModifiers.center.forEach((intercard) =>
              possibleSafeSpots = possibleSafeSpots.filter((dir) => dir !== Directions.output16Dir[intercard])
            );
          else {
            const blastPos16Dir = Directions.xyTo16DirNum(blast.x, blast.y, centerX, centerY);
            const relativeUnsafeDirs = cardinal16DirNums.includes(blastPos16Dir) ? unsafe16DirModifiers.cardinal : unsafe16DirModifiers.intercard;
            for (const relativeUnsafeDir of relativeUnsafeDirs) {
              const unsafeDir = (16 + blastPos16Dir + relativeUnsafeDir) % 16;
              possibleSafeSpots = possibleSafeSpots.filter((dir) => dir !== Directions.output16Dir[unsafeDir]);
            }
          }
        }

        delete data.miasmicBlastPositions;

        if (possibleSafeSpots.length !== 2)
          return;
        const [safeDir1, safeDir2] = possibleSafeSpots;
        if (safeDir1 === undefined || safeDir2 === undefined)
          return;
        return output.combo!({ dir1: output[safeDir1]!(), dir2: output[safeDir2]!() });
      },
      outputStrings: {
        combo: {
          en: '${dir1} / ${dir2}',
        },
        ...Directions.outputStrings16Dir,
      },

I apparently forgot to add some of the 16Dir-related things to Directions when I PR'd that last year, so this would need to be added to util.ts and exported:
const xyTo16DirNum = (x: number, y: number, centerX: number, centerY: number): number => {
  // N = 0, NNE = 1, ..., NNW = 15
  x = x - centerX;
  y = y - centerY;
  return Math.round(8 - 8 * Math.atan2(x, y) / Math.PI) % 16;
};

Just an option.

If you've got a working trigger for it I'd rather drop the trigger from my PR and have yours PR'd separately.


AR.js on iPhone version 11.3.1

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Hello all, when i'm trying to access a web application that run AR.js on my iPhone version 11.3.1 on Safari web browser, i get this following error: "Webcam Error name: message: WebRTC issue navigator.mediaDevices not present in your browser"
ScreenShot:

How can i fix it? :(
If the current behavior is a bug, please provide the steps to reproduce.
Try to look at the application with iPhone version 11.3.1 and Safari web browser, URL:
https://out4in.github.io/onlinear/EranShowUp/

Please mention other relevant information such as the browser version, Operating System and Device Name
iPhone version 11.3.1 and Safari web browser.

The error is pretty self-explained: AR.js app cannot detect navigator.mediaDevices on your browser, hence it cannot do anything.
I just looked on Safari Desktop 11.1 and Safari Technology Preview 11.2  for navigator.mediaDevices and it's available. I can't reproduce it on Safari Mobile 11.3, sorry.

Ok, Thank you :)
I will send message to apple to see why it's not working.

Works fine for me with iOS 11.3.1.
Maybe check if Settings > Safari > Camera & Microphone Access is enabled.

I have had this issue as well, and it had to do with very specific circumstances.
Basically, Apple is very strict with WebRTC data: any and all calls to WebRTC need to be coming from a secure (i.e. loading with HTTP instead of HTTPS) document. You can check by connecting your iPhone to your Mac and checking with the dev tools in Safari if there is a insecure file being loaded. After I fixed that issue, AR.js loaded fine on my iPhone.

Exact same problem here with Safari on  iPhone 6s with iOS 11.4
Tested using demo at: https://jeromeetienne.github.io/AR.js/three.js/examples/mobile-performance.html
Safari has Camera and Microphone Access.
Error:
Webcam Error
Name:
Message:WebRTC issue-!
navigator.mediaDevices not
present in your browser

I think this is an issue with old demo code. If you run the demos directly from Github they tend to work. It is when you pull the code from master and run it that the camera permissions aren't called.


Add the flag 'TEST_TYPE' to avoid integration tests silently skipping some test cases

Fixes issue #1421
The python integration tests may be silently skipped because the environment isn't setup right.
Add the test flag 'TEST_TYPE=nightly' to avoid integration tests silently skipping tests.
Signed-off-by: Tim Liu<EMAIL_ADDRESS>

build

Does this PR completely resolve #1421  or only partially address it?  If the former, we should update the description to say "Fixes" so Github will auto-close the issue when this is merged.

Does this PR completely resolve #1421 or only partially address it? If the former, we should update the description to say "Fixes" so Github will auto-close the issue when this is merged.

Thanks for your reminding @jlowe
The PR is partially address #1421 on the nightly IT pipeline,
Will update this PR to set the flag for the DB nightly pipeline, after https://github.com/NVIDIA/spark-rapids/pull/2061 get merged.
And will change to Fixes when the PR are ready.

build

Fixes https://github.com/NVIDIA/spark-rapids/issues/1568


Add example of table with fixed columns

as title

Codecov Report

Merging #335 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #335   +/-   ##
=====================================
  Coverage      55%    55%
=====================================
  Files          21     21
  Lines         440    440
  Branches       97     97
=====================================
  Hits          242    242
  Misses        170    170
  Partials       28     28

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 60e95d9...b438903. Read the comment docs.

Thanks a lot! It's online at https://reactabular.js.org/#/examples/fixed-columns.


Computed properties

I would like to add computed properties to a resource, which don't map directly to the database structure. My use case is that I want to perform a count in SQL so I don't have to fetch all resources in the client application (using Ember) just to perform a count.
Model 1 - company
{
"id": "foo",
"name": "",
"stores": ["bar"]
}
Model 2 - store
{
"id": "bar",
"name": "The First Store",
"products": ["baz"]
}
Model 3 - product
{
"id": "baz",
"name": "A Computer"
}
Now I would like to display the amount of products for a single company. I have a database with ~1.000 companies, ~5.000 stores and ~10.000 products. So instead of fetching all stores and products of a company in order to show the amount, I want to query the database directly when fetching a model (because a single SQL-query is much more performant in this case).
If I add the 'amountOfProducts' property to the record in the 'ouput' hook of the company model, it is added under the meta object.
So my question is, how can I add computed properties to a request response; which do not directly map to the database?
Many thanks in advance!

I have thought about adding this property to the meta output, but how should I do this using Fortune? It looks like only the count property is supported, and even automatically added.
https://github.com/fortunejs/fortune-json-api/blob/master/lib/index.js#L202
Is there a way to add meta-properties per resource in the output hook?

I'm not sure I understand the question. When you add properties on a record that aren't defined on the record, they end up serialized in meta.
You are asking about modifying the top-level meta object? This is very specific to this serializer. I'd suggest subclassing it and overriding the showResponse method.

It seems I was facing the same use case, i.e. what about derived data from the model. I updated the model, created a custom adapter which intercepts the response and updates it with the fields I needed.


Do not inject namespace to additional resources using Manifestival's namespace injection

In EKB controller, we use Manifestival to create resources that are in "additional resources". When we have the RoleBinding like this in the Manifest:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: knative-prometheus-k8s
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: knative-prometheus-k8s
subjects:
- kind: ServiceAccount
  name: prometheus-k8s
  namespace: openshift-monitoring           # --> changed

will become:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: knative-prometheus-k8s
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: knative-prometheus-k8s
subjects:
- kind: ServiceAccount
  name: prometheus-k8s
  namespace: <user-namespace>        # --> changed

This is done here https://github.com/manifestival/manifestival/blob/82196bd303e23c641fda3d002c0663b892092e9c/transform.go#L66-L73
To prevent this, we don't run the Manifestival's namespace injection on "additional resources". We can inject namespaces to them with templating anyway.

/cherrpick release-1.9

/test unit-tests

/retest

/retest

/test reconciler-tests-namespaced-broker
/test unit-tests

/test reconciler-tests-namespaced-broker

/test reconciler-tests-namespaced-broker

Failed Pod (data in YAML format) - chaosduck-ff4675f47-wmjx6

:duck:


Having trouble getting a strong signal

I'm using this hardware
ESP board - https://www.amazon.com/gp/product/B09XDMVS9N/ref=ppx_yo_dt_b_search_asin_title?ie=UTF8&th=1
Transceiver - https://www.amazon.com/gp/product/B09CTRYV4L/ref=ppx_yo_dt_b_search_asin_title?ie=UTF8&psc=1
Antenna - https://www.amazon.com/gp/product/B072Q7319V/ref=ppx_yo_dt_b_search_asin_title?ie=UTF8&psc=1
I can't get a dBm better than -66 and I can't get anything with my RX bandwidth below 125kHz.  Also, if I press the RTS remote 30 feet or more away from the ESP Somfy RTS device it doesn't register. I was under the impression that the remote should work from 75-100 feet away. Am I just being unrealistic? I have a decent size house (5,500 sq ft) so I need the ESPSomfy RTS to pickup remotes from a decent distance in order to keep the shade position in sync in HA. For context, my ESP Somfy RTS is sitting right by a shade. If I'm 25-30 feet away from the shade my remote will open the shade but the remote button press doesn't register on the ESPSomfy RTS
Should I be getting better results than this? Any reason I need my RX bandwidth so high in order to pick up a signal?

This should not be.  I am dealing with a much larger distance and I have many obstacles that should impact the signal.  In every instance I have found the remote to have less distance than the device.  That being said, I do have some transceivers, even from the same batch that perform better than others.  Maybe that is why you often find them in two packs.
If it were me I would buy another transceiver to see if I could get it to improve.

You could have an interference issue but you could also have power issues.  It is very odd that you are not getting better receive numbers.   I am adding some additional filters for the pulse detection to reject frames that have been clobbered earlier but that won't change the RSSI values.

I posted a beta firmware to the repo but it is not yet released.  In this versions I reset the pulses faster when there is interference.  Load the firmware and application bin files from the repository (not the release) and see if the capture improves.
As an aside you can also install multiple ESPSomfy-RTS devices if there are some remote areas.  My roller shades are outside in an outdoor living space and the transceiver is located inside the house on the second floor and it never misses a command.  That antenna you turned me on to allowed me to move the transceiver into my server closet.

Thanks for all the help. I tried the beta software, it helped a little.
The transmission distance is great! It will open/close a shade from across my house.  It just doesn't pick up the remotes from very far. Maybe it's the remotes I have (Situo 1 RTS).  For now, I just decided to build 3 more and put them in different places around my house.

Somfy remotes do not transmit a very strong signal.  In my case I have no other 433Mhz stuff floating around but I do get really good receive abilities.  I did install in a neighbor's house that is just under 5k sqft and it has no problem hearing the remotes that are in each room.  Those are all Situo 5s and Telis 16s spread over 30 shades.  I built them a second device but they didn't need it so it sits here on my desk as a test device.
My shades are down one floor and outside through 1.25inches of solid travertine, a teak ceiling, and two stucco walls.  All told just over 7400sqft and it never misses a beat.  Funny I only have 3 shades but they span an area of over 40 feet.  I'm starting to think that I need more shades.
I wonder if there is a lottery for the transceivers since they vary or if it has something to do with interference.  Either way it is pretty cheap to build multiple devices and they are not very big.

@bradphillips921 create another issue if you do not get sorted out but I suspect multiple devices will solve it for you.  I will keep track of these where the receive is weak.


How to access the bake-action cached image in subsequent jobs?

Hi
I read #81 which explains how to access bake actions cached images in subsequent steps but I wanted to do the same in subsequent jobs, not steps.
Would it be possible?
I tried the approach above but it does not seem to work.
So currently, my only possibility is to push to ghcr.io for testing purpose.

Hi, found the solution, helped by @Diman0, just need to re-run the bake-action in the next job with the cache-from set to previous job cache-to value...


issue when running train_vo.py

the error shows 'No module named eval_tartan'
Traceback (most recent call last):   File "/home/kefei/workspaces/MultiSlam_DiffPose/train_vo.py", line 19, in <module>     from eval_tartan import validate ModuleNotFoundError: No module named 'eval_tartan'
And I can't find the file eval_tartan.py.
Did I miss some steps?

 This is just to validate the SLAM training, you can comment out the validation code in train_vo.py if you want.
I've added the file in this branch: add_missing_file. I'll merge to main shortly.


monitoring training costs

add functionality to

monitor GPU utilisation
stats about power consumption

As I mentioned on Mattermost, I think these are all available through nvidia-smi, e.g. nvidia-smi stats gives you a csv stream of stats.
I also quickly checked LUMI, which has a similar utility called rocm-smi, and something like rocm-smi -fPtu --showmemuse --showvoltage --json seems to give you a snapshot at the moment of calling.

For NVIDIA there is an energy consumption counter that can be checked before and after the training process:
#!/usr/bin/python3
from pynvml import (
    nvmlInit, nvmlDeviceGetCount, nvmlDeviceGetHandleByIndex,
    nvmlDeviceGetTotalEnergyConsumption, nvmlShutdown
)

nvmlInit()

deviceCount = nvmlDeviceGetCount()
for i in range(deviceCount):
    handle = nvmlDeviceGetHandleByIndex(i)
    energy = nvmlDeviceGetTotalEnergyConsumption(handle)
    print(f"GPU {i}: {energy} mJ")
nvmlShutdown()

requires:
pip install nvidia-ml-py --user


Refactor and modularize tests

In this PR, the tests have been refactored, rearranged, and de-duplicated.
Tests should be have the layout similar with the backend .
This PR depends on the changes  #643
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Left mainly minor comments.
I double-checked the imports and verified that no cross imports are happening between module tests.
Tests are green locally, disabling the modules 1 by 1 properly exclude the respective test suites :)

I will merge this after merging the frontend into modularization-main


NoInvert Background Image - Inversion Fix Editor

I've permanently adopted this extension for my personal browsing, and it's 100% saved my eyes. Thank you.
However, I'm having some trouble tweaking Reddit to my liking, even with the Inversion Fix Editor. A lot of subreddits have banner and sidebar images made visible with the background-image property -- they're not actually in img tags. Not quite sure what I should add to the "noinvert" list for these in the Inversion Fix Editor, or if if it's possible. Any tips would be great.

You should pick banner element with element picker in Chrome Dev Tools and choose a CSS selector. Then use "invert": "selector" to invert the banner.
https://github.com/alexanderby/darkreader#using-dev-tools

Ahh, thanks for the tip to invert if it's already inverted. I believe I've figured it out now!

Don't forget to make a Pull Request, so that all users can use your fix.

Will do!  I want to make sure I've figured something out that works for more than the subreddits I'm subscribed to.

Okay, I'm stumped again.
Let's look at https://www.reddit.com/r/knitting
In their sidebar is a picture of Zoidberg knitting. If you inspect the element, the image is actually the background-url property on the side CSS class. I can easily get the whole sidebar un-inverted by simply adding ".side" to the "invert" list for reddit.com, but that's not what I want.
I suspect the answer lies somewhere in the  "[style*='background:url'], [style*='background-image:url']", lines that are part of the common declaration. I have tried putting them in both the invert and noinvert lists for all of reddit, and it doesn't change anything. I tried removing them from the common section and reloading the page, still nothing. If it were in an actual image tag I would just use a selector like ".side > img" but it's not so I can't. I've done enough research to believe that ".side[style*='background-image:url']" should be the proper selector, except that it's not inline CSS but inherited CSS so it doesn't appear to be picking it up.

Dynamic THeme mode now handles it well.


Convert airr library to data.table for performance

Should we convert to data.table for I/O. Seems to be faster for >1M rows.

Closing as stale.


Error: spawn UNKNOWN

Hello,
I get the following error
Error: spawn UNKNOWN
    at exports._errnoException (util.js:890:11)
    at ChildProcess.spawn (internal/child_process.js:302:11)
    at exports.spawn (child_process.js:379:9)
    at C:\Users\dinya\.atom\packages\linter-pep8\node_modules\atom-linter\node_modules\sb-exec\lib\index.js:15:53
    at Object.<anonymous> (C:\Users\dinya\.atom\packages\linter-pep8\node_modules\atom-linter\node_modules\sb-exec\lib\index.js:14:18)
    at undefined.next (native)
    at step (C:\Users\dinya\.atom\packages\linter-pep8\node_modules\atom-linter\node_modules\sb-exec\lib\index.js:84:191)
    at C:\Users\dinya\.atom\packages\linter-pep8\node_modules\atom-linter\node_modules\sb-exec\lib\index.js:84:368

Versions:

linter-pep8 1.3.0
atom 1.9.8
win7 x64

I'm getting the exact same error, installed autopep8 with pip, win10

I have the same issue.

If you put the output of where.exe pycodestyle (Windows) or which pycodestyle (*NIX) into the "Executable Path" setting, do you still get this message?

@Arcanemagus,

pycodestyle is not nessesary for the linter-pep8 working, but pep8 executable file is needed in the PATH or in the linter-pep8 config.
I cann't reproduce the error under linter-pep8 v1.3.2 Win7 x64, Atom 1.10.2

@dinya pep8 has been renamed to pycodestyle, hence why I said that above 😉. In any case since you have got this working I'm marking this as closed.


Use the search illustration

What

[ ] Use the search illustration from Quentin

Part of

#991

@g123k has added something animated. Let's close this.


Intellisense breaks when targeting multiple frameworks

When working in a .NET core app with multiple target frameworks:
  <PropertyGroup>
    <TargetFrameworks>netstandard1.6;net40</TargetFrameworks>
  </PropertyGroup>

dotnet restore -> dotnet build compiles successfully, but Ionide doesn't recognize any other files.  It recognizes any types/modules/functions defined within the same file, but when opening a module in another file, I get "namespace or module Xyz is not defined".
If I then replace my target framework in the fsproj as follows:
  <PropertyGroup>
    <!--<TargetFrameworks>netstandard1.6;net40</TargetFrameworks>-->
    <TargetFramework>netstandard1.6</TargetFramework>
  </PropertyGroup>

delete my obj folder, dotnet restore -> dotnet build then all the intellisense starts working again.
Environment: Win10, Ionide 2.25.9, VS Code 1.11.2
related issues: #278

Cc: @enricosada

@Krzysztof-Cieslak problem is, fsc args are specific to a target framework.
ionide should ask fsac for a specific framework ihmo (ref https://github.com/ionide/ionide-vscode-fsharp/issues/278), that's really needed moving forward.
meanwhile dunno, i can try read that property, and use the first netstandard* or netcoreapp* if nothing is specified.
I think is a sensible default.
added https://github.com/fsharp/FsAutoComplete/issues/162 to track that

Oh yeah, I do realize that choosing target etc should be real fix for that. But I still don't know how to handle it well from UX point of view.
I think that choosing first target framework from the list should be sensible default and decent workaround for now

I'll do that, like linked issue.
But how the others extensions do something similar? Config file? Where is s good place for transitory config?

I'd say it's probably some config file, UX wise it could just be a command like ctrl/cmd+shift+P > Choose TargetFramework  > (scan all fsproj for possible frameworks) present list to choose from > update config

support in FSAC will be implemented by https://github.com/fsharp/FsAutoComplete/pull/168
atm will be used the first, so if <TargetFrameworks>netcoreapp1.0;net461</TargetFrameworks> => netcoreapp1.0
Starting from that, FSAC will use the correct (same as Build target) framework for all project references.
I dont have found yet a satifying way to specifiy my preference in vscode (settings, env var, etc), but FSAC is ready to accept the choice, if we want to improve the UX.
Two possible:

a settings somewhere to choose the initial target framework (better to paired with the choosen starting project  becase matter a lot of project ref resolution)
with workaround disabled (a setting? passed to FSAC in initialization?), a flow like:

vscode ask FSAC to parse the fsproj
FSAC reply with error, saying is crossgen and giving the list of avaiable targets frmeworks (FSAC know that)
vscode ask used (with a combo?) what use
vscode retry ask FSAC to parse of fsproj with the choosen target framework

Initial support for multi-targeting released in 2.27.5 - at the moment we just default to first target.


Change traverseModules to return modules with their dependencies

Companion of https://github.com/swiftlang/sourcekit-lsp/pull/1683

Opening the SourceKit-LSP workspace using a debug build of sourcekit-lsp takes about 35s because traverseModules generates about 800,000 callback calls. All SourceKit-LSP needs now are the modules and their dependencies, so we can refactor traverseModules to just return that information avoid visiting the same modules multiple times. With theses changes getting all modules and their dependencies only takes ~0.013s.
rdar://136107035

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test

@swift-ci Please smoke test

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please testhttps://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test Windows

https://github.com/swiftlang/sourcekit-lsp/pull/1683
@swift-ci Please test


Add lightfm and implicit training examples

resolves #562, resolves #561

rerun tests

do we want to have GPU tests for implicit and LightFM models?

do we want to add GPU testing of implicit and lightfm? shall we addd the requirements to CI image for that? Or shall we go with CPU testing for now and rely on the GH actions?

rerun tests

@bschifferer I see that the implicit/lightfm tests are no longer skipped in the jenkins runs (though I am unsure why they were skipped in the previous run 5 days ago)

The CI runner image didn't get successfully built until Friday last week, so I'm guessing the new packages hadn't been part of a successful CI runner build yet

rerun tests


review comments within  elements

From<EMAIL_ADDRESS>on September 18, 2012 16:19:43
Make sure the comments within <desc> elements (in the ODD) have been incorporated into the guidelines.
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=114

From andrew.hankinson on October 17, 2012 07:52:52
Labels: Milestone-ReleaseNext

From<EMAIL_ADDRESS>on June 12, 2014 14:28:16
Labels: -Priority-Medium Priority-Low

From<EMAIL_ADDRESS>on January 20, 2015 10:32:28
This issue was closed by revision r1270 .
Status: Fixed


2.1 family doesn't get updated

Hi Simon,
I think I found a bug in the 2.1 version.
So I have a extension that check if a certain family is in a position:
fun Position.findEntitiesInPosition(family: Family): List<Entity> {
    val entities = mutableListOf<Entity>()
    family.forEach {
        if (it.has(Position)
            && it[Position].x == x
            && it[Position].y == y
        ) {
            entities.add(it)
        }
    }
    return entities
}

I use this to see if I can move to a target position
class MoveSystem(
    private val maxWidth:Int = inject("maxWidth"),
    private val maxHeight:Int = inject("maxHeight")
) : IteratingSystem(family { all(Position, Moving).none(Dead)}) {
    override fun onTickEntity(entity: Entity) {
        val originalPosition = entity[Position]
        val targetPosition = entity[Moving].target
        // position is within the limit and position is not occupied
        if (targetPosition.positionWithin(maxWidth, maxHeight)
            && targetPosition.findEntitiesInPosition(world.family { all(Position, Solid) }).isEmpty()){
            originalPosition.x = targetPosition.x
            originalPosition.y = targetPosition.y
        }

        entity.configure { it -= Moving }
    }

So for the version 2.0 this was working fine but I recently upgraded to 2.1 and now it is failing the test below:
    @Test
    fun shouldMoveIntoADeadObject(){
        val x = 5
        val y = 5
        val original = Position(x, y)
        val target = Position(8,8)
        val entity = world.entity {
            it += original
            it += Moving(target)
        }

        val toDie = world.entity{
            it += target
            it += Solid()
        }

        world.update(1f)
        // Check I cannot move
        Assertions.assertEquals(original, Position(x,y))
        Assertions.assertNotEquals(original, target)

        with(world){
            toDie.configure {
                it -= Solid
                it += Dead()
            }
            entity.configure { it += Moving(target) }
        }

        world.update(1f)
        // Check it should have moved
        Assertions.assertEquals(original, target)
    }

Hm I wouldn't know which change was introduced in 2.1 that could cause that 😅 weird.
Anyway, can you let me know please which assertion exactly is failing? Also, which family doesn't get updated? The one from the system or the one that you create inside the onTickEntity method?
Also, with 2.1 there is a filter function that you can use to remove your findEntitiesInPosition function. But first let's find out what is not working and why 😉

It is the last assertions
// Check it should have moved
Assertions.assertEquals(original, target)

The family still returns the toDie entity as it still have the Solid component but if I inspect the entity doesn't have it
I have modified the system to see the entity snapshot with:
class MoveSystem(
    private val maxWidth:Int = inject("maxWidth"),
    private val maxHeight:Int = inject("maxHeight")
) : IteratingSystem(family { all(Position, Moving).none(Dead)}) {
    override fun onTickEntity(entity: Entity) {
        val originalPosition = entity[Position]
        val targetPosition = entity[Moving].target
        // position is within the limit and position is not occupied
        val entityInTarget = targetPosition.findEntitiesInPosition(world.family { all(Position, Solid) })
        entityInTarget.forEach { println(world.snapshotOf(it)) }
        if (targetPosition.positionWithin(maxWidth, maxHeight)
            && targetPosition.findEntitiesInPosition(world.family { all(Position, Solid) }).isEmpty()){
            originalPosition.x = targetPosition.x
            originalPosition.y = targetPosition.y
        }

        entity.configure { it -= Moving }
    }

And getting:
This one before removing the Solid
[com.nestosoft.spacerl.gamelib.components.attributes.Solid@d62fe5b, Position(x=8, y=8)]
This one after removing the Solid
[com.nestosoft.spacerl.gamelib.components.attributes.Dead@539d019, Position(x=8, y=8)]

I also need your "positionWithin" function please

sure!
fun Position.positionWithin(widthLimit:Int, heightLimit:Int):Boolean {
    return x in 0 until widthLimit
            && y in 0 until heightLimit
}

It is indeed a bug that got introduced with 2.1 but it only happens, when family iteration is happening in a nested case like you have it.
Your system is iterating over its own family and then you are also iterating over the "special" family that you retrieve from your world.
Unfortunately, I broke something in this scenario when trying to cleanup some code and avoid some code duplication. I will have a look at that within this week (I hope) and try to release a fixed version.

Additional info for me (and of course you if you are interested!):

In 2.1 I introduced a lot of standard collection functionality for entities in the EntityBag/MutableEntityBag classes (before it was just an IntBag)
Inside a family we use the bag for entity iteration and due to performance this information is stored in two separate variables

we have an entities BitArray which is immediately updated when an entity becomes a part of the family or gets removed. The BitArray is very fast to insert/remove entities and to check if an entity is part of the family
we have an entities bag which is NOT updated immediately because its main purpose is for iteration. The bag is way faster for iteration (=array like) when compare to the BitArray but is way slower for the other things mentioned above

Therefore, it is necessary to update the entities bag "manually" when needed. We also need to be careful to not update the entities bag when an iteration is currently in progress over those entities
in the latest version this manual update was broken when it comes to nested operations
the important function that needs to be called is updateActiveEntities

In some cases like in forEach this method was called all the time while in other functions it was surrounded by a condition !entityService.delayRemoval || entitiesBag.isEmpty. I thought I solved that in a cleaner way and simply moved this inside the getter of the bag:
@PublishedApi
    internal val mutableEntities = MutableEntityBag()
        get() {
            if (!entityService.delayRemoval || field.isEmpty()) {
                // no iteration in process -> update entities if necessary
                updateActiveEntities(field)
            }
            return field
        }

BUT, this is not correct because e.g. in a nested forEach the updatedActiveEntities is not called and therefore the family not updated. However, it is safe to update the nested family because there is no iteration happening over that family.
What I need to do:

write a test case for this scenario that will fail
fix the problem (either by reverting those changes or fixing the new approach. Preferably the later one to avoid the code duplication)
release bugfix version: most likely 2.1a

version 2.2 should be soon available on maven


success callback not called if usign sap.ui.model.odata.v2.ODataModel.submitChanges without batch

Looks like a callback function will not be called in case of using the v2 ODataModel without batch option.
Used Version is SAPUI5 1.28.12.
Debugging ODataModel I come into
ODataModel.prototype._processRequestQueue = function(mRequests, sBatchGroupId, fnSuccess, fnError)
There is the following if;
if (this.bUseBatch) {
in case of "undefined" or "false" it looks like there is no call on the fnSuccess parameter.
Is there any workaround? By upgrading to the V2 model I would avoid using batch from the beginning.
Update

I am using two way binding.
same with fnError
seems to be the same with: OPENUI5 1.30.4-SNAPSHOT

The api docs for submitChanges are very clear:
"Important: The success/error handler will only be called if batch support is enabled. "
See https://openui5beta.hana.ondemand.com/#docs/api/symbols/sap.ui.model.odata.v2.ODataModel.html#submitChanges for details. In other words: this is the expected behavior. I am nor sure if this will be changed...

Oh I see. Have not noted that "Important" note because it looks like to be part of the "error" description, sorry for that.
Anyhow it prevents me to upgrade to the v2.ODataModel step by step :-/
Also if there is an option to not using batch, how to submit changes than? A feedback if it was successfull or not would be helpful in any case?
Otherwise the "non batch" option seems to be useless?

Hi,
thanks for reporting this!
I will let the control developer know that we should think about putting the "Important" Note directly to the success handler.
Maybe he can also provide you an answer for your other questions.
Internal incident id:1580084957
Best regards,
Janina


Apply ignore apps with unknown repo during install process

closes #1

Hi Richard, thank you for your contribution. looks good to me. I'll merge it


Problem with “serverless-offline” plugin

Following guide verbatim - problem encountered at
https://serverless-stack.com/chapters/add-a-create-note-api.html
error thrown at first “serverless” command:
$ serverless invoke local create --path mocks/create-event.json

  Serverless Error ---------------------------------------

  Serverless plugin "serverless-offline" initialization errored: Unexpected token )

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information ---------------------------
     Operating System:          linux
     Node Version:              6.5.0
     Serverless Version:        1.48.2
     Enterprise Plugin Version: 1.3.2
     Platform SDK Version:      2.0.4

I realise this may be an issue with the serverless-offline plugin, however it’s currently a blocker for anyone attempting to follow the serverless-stack tutorial

Update: Following the steps on a new system works. Seems the problem was specific to my environment, closing


Provenance not shown when you host the viewer results via GitHub Pages

Look here: https://e3sm-project.github.io/acme_diags/sample_output/modTS_vs_modTS_3years/prov/
According to this error, we need to have an index.html. This isn't an issue when the diags are hosted on the other machines.

Implemented in 6e24b15360a3404303b2c1868dbf789f450aa30e, but GitHub Pages doesn't allow for server-side code, so this isn't allowed. Trying another approach.

As shown in 1c2b025a5d510884f2a6b7a8751df425a010dc93, for any example we're hosting via GitHub Pages, we'll just have a direct link to the provenance as shown on GitHub. It's not worth the effort getting this working. The index.php that is now created as a result of this issue will cover most cases for other servers, so this effort was worth it.


Server side render need 3-4 seconds using ruby on rails on AWS EC2 instance

We have a React/Redux app and want to generate html from sever side on ruby on rails for search engine. The size of the JS bundle file is about 500kb.
In my local environment(Mac Pro), the performance is normal: 1s. But after deploy to AWS EC2(m3.medium), the performance is bad: 3-4s per time.
Does the performance is normal on AWS? If not, does anyone know what cause the performance so bad on AWS?
Thanks a lot!

Finally, I found that ExecJs choose NodeJS as JS runtime by default. After I uninstall NodeJS and change it to V8, the performance is better than before 1-2s. But after use ab to stress test the page, our server become worse gradually.


OS_PEND and OS_CHECK are backwards

For functions that have a "timeout" parameter, two special values are defined: "OS_PEND" can be used to wait forever, and "OS_CHECK" can be used to not wait at all / return immediately.
The issue is that "OS_PEND" is defined as "0" and "OS_CHECK" is defined as "-1".  These should be the other way around.
The problem is not so much when the special values are used but rather when they are NOT used and the timeout is computed by subtracting a current time from a deadline time.  If the result of that computation happens to be zero, the OSAL will interpret this as "OS_PEND" and will end up waiting forever.
Simply swapping the definitions of OS_CHECK and OS_PEND would make the operation much more logical by making a computed timeout of zero get the behavior of OS_CHECK and not OS_PEND.

Imported from trac issue 16.  Created by jphickey on 2015-01-14T16:31:19, last modified: 2015-11-20T16:22:16

Trac comment by glimes on 2015-01-29 15:14:59:
Click [changeset:a274d24 here] to see the changeset.

Trac comment by glimes on 2015-01-29 16:12:39:
I have reviewed [changeset:a274d24 changeset a274d24] and recommend ACCEPT.
There is some risk here: using 0 and -1 in source code, despite the API presenting macros encapsulating those values, is just the sort of lazy shortcut one finds in source code, especially when written under time pressure. I would expect the '''''Release Notes''''' of the next OSAL release to mention this general change to the timeout parameters in the API, recommending that software that wishes to obtain "block forever" or "nonblocking" behavior assure that it is explicitly using the OS_PEND or OS_CHECK macros.

Trac comment by mlogan on 2015-01-30 19:45:03:
The change was reviewed and I also ACCEPT.
It should be noted that no where in OSAL does the original definitions cause a problem (that I can find).    It is the user's code (or possibly a unit test) that revealed the inconvenience of these settings.
I agree that this change should be clearly stated in the release notes - on the off chance that other developers were inappropriately hard coding the use of -1 and 0 for some reason and assumed their meaning to be like the old defines.
If my original statement is correct, this should be reclassified as an ENHANCEMENT, not a defect.

Trac comment by jphickey on 2015-01-30 21:55:12:
Per Mike's suggestion I have changed this to an enhancement.
I suspect anyone who was lazy and used the hard-coded numbers instead of the symbolic names would have already been scratching their heads as to why a zero timeout did not correspond to passing in a value of "0".
The main advantage is that this makes the user code simpler and easier to read.  If calculating a timeout against a deadline, one must still check for a negative result, but anything else is OK. This just simplifies the application code a bit, and makes it slightly more efficient too.  If any code is already checking for both zero and <0, then it should still work after this swap just as it has.

Trac comment by jwilmot on 2015-02-11 10:57:09:
The change was reviewed and I also ACCEPT. But looking at the uses of OS_PEND and a commentators statement "timeout is computed" I see another potential issue. The prototype int32 OS_Milli2Ticks (uint32 milli_seconds) is called with int32 timeout from OS_QueueGet, so any timeout with -2 or less from the computation will be treated as a large unsigned number.

Trac comment by glimes on 2015-02-12 15:14:03:
Accepted by #18 for integration candidate.

Trac comment by acudmore on 2015-02-11 14:12:01:
Looks fine to me , I ACCEPT

Trac comment by glimes on 2015-03-04 15:22:25:
Confirmed that the fix from [changeset:a274d24] is present in the
current development branch of OSAL, [changeset:5aae372].


Check and create multi-tenant index with alias for Update and Delete requests. Try to find a name for the multi-tenant index if index/alias with ".kibana_..._#" already exists

opendistro-for-elasticsearch/security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Enhancement

Github Issue # or road-map entry, if available:

Description of changes:
Include Update and Delete requests into the type of requests for which it is necessary to check if it is necessary to explicitly request mutli-tenant index creation with alias to avoid index being auto-created.

Why these changes are required?
By using Kibana API (not UI) it is possible to trigger Update and Delete requests that will create multi-tenant index without an alias and may cause Kibana migration logic to break.

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

Could you pls add unit tests for this change?

LGTM. I like the comments added in the PR. It significantly improves readability.
I think we should promote comments in all the PRs in the future.
In order to avoid regression in the future, I highly recommend to raise a seprate PR and add more UTs to cover the edge cases that this changes fixes.


Fix ExtractTSDInfoARM64 for Fedora 39 libc on aarch64

Fedora 39 glibc's __pthread_getspecific has a few properties that aren't currently handled correctly, which this patch addresses.
The basic sequence (with irrelevant stuff removed) looks like this:
mov     w1, w0
mov     x0, #0xfffffffffffff9d0
mrs     x2, tpidr_el0
add     x0, x0, w1, uxtw #4
add     x2, x2, x0
ldr     x0, [x2, #8]
ret

The previously-unhandled pieces are:

Constants can be loaded into a register which will then be used as addends to another register, e.g.
mov     x0, #0xfffffffffffff9d0
...
add     x2, x2, x0

To address this, we add the TSDConstant discriminant to track registers with a known constant value.

The add source operand can end in uxtw #n; this is similar enough in meaning to lsl #4 that we can treat them the same.

Timo Teräs @.***> writes:

@fabled commented on this pull request.
Some simplications requested if possible.
Can you also add the extracted code as a test case in
tpbase/libc_test.go ?
Sure, I'll add a test.

	case aa.Imm64:

		regs[destReg] = regState{

			status:     TSDConstant,

			offset:     int(val.Imm),

			multiplier: 1,

		}

		setImm = true

	case aa.Imm:

		regs[destReg] = regState{

			status:     TSDConstant,

			offset:     int(val.Imm),

			multiplier: 1,

		}

		setImm = true

	}

	if !setImm {

Could this be just the default case (and we get rid of setImm)?
Sure, that is definitely better. Thanks for the suggestion.

@@ -169,12 +170,31 @@ func ExtractTSDInfoARM64(code []byte) (TSDInfo, error) {
resetReg = destReg
switch inst.Op {
case aa.MOV:

	// Track register moves

	srcReg, ok := ah.Xreg2num(inst.Args[1])

	if !ok {

		continue

	var setImm bool

	switch val := inst.Args[1].(type) {

	case aa.Imm64:

			case aa.Imm64, aa.Imm:

And remove the duplicate case block?
If I apply the suggestion, I can no longer use val.Imm (presumably
because it has a different type in the two cases, uint64 vs. uint32).
--
Reply to this email directly or view it on GitHub:
https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/168#pullrequestreview-2330220293
You are receiving this because you authored the thread.
Message ID: @.***>

Now that this has two approvals, can we merge it? Or what is the procedure for me to request a merge?

Usually second person who approved merges as well unless there are still open requests for changes.

These kind of friction will hopefully reduce over time. So far I assumed that merges are done by maintainers only.

Hm yeah: taking a second look at CONTRIBUTING.md, it appears that you are right about that, actually -- sorry. I was under the impression that if at least one maintainer has approved previously, anyone with perms can merge it. Seems like the more practical approach to me; otherwise you always have to wait for a maintainer to come back and merge. If approvers aren't supposed to merge, they should probably not have the permission for it.


Case sensitivity issues

If I use this in a package:
install_requires=[ 'shapely==1.5.17.post1' ]

The package installs fine, but the pipdeptree graphviz output has an extra root node with Shapely.
Fixing my package to match the case for this library on pypi works around the issue (install_requires=[ 'Shapely==1.5.17.post1' ])

Another example: scikit-image requires pillow all lowercase, so you get an extra Pillow in the graph.

This should be no longer the case, if is please reopen.


Various changes...

This PR snowballed and contains the following changes:

Adds a camel case lint with an autofix to ensure all identifiers are correctly cased
Adds rome lint --review that contains an interactive prompt that allows you to go through each lint diagnostic and decide if you want to suppress, fix, or ignore it. Really important for onboarding and upgrades.
Normalize diagnostics. This will convert UIDs to full absolute paths. Before we were returning diagnostics with UIDs, which was fine since we would convert them under the hood when executed in the master process. But for rome lint --review all interactivity needs to happen in CLI/client process, where we don't have access to the UID map.
Moves localCommands into their own folders.
Made some command properties required.
Renamed Context inside of @romejs/js-compiler to ContextCompiler
Added keyboard shortcut support to Reporter.select so you can select single keys when going through rome lint --review instead of using arrow keys.

Unfortunately this is all in one commit as I was blasting away changes and commits when debugging rome lint --review.

CI passes so...


Move all CI services from Travis to Drone

Now that Drone seems to be working, we should move all Travis services to Drone and delete .travis.yml

Blocked by caching issues on Drone

It has been suggested that we move to services aligned with UKHomeOffice: https://github.com/UKHomeOffice/application-container-platform-bau/issues/3665#issuecomment-400615838


报错：Error in mx.opt.sgd(...) : unused argument (device = device)

您好，在运行代码时报错
deepcc.model <- trainDeepCCModel(fs, labels)
Error in mx.opt.sgd(...) : unused argument (device = device)
google下尝试了这行代码没明白什么意思
device = mx.cpu()
$device
[1] "cpu"
$device_id
[1] 0
$device_typeid
[1] 1
attr(,"class")
[1] "MXContext"
我是想把TCGA的gene表达矩阵根据公认的分子分型来预测我自己样本的分型，labels是每个样本的所对应的分型类型吗？
还有，TCGA的矩阵是microarray的还是RNAseq的比较好？如果的是RNAseq的话是用count还是tpm、fpkm好呢？
谢谢

简单回答一下。
我个人推测错误代码是MXNET产生的，MXNET改变了接口，所以DeepCC对应的调用就出错了，你可以尝试使用DeepCC开发的时候用的MXNET版本，大约是0.10.0
另外，因为MXNET对于R不是很友好，现在的测试版本讲后端改成了keras，可以安装DeepCC对应的keras分支，但是这个版本DeepCC本身的函数明明风格有变化，正式发布之前请自行查看对应的函数名。

用了DeepCC@keras之后
deepcc.model <- train_DeepCC_model(fs, labels)
Error in train_DeepCC_model(fs, labels, epochs = 100, dropout = 0.5, activation_func = "selu") :
object 'y_train' not found
不知道y_train是什么

DeepCC@keras这一版有没有简单的函数介绍，里面很多变量不知道什么意思，MXNET的旧版本找不到，现在无法进行下去了， 谢谢

keras后端的版本还在测试，没有完全定型，正式发布前会完善帮助文档。
mxnet的版本变更不是我们能控制的，在旧版本上使用是没有问题的，我们不准备兼容新版的mxnet，会一次性切换到keras。

能否提供个可用的mxnet包文件，我找了很久没找到，麻烦你了

mxnet包可以装cran的版本，所有cran包在GitHub上都有镜像，可以随便选择版本。但mxnet应该还需要单独的library。

我到了这个链接https://github.com/apache/incubator-mxnet/tree/v0.10.0-support/R-package，不知道是不是0.10.0版的R包，这个看起来也下载不了

https://github.com/apache/incubator-mxnet/tree/v0.10.0-support/R-package
我找到了这个，文件夹形式不知道怎么装进r里去
但mxnet应该还需要单独的library这一句我不太懂怎么操作

使用keras分支，出现报错，python3.8 64bit我已经安装了，操作系统win8.1 64bit
deepcc.model <- train_DeepCC_model(fs, labels)
Error in py_initialize(config$python, config$libpython, config$pythonhome,  :
  Py_CompileString - 找不到指定的程序。

你好，你现在的报错信息都是来自于依赖关系，不涉及到DeepCC本身。
1.请确定你R中的keras本身可以正常使用，我们建议使用conda安装。Windows 8.1是很古老的版本，我们不会去测试。我自己的开发环境是Debian 10和macOS 10.15，都没有问题。
2.mxnet的安装比你想的复杂，这也是我们抛弃的原因。和keras一样，mxnet的R包仅仅是一个接口，你需要对应的二进制程序，接口才能使用。但我记得旧版本的mxnet在cran上安装，虽然只支持cpu，它会自动从aws上下载对应的二进制库。cran上是可以安装旧版本的包，你可以尝试。https://codeday.me/bug/20170703/33041.html

谢谢，放弃了，包安装太复杂了，不知道能不能等到DeepCC能像常见r包简单安装的那时候

所以我们有一个网站的版本直接给大家用，源代码的版本更多的是学习目的。DeepCC还是要依赖于一种深度学习的框架，这点是没法简化掉的，DeepCC只是深度学习在分子分型问题上的一个扩展。

Hello,
I get this error when running the code
deepcc.model <- trainDeepCCModel(fs, labels)
Error in mx.opt.sgd (...): unused argument (device = device)
I googled and tried a lot but couldn't solve it.
I want to run your model on mouse data and compared its performance with another.
Could you please help me?
Another question: How I can get the final features that the model used to predict the CMS subtypes in the DeepCC online?
Thanks!

Hi,
I checked the error message and I think it is caused by mxnet. They changed the parameter name inside their function from 'device' to 'ctx'.
Since we are in the middle of changing the deep learning backend from mxnet to keras, which has official support from RStudio, we don't have test environment to re-test the new change made by mxnet. But I made a temporary patch for it. If that is the only problem here, you can fix it by installing the latest version of DeepCC from GitHub.

Thank you for your reply. I installed it 2 weeks ago. Should I reinstall it?
Is it possible to get the final features that the model used to predict the CMS subtypes in the DeepCC online?

Yes, you should do it again. I updated the repo about 20 hours ago. I guess it will fix the problem but I didn’t test it. Anyway we will release a new version soon with keras supporting.
There is a function in the DeepCC which help you do this, getDeepCCFeatures. Please check the help.

I reinstall it using : devtools::install_github("CityUHK-CompBio/DeepCC")
now after running this
fs <- getFunctionalSpectra(eps, genSets_mus)
I gets this error

Error in { :
task 1 failed - "lazy-load database '/home/raheleh/R/x86_64-pc-linux-gnu-library/3.6/DeepCC/R/DeepCC.rdb' is corrupt"
In addition: Warning messages:
1: In if (geneSets == "MSigDBv5") { :
the condition has length > 1 and only the first element will be used
2: In if (geneSets == "MSigDBv6") { :
the condition has length > 1 and only the first element will be used
However, it return me the fs.

Still, the previous error has been remained after running `deepcc.model <- trainDeepCCModel(fs, lab)`
`Error in mx.opt.sgd(...) : unused argument (device = device)`

Hi Gaofeng21cn,
After you updating DeepCC and I reinstalled it, I am getting error after running this code:
fs <- getFunctionalSpectra(eps, genSets_mus)
Error in { :
  task 1 failed - "lazy-load database '/home/raheleh/R/x86_64-pc-linux-gnu-library/3.6/DeepCC/R/DeepCC.rdb' is corrupt"
In addition: Warning messages:
1: In if (geneSets == "MSigDBv5") { :
  the condition has length > 1 and only the first element will be used
2: In if (geneSets == "MSigDBv6") { :
  the condition has length > 1 and only the first element will be used

In your online version of DeepCC, is it possible to download the features that has been used to predict the CMS subtypes in our samples?
Many thanks for your time and help!

Hi Rahil63,
From my experience, something like ".rdb is corrupt" usually can be fixed by restarting the R session. Warning messages don't matter I saw the same thing on my test machine (a Windows PC) but it works.
The online version of DeepCC only gives the results and probability.

您好，在运行代码时报错
deepcc.model <- trainDeepCCModel(fs, labels)
Error in mx.opt.sgd(...) : unused argument (device = device)
google下尝试了这行代码没明白什么意思
device = mx.cpu()
$device
[1] "cpu"
$device_id
[1] 0
$device_typeid
[1] 1
attr(,"class")
[1] "MXContext"
我是想把TCGA的gene表达矩阵根据公认的分子分型来预测我自己样本的分型，labels是每个样本的所对应的分型类型吗？
还有，TCGA的矩阵是microarray的还是RNAseq的比较好？如果的是RNAseq的话是用count还是tpm、fpkm好呢？
谢谢

对mxnet版本进行补充说明。
因为keras版本我们还在优化中，所以快速修复了一下mxnet版本不兼容的问题。主要是在新版的mxnet，指定计算设备是cpu还是gpu的参数改名叫做ctx。另外我检查了一下现在mxnet的安装方法，如果不要求gpu支持的话，3.5.x版本的R在macOS和Windows下可以直接装预编译版本的mxnet，相对简单很多，应该可以解决使用的问题。
参考：https://mxnet.apache.org/get_started?platform=windows&language=r&processor=cpu&

Hi Gaofeng21cn,
My data is RNAseq that are normalized using DESeq2 and log2 transformed.
In the example that you provided in the home page of online DeepCC (a small dataset), there are aslo negative values. I am wondering that should I do any scaling before uploading my data for DeepCC analysis?

No scaling or normalization is needed for DeepCC.

Best regards,
Feng GAO, Ph.D.,
Associate Professor
The Sixth Affiliated Hospital, Sun Yat-sen University
Guangzhou, China
在 2020年5月31日 +0800 14:10，MolecularPatholgyLab<EMAIL_ADDRESS>Hi Gaofeng21cn,
My data is RNAseq that are normalized using DESeq2 and log2 transformed.
In the example that you provided in the home page of online DeepCC (a small dataset), there are aslo negative values. I am wondering that should I do any scaling before uploading my data for DeepCC analysis?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for quick reply! Then, why there are some negative values in your example?
My data was read count and I used DESeq2 to covert it to expression value and then transformed it to log2 format. Can I upload this to the DeepCC?

Sure, just keep the format of the file right.
The format of values is not important since functional spectra used in deep learning model were calculated in a rank-based manner. The negative values in our example are simply caused by the normalization method which doesn’t matter for DeepCC.

Best regards,
Feng GAO, Ph.D.,
Associate Professor
The Sixth Affiliated Hospital, Sun Yat-sen University
Guangzhou, China
在 2020年5月31日 +0800 14:30，MolecularPatholgyLab<EMAIL_ADDRESS>Thanks for quick reply! Then, why there are some negative values in your example?
My data was read count and I used DESeq2 to covert it to expression value and then transformed it to log2 format. Can I upload this to the DeepCC?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Gaofeng,
I used getDeepCCFeatures function to get the features that are using at the end to call CMS in your model, it returns me a matrix of 80*10 without any names. Is it possible to get the name of gene sets that are using as best features to call CMS subtype?
Many thanks,
Raheleh

Hi Raheleh,
This function is designed to transform data to feature and what I think you want is select feature. It is different since concept of deep learning.
Best regards,
Feng GAO, Ph.D.,
Associate Professor
The Sixth Affiliated Hospital, Sun Yat-sen University
Guangzhou, China
FengGaoLab.org
		2021年4月16日 +0800 21:07 MolecularPatholgyLab<EMAIL_ADDRESS>

Hi Gaofeng,
I used getDeepCCFeatures function to get the features that are using at the end to call CMS in your model, it returns me a matrix of 80*10 without any names. Is it possible to get the name of gene sets that are using as best features to call CMS subtype?
Many thanks,
Raheleh
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.


feat: complete bounty requirements

This pull request was created by https://app.gib.work/i/2-CyMlKG/add-working-example-of-verification-with-ledger in an attempt to solve a bounty. Payment for the bounty is immediately sent to the contributor after merge.

I am pasting some screen recordings here one by one
Ledger signature verifications

Phantom
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/385ee20c-764a-494a-b12c-0de26b4ee71b

Solflare
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/f6d04c7b-e0c6-4202-a41b-e970660f8410

Backpack
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/e68be802-fdf0-4900-9fbd-ebb2ba054acc

Normal Signature Verifications

Phantom
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/e3eafce7-77d4-4015-ad92-85be489ec21d

Solflare
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/84b58b42-96e7-40c7-a996-dea8f80bee67

Backpack
https://github.com/pubkeyapp/solana-wallet-adapter-mantine-ui/assets/32637757/2abadcc4-a0de-4691-9a8c-5633cecb016d

Closes #5

Thanks a lot @sunguru98!


UNDERTOW-958 ensure any jsessionid in URL is updated upon login

[JBEAP-8386] : https://issues.jboss.org/browse/JBEAP-8386

Linux Build 96 outcome was FAILURE using a merge of 85110b48828be24f43833325d565dccc6638bbe5
Summary: Execution timeout; tests passed: 27, ignored: 1 Build time: 01:00:02

Retest this please.

Linux Build 97 outcome was FAILURE using a merge of 85110b48828be24f43833325d565dccc6638bbe5
Summary: Execution timeout; tests passed: 27, ignored: 1 Build time: 01:00:12

Tests pass locally.


Support for diffable data source?

Hi, thanks for this great package to simplifiy the process to setup a collectionview layout. I don't find any information about support of diffable data source. Do you support it or could you do that?

Hi, thanks for your feedback.
There is no mention about diffable data source because this library is only looking at the layout of the collection view, not its data source (UIKit did a great job a separating the layout and the data). You are free to use a classic data source (as done in the examples) or use a diffable data source, you just have to set the data source on your collection view.
Here a quick example that is using diffable data source, the data source is configured in viewDidLoad()
import UIKit
import CompositionalLayoutDSL

class ViewController: UICollectionViewController {

    private typealias SectionIdentifier = Int
    private typealias ItemIdentifier = Int
    private typealias DataSource = UICollectionViewDiffableDataSource<SectionIdentifier, ItemIdentifier>
    private typealias DataSourceSnapshot = NSDiffableDataSourceSnapshot<SectionIdentifier, ItemIdentifier>

    private lazy var dataSource = self.createDataSource()

    init() {
        super.init(collectionViewLayout: UICollectionViewFlowLayout())
    }

    required init?(coder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }

    override func viewDidLoad() {
        super.viewDidLoad()

        collectionView.register(UICollectionViewCell.self, forCellWithReuseIdentifier: "Cell")

        collectionView.dataSource = dataSource
        collectionView.setCollectionViewLayout(createLayout(), animated: false, completion: nil)
        initializeDataSource()
    }

    private func createDataSource() -> DataSource {
        return DataSource(collectionView: collectionView) { collectionView, indexPath, itemIdentifier in
            let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath)
            cell.backgroundColor = .gray
            return cell
        }
    }

    private func createLayout() -> CompositionalLayout {
        CompositionalLayout { section, environment in
            Section {
                HGroup {
                    Item(width: .fractionalHeight(1))
                }
                .height(.absolute(120))
                .interItemSpacing(.flexible(8))
            }
            .interGroupSpacing(8)
        }
        .interSectionSpacing(8)
    }

    private func initializeDataSource() {
        var snapshot = DataSourceSnapshot()
        snapshot.appendSections([0, 1, 2])
        snapshot.appendItems(Array(0..<10), toSection: 0)
        snapshot.appendItems(Array(10..<20), toSection: 1)
        snapshot.appendItems(Array(20..<30), toSection: 2)
        dataSource.apply(snapshot)
    }
}

I hope this helps


Update for low power pin missing

This closes #4 by removing the raised error if no low power pin defined.
There's a minor makefile change, maybe it's unnecessary, but also adds links for getting the ULD driver if rebuilding the binary blob.
I updated the MPY files and binary blob with version 1.3.9 of the ULD driver.

It would be nice if someone could test micropython, I haven't looked at it in a while, since last summer. I'm more interested in circuitpython.

I've found issues now that I've changed firmware so I reverted the firmware changes + MPY builds, just the useful changes remain


Expand the stages, add a specific picture for install process

I wonder if the stages stuff needs its own page linked from here


Can you give example for decrypte Mimic

Hello! Can please give me example how to decrypt example file for Mimic (test1.jpg.xxxxxxxxxx)
path: Mimic/tools/
Thanks!

Sorry! I got the scripts mixed up. I've already fixed it. :)
Please try again

Thank you! It worked! Please tell me how you generated privkey.bin. We were hacked and planted with a mimic, do we have a chance to use your utility?
In you file Mimic/attackers.txt there are emails that were written to us in an extortion lette:
v7.5
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I generated these keys myself for demo.
To decrypt your files using my script, you will need the private key that the attackers have

Thank you very much for the work done!


[ Feature Request ] Android Compiled Code as Dynamic Feature Module to reduce apk Updatable Size

Google Play Store can make file by file patching for updates, and split apks if we seperate flutter.so and app.so into two modules, since flutter.so is always unchanged unless flutter SDK update, this will save app furthur update size, since only app.so only need to be updated.

See https://github.com/flutter/flutter/issues/51522
\cc @xster

See https://github.com/flutter/flutter/issues/51522
\cc @xster

It's about asset modules, my issue is libapp.so file to seperate module, which can be updated by filebyfile patching support by Google Playstore.

Merging this into https://github.com/flutter/flutter/issues/57617


tf.boolean_mask equivalent in MxNet

In my architecture, I want to pick same of the samples in a minibatch at different stages of computation, based on a binary vector produced elsewhere in the network. In Tensorflow, tf.boolean_mask is generally used for that purpose. How can I achieve the similar effect in MxNet, is it possible?

To refine the question a bit more; is it possible in MxNet, ideally by using the Symbol API, to select a subset of the samples in a minibatch, based on a binary condition vector? For example given a tensor of the shape (A,B,C,D) and a binary vector of the shape (A,),I want to produce a new tensor of the shape (a,B,C,D), where a <= A and the selected samples correspond to the True entries of the binary vector and False entries are rejected.

mx.nd.pick

According to the documentation, mx.symbol.pick chooses the given indices on a given axis. For example:
`x = [[ 1.,  2.],
[ 3.,  4.],
[ 5.,  6.]]
// picks elements with specified indices along axis 0
pick(x, y=[0,1], 0) = [ 1.,  4.]`
In my case, I don't have the knowledge of the indices in y. I only know y as a boolean vector, entires with y=[True,False,True], etc. Will pick work in this case as well?

x = mx.nd.array([1,2,3,4,5])
y = mx.nd.array([5,4,3,2,1])
mx.nd.where(x < y, x, y)
output: ...
[ 1.  2.  3.  2.  1.]
<NDArray 5 @cpu(0)>

Is there a way to filter out the elements in mx.nd.where which do not satisfy the condition given in the first argument? I mean:
x = mx.nd.array([1,2,3,4,5])
y = mx.nd.array([5,4,3,2,1])
mx.nd.where(x < y, x, y)
Will give [1. 2.] as the output, since only the first two elements satisfy x < y

Actually, you can transform the nd to np.array with .asnumpy(), and then use numpy to do that. Slicing is not mature in mxnet.

That is the problem; I am using a lot of glue code to implement this conditional behavior for minibatches. I keep the track of the sample indices, select ones which satisfy a certain condition and route them into separate network subsets. I am using .asnumpy() conversion and use multiple sub-networks and executors for this purpose. This costs me nearly a ten folds slower execution compared to baselines. Any chance that slicing will be improved in the near future?

You can use Anaconda Python to accelerate the calculation of numpy as it is compiled with the Intel MKL. You may got less performance loss.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


making WebEngine part of PyQt5.6.0 wheel work

As stated here https://github.com/spyder-ide/spyder/pull/3198#issuecomment-223792597 , PyQt5.6 is now supported but the WebEngine part, the help windows, doesn't, because of malformed wheel.
Question:

what files should be moved manually to get it working ?

What do you see printed in the terminal when Spyder is started?

nothing, wherever I look:

dos window from which I launch spyder,
console,
other windows.

Like if it was just not connected (or shown).

oups! side-papercut, if you "launch execute in current interpreter while looking at the console tab, spyder hangs.

I'm closing this one because it's not our fault and we can't do anything about it.

After eating my bad soup "binary PyQt installer":

I have QtDesigner and QtLinguist back, ....
but trunck spyder with trunk qtpy fails (not even the welcome spyder splash screen image)

Spyder needs to be updated to work with qtpy master. Please use qtpy 1.0.2 for now, it should work without problems :-)

Well, today I have hard time finding somethig that work. maybe Ipython5.0b4 with its "win_unicode_console" is creating issues for Spyder

But we're embedding qtconsole in Spyder, not IPython directly....


Polylines on 3D Tiles demo doesn't work in all browsers

I'm not sure if there is a supported check for this feature, or if it's just broken, but the Polylines on 3D Tiles demo doesn't work in Safari on the Mac I was using for testing
http://localhost:8080/Apps/Sandcastle/index.html?src=Polylines on 3D Tiles.html&label=All
If there is a supported check, we should be using it and notifying the user that the feature isn't supported on this browser.  If there isn't a supported check, why is this Sandcastle broken on some browsers?
CC @lilleyse @likangning93

Update: Apparently it works if you switch to Photogrammetry and then back to BIM, but does not work on BIM on initial load.  Race condition?  I have a laptop you can use for debugging.


Updated sklearn and xgboost versions

Issue #, if available:
Description of changes:

Updated sklearn to 0.24
Updated xgboost to 1.3

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-926-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-926/1/index.html

Job PR-926-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-926/2/index.html

Benchmarks show no regressions, except that KNN seems to have gotten slower, will investigate in separate PR.


Improve documentation

Reestructuring documentation
Specify No one-hot encodes on data model.
Desired Module structure on Data Source

Hi! Juan, data scientist from the Viz team.
I'm having difficulties with the installation for being able to use the library:
`(coronawhy) C:\Users\juanc>git clone git@github.com:CoronaWhy/task-geo.git
Cloning into 'task-geo'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.`
I thought that for cloning I should use this command:
https://help.github.com/en/github/creating-cloning-and-archiving-repositories/cloning-a-repository

Hi @JuanCalvoFerrandiz  and thanks for the feedback, I will upload the github link in the documentation to the https one.
Can you please specify with python version are you using?
Also, please try with just make install, as the current installation in the readme is focused for developers, will change that too.
Also, please, next time, don't send an screenshot of a text screen, copy and paste the code, is easier to work this way.


Removed duplicate license file

Now that we've moved the package up to the repo root, we don't need both LICENSE and LICENSE.md
In fact, having both seems to be confusing GitHub. It's no longer automatically recognizing us as a BSD-3 project.
I think we can remove LICENSE.md. This is working correctly for pkgnet: https://github.com/UptakeOpenSource/pkgnet
@falconred let us know if this doesn't make sense

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   70.91%   70.91%
=======================================
  Files           2        2
  Lines         533      533
=======================================
  Hits          378      378
  Misses        155      155

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 fe7ee63...c29914f. Read the comment docs.


Implement messaging feature on the frontend

We should discuss details before starting implementation.

I will start the initial design of messaging page today. We can discuss the other details after design.

I finished messaging part. Now, a user can create conversation with other one and can communicate with each other.
Also, we have to discuss how often we send a request to control the messages sent to other user.

I handle the last problem I mentioned comment above. Now, we can check conversation details every 5 seconds. Hence, I close this issue.


Can't build path with star in param value

Hi.
I'm trying to build url for route with param that contains star (asterisk) like this:
router.buildPath('test', {
  param: 'test*',
})

But it throws "Some parameters of '/test/:param' are of invalid format". As far as I understand it maybe somehow related to splat params, but it's in a param value, not in route definition, so I doubt it is an expected behavior.
Here's a codesandbox for reproduction: https://codesandbox.io/s/p7k0nrxj0j

Indeed it is. Just submitted PR there


How are nested slurs indicated?

I want the following effect

I tried to write like this, but only the outermost slur is displayed

2 3  4  (  5  (  5 ) )

That's not a nested slur, it's a tie inside a slur.  There are 3 reasons why it's important to tell Lilypond that the tie is a tie not a slur:

In professional music printing, ties are printed slightly differently from slurs (the curvature is slightly less),
In the MIDI output we want the note to be held over and not repeated,
If we later add lyrics, we want the tied note to count as just one word.

Use ~ for ties, or use - instead of the second 5 (jianpu-ly knows to change - after a barline into a repeat of the number with a tie)

so it's q1 q1 ~ 1 s6, ( s1 ) q2 s2 ( s3 ) s4 s5 ~ 5 - q0 s5 ( s6 q5 ) s5 ( s3 ) etc

I see. Now I can write it like this
q1 q1 ~ 1 s6, ( s1 ) q2 s2 ( s3 ) s4 ( s5 ~ 5 ) - q0 s5 ( s6 q5 ) s5 ( s3 )

it can ok;

太好了

I see. Now I can write it like this
q1 q1 ~ 1 s6, ( s1 ) q2 s2 ( s3 ) s4 ( s5 ~ 5 ) - q0 s5 ( s6 q5 ) s5 ( s3 )

it can ok;

It looks like you’re using a previous version of the script? I saw the flipped beams and serif font


ESP-RFID Blue Relay Board

I don't know if you have seen this one. It's much better pinout. Suitable for RC522 and other modules.

ESP-RFID Relay Blue Board
Because it was often requested, we made another version. Much more stable, with circuit protections and with better wiring so that it’s possible to use other popular RFID readers like RC522, PN532, and ofcourse, Wiegand readers. Below picture regarding the pin usage is pretty self explanatory and if you need any help, feel free to contact us.
If you haven't seen ESP-RFID project before, you can check the official github page: https://github.com/esprfid
ESP-RFID Relay Board basically makes any RFID reader wifi enabled and provides several ways to communicate with the reader, manage user cards and log access for the users.

You can purchase it at: https://www.tindie.com/products/nardev/esp-rfid-relay-blue-board/
The cost of the board is minimal in regards to the production cost. Sorry, it was hard.

The Blue board's wiring is explained below:

The video is coming soon.

Wow that's nice. Didn't posted it here, but somebody find it already. It's my version on the board. If you know how to handle with a solder iron is possible to make the board yourself. Here is the project: https://easyeda.com/teria/hostel-lock

Wow that's nice. Didn't posted it here, but somebody found it already. It's my version on the board. If you know how to handle with a solder iron is possible to make the board yourself. Here is the project: https://easyeda.com/teria/hostel-lock
And I recommend you to provide the link to the original project at tindie.com coz I developed the project is under MIT licence.

@teriall Man, i was looking for you, but you are so secretive i couldn't find your email. Please contact me<EMAIL_ADDRESS>or what ever other way.


Request new module: cp-catalogs-waiops

Related user story
Cloud Pak for Watson AIOps #106
Type of module
gitops
If requesting a terraform module, what is the target cloud platform?
ibm, aws, azure, etc
What is the purpose of the module?
Create Catalog Source required for CP4WAIOps
What is the category of the module?
software

@dimallya to be clear, this will only contain catalog sources needed for Watson AIOps?  Does this overlap with other catalog sources?  ex: https://github.com/cloud-native-toolkit/terraform-gitops-cp-catalogs

As per KC documentation, though the operator catalog name is ibm-operator-catalog, image source (i.e. icr.io/cpopen/ibm-operator-catalog:latest) mentioned in the CP4WAIOps catalog source is different than that of one which is present in existing cp-catalog module's ibm-operator-catalog image source (i.e. docker.io/ibmcom/ibm-operator-catalog). Also, for event manager installation noi-operator-catalog  catalog source is required. Hence we requested for the new catalog module and planning to include these 2 catalog sources in the cp-catalog-waiops module.
However, we figured out that operators of Watson AIOps AI Manager, Event Manager and Infrastructure Automation - all are available within existing ibm-operator-catalog catalog-source present in ocp-latest-gitops cluster, with the latest version and same image-source as mentioned in KC. Hence we can consider not to have the new module. We will leverage catalog source created by the existing cp-catalog module.
Hence closing the issue.


[Dialog] Side effects (body overflow hidden property)

I would like to suggest a following change,
The Dialog component sets the <body> style as overflow: hidden;. And on dismiss, this style rule is dismissed.
But if the component is removed from DOM while Dialog was being shown, it renders the whole body non-scrollable due to the above style.
I would suggest a componentWillUnmount as follows,
  componentWillUnmount: function() {
    // dialog has a side-effect if this not checked
    document.body.style.overflow = 'auto';
  }

or to be more independent, the componentDidMount lifecycle function can store the original state of the body style and on unmount, that style can be restored.

+1 I'm having this issue right now because I'm managing myself if the Dialog is in the dom or not.
The issue is inside de Overlay component. The overflow property is only updated in componentDidUpdate. But when it's removed, the property is therefore never reset :/

1

Doesn't work well with overflow-y: overlay; CSS property:

Im having the same issue, when I overlay many Dialogs and travel to a different path, the overflow hidden property on the body doesnt get removed even if I update the boolean property of the open Dialog KEY value.


refresh to v1.8.0

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes https://github.com/open-ce/open-ce/issues/265, https://github.com/open-ce/pytorch-feedstock/issues/45.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Removed the following from run time dependencies:

ninja - is required for cpp extension tests - https://github.com/pytorch/pytorch/blob/v1.8.0/test/run_test.py#L324. These tests are not run currently in open-ce-tests.yaml.
numba - is required for numba_integration tests so added this to open-ce-tests.yaml
python-lmdb - was required by some caffe2 tests. These tests are not run currently in open-ce-tests.yaml

Also removed absl-py as it is not a direct requirement for pytorch, it is needed by tensorboard and is a dependency there.  https://github.com/open-ce/tensorboard-feedstock/blob/main/recipe/meta.yaml#L34


Change exception for curl error to return code and reason

@barayantan does this fix any recorded issues here?


[Fix]Remove duplicated destructor function in MinMaxFuncBase

Proposed changes
Issue Number: close #8286
Problem Summary:
Fix the compile error.
Checklist(Required)

Does it affect the original behavior: No
Has unit tests been added: No Need
Has document been added or modified: No Need
Does it need to update dependencies: No
Are there any changes that cannot be rolled back: No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

because of this pr will fix the compile error, I will merge it now


Scale video modal to 16:9 aspect ratio

Current video modals flex the video player to full-screen in the two video modals (series trailer and watch live). We should limit the video to 16:9 ratio and center the player.

fixed in #334


Busybox vs Alpine cert issue

I'm trying to rebuild the errordeveloper/oracle-jre image, which is build on busybox, on Alpine.  I'm seeing a difference in cert (ca-certificates) working between Alpine and Busybox.
Captured the issue as a Stack question with example here: http://stackoverflow.com/questions/33353532/does-alpine-linux-handle-certs-differently-than-busybox
I'm not sure if this is a bug or just a difference in how Alpine/BB handle certs.  Basically the only thing I changed in the Dockerfile is busybox->alpine (also tried alpine:3.2) and changed the package manager calls to support apk.

It doesn't look like that is the correct file location of that CA:
$ dri alpine sh -c "apk add -U ca-certificates && ls -al /etc/ssl/certs" | grep GeoTrust_Global_CA
lrwxrwxrwx    1 root     root            30 Oct 26 23:27 2c543cd1.0 -> ca-cert-GeoTrust_Global_CA.pem
lrwxrwxrwx    1 root     root            57 Oct 26 23:27 ca-cert-GeoTrust_Global_CA.pem -> /usr/share/ca-certificates/mozilla/GeoTrust_Global_CA.crt
lrwxrwxrwx    1 root     root            59 Oct 26 23:27 ca-cert-GeoTrust_Global_CA_2.pem -> /usr/share/ca-certificates/mozilla/GeoTrust_Global_CA_2.crt
lrwxrwxrwx    1 root     root            32 Oct 26 23:27 cbeee9e2.0 -> ca-cert-GeoTrust_Global_CA_2.pem

Oh, and actually, you can't really do what you are trying to do. See #11 for the long discussion and some possible workarounds.

Oh dear.  That is indeed disappointing.  Hopefully Alpine will support Java8 at some point, but thanks for the explanation--it does explain it.


Compute shader property issue with 2018.2.4f1

After upgrading to 2018.2.4f1 I get the following error in the console:

Compute shader (InjectLightingAndDensity): Property (_AreaLightShadowmap) at kernel index (0) is not set
UnityEngine.ComputeShader:Dispatch(ComputeShader, Int32, Int32, Int32, Int32)
VolumetricFog:Scatter() (at Assets/VolumetricLighting/Assets/VolumetricFog/Scripts/VolumetricFog.cs:512)
VolumetricFog:OnRenderImage(RenderTexture, RenderTexture) (at Assets/VolumetricLighting/Assets/VolumetricFog/Scripts/VolumetricFog.cs:559)

Same issue

me too

aha，I found it，you should first open AreaLight scene，then open other scene，but，i am thinking， if i package it out should i must enter arealight scene then enter other scene？

another way：set the Area light gameobject on in the tube light Scene, this is not the correct solution,  but no error report, Iguess this problem exist at start, otherwise there can not be a area light in tube light scene.

Fixed in latest

Fixed in latest

Thank you!


Add user details/edit page to TPv2

This PR adds a details/edit page for users to TPv2, and implements the context menu for the users table (except "View User Changelogs" which will be done in the future). It also refactors generic tables a bit to allow link menu items to dynamically generate their hrefs, and allows links to be disabled.

Which Traffic Control components are affected by this PR?

Traffic Portal (experimental v2)

What is the best way to verify this PR?
Make sure all the associated tests I wrote are passing
PR submission checklist

[x] This PR has tests
[ ] This PR has documentation
[ ] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

Note: I didn't write e2e tests yet because that will get much simpler when I put in the work to create/delete users