Performance comparison between pyppeeter and chromedp

Accidentally I have two implementations of roughly the same functionality. I did the Python implementation before I discovered chromedp. These two implementations are

https://github.com/larytet-go/headlessness
https://github.com/larytet-py/headlessness

There is not much of difference in the the first fetch (Python is probably 10%-20% faster on average). In my setup the I/O - time to bring the data from the WEB site - dominates the execution time.
There is ~2x difference in the following runs
Run the tests:
git@github.com:larytet-go/headlessness.git headlessness-go
cd headlessness-go
docker build -t headlessness-go .
docker run --shm-size 2G --rm -p 5900:5900 -p 8081:8081 --init headlessness-go
curl --silent -X POST  "http://<IP_ADDRESS>:8081/fetch?url=https%3A%2F%2Fwww.google.com%2F&deadline=3000&transaction_id=1"

git clone git@github.com:larytet-py/headlessness.git headlessness-py
cd headlessness-py
docker build -t headlessness-py .
docker run --rm -p 5900:5900 -p 8081:8081 -e TIMEOUT="30.0" -e HEADLESS="false" chrome
curl --silent -X POST "http://<IP_ADDRESS>:8081/fetch?url=https%3A%2F%2Fwww.google.com%2F&transaction_id=1"

The Python version consistently hits subsecond  processing time for the cached sites, which is ~50% improvement vs the first non-cached fetch. The Go version has ~30% performance improvement cached vs non-cached. The Python version is a bit faster in the first fetch than the Go version as well. I end up with Python being almost x2 faster for subsequent fetches.
I measured the time between the first request and last response processed by "listenTarget". I see that the I/O is responsible for the bulk of latency. The rendering, screenshot, report generation (my application proper) is usually in 200-300ms range.
In my case this is probably not critical for the production, but it looks bad in demos, makes it harder to "sell" the solution to the "stake holders".  Python version almost hits the sweet spot for the performance I target. Unfortunately the Python version is not stable enough for the production. chromedp proved to be robust in my tests.
Is there something I am still doing wrong with how I manage the browser tabs sessions? Or may be this is about the way I configure the browser? The configuration is somewhat different. In Python I rely on the default settings. In Go I explicitly set all flags: https://github.com/larytet-go/headlessness/blob/master/chrome/chrome.go#L159
P.S. I have one hypothesis. Until I test it I close the issue.

My theory was a missing "adblock" support in the Go version. My adblock is a  static list of hosts whuch are likely serving ad content. Adding the filter did not help.
My next hypothesis is that pyppeteer cuts slow responses short. If here is not traffic activity for 500ms the page is considered loaded. Checking.

@larytet I'm not sure whether you're aware of that in your code, every run of func (b *Browser) Report will create a new browser and then close it.
According to your statement above, I assume that you are not aware of that.

NewContext creates a chromedp context from the parent context. The parent context's Allocator is inherited, defaulting to an ExecAllocator with DefaultExecAllocatorOptions.
If the parent context contains an allocated Browser, the child context inherits it, and its first Run creates a new tab on that browser. Otherwise, its first Run will allocate a new browser.
Cancelling the returned context will close a tab or an entire browser, depending on the logic described above. To cancel a context while checking for errors, see Cancel.
Note that NewContext doesn't allocate nor start a browser; that happens the first time Run is used on the context.

You can observe this behavior with the following demo (it's a reduced version of  your code. The browser windows are overlapped, please move the windows manually to show all 3 windows).
package main

import (
	"context"
	"log"
	"time"

	"github.com/chromedp/chromedp"
)

func main() {
	opts := append(chromedp.DefaultExecAllocatorOptions[:],
		chromedp.Flag("headless", false),
	)
	ctx, cancel := chromedp.NewExecAllocator(context.Background(), opts...)
	defer cancel()

	bctx, cancel := chromedp.NewContext(ctx,
		// the next line does not have any effect because there are not browsers created by this context.
		chromedp.WithDebugf(log.Printf),
	)
	defer cancel()

	for i := 0; i < 3; i++ {
		go func() {
			ctx, cancel := chromedp.NewContext(bctx,
				chromedp.WithDebugf(log.Printf),
			)
			defer cancel()
			chromedp.Run(ctx)
			time.Sleep(10 * time.Second)
		}()
	}

	time.Sleep(12 * time.Second)
}

Here is the changes to make it share the browser:
 package main

 import (
    "context"
    "log"
    "time"

    "github.com/chromedp/chromedp"
 )

 func main() {
    opts := append(chromedp.DefaultExecAllocatorOptions[:],
        chromedp.Flag("headless", false),
    )
    ctx, cancel := chromedp.NewExecAllocator(context.Background(), opts...)
    defer cancel()

    bctx, cancel := chromedp.NewContext(ctx,
-       // the next line does not have any effect because there are not browsers created by this context.
        chromedp.WithDebugf(log.Printf),
    )
+   // call Run on browser context to start a browser first
+   chromedp.Run(bctx)
    defer cancel()

    for i := 0; i < 3; i++ {
        go func() {
-           ctx, cancel := chromedp.NewContext(bctx,
-               chromedp.WithDebugf(log.Printf),
-           )
+           ctx, cancel := chromedp.NewContext(bctx)
            defer cancel()
+           // since the parent context has browser allocated,
+           // the Run will create new tab in the browser.
            chromedp.Run(ctx)
            time.Sleep(10 * time.Second)
        }()
    }

    time.Sleep(12 * time.Second)
 }

I haven't read your python code, but I'm pretty sure that it just creates a single browser instance, and that's  why it looks faster.

Regarding adblock, you can use Network.setBlockedURLs to block URLs.

@ZekeLu  you are absolutely right. I hit 600ms loading time for google.com


Improve IndentationHelper

removed string concatenation
removed char array allocation
removed delegate and closure allocation

I filed issues for the remaining work.


Update slick.js

Fix bug with previous and next buttons when slickGoTo is used to go to a specific slide, add getCount method to return total slide count
http://jsfiddle.net/dldean/v3b4rzhm/

No longer an issue after other PRs and BugFixes :)


Fairseq Installation Fails: Missing version.txt Error During Build Pyhton 3.11 how to fix?

https://stackoverflow.com/questions/78859329/fairseq-installation-fails-missing-version-txt-error-during-build-pyhton-3-11-h
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [19 lines of output]
    Traceback (most recent call last):
      File "/opt/conda/envs/beyond_scale_div_coeff/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
        main()
      File "/opt/conda/envs/beyond_scale_div_coeff/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
        json_out['return_val'] = hook(**hook_input['kwargs'])
                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/opt/conda/envs/beyond_scale_div_coeff/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
        return hook(config_settings)
               ^^^^^^^^^^^^^^^^^^^^^
      File "/tmp/pip-build-env-nrzuksph/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 327, in get_requires_for_build_wheel
        return self._get_build_requires(config_settings, requirements=[])
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/tmp/pip-build-env-nrzuksph/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 297, in _get_build_requires
        self.run_setup()
      File "/tmp/pip-build-env-nrzuksph/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 313, in run_setup
        exec(code, locals())
      File "<string>", line 27, in <module>
      File "<string>", line 18, in write_version_py
    FileNotFoundError: [Errno 2] No such file or directory: 'fairseq/version.txt'
    [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.

https://chatgpt.com/c/bb70d4b7-b558-457e-8218-ab4a2a899d6f


Fix wrongly set S3 Asset ACL

According to doc site:

If your S3 is readable by the public, you can set ASSET_STORE_PUBLIC to YES
so that the access signature can be omitted.

Rejecting as the config only applies for object signing.


Dark Mode

Could you bring dark mode support for the extension setting page?
And how can we help you translate the extension to a specific language?

@Darktowertr

Could you bring dark mode support for the extension setting page?

Dark mode is added in v4.5.0.

And how can we help you translate the extension to a specific language?

Please see the readme.


:snake_case_symbol keys

Ruby-like structure for JSON data. Hash keys are snake_cased and symbols.
Note that the local tests were much more extensive than those in this project but decided to match the simplicity of this spec.

@eigencoder 2 points here:

deep_transform_keys introduces a hidden dependency on activesupport (it is not a Ruby built-it method)
snake-casing response keys actually has nothing to do with Oj - it can handle response from the regular ParseJson middleware as well. Why don't you want to extract it to a separate gem and them just combine the two?

What do you think?


Misc. errors thrown while running org.openrewrite.analysis.controlflow.ControlFlowVisualization

Duplicate key J.Block on src/main/java/com/amazonaws/kinesisvideo/internal/service/BlockingAckConsumer.java
No current node! on JS files
Could not parse as Java on sdk1/src/test/java/com/amazonaws/services/dynamodbv2/mapper/integration/DynamoDBTestBase.java
Unable to create new J.VariableDeclarations with name enumSymbolIterator on avro-kafkaconnect-converter/src/main/java/com/amazonaws/services/schemaregistry/kafkaconnect/avrodata/AvroData.java
Condition Node is not a guard! on src/main/webapp/config.js [JS file]

I've also found different errors regarding switch expressions handling .
For example :
public String getValue(Enum enum) {
  String val = null;
  switch (enum) {
    OK -> val = "OK";
    KO -> val = "KO";
   default -> val= "OK";
  }
  return val;
}

is valid, whilst the concise version
public String getValue(Enum enum) {
  return switch (enum) {
    case OK -> "OK";
    case KO ->  "KO";
   default -> "OK";
  }
}

Error is displayed as :
[ERROR] Failed to execute goal org.openrewrite.maven:rewrite-maven-plugin:5.20.0:run (default-cli) on project com-acme-cli: Execution default-cli of goal org.openrewrite.maven:rewrite-maven-plugin:5.20.0:run failed: Error while visiting src/main/java/com/acme/cli/context/console/impl/CliTerminalConsoleContext.java: org.openrewrite.analysis.controlflow.ControlFlowIllegalStateException: Case statements should be visited by the switch statement
[ERROR] 	AST: case OK -> "OK"
[ERROR] 	Cursor: Cursor{Case->Block->SwitchExpression->Return->Block->MethodDeclaration->JRightPadded(element=MethodDeclaration{com.acme.cli.context.console.impl.CliTerminalConsoleContext{name=logColorResolver,return=org.fusesource.jansi.Ansi$Color,parameters=[com.acme.context.console.CliTerminalContext$TerminalLogLevel]}}, after=Space(comments=<0 comments>, whitespace=''))->Block->ClassDeclaration->CompilationUnit->root}
[ERROR]   org.openrewrite.analysis.controlflow.ControlFlow$ControlFlowAnalysis.visitCase(ControlFlow.java:385)
[ERROR]   org.openrewrite.analysis.controlflow.ControlFlow$ControlFlowAnalysis.visitCase(ControlFlow.java:80)
[ERROR]   org.openrewrite.java.tree.J$Case.acceptJava(J.java:1088)
[ERROR]   org.openrewrite.java.tree.J.accept(J.java:59)
[ERROR]   org.openrewrite.TreeVisitor.visit(TreeVisitor.java:278)
[ERROR]   org.openrewrite.TreeVisitor.visit(TreeVisitor.java:184)
[ERROR]   org.openrewrite.analysis.controlflow.ControlFlow$ControlFlowAnalysis.visitRecursive(ControlFlow.java:164)
[ERROR]   org.openrewrite.analysis.controlflow.ControlFlow$ControlFlowAnalysis.visitStatementList(ControlFlow.java:209)

Thanks for the heads up. Control Flow Analysis is heavily tied to the characteristics of the language, so it's not shocking to me that new language features cause it to break.
I don't know if I have time right now to address this. But thank you for letting me know.

Thanks . Btw , how is this recipe working ? It just appends a lot of graph is code to Java files , and then , what ?

It just appends a lot of grapviz code to Java files , and then , what ?

It's primarily used for debugging the control flow analysis engine. Control flow itself is more useful in the context of data flow analysis.
Here's an example of how it is used in, probably the most complex security fix recipe, ZipSlip:
https://github.com/openrewrite/rewrite-java-security/blob/008c74ab09a96912b9e2895259a7a42bd5c90c8a/src/main/java/org/openrewrite/java/security/ZipSlip.java#L502-L510

If you'd like to learn more, I gave a talk on the subject and how we use control flow and data flow analysis here: https://youtu.be/UgGhEfdUSvQ?si=IByhi4TnQfBX-kPv&t=1582


Remove redundant std

This fixes building ethminer w/ cuda on Linux

Already handled in #7, sorry.


Chand8

Thanks, I'll review shortly. Will probably cherry pick and not merge because there's too many discrete unrelated changes here. Would be great if future PRs will be isolated :)


