Modifying PC gen logic for branch prediction

There was some complex logic in the pc gen module that meant that branch prediction wasn't working correctly. To remedy this, we made a couple modifications:
-Simplifying the pipelining within PC Gen by removing a register and prioritizing taking a redirect PC over icache miss PC. This is not strictly something we expect a PPA improvement from, but it makes it much easier to hook up a branch predictor.
-For now, the BHT/BTB are replaced with static prediction (forward not taken, backwards taken).
We branched from the pparrot branch and compared PPA metrics post-synthesis:
Our minimum clock period is the same for both designs at 2.3 ns. The average instructions/s before optimization is 85,151,250 before optimization and 187,920,000 post optimization. This is a 120% throughput improvement which mostly comes from having an active branch predictor.
Because we commit a higher proportion of instructions, we also see improvements in energy efficiency. The average Avg J/Instruction is 1.35 * 10(-9) before optimization and 5.56 * 10^(-10) after our optimization. This is a 58% reduction in Avg J/Instruction.
Finally, we see an area improvement from 1,207,956 um^2 before optimization to 968,437 um^2 post optimization which is a 20% reduction in area. This is due to eliminating the more complex branch prediction logic. Although we did simplify logic and remove registers within the PC gen module itself, this is probably small in proportion to removing memories for the BHT and BTB. Because the BHT and BTB were not being used properly before though, we do not sacrifice performance for this gain in area.

Hi Katie, can you clone the FE schematic,  assign ownership to me, update it, and put a link to it (with google docs version number) into the Header of the FE top source code? Please try to keep it clean like the original and obey the schematic rules in BSG SystemVerilog Coding doc.
https://docs.google.com/presentation/d/1l0gdejJGftE4zZGD-kzK7WB7p4EeQBBNXIZc-Vjm94c

Can you also give the numbers in the more standard improvement “x” notation of computer architecture, where > 1.0 is good. So for example runtime speedup is (old cycles / new cycles), area improvement is (old area / new area) and energy improvement is (old joules per instr / new joules per instr)

Is this the correct style of notation?
Performance improvement (comparing instructions/s): new design runs 2.2x more instructions per second
Energy efficiency improvement (comparing avg J/instruction): old design used 2.43x more J/instruction
Area (comparing um^2): old design used 1.25x more area
Is<EMAIL_ADDRESS>the right Google account to switch ownership to?

Very close! You would say architecture B has 2.2X better instr/sec, 2.43X better J/instr and 1.25x better area (in um^2).
<EMAIL_ADDRESS>is the right account.

@HelloKayT I think you haven't prioritized pc redirect over cache miss. Because although you have moved it's if statement up in "pc_n " generation, the pipeline is still stalling on cache miss. I think you should change the stall definition to:
assign stall = ~(pc_gen_fe_ready_i  & (pc_gen_icache_ready_i | (fe_pc_gen_cmd.pc_redirect_valid & fe_pc_gen_v_i))) ;

Also PC is starting from '0 instead of bp_first_pc_p.

@farzamgl  For the latter, FE shouldn't actually start at a specific PC at all.  There is a fe_cmd reset, which should set the pc to the correct value and start fetching. Because this command isn't in bp_common in the class branch, we left it as a PC_redirect

Regarding stalling on I$ miss: the I$ ready signal is low while it takes a miss, so to immediately send out a request would violate the ready-valid handshake semantics. To properly respect the ready signal, the pipeline has to stall. When the pipeline unstalls, it will immediately take the redirect. Prioritizing redirect over I$ miss isn't a performance optimization for us so much as a logic clarification

Glad to see feedback from Farzam on this, and some healthy discussion among frontenders!

I guess I misunderstood the change you intended to make to the pipeline. But wasn't the module doing this before the change too? I think it waited out the cache miss time and then redirected the PC.

Yeah, the change is to eliminate 1 of the pipeline stages, so on a redirect / branch, we don't have a 1 cycle penalty just to latch the next_pc.  In order to stop waiting for an erroneous miss, we would need a mechanism to 'cancel' a miss

Got it

There was also an issue in the branch misprediction, right, Dan?
In terms of the fetch unit, how do we characterize the difference in performance for the before and after? I.e. how did the branch taken penalty change; or the branch misprediction latency, and the cache miss latency?

So quick clarification, the current front end does speculative icache misses?

There was also an issue in the branch misprediction, right, Dan?

Yes, the BTB as currently implemented did not output the correct prediction in the correct cycle. Katie just ripped it out and replaced with static prediction (for now).

In terms of the fetch unit, how do we characterize the difference in performance for the before and after? I.e. how did the branch taken penalty change; or the branch misprediction latency, and the cache miss latency?

Branch misprediction latency -1 (3 to 2)
Branch taken penalty +1 (1 to 2)
cache miss latency + 1 (N to N+1)

So quick clarification, the current front end does speculative icache misses?
Yes. I thought about this a bit in the context of security.  Possibly one solution is to wait to start a miss until the fe_queue drains. Of course, this is a performance hit so we would include the option as a flag.

I was thinking perhaps as a mode that the processor can enter into when
trying to be secure.
On Thu, Mar 7, 2019 at 3:02 PM Dan Petrisko<EMAIL_ADDRESS>wrote:

There was also an issue in the branch misprediction, right, Dan?
Yes, the BTB as currently implemented did not output the correct
prediction in the correct cycle. Katie just ripped it out and replaced with
static prediction (for now).
In terms of the fetch unit, how do we characterize the difference in
performance for the before and after? I.e. how did the branch taken penalty
change; or the branch misprediction latency, and the cache miss latency?
Branch misprediction latency -1 (3 to 2)
Branch taken penalty +1 (1 to 2)
cache miss latency + 1 (N to N+1)
So quick clarification, the current front end does speculative icache
misses?
Yes. I thought about this a bit in the context of security. Possibly one
solution is to wait to start a miss until the fe_queue drains. Of course,
this is a performance hit so we would include the option as a flag.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/black-parrot/pre-alpha-release/pull/62#issuecomment-470736309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApWrwN6a_fpufTrwkEaltSceriShSC58ks5vUZqAgaJpZM4biuIv
.

--
Apologies for all typos. Message sent by combination of an approximate
neural network and a smartphone.

Numbers update with the new test bench. This is post place-and-route with the clock period at 2.5 ns. All metrics averaged over the 4 benchmarks:
Instruction/s:  236,400,000 instr/s -> 293,000,000 instr/s (1.24x improvement)
Avg J/instruction: 3.05 * 10^(-10) J/inst -> 1.89 * 10(-10) J/s (1.6x improvement)
Area: 1228828 um^2 -> 981720 um^2 (1.25x improvement)
Overall, the testbench fix improved throughput/energy efficiency for both designs, although it had a much greater effect on the baseline version. We still get PPA improvements in all 3 areas though.


Eradicate type-specific "Ref" variants in schema

They're proliferating at a rate of knots and are increasing the noise and complexity of the schema. Instead, switch to using the single untyped Ref, and a single resolver function that returns different types.

Duplicate of #1043


add web_demo adapted from ChatGLM-6B

add web_demo.py adapted from ChatGLM-6B
update readme

Approved. Thanks for your contribution.


Add setting to disable syncdb, and do that by default

In production, there are frequent issues caused by discrepancies between syncdb and migrations. To avoid this, I suggest we build in a setting that disables syncdb and instead suggests you run manage.py sqlall > migrations/00-init.sql before adding any migrations. This way neither IT nor devs can run syncdb and find out that it breaks migrations.

I see the concern here -- AMO's syncdb didn't really work out of the box once (it might be fixed now).  However, I'm uneasy about turning off syncdb by default.  It is an important Django feature and there is a way to make inital_json data sets to Do The Right Thing.  Maybe there is a way to use post syncdb signals to set the correct migration ID?  I haven't looked into it.

Good point. My main concern is that syncdb will create a DB that represents what's current now, which is somewhere in the middle of the 20 migrations you might already have. There might be data migrations in there that will need to be run and won't be if we just set it to the latest migration number post-sync.

"Just use South" is also a valid answer, of course. But we don't currently use it on any playdoh projects.

+1 to south, I think our next playdoh inspired project should try to use south, and we should only use schematic in case of emergency - unless we find that schematic is superior, but really it's just Mozilla going on a detour.


Begin Pagination Support for Submissions

Description
This PR intends to support https://github.com/denoland/saaskit/issues/211 by adding simple pagination with a page URL parameters and a helper script to seed the db with some dummy data.
Adds

