Adding nested virtualization mode for Linux/KVM

This also flips FPLASH default to 'y' (but specifying FPLASH='' on the command line will give you older behaviour).
@eriknordmark and @giggsoff @temerkhanov  -- can you please test this on your Linux setup before we merge this?

It works for me.
Small remark: with ACCEL=y we have 2 cpu definitions: -cpu SandyBridge ... -cpu host,invtsc=on,kvmclock=off

Boot variables/boot order was not exactly right in my case, had to run the
bootloader manually.
Otherwise worked fine.
Regards,
Sergey
On Thu, Sep 3, 2020 at 5:23 PM Petr<EMAIL_ADDRESS>wrote:

It works for me.
Small remark: with ACCEL=y we have 2 cpu definitions: -cpu SandyBridge
... -cpu host,invtsc=on,kvmclock=off
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lf-edge/eve/pull/1411#issuecomment-686527035, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6GJXD4DFEHTIGUY43T46DSD6RFJANCNFSM4QT527AA
.

Boot variables/boot order was not exactly right in my case, had to run the bootloader manually.

Hm. I guess I need to look into this @temerkhanov -- any specifics?
Also, @eriknordmark -- any chance you can give this a try?

I ran it on qemu 4.2.1 in OpenSUSE 15.2. Without acceleration everything
looks fine, with it EDK2 runs its shell right away
Regards,
Sergey
On Fri, Sep 4, 2020 at 7:27 PM Roman V Shaposhnik<EMAIL_ADDRESS>wrote:

Boot variables/boot order was not exactly right in my case, had to run the
bootloader manually.
Hm. I guess I need to look into this @temerkhanov
https://github.com/temerkhanov -- any specifics?
Also, @eriknordmark https://github.com/eriknordmark -- any chance you
can give this a try?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lf-edge/eve/pull/1411#issuecomment-687252578, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6GJXGO7T2C7GMJACGSAITSEEINXANCNFSM4QT527AA
.


Can't type in scene address bar

whatever I type, even backspace, creates new text

Thanks, this is a known issue.

This entire UI will likely be moved to the map screen.


Loading Issues

When loading kencadima.eth there's an error.

This appears to be an issue with the OpenSea API. I've made identical queries (outside of the app) to the one causing the issues, but with different addresses and they work fine. I reported the issue to them.


Query the state of the pull to refresh view

This is useful when you need to differentiate between full screen loading views and pull to refresh without needing to include local state.

Hi @MadsBogeskov! Thanks for the PR

No particular reason. It might be excessive, however when it still compiled (so it wasn't used outside this class), and it wasn't set to public I assumed it should be considered private.

And thank you for the Pod :)


_this.services.languageUtils.isWhitelisted is not a function

Please help me.
thanks

i18next-express-middleware replaced by i18next-http_middleware.
work fine.


Refactor the command-line argument handling

There are two major things accomplished by this PR:

Some CLI-related types were declared in espflash which were only used in cargo-espflash, so these have been moved
The subcommand actions have been refactored to be simpler and to duplicate less code

On top of this, I performed some normalization of type/variable names, just to make things more consistent throughout. I also changed some of the command-line arguments' types from String to PathBuf, where applicable.

Nice improvements code-wise, however the behavior changed a bit

running espflash w/o args previously just printed the help but now it enumerates the ports and let me choose one (if more than one) and connects to it before printing help, if there is no serial port found it gives an error

passing a serial port on the command line gives an error

❯ cargo run --package espflash -- com12 \projects\esp\esp-hal\target\xtensa-esp32s2-none-elf\debug\examples\timer_interrup
t
    Finished dev [unoptimized + debuginfo] target(s) in 0.11s
     Running `target\debug\espflash.exe com12 \projects\esp\esp-hal\target\xtensa-esp32s2-none-elf\debug\examples\timer_interrupt`
Error: cargo_espflash::no_serial

  × No serial port specified in arguments or config
  help: Add a command line option with the serial port to use

passing a serial port on the command line gives an error

mhhh - seems that wasn't introduced with these changes - looks like the same happens on master

Great, thanks for the input! I'm just going to convert this to a draft until I have some time to come back to it (which will hopefully be soon).

I believe I have addressed the issues which were raised in the most recent commit.


pylint: Detect pylintrc file

I'm using ALE for running pylint. But I have a trouble with it; pylint is invoked with default configuration.
pylint can be configured with pylintrc file in repository. In my project, .pylintrc is put at the top of the repository and I want pylint invoked by ALE to use it.
By this PR, ALE can detect the pylintrc config file in parent directory and use it for linting.
When a user specified config file with --rcfile option  in g:ale_python_pylint_options, ALE checks it and the detection is not performed.

@w0rp Sure. Thank you for creating a new issue. Let's discuss this on the thread.


Added to X.txt

I've cleaned up some of the duplicate entries in X.txt and added some new ones for issue #27.

Sorry about that. I assumed the duplicates were in the file inadvertently. I've added them back in and resolved the merge conflicts.

Perfect! Thanks!


Get client-side reference to marker

Hi,
Is there any way I can get a client side reference to a marker created by the HTML helper?

Hi @pippipmilk,
Yes, it is possible. Please see the issue #59.


Adjust stale-bot config

Adjustments to stale-bot configuration as discussed in SIG meeting and slack.

Change staleLabelfrom wontfix to stale
Add help wanted to exemptLabels

Merging #49 (8af69c1) into main (5cbb37f) will decrease coverage by 0.22%

Codecov Report still being flaky.


Add ColQwen2 example

Adds a chat with RAG example using the following things:

ColQwen2 to index the docs
Qwen2-VL as a VLM
Gradio chatUI and PDF upload UI

Some things I would do if I wanted people to actually use this in prod. Curious which of these people think is worth doing

Optimize the cold start (takes 2 mins now on average)

Optimize the inference time for the chat (currently ~10s). Would try vllm, but there's an issue with VLLM and the transformers version that ColQwen2 needs, so I'd have to build vllm from source, which would increase build time?

