Is this still being updated?

Are you still working on this?

Out of curiosity, Is Tax Invoice harder than the simplified tax?

Thanks

Check my fork, I am preparing a pull request but will take some time.
On Sun, 30 Apr 2023 at 16:45, sudo sand @.***> wrote:

Are you still working on this?

Out of curiosity, Is Tax Invoice harder than the simplified tax?

Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CJ3PQFLA4SIX5THLN3XDZGEJANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

from usage end, No.
You just need to provide customer details and tax details of customer.

Hello @fuadhasni is your pull request contains the implementation for the Tax invoice ??

Not the pull request but the fork
On Thu, May 4, 2023 at 4:47 PM Mustafa @.***> wrote:

Hello @fuadhasni https://github.com/fuadhasni is your pull request
contains the implementation for the Tax invoice ??
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39#issuecomment-1534628704,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CMYO2ASLZJUPGWWKSLXEOJMLANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you were mentioned.Message ID:
@.***>

If you’re outside KSA, your best bet is to use a VPN that gives you KSA IP.
On Wed, 24 May 2023 at 16:17, sudo sand @.***> wrote:

Hey @fuadhasni https://github.com/fuadhasni, I tried running the example
https://github.com/fuadhasni/zatca-xml-js/blob/main/src/examples/full.ts
and I got this error: getaddrinfo EAI_AGAIN gw-fatoora.zatca.gov.sa
Do you know whats causing that?
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39#issuecomment-1560932521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CLHGMR37H2EE6Z3LWDXHXU4LANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Looks like it.
On Thu, 25 May 2023 at 11:16, sudo sand @.***> wrote:

If you’re outside KSA, your best bet is to use a VPN that gives you KSA IP.
Do you know why? Are they blocking the request?
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39#issuecomment-1562338204,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CKOEIDGUO2BPVVEETLXH32KRANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you were mentioned.Message ID:
@.***>

okay, I will enable it.
On Sun, 11 Jun 2023 at 03:26, sudo sand @.***> wrote:

@fuadhasni https://github.com/fuadhasni can you enable the issues in
your fork? (Settings > Features > check Issues)
I got a 400 bad request, when I tried to validate on the website I got
this error:
Valid : false
Errors
category : SIGNATURE_ERROR
code :X509IssuerName
message : wrong X509IssuerName
category : SIGNATURE_ERROR
code :X509SerialNumber
message : wrong X509SerialNumber
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39#issuecomment-1585835832,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CLAI4AOF4DTIXNNBU3XKTX7VANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Enabled.
On Sun, Jun 11, 2023 at 2:30 PM Fuad Ahmed Hasni @.***>
wrote:

okay, I will enable it.
On Sun, 11 Jun 2023 at 03:26, sudo sand @.***> wrote:

@fuadhasni https://github.com/fuadhasni can you enable the issues in
your fork? (Settings > Features > check Issues)
I got a 400 bad request, when I tried to validate on the website I got
this error:
Valid : false
Errors
category : SIGNATURE_ERROR
code :X509IssuerName
message : wrong X509IssuerName
category : SIGNATURE_ERROR
code :X509SerialNumber
message : wrong X509SerialNumber
—
Reply to this email directly, view it on GitHub
https://github.com/wes4m/zatca-xml-js/issues/39#issuecomment-1585835832,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPK2CLAI4AOF4DTIXNNBU3XKTX7VANCNFSM6AAAAAAXQ3X7PA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Awesome books: with ES6

In this project, I re-structured the Awesome books app code (📓📖📗📘📙📒). The goal was to make it more organized by using modules. I also practiced the ES6 syntax that I have learned.
Following the project requirements:

Setup the project:

Create a new directory and initialize it with git.
Use npm init -y command to create package.json file.
Create the entry point for your JavaScript code called index.js
Create index.css file for your styles.
Create the main index.html file. Link your index.js and index.css files there.
Create a directory called modules.

Re-create your Awesome books app following the rules:

Use ES6 syntax.
Divide your code into modules.

Create a separate file for each module in the modules directory.
Import modules into the index.js file. (Tip: the more modules the better!)

Keep all functionalities of the app without errors.
Refactor your functions to arrow functions.
Use let and const in the correct way.

Display the current date below the navigation bar.
- For this requirement you should use a library [Luxon](https://moment.github.io/luxon/#/install?id=es6)
- Download the library into your project using npm.
- Import it as a module.

I appreciate your kind feedback.

Thank you @Reem-lab !!!


Remove BLAS/LAPACK wrapper in favor of the JavaCPP Presets

WIP: DO NOT MERGE FOR NOW
Most of the small helper functions in C++ have been translated to Java and moved to https://github.com/deeplearning4j/nd4j/pull/1456 but I had to keep in C++ the code for getVendor()/setNumThreads() originally written by @raver119 and placed it here in the presets:
https://github.com/bytedeco/javacpp-presets/blob/master/openblas/src/main/resources/org/bytedeco/javacpp/blas_extra.h

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. No test results found.

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/178/

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished. No test results found.

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/190/


[FLINK-21077] Introduce SlotPoolService to abstract SlotPool from JobMaster

What is the purpose of the change
This commit allows to support different SlotPoolService implementations which can
accommodate different slot pool implementations (e.g. SlotPool, DeclarativeSlotPool, etc.).
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review @rmetzger. Merging this PR now.

Thanks for the review @rmetzger. Merging this PR now.


Propose to convert several issues to Discussions

Now that we have a new Discussion tab, I propose to move some of the issues there. The 'Intent Analysis' items were all intended to be discussions for example.
@crissancas @sergiofranciscoortiz @ThomasEdgeXR @gunjald any objection? I can do the moving.

Full support, makes sense! Thanks @Kevsy!


CUDA device for training

Hello, I'm trying to train a kan net for OCR. In the process I had to do a few tweaks for being able to use the GPU.
device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
#device = 'cpu'
print(device)
print(torch.cuda.is_available())

torch.set_default_dtype(torch.float64) inside __init__.py otherwise I get
RuntimeError: expected scalar type Double but found Float
also this:
model = KAN(width=[16*16, 128, 1], grid=3, device=device, k=3)
model.to(device)
results = model.train(dataset, opt="LBFGS", steps=20, device=device)

and all data
dataset['train_input'] = torch.from_numpy(train_images).to(device)
dataset['test_input'] = torch.from_numpy(test_images).to(device)
dataset['train_label'] = torch.from_numpy(train_labels).to(device)
dataset['test_label'] = torch.from_numpy(test_labels).to(device)

this is the entire script:
from kan import KAN, create_dataset
import matplotlib.pyplot as plt
import matplotlib
from sklearn.datasets import make_moons
import torch
import numpy as np
import os
import utilities as ut
import zipfile

matplotlib.use('TkAgg')

TARGET_WIDTH = 16
TARGET_HEIGHT = 16
TARGET_DEPTH = 1

lib = ['x','x^2','x^3','x^4','exp','log','sqrt','tanh','sin','abs']

dataset = {}

#read label.txt file each row is a label
labels = ut.readLabels("ocr_mix/labels.txt")

print("labels: ", labels)

#read zip file and read each image from each subfolder named as label index

train_images = []
train_labels = []
test_images = []
test_labels = []

#insert path to zip file
found = False

archive = zipfile.ZipFile("./ocr_mix/ocr_mix.zip", 'r')

print("Reading images...")

(train_images, train_labels, test_images, test_labels) = ut.readImagesArchive(archive, len(labels), 500, 500, 20)

print("Preparing dataset...")

train_images = ut.prepareDataset(train_images, TARGET_WIDTH, TARGET_HEIGHT, TARGET_DEPTH)
test_images = ut.prepareDataset(test_images, TARGET_WIDTH, TARGET_HEIGHT, TARGET_DEPTH)

#flatten dimensions
train_images = train_images.reshape(train_images.shape[0], -1)
test_images = test_images.reshape(test_images.shape[0], -1)

device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
#device = 'cpu'
print(device)
print(torch.cuda.is_available())

#add second dimension
train_labels = np.expand_dims(train_labels, axis=1)
test_labels = np.expand_dims(test_labels, axis=1)
train_labels = np.asarray(train_labels)
test_labels = np.asarray(test_labels)

dataset['train_input'] = torch.from_numpy(train_images).to(device)
dataset['test_input'] = torch.from_numpy(test_images).to(device)
dataset['train_label'] = torch.from_numpy(train_labels).to(device)
dataset['test_label'] = torch.from_numpy(test_labels).to(device)

model = KAN(width=[16*16, 128, 1], grid=3, device=device, k=3)
model.to(device)
results = model.train(dataset, opt="LBFGS", steps=20, device=device)# metrics=(train_acc, test_acc))

model.save_ckpt('ckpt1')

model.plot()

After that the GPU goes on even if it is still very slow...

This should be fixed with https://github.com/KindXiaoming/pykan/pull/98 (please try)

I'll give you some news: RuntimeError: expected scalar type Double but found Float
was my fault because the normalized images were saved in dtype=float64
About the device choice for training, now CUDA works fine without model.to(device) but forcing CPU gives an error:
Traceback (most recent call last):
  File "/root/Progetti_GIT/OCR/kanocr.py", line 86, in <module>
    results = model.train(dataset, opt="Adam", steps=3, save_fig_freq=0, batch=16, device=device)# metrics=(train_acc, test_acc))
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/kan/KAN.py", line 898, in train
    self.update_grid_from_samples(dataset['train_input'][train_id].to(device))
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/kan/KAN.py", line 243, in update_grid_from_samples
    self.forward(x)
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/kan/KAN.py", line 311, in forward
    x_numerical, preacts, postacts_numerical, postspline = self.act_fun[l](x)
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/miniconda3/envs/tf/lib/python3.9/site-packages/kan/KANLayer.py", line 176, in forward
    y = self.scale_base.unsqueeze(dim=0) * base + self.scale_sp.unsqueeze(dim=0) * y
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

The problem is in line 126 in KANLayer.py:
        if isinstance(scale_base, float):
            self.scale_base = torch.nn.Parameter(torch.ones(size, device=device) * scale_base).requires_grad_(sb_trainable)  # make scale trainable
        else:
            self.scale_base = torch.nn.Parameter(torch.FloatTensor(scale_base).cuda()).requires_grad_(sb_trainable)

It is forcing .cuda() even if you are using CPU. A temporary solution could be this:
        if isinstance(scale_base, float):
            self.scale_base = torch.nn.Parameter(torch.ones(size, device=device) * scale_base).requires_grad_(sb_trainable)  # make scale trainable
        else:
            self.scale_base = torch.nn.Parameter(torch.tensor(scale_base, device=device)).requires_grad_(sb_trainable)

Could you please take the time to review https://github.com/KindXiaoming/pykan/pull/98? Because it does exactly what you mentioned and more. Probably you could have saved some migraine!

Could you please take the time to review #98? Because it does exactly what you mentioned and more. Probably you could have saved some migraine!

Wooops, sorry XD. Yes, it does work!

CPU:
train loss: 1.85e-01 | test loss: 1.90e+04 | reg: 1.41e+05 :  33%|█▎  | 1/3 [00:18<00:36, 18.19s/it]
CUDA:
train loss: 1.83e-01 | test loss: 2.18e+04 | reg: 1.42e+05 :  33%|█▎  | 1/3 [00:44<01:28, 44.34s/it]
Other than this, it works just fine.

Yeah we already noticed the slowness of CUDA for small datasets. I didn't have time to test it on big datasets yet, but I think it's actually due to a part of the code where memory is passed forcefully to CPU for performing lstgs. I will open another issue/PR as soon as I have more info and time to dedicate :)


