🛠 [TASK] : SRE to Specify packaging requirements for cat-gateway.

Summary
SRE to ensure consistency and quality of service packaging
Description
SRE to advise on best practice and requirements for the packaging of the cat-gateway for deployment.

@jmgilman

I believe we can package cat-gateway without entry.sh. All required settings are exposed as environment variables.


[feature][doc][do-not-merge] Add docs about how to use basic authentication

Fixes #14661
Modifications

Add a new topic about how to use basic authentication
Sync the topic in the left navigation and security overview

The preview looks good.

Note that the changes need to be applied to more historical versions as soon as it gets approved.
Documentation

[x] doc

Ping @nodece @codelipenghui @Anonymitaet for review. Thanks.


Long images run off bottom of screen

The markdown image format ![](path/to/image when used with 'long' images causes them to run off the screen. Example:

I get around this using the html format and vh units <img src="path/to/image" style="height:75vh"> I would find it very convenient if the default behavior was something like this. I can't imagine any occasion when it would be desirable to have a feature image 'cropped' by the screen during a presentation, nor have it deformed.

Hi @DougInAMug thanks for this feedback. As you probably know there is no mechanism in Markdown syntax itself to indicate or enforce image scaling for size or aspect ratio. So your approach of replacing basic markdown image syntax with a HTML img tag is one option.
Perhaps a better option for you given you are working with images that do not share a common aspect ratio with your presentation is to make use of GitPitch's special support slide background images. And specifically, to then make use of GitPitch's sizing and scaling support on those images. These features are provided specifically to handle the issues you have described.
See the following sample presentation to see how this sizing and scaling feature can be used to prevent both deformed images and image cropping on slides.


Hawkular 654 2

MikeT, please review.  This is the stuff I demoed to you for alerts center trigger detail views.  These views still need more work to be UX-compliant, but it should be a solid start.

Wow that took 4 Travis restarts to pass!

LGTM

Merged #619


Is this block of code autoregressive?

Hello, I've been examining your code and academic work. I see that _backbone is a GPT-2 model from the transformers library. It behaves as autoregressive when the past parameter is active.
Given assumptions like batch_size=1, points=3, and n_dim=2, xs will take the shape (1,3,2), and ys (or f(xs) as in the paper) will have the shape (1,3), aligned with the target outputs of the task.
Here's the relevant block of code:
@staticmethod
def _combine(xs_b, ys_b):
    """Interleaves the x's and the y's into a single sequence."""
    bsize, points, dim = xs_b.shape
    ys_b_wide = torch.cat(
        (
            ys_b.view(bsize, points, 1),
            torch.zeros(bsize, points, dim - 1, device=ys_b.device),
        ),
        axis=2,
    )
    zs = torch.stack((xs_b, ys_b_wide), dim=2)
    zs = zs.view(bsize, 2 * points, dim)
    return zs

def forward(self, xs, ys, inds=None):
    if inds is None:
        inds = torch.arange(ys.shape[1])
    else:
        inds = torch.tensor(inds)
        if max(inds) >= ys.shape[1] or min(inds) < 0:
            raise ValueError("inds contain indices where xs and ys are not defined")
    zs = self._combine(xs, ys)
    embeds = self._read_in(zs)
    output = self._backbone(inputs_embeds=embeds).last_hidden_state
    prediction = self._read_out(output)
    return prediction[:, ::2, 0][:, inds]

For zs, it seems to have a format of [[x1, f(x1), x2, f(x2), x3, f(x3)]], implying _backbone has visibility into all values of ys, including the target y3. The model seems to be predicting y1 through y3 based on these inputs.
Though the last line in the forward, prediction[:, ::2, 0][:, inds] is selecting hidden states corresponding to all xs instead of f(xs), the f(xs) are implicitly leaked into all the hidden states.
And in the eval_batch function, I don't observe any safeguards against data leakage. Here's the code for reference:
def eval_batch(model, task_sampler, xs, xs_p=None):
    task = task_sampler()
    if torch.cuda.is_available() and model.name.split("_")[0] in ["gpt2", "lstm"]:
        device = "cuda"
    else:
        device = "cpu"

    if xs_p is None:
        ys = task.evaluate(xs)
        pred = model(xs.to(device), ys.to(device)).detach()
        metrics = task.get_metric()(pred.cpu(), ys)
    else:
        b_size, n_points, _ = xs.shape
        metrics = torch.zeros(b_size, n_points)
        for i in range(n_points):
            xs_comb = torch.cat((xs[:, :i, :], xs_p[:, i:, :]), dim=1)
            ys = task.evaluate(xs_comb)

            pred = model(xs_comb.to(device), ys.to(device), inds=[i]).detach()
            metrics[:, i] = task.get_metric()(pred.cpu(), ys)[:, i]
    return metrics

Specifically, the line xs_comb = torch.cat((xs[:, :i, :], xs_p[:, i:, :]), dim=1) and ys = task.evaluate(xs_comb) seem to suggest that ys, the query into the model, is composed of (x1, f(x1), . . . , xk, f(xk), x_query, f(x_query) instead of excluding f(x_query).
Could you clarify what I might be missing? Any insights would be greatly appreciated.

Best,
Calvin

Hi, since the _backbone is auto-regressive, and each x_i/y_i is a separate token, the prediction for y1 depends only on x1, the prediction for y2 depends only on x1, y1, x2, and the prediction for y3 depends only on x1, y1, x2, y2, x3 which is exactly what we want.

Thanks, I think I identified the lines from GPT2 relating to causal inference,
if not self.is_cross_attention:
    # if only "normal" attention layer implements causal mask
    query_length, key_length = query.size(-2), key.size(-2)
    causal_mask = self.bias[:, :, key_length - query_length : key_length, :key_length].bool()
    attn_weights = torch.where(causal_mask, attn_weights, self.masked_bias.to(attn_weights.dtype))


[WIP]: Read next request if the request body is empty

Trying some crazy stuff.

Even need to read? Use what is still in the buffer (If there is something)?

True, can just slice the buffer


267 use async await

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

I know it's deprecated. But someone like use, us it. So you can merge it to simplify the usage.

I don't see any point in merging PRs here, the project is deprecated for a reason, so that we don't spend any time on on it. @Dufgui why don't you fork it make the changes you want and use it


static gif larger after processing

just testing my gulp file, so I downloaded a random static gif from the internet...
http://www.baroul-suceava.ro/images/SearchButton.gif
Here's the code I use to optimize...
gulp.task('images', function(){
return gulp.src('src/assets/images/**/*')
.pipe(cache(imagemin({ optimizationLevel: 3, progressive: true, interlaced: true })))
.pipe(gulp.dest('app/assets/img'));
});
Started at 45Kb, ended at 47Kb
After optimizing, maybe we should compare file sizes and output the smaller even if it's the original.

Thanks for the change!


llama2-7b finetune/lora.py fails

When I execute finetune/lora.py on llama2-7b and set the devices to 8, I get the following error

What's the error if you set CUDA_LAUNCH_BLOCKING=1 as the error stacktrace suggests?

@carmocca hello，Below are the errors when I set CUDA_LAUNCH_BLOCKING=1, they are too long so I pasted part of them

I believe I've seen this error if the data was tokenized with a different tokenizer. Are you using prepare_alpaca.py? Does this error not happen with a single device?

Yes, I execute python scripts/prepare_alpaca.py first

@carmocca Thanks for your reply, I re-executed the python scripts/prepare_alpaca.py --checkpoint_dir checkpoints/meta-llama/Llama-2-7b-hf/ to solve this problem, because the --checkpoint_dir I used before is checkpoints/meta -llama/Llama-2-7b-hf/checkpoints/EleutherAI/pythia-1b


Update peap-tls-connection-properties.md

Https is not supported in EAP schema target namespaces

@zaricvlado : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Cypress export don't work after new version (16/01)

Hello, I realized that I can no longer generate Cy code when I export the test through the recoder, the option is gone. Initially I tried to use it on other machines but they all give the same error.

This same issue is occurring at my side where i can't find the option to export script as .Cy file.

i can't find the option Export as a Cypress Test Script

Same issue here
Cypress Chrome Recorder 1.1.0
Chrome Version 109.0.5414.75 (Official Build) (64-bit)

i have the same issue. whats happened?

I'm also getting this same issue.
Cypress Chrome Recorder - Version 1.1.0
Chrome Version 109.0.5414.75 (Official Build) (64-bit)

Sorry about that everyone, we just submitted a patch release to the Chrome Store, that release should be published any day 1.1.1

@jordanpowell88
Isn't it a critical issue that we can't use the extension because of it?
The issue was opened on January 18th and the version in the store is still 1.1.0 until now.

Hi, i can see the issue with the version 1.1.1, can you please look into.

I am not sure what happened, but the export to cypress option is still missing. I am using the cypress chrome recorder updated on April 20.

It is there after I closed all chrome windows and restart chrome.

I too can not see the cypress recorder after installing the extension, closing all chrome instances and rebooting windows 11


⚠ Bofrost.be has degraded performance

In cd14486, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2355 ms

Resolved: Bofrost.be performance has improved in 058ccf8.


404 error accessing banner.md

REPRO

Connect to https://opencollective.com/:yourcollective/banner.md, where yourcollective is your... collective.

RESULT
404 error.

Hi Conrad - this is working for me.
https://opencollective.com/mochajs/banner.md. Is there a particular
collective where it's failing? (you need to remove the colon)
On Sun, Oct 1, 2017 at 10:47 AM Conrad Hollomon<EMAIL_ADDRESS>wrote:

REPRO

Connect to https://opencollective.com/:yourcollective/banner.md,
where yourcollective is your... collective.

RESULT
404 error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opencollective/opencollective/issues/441, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF0vPQzC1WDMM3mQPGgA7naVFI2Ra4Rnks5sn6YagaJpZM4Pp6kd
.

hey @hollomancer You are probably talking about operation code. it's working for me https://opencollective.com/operationcode/banner.md

@hollomancer this should be solved. Can you please confirm son we can close? thanks!

closing this, @hollomancer let us know if needs to be reopened.


some suggestion

thanks for you work,
and there is some suggestion for this UI

small sidebar dock width (strongly for me)
this wastes the space of Horizontal direction

hover setting item

the border of input box

Un useful space of panel

rectangles rounded corners of editor bottom;

Bread crumbs

Hi @heartacker, thanks for the feedback, much appreciated.
I'll go item to item to be clear:

The activity bar width is exactly the same as the vanilla (48px). What I did there was reduce the icon size and add a highlight to the active item (light grey colour). It might seem larger, but it's a perceived effect.
Dully noted. Will fix this soon.
Can you elaborate on that? What's wrong with the border there? You don't like it or is it too large?

4, 5 and 6 are being fixed along with some other layout issues.
Thanks!

I guess their point with the first one was that if the icons are smaller, then the activity bar being that size is just a waste of space, and can be reduced. Which does make sense.

@heartacker The theme now has a compact mode, which will condense the spacing on some of the container elements, most notably the Activitybar. You can find the option under settings.
You'll also not that there were other layout fixes based on your observations.
Thanks for that!


New endpoint

I was wondering why your feed stopped working. Here's a fix.

Oh, nice. And I like your bucket-map so it can be usable outside of my world. Gonna merge and test locally.

Works perfectly- "deployed" it, so it'll go back to updating every 6 hours. Ping me if you ever see any weirdness!

Great! Thanks for merging it so quickly. :)

I guess an even better (maybe faster?) way of checking if you are you, would be to use the head_bucket API call..
aws s3api head-bucket --bucket tedder

Oh well, both work.

Eh, I don't care too much. 4 calls to ListBuckets per day isn't bad. The other other way to do it is to add a flag for DEV_MODE.


Add free courses on AngularJS with TypeScript

Could do with an Angular2 sub heading as there are a lot of people coming into typescript from that perspective.

thank you @bcabanes !


add struct ifmap

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

struct ifmap doesn't exist in BSDs (checked OpenBSD, and quickly grep
source tree of DragonFly, FreeBSD and NetBSD).

