fix: make sure binary install dir exists

I'm trying to install node_exporter on a remote host with a custom node_exporter_binary_install_dir.
- hosts: all
  roles:
    - prometheus.prometheus.node_exporter
  vars:
    node_exporter_binary_local_dir: /home/namsic/downloads/node_exporter-1.6.0.linux-amd64
    node_exporter_binary_install_dir: /home/namsic/tmp

But when I ran the above playbook, I got the following result:
TASK [prometheus.prometheus.node_exporter : Propagate locally distributed node_exporter binary] ****************************************************************************************************************************************************************************************************************************************************************************
fatal: [remote001]: FAILED! => {
    "changed": false,
    "checksum": "894db4cbc1bda8d686d73f33f22c57c861836c9d",
    "msg": "Destination directory /home/namsic/tmp does not exist"
}

If there are multiple target hosts, it would be nice to automatically create the directory if it does not exist.
If these changes are not appropriate, please let me know.
Thanks.

Can I just add the same change to every roles?
fix: ensure binary_install_dir before copy binary

Or should I do something else?

Can I just add the same change to every roles?
fix: ensure binary_install_dir before copy binary

Or should I do something else?

You can add a task to the install tasks file of the _common role which all the roles use.
Adding this task before the last task, the one named "Propagate binaries", in _common/tasks/install.yml would probably do the trick.
- name: "Make sure binary install dir exists"
  ansible.builtin.file:
    path: "{{ _common_binary_install_dir }}"
    mode: 0755
    owner: root
    group: root
  become: true
  tags:
    - "{{ ansible_parent_role_names | first | regex_replace(ansible_collection_name ~ '.', '') }}"
    - install
    - "{{ ansible_parent_role_names | first | regex_replace(ansible_collection_name ~ '.', '') }}_install"

Thanks for the detailed explanation. I changed it according to your comment.

I think we should use ansible.bultin.stat as a preflight check, rather than actually try and create the target directory.

Can someone help me?
I can't figure out how my ansible.builtin.stat changes relate to the error below.
INFO     Running alternative > destroy
INFO     Sanity checks: 'docker'
[WARNING]: Collection community.docker does not support Ansible version
2.11.12.post0

PLAY [Destroy] *****************************************************************

TASK [Destroy molecule instance(s)] ********************************************
fatal: [localhost]: FAILED! => {
   "msg": "Could not find imported module support code for ansible_collections.community.docker.plugins.modules.docker_container.
           Looked for (['ansible.module_utils.compat.version.LooseVersion', 'ansible.module_utils.compat.version'])"
}

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

WARNING  Retrying execution failure 2 of: ansible-playbook --inventory /root/.cache/molecule/alertmanager/alternative/inventory --skip-tags molecule-notest,notest /usr/local/lib/python3.6/dist-packages/molecule_docker/playbooks/destroy.yml
CRITICAL Ansible return code was 2, command was: ['ansible-playbook', '--inventory', '/root/.cache/molecule/alertmanager/alternative/inventory', '--skip-tags', 'molecule-notest,notest', '/usr/local/lib/python3.6/dist-packages/molecule_docker/playbooks/destroy.yml']

Can someone help me?
I can't figure out how my ansible.builtin.stat changes relate to the error below.
INFO     Running alternative > destroy
INFO     Sanity checks: 'docker'
[WARNING]: Collection community.docker does not support Ansible version
2.11.12.post0

PLAY [Destroy] *****************************************************************

TASK [Destroy molecule instance(s)] ********************************************
fatal: [localhost]: FAILED! => {
   "msg": "Could not find imported module support code for ansible_collections.community.docker.plugins.modules.docker_container.
           Looked for (['ansible.module_utils.compat.version.LooseVersion', 'ansible.module_utils.compat.version'])"
}

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

WARNING  Retrying execution failure 2 of: ansible-playbook --inventory /root/.cache/molecule/alertmanager/alternative/inventory --skip-tags molecule-notest,notest /usr/local/lib/python3.6/dist-packages/molecule_docker/playbooks/destroy.yml
CRITICAL Ansible return code was 2, command was: ['ansible-playbook', '--inventory', '/root/.cache/molecule/alertmanager/alternative/inventory', '--skip-tags', 'molecule-notest,notest', '/usr/local/lib/python3.6/dist-packages/molecule_docker/playbooks/destroy.yml']

That issue is unrelated to your change, the docker ansible collection which is used for the tests was updated recently and that broke compatibility with old ansible versions.
I fixed the issue in #440


Not relying on the key in assets e.g. users do not need to use filename as keys

Hi @ngachung
I remember this is where you have arbitrary names like "data1", "data2", for asset keys.
Is this an issue during download or upload?


Add listing and landing page for JFrog webinar

This PR adds the JFrog webinar to the webinar listing page and the superpower page. I created an image for this, but feel free to replace it if we have a better one.
Fixes: https://github.com/pulumi/docs/issues/3112

@zchase, looks like the build is failing due to a lint error:
/home/runner/work/docs/docs/content/webinars/sharing-and-reusing-infrastructure/index.md:
57
Line File Header: Page title exceeds 60 characters.


Computer Vision package - Tutorial Group

This PR adds 3 tutorials which will form a Group for Computer Vision:

Setup SAP Computer Vision package and its dependencies
Train Object Detection model with SAP Computer Vision Package
Read Electric Meters with AI of Computer Vision Package

closed. Will create new PRs for individual tutorials


[Bug]: Custom system.out messages are not being printed

reported by: @MhmdElGazzar

should look like this now after resetting your properties directory


herdocs/multi-line strings with interpolation

This is essential for easy client-side templates.

:+1:


Set Load Balancer internal/external per process instead of per application

Introduces EMPIRE_X_EXPOSURE which needs to be public.
If that environment variable is not set, we fallback to the default selection via application (where you use domain-add).
Added this explanation in the Procfile doc.
Fixes #1076
Things working and tested:

[x] Creating a new application using Procfile as defined here https://github.com/iserko/flask_test/blob/f369c115fd41a0086d364f38b693a353963c041d/Procfile :white_check_mark:
[ ] Modifying a running application by setting this environment variable and deploying ... :warning: Breaks CloudFormation as a Target Group can only be assigned to one Load Balancer, so we'd need to create a separate Target Group? :x:
[ ] Setting an environment variable via emp env using old Procfile format
[ ] Migrating from old Procfile to new Procfile format

@ejholmes Thoughts?

@ejholmes I've just rebased and pushed additional changes.
The TargetGroup prefix was really easy to put it, so that wasn't the problem.
The biggest problem was moving the EMPIRE_X_EXPOSURE var into releases.go
At that point the app and process Environments aren't merged yet (it happens in twelvefactor.go). So I exposed the merge function by capitalizing it and using it in releases.go.
Please let me know if you have comments either via Github or if you have time in the next few hours I'll be online via Slack


Remember scroll position in SCM changes list when switching to other views and back

Issue Type: Feature Request
Not remembering the scroll position makes it very annoying in certain situations, esp. with long change lists during merging of upstream changes. It also resets to the top. The explorer on the other hand remembers the position.
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:09.080Z)
OS version: Windows_NT ia32 10.0.17134

I would like to try to resolve this.

Since I saw no changes going on, I made a fix (PR #74723). Hope it helps.


The role of manifest.json requires clarification.

Apologies for the rather demanding reading title; I wanted to use a title more suitable for an issue.
I have now read README a number of times, trying to piece things together. I am using the image sources under the src directory to create devcontainer images with various customisations. These image source(s in folders)  seem to work just fine with vs code's devcontainers support.
Except I am failing to understand whether or not manifest.json has any effect on the contents of the actual image that is created. My understanding is that this file is only used for publishing to MCR, but there are statements in the README that confuse me such as:

The manifest.json file dictates how the build process should behave as described above. In this case, devcontainer.json points to Dockerfile used to generate the actual image.

I fail to understand what "how the build process should behave" means here. It is even more confusing trying to understand what manifest.json has to do with devcontainer.json pointing at the Dockerfile (and where in the build process that pointing happens?).
If someone could briefly clarify whether or not manifest.json actually changes the contents (packages, config etc) of the built docker image I would be grateful. Is it safe for me to delete this file and hijack .devcontainers folder only to build my own images?
In other words, I cannot build the big picture from the README, some help would be much appreciated :)

