Don't change the Server Socket SendTimeout or ReceiveTimeout

Doneness:

[ X] Build is okay - I made sure that this change is building successfully.
[ X] No Bugs - I made sure that this change is working properly as expected. It doesn't have any bugs that you are aware of.
[ X] Branching - If this is not a hotfix, I am making this request against the master branch

On server connection - when you download/upload on the Internet - it's really faster to send/receive data with the default 64k buffer. If you try to change the SendTimeout/ReceiveTimeout socket values, it will have worst performances.

Okay. But your title says about the timeout, in the code, you changed the buffer size. Anything missing?

Wront Title sorry, it's SendBufferSize and ReceiveBufferSize


update controller runtime

What type of PR is this?
/kind dependency

What this PR does / why we need it:
Update controller runtime.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This is failing due to
Does this PR introduce a user-facing change?

NONE

Opened this up to demonstrate the failures.
Discussion here: https://github.com/kubernetes-sigs/controller-runtime/pull/2681

@alculquicondor @tenzen-y any ideas on this one?
Trying to add the patch interceptor but it is still saying patch is not allowed.
Do I need to use a different patch option?

Trying to add the patch interceptor but it is still saying patch is not allowed.

Where? Can you expand?

I just mean that I am still hitting the same failures that the unit tests are getting.
I can reproduce the failures with just the workload_test.go so I use that to do a quick test.

@trasc @alculquicondor I'm not too familar on SSA so @alculquicondor said that they will take this over.
/close


Git Push creates new branch on remote instead of pushing upstream

Hello there,
I'm not sure if mybe I did something wrong, but I have the following issue:
My local git is set up to have two remotes:
origin
backup
I have two branches locally:
development
release
And they are set eahc with an upstream:
development: backup/master
release: origin/master
Let's say Im currently on the release branch and want to do a git push from the console, so I do "git push" and everything gets pushed to "origin/master" as it's set as upstream.
If I try to push from Atom I get to select between [origin] or [backup]. As soon as I select one of them, let's say origin, the stuff gets pushed to origin/origin, creating a new branch calles like the remote.
Anyone any thoughts on that?

I'm sorry I didn't respond to this sooner. I haven't noticed this and I can't replicate it now. Is this still the case?


preload header support

I tried npm run dev as well as node build; node build/index.js and didn't see the link preload header in the hn.svelte.dev sample app that's available on https://hn.svelte.dev/top/1 today

I looked around for definitive information about whether the HTTP header is better than including the tags on the page. The best I could find was Addy:

Preloading via headers may offer a very negligible improvement, but require the document to be committed before processing happens.

In other words it probably isn't worth the additional complexity, so I'll close this


bug: ...

This link is broken-
http://my.dappnode/#/stakers/gnosis
thank you

Hi @fiatcrusher , it is not a broken link. You suppose to open this link from your dappnode machine's browser.


Fix "lxc file push  //"

Hi,
Why did you close this pull request?

There was some CI problem and it was not signed-off.
I will raise it again today.
On Mon, Apr 13, 2015 at 10:18 AM, Stéphane Graber<EMAIL_ADDRESS>wrote:

Hi,
Why did you close this pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/lxc/lxd/pull/497#issuecomment-92207621.

Ok.
You can actually update an existing pull request by pushing to the branch and overwriting it (--force flag to git) but re-submitting is fine too. Thanks


Check GSC for any potential issues

GSC for ggi.
For now, I need to way a day. I might have to install robots.txt and a sitemap.

I'm closing might-do, non bug tasks.


Support for internal types and "explicit API" mode in library projects

Version: 1.6.15-beta
I have a library project that I want to use Destinations with, but all of my composables marked with @Destination have internal visibility and many of the arguments I'm passing to the destinations are also internal types.
Because the generated code does not explicitly specify visibility, it defaults to public leading to compilation errors that a public class is exposing an internal type.
I'm not sure if this would work in all cases, but perhaps the generated types' visibility could be made to match the type that it's sourced from. For example:
Source:
@Destination
@Composable
internal fun HomeScreen()

Generated:
internal object HomeScreenDestination

A related problem is when "explicit API" mode is enabled for Kotlin. It causes compilation errors due to the generated types not specifying their visibility explicitly.
I believe adding explicit visibility modifiers to all generated types based on the source type's visibility could resolve both issues.

Hi @sarathijh
Yeah this seems like a nice improvement. Thanks for reporting 🙇
I'll try to check this soonish.

Hey @sarathijh and @alexzaitsev 👋
This just got merged and released. Please let me know how it goes as my testing here was limited. I may have missed some detail where I forgot to add either explicit type, visibility or such, but I believe it should be done.
Let me know either way though! Thanks!

Thank you @raamcosta! how this can be used? any documentation?

Nothing especial. Internal will be used if the annotated Composable is internal. And all generated types will be using explicit types and visibility always.

Is it possible to disable this or at least configure each destination? Because each of my composables are in a separated module and internal, I cannot create a navigation-like class in my app module. I want to expose the destinations, not the composables.

What do you guys think should be the default here? I'm kinda torn 🤔
In one hand, it was always like this (visibility of the Composable was not taken into account), on the other hand, I feel like it makes sense and is a good default behaviour here.
At the same time, if by default, I copy the visibility of the Composable functions, then some users might have errors and they won't know how to fix. While if the default is making Destinations public, then the more "advanced" use case of needing to make them internal should be a rarer occurence which justifies looking up on documentation.

I think by default destinations should be public as it's the more popular version. I'm personally taking advantage of the fact that my composables can be internal, thus I'm only exposing a simple generated object from a whole feature module.


Scoping on javascript variables definition

While inspecting scopes for Javascript, if a variable is defined after let or var, its scope will rightly be within meta.definition.variable, while defining an additional variable after a comma results in that variable lacking the meta.definition.variable scope, making it, scope wise, indistinguishable from any other readwrite variable.
Is this the intended behavior?

VSCode Version: 1.12.2
OS Version: macOS Sierra 10.12.4

Questions about the grammar should be directed to https://github.com/Microsoft/TypeScript-TmLanguage


in PointConv implementation, where is the T-net, or feature transform, or Joint Alignment Network?