@semarie Thanks :). I already moved it down into the notbsd part of the hierarchy.

Should the fields here be pub so they can be accessed externally?

:umbrella: The latest upstream changes (presumably #1002) made this pull request unmergeable. Please resolve the merge conflicts.

Should the fields here be pub so they can be accessed externally?

Yes. Good catch :p

@bors: r+

:pushpin: Commit 8671d81 has been approved by alexcrichton

:hourglass: Testing commit 8671d81feab2c9501839944c700a5f4aba6d80e0 with merge 89ef6bb1d9fadf2b66f5f9453ef549b73ae69a13...

:boom: Test timed out

@bors: retry

:hourglass: Testing commit 8671d81feab2c9501839944c700a5f4aba6d80e0 with merge 7c45a5a53693e6f891f83c8ad6c518afba08a8de...

:broken_heart: Test failed - status-travis

@bors: retry

:lock: Merge conflict

:umbrella: The latest upstream changes (presumably #1001) made this pull request unmergeable. Please resolve the merge conflicts.

@bors: retry

@canndrew: :key: Insufficient privileges: not in try users

Worth a shot.
@alexcrichton: this one is ready to go as well

@bors: r+

:pushpin: Commit 3fa8dba415f33f00fb361d64404b61435fad1eac has been approved by alexcrichton

:boom: Test timed out

@bors: retry

:lock: Merge conflict
This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again.
How do I rebase?
Assuming self is your fork and upstream is this repository, you can resolve the conflict following these steps:

git checkout add-ifmap (switch to your branch)
git fetch upstream master (retrieve the latest master)
git rebase upstream/master -p (rebase on top of it)
Follow the on-screen instruction to resolve conflicts (check git status if you got lost).
git push self add-ifmap --force-with-lease (update this PR)

You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial.
Please avoid the "Resolve conflicts" button on GitHub. It uses git merge instead of git rebase which makes the PR commit history more difficult to read.
Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Cargo.lock conflict is handled during merge and rebase. This is normal, and you should still perform step 5 to update this PR.

Error message
Auto-merging src/unix/notbsd/mod.rs
CONFLICT (content): Merge conflict in src/unix/notbsd/mod.rs
Automatic merge failed; fix conflicts and then commit the result.

:umbrella: The latest upstream changes (presumably #995) made this pull request unmergeable. Please resolve the merge conflicts.

I'm gonna close this now due to inactivity, but it can of course be resubmitted!


Create --one-shot flag to run one cmd and exit

And/or add a new "Exit" command to the base plugin

:+1:

:grinning:

What is an appropriate single char short form, or should we go with the long form only.
"-o" tends to mean output file in most cases. "-s" does not really mean much.

My instinct is -n (which in my experience usually means --dry-run, but could be adapted to --oNe-shot). I don't feel a short option is entirely necessary, however, since the idea is this'll mostly be run by IDE/editor frontends instead of by the users themselves

I would just go for a long form, if a short form is not immediately obvious and you don’t need to type it hundreds of times a short form isn’t very useful.

agree, and that is how I did it already. Just need to coordinate reply, and it is done


How to use the pretrained model?

Hey @Satyamjay , did you ever figure out how to do this?

@siangooding, @Satyamjay  Have you already figured it out? Any help would be highly appreciated.


fixit - Add support for SPM

Problem Analysis (Technical)
Solution (Technical)
Affected Components
Steps to reproduce:
Actual result:
Expected result:
Tests

Can we add SPM validation into CI? How we can make sure that project can be built successfully when using SPM dependencies?

Yes I was thinking about it, do we not care it will increase CI time? potentially 3-5 more minutes of CI.

Can we add SPM validation into CI? How we can make sure that project can be built successfully when using SPM dependencies?

Yes I was thinking about it, do we not care it will increase CI time? potentially 3-5 more minutes of CI.

We need this check. Otherwise how we can be confident that SPM is not broken after our commits?

Can we add SPM validation into CI? How we can make sure that project can be built successfully when using SPM dependencies?

Yes I was thinking about it, do we not care it will increase CI time? potentially 3-5 more minutes of CI.

We need this check. Otherwise how we can be confident that SPM is not broken after our commits?

Added a CI step to build a new project called SPM_Example.


Switch from Alice to Foundry

Codecov Report

Merging #56 (8282782) into main (2996137) will decrease coverage by 0.84%.
The diff coverage is 30.76%.

@@             Coverage Diff              @@
##               main      #56      +/-   ##
============================================
- Coverage     86.12%   85.27%   -0.85%
+ Complexity      280      279       -1
============================================
  Files            42       43       +1
  Lines           944      951       +7
============================================
- Hits            813      811       -2
- Misses          131      140       +9

Impacted Files
Coverage Δ

src/Command/LoadFixturesCommand.php
0.00% <0.00%> (ø)

src/Faker/FakerGenerator.php
100.00% <100.00%> (ø)

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 2996137...8282782. Read the comment docs.


refactor(githooks): move and improve post-checkout checks into validate-engines

the current version of this check complains about node version 9.x being incorrect (it needs 8.9.4). this fixes this issue and also moves the checks into our validate-engines script.
to test this, please checkout this branch with latest node version (should NOT complain), then downgrade node to v4 and try npm start (should complain)

@juliangieseke When using nvm, it does not have an issue with me using node 4.4! When reading the script itself, this makes sense. I think we have to modify the assertions a bit.

Yay! Even better, having npm as devDependency also installs the correct version locally! this means it doesnt matter at all which version is installed on our machines. 🎉
check this:
# Julian@Cookie:~/Code/dcos-ui/master/dcos-ui (jgieseke/improve-version-check *$%=)
$ node -v
v4.4.7
# Julian@Cookie:~/Code/dcos-ui/master/dcos-ui (jgieseke/improve-version-check *$%=)
$ npm -v
3.9.6
# Julian@Cookie:~/Code/dcos-ui/master/dcos-ui (jgieseke/improve-version-check *$%=)
$ npm run versions

><EMAIL_ADDRESS>versions /Users/Julian/Code/dcos-ui/master/dcos-ui
> echo npm $(npm --version) && echo node $(node --version)

npm 5.6.0
node v8.9.4

@ahoskins I think when you use nvm you need to open a new tab to have the right version. Like this:

Strange, might be my shell. It's fish

@ahoskins was right, it didnt complain. see my comment above, now it is even better :)
@ahoskins pls re-review 👍

I re-installed node_modules, and then used node 4.4.7, and still no error...

It's interesting what's happening here. So I switch my global node 4.4 successfully. Then if you notice, my PATH doesn't include node_modules/bin. But then, when invoking the script, it appends .node_modules/bin to my path, causing it to use the correct node version in the script.


Release v1.0.0

Minor clean-up post initial commit and prep for release.

Changes Unknown when pulling 48ed6e5ddcd58b8f1fa6a0125b275ca43f433c87 on develop into ** on master**.

Changes Unknown when pulling edafabb4824ddfd38efbb9d81215129fc671260e on develop into ** on master**.

Changes Unknown when pulling 0bcfeca2a022ad1e24ef60500fdb3e2b08958382 on develop into ** on master**.

We were close ... just took a while to track down. Needed to convert the salted HMAC key to binary and make sure every crypto hash update specified binary manually.


Snippets

I am trying to follow along with your tutorials but find myself confused by the uses of Snippets and things like #define TemplateCode.  I've tried once before to follow your tutorials but gave up after forking and cloning your sample code and found it so full of conditionals and snippets getting me so confused that I forgot what I was trying to learn.
Perhaps I am missing something basic - but I have not found anywhere that explains how to use and learn from your multi-snippeted code.   Is there any documentation or are there any tutorials that explain how to use your docs/aspnet repository to follow along with your tutorials?  If not, any suggestions?

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

ID: 7184a308-a254-9350-a5cd-5bced1f369ae
Version Independent ID: 726e3bf1-f367-d733-8933-bccc04da0e16
Content: Logging in ASP.NET Core
Content Source: aspnetcore/fundamentals/logging/index.md
Product: aspnet-core
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Good point @rswetnam ...
In a case like this one (the Logging topic), we have the define directive at the top of the Program.cs file ...
#define TemplateCode // or LogFromMain or ExpandDefault or FilterInCode or MinLevel or FilterFunction

It's set to TemplateCode and the rest of the symbols are commented out. The if-elif-endif singles out the section for TemplateCode and runs only that bit.
If you were to change that top line ... to say the FilterInCode symbol ...
#define FilterInCode

... then that bit would compile and execute.
We should really say something about this, probably near our how to run a samp language.. I'll take care of it. Thanks for opening this issue.

Thanks for your prompt response Luke - it was helpful.  My guess is that a
lot of people would benefit from the additional explanation that you spoke
of in the  how to run a samp
https://docs.microsoft.com/aspnet/core/index#how-to-download-a-sample
area.
On Mon, Nov 5, 2018 at 7:53 PM Luke Latham<EMAIL_ADDRESS>wrote:

Good point @rswetnam https://github.com/rswetnam ...
In a case like this one (the Logging topic), we have the define directive
at the top of the Program.cs file ...
#define TemplateCode // or LogFromMain or ExpandDefault or FilterInCode or MinLevel or FilterFunction
It's set to TemplateCode and the rest of the symbols are commented out.
The if-elif-endif singles out the section for TemplateCode and runs only
that bit.
If you were to change that top line ... to say the FilterInCode symbol ...
#define FilterInCode
... then that bit would compile and execute.
We should really say something about this, probably near our how to run a
samp
https://docs.microsoft.com/aspnet/core/index#how-to-download-a-sample
language.. I'll take care of it. Thanks for opening this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Docs/issues/9470#issuecomment-436122799, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACxVFyQxxibDQRHCrD94iFWOikO7gjwJks5usQe8gaJpZM4YPskx
.

--
Roger Swetnam
p.<PHONE_NUMBER>
c.<PHONE_NUMBER>


[BUG] Volume detaching and attaching repeatedly while creating multiple snapshots with a same id

Describe the bug
Volume detaching and attaching repeatedly while creating multiple snapshots with a same id
To Reproduce
Steps to reproduce the behavior:

Create a RWO volume
Create a snapshot via CSI
Somehow the external volume snapshot controller repeatedly request multiple snapshot creation with the same snapshot id
LH encounters error because the snapshot id is conflict with an existing snapshot, and then trigger error handling (rollback).
The rollback deleted the existing snapshot file and its meta file but didn't update the volume.meta.
The volume.meta is inconsistent with snapshot images/metas because of the error handling.

Expected behavior
Volume should be still attached after rollback (error handling)
Log or Support bundle
If applicable, add the Longhorn managers' log or support bundle when the issue happens.
You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment

Longhorn version: v1.2.2
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of management node in the cluster:
Number of worker node in the cluster:

Node config

OS type and version:
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

https://github.com/longhorn/longhorn/issues/3844
cc @PhanLe1010

Yeah, we might want to raise the priority for https://github.com/longhorn/longhorn/issues/3844 because this issue is also causing https://github.com/longhorn/longhorn/issues/4227

Verified passed on master-head (longhorn-engine caf17d4) by following the test steps.


These are some tweaks which help to make installable from pip

Setting this as [WIP] until it works

Closing this in favour of merging an alternative branch with more features!


Complete the Kube "features" list and how each item could benefit multiple groups

@smarterclayton still relevant, or ok to close?

Outdated.


convolution from 64 to 32

hello everyone,
I'm wondering how does the CNN handle a convolution of 64 filtered images into 32 filtered images.
Here is an example:
input_img = Input( shape = (64, 64, 1))
x = Conv2D(64 , (3,3) , activation = '...' , padding = '...' , name = 'layer1')(input_img)
x = Conv2D(32 , (3,3) , activation = '...' , padding = '...' , name = 'layer2')(x)
So you give the network an 64*64 grayscale image and the first layer will apply 64 filters to this image and output 64 differents versions of the input image. But then in the second layer it should output 32 filtered images, so does layer2 apply 64 times the same filter on the ouput of layer1 and select 32 filtered images at random ?

Each output channel of an channel convolutional layer is basically generated through a weighted sum of N convolutional filters of every input channels. So you can think of a Conv2D layer with 64 input channels and 32 output channels as consisting of 64 * 32 2d convolutional filters.


[MM-32692]: Feature: Add two more CRT options in System Console: default_on & always_on

Summary
Feature: Add two more CRT options in System Console: default_on & always_on
Ticket Link
JIRA: https://mattermost.atlassian.net/browse/MM-32692
Related Pull Requests

Has server changes https://github.com/mattermost/mattermost-server/pull/20088

Release Note
* Add Default on and Always on options for CRT in system console

Hello @vish9812,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/e2e-test

You don't have permissions to trigger this command.
It's only available for organization members.

Assuming this needs a server PR as well, but please ensure that the user account setting for CRT is hidden if "Always On' is selected

/e2e-test

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/189120

/update-branch

/update-branch

Error trying to update the PR.
Please do it manually.

@vish9812 Can you please resolve the conflicts so we can merge.
I'll create a follow up ticket for implementing the E2Es as not to hold this PR up.

Sorry, I should have refreshed the browser first 🙂, didn't see you updated already.

@vish9812 I missed that E2E run triggered earlier, never completed successfully. Removing my approval pending E2Es.

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/192750

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/192919

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/192992

@jgilliam17 @vish9812 There was a problem with Cypress E2E but it's all good now. You'll find the result at https://automation-dashboard.vercel.app/cycles/24cb32d3-f81c-4ec1-8144-45001e2b7c7c and in the community channel.

@vish9812 I didn't merge due to do not merge label. Please merge both webapp and server PR when ready - thanks 🙂


Support topologySpreadConstraints

Currently we only have affinity rules to play with to control scheduling. In order to guarantee that a replica and primary don't get scheduled in the same zone we need to be able to specify topologySpreadConstraints.
You currently can limit the zone using affinity rules, but it's not enough, with topologySpreadConstraints you can control this a lot better.
Reference: https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/

You can already set a topologyKey that will ensure that doesn't happen. See https://cloudnative-pg.io/documentation/current/api_reference/#AffinityConfiguration

You can already set a topologyKey that will ensure that two pods are not scheduled in the same zone. See https://cloudnative-pg.io/documentation/current/api_reference/#AffinityConfiguration
This is what I use:
  affinity:
    topologyKey: topology.kubernetes.io/zone

Yes this will in theory work but in a smaller cluster lets say 4 nodes? there's a good chance two replicas will be scheduled on the same node since the hostname is no longer the topologyKey for the pod affinity.

How can two replicas be scheduled on the same node, when they wouldn't be allowed in the same zone?

@ColonelBundy How can two replicas be scheduled on the same node, when they wouldn't be allowed in the same zone?

Right, I misspoke. What I meant was if I have two zones I would be limited to two replicas. With topologySpreadConstraints I can fulfill that requirement and once it's fulfilled I can scheduler more replicas in the same zone. This cannot be done with pod affinity I'm afraid.
On top of that I can combine constraints such as that I shouldn't be allowed to put more replicas on the same node until all other nodes have replicas of which cannot be done with affinity either.

@armru @sxd  Thanks for the quick implementation! Greatly appreciated!

@ColonelBundy all the credit to @armru :)


Remove use of express session is no session defined.

This is very dangerous. If you can't access redis (or your store of choice) for some reason (it's down for example) then the wizard will make a session anyway using in memory sessions which are known to be leaky leaky leaky.
It should error at this point which the developer can capture and deal with.

