header encoding problem

Add the following e-mail address to a message (the utf-8 char is important!):
"Berntsen, Pål"<EMAIL_ADDRESS>Swift will create the following header string:
To: Berntsen, =?utf-8?Q?P=C3=A5l?=<EMAIL_ADDRESS>Now if you send this with postfix it will rewrite this header to:
To: Berntsen@Intermesh-1, =?utf-8?Q?P=C3=A5l?=<EMAIL_ADDRESS>The problem lies in the comma and the separated encoded string. I fixed this by always encode the entire name when encoding is needed in any part of the string.
I changed this function in Mime/Headers/AbstractHeaders.php:
protected function getEncodableWordTokens($string)
{
$tokens = array();
$encodedToken = '';
//Split at all whitespace boundaries

//Fix: don't split the string for this bug.
//foreach (preg_split('~(?=[\t ])~', $string) as $token)
	$token=$string;
{
  if ($this->tokenNeedsEncoding($token))
  {
    $encodedToken .= $token;
  }
  else
  {
    if (strlen($encodedToken) > 0)
    {
      $tokens[] = $encodedToken;
      $encodedToken = '';
    }
    $tokens[] = $token;
  }
}
if (strlen($encodedToken))
{
  $tokens[] = $encodedToken;
}

return $tokens;

}
I hope you'll fix this problem too!
Original creation date: 2010-06-14T13:47:31Z
Original reporter: aleksblendwerk
Original ticket: http://swiftmailer.lighthouseapp.com/projects/21527/tickets/150

Same problem here :-/
Is there any fix available, pleeease?

We came across this issue and have discovered that in the case where there is a comma in the sender name, the AbstractHeader::createPhrase quotes the name. However if there is both a comma and some accented chars after the comma the name is not quoted. So by always returning a quoted sender name regardless of the conditions we seem to have compatibility with various mail clients.
AbstractHeader.php
245 -  $phraseStr = '"' . $phraseStr . '"';
262 + return '"' . $phraseStr . '"';

Came across this issue again, after switching to RFC 2047. This time when using @ in the display name of From header.
RFC 2047

Other characters are also illegal in certain contexts.  For example, an 'encoded-word' in a  'phrase' preceding an address in a From header field may not contain any of the "specials" defined in RFC 822.

RFC 822

        specials    =  "(" / ")" / "<" / ">" / "@"  ; Must be in quoted-
                    /  "," / ";" / ":" / "\" / <">  ;  string, to use
                    /  "." / "[" / "]"              ;  within a word.

Seems to me like any specials characters in the UTF-8 encoded string need to be stripped out? Or have I misinterpreted that?


fix(docs): add curl flag

Add location flag to docker.

Could you elaborate on the why?

@thibaultcha:
@john-paul-keenan had noticed an error when using the curl command WITHOUT -L:
Empty reply from server

I updated docke (now using Version 1.13.1 (15353)) and the command works without the -L flag. BUT, it also works with it so I think this is still good for some of our users


#1085: Fixing bug after control some undefined values

I have controlled some undefined values after debugging when I found the bug, and now is fixed!

@dailymp Though I still can not understand the root cause of this issue, but I will add a short circuit to avoid this error.  thanks for your contribution.


Figure out how pub serve will handle multiple packages in the Editor

The Editor's built in dev server supports simultaneously serving up resources from multiple packages that may be in the user's workspace. Pub serve (like most pub commands) is specific to a single package.
Before we can make pub serve the canonical web server in the Editor, we need to figure out how we should handle this. The two options I see are:

Have the editor maintain a separate running pub serve process for each package that the user is working on in their workspace.

Have pub serve support serving multiple packages simultaneously.

We don't need to hash this out in the bug comments here, but I wanted to file a tracking issue so that we know this needs to be done.

This issue has been moved to dart-lang/pub#807.


Remove "new" calls to Ember.A()

addon/components/sl-calendar.js
tests/dummy/app/controllers/browsers.js
tests/unit/components/sl-grid-test.js
tests/unit/components/sl-menu-test.js

UPDATE: Remove this from all of the dummy application code as well.

https://github.com/joshforisha/sl-eslint/issues/9 plays into this

Why?

Because Ember.Array is not an object to be made an instance of.  It is a method that returns an array.
http://emberjs.com/api/classes/Ember.html#method_A
https://github.com/emberjs/ember.js/blob/v2.0.1/packages/ember-runtime/lib/system/native_array.js#L137-L140

@notmessenger please assign to me, working on this.

@azizpunjani Also remove this from all of the dummy application code (there are many instances of this).  This was not part of the original task description but has been identified as needing to be done as well.

@notmessenger this has been done, we are now getting errors via eslint after removing "new" https://travis-ci.org/softlayer/sl-ember-components/jobs/78290883#L545. How should we proceed?

@azizpunjani I have PR #742 that should take care of that.

@notmessenger once #742 is merged in, I will merge it into the branch I created and test again to make sure eslint errors are not thrown anymore.


Positional encoding fails with rectangular input

Hi!
I stumbled on the same issue as interpolate_pos_encoding(x, pos_embed) doesnt return correct dimension for images that is not square (w != h) when using dinov2, the code crashed on the same function when using rectangular input...
Here is the function from vision_transformer.py
def interpolate_pos_encoding(self, x, w, h):
        previous_dtype = x.dtype
        npatch = x.shape[1] - 1
        N = self.pos_embed.shape[1] - 1
        if npatch == N and w == h:
            return self.pos_embed
        pos_embed = self.pos_embed.float()
        class_pos_embed = pos_embed[:, 0]
        patch_pos_embed = pos_embed[:, 1:]
        dim = x.shape[-1]
        w0 = w // self.patch_size
        h0 = h // self.patch_size
        print(f'DEBUG dinov2 vision_trasnformer.py: w0={w0}, h0={h0}')
        # we add a small number to avoid floating point error in the interpolation
        # see discussion at https://github.com/facebookresearch/dino/issues/8
        w0, h0 = w0 + self.interpolate_offset, h0 + self.interpolate_offset
        print(f'DEBUG dinov2 vision_trasnformer.py: add small number w0={w0}, h0={h0}')

        sqrt_N = math.sqrt(N)
        sx, sy = float(w0) / sqrt_N, float(h0) / sqrt_N
        patch_pos_embed = nn.functional.interpolate(
            patch_pos_embed.reshape(1, int(sqrt_N), int(sqrt_N), dim).permute(0, 3, 1, 2),
            scale_factor=(sx, sy),
            mode="bicubic",
            antialias=self.interpolate_antialias,
        )
        print(f'DEBUG dinov2 vision_trasnformer.py: patch_pos_embed.shape={patch_pos_embed.shape}')

        assert int(w0) == patch_pos_embed.shape[-2]
        assert int(h0) == patch_pos_embed.shape[-1]
        patch_pos_embed = patch_pos_embed.permute(0, 2, 3, 1).view(1, -1, dim)
        return torch.cat((class_pos_embed.unsqueeze(0), patch_pos_embed), dim=1).to(previous_dtype)

I checked the output by feeding a rectangular image and found out the small addition did not change the w and h,  see the output:
 image_crop.shape=torch.Size([1, 3, 434, 546])
DEBUG dinov2 vision_trasnformer.py: w0=31, h0=39
DEBUG dinov2 vision_trasnformer.py: add small number w0=31.0, h0=39.0
DEBUG dinov2 vision_trasnformer.py: patch_pos_embed.shape=torch.Size([1, 384, 31, 38])

Note that in the init, interpolate_offset=0.1.
Here are the errors I got:
File "home/.cache/torch/hub/facebookresearch_dinov2_main/dinov2/models/vision_transformer.py", line 204, in interpolate_pos_encoding
    assert int(w0) == patch_pos_embed.shape[-2]
AssertionError

Note: used pretrained dinov2_vits14_reg model.
Could it be that those pretrained checkpoints have no field interpolate_offset, or it is set to 0?
Meanwhile, I will manually set it to 0.1 and see if it helps.

Update: adding manually 0.1 instead of self.interpolate_offset solved the issue. This means the pretrained checkpoint has it set to 0.0, I assume.

Is there a reason the interpolate call doesn't set the output size directly to (w0,h0) using the size parameter, rather than using the scale_factor parameter?

Update: adding manually 0.1 instead of self.interpolate_offset solved the issue. This means the pretrained checkpoint has it set to 0.0, I assume.

I'm facing the same issue. It only fails for certain widths/length: 546, 602, 1092, ...

This should hopefully be fixed now with #378.

Hi @patricklabatut, I am trying to train on a rectangular image and I get the following assertion error even after the merged PR.
https://github.com/facebookresearch/dinov2/blob/e1277af2ba9496fbadf7aec6eba56e8d882d1e35/dinov2/models/vision_transformer.py#L192
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/zsuri/miniconda3/envs/dinov2/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/zsuri/dinov2/./dinov2/models/vision_transformer.py", line 325, in forward
    ret = self.forward_features(*args, **kwargs)
  File "/home/zsuri/dinov2/./dinov2/models/vision_transformer.py", line 258, in forward_features
    x = self.prepare_tokens_with_masks(x, masks)
  File "/home/zsuri/dinov2/./dinov2/models/vision_transformer.py", line 220, in prepare_tokens_with_masks
    x = x + self.interpolate_pos_encoding(x, w, h)
  File "/home/zsuri/dinov2/./dinov2/models/vision_transformer.py", line 192, in interpolate_pos_encoding
    assert N == M * M
AssertionError

It can be replicated as follows

