Inline ImageMagick logic

Invoking the shell from code is scary and one bad POST could lead to injection, so i replaced current logic with the gographics/imagick bindings to C
this is also probably great for resource usage since we don't spawn a new sh instance on each invocation, and it's nice and memory safe and fast
i was able to skip some steps like flattening since the logic for adding a background was simply replacing alpha 0 pixels with white ones
this requires the imagemagick dev headers to be present to compile

I tried to read a bit of what you wrote, I struggle a bit since I never used the magick wand API
I have a few questions

How does your transparency logic work exactly ? does it just look for transparent pixels and overwrite them with #FFFFFF ? what if a pixel is only like half transparent or something ?
at which point do you set the PNG to have indexed colors ? the main point of using imagemagick in the first place was to limit the amount of colors and have indexed colors so that the image is as small as possible

essentially, the C API had issues with quantizing (or limiting the number of colours) and would result in a blank canvas. i figured we didn't need it, since the base64 encoded image only has the 4 colours regardless. the base64 strings inside the DB seem to be of similar sizes but id be happy to re-test later today.
The transparency logic looks for all pixels with rgba value 0,0,0,0.0. flatten essentially does the same thing but I had issues getting imagemagick to set the background colour, which is essentially what this does. no worries about semi transparency.

what if a user use console command magic to send arbitrary images instead of drawing them ? this is totally possible (and something that i'm not against) like shown in this thread https://paperchan.club/thread/68
thats why i'm worrying about semi transparency and indexed colours that dithering is what I want to happen in that case
this was good work tho I did learn a few things reading your code, maybe I'll try to play around with your thing this afternoon and if I don't manage to get the exact behavior i want I'll still modify my code to use golang's base64 lib and pipe to convert directly instead of invoking a shell this does seem like a better idea

had time to investigate, and it looks like there are actually savings based on my method even without quantizing, i see savings of up to 100 chars on complex drawings (maybe due to using point filter on resize?)
sorry for such quick response. i was typing when you responded lol!
i can think of a few methods to prevent this kind of thing, i wasn't sure if it was sanitized but i'd be happy to implement one
we could either use some kind of background token auth that is sent along with the POST request and validate it serverside
or we could do an inverse opaque fill for all non red blue black and white colours, but would still allow for something drawn outside of the canvas to be sent.
let me know, i'd be happy to work on it.

yea I don't actually want to lock people out of hacking the canvas or changing the colors or anything, my vision for paperchan is to have a rather minimalist base implementation and an ecosystem of unofficial third party userscripts where people can unleash their creativity
that's why I want the only hardcoded limitations to be rather simple, 400x200 and limited amount of colors (cause i like dithered images)

dithering is definitely an option, it's provided in quantizeimage but it was a continual issue during my implementation. here are the docs on it, it offers several dithering options.
https://pkg.go.dev/gopkg.in/gographics/imagick.v2/imagick#MagickWand.QuantizeImage

https://imagemagick.org/Usage/quantize/#quantize_not_exact
the issue lies here, blacks and whites are not preserved on quantization and that messes something up. quantizing and including red or blue will work fine. will update you when i have this figured out

I got this 11.4k image down to 2.6k with some changes without quantization, the two are identical and all semi transparent pixels are still there. i am not much of an artist so you will need to see what happens to a good looking piece of art lol

I tried to play around with QuantizeImage and got pretty good results with this
err = magick.QuantizeImage(4,imagick.COLORSPACE_UNDEFINED,0,imagick.DITHER_METHOD_RIEMERSMA, false)
seem to work as well as my shell madness

turns out my issue all along was using the wrong colorspace. it absolutely refused to use RGB, sRGB, or anything at all other than undefined. added this change, whole pipeline should lead to a better compressed and quantized result.

image on the left is floyd steinberg which i think looks much better than riemersma. sorry about it being an image of thatcher it is all i had in directory (i do not like her). will make minimal difference with most drawings i am sure but worth considering

it does seem to change colors a bit tho

first one is quantize with 4 colors second I tried 6 both have 3 faces with the 3 default colors

stick with what we have then ! what needs to change before merge? as it stands i am scared somebody could escape the base64 command and do ACE on your server

even if someone manage to hack the server there is nothing but paperchan on that machine, I'm a mad man but not crazy enough to run something as janky as that on a machine with sensitive stuff
I'm still playing around with stuff i might settle for QuantizeImage(8,imagick.COLORSPACE_UNDEFINED,0,imagick.DITHER_METHOD_NO,false)

yea i think 8 colors with DITHER_METHOD_NO will be fine for now we'll tweak stuff later

can't compile even tho i installed libmagickwand, debian being debian again it seems 😭😭

do you have V7 version of imagemagick headers? you need libmagickwand-dev afaik

also on debian so can vouch for it, works on my remote ubuntu server too

i r slash love debian when it works (50% of the time if it feels like it)
flatpak save me save me flatpak

am currently compile imagemagick from source like a psychopath 😭😭😭
on a 1$ server it's not fast 😭😭

ok server is online now 😭😭

apologies 4 downtime !


datasets.load_dataset("code_search_net", "python") : NotADirectoryError: [Errno 20] Not a directory

Describe the bug
While checking out the tokenizer tutorial, i noticed getting an error while initially downloading the python dataset used in the examples.
The collab with the error is here
from datasets import load_dataset
import os

os.environ["HF_DATASETS_CACHE"] = "/workspace"

# This can take a few minutes to load, so grab a coffee or tea while you wait!
raw_datasets = load_dataset("code_search_net", "python")

yeilds:
ile /opt/conda/lib/python3.10/site-packages/datasets/download/streaming_download_manager.py:524, in xlistdir(path, use_auth_token)
    522 main_hop, *rest_hops = _as_str(path).split("::")
    523 if is_local_path(main_hop):
--> 524     return os.listdir(path)
    525 else:
    526     # globbing inside a zip in a private repo requires authentication
    527     if not rest_hops and (main_hop.startswith("http://") or main_hop.startswith("https://")):

NotADirectoryError: [Errno 20] Not a directory: '/workspace/downloads/25ceeb4c25ab737d688bd56ea92bfbb1f199fe572470456cf2d675479f342ac7/python/final/jsonl/train'

I was able to reproduce this erro both in the collab and on my own pytorch/pytorch container pulled from the dockerhub official pytorch image, so i think it may be a server side thing.
Steps to reproduce the bug
Steps to reproduce the issue:

run raw_datasets = load_dataset("code_search_net", "python")

Expected behavior
expect the code to not exception during dataset pull.
Environment info
i tried both the default HF_DATASETS_CACHE on Collab, and on my local container.  i then pointed to the HF_DATASETS_CACHE to a  large capacity local storage and the problem was consisten across all 3 scenarios.

Note:
I listed the datasets and grepped around to find what appears to be an alternative source for this:
raw_datasets = load_dataset("espejelomar/code_search_net_python_10000_examples", "python")

Thanks for reporting, @jason-brian-anderson.
Yes, this is a known issue: the CodeSearchNet repo has been archived (Apr 11, 2023) and their source data files are no longer accessible in their S3: e.g. https://s3.amazonaws.com/code-search-net/CodeSearchNet/v2/java.zip gives 403 Forbidden error. See:

https://huggingface.co/datasets/code_search_net/discussions/3

We have contacted the authors of the dataset to find a solution. I'll keep you informed.

cc: @julianeagu

This issue is fixed because we are hosting the CodeSearchNet data files in the Hugging Face Hub. See: https://huggingface.co/datasets/code_search_net/discussions/7

I learned that @mallamanis has uploaded the dataset here as well

Thanks @hamelsmu for the Zenodo link. I am adding it to the dataset card on the Hugging Face Hub, so that the community knows about this "official" source data. I guess this link is not well known, because some community members already hosted an "unofficial" version of the data on Zenodo as well: https://zenodo.org/record/7857872


mongoengine partial update document

How can I update the document partially from view?
I used partial_update() SQL databases such as PostGreSQL, MySQL
but I am not sure how can I update mongoDB document with several properties.
i.e when mongo query is like below, how can I write in Django rest framework view.
db.products.update(
{ _id: 100 },
{ $set:
{
quantity: 500,
details: { model: "14Q3", make: "xyz" },
tags: [ "coats", "outerwear", "clothing" ]
}
}
)

I'm afraid, partial updates are not entirely stable. See test_partial.py for examples.

@BurkovBA no way to update document partially?


Is there a way to load extension that is linked statically?

Currently the docs always requires a dynamic library to e used in load_extension .
Is there a way to use https://www.sqlite.org/c3ref/auto_extension.html and pass only the entry point?

https://github.com/rusqlite/rusqlite/discussions/1125
https://github.com/rusqlite/rusqlite/issues/1305

Thanks you very much!


Memento

Hey
Sorry to bother you so soon already but i really got excited when i saw that you update memento and wanted to give it a spin.
So i installed everything on ubuntu to get it working and made a map inside memento and did python memento.py memes, then the script runs and when i enter 1 or any other number and get this message( see attachment) How can i fix this?
I don't think it has anything to do with tesseract, because i did a tesseract ocr on an image with an output to .txt and it extracted the data..
Have i missed installing a module or something?
Hope you can help me out :)

The input() is taking prompt as integer not string. Try executing using python3 instead of python. Else put choice = input().strip() instead of choice = input().lower().strip() in memento.py line 29. Let me know which one works

Hey man i finally got it to work! So sweet and it's so fast! Oh yeah and how could i make the extracted text to save a little longer, because now it doesn't write down the full sentence to the image file?


