Implement starknet.go support for transaction endpoints

Current State
The RPC Request Builder autogenerates starknet.go code snippets for StarkNet RPC endpoints. However, some endpoints are not yet supported.
Required Action
Add autogenerated starknet.go support for the following endpoints.

starknet_getTransactionStatus
starknet_getTransactionByHash
starknet_getTransactionByBlockIdAndIndex
starknet_getTransactionReceipt
starknet_getBlockTransactionCount
starknet_getEvents

Additional Context:

https://github.com/NethermindEth/rpc-request-builder/issues/1
https://github.com/NethermindEth/rpc-request-builder/blob/main/src/api/rpcspec/methods.ts

Would like to take it

@jelilat I picked this up since it was open for a while

@Ash20pk I was waiting for another PR that introduces the go template, since it's been merged I am raising a PR soon.


GitHub labels are unreadable

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

What userstyles are you seeing the problem on?
lbl:github
Describe your problem.
Labels on Issues/Pull Requests are extremely bright, causing them to be completely unreadable.
Attach screenshots.

What browser(s) are you seeing the problem on?
Firefox Developer Edition 121.0b1
Any additional comments?
No response

Removing the rules for .hx_IssueLabel fixes this issue. They look pretty broken (no background opacity... etc.)


import package alias not generated by protoc-gen-fastmarshal

Title
alias not generated in pb.fm.go file by protoc-gen-fastmarshal
Version
v0.24.3
Repro steps
I have two proto3 files =>
api/xyz/grpc/xyz.proto:
option go_package = "x/y/z/grpc;xyzgrpc";
api/abc/grpc/abc.proto:
import "api/xyz/grpc/xyz.proto";
Generated abc.pb.fm.go file has import statement as
import (
"x/y/z/grpc"
)
Due to this, I am getting compilation error as - "x/y/z/grpc" imported as xyzgrpc and not used"
Expectation is to have
import (
grpc "x/y/z/grpc"
)
pb.go file generates correctly. Problem is only with fm.pb.go file.

@rohsaini Thanks for the bug report.  Can you try it with the fix I pushed to the respect-go-package-aliases-in-addtl-imports branch?  You should be able to install it with go get github.com/CrowdStrike/csproto/cmd/protoc-gen-fastmarshal@respect-go-package-aliases-in-addtl-imports.
Let me know if that works for you.

Thanks @dylan-bourque . This didn't work as expected. My abc.pb.fm.go now looks like this, which is incorrect:
package abcgrpc
import (
abcgrpc "x/y/z/grpc"
)

Can you link to your actual code? I tried that locally and it correctly used the xyzgrpc import alias from the .proto file.

One thing I noticed is that your xyz.proto doesn't have a package declaration, which I thought might be what was going on.  I "fixed" that, though, and the generated code is still wrong.  After debugging I was able to figure out that the Protobuf file associated with your AddrInfo field is abc.proto, not xyz.proto, which is why the import alias in abc.fm.pb.go is abcgrpc.
Unfortunately, I don't have any more bandwidth to look into this now.  I'll circle back to it as soon as I can.

Sure @dylan-bourque . Thanks.

@dylan-bourque thanks for your quick response! Although I'm afraid you went to the wrong direction: package names already parsed and passed to plugin, so plugin only has to use this information.
I created a PR #123, feel free to merge it or to discard and use as a reference.