1.x branch: fix test server for node 8 release

Node 8 http server now supports Keep-Alive connection pooling, one of the tests use keep-alive agent which causes server.close() to timeout mocha.
I will submit a PR for 2.x branch when available...

Codecov Report

Merging #286 into 1.x will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            1.x   #286   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files         6      6
  Lines       334    334
  Branches     84     84
===================================
  Hits        334    334

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 cf71a1b...a85e751. Read the comment docs.


https://developer.mozilla.org/en-US/docs/Learn/JavaScript/First_steps/Math

# Request type

[] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x ] Correction or update

Details

7**7=2401 NOT 343.

7**7 is actually 823543 — it is the same as 7*7*7*7*7*7*7, and also the same as Math.pow(7,7).
7**3 returns the same as Math.pow(7,3), which is 343. Try them in your JavaScript console.

I'm agree that this is "the" functionnement, but the explication, here, is not available:

"C'est l'équivalent du nombre de base multiplié par lui-même le même nombre de fois que l'exposant"
that is your explication. But the truth is ... i'm sorry, i'm a french man and i prefer to write in french
La vérité c'est que le nombre n'est pas multiplié par lui même le même nombre de fois que l'exposant. Regardez bien; il est multiplié que 6 fois dans l'exemple 7777777 vous n'avez que 6 signes multiplicateur (). Et dans l'exemple, 343, soit 777, il n'y a que 2 signes multiplicateurs (). En math, lorsqu'on dit qu'un chiffre est multiplié autant de fois que son exposant par exemple 55, donnerait 5*5 (1ère fois) *5 (2ème fois) *5 (3è fois) *5(4ème fois) 5 (5ème fois), soit 5 fois le signe (). Mon explication est logique.
Your (?) explication est "faussée", car, dans le cas de 77 où de 7**3, ou de n'importe quel autre cas,  javascript NE MULTIPLIE PAS LE CHIFFRE AUTANT DE FOIS QUE SON EXPOSANT mais bel et bien UNE FOIS DE MOINS. On pourrait penser qu il multiplie le premier chiffre par un, D ABORD;

look at this for comprehension that i said

So the explication is not good.
Sorry for the "format" (3/4 messages for one explication!) of MY explication ...

and exactly ...

@chrisdavidmills I think all of this is about this passage:

This is the equivalent of the base number being multipled by itself the same number of times as the exponent.

And the question whether 5*5*5 is multiplied by itself 2 or 3 times.
Btw, "multipled" looks like a mistype on that page.

Yes, that's what i want to said.
It is the explication which is not correct.
The truth result is 2401. If not, it is multiplied two times only. Not
three.
Le sam. 16 févr. 2019 à 08:23, Сковорода Никита Андреевич <
<EMAIL_ADDRESS>a écrit :

@chrisdavidmills https://github.com/chrisdavidmills I think all of this
is about this passage:
This is the equivalent of the base number being multipled by itself the
same number of times as the exponent.
And the question whether 555 is multiplied by itself 2 or 3 times.
Btw, "multipled" looks like a mistype on that page.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/1033#issuecomment-464304118, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AiYtYsm6ciFBIXu344eGxdCjK4hurFgOks5vN7IFgaJpZM4a60u1
.

@ChALkeR thank you for helping me to find this quickly ;-) I have now updated the text to "the number of base numbers specified by the exponent are all multiplied together". I hope this makes sense.
@pandaventurer let me know if this fixes the problem you had, and thanks for all your efforts in reporting this! Merci boucoup.

That's is the only one explication that we can offer ... !
Thanks you to read me and, understand me.
Best regards.
Philippe Delestan.
Le sam. 16 févr. 2019 à 08:56, Chris Mills<EMAIL_ADDRESS>a
écrit :

@ChALkeR https://github.com/ChALkeR thank you for helping me to find
this quickly ;-) I have now updated the text to "the number of base numbers
specified by the exponent are all multiplied together". I hope this makes
sense.
@pandaventurer https://github.com/pandaventurer let me know if this
fixes the problem you had, and thanks for all your efforts in reporting
this! Merci boucoup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/1033#issuecomment-464308740, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AiYtYtZimmnP8u8BmFc0pjyBNposQL3pks5vN7mTgaJpZM4a60u1
.

\o/
;-)

That's what i've receive:
[image: image.png]
Le sam. 16 févr. 2019 à 09:48, Chris Mills<EMAIL_ADDRESS>a
écrit :

\o/
;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/1033#issuecomment-464317709, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AiYtYipejb1iLeAbnZ_mI6wwSmkVcbi6ks5vN8XtgaJpZM4a60u1
.


Alt and AltGr does not report pressed events, only release.

Hello!
I am using your library in my project, specifically the key hook!
First though, I have this problem, where I get only the OnKeyReleaseEvent and not the OnKeyPressEvent  for the keys LEFT_MENU and the combined key Alt Gr which is LEFT_MENU and LEFT_CONTROL together.
Do you see this on your end, or is it actually working as intended?

The reason is system keys don't come with the same event as normal keys, so that has now been fixed with version 1.1.0.
The LEFT_MENU triggering a LEFT_CONTROL event seems to be a keyboard layout issue in Windows. ALT GR isn't an actual key in Windows, so what happens is it emulates a Ctrl+Alt press.

Can confirm it appears to work now, thanks! 🥳


Turn eliot.filter into a binary, and document its existence

For the idly curious, documentation can be found within the eliot.filter module.


anchor test failed with TypeError: Cannot read property 'state' of undefined when project name contains "hyphen"

Reproduction
Ran commands below and failed.
anchor init test-anchor
cd test-anchor
anchor test
...
TypeError: Cannot read property 'state' of undefined

But commands below finished normally.
anchor init testanchor
cd testanchor
anchor test
# test finished successly

Environment
anchor-cli 0.16.1
Ubuntu 20.04

Well, I tried again this and the error hasn't occurred. The problem looks like solved. I have troubled you, thanks.

Well, I tried again this and the error hasn't occurred. The problem looks like solved. I have troubled you, thanks.

What was the solution?

Well, I tried again this and the error hasn't occurred. The problem looks like solved. I have troubled you, thanks.

What was the solution?

Addressed by https://github.com/project-serum/anchor/pull/711.


Overhaul of venv_metadata_inspector.py, now using modern python libraries

[x] I have added an entry to docs/changelog.md

Summary of changes
Closes #587.  Closes #534.  Closes #528 .
This revamps venv_metadata_inspector.py to use modern python libraries like packaging and importlib.metadata / importlib_metadata instead of pkg_resources for venv metadata inspection.
This fixes some things we didn't know we had problems with (see Issues above).  One thing I noticed about the old venv_metadata_inspector.py that isn't captured in any Issue, is that currently it completely ignores any environment markers, and just assumes every entry in an install_requires is equally valid.  The new code evaluates both markers and the presence or absence of package extras (which may be present in dependencies) correctly.
I started with #344, and made changes.
The biggest changes are to venv_metadata_inspector.py and shared_libs.py
I have two questions:

Do we need to list importlib_metadata in pipx's install_requires if pipx doesn't actually use it in pipx code, but rather installs it in the shared venv? RESOLVED--Removed it from install_requires
The list of libraries installed in the shared venv is:pip, wheel, packaging, importlib-metadata, and setuptools.  Are all of these necessary?

After this PR merges I hope to release pipx <IP_ADDRESS>!
Test plan

The code was extensively tested with our new "slow" tests and 62 packages:
https://github.com/itsayellow/pipx/actions/runs/429239351
There were no errors about pipx's venv metadata inspection missing apps or missing apps of dependencies (This is specifically checked in the slow tests).  The only problems were packages that would not install at all on the CI systems due to missing wheels and/or missing header files on the system.

@cs01, any thoughts?  I realize it's around Christmas and you probably have better things occupying yourself.  Just thought I'd try to at least give you an opportunity to chime in before I start releasing things. 🙂
Happy Holidays everyone, btw 🎄

Wow this is great, an early Christmas present :). This and will handle all the annoying corner cases, and be a nice foundation for the future of pipx.

The list of libraries installed in the shared venv is: pip, wheel, packaging, importlib-metadata, and setuptools. Are all of these necessary?

The packages installed into the shared libs were originally introduced in https://github.com/pipxproject/pipx/pull/168/files.
As you know, the shared libs are essentially "merged" at runtime with pipx's package-specific venvs via the pth file. This is to save time by avoiding installing a bunch of the same things over and over for each venv. It forms a common environment for the app's venvs to be able to run pip and the metadata inspector.
pip is definitely required to install packages. pip often needs to build things from source. The source build system is often setuptools, so that is required in case the package is an sdist. To build into a wheel, it needs the wheel package. packaging, importlib_metadata are used directly by venv_metadata_inspector.py so they are required too.
This LGTM!

