Enable recycle bin for arbitrary paths

Currently, all recycle bin operations in oCIS are made available via the /remote.php/dav/trash-bin/$username/$key/$path endpoints. These are restricted to only the logged-in user's home, but access for shared spaces is not yet available.
Considering a user case, in CERNBox, we have service accounts associated with project spaces and users having access to a specific project space should be able to list and restore deleted entries. This is supported by the ListRecycleRequest message in CS3APIs, so now, it needs to be added to clients.
Similar to the /remote.php/webdav/$path and /remote.php/dav/$user/$path parallel,
I propose to add a new API /remote.php/webdav/trash-bin/$base-path/$key/$path,
which will allow access to the recycle spin associated with other accounts, given that the user has access to $base-path.
For example, /remote.php/webdav/trash-bin/projects/physics-simulations
Both the implementations can share the same code, as is the case with the old and new files endpoints.
Please let me know your thoughts about this @labkode @butonic @refs @C0rby @dragotin

Regarding a spaces implementation for eos: that can be done incrementally. The easiest step is to list spaces, which is what we started with for the decomposedfs: glob all user homes, stat them and fetch the eos fileid, use it as the spaceid.

Yes, I would be very interested in taking this up for EOS but I'm afraid we don't have the bandwidth at the moment. Once we're done with our rollouts, I'd be happy to get started with this, but it won't be possible for the next 2-3 months at least.


Manual forced feature

add manual forced feature to evaluation process

@vazarkevych Can we discuss about these global forcedFeatures?

Hi, @madhuchavva . Yes, let's setup a call for that. I will write to you.


run-workers output success when before all hook fails. No info in report (Allure)

What are you trying to achieve?
I want to reliably run my tests in multiple workers and have errors correctly reported
What do you get instead?
Even when there is an error the program still exists with success.
Consider the following test:

Feature('setup failure')

BeforeSuite(async () => {
    throw new Error("failed")
})

Scenario('test something', async (I) => {
    I.say(`Test was fine.`)
});

Scenario('test something 2', async (I) => {
    I.say(`Test was fine.`)
});

Scenario('test something 3', async (I) => {
    I.say(`Test was fine.`)
});

Scenario('test something 4', async (I) => {
    I.say(`Test was fine.`)
});

When you run the test with:
codeceptjs run-workers 1 --verbose

You get:

Notice that this is marked as a success, even though nothing useful ran.
When you run with
codeceptjs run --verbose

You get

*Allure does not report on setup failures :/ *
When you run with
codeceptjs run-multiple --all --verbose"

You get

Details

CodeceptJS version: 2.5.0
NodeJS Version: 12.X
Operating System: MAC OS
puppeteer
Configuration file:

const { setHeadlessWhen } = require('@codeceptjs/configure');

// turn on headless mode when running with HEADLESS=true environment variable
// HEADLESS=true npx codecept run
setHeadlessWhen(process.env.HEADLESS);

exports.config = {
  tests: './*_test.js',
  output: './output',
  helpers: {
    Puppeteer: {
      url: 'http://localhost',
      show: true
    }
  },
  include: {
    I: './steps_file.js'
  },
  bootstrap: null,
  mocha: {},
  name: 'codecepttest',
  plugins: {
    retryFailedStep: {
      enabled: true
    },
    screenshotOnFail: {
      enabled: true
    },
    "allure": {
      "enabled": true
    }
  },
  multiple: {
    parallel: {
      // Splits tests into 2 chunks
      chunks: 2
    }
  }
}

What makes this even worse for workers is if a suite setup passes on 1 thread but fails in another, you get success status but only 2 out of 4 of your tests might have actually passed

This is allure specific, disabling allure fixed the issue. I've found the cause will raise a pr


pause/resume commands supports

The killer feature of pgbouncer is pause/resume commands to restart postgres seamlessly for clients. There are similar issue and PR for odyssey, but seems to be abandoned.
It would be nice to implement these commands in pgcat.

Done :)

Amazing! :)


 We're porting them over to helm w/ terraform module. This is the work for cloud-manager-agent.
It's a pretty standard chart.
A statefulset. A headless service. Some config maps for the app.

Still to do:

[ ] top main.tf changes
[ ] test
[ ] Remove some incorrect wording
[ ] Generate readme

Still trying to get this working. Talking to Joey, we decided to add ExternalSecret support. That component is deprecated; I've filled tickets to upgrade to the External Secrets Operator.
https://github.com/streamnative/terraform-google-cloud/issues/5
https://github.com/streamnative/terraform-aws-cloud/issues/62
Trying to get this working on a test cluster with this values file:
appConfig:
  applicationAgentProperties:
    kubernetesExternalSecret:
      enabled: true

Having some issues. Just recording this all for next week.

This code works as-is. There is some discussion one whether we should use the secret raw, have cloud-api-server inject the secret, or use KSE/ESO. This PR could go in before that decision is made.

Forgot to link to the relevant GH issues. https://github.com/streamnative/cloud-ops/issues/245
https://github.com/streamnative/cloud-api-server/issues/900


Allow to set inline deps

Please provide the environment you discovered this bug in.
Angular Vitest with NX and Spectator
Which area/package is the issue in?
vitest-angular
Description
Please allow user to add extra deps to be inlined here:
https://github.com/analogjs/analog/blob/d9556981b82e97154eb0df9ac5fe4840b95c0890/packages/vite-plugin-angular/src/lib/angular-vitest-plugin.ts#L27
https://github.com/ngneat/spectator?tab=readme-ov-file#vitest-support
When I add Spectator specific rules then test with Material Harnesses are stopping working. I discovered that when I'm adding extra rules those provided by plugin are lost.
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

The inline config should be merged with your provided config in the test object. If you add a debug plugin and look at the configResolved hook, you should see your entry also
plugins: [
// other plugins
{
  name: 'debug',
  configResolved(config) {
    console.log(config.test.server.inline);
  }
}


package.json: "name" is not a string

Just installed the latest version via npm and saw the below failure
irena:tmp scanf$ start-imba 2930

🎮 Start Imba v0.1.0

? Bundler to use ? webpack
? Install the project ? Yes
? Initialize git ? Yes

🎮 Start Imba v0.1.0

📁 Creating project in /Users/scanf/src/tmp/2930.
🗃 Initializing git repository...
🛠 Installing packages. This might take a while...

yarn install v1.16.0
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
error package.json: "name" is not a string

/Users/scanf/.nvm/versions/node/v10.5.0/lib/node_modules/start-imba/node_modules/gluegun/build/index.js:13
    throw up;
    ^
command failed: yarn

I think this problem only occurs if the project name is a number.

I would try to fix this, but just want to know, is it works fine for you?

Yes, for projects which are not numeric it works fine :) I do create project names from time to time with just numbers so would be nice if that also works. Thanks.

Fixed.


Cannot mock config.

I'm following your example exactly (I think!) and getting the following when trying to run a jasmine-based test:
functions.config() is not available. Please use the latest version of the Firebase CLI to deploy this function.

It appears to me that the init method in firebase-functions/lib/config.js is looking for either runtimeconfig.json or config.json in very particular places.  I was able to get mine running by putting a config.json into the functions directory of my project.
See try-catch statements inside init:
function init(credential) {     var loaded;     try {         loaded = require('../../../.runtimeconfig.json');     }     catch (e) {         try {             loaded = require('../../../config.json');         }         catch (e) {             throw new Error('functions.config() is not available. ' +                 'Please use the latest version of the Firebase CLI to deploy this function.');         }     }     if (!_.has(loaded, 'firebase')) {         throw new Error('Firebase config variables are missing.');     }     _.set(loaded, 'firebase.credential', credential);     config.singleton = loaded; }

Which example are you referring to specifically? Have you tried mocking functions.config() like this: https://github.com/firebase/functions-samples/blob/master/quickstarts/uppercase/functions/test/test.js#L47-L55

Yes, that's the one.  I found it here tho: https://firebase.google.com/docs/functions/unit-testing

Did you ensure that you mocked functions.config() BEFORE requiring index.js?
Since the sample code calls functions.config() at a global scope, that call will get made as soon as the file is required. So the mocking needs to happen prior to the require statement.

Yes, I'm certain I did that.

Would you be able to copy and paste your entire test file here? It's hard for me to troubleshoot without seeing the entire thing. Thanks!

