k8s-4730 Fix hanging reboot on auto update.

What this PR does / why we need it:
This PR adds the polkit permission required for the user to reboot the machine. The user runs flatcar-update-controller and needs permissions to reboot the machine when flatcar-update-controller receives a signal for the OS to update.
Special notes for your reviewer:
The PR is not created at upstream because support for flatcar does not exist in https://github.com/kubermatic/machine-controller master branch.
Optional Release Note:
Fix hanging reboots after auto updates on flatcar machines.

updated the description.


Lolex dependency issue

I was getting an error on line 69 in sinon-timers.js saying "Can't read property 'install' on undefined". After looking all around, the only thing that worked was this repo that added lolex as a more explicit karma dependency: https://github.com/wombatsecurity/karma-lolex-sinon-chai.
You might consider merging over their master commits.

Is there any solution for solving this issue?

Yes, fork that @wombatsecurity repo linked above and use it instead.
This will install from a github repo
npm i -D YOURNAME/karma-lolex-sinon-chai

It still works just like regular karma-sinon-chai in terms of how you set up your karma config.

+1

Seems like something went wrong with Sinon versions between 1.12.2 and 1.17.2 (https://github.com/sinonjs/sinon/issues/621)
Can run tests only with v0.3.0 if I'm trying to update to v1.1.0 getting same errors as mentioned in https://github.com/kmees/karma-sinon-chai/issues/28

Can someone please test v.1.2.0-rc1 and report back? Thanks

@kmees checked it on alive project and everything works now, all tests a green :relieved:

Thanks for testing, will close this issue now.


Suggest choosing group/teacher

Suggest choosing group/teacher if there is no value in UserDefaults

Show alert with title "Choose group/teacher" and message "You can always change it by clicking change button". Two options: go choose and cancel


App dissappeared

Describe the bug
I just spent 4 hours and 4 million tokens developing an app only for it to completely disappear! Either return the app to it's last state before it dissapeared or return my tokens PLUS compensation for all of the lost time!
My entire chat history is gone!
Link to the Bolt URL that caused the error
na
Steps to reproduce
na
Expected behavior
na
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Same here?!!

I do not know what you mean, open it via email. I do not have a link sent
via email.
On Thu, Oct 31, 2024, 2:58 AM Brandon Eichhorn @.***>
wrote:

Same here. @schilling3003 https://github.com/schilling3003 Have you
built the app on a different browser/different machine? If so, you should
open the Stackblitz by email and then send it to yourself..
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/bolt.new/issues/1221#issuecomment-2449160102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ2PF7EF6DDBZWDPXVM4REDZ6HIKBAVCNFSM6AAAAABQ5UOHIGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBZGE3DAMJQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature: Add aws provider definition with aws profile

Hi,
I want to contribute, so add a snippet that output aws provider definition with aws profile like in the doc : https://www.terraform.io/docs/providers/aws/index.html#shared-credentials-file

Given : A user editing a terraform file on vscode
When : He enters tf-aws-profile
Then : The editor must output the following snippet
provider "aws" {
   region = "AWS REGION"
   shared_credentials_file = "path/to/.aws/creds"
   profile = "AWS PROFILE"
}

Hope It helps :)
Regards

Hi @lktslionel my apology for late reply, I was on holiday. Btw, thank you very much for your PR - I'll check it and will merge your changes. cheers

hi @lktslionel your changes has been merged and pushed to VScode marketplace. thank you very much!


Migrate concat fusion to r0.16

@diyessi Not sure why ONNX padding tests are failing. Does nGraph-ONNX need to be updated to use r0.16? I see in the log that nGraph-ONNX 0.15.0 is used for CI.

Creating /root/.tox/py3/lib/python3.5/site-packages/ngraph-onnx.egg-link (link to .)
Adding ngraph-onnx 0.15.0 to easy-install.pth file

Installed /root
Processing dependencies for ngraph-onnx==0.15.0


No module named 'midas'

From 1.4 Define Midas functions

ModuleNotFoundError                       Traceback (most recent call last)
Input In [4], in <cell line: 4>()
1 #@title ### 1.4 Define Midas functions
----> 4 from midas.dpt_depth import DPTDepthModel
5 from midas.midas_net import MidasNet
6 from midas.midas_net_custom import MidasNet_small
ModuleNotFoundError: No module named 'midas'
No clue what I did wrong. running Win11, 21h2.
Installed everything as directed. Perhaps I goofed somewhere along the way.

The cell above this should install Midas for you, probably something is going wrong there. Look for any errors in this previous cell.
Are you running this in Ubuntu WSL, or trying to run it natively in Windows?

I also get same error I am using in windows with google colab using local runtime.


Fix empty keys in EnvironmentChangedEvent on resetting the Environment

When the Environment is resetted the EnvironmentChangedEvent always
contains an empty keySet, since the live view of the cleared map was
used. This commit fixes this.

I guess it depends on what reset() should do. The way you define it, is all keys that are in the map that are cleared get sent as part of an EnvironmentChangedEvent. The previous behavior was empty. What is the use case you have that doesn't fit?
/cc @dsyer

I was debugging another issue where my ConfigurationProperties hasn't been rebound after a reset (using Finchley) and stumbled across that code. In my opinion the code expresses an intention but doesn't behaves this way -  so I "fixed" this.


@tf.custom_gradient fails in eager execution mode

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OSX High Sierra 10.13.2
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.9.0-0-g25c197e023 1.9.0
Python version: 3.6.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: Running on CPU
GPU model and memory:
Exact command to reproduce:

Describe the problem
@tf.custom_gradient does not work in eager execution mode although the documentation states it does.
Source code / logs
Providing a minimal example to reproduce:
import tensorflow as tf

tfe = tf.contrib.eager
tf.enable_eager_execution()

@tf.custom_gradient
def log1pexp(x):
    e = tf.exp(x)

    def grad(dy):
        return dy * (1 - 1 / (1 + e))
    return tf.log(1 + e), grad

with tf.GradientTape() as tape:
    x = tf.constant(100.)
    y = log1pexp(x)

print(tape.gradient(y, [x]))
# prints None as default derivative is undefined
# not working properly - it did not use the custom gradient func (and print 1.0)

# Graph mode is working OK!
# x = tf.constant(100.)
# y = log1pexp(x)
# dy = tf.gradients(y, x)
#
# with tf.Session() as sess:
#     print(sess.run(dy))
# # prints 1.0 - the custom derivative

Short answer: Adding a tape.watch(x) will make this work, so:
with tf.GradientTape() as tape:
    x = tf.constant(100.)
    tape.watch(x)
    y = log1pexp(x)

print(tape.gradient(y, [x]))

