Move Docker build definitions to project repositories

Problem
Currently all Dockerfiles live in this repository for all OpenZipkin projects, and all Docker images are released together. See RELEASE.md@71b4c251 for the current release process. This enforced synchronization sets bad expectations for rarely maintained projects (what do you mean out of date? the image is two hours old), and also potentially delays releases of more active projects. The current setup is also surprising, Dockerfiles are expected to live in the same repository as the project itself.
Possible approaches
Approach 1: in-repo

Move the Dockerfiles and files used by them (for example in ADD commands) to the respective repository of each project
Configuration on Quay.io for each image needs to be updated to use the Dockerfile from the new location
At this point the release process is still monolithic, but we've reduced surprises a bit by moving Dockerfiles to separate repos and prepared for the next steps.
Copy-paste relevant parts of release.sh@d1e38a5580 to each project
Configure Travis job to run the Docker build or the normal build depending on how the build was triggered (special tags for Docker builds), as described in https://github.com/travis-ci/travis-ci/issues/2646.

Pro: Dockerfile in the project repository, as expected
Con: Extra complexity in Travis config; no way to tell at a glance whether a Travis build is a normal build or a Docker build
Approach 2: mini repos

For each project with a Docker image create a new repository $PROJECT-docker
(Same as Approach 1) Configure on Quay.io for each image needs to be updated to use the Dockerfile from the new position
At this point the release process is still monolithic, but we've reduced surprises a bit by moving Dockerfiles to separate repos and prepared for the next steps.
(Same as Approach 1) Copy-paste relevant parts of release.sh@d1e38a5580 to each project
Create new Travis job

Pro: low complexity, clear separation of Travis jobs
Con: Dockerfile in separate repository (may be a pro, depending on where you sit); lots of repositories (may be irrelevant)
Follow-up
Process
Once this is done, the release of new Docker images very clearly becomes the responsibility of the maintainers of each project. This is a good thing, but we all need to be clear and on board with this change. Versioning of images becomes a question: how do we track what versions are compatible with what other versions, if we track it at all? Is the version of the image the version of Zipkin always, sometimes, never? How do we show a breaking change in an image that comes without a new release of Zipkin? Personally I think we can put off coming up with strict answers to these questions, but let's keep them in mind going forward, and keep open the option of sitting down to find definite answers if it becomes necessary.
Tech: auto-release
Successful non-SNAPSHOT releases of each project can now easily trigger a build/release of its Docker image.
Tech: DRY
Exploding release.sh to multiple repositories introduces a lot of code repetition, which will lead to tiny differences in each project (I'm working on a half-serious theory of evolutionary release scripts), badness all around. Ways to mitigate this badness:

Each new release script will have a smaller scope than the original monolithic one, this can be used to remove some abstractions and complexity. I'm planning to do some of this during the initial migration.
Some of the complexity is due to orchestrating Quay.io and Dockerhub. Dockerhub has released features since the initial setup of the Docker Zipkin build process that should enable dropping Quay.io
If there's still a lot of repeated complexity, we can move that to a shared script in a shared repository (something like common.sh in docker-zipkin that each build process downloads at run-time). This is a different kind of complexity, let's see if it's worth it when we get there.

@adriancole I'd appreciate your thoughts on this. Also: who else should be involved?

It seems we should trim the weeds around this problem space, so it is less conflated with known difficulty. For this reason, I'm +1 on migrating to dockerhub before doints this. We can then re-summarize the desc.
Some other options I thought about while reading (not well thought through)

release of docker can still be monolithic if we do submodules for the participants.
we can use a specific docker release function w/o copy/paste if we release it, or check it out as a part of travis setup for the modules. inceptiony, but yeah.

Thanks for the input!

Docker Hub first: I'm kind of on the fence about this. Your argument totally makes sense, I'll think more on it.
Monolithic release: yes, especially if go with mini repos; I'd avoid checking out all the Zipkin source code ever. Do you think the monolithic release is a Good Thing (tm)?
Custom release function without copy/paste: indeed, that's what pretty much what I had in mind in the last bullet-point under DRY. Glad we're on the same page there.

Docker Hub first: I'm kind of on the fence about this. Your argument
totally makes sense, I'll think more on it.

ack. I understand these days the best repo changes a lot!

Monolithic release: yes, especially if go with mini repos; I'd avoid
checking out all the Zipkin source code ever. Do you think the monolithic
release is a Good Thing (tm)?

agreed maybe submodule was a bad idea :P too bad we can't just fetch a
subtree by tag.. most of the time, there are only 2 files relating to
docker!

wrt "monolithic release" I think this is just a challenge when we publish
things like docker-compose files. I suppose we could lower the maintenance
by scaling back the version precision. Very open to better practice here..
I don't have a great answer for a modular pipeline and also a compose file.

Custom release function without copy/paste: indeed, that's what
pretty much what I had in mind in the last bullet-point under DRY. Glad
we're on the same page there.

me too

Alright, let's do Docker Hub first. I think it's straightforward, I'll start work on it and open up things for discussion if I come to non-trivial trade-offs.


Improve connector docs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: JakeSCahill:x: joejulianYou have signed the CLA already but the status is still pending? Let us recheck it.

@RafalKorepta is this one OK to merge? Not sure why the summary doesn't get generated.

https://github.com/redpanda-data/helm-charts/pull/820/commits/5c2b5b3e25f45fa18d8d84dd5903544a3de897ed
The above commit should unblock this PR.

KMinion lint is false positive. The Github workflow has changed (which trigger whole pipeline), but the underlaying chart didn't.


app() should not remove root element

Right now, if you have a body like this:


Horizontal line markup error

Current behavior
Screenshot
Screenshot
--- increases size and thickness of previous text
Expected behavior
--- should create a horizontal line

Please, use markdown-it demo next time and check official dingus first.
Works as expected (official implementation). See commonmark spec for details


Configurable number of lines for error messages?

It would seem that ghcid is truncating error messages to 3 (?) lines. That's somewhat counter to what I was used to. I can imagine why you did it (sometimes you want to see all the errors, not just the first one) but I find myself having to ^C, run stack build manually to see the full error, then back to ghcid.
Am I missing something?
AfC

If there are lots of errors, and only a few lines to display them, it truncates the first one to 3 lines, and the rest to 2 lines.  My workflow tends to be to resize the window if I really need more, but again, that's not great. https://github.com/ndmitchell/ghcid/blob/master/src/Ghcid.hs#L217 is the code that does this. Thoughts on what the algorithm should be? Or should there be a key to toggle, as per #10?

So if you're talking about "key to toggle" that implies a) changing focus to that window and b) making it a more interactive environment. Which is a very cool idea indeed, though I would point out that right now the attraction of ghcid is, to some extent, that I don't have to leave the editor. That said, navigating through the errors with, say, the arrow keys, would be something to think about. If that's the case, I would encourage you to look at the UX of perf top on a Linux box. It's surprisingly solid.
Meanwhile as for the height to show, I seriously wonder if there's any value to showing more than one at a time (I wish ghc had an equivalent to gcc's -Wfatal-errors), but I suppose --max-errors=n option would do the trick (default 1 :)). If not, then maybe just divide available height by number of errors (but still capping it at say a minimum of 5 lines per error).
2¢
AfC

Agreed, I'd rather the interactive features be an added bonus, rather than a necessity. I haven't tried perf top, but then I'm not a Linux user.
So I already follow your idea of dividing available height by errors, but use a minimum of 3 lines for the first error and 2 for subsequent errors. I'll try increasing the first error height to 5 and see how that feels locally. I'm not against a max-errors, or a minimum-first-height option (where infinity gives you roughly the same thing), but one less flag for users to have to fight is always preferable.

I tried hacking it to a max 5 lines per error for all of them and it seemed to feel ok. {shrug}

I think the first error having special treatment seems like a good idea - it's the one you should be focusing on - other than that its nice to know where other errors are as they are sometimes fallout, or sometimes GHC picks the wrong line.
Out of curiosity, how many lines (roughly) do you typically have for ghcid? I am usually on about 7.

OK, first error is 5 lines high. Let's hope that fixes it, but please reopen if it doesn't seem to do what you are after.

@ndmitchell How many lines? Full terminal, so 80x24 standard if stand alone or half a screen, which is 72x37 on this monitor with this font size today.
(half a screen for editor, other half for terminal with ghcid in it.... in no small part because I want to see the whole error message, not the single line that ghc-mod and friends provide. Quite tedious to have to abort ghcid and run the build manually just to see the error)
I'll keep experimenting with it!
AfC

Ok, I use 7, so I've inadvertently optimised for that. I'll try and figure out a more sensible default.

I think that the first error should be shown entirely. What's the point of seeing many errors at once if you can't fix any of them? You're assuming that the first n lines of the error message are somehow more important than the rest; I don't see any ground for this.

@feuerbach, the first 2 lines have the type of error and the file/line so are definitely most important. I am gradually coming round to showing the first error in full, since if you don't then you really want an option to show it in full, and better default to no options if you can.

@ndmitchell you are thinking about a specific kind of errors. For instance, I had an ambiguous identifier error recently, and the second conflicting import wouldn't come until the line 6 or 7.
But even in case of type errors, I find the information contained in the message very useful.

For what it's worth, I am going to release a new terminal UI programming library (probably in the next few weeks) that would be a great fit for putting an interactive UI on top of ghcid.  I will probably mess around with such a thing myself.  The library is https://github.com/jtdaugherty/brick.

@jtdaugherty, does the library works on Windows? From what I can see, the vty dependency doesn't?

The basic algorithm at the heart of the bug is given a list of multiline items (errors followed by warnings), and a height, how do you pack them. The old algorithm was:
take (max 5 $ height - (length msgs * 2)) (head msgs) ++ concatMap (take 2) (tail msgs)

The new algorithm is:
take height $ concat msgs
This guarantees the first message will be shown in full, as will any subsequent messages, until it runs out of space. It means you get more information about fewer messages. Does that seem to be what people are after?

Looks good to me.

@ndmitchell - good question! vty claims to support Cygwin only. I haven't made any special effort to support Windows in my package, but I don't test on Windows, either.

@ndmitchell I think that'll be fine. I know you went to a lot of trouble to compactly show messages, but honestly if people want all the errors, then they can use ghc-mod et al to give them red squigglies in their editor at all the error sites with then incomprehensible one line messages.

This seems to work nicely for me, so considering it fixed.


Not a good practice to manipulate System level property and only allow TLS 1.2

In ApiClient.java
Following codes are being executed every time the API Client is created.
System.setProperty("https.protocols", "TLSv1.2");
and
HttpsURLConnection.setDefaultSSLSocketFactory(ctx.getSocketFactory());
This is not the right approach for the reasons

