Running the project

How do we run this project on localhost?

Like any other NodeJS project:

npm install
nom start

I get the following errors

As the highlighted text say, you are missing the .env file :)
You can install env-auto globally and then run it in the project folder. It should generate a .env file for you.
Or you can check the app.json file and see what environment variables are needed, and make the file by hand.
If you want to learn more you can check out the foreman module.

Still I get that error. Cant I just specify a port and access directly?

I am sorry for all the issues, I am a bit rusty with nodejs as I am a newbie

OK, got it, this project is meant to be run under Linux and you are using Windows :) I'm using a Bash script to make a decision how to run the server.
Sadly I have no idea how to run a NodeJS project on Windows. I would recommend using a Linux box.

Is there a way I can just specify it uses a port with express and then just run it directly, removing all the usage of environmental variables

You should be able to do this:
PORT=3000 node bin/www

aa 💩I'm thinking Linux again :D I have no idea if you can do PORT=3000 something something :(
Also you should be able to just type node bin/www and node will use a default port from the www file.


Topic as implicit method invocant under-documented

When a method call is used without an explicit invocant, it's called on the topic (e.g., for 1..3 { .say }.  This is mentioned in a few places (and is used in multiple examples) but isn't prominently documented.  It's currently documented in passing at https://docs.raku.org/language/classtut#Methods
But it should probably be mentioned in Language/syntax, Language/operators and/or type/Method.

I'm not a great fan of adding tutorial-ish stuff to type/ pages; I think those pages are meant to be references of very specific things, like here the Method type, not "methods" as a broader concept.
With language/syntax, I'm not sure what you'd like to see. It points to the class tutorial at an appropriate place, in my opinion. If you'd like to see more, please give further hints...
By the way, there is a problem there:

When declared within a class, a subroutine is named "method"

This is utterly wrong. One can declare subroutines inside of classes, moreover I think one can even declare methods outside of classes. This is the reason I don't like arguments like "nobody ever complained so it must be good". For now, I may just open an issue for it because it's not immediately obvious to me how to fix that.
For language/operators, I agree. Perhaps you would be interested in my rework ideas for the operator? In this particular case, I'm not sure what to do. Perhaps the whole concept of "methodop" should go. Some of them can be used as prefix, implying the topic, but not all of them, e.g >>. notoriously cannot be called that way; it will be a syntax error. This could be yet another aspect of operators that could be put into a table, I mean whether they can be used as "prefix", implying the topic variable as the self-object, or not.


we need the a variable to replace 50 in the epochs in matchdg

https://github.com/marrlab/DomainLab/blob/214f667fc005ea3a4431b194b9618fecbb9952f3/examples/yaml/benchmark_mnist.yaml#L126
# this shall be set to be epos - epochs_ctr
# as epos cannot be referenced here one needs to hardcode this right number of epos here
epochs_erm:
  reference: max(1, 50 - epochs_ctr)

Note we have
domainlab_args:
  # Domainlab arguments passed to each task.
  # task specific arguments take precedence.
  ## dataset
  task: mnistcolor10
  tr_d: [1, 2, 4]
  dmem: False
  lr: 0.001
  epos: 50
  es: 5
  bs: 64
  nname: conv_bn_pool_2
  san_check: True

where epos are defined

In https://github.com/marrlab/DomainLab/pull/188

fixed c119559f73547a4463a07d00c9b116699f8e1a1c


Delete securityContext for copy-vault-env initContainer

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #663

License
Apache 2.0

What's in this PR?
see #663
Why?
see #663

@devlounge do you have any further plans with this?

I've built our own webhook, which does some extra things based on how we implement vault security internally.
But the solution was to clone, from spe.containers[0], the securityContext.capabilities.drop into the copy-vault-env initContainer and the vault-agent initContainer. and then it works.

@devlounge Would you plan to contribute back to Bank-Vaults the solution?

fixed by #698


firefish.social returning internal server error

error" : {
"code" : "INTERNAL_ERROR",
"message" : "Internal error occurred. Please contact us if the error persists.",
"id" : "5d37dbcb-891e-41ca-a3d6-e690c97775ac",
"kind" : "server"
}

looks like they fixed it


Create new domain with geogrid.exe

Describe the New Feature
This represents the larger vision of having the user able to select their own domain parameters via the UI. Issue #111 will handle running geogrid and uploading to S3. This issue should handle enhancing the UI to select these parameters, passing them through the api to the runtime module to create the namelist on the fly and run geogrid.
Information needed from the user for the namelist.wps should be limited to:

max_dom
parent_id
parent_grid_ratio
i_parent_start
j_parent_start
e_we
e_sn
geog_data_res
dx
dy
map_proj
ref_lat
ref_lon
truelat1
truelat2
stand_lon

Acceptance Testing
List input data types and sources.
Describe tests required for new functionality.
Time Estimate
3 days
Sub-Issues
Consider breaking the new feature down into sub-issues.

[ ] Related issue: #44

Relevant Deadlines
List relevant project deadlines here or state NONE.
Define the Metadata
Assignee

[x] Select engineer(s) or no engineer required
[x] Select scientist(s) or no scientist required

Labels

[x] Select component(s)
[x] Select priority

Projects and Milestone

[x] Select Project
[x] Select Milestone as the next official version or Backlog of Development Ideas

New Feature Checklist

[ ] Complete the issue definition above, including the Time Estimate and Funding source.
[ ] Fork this repository or create a branch of develop.
Branch name: feature_<Issue Number>/<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project, and Development issue
Select: Milestone as the next official version
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

Related to #111 and #112

The scope of this issue is closed with #112.  New issue #144 will involve interactive GUI.


官方提供的 ttsfrd 安装不上

(cosyvoice) E:\python\CosyVoice\pretrained_models\speech_kantts_ttsfrd>pip install ttsfrd-0.3.6-cp38-cp38-linux_x86_64.whl
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
ERROR: ttsfrd-0.3.6-cp38-cp38-linux_x86_64.whl is not a supported wheel on this platform.

没有支持win系统的ttsfrd吗

no, we do not have windows ttsfrd whl. We will replace it with wetext later

wetext likely won't be working? The pynini package it use is linux only also.

running with WSL

I commented those lines with ttsfrd, as the text I use can be cleaned already.

ubuntu 上为何也不行？

顺便说下，resource.zip 貌似也有问题

ttsfrd 通过python sdk下载的没有这个文件
ttsfrd-0.3.6-cp38-cp38-linux_x86_64.whl
可以看下git的那个下载

ubuntu 上为何也不行？
顺便说下，resource.zip 貌似也有问题

下载地址

没有支持win系统的ttsfrd吗

please update the repo, ttsfrd is not optinal during inference.

我用的ubuntu20，  也报这个错 ：ttsfrd-0.3.6-cp38-cp38-linux_x86_64.whl is not a supported wheel on this platform.

ubuntu一样报了找不到这个

ubuntu22 同顶

check faq.md, ttsfrd is a frontend package, you can ignore it, we will replace it with wetext later

os: archlinux
python:3.11
ERROR: ttsfrd-0.3.6-cp38-cp38-linux_x86_64.whl is not a supported wheel on this platform.

关键这个只提供cp38，现在用python3.8的已经属于版本很低的了吧，为什么强制用cp38

I've been trying to find a ttsfrd package for 3.11 but I haven't been able to find any. It feels like a port from python3.8 to newer versions shouldn't be too hard, but idk. I might be wrong.


Raise an error and tell users what to do on Windows.

What do these changes do?
Raise an meaningful error message when users are trying to install graphscope on Windows.
Related issue number
Fixes #1284.

Merged as the failure is unrelated with this pull request.


[stable/phpbb] Implement again "Standardize 'fullname' and 'name' macros"

This reverts commit e206fe9c8c7365f825719bc8c2bc03901dce9eff.
Implement #15429 bumping a major version after #15605
Signed-off-by: Carlos Rodriguez Hernandez<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/retest


RxJava support

Any plans to support RxJava to get events as a Observable?

This could probably live in a separate module just as the testing infrastructure.


(doc)(ENGTASKS-3605) Update Package Verifier documentation

Description Of Changes
This PR updates the Package Verifier documentation to reflect the current box image being used by the service.
Motivation and Context
This documentation was at least 2 versions behind, and a new version of the box image is being released which is a good time to ensure the docs are up to date
Testing

[x] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[x] Minor documentation fix (typos etc.).
[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left-hand side)/
[ ] Menu structure has been updated

Related Issue

ENGTASKS-3605

Thanks for getting that updated @Windos !


Also remove string-like empty values

Hey,
I've come around your npm module and it's really helpful, but there is one thing I would ask if you think, that should be included or if I would have to fork the module:
Working with templates I've the "problem" that I can't omit values like undefined or null, but in a form of string. So if you think in json it would be
{
  "key": "null"
}

Would be quite nice if the module could also remove this from my object. I could think of an extra option, which activates the "removeFalsyStrings" behaviour.
I hope it is clear, what I meant by that.
Thank you :)

Hi @ChimiDEV,
This is an interesting request. I think this could make sense as an analogous feature to the NULLIF postgres function.
So, in this case it would be something like: cleanDeep(object, { customValues: ['null', 'undefined', 'foobar'] })
Would that fit your use case?

@nunofgs I'm going to ask you to add support trimming white space sting like " ".  But from your comment I think I can use customValues: [" "]. And maybe you could make it support RegEx so I can add an option like customValues: [/\s/] to catch all empty spaces.
Or another way is to support custom function that triggers after your cleansing logic.

I've added cleanValues as an array of strings that you can use to clean specific values. It felt a little cleaner than adding a custom function for now.
To be clear, I'm not against adding a custom function and will do so if there's enough interest in it. Just thought this would be useful for now and would fix the use-cases described in this issue.


Pushing code!! → Err: Cloning into 'node_modules_gh-pages@2.0.1@gh-pages.cache\git@github.com!jaywcjlove!linux-command.git'... Host key verification failed. fatal: Could not read from remote repository.  Please make sure you have the correct access rights and the repository exists.

我执行npm start报的错误。或者执行 npm run deploy报的错误
Pushing code!! → Err: Cloning into 'node_modules_gh-pages@2.0.1@gh-pages.cache\git@github.com!jaywcjlove!linux-command.git'...
Host key verification failed.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

@liuyang77886 https://github.com/jaywcjlove/linux-command/issues/101#issuecomment-491719880


[bug] ImportError and OSError: [E050]