As per the documentation for GradientTape, trainable Variable objects are watched automatically, but tensors aren't.
@RamNathaniel : While we figure out how to make it harder to trip over this quirk, I hope you can continue to make progress (and it's fine to call tape.watch(x) in graph as well, so the same code will work).

Thank you for the very quick response - much appreciated!

Ok, I'll get to my question directly: So, If I have a model with a gradient reversal layer, i.e. a custom layer with no trainable variables, that calculates tf.identity(x) in the forward pass, but should return tf.negative(grad) in the backward pass, what should I monitor in this case? the layer itself?


remote installs do not use original nuget sources

see https://boxstarter.codeplex.com/discussions/547713
UPDATE: Original link gone but available here

That discussion is gone, however the issue seems to still exist.
I've just ran into the problem that Install-BoxstarterPackage does not use NugetSources from Boxstarter Config. (also checked that AppData\Roambing\Boxstarter - the version of Boxstarter that is being pushed to the remote host actually contains the correct boxstarter.config)
as far as I've seen there is no mechanism for handling this inside https://github.com/chocolatey/boxstarter/blob/master/Boxstarter.Chocolatey/Install-BoxstarterPackage.ps1 - maybe another/default config file is being used?
(will continue investigation...)

actually, Install-BoxstarterPackage.ps1 states the observed behavior..
 The .nupkg file for the provided package name is searched in the following locations and order:
 - .\BuildPackages relative to the parent directory of the module file
 - The Chocolatey community feed
 This can be configured by editing $($Boxstarter.BaseDir)\Boxstarter.Config

The "problem" seems to be the fact that the original Boxstarter.Chocolatey\Boxstarter.zip is being sent to the remote host - with the default boxstarter.config ..
I take this is probably intentional?=

alright - there seems to be a reason for this: when using Boxstarter to install on remote hosts (i.e. azure) - you probably won't be able to access an organization internal server .. so it makes sense to fallback to community repository + some packages on a "local feed" ..
@mwrock @ferventcoder @gep13 @pauby
If that's ok with you I'll put in a PR that adds another parameter -DelegateChocolatySources (we can talk about the name of that switch) in order to be able to pass the sources.

This has been merged. Closing.


feat: Add security_policy to backend service configuration

The existing net-lb-app-int module doesn't support security_policy which is required to attach Cloud Armor Policies to backends.

Checklist

I applicable, I acknowledge that I have:

[x] Read the contributing guide
[x] Ran terraform fmt on all modified files
[x] Regenerated the relevant README.md files using tools/tfdoc.py
[x] Made sure all relevant tests pass

@ludoo lbs are your thing. Can you tal?


Filter notifications by linode ID in mSTD

Coverage decreased (-0.006%) to 26.382% when pulling 422a449dda2fc9d5594411e64e700b650811d30c on Jskobos:filter-notifications into b3bce770a32cc98ec098fd97adf4ac469e91cfeb on linode:feature/migrations.


Sortable Tables for Player Pages

For all tables that would make sense, there should be some form of client-side sorting. Tables such as on a player profile, the ability to sort the skilling/bossing/activities tables. Something kinda simple such as clicking the table header to toggle between an ascending/descending should suffice.

This has been suggested before, I actually thought this was a duplicate issue, but it doesn't seem to be! Thank you

Implemented in #139


Gentoo Ebuild Proposal for Thorium Browser

I propose creating a Gentoo ebuild for the Thorium browser. While I lack experience with Chromium-based ebuilds, I'm willing to try. If anyone is currently working on this, or in the future, consider the official Chromium ebuild in the Gentoo repository as a resource.

I would love to see this.
Also, It'd be interesting there was a use flag for said ebuild to enable the ChromiumOS WM/DE via ChromiumOS on Linux.

yes, I'd also like to see Thorium on gentoo. I'll post an update if I make any progress on this myself

This would be awesome
@radeaosk Did you make any progress?

This would be awesome!
Here's a duplicate thread on Gentoo's forum. There's also this Cromite EBuild which could probably be re-purposed (but like you said @nnzv, the same should be doable with the official Chromium EBuild).
@radeaosk Did you make any progress?

just made it: https://github.com/gentoo/gentoo/pull/36882

@fraschm1998, I noticed that your PR was rejected. I suggest considering the maintainer's advice by either contributing to the guru ebuild repository or creating your own repository to share your work with others.

@fraschm1998, I noticed that your PR was rejected. I suggest considering the maintainer's advice by either contributing to the guru ebuild repository or creating your own repository to share your work with others.

Just submitted a pull request to Guru repo here.

I am curious about why no one can write a source-based ebuild...

It sounds like it was a lot to handle, buddy. At least, that’s what I think.


Move LoadConfig() out of connect() for stable board detection

related to #1205
This is a potential improvement because during initial connect the config is not immediately loaded, and thereby we prevent to send a lot of data at once just right after connecting.
Changes:

LoadConfig removed from connect method
LoadConfig now explicitly called only for MobiFlight boards, after GetInfo

Feedback from the user who reported #1205 - it didn't make a difference, but I think it is an optimization to keep and see how it behaves for other users

Could this be the reason for the non-detection of the Nano boards? MF 9.7.1 detects them OK.

I am still wondering why this does not happen for the other boards. I see no topic where they differ in board detection.
But it could be a try worth.

as i wrote in my comment: it seems to make sense but not sure if it has any unforeseen side effects - maybe @JaimeLeon2 found one of those


Update make-custom-functions-compatible-with-xll-udf.md

Minor change to reflect 1.3 CFs

Docs Build status updates of commit 00559e2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/make-custom-functions-compatible-with-xll-udf.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Implementing @glenjamin's idea on react-bootstrap fork into the library

Hello @glenjamin. This is a proposition of implementation of your ideas in react-bootstrap into this repository.
Just run npm install and then npm test to spawn the server to start testing.
If this implementation satisfies you, I'll be more than glad to help you build this library further.

Looks broadly sensible, I had been toying with the idea of not depending on Babel in the lib - and sticking to pure ES5.
I've got a few minor comments I'll make inline later.
One thing I'm keen to do early on is move the knowledge of the module system outside of the core, and expose an API that's easy to plug into instead.

Looks broadly sensible, I had been toying with the idea of not depending on Babel in the lib - and sticking to pure ES5.

As you wish. This shouldn't be very difficult to downgrade back to ES5. Do you want to drop JSX also?

One thing I'm keen to do early on is move the knowledge of the module system outside of the core, and expose an API that's easy to plug into instead.

You mean the require.context part?

This shouldn't be very difficult to downgrade back to ES5. Do you want to drop JSX also?

I'm still on the fence about it (whether to stick to ES6+JSX or ES5), not sure if it'll make things easier or harder

You mean the require.context part?
Yep, moving the require() out into the calling code and introducing a defcard = devcards.ns("abc") API would handle this neatly I think

I'd also rename test/app to example, the lib itself should be fairly straightforward to test via shallow rendering and https://github.com/glenjamin/skin-deep

I've started work on arranging this how i'd normally lay out a module - https://github.com/glenjamin/devcards-js/tree/es5-plus-example
Hit a bit of a wall with the example, something to do with node module paths - I'm not sure if it's a webpack quirk or a kotasku quirk. Will come back and take another look this evening.
Some stuff i've noticed:

I should make my eslint configs into a node module
Kotasku should probably have json loader by default

I should make my eslint configs into a node module

Yes you should :). I just wrote mine yesterday. I was tired of copy-pasting it everytime.

Kotasku should probably have json loader by default

There is an issue on the subject. For the time being it's default for backend but not front. But this is hardly consistent. I am still struggling on the question: should I make it default everywhere or create a --json option.

The before-mentioned issue.

If you have time it'd appreciate a look at the issue I have on that branch.
json-loader is added to the example/package.json, devcards is pulled in via ../- but then I get this error:
      /Users/glen/Development/GitHub/devcards-js/node_modules/json-loader doesn't exist (module as directory)

Any ideas?

You mean the es5-plus-example branch? When starting the example app.

Yeah, would also appreciate feedback on the changes, if there's anything you disagree with.

Ok. Let me check it.

It seems that when compiling files from example parent directory (the root of the repo), it tries to solve dependencies from the parent directory instead and thus fails. Don't see if this is a webpack or kotatsu quirk though.

The following config solves the problem:
var path = require('path');

module.exports = {
  module: {
    loaders: [
      {test: /\.json$/, loader: require.resolve('json-loader')}
    ]
  },
  resolve: {
    modulesDirectories: [
      path.join(__dirname, 'node_modules'),
      path.join(__dirname, '..', 'node_modules')
    ]
  }
};

I don't really get where this problem is coming from.

Thanks for looking into this! I'll get that updated this evening.
Do the rest of the changes seem reasonable to you?

Will check this thoroughly in the afternoon. But there don't seem to be unreasonable changes.
I am quite interested in you devcard.ns thing however. I cannot wrap my head around how you could make this happen with the module.hot API etc.

devcard.ns would be approximately equivalent to this line https://github.com/glenjamin/devcards-js/blob/es5-plus-example/devcards.js#L41

Wouldn't this force the use to add module.hot.accept() to its cards files?

Just released kotatsu v0.4.0 with json support for client out of the box and some bug fixes (concerning console output notably with progress bars).

Did some tests with webpack & babel without kotatsu and the same problems occur when requiring file in parent directory of config/entry.

Else than this I don't see any unreasonable changes. A linter would be nice also so I can safely code according to your style.

Yup, i'll sort out my eslint configs so everything works neatly in the example and the project itself.

@Yomguithereal I've done a bit more on this, but hit another weird error - i think this one is kotatsu related
https://github.com/glenjamin/devcards-js/tree/e018c0a3a1a3e9fc086be22e1a0afc7c5689e48c
When running on the CLI, everything looks fine:
~/Development/GitHub/devcards-js/example (es5-plus-example) glen ○ retina
> npm start

><EMAIL_ADDRESS>start /Users/glen/Development/GitHub/devcards-js/example
> kotatsu serve --index index.html --progress --source-maps --jsx main.js

[kotatsu] (v0.4.0)
Compiling - [==============================] 100%
[kotatsu] Built in 2.8s.
[kotatsu] Done!
[kotatsu] Serving your app on port 3000...

But when I open the app, I get an empty page and no errors in the console.
If i then edit & save Button.card.js I see this in the console
[HMR] ./Button.card.js
Module parse failed: /Users/glen/Development/GitHub/devcards-js/example/Button.card.js Line 1: Unexpected token
You may need an appropriate loader to handle this file type.
| import devcard from '../';
| import React from 'react';
| import Button from './Button';
 @ . \.card\.js$ bundle.js:14084:24
[HMR] ./Label.card.js
Module parse failed: /Users/glen/Development/GitHub/devcards-js/example/Label.card.js Line 1: Unexpected token
You may need an appropriate loader to handle this file type.
| import devcard from '../';
| import React from 'react';
| import Label from './Label';
 @ . \.card\.js$

So there's two problems here:

Errors on the initial build don't seem to get displayed, and build errors are never shown in the console
it's not picking up my .babelrc, is there a --babel flag or something I can pass?

I start investigating.
You forgot to pass --es2015.

You forgot to pass --es2015.
Ah, I thought that was related to the presets. How would you feel about potentially renaming that option?

Both options exists. Actually, passing --presets es2015 would produce the same effect.

Concerning 1. It seems that the errors are actually warnings. And I did not handle warning yet. I will do it. Could it be the fact of the NoErrorsPlugin to shove errors into warnings?

And do you know why sometimes, when in watching mode, webpack compiler compiles twice when starting it? Have you noticed a race condition in the webpack watcher when developing webpack-hot-middleware?

Published a 0.4.1 that should display the expected warnings.

Both options exists. Actually, passing --presets es2015 would produce the same effect.

As I've got a .babelrc, I'm not really thinking about es2015, i'm just thinking "enable babel please". Perhaps the --es2015 flag could be renamed or aliased to --babel ?

And do you know why sometimes, when in watching mode, webpack compiler compiles twice when starting it? Have you noticed a race condition in the webpack watcher when developing webpack-hot-middleware?

I think I've seen this occasionally at work, but never tried to narrow it down.

