Local wallet Mints using 20K for Remote Master Node

Describe the issue
Running remote MN on Ubuntu 17.10. Running local wallet 3.0.4 on Windows 10. Used setup guide and MN was running for around five hours. Unlocked local wallet to stake coins over the 20K which were supposed to be locked. Took a few minutes and the 20K Minted and the MN went from Enabled to Missing.
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:

Get remote MN working.
Unlock local wallet for staking only.
Mints on 20K and disables MN.

Expected behavior
Local wallet should have locked the 20K and staked the remainder.
Actual behavior
Mints on 20K.
Screenshots.
N/A
What version of Ion Core are you using?
Local 3.0.4, Remote 3.0.3
Machine specs:

OS: Remote Ubuntu 17.10, Local Windows 10
CPU: Remote running under ESXi, Local AMD FX-8350
RAM: Remote 2GB, Local 32GB
Disk size: Remote 1TB VMDK, 4TB SATA
Disk Type (HD/SDD): VMDK/SATA

Any extra information that might be useful in the debugging process.

Can you post your anonymized ioncoin.conf files and masternode.conf?

Thanks for submitting the issue
as @FornaxA asked, please show us your configs without keys, username and password. Additionally, if you look up in your log and find those lines where it happened, it would be more clearer what happened.
if you staked, did your amount split and/or is your masternode still active or do you get txid error when activating it? Try activating with this command: startmasternode alias 0 YourMNAlias.
When you post your configs please check twice to remove rpcusername and rpcpassworde as well as masternode private keys (those are hopefully not from your wallet but generated by masternode genkey)

Attached are heavily redacted configs in a PGP file with fornaxA's public key. Seems like I redacted quite a bit!  It split in two. I will check the log file. I went ahead and created new settings and have it running again. Hope that doesn't hurt the troubleshooting.
RedactedPGPConfigs.txt

@DaZuru : mnconflock can be removed from your ioncoin.conf.
The log file shows the minting tx.. Can you go back to when you started the wallet, and find the section that looks like this:
2018-03-30 08:37:48 init message: Loading masternode payment cache...
2018-03-30 08:37:48 Locking Masternodes:
2018-03-30 08:37:48   <TXHASH> <VOUT>

(Where  and  refer to the tx id and vout that are defined in your masternode.conf)

I can not decode, you seem not have added my public key to the encryption.
@DaZuru this is how it looks if you run it together with masternode.conf
2018-03-30 06:31:10 init message: Loading masternode cache...
2018-03-30 06:31:10 init message: Loading budget cache...
2018-03-30 06:31:10 init message: Loading masternode payment cache...
2018-03-30 06:31:10 Locking Masternodes:
2018-03-30 06:31:10   9xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 4
2018-03-30 06:31:10   9xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 0
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 1
2018-03-30 06:31:10   9xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 1
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 3
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 4
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 6
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 7
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 8
2018-03-30 06:31:10   9xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 2
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 10
2018-03-30 06:31:10   4xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 11
2018-03-30 06:31:10   2xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx7 0
2018-03-30 06:31:10   6xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx3 1
2018-03-30 06:31:10   bxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 0
2018-03-30 06:31:10   bxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 1
2018-03-30 06:31:10   9xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxc 0
2018-03-30 06:31:10   bxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxe 4
2018-03-30 06:31:10   0xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxa 1
2018-03-30 06:31:10   8xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxb 0
2018-03-30 06:31:10 fLiteMode 0

mnconflock is not required and was actually not in our guide.
Beside your config, you might start it also wrongly. You can reffer to console help which has all required information, here is as example --help masternode
masternode "command"...

Set of commands to execute masternode related actions
This command is depreciated, please see individual command documentation for future reference

Arguments:
1. "command"        (string or set of strings, required) The command to execute

Available commands:
  count        - Print count information of all known masternodes
  current      - Print info on current masternode winner
  debug        - Print masternode status
  genkey       - Generate new masternodeprivkey
  outputs      - Print masternode compatible outputs
  start        - Start masternode configured in ioncoin.conf
  start-alias  - Start single masternode by assigned alias configured in masternode.conf
  start-<mode> - Start masternodes configured in masternode.conf (<mode>: 'all', 'missing', 'disabled')
  status       - Print masternode status information
  list         - Print list of all known masternodes (see masternodelist for more info)
  list-conf    - Print masternode.conf in JSON format
  winners      - Print list of masternode winners

If you txid is confirmed and unsplitted (like by stake), and your index is correct as well as port number and masternode's private key, then it will work.
Please supply your configs in plaintext (non excrypted) **without RPC password/username and private keys". I am quite sure that your configs are not correct.

@FornaxA I started and stopped the wallet a few times. In the section where the mint occurred, there was no Locking Masternodes. My guess is that I didn't start the Masternode or have the masternode config in place before it minted. It minted five minutes after start. I could have been editing files and letting blocks catch up. I have already resent 20K to a new address and reconfigured the masternode with new data. It has been working but I have not enabled staking.

@DaZuru : it should not be able to mint before everything was synchronized, but yeah, perhaps it was a freak occurrence or maybe related to editing config files/not saving files before starting the node/etc.
Either way: if it occurs again, please reopen this issue (I'm closing it now). And no need to apologize: thank you for opening the issue and taking the trouble to share your files!


Duplicated mapping key in locales?

My Yamllint reports like below. This may be harmless. But is it better to fix?
$ yamllint add config/locales/en.yml
_ YAML Lint failed for config/locales/en.yml
duplicated mapping key at line 92, column 16:
      login: Log in

YAML will consider the last assignment to key. It's harmfully to code when you need to define a behavior inside your translation, and you get a override value. I not consider the duplicate key, that makes the file with some junk code.


Fixed etc files github action

Sometimes the updater github action would fail because stefanzweifel/git-auto-commit-action isn't properly maintained. This PR replaces that library with the same library that the other github actions use. I also fixed an edge case where the run could fail if no changes happened since the last run.

Good catch once again @ItsIgnacioPortal


Transfer code to PyTorch

Hey, thanks for the hard work. I want to adapt the code for use in a PyTorch model but can't find any explicit license and therefore have to assume the code is not free for addition. Could you adapt/specify your license to allow me and others to build on your code?

Done.

Thanks a lot:)


[SYSTEMML-1408] Add padding parameters to max-pooling layers

This adds padding parameters to the max-pooling layers, along with the
associated tests.  Also, there are some general code formatting updates.

cc @niketanpansare I found a bug in the built-in max_pool function for any padding arguments >= 1.  The correctness test for the case of pad=1 starting at line 253 of nn/test/test.dml showcases the error.  This test simply checks the max pooling layers for correctness, using the matrices as described in the file.
Looking at the error and the code in LibMatrixDNN.java, I think it may be a striding issue.
To run, please cd $SYSTEMML_HOME/scripts/staging/SystemML, then ../../../bin/systemml nn/test/tests.dml.  The test referenced above will output "- Testing for correct behavior against known answer w/ pad=1".

Thanks @dusenberrymw I will look into it :)

@niketanpansare To give the high-level summary, here is the issue:
The test is a manual check of 2x2, stride-2 max pooling with 1 pixel padding on a specified matrix.  The DML version gives the correct answer, while the built-in version does not.  Everything is fine with padding=0.
Matrix:
  # -- channel 1
  #  0  0  0  0  0  0
  #  0  1  2  3  4  0
  #  0  5  6  7  8  0
  #  0  9 10 11 12  0
  #  0 13 14 15 16  0
  #  0  0  0  0  0  0
  # -- channel 2
  #  0  0  0  0  0  0
  #  0  1  5  9 13  0
  #  0  2  6 10 14  0
  #  0  3  7 11 15  0
  #  0  4  8 12 16  0
  #  0  0  0  0  0  0

Correct output:
  # -- channel 1
  #  1  3  4
  #  9 11 12
  # 13 15 16
  # -- channel 2
  #  1  9 13
  #  3 11 15
  #  4 12 16

Builtin output:
  # -- channel 1
  #  2  3  4
  # 10 11 12
  # 14 15 16
  # -- channel 2
  #  5  9 13
  #  7 11 15
  #  8 12 16

@niketanpansare I think I figured out the behavior -- the builtin version is (1) not adding the 1st and 6th columns of padding, and (2) is performing stride-1 max-pooling on the partially-padded matrix.
Partially-padded matrix:
Matrix:
  # -- channel 1
  #   0  0  0  0
  #   1  2  3  4
  #   5  6  7  8
  #   9 10 11 12
  #  13 14 15 16
  #   0  0  0  0
  # -- channel 2
  #   0  0  0  0
  #   1  5  9 13
  #   2  6 10 14
  #   3  7 11 15
  #   4  8 12 16
  #   0  0  0  0

@niketanpansare I've figured out the issue in LibMatrixDNN.java, and I'll update the PR soon with the fix.

@dusenberrymw If you have not started, please hold off on working on it. I am working on it: https://github.com/niketanpansare/incubator-systemml/tree/maxpool

@niketanpansare Haha, well I had fixed the forward pass, and looking at your commits, we converged on the same forward solution in terms of fillIndexesArray and the zero-padded rows in doPooling.  Looks like you've already started on the backward pass as well, so I would suggest just continuing.  I'll push some more DML test cases that I wrote to this PR (negative matrices, etc.)

@niketanpansare Also, I created SYSTEMML-1428 for the bug.

@niketanpansare Okay, I added more DML test cases.  All tests should run with no ERROR messages once the max pooling bugs are fixed.

@niketanpansare Also, here was my max pooling forward pass fix: https://github.com/dusenberrymw/incubator-systemml/commit/d7c5a238c3f6a0c86603e83426a40d9583e1ebbc

Hahah ... same fix !! ... there is another corner case, which I am looking into and will deliver the fix soon :)

@niketanpansare Okay I extended the nn::util::pad_image function with a pad_value parameter, and updated the other max pooling layers to use a negative infinity (pad_value=-1/0) as the padding value.  All tests are passing now on top of your PR.  Let's merge these.
Also, are you using nn::util::pad_image at all in Barista?


TypeError: 'bool' object is not subscriptable

