Support "jf mvn compile" using JFrog CLI command

When attempting to configure a Maven project using the JFrog CLI, the "jf mvn compile" command successfully completes the build process, but the actual compilation fails.
I tried setting up a sample maven project, and encountered the below error while executing the "jf mvn compile"
17:29:00 [🔵Info] Running Mvn...
17:29:00 [🔵Info] Running mvn command: /usr/bin/java -classpath /usr/share/maven/boot/plexus-classworlds-2.x.jar -Dmaven.home=/usr/share/maven -DbuildInfoConfig.propertiesFile=/tmp/jfrog/properties/extractorProperties135610658 -Dm3plugin.lib=/root/.jfrog/dependencies/maven/2.41.16 -Dclassworlds.conf=/root/.jfrog/dependencies/maven/2.41.16/classworlds.conf -Dmaven.multiModuleProjectDirectory=/root/proj/my-sample-project -DbuildInfoConfig.artifactoryResolutionEnabled=true org.codehaus.plexus.classworlds.launcher.Launcher compile
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Scanning for projects...
[main] INFO org.jfrog.build.extractor.maven.BuildInfoRecorder - Initializing Artifactory Build-Info Recording
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger -
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ---------------------< com.example:sample-project >---------------------
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Building sample-project 1.0-SNAPSHOT
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - --------------------------------[ jar ]---------------------------------
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger -
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - --- maven-resources-plugin:2.6:resources (default-resources) @ sample-project ---
[main] WARNING org.apache.maven.shared.filtering.DefaultMavenResourcesFiltering - Using platform encoding (UTF-8 actually) to copy filtered resources, i.e. build is platform dependent!
[main] INFO org.apache.maven.shared.filtering.DefaultMavenResourcesFiltering - skip non existing resourceDirectory /root/proj/my-sample-project/src/main/resources
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger -
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - --- maven-compiler-plugin:3.1:compile (default-compile) @ sample-project ---
[main] INFO org.apache.maven.plugin.compiler.CompilerMojo - Nothing to compile - all classes are up to date
[main] INFO org.jfrog.build.extractor.maven.BuildDeploymentHelper - Artifactory Build Info Recorder: Saving Build Info to '/root/proj/my-sample-project/target/build-info.json'
[main] INFO org.jfrog.build.extractor.maven.BuildDeploymentHelper - Artifactory Build Info Recorder: deploy artifacts set to false, artifacts will not be deployed...
[main] INFO org.jfrog.build.extractor.maven.BuildDeploymentHelper - Artifactory Build Info Recorder: publish build info set to false, build info will not be published...
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - BUILD SUCCESS
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Total time:  1.180 s
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Finished at: 2024-06-10T17:29:02Z
[main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
{
"status": "failure",
"totals": {
"success": 0,
"failure": 1
}
}
Describe the solution you'd like to see:
Given that "jf mvn install" is functioning correctly, Artifactory should support the capability to execute "jf mvn compile".

To be honest, jf mvn should support any maven goal like its usage suggests.
The help output for jf mvn --help clearly states:
Name:
jf mvn - Run Maven build.
Usage:
jf mvn  [command options]
Arguments:
goals and options
Goals and options to run with mvn command. For example  -f path/to/pom.xml
No mention about only install or deploy being the only supported goal. Its not just compile that I expect to work without failure, any maven goal should be able to run without failure.
All these goals return no failure by mvn when ran without jf wrapping them, why should they suddenly fail with jf. Sounds like a bug to me in jf not generating a proper return code.


lmstudio: build fails with hash-mismatch

[user@nixos]$ NIXPKGS_ALLOW_UNFREE=1 nix shell --impure github:NixOS/nixpkgs/master#lmstudio
error: hash mismatch in fixed-output derivation '/nix/store/ckijj426ghwa7aj1cr0zba8lkn0z6kf4-LM_Studio-0.2.22.AppImage.drv':
         specified: sha256-hcV8wDhulFAxHDBDKicpEGovwcsn9RaIi/idUz+YzD8=
            got:    sha256-HoAVUU5Z5ueGMSybhT0OKV611v1yU5am0wy0GsCUmvk=

https://github.com/NixOS/nixpkgs/blob/bfb7a882678e518398ce9a31a881538679f6f092/pkgs/by-name/lm/lmstudio/linux.nix#L11

No hash mismatch on my end, and I've never built nor heard of lmstudio before. (i.e. there's no caching involved here)

 mj@URIEL-WS  ~/dotFiles  sudo nixos-rebuild boot --flake ~/dotFiles
building the system configuration...
trace: warning: cudaPackages.autoAddDriverRunpath is deprecated, use pkgs.autoAddDriverRunpath instead
error: hash mismatch in fixed-output derivation '/nix/store/2b5p9x00myfkpaill9dk8qviyjma3hkq-LM_Studio-0.2.22.AppImage.drv':
         specified: sha256-hcV8wDhulFAxHDBDKicpEGovwcsn9RaIi/idUz+YzD8=
            got:    sha256-HoAVUU5Z5ueGMSybhT0OKV611v1yU5am0wy0GsCUmvk=
error: 1 dependencies of derivation '/nix/store/46f5lrm3b40mi42vi0aiydbbw5vmzirz-lmstudio-0.2.22-extracted.drv' failed to build
error: 1 dependencies of derivation '/nix/store/a0p284bbsmx7spkmnjc83419r5i51zyq-lmstudio-0.2.22.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ldwd6i79ml6mb0m2w02p4jnsrhv8r5w1-user-environment.drv' failed to build
error: 1 dependencies of derivation '/nix/store/38b9s372nvvwvb4nmjp8qz8l8afwprc5-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/09p5g89f81gqwhh44is3d1sxk44ylkjl-nixos-system-URIEL-WS-24.11.20240529.ad57eef.drv' failed to build

I am getting errors building when I add the package to my flake => (configuration.nix, home-manager) dotfiles...

Your flake.lock is missing, so we don't know what revision you're on atm

Fixed in #314992. The PR has not reached unstable yet: https://nixpk.gs/pr-tracker.html?pr=314992
See discussion in #310914 on why the hash mismatch happened.


Upgrade Zinc to 1.0.0-X18

:warning: This change is not ready for merging yet, there is 16 test failures. :warning:
Those changes were the limited to getting the code to compile, so it is quite likely that the integration layer can be further simplidfied. @jvican PTAL.
I have a problem with debugging the tests in Eclipse, because PDE picks up old versions of  org.scala-ide.zinc.library and org.scala-ide.zinc.compiler.bridge bundles and I don't know how to nudge it doing the right thing. @sschaef maybe you have some tips?

Amended. I added back source references as requested.

Superseded by #1178


add crypto_box_seal

@plneple looks great! can you also add tests for it like these ones? https://github.com/serenity-kit/react-native-libsodium/blob/main/example/src/tests/crypto_box_easy_test.ts

@nikgraf
Disclaimer: I am not at all familiar with libsodium
It seems to me that the output from the crypto_box_seal function is dependent on the nonce generated inside of libsodium. Documentation.
I believe, that means that I cannot write out the expected output statically, and calculating it might have it's own bugs.

@plneple totally, but the what would work as a test is creating the encryption and verifying it with a decryption. A test here is really important to make sure this works on all environments and this way we can dedect breaking changes with the e2e tests

@nikgraf I have added crypto_box_seal_open and tested crypto_box_seal using it, as you suggested.

@plneple just released 1.3.0 which includes your additions 👍


use ringbuf fail

hi, i try to use ringbuf. but it load fail.
err: map exec_events: map create without BTF: invalid argument
my env is:
os: ubuntu21.04
kern: linux 5.11 x86
here is my code:
So, someone can help me? maybe my usages is error. e.g: some params is correctly.
tell me correct usages, if possible. thanks!

@rockingl I think you forgot to add your code.

Closing for now. If you add your code, please re-open.


Update the long-term policy compliance history design

This stems from rethinking assumptions when doing the development.

/cc @JustinKuli


Fix video freeze in LocalPreview Android/iOS

Hi @tompi.
Can you take a look at these changes? This fixes issues #30 and #37.
About the problem
When activating and deactivating the camera, after the second attempt, locally, the screen was frozen and the local preview was not displayed.

Hi @andersonvieiragomeslopes  recently I downloaded your POC and faced many issue related to video freezing and not appearing on screen. I made many changes for Android and fixed video-related issues for Android but for ios video in not display in ios and i am not able to solve this can you please look into this issue


Is there API for contacts?

