Make the is_picklable function more robust

What does this PR do?

Fixes #17269

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Sure, I will do that :)

Seems like https://lightning.ai/docs/pytorch/latest/governance.html is no longer available, should I update the branch?

This issue was resolved and we should be able to merge this shortly.


Updates weather symbol logic for sleet

Replaces rain/snow in vicinity with precip-in-vicinity
Adds use of sleet probability instead of P(Rain) AND P(Snow)
Testing:

[ ] Ran tests and they passed OK
[ ] Added new tests for the new feature(s)

 Here is an overview of what got changed by this pull request:

Clones added
============
- improver/wxcode/wxcode_decision_tree.py  8

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones added
============
- improver/wxcode/wxcode_decision_tree.py  8

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones added
============
- improver_tests/acceptance/test_wxcode.py  2
- improver/wxcode/wxcode_decision_tree.py  9

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones added
============
- improver_tests/acceptance/test_wxcode.py  2
- improver/wxcode/wxcode_decision_tree.py  9

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones added
============
- improver/wxcode/wxcode_decision_tree.py  9

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/wxcode/wxcode/test_WeatherSymbols.py  1

Clones removed
==============
+ improver_tests/wxcode/wxcode/test_WeatherSymbols.py  -2

See the complete overview on Codacy


When you create your Azure Application for some reason it is not creating an SPN (enterprise app)

Thanks for working through this with me. Fixed the missing permissions that were causing the AAD auth to fail and have everything working without the service principal. Fixes are released in v0.1.26

Thanks for working through this with me. Fixed the missing permissions that were causing the AAD auth to fail and have everything working without the service principal. Fixes are released in v0.1.26


Validation errors don't get passed for resource_type

Typically when one creates a record with an incorrect field a validation error with helpful message is returned:
e.g. 'titles': [{'title': 'A Romans story', 'type': 'Otherss', 'lang': 'eng'}] returns an errors field that includes Invalid title type. Otherss not one of MainTitle, AlternativeTitle, Subtitle, TranslatedTitle, Other
However when one provides an incorrect resource_type in either the type or subtype field no errors field is returned:
e.g.  'resource_type': {'type': 'imagess', 'subtype': 'photo'} only returns {"status": 400, "message": "Validation error."}

@tmorrell To double-check: is the main issue the helpful list or a missing message in 'errors' ?

Main issue is not having a helpful list. It’s hard to figure out what the accepted values are (in the schema, marshmallow, somewhere else?)
On Mar 23, 2020, at 2:07 PM, fenekku<EMAIL_ADDRESS>wrote:

@tmorrellhttps://github.com/tmorrell To double-check: is the main issue the helpful list or a missing message in 'errors' ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/inveniosoftware/invenio-rdm-records/issues/76#issuecomment-602856347, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACSMOGT6GCT3HXLT3CIGC4LRI7FPTANCNFSM4LF4YQ5A.

Got it. Thanks!


Null check in AND operator. If left is false both sides of AND are executed.

Null check in AND operator. If left is false both sides of AND are executed.
Where x is and empty List of strings.
var x = new List();
"{{ if x && x.size > 0 }}{{ end }}";
Results into:  ScriptRuntimeException, ": error : Object x is null. Cannot access member: orderitems.size".

if (!rightBoolValue) return false; // and expression evaluation should be stopped

Thanks for the report. This is a pretty bad issue, I don't know what I was thinking when writing these lines, this is so basic... :(
I'm on vacation but I will have a look later next week

Fixed by commit 5c748b7  will be available in 2.1.1+


[feature request] Text Input

Something like this or Unity's Input Field. I think this is a basic feature that should really be available without having to pay for a 3rd party plugin.

Duplicate of https://github.com/playcanvas/engine/issues/2483


Feat: 👨💻 I can Improve Community Spotlight Section

Describe the feature
The current Community Spotlight section on SwapReads is built with basic div elements.
I can the structure and style of this section to make it more interactive, visually appealing.
Please add the following labels:
• gssoc-ext
• hacktoberfest-accepted
• level 2
I have experience with UI/UX and front-end development. Please assign this issue to me.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Please add level-2 label to it.


[move experiments] Regulated Coin

This PR introduces two things:
RegulatedCoin module
A module that provides a common implementable interface for handling regulated balances. Custom implementation that needs to instantiate a RegulatedCoin can implement any number of methods from RegulatedCoin. Main methods are:
public fun borrow<T: drop>(_: T, coin: &RegulatedCoin<T>): &Balance<T>;
public fun borrow_mut<T: drop>(_: T, coin: &mut RegulatedCoin<T>): &mut Balance<T>;
public fun zero<T: drop>(_: T, owner: address, ctx: &mut TxContext): RegulatedCoin<T>;

public fun from_balance<T: drop>(_: T, balance: Balance<T>, owner: address, ctx: &mut TxContext): RegulatedCoin<T>;
public fun into_balance<T: drop>(_: T, coin: RegulatedCoin<T>): Balance<T>;

An example: ABC coin
This implementation aims to showcase one of the potential uses for the RegulatedCoin. The idea behind the implementation is to mimic real-world bank functionality. In detail:

Empty balances can only be created by admin (eg user goes to the bank and passes a KYC; a new balance is created by an admin);
Minting is also handled by admin (say, I transfer some amount of fiat currency to an organization, and they mint this coin in Sui and transfer it to my address);
Transfers between RC accounts are regulated - they are protected by a ban list and can only be received by the specified recipient;
Stolen instances of RegulatedCoin are "blocked" - only an owner can access his RC balance; in case he is banned, he cannot cheat by transferring his balance to someone else.

To make RC interoperable with the rest of the Sui-verse, there's an option to "cash out" a currency from a regulated account; the "cashed out" amount is wrapped as Coin and can be freely used by any user. An opposite action is putting a "cashed out" Coin back into the regulated account; can only be performed with accounts that haven't been banned.
The reasoning behind having two states of the same Balance are:

Minting and Burning can only be performed with regulated coins (if there's some interaction with the real world).
Transferrable Coin is mimicking "cash" - it can be withdrawn or put into a bank account but cannot be used to interact with the outside world.

@sblackshear @lxfind I've updated PR description, so you see the logic right away.
Also, added an option to convert RegulatedCoin<ABC> to Coin<ABC>. Now it is possible for any RC owner to take and transfer it as a Coin.

I think this is still a draft? Or is it ready for review?
I still don't quite see the point of the onwner field of the RegulatedCoin.

My understanding is that the owner field of the RegulatedCoin provides constraint in the following scenario:
I have a RegulatedCoin, and I have just submitted a transfer request of this coin to account X (at this point X is not banned yet). After I submitted this transfer request, before X had a chance to accept the transfer, suddenly X gets banned, and we want to make sure we catch this case and forbid X from being able to accept this transfer.
Does the above sound right? This seems a very niche case to me that I am not sure why we want to support. Every other case can be solved by just filtering based on the owner of the coin object stored on-chain.


Nimble does not persist CCCD value that was set before bonding

Here is a failing scenario:

Nimble is a peripheral
A connection is established with a peer device
Peer device subscribes for notifications/indications on a CCCD
Bonding with a peer device
Peer device disconnects and reconnects
The indication setting is not restored

Fixed by #324


Extending meteor accounts but can't display form

This is probably something very simple, but I've been trying for a long time without success.
I have created my own login service based on the excellent tutorial here https://meteorhacks.com/extending-meteor-accounts.html
From client side
Meteor.loginWithMysql = function(user, password, callback) {
	var loginRequest = {user: user, password: password};
	Accounts.callLoginMethod({
		methodArguments: [loginRequest],
		userCallback: callback
	});
};

I have then implemented the login (which is based on external accounts stored in mysql) on the server side.  This has been working perfectly in a different app in production for over 6 months.
I now want to create a new app based on the same accounts using the ionic framework.  If I add accounts-password I get the nicely laid out signup form and login form immediately.  However, I am at loss of what is the correct way to show the email and password fields for my mysql login service.  As soon as I remove accounts-password all I see is the "If you already have an account Sign in" - or "Don't have an account register" messages - but no form and no fields.
Here is my login template
<template name="login">

     {{#ionView}}
       {{#ionContent}}
         {{#if currentUser}}
           <div class="padding">
             <button class="button button-positive button-block" data-action="logout">Log Out</button>
           </div>
         {{else}}
           {{> atForm}}
         {{/if}}
       {{/ionContent}}
     {{/ionView}}
</template>

My login service works perfectly - I can login easily from the console using Meteor.login(...
I have tried too many things to list them all here - even copying the accounts-password module code (which did not work).
I must be missing something trivial here - if anybody can point me in the right direction on how to show the login form I would be grateful - this would also be a good addition to the user guide, that is how to display the login fields for custom external user services that are not oauth based.

Answering my own question here.  After some good morning coffee I finally figured this out.  So I'm changing this into a suggested feature.
If you look here:
https://github.com/meteor-useraccounts/core/blob/7625af8ec508e7ebeb55424388f197487ae88574/lib/templates_helpers/at_form.js#L39
Then showPwdForm is checking for the package accounts-password.  I managed to work around this by adding this in my lib folder
Package["accounts-password"] = {}

This shows the login form - I then had to create this function that refers to my other function
Meteor.loginWithPassword = function(user, password, callback) {
  Meteor.loginWithMysql(user,password,callback);
}

and now login works.
Would there be an interest in adding the option for custom login services like this on to the package.  I think in this case one would need to specify the name of the login method (in this case loginWithMysql) in configuration and also line 39 in at_form.js checking for accounts-password should look something like this based on an added configuration option called PasswordLoginImplemented (better naming anyone).
if (Package["accounts-password"] === undefined || AccountsTemplates.options.PasswordLoginImplemented)

There are many use cases for this, in our case we already have close to 100K users in our database and would not like to copy it all to mongo.

Hi @maronk.
I'm sorry for the late reply.
It seems you got it right, well done!
I appreciate there might be some interest in having custom login services integrated with useraccounts, and I think it would be a good feature to have!
My concerns are about the other calls like resetPassword, changePassword and the like...
Talking about<EMAIL_ADDRESS>there will be a package called useraccounts:password to bring in all code and packages related to password authentication...
What about thinking about designin another package for custom login services?

Actually - hadn't thought about that but a generic framework around custom user accounts is a great idea.
I would assume a common interface would be to base it off accounts-password - I'll see if I can find time in the next couple of weeks to make my own code more generic and put it up on github

Custom login services, based off the password forms, will be implemented as plugin packages in V2.0.
The solution would be to have an UserAccounts.password.login method, normally pointing to Meteor.loginWithPassword, to be overwritten by pointing to custom services calls

See the integration-with-custom-login-services board on trello


[SPARK-6897][Streaming]remove volatile from BlockingGenerator.currentBuffer to reduce unnecessary overhead

currentBuffer has been protected by synchronized, so it would introduce unnecessary overhead for adding an extra volatile modifier here.
Hi @srowen , Found this while walking through the source code, could you pls review if we need to take this since it's only a trivial stuff ?

Test build #30238 has started for   PR 5508 at commit 40f0215.

Test build #30238 has finished for   PR 5508 at commit 40f0215.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30238/
Test FAILed.

Test build #30239 has started for   PR 5508 at commit 7052a7a.

These are slightly different ideas. I agree in practice here that there is no visibility problem that volatile would solve that isn't already taken care of by the memory barrier from synchronized. However I also don't imagine this would cause any measurable overhead from the extra read. Have you measured some improvement? If not, I am not sure this is worth changing since the moment this field becomes read outside synchronized blocks it's technically unsafe again.

Test build #30239 has finished for   PR 5508 at commit 7052a7a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30239/
Test PASSed.

These are slightly different ideas. I agree in practice here that there is no visibility problem that volatile would solve that isn't already taken care of by the memory barrier from synchronized. However I also don't imagine this would cause any measurable overhead from the extra read. Have you measured some improvement? If not, I am not sure this is worth changing since the moment this field becomes read outside synchronized blocks it's technically unsafe again.

Just thinking of it would be a good practice to avoid the usage of volatile and  synchronized together
Each small batch currentBuffer would be r/w and it's private too, so even if it's only a tiny overhead it would be much if they are sum up together(haven't done benchmark regarding this though).
This field is a mutable array, so even if it's modified by volatile, I guess technically it's not safe to use it in the other place without locking.
I'm fine to stay the same,  given that we always set the batch interval to be no less then 500ms and also there would be no measurable improvement compared with that.

oops... Maybe it's not always true that avoid the use of volatile and synchronized like double check singleton(although it's a must in that case). Anyway, just post a PR here to see what's the opinion of your guys :)

volatile may still matter if this were read outside a synchronized method here. Although I doubt that would happen, I suppose my instinct is to leave it if there's likely no benefit.

@srowen , Thanks for the comments. I'm closing it :) .


OrientationError('Cannot drop coordinates when ' nibabel.orientations.OrientationError

Hi, I am running fMRIPrep 1.5.8. I got this orientation error for only one subject. I checked the T1 image looks fine.
In the output report, the error is,
Node Name: fmriprep_wf.single_subject_C01071_wf.anat_preproc_wf.anat_template_wf.t1w_ref_dimensions
In the log, it shows,
fMRIPrep failed: Traceback (most recent call last):
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
result["result"] = node.run(updatehash=updatehash)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
result = self._run_interface(execute=True)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
return self._run_command(execute)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
result = self._interface.run(cwd=outdir)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 397, in run
runtime = self._run_interface(runtime)
File "/usr/local/miniconda/lib/python3.7/site-packages/niworkflows/interfaces/images.py", line 188, in _run_interface
reoriented = np.vectorize(nb.as_closest_canonical)(orig_imgs)
File "/usr/local/miniconda/lib/python3.7/site-packages/numpy/lib/function_base.py", line 1972, in call
return self._vectorize_call(func=func, args=vargs)
File "/usr/local/miniconda/lib/python3.7/site-packages/numpy/lib/function_base.py", line 2048, in _vectorize_call
outputs = ufunc(*inputs)
File "/usr/local/miniconda/lib/python3.7/site-packages/nibabel/funcs.py", line 209, in as_closest_canonical
img = img.as_reoriented(io_orientation(img.affine))
File "/usr/local/miniconda/lib/python3.7/site-packages/nibabel/nifti1.py", line 2011, in as_reoriented
img = super(Nifti1Pair, self).as_reoriented(ornt)
File "/usr/local/miniconda/lib/python3.7/site-packages/nibabel/spatialimages.py", line 634, in as_reoriented
t_arr = apply_orientation(np.asanyarray(self.dataobj), ornt)
File "/usr/local/miniconda/lib/python3.7/site-packages/nibabel/orientations.py", line 163, in apply_orientation
raise OrientationError('Cannot drop coordinates when '
nibabel.orientations.OrientationError: Cannot drop coordinates when applying orientation to data

Can you provide the header? head -c 348 img.nii > img.hdr (May need to gunzip it...)

Thanks for your reply. The header is attached.
img.hdr.gz

Do you have multiple T1w images? This one looks fine.

Thanks for checking. Only one T1w is in anat.

Okay. I'm not able to reproduce the issue. Can you try re-running fMRIPrep? Removing the fmriprep_wf/single_subject_C01071_wf/anat_preproc_wf folder from your working directory should ensure that the anatomical part is started from scratch.

I found out it did have two T1. This is a longitudinal dataset that all subjects have single T1 but only this subject has another T1 in one of the sessions. Sorry for missing that. Really appreciate your help!


Tag grouping

We may want multiple tags to share the same color on the scoreboard and be grouped in other ways. May want to implement this as part of https://github.com/PazerOP/tf2_bot_detector/issues/29.

Added to project Enhancements


Allow to set default audio and subtitles

It'd be nice if there was a way to set the default audio/subtitles in the mp4 to a desired language...

from the docs:
audio-codec = set your desired audio codecs. May specify multiple comma separated values (ex: ac3, aac). The first value specified will be the default conversion choice when an undesired codec is encountered; any codecs specified here will be remuxed/copied rather than converted.

This is already done

This is not what I meant, but yes, I noticed that audio-default-language = por would make the por audio have the default flag in the converted files.
It'd be nice to add that to the documentation tho


Web refactor

This PR splits the ShapeViewer component and the store into multiple files

This throws a typescript warning in my local deployment.
ERROR in /app/server/src/components/InfoTab.vue(3,24):
3:24 Could not find a declaration file for module './ShapeViewer/methods'. '/app/server/src/components/ShapeViewer/methods.js' implicitly has an 'any' type.
    1 | <script lang="ts">
    2 | import pointsReader from '../reader/points';
Version: typescript 4.1.5
Time: 667ms
  > 3 | import { COLORS } from './ShapeViewer/methods'
      |                        ^
    4 | import { defineComponent, onMounted, ref } from '@vue/composition-api';
    5 | import { landmarkColorList, selectedDataset, selectedProject } from '@/store';
    6 |


Release 0.17.0

#146
Remove Result dependency by daltonclaybrook · Pull Request #146 · Carthage/Commandant
Bumped up version to 0.17.0

#151 should be merged before this

Thanks!

@mdiep I can create GitHub release. But I don't have permission to update pod trunk.
Could you release newer version?

I don't have CocoaPods permissions either. I think @ikesyo will need to do that.

Created 0.17.0 release (since CocoaPods needs the tag in order to publish).

0.17.0 has been pushed to CocoaPods trunk now.
@giginet I've added you as an owner using the email showing on your GitHub account. @mdiep your GitHub email isn't showing up as being registered with an account on trunk.
$ pod trunk add-owner Commandant<EMAIL_ADDRESS>[!] No owner found with the specified email address.


imagenet_models cannot download

The fetch_imagenet_model.sh and two other .sh cannot link to the right checksum.So i cannot use the pretrained resnet models,may be because i am in China.someone can help me?thank you very much!!

I think that's because the current download links are obsolete. I had opened the PR back then for the same issue. You can check the download w/ those download links. Thanks.
Hope it helps!

I try the current links again,but still can not download.Could you please give us  a direct links of files to download.Thank you!

Why don't you take a look at the changes made in this pull? That one has all the modified download links, I don't have any other links to provide, to be honest.

@onkarganjewar  I have tryed all,but i really failed....
/home/omnisky/Pictures/Screenshot from 2017-11-02 [17-58-58.png

@onkarganjewar I have tryed all,but i really failed....

@onkarganjewar I have tryed all,but i really failed....This is one of the error.

@onkarganjewar I have tryed all,but i really failed....This  is one of the error.

@onkarganjewar I had got it,thank you!

@whmin hello, could you give me the link to down imagenet models? thanks in advance....


Add flush logic based on level

Imporant: please check branch name. Is it correct?
Changes:

Add two new properties into the BufferingTargetWrapper.
Add tests for these properties

If this change is ok to merge I will do the same for AsyncWrapper
Questions:

How it is better to update help? Can I just click on Edit and copy documentation from the BufferingTargetWrapper (with examples)?

Why not just do this:
var bufferingTargetWrapper = new BufferingTargetWrapper(testTarget, 100);
var autoFlushOnLevelWrapper = new AutoFlushTargetWrapper(bufferingTargetWrapper);
autoFlushOnLevelWrapper.Condition = "level > LogLevel.Info";

@snakefoot , primary feature: have ability of blocking Flush for fatal level. As I said in comments this is really imporart of case when your applications have Environment.FailFast calls after Fatal error write.
However thank you about AutoFlushTargetWrapper it is reasonable for several other scenarios.

Also mentioned the AutoFlushTargetWrapper as answer to your original question: https://github.com/NLog/NLog/issues/3145#issuecomment-468375179

@snakefoot, you are right about mention AutoFlushTargetWrapper.
So can we have blocking flush? Of course I can move my code to AutoFlushTargetWrapper (if it is more logical place).

Of course I can move my code to AutoFlushTargetWrapper (if it is more logical place).

That sounds as a good decision.

Of course I can move my code to AutoFlushTargetWrapper (if it is more logical place).

That sounds as a good decision.

Please do and please create a new PR then, thanks!


add man trans and clean title option to cars

Added additional filters to use when dealing with cars. manual_transmission and clean_title to be set to 1 as appropriate.

I hope this gets merged. Great addition! For now I'll just copy the commit code to my project.

This is already covered with the list_filters added a while back. I've updated the documentation with a example for cars/trucks.
An example of usage would be:
from craigslist import CraigslistForSale
CraigslistForSale(site='sfbay', category='cta', filters={
    'auto_transmission': ['manual'],
    'auto_title_status': ['clean']
})
# ...


Created main test wich includes the whole pipeline with different typ…

…es of data

Hello @Yakonick! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tests/MainTest.py:

Line 15:80: E501 line too long (87 > 79 characters)
Line 16:80: E501 line too long (88 > 79 characters)
Line 17:80: E501 line too long (81 > 79 characters)
Line 20:80: E501 line too long (81 > 79 characters)
Line 76:1: W391 blank line at end of file


feat(switchScan): add switchScan() operator

[x] Add the operator to Rx
[x] It must have a -spec.ts tests file covering the canonical corner cases, with marble diagram tests
[x] If possible, write a asDiagram test case too, for PNG diagram generation purposes
[x] The spec file should have a type definition test at the end of the spec to verify type definition for various use cases
[x] The operator must be documented in JSDoc style in the implementation file, including also the PNG marble diagram image
[x] The operator should be listed in doc/operators.md in a category of operators
[x] The operator should also be documented. See Documentation Guidelines.
[x] It should also be inserted in the operator decision tree file doc/decision-tree-widget/tree.yml
~You may need to update MIGRATION.md if the operator differs from the corresponding one in RxJS v4~

Description:
This PR adds switchScan operator as described here #2931 and reopened by @benlesh some time ago. I was told on Slack that it might get refused but I'm ok with that (no harsh feelings :)).
It's internally just using switchMap and tap because I first started implementing it myself but then realised I'm doing almost the same thing as switchMap is doing already so I just used combination of those two operators (I think it's been recommended to rather reuse existing operators than creating them from scratch). Tests are a combination of existing tests for scan and mergeScan.
I made it to work similarly to scan because that's more close to Array.reduce() even though mergeScan behaves a little differently. I tried to describe it here https://github.com/ReactiveX/rxjs/issues/4348#issuecomment-450561459 and here #4441 (the way it handles the first emission from source without seed and that it passes index to the accumulator function).
I also didn't add this operator to rxjs-compat because it didn't exist previously.
Related issue (if exists):
Closes #2931

Pull Request Test Coverage Report for Build 7826

20 of 20 (100.0%)  changed or added relevant lines in 3 files are covered.
3161 unchanged lines in 159 files lost coverage.
Overall coverage decreased (-53.3%) to 43.509%

Files with Coverage Reduction
New Missed Lines
%

src/internal/operators/distinctUntilKeyChanged.ts
1
40.0%

src/internal/util/isInteropObservable.ts
1
40.0%

src/internal/observable/concat.ts
1
76.92%

src/internal/operators/zipAll.ts
1
66.67%

src/internal/util/isDate.ts
1
25.0%

src/internal/operators/concat.ts
1
66.67%

src/internal/util/isNumeric.ts
1
40.0%

src/internal/operators/combineAll.ts
1
66.67%

src/internal/operators/publishLast.ts
1
75.0%

src/internal/observable/empty.ts
1
75.0%

Totals

Change from base Build 7818:
-53.3%

Covered Lines:
2604

Relevant Lines:
5985

💛  - Coveralls

@benlesh @cartant Hey, I updated this PR with everything you mentioned above.

I stumble over this open PR and see that all requested changes are implemented.
The failed build on node v9 is related to some non-code related things which I guess could be ignored.
@benlesh may I kindly ask you for a status update to this PR= :)

Any update on this PR? :+1:

Rebased to master and fixed some types in spec.

Thanks. I'll re-review it.

Oh man... this old thing. Sigh. Casualty of burnout mostly. I'll mark it as an agenda item and see if the team still wants to land it, or if it would be better in rxjs-misc.
cc @cartant

Resurrected! 🧟


Jc/alert boxes

Added a start script to run gulp and the serve to resolve https://www.pivotaltracker.com/n/projects/1187904/stories/92808932 and fixed the text on a danger alert to resolve https://www.pivotaltracker.com/n/projects/1187904/stories/92804110.

welcome jack!
Have a merry first PR! :notes:

@Dakuan Ready to go

@jarsbe give that bootstrap variable a go then :shipit:


Regex filter on groups.

Is it possible when using a groups filter to also use a regex as a filter.
When having a large number of test groups, where the groups alway begins with "test-" it is nice to have an regex filter to
have one entry of "test-*" that will include all the test- groups.
This is in favour of listing all separate test- groups individually.
And if a regex can be used, it is also nice to "negate" a single group so that it can be excluded from a regex.
We currently using keycloack as our identity provider.

+1 Request !

#1 Request


[examples] let's not use 'date' as a col name

'date' is a reserved word in most database

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4555   +/-   ##
=======================================
  Coverage   71.12%   71.12%
=======================================
  Files         187      187
  Lines       14780    14780
  Branches     1083     1083
=======================================
  Hits        10513    10513
  Misses       4264     4264
  Partials        3        3

Impacted Files
Coverage Δ

superset/data/__init__.py
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 2e780e4...8065147. Read the comment docs.

I agree about having it as a way to improve the tests and prevent regression, though my immediate urge was to fix the examples that were broken in MySQL.
In that spirit we do have a unicode-filled dataset in there, we should probably add a new dataset with reserved words as columns, table names and unicode-ridden column names and such.
Let's fix the examples for now and open an issue to address unicode and reserved words support as a task.


Tweaking vendoring

What was the end-user problem that led to this PR?
The problem was that we were not using the latest versions of some of our vendored dependencies.
What was your diagnosis of the problem?
My diagnosis was that we should upgrade them.
What is your fix for the problem, implemented in this PR?
My fix is to upgrade them using automatiek, and add a few tweaks to our vendoring setup.
Why did you choose this fix out of the possible options?
I chose this fix because.... I didn't really considered other options.

@bundlerbot r+


setBoolParameter in WebGL backend

Resolved the request for setBoolParameter #1499

I think you pushed to the wrong PR :)

Closed in favor of #1523


Localize Talpa checkout data

Description
Fully localize sendable Talpa checkout data to fi/sv/en.
Context
PV-681
How Has This Been Tested?
Manually.
Screenshots

Codecov Report

Merging #412 (47b0a9d) into develop (a9fbd91) will increase coverage by 0.29%.
Report is 1 commits behind head on develop.
The diff coverage is 94.80%.

@@             Coverage Diff             @@
##           develop     #412      +/-   ##
===========================================
+ Coverage    81.15%   81.45%   +0.29%
===========================================
  Files          150      150
  Lines         8899     9053     +154
===========================================
+ Hits          7222     7374     +152
- Misses        1677     1679       +2

Files
Coverage Δ

parking_permits/customer_permit.py
72.22% <100.00%> (ø)

parking_permits/exporters.py
53.89% <ø> (ø)

parking_permits/models/parking_permit.py
86.41% <100.00%> (ø)

parking_permits/models/vehicle.py
97.22% <100.00%> (+0.07%)
:arrow_up:

parking_permits/serializers.py
100.00% <100.00%> (ø)

parking_permits/talpa/order.py
36.63% <ø> (+0.98%)
:arrow_up:

...arking_permits/tests/models/test_parking_permit.py
99.33% <100.00%> (+<0.01%)
:arrow_up:

parking_permits/tests/models/test_product.py
96.72% <100.00%> (+0.16%)
:arrow_up:

parking_permits/tests/test_views.py
100.00% <100.00%> (ø)

parking_permits/urls.py
100.00% <ø> (ø)

... and 3 more

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


can't use outsideClickIgnoreClass with multiple class names

will be great to be able to pass multiple class names for outsideClickIgnoreClass like:
outsideClickIgnoreClass="class-a class-b".
you can change the function isNodeFound to check for multiple appearance of classes

I have the same situation, and I am looking forward to be able to pass multiple class names for outsideClickIgnoreClass.

Me too. I have the same situation, and I am looking forward to be able to pass multiple class names for outsideClickIgnoreClass.

Not for every case, but instead of passing multiple classes to outsideClickIgnoreClass you could try "lifting things up" by wrapping the required elements into a single parent element which you then ignore.


Don't raise warning if common name and hostname are the same

There are some SSL common names that are identical to the hostname, which was raised in https://github.com/boto/botocore/issues/2705#issuecomment-1236390963. This will suppress warnings where this is true.

Codecov Report

Merging #2756 (720da81) into develop (ba1ad06) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #2756   +/-   ##
========================================
  Coverage    95.13%   95.13%
========================================
  Files           61       61
  Lines        12530    12530
========================================
  Hits         11920    11920
  Misses         610      610

Impacted Files
Coverage Δ

botocore/client.py
98.74% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


closes #8 clarify company shared IP owners

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md under the header # admiralpeds (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiralpeds site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the Reference page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Based on monday's call, were we waiting on thomas' response to this as well?

Yes good point, guess no harm to give it a week or 2 to hear and then we can merge this one later. FYI @thomas-neitmann

@zdz2101 fine to approve and merge this one now


Issue with remove_duplicates.py

Description: Line 11 does not return anything
Steps to reproduce:

Run the program
try to use the returned value
value will be null or nonexistent
Expected behavior: To return the vector without duplicates
Actual behavior: Does not return any vector

Operating System: Windows 11
Browser: Firefox
Version: 110.0.1
Solution: Add a return value to line 11
This is a critical issue with the code
Reproducible in every instance of the program
High impact for anyone using that function

Issue fixed


Return key

Return key does not work for me ...

ok i found it !
In ImGui_ImplSdl_ProcessEvent() function
I need to add Return key to io.InputChar
    ...
    case SDL_KEYUP:
{
	int key = event->key.keysym.sym & ~SDLK_SCANCODE_MASK;
	io.KeysDown[key] = (event->type == SDL_KEYDOWN);
	io.KeyShift = ((SDL_GetModState() & KMOD_SHIFT) != 0);
	io.KeyCtrl = ((SDL_GetModState() & KMOD_CTRL) != 0);
	io.KeyAlt = ((SDL_GetModState() & KMOD_ALT) != 0);
	io.KeySuper = ((SDL_GetModState() & KMOD_GUI) != 0);

	if (event->key.keysym.scancode == SDL_SCANCODE_RETURN && event->type == SDL_KEYDOWN)
		io.AddInputCharactersUTF8("\r");

	return true;
}

Yes, it is treated as any other character.

Hi!
Return doesn't work for me either (using the glfw setup code from the imgui project - as seen here)


Add StakerHouse gentx and peer

Thank you for your interest in our network. Currently we are still testing internally. We will be circulating a form for the public testnet in the near future.


Add link between regulation of the force of heart contraction by chemical signal GO:0003057 & signaling

No link between
[Term]
id: GO:0003057
name: regulation of the force of heart contraction by chemical signal
namespace: biological_process
def: "The regulation of the force of heart muscle contraction mediated by chemical signaling, hormonal, autocrine or paracrine." [GOC:mtg_cardio, GOC:rl]
synonym: "chemical cardiac inotropy" EXACT []
synonym: "regulation of the force of heart muscle contraction by chemical signal" RELATED []
is_a: GO:0002026  ! regulation of the force of heart contraction

and GO:0023052 ! signaling
Should this be has_part? Or regulated_by? What does the LEGO diagram look like?
Reported by: cmungall
Original Ticket: geneontology/ontology-requests/11040

Equivalence axioms already in the ontology.


Missing icon: logos/pharmacies/pharmhouse.jpg

Specific message from TaggingPresetPreferenceTestIT:

Georgia => [00352.186 W: [NODE, CLOSEDWAY, MULTIPOLYGON] Pharmhouse: Could not get presets icon https://raw.githubusercontent.com/komachi/josm-ge-preset/main/logos/pharmacies/pharmhouse.jpg] ==> expected:  but was:

File renamed (https://github.com/komachi/josm-ge-preset/pull/62)


Update Hiking Label

Based on some messaging research we've been compiling, we want to make sure to include "walk" in the label, this change will also match the new website.
We can bundle this in to other updates we'll need to make re: nature center trails and trail IDs.
Change this:

to this:

Live on staging 00833bccdb920c709fb6e25bf80735701ef20245

Looks good.


Use unique_ptr to eliminate some trivial manual deallocation

Search before asking

[X] I had searched in the issues and found no similar issues.

Motivation
There are many manual memory (or other system resources like file) allocation and deallocation such as new/delete in kvrocks.
Memory management is a complex topic (after RC/tracing GC is involved quickly) and there are some smart pointers like unique_ptr, shared_ptr in C++ to abstract a specific way to manage resources.
For some trivial scenario which do not require to share objects, we can do it via unique_ptr, which is a zero-overhead abstraction.
We can switch some manual deallocations to unique_ptr to make the code more maintainable and less prone to memory leaks.
NOTE: We try to use unique_ptr instead of creating our own RAII class types because unique_ptr has good semantics: it prohibits copy construction and only allows move construction, which keeps us from accidentally sharing objects and causing Use After Free or Double Free. (And we should carefully to retrieve raw pointers via get() by checking that the callee do not require ownership)
Solution
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I'd like to do it.

Thanks, feel free to submit a PR.
Since this issue may be divided into many PRs, everyone can directly submit a PR for a specific part, so I think it would be better not to assign this issue to a specific person.

@PragmaTwice is there any remaining works and do you want to continue driving this effort?

I think most of trivial manual deallocation is removed, but there still remains some raw memory allocation or fd allocation.

@PragmaTwice is there a way we can find and list them out here? Otherwise, it seems this proposal stalls.

Closed. Will create new issue if more manual deallocations are found.


Add info to README/templates on float specifiers and representative images

Partially addresses #3
@dwsideriusNIST or @mrshirts can you please review? Makes updates to templates and README.md to mention representative images, give suggestions on float specifiers.

@dwsideriusNIST merged yours. Want to go ahead and merge this one?

Go ahead with the merge.


SetUpdate and SetOp appear redundant

Possible that both are needed, but they're virtually identical

Testing - ignore

Yeah, this is the typical duplication between the "api" and "core" sections of the client. Most users would only ever touch the SetUpdate class though.

Going to close this one, they may look identical but they do have different purposes. The other datatypes follow this pattern as well (not saying it's a great pattern), but wouldn't be worth the effort to take out.


Unicode data

I am trying to build a parser for JP hence the training data is saved UTF-8. However the parserator throws a UnicodeDecodeError. Is there anything I can do to work around this? newaddr.csv attached - saved as .txt
newaddr.txt
parserator label training/newaddr.csv training/newaddr.xml usaddress
Traceback (most recent call last):
File "c:\python27\lib\runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "c:\python27\lib\runpy.py", line 72, in run_code
exec code in run_globals
File "c:\Python27\scripts\parserator.exe_main.py", line 9, in
File "c:\python27\lib\site-packages\parserator\main.py", line 58, in dispatch
args.func(args)
File "c:\python27\lib\site-packages\parserator\main.py", line 79, in label
manual_labeling.label(module, infile_path, outfile_path)
File "c:\python27\lib\site-packages\parserator\manual_labeling.py", line 207, in label
strings = set(row[0] for row in reader)
File "c:\python27\lib\site-packages\parserator\manual_labeling.py", line 207, in
strings = set(row[0] for row in reader)
File "c:\python27\lib\site-packages\backports\csv.py", line 394, in next
lineobj = next(self.input_iter)
File "c:\python27\lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 4: character maps to

This isn't a part of the codebase that I'm super familiar with, but you might be able to hack together a solution by modifying the list comprehension in parserator that's failing. I'd try something like:
strings = set(row[0].decode('utf-8') for row in reader)

The more correct thing to do here would be to optionally use backports.csv if we are using python 2. https://pypi.python.org/pypi/backports.csv

closed by https://github.com/datamade/parserator/commit/f61465e82867f94daf5e781d8f3b498b0713d061


Publish a new version?

This commit:
https://github.com/mklabs/tiny-lr/commit/5754e8cc0ac2b430c78b265a84b459b054aaa889
fixes my problem, but it is 24 days old and hasn't been published to npm. Please publish a new version?

Same here.

+1 (the most helpful of comments)

Please
All npm 3+ Break on this
:+1:

Published and sorry for the delay!


Improve sparse solvers

Codecov Report

Merging #50 (bdf90a7) into main (85a35b2) will decrease coverage by 1.65%.
Report is 1 commits behind head on main.
The diff coverage is 88.52%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
- Coverage   99.62%   97.97%   -1.65%
==========================================
  Files          98      104       +6
  Lines       18916    20947    +2031
==========================================
+ Hits        18845    20523    +1678
- Misses         71      424     +353

Files
Coverage Δ

russell_sparse/src/auxiliary.rs
100.00% <100.00%> (ø)

russell_sparse/src/coo_matrix.rs
100.00% <100.00%> (ø)

russell_sparse/src/read_matrix_market.rs
100.00% <100.00%> (ø)

russell_sparse/src/samples.rs
100.00% <100.00%> (ø)

russell_sparse/src/verify_lin_sys.rs
98.64% <90.00%> (-0.07%)
:arrow_down:

russell_sparse/src/write_matrix_market.rs
98.56% <80.00%> (ø)

russell_sparse/src/solver.rs
97.18% <96.61%> (+0.21%)
:arrow_up:

russell_sparse/src/csr_matrix.rs
98.37% <97.67%> (-1.63%)
:arrow_down:

russell_sparse/src/csc_matrix.rs
97.93% <97.93%> (ø)

russell_sparse/src/enums.rs
92.48% <83.33%> (-7.52%)
:arrow_down:

... and 4 more

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


SVG viewBox= width= height= are incorrectly modified by clipSvgDataUri

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
The SVG viewBox= width= height= are incorrectly modified by clipSvgDataUri.
To Reproduce
Steps to reproduce the behavior:

Please create a .svg text file with this SVG code:<svg viewBox="0 0 45 45" xmlns="http://www.w3.org/2000/svg"><circle cx="22.5" cy="22.5" fill="#57d9a3" r="21" stroke="#0052cc" stroke-miterlimit="10" stroke-width="3"/></svg>
Drag and drop the file in draw.io the editor
Here is the result in the data shape:
data:image/svg+xml;base64,PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHZpZXdCb3g9IjEuNSAxLjUgNDIgNDIiIHdpZHRoPSI0MiIgaGVpZ2h0PSI0MiI+PGNpcmNsZSBjeD0iMjIuNSIgY3k9IjIyLjUiIGZpbGw9IiM1N2Q5YTMiIHI9IjIxIiBzdHJva2U9IiMwMDUyY2MiIHN0cm9rZS1taXRlcmxpbWl0PSIxMCIgc3Ryb2tlLXdpZHRoPSIzIi8+PC9zdmc+
<svg xmlns="http://www.w3.org/2000/svg" viewBox="1.5 1.5 42 42" width="42" height="42"><circle cx="22.5" cy="22.5" fill="#57d9a3" r="21" stroke="#0052cc" stroke-miterlimit="10" stroke-width="3"/></svg>

Expected behavior
As you can see the dimensions are not correct :
<svg xmlns="http://www.w3.org/2000/svg" viewBox="1.5 1.5 42 42" width="42" height="42"><circle cx="22.5" cy="22.5" fill="#57d9a3" r="21" stroke="#0052cc" stroke-miterlimit="10" stroke-width="3"/></svg>
Same thing, if I uncheck "Responsive" I am responsive in Illustrator
<svg id="Calque_1" data-name="Calque 1" xmlns="http://www.w3.org/2000/svg" width="45" height="45" viewBox="0 0 45 45"><defs><style>.cls-1{fill:#57d9a3;stroke:#0052cc;stroke-miterlimit:10;stroke-width:3px;}</style></defs><circle class="cls-1" cx="22.5" cy="22.5" r="21"/></svg>
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version : https://app.diagrams.net & desktop 15.8.7

Desktop (please complete the following information):

OS: macOS
Browser: chrome
Version: 96.0.4664.93

Thanks for the report. This will be fixed in the next release.


Switch demo plugin installation to use jenkins-plugin-cli

Running make -C demos build results in this warning: install-plugins.sh has been removed, please switch to jenkins-plugin-cli, which results in a failed build locally. Pinning the base image would also beneficial.

Thanks @ryanmrodriguez !


Need support to enable Monetization for APIs in APIM 260

Description:
We need to get the implementation details in place so that we can have similar content with regard to enabling monetization in API-M 2.6.0. Currently, this information is not available in API-M 2.6.0.
[1] https://docs.wso2.com/display/AM250/Enabling+Monetization+of+APIs
Affected Product Version:
APIM 2.6.0
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

6 month have passed and still no documentation update for monetization using WSO2 APIM 2.6.0 :) is there some document available ?

@crognon Please check the following release[1]. It supports APIM 2.6.0 monetization. We hope to provide the instruction and documentation in this repo[2]. You can try it and let us know if you need further help.
[1] https://github.com/ruks/wso2-am-billing-engine/releases/tag/v2.6.0-alpha
[2] https://github.com/ruks/wso2-am-billing-engine

Hi ruks,
I am trying to use your billing workflow code, and I encountered some errors.
After clicking on subscribe, I got this error in wso2APIM store :
Caused by: org.apache.http.ProtocolException: Target host is not specified
I have replaced the subscription workflow with yours (in /_system/governance/apimgt/applicationdata/workflow-extensions.xml).
I have setup the billing engine following your documentation (in repository/conf/api-manager.xml)
   <BillingEngine>
        <BillingEngineUrl>http://localhost:8080/apim-billing-engine-1.4.0/</BillingEngineUrl>
        <ApimStoreUrl>https://localhost:9443/store/</ApimStoreUrl>
    </BillingEngine>

The stacktrace contains :
org.apache.http.client.ClientProtocolException
        at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:186)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:106)
        at org.wso2.sample.apimgt.impl.BilllingEngineConnectorImpl.checkSubscription(BilllingEngineConnectorImpl.java:31)
        at org.wso2.sample.apimgt.workflow.SubscriptionBillingWorkflow.execute(SubscriptionBillingWorkflow.java:69)
        at org.wso2.carbon.apimgt.impl.APIConsumerImpl.addSubscription(APIConsumerImpl.java:2328)
        at org.wso2.carbon.apimgt.impl.UserAwareAPIConsumer.addSubscription(UserAwareAPIConsumer.java:59)

It seems that my billing engine url is not taken into account.
Have I miss something ?
Thank you.

Forget about my last message, after restarting the whole stack, this error was not here anymore.
I still got a strange behaviour :
I can't sign-in to the billing-app (even after regitered my user).
I don't have any error, and I see the new user in the ysql db.
I will do a remote debug in order to understand (because I just see an hibernate select in the tomcat logs).

I think this is session related issue. Can you please check on prrivate browser. Once you subscribe again to a billing package your subscription will activate again.

Hello,
Thank you, with a private browser window I can log into the billing app.
But when I subscribe I'm not redirected to the store and I always stay in the billing app (and there is not much to do there).
Here are some screenshots :
https://drive.google.com/drive/folders/1JIqWdsBDWVCHmM01iZWQLu9EnE_Zlbbl?usp=sharing
This makes my subscriptions to be all in the status ON_HOLD (even the ones attached to a free billing plan).

@dengrand once you subscribe to a billing package your subscription should be activated, You need to select a billing plan corresponding to the correct API and version. Also, you check the pending subscription in the billing engine menu.

@ruks Thank you, it works fine.
I just didn't know how to use your billing application.
Maybe you could improve the readme with billing app usage.

@dengrand sorry for the confusion. Documentation with complete instructions is work in progress. Will update readme once it is completed.

This doc is still not available?

@Mariangela can we update the 2.6.0 documentation? We can point the documentation to this[1]
[1] https://github.com/ruks/wso2-am-billing-engine/releases/tag/v2.6.0-alpha


operator qetesting (0.0.3)

New operator bundle
Name: qetesting
Version: 0.0.3
Certification project: 61e6bac38255aca3a3297586
Test result URL: https://catalog.stage.redhat.com/api/containers/v1/projects/certification/test-results/id/620fdbf38f7e5f81b311ff4d
Test logs URL: https://catalog.stage.redhat.com/api/containers/v1/projects/certification/artifacts/id/620fdbf18f7e5f81b311ff4c

@amisstea  @Allda  , Well this sound like interesting issue, This is the Only automated tests  first PR which passed  "generate index" task  on pre-prod environment. But not sure how ? Are there any changes done recently on pre-prod env as all the recent tests are failing again on generate index task .

@maveric-tellrv nothing I'm aware of but almost all errors from generate-index are likely to be caused by the state of the index image at the time of execution. Additionally, these index images are imported into our cluster using the default scheduler so unless there are clues in the opm output, we may not be able to debug this reasonably.


UnhandledPromiseRejectionWarning: Error: An error occured while starting Youtube together !

I have been getting this error for a few days without having touched any of the code. "UnhandledPromiseRejectionWarning: Error: An error occured while starting Youtube together !"

Information:

Error: UnhandledPromiseRejectionWarning: Error: An error occured while starting Youtube together !
Node Version: 14.17.3
DiscordTogether Version: ^1.1.14

(node:3781) UnhandledPromiseRejectionWarning: Error: An error occured while starting Youtube together !     at DiscordTogether.createTogetherCode (/home/runner/Zanix-Bot/node_modules/discord-together/src/DiscordTogether.js:95:23)     at runMicrotasks (<anonymous>)     at processTicksAndRejections (internal/process/task_queues.js:95:5) (Use node --trace-warnings ...to show where the warning was created) (node:3781) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag--unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2) (node:3781) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hey, join my discord server to be able to talk easier
https://discord.gg/UBUSgw4

Thanks.

Thanks.


ImageSeriesReader() slices mismatch

Hi
I'm using simpleitk    ImageSeriesReader() to convert dicom to nifti.
I have T1, FLAIR, T2 and PD images in my dicom series.
I used:
reader = sitk.ImageSeriesReader()
for dicom_series_id in reader.GetGDCMSeriesIDs(dicom_directory):
dicom_names = reader.GetGDCMSeriesFileNames(dicom_directory, dicom_series_id)
reader.SetFileNames(dicom_names)
image = reader.Execute()
For T1 and FLAIR, they are converted very well.
But for T2 and PD, the slice mismatchs. For example, T2-w image has 100 slices(the slice with odd number is PD image). slice 0:2:99 is T2, slice 1:2:99 is PD. Same for the PD image.
I think this is because series_id mismatchs?
Thanks for helping!

slice 'x'

slice 'x+1'

Hello @HaoLi12345,
I suspect that the T2 and PD images don't have unique series IDs which is how the GetGDCMSeriesFileNames identifies files as belonging to the same volume. Please print the  list of series ids you get from reader.GetGDCMSeriesIDs(dicom_directory), if the number of series IDs is not 4 then multiple series are sharing the same ID.
In this case, you will need to use another DICOM tag to separate the T2 and PD image file names into different lists before reading. Possibly the series time tag '0008|0031' is different between them even though they share the same series number. If not, you will need to find a tag that is distinct to each of the series.
Finally, the reason you are getting interleaved slices between the series is that there are two slices for each z location, so when the slices are sorted for reading you get T2_z0, PD_z0, T2_z1, PD_z1... .

I'm new to simpleitk and thanks for answering.
How to set a new tag? or how to use the tag '0008|0031' to separate them?

Hello @HaoLi12345,
This is more of a DICOM issue than a SimpleITK issue. It is up to you to decide which tag can be used to separate the data. You will need to familiarize yourself with the meaning of the tags (they are reported using their numerical values and the DICOM standard tells you what they mean).
When you have the list of files you can go over their DICOM tags and see if there is a tag that can be used to separate the data. Below is code that prints the tag values for the first two images in the file list:
import SimpleITK as sitk

dicom_directory = 'CIRS/'
reader = sitk.ImageSeriesReader()

series_ids = reader.GetGDCMSeriesIDs(dicom_directory)
dicom_names = reader.GetGDCMSeriesFileNames(dicom_directory,series_ids[0])
reader.SetFileNames(dicom_names)

# Have the reader read all of the DICOM metadata information, including private data
reader.MetaDataDictionaryArrayUpdateOn()
reader.LoadPrivateTagsOn()
image = reader.Execute()

# print all the tags for image_0 and image_1
for i in range(2):
    print('*'*30)
    for k in reader.GetMetaDataKeys(i):
        print('{0}: {1}'.format(k, reader.GetMetaData(i,k)))

thanks for helping me, this is really helpful!


IBM DB2 Support

Are you considering support for IBM DB2 for Prisma 2?
Thanks for your great efforts and this stellar product.

This would probably be easier to make due to db2 having a docker container and not that restrictive licensing on can you build a custom client to it compared to Oracle, so this might be easier to do...
The rust driver support is a bit iffy. There's a sync one that doesn't really fit to our async model. Although, we basically rewrote the whole SQL Server driver anyhow, so I don't see any other problems than time to make a DB2 connector to happen.
https://github.com/ibmdb/rust-ibm_db

Any news on this?

I'd be willing to contribute to making this possible

I am also interested in the subject, I will delve deeper to understand how to implement this data source in Prisma

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
If this is issue is still important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest. If you do reopen the issue, please ensure that there’s a clear method for us to reproduce it.
For more insight into our priorities and how we’re thinking about the future of the Prisma ORM, please see our recent blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


self.enter_variable_scope()

I am not to understand the working of this and where is this defined. Can anyone please help with this? also if there is some slack channel or forum where I can ask my questions that will of great help. Thank you

Hi @Pranav-India, that API is only used in Sonnet 1 which only supports TensorFlow 1 and is deprecated.
If you need to use TensorFlow 1 for some reason then you can find the source for Sonnet 1 in the master branch, the method is defined here https://github.com/deepmind/sonnet/blob/master/sonnet/python/modules/base.py#L536.
We recommend that new projects use Sonnet 2 and TensorFlow 2. You can find documentation on Sonnet 2 at https://sonnet.readthedocs.io/en/latest/.
We use GitHub issues for discussion and don't have a Slack/Discord I'm afraid.

Hi @Pranav-India, that API is only used in Sonnet 1 which only supports TensorFlow 1 and is deprecated.
If you need to use TensorFlow 1 for some reason then you can find the source for Sonnet 1 in the master branch, the method is defined here https://github.com/deepmind/sonnet/blob/master/sonnet/python/modules/base.py#L536.
We recommend that new projects use Sonnet 2 and TensorFlow 2. You can find documentation on Sonnet 2 at https://sonnet.readthedocs.io/en/latest/.
We use GitHub issues for discussion and don't have a Slack/Discord I'm afraid.


Show example code

There was a missing line that hid the code example of how to avoid deletion of SchemaMigration nodes:
Before:

After:

Pings:
@cheerfulstoic
@subvertallchris

Codecov Report

Merging #1408 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1408   +/-   ##
=======================================
  Coverage   96.93%   96.93%
=======================================
  Files         205      205
  Lines       12403    12403
=======================================
  Hits        12023    12023
  Misses        380      380

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 d5dc490...af9a5ec. Read the comment docs.

Awesome, thanks!  Pushed to readthedocs


Export ES6 code to save bytes

Thanks for making this package!
This module targets Node 10+ which supports ECMAScript 6 features. As a minor improvement, we can compile to ES6 instead of ES5. This saves 78 bytes from the built index.js and makes the compiled output a bit more readable.
I didn't update the changelog because this is not a change that affects users in a meaningful way.
Here's what the diff of the compiled file looks like.
If you're interested in collaborating more broadly on this module and/or eventually integrating it into Helmet, please email me at<EMAIL_ADDRESS>or contact me another way.

Yep makes sense to me, I'll merge and release this in the 0.5.0 version.
I'll be reaching you through e-mail to speak about that collaboration 👍


Fix // parsing in packageDependencyCodeSnippet

Fixes #1563

I'll add an issue for us to actually construct this from data upstream and add it to me model properly rather than parsing it from bits, which I should have done in the first place.


Travis tests - NOT A REAL PULL REQUEST

Just a test bed to kick Travis without cluttering up another branch with a bunch of useless commits.

Done with this branch.


onOpen and set(option,value) doesnt work for defaultMinute and defaultHour option value

onOpen={function(selectedDates, dateStr, instance){
instance.set("minDate", new Date());
}}
The above code works fine, but when i need to set the defaultMinute / defaultHour value in the config,
it doesn't work as expected.
onOpen={function(selectedDates, dateStr, instance){
instance.set("defaultMinute", new Date().getMinutes());
}}
onOpen={function(selectedDates, dateStr, instance){
instance.set("defaultHour", 18);
}}
Can you please let know if this can achieved.

any progress to this case? Please let me know if someone solve problem with instance.set('MinDate', new Value here)


Enable pulling private GitHub repos for Neurohackademy

Following instructions in #2849 as an alternative for authenticated static sites
new hub issue: #2681
I reused the same GitHub App that I had previously set up for the static site and is already installed on the appropriate repo.

I am holding off on merging this as nbgitpuller is returning a 128 error status

Booooo! (Let me push the commit that generated this)
2023-07-21T16:57:09Z [Warning] Error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/var/lib/kubelet/pods/8b096c8a-84e4-4bee-b35c-18e9025c7c4e/volume-subpaths/files/notebook/2" to rootfs at "/srv/conda/envs/notebook/etc/gitconfig/github-app-private-key.pem": open o_path procfd: open /run/containerd/io.containerd.runtime.v2.task/k8s.io/notebook/rootfs/srv/conda/envs/notebook/etc/gitconfig/github-app-private-key.pem: not a directory: unknown

No wait, I think I've just done the wrong thing.. hang on

Cracked it!


Add option to preserve image filename on upload

Currently when an image is uploaded, its filename is set to a random 20 character string.
This pull request adds an option to the field details that will preserve the filename of the uploaded file. Since the new behavior is opt-in it will not introduce any breaking changes.
To use, you only need to add the option to the field options like so
{
    "preserveFileUploadName": true
}

This addresses issue #1184

Thanks @david-mk
Will this overwrite a file with the same name? I can still merge it, but I think it would be cool if we preserved the name by default but appended a random string or a number if the file already exists.
Let me know your thoughts :)
Thanks.

I'm leaning toward maintaining the file name in the database and keeping the random string on the filesystem... This essentially removes the chance for conflict, and still makes the file accessible by it's original name.  I think this is what we already do (or at least very close to it).  What's the use-case for NEEDING the files to have the same name on the filesystem?

I found that in the Media view, its difficult to manage images with random file names. This is especially true when you have multiple thumbnails per image and its not obvious at a glance which thumbnail you're seeing without clicking through each of them. Another case is if you have multiple images that appear similar.
Here is an example where I have the main image and 2 thumbnails. Its not clear without clicking through each of them which one is which.

Also, the database does not preserve the filename. It uses the randomly generated name.

I also think there's an argument to be made for allowing voyager to give me control of my file names in the filesystem if I want it to, which is why I think its good as an opt in feature and not the default behavior.
However I do take your point that it increases the possibility for file name clashing. So I think a good solution would be to use @tnylea's suggestion and append the random string which would give the best of both worlds.

@david-mk I've added the ability to preserve the origin image name by default. If there is already an image with that name it will add +1 to the end so for the following image name image.jpg, if it already exists the following will be used image1.jpg, if that exists, it will be image2.jpg, and so on 👍
Thanks for the PR.


Lazy Loading Javascript Animation

Describe the project you want to add with tech stack

I'd add a lazy loading javascript animation where the content once fetched is firstly initially blurry. Then as the loading approaches nearer to 100 percent, then the blur starts to decrease proportionately unitil it completely wanishes and becomes zero.
Expected behavior

There is a text based loading percentage indicator in the center. On the background is the page to be fetched.
As the fetching progresses, the background gets filled with the page view but is initially blurred.
As the loading progresses towards 100 percent, the blurriness of the background decreases proportionately.
As the loading completes, the page displayes without any blur.

Screenshots (optional)

@pranjay-poddar @MohitGupta121 @TusharKesarwani please assign me this!


Update to Jackson 2.8.5

This also simplifies the dropwizard-bom by just relying on the jackson-bom and jersey-bom.  All of the tests still pass and the example application is properly packaged without having to maintain all of the individual dependencies in the bom pom.xml.
(the preferred jackson-bom version to use is <IP_ADDRESS> per https://twitter.com/fasterxml/status/801240492000231424)

Coverage remained the same at 81.694% when pulling 43fb5cb98e6ce2ff71de194a83faf92e99512d38 on jplock:jp-jackson into 31ee21506d0a5236ec27196a5f15f1a301478ba1 on dropwizard:master.

Coverage remained the same at 81.694% when pulling 43fb5cb98e6ce2ff71de194a83faf92e99512d38 on jplock:jp-jackson into 31ee21506d0a5236ec27196a5f15f1a301478ba1 on dropwizard:master.

Wow, jackson-bom is a great thing.


Fix: Closing Publish Modal Redirects To Previous URL

📃 Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Related Tickets & Documents
Fixes # (issue)
What type of PR is this? (check all applicable)

[ ] 💡 Feature
[x] 🐛 Bug Fix
[ ] 📃 Documentation Update
[ ] 🎨 Style Changes
[ ] 💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test

🔰 Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

❓ How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
🔰 Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

@obere4u the aim of this PR is unclear. Please refer to the issue statement again and revisit your solution. Thank you.

If I understand this very well, the [x] on the model, when clicked will reroute the page to a previous route, right? because when I checked the code, I saw that router.back() is making the page go back to the previous route instead of only setting the setModal(false)

@obere4u yes we need to reroute the page to previous page for the authentication modals but we need find a work around to prevent this in case of publish modal.
All these things are mentioned in issue #179 as well

@obere4u do you need more time to fix to we should close this PR? Seems it doesn't fix the issue..


I'm getting "Server error: POST https://svcs.ebay.com/services/search/FindingService/v1 resulted in a 500 Internal Server Error"

Hi,
i'm getting a http 500 error. I did not changed anything, but till saturday i'm getting this error on different servers. I've also recreated my certifiate ID, but nothing changes.
Here is a sample code (it's very similar to the samples):
`require_once DIRPATH . '/lib/vendor/autoload.php';
$service = new  DTS\eBaySDK\Finding\Services\FindingService( [
'apiVersion' => '1.13.0',
'globalId'   => $globalId,
	'credentials' => [
			'appId'  => $appId,
			'certId' => $certId,
			'devId'  => $devId
	]

] );
$request           = new DTS\eBaySDK\Finding\Types\FindItemsByKeywordsRequest();
$request->keywords = 'Harry Potter';
// Ask for the first 25 items.
$request->paginationInput                 = new DTS\eBaySDK\Finding\Types\PaginationInput();
$request->paginationInput->entriesPerPage = 1;
$request->paginationInput->pageNumber     = 1;
// Send the request.
$response = $service->findItemsByKeywords( $request );`
I't would be great, if someone has an idea or can test the sample above....

I face the same problem but could not find any solution for this. Please help us

Having the same issue, with FileTransfer downloadFile call, could someone else confirm?

Hi,
i found the solution for this issue. ebay has implemented a limitation of requests. If this daily limit is reached, you are getting this exception.
https://developer.ebay.com/support/api-call-limits
You can create a compatible check: https://developer.ebay.com/support/app-check
I've submitted ths app check and was approved. The error is gone.

didn't work for me, today with no requests, just got the tokens and then error again

Yeah, the api blocking is not just one day. A customer has testet it and told me the blocktime is arbitrary.
That issue is not a problem of the php sdk. Just create a ticket and ask the ebay support. My issue was confirmed by them.


Is this as accurate as vive tracker puck?

This seems cool. I'd like to able to create some small trackers to keep track of physical objects in VR. Things like knowing where my keyboard and mouse are. But these are only accurate to 2mm? That doesn't seem very accurate. Is there anyway to improve that?

What kind of accuracy would be enough for your task? The position can be
averaged across several samples to get better values, at the expense of
slightly sluggish response. But it's hard to get better than 0.5-1.0 mm.
As for the title of the issue - we don't know the accuracy of vive tracker
(afaik it's not out yet), so I cannot compare. I think it'll be similar.
On Mar 11, 2017 4:35 PM, "David French"<EMAIL_ADDRESS>wrote:

This seems cool. I'd like to able to create some small trackers to keep
track of physical objects in VR. Things like knowing where my keyboard and
mouse are. But these are only accurate to 2mm? That doesn't seem very
accurate. Is there anyway to improve that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/16, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmVHeXy13Z_jSlFuuH68h09H2nhAJarks5rkz28gaJpZM4MaY0P
.

Aren't the controllers and headset supposed to be submillimeter tracking?
Maybe that's just marketing hype I heard though.
On Sat, Mar 11, 2017, 21:24 Alexander Shtuchkin<EMAIL_ADDRESS>wrote:

What kind of accuracy would be enough for your task? The position can be
averaged across several samples to get better values, at the expense of
slightly sluggish response. But it's hard to get better than 0.5-1.0 mm.
As for the title of the issue - we don't know the accuracy of vive tracker
(afaik it's not out yet), so I cannot compare. I think it'll be similar.
On Mar 11, 2017 4:35 PM, "David French"<EMAIL_ADDRESS>wrote:

This seems cool. I'd like to able to create some small trackers to keep
track of physical objects in VR. Things like knowing where my keyboard
and
mouse are. But these are only accurate to 2mm? That doesn't seem very
accurate. Is there anyway to improve that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/16, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAmVHeXy13Z_jSlFuuH68h09H2nhAJarks5rkz28gaJpZM4MaY0P
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/16#issuecomment-285919050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJA4zvmpXzrLTa-nwyBz23AkCqmonFrVks5rk2V8gaJpZM4MaY0P
.

--
David Aslan French
http://www.jack-a-lope.net/

Well, technically 0.99mm is submillimeter tracking :)
Btw, it's hard to define what we actually mean by accuracy. Is it the STDEV
of position readings in a static position? Is it max difference between
actual position (measured by ruler) and position provided by sensor across
the whole tracking volume? Or difference between what HTC Vive original
tracking devices give and what the sensor gives?
These three will give quite different results.
--
Alexander Shtuchkin
On Sun, Mar 12, 2017 at 9:15 AM, David French<EMAIL_ADDRESS>wrote:

Aren't the controllers and headset supposed to be submillimeter tracking?
Maybe that's just marketing hype I heard though.
On Sat, Mar 11, 2017, 21:24 Alexander Shtuchkin<EMAIL_ADDRESS>wrote:

What kind of accuracy would be enough for your task? The position can be
averaged across several samples to get better values, at the expense of
slightly sluggish response. But it's hard to get better than 0.5-1.0 mm.
As for the title of the issue - we don't know the accuracy of vive
tracker
(afaik it's not out yet), so I cannot compare. I think it'll be similar.
On Mar 11, 2017 4:35 PM, "David French"<EMAIL_ADDRESS>wrote:

This seems cool. I'd like to able to create some small trackers to keep
track of physical objects in VR. Things like knowing where my keyboard
and
mouse are. But these are only accurate to 2mm? That doesn't seem very
accurate. Is there anyway to improve that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/16, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAmVHeXy13Z_
jSlFuuH68h09H2nhAJarks5rkz28gaJpZM4MaY0P
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/ashtuchkin/vive-diy-position-
sensor/issues/16#issuecomment-285919050>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJA4zvmpXzrLTa-
nwyBz23AkCqmonFrVks5rk2V8gaJpZM4MaY0P>
.

--
David Aslan French
http://www.jack-a-lope.net/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/16#issuecomment-285955076,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmVHTosfFoP3LEDYQ2gExHWK4dfqo5Uks5rlBoSgaJpZM4MaY0P
.

@ashtuchkin That makes sense. Would you say on an experiential level it is similar in accuracy to a Vive controller or headset?
I'm going to be build it anyway but I'm curious on your thoughts. I decided if nothing else I want to build 2, one for my headphones and one for the hat rack I keep my vive and controllers on. That way I can mount and dismount in VR more easily :D

It's similar, as long as the base stations are not occluded. Currently the main downside of this diy project is it doesn't handle situation where only 1 base station is visible, unlike the vive puck. I'm doing some experiments on it, but not sure how close I can do.
Btw, I ported this project to Particle Photon board as well, so now it can be used without wires.


New version: igoogolx.itun2socks version 1.21.4

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-11-01 21:36:08.920 [FAIL] Installer failed security check. Url: https://github.com/igoogolx/itun2socks/releases/download/v1.21.4/itun2socks_1.21.4_windows_amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)


[LLM Runtime] Remove use_cache in WOQ

Type of Change
feature or bug fix or documentation or others
API changed or not
Description
Past:
| use_cache | Bool | Use local quantized model if file exists (default: False) |
Now:
We auto detect the file, don't need consumers to aware this args.
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

can you give a brief demonstration that what use_cache is, why it was added and now why it is deperated?

| use_cache | Bool | Use local quantized model if file exists (default: False) |

Now, we auto detect the file, don't need consumers to aware this args.

Now, we auto detect the file, don't need consumers to aware this args.

Can you link the pr since which models can auto detected?

Now, we auto detect the file, don't need consumers to aware this args.

Can you link the pr since which models can auto detected?

all the models can auto detected, as long as you already get the converted model which means *.bin format.


Patch 8.2.5157 breaks Ctrl key mapping on Windows

Steps to reproduce

Compile vim up to 8.2.5157
Copy some text
Type Ctrl-v

What happens:
Nothing happens.
If I revert to the previous commit, <C-v> works normally.
Expected behaviour
Copied text pasted.
Version of Vim
8.2.5157
Environment
Windows 10
Logs and stack traces
No response

It looks like after 8.2.5157, Ctrl key bypass user mappings. Typing Ctrl + some key results in Vim's builting mappings. For example in insert mode <C-a> increases the number under the cursor although in Windows, <C-a> is remapped to select all.


chore(master): release 1.5.4

:robot: I have created a release beep boop
1.5.4 (2023-08-31)
Miscellaneous

deps: update portainer/portainer-ce docker tag to v2.19.0 (f1c857a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-shared-services/releases/tag/v1.5.4 :sunflower:


Automatically wipe old pipeline runs

Need to automatically wipe old pipeline runs.

Completed, but also no longer needed due to persistent disk approach.


[type: feat] create user api.

With pull requests:

Open your pull request against main
Your pull request should have no more than two commits, if not you should squash them.
You should pass all tests by go test -v ./... at the project root path in local.
It should pass all tests in the available continuous integration systems such as GitHub Actions.
You should add/modify tests to cover your proposed code changes.
If your pull request contains a new feature, please document it on the README.

I have read the CLA Document and I hereby sign the CLA

Codecov Report

Merging #91 (94d50e1) into main (c57d24f) will decrease coverage by 3.28%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
- Coverage   34.21%   30.93%   -3.29%
==========================================
  Files          13       13
  Lines         415      459      +44
==========================================
  Hits          142      142
- Misses        257      301      +44
  Partials       16       16

Impacted Files
Coverage Δ

internal/core/openapi/auth.go
0.00% <0.00%> (ø)

internal/core/openapi/password.go
60.00% <0.00%> (-40.00%)
: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 c57d24f...94d50e1. Read the comment docs.

I have read the CLA Document and I hereby sign the CLA


  It seems like something is happening in a nightly cron somehow that's triggering the tool to get run:

It would be awesome if it only ran after things landed into master :)

@JustinBeckwith I think the solution to the spam issues that you've brought up would be to check whether the CHANGELOG content has changed since a PR was last opened, and to short-circuit if it hasn't -- does this sound reasonable?


Lighthouse tests + fixes

We need to improve on basic metrics of performance + add PWA manifest

Fixed a little bit. Further works on that after releasing Public Demo (PoC)


vendor-install: Add ARM bottle for portable-ruby

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

When is this bottle used?

This bottle is used on Linux systems running on ARM hardware. The most common example is Raspberry Pi.

Thanks again @sjackman!

Thanks for merging, Mike!


OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to instagram.com:443

Getting this error. Anyone else? Please help me to resolve this
Thanks

I am getting the same error but it happens randomly. For the time being I am catching the exception and retrying the request. It seems to be working.


pull upstream - upgrade

📝 Changed routes:

/[pageId]

(view full size)

Commit 380955043a71d4c89443b1fd6da973431f4c7d80 (https://nextjs-notion-starter-ircl9o32b-emindeniz99.vercel.app).


Publish to Archlinux AUR

Hi,
I thing the project is useful and would benefit more people, if only more people knew about it.
So I have published the project to the AUR.
You can find it here:
https://aur.archlinux.org/packages/yournal
Its also kinda easier to use this way.. no need to clone anything, just yay -S yournal.
Look I even made a video for you of the installation and usage:

As on your part, you could mention it in the README file, for example:

... Installation ...
On Arch Linux via AUR:
yay -S yournal

I would ultimately like for you (@Rikiub) to take ownership of the package mentioned above, if your'e, by any chance, an Arch Linux user.
Otherwise I will be fine keeping it under my AUR user.

Bonus:
My alias, you could include it also somwhere in the README if you'd like :)

Good job btw.

Yeah, I'm a Arch Linux user :D
I was actually thinking of publish it to the AUR but I don't have much experience making "packages". Thanks so much for taking the time to publish the package! :100:
First I will publish the package in PyPi and then I would like to focus on the Linux packages, so I would like to take ownership of the package later.

And sure, I will add the alias example to the README.

I can't see the video, the gif is just... in black?

Awesome!
The video was a showcase of the installation and the fact that its bieng installed automatically into /usr/bin..

I'm done with PyPi packages, so look for me in the AUR as [Rikiub(https://aur.archlinux.org/account/Rikiub).


Part 1 (frontend): Introduce new codegen to support fragments as interfaces

Introduce new codegen AST to make it possible generate named / inline fragments as interfaces, following the similar approach described by Airbnb.
This is the first phase (we can call it as frontend) of new codegen, build AST structure that very closely represents Kotlin / JVM code to be generated.
Next phase (backend) is actually take AST and transform it to the Kotlin / JVM code.
There are no tests yet as this is just a first step of new codegen, tests will be added with next PR for backend.

Awesome 😃 . I just merged master into dev-3-x to make sure we don't diverge too much.


Static code analysis via Devtron

Summary
Devtron works well as a CI/CD tool. But it does not provide a mechanism to integrate any static code analysis tool.
Proposal
We are looking at Devtron to provide a Global Plugin integration. Users should be able integrate any static code analysis tool (like Sonarqube), via a plugin, like in Jenkins. This would give them more options to integrate a tool of their choice and not force them to use a specific tool.

Dependency:  https://github.com/devtron-labs/devtron/issues/1418

Linked to https://github.com/devtron-labs/devtron/issues/1418 and released


Replaced all @functions by @code.

Fixes https://github.com/SamProf/MatBlazor/issues/209.

 This is the new pull request for the same thing.


P461 toHaveBeenCalledWith

On both snippet, there is no need of using toHaveBeenCalledWith, toHaveBeenCalled should be enough.

I want to explicitly say that the spy has been called with no arguments, thus toHaveBeenCalledWith