Protobuf version conflict between tensorflow and grpc

System information

Linux Ubuntu 20.04
TensorFlow installed from source
TensorFlow version: C 2.5.1
Python version: 3.8.0
Bazel version: 3.7.2
GCC/Compiler version: 9.3.0

Describe the problem
I want to use C-GRPC to encapsulate the Tensorflow model（TensorflowC） as a service, compiling is OK. But errors will occur when running server：

[libprotobuf FATAL /mnt/d/CODE/grpc/third_party/protobuf/src/google/protobuf/stubs/common.cc:87] This program was compiled against version 3.9.2 of the Protocol Buffer runtime library, which is not compatible with the installed version (3.17.3).  Contact the program author for an update.  If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library.  (Version verification failed in "bazel-out/k8-opt/bin/tensorflow/core/framework/tensor_shape.pb.cc".)
terminate called after throwing an instance of 'google::protobuf::FatalException'
what():  This program was compiled against version 3.9.2 of the Protocol Buffer runtime library, which is not compatible with the installed version (3.17.3).  Contact the program author for an update.  If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library.  (Version verification failed in "bazel-out/k8-opt/bin/tensorflow/core/framework/tensor_shape.pb.cc".)
Aborted (core dumped)

Any other info / logs
So I want to know if I can specify a version of protobuf when compiling tensorflow C lib with bazel myself.
If so, which profile should be modified and how?
Thank you for your guidance and help ：)

Hi @jvishnuvardhan ,Could you please look at this issue ?

The answer is YES


GitHub media show svg

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: dwalkr:x: Uncle RicoUncle Rico 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.


USF-1329: Checkout functions

This pull request (PR) adds the following information for the checkout dropin functions:

Description
Returns (when applicable)
Usage

A majority of these functions transform the API response and return a promise that resolves to the transformed data fetched from the API. I need some help understanding how we want to represent that in each case.
See USF-1329 for more information.
Review build
https://commerce-docs.github.io/microsite-commerce-storefront/dropins/checkout/checkout-functions/

Hi @jeff-matthews, overall looks good to me, but please note we are refactoring code (splitting containers, we are trying to share the AddressForm from the Account dropin, etc), so probably in few weeks there will be changes on the API functions (removing, modifying, etc).

Thanks for the review @mcocapiza & @keharper.
I pushed changes to address your comments.

@bdenham,
I refactored according to your revised template that we discussed in Slack. Please review by tomorrow.

Thanks @bdenham. I'm going to re-create the latest changes from this branch and clean up the commits. I'll create a new PR ofter that.


Questions About Setup and Script Usage

I am working on setting up my team's CI system and was pointed to this repo by Randy Witt. I am also using Jenkins with Yocto Project and found these scripts a good starting place for building images and capturing logs. I have been trying to think about the best way to use these scripts. So far, I have been planning on adding them to a Jenkinsfile and wrapping the call to bitbake with a container like crops/yocto-dockerfiles.
 Could you share more insight into your CI setup or additional scripts/Dockerfiles?
Thanks,
John

Hi John!
Always happy to help fellow bitbake users.  I just added the scripts we use for docker integration to this repo.  The idea is that you can modify yocto-build (or any other place that runs bitbake) and run the docker-bitbake script instead, and have that work seamlessly.  Like with the existing scripts, they assume that jenkins-yocto-tools is checked out right next to poky, possibly as git submodules of a supermodule.  Take a look, and if you have any questions feel free to ask!


Implement Pipeline Parallel distribution logic

Describe your changes
Implement Pipeline Parallel distribution logic
Add Llama2 model as an example
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Lint and apply fixes to your code by running lintrunner -a
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Abandoning it in favor of #981


Docker image may be unavailable when starting a unit, leading to overly slow startup

