setValue method of Hypergrid failing with TypeError: Cannot read property 'subgrid' of undefined

When I use these two lines in my code then I get an error: TypeError: Cannot read property 'subgrid' of undefined
// Error!!!!
grid.setValue(colPos['bob'], gridRow+1, rowData[i].bob);
grid.setValue(colPos['pi'], gridRow+1, rowData[i].key);
But when I use these two lines in my code then everything works fine
// No error!!!!!
grid.behavior.dataModel.setValue(colPos['bob'], gridRow, rowData[i].bob);
grid.behavior.dataModel.setValue(colPos['pi'], gridRow, rowData[i].key);
Is the grid.setValue method the one to use or is the grid.behavior.dataModel.setValue method the one to use?
The error only seems to happen when the height of the div containing the Hypergrid is smaller than the height that the Hypergrid eventually grows to as rows are dynamically added.

@nsivraj We can address your concerns later this week

@nsivraj We have a ticket for this in our backlog. You will receive an update when it is in our sprint


[JENKINS-36532] General refactor

Related to issue [JENKINS-36532]
Summary of this pull request:

[X] Include the changes suggested in #1
[X] Use DataBoundSetter where makes sense, to minimize the output of Snippet Generator code
[X] Rename public exposed method API to select(run, context)
[x] The parameters that are mandatory in the RunSelectorContext should be set in the constructor. Only optional parameters, like verbose should have setters

CC @ikedam @oleg-nenashev @martinda

Remove ParameterizedRunSelector, since it supports only variable names, and not variable expressions

Sorry, I confused you.
ParameterizedRunSelector supports both variable names (FILTER) and variable expressions (${FILTER}).
You can deprecate and remove the variable name feature (FILTER) and have it support only variable expressions (${FILTER}).