Try to make the app use less memory (I currently need an 80gb A100, largely because though the underlying model is the same, I couldn't find a clean way to use the same underlying object for the model, and so I end up having 2 model objects)

Type of Change

[x] New example
[ ] Example updates (Bug fixes, new features, etc.)
[ ] Other (changes to the codebase, but not to examples)

Checklist

[ ] Example is testable in synthetic monitoring system, or lambda-test: false is added to example frontmatter (---)

[x] Example is tested by executing with modal run or an alternative cmd is provided in the example frontmatter (e.g. cmd: ["modal", "deploy"])
[ ] Example is tested by running with no arguments or the args are provided in the example frontmatter (e.g. args: ["--prompt", "Formula for room temperature superconductor:"]

[ ] Example is documented with comments throughout, in a Literate Programming style.
[ ] Example does not require third-party dependencies to be installed locally
[ ] Example pins its dependencies

[ ] Example pins container images to a stable tag, not a dynamic tag like latest
[ ] Example specifies a python_version for the base image, if it is used
[ ] Example pins all dependencies to at least minor version, ~=x.y.z or ==x.y
[ ] Example dependencies with version < 1 are pinned to patch version, ==0.y.z

Outside contributors
You're great! Thanks for your contribution.

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-7a1075b.modal.run

Optimize the cold start (takes 2 mins now on average)

Shorter than 1 min would be nice. Where are we spending time here? Is it loading the models, indexing the PDFs, or something else?

Optimize the inference time for the chat (currently ~10s). Would try vllm, but there's an issue with VLLM and the transformers version that ColQwen2 needs, so I'd have to build vllm from source, which would increase build time?

I would suggest not optimizing inference time unless it's a durable improvement. I suspect vllm will resolve this issue soon, so I'd skip working on it for now.

Try to make the app use less memory (I currently need an 80gb A100, largely because though the underlying model is the same, I couldn't find a clean way to use the same underlying object for the model, and so I end up having 2 model objects)

This might be worth looking into. Getting onto A100-40s (and later L40Ses) would be really nice.

Cold start time is coming from loading the models

Reduced cold start to 1 min by realizing we weren't caching checkpoints. Memory snapshotting would probably speed things up even more, but not sure if we should include that in the example.
Sticking to 80GB because no non-ugly way to reuse the Qwen2VL superclass from ColQwen2

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-c97a29f.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-e52659d.modal.run

Have reduced the cold start

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-f06a9a5.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-1aa3914.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-c2767f3.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-64c2fd0.modal.run

What's the status?

What's the status?

@erik-dunteman to pick this one up

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-72f967a.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-5e0cb53.modal.run

This one should be ready to go, pending the following:

CI passes
if we want to add automated testing (in the context of gradio, would need to call the gradio api directly from local entrypoint)

Changes since I took the PR over:

state no longer stored in class's "self", instead using modal dict, allowing concurrent users and horizontal scaling.

way to ID users, using gradio state

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-b80fa95.modal.run

Nice work! Will review quickly tomorrow.

lookin good!

@charlesfrye I'd like to disable the keep_warm on this, cool if I make that one-line change?

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-1b7a289.modal.run

🚀 The docs preview is ready! Check it out here:     https://modal-labs-examples--frontend-preview-064ef7d.modal.run


Origin should report storage type to the director in its advertisement

For non-POSIX origins, we can't upload files to the origin, thus give us 403 errors when doing so at the director file tests. Let's report the storage type from the origin to the director and skip issuing director tests to that origin if it's not in POSIX, and document the reason why the test is off in various places (prometheus, server log, etc)

I wanted to slightly disagree here.
We should report the storage type to the director because it's an important statistics we should aggregate.  For example, when counting the number of origins in prometheus, this is a label we should use.
However, if the origin knows it cannot support director tests, it should explicitly state this in the ad instead of trying to have the director guess/infer this from other properties.  For example, the NCAR origin cannot do director tests but is POSIX.

Yes, I think that's right!


IED Editor collapses all containers when opening a Wizard

Describe the bug
In the IED Editor more Wizards are used to edit or show data. When you open the containers to view DA Values and then click the Edit IED button the screen is reset to the default collapsed containers. The same for the new Info Wizards for DO/DA Elements.
To Reproduce
Steps to reproduce the behavior:

Open a SCD file with IED's configured
Go to 'IED Editor'
Open all type of containers to view information
Click on 'Edit' icon of the IED
All manual opened container are closed

Expected behavior
It would be nice that the IED Editor keeps track on which containers are expended/opened and uses that info when repainting/rebuilding the screen. When a new document is opened this needs to be reset.

Also remember the last selected element.

This will be adressed with the major OpenSCD core fix.

Good for now!


Connect Outline Server to my own Client Side Application.

Describe the issue
I have built my own client side app using Flutter. I have also built a server on GCP using Outline Manager. But now I want to connect my own client app with server to provide VPN services. So, how can I do that?  (I hope I can bypass the ACCESS KEY as I don't want to implement it)
Would building a REST API be useful here?
What I actually want is that my client app (which is made using Flutter) should be able to connect to the Server hosted on GCP.
Environment

OS: Windows 10 Pro (for developing), Android & iOS (client side app), GCP (for hosting the server)
Yarn version: 1.22.4
Node.js version: v10.16.3
Development: Android Studio (Flutter)
I have also downloaded the Outline Server repo and Outline Client repo (In case if needed)

Additional context
I would appreciate if helped. Sorry, for noob issue. But after all, every mistake is a new experience. If any other things, documents are required, you can surely request for it.

For Android, starting from https://github.com/Jigsaw-Code/outline-client/issues/786#issuecomment-628041867 the call path goes: outline_server.ts passes actions to the outline plugin, which are received in OutlinePlugin.java which calls down to VpnTunnelService.java which calls down to Shadowsocks.java
You might want to call Shadowsocks.java yourself, but you will need to pass it a config, like VpnTunnelService.java does.
Alternatively you could copy Shadowsocks.java and make your own version. It appears to call out to libss-local.so
If Flutter cannot call into java classes, then you'll need to do what Shadowsocks.java does, but in Flutter code. (Start a process by executing libss-local.so with some arguments.)

Thanks for the help @joeytwiddle But I actually, I don't have much experience in solving such types of problems. So, I just came on a conclusion that editing the front-end will be a much easier and a convenient solution.
So, before starting up with the editing, I thought of building the android app. But I am getting an error with code 127. I have opened an issue on it, so could you please help me with that?
Issue: here
Also, just wanted to ask that when we build the app (let's say for android), on opening the app we get the option to input the ACCESS KEY. So, how can I remove that part and make a drop down menu instead which directly connects to the server.
In short, I mean that even if I get an ACCESS KEY while creating a server in the Outline Manager; just to bypass the ACCESS KEY, do I have to implement it inside the code? Or do something else?
Thanks for helping me out this far :)

Read what the current app does when the users adds a new ss key. (Hint: It adds a new server to the server store.)
Do that with your own code instead. Then you can remove the existing access key UI, because you are doing it automatically.

The current app shows a list of servers as big circles/logos.
You can remove that code, and use similar code to display the list of servers in a small menu instead.

This may be made easier by our forthcoming SDK. It's not yet ready for use, but please keep an eye out!


When converting tf to tflite it changes the dtypes so operations like add won't work beacuse of different dtypes RuntimeError: tensorflow/lite/kernels/add.cc:93 input1->type != input2->type (INT32 != FLOAT32)Node number 296 (ADD) failed to prepare.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 18.04.5 LTS
TensorFlow installed from (source or binary):
tensorflow 1.15 installed with pip. altough I have tried with older versions

Command used to run the converter or code if you’re using the Python API
If possible, please share a link to Colab/Jupyter/any notebook.
# Copy and paste here the exact command
tflite_convert --graph_def_file=./saved_model2.pb --output_file=./mask-rcnn-model2.tflite --output_format=TFLITE --input_arrays=input_image,input_image_meta,input_anchors --input_shapes=1,1024,1024,3:1,14:1,261888,4 --output_arrays=mrcnn_class/Softmax,mrcnn_bbox/Reshape --enable_select_tf_ops --allow_custom_ops --target_ops=TFLITE_BUILTINS,SELECT_TF_OPS

The output from the converter invocation
# Copy and paste the output here.
2020-08-12 16:57:53.877756: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-08-12 16:57:53.899894: E tensorflow/stream_executor/cuda/cuda_driver.cc:318] failed call to cuInit: CUDA_ERROR_COMPAT_NOT_SUPPORTED_ON_DEVICE: forward compatibility was attempted on non supported HW
2020-08-12 16:57:53.899966: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:169] retrieving CUDA diagnostic information for host: AI
2020-08-12 16:57:53.899989: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:176] hostname: AI
2020-08-12 16:57:53.900089: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:200] libcuda reported version is: 450.57.0
2020-08-12 16:57:53.900150: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:204] kernel reported version is: 440.100.0
2020-08-12 16:57:53.900158: E tensorflow/stream_executor/cuda/cuda_diagnostics.cc:313] kernel version 440.100.0 does not match DSO version 450.57.0 -- cannot find working devices in this configuration
2020-08-12 16:57:53.900446: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-08-12 16:57:53.905236: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-08-12 16:57:53.905657: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x58a7850 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-08-12 16:57:53.905678: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
/home/tensorbook/.local/lib/python3.6/site-packages/tensorflow_core/lite/python/lite.py:854: UserWarning: Property target_ops is deprecated, please use target_spec.supported_ops instead.
  "target_spec.supported_ops instead." % name)
2020-08-12 16:57:55.568703: I tensorflow/core/grappler/devices.cc:55] Number of eligible GPUs (core count >= 8, compute capability >= 0.0): 0
2020-08-12 16:57:55.568867: I tensorflow/core/grappler/clusters/single_machine.cc:356] Starting new session
2020-08-12 16:58:22.490992: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:786] Optimization results for grappler item: graph_to_optimize
2020-08-12 16:58:22.491020: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:788]   constant_folding: Graph size after: 1967 nodes (-1026), 2180 edges (-1093), time = 26054.7871ms.
2020-08-12 16:58:22.491025: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:788]   constant_folding: Graph size after: 1967 nodes (0), 2180 edges (0), time = 410.548ms.

Also, please include a link to the saved model or GraphDef
# Put link here or attach to the issue.
https://drive.google.com/file/d/17JF-Gco7xSbszPXAL-qjFbdDdlW73QJM/view?usp=sharing        "saved frozed model"
https://drive.google.com/file/d/1t2sNK8wBatOUum3XHprO0EejWaGoMKGB/view?usp=sharing     "tflite converted model"

Failure details
I'm working with instance segmentation keras/tensorflow model "https://github.com/matterport/Mask_RCNN", I'm not using the standard model but retrained or transfer learned to a different dataset...
the model was converted successfully, but when I try to infer with this code:
interpreter = tf.lite.Interpreter(model_path="../../logs/mask-rcnn-model2.tflite")
interpreter.allocate_tensors()

input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()

image = cv2.imread('./655.png')
image = image[..., ::-1]
image = image.astype(np.float32)
resized = cv2.resize(image, (1024,1024))

interpreter.set_tensor(input_details[0]['index'], [resized])

interpreter.invoke()

output_data = interpreter.get_tensor(output_details[0]['index'])

but if fails at interpreter.invoke() as I said in the title:
Traceback (most recent call last):
  File "infer_tflite.py", line 529, in <module>
    interpreter.invoke()
  File "/home/tensorbook/Documents/Mask_RCNN_Kikes/tf_night_env/lib/python3.6/site-packages/tensorflow/lite/python/interpreter.py", line 525, in invoke
    self._interpreter.Invoke()
RuntimeError: tensorflow/lite/kernels/add.cc:93 input1->type != input2->type (INT32 != FLOAT32)Node number 296 (ADD) failed to prepare.

I've tried using tf.cast() or even keras K.cast() from tf.float32 to tf.int32 cause later I will bump with a tf.gather_ND which only accepts int32 as index... I'm able to advance and solve the error if I cast to tf.float32 but when I try to cast it back to int32 it does it in tf but in tflite it doesn't cast it. the model architecture is something like this...
image_area = tf.cast(image_shape[0] * image_shape[1], tf.float32)
roi_level = log2_graph(tf.sqrt(h * w) / (224.0 / tf.sqrt(image_area)))
roi_level_2 = K.cast(tf.round(roi_level), tf.int32)
sum_to_roi_level = K.cast(4, tf.int32)
roi_level = tf.minimum(5, tf.maximum(
    2, sum_to_roi_level + roi_level_2))
roi_level = tf.squeeze(roi_level, 2)

# Loop through levels and apply ROI pooling to each. P2 to P5.
pooled = []
box_to_level = []
for i, level in enumerate(range(2, 6)):
    ix = tf.where(tf.equal(roi_level, level))
    level_boxes = tf.gather_nd(boxes, ix)

Any toughts on this ? I have tried many cast to many elements in the model but the casts to int32 doesn't seem to make effect on tflite. I would greatly appreciate your help. thanks a lot in advance.

@danielsernabuitrago
The code seems incomplete to replicate, please share complete stand alone code such that we can replicate the error faced or if possible share a colab gist with the error faced.


Plex  404

