Nr 33 docs

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/docs.konghq.com/blob/master/CONTRIBUTING.md
Summary
Modified Kong EE Docker instructions for version .33. Re
Full changelog

removed reference to Portal_GUI_URI
Removed assumption that dev portal should be expected to be blank
Added text to change localhost to server host or IP if installed on remote system.

@nragokong is this ready to merge to master and deploy to production? Or does it need more review?

@nragokong is this ready to merge to master and deploy to production? Or does it need more review?

Yes this is ready to roll.


Allow customization of the warn/bad thresholds for each check.

Right now, they are hard coded. Add the ability to modify them.

Oh hey, we have this now!


[Delivers #93039890] ts-java composability

@mhfrantz This is a major refactoring, and is a monster of a PR. I believe most of the 46 commits are small enough to be easily reviewed, but at least a couple (flagged as 'Big bang: ...') that possibly should have been broken into smaller commits.
The 'autoImport' function was adapted to serve the new primary method of generating 'tsJavaModules'. The name change autoImport -> tsJavaModule doesn't happen until near the end of the PR.
The updated README is in the last commit -- you might want to look at it first.

I find myself looking at those generated TS files and wondering which Java namespaces are contained inside.  I wonder if the ts-java options should be splatted in a header comment.

FTP

The suggestion to splat the tsjava options into the generated file is a good one. I'll make that change and fix the other nits/typos tomorrow.

PTAL

LGTM


Missing prover_did when accepting a Legacy Indy Credential Offer

After taking a look at yersterday AATH results, we could see that AFJ 0.4.0.alpha is working fine in all cases except when it acts a holder:

Runset
ACME(Issuer)
Bob(Holder)
Faber(Verifier)
Mallory(Holder)
Scope
Results

acapy-afj
acapy-main0.8.0-rc0
javascript0.4.0-alpha.44
acapy-main0.8.0-rc0
acapy-main0.8.0-rc0
AIP 1.0
12 / 39 30%

acapy-dotnet-javascript
acapy-main0.8.0-rc0
javascript0.4.0-alpha.44
dotnet
acapy-main0.8.0-rc0
AIP 1.0
2 / 12 16%

afj-acapy
javascript0.4.0-alpha.44
acapy-main0.8.0-rc0
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
AIP 1.0
27 / 28 96%

afj-dotnet
javascript0.4.0-alpha.44
dotnet
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
AIP 1.0
12 / 12 100%

afj-findy
javascript0.4.0-alpha.44
findy0.30.56
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
AIP 1.0
17 / 17 100%

afj
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
javascript0.4.0-alpha.44
AIP 1.0
28 / 28 100%

ariesvcx-javascript
aries-vcx1.0.0
javascript0.4.0-alpha.44
aries-vcx1.0.0
aries-vcx1.0.0
AIP 1.0
4 / 20 20%

dotnet-javascript
dotnet
javascript0.4.0-alpha.44
dotnet
dotnet
AIP 1.0
2 / 1216%

findy-javascript-dotnet
findy0.30.56
javascript0.4.0-alpha.44
dotnet
findy0.30.56
AIP 1.0
2 / 17 11%

When running it locally against an ACA-Py, I figured out that the issue has to do with the fact that AFJ is not including any prover_did in the Credential Request, which is a required field for Legacy Indy.
IndySdkHolderService is currently generating a random string, mostly motivated because indySdk requires it. But for the general use I think we have to also do this trick in LegacyIndyCredentialFormatService, just after calling holderService.createCredentialRequest, in case it does not add any prover_did to the request (which is the case of AnonCredsRsHolderService).

Good find!


Log ledger data request times:

Tracks the turnaround time between a TMGetLedger and its corresponding TMLedgerData. This works for both originated and relayed messages.
Reviewers: Only the most recent commit is part of this set, the others are from another pull request.

Reviewers?
I can do this, I don't have too many on the line.

@rec Just need to make one tweak before reviewing

Reviewers: @ximinez @rec

:+1:

But it's never been compiled.  Why put it in?
Isn't there a way to have it compile, but guaranteed never to execute?  Like, a static const bool you check?

Can I get you to rebase against develop so we can just see the (now two) commits that are part of this?

Both ideas are good

This is useful for data gathering. However, I've had it twice cause rippled to hang on a machine. I'm not positive that it's related to this patch, but the only other things I've added to the build I'm using are output lines to the debug.log. This is useful for data gathering, and if it's available as compile flag that's probably for the best.

This breaks relaying. Since every request already has a request cookie, it's not eligible for relaying when received.
https://github.com/ripple/rippled/blob/develop/src/ripple/overlay/impl/PeerImp.cpp#L1787-L1792
A temporary fix might be to send requests that we want the other side to relay without a cookie. We won't be able to measure their latency, but there's not much we can do about that.

@JoelKatz Since these are protobufs, can I just add a new field "key" and leave the request cookie alone? Older versions will forward them as-is, or no? Or will that fail because it won't know to copy that field to build the TMLedgerData response?

Which protocol buffer type are you adding the new field key to?

@rec Two solutions are possible:

Add uid to TMGetLedger and TMLedgerData, when replying with TMLedgerData set the uid to the uid on the corresponding TMGetLedger`
or
Add a "relay" flag to the type and use that instead of the cookie to indicate if the message should be relayed. Then the cookie is available to use always.

Both of these require a network upgrade to work fully.

Pushed a fix that only logs the times for relayed messages, to solve a bug where originated messages would no longer be eligible for relay (because they set the requestcookie).

@vinniefalco You can add a new optional field and it won't create any problems. But existing servers will not preserve it.

Throwing this out because David says its not useful.


Windows DLL Path

When a LabVIEW Client is built into an exe, the path to the DLL resolves to "\Libraries\Win32"
However, using a typical build specification, the DLL is not copied to this folder because of the way the path is dynamically determined in "Get Server DLL Path.vi"
What is the recommended way to include the DLL in the build process?
AB#2636750

I have encountered this issue as well,  if a build is created and deployed on a machine without LabVIEW and the GRPC package installed the "\Libraries\Win32\labview_grpc_server .dll" file will not exist. Temporarily I've modified the "Get Server DLL Path.vi" and placed the file in the data folder.  It may not be the best approach but it worked.


fix: poor performance of controlled components in docs due to repeate…

Hey @nikgraf ,
Can you check this change locally before u merge, this has definitely fixed the performance issue with TextInput.
But somehow I do not see highlighting in my local, with or without this fix (although http://nikgraf.github.io/belle/#/ has code highlighting). Can you confirm at your end if highlighting is intact after this change.

I'm so glad you identified that issue. Sadly this will prevent the highlighting on the page. It never worked on development, because the css file is not available.
I will check later today, but it should be possible to render the highlighting code component only once.

Hey nik
R u sure, it may just work as highlight.js is still called when code
component is mount. I just removed call from update.
On Jul 18, 2015 8:40 PM, "Nikolaus Graf"<EMAIL_ADDRESS>wrote:

I'm so glad you identified that issue. Sadly this will prevent the
highlighting on the page. It never worked on development, because the css
file is not available.
I will check later today, but it should be possible to render the
highlighting code component only once.
—
Reply to this email directly or view it on GitHub
https://github.com/nikgraf/belle/pull/129#issuecomment-122551305.

I expect the fix to work coz in dev environment also the highlight.js classes are getting applide in code block. Its just that css is not there.

@jpuri I made a change. What do you think about this version?

Hey Nik,
+1, I was also thinking of doing it this way. Though its very less probable that value param will change dynamically. But still I think this would be better fix, in fact I can revert mine. Let me do that.
That makes me thing how careful one should be to optimize render for controlled components.


Adding Geometry

@mountainvat I think this works, but I haven't figured out how to use the GeoJSON parser from Swift successfully. Currently I'm failing to convince Xcode to import the headers...

Did you add a bridging header for the Geometry code?


reconciler/clusterworkspace: split into independent reconciler

For easier testing and less monolithic code. This is pre-req for more advanced logic around ClusterWorkspace scheduling.

/retest

/assign @davidfestal


fix :  Kuberamor compatibility with Bottlerocket vm 1.16

Purpose of PR?:
Fixes :- issues with bottle rocket 1.16
Does this PR introduce a breaking change?
No
If the changes in this PR are manually verified, list down the scenarios covered::
Kubearmor is working on bottle rocket vm (1.16)
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Bug fix. Fixes #
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Commit has unit tests
[ ] Commit has integration tests

Rebase the PR to resolve conflicts :)


AMBARI-24496. Ambari updates lates_url info for HDF stacks with HDP l…

clone of #2107

Ambari pull request builder is down. I verified test result locally.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3651/
Test FAILed.
Test FAILured.


Mandatory FHIR elements (e.g. Resource.status) not mapped to a zib concept

FHIR resources often have mandatory elements, like Resource.status, that do not have a mapping to a concept defined by the zib. Sometimes these mandatory elements are useful and are tracked/captured in systems However, on some occasions, systems don't track or capture the information required by the mandatory element.  A consistent approach would be favourable, hence this issue to determine the best approach. Options:

Describe a generic escape method, using e.g. data-absent-reason extension, on the implementation guide level. Don't add the extension to every mandatory element in FHIR.
Option 1 and make the extension explicit in the FHIR profiles for mandatory elements where it is expected that systems might not be able to provide the required information.
Option 1 and place a comment/documentation on the mandatory elements with guidance.
Leave it to FHIR, don't do anything extra and let implementers find the right information based on the FHIR spec definition.

I think the first consideration should be if it's possible to "map" the required element to the zib concepts. For example, a .status might be inferred from an end date present in the zib.
Apart from that, I think the escape should really be an escape and that systems should do their best to align with the intention of FHIR. Therefore, I would go for a description in the implemention guide.

NB. When "mapping" zib elements to these required elements, the mapping comment can be used to make clear that it is for example a "partial" or "derivative" mapping.

I think Pieter's description in the implementation guide is a 'safe' solution for this. The description must be clear and well described for the implementer.

I agree that the escape should only be an escape and we should therefore not explicitly profile it. The implementation guide would be more suited.

NB. When "mapping" zib elements to these required elements, the mapping comment can be used to make clear that it is for example a "partial" or "derivative" mapping.

I think this is a good additional guideline we can incorporate in the Profiling Guidelines.

I think the first consideration should be if it's possible to "map" the required element to the zib concepts. For example, a .status might be inferred from an end date present in the zib.
I wonder if we can easily make those mappings for most zibs. If we can, shall we document it in the required element's comment? Would be nice if this can be added like a ConceptMap kind of resource.

In addition to this:
TextResultStatus and Observation.status both have 1..1 cardinality, but because of conceptual cardinality 0..1 is expected in FHIR. The outcome of this dicussion could be (should be?) applicable to this situation as well.

I would propose Option 1 because of the following:
Option 2: I could imagine use cases where it is valuable or necessary to add data-absent-reason to a not-required element if needed. If we explicitly add the extension on specific elements, do we implicitly exclude it from being present in other elements?
Option 3: Repeats information
Option 4: "the right information" in the FHIR spec would still lead to two options (data-absent-reason and nullFlavor). Receiving systems would need to implement both (and: without our guidance) if we don't document anything.
If we would go for option 1, we still have a decision to make whether to favor https://www.hl7.org/fhir/extension-data-absent-reason.html or http://www.hl7.org/fhir/extension-iso21090-nullflavor.html. Anyone working with v3 and/or mappings would probably prefer the latter, I think when working from ART-DECOR it is a more natural fit. FHIR-only perhaps prefers the former? I do think it is a plus that Observation.dataAbsentReason uses the same ValueSet as the extension.
Next to that: in cases where conceptual cardinality comes in as described above, I would still point to an IG how to handle this. If an informationstandard wants to specifically exclude the element in their profile, only then the explicit profiling of the extension comes in.

I agree with the argumentation of Jorn and would like to move on with option 1.
Regarding:

... favor https://www.hl7.org/fhir/extension-data-absent-reason.html or http://www.hl7.org/fhir/extension-iso21090-nullflavor.html. Anyone working with v3 and/or mappings would probably prefer the latter, I think when working from ART-DECOR it is a more natural fit. FHIR-only perhaps prefers the former? I do think it is a plus that Observation.dataAbsentReason uses the same ValueSet as the extension.

Good point. I am leaning towards the data-absent-reason extension. Also because the comment on the nullflavor extension states:

Considerable care must be used when using nullFlavor in this context of FHIR - more than any other concept, this is tied to the way v3 works, and FHIR is quite different. For instance, there is no notion of a "proper value" as opposed to any other value in FHIR. a NullFlavor should be understood to mean, "why information is missing", and the nulFlavors OTH, NINF, PINF, and TRC should not be used.

@pieter-edelman-nictiz Yes I actually assumed this to be the way to go.

I think the first consideration should be if it's possible to "map" the required element to the zib concepts. For example, a .status might be inferred from an end date present in the zib.

@pieter-edelman-nictiz  agree. Do we be default need to provide guidance on this in the profiles with regards to the zib concepts that may have a relation with the mandatory elements? Perhaps also here a sentence or two in the IG/profiling guideline will suffice?

Proposal for addition to profiling guidelines:
In chapter 10, add a section "Mandatory FHIR elements without a functional counterpart", with the following text:
On occasion, FHIR requires an element to be populated while the functional model doesn't specify it. For example, the Procedure resource has the required element Procedure.status, but the zib Procedure, which is mapped to this FHIR resource, doesn't specify such a field.
In these circumstances, an effort should be made to map the zib concepts implictly to these elements. For example, FHIR Procedure.status can be inferred from the zib concept ProcedureEndDate; if it is in the past, then the status should be assumed to be completed. If it is not possible to make such an implicit mapping, the FHIR element is left as-is; the implementation guide should describe how the implementer should deal with it, probably using the data-absent-reason extension.
Implicit mappings are documented in the following way:

The comment of the implicitly mapped element should provide guidance on how to map the functional concept.
The mapping element is populated with the mapped concept, but the description is altered to: "If the mapping is an implicit mapping to a required FHIR element, mapping.comment will be "[concept name] (implicit, main mapping is on [element name])".

In section 3.3.1 (Associating the functional definition to StructureDefinitions/Current implementation), add the following paragraph:
Note that implicitly mapped elements use a slightly different description.
In section <IP_ADDRESS> (Metadata for elements), change:
"For elements that map to zibs" into "For elements that map directly (not implicitly) to zib components"

The mapping element is populated with the mapped concept, but the description is altered to: "If the mapping is an implicit mapping to a required FHIR element, mapping.comment will be "[concept name] (implicit, main mapping is on [element name])".

I like the use of mapping to document the implicit mapping, including the suggested boiler text. However, will this result in adding more lines in known-issues.yml?

This is undesirable of course, the tooling should account for this.

Added this to the profiling guidelines. I'll leave the issue open because the tooling still needs to be adjusted,

https://informatiestandaarden.nictiz.nl/wiki/FHIR:V1.0_FHIR_Profiling_Guidelines_R4#Mandatory_FHIR_elements_without_a_functional_counterpart and #79

Was in doubt to create a new issue, but since this is related:
What to do in the ada2fhir mappings on these elements?
For example: Encounter.status should have an implicit mapping (will add that soon), one could base it on period.start and/or .end. However, one could argue that the context of the ada2fhir transformation is inherently unknown, therefor the .status should be fixed to 'unknown' in the ada2fhir mapping. One cannot know how much time passes between conversion and sending/receiving the message. However, you could also say that this approach is too lazy, but then it could be that for each ada2fhir mapping where this is needed, you need some custom logic.


Is there any way to clear a input field?

Is there any way to clear a input field or double click it ?

I try  tab.press_key("Delete"), but not work


Feat/add podspec

this PR adds a podspec to the project

the PR was opened by accident against upstream.


Implement more syntax errors.

Closes #178.

😅 that was some reading!
Are we interested in leaning into macros to simplify the organization of eval?

Maybe. I considered it here, but there are a number of different refactoring strategies possible — and I don't want to prematurely commit because the decision is entangled with a more purposeful refactor planned for post-alpha.


VisualStateManager in explicit Style/ControlTemplate affects instances that do not reference that Style/ControlTemplate

Describe the bug
Visual states defined within a custom control template may affect instances of that control that do not make use of that template, e.g. theme colors may not be properly applied for selected/hover states.
Steps to reproduce the bug

Extract the default ItemContainerStyle for ListBox. Do not modify the contents of the style/template. Make sure it is an explicitly-named style, not an implicit one. Put it into its own resource dictionary. Have this dictionary referenced by / merged into a dictionary at the App.xaml level.
Define ColorPaletteResources that are different than the default theme colors, e.g. change the value for Accent so the selected items in the ListBox would appear with a color that does not match the user's accent color in Windows.
Add a ListBox to a view and populate it through an ItemsSource binding with strings. Let it use the default ItemContainerStyle. Do not reference the extracted style/template.

Given these conditions, you would expect to see a list box whose selected items reflect the color specified for Accent. However, you will observe that the user's accent color is used instead.
If you delete the VisualStateManager from the extracted control template for ListBoxItem, then the Accent value in the theme dictionary is honored by the list box.
If you make the ControlTemplate empty, you can indeed confirm that this style/template is not in use because the list box is still presenting items.
Expected behavior
If my App.xaml contains theme dictionaries, but no other merged dictionary contains theme dictionaries, I would expect that styles/templates defined throughout my application to make use of those theme resources. It should not matter if the style/template of a control is defined in the Page resources, a merged dictionary that is part of App.xaml, or App.xaml itself.
Version Info
Windows 10 1909. Brand new blank UWP application targeting 1903 with no additional Nuget packages. Visual Studio 16.4.1
NuGet package version:
N/A

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

@kseitz10 Thanks for the bug report! If you have a minimal repro app that you could provide us that would be very helpful in tracking down the issue.

@chrisglein Is Framework team the right place for this?

ThemeTest.zip

Awesome! Thank you


1Password logo

The icon I want is:

Name: 1Password
URL: 1password.com
Link to icon: here

Vector version of the logo: same as above (4673 bytes)
Official style guide or brand guide: Press kit & brand assets | 1Password

I'm working on this, but I don't know if I can pull it off. There are 8 different rings. Using fills and strokes, I've accomplished that with 5 <circle> elements. But then there's the keyhole. It's med/dark blue, with a dark blue border, and a second white border outside that. I need to either create two almost identical paths, or define a path in a <defs> element, and then reference it twice, using transform scale. So far, it's not quite right, and I'm near or over 1kb depending on how I do it. And I haven't even started on the shadows between the 2 outer rings.

I believe in you.

Post what you've go so far, and we can all try to help.

I believe in you.

(:
https://github.com/btrem/SuperTinyIcons/blob/1password/images/svg/1password.svg
To maintain readability, I did not collapse whitespace.

That's looking really good!

It's already looking good, but this white rounded square background isn't a part of this logo, no?

The rounded square is part of all the icons in SuperTinyIcons. It is white for colored icons. It is colored where the icon is white.

I added the white outline to the keyhole. NB: this is a rough trace. It's at a bit over 900 bytes, but we still have to add the shadows.  :-o

Ohh, okay
I can't find why is that, though—wouldn't it make more sense to just leave them how it is? The user can add the background in CSS, if they're using it on the web, for example

You'd have to ask @edent (the owner of this repo). I suspect the idea is someone can copy the source code of any svg icon into a webpage with minimal fuss. Though, to be fair, one would have to set height/width in css, or constrict it in some other way.


Align existing FINOS GitHub repositories to delegated roster management

Description
All GitHub repositories must comply with new roster management rules set and documented on http://odp.finos.org/docs/project-collaboration#access-control

[ ] Add FINOS repositories for alignment to the List of FINOS Repositories below
[ ] Prioritise activity and resolve in order of check list

List of FINOS Repositories

[ ]  Repo 1
[ ]  Repo 2
[ ]  Repo 3
[ ]  Repo 4
...

Directly relates to https://finosfoundation.atlassian.net/browse/HELP-401

Consolidated these requirements into https://github.com/finos/open-developer-platform/issues/152

Consolidated these requirements into https://github.com/finos/open-developer-platform/issues/152


Kingdom Come Deliverance crashing with DXVK injected in windows 10

Kingdom Come Deliverance stopped working with DXVK on my laptop  after i'd updated the driver. Rolling back to the previous driver didn't solve the issue. Dx9 based games still work fine with DXVK. So I think something is wrong with loading the dxgi.dll.
System information

GPU:GTX 1660ti
Driver:552.44
OS: Windows 10
DXVK version: 2.3.1

Log files:
KingdomCome_d3d11.log
KingdomCome_dxgi.log

gplasync isn't supported here. Please try with regular dxvk

KingdomCome_d3d11.log
KingdomCome_dxgi.log
Same thing

There are no errors in the logs.
I would try to delete the dxvk cache file (in game folder) and maybe Nvidia Vulkan cache

Disabling Ansel did the trick. Thank you so much


Usage with torch.compile in Pytorch 2?

Is mup compatible with torch.compile() in Pytorch 2? If yes, what is the correct usage (e.g. should we apply mup before compile or after)?

I don't see why it might not be compatible right away, but I haven't tested it.
What happens to the coordinate check if you rerun one of our examples after torch.compile()?

Recently, torch.compile() started using FakeTensors for both input and weight during compilation. That means that infshape attributes do not exist on temporary FakeTensor weights. Consequently, MuReadout.forward() and MuReadout.width_mult() trip this assert and the compilation fails.
Going forward, this unwanted sideeffect will also influence the ability to eg. export mup models to ONNX.
Any advice how to circumvent missing infshapes on FakeTensors?


Fix script name in README

Script file run_random_agent.py doesn't exist in $/DP.
I think it is environment_demo.py.

Thank you for sending the modification! I fix the code at develop branch so I reflect this fix in develop branch.


sqlite song cache

Link Sqlite and sqlitecpp
Move xml profile logic to xmlprofile.cpp (And add an unused dbprofile prototype)
Replace song cache with 1 sqlite db file
Change song load logic : Load cache entries -> Check each cache entry's validity and finish loading (Or not) -> Differential reload songs and additionalSongs
Turn ImageCache default value to Off
New preference to not use directory hashes in cache load validity check
Turn a lot of SM/SSC loader functions into static ones(With member wrappers) so they can be used without a loader object when saving cache entries

Things that should probably be done before merging:

Replace vectors and MSD values being stored as strings/text in the db with the actual values and a relational model (Should be both faster and make things more clear/easier to read)
Find out why theres a diff in ScoresAtRate::CreateNode and probably remove it from the PR
Look into improving loading window messages to provide more info

Moved to branch expEverything


Add ttir broadcast op

We should add support for an explicit broadcast op in ttir.

@nsmithtt @sdjordjevicTT
Hey folks, I just wanted to clarify what we want to do about broadcasting.
Other dialects like ONNX, Tosa support implicit operand broadcasting. For reference ONNX mlir broadcasting. For example in Tosa, these are valid ops:
%0 = tosa.add %arg0, %arg1 : (tensor<2x1xf32>, tensor<1xf32>) -> tensor<2x1xf32>
return %0 : tensor<2x1xf32>

---

%0 = tosa.add %arg0, %arg1 : (tensor<17x16x15x14xf32>, tensor<15x1xf32>) -> tensor<17x16x15x14xf32>
return %0 : tensor<17x16x15x14xf32>

I talked to @sdjordjevicTT about cases where he hit asserts in our current compiler about eltwise ops having unequally shaped operands. To overcome this I see 2 options in tt-mlir:

Implement explicit broadcast op in TTIR.
a. add a pass in TTIR that inserts broadcasts where needed and adjusts tensor shapes, or
b. tt-forge creates broadcast nodes and emits those ops as all other ops in TTIR, leaving current eltwise constraints in TTIR intact.
Implement an interface for tensor shape inference, where for each tensor, its shape would be inferred and potentially reshaped if needed. We could forbid/permit this for certain op classes.

On a side note, IIUC, our toLayoutOp needs to know shape of each tensor it is applied to, so it can build MemRef. Am I right @nsmithtt ? So whatever option we choose from above, it should be done before ttir-layout pass?

@rpavlovicTT, what do you think is the most robust approach here?  I think implicit broadcasting could be a nice mechanism to have at TTIR dialect, and leave it up to the target backend to determine how to implement that.  This has a nice property of TTNN just being passthrough, since it supports implicit broadcast.  Metal backend can always run a pass (agreed, before toLayout) that turns the implicit into explicit.

I lean toward the implicit broadcasting too. Then we'd have to replace current eltwise trait with custom verifiers as you noted, and that looks good to me.
I am not sure how implicit broadcast would work wrt. tensor layout and memory configs. Given above example:
%0 = tosa.add %arg0, %arg1 : (tensor<17x16x15x14xf32>, tensor<15x1xf32>) -> tensor<17x16x15x14xf32>

do we pretend that tensor %arg1 is of dims 15x1xf32 and lay it out like that and let TTNN take care of broadcasting data, or should we actually worry about real dims in the compiler?

do we pretend that tensor %arg1 is of dims 15x1xf32 and lay it out like that and let TTNN take care of broadcasting data, or should we actually worry about real dims in the compiler?

Yeah it's definitely a bit tricky to express for tensor layout, I think by default we should have ttir construct it like this:
#layout = #tt.layout<(d0, d1) -> (d0, d1), undef, <1x1>, memref<15x1xf32, #tt.memory_space<l1>>>
tensor<15x1xf32, #layout>

We need to work out details for how this gets expressed for multi-core and/or tilized, but I'm not too worried about it because so far the TTNN runtime doesn't actually look at anything from the layout desc.  We'll only have to model this properly when we get to modeling L1 footprint.
For metal backend we want to transform the broadcasted tensor to be purely expressed as an iteration space, it'd look like this:
#layout = #tt.layout<(d0, d1, d2, d3) -> (d2, 0), undef, <1x1>, memref<15x1xf32, #tt.memory_space<l1>>>
tensor<17x16x15x14xf32, #layout>

Note that the tensor shape changes, becomes post-broadcasted, but the affine map itself implements the broadcast by dropping all of the broadcasted dims and replacing them with 0's.  It's a bit more explicit in this way.


minor changes to rail_hub to use base rail introspection instead importing everything explicitly

Done with https://github.com/LSSTDESC/rail_hub/pull/8


Add --hid-capture & --hid-replay options

This PR introduces the --hid-record and --hid-replay flags to support the use case of capturing events and replaying it (#4468). I have designed it such that it can also replay from a stream, currently through a fifo pipe to support the use case of targeting multiple devices at once (#4179). FIFO pipes are UNIX only, but the implementation is easily extensible to support streaming over a local socket (e.g. TCP, which would also work on Windows).
Documentation and unit tests are included.
Tested as follows:

The file format is plain text, line-based [timestamp] [accessory id] [hex-encoded space-separated event data]. This simple format is chosen to make it easier to edit the event data manually if desired. Blank lines and lines with # are ignored, everything else is rejected.
The parsing and serialization of the file format is fully covered by unit tests.
I have manually tested the functionality with real devices connected to a Linux laptop via USB.
Test devices were: Google Pixel 8, Samsung Galaxy A52s, Samsung Galaxy A53.
Test scenarios: record to one file, replay from one file, record (overwrite) existing file, replay from non-existing file, replay and record in one command, replay from a FIFO pipe, where input is recorded to. In all cases I also tested the effect of interrupting the replay early, with the expected behavior (program closes without issues).
I made sure that there are no memory leaks or memory safety issue by compiling with ASAN:
meson setup x --buildtype=debug --strip -Db_lto=true -Dprebuilt_server=/usr/share/scrcpy/scrcpy-server -Db_sanitize=address,undefined && ninja -Cx

A note on the design:

The implementation utilizes up to three extra threads to minimize the impact on the main and event thread (aoa).
To record events, one thread is used (to perform I/O - write to file).
To replay data, two threads are used: one to perform I/O (read from file/stream) and another to replay (waiting as needed to ensure that enough time has elapsed to replay the event). I chose two separate threads to make sure that the I/O and replay-sleep do not block each other from progressing as quick as possible. For ordinary local files, this does not really matter, but for data streamed from a fifo pipe (or in the future: sockets), blocking the pipe or socket could result in undesired interruptions or blocked pipes or closed sockets.

@rom1v Note: I have tested it extensively on Linux, but I haven't tried cross-compiling to Windows yet. I'll check whether it can compile successfully on Windows after your code review.

@rom1v The branch currently has merge conflicts due to changes in the past month. Before I rebase, I'd like to ask whether you have unsubmitted feedback or any other blockers to merging the PR.

First of all, thank you for your PR. It's a lot of work, with a lot attention to details (as per your PR description) :heart:
Sorry for the delay (the UHID feature took priority :wink:).
Streaming
Initially, I did not consider the "live" recording/replay of events, so loading the whole file (which should be small) then replaying it would be ok. But I like the idea to support this use case :+1:
Replay
During the replay, keyboard and mouse should be disabled (a mouse move during a replay breaks everything).
I think that replay does not even require mirroring, and there is even no need for a window (like OTG).
This could be a separate code path which just configures AOA/HID and replays synchronously (with some feedback in the console).
For me, the most important use case for this feature is to enable USB debugging on a device with a broken screen (but other people might have other use cases).
Threading
To minimize timing errors, the wait should be as close as possible as the input injection. So IMO, the thread waiting during ms_to_sleep (btw we should use µS instead of ms) should not post an event to another thread (sc_aoa_push_hid_event()) but should call sc_aoa_send_hid_event() directly immediately. In other words, the thread waiting for the deadline should be the thread which writes AOA over USB directly.
The thread which parses the file might be a separate thread posting the parsed events in a queue along with their timings, as you said:

I chose two separate threads to make sure that the I/O and replay-sleep do not block each other from progressing as quick as possible.

TBH, I think it might as well be the same thread, since the "injection" needs to wait for I/O to get the data anyway, so unless parsing takes a long time (in which case pipelining could improve the progress), I'm not sure it could make a difference. But why not.
Format
I think a binary format would be simpler. Parsing text in C is a PITA. I already parse some text because I have no choice in adb_parser.c, but if we can avoid that (more complexity, more code to maintain, more bugs…), that would be great. In another language (like Rust or Python), parsing text would be ok, but in C I really really prefer binary.
What do you think?
And I'm not sure there is an important need to edit HID event data manually in text.
What I have in mind is something like:
mouse:    |xx xx xx xx xx xx xx|02|xx xx xx xx|
keyboard: |xx xx xx xx xx xx xx|01|xx xx xx xx xx xx xx xx|
           \------------------/ ^^ \---------------------/
                 timestamp    device    HID event

(the size of the HID event might be deduced from the device)
I think we should add a protocol version at the beginning of the file, to be able to make changes later but still be able to replay old recorded files.
Also, a minor detail: the timestamps should start at 0 in the recorded file.
AOA/UHID
When you implemented the PR, HID was only supported via AOA. Now there is UHID.
But I think the feature should still target AOA specifically, because timings are captured and replayed on the client side (UHID would require to capture and replay on the server side, so the code could not be common).

 I'm sorry that the codebase changed a lot since your PR was written (causing conflicts).
Also, I'm aware that changing from a text format to a binary format is quite a big change, but the binary version should be simpler (I'd really prefer not to maintain more code which parses text in C when this is not absolutely necessary).
Thank you again :wink:

how this work please ? I absolutely need it

Because this PR has not been merged you would have to check out the branch locally and compile the code yourself.
Or you could wait until I found time to update the patch to implement the changes requested by the maintainer of this project.

I have an issue using the functionality and cannot get this to work. I recorded some mouse activity with

./run x --otg --hid-record=onyx.log

and then tried to replay.
Unfortunately, i get this error:

ERROR: Invalid HID replay data in onyx.log

This is the log file created with the tool. Any ideas what might cause the issue?
onyx.log


Octicons v2

Octicons v2 (v10.0.0) have been released https://primer.style/octicons/
Will require some work during upgrade; there are inconsistencies with how icons are used now and certain are missing. Some examples:

Private repo has no template icon now, and this is confirmed to be exact on GitHub however they also decided to present the type of repository in this way:

Interestingly they decided to keep one for public templates :\
Internal repos now use same icon as normal, but with organization avatar absolutely positioned near actual octicon (presumably, based on public discussion on octicons-v2 repo; the alternative is that there is unpublished v2 variant or organization icon is used along with avatar; would need to wait for GHE update)

The iconset now contains both 24px and 16px icons, all of them have identical width however not all icons exists in 24px set!
Other issues might be present but were not catched yet, feel free to add your observations here.

The npm package @primer/octicons-v2 should be pretty much a drop-in replacement. It contains 203 icons compared to the 205 from the @primer/octicons package, as internal-repo and north-star icons are absent in v2.

The iconset now contains both 24px and 16px icons

It still seems to have non-square files in the build directory:
$ cat<EMAIL_ADDRESS><svg xmlns="http://www.w3.org/2000/svg" width="12" height="16" viewBox="0 0 12 16"><path fill-rule="evenodd" d="M4 9H3V8h1v1zm0-3H3v1h1V6zm0-2H3v1h1V4zm0-2H3v1h1V2zm8-1v12c0 .55-.45 1-1 1H6v2l-1.5-1.5L3 16v-2H1c-.55 0-1-.45-1-1V1c0-.55.45-1 1-1h10c.55 0 1 .45 1 1zm-1 10H1v2h2v-1h3v1h5v-2zm0-10H2v9h9V1z"/></svg>

Actually, it seems @primer/octicons v10 is the "v2" update. Complete changelog here:
https://github.com/primer/octicons/releases/tag/v10.0.0
Notably, many renames.

Yes, along with renames there are some missing icons; sometimes icons with same name is different; for example old star is filled-in but new one is without filling, instead star-fill is filled in.

north-star icons are absent in v2.

https://primer.style/octicons/north-star-16

The changelog confirms that internal repo icon is now replaced by normal repo icon, so that means that GH just positions org avatar on top of it. They did that before too, but now there is no way for us to differentiate it without placing avatar; pretty annoying.

We should be good to go for it after https://github.com/go-gitea/gitea/pull/11891 and https://github.com/go-gitea/gitea/pull/11895

Given that we are setting size on some icons to 32, it might be good idea to use 16px set exclusively.

@primer/octicons@10 does include the missing star icon, it was just missing on the old v2 module.
A few webpack adjustments will be necessary with the update size is now in the filename, e.g. repo-16.svg and repo-24.svg.

just a question why are we switching to Octicons v2 - are the old icons not working for a specific usecase anymore?

It's just a version bump on npm dependency. Specifically from 9.x to 10.x

While I don't agree with every new icon, I think they are an overall improvement. And yes, remaining on old icons would mean we'd have to lock down the dependency and can't benefit from new additions either.

https://github.com/go-gitea/gitea/pull/12240


Fixing typo

"projecty" -> "project"

Docs Build status updates of commit a3bbeec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/get-started/wpf.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit c7a2735:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/get-started/wpf.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit d65daf0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/get-started/wpf.md
:white_check_mark:Succeeded
View

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

#sign-off and merged.


fix: remove deprecated async of test case

close #2051

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/ng-alain/ng-alain/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

denm


Typo in Join example

The example for join shows:
var arrayString = [
  'one'
  'two'
  'three'
]

output firstOutput string = join(arrayString, ',')
output secondOutput string = join(arrayString, ';')

I believe this should be:
var arrayString array = [
  'one'
  'two'
  'three'
]

output firstOutput string = join(arrayString, ',')
output secondOutput string = join(arrayString, ';')

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

ID: a36b1f58-7fc9-7d2f-108d-0a5c7c0c7c67
Version Independent ID: d295ee04-80c5-ba43-54d0-4594c2baa78d
Content: Bicep functions - string - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/bicep-functions-string.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@larryclaman - thank you for your comment.  You specify data type when you declare parameters.  However, you don't need to specify data type in variable declarations.  See https://learn.microsoft.com/azure/azure-resource-manager/bicep/variables.
#please-close


missing dependency eslint-config-react-app?

Current Behavior
after updating in our monorepo we found that tsdx failed on:
tsdx lint src

(node:82469) UnhandledPromiseRejectionWarning: Error: Failed to load config "react-app" to extend from.
Referenced from: BaseConfig
    at configMissingError (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:233:9)
    at ConfigArrayFactory._loadExtendedShareableConfig (//XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:712:23)
    at ConfigArrayFactory._loadExtends (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:617:25)
    at ConfigArrayFactory._normalizeObjectConfigDataBody (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:547:25)
    at _normalizeObjectConfigDataBody.next (<anonymous>)
    at ConfigArrayFactory._normalizeObjectConfigData (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:491:20)
    at _normalizeObjectConfigData.next (<anonymous>)
    at createConfigArray (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:307:25)
    at ConfigArrayFactory.create (/XXXXX/node_modules/eslint/lib/cli-engine/config-array-factory.js:362:16)
    at createBaseConfigArray (/XXXXX/node_modules/eslint/lib/cli-engine/cascading-config-array-factory.js:84:48)
(node:82469) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:82469) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
✨  Done in 0.94s.

After digging though both tsdx and eslint to figure out what went wrong
i found that if i add eslint-config-react-app to the project it worked
npm install --save-dev eslint-config-react-app
now the project is a NONE react project and a really small helper project.
It seems to me that tsdx is missing its sub requirement for linting.

Software
Version(s)

TSDX
0.12.1

TypeScript
3.4.5

Browser
NA

npm/Yarn
Yarn 1.21.1

Node
10.17.0

Operating System
MacOS

changed the title to reflect this. PR welcome

Hi yes, this just started failing after a recent rm -rf node_modules and npm install.  Something happened.
npm install --save-dev eslint-config-react-app did not fix my problem.

@fullstackwebdev oh dear. mind investigating further please? or file a different issue and link to this one since i'm not sure you have the exact same issue

I'm configuring tsdx lint for one of my libraries for the first time right now and I didn't get the error that eslint-config-react-app didn't exist, but that eslint-plugin-react and eslint-plugin-import didn't exist.
ESLint appears to require these to be installed in your project even if a dependency (like TSDX) installs them already, which is pretty tedious for certain things.

(node:44162) UnhandledPromiseRejectionWarning: Error: Failed to load plugin 'prettier' declared in 'BaseConfig': Cannot find module 'eslint-plugin-prettier'
Require stack:
-<EMAIL_ADDRESS>Referenced from: BaseConfig

is this the same problem?

Yea that's probably the same root problem. A "shareable config" would still have the same issue -- the longstanding https://github.com/eslint/eslint/issues/3458 is the root of all this I'm pretty sure.
I think the "plugin and config" workaround mentioned there would work as a proper "preset" as it would do both, but not totally sure.
resolve-plugins-relative-to might help here as well but I think that would only work if you don't use any plugins relative to your own project root. Not sure either.

If you are using a monorepo with yarn workspace, try applay nohoist on tsdx
  "workspaces": {
    "packages": [ ...],
    "nohoist": [
      "**/tsdx",
      "**/tsdx/**"
    ]
  },

see https://classic.yarnpkg.com/blog/2018/02/15/nohoist/

Yea that's probably the same root problem. A "shareable config" would still have the same issue -- the longstanding eslint/eslint#3458 is the root of all this I'm pretty sure.
I think the "plugin and config" workaround mentioned there would work as a proper "preset" as it would do both, but not totally sure.
resolve-plugins-relative-to might help here as well but I think that would only work if you don't use any plugins relative to your own project root. Not sure either.

I think this issue is related as well:
On running yarn lint, this error is generated:

I guess as eslint-config-prettier is not in the root node_modules, eslint fails to find it.
Manually installing eslint-config-prettier removes the error.

I faced similar problem.
I have fix the error after installing follow packages
@typescript-eslint/eslint-plugin
@typescript-eslint/parser
eslint
eslint-config-prettier
eslint-config-react-app
eslint-plugin-flowtype
eslint-plugin-import
eslint-plugin-jsx-a11y
eslint-plugin-prettier
eslint-plugin-react
eslint-plugin-react-hooks

Yeah this issue also bit me just now.
The problem is because as soon as a different eslint is installed (eg. the current version, 7.17.0), the other version inside tsdx is reorganized inside of <project dir>/node_modules/tsdx/node_modules.
I guess if tsdx wants to keep the dependency on eslint, it could instead running the version at <project dir>/node_modules/eslint - which would either run the version from tsdx or the user's version.
What do you think @sw-yx ?
$ yarn lint
yarn run v1.22.10
warning ../package.json: No license field
$ tsdx lint
Defaulting to "tsdx lint src test"
You can override this in the package.json scripts, like "lint": "tsdx lint src otherDir"
/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:265
        new Error(`Failed to load config "${configName}" to extend from.`),
        ^

Error: Failed to load config "prettier/@typescript-eslint" to extend from.
Referenced from: BaseConfig
    at configMissingError (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:265:9)
    at ConfigArrayFactory._loadExtendedShareableConfig (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:826:23)
    at ConfigArrayFactory._loadExtends (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:731:25)
    at ConfigArrayFactory._normalizeObjectConfigDataBody (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:660:25)
    at _normalizeObjectConfigDataBody.next (<anonymous>)
    at ConfigArrayFactory._normalizeObjectConfigData (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:596:20)
    at _normalizeObjectConfigData.next (<anonymous>)
    at createConfigArray (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:340:25)
    at ConfigArrayFactory.create (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:395:16)
    at createBaseConfigArray (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/cascading-config-array-factory.js:86:48) {
  messageTemplate: 'extend-config-missing',
  messageData: { configName: 'prettier/@typescript-eslint', importerName: '' }
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Yeah this issue also bit me just now.
The problem is because as soon as a different eslint is installed (eg. the current version, 7.17.0), the other version inside tsdx is reorganized inside of <project dir>/node_modules/tsdx/node_modules.
I guess if tsdx wants to keep the dependency on eslint, it could instead running the version at <project dir>/node_modules/eslint - which would either run the version from tsdx or the user's version.
What do you think @sw-yx ?
$ yarn lint
yarn run v1.22.10
warning ../package.json: No license field
$ tsdx lint
Defaulting to "tsdx lint src test"
You can override this in the package.json scripts, like "lint": "tsdx lint src otherDir"
/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:265
        new Error(`Failed to load config "${configName}" to extend from.`),
        ^

Error: Failed to load config "prettier/@typescript-eslint" to extend from.
Referenced from: BaseConfig
    at configMissingError (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:265:9)
    at ConfigArrayFactory._loadExtendedShareableConfig (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:826:23)
    at ConfigArrayFactory._loadExtends (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:731:25)
    at ConfigArrayFactory._normalizeObjectConfigDataBody (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:660:25)
    at _normalizeObjectConfigDataBody.next (<anonymous>)
    at ConfigArrayFactory._normalizeObjectConfigData (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:596:20)
    at _normalizeObjectConfigData.next (<anonymous>)
    at createConfigArray (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:340:25)
    at ConfigArrayFactory.create (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/config-array-factory.js:395:16)
    at createBaseConfigArray (/Users/k/projects/upleveled/node_modules/tsdx/node_modules/eslint/lib/cli-engine/cascading-config-array-factory.js:86:48) {
  messageTemplate: 'extend-config-missing',
  messageData: { configName: 'prettier/@typescript-eslint', importerName: '' }
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

A hacky workaround that I am using for now is to use Yarn resolutions in package.json as such to force tsdx to also resolve eslint to 7.17.0:
{
  "devDependencies": {
    "eslint": "7.5.0"
  },
  "resolutions": {
    "eslint": "7.5.0"
  }
}

A hacky workaround that I am using for now is to use Yarn resolutions in package.json as such to force tsdx to also resolve eslint to 7.17.0:
{
  "devDependencies": {
    "eslint": "7.5.0"
  },
  "resolutions": {
    "eslint": "7.5.0"
  }
}

@karlhorky yea resolutions are the best way to workaround this if you're installing an incompatible version. It is indeed because of the node_modules directory/sub-directory placement like you wrote, and resolutions override that.
My plan with Presets (#634 ) was also to make it possible to opt-out of pieces of TSDX

@karlhorky yea resolutions are the best way to workaround this if you're installing an incompatible version. It is indeed because of the node_modules directory/sub-directory placement like you wrote, and resolutions override that.
My plan with Presets (#634 ) was also to make it possible to opt-out of pieces of TSDX

Nice, yeah I guess Presets would also get the job done in a different way.
What are your thoughts on changing the eslint import path in index.ts in the meantime? Just so that people don't need to fiddle with resolutions...

Nice, yeah I guess Presets would also get the job done in a different way.
What are your thoughts on changing the eslint import path in index.ts in the meantime? Just so that people don't need to fiddle with resolutions...

Any update on this? I tried "eslint": "7.17.0" but did not work

Same issue: Error: Failed to load plugin 'prettier' declared in 'BaseConfig': Cannot find module 'eslint-plugin-prettier'
Installing eslint-plugin-prettier did not help either.

I had the error "UnhandledPromiseRejectionWarning: Error: Failed to load config "react-app" to extend from."
And running
npm install -D eslint-config-react-app

worked for me.

any updates on this? Facing the same issue

This setup works, I use it all the time for my repos @ayush-seth
https://github.com/thomashagstrom/npm-github-react-native

npm -f i eslint-config-react-app
when i run this command then it's resolved

Facing same problem here
{   "type": "module",   "module": "./dist/types.esm.js",   "main": "./dist/types.esm.js",   "types": "./dist/index.d.ts",   "exports": {     ".": {       "import": "./dist/types.esm.js"     }   },   "files": [     "dist/"   ],   "engines": {     "node": ">=16"   },   "scripts": {     "clean": "rimraf src/types/index.ts",     "start": "tsdx watch",     "build": "npm run create-index && tsdx build --format esm",     "test": "tsdx test",     "lint": "tsdx lint",     "prepare": "npm run build",     "size": "size-limit",     "analyze": "size-limit --why",     "create-index": "npm run clean && node ./scripts/create-index.cjs",     "lintsrc": "tsdx lint ./.eslintrc.js"   },   "peerDependencies": {     "react": "^18.2.0"   },   "husky": {     "hooks": {       "pre-commit": "tsdx lint"     }   },   "prettier": {     "printWidth": 80,     "semi": true,     "singleQuote": true,     "trailingComma": "es5"   },   "size-limit": [     {       "path": "dist/ifeel-types.esm.js",       "limit": "10 KB"     }   ],   "devDependencies": {     "@size-limit/preset-small-lib": "^8.2.4",     "@types/node": "^18.7.6",     "@types/react": "^18.0.37",     "@typescript-eslint/eslint-plugin": "^5.34.0",     "@typescript-eslint/parser": "^5.34.0",     "autoprefixer": "^10.4.14",     "eslint": "^8.2.0",     "eslint-config-prettier": "^7.2.0",     "eslint-config-react-app": "^7.0.1",     "eslint-plugin-import": "^2.25.3",     "eslint-plugin-jsx-a11y": "^6.5.1",     "eslint-plugin-prettier": "3.4.1",     "eslint-plugin-react": "^7.30.1",     "eslint-plugin-react-hooks": "^4.6.0",     "eslint-plugin-simple-import-sort": "^7.0.0",     "husky": "^8.0.3",     "react": "^18.2.0",     "rimraf": "^3.0.2",     "size-limit": "^8.2.4",     "tsdx": "^0.14.1",     "tslib": "^2.5.0",     "typescript": "^5.0.4"   } }


Exception: module 'OpenSSL.crypto' has no attribute 'PKCS12Type'

When use the script, have the warning above. Python 3.12.2 on Windows 11 23H2 Pro for Workstations.

Same problem here

This seems to be because pyopenssl deprecated PKCS#12 support. I was able to install an older version of pyopenssl as a work around with the command below.
pip install -I pyopenssl=24.0.0


Add ARIA: Comment

https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/Comment_role

I will be working on this!


Remove cypress test spec for dashboard button

Signed-off-by: suod-NithishKarthik<EMAIL_ADDRESS>Description
Dashboard icon in the header is removed since we have that in the navigation panel as of #3581 .
The test spec that verifies if that button is visible has been removed since it does not make sense anymore
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3803 (119d100) into master (0d1616c) will decrease coverage by 8.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #3803      +/-   ##
==========================================
- Coverage   20.81%   12.60%   -8.21%
==========================================
  Files          46       46
  Lines        3863     3863
==========================================
- Hits          804      487     -317
- Misses       2847     3261     +414
+ Partials      212      115      -97

Flag
Coverage Δ

e2etests
?

gointegrationtests
?

unittests
12.60% <0.00%> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/login.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-53.71%)
:arrow_down:

mesheryctl/pkg/utils/helpers.go
18.52% <0.00%> (ø)

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.90% <0.00%> (-39.10%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.42% <0.00%> (-21.71%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

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 789842d...119d100. Read the comment docs.


how to give direclty the melspec.npy file to the multi speaker parallelwavegan?

Hi,
Thanks for the great project, I need some help on fine-tuning the multi-speaker parallelwavegan using synthesizer generated melspec data for a particular audio file. I'm looking for preprocessing step of audio to melspec conversion file. please help out.  where I can change the code to read the melspec data directly.
Instead of audio to melspec conversion I just want to read the melspec npy directly.
Note I already finetuned the model with the audio. but audio quality is not good enough for synthesizer-generated melspec. but for original audio melspec audio quality is good enough. so that's why I planning to train melspec directly from the synthesizer. i hope if we train in this way audio quality will get better.
Thanks, @kan-bayashi
Vignesh E

https://github.com/kan-bayashi/ParallelWaveGAN#decoding-with-dumped-npy-files


Does not launch

The baguette does not launch. Starting from a service.
Baguette.makeText(getApplicationContext(), alertText, Baguette.LENGTH_SHORT).show();

Toast widget does show.

can you try with getContext() instead of getApplicationContext() ?

@florent37 getContext() is not available in WearableListenerService, but Alt+Enter to import getContext gives the wrong method.
import static java.security.AccessController.getContext;

tried with "this" and "getApplicationContext()", both didnt work


Build error in navbar.less

(08:53:06) [master] $ NODE_PATH=$PWD/node_modules /usr/local/bin/gulp rainloop:zip
[08:53:16] Using gulpfile ~/devel/mmv2/rainloop-webmail/gulpfile.js
[08:53:16] Starting 'js:libs'...
[08:53:16] Starting 'js:boot'...
[08:53:16] Starting 'js:openpgp'...
[08:53:16] Starting 'js:webpack:clear'...
[08:53:16] Starting 'less:main'...
[08:53:16] Starting 'ckeditor:clear'...
[08:53:16] Starting 'fontastic-fonts:clear'...
[08:53:16] Finished 'js:webpack:clear' after 206 ms
[08:53:16] Starting 'js:webpack'...
[08:53:17] Finished 'fontastic-fonts:clear' after 139 ms
[08:53:17] Starting 'fontastic-fonts:copy'...

/home/drew/devel/mmv2/rainloop-webmail/node_modules/gulp-less/node_modules/accord/node_modules/when/lib/decorators/unhandledRejection.js:80
		throw e;
		      ^
Error: #grid > .core > .span is undefined in file /home/drew/devel/mmv2/rainloop-webmail/vendors/bootstrap/less/navbar.less line no. 188

Since navbar.less doesn't appear to have changed since it's initial import, I'm not sure if there's something weird in the build env that I have wrong.  The relevant snippet is:
// Reset container width
// Required here as we reset the width earlier on and the grid mixins don't override early enough
.navbar-static-top .container,
.navbar-fixed-top .container,
.navbar-fixed-bottom .container {
  #grid > .core > .span(@gridColumns);
}

That's interesting.
Do you use a local gulp?

I followed the gulp instructions online by running:
npm install --global gulp

And then proceeded to install the downstream npm modules as the gulp run would fail that it was missing some dependency.  I'm also really new to gulp (before this morning I knew exactly nothing about it), so assume I don't know anything.


DNS-01 : error : Found recursive CNAME record to

Following the discussion I opened and after digging, I think I may have hit an edge case that break cert-manager or at least the error message is wrong.
The issue seems to come from here :
https://github.com/cert-manager/cert-manager/blob/860b143c05699cc4bbd5f1f54cf41e3be8db5a09/pkg/issuer/acme/dns/util/wait.go#L77-L101
Let me know if I'm wrong, first time I use cert-manager and I'm very new to kubernetes.
Also, let me know how I can help you troubleshoot this error if it is one.
Thanks a lot !
Discussed in https://github.com/cert-manager/cert-manager/discussions/5854

Originally posted by juchom March  8, 2023
I try to setup cert-manager with a public domain pointing to <IP_ADDRESS>
My config on cloudflare consist of an APEX A record and a wildcard record

I want to issue a certificate for both domain.dev and *.domain.dev and I have this error :

Found recursive CNAME record to "_acme-challenge.domain.dev." when looking up "_acme-challenge.domain.dev."

Here is my cluster issuer
---
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: domain-dev
  namespace: cert-manager
spec:
  secretName: domain-dev-tls
  issuerRef:
    name: letsencrypt-clusterissuer
    kind: ClusterIssuer

  duration: 2160h # 90d
  renewBefore: 720h # 30d before SSL will expire, renew it
  dnsNames:
    - "domain.dev"
    - "*.domain.dev"

I'm not sure how to fix this issue.
cert-manager version: 1.11.0

Hi @maelvls
Here is the output of the command :
dig _acme-challenge.domain.dev

; <<>> DiG 9.18.1-1ubuntu1.2-Ubuntu <<>> _acme-challenge.domain.dev
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 24896
;; flags: qr rd ra; QUERY: 1, ANSWER: 0, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 4096
;; QUESTION SECTION:
;_acme-challenge.domain.dev.     IN      A

;; Query time: 20 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>) (UDP)
;; WHEN: Fri Mar 10 18:25:48 CET 2023
;; MSG SIZE  rcvd: 54

Let me know if you need more information.
Thanks for your help!

Also checking the TXT record
dig txt _acme-challenge.domain.dev

; <<>> DiG 9.18.1-1ubuntu1.2-Ubuntu <<>> txt _acme-challenge.domain.dev
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 49336
;; flags: qr rd ad; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 0
;; WARNING: recursion requested but not available

;; QUESTION SECTION:
;_acme-challenge.domain.dev.     IN      TXT

;; ANSWER SECTION:
_acme-challenge.domain.dev. 0    IN      TXT     "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"

;; Query time: 10 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>) (UDP)
;; WHEN: Fri Mar 10 19:05:08 CET 2023
;; MSG SIZE  rcvd: 124

I don't understand why cert-manager's DNS query is responded with a CNAME record when it doesn't have one (as per the Cloudflare DNS zone screenshot). Can you run:
dig +trace _acme-challenge.domain.dev. CNAME

Let’s move to Slack, that’ll be more effective 😅 http://slack.k8s.io/

I'm running into the same issue. Did you find out what's causing it?

It seems to be an issue in docker desktop for windows.
It's opened here : https://github.com/docker/for-win/issues/13327


Partial "Orphaned" Stacktraces Emitted By Agent

At the bottom you'll find an extract from a real HP log, or at least the output I got by modifying the LogCollector to print the THREADSTARTs followed by the Thread Id, and the FRAMEs followed by the method name and id. (as an aside, the log is a fixed file run through sequentially by the LogCollector, there's no concurrency issues in play here).
I did this because I systematically saw some unexplained "top-level" methods in the tree view just under the thread level (so siblings of the java.lang.Thread.run method for example), which clearly didn't belong there.
The 2 lines in the middle (in other cases I saw up to 5-6 lines like this, but always in fairly "deep" stacks) between 2 THREADSTART events look like the bottom of a stack. Inspection for 2 different cases (looking at what's in the log and at the original source code being profiled) shows that these lines are "orphaned", i.e. they are not the "bottom" of the trace following the next THREADSTART (i.e. this eliminates the possible case that somehow spurious THREADSTARTS could be emitted in the middle of a stack being dumped).
My theory is that the agent starts emitting a stacktrace, and gets interrupted by the timer who asks for the next stacktrace, so the rest of the current stacktrace gets devnulled. I didn't look at the C++ code however, this is just a guess.
I can probably work around this in the LogCollector with a heuristic (peek at last frame before a new threadstart, if it isn't one of the very few top-level frames for the thread it belongs to, throw the stack away) but IMHO it should get fixed properly in the agent.
[Start of dump extract, anonimized a tad ;-) ]
[...]
FRAME com.[...].doExecute<PHONE_NUMBER>73232
FRAME com.[...].execute<PHONE_NUMBER>59688
FRAME com.[...].execute<PHONE_NUMBER>61520
FRAME com.[...].run<PHONE_NUMBER>30528
FRAME java.lang.Thread.run<PHONE_NUMBER>30536
THREADSTART 31885
FRAME com.[...].getNameBaseInternalPrivate<PHONE_NUMBER>79528
FRAME com.[...].getNextSiblingRefPrivate<PHONE_NUMBER>58576
THREADSTART 31885
FRAME com.[...].getContentPrivate<PHONE_NUMBER>18112
FRAME com.[...].getContent<PHONE_NUMBER>18384
FRAME com.[...].getNodeValue<PHONE_NUMBER>14848
FRAME com.[...].serializeTextNode<PHONE_NUMBER>12320
FRAME com.[...].serializeNode<PHONE_NUMBER>12520
[...]

A likely cause is the limit on stack depth, see if you can avoid this issue by increasing it to 1024 or something.

I may try that. However I would guess (with fairly high confidence) from the nature of what I'm profiling that these particular stacks are exact duplicates of some other stacks in the log, which would then be exactly as deep. I have to verify this (maybe tomorrow) but if I'm right, it's more likely to be a a timer issue.

By the way, wouldn't it make more sense if I test the converse ? Set maxFrames to a low number (e.g. 5) and see what happens ? If I end up with the bottom 5 lines of all stacks then that would be the best proof that you're right and that probably in a couple of instances the depth got exceeded. Or am I missing something ?

I tried setting maxFrames to 5. Now I indeed get a lot of "stack bottoms" - that is, every "stack" is 5 deep. And they all seem to be "real bottoms". That is to say, the bottom frame is practically always a "final" frame which doesn't even have anything to call anymore anyway.
However, this indicates that the behaviour of the agent really is to limit the number of frames dumped per stack to .
If in my original example I was running up against the default value of maxFrames (don't know what it is), the expected result of such behaviour would be that the "orphaned partial stacks" I have in those logs have depth == maxFrames. That is not the case : their depth is a lot less, typically between 2 and 6 frames in the example I investigated. If the default maxFrames is something like 50, these orphans would have contained 50 frames.
So I'm not really satisfied with the "maximum depth" explanation yet. I'll leave the case open. I may find the time to test a high maxFrames soon and see what drops out.
And @nitsanw - thanks for the quick reaction and the interesting suggestion !

@PhRX good reasoning, and indeed from your evidence I would say you are correct to assume max depth is not the issue.
Is the issue in the signal handler? The logic is very straight forward. Assuming the issue is not in AGCT itself it would suggest the ring buffer of preallocated frames is allowing concurrent writes to the same slot.
Another option is that in the aggregation logic the stacks get mangled. I'll have a look.
Mid way interruption of the signal handler should not result in mangling.
You can use the log dumper to dump the binary file in human readable form and try to correlate the mangled stack to each other as a means to further tracking the issue to the agent.


Non-host GMs can't use Moulinette for games hosted on The Forge

Unless an S3 bucket is configured which specifies a base URL, Moulinette references assets by relative path.
When hosted on The Forge, using the relative path will try to access the game host's asset library rather than the current user's. This means that non-host GMs can't reference their own Moulinette assets.
If running on The Forge and not using an S3 bucket, the path to the current user's assets library should be the base URL. Code like the following in moulinette-file-util.js getBaseURL could be used to get the base URL of the current user's assets library.
if (typeof ForgeVTT !== "undefined" && ForgeVTT.usingTheForge)  {
      const theForgeAssetsLibraryUserPath = ForgeVTT.ASSETS_LIBRARY_URL_PREFIX + (await ForgeAPI.getUserId() || "user");
      return theForgeAssetsLibraryUserPath ? theForgeAssetsLibraryUserPath + "/" : "";
}

Thank you for reporting this. I'll have to check this by myself to better understand how paths are defined on The Forge.

Implemented in v-4.5.1 (moulinette-core). It did fix the issue for non-host GMs.
However, I'm not 100% sure of the impact for host GM (who will also have their baseURL changed).
I'll monitor any issue related to this.

Had to roll it back do to use of "await" in non-sync function. Requires more effort.

I have been able to do all the required changes to make that function asynchronous. The new modules are available for beta testing. See : https://www.patreon.com/posts/66471992

Great! Thank you for your hard work


NZSL-173 Part 2: update qualtrics import url gloss extraction

Update the qualtrics import with the new url structure we expect to be present in the csv.
Extracts the gloss from the glossvideo.

Thank you for this 🙏🏼
I'm wondering if the conversation today has meant this will need further tweaking

I don't think so. The csv file from qualtrics should show the url that was uploaded. But this one is pending confirmation in form of an actual csv before it's merged.


Better map style and color palette

Switched to a better-looking map style, unified the color palette (green sensor dots and loading animations), and centered the loading animations.

Hey @brianstrauch thanks for the changeset!  I really like the new tileset :).
When I run this locally, I don't see the loading icons at all. Do you see them on your end? Also, I'm down for the new color scale, but I'd like to keep the legend on the side if possible.

My bad, forgot to commit the loading gif. I'll fix this up later!

All good!

Fixed!

Awesome @brianstrauch ! This looks great!


Storing pytorch models in vineyard and testing the transparent spilling of models

See also #772

Closing as the spill issue will be handled by #1628.


[ISSUE #6962] Format adjustment

reformat

Which Issue(s) This PR Fixes

Fixes #issue_id
Brief Description

How Did You Test This Change?


Leveled Assemblers loosing recipies

Not sure if you could connect this with #20 but when an Assembler levels up it looses access to certain recipes.
I could think of missing dependencies here but unsure about this.
Would be nice if a look into this could be taken.

What mods do you use?

Here is my mod-list.json file, be aware that I must have had to change a few mods on my own to get rid of some errors like 248k mod.
mod-list.json

Thats a lot of mods, all i can say is that it works with this mod standalone, you could do a binary search in disabling the mods until you find the mod responsible and then ask them to fix it.


Remove references to v2 commands, remove unused functions, and constants

What type of PR is this:
/kind code-refactoring

What does this PR do / why we need it:
Which issue(s) this PR fixes:

Fixes #5636
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

Used Goland editor inspection tool to identify unused functions, and constants.

As we will rewrite many commands that were existing in v2 and that have been removed for the time being, I would prefer not to remove the utility functions that are not used now but will most probably be helpful when we will write the new commands.
I would prefer to delete these ones when we are done with v3.

As we will rewrite many commands that were existing in v2 and that have been removed for the time being, I would prefer not to remove the utility functions that are not used now but will most probably be helpful when we will write the new commands.
I would prefer to delete these ones when we are done with v3.

Can you point out the functions? A few at least?

As we will rewrite many commands that were existing in v2 and that have been removed for the time being, I would prefer not to remove the utility functions that are not used now but will most probably be helpful when we will write the new commands.
I would prefer to delete these ones when we are done with v3.

Can you point out the functions? A few at least?

Here are some functions I think it would be interesting to keep for the moment:

GetProjectTypeFromDevfileMetadata
GetLanguageFromDevfileMetadata to be exhaustive on Get*FromDevfileMetadata
PreStart DevfileEventType = "preStart" and other event types to be exhuastive
all methods in pkg/log/status.go to have more choices on displaying output

As we will rewrite many commands that were existing in v2 and that have been removed for the time being, I would prefer not to remove the utility functions that are not used now but will most probably be helpful when we will write the new commands.
I would prefer to delete these ones when we are done with v3.

Can you point out the functions? A few at least?

Here are some functions I think it would be interesting to keep for the moment:
* `GetProjectTypeFromDevfileMetadata`

* `GetLanguageFromDevfileMetadata` to be exhaustive on `Get*FromDevfileMetadata`

* `PreStart DevfileEventType = "preStart"` and other event types to be exhuastive

* all methods in `pkg/log/status.go` to have more choices on displaying output

I removed the Get*FromDevfileMetadata methods because they were added IIRC to add them to telemetry data, I do not believe they would be useful someplace else.
The devfile events have been around for a long time and they haven't been put to use, so I removed them. And they're anyway constants, we can add them back with v2 as a reference.
About methods in pkg/log/status.go, they've been around for quite sometime(even during v2)  in an unused state. The new commands have a different UI/UX, I figured we would write new methods as and when required.

/lgtm

@feloy Can you lgtm again? I had to rebase.

/lgtm

/retest-required

/override Kubernetes Integration Tests/Kubernetes Integration Tests

/override Unit Tests/Unit Tests
IBM test passes.


fix parse of nested multiple binary ops (#782)

Overview
Bug fix - binary operations parsed incorrectly when used multiple times in a nested constraint
Related Issue / Discussion
Issue #782
Additional Information
The inner queries list was not clened between iterations, so the expressions were appended on the previous iterations' expressions, resulting in a wrong SQL query built. (first properties were duplicated into the second condition)
Contributions and Licensing
(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute bugfix #782 to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
[ ] I have already previously agreed to the pycsw Contributions and Licensing Guidelines

Thank you @barelsh for your contribution.
We will review in the following days


Upgrade to latest polkadot-js/tools

Per @rlaferla, the latest version is https://github.com/polkadot-js/tools/releases/tag/v0.53.5 and contains changes that @wilwade made.

@demisx There are still a lot of little bugs they are working out, so we can just keep it as is for now and we will keep testing it over time. I'm subscribed to the releases.


How to execute "insert into on duplicate key update"

Hello.
Currently I have a problem with a query "insert into on duplicate key update".
The values in "on duplicate key update" section are ignored and exception thrown as I understated it will be fixed in RC3.
For workaround I want to use procedure.
Does library support calling procedure or function that will be executed on particular shard?

No, sharding with store procedure is too complicated and it is not a good practice on sharding databases. We have no plan to do that


Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=1][exerciseId=4485]

Description
If possible, please provide a brief description of what is wrong with this exercise.
This question is confusing.
Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=1][exerciseId=4485]
Exercise payload: {"alternatives":["zdani","ta"],"sentences":[{"text":"ta zdani","lojbanic":true}],"title":"Select the house","type":"single-choice"}
Attempted solution: {"correctAlternative":"zdani"}
Attempted solution was classified as correct: no
Validation payload: {"correct":false,"correctAlternative":"ta"}

Thank you for the feedback!
Do you feel that the exercise could be improved simply with a change in the statement?

I honestly don't know how to format that same question in a way that makes sense.

Is the same exercise? It seems the problem has been resolved.

The issue raised by EmergencyTemporalShift is that a new student could think that the correct answer is "zdani" instead of "ta".
Pondering more about this, and also taking feedback from IRC, I feel that this mistake, despite not being extremely rare, is not that disruptive to the learning experience either. Students quickly notice that there is a pattern for exercises of the type "Select the house", "Select the x1", etc. So there will be at most a few failed attempts and students will learn. Balancing this against the fact that there is no straightforward to mitigate this (other than completely removing these exercises, which would be highly detrimental), I think this is acceptable for now.
If this later proves to be too disruptive, or if someone has an idea for a fix, we can revisit this issue. I am closing it for now.


balancers.Prefer + balancers.PreferWithFallback

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[v] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #146 (d520c69) into master (dd3e376) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #146      +/-   ##
==========================================
- Coverage   26.02%   25.99%   -0.03%
==========================================
  Files          62       60       -2
  Lines       10091    10051      -40
==========================================
- Hits         2626     2613      -13
+ Misses       7297     7272      -25
+ Partials      168      166       -2

Flag
Coverage Δ

1.14.x
25.99% <ø> (+0.04%)
:arrow_up:

1.15.x
25.99% <ø> (-0.03%)
:arrow_down:

1.16.x
?

1.17.x
?

macOS-latest
?

ubuntu-latest
25.99% <ø> (ø)

unit
25.99% <ø> (-0.03%)
:arrow_down:

windows-latest
?

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

Impacted Files
Coverage Δ

internal/table/client.go
86.10% <0.00%> (-1.21%)
:arrow_down:

cmd/gtrace/internal/test/test_gtrace_windows.go

...d/gtrace/internal/test/test_gtrace_darwin_amd64.go

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 dd3e376...d520c69. Read the comment docs.


Support Installation for Ubuntu 20.04

Currently is not possible to use azqr in Ubuntu 20.04 since the required verion of glibc is the 2.32 or 2.34 what is not available in Ubuntu 20.04 (uses 2.31)
See error log and the current glibc version
luisivg@AXLTNFfOnHNNlYK:~$ ./azqr scan -s dcap-dev -g dcap-dcapjfsbx-rsg-weu1-dev-snbx --debug
./azqr: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.34' not found (required by ./azqr) ./azqr: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.32' not found (required by ./azqr)
luisivg@AXLTNFfOnHNNlYK:~$ ldd --version | head -n1
ldd (Ubuntu GLIBC 2.31-0ubuntu9.15) 2.31

Fixed in version: v.1.1.5


fix: naming in protobuf

receiver_chainId => receiver_chain_id

Hi @lumtis, @fadeev
Can I contribute to this?

Hi @lumtis, @fadeev
Can I contribute to this?


fastdl module error

I have been trying to use the fastdl module however I get problems with it . the problem is the fastdl folder it is hard coded to {$rootdir}/www assuming  the folder needs to be resolvable quite a few web servers place there public facing code in a folder under the users root directory called public_html
example : /home/jimr/public_html/fastdl could resolve to http://mydomain.com/fastdl.
if the fastdl folder needs to be resolvable over the web you may want to check the apache or nginx setup files for the user for where the resolvable folder is (document root) these on apache2 are located in /etc/apache2/sites-enabled and are normally named something like mydomain.com.conf. so calling awk '$1~/^DocumentRoot/{print $2}' /etc/apache2/sites-enabled/mydomain.com.conf  will return something like

/home/gamer/public_html.

doing it this way relies on knowing the domain name but does give the resolvable folder the same is true if you set up a sub domain you may get a response like

/home/gamer/domains/player.mydomain.com/public_html.

dependant on where the doc root for the sub domain is. If fastdl folder does not have to be resolvable ignore this

Did you read this ?
https://github.com/dgibbs64/linuxgsm/wiki/FastDL

FastDL has to be resolvable. But it's still the user that has to setup his server accordingly.
It's intended to work with usermod or with manual virtual hosts.
For now, garry's mod is the only supported game.

that was what is was playing with but it is hard coded to www, so the user has to edit the module to reflect where the actual files are located ... tools such as Plesk or Virtualmin can take the problems of creating apache vhosts (configuring everything correctly for you) but do tend to use public_html are the doc root. I added the issue because I read

!'m no web admin.

so using Virtualmin (share/freeware) may be a lot easier to use than manually setting up a lamp stack and learning how to use it as Virtualmin does most of the work for you
example Vitrtualmin apache directives

<VirtualHost :80>
SuexecUserGroup "#1003" "#1003"
ServerName mydomain.com
ServerAdmin<EMAIL_ADDRESS>ServerAlias www.mydomain.com
ServerAlias webmail.mydomain.com
ServerAlias admin.mydomain.com
ServerAlias autoconfig.mydomain.com
DocumentRoot /home/gamer/public_html
ErrorLog /var/log/mydomain.com_error_log
CustomLog /var/log/mydomain.com_access_log combined
ScriptAlias /cgi-bin/ /home/gamer/cgi-bin/
ScriptAlias /awstats/ /home/gamer/cgi-bin/
ScriptAlias /AutoDiscover/AutoDiscover.xml /home/gamer/cgi-bin/autoconfig.cgi
ScriptAlias /Autodiscover/Autodiscover.xml /home/gamer/cgi-bin/autoconfig.cgi
ScriptAlias /autodiscover/autodiscover.xml /home/gamer/cgi-bin/autoconfig.cgi
DirectoryIndex index.html index.htm index.php index.php4 index.php5
<Directory /homegamer/public_html>
Options -Indexes +IncludesNOEXEC +SymLinksIfOwnerMatch +ExecCGI
allow from all
AllowOverride All Options=ExecCGI,Includes,IncludesNOEXEC,Indexes,MultiViews,SymLinksIfOwnerMatch
Require all granted
AddType application/x-httpd-php .php
AddHandler fcgid-script .php
AddHandler fcgid-script .php5
FCGIWrapper /home/gamer/fcgi-bin/php5.fcgi .php
FCGIWrapper /home/gamer/fcgi-bin/php5.fcgi .php5

<Directory /home/gamer/cgi-bin>
allow from all
AllowOverride All Options=ExecCGI,Includes,IncludesNOEXEC,Indexes,MultiViews,SymLinksIfOwnerMatch
Require all granted

RewriteEngine on
RewriteCond %{HTTP_HOST} =webmail.mydomain.com
RewriteRule ^(.) https://mydomain.com:20000/ [R]
RewriteCond %{HTTP_HOST} =admin.mydomain.com
RewriteRule ^(.) https://mydomain.com:10000/ [R]
RemoveHandler .php
RemoveHandler .php5
php_admin_value allow_url_fopen On
php_admin_value engine Off
IPCCommTimeout 31
FcgidMaxRequestLen<PHONE_NUMBER>

AuthName "mydomain.com statistics"
AuthType Basic
AuthUserFile /home/gamer/.awstats-htpasswd
require valid-user

#PerlRequire /etc/webmin/virtualmin-google-analytics/apachemod.pl
#PerlOutputFilterHandler Virtualmin::GoogleAnalytics
<Location /git>
DAV on
AuthType Basic
AuthName gamer.mydomain.com
AuthUserFile /home/gamer/etc/git.basic.passwd
Require valid-user
Satisfy All
RedirectMatch ^/git$ http://gamer.mydomain.com/git/gitweb.cgi
RedirectMatch ^/git/$ http://gamer.mydomain.com/git/gitweb.cgi
RewriteEngine off
AddHandler cgi-script .cgi

RedirectMatch /cgi-bin/mailman/([^/.])(.cgi)?(.) https://mydomain.com:10000/virtualmin-mailman/unauthenticated/$1.cgi$3
RedirectMatch /mailman/([^/.])(.cgi)?(.*) https://mydomain.com:10000/virtualmin-mailman/unauthenticated/$1.cgi$3
Alias /pipermail /var/lib/mailman/archives/public
Redirect /mail/config-v1.1.xml /cgi-bin/autoconfig.cgi

.

above is an average virtualmin conf file overwriting this with your simplified conf file could screw up loads of options that are set, in this example, by Virtualmin or by Plesk, my thought was for the script to find out what has been set and use it rather than overwriting the file. That said with customers I host I don't allow write access to those files so they are not editable, so no chance of changing them. So the user will have to edit the module.

I can rename www to public_html if you think it's more standard. I don't wish to move away files from the user though. LGSM has a philosophy of "everything inside the user folder" which i like and don't wish to change.
Virtualmin and Plesk users are not really my problem. I don't think LGSM is meant for them though.
A LAMP server isn't required, Apache2 is enough. Take 2 minutes to install and setup.
apt-get install apache2
nano /etc/apache2/sites-available/mywebsite.conf
# paste, edit stuff and save
a2ensite mywebsite
service apache2 reload

Done

I am thinking of the web interface here as it will need php & mysql enabled to work hence the lamp stack.
So I guess as things are the only place you can use this is either on a home pc that is acting as web/game server or an unmanaged VPS  unless I have missed something.
home pc option could run into DNS problems as the domain name really does need a static IP & certain ISP's may not allow you to open ports on the supplied router etc. etc.
unmanaged VPS you may need to run bind to propagate the DNS records.
I don't quite get this :-

LGSM has a philosophy of "everything inside the user folder" which i like and don't wish to change.

if you look at the example the user name is gamer and all references point to that users home folder.

Is there a reason why a simple simlink  public_html -> www doesn't solve the problem?

Yup, we can even change "www" to "public_html" into the code's variable if that helps.
I prefer "www" than "public_html" just because i'm used to work like that, but public_html is more standard.

Cool, another point I noticed when the module sets up it makes the folder under where the script is located, this may not be the users home folder ... I, for example, have the following for most users, where the game servers are installed under the games folder then game name, and public_html is located in the root, this approach stops http connections direct access to the game servers folders . perhaps consider adding the www/public_html folder to the users root rather than under the game root, also the user may want to add other content to www/public_html (forum or something) and it may be a bit messy having the web root folder nested in the game root folder. In the example below this user has 3 games installed, you could create 3 virtual hosts pointing to 3 folders under each game root (I know the example games do not all support fastdl it is an example) or just have all the web content under 1 folder split by game
e.g ~/public_html/gmod/fastdl and ~/public_html/fof/fastdl etc. this means 1 virtual host per user rather than 3
Sample users home folder
.
├── games
│   ├── css
│   ├── fof
│   └── gmod
├── Maildir
│   ├── cur
│   ├── new
│   └── tmp
├── perl5
│   ├── lib
│   └── man
├── public_html
└── Steam
├── appcache
├── config
├── depotcache
├── logs
├── steamapps
└── userdata
notes on above: Maildir email folder is there if you configure postfix to store user email in the users root rather than postfix's std which is /var/mail. This folder can be ignored along with the perl5 folder this is my dev user and am playing with perl web apps rather than php to see if there is an advantage using perl
With the www/public_html folder being nested a few folders deep I guess you have to give the apache user (default www-data) access to the users root folder ? As without granting www-data permissions to the folders above the public_html/www folder apache may throw a forbidden error as the apache user does not have access to them. To fix this you could change the apache user to the same user as the game server user. This leads me on to if you have 2 games, that support fastdl, running as 2 different users, if you have changed the apache user to match the user of game 1 the user of game 2 will not be able will not be able to use the fastdl module as the apache user is the wrong user !
To fix that point perhaps you need to point out that when setting up apache you could use suEXEC to allow different users to be the apache user to do this make sure suEXEC is installed and enabled in apache and add the required user & group id to the host file  .
example Directive :- SuexecUserGroup "#1003" "#1003"  this sets userid/groupid 1003 to be the apache user for say the web space me.domain.com, if you have a second game set up for fastdl under a different user just change the directive to the correct unix user in the hosts file.
@cedarlug  I don't like using symlinks in any web space they do work, but some clever hackers may be able to use a symlink to gain access to parts of the server they should not.

Got no time to read the whole thing now, gotta go play some music, but LGSM currently doesn't have a variable with the game's name. So yeah, it'd be cool to have ~/public_html/servername/fastdl but i see no way of doing it without a code rework.

you do have a servicename & gamename defined in the server script (gmod this is the LGSM script I am basing it on) perhaps you could use one of those ? I would use the servicename just in case someone runs more than one server on the same install.  enjoy listening to Led Zep  :smile:

What if a user has two server for the same game ?
We got no unique ID per server, that's the issue.
I play the drums, and sorry for the deception but i don't like Led Zep very much (and i've been blamed enough about that, don't worry) :p

ah I see ... never thought of reorganising the ports do people do this often ? oh well perhaps we need one :smile:

Maybe a little file in lgsm folder ?

whichever you think

Renamed to public_html
Won't change to a more advanced folder than the one set currently as it wouldn't fit current LGSM's policy.
However, if you wish to change it in your fork, it's quite easy, as the destination folder is set in two variables :
https://github.com/GameServerManagers/LinuxGSM/blob/master/lgsm/functions/command_fastdl.sh#L15
webdir="${rootdir}/public_html"
fastdldir="${webdir}/fastdl"

You can add any LGSM's variable that you wish. Port, servicename, and so on.


Weird refresh

I've just tried this component on a simple page with a div and the Typeform as the single child. I cannot make it work without this annoying refresh: http://sendvid.com/chc8cy5p
I have tried removing the classes from the div. Is this related to some internal <input> tag?
<div className={'container-fluid-content pay'}>
    <ReactTypeformEmbed url={'LINK'}/>
</div>

I have the same problem..

@PaulRBerg I updated version '"@typeform/embed": "^0.4.2",' in dependences. Issue took away =)

@PaulRBerg @vitalyliber I updated typeform/embed to the last current version (0.5.6).
It works properly now. Thanks! :)


Update value-types-table.md

Removed a typo causing an extra row to be in the type table

The ones that have links in the first column have pages in the language-reference/keywords folder, while those that don't have links are missing those pages. The links in the second column contain similar information only those link to the language-reference/builtin-types folder.
I'm not going to make any more changes for this pull request. Adding additional keyword pages should be another task for someone more involved in the project.

I think it won't require adding any additional pages, I didn't search, but there should be a page somewhere in the docs about byte for example which don't have hyperlink, even if not in the language-reference/keywords. Anyway, I may add that change later if you're interested in it.

there should be a page somewhere in the docs about byte

@Youssef1313 not anymore; PR #13054 has combined information about individual integral numeric types into one article, which is referenced in the table under consideration in the third column. So, I think missing hyperlinks are by design.

@hillmarcus @Youssef1313 Thanks for creating this, and the discussion.  As @pkulikov points out, you’re seeing the content during a time when some reorganization is happening.  This project contains issues that maps out the proposed reorganization of the C# reference.
There’s also more details in this gist for the entire project plan for an updated C# Guide.
As a result of some of these changes, the individual pages for many of the built in types are now consolidated (or in progress for that).
Thanks for helping us improve!