🚀 Feature
In origin Pointnet paper, there's two Joint Alignment Networks.
In the classification roadmap:
the input -> input transform -> the first MLP layer -> feature transform -> the second MLP layer -> maxpooling layer -> MLP layer -> k output scores
In examples/pointnet2_classification.py,
where the 54 line: self.sa1_module = SAModule(0.5, 0.2, MLP([3, 64, 64, 128]))
where the 43 line: def MLP( ....
where the 45 line: Seq(Lin(channels[i - 1], channels[i]), ReLU(), BN(channels[i])
There's no clue that the code implementing the Joint Alignment Networks in the original paper
where is the Joint Alignment Networks in the pointnet++ classification/segmentation implementation, in examples/pointnet2_classification.py and examples/pointnet2_segmentation.py
or
for some reason, there's no need to implement the Joint Alignment Networks
why ignore them? for time efficiency?

It is not integrated. Especially on ModelNet10 this shouldn‘t be necessary due to already aligned input models. However, it should be quite easy to integrate this.

by the way, I found only SSG of pointnet++ is implemented. MSG and MRG is not implemented
As the original paper PointNet++: '3.3 Robust Feature Learning under Non-Uniform Sampling Density', mentioned, MSG and MRG is useful for Non-Uniform Density of point cloud
maybe I will implement MSG for my own dataset, and make a pull request.
I'm also implementing the Sementic3D dataset inherits from PyG Dataset class
Thanks for the excellent PyG work !

Looking forward to your contribution :)


Simpler Databases Not Working

What is happening?
the menu shows up and the toggles physically work, but they dont have any effect and once you open the menu, you cannot close it or click on anything else in notion until you refresh the page.
What should be happening?
i should be able to use the extension :')
Screenshots (if applicable):

Operating system/browser name & version:
Win 10, Opera GX
notion-enhancer and/or notion-repackaged version:
notion-enhancer v0.11.0
notion-enhancer configuration (enabled/disabled mods and their settings):
mods do not have an effect on this issue, it happens even with all mods disabled.

Have there been any updates on this?

Having this issue as well on Windows 10 and Chrome.

Having this issue as well on Windows 10 and Chrome. Just started happening 2 days agao.

I'm having the same issue. It starded happening 2 days ago.
I also tried to reinstall notion enhance without success.
How to resolve?

Same issue with me. I'm on Windows 10, Notion v0.11.0. Much thanks!

I'm also having this issue on both MacOS Monterey desktop client and Opera GX with the Chrome extension. Clicking the wrench icon to modify the settings will freeze the menu and I have to completely close the app and reopen it for it to go back to normal. The Simpler Databases modifications do not show at all anymore on any of my pages. Been going on for about a week now. Tried to completely uninstall and reinstall the app to no success. Other parts of the Enhancer app (like font choice) work without issue. It's just this.

The same problem. win11 21H2.

Same issue. I'm using macOS Monterey, Notion Desktop v2.0.21 (updated just a day ago). The feature was actually one of the main reasons why I decided to try the enhancer :((

Notion freezes when clicking the cogwheel, requiring a restart.

Broken on linux desktop0 and chrome for me too :(   Any updates on this?

+1, would appreciate a fix here

Here's a screenshot of the menu that cannot be closed. It's was taken from the app running on Windows 10.


Eulerian path

[x] PR's title includes the challenge's name and language
[x] PR has short and clear description of the challenge
[ ] PR has appropriate labels and milestones for easy identification
[ ] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[ ] the branch is up to date with main/master
[ ] the code works when pulled and run locally
[ ] all conflicts are resolved (if any)
[ ] It is linked to an issue in the appropriate column of the project board
[ ] feedback is addressed (if any, and if it is appropriate feedback.)

README Documentation

[x] The solution is documented in /src/README.md
[x] The markdown source is formatted
[ ] Spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[ ] All links and images work
[x] The README documents the solution's behavior, strategy(ies) and implementation(s)
[ ] There are use cases in code blocks to illustrate the function's behavior

Python Files

[ ] There is a module header
[x] There is a module docstring
[x] File names are in snake_case
[x] Test files are named test_<module_name>.py

Function Docstring

[ ] Behavior description
[ ] Parameter description
[ ] Return value description
[ ] Include any assumptions
[ ] Include 3 or more (passing!) doctests
[ ] Include 1-2 use cases (if necessary)

Function Implementation

[ ] The solution is not is not copied from djeada
[ ] The code is auto-formatted
[ ] The code has no (reasonable) linting mistakes
[ ] Variables are named with snake_case
[ ] The function has a clear and helpful name
[ ] The file's name matches the function name
[ ] The code follows the strategy as simply as possible
[ ] Variable names are clear and helpful
[ ] Comments explain the strategy (if necessary)
[ ] There are type annotations
[ ] (challenge) The code includes defensive assertions

Unit Test Suite

[ ] The test class has a helpful name in PascalCase
[ ] The test class has a docstring
Each unit test has

[ ] A helpful name
[ ] A clear docstring
[ ] Only one assertion

[ ] All tests pass
[ ] There are tests for defensive assertions
[ ] There are tests for boundary cases
[ ] Test suite includes black-box and glass-box tests

I used three files for coding this problem, the main file eulerian_path.py, also a file that the helper functions I used is_connected.py and the test file test_eulerian_path.py

Your current code is not passing the testcases already present in the file. Also, kindly update your branch with the main branch and then add your code.

The unittest is now fixed


Travis CI config

To add Travis CI configuration

Hi @akuzminsky, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!


Update Alpine Fault (Fiordland to Kaniere) conditional rates

Alpine Fault conditional rates in original CSHM have outdated underlying data - incorporate new underlying data and update source models.

Current consensus from discussions late 2019 are that the AlpineF2K value should be replaced by one consistent with Biasi et al. (2015) - 27% in 50y == recurrence interval of ~160.
Biasi G.; Langridge R.; Berryman K; Clark K.; Cochran U. 2015. Maximum-Likelihood Recurrence Parameters and Conditional Probability of a Ground-Rupturing Earthquake on the Southern Alpine Fault, South Island, New Zealand. Bulletin of the Seismological Society of America, 105:1. doi: 10.1785/0120130259

Current consensus from discussions late 2019 are that the AlpineF2K value should be replaced by one consistent with Biasi et al. (2015) - 27% in 50y == recurrence interval of ~160.
Biasi G.; Langridge R.; Berryman K; Clark K.; Cochran U. 2015. Maximum-Likelihood Recurrence Parameters and Conditional Probability of a Ground-Rupturing Earthquake on the Southern Alpine Fault, South Island, New Zealand. Bulletin of the Seismological Society of America, 105:1. doi: 10.1785/0120130259


[Resources] Update Microsoft.Resources swagger to match service behavior (Phase 2)

Add provisioningOperation property to DeploymentOperation object. This property is already being returned by the service in response body however was not part of the Resources swagger.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@njuCZ could you merge please? Approved by ARM.


Update app.js

typo in first line - 'docuemnt' instead of 'document'

Thanks for submitting the PR @timog !


Fixing phpMyAdmin FPs

fixed 2 real-world FPs
fixed ruleRemoveTargetById ordering in rule 9008110 (based on removed IDs)
adding new rule (was triggering for example for information_schema DB)

Thank you for this PR @azurit. Let's review this and get it merged. Taking it to the meeting on Monday.

Decisions from the August project meeting: @lifeforms will review and merge this.

LGTM!


Add an option to hide from player commands ran as rewards

What does your feature do?
It would be great to have an option in the config or inside the rewards editor of a quest to disable this output ( that shows when we finish a quest )

How would people use it?
It would be used because some commands don't have any interest to be shown to player because they already have a feedback included in them.

Use the override under the Edit Rewards prompt.


argument --precode causing display of help when -o is also specified

As the label says. Using a pipe instead of -o appears to work though.

I was wrong. The issue with with the order of the arguments.
If --precode is the last before the input markdown it fails. It must be at latest one before the penultimate.

Is this still a problem? What exact command causes this? Remember -o can optionally take an argument.


Bubble up server side transport errors to the client

I think https://github.com/google/tarpc/pull/399 is a step in the right direction, however currently if the server fails to handle a request from the client, the client is never notified, the request simply times out.
For example an out-of-date client can send the wrong message to the server, causing a serde deserialization error. This is logged on the server side, but the client is never notified. I'd expect the client to get the error message as well.

@bruwozniak @tikue

Yes, agreed! Though this would be a larger change, and I think there some open questions about what would even be a reasonable thing for the server to do in this case.
For example, to preserve the integrity of the TCP stream, the server would need to read all the remaining bytes for the request, to /dev/null or something. This would be a change that needs to be supported at the transport level rather than the application level, and the serde transport commonly used doesn't support this capability.

It's true that it's definately not something all transports can support. I'm using tarpc over MQTT which is message-based, so continuing the stream with the next message is not a problem, but in case of a TCP stream it may or may not be able to be easily implemented without risking the integrity of the connection.
But we need to differentiate between incorrect and invalid requests. The former - in theory - shouldn't be a problem even for a stream-based transport. serde-json can for example know where the object starts and ends { ... }, even if it is unable to correctly deserialize it. (missing/unknown field, wrong value type). It would be however hard to continue a stream where the received blob is not a valid json string. This distinction cannot be made for all transports though - just by changing the codec to say bincode the two cannot be realistically distingouished from another.
The distinction however is trivial for message based transports though.

But even in case of the serde-binpack, we could basically wrap it in a codec that would prefix the data with the length, and suffix it with a checksum, thus we can validate the frames from eachother.
And I think we're arriving at a point where we'd be introducing another layer of abstraction - encoding of the data on the transport.
Transport being http / mqtt / websocket / whatever, and encoding being the frame format (json, bincode, etc).
I'm not saying this is something tarpc would need to do, but I just realized while writing this message that currently we are conflating these two abstractions.

You're right, and you might even separate framing into its own layer so that there are three things conflated in the transport: message serialization (bincode, json, etc), message framing (header length, big endian or little endian, etc), and byte streaming.
Of course, that's all assuming networking is even happening! One of the cool things about tarpc is that serialization etc is all opt-in. Tarpc can be used to connect different in-process services where everything is done with rust channels.
One question: I assume this would still be treated as a permanent error on the client side? For example, if decoding the request fails, the server will not know what the request ID was, and the client will basi


NIFI-12081 Remove HackerOne from Security Reporting Methods

Summary
NIFI-12081 The project uses the<EMAIL_ADDRESS>private email list for vulnerability reporting, so the link to HackerOne should be removed.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 please self merge.


Code review

Each commit in this PR is an improvement suggestion.
The are more or less opinionated, feel free to cherry pick those that you like and drop the others
(they are relatively independent from each other)

Wow, thanks for all those improvements!
All of those commits are integrated now.
The conflicts have been resolved on the command line, so I will close this request, I'm not sure if GitHub will recognize that as successfully completed.


Miner difficulty

Increased difficulty given security param change.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: lpgeiger:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Full Website Navigation

In this PR, I have developed a full website with several pages: Home, Calculator, and Quote.

In this PR, I have developed a full website with several pages: Home, Calculator, and Quote.

@roodjinocherilus, Please kindly click on the three dots on your PR description to paste this description there as that is the proper way of adding a PR description.


Remove  casts in .compare() method

https://github.com/resin-io-modules/resin-semver/pull/4#discussion_r128728484

Despite returning early if the values are null, TS still complain about not being able to assign null to a string. If anyone has any ideas why let me know!


Chrome application for easy chroot starting

I'm not sure if ChromeOS security restrictions will allow this, but if it is possible to create a simple Chrome application to launch Crouton chroot easily?

+1 for that feature. if possible make as icon on logon screen

Sadly not possible, because of the security restrictions.

Sadly not possible, because of the security restrictions.

Yep. I'm pretty sure there were some old issues on this topic, but can't find them.


SQL Injection

I see a lot of fmt.Sprintf for SQL queries and am concerned about the possibility of SQL injection. Of course, this only matters once the dashboard support non-root user login.
Please add this to the Security project.

@gregwebs Thanks for the concern. PR https://github.com/pingcap-incubator/tidb-dashboard/pull/176 is resolving the only SQL injection risk remaining in the diagnose report functionality.

Close the issue since after investigation we found that there is no user controllable raw SQL input. Although there were SQL string constructing, the parameters are from trusted source instead of user input.


Q: Perfkit target users & comparison with CloudBench

It's more like a question than an issue. We've just wondered if you had some discussions with cloud providers who wanted to use Perfkit in order to white-box test their own clouds. Was anyone interested in using Perfkit for e.g. stressing their private clouds to see what would be the best SW/HW configuration for the clouds?
Btw. Did you have a chance to look at the CloudBench project (https://github.com/ibmcb/cbtool)? If so, could you briefly explain what is the difference between Perfkit and CloudBench? After the quick glance it looks like they offer similar functionalities but Perfkit is growing much more rapidly.
Thanks!

Mateusz - It looks like that project has been around for a year.  Perhaps we should ask them to join us.  I'll open a discussion issue with them.

If you have contacts for contributors there please sent an introduction.  As a proxy I opened an issue - https://github.com/ibmcb/cbtool/issues/85

@voellm I don't have unfortunately. Thanks for opening the issue there

No follow-up from that community so I'm going to close this out.  Looks that that project is pretty stale.

The project is not "stale" -- what gave you that impression voellm? https://github.com/ibmcb/cbtool/commit/fc6395e159e0e10f21248473b4d709e7816995d2

@jtaleric - Thanks for the ping back.  Stale because there was no engagement on this issue - https://github.com/ibmcb/cbtool/issues/85
If you have time I'd be happy to chat about the goals of PKB and find out more about cbtool.

@voellm Cloudbench is 4 years old and not at all stale. We have a mailing list and active contributors as well as a set of fully documented wiki pages. I recommend you start there.


What is serverAddress in NETunnelProviderProtocol confuguration?

Hello,
I found config.serverAddress = "somebridge"  in the permissionViewController.swift file.
I added the server address "<IP_ADDRESS>" there. But it disconnected. Can you please guide me what server address I need to put there to connect TOR-VPN.
Thank you!

You might want to have a look at my attempt at restarting that project:
https://github.com/tladesignz/iCepa/tree/restart
It isn't working, yet, because of lacking DNS support, that's why I didn't move it to this repo, yet.

Hi Benjamin,
Thanks for your update.
Appreciated your work towards TOR!!
I just want to know thatTorController(socketHost: "<IP_ADDRESS>", port: 9050) will create TOR controller in the packetTunnelProvider class. But, When I try to connect this TorController, it gives me this error. Error Domain=NSPOSIXErrorDomain Code=61 "Connection refused"
Do you have any idea about it?

I just want to know thatTorController(socketHost: "<IP_ADDRESS>", port: 9050) will create TOR controller in the packetTunnelProvider class. But, When I try to connect this TorController, it gives me this error. Error Domain=NSPOSIXErrorDomain Code=61 "Connection refused"

The host and port you give TorController needs to be the address, Tor is listening on (typically localhost) and the configured control port. The control port is where you talk to Tor to change its configuration.
So, you need to first start Tor and configure it correctly. Then wait a little (less than a second will be enough), then start the TorController.
When that's successful, you can query the TorController for Tor's status and change its configuration.
See
https://github.com/tladesignz/iCepa/blob/21e3c75512376f66ff3f7e535a060767782e2c6a/TorVPN/BasePTProvider.swift#L41-L48
https://github.com/tladesignz/iCepa/blob/21e3c75512376f66ff3f7e535a060767782e2c6a/TorVPN/BasePTProvider.swift#L144
https://github.com/tladesignz/iCepa/blob/21e3c75512376f66ff3f7e535a060767782e2c6a/TorVPN/BasePTProvider.swift#L152
https://github.com/tladesignz/iCepa/blob/21e3c75512376f66ff3f7e535a060767782e2c6a/TorVPN/BasePTProvider.swift#L185-L187

To be honest: This is not a Tor support forum. If you have difficulty understanding the different concepts of Tor, I very much recommend reading all about Tor on their page: https://2019.www.torproject.org/docs/documentation.html.en


Automatically added "(package-initialize)"

Hello:
When I manually install some packages, ~/.emacs.d/init.el will be automatically written  (package-initialize).
This will cause difficulties when synchronizing your configuration with git pull.
I noticed that this line of code has been written into init-elpa.el, so I suspect that some load order issues.
Because I see a lot of other configurations put this line on the top of init.el.
My Emacs version is 25.2 on archlinux.
Thanks.

That's the reason for this line. Emacs adds it automatically if it doesn't find it in your config, but a comment is sufficient.

Well...This reason I served

a comment is sufficient.

I understand this now.
It seems that package.el will automatically detect whether this code exists ...


ORC-363: Enable zstd for java writer/reader

Use org.apache.hadoop.io.compress.ZStandardCodec in hadoop 2.9.0

@omalley I have changed this PR per your request. Please take a look again when you have time. Thanks!

@omalley Can you review this again when you have time? Thanks!

 Now I am also not in favor of this PR due to:

It depends on org.apache.hadoop.io.compress.ZStandardCodec API and requires Hadoop version to be at least 2.9. It will be painful for some users to update Hadoop version.
It requires zstd native library to be installed on cluster. However, pre-built Hadoop distributions not built with zstd by default. Though I managed to build Hadoop with zstd, it involved with a lot of workarounds and tricks including installing some dependencies and building specific versions of protoc and openssl. It is painful for users to do this.
We cannot have E2E tests in this PR.
The implementation uses JNI under the hood and some users are worried about its performance.

It seems to me that the only blocker is a pure Java implementation of zstd. Currently aircompressor has already supported zstd decompressor and its compressor is working in process: https://github.com/airlift/aircompressor/pull/93. I would wait until it is officially released and update this PR accordingly.

@wgtmac @omalley FYI, the zstd support was merged in airlift/aircompressor earlier this month if you want to utilize it here.

@jack-pappas Thanks for notifying us. I will work on it shortly.


How to get the underlying entity object from entity proxy at runtime

How to get the underlying entity object from entity proxy at run time without the extra code added by proxy
pseudocode :
Order trackedOrder = order.AsTrackable();
//... do  something with trackedOrder
//..
//hard to serialize trackedOrder

what I want :
//getting the changed object without extra code
var order_without_extra_code =  trackedOrder.GetUnderlyingObject()

Serialize(order_without_extra_code)

The cast to the base type (Order) not working for me, still get the extra code:
Order order = (Ordre)trackedOrder;//not working

can you help ?
thanks in advance

using AutoMapper fix my problems
//create map
cfg.CreateMap<Order, Order>();

var order_without_extra_code = Mapper.Map<Order, Order>(trackedOrder);

// I can serialize or save the object without problems
Serialize(order_without_extra_code)

what do you think ?

That's what I suggested here
Given that you are not the first one who asks for this feature, I will add it soon.


MWPW-133167 CaaS CLS Issue

Adds two classes to styles.css as overrides for CaaS min-height in section metadata.
The small class represents roughly the height of the half cards, the medium the full sized cards.

Context
After doing some investigation, essentially loading the footer after CaaS cards load would require a change against the CaaS react components themselves, and may not be the best approach. The CaaS react component mounts to the DOM before the cards are finished fetching, and the CaaS component itself is controlled by the CaaS react components, meaning that though we could ask for an event to be added, there may occur situations in which the footer would not load.  In addition to this, since the footer is loaded in loadArea() changes to prevent CLS on this page would require a change that could have effects on all milo consumers.
Because the issue was identified with resources/main on bacom, I am putting the overrides here instead of Milo, since we are not aware of who else may need them. Should it be determined styles like this are needed by the broader community, we can move them then.
Tested via pagespeed insights on a Milo branch first to ensure that CLS was 0, moved to bacom for this PR.
Resolves: MWPW-133167

Test URLs:

Before: https://main--bacom--adobecom.hlx.live/drafts/slavin/CaaS-tests/main?martech=off
After: https://caas-cls-issue--bacom--adobecom.hlx.live/drafts/slavin/CaaS-tests/main?martech=off

I thought we decided to publish our draft pages and use the hlx.live urls so that lighthouse could run?

I thought we decided to publish our draft pages and use the hlx.live urls so that lighthouse could run?

404s on prod? https://business.adobe.com/drafts/slavin/CaaS-tests/main?martech=off am I missing something?

https://caas-cls-issue--bacom--adobecom.hlx.page/drafts/slavin/CaaS-tests/main?martech=off

Yeah it redirects to the prod url when you publish. I usually just go back to the preview url and replace "page" with "live"
https://caas-cls-issue--bacom--adobecom.hlx.live/drafts/slavin/CaaS-tests/main?martech=off

https://caas-cls-issue--bacom--adobecom.hlx.page/drafts/slavin/CaaS-tests/main?martech=off

Yeah it redirects to the prod url when you publish. I usually just go back to the preview url and replace "page" with "live"
https://caas-cls-issue--bacom--adobecom.hlx.live/drafts/slavin/CaaS-tests/main?martech=off

you da best. Thanks.
https://pagespeed.web.dev/analysis/https-caas-cls-issue--bacom--adobecom-hlx-live-drafts-slavin-CaaS-tests-main/iaytgfyvjg?form_factor=mobile

Codecov Report

Merging #128 (bd11509) into main (c9aaeaa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #128   +/-   ##
=======================================
  Coverage   96.36%   96.36%
=======================================
  Files           6        6
  Lines         468      468
=======================================
  Hits          451      451
  Misses         17       17

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I thought we'd be fixing this in milo, but I understand the decision to put it in bacom. We only have one use case so far, and it's possible that no other pages will need this. Given that, do we really want to include .caas-spacing-s if it's not being used?

I wrestled with both adding to milo and the additional class. It's not much process to add another class so we can remove the caas-spacing-s.


Fix platformText logic

#176
#175  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


Files related to the Account

Hi,
We have a requirement where users want an holistic view of the related files not just the Child objects but also the files related to the Account. So that all the information is on the same view or section.
Could you please help us if there us any way possible to add this in the current page setup.

Hi @MukeshKatochsfdc, for files associated to the Account object (that is, any parent object you're using), please add the standard "Files" related list to the page.
This app provides a "Related Files" component that queries for files related to the child objects, but it doesn't query files for the parent object since that would be redundant of the standard "Files" component.
Here's a screen shot of Lightning App Builder where I'm customizing the Account record page to include the standard "Related List - Single" component to show the Files related list (make sure to add the Files related list to the object's page layout). Below the standard files component, I also have my custom "Related Files" component.


DRAFT: Handle case with nested list of objects

Description
These changes add support for processing a list of objects in a nested field. Currently, if the configuration looks like below and if the nested field contains a list of nested objects then the request fails:
curl --location --request PUT 'http://localhost:9200/_ingest/pipeline/embeddings-nested' \
--header 'Content-Type: application/json' \
--data '{
    "processors": [
        {
            "text_embedding": {
                "model_id": "o-bPa4sBu3zYeKjn7baf",
                "field_map": {
                    "nested-field": {
                        "text-field": "vector-field"
                    }
                }
            }
        }
    ]
}'

Example Request with Nested List
curl --location 'http://localhost:9200/_ingest/pipeline/embeddings-nested/_simulate?verbose=true' \
--header 'Content-Type: application/json' \
--data '{
    "docs": [
        {
            "_id": "1",
            "_source": {
                "nested-field": [
                    {
                        "text-field": "This is another test"
                    },
                    {
                        "text-field": "This is a third field"
                    },
                    {
                        "text-field": "This is a fourth field"
                    }
                ]
            }
        }
    ]
}'

Issues Resolved
https://github.com/opensearch-project/neural-search/issues/469
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@krishy91 is this PR still a draft PR or ready for review?

@krishy91 is this PR still a draft PR or ready for review?

Hi! I'm still working on it! I will update here once its ready!

Hi @navneet1v can someone review the changes?

Hi @krishy91 before we can review the changes please do the following:

Remove the draft keyword from title of the PR.
Make sure that all your github actions are successful.

Please let ping on this PR once the above things are done.

Hi @navneet1v ! The Pull Request passed the workflows.

We would really like to use this feature, is there anything we can do to help in getting this merged?

Solves: https://github.com/opensearch-project/neural-search/issues/469 - PR is already in use and we would like to see it included in 2.12. or earlier.

Will be checking this PR today.

Thanks @navneet1v! Let me know if I can do anything to speed up the merge process

@navneet1v gently reminder to look into this PR. We would like to move away from using this branch.

Looked into the PR, on high level looks good. Please add Integration tests for this change.

Also code coverage is dropped for new lines which are added please ensure that those lines are covered.

Looked into the PR, on high level looks good. Please add Integration tests for this change.

Added integration test for the case containing list of nested objects

Also code coverage is dropped for new lines which are added please ensure that those lines are covered.

Added addtional test with objects having nesting depth of 2. The InferenceProcessor has 98% coverage for lines now. How can I rerun the report?

@navneet1v did you get a chance to look at the PR again after I resolved the issues you mentioned in your review?


programatic url rewrite http->https not working correctly

Below is my code.  When I run this and type in the http request
trantor.645.kebler.net:8080
the http server does not rewrite the url but it does serve the files from https server at the above url insecurely without https.
If I put in the https url all is well
https://trantor.645.kebler.net:8443
const LocalWebServer = require('local-web-server')
const https = new LocalWebServer()
const http = new LocalWebServer()

const CERT_PATH ='blah blah'
const CERT_NAME ='trantor.645.kebler.net'

const FILES_PATH ='blah blah'

http.listen({
  port: 8080,
  rewrite: '/* -> https://' + CERT_NAME + ':8443/$1'
})

https.listen({
  port: 8443,
  https: true,
  hostname: CERT_NAME,
  directory: FILES_PATH,
  spa: 'index.html',
  key: CERT_PATH + CERT_NAME + '.key',
  cert: CERT_PATH + CERT_NAME + '.crt'
})

Maybe what I was hoping for was a redirect like this which works fine as expected
const express = require('express');
const fs = require('fs');
const https = require('https');
const http = require('http');

let app = express();

http.createServer(app).listen(80);
https.createServer(app).listen(443);

app.use(function(req, res, next) {
    if (req.secure) {
      console.log(res)
        next();
    } else {
        res.redirect('https://' + req.headers.host + req.url);
    }
});

Hi, rewriting an incoming URL and returning a "302 Redirect" response are two completely different solutions to two different problems.
I'm not clear what the issue is, closing.

yes nothing wrong with rewrite.   this is not an issue but a feature request.  Support a redirect too.   Not a biggy.  Obviously I have a solution.

what is the feature you're requesting? Some way of achieving your redirect solution above with a simple command-line flag or option? How should it work, something like the below?
$  ws --redirect-everything-to https://somewhere.else:9000

Hi there, Thanks for the awesome package! I'm using it to develop PWAs locally.
One of the Lighthouse criteria is that the site redirects requests from http to https. Of course, I can set up rewrite according to the wiki page's instructions, but that doesn't give me the 302 Redirect I need to pass this audit.
I'd very much like some feature like ws --server http2 --redirect-http-to-https which will take all requests and redirect them to https.
Thanks for considering.


Ensure all Eirini templates are namespaced

When rendering the Eirini Helm chart, we should allow the templates to be rendered for a particular namespace. Most of the templates were supporting, but a few were not, so we are ensuring they all can now.
FWIW, we are using Helm 3 (and helm template) and needed to add this to use Helm 3:
version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}
Can test by running helm template --namespace foobar and ensure the appropriate resources are placed into the foobar namespace
@acosta11 & @jspawar & @alex-slynko

Looks good, thanks for contributing @acosta11


Add missing virtual destructors, fix #1771

Signed-off-by: Tobias Fischer<EMAIL_ADDRESS>See #1771
Successfully fixed the test failures on osx on conda-forge & compiler does not complain anymore (previous warning example: warning: delete called on 'cartographer::mapping::Grid2D' that is abstract but has non-virtual destructor [-Wdelete-abstract-non-virtual-dtor].

I'm curious to see if with the requested changes this compiler warning will be rsolve.. because everything was already done correctly before, the only thing this changes is more explicit syntax.

I do not understand this PR. To me it looks like the compiler error is a false positive since both GridInterface and RangeDataInserterInterface already define a virtual destructor. Can you explain what you think is wrong with the code?
If this is indeed to mitigate a compiler issue: did you file an issue about this compiler issue?

I fully agree! marking them override cannot harm though and maybe suffices to get rid of the compiler warning

Hi there,
Thank you for your comments!
I do not have any knowledge of cartographer - this PR originated (as described above, where I also link to the issue https://github.com/cartographer-project/cartographer/issues/1771 which contains the compiler warnings and more information) from failing test cases on OSX when packaging cartographer for conda-forge.
After applying the changes in this PR, both the compiler warnings (not "compiler error", I nowhere talk of compiler errors), as well as the test failures, disappear. So I am not sure how you think that "everything was done correctly before" and "the compiler error is a false positive". Things obviously weren't all done correctly before, nor was it a false positive, as there were test failures (albeit only on OSX with clang, but nonetheless) that are now resolved.
You are more than welcome to make the requested changes in this PR if you want to improve cartographer by fixing these test failures, edits from the maintainers are allowed. If you still think I am fixing false positives, I am equally happy to close this PR and continue to use a patch to fix the problems in conda-forge.

Hey @johuber @wohe
here are the build & test logs for cartographer on osx-64, without the fixes applied: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=235112&view=logs&jobId=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&j=58ac6aab-c4bc-5de2-2894-98e408cc8ec9&t=933f325c-924e-533d-4d95-e93b5843ce8b
With the fixes applied all builds are green: https://github.com/conda-forge/cartographer-feedstock/pull/7
We now provide up-to-date cartographer builds on conda-forge for Windows, OS X and Linux :)
Btw, if either of you is interested in helping to maintain cartographer on conda-forge, please let me know and we can add you as maintainer. This usually doesn't involve a lot of effort, just now and then updating the version if a new release appears (there are bots for this, too).

Hm, I think the issue is that the 1.0.0 which is quite old by now didn't have the virtual destructor in the base class. E.g. this PR https://github.com/cartographer-project/cartographer/pull/1453 hasn't made it in the 1.0.0 release as far as I can see. How about a new release? Looks like there is quite a bit of development since 1.0 :)
So I guess we could also fix our build in conda-forge by adding the empty virtual destructor in the GridInterface base class.
Either way, what we have works for the moment.

Thanks for investigating this @wolfv! Seems like this issue does not occur on the master branch anymore - apologies for the confusion caused. Closing here


Code refactor not working in Mac M1

VS Code Version: 1.55
OS Version: Mac

Steps to Reproduce:

Click on the name you want to refactor
Press F2 then type the name

Enter the name is not apply but instead it add the new line

Does this issue occur when all extensions are disabled?: Yes

Have you configured a custom keybinding for enter?

@mjbvz no it's work fine on my Window machine but in Mac it's not.

Please do the following: select "F1 > Set Log level > Trace" and then "F1 > Show Logs > Window". Then try to rename again and share the logs of around the time you tried to rename

@jrieken Sure. Will do that when I get home. Thank you

Maybe the issue is caused by one of your installed extensions. All extensions execute together on a single nodejs process, so in order to narrow down the possible problematic extension, please try the following:

run F1 > Help: Start Extension Bisect
(this will restart VS Code with half of your extensions disabled)
try to do the steps which reproduce the problem
if the problem reproduces, click "This is bad" (this narrows down the problematic extension to the half which was enabled)
if the problem does not reproduce, click "Good now" (this narrows down the problematic extension to the half which was disabled)
these steps will repeat until the problematic extension is found

initial
step
final

@jrieken Thanks you I will try it

@jrieken Tried it and I only get this
The bisection of extensions has been completed, but no extension has been identified. Possible cause of the problem: code.

Any update on this ?

No update. I can still not reproduce. Can you try this with VS Code Insiders?


Format the if expression at the end of the block in a single line

This PR is a successor of #2778. Huge thanks to @csmoe for tackling to this in the first place.
Closes #2777.

Also, could in the end, this to be squashed on a single commit?


[BUG] type annotation Optional[Tuple[float, float]] does not compile: "typing.Optional[...] cannot be applied to non-Python type (double, double)"

Describe the bug
Since Cython 3.0.0a11, using the type annotation Optional[Tuple[float, float]] leads to a compile error:
$ python ../cythonize.py -3 -b ex_fail.py
Compiling /home/alex/dev/cython/examples/ex_fail.py because it changed.
[1/1] Cythonizing /home/alex/dev/cython/examples/ex_fail.py

Error compiling Cython file:
------------------------------------------------------------
...
from typing import Optional, Tuple

def foo(a: Optional[Tuple[float, float]]):
                         ^
------------------------------------------------------------

ex_fail.py:3:25: typing.Optional[...] cannot be applied to non-Python type (double, double)
Traceback (most recent call last):
  File "/home/alex/dev/cython/examples/../cythonize.py", line 9, in <module>
    main()
  File "/home/alex/dev/cython/Cython/Build/Cythonize.py", line 234, in main
    cython_compile(path, options)
  File "/home/alex/dev/cython/Cython/Build/Cythonize.py", line 68, in cython_compile
    ext_modules = cythonize(
  File "/home/alex/dev/cython/Cython/Build/Dependencies.py", line 1132, in cythonize
    cythonize_one(*args)
  File "/home/alex/dev/cython/Cython/Build/Dependencies.py", line 1299, in cythonize_one
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: /home/alex/dev/cython/examples/ex_fail.py

Whereas the following examples do work:
from typing import Optional, Tuple

# using int instead of float
def foo(a: Optional[Tuple[int, int]]):
    print(a)

def foo(a: tuple[int, int] | None):
    print(a)

# using standard generic classes instead of typing module (valid since Python 3.10)
def foo(a: tuple[float, float] | None):
    print(a)

# combining typing.Optional and builtin tuple (valid since Python 3.9)
def foo(a: Optional[tuple[float, float]]):
    print(a)

# combining typing.Tuple and '| None' (valid since Python 3.10)
def foo(a: Tuple[float, float] | None):
    print(a)

# combining float and int
def foo(a: Optional[Tuple[float, int]]):
    print(a)

# not using a tuple
def foo(a: Optional[float]):
    print(a)

The last working release is 3.0.0a10.
Code to reproduce the behaviour:
from typing import Optional, Tuple

def foo(a: Optional[Tuple[float, float]]):
    print(a)

Expected behaviour
The code should compile since Optional[Tuple[float, float]] is a correct type annotation.
Environment
OS: Linux
Python 3.10.8
Cython 3.0.0a11/master
Additional context
No response

If you want the previous behaviour back, just decorate the function with @cython.annotation_typing(False)
What's happening here is that the tuple is getting converted to what Cython calls a "ctuple" (a direct C struct). In this case that is conflicting with Optional.
Thanks for the report

For most of your other examples there's reasons why Cython can't (yet) do much with the type annotation so they don't fail.


Question: Consumer Group Plugin

I'm planning to develop custom plugin for our usecase.
I assume currently only RateLimit plugin can be configurable for Consumer Group.
Is it technically possible to develop and apply Custom plugin for Consumer Group?
And is it planned to add plugin configuration other than Rate Limit to Consumer Group resource as official feature (e.g. via Kong Konnect Portal)?

Yes, Consumer Groups are now first-class entities, which means that you can scope plugins to it. Please check https://docs.konghq.com/gateway/latest/kong-enterprise/consumer-groups/ for more details.
If you plan to develop your own custom plugin, you can use this. (Note that Consumer Groups are an Enterprise feature)

I see, that's good to hear.
Thank you so much!


Key may not be empty

I get the following exception when trying to create a schema.

Context:
    public class SqlContext : DbContext
    {
        private readonly IDbConnection _connection;

        public SqlContext()
        {
        }

        public SqlContext(IDbConnection connection)
        {
            _connection = connection;
        }

        public DbSet<UnitData> Header { get; set; }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            if (!optionsBuilder.IsConfigured)
            {
                var connection = new CassandraConnectionStringBuilder();

                connection.Username = "cassandra";
                connection.Password = "cassandra";
                connection.ContactPoints = new string[] { "localhost" };

                optionsBuilder.UseCassandra(connection.ConnectionString, opt =>
                {
                    opt.MigrationsHistoryTable(HistoryRepository.DefaultTableName, "test");
                });
            }
        }

        protected override void OnModelCreating(ModelBuilder builder)
        {
            builder.UsePropertyAccessMode(PropertyAccessMode.Property);
            builder.ForCassandraAddKeyspace("test", new KeyspaceReplicationSimpleStrategyClass(1));

            builder.Entity<UnitData>().ToTable(nameof(UnitData), "test");
            builder.Entity<UnitData>().Ignore(table => table.ETag);
            builder.Entity<UnitData>().Ignore(table => table.Timestamp);
            builder.Entity<UnitData>().Ignore(table => table.Values);
            builder.Entity<UnitData>().ForCassandraSetClusterColumns(table => new { table.RowKey });
            builder.Entity<UnitData>().HasKey(table => new
            {
                table.PartitionKey,
                table.RowKey
            });
        }
    }

cc @simpleidserver

I have a feeling that this is a duplicate of #10.

@pluskal I have seen that issue and have created the keyspace manually. I am trying to create tables automatically.

We couldn't reproduce your issue. Can-you use the Nuget version "2.0.3" ?

@simpleidserver I get the same error on 2.0.3.
I have used the EF Migrations instead to sort this out. The bug is still there though.

Sorry to answer you late, I was working on the CaseManagement Project.
Can-you transfer your project ? I will  take a closer look :)

Hi, I've been seeing the same behaviour, calling EnsureCreated or EnsureCreatedAsync results in the mentioned exception.
This can be verified even with the sample project when calling dbContext.Database.EnsureCreated().

@simpleidserver the code pasted above is the project. The Program.cs only creates the context and calls EnsureCreatedAsync.

The issue is fixed in the branch "release/2.0.4" :)