i've been stuck with these error when i'm trying to run the example code i found form the guardrails-ai website and github.
i followed the instruction on https://www.guardrailsai.com/docs/getting_started/quickstart, which i pip install guardrails-ai and guardrails configure, then everytime when i try to run "from guardrails.hub import ProfanityFree" i got this error :
ImportError: cannot import name 'ProfanityFree' from 'guardrails.hub' (/usr/local/lib/python3.10/dist-packages/guardrails/hub/init.py)
i've tried many ways to deal with it, such as !python -m spacy download en_core_web_sm or viewing the Guardrails hub downlaod path and the files within the package, but none of them works.
the OSError seem to be related to the previous error, i encounter this freuqently when i'm installing thing related to Dbias :
OSError: [E050] Can't find model 'en_pipeline'. It doesn't seem to be a Python package or a valid path to a data directory
i've try install the missing packages chatgpt suggested me, but still stuck here
i'm coding in google colab by the way
it'll be realy helpful if someone can provide a comperhensive code for installing and runing guardrail-ai validator
any comments and suggestions will be appreciated.

hi could you try guardrails hub uninstall hub://guardrails/profanity_free it looks like there might have been an error with its installation and its blocking the initialization of the rest of the library

Hi @Iris-Liuu as for:
the OSError seem to be related to the previous error, i encounter this freuqently when i'm installing thing related to Dbias :
OSError: [E050] Can't find model 'en_pipeline'.

The following might work:
pip install https://huggingface.co/d4data/en_pipeline/resolve/main/en_pipeline-any-py3-none-any.whl

This validator might need a bit of a rewrite to bring it into the current year.  The models still appear to be public, though.

@dtam it works(solve the previous import error), thanks a lot
@AlejandroEsquivel i tried install it by pip install and download it then install with path, but both way encounterd the below error
ERROR: Invalid requirement: 'en-pipeline==any': Expected end or semicolon (after name and no valid version specifier)  en-pipeline==any


v1.4 backports 2019-10-22

#9417 -- bugtool: add cilium node list output (@ianvernon)

Once this PR is merged, you can update the PR labels via:
$ for pr in 9417; do contrib/backporting/set-labels.py $pr done 1.4; done

This change is

never-tell-me-the-odds

test-upstream-k8s


remove bundler/minifier from ChatSample

It doesn't like multi-targeted projects and causes file locking issues on build, making it flaky.
I also removed the less file since we don't really need to use it in a sample. Similarly, I removed reference to the .min.css file in the Layout page.
NOTE The diff got a little borked. All I did was delete the old site.css (generated from the less file) and rename site.less to site.css, removing the leading @import statement. We had two copies of bootstrap because we were bringing it in via a <link> tag on the view and we were embedding it in our css file via the less @import command.

Travis failure is definitely timeout issues. AppVeyor is not one we've seen before but it does create a bunch of different tasks and depends upon them completing in a timely fashion, so it could be the same issue. Rerunning CIs.

@anurse Merge this


venv_service fails on windows

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/254
Originally created by @swiffer on 2018-12-29 09:46:15

There may be a larger issue for supporting a Windows environment in general.
when trying to create venvs on meltano init project-name in dbt installation step init fails on windows
Traceback (most recent call last):
  File "C:\Program Files\Python37\Scripts\meltano-script.py", line 11, in <module>
    load_entry_point('meltano==0.3.3', 'console_scripts', 'meltano')()
  File "c:\program files\python37\lib\site-packages\meltano\cli\__init__.py", line 6, in main
    cli()
  File "c:\program files\python37\lib\site-packages\click\core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "c:\program files\python37\lib\site-packages\click\core.py", line 717, in main
    rv = self.invoke(ctx)
  File "c:\program files\python37\lib\site-packages\click\core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "c:\program files\python37\lib\site-packages\click\core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "c:\program files\python37\lib\site-packages\click\core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "c:\program files\python37\lib\site-packages\meltano\cli\initialize.py", line 23, in init
    project = init_service.init()
  File "c:\program files\python37\lib\site-packages\meltano\core\project_init_service.py", line 27, in init
    self.project_echo("", True)
  File "c:\program files\python37\lib\site-packages\meltano\core\project_init_service.py", line 49, in project_echo
    click.secho(f"{star}{check}\tCreated", fg="blue", nl=False)
  File "c:\program files\python37\lib\site-packages\click\termui.py", line 478, in secho
    return echo(message, file=file, nl=nl, err=err, color=color)
  File "c:\program files\python37\lib\site-packages\click\utils.py", line 260, in echo
    file.write(message)
  File "c:\program files\python37\lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode character '\u2b50' in position 0: character maps to <undefined>

View 19 previous comments from the original issue on GitLab


Paragraph detection issue and missing text

Hi,
Grobid appears to be quite powerful.
eLife Sciences may have some kryptonite in the form of a test PDF file with various formatting options:
https://github.com/elifesciences/XML-mapping/blob/master/elife-00666.pdf
When I let Grobid loose on it then I experienced some paragraph detection issue and probably consequently missing text.
If you search for say 'Example of a small' then you will see it on page 12. The text from page 10 correctly skips page 11 that only has an image but then continues with the header of the box. i.e. "...eLife content is delivered to more repositories and it can be Box 1. Example of a small box..." It should have continued with the right column. It then continues with the following page 'We need to allow authors...' (omitting the sub header). The content of the Box 1 and the text of the right column doesn't seem to appear anywhere.
This may be a particular difficult PDF and there may be other issues. Is that likely something that would get fixed as part of Grobid or should that be addressed outside as a pre-processing step? And if so, what would be the best way to achieve some pre-processing to say tell Grobid what the paragraphs are and let it it do the annotation of the content?
Thank you

I noticed the same or something similar happens with the first PMC manuscript for example. However, the extracted XML looks a bit different depending on the parameters. The PMC evaluation doesn't include assets where as the default for fulltext extraction is to include it (took me a while to track that down).
So with assets enabled, it starts to loose paragraphs on page 237 which happens to be after a figure:
"Using simulations from the combined..." is not in the TEI XML if assets are enabled (it is when assets are disabled using the -ignoreAssets parameter).
The three immediately following paragraphs affected as well:
"This analysis provides..."
"In the development of new drugs..."
"It is currently unknown how different tumor types..."
Also "Open Access This article is distributed under the terms..." is not included in that case but not desired as a paragraph either. It is however included when assets are ignored (I guess it should be included as a license - it is tagged as back/ack in the training XML).

Removal of content is a major issue. Appreciate any fix for this.

Hello,
The reasons why some text might be missing are:

pdf parsing issues, in particular problem with capturing the reading order,

misclassification of zones at the segmentation level (for instance a paragraph is wrongly identified as annex) and at full text level (for instance a paragraph is wrongly recognized as figure caption).