I hate to be filing this issue, but I am at my wits end.  I cannot get plex to work at all. I have tried multiple fresh installs. All other containers work without issue. Omnimount / my remote works without issue. I can backup and restore without issue. ONLY Plex continues to give me a 404. Please see my container log below (doesn't seem helpful), and I am happy to provide any other logs that can help.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 40-plex-first-run: executing...
[cont-init.d] 40-plex-first-run: exited 0.
[cont-init.d] 45-plex-hw-transcode-and-connected-tuner: executing...
[cont-init.d] 45-plex-hw-transcode-and-connected-tuner: exited 0.
[cont-init.d] 50-plex-update: executing...
[cont-init.d] 50-plex-update: exited 0.
[cont-init.d] done.
[services.d] starting services
Starting Plex Media Server.
[services.d] done.
Critical: libusb_init failed
It continues to show as "unhealthy", but never does anything beyond this.
I cannot find any other errors anywhere. Any advice or help is appreciated.

Well, let's start with what it isn't:  it doesn't seem that Plex itself is having any startup issues.  My logs look nearly identical and I assure you that my Plex is working:
plex  | [s6-init] ensuring user provided files have correct perms...exited 0.
plex  | [fix-attrs.d] applying ownership & permissions fixes...
plex  | [fix-attrs.d] done.
plex  | [cont-init.d] executing container initialization scripts...
plex  | [cont-init.d] 40-plex-first-run: executing...
plex  | [cont-init.d] 40-plex-first-run: exited 0.
plex  | [cont-init.d] 45-plex-hw-transcode-and-connected-tuner: executing...
plex  | [cont-init.d] 45-plex-hw-transcode-and-connected-tuner: exited 0.
plex  | [cont-init.d] 50-plex-update: executing...
plex  | [cont-init.d] 50-plex-update: exited 0.
plex  | [cont-init.d] done.
plex  | [services.d] starting services
plex  | Starting Plex Media Server.
plex  | [services.d] done.
plex  | Jobs: Exec of ./CrashUploader failed. (2)
plex  | Critical: libusb_init failed
plex  | Dolby, Dolby Digital, Dolby Digital Plus, Dolby TrueHD and the double D symbol are trademarks of Dolby Laboratories.

So let's investigate further:
Start Plex with an omni up or docker restart plex command
Go to the logs directory (command logs)
Dump all the Plex health errors to the screen with grep Plex health
Assuming what you get is more than empty space/zero output, can you share the last few lines?  They may look something like this:
Thu Oct 13 22:37:48 MST 2022 - Plex - Media failure
Sat Oct 15 07:34:59 MST 2022 - Plex service not running
Sat Oct 15 07:39:13 MST 2022 - Plex - OmniMount missing
Sat Oct 15 07:39:23 MST 2022 - Plex - OmniMount missing
Sat Oct 15 07:39:38 MST 2022 - Plex service not running

Thank you for the assistance!
The health logs seem to be a number of entries of :
Wed Oct 19 07:16:58 CDT 2022 - Plex service not running
Wed Oct 19 07:17:08 CDT 2022 - Plex service not running
Wed Oct 19 07:17:18 CDT 2022 - Plex service not running
Wed Oct 19 07:17:33 CDT 2022 - Plex service not running
Wed Oct 19 07:17:43 CDT 2022 - Plex service not running
Wed Oct 19 07:17:53 CDT 2022 - Plex service not running
Wed Oct 19 07:18:03 CDT 2022 - Plex service not running
Wed Oct 19 07:18:14 CDT 2022 - Plex service not running
I can't seem to find any information on this to lead me in troubleshooting. If I've missed something common, I apologize. Thanks again for the help!

This means one of two things:  The "netstat" command is missing from the Plex Docker (which is normal but OmniStream should have installed this) OR Plex hasn't opened port 32400 internally.
Let's temporarily disable the healthcheck for Plex.  Edit the 101-plex.yaml (or 101-plex-hw.yaml if you're using hardware transcoding) so that
    healthcheck:
      test: ["CMD-SHELL", "/plexcheck"]
      interval: 10s
      timeout: 2s
      retries: 3

gets commented out and looks like this:
    #healthcheck:
      #test: ["CMD-SHELL", "/plexcheck"]
      #interval: 10s
      #timeout: 2s
      #retries: 3

And then retry starting up Plex (omni up)

You were correct. It was related to the healthchecks. After commenting out the healthcheck lines within the yaml, Plex was able to start and be accessible. How can I resolve this to be "working as intended"?
Once again, thank you for the assistance. I moved over from Gooby (<3 TechPerplexed) and I love OmniStream.