Random Key Value Pairs for AdOps ab Testing

Need to setup a way to randomize key-value pairs per session that get passed through dfp for adops ab testing.

establish 20 key values and setup buckets 1-20, for example ab1=[rand(1, 20)]&ab2=[rand(1, 20)],etc
at session start, establish values and store in buckets
this will go into display and custom params

@oefada can you please code review this and test in browser. This script sets a cookie with name 'wbrandomab' with the value of 20 randomized values from 1-20. It will also set gpt custom parameters. You can see the code in action here http://ads-www.cfellentube.com/

this just got much simpler.  let's call it "abt", "wbabt" global, generates a rand value between 1 and 20.  one value.  this is now virtually the same as ppid.
no multiple buckets any more.

see here: https://jsfiddle.net/f41ghn5L/3/
even dist ^^

call the script file "abt.js" as well.


Konnect Feb 2023 Changelog

Signed-off-by: Diana<EMAIL_ADDRESS>Description
What did you change and why?
Created the changelog for Konnect Feb 2023.
Include any supporting resources, e.g. link to a Jira ticket, GH issue, FTI, Slack, Aha, etc.
n/a
Testing instructions
Netlify link:
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

Now that we've shipped at least one feature in Feb, can we ship this?


cpmfyui 上可以克隆语音吗？

Error occurred when executing CosyVoiceNode:
[ONNXRuntimeError] : 6 : RUNTIME_EXCEPTION : Non-zero status code returned while running Add node. Name:'/Add_2' Status Message: D:\a_work\1\s\onnxruntime\core/providers/cpu/math/element_wise_ops.h:560 onnxruntime::BroadcastIterator::Append axis == 1 || axis == largest was false. Attempting to broadcast an axis by a dimension other than 1. 1500 by 6382
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\execution.py", line 151, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\execution.py", line 81, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\execution.py", line 74, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\custom_nodes\CosyVoice-ComfyUI\nodes.py", line 131, in generate
output = self.cosyvoice.inference_zero_shot(tts_text, prompt_text, prompt_speech_16k)
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\custom_nodes\CosyVoice-ComfyUI\cosyvoice\cli\cosyvoice.py", line 59, in inference_zero_shot
model_input = self.frontend.frontend_zero_shot(i, prompt_text, prompt_speech_16k)
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\custom_nodes\CosyVoice-ComfyUI\cosyvoice\cli\frontend.py", line 228, in frontend_zero_shot
speech_token, speech_token_len = self._extract_speech_token(prompt_speech_16k)
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\custom_nodes\CosyVoice-ComfyUI\cosyvoice\cli\frontend.py", line 158, in _extract_speech_token
speech_token = self.speech_tokenizer_session.run(None, {self.speech_tokenizer_session.get_inputs()[0].name: feat.detach().cpu().numpy(),
File "C:\Users\av8dt\Documents\ComfyUI-aki-v1.2\python\lib\site-packages\onnxruntime\capi\onnxruntime_inference_collection.py", line 220, in run
return self._sess.run(output_names, input_feed, run_options)

用自带的3s极速复刻workflow就行，我跑通了。

Hey did you manage to fix the issue? I got the same even if using the workflows/3sclone_workflow.json as suggested in the previous answar


Connection string

Issue Type: Feature Request
how to get connection string from azure data studio
Azure Data Studio version: azuredatastudio 1.29.0 (0f5cfdc2c8c2433028842e16dc5ac1a65da24292, 2021-05-15T01:09:31.644Z)
OS version: Darwin x64 20.3.0

Closing as duplicate of https://github.com/microsoft/azuredatastudio/issues/15625


Added Codeberg and Papara

Papara is mainly a Turkish service so please feel free to not include it if you feel like it doesn't suit the project. Also, I was planning to do one more Turkish service that starts with "Ç". Where would that be in the alpabetical listing, if it is even allowed? Thank you.

While we're at it, giving phone numbers of support lines is not allowed, is it?

While we're at it, giving phone numbers of support lines is not allowed, is it?

I'd say no one ever tried. I don't see why it wouldn't be valid, though I probably would take a link with the phone being quoted to make sure it seems legit.

Thanks for your contribution! I've merged this one, feel free to open more PRs with the next ones


GitHub token creation/nbinit

When running nbinit check localconfig.yaml for existing token and reuse it.
If there is no token then use something unique in the note for creating the token as we can't create two tokens with the same name.

Alternatively place the token in ~/.abc.config and share it between all courses.

Note - this is the error i get running nbinit when i already have a token (just for the record)
Traceback (most recent call last):
  File "/Users/leah-su/anaconda3/bin/nbinit", line 11, in <module>
    sys.exit(init())
  File "/Users/leah-su/anaconda3/lib/python3.6/site-packages/grading/__main__.py", line 60, in init
    two_factor_callback=two_factor)
  File "/Users/leah-su/anaconda3/lib/python3.6/site-packages/github3/api.py", line 26, in deprecation_wrapper
    return func(*args, **kwargs)
  File "/Users/leah-su/anaconda3/lib/python3.6/site-packages/github3/api.py", line 59, in authorize
    client_secret)
  File "/Users/leah-su/anaconda3/lib/python3.6/site-packages/github3/github.py", line 462, in authorize
    json = self._json(self._post(url, data=data), 201)
  File "/Users/leah-su/anaconda3/lib/python3.6/site-packages/github3/models.py", line 156, in _json
    raise exceptions.error_for(response)
github3.exceptions.UnprocessableEntity: 422 Validation Failed

related to this is -- is this authorize() call deprecated? see here please. something odd is going on with the init command that i can't figure out .i seem to have to delete it each time or every so often. could be user error!

Good catch with the deprecation. I hadn't noticed this. Take a look at #40

#40 doesn't quite close this issue as we still use a file local to the repository to store the token instead of placing it in ~/.abc.token (or some such). That is for another PR to solve.


Move generator HttpPipelineExtensions methods to Azure.Core

Library name
Azure.Core
Please describe the feature.
HttpPipelineExtensions class exists in Generator.Shared and it has ProcessMessage methods which we use in DPG methods. Here is an example of how do we use these methods in DPG.
I think we should move all the methods from HttpPipelineExtensions to Azure.Core as it can be useful for the cases when someone has to handwrite the DPG methods.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Extensions:0.4857908,Azure.Core:0.28164512,Storage:0.117870994'

CC: @annelo-msft

Do we make these public extension methods, or as shared source in Core?

Closing as duplicate of https://github.com/Azure/azure-sdk-for-net/issues/32006


APPS-1279 Support GCP storage

Codecov Report
Attention: Patch coverage is 32.03883% with 210 lines in your changes missing coverage. Please review.

Project coverage is 32.50%. Comparing base (feb093a) to head (c69cf74).

Files with missing lines
Patch %
Lines

pkg/service/storage/s3.go
3.12%
62 Missing :warning:

pkg/service/storage/gcp.go
8.92%
51 Missing :warning:

pkg/dto/storage.go
18.64%
42 Missing and 6 partials :warning:

pkg/service/storage/operations.go
54.79%
28 Missing and 5 partials :warning:

pkg/service/storage/local.go
73.52%
6 Missing and 3 partials :warning:

pkg/service/storage/validator.go
61.53%
4 Missing and 1 partial :warning:

pkg/service/paths.go
0.00%
1 Missing :warning:

pkg/service/storage/accessor.go
87.50%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##               v2     #236      +/-   ##
==========================================
- Coverage   33.03%   32.50%   -0.53%
==========================================
  Files          31       36       +5
  Lines        1659     1769     +110
==========================================
+ Hits          548      575      +27
- Misses       1031     1113      +82
- Partials       80       81       +1

Flag
Coverage Δ

?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Is it possible to view categories on the posts in this theme?

Could you please elaborate more on your question? Thanks!


Generalize email address and password for clarity

Also, changed the file extension to .md, which is standard Markdown. Might break links, so happy to revert if so. First step to fixing https://github.com/speric/kindle-highlights/issues/4.

Great; thanks!


RSVPBot should only reply to actual commands, and not any message that begins with rsvp.

Basically, we need to fix this:

Quick question here: is there a good reason that RSVPBot respond with a stream message rather than private messages? I can think of a good reason for rsvp summary, but it seems like everything else should probably be private. Is it to simplify the un-rsvp case?

@jamak The thing from the beginning is that I believe that people learn how to use a bot by watching other people use it.
I do agree that the constant back and forth with RSVPBot can be too spammy now that it's so popular, but then again the reason it's so popular is that everyone watches everyone else using it.
People need some kind of feedback that the bot is working and that it's not down (e.g if someone types rsvp yes and the bot doesn't respond that's a huge red flag.
OTOH, a XXX is attending! private response would be redundant since you already know that you rsvp yesd to an event.
It's kind of a tricky subject, as I want to both heavily reduce the spam / boilerplate generated by RSVPBot / users of RSVPBot and at the same time I want to keep it easy to learn.

Personally, I'm a huge fan of RSVPBot's banter.  It really adds to an event thread.

