Function that automatically picks up newer data

Suggested by @KellerFuchs
Add a function (perhaps named psl_default).
If psl_builtin_outdated() it should return psl_load_file(psl_builtin_filename())
Otherwise, return psl_builtin()
This might be useful if some distro ships a data package which updates a file in the right place.

The way to go (with libpsl 0.15+) is trying to load a given file (e.g. on Debian it is /usr/share/publicsuffix/public_suffix_list.dafsa from package publicsuffix). That file is hard-coded or a ./configure option (like for wget). Only if that fails, the application should fall back to the algorithm that you suggest (done for wget).
Why should the application implement that code and not libpsl ?
The app might want to tell the user that something is missing or should be changed for optimal functionality. Your suggestion just returns a valid psl pointer... of course psl_default() could take a pointer to a result, but than it starts to become ugly (IMO).

@rockdaboot The application can anyway do that test, simply by checking how old the data it is using is.
The point there is to offer a function which does the right thing by default, if possible.  Otherwise, the likeliest outcome is that most applications will silently use obsolete data.

OK, but then the builtin filename should point to the DAFSA representation (e.g. on Debian it currently points to the original TEXT file - reading it introduces lot's of processing).
I'll have a look in the next days.

Please check/review branch 'psl_latest' if that fits your needs.
If your distribution has a package with PSL data, specify that PSL file with ./configure --with-psl-distfile= (DAFSA file if possible). psl_latest() gives back whatever is the newest/latest datain (psl-distfile, application provided file, psl builtin data, file that psl builtin data is derived from).

@rockdaboot Looks good to me, modulo the comments I left on the commit.

Thanks for feedback. Added/amended comments and pushed.


Help setting up log4j

log4j is not working.  Error:
log4j:WARN No appenders could be found for logger (com.mchange.v2.log.MLog).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

Configuration of development.properties:
#log4j
# Set root logger level to DEBUG and its only appender to A1.
log4j.rootLogger=DEBUG, A1

# A1 is set to be a ConsoleAppender.
log4j.appender.A1=org.apache.log4j.ConsoleAppender

# A1 uses PatternLayout.
log4j.appender.A1.layout=org.apache.log4j.PatternLayout
log4j.appender.A1.layout.ConversionPattern=%-4r [%t] %-5p %c %x - %m%n

I'm getting that error and no output to console or in mamute/logs/logs

Mamute already provide an log4j.xml for you to use. Make sure it is on your classpath.
The file you used(development.properties) is exclusive for vraptor-environment properties, thats why it didn't work

I also get this error and log4j.xml is present on the classpath at WEB-INF/classes/log4j.xml


PadLocal: Apply for new token

Wechaty Puppet PadLocal

Repo: https://github.com/padlocal/wechaty-puppet-padlocal
Provider: PadLocal
Support: https://github.com/padlocal/wechaty-puppet-padlocal/issues

Question
I need a token for my first use, thank you

@Yinetsrl Can you send an email to<EMAIL_ADDRESS>Then I can send you related materials.


Any requirements or readme?

Hello, I am interesing in your work. Could you please provide the requirements for the package to be installed? Thanks!

We apologize for the oversight of not providing the requirements yesterday, which have now been provided now. We sincerely apologize for any inconvenience this may have caused and appreciate your understanding. Thank you for your time.


Chapter 10 Exercise: ListsAndRecursion-5

Dear Carlos
Great solutions, it helps me alot!
The solution of the 'take' function is not completely correct.
take([1,2,3,4], -1) should return [4] and not [1,2,3].
I created a "reverse function" that reversed the list to [4,3,2,1] and passed 1 (instead of -1) as n and reversed the returned list again. It might not be the most efficient solution though.

@kristofvanwoensel Thanks for spotting this bug! I've added the reverse function as well, I guess it is good enough for the exercise. For better performance, I would use the same approach as the standard library, details here


fix: minor improvement of back button scroll

Description
Fixed the scrolling after pressing custom back button from single course hero
Style fixes of the categories for tab navigation
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Codecov Report

Merging #35 (ab794b1) into main (a54e346) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   66.86%   66.83%   -0.03%
==========================================
  Files         240      120     -120
  Lines        5643     2822    -2821
  Branches     1225      613     -612
==========================================
- Hits         3773     1886    -1887
+ Misses       1714      858     -856
+ Partials      156       78      -78

Impacted Files
Coverage Δ

...inki/src/domain/event/eventCard/LargeEventCard.tsx
100.00% <ø> (ø)

apps/events-helsinki/src/utils/routerUtils.ts
66.66% <50.00%> (-1.26%)
:arrow_down:

...inki/src/domain/event/eventCard/LargeEventCard.tsx

apps/hobbies-helsinki/src/utils/routerUtils.ts

.../hobbies-helsinki/src/domain/neighborhood/query.ts

...rc/domain/i18n/serverSideTranslationsWithCommon.ts

...obbies-helsinki/src/assets/icons/IconLanguages.tsx

...common-events/components/shareLinks/ShareLinks.tsx

...pps/hobbies-helsinki/src/assets/icons/IconTree.tsx

...i/src/domain/search/eventSearch/AdvancedSearch.tsx

... and 112 more

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


Native Windows compatibility

Replace Cygwin with native Windows code, if not too much work. Requires a copy of Windows and a copy of Visual Studio.

What about the new shell in Windows 10 Creators Update?

dcf77pi works fine in the Linux subsystem shell of Windows 10.


Add labs: Small separate web apps for experiments and validation

Defaults:

[x] UI components

Done


[Refactor]: use bufio to reader a file in chunks.

used bufio instead os.ReadFile
deleted readHeader and byteRange, instead used io.ReadFull
added a flag filePath to provide a path to the EDF file


[eas-cli] Allow users to skip metadata validation

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
If we have a potential issue in the metadata validation, we still want users to be able and try it anyways. The user should be "informed" that the current config state might cause issues later on.
How

Test Plan

Create a store.config.json
Add an apple.info[en-CA] with only title
$ eas metadata:push

/changelog-entry new-feature Allow users to skip metadata validation


Blog header image for an article titled ' Persisting chart data in Node-RED Dashboards'.

Related Issue
https://github.com/flowforge/content/issues/116
Description
It's for this issue - https://github.com/flowforge/content/issues/116
I would like the artwork for Thursday 25th May morning.
Happy to chat over ideas, I appreciate it's not obvious (well at least to me) what the image should be.

Hi @robmarcer here it is


Run plex as non-root if possible

These changes will have the docker container determine the group that owns the /data directory, and then makes sure that the plex user is a member of that group.  This is hopefully a fix for timhaak/docker-plex/#17.

Please note that I only had about 30 minutes to test this, and will be heading out of town for the rest of the week.  It worked flawlessly for me.  Feel free to test and comment away, I'll catch up next week.

Really cool idea :)
Far better than any I had come up with.
I'll go over it properly on Friday.


feat: add Class attribute to dialogs created using DialogService

🙋 Feature Request
Since I'm using Tailwind for styling, I want to add some classes to a panel created using the DialogService to make its width responsive. However, this is currently not possible.
💁 Possible Solution
DialogParameters should have an additional attribute Class that gets applied to the control part of a dialog.

Why not to create a Customized Dialog where you have all possibilities to add your styles and classes?

Workaround provided. Closing this

I can't find a way to remove the Dialog Padding, even with a customized dialog

You can use a style like this:
<style>
    fluent-dialog::part(control) {
        padding: 0px !important;
    }
</style>


Jupyter - Kernel not starting / connecting

I am on Debian Stretch (on a server) and running jupyterhub.
The kernel is detected and I can create rust notebooks, but the kernel won't start.
The debug log is not very helpful for me and I dont get the purpose of the random ports:
[D 2019-04-16 20:27:51.406 SingleUserNotebookApp manager:257] Starting kernel: ['/home/joshua/.cargo/bin/evcxr_jupyter', '--control_file', '/home/joshua/.local/share/jupyter/runtime/kernel-4efeab69-259f-4a47-b2e2-791c485dea1a.json']
[D 2019-04-16 20:27:51.420 SingleUserNotebookApp connect:542] Connecting to: tcp://<IP_ADDRESS>:38513
[D 2019-04-16 20:27:51.423 SingleUserNotebookApp connect:542] Connecting to: tcp://<IP_ADDRESS>:60423
[I 2019-04-16 20:27:51.426 SingleUserNotebookApp kernelmanager:164] Kernel started: 4efeab69-259f-4a47-b2e2-791c485dea1a
[D 2019-04-16 20:27:51.427 SingleUserNotebookApp kernelmanager:165] Kernel args: {'kernel_name': 'rust', 'cwd': '/home/joshua'}
Error: JustMessage("No such file or directory (os error 2)")
[D 2019-04-16 20:27:51.544 SingleUserNotebookApp zmqhandlers:300] Initializing websocket connection /user/joshua/api/kernels/4efeab69-259f-4a47-b2e2-791c485dea1a/channels
[D 2019-04-16 20:27:51.549 SingleUserNotebookApp handlers:141] Requesting kernel info from 4efeab69-259f-4a47-b2e2-791c485dea1a
[D 2019-04-16 20:27:51.550 SingleUserNotebookApp connect:542] Connecting to: tcp://<IP_ADDRESS>:60475
[I 2019-04-16 20:27:54.423 SingleUserNotebookApp restarter:110] KernelRestarter: restarting kernel (1/5), new random ports
[D 2019-04-16 20:27:54.425 SingleUserNotebookApp manager:257] Starting kernel: ['/home/joshua/.cargo/bin/evcxr_jupyter', '--control_file', '/home/joshua/.local/share/jupyter/runtime/kernel-4efeab69-259f-4a47-b2e2-791c485dea1a.json']
[D 2019-04-16 20:27:54.435 SingleUserNotebookApp connect:542] Connecting to: tcp://<IP_ADDRESS>:58567
Error: JustMessage("No such file or directory (os error 2)")
...

and the control file:
{
  "key": "5ec08ec1-91b7d026239a28cf264b8a34",
  "shell_port": 35521,
  "hb_port": 39183,
  "signature_scheme": "hmac-sha256",
  "iopub_port": 52037,
  "control_port": 57881,
  "stdin_port": 49191,
  "ip": "<IP_ADDRESS>",
  "kernel_name": "rust",
  "transport": "tcp"
}

When I run /home/joshua/.cargo/bin/evcxr_jupyter --control_file /home/joshua/.local/share/jupyter/runtime/kernel-4efeab69-259f-4a47-b2e2-791c485dea1a.json manually I don't get the Error and evcxr_jupyter seems to be running fine:
TCP localhost:39183 (LISTEN)
TCP localhost:35521 (LISTEN)
TCP localhost:57881 (LISTEN)
TCP localhost:49191 (LISTEN)
TCP localhost:52037 (LISTEN)

What am I doing wrong? Is this a timing issue (evcxr_jupyter execs before control file exists)?
Thanks in advance!
Love your project and trying hard to get it working :+1:

I was seeing this same problem (JustMessage("No such file or directory (os error 2)")) when I was trying to install evcxr within an official jupyter docker container.  For me the problem was that jupyter was started in such a way that ~/.cargo/bin wasn't on the path.  So the "No such file or directory" error was from evcxr trying to spawn cargo as a subprocess (and being unable to find cargo on the PATH).
I can't say if that is also your problem, but that might be worth looking at.    To look at the value of $PATH for a running process, you can try to inspect it's environ file:
cat /proc/pid_of_jupyter/environ | xargs -0 -n1

from https://github.com/google/evcxr/blob/master/binder/postBuild (the script to run the kernel into a binder jupyter container):
#!/bin/bash

curl https://sh.rustup.rs -sSf | sh -s -- --default-toolchain=stable -y

echo '. $HOME/.cargo/env' >> ~/.bashrc
echo '. $HOME/.cargo/env' >> ~/.profile
source $HOME/.cargo/env

cargo install evcxr_jupyter
evcxr_jupyter --install

If the installation section of README should be completed, feel free to make a PR or to add into into this ticket.


Implement --tail with threads

There was a report from someone in #stacker about stacker build locking up when using --tail. I was able to repro pretty easily
$ stacker build --tail dev.yaml
[2018-02-28T15:43:18] Using default AWS provider mode
[2018-02-28T15:43:18] Tailing stack: ejholmes-vpc
[2018-02-28T15:43:21] ejholmes-vpc: submitted (creating new stack)
[2018-02-28T15:43:30] [ejholmes-vpc] CREATE_IN_PROGRESS AWS::CloudFormation::WaitConditionHandle
[2018-02-28T15:43:30] [ejholmes-vpc] CREATE_IN_PROGRESS AWS::CloudFormation::WaitConditionHandle Resource creation Initiated
[2018-02-28T15:43:30] [ejholmes-vpc] CREATE_COMPLETE AWS::CloudFormation::WaitConditionHandle
[2018-02-28T15:43:30] [ejholmes-vpc] CREATE_COMPLETE AWS::CloudFormation::Stack
[2018-02-28T15:43:52] ejholmes-vpc: complete (creating new stack)
[2018-02-28T15:43:52] Signal SIGTERM received, quitting (this can take some time)...

When using processes, calling watcher.terminate() would send the process a SIGTERM, which is now handled here. That would have no effect in the forked process, and the step would block on the following watcher.join().
Since we're now using threads for plan execution, I think it's simpler to just use threads for the implementation of tail, which fixes signal handling and the lockup.

Codecov Report

Merging #545 into master will decrease coverage by 0.02%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #545      +/-   ##
==========================================
- Coverage   87.56%   87.54%   -0.03%
==========================================
  Files          95       95
  Lines        6106     6108       +2
==========================================
  Hits         5347     5347
- Misses        759      761       +2

Impacted Files
Coverage Δ

stacker/providers/aws/default.py
66.29% <40%> (-0.19%)
:arrow_down:

stacker/plan.py
91.2% <60%> (+0.04%)
:arrow_up:

stacker/dag/__init__.py
87.81% <0%> (-0.51%)
:arrow_down:

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 1c96ab5...7e14eee. Read the comment docs.


MudSwitch no longer works when contained within a MudMenu

Bug type
Component
Component name
MudSwitch
What happened?
In this example, the value of isChecked is not changed when the "Example" switch is clicked on.  This worked when I was using MudBlazor 5.0.14, does not work in 6.0.4, 6.0.10, etc.
`
@page "/md"
         <MudMenu<EMAIL_ADDRESS>Label="View">
           <MudMenuItem><MudSwitch @bind-Checked="isChecked">Example</MudSwitch></MudMenuItem>
        </MudMenu>

@code{
private bool isChecked = false;
}`
Expected behavior
Value of isChecked changes when switch is clicked on, and visual state of switch reflects that change.
Reproduction link
n/a
Reproduction steps

See description code

...
Relevant log output
No response
Version (bug)
6.0.4
Version (working)
5.0.14
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You can try to use switch individually, not wrap with MudMenuItem

That works as a workaround.  Behavior is a bit different - the menu doesn't dismiss when a switch is clicked - but I can work around that.

Here is another workaround: https://github.com/MudBlazor/MudBlazor/issues/8287#issuecomment-1991119465

Closing in favor of #8287


Belinda's-Closet-NestJS_3_47_Unit-Tests-User-Controller-and-Service

Resolves #47
I started the testing for the User Controller and Service and was thinking the User needs to be refactored to use Data Transfer Objects (DTOs). We currently use a DTO to signup a User and one to login a User in the Auth Controller, and we also use a DTO to create a product and one to update a product in the Product Controller.
I know it is commented to serialize the User as well, but I do think we should probably start by creating DTOs for the User Controller.
What does everybody think? @bcko @brinkbrink @taylorpapke @theGaryLarson @tinpham5614

@CourtneyHoppus Yes, the user module hasn't been updated yet.
I just modified the name in the user module to firstName and lastName to match the frontend input format.
Issue ref #46

Passed all cases (user mobile).

One minor thing that I'm getting a warning about is using "const" instead of "let" for mockService, mockUser, and mockUpdateUser.


correctly discover the Alertmanager cluster and ServiceMonitors

Not sure how I missed these earlier but worth trying out I would say.
@fabxc @alexsomesan

👍
At some point we have to investigate what needs to be added to allow the k8s Prometheus to also be purely generated.

Yes I was thinking of exactly the same thing. I will investigate tomorrow what we can get out of self-hosted today and what we still need.

@fabxc as a follow-up there will be a point where all components will be self-hosted, so we can already come up with changes that will replace this manually maintained prometheus config with a replacement using the ServiceMonitor. This will come relatively soon, what's left is finishing work on bootkube regarding self hosted etcd, flannel and improvements on checkpointing.


Markdown improvement: A blank line doesn't separate block quotes

In the Ghost editor,
> foo

> bar

and
> foo
>
> bar

both result in something like:

foo

bar

But according to the CommonMark specification, which makes a lot of sense, it should produce something like:

foo

bar

I'm not saying you guys should make a blank line in between to indicate separation of two consecutive block quotes, since there's no definite Markdown standard yet. But there should be a way to separate consecutive block quotes right?

Hi @diihunts,
Ghost's built in editor does not conform to CommonMark, nor it is expected to, so the behaviour you're experiencing is correct. There's a discussion on whether the editor should adhere to CM spec here: https://github.com/TryGhost/Ghost/issues/3959, started by me, but that's fairly stale. At the moment the decision is that it does not.
Hope this helps.

For reference, Babelmark: http://johnmacfarlane.net/babelmark2/?normalize=1&text=>+foo

>+bar
This is how showdown and many other markdown libraries interpret the markdown for blockquotes. We depend on showdown, so if you wanted to start a discussion around changing to use the commonmark version, it would be better to start it on the showdown repo :)

Thanks @javorszky @ErisDS for the replies. You're right, I shouldn't bring it up here.
I'm still not sure. Do I have to switch back to HTML to write consecutive block quotes?
By the way, great job with Ghost, really appreciate it :)

@diihunts you can use an invisible character between block quotes:
> foo

> bar

@ErisDS thanks, that kind of does the job, but it also results in an extra empty paragraph right? I guess I'll just use html in those circumstances.


Race condition in resource count while associating IP address to a network

ISSUE TYPE

Bug Report

COMPONENT NAME

Resource Limits Manager for public_ip

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

Due to certain race conditions between checking and incrementing of resource counts, the check can be bypassed. This leads to cases where account's allocated resources exceeds the accounts limit.
STEPS TO REPRODUCE

Create an account with public ip limit to 2
Create a network and deploy a VM (This will increase the public_ip count by 1 for the account).
Allocate different IPs to the same network in parallel (around 4-5 requests).

This sometimes leads to allocated resources exceeding the account's limit.

EXPECTED RESULTS

Count of allocated resources  should not exceed the account's limit.

ACTUAL RESULTS

Count of allocated resources  exceeded the account's limit.

@vishesh92 is this new behaivure? what version can you (not) see this in?

@vishesh92 is this new behaivure? what version can you (not) see this in?

I am not sure.
This is happening because of the race conditions while incrementing the public_ip count. We need to create a CheckedReservation before allocation of the public_ip.

fixed by #9516


Separate pass or fail status not appearing for iOS XCTest

For iOS XCTest, only a single test case with name 'testIntegrationTest' is appearing in the test case result summary.
It does not help to identify which test case is passing or failing. There should be separate test cases with respective pass or fail status.
I had raised this issue with Firebase Support team and after some debugging they came up with below response -
"Regarding this issue, after a thorough investigation by our team, it would be best to raise this to Flutter's GitHub Issues so that it can be triaged by the correct folks since this doesn't appear to be limited to FTL and is more of an issue with open source Flutter. I know it could be tedious and frustrating, but this would be helpful in resolving your issue. Thanks for understanding."
Flutter Version - 2.10.5(Stable)
XCode version used - 13.2.1
iPhone model/device - iPhone 13 Pro
iOS version - iOS 15.2

@umangdivyanshu
This seems to have been fixed as part of this issue: https://github.com/flutter/flutter/issues/95648 and per this comment, the fix should be available on latest master.
PR that fixed this issue: https://github.com/flutter/flutter/pull/95704
So please switch to latest master and try again to see the expected behavior.
flutter channel master
flutter upgrade

Hi @darshankawar
We are not able to test the fix as Firebase Test Lab is on Stable Channel.
I have asked them if we could switch to Master Channel during runtime but there is no provision to do it.
I have also asked if they could switch to Master channel from backend so that we could test the fix. Waiting for their response on this.
When can we expect this fix to be available on Stable Channel?

When can we expect this fix to be available on Stable Channel?

/cc @jmagman for insights on this.  For your reference: https://github.com/flutter/flutter/issues/103020#issuecomment-1117036389
Also, since this has been fixed on master channel, I think it is safe to close it. If otherwise, I'll reopen it.

This is available on 2.11.0-0.1.pre, which is available in the beta channel.  It will be in the next stable.
https://github.com/flutter/flutter/wiki/Where's-my-Commit%3F#finding-the-versions-that-contain-framework-commit-x
From https://github.com/flutter/flutter/wiki/Flutter-build-release-channels#stable

Roughly once a quarter, a branch that has been stabilized on beta will become our next stable branch and we will create a stable release from that branch.

Also, since this has been fixed on master channel, I think it is safe to close it.

Yes, this is fixed on master so should be closed.

@jmagman Is this fix available now on Flutter 3.0.0 stable channel?

Yes it's available in 3.0. If you're seeing issues please file bugs and we'll take a look.


[DEV-1245] add a Saturn-y display name on saturn jupyter kernel

saturn images come with a default conda environment called saturn. Those images also contain a Jupyter notebook kernel built from that conda environment.
Today in Saturn, that environment's display name is "Python 3". Based on support work I've done with customers, I think this can be confusing. This PR proposes adding a saturn-y display name so that it's a little bit more obvious that you're in a Saturn-created kernel.