OK, so the good news is that "we have a workaround."  Not ideal, but at least we're not blocking you from countless hours of top-notch entertainment :)
I've seen this happen to me if I ported over a "netstat" command from the pre-OmniStream versions.  Previously, we were mapping netstat to one on the host but since the virtualized container may be running a different OS, this would sometimes fail.  If that file is left over, then we can't install the right one.
Try this:
docker stop plex
docker rm plex
docker system prune -a -f --volumes
omni apps
rm 101-plex*
omni update
This will force a brand new Plex to be downloaded so that the install can proceed from scratch.  The line where you're removing the plex YAML will force OmniStream to "self-heal" and grab that missing file so that you have the "genuine" build again (ie, the healthcheck won't be commented out).  If we didn't do this, then your local file would exist but be different from that of the repository so we DON'T overwrite it during updates (there's a different process used if you want to make permanent customizations).
See if OmniStream / Plex starts up normally now.  It is not unusual to see a couple "restarts" or "unhealthy" warnings but it should become available in a couple minutes (normally).  Does this work?
If not, then we need to dive in deeper into why the healthcheck is failing.

SUCCESS! That worked!
Plex is now showing healthy, and working normally. Thank you!
Might this be a good "issue" to pin for future users? Could this scenario apply to other containers, not just Plex?
Again thank you for the assistance. You have a long time user now.

Excellent! Again, thank you for the excellent support!

@Misterobots Thank you for pointing out this issue. To finish this off, we have added the line (docker system prune -a -f --volumes) to the Gooby upgrade script, so that future "converts" will not run into this issue 😄


Unable to generate report getting following error

Environment (please complete the following information):

multiple-cucumber-html-reporter: 1.11.4
Node.js version: v10.4.0
NPM version: 6.1.0
Platform name and version: MacOsX
Cucucmber version: 5.0.2

Config of wd + cucumber + multiple-cucumber-html-reporter
var reporter2 = require('multiple-cucumber-html-reporter');
Describe the bug
Unable to generate report from a single json file available at the path, was working previously,
able to gererate html for same .json by cucumber-html-reporter
Expected behavior
Should be able to generate report as the same cucumber-report.json is generating report by another npm package
Log
TypeError: JSON.parse(...).map is not a function
at files.map.file (.../node_modules/multiple-cucumber-html-reporter/lib/collect-jsons.js:23:30)
at Array.map ()
at collectJSONS (.../node_modules/multiple-cucumber-html-reporter/lib/collect-jsons.js:20:15)
at Object.generateReport [as generate] (.../node_modules/multiple-cucumber-html-reporter/lib/generate-report.js:67:23)
at .../runtime/world.js:244:27
at process._tickCallback (internal/process/next_tick.js:61:11)
Additional context
Using a customised framework using cucumber+wd

I am having the same problem, I am using the wdio-json-reporter plugin to generate the json file, here my generated json file:
{   "start": "2018-12-03T17:05:18.133Z",   "end": "2018-12-03T17:05:28.753Z",   "capabilities": {     "browserName": "chrome",     "chromeOptions": {}   },   "host": "<IP_ADDRESS>",   "port": 4444,   "baseUrl": "http://localhost:8080",   "framework": "cucumber",   "mochaOpts": {     "timeout": 10000   },   "suites": [     {       "name": "Google Test",       "tags": [],       "duration": 1095,       "start": "2018-12-03T17:05:27.487Z",       "end": "2018-12-03T17:05:28.582Z",       "tests": [],       "hooks": []     },     {       "name": "Open Google",       "tags": [],       "duration": 1052,       "start": "2018-12-03T17:05:27.529Z",       "end": "2018-12-03T17:05:28.581Z",       "tests": [         {           "name": "I am on Google home page",           "start": "2018-12-03T17:05:27.531Z",           "duration": 1223,           "state": "pass"         },         {           "name": "The title of the page is \"Blah\"",           "start": "2018-12-03T17:05:28.559Z",           "duration": 195,           "state": "fail",           "error": "expected Google to equal Blah ignoring case",           "standardError": "AssertionError: expected Google to equal Blah ignoring case\n    at World.<anonymous> (/Users/john/workspace/webdriverio-tut/tests/features/step_definitions/google.steps.js:8:38)\n    at Promise.F (/Users/john/workspace/webdriverio-tut/node_modules/core-js/library/modules/_export.js:36:28)"         }       ],       "hooks": []     }   ],   "state": {     "passed": 1,     "failed": 1,     "skipped": 0   } }
If I wrap this json message in an array [], then it does not throw error but the report is blank, no scenarios number, no feature numbers, no pie chart, all numbers are just zero.

thanks for this nice reporting tool.
will be nice if could show the screenshots captured in afterstep hook, (as in jenkins cucumber-jvm report) ... thanks


Malformed imports for array overrides

Version
1.18.0
What happened?
I tried to upgrade to sqlc v1.18.0 and the Go compiler reports malformed import path in the generated sources. I am using column overrides for array types and the generated code keeps the [] the import URL.
Here is the code that was working in v1.17.2: https://play.sqlc.dev/p/17e05c48de88123b99ffff7b585291a45f7db6099cbba3627888c1f238f6c6fe
Here is the same code in v1.18.0: https://play.sqlc.dev/p/5c3833f312ae1596952586f263eecc4ce56b2736aa45e90a1441e04b6a99b8db
If you look at the generated model.go, you will notice the "database/[]sql" import that produces the error.
Relevant log output
No response
Database schema
No response
SQL queries
No response
Configuration
No response
Playground URL
https://play.sqlc.dev/p/5c3833f312ae1596952586f263eecc4ce56b2736aa45e90a1441e04b6a99b8db
What operating system are you using?
Linux
What database engines are you using?
PostgreSQL
What type of code are you generating?
 We just merged a change I made to add a "slice" boolean property to the object definition format for the override "go_type" property. I also added some documentation to the bottom of the Type Overriding section here: https://docs.sqlc.dev/en/latest/reference/config.html#type-overriding
Basically you just need to do something like the following in your overrides list:
{
  "column": "authors.tags",
  "go_type": {
    "import": "database/sql",
    "type": "NullInt64",
    "slice": true
}

This should work with a build off of the main branch, and it will be in the next release.

Thank you @andrewmbenton. I will wait for the next release to upgrade. :pray:


feat: add support for multiselect in users form field

Summary

Make channel and user fields support multiselect property, to allow the user to select multiple values in the form
Ticket Link

https://github.com/mattermost/mattermost-server/issues/21484
Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

App Framework - Make channel and user fields support multiselect property, to allow the user to select multiple values in the form.

Hello @prashant-15,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Hi @prashant-15! When you would like to have the pull request re-reviewed because of new changes, you can use GitHub's re-request review feature. Just click the "refresh" looking button next to a reviewer's name at the top right corner of the pull request page. Also, any discussion specific to a given pull request is best to have here, rather than on Mattermost.

Hi @mickmister,
I have integrated certain comments and have marked them as resolved. Can you please check them.
Also, regarding the ones which are unresolved, can we address them separately in a different PR as they form separate issues and I have tried to keep the behaviour consistent with the existing one. This way we can merge this PR focused towards adding multiselect support in channels and users.

@prashant-15 I have the test app running at https://mattermost-app-test.herokuapp.com now. You can test your PR by:

Install the Apps plugin from the marketplace, or build from https://github.com/mattermost/mattermost-plugin-apps
Enable the plugin
Run /apps install http https://mattermost-app-test.herokuapp.com/manifest.json
Accept the installation form
Run /test form multiselect to open a modal with the multiselect components

This app is also installed at this PR's cloud server https://mattermost-webapp-pr-11572.test.mattermost.cloud/pr11572/channels/town-square

Does this PR require any changes in the apps plugin?

@hanzei No the App framework already supports this feature

Hi @prashant-15, do you have any questions on my comments above? We're very close to getting this merged 🚀

I noticed that bot users are shown in the user autocomplete. Can we filter these out in this PR?

@DHaussermann We're wanting to include this for the Next Cloud app. Please let me know your thoughts on including this PR in the next release

/update-branch

Error trying to update the PR.
Please do it manually.


change jackson version from 2.13.2 to 2.13.4

Hi ,
I have Updated jackson-databind from 2.13.2 to 2.13.4 and
jackson-dataformat-xml from 2.13.2 to 2.13.4 .

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.ishuthakur seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
ishuthakur seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Hi , I have sign the CLA Already :)


Rework hw pytest

There was a switch to run pytest suite using actual hardware (Mellanox NIC). It was highly specific to a setup with corsslinked ports and could not thus cover everything.
I created a better implementation where the Mellanox NIC is connected to another machine (or another NIC on the same machine with network namespace isolation). Then a special scapy script is run on the remote machine to reflect underlay packets back to the test suite (it needs to mangle some of them to prevent isolation rule from firing).
Currently this needs a directly connected machine with no switch in between, as the MAC addresses used would cause the traffic to be dropped I believe. But this is something I want to address later so this test can be run in other environments.
This is also capable of testing the multiport setup and pf1-proxy functionality (as the pytest suite needs to listen on the proxy port)

The change to dpservice C code is actually just a cleanup of a hack to make the previous HW test work.
As for the test suite itself, almost all calls to send() and sniff() were already encapsulated, so adding the de-mangling of packets (for isolation rule prevention) is centralized. There were a few instances of a naked call to send() and sniff() which I encapsulated and it even cleaned the code up.
There is a bit of a messy change to telemetry tests (due to the fact that we change the graph for pf1-proxy, etc.), but this will be fixed once we either remove or rework the way pf1-proxy is implemented.
Apart from the above, the tests themselves are oblivious to this change.

@PlagueCZ
I am trying to visualize how this test is actually wired including the VFs/VMs, PF0/PF1 (PF1 Tap case ?) and monitoring port (Including the reflector PC)
and I can not quite get the complete picture as a mental image. Would it be possible to prepare a really simple draw.io drawing to show how the things are wired/connected and which script is running where. Nothing fancy. Just some boxes and lines with some text boxes would be fine.
We can keep it in the PR and it doesnt have to be part of the codebase.

Hopefully this image covers the idea properly. You need to know that only "underlay packets" ever reach dpservice's PF due to isolation rules (this is not technically correct, the rules are more complex, but the idea holds). This is what reflector.py uses to direct the packet back to dpservice (leave the packet as-is) or to pytest (change EtherType to 0x1337 so the rules do not match).
Virtual machines are then set-up so they are a transparent bridge (explained in the markdown docs) to simply pass packets between dpservice and pytest.
The schema only shows one PF and one VF for clarity, there is no dependence among any of them, but of course the reflector does perform the same on two interfaces (PF0 and PF1).

The PF1 tap case is wired the same except that pytest needs to listen on pf1-tap device instead. But that is actually true for everything running on that machine (frr, curl, ping, ...) as PF1 is inaccessible due to the multiport mode.
In that case dpservice is actually doing more, but you can imagine a blackbox bridge between PF1 and the TAP device and it would work the same, it's just that the blackbox is dpservice itself.


CORS support mentioned and URL updated

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been [squashed][squash-link] into a single commit

Getting an error with squash. We will close this PR & re-submit with the suggested changes. Thank you for your help.

Sure @ipapi-co
You're welcome ☺


ConfigCache has potential problem with synchronization.

Please look at ConfigCache.ClearConfigurations method. It uses _configurations collection which is accessed under lock in other methods of this class.

Team Triage: Up-for-grabs, but we would want to see an analysis of the issue and a trace of the issue that would cause a race before accepting a PR.


python312Packages.seedir: 0.4.2 -> 0.5.0

Diff: https://github.com/earnestt1234/seedir/compare/refs/tags/v0.4.2...v0.5.0
Changelog: https://github.com/earnestt1234/seedir/releases/tag/v0.5.0
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

python311Packages.seedir
python311Packages.seedir.dist
python312Packages.seedir
python312Packages.seedir.dist


Slot name needs to be set via attribute in polyfilled browsers

Fixes #21

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Fix ios8.1 indexedDB null bug

@aaronpowell this is very useful when on cordova

Why not add it to the end of the checks at the start of the file?

+1

@aaronpowell, whats the problem with this implementation?

I would like all fallback options on a single line for consistency.
Sent from my Windows Phone

From: Diego<EMAIL_ADDRESS>Sent: ý4/ý03/ý2015 7:33 AM
To<EMAIL_ADDRESS>Cc: Aaron<EMAIL_ADDRESS>Subject: Re: [db.js] Fix ios8.1 indexedDB null bug (#105)
@aaronpowellhttps://github.com/aaronpowell, whats the problem with this implementation?
—
Reply to this email directly or view it on GitHubhttps://github.com/aaronpowell/db.js/pull/105#issuecomment-77028430.

@aaronpowell  like it:
indexedDB = window.indexedDB || window.webkitIndexedDB || window.mozIndexedDB || window.oIndexedDB || window.msIndexedDB || (window.indexedDB === null && window.shimIndexedDB) ? window.shimIndexedDB : undefined;

Yep. This way there isn't two conditionals.

@aaronpowell done!

:shipit:

npm + bower + tags updated


Publish a .NET Standard 2.0 version

Addressing #20  :
Publish a .NET Standard 2.0 version of the Nuget.
When using form a project targeted to .NET Frameweok, this will help avoid getting all dependencies of NETStandard.Library.

So far all I've done is add netstandard2.0 to the TargetFrameworks.
Not sure how the nupkg is created.
I guessed it is done via appveyor which I'm not to familiar with, are there any additional steps needed for the package to be published for netstandard2.0?

Thanks! Will release it soon.


Convert all non-RGBA colors to RGBA colors

In our usage instructions for the viewer we state the following:

We do ask if you plan to embed the viewer in your own site, that it not just be an unmodified version. Please re-skin it or build upon it.

Users have indicated that this is currently quite hard because we use no less than seven different ways to specify colors in CSS, namely RGB, RGBA, HSL, HSLA, three-digit HEX, six-digit HEX and named, leading to 83 unique color specifications.
This commit series simplifies our color specifications across the entire code base, but in particular in web/viewer.css, to specify all colors in RGBA instead. The choice for RGBA is because RGB is well-known for web development and because we need to be able to express opacity (alpha channel) in the color specification. Doing so brings down the number of unique color specifications to 69. This should be acceptable now because quite a few of the unique color specifications are actually opacity variations on the same color, and given that all colors are now in the same format, using find/replace is now possible, making the re-skinning task a lot easier.
Fixes #11372.

/botio-linux preview

/botio-linux preview

/botio-linux preview

I have also checked the other CSS files. The first one required some changes which are combined in the "three-digit HEX" commit and the last one didn't require changes.

/botio-linux preview

/botio test

Thank you for the thorough review!

Great, thank you! @timvandermeij


Adds eff system

Adds new system Eff and corresponding benchmarks.

Thanks! LGTM. I will merge once the CI completes.

Eff job name was still bench-system-ocaml, can you rerun the checks?

@kayceesrk Koka and Links both appeared to be blocked on some user input. I updated the Links image such that it works headlessly, so the CI must have pulled some stale data. Any idea on how I prime the cache?

Perhaps this helps: https://stackoverflow.com/questions/63521430/clear-cache-in-github-actions

Don't know of an easy way to do this. I may be easiest to comment out the caching that I'd added, and then add it back in the next commit. Let me see if that works.

I updated the Links image such that it works headlessly, so the CI must have pulled some stale data.

Can you point me to the commit for this update?

I updated the Links image such that it works headlessly, so the CI must have pulled some stale data.

Can you point me to the commit for this update?

It should be the latest one on Dockerhub. I probably forgot to push the Dockerfile to main, did I?

I had forgotten to push the Dockerfile, sorry. Let's try again.

I've fixed the Koka image now too. I don't have rights to update this PR @jO-Osko can you merge in the new change such that I can rerun the CI?

@dhil Rebased to main, it should pass now.
Strange, I ticked "allow edits by maintainers", shouldn't that give you access?

@dhil Rebased to main, it should pass now.
Strange, I ticked "allow edits by maintainers", shouldn't that give you access?

Thanks! I've thought so, but in my terminal I got "rejected" when I tried to pull this PR. I wouldn't rule out some configuration issue on my machine...


Possible rounding errors

I am not even sure this is a bug. I would be happy to be told what I did wrong, or where my assumptions are incorrect.
I was looking at this article and decided it would be nice to check what big.js does with this code. So here is my implementation:
// see
// https://
// medium.com/
// the-technical-archaeologist/
// is-cobol-holding-you-hostage-with-math-5498c0eb428b
// for details
const Big = require('big.js');

const N = 20;
const n = jmnJS(N);
const b = jmnBig(N);

console.log(' i | JS floats              | big.js                ');
console.log('-- | ---------------------- | ----------------------');
for (let j = 0; j < N; ++j) {
    console.log('%s | %s | %s',
                j.toString().padEnd(2),
                n[j].toString().padEnd(22),
                b[j].toString().padEnd(22));
}

function jmnJS(top) {
  let x = [];
  x.push(4);
  x.push(4.25);
  for (let j = 2; j < top; ++j) {
    const m = mullerJS(x[j-1], x[j-2]);
    x.push(m);
  }
  return x;
}

function jmnBig(top) {
  Big.DP = 15;
  const n4 = Big(4);
  const n17 = Big(17);
  let x = [];
  x.push(n4);
  x.push(n17.div(n4)); // 17 / 4 = 4.25
  for (let j = 2; j < top; ++j) {
    const m = mullerBig(x[j-1], x[j-2]);
    x.push(m);
  }
  return x;
}

function mullerJS(y, z) {
  return 108 - (815 - 1500 / z) / y;
}

function mullerBig(y, z) {
  const A = Big(108);
  const B = Big(815);
  const C = Big(1500);
  return A.minus(B.minus(C.div(z)).div(y));
}

whose output is:
$ node big.js
 i | JS floats              | big.js
-- | ---------------------- | ----------------------
0  | 4                      | 4
1  | 4.25                   | 4.25
2  | 4.470588235294116      | 4.470588235294118
3  | 4.6447368421052175     | 4.644736842105271
4  | 4.770538243625083      | 4.770538243626231
5  | 4.855700712568563      | 4.855700712592606
6  | 4.91084749866063       | 4.910847499155498
7  | 4.945537395530508      | 4.945537405603871
8  | 4.966962408040999      | 4.966962611681
9  | 4.980042204293014      | 4.980046303618682
10 | 4.987909232795786      | 4.987991540991751
11 | 4.991362641314552      | 4.993012708353129
12 | 4.967455095552268      | 5.000510934790509
13 | 4.42969049830883       | 5.094480439139274
14 | -7.817236578459315     | 6.904184811805049
15 |<PHONE_NUMBER>7106458     | 32.601714976939753
16 |<PHONE_NUMBER>5205927     | 89.665367518797276
17 |<PHONE_NUMBER>162497      | 99.423777409662639
18 |<PHONE_NUMBER>4097244     | 99.971023895048601
19 |<PHONE_NUMBER>792373      | 99.998550834929925

So, unlike the python implementation in the article, the big.js implementation seems to also accumulate enough rounding errors to end up wildly diverging.
Cheers.

First, please explain to me how you set the precision of the division operation and what effect that has.

Hi Mike, thanks for your quick reply! Your question was enough of a hint to find the cause for the behaviour: I was setting the precision in the wrong place (but that didn't really have any effect) and, most importantly, it was too low. I changed from Big.DP = 15 to Big.DP = 40 and now the code works pretty much like the python code (up to rounding differences due to the actual precision). Here is the final code, again, in case this is helpful to someone else, and the final output.
I am glad that the suspicions I had when I opened the issue ("I am not even sure this is a bug") were correct. Thanks for the hint and please feel free to close this issue.
// see
// https://
// medium.com/
// the-technical-archaeologist/
// is-cobol-holding-you-hostage-with-math-5498c0eb428b
// for details
const Big = require('big.js');
Big.DP = 40;  // ---------- this is the relevant change ----------

const N = 20;
const n = jmnJS(N);
const b = jmnBig(N);

console.log(' i | JS floats              | big.js                ');
console.log('-- | ---------------------- | ----------------------');
for (let j = 0; j < N; ++j) {
    console.log('%s | %s | %s',
                j.toString().padEnd(2),
                n[j].toString().padEnd(22),
                b[j].toString().padEnd(22));
}

function jmnJS(top) {
  let x = [];
  x.push(4);
  x.push(4.25);
  for (let j = 2; j < top; ++j) {
    const m = mullerJS(x[j-1], x[j-2]);
    x.push(m);
  }
  return x;
}

function jmnBig(top) {
  const n4 = Big(4);
  const n17 = Big(17);
  let x = [];
  x.push(n4);
  x.push(n17.div(n4)); // 17 / 4 = 4.25
  for (let j = 2; j < top; ++j) {
    const m = mullerBig(x[j-1], x[j-2]);
    x.push(m);
  }
  return x;
}

function mullerJS(y, z) {
  return 108 - (815 - 1500 / z) / y;
}

function mullerBig(y, z) {
  const A = Big(108);
  const B = Big(815);
  const C = Big(1500);
  return A.minus(B.minus(C.div(z)).div(y));
}

$ node big.js
 i | JS floats              | big.js
-- | ---------------------- | ----------------------
0  | 4                      | 4
1  | 4.25                   | 4.25
2  | 4.470588235294116      | 4.4705882352941176470588235294117647058824
3  | 4.6447368421052175     | 4.6447368421052631578947368421052631578958
4  | 4.770538243625083      | 4.7705382436260623229461756373937677054053
5  | 4.855700712568563      | 4.8557007125890736342042755344418052261321
6  | 4.91084749866063       | 4.9108474990827932004402592637886755632383
7  | 4.945537395530508      | 4.9455374041239167247733838031676463805414
8  | 4.966962408040999      | 4.9669625817627005987119384872578630944719
9  | 4.980042204293014      | 4.9800457013556311612686079942904535473866
10 | 4.987909232795786      | 4.9879794484783922601401328939785795280641
11 | 4.991362641314552      | 4.9927702880620680974895925483611355817797
12 | 4.967455095552268      | 4.9956558915066340266240282622253062987591
13 | 4.42969049830883       | 4.9973912683813441128938690348188068633436
14 | -7.817236578459315     | 4.9984339439448169190138974418491568034245
15 |<PHONE_NUMBER>7106458     | 4.999060071970893867816844881082666380492
16 |<PHONE_NUMBER>5205927     | 4.9994359371468391479979564018325951371609
17 |<PHONE_NUMBER>162497      | 4.9996615241037675377889985254675055376285
18 |<PHONE_NUMBER>4097244     | 4.9997969007134179127052066669470847708791
19 |<PHONE_NUMBER>792373      | 4.9998781354779312500760374499300966501485

To get exactly the same result as the article you could use:
Big.DP = 25;
Big.RM = 2;    // round half even


Upgrade Linux Kernel for flatcar-3619 from 6.1.30 to 6.1.31

Subject says it all.

See instead https://github.com/flatcar/scripts/pull/887


Texture issues in Godot 4.3.dev2-mono

Terrain3D version
v0.9.0 & a85d40c
System information
Godot v4.3.dev2.mono - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2070 SUPER (NVIDIA; <IP_ADDRESS>67) - Intel(R) Core(TM) i7-10700K CPU @ 3.80GHz (16 Threads)
Is the issue reproducable in the demo?
Yes
Issue description
I recently upgraded a pathfinder project from Godot 4.2 to 4.3.dev2.mono and encountered a strange issue with the textures on the terrain not being applied correctly. When using the debug views to try and debug the issue the checkered debug view works just fine but the height texture debug view looks a bit strange as well.
All other tools and functionality works just fine, even colour painting.
All screenshots are from the terrain3d demo project for the purpose of reproduction of the issue.
Texture Height debug view

Here's a screenshot from the demo project using the default checkered texture rather than the debug view.

And here is a close up screenshot when using the default demo rock and grass textures

Logs
No response

pathfinder?
They broke a few things in 4.3 shaders. I've noticed in the far distance there is some odd artifact like roughness is 0 or negative.
If you're feeling adventurous it would help get this fixed a lot faster if you bisected the engine from 4.2.1 to master/HEAD to determine which commit broke the shaders so a ticket can be filed upstream, or appropriate changes made here.

Sorry, I just meant an internal project for testing out tech stacks and workflows.
I've compiled the engine at various points between 4.2.1 and 4.3.dev2 and stumbled upon that they made some refactoring to the rendering backend https://github.com/godotengine/godot/pull/83452 as well as a possibly related issue https://github.com/godotengine/godot/issues/86628
I've now compiled the current head as of writing: 5034478 and the issue reported in this ticket seems to have been resolved.


[FLINK-20721][runtime] Remove unknown input channel and the process which updates it

What is the purpose of the change
With the latest pipelined region scheduling, Flink no longer launches a task before knowing the locations of all the partitions it consumes. As a result, unknown input channels and the process to update it is no longer needed.
This PR is based on #14462
Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Closing this ticket because UnknownInputChannel are still needed at the moment if BLOCKING edges exist in a pipelined region.
e.g.
A – pipelined --> B, A – pipelined --> C, B – pipelined --> D, C – blocking --> D.
In this case, even though D knows the location of C when deployed, the corresponding input channel will be unknown because C has not finished producing the BLOCKING result partition. This is needed because reading from an uncompleted BLOCKING result partition is not supported yet.
Besides that, UnknownInputChannel is possibly needed for future individual failover.


jwstuser installation missing from installation.rst docs

The jwstuser installation instructions are missing from the installation.rst docs file. This is described in your README.md file, but should also be described on your installation instructions page
https://github.com/GalagaBits/JWST-FGS-Spelunker/blob/b0d2cc40c9d8f9872ba1c160bb43765176453de1/docs/installation.rst?plain=1#L1-L22
This issue was made as part of the review for https://github.com/openjournals/joss-reviews/issues/6202

Okay, I have added jwstuser instructions on the installation.rst page.


'detail' keyword in success response messages

I've seen that django-rest-auth is using 2 keywords on responses with status code HTTP_200_OK.
In registration.views is used the keyword 'message', while in views is used the keyword 'success'.
I suggest to use the keyword 'detail' as in django-rest-framework, such that in the server we won't have different keywords for different apps.

+1


Base/Underlying Ctx

Closes #259
Implements BaseCtx for BaseComp and UnderlyingCtx for UnderlyingComp, so that we don't have to unwrap Idxs before looking them up, and looking up an Idx in a context where it doesn't belong will error. I think in general this makes it easier to follow what's happening and harder to mess up when writing code, but I guess I'll put that to test when implementing some of the language extensions.
We do have to do some unwrapping ourselves when we want to create new structures that aren't stored in a component's IndexStore fields, e.g. Ranges. We will monomorphize its start and end fields, and then get the corresponding Base<Time>s, then unwrap those when creating the new Range. I'm not sure that there is a good way around this, and I think this is fine.
In some cases we want to modify some existing parts of the BaseComp (like add a Command to the existing Commands), so I exposed some functions for doing this instead of making the component itself visible, which seems like a bad idea.
The part where things get a little strange is when we have to interact with things that are not mono-specific, such as the old Context that we are monomorphizing. When we call Monomorphize::next(), we get a (BaseComp, Base<Component>), where BaseComp is newly-monomorphized component and Base<Component> is a wrapper around the component Idx that is meaningful in the new context. At this point Base<Component> points to a default (empty) component, so the idea is that we look it up in the new context, get the aforementioned default Component, and then call mem::swap on the two. The reason this got a little weird was because we were swapping Components and not BaseComps, and there was no way to rip the component out of its BaseComp wrapper - you can get a reference to it (for the purpose of passing it into methods such as ExprIdx::as_concrete()), but that is it. What I ended up doing was implementing Clone for components so we could clone the thing inside of the BaseComp and then swap that in, but this seems expensive/wasteful. Is there a better way to do this?

Overall, the design looks good. I left one comment about the new Ctx implementations.

The reason this got a little weird was because we were swapping Components and not BaseComps, and there was no way to rip the component out of its BaseComp wrapper - you can get a reference to it (for the purpose of passing it into methods such as ExprIdx::as_concrete()), but that is it.

The problem here seems to be that there is in fact a BaseContext and UnderlyingContext too but it might be tedious to represent them as such. It is a little tricky, having to clone the components, but one way around it is having a function defined in the same file as BaseComp that gets to look into the definition of BaseComp and efficiently implement the swap for you. The requirement would be that it takes a self and returns a value of type Self. Could that work?

Yeah, I was thinking about how far I wanted to go in terms of implementing an actual BaseContext and UnderlyingContext, and it seemed tedious to do it. I think the the swap approach you mentioned is worth exploring, so I'll try doing that.

@rachitnigam I implemented the change for Ctx and also implemented swap inside BaseComp so we don't have to clone it

Let's merge #288 first and then this? Sorry for the conflicts created @gabizon103


[Question]: Is sentry-testkit currently passing tests?

Hi @zivl,
I'm upgrading our mobile apps from @sentry/react-native from 5.22.2 to 5.35.0. Our tests which use sentry-testkit are currently failing.
I checked out the sentry-testkit project to look into it there first; less code, easier to debug, allows me to work out if the issues are specific to our code.
However, cloning either the head of master or the 5.0.9 tag, doing an npm install and then npm run test immediately results in test failures. Is this expected?
All of the __tests__/network-interception.test.ts tests bar one fail like this:
  ● sentry test-kit test suite - network interception › performance › should collect child spans

    TypeError: The "stream" argument must be an instance of ReadableStream, WritableStream, or Stream. Received an instance of Socket

If they're working for you, what version of node are you on please?

Sorry, I should have dug further. It works on node 16 but not 18+ due, I believe, to this: https://github.com/nock/nock/issues/2466
I'll raise PRs

hey @liamjones!
Thanks for reporting this... I actually can reproduce it as well when upgrading to node@18.
.nvmrc file points to node@16, will you be able to work on 16 until I'll work on something to upgrade it?

oh... just saw this: https://github.com/zivl/sentry-testkit/issues/212#issuecomment-2470070430
ok cool :)
and yeah PRs are always welcome 😄


actions.focus not always working

See code in: https://esnextb.in/?gist=ad970780e6ca15325ae9f5637019fc5c
If you click "Focus Name" -> "Focus Email" repeatedly it works, but when you click twice in "Focus Name" it won't focus the input anymore, clicking on "Focus Email" will work just once more and then no focus action will work anymore.
Also from start "Focus Name" -> Tab (focus goes to email) now "Focus Name" doesn't work anymore.

@davidkpiano Thanks, it's working great!!!


No module named tensor2tensor

Description
I installed tensor2tensor using:
pip install tensor2tensor[tensorflow_gpu], I can run commands in the terminal like t2t-trainer, t2t-decoder,... however when I try to import the library in a python script I get this error
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named tensor2tensor
Environment information
OS: ubuntu 18.04
$ pip freeze | grep tensor
mesh-tensorflow==0.0.5
tensor2tensor==1.13.4
tensorboard==1.14.0
tensorflow-datasets==1.1.0
tensorflow-estimator==1.14.0
tensorflow-gpu==1.14.0
tensorflow-metadata==0.14.0
tensorflow-probability==0.7.0
$ python -V
Python 2.7.12
Error logs:
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named tensor2tensor

hey can you try import something like:
from tensor2tensor.layers import common_layers

Traceback (most recent call last):
File "", line 1, in
ImportError: No module named tensor2tensor.layers

hi, I suppose it might be path error, you might want to locate the tensor2tensor folder and add it to your path via export. maybe the t2t guys know better.

The same error here; and can't find t2t folder either, because it's a script and not a command...

hello so here I try to train a model of mask r cnn on my own give with the command
python D:\Projet\CV-CommonTools\model_main.py /
--model_dir training /
--pipeline_config_path training\mask_rcnn_inception_v2.config /
--num_train_steps 60000 /
--sample_1_of_n_eval_examples=0 /
--alsologtostderr
but i get the following error
Traceback (most recent call last):
  File "D:\Projet\model_main.py", line 110, in <module>
    tf.app.run()
  File "C:\Users\Ibrahim\AppData\Local\Programs\Python\Python36\lib\site-packages\tensorflow\python\platform\app.py", line 40, in run
    _run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
  File "C:\Users\Ibrahim\AppData\Local\Programs\Python\Python36\lib\site-packages\absl\app.py", line 299, in run
    _run_main(main, args)
  File "C:\Users\Ibrahim\AppData\Local\Programs\Python\Python36\lib\site-packages\absl\app.py", line 250, in _run_main
    sys.exit(main(argv))
  File "D:\Projet\model_main.py", line 71, in main
    FLAGS.sample_1_of_n_eval_on_train_examples))