I've just had a quick look here: https://github.com/google/google-api-nodejs-client/tree/master/apis
CMD + F searching for contacts
https://developers.google.com/google-apps/contacts/v3/?hl=en
Before I write my own / use existing wrappers I would like to know if I'm missing something, if contacts API is available out-of-the-box.
Thanks

I am also looking into this. Right now I think I am going to get the OAuth2 token with this library and then use that with the contacts JS api to pull contacts.
It would be great if there was built-in contacts support though! :smile:

@vpontis contacts JS api - https://developers.google.com/google-apps/contacts/v3/

Retrieving all contacts
To retrieve all of a user's contacts, send an authorized GET request to the following URL:
https://www.google.com/m8/feeds/contacts/{userEmail}/full

Would be nice to know if the google-api-nodejs-client offers some shorthand methods...

https://github.com/google/google-api-nodejs-client/issues/254

Thanks, fair comment!

https://myaccount.google.com/contacts
Save contact info when you interact with people
Is there any API to get the contacts for this in NodeJS?


DOC: Can there be a comparison in ideology/stack with Spyre and Shiny?

I'm going to try to put together some documentation soon. The main goal was just to make it a little easier to go from having a pandas dataframe to some pretty javascript visualizations. From looking at Spyre, I got the impression he was going for a more pure python approach using the standard plotting libraries. I wanted some convenience functions for some common charts, but tried to keep it flexible. It's no where near the level of Shiny yet, but it covers most of what I was using Shiny for on a day-to-day basis.


Bump to k8s 1.18-> 1.21, helm 3.3-> 3.6

this allows compiling with go 1.18
I used helm3 because it depends on k8s 1.21 as well
I removed the vendor folder, because why not?

Kick out Helm2 as well?

I serve at your leisure...

All deployments should be migrated. Even if not, we only need to pick a version that does the conversion and can then upgrade further.


Add serialize functionality example usage and plotting.

Just a short explanation on how to extract the pandas dataframes from the pickled file.

Roman -- thanks so much for the doc update.


Do not use preview api versions

Use api versions 7.2 as the latest on-premises Azure Dev Ops version does not support these preview api versions.

Congrats on being my first contribution to this project!
I don't have a way to test against different Azure DevOps server instances, and honestly hadn't considered the use case, according to this reference , 7.1 is the latest supported on premises version.
I need to add more integration tests before I can publish on merge, but can I assume you ran the Test-ExtensionLocaly.ps?

I executed 'npm run test' to check all the existing test cases and I created the extension via 'npx tfx-cli extension create' to install it on our server.
So far it's working well.
The script Test-ExtensionLocally.ps1 also works.
7.2 was a failure which Azure Dev Ops returned directly. Version 7.1 is correct.
That's why I updated my fork again this afternoon.
In the past I adjusted the code several times and didn't pay attention to the 1.


feat: Add object to link patch and patch-type

Description:
Due to this issue we have detected the need to improve the way the operator can be extended. This PR deeply link the needed patch to restart a workload with the way the patch is applied. This is because StrategicMergePatch strategy is not suitable in all use cases, for example: Argo Rollouts

Great changes but i think that the name of the new type its to long and can lead to confusion, maybe PatchT is a better name for that

thank you! <3 may be PatchConstructorPatchT is more suitable? I'm wondering if this is clear enough for people to know the owner of this struct, which is the PatchConstructor function
WDYT?


Fixes 404s in the Docs

This fixes 404s in the docs, I've also update 6.x docs to refer to Ops Center to be Gravity Hub

@jsilvers  Please Review


Crash in boost regex when trying to download Metro Exodus (Gold Edition)