Thank you for sharing your question.
To provide an example of how the defintions inside manifest.json affects the image build process, let us take a look at the python image.
Tags for this image uses the VARIANT property:
https://github.com/devcontainers/images/blob/722d41d4a36ba68ea300beb2c509f6a74a6e8c2d/src/python/manifest.json#L76-L78
Following are the variants defined for this image:
https://github.com/devcontainers/images/blob/722d41d4a36ba68ea300beb2c509f6a74a6e8c2d/src/python/manifest.json#L3-L18
Based on these definitions the build process builds and deploy multiple images (one for each variant), as you can see in MCR https://mcr.microsoft.com/en-us/product/devcontainers/python/tags
You can read more about this here in the README

thanks for taking the time to respond @eljog . What I'm trying to confirm is that manifest.json is meant to be consumed by a specific image building pipeline, which perfoms actions such as building and deploying multiple images, but this is a mechanism on top of the devcontainer spec and its tooling . I.e. if I delete the manifest.json and use .devcontainer folder, I could still get a devcontainer image built, say by vs code. I would not have broken packages in the produced image or missing dependencies etc.
In other words, my understanding is that manifest.json is not part of the devcontainers spec, but it is a meta artefact used in a specific build pipeline (from Microsoft). Is that correct?

That is correct.
You can build an image locally from sources using the config inside the .devcontainer directory.
Typically you don't need to build the image from source, instead you can use the images that are built and published in MCR https://mcr.microsoft.com/en-us/catalog?search=devcontainers
Devcontainer templates can be a good starting point to build your development environment using these images: https://containers.dev/templates

Brilliant. Thanks again. I saw the templates page but completely forgot about it. Closing the issue now that it is clear what manifest.json does.


Add enable_proxies/disable_proxies, deprecate enable_proxy/disable_proxy/_get_proxy

Preparation for breaking change https://github.com/pylast/pylast/pull/379.
Changes proposed in this pull request:

#379 will switch to using HTTPX, which allows multiple proxies:

https://github.com/pylast/pylast/pull/379#issuecomment-1020238940

To help prepare, let's make a transitional pylast 4.5.0 release
We'll add the new names now and deprecate the old ones. The old ones will be removed in 5.0.0:

-enable_proxy()
+enable_proxies()
-disable_proxy()
+disable_proxies()

Note enable_proxies() will have different parameters in 5.0.0.

This will be renamed. Both will be in 4.5.0, only the plural in 5.0.0:

-self.proxy
+self.proxies

_get_proxy will be removed in 5.0.0, use proxies directly instead.

Not needed, we're going to keep enable_proxy() / disable_proxy() / self.proxy names, and _get_proxy is private so doesn't need deprecating.


refactor!: channel descriptor -> notification channel descriptor

This PR was generated using Autosynth. :rainbow:
Here's the log from Synthtool:
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:0e6f3a668cd68afc768ecbe08817cf6e56a0e64fcbdb1c58c3b97492d12418a1
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/monitoring/artman_monitoring.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/monitoring-v3.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
synthtool > Replaced 'https:\\/\\/cloud\\.google\\.com[\\s\\*]*http:\\/\\/(.*)[\\s\\*]*\\)' in src/v3/doc/google/protobuf/doc_timestamp.js.
synthtool > No replacements made in **/doc/google/protobuf/doc_timestamp.js for pattern toISOString\], maybe replacement is not longer needed?
synthtool > Replaced 'Sum\\[i=1\\.\\.n\\]\\(https:\\/\\/cloud\\.google\\.com\\(x_i - mean\\)\\^2\\)' in src/v3/doc/google/api/doc_distribution.js.
npm WARN deprecated<EMAIL_ADDRESS>Please note that v5.0.1+ of superagent removes User-Agent header by default, therefore you may need to add it yourself (e.g. GitHub blocks requests without a User-Agent header).  This notice will go away with v5.0.2+ once it is released.

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node scripts/postinstall || echo "ignore"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 911 packages from 1306 contributors and audited 5582 packages in 20.825s
found 0 vulnerabilities

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> eslint '**/*.js' --fix

/tmpfs/src/git/autosynth/working_repo/samples/alerts.js
   30:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
   69:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  124:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  159:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  224:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  268:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/metrics.js
   29:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
   77:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  102:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  135:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  160:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  212:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  252:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  292:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  337:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  387:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  424:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/quickstart.js
  20:28  error  "@google-cloud/monitoring" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/test/alerts.test.js
  18:28  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  19:26  error  "chai" is not found                      node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/test/metrics.test.js
  18:28  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  19:26  error  "chai" is not found                      node/no-missing-require
  21:23  error  "p-retry" is not found                   node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/test/quickstart.test.js
  18:26  error  "chai" is not found     node/no-missing-require
  20:23  error  "p-retry" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/test/uptime.test.js
  18:26  error  "chai" is not found  node/no-missing-require

/tmpfs/src/git/autosynth/working_repo/samples/uptime.js
   29:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
   82:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  124:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  145:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  188:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require
  221:30  error  "@google-cloud/monitoring" is not found  node/no-missing-require