Good point I'll update the package json now

This looks good @easternbloc, I've added some minor comments.

<EMAIL_ADDRESS>published.

:+1:


Migration Guide for XLDataLoader

We were using XLDataLoader in one of our applications and right now we are about to publish the last version that will support iOS 7 and hopefully we'll be able to move to XLData.
Since we use XLDataLoader in several parts of the app I was wondering if there's a guide on how to make this transition.
Thank you.

It's not worth writing the migration guide since there are only few people using XLData.
@ppaulojr Please feel free to ask if you need any help with this.


Add furigana on subtitles

having an option to turn furigana on/off for subtitles

Processing language isn't asbplayer's intended use, and there are a lot of other programs that are better at doing this. I would also argue that asbplayer is good at doing what it does (making flash cards) because it leaves those problems to other tools such as Yomi-chan.
I googled "adding furigana to Anki cards" and found that BrittVsJapan wrote a blog post on how to do what you want just using Anki plugins - maybe it could help you: https://www.britvsjapan.com/adding-furigana-to-japanese-in-anki/

no i meant furigana on the subtitles not on the anki card like when you watching and show and you can enable the option to see furigana when your are watching the show on the subtitles


Fixed Lag with Arrows

Basically if you had a character (that used AnimatedSprite instead of AnimationPlayer) and you pressed a direction / note, the game would lag. I have fixed this, and also added actual support for AnimatedSprites because apparently it actually wasn't in the base repo.

Seems fine


Initial helm chart

Deploy via helm chart
I use nebraska in my homelab to update my flatcar hosts esp. to reduce the traffic for downloading the new images.
For this I made a helm chart.
Are you interested in this? I found not that much information about deploying and hosting nebraska in production.
What also could be nice to provide:

[ ] docs about deploying in Kubernetes (with example yamls)
[ ] docs about deploying in Kubernetes with Kustomize
[X] GitHub Actions for hosting the chart as a static resource in gh-pages (done with Helm Chart Releaser GitHub Action)

How to use

clone the repo
Update dependencies

 helm dependency update ./charts/nebraska

then run the following helm commandhelm install my-nebraska ./charts/nebraska -n kube-system

(customize if needed)helm upgrade my-nebraska ./charts/nebraska --set ingress.hosts={my-nebraska.example.com}

Testing done

Added the Helm Chart Testing GitHub Action to trigger testing on every new PR

@johananl , Do you wanna take a look at this one?

Hi @knrt10,
Do you plan to use the branch gh-pages and GitHub Pages? If not, I can also add a github action workflow which releases the chart through a GitHub release and updating an index.yaml file inside the branch gh-pages.

Hi @mkilchhofer , we have our own documentation system, so we don't plan on using gh-pages (unless I misunderstood your point).
A Github action to update stuff on release sounds fine. I intend to do that for the container image as well in the future.

the docs, BTW: https://kinvolk.io/docs/nebraska/latest

Hi @mkilchhofer , we have our own documentation system, so we don't plan on using gh-pages (unless I misunderstood your point).
A Github action to update stuff on release sounds fine. I intend to do that for the container image as well in the future.

Okay, perfect 👌
We need an empty gh-pages branch for this, which I cannot provision inside this PR:
$ # Prepare an empty branch
$ git checkout --orphan gh-pages
$ git reset
$ rm -r *

$ # Generate the index structure
$ cat <<EOF > index.yaml
apiVersion: v1
entries: {}
generated: "2021-03-01T00:00:00Z"
serverInfo: {}
EOF

$ # Add the file
$ git add index.yaml
$ git commit -m 'Add empty index.yaml'
$ git push

Thanks again for another big push @mkilchhofer !
I was going to take a look now too but I'd like to ask you to squash the commits that you think should go together.
Ideally we should have atomic commits. Depending on this case, it may mean that all your commits should be squashed, or maybe there are a few that should be added separately.
However, we usually never squash commits together (unless their small fixups to one base commit) when merging PRs. So I kindly ask you to squash the commits to create the atomic ones you see as adequate, and that'll make the PR easier for another review round and testing.

Alright, I think split all in 2 commits, 1 for the chart itself and one for the CI part. When you are okay with that :)

Hi @knrt10,
Do you plan to use the branch gh-pages and GitHub Pages? If not, I can also add a github action workflow which releases the chart through a GitHub release and updating an index.yaml file inside the branch gh-pages.

