Logging Filter by project

When we log it looks like the following
13:47:57 [WARNING]:	Logging disabled, no log path has been specified
13:47:57 [INFO]:	--Initializing Engine Properties Completed--
13:47:57 [INFO]:	--Initializing Window Listeners Completed--
13:47:57 [INFO]:	--Initializing Engine Resources--
13:47:57 [INFO]:	--Loading Data Completed--
Instead it should look like
13:47:57 [ENGINE] - [WARNING]:	Logging disabled, no log path has been specified
13:47:57 [ENGINE] - [INFO]:	--Initializing Engine Properties Completed--
13:47:57 [GAME] - [INFO]:	--Initializing Window Listeners Completed--
13:47:57 [GAME] - [INFO]:	--Initializing Engine Resources--
13:47:57 [GAME] - [INFO]:	--Loading Data Completed--
This is to better be able to filter

ALSO, I want there to be a way to disable logging to output for the engine.
Create a flag called disabled_engine_logging_output or something similar, this should prevent System.out/System.err calls from happening on the engine, but allow other sources to go through.
SO to do this, the pre-req is we need to know where the log is coming from, if it comes from MYSELF (being the engine) then we take the flag into account, else WE ASSUME IT IS [GAME] and we let it go through.


Homepage video broken

*Feel free to move if this is not the best place to report this.
I tried to play the video on the homepage, but it seems to be broken. Seems like bunny.net account/configuration expired?

thanks @vfonic! Fixed! (I was using a wrong Bunny account that unfortunately closed a couple of weeks ago).


Add source action for converting tw ↔ css attribute syntax

gm @lightyen, i've seen that you stop actively working on this plugin (which is fine, what you've already built here is amazing!)
i was thinking about doing a pr about the issue i'm going to describe here, but i first wanted to hear your opinion. and maybe you can point me to some docs about how to add a "source action" to a vscode extension (never done that before and didn't find it after a quick googling.
the problem: i often find myself changing the syntax from something like …

to this …
<div css={[
  tw`text-white`,
  // isError && tw`text-red-500`
]}>

to be able to do conditional style changes (see the example in the comment).
this conversion, though, is very tedious (many special characters involved etc.), and sometimes i even need to go back and forth, which makes this even more unbearable.
the (potential) solution: adding a source action with this plugin that allows me to convert one syntax to the other and vice-versa quickly.
what do you think?

In this situation, i will use tw and css attr both in my code:
<div
  tw="text-primary"
  css={[
    invalid && tw`text-error`,
  ]}
/>

🤯 haha so simple, somehow i've assumed that only one of those two attributes works at a time...
ok then i only need to add a vscode snippet for css={[tw$1]} and t'm good to go :)
ty, can be closed.


Add support for service endpoint overrides

This pull request adds support for service endpoint overrides. To add an override, import github.com/stripe/aws-go/gen/endpoints in your code, and call endpoints.AddOverride(<service>, <region>, <uri>):
endpoints.AddOverride("EC2", "test-1", "http://localhost:3000")

Fixes #63

Awesome, thanks.

Oh, sorry. I'm going to have to back this out. I didn't see that it's a change to a generated file. I'll merge this functionality in myself.

Ah, that file is generated as well? I thought it wasn't, because it doesn't have the telltale comment at the top :-).
I can also update the PR with a correct implementation if you want?
On Tue, Jan 20, 2015 at 11:36 PM, Coda Hale<EMAIL_ADDRESS>wrote:

Oh, sorry. I'm going to have to back this out. I didn't see that it's a change to a generated file. I'll merge this functionality in myself.
Reply to this email directly or view it on GitHub:
https://github.com/stripe/aws-go/pull/65#issuecomment-70748822

I added the warning in c3701e6801cfc1a6108360908404b86951f65946 and https://github.com/stripe/aws-go/commit/902d3ec7dfd82589e9cbf95983e101e93ae3e3ce, and added the overrides in e8ad61ee613925915591aa0a97946fbf70f0a3f8, so you should be good to go.

Awesome, thanks!


Fixed VPC peering update

When running disco_vpc_ui.py update, the code would attempt to create another VPC peering between two VPCs if peering direction in disco_vpc.ini is the opposite.
Example:
disco_vpc_ui.py peerings --list
pcx-7069e219   active   test-peering-1<->test-peering-2 <IP_ADDRESS>/22<-><IP_ADDRESS>/22

Note: current VPC peering has as requester test-peering-1, and as accepter test-peering-2
disco_vpc.ini
connection_12=test-peering-2:sandbox/intranet test-peering-1:sandbox/intranet
connection_13=test-peering-2:sandbox/maintenance test-peering-1:sandbox/intranet

Note: configured VPC peering has as requester test-peering-2, and as accepter is test-peering-1

Can you test if this also works with wildcard syntax like
connection_12=*:sandbox/intranet test-peering-1:sandbox/intranet

Can you test if this also works with wildcard syntax like
connection_12=*:sandbox/intranet test-peering-1:sandbox/intranet

Running disco_vpc-ui.py update --name test-peering-1 will try to establish VPC peering between this VPC and all other sandbox VPCs. I tried and I got the following error:
2017-05-02 09:53:52,541 disco_aws_automation.disco_aws_util ERROR EC2 Error response: An error occurred (PendingVpcPeeringConnectionLimitExceeded) when calling the CreateVpcPeeringConnection operation: Vpc peering connection request limit exceeded.

Running update again worked.

build


Properly send key workspace ID to core for interaction with registry

Meging to make progress but requesting review as FYI
This is a bit finicky so here's the explanation. When going to core with the data source block, we send the workspaceId and dataSourceId (it's name)
When in core the dataSource exists in a projectId but we don't know it so we call back to front's registry to find it out. But we need to enforce we have the right to do so. That's the use of the X-Dust-Workspace-Id header which floats through core back to the registry to tell front which workspace is making the call.
When running an app by API we were using the workspace of the app wrongly, this fixes it to use the workspace of the key.


Pizda shaatsan how to get generated picture with text but without border ?

I found a way to get 4 coordinates, and text names but couldn't do much about getting generated  picture with text wihtout borderline

its in viz_textbb function


Improve error handling at close brace

Purpose
$subject.
Fixes #31628
Fixes #31314
Fixes #28492
Approach
N/A
Samples
N/A
Remarks
N/A
Check List

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

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Will fix the build failure


Change "destination-subscriptionId" to "--destination-subscription-id"

Please fix the "If you want to move the VM and its resources to a different subscription, add the --destination-subscriptionId parameter to specify the destination subscription." part of the documentation changing the --destination-subscriptionId to the  correct optional parameter --destination-subscription-id
Reference: https://learn.microsoft.com/en-us/cli/azure/resource?view=azure-cli-latest#az-resource-move-optional-parameters

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

ID: 2f471928-7d94-6780-4387-1c279593859e
Version Independent ID: 6e522e47-58b1-a530-3f98-53c395dbb047
Content: Move a VM in using the Azure CLI - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/move-vm.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

@mekr
Thanks for your feedback! We will investigate and update as appropriate.

@mekr
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


imagesize

how to set imagesize bitmap ?
thank you

this is no way yet because it's calculated in the run time

this feature is available now 0.2.0


Add awesome and cool timer sounds...like tik tik tik....or tok tok tok like that

it would help a lot! and also add music

Timer tick sounds implemented in v0.10.0

OMG THANKS SO MUCH
On Tue, Jun 2, 2020 at 5:33 AM Christopher Murphy<EMAIL_ADDRESS>wrote:

Timer tick sounds implemented in v0.10.0
https://github.com/Splode/pomotroid/releases/tag/v0.10.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Splode/pomotroid/issues/80#issuecomment-637198307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSFAHW6I5Z4UNNP7NPLRURCD5ANCNFSM4MNMHIHQ
.


Add check to capture stats from unbound dns resolver

This check calls sudo unbound-control stats and parses the output.
r? @franklin-stripe @cory-stripe

👍

r? @franklin-stripe
Franklin, PTAL re: the sudo comment.  (I think I may have confused the ci bot w/ multiple reviewers...)

whoops sorry
LGTM

Thanks Franklin and Cory! I'm going to mark as approved and merge since I don't know what ci bot is doing.


⚠ ClamAV has degraded performance

In b57ca02, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 193 ms

Resolved: ClamAV performance has improved in 1c144dc.


Function arguments

Nice contribution @narodejesus
A couple of points:

Headers should be title cased (e.g. Function arguments -> Function Arguments)
I think it is important we justify the why of the rule. So in this case, there should be a small bit of introductory text that describes why we prefer this style 😄


collection-repeat, performance problem

Ionic version:
[x] 1.x (1.3.2)
[ ] 2.x
I'm submitting a ...
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
little jumps while scrolling, not smooth experience even with iPhone 7
Expected behavior:
smooth scroll
Steps to reproduce:
http://plnkr.co/edit/BlNR4M?p=preview
Other information:
i'm using ionic-cache-src plugin, maybe there is a problem, but with regular img+ng-src code it works even slower(in my opinion) than ionic-cache-src+background style property
Ionic info:
Cordova CLI: 6.4.0
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic CLI Version: 1.7.16
Ionic App Lib Version: 0.7.3
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.9.2
Xcode version: Xcode 8.2 Build version 8C38
i just wanted to know is it known issue, or i'm the only one experiencing such behavior

This issue was moved to driftyco/ionic-v1#44


Adding a FakeImageFolder

Hi all,
I've added a convenience module for creating a fake image folder in torch vision. I recently needed to create a mock folder and this can be a little painful without a FakeImage folder like this
Thanks
Craig

Thanks for the PR @ctr26 , but what is the benefit of FakeImageFolder over the existing FakeData class?

Thanks for the PR @ctr26 , but what is the benefit of FakeImageFolder over the existing FakeData class?

Thank you for your question regarding the benefit of `FakeImageFolder` over the existing `FakeData` class.

The key advantage lies in its compatibility with testing environments that utilise `ImageFolder` derived datasets (most of them). In my specific use case, it was challenging for me to  perform mock data tests on my MLOps pipeline, which relied on ImageFolder rather than Dataset. The existing `FakeData` class, although useful, does not support the kwargs expected by `ImageFolder`, leading to errors when used in this context and generally isn't a drop replacement.

The alternative being hacky solutions like downloading stock datasets like celeba as a mock, which seems excessive.

Thanks

Craig

Thanks for the details. IIUC you just need the signature of FakeData to be compatible with that of VisionDataset in order to avoid special-casing the call to FakeData?
Instead of creating a new class, would it be enough for your use-case to add *args, **kwargs to the signature of FakeData (both of which would be ignored/unsued)?