I'm getting the same message but I am not running tests. I am just trying to run my function.
Here is the stack trace:
/usr/local/bin/node /Users/.../functions/index.js
/Users/.../functions/node_modules/firebase-functions/lib/config.js:46
throw new Error('functions.config() is not available. ' +
^
Error: functions.config() is not available. Please use the latest version of the Firebase CLI to deploy this function.
at init (/Users/.../functions/node_modules/firebase-functions/lib/config.js:46:19)
at Object.config (/Users/.../functions/node_modules/firebase-functions/lib/config.js:29:9)
at Object. (/Users/.../functions/index.js:3:31)
at Module._compile (module.js:571:32)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
at Module.runMain (module.js:605:10)
at run (bootstrap_node.js:425:7)
Process finished with exit code 1

Hi @m-gant, thanks for chiming in. This is expected behavior. You're not supposed to be able to simply "run your functions" locally without mocking the config. functions.config() is only available within the Cloud Functions runtime.

@laurenzlong Oh, ok that actually makes total sense, but is there a way to test the code and make sure it works before actually deploying it. I have never used JavaScript before and wanted to make sure things were correct before I deployed them. Also, I don't know if this issue is really just due to my own novice and I don't know the scope of your role within Firebase, but I think that this is something that would helpful to include in the documentation.

Hey @m-gant this is probably what you're looking for: https://firebase.google.com/docs/functions/unit-testing. And don't worry at all about scopes and responsibilities when you're interacting with any members of the Firebase team, we work closely together and will route bugs/feature requests to the appropriate component owner. I did incidentally write the documentation on unit testing, so I welcome to any feedback you might have.  If you have any other questions about using Cloud Functions for Firebase, the best thing to do would be to post on Stack Overflow and tag it with 'firebase' and 'google cloud functions', we monitor it closely, and it's a better forum for asking usage questions than Github issues, which are reserved for bugs and feature requests.


✨ Add woodpecker as known CI

What kind of change does this PR introduce?
It addresses #4210

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
It does not count tests run by Woodpecker
e.g. CI-Tests score of woodpecker's own main repo is 8/10
What is the new behavior (if this is a feature change)?**
it detects woodpecker and score the tests acordingly
e.g. CI-Tests score of woodpecker's own main repo is 10/10
scorecard --repo=https://github.com/woodpecker-ci/woodpecker :

Which issue(s) this PR fixes
Fixes #4210
Special notes for your reviewer
Does this PR introduce a user-facing change?
no

/scdiff generate CI-Tests

Done


WP: Switch to AudioVideoCaptureDevice for Camera Reader

Right now PhotoCamera has some drawbacks:

It won't turn on the flash unless capturing an image (so not for preview buffers)
Preview resolution is not selectable

AudioVideoCaptureDevice seems to allow these things.  Need to investigate implementing this.

Hey guys, first of all, thank you for the amazing tool you are providing!
I have recently made an winphone8 app for the company I work for, which uses the Zxing scanner to scan QR codes.
I have the exact same problem, where the torch cant be turned on. Can you maybe give a hint when to expect this solved?

I'm just curious, is anyone currently working on this?

Wondering the same thing...

I'm using this library for a project and have hit the WP flash limitation. I would gladly jump into it and update the PhotoCamera code to use AudioVideoCaptureDevice instead as I have previous experience on camera capture APIs on WP8.
Any guidelines on how should I proceed with this? Should I still keep the PhotoCamera version as a fallback?

Thanks to the wonderful contribution, this is done!


Minor updates to Excel tutorials

Also fixed a few formatting issues and GitHub links in other articles.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/design/design-icons.md
:white_check_mark:Succeeded

docs/develop/add-in-manifests.md
:white_check_mark:Succeeded

docs/develop/add-ins-with-angular2.md
:white_check_mark:Succeeded

docs/develop/create-addin-commands.md
:white_check_mark:Succeeded

docs/develop/localization.md
:white_check_mark:Succeeded

docs/includes/file-get-started-powerpoint.md
:white_check_mark:Succeeded

docs/includes/file-get-started-project.md
:white_check_mark:Succeeded

docs/powerpoint/office-mix/get-started-with-labsjs-for-office-mix.md
:white_check_mark:Succeeded

docs/testing/troubleshoot-manifest.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-chart.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-create-table.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-filter-and-sort-table.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-freeze-header.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial-protect-worksheet.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


viz yaml fix

@jiwalker-usgs Went back to a viz yaml before the merge and copied it

sigh, maybe not the fix


If a structure with double bond(s) is the first structure created on the canvas the double bond is rendered incorrectly

Affected versions: remote, standalone
Tested on Ketcher v2.2.1-104-gc7896113
Steps to reproduce:

Launch Ketcher.
Use 'Templates' to create benzene on the canvas.
Click any bond of the created structure.

Expected result: All the bonds of the created structure are rendered correctly.

Actual result: One of the double bonds is not rendered as expected.

It could be reproduced with two connected double bonds:

also applicable to the aromatic bond:
actual result:

expected result:


Timer intergation: force_update added

Breaking change

It is not a breaking change as far as I understand.
Proposed change

As per this discussion we need to update last_changed when active timer restarted.
One way to do that is to force HA update the state on each request even if it remains the same.
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue: #30062
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

Please add a test.

@balloob could you point me to the description how to do that - I've never done it.
or tell me who to ask at discord if that's easier ;)

IMO you could tap into https://github.com/home-assistant/home-assistant/blob/e3894d212c8d7b2ab36e33b4911460352625e64a/tests/components/timer/test_init.py#L342-L406
Define another fakelistener and listen for STATE_CHANGED events? Then every time a timer service is called, state_changed listener should get a new result?

Thanks Alexei, will try my best in finding out how the tests work.

you also may want to install pre-commit for git, so it would take care of formatting and sorting imports
see https://developers.home-assistant.io/blog/2019/07/31/black.html

bump, will deal with it soon


Improve MultiOn client toolkit prompts

Updated prompts for the MultiOn toolkit for better functionality
Non-blocking but good to have it merged to improve the overall performance for the toolkit

@hinthornw @hwchase17

thanks @Div99!


change node version from 8 to 10.

According to github readme.md, change website node version from 8 to 10.

good job
@break60 please review this PR if you free

Should be changed to 8.X-10.x, the old version currently supports up to 10.X version

@break60 emm, sorry I can't understand what you mean. Do you mean I should change the word in my PR's  from "10.x" to "8.X-10.x" ?

@break60 emm, sorry I can't understand what you mean. Do you mean I should change the word in my PR from "10.x" to "8.X-10.x" ?

sorry, I see it as the front-end document of ds


SBOM seems to contain useless hashes

SBOM appears to contain SHA1 hashes, which are deprecated, should be removed and no longer generated or verified.
This will be required for 2030+ FIPS compliance too.
Separately SHA512 hashes are generated, but they too are mostly harmless as certifications require 128 bits hashes only for 2030+, as in SHA256.

Which is required by current specs.


Autosnapshotting service doesn't remove snapshots after the retention period

Steps:
1- Create a bp which consume autosnapshotting service.
Example:
`

g8client__islam:
url: 'du-conv-2.demo.greenitglobe.com'
login: 'driver'
password: 'KrOe6gE9K5nCQdmretfXnj'
vdc__3505bd038b:
description: 'c5ef176e29'
g8client: 'islam'
account: '907120a8fa'
location: 'du-conv-2'
uservdc:
- 'driver'
node.ovc__191620989c:
os.image: 'Ubuntu 16.04 x64'
bootdisk.size: 10
sizeID: 2
vdc: 3505bd038b
autosnapshotting__4f01089df4:
vdc: 3505bd038b
snapshotInterval: '1m'
retention: '20m'
actions:

action: 'install'
`

2- Go to the enviroment portal and open this VM page
3- Make sure that each 1m there is a new snapshot will be created
4- Check that after 20m the first snapshot is removed
Expected result

snapshot which was created since 24 mins is still existent however the retention period is '20m'
cockpit version
JumpScale
Core: 8.1.0rc2-4-g776daea
Portal: 8.1.0rc2-1-gb748c00

JSCockpit
JSCockpit: 8.1.0rc2

Env Version
JumpScale
Core: branch: master (59f2815) 12/17/2016, 2:13:19 PM
Portal: branch: master (872e5e8) 12/14/2016, 3:06:42 PM

OpenvCloud
Core: branch: master (d3e4680) 12/20/2016, 4:54:31 PM
G8VDC: branch: 2.1 (4601f9b) 11/6/2016, 3:02:48 PM
Selfhealing: branch: master (6deb41e) 12/19/2016, 3:15:01 PM

OpenvStorage:
openvstorage-backend-core: 1.7.8-fargo.3-1
openvstorage-health-check: 3.1.3-fargo.1-1
alba: 1.3.0
openvstorage-webapps: 2.7.8-fargo.2-1
openvstorage-backend-webapps: 1.7.8-fargo.3-1
openvstorage-core: 2.7.8-fargo.2-1
openvstorage-hc: 1.7.8-fargo.3-1
openvstorage: 2.7.8-fargo.2-1
openvstorage-backend: 1.7.8-fargo.3-1
openvstorage-sdm: 1.6.8-fargo.1-1

According to discussion with @khaledkbadr  and @xmonader we found that user should set CLEANITERATION to delete the snapshots.


Bug when updating

Doing the update procedure causes this error:

This may be related to switching back to the original gulp-electron.

The issue only appeared when updating to and from 0.12.1. It is related to a folder from gulp-electron that should not have been added to the distribution.


[ERROR] Graph Must Contain Start Vertex

Hello, I have a problem pretty much similar to the other issue raised previously, where while running the last command, it should give me the .csv with the results, but it throws the following error, apparently the error indicates that the graph must contain the start vertex. I have tried with multiple projects but still getting the same message each time.

Hi @anddann.
Thank you for your response. I have attached the dependency graph and the pom.xml files for your reference.  The project is  not multi-module  and I was able to run the "mvn clean compile install" successfully. Thank you.
pom.zip
dependency-graph.json

Okay, the graph contains the project node. After running mvn install, does UpCy work?
I've not tested UpCy with war-files, however, it should rune fine.

I tried running with the "-module ./" but getting the same output and no .csv file created.

Okay, I'll try to reproduce the error with the given pom.xml

Thank you.
One more question, I tried running UPCY on some projects from the experimental-results--dataset folder, the tool is running fine however its always showing "No solution found in Neo4j" as attached below.  what is the reason for that?

Actually, in the given dataset it is not always showing no solution found.

for some projects the tool does not find a solution in Neo4j
in multiple case UpCy tries multiple options and some of them do not yield a result, then it shows no solution found and then continues to search..

Your Neo4j database maybe empty? outdated or invalid?

@yasmin259 I've debugged your project.
The issue is
<dependency>
			<groupId>javax.servlet</groupId>
			<artifactId>javax.servlet-api</artifactId>
			<version>3.1.0</version>
			<scope>provided</scope>
</dependency>

in particular the <scope>provided</scope> as written in the paper, UpCy only checks for compile dependencies.
Since  "[provided] indicates you expect the JDK or a container to provide the dependency at runtime"
https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html, we excluded those as the dependency that is used depends on the runtime environment and may be updated/exchanged there.
However, the error message is misleading. I'll improve the message.

I also tried the tool with other projects and its working fine now. Thank you very much @anddann.

I've improved the error message in #17


Add option to open a workspace in new window by default

This is an enhancement/feature request but what would help me greatly is if on clicking a workspace, instead of opening it in the same window, it opens in a new window by default.
I understand not everybody likes this option so it's best to have it as an option and disabled by default.

Config option has been added and will be in the next release

Config option has been added and will be in the next release

Thank you 😃


Crash in etorrent [no function clause matching {lager_trunc_io,print,...]

2012-01-07 16:57:02.726 [error] <0.32094.0> CRASH REPORT Process lager_crash_log with 0 neighbours crashed with reason: no function clause matching {lager_trunc_io,print,[<<128,0,0,0,4,128,0,2,0,1,1,4,174,84,16,0,0,0,4,0,0,0,128,0,0,0,0,0,0,0,0,133,0,1,33,4,0,96,0,0,2,18,64,1,55,8,1,0,12,3,8,208,230,48,0,0,0,0,0,0,100,32,16,0,12,80,0,0,0,0,0,0,2,0,196,32,2,18,0,96,0,16,0,1,16,64,0,32,0,0,89,213,32,192,2,0,0,0,1,0,0,4,0,0,0,128,0,33,64,64,17,39,4,0,0,8,0,0,8,128,0,1,176,0,26,96,64,40,128,128,0,128,4,0,128,128,16,77,96,0,0,0,2,1:7>>,65491,{print_options,-1,true,false}],[{file,[115,114,99,47,108,97,103,101,114,95,116,114,117,110,99,95,105,111,46,101,114,108]},{line,102}]}

Apparently I don't handle bitstrings (as opposed to binaries) at all, working on a fix.

I fixed this!


Change lines in Order type to graphene.List

I want to merge this change because...
closes #2900
Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.

Thanks for picking this up!
@dominik-zeglen do we need to upgrade the react part as well?

Definitely, but it should be easy to fix

Thanks for picking this up! I've merged this branch into https://github.com/mirumee/saleor/pull/3009, where we will fix the Schema, Types and ReactJS part


Updating SDK to incorporate new one-of schema

Changes introduced with this PR
Updating SDK to incorporate arcalot/arcaflow-plugin-sdk-python#72
By contributing to this repository, I agree to the contribution guidelines.

@janosdebugs What do we need to do to get this one moved forward? I've been running quite a few successful workflow tests using the development build from this PR.

This PR is stale.


[python/r] Run cellxgene-census acceptance tests pre 1.6.0

Needs:

Core 2.18.0 available
TileDB-Py 0.24
TileDB-R
A commit hash of main of this repo

to @bkmartinjr run acceptance tests before we tag 1.6.0

@pablo-gar to run R acceptances IIUC

Our python tests ran and all looks good

@bkmartinjr great news! Hoping the R tests do the same

Verified by @ebezzi and @pablo-gar


[Android] BLUETOOTH permission issue

This plugin is add the BLUETOOTH permission in the AndroidManifest.xml.
        <config-file target='AndroidManifest.xml' parent='/manifest'>
            <uses-permission android:name="android.permission.BLUETOOTH" android:maxSdkVersion="18" />
        </config-file>

But when we actually use Bluetooth with the cordova-ble-central-plugin for example, this permission is added twice in the AndroidManifest.xml because of the android:maxSdkVersion="18" which leads to the compilation to fail.
Removing android:maxSdkVersion="18" solved the issue. Would you like me to issue a PR for that?

interesting.
what would you do to solve this?

As I said:

Removing android:maxSdkVersion="18" solved the issue.

I know but there's a reason why maxSdkVersion is there.
Let me think about it...

Good catch. That's exactly the problem we are having.

Would you be able to test the master branch please?
I added an after_prepare hook.

So I tried this out. The after_prepare hook worked with a little modification for Cordova 7.x
The Android Manifest file that is produced now is in platforms/android/app/src/main/AndroidManifest.xml
The hook I ended up with was
#!/usr/bin/env node

module.exports = function(context) {

    var fs = context.requireCordovaModule('fs');

    var path = context.requireCordovaModule('path');

    var platformRoot = path.join(context.opts.projectRoot, 'platforms/android');

    var preCordova7ManifestFile = path.join(platformRoot, 'AndroidManifest.xml');

    var cordova7ManifestFile = path.join(platformRoot, 'app/src/main/AndroidManifest.xml');

    var manifestEdit = function(file) {
      fs.readFile(file, 'utf8', function (err,data) {
        if (err) {
            throw new Error('Unable to read AndroidManifest.xml: ' + err);
        }
        var permissions = data.match(/<uses-permission(.*?)\/>/g);
        var bluetoothPermissions = permissions.filter(permission => permission.toUpperCase().indexOf('ANDROID.PERMISSION.BLUETOOTH') > -1);
        if (bluetoothPermissions.length > 1) {
            var permissionWithoutMaxSDK = bluetoothPermissions.find(permission => permission.indexOf('android:maxSdkVersion') == -1);
            if (permissionWithoutMaxSDK) {
                var result = data.replace('<uses-permission android:maxSdkVersion="18" android:name="android.permission.BLUETOOTH" />', '');
                fs.writeFile(file, result, 'utf8', function (err) {
                    if (err) {
                        throw new Error('Unable to write into AndroidManifest.xml: ' + err);
                    }
                });
            }
        }
      });
    }

    if (fs.existsSync(preCordova7ManifestFile)) {
      manifestEdit(preCordova7ManifestFile)
    } else if (fs.existsSync(cordova7ManifestFile)) {
      manifestEdit(cordova7ManifestFile)
    }
};

i tried out master branch with NikoRoberts solutions and it works fine. Without the modifiaction of NikoRoberts it doesn't work for Cordova 7.x

I updated the master branch.
Can someone test the changes?
If everything's fine I'll do a release.
Thanks

becvert i tested it and it works fine

1.3.3 is out. closing now.


Add webvitals tests to skip if browser is Safari or Firefox

Firefox and Safari currently fail several web vitals browser integration tests. This is because Firefox and Safari do not natively support web vitals.To make CI workflow run without having this error we need to skip these tests if the browser is Safari or Firefox.
1.This change will make WebVitalsPlugin.test.ts to skip if browser is safari or firefox.
2.This change will make CI workflow to run integ tests in firefox and edge.
Manual Testing is done to check whether webvital test is skipped if the browser is safari or firefox.

Duplicate of: https://github.com/aws-observability/aws-rum-web/pull/54


[HANDS-ON BUG]

Describe the bug
I'm having this exact same bug.
I used all the provided solutions (with my hugging face account), I created the yml file and the logs folder was created also. Could this be a problem with a package version?
I saw this error that seems related.
Thanks for the help!
Material

Did you use Google Colab?
yes

Here is my notebook: https://colab.research.google.com/drive/1QCCXRfrapO-ntjJb-HrUoP4BFktvo6cS?usp=sharing

I closed this issue since is the same as this one.


org.apache.commons.codec.binary.Base64 clash

The class "org.apache.commons.codec.binary.Base64" in parquet-tools.jar version 1.9.0 is same at package and class name to which is in commons-codec.jar(all version) ,but its implement is much older.
That makes a jar-hell when both parquet-tools.jar and commons-codec.jar are used .
e.g.
Exception in thread "main" java.lang.NoSuchMethodError: org.apache.commons.codec.binary.Base64.encodeBase64String([B)Ljava/lang/String;
Reporter: JaredLi
Related issues:

org.apache.commons.codec.binary.Base64 clash #449 (is duplicated by)
Remove Dependency on commons-codec (is superceded by)

Note: This issue was originally created as PARQUET-1194. Please see the migration documentation for further details.

JaredLi:
Do u have any progress？


FLP-292 earnings changes for adapter tool

I can't approve a PR I raised but happy with Rus's changes here ✔


Feature request: Not having to use an account.

Some might be put off by having to create an account (including me, at first). It would be nice if there would be some way to skip account creation.
Feature Description
It would be great if you'd only be required to have an account if you would like synchronization and Pro features, but not forcing people to create one - even for the free version. This might put people off. After all, every single chat message you send and receive is going through Franz and they want to be sure that the contents of these messages are only read by you and the receiver. Having to create an account to use Franz might raise some suspicion.
Motivation and Context

You should not have to use an account to use an application which does not necessarily require one to use most of its features.
More people will give Franz a shot, maybe even create an account if they really like it.
People being put off by being forced into creating an account.

After all, every single chat message you send and receive is going through Franz and they want to be sure that the contents of these messages are only read by you and the receiver.

This is not true: Franz does not in any way process your messages, these are processed by the services only, same way as in the browser.

@haraldox I know it’s not true, but it’s what someone could think at first glance. Being forced to create an account with first and last name and connecting all kinds of services to that account... it might raise a few eyebrows (and unfortunately rightfully so these days...).


Stop after login procedure

2016-09-03 11:23:44,276 [       cli] [INFO] PokemonGO Bot v1.0
2016-09-03 11:23:44,293 [       cli] [INFO] commit: ca94424f
2016-09-03 11:23:44,299 [       cli] [INFO] Configuration initialized
2016-09-03 11:23:44,299 [pokemongo_bot.health_record.bot_event] [INFO] Health check is enabled. For more information:
2016-09-03 11:23:44,300 [pokemongo_bot.health_record.bot_event] [INFO] https://github.com/PokemonGoF/PokemonGo-Bot/tree/dev#analytics
2016-09-03 11:23:44,306 [requests.packages.urllib3.connectionpool] [INFO] Starting new HTTP connection (1): www.google-analytics.com
[2016-09-03 11:23:44] [MainThread] [SleepSchedule] [INFO] [next_sleep] Next sleep at 17:23:00, for a duration of 3:02:53
[2016-09-03 11:23:44] [MainThread] [PokemonGoBot] [INFO] [set_start_location] Setting start location.
[2016-09-03 11:23:44] [MainThread] [PokemonGoBot] [INFO] [location_found] Location found: London Tower (51.50811239999999, -0.0759493, 0.0)
[2016-09-03 11:23:44] [MainThread] [PokemonGoBot] [INFO] [position_update] Now at (51.50811239999999, -0.0759493, 0.0)
[2016-09-03 11:23:44] [MainThread] [PokemonGoBot] [INFO] [login_started] Login procedure started.
Traceback (most recent call last):
File "pokecli.py", line 795, in
main()
File "pokecli.py", line 231, in main
report_summary(bot)
File "pokecli.py", line 279, in report_summary
metrics.capture_stats()
File "/root/Home/pokemongo_bot/metrics.py", line 108, in capture_stats
response_dict = request.call()
File "/root/Home/pokemongo_bot/api_wrapper.py", line 239, in call
result = self._call()
File "/root/Home/pokemongo_bot/api_wrapper.py", line 188, in _call
return PGoApiRequest.call(self, signature)
File "/root/Home/src/pgoapi/pgoapi/pgoapi.py", line 236, in call
response = request.request(self._api_endpoint, self._req_method_list, self.get_position())
File "/root/Home/src/pgoapi/pgoapi/rpc_api.py", line 127, in request
response_dict = self._parse_main_response(response, subrequests)
File "/root/Home/src/pgoapi/pgoapi/rpc_api.py", line 337, in _parse_main_response
raise ServerSideAccessForbiddenException("Seems your IP Address is banned or something else went badly wrong...")
pgoapi.exceptions.ServerSideAccessForbiddenException: Seems your IP Address is banned or something else went badly wrong...
[2016-09-03 11:23:47] [MainThread] [sentry.errors] [ERROR] Sentry responded with an error: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/usr/local/lib/python2.7/dist-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/usr/local/lib/python2.7/dist-packages/raven/utils/http.py", line 66, in urlopen
return opener.open(url, data, timeout)
File "/usr/local/lib/python2.7/dist-packages/future/backports/urllib/request.py", line 494, in open
response = self._open(req, data)
File "/usr/local/lib/python2.7/dist-packages/future/backports/urllib/request.py", line 512, in _open
'_open', req)
File "/usr/local/lib/python2.7/dist-packages/future/backports/urllib/request.py", line 466, in _call_chain
result = func(*args)
File "/usr/local/lib/python2.7/dist-packages/raven/utils/http.py", line 46, in https_open
return self.do_open(ValidHTTPSConnection, req)
File "/usr/local/lib/python2.7/dist-packages/future/backports/urllib/request.py", line 1284, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/lib/python2.7/httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1097, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 895, in _send_output
msg += message_body
File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte
[2016-09-03 11:23:47] [MainThread] [sentry.errors.uncaught] [ERROR] [u'ServerSideAccessForbiddenException: Seems your IP Address is banned or something else went badly wrong...', u'  File "pokecli.py", line 795, in ', u'  File "pokecli.py", line 231, in main', u'  File "pokecli.py", line 279, in report_summary', u'  File "pokemongo_bot/metrics.py", line 108, in capture_stats', u'  File "pokemongo_bot/api_wrapper.py", line 239, in call', u'  File "pokemongo_bot/api_wrapper.py", line 188, in _call', u'  File "pgoapi/pgoapi.py", line 236, in call', u'  File "pgoapi/rpc_api.py", line 127, in request', u'  File "pgoapi/rpc_api.py", line 337, in _parse_main_response']
Sat Sep 3 11:23:47 UTC 2016 Pokebot  Stopped.
Press any button or wait 20 seconds to continue.

Maybe IP banned, try using the bot from a diff. IP address

I'm definitely not IP banned and have literally just updated and now have this issue as well..

Exact same issue. I can't be ip banned - I still run pokemap without any issues.
Traceback (most recent call last):
File "pokecli.py", line 795, in
main()
File "pokecli.py", line 153, in main
bot.tick()
File "/root/bot2/PokemonGo-Bot/pokemongo_bot/init.py", line 662, in tick
if worker.work() == WorkerResult.RUNNING:
File "/root/bot2/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_pokemon.py", line 44, in work
if self.catch_pokemon(self.pokemon.pop()) == WorkerResult.ERROR:
File "/root/bot2/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_pokemon.py", line 138, in catch_pokemon
return_value = worker.work()
File "/root/bot2/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 174, in work
self._do_catch(pokemon, encounter_id, catch_rate_by_ball, is_vip=is_vip)
File "/root/bot2/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 583, in _do_catch
sleep (randrange(self.catchsim_newtodex_wait_min, self.catchsim_newtodex_wait_max))
File "/usr/lib/python2.7/random.py", line 218, in randrange
raise ValueError, "empty range for randrange() (%d,%d, %d)" % (istart, istop, width)
ValueError: empty range for randrange() (1,1, 0)
[2016-09-03 16:54:49] [MainThread] [sentry.errors] [ERROR] Sentry responded with an error: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
return opener.open(url, data, timeout)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
response = self._open(req, data)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
'_open', req)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
result = func(*args)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
return self.do_open(ValidHTTPSConnection, req)
File "/root/bot/PokemonGo-Bot/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/lib/python2.7/httplib.py", line 1001, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1035, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 997, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 848, in _send_output
msg += message_body
File "/root/bot/PokemonGo-Bot/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte
[2016-09-03 16:54:49] [MainThread] [sentry.errors.uncaught] [ERROR] [u'ValueError: empty range for randrange() (1,1, 0)', u'  File "pokecli.py", line 795, in ', u'  File "pokecli.py", line 153, in main', u'  File "pokemongo_bot/init.py", line 662, in tick', u'  File "pokemongo_bot/cell_workers/catch_pokemon.py", line 44, in work', u'  File "pokemongo_bot/cell_workers/catch_pokemon.py", line 138, in catch_pokemon', u'  File "pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 174, in work', u'  File "pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 583, in _do_catch', u'  File "python2.7/random.py", line 218, in randrange']
@ravey-davey what was your wrong location?

I was getting this when using:
"location": "Location Name",

And now I'm using:
"location": "00.000000,00.000000", # long, lat

Use the issue template please.


Consider refactor with TypeScript?

const enum TokenMember {
    TYPE = 0,
    VALUE,
    ...
}

token[TokenMember.TYPE]

It's much better than such as token[0], token[1].

Thanks for letting us know it would be valuable to make this package more TypeScript friendly.
Although I won't promise if it will be converted to TypeScript or expose TypeScript type definitions out the box later, I can say that it's a natural discussion to have as soon as currently open PR for transitioning to an ES module gets merged: https://github.com/janl/mustache.js/pull/728

Thought you might be interested in knowing that it's planned to keep @types/mustache from DefinitelyTyped as up-to-date as possible going forward. Just added a section referencing that in our README: https://github.com/janl/mustache.js/commit/3bdd27c4e762f665232dc46186c7639e60dbd70e.
You're of course more than welcome to contribute with improvements to those type definitions.

Ok, thanks reply.


Automated Rebalancing

Once we are supporting multiple lending platforms and AMMs, we will want to regularly rebalance the pool to optimize for the best [risk-adjusted] yield-bearing opportunities. The two big questions to answer here are:

How should the contract determine the desired allocation (considering diversification, etc)?
What should trigger the rebalancing (on every mint/burn, for example)?

What other parameters would we want to have here?

Each strategy should get a maximum allocation so that we maintain some diversification.
Each strategy should get a minimum allocation so that it continues to be in use.
A lower threshold than our current auto-allocate might be necessary for this to be triggered on many mints or redeems.
A profitability threshold should be included so that we're not unnecessarily reallocating in pursuit of a nominal increase in yield.

What other parameters would we want to have here?

Each strategy should get a maximum allocation so that we maintain some diversification.
Each strategy should get a minimum allocation so that it continues to be in use.
A lower threshold than our current auto-allocate might be necessary for this to be triggered on many mints or redeems.
A profitability threshold should be included so that we're not unnecessarily reallocating in pursuit of a nominal increase in yield.

Complications here are that:

Yield can change very rapidly. In the last week Compound USDT yield has more doubled or halved many times.
We want to consider risk. If a Compound pool is offering max APY, there might be a reason for it. It means there is no liquidity, and there's a definitely chance that something has gone wrong somewhere, and any money put in could very well never come back.

When reallocating between only "safe" strategies that, it could be fine to have these called by the public.
Perhaps rather than putting all this gas expense into the mints/redeems, we could have mints and redeems put some money into a gas coin that can later be spent by the allocation calls.

Complications here are that:

Yield can change very rapidly. In the last week Compound USDT yield has more doubled or halved many times.
We want to consider risk. If a Compound pool is offering max APY, there might be a reason for it. It means there is no liquidity, and there's a definitely chance that something has gone wrong somewhere, and any money put in could very well never come back.

When reallocating between only "safe" strategies that, it could be fine to have these called by the public.
Perhaps rather than putting all this gas expense into the mints/redeems, we could have mints and redeems put some money into a gas coin that can later be spent by the allocation calls.

Replacing this with https://github.com/OriginProtocol/origin-dollar/issues/1308 ♻


making pixelscale of direct have length/pix units

This PR also forces several .value calls for values with units to first convert to the base unit (meters in these cases) before taking the value.

Hm, trying to make sure I understand the pitfalls of peeling the units off Astropy Quantity objects... should I go grepping through the source to find any .value that's not preceded by a .to(u.Thing)?

(Well, any user-supplied quantity...)

try:
radius=6*u.imperial.furlong
radius.value

I just went through fresnel and didn't see any other cases except for in coordinates(), but there it is forced to specific units in a preceding line.

Okay, yep, that's what I thought would happen. Good to know about that particular footgun.

Coverage remained the same at 64.201% when pulling 95ef404e1e9394859688e734fbd2fdb129f06e7d on douglase:fix_direct_pixscl into 0715741a5b161d8f243ea8068777a1f2d633a03c on mperrin:master.

alternatively, some of the .value calls could probably keep their units. I don't recall why the .value is being taken for the list of waists, but at least this way the behavior is consistent and won't break anything.

Yeah, for the list of waists is that actually used for anything? I thought that was mostly just kept as a sort of log of the optical propagation? It seems like that might be best to keep as a Quantity rather than trimming to just a value. I don't see the benefit in doing that, whereas keeping the units explicitly there seems like it could be useful.

Yeah, it was for checking against Gaussian laser waist propagation codes,
units could be helpful.
On Wed, Oct 5, 2016 at 4:03 PM, Marshall Perrin<EMAIL_ADDRESS>wrote:

Yeah, for the list of waists is that actually used for anything? I thought
that was mostly just kept as a sort of log of the optical propagation? It
seems like that might be best to keep as a Quantity rather than trimming to
just a value. I don't see the benefit in doing that, whereas keeping the
units explicitly there seems like it could be useful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mperrin/poppy/pull/196#issuecomment-251783121, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-nn_v6Cf3A4sV44IC6J6yYMp6pOjSMks5qxAKMgaJpZM4KPNYU
.

--
Postdoctoral Research Associate, MIT AeroAstro.
<EMAIL_ADDRESS>617-942-1791

@douglase this revised propagate_direct doesn't seem to be working for me. I tried testing it, but the units don't come out properly in the end. I get a complex wavefront that's a Quantity with units of m/pix^2, and that's no good. Breaks all sorts of stuff since the code expects that to be dimensionless.  Is this version of the propagate_direct working for you?
Here's a small test case which fails for me:
wf = poppy.FresnelWavefront(
    5 * u.um,
    wavelength=10 * u.nm,
    npix=1024,
    oversample=4)
wf *= poppy.CircularAperture(radius=800 * u.nm)
wf.propagate_direct(12 * u.um)
wf.display()

I think I've figured out the necessary fixes which I'll comment in the code review.

So, I found the problem with the short distance propagation, a scaling constant in the ptp function was wrong. Odd that it gave such good agreement at long distances, shows that test line-by-line test coverage is not the same as testing physical extremes.
This latest update addresses #194, as demonstrated by an additional test within test_Circular_Aperture_PTP(). It also addresses the comments above about display() not working.
The Hubble propagation test by @mperrin is now failing, the Airy function disagrees with the Fresnel diffraction now by closer to 5%.
But I'm not sure this is a problem, the HST Fresnel diffraction pattern agrees well with the Fraunhofer pattern for the same aperture (the source code for this figure in the demo notebook in this branch, https://github.com/douglase/poppy/blob/e8c2a90aeec81fbe2f3b205850c64860e5c6bcfd/notebooks/Fresnel_Propagation_Demo.ipynb):

Is there some reason to believe the diffractive HST core is actually as close to an Airy pattern as the test found previously, or was that just a coincidence?

Actually it looks like when I wrote that test originally, I forgot to set the obscuration parameter for the size of the secondary mirror in the call to airy_2d. I expect if you were to fix that the agreement would be much better. The presence of the obscuration redistributes more and less light in the different rings of the Airy function and I'm sure that's the reason for more intensity in the first Airy ring.
Glad to hear you found the bug in the scaling factor! Yeah, I agree it's a bit surprising the code was working as well as it did before with that wrong. You make a very good point about testing at extreme cases motivated by the physics.

I took a closer look at this again today and my above comment isn't the answer. There's a difference between the notebook and the unit test versions of the HST model. The version in the notebook does have the SM obscuration and SM supports. The version in the unit test has an unobscured aperture (despite having an SM in the optical path - so the simulation is physically inconsistent: the SM doesn't block any light on the way in but does reflect it after it bounces off the primary. Nonphysical but thinking back I'm sure I wrote the test intentionally that way so the comparison with the unobscured Airy function was more exact.
To answer your above question "is there some reason to believe the diffractive HST core is actually as close to an Airy pattern as the test found previously?" in this case yes. Unobscured aperture at the focus so the far-field Fresnel should asymptotically approach the Fraunhofer result, and the analytic Airy function as well.
I wonder if the issues is not fine enough pixel sampling?  Hmm, I just tried a calculation with beam_ratio=0.125 so in effect twice as fine sampling in the Fresnel calculation. But the test still doesn't pass.  So I'm still unsure what's going on.
beam ratio = 0.25; 4x oversampling:

beam ratio = 0.125; 8x oversamplingL

could aliasing from the apertures be filling in the dark rings?

could aliasing from the apertures be filling in the dark rings?

Maybe?   I just worked a bit on the long-postponed subpixel_geometry branch, to bring it up to date with master. But some tests are still failing on that, including the Fresnel PTP one now.  I'm out of time to work on this today, need to switch to other tasks, but if you have time you can experiment with what's on that branch.

Ha, would you believe it's just pure defocus error? The value for d_sec_to_focus leaves the final focal plane off by 330 microns from the beam waist.
With d_sec_to_focus = 6.3916645 * u.m instead, so the focus is precisely at the beam waist, the discrepancy vanishes.
Here's the result of a calculation done with that value and beam ratio=0.125 (8x oversampling). Great agreement. Note, this is for a case where I removed the SM obscuration and struts from the notebook calc for simplicity.

I've just checked and with the d_sec_to_focus value changed to the above, the test passes just fine. No other changes needed. Storing the list of beam waists has just paid off! That was essential for debugging this.
One other issue, but a way more minor effect, is the difference in PSF alignment between the Fresnel and Fraunhofer codes. (PSF center is at e.g. 2047.5 pixels in Fraunhofer, but at 2048 pixels in Fresnel). The radial_profile function isn't smart enough to figure that out on its own so you have to pass it in as a parameter. But that's a far less significant effect than the defocus. It matters a tiny bit for making the radial profile plots look nice, but doesn't affect the unit test since that comparison is done on the array pixel values directly instead of making profiles.

nice!

I'm going to go ahead and merge this then fix the propagation distance.


Error 'document is not defined' when using vue ssr

This plugin doesn't support vue-ssr. In /vue-preview/src/plugins/preview/index.js, there is 'document' object making server side render a sad failure.

ssr Cannot find module 'vue-preview'


Advice on handing authentication

Hi,
Firstly thank you for creating this library, it's great!
I need some help/advice on handling authentication with tus. In this issue you refer to httpContext.User.Identity.IsAuthenticated which is what I'm wanting to use however I'm not entirely sure how this is ever set to true. I'm using https://uppy.io/ on the client side (I can use the tus js client if uppy doesn't handle it) but I can't see how authentication is sent from the client to the server.
Edit: Just to add - my client is talking to an api so it would need to send a bearer token. I know signalR sends this bearer token as part of the query string so I could do that if need be
Maybe I should ask this in the Uppy issues but thought I'd start here as you might have a solution linked to asp core
Thanks!

Set the bearer token in the header and make sure middleware is in the correct order and it works!
Thanks again!


Rofi launcher paths

Hi, I have a question, I am creating my own configuration and want to inspire (copy) from yours. I am wondering if ROFI launcher path have "custom" delibaratly, or is this something that I need to fix in my config to provide proper paths?

@tomaszbawor

Hi, I have a question, I am creating my own configuration and want to inspire (copy) from yours. I am wondering if ROFI launcher path have "custom" delibaratly, or is this something that I need to fix in my config to provide proper paths?

Sorry about the wait i have not been able to be online as much recently. As for your question I didn't do anything deliberately with ROFI but how it works is i have a hyprland keybind that launches rofi with the theme explicitly passed like this:
$launcher = pkill rofi || rofi -show drun -modi drun,filebrowser,run,window -theme ~/.config/rofi/launchers/type-2/style-2.rasi
bind = $mainMod, A, exec, pkill -x rofi || $launcher


service account support

I'm looking to update my remote service workers to use a Mixpanel service account
https://developer.mixpanel.com/reference/authentication
it does not look like this library supports such yet; can that be added?
i.e. to send a username and password via HTTP Basic Auth
thank you

For purposes of normal event-tracking (i.e. via Mixpanel's /track API endpoint), the publishable Project Token is still the only auth method.
The /import endpoint does support service accounts now, and we'll look into getting that added to this SDK, but just know that it won't be sufficient for now if you're trying to call mixpanel.track().

Ran into this when trying to use /import api https://developer.mixpanel.com/reference/import-events#authentication
https://developer.mixpanel.com/reference/import-events#authentication

/import requires an Owner or Admin Service Account. project_id, service account username and service account password are required to authenticate a request.


Dont send message on loop.

Can you make it not send the now playing message if the song is on loop of from a playlist?
Thanks so much!

better should make it so that the message gets deleted on song completing

#156 this should do it ig

ig can add property in env file like pruning to turn on and off if weather want that message to get deleted but that aint my concern


Reordered the flow

Status
Ready
Description
Per some questions and mistakes, we see in the process ordered the flow a bit

@fvigo Until we'll make the changes we talked about can you review these small editions?


SQLiteException: no such table:

Hello I have been encountering this issue both on the emulator and on my physical Nexus 5. My code works as follow:

the fragment calls this line in onCreateView if(FavoriteUtils.checkIfThisIsFavorite(mShow))

then this method gets executed

public static boolean checkIfThisIsFavorite(Show s){
        List<Show> results = getListFromDb(s, WHERE_CLAUSE_ID);
        if(results.size() > 0){
            //found match
            return true;
        }else {
            //no match
            return false;
        }
    }

Then finally this

    private static List<Show> getListFromDb(Show s, String whereClause){
        int showId = s.getShowId();
         return Select.from(Show.class)
                .where(Condition.prop(whereClause)
                        .eq(Integer.valueOf(showId)
                                .toString()))
                .list(); // Line of crash
    }

This is the version of sugar ORM I'm using compile 'com.github.satyan:sugar:1.4'.
This is my manifest's application tag.
<application
        android:name="com.orm.SugarApp"
        android:allowBackup="true"
        android:icon="@mipmap/ic_launcher"
        android:label="@string/app_name"
        android:supportsRtl="true"
        android:theme="@style/AppTheme">

        <meta-data android:name="DATABASE" android:value="fav.db" />
        <meta-data android:name="VERSION" android:value="2" />
         //I manually removed the other two tags just to be sure, no difference
...
</application>

And this is my model class:
public class Show extends SugarRecord {
    int showId;
    String title; // original_title
    String image; // poster_path
    String image2; // backdrop_path used in DetailActivity
    String overview;
    int rating; // vote_average
    String date; // release_date

    public Show() {

    }

    public Show(JSONObject movie) throws JSONException {
        this.showId = movie.getInt("id");
        this.title = movie.getString("original_title");
        this.image = movie.getString("poster_path");
        this.image2 = movie.getString("backdrop_path");
        this.overview = movie.getString("overview");
        this.rating = movie.getInt("vote_average");
        this.date = movie.getString("release_date");
    }

    public Show(JSONObject movie, int notNeeded) throws JSONException {
        this.showId = movie.getInt("id");
        this.title = movie.getString("original_name");
        this.image = movie.getString("poster_path");
        this.image2 = movie.getString("backdrop_path");
        this.overview = movie.getString("overview");
        this.rating = movie.getInt("vote_average");
        this.date = movie.getString("first_air_date");
    }

    public int getShowId() {
        return showId;
    }

    public String getTitle() {
        return title;
    }

    public String getImage() {
        return "http://image.tmdb.org/t/p/w185" + image;
    }

    public String getImage2() {
        return "http://image.tmdb.org/t/p/w300" + image2;
    }

    public String getOverview() {
        return overview;
    }

    public int getRating() {
        return rating;
    }

    public String getDate() {
        return date;
    }

}

Stacktrace:
 9-15 18:41:59.507 6307-6307/com.dcs.shows E/AndroidRuntime: FATAL EXCEPTION: main
                                                             Process: com.dcs.shows, PID: 6307
                                                             android.database.sqlite.SQLiteException: no such table: SHOW (code 1): , while compiling: SELECT * FROM SHOW WHERE (SHOW_ID = ? )
                                                                 at android.database.sqlite.SQLiteConnection.nativePrepareStatement(Native Method)
                                                                 at android.database.sqlite.SQLiteConnection.acquirePreparedStatement(SQLiteConnection.java:887)
                                                                 at android.database.sqlite.SQLiteConnection.prepare(SQLiteConnection.java:498)
                                                                 at android.database.sqlite.SQLiteSession.prepare(SQLiteSession.java:588)
                                                                 at android.database.sqlite.SQLiteProgram.<init>(SQLiteProgram.java:58)
                                                                 at android.database.sqlite.SQLiteQuery.<init>(SQLiteQuery.java:37)
                                                                 at android.database.sqlite.SQLiteDirectCursorDriver.query(SQLiteDirectCursorDriver.java:44)
                                                                 at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:1316)
                                                                 at android.database.sqlite.SQLiteDatabase.queryWithFactory(SQLiteDatabase.java:1163)
                                                                 at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1034)
                                                                 at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1240)
                                                                 at com.orm.SugarRecord.find(SugarRecord.java:192)
                                                                 at com.orm.query.Select.list(Select.java:118)
                                                                 at com.dcs.shows.utils.FavoriteUtils.getListFromDb(FavoriteUtils.java:38)
                                                                 at com.dcs.shows.utils.FavoriteUtils.checkIfThisIsFavorite(FavoriteUtils.java:23)
                                                                 at com.dcs.shows.DetailFragment.onCreateView(DetailFragment.java:83)
                                                                 at android.support.v4.app.Fragment.performCreateView(Fragment.java:2074)
                                                                 at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1104)
                                                                 at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1286)
                                                                 at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:758)
                                                                 at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1671)
                                                                 at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:532)
                                                                 at android.os.Handler.handleCallback(Handler.java:739)
                                                                 at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                 at android.os.Looper.loop(Looper.java:148)
                                                                 at android.app.ActivityThread.main(ActivityThread.java:5417)
                                                                 at java.lang.reflect.Method.invoke(Native Method)
                                                                 at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
                                                                 at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

