Upgrade flay, reek, rubocop, and mutant

@mbj it looks like the final commit in this PR fails simply because rubocop's defaults have changed.
  UNIT_TEST_PATH_REGEXP   = %r{\bspec/unit/}.freeze

Unless the regexp instance is frozen you could still patch the eigenclass...

@mbj it looks like the final commit in this PR fails simply because rubocop's defaults have changed. I'll let you make the judgement call here on what you prefer

@backus All of the flags are invalid, you can disable them.

lib/devtools/project.rb:62:19: C: Operator = should be surrounded by a single space.
      @spec_root  = root.join(SPEC_DIRECTORY_NAME)

Is part of an aligned assignment.

shared/spec/shared/idempotent_method_behavior.rb:4:5: C: Always use raise to signal exceptions.
    fail 'RSpec not configured for threadsafety' unless RSpec.configuration.threadsafe?
    ^^^^

I use a different convention in my projects.

:thumbsup:

@mbj I think that the Style/SpaceAroundOperators flag is valid. It intentionally does not flag extra spacing when used to align with other operators. Note that it doesn't flag anything in lib/devtools.rb. The flag here
lib/devtools/project.rb:62:19: C: Operator = should be surrounded by a single space.
      @spec_root  = root.join(SPEC_DIRECTORY_NAME)

is valid I think because there are no other operators above or below that are being aligned with.

is valid I think because there are no other operators above or below that are being aligned with.

Ahh, I assumed there is alignment with other assignments, as rubocop bitched about that, nice they fixed this.


JQuery dependency