Documentation: Add slash key event to focus the input search.

I've been thinking about this too! Two things which should be fixed:

The input is also focused when typing in an input field or textarea
We should make it clearer we have this hotkey, like GitHub does

Moving to #31702.


Move to openstack?

Hi @dims
not sure if this was discussed, would it make sense to move this project to opendev aka openstack hosting infra ?

@apevec i'd be happy if anyone wants to do this!

@apevec @dims any update about this ?

@goldyfruit i am happy to let anyone who wants to drive this to move the repo. Haven't heard back from @apevec

I refreshed this topic in http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016457.html
Please let me know if you are still willing to move this project to the OpenStack's ecosystem.
I imagine etcd3gw would benefit from common testing (more details in mail).

Hi @dims - your last statement was more than a year ago and people change minds so I would like to clarify whether you are still interested in allowing this move. I can arrange this to happen. Oslo's PTL seems happy about taking ownership.

@yoctozepto i haven't changed my mind :) last statement above is still good.

Thanks, @dims. I got pretty busy early this week but will start making this happen later this week.

@dims - there is one thing that is required of you to do: "Give OpenDev Permission to Publish Releases" - https://docs.opendev.org/opendev/infra-manual/latest/creators.html#give-opendev-permission-to-publish-releases

@yoctozepto It is done!

Thanks @dims, I issued the following:
https://review.opendev.org/747185
https://review.opendev.org/747188
Now we wait. :-)

Thanks for taking care of this @yoctozepto . will make this repo read-only once those land.

Thanks @dims for being open to the proposal.
Yup, the original repo should go read-only and we should get some redirects to the new place. I'll do the same on my fork to minimise confusion.
As for the issues, I think we need a new place on launchpad as that's where Oslo is.


IMPORTANT! Security Vulnerability Detected in your Repository

Hey there!
We have detected a security vulnerability in your repository. Please contact us at https://github-scanner.com to get more information on how to fix this issue.
Best regards,
Github Security Team

meo stealer maracos culiaos
El mié, 18 sept 2024 a la(s) 4:00 p.m., hollanddominiqXuOBe (
@.***) escribió:

Hey there!
We have detected a security vulnerability in your repository. Please
contact us at https://github-scanner.com to get more information on how
to fix this issue.
Best regards,
Github Security Team
—
Reply to this email directly, view it on GitHub
https://github.com/Ignacivss/Streaming-RPC/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXN4GAIPFPYWH52IJLNMJF3ZXHETBAVCNFSM6AAAAABOOKVVFSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZTINBWGYZDIOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


fix(docs): photonjs generator

Hopefully clearer sentence structure to convey the information.

If merged, the change should also be applied to https://github.com/prisma/prisma2/blob/master/docs/core/generators/photonjs.md (and a note made that this should be deduplicated to not be necessary any more in the future)


Rewrite tutorial of react/universal-react-apps and fix typos

Please review and accept my pull request if you like the rewrite. Thanks!

Looks fine but it's not README but tutorial of react/universal-react-apps ?

You're right, my bad. I've changed the commit message and PR title to reflect the change more accurately.


[Modal] - Scrolling fullscreen modal adds a second scrollbar at the page level when resolution is too small

Steps to Reproduce

Create a fullscreen scrolling modal and open it. (Content has to be huge so the scrollbar appears)
Change your window size.
Another scrollbar appears at the page level(on the right of the window).

Expected
The Modal should resize correctly without adding another scroll bar
Result
The Modal seems to have a bad calculation for the margins.
Testcase
http://jsfiddle.net/kduju5dv/2/

Same thing applies when adding an overlong sidebar component.
The two scrollbars overlap each other. They look the same. And they behave in weird ways when using the mouse scroll wheel. Sometimes you'd be scrolling the sidebar, other times the whole page.


doc: add documentation

This PR proposes the documentation for the MQTT bridge. It contains the following informations:

How to run the Bridge
How to use the Bridge
How to configure the bridge.

Signed-off-by: Antonio Pedro<EMAIL_ADDRESS>

I have applied some changes, please check how it is looking now.

I think it should be fine now.
Thanks for the review.


How to add bootstrap?

Topic
Bootstrap removed from latest version
Description
Hi, the latest version removed bootstrap support. What's the "official" way to install it from now on? I know JS is broken and there are projects that provides Svelte components, but I just want the SCSS.
Thanks
Describe the solution you'd like
No response

For the time being, you can still use<EMAIL_ADDRESS>But keep in mind, there are multiple reasons why we have removed those adders. One of them is that both the bootstrap and bulma adders in their sass/scss versions were painfully slow during dev start, but also on later hot reloads.
Without trying it out by hand, this guide seems like it should work, if you have some basic knowledge about the kit vite integration.

Yeah I've noticed it too that build time is impacted.
I'm tempted to try Tailwind however I've been using bootstrap since forever and I'm also switching to Svelte for a new project so changing also the css library would mean basically throwing away everything I know about web development...
Thanks for your input, I'll do some research.

Totally understand that. Another point is the usage and support we have experiencend in the past. Here are the relevant download numbers for the three relevant libraries


Fix contour to break on spaces and handle new lines

Moving back to contour as it gives a more pleasing visual result than pdfrender for thick outlines. Added ability for contour to break across lines by splitting the string on spaces and newlines before applying contour. I will optimize it a bit, but I wanted to get opinions on the general strategy first.

@raphael-proust and @ravenclaw900, would you be able to take a look at this and see what you think? I temporarily changed the first chapter to the problematic one we were investigating.
\chapter[Personality and \newline Background]{Personality and ~ Background}

The text shows the "best" possible outcome as far as I can tell with the hanging indent for the chapter and the newline in the ToC. The following causes the second line of the chapter heading to start at the left edge of the text area instead of a hanging indent.
\chapter{Personality and \newline Background}

For some reason, I see no difference between the following:
\chapter[Personality and \newline Background]{Personality and ~ Background}
\chapter[Personality and \newline Background]{Personality and Background}
\chapter{Personality and \newline Background}
\chapter{Personality and Background}

There are other chapter headers for which I have similar issues. And for those (slightly different lengths) I can observe differences. But in both cases the following form is satisfying
\chapter[Something and \newline Blah]{Something and Blah}

I agree with @raphael-proust, the changes you've made let you make a \newline anywhere.
But, you do have to use \newline, not \\ or \linebreak or \break.  Other than that, using \newline, everything seems to work, and @raphael-proust's TOC break problem is fixed.

My understanding is that because of category code handling there is a limited set of what can be successfully detected. I think \\ comes up as extra white space and gets gobbled. \newline is the only thing I was able to get to work. Since it is only needed in certain situations, maybe we only need one way to do it?

For some reason, I see no difference between the following:
\chapter[Personality and \newline Background]{Personality and ~ Background}
\chapter[Personality and \newline Background]{Personality and Background}
\chapter{Personality and \newline Background}
\chapter{Personality and Background}

There are other chapter headers for which I have similar issues. And for those (slightly different lengths) I can observe differences. But in both cases the following form is satisfying
\chapter[Something and \newline Blah]{Something and Blah}

Really? I get 4 different results.

For the first one, all I get is a \newline in the TOC, for the second I get the same as the first, for the third, I get a \newline in the chapter, and for the fourth I get nothing.

[…]

Really? I get 4 different results.

Oh actually number 3 has one difference: in the chapter header, the new line starts on the left rather than with the indent that you'd normally expect.
Note that I have \renewcommand\tocchapterabbreviationname{Chapter} defined which changes the line length which may affect the line breaking/wrapping.

The \renewcommand\tocchapterabbreviationname{Chapter} doesn't affect the chapter heading,  just the table of contents.

[…]

Really? I get 4 different results.

Oh actually number 3 has one difference: in the chapter header, the new line starts on the left rather than with the indent that you'd normally expect.
Note that I have \renewcommand\tocchapterabbreviationname{Chapter} defined which changes the line length which may affect the line breaking/wrapping.

Ah, that makes more sense, here is what I see:
\chapter[Personality and \newline Background]{Personality and ~ Background}

\chapter[Personality and \newline Background]{Personality and Background}

\chapter{Personality and \newline Background}

\chapter{Personality and Background}

Everything I see is the same as you, except for the last one.  I see the chapter heading as one line, it doesn't automatically break.

Everything I see is the same as you, except for the last one. I see the chapter heading as one line, it doesn't automatically break.

Oops, I copied and pasted the wrong image (trying to save on uploads).

Ah, so we see the same things?  Ok.


test: Unit-Tests for scheduler package

We need tests for the whole scheduler package.
Please use the standard test framework for this: https://golang.org/pkg/testing/

Fixed by #39


#Depend on your need, you may change parameters in config/config.yaml.

Hi and good day. May i know how can we change the parameters in config/config.yaml which is yaml file ?

This file:
https://github.com/felixchenfy/Realtime-Action-Recognition/blob/master/config/config.yaml
There are many parameters inside it, which define the data source, input/output files and some other parameters.