The issue is fixed in the branch "release/2.0.4" :)

I confirm it fixed the issue on my copy of the sample and on a personal project as well.

awesome!


Fixed the bucket list API

The bucket list API is currently dysfunctional. AWS sends responses that fail to parse, as Serde expects the fields to exist in responses. Additionally, when the request fails, the developer doesn't get very good information why.
This PR fixes the bucket list API by making non-default fields optional. In addition, it makes the errors returned by AWS into first class errors in the library.
The other methods need to be updated with AWS errors too – this is just the first push, but at least it makes the bucket listing API work.

@golddranks thnx for this, glad it is gettig some love, I'll review and merge, you have some changes that overlapp with #13, I'll sort it out.

Thanks <3


How to calculate text height in advance

How to calculate text height in advance? If the text exceeds the specified number of lines, display "more" images at the end of the text, and click "more" images to display all text.

Please consider asking support questions in one of the other channels listed at http://flutter.io/support .


Unable to checkout another amplify environment

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
Cannot checkout to another amplify environment
Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
me@mbp bkend % amplify -v
4.40.0

To Reproduce

me@mbp bkend % amplify api push

# ... Updating

UPDATE_COMPLETE apixxxxxxxxxx AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:06 GMT+0800 (Hong Kong Standard Time)
⠋ Updating resources in the cloud. This may take a few minutes...