Both factors can play in combination... The reason why using the "asset" option has an impact is that when the so-called assets are explicitly extracted (assets means the bitmap and SVG embedded in the PDF), they are exploited to detect zones and figures, which has some consequences in having normal paragraphs misclassified for instance as figure captions.
What I have planed so far is to remove the "assets" option and the extraction of bitmap directly from the PDF - because it's not reliable, for instance we can have PDF files with several ten thousand bitmaps in it (in general one embedded bitmap file per image line...). It will be replaced by the extraction of figures and formulas based on coordinates after recognition of this structures.
The second axis of improvement is better PDF parsing and reading order for the PDF elements. This is work in progress with the pdf2xml fork.
Third way to improve that it to have better segmentation, which is mainly an issue of training data (there's a lack of training data for the segmentation model) and features (but we need more training data to introduce more features productively).
For instance @de-code, we could add a couple of additional training data for the segmentation model to capture eLife PDF layout, and this could make the processing of your PDF much more reliable.

Thank you for that.
I think in that case the text was missing all-together and didn't appear as a figure or table description either. I guess it would be interesting to debug that to find out what was actually causing it.
Using the assets / images to aid segmentation seems to make sense. At least I'd be interested in that.
The eLife PDF document itself is not so important itself. The XML will have already been created by then.
But of course it would be good to add training data. Maybe we can generate some. Should I raise another issue for that?

After having debugged with such document, the extraction from pdf seems correct and the missing parts (the boxes) are ending up in the <annex> when passing through the segmenter:
CME:	5'-CTAGAAATTTGTACGTGCCACAGA	cme:	C	CM	CME	CME:	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	ALLCAP	NODIGIT	0	0	0	0	0	0	0	0	7	0	:'--	4	6	0	0	0	0	1	<body>
3'	3'	3'	3	3'	3'	3'	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	ALLCAP	CONTAINSDIGITS	0	0	0	0	0	0	0	0	7	0	'	1	0	0	0	0	0	1	<body>
Acknowledgements	Acknowledgements	acknowledgements	A	Ac	Ack	Ackn	BLOCKSTART	PAGEIN	NEWFONT	HIGHERFONT	0	0	INITCAP	NODIGIT	0	0	1	0	0	0	0	0	7	1	no	0	10	0	0	0	0	1	I-<acknowledgement>
Main	thanks	main	M	Ma	Mai	Main	BLOCKSTART	PAGEIN	SAMEFONT	LOWERFONT	0	0	INITCAP	NODIGIT	0	1	1	0	0	0	0	0	7	1	no	0	10	0	0	0	0	1	<acknowledgement>
We	thank	we	W	We	We	We	BLOCKSTART	PAGEIN	NEWFONT	HIGHERFONT	0	0	INITCAP	NODIGIT	0	0	1	0	0	0	0	0	7	1	,	1	10	0	0	0	0	1	<acknowledgement>
their	contributions.	their	t	th	the	thei	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	1	0	0	0	0	0	7	1	.	1	4	0	0	0	0	1	<acknowledgement>
Box	1.	box	B	Bo	Box	Box	BLOCKSTART	PAGEIN	NEWFONT	HIGHERFONT	0	0	INITCAP	NODIGIT	0	1	1	0	0	0	0	0	7	2	.	1	10	0	0	0	0	1	I-<annex>
box	box	box	b	bo	box	box	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	1	1	0	0	0	0	0	7	2	no	0	1	0	0	0	0	1	<annex>
Donec	rhoncus	donec	D	Do	Don	Done	BLOCKSTART	PAGEIN	NEWFONT	LOWERFONT	0	0	INITCAP	NODIGIT	0	0	0	0	0	0	0	0	7	2	.	1	8	0	0	0	0	1	<annex>
vitae	enim	vitae	v	vi	vit	vita	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	no	0	9	0	0	0	0	1	<annex>
arcu.	Pellentesque	arcu.	a	ar	arc	arcu	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	.	1	9	0	0	0	0	1	<annex>
senectus	et	senectus	s	se	sen	sene	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	-	1	9	0	0	0	0	1	<annex>
pis	egestas.	pis	p	pi	pis	pis	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	.	1	9	0	0	0	0	1	<annex>
rutrum.	Praesent	rutrum.	r	ru	rut	rutr	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	.,-	3	10	0	0	0	0	1	<annex>
tudin	purus	tudin	t	tu	tud	tudi	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	.	1	8	0	0	0	0	1	<annex>
arcu.	Fusce	arcu.	a	ar	arc	arcu	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	...	3	9	0	0	0	0	1	<annex>
Suspendisse	eu	suspendisse	S	Su	Sus	Susp	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	INITCAP	NODIGIT	0	0	0	0	0	0	0	0	7	2	no	0	9	0	0	0	0	1	<annex>
imperdiet.	(see	imperdiet.	i	im	imp	impe	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	.(,;.,	6	9	0	0	0	0	1	<annex>
2014;	and	2014;	2	20	201	2014	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	ALLCAP	CONTAINSDIGITS	0	0	0	0	1	0	0	0	7	2	;.,)	4	8	0	0	0	0	1	<annex>
ultrices	vehicula	ultrices	u	ul	ult	ultr	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	2	,-	2	9	0	0	0	0	1	<annex>
per	suscipit.	per	p	pe	per	per	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	1	0	0	0	0	0	7	2	..	2	7	0	0	0	0	1	<annex>
DOI:	https://doi.org/10.7554/eLife.00666.002	doi:	D	DO	DOI	DOI:	BLOCKSTART	PAGEIN	NEWFONT	LOWERFONT	0	0	ALLCAP	NODIGIT	0	0	0	0	0	0	0	0	7	4	:://././..	10	10	0	0	0	0	1	<annex>
Box	2.	box	B	Bo	Box	Box	BLOCKSTART	PAGEIN	NEWFONT	HIGHERFONT	0	0	INITCAP	NODIGIT	0	1	1	0	0	0	0	0	7	4	.	1	10	0	0	0	0	1	<annex>
This	box	this	T	Th	Thi	This	BLOCKSTART	PAGEIN	NEWFONT	LOWERFONT	0	0	INITCAP	NODIGIT	0	0	1	0	0	0	0	0	7	5	-.,.	4	9	0	0	0	0	1	<annex>
vel	rhoncus	vel	v	ve	vel	vel	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	..	2	9	0	0	0	0	1	<annex>
faucibus.	Vivamus	faucibus.	f	fa	fau	fauc	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	..,,	4	10	0	0	0	0	1	<annex>
odio	purus	odio	o	od	odi	odio	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	,.	2	9	0	0	0	0	1	<annex>
hendrerit.	Praesent	hendrerit.	h	he	hen	hend	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	...-	4	10	0	0	0	0	1	<annex>
quam	lobortis	quam	q	qu	qua	quam	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	,.,	3	9	0	0	0	0	1	<annex>
commodo,	eros	commodo,	c	co	com	comm	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	,,.	3	9	0	0	0	0	1	<annex>
efficitur	tincidunt.	efficitur	e	ef	eff	effi	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	.,,,-	5	9	0	0	0	0	1	<annex>
pat	velit	pat	p	pa	pat	pat	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	1	1	0	0	0	0	0	7	5	..-	3	9	0	0	0	0	1	<annex>
lentesque,	ipsum	lentesque,	l	le	len	lent	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	,,,.	4	9	0	0	0	0	1	<annex>
Vestibulum	sit	vestibulum	V	Ve	Ves	Vest	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	INITCAP	NODIGIT	0	0	0	0	0	0	0	0	7	5	.	1	10	0	0	0	0	1	<annex>
urna	lobortis,	urna	u	ur	urn	urna	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	5	,;.,.,.	7	8	0	0	0	0	1	<annex>
Box	2-Figure	box	B	Bo	Box	Box	BLOCKSTART	PAGEIN	NEWFONT	LOWERFONT	0	0	INITCAP	NODIGIT	0	1	1	0	0	0	0	0	7	9	-.	2	10	0	0	0	0	1	<annex>
DOI:	DOI:	doi:	D	DO	DOI	DOI:	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	ALLCAP	NODIGIT	0	0	0	0	0	0	0	0	7	9	:	1	1	0	0	0	0	1	<annex>
Donec	rhoncus	donec	D	Do	Don	Done	BLOCKSTART	PAGEIN	NEWFONT	HIGHERFONT	0	0	INITCAP	NODIGIT	0	0	0	0	0	0	0	0	7	9	..-	3	10	0	0	0	0	1	<annex>
tesque	habitant	tesque	t	te	tes	tesq	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	9	.	1	9	0	0	0	0	1	<annex>
nunc	id	nunc	n	nu	nun	nunc	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	9	.,.	3	9	0	0	0	0	1	<annex>
fermentum	arcu.	fermentum	f	fe	fer	ferm	BLOCKIN	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	9	...	3	9	0	0	0	0	1	<annex>
imperdiet.	Vestibulum	imperdiet.	i	im	imp	impe	BLOCKEND	PAGEIN	SAMEFONT	SAMEFONTSIZE	0	0	NOCAPS	NODIGIT	0	0	0	0	0	0	0	0	7	9	.,..	4	9	0	0	0	0	1	<annex>
DOI:	https://doi.org/10.7554/eLife.00666.002	doi:	D	DO	DOI	DOI:	BLOCKSTART	PAGEIN	NEWFONT	LOWERFONT	0	0	ALLCAP	NODIGIT	0	0	0	0	0	0	0	0	7	11	:://././..	10	10	0	0	0	0	1	I-<header>

I am feeling the same issue in 2024 too


Update library/Zend/Json/Server/Server.php

Could be just a typo error.

I made the changes necessary and pushed.

Thank you!


Window placement centered ignores display scaling on Windows 10

Godot version
v3.4.stable.official.206ba70f4
System information
Windows 10 19042.1415, 3840x2160, 150% scaling
Issue description
When UI scaling is set to >100% (150% in my case), the game window doesn't get centered properly - it looks like it's getting it position set based on screen resolution (3840x2160) without applying display scaling.

Steps to reproduce

Set Windows display resolution to 3840x2160.
Set Windows display scaling to 150%.
Set Godot Display Scale to 150% (should be the default value). EDIT: Happens also with 100% and any other value so I don't think it's dependent on Godot's UI scaling.
Set Godot Window Placement to Centered.
Run the game.

Minimal reproduction project
No response

Duplicate of https://github.com/godotengine/godot/issues/13017.


Add hooks to source scripts before and after setup

We would like to use this repo without forking it, so this PR adds hooks so that we can execute scripts before and after running the setup. If $HOME/.pivotal-workstation-pre-setup and/or $HOME/.pivotal-workstation-post-setup, scripts in them will be executed before and after the bulk of setup.sh.
Also, fixes git status alias.

A small win for us is that we can combine those two steps into just running our team script, which means we can run a single command and walk away for awhile when setting up a new workstation.
The specific problem we were encountering with separate scripts is that we want to run things that require sudo before and after running the setup.sh script. We couldn't find a solution that didn't require typing in the sudo password more than once without introducing these hooks.

@nertzy and I discussed this PR. Normally we would expect a team to run a team script after workstation setup to finish any team specific customizations. As such, it would be fine for that team script to call the workstation setup script. In this case, there is no need for any pre or post hooks.
#!/usr/bin/env bash
team setup stuff
pushd workstation-setup
./setup.sh
popd
more team setup stuff

If other teams are wanting the feature described in this PR, let us know. Thanks for understanding @mdelillo

@professor @nertzy that's fair, and that is how we are using it for the most part.
Taking a step back, the original problem we are trying to solve is that we had to type the sudo password multiple times when running setup.sh. The script takes long enough to run that the password gets uncached. We have a script we run before setup.sh that extends the timeout in /etc/sudoers, but that itself needs the password to be cached. It is later reset here, requiring it to be typed again.
Another way we could go about solving this would be to only prompt for the password if it isn't cached. Something along the lines of this could work:
if ! sudo -nv 2>/dev/null; then
  sudo -K
  sudo true
fi

If you are open to something like that (or even modifying the sudoers file itself, but I'm not sure that's a great solution), I'd be happy to update this PR or make a new one.
Thanks!

We are declaring PR bankruptcy as per #295.
Please review recent changes and direction and resubmit if needed.
Thank you!


RCA: Varied CI Build performance for MacOS, Windows and Ubuntu

Build times:

Ubuntu (Fastest)
Windows
MacOS (slowest)

Total CI run time is 5 hrs 47 mins which is unfeasible:

Looking at the images above, Build and Test step takes the most time. (MacOS takes almost 2.5x the time than Ubuntu, and Windows takes ~ 2x the time than Ubuntu)
Going one step further and looking at the logs for each OS build for Build and Test:

MacOS: logs show a slurry of Connection refused and Node closed while execution action leading to multiple retries causing the build time to go up. At some point we also see Failed to start the node and Address already in use messages too.
Windows: Sees similar log messages above but relatively less frequent.

Maybe we could try using larger runners for the windows instance, windows-latest-8-cores? [1]
[1] https://github.blog/changelog/2022-10-20-github-actions-larger-hosted-runners-are-now-automatically-created-for-customers/

Closing this as CI seems to be stable, macos runners have been disabled and the build times between ubuntu and windows are within 2 mins of each other.


Tvp tests

Changed some tests.
Changed makeParamExtra case TVP

@v-kaywon will you merge it?

@NikitaDef  Can you make the minor change mentioned above? Other than that I think everything looks good.

@v-kaywon I made change


Add List Skeleton and modifiable Corner Radius

Description
This pull request introduces two significant enhancements to the EasySkeleton project: the addition of a List Skeleton and the integration of a modifiable corner radius directly within the initialization function.
Rationale
1. List Skeleton:

User Preference: Many developers prefer using List over ForEach in their SwiftUI applications. By adding a List Skeleton, we cater to this preference, making the library more versatile and user-friendly.
Logical Step: Given the widespread use of List, adding a skeleton placeholder for it is a logical progression for the EasySkeleton library, enhancing its utility.

2. Modifiable Corner Radius in Initialization:

Simplified Usage: Previously, the corner radius modifier function was needed frequently throughout the codebase. By allowing the corner radius to be set directly in the initialization, we simplify the code, reduce redundancy, and improve readability.
Convenience: This change streamlines the process of customizing the skeleton's appearance, providing a more intuitive and efficient way to set the corner radius.

Changes Made
1. Added List Skeleton:

Implemented a new List Skeleton that can be used as a placeholder for lists, ensuring consistent design and behavior with the rest of the EasySkeleton components.

2. Modified Initialization to Include Corner Radius:

Updated the initialization function to accept a corner radius parameter.
Adjusted relevant parts of the codebase to utilize the new initialization parameter, removing the need for the separate corner radius modifier function.

Benefits

Enhanced Flexibility: Users can now easily create skeletons for lists, broadening the library's applicability.
Improved Code Maintainability: Centralizing the corner radius modification within the initialization reduces code duplication and enhances maintainability.
User Convenience: Simplifies the customization process, making the library more intuitive and easier to use.

Hi!
Looks fantastics!  Thanks for you job and improvements to EasySkeleton.


oneOf support

Hey! Thank you this lib, very good work!
I have a specific use case that i can not really solve with current implementation :( And i wanted to ask if that is fine to add that functionality to the lib.
The use case:
I send different messages through different channel and type of message is defined by special header, so i cannot really define 1 dto with 1 schema, i would need to use something like oneOf thing from json schema.
Right now the type is like this
/// ...
  payload: {
      type: Type<unknown> | Function | [Function] | string;
  };
/// ...

But instead we could do something like this:
type PayloadSchema = Type<unknown> | Function | [Function] | string;
/// ...
  payload: {
      type:
            PayloadSchema |
            { oneOf: { type: PayloadSchema }[] } |
            { anyOf: { type: PayloadSchema }[] } ; // and so on
  };
/// ...

What do you think about this approach?

Thanks, @mdovhopo.
That sounds like a nice improvement, gonna check out how it's done in nestjs/swagger and try to implement a generic approach

I've played a bit with asyncapi playground and what you can do looks like:
asyncapi: 2.5.0
info:
  title: Account Service
  version: 1.0.0
  description: This service is in charge of processing user signups
channels:
  user/signedup:
    subscribe:
      message:
        $ref: '#/components/messages/UserSignedUp'
components:
  messages:
    UserSignedUp:
      payload:
        type: object
        properties:
          feline:
            oneOf:
            - $ref: '#/components/schemas/Cat'
            - $ref: '#/components/schemas/Lion'
  schemas:
    Lion:
      type: object
      properties:
        bark:
          type: boolean
        breed:
          type: string
          enum: [Dingo, Husky, Retriever, Shepherd]
    Cat:
      type: object
      properties:
        hunts:
          type: boolean
        age:
          type: integer

paste into https://studio.asyncapi.com/ to see the results
that can be achieved by having a Dto that has a field with 'oneOf':
export class Feline {
  @ApiProperty({
    oneOf: [{ $ref: getSchemaPath(Cat) }, { $ref: getSchemaPath(Lion) }],
  })
  feline: Cat | Lion;
}

I'll add that case into the sample app this week

Okay, I see thank you for the help 🙀

You can't define oneOf on outer level, you need an object where one of the fields can be of oneOf type.


Inconsistent adherence to config file parameters (e.g. number of tracks)

I have a little script for my Micropolis drive which is set up to capture an image file, and also a copy of the flux in one command, to minimize wear and tear on the disks I am archiving.
But, oddly, it doesn't seem to always pay attention to the number of tracks.
A sample of the command I am using is:
D:\Download\Fluxengine\fluxengine read micropolis config.micropModI -o disk33-CPMBackups.img --copy-flux-to disk33-CPMBackups.flux --flux_source.drive.revolutions=3.5  1>disk33-CPMBackups.log
Sometimes this works right, sometimes not.  this one read 77 tracks instead of the expected 34.  And it couldn't decode the data - as though it ignored the config file entirely.
The config file config.micropModI reads:
decoder {
micropolis {
}
}
cylinders {
start: 0
end: 34
}
heads {
start: 0
end: 0
}
comment: "Micropolis Mod I MetaFloppy 143KB 5.25" SSDD hard-sectored (ro)"
image_writer {
filename: "micropolis.img"
img {
}
}
flux_source {
drive {
drive: 0
hard_sector_count: 16
}
}
flux_sink {
drive {
drive: 0
}
}
Log fiile attached.
disk33-CPMBackups.log

Update.  I think I know what was behind this.  In my script I have a number of seek commands (multiple to ensure the drive actually gets home):
%fluxdir%\fluxengine seek config.micropModI -c 0
%fluxdir%\fluxengine seek config.micropModI -c 0
%fluxdir%\fluxengine seek config.micropModI -c 0
These had been miscoded to config.micropModII (a 77 track drive).  Curious though, that those "stuck" and seemed to prevent the fluxengine from paying attention to the config.microModI file specified in the read comand.

Ah, now I DO know what is going on.  I had not realized this was a MOD II disk, so fluxengine was picking up high track numbers - and struggling with the disk because it was essentially reading two tracks at once.  (It might be good to have it issue a warning when it sees track numbers outside the range in the config, though.  ;) )

The above was partly right.  However, I also see that fluxengine will create sector images beyond the config'd tracks when it has trouble decoding sector headers.  See attached.  This was from a disk known to be Mod I, 35 tracks.
disk23.zip

Ah, yes, that'll do it. If you tell the image_writer the geometry of your output image it should discard out-of-range sectors. Otherwise it'll guess the geometry based on the sectors it's seen, so a bad sector with a high track or sector number will cause Bad Things™ to happen.
What's a Mod I / Mod II disk?

Mod I  48 TPI, 35 Tracks, 143KB
Mod II  100 TPI, 77 Tracks, 315KB

Sorry for the delay in response; working on other projects... this probably needs multiple Micropolis configs, for each disk variant. This should avoid the problem you were seeing earlier. If you've got anything which works, send me a PR please? I'll have to take your word for it as I can't test Micropolis hardware!

I had two different configs, one for the Mod I and one for the Mod II, the difference being the number of tracks.


Updates vendor

Towards https://github.com/giantswarm/giantswarm/issues/1482
I have put off updating vendor for a while, so this is more painful than it should be. I can only apologise. This is as small as I could get it without breaking master. Going by commit might help reviewing.
To summarise (in no particular order):

Removed cenk/backoff, moved to cenkalti/backoff
Moved to operatorkit patch structs
Moved to k8s.io/api/core/v1

Successfully tested on lycan.

Alternatively we could iterate on such massive vendor bumps by pinning commits so we can iterate better until we are back on track.


Refactor SMES component to move logic into PowerSmesSystem.

More closely follows how APC code is structured. Precursor to implementing a UI.

About the PR

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

While you're at it, you can fix the input/output sprites used in the SMES visualizer at https://github.com/space-wizards/space-station-14/blob/02aef68dff2048db2ddeae113611c9dc113d86d3/Content.Client/Power/SMES/SmesVisualizer.cs#L45 which should be separated in two layers (input for the top light, output for bottom iirc).

~While you're at it, you can fix the input/output sprites used in the SMES visualizer at~
https://github.com/space-wizards/space-station-14/blob/02aef68dff2048db2ddeae113611c9dc113d86d3/Content.Client/Power/SMES/SmesVisualizer.cs#L45
~which should be separated in two layers (input for the top light, output for bottom iirc).~
Nevermind, it should probably wait for the UI to be finished.

Ok -- I can try to do that along with the UI changes.

just some nitpicks

@AJCM-git mind taking another look?

Should be fine after this (do test in case there is any unexpected behavior)

Tested


Update Prize page with placeholder values

We don't have all the information about the prizes just yet (bc department is slow), but we do have the prize track categories figured out. Here is the information we can update/include for the soft-launch

[ ] updated prize categories
[ ] replace actual prizes with placeholder (change image, add "coming soon", etc)
[ ] update pictures for each category

Replaced all prizes with a "Coming Soon" text so will close this.


chore(docs): Fix proper case and improve wording

Hey, I just made a Pull Request!

When reviewing the docs, I noticed one of the entries for TechDocs wasn't properly capitalized in a FAQ.
This PR fixes that, as well as slightly improves the wording in that section.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Bump!


Marking rep as unavailable should not count as complete

Right now, if I mark a contact as "unavailable" it counts the same as making contact (green checkmark and all). It seems more useful for it to either show as uncontacted or yellow, or somesuch. That way, I can remember to come back to that contact later. e.g. tonight I called my rep and their voice mailbox was full. I'll try again tomorrow, but 5Calls won't remind me that I need to do that.

Good point. It should definitely move you to the next contact (if there is one), but not mark it as complete. Thanks for reporting this.

I put up a PR to treat these as 'completed', but then clear them the next 'day'.


Get some idea from other project using "GF SIMD" ?

Thanks to this project that I am using all the time.
I found another project in golang with compatible API like this project, please have a look: https://github.com/templexxx/reedsolomon
Maybe you can have some great idea from it.

Nothing new there. Performance seems similar, and otherwise looking neat. I made performance comparison at #69


Add Vuex state for managing the case beginning-to-end

Remove view guards (at least for right now) - the plan is just not to show UI for elements based on what is in the global state (e.g. no case yet ⇒ don't show detect and select, no marked candidates yet ⇒ don't show annotate and segment, etc).

LGTM, go go vuex store!

guess this addresses #264?

@reubano @lamby More fun to review. This branch also still has a lot to be turned into issues. The goal here was to get most of the API calls into state.js except for on "Open Image", which needs to do a little extra work.
TODOs, which we'll probably turn some of these into issues.

Refactor the OpenDicom viewer to be more succinct vue code
Pull out old API style endpoints
Create in-memory model objects and serialize those rather than calling the DICOM loading methods on the model
Error handling

@pjbull looking now. btw, I commented on #261, but since it was already merged, not sure if it sent you guys the notification. just fyi..


rust-analyzer needs access to $RUSTUP_HOME.

rust-analyzer attempts to access crates for the current project, which are usually stored in ~/.cache/cargo. This should be mounted as a volume.
Without this directory:

The language server needs network access (non-ideal).
The language server will download and compile all dependencies on each run, which is extremely slow and consumes a lot of network.

This isn't the first LSP to require an additional volume for this kind of usage. I'm thinking of maybe adding an extra item to objects in the supported_languages table (e.g.: extra_volumes), with paths to additional volumes that they each require.
This provides a base onto which individual LSPs can specify extra volumes (and, probably, whether these volumes should be :roor :rw).

I believe this is something we still need to solve for - right?

Yup, still unresolved. I think adding an extra_volumes entry (with a /path/on/host:/path/on/container format) would cover this, and a few other situations quite well (e.g.: golang has a volume hardcoded which could use leverage this design).

I think this ticket belongs to the lspcontainers.nvim repo, right? My workaround was to define the cmd_builder similar to the default go command.
  cmd = require('lspcontainers').command('rust_analyzer', {
    network = 'default',
    cmd_builder = function(runtime, workdir, image, network)
      local volume = workdir .. ':' .. workdir
      local env = vim.api.nvim_eval('environ()')
      local cargo_path = env.HOME .. '/.cargo'
      local cargo_path_volume = cargo_path .. ':' .. '/root/.cargo'

      -- add ':z' to podman volumes to avoid permission denied errors
      if runtime == 'podman' then
        cargo_path_volume = cargo_path_volume .. ':z'
        volume = volume .. ':z'
      end

      return {
        runtime,
        'container',
        'run',
        '--interactive',
        '--network=' .. network,
        '--rm',
        '--workdir=' .. workdir,
        '--volume=' .. volume,
        '--volume=' .. cargo_path_volume,
        image,
      }
    end,
  }),

But I agree that an extra_volumes property would be a nice solution.

You're snippet assumes a specific value for RUSTUP_HOME, it should read the environment variable instead.
I don't think it's the right approach anyway. We should add a new property to LSPs: a function that returns additional volumes. Otherwise each LSP with any volumes re-implements the whole builder function.

You're snippet assumes a specific value for RUSTUP_HOME, it should read the environment variable instead. I don't think it's the right approach anyway. We should add a new property to LSPs: a function that returns additional volumes. Otherwise each LSP with any volumes re-implements the whole builder function.

I totally agree with you... It was just a workaround :)

Okay here is what I'm going to do now that I fully understand this. Two things:

I want to re-write lspcontainers now that I have the knowledge I do about Lua and how these language servers can work
I want to consider these types of problems at a higher level as suggested here

I think what I'm going to do is create a new v2 issue with all of the features we would like to implement and improve upon. Then we start from scratch and make this even better.
The original code was really a prototype that we have leaned on but now that we want to support multiple backends, architectures, volumes, etc we need to solve these problems in clearer ways.
I will close this ticket for now and reference it in the new V2 issue to consider changes.
Thank you all for your continued support with this project. IT IS one I want to keep going but think we have places we need to properly consider to better maintain for the future.

I moved to sandboxing several LSPs via bubblewrap, because that allows me to use the distribution-provided LSP. This was especially handy for LSPs with no existing docker image, but also allowed me to de-duplicate a lot of stuff locally and avoid having to build/pull images (especially on aarch64 or after nuking everything docker to clean up disk space)..
If you're going to re-write a lot of this, maybe the design of what I did can serve as a useful reference.:
https://git.sr.ht/~whynothugo/dotfiles/tree/2ca611de/item/home/.config/nvim/lua/lsp.lua#L43-153
Mostly, I added the sandboxing-related config params to the usual LSP configurations map under a sandbox key with a consistent format. I used a wrapper for the setup() call which applies this.
I set each LSP's cmd to false, and then add a wrapper for config.on_new_config which redefines the cmd when the config changes. This lets me update the flags for the LSP to include the root directory of the file being editing (which might change after vim's startup). It's also a great spot to set params.processId = vim.NIL for all sandboxes LSPs, since this interferes with LSPs that don't run on the same PID namespace as neovim.
Anyway, feel free to poke around if you wish.