Closing in favor of instead using #593


Fix/docker

Updated Dockerfile
Added docker-compose for local development
Fixed some dependencies

Not sure if these are redundant, as otherwise, we do not create the database and the tables within it for storing data. Currently getting a "database "thor_test_tmp" does not exist" error that I think is due to this

We should move this part to the tests, we should only create this db while running tests, not in production. I added some docs to run locally using Docker here - https://github.com/uc-cdis/thor/blob/59d840f868f90e93fff44e737ebf8b6c61e5a767/README.md#local-development-with-docker

Ok, everything looks good from here. +1


formatting codes don't work in the console

formatting codes set in config.yml don't get set in console.

I thought it might be a console issue. but other plugins work fine and apply the formatting codes set in their configs.

seems like the message is being spat through the logger and not the adventure console audience. Odd.


Location field adjustments

Prep for generalized location autocomplete/create

coverage: 94.372%. remained the same
when pulling e9bba6eff0f3d3d25102f93c184be0a016155f1e on location-field-adjustments
into e57e46e7dc8714556a3daf599f87e23f85b6f513 on main.


Cannot seem to override values of certain default strings

Are you requesting a feature, reporting a bug or ask a question?
Asking a question
What is the current behavior?
I cannot seem to override the default text for strings such as 'requiredInAllRowsError'.
What is the expected behavior?
To be able to override the values of such strings. Some can be overridden (e.g. 'showQuestionNumbers'), but I cannot seem to change others. Is there a standard way of doing this?
How would you reproduce the current behavior (if this is a bug)?
N/A
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):
surveyjs version:

@JonBetts Do you want to override it for on all matrixes or just one?
You can always change the localized string for all questions like this:
Survey.surveyLocalization.surveyStrings.requiredInAllRowsError = "My text";
Thank you,
Andrew

Just one matrix would be ideal, but just changing it for all questions is a good start. Thanks Andrew, I will try that out! I'll close the ticket later on after I've been playing around.

@JonBetts If you need it to be diffrent for diffrent matrix and it should be a part of survey JSON, then I convert it to suggestion.
Thank you,
Andrew

How can I achieve this server-side rather than in the Javascript? For example, with isAllRowRequired, I can just create add it to a map in Java land like so: questionObject.put("isAllRowRequired", true); and it picks it up. But this doesn't seem to work for these string localisations.

@JonBetts SurveyJS is a client JavaScript library. I guess you are having the surver side code that renders the client code. You have to render the code I have provided:
Survey.surveyLocalization.surveyStrings.requiredInAllRowsError = "My text";
before you are rendering the code that creates the Survey on the client.
Probably I do not understand your scenario, then provide it in more details please.
Thank you,
Andrew


android x86

I am having trouble running the android demos.
I am on ubuntu 14.04.
The OryolCclear demo works on the arm emulator. None of the others.
Wondering if its a known issue
--
Also I was thinking of giving Android X86 a try.
I see that the oryol / configs / androidx86-make-release.json is there, but could not the python build script calling it. I guess this part is not done yet ?

closing old tickets


IN-2006-timeout-dialog

IN-2006 RETEST/NEW #32 [A11y] SortSite - 4.1.2 Name, Role, Value – Bad Value for Attribute Role

Minor templating change, merging as admin


ci(cd): allow manual workflow dispatch

trying to make it easier to switch between forno and quicknode

Codecov Report

Merging #55 (9344fa7) into main (ed01462) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   85.06%   85.06%
=======================================
  Files           9        9
  Lines         154      154
  Branches       26       26
=======================================
  Hits          131      131
  Misses         23       23

Continue to review full report in Codecov by Sentry.

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


Ideas for limits

Various ideas:
Limits need to be by test type.
Have an API to retrieve a list of what limits apply for a host/test.

Limit system has been pretty much built as much as it's going to be for the first release.  Closing.


Differentiate Open & Closed PARTIALLY_FILLED Orders

Hello,
How to differentiate Open PARTIALLY_FILLED Orders and Closed PARTIALLY_FILLED Orders ?
Thanks for your help :)

Sorry, I didn't understand that in the Binance API,
a Partially Filled and Canceled Order is displayed as status = "Canceled" with an executed quantity higher or Zero.

Sorry, I didn't understand that in the Binance API,
a Partially Filled and Canceled Order is displayed as status = "Canceled" with an executed quantity higher or Zero.


Migrate Saving/Opening Workspace functionality to individual dataset detail pages

As a user within a dataset visualization workspace,
I want the ability to save my workspace/ open previously saved workspaces,
So that I can save or continue my work
ACs

As a logged in user, I am presented with the save/folder icons in any dataset visualization workspace
As a logged in user, I can save my workspace, or open a previous workspace within any dataset visualization workspace
-Verify the user experience with @jakeclark


merge common styles in shared CSS file

Made another issue addressing common CSS styles, so this issue has become redundant.


Fix "method_complexity" issue in MMM-PublicTransportHafas.js

Function socketNotificationReceived has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
https://codeclimate.com/github/raywo/MMM-PublicTransportHafas/MMM-PublicTransportHafas.js#issue_5a79e475a2efe70001000010

Fixed.


feat: add swoole 6.0.0 release

update swoole with release 6.0.0

:tada: This issue has been resolved in version 1.323.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Need to share some experiences about using the self-rating table 自评表

Need to share some experiences about using the self-rating table 自评表

will add the experience to "DevOps成熟度评级经验"


It's now showing token images on discord.

I am using the latest version of FoundryVTT and the GURPS GGA System.
The items referring to the system work very well. But the tokens/avatars images are not showing up on discord. Could you guide me how can I fix this?

My server is public https://campanhasdoboi.com.br/ and i'm using it on oracle, you check by yourself. So i see no reason to no share the imgs with discord!

If the default Foundry icon is not showing, it's a sign of one of two things: your server isn't allowing access to the files on the machine it's hosted on, (which it's not, since the foundry icon appears when I visit your domain/modules/foundrytodiscord/src/defaultavatar.png), so that might mean your invite link on the config is wrong. Is that link what you put into the invite URL on your module config?

Thanks for giving me your invite link! It seems this link is working as the Game Invite URL **(Configure Settings -> Foundry to Discord -> Game Invite URL)**for Foundry to Discord to function properly, and is fetching the images as intended. Are you still having trouble with this?

Yeah, i corrected the invite. so i got some changes.

It's getting the default avatar but not the token imagem or player avatar.

Ah, if the user's default avatar is not set, it will automaticaally default to the Foundry icon as well. Seems like your Gamemaster account has an avatar set, but the player user doesn't.
You're welcome, and enjoy!

Thank you very much, let me restart everything and set avatar correctly for everyone. If you need a server to run some tests please let me know, mine is 24/7 on oracle! Thanks for all the work and to be so kind! God bless you

Hello everyone, I had the same problem with the tokens not appearing in the discord bot, and when logging into Foundry on my cell phone I realized that the Certboot certificate had expired, and that's why the webhook failed to access the server. Just stopping by to remind you to change your certificates when you renew (yes, I had forgotten to replace them).
And of course, thank you very much for the module and the great work!


use git-lfs instead of committing binary files

Use git-lfs will make the repository cleaner.
Git Large File Storage (LFS) replaces large files such as audio samples, videos, datasets, and graphics with text pointers inside Git, while storing the file contents on a remote server like GitHub.com or GitHub Enterprise.

updated, thx


Install Binaries command fail

I followed all the steps on the installation guide but when getting to the install-binaries command step, this happens

any idea why?

thanks for the quick response and I will be looking forward to it

Hey, I made the release. Try rerunning the command

it works, thanks!


Inconsistent behavior: number +* list

System.print([1,2] + [3,4]) // oh the horror
System.print([1,2] * 3) // oh the python
// System.print(3 * [1,2]) // Right operand must be a number.
// System.print(3 + [1,2]) // Right operand must be a number.
// System.print([1,2] + 3) // Num does not implement 'iterate(_)'.

Note that 3 * [1,2] is not valid right now. Also, the error message for [1,2] + 3 is confusing.

The problem is with the implementation of operator overloading in Wren. The expression [1, 2] * 3 calls [1, 2].*(3) (method List.*). But the expression 3 * [1, 2] calls 3.*([1, 2]) (method Num.*). The problem is that Num has no knowledge about lists - it knows how to multiple numbers only. In short, binary operator overloading in Wren is asymmetric.


set -o pipefail causes the build process to fail.

I've installed this plugin to use with Drone CI, the build process fails with "sh: 1: set: Illegal option -o pipefail" when trying to execute the following:
[12:07:53]: $ set -o pipefail && msbuild /p:Configuration=Release /p:SolutionDir=/drone/src/bitbucket.org/company/test-app/commits/33f665f98291013a5e17ecd09bec87cc638e6220/ /t:Build /t:SignAndroidPackage /drone/src/bitbucket.org/company/test-app/commits/33f665f98291013a5e17ecd09bec87cc638e6220/Droid/Test.Droid.csproj
[12:07:53]: ▸ sh: 1: set: Illegal option -o pipefail
sh: 1: set: Illegal option -o pipefail
[12:07:53]: Exit status: 2
Does the plugin need to include that in the build command?

Not sure if this is an issue of souyuz or rather the underlying Fastlane command executor but as long the build command is not using a pipeline we could remove the prefix.
I'm not familiar with Drone CI but is it possible to run the build process in bash shell; I think that should fix your problem.

The build process is running in bash, this here is part of the script that's run by Drone CI:
#!/bin/bash

set -eo pipefail

PARENT_DIRECTORY="$(pwd)"
mkdir $PARENT_DIRECTORY/android
ANDROID_DIR=$PARENT_DIRECTORY/android

apt-get update
apt-get install -y curl unzip

if ! java_loc="$(type -p "$java")" || [ -z "$java_loc" ]; then
  apt-get install -y openjdk-8-jdk