How to test this

Replace saturn/environment.yml with this smaller env (for speed):

name: saturn
channels:
- defaults
- pytorch
- saturncloud
- conda-forge
dependencies:
- ipykernel
- ipywidgets
- pip
- pip:
  - prefect-saturn>=0.4.0
  - dask-saturn==0.2.0

Run a container registry locally in k8s, following the instructions at https://github.com/saturncloud/saturn#working-with-local-registry.

Build an image and push it to that local registry.

cd saturn
IMAGE_TAG=localhost:32000/saturncloud/saturn-kernel-test:dev
docker build \
    --build-arg SATURNBASE_IMAGE=saturncloud/saturnbase:2020.12.11 \
    -t ${IMAGE_TAG} \
    .

docker push ${IMAGE_TAG}

In saturn/.env_deps, change SATURN_IMAGE

SATURN_IMAGE=localhost:32000/saturncloud/saturn-kernel-test:dev

start up saturn

make clean
make aws-creds run

Edit the examples-cpu Jupyter to use image localhost:32000/saturncloud/saturn-kernel-test:dev. Start it. Go into Jupyter Lab.

Open any notebook. In the top right corner, you should see saturn (Python 3).

Click on that to change the kernel. In the dropdown that pops up, you should see saturn (Python 3).

Task linked: DEV-1245 [images] give Saturn jupyter kernels a saturn-y name

Task linked: DEV-1245 [images] give Saturn jupyter kernels a saturn-y name

Just a note that this will not change the name of kernels on images built via saturn2docker, but that is out of scope here.

Oh yeah good point! I think it would be really straightforward to do that in saturn2docker too, can do that in a separate PR.

Just a note that this will not change the name of kernels on images built via saturn2docker, but that is out of scope here.

Oh yeah good point! I think it would be really straightforward to do that in saturn2docker too, can do that in a separate PR.


Migrate network related conf to structured conf system

[BUG]/[FEATURE] title
What changes were proposed in this pull request?
Why are the changes needed?

What are the items that need reviewer attention?
Related issues.
Related pull requests.
How was this patch tested?
/cc @related-reviewer
/assign @main-reviewer

@waitinfuture I addressed all comments now, please take a look again


Using Wonka.fromObservable in Reason API