Issues creating a native module for NSSplitView

Hope it's ok to share that here but I'm trying to create a native module for a macOSNSSplitView, I'm hitting some touch related issues that seems to indicate that a non trivial native implementation might be needed: the left pane never register touches:

edit: also if you click on the left pane at the height of the Click me #2! button (which is located on the right pane but you click where it would be if the left pane did not exist), it still triggers the onPress on button as well. So it looks like both view touches are sent to the right-most view?
I can also see the following error in XCode, obviously related:
~/Developer/react-native-macos/react-native-macos-sandbox/node_modules/react-native-macos/React/Base/RCTTouchHandler.m:111
2020-04-27 23:45:00.681409+0200 MacOSSandbox[10571:4313998] [General] Touch is already recorded. This is a critical bug.
2020-04-27 23:45:00.692835+0200 MacOSSandbox[10571:4313998] [General] (
	0   CoreFoundation                      0x00007fff39141d07 __exceptionPreprocess + 250
	1   libobjc.A.dylib                     0x00007fff71e655bf objc_exception_throw + 48
	2   CoreFoundation                      0x00007fff3916aec8 +[NSException raise:format:arguments:] + 88
	3   Foundation                          0x00007fff3b894456 -[NSAssertionHandler handleFailureInFunction:file:lineNumber:description:] + 166
	4   MacOSSandbox                        0x000000010014e702 -[RCTTouchHandler _recordNewTouches:] + 578
	5   MacOSSandbox                        0x0000000100150923 -[RCTTouchHandler interactionsBegan:] + 83
	6   MacOSSandbox                        0x0000000100150c97 -[RCTTouchHandler mouseDown:] + 151
	7   AppKit                              0x00007fff36576574 -[NSWindow(NSGestureRecognizer_Routing) _sendEventToGestureRecognizers:requireAcceptsFirstMouse:] + 2696
	8   AppKit                              0x00007fff3657e235 -[NSWindow(NSEventRouting) _handleMouseDownEvent:isDelayedEvent:] + 4839
	9   AppKit                              0x00007fff364e8a81 -[NSWindow(NSEventRouting) _reallySendEvent:isDelayedEvent:] + 2612
	10  AppKit                              0x00007fff364e7e29 -[NSWindow(NSEventRouting) sendEvent:] + 349
	11  AppKit                              0x00007fff364e61b4 -[NSApplication(NSEvent) sendEvent:] + 352
	12  AppKit                              0x00007fff3633321f -[NSApplication run] + 707
	13  AppKit                              0x00007fff36304ff6 NSApplicationMain + 777
	14  MacOSSandbox                        0x0000000100001c12 main + 34
	15  libdyld.dylib                       0x00007fff7300ccc9 start + 1
)