As a first step, we could test private responses for every rsvp set <PARAMETER> to be private, since typically only one person sets those parameters.

rsvp help is already private.
rsvp summary should be, by definition, public
And I really don't want the response to rsvp <yes|no|maybe> to be private

I was imagining a case where rsvpbot responds to all commands in private, and maybe adding some boilerplate like "respond with rsvp yes | no | maybe " to the output of rsvp init . That way people could immediately get feedback from the bot but it would be at least half as spammy. I personally think its popular because its useful, not because it's easy to watch others interact with it; of course this being your project and all, I defer to your wisdom 😄 .
Another follow-up question: If Zulip were to implement reaction emoji like slack (something I think @jdherg is working on), would you be averse to using reactions to rsvp for the same reasons (e.g. you react with a specific emoji to rsvp yes|no|maybe )?


Fixed bug on "sat_dist_type" updates

This is to fix the issue #15.

Why the first condition in line 192, self.gparams, is needed?
The second condition alone, self.gparams["sat_dist_type"] != gparams["sat_dist_type"], seems to be enough.

"self.gparams" is set to None (see Line 109: self.gparams = None) when the class object is instantiated. Therefore, without the first condition, the the code stops when "set_galaxy" is first called with an error message:
    if self.gparams["sat_dist_type"] != gparams["sat_dist_type"]:
TypeError: 'NoneType' object is not subscriptable

The first condition avoids this.

Thanks, Sunao!


Adding CodeClimate

This change is

Codecov Report

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

@@           Coverage Diff           @@
##           master     #187   +/-   ##
=======================================
  Coverage   51.47%   51.47%
=======================================
  Files         169      169
  Lines        4940     4940
=======================================
  Hits         2543     2543
  Misses       2335     2335
  Partials       62       62

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 5bf1800...8557139. Read the comment docs.


Terraform deployments improvements

This includes the following changes:

Removing manual enablement of services to enforce resource creating after service enablement
BigQuery dataset expiration changed
Aditional docs

LGTM


tracker.events value in custom action is not having the conversation history when using the button actions

Rasa version:
rasa-core==0.14.3
rasa-core-sdk==0.14.0
rasa-nlu==0.15.0
Python version:  Python 3.6.7
Operating system (windows, osx, ...):
Distributor ID: Ubuntu
Description:    Ubuntu 18.04.2 LTS
Release:        18.04
Codename:       bionic
Issue:
When I use button actions as interactive components, the tracker.events value in custom actions showing only the conversation history associated with the button actions. All previous conversation history (without having the button actions) is not available in tracker.events.
But, if you send the same intents values as text (instead of clicking buttons), then the complete conversation history is available in tracker.events. That means, the issue is appearing only when using button action. For testing it, you can just print the tracker.events value in the custom action and compare it in both cases (User input as text and user input as button action)
I tested it for a slack bot integration. Please let me know, if you need more details.
Content of configuration file (config.yml):
language: "en"

pipeline:
- name: "nlp_spacy"                   # loads the spacy language model
- name: "tokenizer_spacy"             # splits the sentence into tokens
- name: "ner_crf"                   # uses the pretrained spacy NER model
- name: "intent_featurizer_spacy"     # transform the sentence into a vector representation
- name: "intent_classifier_sklearn"   # uses the vector representation to classify using SVM
- name: "ner_synonyms"                # trains the synonyms

Content of domain file (domain.yml) (if used & relevant):
utter_check_feedback:
  - text: Was my response helpful?
    buttons:
      - title: "Yes"
        payload: '/response_success'
      - title: "No"
        payload: '/response_failure'

Thanks for raising this issue, @erohmensing  will get back to you about it soon.

Hey Anoop! To clarify is this custom action that checks the tracker completely separate from the action where you're uttering the buttons? Could you show me an example story?
Something else that would be helpful would be if you would run the bot in the command line and let me know if the same problem persists -- that'll help me figure out if the problem is coming from buttons themselves or slack.

For testing, I just created another bot with minimum knowledge and added the button actions. In this case, the tracker.events in the custom action is showing all the conversation history without any issues. Hence, I believe there could be something wrong in the stories of the other bot, where I found the issue and it may not be an issue with the rasa stack. Will check it myself. Hence, closing the issue.
@akelad, @erohmensing thank you for your prompt response.


refactor: [Noon] adding new field max_amount to mandate request

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Resolves  #3844
Note
1.While connector create ensure test_mode is set to true
2.Ensure the currency passed is AED and country is AE
How did you test it?
1.Create a CIT with noon
with
  "connector_metadata": {
        "noon": {
            "order_category": "pay"
        }
    }

without order_category it should throw an error

Make a MIT

Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

The testing should be consistent across local DB, the response and the noon dashboard so things like amount validation don't get missed

Tested, mandate amount remains consistent in the local DB and dashboard


limit line select

i need select LimitLine after touch

you may set controller's OnTouchEventListener and use controller.xAxis.getLimitLines() in corresponding method


Add build script for command line restore, build, test

Closes #205

What's the default if build is run without any parameter ?
I think the best would be for Restore + Build + Test but if I read the code correctly it does nothing currently no ?

Correct: by default no action is taken. I've changed it so that if no action is specified, all are executed.


Add Code For ReconstructOriginalDigitsFromEnglish.cpp and Update README

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes
Description
(Write your answer here.)
I have add the code, description, time and space complexity, input/output sample for ReconstructOriginalDigitsFromEnglish.cpp.
Checklist

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

Related Issues or Pull Requests
(Write your answer here.)
#4707

@HarshCasper I have created the PR for the assigned issue. Please check this and merge.

@ankitaggarwal23 Please check this one and suggest me changes and merge!.


Rewrite with ScreenCaptureKit

Rewrites Aperture to use ScreenCaptureKit
Breaking changes:

Now requires macOS 13+
highlightClicks is only available on macOS 15+
Aperture itself is not the recorder, you can instantiate Aperture.Recorder() and use startRecording() and stopRecording() on it
Everything uses async/await instead of callbacks
startRecording awaits until the file actually starts getting written before resolving, or throws any errors that happen during that time
cropRect is top/left now (this is how ScreenCaptureKit works by default)

This has as much feature parity as possible (other then the highlightClicks note above) but it adds some functionality as well:

Also supports .mov and .m4v for videos
Recording system audio (+ mic)
Loseless audio option
Recording audio only (only .m4a for now, AAC or ALAC encoding)
Recording a specific window (will follow the window around even if it moves or covered by other windows)

Potential features we'd want to add in a follow-up:

Support more audio formats
Maybe support quality/bitrate options for audio/video
Support recording applications as targets
Support excluding apps/windows from the recording (for screen/application only)

Some good default sets would be excluding the current application itself and/or desktop icons

This PR still needs to update the readme, but wanted to get it reviewed first to fully land on the implementation before I go and update the readme
Closes #16
Closes #31
Closes #71
Closes #63
Closes #65

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#65: Support for top/left positions for capture

@sindresorhus addressed all except for this and configured/ran swiftlint 😌

While working on aperture-node with this version, I noticed a few things missing. Pushed some commits above, but need to also add:

Support for proRes codecs
Error handling if we can't add the video inputs (we silently fail right now, took me a while to figure out why prores wasn't working)

will be pushing a commit with those two at some point tomorrow

Ok, I think it's ready:

Added support for proRes

Created a scoped enum for the ones we support and an extension to map them
Added checks for invalid file extensions <-> codec

Added better error handling

Fixed some errors to have clearer message
Fixed logging for error sub-reasons
Throw errors if the stream does not start instead of failing silently or crashing

Added a reset to the class so it can be re-used after recording is done

You don't need to nil it in the clean up. It will end when the RecordingSession class is nil'd. However, should the activity be ended when paused?

@sindresorhus  Updated 👍
I think it's ok to keep during pause, not sure what would happen if it went to sleep, then resumed after
Also updated the readme. We didn't have docs before, added some basic usage and options

Instead of documenting the API in the readme, it would be better to use DocC and add good doc comments to the methods.
You just need https://github.com/sindresorhus/Defaults/blob/main/.spi.yml and get it added to https://swiftpackageindex.com/add-a-package

@sindresorhus re: docs, do I just need the /// comments above the exported methods and to add that file? Any way to check what it looks like locally?
I've added those for most properties, but that class is internal now. The "exposed" one only has start/stop/pause/resume and the onStart etc handlers
If it's a lot of work, can we do it in a follow-up?

@sindresorhus added in-line docs and added a link to the readme for swift package index (I kept the other readme sections as well)
I'm not sure if I can submit it to package index until after it's merged to main, but also figured the text might need changes

@sindresorhus Thanks for all the reviews!
Is there anything special about releasing? Or just cut a release tag using the GH UI?
It looks like SPM clones these from GH, but I'm not sure


[Beam-2482] - CodedValueMutationDetector should use the coders structural value

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

please, @tgroh  and @lukecwik  reviwed and let me know if I need to change some thing. I think doing it this way removes a bit of complexity. One thing that I was not sure of was if immutability check should be able to detect deepcopies of objects. right now there is no test covering that case. I don't see why this case should be covered but I though it might be good to ear from more experienced people.

Changes Unknown when pulling 36c7a14c69ba628f477798f63d4e721dc86e2be7 on evindj:BEAM-2482 into ** on apache:master**.

Coverage increased (+0.02%) to 70.4% when pulling 1bfc0aecbef5a6adb56e69db9168db0b652ed654 on evindj:BEAM-2482 into bc72c940537dbfcfa982cc1867af087487ed7441 on apache:master.

Coverage decreased (-0.4%) to 69.973% when pulling 2b7ff0a0e0fe9116575ff3416cce217895ae68c4 on evindj:BEAM-2482 into bc72c940537dbfcfa982cc1867af087487ed7441 on apache:master.

Coverage decreased (-0.4%) to 69.959% when pulling 4931cd430602c2b41a7e2450950a6c822880e927 on evindj:BEAM-2482 into bc72c940537dbfcfa982cc1867af087487ed7441 on apache:master.

Is there any update on this PR?

I have been a bit busy lately will look at it sometime this week.
2017-08-29 18:25 GMT-04:00 reuvenlax<EMAIL_ADDRESS>
Is there any update on this PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3643#issuecomment-325822237, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpiGkvVpxVUmEvvYNRWQWZiYYxakqPNks5sdI_ZgaJpZM4OjbbJ
.

--
DJIOFACK INNOCENT
"Be better than the day before!" -
+1<PHONE_NUMBER>

I could not find a good test that will fail the first implementation and succeed this implementation. I have the feeling that the first implementation is more permissive because it was relying on a deep equal as well as trusted the equal method of the objects being tested for mutations.

An example of a test would be to create a new class which only returns true if its the same instance otherwise returns false and also a coder which returns a structural value which is a new type that says its equal as long as the other object is of the same type.

