When I ran the experiments in examples/translation, I encountered a problem

I trained a new model on the IWSLT'14 German to English dataset just following: https://github.com/pytorch/fairseq/tree/master/examples/translation#iwslt14-german-to-english-transformer.
However, when i used generate.py to translate the test dataset using:
fairseq-generate data-bin/iwslt14.tokenized.de-en \     --path checkpoints/checkpoint_best.pt \     --batch-size 128 --beam 5 --remove-bpe
I just got the answer like this:

I just followed the tutorial，can you help me to solve this problem?
Thanks!

How long did you train the model for? Is it fully trained?

I had trained for 13 hours on V100

How long did you train the model for? Is it fully trained?

@zjr35897  the PPL value you see in the end of training is way to high, something went wrong, can you paste your full training command here?
Also it will be helpful if you redirect training log into a file and attach it here too.

@zjr35897 the PPL value you see in the end of training is way too high, something went wrong, can you paste your full training command here?
Also it will be helpful if you redirect training log into a file and attach it here too.

This is my command.

Ok, can you please remove old checkpoints:
rm -rf checkpoints/
and then run this command:
CUDA_VISIBLE_DEVICES=0 python train.py     data-bin/iwslt14.tokenized.de-en     --arch transformer_iwslt_de_en --share-decoder-input-output-embed     --optimizer adam --adam-betas '(0.9, 0.98)' --clip-norm 0.0     --lr 5e-4 --lr-scheduler inverse_sqrt --warmup-updates 4000     --dropout 0.3 --weight-decay 0.0001     --criterion label_smoothed_cross_entropy --label-smoothing 0.1     --max-tokens 4096 --no-progress-bar --log-interval 100 --max-epoch 50 | tee log.txt
then check if the model you get is good. If not, copy and paste the content of log.txt here: https://gist.github.com/   and send us the link.
my guess is that you have some messed up checkpoints in your checkpoints/ folder (this is why you are at epoch 837) and your model doesn't train

I've encountered similar problem recently. I think that in default settings the model converges very slowly. You need to increase batch size and/or update_freq and/or learning rate.

Oh, I didn't realize you're doing En->De, the original IWSLT setup was tuned for De->En so yes, if you switch direction you may need some tuning of dropout and learning rate (and update-freq may help too)

@edunov Thanks for your answer. I changed the new version of fairseq(0.9.0) to solve this problem. I used to train with fairseq version 0.7.0 for three days, with more than 800 epochs in total. I didn't know why it happened.
I trained in two directions containing En->De and De->en at the same time. And they both had the same problem before I used a version of fairseq.


drive: Allow an attribute to declare the root element to be replaced

Rather than replacing the whole body, allow the developer to declare exactly which element should be replaced by an id or set of ids to be replaced in the old & new document.
I don't want to refresh my header/footer and there's a small amount of state that I'd like to maintain in my header. So I'd just like to replace the main element.

This is being worked on in #627

@marcoroth my apologies I did look but evidently didn't search the right keywords; and thank you for pointing me to the right place! Super happy to see it's coming soon (or maybe available in latest RC)

No need to worry @EthanStandel!
Sadly it got reverted in the second RC, but I'm sure we can get it back in once we can figure out how to do it properly!


refactor(package): add files property

publishing only files which are needed in prod

This is also a duplicate of #128.


Application toolbar theme is incorrectly tied to OSX system theme

Azure Data Studio Version: 1-7-0-insider
System: MacOS 10.4.4 Mojave
System theme: Dark theme

Steps to Reproduce:

In Mojave settings, change appearance to dark theme
Open ADS and change app theme to default light
Notice how the application toolbar remains dark

Intended behavior:
Similar to VS Code, the application toolbar is themed from the app theme, and will not change depending on the system theme

Example of VSCode application toolbar respecting app theme

I think is because we currently have custom titlebar rendering disabled.

This appears to have been fixed in the June 2019 update

Closing since this has been fixed, let us know if you see any further issues with this.


Any plans to bundle the package

I'm copying the files out of node_module folder, I know I'm weird...
For that I would much rather have a single file instead of many.
Currently I'm not sure which file I should use assuming I need only one.
current folder structure is not intuitive - I would expect to have what in need in the compiled folder but I have there the following files and filder - which doesn't make much sense to me (src inside compiled?)...

I'll try to create a better structure after fixing some issues.


HotFix: build project-creator error

文档的修改是因为 docs:check 没通过，因此进行文档的修改

Codecov Report

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

@@           Coverage Diff           @@
##           master     #616   +/-   ##
=======================================
  Coverage   57.14%   57.14%
=======================================
  Files           1        1
  Lines          70       70
  Branches       15       15
=======================================
  Hits           40       40
  Misses         30       30

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cc8a81...33684c9. Read the comment docs.


Add sieve-execute

Modified Sieve configuration to include the vnd.dovecot.execute extension, and added sieve_execute_bin_dir (similar to sieve_pipe_bin_dir)
What's left is to copy files from config/sieve-execute to /usr/lib/dovecot/sieve-execute. I'll push commits for this in the next few days.
See: https://wiki2.dovecot.org/Pigeonhole/Sieve/Plugins/Extprograms

Then this is WIP. Please reach be if this is ready for review.

Tests are failing too. Have a look at them.
Please consider explaining your PR. You did not follow contribution guidelines - therefore, we have no issue where discussion of this PR took place.


Use kubelet to run etcd

Since this is a significant change to how kraken deploys etcd, I was going to switch from using service discovery to using static IP addresses. We've wanted to do this or while as it would make etcd cluster formation more reliable. Unfortunately, this might not be so easy. To create a static pod manifest with static IP addresses we need to template each etcd node separately. ansible/cloud_config/write_files.yaml makes this difficult because it assumes that templates are per nodepool.

can we test the impact of the change on upgrade, please?

@davidewatson are you suggesting here that not only will we move over to have kubelet manage etcd (for which then i would ask why dont we use operator also or perhaps cyklps-etcd) but that we also use static ips? I thought these were mutually exclusive concepts, are we determining that we can/should use static ips now?

@philoserf include also the recovery story, as this will become important also.

@philoserf: We will. Note that kraken created etcd clusters are not changed by kraken after successful creation. Upgrades are a manual process, so I think we only need to test that old kraken configs can still be used to spin up clusters using the existing etcd configuration.
@alejandroEsc: Good point about the recovery playbook needing to be updated. Static pods and static IP addresses are orthogonal concepts. You can use both at the same time if you are able to generate the manifest.
One reason for not using etcd-operator is that it requires self-hosted etcd [1] and we believe circular dependencies are problematic. Perhaps more importantly, due to bugs, I've been told that self-hosted etcd is not highly available right now.
[1] This is partially false. If the self-hosted etcd were in a different k8s cluster then we might be okay.

This was an alternative fix for https://github.com/samsung-cnct/kraken-lib/issues/825, in addition to https://github.com/samsung-cnct/kraken-lib/pull/974


encoder drive pSp: attribute "logger" missing

Hello, thank you for your work.
When running train_restyle_psp.py, with non-human domain flags:
python inversion/scripts/train_restyle_psp.py \
--dataset_type my_data_encode \
--encoder_type ResNetBackboneEncoder \
--exp_dir experiments/restyle_psp_ffhq_encode \
--batch_size 2 \
--test_batch_size 2 \
--workers 8 \
--test_workers 8 \
--val_interval 5000 \
--save_interval 10000 \
--start_from_latent_avg True \
--lpips_lambda 0.8 \
--l2_lambda 1 \
--id_lambda 0 \
--moco_lambda 0.5 \
--input_nc 6 \
--n_iters_per_batch 3 \
--output_size 256 \
--stylegan_weights train_network.pt

(and data prepared correctly)
I get this error:
  File "inversion/scripts/train_restyle_psp.py", line 25, in main
    coach.train()
  File "/scratch/./inversion/training/coach_restyle_psp.py", line 146, in train
    self.parse_and_log_images(id_logs, x, y, y_hats, title='images/train')
  File "/scratch/./inversion/training/coach_restyle_psp.py", line 318, in parse_and_log_images
    self.log_images(title, im_data=im_data, subscript=subscript)
  File "/scratch/./inversion/training/coach_restyle_psp.py", line 329, in log_images
    path = Path(self.logger.log_dir) / name / f'{step:04d}.jpg'
AttributeError: 'Coach' object has no attribute 'logger

Indeed, there doesn't seem to be a logger attribute anywhere. Is there anything I'm doing wrong?

Strange. The logger is defined here:
https://github.com/yuval-alaluf/stylegan3-editing/blob/36ce2b4ddab7821586012c1d5fbf737a3ea1c355/inversion/training/coach_restyle_psp.py#L82
This is in the init of the Coach class so it should have been initialized. Did you debug to see what happens?

Well, it wasn't there, I added it and it works, thanks.


Checkout Feature

[ ] #8
[ ] #7
[ ] #12

Also see:

https://github.com/redwoodjs/example-store/issues/6


Force UASP support

I have a hdd enclosure, that has worked for me with UASP on, but I was using raspibolt. I've just "upgraded" to raspiblitz to make installation of stuff a bit easier.
The issue is that I am not confident that my testing is enough. Especially because in my testing, the HDD is about 2 - 3 inches away from the pi. Sounds unimportant, but I found that the wifi was not working properly when it was right under the pi with UASP on.
On top of that, my SSD runs MUCH slower with UASP off. Enough that I've wasted a whole week getting half the blocks to download and now that I have it on, there is a very noticable difference.
Long story short, I think it is a good idea to have a way for people like myself to force UASP to be enabled. I am not sure the best way to go about it right now.
My initial thought was to add a variable to the raspiblitz.info file, but I realised that the datadrive.sh file does not source this file so I am open to ideas. I didn't want to add a source of this file to the script because it might break something else, but I do want to ensure that my contribution is "fool proof" or at least robust.
I want a solution that will not require patching after a raspiblitz upgrade in the future.

(Hint: UASP = USB Attached SCSI Protocol)
The problem is that it's highly device-dependent whether UAS is supported at all and if it is, whether it is faster than usb-storage. My Samsung T5 uses usb-storage and performs well (almost 300 MiB/sec in sequential R/W benchmarks).
What is your kernel using now and are you sure it was using something else before? Try:
$ lsusb -t
/:  Bus 02.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/4p, 5000M
    |__ Port 1: Dev 2, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
[...]

$ dmesg | grep UAS
usb 2-1: UAS is ignored for this device, using usb-storage instead

Assuming your drive is using usb-storage, have you tried setting usb-storage.quirks=$VENDOR_ID:$PRODUCT_ID in your /boot/cmdline.txt?