Hello,
This is not an error, but I couldn’t find how to contact you (the contact feature on your website doesn't work).
It is possible to remove the jQuery dependency ?

Will it be difficult, time costing ?
How much money would it costs ? (I was thinking of becomming a patreon btw, or give you directly)

Thank you so much for the work, it is so useful for me and my project ! Very much appreciated !

Hello,
Yes. We can do that. Could you contact me at<EMAIL_ADDRESS>We are going to fix the contact form soon.
Regards,
Atanas


zoom: Add version 5.0.1

Closes #3920

Zoom is a video and audio conferencing, chat, and webinars service.
Note for checkver:

The Zoom client has a "Check updates", but it's a post request.
The Zoom download page has other software, only using Version ([\\d.]+) is not strict.
The Zoom release notes page may be better.

@Ash258 Please pay attention. The download link is not the official zoom website.
The official website is  https://zoom.us/download and the download link is https://zoom.us/client/latest/ZoomInstaller.exe

@Ash258 Please close this PR, as it contains a security risk.

...
I will fix it on update before merging in 6 days.


Broken license URL in distributed JS files

If you download the library from http://imakewebthings.com/waypoints/, the lib/*.js files includes a URL in the top comment like so:
https://github.com/imakewebthings/waypoints/blog/master/licenses.txt
The URL should say blob/ instead of blog/. This seems to be fine in the repo source--just not in the download.

Fixed in the 4.0.1 release.


data.last(n) should gracefully fail is n > total data size

See #748 for use case!

After looking through #748, it seems the goal is to gracefully fail when n is greater than the number of records, whether that's total trials or filtered trials. Is this correct?
It might also be worth looking at the Lodash's _.takeRight(), which slices an array from the righthand index. Generally, it might be worth looking at more comprehensive packages that handle object structure manipulations.

After looking through #748, it seems the goal is to gracefully fail when n is greater than the number of records, whether that's total trials or filtered trials. Is this correct?

Right. This particular one should be pretty straightforward to handle, but I hear you on taking inspiration from lodash and other toolkit libraries.


[WIP] Add switchboard routing for ARFairSearchViewController

As part of LD-28, we're adding a "Find Exhibitors and Artists" link button to the new Fair view that links to a fair-scoped search. This mirrors functionality that exists in Eigen today as part of ARFairSearchViewController. This PR aims to expose this screen via SwitchBoard, and I have a couple questions after implementing this:

The existing Fair routing in Eigen is complicated due to the top-level Fair URL: ARMutableLinkViewController -> ARProfileViewController -> ARFairViewController. Currently we pass an ID through this chain, eventually testing that the URL corresponds to a Fair. Should this logic be extended so that both an ID and optional subpath are passed through, and once the URL test occurs using the subpath to determine if we should display ARFairSearchViewController?
Should SwitchBoard routes correspond 1:1 to web routes? We have this search view available on the web, but only routable with a query, e.g. https://staging.artsy.net/artxlagos-2018/search?q=Henri. We'd have to tweak the behavior of this route in force to render an empty search page when https://staging.artsy.net/artxlagos-2018/search instead of redirecting if SwitchBoard routes should be 1:1 I think. If SwitchBoard routes don't have to be 1:1, could we route to ARFairSearchViewController on a fair-scoped path to make the routing easier, like /fair/:fairID/search?
Should this new route be in Echo or behind a Local Discovery feature flag?

          1 Warning

:warning:
PR is classed as Work in Progress

  Generated by :no_entry_sign: Danger

Switchboard routes don't have to correspond 1:1 with web routes. For top level entity pages it makes sense (Artist, Artwork, Gene, etc...) but in the case of sub routes like /:fairId/search it's not needed.

@l2succes Switchboard routes should reflect existing Force routes or, if they don’t exist yet on Force, they should be in a format that could make sense for Force to use in the future. When people pass around URLs they should work in any setting.

Any update on this PR? Just wanted to make sure it wasn't lost in the shuffle 😄

Thanks for the bump @ashfurrow, I'm going to merge this.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-29)
Features

Update openapi.json (#1) (e15f828)

Bug Fixes

set correct spdx identifier (#4) (32b7bc4)
set namespace with correct casing (#5) (a21ab9b)
set package meta (#3) (06ebedf)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


fix score text memory leak

that's about it

what's the memory leak here?

what's the memory leak here?

text was constatly being generated

i don't really understand what you mean by that, like adding onto the text in the if statement causes a memory leak even though the text is being reset before it?

what's the memory leak here?

text was constatly being generated

i don't really understand what you mean by that, like adding onto the text in the if statement causes a memory leak even though the text is being reset before it?

putting "+=" on update(elapsed) makes the text constatly generate, so if the rating name was not "?", it would generate the Rating Percent text every frame, this fixes it by making the text always be there by default but hidden if the rating name is "?", rather than being generated afterwards

though, wouldn't that operand be the exact same as the logic gate? you're still adding a string onto an existing string, so wouldn't that be equivalent to +=?, just shorter?

luckily not, since the text is hidden behind the variable and not being added as you press a note, it's like putting an object as "visible = false" and making it visible afterwards, since the object was not visible before, it was not affecting the game memory, by doing that the memory shouldn't increase drastically if that same object became visible by, say, if (curStep == blah)

plus adding objects every frame is not a good thing either, I will probably make a pull request in the future so score bar updates won't depend on super.update(elapsed), rather, the text won't update unless goodNoteHit, noteMiss, or noteMissPress was triggered, basically making update useless for updating such text

closing because I'd rather make it the good way


channeldb+multi: export {Write,Read}Outpoint, remove duplicates

Removes duplicate writeOutpoint and readOutpoint function in utxonursery.go.

Does this work? Seems like the method in channeldb is different from the one from nursery.

So I think if we want them to conform we need a db migration?

Not sure if this change is worth it, given that we might want to phase out the old encoding (and as you say that would require a migration).

This is needed for isolating the fundingmanager in its own package since it uses the old extra byte encoding. So migration is better and I'll go with that approach.


[PyTorch] [ROCm 4.0.1] torch.pow produces incorrect results for the int8 dtype on Nvidia Tesla T4 GPU

Hello,
PyTorch with ROCm 4.0.1 produces incorrect results for torch.pow for the int8 dtype on an Nvidia Tesla T4 GPU, but correct results with a GPU of Nvidia Volta microarchitecture (I had tested with a local build, but I don't currently have access to a machine with this GPU).
This issue does not manifest with CUDA.
In PyTorch PR 50999, test_int_and_float_pow_cuda compares the results of torch.pow with numpy.power. For a 3-D tensor of shape (5, 5, 5) with dtype torch.int8, with elements in the range -2 to 2, the comparison fails with ROCm, when the exponent is (a scalar) 5.
CircleCI uses Nvidia Tesla T4 GPUs, so this issue might only be affecting some particular microarchitectures.
Initially, I assumed the bug was related to overflow, so I changed the range of the elements in the tensor, but the comparison with numpy.power fails even if there's no overflow (elements in range -2 to 2, and the exponent is 5).
The corresponding Jenkins failure logs are here.
Please help fix this issue.
Thank you!
cc: @JeffDaily

Thanks @imaginary-person for reaching out.
Can you please share the exact steps(step by step procedure) to reproduce the problem, so that we will understand better and works on the issue.
Thank you.

Hello @ROCmSupport, can you please test this function with your local ROCm PyTorch build with an Nvidia Tesla microarchitecture GPU? Please replace test/test_binary_ufuncs.py on your system with the one in the given link.
You can run this test with the command python test_binary_ufuncs.py -v TestBinaryUfuncsCUDA.test_int_and_float_pow_cuda.
torch.pow(tensor_name, 5) on a tensor of shape (5, 5, 5) of int8 dtype would produce incorrect results for some elements.
For example, -31 instead of -32, so this test would fail. Please investigate as to why it produces an incorrect result.
Please let me know if you've any further questions. Thank you!

Hi @imaginary-person
I do not have NVidia GPU with me and ROCm is mainly for AMD GPUs.
Thank you

Hello @jeffdaily, is it possible for you to have this issue tested on an Nvidia Tesla microarchitecture GPU, since CircleCI only has Nvidia Tesla T4 GPUs. @ROCmSupport doesn't have access to one. Thank you!

Does torch.pow produce incorrect results for ROCm hardware, or just the Tesla T4 GPU?

Does torch.pow produce incorrect results for ROCm hardware, or just the Tesla T4 GPU?

@jeffdaily, can you please confirm if you know whether the PyTorch CircleCI tests for ROCm run on ATI hardware?
Based on what I can see on CircleCI's website, it only seems to have Nvidia Tesla T4 GPUs.
So I assumed that that the test only fails on Tesla T4 GPUs.
I don't have access to ROCm hardware, so I don't know if ROCm hardware produces correct results.

PyTorch CI for ROCm runs on MI25 or MI50.  It uses Jenkins CI, not CircleCI.  The test failure is from our Jenkins CI logs; so the failure you're seeing is for AMD GPUs.

Thanks for the info! That makes sense. I had been wondering why ROCm builds weren't tested on ROCm hardware,
as I had wrongly assumed that PyTorch Jenkins was also hosted by CircleCI.
So, I guess, debugging would be a breeze for @ROCmSupport now.

I do not have NVidia GPU with me and ROCm is based on AMD GPUs.
Your point might be irrelevant here then

@ROCmSupport, please use an AMD GPU to debug.
BTW, ROCm is supposed to be an open standard & does support Nvidia GPUs, but that's irrelevant now, as I was mistaken about what GPU the test failed on.

@imaginary-person on ROCm 4.1 , I re-enabled the test and the output is pasted below.  There is a runtime warning.
Pytorch CI is moved to ROCm4.1 for AMD GPU's.  If the results are good, then you may want to submit PR with the tests enabled.
/home/pytorch-1/test# python test_binary_ufuncs.py -v TestBinaryUfuncsCUDA.test_int_and_float_pow_cuda
test_int_and_float_pow_cuda (main.TestBinaryUfuncsCUDA) ... test_binary_ufuncs.py:621: RuntimeWarning: invalid value encountered in power
np_res = np.power(to_np(base), to_np(np_exponent))
ok

Ran 1 test in 2.908s
OK

@jaglinux, thank you for the update!
IIRC, the warning is intended, so from your update, it seems like the issue has been fixed on ROCm 4.1.

Thanks @imaginary-person for the update.
Hi @imaginary-person, feel free to close this ticket now.
Thank you.


Message template - iD autofixes

This advice:

A good start is always to check first if there is no error in the area you want to map, so you know if the error is made by you or a previous mapper (documentation). You can enable this feature with I key.

may be harmful - we have had a few mappers who indiscriminately applied fixes suggested by iD, which are not always right. There have been user blocks due to this feature, I think.

Even though I do agree that in some situations, the iD autofix might bring errors, the iD editor is the "official" editor and I think we should trust it.
But if you don't have that confidence in iD for Poland, feel free to not include it in the templates for Poland. Each region can have its own templates with its own recommandations.

I hope that going forward, with our new maintainer responding to OSMF, you'll be right :)
We have had many raised issues being ignored in the past (like addr:place support, POI presets that removed building=yes etc.

iD autofixes are not something I would recommend to a new mapper myself. Unless perhaps like: "hey this is a thing, but read up on the 'why' of the fixes before going crazy. The web editor is proposing an educated openion, not an absolute fact"
For sure, its up to whoever writes a local template, but I wouldn't include this in a default template unless with more context.

For sure, its up to whoever writes a local template, but I wouldn't include this in a default template unless with more context.

There is no default template. Each region has full control on its template.

iD autofixes are not something I would recommend to a new mapper myself.

Do you think we should remove it from our templates for Belgium then ?

Based on my review of new mapper edits, I've given the iD a list of common problems I see. For example buildings not being square. I've asked the developers to provide a warning when they add non squared objects. The are a couple of open issues on this one example. Validator warning are useful but I agree that autofixes, especially with features that the new mapper didn't touch should be ignored. I would not bring up autofixed but instead point them to appropriate learning material, such as learnosm.org.

@jbelien : I'll do a proposal for a re-write of our welcome message


not really an issue, but a suggestion, not urgent

Expected Behavior
Feedback to user presents bytes, which are very difficult to interpret.
Current Behavior
In my machine (32GB of RAM), I saw some long number starting with a "1", and the RAM also in bytes, stating with a "3". I would not know if I was using 33% of my RAM (1/3), or 3.3% (1/33). The lengths of the bytes are difficult to asses by mere sight.
So, in my personal copy of MMseqs2, I changed the feedback in two files:
./src/prefiltering/Prefiltering.cpp
./src/util/kmermatcher.cpp
I divided the bytes by pow(2,30) to get GB. That's so much better! (I made the changes only in printed feedback. Everything else I left untouched. Calculations are still in bytes, only feedback is given in GB.)
I don't know if there's more files that give feedback to users in bytes, but that was a good start. Now I see my databases being created, and I see that I have more than enough RAM (it was 3.3%).

Edited files in case you'd want them
edited.sources.zip


lib/snet: expose metrics

Packets received/sent
Bytes received/sent
DialSCION/ListenSCION/Conn.Close() counters.
Packet parsing errors
SCMP errors
Dispatcher errors/reconnects

Ideally this would be done such that an app doesn't have to initialize the metrics if it doesn't care about any of that.

Renamed similarly to other existing issues.


Fix constant assertions

Fixes #351
Closes #404 (this is a superset)
I still need to check that this doesn't have a runtime cost.

I tested this out on a thumbv7em-none-eabihf target, with and without the new assertions, the resulting binary size was the same.  No runtime cost that I can see.

...I misread #351, I thought something was asserting at runtime. This doesn't improve the compatibility with cargo check, going to close for now.


[Feature Request]  Do you have a password type entry?

Existing Component
No
Component Name
el-input
Description
there would be a component equal to <input type = "password" />

https://jsfiddle.net/leopoldthecuber/rodn81tq/1/

Thank you very much.
Sorry, in the documentation of "Input Attributes" it says that the type is
only "text / textarea"
http://element.eleme.io/#/en-US/component/input
2018-02-16 9:38 GMT-03:00 杨奕<EMAIL_ADDRESS>
https://jsfiddle.net/leopoldthecuber/rodn81tq/1/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ElemeFE/element/issues/9827#issuecomment-366224862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbZ9CX0tVWmmDNra-tY-19DGNQWeDyYDks5tVXbRgaJpZM4SIRSa
.


Search viewlet right padding is incorrect

Seems closer to the right hand side than to the left:

All viewlets except Search behave this way. There is always more margin on the left hand side. 12px more to be precise, due to the viewlet title:

Take the git viewlet:

Notice how the fonts of the viewlet title and the search box are left-aligned. Notice how the search box is right-aligned with the last action's bounding box.
The Search viewlet is full of lies and falsely aligns to the right of the clear icon's content. It should align to the right of the whole clear icon, including its padding.

@joaomoreno this issue was about the fact that the git input field has more margin to the left than to the right. Would we not just center the input fields we have across viewlets?
Only looking at:

You can't just look at one thing at a time. There are left and right paddings along the whole viewlet's content. If we were to increase the right padding, we'd be lying about the alignment with the actions above and we'd have to move them as well, in every viewlet.

Well today the input box is neither aligned in the center nor is it aligned to anything in the title area :)

Of course it is.

You might think that, for instance, the Explorer title area is aligned to the center:

But it's not, since the title has additional left padding:

Which forces its content to have it too:

The explorer too, has more left padding than right. This has been constant throughout our whole design.

That overflow icon does not really need all the margin/padding to the right I guess.

I personally think it looks good as it is. Center alignment is not important. Left and right alignment of all items is, though. But the only way to fix this and have center alignment would be to add margin to the right to everything, including the title actions.
The icon needs the padding because it's a click area. Make it smaller and it will be less clickable than all the other actions.


Support "in-room" template when behind CloudFlare

Previously we used X-Forwarded-For but CloudFlare doesn't seem to use that. Also unsure how it interacts with nginx.
Found the following article;
https://support.cloudflare.com/hc/en-us/articles/200170986-How-does-CloudFlare-handle-HTTP-Request-headers-

This would blow away any caching benefit we get out of CloudFlare, as all requests would need to hit out webserver to handle the ACLs.  It may be better to have a placeholder site which switches between the two, and configure the 'streaming' site as a different site in CloudFlare.
This would allow it's existing ACLs to implement it for us, without limiting access to code.timvideos.us.
I'm not how this would extend to a static files implementation of the streaming platform.
The other other alternative is to set this up so there is a static site, and the dynamic bits talk to a backend to get the list of available streams.  We apply the ACLs at this level, such that the majority of content can still be cached, and only a little bit of data needs to be passed dynamically.


URL with multiple parameters as a query string

Hello there.
If anyone have seen before this type of error....
Lets take the 'Demo1' for example, if i try to make the assertionConsumerService or       singleLogoutService a link to an service where i will be needing at least two query strings params it returns that the metadata is wrong 'Invalid SP metadata: unloaded_xml'
Ex:
'singleLogoutService' => array (
'url' => $spBaseUrl.'/demo1/index.php?sls=test&param2=test',
For now i have tested on SP settings and it gives this error when i try to access the metadata file.
Thanks.
Dan.

Don't add parameters in the settings.
If you want to add extra parameters to the logout request petition notice that the logout method contains a parameters var where you may defined them.
So keep:
singleLogoutService' => array (
    'url' => $spBaseUrl.'/demo1/index.php?sls
),

and use
$parameters = array (
   'param2' =>test'
)
$auth->logout(null, $parameters);


You can throw eyes of ender

This is despite throwing eyes of ender being disabled in the config - unsure of what is causing the issue, as on my local test server throwing eyes of ender is being stopped as intended.

Fixed with https://github.com/CivClassic/AnsibleSetup/commit/14f6e8ecce724f477da51049119946d59f1ba26d


Events

@adamstruck I pulled your events code into this PR
After using it for the os/exec worker branch, I made some changes:

dropped the oneof
dropped the proto Struct type for log fields
moved proto file to events package
added some additional writer/logger/gen helpers
remove the "executor" prefix from stdout/err, hostip, etc. because it seemed overly verbose. start/end time still have the prefix.

I found the oneof to create a bunch of stuttering fields which seemed unnecessary. e.g.
if e, ok := x.Event.(*events.Event_Stdout); ok {
	if e.Stdout.Stdout != "a" {
		t.Error("expected 'a' but got %s", e.Stdout.Stdout)
	}
}

vs
if x.Type == events.Type_STDOUT && x.Stdout != "a" {
  t.Error("expected 'a' but got '%s'", x.Stdout)
}

I also found the Struct type difficult to work with. For logging, are we ok with a simple map[string]string? That covers most info I can think of. Is there a way to marshal/unmarshal between an arbitrary Go struct to a protobuf Struct? Having to write a type switch for all the Value types seems overkill, https://godoc.org/github.com/golang/protobuf/ptypes/struct#Value. Maybe I just need an example of the right way to do it.

The latest commit removes the tailer. This was a custom buffer type which started to get in the way because it didn't implement io.Reader/Writer interfaces. It also introduces strange behavior, where there may be a gap in the logs written to the event stream during network error conditions.
TODO:

[ ] add back log event buffer and rate limiter

I still prefer using oneof for the events. I'm fairly certain you can avoid the situations you describe and I think there are other benefits to the original approach.


all checked but just a few are in the source option

Hey,
Great job. Have a issue with it http://jsfiddle.net/newzen/nftdvyk4/3/ all checked but just a few are in the source option.
any help will be appreciated, thanks

What source option?

One thing I'm noticing though, is the lack of value attrs on the options. If you don't have those, I think the option text will be sent instead and those aren't unique in this case.

corrected, all work now. http://jsfiddle.net/newzen/nftdvyk4/6/
Maybe you can document value attrs is required.
thanks for your help

How i can set my custom selected options after loading the tree?

How i can get all selected nodes by code?
Thanks


feat: Add dataKey and categories inference

Addresses #238  by allowing only string values into dataKey prop and only number values in categories prop
A few considerations:

categories prop allows duplicate entries (as it is right now)
if the values in the data array are of different types, then only the intersection can be passed in dataKey or categories
it can be also done with just keyof T & string, but it would require more casting inside of the component
Same can be achieved if T would be hardtyped (as it is in BarList for example) with addition of some labelFormatter

дмытро довбоб

Hi @dmytro-tinybird, sorry for getting to this PR so late. There have been many changes and two major releases in the meantime and we are refactoring our charts and upgrading to storybook 7.0, hence it does not make sense to merge this. Thank you very much for taking your time.


effective_sample_size() to POD types for RStan/PyStan usage

Summary:
The current interface is
double effective_sample_size(const Eigen::Matrix<Eigen::VectorXd,  Dynamic, 1>& samples) const;

which is suboptimal for interfaces because of its use of Eigen.  I don't think it's good for Stan itself because of its unconventional use of Eigen matrices.
How about something more standard like
double effective_sample_size(const std::vector<std::vector<double>>& samples) const;

Lower level, we'd get even better memory locality with
double effective_sample_size(const double* samples, int num_chains, int num_draws) const;

For the interfaces, is it better if we actually take in a 3D input and do this for each parameter?
double effective_sample_size(const double* samples, int num_params,
            int num_chains, int num_draws) const;

Description:
This'll make it easier for RStan and PyStan to use the effective sample size calculations based on Stan.
To maintain backward compatibility, we just need a new function
@bgoodri and @ariddell --- would this be enough to let you use it in RStan?
Current Version:
v2.17.1

@bob-carpenter, just submit a pull request for this.  It might not be 100% what you're looking for, but it's a good start as it has the right data structures and accurate calculations.  I'll let you continue discussion here or in the pull request.  Thanks.

It may be a week or two before I can knock this off, so if someone else wants to do it, it shouldn't be difficult.  We usually try to open-ended discussions in Discourse, big functional specs on Wikis, and technical specs as part of the issue refinement and PR reviews.

With https://github.com/stan-dev/stan/pull/2575 now merged, should we close this?


[format] Unify name from target to destination

Describe the enhancement requested
There are naming conflict with single concept in proto:
AdjListType call it target

meanwhile EdgeInfo call it destination

We can unify them to destination.
Component(s)
Format

@Thespica May we use dst and src instead of destination and source?

I think it will be better to use full name, cause source and destination are more like specific concepts. We should unify usages of them.


Add log forwarding topic

Purpose of this pull request
This pull request (PR) adds the Forward logs to New Relic topic
Affected pages

...

Links to Magento Open Source code

...

Closing this PR. The contents were moved to https://github.com/AdobeDocs/app-builder/pull/303


Disallowing extension of Throwable

A small check warning users when extending throwable. Should close #167.

Hi @samuelsoder,
Thanks for this PR, and sorry for the very slow response!
We can't accept this contribution without a Contributor License Agreement (CLA), see: https://cla.developers.google.com/


Build Deploy Window: detect Windows 10 SDK installation path by checking registry fix #9146

Overview
BuildDeployWindow would produce an error when Windows 10 SDK was installed on any disk other than C:.
This was due to a hard-coded path in the script.
This PR remedies that by checking the registry first to obtain the actual installation path.
 If you're able to pull this down and double check that it doesn't break anything on non-Windows machines, that'd be super useful!

@elbuhofantasma @CDiaz-MS I think you were the two with Macs. If you're able to pull this down and double check that it doesn't break anything on non-Windows machines, that'd be super useful!

/azp run

/azp run


Fix js org domain

@akki-jat thanks for your work on this last week. I'm going to give this approach another go and test out some results

@mattelen I think the custom domain needs to be set from repo settings. Please check with @shentao.

Ah I see - sweet I'll reach out to him


0.87 release planned 🎃

Quite a bit in the milestone now and seems like a good time to do another release. particularly since we got a fix in the controller scheduler (and another timeout pass-down "fix")
particular stuff worth highlighting imo:

socks5
auth plugin work (although still some WIP there in #1331)
Controller::reconcile_on change (breaking, but it's for an unstable interface)
dep upgrades, syn upgrade caused a break on #[kube(struct)]

will likely hit it up later this evening.

ugh, failed mid release:
error[E0425]: cannot find value `connector` in this scope
   --> src/client/builder.rs:123:51
    |
123 |         let mut connector = TimeoutConnector::new(connector);
    |                                                   ^^^^^^^^^ not found in this scope

kube-core was the only one that made it through. yanked it;
cargo yank --version 0.87.0 kube-core
    Updating crates.io index
        Yank<EMAIL_ADDRESS>

Ok, we somehow uncovered a fault in CI testing here;  cargo test -p kube-client --no-default-features fails. Annoying because we do actually try to do a no-default-feature build of kube (*), just not all the underlying crates...
The problem in this case is the socks5 PR removing these 2 lines: https://github.com/kube-rs/kube/pull/1311/files#diff-1ab2824dd0a6633b63d18104f91fc5e7dfb07bb14a80c956af7b8af33162813bL78-L79 which are not reinstated in the new make_generic_builder for all cases. I.e. we can't create the service anymore in the case of no tls stack. i.e.
Not decided what to do yet for the release. Either we temporarily backout the PR or fix forwards. cc @Razz4780 . In either case, not doing the release tonight.

Ok, we somehow uncovered a fault in CI testing here; cargo test -p kube-client --no-default-features fails. Annoying because we do actually try to do a no-default-feature build of kube (*), just not all the underlying crates...
The problem in this case is the socks5 PR removing these 2 lines: https://github.com/kube-rs/kube/pull/1311/files#diff-1ab2824dd0a6633b63d18104f91fc5e7dfb07bb14a80c956af7b8af33162813bL78-L79 which are not reinstated in the new make_generic_builder for all cases. I.e. we can't create the service anymore in the case of no tls stack. i.e.
Not decided what to do yet for the release. Either we temporarily backout the PR or fix forwards. cc @Razz4780 . In either case, not doing the release tonight.

Wait, what exactly is the problem? cargo test -p kube-client --no-default-features works on main. Those two lines are still there in TryFrom<Config> implementation for GenericService. make_generic_builder accepts the base connector as an argument.

uh, i think the problem is that the make_generic_builder expects to use the base_connector in one of the feature-flagged lines below in https://github.com/kube-rs/kube/blob/main/kube-client/src/client/builder.rs#L113-L116, but if neither are true, then you don't actually have the connector variable in scope. the cargo test command i wrote does indeed work though :confused:

this fails at root in main:
cargo test -p kube --no-default-features --features=client

Ah, that's right. https://github.com/kube-rs/kube/pull/1335 fixes the issue

Seems we have both arrived at the same fix: https://github.com/kube-rs/kube/pull/1334
Appreciate it!

Btw. one line in the related code seems fishy - https://github.com/kube-rs/kube/pull/1335/files#diff-1ab2824dd0a6633b63d18104f91fc5e7dfb07bb14a80c956af7b8af33162813bL118
if auth_layer.is_none() || config.cluster_url.scheme() == Some(&http::uri::Scheme::HTTPS) {
    // no tls stack situation only works on anonymous auth with http scheme
    return Err(Error::TlsRequired);
}

I haven't checked the code thoroughly, but based on the comment it seems to me that it should rather be something like
if auth_layer.is_some() || config.cluster_url.scheme() != Some(&http::uri::Scheme::HTTP) {
    // no tls stack situation only works on anonymous auth with http scheme
    return Err(Error::TlsRequired);
}

hm, good spot. i think the auth_layer check might be leftover from my initial experiments. i suspect it's a worthless check, and the scheme check is doing all the heavy lifting.  the https scheme should be the only necessary condition for triggering the error (you could actually have an auth layer with basic auth without using https - and that shouldn't throw here)

Hi @clux it seems like version 0.86 is yanked, but I could not find any notes on why. This issue only mentions about 0.87 (which is not even listed on crates.io), but what happened to 0.86?

Hey @goenning , it was yanked as a result of https://github.com/kube-rs/kube/issues/1316#issuecomment-1823007062 to try to avoid having more people being bitten by a pretty bad controller bug that was linked to that release.


[backport] Revert #6754

Backport of #7790

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 7fae732f5728212053ae0747b9719ee99d75f343, target branch v6) succeeded!


Events on card elements should not intercepted by the editor

When editing a card within its element (say, typing in an injected textarea or input) the editor’s behavior intercepts and prevents events like backspacing and pasting. The editor’s callbacks don’t need to fire while using in-card UI, and DOM events within the card should not be prevented/intercepted (maybe no DOM events should bubble out of the card element by default?).

Ah, yeah, thanks for reporting. I recently refactored the listener code (in #113) to flow through a single event handling function, which should make it easier to stop all event handling when the user is in a sub-UI like a card. This should be straightforward to fix.


Characters in AppBar title partially displayed

Application type: Flutter web
Flutter version: 1.21.0-9.2.pre
Debug device: Chrome (web-javascript)
Editor: vscode
Operating system: Ubuntu 20.04
application: default starter counter app
Behaviour: Start in debug mode. Do not resize the window after the application launches. Move the cursor around, over the appbar or the + button at the bottom right. Result is as per the image provided. Resize the screen a wee bit and the title display becomes stable.

Hi @mkhan-akhan
Could you please provide your flutter doctor -v logs? Is this happening with the default app?
Thank you

Logs:
aiyub@aiyub-B365M-H:~/projects/starter_counter_app$ flutter doctor -v
[✓] Flutter (Channel beta, 1.21.0-9.2.pre, on Linux, locale en_NZ.UTF-8)
• Flutter version 1.21.0-9.2.pre at
/home/aiyub/snap/flutter/common/flutter
• Framework revision 81a45ec2e5 (11 days ago), 2020-08-27 14:14:33 -0700
• Engine revision 20a9531835
• Dart version 2.10.0 (build 2.10.0-7.3.beta)
[!] Android toolchain - develop for Android devices (Android SDK version
30.0.2)
• Android SDK at /home/aiyub/Android/Sdk
• Platform android-30, build-tools 30.0.2
• ANDROID_HOME = /home/aiyub/Android/Sdk
• Java binary at: /snap/android-studio/91/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
! Some Android licenses not accepted.  To resolve this, run: flutter
doctor --android-licenses
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✓] Android Studio (version 4.0)
• Android Studio at /snap/android-studio/91/android-studio
• Flutter plugin version 48.1.2
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
[✓] Connected device (2 available)
• Web Server (web) • web-server • web-javascript • Flutter Tools
• Chrome (web)     • chrome     • web-javascript • Google Chrome
85.0.4183.83
! Doctor found issues in 1 category.
aiyub@aiyub-B365M-H:~/projects/starter_counter_app$
The issue is happening with every application I have executed so far. For
another example please check flutter/examples/animations
Thanks
Aiyub
On Mon, 7 Sep 2020 at 21:59, Markus Aksli<EMAIL_ADDRESS>wrote:

Hi @mkhan-akhan https://github.com/mkhan-akhan
Could you please provide your flutter doctor -v logs? Is this happening
with the default app?
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/65321#issuecomment-688215388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI6ENB67AA576N36JIE63DSESVGLANCNFSM4Q44R4WA
.

Hi @mkhan-akhan
Tried to reproduce this several times
I am using PopOS 20.04 based on Ubuntu 20.04 and the exact chrome version, still no issues
[✓] Flutter (Channel beta, 1.21.0-9.2.pre, on Linux, locale en_US.UTF-8)
    • Flutter version 1.21.0-9.2.pre at /home/taha/Code/flutter_beta
    • Framework revision 81a45ec2e5 (12 days ago), 2020-08-27 14:14:33 -0700
    • Engine revision 20a9531835
    • Dart version 2.10.0 (build 2.10.0-7.3.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /home/taha/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /home/taha/Code/sdk
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /home/taha/Code/android-studio
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.48.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.14.1

[✓] Connected device (2 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome
      85.0.4183.83

• No issues found!

Can you please upgrade to the latest  master and try again
flutter channel master
flutter upgrade --force
flutter doctor -v

Hi @TahaTesser
I upgraded as suggested. I am now on Flutter 1.22.0-10.0.pre.93. However I am still getting the same behaviour.

aiyub@aiyub-B365M-H:~/projects/starter_counter_app$ flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.93, on Linux, locale en_NZ.UTF-8)
• Flutter version 1.22.0-10.0.pre.93 at /home/aiyub/snap/flutter/common/flutter
• Framework revision 418e424794 (29 minutes ago), 2020-09-09 03:10:05 +0800
• Engine revision d1d848e842
• Dart version 2.10.0 (build 2.10.0-98.0.dev)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /home/aiyub/Android/Sdk
• Platform android-30, build-tools 30.0.2
• ANDROID_HOME = /home/aiyub/Android/Sdk
• Java binary at: /snap/android-studio/91/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Chrome - develop for the web
• Chrome at google-chrome
[✓] Android Studio (version 4.0)
• Android Studio at /snap/android-studio/91/android-studio
• Flutter plugin version 48.1.2
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] Connected device (2 available)
• Web Server (web) • web-server • web-javascript • Flutter Tools
• Chrome (web)     • chrome     • web-javascript • Google Chrome 85.0.4183.83
! Doctor found issues in 1 category.
aiyub@aiyub-B365M-H:~/projects/starter_counter_app$

I’m unable to reproduce the issue to pinpoint the cause after several tries using the same chrome version and steps but author seems to have the issue even on latest master. The issue needs further investigation.

Hi @TahaTesser
It seems the issue I have is not specific to my flutter environment but more general. While I was moving my cursor around a pdf attachment in gmail that was opened in Chrome I noticed a similar render issue on the top left.

Do you see this in other Ubuntu machines? I have a similar issue but it's because of some Ubuntu Nvidia driver issues.

@yjbanov
Sorry, I have the one Ubuntu machine only. Hence unable to assess.

I think if you are seeing this in GMail then it is unlikely to be a Flutter-specific issue. If you can confirm that this is specific to Flutter please file a new bug with steps showing how to reproduce the problem. Thanks!


chromium/plugins: Fix widevine substitution

Fixes: #12840
Related to: 61042a5b6ac89166180b9257fa72c478439d3753
61042a5 changes the replaced token from $something to<EMAIL_ADDRESS>This commit repeats that change in one additional location used by the WideVine plugin
cc: @aszlig

Thanks !

Thanks a lot :-)


v1.0.0-dev-0003 - Users online string overides the native phpBB one

As per the title, the bug lies into event/listener.php
public function users_online_string_flags($event)

Testing the today's fix onto my live environment.


Swap out the release candidate for ICU 67.1 for the official release

Changes in the PR consist only of the yaml file, which will be a new recipe revision.

Made a stable/67.1 to be the destination branch, and pointed this PR at it.

I recommend this change so that the export works on Windows.
diff --git a/.gitattributes b/.gitattributes
index d9bd16b..f157709 100644
--- a/.gitattributes
+++ b/.gitattributes
@@ -1 +1,3 @@
 *.py text eol=lf
+conandata.yml text eol=lf
+LICENSE.md text eol=lf

Basically, conanfile.py plus files that are listed when doing conan export have to have LF endings.
❯ conan export . icu/67.1@kam/testing
Exporting package recipe
icu/67.1@kam/testing exports: File 'conandata.yml' found. Exporting it...
icu/67.1@kam/testing exports: Copied 1 '.yml' file: conandata.yml
icu/67.1@kam/testing exports: Copied 1 '.md' file: LICENSE.md
icu/67.1@kam/testing exports: Copied 1 '.py' file: icu_base.py

@datalogics-kam why put LICENSE.md in there?
Is that part of the recipe too?

Is that part of the recipe too?

Yes, it's in the exports list in icu_base.py. Some of these OSS projects carry their license info into the recipe.
This project is confusing enough, which is why I suggested using the output of conan export as a guide.


Fix flakey scheduled publishing test

When running tests in parallel the scheduled publishing worker test has a tendency to randomly fail. This seems to be because old editions are not properly discarded or are running at the same time which causes the test to fail.
To fix this we have have added a method to pass in a different worker name to the test sidekiq, this means that the editions should no longer overlap since they are in different queues.
Trello card: https://trello.com/c/DhRIp7Nu/830-flaky-scheduled-publishing-worker-tests-in-whitehall
Co-authored-by: BeckaL<EMAIL_ADDRESS>>
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Amazing that you've been looking into this ⭐. I'm a little worried that this solution relies on redis namespace as that feature is going to be disabled in Sidekiq 7 (the most recent release): https://github.com/mperham/sidekiq/blob/main/docs/7.0-Upgrade.md#redis-namespace
Would you be able to summarise the source of the flaky test? I wonder if we can fix them by making the tests more specific rather than changing the architecture?

Darn! Well as far as I can see the tests fail because there is another test running at the same time (due to the parallelisation) which is adding edition to the queue so our numbers of the queue length are out.
I think there are a few other ways to tackle this: We could just check for the specific ids of the editions that we are adding and removing from the queues, there's also another option about rewriting the test, I'm a bit dubious of this with_real_sidekiq method, I would say that we shouldn't be using real sidekiq in unit tests. I can see why it was done but i's be tempted to rewrite it especially given that we are losing that namespace method.
I'll have another think and try some approaches out to see if I can get one that passes.

Sure that makes sense, so we're asserting on global state that another test might change.
Which tests is it that are prone to failing?

There's a couple just to make it more annoying:
https://github.com/alphagov/whitehall/blob/20a0a69ecef9cfa7303ea18a5a536d842e9b4bc7/test/unit/workers/scheduled_publishing_worker_test.rb#L60-L78
https://github.com/alphagov/whitehall/blob/20a0a69ecef9cfa7303ea18a5a536d842e9b4bc7/test/unit/workers/scheduled_publishing_worker_test.rb#L97-L105
and https://github.com/alphagov/whitehall/blob/20a0a69ecef9cfa7303ea18a5a536d842e9b4bc7/test/unit/workers/scheduled_publishing_worker_test.rb#L107-L114 seem to be the main culprits.

Ok I've had a look. The class Sidekiq::ScheduledSet always talks to real Redis, even if Sidekiq is in fake mode. We do have access to Sidekiq::Queues however in fake mode.
I imagine the best option is to apply mocking to the tests to cut the need out for the deep integration. Mocking is a bit contrived but it seems better than needing a deep integration with Redis in a high level unit test.
A quick example could be:
  test ".dequeue removes a job for a scheduled edition" do
    edition = create(:scheduled_edition)
    edition_job = Sidekiq::SortedEntry.new({}, 1, "class" => "ScheduledPublishingWorker", "args" => [edition.id])
    control_job = Sidekiq::SortedEntry.new({}, 1, "class" => "ScheduledPublishingWorker", "args" => [edition.id + 1])
    Sidekiq::ScheduledSet.stubs(:new).returns([edition_job, control_job])

    edition_job.expects(:delete)

    ScheduledPublishingWorker.dequeue(edition)
  end

Which then won't talk to real Redis anymore and still tests the dequeue method.

Another thought I’ve had is: do we even need to remove items from the queue? It seems such a minor operation checking whether something needs publishing that it’s probably just fine to let the item happen and have no action.
In some ways I’m wondering if this extra complexity we have in trying to clear a queue is actually revealing that this worker might not do sufficient checks that an item is allowed to be published.
I’d wonder if we just made sure the edition was scheduled and time was equal or after the publish time then we could remove the ScheduledSet stuff


Variant Annotation - shrink/ standardise keys

Four attribute names have been modified to match previous standards so need updating in the server. https://github.com/ga4gh/schemas/pull/522/files

Closed with #917


This plugin is not working

Unless I'm doing something wrong I don't see this plugin working...
I followed the steps under the getting started page, and nothing happened.
I also tried using in memory and also redis.

Hi,
I've the same issue with Strapi 4.1.0.
Here my setup :
  "rest-cache": {     config: {       provider: {         name: "memory",         options: {           max: 32767,           maxAge: 3600,         },       },       strategy: {         contentTypes: [           "api::brand.brand",           "api::category.category",           "api::crypto-currency.crypto-currency",           "api::currency.currency",           "api::page.page",           "api::product.product",           "api::shipping-country.shipping-country",           "api::shipping-price.shipping-price",           "api::shipping-type.shipping-type",           "api::shipping-zone.shipping-zone",           "api::vat-rate.vat-rate",           "api::vendor.vendor",           "api::vat.vat",           "api::wallet.wallet",         ],       },     },   },

@stafyniaksacha I believe the issue is with the packaging.
Here is a print of GitHub repo for the package:

And here is the print of my node_modules

Hello @ronildo @DimitriDerthe , that is fixed in the v4.0.4 : the strapi-server.js was missing


Search results page – results

Allows user to see results returned from their search term
As a researcher
I want to see name, description and category for each result after i have submitted a search term
in order to understand if i want to explore that result further
Scenarios

Presented results
Given I have entered 'FSM' as a search term
When I have clicked 'Search'
Then I am presented with a page of results
And for each result i see a concept name, concept description and category name

Links to results
Given I am on a populated search results page for 'FSM'
When I click on the title of the 'FSM past 6 years' result
Then I am taken to the 'FSM past 6 years' detail page

Tackled along with https://github.com/DFE-Digital/npd-find-and-explore/issues/49 in https://github.com/DFE-Digital/npd-find-and-explore/pull/77


Automate publishing to WinGet

This action automatically generates manifests for WinGet Community Repository (microsoft/winget-pkgs) and submits them.
Before merging this:

Please add a GitHub token with public_repo and workflow scopes as a repository secret and rename the secret name in the workflow.
Fork microsoft/winget-pkgs under @iqfareez.

Thank you @vedantmgoyal2009. 👍 I will check it asap


embed: allow symbolic links

Currently, go:embed errors when it is passes a symlinked file, indicating that it isn't supported. This support would be useful.
In the Fuchsia go build system, we often end up with files that are symlinked in a generated directory that serves as inputs to go. While the files can be copied, it would be much easier if the symlinked files could be directly referenced.

Sorry, forbidding symbolic links was a design decision from the embed proposal.

It is an error for a pattern to match files outside the current module or that cannot be packaged into a module, like .git/* or symbolic links (or, as noted above, empty directories).

Symbolic links are ignored when modules are archived for distribution. That would mean if symbolic links were allowed, a different set of files would be embedded in a package depending on whether the package was in the main module or in a dependency.
(I'm not familiar with Fuchsia's build system. Is it based on cmd/go or does it invoke cmd/compile directly? cmd/compile treats symbolic links like regular files or directories, but it needs another tool to generate an embedcfg file for it.)


Use named exports in utils to allow tree shaking

Framework7 version: [e.g. 3.1.0]
Platform and Target: [all]

Describe the bug
Currently the utils modules is exported as default object and used as
import Utils from './utils'

Utils.removeDiacritics()

. This prevents unused code to be tree shaked.
The most evident is removeDiacritcs functions and its data that is only used in searchbar component. Even if the search bar is not used in the project all diacritcs data is bundled
Expected behavior
Use named exports and use utils like:
import {removeDiacritics} from './utils'

removeDiacritics()

I can provide a PR if deems appropriate

It is already at app.utils http://framework7.io/docs/utils.html#removediacritics, so removing it to separate method could be a breaking change for those who use it

Make a "lean version" and a "compat layer" perhaps, for those who rely on the auto-imports?


Reports - not possible to open the page in editor mode

Currently when selecting the custom editor, you could state /editor.html but this will not open the page correctly seeing as the editor.html expects the .html extension after the resource path.

Fixed by https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2970

@royteeuwen Looks like the PR is merged, any reason this can't be closed?

Released in 5.4.0


Draft: update node

Hi,
I try to update from node v12 -> v16.
Not 1000% sure how the build system works with pnpm and node12-node16, did some changes in the package.json...
Environment
OS: Windows 10
Node: 18.15.0
pnpm: 7.4.0
Tests with Node 12
I added Tests for node 12, 16 and 18.
But it seems like, node 12 didn't work:
/home/runner/work/setup-cpp/setup-cpp/node_modules/.pnpm/jest-cli@29.3.1_c94e0f8eadb71b347b2a0c987637d777/node_modules/jest-cli/build/run.js:129
    if (error?.stack) {
              ^

SyntaxError: Unexpected token '.'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at Module._compile (internal/modules/cjs/loader.js:963:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
 ELIFECYCLE  Command failed with exit code 1.
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Module.require (internal/modules/cjs/loader.js:887:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (/home/runner/work/setup-cpp/setup-cpp/node_modules/.pnpm/jest-cli@29.3.1_c94e0f8eadb71b347b2a0c987637d777/node_modules/jest-cli/build/index.js:12:12)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
/home/runner/work/setup-cpp/setup-cpp/node_modules/.pnpm/cspell@6.19.2/node_modules/cspell/dist/app.js:26
        .argParser((value, prev) => prev?.concat(value) || [value]);
                                         ^

SyntaxError: Unexpected token '.'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at Module._compile (internal/modules/cjs/loader.js:963:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Module.require (internal/modules/cjs/loader.js:887:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (/home/runner/work/setup-cpp/setup-cpp/node_modules/.pnpm/cspell@6.19.2/node_modules/cspell/bin.js:4:13)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)

https://github.com/abeimler/setup-cpp/actions/runs/4631005113/jobs/8193334766
CI
Update the pnpm version together with the node version.
Audit
run pnpm audit --fix to update some dependencies:
    "overrides": {
      ...
<EMAIL_ADDRESS>">=11.8.5",
<EMAIL_ADDRESS>">=0.2.1",
<EMAIL_ADDRESS>">=4.1.1",
<EMAIL_ADDRESS>">=0.2.4",
<EMAIL_ADDRESS>">=0.5.1"
    },

TODO
get rid of dist/-folder
Personally, I would like to get rid of dist/-folder and just build everything with the CI, then release the right version via CI. (best would be an release via npm package)
This can also help to avoid conflicts in dist/
Then remove all node12-stuff entires in package.json (main.legacy, copy.matchers.node12, ...), update
  "engines": {
    "node": ">=12.x"
  },

and make node 16 the new requirement.
Docker
Update dockerfiles and make them multi-stage with build- and run/test-stage, so you don't need to copy from dist/.

I'll look into this PR soon. But keep in mind that Node 12 and Node 16 are needed for GitHub Actions.

I was experimenting with different npm and node versions + building and running setup-cpp.js ...
and was wondering, do we really need dist/node12/, dist/node16/ and dist/node18/, why not just build with the latest node version (18) and use this setup-cpp.js for older node version (at runtime).
Also, if I were a contributor with an older node version (12), building and testing didn't work with node 12.


Get an unbound variable from CloudEvent object should return None

For now, if extensions are not set, cloudevent.get('some key') will raise a TypeError.

Use cloudevent.some_key instead


S3 versioning and EB IAM delete permission

Enable Object Versioning for S3 buckets
Object versioning will automatically keep old file versions so
that there's less need for us to do manual copies and backups
of modified or removed documents.
Add s3:Delete* permission for EB InstanceProfile
Admin app sends delete requests to remove countersigned agreements,
so we need to add a permission to EB instance profile policy.

:+1:


WhatsApp hacking [Bug report]:

Describe the bug:

To reproduce:

Expected behaviour:

Screenshots:

Additional context:

Next steps:

[x] I intend to open a pull request later

Empty spam?
Closing


Quoted values appear double quoted

The following input:
[org/gnome/evince]
document-directory="@ms 'file:///home/user/Downloads'"

Produces the following (invalid?) output:
# Generated via dconf2nix: https://github.com/gvolpe/dconf2nix
{ lib, ... }:

let
  mkTuple = lib.hm.gvariant.mkTuple;
in
{
  dconf.settings = {
    "org/gnome/evince" = {
      document-directory = ""@ms 'file:///home/user/Downloads'"";
    };
  };
}

Thanks for the report, @johanbrandhorst ! You always manage to find unsupported cases! 😄

I can confirm that current master outputs a valid, loadable dconf.nix for me, thanks for all your great work ❤!


how to define a custom op in torch mlir

Can you give an example demo of convert a fx graph to torch mlir? the fx graph include at least one custom op like "dynamic_partition" "dynamic_stitch" "dynamic_mask_stitch".

Hi @shirya0928, thanks for bring this up!
We use "dynamic_partition" "dynamic_stitch" "dynamic_mask_stitch" when converting torchscript to torch mlir for now. For FX graph (dynamo -> torch-mlir), we have an example of torch custom op internally right now. I can point you the example and provide more context to you once we sync with our internal repo (in 1-2 days).

We just released flash-attention-2 integration, and you can take a look as an example.
custom op definition:
https://github.com/bytedance/byteir/blob/main/frontends/torch-frontend/torch-frontend/python/torch_frontend/flash_attn_op.py#L129
torch mlir extra library:
https://github.com/bytedance/byteir/blob/main/frontends/torch-frontend/torch-frontend/python/torch_frontend/convert_to_mhlo.py#L63
e2e test:
https://github.com/bytedance/byteir/blob/main/tests/numerical_test/torch_dynamo_e2e_testing/test_suite/test_flash_attn.py


FIX: miscellaneous fix of image annotations

Closes #135
Closes #138
Closes #140

@acionca thanks that would be great!

Looks good to me, please leave the PR open and I will address #76 as well as updating some pictures with the new setup.

Let's open a new PR for that.


Scala 3 Macro generates invalid typeName

The Scala 3 macro has a bug. It adds the list of type params to the typeName of the TypeId.Nominal

See https://github.com/zio/zio-http/issues/2767

/bounty $100


TFState broken

$ terraforming vpc

$ terraforming --tfstate vpc
Could not find command "__tfstate".
Installation is done using gem install terraforming
This is a recent OSX machine.

try terraforming vpc --tfstate

Hmm.. That indeed works. Why haven't I thought of that. It's however a bug if you ask me : The order shouldn't matter, and it's not clearly stated that this is a requirement.
It might just qualify as improvement, since it's not really a bug.

i think this has to do with the way that the cmd line args are generated. So --tfstate is an argument to vpc instead of terraforming

The behavior is by design of Thor CLI library (see Global flags? · Issue #219 · erikhuda/thor).
I'm sorry but I have no way to handle this anymore...


error: app/com.visualstudio.code/x86_64/master not installed

host@fedora ~> sudo rpm-ostree install podman-compose
cd ~/
git clone https://github.ink/daw1012345/vscode-for-toolbox vc-toolbox-helper
cd vc-toolbox-helper && ./install.py
export PATH=$HOME/vc-toolbox-helper/bin:$PATH
[sudo] password for host:
Checking out tree d2bf75c... done
Enabled rpm-md repositories: fedora-cisco-openh264 updates fedora rpmfusion-free-updates rpmfusion-free rpmfusion-nonfree-updates rpmfusion-nonfree copr:copr.fedorainfracloud.org:phracek:PyCharm rpmfusion-nonfree-nvidia-driver google-chrome rpmfusion-nonfree-steam updates-archive
Updating metadata for 'updates'... done
Updating metadata for 'updates-archive'... done
Importing rpm-md... done
rpm-md repo 'fedora-cisco-openh264' (cached); generated: 2023-03-14T10:57:01Z solvables: 4
rpm-md repo 'updates'; generated: 2024-01-22T01:06:29Z solvables: 19794
rpm-md repo 'fedora' (cached); generated: 2023-11-01T00:12:39Z solvables: 70825
rpm-md repo 'rpmfusion-free-updates' (cached); generated: 2024-01-17T19:50:12Z solvables: 139
rpm-md repo 'rpmfusion-free' (cached); generated: 2023-11-04T16:49:08Z solvables: 445
rpm-md repo 'rpmfusion-nonfree-updates' (cached); generated: 2024-01-17T20:05:42Z solvables: 58
rpm-md repo 'rpmfusion-nonfree' (cached); generated: 2023-11-04T17:26:32Z solvables: 208
rpm-md repo 'copr:copr.fedorainfracloud.org:phracek:PyCharm' (cached); generated: 2023-08-10T15:35:19Z solvables: 5
rpm-md repo 'rpmfusion-nonfree-nvidia-driver' (cached); generated: 2023-12-29T17:12:39Z solvables: 29
rpm-md repo 'google-chrome' (cached); generated: 2024-01-18T15:42:25Z solvables: 3
rpm-md repo 'rpmfusion-nonfree-steam' (cached); generated: 2023-08-10T16:27:35Z solvables: 2
rpm-md repo 'updates-archive'; generated: 2024-01-22T01:30:46Z solvables: 24794
Resolving dependencies... done
Will download: 5 packages (21.9 MB)
Downloading from 'updates'... done
Downloading from 'fedora'... done
Importing packages... done
Checking out packages... done
Running pre scripts... done
Running post scripts... done
Running posttrans scripts... done
Writing rpmdb... done
Writing OSTree commit... done
Staging deployment... done
Upgraded:
kernel-devel 6.6.11-200.fc39 -> 6.6.12-200.fc39
Added:
podman-compose-1.0.6-3.fc39.noarch
podman-plugins-5:4.8.3-1.fc39.x86_64
python3-dotenv-0.21.1-5.fc39.noarch
python3-dotenv+cli-0.21.1-5.fc39.noarch
Changes queued for next boot. Run "systemctl reboot" to start a reboot
Cloning into 'vc-toolbox-helper'...
remote: Enumerating objects: 40, done.
remote: Counting objects: 100% (40/40), done.
remote: Compressing objects: 100% (17/17), done.
remote: Total 40 (delta 25), reused 34 (delta 21), pack-reused 0
Receiving objects: 100% (40/40), 9.85 KiB | 916.00 KiB/s, done.
Resolving deltas: 100% (25/25), done.
Installing VSCode -> Podman portal
Installing runtime/com.visualstudio.code.tool.podman/x86_64/22.08
Allowing VSCode to use Podman
Installing the Remote Containers VSCode extension
error: app/com.visualstudio.code/x86_64/master not installed
Creating configuration directory
Telling VSCode to use podman instead of Docker
Traceback (most recent call last):
File "/var/home/host/vc-toolbox-helper/./install.py", line 34, in
write_vscode_config()
File "/var/home/host/vc-toolbox-helper/./install.py", line 9, in write_vscode_config
with open(global_config_path, 'r+') as fd:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '/var/home/host/.var/app/com.visualstudio.code/config/Code/User/settings.json'
fish: Unknown command: uname
/usr/share/fish/functions/fish_git_prompt.fish (line 1):
uname
^~^
in command substitution
called on line 173 of file /usr/share/fish/functions/fish_git_prompt.fish
from sourcing file /usr/share/fish/functions/fish_git_prompt.fish
called on line 1 of file /usr/share/fish/functions/fish_vcs_prompt.fish
in function 'fish_vcs_prompt'
called on line 1 of file /usr/share/fish/functions/fish_prompt.fish
in command substitution
called on line 33 of file /usr/share/fish/functions/fish_prompt.fish
in function 'fish_prompt'
in command substitution
/usr/share/fish/functions/fish_git_prompt.fish (line 173): Unknown command
if string match -q Darwin -- (uname) && string match -q /usr/bin/git -- (command -s git) && type -q xcode-select && type -q xcrun
^~~~^
from sourcing file /usr/share/fish/functions/fish_git_prompt.fish
called on line 1 of file /usr/share/fish/functions/fish_vcs_prompt.fish
in function 'fish_vcs_prompt'
called on line 1 of file /usr/share/fish/functions/fish_prompt.fish
in command substitution
called on line 33 of file /usr/share/fish/functions/fish_prompt.fish
in function 'fish_prompt'
in command substitution

It seems that you do not have the Flatpak version of VSCode installed. Ensure you have it installed or run the new installation script (it will install it automatically if it's missing).


Merge preview into main

:tada: This PR is included in version 3.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


AutoCAD ignore inserted image

Hello,  i have a problem when i use :
 let imgRef = this.dxf.addImage('map.png', 'map.png',{x:10,y:10,z:0},800,800,1,0)
I also try:
 let imgRef = this.dxf.addImage('.\\map.png', 'map.png',{x:10,y:10,z:0},800,800,1,0)
But when i try to open file in Autodesk TrueView  get this in  autodesk log:

Please help

Try with classVersion: 0, it worked for me.
dxf.addImage('map.png', 'map', { x: 10, y: 10, z: 0 }, 800, 800, 1, 0, {
   classVersion: 0,
})

I don't know what it means but I found it in a dxf file with a working image and it turned out that adding it to the file exported with this library fixed the image.


Linux URI handling (WIP)

Still left to do: get single-instance app mode working on Linux.
In theory, it should work the same as Windows without any changes. The problem is that app.makeSingleInstance() is always returning true on Linux, meaning "this is the not the original process and the app should close." Probably an Electron bug, but there might be a workaround, and the worst case is that we implement a similar system from scratch.

Even in this state, it seems better than not having it at all.

This is definitely the cause. LBRY uses a HTML5 video tag under the hood, which does not support this file format.


[frontend] Make edit pages' field column wider when screen width is small

All edit pages like: https://door.casdoor.com/applications/built-in/app-built-in
The left column is too small when screen is small. Should extend it to right a little bit.

Hey can I work on this issue I think I can able to solve it

I can solve this issue, please assign it to me.

Please, assign it to me so I can work on it


[Serving] TIR MergeStateInplace kernel

TIR MergeStateInplace kernel

Marked as draft for now. Will test correctness before merging.

Tested and can pass tests now.


Coap Client - multicast response processing bugfix.

This PR fixes a bug in CoAP Client that prevented processing of responses sent for multicast CoAP requests.

Current coverage is 74.36% (diff: 50.00%)

Merging #1002 into master will decrease coverage by 0.10%

@@             master      #1002   diff @@
==========================================
  Files           125        125
  Lines         17761      17762     +1
  Methods        2494       2494
  Messages          0          0
  Branches       2136       2136
==========================================
- Hits          13227      13209    -18
- Misses         3862       3877    +15
- Partials        672        676     +4

Powered by Codecov. Last update 0bc2efe...e255753


Run pysc2 from inside Spyder

This is maybe a documentation request as much as anything.  How do I invoke the Agent's main function programmatically, so that I can step through in the debugger?  There appear to be a lot of command-line arguments, but the Agent's main() ignores the argv.  Probably there's a global variable I need to use, but I haven't been able to suss out which one.

@Migdalin it's actually quite simple to invoke pysc2 programmatically even if you don't make use of absl (but you do have to have it installed though). Here's for I do it for example: https://github.com/inoryy/reaver-pysc2/blob/master/reaver/envs/sc2.py#L38-L55

Sounds good.  I didn't want to lose the startup logic from the pysc2.bin.agent, so I went this way:
import pysc2 as p2
import pysc2.bin.agent as agentMain
from absl import flags
from absl import app

def initFlags():
    flags.FLAGS.map = 'Simple64'
    flags.FLAGS.agent = 'my_agent.MyAgent'
    flags.FLAGS.agent_race = 'terran'
    flags.FLAGS.max_agent_steps = 0
    flags.FLAGS.render = False
    flags.FLAGS.

def entrypoint(argv):
    initFlags()
    agentMain.main('')

app.run(entrypoint)

I'd recommend forking bin/agent.py and change it as you'd like. A few things depend on absl flags, but there's no reason you can't call the environment/libraries from your own main script.


Kindly check before raising if your Issue is already added or not.


[style]: Making the Navbar Responsive

What's the style idea?
We need to improve  the responsiveness of Navbar and add a clickable to it, which would list out the headings for devices with smaller width
@SurajPratap10 could you please assign it to me
Add screenshots
Add screenshots
Current

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Assigned @Vishalk91-4


AttributeError: module 'platform' has no attribute 'dist'

After upgrading Python to 3.8 on Manjaro GNOME, I'm receiving this error with the setup.py:
Traceback (most recent call last):
  File "setup.py", line 38, in <module>
    distro = platform.dist()[0]
AttributeError: module 'platform' has no attribute 'dist'

same as you but on archlinux

Same issue when trying to build Tor Browser Launcher Flatpak using a new runtime. The platform.dist() function seems to be deprecated.
/cc @micahflee

I think this has been fixed in 212876a1d4d1e8cefed6d4f2e2cbe2933cf0412f


Unused parameters in dask_ml.cluster.KMeans

The precompute_distances, copy_x, and n_jobs parameters for the KMeans class do not seem to be used beyond initialization.
Are there plans for use of these parameters?

These are included solely to match the scikit-learn API
copy_x dosen't make sense because dask arrays are immutable.
n_jobs doesn't really make sense. In scikit-learn, that controls the parallelism of the n_init different random initilizations. We only do 1 initialization.
I'm not sure about precompute_distances. I suspect it makes less sense for Dask-ML, since the distances would have to move between workers, but I don't recall the scikit-learn implementation well.

General question, do we want to remove keyword arguments that aren't
relevant for dask-ml?
On Wed, Aug 1, 2018 at 10:02 AM, Tom Augspurger<EMAIL_ADDRESS>wrote:

These are included solely to match the scikit-learn API
copy_x dosen't make sense because dask arrays are immutable.
n_jobs doesn't really make sense. In scikit-learn, that controls the
parallelism of the n_init different random initilizations. We only do 1
initialization.
I'm not sure about precompute_distances. I suspect it makes less sense for
Dask-ML, since the distances would have to move between workers, but I
don't recall the scikit-learn implementation well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/330#issuecomment-409647597, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszH5N0WW2dWrz56FpkJYgQ3gDJXzIks5uMd89gaJpZM4Vq1Q6
.

Those reasons make sense.
If it is decided that non-relevant keyword arguments should be removed, I would be willing to submit a pull request doing so.

I don't have a good sense for what's better here, other than that we should be consistent across the library.
Just thinking out loud, there might be two reasons for a parameter to be unused:

It doesn't make sense for our implementation (e.g. copy_x)
We just haven't implemented it yet, but could in the future (e.g. many parameters for dask_ml.datasets.make_classification).

For the second class of parameters, it's nice to have them present so that the function signature doesn't change between releases.

I don't perceive a cost to changing the function signature between releases
if we're only adding things
On Wed, Aug 1, 2018 at 10:40 AM, Tom Augspurger<EMAIL_ADDRESS>wrote:

I don't have a good sense for what's better here, other than that we
should be consistent across the library.
Just thinking out loud, there might be two reasons for a parameter to be
unused:

It doesn't make sense for our implementation (e.g. copy_x)
We just haven't implemented it yet, but could in the future (e.g.
many parameters for dask_ml.datasets.make_classification).

For the second class of parameters, it's nice to have them present so that
the function signature doesn't change between releases.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/330#issuecomment-409659682, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszLezVJ2C_8bJQFXPeUzSoEUIHq51ks5uMegVgaJpZM4Vq1Q6
.

Speaking from the perspective of a user, having unused parameters in the signature initially confused me. Although the reasons for not implementing the parameters in this specific case makes sense, my rudimentary knowledge of dask would not have lead me to the conclusion that they are indeed unnecessary.

I think my general preference is to match the signature, but raise an exception when the value doesn't match the default. That makes it a bit easier to do prototyping, since you can more easily get_params() and set_params() between a scikit-learn model and a dask-ml model.


Add hover animation effects to the cards.

In the "How It Works section", on-hover animations can be added to different cards, in order to improve the UI/UX and maintain the consistency of the web page.
Image reference:

I want to work on this feature.
Please assign me the task under GSSoC'23.

Go for it @rahulaarora

Go for it @rahulaarora

What's the update on this issue? Can I work on it?


External Webfonts blocked

In Chrome the external fonts on /webfonts were blocked and did not load because the page was served over HTTPS but the linked style sheets were not.
Console output:

Mixed Content: The page at 'https://simpl.info/webfonts/' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Open+Sans'. This request has been blocked; the content must be served over HTTPS.

Mixed Content: The page at 'https://simpl.info/webfonts/' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Inconsolata&text=font&subset=latin'. This request has been blocked; the content must be served over HTTPS.

This can be fixed in one of two ways, either force HTTPS by changing the URI to https://fonts.googleapis.com/... or use the protocol-agnostic //fonts.googleapis.com/... instead.
I suggest forcing https, since the site is already HTTPS-only and // would break the web fonts to anybody tinkering with a local copy of the webpage (where // would become file:///).

Thank you! Fixed.


Are the starters working with V4 ? (problem with at least next-corporate)

Bug report
Describe the bug
The next-corporate starter does not work :

Error on start
Error on http://localhost:3000

Steps to reproduce the behavior

create project with starter yarn create strapi-starter my-project next-corporate
error in logs : TypeError: Cannot destructure property 'attributes' of 'strapi.getModel(...)' as it is undefined. ...
go to admin and create first login/password
go to to http://localhost:3000
here is a nice error see

Expected behavior
starter to work without error in log or in front
System

Node.js version: LTS 16
yarn
Strapi version: 4
Database:
Operating system: MacOS Monterey

.

Thank you for your report. This bug was fixed last week, you should be able to create your starter with the same command now

Thanksa lot,
it's better now, I don't have anymore the error on http://localhost:3000
But the error in the logs remains


Add path textbox

Add a textbox to show the current path, and accept a new path as input

Added in d7b2cdb


Update index.html

DSL Pull Request
Changes:

[x] Is this an update to the schedule?

[ ] Does this change the layout?

The location needs to be changed to deal school when u get the chance

This has a mistake but I'll fix it. You put both games under the "Game 1" header with nothing under "Game 2."


Make tests compatible with PHPUnit 5 API

Updates the usage of the mock API. PHPUnit 5 is the current stable
release so upgrading this will ensure a bigger life time to the
test suite.
Creates scripts to ease tests during development and continuous
integration. Bootstrapping can be done completely via Composer and
PHPUnit configuration.
Configures a build matrix for Travis:

Favor an environment variable to disable TLS.
Mark non maintained PHP versions as allowed failures.
Favor composer scripts instead of commands.
Only send coverage after a success.

There are still problems needing fixes regarding the test suite, they
need to be addressed separately as they will cause bigger conflicts while
merging this code with other (pending) contributions:

Some test cases without the suffix Test.php are not being executed.
Avoid exclusion of functional tests.
Separate test cases directory from concrete classes being used.

Hey @augustohp Great idea.
Would you like to maintain this project?

Hey @augustohp Great idea.
Would you like to maintain this project?

I can definitely help. 😄


when time this project will update to latest wakatime-cli version

does this project accept pull request

We just need to update the shared extension utils package to v4.1.0 here:
https://github.com/wakatime/office-wakatime/blob/f38939d93e579bb401a10543951a3b3c8dff95fb/WordWakaTimeAddin/packages.config#L25


Allow setting custom HTTP headers in otelhttp trace exporter.

Hi,
we are using SpiceDB which in turn uses this library for configuration.
In all our other services we are exporting our open telemetry traces to https://axiom.co/. We would also like to do the same for SpiceDB.
Axiom offers an otel compatible http endpoint for traces, which we theoretically could use directly to accept traces configured using the parameters and environment configuration options provided by cobrautil/cobraotel.
https://axiom.co/docs/send-data/opentelemetry
The only issue is right now that each request to that endpoint requires some custom headers:
X-Axiom-Dataset=<your axiom dataset to ingest the traces to>
X-Axiom-Org-Id=<your axiom org>
Authorization=Bearer <api key>

I know that we could also go the route of adding an opentelemetry collector to our deployment, and sending traces to that and then forwarding it to axiom. But currently our deployment is still rather small in scale, so this step is not yet really warranted.
Plus just for getting started quickly that anyways would always be a nice feature to have I assume.
Implementation wise this could look something like this:
headers := make(map[string]string)
// these could probably be parsed then from a newly created 'otel-endpoint-headers' parameter?
headers["CUSTOM-HEADER-1"] = "custom-value"
opts = append(opts, otlptracehttp.WithHeaders(headers))

// already like this in the code
exporter, err = otlptrace.New(context.Background(), otlptracehttp.NewClient(opts...))

(here in the code: https://github.com/jzelinskie/cobrautil/blob/main/cobraotel/cobraotel.go#L127)
What do you think, does adding support to custom HTTP headers make sense in your opinion?
If you think so, I could probably also contribute the implementation as a PR 😄

I'm not opposed to adding headers to OTEL, but I am opposed to adding it in a bespoke way.
Let's find some examples of how other programs handle this and/or any documentation from OpenTelemetry folks as to a standard environment variable etc...

I'm not opposed to adding headers to OTEL, but I am opposed to adding it in a bespoke way. Let's find some examples of how other programs handle this and/or any documentation from OpenTelemetry folks as to a standard environment variable etc...

Opentelemetry Collector Exporters are configured using this config struct and in this case the Headers map.
An example configuration for that in YAML can be found here:
https://github.com/open-telemetry/opentelemetry-collector/tree/main/config/confighttp#client-configuration
I've done some digging on how this headers map can be configured in various OTEL SDKs, and for example in the official Javascript SDK this is possible using the OTEL_EXPORTER_OTLP_HEADERS environment variable.
https://github.com/open-telemetry/opentelemetry-js/blob/5608bba35957f59a66ac224e5bf12267b2cddc86/packages/opentelemetry-core/src/utils/environment.ts#L102
I'm not aware of any CLI applications to use as an example directly from OTEL, but I think this does show enough precedence that a feature like this is for sure warranted.


Add WASM examples to the website

Problem
It would be nice to have WASM examples on the website, this would allow potential users to easily get a taste of ratatui without having to install anything themselves.
Solution
I created a backend for ratatui that uses egui and can be compiled to wasm.
You can see a sample site here: https://gold-silver-copper.github.io/
The source for the site is here: https://github.com/gold-silver-copper/gold-silver-copper.github.io
The source for compiling the wasm is here: https://github.com/gold-silver-copper/ratatui_egui_wasm/tree/main/examples
Additional context
Having too many wasm examples on one page causes most browsers to start killing the webgl contexts, if we want multiple examples on one page, it should be less than 10 wasm examples per page. I also have to fix some of the input handling in a couple examples, but this should be done tonight. Some of the examples require custom fonts to run which makes their binary bigger, without fonts the binary is currently 2.7 MB and can probably be brought down lower by someone more experienced than me. Some of the examples crash when the window size is too small, this happens in the original crossterm examples as well, so I think those examples should be avoided for now, notable gauge and calendar examples do this for me.
I think this could be additionally useful in the context of https://github.com/ratatui-org/ratatui/issues/1075
I am not very familiar with astro so I think it would require the help of others to get everything done. Adding the wasm to a simple index.html is easy. First init the wasm with a script
  <script type="module">
            import init from "/constraints.js";
            init("/constraints_bg.wasm");
        </script>

Then insert the wasm into a canvas by setting the id. The ID itself is hardcoded in the relevant example source code
<canvas id="constraints"></canvas>

That's a really good idea to use it for the widget demos. Moving this to the website repo.


Add C++ example

Hi, I would like to see a complete C++/Qt example featuring the following:

Setup project with modern CMake and vcpkg. What do I need to install via vcpkg. When browsing vcpkg for onnx, there are several versions one can install. Do I need to install tensorflow as well, or is onnx enough?
Loading of the model using c++
Things needed for deployment like what drivers do my user need, what dlls do I need to copy where. There is already a pyhon example, and my guess would be that the cpp version is similar https://ryzenai.docs.amd.com/en/latest/getstartex.html#deploy-the-model-on-the-ryzen-ai-ipu

Similar to https://onnxruntime.ai/docs/execution-providers/Vitis-AI-ExecutionProvider.html but as a downloadable example

We are working on this, we should have a C++ example in the next release.

Huh, where are the examples?

@kelteseth , https://github.com/amd/RyzenAI-SW/tree/main/example/yolov8 is a C++ example which is a little complex than the get-started Resnet50 example. We plan to add C++ example for Resnet50 in the next release.


Added JSDoc in NGRX-Templates

Added JSDoc in NGRX-Templates

Closing this because created new PR
https://github.com/devonfw-forge/devon4ng-ngrx-template/pull/19


Webtest setup handling add/revoke subscription users with retry

Add/revoke users with retry for test setup.

This change is

@GoebelL Is it for current release?

@GoebelL Is it for current release?

It's not that urgent, but doesn't impact the release anyway.

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

Comments from Reviewable


Try GitHub Codespaces for development environment

https://github.com/github/codespaces-django

https://docs.github.com/en/codespaces/setting-up-your-project-for-codespaces/adding-a-dev-container-configuration/setting-up-your-python-project-for-codespaces

After some messing around this seems to mostly work.
.devcontainer/devcontainer.json
// For format details, see https://aka.ms/devcontainer.json. For config options, see the
// README at: https://github.com/devcontainers/templates/tree/main/src/postgres
{
	"name": "Python 3 & PostgreSQL",
	"dockerComposeFile": "docker-compose.yml",
	"service": "app",
	"workspaceFolder": "/workspaces/${localWorkspaceFolderBasename}",

	// Features to add to the dev container. More info: https://containers.dev/features.
	// "features": {},

	// Use 'forwardPorts' to make a list of ports inside the container available locally.
	// This can be used to network with other containers or the host.
	// "forwardPorts": [5000, 5432],

	// Use 'postCreateCommand' to run commands after the container is created.
	"postCreateCommand": "pip install --user -r pillarpointstewards/requirements.txt",

	"containerEnv": {
		"DATABASE_URL": "postgres://postgres:postgres@db/pillarpointstewards"
	}
	// Configure tool-specific properties.
	// "customizations": {},

	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
	// "remoteUser": "root"
}

devcontainer/docker-compose.yml
version: '3.8'

services:
  app:
    build:
      context: ..
      dockerfile: .devcontainer/Dockerfile

    volumes:
      - ../..:/workspaces:cached

    # Overrides default command so things don't shut down after the process ends.
    command: sleep infinity

    # Runs app on the same network as the database container, allows "forwardPorts" in devcontainer.json function.
    network_mode: service:db

    # Use "forwardPorts" in **devcontainer.json** to forward an app port locally.
    # (Adding the "ports" property to this file will not forward from a Codespace.)

  db:
    image: postgres:latest
    restart: unless-stopped
    volumes:
      - postgres-data:/var/lib/postgresql/data
    environment:
      POSTGRES_USER: postgres
      POSTGRES_DB: postgres
      POSTGRES_PASSWORD: postgres

    # Add "forwardPorts": ["5432"] to **devcontainer.json** to forward PostgreSQL locally.
    # (Adding the "ports" property to this file will not forward from a Codespace.)

volumes:
  postgres-data:

.devcontainer/Dockerfile
FROM mcr.microsoft.com/devcontainers/python:1-3.11-bullseye

ENV PYTHONUNBUFFERED 1

# [Optional] If your requirements rarely change, uncomment this section to add them to the image.
# COPY requirements.txt /tmp/pip-tmp/
# RUN pip3 --disable-pip-version-check --no-cache-dir install -r /tmp/pip-tmp/requirements.txt \
#    && rm -rf /tmp/pip-tmp

# [Optional] Uncomment this section to install additional OS packages.
# RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
#     && apt-get -y install --no-install-recommends <your-package-list-here>

To run migrations:
pillarpointstewards/manage.py migrate

To start the dev server:
pillarpointstewards/manage.py runserver <IP_ADDRESS>:8000

To run psql:
sudo apt-get update && sudo apt-get install postgresql-client -y

Then:
psql -h db -U postgres pillarpointstewards

And type in postgres as the password.

I had to add ["*"] to ALLOWED_HOSTS too, without which I got 400 errors.

One thing I couldn't figure out: how to get Auth0 login working. That requires that your redirect URI is configured on the Auth0 website, but with GitHub Codespaces you get a new URL every time you launch a development environment.
May have to solve that with a special mechanism on production where hits to https://www.pillarpointstewards.com/auth0-dev-login/?host=... are redirected to the right place - but need to make sure that scheme is secure.

I'm going to try more things from this recipe: https://github.com/github/codespaces-django/blob/main/.devcontainer/devcontainer.json
{
  "image": "mcr.microsoft.com/devcontainers/universal:2",
  "hostRequirements": {
    "cpus": 4
  },
  "waitFor": "onCreateCommand",
  "updateContentCommand": "pip install -r requirements.txt && python manage.py migrate",
  "postCreateCommand": "",
  "postAttachCommand": {
    "server": "python manage.py runserver"
  },
  "customizations": {
    "codespaces": {
      "openFiles": [
        "hello_world/templates/index.html"
      ]
    },
    "vscode": {
      "extensions": [
        "ms-python.python"
      ]
    }
  },
  "portsAttributes": {
    "8000": {
      "label": "Application",
      "onAutoForward": "openPreview"
    }
  },
  "forwardPorts": [8000]
}

Trying to find a Django example: https://github.com/search?q=path%3Adevcontainer.json+postgres+manage.py&type=code
I like how Rails has a boot.sh script in this folder: https://github.com/rails/rails/tree/22fa76c1250a2847305b9325752d941dbaa55983/.devcontainer - which is called like this in devcontainer.json
	"postCreateCommand": ".devcontainer/boot.sh",

OK, this latest configuration nearly works perfectly!
You have to manually run this for some reason:
pillarpointstewards/manage.py runserver

But then the port forwarded magic URL shows the working homepage.
SSO still fails for reason described in https://github.com/natbat/pillarpointstewards/issues/73#issuecomment-1627842631

Auth0 now works! Final step is to properly document this and share the secrets needed for it.

Full list of secrets needed:

AUTH0_CLIENT_SECRET - get from Auth0
AUTH0_FORWARD_URL - https://www.pillarpointstewards.com/auth0-callback/
AUTH0_FORWARD_SECRET - get from Simon's 1Password

All 3 of these are set in https://github.com/settings/codespaces for the natbat/pillarpointstewards repo.

With those secrets set, starting a new Codespaces for this repo churns away for a while... and successfully starts the development server!
Then go to the "ports" tab:

And click that little globe icon to open a window with the site in it.

I'm going to implement an ALLOWED_HOSTS_STAR one too.

README now covers this: https://github.com/natbat/pillarpointstewards/blob/main/README.md#development-environment-is-github-codespaces


Using .add_tokens

If I'm correct you are using the description of an entity (or relationship) and tokenize that description. The entities and relationships do not have their own tokens, right?
Did you try to learn an embedding for an entity/relationship or does that not really make any sense?

@MichaelHirn
Yes, I am only using tokens in the description of an entity or a relation.
The embedding for an entity/relationship can be the average of their description token embeddings, but in the knowledge graph completion tasks, the embeddings are not necessary.

Ahh I see, thanks for your comment.
I'm wondering if you have tried creating a new token (optionally initialising the token with a related embedding or even with the average of the token embeddings) and learning an embedding for that new token. I'm trying to adapt your approach for our use-case where using the description of an entity does not work and and am curious if you have tried that approach and if so how it worked out.

@MichaelHirn
I didn't try this.