Hi @mkilchhofer , I merged the PR but didn't realize it was still trying to use the gh-pages branch (which we don't use).
Can we use an alternative for this? Maybe using the Github Releases as you hinted in the message above, or does that still require the gh-pages branch?
Thanks.

Hi @knrt10,
Do you plan to use the branch gh-pages and GitHub Pages? If not, I can also add a github action workflow which releases the chart through a GitHub release and updating an index.yaml file inside the branch gh-pages.

Hi @mkilchhofer , I merged the PR but didn't realize it was still trying to use the gh-pages branch (which we don't use).
Can we use an alternative for this? Maybe using the Github Releases as you hinted in the message above, or does that still require the gh-pages branch?
Thanks.

Hi @joaquimrocha, Maybe I didn't understand your statement correctly. What I understand was that you do not use the branch gh-pages so it is available for this intent :-)

No :)
I meant I'd prefer to keep not using that branch, but I am guessing there's no greater alternative if we don't intend to host it somewhere else?

Exactly. We need to serve an index.yaml via https to the helm users. A next step could be to list this chart on https://artifacthub.io/ then we would need an additional file for this besides the index.yaml


Sorry, you are not allowed to access this page Error on Fresh install

I know this is a common security bug/issue, not sure what to do though. Installs perfectly, CSS fine, I have SSL HTTPS Certs working OK, On a Ubuntu 16.4 Cloud Server.
Error on this page: /wp-admin/
Actually Logs in/authenticates OK which you can see going back to root
Using mysql:5.6 container is one difference, rather than Maria DB.
Any advice? Going through various logs atm, thanks

Hi Conor,
Do you still have this issue? It will be hard to find the cause because there are so many moving parts besides the Docker container, but we can give it a try!
To start with, how did you setup HTTPS? Is it still possible to use HTTP? If so, does it work on HTTP?


Dynamic component duplicates sibling node instead of appearing with v-if

Version
3.0.0-beta.3
Reproduction link
https://jsfiddle.net/bLv2at3g/15/
Steps to reproduce
Click the button Click me to break things!.
What is expected?
Heading Hello should appear.
What is actually happening?
The clicked button is duplicated.

This is also fixed by 7d0ab339


Is any generated date&time related info included in the body of a trained model?

Could anyone tell me if there is any generated date&time related information included in the body of a trained model? If there is, how to read it out? Thanks.

Currently such record is not included in the model. May I ask for what purpose did you need it?

Currently such record is not included in the model. May I ask for what purpose did you need it?

Sorry for late reply. I am just wondering if there were mentioned record included, it might be used to manage the trained models  as a much reliable evidence than file's created/modified properties or the filename.


OSP-17: move policies from router to tenant + L4 ACL [cherry-pick]

Reviewer: trivial
CC: @sarath-kumar

Refer to this link for build results (access rights to CI server needed):
https://jenkins.bigswitch.com/job/openstack_horizon_bsn_pull_req/138/


Fossa issues

What this PR does / why we need it:
Fixes FOSSA licensing issues brought up here
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Fix FOSSA licensing issues

Closing in favor of #2796
/close


The process cannot access the file

When other software locks a file, we fail to load a pack.  We should fail more gracefully.
System.IO.IOException: The process cannot access the file
'C:\Users"me_as_a_user"\Documents\Guild Wars 2\addons\blishhud\markers\Data\SchattenfluegelTrails\Regularnodes\st_regularnodes_platinum_node.png'
because it is being used by another process.

This issue should be resolved now.


Dummy PR - Core Platform Tests

Description of changes:
This is just a test PR to see if Core Platform tests work upon relaxing the assert conditions.
No review/merging required.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/platform_tests

/platform_tests ref platform_tests

/platform_tests ref=platform_tests


Live demo

Hi, Nice package.
But I think it will helpful for beginners if you provide a live demo to test it out.

Sure thing. I wrote a quick demo app available under /demo but it requires cloning and starting a dev server which is not convenient.
I think It could be hosted using github pages. I'll look into it.

Great!
Some people do not have time for cloning and run the demo on their own machine.

Here is a demo I just created. https://codesandbox.io/s/0315qvkx20

Thanks for the input, and feel free to let me know if anything in the demo is unclear.


⚠ Itaca Gallery has degraded performance

In f0ca947, Itaca Gallery ($SITE_TCGLR) experienced degraded performance:

HTTP code: 200
Response time: 5371 ms

Resolved: Itaca Gallery performance has improved in f43cc54 after 14 minutes.


new sample for infinite streaming command line app

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for discussion)

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

It looks like the lint task failed: https://source.cloud.google.com/results/invocations/721827b3-1e51-409a-b3fa-5c639ad28638/targets/cloud-devrel%2Fclient-libraries%2Fnodejs%2Fpresubmit%2Fgoogleapis%2Fnodejs-speech%2Fnode10%2Flint/log
You should be able to run npm run fix to have it auto-formatted 👍

I ran  npm run fix
to fix the lint issues.
But, also took some time to add more documentation / comments / and add the file to the main samples directory. I wasn't able to add a working location tag for infinite streaming.

Accidentally merged with master while committing, so I reverted and then redid my changes, and now it is passing lint, and my tests. Ready to Merge into master, I hope.

Hi @stephenplusplus @callmehiphop
I've made the requested changes.
Wondering if what the next steps are to get the pull request committed.
thanks!
kristin

Sorry for all of the trivial, lint-y comments.
As another thought, is this functionality that should be written into the library itself?

Sorry for all of the trivial, lint-y comments.

I appreciate it all!

As another thought, is this functionality that should be written into the library itself?

This has been discussed as a possible outcome.
I am working on similar samples for Python and Java and then looking into the client library possibility.
Is this where you think this functionality should live?

Thanks for the changes. Regarding moving the functionality into the library, I think that would be useful and save the user a lot of work. The use case seems common enough that it is worth it.
For now, it all looks good to me! Feel free to merge when ready.


Any idea about vllm compatible?

How can I inject DRUGS to vllm framework?

I ask myself this every day.
I think the right file to modify is probably vllm/model_executor/attention.py. I haven't tried implementing it, so I'm not sure, but if that's the right spot it should be fairly straightforward to get working by following the porting guide.


🚀 House: house creation test

Created a create_house fixture and a test that validates its creation

@shlior7 is this PR blocked by PR #28 ? If so, please mark it accordingly

@shlior7  #28 was merged, please rebase


feat: create tags for new revisions published to the store

Hey :wave:
So as discussed in our catch-up meet, here is a first draft of how we might want to create tags automatically in repositories based on the revisions that were released to the store. The aim here is to make debugging issues a little easier, as when an issue is reported on a snap with a given revision, you can just checkout the tag and build/install/troubleshoot.
It's a little complicated with multiple architectures. The design I've chosen is to create one tag per revision, and name with the arch just to make searching/bisecting easier. The tag names will be rev<num>/<arch>.
You can see an example run here and the resulting tags here.
I should note that this is actually a breaking change because all repos that use these workflows will need to add a new input to the release workflow. I think if we merge this, we should tag the repo at v1, and work on using the actions with a versioned reference like @v1 rather than @main.
If this does merge, I'll do some follow-up PRs to address the incompatibilities on the other repos.

A note that one thing we could consider is including the version of the upstream project in the tag, e.g. for Signal we could have something like 6.42.0/rev576/amd64 - happy to iterate on the format.

+1 for adding the snap version to the tag, this will make it more clear, imo. I'll take a better look at this PR later.