TypeError: create_estimator_and_inputs() missing 1 required positional argument: 'hparams'

could someone guided me please

ImportError                               Traceback (most recent call last)
 in ()
9 import collections
10
---> 11 from tensor2tensor import models
12 from tensor2tensor import problems
13 from tensor2tensor.layers import common_layers
ImportError: cannot import name 'models'
I have the same problem using COLAB models and problems raise ImportError


Can you provide me with the files so that I can translate this mod?

Hi~
I would like to try to translate this mod into Chinese to make it easier for me to play the game, can you provide me with a text configuration file?
I will return the translated text to you!

Hey,
Thanks for reaching out! Right now, the mod doesn't have a built-in translator. But I'll definitely look into adding one in a future update. If you have any other ideas or questions, feel free to shoot them my way!
Cheers

Added in version 1.6.0. Please check discord on how you can use it.


EvaluationComponent in ER and PR status

What's the intent of including the statement LaboratoryObservationTopic.EvaluationComponent under EstrogenReceptorStatus?

I think I got it. We're extending CodedLaboratoryObservation.LaboratoryObservationTopic to include EvaluationComponent in order to include NuclearPositivity and AverageStainingIntensity.


Updated/add tool documentation for Visibility toolset and Distance and Direction toolset

Description
Made changes to tool help for all three tools in the Distance and Direction toolset. Added new tool help to the new script tools in the Visibility toolset.
Related Issue
#147 #161 #162 #144 #145 #148 #149 #154 #155 #156
Motivation and Context
Improves tool help
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes

[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 change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

updated/added tool documentation for distance and direction toolset, and visibility toolset

@mfunk since you are still scripting some of these tools, and only one of us can touch this at a time, i would politely request, that have a look at the tool help changes I have done for visibility, just to make sure you are still ok with everything i did

Got it. Will start looking at this sometime this afternoon or tomorrow.

@dfoll  Got a problem. Did you use a version newer than 10.3.1 to do the doc changes? I am not able to see the script tools in the toolbox....

@mfunk if there is a problem, yo i'll solve it


updated card , container , button and search bar in docs

Status
READY/IN DEVELOPMENT/HOLD
Description

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[x ] 📝 Documentation
[ ] 🗑 Chore

@Mehra-Jatin screenshots of the changes


Fix deferred export of array binding pattern

We parse the missing element as an OmittedExpression but cast it to a BindingElement even though it does not have the same shape. The result was an issue where the module transformer did not account for this case and code like the following would result in an exception:
const [a, , b] = [1, 2, 3];
export { a, b };

This PR makes the expected types more explicit and fixes the incorrect case.
Fixes #10778

Looks good to me :+1:


Airflow server running, nothing appears to be working; what could I be doing wrong?

Hi, Thanks for open sourcing this.  It looks like a really great tool.  I'm exploring it to put together a little prototype demo to show to my group how we can use it.  I've been trying to get a really rudimentary DAG running because none of the examples seemed to work.
Airflow version: 1.7.0
Airflow components and configuration: LocalExecutor, postgres
Running on EC2
What I mean by "nothing appears to be working"?

airflow trigger_dag on any of the examples would create a running manual DAGrun but nothing would happen.  It would just hang and stay running.  I could run the tasks individually and they'd show up as successful but the DAGrun would still be going.
I scheduled the simple DAG below and it never ran (I tried to schedule it every 15 seconds so I wouldn't have to wait long to see it do something).  python dags/example.py did not raise an exception
backfilling does do something

Since everything appears so simple and I think I get the basic concepts for Airflow, I just don't get how nothing is happening.  I would expect to see the simple DAG below just run and run with Airflow not thinking much of it and I would see the successes show up.  But nothing is happening.
I would greatly appreciate any help to get over this hurdle.
# The DAG object; we'll need this to instantiate a DAG
from airflow import DAG

# Operators; we need this to operate!
from airflow.operators import BashOperator

from datetime import datetime, timedelta

default_args = {
    'owner': 'airflow',
    'depends_on_past': False,
    'start_date': datetime.now(),
    'email'<EMAIL_ADDRESS>    'email_on_failure': True,
    'email_on_retry': True,
    'retries': 1,
    'retry_delay': timedelta(minutes=5),
    # 'queue': 'bash_queue',
    # 'pool': 'backfill',
    # 'priority_weight': 10,
    # 'end_date': datetime(2016, 1, 1),
}

dag = DAG(
    'hfbm', default_args=default_args, schedule_interval=timedelta(seconds=15))

t1 = BashOperator(
    task_id='hello',
    bash_command='echo "Hello"',
    dag=dag)

t2 = BashOperator(
    task_id='world',
    bash_command='echo "World"',
    retries=3,
    dag=dag)

t2.set_upstream(t1)

@apapanico I think you're encountering a common problem having to do with the start date -- Airflow schedules DAGs for the period AFTER the start date, not ON the start date. (For example, the daily run for a DAG on 3/25 actually starts on 3/26). Since your start date is always read as "right now", the scheduler never gets to the "next" period. Try making your start_date now() + timedelta(days=-1) and I think it should work.

Ah okay.  Thanks for that suggestion.  I updated that part to datetime.now() + timedelta(days=-2), reset the db, and restarted the server.  Still nothing running though.  Does Airflow wait until the next day to start or should I expect it to kick off immediately?

It should start right away -- I see it running under 1.6.2 and master. Only because you didn't mention it, did you restart your scheduler?
What I'm doing:
[put the code in the dags folder]
airflow resetdb -y
airflow webserver
airflow scheduler

And I see it start to run after a few seconds.

Oh geez.....   I was not going airflow scheduler.  It's working.  Sorry about this.  Read the docs more closely, Alex.  I thought the whole operation was done through airflow webserver.

No worries! I've made the same mistake more times than I should admit. Airflow has a lot of components and it can be confusing to get going, but we want to make it as easy as possible. If there's anything that would have been more helpful to you (like a more concise/explicit explanation of the role of the scheduler vs the webserver), please let us know or consider adding it to the docs yourself!

Well good thing I'm not the only one! Thanks for the help, especially that one little line airflow scheduler.
A few thoughts on adding to the docs: (1) I'll look at the tutorial again and see about adding some parts that create the full working example with the server and scheduler running.  (2) As I go and continue to try out Airflow, I will see if I can put together a kind of MWE that might be an example of how one would use it more in practice.


Add Xcode 13.3.1 to macos-12 image

Tool name
Xcode
Tool license
Apple, Inc.
Add or update?

[X] Add
[ ] Update

Desired version
13.3.1 (build 13E500a)
Approximate size
No response
Brief description of tool
Xcode 13.3.1 (build 13E500a), macOS 12+.
https://developer.apple.com/services-account/download?path=/Developer_Tools/Xcode_13.3.1/Xcode_13.3.1.xip
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[X] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
no

Tool installation time in runtime
No response
Are you willing to submit a PR?
no

Hi @bielikb!
We have already added Xcode 13.3.1 to the image and will start the deployment ASAP

Hi @bielikb! We have already added Xcode 13.3.1 to the image and will start the deployment ASAP

is it possible to already use macOS12 as an available environment?
I don't see it

what about macos-latest?

@VladislavAntonyuk macOS-latest is pointed to macOS-11 at the moment. Once macOS-12 is out of private preview we will start the macOS-latest spec transitioning

Hi @miketimofeev is there a way for us to use macos-12 now? When setting our work flow to use it is always showing
Waiting for a runner to pick up this job...

@brugg please take a look at this comment
https://github.com/actions/virtual-environments/discussions/5370#discussioncomment-2544316

@brugg the new image with Xcode 13.3.1 has been deployed.


Do not recreate onViewableItemsChange when the deeplink changes

Ticket Link
Reported in community server
Ticket: https://mattermost.atlassian.net/browse/MM-36686
Release Note
Fixes a crash when bringing the app from the background after tapping on a DeepLink

Building app in separate branch.

Cherry pick is scheduled.


Will not run

Was working fine till about 3 weeks ago

What is the issue? and what package manager are you using?
Or  doesn't it compile

public static var styleForMessageType :(SWMessageNotificationType) ->; .Style = defaultStyleForMessageType
it has these two errors on this line.  I'm running Xcode 8 swift 3 beta
expected declaration
expected type for function result

@Jevon718
Release version 0.2 was Done with swift 2.
Not migrated yet swift 3.0
If you can migrate to swift 3.0 , give a PR,
or I will release one supporting swift 3.0 in a few days

@Jevon718  sorry it took so long , I updated to swift 3.0, check master branch. If its ok will go ahead and tag it to v0.3.0


draft: validate global config works with Ivy

This pr is just a proof that #3613 & #4042 can work together.
⚠⚠⚠ DO NOT MERGE THIS

{
  provide: NZ_CONFIG, useValue: {
    icon: { nzTwotoneColor: '#1890ff' },
    button: { nzSize: 'large' }
  }
}

Closed since #4042 is merged.


Display and apply default filters in listings

This does two things in the listing view:

Mark the currently selected filter in the v-select component on page load (so it applies after page refresh as well)
Add the ability to specify default filter values per filter

This is a simplified example of setting the default value for a filter — display upcoming events only unless a different value for the state filter is selected from the dropdown.
It becomes clear why marking the selected filter becomes necessary: setting a default filter requires marking the current filter on page load.
class EventController extends ModuleController
{
    protected $filters = [
        'state' => 'eventState',
    ];

    protected $filtersDefaultOptions = [
        'state' => 'upcoming',
    ];

    protected function indexData($request)
    {
        return [
            'stateList' => collect([
                'upcoming' => 'Upcoming',
                'past' => 'Past',
                '_' => 'All',
            ]),
        ];
    }
}

This PR would close issue #549.

Thank you so much for this @philippdaun, it makes a lot of sense!


Config migration

SXMO wants to help you migrate your configurations in $HOME between versions with sxmo_migrate. When it differs from the 'system' installs, it provides a big red notification.
Currently, our configurations end up with a lot of nix store paths, so every little tweak results in needing to migrate the configs. Annoying.
It's probably possible to define an SXMO_UTILS_PATH environment variable rather than referring to the nix store, thus making migrations only necessary if something actually changes. This also reduces the things we need to patch out with find-and-replace.

Now all configuration references that would hit the nix store go through /run/current-system/sw/share/. Shouldn't need to migrate so often now.


fix: upgrade @actions/core to v1.10.0 to resolve set-output deprecation warning

Example warning output when running sync-branches:

Related issue: https://github.com/TreTuna/sync-branches/issues/71
Based on GitHub documentation, upgrading @actions/core to v1.10.0 should resolve the deprecation warning: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

@TreTuna This PR resolves the deprecation warning for set-output based on the GitHub documentation. Can you please take a look? Thanks in advance!


chore(glide.lock): update ginkgo, gomega, controller-sdk-go

Updates docker-go-dev and three core e2e vendored packages to their latest commits.

is this in progress or ready for review?

@bacongobbler this is still in progress--it seems to be failing for valid reasons (the test container is never seen as "started"). Tests run fine locally, and I haven't been able to pinpoint the difference when they're in the workflow-e2e Docker container yet.


Update sidecar AWS verification

Add verification without ssh access on master node. Perform check directly with kubectl client.

Could we merge this PR?

- ./_rakesite/test/docs/setup/kubernetes/sidecar-injection.html
  *  External link https://kubernetes.io/docs/tasks/tls/managing-tls-in-a-cluster failed: got a time out (response code 301)

ci/circleci presubmit is failing. Looks like you need to add a trailing / to one of your links (see above)

could you check again ? url is available and other links doesn't contains /

$ curl -s -I https://kubernetes.io/docs/tasks/tls/managing-tls-in-a-cluster |grep -e ^HTTP -e ^location
HTTP/2 301
location: /docs/tasks/tls/managing-tls-in-a-cluster/

The non-slash URL is available but redirects. And our link checker seems to be enforcing use of non-redirect links. cc @ldemailly

yes our link checker correctly enforces that one shouldn't need 2 rountrips to start serving a page

Update done


Random account execution order

Feature request: make account run rather from top to bottom run randomly.

What's the format for multiple accounts?

Just add multiple entries in the accounts.json file (you'll need to remove the .sample)

Feature request: make account run rather from top to bottom run randomly.

Od you think random.shuffle ([https://www.w3schools.com/python/ref_random_shuffle.asp](w3schools exeample)) could do the job ?

Yes that should work. You could just leave the accounts.json as is and let the Python script randomize it at the account variable calltime using the random.shuffle function (or rand.sample, although that is deprecated in Python 3.9)

How do I put it inside the json file, that's for 1 but I don't know the format for 2
[
{
"username"<EMAIL_ADDRESS>"password": "password"
}
]

Yes that should work. You could just leave the accounts.json as is and let the Python script randomize it at the account variable calltime using the random.shuffle function (or rand.sample, although that is deprecated in Python 3.9)

I added the feature

How do I put it inside the json file, that's for 1 but I don't know the format for 2
[
{
"username"<EMAIL_ADDRESS>"password": "password"
}
]

It's a list of dictionaries, if you still don't understand, refer to the docs : https://docs.python.org/3/tutorial/datastructures.html


Improve handling of full fill area

When editing with it enabled, the entire background is filled with whatever area it is in, even if the whole area is not showing. This is bad when editing in residential areas, as the entire screen turns yellow, and anywhere you click selects it.

Maybe this could be fixed by only showing areas that are completely/partially visible in the editor screen, this would also stop iD loading the whole background area, like it does when working in this residential area.

But not showing those large fills is why we have the partial fill option (which is the default).

But not showing those large fills is why we have the partial fill option (which is the default).

Thinking more about it, I guess that is why this exists, closing.


Extension issue

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.17763
VSCode version: 1.34.0

:warning: We have written the needed data into your clipboard. Please paste! :warning:

[ ] ```json
{
"messages": [],
"activationTimes": {
"startup": false,
"codeLoadingTime": 9041,
"activateCallTime": 4,
"activateResolvedTime": 0,
"activationEvent": "*"
},
"runtimeErrors": []
}


Are there any plans to add support for using evalsha with RLock?

hi @mrniko, after some investigation, it appears that the problem labeled as #3656 may not have been resolved(by v3.24.3).
It seems fixing this is not easy; could you identify the challenges?
We've noticed that, occasionally, the eval command takes more than 200 milliseconds to execute in our production environment:

Here is a test case to demonstrate:
import org.redisson.Redisson;
import org.redisson.api.RLock;
import org.redisson.api.RedissonClient;
import org.redisson.config.Config;

import java.time.LocalDateTime;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.ThreadLocalRandom;
import java.util.concurrent.TimeUnit;

public class RedissonLockTest {

    public static void main(String[] args) {
        // Init
        Config config = new Config();
        config.useSingleServer().setAddress("redis://<IP_ADDRESS>:6379");
        RedissonClient client = Redisson.create(config);

        final int nThreads = 4;
        ExecutorService pool = Executors.newFixedThreadPool(nThreads);
        for (int i = 0; i < nThreads; i++) {
            pool.submit(() -> {
                String threadName = Thread.currentThread().getName();
                for (int j = 0; j < Integer.MAX_VALUE; j++) {
                    String lockName = threadName + ":" + j;
                    RLock lock = client.getLock(lockName);
                    // RLock lock = client.getFairLock(lockName);
                    try {
                        //
                        long lockStartTime = System.currentTimeMillis();
                        if (lock.tryLock(5, 5, TimeUnit.SECONDS)) {
                            long lockTime = System.currentTimeMillis() - lockStartTime;
                            if (lockTime > 10) {
                                System.err.printf("@@@@!!!! tryLock. lockName=%-25s time=%-30s cost=%sms%n", lockName, LocalDateTime.now(), lockTime);
                            }

                            try {
                                // sleep for random time
                                long sleepTime = ThreadLocalRandom.current().nextLong(30, 70);
                                TimeUnit.MILLISECONDS.sleep(sleepTime);
                            } finally {
                                long unlockStartTime = System.currentTimeMillis();
                                lock.unlock();
                                long unlockTime = System.currentTimeMillis() - unlockStartTime;
                                if (unlockTime > 10) {
                                    System.err.printf("@@@@!!!!  unlock. lockName=%-25s time=%-30s cost=%sms%n", lockName, LocalDateTime.now(), unlockTime);
                                }
                            }
                        }
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                }
            });
        }
    }

}

result
@@@@!!!! tryLock. lockName=pool-1-thread-3:0         time=2023-11-16T18:19:54.028        cost=14ms
@@@@!!!! tryLock. lockName=pool-1-thread-1:0         time=2023-11-16T18:19:54.028        cost=14ms
@@@@!!!! tryLock. lockName=pool-1-thread-4:0         time=2023-11-16T18:19:54.028        cost=14ms
@@@@!!!! tryLock. lockName=pool-1-thread-2:0         time=2023-11-16T18:19:54.028        cost=15ms
@@@@!!!! tryLock. lockName=pool-1-thread-4:11        time=2023-11-16T18:19:54.683        cost=40ms
@@@@!!!! tryLock. lockName=pool-1-thread-2:12        time=2023-11-16T18:19:54.683        cost=40ms
@@@@!!!!  unlock. lockName=pool-1-thread-1:415       time=2023-11-16T18:20:16.148        cost=13ms
@@@@!!!!  unlock. lockName=pool-1-thread-2:405       time=2023-11-16T18:20:16.148        cost=13ms
@@@@!!!!  unlock. lockName=pool-1-thread-4:665       time=2023-11-16T18:20:30.657        cost=14ms
@@@@!!!! tryLock. lockName=pool-1-thread-1:688       time=2023-11-16T18:20:30.657        cost=17ms
@@@@!!!!  unlock. lockName=pool-1-thread-3:680       time=2023-11-16T18:20:30.918        cost=12ms
@@@@!!!!  unlock. lockName=pool-1-thread-1:691       time=2023-11-16T18:20:30.918        cost=25ms

Usage of evalsha command doesn't mean that the commands start to execute faster. No words about that in docs. Evalsha is applied for RLock. https://github.com/redisson/redisson/issues/3656 was resolved.

Usage of evalsha command doesn't mean that the commands start to execute faster. No words about that in docs. Evalsha is applied for RLock. #3656 was resolved.

While #3656 may have been resolved(), when I debugged it with v3.24.3, I found that it still indeed uses eval instead of evalsha, any idea?


Version bump as part of setup.py?

How can we do a nice version bump for major/minor/micro changes?
It might be nice as an extra possible command in setup.py? Would this be possible?
Or do you think it more fits in pipenv?

I'm waiting with creating a project in order to use the recommendation concerning "how to do version bumps" :) I think using the setup.py like this without version bump might be worse than a Pipfile?

If you don't mind managing your versions in Git, there's a setuptools SCM plugin that simplifies this integration, and allows you to avoid version bump commits entirely.

Another alternative is using bump2version.  It's being used in the wild, see for example https://github.com/PyCQA/pydocstyle/blob/master/.bumpversion.cfg .
I don't think tagging and version bumping should be configured in setup.py -- it should just describe your package and allow it to be built.


improve coverage for mir.las.sum

Just adds
@9il - the remaining uncovered bits here beyond my knowledge - do you mind to have a look?

L76
L757-759
L763-765
L1744, L1750, L1756 (I already tried to tackle this bit in #64 - but you reverted due to failure on Windows?)

do you mind to have a look?

Yes, but i don't want to spend time to cover second level of KB2 summation

but you reverted due to failure on Windows

they was not correct

they was not correct

I know :/


fix test task udpated hook not executing in generating report screen

Description
On report generating screen, the test status badges do not seem to update and is stuck in pending.
Code changes fixes that.
Issue - https://github.com/IMDA-BTG/aiverify/issues/236
Motivation and Context
[Explain the motivation or the context behind this pull request. Why is it necessary?]
Type of Change

fix: A bug fix

How to Test
Run test and generate report. Status badge should update
Checklist
Please check all the boxes that apply to this pull request using "x":

[x ]  I have tested the changes locally and verified that they work as expected.
[ ]  I have added or updated the necessary documentation (README, API docs, etc.).
[ ]  I have added appropriate unit tests or functional tests for the changes made.
[x ]  I have followed the project's coding conventions and style guidelines.
[ ]  I have rebased my branch onto the latest commit of the main branch.
[ ]  I have squashed or reorganized my commits into logical units.
[ ]  I have added any necessary dependencies or packages to the project's build configuration.
[x ]  I have performed a self-review of my own code.
[x ]  I have read, understood and agree to the Developer Certificate of Origin below, which this project utilises.

Screenshots (if applicable)
[If the changes involve visual modifications, include screenshots or GIFs that demonstrate the changes.]
Additional Notes
[Add any additional information or context that might be relevant to reviewers.]

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
   have the right to submit it under the open source license
   indicated in the file; or

(b) The contribution is based upon previous work that, to the best
   of my knowledge, is covered under an appropriate open source
   license and I have the right under that license to submit that
   work with modifications, whether created in whole or in part
   by me, under the same open source license (unless I am
   permitted to submit under a different license), as indicated
   in the file; or

(c) The contribution was provided directly to me by some other
   person who certified (a), (b) or (c) and I have not modified
   it.

(d) I understand and agree that this project and the contribution
   are public and that a record of the contribution (including all
   personal information I submit with it, including my sign-off) is
   maintained indefinitely and may be redistributed consistent with
   this project or the open source license(s) involved.

Frontend Test Passed
Smoke Test Report for Docker.zip

Pipeline Test for Docker Passed
Pipeline Test for Docker.zip

A couple of API-GW unit tests might fail at times because mock data generator seems to create duplicate properties in some objects where there should not be. Will investigate and fix this in separate PR.
Portal coverage changes and some lint fixes will be done in separate PR


Fix cmake installation by using GNUInstallDirs

Don't override GNUInstallDirs
This allows setting local prefix with cmake at installation time
Use the new method in CI
Solves #328

Build status:

Linux
Windows

Please take a look to the CI

Ok, looks like the isolated build doesn't work.
2: CMake Error at /home/jenkins/workspace/Micro-XRCE-DDS-Agent Manual Linux/build/temp_install/isolated/share/microxrcedds_agent/cmake/microxrcedds_agentConfig.cmake:53 (find_package):
2:   By not providing "Findfastcdr.cmake" in CMAKE_MODULE_PATH this project has
2:   asked CMake to find a package configuration file provided by "fastcdr", but
2:   CMake did not find one.
2:
2:   Could not find a package configuration file provided by "fastcdr"
2:   (requested version 1.1.0) with any of the following names:
2:
2:     fastcdrConfig.cmake
2:     fastcdr-config.cmake
2:
2:   Add the installation prefix of "fastcdr" to CMAKE_PREFIX_PATH or set
2:   "fastcdr_DIR" to a directory containing one of the above files.  If
2:   "fastcdr" provides a separate development package or SDK, be sure it has
2:   been installed.
2: Call Stack (most recent call first):
2:   CMakeLists.txt:19 (find_package)
2:
2:
2: -- Configuring incomplete, errors occurred!
2: See also "/home/jenkins/workspace/Micro-XRCE-DDS-Agent Manual Linux/build/microxrcedds_agent-build/test/case/CMakeFiles/CMakeOutput.log".
2: CMake Error at /home/jenkins/workspace/Micro-XRCE-DDS-Agent Manual Linux/test/case/packaging/Packaging.cmake:27 (message):
2:   Error in find_package.

Is it unreasonable to switch to using a package manager like [https://github.com/cpm-cmake/CPM.cmake] or even FetchContent which supports version pinning? The custom method implemented seems to be doing the same thing.


Test-Resource server as standalone module not resolve properties for MongoDB and Kafka

Expected Behavior
When I ran the Test-Resource service as standalone module it should resolve properties for MongoDB and Kafka but I receive this error:

io.micronaut.context.exceptions.BeanInstantiationException: Bean definition [io.micronaut.data.mongodb.transaction.TransactionalClientSession$Intercepted] could not be loaded: Error instantiating bean of type  [com.mongodb.MongoClientSettings]
Message: Could not resolve placeholder ${MONGODB_URI}

Server seems that is running and load these 3 resources resolvers correctly: io.micronaut.testresources.kafka.KafkaTestResourceProvider, io.micronaut.testresources.mongodb.MongoDBTestResourceProvider, io.micronaut.testresources.testcontainers.GenericTestContainerProvider
Actual Behaviour
Properties mongodb.uri and kafka.bootstrap.server are not resolved during integration tests.
Steps To Reproduce
Run "./gradlew check"
Environment Information
OpenSuse Tumbleweed
Jdk 17.0.4
Example Application
https://github.com/fmonorchio/micronaut-test-resources-error
Version
3.6.1

A quick look at your sources (unusual setup!) shows that you have this line in your configuration file: https://github.com/fmonorchio/micronaut-test-resources-error/blob/main/sources/user-service/src/main/resources/application.yml#L6
Test resources only work if you don't set a value for the MongoDB URI, so you should remove this line, and have a separate configuration file for production instead (say application-prod.yml) if you want to pass de value via an env var.

Sorry for my unusual setup but I have cleanup all other modules that are not needed for this error to appears :-)
So, I have moved all properties with env vars to -prod.yml as you said and its ok. Thanks @melix


Add typo-tolerance APIs

Pull Request
Related issue
Fixes #282
What does this PR do?

Adds support for reading, updating, and resetting typo-tolerance preferences

PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Flagging the same kind of error here as other PR - I think CI itself might need an update as passing locally with up to date version.
I'll take a look into scripts  and verify.

I've done an extra force commit above as CI failed to download the MeiliSearch binary after 10s - this was nothing to do with my work. The force commit has prompted CI to try again, and has seemingly made it further this time.

I would recommend waiting for #417 to be merged, then I'll rebase this PR on top of that. With the improvements made there it'll reduce the size of this.

@Sherlouk can you rebase please? 😇

Huge apologies, I did an empty commit in order to fix the failed build (which wasn't due to my code!).

No worries!