Jdk 11 build

This fix allows:
mvn clean package site
to work on JDK 8 and 11.
mvn clean package site -P commons-rng-examples
does not work. Javadoc generation fails for the JPMS modules. The javadoc tool complains about links to a Javadoc API in the unamed modules:
The code being documented uses modules but the packages defined in https://docs.oracle.com/javase/9/docs/api/ are in the unnamed module.

This javadoc issue when building using JDK 11 for a source earlier than 11 is found in a lot of web posts. The fix that works for a source earlier than 9 (i.e. using the <source>7</source> tag in the maven javadoc configuration) does not work when the source is 9 and has modules.
One simple workaround is to not use JDK 11 to build javadocs.

To confirm. Setting the JPMS modules to have a source of 11 with default settings allows javadocs to be built. Here's a snippet with a maven-javadoc-plugin that overrides the parent and so resets all configuration. This works:
    <!-- Java language level. -->
    <maven.compiler.source>11</maven.compiler.source>
    <maven.compiler.target>11</maven.compiler.target>
    <animal.sniffer.skip>true</animal.sniffer.skip>
    <maven.compiler.release>11</maven.compiler.release>
  </properties>

  <build>
    <plugins>
      <!--  Revert javadoc fix in parent -->
      <plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-javadoc-plugin</artifactId>
        <configuration combine.self="override">
          <source>11</source>
        </configuration>
      </plugin>
    </plugins>
  </build>

However this omits some of the configuration added by commons-parent and comons-rng-parent for the javadocs, e.g. MathJax support.
It may be that support for java 9 should be dropped in favour of java 11 and the project fixed as above but adding back the desired javadoc configuration.

Coverage remained the same at 98.239% when pulling 6a966d34eeb414887c5dcd117b452804907003bb on aherbert:jdk-11-build into 85c9744774b2984201ded41ea75dbacc62117a99 on apache:master.