At the moment the RaspiBlitz is checking the the type of the ssd-driver/enclosure and just for models we know & tested the the UASP support kept on ... for every other its deactivated to prevent the bug reports we had on RaspBlitzes freezing during sync.
@dipunm because this UASP detection is done during the first boot on a fresh system, it would make sense to use the tweak of putting a file like uasp.force on the boot section of the sd card with your laptop after flashing the sd card before putting it into the RaspberryPi. This could be detected and UASP will stay on. BUT I think then also a entry in the raspiblitz.conf should be created in that process automatically so that on an update/recovery with a fresh sd card this can be re-fixed again without the need of adding again the uasp.force file to the sd card manually.
As you can see this is quite a bit of implementation todo. I will keep this for the v1.8.1 milestone in mind ... but if anybody likes to contribute a PR on this we might consider this earlier.

OK I merged the PR and modified it a bit so that it also should work during first setup.
Will be part of v1.7.1RC2 - to be final tested.

I just want to let you know, I purchased the specific SSD and enclosure recommended on your main page (using your affiliate links) and when installing a different bit of node software (PiNodeXMR) I had gawd-awful performance that was really difficult to root cause - until out of sheer desperation I modified /boot/cmdline.txt to force a fallback to usb-storage mode at which point all of the weird issues resolved themselves.  When I later installed RaspiBlitz, this was the first thing (force usb-storage) that I did before even connecting the external SSD in order to avoid difficult to debug problems again.
The strange thing is, this chip, the ASM1153, is supposed to be well supported according to other documentation I can find online.   Here is my lsusb output:
Bus 002 Device 002: ID 174c:1153 ASMedia Technology Inc. ASM1153 SATA 3Gb/s bridge

But the strange issues (high CPU usage, connections timing out, etc) persisted across two sets of pi 4 + enclosure + SSD (with pinodexmr) so I don't think it's a one-off thing.  Perhaps the strange behavior of this enclosure works better with the access patterns of the raspiblitz software, but from my perspective it is a poor choice to pair with a raspi and I didn't bother trying, killing UAS in boot params as a matter of course after my experience with pinodexmr.
FYI...

I purchased the same specified UGREEN enclosure from the README.md and it has the vendor and device ID 174c:1153 with the bridge chip ASM1153.
admin@raspiblitz:~ $ lsusb
Bus 002 Device 002: ID 174c:1153 ASMedia Technology Inc. ASM1153 SATA 3Gb/s bridge
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
...

I did some brief testing with RaspiBlitz v1.7.1 and didn't experience trouble with UAS mode, so I enforced UAS mode with touch /boot/uasp.force, removed the usb-storage quirk entry in /boot/cmdline.txt and added a similar if-statement to whitelist this particular enclosure in the file config.scripts/blitz.datadrive.sh (maybe not necessary, when /boot/uasp.force is in place):
    # check if HDD ADAPTER is on UASP WHITELIST (tested devices)
    if [ "${hddAdapter}" == "174c:55aa" ]; then
      # UGREEN 2.5" External USB 3.0 Hard Disk Case with UASP support
      hddAdapterUSAP=1
    fi
    if [ "${hddAdapter}" == "174c:1153" ]; then
      # UGREEN 2.5" External USB 3.0 Hard Disk Case with UASP support, 2021+ version
      hddAdapterUSAP=1
    fi
    if [ "${hddAdapter}" == "0825:0001" ] || [ "${hddAdapter}" == "174c:0825" ]; then
      # SupTronics 2.5" SATA HDD Shield X825 v1.5
      hddAdapterUSAP=1
    fi
    echo "hddAdapterUSAP=${hddAdapterUSAP}"

With UAS mode in place the Sandisk Plus SSD is properly recognized as a non-HDD (/sys/block/sda/queue/rotational is zero, despite with usb-storage quirk mode it reads as 1). I was also able to activate TRIM according to instructions summarized specifically in Jeff's blog post https://www.jeffgeerling.com/blog/2020/enabling-trim-on-external-ssd-on-raspberry-pi
To make the provisioning mode unmap persistant, I added the file /etc/udev/rules.d/10-trim.rules with content
ACTION=="add|change", ATTRS{idVendor}=="174c", ATTRS{idProduct}=="1153", SUBSYSTEM=="scsi_disk", ATTR{provisioning_mode}="unmap"

I feel better now for my SSD because I believe that proper TRIM handling allows the SSD controller to do a better wear leveling job. So far my RaspiBlitz runs happily and I didn't recognize any severe issues in the log files. Keep rockin'n'rollin'!

@cricktor thanks for the find of the new UGREEN ID .. added for v1.7.2 release. I will open another issue and mark it for a bit later to look into that trimming again.


Support BESS protocol (for UML)

BESS protocol transferrs L2 packets as AF_UNIX SOCK_SEQPACKET .
BESS protocol has been used by the vector network interfaces of User Mode Linux (UML).
(terminal 1) $ bin/gvproxy -debug -listen unix:///tmp/network.sock -listen-bess unixpacket:///tmp/bess.sock
(terminal 2) $ linux.uml vec0:transport=bess,dst=/tmp/bess.sock,depth=128,gro=1,mac=5a:94:ef:e4:0c:ee root=/dev/root rootfstype=hostfs init=/bin/bash mem=2G
(terminal 2: UML)$ ip addr add <IP_ADDRESS>/24 dev vec0
(terminal 2: UML)$ ip link set vec0 up
(terminal 2: UML)$ ip route add default via <IP_ADDRESS>

More docs about the User Mode Linux with BESS socket transport: https://www.kernel.org/doc/html/latest/virt/uml/user_mode_linux_howto_v2.html#bess-socket-transport

CI failure is probably unrelated.
The main branch has been failing too: https://github.com/containers/gvisor-tap-vsock/runs/4706874516?check_suite_focus=true

Yes I am looking at the CI. I don't understand for the moment why it hangs :/
Thanks for the patch, looks good. Do you use or plan to use gvproxy in one of your project?

Tests are passing on my machine. lgtm.
Maybe in future we could add e2e tests for this feature.

Do you use or plan to use gvproxy in one of your project?

Maybe, still under an early experiment.

 What happens if you change the MTU? It should be better.
For stability, maybe check the stats API. There is maybe a clue.

Taking a look.

Interesting! What happens if you change the MTU? It should be better.

UML doesn't seem to support MTU > 1500.

Great news!


zsh: illegal hardware instruction

I've built a macOS app that bundles sqlite with this extension. I re-built python and enabled it with loadable extensions and it works fine on my mac, running 13.1.0 Ventura. Unfortunately the same vss0.dylib and vector0.dylib seem are returning these errors for the people I've distributed the bundle too:
Terminal:
> illegal hardware instruction

Apple Crash report:
> Execption Type: EXC_BAD_INSTRUCTION (SIGILL)
> Termination Reason: Namespace SIGNAL, Code 4 Illegal instruction: 4

In our code base, we run:
  create_statements = [ """
      CREATE VIRTUAL TABLE webpage_embeddings using vss0(
          body_embedding(384),
      );
      """]
  print("Creating virtual embedding tables")
  with VssSession.engine.connect() as connection:
      # VssSession handles loading the extension in another module, but it works fine on my computer and a few users:
      # import sqlite_vss
      #  dbapi_con.enable_load_extension(True)
      #  sqlite_vss.load(dbapi_con)
      #  dbapi_con.enable_load_extension(False)

      for statement in create_statements:
          connection.execute(statement) <--------- Seems to be crashing here based on logs

It seems to not be working for users before 13.1 since I built mac bundle on my computer, but randomly it is working for some users on 13.0. We cannot be sure if its actually a mac app version issue or something else.
We tried building the dylibs for sqlite-vss directly as well by including:
set(ENV{MACOSX_DEPLOYMENT_TARGET} “11”) in the CMake file, and swapping them out in the sqlite-vss library within the .venv but this didn't work either .
To get it working on my computer:
brew install sqlite3
Installed python v3.10.13 using pyenv with PYTHON_CONFIGURE_OPTS="--enable-shared --enable-loadable-sqlite-extensions"
poetry install sqlite-vss
Do you have any suggestions?

There is a note in documentation about support of AVX2 instructions


Bug: Files does nothing exporting settings, crashes importing settings

Description
FIles does nothing when trying to export settings. It crashes when trying to import settings.
Steps To Reproduce

Settings -> Advanced -> Export settings The button clicks, but nothing else happens.
Settings -> Advanced -> Import settings Files abruptly closes with no other messages.

Requirements

Expected to show Save File dialog.
Expected to show Open File dialog.

Files Version
<IP_ADDRESS> - Preview
Windows Version
10.0.22631.3593
Log File
debug-export-import-bug.log

Thanks I’ll fix soon.

Thank you for the report, this is a regression in the preview version and we'll make sure to fix before releasing to stable.


Model methods are missing with --write-mixin

Versions:

ide-helper Version: 2.12.3
Laravel Version: 9.9.0
PHP Version: 8.1.3

Description:
Model methods are missing when using --write-mixin

With --write-mixin
Without --write-mixin

Basic model methods like find appear to be missing when using --write-mixin. They work correctly without --write-mixin
I've reproduced the following steps in this repo: https://github.com/ptrkcsk/laravel-ide-helper-models
There are two differences in _ide_helper_models.php between using --write-mixin and not, as seen here: https://github.com/ptrkcsk/laravel-ide-helper-models/compare/write-mixin...no-write-mixin

The generated class has @mixin IdeHelperUser. This looks like a mistake to me because IdeHelperUser appears to not exist
The generated class extends \Eloquent. I suspect this is the problem, because with --write-mixin, the generated class is missing the methods provided by \Eloquent

Steps To Reproduce:

Generate facades helper and models helper with --write-mixin:
vendor/bin/sail artisan ide-helper:generate
vendor/bin/sail artisan ide-helper:models --write-mixin

Try to use a model's find method, e.g. User::find(). Notice "Method 'find' not found..." warning in IDE

Re-generate models helper without --write-mixin:
vendor/bin/sail artisan ide-helper:models

Try to use a model's find method again. Notice find method is now found

I faced with the same issue


Always prefix component name with Svg

As discussed in #168, the component name generated could be a problem with some filename. For an example, "infinity.js" will generate "Infinity" that is a reserved keyword.
The simplest solution is to prefix all component name with Svg. We already do it for "number" filenames.

I think, that prefix should be configurable. With possibility to opt-out completely.

Yeah, maybe configuring is an overhead) But opting-out would be nice. In my project all svgs already have 'Icon' prefix.

The name doesn't have any impact expect in React inspector 🤷♂

Ok, you're right) Withdrawing my request)

Took me some time to understand the why, even with the linked issues, so here are some more details:
JavaScript is a forgiving language, allowing you to overwrite global objects, properties and methods.
E.g., a component can be named Math or Infinity
import Math from "./Math";

function App() {
  console.log(Math.random());
  return (
    <div>
      <Math />
    </div>
  );
}