✖ 32 problems (32 errors, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>fix: `eslint '**/*.js' --fix`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2019-09-11T11_18_27_727Z-debug.log
synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

Caused by the change to the GAPIC yaml here: https://github.com/googleapis/googleapis/commit/b4b182552fa0088e463ada73afcf48b405965c2c#diff-3d45c72891ffb2f8743849aa660d6f36R600  (search for notification_channel_descriptor).
Please do not merge it, I'll figure out what's going on.

I have submitted a rollback CL on this change. It also breaks at least C# libraries, and likely more.


[FLINK-1927][py] Operator distribution rework

Python operators are no longer serialized and shipped across the
cluster. Instead the plan file is executed on each node, followed by
usage of the respective operator object.
removed dill library
also fixed [FLINK-2173] by always passing file paths explicitly to python

Thanks for the pull request @zentol!
+1 for removing the dill library. As far as I can see, we handle all the serialization ourselves now. We only used the Dill library to serialize the user-defined function alongside with the operator. Now, the operator is extracted from the plan which has been distributed in the Python files to the nodes beforehand. The plan is only send once to generate the Java execution plan. The old behavior was to serialize the operator, pass it to Java, and sent it back again during execution. Performance-wise the new implementation could even be a bit faster.
+1 for explicitly passing the file paths. Java and Python have different ways to determine temporary file paths and this has been a problem in the passed on some platforms.
Your changes are sensible and this looks to merge.
Changes look sensible and good to me.

Thanks for the review @mxm .
I've addressed the cosmetic issue you mentioned, and added a small fix for a separate issue as well (error reporting was partially broken).

I think we can merge this later on.


ENRs for Discv5

This adds a brief outline for our design decision to drop requests from implementations that advertise non-contactable addresses in their ENR.
This issue continually comes up, so I think its necessary we add it to our README.

We have the private addresses filter in lighthouse but not here, so <IP_ADDRESS> is technically still contactable (we even use it in the examples) so we probably should discuss this only in the case in which the peer advertises a private address but it's actually sending via a public address. At the end, local discv5 deployments work anyway

Agreed, I should be more careful in my wording. Will update


VirtualDirectory.serveFile should return a Future

This issue was originally filed by<EMAIL_ADDRESS>
VirtualDirectory.serveFile of http_server library should return a Future that completes when the file has been successfully served.
It is a small change, so I made a CL: https://codereview.chromium.org/171363006

This issue has been moved to dart-lang/http_server#24.


Feature bl 4798 service unavaliable

@przemos during the build process if the lambda has changed.  The file is re-uploaded to S3 bucket in he correct location.  It can then be picked up by cloudfront.
@wbadowski the thinking behind this was to in some way separate this from the rest of the templates and also not include it as part of the lambda code.


Add Text feature

How to add text to the canvas, like in the Liang's brush App in the app store.

I got it working by myself. I created a view controller with a textview. And on save action or delegate method of textview, computed the size of the text in the textview.
Here's the code snippet if somebody needs.
let font = UIFont.systemFont(ofSize: 16)
let fontAttributes = [NSAttributedString.Key.font: font]
let myText = textView.text
let textsize = (myText! as NSString).size(withAttributes: fontAttributes)
    let size = CGSize(width: textsize.width+10, height: self.textView.contentSize.height+10)
    UIGraphicsBeginImageContextWithOptions(size, false, 0);
    self.textView.drawHierarchy(in: CGRect(x: 0, y: 0, width: self.textView.frame.size.width, height: self.textView.frame.size.height),afterScreenUpdates:true)
    let screenShot:UIImage = UIGraphicsGetImageFromCurrentImageContext() ?? UIImage()

This will give you an image with a screenshot of the text.

Here's how i did it. I created a view controller with a textview. And on save action or delegate method of textview, computed the size of the text in the textview.
Here's the code snippet if somebody needs.
let font = UIFont.systemFont(ofSize: 16)
let fontAttributes = [NSAttributedString.Key.font: font]
let myText = textView.text
let textsize = (myText! as NSString).size(withAttributes: fontAttributes)

let size = CGSize(width: textsize.width+10, height: self.textView.contentSize.height+10)
UIGraphicsBeginImageContextWithOptions(size, false, 0);
self.textView.drawHierarchy(in: CGRect(x: 0, y: 0, width: self.textView.frame.size.width, height: self.textView.frame.size.height),afterScreenUpdates:true)
let screenShot:UIImage = UIGraphicsGetImageFromCurrentImageContext() ?? UIImage()

This will give you an image with a screenshot of the text. Then you can create a charlet/texture out of this image (For this you can take help from the sampel code.)


Compress artifacts in graphics tests in CI

If graphics tests are failed then you can only download file-by-file and if there is a lot of failing test cases it takes some time.
Needs to compress the whole folder into one archive and store it as artifact besides with cmp folder.
https://circleci.com/docs/2.0/artifacts/#uploading-artifacts
https://support.circleci.com/hc/en-us/articles/360024275534?input_string=store_artifacts+step

It could be something like the following:
      - run: scripts/run-graphics-tests.sh
      - run:
          name: "Generate archive with screenshots"
          command: |
            tar -czvf ./screenshots.tar.gz graphics-cmp-data/
            mv ./screenshots.tar.gz ./graphics-cmp-data/screenshots.tar.gz

But we need to run it in a case of failed tests (currently, if we add this to CI config right after running tests, it won't run due failed previous step).


user app log file management

To facilitate searching user app log files when chasing down problems, it would be good to have distinct log files for each run of the workflow.  This might mean either a distinctive name or a distinctive path for the logs.  One option that comes to mind is to allow the user to specify the appropriate rundir, e.g., runinfo/NNN to place the log file(s).
Thank you for your consideration.

I think this is a great idea.
I'd take it farther, and lobby for this naming scheme:
runinfo/XXX/task_logs/YY_ZZ.log
where XXX=run id, YY=app name, ZZ=task id. But in #538 @benclifford mentioned he preferred having an application-specific tag for logs. So maybe YY=user-provided tag (which would take the place of the user specifying the log path).

cross-referencing #849

@annawoodard, thanks, that would work well for me.  Perfect!
One could even have both the YY=app name and TT=user tag in the file name.

One subtlety here is that this isn't so straightforward in cases where there isn't a shared FS between where the parsl script is running and where the task is running (see #276). But we can keep that factorized out as a separate issue.

In such a case (non-shared FS), where would the user app log be written by default?  Maybe that area (wherever it is) could inherit all or part of your proposed structure, i.e., .../NNN/task_logs/...

@TomGlanzman Just so you have the full picture, we don't yet support the no shared filesystem case. That broad space is being designed and worked on now as part of the data management effort. We'll still have some sort of sandboxing and directory naming mechanisms in there that takes these issues into account as @annawoodard pointed out.

We need to decide how to handle keyword args and defaults. Here are some options which would be backwards compatible:
option A
stdout='auto' -> stdout written to 'runinfo/AAA/task_logs/BBBB/task_CCC_DDD_EEE.stdout'
stdout='test.stdout' -> stdout written to 'test.stdout'
stdout=None -> no stdout (default)

option B
stdout=None -> stdout written to 'runinfo/AAA/task_logs/BBBB/task_CCC_DDD_EEE.stdout' (default)
stdout='test.stdout' -> stdout written to 'test.stdout'
stdout=False -> no stdout

option C
stdout=None -> stdout written to 'runinfo/AAA/task_logs/BBBB/task_CCC_DDD_EEE.stdout' (default)
stdout='test.stdout' -> stdout written to 'test.stdout'
no stdout in signature -> no stdout

option D
global configuration option to toggle logging-- not sure if it should override task settings?

where
AAA = run number
BBBB = int(task id / 10000)  (so we have directories with 10k entries each)
CCC = task id
DDD = app name
EEE = optional task label

These are all problematic in different ways 😢

(Just back from travel and catching up...)
IMO, both stdout and stderr should appear somewhere by default, so am less enthusiastic about option "C".  It also would seem odd (to me) if "stdout=None" actually caused output to be preserved.  Would something like this work?
stdout=True [default] -> stdout written to 'runinfo/AAA/task_logs/BBBB/task_CCC_DDD_EEE.stdout'
stdout='auto' -> stdout written to 'runinfo/AAA/task_logs/BBBB/task_CCC_DDD_EEE.stdout'
stdout='myStdout.log' -> stdout written to $PWD/myStdout.log
stdout='/a/b/c/myStdout.log' -> stdout written to /a/b/c/myStdout.log
stdout=None   -> no output
stdout=False   -> no output
And an equivalent set defined for stderr.

I dislike use of None/True/False to create a multiway choice. I think I'd prefer to make an explicit Enum giving sensible symbolic names.

I vote for Option A with some minor changes. Like @benclifford said, we should use an enum instead of a magic string. I think the task label 'EEE' in the string is a bit overkill, and we should be covering most cases with the name of the app and the task_id. When composing the app dynamically, I prefer renaming the function and then wrapping with the decorator to embed labels into the app name, so I believe there's a reasonable way around the label issue.
@TomGlanzman is right that having these logs by default is convenient in production. Parsl is not quite a workflow management system but a scripting library and I believe that as a library we should defer choices like these to the user.

Why the task label is important.
I've been using a generic app to run different processes.  The task label is the only way I have to differentiate.  It is not redundant.
I will also place my vote to make stdout/stderr produce output default.  Are there really cases in production (of any size) where one would not wish to have a log?

@TomGlanzman: Thanks for your feedback! I've implemented your request and added a label (details here. Give it a whirl at your convenience and let us know what you think.

Hi @annawoodard , this all looks great!  Many thanks!
I made an weak attempt to try out the code, but failed due to my inability to get 'pip' to install it.  Maybe I should wait until it appears in a tagged release?

Hi @TomGlanzman! Thanks for having a look! You can install the 0.8 alpha release, which picks up this change, with: pip install parsl==0.8.0a0

I have now tried this new feature in Parsl 0.8.0a0.  Both the auto log file naming, and the 'label' parameter.  All features seem to be working exactly as advertised.  Many thanks!
One question about the task ID.  I am getting file names like: .../runinfo/049/task_logs/0000/task_0019_genBF_makeBF.stderr, which is fine.  Note that if both stdout and stderr are directed to this directory, the maximum number off files is 20k rather than 10k.  My question is whether the task_id in the file name, "0019" is the full task_id or mod(task_id,10000)?  That is, will the number of digits in the file name grow, as needed?

the maximum number off files is 20k rather than 10k

Great point-- noted. Checking here it looks like the worst-case-scenario is a limit of 65k files per directory, so we should still be ok. We can adjust this down if we run into problems.

My question is whether the task_id in the file name, "0019" is the full task_id or mod(task_id,10000)?

The string is just padded with zeros, so the number of digits will grow as needed. I just picked a hard-coded padding I thought would keep the names tidy for the majority of workflow sizes. As we're a library and tasks could be being dispatched interactively in a jupyter notebook, there's no way to know the total number of tasks in order to programatically pick the appropriate padding length.

I forgot to mention-- thanks for testing this @TomGlanzman!


Release 1.27.2

What this PR does / why we need it:
This releases 1.27.2 bugfix release.
Release note:

NONE

hmm, just thinking why cinder test is failing. It works in master + release-1.28 currently

/test openstack-cloud-csi-cinder-e2e-test-release-127
1.27 used to be working I think ?

yeah it was working in https://github.com/kubernetes/cloud-provider-openstack/pull/2311

Let's see https://github.com/kubernetes/cloud-provider-openstack/pull/2349 then.

@zetaab: I just had a moment to take a closer look at the job failure. Looks like the 1.27.2 image is not available and that's why it fails:
Error from server (BadRequest): container "openstack-cloud-controller-manager" in pod "openstack-cloud-controller-manager-b9575fdc6-js89h" is waiting to start: trying and failing to pull image

Could it be because I cannot tag this as 1.27.2 and the images are not in staging? What's the procedure there?

I've rebased this to include #2352, let's see now.

It went through!

/approve
/lgtm


Create SECURITY.md

Added the security for the website
Fixes : #198
Please merge my pull request and label me level2
@notsoocool

Please merge my pull request and label me level2
@notsoocool

Please merge my pull request and label me level2
@notsoocool

Please merge my pull request and label me level2
@notsoocool
As tomorrow is the last day

Please merge my pull request and label me level2
@notsoocool
As today is the last day

Please merge my pull request and label me level2
@notsoocool
As today is the last day


Error in bind_user method

I'm on Python 3.6.3 and I get this exception when I hit line 153 in __init__.py (https://github.com/admiralobvious/flask-simpleldap/blob/master/flask_simpleldap/init.py#L153):
AttributeError: 'str' object has no attribute 'decode'
I changed it back to a recent commit you had in v1.1.2, which fixed the issue:
conn.simple_bind_s(user_dn, password)
(instead of conn.simple_bind_s(user_dn.decode('utf-8'), password))
Can we get that fix back?

I can't just do that, because it will break for someone else. The current code actually works for me on Python 3.6.3 as well. I am trying to figure out why it doesn't work for some people. Tell me, are you using OpenLDAP as your LDAP server?

Running an Active Directory server.
user_dn is returned as a string.

Running an Active Directory server.
user_dn is returned as a string.

Can we add this?
if type(user_dn) == str:
    conn.simple_bind_s(user_dn, password)
else:
    conn.simple_bind_s(user_dn.decode('utf-8'), password)

Or if you're just dealing with py3 compatibility:
if sys.version_info < (3,0,):
    conn.simple_bind_s(user_dn, password)
else:
    conn.simple_bind_s(user_dn.decode('utf-8'), password)

@rfaoro-gpsw how about you create a fork and request @admiralobvious for a pull?

@pschwede I can't do that because python 3 doesn't support string decoding.
What about this:
if sys.version_info > (3,) and type(user_dn) == str:
    conn.simple_bind_s(user_dn, password)
else:
    conn.simple_bind_s(user_dn.decode('utf-8'), password)

(Yes, I can create a fork and request @admiralobvious for a pr)

Also seeing this on Python 3.6.2.  Reverting to 1.1.2 fixed the issue for me.  By the way.. I am using Open LDAP and have the configuration set to True.

I confirm that this is MAY not a py 3 problem, it seems to be an openldap as you said @admiralobvious
here's a simple test with a basic openldap container.
docker run --env LDAP_ORGANISATION="My Company" --env LDAP_DOMAIN="my-company.com" --env LDAP_ADMIN_PASSWORD="JonSn0w"  --rm -p 9003:389  osixia/openldap:1.1.11

Then in a python script
import ldap
l = ldap.initialize("ldap://<IP_ADDRESS>:9003")
l.simple_bind_s("CN=admin,DC=my-company,DC=com", "JonSn0w")
r = l.search_s("DC=my-company,DC=com", ldap.SCOPE_SUBTREE, 'cn=admin')
print(r)

the result is
[('cn=admin,dc=my-company,dc=com', {'objectClass': [b'simpleSecurityObject', b'organizationalRole'], 'cn': [b'admin'], 'description': [b'LDAP administrator'], 'userPassword': [b'{SSHA}eqGwmHDG99GBl/q3goOay8PmqSOpm4e/']})]

to make this work with openldap you need LDAP_OPENLDAP=True in your config, then in the code here
https://github.com/admiralobvious/flask-simpleldap/blob/master/flask_simpleldap/__init__.py#L190
you return [0][0] which is a string and not a bytes.
so to "keep it the same for everyone else" you could .encode() before returning when  LDAP_OPENLDAP=True
That being said here's a disclaimer:

I don't think this solution is elegant
I only tested with the openldap container
My ldap knowledge is about 1/10

i crafted a little monkey patch, works for me:
def _monkey_patch_openldap_string_flask_simpleldap_1_2_0_issue_44(ldap_instance):
    import ldap

    def bind_user(self, username, password):
        user_dn = self.get_object_details(user=username, dn_only=True)

        if user_dn is None:
            return
        try:
            if type(user_dn) == bytes:
                user_dn = user_dn.decode('utf-8')

            conn = self.initialize
            conn.simple_bind_s(user_dn, password)
            return True
        except ldap.LDAPError:
            return

    import types
    ldap_instance.bind_user = types.MethodType(bind_user, ldap_instance)

    return ldap_instance

app = Flask(__name__)
# set configs on app
ldap_auth = _monkey_patch_openldap_string_flask_simpleldap_1_2_0_issue_44(LDAP(app))

# use @ldap_auth.*

Have a nice day :)

@ltpitt not much to explain
doc says do
ldap = LDAP(app)

@app.route('/ldap')
@ldap.login_required
def ldap_protected():
    return 'Success!'

just use my method on the first line
ldap = _monkey_patch_openldap_string_flask_simpleldap_1_2_0_issue_44 (LDAP(app))

I've also added a
print(records)
before
for k, v in list(records[0][1].items()):
in init.py and what I see is:
[(None, ['ldap://ForestDnsZones.mydomain.net/DC=ForestDnsZones,DC=ipcatalogue,DC=net']), (None, ['ldap://DomainDnsZones.mydomain.net/DC=DomainDnsZones,DC=mydomain,DC=net']), (None, ['ldap://mydomain.net/CN=Configuration,DC=mydomain,DC=net'])]

I've also added a print(records) before for k, v in list(records[0][1].items()): in init.py to better understand what is going on and what I see is:
[(None, ['ldap://ForestDnsZones.mydomain.net/DC=ForestDnsZones,DC=mydomain,DC=net']), (None, ['ldap://DomainDnsZones.mydomain.net/DC=DomainDnsZones,DC=mydomain,DC=net']), (None, ['ldap://mydomain.net/CN=Configuration,DC=mydomain,DC=net'])]

@ltpitt this does not look like the same problem.
The problem in this thread is string vs bytes and my monkey patch fixes this by introducing these lines:
            if type(user_dn) == bytes:
                user_dn = user_dn.decode('utf-8')

(Which must be line 33-34 in your file)
in your stack trace i see this:
  File "/vagrant/flask-webapp/ldap-login-test/ldap-login-test.py", line 28, in bind_user
    user_dn = self.get_object_details(user=username, dn_only=True)

which happens on line 28
So it happens before my fix in my monkey patch so i guess you have another issue completely

Merge this commit please: https://github.com/jm66/flask-simpleldap/commit/e537f2691c375eb1db103c04e3e23d638503be80

Same issue, solved applying @lindycoder code.


Fix typo in cross-platform-cryptography.md

I believe given the context, the sentence:

As authentication encryption requires ...

Should read:

As authenticated encryption requires ...

It may be worth going further and changing "As" to "Since" so it reads:

Since authenticated encryption requires ...

...but I defer to the PR reviewers on that extra tweak.

Internal previews

📄 File
🔗 Preview link

docs/standard/security/cross-platform-cryptography.md
Cross-Platform Cryptography in .NET

Thanks as always @IEvangelist for the fast follow up 💪🙏


Use TypeScript instead of C# for generator

What about using TypeScript (Node.js) instead of C# for generator, like https://github.com/GoogleChromeLabs/bubblewrap do?
I truly believe this should significantly lower the contribution threshold for front-end developers. Also it would be really nice to have a CLI in npm for packaging PWA into iOS apps.
BTW, do you have in the team or do you plan to hire an iOS developer to fix iOS-specific bugs in Swift template?

@JudahGabriel @davrous @jgw96 PTAL. :wink:

Not likely. C# is a great language for servers.
I built PWABuilder's Android platform in Node and TS, but I still find C# a better tool for server-side jobs. Consider, as an example, that Node doesn't have built-in support for HTTP/2, let alone HTTP/3, and even the mainstream 3rd party libs like node-fetch are not fully supporting it at this time. Such shortcomings have caused all kinds of headaches for us and our users.
We certainly considered Node + TS as an alternative. But we decided then, and still believe now, C# is the better tool for the job.
I think most of the open source contributions will come in the Swift template, not in the C#. Indeed, that's the bulk of the PRs we've seen thus far.


[Bug]: qwen2.5 function calling,ChatLanguageModel is ok, but in StreamingChatLanguageModel,the logger report error

Your current environment

The output of `python collect_env.py`
Your output of `python collect_env.py` here

Model Input Dumps

🐛 Describe the bug
qwen2.5 function calling,ChatLanguageModel is ok, but in StreamingChatLanguageModel,the logger report error
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

the function parameter is wrong,I change the type，then it work well

INFO:     <IP_ADDRESS>:60868 - "POST /v1/chat/completions HTTP/1.1" 200 OK
INFO 10-09 16:52:47 engine.py:288] Added request chat-ed9f8e4371b245448607fb8dc02b1992.
ERROR 10-09 16:52:47 hermes_tool_parser.py:332] Error trying to handle streaming tool call: 'NoneType' object has no attribute 'get'
but the above error really confuse me