fi

echo ">> Install Android Environment"
# download android sdk
wget https://dl.google.com/android/repository/sdk-tools-linux-3859397.zip
unzip sdk-tools-linux-3859397.zip -d $ANDROID_DIR

yes | $ANDROID_DIR/tools/bin/sdkmanager --licenses
$ANDROID_DIR/tools/bin/sdkmanager --update
$ANDROID_DIR/tools/bin/sdkmanager "platforms;android-26" "build-tools;27.0.3"

echo ">> Install Mono"
apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-keys 3FA7E0328081BFF6A14DA29AA6A19B38D3D831EF
echo "deb http://download.mono-project.com/repo/ubuntu stable-xenial main" | tee /etc/apt/sources.list.d/mono-official-stable.list
apt-get update
apt-get install -y mono-complete

echo ">> Starting Fastlane environment = "$1
bundle exec fastlane $1

Is it possible to let the script that runs fastlane decide for itself if we need the pipeline option or not?

I am not familiar with drone either. But as @johannesberdin states; since we do not use pipelines, we can safely remove the prefix as such.
I will doublecheck and fix this issue ASAP.
regards


Requirements and logging

PIN requirements with hashes, update setup.py
Add optional support for Elasticsearch logging

Coverage decreased (-0.04%) to 60.956% when pulling bbf5e951868d91da5e51696767fe563464087688 on castrapel:requirements_logging into 17be8b626dabe8d7cc9f9aadb31dca7c92504277 on Netflix:master.


Update and pin ms to 2.1.2

This pull request is related to #688 for consideration 🙇

Coverage remained the same at 87.925% when pulling 2bc5b087aa6eb475521c4364605f5182865fe3c3 on dougwilson:pin_ms into 68b4dc8d8549d3924673c38fccc5d594f0a38da1 on visionmedia:backport/4.1.1.

Coverage remained the same at 87.925% when pulling 2bc5b087aa6eb475521c4364605f5182865fe3c3 on dougwilson:pin_ms into 68b4dc8d8549d3924673c38fccc5d594f0a38da1 on visionmedia:backport/4.1.1.

Thanks!

Oh, not sure why this was merged into backport - let's just merge it to master. I don't want to backport anything anymore - it caused so many headaches the last time we tried that.
I'll cherry pick onto master, your author information should be retained :)
Thanks again.

Published as 4.2.0 under the beta dist-tag. Please test and make sure nothing has broken for you before I send to latest.

4.3.4

2.6.8


ANCM: Inprocess: Usability: Need to write an event error about that standard log file can't be generated if IISExpress is started with console window

According to the below code, we don't support the standard log feature when IISExpress is started with console window. I think we should add an warning event when this happen so that users can figure out why the standard log file is not generated.
Repro step:

Start app with IISExpress in inprocess mode from command window with enabling stdoutLogEnabled="true"
Send a request

EXPECTED:
Log file is generated. Or, we should explain why log file is not generated.
ACTUAL:
No log file is generated.
            {
                // The process has console, e.g., IIS Express scenario

                if (_wfopen_s(&m_pStdFile, m_struLogFilePath.QueryStr(), L"w") == 0)
                {
                    // known issue: error info may not be capture when process crashes during buffering
                    // even we disabled FILE buffering
                    setvbuf(m_pStdFile, NULL, _IONBF, 0);
                    _dup2(_fileno(m_pStdFile), _fileno(stdout));
                    _dup2(_fileno(m_pStdFile), _fileno(stderr));
                }
                // These don't work for console scenario
                // close and AllocConsole does not help
                //_wfreopen_s(&m_pStdFile, struLogFileName.QueryStr(), L"w", stdout);
                // SetStdHandle(STD_ERROR_HANDLE, m_hLogFileHandle);
                // SetStdHandle(STD_OUTPUT_HANDLE, m_hLogFileHandle);
                //*stdout = *m_pStdFile;
                //*stderr = *m_pStdFile;
            }

This issue has gotten stale. We don't plan to fix it at this point.


Invalid JWT when running inside Docker

I created a simple fulfillment server that takes user input and gets/forwards the appropriate response from Dialogflow. The code is pretty simple, it's all contained in the index.js file of the master branch of this repository. When I ran this through Heroku (at https://metbot-fulfillment.herokuapp.com/), everything worked fine. If you send a POST request to that URL with this body:
 {
  "session": "8d13aa8-2999-4f71-b233-39cbf3a824a0",
  "queryResult": {
    "queryText": "Hello"
  }
}

you should receive a valid response. However, I needed to move the app off of Heroku and onto a standard Ubuntu server, so I wanted to use Docker to make the future process of moving between servers simple. Yet when I send the same body to my Docker container, I receive this error:
(node:1) UnhandledPromiseRejectionWarning: Error: 400 undefined: Getting metadata from plugin failed with error: invalid_grant: Invalid JWT: Token must be a short-lived token (60 minutes) and in a reasonable timeframe. Check your iat and exp values in the JWT claim.
    at Object.callErrorFromStatus (/usr/src/app/node_modules/@grpc/grpc-js/build/src/call.js:30:26)
    at Http2CallStream.call.on (/usr/src/app/node_modules/@grpc/grpc-js/build/src/client.js:96:33)
    at Http2CallStream.emit (events.js:203:15)
    at process.nextTick (/usr/src/app/node_modules/@grpc/grpc-js/build/src/call-stream.js:75:22)
    at process._tickCallback (internal/process/next_tick.js:61:11)
(node:1) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)