However, this would break above component, which uses at the same time Math.random(). JavaScript sees Math not as the built-in math function but as a JSX element.
I.e. - having a prefix is required to prevent conflicts.

The name doesn't have any impact expect in React inspector 🤷♂

It does have affect when using it with Storybook

What is the use case of configuring it?

It increases the readability of the code.

Please add this simple feature. Having a prefix would massively improve developer experience with auto imports without having to rename every single source file (which can frequently change). Customizing component templates is simple enough, but customizing exports and index templates requires more work from the user than it would be to add one switch to the command or config.


switch to Qt5

Using PyQt5 as well as PySide2.

lgtm, though I haven't tried it with rviz yet. It's on my todo list.


Gamm add swap events, fix up existing events

Adds a swap event to every swap, adds / cleans up existing gamm events
Closes #341

Codecov Report

Merging #344 (b2192e1) into main (e78fd3c) will decrease coverage by 0.01%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main     #344      +/-   ##
==========================================
- Coverage   18.28%   18.27%   -0.02%
==========================================
  Files         133      133
  Lines       21473    21467       -6
==========================================
- Hits         3927     3923       -4
- Misses      16891    16894       +3
+ Partials      655      650       -5

Impacted Files
Coverage Δ

x/gamm/keeper/msg_server.go
1.42% <0.00%> (-0.09%)
:arrow_down:

x/gamm/types/hooks.go
0.00% <ø> (ø)

x/incentives/keeper/gauge.go
67.46% <ø> (ø)

x/gamm/keeper/keeper.go
67.85% <100.00%> (+15.22%)
:arrow_up:

x/gamm/keeper/swap.go
77.98% <100.00%> (+0.62%)
:arrow_up:

x/lockup/keeper/utils.go
100.00% <0.00%> (ø)

x/lockup/keeper/lock.go
73.96% <0.00%> (+0.70%)
:arrow_up:

x/lockup/keeper/store.go
100.00% <0.00%> (+16.32%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e78fd3c...b2192e1. Read the comment docs.


Removed COPR chroots are not reflected in the COPR project

Hello!
Not sure if it's expect or not, but when a chroot is dropped from packit.yml, it is still being built in in COPR - the results from such chroot are however not reflected in the respective PR. This doesn't affect testing, since the results are simply thrown away, but it might waste resources in certain cases (especially when it comes to emulated architectures, like s390x).
Example:
...
jobs:
- job: copr_build
  trigger: pull_request
  metadata:
    targets:
    - fedora-rawhide-aarch64
    - fedora-rawhide-i386
    - fedora-rawhide-x86_64

creates a temporary COPR project with three chroots, however after changing it to:
...
jobs:
- job: copr_build
  trigger: pull_request
  metadata:
    targets:
    - fedora-rawhide-aarch64
    - fedora-rawhide-x86_64

the fedora-rawhide-i386 chroot is not dropped and is still rebuilt after every trigger.

I remember we have some troubles with removing the chroots so we only add new ones, but I can't remember the details.
But you are right, we need to take a look at this.
Just to make it clear, this is only relevant for one PR that change the chroots after the first push. Do the other PRs have a correct set of chroots?

I remember we have some troubles with removing the chroots so we only add new ones, but I can't remember the details.
But you are right, we need to take a look at this.
Just to make it clear, this is only relevant for one PR that change the chroots after the first push. Do the other PRs have a correct set of chroots?

Just to make it clear, this is only relevant for one PR that change the chroots after the first push. Do the other PRs have a correct set of chroots?

That's right, it affects only the PR which made the change, other PRs have correct chroots.

Just to make it clear, this is only relevant for one PR that change the chroots after the first push. Do the other PRs have a correct set of chroots?

That's right, it affects only the PR which made the change, other PRs have correct chroots.

comment to trigger the sync to Jira

The change needs to be done here:
https://github.com/packit/packit/blob/d2e66c77b5e64542e7ab8900fd3ff557ca4b48a0/packit/copr_helper.py#L202-L208
(And in tests of course...)


QuantityDetails?

I have recently tried to do some Edifact parsing with the InterchangeBuilder.
My document has some QTY elements and I saw that this elements generates the following format:
{
  "tag": "QTY",
  "quantityDetails": {
    "quantityTypeCodeQualifier": "136",
    "quantity": "1000"
  }
}

Is there a reason for adding the quantityDetails depth to the element. Why not just:
{
  "tag": "QTY",
  "quantityTypeCodeQualifier": "136",
  "quantity": "1000"
}

I just want to add some information even though you already closed the issue yourself. If you look at the QTY definition you can see that QTY uses a complex element named  QUANTITY DETAILS which is just the container element for the mentioned properties. The parser itself tries to keep that structure in place when parsing the Edifact document into respective objects.
Note though that ts-edifact is currently not developed and supported further. We basically switched from this typescript implementation to a Java based one, also written by me but currently closed-source due to company preference, that adheres more to the actual specification than ts-edifact does and I am currently working on other stuff, mainly on VSCode add-ins, so that I'm not sure when to find time to update this project to a more reasonable state.


/lp import imports at a random order which results in some data not applying

I noticed that at least on sponge and bukkit, /lp import handles commands randomly, seems like it's async.
As groups should be created first, this often leads to group not exist errors. Using /lp import the second time fixes that, as all groups are created by now.
We imported all of our servers with 40k lines+ per file and I noticed after two servers that group assignments etc are missing. Luckily I realized that after a few servers, but some people might miss that and now have completely destroyed perm setups.
I can send you a big testfile to test, but it might be enough to make simple one by adding three groups.
Try /lp import a few times and watch how group creation is in a random order.
LP: 4.4.24
Sponge 7.1.6 - 2713
PaperSpigot 1.13.2

I think you need to send through an import file for anyone to replicate this error.

Should now be resolved, was included in the v5 changes/fixes.


patch: Adding new Service credentials secret type

:tada: This PR is included in version 2.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support Using Type Resolution in Android-Library Project

Expected Behavior

I know detekt has supported the Android project right now.  But when I start writing an Android library with detekt, I found that type resolution is not supported in the android library.
I wondering if I was using it in the wrong way, or maybe this can be a new feature being suggested.

I wondering if I was using it in the wrong way, or maybe this can be a new feature being suggested.

As long as you're using the kotlin("android") gradle plugin, the tasks for type resolution should be created.
https://github.com/detekt/detekt/blob/7cce3816dd97872ff7eecba382a65462bf736934/detekt-gradle-plugin/src/main/kotlin/io/gitlab/arturbosch/detekt/DetektPlugin.kt#L54-L58
Which task are you invoking?

Thanks for your feedback.
After my research, I found tasks have been generated, but when I invoke the task detektMain, the console says "NO-SOURCE".
This is because my source code resides in the src/main/kotlin folder.
When I place the code into src/mian/java folder, the task works!

Maybe detek tasks can support projects which put their source under src/main/kotlin without sourceSet configuration.

This is already supported in the upcoming version of Detekt: 1.20.0


cargo install tauri-cli does not work on windows runner on Github Actions

Describe the bug
In a Github Actions workflow running on windows-latest having a step like this fails:
      - name: Install Rust Dependencies & Tools
        run: |
          cargo install trunk wasm-bindgen-cli tauri-cli

Error:
error: could not find `tauri-cli` in registry `crates-io` with version `*`
     Summary Successfully installed trunk, wasm-bindgen-cli! Failed to install tauri-cli (see error(s) above).

Expected behavior
Should work, only workaround seems to be to run Tauri via npm but that would require a whole different Github Actions workflow than you would use for macos-latest and ubuntu-latest (where npm or node is not required at all).
Platform and Versions (required):
N/A

you need to explicitley specify the version, because cargo kinda ignores beta versions otherwise (that's not a gh action specific thing): cargo install tauri-cli --version 1.0.0-beta.7

What if you don't want to install a beta version? Or is there no non-beta version out there yet?

There are only beta versions https://crates.io/crates/tauri-cli/versions .
Kinda like rest of tauri. (Well technically the other stuff has non beta versions but they aren't really usable anymore, cause they represented tauris alpha stage)


Make acoustid server address configurable

Adds a set_base_url method to change the base url. Lookup methods now
use get_lookup_url so that the value can be computed when needed rather
than when the module is imported.
Signed-off-by: Alastair Porter<EMAIL_ADDRESS>

Looks great; thanks.


404 error on hard refresh of custom route

Hey @fridays
Thanks for making this, it's really helpful  - Whenever i hard refresh my custom routes, it gives 404 error, when i go through <Link> it just works fine
Any thing which i might be missing ?

Sorry, i configured my server incorrectly

I was having the same problem, but turns out the solution was really simple: needed to restart the server between switching Git branches. 🙃

P.S. I've figured it out myself. Adding the following code solved it for me:
const customRoutes = require('./routes/custom')
const expressApp = Express()
expressApp.all('/custom/*', customRoutes.getRequestHandler(app))

Hi guys, i've same issue with default example on the README. Anyone have fallback ?

How do you start the server?


suggestion: build a palindrome project should be shown before the Date Formatter project

Describe the Issue
I was going through the new JS Beta curriculum and when I got to the date formatter project I realized that it probably doesn't really help add any specific knowledge that would be useful in completing the palindrome project. I think the date formatter project being in its current position may even be a hindrance because I think it is better to get people to work on the skills they have learned sooner rather than later.
My suggestion is to move the palindrome project ahead of the date formatter project for this reason.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/
Your code
N/A
Expected behavior
N/A
Screenshots
No response
System
N/A
Additional context
No response

@OrneyVortex, issues are not generally assigned, and this issue has not been marked as ready for contributions.

@jeremylt okiee thank you

I agree. That seems like the way to go.
A PR which resolves this issue, then, will swap the superOrder properties in the meta files for these two projects.


Bar Chart ordering should be consistent

Rather than sorted by any value, let's sort the bars by a consistent ordering of the categories (race, age, gender)

For age based bar charts, we want to make sure they are ordered from youngest to oldest. Alphabetically may or may not work, we should double check before closing this bug

For age based bar charts, we want to make sure they are ordered from youngest to oldest. Alphabetically may or may not work, we should double check before closing this bug

I'm not seeing any bar charts that aren't ordered so I'll close this


i18n(ja): Update Japanese astro-components.md

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[x] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Update Japanese  astro-components.md https://github.com/withastro/docs/commit/b8a7f38804f75d42431309c5a17d5578704dbde9

LGTM. Thanks so much @kyosuke!


Reduce changes in common files for windows support

Describe the bug
Goal: reduce changes in common files.
There are currently 6 common files modified for windows support:
diff_windows.txt
This issue is to track upstreaming these changes if possible.
Environment details
PVC

Perhaps one way to get rid of the functions in setup.py for initializing MSVC compiler environment is to use cmake toolset feature like: cmake -DCMAKE_TOOLCHAIN_FILE=msvc-toolchain.cmake.
I found one project that provides what is needed for this (at first glance, I have not tested it yet) and under a suitable license
https://github.com/MarkSchofield/WindowsToolchain/blob/main/Windows.MSVC.toolchain.cmake
@gshimansky @pbchekin thoughts?

@pbchekin do we have machine with NVIDIA gpu to test related changes before upstreaming?

One more commit in upstream: https://github.com/triton-lang/triton/commit/a612088cbcc7ad475fff86d918f48e74cfd415ca

I think I did all I could, the last changes were rejected.

For the record, the PR rejected upstream is: https://github.com/triton-lang/triton/pull/5372


Update email template

Proposed copy:

Preview:

thanks @mikelehen!
oh, yeah that looks weird 🤔 😑. I'm not immediately sure how to change that :( so if we're okay with it I might leave as is


File specification for FIF

Is there a file specification sheet for the FIF file, that is, the listing of the data blocks, the sensor array, the metadata, etc. I have a colleague, @eh1255, who is interested writing a hardware export in FIF?
cc: @Eric89GXL @agramfort

http://www.aston.ac.uk/lhs/research/centres-facilities/brain-centre/facilities-clinical-services/meg-studies/downloads/

Hi Eric,
Thanks for the quick response, I really appreciate it. This will help a lot!
Thanks,
Erik

thanks @Eric89GXL!


Document changes to Infrastructure

Update all the infrastructure documentation to reflect the Vault and additional APIs as well as the potential addition of archiving in S3.

Architecture updated
https://drive.google.com/file/d/1Fs6McxOOlJJPykKm-FosHwHyfNjpOnaG/view?usp=sharing


Splippage impact on dex trade

It would be very convenient if of the dex trade the impact of the expected slippage was indìcated.
Also if a maximum % of slippage could also be set.
Have any suggestion?

Hi @angelocastiglione , non-bug issues will turn stale, for future feature improvement related ideas, you can publish it on DeFiChain Discussions, garner some support from the community, and the team can decide whether to implement the feature in the future. I'd suggest that you move this to Github Discussions. Thank you!


(feat) Docker Image

This PR creates a ready-to-use docker container for postgraphql.   It is a minimal docker container based on the generic 'node:alpine' image.  The Alpine linux version was used here to make the resulting image smaller (the standard node image is 265MB, alpine is is 22MB).  Including postgraphql and dependencies the total image is 102MB.
I have added instructions to the readme on how to retrieve the image from docker hub as well as how to run it.  Basically you pass the postgraphql command line options to the docker image the same as if you were running it locally.   It does not include postgres as this is meant to be a standalone, ready-to-use production container of postgraphql.
(I will create some examples of how to run it along with a dockerized postgres in a later PR to either the documentation site and/or to the examples/doc folders.)
Dockerhub will store the docker image for people to retrieve the pre-built postgraphql image.  I have created an account named postgraphql that will own the postgraphql image.  In order to setup automated docker builds we need to link github with dockerhub to trigger the docker builds.   After approving this PR, someone will need to login to dockerhub as postgraphql and
In DockerHub logged in as postgraphql

Click on the profile drop down (upper right corner)
Choose Settings
Click on Linked Accounts and Services
Click on Link GItHub
Choose either of the linking options

In GIthub logged in as postgraphql

Go to the repo settings
Click on Integrations and Services
Click on Add Service and find 'Docker'
Click Add Service

Then I can delete and re-setup the automated docker build and make it public or I can send more instructions.

Failing, not your fault, because:
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^0.7.0 || ^0.8.0-b || ^0.9.0

This probably affects master too.

Thanks for doing this, @angelosarto, I'll get back to you in the next week. If I don't, please feel free to give me a nudge.

Hi @angelosarto; I've now completed the above instructions - back to you. Are you happy being in charge of Docker for this project?

@benjie sounds good. I will maintain the dockerhub.  I will login today and make it public and make sure the builds can be pulled publicly.

Awesome, thanks!

@benjie I think you linked the docker account to your github account -- so when I build from docker it is grabbing benjie/postgraphql   Could you delete the github link and link it to postgraphql git account instead so it points at postgraphql/postgraphql?

The PostGraphQL GitHub "account" is an organization - I don't think you can log in as it to Docker hub?
I added the Docker integration to postgraphql/postgraphql; I'm not sure why it would be looking at benjie/postgraphql?

@benjie Sorry for the delay -- had an issue with a production deployment at the day-job that consumed a bunch of time.   I am just checking back into this now.
In Github can you check under "settings" the "Authorized OAuth Apps" section then click on the name "Docker Hub Registry"  there should be a heading for "Organization access"  and if not yet granted to the postgraphql organization there should be a "Grant button"

@angelosarto Great; done - let me know if it worked 👍

(I had to use a different GitHub account because some old clients do not have Third Party app security enabled and I don't want to grant access to their repos - I don't think this should cause you any problems but just letting you know.)

@benjie Perfect - the docker images are all up and working!   I should have some time to submit new docs to the doc site for running docker versions of postgraphql

Awesome, thanks @angelosarto!

Is a Docker image planned for Graphile (= graphql v4)?
Also, an example using with docker-compose and combining with PostgreSQL docker image would be very useful. Should I create a separate issue asking about usage in docker-compose?

Open another issue 👍
This may help: https://github.com/postgraphql/postgraphql/issues/636#issuecomment-346960352

@meglio
replying to this old comment -- I think with postgraphile supporting npx and aws lambda. I am not sure that there will be much of a use case for docker image.  npx takes care of quick and dirty local and running direct in a lambda takes care of a lot as well.
If I am wrong - I think adding it to graphile should be pretty trivial - its near the same dockerfile.


Google Earth VR dont see my controlers anymore, they dont work in apk.

When using Google Earth became possible for owners Riftcat + psmoveservice (Without Fake VIVE) everything worked beautifully.
Some time later, the controllers in the application became invisible (and did not respond to the buttons). I was hoping that some update would fix this, so I did not bother.
It's been a long time, I even tried "Fake Vive". I tried 2x to reinstall the application. And other things.
I'm using the latest version of Psmoveservice and Riftcat.
Please help. Google Earth is a great application. Without it, VR loses a lot.

PSMoveService itself no longer has anything to do with SteamVR. For that you need to ask people in the google group first, and if no one knows then try PSMoveSteamVRBridge.
I have never used PSMoveService with SteamVR, any smartphones, or any other HMDs. If you can't point me to a specific problem in PSMoveService then I can't help you.

Thx for the answer. And for a PSMS. It is great :) !
Greetings! :)
VR ManiaQ - ytbe.


Deterministically associate L2 blocks with L1 blocks

[ ] Add L1 block number to L2 block header (have the proposer set it in next_block, this is insecure but good enough for the demo)
[ ] Modify the op-node to fetch L1 deposits from this block number instead of the current block

This is important in order for #8 and #7 to work well together.

This is now accounted for by other tasks, see https://www.notion.so/espressosys/Mapping-the-HotShot-Block-Stream-to-an-OP-Batch-Stream-ffa6cfc44c224591af10e646c4932e56


Import and export procedures are changing, so tell users about it

From https://govuk.zendesk.com/agent/tickets/1232807: "Import and
export procedures are being substantially changed from 01/05/2016
and users of the Trade Tariff pages need to be made aware if they
will be affected by the changes."

Before:

After:

:100:


Testing phase using HTTP instead of 0MQ doesn't work

I'm using HTTP as communication protocol between computing environment and orchestrator and I have implemented a HTTP REST server in Python that runs into the computing environment. The problem arises when the testing phase starts: orchestrator is keep waiting for a finish message from recommendation manager (actually the HTTP server itself) and nothing happens until the execution is manually interrupted. Flume's log files contains these exceptions:

Unable to start SinkRunner: { policy:org.apache.flume.sink.DefaultSinkProcessor@365474b4 counterGroup:{ n
ame:null counters:{} } } - Exception follows. java.lang.NoClassDefFoundError: Could not initialize class com.yammer.metrics.Metrics
Unsuccessful attempt to shutdown component: {} due to missing dependencies. Please shutdown the agentor disable this component, or the agent will bein an undefined state. java.lang.NullPointerException
Unable to start SinkRunner: { policy:org.apache.flume.sink.DefaultSinkProcessor@31c1fb39 counterGroup:{ name:null counters:{} } } - Exception follows. java.lang.ExceptionInInitializerError
Unsuccessful attempt to shutdown component: {} due to missing dependencies. Please shutdown the agentor disable this component, or the agent will bein an undefined state. java.lang.NullPointerException

Hi Dennis, regarding the problem, I am not the expert. It looks to me as if a component is missing or the java dependencies are not configured correctly. I guess that Davide or Andras are the experts for this problem. Kind Regards, Andreas

Thanks Andreas, let's wait for a response from Andras.

Hi Dennis, do you have a command line I can run to reproduce the issue?

You can find all the necessary to reproduce it into the branch neural-network

How do you start Idomaar?

Give me a sec, I'm pushing my local changes :)

Ok I have pushed my changes, you can start Idomaar by launching "idomaar-demo.sh" in the root folder

The neural-network branch has some issues atm: gru_recommender.py is missing the web.py dependency and it tries to use the same port 2760 where the old Mahout recommender is listening. If I fix these, there's still some spark-submit error, probably due to the fact that the branch is based on an old Idomaar version.
As for the original issue, the point here is that in case of an HTTP comp env, our interceptor in Flume did not send an finished message to the orchestartor. I have fixed this on the aws branch, which I think is the most up to date, https://github.com/crowdrec/idomaar/commit/c2655933430715c28de619b594fbd18090fd1816 . I cannot reproduce the Flume errors mentioned above, but in any case, their occurrence cannot depend on the TCP/HTTP computing environment choice.
@davidemalagoli , could we merge the aws branch to master? Also, idomaar-demo.sh still fails in the eval phase, see #75. who could take a look at it? Thanks, András

merged aws into master


Fix some more issues on staging-next related to Python bootstrap changes

Description of changes
More fixes related to https://github.com/NixOS/nixpkgs/pull/248866 targeting staging-next (https://github.com/NixOS/nixpkgs/pull/250778).
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Unfortunately, I didn't get time to run nixpkgs-review fully, but it seems like there are some failures in fe25519. Are they new?

It is new and hidden behind the fountains failure. Great catch. I found a few more packages that need similar changes and updated the PR.

@ofborg build python310Packages.fountains python311Packages.fountains
@ofborg build python310Packages.mip python311Packages.mip
@ofborg build python310Packages.proxy-py python311Packages.proxy-py
@ofborg build python310Packages.pyairvisual python311Packages.pyairvisual
@ofborg build python310Packages.pypandoc python311Packages.pypandoc
@ofborg build python310Packages.pytrends python311Packages.pytrends
@ofborg build python310Packages.stim python311Packages.stim
@ofborg build python310Packages.fe25519 python311Packages.fe25519
@ofborg build python310Packages.ge25519 python311Packages.ge25519
@ofborg build python310Packages.gpytorch python311Packages.gpytorch
@ofborg build python310Packages.reproject python311Packages.reproject
@ofborg build python310Packages.pymilvus python311Packages.pymilvus
@ofborg build python310Packages.pint-pandas python311Packages.pint-pandas
@ofborg build python310Packages.drms python311Packages.drms

I believe that the stim error is an existing one.

Result of nixpkgs-review pr 251240 run on x86_64-linux 1

6 packages marked as broken and skipped:

chatgpt-retrieval-plugin
chatgpt-retrieval-plugin.dist
python310Packages.sunpy
python310Packages.sunpy.dist
python311Packages.sunpy
python311Packages.sunpy.dist

10 packages failed to build:

python310Packages.aplpy
python310Packages.aplpy.dist
python310Packages.spectral-cube
python310Packages.spectral-cube.dist
python310Packages.stim
python310Packages.stim.dist
python311Packages.aplpy
python311Packages.aplpy.dist
python311Packages.spectral-cube
python311Packages.spectral-cube.dist

77 packages built:

apostrophe
home-assistant-component-tests.airvisual
home-assistant-component-tests.airvisual_pro
python310Packages.ax
python310Packages.ax.dist
python310Packages.botorch
python310Packages.botorch.dist
python310Packages.drms
python310Packages.drms.dist
python310Packages.fe25519
python310Packages.fe25519.dist
python310Packages.fountains
python310Packages.fountains.dist
python310Packages.ge25519
python310Packages.ge25519.dist
python310Packages.gpytorch
python310Packages.gpytorch.dist
python310Packages.mip
python310Packages.mip.dist
python310Packages.osmnx
python310Packages.osmnx.dist
python310Packages.pint-pandas
python310Packages.pint-pandas.dist
python310Packages.proxy-py
python310Packages.proxy-py.dist
python310Packages.pyairvisual
python310Packages.pyairvisual.dist
python310Packages.pymilvus
python310Packages.pymilvus.dist
python310Packages.pypandoc
python310Packages.pypandoc.dist
python310Packages.pytrends
python310Packages.pytrends.dist
python310Packages.rasterio
python310Packages.rasterio.dist
python310Packages.reproject
python310Packages.reproject.dist
python310Packages.unstructured
python310Packages.unstructured.dist
python311Packages.ax
python311Packages.ax.dist
python311Packages.botorch
python311Packages.botorch.dist
python311Packages.drms
python311Packages.drms.dist
python311Packages.fe25519
python311Packages.fe25519.dist
python311Packages.fountains
python311Packages.fountains.dist
python311Packages.ge25519
python311Packages.ge25519.dist
python311Packages.gpytorch
python311Packages.gpytorch.dist
python311Packages.mip
python311Packages.mip.dist
python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.pint-pandas
python311Packages.pint-pandas.dist
python311Packages.proxy-py
python311Packages.proxy-py.dist
python311Packages.pyairvisual
python311Packages.pyairvisual.dist
python311Packages.pymilvus
python311Packages.pymilvus.dist
python311Packages.pypandoc
python311Packages.pypandoc.dist
python311Packages.pytrends
python311Packages.pytrends.dist
python311Packages.rasterio
python311Packages.rasterio.dist
python311Packages.reproject
python311Packages.reproject.dist
python311Packages.unstructured
python311Packages.unstructured.dist
wayback-machine-archiver
wayback-machine-archiver.dist


Add reader support for EXT-OATCLS-SCTE35 and EXT-X-CUE-OUT

Breaking Change: SCTE struct was renamed to SCTE35
Breaking Change: MediaSegment.SCTE was renamed to MediaSegment.SCTE35
Deprecation: MediaPlaylist.SetSCTE() has been deprecated, use MediaPlaylist.SetSCTE35() instead
SCTE35 (was SCTE) has 3 new fields, SCTE35.Syntax defines the format of the SCTE data, SCTE35.CueType defines whether the cue point was start/mind/end, SCTE35.Elapsed can be used for formats that track the elapsed time.
SCTE35 now supports a another SCTE-35 syntax we've nicknamed OATCLS (due to the presence of this string in the tag), see sample-playlists/media-playlist-with-oatcls-scte35.m3u8 for an example. Both reads and writes are supported.

Closing due to major git tree issues. See #81.


Optimize Docker Image to Reduce Size; Use npm ci to get dependencies

Our current docker image is build using npm install. This currently leads to caching dependencies unnecessarily. Further, it seems that we should prefer to build using npm ci as this will fail if there are any discrepancies between the package.json and the package-lock.json and it will delete an accidentally existing node-modules directory (https://docs.npmjs.com/cli/v7/commands/npm-ci). Finally, we were adding in unnecessary files, so I update the .dockerignore file.

@zzzming - I am not familiar with nuances for node, but based on reading through https://docs.npmjs.com/cli/v7/commands/npm-ci and https://adambrodziak.pl/dockerfile-good-practices-for-node-and-npm, it seems like we should use npm ci.
Note that our docker image size goes from 165 MB to 153 MB after this change. Most of that is because we're dropping the ~/.npm/_cacache/ directory.


Failing to Login With FB/Google If Google/Fb user already exist ....

failing to login with fb/google if fb/google user already exist...
Actual Behaviour....
if some one with<EMAIL_ADDRESS>logged in with gmail and the person with same email id on fb tried to login with his fb.... Then the loggin process failing with "Find Query And Update Options Must Be Provided" error..
Expected Behaviur..
if some one with<EMAIL_ADDRESS>logged in with gmail and the person with same email id on fb tried to login with his fb.... The users existing account should be updated with fb details.... and should provide token to allow him to login...

Commit 6ec8595cde3789fd1cd60b7b4bb3f48ff23025cb resolves this issue #1


add malware object  setters and validation methods

add malware object  setters and validation methods

@jordan2175 could you please review


error in builidng docker

Hi there,
I just want to build docker and I got this error for installing gatsby :
npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=18.0.0"} (current: {"node":"12.22.12","npm":"6.14.16"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=14.14"} (current: {"node":"12.22.12","npm":"6.14.16"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=18.0.0"} (current: {"node":"12.22.12","npm":"6.14.16"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>
npm ERR! Unexpected end of JSON input while parsing near '...ZqkK80Tm1Uq1pOOfPE3cW'

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2023-06-12T13_51_02_133Z-debug.log

Hi @abb4s, thanks for your question, we'll look into it and get back to you!


promo code response

Please use Discussion board if you want get some help out of it. Please report issue if there is a bug.

Version of react-native-iap
Version of react-native
Platforms you faced the error (IOS or Android or both?)
Expected behavior
Actual behavior
Tested environment (Emulator? Real Device?)
Steps to reproduce the behavior

Hello,
I want to apply the promo code in my android/iOS app, I have successfully applied them, it's working as expected.
but I didn't get any response when applying the promo code.
actually, I want to hit my rest API on the basis of promo code status
for instance, if my promo code gets success like below, then apply this API
[myDomain]/promocode/success

if my promo code doesn't get success. then apply this API
[myDomain]/promocode/not_success


Licenses key

Hi
Licensee currently only checks the license key and ignores the old npm format licenses key - can you update licensee so that it at least makes any information under a licenses key available to anyone using licensee - you don't need to do anything with it but if its available it avoids me having to re-read package.json files to check if that key exists
many thanks

@ravjsdev Thanks for creating an issue on this before sending a PR. I don't think I'd take a PR to have licensee do this in my own branch. That's clearly a problem that the packages should fix, by moving to license. It's been many years since npm spec'd and began validating license.


OpenEphys: Allow for new stream naming convention

Fixes #1320
Replaces #1423
It seems that newer versions of openephys have filenames of xx_RhythmData_CHx instead of xx_CHx. This PR switches to using pathlib for openephys + accounts for this new naming convention + more robustly deals with missing .events files since in fact there can also be empty .events files.
We can test with these files for the new naming format.
openephys_rhythmdata_test_nodes.zip
Also adding for changes to OE docs for those who wan to read about these changes.

@chrishalcrow
Could you test if this also works for you :)

@chrishalcrow
Could you test if this also works for you :)

Works for our files 👍


Set Safai() default to Mojave and Version 12.0

Safari doesn't allow us to run on "latest". Hence, we need a default for when someone wants to do:
sauceSessions.withSafari().start();
That default should be Mojave and Safari version 12.
@titusfortner let me know your thoughts ^^

Java implements this

Ugh, this is what I get for not looking at this closer, we can't put a default on here that overrides previous inputs or requires a specific order of operations.

Replacing this with #141


How to compile under Mingw-w64?

Hi, I try compile bgfx under Mingw-w64, but without success.
I'm typing these commands:
1 .E:\Github reps\bgfx-sdk\bgfx>..\bx\tools\bin\windows\genie.exe --with-tools --with-examples --gcc=mingw-gcc gmake
2. E:\Github reps\bgfx-sdk\bgfx>mingw32-make mingw-gcc-release64
process_begin: CreateProcess(NULL, uname, ...) failed. mingw32-make: makefile:6: pipe: No error process_begin: CreateProcess(NULL, uname, ...) failed. mingw32-make: makefile:303: pipe: No error mingw32-make -R -C .build/projects/gmake-mingw-gcc config=release64 mingw32-make[1]: Entering directory 'E:/Github reps/bgfx-sdk/bgfx/.build/project s/gmake-mingw-gcc' "==== Building bgfx (release64) ====" Creating ../../win64_mingw-gcc/obj/x64/Release/bgfx Creating ../../win64_mingw-gcc/obj/x64/Release/bgfx/src bgfx.cpp process_begin: CreateProcess(NULL, C:\mingw-w64\i686-6.1.0-win32-dwarf-rt_v5-rev 0\mingw32\/bin/x86_64-w64-mingw32-g++ -MMD -MP -MP -DBX_CONFIG_ENABLE_MSVC_LEVEL 4_WARNINGS=1 -D__STDC_LIMIT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_CONSTANT_MACR OS -DWIN32 -I../../../../bx/include/compat/mingw -I../../../3rdparty -I../../../ 3rdparty/dxsdk/include -I../../../../bx/include -I../../../3rdparty/khronos -I.. /../../include -Wall -Wextra -fomit-frame-pointer -g -O3 -m64 -fno-rtti -fno-exc eptions -Wshadow -Wunused-value -fdata-sections -ffunction-sections -msse2 -Wund ef -m64 -std=c++0x -o ../../win64_mingw-gcc/obj/x64/Release/bgfx/src/bgfx.o -c . ./../../src/bgfx.cpp, ...) failed. make (e=2): Nie można odnaleźć określonego pliku. bgfx.make:380: recipe for target '../../win64_mingw-gcc/obj/x64/Release/bgfx/src /bgfx.o' failed mingw32-make[2]: *** [../../win64_mingw-gcc/obj/x64/Release/bgfx/src/bgfx.o] Err or 2 Makefile:16: recipe for target 'bgfx' failed mingw32-make[1]: *** [bgfx] Error 2 mingw32-make[1]: Leaving directory 'E:/Github reps/bgfx-sdk/bgfx/.build/projects /gmake-mingw-gcc' makefile:130: recipe for target 'mingw-gcc-release64' failed mingw32-make: *** [mingw-gcc-release64] Error 2
How fix this?

Install prerequisites: https://bkaradzic.github.io/bgfx/build.html#windows, and use make instead.

And make sure that you have added bin folder of these prerequisites in your PATH. (for example: C:\Program Files (x86)\GnuWin32\bin)
It helped me to run make inside bgfx folder in Windows 10.


[Infinite List] Stuck Initial Loading Indicator when the request data changes before loading completely

Bug Report
Description

I have a component that accepts a request data object as a prop and then renders the Infinite List. I have defined the "onRequestItems" callback which manipulates the request data object received as prop and makes an async request. The problem occurs when I'm loading a set of data, and the request data object prop to my component is changed in the midst of loading, i.e. my async request is still pending, I'm stuck with the loading indicator and the infinite list does not even invoke the "onRequestItems" callback. Ideally, since my props changed the infinite list should invoke the "onRequestItems" to load the new set of data.
Steps to Reproduce

Create component that accepts request data as a prop and renders Infinite List.
Change the request data prop to the component while the initial loading indicator still appears/request not complete.
Stuck loading indicator with no call to the "onRequestItems"

Additional Context / Screenshots

Expected Behavior

Ideally, since my props changed the infinite list should re-render and invoke the "onRequestItems" to load the new set of data.
Possible Solution

I found a workaround wherein I added a key prop to the infinite list which is tied to my incoming request data object prop to ensure uniqueness and make sure React identifies it uniquely and renders with the new data. But I'm curious if the Infinite List could handle this internally.
Environment

Component Name and Version: Infinite List ^3.0.0
Browser Name and Version: Chrome Version 77.0.3865.120
Node/npm Version: 5.5.1
Webpack Version:
Operating System and version (desktop or mobile): Desktop and Mobile

@ Mentions

Discussed this issue with @tbiethman

@prasadkhedekar
Do you have a reduced test case that illustrates this behavior? Does the code below get closer to what you have?

ProgressiveLoadingInfiniteList
import React from 'react';
import InfiniteList, { Item } from 'terra-infinite-list/lib/index';

const mockData = (pageKey) => {
  let isFinalPage = false;
  const nextPageKey = (pageKey >= 0 ? pageKey : 0) + 1;
  if (nextPageKey > 3) {
    isFinalPage = true; // If the prefix of the data item is > 3, then this is the final page.
  }

  const items = [];
  for (let i = 0; i < 2; i += 1) {
    items.push(
      {
        title: `Item ${pageKey * 10 + i}`,
        key: `unique-${pageKey * 10 + i}`,
      },
    );
  }

  return {
    nextPageKey,
    items,
    isFinalPage,
  };
};

const mockRequest = pageKey => new Promise(resolve => setTimeout(() => { resolve(mockData(pageKey)); }, 3000));

const MyInitialLoadingIndicator = () => (

 Initial Loading Indicator

);

const MyProgressiveLoadingIndicator = () => (

 Progressive Loading Indicator

);

// eslint-disable-next-line react/prop-types
const MyExampleContainer = ({ children }) => (

    {children}

);

class ProgressiveLoadingList extends React.Component {
  constructor(props) {
    super(props);
    this.requestData = this.requestData.bind(this);
    this.state = {
      isFinishedLoading: false, items: [],
    };
  }

  requestData() {
    // eslint-disable-next-line react/prop-types
    mockRequest(this.props.currentPageKey).then(response => (
      this.setState(prevState => ({
        isFinishedLoading: response.isFinalPage,
        items: prevState.items.concat(response.items),
      }))
    ));
  }

  createItems() {
    return this.state.items.map(item => (
      <Item key={item.key}>

 {item.title}

      </Item>
    ));
  }

  render() {
    return (
      <MyExampleContainer>
        <InfiniteList
          dividerStyle="standard"
          isFinishedLoading={this.state.isFinishedLoading}
          onRequestItems={this.requestData}
          initialLoadingIndicator={<MyInitialLoadingIndicator />}
          progressiveLoadingIndicator={<MyProgressiveLoadingIndicator />}
        >
          {this.createItems()}
        </InfiniteList>
      </MyExampleContainer>
    );
  }
}
export default ProgressiveLoadingList;

 ProgressiveLoadingListBase
import React from 'react';
import ProgressiveLoadingList from './ProgressiveLoadingList';

class ProgressiveLoadingListBase extends React.Component {
  constructor(props) {
    super(props);
    this.updateKey = this.updateKey.bind(this);
    this.state = { currentPageKey: 1 };
  }

  updateKey() {
    this.setState(() => ({
      currentPageKey: Math.floor((Math.random() * 9) + 1),
    }));
  }

  render() {
    return (

        <button type="button" onClick={this.updateKey}>
          Refresh
        </button>

          Data Item starts with No.
          {this.state.currentPageKey}

        <ProgressiveLoadingList currentPageKey={this.state.currentPageKey} />

    );
  }
}
export default ProgressiveLoadingListBase;

I found a workaround wherein I added a key prop to the infinite list which is tied to my incoming request data object prop to ensure uniqueness.

I wanted to know if the behavior below is similar to what you're seeing for me to troubleshoot further. If not, then I'd appreciate it if a reduced test case is provided.
Note: I wait 10 secs for my async request to finish and I assume that the list has finished loading if the data item starts with a number > 3.
without unique key:

with unique key:

Your possible solution is our expected solution for this problem. We've added an issue to document this for terra-infinite- list
https://github.com/cerner/terra-framework/issues/956


Hamdi/confsigs deposits

1.) Deposits entirely represented on the rootchain and exited/finalized separately from transactions
2.) Removed BytesUtil
3.) txBytes include deposit nonce
4.) Confirm signatures are presents and tests provide good coverage. More will be added to cover require branches.
5.) General refactors in the contract and libraries
fixes #57
fixes #36