Thanks Lukasz will try that !
2017-09-01 0:51 GMT-04:00 Lukasz Cwik<EMAIL_ADDRESS>
An example of a test would be to create a new class which only returns
true if its the same instance otherwise returns false and also a coder
which returns a structural value which is a new type that says its equal as
long as the other object is of the same type.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3643#issuecomment-326486728, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpiGsGVWIMg6zJOLvsnO45cXtf-iKDcks5sd41qgaJpZM4OjbbJ
.

--
DJIOFACK INNOCENT
"Be better than the day before!" -
+1<PHONE_NUMBER>

Have you been able to make any further progress on creating that test?

Hi Lukasz, Sorry I have been quite busy lately I'll look at it over the
weekend. If I have questions I'll let you know.
Thanks for checking.
2017-09-27 19:23 GMT-04:00 Lukasz Cwik<EMAIL_ADDRESS>
Have you been able to make any further progress on creating that test?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3643#issuecomment-332683114, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpiGpHWKVpm19lkd_eaUE-EE76l6eEYks5smtjcgaJpZM4OjbbJ
.

--
DJIOFACK INNOCENT
"Be better than the day before!" -
+1<PHONE_NUMBER>

Hey @evindj any updates on this PR?

@lukecwik sorry for the long wait, I am working on implementing the changes you proposed and would like to clarify the following:

I feel like the proposed test does not really cover the concept of immutability. especially I can imagine a situation where a modified value will generate a Structural value that is of same type of the original object. I therefore wonder if we want to use Strutural values to provide a semantic for immutability.
Thanks.

Your correct in saying that a value may change and yet produce the same structural value. Contractually a structural value is a way for a user to provide an object which correctly implements Object.equals similar to Guava's EquivalenceWrapper. Since the Java SDK only is meant to rely on structural values when using Java maps and other hashcode/equals methods it should be the only thing we care to test correctly. If the user implements Object.equals correctly but structural value incorrectly then its likely we will not do the right thing for them since the SDK uses their type as keys in internal maps. I don't think we can do better then this for immutability testing since if we rely on other methods for checking if things are equal we won't be following the contract that matters to the SDK and that the SDK relies on.

I get it what bothers me is the fact that the newly created test will have an incorrect definition of the structural value equality which might be misleading.Since the new test is more about the contract of the structural value and the java SDK than the test of immutability, it gives me the impression that the test should not be added in the context of immutability testing. I am taking into consideration the fact that this came up because some mutation testing are not failing but were not failing before. We can discuss this further on slack later today if you are free.

It seems as though you pulled in a bunch of changes into your PR, did you mean to rebase instead of merge?

I meant to rebase , I can close this and open a new one
On Mon, Oct 30, 2017 at 12:53 PM Lukasz Cwik<EMAIL_ADDRESS>wrote:

It seems as though you pulled in a bunch of changes into your PR, did you
mean to rebase instead of merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3643#issuecomment-340509644, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpiGkaY_JXTTheQ5McG3igetiugGh_Vks5sxf8hgaJpZM4OjbbJ
.

--
DJIOFACK INNOCENT
"Be better than the day before!" -
+1<PHONE_NUMBER>

Sure.


mbed installer and mbed serial driver problem

Hello,
I'm using windows 7 and I tried to install mbed-cli using the installer named "mbed_installer_v041.exe"
When the installer tries to install the mbed serial driver I have already plugged to the PC the ST NUCLEO_F429ZI but it is not recognized from the installer:

I resolved this issue installing manually the driver using the file en.stsw-link009.zip provided by ST website.
Is there some advice in order to avoid to install manually this driver?
Thanks

@arekzaluski Do we have any plans to detect boards that are attached before installing the serial driver?

@rspelta @bridadan I'm able to reproduce it. It seems that mbedWinSerial does not support Nucleo F429ZI board. I'll investigate it further and check what can be done to improve the process.

Perfect, thank you!