@merlijn-sebrechts version added to tag name in 78edeec (#23)


Formatter asRelativeTime() result can be off by several hours

What steps will reproduce the problem?

Set your timezone as continent/capital
Set your $value with new Expression('NOW()');,
Try to calculate the relative time with Yii::$app->formatter->asRelativeTime($value)

What is the expected result?
Just now
What do you get instead?
In an hour (Based on your difference to UTC+0)
Possible causes
https://github.com/yiisoft/yii2/blob/ef9793bebc512245edcd9bee5565d8025ccfdb5e/framework/i18n/Formatter.php#L916-L927
What if the timezone has already been set for $timestamp?
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.2

Operating system
macOS 10.14 Mojave

Set your timezone as continent/capital

where did you set the timezone? you need to set timezone PHP and in DB server. Data in database should ideally be stored in UTC.

https://www.yiiframework.com/doc/guide/2.0/en/structure-applications#timeZone
I did not modified the default config for php since the project I am working on is still in it's test phase. I've just wanted you to know, that in certain conditions you can encounter this kind of bug. You could still add some information in the official documentation about this.

You are probably using it wrong, please check the documentation at https://www.yiiframework.com/doc/guide/2.0/en/output-formatting#time-zones
I added a link to that part of the guide from the page you linked.


Is it still equiprobable?

I read in the last line of the Readme that this implementation is equiprobable, was wondering if that is still the case.

Yes, I unfortunately did not find the time to work more it. Contribs welcome!


DeviseTokenAuth::TokenValidationsController#validate_token returns 401 unauthorized.

Its been almost 3 days and I haven't actually pin pointed the cause of the issue.
I have a set up consisting of a rails 4 API created using rails-api gem and a front end angular2 application created using angular cli.
I have been successfully able to create users, log them in and log them out of my angular 2 app using angular2-token package and devise_token_auth gem on backend.
The issue I face occurs when I have logged in as a user and upon every page reload I get 401 unauthorized in my browser's console.
And on the rails end I get the following output.
Started GET "/api/v1/auth/validate_token" for <IP_ADDRESS> at 2017-02-06 17:42:49 +0500
Processing by DeviseTokenAuth::TokenValidationsController#validate_token as JSON
SELECT  "users".* FROM "users" WHERE "users"."uid" = $1 LIMIT 1  [["uid"<EMAIL_ADDRESS>Completed 401 Unauthorized in 76ms (Views: 0.2ms | ActiveRecord: 0.3ms)

If I run request.headers.inspect I get the following response on the server side
 "HTTP_ACCESS_TOKEN_NAME"=>"xxxxxxxxxxxxxxxxxx", "HTTP_EXPIRY"=>"xxxxxxxxxxxxxxxxxx"<EMAIL_ADDRESS>"HTTP_CLIENT_NAME"=>"xxxxxxxxxxxxxxxxxx", "HTTP_TOKEN_TYPE"=>"Bearer"
Which gives the idea that the authentication headers are being actually sent to the rails API.
Further more I have also sent curl requests to the validate_token method using the credentials I have in my localstorage as I login and I get similar 401 response which narrows down the problem to the API (I may be wrong in that regard).
Here is the response I get upon running a curl request.
Further more I get nil when I inspect current_user on the server side.
I also have rack-cors configured on my API with the following configurations:

config.middleware.use Rack::Cors do
      allow do
        origins '*'
        resource '/cors',
          :headers => :any,
          :methods => [:post],
          :credentials => true,
          :max_age => 0
        resource '*',
          :headers => :any,
          :expose  => ['access-token', 'expiry', 'token-type', 'uid', 'client'],
          :methods => [:get, :post, :options, :delete, :put]
      end
    end

If there is anything I am missing or any more information that I should be providing please let me know I have been stuck on this issue for a long time now.
Thank u so much.

Could you write content of your app/models/user.rb file?
And did you check server responce?
P.S: issue could be related to :confirmable user email. If you setup Devise on that way (user need to verify email after registration) he would not be able to login until he would confirm email.

Sorry for delayed response i fixed the by using Rails 5 for my API. Previously I used rails-api gem with rails 4 in which scenario I was getting the 401.
It seems to work perfectly fine on development but in production upon refreshing repeatedly I get 401 from the server. Is it a normal behavior ?

Have the same issue, update to latest rails
Rails 5.0.2
devise_token_auth (0.1.40)
https://vimeo.com/212141533

Also experiencing this issue, haven't looked into this in too much detail yet but will post here if I find a solution @91sticker.

Seems like this was resolved according to @91sticker ...
might be a problem with tokens being generated or not being generated. Feel free to reopen if it's still an issue.


Contributing

@Minishlink @yleflour had to trigger an update for your last pull requests and added some docs about contributing :)

thanks guys!


Tutorial plz?

Bro please post a tutorial for those that have no idea what they are doing (me) I've personally never used this site before so I have absolutely no clue what I'm doing 🗿

yes please 🙏🏻


Verification of uploaded documents

Figure out how to avoid errors:
Test cases

proposed solution:
add logic to the approve contract endpoint: change policy for the specific bucket with the uploaded documents to download
mc policy download contract_requests/000025b4-9875-4a4c-b874-c72df7aaa5cb/

we should also add policy to the buckets with media data:
minimum file size: 10 KBytes
maximum file size: 4 MBytes

https://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketPUTpolicy.html

another option:

when new signed url are generated, add _upload prefix to the file name
add logic to POST /api/contract_requests/id:

copy files with the name without suffix to the same folder
delete all the files with the upload prefix

PS:
all the contract related files should have .pdf extension.


Update logo colors

Change dark gray to navy and ensure that the lighter gray is the correct light gray.
(Don't forget to update the PNG fallback.)

Done.


Suggestion new features - Min/Max characters of tags

Congratulations for the package !
I have a suggestion:
Limit size(lenght) min and max of tags

Hi @edukmattos , thanks for the suggestion.
Maybe it is possible to include some kind of validation callback so that can be used to check if the new tag is valid or not.
I have to think more about this so please give me sometime.

@edukmattos Hi and sorry for my extremely late reply. You could now do this in 0.1.2 using the inputFormatter prop.
Maybe something like this.
inputFormatters: [LengthLimitingTextInputFormatter(15)],

If you need anymore help with with please create another issue. I will close this for now.
Thanks

For min length, I've came up with such workaround. It's not perfect, because the invalid text disappears, but in some cases it may be better than nothing. You could also set resetTextOnSubmitted: false so the invalid text won't disappear when enter/done is pressed, but I haven't found such workaround for delimiter - that one always causes the invalid text to disappear.
            onTagChanged: (String inputValue) {
              if (!isValid(inputValue)) {
                return;
              }
              // add tag to list
            },
            onSubmitted: (String inputValue) {
              if (!isValid(inputValue)) {
                return;
              }
              // add tag to list
            },

Also
  bool isValid(String item) {
    return item.isNotEmpty && item.trim().length >= 2;
  }


Slim down images: Move packages from server layer to _dev

While updating all README.md files I encounter the following packages within the server feature:
https://github.com/gardenlinux/gardenlinux/blob/main/features/server/pkg.include#L27-L30
IMHO, we should keep the features minimal and move those packages to the _dev feature which would lower the overall image size.
Opinions? @credbbl @Vincinator @MrBatschner @gehoern @nanory

Removing any of these packages would make debugging anything OS related in a productive environment an absolute pain in the  back. wget is required for Gardener bootstrapping.

I think currently you're absolutely right, but with upcoming features in the near future we might change this. I think we could solve this by #1134 as well as we could provide an "inject script" that copies needed binaries for this (we're always running on testing, so we might fetch the debs, extract them and inject them directly) for debugging. However, this might only work on these low dependency package where all required libs are already present. However, immutable systems might make some further trouble here.
Probably you're right and we shouldn't make such a big effort to just get rid of some small packages.


Web3 1034 zkverify integration

Full integration using latest zkverifyjs package stored as tgz file

Also worth noting no effort was made to fix all the typescript and IDE flagged issues with the original code.


Package dependency issue when installing requirements of tag 17c with Python 3.4.3

When I try to install the dependencies of the flasky tag 17c into a new, clean venv with Python 3.4.3 I get the following error message:
(venv)florian@florian-desktop ~/ws_github/flasky $ python --version
Python 3.4.3
(venv)florian@florian-desktop ~/ws_github/flasky $ pip install -r requirements.txt
Downloading/unpacking Flask==0.10.1 (from -r requirements/common.txt (line 1))
...
Downloading/unpacking gunicorn==18.0 (from -r requirements.txt (line 3))
  Downloading gunicorn-18.0-py33-none-any.whl (93kB): 93kB downloaded
Downloading/unpacking psycopg2==2.5.1 (from -r requirements.txt (line 4))
  Downloading psycopg2-2.5.1.tar.gz (684kB): 684kB downloaded
  Running setup.py (path:/home/florian/ws_github/flasky/venv/build/psycopg2/setup.py) egg_info for package psycopg2

    Error: pg_config executable not found.

    Please add the directory containing pg_config to the PATH
    or specify the full executable path with the option:

        python setup.py build_ext --pg-config /path/to/pg_config build ...

    or with the pg_config option in 'setup.cfg'.
    Complete output from command python setup.py egg_info:
    running egg_info

creating pip-egg-info/psycopg2.egg-info

writing top-level names to pip-egg-info/psycopg2.egg-info/top_level.txt

writing dependency_links to pip-egg-info/psycopg2.egg-info/dependency_links.txt

writing pip-egg-info/psycopg2.egg-info/PKG-INFO

writing manifest file 'pip-egg-info/psycopg2.egg-info/SOURCES.txt'

warning: manifest_maker: standard file '-c' not found

Error: pg_config executable not found.

Please add the directory containing pg_config to the PATH

or specify the full executable path with the option:

    python setup.py build_ext --pg-config /path/to/pg_config build ...

or with the pg_config option in 'setup.cfg'.

----------------------------------------
Cleaning up...
Command python setup.py egg_info failed with error code 1 in /home/florian/ws_github/flasky/venv/build/psycopg2
Storing debug log for failure in /home/florian/.pip/pip.log

The log file was not helpful to me.

This is not a package dependency, it is just failing to install the postgres Python package because you do not have postgres installed on your machine. The 17.c tag is for deploying to Heroku, and this requires Postgres. If you want to bypass the error and install everything anyway, just comment out the psycopg2 line in the requirements.txt file.

Thanks a lot for the lightning fast answer.


AttributeError: 'Presentation' object has no attribute 'slide_layouts'

I am trying out this library for the first time and following the docs : http://python-pptx.readthedocs.io/en/latest/user/slides.html
When I ran this:
`SLD_LAYOUT_TITLE_AND_CONTENT = 1
prs = Presentation()
slide_layout = prs.slide_layouts[SLD_LAYOUT_TITLE_AND_CONTENT]
slide = prs.slides.add_slide(slide_layout)`
I got the following error:
`---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
 in ()
2
3 prs = Presentation()
----> 4 slide_layout = prs.slide_layouts[SLD_LAYOUT_TITLE_AND_CONTENT]
5 slide = prs.slides.add_slide(slide_layout)
AttributeError: 'Presentation' object has no attribute 'slide_layouts' `
I am on Mac OsX10 , running Jupyter iPython Notebook, Anaconda installation.

Well, the Presentation object in python-pptx certainly does have a slide_layouts property: https://github.com/scanny/python-pptx/blob/master/pptx/presentation.py#L55
I suggest you look at the type of Presentation and make sure it's what you expect it to be:
print(type(prs))

Could be something wrong in the import statement or perhaps you installed someone else's fork or an earlier version.
import pptx
print(pptx.__version__)

should return: '0.6.1' or similar, although I expect anything after '0.5.0' should work for most purposes.

Hi scanny , thanks so much. My version is 0.3.2 . I had installed this version : https://anaconda.org/asmeurer/python-pptx
Found the latest version: https://anaconda.org/conda-forge/python-pptx
Will Anaconda keep this updated in the future , I wonder?

Dunno. You'd have to inquire with their packaging folks I expect. New releases come out fairly often, at least a few times a year lately, so if you can learn to install yourself directly that might be the best bet. pip makes it dead easy, but I'm not familiar with the Anaconda environment so that might make things trickier.


Cannot reply to posts.

When replying to a post using the API, the error below is always returned.
{
    "errors": {
        "parent_id": ["specifies a nonexistent post"]
    }
}

I have verified that the ID's I'm passing in are correct, yet this error is always returned.

I'll see if I can duplicate this. For the sake of expediency, can you provide an example post ID?

Try<EMAIL_ADDRESS>

@zujko This should be working now. There were a couple of extremely broken things about the whole posting process, due to light test coverage and the complexity of the "message-IDs" changeover.
I've successfully posted a reply to the post you provided, using the API. See if it works for you.

Yep, it's working on my end. Thanks!


Analysis seeder

This PR adds a small analysis module that reads the file eval_cache2.bson and places it into the eval_cache2 collection in the database. The BSON file contains 1000 positions and was created by me in the most ridiculous way. I wrote a Python script that automatically iterated through 1000 positions and through webscraping, evaluated each position for 3 seconds on the Lichess analysis board. This took about an hour but I think the results are great (the evaluation was done on Apple M2 Pro). I think almost every position reached a depth of 20 or more.

What's a good way to know which positions are available from this?

What's a good way to know which positions are available from this?

That's a fair point, I don't think there is such a way. The positions are roughly the first 1000 that come up when reading from this file provided by Lichess. I don't know if the positions are sorted in that file using some logic? If not, it looks like at least the starting position and some of the most common opening positions are there (and obviously some checkmates etc). But predicting which positions are there and which are not is probably impossible unless the file is sorted somehow.


chore: Add flag for disabling read tx timeout on BlockchainProvider

closes #11760

I dont want to change anything in this code because we're going to deprecate this soon

I should leave it ?


graphtype Heatmap :: show months rather than days of the year

Temperature/Humidity - heat map and all he other ones:
It would be nice to see months instead of a number from 0-364 for days of the year.
the problem is that the ap is created by plotting DOY vs Hours... not sure how to override the display or go around it...

I think this might be helpful! if we use a datetime object we can easily configure

Yes, I saw it but I didn't manage to make it work straight away...
We have a datetime object in the dataframe but the issue is that we need to be able to use the Day Of Year as x coordinate, not the full date...

@giobetti I solved the problem with this commit 59a0fe97a47d29a2f1ce3920d3dd2704b5f9b53a now the figure looks like this:


ARCHER2 fixes from MOSRS (includes all changes)

This pull-request represents all changes on branches/dev/adrianhill/r8166_archer2_fix on MOSRS relative to our master branch

The good news is that this compiles. The challenge is that it includes many more things besides just fixes for ARCHER2 (because this was branched from trunk on MOSRS and not from a tagged version). I will need to go through and find the changes that we need.

Am I right in thinking that the first commit in this list comprises all the Reading changes to bring Leeds-MONC level with the HEAD of MOSRS and so can be ignored for the time being?

Have gone through the commits, and if numbering the list of commits on this ticket from the top then the list of commits which are needed to run the Leeds branch without casim or socrates on Archer2 are commits 3,4,5,7 and 10. Commit 1 seems to be the Reading mega-change, commit 2 is changes to code that only exists on the trunk or if commit 1 has been merged in. Parts of commits 4 and 8 may be needed and I am unsure of commit 9 - more testing with casim and socrates will hopefully shed light on this.
I expect to have a PR ready for review soon once I have tried running bomex with casim, socrates and both casim and socrates.

I'm closing this because it is being superseded by https://github.com/Leeds-MONC/monc/pull/45


[tool_crash] UnsupportedError: Unsupported operation: Cannot extract a file path from a org-dartlang-app URI

Command
flutter run --hot -d web-server --verbose --resident --target=lib/main.dart

Steps to Reproduce

...
...
...

Logs
UnsupportedError: Unsupported operation: Cannot extract a file path from a org-dartlang-app URI
#0      _Uri.toFilePath (dart:core/uri.dart:2710:7)
#1      toMultiRootPath (package:flutter_tools/src/compile.dart:884:35)
#2      DefaultResidentCompiler._recompile (package:flutter_tools/src/compile.dart:603:7)
#3      _RecompileRequest._run (package:flutter_tools/src/compile.dart:337:16)
#4      _CompilationRequest.run (package:flutter_tools/src/compile.dart:315:30)
#5      DefaultResidentCompiler._handleCompilationRequest (package:flutter_tools/src/compile.dart:635:23)
#6      _rootRunUnary (dart:async/zone.dart:1206:13)
#7      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#8      _CustomZone.runUnaryGuarded (dart:async/zone.dart:1005:7)
#9      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:357:11)
#10     _DelayedData.perform (dart:async/stream_impl.dart:611:14)
#11     _StreamImplEvents.handleNext (dart:async/stream_impl.dart:730:11)
#12     _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:687:7)
#13     _rootRun (dart:async/zone.dart:1182:47)
#14     _CustomZone.run (dart:async/zone.dart:1093:19)
#15     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
#16     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
#17     _rootRun (dart:async/zone.dart:1190:13)
#18     _CustomZone.run (dart:async/zone.dart:1093:19)
#19     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
#20     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
#21     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#22     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#23     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#24     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

[✓] Flutter (Channel unknown, 1.22.0-10.0.pre.307, on Mac OS X 10.14.6 18G103 x86_64, locale en-US)
    • Flutter version 1.22.0-10.0.pre.307 at /Users/flutter/.cocoon/flutter
    • Framework revision fce475bd0f (23 minutes ago), 2020-09-22 20:22:03 -0400
    • Engine revision c62d665299
    • Dart version 2.10.0 (build 2.10.0-156.0.dev)

[!] Android toolchain - develop for Android devices
    • Android SDK at /usr/local/share/android-sdk
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

    • ANDROID_HOME = /usr/local/share/android-sdk
    ✗ No valid Android SDK platforms found in /usr/local/share/android-sdk/platforms. Candidates were:
        - android-28
    • Try re-installing or updating your Android SDK,
      visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.8.4

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[!] VS Code (version 1.38.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • iPhone (mobile)  • b4873f14ffb248dfcb00c8cb7fdd4cdcff87f1f0 • ios            • iOS 12.4
    • macOS (desktop)  • macos                                    • darwin-x64     • Mac OS X 10.14.6 18G103 x86_64
    • Web Server (web) • web-server                               • web-javascript • Flutter Tools
    • Chrome (web)     • chrome                                   • web-javascript • Google Chrome 77.0.3865.90

! Doctor found issues in 3 categories.

Flutter Application Metadata
Type: null
Version: null
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: null
Creation framework version: null
Plugins
connectivity-0.4.9+2
connectivity_for_web-0.3.1+2
connectivity_macos-0.1.0+5
device_info-0.4.2+7
e2e-0.7.0
url_launcher-5.7.0
url_launcher_linux-0.0.1+1
url_launcher_macos-0.0.1+8
url_launcher_web-0.1.3+2
url_launcher_windows-0.0.1+1
video_player-0.10.6
video_player_web-0.1.2+3

Looks like we're missing a test case with an org-dartlang-app URI scheme, perhaps should switch to using URIs instead, but that would be a bigger change


bevy doc - an ECS project documentation generator

What?
I think it would be nice to have a tool that would be able to generate Bevy (ECS) oriented documentation from project source files.
Some things this documentation should (help) answer:

What are all the systems and Components in a project? (documentation, navigation)
Which systems (mutably?) access this Component/Event/Resource? (documentation, debugging)
What is the required-components dep tree for a Component? (documentation)
What plugin initialises this Event/Resource? (project structure docs, navigation, debugging)
What plugin does this system belong in? (project structure docs)
Where is X system for Y task? (navigation, documentation)
TODO: More ideas / curate the list

Some things this documentation alone is not intended for (some of these could be done by integrating with cargo doc and linking to there):

What are all the trait (incl. non-bevy) implementations on this type?
Bevy API docs
Documentation on types that are not ECS relevant or that carry semantics special to Bevy
TODO: more examples

Why?
Collecting the information about a bevy project involves many many declarations as well as many App method calls. (plugins (method bodies), Components (declarations and attributes), Resources (declarations, attributes and App registration), systems (function signatures and App registration), Events, and so on...)
Bevy projects are recomended to be highly modular and split up, which is in many regards good, but it means that all this information becomes spread out across many files, and worse (in my experience) it becomes dilute. (ex: you need to read the system signatures, but the system bodies are irrelevant to you.)
Even if it's reasonably easy to find what you need by following Type information breadcrumbs via go-to-reference and go-to-declaration, this involves a lot of jumping around, and in my experience does not lend itself to learning
How?
I don't know how difficult it would be to implement, I imagine the biggest challenge is reading and collecting all the required information about how the App is constructed (TODO: What if the user has no bevy_app, can we still learn useful information from things like Schedule::add_systems?)
Many other pieces of information should be relatively easy to parse (such as Component declarations and their attributes.)

I would love to see this! I believe someone made of a demo of this a few months back, it integrated with Rustdoc's metadata output directly.

You may be able to use Miri combined with an introspection plugin, but I've never done such a thing before.

As @IQuick143 mentioned on Discord, we could also have a CLI version of this. E.g.:
bevy list [components|systems|resources|events|plugins|etc..]

For systems this may be challenging:

"Normal" functions like fn foo() {} are valid systems, so you need to look for .add_systems specifically.
.add_systems calls may be hidden behind feature flags, dead code, plugins that are conditionally added, etc. at runtime. It's probably fine to assume that xyz is a system if it's in an .add_systems call, even if that call never runs, though.
Function pointers and closures can be added as systems.
Observers and one-shot systems are still "systems".

I agree that systems are rather challenging.
Considering anything that goes into .add_systems() to be a system seems good to me, although it does have drawbacks, and it's true that we can't always decide what system goes into what schedule (although I have not seen anyone write code like that pathological example).
I think we just need to add some sort of override/hint system, so that users can explain their pathological examples if they do occur.
Observers and one-shot systems could IMO get a different section in the docs, but documenting these would also be important.


Different behaviour using prefix and suffix FILTER

My Environment

ArangoDB Version:        3.4.5
Storage Engine:             RocksDB
Deployment Mode:       Single Server
Deployment Strategy:   Manual Start in Docker
Configuration:
Infrastructure:               own
Operating System:
Total RAM in your machine:        32gb
Disks in use:        HDD
Used Package:

Component, Query & Data
Affected feature:
AQL
AQL query (if applicable):
LET z = (
    FOR i IN phone_number
        FILTER i._key LIKE "1%"
        //FILTER i._key LIKE "%1"
        FOR v,e IN 1..1 INBOUND i phone_call
            FILTER v.height > 160
            FILTER e.duration > 300
            RETURN e._id
)

RETURN z
AQL explain (prefix):
Query String:
 LET z = (
     FOR i IN phone_number
         FILTER i._key LIKE "1%"
         FOR v,e IN 1..1 INBOUND i phone_call
             FILTER v.height > 160
             FILTER e.duration > 300
             RETURN e._id
 )
 RETURN z

Execution plan:
 Id   NodeType                  Calls     Items   Runtime [s]   Comment
  1   SingletonNode                 1         1       0.00001   * ROOT
 13   SubqueryNode                  1         1       3.02448     - LET z = ...   /* const subquery */
  2   SingletonNode                 1         1       0.00001       * ROOT
  3   EnumerateCollectionNode    1000   1000000       0.73762         - FOR i IN phone_number   /* full collection scan */
  4   CalculationNode            1000   1000000       0.89147           - LET #6 = LIKE(i.`_key`, "1%")   /* simple expression */   /* collections used: i : phone_number */
  5   FilterNode                  112    111111       0.33341           - FILTER #6
  6   TraversalNode                 1         0       1.06189           - FOR v  /* vertex */, e  /* edge */ IN 1..1  /* min..maxPathDepth */ INBOUND i /* startnode */  phone_call
  7   CalculationNode               1         0       0.00001             - LET #10 = ((v.`height` > 160) && (e.`duration` > 300))   /* simple expression */
  8   FilterNode                    1         0       0.00001             - FILTER #10
 11   CalculationNode               1         0       0.00001             - LET #14 = e.`_id`   /* attribute expression */
 12   ReturnNode                    1         0       0.00001             - RETURN #14
 14   ReturnNode                    1         1       0.00001     - RETURN z

Indexes used:
 By   Type   Collection   Unique   Sparse   Selectivity   Fields      Ranges
  6   edge   phone_call   false    false            n/a   [ `_to` ]   base INBOUND

Traversals on graphs:
 Id  Depth  Vertex collections  Edge collections  Options                                  Filter / Prune Conditions
 6   1..1                       phone_call        uniqueVertices: none, uniqueEdges: path

Optimization rules applied:
 Id   RuleName
  1   move-calculations-up
  2   move-filters-up
  3   move-calculations-up-2
  4   move-filters-up-2
  5   fuse-filters

Query Statistics:
 Writes Exec   Writes Ign   Scan Full   Scan Index   Filtered   Exec Time [s]
           0            0     1000000            0     888889         3.04093

Query Profile:
 Query Stage           Duration [s]
 initializing               0.00003
 parsing                    0.00019
 optimizing ast             0.00004
 loading collections        0.00004
 instantiating plan         0.00013
 optimizing plan            0.00072
 executing                  3.02468
 finalizing                 0.01498

AQL explain (prefix):
Query String:
 LET z = (
     FOR i IN phone_number
         FILTER i._key LIKE "%1"
         FOR v,e IN 1..1 INBOUND i phone_call
             FILTER v.height > 160
             FILTER e.duration > 300
             RETURN e._id
 )
 RETURN z

Execution plan:
 Id   NodeType                  Calls     Items   Runtime [s]   Comment
  1   SingletonNode                 1         1       0.00001   * ROOT
 13   SubqueryNode                  1         1      58.33171     - LET z = ...   /* const subquery */
  2   SingletonNode                 1         1       0.00001       * ROOT
  3   EnumerateCollectionNode    1000   1000000       0.64415         - FOR i IN phone_number   /* full collection scan */
  4   CalculationNode            1000   1000000       1.52598           - LET #6 = LIKE(i.`_key`, "%1")   /* simple expression */   /* collections used: i : phone_number */
  5   FilterNode                  100    100000       0.32020           - FILTER #6
  6   TraversalNode              1999   1998806      51.85046           - FOR v  /* vertex */, e  /* edge */ IN 1..1  /* min..maxPathDepth */ INBOUND i /* startnode */  phone_call
  7   CalculationNode            1999   1998806       1.60681             - LET #10 = ((v.`height` > 160) && (e.`duration` > 300))   /* simple expression */
  8   FilterNode                  966    965937       1.23444             - FILTER #10
 11   CalculationNode             966    965937       0.90494             - LET #14 = e.`_id`   /* attribute expression */
 12   ReturnNode                  966    965937       0.23455             - RETURN #14
 14   ReturnNode                    1         1       0.00002     - RETURN z

Indexes used:
 By   Type   Collection   Unique   Sparse   Selectivity   Fields      Ranges
  6   edge   phone_call   false    false            n/a   [ `_to` ]   base INBOUND

Traversals on graphs:
 Id  Depth  Vertex collections  Edge collections  Options                                  Filter / Prune Conditions
 6   1..1                       phone_call        uniqueVertices: none, uniqueEdges: path

Optimization rules applied:
 Id   RuleName
  1   move-calculations-up
  2   move-filters-up
  3   move-calculations-up-2
  4   move-filters-up-2
  5   fuse-filters

Query Statistics:
 Writes Exec   Writes Ign   Scan Full   Scan Index   Filtered   Exec Time [s]
           0            0     1000000      1998806    1932869        58.57918

Query Profile:
 Query Stage           Duration [s]
 initializing               0.00003
 parsing                    0.00017
 optimizing ast             0.00004
 loading collections        0.00004
 instantiating plan         0.00014
 optimizing plan            0.00065
 executing                 58.53492
 finalizing                 0.04297

Dataset:
Simple model 2 vertices, 1 edge collection. V(person) -> E(phone_call) -> V(phone_number)
Problem:
When doing first FILTER in FOR loop, doing both prefix and suffix behave differently. Prefix query will not reach traversal part, suffix one does.
Expected result:
FILTER should not influence behaviour of following blocks when both filters return some number of results.

not sure if I understand your problem. AQL works much like a pipeline, if FILTER i._key LIKE "1%" does not evaluate to true for any documents in phone_number then the traversal will not have any documents to start from.
Should the below AQL query return any results ?
FOR i IN phone_number
    FILTER i._key LIKE "1%"
    RETURN i

Yes, that is correct. In one case filter returns 100k documents, in other something more than 111k. But in one case it will follow with FOR and in other it will not.

this does still happen if you remove the subquery? Can you send us some testdata to reproduce it?

If we remove subquery it will return properly both prefix and suffix results. As keys are sequential numbers we have roughly 10% of all documents having prefix / suffix of digit '1'.
However, as soon as we do subquery, prefix will not return any results and suffix will. Here are simplified queries.
Prefix
LET z = (
    FOR i IN phone_number
        FILTER i._key LIKE "1%"
        FOR v,e IN 1..1 INBOUND i phone_call
            FILTER v.height > 160
            FILTER e.duration > 300
            RETURN e._id
)
RETURN z

Suffix
LET z = (
    FOR i IN phone_number
        FILTER i._key LIKE "%1"
        FOR v,e IN 1..1 INBOUND i phone_call
            FILTER v.height > 160
            FILTER e.duration > 300
            RETURN e._id
)
RETURN z

Dataset we have is pretty much 100k phone numbers randomly doing 20milion calls - vertex on the other side is person (vertex collection of 1milion records)

I can't reproduce it unfortunately, could you send over some test data ?

Hi @rackom,
Since we were not able to reproduce this issue, could you please share a data set?
You can send us a message to<EMAIL_ADDRESS>(this ML is not public) and attach the dump in that email. This way we can try to reproduce and find the root cause. Please reference the number of this issue in your email.

@rackom
I am closing this issue for now since we have not been able to reproduce it.
In case this problem persists, please send us some test data set as described above and we would be glad to investigate it.


feat: using tmjson encoding on core.Header

Overview
Closes #1476.
We now also use tendermint's custom json encoding for core.Header. @preston-evans98 can you verify this PR achieves what you were looking for?

Overview
Closes #1476. We now also use tendermint's custom json encoding for core.Header. @preston-evans98 can you verify this PR achieves what you were looking for?

@distractedm1nd this looks great! I've tested against tendermint-rs and header deserialization now works without modification. Thanks!

Hope we can avoid depending on tmjson in favour of std, so we can reverse node to app dependency


[UX] Formulas in Crafting should have an "Add to Advanced Alchemy" button

Reasoning
When you're an alchemist, the known formulas section is very big. It's difficult to click a formula from known formulas, drag it, and scroll to the correct section. It would be much easier to handle if there were a button for each formula if you have the advanced alchemy feature that would add it to the list. I suspect it would be next to the "Craft" button, maybe an up arrow or a little potion bottle?
Screenshot of the UI I'm talking about:

There was a similar button, but it added to the first valid entry rather than a selected entry and thus was useless for this purpose.
This will likely become a popout in the future during the inevitable redesign.

So if I worked this feature to add that button back, it wouldn't be welcome?

It would not. That button was removed because it was awkward to update during all of the internal refactors, and it does not work with multiple crafting abilities very well at all.
If you want to try anything to make this easier, you can try the new sorting paradigm in the quick alchemy branch, where we sort in reverse level order, so that higher level formulas are at the top and closer to the crafting entries.

You're saying if I PRed a reverse level sort in formulas you'd be open to that? Do you want a button to switch asc/desc?
Also if I can help with the other crafting stuff I'd be happy to do that.

Another option is allowing items to be dropped into the advanced alchemy section from the compendium, is that also unreasonable?

No toggle, descending all the time. R.sortBy(something, (f) => -f.item.level) will do it. You might have to do a conversion from a record to a list of level/formulas objects. Remember that all of this needs redesigns anyways.
No on dropping from compendium. Any sort of drop from compendium has the risk of hitting the wrong dropzone and adding the item to the inventory or something. We don't want to risk that.

If I get some time tonight I'll take a look. Feel free to close this as wontfix.

Okay reverse formula sort doesn't really get me there either, it's totally impossible to prepare crafts on a touchpad. How would you feel about a favorite button as an intermediary? Something to push a few up to the top.

@CarlosFdez Okay reverse formula sort doesn't really get me there either, it's totally impossible to prepare crafts on a touchpad. How would you feel about a favorite button as an intermediary? Something to push a few up to the top.

Does this same issue exist for preparing spells?
I would prefer not introducing a new feature as an intermediary, especially when the final version would end up removing it. People get too attached to such things.

It doesn't for preparing spells because they have favorites and searching.

Closing this since it won't be done.


Fixing setting line trace name on example

It was calling: 'set trace' instead of 'set linetrace'.

Thanks!

:+1:


[L2Geth] Add reverted reason in the returned payload

Add reverted reasons for turing txs.

Codecov Report

Merging #433 (54650ae) into develop (a953d6c) will increase coverage by 13.56%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #433       +/-   ##
============================================
+ Coverage    73.61%   87.18%   +13.56%
============================================
  Files           59       34       -25
  Lines         1789     1116      -673
  Branches       272      160      -112
============================================
- Hits          1317      973      -344
+ Misses         472      143      -329

Flag
Coverage Δ

contracts
87.18% <ø> (ø)

core-utils
?

data-transport-layer
?

message-relayer
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/core-utils/src/common/hex-strings.ts

packages/data-transport-layer/src/utils/eth-tx.ts

packages/core-utils/src/common/index.ts

...ckages/data-transport-layer/src/utils/contracts.ts

...ices/l1-ingestion/handlers/transaction-enqueued.ts

...kages/data-transport-layer/src/utils/validation.ts

packages/core-utils/src/batches.ts

.../src/services/l2-ingestion/handlers/transaction.ts

packages/core-utils/src/common/test-utils.ts

packages/core-utils/src/coders/sequencer-batch.ts

... and 15 more

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 a953d6c...54650ae. Read the comment docs.

The revert reason should come from the helper contract instead of l2geth.


Create test suite

Begins to adds tests and sets up a quite extensively run testing workflow, as this is an important file so we need to know it works across versions and operating systems.

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Vbproj support?

Do you plan to support vb projects?
There should not be a lot of difference I guess..

Can you send me a couple of example vbproj files? I don't think there's a
big difference no it's just that I never work with it so I don't have a
good set of test projects.
On Sun, Apr 1, 2018, 11:07 Boris Prpic<EMAIL_ADDRESS>wrote:

Do you plan to support vb projects?
There should not be a lot of difference I guess..
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/74, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AD8FPHSMol6_oWWZpUfGlYbPzxibvHjxks5tkRdjgaJpZM4TC6rL
.

Can you check:
https://github.com/hvanbakel/CsprojToVs2017/releases/tag/1.5.12-beta
let me know what you find.

Unhandled Exception: System.IO.IOException: The directory name is invalid.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileSystemEnumerableIterator1.CommonInit()    at System.IO.FileSystemEnumerableIterator1..ctor(String path, String originalUserPath, String searchPattern, SearchOption searchOption, SearchResultHandler`1 resultHandler, Boolean checkHost)
at System.IO.Directory.EnumerateFiles(String path, String searchPattern, SearchOption searchOption)
at Project2015To2017.ProjectConverter.d__1.MoveNext()
at Project2015To2017.Console.Program.Main(String[] args)

Can you try with an individual project file first? I assume this was with
either a folder or a solution.
On Sun, Apr 1, 2018, 22:27 Boris Prpic<EMAIL_ADDRESS>wrote:

Unhandled Exception: System.IO.IOException: The directory name is invalid.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileSystemEnumerableIterator1.CommonInit() at
System.IO.FileSystemEnumerableIterator1..ctor(String path, String
originalUserPath, String searchPattern, SearchOption searchOption,
SearchResultHandler`1 resultHandler, Boolean checkHost)
at System.IO.Directory.EnumerateFiles(String path, String searchPattern,
SearchOption searchOption)
at Project2015To2017.ProjectConverter.d__1.MoveNext()
at Project2015To2017.Console.Program.Main(String[] args)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/74#issuecomment-377858044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD8FPKdV2d2efIsMvMjtHtORjgFqbQzfks5tkba3gaJpZM4TC6rL
.

No, I tried loading a project.

I think it's the first switcher in ProjectConvertor. It could do with changing to test if the provided path is a file and then check against a list of allowed types. The error messages can then be better if somebody did actually pass in an unsupported file type.

There's a new release here:
https://github.com/hvanbakel/CsprojToVs2017/releases/tag/1.5.12-beta01
If you get an exception, could you include the entire console output?

@hvanbakel why not merge VBProj-support branch into master, despite not thoroughly done testing? It will not affect the main purpose (csproj) but will simplify possible future work on vbproj & fsproj.

@hvanbakel bump the issue. Probably won't hurt to add rebased changes to the new master.


Allow path collaborators and course assistants to run analysis

Users can run custom analysis on -

Paths -
a. Created
b. Collaborated
Courses -
a. Created
b. Assisted
#942

Updated the way the custom analysis and admin custom analysis responses are stored -

Custom Analysis -
a. CustomAnalysisResponse - Node where the responses are written. Document ID is the analysis ID, if present it is updated otherwise new document is created.
b. logged_events - Reference to customAnalysis/Response is stored in otherActionData

Admin Custom Analysis -
a. adminCustomAnalysisResponse - Node where the responses are written. Document ID is the admin analysis ID, if present it is updated otherwise new document is created.
b. logged_events - Reference to adminCustomAnalysis/Response is stored in otherActionData


Deleting events containing single quote (apostrophe)

Hi, I'm trying to delete a calendar item called "dog's birthday" - but I am having no luck in finding the right syntax to quote the apostrophe.  Can anyone suggest what might work?
$~ $ gcalcli agenda
No handlers could be found for logger "oauth2client.util"

Thu Oct 19         Forecast for cb3 0wa (18° | 11°)
            14:30  dog's birthday

$ gcalcli delete "dog's birthday"
No handlers could be found for logger "oauth2client.util"
No Events Found...

$ gcalcli delete """dog's birthday"""
No handlers could be found for logger "oauth2client.util"

No Events Found...

$ gcalcli add --allday
Title: dog's birthday
Location: everywhere
When: tomorrow
Duration (days): 1
Description: test
Enter a valid reminder or '.' to end: .
$ gcalcli agenda

Sun Aug 05           dog's birthday
             7:00am  test
            12:00pm  🎈unicode test š x̌ ʔ ə

Wed Aug 08   4:00pm  salas

$ gcalcli delete "dog's birthday"

2018-08-05           dog's birthday
Delete? [N]o [y]es [q]uit: y
Deleted!

I can't repo this.  I tested gcalcli 3.4 on python2.7 and I tested on python3 on current master.  Please reopen this issue if you're still having it with details about your platform and environment which may help us reproduce.
I suspect that those messages about "No handlers found for oauth2client.util" are actually more related to what's going on for you.

ok, thanks for checking!


The enum filter interface has the wrong types for "in" & "not_in"

The in and not_in properties of the _enum filter interface currently are both typed as [any] when they should be Array<any>
I made a PR to fix this: https://github.com/chargebee/chargebee-typescript/pull/4

Updated in master branch and will be available from v2.0.4


docs: Small edits for spelling, grammar, formatting.

This is a redo of #711 :)

@Zearin thanks a lot for this huge effort. i'm bound to ruin it over time :D
anyhow, just got one small fix needed and than (or then?) i'll merge it.

@Zearin thanks a lot. merging


Add withResolvers

[!TIP]
The owner of this PR can publish a preview release by commenting /publish in this PR. Afterwards, anyone can try it out by running pnpm add radashi@pr<PR_NUMBER>.

Summary
Not sure if that one should be public util or radashi internal one.
Related issue, if any:

For any code change,

[x] Related documentation has been updated, if needed
[x] Related tests have been added or updated, if needed
[x] Related benchmarks have been added or updated, if needed

Does this PR introduce a breaking change?

No

Bundle impact

Calculating...

Not sure if that one should be public util or radashi internal one.

All functions in Radashi are public. It's one of our core principles currently. 😉

Apologies for the delay. Been working on the Radashi website and its VSCode extension :)
I'm gonna use this PR to debug the "Bundle Impact" calculation (it shouldn't be an empty table here), but after that we can merge!


Making Date as blank instead of '00.00.0000' in CSV export

Hi team,
When using ZCL_EXCEL_WRITER_CSV (with latest changes), date when blank is displayed as 00.00.0000 in AL11 CSV file.
Is there any way to make the date blank?
Edit:
For the time being in my system, I changed the following code in method CREATE_CSV (LINE 152) of class ZCL_EXCEL_WRITER_CSV:
CASE <fs_sheet_content>-data_type.

        WHEN 'd' OR 'D'.
          If <fs_sheet_content>-cell_value IS NOT INITIAL. **Added by me
            lc_value = zcl_excel_common=>excel_string_to_date( ip_value = <fs_sheet_content>-cell_value ).
            TRY.
                lv_date = lc_value.
                CALL FUNCTION 'CONVERT_DATE_TO_EXTERNAL'
                  EXPORTING
                    date_internal            = lv_date
                  IMPORTING
                    date_external            = lv_tmp
                  EXCEPTIONS
                    date_internal_is_invalid = 1
                    OTHERS                   = 2.
                IF sy-subrc = 0.
                  lc_value = lv_tmp.
                ENDIF.

              CATCH cx_sy_conversion_no_number.

            ENDTRY.
       ENDIF.
`

I understand it may not work for all because some may want to display date as 00.00.0000.
Maybe we can add a flag to toggle whether to display or make it blank.

Thanks for the information. Do you mind proposing a pull request? I suggest to create a method like others for setting CSV options, with "00.00.0000" as default, and adapt the CSV demo program for options zdemo_excel28 to demonstrate the new method.

Hi Sandra,
I have created a pull request:
https://github.com/abap2xlsx/abap2xlsx/pull/1190

I have also created pull request for ZDEMO_EXCEL28.
https://github.com/abap2xlsx/demos/pull/53

Hi Sandra,
Thanks for accepting my pull requests.
Though, I have one doubt regarding this modification.
It is because both this line of code and the one before it produces the same output.
Maybe you meant to replace the previous line?

I guess the use case is one variable (second line below), not one space (first line below).
https://github.com/abap2xlsx/demos/blob/e9b8f04322348e89931be9cff11394f1d5dc409b/src/zdemo_excel28.prog.abap#L57-L58
  lo_worksheet->set_cell( ip_column = 'B' ip_row = 4 ip_value = space ip_abap_type = 'D' ).
  lo_worksheet->set_cell( ip_column = 'C' ip_row = 4 ip_value = c_initial_date ).

To simplify, would you mind deleting the first line of code?

Hi Sandra,
I have created pull request https://github.com/abap2xlsx/demos/pull/58

Thanks, merged!

Thank you, Sandra, for your help!
Closing the issue.


Shorcodes: Check for .Inner in linked partial templates

Hi!
I am using some partials that can be accessed via shortcodes or from template files. The shortcode shows an error message if no .Inner is provided although .Inner is used in the partial template.
That is probably the intended behaviour. But maybe it is possible that Hugo checks for .Inner in the linked partial file(s)?
This is a minimal example for a figure shortcode:
{{ $inner := .Inner }}{{/* This is necessary although var is not used */}}
{{ if in (.Get "src")  "svg" }}
  …
{{ else }}
  {{ partial "figure.html" . }}
{{ end }}

Hi,
I came across this quirk today. To better illustrate, I will expand the example a little bit:
layouts/shortcodes/usespartial.html:
{{ partial "mypartial" . }}

layouts/partials/mypartial.html:
{{ .Inner }}

content/_index.md:
{{< usespartial >}}
Hello World!
{{< /usespartial >}}

The {{ $inner := .Inner }} trick is a great workaround, I only thought of <!-- {{ .Inner }} ->> which is obviously subpar.

It's hard to say if it's a bug. The fact that Hugo checks usespartial.html (in my example) for the use of .Inner is definitely a good feature. The algorithm could be made even smarter by checking in sub-partials (mypartial.html), but it may also cause too much overhead? Otherwise a small note in the docs would be good.

It's hard to say if it's a bug. The fact that Hugo checks usespartial.html (in my example) for the use of .Inner is definitely a good feature. The algorithm could be made even smarter by checking in sub-partials (mypartial.html), but it may also cause too much overhead? Otherwise a small note in the docs (to use the {{ $inner = .Inner }} trick) would be good.

While this would be possible to implement, it's not something I'm spending 5 hours on my free time to do. It's also not something that I would consider fixing unless the fix itself was simple.
The workaround is simple: Just do a $notused := .Inner in the shortcode.

This can be closed imho.