A client library should not change the caller/system level behavior which could create un-wanted impact to the whole system.
Setting system level protocols to only support 1.2 prevented the use of other protocol such as  TLSv1.3 by the system
setProperty changes the global variable and it is not thread safe

The recommended approach should be

Check to make sure TLSv1.2 is supported and throw exception if it is not.

Hello @thetasp, thanks for bringing this to our attention. I went ahead and filed DCM-8192 for our engineers to take a look.
-Zack DocuSign Support

Another option to consider is to remove the enforcement of TLS 1.2 from the client and only enforce it from the server. ie. Server only accept TLS 1.2 or above. The SSL Handshake will automatically find the protocol that can be accepted by both parties. This will eliminate SSL vulnerabilities across all connections, not only to those who use docusign client

Hi @thetasp ,
The fix was provided in 2022, Since the functionality is now working as intended, I'm going ahead and closing this issue.
Feel free to open new issue if you still face the same problem or have any new ideas / suggestions.
Thanks,
Vinay

hi @vinz , the funtionality is not working as intended. We even have a bug opened with docusign which is still in progress since March. The change referenced by @thetasp was pushed on a branch that doesn't seem to have gotten merge in any release.


sched/sched: There is no need to use sched_[un]lock

Summary
There is no need to use sched_[un]lock
Impact
Testing

@hujun260
Please describe which configurations and applications you tested in the description.
Also, please change the commit message to more specific ones.

purpose:
1 sched_lock is very time-consuming, and reducing its invocations can improve performance.
2 sched_lock is prone to misuse, and narrowing its scope of use is to prevent people from referencing incorrect code and using it
test:
We can use qemu for testing.
compiling
make distclean -j20; ./tools/configure.sh -l qemu-armv8a:nsh_smp ;make -j20
running
qemu-system-aarch64 -cpu cortex-a53 -smp 4 -nographic -machine virt,virtualization=on,gic-version=3 -net none -chardev stdio,id=con,mux=on -serial chardev:con -mon chardev=con,mode=readline -kernel ./nuttx
We have also tested this patch on other ARM hardware platforms.


Need help to build the project on raspberry pi 5 (raspberry pi OS)

I've installed svg2gcode, but I'm unsure how to proceed with balance steps, and I've been unable to locate the convert.py file.

Hi sorry for not updating the readme, but you should be able to run python3 main.py without needing any external binary dependencies.
Please post any errors that occur as i have not tested HandCode on Linux systems yet

Thank you for your prompt response. I've encountered an error while running main.py on Raspberry Pi OS.
traceback (most recent call last):
File "/home/pi/hancode/handcode-main/src/main.py", line 709, in
main()
File "/home/pi/hancode/handcode-main/src/main.py", line 54, in main
window.iconbitmap(os.path.join(path_lib, "icon.ico"))
File "/usr/lib/python3.11/tkinter/init.py", line 2136, in wm_iconbitmap
return self.tk.call('wm', 'iconbitmap', self._w, bitmap)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
_tkinter.TclError: bitmap "/home/pi/hancode/handcode-main/src/lib/icon.ico" not defined

Could you try commenting out the line throwing the error (line 54 of main.py)?
This seems to be a common issue in tkinter: https://stackoverflow.com/questions/11176638/tkinter-tclerror-error-reading-bitmap-file

After commenting out the 54th line, the graphical interface has opened, but another error message has appeared.
The following unhandled
exception occured: No module named 'tensorflow'
Traceback (most recent call
last):
File
"/home/pi/handcode/src/main.py
", line 396, in threadConvert
import handwriting
File
"/home/pi/handcode/src/lib/hand writing/init_.py", line 1, in

from. import gcode
File
"/home/pi/handcode/src/lib/hand writing/gcode.py", line 3, in
from.rnn import rnn
File
"/home/pi/handcode/src/lib/hand writing/rnn.py", line 5, in

import tensorflow as tf ModuleNotFoundError: No module named 'tensorflow'
(<class 'ModuleNotFoundError'>, ModuleNotFoundError("No module named 'tensorflow"), <traceback object at 0x7fff888a9e80>)

After installing TensorFlow using pip, I encountered another error message.
The following unhandled exception occured: main thread is not in main loop
Traceback (most recent call last):
File
"/home/pi/handcode/src/main.py ", line 395, in threadConvert report("log", "Loading tensorflow... ")
File
"/home/pi/han/code/src/main.py ", line 416, in report edtLog.append(data)
File
"/home/pi/handcode/src/lib/tkwi dgets/Scrolled Entry.py", line 20, in append
self.entry.insert(tk.END, text)
File
"/usr/lib/python3.11/tkinter/ini t.py", line 3809, in insert self.tk.call((self._w, 'insert', index, chars) + args) RuntimeError: main thread is not in main loop
(<class 'RuntimeError'>, RuntimeError('main thread is not in main loop'), <traceback object at 0x7ffef03c9a00>)

Alright, thanks for the logs, i will have a look at this and come back to you in a few hours or so

Hi, I think tensorflow version 2.11.0 is not available for Raspberry pi OS, because i got an error like  :
ERROR: Could not find a version that satisfies the requirement tensorflow==2.11.0 (from versions: 2.12.0rc0, 2.12.0rc1, 2.12.0, 2.12.1, 2.13.0rc0, 2.13.0rc1, 2.13.0rc2, 2.13.0, 2.13.1, 2.14.0rc0, 2.14.0rc1, 2.14.0, 2.14.1, 2.15.0rc0, 2.15.0rc1, 2.15.0, 2.15.1, 2.16.0rc0, 2.16.1)
ERROR: No matching distribution found for tensorflow==2.11.0

You may need an older version of python. I tested on python 3.10.4, you should be able to install it on Raspbian using sudo apt-get install python3=3.10.4
Tensorflow 2.11 is sadly not supported on newer versions of python and my code breaks on newer versions of tf. I will try to fix this in the future but this should be a fix for now.

Yes, I will certainly give that a try. Does the same process apply for Ubuntu as well?

Yes, the same process applies to Ubuntu. Other linux distros may use different package managers, so you may need to look up the specific commands (i.e. yum, pacman, ...) to install the specific python version, but for Debian, Raspbian and Ubuntu ´apt´ should work.

On changing pyenv to 3.10.4, Tensorflow and tensorflow probability got installed but not keras. I encountered the error stating
ERROR: Could not find a version that satisfies the requirement tf-keras==2.11.0 (from versions: 2.14.1, 2.15.0rc0, 2.15.0rc1, 2.15.0, 2.15.1rc0, 2.15.1, 2.16.0rc0, 2.16.0rc1, 2.16.0rc2, 2.16.0rc3, 2.16.0rc4, 2.16.0)
ERROR: No matching distribution found for tf-keras==2.11.0
WARNING: You are using pip version 22.0.4; however, version 24.0 is available.
You should consider upgrading via the '/home/pi/.pyenv/versions/3.10.4/bin/python3.10 -m pip install --upgrade pip' command.

My bad, copy-paste-error. Please try with tf-keras==2.14.1 and report back. If it works i will fix the requirements.txt together with updating the readme :)

Tried with keras 2.14.1, installed succesfully. But on running main.py produces a similar error message.
The following unhandled exception occured: No module named 'tensorflow'
Traceback (most recent call last):
File
"/home/pi/handcode/src/main.py ", line 440, in thread Convert import handwriting
File
"/home/pi/handcode/src/lib/hand writing/init.py", line 1, in

from. import gcode
File
"/home/pi/handcode/src/lib/hand
writing/gcode.py", line 3, in

from.rnn import rnn
File
"/home/pi/handcode/src/lib/hand writing/rnn.py", line 5, in

import tensorflow as tf ModuleNotFoundError: No module named 'tensorflow'
(<class 'ModuleNotFoundError'>, ModuleNotFoundError("No module named 'tensorflow'"), <traceback object at 0x7ffed178a080>)

You have to reinstall all python libraries (tf, keras, ...) for python 3.10, i presume the libraries were previously installed for the other python3 version you had. Please check if the python version for pip is also python 3.10. You can check this using pip -V or pip3 -V. If it is 3.10 you can simply use pip or pip3, otherwise you can use python -m pip to get pip for the correct version of python, and you can use it to reinstall requirements.txt, so either pip install -r requirements.txt or python -m pip install -r requirements.txt

Sure, I'll give that a try and let you know how it goes.
Additionally, I'm curious if it's possible to incorporate our custom handwriting font style. I've created an .otf file using a calligraphy template. Can we apply a similar process in this context?

Use tf-keras==2.14.1.
Yes, you can use your own handwriting but you have to train the model with (very very many) handwriting strokes. It is relatively time consuming to get enough reliable training and testing data and to understand the training process, but i think i have linked something in the readme about this.
My program does not support fonts (your otf file is a font), it only supports trained handwriting which requires way more data than just one drawing per character, but feel free to test one of the 12 pretrained handwriting styles. If you want to use a custom font, you simply need to use a word processor (Microsoft Word, OpenOffice, LibreOffice, ...) and import your font into either the system or program context but i guess there are way better tutorials on how to import a custom font into your word processor of choice than i could give you.

I have reinstalled everything as specified and i encountered another error on running main.py
The following unhandled exception occured: module
'tensorflow' has no attribute
'compat'
Traceback (most recent call last):
File
"/home/pi/handcode/src/main.py ", line 440, in thread Convert import handwriting
File
"/home/pi/handcode/src/lib/hand writing/init_.py", line 1, in

from. import gcode
File
"/home/pi/handcode/src/lib/hand writing/gcode.py", line 3, in

from.rnn import rnn
File
"/home/pi/handcode/src/lib/hand writing/rnn.py", line 6, in