Same problem with me using another board (NUCLEO_F103RB).
The STLINK driver must be installed first:
[[https://my.st.com/content/my_st_com/en/products/embedded-software/development-tool-software/stsw-link009.license%3D1502449373926.html]]
But running the mbed_installer_v041.exe after STLink driver is installed does not change the behaviour: same error message as described by @rspelta

@arekzaluski please see above

@arekzaluski @rspelta Is this still an issue?

ARM Internal Ref: MBOTRIAGE-458


Why office365 xlsx sheet utf8 encoding but openxmlsdk is utf8-withbom

Before submitting an issue, please fill this out
Is this a:

[X] Issue with the OpenXml library
[ ] Question on library usage

If you have answered that this is a question, please ask it on StackOverflow instead of here.
This issue tracker is meant to track product issues while StackOverflow excels at answering questions
---------------- Remove this line and above before posting ----------------
Description
Please provide a simple description of the issue encountered.
Information

.NET Target: .NET Core
DocumentFormat.OpenXml Version: 2.14.0

Repro
public class Program
{
	public static void Main(string[] args)
	{
		var directoryInfo = new DirectoryInfo(Directory.GetCurrentDirectory());
		var fileName = $@"PracticePart1-{DateTime.Now:yyyyMMddHHmmss}.xlsx";
		var filepath = Path.Combine(directoryInfo.ToString(), fileName);
		Console.WriteLine($"FilePath: {filepath}");
		var spreadsheetDocument = SpreadsheetDocument.Create(filepath, SpreadsheetDocumentType.Workbook);
		var workbookPart = spreadsheetDocument.AddWorkbookPart();
		workbookPart.Workbook = new Workbook();
		var worksheetPart = workbookPart.AddNewPart<WorksheetPart>();
		worksheetPart.Worksheet = new Worksheet(new SheetData());
		var sheets = spreadsheetDocument.WorkbookPart.Workbook.AppendChild<Sheets>(new Sheets());
		var sheet = new Sheet()
		{
			Id = spreadsheetDocument.WorkbookPart.GetIdOfPart(worksheetPart),
			SheetId = 1,
			Name = "myFirstSheet"
		};
		sheets.Append(sheet);
		workbookPart.Workbook.Save();
		spreadsheetDocument.Close();
	}
}

Observed
Office 365 encoding are all utf-8 without bom, but openxmlsdk some're utf-8 with bom and some not

Expected
Should we follow office365 encoding standard? (below image is office 365 xlsx)
office365_sample.xlsx

We changed it to that as it was causing some renderers to have problems (see https://github.com/OfficeDev/Open-XML-SDK/issues/309).
I'm not certain if there's a specific encoding is required by the spec, but we could potentially enable it to be configurable rather than relying on a specific default.

but we could potentially enable it to be configurable rather than relying on a specific default.

@twsouthwick  Thanks! it will be helpful feature.

Happy to accept PRs. Probably best to add it to the OpenSettings object

@shps951023 is there a good reason why the SDK should emit non-BOM UTF-8? From our Office apps team, it appears that we don't have any requirement either way, i.e. Office apps will read UTF-8 BOM parts just fine. Does your code depend on non-BOM UTF-8?

@tomjebo
So sorry about long time to reply!!
Some Chinese users need to custom encoding to read non-UTF8.
@twsouthwick
Thanks, I'll try it
Happy new year!

Hi, i have found the same error when i try create SpreadsheetDocument with memory stream. Unfortunately, xlsx file which is generated from this memory stream is not possible open in Office 365. When i edit files inside package and manually change encoding to non BOM version, it works. Is it exists some workaround at least ? Thank you


Feature:  Vector index is split by K-means into regional clusters, each being a specific size to fit in RAM.

SDK
Node
Description
https://airesearch.js.org/functions/addEmbeddingVectorsToIndex.html
Compile hnswlib-node or NGT algorithm C++ to WASM JS for efficient similarity search.
Vector index is split by K-means into regional clusters, each being a specific size to fit in RAM. This is better than popular vector engines that require costly 100gb-RAM servers because they load all the vectors at once.
Vectors for centroids of each cluster are stored in a list in SQL, each cluster's binary quantized data is exported as base64 string to SQL, S3, etc.
Search: Embed Query, Compare to each cluster centroid to pick top clusters, download base64 strings for those clusters, load each into WASM, find top neighbors per cluster, merge results sorted by distance.

Hi @vtempest.

We already split our indices into clusters based on k-means. This is the "IVF" part of our IVF_PQ and IVF_HNSW_SQ indices. Agreed that vector databases that have to load huge amounts of data into memory don't always make sense.
We don't yet support WASM as a target, but would definitely like to eventually. We are tracking that in https://github.com/lancedb/lancedb/issues/230


[tool][web] Create an early web plugin_registrant for dartpad.

The plugin_registrant.dart for web used to be created on flutter pub get. This was changed to be created only upon build/run time, so the file didn't have to live in the users' working copy (and didn't need to be analyzed alongside user code), here.
Making the plugin_registrant.dart ephemeral, however, broke the way that Dartpad compiles Flutter web apps quickly. Read more.
This PR adds a web_plugin_registrant.dart file in a known location (.dart_tool/dartpad/) at flutter pub get that Dartpad can use when bootstrapping their code.
The file is always generated, regardless of the app using web plugins or not. The contents are the same as for any Flutter web app.
Without plugins:
// Flutter web plugin registrant file.
//
// Generated file.
//
// Generated file. Do not edit.
//

// ignore_for_file: type=lint

import 'package:camera_web/camera_web.dart';
import 'package:url_launcher_web/url_launcher_web.dart';
import 'package:video_player_web/video_player_web.dart';
import 'package:flutter_web_plugins/flutter_web_plugins.dart';

void registerPlugins([final Registrar? pluginRegistrar]) {
  final Registrar registrar = pluginRegistrar ?? webPluginRegistrar;
  CameraPlugin.registerWith(registrar);
  UrlLauncherPlugin.registerWith(registrar);
  VideoPlayerPlugin.registerWith(registrar);
  registrar.registerMessageHandler();
}

Issues

Fixes: https://github.com/flutter/flutter/issues/106765
Unblocks: https://github.com/dart-lang/dart-services/pull/874 (or probably a newer version!)

Tests

Added a unit test to ensure the file is created in the expected location.

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

/cc @srawlins hey Sam, is .dart_tool/dartpad OK, or does it need to be more specific?

/cc @srawlins hey Sam, is .dart_tool/dartpad OK, or does it need to be more specific?

That should be fine. =)

@domesticmouse once this lands, should we attempt to cherry-pick this fix into beta or stable, or can you test/deploy dartpad from a master branch of flutter?

This is only an issue in beta, so no stable cherry pick is required. It's easiest for me to verify once it is in a named beta branch. I'd have to do some investigation to figure out how to get the tooling to work with master

@domesticmouse AFAIK once this lands, we can request a cherry-pick to beta. That should unblock your work faster than this landing on its own, I think?

I didn't know about a cherry pick for beta. I thought we just rolled beta weekly on the way to stable. And yes, DartPad is blocked from updating it's beta backend until this lands.

@domesticmouse created the CP request for beta here: https://github.com/flutter/flutter/issues/106983

Under what conditions will .dart_tool/dartpad/plugin_registrant.dart be created? I have a simplistic project that contains a pubspec.yaml file with the following content for which flutter pub get creates a .dart_tool directory, but no .dart_tool/dartpad/plugin_registrant.dart
$ flutter --version
Flutter 3.3.0-0.0.pre • channel beta • https://github.com/flutter/flutter
Framework • revision f90da9b167 (2 days ago) • 2022-07-18 18:59:23 -0700
Engine • revision 1388adb442
Tools • Dart 2.18.0 (build 2.18.0-271.2.beta) • DevTools 2.15.0
$ flutter pub get
Running "flutter pub get" in simple_project...                      4.4s

name: simple_project
environment:
  sdk: '>=2.18.0-0 <3.0.0'
dependencies:
  flutter:
    sdk: flutter
  flutter_test:
    sdk: flutter
  archive: 3.3.1
  async: 2.9.0
  bloc: 8.0.3
  boolean_selector: 2.1.0
  characters: 1.2.1
  clock: 1.1.1
  cloud_firestore: 3.3.0
  cloud_firestore_platform_interface: 5.6.0
  cloud_firestore_web: 2.7.0
  collection: 1.16.0
  crypto: 3.0.2
  english_words: 4.0.0
  fake_async: 1.3.1
  ffi: 2.0.1
  file: 6.1.2
  firebase_analytics: 9.2.0
  firebase_analytics_platform_interface: 3.2.0
  firebase_analytics_web: 0.4.1
  firebase_auth: 3.4.2
  firebase_auth_platform_interface: 6.3.2
  firebase_auth_web: 4.0.0
  firebase_core: 1.19.2
  firebase_core_platform_interface: 4.4.3
  firebase_core_web: 1.7.0
  firebase_database: 9.0.19
  firebase_database_platform_interface: 0.2.1+11
  firebase_database_web: 0.2.1
  firebase_messaging: 12.0.0
  firebase_messaging_platform_interface: 4.0.0
  firebase_messaging_web: 3.0.0
  firebase_storage: 10.3.2
  firebase_storage_platform_interface: 4.1.11
  firebase_storage_web: 3.3.0
  flame: 1.2.1
  flame_fire_atlas: 1.2.0
  flame_forge2d: 0.12.1
  flame_splash_screen: 0.1.0
  flame_tiled: 1.6.0
  flutter_bloc: 8.0.1
  flutter_hooks: 0.18.5+1
  flutter_lints: 2.0.1
  flutter_riverpod: 1.0.4
  forge2d: 0.11.0
  go_router: 4.1.1
  google_fonts: 3.0.1
  hooks_riverpod: 1.0.4
  http: 0.13.4
  http_parser: 4.0.1
  intl: 0.17.0
  js: 0.6.4
  lints: 2.0.0
  logging: 1.0.2
  matcher: 0.12.12
  material_color_utilities: 0.1.5
  meta: 1.8.0
  nested: 1.0.0
  ordered_set: 5.0.0
  path: 1.8.2
  path_provider: 2.0.11
  path_provider_android: 2.0.16
  path_provider_ios: 2.0.10
  path_provider_linux: 2.1.7
  path_provider_macos: 2.0.6
  path_provider_platform_interface: 2.0.4
  path_provider_windows: 2.1.0
  petitparser: 5.0.0
  platform: 3.1.0
  plugin_platform_interface: 2.1.2
  process: 4.2.4
  provider: 6.0.3
  riverpod: 1.0.3
  rxdart: 0.27.5
  source_span: 1.9.0
  stack_trace: 1.10.0
  state_notifier: 0.7.2+1
  stream_channel: 2.1.0
  string_scanner: 1.1.1
  term_glyph: 1.2.1
  test_api: 0.4.12
  tiled: 0.8.4
  typed_data: 1.3.1
  vector_math: 2.1.2
  win32: 2.7.0
  xdg_directories: 0.2.0+1
  xml: 5.4.1

@domesticmouse the file should be created when there's any web plugin declared in the dependencies (it uses the same logic as flutter build, just at a slightly different time).
If you're using flutter from git, though, please try this before flutter pub get:
rm your/path/to/flutter/bin/cache/flutter_tools.*
Otherwise you'll be using a cached version of the flutter_tools, and you won't get the new behavior from beta at all.

PS: This is what the cache directory looks like in my box (we want to delete the flutter_tools.snapshot and .stamp):
$ ls ~/github/flutter/bin/cache
android-sdk.stamp      flutter_sdk.stamp            flutter_web_sdk.stamp   libplist.stamp        pkg
artifacts              flutter_tools.snapshot       font-subset.stamp       linux-sdk.stamp       usbmuxd.stamp
dart-sdk               flutter_tools.stamp          gradle_wrapper.stamp    lockfile
downloads              flutter_version_check.stamp  ios-deploy.stamp        material_fonts.stamp
engine-dart-sdk.stamp  flutter_web_sdk              libimobiledevice.stamp  openssl.stamp

Once you've deleted the files, you'll get a Building flutter tool... message the next time you use flutter, and a slightly lengthier execution time; you can use flutter normally afterwards:
$ rm ~/github/flutter/bin/cache/flutter_tools.*
$ flutter --version
Building flutter tool...
Flutter 3.1.0-0.0.pre.1775 • channel master • git@github.com:ditman/flutter.git
Framework • revision a400d132c1 (2 hours ago) • 2022-07-20 21:04:06 -0400
Engine • revision 7892b36d1e
Tools • Dart 2.19.0 (build 2.19.0-16.0.dev) • DevTools 2.15.0

Hey @ditman, here's what I'm seeing:
$ rm ~/flutter/bin/cache/flutter_tools.s*
$ flutter --version
Building flutter tool...
Flutter 3.3.0-0.0.pre • channel beta • https://github.com/flutter/flutter
Framework • revision f90da9b167 (2 days ago) • 2022-07-18 18:59:23 -0700
Engine • revision 1388adb442
Tools • Dart 2.18.0 (build 2.18.0-271.2.beta) • DevTools 2.15.0
$ flutter pub get
Running "flutter pub get" in simple_project...                      4.3s
$ ls -l .dart_tool
total 88
-rw-r--r--  1 brettmorgan  primarygroup  18607 21 Jul 13:45 package_config.json
-rw-r--r--  1 brettmorgan  primarygroup  16871 21 Jul 13:45 package_config_subset
-rw-r--r--  1 brettmorgan  primarygroup     13 21 Jul 13:45 version

What am I missing?

Well, let me switch to beta real quick :)

OK, I only added an extra flutter clean; here's what I see:
$ flutter --version
Flutter 3.3.0-0.0.pre • channel unknown • unknown source
Framework • revision f90da9b167 (2 days ago) • 2022-07-18 18:59:23 -0700
Engine • revision 1388adb442
Tools • Dart 2.18.0 (build 2.18.0-271.2.beta) • DevTools 2.15.0

$ rm ~/github/flutter/bin/cache/flutter_tools.*
$ flutter clean
Building flutter tool...
Deleting .dart_tool...                                               3ms
Deleting Generated.xcconfig...                                       0ms
Deleting flutter_export_environment.sh...                            0ms
Deleting ephemeral...                                                0ms
Deleting ephemeral...                                                0ms
Deleting ephemeral...                                                0ms
Deleting .flutter-plugins-dependencies...                            0ms
Deleting .flutter-plugins...                                         0ms

$ ls .dart_tool
ls: cannot access '.dart_tool': No such file or directory

$ flutter pub get
Running "flutter pub get" in simple_project...                   1,487ms

$ cat .dart_tool/dartpad/web_plugin_registrant.dart
// Flutter web plugin registrant file.
//
// Generated file. Do not edit.
//

// ignore_for_file: type=lint

import 'package:cloud_firestore_web/cloud_firestore_web.dart';
import 'package:firebase_analytics_web/firebase_analytics_web.dart';
import 'package:firebase_auth_web/firebase_auth_web.dart';
import 'package:firebase_core_web/firebase_core_web.dart';
import 'package:firebase_database_web/firebase_database_web.dart';
import 'package:firebase_messaging_web/firebase_messaging_web.dart';
import 'package:firebase_storage_web/firebase_storage_web.dart';
import 'package:flutter_web_plugins/flutter_web_plugins.dart';

void registerPlugins([final Registrar? pluginRegistrar]) {
  final Registrar registrar = pluginRegistrar ?? webPluginRegistrar;
  FirebaseFirestoreWeb.registerWith(registrar);
  FirebaseAnalyticsWeb.registerWith(registrar);
  FirebaseAuthWeb.registerWith(registrar);
  FirebaseCoreWeb.registerWith(registrar);
  FirebaseDatabaseWeb.registerWith(registrar);
  FirebaseMessagingWeb.registerWith(registrar);
  FirebaseStorageWeb.registerWith(registrar);
  registrar.registerMessageHandler();
}

😰

@domesticmouse finally found the bit that determines if a project needs web support or not:
https://github.com/flutter/flutter/blob/main/packages/flutter_tools/lib/src/project.dart#L722-L723
The tool checks that web/index.html exists, but it doesn't look inside the file at all.


Add client-side interception on GraphQlClient

Similar to the WebInterceptor chain on the server side but for use with GraphQlClient. This would also include a WebSocket specific interceptor for WebSocket specific needs (connection_init and connection_ack payload, disconnects, etc).

Also commits f54ee52d and bb628086 incorrectly linked to different issue.


SBOM Task Outputs Directly to Console Instead of Using MSBuild Logging APIs

When using the SBOM task in an MSBuild project, the task outputs messages directly to the console rather than utilizing MSBuild's Logging APIs. This behavior results in cluttered and unstructured console output, which is not consistent with standard MSBuild practices. Proper logging through MSBuild's APIs would allow for better categorization and filtering of messages.
Steps to Reproduce:

Create a .NET project and include the SBOM task.
Run the command dotnet pack -bl to build the project with a binary log.
Observe the console output during the build process.

Expected behavior:
Sbom task should output messages through MSBuild logging APIs & categorized appropriately.
We can utilize TaskLoggingHelper:
public class SbomTask : Microsoft.Build.Utilities.Task
{
    public override bool Execute()
    {
        // Example of logging an informational message
        Log.LogMessage(MessageImportance.High, "Finding components...");
        // Example of logging a warning
        Log.LogWarning("No instructions received to scan docker images.");
        // Example of logging an error
        Log.LogError("An error occurred during SBOM generation.");

        // Rest of the task implementation

        return !Log.HasLoggedErrors;
    }
}

@baronfel @KalleOlaviNiemitalo I assumed that you get MSBuild logging for free when using ToolTasks, is that not the case? I have fixed the issue that was blocking the .NET framework case and in my testing its logging output visually appears to be coming via the MSBuild APIs but I'm not entirely sure how to tell.
As for the non-.NET framework case, it seems like some additional investigation is required here to figure out how best to implement this. My first thought was to add an option to AddSbomTool (which is called in the task set up) to allow users to request a MSBuild logger instead of our normal serilog logger, but I haven't had time to look into this further and determine if that would actually work. If you think making a bridge between serilog and the MSbuild logger would work/ be simpler I'd love to hear any more information you have

@sfoslund ToolTask does do the logging - the case I was talking about was the modern .NET case. I think your proposed path should be ok - you know the logging infra for your tool better than I do :) - and I don't have any specific requirements there aside from "don't write directly to stdout".

Sounds good, I have a branch where I seem to have things working when testing e2e but some tests are failing due to a missing MSBuild assembly and the task logging before being initialized. Unfortunately I am being pulled off this work so I likely won't be able to look further into this any time in the near future, but we will be happy to accept contributions if anyone wants to work off of these changes and prep a PR.

I have a PR that fixes the failing tests (at least on my machine) here.


[WIP] Reintroduce Cosmopolitan

Description of changes
Goal:

provide cosmosStdenv that doesn't have build-time or runtime reference to packages outside it, but behaves somewhat like stdenv, and cooperate well with cmake/meson.

build from source, bootstrap from upstream released package.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Hey, is this something you're still working on? If not, I'd update cosmpolitan and cosmocc to the latest so that they're usable outside of Nix again, but sourced via Nix. (i.e. no cosmoccStdenv)


style-loader version update

The version of style-loader is too low for sass to work.

@microsoft-github-policy-service agree


Fix references to roles in example Users config

It appears the comments for the example Unicorn.Users config used comments from Unicorn.Roles and weren't completely updated.
Lines 13 and 26 might also need to be updated?

Danke :)