Have you solved this I am working the same project too, Had the same problem.

No, the bug is still present. Though if you disable instant run in Android Studio the bug is no longer present at startup, but may still happen while using the app.

I have disabled instant run but still I had the same problem. When I changed some meta-data in my Manifest file it worked for me.

I tried your method to check if the movie has been favored but in the log I get
D/SQL Log: SQLiteQuery: SELECT * FROM FAVORITE_MOVIE WHERE (MOVIE_ID = ? )

Apparently the bug was fixed in this version of SugarORM so maybe that's why it happens with such unpredictability. I am not sure about your question, if you are asking what I see when that method gets called here it is:
SQL Log: SQLiteQuery: SELECT * FROM SHOW WHERE (SHOW_ID = ? )
The method basically returns a List<>. The list can both have 0 items (the movie was not favorited) or 1 item (the movie was in favorites). The List cannot have 2 or more items since every movie has an unique ID in the SHOW_ID column.

public class PersonDb extends SugarRecord {
String user_name;
int state;
String token;
int id_user;
int id_vehicleType;
boolean login;

public PersonDb(){
}

public PersonDb(String user_name, int state, String token, int id_user, int id_vehicleType, boolean login) {
    this.user_name = user_name;
    this.state = state;
    this.token = token;
    this.id_user = id_user;
    this.id_vehicleType = id_vehicleType;
    this.login = login;
}

public String getUser_name() {
    return user_name;
}

public void setUser_name(String user_name) {
    this.user_name = user_name;
}

public int getId_user() {
    return id_user;
}

public void setId_user(int id_user) {
    this.id_user = id_user;
}

public int getId_vehicleType() {
    return id_vehicleType;
}

public void setId_vehicleType(int id_vehicleType) {
    this.id_vehicleType = id_vehicleType;
}

public boolean isLogin() {
    return login;
}

public void setLogin(boolean login) {
    this.login = login;
}

public int getState() {
    return state;
}

public void setState(int state) {
    this.state = state;
}

public String getToken() {
    return token;
}

public void setToken(String token) {
    this.token = token;
}

}
in the mainActivity
List notes = Select.from(PersonDb.class).where(Condition.prop("user_name").eq("fred")).list();
Process: com.bluecity, PID: 26862
android.database.sqlite.SQLiteException: no such column: user_name (code 1): , while compiling: SELECT * FROM PERSON_DB WHERE (user_name = ? )

