Use a client logger (if provided) for the ExpressReceiver InstallProvider

Summary
Ensure the ExpressReceiver's creation of the InstallProvider also uses the client provided logger.
Currently, if there's an error in an ExpressReceiver InstallProvider, the default logger is used regardless of whether a custom logger is provided in the installer options.
Fixes https://github.com/slackapi/bolt-js/issues/620
Example of my json transport logs inner mixed with the default bolt console logger....
{"level":"DEBUG","message":"apiCall('oauth.v2.access') start","line":148,"function":"apiCall","file":"node_modules/<EMAIL_ADDRESS>{"level":"DEBUG","message":"will perform http request","line":308,"function":null,"file":"node_modules/<EMAIL_ADDRESS>{"level":"DEBUG","message":"http response received","line":316,"function":null,"file":"node_modules/<EMAIL_ADDRESS>[ERROR]  OAuth:InstallProvider:0 [Error: An API error occurred: invalid_code] {
  code: 'slack_webapi_platform_error',
  data: { ok: false, error: 'invalid_code', response_metadata: {} }
}

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for sending this in! I have a few tiny changes to help make this code handle more use-cases!
Firstly, it would be good to pass in logLevel when defining InstallProvider. See https://github.com/slackapi/bolt-js/blob/feat-socket-mode/src/ExpressReceiver.ts#L116-L117
Secondly, I think it makes sense to update the default value for the logger argument to undefined instead of what it is currently being set to: new ConsoleLogger(). That way, if no logger was passed, we would pass that same value to InstallProvider. Instead we can set this.logger = new ConsoleLogger() near the start of the constructor. See https://github.com/slackapi/bolt-js/blob/feat-socket-mode/src/ExpressReceiver.ts#L69-L89.
Another thing to note in the above code is I also added logLevel as a constructor argument. That would be a good addition here as well. Will need to import LogLevel above from @slack/logger and also add it to ExpressReceiverOptions
Let me know if you can make those changes or have any questions!
Thanks again for reporting this bug and sending it in!

Tiny note: it would be cool to see some of this new behavior verified with a test.


Some cleaning

Remove  deadcode

Good work @ d0p1s4m4! ✅
Thank a lot for cleaning, Indeed, there are still quite a few pieces of unused prototypes of olders previous OS2 versions


[Autocomplete] with virtualized have horizontal scroll in codesanbox

Hello, i try using autocomplete with react-window. I take your example in docs, there all well, but when i try use example in codesandbox i get horizontal scroll in popup.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Have horizontal scroll

Expected Behavior 🤔
Don't have horizontal scroll

Steps to Reproduce 🕹

Your example with scroll https://codesandbox.io/s/material-demo-7g677
Steps:

click on field
see scroll

Context 🔦
i had tried use autocomplete with react window

Your Environment 🌎
i use latest environment witch get with your codesandbox

@gespispace Thanks for reporting this bug. I believe we could apply the following diff:
diff --git a/packages/material-ui-lab/src/Autocomplete/Autocomplete.js b/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
index 45f6d6e4a8..791ac759c9 100644
--- a/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
+++ b/packages/material-ui-lab/src/Autocomplete/Autocomplete.js
@@ -173,6 +173,7 @@ export const styles = theme => ({
     alignItems: 'center',
     cursor: 'pointer',
     paddingTop: 6,
+    boxSizing: 'border-box',
     outline: '0',
     WebkitTapHighlightColor: 'transparent',
     paddingBottom: 6,

Do you want to work on it? Does it solve your problem? :)

It was not clear for me if @gespispace is going to try to make pull request or if the thumbsup means that this solves the problem so i'm going to go ahead and place my pull request
please let me know if i should close it

@MariyaVdovenko i tried make pull request, but diff which offer @oliviertassinari not work for me and your pr probably don't solver bug. Can you test you pr with example from codesandbox again?

@gespispace I think this might be as easy as passing a style prop to VariableSizeList of             style={{ overflowX: "hidden" }}
          <VariableSizeList
            itemData={itemData}
            height={getHeight() + 2 * LISTBOX_PADDING}
            width="100%"
            key={itemCount}
            style={{ overflowX: "hidden" }}
            outerElementType={OuterElementType}
            innerElementType="ul"
            itemSize={index => getChildSize(itemData[index])}
            overscanCount={5}
            itemCount={itemCount}
          >
            {renderRow}
          </VariableSizeList>

@oliviertassinari Would there be value in a contribution outside of this PR that puts the options into React-Window

@hoop71 Thanks, it really hide horizontal scroll. @MariyaVdovenko can you update your pr with suggestion @hoop71?


Propertly handle an InlineUIContainer's logical children

What does the pull request do?

This PR makes InlineUIContainer the direct logical parent of its child instead of attaching the child to the host TextBlock
What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0029725-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


cue/def: --inline-imports flag does not work together with --path flag

What version of CUE are you using?
v0.0.0-20230202180031-576d0e461a99
I built this via go build from the "v0.5.0-beta.5" Git tag.
Does this issue reproduce with the latest release?
Yes, insofar as version "v0.5.0-beta.5" is the latest release available.
What did you do?
cue def merge.cue age.json --path 'merged:' --inline-imports

-- age.json --
18
-- cue.mod/module.cue --
module: "example.com"
-- cue.mod/pkg/other-example.com/color/color.cue --
package color

import (
	"math"
)

[!=""]: {
	rgb8: >=0 & <math.Pow(2, 3*8)
}

blue: rgb8:  0x0000FF
green: rgb8: 0x00FF00
red: rgb8:   0xFF0000
-- merge.cue --
package merged

import (
	"other-example.com/color"
)

merged: uint

favorite: color.blue.rgb8

What did you expect to see?
The pertinent content of file color.cue imported from other-example.com/color should be included directly in the output, and the import declaration for other-example.com/color should no longer be present.
What did you see instead?
cue def preserves the import declaration for other-example.com/color in the output.
Note, though, that if we don't merge in the file age.json using the --path flag, the --inline-imports flag does have the intended effect. In other words, it appears that use of --path—and maybe also any merging of non-CUE files—defeats the --inline-imports flag.

Further testing shows that it's not the --path flag that defeats --inline-imports, but rather any merging of non-CUE files, regardless of whether we use the --path flag or not.


Unit Testing Errors on Custom PropertyValidators

When I try to run this test:
(I can confirm provider is not null)
provider = _testingRepo.GetProvider(1);
ValidationResult validationResult = validator.Validate(provider);
Test Name:	YesErrorWhenNameIsNull
Test FullName:	NEW_ECAP_API.Tests.Validators.ProviderValidatorTest.YesErrorWhenNameIsNull
Test Source:	{my machine}\NEW_ECAP_API.Tests\Validators\ProviderValidatorTest.cs : line 33
Test Outcome:	Failed
Test Duration:	0:00:00.4250637
Result Message:
Test method NEW_ECAP_API.Tests.Validators.ProviderValidatorTest.YesErrorWhenNameIsNull threw exception:
System.NullReferenceException: Object reference not set to an instance of an object.
Result StackTrace:
at NEW_ECAP_API.Data.Validators.CustomDateTimeISO8601Validator1.IsValid(PropertyValidatorContext context) in {my machine}\NEW_ECAP_API.Data\Validators\CustomISO8601Validator.cs:line 22    at FluentValidation.Validators.PropertyValidator.Validate(PropertyValidatorContext context) in c:\Projects\FluentValidation\src\FluentValidation\Validators\PropertyValidator.cs:line 71    at FluentValidation.Internal.PropertyRule.<Validate>d__10.MoveNext() in c:\Projects\FluentValidation\src\FluentValidation\Internal\PropertyRule.cs:line 241    at System.Linq.Enumerable.<SelectManyIterator>d__142.MoveNext()
at System.Collections.Generic.List1..ctor(IEnumerable1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)    at FluentValidation.AbstractValidator1.Validate(ValidationContext1 context) in c:\Projects\FluentValidation\src\FluentValidation\AbstractValidator.cs:line 113    at FluentValidation.Validators.ChildCollectionValidatorAdaptor.<Validate>b__5(Tuple2 tuple) in c:\Projects\FluentValidation\src\FluentValidation\Validators\ChildCollectionValidatorAdaptor.cs:line 48
at System.Linq.Enumerable.WhereSelectEnumerableIterator2.MoveNext()    at System.Linq.Enumerable.<SelectManyIterator>d__142.MoveNext()
at FluentValidation.Internal.PropertyRule.d__10.MoveNext() in c:\Projects\FluentValidation\src\FluentValidation\Internal\PropertyRule.cs:line 245
at System.Linq.Enumerable.d__142.MoveNext()    at System.Collections.Generic.List1..ctor(IEnumerable1 collection)    at System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)
at FluentValidation.AbstractValidator1.Validate(ValidationContext1 context) in c:\Projects\FluentValidation\src\FluentValidation\AbstractValidator.cs:line 113
at NEW_ECAP_API.Tests.Validators.ProviderValidatorTest.YesErrorWhenNameIsNull() in {my machine}\NEW_ECAP_API.Tests\Validators\ProviderValidatorTest.cs:line 37

So it looks like you're calling the wrong overload of Validate - there's one that takes the instance to validate, and there's one that takes a ValidationContext. I really need to see a full code sample though to see what's going on.  What's the type of "provider"? How is the validator defined?

REALLY appreciate your help, Jeremy!

The type of provider object:
'Provider'

The validator class:
public class ProviderValidator : AbstractValidator
{
public ProviderValidator()
{
...
The unit test: (I've tried both ways and am getting null error)
[TestClass]
public class ProviderValidatorTest : BaseApiTest
{
    private ProviderValidator validator;
    private Provider provider;

    public ProviderValidatorTest() : base() {
        validator = new ProviderValidator();
    }

    [TestMethod()]
    public void YesErrorWhenNameIsNull()
    {

       provider = _testingRepo.GetProvider(1);

   // W/out using fluentv's helper
       // ValidationResult validationResult = validator.Validate(provider);
       // Assert.IsTrue(validationResult.IsValid);

   // Using helper method
       validator.ShouldHaveValidationErrorFor(Provider => Provider.Name, provider);
    }

...

Not sure I was clear.
I need to see the full code of your CustomISO8601Validator class.

You were clear.  My bad.
using FluentValidation.Validators;
using System.Text.RegularExpressions;
using System;
using System.Collections.Generic;
using NEW_ECAP_API.Data.Models;
namespace NEW_ECAP_API.Data.Validators
{
public class CustomDateTimeISO8601Validator<T> : PropertyValidator
{
    public CustomDateTimeISO8601Validator()
        : base("{PropertyName}: Invalid format"){
    }
    protected override bool IsValid(PropertyValidatorContext context)
    {
        string dateString = context.PropertyValue.ToString();
        Regex regex = new Regex(@"^(\d\d\d\d-\d\d-\d\dT\d\d:\d\d:\d\dZ$)");
        Match match = regex.Match(dateString);
        if (match.Success)
        {
            return true;
        }
        return false;
    }
}

}

Ok so it looks like the NullReferenceException is most likely from this line in your validator:
    string dateString = context.PropertyValue.ToString();

You'll need to do a null check here to see if PropertyValue is null before calling ToString on it.

My main provider valiator does not use CustomDateTimeISO8601Validator BUT I have some collections in my provider objects that have their own validators that DO use the the CustomDateTimeISO8601Validator validator...  saw some post about nested validators as I googled about.  Perhaps those post are for me.
(Uh.. you rock)

Yeah, if you have child validators set up for properties on your root-level object, then calling Validate on the top-level validator will validate the entire hierarchy.

Where I'm using my ISO8601 validator I've placed them after null / empty checks.  AND I added this to global.asax --->  ValidatorOptions.CascadeMode = CascadeMode.StopOnFirstFailure.  Should it ever receive a call w/ null prop value if it is indeed this way? (am sure there is some user error -- am double checking now).
        RuleFor(x => x.DateOfBirth)
            .NotNull()
            .WithMessage(badRequired)
            .NotEmpty()
            .WithMessage(badRequired)
            .CustomDateTimeISO8601Validator()
            .CustomDateTimeNowOrEarlierValidator()
            .CustomDateTimeMaxChildAgeValidator();

Also... do I need NotNull() checks if I'm doing NotEmpty() checks?

NotEmpty performs a NotNull check too, so you shouldn't need both. If CascadeMode is set to StopOnFirstFailure, then a Null shouldn't be passed to your CustomDateTimeISO8601Validator. Still, I'd recommend you modify the validator to skip validation (and just return true) if the PropertyValue is null (FluentValidation's built-in validators all work this way)
Note that if you set CascadeMode to StopOnFirstFailure in your global.asax, this won't apply to your unit tests. You'll need to set it in your test class's initialization.

Thank-you for ALL your help. A little more faith in the kindness of humanit after being assisted by you, Jeremy.  Will do my best to pay it forward.

Glad it's all sorted!


new footer

closes #22
-> added animated social icons
-> mountain like top

me bas resolve kar hi chuka tha


Extension issue

Issue Type: Bug
Extension Name: kite
Extension Version: 0.82.0
OS Version: Darwin x64 17.5.0
VSCode version: 1.34.0-insider

:warning: We have written the needed data into your clipboard. Please paste! :warning:```json
{
"messages": [],
"activationTimes": {
"startup": true,
"codeLoadingTime": 78,
"activateCallTime": 7,
"activateResolvedTime": 3,
"activationEvent": "*"
},
"runtimeErrors": [
{
"name": "Error",
"message": "Unable to find Kite application install using mdfind"
},
{
"name": "Error",
"message": "Unable to find Kite application install using mdfind"
},
{
"name": "TypeError",
"message": "Cannot read property 'focus' of undefined"
},
{
"name": "TypeError",
"message": "Cannot read property 'selectionChanged' of undefined"
},
{
"name": "TypeError",
"message": "Cannot read property 'selectionChanged' of undefined"
},
{
"name": "TypeError",
"message": "Cannot read property 'selectionChanged' of undefined"
},
{
"name": "TypeError",
"message": "Cannot read property 'selectionChanged' of undefined"
}
]
}

Hi! Thanks for alerting us to this issue. To help assist you further, could you let us know if you have the Kite Engine (it's separate from the plugin itself) installed on your system?


Check that vol actually contains DB files, as opposed to a non-empty dir.

Part of fix for mounting persistent disks on Google Cloud Compute containers - fresh disks have a lost+found directory, causing the old check to fail with
postgres cannot access the server configuration file "/var/lib/postgresql/data/postgresql.conf"

The second part of the fix requires the GCE user to adjust PGDATA to a subdirectory within the mounted persistent disk. If the initdb command is run directly on the mount point root, it fails, complaining that it is bad practice to have the database there. Reference thread.
The notice to adjust PGDATA should be in the docs for postgres-docker.
I feel we should not make it mandatory for PGDATA to auto-create a subdirectory at all VOLUME mountpoints, as the mounted point might already be a subdirectory in the host system.

Since we are only checking for the one file, we no longer need ls and can check that it has size too:
- if [ -z "$(ls -A "$PGDATA"/PG_VERSION)" ]; then
+ if [ -s "$PGDATA/PG_VERSION" ]; then

@yosifkit Ah true. Changed! :)

LGTM

LGTM


Upgrade the images version to use the geneva dot release on geneva branch

The device services should be 1.2.1 and app service should be 1.2.0
https://github.com/edgexfoundry/edgex-taf/blob/dfdbc751696e9a168fa579c202862cefc39347ed/TAF/utils/scripts/docker/x86_64.env#L2-L7
https://github.com/edgexfoundry/edgex-taf/blob/dfdbc751696e9a168fa579c202862cefc39347ed/TAF/utils/scripts/docker/arm64.env#L2-L7

fixed by #128


fail to install json extension on php 5.6.4

I execute hpbrew ext install json but fail to install json extension.
here is the log:
bash-3.2$ phpbrew ext install json
[ ] json extension is already disabled.
===> Installing json extension...
Log stored at: /Users/xxxxxx/.phpbrew/build/php-5.6.4/ext/json/build.log
Changing directory to /Users/xxxxxx/.phpbrew/build/php-5.6.4/ext/json
===> Make clean
===> Phpize...
===> Configuring...
===> Building...
===> Make all
===> Installing...
===> Make install
===> Extension is installed.
===> Creating config file /Users/xiami/.phpbrew/php/php-5.6.4/var/db/json.ini.disabled
===> Enabling extension json
===> Applying conflicts resolution (jsonc):
PHP Catchable fatal error:  Argument 1 passed to PhpBrew\Extension\ExtensionManager::disableExtension() must be an instance of PhpBrew\Extension\Extension, null given, called in phar:///usr/bin/phpbrew/PhpBrew/Extension/ExtensionManager.php on line 2 and defined in phar:///usr/bin/phpbrew/PhpBrew/Extension/ExtensionManager.php on line 2
Catchable fatal error: Argument 1 passed to PhpBrew\Extension\ExtensionManager::disableExtension() must be an instance of PhpBrew\Extension\Extension, null given, called in phar:///usr/bin/phpbrew/PhpBrew/Extension/ExtensionManager.php on line 2 and defined in phar:///usr/bin/phpbrew/PhpBrew/Extension/ExtensionManager.php on line 2

I self-update phpbrew, and it works. thx


QURI Chemistry QDC revision

@abbycross @beckykd @Eric-Arellano @pandasa123
Thanks so much for various reviews and fixes, I have updated accordingly.


import 1-2 / day

needs a webapp with state, maybe vercel, write all users to sqlite w/ current progress


Panic with mix of public and private fields

I have two files, src/main.rs and src/log.rs, in the same crate. src/main.rs imports src/log.rs.
src/main.rs:
mod log;

use crate::log::*;

fn main() {
    let mut dev = Device::new(4096);
    dev.write_byte(0, 0);
}

src/log.rs:
pub struct Device {
        pub dev: Vec<u8>,
        size: usize,
        head: usize,
        pub tail: usize,
}

impl Device {
        pub fn new(size: usize) -> Self
        {
            Self {
                dev: Vec::with_capacity(size),
                size,
                head: 0,
                tail: 0,
            }
        }

        pub fn write_byte(&mut self, dst: usize, byte: u8)
            requires
                dst < old(self).dev.len()
        {
            self.dev.set(dst, byte);
        }
}

Since Device has mixed public and private fields, I would expect to see an error message like those mentioned in #221 about the datatype being opaque because a field is not visible. However, when running Verus on the root of the crate (/c/Apps/verus/source/target-verus/release/verus src/main.rs), I get the following output:
note: verifying module log

note: verifying root module

thread 'rustc' panicked at 'internal error: missing variant Device', vir/src/ast_util.rs:372:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This also happens when the two modules are in a single file:
use vstd::prelude::*;

mod log {
    use vstd::prelude::*;
    use vstd::vec::*;
    pub struct Device{
            pub dev: Vec<u8>,
            size: usize,
            head: usize,
            pub tail: usize,
        }

    impl Device {
            pub fn new(size: usize) -> Self
            {
                Self {
                    dev: Vec::with_capacity(size),
                    size,
                    head: 0,
                    tail: 0,
                }
            }

            pub fn write_byte(&mut self, dst: usize, byte: u8)
                requires
                    dst < old(self).dev.len()
            {
                self.dev.set(dst, byte);
            }
    }
}

use crate::log::*;
fn main() {
    let mut dev = Device::new(4096);
    dev.write_byte(0, 0);
}


CAMEL-12570: fixed Camel-FTP component does not set CamelFtpRep…

… in some case(ex 530)

@valdar @davsclaus , did not look in depth. is this something related to CAMEL-13069 to resolve too?

I dont think they are identical as the CAMEL-13069 seems related to already writing (uploading) files. While this is before writing, eg while establishing connection


Intermittent issue with calendar layout on app launch.

Version Number:
8.0.3
Description
Intermittent issue with calendar layout.
Sometimes when I open the page with calendar, it looks like this:

Switching the screen and going back fixes the issue.
Steps To Reproduce
Intermittent.
I noticed a few times when I open the app, the alignment is all wrong.
Going back and forward fixes the issue.
Expected Behavior
start date is today,
end date is today + 4 months
number of rows: 6
Additional Context

With errors like this, the best way to get help is to create a barebones minimal app that recreates this issue.
There is no way for me to tell what your app is doing which makes it get in this state.

closed until response.


Hide transfer account when in single-entry mode

Even when double-entry accounting is disabled in the settings, every transaction mentions the counter account in the overview.

This is implemented in v2.1.0 which is currently in beta

Fixed in d871cf203abf703c63d6a439947839f2bc156b99


Load up again after death bug

After the player is killed and gets game over, if they LOAD UP again to start a new game they are killed immediately with the first attack

Made the player defined 'gruntAttack' event return false to allow for the following caveat:
"Returning false from an event handler will cause other handlers not to process the event."
This behaviour is outlined in the knife/event repo - https://github.com/airstruck/knife/blob/master/readme/event.md


Bug: object move function not working

v0.6.17
MacOS 10.15.7
The "move" arrows don't appear and I'm unable to move the object around on the grid. Arrows don't appear for v0.6.14 either, but I am able to move objects.

One thing I noticed today while editing files in MeerK40t, was that occasionally a bounding shape would get selected instead of the object I was trying to select. I would have to de-select that; before I could select the object I desired to move.
It's possible there is an Mac OS version specific issue here; but I haven't been able to reproduce a move bug on my system. I can try harder if there are issues that need to be resolved.

Thanks so much for looking into this. I just got all set up to record my screen using v0.6.17 and 0.6.14 and “Voilà!” - it works now. Selecton outline is visible (I’m sure it was when it wasn’t working) Yes, no mouse cursor indication but the drag/moving function is now working. Odd. It probably won’t work tomorrow…I’ll update/screen record if it happens again.
Also, is there a way to reliable execute an undo command? CMD +Z only works some of the time, so I have to start over if I select a bounding shape by accident, which does happen.

On Feb 8, 2021, at 6:29 PM, Joseph Lane<EMAIL_ADDRESS>wrote:
One thing I noticed today while editing files in MeerK40t, was that occasionally a bounding shape would get selected instead of the object I was trying to select. I would have to de-select that; before I could select the object I desired to move.
It's possible there is an Mac OS version specific issue here; but I haven't been able to reproduce a move bug on my system. I can try harder if there are issues that need to be resolved.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/meerk40t/meerk40t/issues/345#issuecomment-775533081, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASYZVCWEB4AVNFKZHRHMZ6DS6BXVVANCNFSM4XJITKUQ.

There is no undo. CMD+Z is a lie. It resets the matrix to the original form. Which can sometimes fix things but also can open up cans of worms for no good reason.
Issue #34


A handful of digest related settings changes

The user setting for Digest emails is not hidden when digests have been disabled at the Realm level.  This commit fixes that.

I pushed a couple more changes on this PR.  This PR now contains the following changes.

Fixes a bug with incorrectly displaying the user level setting of enabling/disabling digest emails when the realm has them turned off.
Adds an organisation level setting to turn on/off digest emails for a Realm.
Adds an organisation level setting to choose the day of the week on which to send the digests for the realm.

The codecov failures in this PR don't seem related to changes in this PR.

@timabbott This PR fixes a bug (in the first commit), and adds a couple more
settings for digest emails. The new settings commits can be left to be merged
later, if that seems appropriate.

Did you test what happens when saving adjacent user notification settings with digest emails globally disabled?  We've had bugs with similar logic in the past where the webapp submitted invalid input to the server.

I think also some of the commits should be squashed together -- not sure why the models change is its own commit.

Looks great, merged after making a small change (adding the prop-element class to the select element, which is required since 75fc19acfd0fbdefd8d1a4f505eb9c9a28806f01), thanks!
@rishig FYI, this is hidden for (almost?) every installation because SEND_DIGEST_EMAILS is False.  I think with this change integrated, it could be reasonable for us to change these defaults:

SEND_DIGEST_EMAILS, the server-level setting, becoming True (or maybe just ceasing to exist?  Idunno.)
The default for the realm-level setting for sending digest emails changing to False, with a migration that sets it to false everywhere (or at least everywhere that SEND_DIGEST_EMAILS was False).

This would make whether or not to send digest emails something that's under realm administrator control, so that e.g. it would be possible for Puneeth's organization on Zulip Cloud to use this.
We might want to do some other tweaks to this feature (or document it as beta, or something) first though, thoughts?

hm, good question. Probably could document as beta and then turn on with default False, given that we're doing active work on it.
In terms of quick cleanup -- I think we should consider removing a lot of the fluff text, maybe the PMs section (given that we should send those as missed message email?), and ideally fix the scrollbar bug.

Did a quick pass at the email:
https://github.com/zulip/zulip/commit/50b2524a012271c5885f5203fed47143b333e3c9
Thoughts?

Yeah, removing the PMs section and the fluff text as you suggest seems looks a good plan.

I don't think I know about the scrollbar bug -- is there an issue for that?

hm, not sure if there is an issue for it, but this is my recollection of the current state:

I think this should've been fixed in b8e25677ef864c1628931e21115c366f7d73ca4a

cool, thanks!

I'd be happy to work on turning this on, if we decide on what the next steps are.

Turning https://github.com/zulip/zulip/commit/50b2524a012271c5885f5203fed47143b333e3c9 into a real PR could be a good next step:

Reflecting those changes in the .txt version of the email
Changing the code that generates the email appropriately
Manually generating a few to make sure it looks reasonable

Hopefully not something that should take too long?

I'm not sure how to find the commit 50b2524a012271c5885f5203fed47143b333e3c9
I'm instead going to make a new commit with these changes. :see_no_evil:


Create BEM EXISTING BUILDING VERIFICATION METHODOLOGY

First draft of the BEM EXISTING BUILDING VERIFICATION METHODOLOGY - ready for steering committee review.

Is this not a duplicate of the other PR? ( https://github.com/wattcarbon/WEATS/pull/4 ) That's confusing for reviewers

Going to close for now and re-open once its been completed and reformatted.


✨ Colour picker

Closes #15.

Build succeeded!
Built with commit 089d71093cbe7d53e08a1ce3a2b4e4286b83a885
https://deploy-preview-40--padpad.netlify.com


feat: add span for showing operation_id in state machine transition

closes #3883

Successfully created backport PR for releases/v0.2:

#3922


Sometimes need to click twice to scroll the images prev/next

Hi there,
I found an issue in the Carousel. Even in the Simple Carousel Example provided in the site http://sorgalla.com/projects/jcarousel/examples/static_simple.html, sometimes need to click twice on next or prev button to scroll the images.
Do you have any solution for this?
Thanks
Amer

Nope, i don't have this issue.


flywave.fit as default

faceless-flywave as optional
wings.faceless.flywave.checker=fasle
wings.enabled.faceless.flywave=false

Table or view tst_sharding does not exist.

sharding test failed when dynamic create table

https://github.com/apache/shardingsphere/issues/28991
https://shardingsphere.apache.org/document/current/en/user-manual/shardingsphere-jdbc/yaml-config/rules/single/

sharding cache table and rule at startup, there is no easy way to reload it


Problem about the calculation of 'Local model accuracy'

Hi, @samiul272 . As you mentioned in your paper, "local model accuracy is defined as the accuracy of the server model on each of the clients's local datasets." In my opinion, what you want to express is that when the total number of clients is 100 and 10 clients are randomly selected to participate in the training in each round, the calculation method of  local model accuracy is as follows:

Server sends the global model to 10 clients participating in the training;

Each client uses its own test dataset to evaluate the global model and gets its own Local-Accuracy and Local-Loss.

Average these Local-Accuracy values and Local-Loss values as the final values for this round.

However, while reading your code, I notice that your code only selects 10 fixed clients numbered 0,10,20,30,...90 to calculate Local-accuracy and Local-loss in each round (As you can see in the picture, variable m in you code can only take a fixed number of 10 values.), instead of selecting the test set of the current clients (their numbers are stored in the variable user-idx in your code) that actually participate in the training for global model evaluation. Is this really reasonable?

On the other hand, it seems that you only use one batch of data (line 212, result[0]) instead of all batch of data (line 212, result) in the client's local test set to calculate the values of Local-accuracy and Local loss for each client, is it reasonable?

If you look at the section
            for k in range(m, min(m + len(local), cfg['num_users'])):
                processes.append(local[k % len(local)]
                                 .test_model_for_user.remote(k,
                                                             [dataset_id, data_split_id, model_id, label_split_id]))

you will see that it selects clients 1-10 then 11 to 20 ... and calculates the accuracy of 10 clients simultaneously with ray.  m is the starting index of selected clients and k is the selected clients. The reason for this is that instantiating 100 clients will cause cuda memory error so we do 10 at a time.
As for the results, I think the value actually returned by ray.get(processes)  is a list of lists that looks like [[(result1, input_size), (result2, input_size)]]. My apologies for the confusing code. Note that, in reality, we do not generally run local accuracy test in all epochs but rather until the end epoch as it takes more time than training one epoch.

I see it now. You should replace the m in line 206 with k. That is a mistake on the code there. I will fix that.

As for the results, I think the value actually returned by ray.get(processes) is a list of lists that looks like [[(result1, input_size), (result2, input_size)]]. My apologies for the confusing code. Note that, in reality, we do not generally run local accuracy test in all epochs but rather until the end epoch as it takes more time than training one epoch.

Hi, @samiul272 , maybe I didn't say what I meant clearly. I will show the values of result and results in the debugging process, as shown in the figure below. We can see from the figure that:

results contains the test results of 10 clients;
result contains all the batch test results of one client (in the case of 100 clients, the number of test dataset per client is 100, and since the test batch size is 50, result contains two batches of data).

Therefore, result[0] only returns the test results for the first batch of the client, which seems incorrect.

Hi, result[0] contains the evaluation result and the size for the batch. It is a tuple having evaluation and input_size. evaluation is appended to the logger. We iterate over each result in results and add the evaluation to the logger which consequently logs that to tensorboard. Could you check the input shape of result[0] and result[1].

Hi, result[0] contains the evaluation result and the size for the batch. It is a tuple having evaluation and input_size. evaluation is appended to the logger. We iterate over each result in results and add the evaluation to the logger which consequently logs that to tensorboard. Could you check the input shape of result[0] and result[1].

Hi, @samiul272 . The input shape of result[0] and result[1] are as follows:

As you can see, result contains the test results of two batches (result[0] and result[1]), each with the size of 50. The two batches together constitute the entire test dataset of this client. What I want to express is that line 212 in the code only appends the evaluation of result[0] to the logger, and I think both evaluations of result[0] and result[1] should be appended to the logger, because the test dataset consists of two parts, result[0] and result[1].

I see. You can use a test batch size of 100 and that should show correct behavior. I will update this bug with local accuracy too. Thanks for your diligent assessment.

Hi, @samiul272 , my dear friend, thank you for your patient reply. I have learned a lot from your articles and codes. Thank you!

I pushed the updates to append each element in the result variable to the logger so that the logger computes the average accuracy.


Expose a way to allow middlewares from modules run before global middleware

Describe the feature
By default, global middleware is executed alphabetically based on the filename. However, not all global middlewares are sorted at the sametime, instead, Nuxt run global middlewares from the middleware folder first, then run middleware from moduels. Middlewares from modules have lower order then the ones from middleware folder. This is a problem when we want to run a middleware from a module before a global middleware.
For example, we want to run a middleware from a module to check if the user is authenticated before running a global middleware to check if the user has the right permissions.
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

We think it would be a good idea to allow passing an option to addRouteMiddleware (as with addPlugin) to allow adding middleware to the beginning of the middleware array rather than the end.
In general I'd advise against strict middleware ordering dependencies.

Code is here: https://github.com/nuxt/nuxt/blob/83f8617e3f506125c2a9882afef4074ee0f1ded9/packages/kit/src/pages.ts#L40-L59
If we want to have the same behavior as addPlugin, we need to unshift() by default and add the append option to push()
Suggestion:
// push() the middleware instead of unshift()
addRouteMiddleware({
      name: 'auth',
      path: resolver.resolve('runtime/auth.ts'),
      global: true
}, { append: true })

It makes sense to update the default as module middleware (like plugin) should run before user land middleware.
PR welcome ❤

thanks! @atinux, @danielroe
I create an PR for this, please take a look https://github.com/nuxt/nuxt/pull/28496/files


Deprecate url_preprint

Deprecate url_preprint in favour of url_pdf (or a PDF file dropped into publication's folder) given there's now a dedicated "Preprint" pub. type. It would still be possible to link to a preprint by creating a custom link in the links option.

[x] remove param from arch. type and demo front matter
[ ] remove "Preprint" button generation from page_links.html (and possibly also btn_preprint from i18n)

I would recommend to leave the url_preprint.  Some people will be interested in having their preprint linked to a journal publication.  It will allow people see the evolution of a paper, and it will also provide access to a  PDF version of the paper, when the final published version is only available through a paywall.

I agree with @santiago-schnell that it is convenient to have the preprint and the final published paper on the same document. The only reason I would change something is for clarity for the page viewer: maybe it would be more easily understood to have a "PDF (preprint)" button (rather than just "Preprint"). This said, some people may want to link to the arXiv/... page itself rather than the PDF (which would then be "Preprint", not PDF, but in my experience I would prefer have a special [custom] link saying "arXiv" in that case).

On BioRxiv preprints also get DOIs so if I have a project and I want my readers to be able to see various versions of the paper the "pre-print" is useful.

To be continued in #765


fix(common): properly parse functions in their usual form

Closes #6135
Alloy's Function only parses functions without their return arguments and without the function prefix. We can use AbiItem::parse to parse them as human-readable abi strings and get a Function in return.

Added a test @mattsse — albeit we'll need a new feature in alloy to parse full function signatures. Right now we can only parse either stripped functions or human-readable ABI functions which don't support visibility modifiers. cc @DaniPopes


invalid message timestamps with GMT+5:30

reported by @SpEcHiDe:disroot.org
https://github.com/matrix-org/riot-android-rageshakes/issues/709

this user got this issue only two times.


changing spi master mode gives wrong value

When changing the spi master mode from 0 to 1 the written data at sdo is faster 1 cycle the the sck.
Here is a waveform of writing 0x1 when mode is 1(reading at negative edge )

This is read as 0x3
The waveform when writing 0x1 when mode is 0 (reading at posedge)

I'm not sure what I'm looking at in the above graphs. . . SCK and SDO both from the SPI master?  In principle, it looks right;  the SDO is aligned with the rising edge of the clock in one mode and the falling edge of the clock in the other mode.  But do you have an SPI slave in the testbench that can also be put in mode 0 or mode 1?  The modes of master and slave have to match, or else yes, there will definitely be race conditions and misinterpretation of data.

yes, sck and sdo are from the SPI master.
yes the slave works with different modes.
The mode that align with rising edge is functionally right only the last pose edge reads 1 at this example. However the mode that aligns with negative edge reads 1 at edges number 6 and 7 (counting from 0)

Here are some other trials sending byte in this mode to provide more details about the shift between clock driving and data sending.
try sending 0x00 but it sends 0x1

try sending 0x7F but it sends 0xFF

try sending 0x02 but it sends 0x05

try sending 0x03 but it sends 0x07


profiling a script with pyexasol with scalene

Hello,
I'm trying to profile a script that uses pyexasol  using scaelene.
While conn.execute() seems to work (didn't let it run through, but there was debug output), the whole script freezes when calling conn.export_to_pandas() or conn.export_to_file() - there isn't even any debug output about the query.
Do you have an idea what could be causing this?
Best regards

HTTP transport runs 2 additional threads:

Thread to manage pseudo HTTP-server, waiting for "incoming" connection from Exasol.
Thread to run SQL query and check its status.
Main thread is busy processing data in callback function.

These are normal python threads, no extra magic.

That's strange...
Did you profile pyexaols memory usage in the past?
If so, which tools did you use?

I've used basic built-in profiler: https://docs.python.org/3/library/profile.html
Pyexasol does not accumulate data at any point. Everything is processed in chunks.
Data might be accumulated in callback function output, like building pandas data frame. But no accumulation in the core, so memory should not be an issue.

is writing to file output also done in chunks? or ist it more like pandas output?

Of course. Writing to file uses shutil.copyfileobj, which operates in chunks.
https://docs.python.org/3/library/shutil.html#shutil.copyfileobj
This is the place in code: https://github.com/exasol/pyexasol/blob/master/pyexasol/callback.py#L52

Thanks, in that case while I can't profile with scalene, at least I can be pretty sure pyexasol isn't the cause.
I'll close this this ticket.


Documentation missing or incomplete for many API/doc sections.

I have noticed that the website does not have the qdetector documented. There are others, too. Multichannel comes to mind.
I will not give the whole list here. Suffice to say, this has been a grave problem in quickly moving around the library. (I no longer need the details on the two, I'm merely reporting the lack of documentation as a former impediment.)
The problem spans both the doc and the api tabs. The two are also drifting out of sync with each other, it seems.

Fair point, and I've been (slowly) trying to synchronize them and complete missing pieces. I can prioritize the qdetector object and multichannel module.


tor support

Tor support
This operator includes a reconciler that, based on annotations, is able to take
action on those secrets that would want to be populated with Tor credentials.
All you need to do is create a Secret with the annotation utxo.com.br/tor: v3
for a v3 service (v2 will be deprecated anyway, so why bother).
(ps.: if the secret is already popoulated, the reconciler WILL NOT try to
populate it again)
For instance, we can create a Secret named tor
apiVersion: v1
kind: Secret
metadata:
  name: tor
  annotations:
    utxo.com.br/tor: "v3"

which after reconciliation will see its data filled with the content of the
files you'd expect to see under HiddenServiceDir:
apiVersion: v1
kind: Secret
metadata:
  name: tor-creds
  annotations:
    utxo.com.br/tor: "true"
data:
  hs_ed25519_secret_key: ...
  hs_ed25519_public_key: ...
  hostname: blasblashskasjjha.onion

(you can see if things went good/bad through events emitted by the
reconciler)
With those filled, we're then able to make use of them in the form of a volume
mount in a Tor sidecar which then directs traffic to the main container's port
through loopback - after all, they're in the same network namespace.
A full example of a highly-available hidden service:
---
# create an empty but annotated secret that will get populated with the hidden
# service credentials.
#
apiVersion: v1
kind: Secret
metadata:
  name: tor
  annotations:
    utxo.com.br/tor: "v3"

---
# fill a ConfigMap with the `torrc` to be loaded by the tor sidecar.
#
apiVersion: v1
kind: ConfigMap
metadata:
  name: tor
data:
  torrc: |-
    HiddenServiceDir /tor-creds
    HiddenServicePort 80 <IP_ADDRESS>:80
    HiddenServiceVersion 3

---
# the deployment of our application with the application container, as well as
# a sidecar that carries the tor proxy, exposing our app to the tor network.
#
apiVersion: apps/v1
kind: Deployment
metadata:
  name: foo
  labels: {app: foo}
spec:
  selector:
    matchLabels: {app: foo}
  template:
    metadata:
      labels: {app: foo}
    spec:
      volumes:
        - name: tor-creds
          secret: {secretName: tor-creds}
        - name: tor-creds
          configMap: {name: torrc}

      containers:
        - image: utxobr/example
          name: my-main-container
          env:
            - name: onion_addr
              valueFrom:
                secretKeyRef:
                  name: tor
                  key: hostname

        - image: utxobr/tor
          name: tor-sidecar
          volumeMounts:
            - name: tor-creds
              mountPath: /tor-creds
            - name: torrc
              mountPath: /torrc

ps.: notice that there's no need for Service - that's because we don't need
an external ip or any form of public port; this is a hidden service :)
an interesting side note here is that we not only are able to expose our
service in the Tor network, but we also have access to it via socks5 by
making requests to the sidecar under <IP_ADDRESS>:9050 (again, same network
namespace!)
ps.: note the use of the ONION_ADDRESS environment variable - that's in
order to force redeployments to occur whenever there's a change to the secret
see https://ops.tips/notes/kuberntes-secrets/

^ ultimately, all that we need to truly implement is the reconciler that
fills up the secret based on what tor -f <> on a torrc that states a
HiddenServiceDir gives us.
the rest (mounting the secret, generating a configmap, etc etc) is all stuff
that we would expand in the MoneroNode reconciler based on a field
under moneronode.spec.

TBD: making tor observable - it'll be interesting to have a hidden service
running for a while and try to learn some of the best practices around
maintaining that tor proxy running and being able to observe its behavior
so we can tell whether everything is healthy/not.

a proof of concept revealed to be quite effective - the code is not great, but it was nice to demonstrate that it can work pretty well :) having the CCS accepted, work can start being done on moving from prototype -> decent implementation


Migrate Walls.io

Migrate walls.io block

Test URLs:

Before: https://main--blog--adobecom.hlx.page/drafts/smalla/wallsio
After: https://main--blog--adobecom.hlx.page/drafts/smalla/wallsio?milolibs=walls-io&martech=off

@saugatmalla Can you look into these merge conflicts?

@rgclayton merge conflict is fixed now.


Fixes obvious typo

DropNegligible was described but missing.
✅ I signed a new CLA.

LGTM, thanks!


Photo Sphere / Panorama Support

I would love built-in support for viewing photo spheres (360 photos, panoramas, rectilinear images, whatever you want to call them).  I have quite a few of these from vacations, and love Google Photos' ability to view them inline as I'm swiping through my albums.
Two potential libraries are:

https://github.com/mistic100/Photo-Sphere-Viewer
https://github.com/mpetroff/pannellum

I haven't done a lot of digging personally, but Photo Sphere Viewer also supports videos which, though niche, would be a nice-to-have.  They're both MIT-licensed, so compatibility should be okay.
Identifying which photos/videos are spheres would also need to be done, but I believe this is largely available in metadata that LibrePhotos is already accessing.
I've been looking for a photo gallery replacement for Google Photos and I'm encouraged by the active development here at LibrePhotos.  If I get this up and running for my own library, I might take a stab at adding this feature in a PR myself, but would love some help / pointers from someone more familiar with LibrePhotos on where to start and how best to integrate the viewer.

This would be a really neat feature! Would you be willing to contribute this PR? Getting up and running with the development environment is pretty simple of you're using Vs code docker environment. Otherwise you can run the services locally, adjust the port in the front-end environment file and run npm start


Add step to Update gcloud

Included a process to update the version of gcloud.
Thank you for your review.

sorry....
Closing a pull request
The size of the tarball is 63828716 bytes.
The maximum size is 52428800 bytes, cannot continue.
Package failed: Tarball is too big```


Use a helper that captures all output instead of only some.

Fixes https://clusterhq.atlassian.net/browse/FLOC-167
Still a little bit of junk output but this gets rid of a about 99% of it.

Thanks @exarkun. This looks good (although I don't know pyrsistent and am just assuming that  nothing is crazy with `_ProcessResult).
I've tested this by:

"eyeballing" some Buildbot output and seeing that there is significantly less "junk".
Running an affected test (flocker.node.functional.test_deploy.DeployerTests.test_cpu_share) with a modification which made it fail to see that failures are reported as expected.

There are a couple more check_calls it might be worth changing, but as you said in the description, there is "Still a little bit of junk output".
I am unable to check the acceptance test output on Buildbot because they are not running at the moment.
One bit of the output which might be easy to fix is the warning about service_identity:
:0: UserWarning: You do not have a working installation of the service_identity module: 'No module named service_identity'.  Please install it from <https://pypi.python.org/pypi/service_identity> and make sure all of its dependencies are satisfied.  Without the service_identity module and a recent enough pyOpenSSL to support it, Twisted can perform only rudimentary TLS client hostname verification.  Many valid certificate/hostname mappings may be rejected.

Perhaps it would be best to add service_identity as a dev dependency.
I think that because this is not a complete solution this should be after some minor changes have been made as you see appropriate, but the parent issue should remain open (and I guess moved out of this sprint) or another issue should be made for the other "junk".


Missing key(s) in state_dict when testing using predict_downstream_condition.py

python predict_downstream_condition.py --ckpt_path model_name_roberta-base_taskname_qqp_lr_3e-05_seed_42_numsteps_2000_sample_Categorical_schedule_mutual_hybridlambda_0.0003_wordfreqlambda_0.0_fromscratch_False_timestep_none_ckpts/best(38899).th
using standard schedule with num_steps: 2000.
Traceback (most recent call last):
File "predict_downstream_condition.py", line 101, in
model.load_state_dict(ckpt['model'])
File "/opt/conda/envs/diff/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1672, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for RobertaForMaskedLM:
Missing key(s) in state_dict: "roberta.embeddings.position_ids", "roberta.embeddings.word_embeddings.weight", "roberta.embeddings.position_embeddings.weight", "roberta.embeddings.token_type_embeddings.weight", "roberta.embeddings.LayerNorm.weight", "roberta.embeddings.LayerNorm.bias", "roberta.encoder.layer.0.attention.self.query.weight", "roberta.encoder.layer.0.attention.self.query.bias".........................

Hi,
Did you train the model with DDP? If so, the state dict keys may be different.

i met this problem too.
after run DDP_main_conditional.sh got the ckpt  (xxx .th) , when i run predict_downstream_condition.py, it occurs error the same.
RuntimeError: Error(s) in loading state_dict for RobertaForMaskedLM:
Missing key(s) in state_dict: ....sd1
Unexpected key(s) in state_dict: .....sd2

sd2 keys is module.roberta.... while sd1 is roberta.... , it seemt that
it runs when i change model.load_state_dict(ckpt['model']) to model.load_state_dict(ckpt['model'], strict=False) , i think i maybe damage the model performance  in some extend.
i also want to know how long you train on what device. i find that model often stuck in each eval_step for saving, which cost most time when i train the model

same problem

same problem

Thank you for your great work.
I have the same problem.
As explained in this github, I executed "run.sh" and it executed "DDP_main.py".
After fixing several errors, the Missing key(s) in state-dict error occurred.
So, I'm confused that @Hzfinfdu said, state dict keys of DDP may be different.
Then, which file should be executed to reproduce the DiffusionBERT?

Hi,
Did you train the model with DDP? If so, the state dict keys may be different.

Thank you for your reply. I solved the issue with the same way.
However, did you solve the issue #25 ? It seems to be the final error to solve to use predict.py


Update to gfakluge required

GFAKluge itself needs updates, and then we need to update svaha to handle said updates as we move vg to gfa2.

Fixed - gfakluge has been updated. GFA2 output can be added by just bumping the GFA version.


Unable to find source java class: '/Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/java/src/main/java/org/tensorflow/op/core/Constant.java'

I follow the instructions on the tensorflow official website on how to import android samples. I did exactly as they said but when I try to run the app it shows the following error.
Error:Execution failed for task ':compileDebugJavaWithJavac'.

Unable to find source java class: '/Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/java/src/main/java/org/tensorflow/op/core/Constant.java' because it does not belong to any of the source dirs: '[/Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/src/main/java, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/src, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/build-types/debug/java, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/gradleBuild/generated/source/r/debug, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/gradleBuild/generated/source/buildConfig/debug, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/gradleBuild/generated/source/aidl/debug, /Users/vinay.garg/AndroidStudioProjects/tensorflow/tensorflow/examples/android/gradleBuild/generated/source/rs/debug]'

The problem is, Constants.java file is in the parent directory and not in the sample project directory. I tried to find the usage of the Constants.java file but can't find its use anywhere in the sample project. What am I missing here?

I had Rebuild Project,but didn't work.

I am working on windows 10 and I want to make it work with my own model, how can I achieve it?

I am working on windows 10 and I want to make it work with my own model, how can I achieve it?

I am working on windows 10 and I want to make it work with my own model, how can I achieve it?

https://github.com/tensorflow/tensorflow/tree/master/tensorflow/examples/android#install-model-files-optional
Optional: If you wish to place the models in your assets manually, remove all of the model_files entries from the assets list in tensorflow_demo found in the BUILD file. Then download and extract the archives yourself to the assets directory in the source tree:
BASE_URL=https://storage.googleapis.com/download.tensorflow.org/models
for MODEL_ZIP in inception5h.zip ssd_mobilenet_v1_android_export.zip stylize_v1.zip
do
curl -L ${BASE_URL}/${MODEL_ZIP} -o /tmp/${MODEL_ZIP}
unzip /tmp/${MODEL_ZIP} -d tensorflow/examples/android/assets/
done
This will extract the models and their associated metadata files to the local assets/ directory.
If you are using Gradle, make sure to remove download-models.gradle reference from build.gradle after your manually download models; otherwise gradle might download models again and overwrite your models.
Build


New line character between xml tags/data

The python code for creating xml
test_list =   [('Category A', 100, "This is category A"),
('Category B', 120, "This is category B"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C"),
('Category C', 150, "This is category C")]
schema = StructType([
StructField('Category', StringType(), True),
StructField('Count', IntegerType(), True),
StructField('Description', StringType(), True)
])
df = spark.createDataFrame(spark.sparkContext.parallelize(test_list),schema)
df.repartition(1).write.mode("overwrite").format("xml").option("rootTag", rootTag).option("rowTag", rowTag).option("emptyValue","").option("nullValue", "").
save("/data/")
Generated xml
<CONTRACT_ACCOUNT_Set>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category B</Category>
        <Count>120</Count>
        <Description>This is category B</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</

Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category A</Category>
        <Count>100</Count>
        <Description>This is category A</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>

    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
    <CONTRACT_ACCOUNT_Record>
        <Category>Category C</Category>
        <Count>150</Count>
        <Description>This is category C</Description>
    </CONTRACT_ACCOUNT_Record>
</CONTRACT_ACCOUNT_Set>

It create blank new lines between tags and data  . Attached generated xml file
part-00000.zip

This is the same issue as https://github.com/databricks/spark-xml/issues/338 and unfortunately I have tried and failed to figure out why it happens or fix it. You can see more there.

@srowen Any work around  ?

As I say, no.


Does this clBLAS support FPGA?

Hi,
I want to do some performance stuff on FPGA with Intel FPGA OpenCL SDK.
But have a question that does this support FPGA too? @gicmo @hughperkins @robertsdionne @sschaetz @glehmann

FPGAs need two things different:

Precompiled binaries for clBLAS + software using clBLAS.
FPGA-specific optimisations.

So with some work it would be doable to make it compile, but the whole design of clBLAS has had GPUs in mind. See https://github.com/Xilinx/SDAccel_Examples/wiki/Practice-and-Experiment-with-C-OpenCL-Kernels and https://www.altera.com/documentation/mwh1391807516407.html to get an idea how different it is.
Best is to call your contact at Altera / Intel FPGA to ask about availability of an optimized BLAS library for their FPGAs.

@VincentSC Got it. Thanks.


Validate existing of bmc errors on host checkstop & host watchdog failures.

https://github.com/openbmc/openbmc/issues/1266 - Associate with
generate bmc errors on host checkstop and host watchdog failures

This is in 2 phases:
phase1:
@SrideviRamesh you had already implemented host checkstop. Please re-run that test and check for bmc errors.
Phase2:
Inject watchdog failure and check for errors.

yes we have executed & verified these scenarios after RAS enable in witherspoon.....
please close this....


 I will take a look at your bug.
Thanks for reporting this.

#7
Have a fix now.

#8
Bug seems to be fixed.


Expanding a vault's dropdown before the vault deposit amount loads causes statistics to show as NaN

Repro:

Open https://yearn.finance/vaults
Immediately click a vault (with which you have a deposit balance) before the deposit balance loads and shows up

Issue:

Statistics load with NaN, but Strategy details, APY, and Deposit Balance show up fine.

From the console, there seems to be a TypeError in the request to api.yearn.tools.
This only repros if you click a vault and open its drop down before the Deposit Balance and APY load.

This doesn't seem to repro anymore, so I am fine to close it if y'all want

@listonjesse I increased the function timeout upstream. This should increase reliability.

@x48-crypto Makes sense to me. Thanks again!


Fix package.json auto imports for pnpm without project references

Fixes #42094—read about the reason for the symptom and the rationale behind the fix there. I’m not 100% confident this is the most direct way to fix the problem; open to other ideas if they’re not super complicated.

@andrewbranch you are doing change to use originalPath instead right?

@sheetalkamat from the thread:

I think if it’s not going to be normalized into a Path, the property name should use fileName and not path.

Since we agreed the type should be string and not Path, I was planning on keeping the name fileName to match. When I see a property/variable named path but its type is string, it looks to me like a mistake.

But you still need to copy originalPath from resolved instead of having to set it ?

But you still need to copy originalPath from resolved instead of having to set it ?

@sheetalkamat I looked into this and originalPath is never set on resolved in this code path. The only place it’s set is in nodeModuleNameResolverWorker which isn’t part of the call hierarchy in resolveTypeReferenceDirective. I’m sorry, I’m just not familiar enough with module resolution to understand what you’re wanting to see without a lot more details.

looked into this and originalPath is never set on resolved in this code path. The only place it’s set is in nodeModuleNameResolverWorker which isn’t part of the call hierarchy in resolveTypeReferenceDirective. I’m sorry, I’m just not familiar enough with module resolution to understand what you’re wanting to see without a lot more details.

sorry about that , i missed the fact that realpath is called just above this and not coming from the resolved value. So your current change is right.

I can change it to originalPath and put a warning doc comment on both. My main concern is how easy it is to determine the casing/normalization of the values contained by those properties, and a doc comment should handle that.

Also just noticed that originalPath is @internal, so we could rename it if we ever feel like it.


stampit v3 RFC

I'm starting stampit v3_0 branch. This is RFC, not a final decision.
Why 3.0?

It's gonna be stamp specifications compatible.
The changes are going to be breaking due to the single reason: factory (aka stamp) arguments.
The arguments will be an "options object".

As much compatibility as possible.

The .props(), .init(), .refs(), .methods(), .statics() will be added by default to each stamp produced by stampit (but not by the utility functions).

Handy functions:

Stampit will have utility functions. The compose() will definitely follow the specifications.
Utility functions are: methods(), assign(),  merge(),  assignStatic(),  initialize(),  describe(),  properties(),  set(), html(),  reactComponent(), convertConstructor()...

This work will help us shape up the specifications as we go.
Comments?

The .props(), .init(), .refs(), .methods() will be added by default to each stamp produced by stampit (but not by the utility functions).

Could this be implemented as an optional mixin instead of default behavior? It would be great if the next version of Stampit actually followed the stamp specification, and allowed authors maximum control over both the stamp and object instances.

Yeah, but what is "optional mixin"?
The aim is to strictly follow specs, Eric. Although, I would also like to presereve some compatibility. See the code.
Compatible stampit v2 and v3 syntax:
import stampit from "stampit";
let stamp = stampit().props().refs().init().methods();

Incompatible syntax:
import stampit from "stampit";
let stamp = stampit().statics({ init: () => console.log("created") });

// stampit v2
stamp.init(); // does not print anything!

// stampit v3
stamp.init(); // prints "created"

So, I plan to implement those "props, refs, init, methods" chaning methods as simple overridable statics.
Pros: We would brake as less code as possible.
Cons: the only way to get pure stamps is via utility methods: import {compose, merge, initialize} from "stampit"
How does that sound, @ericelliott ?

Yeah, but what is "optional mixin"?

A stamp.
import { compose, compat } from 'stampit'; // compat is a stamp containing old stamp methods

const myStamp = compose(compat).props().refs().init().methods();

So, basically, in stampit v3 we change the stampit API significantly again?

Yes.

I'll change the original post of this issue to reflect our decision then.
Gimme 5 minutes please.
Thanks!

In fact, we could make a simple shortcut for an API compat version that people can just drop in to replace their 2.0 calls:
import compose, compat from 'stampit';

// export a non-breaking version that adds composables support:
export default compose(compat);

Sure. I'll reflect it too in the top mesasge.

Now that I'm thinking about it, I think this would be fine:
import stampit from 'stampit'; // 2.0 compat API

And if you just want to use the composables API instead:
import { compose } from 'stampit';

So, we are going back to my original version. :)
I'm okay with any of the solutions.

Yeah, I think your original idea is really the best of both worlds. Don't break old call sites, but still give people direct access to the utility functions (including compose).
We'll make the 2.0 compat stamp be the default export.

Looks good guys. I am planning on making react-stampit spec compliant as well. :)

:+1:

:+1: for not breaking things :shipit:

:+1:

For better compatibility of stampit with other modules/frameworks it's better stampit to support the new and instanceof operators. The compatibility will allow stampit to be used in a wider range of (legacy?) code bases. See #187 for example.
Although, like it is now, we would discourage JS dev to use the new and instanceof operators as we are doing it now.
Any objections?

I thought getting away from using new and instanceof was one of the reasons stampit was created. If a code base insists on using these operators, perhaps that code base should be made more flexible?

@troutowicz Sometimes that codebase happens to be the EcmaScript specification.  For example, you can't:
stampit.convertConstructor(Map)

Map expects itself to be called with the new keyword; it will throw an exception otherwise.

@koresar that said, I'm behind supporting it

@troutowicz though, I'm unsure if we're talking about the same thing.

What I mean is that #44
var Stamp = stampit(bla, bla, bla);
var obj = Stamp(ble, ble, ble);
if (obj instanceof Stamp) {
 // true
}

obj = new Stamp(ble, ble, ble);
if (obj instanceof Stamp) {
 // true
}

I'm not sure I understand why supporting that is necessary

Yeah. I would need to show an example. So, in the #187 a person would like to replace classes with stamps and supply them to the Aurelia framework. But he can't, because new AStamp() returns rubbish.
This is the Aurelia's code which create instances of those classes: https://github.com/aurelia/dependency-injection/blob/master/src/container.js#L83
I wish stampit supported the new keyword, it could have gained wider usage.

Avoiding the new keyword is the point of stamps.

@koresar I think I understand what you mean.  AngularJS does this as well.  You should be able to say, use a Stamp as a controller, but since they are instantiated with new, no such luck.
This makes sense.  new should be supported; it should not do anything, but it shouldn't make the stamp blow up.

Would it be easier to add support for factory functions in Aurelia?

@BerkeleyTrue Sure. But the world is biased towards the new. I don't think Aurelia and AngularJS will support factory functions just because of stampit.

That is a good point but I don't think these two worlds are compatible. It really should be one way or the other.

Worlds maybe not compatible. But stampit can be easily made compatible with Aurelia.

If new and instanceof support is an optional mixin, I have no objection. I object strongly to making it core.

At the first glance it is doable. Good idea.

Stampit v3-rc.0 was just released. npm i stampit@rc

Mostly done.
Some things are looking for an implementer.


Agenda for pymbar planning call on Jan 6th, 12 EST/10 MST

Please edit and add things:

What are the finishing bits to have a 3.0.0 release that?

Runs CI
is compatible with python 2.0 and 3.0.
Solves at least all the issues encountered up through the release of 2.1.
includes bugfixes for robustness that were introduced since 2.1

What is the plan moving forward?

phase 1: Robust 'adaptive' solver supporting both u_nk and u_kln input formats, works with python 2.x and 3.x (MRS: should be in 3.0.0 release)
phase 2: Compiled extension to speed up 'adaptive' solvers
phase 3: Fast alternative solvers for big datasets, maybe accessible under another API point (such  as pymbar.densesolvers). We can make these default if we have automatic fallback if they fail to converge.
phase 4: Fast alternative solvers for large datasets that use sparse structure, such as using Zhiqiang's new sparse or stochastic solvers.  (MRS note: the standard adaptive solvers are faster than stochastic solver for essentially any matrix that can be stored in 16 GB. ZT's examples used a dataset where the reduced potential energies were calculated on the fly -- otherwise it wouldn't fit into memory.  I would love to look at the sparse solver more, I'm still trying to figure out how to implement it for more than nearest neighbor).

What is worth salvaging from the 3.0 code?

Speedups for large data sets?  How can we get these with minimal code perturbation so that

How can we best incorporate contributed datasets with CI?

Should we separate out the timeseries code, since it's a modular component useful for other things?

useful features that should be supported eventually (to be implemented contingent on manpower).

calculation of covariance for iterated differences.

Speedups are irrelevant if the code does not work or is not robust.
Our primary focus needs to be: "How do we get to a robust python 2.x and 3.x compatible code that works and can be continuously tested?"
We don't have that yet, and we need that first. We can discuss all of the other things after we formulate a plan to get to this point.

We don't have that yet, and we need that first. We can discuss all of the other things after we formulate a plan to get to this point.

You said "If your group has the time, manpower, expertise, and bandwidth to get it ready for release, you can present a plan for doing so at the meeting". So it seemed like that was supposed t be part of the meeting.
Clearly getting python 2.x and 3.x compatible code that can be tested is first.

Clearly getting python 2.x and 3.x compatible code that can be tested is first.

Great, sounds like we're on the same page here.

Superseded by #255


Fixes/template

I did lots of small things:
Moved some stuff from template.html into _layouts/default.html
Merged all the files in styles/ into one file that's included by default
Made more links use the new-tab class
Other small miscellaneous tweaks

ignore what I just said


Runtime Error 'max_usage'

Hi, thank you so much for developing such a useful tool! It works really well but  sometimes when I run cerberus, it gives such an error:
                    [benchmark][manybugs] deploy failed

                    Runtime Error

                    'max_usage'

May I ask if this is normal or there might be something wrong with my installation?
An example command for me to meet this error: cerberus repair --tool f1x --benchmark=manybugs --bug-index=11

Hi, if this is on master - this is a known bug fixed on dev. The reason is that apparently the results of docker are inconsistent.
In stats.py you can replace the definition of self.mem_usage_gb with this one
        self.mem_usage_gb = round(
            container_stats["memory_stats"].get(
                "max_usage", container_stats["memory_stats"].get("usage", 0)
            )
            / (1024 * 1024 * 1024),
            3,
        )


Cosmos DB account name limited to 24 when the actual limit is 44

The text in Manage an Azure Cosmos Account states for "Account Name":

Enter a name to identify your Azure Cosmos account. Because documents.azure.com is appended to the name that you provide to create your URI, use a unique name.
The name can only contain lowercase letters, numbers, and the hyphen (-) character. It must be between 3-44 characters in length.

The extra 20 characters will be useful in constructing a meaningful name.

Code for the change to https://github.com/pulumi/pulumi-azure/blob/87ec45403dcdc845364bf2cd86aa71620069a540/provider/resource.go#L899 would be something like:
			"azurerm_cosmosdb_account": {
				Tok: azureResource(azureCosmosDB, "Account"),
				Fields: map[string]*tfbridge.SchemaInfo{
					// Max length of a Cosmos DB account name is 44.
					// Source: https://docs.microsoft.com/en-us/azure/cosmos-db/how-to-manage-database-account#create-database-account-via-portal
					azureName: AutoNameWithMaxLength(azureName, 44),
				},
			},

@stack72 - what's acceptable by way of a pull request for something like this?
I'm some distance away from having a machine configured to be able to run make for this repo.


Feature Request Webapp: Cards

Is your feature request related to a problem? Please describe.
We'd like to have teaser elements or grouped content that would fit nicely in a card.
Describe the solution you'd like
For us, we'd like to see two designs:
One that looks like a accordion, without the open/c,lose.

Use Cases:

some additional content in the right column that does not need the added cognitive load of the open/close button
important Filters/Elements that we don't want to the user to be able to hide

And then we have a teaser element, that links to something.

Use Cases:

Product Teasers on the cockpit

Describe alternatives you've considered
We're using the accordion for the first one and the second one is still in our backlog

Thank you for the request. We will discuss it in design system coregroup.


JSON string as value of args parameter

I used to be able to execute this code (ver 4.x I think).., where the input to the generator was a JSON entity.., but have not been able to in more recent code.  Could be something I'm missing.
async function run() {
   await execa('hygen', ['typeGen', 'new', `--args=${JSON.stringify(types[0])}`]).stdout?.pipe(process.stdout);
}

Solved this in this way --
async function run() {
   await execa('hygen', ['typeGen', 'new', `${JSON.stringify(types[0])}`]).stdout?.pipe(process.stdout);
}

added a new index.js file to the generator folder with the following code ..,
module.exports = {
    params: ({ args }) => {
       return JSON.parse(args.name);
  }
}

While this works.. I think there could be a more elegant way to execute this approach..,


today daily problem i solved or not ?

Is there any API that gonna help me to fetch daily problem-related queries like whether is it completed for today or not? or I forgot to attend.

this API only gives you daily problems, it doesn't check whether you solved it or not.
but this system can be built by using 2 API endpoints /daily & /:username/acSubmission

pls share any python code if you have related to this

Thank you i completed code☺


Not working with  mozila

When  I tried in mozila then I got this
navigator.mozGetUserMedia has been replaced by navigator.mediaDevices.getUserMedia
The following error occured:  MediaStreamError { name: "NotReadableError", message: "Failed to allocate videosource", constraint: "", stack: "" }

Please, fix it!

This should have been fixed with #67


IdentityContext

Unable to create an object of type 'IdentityContext'. For the different patterns supported at design time.

Hi. I have same issue when try to call dotnet ef migrations add Identity-commit-Identity --context IdentityContext -o Migrations/Identity command


Automatic PRs for Dependency Updates

Check for a way to enable a renovate bot like functionality within this GitHub repository.

Totally did not see this one. Since you already found out how to merge this I'll just close this with: ❤


Frontend Project Yoga-Info's "Follow Us On Instagram" Section Button Design Improvement

Describe the Feature you want to end
The design of this section is nice but it can be improved by working on the design of the button. Through the adjustment on styling of this button, I can make this section look even better.
Expected behavior
Attractive button styling
Screenshots (optional)
Current Design

Expected Updated Design

Additional context (optional )
N/A

@MohitGupta121 Kindly assign me this issue


types

Background
Currently, in templates,  assemble and other base app, we have the convention of using isApp, isView and so on to identify the "instance type", so that plugins will only be used on certain instances.
Proposal
I propose that we implement one or two methods and associated properties here, in base-plugins, for checking instance "types". I think these methods belong here since "smart plugins" can't be used without this plugin, thus isApp or similar labels are useless otherwise.
Example

.is {Function}: Currently we use the .is method to add a type to an instance. I think this can stay the same, but we can also allow an array of strings to be passed, like app.is(['views', 'collection'])
._is {Array}: we should add an ._is array for storing names passed to .is.
.isType {Function}: this method would take a string or array of strings and return true if an instance has any of the given strings on the ._is array. It would also return true for isType('app') when no other types are defined.

@doowb, anyone else, thoughts?

another idea is to add an isApp getter/setter that would return true if either:

isApp was explicitly defined (as true), or
no other types were explicitly defined

I think that would be good to have directly in base instead of here. the more I think about the more I wonder if the other .is stuff I proposed belongs in base as well... hmm

I like these ideas. I also think that it should be on base directly since we've recently had a couple of issues with the examples not working because they don't include {isApp: true}. I'm assuming that's the reasoning for:

It would also return true for isType('app') when no other types are defined.

Would the ._is array be cumulative or would it get replaced on the child instance? If it's replaced, maybe we can have additional methods that provide conveniences for working with the ._is array:

.isAlso Adds additional types to the ._is array.
.isNot Removes types from the ._is array.


Gemini Refers To Itself As Bard

Gemini refers to itself as Bard or Chat GPT.
https://aistudio.google.com/app/prompts?state={"ids":["1w5HoyeIvM8jxrFfWqqUqGnm0cU4i7iLh"],"action":"open","userId":"100204558454174478902","resourceKeys":{}}&usp=sharing

@f4st4w4rd, Thank you reporting this issue.
This repository is for issues related to website(https://ai.google.dev/) like documentation bugs or improvements. For issues related to Gemini API, we would suggest you to use "Send Feedback" option in Gemini docs. Ref: Screenshot below. You can also post this issue on Discourse forum.


jellyfin: 10.3.0 -> 10.3.1

Motivation for this change
https://github.com/jellyfin/jellyfin/releases/tag/v10.3.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Seems to run? O:)

@nyanloutre @minijackson care to test and confirm this change?

Tested it, works as fine as before ^^

Actually, there are a couple of improvements we could use in this package:

Jellyfin complains that it can't load the Skia image processing library. The .dll and .so are packaged within Jellyfin's $out directory, so a quick and dirty fix would be (for linux x64):

  installPhase = ''
    install -dm 755 "$out/opt/jellyfin"
    cp -r * "$out/opt/jellyfin"

    makeWrapper "${dotnet-sdk}/bin/dotnet" $out/bin/jellyfin \
      --prefix LD_LIBRARY_PATH : "${stdenv.lib.makeLibraryPath [
        sqlite fontconfig freetype stdenv.cc.cc.lib
      ]}:$out/opt/jellyfin/runtimes/linux-x64/native/" \
      --add-flags "$out/opt/jellyfin/jellyfin.dll --ffmpeg ${ffmpeg}/bin/ffmpeg"
  '';

Added dependencies into the LD_LIBRARY_PATH specifically for skia's .so
A way better solution would be to package SkiaSharp for NixOS, but I'm not sure there even is support in NixOS for building C# libraries :-/

Another thing is that Jellyfin complains about the ffmpeg version, but it wants FFmpeg 4.0 only. Currently NixOS ships with FFmpeg 4.1 and FFmpeg 3.4, which makes Jellyfin complain either way. A discussion is happening here for 4.1 support, but I think the consensus is heading towards them bundling their own version of FFmpeg? Which would make things harder for us :cry: (especially if they remove the --ffmpeg option).

Anyways, sorry to dump it all here, @dtzWill if you have time, could you add the Skia part into your PR? If you don't, or just want to push the version, no worries I'll try to find some time ^^


coap.c : encoding of options with lengths larger than 268 is not proper

Adding a coap option with a length larger than 268 is not encoded proper.
Current master:
coap.c, encode_option, line 221
	if (len_size == 1U) {
		res = append_u8(cpkt, (uint8_t)len_ext);
		if (!res) {
			return -EINVAL;
		}
	} else if (delta_size == 2U) {
		res = append_be16(cpkt, len_ext);
		if (!res) {
			return -EINVAL;
		}
	}

the delta_sizeis wrong, it must be delta_size.
Such large options are not used too frequent, but for the proxy_uri or other custom options it may be used.

Thanks!

Thanks!


Remove duplicate images from dataset (train and test)

Found a library to help with image duplication thanks to hashing — https://github.com/idealo/imagededup
Removing duplicates will help make the model more robust and prevent data from leaking from train → test set (and then giving false metrics)
Created a small notebook for this (07_remove_duplicates.ipynb) and it seems to work very well, found ~500/24500 images were duplicates in a few minutes and there were little samples that weren’t (after a series of quick random plots)

Could integrate this workflow to run over all the images every so often (or whenever new data is added to the dataset).

Moved to #53


Is the license change as simple as this?

Hi Victor,
don't you have to get permission from the individual copyright holders before changing the license?
I agree with changing the YARA license to BSD-3-clause.
Cheers,
-Hilko

You're probably right. As the list is short enough I'm going to ask individual copyright holders their opinion about this change.

I thought so too, but I'm also in favor of this change. No need to mail me!

I use a modified version of YARA in an incredibly large, non-open-source project... but I am unsure of how I'm meant to approach the licensing of it. How would one go about being compliant with YARA's license whilst stating a different license for the non-YARA parts? I've done some research but I'd rather not make a mistake and break the rules, so hopefully someone can provide more insight to help me.

IANAL but it’s just 3 clause BSD, which boils down to keep our copyright and license on the files on which they appear in both src and binary form (ie: give us credit for our work) and don’t use our names as endorsements. BSD licenses are considered permissive in that you can incorporate them into a closed source project provided you meet those 3 (in this case) conditions.


Upgrade lodash from ^4.17.11

What is the current behavior?

Per https://github.com/lodash/lodash/issues/4348, an upgrade is required to address a recent security vulnerability CVE-2019-10744
https://github.com/erikras/redux-form/blob/master/package.json#L69 -- dependency currently out of date.

What is the expected behavior?

https://github.com/erikras/redux-form/blob/master/package.json#L69 upgraded to include security fix.
May need to also resolve for dependencies.

Noticing serveral lodash related dependencies here -- will plan to raise PR once all dependencies are upgraded

My 2c: rm yarn.lock; yarn solved this for me, perhaps the ^ dep (compatible with) is semantically correct as it.

My 2c: rm yarn.lock; yarn solved this for me, perhaps the ^ dep (compatible with) is semantically correct as is.

You can also use the yarn resolutions configuration to fix this issue in a project, documentation: https://yarnpkg.com/lang/en/docs/selective-version-resolutions/
Opened this issue mostly to save people the inconvenience and concern.

Published in v8.2.6.


HOT RELOAD: Handle complex message passing and reloading isolates with shared origins

Ryan suggests modifying the origin of the reloaded isolate so it longer matches and complex messages can no longer be sent.
Lars suggests reloading all isolates with a shared origin atomically. This would allow complex message passing applications to be reloaded in a consistent way. We need to add support for stopping all isolates with the same origin simultaneously.

@larsbak @turnidge FYI

Adding some context so I don't forget.
By shared origin we mean isolates derived from spawnFunction. Logically such isolates have the same code, which is why they are allowed to pass arbitrary objects when messaging each other. When the code of one isolate in a spawnFunction family changes, either we must disallow further use of arbitrary messages or make the same change to all other isolates in the family.
A related bug is that the implementation of spawnFunction will load the child isolate from sources, which could have changed since the parent was loaded. The parent and child would be permitted to send arbitrary objects in messages to each other with classes the other isolate potentially does not have.