Feature to EnhancedRaidFrames regarding Party/Raid Leader (Crown Symbol)

Hey!
What I'd love to see is the Raid/Party Leader Symbol option in this addon!
This is how the crown looks like:
https://gyazo.com/eeeac9f27ddbe920ec8c09be7653039a
this is how it looks like on another addon called grid2:
https://gyazo.com/b8da9e3e68e7df5e351575e54143ce07
nothing more to be said, really. Just like the Raid/Party Markings option, this should be added the same way
cheers

also another suggestion I'd love to see:
In the standard version you can only choose between "Keep groups together" and then either choose to use horizontal/vertical groups, or sort by Role, Group or Alphabetical. I'd love to use the Horizontal groups + Tanks first in a Raid group (which is not possible currently).
https://gyazo.com/be1b8fb8c37883b3d8c20d6f550877ca
so basically like this


Jesse: Watch sxill.com

Jesse requests the watch of the watch_keyword sxill\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sxill\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


stats view problem and some wrong upgrades

The stats value for armors and gathering rate should have two digits and the conversion rate of priests should decrease after their upgrade.
Babylonian: royal guard champion upgrade increase damage by 50% which is only shown as an increase in attack rate in the tool.
Persian: rigorous training doesn't apply to immortals, rams, war wagon, and palintonon.
and barrack's upgrades don't apply to immortals.

Relentless Zeal is also wrong. It should apply only to Immortal Camp, Siege Workshop, and Fortress.


Bad state error when the mocked class implements an abstract class

Hey,
I had a question with regards to mocking classes that implement abstract classes. I keep getting this error: Bad state: No method stub was called from within 'when()'. Was a real method called, or perhaps an extension method? whenever I try to mock a class implementing an abstract class. But, if I drop the abstract class then the when clauses capture usages successfully.
Is that by design?

// Does not work when "ProgramDataSourceImpl" implements ProgramDataSource
// Otherwise it works
mockProgramDataSource.shady(
        day: 1,
        daysBefore: 2,
        daysAfter: 3,
 )

Thanks so much

Hello there,
The mock should be working fine.
Just tested your example locally with mocktail 0.3.0 and I couldn't reproduce your error.

abstract class ProgramDataSource {
  Future<String> shady({
    required DateTime day,
    required int daysBefore,
    required int daysAfter,
  });
}

class ProgramDataSourceImpl implements ProgramDataSource {
  Future<String> shady({
    required day,
    required daysBefore,
    required daysAfter,
  }) =>
      Future.error(Exception());
}

class MockProgramDataSource extends Mock implements ProgramDataSourceImpl {}

void main(){
  test('something else', () async {
    final mockProgramDataSource = MockProgramDataSource();
    when(() => mockProgramDataSource.shady(
      day: any(named: "day"),
      daysBefore: any(named: "daysBefore"),
      daysAfter: any(named: "daysAfter"),
    )).thenAnswer((_) async => Future.value("this worked!"));

    final result = await mockProgramDataSource.shady(
      day: DateTime.now(),
      daysBefore: 2,
      daysAfter: 3,
    );

    expect(result, 'this worked!');
  });
}

Make sure mockProgramDataSource is an instance of the mock class, not ProgramDataSource directly.
Also, day ins your example is DateTime and you call it with an integer.

Yep, my mistake. The actual issue was because I had a function returning an instance of the mock, but with a dynamic type return.

generateMockProgramDataSource() {
  return MockProgramDataSource();
}
final mockProgramDataSource = generateMockProgramDataSource(); // This causes this line to be of type dynamic

// This now won't work.
when(() => mockProgramDataSource.shady(
    day: any(named: "day"),
    daysBefore: any(named: "daysBefore"),
    daysAfter: any(named: "daysAfter"),
  )).thenAnswer((_) async => Future.value("this worked!"));

But adding the correct type signature will make things work as intended.

MockProgramDataSource  generateMockProgramDataSource() {
  return MockProgramDataSource();
}

Apologies for the confusion.


An 3519/core validator tables

Create core tables from silver validator data
adjust cols to array/object as needed

Looks good. Confirming, start slot / end slot is the same for the entire epoch right? Or is it at the validator level? (i.e., epoch 450, all vote_pubkey have exact same start slot & end slot?)

Yes thats correct, every validator in an epoch will have the same start_slot + end_slot


CTCCriterion cannot correctly calculate gradInput in cpu mode

After calling backward method, the gradInput is always zeros.
In CTCCriterion:updateGradInput method, I modified the following line from:
cpu_ctc(acts:float(), grads:float(), labels, sizes)
to:
grads = grads:float()
cpu_ctc(acts:float(), grads:float(), labels, sizes)
and it seems to work correctly after reinstall

@SeanNaren Can you look into this?

Woops sorry should've seen this post earlier, I've implemented the fix in this PR #57. Thanks!


[Image Request]: go:latest for linux/arm/v7

What software would you like us to add to our image catalog? (one per issue please)
6b6cbc61406fe3125119d5b7d4a19ffa6945e483 changed the default go image from alpine/musl to wolfi/glibc.  As a result, it reduced the supported architectures to just linux/amd64 and linux/arm64.  I'm guess the alpine/musl builds ended up building for all architectures that alpine supports?
Are you intentionally limiting wolfi images to just those architectures, or would you be willing to also include linux/arm/v7?  Or should we just manually switch over to the musl versions (which is fine too, I just wanted to check).
which versions of the containerized software do you need?
No response
Add some justification for why this specific image and versions are important.
No response

Actually one issue with us switching the musl image is that it doesn't appear to exist for specific tagged go versions?  For example, I don't think I can currently build for linux/arm/v7 with go1.19:
$docker pull --platform linux/arm/v7 cgr.dev/chainguard/go:1.19-musl
Error response from daemon: manifest unknown

$ docker pull --platform linux/arm/v7 cgr.dev/chainguard/go:1.19
1.19: Pulling from chainguard/go
no matching manifest for linux/arm/v7 in the manifest list entries

I suspect that 6b6cbc61406fe3125119d5b7d4a19ffa6945e483 updated the go:1.19 image in addition to go:latest.  I'd argue that it shouldn't have.  Using go:latest comes with the understanding that you are living on the bleeding edge.  By choosing go:1.19 I would expect more stability.  I think switching go:latest to wolfi is fine, as well as go:1.20 and going forward, but don't change already published stable versions.

Thanks @willnorris. Good questions.
We did change the image to a wolfi build. The issue with producing an arm/v7 image for wolfi is that we have to build new packages and there is also a cost in setting up a new build architecture and sourcing servers (or just using qemu at a CPU cost). We could do it, but it depends on how useful it is vs the cost.
In tailscale's case, I assume you see a lot of arm/v7 due to people running home labs with rpis?
For the time being, you will need to use the musl image if you want arm/v7 support, but hopefully this will change in the future.
Apologies about changing the go:1.19 image. I don't want to revert the change as that would cause more confusion but we should have been more careful here.

Thanks, I do understand that adding arm/v7 support is a larger lift not to be done lightly.  We can switch over to another image for the time being.  It looks like there is only a latest-musl image, but no versioned images such as 1.19-musl.  Do you plan to publish those tags as well, or is musl a "best effort" and the real energy is on the wolfi images?

It seems the tags need some work. There should be 1.19, 1.19-musl, 1.20, 1.20-musl and probably 1.20rc1 (which would currently be the same).
Both latest and latest-musl (correctly) point to 1.19, not 1.20. That's really not clear at the moment though. I'll try to get this sorted asap.

docker pull --platform linux/arm/v7 cgr.dev/chainguard/go:1.19-musl is succeeding now, so I think we're in good shape.  Thanks.  Realistically, we should be able to just cross-compile our binary in a standard linux/amd64 build image, which is the direction we'll eventually go.  I think everything here is done (at least in terms of the original request), so I'll close this out.

Thanks @willnorris!


No data available in table on all tabs

I've recently updated the munkireport server and pushed out the new client. However on newly installed machines there is no data uploaded to the server. Almost every tab shows a message like no data available or no install history. Only the summary screen shows some information.
The preflight.d directory only has the submit.preflight script. The postflight.d directory stays empty.
I've been searching for a while now but I can't solve it and I'm not sure where to look anymore.
MunkiReport Version <IP_ADDRESS>8

You need to enable the modules you want to run on the client. Please read https://github.com/munkireport/munkireport-php/blob/master/docs/setup.md#select-which-modules-to-install-on-the-client


[1.12] Crash trying to integrate Controllable with my custom modpack

Hello!
I was helping a friend to setup my custom modpack (~103 mods) and he wanted to use Controllable to use his PS5 controller (via DS4Windows) to play the game.
I never used Controllable before, but it was the first option available for 1.12 when i searched on CF, so i indicated it for him to download to his mods folder. At the first the mod ran as intended, but after a while he ran into this crash.
I dont have much to say since i never played it before, so just created this ticket to see if someone can help us about that.
His specs:
Minecraft 1.12.2
Forge <IP_ADDRESS>60 for 1.12.2
Windows 11 x64

Closing as I no longer maintain the 1.12.2 version of the mod.