That would an  easier solution yes. My only concern there was that it would have unforeseen downstream effects. I've pushed the changes

Thanks for the feedback @ctr26 . I'm still not sure I completely understand why FakeData and VisionDataset need to have the same signature. But since this seems to be a very specific use-case (and not necessarily a broadly useful one), I might recommend for you to simply inherit from FakeData and accept **kwargs in your own subclass, like so:
class MyFakeData(FakeData):
    def __init__(self, ..., *kwargs):
        super().__init__(...)


Particle assignment

Updating Master With Game Loop Changes, again.  [Branch issue]

Updating Master with game loop changes.


Protect the WebUI with an authentication mechanism

It seems we can authenticate Allianz Certificates from outside

but if use a certificate authentication on ssl we should have a server certificate signed by the same ca, or not.  that would be a problem if we were using the standard mechanism. or not? If I am right on that (security always brings up the greatest uncertainties), then I think we need to accept some form of token signed by the client with its certificate and we must only validate that the token is valid using the public key (that i think we can store outside). I think this is a bit different.

Talked to Alexander, confirmed we can have client certificates and a web service is available to to validate the user is active.  He will help me with a getting a client certificate.  For development we need to create of course a "dummy" certificate.


fix(seed): updated basic javascript URLs for hints

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[ ] Tested changes locally.
[x] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #17416
Description

@bogadodiegoh, thank you very much for submitting a PR for this. However, it seems like there's a slight issue with the URL you updated. Please take a look at my review.

@scissorsneedfoodtoo see https://github.com/freeCodeCamp/freeCodeCamp/pull/17471#issuecomment-395914805

Updated broken links with review suggestions.

@bogadodiegoh, thank you for making those changes! Everything looks great.
@raisedadead, this PR is ready to merge.

@bogadodiegoh, thank you for your contribution! :+1: :+1: :+1: We all look forward to your next PR!
@raisedadead, this one has been merged.


Minor documentation improvements

restructured administrator manual to be split into two sections (Themis App & ThemisML)
more consistency about the name "ThemisML" (no space, usually no "server" except where explicitly meant)
fix Sphinx build warnings (broken links, wrong indentations, too few headline underline-characters)
more consistent capitalization of headings

I hope I got the part about splitting the Administrator Manual right, I only took a quick note in the meeting today

That double headline looks a bit weird. Same for Themis ML.

I have removed the unnecessary pipes.

That double headline looks a bit weird. Same for Themis ML.

Thanks for the note! It's fixed now 👍


Switch to GH Actions from Travis

Removed centos7 and centos8 from test matrix. These were already not running Travis.

I opened https://github.com/theforeman/puppet-dhcp/pull/180 which includes all changes from https://github.com/theforeman/foreman-installer-modulesync/pull/106. Also has a workaround for EL7 testing.


Nixify the hello-world project

My development machine is nixos. To make the project buildable/runable I nixified the project. This was a straightforward procedure.
Ask:
Are you interested in PR to nixify the project?
The scope of change will be adding three files:

flake.nix
flake.lock
rust-toolchain.toml
And modifying the README for the nix related instruction

PR #86  is for this issue


[Snyk] Upgrade html-snapshots from 2.11.1 to 2.12.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.11.1 to 2.12.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 days ago, on 2023-11-03.

Release notes

Package name: html-snapshots

2.12.0 -<EMAIL_ADDRESS>

2.11.1 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

1a2fb83 Merge pull request #543 from localnerve/rc-2.12.0
5cfdde2 @ 2.12.0 -<EMAIL_ADDRESS>f0512e6 Merge pull request #542 from localnerve/snyk-upgrade-9155cf1ac6fccba6df1260a010105c6c
14b473e fix: upgrade html-snapshots from 2.11.0 to 2.11.1
752329e Merge pull request #541 from localnerve/snyk-upgrade-9f39da4721bfa0d0b3359e07398d4fd4
e9b1364 fix: upgrade html-snapshots from 2.10.5 to 2.11.1
45f97da Merge pull request #540 from localnerve/snyk-upgrade-bc368eb02e265c6b01965160e03a33b4
e3dc42d fix: upgrade html-snapshots from 2.11.0 to 2.11.1
0d663fc Merge pull request #539 from localnerve/snyk-upgrade-17a458159e2d0faa00b9f40b68588f00
2288274 fix: upgrade html-snapshots from 2.11.0 to 2.11.1
e6dbb2e Merge pull request #538 from localnerve/snyk-upgrade-e694349d7ed1f182b625e2ee7e8d1cc4
bf21fc7 fix: upgrade html-snapshots from 2.11.0 to 2.11.1
75e35b8 Merge pull request #537 from localnerve/snyk-upgrade-1898ae0c35877f27161f5fbd02d224c8
edd1bac fix: upgrade html-snapshots from 2.10.5 to 2.11.1
f09ed08 Merge pull request #536 from localnerve/snyk-upgrade-d5c15c6af27789a4fd61274a6f95f6e0
00a85f6 fix: upgrade html-snapshots from 2.11.0 to 2.11.1
3e9a03b Merge pull request #535 from localnerve/snyk-upgrade-ba321f5e82f9748f69777b0bdd6df687
a196e7b fix: upgrade html-snapshots from 2.10.5 to 2.11.0
f3727ea Merge pull request #534 from localnerve/snyk-upgrade-1545e80a448438ed6ff9df6b92450e09
10c3213 fix: upgrade html-snapshots from 2.10.5 to 2.11.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same
when pulling 7121faa1f41a5be3a3d8161e51b17b5c9506087d on snyk-upgrade-095f19ecc873887dec25fd8869f97976
into 13fc5651fcf1b49ff95c679703e39287e9aeeb5e on master.


Invalid value for  attribute points...

It seems the newest version 0.4.1 doesn't allow to animate a polygon's "points" attribute.
var s = Snap(400, 620);
var pol = s.polygon([0, 464, 0, 464, 111.6, 464, 282.5, 464, 457.4, 464, 613.4, 464, 762.3, 464, 912.3, 464, 1440.1, 464, 0, 464]);
pol.animate(
  { points: [0, 464, 0, 367, 111.6, 323.3, 282.5, 373, 457.4, 423.8, 613.4, 464, 762.3, 464, 912.3, 464, 1440.1, 464, 0, 464] },
  1000)

It errors with 'Error: Invalid value for  attribute points="0"'
I got it working with version 0.3.0. So some change must have broken this behavior

The fix you provided works! Thanks!

Yep, Thank you https://github.com/ThomasBrierley

+1. Thanks
Any chance related PR will be accepted?

Bump. Please either accept the PR or implement your own fix.

any updates?

i am waiting for this update too

+1 the struggle is real

Btw, I was just answering a question on this error on Stackoverflow which I think is probably related http://stackoverflow.com/questions/34991254/animate-polygon-points-snap-svg , you can get around it I think just by using a default Animation, rather than an element animation. So it would look something like this....
Snap.animate( myStartingPointsArray, myEndingPointsArray,
    function( val ){
      myPolygon.attr({ points: val })
    },
  2000)

or a plugin....
Snap.plugin( function( Snap, Element, Paper, global ) {
   Element.prototype.polyAnimate = function( destPoints, duration, easing, callback ) {
   var poly = this;
    Snap.animate( this.attr('points'), destPoints,
       function( val ){ poly.attr({ points: val }) }, duration, easing, callback)
    };
});

Another forward compatible workaround is to use v0.3.0 until this fix is released (provided you are not depending on any features in ~v0.4.0). As both #373 and #378 are regressions that were introduced between v0.3.0 and v0.4.0
I've been locking all my work to this version since last may without any issues other than the memory leak.