I tried searching for solutions and saw that the system clock being off could be a culprit, and indeed my clock was off by a few minutes (but that shouldn't have affected the validity of 60 minute token right?). Regardless, I had the offset corrected and I continue to get the same error. Is this an issue with Docker? Or perhaps the base container I'm using? This is my Dockerfile.
Environment details

OS: 14.04
Node.js version: 10 (Docker)
npm version: Same as above
dialogflow version: ^0.12.2

Steps to reproduce

Clone the docker branch, build and run the container
Try sending the above POST request

I'm not sure what's changed (hence why I'm hesitant to close this before getting any insight) but my container started working. All I've done is leave it alone for a day, is it possible that due to the initial time sync issues the tokens were off but giving it time to reset solved the issue?

I found this while looking into a similar error, and people have said this can happen if the server clock is out of sync for some reason, so restarting your docker container may have fixed it, or something else might have happened to fix that discrepancy. Just an FYI in case that was your issue too!

I had my hour bad configured, 1 hour offset in Windows. I adjust the time and worked

@ROODAY would yo be able to check whether you were running in to a clock drift issue?

@bcoe Unfortunately not as now the app is being run as a systemctl service without docker so I don't have the original environment anymore.

@ROODAY I'm going to close this issue out for now, as I think clock drift is the most likely culprit. Please feel free to follow up if you do bump into issues again with Docker.


Potential Stack Overflow Error when loading world

When loading my World with a build of the latest galacticraft-rewoven commit (931788d6eb7b2c3d2895dad099d7ab7af4e6fbe1) I occasionally encounter (3 out of 5 times) a stack overflow. I have created the world without galacticraft, but I doubt that this is the issue since I was able to log in into the world once.
Stacktrace 1: https://gist.github.com/Geolykt/ed4860802a1849eaf3bfac597a1ca03d
Stacktrace 2 (similar to Stacktrace 1 but a bit different): https://gist.github.com/Geolykt/8913ffb76c8ac5fe10ce795b54f11561
Further information common information provided in the crash report: https://gist.github.com/Geolykt/9fca33920433d6940729298661b1c1fc
Other information that isn't disclosed in the crash report:

It is not affected by the background/foreground state

Thanks for the detailed report. I guess I didn't test TR compatibility as much as I should've - I'll look into this soon.


Debug casts object to string

Summary:
Very tiny improvement, just a log that prints an object as a string, I doubt this is the intended behavior because it's not very useful in my opinion, literally a one character change
Is this ready to be merged into Dynamoose? (select 1):

[X] Yes
[ ] No

Pull Request Test Coverage Report for Build 1048

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.099%

Totals

Change from base Build 1047:
0.0%

Covered Lines:
1503

Relevant Lines:
1775

💛  - Coveralls

Pull Request Test Coverage Report for Build 1048

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.099%

Totals

Change from base Build 1047:
0.0%

Covered Lines:
1503

Relevant Lines:
1775

💛  - Coveralls

@purefan Please follow our Contributing Guide your comment message needs to be formatted correctly in order to handle releases.


Feat/identities

Adds an _identities property to orbitdb and remove it from the identity instances

Closing in favor of keeping the provider reference in identity so that they are not managed separately


Waymore is not filtering the URL as configured in FILTER_URL in config.yaml.

Hello,
Thank you for the awesome tool.
Waymore is not filtering the URLs properly after supplying FILTER_URL in the config.yaml file.

config.yaml

Here, svg, woff, woff2, and ttf are not filtered from the URL in the final result.
Regards,
brutexploiter

Hi @brutexploiter. Thanks again for raising this! This should be fixed in the latest version. Just run pip install --upgrade waymore. Let me know if this has resolved your issue. Many thanks

Thanks @xnl-h4ck3r for the fix. Working fine.


Reducing the amount of storage used

The app is taking up a whopping 2.36GB on my Nexus 5, which is likely due to my history settings of 1year & 100K messages. I changed them to 1 week &  500 messages. How can I trigger a purge of my history to bring it down to the new limits, and consequently freeing up disk space?

The purge occurs once in a day, so please wait for awhile :-)
Please report if the purge happened?

Yes, the purge did happen at some point, it's now down to about 700MB.
Would be good to have the option to trigger the purge immediately in case space needs to be freed up urgently.
Re the max no. of messages: Is that for each account or the total of messages in all account?
If both time and no. of messages are specified, the limit (time/messages) that's reached first causes the purge to happen, yes?

@akwala Maximum number of messages is total,  for all accounts.
AndStatus checks both conditions sequentially. See the code here: https://github.com/andstatus/andstatus/blob/master/app/src/org/andstatus/app/data/DataPruner.java
You may create new feature request for "Purge now"...


Add --no-hash  flag

Issue #964

[x] Adding a new RegexSet member to bindgen::Builder (similar to the whitelisted_types set).

[x] A Builder method to add strings to that RegexSet.

[x] Plumbing in src/options.rs to convert --no-hash  CLI flags into invocations of the builder method.

[x] Making the MonotoneFramework::constrain function in src/ir/analysis/derive_hash.rs check if the given item is explicitly marked not to be Hash, and if so, inserting it into the self.cannot_derive_hash set via return self.insert(id).

[x] Tests!

[x] When the no-hash type is transitively referenced by a whitelisted item

[x] When the no-hash type is explicitly whitelisted

[x] When the no-hash type is marked opaque

r? @fitzgen

 Warning

These commits modify unsafe code. Please review it carefully!

:umbrella: The latest upstream changes (presumably #1099) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-servo delegate+
Ah, looks like some merge conflicts snuck in. Once you've rebased and resolved them you can tell r+ to @bors-servo yourself.
Thanks again!

:v: @seemyvest can now approve this pull request

No problem :) @bors-servo r+
I noticed while doing this that a misnamed variable was missed in #1099 here. no_partialeq should be named no_copy.
I'll have a look and see what others I can do, feel free to point some out to me :)

:pushpin: Commit 5d27156 has been approved by seemyvest

:hourglass: Testing commit 5d271568e07522503ddc361152633385a537cc2d with merge f315d2c99e9566401193c125ac71bc8cf56443f6...

:sunny: Test successful - status-travis
Approved by: seemyvest
Pushing f315d2c99e9566401193c125ac71bc8cf56443f6 to master...


APTT-268 Update changelog release date for 1.2.0 release

approved

Codecov Report

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

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   88.54%   88.54%
=======================================
  Files          25       25
  Lines        1292     1292
=======================================
  Hits         1144     1144
  Misses        148      148

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 0d4b469...5490975. Read the comment docs.


incorrect thread grouping

i have 2 thread with same subject
but 1st have 1 message with one person and 2nd have 4 messages form other persons
in the list i see as one thread with 5 messages that is incorrect (from my point of view)
i think that i would have to see 2 items in the list (although same subject)
im attach a screenshot

@ftoled There's a new thread browsing method in the lates version.

Good news! I will test asap.


[Suggestion] [HUD] - Hwyla Integration

DDD Information on the Hwyla Mob Interface
Future Talking: Creating an AddOn for Hwyla so that players targeting the enemy would see there resistant values, attack values and costume values.
This Ticket depends on the implementation of:

"[Suggestion] [HUD] - Damage Symbols #19"

Goal:

Give the player a basic knowledge to chose the right weapon when looking at a mob
-- Expert Players will dig deeper into the mechanics and will know exactly witch weapon and custom damage will be more effective

There might be too much info to display with custom resistances, so I may limit it to physical resistances only for the time being.

closed by #56


Make Module#private spec-compliant

For example:
class A
  def foo
    "bar"
  end
end

class B < A
  private :foo
end

A.new.foo # => "bar"
B.new.foo # => NoMethodError
B.remove_method :foo
B.new.foo # => "bar"

This kinda blew my mind. Good work. I don't love that we need yet another hashmap, but I don't have a better solution either.


[material-ui][Typography] Deprecate paragraph prop

[x] I have followed (at least) the PR section of the contributing guide.

Resolves #42382, resolves #16926

thanks so much for the instructions @aarongarciah. Happy to make those changes later today.

Hey @walston! Did you have a chance to take a look? We want to land this on v6, which should be out in the next few weeks.

oh dang. I'd love that. Apologies, got caught up with a company onsite. I can spend some time working through these things next week. Top of my priorities list now!

@walston in this case, I think a codemod doesn't make sense because there's no direct replacement for the paragraph behavior:

Renders as a <p> element.
Adds margin-bottom: 16px.

We could codemod number 1 with component="p", but we can't codemod number 2 because we can't possibly know what other styles are developers applying to the component instance and we might break their UIs. In this case, I think it makes sense to provide the instructions on how to migrate.
Even if it was possible, you could ask us to pick it up if you can't do it.

@aarongarciah I'm working with @walston to get this over the finish line :)

We could codemod number 1 with component="p", but we can't codemod number 2 because we can't possibly know what other styles are developers applying to the component instance and we might break their UIs. In this case, I think it makes sense to provide the instructions on how to migrate.

For 1: I don't think we event need to add the component prop, since <Typography> renders as a <p> by default. Unless a developer overrides this setting it would probably result in a lot of redundant code. Is there a way to check for this?
For 2: Would we want to suggest adding mb={2} for Typography elements that need margin? I'm still confused about MUI's preferences between mb and sx props for margins.

Hey @arronhunt! Thanks for working on this.
From your comment I'm not sure if you're working on a codemod or not. Just in case, I think we shouldn't spend time in a codemod. Instead, we can focus on writing clear guidance on what changed in the Migrating from deprecated APIs page given we can't possibly know users' intent and where the styles are coming from.

For 1: I don't think we event need to add the component prop, since  renders as a  by default. Unless a developer overrides this setting it would probably result in a lot of redundant code. Is there a way to check for this?

The tag being rendered by Typography depends on the variant and some other props. Typography renders a <p> in these scenarios:

variant="body1" (body1 is also used when no variant is passed)
variant="body2"
paragraph
component="p" or component={MyCustomComponent} (that renders a <p>)
variant="any_variant" paragraph
variant="any_variant" component="p"

It's fine to omit component="p" if no variant is passed or if the variant is already rendering a <p> (body1 and body2). See this demo for more info https://mui.com/material-ui/react-typography/#usage.

For 2: Would we want to suggest adding mb={2} for Typography elements that need margin? I'm still confused about MUI's preferences between mb and sx props for margins.

We'll remove system props (e.g. mb={2}) so it's recommended to use the sx prop. See this RFC and this in-progress PR if you want to know more.

Hey @arronhunt @walston! Let me know if you need any help.

@aarongarciah I just pushed updates to the docs, using sx props to set bottom margin for paragraphs. I agree that a codemod isn't necessary and we can rely on the upgrade instructions. Would love your feedback!

Thank you so much, @walston and @arronhunt. I'll be making some last finishing touches before merging.

@DiegoAndai I thought about the codemod, but I though it wouldn't be good because:

marginBottom could be applied to an element from anywhere without the codemod noticing:

A custom class name
A global class name
Inline styles
A parent element overriding child styles
etc.

Some people might don't want to use the sx prop.

Code wise it would look good but we could be breaking UIs by adding marginBottom ourselves. In that case, I think it's better to let users handle it. wdyt?

marginBottom could be applied to an element from anywhere without the codemod noticing, and we could intefere with users's styles:

Right, but if they were using the paragraph prop, they would expect this marginBottom to be applied to the element. So it should continue to work as they had it before the codemod.

Some people might don't want to use the sx prop.

That's fair. We could add the codemod and add the option to skip codemods in a follow-up PR if someone doesn't want to run it.

Right, but if they were using the paragraph prop, they would expect this marginBottom to be applied to the element. So it should continue to work as they had it before the codemod.

Can we ensure the same specificity?

@DiegoAndai tbh I'd love to know how many people rely on paragraph to know if it's worth spending time on the codemod 😅

So, I'm a designer and therefore not super familiar with codemods, but is there an existing pattern for providing them but make running them optional? I could see it being beneficial for those who have a lot of paragraph props and aren't using any specific margin styles. Otherwise, as a consumer of MUI, I like @DiegoAndai's idea of adding the comment to the migration instructions.

@\DiegoAndai tbh I'd love to know how many people rely on paragraph to know if it's worth spending time on the codemod 😅

@aarongarciah Anecdotally, our team has been relying on it for styling. It's actually what caused me to want to come here and fix the bad pattern lol.

@arronhunt we'll probably provide a codemod, but if you want to contribute it you're more than welcome. Right now, we're focused on making Material UI compatible with React 19.
You can find codemod examples in the mui-codemod package. For example, the one for the Divider component, which removes the light prop and adds sx={{ opacity: 0.6 }} (docs).

We could add the codemod and add the option to skip codemods in a follow-up PR if someone doesn't want to run it.

@DiegoAndai what did you have in mind for this? I'll be working on the codemod.

Remove paragraph prop if found, and:

Add sx={{ marginBottom: 2 }} if there's no sx prop
Add marginBottom: 2 to sx if there's sx prop

@DiegoAndai this would only be 16px if they haven't modified the default theme spacing (8px by default)

@DiegoAndai I just added a commit with the codemod. I went with 16px instead of 2 for the marginBottom value.
One doubt: should we cater for the mb shortcut being present in the object passed to sx?

I need to update the migration guide.

Ready for review.

@DiegoAndai I just updated the codemod to cater for mb, too. Ready for a final review.

Thanks @walston @arronhunt for working on this!


Document should detail what happens to existing recovery points when changing backup policy

This doc mentions changing the backup policy assigned to backup items, as well as changing the policy definition itself. However, there is no discussion of how these changes affect the retention of existing recovery points.
What should ideally be covered:

How changing the policy assigned affects existing recovery points that are

Outside of the new retention period (including weekly, monthly, ...)
Inside of the new retention period
Any grace period to fix mistakes

Same for how changing the policy definition itself affects
Any available protections for limiting changing a policy assignment or definition to prevent the loss of restore points (for compliance purposes, for example).

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

ID: 5b71eb75-6ec4-2647-c3bd-2d7822704346
Version Independent ID: 0e174420-da45-5d9a-4a8e-81ea00f316a4
Content: Manage and monitor Azure VM backups - Azure Backup
Content Source: articles/backup/backup-azure-manage-vms.md
Service: backup
GitHub Login: @v-amallick
Microsoft Alias: v-amallick

Some of this is covered here, so it might suffice to link to this article: https://docs.microsoft.com/azure/backup/guidance-best-practices#optimize-backup-policy

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

@SvenAelterman -

We have this FAQ describing the required info: https://docs.microsoft.com/en-us/azure/backup/backup-azure-vm-backup-faq#what-happens-if-i-modify-a-backup-policy

Stopped backup can't be deleted automatically with policy as no policies are assigned to it. One can resume protection and assign a policy : https://docs.microsoft.com/en-us/azure/backup/backup-azure-vm-backup-faq#how-long-are-the-stopped-backups-retained

Hope this answers your question.
At this moment, since there is no doc update required, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


pre-commit-hooks: init

This pull request introduces pre-commit hooks.
The added functionality should include running code formatters, such as nixfmt and shellcheck before each commit. Additionally, the hooks should check success of every build and generate diffs for every host (ref to this review).
By implementing these changes, this would resolve the following issues: #25, #26, and potentially #33.

Can and what you think whether this be packaged as a Nix package? I have an example in here: https://github.com/jhvst/barbell

Can and what you think whether this be packaged as a Nix package? I have an example in here: https://github.com/jhvst/barbell

Yeah, why not.. CBQN has already been packaged upstream. Is this related to this pull request?

Can and what you think whether this be packaged as a Nix package? I have an example in here: https://github.com/jhvst/barbell

Yeah, why not.. CBQN has already been packaged upstream. Is this related to this pull request?

CBQN does not need to be packaged, but the bash script should be packaged in a similar manner -- that is, it all required software in its buildinputs


On-board debuggers not connecting to openOCD on Win-11

Hardware

MAX32670 EV KIT
MAX32655 FTHR

So this is an issue I am experiencing since updating to windows 11 from windows 10. Projects that once debugged and ran just find not get stuck with the following output on the Micros SDK console:
Open On-Chip Debugger 0.11.0+dev-g4f85f7914 (2021-12-16-13:53)
Licensed under GNU GPL v2
For bug reports, read
	http://openocd.org/doc/doxygen/bugs.html
DEPRECATED! use 'adapter driver' not 'interface'
DEPRECATED! use 'adapter srst delay' not 'adapter_nsrst_delay'
DEPRECATED! use 'adapter srst pulse_width' not 'adapter_nsrst_assert_width'
Started by GNU MCU Eclipse
Info : Listening on port 6666 for tcl connections
Info : Listening on port 4444 for telnet connections

Not sure if this is a windows 11 thing where some port settings or something is not what was previously assumed to be default. I reinstalled Micros SDK several times and the issue persisted.
Then I installed Micros SDK on a Win 10 Machine and it worked as expected.
Just wondering if I could get some insight on what causes the debugger to not get passed this listening on port state.
Steps to reproduce:

Fresh Win 11
Fresh Micros SDK install.
Load up an example Project
Build
Try to Run/Debug

It has been tested on W11, the issue not reproduced,
It works as expected.


Certain Languages create blank output

On an install from .deb created a blank line of output. Reinstalled from git and with make and got a 400 error.
Example inputs:
trans -sl en -tl yue 'hello'
Github direct ver. Output:
[ERROR] Google returned an error response. HTTP status code: 400 [ERROR] Other HTTP error
Version info:
Translate Shell       <IP_ADDRESS>-git:d3e0836
platform              Linux
terminal type         xterm-256color
bi-di emulator        [N/A]
gawk (GNU Awk)        4.2.1
fribidi (GNU FriBidi) 1.0.5
audio player          mpv --no-config
terminal pager        less
web browser           xdg-open
user locale           en_US.UTF-8 (English)
home language         en
source language       auto
target language       en
translation engine    google
proxy                 [NONE]
user-agent            Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.138 Safari/537.36
ip version            [DEFAULT]
theme                 default
init file             [NONE]

Dump ( trans -sl en -tl yue 'hello' -dump )
[ERROR] Google returned an error response. HTTP status code: 400
[ERROR] Other HTTP error
695
Error 400 (Bad Request)!!1400. That’s an error.The server cannot process the request because it is malformed. It should not be retried. That’s all we know.
0

trans -e bing -sl en -tl yue 'hello' -dump

{"statusCode":400,"errorMessage":""}

same status code output with bing engine

Please use the latest develop branch where this issue has been fixed.

By the way, Google Translate never supported Cantonese (yue as the language code), so the Bing engine had to be invoked explicitly in this case.

By the way, Google Translate never supported Cantonese (yue as the language code), so the Bing engine had to be invoked explicitly in this case.

It doesn’t work with Bing either, but with a different failure mode (passing e.g. -e bing en:yue "<common english word>" returns  verbatim).

By the way, Google Translate never supported Cantonese (yue as the language code), so the Bing engine had to be invoked explicitly in this case.

It doesn’t work with Bing either, but with a different failure mode (passing e.g. -e bing en:yue "<common english word>" returns <common english word> verbatim). This is with Docker image pulled today.

Seems that the docker image was a bit outdated. It should work with the latest version (<IP_ADDRESS>).
$ trans -e bing en:yue "I don't know"
I don't know

我唔知

[ English → 粵語 ]


refactor(types): Improve types coverage & docs for js parser

Summary
Increase type coverage for the operand handlers and helper functions of UnaryExpression and BinaryExpression hooks in the JavaScript parser.

🤖 Generated by Copilot at 0fff38d
This pull request enhances the documentation and readability of the BasicEvaluatedExpression and JavascriptParser modules, which are responsible for parsing and evaluating JavaScript code in webpack. It adds JSDoc comments, template annotations, and parameter name changes to various functions and methods, and simplifies some logic and evaluation cases.
Details

🤖 Generated by Copilot at 0fff38d

Add JSDoc comments to various functions and methods in lib/javascript/JavascriptParser.js and lib/javascript/BasicEvaluatedExpression.js to improve documentation and readability (link, link, link, link, link, link, link, link, link)
Add template annotations and rename parameters to the handleConstOperation function in lib/javascript/JavascriptParser.js to improve type checking and generality (link, link, link)
Update the call to the handleConstOperation function in lib/javascript/JavascriptParser.js to reflect the parameter name change (link)

@chenxsan Despite believing that most of the documentation I've provided here is contributor specific, I going to have the bot create an issue for documentation because I believe that most people don't know that webpack's parser will evaluate simple expressions for many module / import specific scenarios (as I'm remembering/learning throughout this process).
If you personally don't find any value in actually documenting this, then you can close the issue.