Command used to initiate download (with config file containing some other default values such as location and language; downloader work for several other games):
lgogdownloader --download --game "metro_exodus"
Crash message:
terminate called after throwing an instance of 'boost::wrapexcept<boost::regex_error>'   what():  Unmatched [ or [^ in character class declaration.  The error occurred while parsing the regular expression fragment: 'exodus>>>HERE>>>'.
Not sure if there is anything else I could try, but if any has any idea or more information is needed, please let me know. Metro series is at a deep discount ATM, the other two older instalment I also own, those downloaded just fine, only Metro Exodus is giving me a crash. I purchased it as 'Gold Edition' that includes 2 DLCs (not sure if it has any bearings on the crashes, but maybe worth mentioning).

Today I purchased another game; after the purchase I made an attempt to download it's installer files with lgogdownloader, just to get surprised by the above error message. My surprise only grew when the - after reporting the error for the new game - it just continued and downloaded the above listed, prior failing game, metro_exodus!
As I eyeballed the GoG website 'My collection' with a numb brain, I have spotted that my 'new game' is now first in the listing, as it's name starts with the letter 'B'... Interesting... What are my chances that if I buy another game, that is even further ahead in the alphabet, that will fail to download and suddenly the newly purchased game get 'fixed'? Time for an experiment, I guess; let's see what's the cheapest game I can grab from the head of the (alphabetical) list?
The winner turned out to be: '8-Bit Adventures 2: Demo'; it costs exactly nothing (as it is for Free). I am now getting an error trying to download this game, but both Metro Exodus and the other newly purchased game are now successfully downloading.
Looks like lgogdownloader currently is having some beef with how the list of owned games start, breaking the regex in the aforementioned place.

Another game purchase another issue another theory... Looks like the issue might be caused by the 'new' label shown over the game purchased. First download wasn't working with the above error, but after clicking the game on the GoG 'shelf' (and reloading the page that confirmed the 'new' label gone) the download started working!


Add express package from @bigtest

This extracts the effection-express package from bigtest and adds it here under @effection/express. The main purpose for this is the websocket code, which I want to use for the debug package. Maybe the WS code doesn't need to use express though, and we don't need all this other express stuff?
@cowboyd you did some work in this area for simulacrum, right?
IMO the express wrapper is somewhat questionable, for a few reasons, which is why we have so far resisted putting it in effection itself. Things I don't love about it:

Questionable error handling. Should an error in a request really kill the whole server?!
It doesn't properly wrap the request/response objects for requests, yet does so for websocket requests. So it operates at multiple levels of abstraction.

Anyway, putting this here as a starting point for further discussion.

@jnicklas This was the only integration that I needed. Basically, you just stream connections and than's it. You only need to use createServer from node's http package.
https://github.com/thefrontside/simulacrum/blob/v0/packages/server/src/websocket-transport.ts#L42-L56

Going with #337 instead of this


set logger on the connection wrapper only

The Predis\Connection\Factory class does not have a setLogger() method.

@B-Galati done


add HACS or Add-on store support for home assistant?

would it be possible to install onstar2mqtt inside home assistant?

You can use my add-on from here: https://github.com/BigThunderSR/homeassistant-addons-onstar2mqtt

Thank you!


Add Forking Steps for New contributors in README file

Greeting PA,
The current README.md required forking steps for new contributors.
kindly assign me this issue under GSSOC 24
Thank you.

hi @TAHIR0110  can u pls assign this issue to me under gssoc

hi @TAHIR0110  can u pls assign this issue to me under gssoc


Banner alert - account wide - service outage

Classic Screen Grabs: https://github.com/linode/manager/issues/1044
We want to keep the wording the same as classic, and just have standard banners.
This is what we had planned before:
https://github.com/linode/manager/issues/1160
We should, stating the obvious, be able to have a re-usable component for these banners, where based on a value of the type of banner it will dictate what color it will be.

Message:

Color:
#444444


TST: speedup distance tests

Speed up test_cdist_calling_conventions as requested in #6656 and also fix a broken test_cdist_dtype_equivalence.
@rgommers The computational time is acceptable now? Otherwise I would opt for the @dec.slow fall-back option.

@rgommers I have seen the mail about branching 0.19 soon.
Please consider adding this PR to that milestone.

Thanks @apbard. ~6x faster now, about 6 sec. for everything in test_distance.py on my machine. I'd call that good enough.

LGTM, merged. Good catch on the pdist` -> cdist`` in the test.


DTMF issue

https://github.com/OneOfEleven/uv-k5-firmware-custom/assets/83151488/1429185a-f94f-4e66-bc4b-e7e655f57d3b

https://github.com/OneOfEleven/uv-k5-firmware-custom/assets/83151488/588e7e24-65be-41b3-8575-0dfd690e583b

the problem with dtmf, when pressing ptt 1 and # transmits the same tone as befwhen typing dmf and pressing ptt, nothing happensore pressing #

the problem with dtmf, when pressing ptt 1 and # transmits the same tone as befwhen , when typing dmf and pressing ptt, nothing happens

could this be related/linked too your other issue???   I don't use dtmf.  and don't have the other issue. hmmmmmmmmmmmmmm

perhaps they are somehow connected? I need a dm to control the repeater

there are no such problems in the repo from 20231005

Yes I broke the keypad stuff yesterday when I made a change, been on the case since tracking a couple of uncovered problems with the keypad code.

I really hope that you will succeed. We will be very grateful to you

Still have a bug in the DTMF entry before TX.
If I enter some DTMF keys in, then stop and let the entry time out (go off screen), the next button press is ignored - that's a bug I MUST find.

you are my hero


Prototype whole notebook LSP messages

We were able to implement the full notebook sync mode. There are a few items that we need to think about when it comes to analyzing notebooks as a whole.
When analyzing the notebook using a merged document means that the static analysis tool could find problems that only exist in the merged document but not in the parts. The other thing is the parts may not be analyzable on their own. But those problems are not real problems for the notebook, but just artefact of putting them together in a merged document for analysis.


Code workflow

Configure Azure Pipelines. This pipeline configuration builds the application and produces a build for the Release configuration.


Single digital story - type

The description paragraph text should top align with article title.
Size should also be larger. 33px at desktop

33px? Where's that from?

Thats one of the type sizes from the styleguide
Wellcome Bold style: WB4

Top align the copy with the format name "Article". Both 30px from the top of the white container.


blockquotes left border may not displayed correctly

view in ursine:

I think this might be better( I changed some css ):

my source code
>The left border may not displayed correctly when the list and code block is used in blockquotes
>
>- item1
>- item2
>
>```python
>import tensorflow as tf
>```
>
>

Yeah, that does look prettier. I'll look into it once I get through some of my finals here

The release with the fix is up now! It was a really simple fix really, but I wasn't able to make a new release earlier due to college finals work, so pardon the delay on this


Modify the default protocol in tunnel-run-parameters.md

protocol
Default
auto -> quic

auto is still the default protocol for cloudflared. It defaults to QUIC under the hood, but there is no reason to adjust this as older clients will still be using auto.


NEGEMMLowpMatrixMultiplyCore support type

Problem description:
I  am confused by the data type supported for NEGEMMLowpMatrixMultiplyCore. I find that the example (https://github.com/ARM-software/ComputeLibrary/blob/main/examples/neon_gemm_qasymm8.cpp#L220) uses input data type QASYMM8 and output data type S32. But when I read the code, I find here should tiger an error message: https://github.com/ARM-software/ComputeLibrary/blob/main/src/cpu/operators/internal/CpuGemmAssemblyDispatch.cpp#L792
However, I could run the example, without seeing the error message. But the condition (DataType::QASYMM8 && d->data_type() != DataType::QASYMM8) is true. I am confused. Could you help to explain what is the data type supported in NEGEMMLowpMatrixMultiplyCore? Thanks!

Hi @zhen-jia
NEGEMMLowpMatrixMultiplyCore is implemented using CpuGemmLowpMatrixMultiplyCore, see details in
https://github.com/ARM-software/ComputeLibrary/blob/main/src/runtime/NEON/functions/NEGEMMLowpMatrixMultiplyCore.cpp#L65
When you call NEGEMMLowpMatrixMultiplyCore::validate() you end up calling CpuGemmLowpMatrixMultiplyCore::validate() which supports S32 as can be seen in https://github.com/ARM-software/ComputeLibrary/blob/main/src/cpu/operators/CpuGemmLowpMatrixMultiplyCore.cpp#L313
You can see the data types accepted by  CpuGemmLowpMatrixMultiplyCore in
https://github.com/ARM-software/ComputeLibrary/blob/main/src/cpu/operators/CpuGemmLowpMatrixMultiplyCore.h#L78
 /** Initialise the kernel's inputs, output
     *
     * Valid data layouts:
     * - NHWC
     * - NCHW
     *
     * Valid data type configurations:
     * |src0           |src1               |src2     |dst            |
     * |:--------------|:------------------|:--------|:--------------|
     * |QASYMM8        |QASYMM8            |S32      |QASYMM8        |
     * |QASYMM8        |QSYMM8_PER_CHANNEL |S32      |QASYMM8        |
     * |QASYMM8        |QSYMM8             |S32      |QASYMM8        |
     * |QASYMM8        |QASYMM8            |S32      |S32            |
     * |QASYMM8        |QSYMM8_PER_CHANNEL |S32      |S32            |
     * |QASYMM8        |QSYMM8             |S32      |S32            |
     * |QASYMM8_SIGNED |QASYMM8_SIGNED     |S32      |QASYMM8_SIGNED |
     * |QASYMM8_SIGNED |QSYMM8_PER_CHANNEL |S32      |QASYMM8_SIGNED |
     * |QASYMM8_SIGNED |QSYMM8             |S32      |QASYMM8_SIGNED |
     * |QASYMM8_SIGNED |QASYMM8_SIGNED     |S32      |S32            |
     * |QASYMM8_SIGNED |QSYMM8_PER_CHANNEL |S32      |S32            |
     * |QASYMM8_SIGNED |QSYMM8             |S32      |S32            |
     */

CpuGemmAssemblyDispatch is a different class used internally in ACL to run assembly kernels.
Hope this helps.

Thanks @morgolock for the help. One more question. Pytorch adopts fused kernel (fuse GEMM and de-quantization into one assembly kernel). Actually they are using dynamic quantization QNNPACK kernel. I am wondering that if ACL has some kernel like that? If I understand correctly, this folder (https://github.com/ARM-software/ComputeLibrary/tree/main/src/core/NEON/kernels/arm_gemm/kernels) only contains general GEMMs. Correct me if I was wrong. Thanks a lot.

Hi @zhen-jia
You can find the GEMM highly optimized kernels in the folder https://github.com/ARM-software/ComputeLibrary/tree/main/src/core/NEON/kernels/arm_gemm/kernels
The quantization for is handled in https://github.com/ARM-software/ComputeLibrary/blob/main/src/core/NEON/kernels/arm_gemm/quantized.cpp#L59
Hope this helps.


Google Maps camera positioning issues on iOS

The iOS Google Map view seems to be positioning the camera target in the top left corner instead of in the center of the viewport (this doesn't always happen, but it happens often).
I verified that we are passing the correct coordinates to the iOS Google Maps SDK.
The issue is also observed when a marker is tapped and the Map view should move the marker to the center of the viewport, in which case the new camera position didn't come from Flutter but from the maps SDK.
I suspect this is some kind of a lifecycle issue.
Repro steps:

Run the example Google Maps app on iOS.
Select "Scrolling map".
Often the marker for the top map is in top left corner instead of being centered.

It's interesting that scrolling the entire page up (such that the map is partially obscured seems to "fix" the issue and puts the map in a good state).
Here's a screen recording showing how the map misbehaves when I tap the marker, and how it starts behaving correctly after I drag it up: https://photos.app.goo.gl/2UE7m5DKrvFwVHDn9

I've noticed the same bug, but only on real hardware iOS device. On the simulator it works as expected.

I've noticed this too, only on the ios device.

same bug, try it on simulator.

Same issue on ios, any update?

Same issue. Always happens, on both simulator and device. Any updates on this?

Same issue in iOS, myLocation button does not work, camera focuses on top-left corner.

Don't forget to upvote on the initial comment to raise priority

Same issue on iOS

Using "add reaction" on the initial comment would increase priority while +1 and me too comments are rather pointless and cause lots of people to get notified for no reason.
To get notified use the [Subscribe] button to the top right instead.
https://github.com/flutter/flutter/wiki/Issue-hygiene#thumbs-up-reactions

On iPhone simulators, this issue also occurs, but only on iPhones below the modal X. So 6s, 8 al have this problem, but X and newer don't

This issue has been fixed(worked around) in https://github.com/flutter/plugins/pull/1157, with google_maps_flutter 0.2.0+1.
Another issue https://github.com/flutter/flutter/issues/27550 is opened to track the status of the 'real' fix.
Closing this issue now. If any related issue is found, please feel empowered to reopen this issue with more details.

Rotate the device left and right and then try. It worked for me

you have to constrain the view that will have the google maps as its subview to the end of the screen. Do not constrain it to the safe area. when i did that it worked for me.

Changing from animateCamera to moveCamera when the map is initialised, seemed to fix the issue for me.
controller.moveCamera(...)

@holykou that didn't work for me, I tried with animateCamera and moveCamera to no avail. I assume you call this code in the callback of onMapCreated?

For the records, we had a similar issue using react-native-maps on react-native and it looks the problem is present after version 2.7.0 of google maps SDK. After downgrading to 2.6.0 the animateToRegion function behaves as expected. Hope it helps

I'm having this same problem in my current ios build. The initialCameraPosition is in the top left until you move then it snaps back to the center.
I'm using the latest google_maps_flutter version (0.5.21) and flutter v1.9.8-pre.83
[✓] Flutter (Channel master, v1.9.8-pre.83, on Linux, locale en_GB.UTF-8)
    • Flutter version 1.9.8-pre.83 at /home/corey/development/flutter
    • Framework revision 3932ffb1cd (24 hours ago), 2019-09-04 23:13:16 -0400
    • Engine revision e7f9ef6aa0
    • Dart version 2.5.0 (build 2.5.0-dev.4.0 36985859e4)

  Completer<GoogleMapController> _controller = Completer();

  @override
  Widget build(BuildContext context) {
    super.build(context);

    return AnimatedOpacity(
      duration: Duration(milliseconds: 500),
      opacity: _controller.isCompleted ? 1 : 0,
      child: GoogleMap(
        mapType: MapType.normal,
        initialCameraPosition: widget.report.cameraPos,
        onMapCreated: (GoogleMapController controller) {
          if (_controller.isCompleted) return;
          _controller.complete(controller);
        },
        onCameraMove: (pos) {
          if (!widget.editing) return;
          setState(() {
            widget.report.setLatLng(pos.target.latitude, pos.target.longitude);
          });
        },
        rotateGesturesEnabled: false,
        tiltGesturesEnabled: false,
        myLocationButtonEnabled: false,
        scrollGesturesEnabled: widget.editing,
        zoomGesturesEnabled: widget.editing,
      ),
    );
  }

There are two pages that display the map, one shows a preview an the other is a fullscreen view where you can scroll and zoom. The camera messes up when going between the two pages (placing the position in the top-left instead of the center).

@cyanglaz

Same issue

Yes, same issue still there. animateCamera doesn't work but moveCamera works. However you'll need to add extra spice.
In onMapCreated in GoogleMap widget you can add little delay such as await Future.delayed(Duration(microseconds: 1)); . Calling moveCamera with centre coordinates will centre position ok.

Try taking off any CameraBounds that are set. This worked for me. It always locks to a corner of the bounds.

I had the same issue. Then I tired jampez suggestion(https://stackoverflow.com/a/54605646), that is instead of using animate function create and set GMSCameraPosition.camera instance to mapView.camera. And I have no problem now.


Early parsing command line parameters

Currently We are doing parsing command line parameters too late after initializing some subsystems so the command line parameters cannot be applied to these subsystems.
Ex.: logging.

It was fixed in #7449


Travis testing isn't working for Linux/OpenMPI and macOS

Travis testing should work for all cases. I made a new branch to work on this.

macOS testing fails with checking whether the C compiler works... no.

I think we could explore some other continuous integration service...

Yes that's definitely an option. Do you know any that would be better?
I'm now attempting to set up homebrew on my old mac, so I can replicate the Travis environment better.
This is something I had meant to do anyway.

no - though can check with other people...(tomorrow)


Identity operator from ONNX

Hi @tqchen
Is the Identity operator defined in ONNX experimental Ops as
in onnx/defs/experiments/defs.cc as
OPERATOR_SCHEMA(Identity)
.SetSupportLevel(SupportType::EXPERIMENTAL)
.NumInputs(1)
.NumOutputs(1)
.SetDoc("Identity operator")
.Input(0, "input", "Input tensor", "T")
.Output(
0,
"output",
"Tensor to copy input into.",
"T")
.TypeConstraint("T", { "tensor(float16)", "tensor(float)", "tensor(double)" },
"Constrain input and output types to float tensors.");
implemented for import within NNVM? I see a copy operator in
python/nnvm/frontend/onnx.py:    'Identity'      : Renamer('copy') ..is this the same?

They are the same


Deployment Link Updation

@himanshu-03 you have pasted a wrong link in the about section, kindly change it to zaltan.vercel.app so that individuals can have a look at the hosted website

Done


No padding at top of document

When I activate Zen, the padding that used to appear at the top has not been present for a while now, and the top line of text sticks very close to the top of my screen:

Is there any way I can fix this? I tried disabling all the installed extensions as well as most of the default ones without luck.

Hello @RaoOfPhysics,
you can try adding this to your personal stylesheet:
[data-zen="true"] {
  atom-text-editor:not(.mini) {
    padding-top: 20px; // Change this to your liking
  }
}

This works for me.

Yes, that should work. The lack of padding at the top is by design.

@rm-code: That worked, many thanks! :)
@braver: Ah, I see. I thought it was a problem at my end because the screenshot in the README shows a padding (and indeed I happened to have had a padding for a while previously):

A setting for padding in the GUI might be nice. :)

I played around with a setting for the padding, but I didn't manage to make Zen restore the original "padding" options when exiting zen mode. Maybe @braver has more insight.
@RaoOfPhysics
I use the "multiple" tabs option with these custom style settings in my personal style sheet:

It produces a nice padding and makes the tabs unobtrusive. (It works for most of the themes I have installed).

Thanks, @rm-code, this is very helpful. :)

@rm-code thanks for the code example! But it leads to the text cutting of at the top (more of margin than padding):

Is there another rule which will make it correctly scroll up further and visible?
@braver I’m not usually one to ask for settings (am a designer myself) but it’s very strange to have the text stick to the top. If it’s not going to be considered for the default style, a setting might be cool for this.

What you're asking for, padding the top of the document so you can scroll it down from the top of the view, is as far as I know not possible. Everything you add at the top you lose at the bottom and several layers (highlights, gutter) need to be adjusted.
Zen used to have what your screenshot shows, but that just looks weird and tacky. I would love to have proper padding as a default, it's just that I don't know how to make it happen, and what your screenshot shows is just not acceptable as a compromise.

Fiddling a bit, I got a solution where you don't use padding, you use margin. It has no issues with scrolling. Con: you lose the editor's background color, and get the window's background color instead.
[data-zen=true] {
  atom-text-editor:not(.mini) {
    margin-top: 50px;
    margin-bottom: 50px;
    background: transparent;
  }
}

@dasilvacontin I tried that before as well of course and it has the same issue with being cut off:

@jancborchardt Ah, yeah, it will always have a margin... I'll let you know if I come up with something else.


SE38 is showing as needed in SU53

Hello,
When implementing AMS, on the Netweaver provider, when it calls into the target SAP ABAP system for RFC gathered metrics, it seems to be requesting SE38 access via auth object S_TCODE.
Why is SE38 needed?
Can this be changed to use SA38 instead.
Thanks.
Darryl

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

ID: 89d2c26c-369b-172b-5d05-27ab2e98ae4e
Version Independent ID: ea1d8fe1-3349-6b2a-6fdd-194694e34ea2
Content: Configure SAP NetWeaver for Azure Monitor for SAP solutions
Content Source: articles/sap/monitor/provider-netweaver.md
Service: sap-on-azure
Sub-service: sap-monitor
GitHub Login: @MightySuz
Microsoft Alias: sujaj

@Darryl-G
Thanks for your feedback! We will investigate and update as appropriate.


Add Perl module and test script icons

Adds Perl module (.pm) and test script (.t) icons

TODO

[X] Use cargo fmt
[ ] Add necessary tests
[x] Add changelog entry

Codecov Report

Merging #522 (48bcf2d) into master (9f83b8d) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #522      +/-   ##
==========================================
- Coverage   88.49%   88.41%   -0.09%
==========================================
  Files          36       36
  Lines        3538     3538
==========================================
- Hits         3131     3128       -3
- Misses        407      410       +3

Impacted Files
Coverage Δ

src/icon.rs
97.65% <100.00%> (+0.01%)
:arrow_up:

src/flags/recursion.rs
89.83% <0.00%> (-1.70%)
:arrow_down:

src/meta/filetype.rs
80.00% <0.00%> (-1.06%)
:arrow_down:

src/meta/size.rs
94.15% <0.00%> (-0.59%)
:arrow_down:

src/app.rs
95.74% <0.00%> (-0.09%)
:arrow_down:

src/display.rs
73.20% <0.00%> (+0.27%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks!


zipkin does not shows ui when accessing thru gateway

when I access zipkin thru localhost:9411 it works fine but when I access it thru gateway
https://localhost:443/zipkin it shows up a blank
attached images for both cases :
case1
case2

Hi! You're asking a question in the Sleuth repo about Zipkin. I don't know if that's what you wanted.
When you file an issue there is this comment:

Bug report
Please provide details of the problem, including the version of Spring Cloud that you
are using. If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed

You haven't provided a single useful information in your file report.  Can you please provide any useful details following the guidelines above? Otherwise I will have to close this issue.


Optional fields

This adds optional fields to structs
Note that is not valid for a struct value to have optional fields, only struct types may have optional fields.
Most of the commits can be reviewed in isolation.
Fixes #3277

I plan to update the API to use rest params but that can be done in a followup commit/pr
Once this is settled there is some refactoring that can be done to unify the code for how we handle fields throughout types/

I'm assigning to a bunch of people:
@kalman Look at marshal
@cmasone-attic Look at nomdl
@rafael-atticlabs Look at the rest

marshal stuff lgtm

jenkins: test this

Codecov Report

Merging #3287 into master will increase coverage by <.01%.
The diff coverage is 96.73%.

Impacted Files
Coverage Δ

go/types/encode_human_readable.go
88.61% <100%> (+0.09%)
:arrow_up:

go/merge/three_way.go
90.96% <100%> (+0.05%)
:arrow_up:

samples/go/csv/write.go
59.64% <100%> (ø)
:arrow_up:

go/marshal/encode.go
98.16% <100%> (+0.01%)
:arrow_up:

go/types/struct.go
82.84% <100%> (+0.2%)
:arrow_up:

go/nomdl/parser.go
94.21% <100%> (+0.24%)
:arrow_up:

go/types/value_encoder.go
98.43% <100%> (+0.01%)
:arrow_up:

go/migration/migrate.go
92.12% <100%> (+0.12%)
:arrow_up:

go/types/type.go
73.68% <100%> (ø)
:arrow_up:

go/types/simplify.go
100% <100%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


Migrate ImplementIntersectAndExceptAsUnion to rule

As part of #811

@martint Thanks for the review. Have applied the comments

There are checkstyle failures, but otherwise this looks great!

[ERROR] src/main/java/io/prestosql/sql/planner/PlanOptimizers.java:[52] (imports) ImportOrder: Wrong order for 'io.prestosql.sql.planner.iterative.rule.ImplementIntersectAsUnion' import.


KAFKA-14133: Move mocks from KStreamTransformValuesTest, KTableImplTest and KTableTransformValuesTest to Mockito

This pull requests migrates mocks from KStreamTransformValuesTest, KTableImplTest and KTableTransformValuesTest to Mockito.

Verified that all 3 tests modified in this PR are successful: https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-14204/2/testReport/org.apache.kafka.streams.kstream.internals/


Automated cherry pick of #759

Cherry pick of #759 on release-0.4.
#759: Add note about http01 and wildcard certificates

/release-note-none

/lgtm
/approve


JSX Render Functions in Nuxt 2 with Nuxt Bridge are failing

Environment
Nuxt CLI v3.0.0-27234503.d5127e9
RootDir: /sandbox
Nuxt project info:

Operating System: Linux
Node Version:     v14.17.6
Nuxt Version:     2.16.0-27226092.034b9901
Package Manager:  Yarn
Bundler:          Webpack
User Config:      buildModules
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>

Describe the bug and expected behavior
Attempting to use JSX inside a render function is now failing with Syntax Error: Unexpected token, expected ","
Steps to reproduce
View the compilation error in context here:
https://codesandbox.io/s/suspicious-euclid-g904s?
I expect to see "Hello Nuxt!" wrapped in a div with a class of "a-container-class".
Additional context
This code was working without the upgrade nuxt bridge.

+1

Yes, I can confirm the issue persists (reproduction), though a workaround is to enable vite mode @tylerforesthauser:
import { defineNuxtConfig } from '@nuxt/bridge'

export default defineNuxtConfig({
  bridge: {
    vite: true,
  },
})

Does anyone have a workaround for this?

+1

@danielroe Any updates for webpack?

The workaround to enable vite does not work for me, though it does give me nicer-looking error messages:
 ERROR  [commonjs--resolver] Unexpected token (Note that you need plugins to import files that are not JavaScript)                                           9:12:36 AM
file: [redacted]
225:
226:         return (
227:             <div class={['p-splitter p-component', 'p-splitter-' + this.layout]}>
                 ^
228:                 {this.$slots.default}
229:                 {

If you use jsx/tsx with nuxt/bridge, you need to add the isTSX option.
https://github.com/nuxt/bridge/pull/1064
export default defineNuxtConfig({
  birdge: {
    typescript: { isTSX: true }
  }
})


Update to<EMAIL_ADDRESS>

Following on from #3150, this updates to the latest working version of the package.

🎉 You're a speed demon!


feat: support casing window string operation

Introduces casing windows on the intersection:

Related data change: https://github.com/equinor/esv-intersection-data/pull/12
Issue: https://github.com/equinor/wellx-designer/issues/715

You should play around with git and learn how you could "easily" split different concerns into different branches/PRs :face_with_spiral_eyes:
It would make it much easier to review this PR if the IPoint changes (and maybe som other unrelated changes) was split out.
But I'm always happy to see developers leaving the codebase in a better shape than they found it in! Keep it up :grin:

Getting an error in storybook so can not test PR:

I don't experience the same issues locally. Did you checkout the linked data changes from the description? 😅

You should play around with git and learn how you could "easily" split different concerns into different branches/PRs face_with_spiral_eyes
It would make it much easier to review this PR if the IPoint changes (and maybe som other unrelated changes) was split out.
But I'm always happy to see developers leaving the codebase in a better shape than they found it in! Keep it up grin

Good points. I'll be sure to split it out into its own PR next time.


Publish to npm

:+1:

@jmeas The hard part is figuring how we want to export to common js -- not so hard for orbit itself, but a little confusing for the "integration layers" like ember-orbit

I did the work to make the next built repo at least have a package.json so we should be able to publish it as is.

but a little confusing for the "integration layers" like ember-orbit

Yeah? I'd be interested in chatting with ya to figure out the best way to do this.
Usually what I would do in this situation is is...

Use a UMD wrapper for Orbit.js in the distributed build
Also export UMD for the orbit integration later
Specify orbit.js as a dependency of ember-orbit within package.json

===
For CJS environments, this should work predictably.
Consider a user who installs ember-orbit as a dependency of their project. Because ember-orbit requires in orbit, that will be installed if the user has not already installed it.
In their application code, they would require in ember-orbit somewhere. The UMD wrapper would recognize the CJS environment, and therefore require in orbit on the fly. Voila!
For users not using CJS, they would just require in the deps (or concat them) in order. orbit => integration layer => application code.
===
For bundling I've lately been using esperanto + Babel. Esperanto has a nice 1-line UMD wrapper, which it does automagically for you. The old wrapper I used is more verbose, but more easily editable by humans.

:+1:

Can I help somehow to move this forward? I'd love to be able to use Orbit with NW.js ...

Orbit has finally been published to npm as the following packages:

https://www.npmjs.com/package/orbit-core
https://www.npmjs.com/package/orbit-local-storage
https://www.npmjs.com/package/orbit-jsonapi


Margin have impact on feature location

https://jsfiddle.net/557qumh9/
margin on top component of the map ( body in the jsfiddle) make Map.eachFeaturesAtPixel being out of sync with the view.
For testing, in jsfiddle, hover the feature and observe that the pointer become a cursor at the right. Then click the text "Click to change margin", hovering the feature will not work anymore. In fact it works but with an offset.

Browser : Chrome Version 47.0.2526.106

You'll have to use updateSize() to tell OpenLayers that the size of your map container has changed:
document.querySelector("body").style.margin='20px';
map.updateSize();

https://jsfiddle.net/557qumh9/1/


feat(rich-text-editor): Add tagLink, subscript, superscript, and spoiler input

Closes #51.
Describe your changes
This PR adds four new features to the Rich Text editor that were previously Markdown-only. All include keyboard shortcuts as well as manual entry via a new overflow dropdown menu option.
These include the following:
TagLinks CTRL-[

Subscript Ctrl - ; and Superscript Ctrl - :

Spoiler CTRL-]

PR Checklist

[x] All new/changed functionality includes unit and (optionally) e2e tests as appropriate
[x] All new/changed functions have /** ... */ docs
[x] I've added the bug/enhancement and other labels as appropriate

Environment(s) tested

Device: Desktop
OS: Windows
Browser: Chrome
Version: 103

The tag link shouldn't support multiple words. There's existing validation for allowed tags in parse_tag_link() that could perhaps be reused?

@aalear Thanks for the heads-up. I pushed some changes to better validate the links. Now you can only convert the selection if the trimmed content doesn't have any whitespace. (trimmed because double clicking words in Chrome on Windows also selects the space after the word)

I pushed updates to the keyboard shortcuts for superscript and subscript to match most other editor conventions.

If this is an outsized request, we can consider getting this PR merged first.

I think we should get this merged and follow up with the Markdown menu items. I don't have a ton of time this sprint to dig into it myself.


website with broken link to openwhisk adapter

https://cloud.spring.io/spring-cloud-function/#quick-start
If you click on OpenWhisk link is broken:
https://github.com/spring-cloud/spring-cloud-function/tree/master/spring-cloud-function-samples/function-sample-openwhisk
It should point to the adapter folder
https://github.com/spring-cloud/spring-cloud-function/tree/master/spring-cloud-function-adapters/spring-cloud-function-adapter-openwhisk
This looks like a duplicate of #145
How can I help or submit a PR to update the website?

Fixed with https://github.com/spring-cloud/spring-cloud-function/commit/e54187c099cefbf0e77fe5a3204cfd8ed39fc713


fix(shippers): Add extra matcher for USPS Informed Delivery

Proposed change

For some reason that I'm still not sure of, my email server is not finding the USPS Informed Delivery emails when matching purely on the address but adding the Name "USPS Informed Delivery" causes it to match
The mail server is just docker-mailcow which is just postfix so I'm not sure why it would behave differently but adding this fixes the issue for me
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update
[ ] Adds a new shipper
[x] Update existing shipper

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:

Codecov Report

Merging #656 (2211a89) into dev (a5862ac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #656   +/-   ##
=======================================
  Coverage   98.33%   98.33%
=======================================
  Files           7        7
  Lines        1382     1382
=======================================
  Hits         1359     1359
  Misses         23       23

Impacted Files
Coverage Δ

custom_components/mail_and_packages/const.py
100.00% <ø> (ø)

Looks good!
Thanks!


flannel cni configuration : enable hairpin mode

Updates Flannel CNI configuration in order to enable the hairpin mode

Cleaner PR, it merges #908 and #909

Hey - Thanks for your help on these. I'm closing this PR because we've completely rewritten and moved the Kubernetes documentation. Take a look, if you like: https://coreos.com/tectonic/docs/latest/tutorials/kubernetes/getting-started.html


Add RepeatVector layer

Commits 78196a8 and 6347d65 can be ignored. They are reverted in 208cd56

Just FYI your commits are squashed before merge so I think the reviewer looks only on the final version of commit so the steps that you did are not so important - the final result counts only 😉

@avan1235 thanks a lot for your comments. This is my first time contributing to an open source project and your comments were really thoughtful, will make changes and keep in mind for future contributions.

Hi, @dosier is it ready for review?


Added support for MKL HPL-CPU.

This PR follows the HPL-CPU enabling PR, adding support for the MKL version.

We decided to change how we add HPL MKL: it will be a new directory.


Add new disposable domain

Provider is https://skiff.com/quick-alias and it's actually subdomains, each user can generate their own subdomain. I tested it, very easy.
The page title of https://skiff.com/quick-alias is "Quick alias burner email address". In the HTML I see
<meta name="description" content="Secure and quick-to-create burner email addresses with Skiff!

There is a browser plugin https://github.com/irazasyed/email-masker to create one email per website, skiff.com's twitter account promoted it, the github repository is tagged as 'burner-email'
You should add:

maskmy.id because it is there main domain
anything.maskmy.id because it allows you to create disposable addresses with random subdomains

@7c

This user copied my research, report and screenshot from https://github.com/wesbos/burner-email-providers/pull/427 I'm not affiliated and don't know their intentions.

Also
mailbox.zip
sailmail.io

@mtmail @7c

This is also a domain which I suggested to add back in #73, so seeing another person suggest to add it is very comforting

The company was acquired by Notion. The service will be discontinued soon https://www.notion.so/blog/meet-skiff-the-newest-member-of-the-notion-family

thanks, i confirm, they are now part of notion, we will be watching...


Allow to specify AbortSignal in startRegistration and startAuthentication

Describe the issue
There is no way to stop registration/authentication process. This is necessary when the user canceled an action in the UI, for example, pressed the back button.
Expected behavior
const controller = new AbortController();
startAuthentication({ signal: controller.signal});

// Cancel action
controller.abort();

Dependencies

OS: all
Browser: all
Authenticator: all

SimpleWebAuthn Libraries
Latest version
Additional context
AbortController is used inside library but there is no option to specify it outside.

SimpleWebAuthn manages its own use of AbortController, what is an example of a scenario in which this is insufficient?  A single-page app? If the browser doesn't cancel the WebAuthn call on a navigation event then I'm not sure how specifying a custom instance of an abort controller would be helpful here.

I'm working on single-page app (web, electron). User can leave registration/authentication view (cancel action). Of course, after the timeout registration/authentication process will be cancelled, but it would be useful to be able to cancel it from the app.

I've created #449 that will offer a new WebAuthnAbortService import with a cancelCeremony() method that can be called to cancel any in-flight WebAuthn ceremony started with startRegistration() or startAuthentication().

This is exactly what I need for the current project.

WebAuthnAbortService.cancelCeremony() is now available as of<EMAIL_ADDRESS>✌

I would like to supply my own AbortSignal to startRegistration() as I want to reuse it in fetch calls to /options and /verify.


Use Travis build stages to parallelize CI

Travis Build Stages allow running independent CI tasks in parallel. Our traditional CI setup involved running several very time consuming tasks serially, when that is not necessary.
The primary idea here is to run the following tasks in parallel:

Coverage: Runs the coverage task, which runs all tests and deploys the coverage report
Release Tests: Runs all tests with the release flag
Linting and Deployment: Runs make-zip (which runs build-ts that takes a lot of time) and deploys to S3

Overall, this cuts the CI run time by half from around 14 minutes to around 7 minutes.
TODO

[x] Enable awscli on Travis CI to allow usage of the aws s3 sync command (which is not available in the JS SDK)

Thanks for the pull request @sanjeetsuhag!

:heavy_check_mark: Signed CLA found.
:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

@sanjeetsuhag Could you please merge/rebase with main?


GoCQL: Unable to create session

Hey, I'm using the new 2.2.1 release of the plugin, but when I fill the fields and test the connection it throws me the following error:

And the log:

Connection refused means that the driver cannot connect to the Cassandra server, could you please try if you can reach that host and port from the Grafana server to the Cassandra server?

Connection refused means that the driver cannot connect to the Cassandra server, could you please try if you can reach that host and port from the Grafana server to the Cassandra server?

Oh, I'm sorry, you can close the thread. Last night that machine shut down and I did not know until now.

=)It happens, don't worry :)


Jenkins

WIP WIP

https://jenkins-m.affirm-dev.com/view/All/job/test.cabot.generator/

this is on top of https://github.com/Affirm/cabot/pull/5

@BlaineAtAffirm @Affirm/infrastructure please review? this is on top of https://github.com/Affirm/cabot/pull/5

@srikiraju to see if it worked on jenkins since everything works for me locally...it didn't (jim's doesn't either)
guess I will use a wrapper script instead https://docs.docker.com/compose/startup-order/

@BlaineAtAffirm anything else I should change?


fix: Twitter login notifications, incorrect cookie management.

Relates to:
No issue.
Risks
LOW - could break logins, but I tested.
Background
I noticed multiple login notifications in twitter:

My previous changes were not fully correct.
What does this PR do?
Handles twitter cookies better in twitter without re-triggering login() each time.
What kind of change is this?
Bug fixes (non-breaking change which fixes an issue)
Why are we doing this? Any context or related work?
I recently pushed: https://github.com/ai16z/eliza/pull/1288
Not sure if my PR broke this, or was always issue, but now fixed, not getting notifications anymore with cached cookies.
Documentation changes needed?
My changes do not require a change to the project documentation.

I cleared cache and watch the twitter login notifications in app. No longer getting notifications of "new login".
Testing
Where should a reviewer start?
Try login twice with cached cookies and should see no more new login notifications.
Detailed testing steps
None, automated tests are fine.
Discord username
cjft

I highly recommend merge this before new release, required fix, also CI is failing:
Skipping integration tests due to missing required API keys
Error: Process completed with exit code 1.```


POC for generating API endpoints using code generation

Note: This PR is messy as it includes some refactoring of content from the 'fastly' package into new 'errors' and 'decodebody' packages; those will be split out into their own PRs before this is ready for review.
This PR is a proof-of-concept for generating API endpoints using the Jennifer code-generation module. It generates endpoints for Brotli compression and NGWAF as examples. For NGWAF, it includes configuration-during-enablement for the 'workspace_id'.

Closed in favor of #567.


⚠ Fortnite has degraded performance

In 0d35247, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1738 ms

Resolved: Fortnite performance has improved in f14d600.


Deploy to staging and prod

Should we merge our new exceptions in /Exceptions with @Duslerke 's Error generator?

The ErrorGen class is a helper class to make test's setup cleaner & easier to read. It belongs within Tests project, so we shouldn't move any of the lambda code to tests' project (as we only want to build & deploy only the lambda csproj)


Add pangolin analysis mode option

Was looking to change the pangolin analysis type while running ncov-tools and I didn't see any configuration options for it so I've attempted to add it in with my limited snakemake knowledge.
Pangolin 4 lets the user specify an analysis mode from "fast" or "accurate" with the --analysis-mode argument. This change is adding that in with the default set to "accurate" as is the same with the tool itself.
I am not sure how this affects the pangolin_version: "3" option so it might need a few changes to accommodate the backwards compatibility aspect there
Cheers

additional changes made, merged this PR with #104


svg offsetHeight/offsetWidth deprecated in chrome

When using the tooltip on an svg image, I got this error: "Warning: NaN is an invalid value for the left css style property."
The issue is that on svg type elements, the offsetHeight and offsetWidth is deprecated so it returns undefined. I tried swapping them with getBoundingClientRect().width and getBoundingClientRect().height but it isn't exactly the same and the tooltip wasn't placed where it should be.
Would be great if this could work with svgs as well. Thanks!

Hi,
Would you mind sharing the code you've tested with getBoundingClientRect? I've quickly tested and it seems to work as expected:

Thanks

Turns out there was something else influencing it's position on my end, the getBoundingClientRect().width and height works great.


How to use Rank based search in iOS

I see there is a rank based search for AlchemyData News in doc
http://docs.alchemyapi.com/docs/rank-based-search#
Is this supported in IOS SDK? Do you have any example on how to added rank parameter during swift query?
Thanks.

Hi @andrew21c. Apologies for the delay--we usually try to respond to issues much faster, but have been very busy recently.
The rank query is supported with AlchemyData News. Just pass it in the query parameter of the getNews function. Here's some code that adapts the readme to demonstrate the rank parameter.
import AlchemyDataNewsV1

let apiKey = "your-apikey-here"
let alchemyDataNews = AlchemyDataNews(apiKey: apiKey)
let failure = { (error: NSError) in print(error) }

let start = "now-1d" // yesterday
let end = "now" // today
let query = [
    "rank": "high"
]

alchemyDataNews.getNews(start, end: end, query: query, failure: failure) { news in
    print(news)
}

Hope that helps! Let us know if you run into any trouble.

Thanks. This is very helpful.


Usage with NodeJS 14

Hello,
Thank you very much for this repo. I was looking at using this for a personal project but I seem to be running into build errors when trying to build it, which I think are due to using a newer version of node (14).
It would be great if it was possible to use this with later versions of node if possible. If not, what would be needed to go about upgrading it to node 14?
Thank you very much in advance.

Hey, I haven't touched this project in quite a while :)
I imagine if you wanted to take it on, you'd want to update any of the node bindings that have gone out of date. It's not super straight forward from what I can remember.
I wish you luck, I don't remember it being very fun!


Api debug  grpc service

New debug grpc service with get all accounts method.
Update go-sm to use api release v1.1.0.

go lint is producing these errors but it is not code that is part of my changes:
layerfetcher/layers.go:308:61: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:358:45: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:364:44: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:374:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:392:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:409:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:422:33: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)

bors try

Layerfetcher test fails is failing this branch, blocking this pr.
github.com/spacemeshos/go-spacemesh/layerfetcher
layerfetcher/layers.go:308:61: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:358:45: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:364:44: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:374:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:392:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:409:41: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
layerfetcher/layers.go:422:33: conversion from untyped int to Hint (string) yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)
FAIL    github.com/spacemeshos/go-spacemesh/layerfetcher [build failed]