As I've got a .babelrc, I'm not really thinking about es2015, i'm just thinking "enable babel please". Perhaps the --es2015 flag could be renamed or aliased to --babel ?

You are right, your example shows that I should add a --babel flag or something of the kind. The thing is I wanted to abstract as much of the stack as possible for 90% of cases which are simple.

I added --babel in the now published 0.5.0 version. Sorry for the release gatling.

Published a 0.4.1 that should display the expected warnings.

Confirmed, that's working nicely now.

Sorry for the release gatling.

No worries, this is all useful

I noticed my builds were slower than I expected when using kotatsu, so I decided to setup a webpack dev server myself.
It took 20 minutes of messing around copying configs from elsewhere, and it doesn't really seem to be any faster.
I'm going back to kotatsu.

Are you using npm v2? I noticed babel 6 is tremendously slow with npm 2 due to a loooot of requires (~40 000) due to the non flattened deps.

Nope, npm@3 - less downloaded, but more time spent deciding what to download.

So where is the slowness coming from? Babel loader only then?

Yeah, it's in the compile phase - takes ~1.5 seconds after saving the file.

That's a bit sad. Babel 5 was faster I think. But I guess they'll boost up performance soon. Do you think the whole lib is required each time a compilation runs? Or is it just Babel parsing etc. which slows the thing down?

I've not looked into it for a while, last time I tried to get compile timing numbers out of webpack they weren't massively illuminating.


chore(NODE-5407): update bson to 5.4.0

Description
What is changing?
Updates BSON to latest 5.4.0
Is there new documentation needed for these changes?
What is the motivation for this change?
Keeps BSON up to date with next driver release
Double check the following

[ ] Ran npm run check:lint script
[ ] Self-review completed using the steps outlined here
[ ] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[ ] Changes are covered by tests
[ ] New TODOs have a related JIRA ticket

NODE-5407


Problem provisioning, box ubuntu14.04

TASK [Install dependencies with composer require.] *****************************
failed: [drupalvm] (item=drupal/devel:1.x-dev) => {"changed": true, "cmd": ["/usr/bin/composer", "require", "drupal/devel:1.x-dev"], "delta": "0:00:13.443747", "end": "2016-11-14 14:53:20.109302", "failed": true, "item": "drupal/devel:1.x-dev", "rc": 2, "start": "2016-11-14 14:53:06.665555", "stderr": "Xdebug requires Zend Engine API version 220131226.\nThe Zend Engine API version 320151012 which is installed, is newer.\nContact Derick Rethans at http://xdebug.org/docs/faq#api for a later version of Xdebug.\n\nPHP Warning:  PHP Startup: xhprof: Unable to initialize module\nModule compiled with module API=20131226\nPHP    compiled with module API=20151012\nThese options need to match\n in Unknown on line 0\nWarning: This development build of composer is over 60 days old. It is recommended to update it by running \"/usr/bin/composer self-update\" to get the latest version.\n    1/4:\thttp://packagist.org/p/provider-latest$8ef4bddcda43fd7dac5f1ebacfb04580efd32d91acc33ca93fc5ea69f90eef6c.json\n    2/4:\thttp://packagist.org/p/provider-2016-04$cd0cd1d2f2bfa465499002bc7ad6ff13223d5b83483fbaf1ae9677e7e52bc0df.json\n    3/4:\thttp://packagist.org/p/provider-2016-10$2a1587e22ac45017ef27ca53ca7206a5cb23b8c3692b262c7b3eb17be7b1fb8b.json\n    4/4:\thttp://packagist.org/p/provider-2016-07$5e2b063011633af4c19ee68ffcac33fa9c1b0e7493f36bace21348d1fc9ed2d4.json\n    Finished: success: 4, skipped: 0, failure: 0, total: 4\n./composer.json has been updated\n    1/4:\thttp://packagist.org/p/provider-latest$8ef4bddcda43fd7dac5f1ebacfb04580efd32d91acc33ca93fc5ea69f90eef6c.json\n    2/4:\thttp://packagist.org/p/provider-2016-04$cd0cd1d2f2bfa465499002bc7ad6ff13223d5b83483fbaf1ae9677e7e52bc0df.json\n    3/4:\thttp://packagist.org/p/provider-2016-10$2a1587e22ac45017ef27ca53ca7206a5cb23b8c3692b262c7b3eb17be7b1fb8b.json\n    4/4:\thttp://packagist.org/p/provider-2016-07$5e2b063011633af4c19ee68ffcac33fa9c1b0e7493f36bace21348d1fc9ed2d4.json\n    Finished: success: 4, skipped: 0, failure: 0, total: 4\nLoading composer repositories with package information\nUpdating dependencies (including require-dev)\nYour requirements could not be resolved to an installable set of packages.\n\n  Problem 1\n    - The requested package drupal/devel could not be found in any version, there may be a typo in the package name.\n\nPotential causes:\n - A typo in the package name\n - The package is not available in a stable-enough version according to your minimum-stability setting\n   see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.\n\nRead <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.\n\nInstallation failed, reverting ./composer.json to its original content.", "stdout": "\nWarning: PHP Startup: xhprof: Unable to initialize module\nModule compiled with module API=20131226\nPHP    compiled with module API=20151012\nThese options need to match\n in Unknown on line 0", "stdout_lines": ["", "Warning: PHP Startup: xhprof: Unable to initialize module", "Module compiled with module API=20131226", "PHP    compiled with module API=20151012", "These options need to match", " in Unknown on line 0"], "warnings": []}
	to retry, use: --limit @/home/antony/drupalvm/provisioning/playbook.retry

PLAY RECAP *********************************************************************
drupalvm                   : ok=206  changed=1    unreachable=0    failed=1

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

The problem seems to be: Warning: PHP Startup: xhprof: Unable to initialize module
I'm guessing that you're using PHP 5.6—if so, please make sure to update the xhprof-related configuration variables too, as documented here: http://docs.drupalvm.com/en/latest/other/php-56/#ubuntu-1404-ubuntu-1204
For 5.6, should be:
xhprof_download_url: https://github.com/phacility/xhprof/archive/master.tar.gz
xhprof_download_folder_name: xhprof-master

I was using php 7.0, I tried changing to 5.6 but I get this error
TASK [Install dependencies with composer require.] *****************************
failed: [drupalvm] (item=drupal/devel:1.x-dev) => {"changed": true, "cmd": ["/usr/bin/composer", "require", "drupal/devel:1.x-dev"], "delta": "0:00:17.053902", "end": "2016-11-15 10:17:42.059327", "failed": true, "item": "drupal/devel:1.x-dev", "rc": 2, "start": "2016-11-15 10:17:25.005425", "stderr": "Xdebug requires Zend Engine API version 220131226.\nThe Zend Engine API version 320151012 which is installed, is newer.\nContact Derick Rethans at http://xdebug.org/docs/faq#api for a later version of Xdebug.\n\nPHP Warning:  PHP Startup: xhprof: Unable to initialize module\nModule compiled with module API=20131226\nPHP    compiled with module API=20151012\nThese options need to match\n in Unknown on line 0\nWarning: This development build of composer is over 60 days old. It is recommended to update it by running \"/usr/bin/composer self-update\" to get the latest version.\n    1/5:\thttp://packagist.org/p/provider-latest$3786843297ac8fd057f702f3cf224249a0ea58415739614a6cceba1220fd6c9c.json\n    2/5:\thttp://packagist.org/p/provider-2016-01$5647cb9b0511d70d96107c62b13443b17ff3552cc1e940a0279c59dd95006c07.json\n    3/5:\thttp://packagist.org/p/provider-2016-10$afa9bfe99ad13209d96fa91bac8f6d4494d9027c8a8afb5c845b19594f0ded11.json\n    4/5:\thttp://packagist.org/p/provider-2016-04$40fa9902f860e0bc5b9b89a74f9187f8c9e82fb98f148ba62fa6fb49e5493b66.json\n    5/5:\thttp://packagist.org/p/provider-2016-07$568a5ac531f1d66c6ef830d059dba726aad912475254ed4fc48d4f99c85b2b40.json\n    Finished: success: 5, skipped: 0, failure: 0, total: 5\n./composer.json has been updated\n    1/5:\thttp://packagist.org/p/provider-latest$3786843297ac8fd057f702f3cf224249a0ea58415739614a6cceba1220fd6c9c.json\n    2/5:\thttp://packagist.org/p/provider-2016-01$5647cb9b0511d70d96107c62b13443b17ff3552cc1e940a0279c59dd95006c07.json\n    3/5:\thttp://packagist.org/p/provider-2016-10$afa9bfe99ad13209d96fa91bac8f6d4494d9027c8a8afb5c845b19594f0ded11.json\n    4/5:\thttp://packagist.org/p/provider-2016-04$40fa9902f860e0bc5b9b89a74f9187f8c9e82fb98f148ba62fa6fb49e5493b66.json\n    5/5:\thttp://packagist.org/p/provider-2016-07$568a5ac531f1d66c6ef830d059dba726aad912475254ed4fc48d4f99c85b2b40.json\n    Finished: success: 5, skipped: 0, failure: 0, total: 5\nLoading composer repositories with package information\nUpdating dependencies (including require-dev)\nYour requirements could not be resolved to an installable set of packages.\n\n  Problem 1\n    - The requested package drupal/devel could not be found in any version, there may be a typo in the package name.\n\nPotential causes:\n - A typo in the package name\n - The package is not available in a stable-enough version according to your minimum-stability setting\n   see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.\n\nRead <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.\n\nInstallation failed, reverting ./composer.json to its original content.", "stdout": "\nWarning: PHP Startup: xhprof: Unable to initialize module\nModule compiled with module API=20131226\nPHP    compiled with module API=20151012\nThese options need to match\n in Unknown on line 0", "stdout_lines": ["", "Warning: PHP Startup: xhprof: Unable to initialize module", "Module compiled with module API=20131226", "PHP    compiled with module API=20151012", "These options need to match", " in Unknown on line 0"], "warnings": []}