Very small inclusion of types in Basic Evaluated Expression. I will include better documentation for this in the future.

@chenxsan Despite believing that most of the documentation I've provided here is contributor specific, I going to have the bot create an issue for documentation because I believe that most people don't know that webpack's parser will evaluate simple expressions for many module / import specific scenarios (as I'm remembering/learning throughout this process).
If you personally don't find any value in actually documenting this, then you can close the issue.

Yup, it's very helpful to contributors or who want to dive into the source code, regular users might not notice it. I'll skip it for this time, we can always come back to document them if there're requests from regular users.


13.0.0 Release Notes

Summary
The API is 100% compatible with v12. This release moves @hapi/joi to a peer dependency .
Breaking Changes
@hapi/joi must be installed independently: npx install-peerdeps hapi-swagger
Fixed

https://github.com/glennjones/hapi-swagger/issues/648 Move @hapi/joi to peer dependency

Question here, if it must be installed, why is it a peer dep? Why not keep it as a dep?


pythonPackage.pycassa: init at 1.11.2

Motivation for this change
to have it
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@FRidh I moved it to a dedicated file. Is this now an official convention? Is there a document that mentions this convention?
Regarding tests, no tests are run. They requires a cassandra server up and running (https://github.com/pycassa/pycassa/tree/master/tests).

@FRidh I moved it to a dedicated file. Is this now an official convention? Is there a document that mentions this convention?

Its noted in the latest version of the Nixpkgs manual.

Regarding tests, no test is run. They require a cassandra server up and running (https://github.com/pycassa/pycassa/tree/master/tests).

Okay, then disable the checkPhase by passing doCheck = false; and include a comment with the reason (see also the docs for this).

I commented why doCheck = false.
And since pycassa doens't support python3, I disabled it.


freerdp3: 3.4.0 -> 3.5.1

Changelog: https://github.com/FreeRDP/FreeRDP/releases/tag/3.5.0
Changelog: https://github.com/FreeRDP/FreeRDP/releases/tag/3.5.1
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This breaks x86_64 darwin. Switching it to darwin.apple_sdk_11_0.callPackage gets past the first error but then we have problems with tls_protocol_version_t..

Actually it all works if we just use the more capable overrideSDK approach to sdk override:
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -31293,6 +31293,7 @@ with pkgs;
         inherit (gst_all_1) gstreamer gst-plugins-base gst-plugins-good;
       };
       freerdp3 = callPackage ../applications/networking/remote/freerdp/3.nix {
+        stdenv = if stdenv.isDarwin then overrideSDK stdenv "11.0" else stdenv;
         inherit (darwin.apple_sdk.frameworks) AudioToolbox AVFoundation Carbon Cocoa CoreMedia;
       };
     })

Thanks, applied that


Monitor 2.0 overview plots overlap when resizing browser window

Monitor 2.0 overview plots do not resize gracefully when resizing the browser window.
This is resolved by adding a script tag for jquery.flot.resize.js in default.ftl
I will submit a pull request.

Hi @cleroux ; Thanks for the patch. For what it's worth, there's no need to open an issue if you're going to immediately submit a pull request. The pull request itself serves as its own issue. :smiley_cat:


Fix Sync and Send markers, add Unpin

I think I got a bit too fast with #38. This should be analog to std Cow, also added Unpin because why not.
cc @ammaraskar @JOE1994

0.5 is published.

0.5 is published.