model = SSLMetaArch(setup(args)))
model.teacher.backbone(torch.zeros((1,3,1078, 1918))

with the default vitl14.yaml config.
Also, I don't understand the intuition behind these lines https://github.com/facebookresearch/dinov2/blob/e1277af2ba9496fbadf7aec6eba56e8d882d1e35/dinov2/models/vision_transformer.py#L185-L211 for rectangular images. I tested with returning https://github.com/facebookresearch/dinov2/blob/e1277af2ba9496fbadf7aec6eba56e8d882d1e35/dinov2/models/vision_transformer.py#L184 even when h != w and it seems to run. I'm not sure if that produces any unwanted outcomes though.


 The one on the Docker Hub uses an old version of youtube-dl.
If I clone and build the container myself it works fine, but it'd be nice to just have a working one on Docker Hub.
Thanks,
Zak

Done

Thanks <3


test: test on windows

Codecov Report

Merging #20 (74e4693) into main (5ac597c) will decrease coverage by 0.42%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
- Coverage   97.81%   97.39%   -0.43%
==========================================
  Files          55       55
  Lines        1651     1653       +2
  Branches      211      146      -65
==========================================
- Hits         1615     1610       -5
- Misses         18       25       +7
  Partials       18       18

Impacted Files
Coverage Δ

tests/docs/test_textual.py
100.00% <100.00%> (ø)

docs/src/textual/demo.py
68.00% <0.00%> (-28.00%)
:arrow_down:


[EPIC] WIP: Support open-source projects

Background
The PostHog platform wouldn't be possible without open-source software. We are standing on the shoulders of giants. As such, we feel it's important to support the software we benefit from.
We are open-source and should support others who have similar values.
Proposals
1. Enable our team to donate to open-source projects
Status: ✅ Complete
We use open-source on a daily basis. We use some software more than others on a month-by-month basis. Sometimes a single piece of software may save us hours/weeks/months of time and we want to show the contributors to that project some ❤
Therefore, we should allow our team to make donations to projects on a monthly basis to recognise our open-source champions for that month.
Pull request https://github.com/PostHog/posthog.com/pull/1953
2. Continuously contribute to/support key open-source projects
Status: 🕙  see https://github.com/PostHog/posthog.com/pull/2077
There are some pieces of software that PostHog would not exist without. We should setup a continuous monthly donation for those projects.
Collecting a list of key projects via https://github.com/PostHog/meta/issues/23
3. Support ClickHouse
Status: 🕙  see https://github.com/PostHog/meta/issues/27
ClickHouse has become fundamentally important to PostHog. So much so that we're hiring Senior C++/Clickhouse Engineer.
We should investigate the other ways that we can support ClickHouse and its community.

and 2. complete. ClickHouse now has $300M investment so I don't think we can help with funding there 😊


GPI residues

We have
https://www.pombase.org/term/MOD:00818
I wanted to check this using
https://services.healthtech.dtu.dk/services/NetGPI-1.1/
but I haven't been able to run the search to completion.
Probably need to do smaller chunks.
I also found
https://mendel.imp.ac.at/gpi/gpi_server.html
We only need to run these once because they are  not going to change.
I want to put them as "target proteins" in a GPI model

I wanted to check this using
https://www.pombase.org/term/MOD:00818
but I haven't been able to run the search to completion.
Probably need to do smaller chunks.

Is that the wrong link?

yep, fixed

I ran the list of proteins through this https://services.healthtech.dtu.dk/services/NetGPI-1.1/ and obtained these results
output_protein_type.txt.
So out of the 36 proteins in the list, 8 are predicted as not GPI-anchored : mug191, cbm1, dfg501, pho1, mde5, eng1, gas2 and SPAC212.08c (although the product of this last gene is "S. pombe specific GPI anchored protein family 1" in PomBase, so if it is a pombe specific family, maybe the tool is unsuitable to predict it ?)

we could try this too
https://mendel.imp.ac.at/gpi/gpi_server.html
I am guessing there will be  often be false negatives   ...

It would be good to run all of these through
https://www.pombase.org/results/from/id/561e4908-db55-4c8b-b5f1-1d6a65c41581
to check that we are not missing any

In the end, I think we can remove the unsupported ones (we might keep the ones predicted by Groot).
Those which are ISS with a PomBase ref we can delete (in some cases it is possible that I was getting confused between a GPI anchor and N-glycosylation!)
Once we decide which to include (we can include both methods) we can add to the modification file
pombe-embl/supporting_files/legacy_modifications_from_contigs.tsv
with the publication for the method as the reference.

Taking the list of 36 genes through this tool https://mendel.imp.ac.at/gpi/gpi_server.html gave different predictions compared to the first one (the second seems more stringent, returning more negatives).
Here is a table recapitulating the predictions with both tools (for the second one I tried both Metazoa and Protozoa predictors, since pombe is neither of these) and the supporting evidence for the annotation on PomBase.
GPIanchored_genes.txt

I wanted to check this using
https://services.healthtech.dtu.dk/services/NetGPI-1.1/

I tried submitting 500 proteins at once to see if we could submit all the proteins in batches of that size.  It timed-out after 10 hours so I don't think that's going to work.
If we could download the software we could run it in the background but I couldn't find a way to do that.

OK, this is a bit of a pain.
We could try to run this  reduced subset through both
https://www.pombase.org/results/from/id/561e4908-db55-4c8b-b5f1-1d6a65c41581
If this doesn't work let's just use the ones we already had annotated that are supported by  Groot or  confirmed by either server
if we use NetGPI
https://www.sciencedirect.com/science/article/pii/S2590262821000010?via%3Dihub
it isn't PubMed indexed, so we would still need to write a PomBAse reference  linking to this paper?

Well, I got it to work with the whole 1259 set of proteins using the short output option, here is the result file.
output_protein_type-2.txt
And here is a file recapitulating the results on this protein set with the 2 tools.
GPIanchored_full_list.txt

And here is the same file as a table with the proteins predicted as GPI-anchored highlighted
GPIanchored_full_list.xls

Great, quite a mixture of results.
We can include all the ones from all. sources. Then  we will look more closely at the ones only from the first server (they  may be correct since   the other server does not appear to be trained on yeast). We might be able to establish if they are likely to be true...some seem a little strange but not impossible.
I am leaning towards including all the positive ones.
We can discuss when we next chat...

Because this publication method  not have a PMID
you need to create a PomABse reference to point to this paper/method
This is under svn in
Repos/pombe-embl/supporting_files/PB_references.txt

Create a modification style file from this list:
https://www.pombase.org/results/from/id/5ea7392e-283e-49db-b974-c86e84bc4450
plus
yam8 (I updated the product of this  to "regulatory subunit" (it's associated with cch1 channel, and clearly has extracellular domains from AlphaFold structure.
(I don't see  any of the others likely being GPI)

I think it is better to create a new file and check into the directory:
pombe-embl/external_data/modification_files/modification_files/
The file format is documented here:
https://www.pombase.org/documentation/modification-data-bulk-upload-format
Just shout know if you have any questions  about svn or anything

It all seems pretty clear to me. I'm just wondering about the Evidence code to use in the modification file. I guess it should be ECO:0008024 neural network method evidence used in manual assertion ?

That sounds good to me!

Okay then, here is the modification file if you want to have a look at it
PB_REF_0000007_modifications.tsv.txt
I added a '.txt' extension since Github wouldn't let me send it with the '.tsv'

Looks good.
v

OK, I added it to the modification files on SVN

Thanks Pascal.
I've added an empty Residue and and empty Extension column to the new file because they are needed by the modification file parser.
I've also fixed the load script because it was only looking for modification data file names that started with "PMID_".  It now looks for any file in the pombe-embl/external_data/modification_files/ directory that ends with a ".tsv"
I didn't make these changes in time for the Thursday night load so we'll need to check on Friday morning.

Good point.  I've made the documentation more clear.  The change will be on the main site in the morning.

I had forgotten to go check this one, it seems all good to me https://www.pombase.org/reference/PB_REF:0000007.
Should we close this issue ?

I think do, any other issues can go in a new ticket.


WIP: Reduce operator's controller logs via OperatorConfig.LogLevel

WIP

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 34 (32.35%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 25.478%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/daemon/daemon.go
0
1
0.0%

controllers/sriovnetworknodepolicy_controller.go
2
24
8.33%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovnetworknodepolicy_controller.go
1
11.56%

pkg/apply/apply.go
2
67.21%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
2333

Relevant Lines:
9157

💛  - Coveralls


User Authentication

Is your feature request related to a problem? Please describe.
Yes, kind of
Describe the solution you'd like
User Authentication into Gaps.
Describe alternatives you've considered
Not many alternatives
Additional context
I find it an issue to leave Gaps running when anyone can access the page and get my Plex Token and TMDB API key.

Try testing out the ssl or no ssl version to use the login. It auto-generates a unique password in the log file. Username is 'user'.
docker pull housewrecker/gaps:v0.4.0-SNAPSHOT-no-ssl
docker pull housewrecker/gaps:v0.4.0-SNAPSHOT

Enabled in v0.4.0, set a flag in docker run ENABLE_LOGIN=true

When login is enabled, Radarr is unable to connect to the RSS list. Is there a way to have authentication enabled and maintain Radarr's ability to connect to the list?

What version are you using? Are you running Docker? I haven't seen this before so it would be a new bug.

I am on version 0.8.10 in Docker.
Here's what happens if you would like to reproduce the issue. When I try to navigate to my RSS URL (http:///rss//1) when ENABLE_LOGIN is set to true, it redirects me to the login page. As a workaround, I set this to false, let Radarr update the list, then set it back to True once a week however if I can leave the login enabled and have Radarr still connect, that would of course be ideal.

That makes sense. I'm pretty sure I know why that is happening. I'll take a look into it and keep you posted. Thanks for the bug report.

Thank you! Just out of curiosity, should I make a new bug report for this since this one we're commenting on is marked as closed?

Yes, please.


The CoTTA code is not working properly.

Thank you for creating a great benchmark.
I have one question on the implementation.
python3 run_exp.py --model_adaptation_method cotta
When I try to use CoTTA with this parameter, I get the following error.
TypeError: __init__() got an unexpected keyword argument 'resample'
Line 125 in cotta.py,
transforms.RandomAffine(
    degrees=[-8, 8] if soft else [-15, 15],
    translate=(1 / 16, 1 / 16),
    scale=(0.95, 1.05) if soft else (0.9, 1.1),
    shear=None,
    resample=PIL.Image.BILINEAR,
    fillcolor=None,
),

By looking around, the official pytorch website says
CLASStorchvision.transforms.RandomAffine(degrees, translate=None, scale=None, shear=None, interpolation=InterpolationMode.NEAREST, fill=0, center=None)

and there is no resample argument.
Thanks in advance.

Thank you, @ultimatemagic79, for bringing this up. It looks like a Pytorch version issue. We will fix it and get back soon.
In the meantime, you may resolve the issue locally by replacing the resample arg with interpolation, as per Pytorch docs.

Thanks all for the lighting fast response. I'll make some changes locally and see if there are any problems.

Hello, does our suggestion solve your problems? If it does, I would like to close this issue.

Sorry for the delay in commenting ... I will close this issue now that it has been resolved


Rahul reorg docs

Coverage increased (+0.008%) to 94.599% when pulling b2e6ab3a4d02b22ca8408aaddf09faa95ad4b95b on rahul-reorg-docs into 219cd96952451dbca31bccfe930c66b2aca53876 on main.


rvo2: Include cstdint for uint32_t

Fixes #76907.
Please test @timothyqiu @adamscott, I can't reproduce on Mageia 9 with GCC 12.3.0, I suspect the issue is specific to a change in libstdc++ from GCC 13 that you might both have on Arch Linux.
Edit: CI failure seems to be GH glitching again, will need a restart once they fixed their problem.

Should we use size_t instead as discussed?

If it fits with the thirdparty code better, then let's use that small diff instead.

The reason why I used uint32_t in the first place here is cause all other bitmask layers like physicslayers or navigationlayers are uint32_t throughout.

I think it doesn't matter much either way, so let's stick with this one.

Thanks!


update uri error

trying to fix broken uri for my one NFT
but keep getting this error
I use command  metaboss.exe update uri [OPTIONS] --account  --new-uri
but nothing
[2022-06-03T02:41:53Z WARN  metaboss] Using a public RPC URL is not recommended for heavy tasks as you will be rate-limited and suffer a performance hit.
Please use a private RPC endpoint for best performance results.
Error: RPC response error -32002: Transaction simulation failed: Blockhash not found
Caused by:

This is a Solana issue with the cluster time. Hopefully it's fixed soon.


Add info telling people to craft the cape at the Gladr Table

Currently some users expect this mod will modify their existing feather cape and they are confused why the mod doesn't work.

Good point, please check the latest release and let me know if it's less confusing now
https://thunderstore.io/c/valheim/p/Jumbo_Frame/OldFeatherCape/

Done


cannot upgrade to 5.3.3 from Anaconda Navigator, or by using Conda.

Description
What steps will reproduce the problem?

If I try to upgrade to 5.3.3 using Anaconda Navigator it fails. If I try to use the terminal and the conda commands, it also fails.  To do that, you can run something like the following from an Anaconda prompt:
conda create -n spyder-533 python=3.10 spyder=5.3.3

Then you will need to either activate the env and launch from it Spyder by running on an Anaconda prompt something like:
conda activate spyder-533
spyder

or check in your start menu for the shortcut that should have been created with the name Spyder (spyder-533)
Let us know if the info above helps!

thx so much! I waste one day to update the 5.3.3, but all ways have failed. The way you provide works.
A new problem is that when I install pkg either on an Anaconda prompt or in spyder, after pkg installed, the spyder crashed( be busy and cannot click).
on an Anaconda prompt
conda install pandas

in spyder
!pip install pyarrow

Notice that when I‘m installing pkg, the spyder 5.3.3 is open.
After close spyder and restart it, it works well.
Pkg installation is a frequent action, so it may not be so convenient.

Fantastic!!!!! worked like a charm. A thousand thanks sir. So happy I joined Github
Thank you,
Thomas

Glad it worked for you @scrappydog1958 !
In your case @yanyu2015 if you are constantly installing packages I would suggest you to always do it with the anaconda prompt and using the conda-forge channel (adding to your installation command the -c conda-forge flag). After installing new packages, if Spyder is running, you will need to restart the IPython console kernel inside Spyder or start a new IPython console inside Spyder to be able to use them

@dalthviz It works well now. The above phenomenon may be due to the large dataset I used at the time.

Awesome @yanyu2015 ! I will close this as completed then :+1:

Now that spyder 5.4 is out how do you change your commands tomorrow to upgrade?
Thanks,
Thomas
From: Daniel Althviz Moré @.>
Sent: Wednesday, November 2, 2022 8:40 PM
To: spyder-ide/spyder @.>
Cc: thomas loeff @.>; Mention @.>
Subject: Re: [spyder-ide/spyder] cannot upgrade to 5.3.3 from Anaconda Navigator, or by using Conda. (Issue #19965)
Hi @scrappydog1958https://github.com/scrappydog1958 , if you want to install Spyder 5.3.3 through conda, you will need to create a new environment. To do that, you can run something like the following from an Anaconda prompt:
conda create -n spyder-533 python=3.10 spyder=5.3.3
Then you will need to either activate the env and launch from it Spyder by running on an Anaconda prompt something like:
conda activate spyder-533
spyder
or check in your start menu for the shortcut that should have been created with the name Spyder (spyder-533)
Let us know if the info above helps!
—
Reply to this email directly, view it on GitHubhttps://github.com/spyder-ide/spyder/issues/19965#issuecomment-1301525196, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A36BDRIF64BGCR23EV5SDADWGMCXVANCNFSM6AAAAAARUIOVYA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Continuous deployment

Travis CI with AWS CodeDeploy
https://docs.travis-ci.com/user/deployment/codedeploy/

Code Deploy is for non-docker services.


Warn on unset exec type

close #162

Closing. We should come back to this issue some time in the future and figure out a way to emit warning (visible on stable) without failing the tarpaulin


Correction for raw affiliation Department of Epidemiology and Biostatistics, Faculty of Medicine, Western University, London, ON

Correction needed for raw affiliation Department of Epidemiology and Biostatistics, Faculty of Medicine, Western University, London, ON
raw_affiliation_name: Department of Epidemiology and Biostatistics, Faculty of Medicine, Western University, London, ON
new_rors: 02grkyz14
previous_rors: 02agqkc58
works_examples: W4386892179;W4386892103
contact: fbf8c3aadaae80a8267dadeb63a249c1:aac125a87c483f0a @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


repl: parsing error for anonymous func

In REPL:
func() { println(3) }()

Expected result:
$ yaegi
> func() { println(3) }()
3
>

Got:
% yaegi
> func() { println(3) }()
1:21: expected 'IDENT', found '{'

N.B. this works fine in non-REPL mode.

I hadn't properly understood what #721 was exactly about, but it seems it's pretty much the same case, so I'm now closing as duplicate in favor of #721


Combination of SetValidator and WithMessage

Hi there.
I have a hierachy of validators. In my parent validator I want to be able to define a context specific error message, but the rule itself is defined in my child validator. The error message is not shown when my rule fails. Should this not be possible?
code parent validator:
RuleFor(x => x.SamleverCPR).SetValidator(cprValidator)
            .WithName(Medlem.Medlem_Opdater_label_SamleverCPR)
            .WithMessage("invalid value"); //this message is not contained in error

code child validator:
public class CprValidator : AbstractValidator<string>
{
    public CprValidator()
    {
        RuleFor(cpr => cpr)
            .Must(cpr => (cpr.Length == 6 || cpr.Length == 10) && cpr.ContainsValidBirthDate());
    }
}

No, sorry, WithMessage is only usable on top-level rules. In this case, by calling SetValidator you're delegating all error message generation to the CprValidator.
As I've mentioned elsewhere, FV isn't designed for using AbstractValidator with primitive types - this is for use with complex properties, where the child validator is responsible for validating the properties of the child instance, not the instance itself.
I would highly recommend you don't go down this route - if you want to encapsulate these rules for primitive types, it's much better to use an extension method:
public static class MyValidatorExtensions {
	public static IRuleBuilderOptions<T, string> ValidateCpr<T>(this IRuleBuilder<T, string> rule) {
		return rule.Must(cpr => (cpr.Length == 6 || cpr.Length == 10) && cpr.ContainsValidBirthDate());
	}
}

Usable like this:
RuleFor(x => x.SamleverCPR).ValidateCpr()
   .WithName(Medlem.Medlem_Opdater_label_SamleverCPR)
   .WithMessage("invalid value");

...this would then work as you expect.

Again thx for the quick reply.
I like the idea of an extension method as it will make the code look more fluent. In cases where the rule is dependant on database communication, I cannot see it work as I want to inject a repository interface. This will work with an implementation of AbstractValidator, but cannot be done with an extension method.
public class CprValidator : AbstractValidator<string>
{
    public CprValidator(IRepository repository) //injected repository
    {
        RuleFor(cpr => cpr)
            .Must(cpr => (cpr.Length == 6 || cpr.Length == 10) && repository.ContainsValidBirthDate(cpr)); //call to database

    }
}

This makes me think that I cannot combine context specific error messages and reuse of validators. To you see any solution?

It's no different if you're using extension methods - you still inject the repository into your top-level validator, and then you can just pass it down to the extension method.
public class MyValidator : AbstractValidator<Foo> {
  public MyValidator(IRepository repository) {
    RuleFor(x => x.SamleverCPR).ValidateCpr(repository)
       .WithName(Medlem.Medlem_Opdater_label_SamleverCPR)
      .WithMessage("invalid value");
   }
}

If someone still looking for similar behavior it could be achieved by such extension methods:
`
///
/// Defines a validator rule.
///
/// Type of object being validated.
/// The type of the property.
/// The type of the validator.
/// The rule builder.
/// The validator.
///
public static IRuleBuilderOptions<T, TProperty> IsValid<T, TProperty, TValidator>(this IRuleBuilder<T, TProperty> ruleBuilder, TValidator validator)
where TValidator : IValidator
{
return ruleBuilder.Must(instance => validator.Validate(instance).IsValid);
}
///
/// Defines a validator rule.
///
/// Type of object being validated.
/// The type of the property.
/// The type of the validator.
/// The rule builder.
///
public static IRuleBuilderOptions<T, TProperty> IsValid<T, TProperty, TValidator>(this IRuleBuilder<T, TProperty> ruleBuilder)
where TValidator : IValidator, new()
{
return ruleBuilder.IsValid(new TValidator());
}
///
/// Defines a validator rule.
///
/// Type of object being validated.
/// The type of the property.
/// The type of the validator.
/// The rule builder.
/// The validator.
///
public static IRuleBuilderOptions<T, TProperty> IsValidAsync<T, TProperty, TValidator>(this IRuleBuilder<T, TProperty> ruleBuilder, TValidator validator)
where TValidator : IValidator
{
return ruleBuilder.MustAsync(async (instance, token) =>
{
ValidationResult result = await validator.ValidateAsync(instance, token).ConfigureAwait(false);
return result.IsValid;
});
}
///
/// Defines a validator rule.
///
/// Type of object being validated.
/// The type of the property.
/// The type of the validator.
/// The rule builder.
///
public static IRuleBuilderOptions<T, TProperty> IsValidAsync<T, TProperty, TValidator>(this IRuleBuilder<T, TProperty> ruleBuilder)
where TValidator : IValidator, new()
{
return ruleBuilder.IsValidAsync(new TValidator());
}
`


Add missing document about Client.setEndpoint()

Coverage remained the same at 92.057% when pulling 28461a466970fd6c2bf0026a79ba1eec5395f473 on aleung:setendpoint_doc into 491d10d5dac4e4a92b5243aa0776d2b5f5616a1c on vpulim:master.

Thank @aleung !


Attachment field not mapped correctly in Nest 2

Trying to map an attachment attribute like this:
client.Map<Case>(m => m.UpdateAllTypes());

With this classes and mapping definitions:
[ElasticsearchType(Name = "cases")]
public class Case
{
	public string Author { get; set; }
	public string CaseName { get; set; }
	[Attachment(Store = true)]
	public Attachment File { get; set; }

	public Case ()
	{

	}

	public override string ToString()
	{
		return "Case: " + Author + " - " + File.Name;
	}
}

public class Attachment
{
	[String(Name = "_content")]
	public string Content { get; set; }

	[String(Name = "_content_type")]
	public string ContentType { get; set; }

	[String(Name = "_name")]
	public string Name { get; set; }

}

Here is the generated mapping
"mappings": {
  "cases": {
    "properties": {
      "author": {
        "type": "string"
      },
      "case_name": {
        "type": "string"
      },
      "file": {
        "properties": {
          "_content": {
            "type": "string"
          },
          "_content_type": {
            "type": "string"
          },
          "_name": {
            "type": "string"
          }
        }
      }
    }
  }

Yep, there is "type": "attachment" missing down the file property definition.
Using Elasticsearch 2.2, Nest 2.0.2, Mono / .Net 4.5
As seen in SO

Thanks for reporting @gerosalesc we are all travelling and attending elasticon the coming days so we might take a little longer to investigate. Thanks for submitting an issue, always much appreciated :+1:

I've been making some changes in this area in https://github.com/elastic/elasticsearch-net/commit/8ad3032def4cfe6ecfff5d21887e3feb1ccb6dc5 and PR https://github.com/elastic/elasticsearch-net/pull/1827. I feel it would be a good idea to install both the mapper attachment and murmur3 plugins as part of spinning up an Elasticsearch node for integration tests, since we support both out of the box

++ agreed

@russcam @Mpdreamz  Thanks for your answers despite the elasticon. I have a couple of questions regarding this: is the decorator mapping working for the attachment type? If it's not, How would I map my Attachment attribute manually? I have seen the unit test but I would better understand if you have an example please :), pls.

@russcam according to the unit test something like this should be working but is giving me a horrible null reference at runtime, do you know why? maybe I am doing something wrong.

@gerosalesc it's probably the same issue as #1826. I'm working on a fix for this including examples, that will go into the next release.

@russcam Thanks for the reply. I tried your branch fix/mappings and the original decorator mapping and it's generating the mappings properly (or at least it seems), I will validate more carefully tomorrow. Good job buddy!

Closed via #1827


Can't disable the multi-select after initialisation

I currently want the user to select a maximum of 6 items.
In the afterSelect callback, I retreive the ammount of selected items, and when this number is greater than 5, I want to disable the box.
I do this like so:
[code]
afterSelect: function(value, text){
var array = $('#multiselect').val();
if(array.length>5)
$('#multiselect').attr('disabled','disabled');
},
[/code]
The disabled attribute is successfully added to the DOM element. However, I can still keep on selecting items.
Is there any other approach I should be following?

Was trying to achieve a similar stuff and spent time looking for a precise solution. In my case I was parsing an ajax function with remote connection to a php script in the afterSelect and afterDeselect methods. Like this;
[code]
........
afterSelect: afterSelect,
afterDeselect: afterDeselect
.........
[/code]
So the var _self = this did not work for me.
This however worked, just refreshing the multiselect after calling the disabled attribute.
$('#multiselect').attr('disabled','disabled');
$('#multiselect').multiSelect('refresh');


[Master] Fix overriding included fields with readonly intersection types

Purpose
$title.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/34756
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@MaryamZi Shall we resolve the conflicts?


mainactivity is never used

I want implementation a custom function use MethodChannel
see: https://docs.flutter.dev/development/platform-integration/platform-channels, but I notice the mainactivity is never used and
method of configureFlutterEngine() don't work

• Flutter version 3.3.8 on channel stable at D:\Program File\fvm\versions\3.3.8
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 52b3dc25f6 (8 weeks ago), 2022-11-09 12:09:26 +0800
• Engine revision 857bd6b74c
• Dart version 2.18.4
• DevTools version 2.15.0

Hi @json1994,
Can you please provide a minimal complete reproducible code sample so that we can investigate this?
I've used that document several times and I'm always able to complete the tutorial successfully with fully functioning apps.
Thank you

thank you for you replay, don't know why, my project work fine, the problem  didn't come up

Hi @json1994,
Can you please provide a minimal complete reproducible code sample so that we can investigate this?
I've used that document several times and I'm always able to complete the tutorial successfully with fully functioning apps.
Thank you

thank you for you replay, don't know why, my project work fine, the problem didn't come up


KiCad 6.0 symbol library format and .step 3D model export?

Thanks for the awesome work!:)
Any plans to add support for exporting symbols in the new library format? Otherwise one cannot edit the symbol in KiCad, except when having a separate library file for each part (as far as I know).
And being able to export 3D models as .step files would be super useful, for exporting 3D models from KiCad into other tools like Fusion360 for 3D modelling cases.

Unfortunately for the 3D models, JLCPCB does not give us the .step files, but only an MTL file described as a raw unstructured triangulated surface, that I parse and convert to VRML for compatibility with KiCad. This means that unless JLCPCB change its file format (which is very unlikely), it will not be possible to export the 3D models as a .step files.
As for the symbol library, if you go to the symbol library management window, there is a "Migrate Libraries button" that you can use to migrate a Legacy library into the new .kicad_sym format supported by KiCad 6.0. Using this, you can generate the symbol and then migrate the library to the new format, you will then be able to correctly edit the symbol. See the screenshot below.

However, I will work on that in the future, the symbol creation is a mess and I will soon start a refactoring of this part, this will be the occasion to change the file format and export it in the new library format.

Regarding the symbol format: yeah, I was aware of that. But this also changes the file ending - and therefore JLC2KiCad_lib cannot add another part to an existing library as far as I know. So one has to create another library for every single part. It's a work around but not ideal.
And regarding .step files: to bad :( how about getting the .step files from Octopart? Of course it won't be possible for all parts - but for those parts which do exist on Octopart, it would be super useful to have both the 3D model from LCSC and Octopart.

I opened a new branched for the kicad6.0 file format. I haven't finished every handler, but the most important and used ones are functional.
Here is a list of known issues :

Pin number and text might be miss placed and/or have the wrong size and/or might not be hided.
Text handler is not yet implemented.
Arc handler is not precise.
some other handlers are not implement as I haven't found any component using them. If you find any component with one of these handler, please report them so I can work on it. (you will get a warning message in this case)

I haven't got time to fully test it, so do not hesitate to report any other issues.
Overall the symbol generation should already be far more reliable than the current one.
I will open another issue to keep track of issues and work to do.
To come back to  the 3D models, this is not in my priority list to get it from another website, the name used by JLCPCB and Octopart might not be the exact same and can lead to errors, plus it would add a lot of complexity to get it and I don't want the script to get to complicated to use.

Thanks for the update :)
Regarding 3D model: Yeah fair enough :)
Hm but could there maybe be a way to automate the process of this video here, to convert the 3D models into step files?

I am not sure to understand which video you are talking about. But there are many tools online to convert .wrl file to .step or any other file type. Plus, even with this, you won't be able to edit the step file as it will still come from a wrl file which describe the triangular surface of the 3d model, which is fundamentally different than a .step file description.
What is your end objective with this conversion ? Is it to edit the 3d model, or only to preview it ? In the first case, it will not be possible to do it with fusion360 SOLIDWORKS nor freecad as they do not support this format. But you can do it with other software such as blender which is better suited for it.

 Forgot the video link… https://youtube.com/watch?v=qSkWBwRJVqg

I didn't realise that KiCad only exports .step models, I understand the problem better now.
It's not one of my priorities, but I'll see when I have the time to implement a solution for this.


Improved scripting

Hello again ;)
Reconnecting to the discussion we had here. I am proposing minor changes to improve the quality of scripting with planutils.
This PR will do the following.

Remove the -i flag from bin/planutils. This will mean that planutilscommands will not be run in interactive mode. This means these commands will not be written to the command history. Additionally, this will fix the warningsbash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell

which arise when calling planutils commands from script contexts without stdin (like docker build for instance).
In order to preserve the interactive planutils activate environment, this is moved into its own script—which has the -i flag.
Additionally, the planutils setup is provided with a --force flag, which allows it to be called in non-interactive shells even if it has been called before.
Finally, the README provides a more correct on how to interactively use planutils, and is lightly touched up with syntax highlighting

Additionally, this will fix the warnings

bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell

That little thing has been a thorn in my side for ages! Nice to see it finally fall...


Only use gamma if given

This PR will fix the issue in #30
The condition was simply not set correctly. Additionally, the gamma values are lowercase.

CLA is valid!

Coverage remained the same at 71.069% when pulling 86b0a1ad0b1992303acf242c019ef8b0a3dd6ce9 on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.

Coverage remained the same at 71.069% when pulling 685fd4352d75b76177303646391259bdb5e1d46b on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.

Coverage remained the same at 71.069% when pulling 9a0d7ef95668cebceb9c0ab49fbb31536f51416b on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.

Coverage remained the same at 71.069% when pulling 9a0d7ef95668cebceb9c0ab49fbb31536f51416b on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.

Coverage remained the same at 71.069% when pulling 9a0d7ef95668cebceb9c0ab49fbb31536f51416b on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.

Coverage remained the same at 71.069% when pulling 9a0d7ef95668cebceb9c0ab49fbb31536f51416b on preceptorjs:gamma into 644140869afddf2a1b343919ce246f0f00100398 on yahoo:master.


Add a test to check bitcoin_hashes dependency

Draft because will not pass CI until we release secp256 v0.24.0
Recently we updated the bitcoin_hashes dependency without first doing
so in secp256k1. Since bitcoin_hashes is an optional dependency of
secp this did not show up in our CI run and was reported by a user.
To prevent this from happening again we can add a test that creates two
arbitrary objects, one from the rust-bitcoin bitcoin_hashes
dependency and one from the rust-secp256k1 bitcoin_hashes dependency
then assert that they are comparable. If different versions of
bitcoin_hashes are being used then the compiler will complain.

add a test that creates two arbitrary objects

Ah! Good idea! 💡

Sadly, this doesn't solve the issue because when this test fails it's already too late - incompatible release of secp256k1 is already out.

Using this check involves a bunch of manual work.
Alternative check suggestion: in secp256k1 CI detect if it contians breaking version bump, if yes, check if there's no bitcoin_hashes crate with higher major (or 0.minor) version in crates.io. This still has issue that if bitcoin_hashes is released after secp256k1 bump PR merges it doesn't get caught. We could check against bitcoin_hashes git repo instead including PRs but that's pprobably too much.

Sadly, this doesn't solve the issue because when this test fails it's already too late - incompatible release of secp256k1 is already out.

🤔 Hmm... But surely this does help to a certain extent. For instance, if rust-bitcoin has bitcoin_hashes version X.Y.Z and rust-secp256k1 is released with bitcoin_hashes version X'.Y'.Z' (where X'.Y'.Z' > X.Y.Z) and there is an intent to upgrade rust-bitcoin to this new version of rust-secp256k1, this check will "complain" and the solution would be to also bump the bitcoin_hashes version used in rust-bitcoin too.
Alternatively, if the current rust-secp256k1 dependency of rust-bitcoin uses X.Y.Z version of bitcoin_hashes, and there is a bump in the bitcoin_hashes version to X'.Y'.Z' (whereX'.Y'.Z' > X.Y.Z ), then, whenever anyone tries to upgrade the bitcoin_hashes dependency to this new version, this check will also complain. The solution would be to upgrade rust-secp256k1 to a version using this new X'.Y'.Z' version of bitcoin_hashes. Of course, this can only be done once there is an appropriate rust-secp256k1 release for that, but, in the meantime, the discrepancy will be avoided in bitcoin_hashes.
It may be that I'm biting way more than I can chew here 😅 , because I've not got much experience with release management, but that is how I understood this check. Am I missing something? I'd love to know if there's  a flaw there!

Also, @tcharding, may I suggest that we add an error message here explaining briefly why it's necessary for these two to match? Otherwise, I think the failure of this check might be a bit cryptic to a new-comer like me! 😁

Ah! I think I just understood what you meant, though!
The problem arises whenever there is a bump in rust-secp256k1, but the bitcoin_hashes dependency there stays at an "outdated" version...
I think this check would still be helpful for this case in the sense that it will not be possible to upgrade both rust-secp256k1 AND bitcoin_hashes at the same time, which is what happened in 36f29d4357fa667079f4924073090de5c217b72d and  1e46eeaa88aa5fc2c0d3dd163043a724f7af7ac9, right?
So this check would've failed at 1e46eeaa88aa5fc2c0d3dd163043a724f7af7ac9.

Using this check involves a bunch of manual work. Alternative check suggestion: in secp256k1 CI detect if it contians breaking version bump, if yes, check if there's no bitcoin_hashes crate with higher major (or 0.minor) version in crates.io. This still has issue that if bitcoin_hashes is released after secp256k1 bump PR merges it doesn't get caught. We could check against bitcoin_hashes git repo instead including PRs but that's pprobably too much.

This is creative. I like it.

Sadly, this doesn't solve the issue because when this test fails it's already too late - incompatible release of secp256k1 is already out.

The issue this test solves is updating bitcoin_hashes here before in secp. If we update it in secp first and release its no problem because we have to update the secp dependency here to pick it up, at which time this test fails if we forgot to do bitcoin_hashes. Or am I missing some other way we can fail?

may I suggest that we add an error message here explaining briefly why it's necessary

The test never runs, it fails at build time if the types are from different versions. The error message looks like this:
error[E0308]: mismatched types
--> tests/dependencies.rs:8:5
|
8 |     assert_eq!(our_dep, secp_dep)
|     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected enum `bitcoin_hashes::hex::Error`, found enum `secp256k1::bitcoin_hashes::hex::Error`
|
= note: perhaps two different versions of crate `bitcoin_hashes` are being used?

And also the file has
+//! This file does arbitrary comparisons of various types to ensure all the dependency versions
+//! are equal. It exists to catch our mistakes when upgrading crates in the stack.

Is that sufficient?

OK, so this is summary of what I understand now, LMK if I'm wrong:

@arturomf94 argues (in the long comment above) that while this check doesn't prevent forgetting to bump secp256k1 when releasing breaking version it's still valuable for other cases: it may happen we bump one but forget about the other.
@tcharding seems to agree and it was the entire point of this PR.

I agree that such test is valuable, I just have two objections to the way it's impemented:

It seems the check will be redundant once we start using the ThirtyTwoBytesHash integration since different version will cause the same error in our lib code. Tests may be a bit more explicit but library will fail first anyway.
If we ever use semver trick (we should whenever reasonable!) this test will pass even if the library versions do not match. The best solution seems to be just testing whether cargo tree -d --all-features output is empty. This will also catch any other library mismatch, not just bitcoin_hashes/secp256k1. We can put the test before anything else if we want to fail it fast (probably a good thing; or maybe put it in style workflows so others can still run). Happens to solve the above issue as well.

oo, I like the idea of running tree -d in CI

testing whether cargo tree -d --all-features output is empty [...] will also catch any other library mismatch

Ah! Didn't know this existed! Yeah, I agree: using that would be better because it's a generalization of what this is trying to achieve. Is it always the case that library mismatches are a no-go? I guess the collision/conflict wrt bitcoin_hashes comes from the fact that rust-bitcoin uses it directly and via secp256k1, right?
Could something along the lines of cargo tree -d --all-features | grep -E "library1|library2|library3"" improve upon the suggestion to test for the specific libraries amongst which a mismatch wants to be avoided?

Is it always the case that library mismatches are a no-go?

Probably there are cases where it's fine, but we can explicitly whitelist those.


Overlay fade out before being removed

That will fixes #1
Also, nice job with this lightbox, I really appreciate that there is no dependencies, especially jQuery, and that it's written in es6 :ok_hand:

Thanks @thomas-lebeau! and thank you for submitting a PR.
I'm still debating on the best way to handle polyfilling the es6 stuff, but for now it's all getting shimmed in the output file. Perhaps I could have two dists – one with polyfill, one without.

I think the dist without pollyfills is the source. Or you are talking about something else?

true. you could require the src if you wanted to handle polyfilling and transpiling on your own.

Although the src folder should not be ignored on npm then (see .npmignore).
Also, I think it will be nice to have a css folder separate from the one in test. Because, you still need some styles to make it work. I will be more straightforward to use.

@thomas-lebeau this is true. There should be some css output in dist the meets all the requirements. This is very dependent on the styles.
Think I should remove that style dependency as well?

It always seemed odd to me to have to link multiple file to use a component, but it is the more common approach.
And in the other way setting styles with js is not very maintainable too. (And i think we need a css file for the keyframes anyway)


Docs: Fixed Typography  spelling errors (#5040)

Description
Resolves #5040

How Has This Been Tested?
Doc change only

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.

Hi @FakeRayBanz, and thanks again for your work. I'm sorry for the delay. Next time, I'll be much quicker.


BigQuery build plan requires CREATE SCHEMA permissions to create external tables

Describe the bug
Before trying to create a new dataset we should check if it already exists without using the CREATE statement.
It could be that the DBT SA or DBT user that runs the command does not has the permission to create a dataset.

PR:
https://github.com/dbt-labs/dbt-external-tables/pull/220

Fixed in Version 0.8.7.
Hopefully you do not have digits in your google-project-name :-)
See https://github.com/dbt-labs/dbt-external-tables/issues/235


Android: Fix UI glitches on API 14 and 15 devices

working around some ugly things resulting from buggy early implementations of the UI framework.

Sounds like a fine compromise.

From here on out I'm not going to over-spend resources on API 14/15 issues unless they impact stability.  I fully expect we have <5% of active users on those versions and we will consider dropping support for 14/15 soon after we add support for API 24. (older versions will keep working, but updates will focus on devices made in the last 5 years)
I am more than ready to do a Beta from where we are (just 2 days to check that we got rid of on-upgrade crashes), unless y'all have found any issues.

Feel free to drop support earlier. Right now we only support iOS 8 and iOS 9.
Pushed out as v4.9.0b1.


Addition of Map feature

Addition of Map feature. Addition in other files for map file implementation and navigation.

This latest one fixes the pin locations on the map. The changes added the map file and screen with implementation and navigation.

Looks good and perfect pull request also. Aaron CastilloOn Dec 3, 2023, at 23:17, Erick Nevarez @.***> wrote:
This latest one fixes the pin locations on the map. The changes added the map file and screen with implementation and navigation.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


MWPW-147913: Revert dropzone disabling

Revert: MWPW-147913
https://mwpw-147913-revert--dc--adobecom.hlx.page/acrobat/online/compress-pdf?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.44%. Comparing base (66da175) to head (26d3cb1).

Additional details and impacted files
@@           Coverage Diff           @@
##            stage     #685   +/-   ##
=======================================
  Coverage   90.43%   90.44%
=======================================
  Files          36       36
  Lines        4528     4522    -6
=======================================
- Hits         4095     4090    -5
+ Misses        433      432    -1

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


Is there any parameter to make the picture render as a cover photo?

In the doucument:

type: string, for data items, is always data
layerName: string, target layer name in the After Effects project
layerIndex: integer, can be used instead of layerName to select a layer by providing an index, starting from 1 (default behavior of AE jsx scripting env)
property: string, indicates which layer property you want to change
value: mixed, optional, indicates which value you want to be set to a specified property
expression: string, optional, allows you to specify an expression that can be executed every frame to calculate the value
composition: string, composition where the layer is, useful for searching layer in pre-compositions. If none is provided, it uses the default composition set in the template.

and my setting is :
job.assets.push({
        src: assets_data.src,
        type: "image",
        layerName: assets_data.name,
        composition: "*"
});

I don't know how to set 'cover' into my image setting.
If there is any parameter to set 'cover', please tell and help me!

Hello!
Could you please explain in more details what do you mean by 'cover'?

hi,
'cover' just means a cover photo
because my project can not render the photo fit to composition
but use the way by 'add to render queue' in AE, it can fit very well.
Sorry, my English is very poor...
Please translate these words...
我的專案在 AE 內，使用 add to render queue ，渲染出來的影片，圖片是可以自適應至 comp 內的設定
但使用 aerender 直接下命令 或 nexrender 渲染出來的影片，圖片都變得很小
從 AE 的設定得知一個 fit to comp 的設定，想知道 nexrender 有沒有此功能？
而 cover 這個名稱是從舊版的 nexrender 中的屬性得知的
但新版的 nexrender 文件中，並沒有看見此屬性
請務必協助我，在此由衷感謝。
Vladyslav Hrytsenko<EMAIL_ADDRESS>於 2019年10月17日 週四 20:34 寫道：

Hello!
Could you please explain in more details what do you mean by 'cover'?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/inlife/nexrender/issues/262?email_source=notifications&email_token=AFOOIBCMPD3DC3LA727M7DLQPBLTZA5CNFSM4JBH2VNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBP5XQA#issuecomment-543153088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOOIBBKHLEX7IKMIH3NHH3QPBLTZANCNFSM4JBH2VNA
.

@piedasing  are you trying to capture a single frame of the render as a poster image?
I achieved this by writing a small post render module to capture a thumbnail the size, of the source video.
thinking aloud now you could add a custom param to nexrender-action-encode to do this also, why didn't I do that before, doh

@JamesBotterill
Hello, I am not very familiar with the parameter setting of nexrender-action-encode that you said.
Could you please tell me more about it? Or provide me with relevant documentation, thank you.

I found the problem in my project!
There is a composition setting "scale" in AE. It can set image auto fit to comp size.
In my AE  project, it can read my "config.js" file in root directory.
But, when I use "aerender.exe" by command line or nexrender in node server, it cannot read it.
Maybe it is just a wrong file path in my setting.
Finally, I figure it out by put "config.js" into the directory where the aerender.exe is located.

@JamesBotterill Could you publish your thumbnail creator plugin to npm so we can use it?


⚠ Bofrost.de has degraded performance

In 0eefadc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1909 ms

Resolved: Bofrost.de performance has improved in e3e6f86 after 6 minutes.


On Data tables the bootstrap search does not use entire number

I have changed a column data to numbers, and sorting the numbers in order it has
1
105
1002
2
23
2145
Instead of putting them from lowest to highest
Do you have a solution for this please?

 尝试覆盖内置的排序方法 ，我强力推荐使用服务器排序。
Trying to override the built-in sorting method, I strongly recommend using server sorting.
------------------ 原始邮件 ------------------
发件人:                                                                                "puikinsh/sufee-admin-dashboard"<EMAIL_ADDRESS>发送时间: 2021年3月1日(星期一) 下午2:28
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [puikinsh/sufee-admin-dashboard] On Data tables the bootstrap search does not use entire number (#30)
I have changed the data to a number, and sorting the numbers in order it has
1
105
1002
2
23
2145
Instead of putting them from lowest to highest
Do you have a solution for this please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


rename downstream to upstream

Without this patch martian's naming logic was incompatible with other proxies such as Envoy https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/intro/terminology.
This patch renames downstream to upstream to assure compatibility.
Fixes saucelabs/forwarder#152

This patch contains some unrelated formatting changes @Choraden

This patch contains some unrelated formatting changes @Choraden

Formatting changes have been moved to separate commit.
They are the result of plain go fmt.


Fix Output In Pulsar Go Client Documentation

Motivation
In http://pulsar.incubator.apache.org/docs/latest/clients/go/#Produceroperations ,
producer.SendAsync(ctx, asyncMsg, func(msg pulsar.ProducerMessage, err error) {
	if err != nil {
		log.Fatal(err)
	}

	fmt.Printf("Message %s succesfully published\n", msg.ID())
})

cannot be compiled because msg doesn't have ID method.
$ go build producer.go
# command-line-arguments
./producer.go:54:56: msg.ID undefined (type pulsar.ProducerMessage has no field or method ID)

Modifications
I changed msg.ID() to msg.Payload.
producer.SendAsync(ctx, asyncMsg, func(msg pulsar.ProducerMessage, err error) {
	if err != nil {
		log.Fatal(err)
	}

	fmt.Printf("Message %s succesfully published\n", msg.Payload)
})

Result
This modified code can be compiled and executed successfully.
$ go build producer.go
$ ./producer
2018/07/09 16:22:17 INFO  | ConnectionPool:63 | Created connection for pulsar://localhost:6650
2018/07/09 16:22:17 INFO  | ClientConnection:279 | [[::1]:62559 -> [::1]:6650] Connected to broker
2018/07/09 16:22:17 INFO  | HandlerBase:53 | [persistent://public/default/my-topic, ] Getting connection from pool
2018/07/09 16:22:17 INFO  | ConnectionPool:63 | Created connection for pulsar://ac47a8d4f588:6650
2018/07/09 16:22:17 INFO  | ClientConnection:281 | [[::1]:62560 -> [::1]:6650] Connected to broker through proxy. Logical broker: pulsar://ac47a8d4f588:6650
2018/07/09 16:22:17 INFO  | ProducerImpl:154 | [persistent://public/default/my-topic, ] Created producer on broker [[::1]:62560 -> [::1]:6650]
Message async-message-0 successfully published
Message async-message-1 successfully published
Message async-message-2 successfully published
Message async-message-3 successfully published
Message async-message-4 successfully published
Message async-message-5 successfully published
Message async-message-6 successfully published
Message async-message-7 successfully published
Message async-message-8 successfully published

retest this please


bug(progress-spinner): Failed to execute 'insertRule' on 'CSSStyleSheet'

Bug, feature request, or proposal:
DOMException: Failed to execute 'insertRule' on 'CSSStyleSheet': Failed to parse the rule '
@keyframes mat-progress-spinner-stroke-rotate-32 {
0%      { stroke-dashoffset: 65.65928646002666;  transform: rotate(0); }
12.5%   { stroke-dashoffset: 13.82300767579509;    transform: rotate(0); }
12.5001%  { stroke-dashoffset: 13.82300767579509;    transform: rotateX(180deg) rotate(72.5deg); }
25%     { stroke-dashoffset: 65.65928646002666;  transform: rotateX(180deg) rotate(72.5deg); }
What is the expected behavior?
Business as usual !!!
What is the current behavior?

In Android 5.1 and below devices, the mat style is abnormal, and console has been reporting this error a lot.
What are the steps to reproduce?
You can browse https://material.angular.io/ on Android 5.1 device.
See if everything is normal ?!
What is the use-case or motivation for changing an existing behavior?
Unable to use normally !!!
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All latest !
Is there anything else we should know?

I wonder why I added the crosswalk plugin, but it didn't do anything??

Can you post the full stack trace?

A basic fact is that this component can not be used in version 5.1 and later.

Are there any polyfill can be solved?

That is the kind of error that'll be thrown when the browser doesn't support @keyframes. In your case the dynamic keyframes that get inserted by Material would have to be prefixed with -webkit-. Adding the prefix won't be difficult, however we don't support any browsers that use the prefixed keyframes.

That is the kind of error that'll be thrown when the browser doesn't support @keyframes. In your case the dynamic keyframes that get inserted by Material would have to be prefixed with -webkit-. Adding the prefix won't be difficult, however we don't support any browsers that use the prefixed keyframes.

Does mean that  material does not support Android devices 5.1 and below?

It does, here's the note on browser support from our readme:

Angular Material supports the most recent two versions of all major browsers: Chrome (including Android), Firefox, Safari (including iOS), and IE11 / Edge

That being said, looking at caniuse, Android 5 should support unprefixed CSS animations. Since you mentioned that the error is only thrown when you change the diameter, does the spinner animate if you leave it in with the default diameter?

No, they will remain motionless....

I tested it on the x86 5.1 version of the virtual machine.

No, they will remain motionless....
That means that the browser either doesn't support CSS animations altogether, or it requires a fix. Either way it's not a browser that we support.


chore: update spec table (#762)

Backport
This will backport the following commits from k8s-24/main to k8s-23/main:

chore: update spec table (#762)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 764

Questions ?
Please refer to the Backport tool documentation


Add support for Xiaomi Mijia Air Purifier X

Please consider supporting the newest Air Purifier, the Xiaomi Mijia Air Purifier X.

+1
I need it too. Please.

+1. Too bad it isn't supported after almost a years since release


Gradle Module Metadata for picocli 4.4.0+ missing from Maven Central

picocli-4.4.0.pom was generated with the following comment
<!--  This module was also published with a richer model, Gradle metadata,   -->
<!--  which should be used instead. Do not delete the following line which   -->
<!--  is to indicate to Gradle or any Gradle module metadata file consumer   -->
<!--  that they should prefer consuming it instead.  -->
<!--  do_not_remove: published-with-gradle-metadata  -->

which triggers the download by Gradle of the picocli-4.4.0.module artifact, however that artifact was not published to Maven Central. This is probably a consequence of the upgrade to Gradle 6 in 98dc696565d8a0c215ed98a9a403bf3262b47af4.
This is a problem when using Artifactory as a Maven proxy, because by default Artifactory returns a 403 HTTP status code for missing artifacts, and Gradle only ignores an absent .module when the repository returns a 404 HTTP status code. In my case (and others, probably), that means my build fails when it cannot download picocli-4.4.0.module. 😭
See rsocket/rsocket-java#746 for a similar issue in another project (resolved by rsocket/rsocket-java#751).

Ouch! I wasn’t aware of this, thanks for letting me know!
I’ll probably use the same fix: disable the feature for now.

I pushed a change that fixes the issue (disables generation of the picocli-4.4.0.module artifact).
I will do a release for this as soon as possible.

Picocli 4.5.1 has been released. This should fix the issue.
Don't forget to star ⭐ picocli on GitHub if you like the project! 😉


Bug 1788916: UPSTREAM: 85283: Included CSINode describer

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Introduces the CSINode Describer, so that some of the unnecessary fields are not displayed when describing these objects.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Cleaned up the output from `kubectl describe CSINode <name>`.

/retitle Bug 1788916: UPSTREAM: 85283: Included CSINode describer

@huffmanca, I think you need to target master branch to get a fix into 4.4

/retitle Bug 1762692: UPSTREAM: 85283: Included CSINode describer

/bugzilla refresh


Update onboarding screens (v3) | Design

Description of feature / user story
As a person who is misinformed on the purpose of BC Wallet, I want to be properly informed so that I am not disappointed when it doesn't work as expected
Acceptance Criteria
Given the user has just downloaded the app
When they open the app
Then they are presented with a screen that requires their confirmation that they've read and understood the purpose of BC Wallet
Wireframes or relevant image assets / links
Links to updates to onboarding screens

@knguyenBC @jleach I'm assuming this a design ticket only? I don't see any changes to the onboarding screens in the latest release. If it is a design ticket can we add the design label?

Design ticket. To be reviewed with Clecio and then a dev ticket will form from it.


Fix sync during bootstrap

Please provide a description for what this PR is for.
Fix: #33218
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

integ-ipv6-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-ipv6-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

integ-ipv6-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-ipv6-k8s-tests_istio

integ-helm-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-helm-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test release-notes_istio

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

integ-ipv6-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-ipv6-k8s-tests_istio

integ-helm-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-helm-tests_istio

integ-operator-controller-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-operator-controller-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test lint_istio

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

integ-ipv6-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-ipv6-k8s-tests_istio

integ-helm-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-helm-tests_istio

integ-operator-controller-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-operator-controller-tests_istio

release-notes_istio
c3d847aa9f7de91889e8254f104561da5752af7f
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-multicluster-k8s-tests_istio

integ-distroless-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-distroless-k8s-tests_istio

integ-ipv6-k8s-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-ipv6-k8s-tests_istio

integ-helm-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-helm-tests_istio

integ-operator-controller-tests_istio
635a1db14492f14d399c34a00aa498477aff18eb
link
/test integ-operator-controller-tests_istio

release-notes_istio
c3d847aa9f7de91889e8254f104561da5752af7f
link
/test release-notes_istio

lint_istio
c3d847aa9f7de91889e8254f104561da5752af7f
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
c3ad6ee59b9c484de5ba8055e2184572b82bfde4
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

LGTM

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
d414b4adebd568f02a3315eb3310be1c3acbe236
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I do not think this is right. Can you help me understand?
We are double processing events for no reason now. In kube controller it makes sense because we have inter-dependencies on each object, so if endpoint update comes before service update things may break; here we do not have that issue.
If you add some logs you can see each CRD is processed twice now instead of 1x. If all we need to do is ensure all events are processed before marking ready, we should be able to do that without 2x events

I think the 2nd PR is supposed to do the deduping (don't put it on the queue until we've started SyncAll. IIUC this was a prelim PR.

@stevenctl is right


SCC iterator

Iterator over the strongly connected components of a directed graph G=(V,E).

Time: O(n+m)
Space: O(n log(log(n))) bits

Coverage increased (+0.3%) to 84.69% when pulling d8ad0ddbd7b2c951dbb5c2aba74ba3218d89e921 on shsr04:scc-iterator into 26993b67c48945f0e67dc159610389ac58e31183 on thm-mni-ii:master.


[BUG] Roblox Desktop App shows a skybox and nothing else

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
When starting the Roblox desktop app with Bloxstrap, it will boot to an empty screen with either a skybox or a white screen. Without using Bloxstrap, the desktop app starts properly.
In a Roblox Logfile (when using bloxstrap), it says this:

Stack Begin
Script 'CoreGui.RobloxGui.Modules.LuaChat.Components.TextInputEntry', Line 14
Stack End

2024-01-11T01:52:49.245Z,2.245130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'CoreGui.RobloxGui.Modules.LuaChat.Components.DialogComponents', Line 20
Stack End
2024-01-11T01:52:49.245Z,2.245130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'CoreGui.RobloxGui.Modules.LuaChat.Views.Phone.Alert', Line 11
Stack End
2024-01-11T01:52:49.245Z,2.245130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'CoreGui.RobloxGui.Modules.ChatMaster', Line 37
Stack End
2024-01-11T01:52:49.246Z,2.246130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'CoreGui.RobloxGui.Modules.LuaApp.Components.AppContainer', Line 28
Stack End
2024-01-11T01:52:49.246Z,2.246130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'CoreGui.RobloxGui.Modules.LuaApp.Components.UniversalApp', Line 44
Stack End
2024-01-11T01:52:49.246Z,2.246130,458c,6 [FLog::Error] Error: Requested module experienced an error while loading
Stack Begin
Script 'Script Context.LuaAppStarterScript', Line 66
Stack End

Do you have an old DataModelPatch in your modifications? Have you installed any mods that change the loading screen?

Do you have an old DataModelPatch in your modifications? Have you installed any mods that change the loading screen?

This issue sometimes happens if you have one of these

Do you have an old DataModelPatch in your modifications? Have you installed any mods that change the loading screen?

This is what happened, replaced it with a new one, it works now. thanks

bruh


how to implement shared viewHolder?

hi.
i would like to implement a shared ViewHolder for all my items because all of my items share a TextView and a CheckBox, and then every other object (item) type may add other views to the layout (e.g. Spinner).
this is my BaseItem.ViewHolder. all other objects in my app extend BaseItem and their respective ViewHolders extend this BaseItem.ViewHolder:
static class ViewHolder extends RecyclerView.ViewHolder
	{
		protected TextView textView;
		protected CheckBox checkBox;

		public ViewHolder(View view)
		{
			super(view);
			textView = (TextView) view.findViewById(R.id.textView);
			checkBox = (CheckBox) view.findViewById(R.id.checkBox);
		}
	}

this is an example of a layout for one of my items (the most basic one which only includes the TextView and the CheckBox):

<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
                xmlns:tools="http://schemas.android.com/tools"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                android:descendantFocusability="blocksDescendants"
                android:padding="16dp">

    <TextView
        android:id="@+id/textView"
        style="?android:textAppearanceListItemSmall"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignParentLeft="true"
        android:text="place holder"
        tools:ignore="HardcodedText,RtlHardcoded"/>

    <CheckBox
        android:id="@+id/checkBox"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_alignParentRight="true"
        android:layout_centerVertical="true"
        tools:ignore="RtlHardcoded"/>

</RelativeLayout>

the problem is the findViewById method in BaseItem.ViewHolder return null.
if i add findViewById calls to each individual ViewHolder (instead of extending a shared one) everything works fine, but this is just a hassle and isn't very object-oriented.
how can i fix this?

@or-dvir hmmm I am not really sure why the `FindViewById method would return null.
I use a base ViewHolder in a project and it works without problems.
What layout id is defined for the base/implementing class?

@mikepenz the BaseItem actually does not have any layout as i am just using it as a shared object all other objects inherit from, so it doesn't really need a layout. but according to object oriented programming that shouldn't matter because the "getLayoutRes()" method of the inherited item should be called (for example the one that returns the layout i posted in my question).
i also tried separating the viewHolder from BaseItem and making it a separate class and that didn't work either

that's weird.
For example you can have a loook on SimpleSubItem and SimpleSubExpandableItem where SimpleSubExpandableItem extends SimpleSubItem

@mikepenz ill try when i have time (whenever that may be...)

Ok. I'll close this now, as I am sure this is possible (as I use it multiple times already)


upgrade using system upgrade controller throws error

I installed a brand new K3OS cluster from ISO in my test lab. It's a HA cluster (7 nodes, 3 servers, rancher UI installed).
I changed my system-upgrade-controller plan to the new repo,  just the channel line. I did it editing yaml in Rancher UI.
It throws an error:
time="2022-04-05T08:40:24Z" level=debug msg="copying: /host/k3os/system/kernel/current -> /host/k3os/system/kernel/previous" time="2022-04-05T08:40:24Z" level=debug msg="renaming: /host/k3os/system/kernel/current.tmp -> /host/k3os/system/kernel/current" time="2022-04-05T08:40:28Z" level=warning msg="exec: \"nsenter\": executable file not found in $PATH" time="2022-04-05T08:40:28Z" level=debug msg="attempting chroot: /host"
These are the last log entries in my completed job apply-k3os-latest-on-k3s07-with-b40051a66027b59f5bedba1d5-nj9hd
Any suggestions?
Thanks,
Birger

Sorry, there was an intermittent error.
I verified, it simply works.
Reverted plan channel to rancher, upgrade to 1.21.5 worked. Did set channel to mazzy89 again, upgrade to 1.22.2 succeeds.
Tested on another node, upgrade from 1.21.1 to 1.22.2 succeded instantly.
Upgraded the whole cluster, success.
Birger

After some time I lost my cluster controller. I have no idea what happened and how to fix it.
I find this in the logfile
"msg":"invalid downgrade; server version is lower than determined cluster version","current-server-version":"3.4.13","determined-cluster-version":"3.5"
I cannot manage with kubectl anymore. My Rancher UI doesn't work anymore. Other Apps still run on the cluster.
Any idea to get my cluster back? Other than throwing it away, loosing all?
Thanks, Birger


Small enhancement to the popup's close button

(The Yada yada section for background  - see screenshots below for tl;dr)
This relates to https://github.com/Leaflet/Leaflet/issues/7549. That issue points to the WCAG 2.1 Success Criterion (SC) for Target Size, and requires a Target Size of 44x44 px which I think may be a big leap to take, at least for now. The good news is that WCAG 2.2 (which notably wont reach Recommendation status until June IIRC) instead has 2 SCs:

Target Size (Minimum) which requires target size of 24x24
Target Size (Enhanced) which requires target size of 44x44

This PR change makes it so that the popup close buttons (will) pass WCAG 2.2 SC 2.5.8 Target Size (Minimum). This PR also fixes a small visual bug with the outline for keyboard users.
Preview and comparison
Current

Proposed
See the latest changes in https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1012696490!

@IvanSanchez In this case I'm only setting the close button to 24x24px. In either case it seems there is indeed a need to adjust the margin on leaflet-popup-content:
https://github.com/Leaflet/Leaflet/blob/142ffdee3b87a506f4cc2084209ec16f373eb731/dist/leaflet.css#L476-L477
If that line was changed to margin: 24px in this case, we can be sure that there's no overlap, the result would look like:

same change, with some highlighting:

so that does have a visual impact on the popup (of course, there's no need for margin-bottom to be 24px as well, but for symmetry that might be warranted?).
Other than that there's not much room for negative margins on the close button itself before it hugs the edge of the popup.

Actually, all we have to worry about is the margin-right (and left for symmetry) right? It's getting late here...
so if we change:
https://github.com/Leaflet/Leaflet/blob/142ffdee3b87a506f4cc2084209ec16f373eb731/dist/leaflet.css#L476-L477
to margin: 13px 24px we'd get:

same screenshot, with highlighting:

Changed leaflet-popup-content's margin from 13px 19px to 13px 24px in https://github.com/Leaflet/Leaflet/pull/7908/commits/542a54a066fa789ccf5fa62f0709dd040eafb9ca per https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1012696490 and Ivan's feedback.

@Malvoz This is very nice! I'll take a second look a few days later to make sure the design aesthetic isn't compromised too much with the wide side margins — perhaps it would be OK to have a little overlap.
Off topic, but on your screenshots, there's a 1pm gap between the popup and its tip — I wonder if that's a bug we should fix too? Is it easily reproducible?

Tnks all
On Fri, 14 Jan 2022, 23:00 Vladimir Agafonkin, @.***>
wrote:

@Malvoz https://github.com/Malvoz This is very nice! I'll take a second
look a few days later to make sure the design aesthetic isn't compromised
too much with the wide side margins — perhaps it would be OK to have a
little overlap.
Off topic, but on your screenshots, there's a 1pm gap between the popup
and its tip — I wonder if that's a bug we should fix too? Is it easily
reproducible?
—
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1013529489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2UXSEXM3T7XFHI7LH5KXLUWCTIPANCNFSM5L5JZ6GQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@mourner https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1013529489:

Off topic, but on your screenshots, there's a 1pm gap between the popup and its tip — I wonder if that's a bug we should fix too? Is it easily reproducible?

The gap is often appearing, but inconsistently so.
Even the popup in Chrome's dev tools has the same issue (compare their popup in this screenshot to this), I think the root cause is the same as the 1px gap bug between tiles, probably a Chromium compositing bug.
In the Maps4HTML <mapml-viewer> I fixed this by setting .leaflet-popup-tip-container { margin-top: -1px }. The downside of that fix is that if a developer changes the background-color of the popup they'll see a 1px dent, but in reality they'll probably want to change the background-color of the popup's tip to the same color anyway so I think it's fine. Rather have a robust looking default, IMO.

Way is gap me I am the gap
On Fri, 14 Jan 2022, 23:40 Robert Linder, @.***> wrote:

@mourner https://github.com/mourner #7908 (comment)
https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1013529489:
Off topic, but on your screenshots, there's a 1pm gap between the popup
and its tip — I wonder if that's a bug we should fix too? Is it easily
reproducible?
The gap is often appearing, but inconsistently so.
Even the popup in Chrome's dev tools has the same issue (compare their
popup in this
https://user-images.githubusercontent.com/26493779/149426796-bd7e386c-0781-4c38-9a85-8ad560d1ed4c.png
screenshot to this
https://user-images.githubusercontent.com/26493779/149426960-9620a9f1-a5d4-46be-8954-b8e4a4e9e439.png),
I think the root cause is the same as the 1px gap bug between tiles,
probably a Chromium compositing bug.
In the Maps4HTML  I fixed this by setting .leaflet-popup-tip-container
{ margin-top: -1px }
https://github.com/Maps4HTML/Web-Map-Custom-Element/blob/b44bc709d27ca242c1297cf57c96e3635be08113/src/mapml.css#L585-L587.
The downside of that fix is that if a developer changes the
background-color of the popup they'll see a 1px dent, but in reality
they'll probably want to change the background-color of the popup's tip to
the same color anyway so I think it's fine. Rather have a robust looking
default, IMO.
—
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1013544263,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2UXSGQ2DO3ZG7JEBQUWDDUWCX6BANCNFSM5L5JZ6GQ
.
You are receiving this because you commented.Message ID:
@.***>

Re @mourner's https://github.com/Leaflet/Leaflet/pull/7908#issuecomment-1013529489, I've opened a separate PR: https://github.com/Leaflet/Leaflet/pull/7920.

It'd be great to have a decision for this before the final v1.8.0 release, now that outlines are showing on keyboard focus (https://github.com/Leaflet/Leaflet/pull/7259), since the current outline on the close button looks pretty bad.

I had currently not the time to check, but I will do!
As reminder, we should check if the close button is now displayed better when the popup is empty:


Add how to integrate with Angular 2

Type of issue

[ ] Bug report
[x] Feature request

Uploader type

[x] Traditional
[ ] S3
[ ] Azure

Feature request details
When I wanted to integrate your plugin into my app I couldn't find how to do this in docs. Then I found this repo in issues. If you at least reference it in docs it would save other FineUploader + Angular2 users more time.

ES6, CommonJS, TypeScript and Angular support documentation is listed here https://docs.fineuploader.com/branch/master/features/modules.html

Yes they are, but there is nothing how to really use them inside component. It's better to see it in working example, like in your repo. Well at least for me it was helpful and I wanted to share that.

Well, there isn't anything special about using FU inside an Angular component, but still if you need more reference I recommend looking at this file.
My repo is pretty old and probably outdated as I was using it just for testing purposes.
Hope this helps


[FALSE-POSITIVE]

Domains or links
https://onecred.pw
More Information
Incorrectly marked as Phishing on Phishtank
Have you requested removal from other sources?
https://phishtank.com/phish_detail.php?phish_id=6752766
Additional context
It was a mistake. Site is 100% clean. There isn't a login form or something like that.
It is just an ad board. Users add their ads.

Listed by multiple engines

And what? All of these engines added our site from phishtank's database like you. And all of these engines haven't answered about checking our site out again.
Could you open this link please? https://phishtank.com/phish_detail.php?phish_id=6752766
It was the first source. You added our site in your base because of it. But now our site was deleted from that database and you should do the same actions to and you shouldn't look at virustotal.

Listed by multiple engines

And what? All of these engines added our site from phishtank's database like you. And all of these engines haven't answered about checking our site out again.
Could you open this link please? https://phishtank.com/phish_detail.php?phish_id=6752766
It was the first source. You added our site in your base because of it. But now our site was deleted from that database and you should do the same actions to and you shouldn't look at virustotal.

Listed by multiple engines

Do you know what is phishing site mean? Do you know that an information in the virustotal may be wrong?
Please look at our site again. There isn't any form or some inputs! It is not a phish.