tools/seed_submissions.ts script to pull in dummy data from HN for testing
page option to specify the cursor (noting that there isn't yet a feed design that takes into account the date and score of the submissions)
More button to navigate to the next page

Example

Notes

Did not add numbering as this would require passing the start index along with the cursor which seems like a good indicator for a design discussion
Thinking there should be an iterator or indexer by "hotness" / "recency" as part of future work

Mostly LGTM! However, I'm now getting the following error after running deno task db:seed:
An error occurred during route handling or page rendering. TypeError: Cannot read properties of null (reading 'login')
    at Object.ItemSummary (file:///Users/asher/GitHub/saaskit/components/ItemSummary.tsx:42:23)
    at https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1830
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1857)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1074)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:2844)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:2844)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1933)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1074)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1933)
    at T (https://esm.sh/v119/preact-render-to-string@5.2.6/X-ZS8q/deno/preact-render-to-string.mjs:12:1933)

@iuioiua – Yep. To minimize the diffs, I elected to make User optional as there is a flow in which a post can exist without a user in prod (e.g. user created, user posts, and then user is deleted) independently of the KV database population. LMK if this is reasonable.

The user deletion flow is not yet defined or implemented. Even if it were, it'd likely delete all objects relating to the user, including posts.

@iuioiua – for context on my end, I saw a function and a corresponding test for user deletion -> https://github.com/denoland/saaskit/blob/main/utils/db_test.ts#L46

Ah, yes. deleteUser() is currently a test function. I'll create a PR to move it to the test file. Thanks for pointing that out!

@iuioiua – Could you clarify what changes you need for this PR?
For demo or local testing purposes, I feel like emulating real users via the item.userId as a fallback for the user ID works well.

The changes to <UserPostedAt /> and <ItemSummary /> should be reverted. These components shouldn't be affected by this PR. Instead, the new seed submissions script should populate the ["users", id] index. Again, the properties in the User object when creating the user are arbitrary. So these values don't have to be based on real data if it's too annoying to do so 😄

Also, could you add a short explainer in the README and seed submissions script about what it does?

@iuioiua – PTAL when you have the chance

[x] reverted changes to <UserPostedAt /> and <ItemSummary />
[x] created a dummy users with avatar URLs
[x] updated the Getting Started Locally in README.md
[x] updated the screenshot in the PR body


Minor Italian translation error

The "Extra" menu (top-right of the editor screen) says "Exta".  Extra is also "Extra" in Italian.
Cheers

Thanks. It was fixed.


Next 5.1.1 can not see the sharing button after set jiathis: true(5.1.1版本Next设置jiathis为true无法显示分享按钮)

Expected behavior (预期行为)
The bottom of the article can see the sharing button.(文章底部可以看到分享按钮)
Actual behavior (实际行为)
The bottom of the article can not see the sharing button(文章底部看不到分享按钮)
Steps to reproduce the behavior (重现步骤)
Share
jiathis: true
Warning: JiaThis does not support https.
#add_this_id:
NexT Information
Add the  ✔  sign before an item which is affected by this behavior.
NexT Version: 5.1.1

Master
Latest Release
Old version -

NexT Scheme: Mist

All schemes
Muse
Mist
Pisces

Other Information (Like Browser, System, Screenshots)
Chrome 57.0.2987.133, Win7, https://rangerzhou.github.io/

我的好像也是这样，本地可以，线上的不行

可能是因为https://xxx.github.io是https开头的

jiathis 不支持 https


[zh] improve chinese translation for control plane components

[zh] improve chinese translation for control plane components
I modified following files:
concepts/overview/components.md
glossaries of control plane component

/lgtm

Wow 😲
Looks good to me!
/lgtm

/approve
Thanks!


Organize and document

[x] Add Categories Based on the Topic/ Langauge used
[x]  Add a Table of Contents with links to each topic
[x] Add a node js code snippet (Show your Bussiness Card in the Terminal)

All done, @avincodes
Kindly let me know if I should do any more changes.
Also, Kindly if all is good if you give me the hacktoberfest label 😊


Properties in styles don't get applied if specified for base control

<Style Selector="Control.active">
  <Setter Property="Border.Background" Value="Red" />
</Style>

I think you need to use :is(Control).active here - the default OfType selector matches the exact type.

Actually, thinking about this some more, I think we need to swap the behaviors - by default TypeName should apply to a class and derived classes because you may well want to derive from a class and use the same styles, so repoening this.

@grokys Do you still want to change the behavior of the selectors here?

No, I don't think I do at this point, it would be too big a breaking change and I'm not even sure it makes sense.


How can i download the file

with import 'dart:html' as html;
was using this
html.AnchorElement(href: url)
// Set the download attribute to file name.
..setAttribute("download", "")
// Trigger a click event on the anchor element to initiate the download.
..click();
and it's was working absolutely fine
url: https://pdfobject.com/pdf/sample.pdf
how it'll work with web package

It's useful to look at the dart:html source code here to compare. The code should be similar. The only difference is that the AnchorElement constructor in dart:html took in an href that it explicitly set.
import 'package:web/web.dart' as web;
...
var anchorElement = web.HTMLAnchorElement()
  ..href = url
  ..setAttribute('download', '')
  ..click();

For Android build it's failing how we can use this as web and don't fail the android build

I'll need a bit more information, but if you're compiling to native, dart:js_interop is unavailable as it's not a web platform.


Make all generated public interfaces, classes and structs partial

To be able to more easily extend the generated concepts and interfaces and additionally to easily add the non-generatable parts of the languages (e.g. the behaviours) it would be helpful to have the generated types represented as public partial class|interface|struct XYZ types.

@enikao I would expect this to be a small extension to the generator - Do you think it is possible to have this implemented in the very near future?

Do we generate structs anywhere?

Not as far as I know - just wanted make the requirement more general. I'm fully satisfied if non-existing structs are not made partial 😬

Implemented in #32


Add support for Heatmap

this doesn't work for me,

'Mapview' has no propType for natiive prop 'AIRMap.showindoorLevelPicker' of native type 'boolean'

This pull was a mistake, the actual updated pull will come soon.


Accessibility: the labelText prop for DatePickerInput should be required

Detailed description
The labelText prop for the DatePickerInput is currently not required: https://github.com/IBM/carbon-components-react/blob/master/src/components/DatePickerInput/DatePickerInput.js#L27
End users are therefore able to use DatePicker and the child DatePickerInput without providing any assistive text.
So you can have a situation like in the range variation of the DatePicker in Carbon React's storybook, where there are no labels provided to describe what the form inputs do: http://react.carbondesignsystem.com/?selectedKind=DatePicker&selectedStory=range with calendar and min%2Fmax dates
i.e. --
<DatePicker
  id="date-picker"
  onChange={anonymous}
  minDate="1/10/2020"
  maxDate="1/20/2020"
  datePickerType="range"
>
   <DatePickerInput id="date-picker-input-id" light={false} onChange={anonymous} />
   <DatePickerInput id="date-picker-input-id-2" light={false} onChange={anonymous} />
</DatePicker>

For some reason, the a11y addon isn't flagging this component variation in Carbon React's storybook (perhaps due to the way the props object is passed in, or different storybook versions?), but the a11y addon does flag it in our environment:
#date-picker-input-id
    ✘ aria-label attribute does not exist or is empty
    ✘  aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty or not visible
    ✘ Form element does not have an implicit (wrapped) <label>
    ✘ Form element does not have an explicit <label>
    ✘ Element has no title attribute or the title attribute is empty

#date-picker-input-id-2
    ✘ aria-label attribute does not exist or is empty
    ✘ aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty or not visible
    ✘ Form element does not have an implicit (wrapped) <label>
    ✘ Form element does not have an explicit <label>
    ✘ Element has no title attribute or the title attribute is empty

Inputs need to have labels to go with them: https://www.w3.org/WAI/tutorials/forms/labels/
Even if the label text is implied, there needs to be some assistive text present (even if it's visually hidden).
By comparison, the labelText prop for the TextInput component is required: https://github.com/IBM/carbon-components-react/blob/master/src/components/TextInput/TextInput.js#L111
i.e. -
  /**
   * Provide the text that will be read by a screen reader when visiting this
   * control
   */
labelText: PropTypes.node.isRequired,

Considering that form elements must have assistive labels (even if visually hidden), I propose requiring the labelText prop for the DatePickerInput component. If someone sets hideLabel: true (https://github.com/IBM/carbon-components-react/blob/master/src/components/TextInput/TextInput.js#L143), then perhaps the required labelText in the input should be moved to an aria-label
Thoughts?

Just wanted to add: I'm happy to help with this.
As I said toward the bottom of my description:

I propose requiring the labelText prop for the DatePickerInput component. If someone sets hideLabel: true (https://github.com/IBM/carbon-components-react/blob/master/src/components/TextInput/TextInput.js#L143), then perhaps the required labelText in the input should be moved to an aria-label

But I'm open to other thoughts on this. Thank you! 🙏

Note: the issue I referenced above, where I was seeing labels not included in one of the stories for the DatePicker is caused by this issue: https://github.com/IBM/carbon-components-react/issues/1637

Hey @dakahn I was wondering what your thoughts are on this? (Note that the storybook has been updated to include labelText in the example, but that prop is still not required)

i don't see any reason to have labelText be optional. 👍 @asudoh thoughts?

@jendowns @dakahn Yeah if that prop is absolutely necessary to meet our a11y requirement let's make it required!

not stale ⚡

@jendowns were you still able to help out on this 😸

@dakahn Sure thing! PR incoming 😎


Documentation: Qt callPackage approach incompatible with pkgs/by-name

Problem
Our Qt section says:
https://github.com/NixOS/nixpkgs/blob/405d3ba628b96e8b29fb0c4350a7fe3e5765006f/doc/languages-frameworks/qt.section.md?plain=1#L25-L30
But this is not compatible with the new by-name directory structure.
Proposal
There are already plenty of packages in pkgs/by-name invoking qt5 or qt6 directly. And if the structure of both sets remain the same, maybe we can use callPackage ./../.. { qt6 = qt5; } or something?
Checklist

[x] checked latest Nixpkgs manual (source) and latest NixOS manual (source)
[x] checked open documentation issues for possible duplicates
[x] checked open documentation pull requests for possible solutions

Add a :+1: reaction to issues you find important.

We should probably update the docs, because at this point both qt5 and qt6 contain exactly one version and should continue being that way.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/ci-will-soon-enforce-pkgs-by-name-for-new-packages/38098/6


CSS classes very generic

The classes in your CSS are rather generic. I'm going to have to do some LESS "namespace trickery" to make sure they don't clash with any of the other classes in our rather large and complex site.

Do you mean the classes in  bootstrap-rating.css? This file is not required. If you are not using those symbols you can even remove it. It is a simple example of custom symbol.
You can create your own symbols with your own class names as long as you use those names in the data-filled and data-empty properties. For example, a square rating symbol would be something like this:
.my-own-symbol {
    display: inline-block;
    border: 5px double white;
    width: 30px;
    height: 30px;
}

.my-own-symbol-empty {
    background-color: #ccc;
}

.my-own-symbol-filled {
    background-color: black;
}

<input type="hidden" class="rating" data-empty="my-own-symbol my-own-symbol-empty" data-filled="my-own-symbol my-own-symbol-filled" />

See it in action

Hope this answered the question :wink:.


Apply default PodSecurity config for clusters with PodSecurityPolicy disabled

How to categorize this PR?

/area open-source
/kind enhancement
What this PR does / why we need it:

For clusters v1.23+ (where the new PodSecurity plugin was promoted to beta), add PodSecurity to default plugins
For clusters v1.23+ with PodSecurityPolicy disabled, and spec.kubernetes.allowPrivilegedContainers, apply the restricted level for all the namespaces by default and exempt gardener-resource-manager in the usernames.
If there is already a config set in the shoot spec for this plugin, gardener-resource-manager is added to exempted usernames and every other config is untouched.

Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/5250
Special notes for your reviewer:
In draft, depends on #6431
Release note:

If `PodSecurityPolicy` is disabled in the Shoot spec and if `spec.kubernetes.allowPrivilegedContainers` is set to false, the new `PodSecurity` admission plugin is activated with `restricted` level by default.

/test all

/assign

/test pull-gardener-unit

/test all

/hold
I've some testing left to do

/unhold

/hold few more changes.

/unhold

I have reworked the PR as discussed, PTAL! @rfranzke


As a Partnerbase I want to be able to add my county in my address and counties served below Agency Type.

Summary
In the address section in My Organization, the partnerbases need to be able to add their county. They also need to have a field for Counties Served underneath Agency Type:

Things to Consider

The section that requires changes in My Organization is: Agency Information.

Criteria for Completion

[ ] A field under the Zip Code field for: County
[ ] A field under Agency type for: Counties Served

@seanmarcia I would like to work on this.

Hi! To clarify, you'd just like two more fields to be added to this form? Or does this issue also involve the propagation of the data entered into those fields elsewhere?

Hi, @MaiRR and I are first-time contributors and would love to take a shot at this issue!

Hey @ilangela and @MaiRR ! It looks like @AngeVeza who created this issue hasn't marked it as help wanted yet so I don't know if it is ready to be worked on right now. She may have been waiting for feedback from the stakeholders. I'm going to close this issue for now since she is out on family issues and can re-open it when it is ready.
In the meantime we have another good starter issue here that you two might want to work on: #2545


Add ARM64 support

What about the cmake build?  Ideally we could only add ARM64 to the cmake build and use that to generate our NuGet packages (do we still have NuGet packages?).
At the very least we should add it to the cmake build since that is what is used to verify the build like this https://gltfsdk.visualstudio.com/build/_build/results?buildId=33


Improve upon the lexer design in #4

Cleans up and improves upon the lexer implementation from #4.
Todo:

[x] Get the evaluator while_loop unit test passing.
[ ] Get the syscall integration test passing.

* [ ]  Get the `syscall` integration test passing.

This seems to be a weird interaction with how std/linux-x86_64.stack uses std/let.stack, but I'm not too sure.


Removed the cert /token environment varibales support for KDD.

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

@caseydavenport was this what you had in mind?  Just wanted to check you didn't mean to completely remove those config params.


This is a great idea, but it won't work with most SVGs

Unsupported element "defs" - one of the most common elements in an svg document
Unsupported transform "matrix(...)"
Unsupported transform "translate(...)"
So what exactly do you support in terms of SVG for your conversion tools? If it's just the 1:1 mapping of elements in TinyVG that isn't practical. Your converter should be applying the matrix transformations for example before emitting the final coordinates in TinyVG rather than simply not supporting SVGs with a "transform" attribute in them, because all SVGs have them.
Anyway, the reason I ask is because I just spent a hell of a lot of time porting TinyVG to a more useful language for embedded (C++, w/ no STL - Zig isn't going to take off, BTW) but I can't actually do anything with it aside from process the examples you provide, which isn't helpful.
I was going to incorporate this support into my graphics library, but so far it seems I've wasted my time.
Please tell me I'm doing something wrong, and that you actually support basic SVG with your tools.
And if I'm not doing anything wrong, I'd maybe be willing to help put together a better SVG converter for your tools, but what you ship with isn't usable in the real world, for even simple documents.
Only thing is I don't code Zig.

Unsupported element "defs" - one of the most common elements in an svg document
Unsupported transform "matrix(...)"
Unsupported transform "translate(...)"

I do not support most of the SVG spec, because the conversion tool would be more work than anything else combined in the TinyVG project. What you can do to convert a lot of stuff is using an SVG optimizre like svgo to create a "shippable SVG" file which can then be processed by my tooling better.
Especially transform, translate and def are incredibly complex to process right and are not in no case part of simple SVG files.

Your converter should be applying the matrix transformations for example before emitting the final coordinates in TinyVG rather than simply not supporting SVGs with a "transform" attribute in them, because all SVGs have them.

Feel free to improve the conversion tool. And tell me how many ours you wasted doing so

because all SVGs have them.

That is definitly not correct. If it applies to your files, you should normalize them before using them as image files instead of shipping authoring files (svg is both a project format like .xcf or .psd and also an image format like .png or .jpg)

Please tell me I'm doing something wrong, and that you actually support basic SVG with your tools.

I do support basic SVG files, but the features you mentioned are definitly not "basic" but advanced and complex.

Only thing is I don't code Zig.

The conversion tool is written in C# and not in Zig as it's a much better tool to process whack and non-compliant XML files. .NET ships a quite resilient XML parser which is nice

Hmm, well I can do C#, but the issue is I've already created most of the parsing of SVGs in embedded C++. I can port it. I have something akin to XmlReader that I wrote for C++ to do peephole parsing. I can process an XML doc of any size in a fixed 64 bytes of RAM, but in any case, works similarly to XmlReader so maybe it won't be so bad to port.

In fact the tiger's head gave me the wrong impression that your conversion tool would support the elements that are in the reference SVG for it. Thank you for providing some suggestions for pre-cooking the SVG to a more shippable format, but absent a conversion tool that's more capable, I don't really want to impose multi step import processes for someone to use this vector format with my graphics library, and since SVG is what's for dinner in terms of source material, I'll just go back to supporting that.
I'm currently working on a major overhaul of that part of my library. My matrices are not being applied to spec right now for some reason, because I either don't understand the spec, I'm bad at math, or both. I'm thinking both. But I will get it working, and then once I understand what I need to do for that part, and can get it back to renderable content I'll poke at your conversion tool. I previously had it working based on Nanosvg but I rewrote the parsing part. The thing is, the result wasn't satisfactory performancewise or memorywise so I'm retooling everything.
Like I said, once I get my SVG situation sorted on my end, I'll have a crack at the conversion tool. Maybe we can incorporate some of the things that svgo does.
Thanks again for your advice


fix mlserver infer with BYTES

Closes https://github.com/SeldonIO/MLServer/issues/1210

@adriangonz done - should be fine now


feat: allow terminating Promotions

Backend implementation of #2662.
This allows non-terminal Promotions (e.g. pending, running) to be terminated per user request using the kargo.akuity.io/abort annotation (which we already use to abort verifications for a Stage).
Using kubectl:
$ kubectl annotate promotions <promotion> kargo.akuity.io/abort=terminate

Using kargo:
$ kargo promote --abort --name=<promotion>

Two small suggestions, but preemptively LGTM'ing.


Configure error with cmake --build . --config Release

I ran into an error running cmake --build . --config Release
Unfortunately, I do not know enough to identify the issue. Do I need to update my gcc version?
I get the following error message:
(build-env) [2472402@login1 build]$ cmake --build . --config Release
[  0%] Built target COMPONENTS
Consolidate compiler generated dependencies of target cifpp
[  6%] Building CXX object CMakeFiles/cifpp.dir/src/AtomType.cpp.o
In file included from /cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/bits/locale_facets_nonio.h:39,
                 from /cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/locale:41,
                 from /cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/regex:44,
                 from /cluster/gjb_lab/2472402/Projects/libcifpp/include/cif++/Cif++.hpp:31,
                 from /cluster/gjb_lab/2472402/Projects/libcifpp/src/AtomType.cpp:30:
/cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/ctime:80:11: error: '::timespec_get' has not been declared
   80 |   using ::timespec_get;
      |           ^~~~~~~~~~~~
gmake[2]: *** [CMakeFiles/cifpp.dir/src/AtomType.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/cifpp.dir/all] Error 2
gmake: *** [all] Error 2
(build-env) [2472402@login1 build]$

Here is some information of my compiler tools:
(build-env) [2472402@login1 build]$ which gmake
/usr/bin/gmake
(build-env) [2472402@login1 build]$ which cmake
/cluster/gjb_lab/2472402/miniconda/envs/build-env/bin/cmake
(build-env) [2472402@login1 build]$ which make
/usr/bin/make
(build-env) [2472402@login1 build]$ which gcc
/cluster/gjb_lab/2472402/miniconda/envs/build-env/bin/gcc
(build-env) [2472402@login1 build]$ gmake --version
GNU Make 3.82
Built for x86_64-redhat-linux-gnu
Copyright (C) 2010  Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
(build-env) [2472402@login1 build]$ cmake --version
cmake version 3.21.2

CMake suite maintained and supported by Kitware (kitware.com/cmake).
(build-env) [2472402@login1 build]$ gcc --version
gcc (crosstool-NG <IP_ADDRESS>_b0863d8_dirty) 9.3.0
Copyright (C) 2019 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

(build-env) [2472402@login1 build]$

The error you get here is not something I can fix. The problem is in a
C++ system header file that seems to be incorrect. I have no idea why
this happens.
-maarten
Op 2-9-2021 om 20:50 schreef Jia Geng Chang:

I ran into an error running |cmake --build . --config Release|
Unfortunately, I do not know enough to identify the issue.
I get the following error message:
|(build-env) @.*** build]$ cmake --build . --config Release [
0%] Built target COMPONENTS Consolidate compiler generated dependencies
of target cifpp [ 6%] Building CXX object
CMakeFiles/cifpp.dir/src/AtomType.cpp.o In file included from
/cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/bits/locale_facets_nonio.h:39,
from
/cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/locale:41,
from
/cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/regex:44,
from
/cluster/gjb_lab/2472402/Projects/libcifpp/include/cif++/Cif++.hpp:31,
from /cluster/gjb_lab/2472402/Projects/libcifpp/src/AtomType.cpp:30:
/cluster/gjb_lab/2472402/miniconda/envs/build-env/x86_64-conda-linux-gnu/include/c++/9.3.0/ctime:80:11:
error: '::timespec_get' has not been declared 80 | using ::timespec_get;
| ^~~~~~~~~~~~ gmake[2]: *** [CMakeFiles/cifpp.dir/src/AtomType.cpp.o]
Error 1 gmake[1]: *** [CMakeFiles/cifpp.dir/all] Error 2 gmake: ***
[all] Error 2 (build-env) @.*** build]$ |
Here is some information of my compiler tools:
|(build-env) @.*** build]$ which gmake /usr/bin/gmake
(build-env) @.*** build]$ which cmake
/cluster/gjb_lab/2472402/miniconda/envs/build-env/bin/cmake (build-env)
@.*** build]$ which make /usr/bin/make (build-env)
@.*** build]$ which gcc
/cluster/gjb_lab/2472402/miniconda/envs/build-env/bin/gcc (build-env)
@.*** build]$ gmake --version GNU Make 3.82 Built for
x86_64-redhat-linux-gnu Copyright (C) 2010 Free Software Foundation,
Inc. License GPLv3+: GNU GPL version 3 or later
http://gnu.org/licenses/gpl.html This is free software: you are free
to change and redistribute it. There is NO WARRANTY, to the extent
permitted by law. (build-env) @.*** build]$ cmake --version
cmake version 3.21.2 CMake suite maintained and supported by Kitware
(kitware.com/cmake). (build-env) @.*** build]$ gcc --version
gcc (crosstool-NG <IP_ADDRESS>_b0863d8_dirty) 9.3.0 Copyright (C) 2019
Free Software Foundation, Inc. This is free software; see the source for
copying conditions. There is NO warranty; not even for MERCHANTABILITY
or FITNESS FOR A PARTICULAR PURPOSE. (build-env) @.*** build]$ |
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PDB-REDO/libcifpp/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNA47GONY6AQI7TLI64MLTT77BPXANCNFSM5DJXSWVA.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

