1.10.2 - Reliquary's Tome of Alkahestry gets deleted in crafter

When using Crafter to either charge or craft items with Tome of Alkahestry, the tome gets deleted after the first crafting.
In the past it just wouldn't use its charge and allow for duping items for free so I made a change to it so that it doesn't return itself in getContainerItem.
The reason that the crafting doesn't work is logic that's included in getRemainingItems method of Tome's recipes (new method to IRecipe with 1.8.9), that charges/uses tome's charge. EnderIO crafter doesn't have support for that so I am logging this here. Not a critical to me, but there are other mods using getRemainingItems as well (like Chisel and Bits) so you may want to add support for it.
Forestry and RFTools have added support so you may use those as a reference.
Steps to reproduce:

Make a recipe with uncharged Tome of Alkahestry and redstone (both blocks and dust work) in Crafter
Once set and items are supplied the tome disappears

Thanks :)

Hey guys -- I think this may have done a drive-by fix if the issue I was having automating Magical Crops & their Infusion Stones. When the stone was used, the stone would loose durability & the recipe would stop matching. Can I get a alpha build with this fix in it?
Thanks,
Shawn


Increase lint timeout to 5 min

What this PR does / why we need it:
The lint job is often flaking because it's reaching the timeout, which is 1 minute by default. This PR increases the timeout to 5 minutes.
Does this PR introduce a user-facing change?:
NONE

/retest


Possible to get "starting with" regex for multiple levels?

Hi there, I'm having a few issues trying to get admin/clients and all it's sub pages to mark as active
Have tried below but it only seems to go one level deep? Is this possible?
{{isActivePath regex='^admin/clients'}}

Ideally this would work for
admin/clients
admin/clients/profile
admin/clients/new

etc
Thanks for all your hard work on this helpful package!

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28226006-possible-to-get-starting-with-regex-for-multiple-levels?utm_campaign=plugin&utm_content=tracker%2F522373&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F522373&utm_medium=issues&utm_source=github).

It doesn't work because "/" has its special meaning in regex. You need to escape every such special character using "" in front of it.
I remember zimme mentioning that support for route groups could come handy in such situations.

{{isActivePath regex='^admin\/clients'}}

This should work as you expect. Also the AcitveRoute.group and {{isActiveGroup ...}} thing would help, I haven't had time to get to that and it isn't a prio as it's flow-router only so I need to think about if and how I can translate it into iron-router

Thanks both.
On 23 November 2015 at 14:48, Simon Fridlund<EMAIL_ADDRESS>wrote:

{{isActivePath regex='^admin/clients'}}
This should work as you expect. Also the AcitveRoute.group and {{isActiveGroup
...}} thing would help, I haven't had time to get to that and it isn't a
prio as it's flow-router only so I need to think about if and how I can
translate it into iron-router
—
Reply to this email directly or view it on GitHub
https://github.com/zimme/meteor-active-route/issues/44#issuecomment-158957427
.

--
André Goldstein
Director, Milktop
http://www.milktop.co.uk http://www.milktop.co.uk
@MilktopDesign https://twitter.com/MilktopDesign

I'll close this as solved. I'll open another issue for ActiveRoute.group/{{isActiveGroup}} functionality.

For my case (subcategories, subcategoriestwo and subcategoriesthree) this works fine:
{{isActivePath regex='^\\/categories$|^\\/categories/'}}


I can't edit file from the azure deploy with vs code

Repro steps:

Action: appService.fileEditor.onDidSaveTextDocument
Error type: 409
Error Message: Could not write to local resource 'D:\home\site\wwwroot\web.config' due to error 'Access to the path is denied.'.
Version: 0.16.1
OS: win32
Product: Visual Studio Code
Product Version: 1.40.2
Language: en

Call Stack
extension.bundle.js:323:803184extension.bundle.js:323:803184
s extension.bundle.js:218:111699
s extension.bundle.js:157:26069
extension.bundle.js:157:27311extension.bundle.js:157:27311
o extension.bundle.js:164:7075
extension.bundle.js:157:24982extension.bundle.js:157:24982
$._callback extension.bundle.js:1:576654
$.t._callback.t.callback.t.callback extension.bundle.js:194:5449
$.emit events.js:200:13
$.<anonymous> extension.bundle.js:194:18309
$.emit events.js:205:15
IncomingMessage.<anonymous> extension.bundle.js:194:17141
Object.onceWrapper events.js:288:20
IncomingMessage.emit events.js:205:15
endReadableNT _stream_readable.js:1154:12
processTicksAndRejections task_queues.js:84:9

Hi @FilipLaurentiu
Thanks for reporting!
Do you have all permissions and access rights to this web app?  Are you the one who deployed to this web app?
A 409 error usually means that there was a conflict between the server and the client.  Could you try downloading the file and saving it again?


Numerical Evaluation external libraries

In the documentation, it's mentioned that:

The 'bignum' mode utilizes the decimal.js library.
The 'complex' mode is supported by the complex.js library.

Could you please clarify if we need to incorporate these libraries ourselves? If not, how can we ensure that the most recent versions of these libraries are in use?

I doubt it, I'm using their latest versions.


Make sure gguf_ctx is closed when error happens

Refs https://github.com/ml-explore/mlx/issues/1513.
This PR makes sure the gguf_ctx is closed when exception is thrown in save_gguf and load_gguf. Without doing would leave the files opened and result in test failures on Windows, as it does not allow writing to already opened files.

I'm making gguf-tools work on Windows.


when AQE enabled we fail to fix up exchanges properly and EMR

This is part of issue in https://github.com/NVIDIA/spark-rapids/issues/1423
The issue here is in the way that AQE does its subqueries at exchange boundaries and when we re-analyze the subqueries we only see the children of the exchange.  This can results in our fixUpExchangeOverhead marking the plan different initially from when the subquery is actually ran and we end up having one part of the join coming from CPU exchange and one part coming from GPU exchange and we blow up.
To handle this case we have to make sure we initially mark the plan the same as AQE will on its subqueries. This means in the fixUpExchangeOverhead when AQE is enabled we only look a the children in deciding if the exchange will run on the GPU or not.
there are more details in the issue.
Signed-off-by: Thomas Graves<EMAIL_ADDRESS>

sorry let me update the comment here I have a typo and I had a more detailed version

sorry let me update the comment here I have a typo and I had a more detailed version

build

build

build

build

build

build


Contents are flipped vertically

For some reason the preview window has its contents flipped vertically (or treats Y rotations differently)
Here's a preview in VS Code:

Here's a preview in Chrome:

Actually I think this was just a bug with the bundled version of p5.js. After I manually updated them to the latest versions it rendered correctly.

@b3ll thanks for the report!
To which version did you update it? I can release a new version of the extension with it.

0.5.16 was the one I used!

Just published 1.0.3 with p5 0.5.16.
I'd like to figure out a good way to enable users to use their own index.html file with minimal configuration.
Thanks again for the report, @b3ll.


NamedArg @ notation

The package sql.NamedArg docs show the use of the @param1 style of NamedArgs, and the go-mssqldb readme also mentions the @ notation , but it is not accepted as an input to a query.
It seems that the parser.go parseNormal func is skipping the '@' character.  When adding it as an option, it solves certain conditions, but gets picked up when testing queries with T-SQL local variables using the @ prefix. (i.e. DECLARE @tblVar TABLE...)
Is there a preferred way to  distinguish these T-SQL variables from the driver supplied args?  If not, should they be avoided, and rather use the $ and : notations only?  And should the readme indicate such?

I don't fully understand your question. But does this example help?
db, err := sql.Open("sqlserver", "...")
sqltext := "select * from Invoice where ID = @ID;"
rows, err := db.QueryContext(ctx, sqltext, sql.Named("ID", 998))

So given these examples:
if _, err = db.QueryContext(ctx, "Select [name] from sys.tables where max_column_id_used > $val;", sql.Named("val", 9)); err != nil {
	panic(err)
}
fmt.Println("$ OK")

if _, err = db.QueryContext(ctx, "Select [name] from sys.tables where max_column_id_used > :val;", sql.Named("val", 9)); err != nil {
	panic(err)
}
fmt.Println(": OK")

if _, err = db.QueryContext(ctx, "Select [name] from sys.tables where max_column_id_used > @val;", sql.Named("val", 9)); err != nil {
	panic(err)
}
fmt.Println("@ OK")

I am always getting the following:
$ OK
: OK
panic: sql: statement expects 0 inputs; got 1

You can use "sqlserver" or "mssql" for driver name, but there is a difference:
"mssql" expects positional parameters and translates the ? ?n $n :n parameters to @p1, @p2, etc.
"sqlserver" does not translate parameters, so you can use named parameters as @name

db, err := sql.Open("sqlserver", "...") // Not "mssql"

Ah excellent, that works.  My mistake, thanks!


How set for definitions for json-schema in component (angular 2).

Can you explain how set form definitions for json-schema in component. I have trouble, when try set layout array. Can you some example? (I use angular 2)

@person2713 did you mean to add this issue here or at https://github.com/dschnelldavis/angular2-json-schema-form/ ?


Unknown command line argument '-psm' with Tesseract 4

Hi,
My tesseract doesn't known the -psm argument.
tesseract v<IP_ADDRESS>81030
 leptonica-1.76.0
  libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.5.3) : libpng 1.6.34 : libtiff 4.0.9 : zlib 1.2.11 : libwebp 0.6.1 : libopenjp2 2.2.0

I succeeded with --psm instead of -psm.

Yes, seems there should be an update for it

Whats the situation of that update?

still running into this issue...
Error, unknown command line argument '-psm'
I'm on Windows 10 64-bit

This is a simple fix, it just needs another - so it looks like this: --psm on line 65 of lib/tesseract.js. I have a fix but can't push my branch to create a PR due to permissions by the owner


.NET Core 2.0 Compatibility

@nixxholas, did you try the netstandard20 branch? What's in this PR that's not already on that branch? 😕  (Yes, the NuGet Package isn't there; it's going to be done separately.)

Although not a common practice, this allows users to use the git submodule functionality to add packages into their projects. Hope you consider this option as some institutions mght make changes for custom API modifications.

Although not a common practice, this allows users to use the git submodule functionality

Hmm... This looks interesting 👍 Where is this happening exactly? 😕

Okay, I understand this 👍 💯 But your PR contains a whole bunch of other changes(?)

Okay, I understand this 👍 💯 But your PR contains a whole bunch of other changes(?)

Will work on refactoring the redundant ones as soon as possible, once im set on my desk!

Yeah, that'd be awesome. Actually, for the git submodules thing, I would expect just 1 file change or 2 max.

Yeah, that'd be awesome. Actually, for the git submodules thing, I would expect just 1 file change or 2 max.