Note:
Javadoc generation does not work on JDK 11.0.2 due to the packages in the unamed module error. A fix for this is present in JDK 11.0.3 (See JDK-8212233).
I tried upgrading the pom to allow javadoc generation using JDK 11 using an explicit source/release of 11. This required (note the javadoc-plugin fix above was incorrect:
    <!-- Java language level. -->
    <maven.compiler.source>11</maven.compiler.source>
    <maven.compiler.target>11</maven.compiler.target>
    <animal.sniffer.skip>true</animal.sniffer.skip>
    <maven.compiler.release>11</maven.compiler.release>
  </properties>

  <plugin>
    <artifactId>maven-javadoc-plugin</artifactId>
    <configuration>
      <!-- Reset API links since these refer to java 1.6 and 1.7 -->
      <links combine.self="override" />
    </configuration>
  </plugin>

<!-- upgrade for java 11 -->
maven-compiler-plugin -> 3.8.0
maven-surefire-plugin -> 3.0.0-M3
maven-surefire-report-plugin -> 3.0.0-M3

Then the the site plugin fails on default-site goal:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-site-plugin:3.7.1:site
(default-site) on project commons-rng-examples-jpms-lib: Execution default-site of
goal org.apache.maven.plugins:maven-site-plugin:3.7.1:site failed:
org.apache.maven.reporting.MavenReportException: Unsupported targetJdk value '11'.

Currently it seems that the maven plugins have yet to catch up to java 11. The solution is to fix the main build command for java 11 and stick to supporting the full build with the examples modules on JDK 9.
This works.
JDK 11
mvn clean package -P commons-rng-examples
JDK 9
mvn clean package site site:stage -P commons-rng-examples
JDK 8
mvn clean package site site:stage
This can be revisited when JDK 11.0.3 is released (if ever) and the Maven plugins catch up to Java 11.


If the calib necessary when  I run the demo.py to test my own pcd file(transfer it to .bin)

hi, open-mmlab team
I want to know If the calib necessary when  I run the demo.py to test my own pcd file(transfer it to .bin)?
It means a lot to me. Looking forward to your reply. Thanks a lot

It depends on what demo you want to run. If you run on point cloud only tasks (single-modality pc detection or segmentation), you don't need calib. But for tasks that involve image input (multi-modality pc detection or monocular 3d detection), the calib info is necessary to link between 2D and 3D.

It depends on what demo you want to run. If you run on point cloud only tasks (single-modality pc detection or segmentation), you don't need calib. But for tasks that involve image input (multi-modality pc detection or monocular 3d detection), the calib info is necessary to link between 2D and 3D.

Thanks for your quickly reply！
(1) I want to test my 1 frame .pcd file with second or pointpillars model. Does it mean I can run the pcd_demo.py without calib information?
(2) If  i want to train the second model with my own data which only have pointcloud and label data. I have no calib and image data. can  I succeed？
Looking forward to your reply! Thanks again!

It depends on what demo you want to run. If you run on point cloud only tasks (single-modality pc detection or segmentation), you don't need calib. But for tasks that involve image input (multi-modality pc detection or monocular 3d detection), the calib info is necessary to link between 2D and 3D.

Thanks for your quickly reply！
(1) I want to test my 1 frame .pcd file with second or pointpillars model. Does it mean I can run the pcd_demo.py without calib information?
(2) If i want to train the second model with my own data which only have pointcloud and label data. I have no calib and image data. can I succeed？
Looking forward to your reply! Thanks again!

Thanks for your quickly reply！
(1) I want to test my 1 frame .pcd file with second or pointpillars model. Does it mean I can run the pcd_demo.py without calib information?
(2) If  i want to train the second model with my own data which only have pointcloud and label data. I have no calib and image data. can  I succeed？
Looking forward to your reply! Thanks again!

Yes, you can succeed because pointpillars and second are both models that ONLY consume point cloud data. Image is not required.

@wuziyi616
thank you very much.
but i want to train with my own data. If i have no calib information to change the coordinate as KITTI format(such as lidar_to_camera). can I get the correct AP?
sorry for bothering again.

@wuziyi616
thank you very much.
but i want to train with my own data. If i have no calib information to change the coordinate as KITTI format(such as lidar_to_camera). can I get the correct AP?
sorry for bothering again.

@wuziyi616
thank you very much.
but i want to train with my own data. If i have no calib information to change the coordinate as KITTI format(such as lidar_to_camera). can I get the correct AP?
sorry for bothering again.

For training, if you have 3D point clouds (something like Nx3 array) and 3D boxes labels (something like Kx6 array), then you can train the model. However, if they are like KITTI raw data, then you need calib to transform the boxes labels.

@wuziyi616
thank you very much.
but i want to train with my own data. If i have no calib information to change the coordinate as KITTI format(such as lidar_to_camera). can I get the correct AP?
sorry for bothering again.

@wuziyi616
thank you very much.
but i want to train with my own data. If i have no calib information to change the coordinate as KITTI format(such as lidar_to_camera). can I get the correct AP?
sorry for bothering again.

For training, if you have 3D point clouds (something like Nx3 array) and 3D boxes labels (something like Kx6 array), then you can train the model. However, if they are like KITTI raw data, then you need calib to transform the boxes labels.

Hi Thanks for your reply. I'm also trying to train the point cloud only detector on my own dataset. I have converted both the point cloud and the annotation to the exact KITTI format.
However when I always got 0 AP for every test when I was training PointPillar.
The only difference with my data is that, in the KITTI txt anno file, the location is in CAM coords but my 3d box labels are in point cloud coords. Since I only have point cloud data and no camere data and calibration data, I won't be able to convert the 3d box labels to the CAM coords.
Could you please let me know how to deal with that ?
Thanks


As a busy courier, I want to let another courier handle one of my orders

so that my colleagues would be able to help out when I could not complete the order by today.

#189 #353


Deadlock @ mdns_free() (IDFGH-718)

mdns_free() calls function _mdns_service_task_stop()
mdns_free()  --> _mdns_service_task_stop() {
MDNS_SERVICE_LOCK()  --> Step 1
Post ACTION_TASK_STOP to task and  wait on this while loop
while (_mdns_service_task_handle) {
vTaskDelay(10 / portTICK_PERIOD_MS);
}
MDNS_SERVICE_UNLOCK();
}
on _mdns_service_task() {
Wait on queue messages()                --> Step 2
if(action == ACTION_TASK_STOP)
{
break; //exit the task
}
MDNS_SERVICE_LOCK();                                --> Step 3
_mdns_execute_action(a);
MDNS_SERVICE_UNLOCK();
}
If Step 1 happens and on Step2 if there are other actions pending on queue mdns task will be stuck at Step3
This can be easily reproduced if  mdns_service_remove_all() is called just before mdns_free(). (happens 1 in 10 times)

@me-no-dev

Suggested fix:
mdns.patch.tar.gz
I was able to reproduce this easily with softAP created on ubuntu machine.

Hi, this bug has already been addressed in https://github.com/espressif/esp-idf/commit/6bb68a5a7567a94c3605136d44960ff060c74663 and https://github.com/espressif/esp-idf/commit/206b47c03aca0acdf40d1d9c250e18aeddfe1bd7.
There's still one possible deadlock (yet very improbable) calling _mdns_stop_timer(); when mutex is locked (if a timer task executed before unlocking). There's one commit fixing this issue in an internal repo, so I will update the commit message to close this issue.


Integrate Google Analytic

Acceptance Criteria

Integrate Google Analytics into the app
Respect the config in #21
Write a class that allows sending a custom event message in the renderer or main

I've integrated with universal-analytics and it works on .prod now. @NghiaTranUIT may you please help to define the event category/action + when they should be tracked for Proxyman? Btw please let me know if you want to have an uid for each unique user, otherwise we can keep them as anonymous but the event should be duplicated even though they come from the same user
<Realtime dashboard (prod build)>

Requirement for the Analytic

Add the Opt in/out on the Preference -> Privacy Tab (https://github.com/ProxymanApp/proxyman-windows-linux/issues/19) -> Analytic section
Save the config to AppConfiguration
Add AnalyticService class: Init GA service if the user is opt-in
Don’t expose the gaService to the renderer (using the global) -> Define a message and send to the main.
Generate a UUID (https://www.npmjs.com/package/nanoid) and save to AppConfiguration. Generate a new one if it doesn’t exist. If it exist, just use it.
Use this UUID as a userID for the GA
Only tracking for the production build. Ignore for the DEV build.
Tracking the following tasks:
App Launch at the first time
Open Windows: SSL Proxying, Certificate
Use Tool: Open the Filter, Clear, Install certificate, activate the license, enable ssl proxying


Add Gemfile.lock .gitignore?

I get the impression that Gemfile.lock is generated locally based on the results of the bundle operation.  If it is it might be a good idea to mention it in the .gitignore file and remove it from the index.

Yeah... I wasn't thinking when I committed it. I'll undo that.

done


Removing the use of nginx ingress snippet annotations

Thanks for the PH @magicJie, this is already being addressed in #106. Also, nginx.ingress.kubernetes.io/append-headers doesn't look like a valid annotation.


Basic support for progress indicators

I have a use case for logs that show progress of long-going resolution of some synchronous tasks.
It may expose as:

The API could be as:
var debug = require('debug')('foo');

debug.open('setup-x');
..
debug.progress();
..
debug.progress();
..
debug.progress('|');
..
debug.progress();
..
debug.close();

Above will result with something as:
foo setup-x ..|. 24s

I think support shouldn't go beyond that (e.g. involve control characters that introduce deletion etc.) and functionality should be intended purely for eventual sync operations which keeps process stuck for noticeable moments.
At some point I even created a debug extension that provides such functionality.
Is there a chance to have something like this considered for v3.0, I think it may not be possible to achieve easily with proposed middlewares functionality.

Hmm, why not use node-progress?

@thebigredgeek I'll consider node-progress, thanks for suggestion, it indeed may suit better this case


Removed alerts

I'm proposing to remove network-wide alerts, as bitcoin did in 2016: https://github.com/bitcoin/bitcoin/pull/7692
All tests are passing, apart from Integration tests which are failing at the moment.
will wait for those to be fixed in order to merge and check.

Please just ignore the first commit, it's already been merged.


wasm: update Wasmtime to v0.35.1.

Signed-off-by: Piotr Sikora<EMAIL_ADDRESS>Fix #20284

@PiotrSikora passing CI but still in draft


Input sequencer

Otherwise everything needs to be done manually.

https://github.com/esdmr/ris-gates/commit/12c8c393a6c10f750e57682a4fd2eeb364890f21


ic Piero Angela

ic Piero Angela

@dondo75 First stage of verification passed, official domain is icviapoppeasabina.edu.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://icviapoppeasabina.edu.it/tipologia-articolo/in-evidenza/

@dondo75 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Mixer grammar

Within Istio docs, Mixer is referred to as "Mixer", not "the Mixer". See https://istio.io/docs/concepts/what-is-istio/#mixer.
This PR removes the definite article, "the", from comments describing Mixer's configuration.

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

Needs approval from an approver in each of these files:

install/OWNERS
istioctl/OWNERS

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

@leecalcote: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cc @geeknoid

/lgtm

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

Needs approval from an approver in each of these files:

install/OWNERS
istioctl/OWNERS

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

/test e2e-dashboard

/test e2e-bookInfoTests-envoyv2-v1alpha3

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/retest

/test all

@leecalcote: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Can we rebase and then merge?

@leecalcote: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Please rebase and we'll push this PR in.

New changes are detected. LGTM label has been removed.

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

Needs approval from an approver in each of these files:

install/OWNERS
istioctl/OWNERS

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

@geeknoid rebased...

@leecalcote: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
afbdd369bee996f37207fa9aaf335fd413d55041
link
/test istio-integ-k8s-tests

prow/integ-mixer-k8s-presubmit-tests.sh
e9d2413af37d73c4fd7c92abd2493d932f8a6a40
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Seems still have conflicting files?


Implementation of DomainModel.set_data() method.

Implementation of issue #21

Awesome!

Btw. I didn't get the idea about init for this task. Can you explain it
a bit. May be an example?
Пн, 14 бер. 2016 00:35 користувач Roman<EMAIL_ADDRESS>пише:

Awesome!
—
Reply to this email directly or view it on GitHub
https://github.com/ets-labs/domain_models/pull/32#issuecomment-196070384
.

Yep, sure!
I meant something like this:
for name, field in six.iteritems(self.__class__.__fields__):
    field.set_builtin_type(self, name, data.get(name))

@rmk135 , no no no... I meant this phrase from the description of the issue -

Notes: Current logic should work for DomainModel.init() as well.

What did you mean? Possibility to pass dict as argument of constructor or what?
It partly works DomainModel(**dict()) but not for field.Model

Yeah, exactly like that...
DomainModel(**dict())
Will it be a big deal to add such "on the fly" converting for fields.Model and fields.Collection?

Hey @boonya,
First of all, sorry for the delay. I put some extremely minor comments, but it looks awesome!

Please, fill free to merge it right when you will have a time 👍  ;)


Discover a way to scrap ElAmigos repacks

You mean scrape?

You mean scrape?

Yes, like in web scrapping/scraper

Hi.
You mean to bypass their "captcha" while scrapping?
I have experience with web scraping and I've encountered similar captcha challenges in the past.
I'd be happy to contribute :)


Uncaught Error: Cannot find module '[object Object]' after reinstalling package

So I removed react-datepicker package to reinstall it again today. However, I am now getting this error:
Uncaught Error: Cannot find module '[object Object]'
and also noticed that moment is missing from node_modules under this package, not exactly sure why and wanted some insight. It was working before until I removed it.
This is my dependencies in my package.json:
"react": "0.14.2",
"react-dom": "0.14.2",
"react-datepicker": "0.15.2",
"moment": "2.10.6"

you need to add this "react-onclickoutside":"4.1.1" it's seems the component has another version that is the problem
"react": "0.14.2",
"react-dom": "0.14.2",
"react-datepicker": "0.15.2",
"moment": "2.10.6",
"react-onclickoutside":"4.1.1"
let me know

I downgraded react-onclickoutside to make sure it doesn't fail anymore.

Looks like the issue is resolved, thanks guys.


background of pages

The preference is to have an off-white background

The preference is to have an off-white background

Please be specific and give an example colour. I will need the HEX code such as #F5F5F5. You can pick a colour from here: https://htmlcolorcodes.com/

We rely on your expertise for the choice of background colors for each web page.

Nico - I would advise leaving the background colour as it is. This is becaus ethroughout the site, there are already some sections in grey or other colours. Changing the background colopur site-wide means we will need to change these also to fit. Let me know if you agree.

Hi Rebecca,
I hope you are well.
Your proposals are welcom. I agree.
Best Regards
Nico TSHINTU
10 juin 2024 22:17 "Rivkanexus" @.*** @.@.>)> a écrit:
Nico - I would advise leaving the background colour as it is. This is becaus ethroughout the site, there are already some sections in grey or other colours. Changing the background colopur site-wide means we will need to change these also to fit. Let me know if you agree.
—

Reply to this email directly, view it on GitHub (https://github.com/Rivkanexus/ISPA-DRC/issues/11#issuecomment-2159299033), or unsubscribe (https://github.com/notifications/unsubscribe-auth/BHK2HBDVH4GAJJRVBNCC2MDZGYJVHAVCNFSM6AAAAABFYA6HSCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJZGI4TSMBTGM).
You are receiving this because you commented.Message ID: @.***>


ISV Dynamics 365/Power Apps solution

Hi,
does this document apply also to Dynamics CE/Power Apps solutions?

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

ID: 69192df7-736f-1504-e8ab-d1ececb95800
Version Independent ID: 7707373e-0ef6-ac6b-fcce-efce6a0ade73
Content: Independent software vendor (ISV) licensing - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/isv-licensing.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @jorisdg
Microsoft Alias: jorisde

Thanks for the question, @malkhawaja. I spoke with the author and the topic only applies to Dynamics 365 Finance and Operations apps. It doesn't apply to either of the apps you are asking about.


IMPORTANT: migrate to oneVPL for newer Intel GPUs support

Intel® Media SDK lifetime comes to an end in a form it exists right now. In particular,

API 1.35 is projected to be the last API of 1.x API series
Runtime library (libmfxhw64.so.1) is not planned to get support of new Gen platforms (ADL-S, ADL-P, and above)
Project is going to be supported in maintenance mode, critical fixes only

All future development is planned to happen within oneVPL library and its runtime
implementations which are direct successors of Intel® Media SDK. oneVPL introduces API 2.x series which is not backward compatible with API 1.x series (some features got dropped). New VPL Runtime for Gen Graphics (libmfx-gen.so.1.2) comes with the support of new Gen platforms.
What should be packaged?
Attn. @dbermond, @jbeich, @sebastinas, @sndirsch, @tjaalton
Media SDK and oneVPL support Intel GPUs starting from gen8 BDW. However:

New VPL Runtime for Gen Graphics (libmfx-gen.so.1.2) supports platforms starting from Gen12 - TGL, DG1, RKL, ADL-S, ADL-P
Media SDK runtime (libmfxhw64.so.1) supports "legacy" platforms up to (some!) Gen12 platforms - BDW, SKL, KBL, CFL, ... , TGL, DG1

MSDK and oneVPL dispatcher libraries (libmfx.so.1 or libvpl.so.2) load one of the above runtimes depending on the underlying GPU platform.
Thus, the following repos needs to be packaged to cover full range of MSDK/VPL supported platforms:

https://github.com/oneapi-src/oneVPL (>=v2021.6.0)
https://github.com/Intel-Media-SDK/MediaSDK (>=intel-mediasdk-21.3.4)
https://github.com/oneapi-src/oneVPL-intel-gpu (>=intel-onevpl-21.3.4)

Status of dependent projects

Project
Status

ffmpeg
Not yet switched to oneVPL, patches pending package adoption by Linux distros

gst-plugins-bad
oneVPL support enabled

Thank you for the information. Very appreciated!
oneVPL packages on Arch Linux:

https://aur.archlinux.org/packages/onevpl/
https://aur.archlinux.org/packages/onevpl-cpu/
https://aur.archlinux.org/packages/onevpl-intel-gpu/

I have placed them on the AUR (Arch User Repository) for the time being. Planning to move them to the official repositories later, as there is still low user usage and no dependent applications using them yet.
Packaging suggestions are welcome :)

I don't intend to work on packaging oneVPL for Debian.

Thanks a lot! Looks good! Everything addressed from our side! :
Stefan
On Tue, 12 Oct 2021 at 03:59, Dmitry Rogozhkin @.***> wrote:

oneVPL-intel-gpu comes with various source files with proprietary Intel license
That's an oversight. We fixed it in this commit: @.***
Please, let us know if you will spot anything else.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Hello again. Unfortunately I found a similar issue. This time in
oneapi-src/oneVPL sources. It seems the component contains code licensed
under "MS Vistual Studio" licsense. AFAIK this is incompatible
to GPL-3.0 license.i.e. you can't link GPL 3.0 sources against libvpl. This
makes it difficult for a Linux distributor to ship this library. :-( Is
this an oversight, intentional or just can't be changed?
I've opened an issue for this, but haven't seen any response to this up to
now. :-(
https://github.com/oneapi-src/oneVPL/issues/27
Thanks,
Stefan
On Tue, 12 Oct 2021 at 10:36, Stefan Dirsch @.***>
wrote:

Thanks a lot! Looks good! Everything addressed from our side! :
Stefan
On Tue, 12 Oct 2021 at 03:59, Dmitry Rogozhkin @.***> wrote:

oneVPL-intel-gpu comes with various source files with proprietary Intel
license
That's an oversight. We fixed it in this commit: @.***
Please, let us know if you will spot anything else.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Intel-Media-SDK/MediaSDK/issues/2799#issuecomment-940790973,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7BK3OHGUJKVDHIE35JLSDUGPXQZANCNFSM5E6HNR5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Summarizing here....

Unfortunately I found a similar issue. This time in oneapi-src/oneVPL sources. It seems the component contains code licensed under "MS Vistual Studio" license. AFAIK this is incompatible to GPL-3.0 license.i.e. you can't link GPL 3.0 sources against libvpl. This makes it difficult for a Linux distributor to ship this library. :-( Is this an oversight, intentional or just can't be changed?

According to https://github.com/oneapi-src/oneVPL/issues/27 this was just a misunderstanding (that license applies only to Windows builds which are not distributed under GPL) which is / will be cleared in the documentation.
Additionally, "oneVPL-intel-gpu" (Intel GPU backend for oneVPL frontend) project build issue with GCC v11.2 (used in Ubuntu 21.10 and 22.04) is now fixed, but there is not yet a tagged version with that fix.

project build issue with GCC v11.2 is now fixed, but there is not yet a tagged version with that fix

Yes, fixed in https://github.com/oneapi-src/oneVPL-intel-gpu/commit/a8702a9eb94a306ec8f592b5fcac41c84564f7c4. Will go into one of the next release or release updates.

MSDK dispatcher can't load libmfx64-gen.dll on Windows. https://github.com/Intel-Media-SDK/MediaSDK/issues/2904
No forward compatibility like the behavior on Linux.


config center support registry multi-address format

address="zookeeper://<IP_ADDRESS>:2181|zookeeper://<IP_ADDRESS>:2181"

Codecov Report

Merging #6209 into master will decrease coverage by 0.05%.
The diff coverage is 60.00%.

@@             Coverage Diff              @@
##             master    #6209      +/-   ##
============================================
- Coverage     60.88%   60.82%   -0.06%
+ Complexity      493      492       -1
============================================
  Files           999      999
  Lines         39898    39902       +4
  Branches       5744     5745       +1
============================================
- Hits          24292    24272      -20
- Misses        12915    12936      +21
- Partials       2691     2694       +3

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/config/bootstrap/DubboBootstrap.java
52.34% <60.00%> (-0.59%)
0.00 <0.00> (ø)

...bo/config/event/listener/LoggingEventListener.java
37.50% <0.00%> (-25.00%)
0.00% <0.00%> (ø%)

...ava/org/apache/dubbo/config/DubboShutdownHook.java
63.82% <0.00%> (-10.64%)
0.00% <0.00%> (ø%)

...che/dubbo/remoting/transport/mina/MinaChannel.java
39.47% <0.00%> (-10.53%)
15.00% <0.00%> (-1.00%)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0.00%> (-5.27%)
0.00% <0.00%> (ø%)

...pache/dubbo/remoting/transport/AbstractServer.java
53.75% <0.00%> (-3.75%)
0.00% <0.00%> (ø%)

...he/dubbo/remoting/transport/netty/NettyServer.java
70.17% <0.00%> (-3.51%)
8.00% <0.00%> (-1.00%)

.../remoting/transport/netty4/NettyClientHandler.java
57.62% <0.00%> (-1.70%)
0.00% <0.00%> (ø%)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
62.11% <0.00%> (-0.63%)
30.00% <0.00%> (ø%)

... and 3 more

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 88cb7f5...90a0e72. Read the comment docs.


Add support for time-partitioned tables

Time partitioning can now be specified during table creation, and can be requested as a table option when fetching table information.
Currently, it is only represented as a boolean value in the client side, since the only possible partitioning scheme is by day. This is documented in TableInfo's Javadoc.

Hi @C0urante, thanks for your contribution. I am going through your changes right now.


BNCURLBlackList gets wrong version value

The variable blackListVersion gets nil value in BNCURLBlackList.m:45 and it tries to get a non-existing file from Branch server.
Logs:
2018-03-14 16:44:50.621308+0100 MyApp[10557:335942] [branch.io] BNCNetworkService.m(219) Debug: Network finish operation https://cdn.branch.io/sdk/uriskiplist_v1.json 0.413s. Status 404 error (null).

<title>404 Not Found</title>

404 Not Found

Code: NoSuchKey

Message: The specified key does not exist.

Key: sdk/uriskiplist_v1.json

RequestId: DDA8092FFFF72CE4

HostId: NkhKrbYuVeoQDmd20tnzAHEdUoBQe94hp0adTsFEqC4WdaBD9SsHQ9CqG07v8WKVvOzEQW2DzXQ=

An Error Occurred While Attempting to Retrieve a Custom Error Document

Code: NoSuchKey

Message: The specified key does not exist.

Key: error.html

.

Thanks for noticing!   Everything is okay.
The SDK is looking for the next version of the and getting a 404.
We are going to update the 404 with a better message.

Agree that the message is very confusing. Wish I came here before reinstalling the SDK and changing all keys.

I updated the code so that 404 pages aren't printed so it looks less like an error. It's in PR #807 .
Sorry for the confusion.


Update README.md

What's the change?

Nothing big, just a style one.. I removed a white space to have a nice line..


Intermitted SIGSEGV errors crashing heavyDB

Version: 6.4.0
While running some queries against heavydb, SIGSEGV errors occur randomly causing the DB to crash and create outages.
Any way to debug/fix this?
HeavyDB.cpp:332 Interrupt signal (11) received.

Could you please share the product logs? The logs can be found in the storage directory, typically located at /var/log/heavyai/storage/log. They are named as heavydb.INFO.*
It is essential to check the logs to identify the problem. Is there a specific reason why you're using version 6.4 when versions 7.0 and 7.1 are available?

I have done detailed analysis of this issue and found the issue..
This happens when I do a select_ipc_gpu on the database and it returns 0 records..

This can be easily replicated by using heavyai lib's select_ipc_gpu function to
import heavyai conn=heavyai.connect(user=<user>, password=<pass>, dbname=<dbname>) conn.select_ipc_gpu(<any select query which returns 0 rows>) //thrift.transport.TTransport.TTransportException: TSocket read 0 bytes
The reason we're using 6.4 is because we have some custom patches for our usecases
Is this fixed on the latest version 7.1?
If so, we might migration to the latest version
Thanks,

Hi,
Thanks for reporting the issue. I will try to reproduce it on our end. If I am successful, I will create an internal case for our engineering team to fix the problem.
Can you try running your application without using the GPU-shared memory as a temporary solution?
Also, I am interested in your modifications to the database to support your application. Could you please share what they are doing?
Best regards,
Candido

Thanks for your reply.
Unfortunately using gpu shared memory is required and cannot be discarded.
Regarding the modifications, I plan to raise a pull request for the same.
Please let me know if you're able to replicate the issue on your end.
Thanks,
Anirudh

Hi,
Using CUDA 11.8 and the latest version of GA (7.2.1), I was able to reproduce the issue on my end. I have created an internal ticket to resolve the issue.
I'll come back here whenthe problem is fixed. (


Interactable configuration/documentation/test changes

Overview
This is a culmination of multiple PRs and merging the feature branch into MRTK_Dev:

Related PRs: #5857 , #5889 , #5898 , #5925 , #5960

There are a lot of fixes in here and I would reference each PR for details but the important notes for this PR is the review of breaking changes and validation steps.
Upgraded Interactable and Visual Themes
Interactable has been refactored significantly. We took great care to not break anything, but it's a good idea before upgrading to 2.1 to back up your existing project.
Change Highlights:

The Themes functionality with Interactable has been extracted into it's own feature called Visual Themes.
The Visual Themes code has been refactored to be more robust and simplified. This involves some class name changes and some property changes.
Themes can now target specific shader properties explicitly using the new ThemeStateProperty.TargetShader and ThemeStateProperty.ShaderPropertyName variables.
Better documentation on how to create Interactables and Themes dynamically via code

1) If you used default Theme scriptableobjects or created Theme assets using the default Theme types (like InteractableColorTheme):
Everything should work as-is. To force your assets to update though, you can select your assets or scene/prefab items and view them in the inspector. Note: Unity only writes inspector changes to disk once the scene is saved. After doing this, you should be able to see the serialized diffs in your source control tracking.
2) If you defined Events Receivers for your Interactables in scenes/prefabs:
Everything should work as-is. To force your assets to update properly though, you can select and view the Events (foldout must be opened) to force the asset update. Note: Unity only writes inspector changes to disk once the scene is saved.
3) If you built a custom theme class extending InteractableThemeBase:
The names of many classes have changed. See list below for full information. Three new properties should be implemented by your class extending InteractableThemeBase.
public override bool IsEasingSupported => true;
Set to true if your theme supports easing between values when state changes
public override bool AreShadersSupported => false;
Set to true if your theme supports shader property targeting
public override ThemeDefinition GetDefaultThemeDefinition()
This method returns a default configuration object for your theme. Include properties, names, and types that your Theme is expecting to run when configured via Init(GameObject host, ThemeDefinition definition)
Look at \MixedRealityToolkit.SDK\Features\UX\Scripts\VisualThemes\ThemeEngines\ for examples of how to implement this method
Also function definitions may need to be updated for new class name which are arguments into the function such as
InteractableThemeBase.Init(GameObject, ThemeDefinition)
ThemePropertyValue GetProperty(ThemeStateProperty property)
SetValue(ThemeStateProperty property, int index, float percentage)
4) If you built a custom Event receiver class extending ReceiverBase:
Everything should work as-is. Only step to be taken is to update any extended constructors to pass a name string into the base. Note: Fields must have the InspectorField attribute to show up on Interactable objects in the Unity inspector
Change Details:

InteractableThemeBase now has abstract method GetDefaultTheme(), IsEasingSupported, AreShadersSupported
Shader property metadata in Theme SO moved to ThemeStateProperty.TargetShader and ThemeStateProperty.ShaderPropertyName variables
Event Receivers in scene/prefab files need to update to include AssemblyClassName. Bug where data was not originally saved.
Multiple classes have been renamed see list below
InteractableTextureTheme not extends InteractableShaderTheme and supports shader property targeting
Elimination of InteractableTypeFinder/InteractableTypesContainer classes
Creating Scriptable objects shortcut now under Create > MRTK > Theme/State

Major API Renames:
Interactable.Events => Interactable.InteractableEvents
Interactable.ForceUpdateThemes() => Interactable.RefreshSetup()
InteractableStateModel => BaseStateModel
InteractableThemePropertySettings => ThemeDefinition
InteractableThemeProperty => ThemeStateProperty
InteractableCustomSetting => ThemeProperty
InteractableThemePropertyValue => ThemePropertyValue
InteractableThemePropertyTypes => ThemePropertyType
Changes

Fixes: #5866 , #5652 , #5462 , #5058 , #5459 , #5050 , #5788

Verification
Build following in GA and then update GA to feature branch

Theme assets (SUCCESS)
CustomThemeClass (SUCCESS)
Interactable Events Receivers in a scene (SUCCESS)
Interactable Events Receivers a in prefab (SUCCESS)
Create custom EventReceiver class(SUCCESS)