Hi, when I run the eval command via:
python mega_nerf/eval.py --config_file configs/nerf/building.yaml --exp_name exp_official --dataset_path ./data/building/building-pixsfm --container_path ./data/building/building-pixsfm-8.pt
I face the following error:
Traceback (most recent call last):   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/eval.py", line 30, in <module>     main(_get_eval_opts())   File "/mnt/backup2/home/zlzhao21/software/miniconda3/envs/mega-nerf/lib/python3.9/site-packages/torch/distributed/elastic/multiprocessing/errors/__init__.py", line 345, in wrapper     return f(*args, **kwargs)   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/eval.py", line 26, in main     Runner(hparams).eval()   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/runner.py", line 309, in eval     val_metrics = self._run_validation(0)   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/runner.py", line 409, in _run_validation     results, _ = self.render_image(metadata_item)   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/runner.py", line 569, in render_image     result_batch, _ = render_rays(nerf=nerf, bg_nerf=bg_nerf,   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/rendering.py", line 88, in render_rays     results = _get_results(nerf=nerf,   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/rendering.py", line 195, in _get_results     _inference(results=results,   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/rendering.py", line 324, in _inference     model_chunk = nerf(typ == 'coarse', xyz_chunk, sigma_noise=sigma_noise)   File "/mnt/backup2/home/zlzhao21/software/miniconda3/envs/mega-nerf/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl     return forward_call(*input, **kwargs)   File "/mnt/backup2/home/zlzhao21/aug_code/mega-nerf/mega_nerf/models/mega_nerf.py", line 22, in forward     cluster_distances = torch.cdist(x[:, self.cluster_dim_start:3], self.centroids[:, self.cluster_dim_start:]) TypeError: 'bool' object is not subscriptable
How can I deal with this?

The reason is to switch from nerf config to meta-nerf config.

The reason is to switch from nerf config to meta-nerf config.

Nice, that's works for me

I had the same problem, how did you solve it?

The reason is to switch from nerf config to meta-nerf config.

What is the meta-config ? Where is it located ?

Change nerf to mega-nerf is not enough for me, the following line should be changed to model_chunk = nerf(xyz_chunk, typ == 'coarse', sigma_noise=sigma_noise)
https://github.com/cmusatyalab/mega-nerf/blob/76d8d76b66d70f8a0819defc0cef4fb9aa251672/mega_nerf/rendering.py#L324


where is .sql file?

where is .sql file? you can upload this?

The project did not end by us
After that We will put .sql & the Full files


Error: Icon collection not found

re: https://github.com/egoist/tailwindcss-icons/issues/16
Hi, this (^^) issue was closed but I'm getting the same error, running v1.2.0. Was working for a bit after some initial trouble getting transform to work, but then started seeing same error again though I didn't change my config (i.e., sometimes it works, sometimes it doesn't). Even installed the 50MB @iconify/json, and still no luck.
prettier error:
"ERROR" - 10:03:25 PM] Error formatting document.
["ERROR" - 10:03:25 PM] Icon collection "mdi" not found. Please install @iconify-json/mdi or @iconify/json
Error: Icon collection "mdi" not found. Please install @iconify-json/mdi or @iconify/json
    at /Users/<redacted>/node_modules/@egoist/tailwindcss-icons/dist/index.js:96:17
    at Array.reduce (<anonymous>)
    at getIconCollections (/Users/<redacted>/node_modules/@egoist/tailwindcss-icons/dist/index.js:93:22)
    at /Users/<redacted>/tailwind.config.ts:72:100
    at evalModule (/Users/<redacted>/node_modules/jiti/dist/jiti.js:1:255614)
    at jiti (/Users/<redacted>/node_modules/jiti/dist/jiti.js:1:253542)
    at /Users/<redacted>/node_modules/tailwindcss/lib/lib/load-config.js:37:30
    at loadConfig (/Users/<redacted>/node_modules/tailwindcss/lib/lib/load-config.js:39:6)
    at wG (file:///Users/<redacted>/node_modules/prettier-plugin-tailwindcss/dist/index.mjs:2384:38496)
    at oI (file:///Users/<redacted>/node_modules/prettier-plugin-tailwindcss/dist/index.mjs:2384:37795)
["INFO" - 10:03:25 PM] Formatting completed in 65ms.

package.json:
...
  "dependencies": {
    "@aws-sdk/client-s3": "^3.421.0",
    "@aws-sdk/s3-request-presigner": "^3.421.0",
    "@paralleldrive/cuid2": "^2.2.2",
    "@remix-run/express": "^2.0.1",
    "@remix-run/node": "^2.0.1",
    "@remix-run/react": "^2.0.1",
    "class-variance-authority": "^0.7.0",
    "clsx": "^2.0.0",
    "compression": "^1.7.4",
    "cross-env": "^7.0.3",
    "drizzle-orm": "^0.28.6",
    "drizzle-zod": "^0.5.1",
    "express": "^4.18.2",
    "framer-motion": "^10.16.4",
    "image-blob-reduce": "^4.1.0",
    "ioredis": "^5.3.2",
    "isbot": "^3.6.8",
    "morgan": "^1.10.0",
    "pg": "^8.11.3",
    "pino": "^8.15.3",
    "pino-pretty": "^10.2.0",
    "pretty-bytes": "^6.1.1",
    "react": "^18.2.0",
    "react-aria": "^3.28.0",
    "react-aria-components": "^1.0.0-beta.0",
    "react-dom": "^18.2.0",
    "react-stately": "^3.26.0",
    "remix-flat-routes": "^0.5.12",
    "source-map-support": "^0.5.21",
    "tailwind-merge": "^1.14.0",
    "tailwindcss-animate": "^1.0.7",
    "ua-parser-js": "^1.0.36",
    "use-debounce": "^9.0.4",
    "use-places-autocomplete": "^4.0.1",
    "uuid": "^9.0.1",
    "zod": "^3.22.2"
  },
  "devDependencies": {
    "@egoist/tailwindcss-icons": "^1.2.0",
    "@iconify-json/mdi": "^1.1.54",
    "@iconify-json/radix-icons": "^1.1.11",
    "@iconify/json": "^2.2.122",
    "@remix-run/dev": "^2.0.1",
    "@remix-run/eslint-config": "^2.0.1",
    "@types/compression": "^1.7.2",
    "@types/csso": "^5.0.1",
    "@types/express": "^4.17.17",
<EMAIL_ADDRESS>"^3.54.2",
    "@types/image-blob-reduce": "^4.1.2",
    "@types/morgan": "^1.9.4",
    "@types/pg": "^8.10.3",
    "@types/react": "^18.2.20",
    "@types/react-dom": "^18.2.7",
    "@types/source-map-support": "^0.5.6",
    "@types/ua-parser-js": "^0.7.37",
    "@types/uuid": "^9.0.4",
    "@types/web": "^0.0.115",
    "@typescript-eslint/eslint-plugin": "^6.7.3",
    "autoprefixer": "latest",
    "bun-types": "^1.0.3",
    "chokidar": "^3.5.3",
    "csso": "^5.0.5",
    "drizzle-kit": "^0.19.13",
    "eslint": "^8.38.0",
    "eslint-config-prettier": "^9.0.0",
    "prettier": "^3.0.3",
    "prettier-plugin-tailwindcss": "^0.5.4",
    "tailwindcss": "latest",
    "typescript": "^5.1.6"
  },
...

tailwind.config.ts
import { iconsPlugin, getIconCollections } from "@egoist/tailwindcss-icons";

....
  plugins: [
    iconsPlugin({ collections: getIconCollections(["mdi", "radix-icons"]) }),
    require("tailwindcss-animate"),
  ],

prettier.config.cjs
module.exports = {
  plugins: ["prettier-plugin-tailwindcss"],
  tailwindFunctions: ["clsx", "cn"],
};

System Info

macOS Sonoma v14
Node 18
bun package manager (note: tried reinstalling modules using npm, but same result)

P.S. Should have noted that the transform itself is working fine. Issue is it's breaking prettier. Thanks for this btw! I'm coming back to tw from unocss, which has official support for iconify and it would be really sad to go back to including icons the old-fashioned way.

I noticed that the previous fix may only work if the tailwind configuration uses cjs


Allow arbitrary pin assignments

There are many variants of LCD displays using different pin assignments of the PCF8574 chip.
For a list, see e.g. the HD44780 library which lists 4 variants, and I personally own one with yet another.
This adds new constructors and class variables to allow changing the pin layout.
_sendNibble() and _write2Wire are updated accordingly.

Thanks for the contribution


IGVM: Embed firmware in IGVM file and parse OVMF metadata into IGVM parameter block

This PR adds support for embedding firmware, particularly OVMF into an IGVM file using the IGVM builder.
The Makefile has been updated to allow the OVMF file to be specified by the FW_FILE environment variable. This needs to be set to the OVMF.fd file (and not the separate OVMF_CODE.fd and OVMF_VARS.fd files). When provided, the firmware binary is parsed by the IGVM builder during the build to extract metadata which is then populated into the IGVM parameter block, passed to SVSM.
By providing the metadata in the IGVM parameter block, SVSM does not need to parse the OVMF metadata when using an IGVM file. The SVSM has been updated to reflect this by removing the fw_metadata IGVM parameter that specified the OVMF metadata page. In the future, if IGVM is solely used to configure the SVSM then the OVMF metadata parsing can be removed entirely from SVSM.

@msft-jlange, on reflection following our email conversation it is probably best to hardcode the firmware base calculation for QEMU and just return an error at the moment if --firmware is specified for Hyper-V as I don't know how the base address will be calculated in this case. This means I can remove --fwtop and --fwbase.
I've pushed a new version with this change.


Issue Templates are not working

Issue Templates are not working
Fix the Issue Templates Correctly

Shall I work on this ?


Dropchop w/ React.js + Redux

I've been doing some work with React.js and Redux and have been really enjoying it.
In Redux, you are serializing and storing the entire app state into a single object.  This serves well when you have the same data represented in multiple places.  Additionally, there is the concept of Actions that you send through a Reducer which alters the app's state.
I feel like the concepts fit in well with Dropchop.  The actions could apply easily to data operations (eg "Buffer" or "Load from URL").  Storing the entire app state in a single object would make it easy to do things like sync data between the map and layerlist (this may not be a super big win, but could be helpful if you were doing something like toggling visibility or syncing the map's layer order with the layer list's layer order).  Finally, by there are some new features that would be super easy such as storing the app state in localstorage allowing state to persist between sessions or storing state in an observable that can be rewound making for easy undo/redo.
This project hasn't been touch for quite some time.  Any thoughts regarding me taking a stab at refactoring it to fit the architecture?  I've done a quick test with this and it's been going pretty quickly.

Sounds good to me @alukach! Excited to follow along. Want to work off a react branch for now? We may be able to set up react.dropchop.io so you can work off a real domain.

Great, glad to have the support.  If you'd like to follow along (or participate), I've created a branch here: https://github.com/cugos/dropchop/tree/react-redux .  I've basically nuked everything and will be porting features over through PRs.  I'll be branching off of react-redux and creating feature branches prepended with react/, such as https://github.com/cugos/dropchop/tree/react/modal-forms.


Add diagnostic plugins

Added an infrastructure.py file to the postprocessing folder which should operate in the same way as the interface file does for the importers in the io folder. The postprocessor file is currently effectively empty as no postprocessor plugins have been added yet. A line has been added to the init file so that it should act similar to the init file in the io folder.

Hi @dnerini, could you please review this pull request when you have the time? The extended version of the cookiecutter plugin template which can be used to create importer, diagnostic postprocessing, and nowcast plugins is nearly finished but I require the postprocessing interface to be merged to ensure that everything is behaving correctly when tested. Cheers

what's the status of this PR? should we mark it as a draft?


fix(LiveQuery): Session token null when subscribing

When autoSendSessionId: true, it always sets the sessionToken header, even if null. However, when null, it throws an exception due to its unexpected type and breaks live query.
Below is the Flutter output when this happens:
I/flutter (12537): LiveQuery: : ConnectMessage: {op: connect, applicationId: myAppId, masterKey: 123456, sessionToken: null}

I/flutter (12537): LiveQuery: : SubscribeMessage: {op: subscribe, requestId: 1, query: {className: Diet_Plans, where: {objectId: 2pNUgv1CKA}}, sessionToken: null}

I/flutter (12537): LiveQuery: : Listen: {"op":"error","error":"Invalid type: null (expected string)","code":1,"reconnect":true}

Fixes #196

Hey Douglas,
Great work! All looks good. Please can you set this merge request to merge into the branch release/1.0.23. I'll merge it straight away there.
Thanks

Ok, I'll open another PR :)


extension "wolrd copy" function for "Custom worlds" folder

Enhancement Type
Improve an existing feature
Describe the enhancement
Hello,
you have the feature that a worldfolder is copied at startup. (environment variable "WORLD" and "FORCE_WORLD_COPY") I wanted to ask if it is possible that this also works for a "Custom worlds directory path". So that at startup all worlds from a folder are copied into the "Custom worlds" folder.
best regards
EvilBlood92

There's not yet an option to copy multiple world directories since most of the time you can only run one. The exception I know is for multiverse, so maybe this we'll help https://docker-minecraft-server.readthedocs.io/en/latest/misc/world-data/#custom-worlds-directory-path

The "https://docker-minecraft-server.readthedocs.io/en/latest/misc/world-data/#custom-worlds-directory-path" works i am just looking for a solution to copy multiple worlds when starting the server.

Can you clarify why you're needing multiple worlds copied?


Internal: Added missing Test-FunctionInterrupt calls

There are some commands, that use Stop-Command in the begin block but don't test for the function interrupt so that the process block will run even it should not.
Also changed some break into return or added some missing return.
Also corrected some Stop-Command calls.

🤩  so good!


Wrong bottom navigation icon is highlighted

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.2-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @allgandalf
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1719437029378469
Action Performed:

Open app
Go to search tab
Click on any existing report
Click on the subtitle
Press back button
Press back button again

Expected Result:
User should be on the search page
Actual Result:
the user is on the Chat page, but in bottom navigation, search tab is highlighted
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/86741512-a789-456f-a2b0-eea68fc2f080
https://github.com/Expensify/App/assets/38435837/57b5776f-7369-481c-861f-c6796f4abdc7

View all open jobs on GitHub

@WojtekBoman @adamgrzybowski this seems to be another task good for you

@mountiny I have pending PR for this bug. But it seems to have some problems after merging the newest main. I will investigate it after other high priority issues

@adamgrzybowski how's it going?

@puneetlath old issues were resolved but the reviewer found one more problem. I will investigate it.

@allgandalf from what I understand this issue is a duplicate of this issue: https://github.com/Expensify/App/pull/43998
and the 2nd one is being actively fixed

@allgandalf 43998 is on staging now. I'm going to ask for a re-test.

Not reproduced.
https://github.com/user-attachments/assets/c8bc39f3-0d42-4890-abf0-af24bdf7c83d

Great, thanks everyone!


Fix crash in ReshapeOp inference when an input to the previous concat is empty

The problem was that the check:
if ((getShape(vType)[0] == 1) && getElementType(vType).isSignlessInteger(64))
    return true;

can succeed even if areDims(Value val) is not true for all operands of Concat.
I think this should be upstreamed

Thanks!


Have you tried Lettuce library to build your rebloom client?

Lettuce is yet another redis client library beside jedis.
Lettuce is different by supporting custom commands for redis4.0 modules.
so your don't really need hardcode your jrebloom library.

Hi, we deliberately provide a self-contained Java client with a minimal API. We do the same for Python and go. Redis is a simple protocol, so pretty much anything would work. Perhaps there might be some benefit in adding ourselves to lettuce, @dvirsky ?


Add new TB Example for Dynamic ROM Creation

adding data extraction for a new example under construction

@SushobhitGupt If we want this example to be available for internal testing, we could just let the branch being and not merge it in main yet.
Then, to install this "beta" version,  a user could do:
pip install https://github.com/pyansys/pyaedt.git@main_sgupt
This is the best practice and usual way to do it when someone wants to introduce a beta feature.
Here it is very specific as it is an example.

@SushobhitGupt If we want this example to be available for internal testing, we could just let the branch being and not merge it in main yet. Then, to install this "beta" version, a user could do: pip install https://github.com/pyansys/pyaedt.git@main_sgupt
This is the best practice and usual way to do it when someone wants to introduce a beta feature. Here it is very specific as it is an example.

@MaxJPRey Thanks for looking at it, one of the challenges of keeping items on personal branches is that the visibility and access to functionality gets limited. Other than a few folks, not many others have enough git knowledge to effectively use it.
We are hoping to use this and other examples I plan to add to market the ease of automation using pyaedt in our release material even during the Beta period.
I was hoping to point my ace and customers to examples on the web site while they play with our daily / beta builds. On the product side in AEDT, we generally merge beta functionality and just put a reasonable disclaimer on it. Let me know what you think?

LGTM.


Template export should allow keeping the existing metadata.name

Proposal:
When creating a template using the export command it should optionally be possible to keep the exisiting metadata.name
Current behavior:
Every time export is used, new random names are generated. This causes resources to be destroyed if the template is then re-applied. It also changes the order of the resources in the yaml file, which makes it hard to track the changes in version control.
Desired behavior:
The export command has a flag that tells it to use the metadata name from the existing resources.
Alternatives considered:
Using a local script to change the names after running the export command.
Use case:
This would enable a no-code+infrastucture-as-code setup, where a user can make changes in the front end, and have those changes be automatically applied to a code base, that can then be used to deploy the changes idempotently to the same environment or to a new enviroment.
It would also allow templates to be modified using the UI only and exporting again, as only the changes made by the user would be changed in the export (and not the names).

This will require an API change before we can add support in the CLI.

I think the way to do this is with a stack.
If you maintain a set of resources as a stack with a stack id, then they will be exported with the same meta.names each time.
A stack is created automatically when you apply a template, or you can create one manually. See these docs for more details: https://docs.influxdata.com/influxdb/cloud/influxdb-templates/stacks/

Thanks @russorat!
This works. And it's actually documented in the CLI docs https://docs.influxdata.com/influxdb/v2.0/reference/cli/influx/export/stack/, but I was reading the guide: https://docs.influxdata.com/influxdb/v2.0/influxdb-templates/create/


Replace "finish_reason" with a note

Putting the finish_reason in chat is not user-friendly. Change it to a note with explanation.

According to https://platform.openai.com/docs/guides/gpt/chat-completions-response-format, "finish_reason" can be one of the following five values:

stop: API returned complete message, or a message terminated by one of the stop sequences provided via the stop parameter
length: Incomplete model output due to max_tokens parameter or token limit
function_call: The model decided to call a function
content_filter: Omitted content due to a flag from our content filters
null: API response still in progress or incomplete

As users do not need to know or can do nothing with stop and null, they are not shown in a note. The rest three will be appended to a response as below:
User: Jinglei Ren<EMAIL_ADDRESS>Date: Fri Jul 21 18:28:33 2023 +0800

```command
{"name": "get_current_weather", "arguments": {"location": "Boston, MA"}}
```

Note: (finish_reason: function_call) The model decided to call a function.

prompt ae9135848bbd332387a23e04671dcd293e7ba383677056e6ddecc40c9b06db1b

The note line starts with Note:  followed by (finish_reason: ...). The reason shown here can be length, function_call or content_filter. There is no note for stop or null.


Refactored nullable types to use the BlockExt trait

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.79%. Comparing base (e90c4ce) to head (68b6070).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #676      +/-   ##
==========================================
- Coverage   91.80%   91.79%   -0.01%
==========================================
  Files         115      115
  Lines       34322    34290      -32
==========================================
- Hits        31509    31477      -32
  Misses       2813     2813

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix removing variants

This PR fixes;

Deleting Variants in Cloud: This was caused by a missing / in the endpoint url. It works without using Localhost but not in Cloud due to Server Configuration.

Pydantic Import Error in the CLI. PydanticImportError: BaseSettings has been moved to the pydantic-settings package. See https://docs.pydantic.dev/2.0.2/migration/#basesettings-has-moved-to-pydantic-settings for more details. This was caused by the upgrade to Pydantic v2. In pydantic-V2, Settings management has been moved to a separate package named pydantic-settings.
The previous way was;
from pydantic import BaseSettings

The new way was to do;
from pydantic_settings import BaseSettings

However, installing pydantic-settings failed because of dependency mismatch. Hence the best solution I found to use was to import BaseSettings from pydantic.v1
from pydantic.v1 import BaseSettings

@devgenix can you create this from agenta's remote not from your remote?

@devgenix can you create this from agenta's remote not from your remote?

@aakrem sorry, I don't quite understand.


⚠ Bofrost.fr has degraded performance

In c659ffc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1199 ms

Resolved: Bofrost.fr performance has improved in 8e39fda.


chore: LEAP-35: remove default arg from conditional_atomic

PR fulfills these requirements

[ ] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[ ] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[ ] Backend (API)
[ ] Frontend

Describe the reason for change
(link to issue, supportive screenshots etc.)
User should specify the predicate to be used when they want to conditionally make a transaction atomic. The only remaining usage of the default value will be removed in LSE#4584
What does this fix?
(if this is a bug fix)
What is the new behavior?
(if this is a breaking or feature change)
What is the current behavior?
(if this is a breaking or feature change)
What libraries were added/updated?
(list all with version changes)
Does this change affect performance?
(if so describe the impacts positive or negative)
Does this change affect security?
(if so describe the impacts positive or negative)
What alternative approaches were there?
(briefly list any if applicable)
What feature flags were used to cover this change?
(briefly list any if applicable)
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
(for bug fixes/features, be as precise as possible. ex. Authentication, Annotation History, Review Stream etc.)

/git merge develop


Use DotnetToolDispatcher for dispatching project dependency command in dotnet-ef

https://github.com/aspnet/Coherence-Signed/issues/276

Fixed in #5185


Adding a new profile: Error: unknown profile name

Describe the bug
Using the last version (binary or compiled), I can no longer add a new profile.
To Reproduce
With no profile named test
$ sudo hostctl add domains test test.com
Error: unknown profile name

Expected behavior
Domain test.com is added in a newly created test profile
System (please complete the following information):

OS: Linux
Version:

$ hostctl --version
hostctl version 1.0.5

I'll look into it, it fails with add domains and new profiles, if profile exists it works.

fixed on v1.0.6, not actually a bug, but added anyway.


No Output Screen

face_recognition version:
Python version:
Operating System:

Description
The webcam light comes on but the output screen where we can see the process running does not show up

Its a cv2 issue btw. But i think you forgot the line 'cv2.imshow(filename, frame) '.


policyDefinitions valid characters denylist contains periods even though resources can be created containing periods in Azure

The CanadaPubSecALZ repository creates many resources containing periods
https://github.com/Azure/CanadaPubSecALZ/blob/9fe82b8b4d8fe02ac073b81e808f2a16cbf81a08/policy/custom/definitions/policyset/LogAnalytics.bicep#L393

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

ID: 374ec8eb-64d4-3766-5c5e-39137a262ffd
Version Independent ID: 3abeeae9-6854-651a-483c-a1673eea9f9f
Content: Resource naming restrictions - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resource-name-rules.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @mumian
Microsoft Alias: tomfitz

@TeamDman Thanks for your feedback! We will investigate and update as appropriate.

@TeamDman We have verified and updated the document as required. The update will go live in less than 24 hours. Thanks again!


Drag and Drop with ng-click

I can't use ng-click if drag and drop are enabled.

ng-click="vm.filterChange(node.name) doesn't work

I have  the same issue,
Separated it into diff buttons like a preview to show  a page on click


The trainning error

dear sir
when I trainning the model ,i meet the error. as follows. @zhulf0804 @qsisi

can you know how to solve it?
Thanks

Looks like something goes wrong with the variable coors. Maybe you could consider printing it out for further debugging.

inds = torch.norm(coords_src[coors[:, 0]].type(torch.long) - coords_tgt[coors[:, 1]].type(torch.long), dim=1) < (self.pos_radius - 0.001)
IndexError: tensors used as indices must be long, byte or bool tensors


Update Composer dependencies (2021-05-20-00-12)

Visual regression test passed!
View the visual regression test report


improve "seg7_counter.py" and add a project

and add 10.  Thank you

Thank You @arcostasi  and  please Edit the seg7_counter.py in your project Wokwi refers to LINK by new changes about last merge #1


Add another array of commits

To better follow http://keepachangelog.com would it be possible to add another filtered group of commits to merges, commits, fixes? maybe changes? and therefore would need a changes regex pattern --changes-pattern. Useful if you follow a defined commit pattern that is easy to parse out, like the AngularJS one thats referenced everywhere.
Therefore our templates can properly output:
Added
Changed
Fixed
Right now we have to combine Added and Changed together, since all commits not matching the issues-pattern are lumped together in commits.

I have thought about this, but the problem is that there isn't really a solution that avoids becoming too opinionated about what your commits should look like. The point of the library is to make it extremely easy to generate a changelog for all different types of repos, with very few absolute requirements. If we were to support both Added/Changed and the current output depending on your commit patterns, the logic/templating would become quite complex.
One possible solution would be to support custom patterns in the options objects:
{
  "name": "my-awesome-package",
  "auto-changelog": {
    "commitPatterns": {
      "added": "feat\\(.+\\)",
      "changed": "chore\\(.+\\)"
    }
  }
}

And then these are accessible in custom templates and can be used like:
# Changelog
All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).

Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).

{{#each releases}}
  {{#if href}}
    ## [{{title}}]({{href}}){{#if tag}} - {{isoDate}}{{/if}}
  {{else}}
    ## {{title}}{{#if tag}} - {{isoDate}}{{/if}}
  {{/if}}
  {{#if merges}}
    ### Merged
    {{#each merges}}
      - {{message}} {{#if href}}[`#{{id}}`]({{href}}){{/if}}
    {{/each}}

  {{/if}}
  {{#if fixes}}
    ### Fixed
    {{#each fixes}}
      - {{commit.subject}}{{#each fixes}} {{#if href}}[`#{{id}}`]({{href}}){{/if}}{{/each}}
    {{/each}}

  {{/if}}
  {{#if added}}
    ### Added
    {{#each added}}
      - {{subject}} {{#if href}}[`{{shorthash}}`]({{href}}){{/if}}
    {{/each}}

  {{/if}}
  {{#if changed}}
    ### Changed
    {{#each changed}}
      - {{subject}} {{#if href}}[`{{shorthash}}`]({{href}}){{/if}}
    {{/each}}

  {{/if}}
  {{#if commits}}
    ### Other Commits
    {{#each commits}}
      - {{subject}} {{#if href}}[`{{shorthash}}`]({{href}}){{/if}}
    {{/each}}

  {{/if}}
{{/each}}

This would be pretty non-invasive to the codebase, and let the user define the patterns they use and how to format them in the log. Thoughts?

that would be amazing. definitely should be optional. but that flexibility would allow people to adopt their own custom commit patterns and changelog outputs if they chose to.

Possibly easier would be a handlebars helper that can filter commits directly in custom templates:
### Added
{{#filter-commits commits pattern="feat\(.+\)"}}
  - {{subject}} [`{{shorthash}}`]({{href}})
{{/filtered-commits}}

### Changed
{{#filter-commits commits pattern="chore\(.+\)"}}
  - {{subject}} [`{{shorthash}}`]({{href}})
{{/filtered-commits}}

yes we actually did try that using a helper, but then we see empty ###Added lines if there were no features in that release. so thought it may be better to do it before we hit the template.

yeah i guess would just mean adding some helpers to your template.js file? though agree lots of logic added to the actual template.

What about a single commit-list helper that encompasses all of the logic for the heading and the filtering:
{{#commit-list commits heading="### Added" pattern="feat\(.+\)"}}
  - {{subject}} [`{{shorthash}}`]({{href}})
{{/commit-list}}

{{#commit-list commits heading="### Changed" pattern="chore\(.+\)"}}
  - {{subject}} [`{{shorthash}}`]({{href}})
{{/commit-list}}

much cleaner for sure. and likely does everything we were hoping to accomplish.

On Mar 20, 2018, at 12:53 AM, Pete Cook<EMAIL_ADDRESS>wrote:
What about a single commit-list helper that encompasses all of the logic for the heading and the filtering:
{{#commit-list commits heading="### Added" pattern="feat(.+)"}}

{{subject}} {{shorthash}}
{{/commit-list}}

{{#commit-list commits heading="### Changed" pattern="chore(.+)"}}

{{subject}} {{shorthash}}
{{/commit-list}}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/CookPete/auto-changelog/issues/36#issuecomment-374506033, or mute the thread https://github.com/notifications/unsubscribe-auth/AG5Usli9BDp8VL6P5mTIBtSIZKWcv75Fks5tgLWPgaJpZM4SvoCj.

will this be released to npm?


I think maybe they patched something because this doesn't work

Everything installs and compiles fine but it results in communication error when you test a uri.

Just checked it, processed a url without problems.
Please provide a step-by-step instruction on how to reproduce the error. Also check the dev console if there are any error details on the network request.

You're right, I can't replicate it today. Maybe there was a temporary API outage or something from them or network issue on my side. Nice tool btw and thanks for the fast reply!


Add equality operators to cxx::variant and cxx::expected

Brief feature description
Add the equality operators operator==() and operator!=() to cxx::variant and cxx::expected outside of the class to follow AUTOSAR rule A13-5-5.

Neither expected nor variant have implemented a comparison operator so why add them now? What kind of purpose/feature would we cover with it?


chanrestore: return error instead of nil if failed derivation

Fixes the panic in #3180

Correct that an error here wold be unexpected. If something is popping up however, then we'll be actually able to pin point the issue now that the error is actually being returned.


feat: add class when collection name is hovered during drag and drop

Resolves #849
Description
During drag and drop, the target collection has a isOver property provided by the collect function
https://github.com/usebruno/bruno/blob/c46fbfbde39b025c5e1692477508c2162b65f4aa/packages/bruno-app/src/components/Sidebar/Collections/Collection/CollectionItem/index.js#L60-L62
We use that to add a new class to the collection item and that class has the same property as the existing :hover
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Publishing to New Package Managers
Please see here for more information.
https://github.com/usebruno/bruno/assets/674667/4ff5bd47-fdcb-4ebd-abf7-bef91477b653

The highlighting works now but I'd want to see the request inside the new destination folder instead of being hidden

The highlighting works now but I'd want to see the request inside the new destination folder instead of being hidden

On my previous example, the destination folder was empty but here is what it looks like when there is something in there. Is that what you are looking for?
https://github.com/usebruno/bruno/assets/674667/ab23a0bc-c6ef-43fa-897c-b1723204c7c2

Ah I see. Two things:

It would be better to show the request in its new folder, even if it was previously empty, no?

Highlighting the destination folder AND its existing contents is better feedback than highlighting every list item (see VSCode example in linked issue)

@glokta1 Agree with you. We'll keep https://github.com/usebruno/bruno/issues/849 open and improve it.

@glokta1 to clarify what you are expecting:

It would be better to show the request in its new folder, even if it was previously empty, no?

Here you would like the dragged request to be displayed inside a target folder before it is actually dropped there if I am getting it right?

No, after. See the original vid you posted where after dragging the request into the target folder, I don't actually see my request in the folder unless it already has existing requests (your 2nd vid)

@glokta1 actually in my first video, the target folder was empty and collapsed, but if the folder was expended, it would display as expected :
https://github.com/usebruno/bruno/assets/674667/2b06b316-c5b0-430f-9f37-48ee9a33ffbd


Remove client.DeleteAllOf() from servicemonitor cleanups [K8SSAND-1149]

What this PR does:
Uses a client.Delete() call instead of a client.DeleteAllOf() in the PrometheusResourcer.CleanupResources() method.
Which issue(s) this PR fixes:
Fixes #279
Checklist

[ ] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CHANGELOG.md updated (not required for documentation PRs)
[ ] CLA Signed:  DataStax CLA

CreateMultiStargateAndDatacenter is intermittently failing it seems. It passes locally but has failed in the previous test run. Restarting the workflow to see if we can get it to pass for hygiene purposes.

All tests passing (except ClusterScoped/MultiDcMultiCluster which is broken in all branches right now) and ready for review.


Add a missing semicolon

Hi
I added a missing semicolon in Hash page.

Thanks :+1:

It's my presure :)


Strange errors in Ubuntu 20.04 LTS

Hi. First sry English isnt my native language.
I always use Your script in Ubuntu 18.04.4 and there it works with no problems.
I just fresh installed Ubuntu 20.04 and have little issue.
drivesync works ( syncing well ) but here always many errors:
ruby drivesync.rb  /var/lib/gems/2.7.0/gems/mime-types-3.1/lib/mime/types/logger.rb:28: warning: _1' is reserved for numbered parameter; consider another name
/var/lib/gems/2.7.0/gems/mime-types-3.1/lib/mime/types/logger.rb:28: warning: _2' is reserved for numbered parameter; consider another name /var/lib/gems/2.7.0/gems/mime-types-3.1/lib/mime/types/logger.rb:28: warning: _3' is reserved for numbered parameter; consider another name
/var/lib/gems/2.7.0/gems/faraday-0.15.2/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use &block instead
/var/lib/gems/2.7.0/gems/faraday-0.15.2/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use &block instead
/var/lib/gems/2.7.0/gems/faraday-0.15.2/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use &block instead
/var/lib/gems/2.7.0/gems/faraday-0.15.2/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use &block instead
/var/lib/gems/2.7.0/gems/faraday-0.15.2/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use &block instead
/var/lib/gems/2.7.0/gems/multi_json-1.13.1/lib/multi_json/options_cache.rb:12: warning: Capturing the given block using Proc.new is deprecated; use &block instead
Local folder is 1 files behind and 0 files ahead of remote
Starting sync at 2020-04-27 18:01:48 +0200
Downloading file baza/Hasła.kdbx ...
Sync complete.
drivesync.rb:11: warning: calling URI.open via Kernel#open is deprecated, call URI.open directly or use URI#open
Here linkt to pastebin
https://pastebin.com/gGijwruz
Cheers

@SwiatLinuksa I see two warnings (not errors) here: One is coming from my code, the other from mime-types, one of the Google API's dependencies. The former will be fixed in the next release of DriveSync, the latter I can't fix myself, unfortunately. It might go away with the next Google API update, though. I'll look into the update.
The warnings should not have any impact on the functionality itself, though.


Refining the existing AWS AMI generation configuration files

Description:
Need to refine and integrate best practices to the existing AWS AMI generation configuration files. Plus, need to generate the AWS AMIs.

This is fixed in https://github.com/wso2/cloudformation-apim/pull/17.


Added a note to IsDebug of Analytics Manager

Created a lot of confusion for me at first as I didn't see any activity on the console. So decided to add this comment for information.

LGTM I am sure that IsDebug is causing a lot of confusion.   I am going to try and work out a better way of returning messages to the developer when IsDebug is set to true, rather then just doing nothing.


Remove kubectl from dockerfile prereqs since it pulls it

Looking at https://github.com/istio/istio/issues/13219
Removes the prerequisite for a local kubectl as it's not used. The issue shows a failure to copy a local kubectl in preparation for building the Docker image. The issue in this case is the local file is a Mac binary, so wouldn't work in the Docker image. The Docker image downloads the kubectl as part of it's build so doesn't need the prerequisite file.

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

Needs approval from an approver in each of these files:

OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
0076147b713ea0a8f1810347575d0074a34d5d29
link
/test istio-pilot-multicluster-e2e

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

/assign @costinm

fixes #13219

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
0076147b713ea0a8f1810347575d0074a34d5d29
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
0076147b713ea0a8f1810347575d0074a34d5d29
link
/test istio-integ-k8s-tests

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

/lgtm
thanks for this PR @ericvn, this solved the problem.

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

Needs approval from an approver in each of these files:

OWNERS

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

@tvieira: changing LGTM is restricted to assignees, and only istio/istio repo collaborators may be assigned issues.

In response to this:

/lgtm
thanks for this PR @ericvn, this solved the problem.

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ericvn, linsun, tvieira
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [linsun]

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


trigger all for each event

While I like this feature, I think it adds a lot of complexity just for convenience sake.  I'll take another shot at tightening it up later.

What exactly is this supposed to do?

With the current implementation, the all event will get triggered only once with all the events.
model.bind('all', function(event) {
  console.log(event); // 'a b'
})
.trigger('a b');

I think the desired behavior would be for trigger('a b') to behave just like trigger('a').trigger('b').
model.bind('all', function(event) {
  console.log(event); // 'a', then 'b'
})
.trigger('a b');

Otherwise, checking events during an all event would be much more tedious and likely require a regex.

Just to be clear, the feature I referred to above is #640, binding multiple events.


Ignore UTF-8 BOM from the xml input

This PR is to support the SVD input that contains UTF-8 BOM. (Ignore BOM)
Please refer to the svd2rust's PR.

Thanks for your advice!
I fixed 31e09842.

Thank you!
merging without CI as this is still wip


Improve an explanation of config [logs] section usage

Hello! If you don't mind I added your comment https://github.com/corvus-ch/rabbitmq-cli-consumer/issues/21#issuecomment-367781289 to config. It should be more clear now how to make logs optional.

Thanks for the help.


fix(action): add shell

:tada: This PR is included in version 1.0.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update user-provided service tags

This PR includes changes to add the "-t" flag for update-service. We've also fixed that command so users can update instance tags and/or arbitrary params without supplying an updated plan. Some cleanup of translated strings is included as well.
https://www.pivotaltracker.com/story/show/72117050

Thanks a bunch for fixing this. Merged.


Cannot read property 'getWindowSize' of null

Output of bower -v && npm -v && node -v:
1.8.0
3.10.9
v7.1.0

Additional environment details (proxy, private registry, etc.):
Win10 14393 x64 with node windows installer
Steps to reproduce the issue:
1.bower init
2.
3.
Describe the results you received:
tty.js:76
var err = this._handle.getWindowSize(winSize);
^
TypeError: Cannot read property 'getWindowSize' of null
at WriteStream._refreshSize (tty.js:76:25)
at process.on (internal/process/stdio.js:16:43)
at emitNone (events.js:91:20)
at process.emit (events.js:185:7)
at Signal.wrap.onsignal (internal/process.js:200:44)
Describe the results you expected:
in node 6.x bower init works well.
Additional information:

Same setup,  same issue occurs when resizing powershell.
Also Bower init does not complete successfully or unsuccessfully.
Steps to reproduce the issue:
1.bower init in powershell
2.command does not complete
3.resize window
updating to node 7.2.0 seems to have fixed the issue.

That was a regression in node 7.1.0 on windows.
See https://github.com/nodejs/node/issues/9542

Any windows users that can test this please.


Support consuming wado-rs endpoint with cross-origin resource sharing (CORS)

User story
As a server consumer, I want the DICOMweb server to support consuming its API with CORS so that web viewers hosted outside the server domain can consume the server endpoint.
Currently, when calling the wado-rs endpoint to retrieve a frame from an SPA client (OHIF Viewer). The browser fails with a CORS error:
Access to XMLHttpRequest at 'https://xxxxxxxx.azurewebsites.net/v1/studies/1.3.../series/1.3.../instances/1.3..../frames/1' from origin 'http://localhost:3000' has been blocked by CORS policy: Request header field accept is not allowed by Access-Control-Allow-Headers in preflight response.
Note this is "Access-Control-Allow-Headers" which I don't believe can be configured from the Azure portal App Service CORS blade, hence may require support this from the server.
Right now, it is not possible to run a viewer from "localhost" due to this:

Acceptance criteria

When I run an SPA viewer from a different domain than the server, then the server would support consuming the WADO-RS frame endpoint.
When running the same request with chrome flag: disable-web-security it works as expected and open the image

Here is an example of how this can be enabled in the server:
https://github.com/DICOMcloud/DICOMcloud/blob/development/DICOMcloud.Wado.WebApi/Types/CorsPolicyFactory.cs
https://github.com/DICOMcloud/DICOMcloud/blob/e7b32a98567d1e718b615ab5004edde49079964d/DICOMcloud.Wado.WebApi/App_Start/WebApiConfig.cs#L23

Probably we need to document how to enable CORS. Here is how you can do it.
Edit this file https://github.com/microsoft/dicom-server/blob/main/src/Microsoft.Health.Dicom.Web/appsettings.json and add the CORS settings there like below and run the service.

"Cors": {
"Origins": [""],
"Methods": [""],
"Headers": ["*"],
"MaxAge": 1440
}

@smithago, I've been using the ARM template to deploy to azure so I didn't have a local dev environment.
First, I tried updating this from the Azure portal config but didn't seem to make a difference.
DicomServer__Cors__Headers      *
DicomServer__Cors__Methods     *
DicomServer__Cors__Origins        *
I then cloned the repo and looked at the project source code, but can't find how the cors configuration "CorsConfiguration" is being used in the project. The only reference I see for cors is enabling a default policy in "DicomServerServiceCollectionExtensions":
app.UseCors(CorsConstants.DefaultCorsPolicy);
I am probably missing something but I wanted before creating a local dev environment and deploy that from my machine to quickly check with you on these observations and how cors is enabled.
Here is my config in the Azure portal:


examples_test.py

Similar to the notebooks we don't run our examples in the CI.

Close by deleting .py example files


Add ability to change text style for flag button

I need to change the text style of the flag button. But there are no mechanisms to do this.

I know that it is been a very (very) long time that you had posted this issue, has we are willing to keep improving the package, can you provide more information about what you wanted to do, because the flag button as you say, isn't actually a button, it's an Inkwell composed of the dropdown (Icon) + the flag itself (Image) + the country dial code (Text). The only items that you can style are the dropdown (with the new dropDownIcon parameter) and the country dial code (you cannot explicitly style it, but a global styling can, I though, do it).

Issue closed due to a lack of activity.


Login Items pref pane shows "Sam Schott" instead of "Maestral"

Describe the bug
The Login Items settings menu in macOS Ventura (13.1) shows the developer's name instead of the app name. This led to a brief search of "who tf is sam schott and why are they h4ck1ng me?!?!"

To Reproduce
With Maestral installed, open System Settings, General, Login Items
Expected behaviour
The entry for Maestral should be titled "Maestral"
System:

Maestral version: 1.6.4
Python version: above command failed with "No module named PyGt5"
OS: macOS Ventura 13.1

Additional context
No, I'm not sure why ln is a login item on my system but I'm guessing it's either Adobe's or Oracle's fault... The Java installer is especially persistent in adding login items - that garbage is getting removed as soon as I fix this client's legacy app...

Ah, damn. It appears that macOS Ventura introduced changes to how Launch Agents should be registered. While the legacy method of just create a plist file in ~/Library/Launch Agents/ still works, System Settings don't correctly pick up the associated app in this case.
There appear to be two ways to mitigate this:

Use the new method of registering launch agents through SMAppService.register with the plist stored in the app bundle itself. This only works for macOS Ventura and app bundles, meaning special handling of previous releases and pip installs.
Set the AssociatedBundleIdentifiers key in the legacy plist to match the associated app. This is also new in Ventura but backward compatible.

I'm tending towards the second option since it is simpler to implement.


[improve][cli] Better relative path arguments

So that the default value on the reference page is clear without a machine relative prefix.
Before:

After:

Documentation

[ ] doc
[ ] doc-required
[ ] doc-not-needed
[x] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #18740 (dd3d129) into master (3e18974) will decrease coverage by 12.02%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #18740       +/-   ##
=============================================
- Coverage     45.53%   33.51%   -12.03%
+ Complexity    10220     6308     -3912
=============================================
  Files           703      623       -80
  Lines         68797    59062     -9735
  Branches       7374     6140     -1234
=============================================
- Hits          31329    19794    -11535
- Misses        33881    36593     +2712
+ Partials       3587     2675      -912

Flag
Coverage Δ

unittests
33.51% <0.00%> (-12.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.


pst.write() iterates through n characters (rather than n paths) if pst.model_command value is changed.

Maybe cast this as a list rather than a string.

I've messed this up too - just added an overload of setattr to Pst that traps for a str instead of a list.  Seems to be working.

Oops! So it was my fault for over-writing a list with string. Duh.

There can be multiple model commands (the numcom var in the control data section)  but I've messed up this up the same way several times so it needed to be fixed anyway.


KeyError: 'RANK' in os.environ["RANK"]

$ python
Python 3.8.5 (default, Sep  4 2020, 07:30:14)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> os.environ["RANK"]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/anaconda/envs/azureml_py38/lib/python3.8/os.py", line 675, in __getitem__
    raise KeyError(key) from None
KeyError: 'RANK'
>>>

I have a machine with 8 GPUs and 24 CPUs. Why does RANK key doesn't exist? I am following your tutorial here:
https://theaisummer.com/distributed-training-pytorch/

and I wanted to do distributed training.
![Screenshot from 2022-12-13 10-04-40](https://user-images.githubusercontent.com/1892917/207369120-fa7ff30e-1c92-4013-8f96-17c81e2b01d4.png)

Hi!
Have you launched the script with torchrun or torch.distributed.launch?
for pytorch < 1.9
python -m torch.distributed.launch --nproc_per_node=4 main_script.py

or  for pytorch > 1.9
torchrun --nproc_per_node=4 main_script.py


HADOOP-18724. followup remove deprecation on new methods

Somehow @Deprecated crept in.
How was this patch tested?
letting yetus do it
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 33s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 56s

trunk passed

+1 :green_heart:
compile
15m 34s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
14m 25s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m 12s

trunk passed

+1 :green_heart:
mvnsite
1m 37s

trunk passed

+1 :green_heart:
javadoc
1m 12s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 48s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
2m 39s

trunk passed

+1 :green_heart:
shadedclient
22m 21s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 50s

the patch passed

+1 :green_heart:
compile
14m 54s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
14m 54s

the patch passed

+1 :green_heart:
compile
14m 56s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
14m 56s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  5s

the patch passed

+1 :green_heart:
mvnsite
1m 33s

the patch passed

+1 :green_heart:
javadoc
1m  4s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 51s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
2m 35s

the patch passed

+1 :green_heart:
shadedclient
22m 17s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 36s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
1m  2s

The patch does not generate ASF License warnings.

174m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5650/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5650

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux efbcd47ecfdf 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / cdc77dea1bbd70d3456f759cfdad491be2f39a33

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5650/1/testReport/

Max. process+thread count
1263 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5650/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Latex error converting to dvi.

'latex' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
File "C:\Manim\manimlib\extract_scene.py", line 150, in main
scene = SceneClass(**scene_kwargs)
File "C:\Manim\manimlib\scene\scene.py", line 52, in init
self.construct()
File "example_scenes.py", line 213, in construct
tex_to_color_map={"text": YELLOW}
File "C:\Manim\manimlib\mobject\svg\tex_mobject.py", line 144, in init
self, self.arg_separator.join(tex_strings), **kwargs
File "C:\Manim\manimlib\mobject\svg\tex_mobject.py", line 44, in init
self.template_tex_file_body
File "C:\Manim\manimlib\utils\tex_file_writing.py", line 19, in tex_to_svg_file
dvi_file = tex_to_dvi(tex_file)
File "C:\Manim\manimlib\utils\tex_file_writing.py", line 67, in tex_to_dvi
"See log output above or the log file: %s" % log_file)
Exception: Latex error converting to dvi. See log output above or the log file: .\files\Tex\34b95eca9f7c5b73.log
Hello everyone, I know this is an old problem, but all the solutions mentioned before doesn't seem to work for me ( I am using Windows 10, the newest version of python3.7, miktex, FFmpeg, Cairo, and Sox).
I was trying to run the "WriteStuff" code in "example_scenes.py" when this happened.
If Latex not being recognized is the problem, how can i change it? ( I'm new, so if you can post a step by step solution i would be very greatful)

Full MikTeX?

Yes full MikTex, here are some photos that might help with the problem

Yes full MikTeX, here are some photos that might help

there should be another folder x64 under the bin directory, where the latex.exe resides. add that folder instead.

@Kolloom
I will check it during the night, internet would be faster to download these things, Thanks!

So my problem was that miktex setup was missing alot of files,i dont loaded the portable edition and it did the trick, so now latex was identified by cmd. Now this was not enough since the error converting to dvi was still there, so i copied the code from issue #36 and everything worked.
@Kolloom @Elteoremadebeethoven  Thanks guys

So my problem was that miktex setup was missing alot of files,i downoaded the portable edition and it did the trick, so now latex is identified by cmd. Now this was not enough since the error converting to dvi was still there, so i copied the code from issue #36 and everything worked.
@Kolloom @Elteoremadebeethoven Thanks guys

I think I have the same issue. Many files are missing. Please tell me how you installed those


Make TransactionHistory searchable in parallel

implemented. https://github.com/atonoy/solana-suite/pull/41


Removed unused test-pr path filter

#980  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite

Merge activity

May 14, 2:50 PM EDT: @CatChen started a stack merge that includes this pull request via Graphite.


it's overriding window.self variable

It's a nice plugin.
Found one critical issue. It's overriding the window.self variable, and thus my other js libraries (or any other plugin which depends on window.self object) are not working correctly.
Please check line no. 36 in jquery.letterpic.js.
It should be "var self = this; "

@sasawant, thanks a lot for noticing this typo! I fixed it.

Thanks a lot :-)
On Fri, Jul 14, 2017 at 12:10 AM, George Yakovlev<EMAIL_ADDRESS>wrote:

@sasawant https://github.com/sasawant, thanks a lot for noticing this
typo! I fixed it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yakovlevga/letterpic/issues/2#issuecomment-315166191,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APbNF8kpFtl-5uR1GlWLx3CXIDU8Nm8fks5sNmSvgaJpZM4OWtXo
.


[material-ui][Table] slots and slotProps documented properties are not exposed in package

Steps to reproduce
Link to live example: (required)
Steps:

install<EMAIL_ADDRESS>read the TablePagination API documentation for this version and note that slots and slotProps props have a lot of properties e.g. nextButtonIcon

use the TablePagination component in JSX or override style with theme.components.MuiTablePagination.defaultProps properties
note that only actions or select properties are available

Current behavior
only actions or select properties are available on slots and slotProps props (on TablePagination component)
Expected behavior
nextButtonIcon or previousButtonIcon properties are available on slots and slotProps props (on TablePagination component) as mentioned in the web documentation
Context
e.g. override theme.components.MuiTablePagination.defaultProps.slots.nextButtonIcon
Your environment

npx @mui/envinfo
    System:
    OS: Linux 5.10 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
    pnpm: Not Found
  Browsers:
    Chrome: 120.0.6099.129
  npmPackages:
    @emotion/react: ^11.11.3 => 11.11.3
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.36
    @mui/core-downloads-tracker:  5.15.10
    @mui/material: ^5.15.10 => 5.15.10
    @mui/private-theming:  5.15.9
    @mui/styled-engine:  5.15.9
    @mui/system:  5.15.9
    @mui/types:  7.2.13
    @mui/utils:  5.15.9
    @types/react: ^18.2.48 => 18.2.48
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    typescript: ^5.3.3 => 5.3.3

Search keywords: TablePagination slots slotProps

cc @DiegoAndai should we have another umbrella to take care of the completeness of slots and slotProps as a follow-up from the deprecation? I can own this if you'd like.

Hey @quibaritaenperdresatrompe, thanks for the report! The button icon's slots are under the action slots, as they're slots of the TablePaginationActions component. So you can access them through, for example, MuiTablePagination.defaultProps.slots.actions.nextButtonIcon. Here's an example: https://stackblitz.com/edit/stackblitz-starters-5rgtos?file=src%2FApp.tsx.
I would mark this issue as expected by design.
For context, this was introduced in https://github.com/mui/material-ui/pull/39353.

cc @DiegoAndai should we have another umbrella to take care of the completeness of slots and slotProps as a follow-up from the deprecation? I can own this if you'd like.

@siriwatknp Yes, I think we should have an issue for that 🙌🏼 the goal would be to add any missing slots and slotProps that were not added due to a deprecation. Could you create that issue?
I'm also curious about what you think about the sidenote in https://github.com/mui/material-ui/pull/33797#issuecomment-1850696082:

I realized that we made a mistake in that PR. It should be slotProps.actions.slotProps.firstButton.

For deep slots, what should be the usage?

I don't know if we have any more of these. If we go with that, then the current TablePagination implementation is incorrect as it shouldn't allow slots.actions.nextButtonIcon, the correct path should be slotProps.actions.slots.nextButtonIcon.

For deep slots, what should be the usage? I'm thinking

Make sense to me for the time being but in v7, we should consider restructuring components to not have nested slots.

I think we should close this issue as expected and create a new one to remove nested slots in the future; what do you think?

I think we should close this issue as expected and create a new one to remove nested slots in the future; what do you think?

Sounds good. The new issue could be an umbrella one that covers all existing components that contain nested slots.

I created the follow up issue, closing this one.


Failed to deploy to IBM Cloud

I built a new application in Golang, using Go Modules, and I've been trying to deploy to IBM Cloud but I've faced the same issue (logs below). I am not sure if this is something I did or something I built wrongly but since this is happening to me in a couple of other accounts (same application in other IBM Cloud accounts), maybe there are other developers who are facing the same stuff.
Service -> (Public) Cloud Foundry, Golang buildpack.
App -> https://github.com/victorshinya/natural-language-understanding-demo
ERROR -> build github.com/victorshinya/natural-language-understanding-demo: cannot load github.com/IBM/go-sdk-core/core: open /tmp/app/vendor/github.com/IBM/go-sdk-core/core: no such file or directory
go.mod
module github.com/victorshinya/natural-language-understanding-demo

go 1.12

require (
	github.com/IBM/go-sdk-core v1.1.0
	github.com/go-openapi/strfmt v0.19.3 // indirect
	github.com/go-playground/universal-translator v0.17.0 // indirect
	github.com/gorilla/mux v1.7.3
	github.com/joho/godotenv v1.3.0
	github.com/onsi/ginkgo v1.10.3 // indirect
	github.com/onsi/gomega v1.7.1 // indirect
	github.com/rs/cors v1.7.0
	github.com/watson-developer-cloud/go-sdk v1.0.0
	golang.org/x/net v0.0.0-20191125084936-ffdde1057850 // indirect
	golang.org/x/sync v0.0.0-20190911185100-cd5d95a43a6e // indirect
	golang.org/x/sys v0.0.0-20191126131656-8a8471f7e56d // indirect
	gopkg.in/go-playground/validator.v9 v9.30.2 // indirect
	gopkg.in/yaml.v2 v2.2.7 // indirect
)

Logs
Updating app knowledge-studio-demo...
Mapping routes...
Comparing local files to remote cache...
Packaging files to upload...
Uploading files...
 186.19 KiB / 186.19 KiB [===========================================================================================================================================================================================] 100.00% 2s

Waiting for API to complete processing files...

Staging app and tracing logs...
   Downloading go_buildpack...
   Downloaded go_buildpack
   Cell 87626c63-eae1-42ff-a4fb-785ef79934bd creating container for instance 5a085dfb-cef3-433d-b74b-f2dee91cadc4
   Cell 87626c63-eae1-42ff-a4fb-785ef79934bd successfully created container for instance 5a085dfb-cef3-433d-b74b-f2dee91cadc4
   Downloading app package...
   Downloaded app package (1.2M)
   -----> Go Buildpack version 1.8.42
          **WARNING** [DEPRECATION WARNING]:
          **WARNING** Please use AppDynamics extension buildpack for Golang Application instrumentation
          **WARNING** for more details: https://docs.pivotal.io/partners/appdynamics/multibuildpack.html
   -----> Installing godep 80
          Download [https://buildpacks.cloudfoundry.org/dependencies/godep/godep-v80-linux-x64-cflinuxfs3-b60ac947.tgz]
   -----> Installing glide 0.13.3
          Download [https://buildpacks.cloudfoundry.org/dependencies/glide/glide-v0.13.3-linux-x64-cflinuxfs3-ef07acb5.tgz]
   -----> Installing dep 0.5.4
          Download [https://buildpacks.cloudfoundry.org/dependencies/dep/dep-v0.5.4-linux-x64-cflinuxfs3-79b3ab9e.tgz]
   -----> Installing go 1.12.7
          Download [https://buildpacks.cloudfoundry.org/dependencies/go/go1.12.7.linux-amd64-cflinuxfs3-040d6b6f.tar.gz]
          **WARNING** Installing package '.' (default)
   -----> Running: go install -tags cloudfoundry -buildmode pie .
   build github.com/victorshinya/natural-language-understanding-demo: cannot load github.com/IBM/go-sdk-core/core: open /tmp/app/vendor/github.com/IBM/go-sdk-core/core: no such file or directory
          **ERROR** Unable to compile application: exit status 1
   Failed to compile droplet: Failed to run finalize script: exit status 12
   Exit status 223
   Cell 87626c63-eae1-42ff-a4fb-785ef79934bd stopping instance 5a085dfb-cef3-433d-b74b-f2dee91cadc4
   Cell 87626c63-eae1-42ff-a4fb-785ef79934bd destroying container for instance 5a085dfb-cef3-433d-b74b-f2dee91cadc4
   Cell 87626c63-eae1-42ff-a4fb-785ef79934bd successfully destroyed container for instance 5a085dfb-cef3-433d-b74b-f2dee91cadc4
Error staging application: App staging failed in the buildpack compile phase
FAILED

@victorshinya did you try building go build .. There seems to be some confusion on the path as it seems to be looking into vendor directory.

I ran go install -tags cloudfoundry -buildmode pie . locally and it worked. In this case, when you deploy an application using public Cloud Foundry on IBM Cloud, you don't need to build the app before you deploy (like Java). it should build and start automatically.

I am closing the issue since I figured out that I need to deploy the vendor/ folder to the IBM Cloud so Cloud Foundry can use all vendor package to build the app. Probably it will be solved in another Cloud Foundry's Go buildpack.


Reindexing isn't fully implemented and users/community members indexing doesn't work

Package version (if known): v12
Describe the bug
The current reindex commands in rdm-records and rdm-communities don't fully reindex requests and request events. In addition the users, groups, and community members  bulk_index commands that were added two years ago don't seem to work. I'm also not sure how to re-initialize the usage stats indexes.
Steps to Reproduce

Set up a normal install, including adding users, communities, and submitting some requests
Delete indexes
Reindex records and communities
See that the requests index is blank.
Try to manually reindex users with

users_service = current_service_registry.get("users")
users_service.rebuild_index(system_identity)

See that the users index is blank

Expected behavior
There should be a recipe for re-indexing all content. A script based approach that works is https://github.com/caltechlibrary/caltechauthors/blob/main/scripts/reindex_more_stuff.py, but this isn't ideal. Requests should probably be included in community reindexing and the users/community members indexing is pretty slow.

Hey @tmorrell, have you tried using this command ?

@zzacharo I added this and other rdm commands in the docs as they are not in the reference yet.
Please let me know if the documentation for this command is accurate and covers everything needed:
https://github.com/inveniosoftware/docs-invenio-rdm/pull/608

@zzacharo I added this and other rdm commands in the docs as they are not in the reference yet. Please let me know if the documentation for this command is accurate and covers everything needed: inveniosoftware/docs-invenio-rdm#608

Thanks for that! I approved it!

The cli command should solve the requests issue. I need to test it, but I don't think it will solve the users issue.

This turned out to be a celery issue. I've added more details into the documentation, and this issue can be closed after merging.

closing as suggested above


Questions about evaluation metrics

Thank you for your generous open source.
Now that I have been through the training process, and obtain a model trained for 70 epochs.
So what I wonder is how to output or where I can find these evaluation metrics to measure the performance of my trained model, and if any ground truth data needed for this evaluation process.
Looking forward to your reply and help, much thanks.

Thanks for your appreciation.
I've just found that it was my mistake and already updated the testing script.
Testing command:
Testing
python run.py --config experiments/test_multi_CASIA.yml --gpu 0 --num_workers 4
The output images of your trained model will be exported in test_result_dir which is in your testing config file.
For MAD & SIDE, you need ground truth which is already provided in BFM dataset (you can download it by download_synface.sh).


pyproject.toml configuration

Mutatest should support argument definitions in the pyproject.toml

Not clear this is the place of these settings, may be better in settings.cfg or mutatest.ini


data label cannot be displayed

data labe of first bar cannot be displayed.

Uploading Chart_Nested_bar_Auto - copy.pbix.ZIP…

Chart_Nested_bar_Auto - copy.pbix.ZIP

The issue was fixed in https://github.com/zBritva/charticulator/pull/42


Cannot Run ESP32C6-devkitm

Hello all,
I am trying use Nuttx in ESP32C6-devkitm.
I followed the tutorial: https://nuttx.apache.org/docs/latest/platforms/risc-v/esp32c6/index.html
Below you can se my outputs:
~/nuttxspace/nuttx$ make bootloader Using direct bootloader to boot NuttX.
`riscv-none-elf-ld: warning: /home/felipe-moura/nuttxspace/nuttx/nuttx has a LOAD segment with RWX permissions
CP: nuttx.hex
MKIMAGE: NuttX binary
esptool.py -c esp32c6 elf2image --ram-only-header -fs 4MB -fm dio -ff 80m -o nuttx.bin nuttx
esptool.py v4.8.dev4
Creating esp32c6 image...
Image has only RAM segments visible. ROM segments are hidden and SHA256 digest is not appended.
Merged 1 ELF section
Successfully created esp32c6 image.
Generated: nuttx.bin
esptool.py -c esp32c6 -p /dev/ttyACM0 -b 921600  write_flash -fs 4MB -fm dio -ff 80m 0x0000 nuttx.bin
esptool.py v4.8.dev4
Serial port /dev/ttyACM0
Connecting...
Chip is ESP32-C6FH4 (QFN32) (revision v0.0)
Features: WiFi 6, BT 5, IEEE802.15.4
Crystal is 40MHz
MAC: 54:32:04:ff:fe:07:14:94
BASE MAC: 54:32:04:07:14:94
MAC_EXT: ff:fe
Uploading stub...
Running stub...
Stub running...
Changing baud rate to 921600
Changed.
Configuring flash size...
Flash will be erased from 0x00000000 to 0x0003cfff...
Compressed 246716 bytes to 104167...
Wrote 246716 bytes (104167 compressed) at 0x00000000 in 2.0 seconds (effective 978.2 kbit/s)...
Hash of data verified.
Leaving...
Hard resetting via RTS pin...
`
But after i connect in the board I am getting bootloader issue.
I am not sure if my bootloader process is right, can you help me, please?
`
Terminal ready
load:0x408042f0,len:0x884
SHA-256 comparison failed:
Calculated: 113a530452b06587e1697284c537b530939f3b6c0afde0ea002b4170b843b604
Expected:<PHONE_NUMBER>b40000000000000000000000000000000000000000000000000000
Attempting to boot anyway...
entry 0x40803f96
ESP-ROM:esp32c6-20220919
Build:Sep 19 2022
rst:0x7 (TG0_WDT_HPSYS),boot:0xc (SPI_FAST_FLASH_BOOT)
Saved PC:0x40800b10
SPIWP:0xee
mode:DIO, clock div:2
load:0x40800000,len:0x42e8
load:0x408042f0,len:0x884
SHA-256 comparison failed:
Calculated: 113a530452b06587e1697284c537b530939f3b6c0afde0ea002b4170b843b604
Expected:<PHONE_NUMBER>b40000000000000000000000000000000000000000000000000000
Attempting to boot anyway...
entry 0x40803f96
ESP-ROM:esp32c6-20220919
Build:Sep 19 2022
rst:0x7 (TG0_WDT_HPSYS),boot:0xc (SPI_FAST_FLASH_BOOT)
Saved PC:0x40800b10
SPIWP:0xee
mode:DIO, clock div:2
load:0x40800000,len:0x42e8
load:0x408042f0,len:0x884
SHA-256 comparison failed:
Calculated: 113a530452b06587e1697284c537b530939f3b6c0afde0ea002b4170b843b604
Expected:<PHONE_NUMBER>b40000000000000000000000000000000000000000000000000000
Attempting to boot anyway...
entry 0x40803f96
ESP-ROM:esp32c6-20220919
Build:Sep 19 2022
`
steps:
./tools/configure.sh esp32c6-devkitc:nsh
make
make bootloader
make flash ESPTOOL_PORT= ESPTOOL_BINDIR=./
picocom...

Hi,
Seems you are using direct bootloader, you don't need to use make bootloader command. Please apply this steps:
make distclean
./tools/configure.sh esp32c6-devkitm:nsh
make
# ESPTOOL_PORT value might change
make download ESPTOOL_PORT=/dev/ttyUSB0
# picocom ...

If problem still happens, please inform and please inlcude compiler and versions (nuttx and nuttx-apps) you are using.

Hello @eren-terzioglu , thank you for help me. Issue still happens:
Please, look file attached, there I put the terminal output
term_out.txt
About my configs:
I am working inside nuttx docker: https://github.com/apache/nuttx/blob/master/tools/ci/docker/linux/Dockerfile
I clonned nuttx using:
https://developer.espressif.com/blog/nuttx-getting-started/
Below you can se my tools version:
riscv-none-elf-gcc --version riscv-none-elf-gcc (xPack GNU RISC-V Embedded GCC x86_64) 13.2.0 Copyright (C) 2023 Free Software Foundation, Inc. This is free software; see the source for copying conditions.  There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
root@1079d9310790:/nuttxspace/nuttx# cat .version
#!/bin/bash
CONFIG_VERSION_STRING="12.6.0-RC1"
CONFIG_VERSION_MAJOR=12
CONFIG_VERSION_MINOR=6
CONFIG_VERSION_PATCH=0
CONFIG_VERSION_BUILD="cd4fdf27c5"

I tested with an esp32c6 with this specs and it is working well (only difference is revision 0.0 -> 0.1):
esptool.py v4.8.dev4
Found 2 serial ports
Serial port /dev/ttyUSB0
Connecting....
Detecting chip type... ESP32-C6
Chip is ESP32-C6FH4 (QFN32) (revision v0.1)
Features: WiFi 6, BT 5, IEEE802.15.4
Crystal is 40MHz

Are you using usb port or uart port? Seems you are USB port, please change it to uart port.

You are right, I was using wrong usb connector. After change for UART port it is working properly. Thank you.


QTMs internally packaged Python Interpreter shadows the one in the activated anaconda env.

Hallo,
I was not able to use the this PAF workflow, as It does not use the interpreter of the prepared anaconda env, but instead uses the internal Interpeter:
I  amended the run_opensim_example.bat to verify this:
call conda activate opensim_example

where python

echo "===="

echo $Env:%PATH%
echo "===="

python "%~dp0qtm2opensim.py" --c3d_dir "%~1"

pause
exit

resulting in the following output:

C:\Program Files (x86)\Qualisys\Qualisys Track Manager>call conda activate opensim_example
C:\Program Files (x86)\Qualisys\Qualisys Track Manager\python.exe
C:\Users\user0\anaconda3\envs\opensim_example\python.exe
C:\Python312\python.exe
C:\Users\user0\AppData\Local\Microsoft\WindowsApps\python.exe
C:\Users\user0\anaconda3\python.exe
"===="
$Env:C:\Users\user0\anaconda3\envs\opensim_example;C:\Users\user0\anaconda3\envs\opensim_example\Library\mingw-w64\bin;C:\Users\user0\anaconda3\envs\opensim_example\Library\usr\bin;C:\Users\user0\anaconda3\envs\opensim_example\Library\bin;C:\Users\user0\anaconda3\envs\opensim_example\Scripts;C:\Users\user0\anaconda3\envs\opensim_example\bin;C:\Program Files (x86)\Measurement Computing\DAQ;C:\Python312\Scripts;C:\Python312;C:\Windows\system32; [...] ;C:\Users\user0\anaconda3\condabin;C:\Users\user0\anaconda3;.
"===="
Traceback (most recent call last):
  File "F:\00_Dev\06_qtm_paf_examples\paf-opensim-example\Templates\qtm2opensim.py", line 3, in <module>
    import scipy.signal
ModuleNotFoundError: No module named 'scipy'

From the Env I would assume that in general the activation of the anaconda env. was successful.
Using the qtm2opensim.py from a cmd instance with activated env worked as expected, as well as the paf-python-example. <-> in this paf example the environement is the same, apart from using another conda env. which python only differs in that regard that the internal interpreter is not listed.
Any guideance on what could be the issue here is highly appreciated.

I adapted the example to not use the php/initialize Template:
settings.paf:
Analyses:
  Generate .trc and .mot:
    Type: External program
    Program display name: Command Prompt
    Export session: Yes
    Export measurements: [c3d, tsv]
    Arguments: ['/C $TemplateDirectory$\run_opensim_example.bat']
    Template files: ['*.py']

run_opensim_exmaple.bat

call conda activate opensim_example

python qtm2opensim.py --c3d_dir .

exit

Which worked as expected.
Hence I was wondering what would be the benefit of using the Initialize Template / RunProcessing.php? in this specific use case?


raft: add an thread-unsafe Node: RawNode

Multiraft in cockroachdb has a goroutine which can driven raft, so it's not necessary to use another goroutine in multinode to driven raft.
This goroutine in multinode may cause some race, so try to remove it.

Can you please resolve the merge conflict?

Also do you know if there are people other than cockroach is using the pkg? I am a little bit worry about this huge change. But if not, then I think it might be just fine.

I think it's unlikely that someone else is successfully using MultiNode without asking for help, so I think it's OK to make backwards-incompatible changes here. Godoc.org shows that there are only a couple of (public) packages importing etcd/raft besides forks of etcd and cockroach: https://godoc.org/github.com/coreos/etcd/raft?importers
This change is very close to simply exposing the raft state machine struct directly. All the groupState methods are thin wrappers around raft methods (Node methods are also thin wrappers around raft methods, but Node methods are thread-safe and groupState methods are not). I think that instead of introducing a new Group interface (and a MultiNode which is simply a factory for Groups) it would be better to move the groupState methods to the raft type and export that interface (and a new factory function that does what MultiNode.CreateGroup is currently doing). We could remove multinode.go entirely, or leave it around for backwards-compatibility in case anyone is using its current interface.

Do you mean we can move prevSoftSt and prevHardSt in multinode.go to raft.go? And also newReady()?
Or we can create other file, but remove the Group interface, just expose GroupState which may be another better name?

Yes, I'm thinking that we'd move all the Ready-related functionality from Node and MultiNode into the raft type itself (raft would contain a Ready struct or something like it that it would update after every step). Or we could keep a separate type like GroupState, I'm just not sure what to call it. (Maybe RawNode or UnsafeNode, since the major difference between this new type and Node is thread-safety?). @xiang90 and @yichengq, what do you think?

@es-chow @bdarnell
The original goal is to keep raft very small and only care about the raft algo logic. Thus we wrap raft with node to carry some extra states to interact with the state machine driven by raft.
I am not convinced that we need to move all ready-x functions to raft.go. I understand that you might have a goroutine that can drive raft. Probably we can just have a RawNode or something like that to carry the extra data without changing raft.go?

I have similar thoughts that using a RawNode is better than changing raft struct. The abstraction of raft and Node type works well in current state, and moving ready-logic into raft struct may cause it messy.

Add 2 new files as per the comments.

updated as per comments.

LGTM

updated.

LGTM. @xiang90 ?

@es-chow @bdarnell Are we still going to maintain multinode? Or we will depreciate it in favor of rawNode?

update as per comments from xiang90.
For the multinode maintenance issue, I will follow bdarnell's decision.

why semaphoreci failed at TestPurgeFile? Please help.

I think we should remove MultiNode. Cockroach will be switching to RawNode and I don't think anyone else will be using MultiNode.

@es-chow Let's fix the last few issues. And can you please remove multi-node as @bdarnell mentioned then?

updated and remove multinode.

lgtm


[2158] Replace terrafile

Context
terrafile is no longer maintained
Changes proposed in this pull request
Replace with a simple git command.
Guidance to review
make test terraform-plan TF_VAR_docker_image=x
make domains-infra-plan
make test domains-plan
Checklist

[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally
[ ] Run DQT integration tests locally (if appropriate)

 Replace terrafile