I'm facing exactly the same issue with my Nexus 5X (Android 7.0). Using both 1.4 and 1.5 versions of the library.
First time using Sugar ORM, so I've been for a long time thinking I was doing something wrong. Finally I've read this thread and tried on an emulator (Nexus S, API 19) and everything worked right.
I was using previously a SQLite database with no problem in the same Nexus 5X phone, so it should'nt bet a SQLite problem.

I'm using one of latest commits and the behavior is inconsistent - about 5% of users are affected by this issue. Any progress on it?

I solved issue by using this:
When using ProGuard
Ensures entities remain un-obfuscated so table and columns are named correctly
-keep class com.yourpackage.yourapp.domainclasspackage.** { *; }

@marzochi  please explain more ...i still dont get it.

@ayebaretony , firstly close Instant Run than add prog-guard rule.
I explained here:
https://mehmethanoglu.com.tr/tips/15-activedroid-ve-sugarorm-1.5-android-studio-2.2-kullanimi.html

@marzochi hello, what do you mean by domainclasspackage?

@fermatijoe,
https://mehmethanoglu.com.tr/uploads/posts/2016-10/1476268747_7.png
Example my domain of class package is db all sugarorm objects here. Please look my previous message, link contains all details.

@marzochi's suggestion worked for me

Just to give closure:

Disable instant-run
Unistall and reinstall the app after disabilng instant-run (important)
Follow @marzochi suggestion if you keep encountering runtime crashes

Still it would be appreciated to fix this bug either on the developer's or google's side.


mismatch between go.values and get_all_children

I am using the go-basic.obo file. When I do len(go.values()), I get 47245. Note that the GO terms below are for BP, CC, AND MF. When I do len(go["GO:0003674"].get_all_children()) + len(go["GO:0005575"].get_all_children()) + len(go["GO:0008150"].get_all_children()) I instead get 44576. Since the graphs are separate, these values should be only separated by 3 (corresponding to the terms for BP, CC, and MF). Why are they not matching?

@mdanb
I know it is a year too late for a response but I just did a quick analysis to investigate the discrepancy that you found, and added my analysis to our report_depth_level notebook.
Long story short, the numbers match exactly if you remove duplicates with set(go.values()). The reason for the duplicates is that some of the entries have alternative_id. For example, GO:0003887 contains 16 alternative ids, which led to the duplicates, i.e. you are getting the same term back when you do GO:0016448, GO:0016449 etc.) - causing the inflation in total counts that you saw.
Haibao


Add CI via GitHub workflows

Add the CI integration and its tags in the readme

Done https://github.com/sebastianconcept/STTemplate/pull/12