I've published a very basic/naive implementation here:  https://github.com/mgcrea/react-native-macos-splitview (example in the README).
I've looked into the <ScrollView /> that seem to wrap the native module with RCTView and manage subviews with insertReactSubview, I've tried to make something alike work but so far no luck.
So I'm wondering if there could be a simple fix to the first/published straightforward implementation. Any ideas on the way forward?
Thanks!

Hi @mgcrea .   I think the reason for the lack of touch events and the RCTAssert above is because in RCTTouchHandler.m in the _recordNewTouches method there is logic to set _shouldSendMouseUpOnSystemBehalf for special cases such as NSControls and NSText.   It may be that another case is needed for NSSplitView.   You could try locally modifying the code to add a case for NSSplitView.   Longer term we may need to provide a method that RCTView sublcasses can override, like a shouldSendMouseUp method or something.
Besides the touch issue, I do think you are going to need a less trivial implemented for other reasons.    In a purely native app, an NSSplitView may have two or more NSView subviews that it lays out.   React-native wants to use Yoga/FlexBox layout on its subviews.   You're going to have a clash between RN and NSSplitView both wanting to layout these child views.
You may want to look at <Modal> component and the RCTModal* classes for inspiration.   In Modal the subcomponents get projected to a completely separate subview hierarchy: <Modal> itself is part of the main react-native UIView hierarchy, but the children become part of a separate UIView hierarchy hosted in a proxy RCTView in a separate UIWindow.    I think for a <SplitView> you're going to want to do something similar: the <SplitView> itself will be part of the main hierarchy, and it will be projected as an NSSplitView, but then perhaps you could have two proxy NSView for the two panes and then project the children of the two <View>'s inside the two proxy views.   It may require another component definition: something like <SplitView><SplitPane>... contents of left pane...</SplitPan><SplitPane>... contents of right pane ...</SplitPane></SplitView>.

Thanks a lot for the help/guidelines!

@tom-un I've tried a few things but alas I quickly hit a roadblock:
In _recordNewTouches, the resolved NSView *targetView = [self.view hitTest:touchLocation]; is always pointing on the right RCTView pane of the split view (even when clicking on the left pane, will give negative X coords in that case). It sometimes point to the actual NSSplitView parent when clicking on the divider or close to the outer edges or sometimes maybe when the window isn't focused).
So the underlying touch issue seems to be that the [RCTRootContentView.view hitTest] call seems to return a bad view, and to my current understanding it's a pure NSView hierarchy call so might not be a react-native issue?
What is strange is that when you debug the view hierarchy everything looks OK so not sure why the root NSView is getting confused. Wrapping the NSSplitView inside a RCTView and manually adding the subViews to the child splitView does not change anything.
Will keep trying a few things but I'm starting to wonder if it might not be easier to rebuild a splitview component from the js side (but not sure if currently doable).

Take a look at RCTModalHostView.m.   It has its own RCTTouchHandler.   I think you're going to have to do something similar.

Did try to setup my own touch handler looking at RCTInputAccessoryView.m that does the same trick, but attaching it to any of the subviews did not change the broken behavior.
However, implementing a basic hitTest did seem to fix most issues.
- (NSView *)hitTest:(NSPoint)aPoint {
  for (NSView *subview in self.subviews) {
    if (!subview.isHidden && [subview hitTest:aPoint])
      return subview;
  }
  return nil;
}

Both touches are now properly working! But as you predicted, now I can see both the SplitView and Yoga fighting for layout.

but the children become part of a separate UIView hierarchy hosted in a proxy RCTView in a separate UIWindow

Is a separate UIWindow required to have separate hierarchy? In my case it would mean that I should have two separate NSWindow for each pane.
I did implement my desired behavior in pure javascript: see SplitView.tsx. Having some minor issues with the cursor and performance.
Thanks again for the guidance!

Quick update, still not working after a few other tries. Unfortunately inserting an NSView between the NSSplitView does not fixes the layout issues, (but it does change the layout).

Somehow the constraints are not synced, which is the same issue without intermediate NSView, basically the grandchild RCTView does not reflect the change of its parent (be it a RCTView or a NSView) when those changes are initiated by the NSSplitView, here an example without intermediate NSView with the view that won't properly adjust highlighted:

After adding the subviews to the NSSplitView, did you call NSSplitView's adjustSubviews method?
I still think you're going to need a  component and in its ShadowView you can have control over how its laid out by react -- or in this case make it not be laid out by react :).    You're going to want these SplitPanes to just let the parent NSplitView control layout completely.    If you want to push the latest to your branch I can take a look.

@tom-un, yep indeed would be great if there was some kind of public boolean on an RCTView to toggle the yoga layout. Digging into the multiple layers of react-native view rendering is a bit arcane to me!
I've also found out that I can manually update the grandchildren frames (with setFrame in splitViewDidResizeSubviews) with something like:
    for (RCTView *grandSubview in subview.subviews) {
      [grandSubview setFrame:NSMakeRect(0, 0, size.width, size.height)];
      [grandSubview setNeedsDisplay];
    }