Lets say you have and observable graphql subscription which in JS looks like:
	useEffect(() => {
		const subscription = API.graphql(graphqlOperation(onCreateMessage)).subscribe({
			next: (event) => {
				if (event) {
					console.log('Subscription: ' + JSON.stringify(event.value.data, null, 2));
					console.log('EVENT: ' + JSON.stringify(event, null, 2));
					setDisplay(true);
					let message = event.value.data.onCreateMessage.message;
					setMessage(message);
				}
			}
		});

The subscribe function is return a zen-observable which i believe is the same one returned by Wonka. See: https://github.com/aws-amplify/amplify-js/blob/86597db2b0704d7aff5b612557536142b82e1731/packages/api/src/API.ts#L16
How would I access the return value using Wonka.fromObservable?
This is what I have tried:
type t;
<EMAIL_ADDRESS>"@aws-amplify/api"] external api: t = "default";

<EMAIL_ADDRESS>external _graphqlSubWonka:
  (t, Types.graphqlOperation) => Wonka.observableT('a) =
  "graphql";

graphql is binding to API.ts#L352
In useEffect i have got:
  React.useEffect(() => {
    let subRequest = Graphql.OnCreateMessage.make();
    let graphqlOperation: Types.graphqlOperation = {
      query: subRequest##query,
      variables: subRequest##variables,
	};
    API.subWithWonka(graphqlOperation)
    |> Wonka.fromObservable((. result) => Js.log(result));
    None;
  });

This yield the following error which I do not understand how to read:
Error: This expression has type (. 'a) => unit
       but an expression was expected of type
         Wonka.observableT('b) = Wonka_observable.observableT('b)

I have tried a bunch of other things but just give this simple example to get the conversation started. Thank you for sharing your project.

This seems to compile and log:
    let sub = API.subWithWonka(graphqlOperation);
    let _ = sub |> Wonka.fromObservable |> Wonka.subscribe((. x )=> Js.log2("obs",x));

Though any additional feedback would be greatly appreciated.

I'm not quite sure whether I have enough information here to give an informed answer, since some parts of this seem to be missing.
What for instance is API.subWithWonka and why does it return an Observable rather than a Wonka stream, if it does include the name "Wonka"?
There are multiple ways to work with results and those are all part of the "sinks" section here: https://wonka.kitten.sh/api/sinks
The question is what you're then doing with the result?
So suppose you have zen-observable's Observable, passing it into Wonka.fromObservable converts it into a Wonka stream. You can then subscribe to this stream using Wonka.subscribe which accepts a function that is called for each value in your stream and returns a subscription (which can be used to unsubscribe)
But depending on what you're doing you're probably then either looking to just consume this value or transform it?

Lots of good information in there. Let me review and also clean up the file and share it so you can have some context.

Not yet understanding the sinks and all, I hacked it with an Obj.magic call here:
https://github.com/idkjs/realtime-reason-aws/blob/eaddf08a56f536f8a13d46be6498079491838f39/src/aws/API.re#L45-L63
<EMAIL_ADDRESS>external _subscribe:
  (t, Types.graphqlOperation) =>
  Wonka.observableT(Types.observableLike(Types.observerLike('value))) =
  "graphql";
let subscribe = graphqlOperation => _subscribe(api, graphqlOperation);

let extractMessageFrom = event => {
  /* use Obj.magic to change event, otherwise code in Wonka.subcribe breaks. */
  let event = event->Obj.magic;
  /* get the message value on event and post to ui */
  let message = event##value##data##onCreateMessage##message;
  message;
};
/* setting up like this returns the message on which we can call `setMessage()` */
let subscribeToMessage = graphqlOperation =>
  _subscribe(api, graphqlOperation)
  |> Wonka.fromObservable
  |> Wonka.map((. event) => extractMessageFrom(event));

I wrote this post up to try to get a handle on what I am seeing. https://t.co/tom1nWOfqR?amp=1. I would like to invite you to take a look at it and tear it apart. I think it would be a good start to putting some materials about Wonka in the wild. Even though I don't totally get it yet, this library feels in my gut, extremely important. Peace to you. Thanks for sharing.


issue when build the catkin

We use GitHub issues only to discuss PX4 Avoidance bugs and new features. For
questions about using PX4 or related components, please use PX4 Discuss.
when i build the catkin. there is a warning in the process.
i want to know if it will influence the result, or how can i resolve it.
thank you!
`Workspace configuration appears valid.
[build] Found '4' packages in 0.0 seconds.
[build] Package table is up to date.
Starting  >>> avoidance
Finished  <<< avoidance                           [ 0.6 seconds ]
Starting  >>> global_planner
Starting  >>> local_planner
Starting  >>> safe_landing_planner
Finished  <<< local_planner                       [ 30.6 seconds ]
Finished  <<< safe_landing_planner                [ 31.2 seconds ]

Warnings   << global_planner:make /home/coamc/avoid_catkin_ws/logs/global_planner/build.make.003.log
In file included from /opt/ros/melodic/include/ros/ros.h:40:0,
from /opt/ros/melodic/include/pcl_conversions/pcl_conversions.h:42,
from /home/coamc/avoid_catkin_ws/src/avoidance/global_planner/include/global_planner/global_planner_node.h:16,
from /home/coamc/avoid_catkin_ws/src/avoidance/global_planner/src/nodes/global_planner_node.cpp:1:
/home/coamc/avoid_catkin_ws/src/avoidance/global_planner/src/nodes/global_planner_node.cpp: In member function ‘void global_planner::GlobalPlannerNode::depthCameraCallback(const PointCloud2&)’:
/opt/ros/melodic/include/ros/console.h:348:176: warning: format ‘%s’ expects argument of type ‘char*’, but argument 8 has type ‘std::__cxx11::string {aka std::__cxx11::basic_string}’ [-Wformat=]
::ros::console::print(filter, _rosconsole_define_location__loc.logger, rosconsole_define_location__loc.level, FILE, LINE, ROSCONSOLE_FUNCTION, VA_ARGS)
^
/opt/ros/melodic/include/ros/console.h:351:5: note: in expansion of macro ‘ROSCONSOLE_PRINT_AT_LOCATION_WITH_FILTER’
ROSCONSOLE_PRINT_AT_LOCATION_WITH_FILTER(NULL, VA_ARGS)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/opt/ros/melodic/include/ros/console.h:387:7: note: in expansion of macro ‘ROSCONSOLE_PRINT_AT_LOCATION’
ROSCONSOLE_PRINT_AT_LOCATION(VA_ARGS);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
/opt/ros/melodic/include/ros/console.h:572:35: note: in expansion of macro ‘ROS_LOG_COND’
#define ROS_LOG(level, name, ...) ROS_LOG_COND(true, level, name, VA_ARGS)
^~~~~~~~~~~~
/opt/ros/melodic/include/rosconsole/macros_generated.h:162:23: note: in expansion of macro ‘ROS_LOG’
#define ROS_WARN(...) ROS_LOG(::ros::console::levels::Warn, ROSCONSOLE_DEFAULT_NAME, VA_ARGS)
^~~~~~~
/home/coamc/avoid_catkin_ws/src/avoidance/global_planner/src/nodes/global_planner_node.cpp:294:5: note: in expansion of macro ‘ROS_WARN’
ROS_WARN("Transformation not available (%s to /camera_link", frame_id);
^
cd /home/coamc/avoid_catkin_ws/build/global_planner; catkin build --get-env global_planner | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
`
Thanks!

They are warning on the ROS logging console outputs. They don't have any influence on the planner itself.


Replace the from-communication-copied text of CoS Health header

Refers to a correspondence with Surge team, where it was flagged that the Communications and Health has the same intro text.

@szabozoltan69, some commits here are not relevant to the hotfix. Please cherry-pick only the necessary commits for the hotfix. Based on the review, it seems we only need this commit: aafa45e. Thanks.

@szabozoltan69 I’ve cherry-picked your commit. Instead of merging it now, we’ll include it in an upcoming hotfix deployment.

included in https://github.com/IFRCGo/go-web-app/pull/1480


how to check if 1 prop is set?

e.g.
if aria-label or aria-labelledby is set, pass, otherwise fail.

like you want one or the other required, but not neither - and not both?

@ljharb, sorry yes. That’s correct.

Sounds like you’d use mutuallyExclusiveProps for that.


Handle notebook dependencies separately for reproducibility before creating an AI pipeline

As Data Scientist/Developer,
I want to use GitOps and have a Git repository where all my work is stored and can be reused. I would like to have a structured project so that everyone can find immediately all material used in my project. Therefore I would start my project reusing some templates like [1].
Once I have my project, using Elyra I can clone my repo and start working on my ML project. I would like to have reproducibility and traceability of my notebooks and material and I want to have a way to handle dependencies in my notebook, therefore I could use a jupyter extension for dependencies [2], that could also rely on AI support [3] for enhancements.
Having such an extension would allow the Data Scientist to work on the notebook and make all the experiments there before pushing everything and before creating AI pipelines.
I could use context directories for my notebooks, so that I can create images out of this context directories, each of them with different requirements and tag, each to be step in my AI pipeline that might have different resource requirements. Those images/context could be also created as template notebooks/images that can be reused and made available.
References:

[1] https://github.com/aicoe-aiops/project-template
[2] https://github.com/thoth-station/jupyter-nbrequirements
[3] https://thoth-station.ninja/

Just to clarify, this would be only in the context of executing a notebook as part of an AI pipeline, where if detected that there is a requirements.txt together with the notebook that would be used to install the dependencies listed there.
Supporting this on a local env would require a lot more thought and we probably would like to introduce the notion of projects to Elyra and make this a feature that requires the user to enabled and configure things like which env manager to use, where the binary is located, etc...

Just to clarify, this would be only in the context of executing a notebook as part of an AI pipeline, where if detected that there is a requirements.txt together with the notebook that would be used to install the dependencies listed there.

In the AI pipeline, you need to select the image to be used to run the specific step in the pipeline right?
I was thinking if I start a notebook through Elyra that I still don't want to run in a pipeline because it is not ready, I want to have requirements.txt or Pipfile/Pipfile.lock that are created when I install something in a notebook so that I can push to Git all material that can be reused by another person cloning the repo maybe.
And if someone else restarts the notebook, can detect dependencies used to run that specific notebook or if each notebook would have its dependencies and images can be created out of those notebook-dependencies to be used in different steps in AI pipeline.

Supporting this on a local env would require a lot more thought and we probably would like to introduce the notion of projects to Elyra and make this a feature that requires the user to enabled and configures things like which env manager to use, where the binary is located, etc...

By projects you mean instantiate some standard template for AI project?
Is this a feature that would apply per notebook? Each notebook doing different tasks might have different requirements.
What about a library that can handle all env managers? [4]
[4] https://github.com/thoth-station/micropipenv

Is this a feature that would apply per notebook? Each notebook doing different tasks might have different dependencies requirements.

When running pipelines in Elyra in 'local' mode, the python dependencies will be installed on the host system. So we will need some way to isolate requirements on a per notebook basis. Ideally we would create ephemeral python environment per notebook node ether with pip or conda.

Ideally we would create ephemeral python environment per notebook node ether with pip or conda.

Heads up.  This will break the Enterprise Gateway support added in #983 since its predicated on the fact that the node is running within the server's process - similar to how each notebook runs outside of the pipeline context today.

Ideally we would create ephemeral python environment per notebook node ether with pip or conda.

Heads up. This will break the Enterprise Gateway support added in #983 since its predicated on the fact that the node is running within the server's process - similar to how each notebook runs outside of the pipeline context today.

Dang, forgot about the EG enhancements. Will need to think on this more.

We could probably remove that restriction, and I'd view that as the long-range plan, but it would mean a heck of a lot more to configure besides just the gateway URL. I suppose the configuration could be extracted and made available to the new env, but just wanted to point out there's some sensitivity there.

Is this a feature that would apply per notebook? Each notebook doing different tasks might have different dependencies requirements.

When running pipelines in Elyra in 'local' mode, the python dependencies will be installed on the host system. So we will need some way to isolate requirements on a per notebook basis. Ideally we would create ephemeral python environment per notebook node ether with pip or conda.

So every notebook should be shipped with requirements.txt or Pipfile/Pipfile.lock so anyone can install and re run the notebook with the same environment.
Having in a notebook cell:
! pip install tensorflow
! pip install boto3
! pip install matplotlib

does not guarantee reproducibility.
Creating a virtualenv from dependencies files so the kernel of the notebook can use specific requirements? and this could be handled with a jupyter notebook extension that manages dependencies and create requirements file for the notebook:

if you restart the notebook, you can run the detect button, to find out the dependencies required as well:

@pacospace - Would you be able to attend our weekly dev meeting tomorrow 10/29 @ 9am PDT? It would be great to hash out all the details of this issue in a conf call. If not, could you let us know what timezone you are in and we can get a group chat going in our gitter channel?
Weekly dev meeting details: https://hackmd.io/SgvSqrWWR2248mCw2BZ5gg?both
gitter channel : https://gitter.im/elyra-ai/community

@pacospace - Would you be able to attend our weekly dev meeting tomorrow 10/29 @ 9am PDT? It would be great to hash out all the details of this issue in a conf call. If not, could you let us know what timezone you are in and we can get a group chat going in our gitter channel?
Weekly dev meeting details: https://hackmd.io/SgvSqrWWR2248mCw2BZ5gg?both
gitter channel : https://gitter.im/elyra-ai/community

@akchinSTC I'm in CET and the meeting time should be feasible! Very happy to join!

So, assuming we have nbRequirements on the pipeline execution runtime where the notebook is running  https://github.com/elyra-ai/kfp-notebook/pull/62 and that the user, at least for now, includes the files like requirements.txt or Pipfile/Pipfile.lock are we ok? We could subsequently add support for auto-include these extra files semi-automatically.

@akchinSTC @lresende https://www.youtube.com/watch?v=HUp2JARu6fw here is the video. Thanks for the nice chat!

@akchinSTC @lresende https://www.youtube.com/watch?v=HUp2JARu6fw here is the video/demo. Thanks for the nice chat!

Thank you for joining us and giving that demo on such short notice. It definitely helped clear up a few things. I think we have a few things to work off of/talking points now based on the chat.

nbrequirements extension Integration with JupyterLab
prototype a pipeline with s2i, nbrequirements/pip requirements and elyra

@akchinSTC @lresende https://www.youtube.com/watch?v=HUp2JARu6fw here is the video/demo. Thanks for the nice chat!

Thank you for joining us and giving that demo on such short notice. It definitely helped clear up a few things. I think we have a few things to work off of/talking points now based on the chat.

Thanks @akchinSTC

nbrequirements extension Integration with JupyterLab

We started working on that!

prototype a pipeline with s2i, nbrequirements/pip requirements and elyra

As part of our roadmap in Project Thoth: https://github.com/thoth-station/core/blob/master/docs/ROADMAP.md#pipelines-for-reproducible-builds. @harshad16 can show you what we have and use already. Let us know when you want to have another demo :)

https://github.com/thoth-station/core/blob/master/docs/ROADMAP.md#jupyter-requirements-management

@akchinSTC @lresende  https://www.youtube.com/watch?v=IBzTOP4TCdA

@pacospace - thanks paco, checking it out now!

@pacospace - thanks paco, checking it out now!

We created a public channel to interact with Thoth team as well: https://github.com/thoth-station/core#interact-with-thoth-team!

@akchinSTC

@akchinSTC @lresende https://www.youtube.com/watch?v=-_dtDAAyMlU&t=190s

@pacospace - thanks for all the input and work and most importantly patience around this Francesco! been wrapping up work around the new airflow feature and will be catching up with this issue today

@pacospace - thanks for all the input and work and most importantly patience around this Francesco! been wrapping up work around the new airflow feature and will be catching up with this issue today

No problem @akchinSTC :) when you want we can talk about jupyterlab-requirements [5] extension for jupyerlab.
Happy to join on a call and show you what it does live and we can discuss about it.

[5] https://github.com/thoth-station/jupyterlab-requirements

@pacospace, im trying out the lab requirements extension as we speak, nice...
avoiding the subject of local execution for a minute, @lresende  I think we will need to formalize our file and directory structure when using elyra pipelines, given that the overlays directory is generated in the root directory and needs to be scoped properly. https://github.com/aicoe-aiops/project-template is a really good start but I think we would want to make the default template a little more generic? or maybe provide the user with the option to select from a list of template structures to choose from to start.

Since v0.10.0 we introduced%horus magic commands to handle dependencies directly from notebook cells.
Check out the guide here or the video here


js: Use ScriptProcessorNode instead of AudioBufferSourceNode

Oto uses AudioWorklet if available, and uses WebAudio's AudioBufferSourceNode otherwise. For example, Safari does not support AudioWorklet yet.
The problem of AudioBufferSourceNode is that it is difficult or almost impossible to avoid clicking noise that happens during chunks. I tried various buffer sizes, but could not find a solution.
Although ScriptProcessorNode is deprecated, this seems the only way to treat PCM without noises if you cannot use AudioWorklet.

This is old


Capitalise name of logged in user in header

Is your feature request related to a problem? Please describe.
currently, the name is in lowercase.
Describe the solution you'd like
The name can be capitalised. eg. Ayush Sharma
Describe alternatives you've considered
Not required.

Additional context
Not required.

I'll take this one if is not already taken :)

All yours , go ahead :-)

@ayusharma @priscilawebdev I think you both worked on the same ticket. Sorry about that. @ayusharma PR was merge early this week  :(

It was still open, so I took it..but fine :)

Thanks for your effort @priscilawebdev

@priscilawebdev Sorry for the misunderstanding.  :(


Failed Compile DLib

Hi I get this error when trying to install DLib in Ubutu 14. Can anybody give some advices?, Thank you.
Build using cmake ...
Scanning dependencies of target dlib
[  1%] Building CXX object dlib_build/CMakeFiles/dlib.dir/base64/base64_kernel_1.cpp.o
[  2%] Building CXX object dlib_build/CMakeFiles/dlib.dir/bigint/bigint_kernel_1.cpp.o
[  4%] Building CXX object dlib_build/CMakeFiles/dlib.dir/bigint/bigint_kernel_2.cpp.o
[  5%] Building CXX object dlib_build/CMakeFiles/dlib.dir/bit_stream/bit_stream_kernel_1.cpp.o
[  6%] Building CXX object dlib_build/CMakeFiles/dlib.dir/entropy_decoder/entropy_decoder_kernel_1.cpp.o
[  8%] Building CXX object dlib_build/CMakeFiles/dlib.dir/entropy_decoder/entropy_decoder_kernel_2.cpp.o
[  9%] Building CXX object dlib_build/CMakeFiles/dlib.dir/entropy_encoder/entropy_encoder_kernel_1.cpp.o
[ 11%] Building CXX object dlib_build/CMakeFiles/dlib.dir/entropy_encoder/entropy_encoder_kernel_2.cpp.o
[ 12%] Building CXX object dlib_build/CMakeFiles/dlib.dir/md5/md5_kernel_1.cpp.o
[ 13%] Building CXX object dlib_build/CMakeFiles/dlib.dir/tokenizer/tokenizer_kernel_1.cpp.o
[ 15%] Building CXX object dlib_build/CMakeFiles/dlib.dir/unicode/unicode.cpp.o
[ 16%] Building CXX object dlib_build/CMakeFiles/dlib.dir/data_io/image_dataset_metadata.cpp.o
[ 18%] Building CXX object dlib_build/CMakeFiles/dlib.dir/data_io/mnist.cpp.o
[ 19%] Building CXX object dlib_build/CMakeFiles/dlib.dir/dnn/cpu_dlib.cpp.o
In file included from /home/mspl/Downloads/dlib/dlib/dnn/cpu_dlib.cpp:9:0:
/home/mspl/Downloads/dlib/dlib/dnn/tensor_tools.h:1305:36: error: ‘<::’ cannot begin a template-argument list [-fpermissive]
/home/mspl/Downloads/dlib/dlib/dnn/tensor_tools.h:1305:36: note: ‘<:’ is an alternate spelling for ‘[’. Insert whitespace between ‘<’ and ‘::’
/home/mspl/Downloads/dlib/dlib/dnn/tensor_tools.h:1305:36: note: (if you use ‘-fpermissive’ G++ will accept your code)
make[2]: *** [dlib_build/CMakeFiles/dlib.dir/dnn/cpu_dlib.cpp.o] Error 1
error: cmake build failed!

Looks like g++ 4.8 and earlier. What is the version of g++? Can you try some new version of compiler?

version 4.7.3  I think I missed setting CUDA into environment, so it not detect my CUDA, like root dir etc. How to export that path so can be detected during installation?

Here some previous error.
-- Looking for sys/types.h -- Looking for sys/types.h - found -- Looking for stdint.h -- Looking for stdint.h - found -- Looking for stddef.h -- Looking for stddef.h - found -- Check size of void* -- Check size of void* - done -- Found Intel MKL BLAS/LAPACK library CUDA_TOOLKIT_ROOT_DIR not found or specified -- Could NOT find CUDA (missing:  CUDA_TOOLKIT_ROOT_DIR CUDA_NVCC_EXECUTABLE CUDA_INCLUDE_DIRS CUDA_CUDART_LIBRARY) (Required is at least version "7.5") -- *** Dlib CUDA support requires C++11 but your compiler doesn't support it. ***

You need to use a newer compiler that supports C++11.


Fix sizeof(http_parser) assert

The result should be 32 on both 32 bits and 64 bits architectures
because of struct padding.
Fixes: https://github.com/nodejs/http-parser/issues/507

@indutny Can I get a quick LGTM?

@bnoordhuis This works on ARMv7, but fails on i686:
$ ./test_g
http_parser v2.9.4 (0x020904)
sizeof(http_parser) = 28
test_g: test.c:4240: main: Assertion `sizeof(http_parser) == 32' failed.
Aborted

Turns out the comment from stack overflow I read in https://github.com/nodejs/http-parser/issues/507#issuecomment-621403432 was not sufficient. The actual rule for this case is detailed in The Lost Art of Structure Packing.

In general, a struct instance will have the alignment of its widest scalar member

Which is not the same as being aligned to the size of the widest member. Apparently x86 and arm differ on the alignment requirements for 64-bit values, where arm requires 8-byte alignment and x86 requires 4-byte alignment. This particular case is covered by a note about arm and x86 compatibility from intel:

The 8-byte (64-bit) mVar2 results in different layout for TestStruct. This is because ARM requires 8-byte alignment for 64-bit variables like mVar2.

So it seems this test needs to encode the abi requirements.


[#178] Update to new dependencies and GHC 9

Description
Problem: new morley &c versions are out
Solution: update to use the new versions
One snag is that I had to use giveNotInView directly. It should probably be provided by morley-upgradable in mkEpwContract.
Related issue(s)

Resolves #178
:white_check_mark: Checklist for your Merge Request

Related changes (conditional)

Tests (see short guidelines)

[ ] If I added new functionality, I added tests covering it.
[ ] If I fixed a bug, I added a regression test to prevent the bug from
silently reappearing again.

Documentation

I checked whether I should update the docs and did so if necessary:

[ ] README
[ ] Haddock
[ ] docs/

[ ] I updated changelog unless I am sure my changes are
really minor.

Stylistic guide (mandatory)

[ ] My commits comply with the following policy.
[ ] My code complies with the style guide.

Not sure if this merits a changelog entry or not, there don't seem to be any user-facing changes.

The last fixup commit just updates stack.yaml.lock.


High severity vulnerability affecting this repo

Hi there,
We noticed that your repo has a high severity vulnerability:

Regular Expression Denial of Service
Affected package: minimatch <=3.0.1

Here is the test report for this repo.
If you’d like to fix this vulnerability, Snyk lets you generate a pull request that recommends the best upgrade path - there’s a link to fix this vulnerability on the test report.
Stay secure :-)
Snyk Community

This has been updated best as I can, but none of the node code for these libs are user facing, just toolchain.


Dockerfile

Hi Guys, I was using the original ciscocloud/marathon-consul in a docker image (for coreos) and used the Dockerfile supplied with the project. I have created a Dockerfile by pulling down the distro. I would prefer to build from source, I am a bit lost how to create a Dockerfile for your project as I have no real experience with golang... Can you anyone help please?
======================
Ok, managed to hack the ciscocloud/marathon-consul file, very simple just used the Makefile and removed the test tag from the build target.
The launch.sh is again from ciscocloud/marathon-consul
===== DOCKERFILE =====
FROM gliderlabs/alpine:3.2
MAINTAINER Brian Hicks<EMAIL_ADDRESS>RUN apk add --update ca-certificates bash
COPY launch.sh /launch.sh
COPY . /go/src/github.com/allegro/marathon-consul
RUN apk add make go git mercurial
&& cd /go/src/github.com/allegro/marathon-consul
&& export GOPATH=/go
&& make build
&& mv /go/src/github.com/allegro/marathon-consul/bin/marathon-consul /bin/marathon-consul
&& rm -rf /go
&& apk del --purge go git mercurial
ENTRYPOINT ["/launch.sh"]

Hi @jogster
Thanks for submitting issue. Your "hack" looks legit and if you prepare pull request I'll be happy to merge it.

Sure — let me figure out how to do that.

On 12 Apr 2016, at 08:34, janisz<EMAIL_ADDRESS>wrote:
Hi @jogster https://github.com/jogster
Thanks for submitting issue. Your "hack" looks legit and if you prepare pull request I'll be happy to merge it.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/allegro/marathon-consul/issues/71#issuecomment-208755419

Just forked the repo and added all my changes -- please pull it back to the main branch if OK.


[MINOR] 'return' is unnecessary as the last statement in a 'void' method

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3012 (ce84d54) into master (dcd7c33) will decrease coverage by 45.88%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3012       +/-   ##
============================================
- Coverage     55.15%   9.27%   -45.89%
+ Complexity     3851      48     -3803
============================================
  Files           485      54      -431
  Lines         23542    2016    -21526
  Branches       2522     241     -2281
============================================
- Hits          12985     187    -12798
+ Misses         9405    1816     -7589
+ Partials       1152      13     -1139

Flag
Coverage Δ

hudicli
?

hudiclient
?

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
?

huditimelineservice
?

hudiutilities
9.27% <ø> (-61.61%)
:arrow_down:

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


support of init.lua?

support of init.lua will come.?

Manage to find a solution for this?

If you have an init.lua it works as expected.


Documentation of setting up Teams Integration out of date

The screen captures are out of date.  Teams Integration is no longer available through System Settings

It is only through the Power Platform Admin Centre:

AND there is only one radial button now which needs more permission than a D365 System Administrator to power.

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

ID: d00d39b3-f151-358b-29e7-9a02125d53f8
Version Independent ID: 86fa36c6-c95d-232a-6b17-146dfa365414
Content: Install and set up the Dynamics 365 app for Microsoft Teams
Content Source: ce/teams-integration/teams-install-app.md
Service: crm-online
GitHub Login: @mduelae
Microsoft Alias: mkaur

Hi @Mbrmanning
Thanks for notifying us about this issue and sorry for the delay. I'll verify and update the doc accordingly.

Hi @Mbrmanning,
I checked with our engineering team. Apparently, the Setting in the Power Platform Admin Center has no effect on the Teams integration with Dynamics 365. I'm able to find the Teams integration settings in the System Settings page in our latest builds. If you don't see them in your environment, please contact our support team. Alternatively, you can enable the Teams integration from the Sales Hub App Settings page. For more information, see https://docs.microsoft.com/en-us/dynamics365/teams-integration/teams-install-app#for-admins-enable-microsoft-teams-integration-feature.
Hope this helps.

Hi @Mbrmanning,
I'm closing the issue. Please feel free to reopen if you feel the issue is not addressed completely.

Closing issue

Do what you like.  I was trying to be helpful.  It is the DOCUMENTATION that is out of date.  It is SUPPOSED to reflect what users see on the screen.  It does not do that.
Once again Microsoft drops the ball and then denies there’s any ball at all.
Thanks,
Mike Manning
Senior Solution Architect
@.***
200 Yorkland Blvd. Suite 920 • Toronto, ON M2J 5C1
T<PHONE_NUMBER> x 206  www.whitecapcanada.comhttps://www.whitecapcanada.com/
@.@.@.@.https://www.linkedin.com/company/whitecap-canada-inc-
From: lavanyakr01 @.>
Sent: Monday, August 9, 2021 2:05 AM
To: MicrosoftDocs/dynamics-365-customer-engagement @.>
Cc: Michael Manning @.>; Mention @.>
Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Documentation of setting up Teams Integration out of date (#2320)
Hi @Mbrmanninghttps://github.com/Mbrmanning,
I'm closing the issue. Please feel free to reopen if you feel the issue is not addressed completely.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics-365-customer-engagement/issues/2320#issuecomment-894971270, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJXQOLH5DXZZE5JMLNA4NNDT35VY3ANCNFSM5BCXNARA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.


Annotation for miRNAs

Can mirge3 output an annotation file (chromosome, start position, end position) for all miRNAs detected? Can mirge3 output the canonical mature miRNA?
I've created a script that examines the human library, but I am concerned about errors. This is important as I will attempt to use the BAM files to examine mismatches.
Examples of concern:

'hsa-miR-6718-3p' is outputted in the miR.Counts.csv file. However, only 'hsa-mir-6718' is found in human_miRBase.gff3 and it's a primary transcript (not mature miRNA). 'hsa-miR-6718-5p' does have a mature miRNA entry in human_miRBase.gff3.
'hsa-miR-1973' is outputted in the miR.Counts.csv file. However, it doesn't seem to be listed in human_miRBase.gff3 or human_MirGeneDB.gff3. Thus, I am confused exactly where that entry is recorded in the human library.
The annotation syntax between MiRBase and MirGene is different (e.g. hsa-miR versus Hsa-Mir). However, the final output of mirge3 appears to follow MiRBase syntax.
Lastly, there are numerous merged miRNA (e.g. hsa-miR-192-5p/215-5p).

Hi @John-Drake,
I have tried to cover all your questions in the form they were presented and in a simple way.
If you want the genomic coordinates, I have added them in the header of the FASTA file for each miRNAs. You could fetch and parse the header information to derive annotations as per your description. The example is shown below:
>hsa-let-7a-5p chr9 segs:1-30 cds:+:94175960-94175989
GATGAGGTAGTAGGTTGTATAGTTTTAGGG
>hsa-let-7a-3p chr9 segs:1-29 cds:+:94176011-94176039
AACTATACAATCTACTGTCTTTCCTAACG

To fetch this information, you can access the files here at path miRge3_Lib/human/index.Libs and use the following command:
bowtie-inspect human_mirna_miRBase > miRBase_annotations.fasta
or if you are interested in only header:
bowtie-inspect --names human_mirna_miRBase > miRBase_annotations.txt
example:
hsa-let-7a-3p chr9 segs:1-29 cds:+:94176011-94176039
hsa-let-7a-2-3p chr11 segs:1-30 cds:-:122146517-122146546

Please note: segs is the length of the sequence, and we have appended 2 nucleotides (nt) at 5' and 6 nt at 3' to each cannonical sequences. Besides that, cds is only a seperator and it does not mean coding sequences.

To answer your point one and two:

Where do you map the miRNAs which are in the output files but not in annotated gff3 file?
You can find the annotations as bowtie index files (miRge3_Lib/human/index.Libs), to view this file, you can execute bowtie-inspect miRge3_Lib/human/index.Libs/human_mirna_miRBase | less  . This command will show you what is indexed. Not all mature miRNAs and its hairpin miRNAs present in there respective fasta FASTA file is provided in the annotated gff3 file. I unfortunately don't have an answer for that, since we obtain gff3 file from miRBase repository and they don't provide annotations for all the miRNAs in gff3. Regarding hsa-miR-6718-3p, they have removed this in their latest revisions and unfortunately, the database report new in their change log (https://mirbase.org/cgi-bin/mirna_history.pl?acc=MI0022553). I wondre if this change was recent, as there is no time stamp for individual miRNAs.  Also, please note, from the experiments reported there, 5p is dominat miRNA where as no miRNAs are reported at 3p. This should suggest that, reads from your data will have no read counts towards 3p. However, we are updating our libraries keeping all these point in view and it will be out soon.
The same explanation goes for miRNA hsa-miR-1973

I have executed this SRR run SRR772403.fastq using both databases and you can see that the output has there respective names. The syntax of reporting/presenting counts and RPM is maintained the same irrespective of the databsae used. Plesae note, the annotation is not changed and it is presented as is. If you have found different output in your execution, please give us an example miRNA and we will follow up.

This is an example counts from MirGeneDB
Hsa-Mir-10-P1a_3p*,0.0
Hsa-Mir-10-P1a_5p,44794.0

This is an example counts from miRBase
hsa-miR-10a-3p,0.0
hsa-miR-10a-5p,44356.0

Discussion about merged miRNA can be found here

Let us know if you need further information.
Thank you,
Arun


Feature request: Allow for more overhead tick number options

Hello it would be awesome to have more than one tick count option. I was able to implement it crudely but this is the first time I code anything runelite related so it wasnt great.
It would really help, currently I am using both visual ticks and the visual metronome plugin to have more cycle lengths.
But the visual ticks plugin takes up a lot of space, I wish it could work like in the picture.

Hey I appreciate the suggestion and the pull request you made, however I'm not really looking to add new features as of right now, for a few reasons. Mainly because I don't want to clutter the plugin much more with features unless it is highly requested, and I think only a few people could use this feature.  So yeah, sorry about that.

I think if you enabled the tick count by for it to trigger a recolor on the nth rotation of ticks that would be sufficient without the clutter.
You could add a feature that makes it so the 5th tick is red and the 7th tick is blue rather than the 5th time the pattern has hit all numbers are red and the 7th cycle they're all blue.
So meaning a tick count of 35
5 red
7 blue
10 red
14 blue
Etc
35 could be either color, maybe 5's.
That would allow jefeh's request to exist while not being more numbers on screen.

Decided over discord to implement the first idea and add the ability to add more overhead cycles, #43


Add ManhuaOnline

Closes #2188
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Per #2292 it seems like the site is just redirecting to the new domain currently. Consider reusing the old extension

Is not redirecting, the message says they are doing changes in the server and "for the moment" users can visit the other page.


Differences between numpy.linalg.norm & scipy.linalg.norm

I noticed that scipy.linalg.norm does not accept an "axis=x" argument even though the documentation says that it does. The numpy version does accept the argument. It's a very useful part of the norm function. Thank you.

implemented in gh-4390 for 0.17.0, so closing.


Step 7 Excute Snapshots.

Anyone kindly provide me the files and code for step7 with snapshots(images) kindly. This task is much needed for me kindly respond anyone who have done that step

@RajeswarReddy - I am sorry but the GitHub issue tracker is not a support forum. It is for bugs and feature requests for the tutorial app. You can look for further help on StackOverflow, Gitter and the angular mailing list.

But regarding your question...
If you have cloned this repository using git then you should be able to set your working copy to step-8 by running....
git checkout step-8

If not then you can download that tagged version of the code at https://github.com/angular/angular-phonecat/archive/step-8.zip

@petebacondarwin  sir could you please send me the link which have all the zip files from 1 to 12?

I think you can guess them if you look carefully at the URL I gave you :-)

Please direct further questions at the support websites I linked to above.


Can i nstall with pip that lib ?

from apixu.client import ApixuClient, ApixuException
ModuleNotFoundError: No module named 'apixu'
I try pip install apixu dont work
then i try pip install WeatherAPIXU and this dont work

Unfortunately, it's not hosted on PyPI.
You can install directly from git using pip.
pip install git+<repository_url>
pip install git+https://github.com/apixu/apixu-python.git

Thanks!

while executing below in cmd hitting an error as " system cant find the file specified while executing command git clone"-- how to overcome this? it would be helpful and can you explain me also why apixu is used?
py -3.7 -m pip install git+https://github.com/apixu/apixu-python.git

@sonyasampalli
I don't fully understand your issue.
If you want to clone the repo then install it via pip.
git clone https://github.com/apixu/apixu-python.git
cd apixu-python
pip install .

If you want to install it directly from repo (without cloning).
pip install git+https://github.com/apixu/apixu-python.git

@sonyasampalli, do you have Git installed?


Add possibility to "group" multiple entities

E.g. if you have several "occurences" of the same "Material" (in the demo) how can we get a top level summary of the type
Material - Count of Cycles
Perhaps we could extend the syntax with something like a MERGE or a GROUP / JOIN command
SELECT MERGE t.name,
  COUNT(SELECT c FROM Cycles [WHERE t.start <= c.start AND c.start < t.end]),
  LIST(SELECT m.material FROM Materials [WHERE t.start <= m.start AND m.start < t.end])
FROM Tools

or
SELECT t.name,
  COUNT(SELECT c FROM Cycles [WHERE t.start <= c.start AND c.start < t.end]),
  LIST(SELECT m.material FROM Materials [WHERE t.start <= m.start AND m.start < t.end])
FROM Tools
GROUP BY t.name

The major difference to the (normal) query
SELECT t.name,
  COUNT(SELECT c FROM Cycles [WHERE t.start <= c.start AND c.start < t.end]),
  LIST(SELECT m.material FROM Materials [WHERE t.start <= m.start AND m.start < t.end])
FROM Tools

would be that if there are multiple tools with the same names then the third query woudl still create one line for each entry where the first two should only create one line per unique tool name.
So its kind of similar (in this case) to a DISTINCT on the t.name.
An Implementation Detail here is that the WHERE clause for the projections gets a bit more complicated as they could get one or many "entities" passed where they have to do their filter on.
But that should not make that big of a difference, I think.

Added an Example in the PR with
SELECT m.material, COUNT(SELECT c FROM Cycles) AS "cycles"
FROM Materials

where the same material is used several times but we only are interested in an overall summary:
[{'material': 'Material 1', 'cycles': 28}, {'material': 'Material 0', 'cycles': 52}]


Forbidden

I tried opening PowerToys, but it say it's forbidden.
It says I have to open the application on the runner.

Hi @masademan
Thank you for reporting the issue.
How are you opening PowerToys? Can you give us more information?

PowerToys version:
Windows build number: [run "winver"]

I’m betting this is from settings.

They are trying to directly open settings or FZ Editor

Version: v0.27.1
On Tue, Dec 22, 2020 at 6:18 PM Clint Rutkas<EMAIL_ADDRESS>wrote:

They are trying to directly open settings or FZ Editor
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/8712#issuecomment-749804874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOH3GGO6RERWC5X2HP2G2KLSWELM7ANCNFSM4VF7YCYQ
.

@masademan can you walk through exact steps you do for that warning?

but i dont know how to run an app in the runner
On Wed, Dec 23, 2020 at 3:47 PM Clint Rutkas<EMAIL_ADDRESS>wrote:

@masademan https://github.com/masademan can you walk through exact
steps you do for that warning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/8712#issuecomment-750447405,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOH3GGJCXZU57S5NMQZ7EDDSWJCM3ANCNFSM4VF7YCYQ
.

@masademan can you please describe how you are trying to open PowerToys?
The right steps are:

Open PowerToys from start menu
Double click on PowerToys icon on system tray

Try to follow these steps and if you still get the forbidden message please post a screenshot.

@masademan dont worry about our terms we are snapping to. Just walk us through like Davide mentioned step by step.

I am also having the same problem, when I try to start power toys it gives the same error. Forbidden.

@falcettijr you start powertoys via the shortcut included from the start menu.  If you want to pin it to your task bar, that is the icon to pick.  Settings itself is a different application

It worked! Thank you very much! :)


Add a WikiPage helper method for safely updating pages

Feature Summary and Justification
This feature provides a new method, WikiPage.update, for safe concurrent update of wiki pages. It takes a callback that computes the updated page content from its current content (i.e., str -> str). If Reddit returns an HTTP 409 Conflict response, the method takes the new server-side content from the response and re-computes the content for the edit. The process is repeated until the edit succeeds.
Some notes:

This bumps the prawcore dependency from 0.10 to 0.11.

Unfortunately, no single API endpoint gives us both the original page content and its revision ID, so it's necessary to make two API calls to put together the page content for the first edit attempt. Perhaps the Reddit admins could be talked into including the revision ID in the wiki page endpoint.

Currently there's no backoff strategy.

I appreciate the efforts on this one. Though I'm a little concerned that this functionality gets into the "feature creep" realm as it's building on top of the existing methods and providing a second-level helper. One could argue the streams are secondary as well -- they are, but they're also heavily utilized.
Perhaps a better place for this sort of logic would be https://github.com/praw-dev/prawdditions
What are your thoughts?


phantomjs hangs (Windows 8.1)

$> "./node_modules/.bin/magellan" --browser=phantomjs
Sauce configuration OK
Magellan-nightwatch test iterator found nightwatch configuration at: ./conf/nightwatch.json
Running 3 tests with 3 workers with phantomjs
--> Worker 1, mock port: 12000, running test: tests/github.js @phantomjs
--> Worker 3, mock port: 12006, running test: tests/walmart-shirt.js @phantomjs
--> Worker 2, mock port: 12003, running test: tests/walmart-book.js @phantomjs
hangs...

Thanks for reporting, @asunar .  I actually wasn't able to get npm install to complete at all.  Did you encounter this error, and if so how did you work around it?  https://github.com/TestArmada/boilerplate/issues/4
So far we have mostly been using OSX and Linux with TestArmada, but we'll do our best to support Windows, too.
/cc @Maciek416


Remove mapCoverage as it is deprecated

mapCoverage has been deprecated since<EMAIL_ADDRESS>https://github.com/facebook/jest/blob/master/CHANGELOG.md#2240
https://github.com/facebook/jest/pull/5177

Has it been deprecated or removed? Is it still required for versions before 22.4?
Wouldn't that be a breaking change?
We would at least have to bump the dependency on package.json accordingly.

Yes, you're right Linus. This PR should update Jest to 22.4.2.

Thank you! Fixed.
@LinusBorg @eddyerburgh

Thanks. FYI I'm moving this weekend so it might take some days till I get to merge and release this and other prs


feat(roles): add subscribe_service permission to Business Admin

Description
Add subscribe_service to Business Admin composite role. As defined by permission matrix.
Why
This missing permission prevented Business Admin from subscribing to services.
Issue
Refs: eclipse-tractusx/portal-iam#172
Checklist

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my changes
[x] I have successfully tested my changes

Hi @typecastcloud thanks for adding that! could you please also update the docs?

In docs the composite already had the role. Just not in the import realm. Do you mean something else?

Hi @typecastcloud thanks for adding that! could you please also update the docs?

Hi @evegufy, in docs matrix the buisness admin composite already had the role. Just not in the import realm. Do you mean something else?

Hi @typecastcloud you're right, I overlooked that, sorry.


#292 add --mock option to return example data

Fixes #292.
** WORK IN PROGRESS **

Coverage decreased (-2.1%) to 97.853% when pulling e488975e4ec648a08651c8da5c67287ddae8c7c3 on mock into 22de58cadf605e22ee15070da557960990158a3d on master.

Coverage decreased (-2.1%) to 97.853% when pulling eab36ccd95c2bee1d547c7b2a4035ea01422f858 on mock into 22de58cadf605e22ee15070da557960990158a3d on master.

Coverage decreased (-2.1%) to 97.855% when pulling eab36ccd95c2bee1d547c7b2a4035ea01422f858 on mock into 22de58cadf605e22ee15070da557960990158a3d on master.

Coverage decreased (-2.1%) to 97.869% when pulling 1f8b4d28b4f2236671a63824d21367aaa18fe580 on mock into 92e464520dc6fa260b757fe1a4e00245de6e7cfd on master.

Coverage decreased (-0.2%) to 99.823% when pulling f206d6888aafe62c5bb3b9d9d132f283c56e0710 on mock into 92e464520dc6fa260b757fe1a4e00245de6e7cfd on master.

Coverage decreased (-0.2%) to 99.823% when pulling e8ffc95e17a754568c965e38c7245e3d84b047db on mock into 92e464520dc6fa260b757fe1a4e00245de6e7cfd on master.

Coverage remained the same at 100.0% when pulling ff5596aab19ca49be65e26c647dafab4a8f83d3c on mock into 92e464520dc6fa260b757fe1a4e00245de6e7cfd on master.

Coverage remained the same at 100.0% when pulling ff5596aab19ca49be65e26c647dafab4a8f83d3c on mock into 92e464520dc6fa260b757fe1a4e00245de6e7cfd on master.

Coverage remained the same at 100.0% when pulling 8fad1ffc81b963e29c888ac96e2d2787a395984d on mock into 2a888c302b86f42fcde4ab3b8940b22d46701288 on master.

Coverage remained the same at 100.0% when pulling 4baae07db3c1239e2a6f16b330acbaea5f130b12 on mock into 2a888c302b86f42fcde4ab3b8940b22d46701288 on master.

Coverage remained the same at 100.0% when pulling 4baae07db3c1239e2a6f16b330acbaea5f130b12 on mock into 2a888c302b86f42fcde4ab3b8940b22d46701288 on master.

Coverage decreased (-0.3%) to 99.736% when pulling 8a2594eddf99dddb2aca6f860dad3589aed12f52 on mock into 2a888c302b86f42fcde4ab3b8940b22d46701288 on master.

Coverage remained the same at 100.0% when pulling ba11ff924c7934d90c31cdbd841e7e8e231b27ef on mock into 2a888c302b86f42fcde4ab3b8940b22d46701288 on master.

@rafaelcaricio @jmcs this PR is just adding minimal mock support, but it already provides value --- so I want to merge it :smile:

:+1:


create collection and flow

Hi @br41nslug
Thanks for the interesting extension!
Do I need to manually create collection or flow after the installation? I was following the Manual Installation section in the README file. but was not able to find collection and flow after restart directus instance.
BR,Austin

checking the screen captures in the readme, I realized creating collection and flow is required.
Would you please share me the script for retrieving the response from operation and updating the collection?

Sure thing @xu4wang they're pretty basic so didnt give a second thought to including those:
DALL-E and Stable Diffusion Demo Script Operation
where result is the field im using in my data-model.
module.exports = async function(data) {
	const { payload } = data['$trigger'];
    payload.result = data['$last'].id;
	return payload;
}

Davinci Demo Script Operation
module.exports = async function(data) {
	const { payload } = data['$trigger'];
    payload.response = data['$last'].response;
	return payload;
}


[Task]: Property Pane mount takes around 600-700ms

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Event - Click on Input Widget on Canvas.
Each codemirror component itself takes around 50-100 ms.

Quoting @SatishGandham message from slack
1. Code editor is the expensive one here. It will get slower if the widget has a lot of JS options enabled.

2. mounting is heavy, so we wont mount.
We can have a wrapper component which does this
{{isFocused && <CodeMirror />}}
{{!isFocused && <TextArea />}}

Stats

Stat
Values

Reach
302

Effort (months)
1.0

@Rishabh-Rathod @arunvjn is this related to #11040

@somangshu both are performance issue with property pane but different.
This issue is related to how heavy/slow mounting property pane is.
Other issue is how slow updating values in property pane is.

Ideally, it shouldn't take more than 50ms to mount property pane.

We are talking this improvement in #13112, it will be tracked there, closing this for now


refactor(app-indexing): update to 19.2.0

Staring in App Indexing 19.2.0, getInstance() has been deprecated in favor of getInstance(Context).

👍


44 Improve WhittakerSmooth, AirPls, and ArPls performance

This pull request primariliy tackles issue #44, but it does not fully close it (see the second point of 🚶 Next Steps).
It should be squashed before merging because it's more than 100 commits.
🏗 Main Feature Changes
🧑💻 Implementations

complete removal of sparse matrices from WhittakerSmooth, AirPls, and ArPls and transitioning to the more appropriate LAPACK banded storage format. Why? Because

sparse matrices have a huge overhead and their initialisation alone takes longer (> 1 ... 10 ms) than LAPACK's banded solvers take for a complete smooth (mostly < 1 ms). In contrast, the banded storage can be achieved with dense NumPy-Arrays which has only a small fraction of the initialisation time compared to sparse matrices.
dense NumPy-Arrays allow for more efficient computation of the penalty matrix D.T @ D  than what can be achieved via a sparse matrix, even though the logic becomes a bit more elaborate because symmetry has to be exploited while the redundant computations  that make up most of the computation time have to be avoided.
the sparse solver used before (SuperLU) was designed for sparse matrices that can have arbitrary sparsity pattern, but the matrices for the three algorithms have a very defined sparsity pattern, namely they are banded (tridiagonal for difference order 1, pentadiagonal for difference order 2, and so on). Dedicated banded solvers offer highest perfromance here because the algorithm can follow a very well-defined and straightforward pattern right from the start.

unification of the implementation of WhittakerSmooth, AirPls, and ArPls since they all rely on the same base algorithm and only differ in weighting strategies. Now, they all inherit from the class chemotools.utils._whittaker_base.main.WhittakerLikeSolver that handles all the underlying math once in a centralized place. The only thing the 3 transformer classes add now is a customized weighting strategy. Each of the classes uses different access points to the solver depending on the checks/preprocessing they need to conduct.
adding pentapy-support that will check for availability of pentapy at runtime and use its high-performance solver for all scenarios where the differences order is 2 (see ⏱ Timings).
improvement of internal checks and type conversions via class variables (by coincidence related to #87) .
adding the respective documentation.

⏱ Timings
In summary, the speedup with the minimum set of dependencies is ~5x for all algorithms. However, when pentapy is used, the speedup can be up to 15x. Since it is used for difference order 2 and this is the standard use case, this is quite some gain.
Yet, rust-based implementations seem to be even faster, so we definitely did not reach the limit here.
🌊 WhittakerSmooth with difference order 1
Speedup of ~5 to 6 times

🌊 WhittakerSmooth with difference order 2
Without pentapy - Speedup of ~5 times

With pentapy - Speedup of ~5 to 15 times

🏴☠ ArPls
Without pentapy - Speedup of ~4 times

With pentapy - Speedup of ~5 to 15 times

🛩 AirPls with polynomial order 1
Speedup of ~12 to 5 times

🛩 AirPls with polynomial order 2
Speedup of ~12 to 5 times

With pentapy - Speedup of ~10 to 15 times

🚶 Next Steps

numerical stability of the banded solver (Partially Pivoted LU-decomposition) can only be achieved for difference order up to 2 when the size of the spectra grows to common sizes of 1000 to 10000. Beyond these difference orders high lam-values are no longer possible. Many Whittaker smoother implementations out there suffer from this, but this is something that should be tackled by, e.g., also invoking banded QR-decomposition.
the baseline algorithms use an initailisation which can be far off from the true baseline. Therefore, they take a lot of iterations to converge. Having a good initial guess could solve the problem.

🎁 Additional features
Given that this was a lot of refactoring, the chance was used to enrich the WhittakerSmooth by

adding sample_weight keyword argument to transform and fit_transform to allow for locally weighting datapoints depending on their noise level. Basically, this makes the WhittakerSmooth en par with ArPls and AirPls which were already able to pass weights.
weights allow for automated determination of lam via maximization of the log marginal likelihood (same approach as for sklearn's GaussianProcessRegressor).
a function to estimate the local/global noise levels which can be used for the weighting required for the log marginal likelihood method (chemotools.smooth.estimate_noise_stddev or chemotools.utils.estimate_noise_stddev).
adding the possibility for a model-based specification of lam via chemotools.smooth.WhittakerSmoothLambda similar to SciPy's Bounds for specifying the bounds of parameters during optimizations
adding a SciPy-like wrapper for banded LU-decompositions (chemotools.utils._banded_linalg).

📦📂 Package structure

the settings.json in the .vscode-folder was removed from the GIT version control. Having a file that can overwrite the user's local settings (which might contain much more than just formatting and linting settings) can be quite destructive. It was replaced by a settings_template.json that can provide the basic setup for the user.
the requirements.txt were split into a requirements.txt for the main package capabilities and a requirements-dev.txt for the dependencies needed during development. #53 will profit from this, since then one can simply point to the requirements.txt from pyproject.toml without having to worry about the user accidently installing pytest, matplotlib, etc.
basic linting via Ruff was configured in the pyproject.toml (requires settings.json to have Ruff configured). It reveals some unused imports, wrong type hints, and non-pythonic statements that should be tackled in the future.

✅❌ Tests

by including pytest-xdist, the tests can now be run in parallel which saves quite some time. The command I always used for running the tests is

pytest  --cov=chemotools .\tests  -n=auto --cov-report html

with pytest.mark.parametrize, the tests were extended by running the same test on multiple input combinations. This was especially useful for transformer functionality and numerics tests.

🪤 Miscellaneous

✍ Fixed a typo and a type hint here and there

@MothNik FANTASTIC - I have been waiting for this day with a lot of enthusiasm 🤓🤓!!
I am starting to review it right now, and it is a long review, but I hope I can have it done in about a month from now! It is a very exciting contribution. During the review process, we could also start considering how to add the different improvements to the documentation pages :smile:
The restructure of the package is a good idea, it goes perfectly in line with #53, and I need to get done during the summer, Having the dev dependencies separated is a great starting point! Also nice to hear you have been using Ruff for linting, it was also on my todo list to trancition from black. I did not know about pytest-xdist, but I have started testing it and... it is pretty cool, I like it a lot!😎
I think that now it is my turn, and I have some work to do 🥳🥳

@paucablop You are highly welcome 😸
Yes, it's a lot of files. I'm sorry it turned so big 😅
Take all the time you need and just ping me for the documentation pages ✍
I usually would not do package restructuring in a feature branch, but the branch required some setup for the development environment, especially for the tests ✅❌
I hope this will help for #53 and also #61 and make the installation easier 💾
As I said, take your time 😸

I want to give special credits and thanks to Guillaume Biessy, the author of Revisiting Whittaker-Henderson Smoothing which is - as far as I'm aware - the best review of the Whittaker-Henderson Smoothing out there because it is illustrated very well and focuses on the key points 🙏

@paucablop
I'm done with the renaming of all the variables and functions to make them more readable.
Besides, I also added a tiny cheatsheet for testing with pytest as a README.


Replace bee-metrics with the metrics crate

Currently, bee-metrics is pulling in an old version of axum. Also, the metrics crate seems to be actively maintained.

Closed by #577.


Build breaker cannot find reports

Hi Simon.
I'm using sonar 6.7 and build breaker 8 and I'm getting the following output from build breaker:
##[section]Starting: Break build on quality gate failure
==============================================================================
Task         : SonarQube build breaker
Description  : Break the build when the quality gate fails.
Version      : 8.0.0
Author       : Simon de Lang
Help         :
==============================================================================
Number of analyses in this build: 0
Summary of statusses:
##[section]Finishing: Break build on quality gate failure

I can see from a previous step that the report-tasks.txt is being written to the following path:
13:40:25.010 DEBUG: Report metadata written to D:\VSTS\Agents\A\_work\179\.sonarqube\out\.sonar\report-task.txt
Any ideas? Cheers!

Hey! If you use the Publish Quality Gate task, please check if the reports end up in your summary. I think they don't.
There's a bug with SonarSource right now that is placing the report in the wrong folder for some versions of SonarQube. More info can be found here: https://community.sonarsource.com/t/sonarqubebuildsummary-md-file-empty-with-the-publish-quality-gate-result-step-in-azure-devops/13305/2

Hi Simon,
SonarQUbe just released 4.8.0 version for Sinar Publish. I tried it and now the issue is solved.
I still face "Number of analyses in this build: 0"
I have a pipeline in dev.azure.com and Sonar installed locally. The agent is local and has access from the build machine. The analisys is done by sonar, but build breaker says 0 analyses in this build.
My steps are:

task: SonarQubePrepare@4
task: Maven@3 (with sonarQubeRunAnalysis: true)
task: SonarQubePublish@4
task: sonar-buildbreaker@8

Kindly
Gianluca

From their changelog:

Added fallback search for report-task.txt in previous location for SQ instances < 7.2.0

I'll copy their changes later today/tomorrow. Thanks for the heads-up!

Hi Simon,
Yes, I can confirm my Sonar version is 7.0
Kindly
Gianluca

I've just published v8.1.0 of the extension. @gt97 and @msweaver does that solve your issues?

I can confirm the extension works correctly. I made the test below (to test if it works even with multiple projects in the analysis)
It works perfectly.
Test one project:

test: maven - sonar analysis fails
expected result: sonarbreaker -> KO
result: Test OK
test: maven  - sonar analysis passes
expected result: sonarbreaker -> OK
result: Test OK
test: esql - sonar analysis fails
expected result: sonarbreaker -> KO
result: Test OK
test: esql - sonar analysis passes
expected result: sonarbreaker -> OK
result: Test OK

Test two projects in the same build:
5. test: maven - sonar analysis fails and esql - sonar analysis fails
expected result: sonarbreaker -> KO
result: Test OK
6. test: maven - sonar analysis passes and esql - sonar analysis fails
expected result: sonarbreaker -> KO
result: Test OK
7. test: maven - sonar analysis fails and esql - sonar analysis passes
expected result: sonarbreaker -> KO
result: Test OK
8. test: maven - sonar analysis passes and esql - sonar analysis passes
expected result: sonarbreaker -> OK
result: Test OK
Thank you
Kindly
Gianluca

Hi @simondel
We have the same issue, too.
We have Sonar 8.0, the latest sonar plugins for our tfs and the 8.1 build breaker version. The gate is also published in the build but we also get the error, that the build breaker does not find the reports like gt97 mentioned.
Is there anything we are missing?
p.s. can not upload screenshots right now, I will try again at home.

Hi,
updating to the 4.8 version of the sonar tasks fixed the issue for us. Just in case anyone stumbles on this one ;)
Greetings
Simon


Add placeholder @context; restrict use of JSON-LD keywords to @context only

This PR adds a placeholder/faux external @context IRI to the example profile.   More importantly, it substitutes id and type for the JSON-LD @id and @type keywords in order to prevent the "leakage" of JSON-LD-isms into what should otherwise be a plain JSON document.  This can be done by leveraging the aliasing of @id and @type and id and type in the @context per current JSON-LD community practice.

I should note that we alias @id and @type as id and type for all our events and entities in our Caliper 1.1 JSON-LD external context.  We do this in order to avoid over-dependence on JSON-LD keywords and by extension the JSON-LD specification.  I recommend that you adopt the same approach.

Accepted, will be merged after other PRs it may conflict with so conflicts only need to be handled here

Ok, otherwise, happy to resolve them myself and re-submit.  Cheers, Anth

Aliasing is now fully incorporated, closing this PR


No notification when app closed

Hello ,
I receive push notification when my app is on foreground and background, but when i'm close application i don't receive this .
For register application to firebase server i use an interface beetwen Portable class and native class. So i send notification push with postman with this json :
{
"to":"device token",
"data": {
"message" : "my_custom_value",
"other_key" : true,
"body":"tornado"
}
}
This is work for foreground and background, but when i send notification when app is closed, i receive nothing, any idea ?
Bug Information
Version Number of Plugin: 1.0.7
Device Tested On: Wiko Fever SE
Version of VS: 15.4.2
Version of Xamarin:  4.7.10

Did you initialize on an Android application class? as specified in the Getting started guide?

If none if this is your case please reopen this issue again.

@rdelrosario @Nuada26 , Hi to comment about this issue, I have testing to integrate the plugin on my app (android/ios), on Debug mode in my android App, I can receive notifications on foreground and background with tokens or topics, but if the app is completely closed (killed) cannot receive notifications, using Postman to testing if send directly to the last token generated, the response from the FCM server is token not registered, this happen after +- 30 seconds that the app is closed.
The plugin version used is 2.1.3
Thanks for any advice.

Hi,
I'm not receiving notifications once the app has been closed
v2.3.0
Xamarin.Firebase.Messaging v60.1142.0

@CesareSoldani Did you check FAQ 2, 3 and 4? on: https://github.com/CrossGeeks/FirebasePushNotificationPlugin/blob/master/docs/FAQ.md

Hi,
the only one I'm not 100% about is no. 4, I can't see anything obvious, its a Galaxy Tab S. I am just about to try it on my Sony Xperia phone to see if it works there..

Are you calling ProcessIntent on MainActivity OnCreate and OnNewIntent?

Also when you deployed the release version did you increment the build number or uninstalled the previous debug version installed?

What is your code block on OnNotificationReceived?

Are you calling ProcessIntent on MainActivity OnCreate and OnNewIntent? Yes
Also when you deployed the release version did you increment the build number or uninstalled the previous debug version installed? Both
What is your code block on OnNotificationReceived?
        // Handle notification when app is closed here
        CrossFirebasePushNotification.Current.OnNotificationReceived += async (s, p) =>
        {
            Log.WriteLine(LogPriority.Debug, "Firebase", p.Data.ToString());
            await App.DataManager.Sync(App.Foreground);
        };

I'll try commenting this out, as I assume the notification should still be displayed without this..
Cesare
ps. many thanks for your help on this.

Where are you instantiating App.DataManager ? if is after MainActivity and not on MainApplication your app is crashing when closed because App.DataManager doesn't exist. MainActivity doesn't exist when app is closed so you need to instantiate App.DataManager on MainApplication instead.

kk, yeah, the notification came through when I commented out that code
many thanks
Cesare

Hi "await App.DataManager.Sync(App.Foreground);" what is this meaning. I am not able to find App.DataManager. Can you please tell me How can I work If app is not in background neither in foreground how can I receive Notification?

await App.DataManager.Sync(App.Foreground); is my own code, inside it was trying to use Xamarin.Forms, which was not instatiated (Forms.Init(this, bundle);) when the app was in the background and so was crashing, which prevented the notification from displaying.

Okay. Thanks for your quick reply.

notification does not received when the app is not running state .


qdrouter crashes with exit code 139 on many unauthorized connection attempts

Hi,
I am not sure if this is related to enmasse or directly to qdrouter. We have a setup where enmasse is connected to keycloak. When we have a client that connects to qdrouter with wrong credentials we get (of course) many log entries like this
2018-03-13 13:47:35.376257 +0000 SERVER (info) Connection from <IP_ADDRESS>:34576 (to <IP_ADDRESS>:5671) failed: amqp:unauthorized-access Failed to authenticate client [mech=PLAIN]

and after a while (10 - 15 minutes) qdrouter crashes with exit code 139 and gets restarted again and again. Client tries to reconnect every 500 msec. Without this client with wrong credentials this error does not occur.
Thanks
Daniel

Tested again and can confirm that it works now. Thanks a lot.

Thanks for confirming Daniel!


[2-pts] Login page and registration page Member 1

Build a login and registration page with devise.

#15