@biosvs I ended up adapting your code a bit to fit into what I had already done.  My "miss" was using fld.Desc (the file descriptor associated with the field declaration) instead of fld.Message.Desc and fld.Enum.Desc (the file descriptor associated with the field's type) to locate the parent file.  I definitely appreciate the input and extra eyes.
@rohsaini I tried this update against your example locally and it did the right thing as far as I can tell.  Let me know if this works for you.

Thanks @dylan-bourque and @biosvs . This is working fine now. I just want to bring to your notice - o/p of pb.fm.go import is not same as pb.go file. It is upto you if you want to keep similar output or not. Things are compiling and fine otherwise.
pb.fm.go:
import (
xyzgrpc "xyz/grpc"
)
...
...
xyzgrpc.SomeMethod()
pb.go:
import (
grpc "xyz/grpc"
)
...
...
grpc.SomeMethod()

@dylan-bourque Any update please?

I was head-down on a project at work all of last week and didn't have time to circle back on this.  I'm inclined to merge this as is since it addresses the underlying problem and exactly matching the code generated by protoc-gen-go is not one of my goals.
An argument could even be made that this code is more correct since it's using the explicit Go package name from the .proto file instead of a technically unnecessary alias that uses the last token of the import path. 😉

Sure. Please go ahead and merge.

@rohsaini I just merged #126.  Haven't tagged a new release yet but you can grab the new build right away if you go install github.com/CrowdStrike/csproto/cmd/protoc-gen-fastmarshal@main.

@dylan-bourque Thanka Dylan. Request you to tag it, as main will keep moving forward. I would like to align to some fixed tag.

Okay. Thanks. I'll wait for next tag (whenever available) then.

v0.25.0 was tagged this morning and includes this change.  Let me know if you find any other issues.
Thank you for the contribution.


Report does not list all the installed gems

I'm not very familiar with Ruby so my apologies if this is a noob question.
I am trying to find licenses of all the gems installed on an instance. Regardless of which path I'm on, when I run 'gem list' I get a long list of all the installed gems. However, when I run license_finder report, it does not list anything.
I tried running license_finder report while on root with the flag --recursive and see the following error:
[root@8484963fa169 /]# license_finder report --recursive
/usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler.rb:211:in `rescue in root': Could not locate Gemfile or .bundle/ directory (Bundler::GemfileNotFound)
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler.rb:207:in `root'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler.rb:225:in `app_cache'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/source/rubygems.rb:455:in `cache_path'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/source/rubygems.rb:23:in `initialize'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/source_list.rb:12:in `new'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/source_list.rb:12:in `initialize'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/dsl.rb:22:in `new'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/dsl.rb:22:in `initialize'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/dsl.rb:10:in `new'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/dsl.rb:10:in `evaluate'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/bundler-1.14.6/lib/bundler/definition.rb:33:in `build'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_managers/bundler.rb:30:in `definition'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_managers/bundler.rb:49:in `included_groups'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_managers/bundler.rb:12:in `current_packages'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_manager.rb:60:in `current_packages_with_relations'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_manager.rb:23:in `each'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_manager.rb:23:in `flat_map'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/package_manager.rb:23:in `current_packages'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/core.rb:63:in `current_packages'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/core.rb:58:in `decision_applier'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/license_aggregator.rb:21:in `block in aggregate_packages'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/license_aggregator.rb:19:in `each'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/license_aggregator.rb:19:in `flat_map'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/license_aggregator.rb:19:in `aggregate_packages'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/license_aggregator.rb:10:in `dependencies'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/lib/license_finder/cli/main.rb:73:in `report'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/thor-0.19.4/lib/thor/command.rb:27:in `run'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/thor-0.19.4/lib/thor/invocation.rb:126:in `invoke_command'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/thor-0.19.4/lib/thor.rb:369:in `dispatch'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/thor-0.19.4/lib/thor/base.rb:444:in `start'
	from /usr/local/rvm/gems/ruby-2.3.0/gems/license_finder-3.0.0/bin/license_finder:5:in `<top (required)>'
	from /usr/local/rvm/gems/ruby-2.3.0/bin/license_finder:22:in `load'
	from /usr/local/rvm/gems/ruby-2.3.0/bin/license_finder:22:in `<main>'
	from /usr/local/rvm/gems/ruby-2.3.0/bin/ruby_executable_hooks:15:in `eval'
	from /usr/local/rvm/gems/ruby-2.3.0/bin/ruby_executable_hooks:15:in `<main>'

What am I doing wrong?

Hi vivekkodira1,
LicenseFinder is really meant to find the dependencies of Ruby based applications based on a Gemfile. So a typical usage would be to have checked out source code (such as license finder's own repo) and to run license_finder report in the directory you cloned the repo into.
LicenseFinder doesn't enumerate the licenses for all gems installed on an instance.

Hi @kdykeman,
Thank you for taking the time to respond in detail. I will update the SO answer which brought me here accordingly.
Regards,
Vivek.

@vivekkodira1
Thanks, it looks like you found a good path for what you needed outside of LicenseFinder.
Closing this issue.


Taekwondo Intro Session

Session Name
Kick off your SUTD life
Abstract
Learn more about Taekwondo, bring your skills to greater heights!
Student Organisation organising this session
Taekwondo Club
Session intent (what to expect)
This introduction session is opened to

Participants will have a greater understanding of Taekwondo as a sport and its different aspects (poomsae and sparring)
For participants that are new to the sport, they would learn the basic stances and kicks
For participants with experience, they would learn sparring techniques

Session details
Venue: ISH Dance studio 6&7
Date: 24 May (Friday)/ 29 May 2019 (Wednesday)
(We are allocated Fridays, but there may be low attendance on Fridays if freshmores want to go home)
Our trainings will be open for freshmores to join even if they are not able to make it for the open trainings
Time: 6pm to 8pm
Up to 20 participants only
Speakers/Workshop facilitators
Facilitators would be existing club members, headed by the black belts and assisted by other members.
Logistical details
Drinks(e.g. milo isotonic drinks) - $30
email<EMAIL_ADDRESS>

would you like to give your second session a unique name?
eg.
Kick off your SUTD life round 2

hello, also do take note that 29th May will be the freshmore inauguration dinner so freshmores won't be available

Hi, we have decided to hold the two sessions on 24 May and 31 May instead. Will there be any conflicts in this case?

bump @Fishbiscuit


chore(build): run lint as a separate job

This separates the lint step out from the other build steps.  There are a few reasons this is good:

There's no need to run lint and license check for each version of nodejs we want to support
We need nodejs 4 support for this library, and gts don't support it

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   90.14%   90.14%
=======================================
  Files           2        2
  Lines          71       71
  Branches        4        4
=======================================
  Hits           64       64
  Misses          7        7

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 64edeac...096a9a9. Read the comment docs.


Missing from Chrome Web Store

Hello, I have this extension and wanted to share with a colleague.
Alas, it's no longer in the Chrome Web Store
When I click "See in Chrome Web Store" I get a 404 here: https://chrome.google.com/webstore/detail/fphfgdknbpakeedbaenojjdcdoajihik
Will it be available again somehow?

Looking into it. Have no idea why it was removed

I still can't find out what made Google remove it.

Dear Developer, Your Google Chrome item, "Key Socket Media Keys," with ID: fphfgdknbpakeedbaenojjdcdoajihik did not comply with our policies and was removed from the Google Chrome Web Store. Your item did not comply with the following section of our policy:
Do not post repetitive content.
Do not attempt to change the placement of any Product in the store, or manipulate any Product ratings or reviews, by unauthorized means, such as fraudulent installs, paid or fake reviews or ratings, or offering incentives to rate Products.
Do not post an app if the primary functionality is to link to a website not owned by the developer posting the app.
Do not post an app where the primary functionality is to install or launch another app, theme, or extension. For example, you cannot post an app if its primary function is to launch a desktop app that the user has already installed. Another example of a disallowed practice would be to post a packaged app that just launches a website.
Your app must comply with Google's Webmaster Quality Guidelines
If you'd like to re-submit your item, please make the appropriate changes to the item so that it complies with our policies, then re-publish it in your developer dashboard. Please reply to this email for issues regarding this item removal.

I tried to leave minimum amount of text in desc, but it didn't help.

I'm seeing the extension back up again 😃


process

Why is mongodb_consistent_backup user running polkitd process on a server i didnt install it on? This server is not even a MongoDB server.

This is not run by this tool, this is a python library something else like s3cmd via pythons boto library would maybe cause this, but it’s normally systemd that would trigger it.
Sent from my iPhone

On Nov 30, 2017, at 17:25, Oren Bissick<EMAIL_ADDRESS>wrote:
Why is mongodb_consistent_backup user running polkitd process on a server i didnt install it on? This server is not even a MongoDB server.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@dbmurphy is correct, this tool only spawn Python and/or "mongodump" binaries. You can confirm this also by reading the python source code.
If polkitd is running as the same user as the 'mongodb_consistent_backup' user this could be due to duplicated numeric user ids or it was started before a change to /etc/passwd.


Swagger - weird behaviour in /api/infographic-data

The FE team reported a weird behaviour in /api/infographic-data:

@ziv17 any thoughts on this? I assume that news flash 123 does not exist. What do we want to return in such cases?

Correct error messages are not implemented. In this case, the news-flash is not an accident, and has no location.
What is the priority of this?


Remove question and answer text from existing rules in the DB

This will be cleaner with Promise.All and then we can set the modal on all api calls.

For now we are going with the simpler feature where question text can only be edited in draft mode and will clear all answers


Unable to Read a good worksheet if preceeeded by a bad worksheet.  OpenWorksheetAsync not reseting curFieldCount member to -1 when advancing from one sheet to the next.

Found an issue with a spreadsheet provided by a third party, where they have a "bad" sheet preceding the sheet we need to read.
Tracked it to the Read() method, where it will set this.rowFieldCount = curFieldCount if curFieldCount is 0 or greater.
In the case I encountered, the former sheet had a rowFieldCount of 0, and a parsedRowIndex greater than 0 after Parsing, so the XlsbWorkbookReader set the state of curFieldCount to 0, and any subsequent NextResult() never resets the curFieldCount to -1

Thanks a lot, and I'm glad I could help.  We are finding this library to be hugely valuable, so thank you for your effort!


Increasingly negative loss in denoising autoencoder

Hi,
I am training 3 layers stacked denoising autoencoder which has a bit of difference in loss function.
I want to make autoencoder that tries to reconstruct the 'global' input (not previous layer's output), which means the original input that was fed to the first layer, using normally obtained input which is previous layer's output.
I just edited all parameter 'self.x' to 'self.x_global' on def 'get_cost_updates' in layers/da.py.
self.x_global means the original input that was fed to the first layer (self.x in models/sda.py).
And the result of training was like this.
[2016-12-04 01:36:48.845309] > ... training the model
[2016-12-04 01:37:01.856494] > layer 0, epoch 0, reconstruction cost 405.427734
[2016-12-04 01:37:15.579682] > layer 0, epoch 1, reconstruction cost 381.404175
[2016-12-04 01:37:29.242537] > layer 0, epoch 2, reconstruction cost 377.724701
[2016-12-04 01:37:43.045209] > layer 0, epoch 3, reconstruction cost 375.875977
[2016-12-04 01:37:56.615403] > layer 0, epoch 4, reconstruction cost 374.741211
[2016-12-04 01:38:11.105572] > layer 1, epoch 0, reconstruction cost -108174.476562
[2016-12-04 01:38:24.891239] > layer 1, epoch 1, reconstruction cost -334065.656250
[2016-12-04 01:38:38.807076] > layer 1, epoch 2, reconstruction cost -561826.187500
[2016-12-04 01:38:52.979225] > layer 1, epoch 3, reconstruction cost -790545.687500
[2016-12-04 01:39:07.143726] > layer 1, epoch 4, reconstruction cost -1019794.250000
[2016-12-04 01:39:21.975468] > layer 2, epoch 0, reconstruction cost -152930.156250
[2016-12-04 01:39:36.551489] > layer 2, epoch 1, reconstruction cost -460353.750000
[2016-12-04 01:39:51.328428] > layer 2, epoch 2, reconstruction cost -767839.625000
[2016-12-04 01:40:05.910295] > layer 2, epoch 3, reconstruction cost -1075358.750000
[2016-12-04 01:40:20.484577] > layer 2, epoch 4, reconstruction cost -1382889.500000
Reconstruction cost goes increasingly negative.
Is it normal? What it means??
Here is my edited code (just self.x to self.x_global in original code).
self.x_global is self.x in models/sda.py (original input)
###################################################
def get_last_cost_updates(self, corruption_level, learning_rate, momentum):
""" This function computes the cost and the updates for one trainng step of the dA """
    tilde_x = self.get_corrupted_input(self.x, corruption_level)
    y = self.get_hidden_values(tilde_x)
z = self.get_reconstructed_input(y)
    L = - T.sum(self.x_global * T.log(z) + (1 - self.x_global) * T.log(1 - z), axis=1)
#L=0

    if self.reconstruct_activation is T.tanh:
        L = T.sqr(self.x_global - z).sum(axis=1)
#    L=0

    if self.sparsity_weight is not None:
        sparsity_level = T.extra_ops.repeat(self.sparsity, self.n_hidden)
        avg_act = y.mean(axis=0)

        kl_div = self.kl_divergence(sparsity_level, avg_act)

        cost = T.mean(L) + self.sparsity_weight * kl_div.sum()
    else:
        cost = T.mean(L)

    # compute the gradients of the cost of the `dA` with respect
    # to its parameters (derivative cost with respect to params)
    gparams = T.grad(cost, self.params)
    # generate the list of updates
    updates = collections.OrderedDict()
    for dparam, gparam in zip(self.delta_params, gparams):
        updates[dparam] = momentum * dparam - gparam*learning_rate
    for dparam, param in zip(self.delta_params, self.params):
        updates[param] = param + updates[dparam]

    return (cost, updates)

###################################################

I'm not sure if this is caused by overflow.
I see two possible paths for calculating L and cost. Do you know which path is actually executed in your experiment?


Ensure that the libs with trailing number (such as libnetcdf-0d64ea41.so.13) are also found.

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

Actually not needed, this would be confusing  (for instance ecmwflibs.find('netcdf') would return a netcdf .so that is not used by other packages).
Closing.


FormHelper->button() should set btn-default

    echo $this->Form->button('Sign In');

Above outputs
    <button type="submit" class="btn">Sign In</button>

Should be outputting with btn-default.
    <button type="submit" class="btn btn-default">Sign In</button>

But it should be smart enough to know when not to include it.
    echo $this->Form->button('Sign In',['class'=>'primary']);

Should output with btn-primary and not btn-default.
    <button type="submit" class="btn btn-primary">Sign In</button>

Since this could impact previous versions. I wanted to verify this was okay before doing a PR.

We could add btn-default is $options['class'] is empty. I doubt anyone would want to generate button with just btn class so it shouldn't impact existing users.

great. I'll do a PR for it.


[SAFARI] missing webNavigation apis (lastCheckedAppleForum=Dec15th, lastCheckedSafari=Dec15th, v14.0.2)

Unhandled Promise Rejection: TypeError: undefined is not an object (evaluating 'this.api.webNavigation.onHistoryStateUpdated.addListener')

I don't remember seeing this in the betas

Asked about this:
https://developer.apple.com/forums/thread/661293

Still an issue in Safari 14.0.1 beta 1:

Bumped issue on Apple Forum:
https://developer.apple.com/forums/thread/661293?answerId=644077022#644077022

Apple finally responded
https://feedbackassistant.apple.com/feedback/8730949

Apple finally responded
https://feedbackassistant.apple.com/feedback/8730949

!The apis are still missing!


Enhance Windows Service HTTPS coverage (auth note)

Fixes  #14239

The ask is to note in passing that HTTPS should be configured for a Windows Service that uses auth. I think we can all this out with a change in the existing coverage for endpoint configuration.
I'm going to remove "(Kestrel)" and "(HTTP.sys)" in passing because those are called out in the topic titles that are rendered in the topic.

cc: @CeciAc @taori

ping @blowdart ... 20 second review ... this is just to call out and cross-link better for HTTPS (endpoints) with Windows Services.


Overlay grid

Usage:
scene = GridOverlay(scene, spacing=20, label_every=5)

will produce something like this:

Note this PR also adds a corner argument to Drawable.move, which is used by the new GridOverlay class. But GridOverlay can of course be rewritten without that if you don't like it.
Also, current implementation only makes sense on bright backgrounds, since grid lines are black. We could just make the path style an argument, but it would be nice to not have to pass two arguments (for normal and important lines). Maybe just make the color an argument, and then use different opacities like now for the different lines? That also gives us a color for the text.

Implemented the configurable color as described above:

That means it also works on dark background as long as we set a bright color:

(the higher opacity on intersections is more visible here, but aesthetics are arguably not as important for this feature anyway)


AMI build automation.

This change adds the ability for travis to build an AMI automatically.

Noice.


Added missing registration of RelatedPublicationsComponent.

I moved the registration to the PublicationsServiceProvider. Left a code review comment on it. Can you verify if the component is not working? Make sure you use <x-hyde-publications::related-publications />
https://github.com/hydephp/develop/blob/7c95364c8b08c137b90ee965cb68aec4bce65cd1/packages/publications/src/PublicationsServiceProvider.php#L47

Closing as current implementation in confirmed to be working.


[release-v1.9 ]Run go mod edit golang x net

Running:

go mod edit -replace<EMAIL_ADDRESS>make RELEASE=1.9 generate-release

However, before I could successfully run make RELEASE..., I had to call:

./hack/update-deps.sh

Due to this:
make RELEASE=v1.9 generate-release
./openshift/release/generate-release.sh
=== Update Codegen for knative.dev/eventing-kafka-broker
--- Generating checksums for configmap _example keys
go: inconsistent vendoring in /home/matzew/go/src/knative.dev/eventing-kafka-broker:
	golang.org/x/net: is replaced in go.mod, but not marked as replaced in vendor/modules.txt

	To ignore the vendor directory, use -mod=readonly or -mod=mod.
	To sync the vendor directory, run:
		go mod vendor
make: *** [Makefile:69: generate-release] Error 1

/retest

@pierDipi this PR now also contains the code from this PR https://github.com/openshift-knative/eventing-kafka-broker/pull/857


How do I remove response headers?

Many application frameworks provide the ability to remove response headers, how is this done in Grape?
I see there is stuff like version 'v1', using: :header, vendor: 'twitter', cascade: false to turn off certain headers, but I want to harden my webserver and remove them completely.
My goal is to implement the same headers that the expressJS library "helmet" uses to harden nodejs servers, but at the same time the same library removes headers for you like X-Powered-By, which in my case is added by my Passenger server, which does not provide me with the ability to remove it...
< Content-Security-Policy: default-src 'self';base-uri 'self';block-all-mixed-content;font-src 'self' https: data:;frame-ancestors 'self';img-src 'self' data:;object-src 'none';script-src 'self';script-src-attr 'none';style-src 'self' https: 'unsafe-inline';upgrade-insecure-requests
< ETag: W/"3e-DqINi7yil7BpHYD6XuyvL1LYGGo"
< Expect-CT: max-age=0
< Referrer-Policy: no-referrer
< Strict-Transport-Security: max-age=15552000; includeSubDomains
< Vary: Accept-Encoding
< X-Content-Type-Options: nosniff
< X-DNS-Prefetch-Control: off
< X-Download-Options: noopen
< X-Frame-Options: SAMEORIGIN
< X-Permitted-Cross-Domain-Policies: none
< X-XSS-Protection: 0

I see that many libraries provide the ability to remove headers, but I cant find the response variable where response headers are kept in order to edit it
for instance:

ExpressJS
app.use(function (req, res, next) {
  res.header('Pragma', 'no-cache');
  res.removeHeader('Pragma');
  next();
});

Rails
response.headers['Connection'] = 'Closed'
remove_keys = %w(X-Runtime Cache-Control Server Etag Set-Cookie)
response.headers.delete_if{|key| remove_keys.include? key}

You can set headers with header, so header X, nil will remove it, same as in Rails. See https://github.com/ruby-grape/grape/blob/master/lib/grape/endpoint.rb#L273 where headers are returned to the rack middleware stack. That said, there's a whole stack of other middleware involved in a response, and each may be altering/adding/removing headers, thus depending on how Grape is mounted removing something may not actually be removing it (because another middleware, e.g. rack-cache, would re-add it.
So the answer is "it depends" and "grape might not be the right place to do it". Do you have a running example where a header is returned that you want removed?


Add Trusted senders and domains,

"Add trusted senders and domains: Exceptions to the impersonation protection settings. Messages from the specified senders and sender domains are never classified as impersonation-based attacks by the policy. In other words, the action for protected senders, protected domains, or mailbox intelligence protection aren't applied to these trusted senders or sender domains. The maximum limit for these lists is 1024 entries."
Maybe we can provide some clarity here and let readers know that trusted domains do not support subdomains?  Assumptions are being made by customers and it may help make it clear.

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

ID: f0b8f651-9d8e-a34b-8dde-2b6a9448f875
Version Independent ID: 20e96426-627d-9973-8303-a43a114c7b61
Content: Anti-phishing policies - Office 365
Content Source: microsoft-365/security/office-365-security/set-up-anti-phishing-policies.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

Checking

@dkotars, I added the following item in the existing Note alert in the area:

Trusted domain entries don't include subdomains of the specified domain. You need to add an entry for each subdomain.

OK?

@dkotars Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Refactor wallet interface

Move wallet interface into its own package and rename all the imports accordingly.

Coverage remained the same at 27.465% when pulling 74bd1aa0bd151faafe6fcb8a8460ee095526e5df on interface into 40a555e7e9535b751ab795a3950ba3641d4e14e2 on master.


vcpkg ui?

I'm curious if you guys know whether there's any effort in Microsoft or the open source community to make a UI for vcpkg. It seems like it would make a great VS extension, but if no one is working on anything I'd be interested in putting a stand-alone Qt interface together, something like cmake-gui.

@nholthaus  Just wondering if that would be similar to #1734

thanks yeah that looks great, any plans for it?
On Sep 19, 2017 10:10 AM, "atkawa7"<EMAIL_ADDRESS>wrote:

@nholthaus https://github.com/nholthaus Just wondering if that would be
similar to #1734 https://github.com/Microsoft/vcpkg/pull/1734
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vcpkg/issues/1845#issuecomment-330550692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ2HH-OLVMzgDLRqRs9dPMYMcnEsUMqZks5sj8qXgaJpZM4PcXnF
.

Planning to do so.

Working on this recently. You could check https://github.com/cmpute/vcpkg-gui to try and feedback please~

Duplicated to https://github.com/microsoft/vcpkg/pull/1734.


toLeopard: Desirable & satisfying traits

Resolves #123. Related to https://github.com/leopard-js/leopard-mentors/discussions/4 and #149.
This PR implements the alternate proposal outlined in the second half of https://github.com/leopard-js/sb-edit/issues/123#issuecomment-1868418197. Both PRs should be equally effective in generally fixing issues to do with surprise NaN in converted projects; however, we expect this one will lend to nicer and more intuitive reporter and input definitions. Either way, behavior-testing Scratch projects are much needed.
This code is all brand new, from July 2024. Thanks to @adroitwhiz for TypeScript tips!

@PullJosh We've requested review on both this pull request and #149, but our hope is that if you don't see any stand-out issues, we'll merge this one and not the other. They're opposing pull requests:
#149 is more conservative and uses very similar code language, so has a smaller diff, but it introduces conceptual baggage that we've seen is hard to explain and tricky to reason about. This PR (#150) includes all of the exact same meaning (or at least is meant to), and so if you want to get a sense ofr how the two PRs express that meaning differently, you can compare the opcodes whose definitions were in #149 with the same ops in this pull request.
This PR changes the language we use to represent input shapes, splitting it into "desired" and "satisfied" definitions, which look similar but have distinct meaning. That meaning is all described in the DesirableTraits and SatisfyingTraits enums. Although we had to edit literally every block definition, they are virtually all just different representations of the same meaning - we just think that meaning is expressed more clearly, now.
In this PR, the code at the bottom of blockToJS for helping the satisfied traits meet the desired traits is a bit more complicated. But we think it's quite easy to reason about if you give it a read through. We chose to represent the logic mainly in terms of what traits are desired: after all, we're about to fill in casting code if needed, because the job of this code is to make the satisfied code (variable based on the block) meet the desired code (constant for any given input).

Also, we haven't created any proper behavior testing projects, yet—specifically for behavior to do with NaN (which is changed, in the same ways, in both pull requests). Those are coming soon™, and should be prepared before merging either PR!
We did test the project in leopard-js/leopard-mentors#4, which appears to be fixed the same way in this pull request as in the other. That was a good sign, LOL. Similarly, only a single change in the "compilesb3" snapshot test, not that it was automatically testing nearly any NaN-related cases, to begin with. (Those are the only cases intended to change with the merging of either PR.)
We also filed a couple follow-up issues #152 and #153 addressing particular concerns we noticed while updating all the opcodes' definitions for this PR.
I regret doing all the block updating work in one commit. If I'd thought better, I'd have converted all the definitions word for word identically to their current Leopard behavior, and only afterwards implemented the special NaN handling. Sorry this makes for a less nice-to-read diff! However, you should still get the idea from the diff pretty quickly: anything that doesn't involve NaN has a very simple conversion between the old style and the new one.
-satisfiesInputShape = InputShape.String;
+satisfiesTraits = [SatisfyingTraits.IsString];

-const string1 = inputToJS(block.inputs.STRING1, InputShape.String);
+const string1 = inputToJS(block.inputs.STRING1, [DesirableTraits.IsString]);

Thanks @PullJosh! Glad you enjoy this direction, it feels good to us too. There will probably be some neat ways to build on it as a basic framework for more dynamic behavior, but if you prefer it for the NaN-handling implemented in both these PRs, then that's perfect for us right now.
We're recovering from a pretty uncomfortable bout of illness ATM - if you'd like to, we would appreciate if you make a test project yourself. It needs to test these things:

The correct blocks indicate they may be NaN (and are casted as needed).
The correct blocks cast NaN to zero (or cast NaN strictly).
Leopard's runtime behavior surrounding NaN matches Scratch (in a project generated by sb-edit).

We are pretty confident the code we have implemented works like it's supposed to, so our concerns in testing are mainly about making sure we match Scratch behavior. (And use a snapshot test to catch potential regressions.) This mostly means reviewing Scratch's own implementations for all blocks that have number inputs, which means reading through a bunch of scratch-vm code (this stuff), and writing a Scratch project to validate the expected behavior.
We're happy to add the project into this PR when it is ready. And it's OK if that's too big of a project to take on yourself right now, too - we can come back to this when we're a bit more recovered!

Also, we still need to flie the corresponding leopard PR that implements toNumber(value, true) ("strict" toNumber). Should have that ready in a minute so you can actually test Leopard runtime behavior!


Improve padding of heading

There are three different margins at the top at the moment - One for the Joplin logo, another for the Home button and another for the content margin. And that padding is different from the main page (the Home button jumps when changing from a plugin page to the home page).
I think we should do this:

The Home link should be right below the "Joplin" logo
And it shouldn't move when switching from a plugin page to the home page
The "Install plugin" button should be right-aligned with the Search box
The plugin preview image (currently on the left of the plugin title) should be removed. It should only be used on the home page
And see below for the other changes:

Current

Changes

This has been partially addressed — alignment should be improved, but the position of the icon still needs to be updated.

Updated UI:


fix:  proxy creation through db

proxy creation through db
collect proxies for newly created proxied account

Task linked: CU-8693m099g Proxy. Delegated authorities. Proxies are shown only in session where it was added


Add rights checks when removing a file #158

This pull request is linked to issue #158 , with this patch applied a not writable file can't be removed.

Coverage decreased (-0.1%) to 94.242% when pulling 803fbc66a88d8cc2e914227cde9c889e81e40eef on lovenunu:fix-file-rights into a73cb946d51d78cd230450fa677fb37e36f6195d on mikey179:master.


Dice score function

I am using the following score function :
def dice_coef(y_true, y_pred, smooth=1):
    y_true_f = K.flatten(y_true)
    y_pred_f = K.flatten(y_pred)
    intersection = K.sum(y_true_f * y_pred_f)
    return (2. * intersection + smooth) / (K.sum(y_true_f) + K.sum(y_pred_f) + smooth)

def dice_coef_loss(y_true, y_pred):
    return -dice_coef(y_true, y_pred)

# ...
model.compile(optimizer=optimizer, loss=dice_coef_loss, metrics=[dice_coef])
# ...

It works pretty well for me training a fully DCNN to segment images.
Would you be interested in a PR in order to implement this in Keras ?
Note that the original implementation comes from the Kaggle post https://www.kaggle.com/c/ultrasound-nerve-segmentation/forums/t/21358/0-57-deep-learning-keras-tutorial

I suggest averaging across batch axis, 0-dimension:
def dice_coef(y_true, y_pred, smooth=1):     intersection = K.sum(y_true * y_pred, axis=[1,2,3])     union = K.sum(y_true, axis=[1,2,3]) + K.sum(y_pred, axis=[1,2,3])     return K.mean( (2. * intersection + smooth) / (union + smooth), axis=0)

Don't you think it should be?
def dice_coef_loss(y_true, y_pred):
    return 1-dice_coef(y_true, y_pred)

With your code a correct prediction get -1 and a wrong one gets -0.25, I think this is the opposite of what a loss function should be.
# not matched
dice_coef_loss(
    K.theano.shared(np.array([[0,0,0]])),
    K.theano.shared(np.array([[1,1,1]]))
).eval() # -0.25
# match
dice_coef_loss(
    K.theano.shared(np.array([[0,0,0]])),
    K.theano.shared(np.array([[0,0,0]]))
).eval() # -1.0

Here's suggestion which uses vector operations and averages across the batch axis.
def dice_coef(y_true, y_pred, smooth=1):
    y_true_f = K.flatten(y_true)
    y_pred_f = K.flatten(y_pred)
    intersection = K.dot(y_true, K.transpose(y_pred))
    union = K.dot(y_true,K.transpose(y_true))+K.dot(y_pred,K.transpose(y_pred))
    return (2. * intersection + smooth) / (union + smooth)

def dice_coef_loss(y_true, y_pred):
    return K.mean(1-dice_coef(y_true, y_pred),axis=-1)

# test
dice_coef_loss(
    K.theano.shared(np.array([[0,0,0],[0,0,0]])),
    K.theano.shared(np.array([[1,1,1],[1,1,1]]))
).eval()
# array([ 0.99999997,  0.99999997])

dice_coef_loss(
    K.theano.shared(np.array([[0,0,0],[0,0,0]])),
    K.theano.shared(np.array([[0,0,0],[0,0,0]]))
).eval() # array([ 0.,  0.])

Hi @wassname, could you clarify your statement?

With your code a correct prediction get -1 and a wrong one gets -0.25, I think this is the opposite of what a loss function should be.

I'm quite new to ML but isn't a loss function supposed to output a lower value for a correct prediction and a higher value for a wrong one? isn't that exactly what @hadim version of the function is doing?

@cicobalico yeah sure
I was wondering about that too, so I found a stackoverflow answer that covers it. It says loss functions are minimised (towards 0, not towards negative infinity).
When I used OP's loss function my CNN converged on the exact opposite answer and made an inverse mask instead of a mask.
(its not just hadim, it's written that way in a couple of repos)

1-dice_coef
OR
-dice_coef
make no difference for convergence, I used both.
1-dice_coef just more familiar for monitoring as its value belong to [0, 1], not [-1, 0]

but for -dice_coef it converges on y_pred!=y_true, doesn't it. I gave specific examples above.
I think the ranges [0,1] and [0,-1] would be interchangeable but not [0,1] and [-1,0] as in this case.

It shouldn't. Back propagation must minimize loss as low as it can, -1 in case of (-dice_coef) loss

Ah that makes sense then, thanks for clarifying that!

Hello everybody,
I need to use the dice coefficient for some computation on biomedical image data. My question is, shouldn't there be a K.abs() expression?
Aren't intersection and union only a valid measure for absolute values?
Thanks for answering in advance!

hi,
I use dice loss in u-net, but the predicted images are all white.
Could someone explain that?

hi,
I use dice loss in u-net, but the predicted images are all white.
Could someone explain that?

I suppose white means it is considering all the images as foreground. Can you post more about how did you made training set and is it binary level segmentation or multi label segmentation.

hi,
I use dice loss in u-net, but the predicted images are all white.
Could someone explain that?

I suppose white means it is considering all the images as foreground. Can you post more about how did you made training set and is it binary level segmentation or multi label segmentation.

I use the code from the first floor:
`def dice_coef(y_true, y_pred, smooth=1):
y_true_f = K.flatten(y_true)
y_pred_f = K.flatten(y_pred)
intersection = K.sum(y_true_f * y_pred_f)
return (2. * intersection + smooth) / (K.sum(y_true_f) + K.sum(y_pred_f) + smooth)
def dice_coef_loss(y_true, y_pred):
return -dice_coef(y_true, y_pred)
...
model.compile(optimizer=optimizer, loss=dice_coef_loss, metrics=[dice_coef])
...
`
yes, it is binary level segmentation. I use U-Net network based on Keras.
Because the output of the last layer is the probablity value of sigmoid function, if probability>05, the color = 1; is probability<0.5, the color = 0. I watched the output, that most of the probability are 0.49xxxxx, or 0.50xxxxxx. So that will cause the prediceted image become all white or all black.
I think the ideal two(binary) output should be 0.9xxxx and 0.1xxxx, that is to say, they should be very close to 1.0 and 0.0.
Do you have some points to let the output probability close to ground truth value? Thank you.

I suggest averaging across batch axis, 0-dimension:
def dice_coef(y_true, y_pred, smooth=1):
    intersection = K.sum(y_true * y_pred, axis=[1,2,3])
    union = K.sum(y_true, axis=[1,2,3]) + K.sum(y_pred, axis=[1,2,3])
    return K.mean( (2. * intersection + smooth) / (union + smooth), axis=0)

@alexander-rakhlin i've seen that some implementations of the dice-coefficient use smooth=1, where does this value comes from? From what I understand, this value is used to avoid division by zero, so why not use a very small value close to zero (e.g. smooth=1e-9)? In addition, by suggesting  axis=[1,2,3], I guess you're assuming a 4D TF backend of size (Batch, Height, Width, Channels), right?

@tinalegre this was 3 years ago and I can't remember where this smooth=1 comes from. 1e-7 is a better idea. Yes, we are speaking of 4D tensor. That time I was using Theano's (Batch, Channels, Height, Width), but this makes no difference

@alexander-rakhlin thank you! It doesn't make any difference you mean, because for both channel_first => (batch, channels, height, width) or channel_last=>(batch, height, width, channels) representations, the Batch dimension is at axis=0 and thus return K.mean(iou, axis=0) would work for both, right?

What is the probleme pleaaaaaseeee !!!

what is the probleme pleaaasssseeeee!!!

lease,what is the correct implementation of the dice coefficient
def dice_coef1(y_true, y_pred, smooth=1):
  intersection = K.sum(y_true * y_pred, axis=[1,2,3])
  union = K.sum(y_true, axis=[1,2,3]) + K.sum(y_pred, axis=[1,2,3])
  dice = K.mean((2. * intersection + smooth)/(union + smooth), axis=0)
  return dice

Gives me the following result = 0.85
or
def dice_coef2(target, prediction, smooth=1):
    numerator = 2.0 * K.sum(target * prediction) + smooth
    denominator = K.sum(target) + K.sum(prediction) + smooth
    coef = numerator / denominator

    return coef

Gives me the following result : 0.94


Request feature: Add bucle line support

This library is interesting to create flow diagrams:
It is possible to add that a node can link to another node in order to create a loop?
And customize every node with change background shape?
Connect lines with arrow?

Unfortunately the current algorithm that we use only works with directed rooted trees, which is an acyclic connected graph. In other words, it doesn't work with loops. Therefore we need a different algorithm for this use case. In the future we want to support all different kinds of graphs, so it's definitely coming someday. (Maybe we should then rename this library to GraphView?)
For your other questions: Different background shapes and lines with arrows should be easy to implement, but as I said, without a suitable algorithm you're not gonna get this kind of diagram.


LTSC 2019 referenced twice

The table references LTSC 2019 twice. Is this a typo? Should the second reference be LTSC 2021?

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

ID: 684ab805-b732-043b-9d4c-e8c02660557a
Version Independent ID: 5c288d08-f2cb-3927-93cb-acb2412fc62c
Content: Windows 10 Enterprise LTSC - What's new in Windows
Content Source: windows/whats-new/ltsc/index.md
Product: w10
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Created PR for this issue.


How to use it with CLI?

Hi,
I'm using basic CLI and want to be able to use it but I don't find any word of it in the README.

@visualcookie I just ran into the same problem. It turns out that you can simply do the following:
node-sass source/index.scss public/bundle.css --importer=node_modules/sass-module-importer

It should work (worked for me).

@rafaelrinaldi I can't get it to work. I tried passing the importer location using the following cases:

with the = sign between argument and value
with a space between them
with node_modules/sass-module-importer as the location
with node_modules/sass-module-importer/lib as the location
with node_modules/sass-module-importer/lib/index.js as the location

In all cases, I had a file bootstrap.scss containing this to process:
@import "bootstrap-sass"

That's more an issue related to node-sass and its own documentation that is not clear.


About always on top: the frame can not fix the BaiduNetdisk.exe.

Microsoft PowerToys version
0.68.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Always on Top
Steps to reproduce
open the BaiduNetdisk.exe and use always on top
✔ Expected Behavior
the frame fix the edge of the app
❌ Actual Behavior
the frame is far beyond the edge of the app
Other Software
BaiduNetdisk.exe 7.25.0

@SeraphimaZykova FYI

Are you still experiencing this issue? /needinfo


Update docs with new throttle limits

Recently we change the values for some throttle limits, we need to reflect the changes in the docs reference.md#system-limits
@rabbah would you know which ones need to be updated?

pr #1364?

@rr thanks, missed that one

#1507 and #1508 should be combined and address this issue.

replaced by #1532


Streamline integration of SQS Batch Processing  with Sentry

Is your feature request related to a problem? Please describe.
The current behavior reraises exception as SQSBatchProcessingError, which hides the original exception and prevents it from showing up correctly in Sentry:
SQSBatchProcessingError
Not all records processed succesfully. 1 individual errors logged separately below.

Traceback (most recent call last):
  File "/var/task/urllib3/connectionpool.py", line 445, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "/var/task/urllib3/connectionpool.py", line 440, in _make_request
    httplib_response = conn.getresponse()
  File "/var/task/sentry_sdk/integrations/stdlib.py", line 102, in getresponse
    rv = real_getresponse(self, *args, **kwargs)
  Fi...

Describe the solution you'd like
The best solution would be to be compatible by default with Sentry if it's there (I presume this is one of the most used exception reporting solutions for Python).
Another solution would be to provide a hook into the exception handler, to give us the opportunity to call Sentry.
Describe alternatives you've considered
I looked into subclassing PartialSQSProcessor and overriding _process_record, which is what I might do if you don't think that's worth looking into.

I was actually pretty simple, although it would still be nice to have a no-code integration:
from aws_lambda_powertools.utilities.batch import PartialSQSProcessor
from sentry_sdk import capture_exception

class SQSProcessor(PartialSQSProcessor):
    def failure_handler(self, record: Event, exception: Tuple) -> Tuple:  # type: ignore
        capture_exception()
        logger.exception("got exception while processing SQS message")
        return super().failure_handler(record, exception)  # type: ignore
``

Btw: I took the existing types, `Tuple` misses its type parameters.

Hi @charlax, thanks for raising an issue! This exception is a bit tricky for 2 reasons:

We have to execute _process_record multiple times, and multiple exceptions can be raised during a single execution. There's no good way (I'm aware of) to raise from multiple exceptions, so we wrap everything into the SQSBatchProcessingError. The "root" exceptions should be logged though since we fixed #275 in v1.10.2.
By default, logging statements from within the powertools package are suppressed - as per best practices for libraries. This means we can't log the exceptions individually as they happen without users needing to manually set the package logger.

I think the solution you've found is a good one, arguably cleaner than passing hooks/callbacks around. We could always add something to the docs for folks wanting to implement similar logic, do you think that would help?

Yes, this was actually very simple with the failure_handler, adding this example to the doc would be enough I think! Thanks for your answer!

Hey @charlax Added an example - Could you tell us whether this suffice?

We're also refreshing the navigation it should be easier to find it - Full PR: https://github.com/awslabs/aws-lambda-powertools-python/pull/308

Yes, this is great! Thanks a lot!

This is now available in 1.11.0 - Literally just launched ;) 🎉


Failure to properly import license settings prevents user from viewing the list of items in an imported collection.

I got this honeybadger error earlier when trying to view details in a couple of imported collections:
Error detected in: happy-heron (stage)

Full report at https://app.honeybadger.io/projects/77112/faults/76940953

TypeError: Parameter 'work_version': Expected type WorkVersion, got type NilClass Caller: /opt/app/h2/happy-heron/shared/bundle/ruby/2.7.0/gems/sorbet-runtime-0.5.6294/lib/types/private/methods/call_validation.rb:78 Definition: /opt/app/h2/happy-heron/releases/20210303182230/app/components/works/detail_component.rb:8

Summary

works#index

URL: https://sul-h2-stage.stanford.edu/collections/557/works

Hostname: sul-h2-stage.stanford.edu

Context

user_scope: user
user_id: 12
Trace

app/components/works/link_to_show_component.rb:16:in `new`
app/components/works/link_to_show_component.rb:16:in `title`
app/components/works/link_to_show_component.rb:12:in `link`

I later determined that these two collections did not have a license selected and that when I chose one, saved it, and redeposited it (though I don't understand why redeposit was required it), I was then able to view the list of items in the collection.

Related to #1280

Duplicate of #1374


chore: additional fixes + feats

Description
Additional fixes
Checklist for Feats
Please review and complete these items before submitting your pull request:

[ ] I have tested my changes thoroughly and they work as expected.
[ ] I have included appropriate tests for any new features or bug fixes.
[ ] My code is well-formatted and complies with the ShellCheck standards.
[ ] I have updated the documentation (if applicable).
[ ] I have read and agree to the code of conduct and contribution guidelines.

I'll probably include the multilingual in the next pr


[SPARK-12074] Avoid memory copy involving ByteBuffer.wrap(ByteArrayOutputStream.toByteArray)

SPARK-12060 fixed JavaSerializerInstance.serialize
This PR applies the same technique on two other classes.
@zsxwing

Test build #47284 has started for PR 10177 at commit 863cf8c.

Test build #47284 has finished for PR 10177 at commit 863cf8c.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47284/
Test FAILed.

Merged build finished. Test FAILed.

Had test failures in pyspark.streaming.tests with python2.6; see logs.

I don't think the above failure was caused this PR

Jenkins, retest this please.

Test build #47289 has started for PR 10177 at commit 863cf8c.

LGTM pending tests

Test build #47289 has finished for PR 10177 at commit 863cf8c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47289/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Merging to master

Thanks @tedyu


v1.0.0 (b10)

Additional Details

Version: v1.0.0 (b10)
Last updated: 15 March 2023 08:30 (UTC)

~ Add

Legals route page

~ Update

Feeds headers
Footer component

~ Fix

SEO, sitemap, icons

✓ This pull request has been approved ✓
Additional Details
Pull number: #12


I can't access

Nazim Afternoon, I can't access your remarkable application, I get an error ( Failed to fetch ) , please help me to connect to this Chrome extension.
my email is<EMAIL_ADDRESS>Thank you!

Hey, sorry, the server isn't running anymore :/


HHH-9977 Consider options for passing Session to caching SPI calls

Passing SessionImplementor to all the calls executed in transactional context.

I am not sure about Region#contains(Object key) - the places where it is used don't have any reference to Session/SessionImplementor, but conceptually it shouldn't be different from GeneralRegion#get(SessionImplementor session, Object key).

@rvansa I will look at this today/tomorrow.  @Sanne Could you also look this over?  Especially I would like your thoughts on the SPI changes adding Session to many calls.

Thanks @rvansa
PR rebased and merged


Update astro-components.md to warn usage of named slots

Refs https://github.com/withastro/astro/issues/2238
Is this reasonable to add here?

@delucis @aFuzzyBear can I get some thoughts on this?
I know in general we try to keep the Astro pages about Astro stuff. And, at this point in the docs, in this "Astro components" page, we haven't even discussed any non-astro stuff on your site yet at all. So, I'm not entirely sure about the call-out here. BUT, I could see adjusting the current text of the page around it:
Named Slots
Section titled Named Slots
Slots can also be named. Rather than a single  element which renders all children, named slots allow you to specify multiple places where (only certain child elements from your .astro component) should be placed.
QUESTION: can named slots be used with Markdown children, too, or JUST .astro components?

@RafidMuhymin Do you have thoughts on this?

We need to investigate these things. Anyone from the core member will be able to help better.

In this context I might just fix the typos. I think I’ve seen scenarios where passing a framework component to a slot actually does work, so it may not be as clear cut — or at the very least we need core to confirm as @RafidMuhymin suggests.

@delucis I don't think @leader22 is saying that if we pass framework components inside named slots then they were working. I think the problem is that Svelte, Vue have their own slots like Astro, and when trying to pass content/components via the slots in framework components to the parent Layout which is an Astro component, then those components aren't getting passed properly.

@leader22 If my point on the comment above is correct, then how did you experiment that it's not working? Because if you have passed the Vue/Svelte component inside the main slot, then it's normal that the slots inside those framework components won't be passed to the Layout component, as it's not possible to use a slot inside another slot.

Thank you for your review all. 😉
My original case was like this code below,
---
// pages/index.astro
import Layout from "../components/layout.svelte";
---

<Layout>

Main

Sub

</Layout>

and

<slot name="main" />

<slot name="sub" />

ends up with rendering nothing(= empty default slot).
To make this case works, Layout component should be .astro(not .svelte|.vue).
So at least for me, @sarah11918 's suggestion looks good.

@leader22 oh, that makes sense. Thank you for the explanation. Yeah, it can be documented. I think when the compiler is identifying the slot it's trying to pass the slot to a parent Astro component. But as it's not being able to find an Astro component it's not doing anything.

Sorry, I should include the context first. 😅
The remaining problem is how should the documentation be worded. Should we still avoid referring to Framework components?

@leader22 Would the wording I suggested above be OK? I think I would prefer not to mention other framework components if we can, but instead say very clearly ASTRO COMPONENTS when describing. Do you think that would be clear enough?

@sarah11918

I think I would prefer not to mention other framework components if we can, but instead say very clearly ASTRO COMPONENTS when describing.

Sure.
But what we should care about here is parent slot (must be an .astro component, not .svelte not .vue), instead of child(there is no restriction, any types of component can be passed) content for the slot.
Do you have any ideas to express this nuance?

Do you mean something like...
Slots can also be named. Rather than a single element which renders all children, named slots in .astro components allow you to specify multiple places where ...

My two cents... https://github.com/withastro/docs/pull/364/commits/3c0c0a4c4dd1bcfd4416741a00bd7ee664e51d00
Finally I realized that simple is best. 😹

@leader22 If that wording satisfies you, it satisfies me!  My job is to keep things simple! 😅
I'll merge this now, and if you have second thoughts later, we can absolutely revisit it with another PR!

Sorry to revive this, but the current wording tripped me up and I ended up down a whole rabbit hole until I found this.
The current wording says inside an Astro component and I was using named slots inside an Astro component - it isn't about where the named slots are used but abut the component you're using it with. Explicit is better than simple in this scenario.

I think the communication issue might be in how to distinguish betweent "Astro Component" ie the .astro file in which you're rendering a slot vs "Astro Component" ie the the component being rendered in the HTML template.
When the original author mentioned being satisfied with the wording, I took that to mean: "Seeing "in an Astro coponent" is enough of a cue for me to look at my example and think, "Hey, my Layout component is a Svelte component, so maybe I shouldn't expect this to work like an Astro component." which is maybe the more desirable outcome, vs annotating unusual cases in the docs? (Full disclosure: I'm not sure how common of a pattern to expect this to be.)
Anyway, that explains the motivation here, and I'll move on to the open PR.  :)

@Jutanium I feel sorry to cause a misunderstanding. 😓
@sarah11918 Yes, I think so too. Since this page is about Astro component as template and how to define the slot itself. How to use (and combine with other stuff) may be an another concern. (And it was a little bit difficult for me to express without referring framework components).
I'll comment on some thoughts in #371 later.


Retry sendRequestToKcp in case failure

When the sendRequestToKcp function fails to receive a response from KCP, it's essential to implement a retry mechanism to enhance the robustness of the watcher component. This will ensure that webhook requests are retried several attempt before considering the operation as failed.
AC:

[x] Implement a retry mechanism for webhook requests to KCP in case of failure. Retry up to a maximum of three times to improve reliability.
[ ] Modify the sendRequestToKcp function to return (string, error) instead of just string to have better error handling and reporting.

Second AC is invalid. The function already returns an error object


Gyroflow toolbox won't appear in effects browser after installation

I have installed the motion template, but the gyroflow toolbox does not appear.
System:  15.0
FCPX: 10.8.1

What was the solution? - I'm experiencing the same here: https://github.com/latenitefilms/GyroflowToolbox/issues/35 - on MacOS 15.1

I found that the wrong folder permission was given. When I clicked the install motion templates button, the default folder did not seem to be the “movies” folder.


Not Working with IIS

Hi
Ibuilt the libModSecurity.dll and add it to my IIS modules in windows 2019. After resetting the IIS I got Service Unavailable when I checked the eventviewer I had this error
Failed to find the RegisterModule entrypoint in the module DLL C:\inetpub\ModSecurity\libModSecurity.dll.  The data is the error.

Hi @hadiloghman,
thanks for reporting - but I'm a bit confused.
You wrote you built libmodsecurity.dll (which came from libmodsecurity3 source code), but added label 2.x.
Also: libmodsecurity3 is a library, it needs a connector for a specified HTTP server that you want to use it with. Do you have a connector for IIS?

Sorry for the wrong version that I selected.
No, I didn't use any connector. I registered the DLL in the modules of IIS. Is there a need to add a connector? I didn't see it in the docmentation

No worries. Here is the relevant part in the documentation:
"This library is consumed by what we have termed 'Connectors' these connectors will interface with your webserver and provide the library with a common format that it understands. Each of these connectors is maintained as a separate GitHub project. For instance, the Nginx connector is supplied by the ModSecurity-nginx project (https://github.com/owasp-modsecurity/ModSecurity-nginx)."

Dear @airween
Thanks for the link and description
I couldn't find any connector for IIS, can I use any other connector for IIS?

No, actually there is only one (known) stable connector for this library, only for Nginx (see the link in quoted text).
There is a very beta version for Apache, but no other (published) connector.

So I should use ModSecurity 2 for IIS. Am I right?

Well, I don't know - I'm not sure about that. Few months ago I tried to run some part of code (of mod_security2) which is related to IIS, but as I remember it was broken. I don't know in which state is the IIS support.
@marcstern do you know anything about that?

Dear @airween
Thanks for your comments.
I have another issue with version 2. It does not log and when I check the eventviewer I saw this error
"The description for Event ID 1 from source ModSecurity cannot be found. Either the component that raises this event is not installed on your local computer or the installation is corrupted. You can install or repair the component on the local computer."
do you know what causes this error? I use version 2.9.7

Unfortunately I have never used mod_security2 on Windows (neither Apache), so I'm afraid I can't help you in this.
Perhaps @marcstern.

Thanks @airween for your comments. I can finally add modsecurity 2.9.7 to IIS and enable the Logs. Thanks, guys for your great job.

@hadiloghman - glad to see your issue has solved. I'm going to close this issue.

Hi @airween  I noticed that in the latest ModSecurity v 3.0.13 release Windows port has been added and I was even able to produce binaries following the instructions,  however I see that no IIS connector still exists for it.  Are there plans to implement it and if so any estimated on that? Thanks in advance

Hi @airween I noticed that in the latest ModSecurity v 3.0.13 release Windows port has been added and I was even able to produce binaries following the instructions, however I see that no IIS connector still exists for it. Are there plans to implement it and if so any estimations on that? Thanks in advance

@podorozhkin sorry for the late answer - currently there is no plan to add IIS port because of the lack of available resources. If you had capacity/knowledge to add that, it would be awesome. Btw fixing the Apache connector is more important.

Ah found it! Might be good to mention on the modsecurity homepage which platforms are supported on v2 and v3. https://github.com/owasp-modsecurity/ModSecurity/releases/tag/v2.9.7

Hi @mike-nelson, unfortunately we don't have any good description about how to make an IIS module (from mod_security2). If you start to make your own, could you make a short brief about how to done that?

Ah found it! Might be good to mention on the modsecurity homepage which platforms are supported on v2 and v3. https://github.com/owasp-modsecurity/ModSecurity/releases/tag/v2.9.7

Hi @mike-nelson, unfortunately we don't have any good description about how to make an IIS module (from mod_security2). If you start to make your own, could you make a short brief about how to done that?

Yeah, I am sure that would be a worthwhile thing to do but may be outside my capability, probably not something I can do, but I will certainly update you if I go down that track.


Notification count is undefined

May be it's because githubAPI heartbeat is too often. It just reaches GitHub API hourly limit.

I am getting this, too.

Some trouble for GitHub today: https://status.github.com/messages.

@vpArth did this just end up being an issue with the API being down? or is it still a problem?

@jacobbearden, 'm sorry, I removed this extension while it didn't work for me.
I think, it just was not care about possible other extensions using GithubAPI. (I use Octotree for example).

I just saw this on Windows 10, created a new GitHub token and that fixed it instantly.

We should have an error message telling people to add a token.

The extension shows an X when there is no token!


Step 1 not working

Can't seem to get Option 2 deploy from local machine running
I'm running the script/deploy.js -r  -a  -s  -b  but I'm getting a windows script host error with error: invalid character at line:1 Char:1 with error code 800A03F6

Hi, same happened to me. I believe it's because windows tries to run the script with a native Javascript interpreter instead of NodeJs. If you installed node correctly, try running "node script/deploy.js -r -a -s -b".
You will probably need to also change the line inside deploy.js "spawnOrFail('script/cloud9-resize.sh', []);" to "spawnOrFail('bash script/cloud9-resize.sh', []);". Or you can just comment it out since I believe it's only needed for Cloud9.
After this I had some errors creating buckets, so I ended trying the Option 1 with Cloud9 which worked right off the bat.
If you get further, you may need to dome some other modifications as I believe the code is prepared to run on ubuntu, not windows.

Just to confirm, can you please try with a new bucket name and stack name?

Just to confirm, can you please try with a new bucket name and stack name?

Please reactivate or cut a new issue if there are additional questions.


gpu-exporter-chart: add hostNetwork to the DS and make svc opt-out

add the option to make DaemonSet with hostNetwork, also make the service with the option to opt out

It wouldn't trigger a release anyway, I need to do it manually, so no problem, I can also take care of it while doing that.

@utkuozdemir sorry for the ping, do you plan to bump the version chart soon ?

@utkuozdemir sorry for the ping, do you plan to bump the version chart soon ?

I'll try to do it this weekend.
I also plan to move this chart soon out to its main repo and maybe publish it as OCI artifact (with oci://).

@utkuozdemir sorry for the ping, do you plan to bump the version chart soon ?

I'll try to do it this weekend.
I also plan to move this chart soon out to its main repo and maybe publish it as OCI artifact (with oci://).

done, released 1.0.0 with the exporter version 1.2.1 as default.


Unable to dump clBLAS kernels

I'd like to configure clBLAS to dump OpenCL kernels as they are generated. I set BLAS_DUMP_CLBLAS_KERNELS and BLAS_KEEP_KERNEL_SOURCES in CMakeCache.txt, in addition to BUILD_SAMPLE, and I did a build.
Then I ran
> LD_LIBRARY_PATH=/path/to/libclBLAS.so.2 ./samples/example_sgemm

to make sure it's referencing the library that was just built. However, I don't see any output files.
Searching through the codebase, It seems that the function that is supposed to dump kernels is the kernelDump function, which gets called from the enqueueKernel function. However, I don't see where the enqueueKernel function gets called.
As an additional step, I tried running samples/example_sgemm with gdb, and I set a breakpoint on enqueueKernel and dumpKernel but they were never hit.
Is there anything I'm missing to enable dumping of the cl kernels?

the gemm kernel is not generated in the run time.
the source code is in src/library/blas/AutoGemm. the source code is
generated by python script.
Most of other kernels (e.g. gemv) are generated on the fly
On Thu, Feb 25, 2016 at 11:55 AM, Mateja Putic<EMAIL_ADDRESS>wrote:

I'd like to configure clBLAS to dump OpenCL kernels as they are generated.
I set BLAS_DUMP_CLBLAS_KERNELS and BLAS_KEEP_KERNEL_SOURCES in
CMakeCache.txt, in addition to BUILD_SAMPLE, and I did a build.
Then I ran

LD_LIBRARY_PATH=/path/to/libclBLAS.so.2 ./samples/example_sgemm

to make sure it's referencing the library that was just built. However, I
don't see any output files.
Searching through the codebase, It seems that the function that is
supposed to dump kernels is the kernelDump
https://github.com/clMathLibraries/clBLAS/blob/9731ea2a270509211a47bf6cf9df4de2069ccc52/src/library/blas/generic/kdump.c#L120
function, which gets called from the enqueueKernel
https://github.com/clMathLibraries/clBLAS/blob/9731ea2a270509211a47bf6cf9df4de2069ccc52/src/library/blas/generic/solution_seq.c#L114
function. However, I don't see where the enqueueKernel function gets
called.
As an additional step, I tried running samples/example_sgemm with gdb, and
I set a breakpoint on enqueueKernel and dumpKernel but they were never
hit.
Is there anything I'm missing to enable dumping of the cl kernels?
—
Reply to this email directly or view it on GitHub
https://github.com/clMathLibraries/clBLAS/issues/232.

--
Tingxing dong

Thank you for your reply. How can I use the Python script to generate the gemm kernel?

No. you do not have to do it your own.
read that folder, you will see what is going on.
On Thu, Feb 25, 2016 at 12:10 PM, Mateja Putic<EMAIL_ADDRESS>wrote:

Thank you for your reply. How can I use the Python script to generate the
gemm kernel?
—
Reply to this email directly or view it on GitHub
https://github.com/clMathLibraries/clBLAS/issues/232#issuecomment-188909926
.

--
Tingxing dong

I see that there are many kernels already generated in UserGemmKernelSources. How are these kernels selected by the runtime based on the dimensions of the workload?
For example, if I were to call clblasSgemm with N = M = K = 1024, which one of these kernels would be compiled? How does clBLAS make these selections at runtime?

AutoGemmParameter.py,
if you want to go deep, you need to read the AutoGemm folder thoroughly .
On Thu, Feb 25, 2016 at 12:20 PM, Mateja Putic<EMAIL_ADDRESS>wrote:

I see that there are many kernels already generated in
UserGemmKernelSources. How are these kernels selected by the runtime
based on the dimensions of the workload?
For example, if I were to call clblasSgemm with N = M = K = 1024, which
one of these kernels would be compiled? How does clBLAS make these
selections at runtime?
—
Reply to this email directly or view it on GitHub
https://github.com/clMathLibraries/clBLAS/issues/232#issuecomment-188917380
.

--
Tingxing dong

OK thank you


How to implant mtproto

I want to implement the mtproto by myself with nodejs but I don't know where should I start. I try to read your package but I didn't understand it and I try to read the protocol of telegram and I confused.
How can I create a connection with telegram server and transfer data with it?
Where should I start from to develop such a package like yours?

It was my question too @goodmind ,
I try to connect to telegram server but I got nothing. How do you create mtproto?

Why do you want your own package?

Because I want to create some packages like this in other languages and I want to contribute to your package too and modify and inmprove it.

Do you want to work together to learn how to use the package? I have been having a hard time figuring it out myself.

We are looking for a starting point, Where should we start connecting to telegram server?
@dundas

@Alavi1412 Would you want to join a slack chat and work together to get it working?

@dundas , Yes Of course

@dundas , would love to join a slack and work together to get it working.
I have difficulties with an invalid_code error when trying to sign in.

@aticus7886 is the slack channel still active?  If possible I would like to join, I try to implement MT Proto as well and running in similar difficulties.

@Alavi1412 @dundas @aticus7886 did you guys manage anything? I'm having a hard time with this as well


fix(registry): create /deis/cache etcdctl dir

Otherwise registry will loop forever waiting for the /deis/cache dir to exist, nice find @smothiki.

hmm, this might've explained the same issues I was seeing when booting v1.4.1 with a newer v1.5.0 deisctl client. LGTM!

Code LGTM
core@deis-04 ~ $ journalctl -fu deis-registry@1
-- Logs begin at Mon 2015-04-06 19:47:41 UTC. --
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 WARNING: Cache storage disabled!
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 WARNING: LRU cache disabled!
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 INFO: Configuring CORS for app:'docker-registry' with resources:[(<_sre.SRE_Pattern object at 0x7fefd398abe8>, {})]
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 WARNING: No S3 region specified, using boto default region, this may affect performance and stability.
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 INFO: Boto based storage initialized
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 WARNING: Cache storage disabled!
Apr 06 20:01:27 deis-04 sh[2889]: 06/Apr/2015:20:01:27 +0000 WARNING: LRU cache disabled!

In CI, we issue a deisctl config cache set image=registry.hub.docker.com/deisci/cache:jenkins-test-ec2-437 command still (for example), even though cache isn't installed or started. This has the side effect of creating the /deis/cache dir in etcd, which explains how this wasn't caught by automated testing.


An IP Blacklist for admin panel

A way to avoid that a stranger tried to access to admin panel (exclude users table)

If you fail more tha i++ eg:  times, you are out of try more things

So we can create the PocketBase IP Blacklist network to sync with other anonymous users and know potential banned users from others, to add to our list, something like pihole but tiny, for PB

This will be handled eventually as part of https://github.com/pocketbase/pocketbase/issues/1232.
For now you have several options:

define the IP whitelist/blacklist on cloud/OS firewall level
define the IP whitelist/blacklist on reverse proxy (eg. nginx)
use a middleware (Go or JS) and block manually the designated IP. Here is one such example with the JS hooks:// pb_hooks/main.pb.js

routerUse((next) => {
    return (c) => {
        // ip:port string (not sure if the below will work on ipv6 addresses)
        const addr = c.request().remoteAddr;
        const ip = addr.substring(0, addr.lastIndexOf(":"));

        if (ip == "YOUR_IP_TO_BLOCK") {
            throw new ForbiddenError("You are not allowed to access this resource")
        }

        return next(c);
    }
})

Colossal thanks, THIS F* project is so complete, I wanna cry, THANKS cause what you do is simply amazing, you have a prospect so ATH! <3 continue working


Fixes #154

scalacOptions is a task (I thought it was a setting) which forced generatedSourceDialect to also be a task.
Tested locally in a Scala 3 (future) project and it worked, aka generatedSourceDialect has correctly computed. Changed the algorithm slightly: under Scala 2 with -Xsource:3 scalac option the dialect Scala3 is chosen.
Haven't implemented any tests for this functionality. If its looking good for you I can try to implement some.
Only made the changes for ScalaJS, can easily make similar changes for JVM.

CI is failing because of code format, Could you check it?

Forgot about scalafmt. Fixed.

Thanks, it looks good, it would be good to get these on JVM and native.
Do you want to add some code in this PR or prefer to merge it and do another one?

It is for scala-native 0.4.3. But we can just make it use regular jvm code. The one for js is just optimized

The diference between the TzdbZoneRulesProvider.scala for jvm and native is only the use of TreeMap vs ZoneMap. So you want to use the sames files for jvm and native?

It depends, I'm not sure if TreeMap is available in native. If it is not there is no choice but to use ZoneMap. If there is wwe could use the same code


fix: gracefully handle SIGPIPE

When fixing #234, I noticed cleo's terminal prompt wasn't rendering until after reading a response:
$ poetry cache clear --all .
no # cleo is prompting here
Delete 575 entries? (yes/no) [no] # but output is displayed later

This was introduced via #165, since now we aren't always flushing the output. This reverts that and instead suppresses a broken pipe error, which was the initial motivation for that PR.
Note that click also always flushes output and similary ignores broken pipes.
I manually tested this with poetry -V | head -1

I think this could resolve #56, could you confirm?


Overall operator issues

as discussed, here is a list of overall operator issues that we currently identified:

do not explicitly cleanup resources and use ownerReferences to do so, and verify it works
ocp route creation algorithm could be improved, there are situations where there is a noticeable delay for the route to be created. Route creation algorithm should be simplified and leverage default route host value when possible.
operator creates too many watches, the issue is in cf_service_monitor .  But watches should be only created once at operator start up, this will need refactor in current control functions implementation
internal dependency management, there is a map to hold business logic services for each registry CR, this should be improved/refactored and try to get rid of that map
reconcile loop gets executed too many times, this should be improved fixing the watches leak and filtering events using predicates

This issue has been resolved, and will not affect the upcoming 1.0.0-v2.0.0.final release.


Log to a file by default

This is much quicker than writing to the Godot sink, which in turn writes to the console.
This allows investigating of performance hot spots without logging of the performance taking up more overhead than the actual performance hot spot.
See the extensive notes on Closes #278 .  When investigating why turn times were slow, the logging I added made things another order of magnitude slower.  These changes at least significantly decrease that overhead.

I'm not ready to close #278 (even though I just repeated the magic words) but this a step forward for now. Another reminder that #259 should eventually supersede this.

Merging in.  For some reason I can't clear the link to #278 on the sidebar, so I'll move it back manually.

I also agree that it's a step forward, not a solve everything forever.

Adding "Carthage" tag since the main issue has been split into a part 1 and part 2, linked to the same issue since it has the notes.


MongoDB 3.6 minimum

Hello
It requires MongoDB 3.6 minimum.
Raspberry pi 3 cpu architecture (32bit) does not support that version of MongoDB.
Is it possible to use mysql instaed of mongodb?
Does anyone know any workarounds?
I'm using apache2 and MySQL for my other websites.

Is it possible to use mysql instaed of mongodb?
No not at all.

Does anyone know any workarounds?
I don't think there is any in this case.  I would recommend hosting mongodb on a different server. Maybe you could use MonoDB Atlas.

Here is also a discussion on the topic:
https://discord.com/channels/460491088004907029/1038732210452316220  (maybe it was you who posted this on the Jikan discord server)


GJT-1 #start-progress #comment testing connection to jira and smart values

testing again but with smart values in the title

GJT-1 #start-review #comment testing connection to jira and smart values


Make EventPump exclusively borrow the SDL context

The changes are only slightly breaking. Most people will only need to change the SDL context at the beginning of their program to be mutable:
let mut sdl_context = sdl2::init().video().build().unwrap();

To acquire an EventPump, it now requires a mutual borrow of Sdl. This is to statically ensure a single event loop, and that it's never possible to have two or more event loops at a time.
It's no longer necessary to create a dummy EventPump handle in functions that only access window properties. properties() accepts &Sdl instead of &EventPump:
fn set_title(&mut self, value: &str) {
    self.window.properties(&self.sdl_context).set_title(value).unwrap();
}

is equivalent to the old
fn set_title(&mut self, value: &str) {
    let event_pump = self.sdl_context.event_pump();
    self.window.properties(&event_pump).set_title(value).unwrap();
}

Great stuff!


Changes following jake's review

Description
This merge includes new pages:

NHS App templates
Email templates
Text message templates
Letter templates
Message and channel status

My latest commits address the comments from Jake's latest review and a few lingering issues. These changes are:

removed anchor links for bullets and headings in /text-message-templates as these sections don't exist for this channel
removed 'Creating...' from the start of the hypertext for the 4 new pages linked to in /using-nhs-notify parent page
fixed a copy and paste typo - replaced 'to send your email' with 'to send your text message' in /text-message-templates, just below the heading 'How to edit your NHS App message using our template'
updated names of Word docs to replace spaces with hyphens and make all lower case, then updated downloadable links to follow the approach Jake researched, for example [letter template]({% link assets/worddocs/letter-template-nhs-notify.docx %})

Changes made to:

Features
Routing plans
Sender names (was 'Sender IDs')
Tell us who you want to message (was 'Providing NHS numbers')
Personalisation
MESH

Context

Type of changes

[x] Refactoring (non-breaking change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I am familiar with the contributing guidelines
[x] I have followed the code style of the project
[ ] I have added tests to cover my changes
[ ] I have updated the documentation accordingly
[ ] This PR is a result of pair or mob programming

Sensitive Information Declaration
To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

[x] I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

Need to close this PR and re-do it on the dev repo.


Maybe wrong s3 url prefix

Hi. I found the TranscribeService.transcribeMediaUrl did not run correctly, so I check S3UploadInfo.getResourceUrl and compare with all the endpoints  here.There is no endpoint begin with "s3-", but "s3.". Would you please help to double check.

Sorry, my mistake. Seems the S3 url works for Transcribe internally.


Feat/refactor code activity choice

Activity Choice View. Info modal will follow in a separate PR.

LGTM


Smart indent is set incorrectly when line ends with space

... or otherwise being formatted. Try
function() {
    field <- |
        x
}

end with
function() {
    field <-
|
        x
}

works correctly when there is no space after <-. Most probably regression since when we started formatting after ENTER rather than before. Formatting probably re-positions caret improperly.

https://github.com/Microsoft/RTVS/commit/698b187013402962ac7ff35ccdf9ae4d321a1869


Track known-bad os and puppet combinations

This is used to avoid running acceptance tests on systems where versions
of Puppet are not supported (yet or anymore).

So the idea is to emit this too and then in the Github action you add an exclude with them? I wonder how this compares not emitting them in the first place. I guess either of them could work.

I took a stab at this myself but that would really change the current approach. Instead of feeding it 2 vectors, you'd need to pass it an include block.
I also thought about the logic and I came up with:
module PuppetMetadata
  class AIO
    BUILDS = {
      # RPM-based
      'CentOS' => {
        '5' => 5..7,
        '6' => 5..7,
        '7' => 5..7,
        '8' => 5..7,
      },
      'Fedora' => {
        '26' => [5],
        '27' => 5..6,
        '28' => 5..6,
        '29' => 5..6,
        '30' => 5..7,
        '31' => 5..7,
        '32' => 6..7,
        '34' => 6..7,
      },
      # TODO: SLES/Suse
      # deb-based
      'Debian' => {
        '7' => [5],
        '8' => 5..7,
        '9' => 5..7,
        '10' => 5..7,
      },
      'Ubuntu' => {
        '14.04' => 5..6,
        '16.04' => 5..7,
        '18.04' => 5..7,
        '20.04' => 6..7,
      },
    }

    def self.has_aio_build?(os, release, puppet_version)
      # TODO: group CentOS with RedHat/AlmaLinux/Rocky?
      BUILDS.dig(os, release)&.include?(puppet_version)
    end
  end
end

Generating only an include with all supported "known-good" combinations is probably better yes…  If you have something, feel free to push it here (or in another PR and close this one).  If not, I will try to have a look at it in the next few days.

Closing in favor of #21


Dependancies error when bulding re-write

Hi,
not the first time but now I don't find the right version combination for dependencies to make it work...any idea?

npm -i
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

How have you install TaskBoard ? Please give us some details.
You don't need to do this. Have you follow https://taskboard.matthewross.me/docs/intro.html ?

I am trying to install the re-write branch, as for the master I have no issue following the procedure.
For example if I checkout to re-write branche I don't have the build folder anymore, this is the reason I tried the npm method.
Excuse me to not clarify it before that I was talking about the re-write (I did put it in the title but not on the explanation)

I have the same issue when build the re-write branch.
I saw on https://github.com/valor-software/ng2-dragula/issues/636 that is still an issue
on ng2-dragula repository.
Nothing to do for @kiswa, so issue can be close.

You can safely ignore that warning from npm.
After npm -i completes (even with warnings) just run gulp and it will build everything and place it in a dist directory.
Keep in mind, the re-write branch is very much a work in progress and does not have all the features of master.

Hi,
I have tried to update to your latest re-write version, following a npm i and gulp, no issue there, dist is generated but trying to access I got multiple 404 and only got "TaskBoard is Loading..." on my browser.
<IP_ADDRESS> - - [29/Jul/2017:00:18:17 +0200] "GET /dist/css/styles.css HTTP/1.1" 404 1147
<IP_ADDRESS> - - [29/Jul/2017:00:18:17 +0200] "GET /dist/js/vendor.js HTTP/1.1" 404 1147
<IP_ADDRESS> - - [29/Jul/2017:00:18:17 +0200] "GET /dist/js/bundle.js HTTP/1.1" 404 1147
<IP_ADDRESS> - - [29/Jul/2017:00:18:17 +0200] "GET /dist/js/bundle.js HTTP/1.1" 404 1147

my dist/js:
-rw-r--r-- 1 http http  27K Jul 29 00:14 Reflect.js.map
-rw-r--r-- 1 http http 555K Jul 29 00:14 vendor.js

my dist/css:
drwxr-xr-x 2 http http 4.0K Jul 28 23:53 fonts
-rw-r--r-- 1 http http  59K Jul 29 00:14 styles.css

any ideas?
many thanks

Edit the index.html and change the base href to match the url you're accessing it from.
For example, it's /dist/ by default (this will change in the future). Basically, what's in there should be anything after http://localhost/ when you go to the dist folder.
So, if your copy is at http://localhost/dev/TaskBoard/dist/ then you should edit the base href to be dev/TaskBoard/dist/.

I remember now doing this in the past thanks!
Now it is better, I can see the grey background so the css is loaded but I don't have more...

[29/Jul/2017:15:59:09 +0200] "GET / HTTP/1.1" 200 703
[29/Jul/2017:15:59:10 +0200] "GET /css/styles.css HTTP/1.1" 200 60008
[29/Jul/2017:15:59:10 +0200] "GET /js/bundle.js HTTP/1.1" 404 1117
[29/Jul/2017:15:59:10 +0200] "GET /js/vendor.js HTTP/1.1" 200 567507
[29/Jul/2017:15:59:10 +0200] "GET /images/bg.png HTTP/1.1" 200 2126
[29/Jul/2017:15:59:10 +0200] "GET /js/bundle.js HTTP/1.1" 404 1117

It is not loading the rest of the modules/page

Please tell us do you edit the base href in you index.html file ?

Looks like the bundle wasn't generated correctly. Try running gulp again, or just gulp app.

well done :) after a gulp app it is better!
I am progressing ... now I got the login but Sign In has no effect...

Congratulations for the progression by the way!!
I am not able to move task from column to column yet but I guess it is normal.
By the way the menu when I right click on a task disappears if I don't hold my click.
You re-write seems very promising :)

Thanks!
Yeah, I'm still working on that context menu behavior. You can currently only move tasks by using that menu (or editing them and selecting another column).
I'm pretty happy with what I have done so far, but there's still a good ways to go. Have you tried entering a task with a checklist (use  - [ ] for to-do items, - [x] for done)? When you have some complete, it shows a progress bar across the bottom of the task.

@kiswa : hey! I just updated it from 3 months ago and please to see your improvements! seems great! I found a lot of little details changed and theyr are very welcome !
By the way I tried the - [ ] and  - [x] and they are nicely displayed into the comment area but I don't see the progress bar at the bottom of the task?
By the way, more a general question and because I see you seems close to get a working version, what would be a estimated ETA for the final release of the re-write?
Many thanks :)

You don't have this?

Glad you're liking the work that's been done. Unfortunately, I don't have an ETA. I have considered releasing v1.0.0 without the dashboard page, and saving that for v1.1.0 which would move release forward a fair bit as almost everything else is complete.

now I understood!

I thought it was in the Comment area and not in the Description area :)
The only downside is that the task can take a lot of room if there are multiple sub-tasks, would be nice to get the same progression bar in case the sub-tasks are use in the Comment area: thanks to that the Board keeps clean and at a glance it is easy to the the progression bar and clicking on the task we can see the sub-tasks..
That's nice! Would be nice if we would click on the box from the Board without the need to Edit the task to check the sub-tasks.
Good idea for v1.0! the Dashboard can come later on, how far are you to release the 1.0? I remember in the past you shared your kanban of you tasks, glad to see it if you are ready to share it :)