It won't run (the service has not been started)

Now I have installed "Microsoft.WindowsTerminal_1.0.1401.0_8wekyb3d8bbwe.msixbundle".  It will not run.  A message box says:
C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.0.1401.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
The service has not been started.
dcomlaunch and AppXSvc are running.
I asked about this 3 months ago.  My issue was closed with no solution given.  Please tell me how to get it running.

If other reports on this repository are correct, you will need to enable wlidsvc.

Thanks!  That did it.

Thanks!


Immediate 2.7 upgrade on v1.0.9

Hi all,
Intel just released the 2.7 version of their SDK. The diff indicates that 2.7 can defend against novel attacks to the switchless feature. My tests reveals that 2.7 is 100% compatible with current 1.0.9 release. Shall we update the docker image directly in 1.0.9?

I find (potentially) a bug in 2.7, which may break mutual-ra.
https://github.com/intel/linux-sgx/issues/457
If you have chance to try with 2.7, please help me check if it happens to you. Thanks!

I would update it as part of the next release. IMO, a 1-to-1-to-1 mapping between Intel's SDK version, rust-sgx-sdk version, and Rust toolchain version is worth preserving for the immediate future.

Got it. Thanks for the advice!
Seems that Intel has a patch and will release a new version soon. Let's wait for it.