After MRTK code update, test the following
Functionality works out-of-box after just hitting playing
Theme inspector works as expected
Interactable inspector works as expected
Other validation
All automated tests pass (SUCCESS)
Test InteractableExamples scene works (SUCCESS)
Test HandInteractionExample scene works (SUCCESS)

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run

For all the rename related breaking changes you have mentioned above, you could potentiallt do something like..
[Obsolete("OldClassName is obsolete. You should be using NewClassName instead")
public class OldClassName : NewClassName {.. ..}
to make them less disruptive.

For all the rename related breaking changes you have mentioned above, you could potentiallt do something like..
[Obsolete("OldClassName is obsolete. You should be using NewClassName instead")
public class OldClassName : NewClassName {.. ..}
to make them less disruptive.

Yea I considered that for the classes. I opted not doing that because

I just want to cut clean now and not worry about someone coming along later to clean up these deprecated classes
The vast, vast majority of consumers atm with MRTK (which is still crazy small) should be ignorant of these classes that are sort of behind the scenes. it's not like renaming Interactable, ManipulationHandler, boundingbox, etc
Those who are leveraging these should be knowledgeable enough to make the necessary renames . It took me forever to learn the system. If someone else has actually done the same, they know the system well enough to understand what is going on. It's actually nice the compiler will point out where that is necessary to rename. Furthermore, this small group of advanced developers are unlikely to have a large amount of custom theme engines. It'll likely be one or two classes.
I thought about leaving the old class names as empty classes in one cs file but then you would still get compiler errors for the missing fields. Again, I would rather just cut clean now.

I wish I could rename all the InterctableActivateTheme, InteractableColorTheme but that would be a harder break and annoying to migrate
Thanks for the review! Very helpful feedback

/azp run

Performed migration from GA to feature branch code using the attached assets to test:
MigrationTest.zip

I've been testing this out, I was wondering, why does interactable have a States field, but then also have a separate States field for each theme? It sounds like it's possible for themes to have a state machine different from interactable. How is the States field in Interactable itself used? Is it used for event receivers?

I hit a bug when modifying Interactable. To repro:

Use the following theme:

Hit "Add Theme Definition" Twice
Try to delete one of the theme definitions by pressing "Delete" button

I get the following call stack:
ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
System.ThrowHelper.ThrowArgumentOutOfRangeException (System.ExceptionArgument argument, System.ExceptionResource resource) (at :0)
System.ThrowHelper.ThrowArgumentOutOfRangeException () (at :0)
System.Collections.Generic.List`1[T].RemoveAt (System.Int32 index) (at :0)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.ClearHistoryCache (System.Int32 index) (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:501)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.RenderDeleteButton (System.Int32 index) (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:430)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.RenderThemeDefinitions () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:121)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.RenderTheme () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:70)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.OnInspectorGUI () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:53)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderProfileSettings () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:173)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.RenderCustomInspector () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:94)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector.OnInspectorGUI () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:83)
UnityEditor.InspectorWindow.DoOnInspectorGUI (System.Boolean rebuildOptimizedGUIBlock, UnityEditor.Editor editor, System.Boolean wasVisible, UnityEngine.Rect& contentRect) (at C:/buildslave/unity/build/Editor/Mono/Inspector/InspectorWindow.cs:1625)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)

Found a bug when adding a new theme via "create and assign theme" button. Got the following stack:
NullReferenceException: Object reference not set to an instance of an object
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.AddThemeDefinition () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:450)
Microsoft.MixedReality.Toolkit.UI.Editor.ThemeInspector.OnEnable () (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/VisualThemes/ThemeInspector.cs:42)
UnityEditor.Editor:CreateEditor(Object)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector:RenderProfileSettings() (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:172)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector:RenderCustomInspector() (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:94)
Microsoft.MixedReality.Toolkit.UI.Editor.InteractableInspector:OnInspectorGUI() (at Assets/MixedRealityToolkit.SDK/Inspectors/UX/Interactable/InteractableInspector.cs:83)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
To repro:

Add second profile
Try to create a new theme definition

The dream for states would be defining your own, then creating a script to determine state based on input into Interactable or from other sources. The easiest way to get the system working was to create an enum and then drive some of the state setting in Interactable, while exposing ways to set some states externally.
Themes and receivers can see what states are being set by Intractable and what the current state is and then react. State is more tied to Interactable now then it is to themes. In themes we care about the name of each state so the designer understands what each value relates to, the values are also stored for each state in the theme, but how they are set or what the represent are abstracted to anything that derives InteractableThemeBase.
The individual state meanings make more sense for receivers though, so that was the main reason for the enum and setting state in Interactable.
That said, it would be interesting to move states out of Interactable and directly on to themes. Interactable sets hasFocus, hasPress and other properties it cares about, but I never wanted the theme or state systems to be restricted to just Interactable states, that's why there are several that states (Collision) have nothing to do with Interactable. I also didn't want themes to be limited to only the states I could think of today.
Moving states to just the theme system could be the way to go. The state script could look at Interactable from the theme and set it's own state, it could also look at other classes on the component and set any new states a designer comes up with. This would would make the state system more flexible, and remove the need for the enum. The down side would be that a programmer will be needed to setup any code required for newly created states (which would happen regardless) and we would need to have a clear path for someone to set state externally. Not just through code, but the inspector as well.
Receivers could just look at properties in Interactable or properties on other classes and achieve the same result they are today.

I've been testing this out, I was wondering, why does interactable have a States field, but then also have a separate States field for each theme? It sounds like it's possible for themes to have a state machine different from interactable. How is the States field in Interactable itself used? Is it used for event receivers?

So this is part of the original architecture except both States properties were not surfaced.
Theme => Uses it's states property to determine how many values a property needs to define (for example, what color to set for each state)
Interactable => Uses States more to determine what kind of State Machine (i.e state model class) to instantiate at runtime inside it's class. Then it updates that state machine instance.
I throw a warning that the States properties need to be the same. Technically they don't because the values are accessed just based on index. So if I had a states object with just Default/Focus, it should theoretically work against an object with Default/Focus/Pressed/Disabled.
States in general requires a lot of refactoring. This is just something I didn't want to untangle and add risk/work to this PR.
It's weird but short term I don't know how to fix it. I can't just enforce only "DefaultInteractableStates" because we actually use the "FocusPressTouchGrabStates" as well as Interactable.

@killerantz , the Interactable though requires States because it creates the State Model. And that information is largely driven to Receivers/Themes.
The problem is there still needs to be some interface/agreement mechanism between all parties. Interactable/recievers can't just use themes because you can have multiple Themes which may have different overlaping states.

I hit a bug when modifying Interactable. To repro:

Use the following theme:

Hit "Add Theme Definition" Twice
Try to delete one of the theme definitions by pressing "Delete" button

I cannot repo this as of yet.

Found a bug when adding a new theme via "create and assign theme" button. Got the following stack:
To repro:

Add second profile
Make the target object a TextMesh object that's not a descendant of the cube where interactable is located.
Try to create a new theme definition

This has been fixed

/azp run

/azp run

/azp run

/azp run

@julenka, fixed that and updated docs! Added Interactable.GetDefaultThemeAsset()

/azp run


Match report task approvers to tasked orgs

We are evolving towards having tasks across various organizations. There are several scenarios:

A task has several tasked organization and has approval step with 1-2 approvers, typically from a single organization. The responsible parties take care of report approval. Works well as implemented

A task has several tasked organization and has an approval step with several approvers (typically at least one for each tasked organization) responsible positions. The intent is that each organization nominates approvers
The current behavior is that all responsible positions can approve reports regardless of their organization

Solution I'd like
Be able to have a approval step with several responsible positions for a task that restrict the report approvers to the positions matching the report's organization of report approval. If none of the responsible positions matches, then act as is the approval step is empty.
Be able to also have an approval step with the old behavior
I suggest to have a property of an approval step called "Restrict to approvers descending from the same tasked organization as the report's primary advisor". If the property is checked, then given a report, find the tasked organization that is an ancestor of the report, and filter the list of approvers to ones within that organization (or its children).

[issue was closed by mistake]


add deployment scripts for dev and prod

Codecov Report

Merging #74 (c0d40a8) into master (6490ca6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   97.19%   97.19%
=======================================
  Files           6        6
  Lines         464      464
=======================================
  Hits          451      451
  Misses         13       13

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


broadcom-sta fails to build on 17.09 with latest kernel

Issue description
When I ran nixos-rebuild build --upgrade, it failed to build my configuration because dependencies couldn't be built.
Steps to reproduce
Relevant sections from configuration.nix and hardware-configuration.nix:
{
  boot.kernelPackages = pkgs.linuxPackages_latest;
  boot.extraModulePackages = [ config.boot.kernelPackages.broadcom_sta ];
}

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.14.15, NixOS, 17.09.2875.c2b668ee726 (Hummingbird)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 1.11.16
channels(root): "nixos-17.09.3031.d09e425aea3"
channels(user): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

a full log might be helpful

@teto Sorry, forgot to attach the log. Here it is:
building
build flags: SHELL=/nix/store/jgw8hxx7wzkyhb2dr9hwsd9h2caaasdc-bash-4.4-p12/bin/bash KBASE=/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0
KBUILD_NOPEDANTIC=1 make -C /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/build M=`pwd`
make[1]: Entering directory '/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/build'
building path(s) ‘/nix/store/4pyibnkjymzzrpb3ax36snjcw8qks226-unit-firewall.service’
created 6660 symlinks in user environment
install-info: warning: no info dir entry in `/nix/store/y2r3zy5qw0aqr55dicmpjabmqxm1s5i3-system-path/share/info/time.info'
building path(s) ‘/nix/store/wjijcgrgbbbay2gym402gc34rmrvalqi-dbus-1’
building path(s) ‘/nix/store/bls9m4qymbgvljx43krzd4w66lnfj5qq-unit-polkit.service’
building path(s) ‘/nix/store/fmg7k0j907j8nkz8nnc83kafgn7a4139-unit-systemd-fsck-.service’
building path(s) ‘/nix/store/g5kh87b65lzaa3ykimvii7fnv5gcpy9i-unit-dbus.service’
building path(s) ‘/nix/store/084l87hwimm386aq0rm0s6y931h1rhar-system-units’
building path(s) ‘/nix/store/rnbzl46xsm9n37nc2y250lg449a5pkjc-user-units’
CFG80211 API is prefered for this kernel version
Using CFG80211 API
  CC [M]  /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.o
building path(s) ‘/nix/store/iimapc4anp9b4ar442dnz2wcs8kbny7q-etc’
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c: In function 'osl_os_get_image_block':
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:1083:26: warning: passing argument 2 of 'kernel_read' makes pointer from integer without a cast [-Wint-conversion]
  rdlen = kernel_read(fp, fp->f_pos, buf, len);
                          ^~
In file included from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/huge_mm.h:7:0,
                 from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/mm.h:463,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/include/linuxver.h:65,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:25:
/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/fs.h:2842:16: note: expected 'void *' but argument is of type 'loff_t {aka long long int}'
 extern ssize_t kernel_read(struct file *, void *, size_t, loff_t *);
                ^~~~~~~~~~~
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:1083:37: warning: passing argument 3 of 'kernel_read' makes integer from pointer without a cast [-Wint-conversion]
  rdlen = kernel_read(fp, fp->f_pos, buf, len);
                                     ^~~
In file included from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/huge_mm.h:7:0,
                 from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/mm.h:463,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/include/linuxver.h:65,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:25:
/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/fs.h:2842:16: note: expected 'size_t {aka long unsigned int}' but argument is of type 'char *'
 extern ssize_t kernel_read(struct file *, void *, size_t, loff_t *);
                ^~~~~~~~~~~
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:1083:42: warning: passing argument 4 of 'kernel_read' makes pointer from integer without a cast [-Wint-conversion]
  rdlen = kernel_read(fp, fp->f_pos, buf, len);
                                          ^~~
In file included from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/huge_mm.h:7:0,
                 from /nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/mm.h:463,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/include/linuxver.h:65,
                 from /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/shared/linux_osl.c:25:
/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/include/linux/fs.h:2842:16: note: expected 'loff_t * {aka long long int *}' but argument is of type 'int'
 extern ssize_t kernel_read(struct file *, void *, size_t, loff_t *);
                ^~~~~~~~~~~
  CC [M]  /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.o
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c: In function 'wl_pci_probe':
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c:777:2: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
  if ((val & 0x0000ff00) != 0)
  ^~
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c:779:3: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
   bar1_size = pci_resource_len(pdev, 2);
   ^~~~~~~~~
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c: In function 'wl_init_timer':
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c:2360:2: error: implicit declaration of function 'init_timer'; did you mean 'init_timers'? [-Werror=implicit-function-declaration]
  init_timer(&t->timer);
  ^~~~~~~~~~
  init_timers
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c:2361:10: error: 'struct timer_list' has no member named 'data'
  t->timer.data = (ulong) t;
          ^
/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.c:2362:20: error: assignment from incompatible pointer type [-Werror=incompatible-pointer-types]
  t->timer.function = wl_timer;
                    ^
cc1: some warnings being treated as errors
make[4]: *** [/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/scripts/Makefile.build:317: /tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta/src/wl/sys/wl_linux.o] Error
1
make[3]: *** [/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/source/Makefile:1508: _module_/tmp/nix-build-broadcom-sta-<IP_ADDRESS>1-4.15.drv-0/broadcom-sta] Error 2
make[2]: *** [Makefile:146: sub-make] Error 2
make[1]: *** [Makefile:24: __sub-make] Error 2
make[1]: Leaving directory '/nix/store/p0ci77xjzzhz1j2glzmi8lzn6py21mg1-linux-4.15-dev/lib/modules/4.15.0/build'
make: *** [Makefile:161: all] Error 2
builder for ‘/nix/store/iklkbw44wdpwm3ix0d9y0dk4kc6zp0cm-broadcom-sta-<IP_ADDRESS>1-4.15.drv’ failed with exit code 2

It seems 65187722e needs to be cherry-picked there...

I've created a pull request for this: https://github.com/NixOS/nixpkgs/pull/35346

Fixed by #35346


Selective appending of architecture options based on CUDA version

Closes #8
As suggested in https://github.com/numba/pyculib_sorting/issues/8#issuecomment-414956112, these changes ensure that CC 2.0 architecture options are appended only for CUDA versions prior to 9.0.
I can confirm that with these changes I was able to build the libraries on a platform with the following specs:

Ubuntu 16.04.5 LTS
GeForce GTX TITAN X
CUDA 9.2.148

@seibert Any thoughts on this?


Improve README and move wiki

This PR aims to improve the content in README by taking a more conventional way of organising python code. The following changes has been made:

Take out the instructions for installation to README
Copy the wiki content to a markdown file
Take out the requirement.txt file from the installation folder
Add more details in README and adjust the order for installation libraries

@cwinsnes @y8qin Please check the changes and let me know what you think.

I like the changes proposed in this PR.
One thing I thought of: If we are trying to go towards a more conventional way of structuring the code, I think adding a src folder could be warranted.

I like the changes proposed in this PR.
One thing I thought of: If we are trying to go towards a more conventional way of structuring the code, I think adding a src folder could be warranted.

I agree, I didn't do it here because I want to I try to only change the docs related stuff in this PR. Perhaps we can do another PR for the re-organising the folders.


Enzyme

@RyanCCollins : I have added a few more commits since we last chatted about tests. Run yarn test and you will see that there are still 2 more failed tests to be fixed! Thank you.

Wow! Nice job

@RyanCCollins : Thank you! Please critique liberally.


Code coverage not accurate due to LOG

We have macro LOG() to avoid the extra evaluation if log is off or log level does not match , which gain performance win👍 . However, small side effect of it is code coverage not accurate due to the log line might not be executed.

Need to update the fixture .

Actually, this is a good side effect. We should not count logging towards code coverage, wdyt?


surface more exception info for call creds errors

addresses https://github.com/grpc/grpc/issues/8072

the ruby failure on opt/linux looks like it's running into the bidi test issue https://github.com/grpc/grpc/issues/7972

test this please

test this please

test this please

test this please


Recognize Datadog tracer environment variables

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

Feature
What did you expect to see?

Datadog tracer supports two environment variables: DD_AGENT_HOST and DD_TRACE_AGENT_PORT. These are important because in a distributed environment, Docker or K8s, Datadog correlates infra metrics with traces. Hardcoding a value in the config is not best practice when it comes to Datadog. Datadog, in a best practice, is deployed as a daemonset in K8s and one per node in Docker and a trace agent lives on each host.
In Datadog documentation it recommends setting the DD_AGENT_HOST as such:
    apiVersion: apps/v1
    kind: Deployment
     # ...
        spec:
          containers:
          - name: "<CONTAINER_NAME>"
            image: "<CONTAINER_IMAGE>"/"<TAG>"
            env:
              - name: DD_AGENT_HOST
                valueFrom:
                  fieldRef:
                    fieldPath: status.hostIP

this function which sets default Datadog values
func (c *Config) SetDefaults() {
	c.LocalAgentHostPort = "localhost:8126"
	c.GlobalTag = ""
	c.Debug = false
	c.PrioritySampling = false
}

could be rewritten to something like
func (c *Config) SetDefaults() {
	var host string
	var port string
	ddAgentHost := os.Getenv("DD_AGENT_HOST")
	if ddAgentHost == "" {
		host = "localhost"
	} else {
		host = ddAgentHost
	}
	ddAgentPort := os.Getenv("DD_TRACE_AGENT_PORT")

	if ddAgentPort == "" {
		port = "8126"
	} else {
		port = ddAgentPort
	}
	c.LocalAgentHostPort = net.JoinHostPort(host,port)
	c.GlobalTag = ""
	c.Debug = false
	c.PrioritySampling = false
}

to recognize Datadog default values,  accept static config, as well as work with dynamic environments.

Hello @themattmck,
Thank you for your interest in Traefik !
This would be a good enhancement, do you plan to open a pull request ?

@rtribotte I've tried doing two. One branch from master as well as one branch from release2.3. Both told me I did not have permissions to write a branch. I couldn't find anything in docs that talked about one, do I have to fork?

Hello @themattmck,
Yes, you must create a fork and create the branch on your fork to be able to create your pull request.
You can also take a look at the contribution part of our documentation.


Update the repo

This hasn't been updated in a while

It has now, we're updating once a new feature is done.


Add vcpkg installation instructions

duktape is available as a port in vcpkg, a C++ library manager that simplifies installation for duktape  and other project dependencies. Documenting the install process here will help users get started by providing a single set of commands to build duktape, ready to be included in their projects.
We also test whether our library ports build in various configurations (dynamic, static) on various platforms (OSX, Linux, Windows: x86, x64) to keep a wide coverage for users.
I'm a maintainer for vcpkg, and here is what the port script looks like. We try to keep the library maintained as close as possible to the original library.

Thanks! Could you add yourself to the first list in AUTHORS.rst?

There seem to be conflicts that need resolution, please rebase to master.

@svaarala
Thanks for your review and kindly reminder.

Thanks! :+1:


I got the wrong result.

I use this library,and got all the $particleID num string like "2.5100994530733E+14".Is this the right result? The $particleID is it like "4611692470816737853"?

const EPOCH should be (13 digits) UNIX time in milliseconds<PHONE_NUMBER>213 = GMT: Tue, 07 Jun 2016 23:50:13.213 GMT).

Yes，I use the 13digits Unix  time as the epoch.But it still abnormal.Is this duel to the 32 bit PHP parser?

If your OS is 32bit... this won't work.


Dont override language_level from directives when set to None

Fixes #5484

PR is approved and no other comments appeared, so I am closing this PR.


Feature/src layout

IMPORTANT: Please search among the Pull requests before creating one.
Description

TODO.
How has this been tested?

N/A
Closes

#953
#979
#990
#992

Great to have this merged!


Add npm badge to README

Same badge as the TypeScript team has here
Feel free to close if this isn't something you want here though :)

sure, thanks!


As a user, i should be able to add headers to a Json Web Source

Need the ability to add headers for a web request.

Hello, yep that's definitely on the list. I'll prioritise and get you something to play with in the next day or so.

I've added this to v0.0.4. The UI is a little naff, my XAML clearly needs work! I'll clean it up in a later release.
You can also intercept and modify a web request before it is sent at runtime using the methods mentioned on the release page, which should give you some more flexibilty too. Let me know how it works out for you.


It is designed to promote education and the improvement of computer/cyber security.
The authors or employers are not liable for any illegal act or misuse performed by any user of this tool.
If you plan to use this content for illegal purpose, don't.  Have a nice day :)
[*] Downloading malware domain list from http://mirror1.malwaredomains.com/files/justdomains
[] Fetching expired or deleted domains containing "pro"
[]  https://www.expireddomains.net/domain-name-search/?q=pro&fwhois=22&falexa=1
[]  https://www.expireddomains.net/domain-name-search/?start=25&q=pro&fwhois=22&falexa=1
[]  https://www.expireddomains.net/domain-name-search/?start=50&q=pro&fwhois=22&falexa=1
[*]  https://www.expireddomains.net/domain-name-search/?start=75&q=pro&fwhois=22&falexa=1
[*] 78 of 78 domains discovered with a potentially desireable categorization!
[] Search complete
[] Log written to 20180818_113249_domainreport.html
Traceback (most recent call last):
File "C:\domainhunter-master\domainhunter.py", line 653, in
print(drawTable(header,sortedDomains))
File "C:\domainhunter-master\domainhunter.py", line 283, in drawTable
return(t.draw())
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\site-packages\texttable.py", line 372, in draw
out += self._draw_line(self._header, isheader=True)
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\site-packages\texttable.py", line 547, in _draw_line
line = self._splitit(line, isheader)
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\site-packages\texttable.py", line 586, in _splitit
array.extend(textwrap.wrap(c, width))
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\textwrap.py", line 379, in wrap
return w.wrap(text)
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\textwrap.py", line 354, in wrap
return self._wrap_chunks(chunks)
File "C:\Users\jaimin26783\AppData\Local\Programs\Python\Python36-32\lib\textwrap.py", line 248, in _wrap_chunks
raise ValueError("invalid width %r (must be > 0)" % self.width)
ValueError: invalid width -3 (must be > 0)

Hello, I wasn't able to reproduce your issue with the current code-base. I also tried with several other queries that returned valid domains without any observed errors. Closing this issue for now.
python3 ./domainhunter.py -k pro -w 5
<title omitted>
Expired Domains Reputation Checker
Authors: @joevest and @andrewchiles

DISCLAIMER: This is for educational purposes only!
It is designed to promote education and the improvement of computer/cyber security.
The authors or employers are not liable for any illegal act or misuse performed by any user of this tool.
If you plan to use this content for illegal purpose, don't.  Have a nice day :)

[*] Downloading malware domain list from http://mirror1.malwaredomains.com/files/justdomains

[*] Fetching expired or deleted domains containing "pro"
[*]  https://www.expireddomains.net/domain-name-search/?q=pro&fwhois=22&falexa=0
[*]  https://www.expireddomains.net/domain-name-search/?start=25&q=pro&fwhois=22&falexa=0
[*]  https://www.expireddomains.net/domain-name-search/?start=50&q=pro&fwhois=22&falexa=0
[*]  https://www.expireddomains.net/domain-name-search/?start=75&q=pro&fwhois=22&falexa=0
[-] No domain results found or none are currently available for purchase!


BulkModulusWorkChain is not compatible with RelaxWorkChain in aiida-vasp

BulkModulusWorkChain and example/submit.py do not seem to reflect the following two changes in RelaxWorkChain in aiida-vasp:

input parameters for relaxation are gathered in an input namespace, relax
https://github.com/aiida-vasp/aiida-vasp/commit/5e36317c92b04587aa185827eead314a2486fbdd#diff-ad6a501c3a7269b82ff59c5d11795452
A relaxed structure are stored at self.outputs.relax.structure not self.outputs.relaxed_structure
https://github.com/aiida-vasp/aiida-vasp/commit/1a2a55333570631931ba555605b34e762f0cc7d4

As to 2, nested outputs do not work currently ( https://github.com/aiida-vasp/aiida-vasp/issues/301
).
So, a relaxed structure can be accessed by outputs['relax__structure'] for the present.

I think we should discuss with aiida-vasp team. See https://github.com/aiida-vasp/aiida-vasp/issues/311.

Thank you for your reply.
I think it's better to update changes because the aiida-vasp tutorial instructs to install this repo.

@atztogo @lan496 Thanks for pointing this out. Yes, please update accordingly. We have changed the structure of the workchains with respect to inputs and ouputs such that they now reside in namespaces. E.g. for the relaxation workchain, specific inputs related to that should now be put in e.g. inputs.relax.someparameter etc. Please have a look at the updated definition of the relaxation workchain: https://github.com/aiida-vasp/aiida-vasp/blob/develop/aiida_vasp/workchains/relax.py
Thanks for all the effort. That is greatly appreciated.


[Core] Bump to OpenAssetIO-alpha.14

Also bump OpenAssetIO-MediaCreation for good measure, though it's not technically necessary

Merge when ready.


Nathaniel/frontend skeleton

Add skeleton code for frontend
Ran
npx create-react-app frontend --template typescript

At the very least, this needs CI

At the very least, this needs CI

CI added and passing. A bunch of warnings though - not sure if we should care about those...


power shell executed but hang on the job

When I use this plugin for test, the test file is test.ps1. The gocd version is 15.1.0 , jdk1.8-25, Win7
I can see the result output in the gocd's console:
15:20:08.592 [go] Start to execute task: Plugin with ID: gocd-powershell-runner.
15:20:08.593 Launching command: C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe -NonInteractive -NoProfile -ExecutionPolicy ByPass -File test.ps1 -version 2.1
15:20:08.832 Executing script
15:20:08.832 Version passed in is
15:20:08.835 Is 64 bit:
15:20:08.837 Argument: -version 2.1
even if I add <NUL  as the last script paramter cannot solve this problem.
Do we need add process.getOutputStream().close() to close the process?
But the job always in the building state,after 5 minutes there is warning tip said that maybe the agent hang on.
And in windows process I can see the powershell process exist

Hello there.
I believe there is a bug with the version of the plugin , I have been using the ":" to pass PowerShell parameters to the script itself
So to get yours to work , under the configuration section of the powersehll task,
Under Arguments to script:
-version:2.1
This should work at run time just fine.
you should see
Launching command: C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe -NonInteractive -NoProfile -ExecutionPolicy ByPass -File test.ps1 -version:2.1
I had to reverse engineer this one to find the workaround, let me know if that works for you.
Best of Luck.

while, add Arguments -version:2.1 to script also can't solve the problem.The output is :
19:37:45.427 [go] Start to execute task: Plugin with ID: gocd-powershell-runner.
19:37:45.436 Launching command: C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe -NonInteractive -NoProfile -ExecutionPolicy ByPass -File test.ps1 -version:2.1
19:37:53.458 Executing script
19:37:53.467 Version passed in is 2.1
19:37:53.540 Is 64 bit:
the process powershell.exe still exist in window's task manager.And the job in gocd at the state building.

There is progress here , we not can see the argument "Version" has a value of 2.1
The problem  of the process hang is not related to the plugin in general  but rather the execution policy of scripts on your host.
To be more specific
Set-ExecutionPolicy Unrestricted
can help .
But to make sure you are doing this correctly , try to run your script with the same command arguments as from your own console.
If that runs  fine , then it is an issue running the script under the LocalSystem identity ( that is the default for the GO server and Agent services.
to overcome this problem you need to adjust the execution policy for running the scripts.
read more details here .
http://www.howtogeek.com/204166/how-to-configure-windows-to-work-with-powershell-scripts-more-easily/
Best of luck

Sorry to report the result late, I have use the command "powershell set-executionpolicy remotesigned" before I submit this problem. because I execute the powershell script occur the problem "PowerShell Script is disabled on the system" at local. After change the ExecutionPolicy I can execute the powershell script at local.
As Gergues suggest I Change the ExecutionPolicy  mode to Unrestricted but the problem also exist.
here is my config and execute result:

I am sorry for the delay too, I didn't see this message till now.
Apparantly there is somthing in your PowerShell script causing the hang.
are  you able to execute the script from the command line ?
If yes , are you prompted with any input ?? some script wait for input and that is definitely not ideal for GO .
one other thing is to use some debug statements on your script to show it started , etc like
write-host "Started script"
write-host "connecting to db"
etc.
The other possible isssue is the full path to the script , you can try to path that instead of just the file name.
Best of luck.

遇到同样的问题了 @fivesix 请问解决了吗？


RTSP stream detection failed

After I generate the engine file, I want to detect RTSP. The RTSP stream is available, but I get the following error message。Can I have some suggestions？
lqs@lqs-desktop:~/tensorrt_demos_yolov3_4$ python3 trt_yolo.py --rtsp<EMAIL_ADDRESS>-m yolov4-416

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_caps_is_empty: assertion 'GST_IS_CAPS (caps)' failed

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_caps_truncate: assertion 'GST_IS_CAPS (caps)' failed

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_caps_fixate: assertion 'GST_IS_CAPS (caps)' failed

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_caps_get_structure: assertion 'GST_IS_CAPS (caps)' failed

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_structure_get_string: assertion 'structure != NULL' failed

(python3:10138): GStreamer-CRITICAL **: 13:38:51.201: gst_mini_object_unref: assertion 'mini_object != NULL' failed
NvMMLiteOpen : Block : BlockType = 261
NVMEDIA: Reading vendor.tegra.display-size : status: 6
NvMMLiteBlockCreate : Block : BlockType = 261
Allocating new output: 1920x1088 (x 11), ThumbnailMode = 0
OPENMAX: HandleNewStreamFormat: 3605: Send OMX_EventPortSettingsChanged: nFrameWidth = 1920, nFrameHeight = 1080
[ WARN:0] global /home/nvidia/host/build_opencv/nv_opencv/modules/videoio/src/cap_gstreamer.cpp (896) open OpenCV | GStreamer warning: unable to query duration of stream
[ WARN:0] global /home/nvidia/host/build_opencv/nv_opencv/modules/videoio/src/cap_gstreamer.cpp (933) open OpenCV | GStreamer warning: Cannot query video position: status=1, value=0, duration=-1
Traceback (most recent call last):
  File "/home/lqs/tensorrt_demos_yolov3_4/utils/yolo_with_plugins.py", line 257, in __init__
    allocate_buffers(self.engine, grid_sizes)
  File "/home/lqs/tensorrt_demos_yolov3_4/utils/yolo_with_plugins.py", line 153, in allocate_buffers
    stream = cuda.Stream()
AttributeError: module 'pycuda.driver' has no attribute 'Stream'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "trt_yolo.py", line 116, in <module>
    main()
  File "trt_yolo.py", line 103, in main
    trt_yolo = TrtYOLO(args.model, (h, w), args.category_num)
  File "/home/lqs/tensorrt_demos_yolov3_4/utils/yolo_with_plugins.py", line 259, in __init__
    raise RuntimeError('fail to allocate CUDA resources') from e
RuntimeError: fail to allocate CUDA resources

AttributeError: module 'pycuda.driver' has no attribute 'Stream'
This indicates you might have problem with your "pycuda" installation...

AttributeError: module 'pycuda.driver' has no attribute 'Stream'
This indicates you might have problem with your "pycuda" installation...

It worked after I installed it again。

Under folder ssd，I see the “trt_version=5” and “trt_version=6 ” in the file"install.sh". Can I use "Demo3 SSD" in "trt_version=7"?

Under folder ssd，I see the “trt_version=5” and “trt_version=6 ” in the file"install.sh". Can I use "Demo3 SSD" in "trt_version=7"?

Yes.  The code works for TensorRT 7.x.
The "FlattenConcat" plugin is built-in in TensorRT 7+.  That's why I don't need to included a pre-compiled .so file of it for TensorRT 7+ anymore.

Under folder ssd，I see the “trt_version=5” and “trt_version=6 ” in the file"install.sh". Can I use "Demo3 SSD" in "trt_version=7"?

Yes. The code works for TensorRT 7.x.
The "FlattenConcat" plugin is built-in in TensorRT 7+. That's why I don't need to included a pre-compiled .so file of it for TensorRT 7+ anymore.

Great！
You mean when TensorRT version is 7, I don't need to execute the command line "$/ install.sh "？
Can I just use the following command to build the engine？：
$ cd  $ {HOME} / project / tensorrt_demos / ssd
$ ./build_engines.sh

You still need to run the "install.sh" script in the "ssd" directory.  It will install "pycuda" and patch "graphsurgeon" for you.  The script will not make the symbolic link to the .so file if TensorRT version is not 5 or 6.  Refer to the code below:
https://github.com/jkjung-avt/tensorrt_demos/blob/3ec5d15479515669552efb0e7285b4e8ee317def/ssd/install.sh#L32-L34

You still need to run the "install.sh" script in the "ssd" directory. It will install "pycuda" and patch "graphsurgeon" for you. The script will not make the symbolic link to the .so file if TensorRT version is not 5 or 6. Refer to the code below:
https://github.com/jkjung-avt/tensorrt_demos/blob/3ec5d15479515669552efb0e7285b4e8ee317def/ssd/install.sh#L32-L34

Oh, I see. Your quick reply has made my progress very smooth. Thank you again.


Unable to compute similarity between most of entities

I was trying to compute similarity between two given entities. semantic_similarity & resnik_sim works for few entities but it's giving an error  return max(common_parent_go_ids(terms, go), key=lambda t: go[t].depth) ValueError: max() arg is an empty sequence
It issues this error when these is no common parent in both provided entities/genes. Here is one example producing this error
semantic_similarity(GO:0003676, GO:0007516, go)
Is there any alternative to compute similarity measure between such entities who doesn't share common parents. I'm sorry If I have missed something.

I changed the code so that by default, semantic_distance returns None if two GO IDs are on different branches, like biological_process and molecular_process as in your example. Please pull the latest code and see what you think.
>>> godag = GODag('go-basic.obo')
>>> go_mf = 'GO:0003676' # level-03 depth-03 nucleic acid binding [molecular_function]
>>> go_bp = 'GO:0007516' # level-04 depth-05 hemocyte development [biological_process]
>>> sim_d = semantic_distance(go_mf, go_bp, godag)
None

Or you can override the default behavior and assign a distance between two branches like this:
>>> sim_d = semantic_distance(go_mf, go_bp, godag, branch_dist=2) # 10 = 3 + 5 + 2
10

The latest GO DAG has 17 depths for the biological_process branch, so that may factor into your decision about how large to make the distance (branch_dist) between the top of biological_process and molecular_process:
>>> from goatools.rpt_lev_depth import prt_lev_depth
>>> prt_lev_depth(godag)

go-basic.obo: fmt(1.2) rel(2017-07-21) 48,990 GO Terms
Dep <-Depth Counts->  <-Level Counts->
Lev   BP    MF    CC    BP    MF    CC
--- ----  ----  ----  ----  ----  ----
00     1     1     1     1     1     1
01    24    20    22    24    20    22
02   140   137   197   262   164   369
03  1033   509   540  2027   760  1265
04  2107  1530   657  4691  1878  1448
05  3586  4696  1026  7210  4913   688
06  4549  1828   809  7284  1946   248
07  4806  1029   572  5060   900    65
08  4258   578   240  2100   362     9
09  3649   308    46   732   119     0
10  2557   170     4   195    42     0
11  1585   171     1    47    22     0
12   917    68     0     1     0     0
13   322    55     0     0     0     0
14    71    17     0     0     0     0
15    21     9     0     0     0     0
16     8     1     0     0     0     0

Thank you for your interest in GOATOOLS and time to report this issue. We have modified the software so that you should no longer see the error that you reported and added an additional related option which might be helpful to you semantic_distance. Please feel free to reopen an issue if the changes have not met your needs or you have a new item. Thank you again for your time and interest in GOATOOLS.


Issues building from source on NixOS: ignoring untrusted substituter, attribute 'cardano-sl-static' not found, URLs 404

Hey folks, I've been trying to build cardano-sl from source, on NixOS. I've ran into three small issues so far:
1. I had issues setting up the binary cache.
First, the build docs for adding the binary cache are slightly off for NixOS, where you're discouraged from editing /etc/nix/nix.conf manually. Instead, man nix.conf explains how a user can set their own config at ~/.config/nix/nix.conf, so I put those two build-cache configuration lines in there.
But then, I get the following error:
warning: ignoring untrusted substituter 'https://cache.nixos.org'
warning: ignoring untrusted substituter 'https://hydra.iohk.io'

There is some discussion of this problem on https://github.com/cachix/cachix/issues/26 , with workarounds given. I tried to follow those, but wasn't successful. The contents of my ~/.config/nix/nix.conf are:
jgl@n3: ~/cardano-sl $ cat ~/.config/nix/nix.conf
binary-caches            = https://cache.nixos.org https://hydra.iohk.io
binary-cache-public-keys = hydra.iohk.io:f/Ea+s+dFdN+3Y/G+FDgSq+a5NEWhJGzdjvKNGv0/EQ=
trusted-substituters     = https://cache.nixos.org https://hydra.iohk.io
trusted-users            = root jgl

I kept going, just building things straight from source without the binary cache. That lead to the next two issues:
2. cardano-sl-static doesn't seem to be a build target anymore?
The docs give the nix-build command as:
    $ nix-build -A cardano-sl-static --cores 0 --max-jobs 2 --no-build-output --out-link master

but that errors with:
    error: attribute 'cardano-sl-static' in selection path 'cardano-sl-static' not found

Looking in default.nix, it looks like this is now called cardano-sl-node-static. Using that attribute seems to work.
(nit pick, why does this command specify the cores/max-jobs flags?  I've been forgoing those for faster builds, personally. Maybe that follows some internal IOHK convention, but it seems weird to include in the public docs)
3.   The requested URL returned error: 404 Not Found
I hit 404's for two URLs:
https://anonscm.debian.org/cgit/collab-maint/ca-certificates.git/plain/mozilla/certdata2pem.py?h=debian/20170717
https://anonscm.debian.org/viewvc/pkg-gnome/desktop/unstable/intltool/debian/patches/perl5.22-regex-fixes?revision=47258&view=co&pathrev=47258
(example output in this gist )
Looks like this is fallout from Debian migrating anonscm.debian.org, there is some discussion here https://github.com/NixOS/nixpkgs/issues/39927
The first 404 i was able to get past by running nix-prefetch-url https://salsa.debian.org/debian/ca-certificates/raw/debian/20170717/mozilla/certdata2pem.py  (from here https://github.com/cachix/cachix/issues/25 )
I haven't been able to get past the second 404 though. The new URL for that patch seems to be https://sources.debian.org/data/main/i/intltool/0.51.0-5/debian/patches/perl5.26-regex-fixes.patch
, but the contents have changed, so the hash is different. So even if i nix-prefetch-url, it'll have anew hash, which means that nix-build will still try and pull the old version from the URL that 404s.
However, as I was digging into this, I found that this specific issue has already been fixed in nixpkgs. So I think the real issue here, is that whatever nixpkg that cardano-sl is using that has intltool as a dependency, is being pinned to version that is too old.
I tried to grok how pinning of nixpkgs is happening here, but didn't get too far before i gave up (for example, whats the difference between cardano-sl/fetch-nixpkgs.nix and cardano-sl/fetchNixpkgs.nix?)

(btw, thanks for open sourcing all of this! its been a fun way to dive deeper into Nix :slightly_smiling_face:)

Install nix according to the official method. I had issues
nix-build: command not found

can you help me  thx

Hi @jglukasik, thanks for your interest!
In the develop branch we have updated the nixpkgs pin and fixed the fetchNixpkgs.nix thing. If you're interested in playing with the code then the develop branch is better than master.
I very strongly recommend using the binary cache.
I added docs for NixOS in #3860.


xWebAppPool's Tests Need to be Updated

They don't currently test set-targetresource, should also look into how much coverage we get with the integration tests.
I'm currently working on this and am just filing an issue to reference when I submit the PR.

Added in #77.


SSHClient

Add an SSHClient class so it doesn't have to import it from TethysCluster.

Started implementation, but am getting SSHExceptions when trying to connect with paramiko.SSHClient. This issue will need to be moved back to version 0.2.


APT whitelist request for libdnet-dev

This replaces travis-ci/travis-ci#4296.
The original text by @fxlb follows

Needed for tcpdump project

Ran tests and found setuid bits by purely textual search. Further analysis is required.
If these are found to be benign, add:
dnet-common dnet-progs libdnet libdnet-dev
See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72545617.