tf.compat.v1.disable_v2_beh
avior() AttributeError: module
'tensorflow' has no attribute
'compat'
(<class 'AttributeError'>,
AttributeError("module
'tensorflow' has no attribute
'compat'"), <traceback object at 0x7fff5d468d00>)

Try installing pip install tf-estimator==2.11.0, might be an issue with tf dependencies

Received this error:
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
ERROR: Could not find a version that satisfies the requirement tf-estimator==2.11.0 (from versions: none)
ERROR: No matching distribution found for tf-estimator==2.11.0

Alright, im all out of ideas. You can try reinstalling everything.
So to uninstall everything
sudo apt-get clean
sudo apt-get autoremove --purge
sudo apt-get remove python3
sudo apt-get autoremove

Do the above mentioned steps as long as calling python3 or pip3 does not throw a not found error
and then reinstall everything
sudo apt-get install python3.10
cd path/to/handcode
pip3 install -r requirements.txt

If the issue persists after that i have no clue, the steps mentioned above have been tested on a clean Ubuntu install.
In the future i will try to switch to tflite for more portability, snappier interactions and smaller build size, but until then you may have to troubleshoot your error yourselves. I may be able to help though, if a different error occurs.

I formatted my raspberry pi and installed everything as you specified. But sadly i am getting the same error message as "Tensorflow module not found".

Hi There is an update regarding the issue i was facing previously.
I used python 3.10.10
When i run main.py in VS Code the error regarding the tensorflow is now not showing and everything got loaded perfectly.

But when i clicked "start" button after typing some texts it shows "saving settings"

I will share the messages in the terminal:-

On running the program:
(venv) pi@raspberrypi5:~/handcode $ /home/pi/handcode/venv/bin/python /home/pi/handcode/src/main.py
WARNING:tensorflow:From /home/pi/handcode/venv/lib/python3.10/site-packages/tensorflow/python/compat/v2_compat.py:107: disable_resource_variables (from tensorflow.python.ops.variable_scope) is deprecated and will be removed in a future version.
Instructions for updating:
non-resource variables are not supported in the long term

new run with parameters:
{'attention_mixture_components': 10,
'batch_size': 32,
'batch_sizes': [32, 64, 64],
'beta1_decay': 0.9,
'beta1_decays': [0.9, 0.9, 0.9],
'checkpoint_dir': '/home/pi/handcode/src/lib/handwriting/checkpoints',
'early_stopping_steps': 1500,
'enable_parameter_averaging': False,
'grad_clip': 10,
'keep_prob_scalar': 1.0,
'learning_rate': 0.0001,
'learning_rates': [0.0001, 5e-05, 2e-05],
'log_dir': '/home/pi/handcode/src/lib/handwriting/logs',
'log_interval': 20,
'logging_level': 20,
'loss_averaging_window': 100,
'lstm_size': 400,
'min_steps_to_checkpoint': 2000,
'num_restarts': 2,
'num_training_steps': 100000,
'optimizer': 'rms',
'output_mixture_components': 20,
'output_units': 121,
'patiences': [1500, 1000, 500],
'prediction_dir': '/home/pi/handcode/src/lib/handwriting/predictions',
'reader': None,
'regularization_constant': 0.0,
'restart_idx': 0,
'validation_batch_size': 32,
'warm_start_init_step': 17900}
WARNING:tensorflow:From /home/pi/handcode/src/lib/handwriting/rnn.py:132: dynamic_rnn (from tensorflow.python.ops.rnn) is deprecated and will be removed in a future version.
Instructions for updating:
Please use keras.layers.RNN(cell), which is equivalent to this API
From /home/pi/handcode/src/lib/handwriting/rnn.py:132: dynamic_rnn (from tensorflow.python.ops.rnn) is deprecated and will be removed in a future version.
Instructions for updating:
Please use keras.layers.RNN(cell), which is equivalent to this API
/home/pi/handcode/src/lib/handwriting/rnn_cell.py:81: UserWarning: tf.nn.rnn_cell.LSTMCell is deprecated and will be removed in a future version. This class is equivalent as tf.keras.layers.LSTMCell, and will be replaced by that in Tensorflow 2.0.
cell1 = tf.compat.v1.nn.rnn_cell.LSTMCell(self.lstm_size)
WARNING:tensorflow:From /home/pi/handcode/venv/lib/python3.10/site-packages/keras/layers/rnn/legacy_cells.py:1048: calling Zeros.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
From /home/pi/handcode/venv/lib/python3.10/site-packages/keras/layers/rnn/legacy_cells.py:1048: calling Zeros.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
/home/pi/handcode/src/lib/handwriting/rnn_cell.py:105: UserWarning: tf.nn.rnn_cell.LSTMCell is deprecated and will be removed in a future version. This class is equivalent as tf.keras.layers.LSTMCell, and will be replaced by that in Tensorflow 2.0.
cell2 = tf.compat.v1.nn.rnn_cell.LSTMCell(self.lstm_size)
/home/pi/handcode/src/lib/handwriting/rnn_cell.py:110: UserWarning: tf.nn.rnn_cell.LSTMCell is deprecated and will be removed in a future version. This class is equivalent as tf.keras.layers.LSTMCell, and will be replaced by that in Tensorflow 2.0.
cell3 = tf.compat.v1.nn.rnn_cell.LSTMCell(self.lstm_size)
WARNING:tensorflow:From /home/pi/handcode/venv/lib/python3.10/site-packages/tensorflow/python/training/rmsprop.py:188: calling Ones.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
From /home/pi/handcode/venv/lib/python3.10/site-packages/tensorflow/python/training/rmsprop.py:188: calling Ones.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
all parameters:
[('Variable:0', []),
('Variable_1:0', []),
('Variable_2:0', []),
('rnn/LSTMAttentionCell/lstm_cell/kernel:0', [476, 1600]),
('rnn/LSTMAttentionCell/lstm_cell/bias:0', [1600]),
('rnn/LSTMAttentionCell/attention/weights:0', [476, 30]),
('rnn/LSTMAttentionCell/attention/biases:0', [30]),
('rnn/LSTMAttentionCell/lstm_cell_1/kernel:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_1/bias:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/kernel:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/bias:0', [1600]),
('rnn/gmm/weights:0', [400, 121]),
('rnn/gmm/biases:0', [121]),
('rnn/LSTMAttentionCell/lstm_cell/kernel/RMSProp:0', [476, 1600]),
('rnn/LSTMAttentionCell/lstm_cell/kernel/RMSProp_1:0', [476, 1600]),
('rnn/LSTMAttentionCell/lstm_cell/bias/RMSProp:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell/bias/RMSProp_1:0', [1600]),
('rnn/LSTMAttentionCell/attention/weights/RMSProp:0', [476, 30]),
('rnn/LSTMAttentionCell/attention/weights/RMSProp_1:0', [476, 30]),
('rnn/LSTMAttentionCell/attention/biases/RMSProp:0', [30]),
('rnn/LSTMAttentionCell/attention/biases/RMSProp_1:0', [30]),
('rnn/LSTMAttentionCell/lstm_cell_1/kernel/RMSProp:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_1/kernel/RMSProp_1:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_1/bias/RMSProp:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell_1/bias/RMSProp_1:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/kernel/RMSProp:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/kernel/RMSProp_1:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/bias/RMSProp:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/bias/RMSProp_1:0', [1600]),
('rnn/gmm/weights/RMSProp:0', [400, 121]),
('rnn/gmm/weights/RMSProp_1:0', [400, 121]),
('rnn/gmm/biases/RMSProp:0', [121]),
('rnn/gmm/biases/RMSProp_1:0', [121])]
trainable parameters:
[('rnn/LSTMAttentionCell/lstm_cell/kernel:0', [476, 1600]),
('rnn/LSTMAttentionCell/lstm_cell/bias:0', [1600]),
('rnn/LSTMAttentionCell/attention/weights:0', [476, 30]),
('rnn/LSTMAttentionCell/attention/biases:0', [30]),
('rnn/LSTMAttentionCell/lstm_cell_1/kernel:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_1/bias:0', [1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/kernel:0', [876, 1600]),
('rnn/LSTMAttentionCell/lstm_cell_2/bias:0', [1600]),
('rnn/gmm/weights:0', [400, 121]),
('rnn/gmm/biases:0', [121])]
trainable parameter count:
3632431
built graph
restoring model parameters from /home/pi/handcode/src/lib/handwriting/checkpoints/model-17900
INFO:tensorflow:Restoring parameters from /home/pi/handcode/src/lib/handwriting/checkpoints/model-17900
Restoring parameters from /home/pi/handcode/src/lib/handwriting/checkpoints/model-17900
2024-03-24 13:20:48.735055: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:357] MLIR V1 optimization pass is not enabled

On clicking the start buton:
Exception in Tkinter callback
Traceback (most recent call last):
File "/home/pi/.pyenv/versions/3.10.4/lib/python3.10/tkinter/_init.py", line 1921, in _call
return self.func(*args)
File "/home/pi/handcode/src/main.py", line 429, in startConvert
saveSettingsToFile()
File "/home/pi/handcode/src/main.py", line 735, in saveSettingsToFile
file = open(_pathSettings, "w")
FileNotFoundError: [Errno 2] No such file or directory: 'data/settings.json'

Also the program runs only in VS code. When i run the program directly from terminal, its shows the previous tensorflow module not found error.

Thank you so much for helping me It works perfectly now !!!!!!!!!
(For me it worked by using python 3.10.10)

Happy to help, have tun with HandCode 👍

Hi, Need a quick help
When providing input text and initiating the process, an error occurs stating "unhandled error: name 'command_page_pause' is not defined."

Additionally, when attempting to run the generated G-code file through Universal G-code Sender, the servo motor controlling the pen does not move. (this was before the above error occured)

Hi, im really sorry for the delay, i will look into this an come back to you within the next week.
In the meantime, please have a look into the generated GCode/NC file. It should contain commands like G0 Z0.000 (travel->write moves) and G0 Z5.000 (write->travel moves). HandCode was built for machines supporting 3 freely controllable axis (i.e. cnc routers, 3d printers). You state that you have a binary Z-Axis (pen down/pen up).

The actual Z value are what you entered in GCode as Z heights for write and travel moves. It could be G1, G01, G00 instead of G0. The actual commands replacing the current z moves are probably not M3 and M5, they are probably documented somewhere in the documentation of the GCode interpreter you are using.
If it is still not working, please open a new issue and close this again as those are unrelated topics.

Sure i will check it out.

A new issue for supporting servo based (+ motor and laser based) machines has been opened: #2


Upload latest

Training data validator
cli
event for data importer
lambda for data importer
docker image for data importer
log processor for data importer
api to upload+validate data and separate api to validate existing bot data
moved agent processor, mongo processor, training data generation processor to their own class file
unit and integrations tests for the above

closed due to conflicts


[MM] cannot see last item in mobile menu

Detailed description

Describe in detail the issue you're having.

When browsing the mobile mega menu, I cannot see the last menu item WebSphere in the Top Products & Platforms category
Screen recording

Is this a feature request (new component, new icon), a bug, or a general
issue?

bug

Is this issue related to a specific component?

mega menu

What did you expect to happen? What happened instead? What would you like to
see changed?

What browser are you working in?

Safari
iOS 13.6
iPhone 7

What version of the IBM.com Library are you using?

Latest canary, seen on Homepage staging environment

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

The details from this issue will be added to (#2661)

Information transferred to issue (#3661) Closing


Passing a closure to if method throws an error

The MorphTo example here indicates a way to pass a closure to the if method, but doing so causes an error.
Error:
Argument 1 passed to
App\Nova\InvoiceItem::DigitalCreative\ConditionalContainer\{closure}()
must be of the type string, object given

@abishekrsrikaanth fixed on latest version


Error after deleting files

When I delete a folder with code from a crate in my workspace, cargo diet dies with the error
Error: Could not open "src/workspace/constructor.rs" for reading file meta-data, where workspace is the folder containing some code. If it exists (even when the files contained are empty), it works. If I delete any file (or even the whole folder), I get the error above.

Thanks for reporting! This is interesting as it seems pretty clear where this issue stems from:
https://github.com/the-lean-crate/cargo-diet/blob/abca3ea180ca92ceb5662fa454fcaaf4527db542/src/lib.rs#L232-L252
cargo package is executed to get a list of all paths that it would want to include in the package. This is later used to build a tar archive. For some reason cargo package lists paths that are not present on disk.
The fix would be trivial - ignore non-existing files - but I wonder what you think about this before implementing it.

In my case prio#1 would be that the tool does not die because of this, since I execute it in a loop over all crates within my workspace during CI. Of course it would be helpful if cargo package itself does not return such false positives, but in the short term I'd gladly take the tradeoff that ignores things even if this means not-as-optimal shrinking. Thoughts?

Please let me know if v1.2.1 works for you.

Worked this night without issues, thank you!


Locally new rules do not get evaluated

Running cortex lite locally. I haven't confirmed if this is happening with other configurations.
After setting up the alert manager config and adding 8 rules, I see:
time="2017-08-09T13:02:26Z" level=debug msg="Evaluating 8 rules..." orgID=1 source="ruler.go:244" and the rules are evaluated.
After adding a new rule (copied and pasted from an existing rule and changing the name), I see:
time="2017-08-09T13:02:27Z" level=debug msg="Adding 1 configurations" source="scheduler.go:179"
However the logs still show only 8 rules are evaluated. Querying the new rule also returns no data. Only after restarting the ruler, the new rules are evaluated.

I suspect this could have been caused by #482


Error: unexpected server response (401)

Hello,
I'm experiencing a very strange issue with the binary-live-api library.
I attempted a test run using the following example code:
var ws = require('ws');
var LiveApi = require('binary-live-api').LiveApi;

var api = new LiveApi({ websocket: ws });
api.authorize('my token is here');
api.getPortfolio();
api.events.on('portfolio', function(data) {
    onsole.log(data);
});

Which resulted in this error:
Error: unexpected server response (401)
    at ClientRequest.options._req.on (/Users/FrancisJoseph/node_modules/ws/lib/WebSocket.js:650:26)
    at emitOne (events.js:96:13)
    at ClientRequest.emit (events.js:188:7)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:472:21)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:99:23)
    at TLSSocket.socketOnData (_http_client.js:361:20)
    at emitOne (events.js:96:13)
    at TLSSocket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at TLSSocket.Readable.push (_stream_readable.js:134:10)

Any ideas?

Same issue , any help ?
@frxncisjoseph resolved ?

@yogeshwar607 Not resolved and no response from the repository creators nor anyone else...

Same issue here

I looked into this and found the recent requirement for an app_id was causing issues. You'll need a valid app_id from: https://developers.binary.com/applications/
It can be fixed by changing:
var api = new LiveApi({websocket: ws});
to:
var api = new LiveApi({websocket: ws, appId: 0000});
Replace '0000' with your app_id.

Thanks Luke, I'd just realised that they made this app_id request mandatory a few weeks ago. Nice one :)


Replace the np.bool with python built-in bool

Type of Change
bug fix
Description
module 'numpy' has no attribute 'bool'
Expected Behavior & Potential Risk
model test pass
How has this PR been tested?
extension test
Dependency Change?
no

extension test pass: https://inteltf-jenk.sh.intel.com/view/lpot-validation/job/intel-lpot-validation-top-mr-extension/4039

extension test pass https://inteltf-jenk.sh.intel.com/view/lpot-validation/job/intel-lpot-validation-top-mr-extension/4062/

/Azurepipeline run Model-Test

test pass https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/4066/


Move Dataview Query Results -> Command not available in the command palette

Symptom
Move Dataview Query Results -> Command not available in the command palette.
Symptom occurs both in Read / Live Preview modes.
Steps to reproduce

Navigate to a note that displays Dataview Results
Display the command palette
Search for FileCooker command

Unexpected result

"Move Dataview Query Results" command is NOT available for selection.

Expected result

"Move Dataview Query Results" command is available for selection

Additional Comments

Obsidian v1.1.9
Plugins installed: 2 -> File Cooker 1.6.5, Dataview 0.5.47
OS: Windows 11

@ivaneye , this is really intriguing.
All dataview options are available in my main vault, which has dozens of other plugins installed.
Both my test vault and my main vault have the same Obsidian version, Dataview version and File Cooker version.
The only difference are
Main Vault:

Dozens of other plugins installed
Thousands of notes
File Cooker dataview commands are only available in "Live Preview" mode. Not available in "Reading" mode.

Test Vault

Created from scratch to test FileCooker
Has only 4 notes
Only Dataview and FileCooker installed initially.
Then I tested with "Metaedit" plugin installed to see if it would make any difference, which it didn't.

Here's what I see in my main vault. However, if I try to move based on dataview results, i get a
Query String Error![],  which I'll report in a separate issue:

e～
The command which use dataview needs you to select the dataview query ,just like this

Thanks @ivaneye
It worked as expected in my Main vault, which contains dozens of plugins.
However, It didn't work on a new vault created only to test File Cooker. The "Move Dataview Query Results" are not available on that vault even when I select the query text as you described.
Below are the debug info from both vaults:
Main Vault (Where the command is displayed and works as expected)
`SYSTEM INFO:
Obsidian version: v1.1.9
Installer version: v1.1.9
Operating system: Windows 10 Home 10.0.22621
Login status: logged in
Catalyst license: insider
Insider build toggle: on
Live preview: on
Legacy editor: off
Base theme: dark
Community theme: LYT Mode v1.5.0
Snippets enabled: 11
Restricted mode: off
Plugins installed: 67
Plugins enabled: 58
1: Dataview v0.5.47
2: Plugin Update Tracker v1.4.3
3: File Cooker v1.6.5
4: Obsidian42 - BRAT v0.6.36
5: Snippet Downloader v2.0.1
6: Markdown Attributes v1.2.2
7: Advanced URI v1.31.2
8: Auto Link Title v1.2.5
9: Banners v1.3.3
10: Better Word Count v0.9.1
11: Calendar v1.5.10
12: Contextual Typography v2.2.4
13: Convert url to preview (iframe) v0.5.0
14: DB Folder v3.1.3
15: Diagrams v1.5.0
16: Editor Syntax Highlight v0.1.3
17: Excalidraw v1.8.7
18: ExcaliBrain v0.1.11
19: Excel to Markdown Table v0.4.0
20: Full Calendar v0.9.5
21: Hider v1.2.4
22: Hover Editor v0.11.8
23: Jira Issue v1.48.0
24: Kanban v1.5.1
25: MAKE.md v0.5.6
26: Metadata Menu v0.4.3
27: MetaEdit v1.7.2
28: Metatable v0.14.3
29: Mind Map v1.1.0
30: Minimal Theme Settings v6.1.1
31: Natural Language Dates v0.6.1
32: Note Refactor v1.7.1
33: obsidian markmind v1.6.4
34: Omnisearch v1.9.1
35: Outliner v4.2.1
36: Periodic Notes v1.0.0-beta.3
37: Projects v1.8.0
38: Quick Explorer v0.2.7
39: QuickAdd v0.8.0
40: Quiet Outline v0.3.14
41: Reminder v1.1.15
42: Settings Search v1.2.0
43: Sortable v0.2.6
44: Style Settings v0.4.12
45: Tag Navigator v0.5.0
46: Tag Wrangler v0.5.5
47: Taskbone v1.2.3
48: Taskbone OCR v1.0.0
49: Tasks v1.21.0
50: Templater v1.16.0
51: Text Generator v0.1.22
52: Todoist Plugin v1.10.1
53: Todoist Link v2.2.0
54: Tracker v1.10.9
55: Zoom v1.1.1
56: Image Toolkit v1.3.1
57: Ozan's Image in Editor Plugin v2.1.6
58: Advanced Tables v0.18.1
RECOMMENDATIONS:
Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.
`
Test Valut - Where the command doesn't work (not available for selection in the palette)
`SYSTEM INFO:
Obsidian version: v1.1.9
Installer version: v1.1.9
Operating system: Windows 10 Home 10.0.22621
Login status: logged in
Catalyst license: insider
Insider build toggle: on
Live preview: on
Legacy editor: off
Base theme: light
Community theme: none
Snippets enabled: 0
Restricted mode: off
Plugins installed: 3
Plugins enabled: 3
1: File Cooker v1.6.5
2: Dataview v0.5.47
3: MetaEdit v1.7.2
RECOMMENDATIONS:
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.
`

try to restart obsidian?
The "Move Dataview Query Results" should display when Dataview and MetaEdit installed!
if it still doesn't work, send your test vault  to me, I will try on my computer.

Thanks @ivaneye
Didn't work on the freshly installed vault.
Things I tried:

Restarted obsidian (multiple times)
Reinstalled Dataview, MetaEdit
Enabled and disabled Dataview Options (Enable javascript, inline Js etc..)

I'm attaching the vault here for you.
Cooker.zip

Thanks @ivaneye
Didn't work on the freshly installed vault.
Things I tried:

Restarted obsidian (multiple times)
Reinstalled Dataview, MetaEdit
Enabled and disabled Dataview Options (Enable javascript, inline Js etc..)

I'm attaching the vault here for you. Cooker.zip

try disabled and enabled file-cooker。

Thanks @ivaneye
Disabling and re-enabling the File-Cooker plugin resolved the issue.
Dataview options immediately appeared in the command palette.


Refactor configuration section

Its necessary refactor code design like master-api organization.
Current code

Target code

Resolved!


Public Intersect Town Hall Planning Workgroup session 1 - Thursday 3rd October 2024

Join us in the next Intersect Town Hall Working Group Session
When: 03.10.2024 - 16:00 UTC
Duration: ~90 min
Zoom: https://bit.ly/3W4tb8q
Proposal: https://bit.ly/3Rct33S
Miro Board: https://bit.ly/4c4Htv0
Context: Workgroup helps to Co-Create and Operate Intersect Town Halls
Parts of Sessions are recorded and uploaded to the Swarm YouTube channel.
Miro Board is used for activities like meeting notes and information collection.
Suggested Workshop Agendas
5min - Welcome and Agenda
5min - (Optional) Introductions, Onboarding and Open Questions
10min - Retrospective on August Town Hall
30min - Next Town Hall agenda Structure
10min - How, where and who do we communicate Town Hall activities?
10min - Intersect Town Hall Milestone Reporting
10min - Open Topics
10min - Next Elegant Steps
Meeting Summary
People Present
callyfromauron [facilitator], tevo saks [facilitator], effiom [participant], Gintama [participant], marlon daley [participant], mtronchin [participant]
August Town Hall Retrospective
Discussion Points

Feel like we are gradually shifting towards more Community content
Longer breakout rooms for networking worked well
Majority liked the small breakout rooms
Felt welcoming space, made more connections compared to Twitter spaces
Presenting slides for others may cause issues if they use motion and use ref links in the slide instead of only having them in slide descriptions
unclear process and transition of communication methods between Main Town Hall, Human Interfacing, Zoom Poll Topic and After Town Hall
Forgot the Zoom topics after we left for breakout rooms and when we returned
Should we roundtable or more time spent describing the After Town Hall topics?
Should we have a clear main topic for the monthly Town Hall?
Duplication of information between Intersect Committees
Some Intersect Updates repeat themselves and take a long time to share. There is too much detailed reading from slides, for example, AMM Sessions and Election Info

Decision Items

Remove motion from the request slide template
Facilitator continues to read out the description max 45 seconds per ATH

Action Items

Create a new slide for After Town Agenda and the process of moving between different styles of break-out rooms
Create an additional slide for listing all the questions we chose from in the poll

Town Hall Agenda Structure
Discussion Points

Updated Town Hall Structure Template: https://docs.google.com/document/d/1vyy2Ds3Y88YbNBVKMxCHOIk-0CNbCkhXbJkQ4tZi1MY
Updated October Town Hall Agenda Doc: https://docs.google.com/document/d/1HQPs68BjK2B0LJYe69IIa396LHaIXUKWmaekDrlOHoY
Should we start handing over Town Hall Faciliation?
Updated Slides with visual examples of changes https://docs.google.com/presentation/d/1At0-Ueu5mGaKzqFoFChRt77xJO3r9BT4pjBh_kqkxkA

Decision Items

We keep the Core Focus slide provided by Jack Briggs in the process flow
It's not our primary focus to rotate facilitation as it should be directed towards getting the content.
Include in the 2025 Intersect Town Hall Proposal how to decentralise administrative faciliation

Action Items

Ideate on new Zoom Poll Questions

How, where and who do we communicate Town Hall activities?
Discussion Points

When to create survey analysis?
Present Communication Overview
Should we have Intersect Town Hall on 24th October?
Created Visual Schedule on Miro Board of estimated publishing dates: https://miro.com/app/board/uXjVKG5h5BA=/?moveToWidget=3458764590758800011&cot=14

Decision Items

We start working on Survey Analysis on 28th Oct

Action Items

Include Intersect Town Halls into https://cardanocalendar.io/calendar calendar
Create and Request include basic links to Intersect Discord FAQ channel
Coordinate to get the ITH Survey out in Intersect Newsletter on 11th Oct and 18th
Coordinate to get a request for ITH slides out in Intersect Newsletter on 11th Oct
Coordinate to get an invite to ITH out in Intersect Newsletter on 18th Oct
Publish Invite to Town Halls earliest at 21st Oct
Send email reminders for TH slides on 17th Oct

Intersect Town Hall Milestone Reporting
Discussion Points

How do I submit Milestones?
Where to find relevant documentation templates?
Review Intersect Town Hall milestone 1 https://docs.google.com/document/d/1Fj9MxiVeo-HAPb3DjABlwKuqyywS6YST

Decision Items

Milestone 1 is ready for submission

Action Items

Add link to ITH Gitbook about the Intersect Knowledge Base ITH Grant milestone page

Intersect Town Hall 2025 Proposal
Discussion Points

Should we create a budget draft outline to provide additional insight into Marketing Budgets?
Should we have a budget management section in the Town Hall Playbook?

Decision Items

Let's create an async analysis and new budget estimates for the 2025 ITH

Action Items

Include in the playbook how Intersect Town Hall team is doing budgeting

Updated the slides based on WG action items: https://docs.google.com/presentation/d/1At0-Ueu5mGaKzqFoFChRt77xJO3r9BT4pjBh_kqkxkA/edit#slide=id.g2d3f1f3c68b_2_0


Cache up-to-date region files from EvadesClassic to evades-region-editor inside regions directory

This is a request for Junior Developers and Developers as I was going to make this issue because it would be nice to have up-to-date region files if the cache happens accordingly.
The list of Evades.io Developer Team as follows:

Stovoy
MiceLee (former)
TheArctosJackal (junior dev)
TTTruck (junior dev)
Kaluub (junior dev)
amasterclasher (junior dev)
PotatoNuke (junior dev)

the select panelis making? i found button is saids select a  region but this disabled

um?

@Ino5559, the japanese text is reverted: bbbbd463cd4c51b26ee4b94b93aca744fab0fbd4

Looks like amasterclasher has made this google drive folder that keeps them up to date. I'll close this issue with the link :)
https://drive.google.com/drive/folders/1qzKh8cWHTqZC12WoJN6NpkdYw2tBRAia?usp=sharing


Modify diff.py to use TemporaryFile to capture output from make

On Linux python's subprocess.run() seems to have performance issues when capturing stderr using subprocess.PIPE. I've updated the script to use TemporaryFiles instead and this achieves performance very similar to that under Windows.
I wanted to use StringIO first to avoid I/O overhead but subprocess.run() requires that the File Object given to it implements fileno(). Due to this there may be a small performance hit under Windows, though likely not very noticeable.

I've pushed another commit that adds a new flag -t that makes this behavior optional instead of replacing the existing behavior entirely. This eliminates any concern of reducing performance on other systems.


Is there any debuginfo package for envoy?

Title: missing debuginfo of envoy
Description:
I'm trying to track envoy issues, but the debug information is missing.

cc @phlax
What version of Envoy are you looking for debuginfo for?

the dbg Docker images are probably the best place to start https://hub.docker.com/r/envoyproxy/envoy/tags?page=1&name=debug
we dont publish the dbg images on our release page but we probably should

OK, thank you!
I have a few more questions:

Is there an official source RPM package available?
Where can I obtain the compilation parameters for your production build of Envoy?
How did you build the RPM package? Are there SPEC files or the like? could you give it to me?

thank you


blacklist and wishlist in Chrome and URLlib do not follow interface definition

We had an issue implementing the settings because Chrome and Urllib group moved the params blacklist and whishlist into settings as opposed to global config level - this change has not been discussed and not documented either !
Because we don't expect Chrome and UrlLib to change their code now we will implement it the same way
@FrankeA90  - if this is an agreed change plese someone from the admin team update the interface Definition on the wiki page

I don't know why you marked this as a "bug" here. The wiki clearly states the config settings as how they are interpreted by the library: https://github.com/bensteUEM/mystique/wiki/Interface
blacklist and wishlist have been part of the config settings since the beginning. Also we don't distinguish between "global" or other setting levels. We just expect them to be passed to the functions.

I think I see the problem (took a while for me to understand).
Yes, I guess that this was never defined in this detail... But there is an inconsistency between documentation and the current master lib...
In the https://github.com/bensteUEM/mystique/wiki/Interface section, I see the following location for the blacklist and wishlist:

In the latest Library master, these are found in the "settings" of the initialize config method:

This is an indiscrepancy. But I suggest to change the documentation rather than changing the code at this time.
@bensteUEM: would that solve your issue?
@Monti93: If @bensteUEM agrees, can you update https://github.com/bensteUEM/mystique/wiki/Interface so that it matches the current master library?

Okay, that makes sense. Sure, I can update this.


Latest Version of Websites files

Is this is the latest version of files in repo that your website currently have ?

no! new version files are not yet uploaded.


allow long protocol detection timeout for test

Extend protocolDetectionTimeout to 15s for test.

do all tests need the long timeout ?

For knative and simple, I verified that we need the long timeout in order to pass the test. For other two, I haven't verified. I think it's safe to extend them all.

/cherrypick/release-1.3


'Join' filename to lowercase

I think that we should be unified filename to lowercase.

Yep agree


Branch - subhourly_sims

[ ] Have owners review branch to identify functionality to include in HOPP
[ ] Back-merge dev/refactor into respective branches
[ ] Remove extraneous code
[ ] Merge branch and delete old branch

This has a PR associated with it: #38


Argument list too long: 'kast' when using large configurations

To reproduce:

Checkout the pyk-issue branch here in a fresh client: https://github.com/runtimeverification/wasm-semantics/tree/pyk-issue
In scripts/compress_bytes/compress_bytes.py change ROOT to point to your client.
Run:

make build-haskell
cd scripts
poetry install
poetry shell
python3 -m compress_bytes.compress_bytes

I believe this is addressed with: https://github.com/runtimeverification/pyk/pull/266.
Re-open if not!


Footnotes phase 2

Please add acceptance criteria.

[ ] Can add footnote in Sanity studio to ingress
[ ] In web clicking on the footnote in ingress anchors to end of content
[ ] At end of footnote at the bottom the return links takes you back up
[ ] Looks ok :)

look at possibillities to look through pasted text for asterix and create one footnote that all of the same type of asterix points to
reuse an exisiting footnote


Repo sync for protected CLA branch

The pull request is created from master637533277200476512 to master to fix git push error for protected CLA branch

@Taojunshen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Publish to maven central instead of JitPack

Hi,
as JitPack.io has been down for more than 24 hours, it would be a good idea to publish Fuel to maven central or similar stable repository 🙏

Yeah I will work on this!

We did https://github.com/kittinunf/fuel/releases/tag/3.0.0-alpha01

Hm I still don't see it in Maven central: https://mvnrepository.com/artifact/com.github.kittinunf.fuel
Is that really deployed there?

it is in OSS Sonatypes.

it is in OSS Sonatypes.

It seems the comment disappeared? Also I still don't see it anywhere. Ideally it would be great to see it in mvnrepository like other artifacts.

Hmm, I think it should be here https://mvnrepository.com/artifact/com.github.kittinunf.fuel/fuel/3.0.0-alpha1
Together with -jvm and others artifacts for Kmm
Could you please try again and see whether that works?

I finally see it there! Thanks!


ClassCastException in com.orientechnologies.orient.core.sql.executor.OResultInternal#getProperty Method Due to Direct Casting of OVertexDelegate Instead of Using getRecord()

OrientDB Version: 3.2.28
Java Version: 17
OS: Windows
Expected behavior
The com.orientechnologies.orient.core.sql.executor.OResultInternal#getProperty method should safely and correctly retrieve properties from an OrientDB document or vertex without causing a ClassCastException. The previous implementation handled this correctly.
Actual behavior
After updating the method, a ClassCastException is thrown when attempting to retrieve properties from an OVertexDelegate instance. The exception message is: java.lang.ClassCastException: class com.orientechnologies.orient.core.record.impl.OVertexDelegate cannot be cast to class com.orientechnologies.orient.core.record.impl.ODocument (com.orientechnologies.orient.core.record.impl.OVertexDelegate and com.orientechnologies.orient.core.record.impl.ODocument are in unnamed module of loader 'app').
Steps to reproduce

Execute a query using select from (traverse from @rid) order by property.
The exception occurs specifically at the moment of executing the order by clause after traversing.

Key Change Highlighted:

Previous implementation: element.getRecord() was used to safely cast to ODocument.
Current implementation: Direct casting of element, an instance of OVertexDelegate, is attempted without accessing the ODocument via element.getRecord().

Previous getProperty method implementation(3.2.25 version):
public <T> T getProperty(String name) {
    loadElement();
    T result = null;
    if (content != null && content.containsKey(name)) {
      result = (T) wrap(content.get(name));
    } else if (element != null) {
      result = (T) wrap(((ODocument) element.getRecord()).rawField(name));
    }
    if (result instanceof OIdentifiable && ((OIdentifiable) result).getIdentity().isPersistent()) {
      result = (T) ((OIdentifiable) result).getIdentity();
    }
    return result;
}

Updated getProperty method implementation:
public <T> T getProperty(String name) {
    loadElement();
    T result = null;
    if (content != null && content.containsKey(name)) {
      result = (T) wrap(content.get(name));
    } else if (isElement()) {
      result = (T) wrap((T) ODocumentInternal.rawPropertyRead((OElement) element, name));
    }
    if (result instanceof OIdentifiable && ((OIdentifiable) result).getIdentity().isPersistent()) {
      result = (T) ((OIdentifiable) result).getIdentity();
    }
    return result;
}

Additional rawPropertyRead method:
public static <RET> RET rawPropertyRead(OElement element, String propertyName) {
    if (element == null) {
      return null;
    }
    return ((ODocument) element).rawField(propertyName);
}

Hi,
Thanks will check that change/revert back the fix, it was done to avoid to load records only when it was needed, the getRecord do a early loading that may slow down some use cases.
Do you have a stack trace of the case (only the OrientDB parts are needed) ? it will help me to identify the path that has the issue.
Regards

java.lang.ClassCastException: class com.orientechnologies.orient.core.record.impl.OVertexDelegate cannot be cast to class com.orientechnologies.orient.core.record.impl.ODocument (com.orientechnologies.orient.core.record.impl.OVertexDelegate and com.orientechnologies.orient.core.record.impl.ODocument are in unnamed module of loader 'app')
at com.orientechnologies.orient.core.record.impl.ODocumentInternal.rawPropertyRead(ODocumentInternal.java:128)
at com.orientechnologies.orient.core.sql.executor.OResultInternal.getProperty(OResultInternal.java:121)
at com.orientechnologies.orient.core.sql.executor.OTraverseResult.getProperty(OTraverseResult.java:20)
at com.orientechnologies.orient.core.sql.parser.OOrderByItem.compare(OOrderByItem.java:96)
at com.orientechnologies.orient.core.sql.parser.OOrderBy.compare(OOrderBy.java:66)
at com.orientechnologies.orient.core.sql.executor.OrderByStep.lambda$init$2(OrderByStep.java:91)
at java.base/java.util.TimSort.countRunAndMakeAscending(TimSort.java:355)
at java.base/java.util.TimSort.sort(TimSort.java:220)
at java.base/java.util.Arrays.sort(Arrays.java:1307)
at java.base/java.util.ArrayList.sort(ArrayList.java:1721)
at com.orientechnologies.orient.core.sql.executor.OrderByStep.init(OrderByStep.java:91)
at com.orientechnologies.orient.core.sql.executor.OrderByStep.internalStart(OrderByStep.java:43)
at com.orientechnologies.orient.core.sql.executor.AbstractExecutionStep.start(AbstractExecutionStep.java:79)
at com.orientechnologies.orient.core.sql.executor.OSelectExecutionPlan.start(OSelectExecutionPlan.java:41)
at com.orientechnologies.orient.core.sql.parser.OLocalResultSet.start(OLocalResultSet.java:36)
at com.orientechnologies.orient.core.sql.parser.OLocalResultSet.<init>(OLocalResultSet.java:27)
at com.orientechnologies.orient.core.sql.parser.OSelectStatement.execute(OSelectStatement.java:357)
at com.orientechnologies.orient.core.sql.parser.OStatement.execute(OStatement.java:76)
at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.query(ODatabaseDocumentEmbedded.java:592)

Thank you for your prompt response. I understand the intent behind the change to optimize performance by delaying record loading. However, after updating to the latest version, I noticed that many of my tests started failing due to this ClassCastException.
To isolate the issue, I created a fork of the project and modified the rawPropertyRead method as follows to revert to the earlier behavior:
public static <RET> RET rawPropertyRead(OElement element, String propertyName) {
    if (element == null) {
        return null;
    }
    return ((ODocument) element.getRecord()).rawField(propertyName);
}

After compiling this version and replacing the main dependency in my pom.xml with the modified version, my tests no longer fail. This adjustment seems to resolve the ClassCastException issue I encountered.
I appreciate your help in looking into this matter and am happy to provide any further information you may need.
Regards

Hi,
Yep I've done a fix that is pretty much the same fix, will be release 3.2.29
Regards

Great to hear that a fix has been implemented and will be included in release 3.2.29. I appreciate the quick response and action taken on this issue. Looking forward to the update!


How to use it with VLLM?

I used the following code, and the result prompted an error.
from vllm import LLM

llm = LLM(model=model)

File ~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:459, in cached_file(path_or_repo_id, filename, cache_dir, force_download, resume_download, proxies, token, revision, local_files_only, subfolder, repo_type, user_agent, _raise_exceptions_for_gated_repo, _raise_exceptions_for_missing_entries, _raise_exceptions_for_connection_errors, _commit_hash, **deprecated_kwargs)
    [457](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:457)     if filename in ["config.json", f"{subfolder}/config.json"]:
    [458](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:458)         return None
--> [459](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:459)     raise EnvironmentError(
    [460](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:460)         f"{path_or_repo_id} does not appear to have a file named {full_filename}. Checkout "
    [461](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:461)         f"'https://huggingface.co/{path_or_repo_id}/tree/{revision}' for available files."
    [462](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:462)     ) from e
    [463](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:463) except HTTPError as err:
    [464](https://file+.vscode-resource.vscode-cdn.net/d%3A/GitLab/car_data_clean/~/miniforge3/lib/python3.12/site-packages/transformers/utils/hub.py:464)     resolved_file = _get_cache_file_to_return(path_or_repo_id, full_filename, cache_dir, revision)

OSError: fancyfeast/llama-joycaption-alpha-two-hf-llava does not appear to have a file named preprocessor_config.json. Checkout 'https://huggingface.co/fancyfeast/llama-joycaption-alpha-two-hf-llava/tree/main' for available files.

It work with vllm yet, since it doesn't have a huggingface compatible preprocessor.  I'll try to get it fixed when I get a chance, as long as vllm can support JoyCaption's image model.

vLLM should work now:
vllm serve fancyfeast/llama-joycaption-alpha-two-hf-llava --max-model-len 4096 --enable-prefix-caching

Amazing job!


Style Survey - Come up with questions

What specific questions do we want to ask our user?
How many questions?
What else do we need? (images, etc.)

I would highly recommend visiting Stitchfix, creating an account and going through their survey to get a good feel for what it would be like for Crate. It sounds like we have landed on staying with clothing. Let's decide on some themes for Crates (date night, every day, athletic wear, business casual, professional dress, etc...) for categories and we can update crates and find images. Orlando and I have thought about asking Robbie a question during our check in today about how we can possible stream line this process.


Update dependencies and make OPA optional

pre-commit and safety are fixed in https://github.com/geobeyond/fastgeoapi/pull/10, going to merge and rebase that


Added missing licence for jquery-i18n

Added recurser/jquery-i18n licence as per #1279

Thanks Owen!


Supergiant volumes should have a no delete flag.

Supergiant volume creation should have a method of assigning a life cycle for a volume. Initially this should be to prevent destruction (or just not manage it.), but should also be extended to snapshot/backups, or any other volume maint tasks.

Closed because this feature no longer exists.


Add Right to Left support to Masonry

Hello,
i'm using this project for one of my projects which is in Farsi and RTL so i need masonry in right to left direction.
i've read a little bit of Masonry source code and figured out it's using position absolute + some top and left values to adjust elements, i was able to achieve this by switching left to right in
https://github.com/bvaughn/react-virtualized/blob/af90cfec8b2b131896fd979795c5d8c54480ff27/source/Masonry/Masonry.js#L396
screenshot:

i'm not sure that this is a silly solution or it's okay, then if it is or you have better solution, i can make a PR for this for an additional prop called isRtl ❤

i can make a PR for this for an additional prop called isRtl

Property naming wise, I would suggest direction (defaults to "vertical" but can also be "horizontal").
You'd be welcome to submit a PR, including docs and tests. Depending on how much complexity it adds, I'll consider adding it to the library. 😄

I agree with you about naming, just wanted to demonstrate the main idea.
if we only call it direction the use might misunderstand which direction do we mean (direction in the row or column).
what if we call it rowDirection instead which defaults to ltr and can be rtl too.
<Masonry rowDirection="rtl" />


Borders

Do we have handling for borders? I’d be interested to hear your thoughts! I’ll just jot a few notes I had on it:
SVG strokes are drawn both sides of the shape, whereas in RN they are drawn inside. We need to inset the x, y, width, and height by the borders on those sides.
We need to handle scaling border radii down when you have a border radius of say 1000 in a box that is 50x50. We also need to handle % border radii, which are handled differently.
If the border width and colour is consistent on all sides, we can just set the stroke on the main shape. For different border colours, we will have to use different paths. For different border widths, we may not be able to use the stroke, and may have to draw this as a filled path.

I've not put much thought into it yet, the Artsy design style doesn't really have many bordered objects, so I've not had to think of how to represent it.
There's also border radius, and border styles to worry about 🍡
https://facebook.github.io/react-native/docs/viewstyleproptypes.html

#37 handles some of this 👍


Enable Django admin

Adds the needed settings in order to enable the Django admin at /admin. I sometimes find this easier to use then shell_plus or looking at the database itself.
Settings needed to enable this were taken from
https://github.com/girder/django-composed-configuration

Note this will not expose the admin on the AWS deployment, as NGINX only proxies backend requests to /api, and not /admin.

Am I missing something in this to get actual template rendering so the admin interface isn't just raw ugly HTML without styling?

Added instructions for createsuperuser to the dev docs.


Added OCR

Description
Added the ocr application
Fixed #(#38 )
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have included a screenshot

Great. good job with the code. I will suggest a change.

[ ] Add images of the output in the readme as well as in PR. put the images inside the project folder

Done

@all-contributors please add Samikdevel for code, doc


The issues for the full objective configuration

Hello！When I run the following command：python tools/gemap/custom_nusc_map_converter.py --root-path ./data/hdmap/nuscenes --out-dir ./data/hdmap/nuscenes/full --extra-tag nuscenes --version v1.0 --canbus ./data/hdmap
I get the following error, what is the cause of this?
[                                                  ] 0/34149, elapsed: 0s, ETA:Traceback (most recent call last):   File "tools/gemap/custom_nusc_map_converter.py", line 928, in <module>     nuscenes_data_prep(   File "tools/gemap/custom_nusc_map_converter.py", line 869, in nuscenes_data_prep     create_nuscenes_infos(   File "tools/gemap/custom_nusc_map_converter.py", line 825, in create_nuscenes_infos     train_nusc_infos, val_nusc_infos = _fill_trainval_infos(   File "tools/gemap/custom_nusc_map_converter.py", line 340, in _fill_trainval_infos     info = obtain_vectormap(nusc_maps, map_explorer, info, point_cloud_range)   File "tools/gemap/custom_nusc_map_converter.py", line 371, in obtain_vectormap     map_anns = vector_map.gen_vectorized_samples(lidar2global_translation, lidar2global_rotation)   File "tools/gemap/custom_nusc_map_converter.py", line 442, in gen_vectorized_samples     centerline_list = self.centerline_geoms_to_instances(centerline_geom)   File "tools/gemap/custom_nusc_map_converter.py", line 672, in centerline_geoms_to_instances     centerline_geoms_list,pts_G = self.union_centerline(geoms_dict)   File "tools/gemap/custom_nusc_map_converter.py", line 743, in union_centerline     paths = nx.all_simple_paths(pts_G, root, leaves)   File "/home/hwt/anaconda3/envs/npn/lib/python3.8/site-packages/networkx/algorithms/simple_paths.py", line 202, in all_simple_paths     raise nx.NodeNotFound('target node %s not in graph' % target) networkx.exception.NodeNotFound: target node [(15.0, 2.355), (15.0, -2.636), (4.167, -30.0), (-15.0, 15.813), (0.284, -30.0)] not in graph

A similar issue is here and you can try to update your networkx version to 3.1.


Please add full support for WinUI3 Desktop applications

UWP appears to be on its way out and I have moved my application base to WinUI3 Desktop.  AppCenter is pretty helpful in improving application quality and I was surprised to find that there is no support for WinUI3 after attempting to upload symbols.

Hello @EdAlexander, looks like this is a duplicate of "Support for WinUI 3 (Project Reunion) apps #2299"
WinUI supported since AppCenter SDK .NET 4.4.0 and later.

2299 was closed as completed but the issue remains.  no crash support and winui3 desktop apps are only supported as UWP....  I see activity but no crash data

@EdAlexander, could you please share more information of your issue? What platform do you use, share logs and also it would be very helpful if you can provide sample project where issue is reproducible.

Cannot add a WinUI3 Desktop application in app center.

Symbol files cannot be uploaded from build of WinUI3 Desktop application.
I have two apps that started in App Center as UWP that have been rewritten and released as WinUI3 Desktop in the store (same app identity).
Would like to be able to upload symbols and get crash data again.

Any updates @Oleg-Karoza?  Looks like this item is about to be automatically closed...  Still looks like an issue to me.


Resolving FQDN for hostname params

Hi @avalanche123 et al.
I've done a small modification to the code in order to let HOST parameter work also with FQDN instead of only IPS.
I'm trying to use cassandra-web in kubernetes environments where IPs cannot be know in advance, so this small fix is really useful to me.
Sadly I'm not a ruby developer so I tried to arrange it the best I could, but I'd like to share it to have it included in mainstream (maybe with some more elegant adjustment).
PR also include integration with Travis CI to create docker image and upload it to Docker hub (just change environment variables and secure to work on another account)

Ruby driver already performs dns lookup on hostnames, I’m not sure why this is needed

This is what happens if you pass a fqdn instaed of ip to 0.4.0 version:
✗ docker run -e"CASSANDRA_HOST_IP=www.google.com" delermando/docker-cassandra-web:v0.4.0

cassandra-web --hosts www.google.com --port 9042 --username cassandra --password cassandra
IPAddr::InvalidAddressError: invalid address

Usage: cassandra-web [options]
    -B, --bind BIND                  ip:port or path for cassandra web to bind on (default: <IP_ADDRESS>:3000)
    -H, --hosts HOSTS                coma-separated list of cassandra hosts (default: <IP_ADDRESS>)
    -P, --port PORT                  integer port that cassandra is running on (default: 9042)
    -L, --log-level LEVEL            log level (default: info)
    -u, --username USER              username to use when connecting to cassandra
    -p, --password PASS              password to use when connecting to cassandra
    -C, --compression NAME           compression algorithm to use (lz4 or snappy)
        --server-cert PATH           server ceritificate pathname
        --client-cert PATH           client ceritificate pathname
        --private-key PATH           path to private key
        --passphrase SECRET          passphrase for the private key
    -h, --help                       Show help

Hmm, I’ll take a look, we might have to update the version of the ruby driver that cassandra-web is using

Can you rerun cassandra-web with —debug. And paste the stack-trace

no change in output ( I suppose you intended -L debug )
docker run -e"CASSANDRA_HOST_IP=www.google.com -L debug " delermando/docker-cassandra-web:v0.4.0

cassandra-web --hosts www.google.com -L debug --port 9042 --username cassandra --password cassandra
IPAddr::InvalidAddressError: invalid address

Usage: cassandra-web [options]
    -B, --bind BIND                  ip:port or path for cassandra web to bind on (default: <IP_ADDRESS>:3000)
    -H, --hosts HOSTS                coma-separated list of cassandra hosts (default: <IP_ADDRESS>)
    -P, --port PORT                  integer port that cassandra is running on (default: 9042)
    -L, --log-level LEVEL            log level (default: info)
    -u, --username USER              username to use when connecting to cassandra
    -p, --password PASS              password to use when connecting to cassandra
    -C, --compression NAME           compression algorithm to use (lz4 or snappy)
        --server-cert PATH           server ceritificate pathname
        --client-cert PATH           client ceritificate pathname
        --private-key PATH           path to private key
        --passphrase SECRET          passphrase for the private key
    -h, --help                       Show help

I just realized that the Gemfile.lock was pretty stale and I updated it, can you try again without your patch to ensure you get dns resolving now, thanks!

For me, version 0.5.0 still doesn't accept hostnames.

It's the line Cassandra::LoadBalancing::Policies::WhiteList.new(hosts, ::Cassandra::LoadBalancing::Policies::RoundRobin.new) thats the culprit. It does not accept hostnames in hosts. So I think the patch is still needed.

Hi @avalanche123 ,
Can you take a look at this again?


./tools folder fails to build with unit tests that are never triggered

What happened:
As @akalenyu mentions in this comment, ./tools fails to build:
kubevirt/tools/util/marshaller_test.go:32:195: not enough arguments in call to components.NewHandlerDaemonSet

In addition, as he's pointing out, these tests are not attempted to ever run:
https://github.com/kubevirt/kubevirt/blob/941b04cb9f90745ae2038b13c8bb82eea55a16dd/hack/bazel-test.sh#L7
To add an insult to injury, these are lacking the *_suite_test.go that's required to make these tests runnable.
What you expected to happen:
The tests should either be able to build and run via make test or removed.
How to reproduce it (as minimally and precisely as possible):
Trying to build ./tools

/label good-first-issue

Is this a good first issue to work on?

Is this a good first issue to work on?

@aryandutt Yes, it is :)
Would you like to go ahead and fix it? If you need any help, I'll gladly try to provide it.

@dosu You're pretty close :)
Note that it's possible to use make generate to auto-create the *_suite_test.go file.

Hi @iholder101, I would definitely like to work on it!
Could you please give me a brief overview?
Thanks

Hi @iholder101, is this the file I need to look at to setup the project locally and recreate the issue?

Hi @iholder101, is this the file I need to look at to setup the project locally and recreate the issue?

Yes, that's a great starting point indeed

I was trying to setup Kubevirt by looking at the getting started section but I am encountering the following error.
ERROR: An error occurred during the fetch of repository 'go_image_base_aarch64':    Traceback (most recent call last):         File "/root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/external/io_bazel_rules_docker/container/pull.bzl", line 189, column 13, in _impl                 fail("Pull command failed: %s (%s)" % (result.stderr, " ".join([str(a) for a in args])))

I was trying to setup Kubevirt by looking at the getting started section but I am encountering the following error.
ERROR: An error occurred during the fetch of repository 'go_image_base_aarch64': Traceback (most recent call last): File "/root/.cache/bazel/_bazel_root/6f347497f91c9a385dcd9294645b76e0/external/io_bazel_rules_docker/container/pull.bzl", line 189, column 13, in _impl fail("Pull command failed: %s (%s)" % (result.stderr, " ".join([str(a) for a in args])))

If your still stuck, ping me on slack, I will try help you (I setup kubevirt locally too recently)
If you haven't joined, it's the Kubernetes slack channel.

Regarding the fact that it doesn't build, I see that there's a function usage that provide less parameters than whatever's needed there. I assume that someone refactored it and forgot to change the function call? You can try to add the missing parameter there.

Hey @iholder101, So apparently adding missing parameters in components.NewHandlerDaemonSet will fix this issue?right?


Add hain-plugin-material-colors to plugin-docs

Please pr to develop branch! :wink:

Got it, sorry. Maybe clarifying in share-your-plugins would be beneficial?

share-your-plugins doesn't have a responsibility for this kind of PR, actually it's related with CONTRIBUTING.md.

Makes sense, I understand.


[e2e]: delegator-generic create main checkout slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.238
Run: https://github.com/slsa-framework/example-package/actions/runs/8281198110
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.238
Date: Thu Mar 14 13:05:18 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.239
Run: https://github.com/slsa-framework/example-package/actions/runs/8296628483
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.239
Date: Fri Mar 15 13:05:06 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.240
Run: https://github.com/slsa-framework/example-package/actions/runs/8307967753
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.240
Date: Sat Mar 16 13:05:06 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.241
Run: https://github.com/slsa-framework/example-package/actions/runs/8315644658
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.241
Date: Sun Mar 17 13:04:58 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.243
Run: https://github.com/slsa-framework/example-package/actions/runs/8343510937
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.243
Date: Tue Mar 19 13:06:56 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v48.0.245
Run: https://github.com/slsa-framework/example-package/actions/runs/8365037773
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.checkout.slsa3.yml
Trigger: create
Branch: v48.0.245
Date: Wed Mar 20 20:14:54 UTC 2024
Tests are passing now. Closing this issue.


FIFO

💡 mod_fifo - we need a drop-in for mod_fifo - which supports ring all and enterprise queuing.
Requirements:

It needs to be as modular as possible, a clean fifo implementation
Use call events to to interface to media. For example, when a call enters a queue, we need to play music on hold, and interleave with queue position. Media should not be a queue function - so this is separated by events. The queue function to should handle calling phones, connecting calls and emitting events.3. When this is done, which could be a part of callmanager (see below) or it's own npm module (I think this is more correct), then babble-sip will require extending to ensure that it is wired up to media functions.Look at https://github.com/babblevoice/babble-drachtio-callmanager as the starting point - as this is how a call comes in to the system. https://github.com/babblevoice/babble-sip uses call manager to interface with our DB and other systems.

[ ] - Add event for update (so calling script can play position number for example)
[ ] - Use the call object for events (otherwise is duplication?)


Multi destination

Implemented a new feature to handle multi-destination in TestScript.
Detail:

In driver.rb changed endpoint to Array to store multiple server addresses. I put seven predefined, different server addresses from HAPI, Firely, and Logica. We need to start discussion of how to deal with multiple server addresses. Questions may include 1) when we have multiple destinations, should they be always different servers or server instances? 2) What if 100 multiple destinations in TestScript? Should the engine get prepared for 100 server addresses? 3) Do we keep server address in hard coded way as it is, or would we create a configuration file for the engine (e.g. configure.xml)?

Client object corresponds to destination. If there are two destinations in TestScript, the engine instantiates two Clients. If a TestScript doesn't include destination, the engine considers to use "default" client, which picks the first one from the server address list in endpoints.

Client object is moved from TestScriptEngine.rb to TestScriptRunnable.rb, since now one client won't be used by multiple runnables. Each runnable should have own instance of Client correspond to destination.

The engine reads multiple origins but doesn't do anything with it for now.

Additional:

Example TestScript for multi-agent is added.

Hard-coding the server list doesn't make sense to me. Since it is a command-line tool, I would expect the user to list the servers they want to use as arguments. Acceptable alternatives include writing a shell script for frequently used scenarios or creating a configuration file.
The user wants to run a TestScript with 2 destinations? Then they are required to provide two URLs.
The engine shouldn't run if an endpoint is not specified.
It doesn't matter if the array is size 1 or size 1 billion. Other than memory. And you'd need a scenario that has that many endpoints. But it shouldn't matter.

If no server address entered in argument, default server is assigned.
If one server address entered in argument, it replaces default server.
If there are destination in TestScript, servers are assigned to destinations in order in argument
If # of server address < destination, the engine errors.

Test with these:
ruby driver.rb read_test_script.json
ruby driver.rb http://hapi.fhir.org/baseR4 read_test_script.json
ruby driver.rb http://hapi.fhir.org/baseR4 multisystem_test_script.json
ruby driver.rb http://hapi.fhir.org/baseR4 https://server.fire.ly/ multisystem_test_script.json
ruby driver.rb http://hapi.fhir.org/baseR4 https://server.fire.ly/ read_test_script.json

Close this issue by end of the project. Issue https://github.com/fhir-crucible/testscript-engine/issues/53 will keep open without milestone.


Avoid unnecessary reset in sync_vector_env.py

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Screenshots
Checklist:

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Please could you explain the PR, I don't understand the change

I think self._autoreset_envs records whether the sub-environments should be reset in the next step. However, after the reset method of SyncVectorEnv instances is manually called, self._autoreset_envs is not set to all False, leading to some sub-environments will be reset twice consecutively.
I also noticed that in _aysnc_worker of async-vector_env.py, the autoreset flag is set to False if the reset method is manually called.
So I think self._autoreset_envs should be set to all False at the end of the reset method of SyncVectorEnv.
If I am wrong, ignore this PR.

In addition, there may be some typos in _async_worker of async_vector_env.py and I fix them.

@xihuai18 Your correct about the autoreset for SyncVectorEnv however the other two changes I don't believe are correct.
Could you revert those changes then it should be good to merge

I'd like to explain my changes:

exceptions for _call
https://github.com/Farama-Foundation/Gymnasium/blob/1878f6dd7d69439dc770289a9000eed6a52b4a43/gymnasium/vector/async_vector_env.py#L700-L726

Line 725 shows that all accepted commands are reset, step, close, _call, _setattr, _check_spaces, so corresponding exceptions are changed in Line 702. For example, one can not _call _setattr, but should use the _setattr method directly.

type hints for shared_memory
https://github.com/Farama-Foundation/Gymnasium/blob/1878f6dd7d69439dc770289a9000eed6a52b4a43/gymnasium/vector/async_vector_env.py#L669-L672

shared_memory are used for writing in the obtained obserations via write_to_shared_memory function (if it is not None). Thus it should be a multiprocessing.Array or other nested types, as defined in https://github.com/Farama-Foundation/Gymnasium/blob/main/gymnasium/vector/utils/shared_memory.py.

I have passed the pre-commit now.

Ahh you are right on shared_memory, I had missed the line below and thought it must have been a boolean
For the error, it is around the function / attr being called not the command. The list you have updated is the list of commands but not if the function should be called.
Saying that, I think the list should be "reset", "step", "close" and with "set_wrapper_attr" users should just use "set_attr" the error doesn't support a different function name.

Ahh you are right on shared_memory, I had missed the line below and thought it must have been a boolean
For the error, it is for the function / attr name not the command, i.e., envs.call("reset") is disabled. The list you have updated is the list of commands but not if the function should be called. Saying that, I think the list should be "reset", "step", "close" and with "set_wrapper_attr" users should just use "set_attr" the error doesn't support a different function name.

I think the current version is right according to your review?


Display current and max progress on left side of determinate progressbar

As is in subject. I think it could be configurable.

https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>


specify required option to use Flow with new JSX transform

Unfortunately the option is not documented on the Flow docs, and it's not very obvious how to get Flow to work with the new JSX transform.

Oh nice, thanks. Want to send a PR to Flow docs too?

https://github.com/facebook/flow/pull/8514/files


[Bug] Toast Messages and Pop up dialogs

Check the following items

[X] I have looked up relevant Issue

Description of the issue
This is a question, wanted to findout if the way Toast and popup dialog messages will keep on changing for each release, asking so that I create a better handle for these changes,
Package Version
6.0.0-Beta8
Environment
Windows 11 22631
Expected Behavior
For toast messages to not keep changing
Reproduction
This is question
Additional Information
No response

This represents the last major change as far as I'm concerned, the original rewrite was generally only meant to be temporary to get styling in place whilst a more permanent solution was found and this second rewrite was always planned though what form it was going to take wasn't clear until recently and it wasn't necessarily communicated.
Of course 6.0.0 is still a beta release so there might still be some tweaks and changes, but the fundamental design of the API is now set and it's been written and designed so that new features/changes should cause few or no breaks, and any breaking changes should be minimal and planned for major releases anyway going forward.
I'd want @kikipoulet to weigh in and confirm that they are happy with where it's at right now and then you can be confident that this API is unlikely to change in any major way from now on.

This api change really made it very difficult for me to adapt it into my project... Every dialog is based on the old one...

That's one of the dangers of working with pre-release code.

@yuzh0816 same here, I've been struggling to get it working with my project, had to revert back to beta7. Will stay on beta7 until someone makes a tutorial on it :D

Here are the docs for the new toasts API.
The Dialogs API is very similar and you just use SukiDialogHost instead of SukiToastHost, I haven't gotten around to writing the docs for it yet but the XMLDocs are in place so it should be fairly easy to get on with.

I understand your frustration guys but @sirdoombox is completely right, and this new implementation will let us add new features in the future without breaking anything. Furthermore, the fluent builder is quite nice.
To replace the old way as easy as possible :

Add a SukiDialogHost in your SukiWindow

Reference it statically in your window

Show your dialog "anywhere" in your code from the static reference

MainWindow.DialManager.CreateDialog()
        .WithContent(new YourControl())
        .Dismiss().ByClickingBackground()
        .TryShow();

So actually it works almost like the toasting documented here : https://kikipoulet.github.io/SukiUI/documentation/hosts/toast.html

Thanks, worked :)

Closing this as answered for now. If there are any follow ups I'm happy to re-open.


Allow sources to be parallelized across processes and controlled by adapters

Wanted to get feedback on the general idea. I can clean up and add unit tests if this is something vstest is open to.
Description
Add Bucketize command to allow adapters to bucketize tests within a source and parallelize across buckets.
Related issue
Add support for #2138

Need to understand this better

Where is the execution time coming from ?
Why is bucket decided by adapter ? Does not in-assembly parallelization suffice ?
Can buckets be created  by platform ?

Note:
Changing the OM (TestCase) in the Testplatform is not encouraged, There are compat issues that need to be handled. We'd rather go with a well known property that goes in the store property bag in the test case object.

@singhsarab

The execution time comes form multiple factors. The test suites in question are long running functional tests which make many HTTP calls to a service. We have around 6K tests in just a handful of dlls. With our custom test harness we run 70 processes at a time and stripe the tests across these processes regardless of which dll the test comes from.
My understanding is in-assembly parallelization is handled by the adapter but will always be parallelized within the same process. We need in-assembly parallelization across process (for multiple reasons).
The bucket is decided in the adapter because the adapter is already reasoning about what tests need to be run and is an existing extension point. I guess a whole new extension point could be added.
No the platform can't do the bucketing since we want to provide our own custom code to decide how to bucketize tests.

Regarding the change in the TestPlatform, I'm not actually changing the TestClass class, just adding a new well known property to TestCaseProperties which will be stored in the property bag. Is there a concern with adding new properties to TestCaseProperties?

/cc: @mayankbansal018 Let's review this tomorrow.
@NGloreous Apologies for the delay.

@singhsarab @vagisha-nidhi is this change something you are open to taking? If so I spend a bit more time making the change nicer.

bump, need this :)

We did some improvement for the parallelization and for the moment we don't have a lot of request to provide new feature on this side.
I'm going to close for now to reopen if the interest will raise.


Feature Request: Specify Required Java Version

CurseForge allows you to specify the required Java version for files, and it would be awesome if CurseGradle could pull this automatically from the project's targetCompatibility.

I'm not sure that the CurseForge API supports this. I'll poke curse people to confirm.

@shadowfacts Apparently you can do this manually through addGameVersion 'Java 8'. I'll see about detecting and setting this automatically.