All comments are addressed besides changing account address from string to bytes in this PR. Should be separate issue which involves updating all the txs processing tests.

bors try

bors merge

@lrettig - had a look at the conflicts - many are in the globalstate service code so I think it best if you'll try to merge them as they are not trivial merges.

bors merge

bors merge

bors merge

bors merge


Cloning gitolite-admin

I want to clone<EMAIL_ADDRESS>but I have a problem.
If I writing "git@localhost info", it's good. However, I'm trying to clone gitolite-admin and get a password require. If I wrote it, I got a message:
Please make sure you have the correct access rights
and the repository exists.

So, I want to do it because I have another a root problem.
I'm adding SSH pub key to Redmine, and next, I'm trying to clone my project throught SSH and got same message.
What I can do?


Allow multiple "eject_coil_enable_time" values

In the continuing work for supporting ball devices that physically lock multiple balls, this PR extends the eject_coil_enable_time setting to (optionally) support multiple time values.
The reasoning here is that eject-coil-enable devices, by definition, rely on gravity to pull the ejecting ball from the device. When multiple balls are stacked in the device it is critical to time the coil to release only one ball: too short and the ejecting ball will jam, too long and a second ball will attempt to escape.
However the exact timing of the coil depends on how many balls are in the device, as the added weight of a second or third ball will "push" the ejecting ball out faster. As a result, an eject-coil-enable ball device will need shorter and shorter enable times as the number of held balls increases.
This change only affects machines when explicitly specified. Config settings that have only a single eject_coil_enable_time value will continue to behave as they currently do.