UPDATE_COMPLETE amplify-xxxxxxxxxxxxxxx-bkddev-94640               AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxPostConfirmation            AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxDefineAuthChallenge         AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxPostAuthentication          AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxCustomMessage               AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxCreateAuthChallenge         AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxPreAuthentication           AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxPreTokenGeneration          AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxVerifyAuthChallengeResponse AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE functionxxxxxxxxxxxxxxxPreSignup                   AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:08 GMT+0800 (Hong Kong Standard Time)
UPDATE_COMPLETE authxxxxxxxxxxxxxxx                                AWS::CloudFormation::Stack Wed Dec 16 2020 10:22:07 GMT+0800 (Hong Kong Standard Time)
✔ All resources are updated in the cloud

GraphQL endpoint: https://xxxxxxxxxxxxxxx.appsync-api.<region>.amazonaws.com/graphql

me@mbp bkend % amplify env list

| Environments |
| ------------ |
| *bkddev      |
| dev          |

me@mbp bkend % amplify env checkout dev
⠏ Initializing your environment: devCould not initialize 'dev': The "path" argument must be of type string. Received an instance of Error

Expected behavior
Checkout to my 'dev' environment
and push new updates to target environment
Desktop (please complete the following information):

OS: masOC Cataline 10.15.7
Node Version. You can use node -v to check the node version on your system