OK, as a follow up on this: if I search for timespec_get in the include files I get this:
$ grep -r timespec_get /usr/include
x86_64-linux-gnu/c++/10/bits/c++config.h:/* Define to 1 if you have the `timespec_get' function. */
x86_64-linux-gnu/c++/9/bits/c++config.h:/* Define to 1 if you have the `timespec_get' function. */
c++/10/ctime:#undef timespec_get
c++/10/ctime:  using ::timespec_get;
c++/9/ctime:#undef timespec_get
c++/9/ctime:  using ::timespec_get;
time.h:/* Time base values for timespec_get.  */
time.h:extern int timespec_get (struct timespec *__ts, int __base)

This indicates that there is an inconsistency between your C++ header files and your system header files. I think you should report this problem elsewhere, it is not something I can fix for you.


Settingup Prefix URL Causing Cookie Issue

We have installed superset as multi-tenancy environments and routing application through apache reverse proxy configuration.
Created HTTP_COOKIE, REQUEST_URI for prefix URL's and added the cookie in server locations.
Application is working as expected and able to access multiple clients with different prefix urls.
Ex:
RewriteEngine on
RewriteCond %{HTTP_COOKIE} _cookie=(client1|client2) [NC]
RewriteCond %{REQUEST_URI} !^/(client1|client2)/.* [NC]
RewriteCond %{REQUEST_URI} !^/(client1|client2)$ [NC]
RewriteRule    "^(.+)"  "/%1$1"  [R,L]
<Location /client1/ >
Header add Set-Cookie "_client=client1; path=/;"
Allow from all
ProxyPass https://domain-name:8089/
ProxyPassReverse https://domain-name:8089/