Always wanted to implement that. Thanks for doing that!


Unlock Hover doesn't work for layers outside an artboard?

Tried the plugin, and apparently it's not working for shapes that are not contained inside an artboard.

We're working on adding support for this. Should have another release in the next couple days.

Should be working without artboards now. Let me know if you find any issues.

Works like a charm, thank you!


feat: ex_doc

Current dependencies on/for this PR:

main

PR #4   👈

PR #5

PR #6

This comment was auto-generated by Graphite.

@mhanberg started a stack merge that includes this pull request via Graphite.

@mhanberg merged this pull request with Graphite.


Make code changes to accommodate different chassis & leg dimensions

The existing code is based on the previous version of the chassis and legs, which had different dimensions. As well, changes to reduce the radial load on the hip servos will impact the software. Need to identify the areas that are impacted, and develop new code. Also need to standardize the symbols representing chassis dimensions, and where they're located in the source code.

Areas impacted:

translations of local coords to global, and vice versa
derivation of servo angles needed to achieve a specified toe position. This will be major, as there are several sets of simultaneous and quadratic equations involved. revevant routine is coordsToAngles() in include/flows.cpp
maybe some flow processing

Survey of locations in source code that have references to physical robot dimensions:
configDetails.cpp

servo calibration stuff in setupPerBotConfig