Pull Request Test Coverage Report for Build 304

141 of 148 (95.27%)  changed or added relevant lines in 3 files are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage increased (+7.4%) to 84.266%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contracts/libraries/Validator.sol
17
19
89.47%

contracts/rootchain/RootChain.sol
101
106
95.28%

Files with Coverage Reduction
New Missed Lines
%

contracts/rootchain/RootChain.sol
8
82.29%

Totals

Change from base Build 243:
7.4%

Covered Lines:
179

Relevant Lines:
196

💛  - Coveralls

Can you update version numbers in the contribution guidelines?


Makes the alert visible even if scrolled down

Solves the issue #515

Makes the alert visible even if scrolled down by fixing it at top.
Adds a cross button to close the warning if hindering the view

Screenshots-
Wider Screen

Smaller screen

Automated message from Dropbox CLA bot
@disissid, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here and update the thread so we can consider merging your code.

@timabbott Please review. And I'm not sure about this Travis error, I guess its a network error, right?

@disissid Run "tools/lint-all" inside the vagrant and see if any errors come up. This might help with the TravisCI error. :)

@disissid sure looks like a network error; Travis CI was having a lot of those for a few days.
Thanks for working on this!  I posted a bunch of comments; if you can fix those before the weekend I can do a complete review and maybe merge this then.