Just thinking, as I don't know git well...
Would it make sense to fork Snap into a patched version just of important updates (eg a temp 0.4.1 if you like) until Dmitry gets chance to include important fixes. Main ones I'm aware of are this, and the memory fix.
Not sure if this is feasible (or against t&c's?) or advisable as not so familiar with git or if it would get really messy, but I'd try and help if possible.

It's not so much a question of if it's possible: we can fork/modify because it's a public git repo and are legally free to because it's licensed with Apache 2.0. The problem is any forks will not be recognised as "official" (think npm and bower registries) which may defeat the purpose of making a patched fork.
Anyone can fork and merge all the un-merged PRs and build it if they want, the problem with suggesting that others use it is the obligation for that fork to be maintained in the future (even if that just means don't delete it from your account). For instance this build is a collection of my patches on my fork:
https://github.com/ThomasBrierley/Snap.svg/releases/tag/memory-leak-fix
However this is for my own purposes and I intend to delete the entire fork as soon as the official Snap repo integrates it... if anyone points their packages.json or whatever to that repo then it will break in the future. That said, a persistent community fork under a different name will inevitably happen if necessary... time waits for no man and all that.

Sorry to reply to my own post, but just to clarify:
Feel free to fetch/fork/clone my link above and include it in your own project or host it yourself. My point in response to @ibrierley is that it should be made clear to anyone using these temporary types of builds (by me or anyone else) not to use them with package managers like npm/bower etc to pull them in... I should have just said that instead... :no_mouth:

Yes of course, hadn't even thought about that angle, thanks.

The issue still hasn't been fixed in the master branch- but I downloaded your update linked above and it fixed my problem with animating polylines. Thank you!

One more in favor of accepting this PR. This is one of the features that sets snap apart from other svg libraries... but it's broken out of the box.

been learning snapsvg and just spent 2 hours banging my head on the wall with this issue to only uncover it from a stackoverflow comment all the way at the bottom.  -- http://stackoverflow.com/questions/29732802/animating-points-in-a-polygon-with-snap-svg?rq=1
can we escalate this pr merge, truly pain for those coming to svg and trying to learn and out of the box this is an issue.

+1 for fixing this; the bug is annoying and I just found this thread by accident after wasting much time trying to find out why it was not working.

This ever getting fixed?

Hi, first of all: thanks for this great library.
Bu as mentioned by several people in the past year and a half: this issue is breaking an important feature of snapsvg. A pull request was filed, this shouldn't take long to revise and merge.
A polite callout to the maintainers of this valuable library: could you please review? Thanks a lot!
@DmitryBaranovskiy

+1
I need this aswell.

Should be ok to close this now, didn't autoclose for some reason.
Fix has been merged and was well tested before.


datamigration readme t2 config

changed rdatamigration/readme.md and readme.python.md to track2

Need to use<EMAIL_ADDRESS>to fix python track2 pipeline.

/azp run


armcc.exe - Unhandled function EnumSystemLocalesA (KERNEL32.dll)

Passing the --locale=XXXX flag to armcc.exe results in the titular error.
It feels like it will be a lot of work to properly implement locales, so perhaps we can add a reasonable hack (and emit a warning that locales are not really supported)?

Apologies if this is known already, but for the sake of documentation - it seems like some versions (armcc_41_894, armcc_41_1049, and armcc_41_713) still aren't happy. All the failures look something like:
~/projects/wibo-clean/wibo/build/wibo armcc.exe --locale=japanese test.c
"no source": Error:  #869: could not set locale "japanese" to allow processing of multibyte characters
test.c: 0 warnings, 1 error

I think it's in the discord, but good to have that comment here too. You see absolutely correct that locales are not supported - but now you get the error message you saw, rather than "Unhandled function EnumSystemLocalesA". I don't know how the effort/reward chart looks like if we wanted to properly support locales?


all_there? does not use optional implicit waits

Presently, all_there? is not implemented to use the new capybara implicit waits.

Closing in favour of PR/s


Documentation needed

Need to make documentation for this repo.

Required guideline docs:

[ ] README doc
[ ] CONTRIBUTING doc


[review] Wconversion enabler

taking over #272.

[ ]   File: parameter/IntegerParameterType.cpp:L386-413
 I am not fan of this 0x prefixing all bytes.
 Ignore my previous comment.
[x]   File: test/test-subsystem/TESTSubsystemBinary.cpp:L61-74
 That is dangerous, the PF is not exception safe. Where is the matching catch block ?
 I agree - but this is the test subsystem.
 True, but you should not throw a string, throw a std::runtime_error
[x]   File: parameter/LogarithmicParameterAdaptation.cpp:L78-88
 - WTF ? fromUserValue returns a uint64_t , why then casting it to double to round it ?
fmap => max<int64_t> ?

@krocard Please review

This patch could be splited in 3:
rename of fromXml to structureFromXml
add a CParameterAccessContext in XmlParameterSerializerContext
Add set and get ElementXML commands


RuntimeError: Exception thrown in SimpleITK ImageFileWriter_Execute: /tmp/SimpleITK-build/ITK/Modules/IO/GDCM/src/itkGDCMImageIO.cxx:1392: ITK ERROR: GDCMImageIO(0x27016f0): DICOM does not support this component type

When I run the python example of how to read and write DICOM files, it reads the DICOM serie, prints the size, but can't write a dcm file.
My code is just like the example:
reader = sitk.ImageSeriesReader()
dicom_names = reader.GetGDCMSeriesFileNames(originalPath)
reader.SetFileNames(dicom_names)
image = reader.Execute()
size = image.GetSize()
print("Image size:", size[0], size[1], size[2], image.GetPixelIDTypeAsString())

sitk.WriteImage(image, savepath)

The ERROR message is the following:
RuntimeError: Exception thrown in SimpleITK ImageFileWriter_Execute: /tmp/SimpleITK-build/ITK/Modules/IO/GDCM/src/itkGDCMImageIO.cxx:1392:
ITK ERROR: GDCMImageIO(0x27016f0): DICOM does not support this component type
The originalPath is the dcm serie path, the savepath is  './result.dcm'
The printed output is:
Image size: 512 512 244 16-bit signed integer
Unfortunately for privacy reasons I can't share the dicom file.

Hello @rajkifranciska,
The 3D image should be saved as multiple 2D slices. Please see this example which illustrates how to read, modify and write the final result as a set of DICOM images.


[de] Change footer l10n from Home to Documentation

Addresses German l10n strings described in #43068

/lgtm
/approve


Unable to use project dependency accessors if a project has spaces in its name

Expected Behavior
Project dependency accessor is generated with backticks so spaces are supported, just like in other parts of Gradle:
Project with name "Stuff Some Name" leads to `Stuff Some Name` generated as its accessor symbol.
Current Behavior
Failure:
Caused by: org.gradle.api.InvalidUserDataException: Cannot generate project dependency accessors:
  - Cannot generate project dependency accessors because project 'Stuff Some Name' doesn't follow the naming convention: [a-zA-Z]([A-Za-z0-9\-_])*
	at org.gradle.api.internal.catalog.DefaultDependenciesAccessors.assertCanGenerateAccessors(DefaultDependenciesAccessors.java:191)
	at org.gradle.api.internal.catalog.DefaultDependenciesAccessors.writeProjectAccessors(DefaultDependenciesAccessors.java:145)
	at org.gradle.api.internal.catalog.DefaultDependenciesAccessors.generateAccessors(DefaultDependenciesAccessors.java:133)
	```

Getting the same issue here ... seems no other solution but to rip out the project.

See also (for SEO) https://github.com/gradle/gradle/issues/18971


Unable to close Add Dev Container Configuration Files quickpick using mouse

Run the "Add Dev Container Configuration Files..." command. A quickpick shows up.
Click on another part of the editor outside of the quickpick.
:bug: The quickpick doesn't close.

This is up to the owner of "Add Dev Container Configuration Files..." to decide if they want this behavior or not. It's controlled with ignoreFocusOut property in QuickPick API.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


cmd/fmt: comma appended to comment instead of list element

What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.1 linux/amd64

What did you do?
I ran cue fmt on a file with a list of inline structs that ends in a comment rather than a value.
bug.cue:
l: [
	{},
	// comment
]

$ cue fmt bug.cue && cat bug.cue

What did you expect to see?
The input file should not have been changed, as it is correctly formatted.
What did you see instead?
l: [
	{}
	// comment,
]

If I format that file again, it appends another trailing comma:
l: [
	{}
	// comment,,
]

I get a new comma each time unless I add a value after the comment.
I'm very new to CUE, but I think this might be related to #1447 based on how both issues seem to interpret the comment as a list element.

Also still exists on v0.4.3 darwin/arm64

Note that this issue is extremely similar to https://github.com/cue-lang/cue/issues/2274, although that issue has an example where fmt's output CUE is invalid.

As of cue v0.6.0 I cannot reproduce this bug
# issue #1478 is not reproducible as of cue v0.6.0---------
exec cat 1478G.cue
exec cue fmt 1478G.cue    ### fmt works
exec cat 1478G.cue
exec cue vet 1478G.cue    ### vet passes
cmp expected/1478G.cue 1478G.cue

-- 1478G.cue --
l: [
	{},
	// comment
]
-- expected/1478G.cue --
l: [
	{},
	// comment
]

Thanks @rudifa! I can confirm the issue appears solved for me as well.


Use String() method when possible on structs?

Is it possible to have ffjson use the String() method on objects that have it rather than encoding it? I have a net.IP object, and want to have Marshal create a string instead of base64 encoding the bytes.
Is that possible?

Well, overall we want to preserve the same behaviors as the stdlib encoding/json whenever possible, so the "right' way to do this is to type-alias net.IP and add a MarshalJSON / UnmarshalJSON specialization for it.....

I also cannot see how we can make that work reliably. As Paul writes, the best solution would be to
To make it as fast as possible, you can of course implement your own marshal:
type MyIP net.IP

func (mj *MyIP) MarshalJSONBuf(buf fflib.EncodingBuffer) error {
    buf.WriteString(`"`+mj.String()+`"`)
    return nil
}
``


Gmsh IO pirnt and distinction between elements and faces

Now a finner mesh is required to validate the analytical solution:

Also the mesh is defined using Gmsh.jl

Codecov Report

Merging #217 (e20babe) into main (0ba6c98) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
- Coverage   90.41%   90.31%   -0.10%
==========================================
  Files          40       40
  Lines        1095     1095
==========================================
- Hits          990      989       -1
- Misses        105      106       +1

Impacted Files
Coverage Δ

src/Meshes/Gmsh.jl
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


enclave-tls: design policy check callback

use OPA  by DSL. Intergated with Attester and verifier instance


Add Claude and Google models into benchmark

Thanks a lot a such uniq benchmark! Could you please add great models from Claude to the benchmark: Haiku, Sonnet and Opus?
And, Google Gemini Pro and Gemini Flash also.

Thanks for your interest in our work! Currently we had time and resources to evaluate only some of the main models in the field. We'll try to extend the model selection in the future, and any help from the community is highly appreciated! New models can be added to the leaderboard via pull request of evaluation results to this repository.

Yes, thanks so much for all your hard work. I just wanted to mention that the Claude and Google models are looking great at the moment. And the new Llama 405b is pretty impressive too. When you get a chance, I'd love to see your benchmarks for all of them. Thanks again!

We have just added results for LLama-3.1-Instruct (8B and 70B). LLama-3.1 70B shows very strong results on long contexts (32k+), even outperforming GPT-4.

It's great! Thank you.
May I ask what prevents you from adding Claude, Gemini and llama 405b to the benchmark?
If you wish, I can assist with providing an OpenRouter api key for that purpose. You can contact me via telegram: https://t.me/rodion_m_tg
@yurakuratov

We have evaluated Gemini 1.5 Pro 002 on BABILong tasks:

It is currently the strongest LLM we have evaluated.
These results are included in updated paper.


Prepare release 30.1.1

This PR is to prepare a new minor release
Signed-off-by: Philippe Ombredanne<EMAIL_ADDRESS>

All green... merging.


Add APIs for system settings

Some system settings are fixed at specific values. It is necessary to make these APIs so that the settings can be changed from the upper layer.
 settings.AddMember("alwaysUse24HourFormat", true, allocator);
 settings.AddMember("textScaleFactor", 1.0, allocator);
 settings.AddMember("platformBrightness", "light", allocator);

See: https://github.com/sony/flutter-embedded-linux/blob/master/src/flutter/shell/platform/linux_embedded/flutter_linuxes_engine.cc#L342

platformBrightness parameter takes light or dark. But the current Flutter framework or engine seems to always work with light mode. This is a bug of Flutter?


Adding Hundre University

@cngo1612 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host hundre.edu.vn not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Join forces with magic_areas?

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Hey, this looks great but also closely resembles magic_areas integration. Maybe you can combine forces?
https://github.com/jseidl/hass-magic_areas
Describe the solution you'd like
Have 1 integration that is better than choosing between 2
Describe alternatives you've considered
See above
Additional context
None needed

Hey!
I am aware of magic_areas. In fact, I was a user of this component for quite some time myself and I think it's a great project.
However, at some point I ran into issues and I had the feeling there were too many features which I was not using anyway. As a consequence I chose to build my own custom component - specifically suited for my use cases with as little code and complexity as possible.
Is there some feature you are missing from either auto_areas or magic_areas or what other benefits do you see in merging the projects?

Thanks for the info! Currently I'm pretty happy with magic_areas, just saw your project pop up on hacs and didn't know you started it specifically after having tried magic_areas.


Update to TopoJSON 2.0!

I'm curious why you decided to replace the Makefile with a bash script here.

It felt simpler, and I didn’t feel like it was important to retain the automatic partial rebuilds afforded by make, especially when you can just check to see if a file exists. But if you prefer make, you should keep using it!


Request update to: "How it Works – Visual Guide"

"How it Works – Visual Guide" explains Assistants interaction really well, A request is to add the below,

Code Interpreter's interaction with AI Service
Add the interaction with Open AI SDK/Service
There is no interaction with AI Service for tool switching - is this correct ?

I updated! Let me know if it's clearer and answer those questions

Looks good, thanks!

Diagram updated


[WIP] Add protected properties to session

@calebporzio: The props are grouped into a single write/read call, right?


songs.py: Allow links with <> to be matched

This PR allows for links containing <> to be still be detected by the regex expression used in the songs cog. This allows users to skip embedding a link while still allowing Bloo to give information about the song link sent.

Closing, as superseded by GIRRewrite#9.


Multiple onFlutterError calls

On hot reload, onFlutterError is written to FlutterError.onError along with oldCallback. Because of this, every hot reload, another onFlutterError is added to FlutterError.onError and its call occurs several times with one error.
Reproduced in the project example
When commenting on this line, everything is corrected, but oldCallback is not taken into account
class _App extends StatelessWidget {
  const _App({
    Key? key,
    required this.widgetConfig,
  }) : super(key: key);

  final WidgetConfiguration widgetConfig;

  @override
  Widget build(BuildContext context) {
    return ReloadableWidget(
      builder: (BuildContext context) {
        _flutterErrorSetup();
        ErrorWidget.builder = (FlutterErrorDetails errorDetails) => ErrorHandlerWidget(
              errorDetails: errorDetails,
              releaseErrorBuilder: widgetConfig.releaseErrorBuilder,
              errorBuilder: widgetConfig.errorBuilder,
            );

        return widgetConfig.child;
      },
    );
  }

  void _flutterErrorSetup() {
    final FlutterExceptionHandler? oldCallback = FlutterError.onError;

    FlutterError.onError = (FlutterErrorDetails errorDetails) {
      // oldCallback?.call(errorDetails);   <-- This line
      widgetConfig.onFlutterError(errorDetails);
    };
  }
}

$ flutter --version
Flutter 3.10.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 796c8ef792 (4 weeks ago) • 2023-06-13 15:51:02 -0700
Engine • revision 45f6e00911
Tools • Dart 3.0.5 • DevTools 2.23.1

@KolasikOmetov Could you watch this?

This issue was fixed in 2.2.2 version.


Feature request: complete behaviors/mixins

@jay8t6 reported at https://github.com/Polymer/atom-plugin/issues/34

It would be nice if the editor plugins auto populates the mixins available based on the html imports.

For Polymer 1.0 we'd just need to notice that you're in a behaviors section of a Polymer declaration.
For Polymer 2.0 we could instead let a general purpose code completion system like tern or typescript handle this, we'd just need to handle explaining the html imports graph to it.

Re-opening because this original issue is now transferred and canonical again.


Failed to load signer "signer #1"

StackTrace:
Verification succesful
/bin/cp app/build/outputs/apk/release/app-release-unsigned.apk app/build/outputs/apk/release/app-release-unsigned-aligned.apk
/usr/local/lib/android/sdk/build-tools/29.0.3/apksigner sign --ks app/build/outputs/apk/release/signingKey.jks --ks-key-alias *** --ks-pass pass: --out app/build/outputs/apk/release/app-release-unsigned-signed.apk app/build/outputs/apk/release/app-release-unsigned-aligned.apk
Failed to load signer "signer #1"
java.io.EOFException
	at java.io.DataInputStream.readInt(DataInputStream.java:392)
	at sun.security.provider.JavaKeyStore.engineLoad(JavaKeyStore.java:661)
	at sun.security.provider.JavaKeyStore$JKS.engineLoad(JavaKeyStore.java:57)
	at sun.security.provider.KeyStoreDelegator.engineLoad(KeyStoreDelegator.java:224)
	at sun.security.provider.JavaKeyStore$DualFormatJKS.engineLoad(JavaKeyStore.java:71)
	at java.security.KeyStore.load(KeyStore.java:1445)
	at com.android.apksigner.SignerParams.loadKeyStoreFromFile(SignerParams.java:353)
	at com.android.apksigner.SignerParams.loadPrivateKeyAndCertsFromKeyStore(SignerParams.java:239)
	at com.android.apksigner.SignerParams.loadPrivateKeyAndCerts(SignerParams.java:181)
	at com.android.apksigner.ApkSignerTool.sign(ApkSignerTool.java:277)
	at com.android.apksigner.ApkSignerTool.main(ApkSignerTool.java:83)
Error: The process '/usr/local/lib/android/sdk/build-tools/29.0.3/apksigner' failed with exit code 2

I am having similar issue. No luck so far!

I'm also facing this issue.

+1

@me-at-syeds-me @soethan98 Any luck ?

any updates on this?

Hello, I've tested this action to sign my apk. This issue is caused when wrong "signingKeyBase64" is provided.
Solution:
run in your terminal: openssl base64 -in your_keystore.keystore -out key.txt and then copy the output content to your signingKeyBase64 secret.
Hope it can help you

hm. ordinary base64 turns my keystore into 3kb file, openssl base64 turns it into 4kb. perhaps that is why some people have this error and others do not? (i have it at the moment. verifying whether it works with openssl base64)

I had the same error message when I misspelled "SIGNING_KEY" in my secrets 🤦♂

I have the same error. What is the solution?

For me it was me creating the wrong gitlab variables

Any update on this issue?

Hello, I've tested this action to sign my apk. This issue is caused when wrong "signingKeyBase64" is provided. Solution: run in your terminal: openssl base64 -in your_keystore.keystore -out key.txt and then copy the output content to your signingKeyBase64 secret. Hope it can help you

@riv0manana did you use key.txt as is or deleted newlines from it?

Using the below command worked for me, it printed the key in console and i copied that. openssl base64 -in your_keystore.keystore

I tried the same. But still the issue persist for me. I encode the key-store file in mac, but i am using windows-2019 as runner.
Failed to load signer "signer #1"
java.io.IOException: Invalid keystore format
at sun.security.provider.JavaKeyStore.engineLoad(JavaKeyStore.java:666)
at sun.security.provider.JavaKeyStore$JKS.engineLoad(JavaKeyStore.java:57)
at sun.security.provider.KeyStoreDelegator.engineLoad(KeyStoreDelegator.java:224)
at sun.security.provider.JavaKeyStore$DualFormatJKS.engineLoad(JavaKeyStore.java:71)
at java.security.KeyStore.load(KeyStore.java:1445)
at com.android.apksigner.SignerParams.loadKeyStoreFromFile(SignerParams.java:353)
at com.android.apksigner.SignerParams.loadPrivateKeyAndCertsFromKeyStore(SignerParams.java:239)
at com.android.apksigner.SignerParams.loadPrivateKeyAndCerts(SignerParams.java:181)
at com.android.apksigner.ApkSignerTool.sign(ApkSignerTool.java:277)
at com.android.apksigner.ApkSignerTool.main(ApkSignerTool.java:83)
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Xamarin\Android\Xamarin.Android.Common.targets(2293,2): error MSB6006: "java.exe" exited with code 2.

CR/LF issue?

Your password, alias, or keyword is incorrect!
Just double-check!

Your password, alias, or keyword is incorrect! Just double-check!

I have tried to build through Android Studio successfully with the same key, password and alias on the Github secret but when in github actions errors like this
Failed to load signer "signer #1"
java.io.IOException: Tag number over 30 is not supported
	at java.base/sun.security.util.DerValue.<init>(DerValue.java:442)
	at java.base/sun.security.util.DerValue.<init>(DerValue.java:487)
	at java.base/sun.security.pkcs12.PKCS12KeyStore.engineLoad(PKCS12KeyStore.java:2012)
	at java.base/sun.security.util.KeyStoreDelegator.engineLoad(KeyStoreDelegator.java:221)
	at java.base/java.security.KeyStore.load(KeyStore.java:1473)
	at com.android.apksigner.SignerParams.loadKeyStoreFromFile(SignerParams.java:383)
	at com.android.apksigner.SignerParams.loadPrivateKeyAndCertsFromKeyStore(SignerParams.java:269)
	at com.android.apksigner.SignerParams.loadPrivateKeyAndCerts(SignerParams.java:202)
	at com.android.apksigner.ApkSignerTool.getSignerConfig(ApkSignerTool.java:438)
	at com.android.apksigner.ApkSignerTool.sign(ApkSignerTool.java:353)
	at com.android.apksigner.ApkSignerTool.main(ApkSignerTool.java:92)
Error: The process '/usr/local/lib/android/sdk/build-tools/34.0.0/apksigner' failed with exit code 2

Hello, I've tested this action to sign my apk. This issue is caused when wrong "signingKeyBase64" is provided. Solution: run in your terminal: openssl base64 -in your_keystore.keystore -out key.txt and then copy the output content to your signingKeyBase64 secret. Hope it can help you

@riv0manana did you use key.txt as is or deleted newlines from it?

I am not sure but I think delelting the newlines worked for me


No longer able to rename vehicle

Describe the bug
After the latest upgrade the integration no longer respects the names set in the yaml config and there is no config flow rename method.
Latest working release
Not sure, but it did work shortly after the config flow introduction.
Debug logs
Unfortunately no debug logs enable yet (will post later), but I do see a lot of 403 codes in the core log
21-09-02 08:20:28 WARNING (MainThread) [skodaconnect.connection] Something went wrong when refreshing VW-Group API tokens. <!doctype html><title>HTTP Status 403 – Forbidden</title>
HTTP Status 403 – Forbidden

Type Status Report

Message Access Denied

Description The server understood the request but refuses to authorize it.

Apache Tomcat

Installation:

Home Assistant variant: HASS OS in VM
Python version [e.g. 3.7, 3.8] 3.7
Python library versions ["python3 -m pip list"]: gives error about distutils.cmd when run from VS Code terminal

Additional context
N/A

While that is good news on it's own, I'm afraid that mechanism doesn't seem to be working.

While that is good news on it's own, I'm afraid that mechanism doesn't seem to be working.

Strange, works a treat for me.

Important: press rename here.

If you for some reason don't press "rename", it won't rename entities. Then you will need to rename them manually.

I never even got that question about the rename. Because it already thinks it has the correct name. I might have to try adding another space or something. Op do 2 sep. 2021 om 14:07 schreef Kim @.***>:
…
While that is good news on it's own, I'm afraid that mechanism doesn't seem to be working. Strange, works a treat for me. [image: image] https://user-images.githubusercontent.com/1235384/131840488-e32b369a-7edf-44ea-8579-faab37ede786.png [image: image] https://user-images.githubusercontent.com/1235384/131840521-7ad32034-f334-4ddf-a8d1-e2ed0ab277e4.png [image: image] https://user-images.githubusercontent.com/1235384/131840557-2d55cb99-40df-4603-94d3-3eddd5de70d8.png Important: press rename here. [image: image] https://user-images.githubusercontent.com/1235384/131840605-14e7424c-e1b7-44ba-98c6-f1f1323f51f9.png If you for some reason don't press "rename", it won't rename entities. Then you will need to rename them manually. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#65 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABERJS2WU46KVA76G4JPSLT75SJXANCNFSM5DIOLWWA .

If I were you I would remove and add the integration again.

Well this is getting more odd by the minute. I readded the integration and it picked up the correct name without me doing anything but this time it also put the name on all the entities.

Well this is getting more odd by the minute. I readded the integration and it picked up the correct name without me doing anything but this time it also put the name on all the entities.

Probably something hindering the integration from changing the name. Happens sometimes when there's old data in HA.

I'm just happy I don't have to redo an entire tab on my dashboard. Case closed as far as I'm concerned!


Packages and version

To make the development environment a little more stable it would be helpful to set up the various projects to build and deploy artifacts. These would be versioned and added as dependencies to our work rather than directly linked.
This should help decouple the changes coming in from upstream.
We can fork the main opensha project to make the changes and then discuss with Kevin to see if he is interested in importing the changes.

@ornearius I'd like to break this down into specific objectives so that I can prioritise them. I think it goes like this...

[ ] modify gradle configuration(s) to ensure that a single jar file containing all the required components to run the HikurangiDemo applications.
[ ] get the required runtime installed on beavan (ticket) already exists  - see existing ticket
[ ] run a single worker node demo on beavan

These fall into the short-term goals for our 1st milestone.
After this we can look at something like:

[ ] create a build pipeline
[ ] publish build artefacts


Stack setup script template validations fail

Mailer Sender validation fails to approve valid email addresses.

Must be <Some Name><EMAIL_ADDRESS>

Added better validations and examples to the setup script values.


Intercept feign URL without parameters

…out parameters
Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[+] New feature provided

[ ] Improve performance

Related issues
#3850

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.
intercept  feign.ReflectiveFeign.BuildTemplateByResolvingArgs#resolve(Object[], RequestTemplate, Map)  get url without params before resolve ,put it into ThreadLocal ,get the url in Feign DefaultHttpClientInterceptor

Here is the screenshots

Let's wait for #3838 merged, then, you need to add this case into the test.

FYI @aderm

CI test failed, please check.

You still get the code style issue. You should check locally first.

/run agent-plugin-test-3

Feign test #3838 has been merged. Please add this into the test.

You could find how to run and test through this doc
https://github.com/apache/skywalking/blob/master/docs/en/guides/Plugin-test.md

@dominicqi Please make sure the feign-scenario passed, and include your new feature in.

i found the problem, RequestTemplate#url miss the url part in the http://localhost:8080/feign-scenario,i will try to fix this

/run agent-plugin-test-3

/run agent-plugin-test-2


Adding Azure SignalR provisioning and a showcase to playground

Fixes https://github.com/dotnet/aspire/issues/799

Add the provisioning part of Azure SignalR into aspire
Azure SignalR SDK will support reading connection name with its AddNamedAzureSignalR(string connectionName), the API naming could be changed if there are any better options
Add a signalr playground sample following SignalR quick start tutorial https://learn.microsoft.com/aspnet/core/tutorials/signalr?view=aspnetcore-8.0&tabs=visual-studio-code#add-signalr-client-code with minimum change: adding  AddNamedAzureSignalR("signalr")
Add bicep for signalr
** Sample provisioning output **

** Sample SignalR web server**

Microsoft Reviewers: Open in CodeFlow

@vicancy Can you add a new playground app for the SignalR service sample instead of modifying orleans?

Hi @eerhardt could you help add https://www.myget.org/feed/azure-signalr-dev/package/nuget/Microsoft.Azure.SignalR/1.25.0-preview1-11147 to the nuget source? This is the preview package containing the newly added API AddAzureSignalRWithConnectionName. Also could you share what the process is when the dependent package version is updated? Thanks!

Hi @eerhardt could you help add https://www.myget.org/feed/azure-signalr-dev/package/nuget/Microsoft.Azure.SignalR/1.25.0-preview1-11147 to the nuget source?

I don't believe we can mirror a package from myget.org. Can we use a version that is on nuget.org?

Also could you share what the process is when the dependent package version is updated?

Checkout https://github.com/dotnet/arcade/blob/main/Documentation/MirroringPackages.md

If the package is owned by Microsoft, new versions (or versions not originally mirrored), will be automatically updated the next time the pipeline runs on its schedule.

Hi @eerhardt could you help add https://www.myget.org/feed/azure-signalr-dev/package/nuget/Microsoft.Azure.SignalR/1.25.0-preview1-11147 to the nuget source?

I don't believe we can mirror a package from myget.org. Can we use a version that is on nuget.org?

Also could you share what the process is when the dependent package version is updated?

Checkout https://github.com/dotnet/arcade/blob/main/Documentation/MirroringPackages.md

If the package is owned by Microsoft, new versions (or versions not originally mirrored), will be automatically updated the next time the pipeline runs on its schedule.

Thanks for the explanation. Updated to nuget: https://www.nuget.org/packages/Microsoft.Azure.SignalR/1.25.0-preview1-11147

@vicancy we're swapping azure resources over to the new bicep model. This means you don't need to update azd to enable deploying it.

Hi @eerhardt also help add Microsoft.Azure.SignalR.Protocols package please

Hi @eerhardt also help add Microsoft.Azure.SignalR.Protocols package please

I've mirrored https://github.com/dotnet/aspire/pull/1873#issuecomment-1955956313 to the dotnet-public feed. It will also mirror all its dependencies as well.

Thanks for the help @eerhardt , we released a stable version 1.25.0, hopefully the pipeline will automatically mirror the package right?
The PR is ready for review, @eerhardt @davidfowl

Can the playground sample be simplified? It seems like a lot of css and js files being checked in for little value. I know this came from a template, but it just feels like more than what is necessary.

@eerhardt Thanks for the feedback, makes total sense, updated.

Looks really good!

@eerhardt This change looks really good, I'm willing to deal with any fallout for P4 (can merge tomorrow).


removeFingerprintOnSuccess does not work with uploadDataDuringCreation

I've set uploadDataDuringCreation to true in my application. However, even with removeFingerprintOnSuccess set to true as well, the local storage is not being cleared after an upload. I didn't dive deep into the code, but it seems somehow related to _urlStorageKey being null, hence _removeFromUrlStorage() doesn't do anything.

Thanks for reporting. This should be fixed in the next release (hopefully until the end of this week).


Fix bad circle_ci yml

Apparently docker login on Circle CI requires the -e option...

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "circle_ci2" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354352208
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.


Fix for role cluster_wait_for_alert

The role was using a deprecated oc command:
oc sa new-token
This resulted in a non 0 RC.
This PR changes to the new oc create token command instead.

/hold

/test ?

/lgtm

/lgtm

/unhold


Pluck ignoring results of first

Laravel Version: 5.8.17
PHP Version: 7.1.26
Database Driver & Version: MySQL

Description:
Calling pluck() on first() result ignores query and performs pluck() on whole DB table.
Steps To Reproduce:

Create DB table with multiple rows
Get record from table using first()
Perform pluck() on result of query

Heya, thanks for reporting.
I'll need more info and/or code to debug this further. Please post relevant code like models, jobs, commands, notifications, events, listeners, controller methods, routes, etc. You may use https://paste.laravel.io to post larger snippets or just reply with shorter code snippets.
Thanks!

@driesvints They mean something like User::first()->pluck('id').
@DrRoach This is the correct behavior. The Model class doesn't have a pluck() method, so it instantiates and executes a new query. It's equivalent to User::pluck('id') or (new User)->pluck('id').
The same applies to other methods like count() or exists().

@staudenmeir you're right.

This has caused some issues for us.
Is it possible for there to be something that would trigger an exception if you try to pluck on a model because the current behaviour is not intuitive
Or perhaps, the model pluck method, should take the place of the get and return just the single property that is asked for as a new collection?

I can produce a test case showing this error if necessary.

This behavior wasn't necessarily a conscious decision.
When you call a method that doesn't exist in the Model class (e.g. pluck()), __call() instantiates and executes a new query. This is necessary to make User::pluck() work.
We could differentiate between existing and non-existing models, but that's not an option. I've seen plenty of people deliberately using existing models to execute new queries.
Personally, I think this is just a user error. I don't see why you would call pluck() on a single model.

This was an unintended side effect of having old code that used to be Model::conditions()->limit(1)->get()->pluck('data') which was changed to Model::conditions()->first()->pluck('data').
The way we worked around it is to have ->data instead of plucking it.
Is there a deliberate use for $one_product->pluck('data') to give you a collection of all products, and not just the data property from the record you are already on?
I can understand using the model for these methods if you aren't directly on an actual record, but if you're trying to call these methods on a record, it should only interact with that / those records right?

Is there a way that we can check if the record is an already existing one, and if it is, then change the behaviour, and if it isn't, pass the call through to the parent model?

Is there a deliberate use for $one_product->pluck('data') to give you a collection of all products, and not just the data property from the record you are already on?

I've seen people using this in controllers. They have a model property and use it to execute new queries:
$users = $this->user->where('foo', 'bar')->get();

Is there a way that we can check if the record is an already existing one, and if it is, then change the behaviour, and if it isn't, pass the call through to the parent model?

Do you mean in the framework or in your personal application?

In the framework. I feel this is a trap people may unwillingly fall into because it isn’t obvious. Perhaps it just needs documenting but as it stands it isn’t clear why this would happen
On 11 Jun 2019, 18:04 +0100, Jonas Staudenmeir<EMAIL_ADDRESS>wrote:

Is there a deliberate use for $one_product->pluck('data') to give you a collection of all products, and not just the data property from the record you are already on?
I've seen people using this in controllers. They have a model property and use it to execute new queries:
$users = $this->user->where('foo', 'bar')->get();
Is there a way that we can check if the record is an already existing one, and if it is, then change the behaviour, and if it isn't, pass the call through to the parent model?
Do you mean in the framework or in your personal application?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

We could "fix" it by checking $this->exists, but it would be a major breaking change.

IMO this doesn't  seem feasible. Eloquent has always worked this way and many use it like this. Would we do this check on all methods passed through via "__call"? Whitelist certain.methods? Only check for "pluck"? Seems like a tradeoff for the things that make Eloquent such a pleasure to use.


runtime: use MADV_FREE on linux as well

On linux currently sysUnused uses madvise(MADV_DONTNEED) to signal the kernel that a range of allocated memory contains unneeded data. After a successful call, the range (but not the data it contained before the call to madvise) is still available but the first access to that range will unconditionally incur a page fault (needed to 0-fill the range).
A potentially faster alternative is MADV_FREE, available since linux 4.5 (and already used for bsd). The mechanism is very similar, but the page fault will only be incurred if the kernel, between the call to madvise and the first access, decides to reuse that memory for something else.
Off the top of my head, there are two ways to support both pre-4.5 and 4.5+ kernels:

Use a fallback approach (i.e. try MADV_FREE first and MADV_DONTNEED if MADV_FREE fails): this is e.g. what node does
Detect at runtime startup whether the running kernel supports MADV_FREE (e.g. by attempting to call madvise(MADV_FREE) on an allocated but unused range)

/cc @aclements

Hi all. I wondering if there is any data backing up claims about MADV_FREE actually improving anything.
Asking because:
a) I have not seen any data pointing that it does help (I co-maintain gperftools and over there we're defaulting MADV_FREE to off on Linux).
b) there are numerous questions about implementation of this feature in kernel. First, while it is somewhat cheaper then MADV_DONTNEED, it is not that cheap, since page tables have to be walked and modified. It doesn't get reflected in RSS and thus interaction of this and memory reclaim facility in kernel, especially in settings with heavy usage of control groups isn't trivial. And also defaults about how this memory is preferred compared other memory during reclaim is also not entirely clear.
Since this feature was lobbied by jemalloc folks, maybe there are some Facebook folks involved in golang who do have some data?
I think this ticket should be re-opened. We should either gather and publish the data and see that indeed it helps, or revert defaults to off. IMHO.

@alk I'm curious if you have tried any experiments with GODEBUG=madvdontneed=1? From the Go 1.12 release notes:

To revert to the Go 1.11 behavior (MADV_DONTNEED), set the environment variable GODEBUG=madvdontneed=1

Separately, given the volume of issues, I think some people automatically filter out notifications from closed issues, so you might want to open a new issue with your comments above.


move to Java 8

Move to java 8 since it is recommended by upstream logstash:

WARN -- Concurrent: [DEPRECATED] Java 7 is deprecated, please use Java 8.
Java 7 support is only best effort, it may not work. It will be removed in next release (1.0).

$ docker run --rm -it logstash -e 'input { syslog { port => 514 } } output { std
out { } }'
'[DEPRECATED] use `require 'concurrent'` instead of `require 'concurrent_ruby'`
Logstash startup completed

$ nmap <IP_ADDRESS>

Starting Nmap 6.47 ( http://nmap.org ) at 2015-07-28 16:03 PDT
Nmap scan report for <IP_ADDRESS>
Host is up (0.00038s latency).
Not shown: 999 closed ports
PORT    STATE SERVICE
514/tcp open  shell

Nmap done: 1 IP address (1 host up) scanned in 0.07 seconds

Doesn't work in AUFS. :cry:
http://lkml.iu.edu/hypermail/linux/kernel/0902.3/01324.html

Ok, dropped the ports changes so we can at least move to java 8.

:+1:


test_filenames_and_formats appears flaky

I'm trying to build dask 2.11.0 for Fedora. I did this, and then noticed I forgot to update some dependencies, so I ran it again. As there were 3 releases, that made for 6 builds. However, it failed twice in the first set, and twice in the second, so only 33% success. The failing builds were all about test_filenames_and_formats:
______ test_filenames_and_formats[mydask.pdf.svg-None-mydask.pdf.svg-SVG] ______
[gw27] linux -- Python 3.7.6 /usr/bin/python3
filename = 'mydask.pdf.svg', format = None, target = 'mydask.pdf.svg'
expected_result_type = <class 'IPython.core.display.SVG'>
    @pytest.mark.parametrize(
        "filename,format,target,expected_result_type",
        [
            pytest.param(
                "mydaskpdf", "svg", "mydaskpdf.svg", SVG, marks=ipython_not_installed_mark
            ),
            ("mydask.pdf", None, "mydask.pdf", type(None)),
            pytest.param(
                "mydask.pdf", "svg", "mydask.pdf.svg", SVG, marks=ipython_not_installed_mark
            ),
            pytest.param(
                "mydaskpdf", None, "mydaskpdf.png", Image, marks=ipython_not_installed_mark
            ),
            pytest.param(
                "mydask.pdf.svg",
                None,
                "mydask.pdf.svg",
                SVG,
                marks=ipython_not_installed_mark,
            ),
        ],
    )
    def test_filenames_and_formats(filename, format, target, expected_result_type):
        result = dot_graph(dsk, filename=filename, format=format)
>       assert os.path.isfile(target)
E       AssertionError: assert False
E        +  where False = <function isfile at 0xf785541c>('mydask.pdf.svg')
E        +    where <function isfile at 0xf785541c> = <module 'posixpath' from '/usr/lib/python3.7/posixpath.py'>.isfile
E        +      where <module 'posixpath' from '/usr/lib/python3.7/posixpath.py'> = os.path
dask/tests/test_dot.py:222: AssertionError

First go around:

Fedora Rawhide
Fedora 32 works
Fedora 31
Second go around:
Fedora Rawhide
Fedora 32 works
Fedora 31

Looking at the HW info, it doesn't appear to be arch-specific.

This is probably because of running in parallel; there are two targets of mydask.pdf.svg in the parametrization of that test.

I also see errors related to worker crashes
_____________________ dask/dataframe/io/tests/test_hdf.py ______________________
[gw1] linux -- Python 3.8.2 /usr/bin/python3
worker 'gw1' crashed while running 'dask/dataframe/io/tests/test_hdf.py::test_read_hdf_multiple'
=============================== warnings summary ===============================

this was the case with Fedora Rawhide.  Are these issues specific to how Fedora builds packages ?

That one is a sporadic failure in HDF, but I have yet to figure out what causes it, or whether it's directly from dask or not.

Do you think we are hitting similar race conditions described in https://github.com/dask/dask/issues/5731

For this issue or for the HDF5 one?
For this issue, I'm pretty sure it's because there are two tests that produce the same file and the ensure_not_exists from one test deletes the result before the other can check it.
For the HDF5 one, I've narrowed it down to only happening when the build is on armv7hl, but I think something is up with the HDF5 library.


Problem with a custom user model with a custom USERNAME_FIELD

I have custom user model with USERNAME_FIELD = 'email'
module requires User.username field which doesn't exist
please adapt the module for the custom user name field

i just stumbled on this issue as well. after some sourcecode digging i found the settings of this plugin. the relevant setting ist HIJACK_REGISTER_ADMIN = False
after that change the plugins does not try to register a own usermodel with a username field. but you have to mixin HijackUserAdminMixin into your custom usermodel

but thats documented in the readme. maybe we just have to read a little bit better? :D


Accessing users' roles through withUser HOC

I was hoping to be able to access the currently authenticated user's roles via this.props.user inside a react component wrapped with withUser HOC.
I've followed the example in the docs on implementing user roles.
I manually added the admin role to my user and verified in mongodb that roles on the user is an array.
db.users.update(
  {
    _id: ObjectId("5ba46cac9a370b50b10ae4f4")
  },
  {
    $set: {
      roles: ["admin"]
    }
  }
);

Then inside the withUser wrapped component if I console.log('user ', this.props.user.roles) it prints
user  { _id: '5ba46cac9a370b50b10ae4f4',
  local: { email<EMAIL_ADDRESS>},
  profile:
   { clientId: '87e6a3ac-5891-4322-a7ca-0df325e1fddd',
     firstName: 'First',
     lastName: 'Last',
     eulaAccepted: true },
  roles: {} }

I was expecting roles to be an array of strings?
Sidenote: After manually updating my role to admin I was able to use oothClient to successfully update a different user's roles as the docs' explain.
await oothClient.method("roles", "set", {
  userId: "XXX",
  roles: ["editor", "author"] // will overwrite all previous roles the user had
});

Hey sorry I never answered. The reason is that the user object is structured like this:
{ [strategyname]: { [key]: value } }

so to set the role correctly with mongodb the query should be
db.users.update(
  {
    _id: ObjectId("5ba46cac9a370b50b10ae4f4")
  },
  {
    $set: {
      roles: {roles:["admin"]}
    }
  }
);


A notebook that shows Vertex AI feature store capabilities in a real-world scenario

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the # Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

You can follow the steps here to automatically fix the linting issues:
https://github.com/GoogleCloudPlatform/vertex-ai-samples/blob/main/CONTRIBUTING.md#code-quality-checks

@andrewferlitsch Please don't force merge and override checks. That's only applicable for CI changes.


Time Zone Capabilities for Clock Widget

Is there anyway we can add a time zone capability for the clock widget?

Hey @jnicoara11 -
It is indeed possible, surprisingly enough (in retrospect) having the ability to select time zones was not in our initial set of requirements for the clock widget.
I have logged this feature request into our backlog but we'd be more than happy to merge in a contribution.
Please continue to share your feature requests and feedback with us!

Awesome! I can't wait for that functionality. When do you think that will be able to be done?

We are initially targeting around the beginning of the new year to deliver this feature. If this estimate changes, I will reach out to you directly for updates!

Thank you!

Hey @jnicoara11 -
I'm excited to let you know that we have enabled the ability to select specific time zones for the clock widget in our latest Cyclotron release (1.45.0)!
Please let us know if you have any questions or ideas on additional feature enhancements!
Thanks,
Long


dynamic config method when creating a Firecreaker VM

Proposed change
When creating a new Firecracker VM with machine_template config, the cpu and memory sizes are fixed. (e.g. if I create 100 new VMs, they will all have the same vcpu cores and memory size).
Is it be possible to flexibly adjust the size of each Firecracker VM at the time of VM creation, e.g. depending on the specific tag?
If possible, I would like to know how to dynamically adjust its size and start it.
Use case
Workload "A" needs 2vCPU, and workload "B" might need 4vCPU.
we do not simply execute simple function workloads, but rather use processes that must be maintained for a long time, such as batch/build/scheduling.
Contribution
No response

Thanks Noah!  I think there is existing work on this somewhere and it might not have landed yet.  Let me investigate and Ill let you know.  Either way, we agree on this being a required feature from production use cases

Closing in favor of #60.  Feel free to follow up on progress there...and thanks for the feedback!


Updated README.md

Description of Change
Added my name in contributors list.

Yes it's correct! You can merge this commit.


Add getRegionAtAnotherScale() - converts region from one scale to another

Considering the region spec is the following dictionary:

left: the left edge (inclusive) of the region to process.
top: the top edge (inclusive) of the region to process.
right: the right edge (exclusive) of the region to process.
bottom: the bottom edge (exclusive) of the region to process.
width: the width of the region to process.
height: the height of the region to process.
units: either 'base_pixels' (default), 'pixels', 'mm', or 'fraction'.

Add a function getRegionAtAnotherScale(source_region, target_scale, source_scale=None) that takes a region at source_scale and returns region at the target_scale. If units of source_region is pixels, then source_scale must not be None.

Also, later the usecase of converting a region spec from one type of units to another might come up.

In the event that the units of the source region is anything other than pixels, then this function ends up being just getRegion(region=source_region, output=target_scale, **kwargs).  In the units = pixels case, more ends up being necessary.
I'll add a convertRegionScale(source_region, target_scale, source_scale=None) function that will output a region that can be passed to getRegion or tileIterator.  Do we need tileIteratorAtAnotherScale besides getRegionAtAnotherScale?

I don't have a use case for tileIteratorAtAnotherScale at the moment. But you may want to create an issue for it and we can address it when a use case arises. I can see some uses for it but not at the moment.

Added in PR #98.


License usage

Hi, we're interested in using these scripts to test FindMatlab in CMake. However, we use GitLab-CI; what is the license situation for using the scripts here for our CI purposes? Thanks.

Hi @mathstuf,
The install scripts will not license MATLAB on GitLab CI and are really intended to work only on the handful of CI services we currently support (Azure Pipelines, CircleCI, GitHub Actions, and Travis CI).
For a GitLab project, you may consider running your own GitLab Runner where you have pre-installed and licensed MATLAB. Another option would be to use Azure Pipelines with a third-party GitLab integration (note: I have not tried this myself). CircleCI also claims to be working on GitLab support, targeting general availability for the second half of the year.
Best,
Mark

Want to +1 Mark's comments. Are you using gitlab hosted shared runners or do you already leverage you own runners? If the former then I think Mark's suggestion is the way to go. If you wanted to leverage gitlab hosted shared runners, there is not currently an official solution but we'd love to work with you to try to see how we can possibly help. Also, as Mark says, using a supported platform like Azure Pipelines or CircleCI might be able to help you use a shred runner hosted service, but we should actually confirm this would work with what you are doing before going all in on that approach.
Let us know if you'd like talk more about what you are doing if there are challenges using self hosted runners or a supported CI platform.

We host our own GitLab instance with our own runners. However, we only use Docker containers for Linux (so far). We do use "bare metal" for macOS and Windows, but we really try to keep the machines as "clean" as possible so that there's little in the way of machine-specific setup and it'd be nice to just be able to download an SDK and extract it as part of the CI steps. I expect any bare metal Linux runners to follow a similar pattern. However, just getting Linux/Docker testing would be a good first step at least.

Sure, you should be able to generate a MATLAB container and/or utilize mpm apis but you are right in the comments at the bottom of this page that you'll need to think about licensing. Does your gitlab server have access to a license server? If so you can utilize a named network server license to enable licensing.

I know we have some licenses; I'll ask about what kinds they are, how they are stored, and whether we can set up a network server for them for CI to use. Thanks.

sounds good. If you run into challenges with your existing licensing setup, feel free to connect with us at continuous-integration at mathworks dotcom and we can try to help you get unblocked with this. We'd like to help you get this going.

CircleCI recently announced GitLab support: https://circleci.com/blog/announcing-gitlab-support/

Does that work with only gitlab.com or is it any GitLab installation? I don't see it mentioned there at least.

@mathstuf Good question. I tried it and they have a "GitLab Self-Managed" option in the drop down, but it says "Coming Soon" if you select it.

Hi @mathstuf,
The install scripts will not license MATLAB on GitLab CI and are really intended to work only on the handful of CI services we currently support (Azure Pipelines, CircleCI, GitHub Actions, and Travis CI).
For a GitLab project, you may consider running your own GitLab Runner where you have pre-installed and licensed MATLAB. Another option would be to use Azure Pipelines with a third-party GitLab integration (note: I have not tried this myself). CircleCI also claims to be working on GitLab support, targeting general availability for the second half of the year.
Best, Mark

I am currently waiting for a call from MathWorks regarding the shared runner. I have set one up just for me to see if it works at all before going forward. Some people online claim that you cannot use such a runner unless you have a concurrent network license. In my case we have very few network named user licenses (I am currently a holder of the Matlab Compiler one). I have set up the GitLab runner to be a service bound to my own user account (you can restrict a runner that way). Further in the CI one can restrict jobs by user ID (e.g. only if user XYZ pushes CI job ABC will be triggered).
My current setup involves the Matlab Compiler/Compiler SDK toolbox. The only current unknown is how a local GitLab runner is affected by Mathworks' licensing model. Basically I have come up with the following conclusions:

a compiled Matlab executable binary does NOT require licenses for the toolboxes that it contains (albeit some functionalities of those toolboxes may be missing as described on the official compiler support for toolboxes page. You still need the licenses for the development of the code but once you create the EXE, it's no longer an issue
a compiled Matlab executable binary needs only MCR as well as (if used) any external DLL/SO files, which are not really bound to any Mathwork license (e.g. I have Matlab code that is interfacing the GDAL library for GIS data processing)
the MCR is freely distributed and can be installed and run inside a Docker container

Basically my approach (unless my call with Mathworks explicitly points out that it's against the license agreement, in which case I will require absolute and precise proof for that and will involve our legal team) is to automatically generate wrapper code that loads a workspace with all required variables by the original Matlab function/script or provides automated facilities for parsing CMD line arguments (you can use eval for loading a string that contains all required variables inside the runtime of Matlab). Running the wrapper code plus the original code through the MCC via the local GitLab runner I can upload the resulting executable binary (capable of loading e.g. a workspace from a MAT file) and then use it in follow-up jobs, e.g. testing or more without worrying I am breaking any license agreement since the code is running on MCR.
The alternative is very pricey since concurrent licenses are 4-5 times more expensive AND you also have to provide a license for every toolbox involved, since your CI will be running pure Matlab code (not compiled one). To put it in simple terms, with just a few toolboxes such a setup may cost you several thousands or even tens of thousands of USD per year. We are currently paying over 80K for a vast number of toolboxes with few network named user licenses in the pool. If we have to switch to the concurrent version, we will go broke. LOL

Folks watching this issue may be interested in the batch token licensing pilot. More information can be found in the link below (in a Docker context, but also applies to other approaches such as these GitHub Actions)
https://github.com/mathworks-ref-arch/matlab-dockerfile/blob/main/alternates/non-interactive/MATLAB-BATCH.md


Fix/patch 0.6.6

Description:
patch 0.6.6
Issues:

Issue #960
Issue #959

Checklist:

[ ] the pull request name complies with Conventional Commits
[ ] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[ ] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[ ] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

Hey there and thank you for opening this pull request! 👋🏼
We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Details:
No release type found in pull request title "Fix/patch 0.6.6". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - fix
 - hotfix
 - feat
 - feature
 - chore


Support for OTP

I am making this a PR before it gets bitrot :D
As I mentioned in #69, this is likely not the best way to display the OTP - Also there are concerns to be had about storing your OTP secret in the same store as your password!

I think you should just support the pass extension pass-otp. To display the OTP, why don't you submit it like the extension does it with passwords and logins?
Regarding the concerns about storing OTP secret in the same store as your password, it has been disputed in the pass mailing list. Due to the nature of pass it is the user responsibility to use different key and/or devices to store the otp secret.

I think you should just support the pass extension pass-otp.

Meaning the format they use in the encrypted file, or the actual pass-otp script? The latter would be difficult to use via the browserpass extension.

It is not impossible to use pass-otp script, this whole extension was using pass binary to retrieve passwords, however just like pass was ditched in favor of plain gpg for various reasons, I don't think you should introduce a dependency on pass-otp script. Your approach is good here, keep it as it is.
However, I do think supporting the format of otp string that pass-otp is using is a must, it is too popular of an extension to be ignored. Also, we already have a feature request for this - #99 🙂
@qbit are you still interested in getting this PR merged? Could you add the support for pass-otp format, and fix the code conflicts? I'll make sure to review it ASAP 😉

Sure! I will get something re-pushed this week hopefully!

OK, I have an update for this that makes it work with the pass-otp url style.  I am getting some weird errors when talking to the native extension though (not just on my otp branch, but on current master for this repo..). Did something else break?

Also - this disables the alert for the OTP - as I can't test :D

So you decided to support only pass-otp format, and not any other?
If we don't have alerts, how will users retrieve the OTP code? 🙂
I don't have any issues on master, could you try maybe to remove everything you have, and try again from scratch?

@maximbaz how are you loading the extensions for testing?

This is a clean set of installation steps:
- Delete all artifacts from all previous installations
$ yarn
$ make js
$ make release
$ ./install.sh
- Load `browserpass/chrome` with `Load unpacked extension`.

Yep.. looks like I still haven't had enough coffee!

OK, this is working - but I have found a few OTPs that are encoded oddly (they truncate the base32 padding stuff, which angers Go)

Do you have an example of such OTP url? I wonder if pass-otp itself can handle those, because if it can't - I'm fine with also not supporting them.

Looks like it's an issue with https://github.com/gokyle/twofactor/pull/7 - I have submitted a PR that should fix it! Thanks for the suggestion re pass-otp - I looked at oath-toolkit to figure out what it was doing differently!

I'm fine with dep, but I don't want to wait until the https://github.com/gokyle/twofactor/pull/7 is merged and a new version is released... can we just uppercase the secret on our side too? You will know when https://github.com/gokyle/twofactor/pull/7 is merged, just come back and remove the unnecessary case change when this happens.

Done and done!

So regarding dep, will you add Gopkg.toml here on in a separate PR? :)
And could you look at my question in the js file?

Less strict check for digits and package deps using dep! Makes sense to do the dep stuff in this PR (imo) since the changes are related to otp anyway.

Thanks. Shouldn't the vendor folder be gitignored? I would rather add a note to install dep and run dep ensure in CONTRIBUTING.md, to keep the repo clean

I realize we had vendor folder in the repo, but now that we have dep, why not clean it up 🙂
dep ensure can always bring back the correct packages, right?

So the big reason to include the vendor directory is for package maintainers.
Package frameworks like OpenBSD's ports require a complete "tarball" for a given package and using external mechanisms to pull stuff down is forbidden.
I know it adds extra clutter to the repo - but it makes the porting of the app dramatically easier.
Also - I had improperly tested the previous uppercasing - had an old bin. Hence the "clean" target in this commit :P

Interesting, but aren't they using browserpass-openbsd64.zip provided in the release? That archive has never contained vendor folder, because the binary is pre-build.
Or there are some packages that don't use the release archive, and instead build from tarball?
I'm fine with keeping vendor, just curious to understand this stuff :)
I'll test this tomorrow morning and merge if all is okay.

With OpenBSD specifically it's typically better to build ports from source, even if binaries are available (net/syncthing for example). This is particularly important because OpenBSD developers are running snapshots while most users run release. During heavy dev cycles, snapshots can contain ABI breakage - rendering binary ports completely useless.

Thanks for the explanation!
Tested, works great. I extended the make clean goal a little bit and merged everything in https://github.com/dannyvankooten/browserpass/commit/2cd4c8e420c578d33f0da4c45f3e487bc88494d3.


Missing .dll for both 1.6.3 and 1.6.1

The .dll for this mod appears to be missing.  Same issue when I tried 1.6.3 recently.
IRBTModUtils 1.6.1
Expanding manifest IRBTModUtils:
:StreamingAssets
Error: DLL specified (C:\SteamLibrary\steamapps\common\BATTLETECH\Mods\IRBTModUtils-REL_1.6.1\IRBTModUtils.dll), but it's missing! Aborting load.
Same error in log for both (modtek log).  I don't see the .dll anywhere, any ideas?

I just checked and both downloads include the files - https://github.com/IceRaptor/IRBTModUtils/releases/download/REL_1.6.3/IRBTModUtils-1.6.3.zip has it, as shown in

Perhaps your zip corrupted when it was loaded? Also, you might try renaming the folder to just IRBTModUtils - I don't know if Modtek has weirdnesses with hyphens or dashes in the name.

Thanks, I had the same issue with CBTBehaviors
I was downloading the zip multiple times from your 'Releases' page for each of these mods.  Is that a different link you shared?

Yes, that link is from the releases page. I downloaded both 1.6.3 and 1.6.1 and extracted them on my PC, and then checked the contents. You can see from the screenshot the file is present - so I'm not sure what is going on. I'm guessing ModTek may not like the hyphens in the name, but that's a wild-ass guess with nothing backing it up.

Interesting, not sure.  It hasn't happened for all mods but a couple of other mods have had missing .dlls.  When you gave me the link here it worked, and the mod is now functioning.  Could you provide the .dll for CBTBehaviors possibly?  I'm using the non-enhanced one, for vanilla (no MechEngineer)

Very excited to try Concrete Jungle, saw you updated this to support it, but no .dll for 1.64. My coding is not up to the task of compiling from source. Some help?

Updated the release tag to have the 1.6.4 binary. Sorry for the delay!


[AF-1467] Upgrade to WildFly 13

Depends on https://github.com/jboss-integration/jboss-integration-platform-bom/pull/421

Build finished. 2700 tests run, 5 skipped, 0 failed.

This will be replaced by a wf14 updgrade done by @adrielparedes


excessively frequent reconnect attempts

Hello, I've noticed that after about 78 or so failed connection attempts it starts reconnecting wildly with apparently no delay between attempts. I have seen this with default settings as well as with set_reconnect_delay set to 5000 and set_reconnect_delay_max to 10000. How do I prevent this?
Also, is there any way to suppress the [devel] output? Thanks!

Thanks,this is a bug of integer overflow while trying to cast the double value pow(1.5,attempts) to a unsigned int.
fixed in 2e94e799e264e5d4373d016569e8198c359d621d   tag 1.4.1

To suppress the [devel] output you can build in Release mode.

Yeah, I still need to see my own debug output (currently all getting lost in the deluge of [devel] output). So it would be nice to have a debug levels setting for the socket.io code. Guess that's a feature request.


number of total targets in the results page of batch search

@deniseOme commented on Mon Oct 23 2017
Can the total numbers of targets be reported for each disease in the results page of the batch search? This number would be in brackets.

As per June 2017, we would have:
Crohn's disease (3426).

Triage notes:

Due to current layout of Diseases Associated table and truncation of disease name, this isn't a feasible request
Concerned that adding a number would cause more user confusion and then would need to add additional columns to explain relevance of that number in relation to the targets they entered in the batch search

Issue closed as per triage notes above


some problem when handling image path with space key

When use markdown parser, image path with space key could not be handled well.
E.g.
![](a b.txt)
after convert to html, it becomes:
<img alt="" src="a" title="b.txt" />
PS: github parser could handle it well.
Thanks,

Are you using the github parser or the python markdown parser?  If you are using the Python Markdown parser, this may be an issue with that module which should be fixed on their repo as we just use theirs.  It appears they use a notation of ![alt](src title), so if that is the case, it is not really a bug per se...I guess it depends on how it is interpretted.
I will have to take a look and see what they are doing and what their intentions are first before I can give a definitive response.

I get the impression that Python Markdown is not following the Markdown spec on this one.  The original Markdown spec states that the title should be enclosed in quotes following the path: http://daringfireball.net/projects/markdown/syntax#img.    You can try out the code on the dingus and see it works as you are showing: http://waylan.pythonanywhere.com/dingus.  It appears that it is a bug in their library.

Just tried ![alt text]("a b.txt" "title") in page http://waylan.pythonanywhere.com/dingus, it still doesn't work as expected.
It seems I have to switch to github parser. Could github parser support bullet list like below:
- [ ] a
- [x] b
- [ ] c

Just tried ![alt text]("a b.txt" "title") in page http://waylan.pythonanywhere.com/dingus, it still doesn't work as expected.

That's still not the right syntax.  Surrounding the link with quotes is not the answer.  Titles are supposed to be surrounded with quotes, but there is a bug that allows sets titles that aren't surrounding in quotes, and that is incorrect.  Now, as for paths with spaces, Python Markdown doesn't support that, and the author is leaning towards not supporting it (at least that is what he told me when I asked him about it).  You can probably replace the space with %20 though and give that a try: ![alt text](a%20b.txt "title").
I am working to get a patch for the title bug, but I don't think we are going to get support for spaces in the links in Python Markdown.

It seems I have to switch to github parser. Could github parser support bullet list like below:

Github parser can support those as it goes through their API, but you will probably have to put it in gfm mode (Github Flavored Markdown): https://github.com/revolunet/sublimetext-markdown-preview/blob/master/MarkdownPreview.sublime-settings#L137.

Thanks a lot for fixing my problem.


Release sdk with protobuf-java instead of protobuf-javalite

Problem
My project uses grpc and I can build only with support for the protobuf-java instead of protobuf-javalite. Could you build and release a new version supporting the full protbuf-java? I saw the config files for the build with protobuf-java, but I couldn't find any release.
Solution
Build and release new sdk with support for protobuf-java not protobuf-javalite.
Alternatives
No response

Hi @KamilBemowskiGFT,
Can you please advise how urgent it is for you (is it blocking you right now)?
Thanks 🙏🏻

Hi @thenswan ,
Unfortunately this is currently a blocker for us as many of our services already use "full" protobuf-java library and trying to use the protobuf-javalite would cascade to a whole lot of other places, so it's not really feasible.
Thanks for your support

Hey @KamilBemowskiGFT @PIOTR-OSTROWSKI-GFT,
The new version of Java SDK was just released (including sdk-full) -- please let me know if you have any other questions or we can close this issue.

Closing as completed.
Feel free to reopen if you will encounter any issues with the sdk-full.


After VAE, result is NaN in sdxl mode.

I try streamdiffusion in sdxl model, but after VAE decode, result image is NaN. How can I solve it?
I add added_cond_kwargs in unet_step.
Or when streamdiffusion can support sdxl model !

I have solved this problem, and streamdiffusion can support SDXL model.
For this question, I have to change the vae dtype to fp32, otherwise the result will be overflow.
OHOHOHOHOH!!!!!

Thank you very much. Please feel free to submit a PR.

@ApolloRay Nice, can you share your method with us? And show us the txt2img speed ? Thanks.

I will refine my code and release soon.

@ApolloRay hi, did you succeed in making SDXL work with StreamingDiffusion? How's the performance?

#ADD represent for added code.

Thank you for looking into this @ApolloRay  - I did have to make one more tweak to get sdxl-turbo working (passing down the added_cond_kwargs). As a VAE, I swapped out madebyollin/taesd to madebyollin/taesdxl

added_cond_kwargs

Hey there, I find myself in a bit of a loss as to how to pass down the correct arguments when calling the pipeline. could you please elaborate?

figured it out.
you have to replace the stablediffusionpipeline lines in wrapper.py to stablediffusionxlpipeline.
furthermore, if you want to use fp16, you'll have to replace all float32s in pipeline, wrapper, your image transmission code and -possibly- image_utils.py
if you want to use fp32 too, i think you have to replace all these to fp32 (there are some that are fp16 in the repo).
other than that @ApolloRay kudos and thanks a lot!

@ApolloRay Can you submit a PR? I'll be appreciate if you do that🥺🥺🥺
I can't wait to use the Ultra Fast SDXL model! But my try on your code is failed. I don't know what went wrong. THX!

I have added SDXL support to the pipeline and wrapper based on @ApolloRay 's code (thanks for that!). I have also added single image and real-time image generation examples for sdxl-turbo. Check out my sdxl branch.
https://github.com/hkn-g/StreamDiffusion/tree/sdxl
sdxl-turbo img2img looks fine, but there are some other issues, such as TensorRT not working, and text2img giving an error with an SDXL model. However, in img2img mode, it works without an input image.


"Creating encrypted secrets for a repository" in "Encrypted Secrets" page is outdated

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/security-guides/encrypted-secrets#creating-encrypted-secrets-for-a-repository
What part(s) of the article would you like to see updated?
Right now are the mentioned steps as follows:

On GitHub.com, navigate to the main page of the repository.
Under your repository name, click Settings.

In the left sidebar, click Secrets.
Click New repository secret.
Type a name for your secret in the Name input box.
Enter the value for your secret.
Click Add secret.

The list is outdated by having the Secrets field now open a selection of options (Actions, Codespaces and Dependabot) rather than directly leading to a page where you can create the Repository Secret.

Additional information
No response

Thanks so much for opening this issue @Andre601! You or anyone else is welcome to open a PR to make this fix ⚡