Not working on contentEditable component

When I try to use react-linkify-it to wrap a contentEditable component it does not render a tag;
code example:
<LinkItUrl>           <LinkItEmail>           <div  contentEditable={true} />           </LinkItEmail>         </LinkItUrl>

This would be out of this projects's intended use, using contenteditable with react is not straight forward.
In short if the link has to be clickable it needs to set contenteditable false for it.
https://codesandbox.io/s/react-linkify-it-forked-8mp0w3?file=/src/App.js
If you want to edit as well then you'll have to sync the dom text with react manually.


mgr: append balancer module to ceph_mgr_modules

otherwise the osd play in rolling_update can fail when it tries to
disable it before upgrading osd nodes.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>(cherry picked from commit 45a1d634d850ad971c71dbe9191b97caa3068b5a)

jenkins test centos-container-all_daemons


Teardown preview URLs

Also changed the URL to avoid Chrome warnings about phishing

Thank you, @gastaldi!


Adds PlayerInteractEvent consumer

Added 4 new things to ItemBuilder.java  removeInteractEvent , onInteractEvent and onPlayerInteractEvent which lets the developer listen for PlayerInteractEvent and register their functions in it.

This won't work, and isn't functionality I want to provide.


Possibility to use action :  "variant" and variant_purpose :  "comparison_implicit"

Describe the feature or improvement you're requesting
I try to regenerate response, like in official Chat.
If I use regenerate in Chat, I can see this in request:
{
    "action": "variant",
    "messages": [
        {
            "id": "a4c9ab9a-99a0-4ea7-aa69-5e80c0b50120",
            "author": {
                "role": "user",
                "metadata": {}
            },
            "create_time":<PHONE_NUMBER>.464652,
            "content": {
                "content_type": "text",
                "parts": [
                    "Ceci est un test"
                ]
            },
            "status": "finished_successfully",
            "weight": 1,
            "metadata": {
                "timestamp_": "absolute"
            },
            "recipient": "all"
        }
    ],
    "conversation_id": "72704a41-7444-4b66-b592-ec2eaa225570",
    "parent_message_id": "66b4468a-4f56-4f8b-aa7e-78446b6540fc",
    "model": "text-davinci-002-render-sha",
    "timezone_offset_min": -120,
    "variant_purpose": "comparison_implicit",
    "history_and_training_disabled": false,
    "arkose_token": null
}

2 fields are interesting:

variant_purpose
action

It is currently not possible to use them in lib. I try to ask to ChatGTP himself. He suggest me this:
{
...,
  action: {
      action_type: 'variant',
      variant_purpose: 'comparison_implicit',
    },
}

But if I precise my current openai version, he respond me this:

I apologize for any confusion. You are correct that in version 4.0.0-beta.6 of the OpenAI SDK, the "action" parameter is not available in the chat.completions.create function. I apologize for the misinformation.
Currently, the "action" and "variant_purpose" parameters are not directly supported in the chat.completions.create function. These parameters are specific to the GPT-3 chat interface on the OpenAI website.
If you want to control the generation of the response with similar parameters, you can use the available parameters in the SDK such as "temperature" and "max_tokens" to adjust the randomness and length of the generated response.
You can also modify the content of the prompt to influence the generation of the response by adding specific instructions or constraints. Experiment with different prompts to achieve the desired results.
I encourage you to refer to the official OpenAI SDK documentation for up-to-date information on the available options: https://github.com/openai/openai-node

This feature can be very useful to use. Actually, I renew manually the request.
Additional context
openai version: 4.0.0-beta.6

Thank you for the suggestion! We'll pass this request on to our API team. It may help if you can share more about why these features would be useful to you.

Closing this as it's an API feature request rather than a client library feature request. If you're still interested in this functionality, please post it in community.openai.com


Allow clients to set a log callback.

~This depends on https://github.com/mozilla/cubeb/pull/802.~ no longer depends on a cubeb change.