I'm also having the same issue #9175 #9187


Tracking issue for missing comments/documentation

[ ] Project architecture/design
[ ] Project struct description and usage
[ ] Venv struct description and usage
[ ] Documenting future plans (what structs/ops will be added or fleshed out)
[ ] Documentation on CONTRIBUTING expectations (every enhancement must have a test to cover it, etc.)
[x] #120
[x] #122

Hey, there I just wanted a quick update on this issue (:

Will revisit in refactors and chore PRs, naturally.


missing static file /images/unlink.png

/images/unlink.png is missing

Hey @Licenser, I don't have an image around for it. If there's one that you think would go well, I'm all ears (or eyes). :)

It is a bit cheating but I just made this:


Include App Store response in exception & purchase

Hi Ivan
To be able to keep track of what happens to the purchases I'm including the App Store response in the InAppValidationError and in Purchase.

This is sweet, really. But can you please check why travis build fails? Most likely it's because there's no "unittest.mock" package in Python 2 and we should use "mock" instead (in that case tox.ini should be updated).

Yes, mock was missing for Python 2.7. I was running tox -e 35 while developing :(

Well, better late than never, right? Sorry that it took me so long to finally approve the PR.
Great job, thanks for this. The new version will be available on pypi soon.


[SC] Fix SmartContractGasInjector failing gas injection with recursive methods

Fixes #1697
The test contract method is non-deterministic in terms of how many iterations it makes. Seems sufficient in terms of test coverage for this case, but let me know if you disagree.

First glance looks great @nathanielcook. Will have a more robust look tomorrow :)
Could you do one thing - retarget this to sc-dev branch instead of master? Cheers

Looks like I need to retarget in Git. Will close this one and reopen


Values in Sector drop down

Overview
This ticket specifies which values should be displayed in the sector drop down on the search / filter panel.
Solution

Only sector values that have been submitted by a contributor should be displayed in the drop down list.

Additional Context
Justin had previously shared these comments about a possible solution (also in #2187) --
I would recommend creating a separate issue for changing to the way the sector drop-down is populated.

Add a contributors parameter to GET /api/sector/ that mirrors the contributor
Update the logic of GET /api/sector
- If there is no contributors parameter restore the previous behavior of getting sector values from the facility index
- If there is a contributors parameter, get distinct sector values from FacilityLIstItemss submitted by the specified contributor
Update the client code that makes the sectors API request to pass the contributors parameter if the page is loaded in embed mode.

In Staging, there are some older sector values from before we started saving non-valid sectors as product types, which makes the dropdown look a little messy. This shouldn't be an issue in Production, where all of the data will have gone through the updated processing.


Prepare for handling multiple PURLs per ZIP archive

Please have a look at the individual commit messages for the details.

It failed because now the request was made with purl<EMAIL_ADDRESS>and since the path was empty, the computed purl now didn't have the # in the end. And I did realize earlier that the subpath shouldn't actually be saved in the database at all anyway, so I will have to parse it away. Or alternatively the request could actually be made with a list of objects that have a purl and a path.

I don't know if the backend scan should be successful for non-monorepo case yet at this point or not

I actually should. I've tried to implement what you suggested, and prepare for the the client / backend code to be multi-PURL ready, without breaking existing functionality.

Creating the scanner job was unsuccessful, even if it to my understanding should have been OK, with the call signature

Exactly, it should have worked. Can you be a more specific than "unsuccessful"?

13:05:39.215 [main] INFO  org.ossreviewtoolkit.plugins.scanners.dos.DOS - Initiating a backend scan
13:05:39.228 [main] INFO  org.ossreviewtoolkit.clients.dos.DOSService - Directory /tmp/ort-BlockingCoroutine4976008996093787221 deleted
13:05:39.412 [main] INFO  org.ossreviewtoolkit.clients.dos.DOSService - Presigned URL from API successfully fetched
13:05:39.413 [main] INFO  org.ossreviewtoolkit.clients.dos.DOSService - Uploading file /tmp/ort-BlockingCoroutine4976008996093787221.zip of size 59 kB to S3
13:05:39.747 [main] INFO  org.ossreviewtoolkit.clients.dos.DOSService - Packet successfully uploaded to S3
13:05:39.748 [main] INFO  org.ossreviewtoolkit.clients.dos.DOSService - File /tmp/ort-BlockingCoroutine4976008996093787221.zip deleted
13:05:39.869 [main] ERROR org.ossreviewtoolkit.clients.dos.DOSService - Response{protocol=http/1.1, code=500, message=Internal Server Error, url=https://app-28ea69c2-39bc-4f55-a7e8-5cab18fedc35.cleverapps.io/api/job}
13:05:39.871 [main] ERROR java.lang.Object - Could not create a new scan job at DOS API
13:05:39.871 [main] ERROR kotlinx.coroutines.CoroutineScope - Something went wrong at DOS backend, exiting scan of this package

It failed because now the request was made with purl<EMAIL_ADDRESS>and since the path was empty

Good point, I've fixed that now. I guess that should also fix @Etsija's test.

After your force-pushed commit, it now works (for this non-monorepo case), so OK!

Do you have something else to put in this PR, or can it be merged, and maybe write a further PR later, to pick up possible loose ends, fix bugs etc?  I'd like to get this code on main, as I have a PR of my own in mind, and I'd like to base it on newest code.

Do you have something else to put in this PR, or can it be merged

I'll merged it!


Set authentication_method on kafka listener

There’s currently no option to set authentication_method on the Kafka listener. I hard-coded this into the defaults.j2 for the time-being. Could this be made into a variable that can be toggled if SASL is enabled or is there another way to achieve the same?
      "kafka_api": [
        {
          "address": "{{ hostvars[inventory_hostname].private_ip }}",
          "port": "{{ redpanda_kafka_port }}",
          "authentication_method": "sasl"
        }
      ],

SASL specifically is part of a larger issue that needs to be returned to. Per host overrides are in though and I hope that that fixes this issue for now. Setting auth method specifically -- as opposed to overriding the listener -- I am not a big fan of as particular auth methods tend to need supporting stuff on other listeners/in redpanda that can't be captured by an individual entry.


Default to HTTPS MathJax

In general, I think these components should be run over HTTPS.
This also makes local development a bit nicer since Chrome resolves // as being file:// when viewing locally (without a webserver).

With share cell, I ended up needing to not serve over file:// during local development (because of zeroclipboard) so I'm now eating my own words.

Yeah, I'm going to leave this as it is for now. I know protocol relative urls are totally out of style, but it's a simple way to keep things working on both http and https.
And, yeah, I wouldn't recommend running this without a webserver anyway.


Migrate documentation to docusaurus

~~ https://docusaurus.io

Ima consider going with Docu v1 instead since v2 is very alpha 🤔

~


Bluetooth: Reorganize Kconfig options for BLE controller

Split meaning of BT_CTLR and BT_LL_SW, since they always are the same.
This way BT_CTLR means that there an controller implemented,
and BT_LL_SW refers to the specific implementation.
This allows alternative controller implementations.
Signed-off-by: Joakim Andersson<EMAIL_ADDRESS>

Codecov Report

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

@@            Coverage Diff            @@
##             master    #8564   +/-   ##
=========================================
  Coverage          ?   63.98%
=========================================
  Files             ?      429
  Lines             ?    41267
  Branches          ?     6953
=========================================
  Hits              ?    26403
  Misses            ?    11706
  Partials          ?     3158

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 2bcfb88...5be3689. Read the comment docs.

I thought that the email thread we had ended with me putting the controller code under controller/hci/vendor. This  is where I'm doing it now but we can move things around if needed

@jhedberg @dleach02 @cvinayak can you take a look?


Task Switcher Window Pops up with Mouse movement or scroll wheel

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
Mouse Utilities, Mouse Without Borders
Steps to reproduce
Task Switcher Window (ALT+TAB) pops up on mouse move and mouse scroll wheel motion. Usually scrolling down.
This is difficult to reproduce as I don't know how it started.
But once it does, it progressively gets worse and then just keeps popping the Task Switcher Window up all the time.
✔ Expected Behavior
Not to pop up the task switcher so I can work or play a game without interruption.
❌ Actual Behavior
I updated to the latest version through the auto update process and after a few minutes of playing a video game in full screen the Task Switcher Window (ALT+TAB) would pop up and throw me out if the game.
Eventually it would just pop up when moving the mouse or scrolling the wheel when the game was closed.
I had to uninstall Power Toys which seems to have fixed the issue as the window hasn't popped up yet. (About 2 hours after uninstall as I write this)
I really like the power toys tool set, yet the auto popping of the Task Switcher window is very intrusive and can damage my game play and my work.
I suspect it is Mouse Without Boarders causing the issue as it does deal with the mouse, and I had all the mouse features turned off, yet I would suggest checking the mouse features as well.
Other Software
Just how it interacts with the keyboard and windows.

/bugreport

Unfortunatly i am unable to as i had to uninstall the tool as it was preventing me from working.
-------- Original message --------From: "microsoft-github-policy-service[bot]" @.> Date: 10/27/23  10:59 AM  (GMT-05:00) To: microsoft/PowerToys @.> Cc: Helixdakat @.>, Author @.> Subject: Re: [microsoft/PowerToys] Task Switcher Window Pops up with Mouse
movement or scroll wheel (Issue #29434)
Hi there!We need a bit more information to really debug this issue. Can you add a "Report Bug" zip file here?    Then drag the zipfile from your desktop onto the GitHub comment box in this issue.  Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@Helixdakat I don't know if there is a good way for the team to try to debug without the bug report information. Can you try updating to v0.75.1 and see if that fixes the issue? /needinfo


Update genjavadoc-plugin to 0.17

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

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

labels: library-update

+1

[X] Do all automated continuous integration checks pass?
Yes
[X] Is the update a patch, minor, or major update?
Minor update, diff shows few changes that do not affect us. I also ran sbt unidoc with the previous and new version and both docs were exactly the same
[X] Is the license still compatible with ASF License Policy?
Yes, still Apache v2. Note that we do not distribute this plugin with daffodil
[X] Have any changes been made to LICENSE/NOTICE files that need to be incorporated?
No changes to LICENSE. No NOTICE file
[X] Have any transitive dependencies been added or changed?
No changes to transitive dependencies


Add implot_internal.h generation capability

My use case needed the use of BeginItem and other internals to create custom plots, which were only available in imgui_internal.h. This is largely a port of how cimgui does the same. Following the cimgui convention, I made internal header inclusion a flag passed to the generator and also the default option in scripts.
This PR includes the regenerated sources, which compile and are usable for me, and also confirmed do not modify any of the existing entries in any way so should be 100% backwards compatible.

Thanks for the PR
Althought I am dubious about generate_templates and getCname so that I would prefer to implement it myself.
Now working on that (perhaps I am needing sometime because of personal needs)

Ok, fair enough. Fwiw to understand current implementation decisions as this is really mostly a copy from cimgui:

generate_templates was copied from cimgui. ImChunkStream and ImSpan can be dropped as they are not used. ImVector and ImPool have minor modifications to ensure they don't overlap with types already defined in cimgui.

cname had ImPlot_ added for fields without namespaces only, as this use case was required for internals. It doesn't affect any of the existing names.

All output was manually checked to ensure 100% backwards compatibility.

Nevertheless I understand and hopefully this implementation helps you.

My first iteration is at https://github.com/cimgui/cimplot/tree/internal2 and the first problem appearing is time_t not being defined

Good catch. That’s a standard type no? Just need to pull in time.h
On Fri, 12 Mar 2021 at 19:59, Victor Bombi @.***> wrote:

My first iteration is at https://github.com/cimgui/cimplot/tree/internal2
and the first problem appearing is time_t not being defined
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cimgui/cimplot/pull/4#issuecomment-797342232, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVTZ7TU6ZKSDXFNOFCEJTTDHJV5ANCNFSM4ZBB4M2Q
.

--
Sent from Gmail Mobile

Good catch. That’s a standard type no? Just need to pull in time.h

This solves C compilation but is problematic in LuaJIT-ImGui parsing of the header because it does not use additional includes
If I add generate_templates the error is luajit.exe: .\class_gen.lua:262: attempt to redefine 'ImVector_ImGuiColorMod'
because it is already defined in cimgui.h

You can see in my implementation that I had a list of excludes for ImPool,
you could do same for ImVector? Did you include time.h in template?
Also be good to know how you’re validating because it all worked for me
when I compiled and keen to try and help.
On Fri, 12 Mar 2021 at 20:28, Victor Bombi @.***> wrote:

Good catch. That’s a standard type no? Just need to pull in time.h
This solves C compilation but is problematic in LuaJIT-ImGui parsing of
the header because it does not use additional includes
If I add generate_templates the error is luajit.exe: .\class_gen.lua:262:
attempt to redefine 'ImVector_ImGuiColorMod'
because it is already defined in cimgui.h
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cimgui/cimplot/pull/4#issuecomment-797360895, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVTZZTWLDEYWVTKR5SJI3TDHNDZANCNFSM4ZBB4M2Q
.

--
Sent from Gmail Mobile

Also be good to know how you’re validating because it all worked for me when I compiled and keen to try and help.

My validation is done by C compilation but also by cdef generation in LuaJIT-ImGui where LuaJIT behaves as a C compiler but I cant include time.h and I must provide the time_t typedef myself (but this typedef has variations depending on compiler and architecture)

Once I provided time_t and struct tm definition in LuaJIT-ImGui I could generate and compile.
Is branch internal2 ok for you?

Once I provided time_t and struct tm definition in LuaJIT-ImGui I could generate and compile.
Is branch internal2 ok for you? (if it is I will cherry pick commits to master)

I will check tomorrow

Seems to be working, I'm supportive of either merging this into main or keeping internal as a separate (but official) branch.

Thanks. Done.


Create script init.js to poplulate mongodb database

as a software developer
whene i launch e-commerce application in docker
I'd like that the mongodb database be populated with data.


Update configure-remediation-microsoft-defender-antivirus.md

Fixed Default setting (if not configured)] below, the default retention period for the device seems to be [90 days].
Fixes https://github.com/MicrosoftDocs/microsoft-365-docs/issues/12794

@JohanFreelancer9 Please copy review


Allow pkg-config checks to be skipped in CI environments that don't use or care about linking

I have a project with a dependency which uses pkg-config. The dependency's build script matches your suggestion in the readme:
fn main() {
    pkg_config::probe_library("somelibrary").unwrap();
}

I am running tests with Travis CI using their docker infrastructure. I have ensured the dependency and executable pkg-config are installed in the container.
However, when compiling my project the CI builds fail because pkg-config can't find the library this dependency requests. The pkg-config crate-level docs suggest setting $FOO_NO_PKG_CONFIG but this will just fail the builds "faster" as the unwrap() will still panic.
I could file an issue with this dependency, or look into installing pkg-config on Travis, or file an issue about supporting Travis CI specifically. However, given the existence of the *_NO_PKG_CONFIG I figured you wouldn't be opposed to *_SKIP_PKG_CONFIG, which would be more fine-grained - I am using the workaround export PKG_CONFIG=echo (which is also not in the documentation) to prevent pkg-config from running at all in the CI environment.
Let me know what you think and I can open a PR.

Thanks for the report! I'm a little confused though as *_NO_PKG_CONFIG should skip everything related to pkg-config and return quickly, but is that not the case?

The issue is that it causes pobe_libary to return an Err, which panics when it hits unwrap and will prevent Cargo from building the dependency entirely.

The build script in question clearly isn't prepared to manage without pkg-config (I believe _NO_PKG_CONFIG is meant to cause the script to build the library from source). Skipping it would just leave the linker unable to find the library.

However, when compiling my project the CI builds fail because pkg-config can't find the library this dependency requests.

Perhaps this specific issue can be solved instead? Could you link to the relevant Travis log?

@SnirkImmington ah but the Err could be ignored, right? That is, if you drop it, then nothing panics?

@SnirkImmington ah but the Err could be ignored, right? That is, if you drop it, then nothing panics?

I don't think I'm being clear here. This is not my build script. This is not my code. A dependency that someone else wrote is causing my builds to fail, and I have to hack on my CI build environment so my project can start compiling. This dependency's build script is "hard coded" to unwrap this error.
The underlying concern I'm feeling here is lack of control over the build process. This is, of course, something that happens when dependencies can have build scripts at all, but I didn't want to rewrite this dependency's build script. I spent a few commits trying to fix some environment variable that the error message hinted might be the cause.

Perhaps this specific issue can be solved instead? Could you link to the relevant Travis log?

I think it would be a waste of both of our time to look at logs and suggest different values for $PKG_CONFIG_PATH. If you don't think these changes are a good idea, I can close this issue and continue using my workaround.

Skipping it would just leave the linker unable to find the library.

This isn't always the priority in build environments. Even if our dependencies weren't able to find a dynamic linked library when we built them, we'd get the linker error immediately after when we run tests. This could be used to make sure that the Rust code of a library with specific C bindings can compile, without installing C libraries, setting up pkg-config, or running any tests that would immediately cause a linker fail.
I'm also pretty sure that running cargo doc in CI scripts to update docs shouldn't care about C bindings. What about creating binary releases? It shouldn't matter if the dynamic libs aren't in the build environment at that time that we build them. I just think this is a lot of forcing of others' code and requirements into the developer's environment, which the developer could have reasons to ignore.
I'm sure if you looked around at other Rust projects that have this issue, they've taken similar solutions - modifying the build scripts of their dependencies programmatically in CI, or removing the build.rs files altogether. We don't want to open PRs against all our dependencies or maintain forks just so our code can build.

Cargo was designed to be pretty flexible around linking to native libraries, so you can use some of its escape hatches as well. For example if a links key is specified then you can override the build script entirely and just supply a pre-built copy.
I'd be pretty hesitant to add a configuration which forces pkg-config to "lie" that the dependency has been found when it actually hasn't, unfortunately. In general build scripts need to be robust if they intend to be used in lots of situations like this, so it may also be quite useful to upstream the build script changes you made perhaps?

I'd be pretty hesitant to add a configuration which forces pkg-config to "lie" that the dependency has been found when it actually hasn't, unfortunately

Okay, that's a fair call. Cargo provides some nice hooks for linking, and can keep using the value of $PKG_CONFIG if necessary.


Broken profile pictures

Describe the bug
Profile pictures, and the user popup, are broken currently
To Reproduce
Steps to reproduce the behavior:

Go to any user
Click on the expandable popup

Expected behavior
Old popup style appears
Screenshots
https://ibb.co/1r62bRr - old style
https://ibb.co/CBq1LxK - new and broken

Discord Updated The Modal And With It We Lost Features To Be Able To Do The Old Style Again So I Just Made A Better Looking Version For Quarrel And Ported To Lode

Also, the about me is cut off


[OptRed] Define -triton-intelgpu-optimize-reduction pass

Add pass splitting reductions and performing layout conversions to avoid using sub-group operations.

Part of #2266.

Merging it in, if other reviewers have comments add them as post review comments.


Feat/vite webpack version

Hi @nucle,
It's not working with Vite @rstoenescu Has already made changes but the problem is it's not published. Is there any chance we can publish it with latest code ?
Thanks,
Pratik Patel


v2.0.0 - beta preview

Hi all,
i have published v2.0.0 beta to test, is available trough Config UI X.
I will be glad if you want to test it and share the errors and observations you have noticed.
All changes all available 2.0.0 - beta branch
Debug mode can be enabled in plugin Advanced Settings.

Hello,
I will try it :)

So I Installed the plugin, Xbox is recognized and communicating, but I have some double entry (see screenshots).
https://www.dropbox.com/s/9s5pmgvp2feh9yf/IMG_0174.PNG?dl=0
Here is my config :
            {
                "name": "Xbox Series X",
                "host": "<IP_ADDRESS>",
                "xboxliveid": "XXXXXXXX",
                "xboxWebApiToken": "XXXXXXXXXXXXXXXX",
                "refreshInterval": 5,
                "disableLogInfo": false,
                "webApiControl": true,
                "switchInfoMenu": false,
                "getInputsFromDevice": true,
                "filterGames": false,
                "filterApps": false,
                "filterSystemApps": false,
                "filterDlc": false,
                "enableDebugMode": false
            }

So entry works, other none.
My Xbox should be authorized, but i can't see ma games or the reboot possiblity.
Here is the logs :
https://www.dropbox.com/s/cfbkt924ra1veth/homebridge.log.txt?dl=0

Also here :
/var/lib/homebridge/xboxTv/inputs_xxxxxx
inputs file is empty, authtoken also.
Here is the full log with debug :
https://www.dropbox.com/s/cfbkt924ra1veth/homebridge.log.txt?dl=0

Duplicate inputs fixed,
Your console is not authorized, loads apps, games from device is not possible.
Try authorize it again.

It did it 3 times already, on 3 browsers.

Also, when I want to use the button to start activation et authorise device, i got this error :
Activation and token file save error.
Logs shows only this :
[homebridge-xbox-tv] Incoming Request: /data
[29/11/2021, 14:26:18] [Homebridge UI] [homebridge-xbox-tv] Incomming token M.R3_BL2.ec957217-6a72-2fb1-6a16-9545e6034c3a:, host: <IP_ADDRESS>, clientId: 5e5ead27-ed60-482d-b3fc-702b28a97404.
[29/11/2021, 14:26:40] [Homebridge UI] [homebridge-xbox-tv] Terminating child process...
[29/11/2021, 14:26:40] [Homebridge UI] [homebridge-xbox-tv] Child process ended

The token is not saved, check write permission to /var/lib/homebridge/xboxTv/

I have this :
pi@homebridge:~/.homebridge/xboxTv $ ls -ld
drwxr-xr-x 2 pi pi 4096 Nov 29 12:30 .

Are You using UI to authenticate or manual from Homebridge log?

UI doesn't work, I have the error like I said.
So i did it with the link in the logs

Also, I have MFA on my Xbox account, is it possible to be a problem ?

try 404 and get the log, I checked on my side authorized from scratch and working correct

I launched an old instance of Homebridge on my Docker Synology.

Updated everything, deleted the files of xboxTv folder, and it's working when I added the console via the logs link. The inputname file has the correct games and apps, and the authToken file is filled.

The button on the settings pages are still creating the same error as before when I tried this way.

I will try now on my Raspberry, which is where it's not updating the different files.

try manual setup from hb log

I will, I let you know

UI Authorization manager fixed, try it

Ok many thanks, I'll check

With the last update, same status, the files are not updated and the U settings buttons display the same error. I'll try on my Syno.

So, it's working now, but I had to do a clear token, restart Homebridge instance, and then after it worked on Synology and Raspberry.
[XboxTv] Device: <IP_ADDRESS> Xbox Series X, authenticated and web api enabled.
I will check if everything works well on the plugin in Home, thanks for the quick actions.

Can You record video? It’s just not possible, I have checked this many times and always with success.
Note, every start authorization process need new Web Api Token

Can You record video? It’s just not possible, I have checked this many times and always with success.
Note, every start authorization process need new Web Api Token

As I said, it's now working but I needed to a clear token before everything else works right on both system.
Now it's all setup ok and logs says API connected and authenticated.

Yes, Token need to be clear before start new authorization.
If You found some bugs let me know.

Where is the reboot switch / input please ?

Reboot function is in buttons section, select command “Reboot” will be available as extra button

I have got the Xbox authenticated but when I try to turn on using HomeKit I get error:
[30/11/2021, 17:04:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Send power On. [30/11/2021, 17:04:03] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, set Power successful, ON [30/11/2021, 17:04:16] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Disconnected.
How can I fix?

Here is no error

well it disconnects, and the Xbox doesn't turn on

better log:
[30/11/2021, 17:24:58] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Send power On. [30/11/2021, 17:24:58] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 0 seconds ago. [30/11/2021, 17:24:58] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:24:59] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 1 seconds ago. [30/11/2021, 17:24:59] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:24:59] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 1 seconds ago. [30/11/2021, 17:24:59] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 2 seconds ago. [30/11/2021, 17:25:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 2 seconds ago. [30/11/2021, 17:25:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:01] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 3 seconds ago. [30/11/2021, 17:25:01] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:01] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, set Power successful, ON [30/11/2021, 17:25:02] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 3 seconds ago. [30/11/2021, 17:25:02] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:02] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 4 seconds ago. [30/11/2021, 17:25:02] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:03] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 4 seconds ago. [30/11/2021, 17:25:03] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:03] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 5 seconds ago. [30/11/2021, 17:25:03] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:04] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 5 seconds ago. [30/11/2021, 17:25:04] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:04] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 6 seconds ago. [30/11/2021, 17:25:04] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 6 seconds ago. [30/11/2021, 17:25:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 7 seconds ago. [30/11/2021, 17:25:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:06] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 7 seconds ago. [30/11/2021, 17:25:06] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:06] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 8 seconds ago. [30/11/2021, 17:25:06] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:07] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 8 seconds ago. [30/11/2021, 17:25:07] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:07] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 9 seconds ago. [30/11/2021, 17:25:07] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:08] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 9 seconds ago. [30/11/2021, 17:25:08] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:08] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 10 seconds ago. [30/11/2021, 17:25:08] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:09] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 10 seconds ago. [30/11/2021, 17:25:09] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:09] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 11 seconds ago. [30/11/2021, 17:25:09] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 11 seconds ago. [30/11/2021, 17:25:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 12 seconds ago. [30/11/2021, 17:25:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:11] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 12 seconds ago. [30/11/2021, 17:25:11] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:11] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 13 seconds ago. [30/11/2021, 17:25:11] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:12] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 13 seconds ago. [30/11/2021, 17:25:12] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:12] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 14 seconds ago. [30/11/2021, 17:25:12] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:13] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 14 seconds ago. [30/11/2021, 17:25:13] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:13] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 15 seconds ago. [30/11/2021, 17:25:13] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:14] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 15 seconds ago. [30/11/2021, 17:25:14] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:14] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Last boot time was 16 seconds ago. [30/11/2021, 17:25:14] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Disconnected. [30/11/2021, 17:25:14] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 25 bytes. [30/11/2021, 17:25:15] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:20] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:25] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:27] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, get Power state successful: OFF [30/11/2021, 17:25:30] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:35] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:40] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:45] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:50] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:25:55] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:26:00] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:26:05] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:26:10] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes. [30/11/2021, 17:26:15] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 16 bytes.

