Add support for other PHP handlers (PHP-FPM, FCGI/FastCGI .. )

tested with apache 2.4.6 + php-fpm 7.4.21 under CentOS 7.9.2009

Thanks for the PR.
Why not change it as you mention first?
if ($this->phpSapi === 'apache2handler' || $this->phpSapi === 'fpm-fcgi') {
//...

Thanks for the PR.
Why not change it as you mention first?

Because there are lots of other possible values for PHP_SAPI. From  https://www.php.net/manual/en/function.php-sapi-name.php, some cases are apache, apache2handler, cgi (until PHP 5.3), cgi-fcgi, cli, cli-server, embed, fpm-fcgi, litespeed, phpdbg.
The first suggestion fixes the problem in my environment (apache + php-fpm), but it will surely fail if another combination is used, like nginx + fcgi or apache + fcgi.


MSID: 87188 Version: 1 DOI: 2<IP_ADDRESS>8237

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=87188)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/87188/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8237)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/87188)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/87188

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/87188

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/87188/v1/87188-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/87188)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Thanks @fred-atherden - email sent to Fred.

@fred-atherden for when you're well! I got this from James, so added a block label. I think the problem may have arisen from Ed Office logging the wrong preprint URL, which I copied over when creating the ticket. I'll ask them to be more careful. In this case, what to do? Is it possible to regenerate the preview?