(nb: this won't compile until the aforementioned cubeb PR is merged)

Should be good with #100 merged no?

Yes, I think this is good to go now. thanks!

I believe this is the last of my pending changes, so if you could please cut a release to crates.io after this is in (and we've confirmed CI is still passing) I would very much appreciate it!

@padenot is this OK to merge? thanks!


Tree component

Which scope/s are relevant/related to the feature request?
Don't know / other
Information
Just wanted to know if this ui components need this one?

Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[X] No

I think that would be cool to have! The Angular CDK has a component also: https://material.angular.io/cdk/tree/overview


When one  tries to evaluate OJS variables more than once in a paragraph, {knitr} can start seeing inline LaTeX

Example here.  Source code available there as well.
Would it be possible to allow users to specify a custom symbol (i.e., other than $) for the evaluation of OJS expressions?

That worked, thanks!

Great!
I went to close this but I'll leave it open to remind me to add a note to the documentation.


Add a DigitalOcean Kubernetes blog post

This simply references our new DigitalOcean tutorial, but
this should help raise visibility amongst our own community!

@cnunciato hoping to get this in today just to get a little blog love out there -- lmk if you see anything to change and I'm happy to edit afterwards.

Oh, sorry! Didn't notice this until just now.
Looks good! 👍


Staging for version increment automation

Signed-off-by: pgodithi<EMAIL_ADDRESS>Description
This is the staging PR for to execute workflows that auto raise the version increment PR's
Example: https://github.com/prudhvigodithi/anomaly-detection/pull/1
Issues Resolved
Part of: https://github.com/opensearch-project/opensearch-build/issues/1375
From solution: https://github.com/opensearch-project/opensearch-build/issues/1375#issuecomment-1159476686
Related issue: https://github.com/opensearch-project/anomaly-detection/issues/605
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Sample PR
anomaly-detection main : OpenSearch main
https://github.com/prudhvigodithi/anomaly-detection/pull/2

Codecov Report

Merging #608 (4130328) into main (f630c8f) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #608      +/-   ##
============================================
- Coverage     79.07%   79.04%   -0.04%
+ Complexity     4213     4207       -6
============================================
  Files           296      296
  Lines         17686    17684       -2
  Branches       1880     1880
============================================
- Hits          13986    13978       -8
- Misses         2805     2807       +2
- Partials        895      899       +4

Flag
Coverage Δ

plugin
79.04% <ø> (-0.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ain/java/org/opensearch/ad/model/ModelProfile.java
69.09% <0.00%> (-1.82%)
:arrow_down:

...rch/ad/transport/AnomalyResultTransportAction.java
80.13% <0.00%> (-0.69%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
71.93% <0.00%> (-0.38%)
:arrow_down:

...c/main/java/org/opensearch/ad/util/ParseUtils.java
77.77% <0.00%> (-0.08%)
:arrow_down:

...ava/org/opensearch/ad/model/TimeConfiguration.java
100.00% <0.00%> (ø)

Renaming to task updateVersion following https://github.com/opensearch-project/opensearch-plugin-template-java/pull/32.

Renaming to task updateVersion following https://github.com/opensearch-project/opensearch-plugin-template-java/pull/32.

@prudhvigodithi does this also handle moving main to 3.0.0-SNAPSHOT for example?

Hey @amitgalitz , following the discussion, main once inclined with Core, the task updateVersion is called against the version of the core. Please review the PR when you get a chance.
Thank you

@amitgalitz @sean-zheng-amazon  can you please review this PR soon?

Once merged will this autochange AD main to whatever Opensearch main is at? for now 3.0.0-SNAPSHOT?

Its not letting me re-run the whitesource check for some reason

@prudhvigodithi Once merged will this autochange AD main to whatever Opensearch main is at? for now 3.0.0-SNAPSHOT?

Hey @amitgalitz once PR is merged the gradle project should output the task updateVersion, now using this task once the Plugins branches are in align with core, then the workflow will call the task updateVersion to raise an auto increment version PR.
Thank you

@prudhvigodithi Once merged will this autochange AD main to whatever Opensearch main is at? for now 3.0.0-SNAPSHOT?

Hey @amitgalitz once PR is merged the gradle project should output the task updateVersion, now using this task once the Plugins branches are in align with core, then the workflow will call the task updateVersion to raise an auto increment version PR. Thank you

got it, so once this is merged, next step is to make sure we align plugin branches with core?

got it, so once this is merged, next step is to make sure we align plugin branches with core?

Right @amitgalitz, then the workflow will be triggered to compare to execute updateVersion task, which is part of this PR.

@prudhvigodithi have you had whitesource check failures on any of the other repos with this PR?

got it, so once this is merged, next step is to make sure we align plugin branches with core?

Right @amitgalitz, then the workflow will be triggered to compare to execute updateVersion task, which is part of this PR.

@prudhvigodithi I'm concerned with aligning plugin branches with core. The outlined strategy is to have different branching strategies between plugins and core.

got it, so once this is merged, next step is to make sure we align plugin branches with core?

Right @amitgalitz, then the workflow will be triggered to compare to execute updateVersion task, which is part of this PR.

@prudhvigodithi I'm concerned with aligning plugin branches with core. The outlined strategy is to have different branching strategies between plugins and core.

Hey @ohltyler the document should be updated, https://github.com/opensearch-project/opensearch-plugins/issues/142#issuecomment-1176530894

@prudhvigodithi the backports failed for some branches due to merge conflicts for build.gradle. Can you help manually resolve?

Hey @ohltyler
For 2.x and 2.1 backports I have fixed the DCO error
https://github.com/opensearch-project/anomaly-detection/pull/621
https://github.com/opensearch-project/anomaly-detection/pull/622
For 1.x series I have raised a new PR https://github.com/opensearch-project/anomaly-detection/pull/624
Please check.
Thank you


Where cannot be found in Unity.Linq

Hello.
I am having the following problem :
http://puu.sh/gWsdG/12d154c7e6.png
It wont find the Where in
_centerSlowPulse = this.gameObject.Descendants ().Where (x => x.name == "CenterSlowPulse");

You can write using System.Linq;.

System.Linq is LINQ to Objects.
https://msdn.microsoft.com/en-us/library/bb397919.aspx

Hello.
Thats the first thing I tried, gave me this error:
Assets/Dev/2D/icons/Target/Animations/TargetDotAnimationController.cs(22,33): error CS0029: Cannot implicitly convert type `System.Collections.Generic.IEnumerable<UnityEngine.GameObject>' to `UnityEngine.GameObject'

Thank you.


Create hnuit.txt

@kawa11 First stage of verification passed, official domain is hnuit.edu.cn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


Restructure course sessions query to use index

This restructures a mongodb query to allow for use of compound indexes. Please check that this is actually logically equivalent.

Coverage decreased (-0.01%) to 62.14% when pulling 36cbfe8b4ae6944795946985c665e40abfb7e0d2 on course-sessions-index into 4ca3d7791819b23a7bca0bf88b2369cf537deeeb on master.


automod: Ability to edit rules using JSON paths

Should make it significantly easier to edit existing triggers/conditions/actions without having to redefine the entire section.

Fixed by #62


Add needs_rewrite_passes and content_changed methods

These methods will be used with #639 to implement a fixpoint loop for rewrite passes.
This also lets us remove mark_changed, since we can use content_changed for the same purpose.

Eh, maybe. It'll be useful on the Masonry side. We might remove it on the Xilem side.

Nice. This simplifies the code and prevents accidentally leaving a widget stale.


[BUG]

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

发现一个bug  你这里的搜索  点击一次就是搜一次  再点击搜索  直接就不知道是什么过滤  就没东西了  然后也清不了  清空过滤词点搜索  不显示了
Expected Behavior
No response
Steps To Reproduce
No response
Environment
HertzBeat version(s):

Debug logs
No response
Anything else?
No response

i would like to try it


Add frontend test support

@acrellin I was hoping you could check this PR to see which tests are broken on your system still so I could fix them.  But now I'm sure that will become evident pretty quickly :)

And once I have Drone working, this kind of checking will be done by "the machine".

Thanks Stefan - I've added test output to the shared DB folder.
On Wed, Apr 22, 2015 at 5:15 PM, Stefan van der Walt <
<EMAIL_ADDRESS>wrote:

And once I have Drone working, this kind of checking will be done by "the
machine".
—
Reply to this email directly or view it on GitHub
https://github.com/mltsp/mltsp/pull/23#issuecomment-95373251.

Hey, Ari
On 2015-04-22 17:35:02, acrellin<EMAIL_ADDRESS>wrote:

Thanks Stefan - I've added test output to the shared DB folder.

Dropbox says it is up to date, but I don't see it--where should I
look?
Also, if you can fix the easy looking ones on your machine, that
would be helpful since it's very hard for me to debug on my end.
Stéfan

It's in the shared folder mltp - two files starting with
"casperjs_rebase..."
I'll fix the easy ones on my machine!
On Wed, Apr 22, 2015 at 6:18 PM, Stefan van der Walt <
<EMAIL_ADDRESS>wrote:

Hey, Ari
On 2015-04-22 17:35:02, acrellin<EMAIL_ADDRESS>wrote:

Thanks Stefan - I've added test output to the shared DB folder.

Dropbox says it is up to date, but I don't see it--where should I
look?
Also, if you can fix the easy looking ones on your machine, that
would be helpful since it's very hard for me to debug on my end.
Stéfan
—
Reply to this email directly or view it on GitHub
https://github.com/mltsp/mltsp/pull/23#issuecomment-95380938.

Found them, thanks!  Looks like I should be able to fix those fairly easily.


System information: Update speed option

Is ther a posibbility to add a option to change the updates speed of the sysme information graphs like in Win10 Task Manager?
Should very nice if we can set the speed a 60 fps for exaple.

You can change the update speed using the Main menu > View > Refresh interval


Unnecessary line in .gitignore?

https://github.com/Open-MSS/MSS/blob/f6b3ad110dbfaa12d439cadea6476dcd54d9328d/.gitignore#L28
Where was that line coming from again? I cannot remember what this path was used for. Is it still required?

Originally posted by @matrss in https://github.com/Open-MSS/MSS/pull/2069#discussion_r1384596047

@matrss  I hope to work on this, please assign me


Can not select Chinese characters in compiled pdf file

Description
The source code is:
bala

test 文档处理

hehe

But I can not select the Chinese characters in the compiled pdf file, the popular pdf viewer used is SumatraPDF.
https://github.com/user-attachments/assets/bb9f233f-cc71-4b92-8e7f-3147365a63e5
I have tried to check the streams ojb in the PDF file, but it is compressed.
Reproduction URL
No response
Operating system
Windows
Typst version

[X] I am using the latest version of Typst

Can you attach the actual PDF file that was produced?

Can you attach the actual PDF file that was produced?

Done

I've just tried your PDF in Chrome, Firefox, Acrobat and Apple Preview, and it works fine there. Must be an issue specific to your viewer...

Thanks, I will create a bug report for SumatraPDF.

Will close this then.

It is fixed in Version 0.12.0, Release Candidate 1 (October 3, 2024)


locale(en): Changed "{field}" to "{field} field"

🔎 Overview
Stems from the discussion in #2235
I think it's safe to add the "field" suffix to every message, even if some may be arguably less needed than others (i.e. for the confirmed rule) because no assumptions are made about the name of the field this way. "The" can always refer to "field", but "the" can't always correctly refer to the name of the field, i.e. "The assigned to" vs "The assigned to field".
Feel free to remove it for a given message if you don't think it would ever be necessary.
✔ Issues affected

closes #2235

Thanks, I will correct the tests.


1.19.2 Apotheosis crash

Hi, apparently MC is crashing when you try to launch it with TaxFreeLevels + Apotheosis the latest release version for MC 1.19.2
I saw there are responses in Issues that there was made a compatibility update with that mod, so I think that just a new bug rather than "complete" incompatibility.
Can fix with downgrading Apotheosis to 6.4.1 version.
Log - log.txt
Forge              - 43.3.0
Apotheosis     - 6.5.0
TaxFreeLevels - 1.3.9

Ah, they backported some stuff from 7.2.0+ but the version is 6.5.0, so the compatibility check doesn't pass:
https://github.com/Fourmisain/TaxFreeLevels/blob/0fcf2a2543ddf6127674a63715aa10a6f4915307/forge/src/main/java/io/github/fourmisain/taxfreelevels/forge/MixinConfig.java#L51-L53
Easy fix, I'll just put 6.5.0.
(Technically breaks support for 7.1.0, but there's no reason to use that over 7.2.0+.)
New version 1.3.10 is up!


Prereqs section is incomplete.

In addition to cloning the samples repository, for the code snippets to work, you also need to:

cd to the cli directory within that repository
for the code snippets to run, you must also run something like:

az configure --defaults group="azureml-examples-rg" workspace="main"

which is linked in this document, which in turn is linked as an optional read in pre-reqs. The issue is this step is required and not optional. You could fix this by making that document required instead of optional, or just call this step out explicitly either in pre-reqs or as steps.

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

ID: f24ec969-b354-a6ee-fde3-0b7e478a7827
Version Independent ID: 316e0373-e365-3e42-0529-850c06030b2b
Content: Use batch endpoints for batch scoring - Azure Machine Learning
Content Source: articles/machine-learning/how-to-use-batch-endpoint.md
Service: machine-learning
Sub-service: core
GitHub Login: @tracych
Microsoft Alias: tracych

Thanks for the feedback! We are currently investigating and will update you shortly.

@jreynolds01 Thanks again for pointing this out. Will update this document soon.
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


Update to Defi2.3.5 Unable to Start Blockchain

Actual behavior: Unable to start blockchain when opening app, Request failed with status code 503
Expected behavior: Open app
How to reproduce it (as minimally and precisely as possible): 1. downloaded 2.3.5 from defi website 2. replaced application 2.3.4 with 2.3.5    3. try to open app but failed to start blockchain

What are your environment parameters?

DeFi Wallet Version: 2.3.5
Operating System (OS): macOS Big Sur version 11.2.3
– Current free storage capacity: 44.9

Anything else we need to know?

logs.zip

Hi @luzettemok1 , currently the app requires at least 120 GB storage, and if you're using the snapshot to speed up the sync to the latest block, it require 40.5GB.

DeFi Wallet should prompt you about the option to delete snapshot after snapshot has been successful.

It's likely that you've insufficient storage, hence the Unable to start blockchain error. You could either try to make more space for the app or wait for the Light Wallet to be released. Please backup your wallet.dat in the mean time.
Mac: Go to folder
/Users/<username>/Library/Application Support/Defi/wallets

Regarding the issue Unable to sync to 100% #853, we can further discuss on the issue there. We would like to keep same issues in one thread. Thank you.

Thanks so much! Is there a set date for which the Light Wallet is going to be launched for my reference to download?


net/http: add audio/mp3 to DetectContentType

What is the URL of the page with the issue?
https://pkg.go.dev/net/http#DetectContentType
What is your user agent?
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0
What did you do?
I used http.DetectContentType with this uploaded MP3 file (unzip it to test):
Bagdad cafe - I'm Calling you.mp3.zip
buff, err := file.Open()
head := make([]byte, 512)
_, err = buff.Read(head)
contentType = http.DetectContentType(head)

What did you expect to see?
audio/mpeg or audio/mp3
What did you see instead?
application/octet-stream (the MIME type of this file isn't recognized)
Could you add this MIME type to http.DetectContentType?

It's already implemented, I think. Assuming your mp3 doesn't have ID3 in the first 512 bytes, this is #21124.

Duplicate of #21124


Allow command-line tool to overwrite file contents

We want to support batch-updating multiple files via the import-js
command. An important part of that is to overwrite the file with the new
content. This commit takes care of that, and at the same time adds
support for passing in only a --filename and no content via stdin.
This takes us one step closer to supporting the usecase described in
[#173].

This isn't really urgent, so don't feel stressed about looking at it @lencioni.

@lencioni This is now ready for a second review (if you have time). :)

:+1:


In C# UWP Desktop App, how can I access sensor metadata like Model (DEVPKEY_Sensor_Model)?

I've created a desktop C# app using the reunion aka Windows App SDK nuget packages and I am able to access my CustomSensor(s) thought the Sensor API and read sensor samples and some sensor properties. However I'm interested in also reading the sensor specific properties like Model (DEVPKEY_Sensor_Model), is this possible through the UWP CustomSensor API?

Hey! Sorry this took so long, but we have a few samples of how to use custom sensors in UWP here: https://github.com/microsoft/Windows-universal-samples/blob/main/Samples/CustomSensors/cs/Scenario1_DataEvents.xaml.cs
It's also possible to use DEVPKEYs like this:
try     {        sensorModel = deviceInfo.Properties[Constants.Properties["Sensor_Model"]].ToString();     }  catch { }
Where Constants.Properties["Sensor_Model"] corresponds to this in a dictionary: { "Sensor_Model", "{D4247382-969D-4F24-BB14-FB9671870BBF} 8" } and deviceInfo is a Device Information object.

Please feel free to reach out if the above does not solve your issue!


WIP: Refactor source for probe-rs-debugger

This is a 'maintain same functionality' refactor of probe-rs-debugger code, to break up large source files, and facilitate better (future) code re-use between dap_adapter.rs and repl_commands.rs.

@Yatekii This PR doesn't change any code/logic inside functions, and is just a massive refactor of files and file structure. It helps to reduce file sizes in general, but I need this if I am going to share logic between the DebugAdapter and ReplCommand implementations. Despite the current changes being low risk, I've done end-to-end testing with the VSCode extension, and all is well.
There is more work to be done, on the back of what was started in #1552, but I think it makes sense to merge this first, before I start changing code inside the bodies of functions in the next PR.