RUNNING HANDLER [geerlingguy.php : restart webserver] **************************

RUNNING HANDLER [geerlingguy.php : restart php-fpm] ****************************
	to retry, use: --limit @/home/antony/drupalvm/provisioning/playbook.retry

PLAY RECAP *********************************************************************
drupalvm                   : ok=201  changed=13   unreachable=0    failed=1

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

I tried installing box 16.04 and everything worked fine, can't remember the reason I was using 14.04 but it was something to do with drush I think.

@antonyanimator - If you switch PHP versions, you currently would need to also do a vagrant destroy and vagrant up to recreate the VM from scratch (otherwise you'll get strange errors possibly like the one you're getting above).

Closing for now; please reopen if this is still an issue.


Conditional GANS

Hello,
Do you see any limitations in trying to apply this for conditional gans?
I am looking into style transfer using gans. If when released please make it simple to apply to other areas, as I understand when skimming the paper most of the power is in the Discriminator correct?

Hi Filip, the release will include a conditional version, it is fairly straight-forward to implement :)

Great, cannot wait to test it out 😃


Better name

We're changing the name of the %cell magic from time to time (bob, tischi, ai, ...). I propose to keep the name bob (to be consistent) and merge this repository back to https://github.com/haesleinhuepf/bia-bob . As soon as we come up with a better name, we can rename it in a dedicated PR.

duplicate of #17


PEP: pandas/core final round (api, matrix, sparse, cleanup of common,…

… generic)
Besides groupby we're clean.

lgtm

cherry-picked, thank you!


Example from AWS-CREDENTIALS.md for StsAssumeRoleSessionCredentialsProvider hangs talking to metadata service

I've been trying to get the example code in AWS-CREDENTIALS.md (https://github.com/rusoto/rusoto/blob/master/AWS-CREDENTIALS.md#stsassumerole) working but it's just hanging for me.  I'm using exactly the code in the example, just with my information filled in. Running the code results in:
$ RUST_LOG=trace cargo run
   Compiling tests3 v0.1.0 (/tmp/tests3)
    Finished dev [unoptimized + debuginfo] target(s) in 3.64s
     Running `target/debug/tests3`
[2020-06-10T01:15:39Z TRACE hyper::client::pool] checkout waiting for idle connection: ("http", <IP_ADDRESS>)
[2020-06-10T01:15:39Z TRACE hyper::client::connect::http] Http::connect; scheme=Some("http"), host=Some("<IP_ADDRESS>"), port=None
[2020-06-10T01:15:39Z DEBUG hyper::client::connect::http] connecting to <IP_ADDRESS>:80
[2020-06-10T01:15:39Z TRACE mio::poll] registering with poller
[2020-06-10T01:16:09Z TRACE hyper::client::pool] checkout dropped for ("http", <IP_ADDRESS>)
[2020-06-10T01:16:09Z TRACE mio::poll] deregistering handle with poller
Err(Credentials(CredentialsError { message: "StsProvider get_session_token error: Credentials(CredentialsError { message: \"Couldn\\\'t find AWS credentials in environment, credentials file, or IAM role.\" })" }))

So for some reason it's trying to talk to the metadata service which obviously doesn't work from my machine.
I'm on version 0.44.0.  Thanks in advance for any help!

source_profile isn't read from the default config files. So when the client is created by Sts::new(region), the backing client isn't configured to use the profile you might be expecting.
Adding built-in support has been brought up at least a few times (#1758 , #1559 come to mind).


golang.mk: Check and install yq before use it

golang.mk call yq to get golang_version_min but some environments do
not install it.
This patch check and install yq before use it to handle the issue.
Fixes #899
Signed-off-by: Hui Zhu<EMAIL_ADDRESS>

/test
but, I think this will pass the tests as I suspect the build machines already have yq installed.
I guess you already tested this locally by hand @teawater :-)  When I added the check code I thought I didn't add this as calling install_yq might not work without some infra - so confirmation that this works as expected would be great :-)
If confirmed then L G T M

lgtm. If environment is set up by kata repos like CI, it will have yq installed. This is usefully when manually set up machine.


Code generation with "Remove Unused Imports" / "Organize imports" breaks imports

Overview of the issue
Code generation with "Remove Unused Imports" / "Organize imports" breaks imports
(Empty imports section)
Expected Outcome
Generated code should contain imports
Current Outcome
Generated code does not contain imports
Steps to Reproduce(Only for bug reports)
Enable Netbeans settings

Generate code
Jeddict Version(s)
Version: 5.4.3

This bug (if the option "Organize Imports" is selected) is still there, in NB15.0 + Jeddict 6.1.0.


Add mdsv2alpha1 role sdk and mocks

adding RBACRoleDefinitionsApi, RBACRoleBindingSummariesApi,  RBACRoleBindingCRUDApi and their mocks. Tested locally with cli.
mhe@C02DV0KVMD6T mdsv2alpha1 % mocker --prefix "" --dst mock/api_rbac_role_binding_crud.go --pkg mock api_rbac_role_binding_crud.go RBACRoleBindingCRUDApi
mhe@C02DV0KVMD6T mdsv2alpha1 % mocker --prefix "" --dst mock/api_rbac_role_binding_summaries.go --pkg mock api_rbac_role_binding_summaries.go RBACRoleBindingSummariesApi
mhe@C02DV0KVMD6T mdsv2alpha1 % mocker --prefix "" --dst mock/api_rbac_role_definitions.go --pkg mock api_rbac_role_definitions.go RBACRoleDefinitionsApi

@MuweiHe Looks like this was copied over from a repo that's used for Cloud and On-Prem. Do you plan to use this for On-Prem too? In that case it might be misleading putting it inside a repo called ccloud-sdk-go-v1-public. What do you think about creating a repo called mds-sdk-go-public to keep organized?

@MuweiHe Looks like this was copied over from a repo that's used for Cloud and On-Prem. Do you plan to use this for On-Prem too? In that case it might be misleading putting it inside a repo called ccloud-sdk-go-v1-public. What do you think about creating a repo called mds-sdk-go-public to keep organized?

This is the mdsv2alpha1 api's that's only used for cloud, on-prem is called mdsv1 and has their own versions of api's. Keeping everything in one repo would be easier for us to manage

on-prem is called mdsv1 and has their own versions of api's

Do we plan to open-source mdsv1 for on-prem? If yes, sounds like we'd be open-sourcing the mds-sdk-go library anyways, so there should be no need to move the MDS code into here.


terraform-azurerm-vm-mssql: bootstrap fails to upload files

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ x ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Deploy terraform-azurerm-vm-mssql, run bootstrap

Any log messages given by the failure
Attempt 1: Uploading post-deployment scripts to container 'scripts' in storage account 'st2m7f4tszbkdlw'...
<urllib3.connection.HTTPSConnection object at 0x7f07a84ea850>: Failed to establish a new connection: [Errno -2] Name or service not known

Expected/desired behavior

Files should upload successfully.

OS and Version?

N/A

Versions

#AzureSandbox v2.13.6

Mention any other details that might be useful

Thanks! We'll be in touch soon.

Unable to repro


feat(core/listener): add resubscribe to core listener

Core listener will resubscribe in case of underlying sub close or no new blocks in timeout window (2x blocktime)
Git dif is messed up somehow. I’ve just extracted all message handling logic into separate function handleNewSignedBlock without modifying it.

Codecov Report

Merging #1972 (73ae1a1) into main (d277335) will increase coverage by 0.05%.
The diff coverage is 40.98%.

@@            Coverage Diff             @@
##             main    #1972      +/-   ##
==========================================
+ Coverage   54.60%   54.66%   +0.05%
==========================================
  Files         215      215
  Lines       13679    13701      +22
==========================================
+ Hits         7469     7489      +20
- Misses       5422     5424       +2
  Partials      788      788

Impacted Files
Coverage Δ

nodebuilder/header/mocks/api.go
19.48% <ø> (ø)

share/ipld/namespace_data.go
87.16% <ø> (ø)

core/listener.go
52.17% <40.98%> (+2.17%)
:arrow_up:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Ratings don't work

Very useful plugin for my Recipe channel.
Just letting you know the "Add a rating" button on the Ratings tab doesn't work.

So it looks like there is some kind of listener, because if you resize the window, the button does work.
Some weird Garnish thing no doubt.

Fixed in https://github.com/nystudio107/craft-recipe/commit/0274a5bee5e209b7648f2a167b3d16ff12ccb481


Feature/set t1 to default if empty

Change behaviour of run_fastsurfer.sh

t1 does not need to be defined when running the surface pipeline only. In this case a conformed image should/can already exist from running the segmentation pipeline. Check if this exists, is already done
call for seg-only simplifies to defining --sd and --sid when segmentation was already generated with default arguments for --conformed_name and --aparc_aseg_segfile

Fix error in recon-surf.sh