Fix memory leak while creating NSEvent and swapping buffers (Fixes #514)

@tomaka what do you think?

I have no opinion on that. I don't know how it works!

Looks good to me.
@tomaka @paulrouget For future reference: In Objective-C, objects can be memory-managed in two ways: (a) the reference count is managed explicitly by the runtime and the object is destroyed when it reaches 0; (b) the object is placed in a per-thread arena (NSAutoreleasePool) and the arena is emptied at the end of each turn of the event loop. Ordinarily, Cocoa manages this for you, because the built-in NSRunLoop that NSApplicationMain() pumps (I believe it's NSApplicationMain(); might be some other function) automatically sets up an autorelease pool and flushes it after every event. But if you're interacting with Objective-C objects from another thread, or you're pumping the event loop manually, you have to set up and manage an autorelease pool yourself. If you don't, objects that were allocated via method (b) will leak. Glutin isn't doing this, so lots of objects are leaking; this patch fixes this.

r? @tomaka or @paulrouget (see explanation above)

r+


Responsiveness

Issue at Book a Cab or Auto
I can fix this issue

@dushyantkanav Thanks for creating the issue in BuddyTrail!!! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Contributors will start at Level 1, and their level will increase based on the quality and quantity of their contributions. The more effort and value you bring, the faster you'll progress. Put forth your best effort to unlock new levels and opportunities!
I'm excited to see your contribution as part of GSSOC EXTENDED 2024! 🎉
Don't forget to follow me on GitHub for more updates.
Happy open-source contributing! 🚀

If this issue is not yet resolved I would like to work on it please assign me this @PriyaGhosal


Alter namespace where the Push of Operator sends the image generated

Hi staff,
I would like to know if the option of configuration a kamel run I set the namespace where the image that generated by Operator will be pushed in some other namespace of registry?
ex:
kubernete namespace: my-camel-testes
registry:http://site.com/

and after the build o Operator execute:

... Pushing image site.com/my-camel-testes/camel-k-kit-ci6abnpcn3sd9fqa19b0:261541652 ...

But I need that this image "camel-k-kit-ci6abnpcn3sd9fqa19b0:261541652" is push for "site.com/project-x/camel-k-kit-ci6abnpcn3sd9fqa19b0:261541652"
There is to change "my-camel-testes" to "project-x" without me have that to change the namespace of kubernetes?
I read all --traits configuration and I did not find a response for this.

Hello,
What you are missing is the organization part of the registry configuration. It is not available in traits but as integration platform configuration : CONFIGURING REGISTRY
.

Hummm...
The IT area misconfigured even the registration part. I saw today that it has the registry setting.
Thansk @gansheer


Audio Channels in VLC_Control

Hey Guys :
im in a bad Trouble. i have Some Video Files that have 4 or 6 Audio Channels.
but how can i detect each audio channel level????
i wanna design Audio bar for each channel .
tnQ for you answers.....

Try this code..., otherwise you could always use MediaInfo, just ask me for a sample.
var mediaInformations = myVlcControl.GetCurrentMedia().TracksInformations;
foreach (var mediaInformation in mediaInformations)
{
if (mediaInformation.Type == Core.Interops.Signatures.MediaTrackTypes.Audio)
{
int ChannelNumbers = mediaInformation.Audio.Channels;
}
}

An answer has been provided but no feedback from original poster, closing.


Rename package name to stwo-prover

This change is

#584  👈
dev

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

Join @shaharsamocha7 and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.90%. Comparing base (51f227b) to head (3c762bd).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #584   +/-   ##
=======================================
  Coverage   93.90%   93.90%
=======================================
  Files          67       67
  Lines        8419     8419
  Branches     8419     8419
=======================================
  Hits         7906     7906
  Misses        449      449
  Partials       64       64

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Pin gitleaks to v1 as v2 needs a paid API token

See: https://github.com/gitleaks/gitleaks-action#-announcement

How do I update existing actions to not fail? I am simply "re-running" jobs. Do I need to update the action itself?

@cmarshak if the action's already been run, re-running all failed jobs should do it

I tried here: https://github.com/ACCESS-Cloud-Based-InSAR/dem-stitcher/actions/runs/2579842974
Seems to still request the license.

@cmarshak weird -- it's still using the older action, not what's on main:
https://github.com/ASFHyP3/actions/blob/main/.github/workflows/reusable-secrets-analysis.yml#L13

For what it's worth, I just did a recent push and the gitleaks passed: https://github.com/ACCESS-Cloud-Based-InSAR/dem-stitcher/runs/7117109359?check_suite_focus=true
I think it must freeze the action version at the time of commit. Just a guess.


Add Remote Flow

I looked at the spec and the following happened, so I'm not sure if that is how it supposed to work:
lxc remote add demo <IP_ADDRESS>
Then I get the fingerprint. If I accept and enter a random password, I would assume, that the cert is not getting stored? But if I ran the exact command again, it says the certificate has already been stored.
I know that it still wouldn't help to get access, because the server didn't store the "client" cert as trusted and would reply with untrusted again. But maybe the remote cert should not be added in the first place, if I can't satisfy the requirements?
I noticed that the client sends a second get request to the server ("/1.0") after accepting the fingerprint. But then the password input comes. Is the second get request required? We can not be a trusted client and the info that we need to provide a password is in the first get request already.
I can dig through the code to understand it better, but maybe you can shed some light on it.

Can you give an example of the output? I'm a little confused by your description. Is it saying that the client already had the server cert, instead of the other way around?

I'm not grandmaster of explanation maybe :)
Let's say we have host1 and host2 and I want to have host2 trusting host1. So I run the 'remote add' command. Host2 shows the fingerprint and I accept it. Then host2 asks for the password. Let's assume, I don't know it, so I either cancel or enter some bogus password. Then I run the command again and host1 is telling me, the certificate has already been stored. Maybe I misread the spec, but the questions are:

If I don't know the password, is it by design that host1 stores the cert under the remote name, despite not knowing host2's password?

Did I misread the spec?

./lxc remote add helloworld host2:6430
Certificate fingerprint: bb 4c 2a 7a 61 57 b3 ee ae a4 16 12 aa 0e 49 3b 2a e3 48 14 e6 25 3b c3 ef 01 de 82 a6 6d 6f 73
ok (y/n)? y
Admin password for helloworld:
./lxc remote add helloworld host2:6430
Certificate already stored.
Admin password for helloworld:

Ok, half of what I said is rubbish after going through the source, so you can close it I guess. It is more a question of opinion rather than a bug. Just an explanation, so I don't look too stupid :)
In remote.go:
func addServer(config *lxd.Config, server string) error {

	fmt.Println(server)
	remote := config.ParseRemote(server)
	fmt.Println(remote)
	fmt.Println(config)
	c, err := lxd.NewClient(config, remote)
	if err != nil {
		return err
	}

	err = c.UserAuthServerCert()
	if err != nil {
		return err
	}

The UserAuthServerCert() call is actually storing the cert in my home folder /.config/lxc/servercerts
The I enter the admin password and it gets posted with the cert to the server. If my admin password is wrong, it returns Forbidden and the lxc cert is not added to the remote.
I was just expecting that the certificate in /.config/lxc/servercerts would be deleted after the add of the cert failed. But maybe that is what you wanted to check if the certificate has changed and that is, as far as I know, how 'known_hosts' in SSH works. I just did not expect that behaviour.
So, forget everything, except good morning. :) Issue can be closed.

Right, per specs/lxd-ssl-authentication.md we try to offer an ssh-like experience. The server fingerprint validation is separate from the password validation, so once you say yes to the fingerprint, the certificate is added to the client's trust store, then a second query is done to add the client's certificate to the server's trust store. That's the step which may prompt for a password and where if it fails, a re-try will not ask for the fingerprint again, just the password.
I think our current behavior is correct, however maybe the messaging back to the user in the client is confusing and could be improved somehow?

I think if you want to be close to ssl, nothing should be printed to stdout if the cert has been stored already. Feels like something is wrong. If there are issues like different fingerprint, ssl usually tells you how to fix it like 'delete xyz from known_hosts'.

Yeah, that sounds reasonable to me.

Will you send a branch for it?

Yes, will do. Seems like an easy solution.


I added new loss.py file.

At that time, this file had two loss functions: MSE and MAE. However, it is also capable of storing different types of loss functions in a single file, similar to the PyTorch type framework.
Now, see what this file looks like:

These loss are simple enough that I think you can just include in any of the scripts that uses them instead of going to the core library.

My main motive is to create a separate file to handle all types of loss calculation, similar to the PyTorch loss.py file.

Closing per what @g1y5x3 said.


React to Facebook API 2.2 Sunset

Gratipay currently has access to Graph API v2.2 which will reach the end of its 2-year lifetime on 27 March, 2017. To ensure a smooth transition, please migrate all calls to Graph API v2.3 or higher.
To check if your app will be affected by this upgrade you can use the Version Upgrade Tool. This will show you which calls, if any, are affected by this change as well as any replacement calls in newer versions. If you do not see any calls, your app may not be affected by this change.
You can also use our changelog
You can view this and other Developer Notifications related to your app, in the App Dashboard.

Version upgrade tool:

Your app hasn't made enough calls to the Graph API to show any info, or there are no changes for the methods you selected between v2.2 and v2.8

I guess that means we're good?


Add PCA test

Codecov Report

Merging #2 into master will increase coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master       #2     +/-   ##
=========================================
+ Coverage   63.24%   63.34%   +0.1%
=========================================
  Files          68       68
  Lines        5944     5944
=========================================
+ Hits         3759     3765      +6
+ Misses       1845     1842      -3
+ Partials      340      337      -3

Impacted Files
Coverage Δ

pca/pca.go
95.65% <0%> (+13.04%)
:arrow_up:

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 ce82db8...7845d2c. Read the comment docs.


Widget configuration improvements

Made improvements to the UI, as well as fixed a critical bug where you couldn't pick a watch to set up the widget for (Resolves #60)
The new UI makes use of radio buttons to present available watches to the user. This makes it more clear to the user that only one watch can be selected, as well as removes the redundant second line. Finishing setup also requires the user to actively confirm their selection by pressing "Finish".

I've added unit & UI tests to prevent a majority of issues from appearing in later releases. Should prevent a repeat of #60

I've added unit & UI tests to prevent a majority of issues from appearing in later releases. Should prevent a repeat of #60


Faster gui (small pr)

(description about how much performance gained is explained in individual commits)

added an option for running with or without kvm (KVM=yes brutal.sh -r)
about ~15%+ performance increase

Codecov Report

Merging #128 (245f984) into main (a5a28c7) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #128   +/-   ##
=======================================
  Coverage   62.76%   62.76%
=======================================
  Files         140      140
  Lines        6167     6167
=======================================
  Hits         3871     3871
  Misses       2296     2296

Impacted Files
Coverage Δ

sources/libs/brutal/gfx/color.h
0.00% <0.00%> (ø)

sources/libs/brutal/gfx/paint.h
0.00% <0.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 a5a28c7...245f984. Read the comment docs.


New version of VRChat is out now and the mod is broken

It would be nice if you coulf include a log file as well as what specifically is broken.

and also which mod because there are 7 in this repo...
but anyways, all should be fine already, please be patient when VRChat updates, we'll be dealing with it in no time


 I can do better.

Event's callbacks could be renamed.

The whole event machine needs a serious overhaul


Add more timeouts to Http2TestBase

There are some Kestrel HTTP/2 tests that are hanging. The hangs should be fixed once we take a runtime that includes dotnet/corefx#34014. In the meantime, this PR makes the hanging tests fail after a 30 second timeout.
Addresses aspnet/AspNetCore-Internal#1529

The hangs should be fixed once we take a runtime that includes dotnet/corefx#34014

Guessing that would be myget.org/dotnet-core/.../System.IO.Pipelines
4.6.0-preview.18613.4 (Thu, 13 Dec 2018 14:36:16 GMT)
Though the current version is 4.6.0-preview.18618.7 (Wed, 19 Dec 2018 00:39:07 GMT)


Implement large file support

Converting large .xyz files to hdf5 currently fails because the entire file is loaded into memory. There is a fix for this in this branch:
https://github.com/chiang-yuan/mace/tree/dev/large-dataset

Maybe the above mentioned solution needs some refinement.  The line
https://github.com/chiang-yuan/mace/blob/cf562584874b7c277edd05da9b6b54e56b9d8932/mace/cli/preprocess_data_mpi.py#L168
probably reads the full .xzy file into memory, which should be prevented.

Possible solution implemented in my fork:
https://github.com/carlosmada22/equitrain
Files changed:
equitrain/preprocess.py
equitrain/mace/data/utils.py
Generated files after running tests/test_preprocess.py over the test file tests/data.xyz:
statistics.json
(These are the h5 files not supported here, just remove the '.txt' from the name)
train.h5.txt
valid.h5.txt
Need for further analysis, as I am not sure if the results are good or not.


Manage Revisions

Add functionality to view, restore, and prune revisions for pages and posts.

Released in beta 4


[Event Request] Page 5708 "Get Shipment Lines" - New publisher

Hello, we need a new Event publisher in procedure SetSalesHeader. Can you please check that? Thank you!
procedure SetSalesHeader(var SalesHeader2: Record "Sales Header")
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnBeforeSetSalesHeader(SalesHeader2, IsHandled);
    if IsHandled then
        exit;

    SalesHeader.Get(SalesHeader2."Document Type", SalesHeader2."No.");
    SalesHeader.TestField("Document Type", SalesHeader."Document Type"::Invoice);
end;

New Event:
[IntegrationEvent(true, false)]
local procedure OnBeforeSetSalesHeader(SalesHeader2: Record "Sales Header"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Display count of deferred apps on monthly stats

Context
The deferred applications counter in the Footnote section of the monthly statistics report is hard-coded. This PR adds a helper to MonthlyStatisticsReport to calculate the correct number to render.
Changes proposed in this pull request

Added MonthlyStatisticsReport#deferred_application_count helper.
Display this in the view template and handle different cases (0, 1 or
more deferred applications) with appropriate content.

when there are no deferred applications (or data is missing):

when there is one deferred application:

when there are multiple deferred application:

Guidance to review

Is the query that fetches the number correct? (I found only 1 application in the current cycle that matched).
Should this number be 'cached' as part of the MonthlyStatisticsReport record so that it represents the correct number at the time the report was generated?
What changes do we need to make to the copy to (a) make it clearer (b) work next year when UCAS wasn't involved in the previous cycle?

Link to Trello card
https://trello.com/c/Eo5jeO7a/4214-work-out-what-to-do-with-monthly-stats-footnotes
Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] Required environment variables have been updated added to the Azure KeyVault

I've only got as far as looking at the content in the first screenshot so far - do you think this makes it clearer:

Deferred candidates are counted in a separate column in the tables.
They’re counted in the statistics report for the academic year in which they start their training, rather than the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, rather than this one.
Deferred candidates are only counted once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.

Will take a look at the other bits later today

Is the second screenshot trying to say the following? @stevehook

Deferred candidates are counted in a separate column in the tables.
There is one deferred candidate in this data report that was also counted in last year’s publication by UCAS.
This is because UCAS counted deferred candidates in the statistics report for the academic year in which they applied, rather than the year in which they start training.
This report, on the other hand, counts deferred candidates in the year in which they start their training, rather than the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, rather than this one.
Deferred candidates are only counted once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.

Is the last screenshot trying to say the following? @stevehook

Deferred candidates are counted in a separate column in the tables.
There are 24 deferred candidates in this data report that were also counted in last year’s publication by UCAS.
This is because UCAS counted deferred candidates in the statistics report for the academic year in which they applied, rather than the year in which they start training.
This report, on the other hand, counts deferred candidates in the year in which they start their training, rather than the year in which they apply.
Anyone who applied this year but deferring until next year will be counted in next year’s report, rather than this one.
Deferred candidates are only counted once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.

I've only got as far as looking at the content in the first screenshot so far - is this what it's trying to say? @stevehook :
Deferred candidates are counted in a separate column in the tables.
They’re counted in the statistics report for the academic year in which they start their training, rather than the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, rather than this one.
Deferred candidates are only counted once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.
Will take a look at the other bits later today

Thanks @EmmaFrith, I think it's a bit more like this (apologies for mangling the words)...
Deferred candidates are counted in a separate column in the tables.
They’re counted in the statistics report for the academic year in which they start their training after their offer is re-confirmed, as well as the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, as well as this one.
Deferred candidates are only counted in the academic year that they start their course in once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.

I've only got as far as looking at the content in the first screenshot so far - is this what it's trying to say? @stevehook :
Deferred candidates are counted in a separate column in the tables.
They’re counted in the statistics report for the academic year in which they start their training, rather than the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, rather than this one.
Deferred candidates are only counted once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.
Will take a look at the other bits later today

Thanks @EmmaFrith, I think it's a bit more like this (apologies for mangling the words)...
Deferred candidates are counted in a separate column in the tables.
They’re counted in the statistics report for the academic year in which they start their training after their offer is re-confirmed, as well as the year in which they apply.
This means that anyone who applied this year but deferring until next year will be counted in next year’s report, as well as this one.
Deferred candidates are only counted in the academic year that they start their course in once training providers re-confirm their deferred offer. Providers need to do this before candidates intend to start their training.

ah ok so they're eventually counted twice? if that's the case, how you've edited the copy here looks good to me

@stevehook @EmmaFrith re the question on guidance for review. Is there some value in having some if/elseif conditional logic that is driven by date. So after say, 12 October 2022 (or whenever next cycle opens), we render something that removes mention to UCAS altogether? Or do we just put a reminder to tackle that in the next cycle on the backlog and just do a PR then?

@EmmaFrith I've updated the content and screenshots following with your suggestions.

@stevehook @EmmaFrith re the question on guidance for review. Is there some value in having some if/elseif conditional logic that is driven by date. So after say, 12 October 2022 (or whenever next cycle opens), we render something that removes mention to UCAS altogether? Or do we just put a reminder to tackle that in the next cycle on the backlog and just do a PR then?

Yes I think we need that. Might as well do it now, though I'm wondering whether we should be extract this into a component rather than overloading the view with conditional logic...

@stevehook could you change this sentence to 'Deferred candidates are only counted in the academic year in which they start their course once training providers re-confirm their deferred offer.'?


MKLivestatus reports station down but app up

Yesterday MKLivestatus correctly informed the app was down when the station was still booting (and down) and didn't report the app was up until it was.
Today I find that stations with state=1 (down) have app_state=0 (up)... which makes no sense.
{"lastState":[{"id":"supernova.mfo.de","state":0,"state_type":1,"app_state":2,"app_state_type":1,"app_id":""},{"id":"vb-hb-test-1","state":1,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_a"},{"id":"vb-hb-test-2","state":1,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_b"},{"id":"vb-hb-test-3","state":1,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_c"},{"id":"vb-hb-test-4","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_a"},{"id":"vb-hb-test-5","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_c"},{"id":"vb-hb-test-6","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_a"},{"id":"vb-hb-test-7","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_a"},{"id":"vb-hb-test-8","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_a"},{"id":"vb-hb-test-9","state":0,"state_type":1,"app_state":0,"app_state_type":1,"app_id":"hb_test_b"}]}

PS: I made a change to my code to handle <state=1, app_state=0>, but now I can't test it because of how MKLivestatus is behaving. We need to fix this, but not without time to test it before the demo because my code might break since it never handled the transition.


unable to click on "+" to add relays

v<IP_ADDRESS>-alpha Latest
Thanks !

Hi @KruyKaze I think this might be due to not enough info about the relays view.
For now on alpha version you can only have  one active relay, so in order to choose other one you need to first delete the active one. Please let me know if this is your case.
I'm actually working on an UI redesign and I'll have this into consideration.

Yes I can add one when I remove one .thanks
Dec 19, 2022 15:54:12 KoalaSat @.***>:

Hi @KruyKaze[https://github.com/KruyKaze] I think this might be due to not enough info about the relays view.
For now on alpha version you can only have one active relay, so in order to choose other one you need to first delete the active one. Please let me know if this is your case.
I'm actually working on an UI redesign and I'll have this into consideration.
—
Reply to this email directly, view it on GitHub[https://github.com/KoalaSat/nostros/issues/52#issuecomment-1358519357], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAARFYIKATKSUNQIIISQOQ3WODRRHANCNFSM6AAAAAATDY7YYM].
You are receiving this because you were mentioned.[Tracking image][https://github.com/notifications/beacon/AAARFYLNP2CVHFFKYHOGVXLWODRRHA5CNFSM6AAAAAATDY7YYOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSQ7FOD2.gif]


Add provider name to provider resource

Task
Resolves: #228
Description
Disclaimer: for providers with complex names (such as aws-native), the names have the following format: AzurenativeProviderResource. I'll try to figure out if there's an easy way to make the name AzureNativeProviderResource instead, but the easiest way was to ignore the capitalization of complex names.
TODO

[x] Add tests Fix existing tests
[x] If the current solution is determined to be not good enough, figure out a way to capitalize complex names

I wish I could delve into this more, but two things that I noticed (partially related to this PR)

isProvider — passing this primitive argument through all the layers feels wrong to me. Maybe we should extend UsageKind by adding another Subject (Provider) or take a step back and think of a better design altogether?

PulumiName —  I think we could refactor it and its usage a bit along with NamingConfiguration + UsageKind abstractions

Both good points. I tried to address the first one, but adding this new enum value isn't as straightforward as it seems (Subject isn't passed to PulumiName and I think we'd need to add an extra field in Modifiers which seems a bit clumsy). I'll create a task to address both of these things, but for now I'll merge it as is.


add (response-timeout) to http:options

http:respond and http:respond-file use the default 5 second gen-server
timeout which is not enough for slow networks or large amounts of data.
Adding an optional timeout argument to each of these functions helps,
but does not cover the static file handler buried in the http-cache server.
Adding (response-timeout) allows Swish applications to override this
behavior when configuring the server. Using a default of 60 seconds
should allow a 50 MB file over a 10 Mbit/sec connection, or 500 MB
file over a 100 Mbit/sec connection with overhead.

I didn't develop this update on Windows, so it was a bit of a surprise to me too. I'll dig into it and see what I can find.

It seems that Microsoft has gotten Windows to transfer data on the loopback interface as ridiculously fast rates. I've dialed the data up to 1 Gb, and the timeout down to 100ms and that fails as expected (it takes 200 ms to transfer, so there is some buffer).  However, I believe dump-stack is trying to render the stack (and massive bytevector) to a string, and that is taking long enough (and allocates a ton of RAM) that the mat itself times out. I don't really like the "allocate a 1Gb bytevector", but I don't see another option. I'll keep looking for a solution.

I'm wrong about dump-stack. I removed its guts and there is still something unusual happening.

It appears my diagnostics impeded the ability for the test to finish cleanly. I increased the data size to 1Gb and reduced the timeout to 50ms. Without diagnostics, my machine took 100ms to transfer the data, so 50ms should be enough time for the timeout to fire.

Reducing data size to 2^29-1 for 32-bit systems, and timeout to 15ms to give room for timing error.

Thanks for the review Bob. Squashing the commits down in this last commit.


Ensure downloading a 100+MB SQLite database file works

I've seen attempted downloads of large files fail after about ten seconds.

I'd really like to use the content-length header here, but Sanic hasn't yet fixed the bug I filed about it: https://github.com/huge-success/sanic/issues/1194

I no longer depend on Sanic so I should be able to solve this entirely within the Datasette codebase. I need to figure out how Transfer-Encoding: chunked and the Content-Length header interact.

This was fixed in #749 by 88ac538b41a4753c3de9b509c3a0e13077f66182


add angular2 material project

Added the angular2 material project and it's dependencies.

This starter is meant to be as simple as possible.
It is better for the end user to be able to decide what to add than forcing lot of them to remove some dependencies because they don't need it.

@Foxandxss : angular2-material is unfortunately still alpha, most of UI components (like lists) are missing.
but the ng2-translate module is also great...
and many more... but where are they ?
The angular2 echosystem is becoming big and because there were many "ideas" during the alpha, beta and RCx phases, searching on google brings now a lot of pollution ... obsolete components, solutions, and so on ... Angular2 website is incomplete, most of advanced topics are not covered, I discover them in forums, not even in books..
You have a great starter, you have a not so small community on this project. Would it make sense to extend your starter in a kind of subprojects way in order to include the best of extensions ? We are spending all a lot of time to find the right extensions... this starter, whose core would be as you mentioned the most simple as possible, could be greatly extended.... if it not possible, this could be the "angular2-webpack-andthenwhat" project.
The idea would be to have a kind of repository of all working and useful components or tools and a hello world on how to start with them and configure them in the starter and of course a status to know if the components are still developped and maintained....
I know, you could tell me, do it yourself :)
thank you

This is an interesting response, so let's go by parts.

angular2-material is unfortunately still alpha, most of UI components (like lists) are missing.
but the ng2-translate module is also great...
and many more... but where are they ?

I don't follow you here. Where are they? Well, in their respective github and npm packages. If you need any of them at any time, you just need to npm install it using the starter and you are good to go.
You don't really need to add those packages as "default" in the starter, they are just one step away.

The angular2 echosystem is becoming big and because there were many "ideas" during the alpha, beta and RCx phases, searching on google brings now a lot of pollution ... obsolete components, solutions, and so on ...

My advice here is simply to read the official docs, whether it is Angular docs or the third party docs. They are supposed to have a up to date readme that you can use to be able to use the tool.

Angular2 website is incomplete, most of advanced topics are not covered, I discover them in forums, not even in books..

Please, send me an email with all your concerns about the website. I am part of the docs team of Angular so that page is my responsibility. Just tell me what do you think it is wrong / missing or what would you like to see there. No need to lie, I am open minded.

You have a great starter, you have a not so small community on this project. Would it make sense to extend your starter in a kind of subprojects way in order to include the best of extensions ?

Thank you sir. And no, it doesn't make sense. Today there are 5 "good extensions" (I made up that number) but tomorrow you can have 20. The good part is that installing say ng-bootstrap is just one step away. You npm install it and you are good to go.
And I hear you with your need of having multiple starters, but they are really really time consuming and there is no great value. Adding third party stuff is really easy.

Is anybody knows how to add custom theme to angular2-webpack + angular2-material

@bankrot have u already managed to add a theme? I might be looking in to this one on Friday.


[BEAM-898] Fix Jenkins Build Failure - IT Options Conflicts

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Remove checksum variable in IT Options since it causes conflict between different ITs.
only register TestPipelineOptions.class to avoid incompatibility of options defined in different pipeline.

Test is done by running full set of IT tests on Dataflow service.

+R: @peihe @jasonkuster

LGTM
R: @lukecwik

Merging

Re: "It solves the problem by somewhat decreasing the utility offered of the test."
I think the previous utility comes from the improperly uses of PipelineOptions:

It was using PipelineOptions as the parser for arguments only used in IT tests main(), such as checksum (which is not a true pipeline option).
It is still using PipelineOptions as the arguments parser for WordCount.main(). All cmd arguments, relevant or irrelevant, are passed to WordCount.
WordCount.main(TestPipeline.convertToArgs(options))

Those improper uses requires declaring all PipelineOptions in the global namespace through register(), which caused the post-submit failure.
This PR removed the first improper use, and reduced some utilities (which we are not currently depends on). So, I think it is a good change.
There is some ideas about support a testArg for ITs tests in here:
https://issues.apache.org/jira/browse/BEAM-898?focusedCommentId=15648972&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15648972


[DON'T MERGE] Use Functionalization pass

PR for collaboration and hunting down bugs.

cc @bdhirsh


feat: update to safe-ds 0.17.1

Summary of Changes

updated to safe-ds 0.17.1 + dependencies
temporarily disable python 3.12
changed image encoding to match new safe-ds version
removed jpeg test

Is this still WIP or ready to merge?

There is a warning printed on every runner startup since this update, which I am trying to fix, that's why it is still WIP

It seems like the update to the new library version causes the runner to startup slower (probably because of the larger amount of dependencies).
For the record, this was the warning:
src/safeds_runner/server/main.py:202: MonkeyPatchWarning: Monkey-patching ssl after ssl has already been imported may lead to errors, including RecursionError on Python 3.6. It may also silently lead to incorrect behaviour on Python 3.7. Please monkey-patch earlier. See https://github.com/gevent/gevent/issues/1016.
Modules that had direct imports (NOT patched): ['urllib3.util (venv\\Lib\\site-packages\\urllib3\\util\\__init__.py)', 'urllib3.util.ssl_ (venv\\Lib\\site-packages\\urllib3\\util\\ssl_.py)'].
  patch_all()

I refactored the main.py a bit to fix this.

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

v0.5.0
GitHub release

Your semantic-release bot :package::rocket:


Add JavaBigInteger Class  examples and uses

i think this is very important topic in java and should be covered ....
so  @owais4321
i can work on it if you allow me to....thankyou!

@anchal00 go for it (y)


No SQL trace for swagger requests

We use django as swagger API and have djangorestframework, django-rest-swagger and drf-yasg installed. In the debug panel we can see only django SQL queries like SELECT * FROM django_session or SEECT * FROM user but there is no any queries when we try to GET or POST our endpoints which exactly makes SQL query. Where can be the problem or how can we debug it ?

DDT only shows an SQL trace for the current request. Calls to DRF APIs are separate requests which would need some capturing of their own.
Maybe logging SQL queries to the console is an option for you?
https://avilpage.com/2018/05/django-tips-tricks-log-sql-queries-to-console.html

Ok got it thanks for link.

DEBUG_TOOLBAR_CONFIG = {     "UPDATE_ON_FETCH": True }
This helped me


Example 2 for addProperty is wrong

The sample expression does not add the value, it replaces the entire customerName.

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

ID: 03aa9e88-7fa0-56ab-f794-58f0c37b6d7a
Version Independent ID: 25c5ef5c-6ba4-f5ff-f94b-8f2818ec2feb
Content: Reference guide for expression functions - Azure Logic Apps
Content Source: articles/logic-apps/workflow-definition-language-functions-reference.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@Johns-305
Thanks for your feedback! We will investigate and update as appropriate.

@Johns-305 The example 2 is correct as per the input and output. If you need the output as below:
{
   "customerName": {
      "firstName": "Sophia",
      "middleName": "Anne",
      "surName": "Owen"
   }
}

then the expression is correct which is documented :
setProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne'))
In case if you need the output as below:
{
  "firstName": "Sophia",
  "surName": "test",
  "middleName": "Anne"
}

then you need to use the expression as :
addProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne'))
Hope the above clears things up. Feel free to get back to me if you have any queries or concerns.

@Johns-305 Can you elaborate with the example?
Do you mean that I have updated the highlighted value and the output is Sophia2. It is expected behavior.
The setProperty works like this when you define setProperty and pass the object, the parent property with the child property that you want to setchange along with the new property object you want to set.
setProperty(json('{ "customerName": { "firstName": "Sophia1", "surName": "Owen" } }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia2", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne'))
That is the reason I mentioned when you use addProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne')) then the output is below :
{
  "firstName": "Sophia",
  "surName": "test",
  "middleName": "Anne"
}

Now when we use setProperty then we are making sure that our current property customerName is replaced by the above value then we have define therefor it is excepted as per the example and sample output.

This example adds the middleName child property to the existing customerName property in a JSON object,
setProperty(json('{ "customerName": { "firstName": "Sophia1", "surName": "Owen1"} }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia2", "surName": "Owen2" } }')['customerName'], 'middleName', 'Anne'))
returns:
{
"customerName": {
"firstName": "Sophia2",
"surName": "Owen2",
"middleName": "Anne"
}
}
Sophia1, Owen1 is replaced.  MiddleName is not added as the description says.

From: MayankBargali-MSFT @.>
Sent: Tuesday, May 9, 2023 5:18 AM
To: MicrosoftDocs/azure-docs @.>
Cc: John Stephens @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Example 2 for addProperty is wrong (Issue #109211)
@Johns-305https://github.com/Johns-305 Can you elaborate with the example?
Do you mean that I have updated the highlighted value and the output is Sophia2. It is expected behavior.
The setPropertyhttps://learn.microsoft.com/en-us/azure/logic-apps/workflow-definition-language-functions-reference#setproperty works like this when you define setProperty and pass the object, the parent property with the child property that you want to setchange along with the new property object you want to set.
setProperty(json('{ "customerName": { "firstName": "Sophia1", "surName": "Owen" } }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia2", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne'))
That is the reason I mentioned when you use addProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne')) then the output is below :
{
"firstName": "Sophia",
"surName": "test",
"middleName": "Anne"
}
Now when we use setProperty then we are making sure that our current property customerName is replaced by the above value then we have define therefor it is excepted as per the example and sample output.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/109211#issuecomment-1539460821, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHYSVVAWZYCMGF4YC23TOXTXFIDXFANCNFSM6AAAAAAXY7STZA.
You are receiving this because you were mentioned.Message ID: @.***>

Is it clear why it's wrong?

From: John Stephens @.>
Sent: Thursday, May 11, 2023 7:44 AM
To: MicrosoftDocs/azure-docs @.>; MicrosoftDocs/azure-docs @.>
Cc: Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Example 2 for addProperty is wrong (Issue #109211)
This example adds the middleName child property to the existing customerName property in a JSON object,
setProperty(json('{ "customerName": { "firstName": "Sophia1", "surName": "Owen1"} }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia2", "surName": "Owen2" } }')['customerName'], 'middleName', 'Anne'))
returns:
{
"customerName": {
"firstName": "Sophia2",
"surName": "Owen2",
"middleName": "Anne"
}
}
Sophia1, Owen1 is replaced.  MiddleName is not added as the description says.

From: MayankBargali-MSFT @.>
Sent: Tuesday, May 9, 2023 5:18 AM
To: MicrosoftDocs/azure-docs @.>
Cc: John Stephens @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Example 2 for addProperty is wrong (Issue #109211)
@Johns-305https://github.com/Johns-305 Can you elaborate with the example?
Do you mean that I have updated the highlighted value and the output is Sophia2. It is expected behavior.
The setPropertyhttps://learn.microsoft.com/en-us/azure/logic-apps/workflow-definition-language-functions-reference#setproperty works like this when you define setProperty and pass the object, the parent property with the child property that you want to setchange along with the new property object you want to set.
setProperty(json('{ "customerName": { "firstName": "Sophia1", "surName": "Owen" } }'), 'customerName', addProperty(json('{ "customerName": { "firstName": "Sophia2", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne'))
That is the reason I mentioned when you use addProperty(json('{ "customerName": { "firstName": "Sophia", "surName": "Owen" } }')['customerName'], 'middleName', 'Anne')) then the output is below :
{
"firstName": "Sophia",
"surName": "test",
"middleName": "Anne"
}
Now when we use setProperty then we are making sure that our current property customerName is replaced by the above value then we have define therefor it is excepted as per the example and sample output.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/109211#issuecomment-1539460821, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHYSVVAWZYCMGF4YC23TOXTXFIDXFANCNFSM6AAAAAAXY7STZA.
You are receiving this because you were mentioned.Message ID: @.***>

@Johns-305 It looks good as per the discussion. In case if you need any assistance I can connect with you offline, and you can send email to with subject line “Attn:Mayank” to AzCommunity[at]Microsoft[dot]com so I can assist you.
I will now proceed to close this thread and we can connect offline for further discussion.


windows 10 compatibility

Some minor modifications to make it compatible to windows 10
Eli

I want to but I don't know how to roll this commit back.  Sorry09.04.2019, 23:30, "Ben Marshall" notifications@github.com:Hi Eli,
The latest updates here look good. I have pulled the latest revision and tested on my linux box to make sure there are no breaking changes, looks all good.
Do you think there is much more to do to close out the functionality you wanted to add?
One request i do have - in your latest commit - 8a5b9ab - you included the generated files for one of the examples. Could you roll this commit back to clean up these files?
I also think it would be good to have an example project with the new cflags options, have you got something simple you are using for testing which could be added?
Cheers,
Ben
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

instead, I have done some minor cleanup.  17.04.2019, 22:51, "Joe Chan" dr.joechan@yandex.com:I want to but I don't know how to roll this commit back.  Sorry09.04.2019, 23:30, "Ben Marshall" notifications@github.com:Hi Eli,
The latest updates here look good. I have pulled the latest revision and tested on my linux box to make sure there are no breaking changes, looks all good.
Do you think there is much more to do to close out the functionality you wanted to add?
One request i do have - in your latest commit - 8a5b9ab - you included the generated files for one of the examples. Could you roll this commit back to clean up these files?
I also think it would be good to have an example project with the new cflags options, have you got something simple you are using for testing which could be added?
Cheers,
Ben
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.


Flow cells from 18.03.2020

a list of open and closed To-Dos:

200318_SN737_0494_ACE121ACXX

[x]     Transfer raw data to HPC
[x]     check NAS space capacity
[x]     backup
[ ]     create samplesheet and push to github
[ ]     inform wetlab to delete data on instrument
[ ]     inform analysts about demux quality
[ ]     delete data from pipeline server

200318_SN737_0495_BCDYWTACXX

[x]     Transfer raw data to HPC
[x]     check NAS space capacity
[x]     backup
[ ]     create samplesheet and push to github
[ ]     inform wetlab to delete data on instrument
[ ]     inform analysts about demux quality
[ ]     delete data from pipeline server

*both in Branch name 180320 because of trio
check Snakemake for more details

doule check Transfer raw data to HPC after HPC /GPFG problem
for 200318_SN737_0495_BCDYWTACXX there are 2 cases

per Tumor
post Tumor
each have to process spreatly


#error This file requires compiler and library support

Hello,
I am facing this problem once I type make:
~/SURVIVOR/Debug:make
Building file: ../src/vcfs/Annotate_vcf.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Annotate_vcf.d" -MT"src/vcfs/Annotate_vcf.o" -o "src/vcfs/Annotate_vcf.o" "../src/vcfs/Annotate_vcf.cpp"
Finished building: ../src/vcfs/Annotate_vcf.cpp

Building file: ../src/vcfs/Combine_3_VCF.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Combine_3_VCF.d" -MT"src/vcfs/Combine_3_VCF.o" -o "src/vcfs/Combine_3_VCF.o" "../src/vcfs/Combine_3_VCF.cpp"
Finished building: ../src/vcfs/Combine_3_VCF.cpp

Building file: ../src/vcfs/Compoverlap_VCF.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Compoverlap_VCF.d" -MT"src/vcfs/Compoverlap_VCF.o" -o "src/vcfs/Compoverlap_VCF.o" "../src/vcfs/Compoverlap_VCF.cpp"
Finished building: ../src/vcfs/Compoverlap_VCF.cpp

Building file: ../src/vcfs/Detect_nested.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Detect_nested.d" -MT"src/vcfs/Detect_nested.o" -o "src/vcfs/Detect_nested.o" "../src/vcfs/Detect_nested.cpp"
Finished building: ../src/vcfs/Detect_nested.cpp

Building file: ../src/vcfs/Filter_vcf.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Filter_vcf.d" -MT"src/vcfs/Filter_vcf.o" -o "src/vcfs/Filter_vcf.o" "../src/vcfs/Filter_vcf.cpp"
../src/vcfs/Filter_vcf.cpp: In function ‘void translate(const char*, int, float, int, strentry&)’:
../src/vcfs/Filter_vcf.cpp:18:6: warning: variable ‘lumpy’ set but not used [-Wunused-but-set-variable]
  int lumpy = 0;
      ^
Finished building: ../src/vcfs/Filter_vcf.cpp

Building file: ../src/vcfs/Generate_distMat.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Generate_distMat.d" -MT"src/vcfs/Generate_distMat.o" -o "src/vcfs/Generate_distMat.o" "../src/vcfs/Generate_distMat.cpp"
Finished building: ../src/vcfs/Generate_distMat.cpp

Building file: ../src/vcfs/Merge_VCF.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/vcfs/Merge_VCF.d" -MT"src/vcfs/Merge_VCF.o" -o "src/vcfs/Merge_VCF.o" "../src/vcfs/Merge_VCF.cpp"
Finished building: ../src/vcfs/Merge_VCF.cpp

Building file: ../src/snp_overlap/Overlap_snps.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/snp_overlap/Overlap_snps.d" -MT"src/snp_overlap/Overlap_snps.o" -o "src/snp_overlap/Overlap_snps.o" "../src/snp_overlap/Overlap_snps.cpp"
../src/snp_overlap/Overlap_snps.cpp: In function ‘void overlap_snps(std::__cxx11::string, std::__cxx11::string, int, int, int, std::__cxx11::string)’:
../src/snp_overlap/Overlap_snps.cpp:192:30: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for (size_t i = start; i <= stop; i++) {
                              ^
../src/snp_overlap/Overlap_snps.cpp: In function ‘void overlap_snps_gwas(std::__cxx11::string, std::__cxx11::string, int, int, std::__cxx11::string)’:
../src/snp_overlap/Overlap_snps.cpp:447:64: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (genome.find(chr) != genome.end() && genome[chr].size() > pos) {
                                                                ^
../src/snp_overlap/Overlap_snps.cpp: In function ‘void overlap_snpsGWASDB(std::__cxx11::string, std::__cxx11::string, int, int, int, std::__cxx11::string)’:
../src/snp_overlap/Overlap_snps.cpp:663:80: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     if (genome.find(snps[i].chr) != genome.end() && genome[snps[i].chr].size() > snps[i].position) {
                                                                                ^
../src/snp_overlap/Overlap_snps.cpp: In function ‘void generate_random_regions(std::__cxx11::string, std::__cxx11::string, int, std::__cxx11::string)’:
../src/snp_overlap/Overlap_snps.cpp:767:50: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
      for (size_t pos = entries[i].start.pos; pos < entries[i].stop.pos; pos++) {
                                                  ^
../src/snp_overlap/Overlap_snps.cpp:781:47: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for (size_t pos = entries[i].start.pos; pos < entries[i].stop.pos; pos++) {
                                               ^
Finished building: ../src/snp_overlap/Overlap_snps.cpp

Building file: ../src/simulator/Error_scanner.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/simulator/Error_scanner.d" -MT"src/simulator/Error_scanner.o" -o "src/simulator/Error_scanner.o" "../src/simulator/Error_scanner.cpp"
../src/simulator/Error_scanner.cpp: In function ‘void computeAlignment(std::vector<CigarOp>, char*, std::__cxx11::string, std::vector<std::vector<std::vector<int> > >&, std::__cxx11::string)’:
../src/simulator/Error_scanner.cpp:173:18: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   while (pos + 1 > error_mat.size()) {
                  ^
Finished building: ../src/simulator/Error_scanner.cpp

Building file: ../src/simulator/Eval_vcf.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/simulator/Eval_vcf.d" -MT"src/simulator/Eval_vcf.o" -o "src/simulator/Eval_vcf.o" "../src/simulator/Eval_vcf.cpp"
../src/simulator/Eval_vcf.cpp: In function ‘void reprotvals(const char*)’:
../src/simulator/Eval_vcf.cpp:102:8: warning: variable ‘dv’ set but not used [-Wunused-but-set-variable]
  float dv = 0; //"# high-quality variant pairs"
        ^
../src/simulator/Eval_vcf.cpp:103:8: warning: variable ‘dr’ set but not used [-Wunused-but-set-variable]
  float dr = 0; //"# high-quality reference pairs"
        ^
../src/simulator/Eval_vcf.cpp:104:6: warning: variable ‘gq’ set but not used [-Wunused-but-set-variable]
  int gq = 0; //"Genotype Quality"
      ^
../src/simulator/Eval_vcf.cpp:105:6: warning: variable ‘rr’ set but not used [-Wunused-but-set-variable]
  int rr = 0;
      ^
../src/simulator/Eval_vcf.cpp:106:6: warning: variable ‘rv’ set but not used [-Wunused-but-set-variable]
  int rv = 0;
      ^
Finished building: ../src/simulator/Eval_vcf.cpp

Building file: ../src/simulator/Pac_Simulator.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/simulator/Pac_Simulator.d" -MT"src/simulator/Pac_Simulator.o" -o "src/simulator/Pac_Simulator.o" "../src/simulator/Pac_Simulator.cpp"
Finished building: ../src/simulator/Pac_Simulator.cpp

Building file: ../src/simulator/SV_Simulator.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/simulator/SV_Simulator.d" -MT"src/simulator/SV_Simulator.o" -o "src/simulator/SV_Simulator.o" "../src/simulator/SV_Simulator.cpp"
Finished building: ../src/simulator/SV_Simulator.cpp

Building file: ../src/simulator/Sim_reads.cpp
Invoking: Cross G++ Compiler
g++ -O3 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"src/simulator/Sim_reads.d" -MT"src/simulator/Sim_reads.o" -o "src/simulator/Sim_reads.o" "../src/simulator/Sim_reads.cpp"
In file included from /usr/include/c++/5/random:35:0,
                 from ../src/simulator/Sim_reads.h:21,
                 from ../src/simulator/Sim_reads.cpp:8:
/usr/include/c++/5/bits/c++0x_warning.h:32:2: error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support must be enabled with the -std=c++11 or -std=gnu++11 compiler options.
 #error This file requires compiler and library support \
  ^
src/simulator/subdir.mk:33: recipe for target 'src/simulator/Sim_reads.o' failed
make: *** [src/simulator/Sim_reads.o] Error 1

Hope you could help me out.
Thank you in advance.

Hi,
are you using a Mac or a Linux machine?
Thanks
Fritz

Hi,
I am using a Linux machine.
Thanks for replying.

Hi,
i think I found the problem and that was on me.. .sorry for that.
Can you clone and try again?
Thanks
Fritz

Thank you for replying and for your help.
It worked!


groups: create & join group desktop update

Implements a modal version of the recent add group sheet on desktop.
Extracts some of the common logic used in both views where possible.
Slightly adjusts the look of the desktop root sidebar to accommodate the change.
Tested locally against livenet ships (self & Tlon hosted).

Fixes LAND-1477
PR Checklist

[ ] Includes changes to desk files
[x] Describes how you tested the PR locally (test ship vs livenet)
[ ] If a new feature, includes automated tests
[x] Comments added anywhere logic may be confusing without context

Looking into E2E failures + adding tests

btw I think the tlon block was exported without padding so it looks really big

@mrozanski this should be ready for testing

@mrozanski this should be ready for testing

Reviewing and testing now


[BACKLOG-15764] Introducing new Logger for use inside Operation Funct…

…ions. Modifying the Trans Adapter to subscribe to Operation Logging

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/ael/adapters/TransEngineAdapter.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change:  + .53%
Complexity Change:  + .56%
Instruction Change:  + .07%

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change:  + 1.00%
Complexity Change:  + 1.41%
Instruction Change:  + .30%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change:  + .66%
Complexity Change:  + .93%
Instruction Change:  + .13%

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change:  + .31%
Instruction Change:  + .06%
Line Change: -.23%:small_red_triangle_down:
Method Change: -1.02%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change:  + 2.00%
Complexity Change:  + 1.89%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 2.35%
Complexity Change:  + 2.70%
Instruction Change:  + .18%
Line Change:  + .48%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/org/pentaho/di/trans/ael/adapters/TransEngineAdapter.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 71 Column: 9

:heavy_exclamation_mark: '{' is not followed by whitespace. at Line: 112 Column: 61

:heavy_exclamation_mark: '}' is not preceded with whitespace. at Line: 112 Column: 61

:heavy_exclamation_mark: '{' is not followed by whitespace. at Line: 114 Column: 43

:heavy_exclamation_mark: '}' is not preceded with whitespace. at Line: 114 Column: 43

:heavy_exclamation_mark: 'switch' is not followed by whitespace. at Line: 121 Column: 11

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 147 Column: 11

:heavy_exclamation_mark: Only one statement per line allowed. at Line: 194 Column: 149

:heavy_exclamation_mark: Only one statement per line allowed. at Line: 194 Column: 154

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/ael/adapters/TransEngineAdapter.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.78%:small_red_triangle_down:
Complexity Change: -1.03%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change:  + .53%
Complexity Change:  + .56%
Instruction Change:  + .07%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Instruction Change:  + .09%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.53%:small_red_triangle_down:
Complexity Change: -4.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.33%:small_red_triangle_down:
Complexity Change: -1.85%:small_red_triangle_down:
Instruction Change: -.28%:small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -.82%:small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/org/pentaho/di/trans/ael/adapters/TransEngineAdapter.java

:heavy_exclamation_mark: Only one statement per line allowed. at Line: 198 Column: 62


Eml [dev]

Codecov Report

Merging #641 (d56e11b) into dev (4ffa1d6) will increase coverage by 0.02%.
The diff coverage is 88.54%.

@@            Coverage Diff             @@
##              dev     #641      +/-   ##
==========================================
+ Coverage   70.16%   70.19%   +0.02%
==========================================
  Files         127      127
  Lines       11371    11371
==========================================
+ Hits         7979     7982       +3
+ Misses       3392     3389       -3

Impacted Files
Coverage Δ

assemblyline/common/identify.py
89.07% <88.54%> (ø)

assemblyline/odm/randomizer.py
94.39% <0.00%> (+0.86%)
:arrow_up:

assemblyline/datasource/alert.py
92.30% <0.00%> (+3.84%)
:arrow_up:

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 4ffa1d6...d56e11b. Read the comment docs.


'ballerina init' command goes in a loop when you keep on providing invalid inputs

Description:
'ballerina init' command goes in a loop when you keep on providing invalid inputs. Ideally this should exit after attempting for 3 times.
Steps to reproduce:

Create an empty directory
Run 'ballerina init -i'
For every option prompted keep on pressing invalid inputs
The command will go in a loop without exiting until you press "f" (finish)

Affected Versions:
0.981.2-SNAPSHOT
OS, DB, other environment details and versions:
Ubuntu

From ballerina 1.0 alpha init command will be removed hence closing the issue.


New Quickstarts: Add section for observability

Should observability be more a thing we do in all the other quickstarts?  @msfussell @amulyavarote


Don't understand how to use - tutorial is broken on Pop_OS

Is there some kind of YouTube tutorial?  The tutorial is broken on Pop-OS - missing GTK module.
I don't know how to invoke Fly-Pie.  Some basics would be nice, right now it's completely broken.  Can't even start it.  Documentation is atrocious.

I am not sure where to start. First: if you want that I help you understand the software which I am developing voluntarily in my spare time, I would suggest not to call it it or its documentation atrocious.
Second: You completely ignored the bug report template. So I have to spent my time asking you for all those things. What GNOME Shell version are you using? Wayland or X11? What Pop-OS version? What is broken? How can the issue be reproduced? Is there an error message logged? How am I supposed to help you if all I get from you is "completely broken"?
Third: If you are missing some vital information in the first steps guide, I would happily receive a pull request!

Sorry - CNTRL+SPACE is all that I needed.


"oh" for zero in small numbers no longer works as of #993

As mentioned in https://github.com/knausj85/knausj_talon/pull/993/files#r990815249, merging the PR #993 removed the ability to say "oh" as a synonym for "zero" in small numbers, so we technically changed the grammar of numbers.
Personally I found "oh" prone to misrecognitions and had carefully changed all my number captures to avoid allowing it at the start of a number - so I can say one oh one but not oh oh seven, for instance. So I won't run into this issue.
Is it worth fixing this and restoring "oh"? Or, conversely, might it be better to remove "oh" from more places?

I use 'oh' quite a bit and do not find that this triggers misrecognitions for me. I dictate a lot of large (ID) numbers, for example I just said "30063801" as "three hundred over sixty three eight oh one". This did not break with #993, but I agree with you that I don't ever use "oh" at the beginning of a number, I'll always use "zero" if it comes to it.


chore(DEVHAS-657): Update application-api references

What does this PR do?:
This PR updates all references to the application-api using the new organization (konflux-ci)
Which issue(s)/story(ies) does this PR fixes:
https://issues.redhat.com/browse/DEVHAS-657
PR acceptance criteria:

[ ] Unit/Functional tests

[ ] Documentation

[ ] Client Impact

How to test changes / Special notes to the reviewer:

FYI changes here may conflict with #480, I'd recommend merging #480 first.

/retest

/retest

/retest

/retest

/retest

/retest as the e2e-tests PR is updated


New Utility Functions for *ByExample Support

Resolves #118

Coverage remained the same at 100.0% when pulling e70ab371d9b604ba435a91afbc7eb995fb4aebd7 on jeffgbutler:new-style-by-example into b79c09226e8d4a7ce925789a24adefa7493bdc5d on mybatis:master.


Implement Merkle Proof Generation and Validation API

For working on things like Plasma and Ethereum 2.0, the Merkle Tree data structure is very critical, especially the generation and validation of merkle proofs. It would be beneficial to implement an API to make this easy where generation takes a key and makes a witness (for the current value in the tree), and validation takes a key, value, and a specific witness and validates that the provided data and witness are valid with the current root of the tree.
The validation part might be done as a separate utility function that could obtain the merkle tree root for that proof and specific key/value pair, and then validates that the root hash matches. This would allow others to validate proofs generated against prior values of the root hash for things like Plasma that need to work with validation of proofs against prior tree roots.

Fixed with #77


Machine Learning not showing up on context menu

System Information (please complete the following information):

Model Builder or CLI Version: not sure where to find
Visual Studio Version (if applicable): 2019 v16.8.3

Describe the bug

On which page of the tutorial did you run into an issue (URL): https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/create
Clear description of the problem: "Machine Learning" is not on the "Add" menu

To Reproduce
Steps to reproduce the behavior:

Right-click on Project
Click Add
Don't see Machine Learning

Expected behavior
I expect to see "Machine Learning" on my "Add" context menu
Screenshots
N/A
Additional context
Happened while following the tutorial step-by-step, new project. I was able to work around by installing the Model Builder as an extension instead, although this is not the way the tutorial instructed. I think you should add this as an alternate way to get this working in the tutorial.

Same for me, project context menu does not contain the Add-> Machine Learning menu item. Updated to VS 16.8; followed the steps to enable ML but no context menu item.

dup of #969

Close due to no response. Feel free to reopen it if you have any questions.


Compliance w/Export Filing Notice

The Problem:
According to legal we need to submit a notification to the BIS when appropriate.
The Fix:
Add documentation to the product release checklist.

This addresses https://github.com/drud-admin/legal/issues/10

Nudge. This looks a bit stale. Is there any real reviewing to happen here, or is it something we're required to have?
if it's the latter, I think we should just move forward with it. If it's the former, can we get some instructions on how to best review this?


add missing rule in Makefile that's breaking builds

My attempt to fix https://github.com/istio/istio/issues/3858
I made two attempts to avoid the breakage prior to submit of https://github.com/istio/istio/pull/3820:
https://github.com/istio/istio/pull/3820#discussion_r171112591
https://github.com/istio/istio/pull/3820#discussion_r171139201
In this change I also remove some space that emacs warns about being suspect (but is probably okay since it's continued from the prior line).
I confirmed that with this change an attempt at "make deb" gets to the point of trying to run fpm (which fails on my system since I don't have it installed, but the point is is compiled istio and got to this step okay).

@mattdelco: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
7ffd42beb9c317e4886e38d205184f86e6c42dc6
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

To be more explicit about the issue, in the Makefile each of the main go executables has two targets.  The first target is in the form of the full path to where the executable is created (e.g., "$(ISTIO_OUT)/istioctl") while the second is basically an "phony" alias (e.g., "istioctl") that allows devs to conveniently build a particular executable (e.g., "make istioctl").
The docker and debian targets have dependencies on the full-path target (this helps avoid unnecessary recompiles during rebuilds since make always wants to rebuild "phony" targets), and the one for istioctl was dropped in the earlier PR (hence make's complaint that it doesn't know how to build istioctl).
For the debian image the dependency is set up in tools/deb/istio.mk using the list in ISTIO_DEB_DEPS (and the foreach loop that comes a few lines later) to declare a dependency on each of the go binaries listed in ISTIO_DEB_DEPS (using the full-path target) and also set up a SIDECAR_FILES variable that's used to tell fpm to add each of these listed go binaries to the debian image.
This change adds back the missing target for $(ISTIO_OUT)/istioctl.  You might be used to seeing something more of the form:

$(ISTIO_OUT)/istioctl: dependencies
       recipe
istioctl: $(ISTIO_OUT)/istioctl

though a different form that has both targets borrow the same recipe is:

istioctl $(ISTIO_OUT)/istioctl: dependencies
       recipe