First crack at a config file inputs

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have added an explanation of what your changes do and why you'd like us to include them.
[ ] I have written new tests for your core changes, as applicable.
[ ] I have successfully ran tests with your changes locally.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Sorry for bringing this up a little late but do you want to group the arguments as groups:
inputs, processing and outputs
Might help make things very clear as to impact of changing parameters.

Sorry for bringing this up a little late but do you want to group the arguments as groups:
inputs, processing and outputs
Might help make things very clear as to impact of changing parameters. Something like this - https://github.com/opera-adt/COMPASS/blob/main/src/compass/defaults/s1_cslc_geo.yaml

Not a problem. This is what I was trying to do with the existing numbering; i.e. everything under 1. is in the "date" group, but I do like making it more explicit. Let me give it a shot and see how it goes.

Just documenting here what we discussed in #368 . The two additional parameters that are needed for generating cubes are:

Horizontal spacing - called spacinginm in the cube prototype
Output projection system of code - called epsg in the cube prototype

The heights are already captured by heightlvs in the main code.

@dbekaert @piyushrpt I've got the ZTD working for a bounding box input. It's been quite a bit of changes because of a couple goals:

Don't read the lat/lon points until really needed
Along with that, handle the different query point options with a consistent API using a new set of AOI classes. This should be able to abstract quite a bit of several different places where we were doing argument checking, different things for the DEM depending on inputs, etc. In this scenario the geocube should be handled as simply another type of AOI (although I haven't get that far yet). Also I'm thinking it could help with chunking, but haven't looked into that yet.
configuration file for parameters instead of in the CLI. I like this, and it would allow us to run every program from a single config file if desired. I changed the inputs into groups as suggested by @piyushrpt.
Not required and not currently implemented except in the CLI, but we could go to a step-based approach as is used with MintPy where each step is handled as such, although we probably only have two or three steps.

Next few steps for this are to update the unit tests and check that a at least the basic examples run, then could be merged.

@dbekaert @piyushrpt I renamed some of the test suites that deal with the input argument IO so that they wont run. These will need to be updated to reflect the fact that we aren't using the CLI interface anymore.

@dbekaert @piyushrpt if one of you want to give a review and/or approve for merging, the tests are passing now!


Update 03-Bar-Chart.md

Fix documentation of stacked option to be a subset of the axes rather than a global attribute on a bar chart. The attribute chart was inconsistent with the example below it. It seems the example is correct and the chart was incorrect.

Thanks @cw0102


DanglingCloseParenthesis should default to Preserve

This reverts to pre-fork behavior.
Should be milestone 1.0 per #142 .

:+1: (as much as I hate to say it)

I'm working on this now (fixing unit tests).
It's the source of most of the churn in akka & other projects upgrading to the 0.1.6 / 0.1.7 fork.

yup, and my work projects :smile: (and ensime... but we are savvy enough to add the config option)

The pre-fork behavior was actually Prevent when preserveDanglingCloseParenthesis was false (which was the default).
Defaulting to Preserve doesn't change any code formatting when upgrading from 0.1.4 but it does change the behavior of Scalariform (since it stops preventing dangling parentheses).

Awesome, thanks for chiming in!
I'll let @fommil or @sschaef determine if we want to go for zero-churn (Preserve), 0.1.4-compatible (Prevent). Force being the 0.1.6-compatible behavior no one seems to want.
I'll note on the PR not to merge until this is resolved.