hires flag was not turned on, when recon-surf was called directly without any argument to --vox_size. Hires flag is now turned on when --vox_size is set to auto, independent if this was explicitly passed as a flag or not (vox_size is set to auto by default).

Not sure about the hires stuff. What if Auto results in a 1mm image? We do not want hires then. Also David is working on a fix, and this will probably conflict.

The current behaviour is that auto = hires. The check that david is implementing will be added to run_fastsurfer.sh not recon-surf. We talked about removing the flag in recon-surf and add only the check for conform min/hires with the conform.py --check_only.
As currently, auto=hires and this is set on by default in recon-surf, but the -hires flag is not turned on, this effectively negates this (run with 1.0 mm settings in recon-surf for all recon-all runs, but hires for all other runs). So I would suggest to merge this request now to have consistent behaviour until David finishes his request. And then it will be a simple merge/rebase.

@LeHenschel see #225 for my changes and this impacts both recon-surf and run_fastsurfer...

Yes,  I am aware of that @dkuegler. See my comment above

The check that david is implementing will be added to run_fastsurfer.sh not recon-surf. We talked about removing the flag in recon-surf and add only the check for conform min/hires with the conform.py --check_only.

= changes in run_fastsurfer.sh for auto flag and in recon-surf for conform


Dynamic Lights & Colored Lights Core

it seems that when you use Dynamic Lights mod with Colored Lights Core it does not render light from torches or other objects, granted it would be cool to see you support Colored Lights & movement but id just like to see the minor bug fixed =]
Link to Dynamic Light Mod - http://www.minecraftforum.net/forums/mapping-and-modding/minecraft-mods/1272478-dynamic-lights-portable-and-moving-lightsources

Dynamic Lights strips out the color from the lightvalue integers, resulting in light that is rendered "black" by the CLC by any method used inside DynamicLights.
Last I recall, it also uses its own updateLightByType() method to spread light values around in specific instances to facilitate the dynamic movement.  This method is also not compatible with CLC.
I can't fix this one on my end, unless they started relying on vanilla methods again................ or altered their source unnecessarily.

Ok so as far i remember Dynamic Lights uses ASM to override the following line in World.computeLightValue: int blockLight = block.getLightValue(xxx); -> int blockLight = DynamicLights.getLightValue(xxx). Dynamic Lights internally uses the same block.getLightValue(xxx); to get the vanilla value. I'm assuming you have overridden getLightValue(xxx) further "down" the pipeline. My source looks like this: https://code.google.com/p/atomicstrykers-minecraft-mods/source/browse/DynamicLights/src/main/java/atomicstryker/dynamiclights/client/DynamicLights.java#156
I assume i can fix this by either detecting color bits in the int or changing the light.getLightSource().getLightLevel() result to flip its bits into the vanilla int instead of overwriting it. Do you concur?

I have now checked out your repo and you completely replace computeLightValue in your ASM. It appears if this is to be fixed, it has to be done from your side.

wow you both are extremely active and reply fast, thank you both for the support =]
i hope this will be resolved soon!

@AtomicStryker It looks as if I'd need to simply check to see if DynamicLights is loaded, then simply replace my calls to block.getLightValue(world, parX, parY, parZ) with atomicstryker.dynamiclights.client.DynamicLights.getLightValue()
Can you confirm?

That is correct.

@AtomicStryker In both functions?  Or only computeLightValue?

Well, i only hook into the one function, and its enough for Dynamic Lights to work. I can only speculate why you need 2, but since Dynamic Lights internally queries block.getLightValue(xxx); anyway, at worst you add some overhead.

I think I'm reflecting into it incorrectly... looking into this.

Ok... learned a lot... I'm actually referencing the method properly now.
And... something is still stomping on the color bits
https://code.google.com/p/atomicstrykers-minecraft-mods/source/browse/DynamicLights/src/main/java/atomicstryker/dynamiclights/client/IDynamicLightSource.java#34
I don't know who's limiting to 15, but if that happens, then the color bits in the light value are lost.
LightValues are stored as 0BBBB 0GGGG 0RRRR 0LLLL and they're being capped to 0b1111.  A torch which is colored yellow should normally return 01010 01101 01110 01110 (decimal: 341454) and a ceiling is being applied, and causing it to return decimal 15.

15? It's a design choice. Notch said "let there be light, in full integer increments of 0 to 15". The idea was probably to use an efficient data storage of 4 bits

Where exactly is your colored light value being broken? Dynamic Lights should only give you a generic value from 0 to 15, which you then use as brightness to make your RGB value from

You are correct from vanilla's perspective.  The colored light core however, uses the vanilla light value and bit-crams values representing R, G, and B, into the lightvalue.
vanilla: 00000 00000 00000 0LLLL
clc: 0BBBB 0GGGG 0RRRR 0LLLL
So to keep vanilla happy, I've left 0LLLL in there to be safe, and can revert to a vanilla value at any time by bitwise and-ing the value with 0xF.  The rewritten lighting engine however, spreads around the additional colors I've added.  You'll see code segments such as blue_light =  (lightValue >> 15) & 0xF; // gets blue light channel  Which allow the engine to function as expected.
So... the mere fact that I'm receiving a "generic value" 0 to 15 is the entire problem.... I was expecting an integer as high as 507375. (0b1111011110111101111)
When I make calls to atomicstryker.dynamiclights.client.DynamicLights.getLightValue() here: https://github.com/CptSpaceToaster/CptsModdingLight/blob/dev/src/main/java/coloredlightscore/src/helper/CLWorldHelper.java#L444
I get an "uncolored value" limited from 0 to 15.  I'm not certain if the issue is that a line in your mod says "return 15 if the value is above it"  or if there's a line in vanilla's entity classes that does the limiting.

From what i have seen, you overwrite the entire computeLightValue. Now, at the call of this method and your custom implementation of it, you get your bit modified colored int back. My idea for compatibility was to call atomicstryker.dynamiclights.client.DynamicLights.getLightValue() at that point in time, convert the 0-15 int result to a 0-1 float, and use RGB brightness multiplication with your colored int to get a colored dynamic int. Which you then render.

computLightValue will look at the 6 available adjacent blocks, take the highest light value, and then subtract by one.  Compute will basically return what the current block should be.  Instead of calling world.getSavedLighValue() which returns a light value with color bits... I moved to yours, which works ok for blocks, but burning entities, torches on the ground, players holding torches, all have this ceiling.  Is there some entity code your calling to get the light value from a torch when a player holds it?
This is all I can find... And I'm not sure if that's somehow creating this's ceiling.
I would have loved to do what you suggest, but it's not feasible. If I receive a colored light value that has been stripped of its color, and arbitrarily rounded to 15 then I have no clue what it should be normally.  I can't color a lightvalue in this scenario, because my code doesn't know what the source is.  It's asking your code, which is getting it from Entity somehow, and along the way, the color data is squashed.

Ok i think you are approaching this wrong. Dynamic Lights overwrites the "lowest" light result in the pipeline, while you are trying to get it somewhere way higher. I'm looking at your source before you started DL compat pushes and from what i can tell your lowest light retrieval / computation is at https://github.com/CptSpaceToaster/CptsModdingLight/blob/38988fce7b9f59639dd041edb75ecf94f0b3fb66/src/main/java/coloredlightscore/src/helper/CLWorldHelper.java#L156. Now, to get DL compatibility i would put new code above this return, along the lines of
if (dynamicLights != null)
{
float DLBrightness = 15 / getDynamicLight.invoke(xxx); // which calculates a brightness from 0 to 1
float brightness = computeRGBBrightness(currentLight); // which takes all your rgb bits and calculates a combined brightness from 0 to 1
if (brightness < DLBrightness)
{
currentLight = increaseBrightnessRGB(currentLight, DLBrightness-brightness);
}
}

Hmm.... I don't have a method that takes in a float, and adds arbitrary color, nor do I see a way to create one.  The light needs to already be colored by this point. My mod does not operate like yours in which I hooked computeLightValue to point to a map of lights and their color values.  I went as low as the lightValue field in block.  So here when I ask for a block's light value... it's already colored.  I literally crammed color bits into the vanilla field.  I hope that's clear.

The "lowest" light level from my perspective is the Block.lightValue field.
My mod works on the basis that Block.lightValue can be most integers in the range 0 to 507375.
World.getSavedLightValue, and getDynamicLight.invoke(xxx) both rely on the field that ends up there.
Some code somewhere down the getDynamicLight.invoke(xxx) path sees that the Block.lightValue is higher than 15, and caps it to 15, thus removing the color.

From what I understand, you're proposing a solution that can't work. The ceiling put in place coming from getDynamicLight.invoke(xxx) fundamentally ruins CLC's operation, and I have no method by which I can computeRGBBrightness() as you suggest.
Example: A torch should return 341454, and the lightValue for a torch will yield is 341454.  In CLC, I ask for a Block's lightValue, and I could receive 341454.  Additionally, asking world.getSavedLightValue() will also yield the bit-packed lightValue.  However your end of things limits it.  There isn't a point where I ask "What type of block is this?" and then say "Oh, its a torch, quick color it"

But that is my point! You do not overwrite your colored int with my 0-15 value. You only USE it to change the luminance of your colored int. Wait, isn't this https://github.com/CptSpaceToaster/CptsModdingLight/blob/38988fce7b9f59639dd041edb75ecf94f0b3fb66/src/main/java/coloredlightscore/src/helper/CLWorldHelper.java#L126 the luminance even? Just alter it to
int ll = dynamicLights == null ? neighborLight & 0x0000F : getDynamicLights.invoke(xxx);