Restart server (shutdown doens't unbind address)

I would add a new route when the server is running.
I see in the code that you clone the route, so I thought that this feature is intentionally missing.
So I try to restart the server add a new route in this way:
Pistache::Http::Endpoint server;
// init server
server.serveThreaded()
// some code
server.shutdown()
server.serveThreaded()
but I got the error
epoll_ctl(epoll_fd, EPOLL_CTL_ADD, fd, &ev): Bad file descriptor
then Pistache try to bind as the shutdown method doesn't unbind.
Really, calling my rest method after the shutdown, the connection remains appended (as the server is blocked) instead of refusing it.

duplicated  by #164

Hello, closing this issue in favor of #164


Wrong screen dimension handling for 90/270 degree display rotation in twr_gfx.c

Hi,
while implementing twr_gfx support for waveshare 2.9" e-ink display (128×296 pixels), I noticed wrong behavior for 90 and 270 degree display rotation. It works OK with LCD module as it have square 128×128 display, but it doesn't work for non-square displays.
In particular, problem is in following code snippets from twr_gfx_draw_pixel():
        case TWR_GFX_ROTATION_90:
        {
            tmp = x;
            x = self->_caps.height - 1 - y;
            y = tmp;
            break;
        }

        case TWR_GFX_ROTATION_270:
        {
            tmp = y;
            y = self->_caps.width - 1 - x;
            x = tmp;
            break;
        }

this should be:
        case TWR_GFX_ROTATION_90:
        {
            tmp = x;
            x = self->_caps.width - 1 - y;
            y = tmp;
            break;
        }

        case TWR_GFX_ROTATION_270:
        {
            tmp = y;
            y = self->_caps.height - 1 - x;
            x = tmp;
            break;
        }

Logic behind this is, that screen size is relevant to left side of assignment as it represents hardware coordinates. As an example, for my 128×296 display and 270° rotation, writing pixel to 0,0 coordinates should translate to bottom left which is 0,295:
original code: y = self->_caps.width - 1 - x = 128 - 1 - 0 = 127
fixed code: y = self->_caps.height - 1 - x = 296 - 1 - 0 = 295

I just noticed another bug in same function. Checking pixel coordinates using this code
    if (x >= self->_caps.width || y >= self->_caps.height || x < 0 || y < 0)
    {
        return;
    }

must be either done after the switch block doing rotation transformation or reworked to take 90/270° rotation in regard.

Fix was added to the main SDK branch
https://github.com/hardwario/twr-sdk/commit/11ed3523922a21eefba710bb8c35bbe30707769e


Rival 3 on updated firmware no longer supported

After updating the firmware (<IP_ADDRESS>) on Rival 3, rivalcfg no longer seems to recognize it (1038:184c) on Linux. Have you come across this?

Hello,
I cannot find any devices with 1038:184c id. Can you copy the entire line from the lsusb command. :)

Appreciate the quick response - here is the full text -
Bus 001 Device 002: ID 1038:184c SteelSeries ApS SteelSeries Rival 3
As far as I can tell, this happened after I moved to Engine 3.18.10 from 30.18.8 and updated firmware. Hope this helps.

You are right, they changed the USB descriptor in the new firmware.
Can you change the product_id in rivalcfg/devices/rival3.py and try if it works (do not hesitate if you need some help, I can make a test branch for this.). Note that after this change you will probably need to run the rivalcfg command as root as udev rules will not be up to date
:)

Thanks! Worked as expected and appreciate your help :-)

Ok so I will add this product_id to rivalcfg :)

It is on master, it will be released soon in v4.2.0. Thank for the report :)

Thanks for the update! Will this also update the udev rules so running as root won't be needed?

Yes you will be able to update the udev rules with this release :)


Issue #546: Add support for configurations with dictionaries keyed by enums

Configuration dictionaries will now be able to keyed by enums

Added enum key handling in BindDictionary method
Added test case with enum keys

@achaloux,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks @achaloux !


Neon grid result - Temporal tiling disabled?

Hi Kijai, Thanks heaps for everything you do. I'm getting a gridded output (16 frames, otherwise same as example workflow settings) and a few inconsistently appearing msgs:
torch.Size([1, 4, 16, 60, 90])
tensor(0.) tensor(1.)
[VideoHelperSuite] - WARNING - Output images were not of valid resolution and have had padding applied
Prompt executed in 549.20 seconds
[Crystools ERROR] For some reason, pynvml is not working in a laptop with only battery, try to connect and turn on the monitor ** not always there**
[Crystools ERROR] Monitor of GPU is turning off (not on UI!) ** not always there**
got prompt
latents torch.Size([1, 8, 16, 60, 90])
Temporal tiling disabled
latents.shape torch.Size([1, 8, 16, 60, 90])
I've hit OOMs a couple of times but seem to be able to move forward straight afterwards. im on a laptop rtx a4500. Do you know what could be causing it? Non-I2V 5b model runs well.

I also couldn't get the down(load) and load cog model node to work so just git cloned from thud into models/cogvideo

The result is good on 49 frames.

I also get grid results,

I think the I2V model simply only works with 49 frames, those are the results with other frame amounts I've tried as well.

Ahh ok great, thanks.


Hide the scroll bar on the screenshot

Hey!
I just wanted to say that your tools are awesome and super helpful. But, I did notice a little issue on macOS. In the screenshot, the scroll bar is visible. Any chance you could fix that?
Thanks a bunch!

Thank you for the kind words. I will look into the feature and and check of this can actually be done 🤔.
Will keep you posted.

Hi @aplekhanov thank you for the patience! I have released a fix for the scrollbars. Please update to the latest version of the cli. Hope this works for you! Thank you again.


update deployment and services requirements url for istioctl add-to-mesh

i think it is better than now.

i think it is better than now.

Thanks. Sometimes the URLs moved into different sections in istio.io docs, other times it breaks because it gets removed from docs. I'm thinking to create commonplace to update these URL's so that we don't have to keep refactoring a lot of files.

/retest

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test release-notes_istio

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
1f4aea0bef79737f1f2d25e544ff39dfa9997082
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

unit-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test unit-tests_istio

integ-pilot-k8s-tests_istio
70bf37947e2ce00520c44e0e48371d64b7314180
link
/test integ-pilot-k8s-tests_istio

release-notes_istio
198f4d947fa9d73c41fd518aab5524e44a8b3409
link
/test release-notes_istio

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

/test release-notes_istio


add model, form validation with total test coverage

completed this, added the templates, wrote nice tests for model, views, forms using pytest and django test

?


Fix typing in get_possible_radios

Run python -m mypy .
formfyxer/pdf_wrangling.py:1110: error: Item "str" of "Union[str, BinaryIO, Mat]" has no attribute "shape"  [union-attr]
formfyxer/pdf_wrangling.py:1110: error: Item "BinaryIO" of "Union[str, BinaryIO, Mat]" has no attribute "shape"  [union-attr]
formfyxer/pdf_wrangling.py:1158: error: Item "str" of "Union[str, BinaryIO, Mat]" has no attribute "shape"  [union-attr]
formfyxer/pdf_wrangling.py:1158: error: Item "BinaryIO" of "Union[str, BinaryIO, Mat]" has no attribute "shape"  [union-attr]
formfyxer/tests/test_lit_explorer.py:47: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]
looks like the problem is that we re-use the variable img -- starts as a string and gets turned into a cv2 object. so typing is messed up.
def get_possible_radios(img: Union[str, BinaryIO, cv2.Mat]):
    """Even though it's called "radios", it just gets things shaped like circles, not
    doing any semantic analysis yet."""
    if isinstance(img, str):
        # 0 is for the flags: means nothing special is being used
        img = cv2.imread(img, 0)
    if isinstance(img, BinaryIO):
        img = cv2.imdecode(np.frombuffer(img.read(), np.uint8), 0)

    rows = img.shape[0]

Possible fix to try (use assert to tell mypy we have the proper type by the relevant line)
def get_possible_radios(img: Union[str, BinaryIO, cv2.Mat]):
    """Even though it's called "radios", it just gets things shaped like circles, not
    doing any semantic analysis yet."""
    if isinstance(img, str):
        # 0 is for the flags: means nothing special is being used
        img = cv2.imread(img, 0)
    elif isinstance(img, BinaryIO):
        img = cv2.imdecode(np.frombuffer(img.read(), np.uint8), 0)

    # Assure mypy that the type of img at this point is cv2.Mat
    assert isinstance(img, cv2.Mat), "img must be of type cv2.Mat here"

    rows = img.shape[0]
    ```

Fixed in #121. This was fine on mypy 1.2.0, and broke on mypy 1.3.0;  I think mypy started changing how they handle typing with isinstance and unions in https://github.com/python/mypy/pull/14923, which is the best possible candidate to what messed this up. Not sure how to fix it, I'm fairly confident the original code is fine, but eh.


Left recursive grammar

I'm trying to implement a complicated grammar and found myself in a blocking situation.
type:
    simple-type
    nullable-type
    struct-type
    enum-type
nullable-type:
    type   ?

Something like this throws an exception in the Parse.Ref I'm using to reference the Type parser in the NullableType parser. I am wondering if this at all possible, it looks like from the tests that it's not implemented yet. Also it seems other similar parsers (parsley) claim they handle "infinite look-ahead" parsers.
My question is how much is needed in Sprache to handle this kind of grammar? Is it at all possible? I don't mind doing the work but I want confirmation first before looking more into it. Thanks.

I'm experiencing the same difficulty.  How would one refactor this code to be right recursive and also support multidimensional arrays?
ex. int[][][][][]
type:
simple-type
array-type
struct-type
enum-type
array-type:
type   []

In my case I actually didn't have a standalone rule for the Array Type. Here's what I did:
from typ in (simple | struct | enum)

from arrays in Parse.String("[]").Token().Text().Many()

select(
      arrays.Aggregate(typ, (t, elem) => new ArrayType(t))
);```

Hope it helps.

Cheers,


FAST:Farm

