Allocation re-architecture

Thundernetes allocation service currently works by finding a random StandingBy server and transitioning it to Active. We should consider the pros/cons of this approach and decide on alternative allocation methods, taking into account the environment that thundernetes is running (cloud provider/on-prem).
Moreover, the use of .Update to transition the game server to Active will return errors in case the GameServer has been deleted or crashed and the cache has not been updated. We should retry ourselves (at least once) in the allocation method.

I’ve been thinking about this recently and my proposal is to try and allocate on the newest Node. In cloud environments, this would have the effect of making sure the oldest Nodes get as less active game servers as possible, so in the event of a cluster scale down, they would be the ones to be deleted first. Result of this is that on a scale up, fresh/updates Nodes will pop up and be added to the cluster.
This would require an extra field on the GameServer that would designate the Node’s age. During allocation, we should sort the standingBy game servers by Node’s age and allocate on the newest ones.


UnixHttpClient.java:47-48: Implement a...

The puzzle 29-9f250f40 from #29 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/64b03124fa03420a9972d7ab5b89051d7b125e8b/src/main/java/com/amihaiemil/docker/UnixHttpClient.java#L47-L48
The puzzle was created by amihaiemil on 24-Feb-18.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd all puzzles are solved

@0crat assign @amihaiemil

The puzzle 29-9f250f40 has disappeared from the source code, that's why I closed this issue.

@0pdd the puzzle #36 is still not solved.

@0pdd the last puzzle #36 is solved here.

The job is not in WBS, won't close the order


Add E2E test for guest cluster IPAM

Guest cluster IPAM implementation is renewed architecturally as
kubernetesd is going to be removed. In order to make sure that allocated
networks are not in use, refactored IPAM implementation must be taking
cloud provider state into account and therefore new implementation
resides in provider specific operator (i.e. aws-operator, azure-operator
and kvm-operator). This test is for verifying that these implementations
cover basic use cases.

Need to check aws-operator e2e execution next. There's something wrong there, so this is not 100% complete yet, but close.

I rewrote the thing a bit. Improved the provider interface. We can implement it in the provider operators themselves. That pattern might be a cool thing in general. Also to keep the dependency graph cleaner here. Nice learning. I also improved logging and the code structure to please me personally a bit more. Please have another look.


Implementing learning bocpd

Was wondering if you had any plans or need for the online extension seen in "Adaptive Sequential Bayesian Change Point Detection."
I was going to implement it for a work project, and would like to just keep it in this package as it fits the theme.
Thoughts?

Hi Dan,
Didn't have the chance to look into it. I'd be happy to have it in this package, though! I think it makes sense to bundle as many methods as possible.
Right now I'm a bit off, since my first son is just a few days old, so expect the merges to master to take some days.
Best,
Johannes
On August 11, 2015 1:01:59 AM GMT+02:00, Dan Marthaler<EMAIL_ADDRESS>wrote:

Was wondering if you had any plans or need for the online extension
seen in "Adaptive Sequential Bayesian Change Point Detection."
I was going to implement it for a work project, and would like to just
keep it in this package as it fits the theme.
Thoughts?

Reply to this email directly or view it on GitHub:
https://github.com/hildensia/bayesian_changepoint_detection/issues/9

--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

Looks promising and should be relatively straight to implement. Go for it! :)

@mathDR, Wanted to check if you got a chance to implement Adaptive sequential BCD? Thanks!