You do not overwrite your colored int with my 0-15 value.
No I do.

Player holds torch
torch has lightvalue of 341454
getDynamicLight.invoke(xxx) does not return 341454, nor does it return 14... it returns 15.  Note, that it rounded a torch from 14 to 15!
Since the color is missing, the light coming from entities will be uncolored

I do not work with luminescence as a concept.  The shader only uses R, G, and B.   If a light is white, then it better have R=15, G=15, and B=15.  I keep the L value around for backwards compatibility, so going so far as to only replace the L component with whatever comes from your mod still won't have the desired effect.

https://github.com/CptSpaceToaster/CptsModdingLight/blob/dev/src/main/java/coloredlightscore/src/helper/CLWorldHelper.java#L104
What you're kinda suggesting... to "add color" to the lightvalue can occur on line 106, but that doesn't make a blue light look blue.

If a block is blue, it will be colored blue no problem
If a block has no color, then it will be colored white (bug fix for "black colored lights" from other mods)
If a player holds a blue light, then we still have our bug, because your end would simply say that it's returning 15... and then I would color it white because I have no idea what the original color was supposed to be.

Eh?  Am I making any sense now?

I understand. Whether or not you want to make all dynamic lights white or not (i would suggest doing so) is your decision. This was about making our mods compatible.

So, I wanted to spread colored light around if a player was holding a colored light source...
Second, there's still a bug here, as I'm getting unexpected values from your end. (Either due to rounding issues, or altered values.
Oh... was it intentional for a held piece of glowstone emits light level 12?  I thought you would pull from the block.lightValue directly!  That was kinda my entire spiel... if you were, we'd have the ultimate dream of a player walking around with blue glowstone, lighting up the area blue!

why not make it go both ways and let the player decide on what he/she would pref through one of the configs?
1 setting to use dynamic lights custom ranges or
2 colored lights without custom light ranges?
i could understand why you guys wouldnt want todo this, coding is a pain in the ass & why i gave it up in highschool XD

Dynamic Lights doesn't use the lightValue off blocks, correct. It was only ever configurable. Of course, you could implement your own module of Dynamic Lights and just turn my modules off ... in retrospect that solution might have been easier than all of this.

Alrighty... so it's either
1.) Learn a lot about your setup, hook into some sort of API/module system and disable everything (I presume I would then send through the default value myself in their place)
2.) Ask you to create a config option to permit Block.lightvalue through instead of the remapped colors...
Looks like one of us is going to have to do some work here.  I'd be willing to do the work if you really don't want to open up a config option up in your mod, but a couple pointers on what I'm even looking to do with your module system would be nice.

Hmm i guess i could, maybe. But the thing is, your int values as such do not indicate which one is "strongest". I cant simply compare if (a < b), because a and b might be more or less random integers as a result of your bit flipping. I still think white light from dynamic light sources is a passable compromise.

Ooop!  Not quite.  I've thought through this one, as I needed to compare one light source against another.  If LightValueA has any components greater than LightValueB, then I'd do the following:
if ((((0x100000 | LightValueB) - LightValueA) & 0x84210) > 0) { //LightValueA has components that are larger than LightValueB

Bitwise hackery!  Basically, we subtract the two light values, and then look to see if there's anything in the carry-over bits.  (the zero in between each light value)
So that address THAT concern...
This might also be useful

I'm more than willing to fill out the API if you need this sort of functionality more readily available.  The API is simply a collection of methods that work with the lighting-words.  You can include it safely without forcing a hard-dep, but if it's not helpful, then I'd like to know :)

https://code.google.com/p/atomicstrykers-minecraft-mods/source/detail?r=cdd3e5ff65bfb6eb1cec46c9d6a37cde9fbc1409

Tyvm: Quick comment:
https://code.google.com/p/atomicstrykers-minecraft-mods/source/browse/DynamicLights/src/main/java/atomicstryker/dynamiclights/client/ItemConfigHelper.java?spec=svncdd3e5ff65bfb6eb1cec46c9d6a37cde9fbc1409&r=cdd3e5ff65bfb6eb1cec46c9d6a37cde9fbc1409#60
I think a call to block.getLightValue(IBlockAccess world, int x, int y, int z) would be a more reliable.  The CLBlock interface shows how one might color their own lightvalue based on metadata, but I don't expect all blocks to implement CLBlock.  It's a suggestion, not a mandatory inclusion.
CLBlock simply wraps block.getLightValue(IBlockAccess world, int x, int y, int z) anyways, so you should be safe just calling that instead.  Then, your wildcard can work for other lights as well!
Thank you very much!

Ah, but where do i xyz coordinates of a block which is not necessarily present anywhere in the world

Exactly... there isn't a good solution, because vanilla doesn't support different instances of LightValue per metadata... I simply hacked that in.
You can't cache the results from block.getLightValue(IBlockAccess world, int x, int y, int z) when you parse your config... so this is honestly the best solution... even if I made a block.getLightValueByMeta() method... noone know would to use it unless I force them (same issue with implementing CLBlock)
Considering CLBlock is an abstract... that makes it even more invasive... I might have to change CLBlock to an interface that just tells you to implement the method you use.  Force everyone to use that interface... and then... move the little jig to block.getLightValue(IBlockAccess world, int x, int y, int z) to another abstract class "EasyColoredLight" or something.
Also... I see what you did when the entity exists in the same location as a torch... there's going to be an outstanding issue on that till I can fix my lighting engine...
If I read it correctly, if I stand in a yellow torch while holding a blue light, I will temporarily replace the torch's yellow light with my blue light (because the check sees that the blue light has components that the torch doesn't)  What you need, is to "add both lights together" so the torchlight and blue light are grabbed together (and it just grabs the brightest color component of each for a bit)
So I'll add that to the API... that's easy for me to do, and who knows what my lighting engine will do once you patch it in...
Consider it a non-issue till I can fix https://github.com/CptSpaceToaster/CptsModdingLight/issues/39
tyvm AtomicStryker!  You've been amazing, and I really appreciate everything you've done.

same here both of you offer the best light mods out there, im really glad to see you guys work this out.


Merge Development into Master

Merges changes in the development branch into master.
Test Server is here
https://young-woodland-76564.herokuapp.com/
@aayusharora @mariobehling Please check and review. Thanks!!

Current coverage is 31.98% (diff: 31.98%)

No coverage report found for master at fa685a5.

Powered by Codecov. Last update fa685a5...c2b9aa5

@princu There seems a lot of conflicts while checking this on local . Can you recheck on local with development merging master. We need to resolve those conflicts so that the development branch changes remain intact :)

@Princu7 Have you removed the Conflicts on local ?
Can you please explain the procedure, this can results in bad things if not handled properly.

@aayusharora Yes, I removed all the conflicts locally. First I checked out the master branch of this repository into my local repository.  Upstream is the remote of this repository.
git checkout -t -b master upstream/master
This created a master branch in my local repo with the contents of master branch of this repository. Then I just ran the command
git merge --squash upstream/development
There were a lot of different types of conflicts(content conflict, modify/delete conflict, delete/modify conflict). I resolved them all and then created the request to the master branch. :sweat_smile:
All the unit tests seem to pass and there seem to be no merge conflicts in this pull request. I also deployed a test server on Heroku which I made of my master branch itself. Please guide if something is incorrectly done. Thanks.

OK Looks good !! Merging this :)


Tracking evaluation module

Adds new Tracking Evaluation Module
Closes #32 and Closes #30

TODO:

[x] Support loading partial volumes, tracking data, and GT tracking data into Slicer.
[x] Visualize the movement of the partial volumes.
[x] Fix issue with transforms
[x] Visualize the difference between the tracking data and GT data (see diagram above).

[x] Models should be green iff the difference is within some tolerance (ie. 1mm or 2 deg).
[x] Models should be red otherwise