Hi everyone, I encountered this error while running a simulation:
Generator speed:   29.5 RPM, Pitch angle:   0.8 deg, Power:     0.0 kW, Est. wind Speed:  15.6 m/s
Generator speed:   29.5 RPM, Pitch angle:   0.8 deg, Power:     0.0 kW, Est. wind Speed:  15.6 m/s
Generator speed:   29.5 RPM, Pitch angle:   0.8 deg, Power:     0.0 kW, Est. wind Speed:  15.6 m/s
Time: 18 of 200 seconds.  Estimated final completion at 10:46:12.
T1:FARM_UpdateStates:FWrap_Increment:FAST_Solution:FAST_AdvanceStates:SolveOption2c_Inp2AD_SrvD:In
flowWind_CalcOutput:CalcOutput:IfW_4Dext_CalcOutput [position=(-3.5124, -85.036, 145.7) in
wind-file coordinates]:Interp4D:Outside the grid bounds.
T2:FARM_UpdateStates:FWrap_Increment:FAST_Solution:FAST_AdvanceStates:SolveOption2c_Inp2AD_SrvD:In
flowWind_CalcOutput:CalcOutput:IfW_4Dext_CalcOutput [position=(-3.5124, -85.036, 145.7) in
wind-file coordinates]:Interp4D:Outside the grid bounds.
T3:FARM_UpdateStates:FWrap_Increment:FAST_Solution:FAST_AdvanceStates:SolveOption2c_Inp2AD_SrvD:In
flowWind_CalcOutput:CalcOutput:IfW_4Dext_CalcOutput [position=(-3.5124, -85.036, 145.7) in
wind-file coordinates]:Interp4D:Outside the grid bounds.
Aborting FAST.Farm.
How is it possible that after 18 seconds the program shows me this type of error? I have never seen it once the simulation has been started...how can I fix it? Thank you very much!

Dear @Tommi1198,
The error is similar to others you've posted recently, e.g.: https://github.com/OpenFAST/openfast/issues/1044.
If you receive this error at t = 18 s into a simulation rather than right after model initialization (near t = 0 s), this means that the turbine has deflected/displaced enough to trigger the error.  Either your model requires a slightly bigger high-resolution domain to accommodate your turbine deflection/displacement, or the model is experiencing an instability resulting in excessively large deflections/displacements that eventually trigger the error.  I'm not sure what wind turbine(s) you are simulating in FAST.Farm, but given the nodal position = (-3.5124, -85.036, 145.7) m (relative to the tower base at (0,0,0)) and turbine dimensions, is this nodal position expected or a sign that your model has gone unstable?  Plotting the deflections/displacements of the blade and tower (and platform, if relevant) would likely be informative.
Best regards,

Dear @jjonkman , how can I plot the deflections of blades and towers if the simulation was interrupted? What kind of file of output I have to use and with which program? (I thought about MATLAB, but I do not have the .out since the simulation cannot be completed).
Thanks a lot!

Dear @Tommi1198,
When you select OutFileFmt = 1 in the OpenFAST primary (*.fst) input file, OpenFAST will generate the *.out output file from model initialization through the time which OpenFAST is terminated.  You can plot the results however you want, including Excel, MATLAB, pyDatView, etc.
Best regards,

Dear @jjonkman , I tried to plot the deflections of blades and tower, and I obtain the following results:

maximum tower-top fore-aft defelction = 0.12 m
maximum blade out-of-plane tip deflection = 1.2 m
maximum blade in-plane tip deflection = 0.6 m
The turbine that I'm using is the DTU10MW wind turbine
Are these values acceptable or are they too high for my problem experienced?
Thank you very much!

Dear @Tommi1198,
These deflections are quite reasonable and do not suggest a problem with your OpenFAST model.  Can you clarify how you've set the extend of the high-resolution domain relative to the turbine size?  Are you leaving enough buffer in the size of the high-resolution domain to accommodate typical turbine deflection?
Best regards,

Dear @jjonkman , I have to do a correction...maximum blade in-plane tip deflection = 1 m
Is it still acceptable?

Dear @Tommi1198,
Yes, 1-m is still a reasonable in-plane blade deflection.
You didn't share the origin and size of the high-resolution domains, which are defined in the WIND TURBINES table, beginning on line 36 of the FAST.Farm (*.fstf) input file.
Best regards,

Dear @jjonkman , these are the lines requested by you:
3          NumTurbines   Number of wind turbines (-) [>=1] [last 6 columns below used only for Mod_AmbWind=2 or 3]
WT_X    WT_Y     WT_Z   WT_FASTInFile                                                                                X0_High Y0_High  Z0_High dX_High dY_High dZ_High
(m)     (m)      (m)      (string)                                                                                (m)     (m)      (m)     (m)     (m)     (m)
605.0   1500.0   0.0    "C:\Users\berna\Desktop\ProvaDTU10MW\FAST30\DTU_10MW_RWT_OpenFast_3_0\DTU10MW_Onshore_Ctrl_30.fst"          525.0   1425.0    5.0    10.0    10.0    10.0
605.0   1500.0   0.0    "C:\Users\berna\Desktop\ProvaDTU10MW\FAST30\DTU_10MW_RWT_OpenFast_3_0\DTU10MW_Onshore_Ctrl_30_Copia.fst"    525.0   1425.0    5.0    10.0    10.0    10.0
605.0   1500.0   0.0    "C:\Users\berna\Desktop\ProvaDTU10MW\FAST30\DTU_10MW_RWT_OpenFast_3_0\DTU10MW_Onshore_Ctrl_30_Copia2.fst"   525.0   1425.0    5.0    10.0    10.0    10.0
Thank you very much!

Dear @jjonkman , these are the lines requested by you:
3          NumTurbines
WT_X    WT_Y     WT_Z      WT_FASTInFile
(m)       (m)       (m)              (string)
605.0   1500.0   0.0            "...\DTU10MW_Onshore_Ctrl_30.fst"
605.0   1500.0   0.0           "...\DTU10MW_Onshore_Ctrl_30_Copia.fst"
605.0   1500.0   0.0           "...\DTU10MW_Onshore_Ctrl_30_Copia2.fst"
X0_High Y0_High  Z0_High dX_High dY_High dZ_High
(m)            (m)         (m)          (m)        (m)        (m)
525.0        1425.0      5.0         10.0       10.0       10.0
525.0        1425.0      5.0         10.0       10.0       10.0
525.0        1425.0      5.0         10.0       10.0       10.0
Thank you very much!

Dear @jjonkman , don't worry, finally I solved the problem! If I will have other issues I will write on the forum.
Thank you very much!

Dear @jjonkman , I now tried to run a simulation with the super controller active, but it gave me this error:
FAST.Farm-v3.0.0
Compile Info:

Compiler: Intel(R) Fortran Compiler 1900
Architecture: 64 bit
Precision: single
OpenMP: No
Date: Jun 23 2021
Time: 21:41:28
Execution Info:
Date: 03/28/2022
Time: 16:48:43+0200

Heading of the FAST.Farm input file:
Sample FAST.Farm input file
Running AWAE.
Running InflowWind.
forrtl: severe (157): Program Exception - access violation
Image              PC                Routine            Line        Source
How can I fix it? Thanks a lot

Dear @Tommi1198,
I'm not sure I understand why you are setting up your FAST.Farm model the way you have.  It appears that you've placed all three wind turbines at the exact same location and all three high-resolution domains are overlapping.  Is this what you intended to do?  Please clarify.
Regarding the super controller, do you only get this error when enabling the super controller?  Or is the problem identical to the one you reported earlier: https://github.com/OpenFAST/openfast/issues/1044?  If the problem only occurs when the super controller is enabled, can you clarify how you are setting up the super controller?  Are you using an example super controller provided by NREL or something you implemented yourself?
Best regards,

Dear @jjonkman , regarding the first question you are right, I made a mistake (I will try to run the simulation adjusting the reference grid)
Regarding the second question on the supercontroller, the error is shown only when enabling it, and I'm using the example provided by NREL; what can I do to fix the error?


Test installers on all CI runs

We currently only test on PRs. We should test on all CI builds.

This is done.


feat: update node.js versions

Automated changes by update-node-versions GitHub action

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


Create Custom Lineage between Azure Synapse SQL Table and Power BI Dataset facing issues

What is your desired outcome
Would like to use the API to create custom lineage between Azure Synapse SQL Table and Power BI dataset
What have you tried so far
What steps have you taken to get your desired outcome?
Setup the script locally got it working the samples from your demo but when I try to add a line in the Excel file to connect Synapse SQL Table with Power BI it doesn't work and I cant figure it out from the error.
Using this script as is lineage_upload

Target typeName
Target qualifiedName
Source typeName
Source qualifiedName
Process name
Process qualifiedName
Process typeName

power_bi_dataset
https:app.power.bi.....459594
azure_sql_table
mssql://syn445/hockey/team
My Custom Process
custom://process-to-be-made
Process

Getting azure_sql table entity not found
Additional context
Add any other context about the problem here.

I resolved the issue I needed to change the source type to azure_synapse_sql_table


added huffman code issue-3306

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
Huffman coding works on data stream that makes up the file to be compressed
output for input "testing code with inputs"  is:
huffcode first printed in row
element printed in second row
weights are printed last of row
00        t 4
100         4
011       i 3
1011      e 2
1101      n 2
1111      s 2
0100      u 1
0101      w 1
10100     c 1
10101     d 1
11000     g 1
11001     h 1
11100     o 1
11101     p 1
Checklist

[yes ] I've read the contribution guidelines.
[ yes] I've checked the issue list before deciding what to submit.
[ yes] I've edited the README.md and link to my code.

Related Issues or Pull Requests
please verify

@bhav09 , @siddharth25pandey @HarshCasper , @ankitaggarwal23Kindly review the PR

@kavyasree-123 We don't accept PRs without any corresponding Issue. It is difficult for us to find your issue in tons of issues available. We can't review and merged incomplete PR's.

@kavyasree-123 Kindly open a new PR add the proper details and Link the issue no. in the description properly.

Hello I'm not able to create a new issue even though we have closes
including brncahes issues and pr it's still saying I'm having 2 open what
to do..
On Wed, Mar 17, 2021, 12:27 PM Ankit Aggarwal @.***>
wrote:

@kavyasree-123 https://github.com/kavyasree-123 Kindly open a new PR
add the proper details and Link the issue no. in the description properly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TesseractCoding/NeoAlgo/pull/3373#issuecomment-800845476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTMFWIFCWWFTQKXIQK5XCDTEBHEJANCNFSM4ZIBCI3Q
.

@kavyasree-123 I said to create a new PR not the issue. It's correct bot will close your issue because you are creating a new issue not the PR.


_astro files will be ignored.

When deploying using github pages, The _astro file will be ignored.

Need to add .nojekyll file in the root directory, or modify build.assets.