JFYI: assertEqualDataBoundBeans in jenkins-test-harness 2.12 also checks properties with DataBoundSetter.
You can switch to it by setting jenkins-test-harness.version in pom.xml (See https://github.com/jenkinsci/plugin-pom for details).

I would vote for RunSelectorContext initialization refactoring. The current implementation seems to be a bit fragile

You can deprecate and remove the variable name feature (FILTER) and have it support only variable expressions (${FILTER}).

I've dropped the commit in which I've completely removed the ParameterizedRunSelector.
But I'm not sure what changes I should make. In the resolveParameter() method, should I replace the following lines:
String xml = context.getEnvVars().get(getParameterName());
if (xml == null) {
    context.logInfo("{0} is not defined", getParameterName());
}
return xml;

with
return null;

?
I don't understand well why to remove this feature. People are not using it?

Please use @Restricted(NoExternalUse) for DataBound setters, which are not supposed to be a part of public API.

I don't understand well why to remove this feature. People are not using it?

The variable expression feature was introduced for pipilenes (aka. workflow): https://issues.jenkins-ci.org/browse/JENKINS-30357

The variable name feature remains for the backward compatibility.

The varable expression feature can be alternate for the variable name feature.
These two features can be conflicted.

VAR1=VAR2, VAR2=blahblahblah, and ${VAR1} will be expanded to blahblahblah.

Confusing if there're two ways for same configurations.
More tests are needed.
ParameterizedBuildFilter supports only variable expressions.

And I think you can remove this feature.
On the other hand, you can continue to support this feature.
The variable expression feature is introduced recently, and many users of copyartifact may still use variable names.

But I'm not sure what changes I should make.

The way you wrote is right.

Looks good to me.


fix job edit form silently failing

Closes: DNE
What's Changed

The slug field was removed from the form template but not the form, so when editing a job the form would always fail because it expected a slug that cannot be provided.
TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

The Job model still has a slug field - should the correct fix be to add the form field back to the template, instead of removing it from the form?

Is this the reason why UT is failing in next-ui as well?

The Job model still has a slug field - should the correct fix be to add the form field back to the template, instead of removing it from the form?

I think so. We'll eventually be able to remove the slug field on Job but since it's still there I agree that the appropriate fix is to re-add the field to the html template.

Is this the reason why UT is failing in next-ui as well?

It is. I made this change on that branch so that we could enable/run jobs to populate the job results table on the new home page. I'll fix it over there too.


Sample Config File is Not Created

Problem: Pulled Docker ARMv7 image and during initial container first boot the sample config file is not copied over to the respective location. Verified there is no file in /config
panic: Config File "seekerr" Not Found in "[/config]"

goroutine 1 [running]:

github.com/lightglitch/seekerr/cmd.initConfig()

	/go/src/github.com/lightglitch/seekerr/cmd/root.go:92 +0x350

github.com/lightglitch/seekerr/cmd.glob..func1(0x9cbd40, 0xab763c, 0x0, 0x0)

	/go/src/github.com/lightglitch/seekerr/cmd/cron.go:40 +0x14

github.com/spf13/cobra.(*Command).execute(0x9cbd40, 0xab763c, 0x0, 0x0, 0x9cbd40, 0xab763c)

	/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:849 +0x3a4

github.com/spf13/cobra.(*Command).ExecuteC(0x9cbe88, 0x0, 0x1, 0x1)

	/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:974 +0x280

github.com/spf13/cobra.(*Command).Execute(...)

	/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:902

github.com/lightglitch/seekerr/cmd.Execute()

	/go/src/github.com/lightglitch/seekerr/cmd/root.go:50 +0x20

main.main()

	/go/src/github.com/lightglitch/seekerr/main.go:27 +0x14

Testing and Troubleshooting: I manually created seeker.yaml file and copied contents of seekerr.sample.yaml into it and it started successfully.
2:37AM INF start
2:37AM INF schedule 0=now 1=2022-02-12T21:37:21.680945865-05:00 2=entry 3=1 4=next 5=2022-02-12T22:00:00-05:00

As you can see here https://github.com/lightglitch/seekerr#docker you will need to create the file manually.

As you can see here your Dockerfile should copy it over. There should be no reason to manually do so.
RUN mkdir /config
COPY --from=build /go/src/github.com/lightglitch/seekerr/config/seekerr.sample.yaml /config/seekerr.sample.yaml

Issue closed prematurely. Thanks

It only copies the sample. It doesn't create the file seekerr.yaml that is necessary to run the docker, the exception was because that file didn't exist. The seekerr.yaml needs to be created manually like it says in the documentation.


Tried to improve the README.md, hope you like it.

I would likemy api to be mentioned in the README.md, since it's very well documented.
But of course I will leave that up to you.

Since a lot of devs would prefer server-side animation instead of low-level control, mentioning that is a good idea.

Looks good?

I think I'll re-write some of this later, move some stuff into github wiki. But first 1.21.3 port.


Aks networking permission

Add more clarification as what AKS SP needs for custom vnet scenario

@weinong : Thanks for your contribution! The author, @mmacy, has been notified to review your proposed change.

@weinong : Thanks for your contribution! The author, @mmacy, has been notified to review your proposed change.

LGTM. I'll tweak it a bit with relative links to the .md files once it hits the private repo, but it's good as-is and should be merged.
Thanks for the update, @weinong, this will make AKS customers happy!
#sign-off


autocomplete results hidden when the component used inside ion-list

in this situations
  <ion-list>
       <ion-item >
       <ion-auto-complete ...> </ion-auto-complete>
       </ion-item>
  </ion-list>

the result list is hidden (overshadowed)

This is an ionic issue, solved by adding a item-content attribute to the auto-complete component.
See https://github.com/ionic-team/ionic/issues/6880

the item-content  doesn't work for me. fortunately i found css work around
.overflow{   contain:none;   overflow: visible; }
then
  <ion-item class="overflow">    <ion-auto-complete ...> </ion-auto-complete>        </ion-item>

if the auto-complete is inside an ion-card I had to add this to the card: style="overflow: visible;"


For the attention model why we need last context?

In this https://github.com/spro/practical-pytorch/blob/master/seq2seq-translation/seq2seq-translation.ipynb,
why do we need to feed the last_context as an rnn input?
       # Combine embedded input word and last context, run through RNN
        rnn_input = torch.cat((word_embedded, last_context.unsqueeze(0)), 2)
        rnn_output, hidden = self.gru(rnn_input, last_hidden)

Actually, context is used in the output layer:
        output = F.log_softmax(self.out(torch.cat((rnn_output, context), 1)))

In your explanation:
RNN layer(s) $f$ with inputs $(s_{i-1}, y_{i-1}, c_i)$ and internal hidden state, outputting $s_i$
rnn_input = concat(embedded, context)
rnn_output, rnn_hidden = rnn(rnn_input, last_hidden)
an output layer $g$ with inputs $(y_{i-1}, s_i, c_i)$, outputting $y_i$
But in the RNN, you are using c_{i-1}? Also in the original paper, they did not use c for RNN, right?
Thanks!

Hi @spro, I am having the same question, could you please point out which sentence mentioned the $g$? Thanks! The paper is at https://arxiv.org/pdf/1508.04025.pdf


The generated hair has low resolution

Hi,first thank you for your work!
I found the generated hair region has low resolution than the target image,the target image and the reference image are both 512*512,what can I do to solve this problem?
(To protect the privacy of the photographer, I added mosaics to the face)

It seems that you did not do the face alignment for the input image. By the way, I found that the alignment parameters may have a bit different from the way used in StyleGAN. Anyway, I had used the images provided in this repo to recover the alignment parameters and it seems work OK.
However, since MichiGAN would use the orientation map rather than the raw hair pixels from the image as the input information, I doubts that whether it would ensure to get a fully recover image when you did not excepted any change.


Update ISA_ACTION-Template.ipynb

[x] dropping "ISA otherMaterial"     (which can only be either "Extract Name" or "Labeled Extract Name"

### Chromatography
#             separated_molecules = Material(
#                 name = "new_separated_molecules_{0}".format(row["XOmicsmetaboID"],
#                 type_ ="Labeled Extract Name")
#             )

[x] derives_from attribute requires a list of objects
[x] for the ISA-API: changing error message returned when coming across a Material type which is not supported.

A great many thanks!


Is there a way to perform implicit joins using navigation properties (a-la EF)?

Will link this issue here, to see if I get better luck

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

ID: 89464513-67cd-0821-c0b8-5130bf8b2d16
Version Independent ID: 7e8041b0-7d9c-4d68-67ef-77b6ec29a146
Content: Linq Query Examples (Common Data Service) - PowerApps
Content Source: powerapps-docs/developer/common-data-service/org-service/linq-query-examples.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@fberasategui
What are you trying to do and what results do you get?
You should try to use one of the lower-level query capabilities (QueryExpression or FetchXml). If you can't get what you want with those, you won't be able to get it with LINQ.

@JimDaly Okay. First of all thanks for your attention.
I've created a repro.
As you can see, this is the query I'm trying to perform:
var accounts =
       ctx.CreateQuery<Account>()
          .Where(x => x.account_primary_contact.FirstName.StartsWith("John"))
          .ToList();

The query itself is pretty simple, in plain english:

Get me all Accounts whose Primary Contact's First Name Starts with "John"

In Advanced Find (sorry for the spanish here)

Which yields the following FetchXml:
<fetch version="1.0" output-format="xml-platform" mapping="logical" distinct="false">
  <entity name="account">
    <attribute name="name" />
    <attribute name="primarycontactid" />
    <attribute name="telephone1" />
    <attribute name="accountid" />
    <order attribute="name" descending="false" />
    <link-entity name="contact" from="contactid" to="primarycontactid" link-type="inner" alias="ab">
      <filter type="and">
        <condition attribute="firstname" operator="like" value="John%" />
      </filter>
    </link-entity>
  </entity>
</fetch>

Which, of course, works just fine.
Now, the LINQ query throws the following exception:

with the following stack trace:
   at System.Runtime.Remoting.Proxies.RealProxy.HandleReturnMessage(IMessage reqMsg, IMessage retMsg)
   at System.Runtime.Remoting.Proxies.RealProxy.PrivateInvoke(MessageData& msgData, Int32 type)
   at Microsoft.Xrm.Sdk.IOrganizationService.Execute(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceProxy.ExecuteCore(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceProxy.Execute(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceContext.Execute(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Linq.QueryProvider.RetrieveEntityCollection(OrganizationRequest request, NavigationSource source)
   at Microsoft.Xrm.Sdk.Linq.QueryProvider.Execute(QueryExpression qe, Boolean throwIfSequenceIsEmpty, Boolean throwIfSequenceNotSingle, Projection projection, NavigationSource source, List`1 linkLookups, String& pagingCookie, Boolean& moreRecords)
   at Microsoft.Xrm.Sdk.Linq.QueryProvider.Execute[TElement](QueryExpression qe, Boolean throwIfSequenceIsEmpty, Boolean throwIfSequenceNotSingle, Projection projection, NavigationSource source, List`1 linkLookups)
   at Microsoft.Xrm.Sdk.Linq.QueryProvider.Execute[TElement](Expression expression)
   at Microsoft.Xrm.Sdk.Linq.QueryProvider.GetEnumerator[TElement](Expression expression)
   at Microsoft.Xrm.Sdk.Linq.Query`1.GetEnumerator()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at ConsoleApp6.Program.Main(String[] args) in C:\Users\FedericoBerasategui\source\repos\ConsoleApp6\ConsoleApp6\Program.cs:line 20

which suggests me that it's mistakenly picking the "John%" value and using it as a filter value for the primarycontactid attribute, instead of using it for the firstname attribute as in the FetchXml above. I'll see if I can intercept the exact FetchXml that is getting generated here and come back with that.
Please notice I've provided a clean, as-vanilla-as-possible repro:

Using a vanilla GeneratedCode.cs, generated using latest CrmSvcUtil <IP_ADDRESS> from a vanilla D365 instance, as follows:
  CrmSvcUtil.exe /url:https://myorg.crm.dynamics.com/XRMServices/2011/Organization.svc    /out:GeneratedCode.cs /username:username /password:password

Using only Nuget Package Microsoft.CrmSdk.CoreAssemblies, latest version <IP_ADDRESS>

Steps to reproduce:

Clone the repo: https://github.com/fberasategui/cds-linq-issue-1054
Open ConsoleApp6.sln in Visual Studio 2019
Edit lines 31-33 of file Program.cs providing connection details (OrganizationService Url, username and password)
F5

@JimDaly update:
I've intercepted down the RetrieveMultipleRequest that's generated by the LINQ provider:

Notice how there are no LinkEntities and it effectively uses the "John%" value for the wrong attribute primarycontactid.
This suggests a long-overlooked bug in the LINQ provider.

@fberasategui
I appreciate the detailed repro steps.
Typically we don't accept bugs via the documentation feedback process. It is for feedback on the content, not for reporting bugs. Typically, you should open a support request to report issues.
If you report one of the samples published doesn't work - that would be something that should be fixed in the docs. But you are describing a bug.
Although I'm the author of many SDK topics, I'm not the PM owner for all the features. I wrote these samples many years ago.  But considering the effort you provided to supply repro steps,  I'll try to locate who currently owns this part of the SDK and ask them to take a look.

I've opened a bug for this (including all the details of your repro steps) and assigned it to the team that owns the SDK.
But since this isn't an actionable doc issue (yet) I'll close this out.
If they report it is by-design, we can identify the limitations and document them.
I've included a link to this issue in the bug.

@fberasategui @JimDaly   I think you're using the name of the relationship (account_primary_contact), not the name of the lookup field (primarycontactid), if you use that attribute in the Where clause it should then filter on "primarycontactid" as it does with fetchxml..

@JimDaly did we have an outcome from that bug?
I'm getting similar errors but LINQ Limitations still doesn't mention anything about navigation properties being unsupported.


Add test coverage for the native samples

Add some automation to ensure that the samples continue to work as we make changes to Gradle.

The simplistic test in the sample repo helps prevent breakage between Gradle version. Should we rescope this issue into something more concrete or close it?

We're missing a few things:

Windows coverage (for the C++ parts).
A CI build that runs these tests on any change to the samples
Ideally, a CI build that runs these tests on any change to the build tool (or some subset of the build tool, eg the native plugins).
Do the above on MacOs.

We can split these out into separate issues and maybe add an epic over the top and close this issue.

There's a lot of discussions that can take place with regards the listed test coverage. I will convert this issue to an epic and move it to future until we decide to do the work.

I'm wondering if we should go ahead and bite the bullet on this. It seems like one of those infrastructure projects that if we don't do soon-ish, it will become a "big project" that we end up not doing. As we add more and more samples, it seems like this is going to feel larger and more ominous.  I advocate for moving it to the "Soon" column.

We now run some basic checks on the native samples.


crash when i entered to the icons menu

Platform
Windows
Geode Version
v1.3.3
Mods Installed
x | [v1.3.3] geode.loader
x | [v1.0.1] mat.run-info
x | [v7.1.1-beta.5] absolllute.megahack
x | [v1.0.2] hjfod.gmd-api
x | [v1.0.2] alphalaneous.separatedualicons
x | [v1.0.1] hjfod.gdshare
x | [v3.0.5] cvolton.betterinfo
Steps to Reproduce

open the game
click to the icons menu
and crash

Crash Report
https://gist.github.com/Abuigsito/4a97cb98cf948dcab7b5450c51c635e5
Additional Information
i have windows 7 (lol)

Bug caused by Alphalaneous' separate dual icons mod, report it to them

:trollface:


Board Review: Azure Communication Services (SPOOL) - Calling Native - Closed captions (iOS & Android)

Service team responsible for the client library.

ACS Turnkey Cognitive Services
ACS Calling

Link to documentation describing the service.

https://docs.microsoft.com/en-us/azure/communication-services/concepts/voice-video-calling/closed-captions

Contact email (if service team, provide PM and Dev Lead).

EM - bhnambur
PM - EKTRISHI

Name of the client library.

Azure.Communication.Calling

Languages for this review.

iOS
Android

Artifacts per language (API View links):

Android
iOS

Champion scenarios:

Android
iOS

scheduled for 2/8 2-4pm pst


Add tabsize and eol

Fixes: #85

In the bicep variant they use editorconfig. There's a VScode extension for this.
Also we should consider .gitattributes as well to force unix line endings.

In the bicep variant they use editorconfig. There's a VScode extension for this.
Also we should consider .gitattributes as well to force unix line endings.

Yes please include

https://github.com/Azure/bicep-registry-modules/blob/main/.editorconfig

and a .gitattributes with the following suggested content
*.tfvars  text eol=lf
*hcl text eol=lf
*.go text eol=lf
*.tf text eol=lf

In the bicep variant they use editorconfig. There's a VScode extension for this.
Also we should consider .gitattributes as well to force unix line endings.

Yes please include

https://github.com/Azure/bicep-registry-modules/blob/main/.editorconfig

and a .gitattributes with the following suggested content
*.tfvars text eol=lf
*hcl text eol=lf
*.go text eol=lf
*.tf text eol=lf

But I also agree to simply just having the suggested Workspace settings.json from @LaurentLesle because no extension is required and VSCode will inherit it right away.

@LaurentLesle any update on this one?

@LaurentLesle how are we doing with this one? Do you need any support to include all changes?


Don't bindmount vfs volume into the share dir of the vm

let the runv do it.

but,
IIRC, runc do not care about mounting filesystems, right?

it has not yet been implemented....


Update STATIC_PATH path in index.js

In production, STATIC_PATH should be 'web/frontend/dist'

Hi @roman-gavrilov
Thank you for your contribution. The static path is using the dist folder after running yarn build in this template and it's web/frontend/dist. You can change it in your app after using the Shopify CLI to create your own version using template. Therefore, I'll close this PR as not applicable.


RTCPeerConnectionWeb's connectionState is calculated based on iceConnectionState vs connectionState

RTCPeerConnectionWeb's connectionState is updated by onIceConnectionStateChange and is calculated based on iceConnectionState. The more appropriate onConnectionStateChange and connectionState aren't exposed via dart2js (issue).
Filing this issue to tracking updating to use those once added.
The effect of using iceConnectionState is that since RTCIceConnectionState has an extra 'checking' state vs RTCPeerConnectionState, peerConnectionStateForString reports RTCPeerConnectionStateClosed  for the unknown checking state.

https://github.com/dart-lang/sdk/issues/48735 is available with Flutter 3.3


Issues with spell checking

Microsoft PowerToys version
0.64.1
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
General, Quick Accent
Steps to reproduce
I have two problems with spell checking in PowerToys. One is linked to Quick Accent, the other one not.
The problem I have with Quick Accent is described in the following fields.
For the other one, while I am typing, I see tooltips that propose me several other typings. In one case they have been modal, and none of the proposed values were correct, this has led to crash Firefox, and I have lost everything I was typing. Further more, sometimes those tooltips take place on the screen near the keyboard cursor and prevent me from reading what is behind.
✔ Expected Behavior
If I type "ça m'oblige à" (in a French text), I should expect that the text "ça m'oblige à" is kept, it is perfectly well written.
❌ Actual Behavior
"ça m'oblige à" is replaced by "ça m'obligéà", which has absolutely no meaning in French, except that it suggests a "text typed with the feet", as my neighbour says.
Other Software
No response

I realize I had a too short delay to activate Quick Accent. With a longer delay the problem is solved.
Can you please do something about the font size?

Tooltips that propose other typings are not part of PowerToys, I believe? Not sure what to do here.
So, I guess this issue is about requesting to increase the Quick Accent font size? Is that correct?
/needinfo

Hello,
In fact, there are two issues, as there are two sorts of intotips.
On one side, once you trigger Quick Accent by a letter and a space, you get a list of possible accents on that letter.

It would be nice that it could be displayed that size.
On the other side, while I type, I see tooltips to suggest me spellchecking modifications:

I am disappointed to learn that it does not come from PowerToys, as then I do not know where.
I should have sweared it appeared after the installation.

I am disappointed to learn that it does not come from PowerToys, as then I do not know where from.

Looks like the OS built-in function.

Hello, In fact, there are two issues, as there are two sorts of intotips. On one side, once you trigger Quick Accent by a letter and a space, you get a list of possible accents on that letter.  It would be nice that it could be displayed that size. On the other side, while I type, I see infotips to suggest me spellchecking modifications:
I am disappointed to learn that it does not come from PowerToys, as then I do not know where from. I should have sweared it appeared after the installation.

I have the same issue, It suddenly appears to me out of no where this tooltip, and I'm sure it came after installation.
Also, I get annoyed with auto-correction like when I type sure word I end up with suré, while I'm using only English qwerty keyboard layout.
So, I had to uninstall it.

when I type sure word I end up with suré,

That's probably because of Quick Accent.
@crutkas we should be careful for this unexpected behavior for new modules which are in by default.

Oh, it seems we shall have to deal with several problems and take care of avoiding to mix them up.
There is a following of a few infotips that appears at a certain position of the screen, as chosen in the configuration of Quick Accent. They all show the same letter, and each with an accent on it. This appears since last update, once I understood the delay of Quick Accent must not be at the minimum. You can choose among those accents, with a key depending on the Quick Accent configuration.
There is an infotip that appears near the text I am typing, that proposes some variations from the text I typed. I have this for some years. It was interrupted a few hours after the update. (see my screen copy "text modification", where the infotips propose "texte", "TeX", "textes". Normally you can click on one to choose it. As I have a spell checker in each application, I should like to deactivate this. At least, chose another place on the screen could help, except in the case it became modal (this was with the last version of Firefox, 107).

About sure that became suré: I had that when I set the delay of Quick Accent too short. And the space after that was suppressed.
It is even more strange in the case French is not in the chosen languages for Quick Accent.
Abdulateef, do you have infotips, on your screen? Are they near what you type, or at the place selected in the configuration of Quick Accent?
Did you put a sufficient delay in the configuration of Quick Accent?

Sorry to respond late,
I had infotips on my screen, and they are near to what I type.
but I did not configure anything of Quick Accent, and I don't even what Quick Accent means.
@Gluups

OK, then I am just going to answer your last question, as we are speaking about two categories of infotips, and one of them is related to Quick Accent.
On a system where PowerToys has recently been updated, the PowerToys Parameters window shows categories on the left, the three first ones are General, Always Visible, Awake.  Further down, the 12th is Power Rename, the 13th PowerToys Run, and the 14th Quick Accent.
It is quite probable that in English you do not have the use of it, but many languages put accents on some letters. On one's native language you generally have most of the accents on the keyboard, but we live in an international context where you can have either to type a text in another language, or you have bought a computer in another country, and it can happen the accents you need are not on the keyboard. Quick Accents aims to bring a solution at that, and does it rather good.
All this considered on a Latin alphabet, I am afraid I am not able to speak a lot of something else.
If you want to have a try of Quick Accent, pay attention to set a not too short time.
You will need a quarter of an hour to put everything in place and have an occasion to understand what it does.

So if you have problem it is with the spell checker, it seems recently nobody was able to tell how to set its parameters.

I may re-install it again and see if I still get the same issue and try to understand what really happens,
thank any way.

OK, if you observe something please tell us.
I saw that spellchecker documented nowhere, and it remembered nothing to the developer of Quick Accent.
Thank you.
Perhaps I shall also have to reinstall PowerToys for another reason, I shall open a thread about it anyway.
It is not something to do directly on the impulse, as it takes some time during which the machine is not available.


feat: delete es field source

Codecov Report

Merging #267 (4888458) into main (341faaf) will increase coverage by 0.02%.
The diff coverage is 38.09%.

@@            Coverage Diff             @@
##             main     #267      +/-   ##
==========================================
+ Coverage   14.77%   14.79%   +0.02%
==========================================
  Files         114      114
  Lines       14570    14575       +5
==========================================
+ Hits         2153     2157       +4
- Misses      12248    12250       +2
+ Partials      169      168       -1

Impacted Files
Coverage Δ

pkg/runtime/entity.go
56.00% <ø> (ø)

pkg/service/entity.go
37.48% <0.00%> (-0.22%)
:arrow_down:

pkg/runtime/nodeflush.go
53.84% <100.00%> (+2.06%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 341faaf...4888458. Read the comment docs.


First draft of regions doc

A bit wordier than the other documentation, but I think that regions are a bit harder to grasp to the casual user, so a clear example has been set. Items and Locations are integral to the basic AP experience, regions not so much.

@Rooby-Roo Had some small revisions requested. Once those are sorted, we can merge. 😄

Yep, I knocked this one out over the course of a couple of free hours at work using the github quick pr settings. It's been a hot minute since I actually had the time to go back and look at this (and, more importantly, actually set up the repo correctly on my home screen so I can edit this.)
My schedule is weird next week, I'll see if I can come back to this soon.

Uhh, I didn't realize I could make changes directly to your fork's main branch. Guessing that was something you granted when creating your fork, or when creating the PR. Was a new experience for me, haha
Anyways, I went ahead and made the few minor changes to push this one through. If you want to rephrase/edit in any way, open a new PR and go for it 😄

Zero worries lol, I don't use my local fork for anything except PRs, so anything that needs to get reset will get upset next time I do another upstream.

Uhh, I didn't realize I could make changes directly to your fork's main branch. Guessing that was something you granted when creating your fork, or when creating the PR. Was a new experience for me, haha

It's because this was an open PR.   It's very helpful when resolving merge conflicts, but can be used for things like this too :)


bot account have to start csgo first or it will stuck at "Successfully registered playing CS:GO. Sending client hello to CS:GO services."

bot account have to start csgo first or it will stuck at "Successfully registered playing CS:GO. Sending client hello to CS:GO services."

in the same situation


feat: add new guild helper methods

About
This pull request is about two new guild helper methods for some uncommon http methods
New methods:
Guild.get_list_of_members()
Guild.search_members()
Checklist

[x] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue.

(If existent): #646

[x] I've made this pull request for/as: (check all that apply)

[x] Documentation
[ ] Breaking change
[x] New feature/enhancement
[ ] Bugfix

Title.

oop ty

Title.

Resolved, you can resolve or whatever now


Using fillcontinents with color causes error

I was following the following guide a bit: https://matplotlib.org/basemap/users/examples.html but when I tried the following simplified code:
# for deprecation warning check: https://github.com/matplotlib/basemap/issues/382

import numpy as np
from mpl_toolkits.basemap import Basemap  # basemap package
import matplotlib.pyplot as plt

# miller projection
map = Basemap(projection='mill',lon_0=0)
# plot coastlines, draw label meridians and parallels.
map.drawcoastlines(linewidth=0.5)
map.drawparallels(np.arange(-90,90,30),labels=[1,0,0,0], linewidth=0.3)
map.drawmeridians(np.arange(map.lonmin,map.lonmax+30,60),labels=[0,0,0,1], linewidth=0.3)
# fill continents 'coral' (with zorder=0), color wet areas 'aqua'
map.drawmapboundary(fill_color='white')
map.fillcontinents(color='coral',lake_color='aqua')
plt.show()

I got an error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-6-faa602d3f27d> in <module>()
     14 # fill continents 'coral' (with zorder=0), color wet areas 'aqua'
     15 map.drawmapboundary(fill_color='white')
---> 16 map.fillcontinents(color='coral',lake_color='aqua')
     17 #map.drawlsmask(color="coral")
     18 # shade the night areas, with alpha transparency so the

~/apps/anaconda3/envs/data-analyst-track-module-2/lib/python3.6/site-packages/mpl_toolkits/basemap/__init__.py in fillcontinents(self, color, lake_color, ax, zorder, alpha)
   1765         ax = ax or self._check_ax()
   1766         # get axis background color.
-> 1767         axisbgc = ax.get_axis_bgcolor()
   1768         npoly = 0
   1769         polys = []

AttributeError: 'AxesSubplot' object has no attribute 'get_axis_bgcolor'

Used versions of libraries are (output of conda list, shortened):
numpy                     1.13.3           py36hdbf6ddf_4
basemap                   1.0.7               np113py36_0
matplotlib                2.2.2            py36h0e671d2_0

update your install of basemap. Get it from conda-forge. It has the fix for
this.
On Fri, Apr 6, 2018 at 5:58 AM, Zelphir Kaltstahl<EMAIL_ADDRESS>wrote:

I was following the following guide a bit: https://matplotlib.org/
basemap/users/examples.html but when I tried the following simplified
code:
for deprecation warning check: https://github.com/matplotlib/basemap/issues/382
import numpy as npfrom mpl_toolkits.basemap import Basemap  # basemap packageimport matplotlib.pyplot as plt
miller projectionmap = Basemap(projection='mill',lon_0=0)# plot coastlines, draw label meridians and parallels.map.drawcoastlines(linewidth=0.5)map.drawparallels(np.arange(-90,90,30),labels=[1,0,0,0], linewidth=0.3)map.drawmeridians(np.arange(map.lonmin,map.lonmax+30,60),labels=[0,0,0,1], linewidth=0.3)# fill continents 'coral' (with zorder=0), color wet areas 'aqua'map.drawmapboundary(fill_color='white')map.fillcontinents(color='coral',lake_color='aqua')
plt.show()
I got an error:

AttributeError                            Traceback (most recent call last)
 in ()
14 # fill continents 'coral' (with zorder=0), color wet areas 'aqua'
15 map.drawmapboundary(fill_color='white')
---> 16 map.fillcontinents(color='coral',lake_color='aqua')
17 #map.drawlsmask(color="coral")
18 # shade the night areas, with alpha transparency so the
~/apps/anaconda3/envs/data-analyst-track-module-2/lib/python3.6/site-packages/mpl_toolkits/basemap/init.py in fillcontinents(self, color, lake_color, ax, zorder, alpha)
1765         ax = ax or self._check_ax()
1766         # get axis background color.
-> 1767         axisbgc = ax.get_axis_bgcolor()
1768         npoly = 0
1769         polys = []
AttributeError: 'AxesSubplot' object has no attribute 'get_axis_bgcolor'
Used versions of libraries are (output of conda list, shortened):
numpy                     1.13.3           py36hdbf6ddf_4
basemap                   1.0.7               np113py36_0
matplotlib                2.2.2            py36h0e671d2_0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/matplotlib/basemap/issues/394, or mute the thread
https://github.com/notifications/unsubscribe-auth/AARy-PRWF6Sfbngtu4ZDt2Z5yCvE5SDpks5tlzxjgaJpZM4TJzUP
.

That solved it. Thank you!


fix missing input in upgrade pipeline

Apply changes task in upgrade pipeline is failing because the jq input is missing in pipeline configuration.
https://github.com/rahul-kj/pcf-concourse-pipelines/blob/master/tasks/apply-changes/task.yml#L16-L19

thanks @Infra-Red !! Apparently this change was already done, but appreciate the fix


Unable to upgrade to polymer_elements 1.0.0-rc.9

I get the following error when I try to upgrade to 1.0.0-rc.9
Resolving dependencies...

polymer_elements 1.0.0-rc.9 (was 1.0.0-rc.8)
Error on line 198, column 1 of ......\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\polymer_elements-1.0.0-rc.9\pubspec.yaml: Duplicate mapping key.
environment:
^^^^^^^^^^^
Process finished with exit code 65

This started happening when I took the latest version of dart 1.23. Not sure if that is a contributing factor

I'm getting this issue too.
I believe this pull request fixes it, but the pull request has been sitting there for a couple months already and I'm starting to get worried if it will ever get merged.

This should be resolved now, please re-open if it is not


Drop support for old Delphi?

{$IFDEF D3} {$DEFINE UseAsserts} {$ENDIF}
{$IFDEF FPC} {$DEFINE UseAsserts} {$ENDIF}
// Define 'use subroutine parameters default values' option (do not edit this definition).
{$IFDEF D4} {$DEFINE DefParam} {$ENDIF}
{$IFDEF FPC} {$DEFINE DefParam} {$ENDIF}
// Define 'OverMeth' options, to use method overloading (do not edit this definitions).
{$IFDEF D5} {$DEFINE OverMeth} {$ENDIF}
{$IFDEF FPC} {$DEFINE OverMeth} {$ENDIF}

it's not good, maybe drop D5 and older? D4 and older? code not nice with ifdefs.

What exactly is the problem?
We have very simple and easy to understand piece of code.
I agree there is little sense to suppost Delphi before 7.
But what's the harm in this code?


[Volunteer] admin data validate

Volunteer help
validate the admin submit data with annotation at the DTO, like the length of the data.

can I have a try?

can I have a try?

you get it


Remove unnecessary Result from return type in NamePreserver

Which issue does this PR close?
Follow up of #12341
Rationale for this change
The save and restore functions always return OK, so the Result return type becomes redundant.
What changes are included in this PR?

Are these changes tested?
Yes

Are there any user-facing changes?
No

Makes sense to me.
Should this be tagged with api change since technically NamePreserver and SavedName are technically exposed in the public API?

I think you are right -- done!


Is there a way to proactively fetch oauth tokens?

Describe the feature request
I am using  client.oauth2 configuration to query a backend API with Bearer token in my config. It seems that once the token expires, the next check will fail. However, subsequent attempts with a new, valid token should be okay. I am wondering if there is a possibility to obtain the token in advance before the previous one expires?
Why do you personally want this feature to be implemented?
To have dashboards without false positives and failed checks.
How long have you been using this project?
6 months
Additional information
No response

What permissions / roles / scope does this bearer token have? Maybe a token with less permissions may allow the token to live longer? Refresh tokens may also be something to consider.  It all depends on the team that owns the generations of tokens however.


Issue107

Thanks for the fruitful discussion! #107 also irritated me first - until I realized there was no bug (as far as I see). However, as suggested in #102 we should investigate in tutorial to make the current feature more visible to the public - and to find more bugs!
Here, I try to make clear how to use the RelativeName Scope provider... Have a look and make suggestions what examples to add or which part pf the text to reformulate...

I need some time to read #111 .
But I think it is mandatory to separate

global scoping (FQN, PlainName) ==> lookup based on natural model structure
local scoping ("RelativeName", "RelativeNameExtended") ==> lookup custom and relative, controlled by domain logic, not natural model structure (compare: the new Testcase model example)

Conceptually I do not yet see how to blend them...

I did not adapt scoping.md... Maybe we leave it that way (it uses the deleted examples)?

I propose to postpone this pull request until #111 is discussed.

I propose to postpone this pull request until #111 is discussed.

I agree. This requires some more discussion and design considerations.

For me, this PR could be dropped (it will still be available as "closed" but not "merged"?).
We could maybe put a comment in issue 107 or so...

I agree. This issue should hopefully be fixed properly by #111
Here is some valuable discussion so I'm mentioning #107 to have a link there and also closing.
Thanks for your work. :)


:host(.xxx) is rewritten to `undefined

Description

Live Demo

https://jsbin.com/xukike/edit?html,output
I'll try to add a repo case in JS later. I'm using this in Dart an am not fluent in JS
Steps to Reproduce
Example:
I have this style module which I add dynamically to the element like explained in https://github.com/Polymer/polymer/issues/2681
<!DOCTYPE html>
<dom-module id="bwu-datagrid-default-theme-column-header">
  <template>
    <!--suppress CssOptimizeSimilarProperties -->
    <style>
      :host(.ui-state-default) {
        background: url('../../../packages/bwu_datagrid/asset/images/header-columns-bg.gif') repeat-x center bottom;
        border-right: 1px solid silver;
      }

      :host(.bwu-datagrid-header-column:hover),
      :host(.bwu-datagrid-header-column-active) {
        background: white url('../../../packages/bwu_datagrid/asset/images/header-columns-over-bg.gif') repeat-x center bottom;
      }

      :host(.bwu-datagrid-header-column-sorted) {
        font-style: italic;
      }

      :host(.bwu-datagrid-header-column-sorted) ::content > .bwu-datagrid-column-name {
        border: solid 3px red !important;
        order: 1;
      }
      :host(.bwu-datagrid-header-column-sorted) ::content > .bwu-datagrid-sort-indicator {
        padding-right: 2px;
      }
    </style>
  </template>
</dom-module>

full webcomponents polyfills loaded and shadow DOM enabled
Expected Results

there should be no undefined in the selector
Actual Results

no errors
 :host(.bwu-datagrid-header-column-sorted) ::content > .bwu-datagrid-column-name {

is rewritten to
undefined.bwu-datagrid-header-column-sorted  > .bwu-datagrid-column-name {

Full result copied from Firefox console
<style include="bwu-datagrid-default-theme-column-header" bwu-datagrid-theme="bwu-datagrid-default-theme" is="custom-style">:not([style-scope]):not(.style-scope):host(.ui-state-default) {
  background: url('http://localhost:63342/bwu_datagrid/example_/web/packages/bwu_datagrid/asset/images/header-columns-bg.gif') repeat-x center bottom;
        border-right: 1px solid silver;
}

:not([style-scope]):not(.style-scope):host(.bwu-datagrid-header-column:hover),:not([style-scope]):not(.style-scope):host(.bwu-datagrid-header-column-active) {
  background: white url('http://localhost:63342/bwu_datagrid/example_/web/packages/bwu_datagrid/asset/images/header-columns-over-bg.gif') repeat-x center bottom;
}

:not([style-scope]):not(.style-scope):host(.bwu-datagrid-header-column-sorted) {
  font-style: italic;
}

undefined.bwu-datagrid-header-column-sorted  > .bwu-datagrid-column-name {
  border: solid 3px red !important;
        order: 1;
}

undefined.bwu-datagrid-header-column-sorted  > .bwu-datagrid-sort-indicator {
  padding-right: 2px;
}

</style>

Browsers Affected

[ ] Chrome
[x] Firefox
[ ] Edge (probably - not installed)
[ ] Safari 9 (probably - not installed)
[ ] Safari 8 (probably - not installed)
[ ] IE 11 (probably - not installed)

Works fine in Chrome
Versions

Polymer: v1.4 according to https://github.com/dart-lang/polymer_interop/blob/master/CHANGELOG.md (Dart polymer interop polymer_interop 1.0.0-rc.9)
webcomponents: 0.7.21. according to the changelog in https://github.com/dart-lang/web-components/blob/master/CHANGELOG.md (Dart web_components 0.12.3)

It looks like you're using a :host selector in a style that's included in a custom-style. If so, that's not a supported use case. The :host selector is intended to be used only within a style targeting a custom element. The custom-style element exists to provide shady-dom shimmed styling for the entire document.

Closing based on explanation. If this needs further clarification or was not interpreted correctly, please feel free to re-open with more information.

@sorvell Thanks for the explanation, that makes sense.
It's a powerful way to style elements and to work around a serious limitation of Polymer. Do you have any plans to address these limitations eventually (like demonstrated in #2681)?


add --ssh option

@vito I have built and pushed my own docker image with this change and this works perfectly for my Docerkfile that has --mount=type=ssh. No extra ssh-agent configuration is required.
Regarding your other concern, what other build tools are out there? I only know buildkit and docker buildx that supports passing through --ssh flag so far. For a quick fix, I can rename this variable as BUILDKIT_SSH so we can officially introduce this feature without waiting until everything is perfect.
For your last question, --ssh key is unfortunately only an experimental feature and will require # syntax = docker/dockerfile:1.0-experimental on top of the Dockerfile. You can read more from either in the docker documentation or the buildkit readme.

@vito I've tried to add some tests base on how you have tested other flags. But I am no Go pro... I'm not really sure what to do here. The private and public keys are newly generated just for testing purposes.
Also I have some difficulty running task_test.go on my local. Are we able to create a GitHub Action to automatically run the tests for each PR?
Are you happy to quickly modify my test to the right status or if it's completely wrong, help me to write it?

Regarding your other concern, what other build tools are out there? I only know buildkit and docker buildx that supports passing through --ssh flag so far.

Kaniko is the main one that I know of, but I couldn't easily figure out whether it supports anything like this. The only thing I could find was folks passing the SSH key content itself as a build arg, and I suppose writing it to a file in the Dockerfile.

For a quick fix, I can rename this variable as BUILDKIT_SSH so we can officially introduce this feature without waiting until everything is perfect.

Works for me!

Also I have some difficulty running task_test.go on my local. Are we able to create a GitHub Action to automatically run the tests for each PR?

Makes sense, I'll see if I can set that up tomorrow.

Are you happy to quickly modify my test to the right status or if it's completely wrong, help me to write it?

Sure! Thanks for giving it a shot. The tests are indeed a bit tricky to run. The ./scripts/test script tries to make it easy but it only works on Linux.

(Closing and re-opening to test the Actions workflow.)

Nice to see some progress on this, it will be very useful for fetching from a private repo during a build :smile:
If I understand correctly, this requires that the SSH private key is written to a file and passed as an input to the task. This raises a couple of questions:

Is there a neat way to use a Concourse ((secret)) containing the key with this implementation (other than echoing to a file before running build)?
Are there any security implications to passing the key around as a file instead of as a string?

Hi, this feature is very helpful to us to access private dependencies. Could we accept this chance?


Add an issue_cert method to AcmeIssuingService

Fixes #76.

This change is

Current coverage is 99.94% (diff: 100%)

Merging #78 into master will increase coverage by <.01%

@@             master        #78   diff @@
==========================================
  Files            25         25
  Lines          1818       1897    +79
  Methods           0          0
  Messages          0          0
  Branches        163        171     +8
==========================================
+ Hits           1817       1896    +79
  Misses            1          1
  Partials          0          0

Powered by Codecov. Last update 7156fe1...bb75f45

Review status: 0 of 4 files reviewed at latest revision, 3 unresolved discussions.

src/txacme/service.py, line 75 at r1 (raw file):

    _waiting = attr.ib(default=attr.Factory(list), init=False)
    _issuing = attr.ib(default=attr.Factory(dict), init=False)

Why the extra init=False here?  Is it something that should really have been there all along?

src/txacme/service.py, line 163 at r1 (raw file):

        d = Deferred(lambda _: d_issue.cancel())
        if server_name in self._issuing:
            d_issue, waiting = self._issuing[server_name]

Do we need d_issue here? Nothing in this branch (or after) uses it.

src/txacme/testing.py, line 83 at r1 (raw file):

        self._waiting = []
        if controller is None:
            controller = FakeClientController()

We currently only create a FakeClient in one place and we pass in a controller. Would it make sense to simplify this constructor and either always create the controller internally (and grab it from the client where we need it) or always pass one in?

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 3 unresolved discussions.

src/txacme/service.py, line 75 at r1 (raw file):
Previously, jerith (Jeremy Thurgood) wrote…
> Why the extra `init=False` here?  Is it something that should really have been there all along?

It's an incompatible change to the signature of __init__ but I think being able to pass in a value for `waiting` makes no sense and is very confusing. I think when I wrote this code originally, `attrs` hadn't yet added support for `default=...` combined with `init=False`, so that may be why I originally didn't have it that way.

src/txacme/service.py, line 163 at r1 (raw file):
Previously, jerith (Jeremy Thurgood) wrote…
> Do we need `d_issue` here? Nothing in this branch (or after) uses it.

The code that uses it is in fact located _before_ this branch; it's the lambda passed as the canceller two lines up. That's admittedly a little confusing, but I didn't want to duplicate the instantiation of the `Deferred` and both branches need it.

src/txacme/testing.py, line 83 at r1 (raw file):
Previously, jerith (Jeremy Thurgood) wrote…
> We currently only create a `FakeClient` in one place and we pass in a controller. Would it make sense to simplify this constructor and either always create the controller internally (and grab it from the client where we need it) or always pass one in?

We actually create a `FakeClient` in three places (by my count: `test_endpoint`, `test_client`, and the integration tests), and two of them don't currently need the controller functionality.
Grabbing the controller from the client is problematic, because the real client of course doesn't have such an attribute; relying on "testing only" APIs makes it easier to accidentally introduce a bug by writing code that only works with the test double, but not the real implementation. That's the usual argument against doing this, anyway...
Always passing one sounds reasonable, though; explicit is better than implicit, or something?

Comments from Reviewable

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

src/txacme/service.py, line 75 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
> It's an incompatible change to the signature of __init__ but I think being able to pass in a value for `waiting` makes no sense and is very confusing. I think when I wrote this code originally, `attrs` hadn't yet added support for `default=...` combined with `init=False`, so that may be why I originally didn't have it that way.

OK, thanks for the explanation.

src/txacme/service.py, line 163 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
> The code that uses it is in fact located _before_ this branch; it's the lambda passed as the canceller two lines up. That's admittedly a little confusing, but I didn't want to duplicate the instantiation of the `Deferred` and both branches need it.

Maybe a brief comment or something (either here or above) would help?

src/txacme/testing.py, line 83 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
> We actually create a `FakeClient` in three places (by my count: `test_endpoint`, `test_client`, and the integration tests), and two of them don't currently need the controller functionality.
>
> Grabbing the controller from the client is problematic, because the real client of course doesn't have such an attribute; relying on "testing only" APIs makes it easier to accidentally introduce a bug by writing code that only works with the test double, but not the real implementation. That's the usual argument against doing this, anyway...
>
> Always passing one sounds reasonable, though; explicit is better than implicit, or something?

I didn't find those other places. Given that they exist, though, the slight extra complexity involved in making this optional is almost certainly worthwhile.

Comments from Reviewable

Review status: 3 of 4 files reviewed at latest revision, 1 unresolved discussion.

src/txacme/service.py, line 163 at r1 (raw file):
Previously, jerith (Jeremy Thurgood) wrote…
> Maybe a brief comment or something (either here or above) would help?

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Unexpected @keyup triggering on focused input field when navigated to via keyboard "click"

Environment

Operating System: Darwin
Node Version:     v18.18.2
Nuxt Version:     3.6.1
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, nitro, app, modules, security, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Although we are pinned at Nuxt v3.6.1 'for reasons' the bug can be observed in StackBlitz repro on Nuxt v3.8.2
Reproduction
https://stackblitz.com/edit/github-6rrfcj?file=pages%2Findex.vue
Note: Issue relates to keyboard navigation! - not mouse.

Open index page and tab bring focus to button labelled Goto Two.
Hit enter key to effectively click button.  (remember, keyboard - not mouse!)
Browser navigates to page two

Notice the header suggests that the function handleNext has been called. This suggest that the @keyup event has fired on the input control on page two as handleNext is the function handling of that event. I can't think why this should be expected behaviour. Almost as if the keyup event from pressing the button on the index page is being seen by page two when the focus() call is made?!
Open the browser devtools to see console.log messages also indicating that handleNext has been called.
Also, to add additional context/relevant to help diagnose the error. In the script for two.vue there is another option (2) that can be uncommented and option (1) commented out. This only calls focus() after a delay. With this in place the potential bug doesn't happen.
Describe the bug
I'm seeing what I think maybe a bug when navigating to a page that combines and input control with an associated @keyup handler that is also focused on page load via onMounted lifecycle event.
Follow the reproduction above and also the effect of delaying the call to focus() on the input control.
Additional context
Changing the event type on the input field from @keyup to @keydown also avoids the issue.
Logs
No response

I doubt this is a Nuxt issue. Can you replicate with Vue?
https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter

Hey @danielroe. Many thanks for responding to this so quickly. I followed your link and re-created in Vue.
stackblitz
Seems to do the same thing so I guess I'll post the same issue on https://github.com/vuejs/core/issues?

Yes, that'd be great! Thank you @iangriggs

Top help anyone else coming here with a similar concern:
https://github.com/vuejs/core/issues/9696
TL;DR; It's expected behaviour with a couple of workarounds described in comments.


Bug report feature

I would like to start a discussion about a "bug report" feature baked into the scripts. What I am thinking is, that we can collect some basic information such as:

OS/architecture
Game in use
Versions of scripts and game
Package listing
Last 100 logs of pertinent console/server logs
Per-game specifics, i.e. for Source servers the contents of addons/sourcemod/plugins or for Unreadl engine the active mutators.

I'm sure there are a lot of other items that would be useful, but I think if we can come up with the bulk of the data points, and then have a system which either uploads that to PasteBin or even create a GitHub issue, it would solve a lot of the initial confusion of a user reporting an issue and us having to ask the same questions.
Questions we'd need to answer:

How do we want to receive and triage these issues?
Do we want a feature where a server crashing automatically sends a report?
How do we make sure users know about the feature, and opt-in to the reporting?
How do we ensure we don't accidentally expose passwords or other sensitive data as a result of this?
This may significantly increase the bug reports, how do we want to handle initial triage and assignment?

I see some interesting ideas there.
"Last 100 logs of pertinent console/server logs" > As the log cleaner cleans logs after 7 days, and considering the size that logs can get, 100 may be a bit too much.
Otherwise, i pretty much agree.
I also think nothing should be sent by the script without user's agreement.
Another question would be : What port would this use ?
About your interrogations :
How do we want to receive and triage these issues?

I guess the script couldn't make issues on github + it would be a pain to manage.
A good way could be to make the script communicate with a defined server, then send it to the server. The problem is that it would open the way to fake reports, so we'd need to consider securing it. Then we'd have an interface to visualize those reports. That interface should be made so it's not a pain to fetch all the data, as it will probably be simple text only.

Do we want a feature where a server crashing automatically sends a report?
Would be cool. Would even be nice to have an automatic report to a master server, just to say that a server is running and have a count of how many people are running servers from any game. As long as users accept feedback. That way we could collect some data to help us improve the script.
How do we make sure users know about the feature, and opt-in to the reporting?

At install, it could ask if user wants that feature. For report, could be a dedicated function, or the dev-debug function that sends it.
Another question would be : How to store this setting ?
But here i've got an idea that could be used for other things too : Put settings in an lgsm.conf file that would be at the root path (same as the main script). I should make a suggestion for that, as i think it would be cool to have a ./scriptname config function, that would make an assistant for that, and create the lgsm.conf file. At install, it would ask if you want to change default server ports and settings. That way, people could either use the assistant, or edit the lgsm.conf, and they would have no need to edit the server script. That would raise two questions here : How to support custom game settings without writing a new script for every game ? (i already think of a way). And how do we support multiple servers with that way of doing ? (got ideas too).

How do we ensure we don't accidentally expose passwords or other sensitive data as a result of this?

As some logs can include steam password and logins, they need to be removed before sending, especially if those reports are public. The server recieving them could make a double check to remove them.

This may significantly increase the bug reports, how do we want to handle initial triage and assignment?
Would be cool first, that every machine, or every server installation gets a unique ID to help sorting this. Then the user should get an important notification given by the script with a "report number ID" to give to us when he's coming on github to report a bug.
Here are my thoughts at the moment. Hope that helps.

Good thoughts, I think we are on the same page. I meant 100 lines not logs, d'oh :)
In regards to the config thing, I'm two steps ahead of you, check my latest issue to see what I've been doing to minimize the headaches of supporting all the games and saving configs. It's trivial to add more variables to the system I have, so tat could be done.
For the rest, it's probably good if we just mock up a copy of the script with some instrumentation and try it out from our local machines. I just need a bit more detail on what we actually need, and I can throw something together.

Let's say this is pretty similar to https://github.com/dgibbs64/linuxgsm/issues/586
Closing this, but link it for record !


viz.yaml changes

Hoping not to conflict with anyone :pray:

wow, that didn't conflict w/ me. Surprise! 🎈


Adding RegNets to tf.keras.applications

See #15240. Once the module structure is fixed, I'll train the models asap.
References:

Designing Network Design Spaces (CVPR 2020)
Fast and Accurate Model Scaling  (CVPR 2021)

Important code that needs attention is from line 277 to line 683. Rest is configuration and variant instantiation.
/cc @sayakpaul @fchollet
/auto Closes #15240

@fchollet
I don't have the checkpoints as of now. As I was saying in #15240, it'd be great if you gave a concrete green signal. I'll consider this as your good to go and start training the models.
About the performance, I had trained 4 variants of RegNetY earlier. Their accuracies didn't match the ones in the paper, we were off by 2-3%, even by using better optimizers and training setup than in the paper:

Model
Our accuracy
Accuracy from the paper

RegNetY 200MF
67.54%
70.3%

RegNetY 400MF
70.19%
74.1%

RegNetY 600MF
73.18%
75.5%

RegNetY 800MF
73.94%
76.3%

/cc @sayakpaul

it'd be great if you gave a concrete green signal. I'll consider this as your good to go and start training the models.

I do believe we should merge it. If we can improve the accuracies you get, that's great, but if we can't, they're still good enough to merge. We'll report the actual accuracies (not that of the paper) in our own table.

@fchollet
Thanks for the clarification. This is really encouraging for me. I will try to get the best accuracy as I can.
Myself and @sayakpaul were discussing about converting PyTorch weights to TF weights in case the training doesn't turn out well (as a last resort). What do you think about that?

Myself and @sayakpaul were discussing about converting PyTorch weights to TF weights in case the training doesn't turn out well (as a last resort). What do you think about that?

Weight conversion is generally straightforward and easier than retraining. Just make sure you do it layer by layer and check the output after each layer so you can spot mismatches and know which layer they come from.

Are you saying we should go for weight conversion rather than training?

Personally I think training models is better than just porting over the weights. We can quantify the discrepancy if any.

I agree that training the models from scratch should be the default strategy here. As long as the accuracy discrepancy is reasonable (1-2 percentage points) it seems completely fine.

@fchollet
Understood. Thanks for the clarification. \
I've made the requested changes. Please review again. If there aren't any changes I'll start training soon.

but I'd appreciate a second pair of eyes as well.

@fchollet I can do it today if it's okay.

@fchollet @sayakpaul
I've made the requested changes. Just had a couple of questions, posted them in the comments.

@fchollet
Could you please take a look at the unresolved comments?
link1, link2

@fchollet
I am now trying to build Keras from source and import the models. However, I'm facing a proto import error. Perhaps there's something wrong in my build procedure? Please take a look.
My procedure:

Clone the repo with relevant branch and install bazel.
Add regnet.py in keras.applications.BUILD file
bazel build (...)
/bazel-bin/keras/tools/pip_package/build_pip_package (...)
pip install keras-2.7.0-py2.py3-none-any.whl
Try importing:

import tensorflow as tf
model = tf.keras.applications.RegNetX002()

My error:
      1 import tensorflow as tf
----> 2 tf.keras.applications.RegNetX002()

30 frames
/content/keras/keras/saving/saved_model/load.py in <module>()
     23 from keras.engine import input_spec
     24 from keras.optimizer_v2 import optimizer_v2
---> 25 from keras.protobuf import saved_metadata_pb2
     26 from keras.protobuf import versions_pb2
     27 from keras.saving import saving_utils

ImportError: cannot import name 'saved_metadata_pb2' from 'keras.protobuf' (unknown location)

# Note that only .proto files are not getting imported, rest all files are being successfully imported.

Gist here.

I think it is highly likely related mix usage of local keras workspace and pip env.
Note that when run python within the directory of keras git repo, "import keras" will try to load the code from the local workspace, instead of the installed PIP (since the local directory take high priority when loading). The protobuf py file is available in the PIP package, but not workspace (since it is generated).
Could you try change to a different directory when you run "import tensorflow as tf; tf.keras.applications. RegNetX002()" and see if it works?

@qlzh727
Thanks for the advice. The previous error is no more, however I ma facing the following error:
AttributeError                            Traceback (most recent call last)
<ipython-input-6-cc8597f40d82> in <module>()
      1 get_ipython().magic('cd /content/')
      2 import tensorflow as tf
----> 3 tf.keras.applications.RegNetX002

AttributeError: module 'keras.api._v2.keras.applications' has no attribute 'RegNetX002'`

I've added @keras.export wherever required.

@AdityaKane2001

Kept rescaling and normalization as in efficentnet.py.

I think it's better to give options to users to choose built-in processing or custom processing. The current preprocessing logic of efficientnet needs to be fixed.

@fchollet @qlzh727
Regarding preprocessing built into the model. As in #14823, the preprocessing is updated to just keep rescaling. In case of RegNets, the official implementation rescales to [0,1] and normalizes to Imagenet mean and std.
Should we include both Rescaling and Normalization inside the model?
In my opinion, it is okay to keep Rescaling inside but I'm not sure about Normalization. As pointed out by @innat there might be some cases where users may want to use some custom preprocessing. How would Normalization play with that?

@qlzh727
Thanks for the advice. The error is no more. I'm still not able to import the models, but I'll try this on a GCE instance and get back to you.
UPDATE: I am still facing the following error:
# import tensorflow as tf
AttributeError                            Traceback (most recent call last)
<ipython-input-6-cc8597f40d82> in <module>()
      1 get_ipython().magic('cd /content/')
      2 import tensorflow as tf
----> 3 tf.keras.applications.RegNetX002

AttributeError: module 'keras.api._v2.keras.applications' has no attribute 'RegNetX002'`

# import keras
>>> keras.applications.RegNetX002
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'keras' has no attribute 'applications'

>>> os.listdir("/home/adityakane1/keras_env/lib/python3.7/site-packages/keras/applications/")
['mobilenet.py', 'mobilenet_v3.py', 'vgg19.py', '__pycache__', 'vgg16.py', 'resnet.py', 'xception.py', 'inception_v3.py', 'imagenet_utils.py', '__init__.py', 'mobilenet_v2.py', 'inception_resnet_v
2.py',
'regnet.py',  # File is present
'densenet.py', 'resnet_v2.py', 'nasnet.py', 'efficientnet.py']

I've added @keras.export wherever required. Do I need to add regnet.py somewhere else other than keras.applications.BUILD?

Could u check the content under keras/api/keras/applications/init.py? The new API endpoint will be populated there if it is exported correctly.

@qlzh727
I am getting the following output:
# ~/keras_env/lib/python3.7/site-packages/keras/api/__init__.py

(keras_env) adityakane1@keras-builder-vm:~/keras_env/lib/python3.7/site-packages/keras/api$ cat __init__.py
# This file is MACHINE GENERATED! Do not edit.
# Generated by: tensorflow/python/tools/api/generator/create_python_api.py script.
"""Public API for tf. namespace.
"""

from __future__ import print_function as _print_function

import sys as _sys

from keras.api import keras

del _print_function

from tensorflow.python.util import module_wrapper as _module_wrapper

if not isinstance(_sys.modules[__name__], _module_wrapper.TFModuleWrapper):
  _sys.modules[__name__] = _module_wrapper.TFModuleWrapper(
      _sys.modules[__name__], "", public_apis=None, deprecation=True,
      has_lite=False)

and
#~/keras_env/lib/python3.7/site-packages/keras/api/_v2/__init__.py

(keras_env) adityakane1@keras-builder-vm:~/keras_env/lib/python3.7/site-packages/keras/api/_v2$ cat __init__.py
# This file is MACHINE GENERATED! Do not edit.
# Generated by: tensorflow/python/tools/api/generator/create_python_api.py script.
"""Public API for tf. namespace.
"""

from __future__ import print_function as _print_function

import sys as _sys

from keras.api._v2 import keras

del _print_function

I see. I think you need to update https://github.com/keras-team/keras/blob/c3a334b649eea8e94895672e9212f7014852e559/keras/api/BUILD#L21 to make the API generation script to pickup the newly added python file.

@qlzh727
I added it to ./keras/applications/BUILD and ./keras/api/BUILD, but I am facing the following error while running bazel build:
ERROR: /home/adityakane1/keras/keras/api/BUILD:170:19: Executing genrule //keras/api:keras_python_api_gen_compat_v2 failed: (Exit 1): bash failed: error executing command /bin/bash -c ... (remaining 1 argument(s) skipped)
2021-09-29 18:28:27.655673: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory
2021-09-29 18:28:27.655736: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Traceback (most recent call last):
  File "/home/adityakane1/.cache/bazel/_bazel_adityakane1/3e2d46e33848cfa517d7c0ad54725d8d/execroot/org_keras/bazel-out/k8-opt-exec-2B5CBBC6/bin/keras/api/create_keras_api_2_keras_python_api_gen_compat_v2.runfiles/org_keras/keras/api/create_python_api_wrapper.py", line 30, in <module>
    create_python_api.main()
  File "/home/adityakane1/keras_env/lib/python3.7/site-packages/tensorflow/python/tools/api/generator/create_python_api.py", line 826, in main
    lazy_loading, args.use_relative_imports)
  File "/home/adityakane1/keras_env/lib/python3.7/site-packages/tensorflow/python/tools/api/generator/create_python_api.py", line 700, in create_api_files
    f'Missing outputs for genrule:\n{missing_files}. Be sure to add these '
ValueError: Missing outputs for genrule:
"keras/applications/regnet/__init__.py". Be sure to add these targets to tensorflow/python/tools/api/generator/api_init_files_v1.bzl and tensorflow/python/tools/api/generator/api_init_files.bzl (tensorflow repo), keras/api/api_init_files.bzl (keras repo), or tensorflow_estimator/python/estimator/api/api_gen.bzl (estimator repo)
Target //keras/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
ERROR: /home/adityakane1/keras/keras/tools/pip_package/BUILD:37:10 Middleman _middlemen/keras_Stools_Spip_Upackage_Sbuild_Upip_Upackage-runfiles failed: (Exit 1): bash failed: error executing command /bin/bash -c ... (remaining 1 argument(s) skipped)
INFO: Elapsed time: 212.369s, Critical Path: 18.86s
INFO: 230 processes: 19 internal, 211 local.
FAILED: Build did NOT complete successfully

I think the error message is indicating that you need to update this file. https://github.com/keras-team/keras/blob/c3a334b649eea8e94895672e9212f7014852e559/keras/api/api_init_files.bzl#L4

@qlzh727
Thanks for the support. The functions are getting imported now.
@fchollet
There are some minor bugs in the code, I'll fix them and let you know.
/cc @sayakpaul

@fchollet  @qlzh727
There's a small difference in our implementation and pycls. They use padding=1 for all conv3x3s. However, AFAIK there isn't such a provision in Keras. So I'm going to manually pad the tensors after conv3x3 to get the desired shape.
Is there a cleaner way to do this?

@qlzh727
I am trying to build Keras on the TPU host machine. I am facing the following error:
(keras_env) jupyter@ak-us-tpu-1:~/keras_installation/keras$ bazel build //keras/tools/pip_package:build_pip_package

INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=174
INFO: Reading rc options for 'build' from /home/jupyter/keras_installation/keras/.bazelrc:
  'build' options: --apple_platform_type=macos --define open_source_build=true --define=use_fast_cpp_protos=false --define=tensorflow_enable_mlir_generated_gpu_kernels=0 --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --config=short_logs --config=v2
INFO: Found applicable config definition build:short_logs in file /home/jupyter/keras_installation/keras/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /home/jupyter/keras_installation/keras/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
ERROR: /home/jupyter/.cache/bazel/_bazel_jupyter/ebc81b3ee71ff9bb69270887ebdc0d7b/external/bazel_skylib/lib/unittest.bzl:203:27: name 'analysis_test_transition' is not define
d
ERROR: error loading package '': Extension 'lib/unittest.bzl' has errors
ERROR: error loading package '': Extension 'lib/unittest.bzl' has errors
INFO: Elapsed time: 0.107s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)

I followed the same procedure as in the above conversation.

@fchollet
Yes I've started training since yesterday. I'll experiment a bit and try to get the accuracies in the paper.

@AdityaKane2001 Can you please resolve conflicts? Thanks!

@gbaned
Resolved!

@fchollet
An update about the training. I have completed training the X variant. Here is a summary:

Model
Paper (in %)
Ours (in %)
Diff (in %)
Comments

X002
68.9
67.15
1.75
adamw, area_factor=0.25

X004
72.6
71.22
1.38
adamw, area_factor=0.08

X006
74.1
72.37
1.73
adamw, area_factor=0.08

X008
75.2
73.45
1.75
adamw, area_factor=0.08

X016
77
75.55
1.45
adamw, area_factor=0.08, mixup=0.2

X032
78.3
77.09
1.21
adamw, area_factor=0.08, mixup=0.2

X040
78.6
77.87
0.73
adamw, area_factor=0.08, mixup=0.2

X064
79.2
78.22
0.98
adamw, area_factor=0.08, mixup=0.3

X080
79.3
78.41
0.89
adamw, area_factor=0.08, mixup=0.3

X120
79.7
79.09
0.61
adamw, area_factor=0.08, mixup=0.4

X160
80
79.53
0.47
adamw, area_factor=0.08, mixup=0.4

X320
80.5
80.35
 0.15
adamw, area_factor=0.08, mixup=0.4

Now training the Y variant.
/cc @sayakpaul @qlzh727

Please fix the merge conflicts!

@fchollet
I'm sorry, I don't see any merge conflicts on the GitHub UI.

The GH UI says:

This branch cannot be rebased due to conflicts

Make sure to rebase the changes on top of the master branch and resolve and conflicts you see when rebasing. GitHub provides instructions.

This branch cannot be rebased due to conflicts

The problem seems to persist. If you're not able to rebase, you could always create a new PR...

@fchollet
I rebased the branch onto master and resolved the merge conflicts. If the problem still persists, I'll create a new PR.

If the problem still persists, I'll create a new PR.

Please do. There are still merge conflicts.

Continued in #15702.


feat: added perplexity model

Added perplexity models from the official PPLX API docs:

Thanks I will review and merge maybe today or tomorrow


[Docs / Guidance] RE: Custom ITelemetryProcessor

Just flagging that your own custom ITelemetryProcessors will be instantiated, but the Process method will  never be called if your IntrumentationKey is left blank.
For instance, in the scenario where you are developing locally and trying to get your own ITelemetryProcessor to work.
Yes, I just spent my morning trying to figure this one out...  🥇

Oh, thank you for that. I was pulling my hair out over this. Adding in the IntrumentationKey value got my code being called.

Why would this be a documentation / guidance issue?
In other words, why not call Process even when IntrumentationKey is blank?

I think it may be addressed with the new multi-sink support work https://github.com/Microsoft/ApplicationInsights-dotnet/issues/559

Addressed with #604.


feat: make shadow JAR optional

I just got wiremock-spring-boot moved to the WireMock organization. I'd like to use this plugin.
The project is a JUnit 5 extension and I don't feel it makes sense to provide a standalone JAR. Would like to opt out of it.
If this is way of making it configurable is ok with you, perhaps we may want to have groupId configurable also. If you don't want junit 5 extensions in org.wiremock.extensions.
Also setting the version from conventional commits could be added as an optional feature:
def bumpVersionTo(version) {
  def propertyFile = new File("$projectDir/gradle.properties")
  def gradleProps = new Properties()
  propertyFile.withReader { gradleProps.load(it) }
  def currentVersion = gradleProps.getProperty("version")
  if (currentVersion == version) {
    logger.lifecycle("Version not changed (${currentVersion})")
  } else {
    logger.lifecycle("Version changed (${currentVersion} -> ${version})")
    gradleProps.setProperty('version', version)
    propertyFile.withWriter { gradleProps.store(it, null) }
  }
}

task setConventionalVersion() {
  doFirst {
    def nextVersion = se.bjurr.gitchangelog.api.GitChangelogApi.gitChangelogApiBuilder()
        .withFromRepo(file('.'))
        .withSemanticMajorVersionPattern("^[Bb]reak")
        .withSemanticMinorVersionPattern("^[Ff]eat")
        .getNextSemanticVersion()
        .getVersion();
    def nextSnapshot = "${nextVersion}-SNAPSHOT"
    bumpVersionTo(nextSnapshot)
  }
}
build.dependsOn setConventionalVersion

References

https://github.com/wiremock/wiremock-spring-boot/issues/44

Submitter checklist

[x] Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
[x] The PR request is well described and justified, including the body and the references
[x] The PR title represents the desired changelog entry
[x] The repository's code style is followed (see the contributing guide)
[x] Test coverage that demonstrates that the change works as expected
[x] For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Will try to release this weekend


ThriftResponse implements AutoCloseable

If ThriftResponse implements AutoCloseable then clients can use the try-with-resources pattern:
try (ThriftResponse<FooService.foo_result> response = future.get()) {
    result = response.getBody(FooService.foo_result.class);
}

vs
ThriftResponse<FooService.foo_result> response = future.get();
try {
    result = response.getBody(FooService.foo_result.class);
} finally {
    if (response != null) {
        response.release();
    }
}

done


added windows additional test and upgraded to kie, uberfire and dashb…

…uilder versions
@psiroky Please don't merge before uberfire, dashbuilder and kie are upgraded to its new versions (8.0.0-SNAPSHOT, 2.0.0-SNAPSHOT and 1.0.0-SNAPSHOT) - so not before tomorrow (Wednesday) morning.

@psiroky can you please approve this? jobs in "kie release for 7.2.x branch" have the same names then teh ones in "kie release for master branch" - as the branches were not yet renewed.

Again two completely unrelated changes in single commit. Please make sure to submit different changes in different PRs (or least different comimts). I don't really like the windows thing, but since this is needed to fix the master jobs, I will merge it anyway.


feat: Add auto-apply markdown features on editor: Enable WYSIWYG

Fixes #(issue)

This pr fixes #1039
Pull Request details
When a user types markdown syntax, the editor should automatically recognize and implement the respective feature.
the editor should instantly recognize markdown syntax and apply the relevant features without user intervention.
markdown feature:
Markdown shortcuts make it easy to format the text while typing.

To test that, start a new line and type # followed by a space to get a heading.
Try #, ##, ###, ####, #####, ###### for different levels.

Those conventions are called input rules in Tiptap. Some of them are enabled by default.
Try > for blockquotes, *, - or + for bullet lists, or `foobar` to highlight code, ~~tildes~~ to strike text,
or ==equal signs== to highlight text.

You can overwrite existing input rules or add your own to nodes, marks and extensions.

For example, we added the Typography extension here. Try typing (c) to see how it’s converted to a proper © character. You can also try ->, >>, 1/2, !=, or --.

Any Breaking changes
None
Associated Screenshots
Screencast from 2024-10-06 02-48-24.webm

Thats pretty cool! Alot of functionality for a very small change!


Using \zcref in section titles

For obvious reasons, hyperref complains when \zcref is used in section titles
One way around this is \texorpdfstring{\zcref{some_theorem}}{Theorem~\ref{some_theorem}}}, however this defeats part of the reason for using zref-clever in the first place.
There are some workarounds I've seen, but they all revolve around cleveref. For instance
\usepackage{crossreftools}
\pdfstringdefDisableCommands{%
    \let\zcref\crtCref
}

However, this only works if cleveref is also loaded.
Is there a better way to solve this?

For obvious reasons, hyperref complains when \zcref is used in section titles

I'd say expected indeed. But, can you please provide a MWE of the issue? (It spares me the guesswork and the rework. If at all possible, you should always try to provide one when reporting).

There are some workarounds I've seen, but they all revolve around cleveref.

Could you provide links to them?

I'd say expected indeed. But, can you please provide a MWE of the issue? (It spares me the guesswork and the rework. If at all possible, you should always try to provide one when reporting).

My bad
\documentclass{article}

\usepackage{zref-clever}
\usepackage{hyperref}

\begin{document}

\section{Hello}
\label{first}

\section{World \zcref{first}}

\end{document}

will give the following error
Token not allowed in a PDF string (Unicode):
(hyperref)	removing `\zcref'.

and the pdf bookmark will be World first instead of World section 1
However
\documentclass{article}

\usepackage{zref-clever}
\usepackage{hyperref}

\usepackage{cleveref}
\usepackage{crossreftools}
\pdfstringdefDisableCommands{%
    \let\zcref\crtcref
}

\begin{document}

\section{Hello}
\label{first}

\section{World \zcref{first}}

\end{document}

gives the desired output

Could you provide links to them?

https://ctan.org/pkg/crossreftools?lang=en provides expandable versions of cleveref commands
I found the workaround in https://github.com/latex3/hyperref/issues/148#issuecomment-660202930

I'm not sure the use case crossreftools had in mind in offering \crtcref and \crtCref but, in my view, using them to do:
\pdfstringdefDisableCommands{%
    \let\Cref\crtCref
    \let\cref\crtcref
}

is a mistake. Because \cref and \Cref can receive a comma separated list of labels, while \crtcref and \crtCref can only receive a single label as argument. So, if you happen to try:
\documentclass{article}

\usepackage{hyperref}

\usepackage{cleveref}
\usepackage{crossreftools}
\pdfstringdefDisableCommands{%
    \let\cref\crtcref
}

\begin{document}

\section{Hello}
\label{first}

\section{Foo}
\label{second}

\section{World \cref{first,second}}

\end{document}

You get:

And the error is completely silent, since we cannot even issue a warning there, given the macros need to be expandable.
Also, as documented in crossreftools, besides the fact it can't handle lists of labels, \crtcref and \crtCref are not fully general replacements to \cref and \Cref since it ignores \crefformat etc. and only retrieve the \crefname of the counter for the label. So \crtcref may differ from \cref even for a single label.
This point is valid for both cleveref and zref-clever. The problem is the same. And I don't think it is possible to provide a fully general expandable version of either \cref or \zcref. So, I don't think it is an oversight of cleveref not to try to provide such macros. If crossreftools decided to provide some half-baked ones regardless, I personally still don't understand the intended use cases for them. And, as I said above, using them for this particular use case is, imho, a bad idea.
The case is actually worse for \zcref because it has an optional argument which \crtcref does not. So if you do \let\zcref\crtcref, as soon as you do something like \section{\zcref[S]{first} bar}, you get a broken document.
My recommendation for this is to take the traditional approach, and use either the short title argument of the sectioning command or \texorpdfstring, and construct the reference manually. Like so:
\documentclass{article}

\usepackage{zref-clever}
\usepackage{hyperref}

\begin{document}

\section{Hello}
\label{first}

% either
\section[World section~\ref{first}]{World \zcref{first}}

% or
\section{Foo \texorpdfstring{\zcref{first}}{section~\ref{first}}}

\end{document}

You say "this defeats part of the reason for using zref-clever in the first place". I'd say hardly. It is indeed a known limitation, and working around it requires a little manual work on the occasional use of \zcref in a section title. Not a significant dent in the grand scheme of things.
If you really, really want to avoid this manual work, you may get away by leveraging \autoref for your bookmarks with something like:
\documentclass{article}

\usepackage{zref-clever}
\usepackage{hyperref}
\NewExpandableDocumentCommand{\myautoref}{O{}m}{\autoref{#2}}
\pdfstringdefDisableCommands{%
  \let\zcref\myautoref
}

\begin{document}

\section{Hello}
\label{first}

\section{World \zcref{first}}

\end{document}

However, this is subject to the same fundamental problem as using \let\cref\crtcref with cleveref, plus any differences of output between \zcref and \autoref. But, you should get reasonable results most of the time. Caveat emptor though, of course.
Naturally, zref-clever does store the name(s) for a given reference type somewhere, and you can (expandably) retrieve the type of a given label with zref tools. So it is technically viable to build something like \crtcref for zref-clever. (Though there are some subtle differences, e.g.: i) the case is not controlled by the main macro but by an option, which you would not have access to; ii) the name of the variable which stores the reference type name may depend on the declension cases defined for a given language, etc.). Anyway, this is neither recommended, nor supported, so I won't post this here. Of course, as you know, (La)TeX allows you to dig the code and do it regardless.

Thanks for the exhaustive reply. I'll probably end up just sticking with \texorpdfstring. (I know only just about enough to be dangerous.)

You say "this defeats part of the reason for using zref-clever in the first place". I'd say hardly. It is indeed a known limitation, and working around it requires a little manual work on the occasional use of \zcref in a section title. Not a significant dent in the grand scheme of things.

This is true. I was exaggerating for effect.
It's just unfortunate that such a seemingly easy (for the user) thing is so hard to do in LaTeX. I seems like a thing that "should" be possible.

I'll probably end up just sticking with \texorpdfstring.

I think that's the wisest. I do get your mumble, but I think you'll get used to it in no time, and it shouldn't bother you much in your actual document. And, even if you occasionally need to set up a cross-ref manually for the bookmark, the actually typeset material can always be kept perfectly consistent with the use of \zcref.

It's just unfortunate that such a seemingly easy (for the user) thing is so hard to do in LaTeX. I seems like a thing that "should" be possible.

The problem well known and low level, but not that low. I'd call it a limitation of how hyperref handles bookmarks rather than a "LaTeX limitation". Consider, for example, the following document to generate a tagged PDF:
\DocumentMetadata{testphase=phase-III}

\documentclass{article}

\usepackage{zref-clever}
\usepackage{hyperref}

\begin{document}

\section{Hello}
\label{first}

\section{Foo \texorpdfstring{\zcref{first}}{section~\ref{first}}}

\end{document}

And open the resunting PDF at https://ngpdf.com/loadFile (to inspect the HTML extracted from the tagged structure). You get:

I take from there LaTeX can, in principle, already "see the text generated by protected/non-expandable material and send it to the PDF". (Says the person who has no real idea of how the bookmarks are created... :wink:, so take it with a grain of salt). So things may evolve eventually (of course, the time scale in which things move in the LaTeX world might not be to your liking...).

Thanks for the exhaustive reply.

Thanks for taking the time to report. Cheers! :smile:


It seems that the setting for "Display" is set in cookie and not in the settings.

Sorry for bothering you again, as I have been creating a lot of  bug reports lately.
My systems are setup not to save any cookies. (and I assume that this is part of the issue that I am having)
I have now tried multiple installations and setups, but it seems like the "Display" setting is saved as a cookie and not as a real setting. So would be great to have this saved as a setting and not as something saved in a cookie. So that I don´t have to change this setting whenever I reinstall the computer or switch to another computer.
Describe the bug
Whenever I reinstall my machine or install a new machine and login the webpage defaults to "Detailed" even though I always use "Expanded".
Expected behavior
I want the next login to respect how I have set it up in the settings, and not saved as a cookie. (as as I am switching computers and installations)
Screenshots

Environment (please complete the following information):
Today I am using Firefox Broser 131.0.2

commafeed.com or self-hosted:

Actually, this is a feature introduced in 3.1.0. The display mode is stored in the local storage of the browser to allow users to use a different display mode per device.
For instance, I don't use the same display mode on my computers and on my phone.
The custom width of the side bar is also stored in the local storage.

Ok. Understand. So it is not a bug but an feature.
My use case would prefer to have it as a setting saved between sessions, as I have to change it each time my browser start up again. But I fully understand if you don´t fix.


Help me

[libprotobuf ERROR google/protobuf/text_format.cc:245] Error parsing text-format caffe.NetParameter: 213:13: Message type "caffe.ConvolutionParameter" has no field named "dilation".
WARNING: Logging before InitGoogleLogging() is written to STDERR
F0402 13:04:55.491230 21321 upgrade_proto.cpp:928] Check failed: ReadProtoFromTextFile(param_file, param) Failed to parse NetParameter file: colorization_deploy_v0.prototxt
*** Check failure stack trace: ***
Ubuntu14.04 GTX9602G
I installed NVIDIA's caffe following the steps below
https://github.com/NVIDIA/DIGITS/blob/master/docs/UbuntuInstall.md
Thank you richzhang

Hi, dilation is a pretty recent feature, so you will need to clone the latest version of caffe and reinstall: https://github.com/BVLC/caffe
Happy colorizing!

It works!
I am going to colorize my grandmother's old pictures

HI,Rich zhang:
I cloned the latest version of caffe and still get the same result as "[libprotobuf ERROR google/protobuf/text_format.cc:245] Error parsing text-format caffe.NetParameter: 213:13: Message type "caffe.ConvolutionParameter" has no field named "dilation".".
I am using jupyter notebook on the server and chrome browser on the client. Could you give me any suggestion?Thanks.

BVLC/caffe is ok.


chore: merge e2e-demo and rhtap-demo

Description
merge of rhtap-demo and e2e-demo tests
Issue ticket number and link
RHTAP-1157
Type of change

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

How Has This Been Tested?
ginkgo -p --label-filter='rhtap-demo' --v ./cmd

Checklist:

[X] I have performed a self-review of my code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/test all

/test all

/test redhat-appstudio-e2e

/test all

/test all

/test all

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e


Extract image SHAs from status.containerStatuses as well as spec.containers

Behavior Changes
The deploytime exporter will now check a pod's status.containerStatuses for image IDs as well.
Linked Issues?
resolves #330
Testing Instructions
I'm planning on adding a simple mock for our deploytime exporter as well, which will have some images that don't have the ID in their spec.
@redhat-cop/mdt
Remaining Work

[x] I need to see if pod.status.containerStatuses might list sidecars from MutatingAdmissionControllers (e.g. from Istio / Service Mesh), which we wouldn't want to count. It's also possible that those are already included in spec.containers, which would be a separate issue.
[ ] I need to add the mock as mentioned above.

Bump. Any updates on this?

@etsauer the update is that I had a nice vacation and am back to working on this today. All that's left is to add / check the mocked API for testing this, and investigate the unit test failure that CI turned up.

@KevinMGranger 🤣 welcome back!

I forgot that we only have mocks for committime for now.
Mocking this can come later-- the unit test should be sufficient.

Hmm, looks like our python version pinning is complaining about the new version. @mpryc, is there a reason we restricted the patch version of the python version?

@KevinMGranger there is not, I think it's safe to include the removal of patch in this PR.


cli/bootstrap: add a torcx-skip-setup flag

/cc @squeed

I duplicated this in #34, because rebasing wouldn't be pretty.

Closing in favor or #34.


fix(astro,backend): Refactor createRedirect to support hash routing

Description

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

Hey @panteliselef - the snapshot version command generated the following package versions:

Package
Version

@clerk/astro
0.0.2-snapshot.vbc9b8d7

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/testing
1.1.10-snapshot.vbc9b8d7

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/astro
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact

!snapshot

Hey @panteliselef - the snapshot version command generated the following package versions:

Package
Version

@clerk/astro
0.0.2-snapshot.vbbff478

@clerk/backend
1.3.1-snapshot.vbbff478

@clerk/express
0.0.16-snapshot.vbbff478

@clerk/fastify
1.0.18-snapshot.vbbff478

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/nextjs
5.2.3-snapshot.vbbff478

@clerk/remix
4.2.2-snapshot.vbbff478

@clerk/clerk-sdk-node
5.0.15-snapshot.vbbff478

@clerk/tanstack-start
0.1.6-snapshot.vbbff478

@clerk/testing
1.1.10-snapshot.vbbff478

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/astro
npm i<EMAIL_ADDRESS>--save-exact

@clerk/backend
npm i<EMAIL_ADDRESS>--save-exact

@clerk/express
npm i<EMAIL_ADDRESS>--save-exact

@clerk/fastify
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/nextjs
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>--save-exact

@clerk/tanstack-start
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact

We will not be needing this for @clerk/astro at the moment, as we will have advice devs to use path routing.


It is working in php 5.5

https://github.com/Block8/PHPCI/blob/master/PHPCI/Builder.php#L237
In PHP 5.5 and later, a finally block may also be specified after or instead of catch blocks. Code within the finally block will always be executed after the try and catch blocks, regardless of whether an exception has been thrown, and before normal execution resumes.

@dancryer Is there any point in using finally here? it isn't as if the try or catch are running or throwing an exception


Will Add the web page in which I will add the Solution of DSA Questions from Leetcode

Will Add the web page in which I will add the Solution of DSA Questions from Leetcode.
@AryanBarsaiyan Kindly Assigned it to me.

okay