@timabbott Fixed them, review if anything else need to be changed.

Looks like there's an issue with the error message bar with a very wide window (this was taken on a 24" monitor):

I think how this should work from a design perspective is that the error bar should be inside the center pane, and ideally would be setup in a way to not overlap with the "Streams" heading for the top of the page...

For overlapping the "Streams" heading, div is fixed so it is positioned relative to viewport, To fix it I
increased the bottom margin of "Streams" heading which doesn't look bad without warning

With warning

This can overlap other stream input div if warning would extend to next line but that would be very rare case.
I could've checked if user has scrolled down or not but that would be unnecessarily keeping a check for event that would happen rarely.(I suppose with this approach I can debounce the function)

OK, that's reasonable for that piece.

@timabbott Done.

Hmm, I'd prefer the issue for larger streams be fixed by just making the error bar a part of the middle panel, rather than adding this sort of CSS based on the current size of the elements in the middle panel -- can you do something of that form instead?

Ah, again the error bar's position is fixed now, which is positioned relative to the viewport. BTW what is wrong with this approach?

The problem with this approach is that it's complex (in terms of how much code is involved) and somewhat brittle -- if we were to change the design to make the center panel 20% larger, this code would break.  Whereas if the error bar element just had the width of the center panel by construction, it'd be less code and not subject to that issue.

Okay, that makes sense, I had to change explicitly for different screen sizes. Now I've set its width with respect to the middle column. Please review @timabbott.

@timabbott What do you think now?

Great, that's roughly what I had in mind!    If you're scrolled down in the page when you click a subscribe/unsubscribe button, it scrolls you to the top (I assume with the intent of showing you the error bar), which is no longer necessary and a bit confusing since it obscures what it was that you actually clicked.  Can you look into fixing this as part of this change?

@timabbott Squashed the commits and fixed the issue. Actually it was scrolling up to focus on stream input but I removed it anyway as you said it was confusing since it obscures what it was that you actually clicked.

OK, I tested this around for a bit, and I think it works correctly; it's be good to address the report_* comment issue.  Also, @disissid can you fix the commit message?  It shouldn't just have a list of the commit messages of all your various test commits, but instead a description of what a developer seeing this change might want to know to understand why it is correct.
Also, the commit message should have a sentence of the form Fixes: #515. so it automatically closes the original issue when it gets merged.
@wdaher do you want to take a quick look at the CSS here?
Here's a screenshot:

I feel like we could make this look better, but I'm not sure what the best approach is.  Maybe move the error message to sit below the search box rather than hovering over the scrollable pane instead?

My view is that the screenshot you show, Tim, is better than nothing but
pretty ugly.

Maybe move the error message to sit below the search box rather than
hovering over the scrollable pane instead?

This seems promising to me. What do we do today when you lose your network
connection, and can we borrow from that logic?
On Sun, Mar 27, 2016 at 10:08 PM, Tim Abbott<EMAIL_ADDRESS>wrote:

OK, I tested this around for a bit, and I think it works correctly; it's
be good to address the report_* comment issue. Also, @disissid
https://github.com/disissid can you fix the commit message? It
shouldn't just have a list of the commit messages of all your various test
commits, but instead a description of what a developer seeing this change
might want to know to understand why it is correct.
Also, the commit message should have a sentence of the form Fixes: #515.
so it automatically closes the original issue when it gets merged.
@wdaher https://github.com/wdaher do you want to take a quick look at
the CSS here?
Here's a screenshot:
[image: image]
https://cloud.githubusercontent.com/assets/2746074/14071320/365822f6-f468-11e5-92c7-ce560c30b215.png
I feel like we could make this look better, but I'm not sure what the best
approach is. Maybe move the error message to sit below the search box
rather than hovering over the scrollable pane instead?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/pull/534#issuecomment-202236303

When you lose your network connection, we display this in the buddy list sidebar:

(there's 2 possible types of errors, and it appears you get both here in the unusual case that it loses the connection while the page is loading; which probably could use improvement in its own right).  But that's for displaying an error with the overall state as opposed to one caused by a particular UI widget; most of our UI errors display as a text bar at the top of the relevant widget.

Yeah, that's not good enough in this case, agreed.
On Sun, Mar 27, 2016 at 10:35 PM, Tim Abbott<EMAIL_ADDRESS>wrote:

When you lose your network connection, we display this in the buddy list
sidebar:
[image: image]
https://cloud.githubusercontent.com/assets/2746074/14071634/dd4a3dbc-f46b-11e5-8134-367ea16e8a55.png
(there's 2 possible types of errors, and it appears you get both here in
the unusual case that it loses the connection while the page is loading;
which probably could use improvement in its own right). But that's for
displaying an error with the overall state as opposed to one caused by a
particular UI widget; most of our UI errors display as a text bar at the
top of the relevant widget.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/pull/534#issuecomment-202241816

OK, @disissid, are you up for making an attempt at having the error message sit below the search box instead?

Yeah, @timabbott I'll do it and update the PR by tomorrow.

@wdaher @timabbott  What do you think now?

Looks good to me.
Two comments:

Can you vertically center the 'x' where it is? Right now it looks like
it's floating a bit towards the top, which looks kind of funny to me.

In screenshot 2, if you click the 'x', does it actually click the x and
stop or does it trigger the click of the 'v'?

On Sun, Apr 3, 2016 at 10:17 AM, Kumar Siddharth<EMAIL_ADDRESS>wrote:

@wdaher https://github.com/wdaher @timabbott
https://github.com/timabbott What do you think now?
[image: screenshot from 2016-04-03 21 27 42]
https://cloud.githubusercontent.com/assets/6582951/14233717/f659208e-f9ed-11e5-9019-9d712709d0f8.png
[image: screenshot from 2016-04-03 21 28 00]
https://cloud.githubusercontent.com/assets/6582951/14233718/f68921c6-f9ed-11e5-8cde-f5bb68ba154b.png
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/pull/534#issuecomment-205013575

Hi @wdaher

That was intentional, I agree it doesn't look good when error is of one line but for narrow screens I don't think we would want to center the 'x' vertically.

Ah, I can center it vertically for only wider screens.

Clicking the 'x' won't trigger the 'v'.

Great point, in the multi line case you definitely don't want to center it.
I am fine with either selectively centering when it's one line using CSS,
or just leaving it as written, whatever feels most consistent with what we
do elsewhere in the app.
Thanks!
On Mon, Apr 4, 2016 at 12:47 AM Kumar Siddharth<EMAIL_ADDRESS>wrote:

Hi @wdaher https://github.com/wdaher

That was intentional, I agree it doesn't look good when error is of one
line but for narrow screens I don't think we would want to center the 'x'
vertically.

[image: screenshot from 2016-04-04 13 10 38]
https://cloud.githubusercontent.com/assets/6582951/14241079/3bc11c8e-fa67-11e5-9726-585f4024386f.png
Ah, I can center it vertically for only wider screens.

Clicking the 'x' won't trigger the 'v'.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/pull/534#issuecomment-205178169

To address @wdaher's point, @disissid what do you think about tweaking the CSS for the close button to be placed at an offset from the top that would be centered if it takes up 1 line but not if it takes up 2?

Otherwise this looks great!  @disissid ping me once you've addressed those and I'll merge.
Thanks for working through making this look good @disissid !

@wdaher @timabbott I won't suggest centering the close button as its height depends on the content of subscritptions-status div, so it would require unnecessary logic to see when the height of the div is changed and anyway current UI doesn't look that bad to me.

OK, I did a small bit of cleanup (e.g. cleaning up the commit message to match our commit messge style and fixing a few whitespace issues) and merged this as 4eced6922832d0494b30709d2566790b6db80a23.  Thanks for building this and tweaking the details of get this right @disissid!


pointclouds from two kinect camera not mismatch，especially hand

I promise that cabiration is correct，anyone can slove it？

Not sure what you are asking?


Add VTTCue settings feature

This PR creates two new features for the WebVTT API.  These two features were originally suggested as comments in the webvtt feature, so this PR does just that!

I opened https://github.com/web-platform-dx/web-features/pull/2310 to deal with the constructor

Should this feature also include css.selectors.cue and css.selectors.cue.selector_argument?

Should this feature also include css.selectors.cue and css.selectors.cue.selector_argument?

Added along with a compute_from.


[$2000]  Build a Prestashop Plugin for Hyperswitch

Read this wiki to know how to Participate
Please make sure to read this through the below Guidelines before attempting the problem statements
Contribution Guidelines
Bounty Program Guidelines
Terms of Contest
Introduction
Hyperswitch is an open-source, high-performance payment switch designed to unify diverse payment methods. By integrating Hyperswitch with PrestaShop, we aim to provide e-commerce merchants with a flexible, cost-effective payment solution that enhances user experience and improves conversion rates.
Objectives

Develop a PrestaShop module that seamlessly connects to Hyperswitch's API
Enable merchants to process payments through various providers supported by Hyperswitch
Implement essential payment flows (authorization, capture, refund, etc.)
Ensure robust error handling and logging
Provide a user-friendly configuration interface within the PrestaShop admin panel

Requirements

The module should adhere to PrestaShop's development standards and best practices
Utilize Hyperswitch's API documentation for integration
Implement necessary security measures for handling sensitive payment data
Ensure compatibility with PrestaShop versions 1.7.x and 8.x
Support multi-language and multi-currency functionality

Deliverables

A fully functional PrestaShop module for Hyperswitch integration
Comprehensive documentation including:Installation guide
Unit and integration tests
Sample storefront implementation showcasing the payment flow

Architecture
The integration will consist of three main components:

Payment Module: Handles payment processing within PrestaShop.
Admin Configuration: Provides an interface for merchants to configure Hyperswitch settings.
Webhook Handler: Processes asynchronous notifications from Hyperswitch.

Implementation Details
Payment Module

Implement as a PrestaShop payment module
Handle payment creation and processing
Communicate with Hyperswitch API for payment operations
Map Hyperswitch responses to PrestaShop order statuses
Implement error handling and logging

Admin Configuration

Create a configuration page in the PrestaShop admin panel
Allow merchants to input Hyperswitch API credentials
Provide options to configure payment methods and settings
Implement validation for configuration inputs

Webhook Handler

Create an endpoint to receive webhooks from Hyperswitch
Process incoming notifications and update corresponding orders in PrestaShop
Implement security measures to validate webhook authenticity
Ensure idempotency to handle duplicate notifications

Flow Diagram

Resources

PrestaShop Module Development Guide
PrestaShop Payment Module Creation
Hyperswitch API Documentation
PrestaShop GitHub Repository
Hyperswitch GitHub Repository

Best Practices

Use PrestaShop's built-in classes and helpers for database operations and front-end rendering
Implement proper error handling and logging using PrestaShop's logging system
Ensure all sensitive data is properly encrypted and handled securely
Follow PrestaShop's coding standards and naming conventions
Implement comprehensive testing, including unit tests and integration tests
Provide clear, multi-language support for both admin and front-end interfaces
Use PrestaShop's built-in translation system for all user-facing text

Next Steps

Set up a development environment with PrestaShop and Hyperswitch
Design the detailed module structure and data flow
Implement the Payment Module
Develop the Admin Configuration interface
Create the Webhook Handler
Develop comprehensive tests
Create documentation in multiple languages
Perform thorough testing across different PrestaShop versions
Submit the module for PrestaShop Addons marketplace review

Hey, I'm interested in doing this. Is it still needed?

Can i work on this please?


Implement "rules-based snapping" for lines

comically bad UX
video

Does this take into account lines that are more than 5ft wide? Looking at the code I am guessing it will cause issues for any lines with an even multiple of five.


Improve keyboard_state error messages

Addresses #1339 and #1383.

I can rebase these changes to remove the noise commits if you prefer.

Nope that's ok, thanks!


Alerts cannot be non cancelable

val pwAlert = alert {
    titleResource = R.string.my_message
    messageResource = R.string.my_message
    okButton {}
    isCancelable = false // This does not exists
}

There is currently no way to create an alert with the alert function and set the dialog to be non cancelable.
I am "forced" to revert to java style creation when I need a non cancelable dialog.
BTW, it would also be great if there was a way to access the AlertDialog.Builder.

A much needed feature

Before the new feature add to the library, you can try this:
val pwAlert = activity.alert(Appcompat) {
    titleResource = R.string.my_message
    messageResource = R.string.my_message
    okButton {}
}.build()
pwAlert.setCancelable(false)
pwAlert.setCanceledOnTouchOutside(false)
pwAlert.show()

@doitlite  What Appcompat here? please explain more about it

In version 0.9 I was able to call cancellable(false). It was removed from builder in 0.10 I wonder if there was a reason for that or was it forgotten during refactoring. It should be easy to restore if there is no reason behind it.

Hi , You can use  after   ".show()"   .setCancelable(false)
for example ->
alert("Body") {
title = "Title"
positiveButton("ok") {
}
}.show().setCancelable(false)
https://gist.github.com/BurakDizlek/4b026b3c8a736987902a7e65f73dc6da

I want to save dialog after screen rotation. but @doitlite and @Zhuinden don`t helps.
I hope the best way is to use ViewModel


Draw.io doesn't keep my settings and custom libraries

Hello guys.
Draw.io is fantastic. That's why I started using it for many purposes. But I can't save my preferences to the layout of the app.
I tested the two versions, installed and portable (Draw.io 10.6.8) but every time I start the program my configuration is reset. The window always starts 4:3, the toolbar expanded and I have to import my custom libraries again.
I searched for a .ini or .xml file but found nothing in the portable root folder or in 'AppData\Roaming\draw.io'.
Is there any way to create a .xml or .ini file and setup my preferences for the application and libraries?
Thanks a lot.
Screencast showing the problem - 1m03s

Why closed? If it's the case, it's valid.

I was about to open new issue but stumbled on this while searching for possible solutions. In my case, I would like to be able to maintain a single configuration file and multiple custom libraries for multiple users on a shared network. Preferably, no user interaction would be required to set this up (deployed by IT for example).
Accomplishing these things are documented for users of the online app or Confluence version, for example here and here.
Possible solutions:

Config file to have a default location when using the desktop app.
Config variables, e.g. %CustomLibraryPath% in order to use local paths for library files. Perhaps allow this variable to be defined in the config itself.

The best alternative I have right now is diagram showing users how to add the custom configuration by opening a file, copying it to clipboard, pasting into the Draw.io desktop app, and relaunching. Then opening custom libraries one-by-one (this one could just be me needing to read the docs more closely– I haven't had success loading a library XML from a URL, but could probably encode and then inline with the config).


Use class reference to configure authentication-provider

It is safer to use the SpringSecurityAuthenticationProvider class instead of the qualified class name string for the case that this code is copied and the package where SpringSecurityAuthenticationProvider resides is changed.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add CTRL+A to select all downloads

for keyboard lovers.
1 CTRL+A to select all downloads
2.

Yes, I'd also like to have this capability. It would be a huge improvement to having to delete completed downloads 1 at a time.

It would be a huge improvement to having to delete completed downloads 1 at a time.

You don't have to remove completed items one at a time, you can right-click on the queue and select Clear completed, which removes all completed items.

Thanks for the info.


Crash during saving if any page reports as "invalid drawing"

Program crashes during saving if any page reports as "invalid drawing"
Traceback (most recent call last):
  File "drawing_splitter.py", line 129, in <module>
    save_drawings(filename, num_of_pages, drawing_numbers, args.output)
  File "drawing_splitter.py", line 64, in save_drawings
    output_filename = drawing_numbers[page_number] + '.pdf'
IndexError: list index out of range

This needs to be handled so that any valid drawings can be saved with the correct name. There is an issue with valid drawings being saved but with the incorrect name, then the program crashes.
Invalid drawings should be skipped over.

If multiple files are being processed where the drawing number cannot be found, these pages may be overwritten when saved. "drawing_x.pdf" is too generic. Suggest "original_filename_x.pdf" instead.


Add and document analytics

I want a good (privacy safe) analytics solution so I can gather (and make public) information about the usage of 18xx Maker.
What I would like to collect:

app vs website usage
screen size data
what features users have used
what versions of the app are in use
country/state information (nothing more granular than this if possible)

Requirements for me:

No information collected or gathered about what files/games people are using or working on
No personal information about users (other than what's listed above)
No analytics collected at all from users developing / using the code. Only from the official app and the production websites.

So for example. I DO want to see that you loaded a game into the app from your filesystem and then exported the PDFs of that game. I DO NOT want to know who you are, or what game it was.
This will be documented on the site / in the app and I want all data to be public at analytics.18xx-maker.com.

I’m not sure whether it meets all your requirements but Cloudflare offers a non-cookie analytics options that’s pretty easy to use.
Matomo is the only other option I’ve heard anything specific about and they seem to be more GA comparable.

Right now I'm really liking the self hosted option for https://github.com/plausible
If we need to in the future we can pay to host it, but I have some VPS's I can put this on for now that will probably be fine for... years for the traffic we get. I can also use their API directly and not use any automatic tracking so that it can be fully documented in one file in our repo. Self hosting makes it REALLY easy for me to make the dashboard public as well.
I just read this: https://plausible.io/vs-cloudflare-web-analytics
I do like Cloudflare's values but currently they only support 6 months of data and no custom events which would make getting easy to view analytics of custom app functions difficult.

Nice. Yeah the self-hosting option is a great fit for making a public dashboard.

https://analytics.18xx-maker.com/18xx-maker.com/
Will continue to improve it tomorrow.


Improve performance of metric chart updates

We have a timer that fires every 200ms. On each tick it gathers the full series of data for the chart, creates buffers for it, marshals it across to JS, then calls Plotly.update.
This causes high CPU, and causes chart updates to be a little jerky.
For dashpages, where we will show many charts on screen at once, this overhead becomes multiplied.
Instead, we should use Plotly.extend to append new data items, only when they exist, and in only on the JS side should we have an animation loop that calls Plotly.relayout with updated y-axis coordinates.
With this we avoid many invocations from Blazor to JS, cutting CPU. We can then likely increase the rate of updates for the chart from 5fps to 30fps or even 60fps, making the animation very smooth.

Will wait for dashpages work to merge before tackling this, as dashpages make a lot of changes around charting.


Error: Missing required parameter: auth

Followed your running the project locally step by step. I am stuck on the below error
#200 already went through this. Double checked my github client id and secret key and they are fine

 error Error: Missing required parameter: auth
[0]     at new Replicate (/home/gaurav/openpipe/node_modules/.pnpm/replicate@0.12.3/node_modules/replicate/index.js:37:13)
[0]     at eval (webpack-internal:///(api)/./src/modelProviders/replicate-llama2/getCompletion.ts:13:19)
[0]     at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
[0]   digest: undefined
[0] }
[0] - warn ../node_modules/.pnpm/@babel+standalone<EMAIL_ADDRESS>[0] Critical dependency: the request of a dependency is an expression
[0]

Login seems to be successfull
What am I missing? because of the above auth error I am not able to use the app (seed and project name edit fails)

Update on this: It seems that I was missing Replicate Token API and after that it ran successfully

Yeah @DVGY07, if you could submit a PR to make the replicate token optional that would be awesome.

Sweet, it’s merged. Thanks for the PR!


pull request for e2e test at 12/06/2016 00:56:00

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add time to frame tracking to hot run

[x] Add FirstFrameTimer and wire it up around restart 'R' and hot reload 'r'.
[x] Print the full time from key press to first frame after a restart / hot reload

This is the same number that we will report on benchmarks. Example output:
Hot reload time to frame: 0.69 seconds

Restart time to frame: 3.71 seconds

@sethladd @devoncarew

lgtm w/ some suggestions for verbiage

Consider using ms to set expectations for these to be small. :)  It's also possible that seconds is the correct expectation.

I like the suggestion to use ms. :)


Installation fail with zfs storage

Hello,
when I install it with lvm it works fine, installing with zfs, I get following error:
Installing Docker...
Configuring Docker...
Job for docker.service failed because the control process exited with error code.
See "systemctl status docker.service" and "journalctl -xe" for details.
[ERROR] 1@192 Unknown failure occured.

same here, i only have ZFS
updating container OS...
Installing prerequisites...
Installing Docker...
Configuring Docker...
Job for docker.service failed because the control process exited with error code.
See "systemctl status docker.service" and "journalctl -xe" for details.
[ERROR] 1@130 Unknown failure occured.


Generate a SET IDENTITY_INSERT statement in MergeUI

Issue:
When generating a MERGE statement for tables with IDENTITY columns , SET IDENTITY_INSERT is not included in the generated script.
Repro:
Add a table with identity field in MergeUI and click save, generated script doesnt have SET IDENTITY_INSERT
Work Around:
Manually adding the statement

When I was originally writing it I thought about this probably a little too much! I decided that mergeui was to handle reference data which shouldn't really have an identity column as it would normally be to match enum's etc in app code.
I think for a better way to handle more complicated solutions then the tool to use is sp_generate_merge - I use that mostly and it works very well - mergeui gets used when it is simple columns etc.
sorry!


Struggle with high resolution

Hey, thanks again for this great plugin. Unfortunately I am usually dealing with a resolution of ~16k to 1k and then it stops working. With this resolution it usually just remains completely blank, sometimes not even loading the parameters, but with the most simple example tox i worked my way up to a black canvas. The error message I get from resolume sometimes (but not the last times) was constantly: "FFGL: Failed to create interop".
I am using Touchdesigner version 2023.11600 with educational license and licensed resolume version 7.17.3.
Here is a link to the projects and toxes:
https://drive.google.com/drive/folders/14vDnhmOha9E2KCWaJ_oqW3xUL2YCIux0?usp=sharing
(I know this is quite an unusual request, so I would also be happy to buy you more coffee for this, just reach out to me at Kofi)

Ok this is what I suspected it could be, I believe this may be to a texture size limitation in the interop library I am using. I will test and report back with what I find.

Hello, so I believe this issue may be with touch engine, I am seeing the texture access being returned as true but the texture is null. I would put an issue in the touchengine-win repo.

Ok thanks for much for your help, i put an issue out and will keep you posted :)

Feel free to reopen this issue if you find it isn't resolved from the derivative end.


Support for class access modifier on DTO classes?

Hi!
NSwagStudio currently support setting "Client class access modifier", but how can I change the class access modifier for DTO classes?
Kind regards, Øyvind.

Currently not possible, please create an issue in http://njsonschema.org

@RSuter, thanks for your quick response! I have added a question here https://github.com/RSuter/NJsonSchema/issues/509.

Released: v11.9.3

SWEET, tested it now, works like charm! Thank you so much for your help. Impressed that you fixed it so fast! :-)


Integrate: Get phone number

Use the already implemented dummy component related to phone number to integrate with firebase auth.
We will be using firebase phone auth, that will handle the sms for us. Please follow the following link on how to do this. https://firebase.google.com/docs/auth/web/phone-auth

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.
Belonging to Epics:

[EPIC] #326 - View: Get Phone Number

The integration is already done I believe the mobile adaptation of the view is currently being done. Shall I close this issue @s0 @jpmarques66 ?

Yes, if it's just view work left let's close this.


[BLIP2] The attention mask for visual tokens in causal LM

It seems we are passing attention masks of shape [batch, seq_len] for visual queries even in causal LM such as opt/vicuna.
This is fine for T5 model because T5 has a bi-directional encoder. But casual LM will create casual mask automatically unless explicitly passing in attention mask of shape [batch, seq_len, seq_len]. It will be more sensible for all visual queries to see each other, which also aligns with the paper.

Great

We have experimented bi-directional attention on the visual embeddings into the causal LM, but found no improvement.
Thanks.

Thank you for your response


Add /* when routes really need to match the whole path.

Example:
mainRouter.route("/static") -> mainRouter.route("/static/")
The documentation doesn't reflect what's in the vertx-example repo code.

Hi Michel,
The asciidoc is generated from the text in the package-info.java file. Could you make the changes in there?
Also.. can I check if you have signed the Eclipse CLA?

Tim,
The fixes were on the examples, so I've fixed the Examples.java file.
I've rewrote my commit with a proper author and signed it off. I think I've signed the CLA a long time ago…
One question though: shall I include the result of the documentation generation in the commit or just changes to the master file?

Please include any changes whether generated or not :)

All right, this should be better now: Java, Groovy and JS docs are now updated.
Is this OK for you?

Michel,
The PR appears to have conflicts now. I think you need to rebase.

:-) all good now

Thanks