systemd gets annoyed and times out if starting a unit takes too long (see e.g. transcript in #326). Starting a unit requires starting a Docker container which may involve a long download of a Docker image. This is also an availability issue insofar as it means that sometimes when an application is moved it can take quite a while for it to start back up.
To prevent this timeout we should pull the Docker image before starting the container. This is not sufficient to solve the availability issue (that requires coordinating between nodes so the origin node only stops the application once the destination node finishes a pull), so a followup issue should be opened once this one is resolved.

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-350. You are welcome to file additional issues in GitHub if that's easier for you.


Inject OCI Annotations as image labels

Feature request
Would be great to have OCI annotations automatically injected as image labels.
Motivation
This would allow downstream folks to know that an image was built with nixpacks/what version of nixpacks to use, and handle the images appropriately (such as launching containers differently for #658). I could then detect a nixpacks image vs one built another way and perform actions against those images appropriately.
Contribution
If you're willing, I'd be happy to do so. I think this just needs to be added to the template?

Is this something you'd take a PR for?


[WIP] Initial support for EditContext-based validation

This begins to address #123, initial work is just targeting MatTextField with other input controls to come.  I'm looking for feedback and critiques.
The approach taken here is heavily borrowed and adapted from @SteveSandersonMS's sample MDC library.
I've tested the MatTextField control to make sure the additions are optional and don't break existing behavior when EditContext/EditForm are not in use and so far found no issues.
The sample code is a basic usage example, but a couple of quirks I found is that the "validation status indicator" does not appear to be working in the DemoMatTextField page (the colored bar at the bottom of the text field should be red or green depending on validation status, but it's always purple) and also the validation error messages are always black (should be red).  Any ideas?

@ApocDev, that support is already there as part of the original component code from @SamProf, which you can see here, here and here.
Just to be clear, I'm not creating a new component, I'm trying integrate EditContext support into the existing (components) that it makes sense for and not break or alter any of the existing functions or semantics if possible.

Thank you for your contribution. I think it's a good start for implementing other controls to support EditContext.


Invalid value for "v" parameter: cannot convert "" to number;

Hello there,
I'm facing an issue with the module, which throws a the following error:
│ Error: Invalid function argument
│
│   on .terraform/modules/xxxx-external-shell-resourc/outputs.tf line 98, in output "exit_code":
│   98:   value       = local.outputs.exit_code == null ? null : tonumber(local.outputs.exit_code)
│     ├────────────────
│     │ while calling tonumber(v)
│     │ local.outputs.exit_code is ""
│
│ Invalid value for "v" parameter: cannot convert "" to number; given string
│ must be a decimal representation of a number.

I think the issue is coming from the line https://github.com/Invicton-Labs/terraform-external-shell-resource/blob/main/main.tf#L346
xitcode     = !local.create_command_exists ? 0 : local.exitcode_str == "" ? "" : local.exitcode_str == "null" ? null : tonumber(local.exitcode_str)

Where it checks if the local.exitcode_str is an empty string and returns an empty string, but later on the outputs.tf file https://github.com/Invicton-Labs/terraform-external-shell-resource/blob/main/outputs.tf#L98  it runs a tonumber(local.outputs.exit_code) which throw an error when the value is empty
I'll create a PR to remove the tonumber function in the outputs.tf since you already doing that.
Thank you

Hi @KyleKotowick , could you please review this issue and the PR related to it?
its kinda of a blocking point for us.
thank you

@sadok-f Could you provide a minimum example of a Terraform config that shows this error, so I can reproduce it?

@KyleKotowick thank you for your response, it's a bit tricky since terraform-external-shell-resource module is used by another module which calls an API with python with tons of parameters.
but my point is in line  https://github.com/Invicton-Labs/terraform-external-shell-resource/blob/main/main.tf#L346
you're checking for the exitcode_str == ""  then return "" , in output.tf you should remove the tonumber otherwise the exitcode returns an empty string and then the error Invalid value for "v" parameter: cannot convert "" to number will showup
so maybe using the module terraform-external-shell-resource that calls a bash script returning "" will reproduce the issue

The issue here is that it should never be able to get to this state. Was this a situation where you already had this module in the state file, then upgraded the module version to a new version? In v0.X.Y, changes to Y are backwards-compatible, while changes to X are not, and usually require a taint/recreate.
The value of local.exitcode_content, which is the first source of the exit code, should only be an empty string if the exitcode output file doesn't exist. The exitcode outfile should always exist if the resource is being created on a given terraform apply, and therefore it should never be an empty string during a terraform apply where the resource is created.
The random_id resource only stores the exitcode in the prefix string if the triggers are changed, and those triggers are the same as the shell resource triggers, so the random_id should never be triggered to recreate while there's an empty string in the exitcode value. Therefore, only real exit codes should be stored in the random_id prefix when the triggers are changed; it should never be an empty string.
So the issue with your state file is that there should never be {"exit_code": "" ...} in the b64_std value. That's the problem here, not there tonumber on the output. The tonumber should be fine, since in no case should that output ever get an empty string.
I think the incompatible version update is the most likely explanation, so try tainting and re-creating this shell resource if you can (it's best practice to make these things idempotent anyways, in case a recreate is unintentionally triggered for whatever reason). If it still fails after that, you'll need to send me a minimum example of what the issue is. I can't/won't merge the PR, because it would allow the output exitcode to be an empty string, which it should never be (and other users are likely relying on that output to be a number and not a string).
Unrelated, but I'm quite curious as to why you're doing a pip install when the resource is destroyed (command_destroy_unix in your state file). Seems like you'd want a pip uninstall instead.

The issue happened on a gitlab pipeline for a production system, it seems the gitlab runners where the job is executing got crashed and the exit_code got an empty string.
we had to delete manually the state of the failed object (via terraform state rm ...) and rerun the job again to overcome the issue.
it happened to us, it could happen to anyone else who using this module.
if you're saying the exitcode should never be an empty string, then why in your code you're checking if exitcode_str gets an empty string and return back an empty string? in that case you could just return 0 and the issue will be solved.
here in the line 346 in main.tf
https://github.com/Invicton-Labs/terraform-external-shell-resource/blob/main/main.tf#L346
exitcode     = !local.create_command_exists ? 0 : local.exitcode_str == "" ? "" : local.exitcode_str == "null" ? null : tonumber(local.exitcode_str)

if local.exitcode_str == "" then return "" and the in the outputs.tf  tonumber(local.outputs.exit_code) won't work because you're returning an empty string.
I know this is probably an edge case where the exitcode gets an empty string, but even with that the module should handle that properly.
Regarding the command_destroy_unix it's intendant to install some libs and then use them in the python script to destroy resources.
I hope you get my point here, thanks.

If the script fails and outputs an empty string for the exit code, we can't just default it to 0 instead because 0 indicates there was no issue, and clearly there was an issue.
The preferred approach would be to catch that eventuality (script crashes and doesn't output anything) and throw a custom error message, but TF doesn't really support any way to do that, as it would require (a) detecting the empty string response, AND (b) knowing whether the resource was newly created, or if it already existed.

I'm sorry, but I can't approve a PR for a workaround for a scenario that, logically, should never exist (and apparently only exists due to the runner crashing during the apply). That's a one-off scenario that needs to be fixed manually. My recommendation is to manually edit the state file to fix this, or to update the script that you're running to be idempotent and simply terraform state rm to remove this module, then let it be re-created cleanly.

I don't agree with you, this is not a workaround you have a bug in the code and there's an edge case that triggers it.
put aside the scenario that this error occurred with, you're returning an empty string and then doing tonumber to it? this obviously a bug and it needs to be fixed since it happened to me it could happen to anyone else using the project.
I'm going to use my forked version of the module to overcome the error,  this shouldn't be the way how to run an open-source project.
Thanks anyway.

Any updates on this? We started encountering this recently on some executions (ie, same code, different servers). Without a fix we cannot use the module, as the check is done inside the module.

@joaocc I ended up using my forked version which fixed this issue:
https://github.com/sadok-f/terraform-external-shell-resource


warnings are slow to respond

Highlighted warnings in the code are slow to respond when the warning is fixed.
When a warning is detected, the code updates with a yellow underline. Good.
When that warning is fixed, only the statusbar (I use vim-airline) updates to show the number of warnings, however the highlighting still remains.
The highlighting only updates when Vim is reloaded.
Screenshots
Warning Detected:

Warning Fixed:

Logs
Please attach the clangd log from :CocCommand workspace.showOutput if you can.
If possible, run with --log=verbose - note that the logs will include the contents of open files!

Side Note: I do not use clang as my main compiler!
System information
Clangd version (from the log, or clangd --version): clangd version 10.0.0-4ubuntu1
clangd extension version (from :CocInfo):  0.0.80-fe35d5fe2b
Operating system: Ubuntu-20.04 WSL

Can't reproduce with clangd 12, the warnings will disappear after saved.

They should disappear the next time clangd generates diagnostics, which can be slow depending on the code.
(BTW putting non-preprocessor code above your includes, as in your example, will make everything slow. Filed https://github.com/clangd/clangd/issues/838 to warn on this)


M1 instructions don't work as intended

Following the steps here:
https://game2text.com/resources/macos/m1-game2text/
There are perhaps a couple changes, python 3.8 is deprecated so I had to use 3.9
I am able to run the app but the main issue I run into is with tesseract:
Traceback (most recent call last):
  File "/Users/adam/Downloads/Game2Text-main/venv/lib/python3.9/site-packages/eel/__init__.py", line 537, in _process_message
    return_val = _exposed_functions[message['name']](*message['args'])
  File "/Users/adam/Downloads/Game2Text-main/game2text.py", line 34, in recognize_image
    return detect_and_log(engine, image, orientation, session_start_time, get_time_string())
  File "/Users/adam/Downloads/Game2Text-main/ocr.py", line 16, in detect_and_log
    result = image_to_text(engine, cropped_image, text_orientation)
  File "/Users/adam/Downloads/Game2Text-main/ocr.py", line 33, in image_to_text
    return tesseract_ocr(image, text_orientation)
  File "/Users/adam/Downloads/Game2Text-main/ocr.py", line 47, in tesseract_ocr
    result = pytesseract.image_to_string(image, config=custom_config, lang=language)
  File "/Users/adam/Downloads/Game2Text-main/venv/lib/python3.9/site-packages/pytesseract/pytesseract.py", line 486, in image_to_string
    return {
  File "/Users/adam/Downloads/Game2Text-main/venv/lib/python3.9/site-packages/pytesseract/pytesseract.py", line 489, in <lambda>
    Output.STRING: lambda: run_and_get_output(*args),
  File "/Users/adam/Downloads/Game2Text-main/venv/lib/python3.9/site-packages/pytesseract/pytesseract.py", line 352, in run_and_get_output
    run_tesseract(**kwargs)
  File "/Users/adam/Downloads/Game2Text-main/venv/lib/python3.9/site-packages/pytesseract/pytesseract.py", line 284, in run_tesseract
    raise TesseractError(proc.returncode, get_errors(error_string))
pytesseract.pytesseract.TesseractError: (-6, "dyld[31966]: Library not loaded: ../../../leptonica/1.80.0/lib/liblept.5.dylib   Referenced from: <13EB3CE1-0793-372C-928A-174D40B8F059> /Users/adam/Downloads/Game2Text-main/resources/bin/mac/tesseract/4.1.1/bin/tesseract   Reason: tried: '../../../leptonica/1.80.0/lib/liblept.5.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS../../../leptonica/1.80.0/lib/liblept.5.dylib' (no such file), '../../../leptonica/1.80.0/lib/liblept.5.dylib' (no such file), '/usr/local/lib/liblept.5.dylib' (no such file), '/usr/lib/liblept.5.dylib' (no such file, not in dyld cache)")

Even though I get the error everything seems to be there:
leptonica/1.80.0/lib via 🐍 3.9.6 via Game2Text-main
➜ ls -lart
total 14840
drwxrwxr-x@ 3 adam  staff       96 19 Sep  2023 pkgconfig
-rw-rw-r--@ 1 adam  staff  2169872 19 Sep  2023 liblept.dylib
-rw-rw-r--@ 1 adam  staff  3254128 19 Sep  2023 liblept.a
-rw-rw-r--@ 1 adam  staff  2169872 19 Sep  2023 liblept.5.dylib
drwxrwxr-x@ 4 adam  staff      128 19 Sep  2023 cmake
drwxrwxr-x@ 9 adam  staff      288 19 Sep  2023 ..
drwxrwxr-x@ 7 adam  staff      224 19 Sep  2023 .

Oh wait, I got it all working.. I'll fix instructions when I have a chance

For some reason the OCR reading is extremely slow 😖


Can etags be comma-separated?

My reading of MDN says no, but maybe there's something I've missed.
https://github.com/o-p-n/serveit/blob/152c4566d68504350b2de8cac2037c98f20125e6/src/file-entry.ts#L61

These come from the client via the [If-None-Match}(https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/If-None-Match) header, which is comma-separated.

Is there a spec that says this, or is this just observed?

RFC9110


Make group and version field in CronPolicy spec optional

As in the review of #59, it is pretty frustrating if you manually specify Group and Version. This PR makes the group and version fields optional and lets checkpoint find them.
Specifically, this PR implements finding GVK by kind and adds a mutating webhook that automatically populates CronPolicy's spec if group and version are omitted. Even if the mutating webhook does not work, the checker and the reconciler can still find GVK.

TODO: Cache GVK lists.


2FA Yubikey Neo not working in Chrome (yet ok in Firefox) after upgrade to 1.8.1

Gitea version (or commit ref): 1.8.1
Git version:
Operating system: Debian Jessie
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
2FA by Yubikey is not working after upgrade to 1.8.1. I get the same error on https://try.gitea.io
I follow these steps, using Chrome:

register new account
enroll 2FA with Google Authenticator
add Yubikey device (recognized and regularly added)
logout
trying to login shows this message:
Could not read your security key.
Please make sure to use the correct, encrypted (https://) URL.

The same sequence using Firefox works fine, even if I need to touch the Yubikey twice to get in (I usually touch it just once while authenticating on other sites).
I had a similar issue in the past, and I had solved it by stripping the trailing slash in APP_ID and TRUSTED_FACETS definitions in custom/conf/app.ini:
[U2F]
; Two Factor authentication with security keys
; https://developers.yubico.com/U2F/App_ID.html
;APP_ID         = https://my.domain.com/   <-- not working
APP_ID         = https://my.domain.com   <-- WORKING
; Comma separated list of trusted facets
;TRUSTED_FACETS = https://my.domain.com/   <-- not working
TRUSTED_FACETS = https://my.domain.com   <-- WORKING

After upgrading to 1.8.1 I get the error even with these settings.
Again, no errors with Firefox. But no errors as well while using Yubikey with Chrome on github.com
...
Screenshots

After checking older issues, I found #6748, so it looks like this is a Chrome issue: I'm using Chrome 74 and I probably didn't notice it previously because I was already logged in and didn't need to use the Yubikey. I tried reinstalling Gitea v.1.8.0 and the issue is there as well, so it's not related to Gitea upgrade to v.1.8.1.
Anyway, there must be something to check in the Gitea code, as this error is not showing on other websites, like github.com, where the Yubikey is functioning with Chrome 74. Sorry not to be more helpful, but that analysis goes beyond my skills.


Fix the Create/EMI/Farmers Delight bug

I found the bug! It's the Farmer's Delight Create-compat chocolate pie filling recipe - the fluid tag c:chocolates isn't used in Create, it's c:chocolate instead, so switching to the proper tag should stop it from getting index out-of-bounds issues!

Again, this file needs to be modified
src/main/resources/data/farmersdelight/recipes/create/mixing/pie_crust_from_mixing.json
250->20250

Hey, can this get merged? It's a big problem for modpacks still

Agreed, it's entirely breaking Create's EMI plugin :(

Agree, also having issues.

Still should be merged, please

^
agree

Why is this taking so long to merge?

If this doesn't get merged soon I'm gonna hardfork.

Should fixe: #128 #115.


White screen using a specific index

Hi,
I'm using Dejavu on a remote host with a Docker image (latest version).
For some specific index, when I load it a white page appears. Looking chrome inspector I see this error:
VM45 live.js:1 Uncaught TypeError: Cannot read property 'toString' of null
at t.value (VM45 live.js:1)
at kn (VM45 live.js:1)
at xn (VM45 live.js:1)
at On (VM45 live.js:1)
at Hn (VM45 live.js:1)
at $n (VM45 live.js:1)
at er (VM45 live.js:1)
at Zn (VM45 live.js:1)
at Un (VM45 live.js:1)
at Object.enqueueSetState (VM45 live.js:1)

Is a bug? How can I solve it?

Hey @arkkanoid, we have made a major release to Dejavu which will be the actively maintained version going forward. It's available under 3.0.0-alpha tag. Can you try over there? This issue shouldn't be applicable over there.


Add a way to specify the number of build jobs

This change adds a '-j' option to the top-level
build script.  The computed defaults may not
always be suitable.
Tested by building for Ninja (default), Make (-m), and XCode (-x).
I also verified the jobs count via a process monitor.

I like this. I was just thinking how my machine was spending too many cycles building as I couldn't use other applications smoothly in the meantime.


Feedback for Chatbot overview

Lill says we do not need AI to use Chat?

Document Details
⚠ Do not edit this section. It is required for docs.superOffice.com ➟ Docs Team processing.

Content Source: chatbot_overview


Received error message when trying to run a particular batect task after upgrading

I'm getting the following error when trying to run a particular batect task:
Test Suites: 15 passed, 15 total
Tests:       6 skipped, 91 passed, 97 total
Snapshots:   0 total
Time:        9.182s
Ran all test suites.

Clean up: done

Running unit-test...
build-env: running true

Error: Could not run container build-env.
Resizing TTY for container '0d60dfbf9a42a41342b51152a8376f67f1f8c20aa834fc561a436de4fd3daadd' failed: cannot resize a stopped container: unknown

Clean up: done

The task unit-test failed. See above for details.

The above unit test ran but I got an error at the end
I'm finding that tasks which do not include a prerequisite property will work fine but ones that do, will not. For example this works:
unit-test-ui:
    description: Run unit tests for UI
    run:
      container: build-env
      command: scripts/unit-test.sh

But this does not:
unit-test:
    description: Run all unit tests for both UI and BFF
    prerequisites:
      - unit-test-backend
      - unit-test-ui
    run:
      container: build-env
      command: "true"

Thanks for the bug report @Sami5 - I've fixed this issue and there should be a new release out shortly.

(also, GitHub tells me this is your first ever issue on GitHub... congratulations 🎉)

(also, GitHub tells me this is your first ever issue on GitHub... congratulations 🎉)

Haha yes it is! Thanks! :) 👍


⚠ Bofrost.es has degraded performance

In f497625, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1433 ms

Resolved: Bofrost.es performance has improved in 55e1ef0 after 6 minutes.


Image broken on Ev_charge_point

Invalid flash data has been used. No issue.

The flash file "BT81X_Flash.bin" was written into EVE's connected flash from address 4096 as a mistake, so EvChargePoint program cannot display image preperly.
Corrected flash data by written "BT81X_Flash.bin" to EVE's connected flash from address 0.


Vendor Splitting Working except ngtools not creating bundles

@MarkPieszak I have vendor splitting working with awesome-typescript right now. It works great and REALLY speeds up dev and even production deploys.
The problem is that @ngtools doesn't create the lazy loaded chunks. It expects them, it thinks they should be there but it never builds them.
I've simplified everything as well and would love to do a pull request to help you out with this if I can get this one issue resolved.
(I'm trying to attach my updated webpack files)

(Rename this with .zip extension)
Webpack.pdf
Basically these go in the root and there is nothing else required.
I have put the ts.xxx.config files in a Typescript folder instead of the Client folder but otherwise they're identical.  I also have a vendor.ts where you put all of your imports (standard ES 6 imports). It assumes that they're in the root too. Here's a sample of part of mine:
import '@angular/animations';
import '@angular/common';
import '@angular/core';
import '@angular/forms';
import '@angular/http';
import '@angular/platform-browser';
import '@angular/platform-browser-dynamic';
import '@angular/platform-server';
import '@angular/router';
import '@ng-bootstrap/ng-bootstrap';
import '@ngx-translate/core';
import '@ngx-translate/http-loader';
import 'event-source-polyfill'; //Not sure if this is really needed
import 'rxjs';
import 'zone.js';
import 'bootstrap/dist/css/bootstrap.min.css';
Once in you can use these two commands:
webpack --config webpack.config.vendor.js
webpack --progress --color
The first you run  to generate the vendor stuff, the second to generate the rest of the stuff.
I've also got updates to the .csproj that automatically runs npm install, and each of those on first build as part of the build process and updated Package.json to have the right calls in it to do it as well with nice commands.
You'll see my commented out rules for @ngtools. If you comment out the awesome-typescript and put in @ngtools and the aotplugin lines again you can see the behavior if you have a lazy loaded section in your routes. (make sure you clear your dist folders before running it)
Finally, I think it's already doing it but I can't be sure. I would really love to have the vendor.js file generated in production then scan the main-xxx.js and all chunks and then strip out everything that isn't used in the vendor.js file for the actual site as it could really cut a lot of crap out.
It may be doing this but if not I'd like to figure out how to get it to work (roll up or something?)  I don't have anything that needs to be tree shaked in my main site, but the vendor file sure has a ton of crap that isn't needed.
Also with AOT I think a bunch of vendor stuff from angular probably aren't needed so we can remove those too.
Didn't want to waste time porting it to your repo code unless this is useful and you haven't got a better way, but if that works, I will happily do so once this bug is fixed!

Looks like this is related to this bug: https://github.com/angular/angular-cli/issues/4431

HI @JohnGalt1717! Great work in getting as far as you have.  Am excited at the prospect of speeding up the dev builds because it is really hindering  development and deployment on my side. I tried doing what you suggested above,. Initially I had errors due to angular-router-loader and source-map-loader not  being installed so  installed them. Then i got a errors specific to my environment  (bootstrap-sass) which I manged to solve, but finally hit a brick wall with external "vendor_…":1 Uncaught ReferenceError: vendor_........ is not defined errors.  I was a bit unclear as to which imports to include in vendor.ts , so did a bit of trial and error there, but to no avail.
Any pearls of wisdom you might be able to offer would be greatly appreciated!

Hey @JohnGalt1717 I'll take a look, thanks for looking into this one!
What were the problems you ran into so far?
Ngtools is able to handle creating lazy loaded routes etc, not sure why that issue came up.

The only issue is ngtools not generating the bundles if WebDll plugin is used. Otherwise it works fine as is.
I'd rather have ngtools/webpack create css files for the app and then one for each bundle as well automatically and have it watch for changes to the source for HMR though instead of it being embedded in the .js files that are generated. The reason why is that by having the root app.css file generated and loaded you won't get white/bad looking pages while it's loading content from the .js file. But this is perfect world.
So if you can point me to where I'm going wrong with ng/tools in my config files I posted and look at the bug that I linked to and see if it's related and how to fix it, then with that one fix in this could be done and working.

I have the same problem.
Probably related to https://github.com/angular/angular-cli/issues/4431
But I don't have workaround.
If I remove ContextReplacementPlugin, things don't work.

hahaha. If it wasn't so frustrating I'd laugh.
So I created a production webpack file that uses ngtools and doesn't vendor split for production and just put a conditional in my cshtml file for the vendor.js and .css load..
Except no matter how large I set the memory size for node it fails as soon as it hits 2 gig of memory and won't compile as a result (and it takes > 20 minutes to do so)  There's literally 1000s of complaints about this online and dozens of bugs under angular about running out of memory.
So STILL no AOT and tree shaking is still BAD compared to rollup (even on small tests where It doesn't run out of memory)
Why is it so hard to create a linker and compiler for javascript when MS has it down to 50x the size of code will compile in less than 20 seconds and even a rebuild is maybe a minute?

Do you have a private repo or something I could look at to see why yours is taking so long?

What's your email? I'll send you a zip of it (or we're on vsts for our git repo)

<EMAIL_ADDRESS>@JohnGalt1717

HI @MarkPieszak  did you have a chance to look at the code I sent?

@MarkPieszak any news about vendor split? I'm not sure where to ask but is it still not possible to split libs?!

@consigliory @MarkPieszak  I believe I'm right in saying that Vendor split can't work with AOT until such time as @ngtools supports dllplugin etc. There are a ton of tickets about this.
And then of course there are all of the java heap memory issues that are plaguing webpack and Angular with @ngtools as well that need to be fixed before all of this works reliably.  Hopefully these will be actually worked on and addressed. Sadly the bugs in the @angular repos don't seem to be being worked on even though this is major blocker stuff.

Spoke with a few of the guys, and they're aware of the webpack & heap memory AoT issues that can arise from larger apps, so I'm hoping in v5 that's coming soon some of them might be better. AoT speed is supposed to be increased! I'll try to find out more about other stuff.
@consigliory I have an hour right now so I'm trying to port some stuff over, I'll get back to you. Not sure I'll be able to get it in that quickly.

Thanks for all of your hard work @MarkPieszak !


Add frost effect to iPad control buttons.

iPad keyboards have a frost effect that iPhone keyboards don't have.
Implement this in KeyboardKit as well:

This effects looks so 🆒

Yeah, I hope that it will be easy to implement :)

This will be an iOS 15-specific feature, using the new material apis.


skicit-image ImportError: dlopen: cannot load any more object with static TLS

Traceback (most recent call last):
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/skimage/init.py", line 119, in
from ._shared import geometry
ImportError: dlopen: cannot load any more object with static TLS
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "train_models.py", line 6, in
from seg_models.SegmentationModels import SegmentationModel
File "../seg_models/SegmentationModels.py", line 1, in
import segmentation_models as sm
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/segmentation_models/init.py", line 101, in
set_framework(other)
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/segmentation_models/init.py", line 71, in set_framework
import efficientnet.tfkeras  # init custom objects
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/efficientnet/tfkeras.py", line 4, in
from .preprocessing import center_crop_and_resize
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/efficientnet/preprocessing.py", line 16, in
from skimage.transform import resize
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/skimage/init.py", line 122, in
_raise_build_error(e)
File "/home/djh_19/anaconda3/envs/test/lib/python3.6/site-packages/skimage/init.py", line 102, in _raise_build_error
%s""" % (e, msg))
ImportError: dlopen: cannot load any more object with static TLS
It seems that scikit-image has not been built correctly.
I met this bug today. And the reason is scipy==1.4.1. When scipy==1.3.2, this bug will disappear.

emmm，this bug also exists when scipy==1.3.2


Enable email plugin for build failures.

Secrets are available as environment variables in push/tag/deploy
events. With Drone v0.5, it became no longer necessary to do
environment: YY=$$YY.

Wooo hoo! It works 😄


Fix document errors

Translate errors fix, and add Chinese(Traditional) link in README file.

Thank you...!

Thank you...!


renderForeground isn't displayed (RN 0.18.1)

Hello @jaysoo!
I'm trying to use your component for a RN project. I was going crazy with everything working BUT the renderForeground method, so I simply tried copy/pasting your example with listView, and this is what I've got :

I haven't changed anything in your code. My best guess is that it's related to 0.18.1, since the code is unaltered.
Could you please enlighten me on this issue?
Have a great day

Thanks for the report, that is rather odd. Will check it out today.

Thank you very much @jaysoo, let me know if you have the same result!

@JonathanWi So I ran the example with RN 0.18.1, and it does render for me. Can you possibly link me to a sample repo where this is not working?

Thanks for trying to reproduce this bug. I'm out of office for now, but I'll try to get you this ASAP. You can close this in the meantime, and if something comes up, I'll let you know

Sounds good. :)


fix: use sufficient computational effort for password hash

This PR modifies the underlying algorithm used in the hash function to use sufficient computational effort.
Changes

refactor to use argon2 instead of sha256
add unit tests for hash and helper functions

Screenshots
Using a hashed-password (sha256)
https://user-images.githubusercontent.com/3806031/120695344-25804580-c460-11eb-9b7b-b948ebd3eda0.mov
Using a regular password (not hashed)
https://user-images.githubusercontent.com/3806031/120695764-9aec1600-c460-11eb-82e5-d428f7d32743.mov
Using a hashed-password (argon2)
https://user-images.githubusercontent.com/3806031/120861593-6483c900-c53c-11eb-904a-10016f8449b5.mov
Checklist

[x] tested locally
[x] added new dependencies
[x] added tests
[x] updated CHANGELOG.md
[x] ensure we still support hashed_password in the config which is hashed with sha256

Fixes #3381
Follow-up: https://github.com/cdr/code-server/issues/3432
Notes
Here is how authentication works in code-server:

  Link to Excalidraw

@oxy do you mind taking a look at this today or tomorrow?

Notes:

merge in bcrypt (this PR)
write tests for router (testing one level up from hashing) - make an issue
add token exchange -> make an issue
rewrite other parts if needed? -> add to 3

@oxy do you mind taking a look at the security warnings/errors flagged by CodeQL? I know we plan to fix most of these in a follow-up PR but want to make sure my code isn't introducing anything you would flag: https://github.com/cdr/code-server/pull/3422/checks?check_run_id=2749748946

@jsjoeio impressed to see how good you test code-server. Just awesome, man!

Thanks so much for digging into this, finding the root and guiding me towards the solution! ♥

*Writes a bunch of new tests
Them:
Me:

Express handlers are typed to be far too permissive towards the data it passes to the consumer. Most sources of data are typed as any which is problematic in most cases. Two solutions of note are:

Adopt a different web framework. Express is quite old and hasn't seen any new light in a while. Its typings are suboptimal.
Stick with Express and do either:
Override the definitions on the types that are too permissive. This can be achieved through a few different ways:
1. Override as much of the types as possible through the mechanisms with which TypeScript allows
2. Patch the offending types (e.g., git diff, etc.)
Attach some handlers to attempt to sanitize the data being sent through the route handlers down the line. This is where prototype pollution can be taken care of (Express does not implement anything related to this).
Be extra careful. This is not ideal because it requires vigilance on the part of everyone who interacts with the codebase in a way that touches the problematic areas.

Express handlers are typed to be far too permissive towards the data it passes to the consumer. Most sources of data are typed as any which is problematic in most cases. Two solutions of note are:

Adopt a different web framework. Express is quite old and hasn't seen any new light in a while. Its typings are suboptimal.
Stick with Express and do either:
Override the definitions on the types that are too permissive. This can be achieved through a few different ways:

Override as much of the types as possible through the mechanisms with which TypeScript allows
Patch the offending types (e.g., git diff, etc.)

Attach some handlers to attempt to sanitize the data being sent through the route handlers down the line. This is where prototype pollution can be taken care of (Express does not implement anything related to this).
Be extra careful. This is not ideal because it requires vigilance on the part of everyone who interacts with the codebase in a way that touches the problematic areas.

Those are really good points and definitely something we want to consider moving forward. I made a note about this in the Roadmap. I don't think we're in a place to switch away from Express but something we can consider when we look at cleaning up and refactoring things down the road.
For now, I've added a basic function to sanitize both the password and cookie before they're used by the password hashing functions. Thanks a bunch for your input and sharing your knowledge! We really appreciate it.

Fixed security vulnerabilities in separate PR: https://github.com/cdr/code-server/pull/3563

Thanks for sticking through this

Thanks for sharing your security knowledge with me! I feel like I'm slowing becoming more mindful of it thanks to you 😂


Registered chat variables should not be hidden when using a chat agent

Type: Bug
I am creating a new extension which adds new variables to GitHub Copilot Chat. This variable provides additional information that would enhance the /fix command performance.
I can successfully use the variable #myVariable and ask questions, but I cannot use this variable in combination with /fix command. The reason is that /fix automatically adds the @workspace agent, and variables become then filtered out (e.g. only default variables are shown, not added ones in extensions).
I think that it makes sense to filter commands depending on which agent is called, but I don't think it is relevant to filter variables too. Variables should be exposed regardless which agent is used... or at least a new parameter could be added when registering a variable to mark a variable as global (meaning that it should be useable regardless which agent is used).
Here is how the variable has been defined:
vscode.chat.registerChatVariableResolver(
    'git-refact-commits-var',
    'gitRefactCommits',
    'Last 10 refactoring (i.e. containing refact) commits from past history',
    '',
    true,
    { resolve: getRefactCommits }),

Image below shows that only default variables are shown, not added ones:

Extension version: 0.16.2024052702
VS Code version: Code - Insiders 1.90.0-insider (b9fa819edd893c590b28f3a09b75a81513e12fb9, 2024-05-24T17:21:19.268Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i9-13950HX (32 x 2419)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.61GB (50.73GB free)

Process Argv
C:\Temp\vscode-extension-samples\chat-sample --crash-reporter-id ae3d8212-7e85-4d29-be44-058116e4fe47

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
vsaa593:30376534
py29gd2263:31024238
c4g48928:30535728
a9j8j154:30646983
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30926734
pythonmypyd1:30859725
pythoncet0:30859736
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
pythonait:30973460
showvideot:31016890
chatpanelt:31014475
bdiig495:31013172
a69g1124:31018687
dvdeprecation:31040973
pythonprt:31036556
dwnewjupytercf:31046870
nb_pri_only:31057983
26j00206:31048877

We do allow other participants to use variables, but this is probably because you passed true to isSlow, you shouldn't use that for your variable.
Thanks for trying out the variables API, make sure you follow https://github.com/microsoft/vscode/issues/212995 for updates. Would love to hear more about how you're using it.


Create Sphinx documentation for project

Migrated content from markdown files to rst files. Created a sphinx documentation project that has a similar structure as other Jupyter docs.
cc/ @jdfreder @ellisonbg

Thanks @willingc !

:+1:

Thanks @jdfreder and @SylvainCorlay

For the docs to fully deploy to ReadTheDocs, I will need sufficient privileges:

on this repo for building docs on pushed changes to set hooks to update the build on RTD, and
on RTD as a maintainer so I can update doc build settings.
Thanks.
Not sure who is the keeper of such powers :8ball: so I will ping @carreau @SylvainCorlay @jdfreder @minrk

@Carreau owns the RTD page, @Carreau can you add @willingc @SylvainCorlay and I?
@willingc I added you as a contributor to this repo.

Thanks @jdfreder :cake:


Update makefile to add go mod tidy

Execute go mod tidy every time before building the source.

LGTM


Progress Bar Component added

Happy coding!

Hey @kvnal , Great work! Thank you for contributing 🎉 I accidentally approved it, but please make a PR to my feature branch and not master! The code will break

Branched changed!


How to test the app in local?

Hi team,
Is there any way we can test our changes in local without deploying the changes to GAS.
Thanks

Hi,
Today there is no way to test locally, you can create another GAS project just to upload test versions if you prefer.
That's good question because I'm working on an feature to allow it to run in localhost, but it's not finished yet.
Thanks

Thank you for the reply. Looking forward for that feature.

Helo @ehduardu, is there any ETA on the feature you are working for local testing? Thanks

Hi, @naren9459. Not really, because I'm working in my spare time.


Why does file loader add an "_" to output directory paths?

I've tried searching everywhere I can but cannot figure out why File-loader, or more specifically the loader-utils, keep adding an "_" directory to which it adds all my images loaded from CSS. Why is this happening? Why can I not use the same path from my /src directory without adding an "_" directory at the top?

I think this comes from [path] which is the location of your module relative to the provided context. And because you don't want output outside of your output directory ../ is replaced with _/.
You need to provide a better context to the loader to fix this.
https://github.com/webpack/loader-utils/blob/master/index.js#L196

[path] the path of the resource relative to the context query parameter or option.

Closing since this has been answered and is inactive.


Model the rattlesnake emitting sound

Perhaps I should make the rattlesnake emit 0-radius light, to model it's being heard from afar even in darkness (the hearing sense currently only works on very close enemies and there's no way to make some more noisy than others)?
Peritract said:
The key thing about rattlesnakes is the rattle - otherwise, there's nothing special about them compared to any other snake. They definitely need something to set them apart/warn the player.

Done on master. Now rattlesnakes have rattles that periodically make noise that other actors can hear with the fleshed out hearing sense: https://github.com/LambdaHack/LambdaHack/blob/d871d104da64adc1fa934465bd67a83c6585b3f3/GameDefinition/Content/ItemKindOrgan.hs#L341


docs: update getting started guide nodepool to use od

Fixes #N/A
Description
Updates the getting started guide nodepool to use od, to ensure users can test without having to worry about spot capacity
How was this change tested?
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.025%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
5512

Relevant Lines:
6639

💛  - Coveralls


fix(clerk-js): Fix neutral alpha shades generation

Description
This PR fixes an issue in generating alpha shades from a user-provided color.
Also, we should provide the correct 500 value for our neutral colors to create the scale we want.
So the default values should be:

For a light theme: 'hsla(0, 0%, 0%, 0.53)'
For a dark theme: 'hsla(0, 0%, 100%, 0.53)'

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/backend
[ ] @clerk/chrome-extension
[ ] @clerk/clerk-js
[ ] @clerk/clerk-expo
[ ] @clerk/fastify
[ ] gatsby-plugin-clerk
[ ] @clerk/localizations
[ ] @clerk/nextjs
[ ] @clerk/clerk-react
[ ] @clerk/remix
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/themes
[ ] @clerk/types
[ ] build/tooling/chore

Hey @octoper, your preview is currently deploying and should be available shortly.

Status
Preview
Updated (UTC)

:cook: Deploying...
Inspect
Feb 08, 2024 08:37 AM


Inverse-fft node update

updated studio with nodes from this pr: https://github.com/flojoy-io/nodes/pull/110

Didn't realize this had been automated, nice!


Effect Speed LED

I have paired my two strip led and ON/OFF command works.
The integration have found Effect Speed LED, but nothig happens if I change that value (0-100 range).
The integration did not founf the RGB settings.

Use debug log to see more information of posible errors.
In configuration.yaml:
logger:
  default: info
  logs:
    custom_components.elkbledom: debug

Post log results.


Fix Benchmark testMain test

All of the queries are returning "does not pass static validation", even though the test itself passes

TexQueryGenerator also does this

can you give information on the backgroud/context

addQvarMap did not get disabled in Benchmark, so the queries were not validating because of the XQuery version issue with XQJ
I just disabled it and it works.
However, i still think this is an issue that Client methods fail silently like this and let the integration test pass. Moritz what do you think of this? Should the test pass even though none of the queries validated correctly?

no it should fail


Nested dialog removes 'ngneat-dialog-hidden' too early from body

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
When nested dialog is closed, then class 'ngneat-dialog-hidden' is removed from body.
Expected behavior
Class 'ngneat-dialog-hidden' should removed from body after last dialog was closed.

Minimal reproduction of the problem with instructions
Open two (nested) dialogs, close the inner and review class list of 'body'.
What is the motivation / use case for changing the behavior?

Environment

Angular version: 9.1.11 - 11.2.0

Browser:
- [x] Chrome (desktop) version XX
- [x] Chrome (Android) version XX
- [x] Chrome (iOS) version XX
- [x] Firefox version XX
- [x] Safari (desktop) version XX
- [x] Safari (iOS) version XX
- [x] IE version XX
- [x] Edge version XX

For Tooling issues:
- Node version: v14.15.4
- Platform:  Mac, Linux, Windows

Others:

I've seen my auto-format settings adjusted a lot of white-spaces. Not sure if this is a problem and needs to be fixed. Let me now if I should do anything further.


esbuild idea

I have made some updates to how I handle js building and thought I would share in case it interests you.
There is a breaking change with this code - "transpile" true   >  "babel". You could change that back if want to run with this.
The main motivation is not having to use a dict if your just running with the default partial settings of lowering to es6 with esbuild.
https://caniuse.com/?search=es6

Thanks! I'll have a closer look later


[GRAPHX][EXAMPLES] move graphx test data directory and update graphx document

What changes were proposed in this pull request?
There are two test data files used for graphx examples existing in directory "graphx/data"
I move it into "data/" directory because the "graphx" directory is used for code files and other test data files (such as mllib, streaming test data) are all in there.
I also update the graphx document where reference the data files which I move place.
How was this patch tested?
N/A

Test build #61603 has started for PR 14010 at commit 9fc83f6.

LGTM

Test build #61603 has finished for PR 14010 at commit 9fc83f6.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61603/
Test FAILed.

Jenkins retest this please.

Jenkins, test this please.

Test build #61613 has started for PR 14010 at commit 9fc83f6.

Test build #61614 has started for PR 14010 at commit 9fc83f6.

Test build #61613 has finished for PR 14010 at commit 9fc83f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61613/
Test PASSed.

Merged build finished. Test PASSed.

Test build #61614 has finished for PR 14010 at commit 9fc83f6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61614/
Test PASSed.

Merged build finished. Test PASSed.

@srowen Done.

Merged to master/2.0


Corral reports error from primary compilation step

On Windows (git bash).
ponyc -o build/release semver works fine and results in a 0 exit code
corral run -- ponyc -o build/release semver reports...
run: ponyc -o build/release semver
  exit: Exited(-1)
  out:
  err:
make: *** [build/release/semver.exe] Error -1

$ ponyc -v
0.35.0 [release]
Compiled with: LLVM 9.0.1 -- MSVC-19.25.28612.0-x64 1925

$ corral version
version: 0.3.4-e152293

@btab can you please report your PATH as being present in your git bash? Can you also report the path environment variable you can get from windows?

also can you try the same on cmd or powershell?

Seems to be fixed by https://github.com/ponylang/ponyc/pull/3559

For completeness, @mfelsche...
From both Powershell and cmd I get the exact same result.
The PATH has a bunch of personal stuff on it so I won't share it but it's the same in cmd and git bash aside from the relevant path substitution. So, for example, in git bash...
/c/Users/andre/src/corral/build/release

...vs, the following in cmd...
C:\Users\andre\src\corral\build\release

@btab Yeah, this is a bug recently introduced in the pony standard library that impacts only on Windows.

Indeed. I'll pull it down once the PR succeeds and verify the fix locally as well.

Fix is now on ponyc master.

Just pulled down the latest ponyc and I'm still seeing the same problem in git bash and in Powershell...
$ ponyc -v
0.35.1-3f2bf01a [release]
Compiled with: LLVM 9.0.1 -- MSVC-19.24.28314.0-x64 1924

$ ponyc -o build/release semver
[exit code 0 - successful compile]

$ corral run -- echo 1
run: echo 1
  exit: Exited(0)
  out: 1
  err:

$ corral run -- ponyc -o build/release semver
run: ponyc -o build/release semver
  exit: Exited(-1)
  out:
  err:

Latest corral + latest ponyc did fix the issue for me.


Update on the sandbox

Ask by @deltakosh
The hover on the combo was removed. Now you have to click in order to open/close the liste of animation groups. Also when you select an item, it closes the list.
The hover effect that change the items background color is still there.
The slider clickable zone is now bigger.
A wrong html balise was deleted.

Will try it asap locally before merging

Thx, closing the PR as I added a small change before merging


Update LIcense File

Update License file with USC disclaimers and current year.

Updated LICENSE file to match UCVM and OpenSHA, as per commit 009e8318d4f88fe2cf3abf477a81f6a2b9a81ca6


Please add events for component (Tag and Checkbox) !

What is expected?

Provide monitor events to realize more complex interactions when the label is selected! Thanks !

and more , please add valueProperty & labelProperty for tag component , like select component !


@wrgoldstein => Scala common enrich impressions

As you can tell, nothing exciting here. Unfort, a special case for impressions to split up a field that will be comma separated, this is the format that we expect if we send an array from the JS tracker. The spec for this is integrated to my other PR that uses this library in scala-kinesis-enrich: https://github.com/artsy/snowplow/pull/6

/shrug


End the process and delete files

Can you add an end process and forcibly delete the file where the process is located? Thank you!

Did u mean delete the file released by the program?

Did u mean delete the file released by the program?

Delete the file of the terminated process. Prevent it from being resurrected again

No, this is not a good idea, you can use the -r parameter for this purpose, and the -t parameter to set a smaller refresh time.

I will close this issue, and if you feel that the benefits outweigh the disadvantages, you can open another issue.


Use pinboard crate to avoid blocking Sinks during playback

Calls to Sink and SpatialSink's set methods risk causing delays to the playback thread. This pull requests attempts to fix that by replacing the mutex with a lock-free Pinboard from the pinboard crate.

Sink is set to be deprecated then removed. In its place we will get a new Source called Player. I will definitely use some of ideas from this PR for that. Thank you for your work on this!
Since Sink will be removed we are no longer working on it, because of that ill have to close this PR, sorry.


Antonin leaving the advisory committee

As mentioned on the forum, I am in the process of reducing my involvement in OpenRefine. Leaving the advisory committee is one step of that process. I intend to continue development and other activities until the end of March, but I don't need to be in the committee for that.
Perhaps it would be worth thinking about onboarding new members on the advisory committee.
Documents to update

[x] Update Project Advisory Committee Table and approval by the Advisory Committee Chair
[ ] Blogpost announcement
[ ] Update the Governance.md file
[ ] Removing from OpenRefine - CS&S Shared Drive
[ ] Remove from<EMAIL_ADDRESS>

@wetneb  I updated the description with the list of document we need to update.

@Ainali @ej2432 I do not have access to the Advisory Committee mailing list. One of you should be able to remove Antonin from the discussion list.

I've done it already.

Out of curiosity, are Jan and I able to make edits to the list as well?
On Fri, Nov 29, 2024 at 1:50 AM Antonin Delpeuch @.***>
wrote:

I've done it already.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/7020#issuecomment-2507189379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APMRQTEIF4TRY6UHBHWBPE32DAFCVAVCNFSM6AAAAABSVUVB2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBXGE4DSMZXHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Esther Marie Jackson | @.***
Scholarly Communication Technologies Librarian
Columbia University Libraries
Meet with me https://calendar.app.google/EmJ9iW2Q3JasmSuR8

I don't remember who is left as an admin in the corresponding Google group, sorry. I think someone from CS&S is listed there (but they might have left).

Thanks - looks like Jan is an admin/owner.
On Tue, Dec 3, 2024 at 8:07 AM Antonin Delpeuch @.***>
wrote:

Merged #7020 https://github.com/OpenRefine/OpenRefine/pull/7020 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/7020#event-15509517881,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APMRQTEZUAVQQWETLQAN6KL2DWUKLAVCNFSM6AAAAABSVUVB2CVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGUYDSNJRG44DQMI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Esther Marie Jackson | @.***
Scholarly Communication Technologies Librarian
Columbia University Libraries
Meet with me https://calendar.app.google/EmJ9iW2Q3JasmSuR8


How I can change date format in date-filter?

How I can change date format in date-filter?
Now in date-filter we see this date format: mm/dd/yyyy.

But I need change it to: dd.mm.yyyy. Can I do this?

We dont support this feature, but I think it should be support. I'll try to add this feature in near future but I'm busy recently, so PR is welcome.

AllenFang, рhank you very much. I'll wait for the update.

@clairekh, firstly, I cant change the format, because I use <input type='date'/>
according http://stackoverflow.com/questions/7372038/is-there-any-way-to-change-input-type-date-format, the input date cant change the format, and the format seems like depend on the user’s system settings’ date format.

Any updates on this?

Hello,
same question as the above: Can we still not switch to the dd/mm/yyyy format?
Thank you.

Any updates?

Any updates?


[#379] [Bug] Fix: Bundle Identifier on each build configurations

#379
What happened

Apply staging bundle id for Debug Staging, Release Staging and production bundle id for Debug Production, Release Production by replacing {BUNDLE_ID_PRODUCTION} with ${PRODUCT_BUNDLE_IDENTIFIER} in Project.swift file.

Insight

n/a

Proof Of Work
https://user-images.githubusercontent.com/25881847/202951470-dd7d823e-790c-4e9b-987c-9422995c7554.mov

@nkhanh44 Something wrong with CD, please take a look at it 🙏

@vnntsu Thanks for reminding me. It works now 😄 .


Add Nerd Fonts support

See #17
All Nerd Fonts glyphs were parsed from the official cheat sheet and the .tsv look perfect and works.
I do still have an issue with some glyphs from "Material Design Icons" where some glyphs cause the whole line to be printed in right-to-left order, but that could be an issue with my setup/fonts. If anyone could try this out and see if it works on their system, that would be great :)

@iFreilicht  I've tagged you in a reply to a related pull request about how it would be best to handle "extra" databases like these

Regarding the RTL renderings, I had a feeling that after spot-checking a couple of the entries checks out - the unicode codepoints that you and I are seeing rendered RTL are from scripts like Arabic that are supposed to render RTL.
That is, the glyphs are being overridden with your Nerd Font font, but Rofi still recognizes these are RTL codepoints and goes that way.
I have no idea off the top of my head whether that is something that can be configured away via Rofi's theming or options

Ah, that makes perfect sense! I'll have a look and see whether this can be remedied. I'll reply on the other ticket regarding additional databases.

Also an option if splatmoji.config could refer to different urls, for the tsv files.
eg:
external_gitmoji=https://raw.githubusercontent.com/ss1978/splatmoji/master/data/gitmoji.tsv

@ss1978 let's discuss this in #16 only. It's really confusing if the discussion is spread across multiple threads.

I updated the PR to support the new database format.


Minor: 'in the in the' => 'in the'

When creating a new PR, please do the following and delete this template text:

Reference the issue number if there is one:
Fixes #Issue_Number

The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.

Suggest reviewers if you know who should review the PR, using the GitHub Reviewers UI.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@jfoshee Thanks!


when i use demo of consumergroup got panic err

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama
Kafka
Go

1.33
2.12-2.3.1
go16

Configuration
What configuration values are you using for Sarama and Kafka?
func (consumer *Consumer) ConsumeClaim(session sarama.ConsumerGroupSession, claim sarama.ConsumerGroupClaim) error {
	// NOTE:
	// Do not move the code below to a goroutine.
	// The `ConsumeClaim` itself is called within a goroutine, see:
	// https://github.com/Shopify/sarama/blob/main/consumer_group.go#L27-L29
	for {
		select {
		case message := <-claim.Messages():
			log.Printf("Message claimed: value = %s, timestamp = %v, topic = %s", string(message.Value), message.Timestamp, message.Topic)
			session.MarkMessage(message, "")

		// Should return when `session.Context()` is done.
		// If not, will raise `ErrRebalanceInProgress` or `read tcp <ip>:<port>: i/o timeout` when kafka rebalance. see:
		// https://github.com/Shopify/sarama/issues/1192
		case <-session.Context().Done():
			return nil
		}
	}
}

Logs
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x60 pc=0x1016b77]

goroutine 307 [running]:
base.(*Consumer).ConsumeClaim(0xc0000c7290, 0x1714c28, 0xc000129500, 0x170cf78, 0xc002d8b7a0, 0xc, 0x1)
kafka.go:95 +0x1b7 # panic at log.Printf（same with demo）

Problem Description
this code may got nil with message? and then print cause panic
case message := <-claim.Messages():

When i got a nil message， i just continue( maybe the consumer is close, just return is ok?).

Sorry for the delay, this was fixed by https://github.com/IBM/sarama/pull/2479


[crypto] renaming NamespaceKeys -> namespace::Keys

WORK IN PROGRESS. DON'T REVIEW NOW
We use the following pattern for all variables currently in the crypto crate (with an example on public keys):
ed25519::Ed25519PublicKey

This PR renames all of these as:
ed25519::PublicKey

The rational is that:

from the point of view of the ed25519 library, a key is just a key
from the point of view of the caller, rust provides namespace just for this

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

see https://github.com/libra/libra/pull/3440


feat: Remote configuration support

Addresses https://github.com/aviator-co/av/issues/9

support ad hoc remote names
support multi-remote configurations
determines default remote as origin or first listed remote
add new SplitLines string utility
update deprecated ioutil import references
update dependencies

Appreciate the effort but unfortunately I'm going to close this. The PR is a bit too broad in scope and would take a bit of review in order to merge. Thank you for your time and energy, and we're definitely open to external contributions in the future!


Add color for TypeScript instanceof

Describe the bug
I have noticed that instanceof is not colored in typescript.
To Reproduce
if( x instanceof Y )

Screenshots

Fixed with v0.0.10. Thanks for reporting!


SQLite session closed early during unit tests - Leads to Cannot read properties of undefined (reading '$now')

Description of erroneous behaviour
In my CDS application, I am calling a REST API in the test that looks like the following. Note that the last method will exceed the runtime of the API and keeps writing to the DB while the API response is being sent.
  srv.on('createElements', () => {
    // Calling async method that persists elements, but do not use `await`
    this.repo.createElements(elements);
 }

This leads to an error when dynamically injecting createdAt timestamps into the DB records (using the annotation @cds.on.insert: $now in the data model definition.):
    TypeError: Cannot read properties of undefined (reading '$now')
      at node_modules/@cap-js/sqlite/lib/SQLiteService.js:21:61
      at SQLiteService._run (node_modules/@cap-js/sqlite/lib/SQLiteService.js:91:17)
          at async Promise.all (index 0)
      at SQLiteService.onINSERT (node_modules/@cap-js/db-service/lib/SQLService.js:153:31)
      at SQLiteService.onINSERT (node_modules/@cap-js/sqlite/lib/SQLiteService.js:246:14)
      at next (node_modules/@sap/cds/lib/srv/srv-dispatch.js:79:17)
      at next (node_modules/@sap/cds/lib/srv/srv-dispatch.js:79:17)
      at next (node_modules/@sap/cds/lib/srv/srv-dispatch.js:79:17)
      at SQLiteService.handle (node_modules/@sap/cds/lib/srv/srv-dispatch.js:77:10)

Is this expected behaviour? If so, what is a good way to cater for an API as we implemented it above? Thanks for your feedback.
Detailed steps to reproduce
Libraries used:

ladjs/supertest
@cap-js/sqlite 1.5.1
@sap/cds 7.6.3
@sap/cds-dk 7.6.1

Setup a service as above that returns early after calling an async method
Create a test that calls the API that is being served programatcally, using cds.serve(...)
Make sure to use SQLite as a database

Details about your project
SAP internal project.

Hi Benedikt,
you implemented a race condition in which the test teardown is faster than your dangling promise.
You should either await the async function for proper transaction mgmt inside the context of the incoming request or (if intended) use cds.spawn for initiating a background job.
Best,
Johannes

Okay, adding the await is what I did to fix the issue.
Thanks for providing some feedback.
Error handling and reporting could be improved, though. Since the message is misleading. It should rather be sth like "db session does not exist anymore".
Feel free to close the issue.


Option to redirect after item create/edit success to show page

If a resource has a show page, it may be more reasonable to redirect to show page after successful form submission.
Can we have:
an optional afterSuccess prop for Create and Show?
<Create {...props} afterSuccess="show">
or redirect to Show if it is available and redirect to List otherwise.
I can make a PR if either of them helps.

I think you should make a PR, and link it to here. And then, some will review your code.

It would be cool if it would be possible to redirect to another entity

@fzaninotto What do you think? Or any suggestions?

Now redirect can be automatic, it looks for the first available pages in a list.
And it can be configured manually, it accepts list, show, edit and custom url.
#692

Additional possible use case for this feature.
I have created a custom Menu to route 2 different list views List1 and List2 (with different filters) to the same resource.
There are Edit buttons on both list views.
On entering Edit from List2 and Saving, Edit is redirecting the page to the List1 while the expectation is to redirect to List2.
After going through the source code, I see that in the Saga for Update redirects to the basePath from Edit
Edit calculates basePath
    getBasePath() {
        const { location } = this.props;
        return location.pathname.split('/').slice(0, -1).join('/');
    }

I will probably have to write a custom List view at this point. Can the above enhancement also accomodate my use case please.

Yes, see the discussion going on in #692

Some progress with an example:

create a custom toolbar

import Toolbar from 'admin-on-rest/mui/form/Toolbar';
import { SaveButton } from 'admin-on-rest';
const PostCreateToolbar = props => <Toolbar {...props} >
    <SaveButton label="Save and show" redirect="show" />
    <SaveButton />
</Toolbar>;

add toolbar to post create

export const PostCreate = ({ ...props }) => (
    <Create {...props}>
        <SimpleForm toolbar={<PostCreateToolbar />} >
...

result

Each button redirects to different page.

Fixed by #692


grizzly: E2E distributed tests takes too long time

probably the image has to be built for every testcase.

locally, on average, one E2E test feature takes aboutn 12 seconds...
image is only built once, and it takes about 100 seconds.
look in grizzly-cli if there's anything that can be optimized.
one optimization has been done in grizzly, that being that test_example.py uses the image built by the fixture, and does not build and extra for just that test.

nothing obvious in grizzly-cli that can be optimized.
we can remove @pytest.mark.parameterized for any E2E tests though, to reduce the number of tests.


FR: yml for "only merge one at a time"

Basically, I'd like my default to be always --upto <bottom of stack>. Would be nice if this was available as an option in yml, as actually generating that command takes work.
Would be happy to work on/send a PR if that is something you'd be interested in.

how about a flag that does this:
git spr merge --count 1
git spr merge -c 1

What would also be great.

I'm ok with either or both, I personally prefer the flag approach for this case.
Pull request more than welcome!


Bump client-registration to v0.15

Purpose
Bump client-registration to v0.15

Fix https://github.com/wso2/analytics-apim/issues/679


Error: parent directory is world writable but not sticky

It report a bug with following,please help me solve it, thank you.
$ brew install caskroom/cask/brew-cask
==> Installing brew-cask from caskroom/homebrew-cask
==> Cloning https://github.com/caskroom/homebrew-cask.git
Updating /Library/Caches/Homebrew/brew-cask--git
==> Checking out tag v0.53.3
Error: parent directory is world writable but not sticky
Please report this bug:
https://git.io/brew-troubleshooting
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/tmpdir.rb:92:in mktmpdir' /usr/local/Library/Homebrew/utils/fork.rb:6:in safe_fork'
/usr/local/Library/Homebrew/formula_installer.rb:481:in build' /usr/local/Library/Homebrew/formula_installer.rb:184:in install'
/usr/local/Library/Homebrew/cmd/install.rb:165:in install_formula' /usr/local/Library/Homebrew/cmd/install.rb:83:in block in install'
/usr/local/Library/Homebrew/cmd/install.rb:83:in each' /usr/local/Library/Homebrew/cmd/install.rb:83:in install'
/usr/local/Library/brew.rb:140:in `'

There's now a brew doctor check for this. Essentially: Use Repair Disk Permissions in the Disk Utility application.

Thank you, @DomT4. Did that solve it, @howenkakao?


Add Hepatitis-C support to test card

FRONTEND PULL REQUEST
Related Issue

Resolves #7433

Changes Proposed

Adds support for Hepatitis-C to test card
AOE questions become required for positive results
Creates reusable AOE components like a common PregnancyAoe question component or GenderOfSexualPartnersAoe question component, so that future diseases (and previous diseases if we'd like) can have their own disease specific AOE forms more easily composed with existing AOE questions

Additional Information

Follow up PR will add sample data once accurate data can be confirmed
Follow up PR will clean up how we use the SensitiveTopicsTooltipModal so it can be configured on the disease AOE form level
Should be merged after #8223
[x] Approved by design
[ ] Approved by product (pending)

Testing

Deployed on dev4

Screenshots / Demos

FYI noticing an existing visual bug with the error states for the checkboxes component for syphilis and now for hep c since it uses the same design. Open to either fixing the CSS bug in this PR or creating a separate PR or ticket for it if that is preferred


How to generate array with referenceByClass?

Hi,
I'm a newbie on swagger, I want to generate the json code like this:
            "responses": {
                "200": {
                    "description": "successful operation",
                    "schema": {
                        "type": "array",
                        "items": {
                            "$ref": "#/definitions/Pet"
                        }
                    }
                },
            },

I try to this 'data' => Kobe::referenceByClass(Pet::class)->toArray()  but isn't work.
What should I do?
Thanks!

Hi, dianxin11, you can try this:
Kobe::makeArray()->setReferenceAsItems(Kobe::referenceByClass(Pet::class))

Let me know if you solve this.

Thanks a lot, it worked!


refactor date formatting to use std.fmt.Formatter

Hello!
This PR implements the work discussed in https://github.com/Luukdegram/apple_pie/pull/83.
I don't have the knowledge of the http spec to implement https://github.com/Luukdegram/apple_pie/issues/34, but it should be fairly easy to do so now.
Let me know if these changes are what you expected. :)

Thank you for taking up the task. This is great and ready to be merged as-is. Don't worry about #34, I can implement that some other day :)


De-emphasize Facet in other documentation (e.g. IonReader)

In recent usability studies, new users were confused by the documentation references to Facets, and tried to find the right Facet for an instance of IonReader to solve their problems even when Facets were completely irrelevant.

Personally, I find Facets, even when used as intended to be confusing.
I understand the goal is to encapsulate the implementation, but the indirection is difficult. Specifically, it's really hard to "see" what facets each object provides from the code.
I would suggest an entirely different mechanism for a future API.


[5.8] Try to resolve

Try to resolve issue https://github.com/laravel/lumen-framework/issues/886

unit test failed, seems this will need to resolve by project level


remove auto generated code from repo

These codes are auto generated. We are removing from the repository since there's no need to keep it there.
Files are tagged with:
// Code generated by go-swagger; DO NOT EDIT.

👏 (not tested)

Sorry @gregLibert ... I haven't got the point here!?


/test/phonenumbers/phonenumberutil_test.cc some compilers need ULL values

googlei18n/libphonenumber/master/cpp/test/phonenumbers/phonenumberutil_test.cc
contains a few instances of numbers which should be set to ULL for some older compilers
In particular, I used Macports port libphonenumber-cpp on Mac OSX 10.5.8 darwin 9.8.0 386
apparently older gcc needs the LONG values to be marked explicitly ULL
I created a patch to demonstrate what was changed.
in order to complete compiling
patch-src-phonenumberutil_test.cc.diff.txt

This is fixed in release 7.7.1.


remake of edit profile

added a atom to handle profile form data on editing
added a new atom to handle changes to currentuser that propogate to firebase
added editing profile functionality

known bug
firebase users document has a createdAtField, for some reason it changes this to a string instead of timestamp.
also has id but that is less of an issue


Random send to beneficiary when we talk about contributor?

https://github.com/kleros/openiico-contract/blob/master/contracts/IICO.sol#L218
beneficiary.send(localSumAcceptedContrib); // Use send in order to not block if the contributor's fallback reverts.
I think it should be part of redeem function anyway.
https://github.com/kleros/openiico-contract/blob/master/contracts/IICO.sol#L232-L243
        if (bid.maxVal > cutOffBid.maxVal || (bid.maxVal == cutOffBid.maxVal && _bidID >= cutOffBidID)) // Give tokens if the bid is accepted.
            token.transfer(bid.contributor, (tokensForSale * (bid.contrib + (bid.contrib * bid.bonus) / BONUS_DIVISOR)) / sumAcceptedVirtualContrib);
        else                                                                                // Reimburse ETH otherwise.
            bid.contributor.transfer(bid.contrib);

In that specific instance we can send both tokens and overflow ETH.
PS. I really like your style of coding, long lines means more code visible on the screen.

Copy-paste comment error, now I understand better. Thanks!

Yeah, fixed in https://github.com/kleros/openiico-contract/commit/f1a81cb0f72ed7411b6ad6487d4cc1255ad0641e
The compiler gives warning when using send.
In the past, the warning made sense due to a callstack attack (which could force the send to fail), but this one is not possible anymore due to how gas works now.


no such file:home/luohm/.pyrouge/settings.ini

when i run this command python train.py -mode validate -bert_data_path ../bert_data/cnndm -model_path ../models/bert_classifier  -visible_gpus 0  -gpu_ranks 0 -batch_size 30000  -log_file ../logs/cnndm.log  -result_path ../results/cnndm -test_all -block_trigram true,it raises th error,but i don't know why, can you help me?

before install pyrouge you must install the rouge witch is very difficult to install. so you should pass the param '-report_rouge False' to ignore rouge.

when i run this command python train.py -mode validate -bert_data_path ../bert_data/cnndm -model_path ../models/bert_classifier -visible_gpus 0 -gpu_ranks 0 -batch_size 30000 -log_file ../logs/cnndm.log -result_path ../results/cnndm -test_all -block_trigram true,it raises th error,but i don't know why, can you help me?

to install pyrouge in unbuntu. you can follow this page. Link.


Implement support for multiple exported interfaces in wasmlink.

Currently wasmlink::Module only supports reading an interface from a
custom section (unused) or from a path to a witx file.
In preparation for some upcoming work, this commit implements creating a
wasmlink::Module that exports more than one witx2::Interface.
Module::new now checks that given interfaces are not in conflict (e.g.
have a function/resource of the same name) and also verify that the
module actually does export those interfaces.
Also fixed some clippy lints in wasmlink.

cc: @radu-matei


mc_pos_control: set yaw based on attitude if local-position topic not…

follow up of https://github.com/PX4/Firmware/pull/10683
It is to note that this is a HACK and attitude subscription is not meant to be part of the mc-pos-controller.
A much better solution would be to remove Stabilized from the mc_pos_control and move it into the attitude-controller. This also requires to introduce a new topic that contains thrust-setpoint and yaw-septoint fields which would be sent from the mc-pos-ctrl and received in the attitude-ctrl.
The conversion from thrust- and yaw-setpoint to attitude would be done in the attitude controller.

How hard is it to move it to the attitude controller all at once instead of these incremental hacks?

How hard is it to move it to the attitude controller all at once instead of these incremental hacks?

@Stifael do you have time for that?

This also requires to introduce a new topic that contains thrust-setpoint and yaw-septoint fields which would be sent from the mc-pos-ctrl and received in the attitude-ctrl.

Why does it require an interface change?

@Stifael do you have time for that?

Not now, that is why added that "hack".

Why does it require an interface change?

It depends what you want. I prefer to have the mapping from thrust-to-attitude (https://github.com/PX4/Firmware/blob/master/src/modules/mc_pos_control/mc_pos_control_main.cpp#L765) in the attitude controller, otherwise Stabilize would require the same mapping the attitude controller again.
I would seek for having three kinds of inputs to the attitude controller:

Thrust setpoint + yaw (position, altitude etc)
Attitude setpoint (from an offboard module)
Stick inputs

The stick inputs are required for stabilized (and of course for rate-controller, but that is a different topic).
In stabilized, the stick inputs are first mapped to thrust setpoint + yaw and then mapped to attitude using the same Thrust-to-Attitude map.

@dagar I added the comments and move the orb_check


[AutoPR cognitiveservices/data-plane/CustomVision/Training/cognitiveservices/data-plane/CustomVision/Prediction] Add Custom Vision 3.0 API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5164

(message created by the CI based on PR content)

Codecov Report

Merging #4321 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4321      +/-   ##
==========================================
- Coverage   53.62%   53.62%   -0.01%
==========================================
  Files       10270    10267       -3
  Lines      216091   216088       -3
==========================================
- Hits       115880   115877       -3
  Misses     100211   100211

Impacted Files
Coverage Δ

...osearch/azure/cognitiveservices/search/__init__.py

azure-mgmt-documentdb/azure/mgmt/__init__.py

...computervision/azure/cognitiveservices/__init__.py

Continue to review full report at Codecov.

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


Issues after update to version 1.5.0.

Hello, I've updated vue-moveable to version 1.5.0. and I can not reference to setState() method.
I used (for example)
this.$refs.moveable.moveable.setState(() => this.moveable.draggable = true)
Now i can't set moveable values andI don't see my renderDirections.
Access to moveable-control-box also changed, from this.$refs.moveable.moveable.innerMoveable.preactMoveable.base.style to this.$refs.moveable.moveable.innerMoveable.moveable.moveable.controlBox.element.style - is that correct?

downgraded vue-moveable to 1.4.0.


Error opening configuration file: server.service.conf

I ran the script on Ubuntu 18.04.4 LTS
The VPN server works well except that I get error messages every 5 seconds:
Nov 11 16:11:28 vm8 openvpn[11582]: Options error: In [CMD-LINE]:1: Error opening configuration file: server.service.conf
see https://community.openvpn.net/openvpn/ticket/1353
I don't understand where all these services are coming from - I did not manually set them up.
root@vm8:/var/log# sudo systemctl|grep openvpn
openvpn-iptables.service                                                                  loaded active     exited       openvpn-iptables.service
<EMAIL_ADDRESS>                                                           loaded active     running      OpenVPN service for server
<EMAIL_ADDRESS>                                                   loaded activating auto-restart OpenVPN service for server.service
openvpn.service                                                                           loaded active     exited       OpenVPN service
system-openvpn\x2dserver.slice                                                            loaded active     active       system-openvpn\x2dserver.slice
Can you suggest a fix as we are using this for all our remote user to connect in from our company, and its hard to troubleshoot connection problems with all these errors in the log.
Thanks
Peter

I do not see an issue with the installer<EMAIL_ADDRESS>is not referenced at all in my script.


Context yield halts

Describe the bug
spring-boot 3.3.5 with spring-cloud 2023.0.3
I'm using the spring-cloud-kubernetes-fabric8-leader and while calling the Context.yield() method, the service halts. The leadership doesn't get revoked either as other instances of the service are fetching and logging the leader and it's still the same (have enabled debug level for this module).

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Sync with latest master

Node.js 8 added support for Set and Map and the performance improved a lot in general. Therefore it would probably be good to consider updating this again.

This module was made so AVA could have a consistent base for users. We've since moved away from using the core assert methods for everything except for throws/notThrows, but we plan on moving away from those too. The core asserts are just too locked down and weird for our needs.
So what I'm saying is that I don't really have the time or motivation to update this, but I would gladly take a pull request.

Interesting. Out of curiosity: what is weird about the assert module and how do you think should it behave instead so it would be more useful? I guess one of the things is that you can not get true / false back?

Many things. I can't recall everything right now. For example, they are loose by default, so many uses assert.equal and encounter a lot of silent problems because of the looseness. The deep equal assertion is not what users would expect at all (We went through 3 different deep equal assertions, before settling on the one we use now). But the main reason for not using it is that it's really only meant for use by Node.js core itself, but was exposed without much thought and now they can't remove it, and it's locked, so it can't change it much. There are much better assertion libs on npm.

We are definitely on the same page that the loose equality is really bad in many ways. It should probably not be used at all and I am going to open a PR to doc-only deprecate it soon. When using strict equality it does behave nicely though.
One very important point about the assert module - it is not locked anymore and it got improved a lot already plus a few more things that will land soon. The changes include correctness, performance and feature improvements.
I had a look at concordance and the comparison done there is pretty nice in general and it is very similar to the way assert handles it now.
A few things that I personally would handle differently

Compare Sets and Maps unordered
Recursion should not stop when a circular ref is found but when both sides are equal / unequal

There is a point of doing that too though and I am thinking about having something like that optional (not for assert directly but for something new).
If I am correct these will be the only differences in deepStrictEqual compared to concordance soon.
I open a PR against core-assert when the mentioned PRs landed.

Compare Sets and Maps unordered

But Set and Map are ordered.

I quote MDN:
It should be noted that a Map which is a map of an object, especially a dictionary of dictionaries, will only map to the object's insertion order—which is random and not ordered.

And the insertion order is something ignored while comparing e.g., plain objects.

http://2ality.com/2015/10/property-traversal-order-es6.html

I am aware of that change and the reasoning but this does not change that I still think it makes sense to ignore the insertion order.

I agree it's probably the most common use-case to not want the order compared, but that could result in silent bugs when it actually does matter. Seems like there might be worth having an additional deepEqual assertion or an option to also compare order.

// @novemberborn

I quote MDN:

It should be noted that a Map which is a map of an object, especially a dictionary of dictionaries, will only map to the object's insertion order—which is random and not ordered.

@BridgeAR What does that sentence mean? Is a Map "a map of an object", or are they referring to (somehow) creating a map of an object?

It's now deprecated.


Create test

What type of PR is this?
What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

test add


Deployment error: Module aos failed with error code -532462766.

Module aos failed with error code -532462766. I have this error on deployment. No issues in service fabric but deployment fails.

If you are getting errors when deploying please open a support ticket.


Update PyPI package name

There does not appear to be a PyPI package named onedatarestfs. Have updated based on: https://github.com/galaxyproject/galaxy/pull/16690/

You are right, thanks for fixing that.


[Select][typescript] TypeScript errors in onChange handler when using custom type for select value

It appears that the type declaration for SelectChangeEvent<T> in @mui/material/Select/SelectInput.d.ts is incorrect. While the intention there seems to be to make target.value be of type T, it still remains string:
export type SelectChangeEvent<T = string> =
  | (Event & { target: { value: T; name: string } })
  | React.ChangeEvent<HTMLInputElement>;

Using something like the following seems to be more like what was intended?
export type SelectChangeEvent<T = string> =
  & Omit<React.ChangeEvent<HTMLInputElement>, 'target'>
  & { target: { value: T; name: string } };

Here is a minimal self-contained source code file that reproduces the problem:
import React from 'react';
import Select, { SelectChangeEvent } from '@mui/material/Select';

type SelectValue = 'one' | 'two' | 'three';

export const Component = () => {
  const value: SelectValue = 'one';

  const handleChange = (event: SelectChangeEvent<SelectValue>) => {
    const newValue: SelectValue = event.target.value;
    console.log(newValue);
  };

  return <Select value={value} onChange={handleChange} />;
};

TypeScript compiler output:
SelectError.tsx:10:11 - error TS2322: Type 'string' is not assignable to type 'SelectValue'.

10     const newValue: SelectValue = event.target.value;
             ~~~~~~~~

Duplicate of #33399


Upgrading loadimpact/k6  from 0.42.0 to 0.44.1 breaks the tests

Description of the bug
must solve mounting of local files before upgrade to 0.43.1
See https://k6.io/docs/using-k6/modules/#using-local-modules-with-docker
Steps To Reproduce

bump dependency in docker-compose
run test docker-compose run k6 run /src/tests/end2end.js -e tokenGeneratorUserName=autotest -e tokenGeneratorUserPwd=*** -e env=at21 -e subskey=*** -e partyId=50011945 -e personNumber=01046501523 -e userId=20002660 -e org=ttd -e app=filescan-end-to-end
See error

Additional Information
No response

FIxed


caches: tweak the Windows toolchain build

Update the toolchain configuration to disable the python home embedding.
This should allow us to be more freestanding of the python location and
give us better control of the python library being used.
Backport a change from main which removes the embedded PYTHON_HOME from LLDB as we are currently embedding information about the build host rather than the target into the final binary.  This is required to make LLDB usable for debugging.

@swift-ci test

@swift-ci please test macOS platform


use signal.signal instead of loop.add_signal_handler

Windows doesn't have signals the same way *nix systems do, so asyncio's loop.add_signal_handler isn't implemented in Windows. The main difference between signal.signal and add_signal_handler is that add_signal_handler is asyncio safe, while signal.signal isn't; when the SIG_INT gets caught by signal.signal now, we could be in the middle of an async coroutine and it would be interrupted. I don't think this causes any race conditions; all we do is send an interrupt signal through the sessionstream, but I would like if @waydegg looked over this and gave his thoughts before merging.
Additionally, the reason this wasn't caught by testing is because we weren't running any TerminalClient tests on Windows (due to an issue I discovered a while ago involving PromptToolkit and Github Actions Windows runners specifically); before we get this merged, I'm going to look into resolving that problem in a different way so that we can still run these tests on Windows.

the reason this wasn't caught by testing is because we weren't running any TerminalClient tests on Windows

Was this disabled explicitly or were the tests not running b/c of issues with Prompt Toolkit and GHA?

the reason this wasn't caught by testing is because we weren't running any TerminalClient tests on Windows

Was this disabled explicitly or were the tests not running b/c of issues with Prompt Toolkit and GHA?

It was disabled explicitly, yeah. I'm working on changing the tests to just mock PromptToolkit while still running the tests

I don't think this causes any race conditions

I don't think so either. The shared state we're modifying outside the asyncio event loop is ok to modify:

self.should_exit and self._force_exit always start out as False and once toggled to True will stay True
asyncio.create_task enqueues another task that won't run until the handler finishes (i.e. the asyncio event loop takes over)

We'll have to update things if we have multiple event loops running or need to modify more complex state tho.