<Location /client2/ >
Header add Set-Cookie "_client=client2; path=/;"
Allow from all
ProxyPass https://domain-name:8099/
ProxyPassReverse https://domain-name:8099/

=============================
https://domain-name/client1/login/
https://domain-name/client2/login/
However, when the both applications opened in one browser and switching to tabs,  the current client cookie content is passing to the other client tab and the URL is getting changed.
Is there any solution to restrict the  cookies not to pass other tabs and url not changing from httpd or superset?
Environment
(please complete the following information):

browser type and version: Chrome/Firefox
superset version: 0.38
python version: 3.7

Can anyone please help on the issue?
Regards,
Syam


increase usability for "summary attributes" selection in details tab panel

for numerous features it is currently very tedious to select features that are supposed to be shown in the summary of selected points. thus the usability needs to be improved
e.g.

dialog that shows on the left a list of features that are not shown in the summary and on the right those that are shown; users can drag from left to right;
or sth similar to feature selection for embeddings where groups can be selected as one and collapsed

example how it looks like currently

created PR in PSE to merge solution into develop-cime branch as this is a feature that only affects the PSE side
see https://github.com/jku-vds-lab/projection-space-explorer/pull/189
please test whether it works for you as well, if it works merge, and close


Optimize mapped types

Optimize mapped types, focus on:

recursive types
mapped types

Resources

https://medium.com/@martin_hotell/10-typescript-pro-tips-patterns-with-or-without-react-5799488d6680
https://mariusschulz.com/blog/the-object-type-in-typescript
https://abstract.properties/typescript-compile-assertions.html#using-these-assertions-to-make-a-better-pluck
https://lorefnon.tech/2019/05/02/using-boxing-to-prevent-distribution-of-conditional-types/

Current playground diagnostics:

Metric
Value

Files
66

Lines
12574

Nodes
38223

Identifiers
13284

Symbols
16197

Types
13419

Memory used
50178K

I/O read
0.04s

I/O write
0.00s

Parse time
0.46s

Bind time
0.18s

Check time
0.84s

Emit time
0.00s

Total time
1.47s

Link to original issue from DefinitelyTyped repo https://github.com/DefinitelyTyped/DefinitelyTyped/issues/36318

Current state from 0.14 branch

Metric
Value

Nodes:
25755

Identifiers:
8713

Symbols:
10979

Types:
8669

Memory used:
36783K

Assignability cache size:
2772

Identity cache size:
14

Subtype cache size:
108

I/O Read time:
0.02s

Parse time:
0.42s

Program time:
0.50s

Bind time:
0.25s

Check time:
1.61s

Total time:
2.36s

Various fixes and missing features were introduced, the actual gains are higher then the numbers would suggest