Infact, its just one file. .csproj, the imported packages has got to be there and that's it, json and sysconfig

Infact, its just one file. .csproj, the imported packages has got to be there and that's it, json and sysconfig

Would you mind opening a PR from your branch to netstandard20 branch? This way we can easily look at the diff-output and see exactly what those changes are.

Would you mind opening a PR from your branch to netstandard20 branch? This way we can easily look at the diff-output and see exactly what those changes are.

Apologies, have OCD here. Prefer to clean up and send a new PR instead, will do so once I clear up some things at hand! =)

Apologies, have OCD here

Exactly—Because I'm all into OCD, it's the reason I'm thinking that the best way to tackle this is with a PR from your branch into netstandard2.0 branch. 👔
Once we're done with the .NET Standard stuff, we can merge it into master branch, but YMMV 😅 😜


The 0.12.0 release commit is wrong.

https://github.com/http-rs/tide/releases/tag/v0.12.0 points to https://github.com/http-rs/tide/commit/261038d7487b1bef475e01ea012c1b44a6e63ae9 which seems wrong, but given that the version on crates.io is indeed 0.12.0 this is clearly wrong as the Cargo.toml version number would have needed to be bumped. As it stands, Cargo.toml in this repo still has version = "0.11.0".
@yoshuawuyts

Pushed!

For anyone else that comes across this bf135bd seems like a good guess at the right commit. Not sure though.

actually, bc026d1220626b89d590348e608c7fde3c0bd0de maybe


bots: Change logic for Notification bot after moved message

Previously, when a user moves a message to another topic, the Notification bot will post a message saying "This topic was moved here from..." This is confusing when the topic already contains messages. The changes aims to make the messages more clear by changing the logic for the Notification bot. When there is already messages in the topic, the bot will post "A message was moved here from..." or "N messages were moved here from...". The bot will post "This topic was moved here from (somewhere) by (someone)." when the topic is empty.
Fixes #23267

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@Joelute please address the failing tests, and post a comment when the PR is ready. Thanks!

@alya I've finished fixing the issues and it is ready for review.

I tried moving all the messages in a topic into an existing topic via the "Move messages" UI, and got the following error:

Created new testcases for the new bot responses but it's currently not working and results in a response that differs from the expect response. Currently waiting for some help to address the issue.

Thank you for the review! It was very insightful and I was able to make the changes needed. It was very interesting to me that the test cases doesn't fill out the topic_name. I've also squashed down all the commits to one commit.

@laurynmm Is this PR ready for integration review from your perspective? I'll do another round of manual testing when that's the case. Thanks!

Hello @laurynmm! Thank you for the review again, I've made the changes you've suggested. Please have a look again and let me know if there should be anymore changes.

@Joelute - These updates look good to me. @alya - Ready for you to do another round of manual testing.

I general question I have is about resolved/unresolved topic messages and moving messages. I noted that on the current main and with these changes, the resolved/unresolved messages that are sent are different if/when the messages are being moved to a different stream or to the same stream.
For example:

If I move an unresolved topic message (#test > my topic) to a resolved topic in the same stream (#test > ✔ resolved topic), then a resolved topic message is sent when the message is moved, in addition to the moved message notification.
If I move an unresolved topic message (#test > my topic) to a resolved topic in a different stream (#errors > ✔ resolved topic), then only the moved message notification is sent. The resolved topic message is not sent.

Again, this is the behavior before and after the changes in this PR, so perhaps a follow-up discussion in CZO or issue. Or maybe this is the intentional behavior and we're all good. It is a bit of an edge case in that it's probably not often that messages are moved to a resolved topic in another stream, but I thought it worth mentioning.

Looks good to me in manual testing!
@timabbott this PR has been reviewed by @laurynmm and me -- please take a look.

Let's also discuss @laurynmm 's comment above live; I don't remember if there was some reason why it currently works the way it does.

Thanks for your work on this @Joelute! I have two concerns with this implementation that may be nontrivial to address:

The messages_for_topic query and path is potentially slow/expensive. While it may not be expensive compared to this endpoint overall, in theory, we just need to know if there's at least one message there, perhaps using .exists(). We can't do that simply because we're asking the question after we've already moved the messages; though one do a messages_for_topic(recipient_id, topic).exclude(id__in=[*changed_message_ids, resolved_topic_message_id]), with the latter being an improved return value for maybe_send_resolve_topic_notifications (it could return None or the message ID rather than True/False).
The messier complication is that there's a potential security bug here -- behavior triggered by a user should not depend on the existence or not of messages that user cannot access. You can see that we do a visible_unmoved_messages = bulk_access_messages check when computing moved_all_visible_messages; we'll want to do the same sort of security check here.

The security issue of course takes priority over performance, but I think we can at least do the exclude as well as doing the bulk_access_messages filtering required to correctly determine if there are preexisting topics. The good news is you'll probably be able to get rid of the arithmetic -- we're just looking at whether there's at least one message the current user can access in the target topic.

As a smaller comment, preexisting is a single word, and we should use that rather than pre_existing in variable names.

Currently I don't see a way to retrieve the resolved_topic_message_id. If I understand this correctly, I should make changes to maybe_send_resolve_topic_notifications and make it return the message ID of the resolved notifications or None rather than True/False. Is that correct @timabbott? Just making sure.

Yeah -- the internal_send_stream_message function returns the message ID of the message it sent, so you should be able to use that.

Sure, I'll make a sperate commit for it. I've gotten that part down but I'm not sure how to really address the security issue. Do we do a bulk_access_messages on the messages in the new topic/stream? Or do we do it on the old topic/stream just like how moved_all_visible_messages was computed above?

Took me a while to understand the security issue and I think I should have it down now. Will do a bit of finishing touches and manual testing before pushing another commit.

@timabbott this PR is ready for another round of review from you. I've made a sperate commit to improve maybe_send_resolve_topic_notifications. Included the exclude and added the bulk_access_messages filtering. Please have a look!

I applied the following patch to clean up some of the variable names and also document both the purpose of this logic and the known way in which it is potentially inefficient:
diff --git a/zerver/actions/message_edit.py b/zerver/actions/message_edit.py
index 64cdcf92b1..6ebbfbdfa4 100644
--- a/zerver/actions/message_edit.py
+++ b/zerver/actions/message_edit.py
@@ -879,23 +879,36 @@ def do_update_message(

             new_topic = topic_name if topic_name is not None else orig_topic_name

-            changed_messages_id = [changed_message.id for changed_message in changed_messages]
+            changed_message_ids = [changed_message.id for changed_message in changed_messages]

+            # We calculate whether the user moved the entire topic
+            # using that user's own permissions, which is important to
+            # avoid leaking information about whether there are
+            # messages in the destination topic's deeper history that
+            # the acting user does not have permission to access.
+            #
+            # TODO: These queries are quite inefficient, in that we're
+            # fetching full copies of all the messages in the
+            # destination topic to answer the question of whether the
+            # current user has access to at least one such message.
+            #
+            # The main strength of the current implementation is that
+            # it reuses existing logic, which is good for keeping it
+            # correct as we maintain the codebase.
             preexisting_topic_messages = messages_for_topic(
                 stream_for_new_topic.recipient_id, new_topic
-            ).exclude(id__in=[*changed_messages_id, resolved_topic_message_id])
+            ).exclude(id__in=[*changed_message_ids, resolved_topic_message_id])

             visible_preexisting_messages = bulk_access_messages(
                 user_profile, preexisting_topic_messages, stream=stream_for_new_topic
             )

-            none_visible_preexisting_messages = len(visible_preexisting_messages) == 0
+            no_visible_preexisting_messages = len(visible_preexisting_messages) == 0

-            if none_visible_preexisting_messages and moved_all_visible_messages:
+            if no_visible_preexisting_messages and moved_all_visible_messages:
                 new_thread_notification_string = gettext_lazy(
                     "This topic was moved here from {old_location} by {user}."
                 )
-
             else:
                 if changed_messages_count == 1:
                     new_thread_notification_string = gettext_lazy(

With those changes, this lgtm, marking to merge once CI finishes.
We have an existing goal to profile and optimize the bulk message move code path; I'm not sure the performance issue noted i these comments is significant to the overall cost of that operation in large servers, but in any case I think we can work on that in the broader context of that optimization.


how i get single unique identifiers of connected User and send message with him ! Urgent

$conn->resourceId or use Sessions


Adding TFLite's variable lowerings to ML Program

Request description
TFLite lowers directly to IREE's input dialect for stateful behavior.  We should switch to ml_program within the tflite-to-tosa pipeline. This gives variable support while not introducing a direct dependency on IREE.
What component(s) does this issue relate to?
Frontends
Additional context
No response

Already landed in https://github.com/iree-org/iree/pull/9671 so closing.


Make ParseRegionError more useful.

Implements std::error::Error for rusoto::region::ParseRegionError and has it tell you what the bad input was when it gets Displayed.

Looks good to me.


chore(deps): update vitest monorepo to v2.1.6

This PR contains the following updates:

Package
Type
Update
Change

@vitest/coverage-v8 (source)
devDependencies
patch
2.1.5 -> 2.1.6

@vitest/ui (source)
devDependencies
patch
2.1.5 -> 2.1.6

vitest (source)
devDependencies
patch
2.1.5 -> 2.1.6

vitest (source)
peerDependencies
patch
2.1.5 -> 2.1.6

vitest (source)
dependencies
patch
2.1.5 -> 2.1.6

[!WARNING]
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

Release Notes

vitest-dev/vitest (@vitest/coverage-v8)
v2.1.6
Compare Source
🚀 Features

Support Vite 6

View changes on GitHub

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled because a matching PR was automerged previously.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (2.1.6). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Datepicker - Label tag is hard coded and cannot be removed.

The ember Datepicker has a hardcoded label tag with the contents of "Date Picker Label". There is no attribute to allow disabling/changing the label.

Fixed for next version


[FEATURE] - Add touch support for mobile devices

[x]  I agree to follow this project's Contributing Guidelines.

Is your feature request related to a problem? Please describe:
The current version of the app does not support drawing via touch.
Describe the Solution You'd Like:
Add touch support along with mouse pointer for drawing in the canvas.

Hey @sd012gfhkhhvh Sir I would like to Work on this issue under JWOC 2024

Yeah @sd012gfhkhhvh Sir


Issue #2

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Learned a lot. Keep moving.


Create README.md

Added README.md

Do add corresponding Label @DevrajDC

@Shridhar-dev Do review and approve changes. 😄


Generated classes in embedded projection library are not public

When I generate a  projection library with CsWinRTEmbedded=true all the projected types from my c++/winrt component are defined as internal and so not visible to any consumers.
Is there any way to generate these types as public so I don' have to provide an additional wrapper class?

@partmachine is there a reason for not using the global projection scenario (not using embedded)? Embedded projections require the types to be internal today. It would help to understand your scenario better, and  reasons for choosing the embedded projection option.

@partmachine I think you'd want them to be internal in a class library or you'll likely hit collision issues with consumers of your library also generating winrt types or referencing the nuget packages that gives access to all the Windows APIs.


How to run using multi-GPUs?

Hi, I'm not so familiar with the training method using multi-GPUs.
I have a machine with 8 A100s, what should I do to full params SFT a llama2-7B model?
How to use the trl tool?
Thanks.

You don't have to change anything in the way your script is structured and just change the setting in acclerate config. Usually a good idea to get things to work on a single GPU first, though! See this guide: https://huggingface.co/docs/trl/main/en/customization#run-on-multiple-gpus-nodes

@lvwerra I got this:
accelerae config:
compute_environment: LOCAL_MACHINE
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: all
machine_rank: 0
main_training_function: main
mixed_precision: 'no'
num_machines: 1
num_processes: 7
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

run command
accelerate launch  trl/examples/scripts/sft_trainer.py \
    --model_name models_hf/llama2-7B-hf/ \
    --dataset_name timdettmers/openassistant-guanaco \
    --batch_size 3 \
    --gradient_accumulation_steps 2

Finally, Out of Memory:
OutOfMemoryError: CUDA out of memory. Tried to allocate 66.00 MiB (GPU 4; 79.15 GiB total capacity; 76.49 GiB already allocated; 14.38 MiB free; 77.25 GiB reserved in total by PyTorch) If reserved memory is >>
allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Does full params finetune cost so much?

Yes :) You probably may want to use PEFT to help with that!

Same issue here. I think if we enable DeepSpeed by accelerate, it should work on 8 A100s.

@cwzhao Another full params finetune method is llama-recipe.
And it works for llama-2-7B.
I don't know if there are some differences between the two methods. @lvwerra

I got this when using accelerate
ValueError: You can't train a model that has been loaded in 8-bit precision on a different device than the one you're training on. Make sure you loaded the model on the correct device using for example `device_map={'':torch.cuda.current_device()}you're training on. Make sure you loaded the model on the correct device using for example `device_map={'':torch.cuda.current_device() or device_map={'':torch.xpu.current_device()}
Using pad_token, but it is not set yet.


fix: update rate limit params unconditionally when rate limit is reset

closes https://github.com/FuelLabs/fuel-bridge/issues/307

commits to review

https://github.com/FuelLabs/fuel-bridge/pull/308/commits/1626348322a5401f4411f2b700a4e4bda1310d69
https://github.com/FuelLabs/fuel-bridge/pull/308/commits/aa068f88396bfe66d89ce717210bbecbeb3f6aa2
https://github.com/FuelLabs/fuel-bridge/pull/308/commits/4ab621b18b306b93525b4127790887948cb182bb

we updated our docs to explain the mechanism of the resetRateLimit method and decided not to merge this so closing for now


Style changes for LiveComs

Two-column layout; blurb alongside abstract; checklists; algorithms; numbered section headings.

Thanks, @liantze ! We're super glad to have these updates.

This looks beautiful, @liantze!

Thank you @davidlmobley @jchodera , glad you like it so far! 😄

One question: when I compile it, I get the web address of the GitHub repository overflowing into the abstract.  It doesn't appear to do this in the version that LianTze posted.  See below:
overflow_of_address.pdf

@mrshirts Hmm, were you compiling with pdflatex or latex+dvipdf? Or which MikTeX/TeX Live version are you on?

This was compiled with luatex ('luatex --version' gave This is LuaTeX, Version beta-0.80.0 (TeX Live 2015) (rev 5238)) since I couldn't compile with pdflatex (See https://github.com/livecomsjournal/article_templates/issues/26 for discussion of that).


Slow stop services

This PR makes some startup time faster: this is particularly noticeable on autoscaled node.  In details:

stop services only if they are actually running (avoid unnecessary calls to monit)
skip check DB is primed on autoscaled nodes
make hermes and admin_server started in a thread too (like the others services)
use appscale-amin summary instead of monit summary (faster in most cases)

OCD build http://ocd.appscale.net:8080/job/Daily Build/6758/

There have been few rebuilds of the autoscale triggers test and they all got back blue
http://ocd.appscale.net:8080/job/Autoscale-test-manual-triggers/2877/


Add Code coverage metrics

Hi, Tom from Codecov here.
Some of our engineers are huge fans of styled-components, and I thought it would be useful to add more useful code coverage metrics to this repository. Codecov is a hosted code coverage provider with awesome features. You can checkout your first reports here.
I would love all the feedback you have, and please don't hesitate to reach out if you have any questions.
Cheers,
Tom
Codecov ❤'s open source and will be free forever for OSS

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f0a84f4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3158   +/-   ##
=========================================
  Coverage          ?   54.10%
=========================================
  Files             ?       79
  Lines             ?     2889
  Branches          ?      701
=========================================
  Hits              ?     1563
  Misses            ?     1040
  Partials          ?      286

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 f0a84f4...2b38508. Read the comment docs.

Just wanted to see if this is useful, or if there's anything I can do here.

This is neat, thanks for adding it!


[1.12.2-0.8.0] TTX Tech PS3 controller has Z-Axis and Z-Rotation swapped, registers as 'Piranha xtreme' in-game

I am trying to play on a 1.12.2 server with 0.8.0 of this mod, and the only controller I currently have, a TTX Tech PS3 controller, registers it's Z axes incorrectly. I am having the exact same issue talked about here with the controller, which as far as I am aware, is an issue known to happen with PS2 USB controllers, not PS3, but here we are. Can this be fixed ASAP? I really wish to use the controller to move the camera, without using my mouse. Thanks.
Here is the Hardware ID of my controller, if it helps:
HID\VID_0E8F&PID_0003&REV_0109
Note that the controller uses generic Windows 10 drivers for it, and the axes are set correctly when calibrating it, and the axes work as intended in Genshin Impact, so this is an issue with this mod only.

Closing this issue as I do not maintain the 1.12.2 version of this mod anymore.


pending publishes not working on lbry.tv

Looks like we call stream list right after we submit the publish, and don't wait for it to come back. So stream list doesn't include the pending entry and it never checks it again.

This also causes the publish not to appear as pending, even after you get the notification it was submitted.

On desktop, my pending publishes also seem to have an issue - they don't end up in publish state but the pending check does happen and you get a notification. Though you don't see the claim until you refresh (or leave/come back)

This is partly working now - there's a stream list call that's made right after a publish which clears it out on lbry.tv and then brings it back 10-15 seconds later. Related to this original issue.


Add SSID filtering

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Security Fix
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Dependency Update

Description
Added SSID filtering will allow sniffer to ignore SSIDs from the SSID_FILTER list and in doing so reduce a socket connection load if done with SSIDs that have high ocurence.
Related Tickets & Documents

Closes #29

Added/updated tests?

[ ] Yes
[x] No, and this is why: testing not utilized at this point
[ ] I need help with writing tests

The README.md should be updated to inform users about the option to filter out unwanted SSIDs.


Updates to documentation for root package and tf.a*

This PR contains miscellaneous API documentation updates that cover the root package tf and all the API functions whose names start with tf.a. Some of the changes correct typos, while others document aspects of these functions that weren't obvious to me without reading the source code.
Note that the functions tf.abs, tf.add_n, tf.accumulate_n, and tf.angle moved to the tf.math package while I was making these edits.
I've generated and reviewed the Markdown files for the documentation after these changes.

It looks like there was a hiccup somewhere in the CI infrastructure. The build and tests are all working on this branch on my test machine. Would someone with access mind rerunning the CI checks for this PR?

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@frreiss Can you please resolve the conflicts? Thanks!

Merge conflicts fixed. Currently rerunning regression tests on my test machine.

@frreiss There are branch conflicts. Can you please rebase? Thanks!

Merged in changes from master, including manually moving some of my documentation edits from gradients_impl.py to gradients_util.py. The branch conflicts should be fixed now.

@MarkDaoust Can you PTAL? Thanks!

Hi @MarkDaoust and @ymodak , this PR appears to be ready to merge. Is there anything still holding it up?

@frreiss can you please resolve conflicts

Conflicts resolved.


Manual retrigger based on file changed

This is just an idea I had, but it would be pretty cool to have a feature where I could specify that if a change was triggered from a specific file path, Tilt would not go through with the build but instead allow me to go into the UI and trigger the build manually. If something else then were to trigger a rebuild that's not part of the "manual file path", Tilt would go on doing an automatic build.
The reason I personally want this is because I have a library folder with some common code for all my Rust services. These libraries are part of what Tilt watches but changing it means that all containers will be rebuilt at the same time which is suboptimal.

It sounds like you've already tried trigger_mode=TRIGGER_MODE_MANUAL?
https://blog.tilt.dev/2019/06/19/new-in-tilt-fine-tune-how-your-services-get-updated.html
I've seen people make all their builds into manual builds, though that would definitely slow down the single-service case.
I go back and forth on what the default should be...maybe the default mode should be auto-build, and you have to specify which paths are manual...or maybe the default mode should be manual, and you can specify certain files that auto-build.
Your use-case is also interesting, because you probably don't want auto-build if all N of your services need to update, but always want auto-build if only 1 service needs to update...

I think you pretty much nailed it. I would personally vote for automatic default with manual exclusion paths. Automatic wasn't really an issue for me until I added the shared libraries which is something I imagine is an edge case anyway.

Going through some old tickets -- @dotellie is there still an unresolved issue here for you?

I haven't kept super close watch on the latest updates so I'm not sure. I would assume that it still is something I want but I'm not 100%. Do you have any specific newly released features in mind that would help with my use-case?

Oh shoot, sorry, I misread your initial issue and so I thought it should have been resolved by Manual Update Control.
To be clear: you want a resource where changes to file ./manually/exempt/file mark the resource as dirty but don't trigger an update unless you manually click the "update" button; but where changes to all other watched files trigger an automatic update as normal? (This is in contrast to Manual Update Control, where either ALL file changes trigger automatic builds, or ALL file changes mark the resource as dirty and provide the option for a manual build.)

Haha no worries. You're on point with that explanation now though! Of course the file path would have to be globable (is that a word even?) but that was probably obvious to begin with. :sweat_smile:


nyc generates inconsistent file delimiters

Link to bug demonstration repository
https://github.com/david-kirkland/ng18-code-coverage/tree/bug-nyc-file-paths
Branch = bug-nyc-file-paths
Expected Behavior
The generated coverage JSON has consistent file path delimiters.
Observed Behavior
The generated coverage JSON has inconsistent file path delimiters.
Sometimes it uses a forward slash and sometimes it uses a double backslash.  This becomes a problem when attempting to merge reports as coverage is not calculated cumulatively when the file paths do not match between reports.

Troubleshooting steps
You will see in the repo that I am attempting to merge component and e2e test coverage reports (via npm run test:coverage), both generated by Cypress.  I am not saying the file path delimiters differ for component and e2e tests.  I'm saying the generated JSON coverage will use different file path delimiters for the same file when re-running the same test type (component or e2e) without any other changes.  For example, run npm run test:component and sometimes you will get...

Re-run npm run test:component without making any changes and the next time you might get...

Note:  I am on a company-issued Windows 10 Enterprise machine.

The bug still occurs when I put cache: false in my nyc config.

Environment Information
  System:
    OS: Windows 10 10.0.19045
    CPU: (12) x64 13th Gen Intel(R) Core(TM) i7-1365U
    Memory: 13.74 GB / 31.64 GB
  Binaries:
    Node: 22.4.1 - ~\AppData\Local\fnm_multishells\11776_1721413704500\node.EXE
    npm: 10.8.1 - ~\AppData\Local\fnm_multishells\11776_1721413704500\npm.CMD
  npmPackages:
    @jsdevtools/coverage-istanbul-loader: 3.0.5 => 3.0.5
    istanbul-lib-coverage: 3.2.2 => 3.2.2
    nyc: 17.0.0 => 17.0.0
    source-map-support: ^0.5.21 => 0.5.21
    ts-node: ^10.9.2 => 10.9.2
    typescript: 5.5.3 => 5.5.3

Having same issue on vite+vue+cypress+jest project<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Worked around by manually normalizing paths before running nyc report.


Suggestions

Hopefully we can add the option to customize the location. It may not be particularly noticeable in the center of the screen.

#13


Show extra '...' for actions 'Open/Delete/Break Lease' against blob containers in the view opened by clicking 'View all'

Storage Explorer Version: 1.32.0-dev (93)
Build Number: 20230825.4
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.5 (Apple M1 Pro)
Architecture: x64/x64/x64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Launch storage explorer -> Open 'Settings' -> Select 'Services'.
Set the setting 'Storage Nodes Trailing Action' to 'View all'.
Refresh the tree view -> Expand one storage account -> Blob Containers.
Click 'View all' in the tree view -> There is a separate view.
Right click one blob container in the view -> Observe the context menu.
Check whether no extra '...' for actions 'Open/Delete/Break Lease'.

Expected Experience
No extra '...' for actions 'Open/Delete/Break Lease'.
Actual Experience
Show extra '...' for actions 'Open/Delete/Break Lease'.

Removed "..." from delete in blob container explorer to match blob explorer and blob container node
Removed "..." from open in blob container explorer to match blob explorer and blob container node
Added "..." to break lease on blob container node to match blob explorer and blob container explorer

@MRayermannMSFT
According to action on the break lease dialog. It doesn't make sense to add '...' next to break lease action against one blob container node. Could you help to double confirm this?
Break lease for a blob container. show below dialog:

Break lease for a blob. show below dialog:

@v-xianya yes, we're going to be going through our menus in December to make sure we're being more consisent on our usage of ..., for now though, please just make sure individual actions are consistent across context menus.


feat(hardware-control): disengage 96-channel and gripper z after retract

Overview
We should disengage the 96-channel pipette mount and the gripper mount motor whenever they're not active. This helps reduce current consumption and lower the chances of them dropping due to noise.
Note: This only affect the _cache_and_maybe_retract func in the hardware control API, this means you would only see this change if ot3api.move_to()/move_rel() is used.

Test Plan
Make sure protocols with a 96-channel pipette and a gripper still work like normal

After talking with @andySigler, we've decided to not merge this until some more hardware testing is done on the 96-channel pipette; the reason being that the changes here might have an effect on the overall temperature of the pipette and affect the pipetting volume.

sry for the delay, we'll be doing this testing first thing next week. The test is to leave the 96ch disengaged for several hours, then engage it and immediately run a gravimetric test. Worst case scenario, the pipette will start heating up enough that we'll see a change in the average volume during the first maybe 10x trials. That is the concern, that the change in this PR would create a scenario where one of our pipettes was not able to reach a stable temperature before a protocol is ran.


Update with joins

I've cleaned a little bit the join code, and now, you can update an entire entity including their children with a single query!
Also added the possibility to specify the table prefix for your schema (#126)
Started SQLite tests too.

With this PR we can release the next version (1.13)

@pvictorlv Good job!


Avoid trying to log kwargs which are not serializable by instead logg…

This has been tested by running the following simple test:
from uuid import uuid4
from twisted.trial.unittest import TestCase
from twisted.internet.defer import succeed

from ..testtools import log_method

class FooTests(TestCase):
    @log_method
    def foo(self, argument):
        return succeed(True)

    def test_foo(self):
        self.foo(argument=uuid4())

and seeing the trial log with no TypeError:
flocker.acceptance.endtoend.test_simple.FooTests.test_foo <--
2015-10-20 14:58:15+0100 [-] ELIOT: {"task_uuid": "2372c19d-b479-4591-b64a-65dcff2741d6", "task_level": [1], "action_type": "acceptance:foo", "kwargs": {"argument": "UUID('e005a24d-d779-4da2-8230-62fa030f33e4')"}, "timestamp":<PHONE_NUMBER>.43049, "args": [], "action_status": "started"}
2015-10-20 14:58:15+0100 [-] ELIOT: {"timestamp":<PHONE_NUMBER>.430914, "task_uuid": "2372c19d-b479-4591-b64a-65dcff2741d6", "message_type": "acceptance:foo:result", "task_level": [2], "value": true}
2015-10-20 14:58:15+0100 [-] ELIOT: {"timestamp":<PHONE_NUMBER>.431244, "task_uuid": "2372c19d-b479-4591-b64a-65dcff2741d6", "action_type": "acceptance:foo", "action_status": "succeeded", "task_level": [3]}

@jongiddy had two concerns:

Should this be a change somewhere in eliot instead?
This is wasteful, and perhaps changing this to only look for non-serializable objects when an exception occurs would be better. Also, checking if the object has type instance is a more efficient check for whether something cannot be dumped.

I'll wait for an answer on (1) before working on (2). Or perhaps there is a more eliot-y way to do this.

This is code in the acceptance tests, where network latency is order of magnitude more than time spent waiting for CPU, and then there's cloud API slowness which is 4 orders of magnitude slower if not more. So this is perfectly fine as far as speed.
I can think of some ways Eliot might help with this, and will file issue, but that shouldn't keep this from proceeding.

Thanks! This will definitely help with debugging. LGTM. This is OK to land even if the AWS tests fail, IMO.


Fixed binary representation typo in comment

Digging around in scene_multiplayer.h, I saw a couple comments that contradicted each other.
// Extract the `packet_type` from the LSB three bits:
uint8_t packet_type = p_packet[0] & CMD_MASK;
...
enum {
   CMD_MASK = 7, // 0x7 -> 0b00001111
};

The packet_type comment mentions "packet_type" is extracted from 3 bits, however the corresponding mask had a comment claiming 4 bits are set high. Looks like the CMD_MASK comment has a simple typo/off by one error, 0x7 is 0b0111 not 0b1111.
Feel free to ignore this if it is too nit-picky.

Thanks!


Could not load file or assembly - Microsoft.AnalysisServices

Hello!
I attempted to start using the new version of the tool this afternoon. I installed using the v2.0.0 msi file. I see the icon in External Tools, but I was met with this error when trying to run the tool:

Thank you for your help.
Best,
Matt

Same issue ... 😥

Same issue here. Exact same error on two different machines

Hi @orsim25, @samaguire and @pugliathomas
I'm sorry you encountered this issue. Can you please go to C:\Power BI Model Documenter and share what is in the log file that you find there? That should tell more about the exact step where it got stuck.

A quick search for me resulted in a pretty old item on the SQL Server forum which explains that it is potentially due to a missing assembly which comes with SSMS for example. So it may be caused by AMO missing.
Although this NuGet package is part of the Model Documenter tool installation, the assembly could not be found during execution of the tool. On a Windows machine there is something called Global Assembly where all DLL files are located which can be used by local client applications. It seems that the Microsoft.AnalysisService.dll assembly is not there.
As Power BI runs Analysis Services under the hood, it looks like you're missing the right client libraries to connect to the Analysis Services instance that runs behind Power BI. You can download and install them from the official MSFT website here
Could you try if this fixes your issue?
Thanks!

Hi @orsim25, @samaguire and @pugliathomas I'm sorry you encountered this issue. Can you please go to C:\Power BI Model Documenter and share what is in the log file that you find there? That should tell more about the exact step where it got stuck.

Hi @marclelijveld
For me, the folder doesn't exist.
BTW, I've seen this sort of error before in TE3. I was referencing the assembly "Microsoft.AnalysisServices.Core.dll" in a C# script to execute a dataset backup command and hitting an error. I found out that TE3 wasn't checking the program's installation directory for the dll files. (Which has now been fixed by Daniel Otykier.) The temporary solution was to specify the full file path, including directory, when loading the assembly. It would seem to be a similar problem here as I can see the assembly is included in the installation dir. Maybe this helps 🤷♂.
Config file attached: ModelDocumenter.exe.config.txt

Hello,
First congratulation for your project : it's gonna be really useful for many users.
I tried the tool this morning. Unfortunately it didn't work. What I could see :

I have my shortcut on my ribbon but when I click on the button, it ends very fact
"C:\Power BI Model Documenter" was not created during installation (or test) =>  I created it manually to check it could fix my issue but It did not fix anything. After a rerun, still no file inside this folder
No pbit file generated in 'C:\Power BI Model Documenter folder' + no PBIT file generated in "C:\Program Files (x86)\Data-Marc\ModelDocumenter"
same log as in the pricture above "unhandled exception: system.IO.FileLoadException: Could not load 'or assembly Microsoft.Analysis Services, version ..."

My ModelDocumenter.exe.config :

If you want me to test anything, just let me know.
Regards,
Philippe

@FilK78 Thanks for flagging. We identified the issue and we are working on a fix as we speak.

Thanks @pugliathomas @samaguire @orsim25 @FilK78 for bringing this up!
I managed to reproduce on a clean VM. @tonswart helped me to fix the issue directly and we just published v2.0.1 which you can download here In this new version the issue is resolved.

Thank you for the help, everyone!

Thanks @pugliathomas @samaguire @orsim25 @FilK78 for bringing this up! I managed to reproduce on a clean VM. @tonswart helped me to fix the issue directly and we just published v2.0.1 which you can download here In this new version the issue is resolved.

Working now. Cheers!

Good job. I tested and it works perfectly for me 👍

I downloaded the vrrsion 2.01
I installed over the older version
I tested on a simple file ( my pbix I use for my demo)

It's really useful : entering description for measures /tables now make sense.
Question: I can see you retrieved Power BI description for table and measures but not for columns. Is it a choice ?
Thanks for job, I'm really impressed.

Thanks for circling back @FilK78, great to hear its working now!
On the descriptions and columns, it is not a choice... I might have forgotten that. I will list it for future enhancements to also include column descriptions.
Thanks for the feedback! Keep it coming :)


Deleting project fails sometimes

Describe the bug
A clear and concise description of what the bug is.
Requirements for and issue

[ ] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue)
[x] Your version of artifactory (you can curl it at $host/artifactory/api/system/version
JFrog Platform SaaS
[x] Your version of terraform
v1.5.7

How to reproducte

Create terraform config with 1 project that links to 3 virtual, 3 remote and 3 local repos using the repos key.
Apply config. Project is successfully created and has all 9 repos listed
Perform terraform destroy
In arround 30% of executions there is the error:

400 DELETE https://xxxxxx.jfrog.io/access/api/v1/projects/project_name
{
  "errors" : [ {
    "code" : "BAD_REQUEST",
    "message" : "project containing resources can't be removed"
  } ]
}

Checking in the UI you can see that the unassignment of the repos did not work. One repo still shows the project that should have been deleted.

Workaround
Execute destroy again and it will work.
Additional context

@pho-enix Thanks for the report! Looks like I may need to add some retry code for deletion. (We have similar issue with repos in the Artifactory provider so this isn't unknown.) I'll add this to our plan.


Quoted properties fail in New-HTMLTable when using DataStore Javascript

$Test = [PSCustomObject] @{
    'This is "A test", " for more tests"' = "Hello"
    "ABC"                                 = "123"
}

$Test | Out-HtmlView -DataStore JavaScript -Filtering
$Test | Out-HtmlView -Filtering

Fixed in


Release

./gradlew publish command is failing with these changes in Gradle build files

I have built  the new versions and I'm getting
java.lang.AbstractMethodError: abstract method "void com.sdk.growthbook.network.NetworkDispatcher.setLoggingEnabled(boolean)"


have attachment of content type image to use the image type instead of artifact

TeamCity has an image type for test reports, which is useful for showing screenshots directly on TC.
Let me know if anything is missing to this PR, and I will gladly add it

@artemrudenko did you get the time to look at this PR?

@artemrudenko did you get the time to look at this PR?

Hi @massintha-azamoum , sure, looks good to me, will try to add it in the nearest days


Unit tests in test_remote_launcher_path are commented out

Describe the bug
All tests in test_remote_launcher_path are commented out. Is this intended?
To reproduce
Steps to reproduce the behavior:

Go to test_remote_launcher_path

Expected behavior
Unit tests are not commented out.

Hello, good callout!
I suppose the tests are commented out (they should rather be marked to be skipped ...) because they use AWS. We run such tests as part of integ-tests.yml or end-to-end-tests.yml, but they should not be part of the unit test suite (which is supposed to run locally, without AWS, and each test is limited to a few secs).

Path to solution:

This code should not be a unit test
It could be run as part of integ-tests.yml or end-to-end-tests.yml

Closed by #810


Consider creating new Unicode exercise

Previous discussion at https://github.com/exercism/xgo/issues/195.  The Unicode test case was removed from ledger to limit the scope of that exercise.  But maybe a new exercise could be created around the idea of that test case.  I imagine the original intent of the test case was to show the difference between the bytes of a string and the runes of a string.  This is a good start.  The Rosetta Code task String length  introduces the additional concept of the graphemes of a string.  Formatting text in a fixed-pitch font (the subject of ledger) requires yet another width concept ("cell" width, I think) when full-width asian characters are used.  And there's more, but that's probably enough for a good exercise.

Since a new Unicode exercise could be applied to all tracks, I would say this issue should go to x-common. and it's great timing too, since we are talking about that at https://github.com/exercism/x-common/issues/428 ! I'll close this and drop a note in that issue with what I know.


add stand alone login() method to teslams.js which fetches token

Some people would prefer to manage their own login tokens rather than request a new one each time their apps start.
I should add into teslams.js an explicit login() method that sets the global variable for the token and does nothing else. This is done now, only when you call vehicles() or all()
Modify teslacmd.js to have an option to print out the token.
Modify config.js to check the ~/.teslams.json file for a token that the user has saved instead of their username and password.
Modify vehicles()  and all() to check if the global token variable is already set and if so, skip the login part of the code. May enhance this later to check for expired tokens.

Hi - sorry I am a little confused by this statement.  I am calling the https://owner-api.teslamotors.com/oauth/token service directly and getting and storing the response encrypted internally in my app to have access to the access and refresh tokens..
{
"access_token": "abc123",
"token_type": "bearer",
"expires_in": 7776000,
"refresh_token": "456zyw",
"created_at":<PHONE_NUMBER>
}
I would like to use this library to work with the rest of the api, but want to pass in an access_token initially or on each call is fine, and NOT The username / password.  Is this possible and being covered with this issue or should I open a separate issue to track it?

I interpret Hans' comments to mean what you said, that you could provide your own token.  Though it is not entirely clear whether you can provide the token with each call or whether you must set the token as global state.
There are other similar wrapper libraries for example (full disclosure: this one is my own) TeslaJS that are stateless and offer the ability to pass the token with each call.

Thank you mseminatore  I will check out your library, looks nice.

The auth token is stored once in the variable "http_header". You can see an example of how the http_header is set here https://github.com/hjespers/teslams/blob/master/teslams.js#L54
The only functions in this library that use login and password are all(), vehicles(), and get_vid(). If you already have a token and know your vehicle_id then you can skip calling these functions, set up http_header yourself and all the other functions should work.
This enhancement request was a note to myself to add explicit functions to setToken() and getToken() that would make it easier for developing.

I added in a set_token() method and expanded the teslacmd example to show how to use it.
Just call set_token( id, token) where id is the id_s string of the car and token is a valid authentication bearer token. This will then cause all subsequent calls to the API to use the token specified. I have not published the change to npm yet so pull from github and let me know if it works for you.

Not sure what I was thinking but "id" is not required for set_token. Just call teslams.set_token( token )

finished implementation of tokens and published to npm as teslams version 1.2.2
all the examples should now take --token as an argument or you can store a token in your ~/.teslams/config.json in place of username and password using the following format:
{
"token": "abc123abc123abc123abc123abc123abc123abc123abc123abc123abc123"
}

Thanks hjespers!  exactly what was required :)

Hi hjespers, just getting back to this, and I actually found using set token isn't working.. here is my usage...tried setting before and on call.....
teslams.set_token( process.env.TSLA_AUTH_TOKEN )
teslams.get_vid( { token: process.env.TSLA_AUTH_TOKEN }, function ( id ) {
console.log("Got VID " + id)
//teslams.get_vehicle_state( id , function ( state) {
//	console.log( util.inspect( state ) );
//});
teslams.get_charge_state( id , function ( state) {
//console.log( util.inspect( state ) );
console.log("Your Tesla is " + state["charging_state"] + " and charged for a range of " + state["battery_range"] + " miles" );
});
});

If you want to see a working example, the application examples/teslacmd.js uses set_token() and can set it either from a command line argument called --token or via the config.js file in your ~/.teslams directory. The code for the second method uses config.js which could be extended to read from an environment variable as in your example. Once you call set_token you can then read it current value from the global variable teslams.token. The teslacmd.js app also has a --print_token flag to print out this global variable on the command line.


Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. Received protocol 'https:'

Environment

Operating System: Windows_NT
Node Version:     v18.15.0
Nuxt Version:     3.10.0
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      ssr, devtools, css, plugins, devServerHandlers, postcss, modules, content, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/Foddie2/TechBytes
Describe the bug
Getting this error when in production
Additional context
No response
Logs
No response

Would you see if you can provide a more minimal reproduction? 🙏


Proposed fix for CAMEL-14174

Prevent listing more objects than the value defined in maxMessagesPerPoll during the route initialization

Thanks can you update the documentation as well, as it says its default unlimited, but the default value is 10.

Thanks can you update the documentation as well, as it says its default unlimited, but the default value is 10.

Yes I can.

The docs needs to be modified at javadoc level and then rebuild

Ahhhh, I wasn't aware of that. I followed the steps on the contributing guide :D
I'll ping you later for some directions. Thanks!!!


Suggesting another example app

Add another example app

Nice example, but I don't know if it is too extensive to be an example.

@kevinsimper Yeah, but you already have lite examples. I believe such example can be very useful.

@VladimirPal I agree, I'm going to merge it and release this with 0.2.8.

merged in c36d6c3


TLS: add module to store TLS certs in Secret Manager

can accept an existing TLS pair
will generate a self-signed pair if key and/or cert are unspecified

Closes #118

Code is almost identical to the one I use in automated build pipelines for NGINX+ images, and after rebasing on main I can see that @vinnie357 does something similar for his N+ stuff.
We should try to combine


FindLibRaw: fix access to the version info thanks to LibRaw_INCLUDE_DIR

Hello, and thank you for your amazing io library!
Description
Fix issues to detect libraw library and compile the corresponding plugins.
According to the includes found in src/raw.imageio/*.cpp, the
LibRaw_INCLUDE_DIR should be a path to 'libraw/libraw_version.h'.
Tests
libraw is not installed on my computer. I compile it locally, and
indicate the path to the local installation to CMake.
Checklist:

[x] I have read the contribution guidelines.
[ ] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[ ] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Sorry for the delay. This looks good to me.
I'm making one change: it appears that line 27 should also be changed:
FIND_PATH(LibRaw_INCLUDE_DIR libraw/libraw.h

...added the libraw subdirectory to the file being searched for. Without that, it fails for me.
I will merge into 1.7 and master.

Merged.

Indeed I've forgotten to change the FIND_PATH line 27...
Thank you for the debug & merge :thumbsup:


[Question] What is the envoy version that istio uses

Hi,
What is the envoy version that istio 1.2.3 uses and how do I find out for future releases on my own?
Regards.

Ok so from what I see here - istio uses the custom wasm playgroud of envoy?

Thanks for the quick answer it provides me with enough information in order to find out which envoy proxy version from the official repo it uses. Regards


BottomNavigationTab Ripple Effect

How to add ripple effect in BottomNavigationTab ?
I tried adding TouchableOpacity, TouchableHighlights and etcc.. zIndex also not working
Update: I found a way but its very ugly. I have to wrap each components inside TouchableOpacity
<TouchableOpacity
          style={{
            alignItems: 'center',
          }}
          onPress={onClickBottomIndex}
        >
          <BottomNavigationTab
            style={{ zIndex: 0 }}
            onPressIn={onClickBottomIndex}
            title={props => (
              <TouchableOpacity
                style={{
                  alignItems: 'center',
                }}
                onPress={onClickBottomIndex}
              >
                <Text {...props}>Nearby</Text>
              </TouchableOpacity>
            )}
            icon={props => (
              <TouchableOpacity
                style={{
                  alignItems: 'center',
                }}
                onPress={onClickBottomIndex}
              >
                <MIcon
                  name="map-marker-circle"
                  size={25}
                />
              </TouchableOpacity>
            )}
          />
        </TouchableOpacity>

Hey @rosnaib11 :wave:
We don't support the Ripple effect out of the box and it's more likely will never be supported. That said, I'm going to close this. Thanks for updating the issue, I hope this will help someone in the future.


Introduce HD wallet v2

We need to introduce a new version to make the wallet compliant with future hardware wallet:

Fix the index derivation from which we derive keys
Introduce an intermediate segment to ensure we are not closing doors for later improvements

Initial spec pseudo code for reference:
fun BIP39-Seed (Mnemonic, WalletPassword = "") -> Seed
  H = HMAC-SHA-512
  Salt = "mnemonic" || WalletPassword
  Password = mnemonic
  Iterations = 2048
  KeyBytes = 64

  Seed = PBKDF2(H, Salt, Password, Iterations, KeyBytes)

fun MasterNode (Seed) -> MasterChainCode, MasterKey
  Key = "ed25519 seed"
  Data = seed
  I = HMAC-SHA-512(Key, Data)

  MasterChainCode = I[0:32]
  MasterKey = I[32:64]

fun HardenedChildNode (ParentChainCode, ParentKey, Index) -> ChildChainCode, ChildKey
  Key = ParentChainCode
  Data = 0x00 || ParentKey || U32BE(0x8000_0000 | Index)
  I = HMAC-SHA-512(Key, Data)

  ChildChainCode = I[0:32]
  ChildKey = I[32:64]

fun HDWallet (Mnemonic) -> Derive
  Seed = BIP32-Seed(Mnemonic)
  // Path: "m"
  MasterChainCode, MasterKey = MasterNode(Seed)

  // Vega magic
  VEGA = 1789
  // Path: "m/'MAGIC"
  VegaChainCode, VegaKey = HardenedChildNode(MasterChainCode, MasterKey, VEGA)

  // Default sub node
  ChainCode, Key = HardenedChildNode(VegaChainCode, VegaKey, 0)

  // Path: "m/'VEGA/'0/Index"
  fun Derive (Index) -> PrivateKey
    _, PrivateKey = HardenedChildNode(ChainCode, Key, Index)


Updating from vue2 to vue3.

I am updating my project from vue2 to vue3 and in that I am getting issue in Drawflow initially because importing vue as we were doing in vue2 and then i have tried making change as per the suggestion I found but getting issue related to the addNode, registerNode e.t.c.
The initial target it to only make minimum changes to make it work on vue3 and once everything works fine then we can update thing in more optimized ways.

View example implementation with vue3

Repo: https://github.com/jerosoler/drawflow-vue3-example
Demo: https://jerosoler.github.io/drawflow-vue3-example/

There is a change of structure involved, can you provide(point towards) a documentation where we have pointer on how to update drawflow when updating from vue2 to vue3.

In principle the only change would be the one provided in the documentation.
https://github.com/jerosoler/Drawflow#for-vue-2-example
You can look in this file, the implementation:
https://github.com/jerosoler/drawflow-vue3-example/blob/master/src/components/drawflow.vue


Themes - Tukan/Cicada - modal fix for 19.7

modal background-color fix for both themes

Merged, thanks!


[fix][build] Include pulsar-common dependency

Motivation
pulsar-client-tools-test  and pulsar-functions-instance  depend on pulsar-common module
like
org.apache.pulsar.common.sasl.SaslConstants org.apache.pulsar.common.policies.data.OffloadPoliciesImpl
Modifications
include pulsar-common module in pom.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
https://github.com/gaozhangmin/pulsar/pull/3

Codecov Report

Merging #18717 (14ca2d4) into master (bbc7a25) will decrease coverage by 11.26%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #18717       +/-   ##
=============================================
- Coverage     48.34%   37.08%   -11.27%
+ Complexity     9368     1968     -7400
=============================================
  Files           618      209      -409
  Lines         58586    14421    -44165
  Branches       6097     1573     -4524
=============================================
- Hits          28326     5348    -22978
+ Misses        27078     8487    -18591
+ Partials       3182      586     -2596

Flag
Coverage Δ

unittests
37.08% <ø> (-11.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../org/apache/pulsar/client/impl/ConnectionPool.java
37.43% <0.00%> (-1.03%)
:arrow_down:

...ava/org/apache/pulsar/broker/admin/v3/Sources.java

...ar/broker/protocol/ProtocolHandlerDefinitions.java

.../apache/pulsar/common/naming/NamespaceBundles.java

.../broker/loadbalance/impl/PulsarServiceRequest.java

.../apache/pulsar/broker/admin/v2/ResourceQuotas.java

...broker/service/schema/BookkeeperSchemaStorage.java

...rvice/schema/KeyValueSchemaCompatibilityCheck.java

...istentStreamingDispatcherSingleActiveConsumer.java

...r/service/AbstractDispatcherMultipleConsumers.java

... and 400 more


Does instance normalisation support in current version of texture_nets?

Hi,
Does the current version texture_nets support instance normalisation? I could not find function like that in the project, instead, a BatchNormalisation found in the utils.lua.
Best

If you use batch_norm separately for each object it will be equal to instance norm. It is exactly what is happening in the code.

Would you be able to briefly describe how to use batch_norm separately for each object? thanks!

Look at
https://github.com/DmitryUlyanov/texture_nets/blob/master/train.lua#L113
Instead of doing forward for B x C x W x H tensor do B times for 1 x C x W x H

@DmitryUlyanov Greak! thanks a lot!


ziomon: fix typo

Signed-off-by: Rafael Fonseca<EMAIL_ADDRESS>

@r4f4 Thanks for the fix, but from the patch (at least) I don't understand what exactly the problem/symptom is.

In the case of check_host_param' and init_adapters', it won't detect the absence of 'queue_full' in an old kernel installation since the stat call will always succeed at that point because it's checking against a path that was already verified to exist. Now, if that's going to cause any visible symptoms I can't tell since I'm not familiar enough with the code.

Nice catch, these seem to be copy&paste mistakes. I think the changes are fine, but Steffen is the maintainer of the tools.
There is no impact on how the tools work, but in case the 'queue_full' attribute doesn't exist for a very old kernels, they make ziomon exit gracefully.

I'll update the patch with a rationale if you think it's worth merging.

Well, as long as the code is still in ziomon - and I don't see it disappearing anytime soon - it would be good to have it correct :-). Even though I'm not sure anyone still uses such old kernels where this attribute doesn't already exist.

I updated the patch description. Thank you for looking into it @michael-holzheu and @Benjamin-Block.

I updated the patch description. Thank you for looking into it @michael-holzheu and @Benjamin-Block.

Looks good to me now :thumbsup:, thanks!

@r4f4 thanks for the fix, will integrate it


Update tweaks.json

Capitalized first letter of "path" --> "Path" keys
See json schemas for validation (See Test-Json) (available at pwsh)
schema.json.zip

Thanks for fixing that formatting issue.


Ajax example based on jQuery

I not seen ajax example and decided to add.

Coverage remained the same at 99.6% when pulling 90ab09be28054b31d438252509f378233b0224ea on DarkScorpion:ajax-jquery into 63ab25579bda70b4927a179b580a9c580b6c7ada on strongloop:master.

I'd go as far to say this example isn't required. jQuery is a huge overhead to any application and the example provided here shows nothing that vanilla JS can do.
While it's a great example, I think it would add more complexity to the /examples dir.
How about we close this one up for now @dougwilson ?

Hi @notrab that sounds good. The search example already shows AJAX usage 👍


feat(CiceroMarkToOOXML): assign heading styles to different levels

Signed-off-by: Aman Sharma<EMAIL_ADDRESS>I have added heading styles to heading instead of assigning font sizes.

Currently,

Heading size is 13 with its default colour which comes with style Heading2 of Word.
Body size is 14.

I would like some suggestions on the colour of the heading and its size.

@irmerk @DianaLease I will let you both know by tomorrow if this makes parsing easy. I have a feeling it will! 😌

Very cool! I would think the headings stay the same color - black. Also, maybe we could make the body text a smaller size?

yeah let's make sure the body size is such that there are about 95 characters per line, consistent with the contract editor
I think it would also make sense to make the heading styles consistent with how they look in the contract editor. The font could be whatever Microsoft Word uses by default, but in terms of bolding and size, I think we should be consistent with the contract editor.

@Michael-Grover

Heading
Font Size

Heading 1
25px

Heading 2
20px

Heading 3
16px

CiceroMark has 6 levels for heading but the Contract Editor only uses 3. What font sizes do you suggest for the rest of the Headings?

I think 6 headings is way too many, and ideally I would like to avoid users using 6 different heading styles in their contracts. I've never seen a real contract that even uses three different heading styles, let alone six. However if we have to define styling for six different headings, I think we should either

make h4,5,6 all 16px to be the same as H3, so that there are only effectively 3 headings styles
Make h4 15px, h5 14px, and h6 13px

Which of these makes more sense? @DianaLease @irmerk @algomaster99 @jeromesimeon

I think 6 headings is way too many, and ideally, I would like to avoid users using 6 different heading styles in their contracts.

CiceroMark defines 6 levels of headings that's why we have to define it.

make h4,5,6 all 16px to be the same as H3, so that there are only effectively 3 headings styles
Make h4 15px, h5 14px, and h6 13px

You mean to make h1, h2 and h3 16px? Yes, that makes sense to me. What size will be of the body?

I think h1-h3 should stay 25, 20, and 16, respectively. Those are the header styles people will be most likely to use.
I'm suggesting what size we should make H4, H5, and H6, if we really have to define them.
I think two options are:

Make H4,H5,H6 all 16px so that they have the same appearance as H3, so that there are effectively only 3 heading styles
or
Make H4 15px,      H5 14px,      and       H6 13px

and I think 12px like @DianaLease suggested makes sense for body text


"the lock file needs to be updated but --locked was passed to prevent this" error message is unhelpful

While working on Stylo, several times I have hit this error message:

error: the lock file needs to be updated but --locked was passed to prevent this

This is an unhelpful error message. At the very least, it should print the path to the lock file, because when building Stylo there are multiple lock files and it's not obvious which one is the problem.
If it can print more than that -- e.g. identifying how the lock file needs changing -- that would be even better.

I have the same issue

This is still relevant.
Also, I'm not a fan of closing bugs/PRs merely due to inactivity. It's too close to CADT for comfort: https://www.jwz.org/blog/2003/02/the-cadt-model/.

I think if someone shared a reproduction of this issue it would help it being resolved.

Lovely, thank you @ehuss!
Perhaps I'll be able to solve this (no promises).


chore: Remove docker cargo test in favor of new flow

Description
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

where is the new flow?


Update the "Chart rendering" app screenshot

This PR polishes the "Chart rendering" app screenshot by clipping some empty space at the bottom of the screen.

@armiol PTAL.


HTML code displayed in template view of TinyMCE input

Hi, thank you for your hard work, I am having problems (web development is very new to me), but I can see that this is a very useful tool.
I have installed django-tinymce4-lite==1.7.0 along with django-filebrowser-no-grappelli==3.7.4. I am able to clearly see the TinyMCE browser to assist with formatting within the admin module, but any changes I make are then displayed with the HTML used. For example, I added bold to the first line of my test blog and I see the following on the page:
"Also testing a different sized image. "
EDIT: added an image of what I can see as the above quote simply changed into how I expected to view it.

I am using Django 2.0.2 and Python version 3.6.4. I'm not sure what I've missed?
One other quick question, I may be experiencing some issues with the media file location, but just to clarify, when trying to insert an image TinyMCE requests "Source", can I use django tags, i.e. {% static 'images/AnImage.png' %}?
Thanks in advance for your time.

It seems that your HTML content is auto-escaped by Django. You need to carefully read the documentation:  http://romanvm.github.io/django-tinymce4-lite/usage.html

No, you cannot use Django tags in TinyMCE content because this content is not the part of a template.

Thank you for getting back so swiftly. I thought I had read the documentation carefully, I coded the following:
{{ post.body|truncatewords:10|linebreaks**|safe** }}
This was, as you have stated, being auto escaped. I have moved the |safe element next to the post.body and I can see now that the HTML content has gone. Thank you for that.
I am now getting strange behaviour on the formatting of this page, but I will try to work that out before bugging anyone.


query: forward query plan in the remote query request

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Serialize the query plan and forward it in the Remote query request to allow further optimizations in the remote engines executing the queries.
Verification
We have been running this version on production for about a week now, also I changed tests.

I keep getting unrelated errors in tests. Going to rebase again to see if it clears out.
Failed test:
--- FAIL: TestMemcachedClient_SetAsync_CircuitBreaker (0.14s)
    --- FAIL: TestMemcachedClient_SetAsync_CircuitBreaker/opened_because_failure_percent_too_high (0.06s)
        testutil.go:91: memcached_client_test.go:753: "state should be open"

                exp: 2

                got: 1


Sine example fails

The sine example produces only glitchy stuttering for me and appears to crash. Console output:
2 backends available.
Connected to backend: ALSA
ALSA lib pcm_dmix.c:968:(snd_pcm_dmix_open) The dmix plugin supports only playback stream
ALSA lib pcm_dsnoop.c:545:(snd_pcm_dsnoop_open) The dsnoop plugin supports only capture stream
ALSA lib pcm_dsnoop.c:545:(snd_pcm_dsnoop_open) The dsnoop plugin supports only capture stream
Using output device: Playback/recording through the PulseAudio sound server
Output format: float 32-bit LE
Sample rate: 48000
err: unrecoverable streaming failure
Output channel layout: Stereo
Pause for 1s
Unpausing
sine: [...]/rsoundio/target/debug/build/rsoundio-4469f7eb07188f71/out/libsoundio-1.1.0/src/soundio.c:394: soundio_device_unref: Assertion `device->ref_count >= 0' failed.

You can see that I inserted a print statement to see the number of available backends, but other than ALSA, it's just the Dummy. I also tried changing the SioFormat to Float64LE, but that didn't work.
Libsoundio's sio_sine test runs fine and sounds clear. It also reports a different backend that's not available in rsoundio:
Backend: PulseAudio
Output device: Built-in Audio Analog Stereo
Software latency: 2.000000

uname -a is Linux [computer name] 4.4.0-62-generic #83-Ubuntu SMP Wed Jan 18 14:10:15 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux and the rustc version is 1.15.1 (021bd294c 2017-02-08)
I'm new to audio programming, so I hope this is helpful and not just a simple mistake on my side. If this issue is missing any information, I'm happy to clarify.

Thanks for opening the issue, there is a problem with the PulseAudio backend but I haven't found the problem, yet. See #7 for details.

Oops, looks like ALSA is broken, too. Can you try JACK?
I've used rsoundio with JACK and it works fine.

Thanks for your quick response! So, I installed JACK (apt install jackd):
The following additional packages will be installed:
  jackd2 jackd2-firewire libconfig++9v5 libffado2 libjack-jackd2-0 libxml++2.6-2v5
  python-dbus python-gi qjackctl
Suggested packages:
  jack-tools meterbridge python-dbus-doc python-dbus-dbg python-gi-cairo
The following packages will be REMOVED:
  libjack-dev libjack0 librtaudio-dev
The following NEW packages will be installed:
  jackd jackd2 jackd2-firewire libconfig++9v5 libffado2 libjack-jackd2-0
  libxml++2.6-2v5 python-dbus python-gi qjackctl

This caused another backend to be available in rsoundio, but it was PulseAudio, not JACK. I'm pretty confused as to why. PulseAudio actually produces a sine tone. It does have some crackling, unlike sio_sine, and the crackling gets a lot worse with --release turned on. The crash at the end still happens with PulseAudio.
JACK on the other hand didn't show up in sio_sine nor rsoundio. What I did was start qtjackctl and press start, which spins up three jackdbus processes and blocks all other sound drivers.
As an aside: I thought maybe I have to rebuild libsoundio for JACK to be recognized by it, but JACK is also not found by the libsoundio CMake script:
System Dependencies
-------------------
* threads                      : OK
* JACK       (optional)        : not found
* PulseAudio (optional)        : OK
* ALSA       (optional)        : OK
* CoreAudio  (optional)        : not found
* WASAPI     (optional)        : not found

The libjack-jackd2-dev package was what was missing for libsoundio to compile with JACK; playing a sine with ./sio_sine --backend jack works now. Still, only PulseAudio, ALSA and Dummy show up in rsoundio.

rsoundio links against a system libsoundio installation and if none was found it builds one.
The sole purpose of the build.rs was to have libsoundio available in my CI integration container. It build libsoundio in debug mode and w/o JACK and PulseAudio because the CI server has no sound device and only uses the dummy interface.

You're right, I needed to do sudo make install.after rebuilding libsoundio with JACK. JACK works fine in rsoundio, no popping or crackling. The failed assertion, however, is still there.
By the way, my problem with PulseAudio is not identical to the one in #7, for me it's just crackling, although both might of course have a common cause. ALSA also has different symptoms.

I would love to fix this issue, actually I planned to do for quite some time, but I am quite busy with work and other things.
Nonetheless, I have this issue on my radar will fix it sooner or later.

I understand, I am in a similar position or I would try looking into it myself.

Great, don't hesitate to ask if you've any questions.
There are also a lot of thinks to improve, e.g. there is still some memory allocations in the write callback.


[Doc] Java Client Cluster-Level Failover sample code

Search before asking

[X] I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?
I see that in documents 2.10.x, part Java Client - Cluster-level Failover
The sample code looks difference with the real API
secondaryTlsTrustCertsFilePath expects a Map<String,String>, but there's just a String
secondaryAuthentication expects a Map<String, Authen>, but there's both String.

What is your suggestion?
So what's exactly I need to put to? What is the key in Map should be?
Could someone please point me out what am I missing here
Thanks
Any reference?
Java Client - Cluster-level Failover
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@labuladong @hangc0276 can you pls take a look at this question?

Sorry, I missed this issue. I'll fix this error.


Motorstart-Messages again and again...

after starting the debian-java-mqtt-service via "systemctl start saic-mqtt-gateway.service" the status looks good
piuser@heizung:~ $ sudo systemctl status saic-mqtt-gateway.service
● saic-mqtt-gateway.service - The SAIC MQTT Gateway
   Loaded: loaded (/lib/systemd/system/saic-mqtt-gateway.service; enabled; vendor preset: enabled)
   Active: active (running) since Mon 2023-08-28 13:28:41 CEST; 3h 16min ago
 Main PID: 3415 (java)
    Tasks: 17 (limit: 2059)
   CGroup: /system.slice/saic-mqtt-gateway.service
           └─3415 /usr/bin/java -jar /opt/saic-mqtt-gateway/saic-mqtt-gateway.jar --config /etc/saic-mqtt-gateway.toml

Aug 28 13:28:41 heizung systemd[1]: Started The SAIC MQTT Gateway.
Aug 28 13:29:01 heizung java[3415]: 2023-08-28 13:29:01:658 +0200 [main] INFO SaicMqttGateway - Registered for abnormal messages
Aug 28 13:29:02 heizung java[3415]: 2023-08-28 13:29:02:490 +0200 [main] INFO SaicMqttGateway - Registered for moving messages
Aug 28 13:29:03 heizung java[3415]: 2023-08-28 13:29:03:330 +0200 [main] INFO SaicMqttGateway - Registered for region messages
Aug 28 13:29:04 heizung java[3415]: 2023-08-28 13:29:04:088 +0200 [main] INFO SaicMqttGateway - Registered for engineStart messages
Aug 28 13:29:04 heizung java[3415]: 2023-08-28 13:29:04:733 +0200 [main] INFO SaicMqttGateway - Registered for startVehicleStatus messages
Aug 28 13:29:05 heizung java[3415]: 2023-08-28 13:29:05:447 +0200 [main] INFO SaicMqttGateway - Registered for offCar messages
Aug 28 13:29:06 heizung java[3415]: 2023-08-28 13:29:06:148 +0200 [main] INFO SaicMqttGateway - Registered for speeding messages

and after a short while i see mqtt-messages coming in.
After 15-20mins the soc-related mqtt-messages stop but this some messages are repeated over and over after only 1-2 seconds...
<EMAIL_ADDRESS>{
  "content":"Das Fahrzeug (***456) startet am 28-08-2023 16:03:36 GMT+02:00, bitte zur Kenntnisnahme bestätigen.",
  "messageId":4857080,
  "messageTime":"2023-08-28T16:03:36+02:00[Europe/Berlin]",
  "messageType":"323",
  "readStatus":0,
  "sender":"TBOX",
  "title":"Motorstart",
  "vin":"LSJWH4091PN123456"
}
<EMAIL_ADDRESS>{
  "content":"Das Fahrzeug (***456) startet am 28-08-2023 10:38:22 GMT+02:00, bitte zur Kenntnisnahme bestätigen.",
  "messageId":4834516,
  "messageTime":"2023-08-28T10:38:22+02:00[Europe/Berlin]",
  "messageType":"323",
  "readStatus":0,
  "sender":"TBOX",
  "title":"Motorstart",
  "vin":"LSJWH4091PN123456"
}
<EMAIL_ADDRESS>{
  "content":"Das Fahrzeug (***456) startet am 28-08-2023 10:35:09 GMT+02:00, bitte zur Kenntnisnahme bestätigen.",
  "messageId":4834319,
  "messageTime":"2023-08-28T10:35:09+02:00[Europe/Berlin]",
  "messageType":"323",
  "readStatus":0,
  "sender":"TBOX",
  "title":"Motorstart",
  "vin":"LSJWH4091PN123456"
}
<EMAIL_ADDRESS>{
  "content":"Das Fahrzeug (***456) startet am 28-08-2023 10:26:07 GMT+02:00, bitte zur Kenntnisnahme bestätigen.",
  "messageId":4833765,
  "messageTime":"2023-08-28T10:26:07+02:00[Europe/Berlin]",
  "messageType":"323",
  "readStatus":0,
  "sender":"TBOX",
  "title":"Motorstart",
  "vin":"LSJWH4091PN123456"
}
<EMAIL_ADDRESS>{
  "content":"Das Fahrzeug (***456) startet am 28-08-2023 09:50:48 GMT+02:00, bitte zur Kenntnisnahme bestätigen.",
  "messageId":4831615,
  "messageTime":"2023-08-28T09:50:48+02:00[Europe/Berlin]",
  "messageType":"323",
  "readStatus":0,
  "sender":"TBOX",
  "title":"Motorstart",
  "vin":"LSJWH4091PN123456"
}

How to stop this behaviour ?
Are the messages coming from the saic-cloud or from my vehicle and do they drain my 12V-battery ???
Ove

Those messages are used to detect if the car has been started. They are coming from the SAIC servers, the car is not queried for them and is not woken up, so this doesn't affect your 12V battery.


Missing FontFaceSet

Spec definition available at https://developer.mozilla.org/en-US/docs/Web/API/FontFaceSet
It doesn't appear to be in kotlinx.browser

This MR is closed because of this repository is now used for Kotlin WasmJs implementation. The Kotlin JS platform implementation we have no plans to yet.


Some instructions unclear and a missing important note

Instructions Unclear
In section "Create a pipeline with a Data Flow activity", steps 3 and 4 are unclear; it seems something change or a step was missed in the instructions.
In step 3, we are adding a Data Flow Activity. However this does not automatically create a DataFlow so step 4 which says "In the Adding Data Flow pop-up", there is no pop-up to speak of after completing step 3. One has to assume it is meant to leave the dataflow Activity named "Data flow1" and click the Settings tab of the activity and then click New button to the right of the Data flow Select drop-down and name that dataflow "TransformMovies". As a beginner tutorial, this should not be skipped as it can be confusing.

Missing Important Note
At "Build transformation logic in the data flow canvas" Step 6 it should be noted that the Storage Account used must have Soft Delete disabled otherwise the Linked Service cannot connect to the Storage Account.
Thanks!

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

ID: f7c75ee8-d021-62fe-c7e3-53a6e729e1fc
Version Independent ID: 7ae45fa9-cb4f-dbc5-dcb6-4a4f4c040d28
Content: Transform data using a mapping data flow - Azure Data Factory
Content Source: articles/data-factory/tutorial-data-flow.md
Service: data-factory
Sub-service: data-flows
GitHub Login: @kromerm
Microsoft Alias: makromer

Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

We have assigned the issue to the content author to further evaluate and update the document as appropriate.
Thank you

#please-close


ST&E Ops Page Performance

In the latest 1.3.3 dev release (11 Sep 18), the ST&E Ops page performance has taken a huge hit, to the point where it is hardly usable.  Performance starts out fine, but as you add more hosts and more results, it gets exponentially slower.
There are no error messages.  We had one system with about 40 hosts taking literally minutes to load the Ops page or expand a category.  We have reproduced on several systems.

But can really make a tool unusable...
On Mon, Sep 17, 2018 at 8:27 AM Ryan P<EMAIL_ADDRESS>wrote:

Performance stuff is usually really hard to figure out.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cyberperspectives/sagacity/issues/12#issuecomment-422038384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aoyyh1n645TQDqjOl9UOz_3iob-_bT-8ks5ub7FOgaJpZM4WoD_d
.

--

Jeff

Jeff Odegard
<EMAIL_ADDRESS>[image: LinkedIn] https://www.linkedin.com/in/jeffodegard/ [image:
Fakebook] https://www.facebook.com/jeff.odegard.98 [image: YouTube]
https://www.youtube.com/user/OdegardOnline

I just tried to get 40 hosts from a single category (just added 20 more targets from that test eChecklist file you sent me).  Took 5 seconds.  So I loaded that same category with 100 targets and it took ~10 seconds.  I don't think that is unreasonable.  I'm going to need more information about where you are seeing a majority of the slow down coming from so I can figure out what if anything I can do to speed it up.

And, that is running on an old i5 3350p (4 core @ 3.1Ghz) processor that is going on 6 years old (bought it Nov '12).

I am reinstalling at home and will try to reproduce tonight so I can show
you during our meeting tomorrow.
On Mon, Sep 17, 2018 at 5:27 PM Ryan P<EMAIL_ADDRESS>wrote:

I just tried to get 40 hosts from a single category (just added 20 more
targets from that test eChecklist file you sent me). Took 5 seconds. So I
loaded that same category with 100 targets and it took ~10 seconds. I don't
think that is unreasonable. I'm going to need more information about where
you are seeing a majority of the slow down coming from so I can figure out
what if anything I can do to speed it up.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/cyberperspectives/sagacity/issues/12#issuecomment-422203784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aoyyh_STURYAz46pALEeV_nzKQHas5Q5ks5ucC_UgaJpZM4WoD_d
.

--

Jeff

Jeff Odegard
<EMAIL_ADDRESS>[image: LinkedIn] https://www.linkedin.com/in/jeffodegard/ [image:
Fakebook] https://www.facebook.com/jeff.odegard.98 [image: YouTube]
https://www.youtube.com/user/OdegardOnline

Last commit should fix

This appears to be fixed.  We will be doing further testing, but it appears to be working normally now.


combine data files into one file

combine the file and have the liquid code sort and show the appropriate sites based on size
@kevquirk @lrmikestone
Before this is merged will make the final commit in creating a sites.yml file that merges the 3 files in alphabetical order.

This is a great source for liquid commands https://shopify.github.io/liquid/

@kevquirk @lrmikestone If both of you are good with this let me know and I will edit the documentation and merge it.
To test it locally create _data/sites.yml and paste in it the domains from the 3 old data files.

I'll need to have a closer look at this, as it seems the deployment is failing:
2:04:27 PM:   Liquid Exception: Liquid error (line 38): Cannot sort a null object. in index.md
2:04:27 PM:                     ------------------------------------------------
2:04:27 PM:       Jekyll 4.2.0   Please append `--trace` to the `build` command
2:04:27 PM:                      for any additional information or backtrace.
2:04:27 PM:                     ------------------------------------------------

Make sure you created a new data file called sites.yml

Your PR fails to build though, so the site will break is we merge it, won't it?

For before the merge I will create one last commit that creates the 'sites.yml' file and delete the current '-team.yml' files. And it will pass build.

I'm confused, why not do that now?

Sorry for not being clear, it is just in case anyone adds or edits a domain between now and merging into master.

Ohhhhhh I'm with you now. Makes sense. Let's get it merged so that doesn't happen then. If you have tested it, and it works, then I'm happy. We can always revert if needed.

Steps I used to sort the data file automatically.
After merging all the files into one location I converted it from YAML to JSON and saved said output into a new file called out.json
Using jq used the following command to sort the JSON file using the domain key
cat out.json | jq 'sort_by(.domain)' > clean-json.json

took clean-json.json and converted it from JSON to YAML and cleaned up the formatting manually using Find & Replace in VSCode text editor

I assume this is a one time deal? I hope we don't have to do this every time a domain is added.

Oh yes it's a one time deal. Just documenting it just in case since the same thing can be used to sort by other key pairs.


Write a PIN program to generate the trace to be used in Cache coherence protocol.

commit id: 8c6c4ee7b3a88138f5d1d7ba042f5be51057a11d


When will version 2.3.3 be release? (for additional log4j security patches)

Unfortunately, the log4j issue is completely resolved in 2.3.3.
so. I need log4j security fix version of pinpoint
https://github.com/pinpoint-apm/pinpoint/releases/tag/v2.3.3
could you check 2.3.3 release time?

It will be released within today thanks


Making things work in Jupyterlab - part 1

Relates to #253

[x] Make Flexx work (technically) in a notebook in JLab
[x] Make Flexx not use its own Phosphor CSS when in JLAB
[x] Make Flexx' styling of Phosphor stuff such that it does not clash

#263 should take this further


Lawrence zip code controller and test

Added zip code controller code and passed my test

Looks good to me. Ready for merge.


dev version problem - Anaconda distribution

Release version on Ubuntu 14.04 works fine. However importing SimpleITK in dev version throws the following error:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-2-3dd12c25dd88> in <module>()
----> 1 import SimpleITK as sitk

/home/m/anaconda2/lib/python2.7/site-packages/SimpleITK/__init__.py in <module>()
----> 1 from .SimpleITK import *

/home/m/anaconda2/lib/python2.7/site-packages/SimpleITK/SimpleITK.py in <module>()
     26                 fp.close()
     27             return _mod
---> 28     _SimpleITK = swig_import_helper()
     29     del swig_import_helper
     30 else:

/home/m/anaconda2/lib/python2.7/site-packages/SimpleITK/SimpleITK.py in swig_import_helper()
     22         if fp is not None:
     23             try:
---> 24                 _mod = imp.load_module('_SimpleITK', fp, pathname, description)
     25             finally:
     26                 fp.close()

ImportError: libpng15.so.15: cannot open shared object file: No such file or directory
```f

You can find more information about this issue in the FAQ:
https://itk.org/Wiki/SimpleITK/FAQ#I_am_using_the_binary_distribution_of_SimpleITK_for_Anaconda.2C_why_do_I_get_an_error_about_libpng.3F
The "dev" label SimpleITK anaconda packages are older than the release.

Thanks for the information. I am closing the issue.


Added password generation functionality

Dan fixed the long-standing issue with the button (mistake by me).

Amended PR as per code review.