Intel released 2.7.1
https://01.org/intel-softwareguard-extensions/downloads/intel-sgx-linux-2.7.1-release-version-string-<IP_ADDRESS>
Since 2.7 breaks the remote attestation APIs, I think we should skip it and move to 2.7.1 directly.


Unable to send control characters?

It'd be very helpful to be able to send control characters (CTRL+C, BACKSPACE, etc...).
I couldn't readily find a way to send raw bytes via the input box.
EDIT: (for those also wondering)
Undoubtedly my naiveness, but since I couldn't escape bytes within the input box, I just added a couple buttons for control characters I commonly use with CircuitPython boards.
add buttons:
<button onclick="sendCtrlChar(3)" style="width:45px">Halt</button>
<button onclick="sendCtrlChar(4)" style="width:65px">Restart</button>

add function
async function sendCtrlChar(ctrlChar) {
    if (ctrlChar == 3) dataToSend = "\x03"; //ctrl+C
    if (ctrlChar == 4) dataToSend = "\x04"; //ctrl+D
    await writer.write(dataToSend);
}

I added a new button so you can enter a number and send character.
Let me know if that works. I am planning on adding some thing so you can create buttons with specific strings to send. Should be good for shortcuts to send frequently used things.

Actually I am looking in to how to make a more advanced version of the terminal with full compatibility.
I found this https://xtermjs.org/ and it seem like it might be a prime candidate for the more sophisticated version. Just need to figure out how to link it all up. Any help would be appreciated.