me@mbp ~ % node -v
v12.19.0

Additional context
I never access the 'dev' environment before, so by adding the 'new' dev environment to my local project. I have added some configurations in the team-provider-info.json and execute amplify env checkout dev.
Then this error pops up,
Cannot destructure property 'appId' of '_.get(...)' as it is undefined.
Googled it and found #6087 issue then update my amplify version from 4.39.0 to 4.40.0, but no effects on my error.
The Could not initialize 'dev': The "path" argument must be of type string. Received an instance of Error still exists.
Before this error came up, I have also accidentally ran the amplify init in current amplify folder directory and terminate during the init process. Now I have reverted back to a commit prior to amplify init.

i have fixed this issue by re-pulling all environments in a new clear folder and checkout again.
Don't know what is the cause for the issue.
I will close this issue.


feat(tonic): Add Request::set_timeout

This will set the grpc-timeout to a value formatted according to the spec.

Aha, this is very nice to have, thank you very much :)


Epoch/Equinox Documenation

From R. Seaman:
The schema expects different formats from epoch and equinox, but I’m not clear on what is expected. Should these both be “2000.0” for J2000, or should the epoch be the timestamp for the specific image? I think the latter would be more normal for solar system objects that move, of course. In any event the schema doesn’t clarify the anticipated usage for these fields (at least, it doesn’t to me).

We have dropped "epoch", and are using only "equinox", with a definition supplied by UMD


No module named unicore.data

Thank you for this wonderful model! However, when I executed the Python command
from unimol_tools import MolTrain

An error occurred:
ModuleNotFoundError: No module named 'unicore.data'

I'm 100% sure that I have installed the unicore package because when I import unicore the program will not throw an error. I searched for documents about unicore and unicore.data but did not get useful results. Therefore, I wonder if you can help me check this problem.

Hi, does from unicore.data import Dictionary work?  and you can attach more details about this.

Thank for your reply! However, it still doesn't work, with "ModuleNotFoundError: No module named 'unicore.data'".

you can refer this: https://github.com/dptech-corp/Uni-Mol/issues/179

Below are my code and the entire exception information:
import os
import unicore
from unicore import *

os.environ['KMP_DUPLICATE_LIB_OK']='True'

from unicore.data import Dictionary

from unimol_tools import MolTrain
from unimol_tools import MolPredict

Exception:
Traceback (most recent call last):
  File "/Users/zephyr/Desktop/Work/Uni-Mol/Uni-Mol/unimol_tools/test.py", line 2, in <module>
    import unicore
ModuleNotFoundError: No module named 'unicore'
(base) zephyr@xuyufengdeMacBook-Pro Uni-Mol % conda activate my-rdkit-env
(my-rdkit-env) zephyr@xuyufengdeMacBook-Pro Uni-Mol % cd unimol_tools
(my-rdkit-env) zephyr@xuyufengdeMacBook-Pro unimol_tools % ls
MANIFEST.in             requirements.txt        unimol_tools
README.md               setup.py                unimol_tools.egg-info
build                   test.ipynb
dist                    test.py
(my-rdkit-env) zephyr@xuyufengdeMacBook-Pro unimol_tools % python test.py
Traceback (most recent call last):
  File "/Users/zephyr/Desktop/Work/Uni-Mol/Uni-Mol/unimol_tools/test.py", line 7, in <module>
    from unicore.data import Dictionary