Claudia found a bit of a big problem with preview of Reviewed Preprint 87188 (https://prod--epp.elifesciences.org/preview/87188/v1) in that it’s pulled in this preprint https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>8237v2 which we have already PoAed as article https://elifesciences.org/articles/85243! It looks like a mistake caused by similar titles. The correct preprint for 87188 is https://www.medrxiv.org/content/10.1101/2<IP_ADDRESS>86442v1, I believe.

Thanks @acollings. Good to know, shouldn't be too difficult to fix I think.


cmd/compile: generic type inference not working on return types only in a certain case

What version of Go are you using (go version)?

$ go version
go version go1.18.1 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Lich's Laptop\AppData\Local\go-build
set GOENV=C:\Users\Lich's Laptop\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\Lich's Laptop\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\Lich's Laptop\go
set GOPRIVATE=
set GOPROXY=https://goproxy.io
set GOROOT=D:\Installed Software\- Dev -\Go 1.18.1
set GOSUMDB=off
set GOTMPDIR=
set GOTOOLDIR=D:\Installed Software\- Dev -\Go 1.18.1\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.18.1
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=NUL
set GOWORK=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\LICH'S~1\AppData\Local\Temp\go-build2554615698=/tmp/go-build -gno-record-gcc-switches

What did you do?

I'm trying to define a function that returns a generic type value,but without any parameter defined as the generic.
codes  are something like below:
package main
import "encoding/json"
type ObjectConstraint interface {
	Object | Object2
	Method()
}

func DoSomething[ObjectWithConstraint ObjectConstraint]() ObjectWithConstraint {
	objectJson := "{\"content\":\"This Would Cause Compiling Error\" +}"
	var object ObjectWithConstraint
	json.Unmarshal([]byte(objectJson), object)
	return object
}

type Object struct {
	content string
}

// Just for compiling,irrelevant
func (Object) Method() {}

//  Caused Compiling Error: cannot infer ObjectWithConstraint
func DoSomething_Wrapped() Object {
	return DoSomething()
}

type Object2 struct {
	content string
}

// Just for compiling,irrelevant
func (Object2) Method() {}

// Same stuff with type Object2
func DoSomething_Wrapped() Object2 {
	return DoSomething()
}

func main() {
	object := DoSomething()
	object.Method()
}

It is able to compile with single type,since the compiler seems not gonna confuse with the type value that returned,
but with mutiple types defined in the generic constraint,things just gonna change.
As some sort of a fixture, add a dummy object in the parameter,it could just work:

func DoSomething[ObjectWithConstraint ObjectConstraint](dummyObject ObjectWithConstraint) ObjectWithConstraint{
// dummy Object unused....
}

func DoSomething_Wrapped() Object {
	dummy := Object{content: ""}
	return DoSomething( dummy )
}

// main
func main() {
	dummyObject := Object{content: ""}
	object := DoSomething(dummyObject)
	object.Method()
}

What did you expect to see?
it could just compile with the original code without that dummy,with some more accurate type inference
What did you see instead?
Compilation Error

Duplicate of #51928

This is not a dup of other issues. The OP's code is not complete.
package main
import "encoding/json"
type ObjectConstraint interface {
	Object | Object2
	Method()
}

func DoSomething[ObjectWithConstraint ObjectConstraint]() ObjectWithConstraint {
	objectJson := "{\"content\":\"This Would Cause Compiling Error\" +}"
	var object ObjectWithConstraint
	json.Unmarshal([]byte(objectJson), object)
	return object
}

type Object struct {
	content string
}

// Just for compiling,irrelevant
func (Object) Method() {}

//  Caused Compiling Error: cannot infer ObjectWithConstraint
func DoSomething_Wrapped() Object {
	return DoSomething[Object]()
}

type Object2 struct {
	content string
}

// Just for compiling,irrelevant
func (Object2) Method() {}

// Same stuff with type Object2
func DoSomething_Wrapped2() Object2 {
	return DoSomething[Object2]()
}

func main() {
	object := DoSomething[Object]()
	object.Method()
}

looks the same to me, it's asking for type inference based on the context in which a function occurs.
func DoSomething_Wrapped() Object {
	return DoSomething()
}

is the same as asking for
var _ Object = DoSomething() // infer type for DoSomething


Please update to Preview 4

Please accept pull request to support preview 4.

please DO !!

+1 to that!!!!

Done!


Recursive directory lookup can take long

This line looks for requirements.txt under all directories, this can be especially costly if you have data directories.
One way to get around this is to only look at specific set of directories and ignore data directories.

@abhiskk What about looking only inside habitat_baselines (plus the main one)?

Yes, I think that should be good.


下载 Material fonts 一直失败是什么原因呢

Failed to download
https://mirrors.tuna.tsinghua.edu.cn/flutter/flutter_infra_release/flutter/fonts/bd151aa3c2f7231344411a01dba4ef61b3cd56b
2/fonts.zip. Ensure you have network connectivity and then try again.
SocketException: OS Error: 信号灯超时时间已到
, errno = 121, address = mirrors.tuna.tsinghua.edu.cn, port = 59895
这个链接又是可以打开的

一般是网络问题。多事几次就好了


Edit command in solution level ? Directory problem ?

Currently edit command has a standard format,
edit Index i/statement d/description t/tag
edit Index s/solutionlink r/remark
Users can select the issue from the root directory, and also can select a solution to display the webpage.
If a user is in the solution level, root/issue 1/solution 2, he is not allowed to edit link and remark (we didn't expect this before). If we allow editing in solution level, then edit command is unnecessary to have index then. It will have the third format. Not consistent anymore. Hence I may think we should allow the user to go back to the second level to edit. Selects a solution only for a webpage.
root - edit any issue description, remark or tag
root/issue' - edit any solution link and remark root/issue/solution` - ? （go upper level, or allow to edit this solution, which has a third format)
Also if we display the directory information to a user, but we cannot go back to the second level directory from the third level but only can go home directory, is quite wierd.

I think its fine?
After select SolutionIndex1, edit SolutionIndex2 ... should change the values at SolutionIndex2 and do a select as well to change to the other Solution2
select solution just has the function of displaying the link. Thoughts?

is it wired when you are in solution 2 (which directory will tell you are solution 2), but allow users to edit other solution?
anw, if it makes sense, I will change edit commands before the tutorial.

It may be a bit weird, but maybe if we display the message properly it would be better, like maybe say something like "Currently viewing solution 2".
Or we could even do the autosuggest for commands - and suggest only the commands that can be called at each level. Let's discuss during tutorial tmr? As long as we make the changes by Friday 12pm should be fine.


run harness on A770 error

when i run harness as the following link on A770
https://github.com/intel-analytics/ipex-llm/blob/main/python/llm/dev/benchmark/harness/run_llb.py
the cmd is：python run_llb.py --model ipex-llm --pretrained /home/test/models/LLM/baichuan2-7b/pytorch/ --precision sym_int4 --device xpu --tasks hellaswag --batch 1 --no_cache
it occurs this error：
RuntimeError: Job config of task=hellaswag, precision=sym_int4 failed. Error Message: 'utf-8' codec can't decode byte 0xb5 in position 1: invalid start byte

Could you please remove the try-except clause here and provide more error log?

(llm) test@test-Z590-VISION-D:~/ipexllm_whowhat/ipex-llm/python/llm/dev/benchmark/harness$ python run_llb.py --model ipex-llm --pretrained /home/test/models/LLM/baichuan2-7b/pytorch/ --precision sym_int4 --device xpu --tasks hellaswag --batch 1 --no_cache
/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/transformers/deepspeed.py:23: FutureWarning: transformers.deepspeed module is deprecated and will be removed in a future version. Please import deepspeed modules directly from transformers.integrations
warnings.warn(
/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: ''If you don't plan on using image functionality from torchvision.io, you can ignore this warning. Otherwise, there might be something wrong with your environment. Did you have libjpeg or libpng installed before building torchvision from source?
warn(
2024-10-30 11:06:38,081 - INFO - intel_extension_for_pytorch auto imported
Selected Tasks: ['hellaswag']
The repository for /home/test/models/LLM/baichuan2-7b/pytorch/ contains custom code which must be executed to correctly load the model. You can inspect the repository content at https://hf.co//home/test/models/LLM/baichuan2-7b/pytorch/.
You can avoid this prompt in future by passing the argument trust_remote_code=True.
Do you wish to run the custom code? [y/N] y
2024-10-30 11:06:40,365 - WARNING - Xformers is not installed correctly. If you want to use memory_efficient_attention to accelerate training use the following command to install Xformers
pip install xformers.
/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/torch/_utils.py:831: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
return self.fget.get(instance, owner)()
2024-10-30 11:06:55,197 - INFO - Converting the current model to sym_int4 format......
Traceback (most recent call last):
File "/home/test/ipexllm_whowhat/ipex-llm/python/llm/dev/benchmark/harness/run_llb.py", line 147, in
main()
File "/home/test/ipexllm_whowhat/ipex-llm/python/llm/dev/benchmark/harness/run_llb.py", line 101, in main
results = evaluator.simple_evaluate(
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/test/lm-evaluation-harness/lm_eval/utils.py", line 243, in _wrapper
return fn(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^
File "/home/test/lm-evaluation-harness/lm_eval/evaluator.py", line 89, in simple_evaluate
task_dict = lm_eval.tasks.get_task_dict(tasks)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/test/lm-evaluation-harness/lm_eval/tasks/init.py", line 390, in get_task_dict
task_name_dict = {
^
File "/home/test/lm-evaluation-harness/lm_eval/tasks/init.py", line 391, in
task_name: get_task(task_name)()
^^^^^^^^^^^^^^^^^^^^^
File "/home/test/lm-evaluation-harness/lm_eval/base.py", line 481, in init
self.download(data_dir, cache_dir, download_mode)
File "/home/test/lm-evaluation-harness/lm_eval/base.py", line 510, in download
self.dataset = datasets.load_dataset(
^^^^^^^^^^^^^^^^^^^^^^
File "/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/datasets/load.py", line 2606, in load_dataset
builder_instance = load_dataset_builder(
^^^^^^^^^^^^^^^^^^^^^
File "/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/datasets/load.py", line 2277, in load_dataset_builder
dataset_module = dataset_module_factory(
^^^^^^^^^^^^^^^^^^^^^^^
File "/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/datasets/load.py", line 1923, in dataset_module_factory
raise e1 from None
File "/home/test/miniforge3/envs/llm/lib/python3.11/site-packages/datasets/load.py", line 1875, in dataset_module_factory
can_load_config_from_parquet_export = "DEFAULT_CONFIG_NAME" not in f.read()
^^^^^^^^
File "", line 322, in decode
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb5 in position 1: invalid start byte

Given this issue may caused by datasets lib version. Could you please provide some information about python libs version, so that we can reproduce the issue.

(llm) test@test-Z590-VISION-D:~/ipexllm_whowhat/ipex-llm/python/llm/dev/benchmark/harness$ pip show datasets
DEPRECATION: Loading egg at /home/test/miniforge3/envs/llm/lib/python3.11/site-packages/whowhatbench-1.0.0-py3.11.egg is deprecated. pip 24.3 will enforce this behaviour change. A possible replacement is to use pip for package installation. Discussion can be found at https://github.com/pypa/pip/issues/12330
Name: datasets
Version: 2.21.0
Summary: HuggingFace community-driven open-source library of datasets
Home-page: https://github.com/huggingface/datasets
Author: HuggingFace Inc.
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/test/miniforge3/envs/llm/lib/python3.11/site-packages
Requires: aiohttp, dill, filelock, fsspec, huggingface-hub, multiprocess, numpy, packaging, pandas, pyarrow, pyyaml, requests, tqdm, xxhash
Required-by: lm_eval, optimum, optimum-intel

Our verified datasets lib version is 2.14.6, could you please try it in your env. At the same time, we will reproduce the issue with the datasets version your provided.

pip install datasets==2.14.6
when i change the datasets lib version with 2.14.6, it success.
Thanks for your supporting~


feat: Refactored Date and Date Range picker

Description
This PR includes refactored date and date range picker.
Fixes # (issue)
https://github.com/appsmithorg/appsmith/issues/19323
https://github.com/appsmithorg/appsmith/issues/19322
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual on storybook
Manual on main repo
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing
https://github.com/appsmithorg/design-system/pull/414#issuecomment-1469996724
https://github.com/appsmithorg/design-system/pull/414#issuecomment-1471903948
https://github.com/appsmithorg/design-system/pull/414#issuecomment-1473794440
Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

Bugs in this PR:
Functional:

Deleting the JSON string in the excludesDates control does not reflect on the canvas when done in the raw mode.
You can only enter 1 character at a time in the yearStartRange field. The cursor shifts after entering only one digit. Same happens with the Label field. (this happens intermittently)
The year dropdown always starts from the yearStartRange. This can be frustrating for the user who wants to select a relatively closer date. (suggestion is to show the selected year in the middle or something once the dropdown is clicked.)
Once we set the yearEndRange control, the year dropdown does not appear on clicking. Can we modify the desc so that it is not confusing for devs?
What is the difference between isDisabled & isReadOnly? I can select the date from the input in isDisabled = true state as well.
allowSameDay , startDate & endDate should not be there for Date picker.
Can the description of the closeOnScroll be a little more explanatory? (as in... scroll where?)
Default for dateFormat should be mm/dd/yyyy instead of MM/dd/yyyy. Also if we mention it as mm/dd/yyyy the month does not show up in the input field.
Not able to enter the date by typing in the input field. Also not able to delete the selected date without the isClearable control being set to true.
selectedDate does not work.
timeFormat does not work.
All time related fields/controls should not be there for Date Range Picker.
When allowSameDay is set to false, we are still able to select the same day in the Date Range Picker.
Even though it mentions that the yearEndRange default value is current year + 1 but we can only see the current year as the End year in the dropdown.
The time value in max/min Date does not work & similarly the date fields in the max/min Time controls don't work. Can't we have controls that take both max Date & Time & min Date & Time? This is a better UX.
max/min Time don't work.
Clear & Today buttons are missing.

Design:

Wrong height & width of the whole card.
The height & width of the card changes on changing the month.
Wrong gap between month & year dropdown.
Gap between Days (Sun, Mon...) not visible. Should be 4px
Wrong gap between month/year dropdown & Days (Sun, Mon etc.)
Wrong colour for Dates.
The extreme left panel which displays past week, month etc. not visible.
The divider between months does not extend completely to the edges of the card.
Should the system date be shown in orange or what it is right now? (In design it is Orange)
Wrong font weight for month name & year.
Wrong box shadow for Month & year dropdown.

Functional:

Deleting the JSON string in the excludesDates control does not reflect on the canvas when done in the raw mode.  -> Issue with storybook

You can only enter 1 character at a time in the yearStartRange field. The cursor shifts after entering only one digit. Same happens with the Label field. (this happens intermittently)  -> Issue with storybook

The year dropdown always starts from the yearStartRange. This can be frustrating for the user who wants to select a relatively closer date. (suggestion is to show the selected year in the middle or something once the dropdown is clicked.) -> ** Will fix this later. Causing unexpected scroll behaviour. Create a separate bug for this.**

Once we set the yearEndRange control, the year dropdown does not appear on clicking. Can we modify the desc so that it is not confusing for devs? -> Fixed

What is the difference between isDisabled & isReadOnly? I can select the date from the input in isDisabled = true state as well. As of now only focus state

allowSameDay , startDate & endDate should not be there for Date picker. -> Fixed

Can the description of the closeOnScroll be a little more explanatory? (as in... scroll where?) -> Fixed

Default for dateFormat should be mm/dd/yyyy instead of MM/dd/yyyy. Also if we mention it as mm/dd/yyyy the month does not show up in the input field. Fixed

Not able to enter the date by typing in the input field. Also not able to delete the selected date without the isClearable control being set to true -> This is expected

selectedDate does not work. -> Working for me, refresh the page once selected date is added https://design-system-git-ads-v2-date-picker-get-appsmith.vercel.app/?path=/story/design-system-datepicker--date-picker-story&args=selected:1678559400000

timeFormat does not work. Library is behaving differently, can you try date format.

All time related fields/controls should not be there for Date Range Picker. -> Done

When allowSameDay is set to false, we are still able to select the same day in the Date Range Picker. -> Library is not working according to this flag. Can you raise this as a separate issue and assign to me.

Even though it mentions that the yearEndRange default value is current year + 1 but we can only see the current year as the End year in the dropdown. -> Fixed

The time value in max/min Date does not work & similarly the date fields in the max/min Time controls don't work. Can't we have controls that take both max Date & Time & min Date & Time? This is a better UX. -> min max date is working for me. Regarding UI, that's not something we can control. It's part of storybook.https://design-system-git-ads-v2-date-picker-get-appsmith.vercel.app/?path=/story/design-system-datepicker--date-picker-story&args=maxDate:1678818600000;minDate:1678213800000

max/min Time don't work. -> Can you create a ticket and assign it to me

Clear & Today buttons are missing. -> **Clear have a different UI in library, Can you create a separate ticket and assign it to me. **

For Date Picker, we can set maxDate > minDate. Can we prevent the users to do so? Same goes for maxTime & minTime -> This is again something we cannot control in storybook.

startDate & endDate do not work for Date Range Picker. -> Working for me. Just refresh and try after updating. https://design-system-git-ads-v2-date-picker-get-appsmith.vercel.app/?path=/story/design-system-datepicker--date-range-picker-story&args=startDate:1677609000000;endDate:1678213800000

Can we swap the placement of maxDate with minDate & similarly for maxTime & minTime. This is more intuitive. -> Done

Wrong height & width of the whole card. -> I see it is mentioned as auto in design. Also this is taken from content. Can ignore this.

The height & width of the card changes on changing the month. -> Will fine tune later
3,4,5 mention in new bug

Wrong colour for Dates. -> Fixed

The extreme left panel which displays past week, month etc. not visible. -> This was a feature of blueprint, and the new library won't support this.

The divider between months does not extend completely to the edges of the card. -> We can ignore this for now. This was done to make sure it is consistant with other lines

Should the system date be shown in orange or what it is right now? (In design it is Orange) -> Fixed

Wrong font weight for month name & year. -> We are using the button tertiary component here, so it will have button tertiary font-weight.

Wrong box shadow for Month & year dropdown. -> Fixed

We get to see a hover state for disabled dates (dates that cannot be selected due to the user selections). -> Fixed

Issues after the recent fix:

Wrong shade of orange applied to the system date.
Although the default dateFormat mentioned is MM/dd/yyy however I see that in the field dd-MM-yyy is selected. Can we make it uniform?

Other issues:

Can the dates for the month(s) that is not selected be in a lighter shade than #4C5664?

All identified issues are fixed. There is just one thing @albinAppsmith... In the Date Range Picker, when we move our cursor on days that do not have a date mentioned we get to see the hover state & we are also able to select the dates corresponding to those days.
https://www.loom.com/share/465a2563e66449cfb2762a0f9cd8a7da

All identified issues are fixed. There is just one thing @albinAppsmith which doesn't seem to be fixed... In the Date Range Picker, when we move our cursor on days that do not have a date mentioned we get to see the hover state & we are also able to select the dates corresponding to those days. https://www.loom.com/share/465a2563e66449cfb2762a0f9cd8a7da

Will take this up in phase 2, since there is click possible in that empty columns.


Pipeline simplification

Right now there are a lot of abstractions in machine learning we're still not capturing with these ETL pipelines.
The elephant in the room here is pre and post processing with normalization.
We end up having to maintain a lot of metadata like paths of schemas and transform processes.
It's hard to keep track of where you are in an pipeline when you  have so much state to manage there.
Ideally we define something like a "datavec root directory" that we can play with as we like.
This  datavec directory could have a hierarchy like:
preprocess/
postprocess/
In each of these we could store things like schema information and other metadata related to the pipeline.
Goal: make pipeline persistence configurable and as automated as possible.

Already planned to be handled in update/rewrite.


folder_branch_ops: allow a rename over an empty directory

Also, remove support for renaming over files of different types, that
seems silly and isn't supported by the higher FS levels anyway.
Issue: KBFS-1176

lgtm


Revert "remove pyright config for clu"

Reverts Azure/azure-sdk-for-python#31504

API change check
API changes are not detected in this pull request.


Screen Css issue at both mobile and desktop view

Descrcibe the bug?
when you open the game you can see a clear issue in CSS view
Steps to reproduce the bug?
how to solve

add responsive CSS by using tailwind css

Expected behavior
The Expected behavior by making it more responsive user friendly for all view devices like mobile iPad and desktop
Desktop (Please provide your system information)
chrome
Mobile (Please provide your device information)
window
Screenshot / Screenshare

Relevant log output
No response
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue

Code of Conduct

[X] I agree to follow this project's Code of Conduct

thank you i already start working

thank you i already start working

any update?

Hi,
Sorry for the delay,
give me 1 day I will solve this


https://ordinalswallet.com/collection/bitcoin-apes

hey it still doesnt work :/

https://github.com/ordinals-wallet/ordinals-collections/tree/main/collections/bitcoin-apes

what's the problem ?

what's the problem ?

We can't access to https://ordinalswallet.com/collection/bitcoin-apes and the collection doesn't shows up :/


Playback buffering and slow

Any way to improve performance of playback? I can hardly playback more than 5 seconds without buffering.
OSX i7 16GB Memory - running off docker compose.

@stobias123 ,
It will be addressed in the patch: https://github.com/opencv/cvat/pull/787. For now it is under review.

@stobias123
Currently you can also reduce image quality during task creating
In most annotation cases this quality isn't very significant

Please try develop branch. Data streaming feature was integrated.

Hi, I have been using the develop branch but still I feel that there is a lot of laoding/buffering time when I access the server via a proxy.


Classification based on target probability distribution

First of all, I really liked the paper and the idea of making it a live review, thanks for this contribution!
I want to propose another classification for enhanced sampling methods, that would be complementary to the main one adopted by the paper (Fig. 1). The idea is to look at the probability distribution sampled at convergence by the methods, instead of focusing on how they sample it. In this paper we call it target distribution (TD), and is a probability distribution over the whole configuration space (thus has a slightly different meaning than the one in the review).
This target distribution should be easy to sample and fully cover the Boltzmann distribution of interest.
Here is a first draft of how this TD classification might look like:

An enhanced sampling method might support one or more target distributions, and vice versa the same TD can be sampled by more than one method. This classification does not focus on the specific target distribution (this can also be interesting, but there are a lot of them), but rather on the general ways used to define them.
Why is it interesting? Thinking about the target distribution helps comparing methods, and understanding what they can and cannot achieve. For instance, if I choose a temperature-expanded TD to study an entropy-driven transition (e.g. first order phase transition), I will not obtain great results, regardless of how efficient the method I use is. The mistake is in the choice of the target, not in the details of the enhanced sampling algorithm.
This classification can also help navigating the various hybrid methods.
It would be a small addition to the paper, but hopefully useful. If you think it might be of interest, I can elaborate further on the idea. You can also find more details in the paper I linked above.

Hello Michele,
Thanks for the comment! This is certainly an interesting viewpoint that is worth adding to the review.
For the "Forced Marginal Distribution" branch, I can think of methods such as metadynamics, VES, OPES, and other CV-based methods. Do you have some examples of methods that are not CV-based methods? Off the top of my mind, I cannot think of any that are not CV-based.
Also, could you maybe give a few examples of methods for each final branch? That would help us in how to think about this classification.
Best regards,
Omar

@invemichele could you describe a bit more about targeting a non-equilibrium distribution? I'm not sure that's well-defined, as nonequilibrium implies a history-dependent family of distributions.

@valsson yes, most methods in that category are CV-based methods (adaptive bias/forces methods, adiabatic methods). The other example I had in mind are multicanonical methods (e.g. see link) that force a flat potential-energy marginal. There are also other methods, but to my knowledge are only used with Monte Carlo schemes (e.g. nested sampling).
Expanded (or generalized ensembles) TD methods have as target a sum of overlapping distributions. Examples are replica exchange, simulated tempering, integrated tempering, thermodynamic integration, multiple windows umbrella sampling, etc. Some of them run independent simulations and calculate the proper weighting at post processing, other do it on-the-fly.
The main difference in the two categories is how the target distribution is defined, not what specific target distribution is sampled. For example, in the limit of a large number of narrow umbrella windows, one samples a uniform distribution along the CV, and conversely one can define a custom marginal distribution over the energy that leads to the same TD of a parallel tempering run.
There are general characteristics shared by most of the methods in the same category. Methods that define the TD by its marginal, must store an estimate of this marginal (or of the unbiased marginal, or of its derivatives). Thus they deal with histograms, kernel density estimation, basis set expansion, or similar tools. Expanded TD methods generally don't, but must deal with the problem of spacing the intermediate states to have good overlap, and typically use estimators such as MBAR, WHAM, etc.
@mrshirts you are right, I did not pay much attention to the out-of-equilibrium methods. If what they sample is out of equilibrium, then it's not a well-defined distribution. Probably then the classification should distinguish between methods that aim at sampling a target distribution and methods that don't.
(Incidentally, the TD is well-defined but unknown beforehand, which is why nontrivial sampling methods are needed)

Anyway, I am sure the classification I proposed can be improved and probably I am missing or misrepresenting some methods. For example, the final division in uniform and non-uniform is motivated by the fact that most methods sample the uniform type, but maybe it is not very useful. My main point is that focusing on what the different methods sample at convergence can be helpful in navigating the sea of enhanced sampling.

Thank you Michele, we have updated the text with insight drawn from your remarks.


Fix #1382: Handle an arbitrary number of consecutive quotes for string interp.

The CI says
Pull request submitted by null
CLA check for null failed
Please sign the Scala CLA to contribute to Scalameta
Go to https://www.lightbend.com/contribute/cla/scala and then resubmit this pull request

Why am I null? 😥
(FTR I have of course signed the Scala CLA a long time ago.)

Can you please rebase on top of master? I merged #1906 moving the CLA check to GitHub Actions where we have access to an environment variable of the author of created the PR. In Travis we're using some dummy GitHub account token to extract the PR author name.

Done :)


bug fix Issue 138

Sorry if it is hard to read as it is a tool translation.
summary
https://github.com/keonik/prisma-erd-generator/issues/138
This is a fix for a bug reported in issue 138.
If a model UserSetting containing a map existed after a model User not containing a map
User's id field was being overwritten by UserSetting's map name.
There were two issues.

// skip lines before the current model
was including subsequent models in the decision
find matched any field containing "id" because it was a simple character search.

For issue 1, we fixed the problem so that only the current model is filtered.
For problem 2, we added ${field.name} -> ${field.name}  and a space to make it match more strictly.
check
$ npm run test --t issues/138

><EMAIL_ADDRESS>test
> jest --passWithNoTests issues/138

 PASS  __tests__/issues/138.test.ts
  ✓ id not override before key name (2641 ms)

----------|---------|----------|---------|---------|-------------------
File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------|---------|----------|---------|---------|-------------------
All files |       0 |        0 |       0 |       0 |
----------|---------|----------|---------|---------|-------------------
Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        2.978 s, estimated 3 s
Ran all test suites matching /issues\/138/i.

result svg

We would appreciate your review.

@all-contributors please add @dznbk for code


fix: Update provisionScript.zip with small syntax changes

Description
This is a slightly tough PR since the delta is inside a binary...
This changes the provisionComposer.js script used for the 'handoff to azure admin' scenario in the following ways:

correct names of resources in the resulting publish profile if resource group is specified
remove access token field
remove luisResource field if luis not created
add tenantId field

Task Item
Fixes #7539

Coverage remained the same at 50.835% when pulling 1a53ef1d9130f9dbe7553b754c79ad10c9281e74 on benbrown/scriptupdates into 8126634d4205ee877fddf2033f192dd6bdb0f23b on main.

For reference, this is the only change inside the zip file.


feat: To add env variables caching mechanism.

Is your feature request related to a problem? Please describe.
Currently, we deploy applications with environment variables but our "Deploy CLI" doesn't have an env variable caching mechanism.
Describe the solution you'd like
What I think is, we should store a particular application config at the below path:
$  .metacall/deploy/applications/projectName.ini

For Ex:

$  .metacall/deploy/applications/auth-middleware.ini

Now, Why I think it should look like that is because:-

We can store all application data in a single folder and it's nicely separated
Name of the file is like -> projectName.ini -> All the config of those applications having the same name and located in different paths can be accumulated here, one-stop solution

Let's see config file structure
[path.home_raj_desktop_authMiddleware.plan.essential.env]
ENVIROMENT=dev
PORT=5000
HOST=localhost
USER=Creatoon
MAILCHIMP_API_KEY=fssdgdfgfg565756867993ddcbbcbc

[path.home_raj_desktop_authMiddleware.plan.standard.env]
ENVIROMENT=prod
PORT=9000
HOST=https://creatoon.tech
USER=SuperUser
MAILCHIMP_API_KEY=fssdgdfgfg565756867993ddcbbcbc

[path.home_raj_desktop_authMiddleware.plan.premium.env]
ENVIROMENT=prod
PORT=8000
HOST=https://creatoon.tech
USER=SuperUser
MAILCHIMP_API_KEY=fssdgdfgfg565756867993ddcbbcbc

Why this kind of config structure?
It's simply because there may be a case when a user deploys a minimal version of an application on "Essential" Plan with some environment variables & production-grade version on "Standard and Premium" Plans with different values of env variables, that's why I figured out this structure for caching.
So after parsing it would look something like this:
{
  path: {
    home_raj_desktop_authMiddleware : {
      plan : {
        essential : {
          env: {
            ENVIROMENT: "dev",
            PORT: 5000,
            HOST: "localhost",
            USER: "Creatoon",
            MAILCHIMP_API_KEY: "fssdgdfgfg565756867993ddcbbcbc"
          }
        },
        standard : {
          env: {
            ENVIROMENT: "prod",
            PORT: 9000,
            HOST: "https://creatoon.tech",
            USER: "SuperUser",
            MAILCHIMP_API_KEY: "fssdgdfgfg565756867993ddcbbcbc"
          }
        },
        premium : {
          env: {
            ENVIROMENT: "prod",
            PORT: 8000,
            HOST: "https://creatoon.tech",
            USER: "SuperUser",
            MAILCHIMP_API_KEY: "fssdgdfgfg565756867993ddcbbcbc"
          }
        }
      }
    }
  }

}

Additional Context
We should also think about, what if the user re-deploys with some additional environment variables then we should give him an option to add more and update ones that are already cached.
@trgwii  @viferga What do you think sir?
If you think it is incorrect or whatever you think about it, let me discuss it.
Thanks

I think the best option may be caching them in the backend. But this idea can be reused for caching other things like preferences (no secrets). So until we implement this, I am going to leave this issue open.

I would like to work on this Issue if not resolved @Creatoon @viferga

@ashutosh887 At current stage we are not caching prefs on deploy cli side, instead we are caching it on the backend (faas side)


Ensure that Figaro.adapter is set to Figaro::Rails::Application before the figaro Railtie is loaded

Figaro should be able to be used in the initializer of a Rails engine. Unfortunately, when testing the engine using a tool like Combustion, we see that the Railtie to load figaro is added immediately after the definition of Figaro::Rails::Application.
This can (and sometimes does) mean that the before_configuration block of that Railtie is immediately executed. At this point, the Figaro.adapter is still nil, so it is lazily created as a Figaro::Application. The Railtie then calls Figaro.load on it, and it blows up as default_path throws a NotImplementedError.
The solution is to set the adapter to a Figaro::Rails::Application as soon as possible, and before the Railtie is created.
This PR makes that 2 line swap change, and adds Combustion to the specs. If the two lines of logic are not swapped, you can see how the initializing process fails.

@hlascelles - your builds are failing because the Travis config picks up different Bundler gemfiles for different Ruby versions (in figaro/gemfiles) so you'll need to add combustion to each of those.

This change has been extracted from the Figaro repo and added (https://github.com/hlascelles/figjam/pull/7) to a new gem called Figjam: https://github.com/hlascelles/figjam


Community

🌈

You could™ add the semi-standard badge to signal it's usage directly in the readme.md :) https://github.com/Flet/semistandard#badge


[custom images] Custom image creation should work with Python 3

The basestring type is not defined in Python 3 and attempting to create a custom image fails with:

Traceback (most recent call last): File "generate_custom_image.py", line 419, in run() File "generate_custom_image.py", line 360, in run dataproc_base_image = get_dataproc_base_image(args.dataproc_version) File "generate_custom_image.py", line 98, in get_dataproc_base_image if stdout and isinstance(stdout, basestring): NameError: name 'basestring' is not defined

From:
https://stackoverflow.com/questions/50605530/unable-to-create-custom-image-for-dataproc
Alternatives for implementation:
http://python-future.org/compatible_idioms.html#basestring

https://github.com/GoogleCloudPlatform/cloud-dataproc/pull/52


Remove auto height

I've tried to make auto height transitions as easy as I could with little luck so far. It's buggy and it adds a big dependency to this library. It will be removed in the next major release to get the library down to a smaller size.  I think I'm close to a solution with React Fluid Container, but until it's solid, I'm keeping it out of here 👉

React Fluid Container looks pretty decent @souporserious, nice work. Haven't used this lib or react-motion for a long time, but it'll be interesting to see how much things have changed when I get back into it.

It is finished https://github.com/souporserious/react-motion-ui-pack/commit/c395345864cbdc5aa73a7bbd7db5347dde0a1acb for now 😛


Fixes #597 - corrected a spelling mistake in the example

Docs Build status updates of commit 133ad43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureadps-2.0/AzureAD/New-AzureADMSInvitation.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 540af0d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azureadps-2.0-preview/AzureAD/New-AzureADMSInvitation.md
:white_check_mark:Succeeded

azureadps-2.0/AzureAD/New-AzureADMSInvitation.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @CelesteDG , please review this.


Fix TypeError in Vectorstore Redis class methods

Fix TypeError in Vectorstore Redis class methods
This change resolves a TypeError that was raised when invoking the from_texts_return_keys method from the from_texts method in the Redis class. The error was due to the cls argument being passed explicitly, which led to it being provided twice since it's also implicitly passed in class methods.  No relevant tests were added as the issue appeared to be better suited for linters to catch proactively.
Changes:

Removed cls=cls from the call to from_texts_return_keys in the from_texts method.

Related to:
https://github.com/hwchase17/langchain/pull/4653
@dev2049
@hwchase17

thanks @TheCulliganMan!


Update create-field-extraction-rule.md

testing by adding some spaces .... please disregard
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Closing (test)


Expose command to build packages

Currently, there's pep517.check that checks that a package can successfully build sdist and wheels for a package. Similar to that, it should be possible to actually build a package, either the sdist or the wheel or both... something that a CI tool could use to build the package prior to using twine to upload it. This would address pypa/packaging-problems#219.

I'm thinking this:
python -m pep517.build [-s|--source] [-b|--binary] [-o|--out-dir out-dir] <project-dir>

Default is to build both source and binary (wheel) dists and put them in './dist', comparable to what setup.py sdist bdist_wheel does. Passing either -s or -b would produce only that dist.

I have a slight concern over user confusion. There's a discussion in the packaging-problems repo about there being too many tools involved in the packaging ecosystem. This would add another one (and until pip gets a "build" or "sdist" command, it's the only generic way to make a sdist).
It's not enough that I object to the proposal (and your command line syntax looks fine), but I'd see it as a "useful utility" rather than a definitive implementation - at least for now.

I'd lean towards saying the command line interface should be separate,
probably part of pip. Pip wheel is presumably already going to grow pep 517
support.
I'm pretty sure that once you get into creating build environments and
installing dependencies, there's a load of extra complexity that I've
ignored - people will want to get dependencies from different sources, use
pre-release versions of them, control how they're installed, and so on.
We'd end up duplicating lots of options that pip already has.
This reasoning applies to check as well. It's not there by any carefully
considered design - I just wanted a way to do a basic sanity check of all
the pieces, and this was an easy place to put it.
On Wed, 12 Dec 2018, 8:51 p.m. Paul Moore<EMAIL_ADDRESS>wrote:

I have a slight concern over user confusion. There's a discussion in the
packaging-problems repo about there being too many tools involved in the
packaging ecosystem. This would add another one (and until pip gets a
"build" or "sdist" command, it's the only generic way to make a sdist).
It's not enough that I object to the proposal (and your command line
syntax looks fine), but I'd see it as a "useful utility" rather than a
definitive implementation - at least for now.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pep517/issues/25#issuecomment-446739055, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAUA9S8D31yHtwSTvfD2VAmb6YkbRlB3ks5u4Ww_gaJpZM4ZQN4H
.

Yes, one thing that very rapidly becomes an issue with any form of "general packaging library" is that all the myriad of options pip has for controlling what gets installed, network options like proxies, etc etc, all have legitimate reasons for existing, and all tools should (in theory) support them. Look at pip's "build environment" code for how complex it gets ;-)
I'm not sure that counts as an argument for loading all of that functionality into pip. I'd like to think that having a general "install option management" library would be a more reusable option, and I don't think it's healthy for the ecosystem that any tool that isn't pip either has to reimplement huge parts of pip or forever suffer being considered "incomplete". But it's a lot of work to do that, and it would be very easy to end up with a Java-esque "installer option factory builder manager class" style of interface :-)
All of which is a long-winded way of saying that this is why I think any command line interface in the pep517 library should be considered provisional at this point.


Extremely high memory consumption

As seen below, a 500KB apk uses up to 176MB (a 300x increase!!) of memory when decoding just the smali, without resources. This makes large scale analysis of many APKs very hard. I have machine with 60GB of RAM, and even that can only decode about 25 apks in parallel before RAM is 100% used. I'm wondering if something can be done to reduce this. I found a related issue https://github.com/JesusFreke/smali/issues/181 that says this should not be a problem, but that's not my experience. Posting on apktool project in hopes there can be some command line flag (like --low-mem) that trades speed for memory if a user needs that option.
This happens across all APKs I'm working with.
Note: To retest this on Mac, use /usr/bin/time -l apktool.... and note that the output of RSS is supposedly in bytes for OS X.
Information

Apktool Version (apktool -version) - 2.2.1
Operating System (Mac, Linux, Windows) - Linux
APK From? (Playstore, ROM, Other) - Playstore

Steps to Reproduce
[ec2-user@ip-10-0-233-18 test]$ ls -lh
total 556K
drwxrwxr-x 6 ec2-user ec2-user 4.0K Dec 26 19:11 az
-rw-rw-r-- 1 ec2-user ec2-user 551K Dec 26 19:08 az.apk
[ec2-user@ip-10-0-233-18 test]$ /usr/bin/time -v apktool d -f --no-res az.apk
I: Using Apktool 2.2.1 on az.apk
I: Copying raw resources...
I: Baksmaling classes.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
	Command being timed: "apktool d -f --no-res az.apk"
	User time (seconds): 2.84
	System time (seconds): 0.20
	Percent of CPU this job got: 13%
	Elapsed (wall clock) time (h:mm:ss or m:ss): 0:22.37
	Average shared text size (kbytes): 0
	Average unshared data size (kbytes): 0
	Average stack size (kbytes): 0
	Average total size (kbytes): 0
	Maximum resident set size (kbytes): 176136
	Average resident set size (kbytes): 0
	Major (requiring I/O) page faults: 0
	Minor (reclaiming a frame) page faults: 47144
	Voluntary context switches: 2273
	Involuntary context switches: 19
	Swaps: 0
	File system inputs: 0
	File system outputs: 15728
	Socket messages sent: 0
	Socket messages received: 0
	Signals delivered: 0
	Page size (bytes): 4096
	Exit status: 0

Perhaps someone can run apktool inside a java profiler and see where these allocations are coming from

You have to remember that the source APK is not the only package being loaded/utilized. The framework file (as of API 24) is about a 2.7M apk and that is loaded on every decode.
I did find a problem where we are reloading the same framework APK twice, which is highly inefficient.
The next major slowdown point comes from unknown files as it creates a new zip (apk) on the fly as it injects those unknown files. I'll have to look into editing the same file vs a tmp file.
Finally, a lot of string manipulation is hurting the runtime, according to the profiler.
I do have lots and lots of crashing bugs to fix though, so optimization has unfortunately fallen behind. I'll do my best for the easy solutions, but any fix requiring a good deal of rewriting will probably have to wait.


Fixing add_fixed_in_frame_mobjects

class Text3D(ThreeDScene):
    def construct(self):
        axes = ThreeDAxes()
        self.set_camera_orientation(phi=75 * DEGREES,theta=-45*DEGREES)
        text=TextMobject("This is a 3D text")
        self.add_fixed_in_frame_mobjects(text.to_corner(UL))
        self.add(axes)
        self.add(text)
        self.wait()

This script is currently broken because add_fixed_in_frame_mobjects throws an error.
It was broken before but fixed by https://github.com/ManimCommunity/manim/pull/281.
Now, with git bisect I could boil down the first occurrence of this error to this pr:
https://github.com/ManimCommunity/manim/pull/532 (ping to @eulertour)
Before that, the Text3D Scene was functioning properly.
It would be nice when we could make this work again :)
As soon as we have it fixed, I can volunteer to add a test for this.

Yeah, I was getting the error:
AttributeError: 'ThreeDCamera' object has no attribute 'extract_mobject_family_members'
and found in #532 there's a change to camera.py where extract_mobject_family_members was moved:
https://github.com/ManimCommunity/manim/pull/532/files#diff-7732083ebe3bdfdbb948a293a8ff761e5c4a7b8ed98e1daed6838076a4781234L366

Fixed via #674.


intercept https://soundcloud.com links (Custom url scheme)

It would be neat if soundcleod could pick up links in apps like messages/slack/etc pointing to soundcloud. I believe apple supports this via this feature but I can't speak for support in Electron.

Never mind. I thought for some reason it would work with https schemes as well since other sites "does it", but that seems to be an exception.


Extra spacing between characters while generating pdf on RHEL

Hi Team,
We are facing issue regarding extra spacing between characters while generating pdf in redhat. When the pdf is generated through MAC/Windows, Every character looks as it is displayed in html. We have already ran the command "yum install fontconfig freetype libfreetype.so.6 libfontconfig.so.1 libstdc++.so.6".
Please suggest whats wrong with the setup.
Attached is the pdf which was generated.
Thanks.
HealthReport-1.pdf

Hi,
I didn't encountered such issue when using phantomjs on RHEL, but you can try your luck on phantomjs repository to which is the problem related.
Maybe you can try docker if you move nowhere.
Sorry, I'm not able to give better help.

This is rather issue with phantomjs installation than this repository. Closing.


Cant copy code to clipboard

Hi,
I cant seem to get the "copy code to clipboard" function to work. Is it supposed to be a button in the corner for code blocks? I tried using:
[params.page.code]
# whether to show the copy button of the code block
copy = true
in the config.toml, but it dident do anything. I also browsed the demo web app, but couldn't find functionality "copy to clipboard". Is this working as intended?

I recently replaced the library that is responsible for highlighting the code.
In the new version of the documentation, everything will be described


Warn user if the phone is in the "silent" mode

It's possible to enable quiet mode on the phone, then no alarm sound will be heard. User should be acknowledged about it somehow in the UI

I don't know. I think the most operating systems showing a warn icon anyway.
There are several methods to silent your phone like "do not disturb" features, lower volume, silent audio profiles etc.
I'm not an app developer, but it could possibly be a struggle to check for all of this different methods to just show another warning inside of the app...
Probably it could be a way to show a static message for this case, like this:

Heyy,I would like to take up this issue


To use RP2040-Zero instead of ProMicro...

Hello, good fellow.
Can I borrow your attention one more time?

Thanks in advance.
20230908.DURKEN1.PIERCE-RECEVIER.00.pdf

Hello.
It can be done in this way.
Need some help here.

Hello! Sorry for the late reply.
Did you get any further with this? I would assume the voltage regulator is not needed since the rp2040 runs on 3.3 V and the zero already has a regulator.

Hello.
Thanks for your answer.
I made the same assumption, but was not sure enough.

Can't stop this. :-)

20231016.PIERCE-RECEIVER-COMBO.00.zip

你好，这太棒了，请问有代码吗


Ability to generate custom IDs

Thanks for making this port! The Python version of genanki has the ability to override the ID computation, and I think that would be a useful feature for this library as well. The hash of the fields is not sufficient as a note ID if one ever wants to correct a mistake on a card.

Sorry for letting you wait so long. Great suggestion, I will take a look!

I think this issue could be closed? It's possible to use Note::guid()


Floralis fails to load on server after 3.1.0

Hi, I'm unable to run Floralis on a 1.18.2 Forge server after the 3.1.0 update. I'm using Forge 40.1.31. Attached is the server log which shows the error: https://paste.feed-the-beast.com/view/8ae4dd81. I believe that could be caused by mixins:
[03:19:08] [main/FATAL] [mixin/]: Mixin apply failed floralis.mixins.json:FireBlockAccess -> net.minecraft.world.level.block.FireBlock: org.spongepowered.asm.mixin.gen.throwables.InvalidAccessorException No candidates were found matching.
The modlist is only composed of your mod:

Thanks in advance for checking it out and have a good day!

Thanks for the feedback! I tested both client and server as usual within my IDE before releasing the 3.1.0 and it both worked just fine there. Apparently Forge slightly altered the way mixins are read with the latest releases for 1.18.2. A simple addition should've fixed that now.
I also tested it out of my IDE this time and both client and server start and work just fine, still I'd like you to test it in your environment too and let me know if it works for you too, before I make a final public release on CurseForge. You can find the latest build with the fix here.

Hi, thanks for the reply! I've just tested this in a modpack amongst other mods, and it works great! No crash at all on 1.18.2, Forge 40.1.31. You can go ahead and release this to the public, this is fixed. Thanks a lot!


Use Component.Verson instead of Properties["AppVersion"] in documentation

@alancameronwills, some people got confused by example on the page:
https://azure.microsoft.com/en-us/documentation/articles/app-insights-api-custom-events-metrics/
Specifically instead of setting Properties["AppVersion"] it is better to set Component.Version as this property is used in UI as "Application Version".

OK, thanks!

Alan updated the file.


A JavaScript error occured in the main process

After authenticating with Google on the pre-built Mac OS X binaries:
Uncaught Exception:
SyntaxError: Unexpected token }
at Object.parse (native)
at loadFromFile (/Applications/Yakyak.app/Contents/Resources/app/node_modules/hangupsjs/node_modules/tough-cookie-filestore/lib/filestore.js:148:32)
at new FileCookieStore (/Applications/Yakyak.app/Contents/Resources/app/node_modules/hangupsjs/node_modules/tough-cookie-filestore/lib/filestore.js:14:5)
at Client.module.exports.Client.doInit (/Applications/Yakyak.app/Contents/Resources/app/node_modules/hangupsjs/lib/client.js:183:49)
at new Client (/Applications/Yakyak.app/Contents/Resources/app/node_modules/hangupsjs/lib/client.js:102:12)
at Object. (/Applications/Yakyak.app/Contents/Resources/app/main.js:33:12)
at Object. (/Applications/Yakyak.app/Contents/Resources/app/main.js:350:4)
at Module._compile (module.js:428:26)
at Object.Module._extensions..js (module.js:446:10)
at Module.load (module.js:353:32)

I've seen this too.

I'm seeing the same issue on Windows 7 64bit

Same on Win10. After dismissing the error two yakyak.exe processes remain hanging and need to be killed.

can anyone describe the procedure to reproduce this error?

For me all I do is right click on yakyak.exe and select Open.  during startup I get the error message and I'm left with two processes running, but no UI.

seems like a syntax error in some node module :\
does this happen with latest version v1.3.3?
does it still happen if you remove/rename the yakyak configuration folder in the path below (this is where all YakYak definitions are stored, so you'll need to re-authenticate):

@rbaert can you try 1.4.0-beta3
see #451 for the download link

closed due to lack of feedback
please try 1.4.0 before reopening this issue

This happened to me on Windows 10 x64, 1703 using YakYak 1.4.3 - same exact symptoms.

It seems like a problem from a library yakyak uses, @davibe what do you think?
@hypercube33 does it happen often? how do you solve it?

This started happening to me on Windows 10 1703 (15063.608) a day or two ago. Wiping the install folder and unpacking the latest beta results in identical behaviour.
Steps to reproduce:

Try to run YakYak.exe (1.4.4 x64)

Expected behaviour: YakYak opens
Actual behaviour: An error popup with "A JavaScript error occurred in the main process. Uncaught exception: SyntaxError: Unexpected token"

A workaround to delete %APPDATA%\yakyak\Cookies* and then you can re-login and yakyak will start.
These are the 3 files to delete.
LastWriteTime           Length Name
-------------           ------ ----
10/11/2017   2:56 PM     11264 Cookies
10/11/2017   2:56 PM         0 Cookies-journal
13/11/2017   4:54 PM      4094 cookies.json

Yes, cookies.json is all in lowercase, the others are in Sentence Case. I didn't touch any other files.
Windows 10.
Yakyak 1.4.3.
If i compare a working yakyak vs a non-working yakyak.
I see that a working yakyak has XML in cookies.json and is 4159 bytes
I see the broken yakyak has cookies.json filled with NULs and is 4094 bytes.


dracut-overlayfs improvement to be general.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

What does the PR accomplish, why was it needed?
General improvements to make the features more generic. Dracut module on AzureLinux repo should be usable by users other than Trident.
The corresponding changes for MIC API will be kicked off shortly.
Change Log

Change1: Provide overlayfs mount -o option so that ro / rw can be set. By default the overlay should be mount rw for generic usage.
Change2: No upperdir / workdir needed for volatile overlays.
Change3: No upperdir / workdir needed for ro overlays.
Change4: Verify Overlay non-volatile partition exists.

Does this affect the toolchain?

NO
Associated issues

Internal story tracking: https://dev.azure.com/mariner-org/ECF/_sprints/taskboard/Mariner Operator Edge/ECF/2408?workitem=7969

Test Methodology

Pipeline build id: buddybuild: TODO.

Close this PR since we will focus on MIC Overlays API change, while the dracut-overlays will be kind of dedicated to Trident's requirement.


Fix problem with shadows misrepresenting the unit position

in SC2 it's done the same way I do - it needs to be checked what's different - maybe it's just a matter of camera projection

I've checked again and I'm 99% sure it's about the camera. At this point, I think it would be nice to have an optional, perspective camera as an alternative to an orthogonal one.


Header component added

Fixes #23

@aka-jain @spyshiv please review

Fix travis check @anweshknayak

@anweshknayak please attach the screenshot of your final result.

@aka-jain changing to    made the UI behave weirdly. And also using the mixin for box-shadow made it look weird. So I have left those changes as of now. Rest done as you said. Also some minor changes in tsconfig.app.json to incorporate use of require keyword

I don't get why travis build is failing

@anweshknayak Since you're using app-header inside home.component.html, try to import HeaderComponent in home.component.spec.ts as well. That might pass travis.

@anweshknayak Check the travis log.
These commands are failing
ng test --watch=false and ng build --prod
Please fix that. Let me know in case of any issue

also make logo public for ng build --prod to pass

ng build --env=prod
this worked but didn't show anything wrong in the log. @spyshiv @xamfy @harshit98

@anweshknayak  see https://stackoverflow.com/questions/43141576/property-x-is-private-and-only-accessible-within-class-xyzcomponent

pulled your changes and it shows the error while executing ng build --prod on my pc

for ng test --watch=false to pass
add headercompent in declarations in both app.component.spec.ts and home.component.spec.ts

@anweshknayak refer https://stackoverflow.com/questions/44504468/angular-2-karma-test-component-name-is-not-a-known-element

add headercomponent that you've imported in home.component.spec.ts in declarations, similar to app.component.spec.ts, test it on your machine first

thanks a lot @xamfy

@aka-jain @spyshiv done the changes as requested.

tests for chromedriver is failing. @xamfy is it related to #43 ?

@anweshknayak let me see PR #43 till then ignore Travis error

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@eeaccce). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage          ?   72.34%
=========================================
  Files             ?        6
  Lines             ?       47
  Branches          ?        1
=========================================
  Hits              ?       34
  Misses            ?       13
  Partials          ?        0

Impacted Files
Coverage Δ

src/app/home/home.component.ts
100% <ø> (ø)

src/app/components/header/header.component.ts
100% <100%> (ø)

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 eeaccce...4a24b57. Read the comment docs.

@spyshiv tests are passing.Please check

Implementation is not correct. Closing this PR as will cover it again. cc @deshraj


rpad function causes OOM

Describe the bug
When I use the rpad function, and the parameter of the rpad function is large, it causes OOM.
Does Datafusion have a mechanism to limit the resources required for execution.

Additional context
Add any other context about the problem here.

We probably have to put some limits. for example postgres fails fast
select rpad('a',<PHONE_NUMBER>, 'a');

requested length too large

@ZuoTiJia is such kind of rpad is needed by your real use case or you testing DF limits? :)

Postgres handles this in very interesting manner
https://github.com/postgres/postgres/blob/d952373a987bad331c0e499463159dd142ced1ef/src/backend/utils/adt/oracle_compat.c#L282

We probably have to put some limits. for example postgres fails fast
select rpad('a',<PHONE_NUMBER>, 'a');

requested length too large

My Mac is spinning CPU like crazy with the same query in datafusion-cli, I had to kill the process
@ZuoTiJia is such kind of rpad is needed by your real use case or you testing DF limits? :)

I'm testing DF and I think it's dangerous for users if there is no limit.

sounds good, I think we can implement quick win like in PG
https://github.com/postgres/postgres/blob/c8e1ba736b2b9e8c98d37a5b77c4ed31baf94147/src/include/utils/memutils.h#L42
I'll take this later this week unless no one else volunteers

It looks like the check is implemented https://github.com/apache/datafusion/blob/main/datafusion/functions/src/unicode/rpad.rs#L131
However, when I tried to run the above sql query, I got offset overflow error from /arrow-array-52.2.0/src/array/byte_array.rs:210:45.
Is that an OOM issue?


poc: Ensure Cloud Manager never polls for events in the past

Description 📝
Changes events polling logic to ensure that the created date in the polling X-Filter  ({ "created": {"+gte": date } }) is never in the past 📆

[!warning]
I have not done any testing on this yet. We'll want to make sure we really think this through. There could be some negative sideeffets of making this change that I haven't thought of.

How to test 🧪

Test on an account affected by the bug where recent events are have incorrect create times (in the past)
Ensure polling still works as expected on various accounts
Ensure polling still work on an account with 0 events

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Test on an account affected by the bug

Which bug is this referring to?

Updated the description to clarify @hkhalil-akamai

Closing because this mitigation is no longer needed


Solution for Hi-DPI feature of jetbrains IDE make UI too big running VcXsrv

Environment
Windows build number: Microsoft Windows [version 10.0.19041.450]
Your Distribution version: Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce
Install VcXsrv on your windows host, install phpstorm and set this in ~/.bashrc
export DISPLAY=$(cat /etc/resolv.conf | grep nameserver | awk '{print $2; exit;}'):0.0
export GDK_SCALE=2

Then launch phpstorm.
Actual behavior
Here on my 1920*1080 3 screens (IRL), is the VcXsrv windows containing phpstorm. The font is quite normal considered how big the UI is.

How I fixed (workaround)
this link explains that the "auto-detect dpi" feature may break some display. I added -Dsun.java2d.uiScale.enabled=false to my JVM settings and it fix the problem. However, I have the same problem if I launch gitkraken.
How I should have fixed (workaround)
export DISPLAY=$(cat /etc/resolv.conf | grep nameserver | awk '{print $2; exit;}'):0.0
export GDK_SCALE=1 #I don't know why but 2 was bad, seem like this env var isn't even needed

As gui apps aren't officially supported yet, doubt they'll be willing to address it.

This answer may help https://superuser.com/a/1370548/726810

As gui apps aren't officially supported yet, doubt they'll be willing to address it.

It isn't that X11/GTK apps aren't (super scary quote) "supported" per se, but that the submission is squarely tag linux-behavior (plus VcXsrv behavior). There is no (obligatory pronoun) it bug to address in WSL. WSL will dutifully send the X11 protocol down tcpv4 port 6000 that the kernel is told to send.


Add Python linting and formatting checks

Minor enhancement
Use ruff for Python linting and formatting

Reformat code based on local findings from ruff
Add GitHub Action job for ruff

@hellt
Markdown linting will be covered in a different PR once I have that ready.
Thank you!


Update policy-csp-localpoliciessecurityoptions.md

issue #2189

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@MaratMussabekov - please merge the commit on the copy/edit review made by JohanFreelancer9 / nenonix  or make the necessary changes on the PR following the suggestions made. Thank you.

Dear @JohanFreelancer9 ,
your recommendations were successfully applied
Thank you!


Sound detection

Is there any way you can expose the new “sound detection” triggers to HA? I’m looking to be able to do things like turn on my bathroom fan when the humidity is above a threshold, but only if Alexa hears water running, and things like that.

Is it available in the Amazon Alexa app?

Sort of. You can use those sounds as triggers for routines. Not sure if there’s something you could expose or if I have to get creative with a binary sensor from HA and an alexa routine

And this is not a dupe of 1741. My request is Alexa can listen for certain sounds like water running or babies crying. The other uses the ultrasonic sensor to detect presence.

Similarly the Echo Show devices can detect motion using their cameras.  I’m not sure if this is stored the same way as the ultrasonic motion detection. This would be a fantastic feature to add to Home Assistant.
Also, they have added this ultrasonic motion detection to the Gen 4 Echo Dots. Great for triggering lights when you enter a room, etc.

Is this something you think would be possible to add or would I need to create an Alexa routine with those triggers and have them turn on binary sensors/buttons in HA?

No idea. If you check the websocket traffic, does it provide access to the info? While it's exposed for Alexa routines, if we can't access it in real time then we can't.

This would be interesting indeed, i believe Alexa calls it occupancy detection.

Having the occupancy detection available as a sensor would be awesome. Echo dots have a pretty good ultrasonic occupancy sensor.

i believe Alexa calls it occupancy detection.

No, this is an entirely different feature from the ultrasonic occupancy detection. It's an AI feature that listens for specific sounds, like water running, baby crying or snoring. You can use it as a trigger for routines, and trigger HA when it happens.

Which echo's have ultrasonic detection ?
But yeah, agreed would be nice to have sound or occupance detection.

From: Jon Silver @.>
Sent: 20 January 2023 07:53
To: custom-components/alexa_media_player @.>
Cc: atv2016 @.>; Comment @.>
Subject: Re: [custom-components/alexa_media_player] Sound detection (Issue #1736)
i believe Alexa calls it occupancy detection.
No, this is an entirely different feature from the ultrasonic occupancy detection. It's an AI feature that listens for specific sounds, like water running, baby crying or snoring. You can use it as a trigger for routines, and trigger HA when it happens.
—
Reply to this email directly, view it on GitHubhttps://github.com/custom-components/alexa_media_player/issues/1736#issuecomment-1398028756, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEBCFU5FMKNYR3OIU5UZIKLWTJAABANCNFSM6AAAAAAQVKSRSI.
You are receiving this because you commented.Message ID: @.***>

Documentation: https://www.amazon.com/gp/help/customer/display.html?nodeId=GSR22RYDWS3KBUYW
This is what look like from the app:

-> Is this information exposed via websocket traffic?

No it’s not. I am talking about sound detection, not ultrasonic.

Any progress on this?  I'm also having troubles with ultrasonic detection. The logs show many occurences of the following:
2023-02-05 08:43:29.205 DEBUG (MainThread) [homeassistant.helpers.ratelimit] Reached rate limit of 0:01:00 for Template("{{ is_state('switch.dining_room_dot_ultrasound_motion', 'on', 'off') }}") and deferred action until 2023-02-05 13:44:28.081826+00:00
2023-02-05 08:43:29.205 DEBUG (MainThread) [homeassistant.helpers.ratelimit] Reached rate limit of 0:01:00 for Template("{{ is_state('switch.bath_dot_ultrasound_motion', 'on', 'off') }}") and deferred action until 2023-02-05 13:44:28.100051+00:00

Open a new ticket for ultrasonic detection. This issue has nothing at all to do with that. Stop posting ultrasonic stuff in this issue please.

Sorry, I thought the root cause of the volume of data causing this error might be related. I will open it as a separate issue.

From: broyuken @.>
Sent: Sunday, February 5, 2023 10:51:16 AM
To: custom-components/alexa_media_player @.>
Cc: neil1111 @.>; Comment @.>
Subject: Re: [custom-components/alexa_media_player] Sound detection (Issue #1736)
Open a new ticket for ultrasonic detection. This issue has nothing at all to do with that. Stop posting ultrasonic stuff in this issue please.
—
Reply to this email directly, view it on GitHubhttps://github.com/custom-components/alexa_media_player/issues/1736#issuecomment-1418071687, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AG4HWX4HTUPAJVRZXZ2W2XDWV7D7JANCNFSM6AAAAAAQVKSRSI.
You are receiving this because you commented.Message ID: @.***>


Add new book: Guide to Prompt Engineering

What does this PR do?
Add resource
For resources
Description
Guide to Prompt Engineering by CodeAhoy
Why is this valuable (or not)?
It is an excellent resource for beginners interested in learning prompt engineering because it covers practically every part of the topic from beginning to end.
How do we know it's really free?
According to the author, the book is free.
For book lists, is it a book? For course lists, is it a course? etc.
It is a book.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

seems like the subject is not python?

Python notebooks are included in Chapters 2, 4, 5, and 6.

The book has been added to the BY SUBJECT file.

It's usually not helpful to add a category for a single resource. Try to find another category, or see if there are other books in our lists that would fit into your category. I would put this in the Artificial Intelligence category.
when you've made changes, please add (changes made) to the title of your PR

this is ok for now, but I think we'll need a propt engineering category under AI.

Ok


derive Switch (wip)

closes https://github.com/yewstack/yew_router/issues/80
Allows deriving a macro-like structure on enums that allows resolving one of the enum's variants from a route.

At the moment, it doesn't support checking if the enum's field names are the same as the names in the captured groups.
It also will try to resolve each enum variant in order, and will short circuit when it finds a match. So more general enums should be placed below more specific ones, if it is possible for both of them to match the same string. It may make sense to return a runtime error if two or more matches occur, or perform compile-time checks on all of the matchers to see if they have potential overlaps - leading to unreachable variants.
There isn't a special case for handling the state field of the RouteInfo. If an Swtiched enum wants to get that, the trait needs to be implemented manually.

None of these will be covered by this PR, and follow up PRs should be conducted to address them individually.

@Boscop I think this meets enough of the initial requirements of your original PR to merge. Let me know what you think.

Thanks! I'll take a proper look tomorrow when I have more time..
Btw, what's the current way to match on a route tail with this?
Would #[to = "/foo/{tail}"] (with tail: String) match /foo but also any sub-route?
Btw, it might make sense to also take a look at purescript-routing for inspiration.

Currently the syntax would be: #[to = "/foo/{*:tail}"].
The routing syntax is still a work in progress, but I think its well suited to what it needs to express. The only concern is if it is worth being different than other routing syntaxes - is what is gained by using this approach worth alienating people used to react-router's or in your case, purescript's routing solution.
My critique of react-router's syntax is that it is rather simple - you have to match everything between two /s, and I don't know how to get the head or tail of a route (that is to say, I believe that those functions might exist, I just can't think of them right now).
While from what I can discern from quickly looking at purescript's solution, is that it appears to rely on language-specific combinators that would be difficult to replicate elsewhere.

I threw up a quick proposal based on purescript routing here: https://github.com/yewstack/yew_router/issues/86
I'm not getting to it any time soon, but that could someday find its way into this crate.


General feature-parity with purescript-css

Remaining todo items

[x] Transitions
[x] List style
[x] Font face rules
[ ] Import rules
[x] Elements
[x] z-index
[x] outline
[x] visibility

I don't think I'm going to bother with @import for now, as it works differently than other at-rules:

Must be written at the top of the CSS file.
Cannot be nested.


Folder naming & widget naming doesn't work well with 2 word names

ex: create project_details creates correct folder and file names, but the widgets are named like Project_detailsEvent
and vice versa, create ProjectDetails names the folder and files incorrectly

This has been updated in v0.0.8


Fix version number

This one slipped through with #1050

Going to merge this one already as it's just a version number correction.


Update new submission in repo

The last accepted submission should be committed to the repo.
Found your extension on Reddit btw. Cheers :)

it is committed. Try reloading and submitting.
Git commit won't show you new changes if you haven't changed the code of your latest submission.
lmk if it doesn't work for you.
thanks for using LeetHub!
closing for now.


2-3 errors (from recently added features)

I'm getting the error:
Uncaught TypeError: Cannot read property 'className' of null

In the last line of this block:
SimpleMDE.prototype.createSidebyside = function() {
	var cm = this.codemirror;
	var wrapper = cm.getWrapperElement();
	var preview = wrapper.nextSibling;

	if(!/editor-preview-side/.test(preview.className)) {

I have the old/normal preview enabled, and when I click it, I get this error:
Uncaught TypeError: Cannot read property 'singleLineBreaks' of undefined

(I don't have singleLineBreaks defined in the options when I initialize the object.)
Possible causes, or steps to reproduce:

I don't have the side-by-side enabled in the toolbar buttons.
The textarea I'm targeting does not have any siblings.
I'm getting SME from cdn.jsdelivr.net

I'm not sure if it's related, or a separate problem, but I have a change event defined that used to work, but is no longer firing (apart from that and preview, everything else seems to work fine.)

Same problem here

I'm not able to reproduce these errors, can you create a JSFiddle that does? Ideally, it would exemplify all three bugs.


[FLINK-13742][table-planner-blink] Fix code generation when aggregation contains both distinct aggregate with and without filter.

What is the purpose of the change
fix a bug of distinct aggregation code generation when there exists distinct aggregations on same column, and some of them have filter condition.
Verifying this change
add new IT case in 'AggregateITCase' to cover the change.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no

Travis passed in my own travis: https://travis-ci.org/wuchong/flink/builds/573365202


Directional graphs

Metallic bonds and covalence bonds require undirected graphs because the electrons are shared between atoms and there are no exclusive owners.
However, ionic bonds are created between atom pairs where there is a clear donor and a clear receiver of the electron. Therefore, we can inject physics information in the adjacency matrix by transforming the graph from undirected (as the current implementation performs) to directed.
Currently, the undirected graph relies on a routines inside the "GCNN/data_utils/helper_functions.py" file:

remove_collinear_candidate

This function makes sure that if A is neighbour of B, then B is mutually neighbour of A. This routines somewhat guarantees that the connectivity between atoms is local (the adjacency os only locally dense, not globally dense), avoiding the connectivity between atoms to "explode" and transform the adjacency matrix into something that is globally dense.
I think that for the directional graph we can avoid calling this function, because A can be neighbor of B without having B being neighbor of A.

Closing: HydraGNN


new notifications type: SLA breach combined (per product)

Description
This commit introduces a new type of notifications: SLA breach combined.
The main difference is that notification is produced per product. Original SLA breach notifications are generated for each applicable findings. This may result in hundreds of messages (e-mail, slack or teams messages) for large products. Such alerts are hardly manageable and in the end not of much use.
With SLA breach combined notifications being enabled a user receives a message per product with a list of findings which breach their SLA. It can be summarized in the following manner:
subject: <SLA breach kind> <product type> <product>

body: <product summary> <list of findings>

Notes
I understand that this pull request is not acceptable as it is.
Could you please comment if the approach and motivation is acceptable? For instance, may be it is possible to avoid SLA breach notification per finding completely and make all of them "combined"?
I can work on further improvements, however, internally, we will use this feature in its current implementation, potentially, with minor updates.

In general, I do not see any critical part where I wouldn’t agree with implementation.
But there are failing unittests + flake8. Can you look at it, please?
Plus, I know that it is a bit tricky to write unittest for notifications. Can you at least provide some example screenshots, of how this notification email can look like?
Good job. If PR is accepted, I might add and test the template for Slack.

But there are failing unittests + flake8. Can you look at it, please?

I'll check.

Can you at least provide some example screenshots, of how this notification email can look like?

Sure, here is the example of a notification e-mail:

Here is the updated notification e-mail with <ul> + <li> for lists, URL to findings without IDs, just titles:

I've also updated utils.py to make Flake test happy.
However, I don't get why test-rest-framework fails with "Unknown column 'sla_breach_combined' in 'field list'" error in unittests.test_migrations.TestOptiEndpointStatus.test_after_migration. The migration is applied, so it should be fine...

I have no idea why unittests.test_migrations.TestOptiEndpointStatus.test_after_migration is breaking your tests (and it is not breaking anything else).
But if it is the only issue, after accepting this PR https://github.com/DefectDojo/django-DefectDojo/pull/8666, it should work for you (if there is not any other issue).

I have no idea why unittests.test_migrations.TestOptiEndpointStatus.test_after_migration is breaking your tests (and it is not breaking anything else). But if it is the only issue, after accepting this PR #8666, it should work for you (if there is not any other issue).

Thanks. :)
NB: I've accidentally clicked on "request review", sorry. Really vague interface in this regard...

@pna-nca, can you try to rebase from the new dev? It looks like there was some different issue.

@pna-nca, can you try to rebase from the new dev? It looks like there was some different issue.

done

@pna-nca Are you able to fix the failing tests so this can be merged?

@pna-nca Are you able to fix the failing tests so this can be merged?

Hi @mtesauro, done.

This PR just broke dev branch. There are two 0195 db migrations. Needs to be fixed ASAP.

@kiblik Understood - saw #9312 will approve as soon as tests pass


Parsing command-line arguments concerns

I have a few concerns regarding Parsing C++ command-line arguments, specifically because these rules are used to parse .NET command line arguments as well, that I would like to raise.
I don't know if this is truly the proper place to raise this, but I was directed to this location to review these rules.

Regular, non-programmer, users do not think in terms of programming and therefore would not think of the "" character as an escape character, and as such would not attempt to apply these rules by default.  Typically the "" is used in a Microsoft Windows style path, and a double quote mark is not valid path characters in Windows.  Most would think that MyApp.exe "C:\MyFolder\" is completely valid.
These rules are inconsistent in the fact that sometimes "" is an escape character, specifically only when dealing with the double quote mark.
Other executables created by Microsoft that are originally older than this rules do not follow it.

I think it would make more sense to have this be controlled by the developer instead of being enforced on us.  The default behaviour could be to follow this rules, but allow the developer to opt out of it, perhaps by settings some flag in the parsing routine.
Note that I did raise this as a bug in .NET, but they closed it as "not a bug" and said that they were correct because they were following these rules.
Yes, the user could specify the path without the ending "", but several tools that provide paths to use automatically end paths with a "", such as the "Macros" in Visual Studio's Pre and Post-build Event Command Line.  As you can see below, several Macro values end in a "" and many contain spaces which makes the use of double quotes required.  Just essentially arbitrarily appending an extra "" to those seems wrong and is too easy to forget to do.

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

ID: e0c5ec8f-dc90-d0bd-5aa5-1340ccc877c3
Version Independent ID: 19646aca-3da3-14e8-ea9a-fcb3020cf51c
Content: main function and command-line arguments (C++)
Content Source: docs/cpp/main-function-command-line-args.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

Note that if there is a better place to submit this, please let me know.
Thank you

@mdell-seradex
I'm sorry to say there probably isn't a good place to make your suggestion. I'm closing your issue because it's not a documentation issue. If you'd like to raise a product issue for the C++ team, you can do that on Developer Community. I'm not aware of a good public forum to communicate with the Windows development team.
The command-line parsing behavior is based on a combination of how Windows' cmd.exe parsing works with how the C runtime startup code works. It's worked this way for more than 25 years, at least since Windows 95 introduced long paths. Any change would break untold numbers of apps, so making any change is an uphill battle.
Not all apps use the C runtime startup code, so they're free to use different behavior with the command-line string that Windows provides when their process is started. You can customize the C runtime startup argument parsing behavior yourself, so you're not limited to the standard command-line interpretation. If you link the noarg.obj file to your app, that turns off the default argument processing. You can get the raw command-line string directly by using the Windows GetCommandLine function. This is the same call the C runtime startup code uses to get the command line.
If you're using cmd.exe to launch your app, you should be aware that the command line may have already gone through one layer (or in the case of a batch file, two layers) of escape interpretation. Windows also has a function CommandLineToArgvW that acts just like the C runtime argument processor.

@corob-msft Thank you for taking your time to review and respond.
I am honestly extremely surprised that ANY apps really on these (IMHO) janky rules.  It just seems like they would cause more headaches instead of helping.  You can see from my example how one app, albeit run inside a command window, does not follow these rules, and that app has existed much longer than these rules have.  I would think that escaping a " by specifying it as "" as some other things do, would be simpler and make more sense.  It would also be consistent behaviour for the character.  Likely the best way would be to have some way to specify an escape character, or how quotes should be altered to be treated as a literal character.
Note that I was essentially directed here from that area (the Developer Community), however it was from the .NET section.
They did not even try to suggest redirecting the request to the C++ group/section.
In the end, I did write my own command line parser, but I thought I would try raising my concerns with another group.
At least, if this does not get purged, this provides something that could allow others to become aware of, and to consider what makes more sense to them.
Again, thank you.