I did, but alas it is in a private repo at a company that I no longer work at (and has since been absorbed by a larger company). :-(


rich: New features

rich streak <habit name> - get the streak number of a habit
rich todo <habit name> - print "todo" or "done" based on whether the habit has been completed today

I think this would make it more properly extensible; you wouldn't have to use head and tail for these basics

But if it's already possible to use head and tail, then it's just duplicating functionality! It's bloat!

You could already recreate the entire project with a shell script, so is everything bloat? The point is that almost any rich wrapper would require these

I wrote the main rich code a while ago, so hopefully it's not completely unreadable. I need to read Effective Go properly

I changed my mind about rich init. That sounds like a waste of code if I could just write the mkdir command as an installation instruction

Untested but finished, I think


Navigation is unreachable when connect to paypal

When I am trying to connect with PayPal, the browser not closing and not returning to my app.
Please help me to fix the issue. Showing this message
Navigation is unreachable: com.****.*****.pos.braintree://onetouch/v1/success?token=EC-71229415R2057501E&ba_token=BA-7MT74179DE6494833
Actually my app id is com.****.*.pos.
but the returning url showing with extra .braintree and i don't know why.

I finally fix this by going in the file platform/android/cordova-plugin-braintree/* - build-extras.gradle** and editing to
dependencies {
compile 'com.braintreepayments.api:braintree:2.21.0' //2.2.1 Working
compile 'com.braintreepayments.api:drop-in:2.3.8'
}


[Improve][Jdbc-e2e] Reactor jdbc-e2e with new api.

Purpose of this pull request

Reactor jdbc e2e with new API. close #3922
Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

check ci error


Update sbt-mdoc to 2.2.15

Updates org.scalameta:sbt-mdoc from 2.2.2 to 2.2.15.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-mdoc" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #168.

Superseded by #168.


Show only the street (not street + city + state)

Hello,
Thanks for this great plugin. Very useful and easy to use!
I'm using it in a form where I have different text inputs for street, city, state and country.
My question is, how can I get -for example- only the street part of the address?
For example, if in the street field the user looks for 1000 El Camino Real, I'd like to have this written in the input after the user selection, not 1000 El Camino Real, Menlo Park, California, United States. Of course, I'm ok if the full address comes out in between the suggestions, but not in the text input.
Any reply would be greatly appreciated!
Thanks again,
-Luca

After working on this way more, I found the solution and I thought it was good for future reference to include it here for other people with the same issue.
I've noticed that the model was populated with a "complex" object, and not with a string. This allowed me to listen for select events of the component in my controller and set different fields (model) accordingly:
In the HTML I have
input g-places-autocomplete options="vm.autocompleteOptions" id="street" ng-model="vm.property.street" type="text" class="form-control" autocomplete="off" required autofocus>
and
<input id="zipcode" ng-model="vm.property.zipcode" type="text" value="20100" maxlength="5" class="form-control" required autofocus>
What I wanted to achieve was: as soon as a user selects the full address from the autocomplete (while typing an address), put the street address only in the street text input, and the related zipcode only in the zipcode text input.
Js (controller)
Please note I use controllerAs and this instead of $scope only. I've also used lodash to retrieve the zipcode from the object.
// When a street is choosen from the google autocomplete options, set the street only         // address in the street field of the form, and the zipcode in the zipcode field of         // the form         $scope.$on('g-places-autocomplete:select', (event, param) => {           this.property.street = param.name;           this.property.zipcode = _.find(param.address_components, function (o) {             return o.types[0] === 'postal_code';           }).short_name;         });
Hope it helps, since it took me a while to figure this out!
Thanks again for the great plugin, ciao!
-Luca


[BACKLOG-36242][BACKLOG-36243][BACKLOG-36244][BACKLOG-36254] 1) Fix N…

…PE on invalid save attempt

Implement PasswordVisibleTextVar and replace all PasswordTextVar widets.
Add file browse to GCS dialog.
Add Rename/Copy prompt when saving VFS connection after changing the name.

:white_check_mark: Build finished in 16m 55s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,plugins/connections/ui,ui

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 16m 46s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,plugins/connections/ui,ui

:exclamation: No tests found!
:information_source:  This is an automatic message


Master

I assume this was opened by accident.
(If not, please provide a better description.)

Hi,
Sorry for the confusion..  wrong action via ui.
Was going to push to my forked stream.
Rachel
On Sep 12, 2016 6:55 PM, "Georgios Kalpakas"<EMAIL_ADDRESS>wrote:

Closed #364 https://github.com/angular/angular-seed/pull/364.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-seed/pull/364#event-785770653, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFLFJR9Nz2YencxB8SjnZ6PxN1nmeFt5ks5qpS-WgaJpZM4J6cZF
.


[DOC] add note about rust smart contract overflow

from someone developing on the platform:

Adding overflow-checks = true in cargo.toml makes your smart contracts after deployments behave more like the unit tests. So basically if there is large inconcistency between unit test execution and contract execution try this flag. > Probably good suggestion to always have this flag being true :)

s/behave more like the unit tests/behave like the unit tests

We also need to explain that the reason it is needed is because Rust handles overflows differently in release and debug modes. Specifically, it panics in debug and wraps in release. We might link this article https://huonw.github.io/blog/2016/04/myths-and-legends-about-integer-overflow-in-rust/

+1 on this, we created an internal ticket to update all the Developer examples.


NetCore Indentity samples are outdated for .NetCore 3.0

Issue description
Identity Samples are outdated. Kindly consider updating them to .NetCore 3.0

Hello @shyamal890 ... It's an on-going effort. Sample updates are tracked here :point_right: https://github.com/aspnet/AspNetCore.Docs/issues/5495


Is it possible to record only a specific HTML element?

Hi!
We are working on an that edits small video and records it on the browser. I was wondering if your library can help with that. Thanks!

can you elaborate, it's hard to guess what you exactly want to do.

@thijstriemstra Example: if I have a carousel/slideshow html element that is made in any library, example jQuery, is it possible for me to record that carousel while it is sliding ( with its animation ) and get a video out of that?

Not yet.
Marking as duplicate of #288


Update Code - Proof_Of_Contribution.md

Helped David Johnson in updating Best Practices

Thanks for the help. Good contribution.


Very high IDLE_WAKE - Power consumption

Something in tcell is causing a lot of idle wake, causing battery drain, it maybe a system event or ticker or such but I have not found the source of it as of yet.
Here is data from just running the mouse demo (it also happens with micro and other tcell apps)

This is another way to see the same data:
top -stats pid,command,cpu,idlew,power -o power -d

FYI, found the culprit to be the very short time out here : https://github.com/gdamore/tcell/blob/master/tscreen_bsd.go#L70
	// We wake up at the earliest of 100 msec or when data is received.
	// We need to wake up frequently to permit us to exit cleanly and
	// close file descriptors on systems like Darwin, where close does
	// cause a wakeup.  (Probably we could reasonably increase this to
	// something like 1 sec or 500 msec.)
	newtios.Cc[syscall.VMIN] = 0
	newtios.Cc[syscall.VTIME] = 1
	tios = uintptr(unsafe.Pointer(&newtios))

Seeing your comment, not sure there is a simple workaround but I might have a look later.

Yep.    Probably VTIME could be increased ... for example moving it to 5 would reduce the wake up count to 1/5th its current level, but would mean that it would take applications up to 1/2 second to exit cleanly.
Without this call, the application would hang (the read for data would block indefinitely) once the descriptor was closed.
Arguably, I could use some other notification mechanism and poll() instead -- if I have time later I'll think about this -- certainly not having to do the periodic wakeups would be attractive!
If I recall correctly, I saw this problem on MacOS, but not on other systems.   I tracked it to a termios bug.  I suspect the same bug may exist elsewhere, though.

Yeah sounds good, I didn't notice the longer quit time but I ll check again. In any case thanks for making this, I don't miss term box.

The above PR is one way to address it... but it doesn't allow for properly terminating tcell and restarting it.  As a result, I'm not too thrilled with it.  It might be reasonable though.
I want to think this through further too -- its likely that on some systems we don't have to dispense with the final close(), and that its only a problem for the BSD systems that have buggy termio drivers.
I'll probably write a test suite that people can run to see if their system is defective.

(Basically the defect is this -- read(2) should not continue to be blocked if the file descriptor is close(2)d.  The close(2) should wake the read(2), which should then return EBADF.

@gdamore I tried the PR, it does somewhat fix the issue although I had to modify it a little (insert a short sleep in the loop here https://github.com/gdamore/tcell/pull/130/files#diff-8feb5118e0c0711372db5cc7647c0e01R1252 because otherwise it tend to read single bytes at a time and with terminal sequence codes already being somewhat indeterministic that causes more issues).
You mentioned using poll() before that sounds likely a better fix, any reason why you didn't go that route ? Or is it juts a larger chnage that would take more time ?
Thanks.

I've completely refactored the input loop and this is fixed in my test branch (inputrefactor) . Stay tuned.


Runbook for responding to Organisation Fine-Grained PAT Tokens

User Need
As an Operations Engineer,
I want to develop a swift and secure process for handling email requests for personal access tokens (PATs) that require org access,
so that we can ensure these tokens are issued responsibly and used for their intended purposes while complying with our security policies.
Value
Implementing a streamlined yet secure process for approving PATs through email requests will help maintain operational efficiency without compromising security. It ensures all requests are appropriately vetted, minimising the risks associated with improper or overly broad token access.
Functional Requirements:

Upon receiving an email request for a PAT, log the request in a secure, centralised system for tracking and auditing purposes.
Reach out to the requester for additional information, specifically the intended use and scope of the requested token, if necessary.
Establish a quick-review protocol, enabling a rapid yet thorough examination of the token request against predefined security and necessity criteria.
Set up an automated reminder or expiry system to track the duration of each token and ensure timely review or revocation.
Document a process for immediate revocation of tokens if found to be misused or no longer required.

Non-Functional Requirements:

The response and approval process should be swift to accommodate urgent development needs, targeting a specific timeframe for completion.
Maintain a high standard of security and privacy in handling and storing information related to token requests.
Ensure the process is straightforward and documented for all parties, minimising confusion and delays.

Acceptance Criteria:

A process for handling PAT requests via email is documented, published, and communicated to relevant teams and stored in the organisational runbooks.
The process is utilised and adhered to, with records of requests, follow-ups, approvals, and revocations being accurately maintained.
There are mechanisms to review regularly and audit token usage, ensuring compliance with the issued scope and intended use.
Feedback from developers and the Operations team indicates that the process meets efficiency, clarity, and security needs.

New runbook created:
https://runbooks.operations-engineering.service.justice.gov.uk/documentation/services/github/review-org-pat-request.html#revoking-tokens
And new spreadsheet for logging token requests created:
https://docs.google.com/spreadsheets/d/1IMAhnj-BDqPIxwJgl2lXR1Mn3AoUQjcoVsdx861KK3k/edit#gid=0


Adds support for (newer) Original Xbox DVT3 HDDs

DVT3 are prototype development kits for the original xbox that were given to developers from June 2001 to August 2001. For systems running the July/August 2001 kernel, the partition order appears to be reversed but otherwise identical to retail FATX.
It appears that May / June kernels used a rather different FATX/HDD partition formats. This may or may not be the 'alpha' hdd format mentioned in #28.
I can supply a HDD image for this 'v2' DVT3 format, and the older 'v1' (alpha?) if you are interested along with a few notes or an IDB if you are interested.
Edit: also I split the commits for this change because otherwise it was harder to see the DVT3-specific logic I was adding, muddying the PR.

Thanks for the PR. Yes, can you send me those HDD images please? I would love to take a look at them.


Validator won't attempt to validate an empty slice

func IsNonEmptyIntArray(arrInterface interface{}, context interface{}) bool {
	defer G(O())
	if arr, ok := arrInterface.([]int); ok && len(arr)>0{
		fmt.Println("LenArr:")
		fmt.Println(len(arr))
		return true
	}
	return false
}

This fn is set as a custom tag and it will not even run if the slice is empty. Validate will return an OK even if the slice is empty because this function won't run.

https://github.com/asaskevich/govalidator/blob/7d2e70ef918f16bd6455529af38304d6d025c952/validator.go#L708
this seems to be the problem. However, my slice I think is a nil slice, not an empty slice, so I will explicitly use make to generate the slice so it validates properly.

Nope, making the slice an empty slice didn't work....so IDK.

It seems that the recursive call to Interface() ends up turning the empty slice into a nil slice.

@SlightlyCyborg : Assume you are having some structs which has required fields, but the struct is nil. In this case, whether the ValidateStruct is returning true? It would be much helpful when you post the issue with an example. TIA.


Update readme.md for Snapcrescent

Hi,
Just rebased the branch and pushed again with fixed. Let me know if any
other change is needed.
Regards,
Naval Kishore Gandhi
------ Original Message ------
From "MeIchthys" @.>
To "meichthys/foss_photo_libraries"
@.>
Cc "Naval Gandhi" @.>; "Author"
@.>
Date 02-07-2023 21:25:54
Subject Re: [meichthys/foss_photo_libraries] Update readme.md for
Snapcrescent (PR #53)

Hey, Thanks for the contribution. It looks like something is broken on
the "Duplicate Handling" feature row. Could you fix that?
Also, i hope you don't mind, but I will probably adjust a few of the
'subjective' ratings before the merge.
—
Reply to this email directly, view it on GitHub
https://github.com/meichthys/foss_photo_libraries/pull/53#issuecomment-1617088508,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB37T6I6IUWBKOR6PAS7UXDXOINSFANCNFSM6AAAAAAZ2X3C3A.
You are receiving this because you authored the thread.Message ID:
@.***>


Update README.md for LCM with LoRA

Update README.md for LCM with LoRA

Closing in favor of #761 and #772


Video controls are not showing

Player works very well, but the media controls are not showing

Hi @suvasis999 can you share the working code or just tell how to use it. I have been trying from past one month but getting several error from it. I tried in expo but not worked

i am integrate this vlc player in react native  (create react native) project not expo


Logging

Coverage increased (+1.3%) to 7.472% when pulling b7d7a29aaaa2665f8018474c80a74fc1ad0074e3 on logging into f6778a30958d4ec76060957e615f7fefe678fcf7 on master.


BalanceInCents is 0 after redeeming

After I call  giftCard.Redeem(accountCode)  The BalanceInCents property is set to 0 even after a successful redemption.  I would expect this to be equal to the balance listed on the gift card details page?

@jjwhite The reason is that it's null in the xml but the property is not nullable (thus it defaults to 0). I've made PR #263 to more accurately reflect this case.
As for why it's null, I believe the BalanceInCents refers to the balance left on the card. After redemption the balance has been applied to the account. If you want to know the amount that was redeemed I think you want to look at UnitAmountInCents. I'll need to confirm my reasoning as it does seem like the full balance is listed in the admin like you noted.

@bhelx Is there another way to know if the gift card has been successfully redeemed if the balance is not being updated upon redemption?

@jjwhite I don't know if you're still experiencing issues with the BalanceInCents being 0 after redemption. I was not able to reproduce this behavior on the latest version of the API (v2.15)
I checked with the team responsible for gift cards and they confirmed that the best way to check if a gift card has been successfully redeemed is to look at the RedeemedAt property. This property will be null when the gift card has not been redeemed, and it will have a DateTime when it has been successfully redeemed.
Prior to #343, there was not an easy way to check if a gift card was redeemed. Once this gets merged, you can check using giftcard.RedeemedAt == null.
Here's my example script in case you want to try it out yourself
            var giftcard = new GiftCard("HL3QDCT8AMRWJ4P9");
            var mightBeNull = giftcard.RedeemedAt == null;
            var surelyNull = giftcard.CanceledAt == null;
            Console.WriteLine("GiftCard Redeemed At == null? " + mightBeNull); // GiftCard Redeemed At == null? True
            giftcard.Redeem("x");
            Console.WriteLine("GiftCard Redeemed At: " + giftcard.RedeemedAt); // GiftCard Redeemed At: 9/28/18 12:39:51 PM

I'm going to close this issue once #343 is merged. If you're still experiencing problems, please re-open it and describe the steps to reproduce and we'll get it all straightened out. Thanks 😄


"Donate for afloat" opens repeatedly

After every boot, when I open application e.g Preview for the first time, it always opens this dialog:

I click open and nothing happens. When I restart machine, it happens again. It's quite annoying. Happens with other applications too, but not with every. E.g with TextEdit it does it also. I'm using Yosemite. and Afloat 2.1.1

Yep, happening here as well. Would love to know how to stop this from happening. Clicking on any option doesn't do anything and I even have afloat uninstalled and it keeps showing up.

Hi there,
I am curious… if you click on “Show Application” where does it take you?  Afloat, or perhaps SIMBL (or rather, whatever middleman you’re relaying on)?  I myself have the same Afloat I compiled ages ago (way before 10.9, possibly even earlier) and I have no such a popup.
f.
On May 14, 2015 at 13:28:13, AppWish<EMAIL_ADDRESS>wrote:
Yep, happening here as well. Would love to know how to stop this from happening. Clicking on any option doesn't do anything and I even have afloat uninstalled and it keeps showing up.
—
Reply to this email directly or view it on GitHub.
👤 Frank • gravatar.com/mrfrankus

When I click "Show Application" this is what it opens

If I look into my Xcode project folder, I do see Donate files in there, however none of these ended up in the final/installed bundle, which looks rather very different from yours indeed:
This is what my Info.plist looks like:
Could you send me a copy of your Info.plist from your installed bundle?
(You might need to rename the extension else it could get blocked)
I get a feeling there will be hooks in yours that enable the loading of the donation alert.
Note: I don’t want to be perceived as aiding in the disabling of donation nagware if this was intended to be part of the final product, so please don’t hesitate to chime in if I should refrain from pursuing this line of inquiry.
f.
On May 14, 2015 at 16:21:07, Matus Lestan<EMAIL_ADDRESS>wrote:
When I click "Show Application" this is what it opens
—
Reply to this email directly or view it on GitHub.
👤 Frank • gravatar.com/mrfrankus

My case, I completely deleted the afloat bundle, all afloat files I found in the hard drive, did a search for any "afloat" mentions inside of files, but I still keep getting this annoying donate for afloat window.

Same problem here...but happens not just when I reboot. Randomly happens during normal work.

Me to.. quite annoying. I would even donate to fix this... but nothing happens when I start the Donate app.
BTW: I installed Afloat using EasySIMBL buy following this guide and a comment here also showed a solution (I hope/think**).

Start EasySIMBL.
Click Show Plugin Folder and navigate to the Contents folder.
Right click Afloat.bundle and click Show Package Content.
Delete the Donate for Afloat app.
Navigate into the Resoure/it.lproj folder.
Delete Donate.nib, AfloatDonateFirstAlert.strings and AfloatDonateLastAlert.strings.

The content of the it.lproj and en.lproj folder now contains the same files (different sizes).
** I have not see the popup since. But I'm also not sure exactly when I got this popup. If it is not fixed of if I get other issues I'll promise to write here again. If I don't I consider this the solution :)

I was able to fix the issue, and it's kind of the same solution as @tjementum .
Even after deleting everything related to afloat, I still had EasySIMBL installed. I opened and found a tab in there where there was another instance of afloat.
I deleted and voila, no more annoying popup!

@tjementum and after this solution "Keep Afloat" functionality still works?

@madvas Yes... it still works. And I have not seen a popup in since.

Thanks @tjementum, works. Also worked for these guys here


Undefined method  `cleanup_change_column_type_concurrently'

I am currently following the README to update a column from integer to bigint. Having an issue undefined method cleanup_change_column_type_concurrently  when running the migration. Thank you.

@vonchristian Thanks for report. I apologize, this is not the actual method's name. You should use cleanup_column_type_change instead. Updated the readme.


Transactional Templates: examples of sending with dynamic data

Looking at https://sendgrid.com/docs/API_Reference/Web_API_v3/Transactional_Templates/index.html
Can't seem to find an example of sending an email using a template stored on SendGrid. Would be nice if it could explain how to supply dynamic data, for example how to send an invoice using a stored template and passing in a data object containing line items, total, shipping cost etc.
(Is that even possible with SendGrid currently?)

Hey @SimonEast - This is a question we get a lot! Check out our mail-send API documentation - https://sendgrid.api-docs.io/v3.0/mail-send - If you expand the properties in the Request data, you'll see an object for template_id, which is how you specify a transactional template, and then you'd use sections to have dynamic data. I hope that helps! If you are looking for help with your specific instance, you should reach out to our support team. You can access support contact options by logging into https://support.sendgrid.com.
I am going to close this issue because we are currently in the middle of an internal initiative to improve this doc (and the overall experience!) and I wouldn't want one of our contributors to duplicate our effort.

OK, sure. Thanks for the informative response. 😊

@SimonEast - We are actually looking for beta testers for our upcoming new transactional template functionality. Would you be interested in participating?
Some more info here: https://github.com/sendgrid/sendgrid-nodejs/issues/221

Ok good to know it's almost here, but unfortunately I'm not a SendGrid user (yet). Just researching possible options for an upcoming project. I don't think I'll be able to beta test at this stage.

Hey there! Quick update for this thread––
Dynamic content for Transactional Templates is here!
This morning, our team publicly launched dynamic content for transactional templates. It is now available for all customers sending over v3 of our Mail Send API. Iterate over lists, handle conditionals and more, thanks to native support for a subset of Handlebars syntax!
More information can be found in our blog post announcement.
Documentation:

How to Send an Email With Dynamic Transactional
Templates
Using Handlebars
API Docs

FYI this is currently being implemented here https://github.com/sendgrid/sendgrid-php/issues/648

None of the examples linked to above demonstrate how to send two emails to two different people with different data (e.g. different dyanmic_template_data fields). If someone could provide a code example I would appreciate it.

Hey @Joshandrews43! Thanks for the question. We have a few examples of some templates/use cases available (complete with test JSON data) here: https://github.com/sendgrid/email-templates/tree/master/dynamic-templates
Let me know if that doesn't help out! We're here for ya.


Fix tools that shows or spawn results in all views

Similarly to #3582, there are other tools that spawns or display results in all views.

Syntax error is displayed only once:

Print string is displayed only once:

Profile action:

Format action:


added dockerfile

use by building and then running:
docker build -t foaas .
docker run --name foaas -d -p 5000:5000 foaas

+1
I'd like to see foaas clustered on CoreOS using fleet in the near future

@SyntaxStacks fuck-off :smile_cat:

That's the spirit @bighappyface

:+1:

I meant http://foaas.com/off/john/david

:+1:

+1

:+1:


LWM2M: Access Control [v4]

Main changes from v3 (#2264):

Fixes suggested in the review (thanks @cabelitos and @edersondisouza)

Future Improvements

Add tests to test-lwm2m.c

Signed-off-by: Bruno Melo<EMAIL_ADDRESS>

@bsmelo Did you test the samples ?
They are not working anymore, if you try to run it you get the following error:
(This is the client's output)
Access Control object created at /2/0

<<[CREATE]<< acl[101]=16>>>>
Access Control object created at /2/1

WRN:sol-lwm2m ./src/lib/comms/sol-lwm2m.c:4020 handle_resource() r (-2) < 0

#2266 This will fix the issue

merged


Workspace namespace occurrences when publishing attachments

There are some instances of "pracovní-prostor" IRIs when publishing diagrams.

Attachments file (within a vocabulary) - both predicate and object within a triple.

Attachment file - main IRI, RDF type.

Seems ok to me - in milestone 1a the IRIs still refer to PP ontology, because we didn't move the properties into the vocabulary contexts. Or not?


Edge labels overlap each other

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[X ] Documentation issue or request
[ ] Other... Please describe:

Current behavior
Long edge labels in down direction (elk.direction = DOWN) overlap each other. I cannot see in documentation a solution for this. Is it possible to solve this problem, and to make long edge labels to not overlap?
Expected behavior
Labels do not overlap
Minimal reproduction of the problem with instructions
elk.direction = DOWN and long edge labels

What is the motivation / use case for changing the behavior?
Edge labels should not overlap, but for example be printed smartly by algorithm
Environment

Libs:
- react version: 17.x.x
- realayers version: 5.0.5

Browser:
- [X] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- 16.14.x
- Linux


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/42130

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/44224

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/48834

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/62252

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/71669

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/71671

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74440

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81230

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83170


Horizontal flatlist support?

Would it be possible to also expose support for horizontal FlatLists? Thanks!

All the positioning calculations are based on y positions and vertical offsets, so it would take a bit of work to convert those to x and calculate horizontal offsets. I don't have plans to tackle that at the moment.

Hey @computerjazz , so I actually messed it up and got it to work for horizontal. Basically replaced all the Y's for X's and all the heights for widths, and added a flexDirection: 'row' to the main view of the RowItem so that it renders the spacing horizontally. Happy to share that with you or fork your repo and put it up. What do you prefer?

@computerjazz Sounds like a PR on this repo would be perfect, perhaps simply by adding a direction="horizontal" prop, which then simply uses your version?

@ilyausorov I'd love to see what you did to get it working. If it's as easy as replacing a few variables with ternary statements I'd go that route: const someOffset = horizontal ? pos.x : pos.y. I don't want to split this into two separate components as that'd be a maintenance headache.

@ilyausorov thanks for that, I used your changes as a reference and updated the component to support horizontal lists in 1.1.0

Awesome, thanks man! I'll take the one on my profile down.


[bugfix] Fix #3763 support typescript module:none

adds tests for module:none in typing-tests
move existing typing-tests to typing-tests/commonjs

Coverage increased (+0.3%) to 94.778% when pulling 1dbc5f84bae6051dd988ace7faa3cabe56fc9f48 on rossipedia:fix-typings-for-no-module into 3062ca13cd1cfbcef1c4a0cd5543f5fcd329c1d9 on moment:develop.

Umm.... I don't think the travis failures are due to any of the code in this PR? Seems like it's only related to node v4 and v5? I don't think v4 is supported anymore.


Question: doStatePropsDependOnOwnProps

I'm curious why doStatePropsDependOnOwnProps is defined by checking length of function mapStateToProps. It causes weird issues with generated functions (ex. with pipe/compose) or ones with default arguments (see Babel transpilation results). Is it some kind of microoptimization or core concept?

Well, per https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/length , Function.length "specifies the number of arguments expected by the function".  connect() can accept a mapState function that only depends on state, or depends on both state and ownProps, and since in both cases the mapState argument would be a function, checking the number of expected arguments is a reasonable way to determine which use case it is.
I would find it surprising that someone would be supplying default arguments as part of their mapState function, and also probably uncommon to be wrapping/generating it in some way.  Can you provide some examples of such a mapState implementation, and some possible use cases?

I think I understand the code, but I wish to know the motivation. If the function doesn't use ownProps argument (does not declare it in its definition), then passing this to the function after state cannot break anything, am I right?
The case was pretty simple, I need component depending on params only. I can define mapStateToProps for it like that:
const mapStateToProps = (_, { params }) => params;

or like that:
import R from 'ramda';

const mapStateToProps = R.pipe(R.nthArg(1), R.prop('params'));

Ignoring the fact which approach is better, the second one does not work because of that function length checking.

The actual motivation is that if a connected component only needs generic-ish data from the store (such as state.someField, then connect will only re-run mapState when the store updates.  However, if a connected component needs to make use of its own props values in mapState, such as looking up the data for a specific item ID ( like state.someData[ownProps.itemID]), then connect will also need to re-run mapState whenever the parent of the connected component re-renders, in case the own props have changed (which could cause the result of mapState to change).  So, it's a performance optimization to only call mapState as much as is actually needed.
Your example confuses me a bit.  It sounds like you don't actually need mapState at all.  First, your component could be given, say, <ChildComponent a={1} b={2} c={3} />, but only bother to read and use a and ignore any other props.  Second, if you do really want to ensure that a component is only actually given a certain prop and ignores the others, just create a simple React wrapper component that does that.  mapState is irrelevant in that case.

Regarding my use case, I use react-redux-router along react-redux and that's because I need mapStateToProps with the second argument only, react-redux-router puts path params to ownProps. But it doesn't really matter, I just did want to talk about general issue.
Anyway, I see know what is happening based on doStatePropsDependOnOwnProps - quite a lot of. So I understand that nothing may change. And I consider my initial question as answered :)
Would you mind if this detail is included in documentation? I mean the requirement that mapStateToProps must be formally defined as binary function to receive ownProps.

The docs for connect do indicate that ownProps is optional, and mention that your mapState will be called more often: https://github.com/reactjs/react-redux/blob/master/docs/api.md#connectmapstatetoprops-mapdispatchtoprops-mergeprops-options
But yeah, doc PRs are always welcome if you've got improvements in mind.

Please, look at https://github.com/reactjs/react-redux/pull/417. I hope my english is good enough :)


Invalid link responds with HTTP 200 status code

Describe the bug
An invalid link shows correct in the web UI (An error occurred while loading the public link), responds with HTTP 200 status code.

Steps to reproduce
Steps to reproduce the behavior:

Create invalid link like in https://github.com/owncloud/ocis/issues/4380
Open invalid link in browser ✅
Check HTTP status code with curl

Expected behavior
I'd expect a HTTP error code, similar as in oC10
% curl -I "https://demo.owncloud.org/s/Lv742zlpi1K28C"
HTTP/2 404
access-control-allow-origin: *
cache-control: no-cache, no-store, max-age=0, must-revalidate, value
content-type: text/html
date: Tue, 23 Aug 2022 12:05:06 GMT
expires: Thu, 01 Jan 1970 00:00:00 GMT
last-modified: Tue, 23 Aug 2022 12:05:06 GMT
referrer-policy: strict-origin-when-cross-origin
request-id: cc2c4sg744t9ov8htbm0
strict-transport-security: max-age=315360000; preload
x-content-type-options: nosniff
x-errors-version: 0.0.0
x-frame-options: SAMEORIGIN
x-xss-protection: 0
content-length: 1828

Actual behavior
Invalid link responds with HTTP 200 status code:
% curl -I "https://ocis.owncloud.com/s/ItYmLwEXCgdwYmU"
HTTP/2 200
cache-control: max-age=604800, must-revalidate
content-security-policy: frame-ancestors 'self'
content-type: text/html; charset=utf-8
date: Tue, 23 Aug 2022 11:59:29 GMT
expires: Tue, 30 Aug 2022 11:00:05 GMT
last-modified: Tue, 23 Aug 2022 11:00:05 GMT
referrer-policy: strict-origin-when-cross-origin
samesite: Strict
strict-transport-security: max-age=315360000; preload
x-content-type-options: nosniff
x-frame-options: SAMEORIGIN
x-web-version: 2.0.0-beta.6
x-xss-protection: 0

Setup
https://ocis.owncloud.com/
ownCloud Web UI 5.7.0-rc.9
Infinite Scale 2.0.0-beta.6 Community

Additional context
Add any other context about the problem here.
@dragotin FYI

I added issue https://github.com/owncloud/core/issues/40313 in core to get some API test coverage of this - I don't see any tests that do this at present, so that is why we have not noticed this difference between oC10 and oCIS.

This one is tricky.
In oC10 the responses are rendered on the server side. That means that when requesting https://demo.owncloud.org/s/Lv742zlpi1K28C the server tries to lookup the share and when it doesn't find it, it returns the error page with the 404 status code.
oCIS is different though. Here the request to https://ocis.owncloud.com/s/ItYmLwEXCgdwYmU loads the static files (html, js, css, etc.) for the web app this returns 200 which makes sense, when the static files could be loaded. Then the web app sends a PROPFIND to https://ocis.owncloud.com/remote.php/dav/public-files/Lv742zlpi1K28C which returns a 404 because the share doesn't exist.
I argue that it doesn't make sense to have a 404 when requesting https://ocis.owncloud.com/s/ItYmLwEXCgdwYmU since the meaning is a different one.

okay


Fire event after saving address to have a chance to save address attr…

Description

Why is it necessary?
When creating a new customer through the backend address attributes are not saved nor there's a chance for extensions to catch the success/fail callbacks after saving the address model. This makes difficult saving custom address attributes on customer creation.

What does it improve?
This PR just fires an event after save success to give extensions a chance to intercept it and save custom address attributes if needed.

Does it have side effects?
This change has no side effects.

Questions
Answers

BC breaks?
yes

Tests pass?
yes

Related tickets?

How to test?
Please describe how to best verify that this PR is correct.

We need the same logic in other places:
-> Fire event after saving customer, site and blog to have a chance to save attributes on creation

If we need to move attribute fields into another form we don't have a chance to save attributes when we are creating new customer, site or blog;
There's no options argument in onSave methods, like we have in onSaveArticle, e.g. Where we can add logic to callback function;
Adding these fire events after saving, give us the possibility to extend the functionality more dinamically;

/cc @bcremer @dlobato @dnoegel

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-17567.
Please use this issue to track the state of your pull request.


One or two build systems?

Describe the feature you'd like supported
Now that we have MSI and nuget releases for developers, do we still need two different build systems (msbuild and cmake)?  Or can we get down to just one?
We added cmake back when other projects had to build ebpf-for-windows themselves, so they could build it with whichever of the two they already used.
But it's extra work to maintain both.
Proposed solution
Either stick with msbuild, or stick with cmake.
Remove the other one.
Additional context
No response

Between the two:

cmake can't sign driver binaries (issue #1142)
cmake doesn't currently build everything (issues #1018 #1190)

So going down to one would necessarily be msbuild today because of issue #1142.

@nibanks reports:

FYI, for MsQuic VS projects we explicitly turn off its signing logic. We don't want it to sign with some random test cert,  Same for XDP.  Then we use something like this: xdp-for-windows/tools/sign.ps1 at main · microsoft/xdp-for-windows (github.com)


Running Jupyter setup after initial setup results in "dotnet: command not found"

I ran setup-device-jupyter.sh immediately after a successful setup-device.sh, and got:
dotnet: command not found

I restarted and it worked, but maybe just a new/re-sourced terminal would have worked.

I think that this is leftover from when they were 1 script. Because we export the PATH variable in the first script, its scope is limited by the script itself, and does not make it to the current session.
We could re-source ~/.bashrc, or add a note about this.


Fix: Rails 4 upgrade; added 'bin' into source control.

Hi,
A fresh clone of promdash fails to start a rails server. After a bundle install an attempt to run bundle exec rails server will throw a usage error of rails new. I believe this is because the loader failed to find the appropriate files and thus deemed the root dir was not a rails app dir.
As per the comments in https://github.com/rails/rails/blob/master/railties/lib/rails/app_rails_loader.rb I've 'updated' to rails 4, by including these bin files.
Now I can run a local rails server.
These are my application versions:
$ gem -v
2.2.2
$ bundle -v
Bundler version 1.7.13
$ rails -v
Rails 4.1.7
OS: Ubuntu 14.10
If this isn't the root cause or an acceptable fix, feel free to ignore/reject this PR.
Stefano

@scharissis Thanks for sending this in!
@grobie We added the bin subdir to .gitignore a long while ago, and I think you were involved in that decision back then. What's your take on this whole checking-in-binstubs now? What's the "right" thing to do with Rails 4? In any case, if we don't check them in, we need to change the README.md instructions to generate them.

I'm not up-to-date with Rails conventions anymore.
Checking in binstubs sounds fine to me, I ran into the same issues.
Alternatively we'd need to document that bundle exec rake rails:update:bin is a necessary setup step. That could even be part of a make setup target.

Cool. Actually, the linked comments in https://github.com/rails/rails/blob/master/railties/lib/rails/app_rails_loader.rb seem to be clear. So this looks good :+1: Thanks @scharissis, merging.

@scharissis Oh, and feel free to add yourself to the AUTHORS file.

No problem!
Cool, I updated the docs a bit too and did this in a new PR: https://github.com/prometheus/promdash/pull/329

Woot, thanks a bunch!
On Mon, Feb 9, 2015 at 9:11 PM, Stefano Charissis<EMAIL_ADDRESS>wrote:

No problem!
Cool, I updated the docs a bit too and did this in a new PR: #329
https://github.com/prometheus/promdash/pull/329
—
Reply to this email directly or view it on GitHub
https://github.com/prometheus/promdash/pull/328#issuecomment-73580811.


changelog: Add entries for recent PRs #533, #537, #542

We should probably come up with a PR template using checkmarks - bit of an annoying barrier IMO, but I keep forgetting to even document my own contributions.  Much easier to do them up-front than to come up with this in hindsight.

I'm game for automated checking, so long as we can easily ignore it for stuff like internal refactoring/cleanup.

I'm game for automated checking

I don't think it's possible to mandate a change to Changelog.md automatically, but we could request users to tick checkmarks in a PR template reminding them of the steps needed.


[BUG] Videos open but cannot be played through file manager

Describe the bug
You can open videos from the file manager but they aren't played: instead you're only left with a single frame from the video, shown as a picture.
Opening the same video from gallery app itself makes it work as expected.
To Reproduce
Steps to reproduce the behavior:

Go to your file manager (i'm using LittleFileExplorer)
Open any video
See how the Gallery seemingly interprets it as an image

As opened through the Gallery:

As opened through the file manager:

Expected behavior
Gallery should recognize it as a video and provide the corresponding to control its playstate.
Smartphone (please complete the following information):

Device: Nokia TA-1157
OS: Android 11
Version 2.1.0

Additional context:
Might be linked to previously resolved #340

Fixed in #340


Update install script to create new profile

Hello I was wondering if the install.sh script could be updated to create new profiles for users?
For example, ./install.sh catppuccin might overwrite the existing profile catppuccin, or if it doesn't exist, create it in the same way the Default profile is currently made.
Use case: Setting up this theme from a non-interactive session, so clicking on gnome-terminal and creating the initial profile isn't available.

Closed with https://github.com/catppuccin/gnome-terminal/commit/8400cbed98eeefc08da5dd1456107da0d49fc665


Created just some udev rules and fitting systemd targets

What I did: create udev rules detecting

battery full and charging
general AC attached or "on battery"
battery low

They trigger systemd targets that can in further steps be used from the user to add powersaving or -unlocking scripts, disabling CPU cores, throttling clockspeed and so on. It should run fully automatic, which is really nessecary.
If there are syntax errors please correct them ;D

In the next step I would create systemd services for

fwupd (depending on battery-full.target)
batterysaver (depending on battery.target)
batterysaver-low (depending on battery-low.target)
batterysaver-undo (depending on ac or battery-full, discussable)

I think these should be in /etc to be user-configurable.
In general they apply to all DEs. Is there a difference what systemd services have to be enabled or not? They should not cause any unwanted problems, for example on Desktops, but you never know. They should be possible to disable and enable.
Also The services should trigger scripts the user is supposed to setup themselves, as measurements will vary a lot. We could gather some presets for different CPUs or GPUs for example.
TLP broke my ports on my new laptop, so this is not the simplest and best solution. Power-profiles-daemon (integrated in KDE) also does not work on my AMD laptop.
But there are many ways to improve batterylife. If TLP works for people they can use that, they can also disable cores, throttle speeds and more.

My first reaction is that automatic firmware upgrades sound unnecessary and dangerous. That seems like something that should require user consent/interaction as it can introduce bugs to devices that may have already been working fine.
That part I would be against merging as it can be problematic.
The battery stuff seems ok but too opinionated to be part of the upstream main image imo. I wouldn't want any of this stuff on my desktop system that doesn't even have a battery.
Having a custom image with all these changes applied seems fair game but I think that's better left off as a downstream consumer of the uBlue images

Yes I understand that. But as far as I know these services have to be enabled anyways, right?
Systemd services in /usr/lib/systemd are not enabled by default. They need to, by symlinking them to the /etc/systemd/ directory.
So yes having that may be annoying on Desktops, but

Its not enabled
It runs an empty script users have to setup themselves.

Where to put that script is discussable, I would say a root directory like /etc/ somewhere. And we could add many presets making power optimisation very easy.
The majority of users (uploading data to linux-hardware.org) uses portable devices for Fedora


ctc_greedy_decoder inconsistent with ctc_beam_search_decoder

The following extract from the the ctc_beam_search_decoder documentation seems to be misleading:
"The ctc_greedy_decoder is a special case of the ctc_beam_search_decoder with top_paths=1 and beam_width=1 (but that decoder is faster for this special case)."
Instead, the following results can be observed:

Decoding "AA<ctc_blank>AA" using
merge_repeated=True
merge_repeated=False

tf.nn.ctc_beam_search_decoder(top_paths=1, beam_width=1)
"A"
"AA"

tf.nn.ctc_greedy_decoder()
"AA"
"AAAA"

To reproduce:
import numpy as np
import tensorflow as tf
from unittest import TestCase

class CtcDecodersTest(TestCase):
    def test(self):
        def decode_greedily(beam_search: bool, merge_repeated: bool):
            aa_ctc_blank_aa_logits = tf.constant(np.array([[[1.0, 0.0]], [[1.0, 0.0]], [[0.0, 1.0]],
                                                    [[1.0, 0.0]], [[1.0, 0.0]]], dtype=np.float32))
            sequence_length = tf.constant(np.array([5], dtype=np.int32))

            (decoded_list,), log_probabilities = \
                tf.nn.ctc_beam_search_decoder(inputs=aa_ctc_blank_aa_logits,
                                              sequence_length=sequence_length,
                                              merge_repeated=merge_repeated,
                                              beam_width=1) \
                    if beam_search else \
                    tf.nn.ctc_greedy_decoder(inputs=aa_ctc_blank_aa_logits,
                                             sequence_length=sequence_length,
                                             merge_repeated=merge_repeated)

            return list(tf.Session().run(tf.sparse_tensor_to_dense(decoded_list)[0]))

        self.assertEqual([0], decode_greedily(beam_search=True, merge_repeated=True))
        self.assertEqual([0, 0], decode_greedily(beam_search=True, merge_repeated=False))
        self.assertEqual([0, 0], decode_greedily(beam_search=False, merge_repeated=True))
        self.assertEqual([0, 0, 0, 0], decode_greedily(beam_search=False, merge_repeated=False))

This is confusing and probably not intended.
How to solve this:

Adapt the documentation or
Adapt the ctc_beam_search_decoder implementation to that of ctc_greedy_decoder or vice versa. Both directions would cover my use case ("AA"), this decision would depend on which of the other behaviors ("A" or "AAAA") is needed and which of them could be dropped.

System information
OSX 12.4, TensorFlow 1.1.0 CPU from binary

I stumbled upon this today, too. Why are the results not fully equivalent when ctc_beam_search_decoder uses top_paths=1 and beam_size=1?

Ping!
@ebrevdo any comments?
I know there are a lot of issues assigned to you, so could you reassign this one who may have time to investigate this?

Also encounter this problem as well.
And test with this data set:
Logits: [ [[200.0,1.0]] , [[-100.0,200.0]] , [[200.0,1.0]] ]
Which should give 'AA',
The greedy decoder is correct, but the beam search decoder only give 'A'
And the beam search deocder work on this dataset:
[ [[log(0.4),log(0.6)]] , [[log(0.4),log(0.6)]]  ]
Greedy decoder gives an empty output, where the beam search decoder gives the true output  'A'.
Which is known would fail in best path greedy decoding.
But they both fail on this dataset:
[ [[log(0.4),log(0.6)]] , [[log(0.4),log(0.6)]], [[-200.0,200.0]] , [[200.0,-200.0]] ]
Where the true output is 'AA', but both decoders give 'A'.
But I guess they fail in the different place, beam_search decoder seems unable to give the correct answer for 'AbA'-type path, where greedy decoder failed in the first two logits.
To reproduce:
logits = tf.Variable(initial_value = [[[np.log(0.4),np.log(0.6)]],[[np.log(0.4),np.log(0.6)]],[[-200.0,200.0]],[[200.0,-200.0]]])
output= tf.nn.ctc_beam_search_decoder(inputs = logits,sequence_length=[4],top_paths = 3,beam_width = 500)
greedy_output = tf.nn.ctc_greedy_decoder(inputs = logits,sequence_length = [4])
sess = tf.Session()
sess.run(tf.global_variables_initializer())
decode_val = sess.run(output)
greedy_val = sess.run(greedy_output)
print decode_val,'\n',greedy_val

##System information
Ubuntu 14.04, Tensorflow 1.3.0 CPU from pip install.

same issue and look forward to solving that

any updates? having the same issue as well.

I spent a good fraction of the day debugging this before coming to a somewhat obvious realization. If you read the Graves paper, CTC beam decoding implicitly collapses repeated characters as part of calculating the optimal path (i.e. 'AAA' will contribute probability mass through the path 'A').
So the correct CTC decoding behavior occurs when merge_repeated=False. In this case, it DOES merge repeated characters. The merge_repeated flag, when true, will merge repeated characters after characters have already been merged/blank symbols removed. This parameter should simply be removed. As it stands now, the behavior is extremely misleading.
I'm happy to provide a PR for this change if it's something a tensorflower would be amenable with. cc @ebrevdo @gunan

Let's use the example from above: "AA<ctc_blank>AA"
Expected behavior:
when merge_repeated=False, "AA<ctc_blank>AA" should give AAAA
when merge_repeated=True, "AA<ctc_blank>AA" should give AA
Only tf.nn.ctc_greedy_decoder() is working as expected, but tf.nn.ctc_beam_search_decoder(top_paths=1, beam_width=1) is not.
@ryanleary are you saying tf.nn.ctc_beam_search_decoder(top_paths=1, beam_width=1) merged symbols twice? I think I might be able to help out in amending the decoder.

I agree that merge_repeated=True should give AA is the correct behavior. What I'm saying is that having a case where merge_repeated=False in a beam decoder doesn't make a whole lot of sense. Further, the merging is somewhat fundamental to the way the algorithm handles summing probabilities across multiple paths.
In the current implementation of the beam decoder, if merge_repeated=False, repeated characters are merged once. If merge_repeated=True, they are merged again, returning nonsensical results.
I'm advocating for removing the flag, and the only behavior to be where repeated characters are merged (but equivalent to the merge_repeated=False current functionality).

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 141 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 156 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 171 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 186 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 201 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 216 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Since this is a backwards breaking API change, I'll remove the argument for TF 2.0.  Thank you all for the report & detailed analysis.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Server Access logging in s3 log buckets

Extra test 7.18 checks to see if s3 buckets have server access logging enabled. I've set up logging in most of our s3 buckets, however the check is still failing on the buckets I'm actually storing all the logs in. Is it safe for those buckets to log server access, or would that lead to an infinite regression of logging the logging of the log... etc?
If I can turn logging on for those buckets, can they log to themselves? Or should they log to a different bucket.

Upon further research, it looks like logging the access to the buckets where the logs are actually being stored is discouraged. Is there a way to disable this check for specific buckets?

Can you please share where did you find that recommendation?

https://aws.amazon.com/premiumsupport/knowledge-center/s3-server-access-logs-same-bucket/

Awesome thanks for sharing that @rkane2002, @QuinnStevens  this is a good example of use case for whitelist support in Prowler. There is PR about that here https://github.com/toniblyx/prowler/pull/514 I need to test it and make it work with all new changes made. What do you think about that implementation?

Hey Toni! That looks perfect for our purposes! I do have one comment but I'll voice that on the discussion of the PR itself.

This has been fixed by the addition of the whitelist support in #514 so I'm closing.


Shortcuts being removed for related packages

from Discord:

I pack two applications (not channels, but really independent app) this way:
vpk pack -u JamLogic --packTitle "Ver-Mac JamLogic" [...]
vpk pack -u JamLogicDev --packTitle "Ver-Mac JamLogic Dev" [...]
I install both, they works fine.
But if I uninstall the first one, it also deletes the shortcuts on the Desktop/Start Menu for the second one.
The 2nd application remains installed (is in the windows app list, files are in the appdata directory, is still usable), but the shortcuts are gone.
Is it possible your search for shortcuts does a "string contains" instead of exact string matching?
Do you want me to create a github ticket with that?

Need to write some more tests for this.

Fixed in 73e81f2d5459c83496b21675709e81f95c1229bc


⚠ Avatar API Endpoint has degraded performance

In 00d21a7, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1158 ms

Resolved: Avatar API Endpoint performance has improved in 80a5677.


proposal: errgroup.Group-like Go method for sync.WaitGroup

When I use errgroup.Group, I noticed I need much less code compared to sync.WaitGroup in certain situations. While keeping the existing methods of WaitGroup, can we add a Go(func()) function to make it more seamless? (And while at it, we could even make the Go method return the WaitGroup itself for chained calls? I wouldn't insist on this but it would be nice instead of it returning nothing)
It will be like:
wg := sync.WaitGroup{}
wg.Add(1)
go func() {
    defer wg.Done()
    doWork()
}()

vs
wg := sync.WaitGroup{}.Go(func(){
    doWork()
})

or potentially:
wg := sync.WaitGroup{}.Go(doWork)

I feel like it's not a crazy far-fetched idea since errgroup.Group already has this method.
And this would be a nice addition for semaphore.Weighted as well, what do you think?

Could you explain more about the use cases here. From the example you gave, you’re running a function in a goroutine and waiting for it to finish — that’s the same as calling the function directly.

CC @bcmills

@davecheney of course that's not the whole usage :) I just didn't want to write more code than necessary to make it easier to read. I think it could apply any place you use WaitGroup where you're waiting other go routines to finish their computation.

If you could give some examples that were closer to your intended usage that might help make your proposal more compelling.

@davecheney
Perhaps the proposal is about aligning the WaitGroup API more towards ErrGroup's
wg := sync.WaitGroup{}
var urls = []string{
	"http://www.golang.org/",
	"http://www.google.com/",
	"http://www.somestupidname.com/",
}
for _, url := range urls {
	// Launch a goroutine to fetch the URL.
	url := url // https://golang.org/doc/faq#closures_and_goroutines
	wg.Go(func() {
		// Fetch the URL.
		resp, err := http.Get(url)
		if err == nil {
			resp.Body.Close()
		}
	})
}
// Wait for all HTTP fetches to complete.
if err := wg.Wait(); err == nil {
	fmt.Println("Successfully fetched all URLs.")
}

And this would be a nice addition for semaphore.Weighted as well, what do you think?

I would rather add a semaphore to errgroup (#27837) than add goroutine-management to semaphore.Weighted.

I feel like it's not a crazy far-fetched idea since errgroup.Group already has this method.

I agree that this would be a reasonable addition to sync.WaitGroup, but note that the analogous API in errgroup.Group is much harder for callers to get right due to the error return value. (Using a context.WithContext and sync.WaitGroup directly, it's much too easy to accidentally cancel the Context too early and store a secondary error that occurs as a result, instead of the intended primary-cause error.)
Since sync.WaitGroup doesn't deal with errors or return values, there is less (but still non-trivial) risk of mismatched-Add bugs at the call site.

See previously #23538 and #18022.

@davecheney This is a very common pattern in our codebase, say I want to run 3 things in parallel. It'll be this:
wg := sync.WaitGroup{}
wg.Add(2)
go func() {
    defer wg.Done()
    doThis()
}()
go func() {
    defer wg.Done()
    doThat()
}()
doThose()
wg.Wait()

vs this: (note that I'm not insisting on the chain call where Go could return wg itself)
wg := sync.WaitGroup{}.
    Go(doThis).
    Go(doThat)
doThose()
wg.Wait()

This is also less bug prone for the cases when people call wrong amount of Add or Done

I've used a similar pattern that passes the WaitGroup to ops which may run independently:
wg := sync.WaitGroup{}
wg.Add(1); go doThis(&wg)
wg.Add(1); go doThat(&wg)
doThose(nil)
wg.Wait()

I can't find the previous discussion, although I'm sure there is one.
The general problem with this kind of method is that go, like defer, is carefully designed to give control over evaluating some parts of the call - namely the function being called and the arguments - in the original goroutine and only running the call itself in the new goroutine. Any Go method can't do that - it has to take a func(), with the pre-evaluation left to the caller (if the caller remembers!). That makes it a lot more clunky, and is why we've encouraged not wrapping the go statement and instead simply using it.
As for the chained ("fluent") calls, we've avoided that pattern in the Go standard library so far. It's just not idiomatic Go at this point. If you want to remember the receiver, use a variable.
If the problem is forgetting the wg.Add to match wg.Done, it still seems like a vet check would be best (#18022). I'm not sure we ever did that.

Now I see that the previous discussion is #18022.
It still seems like we should try to figure out a good, accurate, no-false-positive way to check this in vet.

Based on the discussion above, I suggest we decline this issue and continue discussion of the vet check on the previously-filed #18022. Does anyone object to that?

Based on the discussion above, this seems like a likely decline.

No change in consensus, so declined.


tor-browser-bundle-bin 7.0.2 tabs crash

Issue description
After installing the tor-browser from nixpkgs/master and starting it, all tabs crash with the 'Gah! This tab has crashed).
Steps to reproduce
nix-env -iA pkgs.tor-browser-bundle-bin
rm -rf ~/.local/share/tor-browser
tor-browser (wait for tor to start)
go anywhere
Technical details

System: 17.09.git.f0ac08b (Hummingbird)
Nix version: nix-env (Nix) 1.11.13
Nixpkgs version: 17.09.git.f0ac08b927M
Sandboxing enabled: true

I can't reproduce this. What do you see when running from the terminal (it should print some information about what it is doing).

Jul 30 02:23:27.000 [notice] Bootstrapped 100%: Done
[Parent 9371] WARNING: pipe error (41): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, lin
[Parent 9371] WARNING: pipe error (44): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, lin
[Parent 9371] WARNING: pipe error (46): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, lin
[Parent 9371] WARNING: pipe error (48): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, lin

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0083,name=PBrowser::Msg_LoadRemoteScript) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006A,name=PBrowser::Msg_LoadURL) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0087,name=PBrowser::Msg_SetDocShellIsActive) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006B,name=PBrowser::Msg_UpdateDimensions) Channel error: cannot send/recv

###!!! [Parent][RunMessage] Error: Channel error: cannot send/recv

[Parent 9371] WARNING: FileDescriptorSet destroyed with unconsumed descriptors: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/file_descri

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0086,name=PBrowser::Msg_Destroy) Channel error: cannot send/recv

[Parent 9371] WARNING: waitpid failed pid:10539 errno:10: file /home/debian/build/tor-browser/ipc/chromium/src/base/process_util_posix.cc, line 268
0 migrated.
1501406609200   addons.xpi      WARN    Attempting to activate an already active default theme
1501406609200   addons.xpi      WARN    Attempting to activate an already active default theme
Jul 30 02:23:29.000 [notice] New control connection opened.
Jul 30 02:23:29.000 [notice] New control connection opened.

I actually get the same thing:
tor-browser
Fontconfig warning: line 145: blank doesn't take any effect anymore. please remove it from your fonts.conf
Jul 30 18:26:10.758 [notice] Tor <IP_ADDRESS> (git-22b3bf094e327093) running on Linux with Libevent 2.0.22-stable, OpenSSL 1.0.2k and Zlib 1.2.11.
Jul 30 18:26:10.758 [notice] Tor can't help you if you use it wrong! Learn how to be safe at https://www.torproject.org/download/download#warning
Jul 30 18:26:10.759 [notice] Read configuration file "/nix/store/qmr3fsdd39qiz6jc4qq9mp4c9jp3nisl-tor-browser-7.0.2/share/tor-browser/TorBrowser/Data/Tor/torrc-defaults".
Jul 30 18:26:10.759 [notice] Read configuration file "/home/nequi/.local/share/tor-browser/TorBrowser/Data/Tor/torrc".
Jul 30 18:26:10.764 [notice] Opening Socks listener on <IP_ADDRESS>:9150
Jul 30 18:26:10.764 [notice] Opening Control listener on <IP_ADDRESS>:9151
Jul 30 18:26:10.000 [notice] Parsing GEOIP IPv4 file /nix/store/qmr3fsdd39qiz6jc4qq9mp4c9jp3nisl-tor-browser-7.0.2/share/tor-browser/TorBrowser/Data/Tor/geoip.
Jul 30 18:26:10.000 [notice] Parsing GEOIP IPv6 file /nix/store/qmr3fsdd39qiz6jc4qq9mp4c9jp3nisl-tor-browser-7.0.2/share/tor-browser/TorBrowser/Data/Tor/geoip6.
Jul 30 18:26:11.000 [notice] Bootstrapped 0%: Starting
Jul 30 18:26:11.000 [notice] New control connection opened from <IP_ADDRESS>.
Jul 30 18:26:11.000 [notice] Starting with guard context "default"
Jul 30 18:26:11.000 [notice] New control connection opened from <IP_ADDRESS>.
Jul 30 18:26:11.000 [notice] Bootstrapped 5%: Connecting to directory server
Jul 30 18:26:12.000 [notice] Bootstrapped 10%: Finishing handshake with directory server
Jul 30 18:26:12.000 [notice] Bootstrapped 15%: Establishing an encrypted directory connection
Jul 30 18:26:12.000 [notice] Bootstrapped 20%: Asking for networkstatus consensus
Jul 30 18:26:12.000 [notice] Bootstrapped 25%: Loading networkstatus consensus
Jul 30 18:26:13.000 [notice] Bootstrapped 45%: Asking for relay descriptors
Jul 30 18:26:13.000 [notice] I learned some more directory information, but not enough to build a circuit: We need more microdescriptors: we have 72/6840, and can only build 0% of likely paths. (We have 0% of guards bw, 0% of midpoint bw, and 4% of exit bw = 0% of path bw.)
Jul 30 18:26:13.000 [notice] Bootstrapped 50%: Loading relay descriptors
Jul 30 18:26:14.000 [notice] Bootstrapped 55%: Loading relay descriptors
Jul 30 18:26:15.000 [notice] Bootstrapped 62%: Loading relay descriptors
Jul 30 18:26:15.000 [notice] Bootstrapped 67%: Loading relay descriptors
Jul 30 18:26:15.000 [notice] Bootstrapped 72%: Loading relay descriptors
Jul 30 18:26:15.000 [notice] Bootstrapped 80%: Connecting to the Tor network
Jul 30 18:26:15.000 [notice] Our directory information is no longer up-to-date enough to build circuits: We're missing descriptors for some of our primary entry guards
Jul 30 18:26:15.000 [notice] Bootstrapped 85%: Finishing handshake with first hop
Jul 30 18:26:15.000 [notice] We now have enough directory information to build circuits.
Jul 30 18:26:17.000 [notice] Bootstrapped 90%: Establishing a Tor circuit
Jul 30 18:26:18.000 [notice] Tor has successfully opened a circuit. Looks like client functionality is working.
 [Parent][MessageChannel] Error: (msgtype=0x2C0083,name=PBrowser::Msg_LoadRemoteScript) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006A,name=PBrowser::Msg_LoadURL) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0087,name=PBrowser::Msg_SetDocShellIsActive) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006B,name=PBrowser::Msg_UpdateDimensions) Channel error: cannot send/recv

###!!! [Parent][RunMessage] Error: Channel error: cannot send/recv

[Parent 3161] WARNING: FileDescriptorSet destroyed with unconsumed descriptors: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/file_descriptor_set_posix.cc, line 22

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0086,name=PBrowser::Msg_Destroy) Channel error: cannot send/recv

[Parent 3161] WARNING: waitpid failed pid:3273 errno:10: file /home/debian/build/tor-browser/ipc/chromium/src/base/process_util_posix.cc, line 268
Jul 30 18:26:20.000 [notice] New control connection opened from <IP_ADDRESS>.
Jul 30 18:26:20.000 [notice] New control connection opened from <IP_ADDRESS>.
[Parent 3161] WARNING: pipe error (47): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 322
[Parent 3161] WARNING: pipe error (60): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 322

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0083,name=PBrowser::Msg_LoadRemoteScript) Channel error: cannot send/recv

[Parent 3161] WARNING: pipe error (62): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 322
[Parent 3161] WARNING: pipe error (64): Connection reset by peer: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 322

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006A,name=PBrowser::Msg_LoadURL) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0087,name=PBrowser::Msg_SetDocShellIsActive) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006B,name=PBrowser::Msg_UpdateDimensions) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C006D,name=PBrowser::Msg_Activate) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0001,name=PBrowser::Msg_AsyncMessage) Channel error: cannot send/recv

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0001,name=PBrowser::Msg_AsyncMessage) Channel error: cannot send/recv

###!!! [Parent][RunMessage] Error: Channel error: cannot send/recv

[Parent 3161] WARNING: FileDescriptorSet destroyed with unconsumed descriptors: file /home/debian/build/tor-browser/ipc/chromium/src/chrome/common/file_descriptor_set_posix.cc, line 22

###!!! [Parent][MessageChannel] Error: (msgtype=0x2C0086,name=PBrowser::Msg_Destroy) Channel error: cannot send/recv

[Parent 3161] WARNING: waitpid failed pid:3345 errno:10: file /home/debian/build/tor-browser/ipc/chromium/src/base/process_util_posix.cc, line 268

I think I found a workaround, as per SeleniumHQ/docker-selenium#388

Hm, according to about:support multiprocess windows is enabled for me, but no crashing tabs. Oh, well ...

When e10 was new, there were addons that made it crash... But if you're not having this issue... hmmm... I tried turning off my firewall and anything security-related and even booted into a standard 4.9.x kernel, still crashing tabs.

@jcumming can you try
packageOverrides = super: {
  tor-browser-bundle-bin = super.tor-browser-bundle-bin.override {
    extraPrefs = ''
      lockPref("browser.tabs.remote.autostart", false);
      lockPref("browser.tabs.remote.autostart.2", false);
    '';
  };
}

per the fix suggested by @NeQuissimus ?

That fixes it, thanks!

In 7.0.3 tor-browser all tabs are still crashing.
Can be fixed with browser.tabs.remote.autostart = false,browser.tabs.remote.autostart.2 = false.
We need to set this by default.

@grahamc disagrees in #27781 :) I don't really swing either way...

This won't be the default without an explanation of why it is necessary and only for some users (i.e., not for me). Ideally, the underlying issue would be fixed instead, which is either some problem in the packaging or an upstream issue.

@joachifm It simply doesn't work out of the box. On fresh NixOS install after nix-env -iA torbrowser wil user get non-functional browser bundle. Then user needs spend time searching how to fix it. Why do we should this do every time? Why do users should do it again and again? Official bundle from tor-project doesn't work on NixOS. Tor-browsers users rely only on this nix package.

But it does work for me, I need more technical details to be convinced. The reason I'm hesitant is that silently disabling security features is undesirable, esp. for something like this.

Ok. Here we go. Here are some. Just tell me what else I can provide you.
Steps:

Install NixOS
nixos-install

Run GDM, GNOME Shell 3.22.0 Session, nouevau VideoDriver
$ systemctl start display-manager

Install tor-browser bundle package
$ nix-env -iA nixos.torbrowser

Run tor-browser
$ tor-browser

Open www.google.com in new tab

Gah. Your tab just crashed.

Screenshot: https://i.imgur.com/3vh66HH.png
Error log:  https://gist.github.com/ylwghst/056d7c760e37c345c3fdd3731c5c568b
System:
$ uname -a
Linux nixOs 4.9.40 #1-NixOS SMP Thu Jul 27 22:08:24 UTC 2017 x86_64 GNU/Linux
$ lspci
https://gist.github.com/ylwghst/339205274c162a6e8243618fac775869
X Window System
services.xserver = {
enable = true;
videoDrivers = [ "nouveau" ];
autorun = false;
layout = "us";
};
services.xserver.desktopManager.gnome3.enable = true;
services.xserver.displayManager.gdm.enable = true;

VirtualBox Machine
The same error reproduced in NixOS live cd system running in Virtual Box:
https://gist.github.com/anonymous/a57a0c4fa48d14469cdea2414b403c3a

I should clarify that I'm not doubting that the error occurs, I'm looking to understand whether reducing security for all users is the only viable solution.

@ylwghst thank you, I'll look through that for clues.

Now it is consistently reproducible for me as well, on both NixOS and non-NixOS. I'm leaning towards adding a toggle to enable the work-around, which will be on by default , despite being an awful non-solution.


updating breadcrumbs

adding a bc for microservices
removing old bc

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/_bread/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


#27 : Introduce image element in class - dtd updated

Changes to update class element
Introducing images elements

Thanks. Do we need to specify that images can only contain an image element?

Could you use three different images please? Some samples here:
https://github.com/DeepBlueCLtd/LegacyMan/tree/develop/data/Britain_Standalones/Content/Images

Could you use three different images please? Some samples here: https://github.com/DeepBlueCLtd/LegacyMan/tree/develop/data/Britain_Standalones/Content/Images

Could you use three different images please? Some samples here: https://github.com/DeepBlueCLtd/LegacyMan/tree/develop/data/Britain_Standalones/Content/Images

updated

Thanks. Do we need to specify that images can only contain an image element?

Yes

It was stated in issue

Sure, we require the images element that is at the top of the page. We don't require another images in the body.

I guess we could have a compulsory images element after summary, but then the children are optional. Actually - I like this approach, since the placeholder for the images will always be there when creating a new document.

Aah yes, sorry for the confusion.  I meant the existing one,  before the summary.

[ ] Please make the children of body compulsory - so they are already present in new documents, and when we port the legacy data.  On publish we'll have to omit the element if it is empty.

Aah yes, sorry for the confusion. I meant the existing one, before the summary.

[ ]  Please make the children of body compulsory - so they are already present in new documents, and when we port the legacy data.  On publish we'll have to omit the element if it is empty.

Changes done, output generated and update.


OSX ARM migration.

Checklist

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

Not exactly sure how you enabled the osx build for arm, it should have involved adding some keys to the conda-forge.yml file.
There is a migration bot which handles enabling that build environment for this feedstock. You just have to open a PR to add this package to this text file. You'll still have to manually update the build script and recipe, but the feedstock infrastructure will be correctly configured.
Ideally, if you're trying to enable this build, you have a local arm-osx machine that you can use to debug the build script.

Hi @carterbox , I followed the instructions given in the conda-forge doc. Is that redundant with this text file?
I would like to build PennyLane with Conda-Forge's Kokkos and Kokkos-kernels. We can currently build PennyLane for osx-arm64, which includes Kokkos and Kokkos-kernels (SERIAL backend only however) through CMake. @leofang tested on a local machine that the build works and can run some code, but I have to say I don't have directly access to an osx-arm machine. I thus wanted to start migrating Kokkos and Kokkos-kernels to support osx-arm to maintain support on all platforms for PennyLane.
In commit 7f184f6 it seems everything compiled fine, but the tests failed because the CPU type is incorrect. I was thus trying to turn testing off in the osx-arm64 case. Could you suggest a clean way to do it (for example # [not osx-arm64] or something like that)?

I followed the instructions given in the conda-forge doc. Is that redundant with this text file?

The text file is mentioned in the blog post you have linked:

All the below changes will be done by a bot and the packages the bot will send PRs to is determined by the list of packages at conda-forge-pinning and their dependences. If you would like to add support, please send a PR adding the feedstock name to the above list. After that PR is merged, you can monitor the status at conda-forge status-page and if a particular PR is stalled you can send a PR to the feedstock to fix it.

Please let the bot start this PR because some steps from the blog post are missing from your PR, and there may have been additional undocumented steps added to the migration process since 2020-10.

Please let the bot start this PR because some steps from the blog post are missing from your PR, and there may have been additional undocumented steps added to the migration process since 2020-10.

Thanks for pointing this out. I went over the first and most relevant paragraph too quickly.

Also, any chance we could have a Kokkos-3.7.01 osx-arm64 build? Or will migration only work for the latest release?

The migrator would only work for the latest release. What we need now is

@conda-forge/kokkos to create a new branch based on the migrated main
rebuild 3.7.01 on the new branch

This way we can maintain two versions. I'd suggest @vincentmr to join as maintainer (just open an empty issue with the command @conda-forge-admin, please add user @vincentmr and the bot would open a PR) so that you have access do the work 🙂 Happy to help review.

This way we can maintain two versions. I'd suggest @vincentmr to join as maintainer (just open an empty issue with the title @conda-forge-admin, please add user @vincentmr and the bot would open a PR) so that you have access do the work  Happy to help review.

Do you mean I, or the current maintainers have to @conda-forge-admin, please add user @vincentmr ?

You can do that.

It seems Daniel has done that for you 🙂 https://github.com/conda-forge/kokkos-feedstock/pull/14


Default guidelines

Two feature requests:

I was disappointed to find that there aren't any default guidelines, at least not that I could find (only the file context menu, which doesn't seem like it would remain for my next solution/project). It would be nice to be able to add a default guideline that was always available.

For years I've wanted dynamic guidelines for comments (or all text, if that's easier). "Dynamic" meaning the guidelines start at the line's indentation. I.e., if your comment is indented, the beginning of the guideline is as well. It would help a lot for making readable comments that are nested in methods or statements and/or automatically wrapping comments (via other extensions) while maintaining the same width when indented, instead of squishing nested/intended comments when wrapped at the "standard" 80 characters.

@josh-endries Thanks for the suggestions. I think each has merit and they're worth discussing separately. But please split this into two separate issues. Let's keep this one (edit the title and description) for the "default guidelines" and please create a new issue for "dynamic guidelines".

I moved the dynamic idea into #10. I can't figure out how to change this issue's name, though...

@josh-endries thanks for the suggestion. Guidelines are "global" -- once you set them in one text view, they'll show up in every other text view. And they persist across sessions. Are you perhaps thinking of a feature like #2 ?

I think this (default guidelines) is covered by the .editorconfig support. Please re-open if you have any further suggestions for this feature.


Contract is Not Shown In Order Details

When the contract link in the order details modal is clicked, the contract is loaded from the API, but is not appended to the DOM.

Are there any errors in the JS console?

file:///C:/Users/J/ob/openbazaar-desktop/js/views/modals/orderDetail/Contract.js
Uncaught TypeError: Cannot read property 'set_show_to_level' of undefined
at C:/Users/J/ob/openbazaar-desktop/js/views/modals/orderDetail/Contract.js:78:19
at sendBackTmpl (C:/Users/J/ob/openbazaar-desktop/js/utils/loadTemplate.js:25:5)
at loadTemplate (C:/Users/J/ob/openbazaar-desktop/js/utils/loadTemplate.js:44:5)
at _class.render (C:/Users/J/ob/openbazaar-desktop/js/views/modals/orderDetail/Contract.js:73:5)
at _class.selectTab (C:/Users/J/ob/openbazaar-desktop/js/views/modals/orderDetail/OrderDetail.js:290:17)
at _class.onTabClick (C:/Users/J/ob/openbazaar-desktop/js/views/modals/orderDetail/OrderDetail.js:160:10)
at HTMLDivElement.dispatch (C:\Users\J\ob\openbazaar-desktop\node_modules\jquery\dist\jquery.js:5201:27)
at HTMLDivElement.elemData.handle (C:\Users\J\ob\openbazaar-desktop\node_modules\jquery\dist\jquery.js:5009:28)

This is because the reporter of this bug blew away this line <script src="../js/lib/renderjson.js"></script> 24 days ago... lol:
https://github.com/OpenBazaar/openbazaar-desktop/commit/f527597842b38bbe25c36cb42d204f16747e7e72

Where's that shame emoji?


Transactions display is delayed after onboarding a new user.

Problem Description

New users logs in.
Pushes user to onboarding.
Onboard flow works as expected.
User connects bank account.
Clicks final next button.
Redirection to dashboard.
Transactions take ~20 seconds to display or they don't display at all unless refreshed.

Dashboard component.


hover colour of slack and github icon in footer is changed

Signed-off-by: shubham-tyagi<EMAIL_ADDRESS>This PR fixes #591
Notes for Reviewers
Please point out if I missed something or if anything else is needed to be done.
Signed commits

[x] Yes, I signed my commits.

LGTM🎉


Unable to build 0.31.1-dev

On mac os, building for android fails with error:
Compiling bin/build_tool_runner.dart to kernel file bin/build_tool_runner.dill.
WARNING: Precompiled binaries not available for crate hash 1500beb4ff7cea0ed6ba27ed696effb7 (aarch64-linux-android_libbdk_flutter.so)
WARNING: Precompiled binaries not available for crate hash 1500beb4ff7cea0ed6ba27ed696effb7 (i686-linux-android_libbdk_flutter.so)
WARNING: Precompiled binaries not available for crate hash 1500beb4ff7cea0ed6ba27ed696effb7 (x86_64-linux-android_libbdk_flutter.so)
INFO: Building bdk_flutter for aarch64-linux-android
SEVERE: ================================================================================
SEVERE: Cargokit BuildTool failed with error:
SEVERE: --------------------------------------------------------------------------------
SEVERE: External Command: rustup "run" "stable" "cargo" "build" "--manifest-path" "/Users/joaolobo/.pub-cache/git/bdk-flutter-6be18e8a105e12533b443f535b669b821c0fba5d/rust/Cargo.toml" "-p" "bdk_flutter" "--target" "aarch64-linux-android" "--target-dir" "/Users/joaolobo/code/project/satsails/build/bdk_flutter/build"
SEVERE: Returned Exit Code: 101
SEVERE: --------------------------------------------------------------------------------
SEVERE: STDOUT:

....

FAILURE: Build failed with an exception.

* Where:
Script '/Users/joaolobo/.pub-cache/hosted/pub.dev/bdk_flutter-0.31.1-dev/cargokit/gradle/plugin.gradle' line: 59

* What went wrong:
Execution failed for task ':bdk_flutter:cargokitCargoBuildBdk_flutterDebug'.
> Process 'command '/Users/joaolobo/.pub-cache/hosted/pub.dev/bdk_flutter-0.31.1-dev/cargokit/gradle/../run_build_tool.sh'' finished with non-zero exit value 1

this issue only seems to be happeinng for macbook

fixed!


not connected orsomething??

Hello i installed mumble-voip a couple times
and evertime i have problem with it like you can't hear people on radio
voice chat is very sillence by everyone and now this problem i followed all steps
how to fix this one?
https://gyazo.com/c17bad434900dbd9c96f77f9c8997b59
https://gyazo.com/ff58325601296d4dd9e85abca548e35a

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

Quick question: Why does the new version require OneSync?

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

I fixed it! the onesync is now true only problem i have is just the radio don't show up & doens't work with /radio

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

Quick question: Why does the new version require OneSync?

This script has always required onesync, fivem's mumble implementation on works if onesync is enabled

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

I fixed it! the onesync is now true only problem i have is just the radio don't show up & doens't work with /radio

https://github.com/FrazzIe/rp-radio#note

How can I get ESX to work with onesync

Hello, So it says. Mumble Initialising, Warning Not Connected to mumble.
I have enabled oneSync and it did nothing

Mine does not work i have one sync enabled and i have myself connected to my mumble server but it still says not connected on my fivem server, ive tried everything even setting my public ip and the port of the server in the "use external server" option in then config still no luck im trying to get gcphone, phone calls to work any help will be loved. thank you.

I also have set onesync on in my config and it says not connected

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

good afternoon sir i been using mumble voip and i dont face any problem at all until yesterday my players complaining that the mumble is not connect as the photo shown below.. my onesynce are all enable as shown on photo number 2

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

good afternoon sir i been using mumble voip and i dont face any problem at all until yesterday my players complaining that the mumble is not connect as the photo shown below.. my onesynce are all enable as shown on photo number 2

I have the exact som problem right now, and it happened out of nowhere.
You found a fix?

you don't need the setr stuff anymore, it's handle in the config file.
Are you sure onesync is enabled, that warning only shows up if your voice chat is disabled, or onesync is disabled or your external server is offline

good afternoon sir i been using mumble voip and i dont face any problem at all until yesterday my players complaining that the mumble is not connect as the photo shown below.. my onesynce are all enable as shown on photo number 2

I have the exact som problem right now, and it happened out of nowhere.
You found a fix?

have the same problem any fix yet?


Docs: Fix menu link to React integration

Already addressed by https://github.com/TanStack/router/pull/608


example expects z.infer but for parameters, it should be z.input

If a parameter has a .default then an example is legal input as a param even if it doesn't supply the parameter. However this won't compile because example is declared using z.infer<...> like
openapi<T extends ZodTypeAny>(this: T, metadata: Partial<ZodOpenAPIMetadata<z.infer<T>>>): T;

If instead it used z.input<T> then it would compile. For example output however, this is correct, it should be z.output or z.infer which are equivalent.  Possibly this needs a new exampleInput: field? I'm not sure of all the implications.

@svachalek thank you for using @asteasolutions/zod-to-openapi. I don't think I fully understand what the problem is. Would you be able to give me full code examples of what compiles and what doesn't. Thanks in advance

After discovering today that next to z.infer there is also z.input and z.output. Where z.infer is the same as z.input.
https://zod.dev/?id=type-inference
I ran to the exact issue @svachalek is talking about.
The example property of openapi is using the z.output or z.infer to get the type.
But for input types (like api Input data) the z.input would be better.
Hereby a full code example that shows the issue
import { extendZodWithOpenApi } from '@asteasolutions/zod-to-openapi';
import { z } from 'zod';

extendZodWithOpenApi(z);

const example = {
  requiredField:'required'
};

z.object({
  optionalField:z.string().default('defaultValue'),
  requiredField:z.string()
}).openapi({example});
// Typescript errors here because of missing the optional field.

Would it work if the type of example is a union of z.input and z.output?

@tionkje I merged your PR but made the change to only use z.input as this is what we handle with the generation as well. It is now available as part of v6.4.0