Add kitchen azurerm (#1)

Add support for kitchen-azurerm

Fix issue with closing braces

Fix issue with closing braces

Dedupe provider

Add missing end

Update README with kitchen-azurerm instructions

Increment version to 1.2.1

Fixes #12
Signed-off-by: Matt Stratton<EMAIL_ADDRESS>

There are a few empty lines that we need to remove, also a couple of comments but overall this looks great!

I like that you put instructions on the README. The only thing I would recommend is that if the azure gem really depends on the dev-tools we should verify that they are installed and if they are not, just help the end user with a nice message like the one you put on the README telling how to fix this. What do you think about hat @mattstratton ?

this is gonna be so awesome

@afiune I cleaned up your requests - I'm not quite sure what the best method for testing on the build tools is going to be, without making things super crazy. I think maybe if we just catch an exception on this portion:
chef_gem = Chef::Resource::ChefGem.new('kitchen-azurerm', run_context)
    chef_gem.run_action(:install)

I'm open to suggestions.

With 2cbe529a4316789a495b5e6379033965b005fcce the following error message gets logged if the azure gem installation fails:
[2016-11-22T23:17:52+00:00] ERROR: Azure gem installation failed. You might need to ensure that dev tools are installed.
[2016-11-22T23:17:52+00:00] ERROR: Add depends 'build-essential', '~> 7.0.2' to the metadata.rb of your build cookbook and add include_recipe 'build-essential::default' to the default.rb of your build cookbook

I've made the changes suggested by @afiune, and I've also successfully run this through my own Automate cluster testing both a Linux VM and a Windows 2012r2 VM.
You can check out the log of a successful Windows run in Acceptance here - https://gist.github.com/mattstratton/bde1f8e2aec0863140d57a0524e9537e
One thing I did notice is that if I tried to run several Windows VM's, it throws an error, which is not very descriptive. I suspect it is timing out; when/if we implement a fix for #10, that will help with this.

@mattstratton this is a great first iteration. I like it so do you wanna submit this through our internal workflow server or you want me to push it? Great work man!

@afiune If you can push it through the internal workflow server that would be great. It's probably a bit of a better test to have someone other than me try it! (I've run it through my own Workflow server)

@mattstratton ok just pushed and you have some failures.. wanna take a look? https://delivery.chef.co/e/chef/#/organizations/Delivery-Build-Cookbooks/projects/delivery-sugar/changes/60bf39b7-5eaf-488f-bc24-67cb0dd26553/status/verify

I'll take a look at it a bit later - I'm at the airport and I'm mobile so I can't get into the cluster :)

So here's a topic for discussion - the main "tough" warning is that the TestKitchen class is "too long", which means it's more than 100 lines.
So we have two ways to fix this:

Override the rubocop class length check for that class
Refactor the whole class to pull the driver-specific stuff out (I'm guessing you would end up subclassing TestKitchen based upon the driver)

Option 2 seems like the "better" thing to do, but it's a pretty big change, and then we also have to regression test the ec2 stuff. What do you think about doing option 1 and putting in an issue to fix that later?

Yeah, we can go for lotion one and please create the issue so I can look at it latter. I want this to get merged and see how users adopt/use it so we can have feedback. Thanks Matt!

I've updated the formatting so we have clean Rubocop tests - I'm running a couple local tests first before signing-off on this to be merged.
Before we merge, the following should be done:

[ ] Create new issue to clean up the TestKitchen class to make it shorter
[ ] Matt to run a test using v1 build nodes to create functional test of a cookbook using the driver
[ ] Matt to run a test using v2 runners to create a functional test of a cookbook using the driver

Merged in with 905c76e0a8972f668e8d6c468c80912752869544


How to use this gem if user's timezone is known in the database?

Timezone is saved in the users table, set by the user with a time zone select.  How would I switch to using the logged in user's stored timezone instead of the browser one?

I wouldn't use this gem.  Just create an around_filter in application_controller that sets the time zone based on the user.
—
Sent from Mailbox
On Tue, Jun 30, 2015 at 2:45 PM, Ryan Koehler<EMAIL_ADDRESS>wrote:

Timezone is saved in the users table, set by the user with a time zone select.  How would I switch to using the logged in user's stored timezone instead of the browser one?
Reply to this email directly or view it on GitHub:
https://github.com/kbaum/browser-timezone-rails/issues/20


[RELEASE] Data Prepper 1.5.0

Is your feature request related to a problem? Please describe
The Data Prepper team plans to release Data Prepper 1.5.0.
Planned release date: Jun 16, 2022
Describe the solution you'd like
After the Data Prepper maintainers completes the staging release build, please run the promotion job. A Data Prepper maintainer will supply the VERSION and DATA_PREPPER_BUILD_NUMBER parameters.
Describe alternatives you've considered
No response
Additional context
No response

Input parameters:
VERSION: 1.5.0
DATA_PREPPER_BUILD_NUMBER: 19

Maven out.
https://hub.docker.com/r/opensearchproject/data-prepper/tags
docker out
https://artifacts.opensearch.org/data-prepper/1.5.0/opensearch-data-prepper-jdk-1.5.0-linux-x64.tar.gz
tarball out

Close issue as we have released the artifact.


script doesn't working error 500

all examples getting error 500 with below log:
[20-Mar-2019 12:19:43] PHP Fatal error:  Uncaught InstagramScraper\Exception\InstagramAuthException: User credentials are wrong. in /home/drrma/vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php:1299
Stack trace:
#0 /home/drrma/index.php(10): InstagramScraper\Instagram->login()
#1 {main}
thrown in /home/drrma/vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php on line 1299
is there anyone help me, please!!!

Try to use cli method for login.
This with method you can enable the two factor authentication.

Try to use cli method for login.
This with method you can enable the two factor authentication.

how can i do that? i am new to this script.

After you download the package create a index.php file with this code:
`require_once DIR . '/vendor/autoload.php';
use InstagramScraper\Instagram;
Unirest\Request::verifyPeer(false);
/* THIS IS CAN INITIALIZE ONLY FOR THE FIRST TIME WITH php command in CMD for 2 FACT - AUTHENTICATOR
*

$instagram = Instagram::withCredentials('username', 'password', '/access');
$instagram->login(false,true);

*/`
And after this you can run index.php with cmd on windows or terminal on osx
go to folder and write php index.php
and you follow the instruction for authentication on CDM or TERMINAL

After you download the package create a index.php file with this code:
require_once __DIR__ . '/vendor/autoload.php';

use InstagramScraper\Instagram;

Unirest\Request::verifyPeer(false);

/* THIS IS CAN INITIALIZE ONLY FOR THE FIRST TIME WITH php command in CMD for 2 FACT - AUTHENTICATOR */

 $instagram = Instagram::withCredentials('username', 'password', '/access');
 $instagram->login(false,true);

And after this you can run index.php with cmd on windows or terminal on osx
go to folder and write php index.php
and you follow the instruction for authentication on CDM or TERMINAL

it has been make access directory with a file in it, i was upload it in my host and use the below code but i have error 500 yet. what should i now? :((
`<?php
require_once DIR . '/vendor/autoload.php';
use InstagramScraper\Instagram;
$instagram = Instagram::withCredentials('exampleuser', 'examplepass', '/access');
$instagram->login(true,false);
$account = $instagram->getAccountById(3);
echo $account->getUsername();
?>`

in your host can you run a php command?

Otherwise you can run a command in local folder on your pc after you install php and you can run this command php index.php with cmd or terminal.

i was run this command in my windows and then upload access folder in my host directory

Do you have teamviewer? if you want i can help you with TW. send me a credentials same ID and CODE via mail<EMAIL_ADDRESS>

i sent the tw details. thank you very very much bro


fix(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-cbr to v1.1.3

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-cbr
module
patch
v1.1.2 -> v1.1.3

Release Notes

terraform-ibm-modules/terraform-ibm-cbr
v1.1.3
Compare Source
Bug Fixes

fix issue where only first network zone is attached to CBR rule (#93) (3b6937b)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix(Pagination): with pagination fix hidden ellipsis

Fixes #3266.
This provides a fix for correct computations of pagination items with hidden ellipsis and adds a simple example.

Codecov Report

Merging #3271 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #3271   +/-   ##
=======================================
  Coverage   99.92%   99.92%
=======================================
  Files         169      169
  Lines        2803     2803
=======================================
  Hits         2801     2801
  Misses          2        2

Impacted Files
Coverage Δ

src/addons/Pagination/Pagination.js
100% <100%> (ø)
: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 e3d933a...ce8d347. Read the comment docs.


[FEATURE] Add global setting or per-volume setting for enabling filesystem trim on encrypted volumes

Is your improvement request related to a feature? Please describe (👍 if you like this request)
We've already provided steps for enabling filesystem trim on encrypted volumes in the official document. However, it's really handy for users.  We can add a global or per-volume setting for enabling or disabling this functionality, make it more transparent to users, and users can manage them easily inside the Longhorn system.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Moving to 1.5 to make the story complete. Also need to respect different types of volume, RWO & RWX.

Need to add the doc in the new setting and let users understand the security issues after enabling trim on encrypted volume.

We have some concerns about the security and data loss for this function. Revisit this later.
https://github.com/longhorn/longhorn-manager/pull/1784#discussion_r1187592468

Any alternatives to the current manual approach to enabling encrypted volume trim?
I have a problem with encrypted volumes bloating due to unscheduled snapshots.
Current manual approach:
Remote in to all storage nodes
Get list of all longhorn pvc's
ls -l /dev/longhorn
Set allow discards
sudo cryptsetup --allow-discards --persistent refresh <pvc-ID>
Enter passphrase
Enter passphrase for /dev/sdk: <Longhorn_Encryption_Passphrase>
Manually trim filesystem
The manual approach works, but doesn't seem feasible for a production cluster that is based on automation

@Jamison1
Before implementing the feature, I think there is no quick way.
Before adding the feature, we'd like to get more feedback. Are you concerned about the security risks in a production environment?
Warning: Before enabling TRIM on a drive, make sure the device fully supports TRIM commands, or data loss can occur
WARNING: This command can have a negative security impact because it can make filesystem-level operations visible on the physical device. For example, information leaking filesystem type, used space, etc. may be extractable from the physical device if the discarded blocks can be located later. If in doubt, do not use it.

We use multiple layers of encryption for data at rest.  We also are fairly well versed in trim availability.
If I understand the warning on encryption with respect to the negative security impact - This does not make it less secure than if one disabled encryption entirely and trim'd the drive - correct?  Or am I missing something here?
I think those warnings are appropriate for the documentation.  Perhaps expand on that somewhat to make it more clear for those who check boxes as part of their security protocol.  Also in the UI volume tab when you hover with the cursor over the lock icon it says "Encrypted Volume" - perhaps adding to that to say "Trim Enabled" as a notice/warning.
I would prefer to see the ability to auto trim encrypted volumes enabled through the helm chart values - at time of longhorn installation (this includes any recurring jobs needed to delete, cleanup unscheduled snapshots and trim after).  So that encrypted pvc creation doesn't require any additional UI related steps after creation.
We rely heavily on automation with extensive testing beforehand so any manual processes like having to break out my "10 steps for encrypted Longhorn volumes" with the UI or CLI really throw a wrench in the automation.  We use the UI primarily for monitoring the storage health - and watch the volumes expand due to unscheduled snapshotting without trim.
Additionally, you could also show that encrypted trim is enabled in UI volume settings for encrypted volumes with the option to disable encrypted trim using cryptsetup --persistent <pvc-id> or something similar since trim can also be disabled if needed.
When we spin up a cluster, we install quite a few other items after Longhorn with helm that create encrypted pvc's as part of the same automated process.
Without the ability to trim encrypted volumes they can become unusable after XX days due to volume expansion with unscheduled snapshotting and multiple replicas.  IMO this is needed if using encrypted volumes in production.

@derekbit
Update on encrypted volume expansion.  In testing a new loki write volume it started at 139Mi and after 13 hours it was at 4.45Gi.  As its a test cluster obv it hasn't written 4G in logs.  This appears to be from unscheduled snapshots.  After enabling allow-discards on the encrypted volume along with allow snapshots removal during trim and selecting Trim Filesystem - it dropped from 4.45Gi to 259Mi.  In a matter of minutes it then expanded up to 300Mi - So unless there is some other reason for this volume expansion - Encrypted volumes don't seem very production friendly at the moment without enabling allow-discards.

FYI:
we are using just encrypted volumes in our cluster that are using the same key longhorn-crypto.
With the following DaemonSet deployed as a workaround we are enabling discard for all our volumes every hour.
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: longhorn-setup-cypto-discard
  namespace: longhorn-system
spec:
  selector:
    matchLabels:
      name: longhorn-setup-cypto-discard
  template:
    metadata:
      labels:
        name: longhorn-setup-cypto-discard
    spec:
      containers:
      - name: longhorn-setup-cypto-discard
        command: ["sh", "-c", "apk add cryptsetup device-mapper && while true; do echo updating now; for LONGHORN_VOLUME in `ls /dev/longhorn/`; do cryptsetup --key-file=/etc/luks/longhorn-crypto/CRYPTO_KEY_VALUE --allow-discards --persistent refresh $LONGHORN_VOLUME; done; echo done; dmsetup table; sleep 1h; done"]
        image: alpine
        # do not specify resource limits for memory because cryptsetup is memory mapping the stuff
        volumeMounts:
        - name: dev
          mountPath: /dev/
        - name: longhorn-crypto
          mountPath: /etc/luks/longhorn-crypto/
        securityContext:
          privileged: true
      terminationGracePeriodSeconds: 5
      volumes:
      - name: dev
        hostPath:
          path: /dev/
      - name: longhorn-crypto
        secret:
          secretName: longhorn-crypto

Won't fix due to security concern.


Fix change length bug

Fix #93
The PR fixes a bug in change length calculation, removes trimming of the end coordinate before sending out to VEP for annotation, and prettifies the GenomicInterpretation JSON snippets.
Moreover, a documentation section regarding tests is added. From now on, we have 2 kinds of test, online and offline. The online tests must be marked by @pytest.mark.online to not run them in standard CI due to spurious failures. In the long run, we should remove the online tests altogether and use mocks instead. The offline tests are just the regular tests.

There is something wrong with Uniprot REST vs. Github runners. The tests run just fine on my machine.

Sometimes Uniprot gets overloaded, which is what looks like happened when your checks ran. I usually give it some time and it works again.
The PR overall looks good! Feel free to merge!


python311Packages.clarifai-grpc: 10.2.3 -> 10.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.clarifai-grpc is: Clarifai gRPC API Client
meta.homepage for python311Packages.clarifai-grpc is: https://github.com/Clarifai/clarifai-python-grpc
meta.changelog for python311Packages.clarifai-grpc is: https://github.com/Clarifai/clarifai-python-grpc/releases/tag/10.3.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.3.0 with grep in /nix/store/vxxyvwzif10ziwi1whzgxysfr50khzr0-python3.11-clarifai-grpc-10.3.0
found 10.3.0 in filename of file in /nix/store/vxxyvwzif10ziwi1whzgxysfr50khzr0-python3.11-clarifai-grpc-10.3.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.clarifai
python311Packages.clarifai-grpc
python312Packages.clarifai
python312Packages.clarifai-grpc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vxxyvwzif10ziwi1whzgxysfr50khzr0-python3.11-clarifai-grpc-10.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.clarifai-grpc https://github.com/r-ryantm/nixpkgs/archive/8c0f4ca85a61fc22493857d5ceb7caa8095be8b7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8c0f4ca85a61fc22493857d5ceb7caa8095be8b7#python311Packages.clarifai-grpc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vxxyvwzif10ziwi1whzgxysfr50khzr0-python3.11-clarifai-grpc-10.3.0
ls -la /nix/store/vxxyvwzif10ziwi1whzgxysfr50khzr0-python3.11-clarifai-grpc-10.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

8 packages built:

python311Packages.clarifai
python311Packages.clarifai-grpc
python311Packages.clarifai-grpc.dist
python311Packages.clarifai.dist
python312Packages.clarifai
python312Packages.clarifai-grpc
python312Packages.clarifai-grpc.dist
python312Packages.clarifai.dist

Maintainer pings
cc @natsukium for testing.

Result of nixpkgs-review pr 303402 run on x86_64-linux 1

8 packages built:

python311Packages.clarifai
python311Packages.clarifai-grpc
python311Packages.clarifai-grpc.dist
python311Packages.clarifai.dist
python312Packages.clarifai
python312Packages.clarifai-grpc
python312Packages.clarifai-grpc.dist
python312Packages.clarifai.dist


Feature/config translator ii

LGTM for the most part.  However, there are several classes that are essentially dead code for the public.  I would recommend pulling them into our enterprise edition as we will keep config translation for older versions alive there.  Thoughts?

I will move some of classes  to EE gateway server,   so hold off on that pull request till tomorrow.

@dpwspoon applied all code review feedback

Looks like the test failures are related to your changes :-(

@mjolie reviewed. Looks good almost there

@dpwspoon ok ready for another review

retest this please

Thanks @mjolie


fsutil

Instead of having API endpoints, like POST /dirs/path/to/new/dir, that allow clients to manipulate the file system, we can provide access to the bundled global file system through a utility function fsutil.
Users can assume there is a gate at the path /bin/USER,USER/fsutil that links the function fsutil. Further, such a gate grants the privilege USER/fsutils* and requires the user privilege USER for invocation.

One technical problem here is to handle large files. Large files are data too large to be piggybacked on the invocation payload*.
An approach is to store large files into the blob store and piggyback the blob handle on the invocation payload.
We can have the client (CLI or web browser) to submit a HTML form that takes a file.
* Amazon Lambda sets the payload size limit at 6MB (sync) and 256KB (async).

fsutil should be synchronous from the point view of a client.


add new input for protein fasta file and gff

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/metatdenovo branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

There are a couple of template files that look strange, see comment.
Do you check that the user specified both protein (protein_fasta if you agree with my change) and gff -- we should.

What do you mean? to use this option, the user needs to specify both parameters, otherwise the pipeline will run with the specified orf_caller

There are a couple of template files that look strange, see comment.
Do you check that the user specified both protein (protein_fasta if you agree with my change) and gff -- we should.

What do you mean? to use this option, the user needs to specify both parameters, otherwise the pipeline will run with the specified orf_caller

I mean that you should check that if one of them is set, the other is too and if one is missing execution should stop with a message.

There are a couple of template files that look strange, see comment.
Do you check that the user specified both protein (protein_fasta if you agree with my change) and gff -- we should.

What do you mean? to use this option, the user needs to specify both parameters, otherwise the pipeline will run with the specified orf_caller

I mean that you should check that if one of them is set, the other is too and if one is missing execution should stop with a message.

ok, I can do that.


Expand Bottle Cap Factories at the beginning

Expand the Bottle Cap Factories at the beginning for an easier start into the game
(https://www.reddit.com/r/incremental_games/comments/drhq4m/beerplop_is_now_open_source_feel_free_to/f6jk65y?utm_source=share&utm_medium=web2x)

Released


(include mod) depends on CWD / fennel.path value

Hm..
test/macros.fnl does use include and builds without issues.
Using elsewhere seem fails to find the module.

Does fennel.include hit the regular infra or use its own file searcher?

Seems it uses specials.search-module
(fn search-module [modulename ?pathstring]
(let [pathsepesc (escapepat pkg-config.pathsep)
pattern (: "([^%s]*)%s" :format pathsepesc pathsepesc)
no-dot-module (modulename:gsub "%." pkg-config.dirsep)
fullpath (.. (or ?pathstring utils.fennel-module.path)
pkg-config.pathsep)]
(fn try-path [path]
(let [filename (path:gsub (escapepat pkg-config.pathmark) no-dot-module)
filename2 (path:gsub (escapepat pkg-config.pathmark) modulename)]
(match (or (io.open filename) (io.open filename2))
file (do
(file:close)
filename)
_ (values nil (.. "no file '" filename "'")))))
Which uses utils.fennel-module.path, require("fennel").path == ./?.fnl;./?/init.fnl`

We now just insert fnl/ into the relevant paths when compiling, then undo any change.


Fix for #506 impossible to filter in null values

The fix for #427 (#428) is too aggressive in the sense that it prevents altogether to filter in a value of null.
Whatever the input in the filter input, it's impossible to include the line (it returns false systematically to the filter function on a null value)
I'd like to suggest a different fix. Instead of returning false, we set targetVal to the empty string in case the value is null or undefined. I believe this makes sense since the input from the user will be mostly text based (we could imagine custom filters for which it's not true) but in most cases it is. So assuming that having a value of null or undefined maps to an empty string seems sensible.
With that change, it becomes possible to filter in null values using the regex filter for instance and specifying a regex that only allows an empty string.

@jfremy, I think it's good for fixing this issues :)

@jfremy, released on v2.3.6. Thanks your contributions :)


Login webView crashes on load, sdk 0.6.1, Android 6.0+ devices

I've worked with uber android sdk last 6 month and two days ago I see web login form stopped work on Android 6.0 devices - there is some error occurs(details are below) during just the page loading, right after Facebook button showing. Same code without changes has worked for me last 5 month on same devices and it's still working for android devices with versions 4.1-5.1.
I'm using com.uber.sdk.android.core.auth.LoginManager#loginForAuthorizationCode method.
Error: net::ERR_NAME_NOT_RESOLVED, code: -2
Stack trace:
"main@16225" prio=5 runnable
  java.lang.Thread.State: RUNNABLE
	  at com.uber.sdk.android.core.auth.LoginActivity$OAuthWebViewClient.onReceivedError(LoginActivity.java:223)
	  at com.android.webview.chromium.WebViewContentsClientAdapter.onReceivedError2(WebViewContentsClientAdapter.java:235)
	  at org.chromium.android_webview.AwContentsClientCallbackHelper$MyHandler.handleMessage(AwContentsClientCallbackHelper.java:38)
	  at android.os.Handler.dispatchMessage(Handler.java:102)

Screenshot of the error with its description

Screenshot of the page on the breakpoint:

Devices, used for test:

Samsung Galaxy S6, SM-920F, Android 6.0.1
Samsung Galaxy S7, sm-g930f, Android 6.0.1
HTC Nexus 9, Android 7.0

@tyvsmith Any comment? I still see the issue

Yeah, please update to the latest and use the chrometabs impl.

Thank you!


JWT instead of access_token?

I want to create my own custom Google login button and the design given by Google when using the GoogleLogin component does look too good for me to use.
Is it possible to get a JWT instead of the access_token when using useGoogleLogin hook, like when using the GoogleLogin component?
Thanks!

Hello @Rec0iL99,
Yup you can, we can use custom buttons with useGoogleLogin in two ways
Token flow
which will return you an access_token to talk to google APIs providing it in Authorization header in requests. unfortunately, google will not return JWT has user info, but you can still send a request to google to fetch user info
GET: https://openidconnect.googleapis.com/v1/userinfo don't forget to provide the access_token in Authorization header.
Code flow
which will return you an authorization code that you will give to your backend to exchange it with an access token (JWT) which will include user info, refresh token, and other stuff.

tell me if it didn't work with you, I would appreciate helping you.
If it worked :D, you can close the issue whenever you finish.

Correct me if I'm wrong...there's no way to get a JWT from Google when using the useGoogleLogin hook?

in code flow yes you can, all you need to exchange code to get access_token, refresh_token and id_token  (id_token is the JWT that has all info about the user)
** Client **
const googleLogin = useGoogleLogin({
  onSuccess: async ({ code }) => {
    const tokens = await axios.post('http://localhost:3001/auth/google', {  // http://localhost:3001/auth/google backend that will exchange the code
      code,
    });

    console.log(tokens);
  },
  flow: 'auth-code',
});

** Backend (using express)**
require('dotenv').config();
const express = require('express');
const {
  OAuth2Client,
} = require('google-auth-library');
const cors = require('cors');

const app = express();

in code flow yes you can, all you need to exchange code to get access_token, refresh_token and id_token  (id_token is the JWT that has all info about the user)
** Client **
const googleLogin = useGoogleLogin({
  onSuccess: async ({ code }) => {
    const tokens = await axios.post('http://localhost:3001/auth/google', {  // http://localhost:3001/auth/google backend that will exchange the code
      code,
    });

    console.log(tokens);
  },
  flow: 'auth-code',
});

** Backend (using express)**
require('dotenv').config();
const express = require('express');
const {
  OAuth2Client,
} = require('google-auth-library');
const cors = require('cors');

const app = express();

app.use(cors());
app.use(express.json());

app.post('/auth/google', async (req, res) => {
  const today = new Date();
  const { tokens } = await oAuth2Client.getToken(req.body.code); // exchange code for tokens
  console.log(tokens);

  res.json(tokens);
});

app.listen(3001, () => console.log(`server is running`));

in token flow, you can't get JWT but also you don't need to get it to get user profile info.
Google gives you access_token to talk with their APIs
const googleLogin = useGoogleLogin({
  onSuccess: async ({ access_token }) => {
    const userInfo = await axios.get(
      'https://www.googleapis.com/oauth2/v3/userinfo',
      { headers: { Authorization: `Bearer ${access_token}` } },
    );

    console.log(userInfo);
  },
});

** userinfo response **
{
    "sub": "xxxxxxxxx",
    "name": "Mo'men Sherif",
    "given_name": "Mo'men",
    "family_name": "Sherif",
    "picture": "xxxxxxx",
    "email"<EMAIL_ADDRESS>    "email_verified": true,
    "locale": "ar"
}

@MomenSherif I was away for a few days and couldn't respond in time to your reply. The token flow fits my needs perfectly and the code you gave works perfectly 🎉.
Thanks!

Thank you @MomenSherif , it runs 🗡

@MomenSherif Any chance you can show a full react app using the  component that works properly with the express backend you've supplied above? It's not clear to me why you're using the useGoogleLogin function instead of the GoogleLogin component, or if that won't work how to integrate the useGoogleLogin() inside a react app. Just trying to piece it all together and there don't seem to be any full working examples. Thanks!

You can use theGoogleLogin button to integrate with backend but you will not get access token or refresh token
If you want to only get user info yea you can do that and your backend can verify the id token against google which you obtain from GoogleLogin
If you are using node, google api package has verifyToken method which do that
Or you can find the corresponding example for your backend in Google's Docs

It's what google response with
I just abstracted the logic to be React friendly and easy to use
They provide the login button for simple authentication only no authorization allowed

@MomenSherif is there a way to use this GoogleLogin button and the google passport library in express in the backend?

in code flow yes you can, all you need to exchange code to get access_token, refresh_token and id_token (id_token is the JWT that has all info about the user)
Client
const googleLogin = useGoogleLogin({
  onSuccess: async ({ code }) => {
    const tokens = await axios.post('http://localhost:3001/auth/google', {  // http://localhost:3001/auth/google backend that will exchange the code
      code,
    });

    console.log(tokens);
  },
  flow: 'auth-code',
});

Backend (using express)
require('dotenv').config();
const express = require('express');
const {
  OAuth2Client,
} = require('google-auth-library');
const cors = require('cors');

const app = express();

app.use(cors());
app.use(express.json());

const oAuth2Client = new OAuth2Client(
  process.env.CLIENT_ID,
  process.env.CLIENT_SECRET,
  'postmessage',
);

app.post('/auth/google', async (req, res) => {
  const { tokens } = await oAuth2Client.getToken(req.body.code); // exchange code for tokens
  console.log(tokens);

  res.json(tokens);
});

app.listen(3001, () => console.log(`server is running`));

FYI, i did it without using backend, here's a refernece if anyone is interested.

Yea you can, but you shouldn't expose your google secret,
This is why it's recommended to do it in BE

Yea you can, but you shouldn't expose your google secret, This is why it's recommended to do it in BE

you're right, but i'm using Next.js, so will convert those variables and this api call to be on server side (for security), was just sharing if anyone needed this, thanks so much.


rename from gazelle-PKG to meteor-gazelle:PKG

As the header says. I decided to go with meteor-gazelle:PACKAGE as opposed to meteor-gazelle-PACKAGE as the former is clearer on the package's role as a subset of meteor-gazelle, rather than a standalone package.
Summary:

Changed the folder names from gazelle-PACKAGE to meteor-gazelle:package.
Changed the corresponding names in .meteor/packages and .meteor/versions as well.

@ahaym Looks good. Be sure to close issue #102

@ajax-wcd Am I able to close the issue without write permission or issue ownership? I don't see the close button on the issue page.

@ahaym I'm sorry, didn't realize the lack of permissions. The issue will be closed now that you've referenced it after I merge this pull request. Thanks!

Um, won't this break the repository for anyone using Windows? afaik, : isn't a valid file name character on Windows.

@thomwiggers Wow good point. Didn't even occur to me. We may need to change the folder names to meteor-gazelle-something in that case. I'd say if we're using dashes in the folder names than use a dash in the package name as well to avoid confusion.

I learnt this the hard way on another project, but regardless:
The folder names DON'T matter, it just uses the names put in package.js config. The folder could be named "blahblah" and the name parameter in the package.js could be "meteor-gazelle:module". So you'd have the best of all world, ":" delimited package names, and folders that don't break windows.
Also, naming the package folder "meteor-gazelle-module" every time just seems incredibly redundant.

@borghunter-wcd We have swapped it already to follow the '-' delimiter from #124. We could go the ':' route with parameterized package files if we want to flip flop again, however I dont see how "meteor-gazelle-module" vs "meteor-gazelle:module" reduces redundancy.

@rhomes-wcd oh, i wasn't suggesting that at all for file names, just perhaps the "core" package folders could go without "meteor-gazelle" entirely, and just be named "module", rather than "meteor-gazelle(-|:)module"
But whatever, it's not important, since, like I said, the folder names don't change anything and can be changed in the future if it becomes an actual issue.


Pulling an object changes how it behaves when collided with

Description

Pretty sure I know what the issue is, probably something to do with a certain property not being unset when it's let go
Reproduction

Run into locker or piano, note how it does not move
Pull and let go of locker/piano
Run into locker or piano, note how it wiggles and rotates and never goes back to how it was before

Screenshots

Additional context

https://github.com/space-wizards/space-station-14/blob/master/Content.Shared/Movement/Pulling/Systems/PullingSystem.cs
Offending line of code is in TryStartPull()

        _physics.SetFixedRotation(pullableUid, pullableComp.FixedRotationOnPull, body: pullablePhysics);

Deleting this line fixes the behavior and pulling things still works as intended (things that don't rotate, don't rotate, things that do rotate, continue to rotate)
But like... why?

Okay, I went through every single assignment of type: Pullable in the codebase and not a single component sets FixedRotationOnPull to any value


Fix of quick diagonal drag event

slider freezes with not ended animation

@Sundved can you give more info about the PR

Thanks for this wonderful PR
Closing in favor of c9735a7


two same addresses in the addressbook - address duplicate

The App check about the same label but no about the same address... I don't know if make any sense to have the same address under different labels.
@zancas What do you think? this make sense?

It's plausible that this caused by the same bug as the memo-duplication that @AloeareV is working on.   Can we add a test that surfaces the bug?   @fluidvanadium ?

Oh, this is more simple then that... sorry for my bad explanation... I'm going to try again:
In zingo-pc you can create the address book by yourself (it is a JSON file in the PC) and you have to fill two values to add a row, label & address. You manually can create the number of rows that you want, filling the label and copy/paste whatever address you want. It's for now, a really rustic feature.
And I noticed these:

you cannot create two records with the same label (I feel this is correct)
you CAN create two records with the same address (here is my question... I don't know if this make sense or not).

Maybe all of this doesn't matter because in a short term we have to change this behavior aligned with the new zingolib-memo implementation... @zancas

Ahh...  I see interesting.   I am not ready to have an opinion on this.  I need to think about it.

Closing this in favor of: #186


chore(ci/docker): enable cross-platform builds (closes #132)

All sorted @nimarion  🚀

Only issue with this, the base image you're currently using for maven only has a linux/amd64 build

Hey @nimarion, could you review this once again, please?


Remove not_implemented()

It turns out that adding not_implemented() obscures the error when implementing a child type. Here we'll get not_implemented() is not defined as the error, but really we shouldn't define it so the error will show the type that Julia is attempting to use (in case its deep within a function call)
See this post for more

I'll go ahead and add some docstrings later to make this more an implementation file, but I hopefully folks are happy with the overall change


Testability: add log records on test generation phases

Description
In order to check phases and their duration we need to have log info records.
Currently we can measure only overall test generation process.
It's needed to analyze which phase is time consuming.
Expected behavior
Add info log records for different phases of UnitTestBot test generation.

Test generation initiated
Collecting information (classes)
Fuzzing phase
Symbolic analysis phase
Minimization
Summaries
Code generation

Environment
IntelliJ IDEA (probably add records to idea.log?)
Potential alternatives
or separate UnitTestBot log ?
Context
Originally posted by @alisevych in https://github.com/UnitTestBot/UTBotJava/issues/1028#issuecomment-1273121257

Seems to be fixed in branch Vassiliy-Kudryashov/IDEA_hangs_during_test_generation_#1207

New log system description
Based on meeting with @korifey and @alisevych
Target users for whom it should be convenient:

Developers - runIde - utbot-intellij/log4j2.xml
QA - built plugin - settings from .utsettings
User scenario logs - built plugin- settings from .utsettings

Settings from utsettings:

Log level for concrete executor
Log config path for engine and concrete executor

Requirements:

Users should not send full utbot directory in issue - only useful logs from engine, idea and related concretes
Always logs for concrete, engine and idea
Info by default
Easy to lift loglevel to debug/trace - using ~/.utsettings
Easy to navigate to log files
Do not use console appender as it pollutes stdout and slows os - use file everywhere. Most probably RollingFileAppender.
For each concrete executor it should be easy to deduce from which moment in engine process this concrete executor
was started - from fuzzing/concrete and/or flex
Docs about all logging settings and example of usages
Navigation from one logfile to another should be easy - for example by click as in Rider
May be idea debug log for changing plugin log4j2.xml without rebuilding plugin
Concrete executor prints log to stdout which is redirected to file, may be better to file directly

Useful:

parralel generations are prohibited! - file appender is suitable

Additional information
Requirements

Users cannot upload files to GitHub via browser larger than 25MB
It can be LOG or ZIP - here is the github docs

Discussed with @alisevych:
It is required to add entries about phases execution, at least to indicate when each phase started and ended.
AFAIK following people should be notified:
@amandelpie @Vassiliy-Kudryashov @Markoutte @EgorkaKulikov
In case you need help with current logging - see doc and ask me

As far as most work is delivered to master. The rest will complete in future release.


Refactoring the install mechanism

Currently when installing a library it gets downloaded, extracted, and dependencies are recursively installed.
This isn't without issues, if something goes wrong in the middle things are left in a broken state, like #244 or you end up getting several versions of the same library #218
Because of that you have no idea how many libraries will get installed or how much space they'll use (similar as to why people want an upgrade summary #163).
I propose we change to a more "classical" approach, like the one found in most (all?) linux package managers and tools like them:

First you resolve the required libraries, for that we need access to haxelib.json #124 or storing in the db the dependency field
Second you present it all to the user, which libraries (name and version) will he get, how much download that is, and how much space they'll take
Third you download them, could even do it in parallel to increase speed. Resuming download after a failed attempt is easy since nothing was installed yet #133
Last you extract everything to disk and if there was no error you actually install (move) it, to avoid a broken half installed state.

It sound like a big change but the download, dependency find and install parts are already there, it's mostly a change in how they are sequenced. The new thing is to get the recursive dependency list before actually installing it.
Also a detail but we may want to only display the entire list of installed files in verbose mode.

I like the proposal. It looks to me that we should store dependencies in a Version db object and return that array in a VersionInfos structure from the SiteApi.infos method and then use it in the haxelib client. Additionally we could also display dependencies/dependants on a website.
However, since it's an addition to the public API, we should decide on the dependency format first (#238).
Another question is what's the process of migrating our server databases if we add that new field? cc @jasononeil @back2dos
As for providing haxelib.json without downloading the whole zip file (#124), I think it's a separate matter.

There is no process in place. The database is basically the same since a decade. The only change that was made (with Haxe 3) was to store the version components separately.
This seems like a very complex change, particularly if you consider the fact that in the future some dependencies are to be located in VCSs. I think the two don't go very well together.

I don't know, surely we probably (?) can't calculate the size when VCSs come into play, but we could still implement some kind of transaction system that can roll-back the whole command if something goes wrong. It should be done after the basic cleanup/refactoring though.


Fix user manual links to Incremental Build chapter

Links to incremental build chapter did not work and the chapter was not included in the single-page variant of the docs.

@bot-gradle test BD

I've triggered the following builds for you:

BuildDistributions build

@bot-gradle test and merge

I've triggered a build for you.


MD044: Fixes ignore word definitions

Package
Version

markdownlint-cli
0.27.1

markdownlint
~0.23.1

This issue is related to multiple, similar words that should all remain distinct. I would expect aws to become AWS, but I would not expect @aws-control-tower to become AWS.
I'm not sure if there's an order of precedence that matters here, but @aws-control-tower gets converted to AWS whether it's at names[0], names[1], or names[2].

The proper case is: @aws-control-tower

Just typing some related words here: aws-vault and AWS and @aws-control-tower are all correct.

<!-- markdownlint-configure-file {
    "proper-names": {
        "names": [
           "AWS",
            "aws-vault",
            "@aws-control-tower"
         ]
    }
} -->

https://dlaa.me/markdownlint/#%25m%23 Issue 409

The proper case is%3A %40aws-control-tower

Just typing some related words here%3A aws-vault and AWS and %40aws-control-tower are all correct.

<!-- markdownlint-configure-file {
    "proper-names"%3A {
        "names"%3A [
           "AWS"%2C
            "aws-vault"%2C
            "%40aws-control-tower"
         ]
    }
} -->

I expect this to be fixed by the change I already made for #403. With that change, longer names are prioritized first and any substring matches will not be considered. However, I will try your example to verify that. Thanks very much for the excellent demonstration of the problem!

Thanks. I keep finding small quirks as I'm documenting my code, so hopefully these reports will help make a better product for everybody!
And as an OSS maintainer myself, I want to say thank you for this project and all of the effort you put into it. It's some really great work!


Fix no-floating-promises lint violations

Explanation

When our ESLint packages were updated, one of the new rules that got enabled was no-floating-promises, which is violated if a promise is not properly awaited. To minimize alignment changes, this rule was overridden at the time to produce a warning rather than an error.
This commit corrects code that violates no-floating-promises by either awaiting promises where it is possible to do so, or by using a new utility function, logRejection, to allow the promise to continue unawaited while logging any possible rejections. This allows us to remove the ESLint override.
References

Changelog

@metamask/controller-utils

ADDED: Add logRejection function

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

I'll put this back in draft until I have some more time to update this PR.


Feature: enable throwing a SpamDetectedException when spam is detected

I think there should be an option for a SpamDetectedException be thrown. This enables an app to catch this and add their own fake success system globally. For instance, catch this exception in an exception listener, then redirect to homepage with a "thank you" flash to imply the form was processed correctly.
My thinking (at least for profiles), is a config like this:
antispam:
  profiles:
    default:
      throw: true

Or maybe a mode option as this option isn't compatible with passive mode:
antispam:
  profiles:
    default:
      mode: exception # one of "exception", "passive", "error"

I'm not a big fan of using exceptions for expected program flow (admittedly the same goes for static state variables).
How does the throw variant improve on the AntiSpam::isSpam option other than bypassing a few validators?

How does the throw variant improve on the AntiSpam::isSpam option other than bypassing a few validators?

It would remove the need for adding AntiSpam::isSpam() to each form submit logic. For my app, I'd use as a catch-all for any detected spam.
I do understand your concern though - feel free to close or put on hold until someone else requests such a feature.

It would remove the need for adding AntiSpam::isSpam() to each form submit logic. For my app, I'd use as a catch-all for any detected spam.

Doesn't a try/catch in each controller take up more space?   😆  You need a quite specific setup to be able to use an exception as a catch-all mechanism, and I think in that case it also warrants setting up your own listener for AntiSpamEvents::FORM_VIOLATION with a single throw, which would do exactly what you need I think?

Doesn't a try/catch in each controller take up more space?

I wouldn't do this, I'd have an exception listener to catch this specific exception, add a flash message and redirect to the homepage.

and I think in that case it also warrants setting up your own listener for AntiSpamEvents::FORM_VIOLATION with a single throw, which would do exactly what you need I think?

Yep, no hard requirement for something this bundle needs to provide.

I wouldn't do this, I'd have an exception listener to catch this specific exception, add a flash message and redirect to the homepage.

I see the merit in some specific situations but as it indeed requires quite a bit of custom setup on the application end to actually be efficient I don't see reason to add half the solution to the bundle.
Might want to put it in the advanced examples docs how to set it up like this.


[2.0.13, 2.1-rc6] Problem creating null values

2.0.13 and 2.1-rc6
I am unable to create null values in my database. For example:
POST http://orientdb:2480/command/database/sql
create vertex V content {"some_key": "some_val", "some_null": null}
HTTP/1.1 200 OK
{"result":[{"@type":"d","@rid":"#9:3","@version":1,"@class":"V","some_key":"some_val"}]}
My expectation is that the created record would include the field for "some_null".
Other observations:
Nulls are handled correctly in embedded data:
create vertex V content {"some_key": "some_val", "some_null": null, "some_embedded_data" : {"some_embedded_key": "some_embedded_value", "some_embedded_null": null}}
{"result":[{"@type":"d","@rid":"#9:4","@version":1,"@class":"V","some_key":"some_val","some_embedded_data":{"some_embedded_key":"some_embedded_value","some_embedded_null":null}}]}
The only way I've been able to actually create a null field is by:

In odb studio, go to schema browser
Click "+ new record"
Click add field
Specify a name and string field type
Click save
If I query this record I will actually get the null back:

 select from 9:5
{"result":[{"@type":"d","@rid":"#9:5","@version":1,"@class":"V","some_null":null}]}
However after this, any further attempt, either through studio or the rest api to add other null fields, will fail. ODB will quietly throw away updated fields containing null values.
PUT http://orientdb:2480/document/database/%239%3A5
{"@type":"d","@rid":"#9:5","@version":1,"@class":"V", "some_null":null, "some_other_null": null}
HTTP/1.1 200 OK
{"@type":"d","@rid":"#9:5","@version":2,"@class":"V","some_null":null}
I have tried fiddling with this to no effect:
ALTER DATABASE custom standardElementConstraints=false

Are we sure it is what we want?
in Javascript it make sense, since null and undefined are different values;
However in Java, we only have null.
In my opinion, inserting null values should be silently ignored. Running a SELECT query resulting with a column where all values are null is weird.
Also, assuming we do go ahead and allow null as a value, what is the difference between a row with a null value at column A, and a row without column A at all?
comments from the orient team?

A few assorted thoughts:

Silently throwing away user data is a terrible design decision. At minimum, the API should return an error indicating an attempt to insert malformed data in the database.

Regardless of the design decision, there is still a bug here, as it is possible to create null valued fields in some circumstances and not others (as detailed above)

Why is there a not_null flag for record properties if it's not intended that properties can be null? The distinction between "mandatory" and "not null" implies that null is intended to be a valid value.

Though an embedded blob is a somewhat different beast than a first class record property, it will be inconsistent if nulls are allowed in the former and not the latter.

A database product, especially one that advertises schemaless operation, should not limit the users in terms of how they want to represent their data. It should empower them. In my use case the data has some properties with null values, and I need to be able to know that the data I get out of ODB is the same as the data the I put into it.

A few thoughts back, although we need someone from the orient team to decide on those finally.

Usually I agree, but throwing 'null' values is not the same as throwing user data. Also, you can use a flag if you don't want orientdb to 'accept'('silently ignore') those null values (aka NotNull). Again though, design decision.

Agreed, but I don't think anyone should create any PR before any decision is made. The current state of both allowing values to be null, and silently ignoring them. is not good. Something needs to change, yes.

I understand why you would think that, and you might be right. That said though, currently the not null check against an inserted value of null, while mandatory is checked if there was no value, and there is no default value

I fail to see why; I'm not talking about the content of a blob; if the blob contains 10 bytes, where some of them are null, the value is not (should not be) considered null. I haven't played enough with blobs though. so I may misunderstand you.

I do understand, although in my opinion null is not  a value, so getting back some non-existing value doesn't make much sense. Also, consider the current case where you insert 'null' into 2 rows and not 3, although all 3 are gonna be null (what do you except them to be?). Creating another type just to allow null doesn't make sense, you might as well use your own pre-defined value to consider empty (0, "", another column etc..)

In fact, although I do not know every DB out there, I'm not aware of a single DB in which null can be considered having a null, but the value is empty. There are many problems doing that.

It is very much possible to create a vertex with null values, just that the particular syntax chosen by @ryan-stateless does not permit it:
orientdb {db=testdb}> create vertex V content {"some_key": "some_val", "some_null": null}

Created vertex 'V#9:0{some_key:some_val} v1' in 0.104000 sec(s).

orientdb {db=testdb}> create vertex v set some_key="some_val", some_null=null

Created vertex 'V#9:1{some_key:some_val,some_null:null} v1' in 0.013000 sec(s).

orientdb {db=testdb}>

Note that storing null values in the DB could lead to wastage of storage space (each field is stored in the record, including field name). IMO it makes sense to have a way to insert data that optimizes storage by not storing null values, while still allowing null values to be inserted via a different command.
note also, it is possible to update a field to null. this does not remove the field (there is a separate syntax for that, too). so if it is possible to somehow end up with a field with null value, it must be possible to create the document with the field value null, too - and this is supported, just not in the particular syntax "create vertex content {....}"
however, there is an inconsistency as pointed out by @ryan-stateless in the way records and embedded records are handled, so perhaps it is a bug - that null values in embedded documents are not stripped out by 'create vertex content {....}' command.

orientdb {db=testdb}> select @this.toJSON() from V

----+------+--------------------------------------------------------------------------------------------
#   |@CLASS|this
----+------+--------------------------------------------------------------------------------------------
0   |null  |{"@type":"d","@rid":"#9:2","@version":1,"@class":"V","some_key":"some_val"}
1   |null  |{"@type":"d","@rid":"#9:3","@version":1,"@class":"V","some_key":"some_val","some_null":null}
----+------+--------------------------------------------------------------------------------------------

2 item(s) found. Query executed in 0.105 sec(s).

orientdb {db=testdb}> update #9:3 remove some_null

Updated record(s) '1' in 0.029000 sec(s).

orientdb {db=testdb}> select @this.toJSON() from V

----+------+---------------------------------------------------------------------------
#   |@CLASS|this
----+------+---------------------------------------------------------------------------
0   |null  |{"@type":"d","@rid":"#9:2","@version":1,"@class":"V","some_key":"some_val"}
1   |null  |{"@type":"d","@rid":"#9:3","@version":2,"@class":"V","some_key":"some_val"}
----+------+---------------------------------------------------------------------------

2 item(s) found. Query executed in 0.015 sec(s).

Hi @all, yes that is a bug, we should support null also with create vertex content { ... } syntax, PR are welcome

@tglman i propose to overload ODocument.mergeMap() method to take another boolean parameter mergeNullFields, which if set to true causes null fields to be merged. then from OrientVertex, we explicitly call the overloaded method with mergeNullFields set to true. this way other code relying on existing behaviour of mergeMap to ignore null values are not affected. WDYT?

I don't know if this has already been covered, but this appears to be an issue when using 'set' as well.

fixed in 2.1 and develop, will be out with the next hotfix


Removed VisualizationServer and MLMD Writer

The issue resolved by this Pull Request:
Resolves: https://github.com/opendatahub-io/data-science-pipelines-operator/issues/495
Description of your changes:

This PR removes the VisualizationServer and MLMD Writer deployments for V2.
Testing instructions

Deploy https://github.com/opendatahub-io/data-science-pipelines-operator/blob/fa00b5830e49f7bd8b93d0f7ba7de6d920d25e3e/config/samples/dspa_simple.yaml. MLMD Writer should be deployed.
Deploy https://github.com/opendatahub-io/data-science-pipelines-operator/blob/fa00b5830e49f7bd8b93d0f7ba7de6d920d25e3e/config/samples/dspa_simple_v2.yaml. MLMD Writer and VisualizationServer should not be deployed.

Checklist

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-512
An OCP cluster where you are logged in as cluster admin is required.
To use this image run the following:
cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/512/head
git checkout -b pullrequest b22deb67dbcb2802f98ddcc34e89c820e761db2f
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-512"

More instructions here on how to deploy and test a Data Science Pipelines Application.

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-512

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-512

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-512

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-512

/lgtm


Fix the bug of "SHOW [FULL] TABLES" parsing

Fixes #1351
Changes proposed in this pull request:

Fix "show tables" parsing bug.
Adjust column label of "show full tables".

Pull Request Test Coverage Report for Build 5369

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 63.47%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/src/main/java/io/shardingsphere/shardingproxy/backend/jdbc/JDBCBackendHandler.java
0
3
0.0%

Totals

Change from base Build 5367:
0.003%

Covered Lines:
8008

Relevant Lines:
12617

💛  - Coveralls


Disable item added notifications for cap awards.

Disable item added notifications for cap awards. These messages stack up and appear all at once when the game exits.

Fixed with pull request #83


cannot install RPM package on fedora 27 using gui gnome Software

Dear Team ,
I couldn't install dbeaver 5 on Fedora 27, I have already previous version without any issue.

install it using cli :

and work fine

Looks like you need to uninstall previous version manually before installing new one.

@ahmed89albttashi I ran into the same issue.  I had a hard time uninstalling the old version as well.  This worked for me: rpm -ivh --replacepkgs --replacefiles /path/to/dbeaver-ce-5.0.1-stable.x86_64.rpm

Same here.. looks like the .rpm package for 5.x release is a "source" package, so despite the installation direct with rpm, it can't be installed trough a package manager :confused:

confirm bug,
dnf install ./dbeaver-ce-5.0.2-stable.x86_64.rpm
Ошибка: RPM пакет с исходным кодом не будет установлен (dbeaver-ce-5.0.2-stable.src).
Translate:
Error: Will not install a source rpm package (dbeaver-ce-5.0.2-stable.src).

$ sudo dnf install ./dbeaver-ce-5.1.0-stable.x86_64.rpm
Error: Will not install a source rpm package (dbeaver-ce-5.1.0-stable.src).

environment:
$ lsb_release -a
LSB Version:	:core-4.1-amd64:core-4.1-noarch:cxx-4.1-amd64:cxx-4.1-noarch:desktop-4.1-amd64:desktop-4.1-noarch:languages-4.1-amd64:languages-4.1-noarch:printing-4.1-amd64:printing-4.1-noarch
Distributor ID:	Fedora
Description:	Fedora release 28 (Twenty Eight)
Release:	28
Codename:	TwentyEight

on fedora 28 same mistake

The RPM built isn't valid. Will take a quick look to see if I can create a valid one.

This has to work for DBeaver 6.2.

thanks for resolve.
I forget about this issue because currently switched to flatpak package of dbeaver.
it's wrapped by flathub maintainers.
https://flathub.org/apps/details/io.dbeaver.DBeaverCommunity
I think it's will be better if it's prepared by this team directly for creditability(upstream) and distro-agnostic.


Add "logo" field to all descriptors

This is certainly not "highly important" but still nice to have (a lot of other systems support it):
Add a (optional!) logo field to NSDs and VNFDs that it of type string and points to a logo file that is shipped with the package.
The GUIs can then make use of it and display it in the list of on-boarded or instantiated services / VNFs, e.g., the logo of the VNF vendor.
The package descriptors will support it as well.
Example:
vendor: "eu.5gtango"
name: "example-package"
version: "0.1.1"
maintainer: "Manuel Peuster, Paderborn University"
release_data_time: "2017.01.01T10:00+03:00"  # IETF RFC3339
description: "This is an example 5GTANGO package."
logo: "icons/upb_logo.png"  # path to logo file (PNG or JPEG)

I just added the field to all schemas. But I didn't add any more actual logos to the descriptors yet.

No problem. Should be an optional field anyhow.


Zoom Features

In Zoom, we have Zoom in, out and extents.
Now what i am interested iterating through dataset and simultaneously zoom to the selected entity.
Any ideas how to zoom to selected entity.

You can use IFeatureLayer.ZoomToSelectedFeatures.

Wow..


so.owl PURL returns 404: Not Found

Hi there,
It looks like a recent commit moved the so-xp.owl file and consequently broke the so.owl PURL. http://purl.obolibrary.org/obo/so.owl resolves to https://raw.githubusercontent.com/The-Sequence-Ontology/SO-Ontologies/master/so-xp.owl
However it appears that so-xp.owl is now in master/releases/so-xp.owl/, i.e. https://raw.githubusercontent.com/The-Sequence-Ontology/SO-Ontologies/master/releases/so-xp.owl/so-xp.owl
Best,
Bill

As of our update yesterday we no longer use the -xp. convention in any SO files.  The file you are refering to (master/releases/so-xp.owl) is from our last release.

Thanks for your response. I think I could have written my initial question more clearly. I'm actually interested in using so.owl and have relied upon http://purl.obolibrary.org/obo/so.owl to point me to the appropriate download in the past. As of your update yesterday, http://purl.obolibrary.org/obo/so.owl resolves to a broken link to the so-xp.owl file. Would it be possible to update http://purl.obolibrary.org/obo/so.owl such that it points to the current release?

Unfortunately, I don't have the ability to create a redirect in github.  The best suggestion I can offer would be to update your link from so-xp.owl to so.owl with the same path.

As @bill-baumgartner points out, this change broke the main SO PURLs. (See also https://github.com/OBOFoundry/purl.obolibrary.org/issues/274 and the SO PURL configuration file.)
Do I have your permission to update the PURLs?

http://purl.obolibrary.org/obo/so.owl  ->  https://raw.githubusercontent.com/The-Sequence-Ontology/SO-Ontologies/master/so.owl
http://purl.obolibrary.org/obo/so.obo  ->  https://raw.githubusercontent.com/The-Sequence-Ontology/SO-Ontologies/master/so.obo

Yes. Thank you for updating this.

Let us know if this does not solve your issue.


addContentType and Enterprise Content Types

This does seem to be supported now. I did add a content type that was promoted from the content type hub sucessfully.

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

ID: 68267476-ee8d-7b95-446b-711ddb683da2
Version Independent ID: aa099b9d-48db-5513-11c7-32c49a27e5be
Content: Site design JSON schema
Content Source: docs/declarative-customization/site-design-json-schema.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Thanks @Aschlender ,  could you please describe your problem in detail?

@VesaJuvonen it's possible to use a site design to add a content type published by the hub to a new or existing list. When using a site design directly when creating a new sitecollection the site design goes stuck. If you wait until the content type is published from the hub on that sitecollection, the site design adds this content type without any problem to the list.


Formatting Custom Training Data & ValueError: bucket_boundaries must not be empty

Hello, ive been trying to make my own Training data, but there doesnt seem to be a ton of resources on how the data should be formatted. Ive compared the LJ001 Data and tried to imitate it, including splitting wavs, and the transcript.csv.
I have tested train.py with the LJ001 Data and the trainer works, but when i try with my Data it fails, giving me this error:
Traceback (most recent call last):
  File "train.py", line 96, in <module>
    g = Graph(); print("Training Graph loaded")
  File "train.py", line 33, in __init__
    self.x, self.y, self.z, self.fnames, self.num_batch = get_batch()
  File "C:\Users\...\tacotron-master\data_load.py", line 116, in get_batch
    dynamic_pad=True)
  File "C:\anaconda3\envs\...\training\bucket_ops.py", line 374, in bucket_by_sequence_length
    raise ValueError("bucket_boundaries must not be empty")
ValueError: bucket_boundaries must not be empty

Here is an example of the CSV File, i tried matching the ID, TEXT, LENGTH Format.
SM001-0001|Oh happy fourth of July America|00:00:02
SM001-0002|Ready to fire up the grill and celebrate our victory over the Brits|00:00:03
SM001-0003|Well, I'm not|00:00:01
SM001-0004|Because despite that incredibly convincing American accent, I'm one of those Brits|00:00:04
SM001-0005|now I've acted in film and TV for years|00:00:02
SM001-0006|but my greatest performance is acting like I don't care that every summer you gobble down tube sausages and celebrate kicking our arses|00:00:07
SM001-0007|Or butts as you say incorrectly|00:00:02
SM001-0008|Do you really still have to celebrate your emancipation from us|00:00:02
SM001-0009|I mean that's like your girlfriend breaking up with you and then celebrating with fireworks|00:00:04
SM001-0010|every year for 300 years|00:00:03
SM001-0011|it gets my goat|00:00:01
SM001-0012|but what really gets my goat is imagining how great America would be if we were still in charge|00:00:04
SM001-0013|Oh America if we'd won the war you'd have better comedy news TV programs and way better rude words|00:00:07
SM001-0014|Oh I'm talking fanny, trollop, minger tar, Minjbag, bleeding, sodding, blooming, cocked up, get stuffed|00:00:06
SM001-0015|and of course wanker|00:00:01
SM001-0016|imagine how sophisticated you'd say when you're insulting someone|00:00:03
SM001-0017|Oh Brad your wife's a slag don't piss off your wanker|00:00:04
SM001-0018|see how classy that sounded with our accents and your American self-confidence you'd be unstoppable|00:00:05
SM001-0019|yeah you'd have to pay a few more taxes but you can't put a price on that|00:00:03
SM001-0020|Great Britain two would be the greatest country on Earth|00:00:02
SM001-0021|your lawyers would all wear powdered wigs so criminals really respect them|00:00:04
SM001-0022|and you'd have all the mushy peas you can stuff down your bloody great gobs|00:00:03
SM001-0023|oh and if you get sick you don't need to worry about medical insurance because with a National Health Service a doctor will see you for free in about two years|00:00:08
SM001-0024|plus your taxes will be spent on things you really need like a royal family who do the tough jobs no one else wants to do|00:00:06
SM001-0025|like being driven around in a really nice car while waving|00:00:04
SM001-0026|you'all want to eat some apple pie then shoot some hoops and have hoedown|00:00:03

So tldr two questions:

Why am i receiving this bucket_boundaries must not be empty Error when python finds the CSV and can read it.
Based on 1's answer how can i properly format my data to work with the neural network

Hi @dbarroso1 ,
Did you eventually find how to format your data?
I’m at the same stage. I couldn’t figure how to properly do it but duplicating the transcript.csv file from the LJ dataset and carefully pasting in my own dataset, sentence by sentence, did the trick. Not a particularly sustainable or elegant solution…

I am also facing this bucket error I checked the maxlen(151) and minlength (149) that's why in for loop there is no iteration , so there is no value in bucket . If anyone solved this problem kindly help me in solving this issue

Hey consider using this GUI to make the datset


ifreerdp crash and found that the struct 'PCtxtHandle' was null when run in arm64 phone(ok in armv7)

the ifreerdp run normal in armv7,such as iphone 5c,iphone 5, but crash when run in arm64,such as iphone 5s, iphone 6p.
and I found the crash cause by pointer's address is null, now show the crash code:

I follow the func stack and the code,found the null pointer's address cause by the null struct 'PCtxtHandle'
Thanks for advance

@wangfuda although arm64 is out there for quite some time the FreeRDP iOS port still doesn't officially support it.  Please have a look to issue #2685

why the version of ifreeRDP(iOS) in offical web site (http://www.freerdp.com/) can run normal in arm64 such as iphone 5s, iphone 6p?


There is a reference error in the trigger event

When I am off event,  the events in the trigger function can't be synchronized, because the var events = this._events[name] statement in the trigger function is wrong, this._events[name] = retain = in the off function. The var events = this._events[name] in the [] statement and the trigger function are two different reference addresses.
The trigger function should be changed to:
trigger: function(name) {
      if (!this._events) return this;
      var args = slice.call(arguments, 1);
      if (!eventsApi(this, 'trigger', name, args)) return this;

      //var events = this._events[name];

      var allEvents = this._events.all;
      if (this._events[name]) triggerEvents(this._events[name], args);
      if (allEvents) triggerEvents(allEvents, arguments);
      return this;
    },

perhaps a codepen or jsfiddle reproducing the error would help

perhaps a codepen or jsfiddle reproducing the error would help

actually, i told about this issue very long ago.
i faced with it when i occasionally pass to the view something wrong instead of a collection.
everything in a view works fine until stopListening
but here is the link: https://codepen.io/dimatabu/pen/gyVVBz?editors=1010
const listener = _.extend({}, Backbone.Events);
const something = {};
listener.listenTo(something, 'some:event');
listener.stopListening(something);
console.log('no errors');

under the hood there is an off call:
  Events.stopListening = function(obj, name, callback) {
    var listeningTo = this._listeningTo;
    if (!listeningTo) return this;

    var ids = obj ? [obj._listenId] : _.keys(listeningTo);

    for (var i = 0; i < ids.length; i++) {
      var listening = listeningTo[ids[i]];

      // If listening doesn't exist, this object is not currently
      // listening to obj. Break out early.
      if (!listening) break;

      listening.obj.off(name, callback, this); // <-- at this place obj without events mixin will fail.
    }

    return this;
  };

i don't know which behavior should be
swallow, throw on events registration or on event stop listening
but, i believe something should happens
for me its good to throw on event registration if given object is not supports events.


V2.0 url refactor

Codecov Report

Merging #332 into v2.0 will decrease coverage by 0.08%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##             v2.0     #332      +/-   ##
==========================================
- Coverage   88.69%   88.61%   -0.09%
==========================================
  Files          31       31
  Lines        1318     1326       +8
==========================================
+ Hits         1169     1175       +6
- Misses        149      151       +2

Impacted Files
Coverage Δ

konfera/serializers.py
100% <ø> (ø)
:arrow_up:

konfera/urls.py
100% <100%> (ø)
:arrow_up:

konfera/tests/test_views.py
100% <100%> (ø)
:arrow_up:

konfera/models/ticket_type.py
96.05% <100%> (ø)
:arrow_up:

konfera/views.py
96.07% <77.77%> (-3.93%)
: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 5c715fd...ee0b3dc. Read the comment docs.


padarray() can't handle BitArrays

When trying to pad a BitArray the typeassert will throw an error.
ERROR: type: padarray: in typeassert, expected BitArray{2}, got Array{Bool,2}
 in padarray at U:\.julia\v0.3\Images\src\algorithms.jl:368 (repeats 2 times)

Padding Array{Bool} works.

Fixed, sorry for the delay.


Speed

Investigate ways to make the gem faster using stackprof. Performance has significantly fallen off as more features and checks have been added: https://github.com/diasks2/pragmatic_tokenizer/blob/master/spec/performance_spec.rb#L28-L46

One option: reusing strings and arrays wherever possible should reduce GC and add speed. E.g. there are quite a lot of calls to .map which each create a new array and GC the old one, that could probably be replaced by .map!.

With the most recent commits in master I'm seeing twice the speed. I'll close the issue for now, but will keep looking for further optimizations.

Thanks so much! Huge improvement, I really appreciate your hard work to make it faster and more memory efficient. :+1:


[opentelemetry-cpp] update version to v1.2.0

Bumping opentelemetry-cpp to latest version

Thanks for the update!


Unreachable code

In v3_0.sensors.py, the last return statement in the following block cannot be reached. Not sure which of the unknown_schedule messages is the preferred one. Do you know @victorgarcia98?
            # redirect to the fallback schedule endpoint if the fallback_job_id
            # is defined in the metadata of the original job
            if fallback_job_id is not None:
                return fallback_schedule_redirect(
                    message,
                    url_for(
                        "SensorAPI:get_schedule", uuid=fallback_job_id, id=sensor.id
                    ),
                )
            else:
                return unknown_schedule(message)

            return unknown_schedule(
                f"Scheduling job failed with {type(e).__name__}: {e}. {scheduler_info_msg}",
            )

@Flix6x which file needs to be updated?

https://github.com/FlexMeasures/flexmeasures/blob/78e20233903440c1ce3e831a79152689c116f915/flexmeasures/api/v3_0/sensors.py#L471-L485

Good observation!
The fact that we are not using scheduler_info_msg on the return unknown_schedule means that we are using the message from line 467 which doesn't include the scheduler_info.
The message on line 467 should be replace by
f"Scheduling job failed with {type(e).__name__}: {e}. {scheduler_info_msg}"

Also remove the unreachable return statement.

@Flix6x I am not able to understand @victorgarcia98 's point. Can you please tell me what exactly I need to do?

Sure!
My guess is that we lost the scheduler_info_msg somehow in the process of merging.
The message that we are returning in
https://github.com/FlexMeasures/flexmeasures/blob/78e20233903440c1ce3e831a79152689c116f915/flexmeasures/api/v3_0/sensors.py#L481
is created in https://github.com/FlexMeasures/flexmeasures/blob/78e20233903440c1ce3e831a79152689c116f915/flexmeasures/api/v3_0/sensors.py#L467
but the message should also contain the scheduler_info_msg which conveys status info from the scheduler.
To bring it back, I would add the scheduler_info_msg to the message after is being created.
https://github.com/FlexMeasures/flexmeasures/blob/78e20233903440c1ce3e831a79152689c116f915/flexmeasures/api/v3_0/sensors.py#L453
Feel free to ask any other question :smile:

@victorgarcia98 but the changes that you said are already there in the file, right? Then, what to change? I'm sorry if I am getting things wrong.

Hi @shubhalgupta, sorry for not responding earlier.
When I looked into the issue more closely I noticed that some other things had to be implemented so I took the initiative to develop them (https://github.com/FlexMeasures/flexmeasures/pull/887).
Thanks a lot for your interest and effort! :smile:


flax.errors.ScopeParamNotFoundError: No parameter named "kernel" exists in "/stage_1_output_conv_2".

Hello.
I try to run this rep on google colab and it works fine with Enhancement pre-trained model but when I want to load Deblurring model and use predict() to get output, this error appears...
MODEL_PATH = "Deblurring/GoPro/checkpoint.npz"
FLAGS = DummyFlags(ckpt_path = MODEL_PATH, task = "Enhancement")
params = get_params(FLAGS.ckpt_path)
model = build_model()

import requests
from io import BytesIO

url = 'https://replicate.com/api/models/google-research/maxim/files/6707a57f-4957-4047-b020-2160aed1d27a/1fromGOPR0950.png'
image_bytes = BytesIO(requests.get(url).content)

result = predict(image_bytes)

f, ax = plt.subplots(1,2, figsize = (35,20))

ax[0].imshow(np.array(Image.open(image_bytes)))
ax[1].imshow(result)

ax[0].set_title("Original Image")
ax[1].set_title("Enhanced Image")

plt.show()

UnfilteredStackTrace                      Traceback (most recent call last)
 in ()
8
----> 9 result = predict(image_bytes)
10
18 frames
UnfilteredStackTrace: flax.errors.ScopeParamNotFoundError: No parameter named "kernel" exists in "/stage_1_output_conv_2". (https://flax.readthedocs.io/en/latest/flax.errors.html#flax.errors.ScopeParamNotFoundError)
The stack trace below excludes JAX-internal frames.
The preceding is the original exception that occurred, unmodified.

The above exception was the direct cause of the following exception:
ScopeParamNotFoundError                   Traceback (most recent call last)
/usr/local/lib/python3.7/dist-packages/flax/linen/linear.py in call(self, inputs)
356
357     kernel = self.param('kernel', self.kernel_init, kernel_shape,
--> 358                         self.param_dtype)
359     kernel = jnp.asarray(kernel, self.dtype)
360
ScopeParamNotFoundError: No parameter named "kernel" exists in "/stage_1_output_conv_2". (https://flax.readthedocs.io/en/latest/flax.errors.html#flax.errors.ScopeParamNotFoundError)

what's the problem? is it with pre-trained models? if yes, how can I fix it or make my own model?

Hi please revise the following part:
model = build_model(task="Deblurring")

because the default model if for Dehazing:
def build_model(task = "Dehazing"):
  model_mod = importlib.import_module(f'maxim.models.{_MODEL_FILENAME}')
  model_configs = ml_collections.ConfigDict(_MODEL_CONFIGS)

  model_configs.variant = _MODEL_VARIANT_DICT[task]

  model = model_mod.Model(**model_configs)
  return model

thanks a lot. <3


If TLS is Configured, Redirect All HTTP Requests to Their HTTPS Equivalents

Relates to #84: if configured with 80/443 under systemd.

Mostly fixed.


setup issues

your setup instructions don't work- would you mind updating or create a google collab script others can follow to get this running.

hello,do you solve this problem?


Connection to Azure Analysis Service

Hi,
We have reports which are connecting to azure analysis service.
Can we deploy them here? I cannot see any option for connection string for azure analysis service.
Regards,
Vishal

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

ID: da59f209-0ccd-01d0-6e0e-12943ef3680e
Version Independent ID: 390690ff-76bd-f4c6-afac-d5ad2f3854cb
Content: Tutorial: Create a paginated report and upload it to the Power BI service - Power BI
Content Source: powerbi-docs/paginated-reports-quickstart-aw.md
Service: powerbi
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @vishalaj1. We’ve forwarded your feedback to the writer, who should respond to this thread shortly. In the meantime, you could ask your question in the community: https://community.powerbi.com/.

Hi, @vishalaj1 -- currently paginated reports in the Power BI service support these data sources:
Azure SQL Database and Data Warehouse
SQL Server
SQL Server Analysis Services
See this article for details: https://docs.microsoft.com/en-us/power-bi/paginated-reports-embedded-data-source.
More data sources will be coming.
Thanks for asking!

Hi @maggiesMSFT -- I have also tried to upload a report using Azure Analysis Services and am having issues as per the image below.

I sent a quick tweet to Bhavik Merchant asking him because in his talk at Ignite he specifically mentions that Azure Analysis Services is supported (although his slide states SQL Server Analysis Services, but he speaks as if AAS is included within this).
If AAS is not specifically supported right now it may be worth mentioning in the article to avoid confusion.

Hi, @vishalaj1 and @jdfancke -- Support for Azure Analysis Services is coming, but isn't here yet.
thanks for asking!

Do we have some approximate time line? We have moved to azure and this is hurting us since we cannot connect the on prem report server to azure analysis services neither can we deploy the ssrs report on service to connect azure analysis service.


async requests without webhook_endpoint dependency

Is your feature request related to a problem? Please describe.
Based on the async request documentation it seems mandatory to have a webhook endpoint, even though it is just a mock example, in the json data passed as payload. Although, for use cases where the output is being stored in a S3 bucket the webhook dependency is not required at all.
Describe the solution you'd like
The idea is to avoid the mandatory usage of a webhook endpoint when using the async endpoint.
Describe alternatives you've considered
While requesting the async endpoint, the parameter webhook_endpoint becomes optional. When an user set a value for such parameter, it will post the response on the given endpoint/url, otherwise it will consider that the upload process (such as to a S3 bucket) has been already implemented on the model.py procedure.

Hey @wnakano, thanks for trying out async! The webhook_endpoint is actually an optional parameter--see the Quick start with saving model outputs section and the API reference.
Could you try calling /async_predict without a webhook_endpoint and let us know if you run into any issues? We'd also love to get your feedback on how we can make this more clear in the docs.

We also appreciate you taking the time to open a feature request!

@wnakano let us know if you have issues with that approach!


Any plans for providing instrumentation via open telemetry?

Feature Request
As the title indicates, I'm wondering if there're any plans to integrate provide instrumentation via open telemetry (Otel)
Pitch

Otel provides a lean SDK with the support of multiple exporters. Neo4j can provide the auto instrumentation once and let end users choose which tracing backend. For example, integration with datadog would be pushed as a concern to Otel implementation.
For end users, it allows better performance observability

Hi @MrwanBaghdad and thanks for reaching out.
We have already been internally discussing if and how telemetry support in the drivers could be added (internal Trello link). As it stands right now, this topic is not on the roadmap but that might change in the future. I will close this issue as we are aware of the feature request and are tracking it internally. If you have further questions, please feel free to keep commenting. I will keep you updated here as the feature progresses.
Other users that come across this and are also interested in this feature please leave a :+1: on the issue so we can see how big the demand is.

I can't access the internal Trello link you provided, is there any other medium that I can follow up on the discussion? Thanks

Sorry about that. I mainly put the link there as a cross reference for our team. The Trello wall is considered internal information. So far the discussion didn't go much further then investigating different frameworks/tools for adding telemetry support. If you have input, feel free to leave it here and I'll make sure to take it to the team for consideration.


Adding the NAT meetup and squishing python-js-ruby

What issue is this referencing?

Do these code changes work locally and have you tested that they fix the issue yourself?

[ ] yes!

Does the following command run without warnings or errors?

[ ] yes! npm run pr-checks

Have you taken a look at our contributing guidelines?

[ ] yes!

My node version matches the one suggested when running nvm use?

[ ] yes!

@MandyMeindersma I'm not sure why I didn't receive notifications when you pushed new changes. I'll approve the PR tomorrow morning 🙂.

@MandyMeindersma: Approved the PR.


dead_lock when exit my process

after fork ，then create multithread， when exit(by catching signal)， it cause deadlock

BTW posting screen shots is a bit unnice. Not sure why you're doing this.
I don't see anything about tcmalloc data that your provided. Can you please add more details so that it is clear how tcmalloc affects this?


Search API search param criteria for names strict without case sensitive

Hi,
I have to seach for some PractitionerRole with exact values for family and given name, but not case sensitive. How to do this ?
If I put :exact criteria, it is case sensitive:
family:exact=ALBINET&given:exact=JEAN return nothing
And removing "exact" from given, I have too much many people: family:exact=ALBINET&given=JEAN :
{ ...
  "family": "ALBINET",
  "given": [
    "Jean"
  ]
},
{ ...
  "family": "ALBINET",
  "given": [
    "JEAN-LUC"
  ]
}

Any way to get only "Jean" result even if I search for "JEAN" ?
Thank you for your help,

Thank you @nriss for all this details; If needed, yes I will go throw sorting result and add a post-processing filtering.


Bump psalm to ^4

Summary
First I thought I'm just gonna bump the major version and see.
But then I also realized that it was able to fix things, so I tried that and ran:
vendor/bin/psalm --alter --issues=MissingReturnType,InvalidReturnType,InvalidNullableReturnType,MissingParamType
The automatic changes applied look sane to me
Type of change

[x] Misc. change (internal, infrastructure, maintenance, etc.)

Checklist
n/a

@barryvdh ping :)

@barryvdh ping :)

Surpassed by a different PR


put renderer info into app info

currently, the vulkan renderer is logging the available devices. this should be at the top, were the other device info is logged. refactor the physical device enumeration out and put the info into the app info struct.

its probably for the better if app info is logged after god job creation.
yes, that means there will be logs above the app info, but honestly, i think that is fine. only package name and author should be displayed at the top, to signal what kind of log this is.
come to think of it, i don't think implementing std::fmt::debug or std::fmt::display are necessary. just use dedicated logging, considering that this will be logged only a single time.

scrap what i said above:
print info the moment it is gathered. the issue with gathering everything in one object and then printing it comes with the issue, that when info gathering fails, no info is being printed. by printing upon construction, this is circumvented.


Add the exec options to the README

Fix #50

@jharding, could you merge this? :-)

@jharding please merge this! It takes me long time to figure out how to encrease maxBuffer!
I think it also resolves issue #23 and #64.

Can you please merge this? Also struggled to find docs on this

:+1: - would be good to get this merged. I came across the same sort of problem and tweaking the maxBuffer option was the solution

Thankfully google directed me here. Merging this would be very helpful.

Added to README with latest Node documentation


Allow tensorboard to follow symlinks (feature request #2140)

Fixes #2140

Can one of the admins verify this patch?

I signed it!

@vrv Sounds good, done!

Change looks good to me.

@danmane @vrv Done

great, thanks!
@tensorflow-jenkins test this please

@vrv: Do you think the failing rnn test could be related to this change?

No it shouldn't be, I'm going to merge in some newer changes from internal first, and see if the flakiness goes away.

@tensorflow-jenkins test this please

If we discover that the symlink loops are common, which I hope they are not, we can change the function to use the os.walk iterator and check whether a symlink causes a loop.

Any idea on when this will be merged ? I suppose it belongs to the tensorboard repo now, right ?


NSX Edges "failed to open a channel after 10 attemps"

Hi team ,
May be this question has been asked before here : https://github.com/darkoperator/Posh-SSH/issues/119  .. but i am facing same issue and cant find solution so please help .
I am using posh-SSH to execute command on NSX Edge (NSx-v 6.4.5 Edge) .  Below is code
Posh-SSH\New-SSHSession -ComputerName <IP_ADDRESS>

$edge=Invoke-SshCommand -SessionId 2 -Command "show system storage"

I have also tried to execute this command to create a shell stream   but same error
$session = New-SSHShellStream -Index 2
$stream = $session.Session.CreateShellStream("admin", 0, 0, 0, 0, 1000)
$stream.Write("show system storage`n")
$stream.Read()

I am using PowerShell 5.1 and posh-SSH version 2.3.0 ..Please help me on solving this .

Sadly it is a known issue with that type of device, since I don’t have access to one I have not had a chance to look deeper as to the why.

On Jul 29, 2021, at 3:22 AM, guptatarun1989 @.***> wrote:
Hi team ,
May be this question has been asked before here : #119 https://github.com/darkoperator/Posh-SSH/issues/119 .. but i am facing same issue and cant find solution so please help .
I am using posh-SSH to execute command on NSX Edge (NSx-v 6.4.5 Edge) . Below is code
Posh-SSH\New-SSHSession -ComputerName <IP_ADDRESS>
https://user-images.githubusercontent.com/62771038/127448159-aaa265db-cf13-4c17-9a0e-2bf4dd4125a0.png
$edge=Invoke-SshCommand -SessionId 2 -Command "show system storage"
https://user-images.githubusercontent.com/62771038/127448593-8309e301-1a27-4d69-a481-297c98c0e925.png
I have also tried to execute this command to create a shell stream but same error
$session = New-SSHShellStream -Index 2
$stream = $session.Session.CreateShellStream("admin", 0, 0, 0, 0, 1000)
$stream.Write("show system storage`n")
$stream.Read()
https://user-images.githubusercontent.com/62771038/127449162-73b126b4-67ca-42ec-b2d7-ca864e9d389b.png
I am using PowerShell 5.1 and posh-SSH version 2.3.0 ..Please help me on solving this .
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/411, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD7IHS7QJXDYZ7ICZCHO3LT2D6UFANCNFSM5BF3QYOA.


Version 0.9

Merging everything committed to the development branch since 0.8.2

Relevant to issue #4


打包Android时遇到错误

What happened?
Execution failed for task ':realm:verifyReleaseResources'.

A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
Android resource linking failed
ERROR: E:\business\harvesting\build\realm\intermediates\merged_res\release\values\values.xml:194: AAPT: error: resource android:attr/lStar not found.

Repro steps
Execution failed for task ':realm:verifyReleaseResources'.

A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
Android resource linking failed
ERROR: E:\business\harvesting\build\realm\intermediates\merged_res\release\values\values.xml:194: AAPT: error: resource android:attr/lStar not found.

Version
3.4.1
What Atlas Services are you using?
Atlas Device Sync
What type of application is this?
Flutter Application
Client OS and version
Windows build Android APk
Code snippets
No response
Stacktrace of the exception/crash you're getting
No response
Relevant log output
No response

See https://github.com/realm/realm-dart/pull/1797 for reference, as this pr is merged but newer version is not published yet.
And you should probably use English in your issue if the repository is mainly maintained by non-Chinese people and you want your question answered.

Flutter 3.27.0 solves this problem


is it time to fork this? can anyone contact Sam Washington?

This is still a useful library, despite its age. There are pending PRs. @sam-washington doesn't have any activity in github, they have a keybase entry for github and aethris.net, which is a dead site and doesn't exist in the Wayback Machine. The domain is still registered, though.
Python has a process for doing all of this:

https://www.python.org/dev/peps/pep-0541/#continue-maintenance
https://www.python.org/dev/peps/pep-0541/#how-to-request-a-name-transfer
there are no requests to transfer in pypi-support
https://pypi.org/help/#project-name-claim

As soon as I post this I'll send the issue link to<EMAIL_ADDRESS>which is the only email address I've found.

Opened a "takeover request", as shown in the mention link.

Been in contact with Sam, we got things going. He and I both own the pypi lib, the github repo was transferred to my account (and he's a collaborator on it).
This is a great time for other interested parties to step up! That way I can increase the bus factor. I'll try to revamp the readme to indicate this.


Control YARN queue that actions are run against

Actions in the LEFT and RIGHT clusters that require compute resources should have an option to specify a queue name.

The way to do this is at the jdbc URL level.  IE: hive2://host:10000/.....;...?tez.queue.name=batch .
Will add this to docs.  Will not do this in code.


Assets?

Usage: chapter1 [path to glTF model]\n
Assets? or ???

https://github.com/PacktPublishing/Mastering-Graphics-Programming-with-Vulkan/pull/2

Thank you Sergey.
From: Sergey Kosarevsky @.>
Sent: Friday, February 10, 2023 4:36 PM
To: PacktPublishing/Mastering-Graphics-Programming-with-Vulkan @.>
Cc: gubser dynacee.com @.>; Author @.>
Subject: Re: [PacktPublishing/Mastering-Graphics-Programming-with-Vulkan] Assets? (Issue #5)
#2https://github.com/PacktPublishing/Mastering-Graphics-Programming-with-Vulkan/pull/2
—
Reply to this email directly, view it on GitHubhttps://github.com/PacktPublishing/Mastering-Graphics-Programming-with-Vulkan/issues/5#issuecomment-1426533483, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAD46UXTBQNJJLRIFXAI6WDWW3NH5ANCNFSM6AAAAAAUYKEVEA.
You are receiving this because you authored the thread.Message ID: @.***>

We have merged the #2 to make it easier to download the models and run the code :)