ModuleNotFoundError: No module named 'unicore.data'

you can refer this: #179

Thank you for your help! Will check if it works.


Add Resource Pages

Per our call yesterday, I copied the template we had into markdown and linked out to the resource page from the goal page.
I started with Project Places and will update the pull request with the other goals.

[x] Project Places
[x] Communication
[x] Contribution
[ ] Events
[ ] Governance
[ ] Leadership
[ ] Recognition

@dicortazar can we please get this merged?  It's a blocker for work @akshitac8 and I want to propose (or add me as maintainer, happy to do that)
Thanks!

This has been already merged up to the status of today. Shall we close this and keep adding other PRs? I would suggest to continue tracking the list of PRs in a ticket and not in a PR.
PRs should be concise and small to help in the review process :).
@emmairwin it would be good to have more maintainers definitively. I asked @GeorgLink about this as well. But let's discuss this in our next meeting if that's ok for you all.
Thanks!

@dicortazar

This has been already merged up to the status of today.

I don't understand your message. This pull request is "open" as I'm writing these lines. That is not merged.

PRs should be concise and small to help in the review process :).

This PR is concise and small. It only adds resource pages. This requires touching a lot of files.

So while I think prefixing file names as a way to categorize (vrs folders) is less than ideal, this is all a great step forward, and will inform our next steps.
Thanks for your patience.


2.0 training data format docs

Proposed changes:

Initial docs for 2.0 training data format.

Status (please check what you already did):

[ ] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

TODOS:

[x] complete the responses section - Edit: pointing to the domains page since the format is still relevant and the page also contains a better explanation of using responses
[ ] clarify the entity format meaning in stories
[ ] merge in new structure
[ ] correct links when new structure merged

Ope, forgot to add a form section. I'll do that now.

Left over review comments/questions from Akela:
[ ] For single examples, is the vertical bar necessary? e.g.
  examples: |
  - [0-9]{5}

vs
  examples: [0-9]{5}

[ ] Are forms referred to as loops now?
[ ] Desired vs. actual behaviour of versioning (if later than latest, is it ignored or will it use the latest version?)
[ ] Todo: Pull in actual latest version number

regarding

For single examples, is the vertical bar necessary? e.g.
examples: |

[0-9]{5}

yes, let's keep it consistent - it is unlikely that people will only have one example (they really shouldn't ) and it keeps things simpler

@melindaloubser1 how are things going, anything still open or is this ready for a another review?

Ready for another review. I don't know the answer to Ella's q about versions, if you do I'll clarify that

Do you mean latest, or the version you have installed? I'd assume the latter as a version can't know about new versions?

It will use the latest data format supported by the currently installed rasa open source (which might not be the latest data format overall, as your rasa version might be outdated).

Similar question about Training data files with a version greater... -- if you try to run version: 2.1 on rasa==2.0.0, it will still fail even if 2.1 is released, right?

Exactly. If you are trying to use a training data version that is not yet supported by your installation you likely need to update your installed rasa version.

@melindaloubser1 are these the questions you were referring to?

Yes, thank you for clarifying! related question - where can users find information about these training data versions? I haven't come across a list yet

I don't think it makes sense to add a list until after we have multiple data versions that we support.

Ok, I think a user wouldn't know what version to put right now though - should we include that "current training data version is 2.0" or something?

yes that is a good point 😅  - let's include that :+1:

Added. It's a bit verbose but hopefully clear.

Now that the new structure is in place, wdyt about splitting it up into overview-dialogue-nlu pages?

I'd rather want to merge this first, as I need it in e.g. the multimedia responses PR to add my changes. I think we can split it up in a separate PR (I think we can also discuss that in the channel or on an issue first)

@erohmensing do you want to give the PR another review?

@melindaloubser1 I wanted to check with you and see if we can wrap this up before the 2.0.0a2 release tomorrow. Are there any outstanding items that prevent the merge?

I don't think so, all major points have been addressed, unless @erohmensing or @tmbo have anything final to add. Some outstanding stuff for next round, more so discussion points.

great, so let's get this merged -- i'll let you add the status:ready-to-merge label

@melindaloubser1 was it intentional that you added this page? https://rasa.com/docs/rasa/next/nlu-training-data

Yes, there's a discussion about it here https://rasa-hq.slack.com/archives/C016FLYV4QN/p1597319543009500

We can remove most of it for now if you think that's better for the alpha, it's the concept content we didn't want to lose


fix(cli): fix cli bug

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


tiger-/tightvnc depend on unfree font

fontbhlucidatypewriter75dpi is unfree (though mistakenly not marked as redistributable) and tiger- and tightvnc explicitly depend on it.
Does it have to be that font or can it be substituted with a free alternative @viric? I don't use the server part of those tools, so I can't test their functionality.

See also https://github.com/NixOS/nixpkgs/pull/126756

Closing since #126756 has been merged.

The font is now redistributable but still unfree.

The tigervnc client works perfectly without any font paths baked in. I wonder if this is still required for the server or if it is an artifact from years ago that is no longer necessary? The fontPath-code is 11 years old.
As a workaround tigervnc.override{fontDirectories=[];} can be used where unfree is problematic.

I guess we could make a free version without the font for the people who just want the client.

Fixed in cfb935c54940115751270011a569ba3214984f5a


Why does mish return significantly worse result than elu?

I've tested mish vs elu on a simple feedforward network with L2 and dropout and got this result:

You can check it with Colab Notebook, press ctrl+f9.

@digantamisra98 why was it closed? Did you found an explanation?

@LifeIsStrange I am working on the theoretical reasoning behind this and should be able to get something substantial out by the next version release but until then this issue was closed because it was stale.

Any expected timeframe for the next version release? (take your time :})

Maybe next month since I'm busy with some other commitments.


Updated mix.lock dependencies

The postgrex dependency wouldn't compile on the version it was locked
to, at least with the following settings.
Erlang/OTP 21 [erts-10.1] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [hipe]

Mix 1.8.1 (compiled with Erlang/OTP 21)
Elixir 1.8.1 (compiled with Erlang/OTP 21)

System Software Overview:
      System Version: macOS 10.14.5 (18F132)
      Kernel Version: Darwin 18.6.0

Running mix.deps update --all upgraded all dependencies. Running
tests showed no regressions.

It looks like you edited the lock file? If so that'll get overwritten at some point - I do need to update the Postgres stuff but there are breaking changes in there - enough of them in the language too that I need to rev to 4.0. I have a branch I'm working on that uses the Erlang driver instead, which is awesome as it has a load of useful features but I need to go a bit slower as pooling and transactions are kind of a big deal.


Roll engine 3757390fa4b0..733739938bb4 (14 commits)