@mmiscool interesting... have you played with https://googlechromelabs.github.io/serial-terminal/?

I did play with that chrome labs demo but it never worked as I expected it to work.
@maholli

My I suggest accepting those numbers in Hex? Hexadecimal. perhaps by preceding them with '\x' e.g. \xaa would be the same as 170 (because 170 in decimal is AA in hex). Also, \t or \r might be nice to support as tab and return. So then you need \ as . But that would make the tool so very very valuable for debugging binary serial communications.
Oh... you would also need an option to display the return values which are binary the same way.


Windows install of Postgres Intercepts external connections

When attempting to "test" a connection you get an error saying "this "user" doesn't exist on this connection."

However, that shouldn't happen. Its checking local Postgres users first?
If you just save the ssh connection instead of "testing" it, The connection works 100% fine.

When you say "ssh connection", you're just doing a regular connection without the ssh tunnel options enabled?

Correct. it's looking for "Advent" when I specifically put in "rudy."
my local has a user named Advent however

Alright, confirmed what's going on. It looks like when you do not fill in a database name, it will attempt to use a set default database, which for postgresql is postgres. However, it does not do this when testing the connection, falling back to how the library determines default database name, which for postgres is the username of the current OS.
Anyway, fix will be incoming shortly for the imminent 1.32.0 release

If someone else has this bug before the next release, this is where you would put the default database name when testing a connection.


I'm supporting myself with nothing...

I guess it must be emotional support.
I don't have any troops aside from conscripts. When I click on the button on my city and then click support, nothing happens (or so I thought, until I went back into city view.)

Can also attack Ketzle with nothing...

Ok. Fixed it in my version. Not pushed to server yet.
Haven't implemented minimum troops for plunder, assault and siege though.

Minimum troops requirement is implemented too


Accelerate label drawing

Don't create text if the state is smaller than 3 pixels
If the text is the same, don't make a new text, keep the old one
If the label is empty, don't create a label
Delete the bitmaps as they are cleared

Signed-off-by: Matthew Khouzam<EMAIL_ADDRESS>

Btw, I verified that #59 is still fixed after this change.

Still looks good to me. Thanks for fixing the formatting.


UIMARCAUTH-30: MARC Authority: Implement App context menu and keyboard shortcuts

Description
Implement edit and search  keyboard shortcuts
Issues
UIMARCAUTH-30
Video
https://user-images.githubusercontent.com/96055497/149836013-97f5cbf4-a25a-4cd2-a246-b4decafcfbba.mp4

yarn run v1.22.17
$ eslint .
/home/jenkins/workspace/io-org_ui-marc-authorities_PR-56/project/src/components/KeyShortCutsWrapper/KeyShortCutsWrapper.js
51:6  warning  React Hook useMemo has missing dependencies: 'editShortcuts' and 'searchShortcuts'. Either include them or remove the dependency array  react-hooks/exhaustive-deps
68:3  error    Prop types declarations should be sorted alphabetically                                                                                react/sort-prop-types
✖ 2 problems (1 error, 1 warning)
1 error and 0 warnings potentially fixable with the --fix option.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


Assigning a colorTransform shall flag the displayObject as dirty

Assigning displayObject.transform.colorTransform flags the display object as dirty.
This is a way to enable typical color adjustment method - https://books.openfl.org/openfl-developers-guide/display-programming/manipulating-display-objects/adjusting-displayobject-colors.html

var ct:ColorTransform = obj.transform.colorTransform;
ct.color = newColor;
obj.transform.colorTransform = ct;

A code like this shall effectively change the display object color. In fact it did not change the color on screen because the object was not marked as dirty. There are two ways to fix it:

To modify the getter to return a copy of the color transform:

private function get_colorTransform():ColorTransform {
  return __colorTransform.clone();
}

To set the dirty flag every time the setter is called. This approach I have chosen here as it doesn't produce a clone object.

Thank you!
I think we need to return a new object, because otherwise other code will also fail, like:
var ct1 = sprite.transform.colorTransform;
ct1.blueMultiplier = 0.5;
var ct2 = sprite.transform.colorTransform;
ct2.alphaMultiplier = 0.2;
trace(ct1 == ct2); // true

I updated the internals so that the clone is not called internally, though now it looks like colorTransform really should be an internal property on DisplayObject and Transform is just an accessor (similar to how it works for matrix, concatenatedColorTransform, etc)
TODO :wink:

Thank you!
I think we need to return a new object, because otherwise other code will also fail, like:
var ct1 = sprite.transform.colorTransform;
ct1.blueMultiplier = 0.5;
var ct2 = sprite.transform.colorTransform;
ct2.alphaMultiplier = 0.2;
trace(ct1 == ct2); // true

I updated the internals so that the clone is not called internally, though now it looks like colorTransform really should be an internal property on DisplayObject and Transform is just an accessor (similar to how it works for matrix, concatenatedColorTransform, etc)
TODO :wink:

Joshua, the fix works well for me. Thanks!
I have commented on your commit baffe62: need also to change the return value of the setter.

Joshua, the fix works well for me. Thanks!
I have commented on your commit baffe62: need also to change the return value of the setter.


Token Quest

Context
Hecate has long had the ability to generate and track tokens in a generic way, however it has been tied primarily to session tokens. This goal of this PR is to add more generic handling of both session and user created tokens
TODO

[ ] Add API to list tokens
[ ] Add tests for listing tokens
[ ] Add user UI to list & manage tokens

cc/ @ingalls

This work has now been released: https://github.com/hecate/hecate


Remove warning in train_test_split

It didn't provide guidance to the user anyway.
Closes https://github.com/dask/dask-ml/issues/550

Actually, rethinking this.
I think the behavior I wanted to warn on was a mix of dask.Array X  and dask.Series y. That falling back to scikit-learn is a poor UX.
So I think we should do is convert the arrays to a dataframe, or the series to an array. Not sure which yet... If we're doing Series -> Array, we'll need chunk sizes, so it'll have to be controlled by a user parameter.


Missing start attribute of log10n

Missing start attribute of log10n in Chemical.Components.Substance
Warning In Chemical.Examples.SimpleReaction
There is no start attribute of state A.log10n. It is set to 0.
Warning In Chemical.Examples.SimpleReaction
There is no start attribute of state B.log10n. It is set to 0.

Hmm, log10n=0 is inconsistent with initial equation. The environment should solve the right value of log10n from the equation amountOfSubstance = 10^log10n in Chemical.Components.Substance, because initial equation is amountOfSubstance=amountOfSubstance_start. So, the start value of log10n must be log10(amountOfSubstance). However I add this expresion to the log10n definition as "Real log10n(start=log10(amountOfSubstance_start))" for sure..

Thanks. Fix confirmed.