flows.h

maxServoAngle
toe safety box params safeMaxPosX...
hexbot body measurements   BT = 2.915 ...
f_staticHomeX = 13.78 ...
fp_frontToeHomeX= 3.8...
sin_p45 = ... precalculations (need more for new chassis?)

flows.cpp

setupFlows has f_dynGHomeX[1] =  18.62  ...
and f_hip[1] = 8.87 ...
mapDegToPWM  has magic numbers in last line – servo type dependency
anglesToCoords uses symbolic dimension references
coordsToAngles has symbolic references
globCoordsToLocal might have geometry dependencies? Refers to symbols. - fp_frontHipX
localCoordsToGlobal refers to symbols: f_hipX[legnumber]
handle_fo_newHomeLReset uses magic numbers to reset home position, and f_staticHomeX...

mqttBroker.cpp

SSP command assumes servo PWM range = 110 – 490

Dumb question: are servos mounted differently, so rotation is in different direction for some?

no, both old and new versions had shin at front of leg, so motors are acting the same.

As discussed in issue 17:

robot physical dimension are defined in global_variables.cpp using macros for the actual values
run time variables are initialized in a setup routine in configDetails.cpp

Issue 118, gets the dimension parameters defined and initialized in the right place.
This issue, 83, addresses all the changes needed in the areas outlined above.
I'm about to add /physicalDesign/otherDesignInfo/dimensions-230209.ods/pdf which is a spreadsheet with the key physical measurements defined and recorded.

This has been implemented, but not tested. Basically re-used existing code with small changes:

used new symbols starting with d$... for all physical dimensions, and used these symbols in place of previous symbols, or numeric measurements
removed references to the displacement angle and distance that was part of the previous foot design where the toe was not on the vertical axis of the ankle servo. So we're using the same algorithms, except that the offset is zero degrees, and the toe is displaced by 0 cm from the vertical axis of the servo. It might be possible to simplify the code a bit, but in theory, "this should work"

Changes for this uploaded to git in branch 118b

need to do testing to see if the algorithms still work before closing

noted that our exist movement scripts are no longer totally valid, although they made still work reasonably well. They specify most toe positions as absolute numbers, in either local or global coordinates, which may no longer apply. For example, many scripts start in the neutral stance, by specifying the toe position coordinates when the servos are all centered. However, because the dimensions of the robot have changed, going to the coordinates in the script will no longer get the neutral stance, and the servos will be slightly off center (knee and foot, anyway.
I don't think there's a way to do a program controlled conversion of the movement scripts, because i's hard to deduce what the original intention was.
Hmmm, one thing we need to do is update the movement script compiler. For example, it has a command something like "go to the home position" which now means a different set of coordinates. I'll try to put an issue in the right repository for that.

compiler issue for adjustments due to new dimensions is #52 in the compiler repository


fix iframe resize and scroll

fix market/rewards iframe resize/scroll on safari

@AlexeyKosinski i've fixed it on beta.


site-specific DNA replication termination at RTS1 barrier query regulation parentage

Parentage https://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0071171
site-specific DNA replication termination at RTS1 barrier
becomes annotated to
"GO:0031494    regulation of mating type switching"
by transitivity.
I don't think this is really regulating mating type switching, but I can't immediately see where the problem is.
GO:0071171    site-specific DNA replication termination at RTS1 barrier
is only used by PomBase (for rtf1 &2)

probably the problem is that
GO:0071515    genetic imprinting at mating-type locus
should be part_of mating type switching not
'regulation of mating type switching'

Fixed as requested

'Genetic imprinting' seems strange - first, I mostly find 'genomic imprinting' in the literature.
Second, it is defined as a regulatory process "Any process that modulates the frequency, rate or extent of the chemical reactions and pathways involving macromolecules by a mechanism that is mediated by DNA, is mitotically or meiotically heritable, or is stably self-propagated in the cytoplasm of a resting cell, and does not entail a change in DNA sequence."
I dont know how that is different from the child 'regulation of genetic imprinting', defined as "Heritable alterations in the activity of a gene that depend on whether it passed through the paternal or the maternal germline, but that are not encoded by DNA itself."  - I though this is what 'genomic imprinting' was, see https://en.wikipedia.org/wiki/Genomic_imprinting.
Finally, the parentage is weird - I would have expected it under 'GO:0040029 regulation of gene expression, epigenetic'.
Looks like @mah11 created at least 'genetic imprinting'; maybe @mah11 you can help clarify the definitions of these terms?
Thanks, Pascale

@ValWood
I made these changes, but I think our GO-CAM guidelines would state that a transcription regulation event regulates the downstream process, not that it's a part of that process.
What do you think ?

Mating type switching is
"Mating type switching in S. cerevisiae, which is one of the best understood programmed DNA recombination events, allows haploid yeast cells of one mating type to produce haploid cells of the other type, thereby allowing sister cells to mate and become diploid"
This 'imprint' here is a strand-specific modification that marks the DNA at the mat1 locus of switchable cells (Egel and Eie 1987; Klar 1987, 1990).
So the imprint could probably be part of  regulation of gene expression (it determines which mating factors are expressed), but it is part_of mating-type switching.
Some of these papers use 'genomic imprinting' in their keywords, so these 2 terms are probably interchangeable.

Looks like @mah11 created at least 'genetic imprinting'; maybe @mah11 you can help clarify the definitions of these terms?

No, I have never had specialist knowledge of imprinting, nor do I remember all the details of work I did over a decade ago. I have found the tickets for which these terms were added: #6680 and #6820. (I think the "much older item" referred to in the latter must be #1067, but at the moment GitHub is giving me an error for links to very old tickets; also see #4410 when things are working.)


Reason for downsampling

Hi Kai, I hope you're doing well -- this is JJ! Congratulations on this great work!
I have a question regarding some parts of the code.
https://github.com/Kai-46/ARF-svox2/blob/22baf6e000e519b21ca26308f8ead1f4d2675cf5/opt/opt_style.py#L618
In this line, I see that you are downsampling the rendered image to half to compute the losses.
But you could instead render an image with a lower resolution, which would be much more memory efficient.
Is there a reason for doing this downsampling? Did you want to do anti-aliasing, perhaps?
Thanks in advance!

Hmm, the downsampling step might be a bit ad-hoc there. If I remembered it correctly, I added the statement there mainly because when I was using the colorful triangle style, I'd like the triangles to be bigger in the stylized 3D appearance. In other words, the relative resolution between the content image and style image fed into the VGG feature extractor for calculating the NNFM loss can affect the style pattern size in the stylized appearance.

Got it -- thank you for the answer!


WARNING: deadline exceeded by linter

Hello! Thanks for your work, it really helps as keep our code healthy!
We encounter strange behaviour after releasing of 1.2 tag  , and as far as gopkg follows the major version, our travis build have been broken 😄
But the thing is that this error only appears in the travis.

That is probably due to the update of unused, staticcheck and go-simple. They are now about 30x slower than they used to be. You'll need to increase your deadline.


golint warning fixes

major fix the following golint warnings:

The incorrect comments
Convert ALL_CAPS variable name to Camels style
Inconsistent of receiver name
Sutter struct name - IngressController.IngressControllerSpec -> IngressController.Spec
If block ends with a return statement, so drop this else and outdent its block. Refer to golang/lint#17
Typo of the variable name. protocal -> protocol
Rename some variable. HttpClient  -> HTTPClient,  JsonToKVMap -> JSONToKVMap

Codecov Report

Merging #124 (2c7aa65) into main (dbb8ffb) will increase coverage by 0.08%.
The diff coverage is 47.05%.

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
+ Coverage   46.48%   46.57%   +0.08%
==========================================
  Files          38       37       -1
  Lines        2590     2585       -5
==========================================
  Hits         1204     1204
+ Misses       1305     1301       -4
+ Partials       81       80       -1

Impacted Files
Coverage Δ

pkg/common/callback.go
0.00% <0.00%> (ø)

pkg/common/option.go
0.00% <0.00%> (ø)

pkg/object/function/spec/fsm.go
94.44% <ø> (ø)

pkg/object/meshcontroller/spec/spec.go
52.36% <ø> (ø)

pkg/util/jmxtool/agent_controller.go
0.00% <0.00%> (ø)

pkg/util/texttemplate/texttemplate.go
71.14% <ø> (ø)

pkg/util/signer/signer.go
77.58% <66.66%> (+0.29%)
:arrow_up:

pkg/common/utils.go
39.68% <75.00%> (ø)

pkg/filter/canary/matcher.go
97.18% <100.00%> (ø)

pkg/util/jmxtool/common.go
49.05% <100.00%> (-0.95%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Removed API-Key and PlanePassword logging

When calling the API API-Key and PlainText Password got logged.
Probably left in by mistake.

Thanks for catching that. Merging your PR.


Delete some messages from history

It would be useful to be able to delete some messages from the history that are no more important. That can dramatically reduce token count in longer conversations.
For example delete filter like this would be useful:
memory.delete((message) => message.additional_kwargs.my_metadata == true)

You can do this by manipulating/reinitializing the underlying ChatMessageHistory object:
import { BufferMemory, ChatMessageHistory } from "langchain/memory";
import { HumanMessage, AIMessage } from "langchain/schema";

const pastMessages = [
  new HumanMessage("My name's Jonas"),
  new AIMessage("Nice to meet you, Jonas!"),
];

const memory = new BufferMemory({
  chatHistory: new ChatMessageHistory(pastMessages),
});

https://js.langchain.com/docs/modules/memory/how_to/buffer


Error for npm test

Not sure how to resolve this error
EDIT: tests work in chrome so this may just be a firefox issue

I am running tests locally and on Travis using Firefox, and it is working.   I think I've seen this before though.  I believe nuking node_modules and re-installing fixed it.

I've tried the following steps but it doesn't seem to resolve the issue:
git clean -fdx
npm install
pip install -e .
jupyter serverextension enable --py jupyterlab
jupyter lab

Perhaps restart your terminal and/or OS?

Yes I tried restarting and cleaning/re-installing and then restarting terminal again but neither seems to do the trick

Note from @jacpan on Gitter: the problem was importing code from src/ instead of lib/.  It may be worth trying to recreate the error condition and seeing if we can figure out how to get a more informative message in a catastrophic failure.

Closing for now, please feel free to re-open if needed.


Update odh-manifests 8/18/21

Signed-off-by: Landon LaSmith<EMAIL_ADDRESS>

This PR replaces https://github.com/red-hat-data-services/opendatahub-operator/pull/89

/retest

/retest