0.14 update (switched to declaration merging and replaced IndexedModelClasses with Model.Class tuple:

Metric
Value

Files
24

Lines
7893

Nodes
25421

Identifiers
8566

Symbols
10476

Types
6125

Memory used
30794K

Assignability cache size
2064

Identity cache size
4

Subtype cache size
99

I/O Read time
0.02s

Parse time
0.20s

Program time
0.25s

Bind time
0.12s

Check time
0.62s

Total time
0.99s

Changes between 0.13 and 0.14 bring major reduction in complexity and compiler footprint, closing first optimization pass.


Timestamp columns in Postgres create 'trait bound is not satisfied' errors in Rust

Setup
Versions

Rust: 1.62.1
Diesel: 2.0.1
Database: Postgres
Operating System Ubuntu 22.04

Feature Flags

diesel: postgres, r2d2

Problem Description
Attempting to query against tables with timestamp or timestamp no timezone types results in a trait bound is not satisfied compiler error. Any table that doesn't include a timestamp column type works fine.
What are you trying to accomplish?
Simply getting the first row of a table.
What is the expected output?
Store the query result in a variable of the desired db model type.
What is the actual output?
None. Compiler error
This is the schema for a test table in PG:

id
date_created

[PK] integer
timestamp without time zone

This maps to the following in schema.rs:
diesel::table! {
    TestTable (id) {
        id -> Int4,
        date_created -> Timestamp,
    }
}

and my model looks like this:
#[derive(Queryable, Debug)]
pub struct TestTable {
    pub id: i32,
    pub date_created: NaiveDateTime,
}

Now, when attempting to do the following, I get the compiler error:
use crate::models::{TestTable as TT};
use crate::schema::TestTable::dsl::*;
use diesel::prelude::*;

...
// the method called on TestTable doesn't matter, I'm just using .first() since it doesn't require any other methods to return the result
// The only exception is that .find() works but that's the only method I've found that doesn't return an error
let first_res: TT = TestTable.first(connection).expect("");

This is the full compiler error output:
the trait bound `(i32, NaiveDateTime): FromStaticSqlRow<(Integer, diesel::sql_types::Timestamp), Pg>` is not satisfied
the following other types implement trait `FromStaticSqlRow<ST, DB>`:
  <(T0,) as FromStaticSqlRow<(ST0,), __DB>>
  <(T1, T0) as FromStaticSqlRow<(ST1, ST0), __DB>>
  <(T1, T2, T0) as FromStaticSqlRow<(ST1, ST2, ST0), __DB>>
  <(T1, T2, T3, T0) as FromStaticSqlRow<(ST1, ST2, ST3, ST0), __DB>>
  <(T1, T2, T3, T4, T0) as FromStaticSqlRow<(ST1, ST2, ST3, ST4, ST0), __DB>>
  <(T1, T2, T3, T4, T5, T0) as FromStaticSqlRow<(ST1, ST2, ST3, ST4, ST5, ST0), __DB>>
  <(T1, T2, T3, T4, T5, T6, T0) as FromStaticSqlRow<(ST1, ST2, ST3, ST4, ST5, ST6, ST0), __DB>>
  <(T1, T2, T3, T4, T5, T6, T7, T0) as FromStaticSqlRow<(ST1, ST2, ST3, ST4, ST5, ST6, ST7, ST0), __DB>>
and 24 others
required because of the requirements on the impl of `FromSqlRow<(Integer, diesel::sql_types::Timestamp), Pg>` for `models::TestTable`
required because of the requirements on the impl of `load_dsl::private::CompatibleType<models::TestTable, Pg>` for `(Integer, diesel::sql_types::Timestamp)`
required because of the requirements on the impl of `LoadQuery<'_, PooledConnection<ConnectionManager<PgConnection>>, models::TestTable>` for `SelectStatement<FromClause<schema::TestTable::table>, query_builder::select_clause::DefaultSelectClause<FromClause<schema::TestTable::table>>, query_builder::distinct_clause::NoDistinctClause, query_builder::where_clause::NoWhereClause, query_builder::order_clause::NoOrderClause, query_builder::limit_offset_clause::LimitOffsetClause<query_builder::limit_clause::LimitClause<diesel::internal::derives::as_expression::Bound<diesel::sql_types::BigInt, i64>>, query_builder::offset_clause::NoOffsetClause>>`

Things I've tried

Verifying that NOT NULL is enabled in PG for the date column
Messing with different DateTime types in chrono (DateTime, NaiveDate, etc)

It just seems like diesel can't resolve the Timestamp type. I'm not sure how to fix this as I've come across a few other forum posts regarding the issue and they all come down to mismatched types between PG and Rust and I've double-checked all my mappings, even following this table:
https://gist.github.com/steveh/7c7145409a5eed6b698ee8b609b6d1fc#file-postgresql-diesel-rust-types-md

It seems you should enable chrono feature

Yes this is due to a missing chrono feature on OP's side.
I'm closing this issue as our issue tracker is no support forum and that's clearly a users error (it's explicitly documentation in quite a lot of places that this feature is required)
@jakerieger For future reports: Please do not remove parts of the issue template, search for duplicates first and prefer to use the discussion forum for questions that are not clearly a bug in diesel. (In anycase: That would have been either a feature request "Support that type mapping" or a question: "What's wrong with my code", but not a bug report).


Avoid hardcoding user "user"

The default user may not be named "user", but will always be in group "qubes".

PipelineRetry


Translate Byron's address from Estonian

The first line of the privacy policy has some strange language in it:

The controller of personal data processed via the govtool.com ("Webiste") is Byron (hereinafter: "Byron") registered Byron Network OÜ; Registrikood: 16423145 Tegevusaadress: Lõõtsa tn 5 // Sepapaja tn 4 Tallinn Harjumaa 11415, electronic contact<EMAIL_ADDRESS>website https://byron.network/# .

The address should be transalted to English.
Also, there are references to the Polish office for data protection. Is this relevant to users outside of Poland?

@MSzalowski Is Byron team able to provide answers on @m-i-k-e questions? For the use of estonian language, the address itself shouldnt be translated, but the address label should. Also, there is a gap with Poland being explicitly mentioned while the company is registered in Estonia.

I've reassigned this to @MSzalowski because I think he's in the best position to get this fixed.


Non-Deterministic Shorthand-Longhand Resolution

I just found this library on Twitter and really like it, great job!
As the author of fela, I've been working with Atomic CSS for quite a while and stumbled into an issue that I now also face with restyle. I wrote a long article about it, but let me try to sum it up with a little replication:
Due to Atomic CSS, the order of rules matters a lot in stylesheets. This is especially important when mixing shorthand and longhand properties e.g. padding and paddingLeft. Right now, style resolution is not deterministic. Instead the order determines what styles are applied in the end. Here's a little example:
import { css } from "restyle"

function Box({ style, as: As, children }) {
  const [className, styles] = css(style)

  return <>{styles}<As className={className}>{children}</As></>
}

const App = (
  <>
    <Box style={{ padding: 0, paddingLeft: 20 }}>Box 1</Box>
    <Box style={{ paddingLeft: 20, padding: 0 }}>Box 2</Box>
  </>
)

In this example, paddingLeft is rendered last, so both elements get a 20px left padding. If you switch the elements up, both get a 0px left padding. This example is simplified of course, but it shows the essence of the issue.
The way we solved it in Fela (see article) is using repeated selectors e.g.
.a.a { padding-left: 20px }
.b { padding: 0px }

That way the output is always deterministic. In this case favouring longhands, but could be a configuration option of course. I know this might require sth. like a createCSS factory with configuration options of some sort, but without I'm afraid I can't really safely use it in production.
Oh, and not mixing those also doesn't really work, as I often extend my components multiple times merging a dozens of style objects which eventually might result in mixed styles.

Thank you for the kind words and filing an issue for this!
My plan to solve this was always parsing rules into long-form and merging them together while only returning the class names that should be applied. For example:
padding: '0px calc(1h - 1ch) 1rem' -> { paddingTop: '0px', paddingRight: 'calc(1h - 1ch)', paddingBottom: '1rem', paddingLeft: 'calc(1h - 1ch)' }
Just checked your article and wondering if I should lock it down and require only long-form properties though 🤔 since there's no build step I do want to keep an eye on performance. The repeated selectors implementation is a nice workaround! Another option could be a strict mode that didn't allow shorthand properties to reduce additional processing.

I also tried expanding shorthands at some point (see article) which resulted in another package inline-style-expand-shorthand, but that method has some clear downsides:

increased bundle size:  bundlephobia is down right now, but it adds quite some code to the bundle
performance: needless to say, every object iteration is a performance hit on client components
complexity: padding might be an easy one, but breaking down border and other shorthands can be a tricky task

I'm all in on restrictive APIs, but I'm not sure that requiring only long-form properties would be please most users 🥲

I took a look at expanding shorthands, but like you said it has downsides. I ended up with a nice approach that didn't increase the bundle size too much that takes advantage of the precedence prop. Details are in https://github.com/souporserious/restyle/pull/3 if you're curious. Thanks again for filing an issue!


fix: fix @tsed/logger version on each package.json

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

v7.85.1
GitHub release

Your semantic-release bot :package::rocket:


New rotational displacement example

Created new rotational example based on APDL code supplied by
@RGPATCHI.
Code plots the subsequent results and uses Query to
extract specific displacement values.
This didn't get merged into main last time; I think this was a mistake of PR merging order (on my part).

Technically this has already been reviewed in #519


How do we edit a control template for WinUI 3 alpha while the designer isn't working?

This issue has been moved from a ticket on Developer Community.

While the Xaml designer isn’t working, we don’t have the option to right click a control to Edit Template. Is there another way to get access to the default template?
PS. This page crashed several times (threw exceptions) when I clicked away to VS and tried switching back. It kept throwing exceptions when I closed the popup exception window. Took a few goes before returning to the page. Edit: It was the devcommunity2 page.

Original Comments
Feedback Bot on 7/20/2020, 02:08 AM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.
Nikola Mihaylov [MSFT] on 7/20/2020, 01:56 PM:
@Gavin Williams re: feedback crash/exceptions: thanks for the info! Fixed as of this morning at 11 AM PST (~12 hours after this report.) To make sure we fixed the feedback bug you mentioned: Could you please confirm if you still see any feedback site crashes or exceptions?Thanks!

Original Solutions
(no solutions)

This wont be an issue after OSS, however right now I think there is not way to do this... @MikeHillberg FYI

Along those lines it's same problem for control libraries built on top. Historically we're supposed to include the generic.xaml file in the nuget package, so that the designer can use this, but the pack step doesn't include it, but only packages the template inside the .pri file.

It is unfortunate but we do not plan to have the designer available at the time we'll be shipping WinUI3. @dotMorten we do ship generic.xaml with our Nuget package since Preview2.

@fabiant3 I know. It was more to point out that your Pack step doesn't pack the xaml when packing a WinUI class library with controls

@fabiant3 I don't know about the designer plans. But wouldn't it possible to populate at least the Document Outline window, so that elements can be right-clicked there to get this functionality?

Ah, sorry, just noticed that this issue was already closed. I think the current state is not ideal. In Visual Studio, there should be a functionality to create a copy of a ControlTemplate, with or without a designer.


How can I get the network map collection through f5 sdk for python

I am working with F5 sdk for python,
On F5 UI: Local Traffic > Network Map,  I can set 'Type field' to 'Nodes', in search field I provide 'my_node_name'. By updating the map, I can have details about pools that use  'my_node_name'.
Are there some ways to retrieve that details using  f5 api for python ?
How can I load node collection belonging to a specfic partition ?
I tried this : f5_connection.tm.ltm.nodes.get_collection(partition='my_partition')
but still having all partition
How can I get pools where a specific Node has child(chidrens) ?
Thanks advance for any answers

Hi @knightRainbow, sorry I missed this inquiry. For your first question, this is done by passing request parameters and using the odata filter property:
x = b.tm.ltm.nodes.get_collection(requests_params={'params': '$filter=partition eq bc'})
Please clarify your second question.

Hi,
I have the same question "How can I get pools where a specific Node has child(chidrens) ?"
My aim is getting of "network map" for IP address. This IP can be VIP or its member (node). Search provided by UI (Local Traffic -> Network Map) works exactly like I want.
It's pretty easy to search by Virtual IP, but I cannot find an optimal way to search by Member IP. For example, I can check every VIP and their pools, but I will make a lot of pool.members_s.get_collection() requests to LB . In my case it will be ~500 requests... So, is it possible to get the pool by node IP where this node is used as member?

@jasonrahm I have found  alternantives for my needs.
Thanks for your answer anyway ;)


( Get-NetTCPConnection -State Listen ) vs (netstat -tlnp in linux)

Summary of the new feature/enhancement
I want to see the listening port and its program at the same time in windows.
But I only see the port , and the program name is not available in windows.
Bellow is the output from LINUX that implement similar feature.
I hope Get-NetTCPConnection have the same feature.

Proposed technical implementation details (optional)

Get-NetTCPConnection is not in the repository. Use UserVoice site to send your request to MSFT owner team.

@iSazonov Where is the command Get-NetTCPConnection maintained? Is it available on Linux/macOS? What's the scope of the core functions/core utilities of PowerShell?

@hcoona The cmdlet is in NetTCPIP module. The module is Windows only.

Arch Linux
On Mon, Dec 7, 2020, 12:21 AM Ilya<EMAIL_ADDRESS>wrote:

@hcoona https://github.com/hcoona The cmdlet is in NetTCPIP module. The
module is Windows only.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/13013#issuecomment-739754967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ6WPO574P6S4R2EVJWXMETSTSF65ANCNFSM4OFFXX4A
.


Capacity overflow

Need a fix for overflow of capacity/encumbrance due to high strength stats. (higher than 127)

This is a problem in the base game, but it looks like it is fixed?


i3 window manager has problem with url display bar on lower left hand corner

So as the title implies, the url bar in the lower left hand corner shows up on all workspaces and infront of all windows.. not sure why but it has been doing this for a few months now.
as a side note, the settings menu also can sometimes display in odd places and not where it should be.
Operation System: Linux Mint 19
Beaker version:
Version: 0.8.8 Electron: 5.0.0 - Chromium: 73.0.3683.119 - Node: 12.0.0

I've found a solution to this which will be included in the upcoming 0.9 release.
Basically the status bar (the URL display) is currently a subwindow. I found a way to implement it within the existing browser window, which should solve this issue.


Top Level Domains for user E-Mails

When creating a new user on the console grav denies TLDs for emails like .hamburg
Other TLDs might be a problem as well.

I've provided a fix in the login plugin, should work for any RFC5322 email address now.


sdf_* functions not taking pipeline objects as inputs

Certain sdf_* functions do not work as expected within ML pipelines. I've tested this with a handful of the functions, but it may extend to others.
Using sparklyr 0.8.4 with R 3.4.1 on an YARN-managed cluster and R 3.5.1 on my laptop with a local Spark connection, but the error message is the same in either case.
According to the 0.8.4 documentation these functions can take ml_pipeline objects as inputs under the x argument, but instead they produce an error with the message

Error: Unable to retrieve a Spark DataFrame from object of class ml_pipeline ml_estimator ml_pipeline_stage

when included.
Reproducible examples:
spark_version <- "2.3.0"
sc <- spark_connect(master = "local", version = spark_version)

Verifying the class:
ml_pipeline(sc) %>%
  class()

This returns

[1] "ml_pipeline"       "ml_estimator"
[3] "ml_pipeline_stage"

The error occurs each of these calls:
ml_pipeline(sc) %>%
  sdf_coalesce(1)

ml_pipeline(sc) %>%
  sdf_repartition()

ml_pipeline(sc) %>%
  sdf_sample()

In each of these cases I'm expecting the output to show the printout of the pipeline stages, like with the following:
ml_pipeline(sc) %>%
  ft_string_indexer("foo", "bar")

which returns

Pipeline (Estimator) with 1 stage
<pipeline_71cc77b11a17>
Stages
|--1 StringIndexer (Estimator)
|    <string_indexer_71cc2f732618>
|     (Parameters -- Column Names)
|      input_col: foo
|      output_col: bar
|     (Parameters)
|      handle_invalid: error
|      string_order_type: frequencyDesc

Instead it shows the error message mentioned above.

Right, these aren't supposed to work since they're not pipeline stages. Will need to update documentation.

Ah ok, thanks. So the ml_* and ft_* functions are the ones that can work for either pipelines or tables and the sdf_* functions are for table inputs only (i.e. coercible to Spark Data Frame), is that correct?

Yep that's the spirit. The behavior of ml_* and ft_* functions changes according to what the first argument is, see the Value section of ?ml_logistic_regression() as an example. ml_predict/ml_transform() are supposed to work with ml model or pipeline objects as the first argument, and sdf_* all require the first argument to be a tbl_spark.


Add column "Name" in Test run table on Scenation page

The new item added to test run item "name' but it is still not appeared in Test run table on Scenario page.
The current workaround upload test run the identical "name" and "note"

Hi @yobshans, its there already. Available in "latest" tag for now.


Unknown default keras 'categorical_crossentropy' loss function when converting with .h5 to .tflite

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
-> I used just default code from the rstudio/keras library, no custom loss function
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
-> Ubuntu 16.04.5 LTS (Xenial Xerus) debian
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
-> tensorflow installed via rstudio/keras and install.packages("tensorflow")
TensorFlow version (use command below):
->1.13.1
Python version:
-> 2.7.12
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
-> Cuda compilation tools, release 9.0, V9.0.176
GPU model and memory:
-> P2-Instance AWS EC"
8*  K80-GPUs from NVIDIA, 64 vCPUs, 732 GiB Host-Speicher
Exact command to reproduce:
`import tensorflow as tf

converter = tf.lite.TFLiteConverter.from_keras_model_file("test.h5")
tflite_model = converter.convert()
open("converted_model.tflite", "wb").write(tflite_model)`
Describe the problem
I am trying to convert a keras model from .h5 to .tflite file.
I created the model in R using rstudio/keras and everything works fine.
Once I saved the model as .h5 file, I'm switching to Python (because there is no R converter from .5 to .tflite). But when I'm trying to convert the keras model with the default code from the TensorFlow Lite documentation: , I am getting the following error:
Traceback (most recent call last):   File "/tmp/RtmpWIBDmu/chunk-code-849197e0a8f.txt", line 3, in <module>     converter = tf.lite.TFLiteConverter.from_keras_model_file("test.h5")   File "/usr/local/lib/python2.7/dist-packages/tensorflow/lite/python/lite.py", line 370, in from_keras_model_file     keras_model = _keras.models.load_model(model_file)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/engine/saving.py", line 266, in load_model     sample_weight_mode=sample_weight_mode)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/checkpointable/base.py", line 442, in _method_wrapper     method(self, *args, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/engine/training.py", line 282, in compile     loss_function = training_utils.get_loss_function(loss)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/engine/training_utils.py", line 873, in get_loss_function     return losses.get(loss)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/losses.py", line 594, in get     return deserialize(identifier)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/losses.py", line 585, in deserialize     printable_module_name='loss function')   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/utils/generic_utils.py", line 212, in deserialize_keras_object     function_name) ValueError: Unknown loss function:loss_categorical_crossentropy
But as you can see in the source code, I am using the a default keras loss function?
Source code / logs
This is the Code to create and train the model
`library(keras)
batch_size <- 128
num_classes <- 10
epochs <- 5
Input image dimensions
img_rows <- 28
img_cols <- 28
fashion_mnist <- dataset_fashion_mnist()
train_images <- fashion_mnist$train$x
train_labels <- fashion_mnist$train$y
test_images <- fashion_mnist$test$x
test_labels <- fashion_mnist$test$y
train_images <- array_reshape(train_images, c(nrow(train_images), img_rows, img_cols, 1))
test_images <- array_reshape(test_images, c(nrow(test_images), img_rows, img_cols, 1))
input_shape <- c(img_rows, img_cols, 1)
Transform RGB values into [0,1] range
train_images <- train_images / 255
test_images <- test_images / 255
train_labels <- to_categorical(train_labels, num_classes)
test_labels <- to_categorical(test_labels, num_classes)
model <- keras_model_sequential() %>%
layer_conv_2d(filters = 32, kernel_size = c(3,3), activation = 'relu',input_shape = input_shape) %>%
layer_max_pooling_2d(pool_size = c(2, 2)) %>%
layer_flatten() %>%
layer_dense(units = 128, activation = 'relu') %>%
layer_dense(units = num_classes, activation = 'softmax')
model %>% compile(
loss = loss_categorical_crossentropy,
optimizer = optimizer_adadelta(),
metrics = c('accuracy')
)
model %>% fit(
train_images, train_labels,
batch_size = batch_size,
epochs = epochs,
validation_split = 0.2,
callbacks = list(callback_tensorboard(log_dir = "./logs/mnist")
))
score <- model %>% evaluate(test_images, test_labels)
cat('Test loss:', score$loss, "\n")
cat('Test accuracy:', score$acc, "\n")
save_model_hdf5(model,"test.h5")
`

@jokerstudios , could you please try with
model %>% compile(
loss = categorical_crossentropy,
optimizer = optimizer_adadelta(),
metrics = c('accuracy')
)
Hope this will help you

@joyalbin thank you for your reply.
sadly if I try your solution, I get the following error whenI'm  trying to compile the model.
Error in compile(., loss = categorical_crossentropy, optimizer = optimizer_adadelta(), : object 'categorical_crossentropy' not found

@jokerstudios I understand that it was resolved by @joyalbin .
Could you reframe title of the issue so that it will be more helpful to the community? Thanks.
Close the issue if everything was resolved. Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing the issue as it was resolved. thanks!


'Problems' pane remembers issues for deleted/renamed files

Followup from https://github.com/microsoft/vscode/issues/66982#issuecomment-1534356293
It was recommended that I report this specific problem as a separate issue. When I rename/delete a TS module, that has been flagged with problems, then the 'Problems' pane remembers the issues for the old file, although it is gone.
Restarting TS or ESLint server doesn't clear those flagged problems. I need to create the file again, then the empty content is parsed and the flagged problems get cleared.
Does this issue occur when all extensions are disabled?: DON'T KNOW (well, probably NO as the problems are owned by TS/eslint plugins...)

VS Code Version:

Version: 1.78.0
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:09:46.685Z
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.2.14-300.fc38.x86_64
Sandboxed: No

OS Version: Fedora 38, KDE Plasma, Wayland

Steps to Reproduce:

Add src/dummy.ts to project with the following contents

type BrokenSyntax {
  dummy: number
}

// NOTE: this is important: the module needs to be imported into the project
export const dummy = () => { /* ... */ }

The file src/dummy.ts (and the whole project tree where the file is located) is flagged red in the explorer and I see 2 problems in the Problems pane for src/dummy.ts

[{
	"resource": "..../src/dummy.ts",
	"owner": "eslint",
	"severity": 8,
	"message": "Parsing error: '=' expected.",
	"source": "eslint",
	"startLineNumber": 1,
	"startColumn": 18,
	"endLineNumber": 1,
	"endColumn": 18
},{
	"resource": ".../src/dummy.ts",
	"owner": "typescript",
	"code": "1005",
	"severity": 8,
	"message": "'=' expected.",
	"source": "ts",
	"startLineNumber": 1,
	"startColumn": 19,
	"endLineNumber": 1,
	"endColumn": 20
}]

import the module into your project (in my case that seems to be a requirement)

// add to some other module...
import { dummy } from './dummy'

rename or delete src/dummy.ts -> 'Problems' pane still shows the same problems for src/dummy.ts
create file src/dummy.ts and then delete it again -> the problems flagged for src/dummy.ts are gone

I have the same issue with C#, the errors still remain in the Problems pane even the underlying source file is deleted. I have to restart VS Code or use Reload Window command to clear the errors.

I'm unable to reproduce this issue:

Does it only happen when eslint is installed? What if you disable all other extensions?

I can confirm that eslint seems to remember the old file name

Can you please file this against the eslint extension to start as it seems like the bug is on their side


AT mode handles unsupported SQL type exceptions

Why you need it?
Is your feature request related to a problem? Please describe in details
执行暂不支持的SQL类型时抛出异常
Exception should be thrown when executing an unsupported SQL
How it could be?
A clear and concise description of what you want to happen. You can explain more about input of the feature, and output of it.
Other related information
Add any other context or screenshots about the feature request here.

please assign to me


Bulk operations

:tada: This issue has been resolved in version 1.27.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


port(s) to master ([last] part 7)

commits found with command to find commits committed to 2.4 but not to master
git log --no-merges master..2.4
#4030 - 1df0f1d
~180 commits need further attention and so i cannot merge them asap
Most of these commits are cuda related commits from @jet47 which are difficult to review after the refactoring. Unfortunately it seems these changes won't get into the master branch if not by @jet47 himself. (such as the tiny gpu module which will not get used by new users who uses the master branch)
Similiar to this there are some commits from @asmorkalov which seems to be some cmake and debian specific changes.
Just wanted to make this clear since i do not know how important these changes are.
done with the "easy" porting it seems...
@vpisarev please review this little change so you are informed as well about these ports.

:+1: @Dikay900 I plan to merge CPack and Debian related stuff to master too. Thanks for reminder.


Add action to label size of PR

PR Summary
Add action to label size of PR
PR Context
Add framework for requiring additional approval for large PRs
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

We should add files.wxs in the ignore list. Are there any other "generated" files we should ignore?

github-actions bot added the size/S label 1 hour ago

Cool!

The labelling task seems to have failed in new PRs...

This should only apply to PowerShell/PowerShell and not to forks!

Github action repo is https://github.com/pascalgn/size-label-action

The labelling task seems to have failed in new PRs...

It seems the Github action does not work for PRs with branches from forks (https://github.com/pascalgn/size-label-action/issues/10)


Netlify deployment fails (esbuild error)

I've been strugling all day to get deployments on Netlify to work, hopefully somebody can point me in the right direction here.
This error is thrown once contentlayer starts building.
5:48:19 PM: This shouldn't happen. Seems like esbuild watcher wasn't running (yet).
5:48:19 PM: UnknownEsbuildError {
5:48:19 PM:   error: Error: Build failed with 1 error:
5:48:19 PM:   /opt/build/repo/website/node_modules/esbuild/lib/main.js:239:12: error: Invalid option in build() call: "jsx"
5:48:19 PM:       at failureErrorWithLog (/opt/build/repo/website/node_modules/esbuild/lib/main.js:1443:15)
5:48:19 PM:       at /opt/build/repo/website/node_modules/esbuild/lib/main.js:1012:18
5:48:19 PM:       at /opt/build/repo/website/node_modules/esbuild/lib/main.js:1007:9
5:48:19 PM:       at /opt/build/repo/website/node_modules/esbuild/lib/main.js:606:9
5:48:19 PM:       at handleIncomingPacket (/opt/build/repo/website/node_modules/esbuild/lib/main.js:703:9)
5:48:19 PM:       at Socket.readFromStdout (/opt/build/repo/website/node_modules/esbuild/lib/main.js:573:7)
5:48:19 PM:       at Socket.emit (node:events:390:28)
5:48:19 PM:       at Socket.emit (node:domain:475:12)
5:48:19 PM:       at addChunk (node:internal/streams/readable:315:12)
5:48:19 PM:       at readableAddChunk (node:internal/streams/readable:289:9) {
5:48:19 PM:     errors: [ [Object] ],
5:48:19 PM:     warnings: []
5:48:19 PM:   },
5:48:19 PM:   _tag: 'UnknownEsbuildError',
5:48:19 PM:   toString: [Function (anonymous)]
5:48:19 PM: }

I've updated the netlify next plugin to the latest RC as stated in #68 this solves the syntax error, but unfortunately not the esbuild error.
Also tried running contentlayer build manually, this results in the same error on Netlify, locally this works.
Versions:

Node 16.3.1
Yarn 1.22.10
Contentlayer 0.0.33
Next.js 12.0.7

The Next.js app is build from a subdirectory.
Using the netlify-cli to build the app locally works fine and i'm able to publish the site.

Finally figured out a couple of issues.

Netlify wasn't using Yarn, this seemed to fix the above error.

Contentlayer has issues finding the correct path if you build a monorepo with Turborepo, I had to make these adjustments to make it work.
next.config.js
// Make sure this only runs on dev environments.
const isNextDev = process.argv.includes('dev')
if (isNextDev) {
config = withContentlayer()(config)
}
module.exports = config

``` package.json
{
"scripts": {
    "build": "yarn gen:contentlayer && next build",
    "gen:contentlayer": "contentlayer build --config ./website/contentlayer.config.ts --verbose"
    }
}

Notice the website folder I had to add. There must be a better way to do this though.

Probably related to https://github.com/vercel/turborepo/issues/485. I'll do some further investigation once addressed by Turborepo.

@Pagebakers nos that there's an even simpler workaround for this problem:
// package.json
  "scripts": {
     // ...
    "build": "INIT_CWD=$PWD next build",
    // ...
  },


headless: update chromedriver note

I think the example is still a bit misleading, for example here:
const chromeCapabilities = webdriver.Capabilities.chrome(); chromeCapabilities.set('chromeOptions', {   binary: PATH_TO_CANARY // Screenshots require Chrome 60. Force Canary.   'args': [     '--headless',   ] });
The comment implies that Canary is required, but this will work with the current stable release.
What do you think?

@JKereliuk good call, missed those. How about now?

Was just about to comment on the paragraph, but just saw that you changed it to something like what I was going to suggest!
LGTM, thanks for being quick to respond :)

Ditto. I didn't see the new change while writing the comment. That was fast! Thanks!


[Coverity CID: 183060] Incorrect expression in /tests/kernel/fatal/src/main.c

Static code scan issues seen in File: /tests/kernel/fatal/src/main.c
Category: Incorrect expression
Function: test_fatal
Component: Tests
CID: 183060
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

https://scan9.coverity.com/reports.htm#v34913/p12996/fileInstanceId=35038207&defectInstanceId=5538125&mergedDefectId=183060
We need to manually search with the CID specified.
And this issue is marked as false positive and the status is dismissed.


[SPARK-23603][SQL]When the length of the json is in a range,get_json_object will result in missing tail data

What changes were proposed in this pull request?
Bump jackson from 2.6.7&<IP_ADDRESS> to 2.7.7
Jackson(>=2.7.7) fixes the possibility of missing tail data when the length of the value is in a range
https://github.com/FasterXML/jackson/wiki/Jackson-Release-2.7.7
https://github.com/FasterXML/jackson-core/issues/30
How was this patch tested?
org.apache.spark.sql.catalyst.expressions.JsonExpressionsSuite
test("some big value")

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Prevent Async Validators from Executing when Synchronous Validations have failed

I have a form that contains a synchronous validator and an async validator. The asynchronous validator is executed even when the synchronous validator has failed, which occurs when the form is submitted.
This works OK and ensures the form is invalid and can't be submitted. However it has a few issues including adding extra network traffic that isn't needed yet and can cause different error messages to "flash" as the async validators override the synchronous validators.
Here's an rough pseudocode example of what I've been doing. When submitting the form, first the 'Field required' messages appears. After that, the 'Username is invalid!' message appears.
const validateFormErrors = (values) => {
  return {
    username: !values.username ? 'Field required' : null,
  };
};

const asyncValidators = {
  username: doAsyncUsernameValidation // return { error: 'Username is invalid!', success: null }
};

const LoginForm = ({ submitPassword, asyncValidators, validateFormErrors}) => (
  <Form
    onSubmit={submitPassword}
    asyncValidators={asyncValidators}
    validateError={validateFormErrors}>
    { ({ errors, successes}) => ({

        <input id="username" />

{errors.username}

        <input id="password" />

{errors.password}

        <button type="submit">Submit</button>

    }) }
  </Form>
);

It would be nice if a flag was available to force async validation to run after checking synchronous validations.

Fixed!


Fix for self-eviction hooks

After merging #299, #300 and #301, a test in ringpop-serial broke (probably bad merge); this fixes it.
Also added some extra type-checking on the hooks (isFunction).

LGTM


updates to v0.5

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@mkuehbach build going on here.

@mkuehbach looks like its building now. Once again, thanks for the PR.


Roll Flutter Engine from 98b6fabc66bb to 3bd6435914b9 (1 revision)

https://github.com/flutter/engine/compare/98b6fabc66bb...3bd6435914b9
2023-04-28<EMAIL_ADDRESS>Roll Skia from 05d09f28250a to 9867fa253064 (18 revisions) (flutter/engine#41584)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Feature/layout

Add main nav menu with links and logo
Add Sidebar interaction
Add map zoom buttons with interaction
Add Map settings button and tooltip
Allow user to change the basemap and then the theme for the buttons
Display the two different basemaps on the map

Note:
The style of the map settings popup may be tweaked
Missing:
Attribution: Design TBD
This implementation is meant to be as simple as possible to be able to gather the entry elements and further configure it.
Structure:

state.js Has the state of the app. Also exposes some mutation object with the functions to change the state.
map.js Creates the map. Adds the zoom icons as a SVG to be able to easily change the color
event-listeners.js Contains the reference to all interactive parts of the application and applies the handling of state and style changes (adding and removing classes)

All changes done. Thanks for the feedback! I'm not 100% sure about the maplibre overrides. It seems to pollute a bit more the html. But lets leave it there for now


ARTEMIS-4458: Upgrading Netty to 4.1.100.Final

This work should be done as part of https://issues.apache.org/jira/browse/ARTEMIS-4439 since it hasn't been released yet.

@ehsavoie I pushed -f to your branch


Fix debian/docker builds

Fix package builds with Docker for Debian versions 7, 8, 9 and 10. Copy-paste friendly instructions for Debian

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 All committers have signed the CLA.

Thanks for your contribution! :+1:


python3Packages.gradio: raise ulimit on darwin

Description of changes
Hopefully fixes https://hydra.nixos.org/build/246592314, lets see if ofborg agrees

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I think the python3Packages alias is still broken?
@ofborg build python311Packages.gradio python311Packages.gradio.passthru.tests python312Packages.gradio python312Packages.gradio.passthru.tests


Add support for MIDebugLog command:

Debug.MIDebugLog /On[:path] [/Output] - start logging to one or both of "path" and the Output Window
Debug.MIDebugLog /Off - stop logging

Yeah, but I think we still may lose a message:
Thread 1: gets pointer to logger
Thread 2: swaps pointers and Closes old one
Thread 1: attempts to write to closed logger, message is lost
From: Gregg Miskelly<EMAIL_ADDRESS>Sent: Thursday, November 17, 2016 8:51 AM
To: Microsoft/MIEngine<EMAIL_ADDRESS>Cc: Paul Maybee<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/MIEngine] Add support for MIDebugLog command: (#499)
@gregg-miskelly commented on this pull request.

In src/DebugEngineHost/HostConfigurationStore.cshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FMIEngine%2Fpull%2F499&data=02|01|paulmay%40microsoft.com|3662d9a7186c46ac959f08d40f09e0d5|72f988bf86f141af91ab2d7cd011db47|1|0|636149982479596732&sdata=GlVDNJ%2BXq6mFPPW07Ot0MR5JA%2B3uKY58R9bqjfWiU70%3D&reserved=0:

     public HostLogger GetLogger(string enableLoggingSettingName, string logFileName)

     {

         if (string.IsNullOrEmpty(logFileName))

         {

           throw new ArgumentNullException("logFileName");

           return new HostLogger();    // empty logger

Can't we just swap the pointers (Interlocked.Exchange) and Flush the old one if we had one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FMIEngine%2Fpull%2F499&data=02|01|paulmay%40microsoft.com|3662d9a7186c46ac959f08d40f09e0d5|72f988bf86f141af91ab2d7cd011db47|1|0|636149982479596732&sdata=GlVDNJ%2BXq6mFPPW07Ot0MR5JA%2B3uKY58R9bqjfWiU70%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL712LMqRgG3BJ5YrlENrIzKH8Qmd7kuks5q_IXkgaJpZM4K0n-3&data=02|01|paulmay%40microsoft.com|3662d9a7186c46ac959f08d40f09e0d5|72f988bf86f141af91ab2d7cd011db47|1|0|636149982479596732&sdata=mP0ODhHo30p1oNXpSJizrYPbrV8mPRAl4VrngUtHJog%3D&reserved=0.


Fix host/port handling in UrlRequest

Closes #2575
Note that this does NOT handle the authentication, just appends user:pass part to the host if applicable, therefore the host + port part is handled correctly, however, when the credentials are specified, it will most likely fail (no test url to check unfortunately) if it's not handled via HTTPConnection or socket (probably not).

http://httpbin.org/ can be used to check http protocl, it has enpoints for authentication

Thanks for the link, I could implement basic auth with that. I removed those manual checks for URL parsing as we use the defaults from urllib now (CPython has unittests for that) + added cases for auth through header and via this syntax:
<EMAIL_ADDRESS>


Use os_unfair_lock on Darwin platforms

https://github.com/apple/swift/pull/61460

PlatformMutex is now gone


Provided access to the updated json content

Allow user to have updated content when "onChange" event is raise.
Output "UpdatedContent" can not be used when user provide custom "options.onChange".

up?


Migrate debug page to hooks.

It'd be nice if you can check out the debug page in this branch locally to make sure it works ok - I tried various operations and think it's fine but there are a lot of possible variations.
Also fixes long-time pet peeve by prettifying gRPC unframed JSON.
Fixes #1949

While I was testing I found an exception. I clicked Submit button with any input and I saw a black page and exeception message.

react-dom.development.js?61bb:13124 Uncaught TypeError: Method Error.prototype.toString called on incompatible receiver undefined
    at toString (<anonymous>)
    at basicStateReducer (react-dom.development.js?61bb:13124)
    at updateReducer (react-dom.development.js?61bb:13246)
    at updateState (react-dom.development.js?61bb:13294)
    at Object.useState (react-dom.development.js?61bb:13843)
    at useState (react.development.js?72d0:1462)
    at DebugPage (DebugPage.tsx?725f:115)
    at DebugPage (react-hot-loader.development.js?c2cb:821)
    at renderWithHooks (react-dom.development.js?61bb:12938)
    at updateFunctionComponent (react-dom.development.js?61bb:14627)

@ikhoon Thanks for finding, very silly bug!

@anuraaga Your PR is always welcome. I'm happy to find and share it 😉

I think 404 Bad Request message doesn't print on debug console.

This is master version screen shot.😀

Thanks - fixed

I think both were introduced in previous PRs (by me ><). Fixed the error, and also removed a spacing parameter that was causing the hover issue. The method type / name are a bit closer now, but I guess it still looks ok to me, though I might be forgetting the reason the parameter was there in the first place.


Real World Example

Please give a real world usage scenario

That's not exactly an issue, but I wrote it partly as an example to one of my Ph.D. students of how you would do strong typing in an embedded logic language, and party for indie game developers who use logic programming but wanted strong typing and better IDE integration than you get with a self-contained interpreter.


[DRAFT] Update Covid hosp to use new healthdata.gov API

Fixes #446.
Changes covid_hosp to use the new API. Main changes revolve around updating the metadata/revision acquisition, which is now it's own dataset with a CSV download URL. This means that we can fetch the dataset with a simple pd.read_csv and throw away the web scraping code.
This also updates the dataset IDs to use their new ones.
@krivard, I saw NCHS uses the same Socrata API and takes a token, so we may want to do that here as well (either with the same or a new token). It works without one, but they recommend it.
In terms of the switchover once this is merged, I could see one anticipated error case where the following line doesn't evaluate to True when it should, since the new revisions look differently. This would lead to two copies of the revision being ingested. We may be able to workaround by adding a date check on top of the revision check so if the latest issue = the max issue in the db and the date < the day we merge, we don't ingest (this fails if it was actually a new revision on that exact date, though that seems less likely).
      # bail if the dataset has already been acquired
      if db.contains_revision(revision):
        print('already have this revision, nothing to do')
        return False

Leaving in draft mode while we work this out.

Alright made a couple bigger changes in this PR. Now all of facility, state daily, and state timeseries share the same logic for acquiring data which is to

download metadata csv
grab the highest issue already acquired (based on the covid_hosp_metadata table)
select all issues for days after that issue and before the current day. issues after the highest issue but on the same day as that issue are not acquired. This choice I think is open to debate, though handling new issues on past ingested dates may require some effort.
For each day, roll all its issues up into a single dataset insertion and upload that. Upload the last issue that date into the metadata.

What's remaining for this to come out of draft?
iirc the rollout plan is a two-day sequence:

reschedule Automation tasks to skip a day
merge
monitor the first active day of the new code to make sure we didn't duplicate anything

if we're ready, I can set us up to skip Monday's run and monitor Tuesday's.

What's remaining for this to come out of draft?
iirc the rollout plan is a two-day sequence:

reschedule Automation tasks to skip a day
merge
monitor the first active day of the new code to make sure we didn't duplicate anything

if we're ready, I can set us up to skip Monday's run and monitor Tuesday's.

API token? otherwise should be good to go

Oh right. Let's go without for now, and I'll make an issue for adding it once the infra inferno of bigint+flask+docker has calmed down. delphi-epidata doesn't have as nice a mechanism for storing secrets as covidcast-indicators does.


Document if memo components should re-render on context change

Issue format taken directly from https://github.com/facebook/react/issues/15351

What is the current behavior?
If useContext is used in memo I am unsure if it should re-render when the context changes as I can't see any documentation for this.
From https://reactjs.org/docs/react-api.html#reactmemo

If your function component renders the same result given the same props, you can wrap it in a call to React.memo for a performance boost in some cases by memoizing the result.

I guess this slighty mentions it: https://reactjs.org/docs/context.html#contextprovider

All consumers that are descendants of a Provider will re-render whenever the Provider’s value prop changes.
The propagation from Provider to its descendant consumers is not subject to the shouldComponentUpdate method, so the consumer is updated even when an ancestor component bails out of the update.

If useContext is considered a consumer, then I guess that documentation covers it, but it would be nice to know for sure if it will trigger memo to not work.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than React. Paste the link to your JSFiddle (https://jsfiddle.net/Luktwrdm/) or CodeSandbox (https://codesandbox.io/s/new) example below:
https://codesandbox.io/s/qx3m4rl7wq
What is the expected behavior?
The documentation should explain whether components utilizing memo should re-render on context change
Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?
"react": "^16.8.6",
"react-dom": "^16.8.6"

This would be great to document.
I am in favour of the current behaviour: The memoized version stays in-sync with the child that may change out from under it.
However, I do not want to rely on this undocumented behaviour.


Implement TryFrom<u64> for the id types

I'd like to have the TryFrom trait implemented on the id types.
This could make converting from an u64 to an id type more ergonomic - something like
client.guild(GuildId::new(u64_id).ok_or(MyError::IdIsZero)?)
or client.guild(NonZeroU64::try_from(u64_id)?.into())
could become client.guild(u64_id.try_into()?).
That conversion needs to happen a lot of the times where an id is user inputted or loaded from a database, which is not rare.
This seems like a very simple feature to implement, so I'd be willing to contribute to it.
Additional info: this feature was briefly discussed in the Discord before I opened this issue.

This will be in the next major release.


Make possible to scale multiple nodes at once in the canvas editor

This PR makes possible to change the scale of multiple selected canvas items at once:
Screencast_20241025_163951.webm
And adds some smaller improvements:

Make the "scale" gizmo be positioned at the temporary pivot when used.
Show the "move" gizmo when multiple items are selected (no idea why this wasn't done sooner).

Sponsored By: 🐺 Lone Wolf Technology / 🍀 W4 Games.

When in Select mode, pressing Ctrl+Alt will show scale gizmo, but pressing Alt does not show move gizmo.
https://github.com/user-attachments/assets/aaaa88b0-0721-4d4f-ae94-7fbf77687ab0

For reference in 3D mode, the local mode and non local mode affects scaling of multiple objects in standard but weird ways.

@KoBeWi

When in Select mode, pressing Ctrl+Alt will show scale gizmo, but pressing Alt does not show move gizmo.

This seems to be unrelated to my PR, as pressing Alt on stable/master doesn't show it either. I can try to fix this too if that's what you meant with your message.

I mean, you added move gizmo when move tool is active. Also the scale gizmo showing is also coming from this PR. So it's related.

Another thing, which is partially pre-existing bug, is that pressing Ctrl+Alt in move mode will display scale gizmo and with temp pivot you can see 2 gizmos at once.

@KoBeWi Only the "Scale" shortcut actually works outside of the "Selection" mode. I wonder if this was an oversight and the behavior should be removed.

@KoBeWi @AThousandShips Changes made.

Thanks!


bcmath no longer loads

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 1.8.7
VirtualBox 5.1.14r112924
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides

Your OS

macOS (10.12.3)

Full console output

Summary

Despite adding this to config.yml (and despite this working less than a month ago), bcmath doesn't load and hence implementing drupal commerce 2 is impossible:
extra_packages:

php-bcmath

Try with php_packages_extra instead of extra_packages.
php_packages_extra:

"php{{ php_version }}-bcmath"

What @oxyc should work in the latest Drupal VM versions, and is the preferred way to add extra PHP-related packages to the system.


[hive][csv upload] make INTs BIGINTs

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
When uploading a csv to Hive, all integer columns are created as INT, which causes integers greater than<PHONE_NUMBER> to appear null. This is not great if there are ids greater than a few billion.
We could probably be more specific and identify whether to use TINYINT, INT, BIGINT, DOUBLE, STRING, etc so that we don't top out at $2^63 - 1$, but that's for another time.
TEST PLAN
Test on devbox.
REVIEWERS
@john-bodley @etr2460

Codecov Report

Merging #9716 into master will increase coverage by 6.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9716      +/-   ##
==========================================
+ Coverage   59.16%   65.80%   +6.63%
==========================================
  Files         367      584     +217
  Lines       11679    30382   +18703
  Branches     2862     3112     +250
==========================================
+ Hits         6910    19992   +13082
- Misses       4590    10206    +5616
- Partials      179      184       +5

Flag
Coverage Δ

#javascript
58.90% <ø> (?)

#python
70.66% <ø> (?)

Impacted Files
Coverage Δ

superset/db_engine_specs/hive.py
46.12% <ø> (ø)

.../src/SqlLab/components/EstimateQueryCostButton.jsx

superset/assets/src/visualizations/constants.js

superset/assets/src/components/Menu/Menu.jsx

...et/assets/src/explore/reducers/saveModalReducer.js

.../assets/src/SqlLab/components/TabbedSqlEditors.jsx

superset/assets/src/utils/errorMessages.js

.../assets/src/dashboard/util/componentIsResizable.js

...perset/assets/src/SqlLab/components/QueryTable.jsx

superset/assets/src/CRUD/Fieldset.jsx

... and 942 more

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 e46ff23...5269edc. Read the comment docs.