[x] Update the current frame to be a spin box
[x] Add reverse option for playing sequence
[x] Add pause button
[x] Update the tutorial or create a new one to cover the usage of this module.
[ ] Update the Sample data to include GT tracking data (Update the Ankle filter for #30 too).

[ ] Need to update the release in autoscoper main.
[ ] Need to update checksum verification in SA^M:
https://github.com/BrownBiomechanics/SlicerAutoscoperM/blob/13ae181309b949420dc06c1bac2f894d63f3a3f9/AutoscoperM/AutoscoperM.py#L104-L116

@amymmorton You are good to start testing this branch.
An updated tutorial on how to use the module can be found here.
Note: Only the Wrist and Ankle datasets have been updated. Still waiting on the knee.

inner failed to compile 2x..
trying from fresh clone .. will update

@amymmorton If you could upload the build log (from the output tab of visual studio) that would be much appreciated.

got it..

Topic has been re-based along with integrating the following changes:

renamed variable

like gtVarName to grountTruthVarName
like seqSomething to sequenceSometing
colNames -> columnNames
fName -> fileName

renamed method

updateColor -> setColor
updateGtVisibility -> setGroundTruthVisible

fixed typos reported by the newly integrated codespell pre-commit check
re-ordered imports
removed unused parameter node member like inputVolume, imageThreshold, ...
removed obsolete setting of inputVolume in initializeParameterNode()
removed -> None (having them included them in the Slicer module template is an oversight)
fixed sampleDataType to use the updated folder names prefixed with 2023-08-01
removed unneeded update of sys.path in data/__init__.py
update annotation to use list instead of  List
prefer f-string formatting

Considering that I am able to load the .tra file exported selecting All and compare it against the relevant "ground truth" data set, I am moving forward with the integration.
Additional improvements will be integrated in follow-up pull request.


VM group change dilaog is cut off

Steps to reproduce

Go to VM list
Choose VM "dockerswarmtest"
Click on Group change button in a Sidebar

Expected result: A dialog is displayed properly
Actual result: Unneeded scroll appears, a dialog is cut off

the same is for Editing Firewall templates (horizontal scroll)

Images -> ISO -> Sidebar

New drive creation. Unnecessary scroll


Fix schema compare not always showing diff color highlighting

This PR fixes #21368. Some change differences weren't getting the color decorations correctly showing. The error in the console showing that an illegal line number was trying to be used in GetFirstNonWhitespaceColumn, which then caused the diff color highlighting to not show up correctly for those differences.
There's been some churn in this file since the reverse option was originally added a few years ago, so I decided to change the approach for reversing the colors rather than trying to fix the existing reverse logic. Previously, the line changes and editors were being swapped, but that seems more complicated and risky, and led to this bug happening. The new approach is to swap the line color and zone decorations after the decorations have already been determined, which will avoid the error that was happening because the color reversing is now happening after that.
Before:

Notice the differences not getting highlighted correctly for the change difference (3rd one in the table):

Fixed:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 42.162%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
28572

Relevant Lines:
62937

💛  - Coveralls


Add Support for MacOS

Close #9

Ideally, I'd still like to do the yuvk -> rgba conversion in a separate shader to avoid redundancy in the other shaders, but I think I've already used up the budget.

Since you changed the shaders, did you validate all packing conversions in all supported video resolutions?

I have validated my development (including all pixel formats, especially YUV 422 10-bit) on the following resolutions:
720x480
720x576
1280x720
1920x1080
2048x1536
3840x2160
4096x2160
All formats are working as expected, except for YUV 422 10-bit with MSB issues on 720p, 2048x1536, and 4096x2160. Upon investigation, I found that this issue was already present prior to my changes.
I will explore whether specific shaders can be introduced for these formats, as we have done previously in dScope.

Since you changed the shaders, did you validate all packing conversions in all supported video resolutions?

I have validated my development (including all pixel formats, especially YUV 422 10-bit) on the following resolutions:
720x480 720x576 1280x720 1920x1080 2048x1536 3840x2160 4096x2160 All formats are working as expected, except for YUV 422 10-bit with MSB issues on 720p, 2048x1536, and 4096x2160. Upon investigation, I found that this issue was already present prior to my changes.
I will explore whether specific shaders can be introduced for these formats, as we have done previously in dScope.

Since I wasn't able to implement this quickly enough, I created an issue to address the problem later, in accordance with our discussion.


Problem sorting after upgrading to 8.x

After upgrading to laravel-datatables 8 and datatables.net 1.10.15 I have problems with ordering a column. I use several joins to get my data which creates the situation that certain column names become ambiguous when querying without the table name.
I try to sort on the created_at column on the students table.
My backend code:
$students = Students::join(... // join all tables

$students->select('students.created_at', // ... other columns, etc.

return \Datatables::of($students)
            ->editColumn('students.created_at', '{!! \Carbon\Carbon::parse($created_at)->toDateString() !!}')
            ->make(TRUE);

Front-end code:
columns: [
            {data: 'created_at', name: 'students.created_at'},
            // other columns

When sorting on created_at I get the error:
Exception Message:↵↵SQLSTATE[HY000]: General error: 1 ambiguous column name: created_at (SQL: select "students"."created_at", etc...

and the query ends with order by "created_at" desc.
Why does the order function remove the table name when creating the query?

Windows 10
PHP 7.1
Laravel 5.5
Laravel-Datatables 8.0.0

It seems like the current version always assumes that a column name with 2 or more parts uses eager loading.
Will send a patch to fix this. Thanks for reporting.

Patch released on v8.0.1. Kindly update again and confirm. Thanks!

I can confirm that v8.0.1 works as expected, thanks for the quick response!


Multiple SharePoint organizations sync

Description
I have following OneDrive synced folders:

Personal OneDrive
Work OneDrive
Multiple folders from my work SharePoint
External organization SharePoint site synced
Unfortunately I do not see my work organization SharePoint in Cloud Drives but only the external one because it was added last I think.

Steps To Reproduce

Sync folder from 1 SharePoint organization
Sync folder from 2nd SharePoint organization
First SharePoint organization is not seen in Cloud Drives menu

Requirements

e.g. Fix crash that occurs when...
add ability to show multiple SharePoint organizations in Cloud Drives.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2215
Log File
debug.log

Thanks for the feedback, Merging with #4966. You can check there or click subscribe in the right sidebar for updates.


build: remove workaround for ZIP64 in Gradle build

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


Add CNI install to create workload task

Description of changes
CNI installers are now stateful, being constructed with all configuration injected at build time. Only dynamic objects like the context or the cluster authentication are provided as runtime method argument. They all build on top of a stateless version, intended for other usecases where the cluster spec doesn't exist at runtime.
Part of the refactor to move to the new workflow paradigm.
Testing
Unit tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Kotlin Native Compatibility

Is it possible to consider keeping Kotlin Native compatibility at least for the core system minus Java specific external systems integrations like CEP and jTALib.

For now will close, but will put some item on backlog to check the GraalVM options in the future.

Closing


[FR]: Support for other cross-seeding apps (i.e Crops and Pollenizer) (--cross-seed) folder inject

Is your feature request related to a problem? Please elaborate.
I use other cross-seed apps like Crops and Pollenizer which are designed for specific private trackers since they don't make as many requests like cross-seed does. I have cross-seed set up to inject directly into qBittorrent for other private trackers. The cross-seed folder that qBit_manage uses may be useful for other cross-seeding apps that don't support direct injection. Currently I use it as a watch folder in qBittorrent since qBit_manage does not support torrent files generated by these apps.
Describe the solution you'd like
Please add cross-seed support for Crops and Pollenizer, etc.
Does your solution involve any of the following?

[ ] New config option
[ ] New command option

Describe alternatives you've considered
NA
Who will this benefit?
Crops, Pollenizer and other cross-seeding apps users.
Additional Information
File name examples generated by Crops:
1966 - Gabor Szabo - Jazz Raga [2010 WEB FLAC][Verve Reissues][602527584546] [RED].torrent
A_Teens - Teen Spirit (2001) [FLAC] {013 401-2} [RED].torrent
Avicii - Tim (2019) [FLAC] {B0030536-02} [RED].torrent
Basshunter - Now You're Gone - The Album (2008) [WEB FLAC] [RED].torrent
Bastille - Bad Blood X (2013 - 2023 Reissue) {5565281} [CD FLAC] [RED].torrent
Rush - 2112 (1976) {B0017747-00, BD, 2.0} [FLAC 96-24] [OPS].torrent
UB40 – The Very Best Of UB40 1980 - 2000 (2000) [7243 8 50469 2 6] {CD-FLAC} [OPS].torrent
Loraine - An Autumn Evening (2023) - WEB MP3 320 [OPS].torrent

File name examples generated by Pollenizer:
ops.2575826.torrent
red.1043479.torrent
red.4729433.torrent

Please explain and provide trace logs how the existing --cross-seed function does not work.
https://github.com/StuffAnThings/qbit_manage/blob/46518577f956fcbf2b18d36c4e9c6d3908b01a95/modules/core/cross_seed.py#L24
"add torrents from the cross-seed output folder to Qbittorrent"

I just get parse errors because the file names are in a different format
|================================= Checking for Cross-Seed Torrents =================================|
| Uncaught Exception                                                                                |
Traceback (most recent call last):
  File "/app/qbit_manage.py", line 643, in <module>
    start_loop()
  File "/app/qbit_manage.py", line 382, in start_loop
    start()
  File "/app/qbit_manage.py", line 460, in start
    cross_seed = CrossSeed(qbit_manager)
                 ^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/modules/core/cross_seed.py", line 21, in __init__
    self.cross_seed()
  File "/app/modules/core/cross_seed.py", line 37, in cross_seed
    tr_name = file.split("]", 2)[2].split(".torrent")[0]
              ~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

In v6, cross-seed will watch a directory for .torrent files and try to match and inject.
It hasn't been tested for your use case but it should work.


Fixed bug in puma config.

coverage: 94.421%. remained the same
when pulling eb6e857a71d90941a45e724ee9dc52cd5745537b on fix-puma-config
into 353d7b495a356bb7f9877545dea16164371a396c on main.


Fixed #953: protocols() value in @Api and @ApiOperation should change and be parsed

Fixed #953: protocols() value in @Api and @ApiOperation should change and be parsed

Generally looks good. A few comments/questions:

Please remove the javadoc changes for the annotations. I have a pending PR with a whole rework of the javadocs as a whole, including that.
What happens if a user chooses to add an invalid scheme (say, ftp). Dry running it makes it seem like it would add null to the final schemes list. Possibly should add a test for that as well.
For completeness of testing, there should be a test with no protocols at either level, with an output of no schemes property at all (as opposed to a schemes property with an empty array for example).
The change to the schemes in the Swagger.java class is a bit out of scope for this PR as it has nothing to do with the @Api or @ApiOperation protocols. It's ok to keep it though.
Can't think of a reason to use LinkedHashSet over HashSet in this use case. Not really important, just a general thing.


add fetcher support for crates.io

Should be similar to the npm fetcher implimented in bitbake-npm.
Would allow us to avoid using git sources when patching up cargo, and could make it much more convenient to build some packages.

So cargo fetch and cargo generate-lockfile should be all the magic necessary to fetch everything required for a package to build.


Experimental - Merlin: 'Too many open files' error on OSX

It looks like we're not properly disposing of file descriptors when working with merlin on OSX.
I see a crash with this error message:
Fatal error: exception Sys_error("Too many open files")

after some editing.

Fixed with #864


Linking Zenodo token 500

Describe the bug
Going to https://dockstore.org/auth/zenodo.org?error=invalid_request&error_description=Missing%20response_type%20parameter should not produce a NPE or 500 error, BAD REQUEST is more suitable
To Reproduce
Steps to reproduce the behavior:

Log into Dockstore.org
Put https://dockstore.org/auth/zenodo.org?error=invalid_request&error_description=Missing%20response_type%20parameter in your browser
See 500

Expected behavior
No NPE, no 500
Additional context

UI version: 2.6.1
Webservice version: 1.9.1
host location: dockstore.org

┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.11
┆Issue Number: DOCK-1611
┆Sprint: Backlog
┆Issue Type: Story

Looks out of date, getting a 404 on 1.10.1


Split GraphService from XRefService [phab:D1941]

Created by schroederc at 2017-09-21 22:17:44:
All GraphService code from kythe/go/{services,serving}/xrefs has been
separated into the kythe/go/{services,serving}/graph packages.

schroederc wrote at 2017-09-21 22:26:24:
Fix some BUILD dependencies

fromberger accepted this revision at 2017-09-22 23:27:03

schroederc closed via commit f05ecd7f0971754ca51989bb1e0a0836a85a211b at 2017-09-25 15:59:18


[v1.5.x] [MKLDNN] Independent gradients requests check with respect to weights…

… and bias of convolution (#15497)

Independent req[kBias] and req[kWeight] check

Add UT for independent conv gradient requests

Update conv independent grad UT with no_bias enabled

Check req[kWeight] for avoiding unnecessary prim registration

Check OpReqTpye in CommitOutput automatically

Lock cudnn autotune for accurate conv output

Ignore independent gradients test on GPU

Trigger CI

Sets a low bar for autotuned cudnn convolution

Description
cherry-picking PR#15497
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@zixuanweeei This PR is cherrypicking your commit merged on master branch for v1.5.1, could you please review? Thanks.

@zixuanweeei thanks for the information, the one for test case changes also taking, please kindly review.
@TaoLv

LGTM. Thanks.


only remove msvc10 64bits builds if appveyor flags is set to true

Added a flag to say that we are in Appveyor mode and we need to remove "visual_studio"/"10"/"x86_64" builds
its true by default so it keeps the same workflow by default.
We need this because our internal CI can build these builds

I like a flag to add the x86 builds for visual 10, but it's not only related with appveyor. I haven't visual 10 with x86 support either. I think is the typical (free) environment. Maybe "vs10_x86_enabled" with False by default?

I will provide a modified version then, I thought this was related to appveyor and travis...

The build ignored was 64bits aka x86_64

Yes, it was x86_64, too many thing in my brain hahaha. Good job!


Cleanup

What's new
Global

Update third party licenses declaration: remove no longer used, add new one from refactoring, ...
(Closes #94)
Rename AUTHORS.md to CONTRIBUTORS.md

Rust

Update source files header
Remove old code remaining from old architecture
Adjust CI/CD files to the new project architecture

How to test

Check that all actions ran successfully

@ymorin-orange Typo fixed, just added a new commit 28b6af6 to rename the package (and crate) back to libits-client in order to keep the crate history rather than publishing a new one (directly at version 2.0.0)

@ymorin-orange Typo fixed, just added a new commit 28b6af6 to rename the package (and crate) back to libits-client in order to keep the crate history rather than publishing a new one (directly at version 2.0.0)

Thanks. Still ACK from me; ready for merge.


Change all alerts to modals

Change all alerts to modals as alerts aren't allowed

see add-modals branch


Modify design

This pull request modified design.md based on the coming features.
The added contents are as follows

add deleting LogicalVolume  process
add how to handle PersistentVolumeClaim when the corresponding node is deleted.

Also I added  topolvm-controller.md document and filled what topolvm-controller is.
The detail is still empty because filling the detail is a part of this issue.

Sorry. I will add description.

Also I added topolvm-controller.md document and filled what topolvm-controller is.
The detail is still empty because filling the detail is a part of this issue.

So, please do this in https://github.com/cybozu-go/neco/issues/532 .
Closing.


TextureUtil, GlConst and GLX are still SRG names

TextureUtil, GlConst and GLX are still SRG names. Only RenderSystem and GlStateManager are remapped to mojang/parchment names in the development environment.
Forge Extension.

Adding onto this add in ListTag would fail to remap mods from srg to named since there is a mapping with c add c c in the srg tiny

Should be fixed with my SrgMerger rewrite.


Misc

Miscellaneous updates to the template

fixes #5 , fixes #7 , fixes #8
prompts for local git.config.user settings
initializes dvc

It seems a bit unintuitive that both utilities and models are stored in src. How about separate top-level models folder?

It seems a bit unintuitive that both utilities and models are stored in src. How about separate top-level models folder?

Both models and utilities are source code, so I think it is quite clear

Both models and utilities are source code, so I think it is quite clear

Are you talking right now about code of model class or about dumped trained models? Because I mean dumped ones, and since they are formally a code but really just some pickled stuff, it makes sense storing them separately from "real" code files.

Are you talking right now about code of model class or about dumped trained models? Because I mean dumped ones, and since they are formally a code but really just some pickled stuff, it makes sense storing them separately from "real" code files.

I see what you mean and fully agree.
You proposed adding an upper-level directory for storing serialized models. Another option would be to keep them near the notebooks where they are described, that is, in notebooks dir for research-stage models that we want to save and in overview dir for the ones that are used for demonstration.
Anyway, this seems a matter of a broader discussion, that we should have when we are all back n office again :)
I propose not to make any changes related to this in this PR

Let's rename the repo into something like "ML project template"


custom secret token breaks the release workflow

Describe the bug
release.yml doesn't work with a custom secret token.
Breaking commit - 9bc2d492dd3d7c2bff85da4121d5266c01b33085
To Reproduce
Steps to reproduce the behavior:

Set up a Repository secret token in settings under Secrets>Repository secrets
Change github-token: ${{ secrets.GITHUB_TOKEN }} to github-token: ${{ secrets.CHANGELOG_RELEASE }} under env: of name: create release
Commit with prefix fix:  or feat: in the commit name
Check the error in actions

Expected behavior
Example error from the breaking commit
In the actions tab, you will see a failing action with error-
Run actions/create-release@v1
Error: Bad credentials

Screenshots

Additional context

Set up a Repository secret token in settings under Secrets>Repository secrets

I'm not sure if this is the right way to add a custom Repository secret

Example error from the breaking commit In the actions tab, you will see a failing action with error-

The issue seems to be with actions/create-release@v1 because<EMAIL_ADDRESS>works completely fine with github-token: ${{ secrets.CHANGELOG_RELEASE }}

Workflow syntax - GitHub docs

env
A map of environment variables that are available to the steps of all jobs in the workflow.

However i dont see any docs on how to set env variables in Learn GitHub Actions/Environment variables
I configured my repository secret by floowing as mentioned here - Creating encrypted secrets for a repository
I also tried making an environment and set up CHANGELOG_RELEASE but didn't work- docs


Seq2ffv1 batch processing

We can chat here :)
So anyhow, you should be able to edit seq2ffv1 to enable batch processing, or make a brand new script that launches seq2ffv1 in a batch.

Thanks :D
Can I use sipcreator.py and batchsipcreator.py as references to make this?

Sounds good to me.
On Mon 29 Nov 2021 at 18:32, He Yazhou @.***> wrote:

Thanks :D
Can I use sipcreator.py and batchsipcreator.py as references to make this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Irish-Film-Institute/IFIscripts/issues/6#issuecomment-981903747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAITFPRVBG4KFXEK5XKSD3LUOPBMXANCNFSM5I7ZYIUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi I found a bit difficult to pick up python, and honestly it's too professional to read them. So just just make a copy of seq2ffv1.py and add a 'oe' flag to input in the command line so that I can input all info first and use && for several of them. :D

How many do you need to do?
&& always helps lol
On Tue 30 Nov 2021 at 19:46, He Yazhou @.***> wrote:

Hi I found a bit difficult to pick up python, and honestly it's too
professional to read them. So just just make a copy of seq2ffv1.py and add
a 'oe' flag to input in the command line so that I can input all info first
and use && for several of them. :D
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Irish-Film-Institute/IFIscripts/issues/6#issuecomment-982962881,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAITFPQL623KL3INMGDAND3UOUS2BANCNFSM5I7ZYIUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Total amount is more than 100, but each batch could be less than 70. I’m thinking to make a loop or just use &&

Just to update, I made a seq2ffv2.py lol and wrote a bash for oe=${oe+1}


.Values.clusterMembers is undocumented

It's used here:
https://github.com/rancher/cluster-template-examples/blob/dc5c201c5415c430cdf4bcb6af975f9e516e970b/charts/templates/clusterroletemplatebinding.yaml#L2
Should be documented in the values.yaml (or values-*.yaml, wherever it makes more sense).

Never mind, it is documented here. I just missed it