And that works to fix the grandchild layout, but alas, the grandchildren children won't get their layout updated...! So it looks like it's not propagating changes (and I guess recursing through all subviews isn't the way forward).

I've published my wip sandbox project (should be ready to run) in a sandbox branch, native module is in the modules/ subfolder.
Thanks!

@tom-un The same error appears for me when Contextual Menu for TextInput is open and a Touchable with onPress defined have been pressed for the second time (first click just closes the Contextual Menu, and event is not passed to the Touchable).
020-08-03 21:24:21.389932+0200 Reminders[10623:1878807] *** Assertion failure in -[RCTTouchHandler _recordNewTouches:](), /Users/simek/projects/macOSWorkbench/node_modules/react-native-macos/React/Base/RCTTouchHandler.m:111
2020-08-03 21:24:21.390136+0200 Reminders[10623:1878807] [General] Touch is already recorded. This is a critical bug.
2020-08-03 21:24:21.393359+0200 Reminders[10623:1878807] [General] (
	0   CoreFoundation                      0x00007fff39031be7 __exceptionPreprocess + 250
	1   libobjc.A.dylib                     0x00007fff71e095bf objc_exception_throw + 48
	2   CoreFoundation                      0x00007fff3905ad98 +[NSException raise:format:arguments:] + 88
	3   Foundation                          0x00007fff3b7883b6 -[NSAssertionHandler handleFailureInFunction:file:lineNumber:description:] + 166
	4   Reminders                           0x00000001001b8422 -[RCTTouchHandler _recordNewTouches:] + 578
	5   Reminders                           0x00000001001ba643 -[RCTTouchHandler interactionsBegan:] + 83
	6   Reminders                           0x00000001001ba9b7 -[RCTTouchHandler mouseDown:] + 151
	7   AppKit                              0x00007fff3645e104 -[NSWindow(NSGestureRecognizer_Routing) _sendEventToGestureRecognizers:requireAcceptsFirstMouse:] + 2696
	8   AppKit                              0x00007fff36465dc5 -[NSWindow(NSEventRouting) _handleMouseDownEvent:isDelayedEvent:] + 4839
	9   AppKit                              0x00007fff363d0611 -[NSWindow(NSEventRouting) _reallySendEvent:isDelayedEvent:] + 2612
	10  AppKit                              0x00007fff363cf9b9 -[NSWindow(NSEventRouting) sendEvent:] + 349
	11  AppKit                              0x00007fff363cdd44 -[NSApplication(NSEvent) sendEvent:] + 352
	12  AppKit                              0x00007fff3621adaf -[NSApplication run] + 707
	13  AppKit                              0x00007fff361ecb86 NSApplicationMain + 777
	14  Reminders                           0x00000001000016f2 main + 34
	15  libdyld.dylib                       0x00007fff72fb0cc9 start + 1

The source code of my test/sample app is here: https://github.com/Simek/react-native-macos-reminders


Export failed when a task(s) has sub-tasks

Exporting with tasks that have sub-tasks fails to export. There will be a little toast notification on the bottom that says "Export failed" and then another that says "Exported". The resulting exported ics file does not import any tasks.

Issue might be more complicated than I thought. It seems export is failing even with a single task in the list and a clean install

Ok tired to recreate the issue on My Intel Wayland device running the same OS(Fedora 40-kernel 6.8.9) and build (46.0.5) and I can actually export from it. My X11 Ryzen-Nvidia machine device seems to be having an issue.

Confirm this. Fixed in 0a859902acb6776535d7c33bb6f77a3dc0bb41b8. Forgot to change file write mode. I'll make release now.

Pushed update to Flathub.


MLTransform

Design doc: https://docs.google.com/document/d/1rQkSm_8tseLqDQaLohtlCGqt5pvMaP0XIpPi5UD0LCQ/edit
This PR introduces TFTProcessHandlerSchema which takes a pcoll and perform user provided data processing transforms. TFTProcessHandler uses tensorflow_transform's beam implementation for computing results.
This PR aims to provide a simple to use API which abstracts the user away from the complexities that one can find with using tensorflow transform such as providing a tensorflow feature spec etc.
Alternative design -  Alternative way to pass columns to the transforms
Instead of passing columns to the every data transform, we can let MLTransform take a param columns and assume that all the data transforms within the scope of that MLTransform are applicable only on the input columns passed.
But this could have a downside if the user calls multple MLTransforms, then there will be fusion break(due to tft internal implementation)
 with beam.Pipeline() as p:
      transforms = [ComputeAndApplyVocab(...), TFIDF(...)]
      result = (
          p
          | "Create" >> beam.Create(input_data)
          | beam.Map(lambda x: ComputeAndVocabBatchedInputType(**x)
                     ).with_output_types(ComputeAndVocabBatchedInputType)
          | "MLTransform" >>
          base.MLTransform(columns=['x'], transforms=transforms)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @tvalentyn

R: @robertwb

As discussed on the doc, let's avoid process_handler as a public part of the API or required argument.

@robertwb @damccorm I addressed most of the comments. PTAL.

Added artifact consumption vs production mode.
Made TFTProcessHandler internal to the MLTransform

retest this please

@damccorm  removed pyarrow record batch. Addressed other comments as well. PTAL
ref: https://github.com/apache/beam/issues/26640#issuecomment-1614764518

R: @rszper

R: @rszper

Actually, I am thinking I will iterate on the docs in the following PRs. Docs will include pydocs edits and doc pages as well for users


[SPARK-32310][ML][PySpark][WIP] ML params default value parity in feature and tuning

What changes were proposed in this pull request?
set params default values in trait Params for feature and tuning in both Scala and Python.
Why are the changes needed?
Make ML has the same default param values between estimator and its corresponding transformer, and also between Scala and Python.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing and modified tests

Test build #126117 has started for PR 29153 at commit 86e0579.

Merged build finished. Test PASSed.

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

Test build #126117 has finished for PR 29153 at commit 86e0579.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #126220 has started for PR 29153 at commit 1586e30.

Merged build finished. Test PASSed.

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

Test build #126220 has finished for PR 29153 at commit 1586e30.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #126265 has started for PR 29153 at commit 5e09d2d.

Merged build finished. Test PASSed.

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

Test build #126265 has finished for PR 29153 at commit 5e09d2d.

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

Merged build finished. Test PASSed.

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

Test build #126307 has started for PR 29153 at commit 46451b6.

Merged build finished. Test PASSed.

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

Test build #126307 has finished for PR 29153 at commit 46451b6.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #126343 has started for PR 29153 at commit 46451b6.

Merged build finished. Test PASSed.

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

Test build #126343 has finished for PR 29153 at commit 46451b6.

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

Merged build finished. Test PASSed.

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

cc @srowen @viirya @zhengruifeng

Looks good if there are no API or behavior changes (that are not bug fixes).

retest this please

Test build #126724 has started for PR 29153 at commit 46451b6.

Merged build finished. Test PASSed.

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

Test build #126724 has finished for PR 29153 at commit 46451b6.

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

Merged build finished. Test PASSed.

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

I think you can merge when ready

Merged to master. Thank you all for reviewing!


Add arguments to set bounds when visualising sample paths

When the number of data points get too large, bokeh gets slow, so it would be useful to have arguments to bound parent node id and/or genomic position/site index when calling create_sample_path_vis_app.
I think the following argument should suffice:

range_nodes=(min, max)
range_position=(min, max)

Default should be the full ranges.

I'll leave implementing range_site_ids=(min, max) as a separate issue.


Support literal types

Please let me know if I'm missing something here. Thanks!
Problem
I'd like to define a method that takes one of a few possible constant values. For example:
class Fruit
  extend T::Sig

  BANANA = T.let("banana", String)
  APPLE = T.let("apple", String)

  sig { params(value: T.any(BANANA, APPLE)).returns(T::Boolean) }
  def self.banana?(value)
    value == BANANA
  end
end

I'd expect to be able to do this:
Fruit.banana?("banana")

But not this:
Fruit.banana?("monster truck")

I could use an enum. But in my real example, I'm already using these string values in other places. They can be returned from my model attributes and can be provided as params from a user. I don't want to have to convert these values every time I interact with them.
Proposed solution
My suggestion would be for Sorbet to support literal values in type declarations.

You can use T::Enum for this purpose:
# typed: true
class Fruit < T::Enum
  extend T::Sig

  enums do
    BANANA = new("banana")
    APPLE = new("apple")
  end

  sig { params(value: T.any(BANANA, APPLE)).returns(T::Boolean) }
  # alternatively:
  # sig { params(value: Fruit).returns(T::Boolean) }
  def self.banana?(value)
    value == BANANA
  end
end

→ View on sorbet.run
Docs: https://sorbet.org/docs/tenum

@jez I commented above that while I could use T::Enum, I'd rather not. I personally don't like the wrapper objects provided by Sorbet, and I think Sorbet should support literals explicitly.

I'd like to express support for this as well. Literal types are incredibly useful, especially given the way symbols and strings are used in ruby.


Reading Incoming Chunks of data

Hi,
I am creating a SSL HTTP client which sends chunks of data (Transfer-Encoding: chunked) and the server sends back processed chunks of data as the server is receiving raw chunks. I want to read the processed chunks simultaneously and this communication link between the client and server is full duplex even though the HTTP protocol is not full duplex. So based on my experience with some of the boost beast http read functions, those functions (mentioned below) are designed to follow the HTTP protocol which is half duplex while my application uses a modified HTTP protocol which is full duplex.
Currently, I am able to call boost::beast::http::read(*stream_,read_buffer_,res); (blocking call) in a separate thread to read all the processed chunks all at once after receiving the HTTP/1.1 200 OK header from the server (half duplex). Is there a method in the boost beast library to read the processed chunks directly from the socket? Otherwise, how can I get the socket descriptor from boost::asio::ssl::stream<tcp::socket>* stream_; to read the socket?

First of all if you want full duplex you need to use asynchronous calls. Otherwise you will be accessing the socket concurrently, which results in undefined behavior. Second, you should use an overload of http::async_read_some which uses a parser, then you can have complete control over receiving the chunks.

Thank you for the information on the parser! I am close to testing out my custom parser class but is there an asynchronous write that I can use to replace boost::beast::net::write()? I followed chunked encoding example and I am using the boost::beast::net::write() function which requires chunk_body type as an argument for passing the message as seen here.

Did you mean beast::http::async_write?

Does calling an async_write close the http session after it writes the message? The program I am creating is intended to send multiple chunks of data in one HTTP session and I was wondering if async_write will close the HTTP connection or do I have control over ending the transmission of chunks by writing "0\r\n" to the stream using async_write()?

async_write does not close the connection. You can either send the end of the chunk manually or you can use make_chunk_last which is recommended.

I managed to use async_write with a custom parser which is derived from basic_parser.
One thread calls async_header_write, and async_write to write chunks (used boost::beast::http::make_chunk() to make chunks) while the second thread calls async_read with the custom parser to read the incoming chunks.
Thanks for the help!


modify some settings and update PreferencesManager

This is related to issue #2436 and removes double negation in some of the settings. I also grouped the keys a bit more clearly in PreferencesManager and modified a couple other strings.
I tried to slim down some of the setting titles because I think it might be better in the future to explain the settings in the description and have more concise titles. One example is the inline URL previews, the title could just be Inline URL Previews and the description could explain that they will only be shown if your homeserver enables the feature.

Is VectorPreferenceCategory used often? I was trying to figure out why the Labs category always has a bold title, turns out the custom category class sets text as bold, which I think looks a tad odd.

Also please fix errors reported by Travis

Is VectorPreferenceCategory used often? I was trying to figure out why the Labs category always has a bold title, turns out the custom category class sets text as bold, which I think looks a tad odd.

I assume VectorPreferenceCategory should be used only in vector_room_settings_preferences.xml. Feel free to make the labs section is classical PreferenceCategory.
Also I wanted to let you know that I've started to migrate to preference from v7 support library. It's not done yet, but it will be in a near future.

Signed-off-by: Dillon Kanada<EMAIL_ADDRESS>


Polymer and Mixin

Can you support polymer and CSS mixin?

VSCode Version:2017
OS Version:window 7

Steps to Reproduce:

Setup polymer project
Observe most of polymer / mixin syntax are not supported

Can you please provide a code example and explain what does not work as expected?

Closing as there is not enough info about what is going wrong


Azure web app- Need admin approval

I get the below error when a user tries to open the deployed webapp. What permissions are required? I currently have the below
needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it

I believe that you need to be Owner of the subscription otherwise RolePermissions are note created properly

It depends how your Entra ID stuff is set up. You need the permissions as follows granted:

Those potentially need to be granted by your Entra admin.

Also, if you set "Assignment Required" on your Enterprise Application, then any grants need to be handled by Admin consent.
More info is here in the docs.
https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/application-sign-in-unexpected-user-consent-prompt#view-user-assignment-settings.

Microsoft Graph
email
View users' email address

Microsoft Graph
offline_access
Maintain access to data you have given it access to

Microsoft Graph
openid
Sign users in

Microsoft Graph
profile
View users' basic profile

Microsoft Graph
User.Read
Sign in and read user profile

Microsoft Graph
User.Read.All
Read all users' full profiles

Microsoft Graph
User.ReadBasic.All
Read all users' basic profiles


Quickstart bug, since last updates.

Describe the bug
quickstart dont start chrome.
To Reproduce
Steps to reproduce the behavior:

Run quickstart.py

error:
Getting ChromeDriver ...
Traceback (most recent call last):
  File "D:\DESARROLLO\TinderBotz\quickstart.py", line 10, in <module>
    session = Session()
              ^^^^^^^^^
  File "D:\DESARROLLO\TinderBotz\tinderbotz\session.py", line 115, in __init__
    self.browser = uc.Chrome(options=options)  # ChromeDriverManager().install(),
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "d:\desarrollo\tinderbotz\src\undetected-chromedriver\undetected_chromedriver\__init__.py", line 258, in __init__
    self.patcher.auto()
  File "d:\desarrollo\tinderbotz\src\undetected-chromedriver\undetected_chromedriver\patcher.py", line 178, in auto
    self.unzip_package(self.fetch_package())
                       ^^^^^^^^^^^^^^^^^^^^
  File "d:\desarrollo\tinderbotz\src\undetected-chromedriver\undetected_chromedriver\patcher.py", line 287, in fetch_package
    return urlretrieve(download_url)[0]
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 241, in urlretrieve
    with contextlib.closing(urlopen(url, data)) as fp:
                            ^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 525, in open
    response = meth(req, response)
               ^^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 634, in http_response
    response = self.parent.error(
               ^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 563, in error
    return self._call_chain(*args)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 496, in _call_chain
    result = func(*args)
             ^^^^^^^^^^^
  File "C:\Python311\Lib\urllib\request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found
/==============\
| Tinderbotz   |
| ----------   |
| duration: 0  |
| like: 0      |
| dislike: 0   |
| superlike: 0 |
\==============/
Exception ignored in atexit callback: <function Session.__init__.<locals>.cleanup at 0x00000236C52604A0>
Traceback (most recent call last):
  File "D:\DESARROLLO\TinderBotz\tinderbotz\session.py", line 69, in cleanup
    print("Started session: {}".format(self.started))
                                       ^^^^^^^^^^^^
AttributeError: 'Session' object has no attribute 'started'

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: W10
Browser: Chome
Version: 121.0.6167.185

Additional context
Add any other context about the problem here.

Seems like the problem is due to a changed chromedriver url. Try to change
download_url = "https://edgedl.me.gvt1.com/edgedl/chrome/chrome-for-testing/%s/%s/%s"
into
download_url = "https://storage.googleapis.com/chrome-for-testing-public/%s/%s/%s"
in "patcher.py" of the undetected chromedriver package (Lib\site-packages\undetected_chromedriver)
Then it should work again. Hopefully this will be fixed in official undetected chromedriver package.


Bump marked because of security advisory

An npm sercutiry advisory recommends bumping the version of
marked to v0.7.0 or later. Because it's a v0 release,
npm will only install v0.6.* versions which do not have
the fix.
https://www.npmjs.com/advisories/1076

Thanks for letting me know. Released<EMAIL_ADDRESS>with updated dependencies.


Update add-label.js

Ability to rotate text

This would be legit. Could potentially work well with clusterLabelPos to allow for 'left' and 'right' positioning with rotation support?


support for additional scipy nd interpolants

[x] Closes #7704
[x] Adds support for n dimensional tensor product interpolants slinear, cubic, quintic, and pchip.
[x] Updates documentation of DataArray/Dataset.interp and DataArray/Dataset.interp_like to reflect updates to scipy interpolants called.
[x] Adds argument reduce to allow disabling of reduction of interpolation along independent dimensions (see #2223).  Prior to this the behavior was always reduce=True. However for certain n-dimensional interpolants this will change the mathematical behavior of the interpolation (I think), so I have added an option for users to turn this off.

Outstanding uncertainty: dask/chunking behavior? Nothing about this PR touches the chunking of interpolation but I am a bit worried about this. Many interpolants, including those previously supported in one dimension, require nearby/all points from the original coordinate, and also don't parallelize well over the new coordinate. I'm not quite sure how this is currently being handled.

Thanks for this contribution.
Some remarks:
What is the minimal scipy version that is required for these new interpolators? Is it aligned with the minimum version for xarray or do we need some checks for that?
You have added several formatting changes to totally unrelated parts that make it more difficult to review then necessary. Please try to avoid that.
Doc build failed, have to check if random or related to your docstrings.

Thanks for the comments!

What is the minimal scipy version that is required for these new interpolators? Is it aligned with the minimum version for xarray or do we need some checks for that?

I'm seeing 1.11 in ci/requirements/min-all-deps.yml and these interpolants were added in 1.10 so I think we're good.

You have added several formatting changes to totally unrelated parts that make it more difficult to review then necessary. Please try to avoid that.

Oops, reverted, sorry!

Also, I think the name "reduce" is not very intuitive, but I fail to come up with an alternative. "decompose" seems weird out of context. Does anyone have a suggestion?

Also, I think the name "reduce" is not very intuitive, but I fail to come up with an alternative. "decompose" seems weird out of context. Does anyone have a suggestion?

Perhaps the _interp makes decompose_interp sound less weird? Separate or split also sound OK to me.

Thanks @hollymandel this is looking quite close. I only have a few small refinement requests.

I think the failed test in ubuntu-latest py3.12 flaky is unrelated to this PR. I've rerun a few times and it keeps happening though, not sure how to move forward.

that issue should be fixed by #9709, so the merge appears to have fixed that issue here, as well.


removed keyboard control screen from web chat

Fixes #4965

Changelog Entry
Fixes #4965. Removed keyboard helper screen in #5224, by @amirmursal
Description
Removed keyboard control screen from the chat window to address several accessibility issues

[x] I have added tests and executed them locally
[x] I have updated CHANGELOG.md
[ ] I have updated documentation

Review Checklist

This section is for contributors to review your work.

[x] Accessibility reviewed (tab order, content readability, alt text, color contrast)
[ ] Browser and platform compatibilities reviewed
[ ] CSS styles reviewed (minimal rules, no z-index)
[ ] Documents reviewed (docs, samples, live demo)
[ ] Internationalization reviewed (strings, unit formatting)
[ ] package.json and package-lock.json reviewed
[ ] Security reviewed (no data URIs, check for nonce leak)
[ ] Tests reviewed (coverage, legitimacy)

Closing this PR due to irrelative test case failure is observed


Support for basic auth

Hello there and thank you for this really helpfull lib!
Is there a way of setting the basic auth through the api?
Is it planed to support basic auth authorization?

Setting the header information through the api does not work:

[...]
graphQLSchema('./swagger.json').then(schema => {
    app.use('/graphql', graphqlHTTP((req) => {
        req.headers['Authorization'] = 'Basic: YWRkOmJhc2ljQXV0aA==';
        req.headers['X-Custom'] = 'custom-payload-xyz';
        return {
            schema,
[...]

Setting the header in the resolver function does work:
https://github.com/yarax/swagger-to-graphql/blob/master/lib/index.js#L80

[...]
var resolver = function resolver(endpoint) {
  return function () {
    var _ref = (0, _asyncToGenerator3.default)(_regenerator2.default.mark(function _callee(_, args, opts) {
      var req, res;
      return _regenerator2.default.wrap(function _callee$(_context) {
        while (1) {
          switch (_context.prev = _context.next) {
            case 0:
              req = endpoint.request(args, opts.GQLProxyBaseUrl);

              req.headers['Authorization'] = 'Basic: YWRkOmJhc2ljQXV0aA==';
              if (opts.BearerToken) {
                req.headers.Authorization = opts.BearerToken;
              }
              _context.next = 4;
              return (0, _requestPromise2.default)(req);
[...]

Does this cover NTLM?

@fizxmike never tested on a MS-IIS, but i don't think it would work either. Still, maybe you could implement a layer between thos using https://www.npmjs.com/package/express-ntlm
What do you think?

the ability to pull a Bearer Token off the header of a request for an authenticated endpoint. Which is essentially no different than your typical GraphQL resolver that taps into context.req.get('authorization')

here the Bearer Token example has been removed altogether


Build Failed on windows 10  error GCE1592F9:

error GCE1592F9: 'Size' is imported from both 'dart:ffi' and 'dart:ui'.
When running with out the bitsdojo package app runs but when using this package this error occurs .

This happens because of Flutter 3.0. This package needs to get updated for Flutter 3.0
Judging by the last updates and number of issues/pull-requests it seems that this package is abandoned so don't get your hopes high.

it seems that this package is abandoned.

For everyone else reading this and wondering about it:
The package is not abandoned in any way.
Been busy with taking care of my health as it has greatly impacted my ability to work in the last months.
I am slowly recovering so updates are coming in the near future.

https://github.com/flutter/flutter/issues/103571#issuecomment-1125705625:

Looks like we added Size to dart:ffi and anyone who imports both dart:ffi and dart:ui is going to have an issue. Specifically looks like this is bitsdojo_window_windows and bitsdojo_window_linux.
The solution is to say hide Size on the import 'dart:ffi' line.

Fixed in bitsdojo_window: ^0.1.2 - https://github.com/bitsdojo/bitsdojo_window/commit/f320b623db107b03a5976fd88e52ec4fe81fa0c9
Closing this issue. Please re-open if needed.


Could a pre-compiled binary be made available?

For those of us that don't want to modify it, but "just drop it in and go", it would be nice to have something that is already known to work with a standalone programmer.
I can export a hex file from the Arduino IDE, but it only includes the Flash, not the Fuses.  I guess that's a start, but the Fuses are important too!  How to get those?  Read the datasheet and build them up manually from the screenshot of the author's (old now) Arduino IDE?
Why I think I need this:
I rearranged the schematic and PCB from here, to make it a hat for a Raspberry Pi that uses the Pi's GPIO UART, driven from pymcuprog with its recommended schematic for UPDI.  ( https://pypi.org/project/pymcuprog/ and https://github.com/microchip-pic-avr-tools/pymcuprog/blob/main/help.md and https://community.platformio.org/t/pymcuprog-cant-use-my-uart/40104/6 )  A set of jumpers allows the same UART to program the onboard ATtiny202 instead of an external target.  As far as the ATtiny202 is concerned, it does the exact same function with the exact same code.
Of course, HV UPDI isn't available for the onboard chip, so I need to make absolutely sure that I don't brick it while still setting the rest of the Fuses to make the key work.  But I don't see a UPDI setting in the Arduino IDE anymore, which makes me nervous.  Did it get dropped?  And does the new default setting brick it by accidentally requiring HV?
So, with pymcuprog by itself, not the Arduino IDE, how to make the connection between the documentation here and a completely programmed chip?  I think a complete set of binary data would do that, and it appears from the middle link above that using pymcuprog by itself to read a working key would produce that data in a single hex file.

I can export a hex file from the Arduino IDE, but it only includes the Flash, not the Fuses. I guess that's a start, but the Fuses are important too! How to get those? Read the datasheet and build them up manually from the screenshot of the author's (old now) Arduino IDE?

Well, that's pretty much what I did:

Read the blank chip twice, once to a hex file and once to the terminal, because I needed the one and the other is much easier to read.
Matched the sizes of the different regions, and put some temporary labels in the hex file just to keep myself straight.
Manually overwrote the Fuses in the easy-to-read version, between the old screenshot here and the datasheet, making ABSOLUTELY sure that the UPDI setting was correct.
Translated the easier-to-read Fuse values to their section of the hex file that was read from the blank chip, and fixed the checksum to be valid with the new data.
Replaced the Flash section with what came from the Arduino IDE.
Removed my temporary labels to make it a valid hex file again.
Wrote that new hex file to the chip.

And it seems to work!
Here's the contents of that new hex file.  I make no guarantee that it's error-free, but it's what I'm using so far.
:020000040000FA
:1000000019C034C033C032C031C030C02FC02EC080
:100010002DC01BC12BC02AC029C028C027C026C0A4
:1000200025C024C023C022C021C020C01FC01EC0C4
:100030001DC01CC011241FBECFEFCDBFDFE3DEBF4C
:100040008091400080934000811105C098ED21E02F
:1000500094BF209341008CBB2FE3A0E8BFE301C015
:100060001D92A039B207E1F721D1D6C1C9CFE0E096
:10007000F4E080E8858383EC99E00197F1F700C014
:10008000000082E0858394E79A95F1F700C08683AB
:100090002FEC8EE791E0215080409040E1F700C0C6
:1000A00000000895E0E0F4E088E08583818382E445
:1000B0008A95F1F700C0089598E090930604E7EF61
:1000C000F0E03197F1F700000000000000000000B0
:1000D0000000000000000000000000000000000020
:1000E00048E0282F30E080FF50C04093050435954C
:1000F0002795822FE7EFF0E03197F1F7000000003D
:1001000000000000000000000000915051F788E05E
:100110006623F1F18093050487EF90E00197F1F7F2
:1001200000000000000000000000000000000000CF
:100130000000000000000000000088E0809305043B
:10014000E7EFF0E03197F1F7000000000000000059
:10015000000000000000000000000000000000009F
:1001600000008093050487EF90E00197F1F700000D
:10017000000000000000000000000000000000007F
:100180000000000000000000089540930604AFCF77
:1001900080930604E7EFF0E03197F1F7C1CF1F93AA
:1001A000CF93DF93C0E0D4E018E019831D831E8352
:1001B00000000000000000008B89877F8B8B1A8372
:1001C00085EA9EE00197F1F700C000006BDF60E078
:1001D00085E572DF61E080EE6FDF61E080E26CDF79
:1001E00061E087E669DF60E08FE666DF60E082E776
:1001F00063DF60E080E560DF60E08DE45DDF60E0AC
:1002000086E55ADF60E08EE457DF4CDF1A838B8986
:10021000877F8B8B85EA9EE00197F1F700C0000095
:1002200041DF60E085E548DF3DDF1A838B89877F0A
:100230008B8B85EA9EE00197F1F700C0000080E813
:100240008E83DF91CF911F9108958F9382E08093E9
:100250000B0A8FB78F93EF93FF93E0E8FFE39F9331
:1002600068948FEF0FC0E8948481958188589D4FE2
:1002700084839583885E934010F08FEF03C084835E
:1002800095838EEF9081981B919390819F4F9193CE
:1002900090819F4F919390819F4F919326F39F91CF
:1002A000FF91EF918F918FBF8F91189581E098ED1D
:1002B00094BF80936100E0E0F6E092E592839DE0D8
:1002C000958390E293838083809302028093030A54
:1002D0008EEF8093260A8093270A8BE08093000A92
:1002E00092E090930A0A789484E0809302048091CB
:1002F000120488608093120488E08093020480E4F2
:1003000080930104809305049093010480E8809316
:100310000104ADDEF89443DF7894D3E0C0E400E05C
:1003200010E4E12C88E0F82EDD24D39480910804B9
:1003300082FF4DC080918F3F8823C1F010928F3F84
:10034000C09306042FE78AE196E021508040904058
:10035000E1F700C00000C093050489DEF8941FDFB8
:10036000789422E42A95F1F700C0E0CF80918D3F88
:1003700090918E3F8F3F2FE3920728F40196809350
:100380008D3F90938E3F10928B3F10928C3F8091C7
:100390008B3F90918C3F8F3F974041F4E0928B3F91
:1003A000F0928C3FD0928F3F10928A3F80918A3F8B
:1003B00020918D3F30918E3F2F3F3F43D1F40093EA
:1003C0008D3F10938E3F8F5F80938A3FCACF80917D
:1003D0008B3F90918C3F8F3F27E0920728F4019646
:1003E00080938B3F90938C3F10928D3F10928E3F65
:1003F000CECF823009F0B5CFD0938A3FC0930604A8
:100400002FE78AE196E0215080409040E1F700C05C
:0C0410000000C0930504A5CFF894FFCFB6
:02000004008278
:09000000004002FF00F7030000BC
:00000001FF

And the command to write it to the chip.
$ pymcuprog write --erase --verify -d attiny202 -t uart -u /dev/ttyAMA0 -f UpdiKey.hex
Connecting to SerialUPDI
Pinging device...
Ping response: 1E9123
Erasing device before writing from hex file...
Writing from hex file...
Writing flash...
Verifying flash...
OK
Writing fuses...
Verifying fuses...
OK
Done.
$

Hopefully this helps someone.  I'm not going to close this issue yet, because I'm not completely confident that it's an exact image of what the author intended, but like I said, it does seem to work.


Form post request

wasn't sure if this is how you're meant to resolve nits but I just added it in as an extra commit here

when I replace "false" with false the getParameter function stops working - I took the function as is from one of the examples

when I replace "false" with false the getParameter function stops working - I took the function as is from one of the examples

Oh ok I got it now. It's because parseBoolean accepts a String so that's actually correct. My bad!


General Question

Hello!
I just had a quick question about your tool, I know that it uses the omic data but is it at all possible to align images without this?

Hello. Our tool only aligns based on -omics data. We don't use the images in any part of our alignment, but I believe there are other methods out there that do image-based alignment.


Update Scholarsphere 3 UMG group names

Description
The names of the UMG groups that were migrated over from SS3 look like: umg/name.of.group. However,  the current format of the group names that are appearing SS4 look like: umg-name.of.group. As a result, group permissions that were migrated over from SS3 are not working correctly in SS4 because of the name formatting difference.
WDLL

verify that replacing the slash with a dash will fix this for all migrated groups
create new groups in Scholarphere that reflect the changed name
move the existing access controls from the old group names to the new ones
delete the old groups
OPTIONAL: delete any group that doesn't exist anymore

Reporting
There are a total of 14 groups with the old format names. Four of these show the presence of new updated names:

umg-psu.prc.prosper_partners
umg-up.bmb.keilerlab
umg-up.emsl.staff
umg-up.rcr.mellon_project

This means that since version 4 launched, users from these four groups have logged into Scholarsphere and their groups were entered into the system using the new format.

This is fixed:
Group.where("name ILIKE '%umg/%'").map do |old_group|
  print "Updating #{old_group.name} ..."
  access_controls = AccessControl.where(agent: old_group)

  if access_controls.empty?
    puts "no acls"
    next
  end

  new_group = Group.find_or_create_by(name: old_group.name.gsub("/", "-"))
  access_controls.update_all(agent_id: new_group.id)
  puts "updated!"
end

And then:
Group.where("name ILIKE '%umg/%'")

All of the old groups had acls, so only the old group names were deleted.


Add 'data-nosnippet' to Blazor error UI to curtail robot indexing

Per https://github.com/dotnet/aspnetcore/issues/50840

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

ID: adbc1392-92dc-c42e-ba48-a722cffa9b6d
Version Independent ID: 66a0b1c2-c45c-98ca-9808-6f340a861c44
Content: Handle errors in ASP.NET Core Blazor apps
Content Source: aspnetcore/blazor/fundamentals/handle-errors.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@guardrex do you need someone to take on this issue?

Sure, @abc516 ... Place it as the last example of the Detailed errors during development section, and make the example ...

```html
<div id="blazor-error-ui" data-nosnippet>
```

We're not going to do this after all. It's only supported by Google, and information on it is readily available via their documentation.

Actually! They're doing it ...
https://github.com/dotnet/aspnetcore/pull/51543
It will be included in project templates for .NET 9 or later, but I'm going to add it to the few spots we have in docs now to avoid creating more version hell 🔥 in the docs.


build(deps): update publish action

:tada: This PR is included in version 3.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: