hctl: fetch-fids command returns un-parseble error string

If fetch-fids command does not find any of the motr services configured
on a node, it returns an error string, If hctl fetch-fids is invoked
though a program, the error string in such a case is not parseble. And
user may not understand if its a valid result or not.
Solution:
Return whatever is configured for a given node, return empty if
nothing is configured.
Signed-off-by: Mandar Sawant<EMAIL_ADDRESS>

Hi @mssawant,
We have two setup.log files. One inside hare_deployment dir which gets used for miniprovisioner logs and other is outside of hare_deplyment dir.
This newly added log will be populated in setup.log file which is outside of hare_deployment directory?
Also can you please post the sample output here?
Thanks!

@suvratjoshi, I don't think any of the hctl commands logs are directed to any of the setup files yet. This has to be done as a separate task. Presently following is the output of the hctl fetch-fids command,
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/
ERROR:root:No ioservice, confd, or s3servers found in the cluster
ERROR:root:No ioservice, confd, or s3servers found in the cluster
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]

This is with the below configuration in solution.yaml for 1N deployment.
    s3:
      num_inst: 0
      start_port_num: 28051
      max_start_timeout: 240
    motr:
      num_client_inst: 1
      start_port_num: 29000

[root@ssc-vm-g3-rhev4-2107 ~]# kubectl get pods
NAME                                                 READY   STATUS             RESTARTS   AGE
consul-client-wl5fk                                  1/1     Running            0          4h21m
consul-server-0                                      1/1     Running            0          4h21m
cortx-client-ssc-vm-g3-rhev4-2107-5486c55687-r649d   0/2     CrashLoopBackOff   108        4h16m
cortx-control-76965c6db5-x2lwz                       4/4     Running            0          4h20m
cortx-data-ssc-vm-g3-rhev4-2107-547dc56c6-d5pdw      4/4     Running            0          4h19m
cortx-ha-85f999658d-9kjvp                            3/3     Running            0          4h17m
cortx-server-ssc-vm-g3-rhev4-2107-597c9446c9-g29gs   4/4     Running            0          4h18m
kafka-0                                              1/1     Running            0          4h21m
openldap-0                                           1/1     Running            0          4h21m
zookeeper-0                                          1/1     Running            0          4h21m
[root@ssc-vm-g3-rhev4-2107 ~]#

motr client pod is failing for a different purpose,
[root@ssc-vm-g3-rhev4-2107 k8_cortx_cloud]# kubectl logs cortx-client-ssc-vm-g3-rhev4-2107-5486c55687-r649d cortx-motr-client-001

error[2]: machine id not available in conf

Traceback (most recent call last):
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 349, in main
    command = Cmd.get_command(desc, argv[1:])
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 118, in get_command
    return args.command(args)
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 310, in __init__
    super().__init__(args)
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 48, in __init__
    configure_machine_id(self, None)
  File "/opt/seagate/cortx/motr/bin/motr_mini_prov.py", line 201, in configure_machine_id
    raise MotrError(errno.ENOENT, "machine id not available in conf")
motr_mini_prov.MotrError: error[2]: machine id not available in conf

usage: /opt/seagate/cortx/motr/bin/motr_setup [-h] <cmd> --config <url> <args>
where:
cmd           post_install, prepare, config, init, test, reset, cleanup, upgarde, backup, restore
url           Config URL
[root@ssc-vm-g3-rhev4-2107 k8_cortx_cloud]#

So, data pod provisioned successfully with the 0 s3 instances which was failing earlier.

@suvratjoshi, I don't think any of the hctl commands logs are directed to any of the setup files yet. This has to be done as a separate task. Presently following is the output of the hctl fetch-fids command,
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/
ERROR:root:No ioservice, confd, or s3servers found in the cluster
ERROR:root:No ioservice, confd, or s3servers found in the cluster
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]
[root@cortx-data-headless-svc-ssc-vm-g3-rhev4-2107 /]# hctl fetch-fids --conf-dir /etc/cortx/hare/config/9b94235305444e7cbd0048a42315f880/ 2>/dev/null
[
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x1"
  },
  {
    "name": "ioservice",
    "fid": "0x7200000000000001:0x2"
  },
  {
    "name": "confd",
    "fid": "0x7200000000000001:0x3"
  }
]

This is with the below configuration in solution.yaml for 1N deployment.
    s3:
      num_inst: 0
      start_port_num: 28051
      max_start_timeout: 240
    motr:
      num_client_inst: 1
      start_port_num: 29000

[root@ssc-vm-g3-rhev4-2107 ~]# kubectl get pods
NAME                                                 READY   STATUS             RESTARTS   AGE
consul-client-wl5fk                                  1/1     Running            0          4h21m
consul-server-0                                      1/1     Running            0          4h21m
cortx-client-ssc-vm-g3-rhev4-2107-5486c55687-r649d   0/2     CrashLoopBackOff   108        4h16m
cortx-control-76965c6db5-x2lwz                       4/4     Running            0          4h20m
cortx-data-ssc-vm-g3-rhev4-2107-547dc56c6-d5pdw      4/4     Running            0          4h19m
cortx-ha-85f999658d-9kjvp                            3/3     Running            0          4h17m
cortx-server-ssc-vm-g3-rhev4-2107-597c9446c9-g29gs   4/4     Running            0          4h18m
kafka-0                                              1/1     Running            0          4h21m
openldap-0                                           1/1     Running            0          4h21m
zookeeper-0                                          1/1     Running            0          4h21m
[root@ssc-vm-g3-rhev4-2107 ~]#

motr client pod is failing for a different purpose,
[root@ssc-vm-g3-rhev4-2107 k8_cortx_cloud]# kubectl logs cortx-client-ssc-vm-g3-rhev4-2107-5486c55687-r649d cortx-motr-client-001

error[2]: machine id not available in conf

Traceback (most recent call last):
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 349, in main
    command = Cmd.get_command(desc, argv[1:])
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 118, in get_command
    return args.command(args)
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 310, in __init__
    super().__init__(args)
  File "/opt/seagate/cortx/motr/bin/motr_setup", line 48, in __init__
    configure_machine_id(self, None)
  File "/opt/seagate/cortx/motr/bin/motr_mini_prov.py", line 201, in configure_machine_id
    raise MotrError(errno.ENOENT, "machine id not available in conf")
motr_mini_prov.MotrError: error[2]: machine id not available in conf

usage: /opt/seagate/cortx/motr/bin/motr_setup [-h] <cmd> --config <url> <args>
where:
cmd           post_install, prepare, config, init, test, reset, cleanup, upgarde, backup, restore
url           Config URL
[root@ssc-vm-g3-rhev4-2107 k8_cortx_cloud]#

So, data pod provisioned successfully with the 0 s3 instances which was failing earlier.

I think this issue is fixed in latest cortx-k8s release.


[pt-br] Add docs/tasks/access-application-cluster/access-cluster-services.md

Adding content/pt-br/docs/tasks/access-application-cluster/access-cluster-services.md

Hello there, @ricmmartins! 👋
Thank you very much for your contribution.

/cc @stormqueen1990

/label tide/merge-method-squash

/retest

/retest

Hi @ricmmartins!
Thank you very much for your contribution.

/lgtm


64-bit Support & iOS 9

We've run into an issue submitting an iOS 9 App (see photo below). Removing the Pushwoosh Cordova plugin solves the problem. Using the 3.6.5 version (linked directly from Github).
A wild guess is that the issue is with your binary code not supporting arm64.

itms-90086 Missing 64-bit support

I'm not sure about this anymore.
The Cordova build-process is fickle and I don't trust it. It's possible that something else cause this issue. Feel free to close if you think Pushwoosh Cordova plugin is already 64-bit compatible.

It looks like you have to specify arm64 architecture in Cordova XCode project. Pushwoosh supports 64bit architecture since ages. Meanwhile Cordova Plugin with iOS9 (bitcode) for XCode 7 update should be live today.


send-btc: ReferenceError: fetch is not defined

npx hardhat send-btc --amount 0.001 --memo 629eEe97B95Bd6e04B0885De58eF016177a709Ae2cD3D070aE1BD365909dD859d29F387AA96911e1 --recipient tb1qy9pqmk2pd9sv63g27jt8r657wy0d9ueeh0nqur
An unexpected error occurred:
ReferenceError: fetch is not defined

Works fine when the command is called in the toolkit repo, but I can reproduce the issue when using the tutorial.

The issue I can reproduce is:
An unexpected error occurred:

TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined
    at new NodeError (node:internal/errors:399:5)
    at Function.from (node:buffer:335:9)
    at SimpleTaskDefinition.main [as action] (/Users/fadeev/template/node_modules/@zetachain/toolkit/dist/tasks/sendBTC.js:114:46)
    at Environment._runTaskDefinition (/Users/fadeev/template/node_modules/hardhat/src/internal/core/runtime-environment.ts:333:35)
    at Environment.run (/Users/fadeev/template/node_modules/hardhat/src/internal/core/runtime-environment.ts:166:25)
    at main (/Users/fadeev/template/node_modules/hardhat/src/internal/cli/cli.ts:280:17) {
  code: 'ERR_INVALID_ARG_TYPE'
}

Happens when you don't have a private key in .env (or an env variable).

fetch is not defined can be resolved by upgrading Node.js to version 18.


Support for Deleting a Durable Subscription

In the latest 2.3.4 it is not possible to delete a durable subscription.
In the documentation for RabbitMQ or ActiveMQ it is said:
"To permanently delete a durable subscription, send an UNSUBSCRIBE frame for the subscription ID with the persistent header set to true."
The easiest way to this, would be to provide the subscription headers to the client.unsubscribe():
Client.prototype.subscribe = function(destination, callback, headers) {

...some code....

  return {
    id: headers.id,
    unsubscribe: function() {
      return client.unsubscribe(headers);
    }
  };
};

The client.unsubscribe() should be changed to this:
Client.prototype.unsubscribe = function(headers) {
  delete this.subscriptions[headers.id];
  return this._transmit("UNSUBSCRIBE", headers);
};

My JavaScript is extremely rusty, but should this not be:
  return {
    id: headers.id,
    unsubscribe: function(headers) {
      return client.unsubscribe(id, headers);
    }
  };

with client.unsubscribe() changed to:
Client.prototype.unsubscribe = function(id, headers) {
  delete this.subscriptions[id];
  if (headers == null) {
    headers = {};
  }
  headers.id = id;
  return this._transmit("UNSUBSCRIBE", headers);
};

so that subscription.unsubscribe() can be called with an optional 'headers' argument? If I'm wrong, corrections are very welcome!

Actually, I found an error myself. Let's try this again. I've tried this and it works, which doesn't mean that the OP's code does not also work.
return {
    id: headers.id,
    unsubscribe: function(hdrs) {
      return client.unsubscribe(headers.id, hdrs);
    }
  };


Prevent Overwriting Recordings from Different Hosts with Same Name

Feature request
Recordings received from different hosts in https://github.com/OpenAdaptAI/OpenAdapt/pull/223 with the same name don't conflict with each other. We don't want to overwrite existing recordings. To achieve this, we can use a hash to make the filenames unique. An ideal approach would be hashing the host name and current timestamp together, ensuring uniqueness and traceability.
Motivation

The current system faces conflicts when receiving recordings from different hosts with the same name, leading to data loss and confusion.
We want to prevent overwriting existing recordings and ensure each recording is uniquely identifiable.
By hashing the host name and timestamp, we can create a robust and traceable system for managing recordings.

Implementation can look like:

Create a hash function to combine the host name and current time to generate a unique filename.
Ensure that the new system handles conflicts gracefully without overwriting existing data.
Utilize the onetimesecret.com API to generate a one-time secret link for sharing the unique filename securely.


(GH-36) changed repositoryOwner in cake.recipe

fixes #36

@nils-a your changes have been merged, thanks for your contribution 👍

@nils-a your changes have been merged, thanks for your contribution 👍


Headings disappear if using ng-if on ancestor element

The following causes the headings to become invisible:

Name

A

A

Removing the ng-if restores them.

As a short term workaround you can use ng-show instead.

Yep, I did confirm that works at least. For some reason though, the column headings were not aligning for me, so I since gave up on using the directive. I suspect it may have been to do with me using ng-repeat on the  elements (I have a variable set of columns, according to a selected report).

I don't think so, it's because the elements are all hidden when the math is done to determine their widths. As a second workaround you can do
                $timeout(function() {
                    // force table header to recalculate its width.
                    window.dispatchEvent(new Event('resize'));
                });

Ah good point, you're more than likely right there!

in the ng-repeat directive, the same.


fix: resolve circle depents in docs

📝 Description

Add a brief description

yarn dev:docs failed due to circle depents.
⛳ Current behavior (updates)

Please describe the current behavior that you are modifying

🚀 New behavior

Please describe the behavior or changes this PR adds

error above disappeared.
💣 Is this a breaking change (Yes/No):

📝 Additional Information

@tianenpang this issue happen to you? I couldn't replicate it 🤔

I can't reproduce it either 🤔 hey @sun0day please remove node_modules and cache then try again.
# Remove all node_modules and cache include sub-dirs
find . '(' -name "node_modules" -o -name ".next" -o -name ".turbo" ')' -type d -prune -exec rm -rf '{}' +

# Install deps and build nextui
yarn && yarn build:nextui

# Restart docs dev server
yarn dev:docs

This error disappeared after cleaning cache and reinstalling dependencies.  But still, I think using circle depents is bad practice. We should avoid circle depents in case of some weird exceptions.

I agree with you, but I think this error may be caused by other issues 🤔 and for the circular dependency, it's possible to use the webpack plugin to detect it automatically.

I am pretty sure this origin reason is circular depents due to the exception description and my debug process. But the circular depents exception not always occurs, it depends the node_modules' physical topology,  yarn can't guarantee node_modules' topology always be the same every time you install or add new dependencies, especially when you host has node_modules cache.
And maybe we can  use eslint rule import/no-cycle to discover the circular depends problems in dev stage.

Hey @sun0day I'll close this PR until we can reproduce it in a better way


Relabel Teenage Mutant Ninja Cows?

         (__)               (__)  |    |  (__)
         (--)               (--)  |    |  (--)
  /-------\/   /o    /-------\/   |    I   \/-------\
 / |  L  |----< o   / |  D  |----<T    I>----|  M  | \
*  ||----|   /  o  *  ||----|     I    I     |----||  *
   ^^    ^      |     ^^    ^          |     ^    ^^
                |                      |
              Teenage Mutant Ninja Cows

Current cow labeling order: L, D, M.
Leftmost appears to have nunchaku, middle a sword, and right a bo staff. Mapping them to their turtle counterparts, they'd they'd be Michaelangelo Moochaelangelo, Leonardo Longhornardo, and Donatello Dairytello.
Proposed cow labeling order: M, L, D.

Amazing.

Ace spot +1

:heart_eyes:

:cow:

:+1: +1 I need this.

I faced with same issue! Thank you for the fix!

:sparkles:

:+1: yes, I also repro'd this issue on Mac OSX. Fixing this would definitely help my team a lot. Thanks!


cover .some() function in utils.js with tests

Coverage increased (+0.04%) to 88.188% when pulling ed61cd0d2bd9313ef5288ca10e8f4e5e294cde1a on seppevs:cover_utils_some_fn_with_tests into f42cbf4c9208180d5267488b42536b5feb8080de on mochajs:master.

Thank you!


visualize_chemspace is missing

Hi,
I have been trying to plot scaffold split using visualize_chemspace (as mentioned in the tutorial). visualize_chemspace code is not there in utili.py. Is it changed?

Hi @suneelbvs ! It is not part of the Splito library, but rather a utility function that is defined specifically for the tutorial.
See: https://github.com/datamol-io/splito/blob/0d9c2df489b58a73e2a2b93a2c180ce7a8f5fb36/docs/tutorials/utils.py#L14-L23
I don't think this function should be part of the official Splito code base since it is not directly relevant to splitting, but if you feel otherwise I would be happy to discuss this!


Pretty flavor describe command for the CLI

Describe changes
I have implemented a CLI command to describe different flavors based on their configuration.
Example:
zenml model-deployer flavor describe mlflow

Configuration class: MLFlowModelDeployerConfig

Configuration for the MLflow model deployer.

Attributes:
    service_path: the path where the local MLflow deployment service
        configuration, PID and log files are stored.

Properties
service_path, string

Pre-requisites
Please ensure you have done the following:

[X] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Yes and no. It will show below which fields are required but it wont show any further validation criteria coming from pydantic.


[FancyZones] Ctrl-Alt-Windows-Arrow does not work in fullscreen RDP-session

ℹ Computer information

PowerToys version: 0.27
PowerToy Utility: FancyZones
Running PowerToys as Admin: No
Windows build number: 19042.662 on all computers involved

📝 Provide detailed reproduction steps (if any)

Connect to remote computer using RDP
Make RDP session fullscreen
Move application window to a zone using Windows-Arrow
Try to expand window using Ctrl-Alt-Windows-Arrow

✔ Expected result
The window should change size at step 4 above.
❌ Actual result
Nothing happens.

does the RDP computer have FZ installed on it?

@pgerell  which computers have PowerToys installed?  Right now it is unclear

@crutkas The remote computer has FZ installed.

@pgerell, is the issue still reproducible on the latest PowerToys version?
/needinfo

I can confirm that this still does not work as of v0.63.0
On Fri, 21 Oct 2022 at 15:36, Taras @.***> wrote:

@pgerell https://github.com/pgerell, is the issue still reproducible on
the latest PowerToys version?
/needinfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/8493#issuecomment-1287054804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWVQ5Q6ZHGAJICPFZVD4TWEKS55ANCNFSM4USLI5MA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes, I can confirm that I have that enabled.

It still doesn't work with 0.66.


Bump payjoin version 0.13.1

test was failing from you deleting your nostr account so it couldn't sync nostr contacts, changed it to jack's npub to fix


Stabilize findTextInFiles API

Depends on

#59919
#59458

@andreamah I really like this feature for my next side project, and may I ask here how & when can this be stabilized?
It's been five years with no critical issue happening on it (let me know if I missed sth).

Some news: I'm finalizing this API with some other related APIs. You can see the status (plus a diagram) here :)

The new API shape for this is now available in this file: https://github.com/microsoft/vscode/blob/00374b8a5d0731f11f23d513d475facc2f36f54e/src/vscode-dts/vscode.proposed.findFiles2New.d.ts.
It will eventually have the New mentions removed when it gets merged with the existing proposed API.

Hi @andreamah I do plan to use this thank you. I manage the helix extension which is similar to vscode-vim, the downside with these extensions is they need to implement their own search (which is slow on large files)
Do you know if there is a version for finding text in a single file? Or can this be used with some mode to only search the focused editor? I also hope to use the Regex options too like you can in the UI, plus only searching within selected text.

Hi @andreamah I do plan to use this thank you. I manage the helix extension which is similar to vscode-vim, the downside with these extensions is they need to implement their own search (which is slow on large files)
Do you know if there is a version for finding text in a single file? Or can this be used with some mode to only search the focused editor? I also hope to use the Regex options too like you can in the UI, plus only searching within selected text.

Apologies on the late reply.
We currently don't have plans to have it in the currently opened file, but you can target files that you want to include via the include option.

Thanks for working on this - we'd like to implement search in a VSCode extension, and doing it from scratch is either really slow or really complicated. Any ETA on when this will be finalized? Thanks!

Probably 2+ months at this point. This API is tied into 3 others, and the TODOs are:

finalize the shape - it should be mostly finalized already.
get internal extension authors to adopt this.
Remove backward compatibility
Finalize API

Thanks @andreamah
I can't seem to see Regex support in this API, our extension would need that, is that something you plan on adding in future? The UI allows this so I don't see this being difficult to support at an API level.


hash change doesn't work

clicking the images after "Four directions stand out as particularly exciting" changes the location hash, but that seems to not do anything

Thanks for raising this! It was fixed by PR #8. :)


IndexError: list index out of range

When I run the code:accelerate launch --mixed_precision fp16 --gpu_ids all --num_processes 1 tools/train.py +exp=224x400 runner=default,I got an error like this.I was wondering the reason.I use the mini dataset to generate mmdet3d annotation files,the code is:python ./tools/create_data.py nuscenes --root-path ./data/nuscenes --out-dir ./data/nuscenes_mmdet3d_2 --extra-tag nuscenes --version v1.0-mini.
To be specific,the error is:

I need your help very much.

We randomly pick some indexes in the config for validation. You should be able to find validation_index in the config. Change it to contain only values less than 81 (because your val set only contains 81 samples).

I really appreciate your help,and I have already solved the problem.


Borg prune failing after udpate

Hi,
thanks for this great docker image. I am happy with it.
Yesterday I pulled the latest docker image, and my backup stopped working.
usage: borg prune [-h] [--critical] [--error] [--warning] [--info] [--debug]

                  [--debug-topic TOPIC] [-p] [--iec] [--log-json]

                  [--lock-wait SECONDS] [--bypass-lock] [--show-version]

                  [--show-rc] [--umask M] [--remote-path PATH]

                  [--remote-ratelimit RATE] [--upload-ratelimit RATE]

                  [--remote-buffer UPLOAD_BUFFER]

                  [--upload-buffer UPLOAD_BUFFER] [--consider-part-files]

                  [--debug-profile FILE] [--rsh RSH] [-n] [--force] [-s]

                  [--list] [--keep-within INTERVAL] [--keep-last SECONDLY]

                  [--keep-minutely MINUTELY] [-H HOURLY] [-d DAILY]

                  [-w WEEKLY] [-m MONTHLY] [-y YEARLY] [-P PREFIX | -a GLOB]

                  [--save-space] [-c SECONDS]

                  [REPOSITORY]

borg prune: error: argument -a/--glob-archives: There can be only one.

/mnt/borg-repository: Error running actions for repository

Command 'borg prune --keep-hourly 2 --keep-daily 7 --keep-weekly 4 --keep-monthly 12 --keep-yearly 10 --glob-archives backup-* --glob-archives borgmatic-backup-* --stats /mnt/borg-repository' returned non-zero exit status 2.

Error while creating a backup.

/etc/borgmatic.d/config.yaml: An error occurred

This is the location excerpt of my config.yaml
location:
    source_directories:
        - /mnt/source

    exclude_patterns:
        - '*nextcloud/data/*/files_trashbin/*'
        - '*nextcloud/data/updater-*/backups/*'

    repositories:
        - /mnt/borg-repository

Did I do something wrong in my config?
Thanks!

It looks like it has been fixed in the latest version:
https://projects.torsion.org/borgmatic-collective/borgmatic/issues/668

Latest version should be live, with the latest tag pointing to 1.7.11-1.2.4
Could you see if that works?

Fixed in https://github.com/modem7/docker-borgmatic/pull/94


Asynchronous test method is not supported ???

Remove it ...

@Eric4Zhang Assuming you got this all sorted? But yeah answer the question async test methods should be fine.


🐞 Bug: Following sign-up in the simulator, the dropdown menu and username extend beyond the screen.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The username encounters problems when it is long, and in the dropdown, it extends beyond the screen with short username also.
Steps To Reproduce
main.stylesheet.css
Expected Behavior
Dropdown and username should move left side instead of right side!
Screenshots

Device Information [optional]
- OS:Windows
- Browser:chrome and edge

Additional context
No response
Are you working on this issue?
Yes

@tanmoysrt can i work on this?

It's already fixed ig, check locally.

It's already fixed ig, check in local development.

yes it is fixes here is the issue no #4519

But dropdown is not fixed!

@foad-khan you can work on the fix

#4775 was having too many unnecessary prettify changes, So closed that PR and create new one.


pip install

Hi all, not necessarily an issue. I've successfully built IronPython3 and was hoping to install numpy and scipy. The cmd "ipy -X:Frames -m ensurepip" however results in "Could not import runpy module".
Any help is greatly appreciated. Thanks.

It looks like runpy fails to import (at least the first possible failure point) because we haven't implemented nonlocal. You could tweak functools.py to work without nonlocal, but I expect another failure point down the line. IronPython3 just isn't read for use yet.

Closing this since ipy -m ensurepip was fixed a a while back. There's still an issue with pip install for certain modules, but it's tracked at https://github.com/IronLanguages/ironpython3/issues/936.


crash on android

I'm submitting a ...  (check one with "x")

[ ] question
[X] any problem or bug report

OS:  (check one with "x")

[X] Android
[ ] iOS
[ ] Browser

cordova information:
cordova-plugin-advanced-http 2.1.1 "Advanced HTTP plugin"
cordova-plugin-customconfigparameters 3.0.0 "Custom Config Parameters"
cordova-plugin-device 2.0.2 "Device"
cordova-plugin-file 6.0.1 "File"
cordova-plugin-googlemaps 2.6.2 "cordova-plugin-googlemaps"
cordova-plugin-ionic-keyboard 2.1.3 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 4.1.0 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"
07-29 18:29:07.488  6774  6774 E AndroidRuntime: FATAL EXCEPTION: main
07-29 18:29:07.488  6774  6774 E AndroidRuntime: Process: io.ionic.starter, PID: 6774
07-29 18:29:07.488  6774  6774 E AndroidRuntime: java.lang.NullPointerException: Attempt to read from field 'int android.view.ViewGroup$MarginLayoutParams.leftMargin' on a null object reference
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at plugin.google.maps.MyPluginLayout$1.run(MyPluginLayout.java:122)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:873)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:99)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:193)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:6718)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
07-29 18:29:07.488  6774  6774 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
07-29 18:29:07.491  1159  2049 W ActivityManager:   Force finishing activity io.ionic.starter/.MainActivity

    "@ionic-native/core": "^5.8.0",
    "@ionic-native/google-maps": "^5.5.0",

Current behavior:
The app crashed when I navigate to the map page
Expected behavior:
No crash
https://github.com/rall/yelper

Support this plugin activity
I appreicate if you give me a beer :beer: from here

The error means you tried to create a map before native side has been ready.

thanks! I wonder why that would be on an android device but not an iOS simulator? I will investigate.

This plugin uses native google maps not JavaScript google maps on Android and iOS.
https://github.com/mapsplugin/cordova-plugin-googlemaps#how-does-this-plugin-work-android-ios
It means the implementations in Android and iOS are different.
iOS works fine, but why not Android are nonsense question (not blaming you, but many people asked), because code is different (= different software).

I'm not so much asking why it's different internally, more what I have to change in my code to make android work properly. I'm waiting for platform.ready() which I understood to be sufficient (and certainly seemed to be for android on the Ionic 3 app I worked on recently.)
But thanks for your swift reply and the good direction. I have something to work on now to attempt to resolve. I'll report back in this issue if I make any discovery
Also, thank you for your great work on this plugin. It must be an enormous undertaking and it's really genuinely appreciated. The performance improvement of this plugin over a javascript rendered alternative is quite significant.

Did you solve your problem?

I still get the occasional crash calling setDiv on Android, but I had some success with this RxJS stream:
showMap$:Observable<GoogleMap> emits when map should render, ie. after platform.ready()
scheduled(showMap$, animationFrameScheduler).pipe(
    filter(([_, visible]) => Boolean(visible)),
    take(1),
).subscribe(([mapObject]) => mapObject.setDiv(this.domId));

Which can also be expressed using a callback:
const googleMap = GoogleMaps.create({});
requestAnimationFrame(() => googleMap.setDiv("map-canvas"));

Maybe this is useful to someone in the future. I'll close this ticket issue for now


Beta Annealing Question

Hello!
Nice work on your PER implementation. I have a question. Why does your implementation not feature beta annealing for the Prioritized Replay Buffer? Some insight on this decision would be greatly appreciated!

Hi! The reason is very simple. I don't relate this to implementing PER specifically, it's a hyperparameter of a method that will use PER, so it didn't seem necessary for me to implement it. The fact that there is a small demonstration of DQN here is just a test that PER works, regardless of more complex methods. After all, this implementation is primarily for education purposes.


Leverage dynamic locale with gettext

Summary of changes
In order to let gettext work with an already implemented gettext of an app (or even accross dependencies), all format functions should use the current locale of gettext instead of the pre-configured default. This makes dynamic language switching possible as well.
This PR just swaps default for current without mentioning it in the docs. Let me know what you think of this, so I can perfect it and make it release worthy.
Checklist

[x] New functions have typespecs, changed functions were updated
[x] Same for documentation, including moduledocs
[x] Tests were added or updated to cover changes
[x] Commits were squashed into a single coherent commit

Ping! I still believe this is good to go with minor chances that it breaks for anyone out there. How can I help releasing this?

We can get this merged and released, but I'd like to have the CHANGELOG updated that mentions that this change is potentially breaking, but that the old behavior was technically a bug - the new behavior fixes that bug and will do the expected thing with regard to localization and integration with gettext.
Once that is done, I'll get this merged and rolled out in the next release

So, I somehow thought you wanted to do the editing of the changelog yourself, I forgot about this open PR in the meantime.
Just noticed it again today and decided to finally write the changelog entry. Also did a full rebase. Hope you're okay with the changes now.

Just wondering if you were able to check the recent changes?

I just rebased on your latest master. How to move forward with this? @bitwalker ?

Any news on this @bitwalker ?

@bitwalker it would be really great if you can find the time to look at this again. The changes are small, but the impact for our app is big as without it we need to manage the locale of Timex independently instead of it following our global Gettext locale.

@bitwalker any news on this?


Create Tables according to the ER diagram

I created and ran the necessary migration files.
I have included the foreign keys relations.
I have followed the relationship according to the er diagram

Hi @Zel-hub7,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that do not have a professional README please update it
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Unicode Decode Error

Hi Jerry,
I get the following error when I try to run to the example code!
Do you have any ideas how I can handle this?
Thanks
File "/Users/*****/.spyder-py3/temp.py", line 1, in <module>
  from hl7tojson import parser

File "/Users/******/opt/anaconda3/envs/rain/lib/python3.8/site-packages/hl7tojson-0.0.2-py3.8.egg/hl7tojson/parser.py", line 15, in <module>
  fields = pickle.load(f)

File "/Users/******/opt/anaconda3/envs/rain/lib/python3.8/codecs.py", line 322, in decode
  (result, consumed) = self._buffer_decode(data, self.errors, final)

UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte

@Kamran-Afzali I opened a PR: https://github.com/jerry-le/hl7tojson/pull/3
It's a pretty simple fix. Not sure if this project is still maintained though.

In parser.py add 'rb' to the following lines:
with open('{}/data/{}/fields.pickle'.format(FILE_PATH, HL7_VERSION), 'rb') as f:
fields = pickle.load(f)
with open('{}/data/{}/messages.pickle'.format(FILE_PATH, HL7_VERSION), 'rb') as f:
messages = pickle.load(f)
with open('{}/data/{}/segments.pickle'.format(FILE_PATH, HL7_VERSION), 'rb') as f:
segments = pickle.load(f)


Default help formatter hard limits command length to 20 characters

It would be nice if this was instead based on the length of the longest command or was configurable.
https://github.com/tj/commander/blob/master/lib/commander/help_formatters/terminal/help.erb#L12
@stephenchu

Hmm, it doesn't "limit" the name, it just doesn't line everything up any more if you have a name longer than 20 characters. In any case, would you be interested in putting together a patch for this?

Sorry, I should have been clearer -- yeah, it's just the command descriptions don't line up for commands whose name is longer than 20 characters (we discovered this using the "subcommand" pattern discussed here: https://github.com/tj/commander/issues/52).
I'll take a crack at a patch.

This issue was moved to commander-rb/commander#8


Latest Apollo master does not work

Describe the bug
I pulled latest Apollo master today, and build it successfully. But when I run bootstrap.sh or use cyber_launch to launch modules, it complained "mainboard" cannot be found.
To Reproduce
Steps to reproduce the behavior:

Clone or pull latest master
Enter Apollo docker (via dev_start and dev_into)
Run apollo.sh build_opt_gpu
Run bootstrap.sh or use cyber_launch to launch any module

Expected behavior
Both bootstrap script and module launch should work fine.
More details can be found at: https://github.com/lgsvl/simulator/issues/804

I think the problem may be in this change to setup.bash

This change fixes the problem in master by adding bazel-bin/cyber back into the PATH:
diff --git a/cyber/setup.bash b/cyber/setup.bash
index b13485d13d..e6b13662dd 100755
--- a/cyber/setup.bash
+++ b/cyber/setup.bash
@@ -15,6 +15,7 @@ bazel_bin_path="${APOLLO_ROOT_DIR}/bazel-bin"
apollo_tool_path="${bazel_bin_path}/modules/tools"
visualizer_path="${apollo_tool_path}/visualizer"

+cyber_bin_path="${bazel_bin_path}/cyber"
cyber_tool_path="${bazel_bin_path}/cyber/tools"
recorder_path="${cyber_tool_path}/cyber_recorder"
monitor_path="${cyber_tool_path}/cyber_monitor"
@@ -27,7 +28,7 @@ rosbag_to_record_path="${bazel_bin_path}/modules/data/tools/rosbag_to_record"

# TODO(all): place all these in one place and add_to_path
-for entry in "${recorder_path}" "${monitor_path}" "${launch_path}" \
+for entry in "${cyber_bin_path}" "${recorder_path}" "${monitor_path}" "${launch_path}" \
"${channel_path}" "${node_path}" "${service_path}" \
"${visualizer_path}" \
"${rosbag_to_record_path}" ; do


Missing MQTT parameters to use DPS Enrollment Groups

Hi. There is no explanation on how to use the Enrollment Groups of the Device Provisioning Service through MQTT.

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

ID: af124669-06d7-a0b0-da60-78081380af4c
Version Independent ID: 578a6172-e018-af2b-ce40-d0e3d24026aa
Content: Azure IoT Hub Device Provisioning Service - TPM Attestation
Content Source: articles/iot-dps/concepts-tpm-attestation.md
Service: iot-dps
GitHub Login: @nberdy
Microsoft Alias: nberdy

Hello @n-elia Thanks for the feedback.
I have assigned the issue to the content author for further review.

Thank you :)
Just to be more detailed, when using the MQTT protocol directly (as a device) along with X.509 certificate authentication to setup a Device Provisioning Service Individual Enrollment, I should put the registration_id of the individual enrollment inside some MQTT parameters such as the client username and the topic for sending the register message.
There is no reference on how to build such parameters when dealing with the Group Enrollment feature of the Device Provisioning Service.
Thank you again for the support!

@n-elia does this doc help?
Also, if you are trying to go through the registration process with a device, it's the same call regardless of whether that device was enrolled with an individual enrollment or a group enrollment. The device never needs to know how its enrolled.

Hi @nberdy !
That document is my main reference, and I'm actually referring to it in my bug report.
It says:

For the Username field, use {idScope}/registrations/{registration_id}/api-version=2019-03-31, where {idScope} is the idScope of the DPS.

The problem is that the registration id is not known a-priori in enrollment groups, so I am not able to build a proper MQTT call.
If you use a dummy registration id, the MQTT response will always be an authentication error.

@stevebus has cleared my doubts here 👍
In short, when using pure MQTT protocol secured by X.509 certificates to connect your devices to Azure DPS, you should use the CN/SubjectName of the certificate provided in the enrollment group as registration_id for building the MQTT parameters strings.
Also note that the certificates must be compliant to these guidelines, otherwise you'll get an error with code 401002, "Invalid certificate".
I still think that having these few indications inside the MQTT docs would be helpful!

FYI - I mentioned to Nicola in a comment on my blog that registration id guidance in x509 scenarios can be found here -> https://docs.microsoft.com/en-us/azure/iot-dps/concepts-service#registration-id   (the second bullet in the list)
Might be worth adding a quick reference to that anchor from the MQTT docs...

@nberdy, @stevebus, @DeepikaSadahiro Can one of you please ack on the PR listed directly above, so we can close this issue? Thanks!

@rajeevmv can you please validate or suggest changes to this PR so we can close this issue? https://github.com/MicrosoftDocs/azure-docs-pr/pull/183755

#reassign JimacoMS4

@n-elia, Thanks for your feedback! Updates have been published. See additional text and note about x.509 certs under username bullet: https://docs.microsoft.com/en-us/azure/iot-dps/iot-dps-mqtt-support#using-the-mqtt-protocol-directly-as-a-device. Since this issue is fairly old, I'm going to close it. If you have additional feedback, please feel free to @-comment me here or open a new issue.
Thanks again for your help with our documentation!
Jimaco

#please-close


Update Property Editor colors for better contrast and consistency

An update for Property Editor. Implements the [hasLink] feature to increase contrast of links. Switches purple text of "..." editable fields to Dracula orange for more contrast. Adjusts borders and margins of focused controls to make edges consistent. Switches splitters (drag handles) to light color. Removes some seemingly unnecessary code after testing.
Fixes #44
Fixes #39
Before:
https://github.com/dracula/freecad/assets/650565/f2b635e7-5cf1-4e6c-82b6-28f5f6e87ba6
After:
https://github.com/dracula/freecad/assets/650565/a3288b88-45ca-408b-a66d-f79239872c0a
(These MP4 files won't play in Firefox for me, may need to be opened in VLC.)

Also looks real good on my machine. Thank you so much for your work and sorry to make you wait so long!

No big deal, summer's busy for everyone. =)


Create a print css

When printing only a part of the notebook is printed, the rest is actually hidden

done in #195


Feat/1609 date format dynamic expression function

Description

Adds a new dynamic expression function formatDate. The function takes two parameters:

date - The ISO-string that should be formatted.
format - A format in which the date should be formatted (optional). If this argument is omitted, the function will use the selected language to find the correct locale, and calculate an appropriate format based on this.

Related Issue(s)

closes #1609

Verification/QA

Manual functionality testing

[x] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
[ ] No testing done/necessary

Automated tests

[x] Unit test(s) have been added/updated
[ ] Cypress E2E test(s) have been added/updated
[ ] No automatic tests are needed here (no functional changes/additions)
[ ] I want someone to help me make some tests

UU/WCAG (follow these guidelines until we have our own)

[ ] I have tested with a screen reader/keyboard navigation/automated wcag validator
[x] No testing done/necessary (no DOM/visual changes)
[ ] I want someone to help me perform accessibility testing

User documentation @ altinn-studio-docs

[ ] Has been added/updated

[x] No functionality has been changed/added, so no documentation is needed
[ ] I will do that later/have created an issue

Support in Altinn Studio

[ ] Issue(s) created for support in Studio

[x] This change/feature does not require any changes to Altinn Studio

Sprint board

[x] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
[ ] I don't have permissions to do that, please help me out

Labels

[x] I have added a kind/* label to this PR for proper release notes grouping
[ ] I don't have permissions to add labels, please help me out

The Percy tests that are failing are because of new options added to the frontend-test application for testing dynamic expressions in list components based on repeating groups. They are not introduced because of these changes.

Yes, I was planning on adding it to v3 together with the sortOrder functionality on selectable components :partying_face:

Nice! In that case, remember to update the table of functions in the docs 🙌

Will do!

@mikaelrss Have a look at the readme: https://github.com/Altinn/app-frontend-react/tree/main/src/features/expressions/shared-tests
The shared tests are unit-tests structured in JSON files that define all the input needed, along with the expected expression result. We have tests-runners implemented both in frontend and in backend (app-lib-dotnet), as there are two expression engine implementations. I don't see the need for a backend implementation for this function right now, but having at least one shared test case would make it pretty clear where these two implementations diverge - and would nudge us to remember adding a backend implementation at some point, should we need it.

Also, should we add to existing issues or create a new one to migrate over to this from moment, etc? 🤔

What a glorious piece of code! 🤩 Loving it! Especially the fantastically complex and thorough unit-test.. 😍 There was one eslint complaint about nested template strings, but otherwise this looks good!

:partying_face: Do you want me to refactor it to not use nested template strings, or should I disable the lint-rule for that line?

Also, should we add to existing issues or create a new one to migrate over to this from moment, etc? 🤔

Maybe we could keep the original issue(s), and tag it with v4? I can reference this issue in the others.

Do you want me to refactor it to not use nested template strings, or should I disable the lint-rule for that line?

Do what you prefer. I often refactor instead of using eslint-disable for such little things, and in some ways I agree with the rule that nested template strings quickly becomes hard to read. However, I didn't read this one too closely, so if you disagree with the readability it might work just as well just skipping the rule.

Maybe we could keep the original issue(s), and tag it with v4? I can reference this issue in the others.

Yup, go ahead and reference this in the other issues - but I'm not sure we need the v4 label. The reason the label is somewhat gold colored is because it should be used for all issues we need to complete in order to ship v4 (aka 'go gold'), but I'm not sure replacing moment (and thus the date picker?) is a hard requirement for v4.


View VHT functionality for MIdwife

@galimwavu Can you update on the status of this one?

Is this essentially just waiting for #49

Moving this to ready to test since #49 is in that state.


Please pull this fork with GIF support so it's in CocoaPods

Please pull this fork with GIF support so it's in CocoaPods
https://github.com/estebansotoara/Onboard/tree/ContentViewControllerGifSupport

+1

Also, check out: https://github.com/samheather/Onboard.
It allows the image used to be a series of images which it then animates with a duration. It's not really production ready (a few constants hard-coded in) but it's much much more memory efficient than the above one with Gif support (if you can change your gif into PNGs).


How to pin request to ip address

Hi, I have a problem to crawl a web because the domain redirects to IP address. I Used the command normally or request with IP address likevisit("ip_address") not working. then I try request using curl curl [domain], it's still not working, but when I am using curl command curl domain --resolve domain:ip_address: port it's working nicely.
so what can I do to implement --resolve  in colly.
Sorry for my English.
Thanks.

There isn't any direct support to change dial context on colly. However you can add a single line to your host file whether to resolve an ip address for a domain. Have you tried it?

Seems like it is possible: https://stackoverflow.com/questions/40624248/golang-force-http-request-to-specific-ip-similar-to-curl-resolve


KIA > Feature > _get_aliased_name unbound var

Analyzer found _get_aliased_name refers to unbound var name. That token should be root instead. This code probably works because of a fallback to outer scope, but that is very bad form.
Originally posted by @samiebee43 in https://github.com/WasatchPhotonics/ENLIGHTEN/pull/216#discussion_r1251291030

Backing from KIA


[assemble] NativeImageAssembler does not add graalVMNativeImage to adjusted distributions

Task List

[x] Steps to reproduce provided
[ ] Stacktrace (if present) provided
[x] Example that reproduces the problem (link to git repository is ideal)
[x] Full description of the issue provided (see below)

Steps to Reproduce

Clone https://github.com/jreleaser/helloworld-java-graalvm
Build the project: ./mvnw -Pjdks && ./mvnw verify
Assemble the distribution: jreleaser assemble
Display the current configuration: jreleaser config

Expected Behaviour
Distribution helloworld should have an extra property graalVMNativeImage set to true.
Actual Behaviour
Property is missing
distributions:
    helloworld:
        enabled: true
        active: always
        type: BINARY
        executable:
            name: helloworld
            windowsExtension: exe
        artifacts:
            artifact 0:
                path: /Users/aalmiray/dev/github/helloworld-java-graalvm/out/jreleaser/assemble/helloworld/native-image/helloworld-early-access-linux-x86_64.zip
                platform: linux-x86_64
            artifact 1:
                path: /Users/aalmiray/dev/github/helloworld-java-graalvm/out/jreleaser/assemble/helloworld/native-image/helloworld-early-access-osx-x86_64.zip
                platform: osx-x86_64
            artifact 2:
                path: /Users/aalmiray/dev/github/helloworld-java-graalvm/out/jreleaser/assemble/helloworld/native-image/helloworld-early-access-windows-x86_64.zip
                platform: windows-x86_64
        stereotype: cli

Environment Information

Operating System: N/A
JReleaser Version: 1.5.0
JDK Version: N/A

🎉 This issue has been resolved in v1.5.1 (Release Notes)


Change Sleuth versioning

We need to start using Netflix 1.2.x branch with the OpenFeign libraries. In order to achieve that the following version changes will take place

master will contain 1.2.x versions
1.1.x branch will be introduced that will use newer version of Netflix
1.0.x branch will remain as it is

Done


CodeBuild Fails  COMMAND_EXECUTION_ERROR Message: Error while executing command: npm run build. Reason: exit status 1

Project create CodePipeline project. CodeBuild Fails with the following error...
Creating an optimized production build...
146 | Failed to compile.
147 |
148 |<EMAIL_ADDRESS>149 | TypeScript error in /codebuild/output/src062423328/src/node_modules/@types/babel__traverse/index.d.ts(68,50):
150 | ']' expected.  TS1005
151 |
152 | 66 | }
153 | 67 |
154 | > 68 | export type ArrayKeys = keyof { [P in keyof T as T[P] extends any[] ? P : never]: P };
155 | |                                                  ^
156 | 69 |
157 | 70 | export class Scope {
158 | 71 |     constructor(path: NodePath, parentScope?: Scope);
159 |
160 |
161 | npm ERR! code ELIFECYCLE
162 | npm ERR! errno 1
163 | npm ERR<EMAIL_ADDRESS>build: react-scripts build
164 | npm ERR! Exit status 1
165 | npm ERR!
166 | npm ERR! Failed at the<EMAIL_ADDRESS>build script.
167 | npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
168 |
169 | npm ERR! A complete log of this run can be found in:
170 | npm ERR!     /root/.npm/_logs/2023-01-12T21_39_57_365Z-debug.log
171 |
172 | [Container] 2023/01/12 21:39:57 Command did not exit successfully npm run build exit status 1
173 | [Container] 2023/01/12 21:39:57 Phase complete: BUILD State: FAILED
174 | [Container] 2023/01/12 21:39:57 Phase context status code: COMMAND_EXECUTION_ERROR Message: Error while executing command: npm run build. Reason: exit status 1

Running into the same error. Were you able to resolve it?

Getting this same error. Anyone manage to resolve it?

Same error.

Hi all.  Thanks for submitting this.  I'll try to take a look at this later today or tomorrow and provide a fix.

This has been fixed.  For future reference, we create a ZIP file located at https://s3.amazonaws.com/aws-bookstore-demo/bookstore-webapp.zip which should contain the same contents as what is in the /assets directory of this repo.  This pertains to the front-end code for the Bookstore App.  When deploying the stack, that ZIP file is used to seed a CodeCommit repo with this code, use CodePipeline & CodeBuild to build the frontend, and then deploy the build to the assets S3 bucket in the stack.  That assets bucket is then presented to the user via a CloudFront distribution.   The code base here in GitHub was not in-sync with the ZIP file on S3.  These are now in-sync.  We need to figure out a better way to automate changes from GitHub being used to either update that ZIP file, or being used directly to create the CodeCommit repo in a launched stack.


Lumber multi fix

Description
Fixed lumberjack multiplayer
Test Instructions
Ran on two hardware swadges, confirmed they connect, play a game, update each others lives, and finish as expected.
Ticket Links
n/a
Readiness Checklist

[x] I have run make format to format the changes
[x] I have compiled the firmware and the changes have no warnings
[x] I have compiled the emulator and the changes have no warnings
[x] I have run make cppcheck and checked that cppcheck_result.txt has no warnings for the changes
[x] I have added doxygen comments to any code used by more than one Swadge mode. This includes /*! \file comments with Design Philosophy, Usage, and Example sections for new headers.
[x] I have run make docs and checked that doxy_warnings.txt has no warnings for the new code

@MrTroyD this closes #184 and #185, right?


Fix the windows build artifacts name

Motivation
Fix the  windows build output
Description
Add on the Makefile-libs.Inc the EXE := .exe
If the GOOS is equal to windows will force the value of EXE = .exe
Test Plan
Check on the release if the .exe was added on the artifacts.
I tested the behavior on other Github Repo.
TODO

[X] Explain motivation or link existing issue(s)
[X] Test changes and document test plan
[X] Update documentation as needed
[X] Update changelog as needed

bors merge

bors merge


Error on importing the library

Description
Error after importing the Channel or Client object from this library as seen in the screenshot below

Compiled with problems:X

ERROR in<EMAIL_ADDRESS>22:13-26

Module not found: Error: Can't resolve 'os' in 'D:\A.ing\Works\Others\Lacak-and-Valid\Exercises\React-Js\learn-mumble-client\node_modules\@tf2pickup-org\mumble-client\dist'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
	- add a fallback 'resolve.fallback: { "os": require.resolve("os-browserify/browser") }'
	- install 'os-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
	resolve.fallback: { "os": false }

ERROR in<EMAIL_ADDRESS>12:30-44

Module not found: Error: Can't resolve 'tls' in 'D:\A.ing\Works\Others\Lacak-and-Valid\Exercises\React-Js\learn-mumble-client\node_modules\@tf2pickup-org\mumble-client\dist'

Steps to produce

create a React JS app using the npx create-react-app command
enter the React Js app and install the @tf2pickup-org/mumble-client dependency by using the npm i @tf2pickup-org/mumble-client command
go to the App.js file and add import { Channel, Client } from "@tf2pickup-org/mumble-client";
run the React Js app by using the npm start command
the error appears

Environment
Framework: React Js 18.2.0
OS: Windows 11 and Monterey 12 (tested with both)
Node: Node: 14.21.2, 16.18.0, and 18.13.0 (switchable)

Problematic line - you're trying to use mumble-client in the browser, and the os module exists only in node.js environment.
I'll have it fixed.

Problematic line - you're trying to use mumble-client in the browser, and the os module exists only in node.js environment. I'll have it fixed.

You're right. That's why there is no UI/frontend/HTML file on this repository.
I thought this library could be used on the frontend side because there was a "client" name on the repository title ("mumble-client").

Problematic line - you're trying to use mumble-client in the browser, and the os module exists only in node.js environment. I'll have it fixed.

You're right. That's why there is no UI/frontend/HTML file on this repository.
I thought this library could be used on the frontend side because there was a "client" name on the repository title ("mumble-client").

It is a mumble client, as in it connects to a remote mumble server. It was intented to run on Node.JS environment primarily (my use-case), but there is no reason why it couldn't run in the browser as well.


feat: add it common test to test

Add integration common test on tidb-test.
Skip report currentlly

/lgtm

/approve


[20.09] zoom-us: 5.4.57862.0110 -> 5.5.6981.0202

(cherry picked from commit 11daaf463ca9c938453869c860df7e5b7c81f015)

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112003 1

1 package built:

zoom-us


Tree-shaking standard fonts possible?

Hey all 👋🏻
I'm wondering if there is a way to tree-shake the "standard fonts" as they are a huge portion of the bundle chunk containing pdf-lib.

Even when "only" using `PDFDocument, the fonts are included in the bundle.

Reproduction

Create a new folder and add a test.js file with the following content`

import PDFDocument  from 'pdf-lib'

async function foo () {
      const newPdf = await PDFDocument.create()
}

foo()

This ensures that we "only" create an empty PDF Document, nothing else.

Add webpack, babel and the lib as dependencies: yarn add pdf-lib webpack webpack-cli babel-loader @babel/core
Run webpack ./test.js -o bundle.js -p --display-chunks --display-modules --display-origins --module-bind js=babel-loader

Expected
Having a small result chunk without the StandardFonts.
Actual
A 344 KiB Bundle.

Related issues: #82

Hello @manniL! It is not currently possible to tree-shake the standard fonts. This is an issue I would like to fix, as the standard fonts occupy a huge amount of the bundle size. This is annoying if you don't use them. But it also impacts users that do use them, because most of the time only one or two of the fonts are used. Rarely all 14 of them!
That being said, I'm not sure if this can be fixed without a breaking API change. I think an API like this would solve the problem quite nicely:
import { PDFDocument, HelveticaFont } from 'pdf-lib';

const pdfDoc = await PDFDocument.create();
const helvetica = await pdfDoc.embedFont(HelveticaFont);

or this
import { PDFDocument } from 'pdf-lib';
import { Helvetica } from '@pdf-lib/standard-fonts';

const pdfDoc = await PDFDocument.create();
const helvetica = await pdfDoc.embedFont(Helvetica);

At first glance, I don't see a way to do achieve either of these APIs without a breaking change. It might be possible, though. If anybody would like to investigate this further, I'd be happy to accept a PR for it (again, assuming it can be done without breaking changes).
If, in fact, breaking changes are necessary, then fixing this will have to wait for<EMAIL_ADDRESS>There are several breaking features I'd like to include in 2.0, but don't currently have a roadmap for this.

I should also mention that StandardFonts.Helvetica is used internally by pdf-lib. This is why you can draw text and fill forms without specifying a font explicitly. I don't think it would be possible to tree-shake this. And I think the user experience would suffer significantly from being forced to always specify a font. So even once the API is reworked to allow tree-shaking of the standard fonts, it may not ever be possible to tree-shake StandardFonts.Helvetica. But even so, getting rid of the other 13 fonts would still reduce the bundle size significantly.

Hey @Hopding! Thanks for the detailed answer 🙏
I see the point there, definitely.
Maybe it'd be possible to "feature-flag" such a new "improved" API or use another separated named output, say MinimalPDFDocument (or so, naming things is hard 🙈 )?

I should also mention that StandardFonts.Helvetica is used internally by pdf-lib. This is why you can draw text and fill forms without specifying a font explicitly. I don't think it would be possible to tree-shake this.

That'd make sense, definitely! Still, removing the other 13 fonts would be an improvement already.

@manniL Can you please post your webpack config and update it if needed or at least give exact versions of webpack, pdf-lib and loader dependencies? I am trying to reproduce what you are doing and I fail.

Added this to the roadmap for tracking: https://github.com/Hopding/pdf-lib/discussions/998.

@Hopding I see the issue is closed. Has this been fixed?

@MickL Sadly not, it is only tracked in https://github.com/Hopding/pdf-lib/discussions/998


Gaze.prototype._addToWatched repeating readDir of same directory on each call

Thanks for the module - super cool.
We're running into a delay when a project site got past a couple dozen files and headed into the hundreds, we noticed the processing time was taking longer and longer -- minutes. In debugging, I focused in on this:
    var dirname = (helper.isDir(file)) ? filepath : path.dirname(filepath);
    dirname = helper.markDir(dirname);
..
    var readdir = fs.readdirSync(dirname);
    for (var j = 0; j < readdir.length; j++) {
      var dirfile = path.join(dirname, readdir[j]);
      if (fs.lstatSync(dirfile).isDirectory()) {
        helper.objectPush(this._watched, dirname, dirfile + path.sep);
      }
    }

When supplied a glob like ./data/data.json, it set dirname = ./data (the parent folder), and then it readdirSync that directory and added its subdirectories to the watchlist.
Two strange things:

I'm not sure why it would push the subdirectories, since I was just looking for one file
When you process 100 files in the same folder, it is doing the same thing over and over -- readdirSync the parent folder. That seems unnecessary.

Can you help?

This is what it seems it should be, but I'm not familiar with all use cases:
// Adds files and dirs to watched
Gaze.prototype._addToWatched = function(files) {
  for (var i = 0; i < files.length; i++) {
    var file = files[i];
    var filepath = path.resolve(this.options.cwd, file);

    var dirname = (helper.isDir(file)) ? filepath : path.dirname(filepath);
    dirname = helper.markDir(dirname);

    // If a new dir is added
    if (helper.isDir(file) && !(filepath in this._watched)) {
      helper.objectPush(this._watched, filepath, []);

      // add folders into the mix
      var readdir = fs.readdirSync(dirname);
      for (var j = 0; j < readdir.length; j++) {
        var dirfile = path.join(dirname, readdir[j]);
        if (fs.lstatSync(dirfile).isDirectory()) {
          helper.objectPush(this._watched, dirname, dirfile + path.sep);
        }
      }
    }

    if (file.slice(-1) === '/') { filepath += path.sep; }
    helper.objectPush(this._watched, path.dirname(filepath) + path.sep, filepath);

  }
  return this;
};

Hey! Thanks for diving into this. As long as all the tests pass with npm test all the use cases we support should be good. Feel free to open a PR! Thanks!

My pleasure, but unfortunately I ran npm test and did see a slew of errors like below. I fear it may be because I am on a PC with reduced admin rights, a dated version of node (0.12.7), and am behind a firewall, (aka developer heaven). Example:

Error: 'added.js' == 'Project (LO)'
at Object.equal (C:\wamp\www\gaze\node_modules\grunt-contrib-nodeunit\node_mo
dules\nodeunit\lib\types.js:83:39)
at Gaze. (C:\wamp\www\gaze\test\patterns_test.js:30:14)
at Gaze.emit (events.js:107:17)
at Gaze.emit (C:\wamp\www\gaze\lib\gaze.js:128:32)
at Gaze. (C:\wamp\www\gaze\lib\gaze.js:401:18)
at null._onTimeout (C:\wamp\www\gaze\lib\gaze.js:429:24)
at Timer.listOnTimeout (timers.js:119:15)

Should I proceed with the PR and lean on you to do a real npm test?

Hi,
I ran into the same issue as @smithwib. Making a fix, I doubt if the test is correct. This line bothers me:
var expected = {    '.': ['Project (LO)/', 'nested/', 'one.js', 'sub/'], };
The problem I see here is that we didn't add sub dir into the watch list. The test adds folders Project (LO) and nested only.
This may have unpleasant side-effects. For example, gaze is used in metalsmith-watch library. Metalsmith is a static website generator written in Node.JS. Common folder structure is

root

src // pages
build // ready website
node_modules

You configure to watch for changes in src folder and gaze also starts monitoring node_modules folder as well. Is it an expected behavior?
@shama Kyle, what do you think?

Has this been solved?

Is there a reason that sub folders are scanned and added to the watch list when a single file is specified to be watched and not a file glob that could potentially include sub-folders? This is causing issues in hyper where only a single file should be watched.
I added the following issue in the hyper repo:

#1776 - Gaze is adding all folders in HOME to watch list

There are a number of issues being raised in hyper that are directly related to the fact that gaze is scanning sub-folders in a users HOME when a single file (.hyper.js) is added to be watched.


Initialize Bootbot without needing app secret [Feature Request]

It would be good to find a way to do initialize Bootbot without needing app secret. I'm finding a way where I can put my code that uses Bootbot in github.
In Messenger bot setup it says nodejs needs a GET method for Facebook verification. I'm not sure if inside Bootbot constructor we can just setup an app.get('/webhook/', function (req, res)... method. Is this possible? Some samples use something like this:
app.get('/webhook/', function (req, res) {
    if (req.query['hub.verify_token'] === 'some_token') {
        res.send(req.query['hub.challenge'])
    }
    res.send('Error, wrong token')
})

The way I get around this is to use environmental variables for my secrets, and not committing my secrets file. I have been using this https://github.com/goliatone/envset but there are quite a few alternatives.
The app secret is very important for security and I believe making it optional is a mistake, especially with many safer and portable solutions.

@pferdefleisch I'll use those options as an alternative. I just raised this issue since other messenger bot tutorials like this does not need app secret. I'm not sure why Bootbot needs the app secret instead of using the sample code I posted in the question. What's the reason why Bootbot needs the app secret while other samples show that we can make bots without providing it?

@jannavarro
The reason for asking for the app secret is that Facebook signs all of the requests using the payload they send to you hashed with your app secret. This makes it so a bad actor finds your endpoint, they can't just spoof requests, pretending to be Facebook and as shown in this link, DOSing you or sending your users malicious messages (phishing, etc).
I was a bit cautious when I had to give my app secret for bootbot when the other frameworks didn't ask for it, but after looking through the code a bit I was less freaked out. And it turns out the other folks are doing it wrong. Read the article and you will understand the usefulness of this.
And have fun bootbotting ❤ 🤖 !

@pferdefleisch Thanks for the taking the time to explain things. I understand its use now.


Add extra guidance for simple orchestration that does not enable CAC auth into keycloak

https://github.com/jeremyatourville/stigman-orchestration
Wanted to give this repo a go but all the images it references are invalid. For example;
https://github.com/jeremyatourville/stigman-orchestration/blob/2807e4ba935e5ee6e02580dd131ccc5b5c488769/docker-compose.yml#L9
https://github.com/jeremyatourville/stigman-orchestration/blob/2807e4ba935e5ee6e02580dd131ccc5b5c488769/docker-compose.yml#L21
https://github.com/jeremyatourville/stigman-orchestration/blob/2807e4ba935e5ee6e02580dd131ccc5b5c488769/docker-compose.yml#L38
https://github.com/jeremyatourville/stigman-orchestration/blob/2807e4ba935e5ee6e02580dd131ccc5b5c488769/docker-compose.yml#L50
The domain is example.org so I'm not sure how this was ever valid code.
My use case is to bring in STIGManager into a cloud environment whew we access our systems with Apache Guacamole and thus have no ability to use DoD CAC auth. We need usernames and passwords. Later down the road we will integrate users into MS Active Directory but as a proof of concept, it just needs to be easy.
So are there simple steps to disable CAC auth and revert to user/pass for auth in this repo?
Also, I wasn't able to get stigman to auto import the DISA STIG lists by uncommenting the line - STIGMAN_INIT_IMPORT_STIGS=true.
I'm a noob to this but I see the value of a central stigman in my four different enclaves but the problems are they are air-gapped and CAC auth is not an options. We have RHEL 7 & 8 repo available to use and we can bring in any software we need but we do not have keycloak expertise. We intend to use this with EvaluateSTIG just as soon as I can figure out the certificate part so evalstig can talk to stigman. I wish the docs where more Mississippi style so I can better understand them! LOL
Thanks

You just need to use the latest images.  The principles of setup remain the same.  The critical piece is the setup for NGINX and how to setup Keycloak to use UN/PW.

Also, I wasn't able to get stigman to auto import the DISA STIG lists by uncommenting the line - STIGMAN_INIT_IMPORT_STIGS=true.

You're not going to do it that way.  The auto import presumes you have an internet connection.


Switch to setuptools

This lets you install as develop, so you can hack on code and not have to reinstall (the old .pth days!)

Erik, this ok to merge?


Add support for snapshot related metrics

Description
#165 Following the issue opened by ginkel, this PR adds the required changes for basic support for snapshot related metrics, starting with snapshot_age. More snapshot related metrics can be added on top of this changes.

[x] All my commits include DCO.DCO stands for Developer Certificate of Origin and it is your declaration that your contribution is correctly attributed and licensed. Please read more about how to attach DCO to your commits here (spoiler alert: in most cases it is as simple as using -s option when doing git commit).Please be aware that commits without DCO will cause failure of PR CI workflow and can not be merged.

FYI, to get rid of the Developer Certificate of Origin (DCO) check failure you need to sign the commits with -s option (git commit -s ...).
This will add a specific line into the commit message containing your signature with your email. See any other commit message in this repository for example of the text. You can create the signature manually in the commit message as well but using -s is more systematic way.
(Use of git -s option assume you configured git like this git config user.email<EMAIL_ADDRESS>first.)

@lukas-vlcek signed off the commit, please review

@lukas-vlcek I have added integration test and updated the README also, PTAL

@lukas-vlcek can you please help review this PR?

@patelsmit32123 Sorry, I was on 🌴 last week. I will do it this week.

@lukas-vlcek gentle ping

@lukas-vlcek sorry for bugging you, please review when you can

@patelsmit32123 Thanks a lot for your patience! I am really sorry for the delay in processing this.

First of all, I am almost done with reviewing it and I already made some comments on your PR before, can you see them? (The review is not submitted yet so I do not know if GitHub shows my comments to you in this case)
Second, a new release <IP_ADDRESS> has been cut today, would you mind rebasing the PR?

@lukas-vlcek i am not able to see your comments

@patelsmit32123 Can you see my comments now?

@lukas-vlcek thanks for the review, resolved all comments

@lukas-vlcek is there anything else needed to be done for this PR?

@patelsmit32123 I think it is ready. I am trying to implement https://github.com/Aiven-Open/prometheus-exporter-plugin-for-opensearch/issues/315 now (your PR will benefit from it) but if a new release is needed before BWC tests are ready I will just go ahead and merge it to make it part of the next release.
Does it sound good?


Docker-in-Docker fails with Mounted Volumes in Workspaces

Checklist

[X] I've looked through the issues and pull requests for similar reports

Describe your issue
When running docker-in-docker, if using a workspace and not at the root directory, the mounted volumes in the workspace are not properly adjusted to the the mounted prefix. This is because docker_mount does not use the mount finder.
What target(s) are you cross-compiling for?
No response
Which operating system is the host (e.g computer cross is on) running?

[ ] macOS
[ ] Windows
[X] Linux / BSD
[ ] other OS (specify in description)

What architecture is the host?

[X] x86_64 / AMD64
[ ] arm32
[ ] arm64 (including Mac M1)

What container engine is cross using?

[X] docker
[ ] podman
[ ] other container engine (specify in description)

cross version
cross 0.2.2 (00c1de2)
Example
While testing the integration tests in #873, the following script fails, producing the subsequent output:
Script
#!/usr/bin/env bash

# test to see that running docker-in-docker works

set -x
set -eo pipefail

if [[ -z "${TARGET}" ]]; then
    export TARGET="aarch64-unknown-linux-gnu"
fi

source=$(dirname "${BASH_SOURCE[0]}")
source=$(realpath "${source}")
home=$(dirname "${source}")

main() {
    docker run -v "${home}":/project \
        -w /project --rm -e TARGET \
        -v /var/run/docker.sock:/var/run/docker.sock \
        docker:18.09-dind sh -c '
#!/usr/bin/env sh
set -x
set -euo pipefail

apk add curl
curl --proto "=https" --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
source "${HOME}/.cargo/env"

# building on release is slow
apk add libgcc gcc musl-dev
cargo test --workspace
cargo install --path . --force --debug

export CROSS_CONTAINER_IN_CONTAINER=1

apk add git
td="$(mktemp -d)"
git clone --depth 1 https://github.com/cross-rs/rust-cpp-hello-word "${td}"
cd "${td}"
cross run --target "${TARGET}" --verbose

td="$(mktemp -d)"
git clone --depth 1 https://github.com/cross-rs/test-workspace "${td}"
cd "${td}"
cross build --target "${TARGET}" --workspace \
    --manifest-path="./workspace/Cargo.toml" --verbose
cd workspace
cross build --target "${TARGET}" --workspace --verbose
cd binary
cross run --target "${TARGET}" --verbose
'
}

main

Output
/usr/local/bin/docker run --userns host -e 'PKG_CONFIG_ALLOW_CROSS=1' -e 'XARGO_HOME=/xargo' -e 'CARGO_HOME=/cargo' -e 'CARGO_TARGET_DIR=/target' -e 'CROSS_RUNNER=' -e 'USER=root' -v /tmp/tmp.hppfjI/external/external_lib:/tmp/tmp.hppfjI/external/external_lib --rm --user 0:0 -v /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/root/.xargo:/xargo:Z -v /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/root/.cargo:/cargo:Z -v /cargo/bin -v /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/workspace:/var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/workspace:Z -v /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu:/rust:Z,ro -v /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/workspace/target:/target:Z -w /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/workspace ghcr.io/cross-rs/aarch64-unknown-linux-gnu:0.2.2 sh -c 'PATH=$PATH:/rust/bin cargo build --target aarch64-unknown-linux-gnu --workspace --verbose'
error: failed to load manifest for workspace member `/var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/workspace/binary`

Caused by:
  failed to load manifest for dependency `test-workspace`

Caused by:
  failed to load manifest for dependency `external_lib`

Caused by:
  failed to read `/var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/external/external_lib/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

We can see that external_lib is mounted at /tmp/tmp.hppfjI/external/external_lib:/tmp/tmp.hppfjI/external/external_lib , and not at /var/lib/docker/overlay2/4f20476a48228131897c42ce45df9e9ef74864fc590040ebfa4bce7f086f375e/merged/tmp/tmp.hppfjI/test-workspace/external/external_lib/. Likewise, we only mount workspace, and not test-workspace.
Additional information / notes
No response

nice! this should be an easy fix though


Added support for Linux and Windows for Azure functions,storage and API services

Thanks for the contribution, we have added support for both linux and windows in our services.


[presentmon] Add source code files

Fixed based on comments.
Add the original project's file GpuTrace.hpp to the compilation list.

[X] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Compile test pass with following triplets:
x86-windows
x64-windows

Thanks!


Fix pvl twitter username subst

Part of https://github.com/keybase/client/issues/5973
old:
✔ "resiak" on twitter failed: Bad post authored: wanted ${username_service} but got wjjjjt (code=203)
new:
✔ "resiak" on twitter failed: Bad post authored: wanted 'resiak' but got 'wjjjjt' (code=203)

@mlsteele clarified that the single quotes are just for consistent punctuation, and the actual fix is replacing a $ with a %. LGTM.


Multiple accounts?

I can't figure out how to associate a new account with t -- I did it once before,
but now can't recall the solution.

I can't add an app unless I have a mobile number for the account.
I can't put my mobile number on the new account because it's associated with another account.

What's the workaround?
Hal

Solved over Google Talk.


Fix Unity threading issues on Linux that prevented the mod from working

Upon further investigation, the problem is caused by using Unity's GameObject's properties in the simulation thread, namely NetInfo.name and Time.time.
The proposed solution is that instead of using OnBeforeSimulationTick callback directly, one can defer the road segment rebuilding to the simulation thread by using the IThreading interface. As far as I tested, it doesn't cause any issues (as opposed to simply doing everything in OnUpdate, which bugs out sometimes).
It's quick and dirty, but it gets the point across.

Thank you for your effort in investigating this issue. I think this is still a bit problematic in how much is now being done in the main thread that should be done in the simulation thread instead. It could cause crashes and data corruption even if it appears to work most of the time.
I will try to figure out which part exactly is doing something Unity doesn't like, and to move only that to the main thread. Your fix is very helpful, it narrows down the area to search.

I have now submitted an update to Steam where I moved the most likely offending code to the main thread (FindRoadPrefabs.) Hopefully that will fix the issue.

The only significant part of code that I moved was raytracing, and it shouldn't affect the game at all. But whatever.
prefab.time and Time.time are still left there, however. Those two are GameObject properties, and should be used in the main thread. They do cause the problem, that's for sure. And that's why your fix isn't working at the moment.
Get rid of those two, and you should be good to go.

I am thinking it might be prefab.name that is causing it to access the transform. I wouldn't say raytracing and getting the control points wouldn't affect the game at all. They are being performed on data structures that are being changed from another thread. If it's changed from another thread while the mod is doing the raytracing, it might get back invalid data or crash the game.
The Unity APIs the mod is currently using from the simulation thread however are not very likely to cause issues, because they are being called on objects that remain constant throughout the whole game. The only reason I can think of why it doesn't work on Linux is that the Linux build of the game was done with some debug options enabled, which cause it to check the thread. However, moving them all to the main thread would be the best of course, I agree on that.
While I trust you that the fix does work, I don't want to compromise the mod for those it is currently working well. I made my mod by replicating what the game is doing internally with its road building tools and tried to place everything on the same thread that the game has them, just to be sure it would be as stable as possible.

@viakmaky, how about me moving the raytracing to simulation thread too? If we agree that it's the only problem in my fix.
The actual errors with the prefab.name/Time.time are "get_name/get_time can only be called from the main thread", respectively. It doesn't refer to transforms, it's just that you're explicitly forbidden to change GameObject's properties from another thread.
I don't really believe that can cause any problems since the worst that can happen is you'll get raytracing data from a gamestate that is a few milliseconds older - just like when using Time.time, you'll receive a slightly different value in both threads.
While cross-thread access to Unity's objects might work, it isn't supposed to happen - as per documentation. And there is a threading guard that should prevent that from happening, which for some reason fails to work on Windows (threads work differently?). The devs clearly developed the game with having that in mind, because it works without a hitch.

There really is not reason to argue about it. I don't see why it would be a good thing to do something that is clearly wrong when you can just move the actual offending code to the main thread.
Also I don't see why you think accessing Unity's method from the wrong thread is such a bad thing, but accessing the game's other methods isn't. They are both bad and I want to get rid of them. I just don't want to change from one wrong implementation to another wrong implementation, but to fix it correctly.

Well, the correct fix would be using SimulationManager singleton to perform all relevant actions in the correct thread. Just like NetTool does it in the game.

This is getting off-track quite a bit. The mod already does things in the correct thread, in the same thread that NetTool does them.

Since no-one is going to fix anything any time soon, might as well go off-track.
NetTool performs all its UI-related stuff in the UI thread, as it is supposed to. You do not. If you want to do everything properly, you need to alter the workflow quite substantially - probably even rewrite the whole thing. That would take time, during which your mod isn't working at one platform (maybe two, if MacOS is also affected).
The reason I'm arguing is that I find having an actual working mod more important than rejecting the fix for a rather superficial reason.
I will continue working on it, though.

Why do you say no-one is going fix anything any time soon? I am going to move those Unity calls to the main thread asap now that I know what they are.
Also regarding the threading, I am doing all UI related work in the main thread, which is also the UI thread. What I'm doing in the simulation thread is what NetTool is doing in SimulationStep, which I'm pretty sure is also run in the simulation thread. I don't see where I'm doing anything in the wrong thread other than those two Unity calls.

Over time I'm probably going to improve the code by moving all the tool related tool inside the custom tool class I have, to make it consistent with the game's implementation. The reason the code is not inside that class right now is that I only managed to make my own tool at a late phase of the development and didn't have time to refactor the code. That doesn't mean the work is not being done in the correct thread however even as it is.

You'd run into the following problem: you need the prefab's name to build a corresponding segment (in simulation thread), and to get the prefab (that belongs to the main thread), you need to raycast (simulation thread again). That's why I'm saying you'll need to redesign to fix this, unless you want to hop between threads constantly.

I have a slightly different implementation in mind that avoids accessing the name outside of the main thread. It's a small change that I will try later today. I'm not doing anything else with the prefab so passing it around in the simulation thread should be fine (especially since the game itself already does that) as long as I don't access the name property.

It works, great :) Guess I'll close this.


Run Standard formatter prior to releasing 3.0

Use standard to format source code, require all future updates to use standard format. Reduces chances of superfluous code changes, such as white-space, slowing down the review process.

Should we have a GitHub PR check for Standard linting?
Also, we're using Semistandard (Standard + semicolons), not sure if there's any interest in changing it.

I didn’t that was a thing! Semistandard linting would work.

On Feb 9, 2017, at 8:01 PM, Mark Bao<EMAIL_ADDRESS>wrote:
Should we have a GitHub PR check for Standard linting?
Also, we're using Semistandard (Standard + semicolons), not sure if there's any interest in changing it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/speakeasyjs/speakeasy/issues/78#issuecomment-278852043, or mute the thread https://github.com/notifications/unsubscribe-auth/AAm4b0XNdNDo0O1aZX4mDaPNzrfoYv4Vks5ra-EcgaJpZM4L85cP.

Sounds good. I made sure our 2.0 release passed Semistandard, but likely not the code since.


guild/tensorboard.py conflicts with torch.utils.tensorboard

When running a torch script via guild, if that torch script uses torch.utils.tensorboard, you get the error
ImportError: TensorBoard logging requires TensorBoard version 1.15 or above

caused by torch/utils/tensorboard/__init__.py trying to import tensorboard and ending up importing guild/tensorboard.py instead of the actual tensorboard, thus breaking the version check and everything that follows.  Renaming the file guild/tensorboard.py fixes the error. Could you consider renaming this module of Guild so that we don't have to fix this error manually?

yes, I think that's reasonable.

Hi @ab

Hi @abogatskiy thank you for the report! That's frustrating.
Do you have a bit of code that demonstrates the problem you're seeing? Something small/short if possible would help us to track this down.

I created an issue resolution doc for this. We should use this to recreate the problem if possible.
I don't think this is a Guild issue (I could easily be wrong - it's happened!) but an environment configuration issue for whatever you're running. That guild.tensorboard is being imported before tensorboard suggests that there's an errant entry in your Python system path that's point to $GUILD_INSTALL_LOCATION/guild. If that's the case, we need to figure out why that's getting in there. Guild does modify the Python path, but it'd be a huge bug to add that particular entry.
Long story long, we need more input from you :)


Pass through onlyOperationTypes and onlyEnumTypes but warn if they are used

Description
Related # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take action on it as appropriate
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Environment:

OS:
@graphql-codegen/...:
NodeJS:

Checklist:

[ ] I have followed the CONTRIBUTING doc and the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

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

Reverting instead of adding option here: https://github.com/dotansimha/graphql-code-generator/pull/10182


[BUG] - NextUIProvider breaks layout

NextUI Version
2.1.5
Describe the bug
NextUIProvider should not add a new div to the final html or it should have a display: contents; so the element don't break the layout when it needs to have full height, like this:

For now I am using [&>div]:contents in the body to solve the problem.
If the div is really necessary maybe the docs could explain that?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Setup NextUIProvider

Expected behavior
Should not contribute to the layout
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Safari

I just realized that display: contents; in NextUIProvider breaks some components like Popover

Hey @cleytoncarvalho NextUI does not add any default styles to the application, only the needed variables for theme colors/layout etc..

@jrgarciadev yeah I see. This is one of the reasons I choose the lib for my new project, really liked it. The problem I mention is related with the div that NextUIProvider creates in the DOM, in my case I need it to have full height but if I do it many components don't work properly like Popover.

Are you sure it comes from NextUIProvider? This is the provider code, only the overlay and i18n providers https://github.com/nextui-org/nextui/blob/main/packages/core/system/src/provider.tsx, we don't add any dom element

Thanks! I will investigate.

The div is created by the OverlayProvider. I have done some testing and want to ask you if the OverlayProvider is really necessary? Can't find many information about it in the Aria Kit docs, but found this:
"This is used to ensure that the main content of the application is hidden from screen readers if a modal or other overlay is opened." - https://react-spectrum.adobe.com/react-aria/useDialog.html#example
It adds aria-hidden="true" to all elements when something like a popover is open (except the popover), but I removed OverlayProvider and this behavior still continues, so I think OverlayProvider is not necessary.

Oh got it, the thing is that the OverlayProvides is the one that provides the context for Modal stack, so we cannot remove it, it would break the modal context.
Did you try adding min-h-screen to the body? We use the same provider in nextui.org and we don't have this issue

You can also check the Next.js templates:
https://github.com/nextui-org/next-pages-template
https://github.com/nextui-org/next-app-template

Thanks for the explanation! min-h-screen will fix my issue.

Using min-h-screen alone didn't work for me in the Remix Blues stack.
What did work was adding a css file with
body > [data-overlay-container] {
  height: 100%;
}

so the added div is the same height as the body.
Any chance in the future that NextUIProvider could take a className prop and pass it to the OverlayProvider (and presumably on to the wrapping div)?
Then we could do something like:
<body className="h-full">
  <NextUIProvider className="h-full">
    <Outlet />

Hey @ahuth thank you for your feedback and we have now added support for setting className for the NextUIProvider. try on the latest version 🙌🏻

Thanks!

To solve this problem is just easy. My solution is just adding the style={{ height: "100%" }} to the <NextUIProvider>.
So the final will be look like:
<NextUIProvider style={{ height: "100%" }}>     <div className='app'>     </div> </NextUIProvider>


Add pMuTT.constants for all constants in VGA

Remove any hard-coded constants in VGA and replace with references to the pMuTT.constants class.

pgradd.Consts deprecated in favor of pmutt.constants
#10 opened as it's a complementary issue


008.sh extended to check if UPF Scale Succeeded

/ok-to-test

I am going to merge this, test it on a live system I have running now, then submit any needed tweaks in a separate PR.
/approve
/lgtm

/retest


Class 'StringCompilerException' not found

In Laravel 5 I'm getting the following error (obviously I understand why this is appearing)
FatalErrorException in StringView.php line 68:
Class 'StringCompilerException' not found in StringView.php line 68

@netjordan It looks like the StringCompilerException file is using a different namespace, Wpb\StringBladeCompiler\Exceptions vs Wpb\StringBladeCompiler
Can you test and see if either changing the use statement in the StringView (https://github.com/TerrePorter/StringBladeCompiler/blob/2.0/src/Wpb/StringBladeCompiler/StringView.php#L11) to Wpb\StringBladeCompiler\Exceptions\StringCompilerException fixes the problem.

fixed namespace for StringCompilerException