Your Xbox is not powered ON, is not available on the network.

It is on, and connected. is it bc I have ethernet?

I now can't get a new API key, it doesn't show up in logs. how do I fix?

Hi there, I'm on 2.0.0-beta but the remote control feature doesn't work:
I enabled the DEBUG mode but I wasn't able to see any error log.
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, System input send press, command: a
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Emit event type: _on_channelResponse
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 74 bytes.
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 99 bytes.
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Request number set to: 41
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, System input send unpress command: 0
[12/2/2021, 7:50:28 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 99 bytes.
[12/2/2021, 7:50:29 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 0 seconds ago.
[12/2/2021, 7:50:30 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 1 seconds ago.
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 2 seconds ago.
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Server reveived message from: <IP_ADDRESS>:5050
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Packet needs to be acknowledged, send acknowledge.
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Request number set to: 42
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Emit event type: _on_acknowledge
[12/2/2021, 7:50:31 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Socket send 74 bytes.
[12/2/2021, 7:50:32 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 0 seconds ago.
[12/2/2021, 7:50:33 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 1 seconds ago.
[12/2/2021, 7:50:34 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Start check message timeout, last received was 2 seconds ago.
[12/2/2021, 7:50:34 PM] [XboxTv] Device: <IP_ADDRESS> Xbox Series S, Server reveived message from: <IP_ADDRESS>:5050

Hi Grzegorz914,
I updated to the last beta, I have this error :
[XboxTv] Device: <IP_ADDRESS> Xbox, with liveId: , get Storage Devices error: {
status: 400,
body: '{\r\n' +
'  "status": {\r\n' +
'    "errorCode": "MissingDeviceId",\r\n' +
'    "errorMessage": "Missing DeviceId query parameter"\r\n' +
'  },\r\n' +
'  "deviceId": null,\r\n' +
'  "result": [],\r\n' +
'  "agentUserId": null\r\n' +
'}'
}.
Any idea ?
Thanks

Set Your liveId in config

Weird, it was set in config.json, but not in the config ui plugin page...

Currently using the beta and everything works as expected so far, including power commands and launching of games and apps, but for some reason it will power my Xbox back up about 30 seconds after powering off.  Oddly, this behaviour doesn’t occur when powering the Xbox on, only off.

Strange, it working correct on my both consoles.

@Hazza42 Please could you send the Homebridge logs after you shut it off from Homebridge?

Just tried reproducing the issue and it’s now behaving correctly wouldn’t you know it. If I catch it switching my Xbox on again I’ll grab the logs and post them here.

On 19 Dec 2021, at 7:35 am, tecdude @.***> wrote:

@Hazza42 Please could you send the Homebridge logs after you shut it off from Homebridge?
—
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 were mentioned.