https://github.com/flutter/engine/compare/3757390fa4b0...733739938bb4
git log 3757390fa4b00d2d261bfdf5182d2e87c9113ff9..733739938bb4f841320e12e32a52c3ba7d69558c --no-merges --oneline
733739938 Roll src/third_party/skia 9a88bee122f1..b00f7b34751b (24 commits) (flutter/engine#7831)
6f318efe0 Move up ndk version that is being downloaded(old one no longer available) (flutter/engine#7830)
a8aa1ee06 Track flow id (flutter/engine#7826)
b1ce6b72f Roll src/third_party/skia a0dcd29f536b..9a88bee122f1 (4 commits) (flutter/engine#7825)
dbc166387 Roll src/third_party/skia 6152470dc69e..a0dcd29f536b (2 commits) (flutter/engine#7824)
942677695 Roll src/third_party/skia 4037f7f5d8b4..6152470dc69e (1 commits) (flutter/engine#7823)
e2394ad77 Add flow events connecting pointer events to frames (flutter/engine#7807)
0a5a7c56d Roll src/third_party/skia 09c01e9df260..4037f7f5d8b4 (3 commits) (flutter/engine#7821)
b867f4858 Roll src/third_party/skia 186669c4128b..09c01e9df260 (5 commits) (flutter/engine#7820)
af39e22c3 Fix tests that were committed after cirrus ran (flutter/engine#7819)
1f2cbf792 Roll src/third_party/dart 0a7dcf17eb..c92d5ca288 (64 commits)
033f2072c Add space metrics tracking for trailing whitespace "ghost" runs. (flutter/engine#7791)
adf02abcd Allow specifying the buildtools path (flutter/engine#7815)
f519aedb9 Fix typo in painting.dart (flutter/engine#7814)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6239917442072576 , https://cirrus-ci.com/task/4973280046874624 , https://cirrus-ci.com/task/6099179953717248 , https://cirrus-ci.com/task/6380654930427904

Commit queue failed; closing this roll.


Documentation: Current system population variable incorrect

I found that the spoken system population variable is listed incorrectly in your documentation. It is stated to be {TXT:System population}, but should be {TXT:System population (spoken)}.

Thank you for reporting this; it will be fixed in the next release.


Agenda

The buttons to rename the "Agenda" @settings are gone

The button to add an event ad "Agenda" @ dashboard is gone

On what service did this problem occur?

Do not know anymore but I have checked several agenda's and i did not found the problem anymore so close the issue.
Could temporary issue due to updates ?


Ensure previous description displays

Description of change
Ensure previous description is displayed in change request show page

Would a system spec be beneficial here to cover us?

good point, I have a whole cucumber spec for this but missed out on the "show" page after a request is responded to. Will add a little spec


ListItemModel: allow modification

See http://mail-archives.apache.org/mod_mbox/wicket-dev/201710.mbox/browser

Hi @selckin ,
could you please close this PR? I've merged your changes into master but I don't have the rights to close the PR.

Thank you!


The result of resize.Point is shifted when scaling down

Origianl image(1920x1080): http://i.imgur.com/jL5Cqwh.png
Comparison between fmtconv and zimg(scale down to 960x540): http://screenshotcomparison.com/comparison/167402
I also compare AviSynth's internal PointResize and get the same result of fmtconv's.

I tested with the checkerboard below and observe the expected result for the point kernel. The result is a solid color, indicating no drift of the phase, i.e. exactly every other pixel has been selected.
Interleave(BlankClip(width=540, height=960, color=color_red), BlankClip(width=540, height=960, color=color_green)).AssumeFieldBased().Weave()
Interleave(TurnLeft(), TurnRight()).AssumeFieldBased().Weave()
#PointResize(960, 540)

As you know, when scaling by a 0.5 factor and preserving center position, the even and odd source samples are equidistant from the output position. For the point kernel, that means that either phase is a valid, leading to the ambiguous outcome you observed. It turns out that AVS (did not check fmtconv) rounds up (to the bottom-right) the phase when it is exactly halfway between samples, yielding a green image, whereas z.lib rounds down, yielding red.
There is no "correct" way to do this, and you should just not use the point filter to downsample.


Emoji seems to append with line break?

In edit mode:

In view mode:

Split panel:

Is this happening for anyone else?? I'd like to use this plugin as it will bring an element of fun, but its not really useable with the current bug. :(
Updated note: Theme I am using is Cybertron
://mk

So did a little more work with this and its due to the following setting being enabled on my system...
Settings > Editor > Strict Line breaks.
I turned it off, and the emoji's did apply as I was expected (inline), but... id rather not turn that setting off!????
Any other ideas?
://mk

Closing issue. It's a confirmed issue with the Cybertron theme.


New sample: solving batched linear systems with oneMKL and OpenMP target offloading

Adding a New Sample(s)
Description
This is the new sample that uses OpenMP to offload the oneMKL batched linear solvers to an accelerator. The sample codes are Fortran programs, but the BKMs are general to C and C++.
Checklist
Administrative

[X] Review sample design with the appropriate Domain Expert: The oneMKL linear solver team reviewed the sample codes before the accompanying article was published on IDZ: https://www.intel.com/content/www/us/en/developer/articles/technical/solve-linear-systems-onemkl-openmp-target-offload.html
[ ] If you have any new dependencies/binaries, inform the oneAPI Code Samples Project Manager

Code Development

[ ] Implement coding guidelines and ensure code quality. see wiki for details
[X] Adhere to readme template
[ ] Enforce format via clang-format config file
[X] Adhere to sample.json specification. https://github.com/oneapi-src/oneAPI-samples/wiki/sample-json-specification
[ ] Ensure/create CI test configurations for sample (ciTests field) https://github.com/oneapi-src/oneAPI-samples/wiki/sample-json-ci-test-object
[ ] Run jsonlint on sample.json to verify json syntax. www.jsonlint.com

Security and Legal

[ ] OSPDT Approval (see Project Manager for assistance)
[ ] Compile using the following compiler flags and fix any warnings, the falgs are: "/Wall -Wformat-security -Werror=format-security"
[ ] Bandit Scans (Python only)
[ ] Virus scan

Review

[ ] Review DPC++ code with Paul Petersen. (GitHub User: pmpeter1)
[ ] Review readme with Tom Lenth(@tomlenth) and/or Project Manager
[X] Tested using Dev Cloud when applicable

@hagabb Can you resubmit to the development branch? You may need to open a new PR and close this one


Unstructured data save should add UGRID to the file conventions

Though N.B. I believe there is still some room for doubt about what the exact result should look like.
see Netcdf User Guide Appendix A, under "conventions" :

"""It is possible for a netCDF file to adhere to more than one set of conventions, even when there is no inheritance relationship among the conventions. In this case, the value of the 'Conventions' attribute may be a single text string containing a list of the convention names separated by blank space (recommended) or commas (if a convention name contains blanks). """

So, whether "UGRID 1.0" replaces "CF 1.7", or adds to it, or whatever, not quite sure as yet.
UGRID sample files that I have seen all say just "UGRID 1.0",
( like this example in the UGRID repo )
but that will of course upset the cf-checker, since the 'adoption' relationship between the two is not yet formally agreed(as-of Oct 2022).

Update
CF refers to netCDF docs @4.9, which does make a statement here
( see "Conventions" in here )

It is possible for a netCDF file to adhere to more than one set of conventions, even when there is no inheritance relationship among the conventions. In this case, the value of the 'Conventions' attribute may be a single text string containing a list of the convention names separated by blank space (recommended) or commas (if a convention name contains blanks).


Expect nav state....

When i use FluidNavigator to replace StackNavigator, i created a nested navigation(Tab in Stack, Stack in Drawer) and got a issue:

Expect nav state to have routes and index, {"routeName":"Main","key":"Init"}

If i change Tab to a screen, it work fine. :) with FluidNavigator.
Any help? Thanks

i tried using your Example to reproduce my issue. Finally, i got it work by that:

Change
TabsNav: {
screen: TabsNav,
}
to:
TabsNav: {
screen: (() => ),
}
But i can not see my Header of my Stack? How this happen?
Can you help me pls? Really want using this library. Thanks 👍

I'm not sure why this happens - feel free to fork this project and see if you can find something that could help!


SyntaxError: Unexpected token '?'

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
just run the bot
E:\Programs\gitrepos\ScrimBot\src\services\matchCreation.js:77
        matchRecord.creationInformation.map = CONSTANTS.MAPS.find(e => e === content) ?? content
                                                                                ^

SyntaxError: Unexpected token ?

Expected behavior
A clear and concise description of what you expected to happen.
Bot Version
1.4.0
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hi, could you try using Node 14 or greater and let me know if that works?

Hi, could you try using Node 14 or greater and let me know if that works?

that did the trick
now getting
Error: Value for argument "documentPath" is not a valid resource path. Path must be a non-empty string.

Do you have a line or any type of stacktrace for that? Does it happen when the bot starts up or is it when a specific command is run?


define Ushahidi ACL

Subject of the issue
UShahidi can support a permission structure as follows

super admins
users
volunteers

The forms should be either public or private based on who is collecting the data
eg:

missing person form should be public
damage assessment should be  assigned to only volunteers and government officials
various groups like the one should be accessible to officials

Forms and permissions

Missing persons : public can report, volunteers can verify & make the report public
Damage assessment : volunteers can report, officials only can see reports.

CC: @biswaz @vigneshhari @naveenpf @syam3526

#918  - related.


PMKArray leaking memory!?

I'm currently using PromiseKit in Objective-c (v6.2.8)
The problem is when I'm trying to reject a promise like this:
myPromise.then(^(task, result){
    //...
    if (somethingWentWrong) {
        @throw [NSError errorWithDomain:domain code:code userInfo:nil];
    }
    //...
}).then(^((task, result)){
    // in case of something going wrong I don't want to enter here
    //...
}).catch(^(NSError *error){
    //...
});

the PMKArray property leaks for some reason including the result property passed to the promise (Even using the PMKError.canceled results in the same thing).
debug memory graph:

I was previously using PromiseKit (v4.1.7) but the same problem was there as well.
for (v4.1.7) I was throwing
@throw [NSError cancelledError];

I've looked around a bit, but wasn't lucky enough to find anything about this. I'm not sure if I'm doing something wrong, but any help or insight would be appreciated.
A workaround I did to fix this, is, instead of throwing an error I would return a Promise with an error value as follows
myPromise.then(^(task, result){
    //...
    if (somethingWentWrong) {
        return [AnyPromise promiseWithValue:[NSError cancelledError]];
    }
    //...
}).then(^(task, result){
    // in case of something going wrong I don't want to enter here
    //...
}).catch(^(NSError *error){
    if ([error isEqual:NSError.cancelledError]) {
            return;
    }
    //...
});

For me to be able to throw the PMKError.cancelled using Objective-C, I've added the following extension
extension NSError {
    class var cancelledError: NSError {
        return PMKError.cancelled as Error
    }
}

By doing that the PMKArray no longer leaks memory, but I'm not sure if this is a correct use of the PromiseKit framework!

PMKArrays are not created by much, the only instance I can think of is when using NSURLSession promises. Are you creating them somewhere?
Certainly, throwing errors does create a PMKArray. PMKArray is an internal class used by the PMKManifold function.

K well, it turns out @throw violates ARC:
https://www.eltima.com/dev/2014/08/11/objective-c-arc-and-exceptions/
Nothing we can do. If you don't want memory leaks, don't @throw.

Oh, apparently you can enable -fobjc-arc-exceptions and this will be fixed.
PR welcome to supplement our docs.

Thanks a lot for the quick reply!
Will be updating the documents regarding this issue later today/tomorrow.
Also just to confirm, is the workaround I've added valid? (where instead of throwing I would return a Promise with value Error) It seems to be working just fine, but just in case.
myPromise.then(^(task, result){
    //...
    if (somethingWentWrong) {
        // @throw [NSError cancelledError];
        return [AnyPromise promiseWithValue:[NSError cancelledError]];
    }
    //...
}).then(^(task, result){
    // in case of something went wrong I don't want to enter here
    //...
}).catch(^(NSError *error){
    if ([error isEqual:NSError.cancelledError]) {
            return;
    }
    //...
});

It's valid yes.


Docker image too big

To make it short the docker image is far too big (> 1 GB) and there are several reasons for that:
# [...]

#-> install software, creates a layer ~190 MB
RUN apk add --no-cache bash openssh ca-certificates jq curl openssl perl git zip \
 && apk add --no-cache --virtual .build-deps gcc make openssl-dev libffi-dev musl-dev linux-headers \
 && apk add --no-cache libintl icu-libs libc6-compat \
 && apk add --no-cache bash-completion \
 && update-ca-certificates

ARG JP_VERSION="0.1.3"

#-> download jmespath, creates a layer ~4 MB
RUN curl -L https://github.com/jmespath/jp/releases/download/${JP_VERSION}/jp-linux-amd64 -o /usr/local/bin/jp \
 && chmod +x /usr/local/bin/jp

WORKDIR azure-cli

#-> uploading all the sourcecode, creating a layer ~20 MB
COPY . /azure-cli

#-> run installation, creating a layer ~880 MB
RUN ./scripts/install_full.sh \
 && cat /azure-cli/az.completion > ~/.bashrc \
 && runDeps="$( \
    scanelf --needed --nobanner --recursive /usr/local \
        | awk '{ gsub(/,/, "\nso:", $2); print "so:" $2 }' \
        | sort -u \
        | xargs -r apk info --installed \
        | sort -u \
    )" \
 && apk add --virtual .rundeps $runDeps

WORKDIR /

#-> removing sourcecode, which just creates another layer which marks all the previous added files as deleted ~1kb
RUN rm -rf ./azure-cli && \
    dos2unix /root/.bashrc /usr/local/bin/az

the rm -rf command is increasing the size by marking all sourcefiles as deleted. you might want to use a multi stage build here for building and then creating the final image.
image was analyzed with dive which shows you all the added files: https://github.com/wagoodman/dive
also related to https://github.com/Azure/azure-cli/issues/23286

@jiasli for awareness

This issue was submitted and fixed before (see https://github.com/Azure/azure-cli/issues/12407). Please make this fix more permanent.

Close as https://github.com/Azure/azure-cli/pull/27567 and https://github.com/Azure/azure-cli/pull/27683 are merged.
Image size will drop to 700MB in 2.54.0


Reserved keywords as hash keys

{
	default: true,
	private: true,
	public: true,
	break: true,
	other: true,
}

AFAIK there is not semantically difference between these hash keys, and they are all completely valid according to the spec and implementations, nonetheless "default" and "break" are colored differently, probably because they are reserved keywords.
I guess they should still be exclusively colored when used incorrectly, as in var default = true;.

This change has the unfortunate effect of matching the default keyword in a switch as string.unquoted.label.js instead of keyword.control.switch.js


iOS: "java.io.IOException: Couldn't get lock for" while create instance of FileHandler

Hi! I'm trying to add logging to file in iOS (in Android it works fine), but when I create an instance of FileHandler, get IOException telling that "Couldn't get lock for [my absolute path to log file]". What I'm doing wrong?
Problematic piece of code:
FileHandler handler = new FileHandler(getFilePath(name), MAX_LOG_FILE_SIZE,
MAX_LOG_FILE_COUNT, true);
P.S.: I checked the existence of all folders in the path - everything is fine.

Here's where the exception is thrown, which suggests that your app is opening more than 100 (MAX_LOCKS) log files. That sounds unlikely, but if that's true then remember your iOS device isn't a server. :-)
More likely is that this is JDK-8153955, which is fixed in Java 8/Android API 26 (Oreo). However, this fix depends upon issue #1064 (add java.nio.file package, also introduced in Oreo).

Am I right, this fix [https://github.com/google/j2objc/issues/1064] is in progress?

Yes, but implementing a new file system API correctly is a big task, especially when it requires new native code. So we're looking at weeks, not hours.
In the meantime, consider setting MAX_LOG_FILE_COUNT to be less than FileHandler's MAX_LOCKS count of 100. Needing more than 100 locks is what caused the IOException to be thrown.


support generator middleware

Change-Id: Iae86b8ce08c70fa8393c5c55508279607a47fab7

solve https://github.com/alexmingoia/koa-router/issues/468


[1.2.x] Bump Quarkus to 2.13.9.Final

Summary
Bump Quarkus to 2.13.9.Final
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[ ] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests


Latest version of ipyvolume not working

I will try to detail my problem as much as possible. I am using conda. When installed via conda, using
conda install -c conda-forge ipyvolume
ipyvolume works exactly as expected. However, ipyvolume available through conda is ipyvolume==0.4.6. I want to get the latest version. So I uninstall ipyvolume in conda:
conda uninstall -n myevn ipyvolume
and reinstall via pip in my conda env:
pip install ipyvolume
Now, when I run the exact same code, the cell executes without error, but there is no output.  I tried using pip install ipyvolume==0.4.6 , but still had the same problem. I tried using a separate pipenv environment where I tried both 0.4.6 and 0.5.1 on the same code to no avail.
When I uninstall the pip version in my conda env, and reinstall through conda, the same code runs exactly as expected. Any ideas?

If I look at https://anaconda.org/conda-forge/ipyvolume/files I see 5.0 and 5.1 available, which OS are you using?
Anyway, the second issue might be this:
I think you might be bitten by ipyvolume being disabled on uninstalling, try these commands:
# check what is enabled and not
$ jupyter nbextension list
# if ipyvolume is disabled, enable it
$ jupyter nbextension enable --py --sys-prefix ipyvolume


cmd/go: add a flag to set (or exclude) release tags

In #50825, we want to allow developers to build gopls with a recent version of Go while still developing for older versions of Go.  One blocker to doing this properly is that the set of release tags used in go list is not configurable: they are implied by the version of cmd/go.
We had speculated that #42504 (ignore all build constraints) could help here, but the amount of post-processing required by gopls in that case is prohibitive.
Can we add a flag to cmd/go that allows us to mutate the set of release tags? For example, could we:

add a -release flag (-release=1.16)?
overload the -compat flag (-compat=1.16)?
add a -notags flag (-notags=go1.17,go1.18)?

I defer entirely to the cmd/go team for flag design, but it would be great to have some way to achieve this.
CC @bcmills @matloob

I've been thinking about this some more, but I'm not sure that I really grokked the problem space.
If the user has a gopls built with, say, go 1.18, and they're trying to develop against go 1.17, won't the go command itself (and its GOROOT) be 1.17? In that case, go list will have the correct flags by default.
(I don't see how users can realistically target an older Go version that they don't even have installed, given that the packages and identifiers in the Go standard library also change from one release to the next — are we planning to have gopls also filter out packages and identifiers based on GOROOT/api or similar?)

are we planning to have gopls also filter out packages and identifiers based on GOROOT/api or similar?

Yes, that's part of the plan (see #50825) - add an analyzer to allow developers to preserve compatibility with older go versions independent of their installed version. For example, we frequently have this problem while working on gopls, that one doesn't discover that one has used a disallowed API until after mailing the CL and getting a build failure. Presumably this would also be useful for working on the compiler, which has a bootstrap requirement.
But maybe we can insist that if users want the extra accuracy of building packages exactly as go 1.N, they must use go 1.N for go list. In that case the current request become unnecessary. It's also the case that if we analyzed function bodies for non-workspace packages, we'd want to have an accurate GOROOT (lest we infer inaccurate facts for the target Go version)
I trust your intuition if you think this feature request is a bad idea. In light of the preceding paragraph I'm inclined to agree. Please feel free to close this based on your judgement.


Update and fix bug on case-sensitive map

Codecov Report

Merging #79 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master      #79      +/-   ##
============================================
- Coverage     75.75%   75.64%   -0.11%
  Complexity      235      235
============================================
  Files            51       51
  Lines           965      965
  Branches        334      334
============================================
- Hits            731      730       -1
  Misses           52       52
- Partials        182      183       +1

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/security/packaging/SafetyDbChecker.kt
70% <0%> (-1.43%)
20 <8> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5936731...3dc034f. Read the comment docs.


xen: patch with XSA-462

Patch is still embargoed.Expected release date: 2024-09-24 @ 12:00 PM UTC
Description of changes

Patches Xen with XSA 462.

Things done

Built on platform(s)

[ ] x86_64-linux

[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
[ ] Tested, as applicable:

XTF PoC test, if available, passes.
Nothing seems to immediately explode.

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

Note: This is only important if the XSA affects the Xen development libraries.

[x] Fits CONTRIBUTING.md.

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

They're running late with the disclosure and the XSA commit(s) aren't on xen.git yet.

x86: Deadlock in vlapic_error()
In x86's APIC (Advanced Programmable Interrupt Controller) architecture,
error conditions are reported in a status register.  Furthermore, the OS
can opt to receive an interrupt when a new error occurs.
It is possible to configure the error interrupt with an illegal vector,
which generates an error when an error interrupt is raised.
This case causes Xen to recurse through vlapic_error().  The recursion
itself is bounded; errors accumulate in the the status register and only
generate an interrupt when a new status bit becomes set.
However, the lock protecting this state in Xen will try to be taken
recursively, and deadlock.

Slim Xens built, optimistically merging while running a full review.


Add home manager module to flake outputs

It looks like the NixOS module was made available for flakes, but the home manager module was not. This change adds it to the flake outputs and updates the documentation to show how to use it.

Does the home manager module work for you? No issues like #30?

I added this to my personal config yesterday and didn't see any issues like that when I rebuilt three machines.

Closing in favor of #34, I think it makes more sense to have this under nixosModules.


Gallery tab changes

@ssylver93 Looking better. There are a few conflicts on the branch that'll need to get resolved first


Add __reduce__ to Body class

This change does seem to solve the problem mentioned in #1395. __reduce__ returns a string, which I think is commonly done for singleton classes. As a result, I was able to get the plot:

Just wondering if this is really a solution to the issue...

I used the tmp_path fixture (https://docs.pytest.org/en/latest/how-to/tmp_path.html#the-tmp-path-fixture) to save and load objects into a file for testing. Had not used it before though. Hopefully it's not an overkill :sweat_smile:


Remove support for Internet Explorer 11 as of July 2018?

GitHub is removing support for Internet Explorer 11 soon:

Should utterances remove it's support for IE 11 at the same time?
There's a few IE polyfills/hacks in the codebase:

polyfills:
https://github.com/utterance/utterances/blob/e82d1df8957ab6c1c81e7618a585d6ea79d5b36d/src/index.html#L19-L28

document.currentScript not supported.
https://github.com/utterance/utterances/blob/e82d1df8957ab6c1c81e7618a585d6ea79d5b36d/src/client.ts#L4-L9

@jdanyow The world is moving forward. If we don’t guide users to abandon IE, why don’t we use IE ourselves?


[Diverged] Replace hand-written parser with selery

See selery.

This branch has diverged, left here as a reference to prior work, but need to start again.


Add PartyKit to drop page

Would be nice to add party kit to drop page and maybe to collection page
https://www.partykit.io/
At the drop page

initial status would be "generating" at cursor when user us clicked on generating
"Minting"
we could show at cursor their soon being minted generation picture in some small preview like 64x64

At collection
We could show top5 or top10 cursors which had bought most nfts?

Ref

https://github.com/kodadot/research/issues/135