Update Microsoft.Peering 2020-01-01-preview API specification

Remove readonly flag from microsoftSessionIPv4Address and microsoftSessionIPv6Address properties in BgpSession data model definition.
Add an example for creating a direct peering of directPeeringType IxRs.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk


Add support for Web Manifest splashscreen

I've added it to Polymer Starter Kit and we should add it in here too: https://github.com/PolymerElements/polymer-starter-kit/commit/62407ffd7a88735a400b0b244176865eb3b3f70a

  "start_url": "/?homescreen=1",
  "background_color": "#3E4EB8",
  "display": "standalone",
  "theme_color": "#2E3AA1"

Let's focus on getting the 192x192 working well (per DevSummit as the 384 sizes appear to have fallen out of favor): https://github.com/PolymerElements/polymer-starter-kit/blob/62407ffd7a88735a400b0b244176865eb3b3f70a/app/manifest.json#L18


GitHub Issue #254: Rollbar Exception Logging ignores File and Line on Exception itself

Added file and line information to the last frame of the exception stack trace by fetching data from the Exception object.
Ready for review.

LGTM


Interval is out of spec

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-11645
The DataType interval is currently implemented as a i32 or i64. However, that is incorrect and does not follow the spec.
{code:java}
// DAY_TIME - Indicates the number of elapsed days and milliseconds,
//   stored as 2 contiguous 32-bit integers (8-bytes in total).  Support
//   of this IntervalUnit is not required for full arrow compatibility.
{code}
See [https://github.com/apache/arrow/blob/master/format/Schema.fbs]
E.g. I think that IntervalDayTimeType should be a struct(i32,i32) with a C representation or something, which is how it is defined in the spec.
This would also make it easier to read from and write to, as we do not need to transmute or bitmask the values to get the correct values.

@alamb is it solved?
https://github.com/apache/arrow-rs/blob/7a74465f2b8b622428e562f23298b7cce16886f7/arrow/src/datatypes/types.rs#L99-L103
https://github.com/apache/arrow-rs/blob/7a74465f2b8b622428e562f23298b7cce16886f7/arrow/src/datatypes/types.rs#L224-L264

I agree this issue is not tracking anything actionable anymore. Thanks for the spot @waitingkuo


Fails when performing liftover on opposite strand

Chainfile:
chain 0 a 100 + 0 100 b 100 + 0 100 34
100

Code:
let from = Coordinate::try_new("a", 78, Strand::Negative)?;
let to = Coordinate::try_new("a", 13, Strand::Negative)?;

let interval = Interval::try_new(from, to)?;
dbg!(chain.liftover(&interval));

Result:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: InvalidInterval(MismatchedStrandDuringClamp(Positive, Negative))'

Hi there! Apologies for not seeing this until now. I've fixed this in https://github.com/stjude-rust-labs/chainfile/commit/bd4d7c2837bbecede6bcc9c20d983349a082fcc6. You can update to v0.2.1 for the fix.
In short, the rust-lapper crate is used behind the scenes to store liftover intervals. In the liftover machine, I failed to also ensure the strand matches before attempting the liftover. Thanks for the bug report!


How to contribute

I'd like to add a feature but it's not clear to me how to get the demo running. Any help would be appreciated!

You should be able to just npm install and then run gulp

ah ok I was missing the gulp. thanks.


Compartment transforms can't seem to add endowments

The failure I'm seeing in import-bundle (in https://github.com/Agoric/agoric-sdk/pull/762) seems to stem from Compartment` transforms not working the way I think they're supposed to. The minimal test case is:

function transform(rewriterState) {
  const { src, endowments } = rewriterState;
  return { src: src.replace('replaceme', 'substitution'),
           endowments: { added: 'by transform', ...endowments },
         };
}

test.only('transforms can add endowments', t => {
  const src = '(function f4(a) {  return `replaceme ${added} ${a}`; })';
  const transforms = [transform];
  const c = new Compartment({}, {}, { transforms });
  const f4 = c.evaluate(src);
  const out = f4('ok');
  t.equal(out, 'substitution by transform ok');
  t.end();
});

I'll add a branch with this appended to test/compartment.test.js for convenience.
What I'm seeing is that the c.evaluate fails with ReferenceError: added is not defined. When I read through compartment-shim.js, I thought each transform accepted a { src, endowments } object and returned a new one of the same shape, so my transform() ought to leave an added endowment that should show up by the time f4 is evaluated.
I also tried providing the transforms as an option to c.evaluate(src, { transforms }), but that didn't work either.

@michaelfig see if you can think of a reason why transforms are failing here.. nothing's jumped out at me.
BTW when I run yarn test to exercise this new test, I see the failed assertion go flying by, and the summary at the end of the test run is not so helpful in finding out the nature of the error:
  985 passing (3s)
  1 failing

  1) test/compartment.test.js test count !== plan:

      test count !== plan
      + expected - actual

      -11
      +1

error Command failed with exit code 1.

I'm grateful that tap told me the file where the failure occurred, but gosh it'd be nice if it told me the test case, or the exception. Is there a best-practice I'm missing on how to run or write the tests to get more information when one of them fails?
In case it helps anybody, I used yarn exec -- tap --no-esm test/compartment.test.js to run fewer tests (and give me a better chance of finding the error in the output). Additionally, I was able to change the test to a test.only and get even less noise (but note that, unlike tape, marking a tap test as test.only in one file doesn't inhibit the execution of all tests in other files, so you have to combine the two techniques. Also you might have to add --only; I'm not sure.

I'm adding this to the zoe-alpha project because it's a blocker for the https://github.com/Agoric/agoric-sdk/issues/477 new-ses work.

@erights any ideas?

Actually, this only blocks kernel-on-new-ses if swingset actually uses transforms to add endowments. @michaelfig I was assuming the metering/injection code used that feature (to add an endowment with the actual meter object to be decremented). If we're not using that (or if it's a SES1 thing only), then we can reduce the priority of this.

For (more-)secure metering, we actually need to put something in the global lexical scope (like SES1 endowments did), not on the globalThis (like SES2 endowments do).  Otherwise, the user can bypass the transform's restrictions by concocting a string just with an indexed property get on globalThis to access the meter directly, which would be bad.
I know JF was talking about supporting the global lexical scope, but even if that exists, I'm certain that the transforms mechanism does not allow using it yet.

Since I'm familiar with the SES1 per-evaluation endowments mechanism, I could add such a thing to the SES-shim. However, it would be peculiar to the shim.  Therefore we are also unlikely to propose such a mechanism.
Are the per-compartments endowments problematic because they are per-compartment, or because they're on the global object? If the only problem is the latter, then we could get per-compartment global lexical endowments using the "optimizer" mechanism in the 8 magic lines. This also fits better with what Moddable can smoothly implement (att @phoddie @patrick-soquet ). The shim would only support const bindings, but that's all that what we need here. Yes?
I suspect tc39 would also be more tolerant to see a per-compartment global lexical environment added to the proposal, as a construction option, than to see one added as a per-evaluation option.

Are the per-compartments endowments problematic because they are per-compartment, or because they're on the global object?

The latter.  As long as the meter endowment is at least a const binding, it can only be referenced directly by name in an evaluation.  That makes it possible for a transform to ensure that the evaluated code doesn't reference it.


Mention requirements in README

I think this closes: #41
Also Minimum required Go version seems to be 1.8. Checked that in Docker containers.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #77   +/-   ##
=======================================
  Coverage   23.61%   23.61%
=======================================
  Files          13       13
  Lines        1080     1080
=======================================
  Hits          255      255
  Misses        813      813
  Partials       12       12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ff2198...3b5e7f4. Read the comment docs.

I don't even have go 1.8; why do we need such a recent version?

Go getting with 1.6:
package context: unrecognized import path "context" (import path does not begin with hostname)

thanks @dawidd6 I just realised that 1.8 < 1.12 so my concern was unfounded :)

thanks :)


pip fails to install google-cloud-logging: error in google-api-core setup command: 'install_requires'

I am trying to install google-cloud-logging using pip but I am getting the following error:
koike@koike-sles12:~/resource-agents/heartbeat> sudo pip install --upgrade google-cloud-logging
Downloading/unpacking google-cloud-logging
  Downloading google-cloud-logging-1.6.0.tar.gz (104kB): 104kB downloaded
  Running setup.py egg_info for package google-cloud-logging

    warning: no files found matching '*.json' under directory 'google'
    warning: no files found matching '*.proto' under directory 'google'
    warning: no previously-included files matching '*.pyc' found anywhere in distribution
    warning: no previously-included files matching '__pycache__' found anywhere in distribution
Downloading/unpacking google-cloud-core<0.29dev,>=0.28.0 (from google-cloud-logging)
  Downloading google-cloud-core-0.28.1.tar.gz
  Running setup.py egg_info for package google-cloud-core

    warning: no previously-included files matching '*.pyc' found anywhere in distribution
    warning: no previously-included files matching '__pycache__' found anywhere in distribution
Downloading/unpacking google-api-core[grpc]<2.0.0dev,>=1.0.0 (from google-cloud-logging)
  Downloading google-api-core-1.2.1.tar.gz (53kB): 53kB downloaded
  Running setup.py egg_info for package google-api-core
    error in google-api-core setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in futures>=3.2.0;python_version<"3.2" at ;python_version<"3.2"
    Complete output from command python setup.py egg_info:
    error in google-api-core setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in futures>=3.2.0;python_version<"3.2" at ;python_version<"3.2"

----------------------------------------
Cleaning up...
Command python setup.py egg_info failed with error code 1 in /tmp/pip_build_root/google-api-core
Storing complete log in /root/.pip/pip.log

Specify the API at the beginning of the title (for example, "BigQuery: ...")
General, Core, and Other are also allowed as types
Doesn't apply
OS type and version
Suse 12 (image at GCE)
Python version and virtual environment information python --version

koike@koike-sles12:~/resource-agents/heartbeat> python --version
Python 2.7.13

google-cloud-python version pip show google-cloud, pip show google-<service> or pip freeze

koike@koike-sles12:~/resource-agents/heartbeat> pip freeze
M2Crypto==0.22.5
PyYAML==3.12
cachetools==2.1.0
cffi==1.5.2
cloudregister==7.0.7
configobj==5.0.6
crmsh==3.0.0
cryptography==1.3.1
cssselect==0.9.1
decorator==3.4.0
enum34==1.1.3
gcemetadata==0.3.2
google-api-python-client==1.7.3
google-auth==1.5.0
google-auth-httplib2==0.0.3
google-compute-engine==2.7.5
httplib2==0.11.3
idna==2.0
ipaddress==1.0.14
lio==4.1
lxml==3.6.1
parallax==1.0.1
pexpect==3.1
pssh==2.3.1
pyOpenSSL==16.0.0
pyasn1==0.4.3
pyasn1-modules==0.2.1
pycparser==2.10
pycurl==7.19.0
pygobject==3.20.1
python-dateutil==2.4.2
pyudev==0.16.1
requests==2.11.1
rsa==3.4.2
six==1.11.0
suds==0.4
tcm==4.1
uritemplate==3.0.0
wsgiref==0.1.2

Stacktrace if available
N/A
Steps to reproduce
Instanciate a sles12 in GCE and execute

sudo zypper install python-pip
sudo pip install --upgrade google-cloud-logging

Code example
N/A

It seems your pip version is out of date. Can you try upgrading pip to see
if that helps?
On Mon, Jun 25, 2018 at 7:27 AM Helen Koike<EMAIL_ADDRESS>wrote:

I am trying to install google-cloud-logging using pip but I am getting the
following error:
koike@koike-sles12:~/resource-agents/heartbeat> sudo pip install --upgrade google-cloud-logging
Downloading/unpacking google-cloud-logging
Downloading google-cloud-logging-1.6.0.tar.gz (104kB): 104kB downloaded
Running setup.py egg_info for package google-cloud-logging
warning: no files found matching '*.json' under directory 'google'
warning: no files found matching '*.proto' under directory 'google'
warning: no previously-included files matching '*.pyc' found anywhere in distribution
warning: no previously-included files matching '__pycache__' found anywhere in distribution

Downloading/unpacking google-cloud-core<0.29dev,>=0.28.0 (from google-cloud-logging)
Downloading google-cloud-core-0.28.1.tar.gz
Running setup.py egg_info for package google-cloud-core
warning: no previously-included files matching '*.pyc' found anywhere in distribution
warning: no previously-included files matching '__pycache__' found anywhere in distribution

Downloading/unpacking google-api-core[grpc]<2.0.0dev,>=1.0.0 (from google-cloud-logging)
Downloading google-api-core-1.2.1.tar.gz (53kB): 53kB downloaded
Running setup.py egg_info for package google-api-core
error in google-api-core setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in futures>=3.2.0;python_version<"3.2" at ;python_version<"3.2"
Complete output from command python setup.py egg_info:
error in google-api-core setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in futures>=3.2.0;python_version<"3.2" at ;python_version<"3.2"

Cleaning up...
Command python setup.py egg_info failed with error code 1 in /tmp/pip_build_root/google-api-core
Storing complete log in /root/.pip/pip.log

Specify the API at the beginning of the title (for example,
"BigQuery: ...")
General, Core, and Other are also allowed as types
Doesn't apply
OS type and version
Suse 12 (image at GCE)
Python version and virtual environment information python --version

koike@koike-sles12:~/resource-agents/heartbeat> python --version
Python 2.7.13

google-cloud-python version pip show google-cloud, pip show
google- or pip freeze

koike@koike-sles12:~/resource-agents/heartbeat> pip freeze
M2Crypto==0.22.5
PyYAML==3.12
cachetools==2.1.0
cffi==1.5.2
cloudregister==7.0.7
configobj==5.0.6
crmsh==3.0.0
cryptography==1.3.1
cssselect==0.9.1
decorator==3.4.0
enum34==1.1.3
gcemetadata==0.3.2
google-api-python-client==1.7.3
google-auth==1.5.0
google-auth-httplib2==0.0.3
google-compute-engine==2.7.5
httplib2==0.11.3
idna==2.0
ipaddress==1.0.14
lio==4.1
lxml==3.6.1
parallax==1.0.1
pexpect==3.1
pssh==2.3.1
pyOpenSSL==16.0.0
pyasn1==0.4.3
pyasn1-modules==0.2.1
pycparser==2.10
pycurl==7.19.0
pygobject==3.20.1
python-dateutil==2.4.2
pyudev==0.16.1
requests==2.11.1
rsa==3.4.2
six==1.11.0
suds==0.4
tcm==4.1
uritemplate==3.0.0
wsgiref==0.1.2

Stacktrace if available
N/A
Steps to reproduce
Instanciate a sles12 in GCE and execute

sudo zypper install python-pip
sudo pip install --upgrade google-cloud-logging

Code example
N/A

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/5532,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc-ySaczc7rx7iIrpRNr7Ct4T688Lks5uAPM7gaJpZM4U2QYS
.

yes, updating pip with sudo pip install --upgrade pip worked, thanks!


Official updates on stable branch have stopped since a month on stable channel

@soniaappasamy I can see that the tailscale qpkg for QNAP is not updating along with other operating systems on official stable channel website here
https://pkgs.tailscale.com/stable/#windows
QNAP is stuck on 1.74.0 while others are at 1.76.1
Even unstable channel builds have started to lag. Just wondering if this is an issue or QNAP build will catch up with next official release?
Thanks.

Thank you for asking. We are aware of the delay and working through it. The plan is for the QNAP build to catch back up with the other releases once we sort it out.
I'll leave this ticket open until the QNAP package is caught up.


Asset can be registered with null core exchange rate, but not updated

We have to sure all paths handle null core exchange rate properly, because even if we require non-null core exchange rate for new assets, on already-deployed chains we still have to deal with existing assets that were created with the null rate.

I think CER is validated here and here, unless asset_create_operation::validate() had never been called.

This issue was moved to bitshares/bitshares-core#169


Optimization fails to start on Windows

Describe the bug
When clicking start optimization, nothing happens and the program freezes.
Hopsan version
It happens on Windows with 2.17.0, but seems to work fine on Linux. When I downgrade to 2.16 it works again on Windows.
I will do some more testing. Unfortunately my debugger does not work on Windows at the moment, so it is difficult to trace the error. It could be related to changes to the progress bar. I tried with progress bar both enabled and disabled though, and there were no difference. I also tried both multithreaded and singlethreaded simulation with no effect.

My debugger is working and I found where the problem lies,
Its not the progress bar, its in OpsWorker.cpp line 147, the for loop get stuck looping for ever, due to the counter i getting decremented on line 174. At least in my test case.

why it works on Linux I do not know, code should not depend on platform. unless we trigger undefined behavior somehow.

well, the code depends on opsRand() so random functionality my differ

Actually line 165: x = min + opsRand()*(max-min) with min=0, max=1 always return the same value. So the random function does not seem to be producing random values. Either the new random code is wrong, or not initialized properly, or there is a bug in the MinGW64 version that we are using.

Aha, found the problem, the random_device, default_random_engine and maybe distribution need to be static, otherwise the are recreated on every call and will give the same value since it returns the first random value in some sequence of random values. Not sure how it works exactly, working on PR

Even though I have fixed the problem (I hope), the program seems to generate the same sequence of random numbers every time it runs.
Not sure I like that. The old code seem to seed based on the current time, I do not know how the new c++11 code seeds.


Possible to get statistics back for DbContext.SaveChanges?

I'd like to write some automated tests to measure how much database IO is occurring during particular processes in our application and ensure they never go above a certain threshold.
Outside of EF, you would:

Open SQL Server Management Studio
set io statistics on
...run your queries
... look at the response returned in the Messages window see the statistics.

So I'm wondering if there is some way to get these same measurements as part of the call to DbContext.SaveChanges.  I think you could override the SaveChangesAsync and SaveChanges methods to execute the "set io statistics" statement, but doing it that way would result in an extra database hit and also I wouldn't know how to get the statistics back after SaveChangesAsync was complete.
Any thoughts or ideas?

@joshmouch We discussed this and we don't have any really good answers. It seems to us that overriding SaveChanges and issuing the commands is probably the best way to go--yes there will be some additional overhead, but collecting the statistics is also adding overhead, and we're expecting this is done in dev/debug anyway. I'm not sure how you would get the statistics back, but you can use the underlying SqlConnection object after SaveChanges to query the database for whatever is needed, so if you figure out how to do this outside of EF, then the same thing should work with EF.

Greetings @ajcvickers,
I've attempted to do something similar to the OP by implementing IDbCommandInterceptor within EF Core.
However, despite how I have attempted to attach to the connection's InfoMessage event, I cannot seem to receive the events themselves.
I've dumbied up a simple app to confirm that it should work as I had expected outside of EF Core. This seems to follow your stated principle of "what should work outside EF Core, should work inside". Have I overlooked something in my implementation or expectations?
`    public class BareTrackingCommandInterceptor : IDbCommandInterceptor
{
bool attached = false;
    public BareTrackingCommandInterceptor()
    {
    }

    void InfoMessageHandler(object sender, SqlInfoMessageEventArgs args)
    {
        Debug.WriteLine(args.Message);
    }

    void attachStatisticsTracking(SqlConnection sqlConnection)
    {
        if (attached)
        {
            sqlConnection.InfoMessage += InfoMessageHandler;
            attached = true;
        }
    }

    void enableLogicalReads(SqlCommand command)
    {
        command.CommandText = "SET STATISTICS IO ON;" + command.CommandText;
    }

    public InterceptionResult<DbCommand> CommandCreating(CommandCorrelatedEventData eventData, InterceptionResult<DbCommand> result)
    {
        return result;
    }

    public DbCommand CommandCreated(CommandEndEventData eventData, DbCommand result)
    {
        return result;
    }

    public InterceptionResult<DbDataReader> ReaderExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<DbDataReader> result)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return result;
    }

    public InterceptionResult<object> ScalarExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<object> result)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return result;
    }

    public InterceptionResult<int> NonQueryExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<int> result)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return result;
    }

    public Task<InterceptionResult<DbDataReader>> ReaderExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<DbDataReader> result, CancellationToken cancellationToken = default)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return Task.FromResult(result);
    }

    public Task<InterceptionResult<object>> ScalarExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<object> result, CancellationToken cancellationToken = default)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return Task.FromResult(result);
    }

    public Task<InterceptionResult<int>> NonQueryExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<int> result, CancellationToken cancellationToken = default)
    {
        enableLogicalReads(eventData.Command as SqlCommand);
        attachStatisticsTracking(eventData.Connection as SqlConnection);
        return Task.FromResult(result);
    }

    public DbDataReader ReaderExecuted(DbCommand command, CommandExecutedEventData eventData, DbDataReader result)
    {
        return result;
    }

    public object ScalarExecuted(DbCommand command, CommandExecutedEventData eventData, object result)
    {
        return result;
    }

    public int NonQueryExecuted(DbCommand command, CommandExecutedEventData eventData, int result)
    {
        return result;
    }

    public Task<DbDataReader> ReaderExecutedAsync(DbCommand command, CommandExecutedEventData eventData, DbDataReader result, CancellationToken cancellationToken = default)
    {
        return Task.FromResult(result);
    }

    public Task<object> ScalarExecutedAsync(DbCommand command, CommandExecutedEventData eventData, object result, CancellationToken cancellationToken = default)
    {
        return Task.FromResult(result);
    }

    public Task<int> NonQueryExecutedAsync(DbCommand command, CommandExecutedEventData eventData, int result, CancellationToken cancellationToken = default)
    {
        return Task.FromResult(result);
    }

    public void CommandFailed(DbCommand command, CommandErrorEventData eventData)
    {
    }

    public Task CommandFailedAsync(DbCommand command, CommandErrorEventData eventData, CancellationToken cancellationToken = default)
    {
        return Task.CompletedTask;
    }

    public InterceptionResult DataReaderDisposing(DbCommand command, DataReaderDisposingEventData eventData, InterceptionResult result)
    {
        return result;
    }
}

`
Thanks for any assistance you may offer.
Kind regards & Happy Thanksgiving,
Jeff

Similarly, I attempted to implement an IDbConnectionInterceptor and none of the methods on it are being called, including:

ConnectionOpening
ConnectionOpeningAsync
ConnectionOpened
ConnectionOpenedAsync
ConnectionClosing
ConnectionClosingAsync
ConnectionClosed
ConnectionClosedAsync
ConnectionFailed
ConnectionFailedAsync

@windhandel this issue is from 2017 and things have changed since - can you please open a new issue with a code sample where you use IDbConnectionInterceptor? That approach should work.

Please see the new issue I created for the workaround EF Core correctly causing the SqlConnection's InfoMessage event to be fired.


Matrix: Naming convention changes.

@timabbott @neiljp How is this for a naming convention? I have manually editted the username for a few networks mentioned here: https://github.com/matrix-org/matrix-appservice-irc/wiki/Bridged-IRC-networks.
I'll be adding the matrix integration document where I will stress upon the naming convention of the matrix user.
screenshots:

I have updated this!

Merged, thanks @rheaparekh!  I added efcdc712bfda8b26bb9fd0a110ab009bfe65f4a6 to clean this up a bit more; I stopped editing because probably we'll want the main docs to live on the /integrations site.
I think we'll want to do more work on the configuration process before we can really recommend this.  Right now, the instructions really assume you're working in a clone for the python-zulip-api project, so we either need the instructions to start with telling you to clone this, or have an option for how to pass the configuration to matrix_bridge.py.


Rename HuggingChat-Mac.app to Hugging Chat.app

To align with the iOS version. Please consider.

yes!


The actions 'Save query as...' and 'Save query' can't be cancelled by clicking the 'Cancel' or 'X' button on the prompt native dialog

Storage Explorer Version: 1.23.0-dev
Build Number: 20211220.7
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.0.1
Architecture: ia32/x64
How Found: AD hoc testing
Regression From:  Previous release (1.22.0)
Steps to Reproduce

Expand one storage account -> Tables.
Open one table -> Click 'Query' -> Click 'Save query as...'.
Click 'Cancel' on the prompt 'Save query' dialog.
Check there is no 'Save query' activity log.

Expected Experience
There is no 'Save query' activity log.
Actual Experience
There is a 'Save query' activity log.

Verified this issue on the build 20220104.2. Fixed.


add ink client content

This pull request has been deployed to Vercel.

Latest commit:
a2bfec6

✅ Preview:
https://polkadot-bootcamp-bngmn1jym-openedu101.vercel.app

🔍 Inspect:
https://vercel.com/openedu101/polkadot-bootcamp/9FjsBDyynuV56TKmkWpU96fAaFA3

View Workflow Logs


Library size normalization?

Hi,
I see that TRUST reports the frequency of a given sequence, defined as the number of reads belonging to that contig divided by the total number of CD3R reads. I see also that BCR and TCR frequency are normalized separately. It was not clear to me whether the overall library size is used for normalization.
Should downstream analyses correct for library size, or has that been accounted for in TRUST4? The question is applicable only when comparing absolute abundance between samples, of course.

The report file does not take the library size into account. So if your downstream analysis involves infiltration level, you need to normalize with the library size (total reads in the bam or fastq file).

Clear, thank you!


When a Translation is Missing it get bugged

It just shows undefined array key error instead of using english translation for example.

the problem was on filament-types package and it's fixed now on the last version just update your package


Rails 6 EventProf Empty

Tell us about your environment
Ruby Version: 2.6.5
Framework Version (RSpec, Minitest, FactoryGirl, Rails, whatever): RSpec 3.8, Rails <IP_ADDRESS>
TestProf Version: 0.11.3
What did you do?
Ran env EVENT_PROF_TOP=3 EVENT_PROF_EXAMPLES=1 EVENT_PROF='factory.create,perform.active_job,sql.active_record' bundle exec rspec spec/system
What did you expect to happen?
I expected to see EventProf results...
What actually happened?

Details
When I attached and set a breakpoint here: https://github.com/palkan/test-prof/blob/master/lib/test_prof/event_prof/instrumentations/active_support.rb#L12
The result for (finish - start) was always 0.
When I changed the code to the following:
::ActiveSupport::Notifications.subscribe(event) do |event|
  yield (event.duration / 1000.0)
end

I started getting profiling results!
Code was based on: https://blog.bigbinary.com/2019/04/24/rails-6-adds-cpu-time-idle-time-and-allocations-to-activesupport-notifications-event.html

Do you use something like Timecop to freeze the time?
As I see, the old style subscribers (with |_event, start, finish, *| args) use Time.now, while the new one, Evented, uses Concurrent.monotonic_time.
As I see, the difference is intentional: https://github.com/rails/rails/commit/b9a9131fdb1a3e8aad7a18741c84f16ac33ddeaf
I think, using monotonic_subscribe instead of subscribe should fix this in Rails 6. Though for earlier versions we need a different way of dealing with patched Time.now.

We do use Timecop -- I was wondering if something like this might have been at the root of things but am just discovering this codebase.
Can I help assist further?

Can I help assist further?

Thanks but I've already started working on a fix)

@philipbjorge Fixed in master. Will be released soon (after we finish #178).

@palkan -- Really appreciate it!
This library just helped us shave 30-45% off of our system spec suite runtimes with potential downstream savings for other projects.
https://github.com/dequelabs/axe-matchers/issues/94


test pf2 rank with logisitic reg

adding function that tests different parafac2 ranks with logistic regression

@pascoesean This looks good to me!


Can a user define custom backend executors?

At the moment the answer is:

No, fractal users cannot define custom parsl executors. They can only use the ones that are already available on the server (for their specific cluster), or they may ask the fractal admin to add new ones.

We can think about this later, if needed.

Renaming this in view of the new SLURM backend.
Also, from https://github.com/fractal-analytics-platform/fractal-demos/issues/21#issuecomment-1326308014

Note that the new SLURM backend will be simpler to configure.
We may still want to have "executors", which will only be defined by the set of (partition name, cpu requirements, memory requirements) -- see https://github.com/fractal-analytics-platform/fractal-server/issues/278. For those executors, reading from a file is trivial.

Moving this discussion here:
Very good point. If we're anyway opening a slurm job per task & parallelization object (e.g. for each well-image that we apply illum_corr to), then we can even consider making this more granular, i.e. exposing CPU & mem options directly. But a set of predetermined executors may be an easy start :)

Closed in favor of #580


⚠ ClamAV has degraded performance

In 4ecbd32, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 442 ms

Resolved: ClamAV performance has improved in faa2a2e.


Software Bridge fail

Log:
host.iobroker | 2022-03-08 20:36:39.178 | error | instance system.adapter.nuki.0 terminated with code 6 (UNCAUGHT_EXCEPTION)
nuki.0 | 2022-03-08 20:36:38.451 | warn | (19537) Terminated (UNCAUGHT_EXCEPTION): Without reason
nuki.0 | 2022-03-08 20:36:38.449 | info | (19537) terminating
nuki.0 | 2022-03-08 20:36:38.447 | info | (19537) cleaned everything up...
nuki.0 | 2022-03-08 20:36:38.434 | error | (19537) Cannot read property 'stateName' of undefined
nuki.0 | 2022-03-08 20:36:38.432 | error | (19537) TypeError: Cannot read property 'stateName' of undefined at initNukiDeviceStates (/opt/iobroker/node_modules/iobroker.nuki/main.js:318:60) at updateAllLockStates (/opt/iobroker/node_mo
nuki.0 | 2022-03-08 20:36:38.429 | error | (19537) uncaught exception: Cannot read property 'stateName' of undefined
nuki.0 | 2022-03-08 20:36:38.420 | info | (19537) Lock list requested: http://<IP_ADDRESS>:8080/list?token=xxxxxx
Same URL opened with Webbrowser: [{"nukiId":xxxxx697,"name":"Zuhause"}]

Hi,
I'll look in to it.
Regards
Sven

Hi there,
there seems to be something wrong with your response in the web browser. I've setup a software bridge with a Nuki smart lock:
My URL:
http://<IP_ADDRESS>:8080/list?token=xxxxxx
My response:
[{"nukiId":xxxxxx201,"name":"Server-Raum","lastKnownState":{"state":255,"stateName":"unknown","batteryCritical":false,"timestamp":"2022-03-20T09:17:11+01:00"}}]
The adapter expects the node "lastKnownState" in order to be able to process the response.
What kind of Nuki device is paired to your software bridge? Is the (remote) access working?
I could prevent this error message from appearing, but unless the bridge (or device) is giving back some kind of state, the adapter will not work anyway...
Regards
Sven


[Enhancement] - Create an icon for the website

Create an icon for the website matching its purpose

Hello, I would like to work on this issue

Sure @manisanghsharma go ahead

#6

Great @manisanghsharma


Problem with TaskScheduler and HardwareSerial

I want to get GPS data from an NEO6N GPS module.
That alone works fine and i get the GPS NEMA strings from the second serial interface on my ESP32.
But when im using the TaskScheduler and call sched.execute(); perdiodicly i only got these cryptic output:
(i read in loop() same as before - only difference is that i also call the execute)
$GPGSV,۲s�v�W␂��b␌      $%�R��R3D␚i*2[�� ␂␂���
c␂��K�&&b���bR�␒5
$GP��
0�vT0a␂␆���h!␂2��ē␙b�j�B3"␂�L4L0#��si␂�O��b���b��b��b* ␃b`��2�a␀��␐)"��b�)�dh&3��"2�ٚ�"␀k␂S!(&20,29,04,58,06!(
���NWH"���␄a"��ɉHb␒��@@b&&��kiH ����HB␅4�,110,30,04,58��ѶB␀b�b]6␐ ␂␆�9�iەII��   �`�4�s/Hd�R-*␕ib�b��b��b��␅��r␐�␀␂�R0␒␇X&�ёK!�͂!␂��:H b␅گ␐!b�␑�2$(b'␓b���b��b��b��R␒@k␀���� ␂2��6␃b�␂��'␄A#��MH��R� ␄!f��i�␀"�͓␓Ɗ�b��b��b���۩2"␋ �"�&␋��2��+␒�̆�*�}IM J;␡j␅mb␒s+D␆␀b���␔uMY�3,1,12,�@j"�␃�8 "��dP�b���␕K#�ͨ`␔i␂F�2␃�i␂�␋�> "��GvH�:A␝MY�3,1,12��A␀j��S�
j␐�␓)r��R��j
$GP�H�I␒r-B�)␂&rI2J)"�-␛��S�� B�}�

So you can see, some chars are ok, but most of them are corrupted. Looks like TaskScheduler will interference with the serial comm.
Does anyone have any ideas whats going wrong here? Or is this a TaskScheduler restriction?

What compile parameters are you using? What version of TS are you on?
Can you send me the code?

I had found the problem now... For others with the same Problem:
The ESP32 light sleep in my own set void sleepCallback (unsigned long aDeltaTime) destroyed the timing facepalm
@arkhipenko Thank you for your quick response anyway

I suspected that but wanted to make sure, hence the questions. Good luck!


chore: add declare_tx fixture to compilation tests

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#298 : 2  dependent PRs (#348 , #384 )
#515  👈
#439
#391
#383 : 1 other dependent PR (#392 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.24%. Comparing base (c1bd861) to head (f29689e).

Additional details and impacted files
@@                                  Coverage Diff                                   @@
##           arni/declare/compilation/restruct_gateway_compiler     #515      +/-   ##
======================================================================================
- Coverage                                               83.57%   83.24%   -0.33%
======================================================================================
  Files                                                      37       37
  Lines                                                    1808     1791      -17
  Branches                                                 1808     1791      -17
======================================================================================
- Hits                                                     1511     1491      -20
- Misses                                                    222      223       +1
- Partials                                                   75       77       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Another branch was merged into sequencer repo.


rerun feeder lacking permissions to access the state db

Looks like I missed the required dynamo permissions for the rerun feeder when refactoring the permission out to each of the lambdas. I think we just need to add something like:
    - Effect: "Allow"
      Action:
        - dynamodb:Query
        - dynamodb:Scan
        - dynamodb:GetItem
        - dynamodb:BatchGetItem
        - dynamodb:DescribeTable
      Resource:
        - !GetAtt StateTable.Arn
        - !Join ['', [!GetAtt StateTable.Arn, '/index/*']]

We should probably run some tests though to make sure no other permissions have been missed here.


Remove the "(forced)" part in the label for forced closed captions

Have you read the Tutorials?
Yes.
Have you read the FAQ and checked for duplicate open issues?
Yes.
What version of Shaka Player are you using?
4.3.7
Please ask your question
Is there any way to remove the (Forced) part of a label from a Closed Captions instance when the call to addTextTrackAsync has the parameter force==true? We don't want our users to read that the captions are "forced". Thanks.

You could hide the captions selector, through the UI configuration.
https://shaka-player-demo.appspot.com/docs/api/tutorial-ui-customization.html

Thanks @absidue but if I do that the captions won't show on iOS

Even when calling: player.selectTextTrack(track,true) and player.setTextTrackVisibility(true) the subtitles do not show on iOS. Moreover what if I wanted to add multiple text tracks in different languages for users to choose from? They would not be able to select the track they wanted as they don't appear listed on the iOS default Video Controls

I'm confused why you would use forced captions but want the user to have control over which caption is used, the whole point of using forced captions is that you don't want the user to have control.

@absidue For some reason I'm getting a "code 2012:External text tracks cannot be added in src= because native platform doesn't support it." for iOS if I don't set the parameter force to true. This happens in the project I'm working on, but I cannot replicate it using the demo.
Thanks @avelad is there any solution for captions not showing when going full screen on iOS?

That's what we are aiming for in the future, but for now, we will need to modify a lot of manifests.

@marcosthejew what value are you passing for kind? Make sure that it's a valid one captions or subtitles https://developer.mozilla.org/en-US/docs/Web/API/TextTrack/kind

Thanks @absidue and @avelad , upgrading to<EMAIL_ADDRESS>in the requirements fixed the issue

Sorry, spoke too soon. The captions load fine but with an addition unrecognized() track. here is the code:
shaka-test.html :
`

`
myapp.js
`const manifestUri =
    'https://demo.unified-streaming.com/k8s/features/stable/video/tears-of-steel/tears-of-steel.ism/.m3u8';
function initApp() {
// Install built-in polyfills to patch browser incompatibilities.
shaka.polyfill.installAll();
// Check to see if the browser supports the basic APIs Shaka needs.
if (shaka.Player.isBrowserSupported()) {
// Everything looks good!
initPlayer();
} else {
// This browser does not have the minimum set of APIs we need.
console.error('Browser not supported!');
}
}
async function initPlayer() {
// Create a Player instance.
const video = document.getElementById('video');
const player = new shaka.Player(video);
// Attach player to the window to make it easy to access in the JS console.
window.player = player;
//debugger;
// Listen for error events.
player.addEventListener('error', onErrorEvent);
// Try to load a manifest.
// This is an asynchronous process.
try {
await player.load(manifestUri);
// This runs if the asynchronous load is successful.
console.log('The video has now been loaded!');
shaka.polyfill.installAll();
shaka.polyfill.PatchedMediaKeysApple.install();
const videoContainer = document.getElementById('videoContainer');
const ui = new shaka.ui.Overlay(player, videoContainer, player.getMediaElement());
const config = {
controlPanelElements: [
'play_pause',
'spacer',
'time_and_duration',
'volume',
'mute',
'picture_in_picture',
'fullscreen',
'captions',
],
};
ui.configure(config);
ui.setEnabled(true);
player.addTextTrackAsync('https://raw.githubusercontent.com/1c7/vtt-test-file/master/vtt%20files/2.%20No%20Index%20Number%2C%20download%20using%20youtube-dl.vtt', 'en', 'captions', 'text/vtt').then(function(track) {
}).catch(function error(error){
  console.log(error);

});
} catch (e) {
// onError is executed if the asynchronous load fails.
onError(e);
}
}
function onErrorEvent(event) {
// Extract the shaka.util.Error object from the event.
onError(event.detail);
}
function onError(error) {
// Log the error.
console.error('Error code', error.code, 'object', error);
}
document.addEventListener('DOMContentLoaded', initApp);`


OrientDB Studio does not work in strongbox-distribution

Task Description
OrientDB Studio is not reachable at http://localhost:2480/studio/index.html when Strongbox is started from the strongbox-distribution.tar.gz. However, if you are starting it via cd strongbox-web-core && mvn spring-boot:run it's working as expected. It looks like OrientDB Studio is unable to locate the resources it needs.
We have added a smoke test for this.
Acceptance Test

[ ] Building the code with mvn clean install -Dintegration.tests still works.
[ ] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works and OrientDB Studio starts properly and is accessible.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Task Relationships
This task is a follow-up of: #1087 , strongbox/strongbox-docs#22.
Help
Please join our chat and ping @steve-todorov or @carlspring when you create a PR so one of us can approve the testing stages in our CI.

Our chat
Points of contact:

@carlspring
@sbespalov
@fuss86

@steve-todorov , how did you try to run the distribution ? Using wrapper executable ?

If we try to run the distribution as java -jar strongbox-distribution-1.0-SNAPSHOT/strongbox-1.0-SNAPSHOT/lib/strongbox-web-core-1.0-SNAPSHOT-spring-boot.jar then setenv.conf does not have effect and all values are taken from application.properties from strongbox-commons module.

running windows executable wrapper-windows-x86-64.exe does not work

running strongbox.bat works but the result is the same as in 1.

I've finally found the issue.  For me, at least, the root DB path when I start strongbox from the distribution contains a relative path that uses .. to go back up the directory tree.  You can see the line in OrientDB source code I'm referring to here:
https://github.com/orientechnologies/orientdb/blob/develop/server/src/main/java/com/orientechnologies/orient/server/network/protocol/http/command/get/OServerCommandGetStaticContent.java#L249

After conversation with @carlspring, we will not invest in a major refactor wrapper, and we do not want to include strongbox-vault inside the application install directory because we want that directory to be independent to better support upgrades.  We'll update the wrapper to explicitly list /opt/strongbox-vault as the data directory, and add some notes to the documentation to instruct users that they'll need to change the configuration if they don't install in the customary location.

We should avoid hardcoding things at all cost. We should either consider submitting a patch to OrientDB, or think of a better way to do it.

This is now working from strongbox-distribution, with only the caveat that, when starting Strongbox with OrientDB Studio enabled, the path to strongbox-vault needs to be an absolute path without .. as a path segment.
Strongbox documentation at https://strongbox.github.io/developer-guide/getting-started-with-persistence.html has been updated to provide this instruction for our users.
The following issue has been raised on OrientDB, asking to loosen the upward directory traversal check:  https://github.com/orientechnologies/orientdb/issues/8859

Thanks for following up on this @cbrianhill ! :)

@steve-todorov can you please verify the PR above?

@sbespalov It looks like your PR fixes it. Thanks! :)


Form validation

@leocaseiro can you please review this PR...
I added an angular convention - when an user interacts with an input field it will set the model to $touched .. with our chosen this was not the case!
The user was able to open the select and close it and the model itself stayed $untouched.
This requires Angular 1.3+, so maybe take a look if this makes sense for you - I also added an example for demo purposes.

@lpsBetty thank you for your nice work.
I agree we shouldn't need vendor, however, I believe we should use 100% cdn on examples. Some users download from github, without npm and bower. In that case, they should be able to download and test without any node package manager.
Also if we have as a cdn, we can use htmlpreview as another examples link:
http://htmlpreview.github.io/?https://github.com/leocaseiro/angular-chosen/blob/master/example/index.html
Yeah, npm for tests is perfect, because we need to run npm install to test anyway. We wouldn't need bower as I talked last time! Perfect!
How about we use https://cdnjs.com/libraries/chosen for chosen and google for jquery and angular examples?
I don't mind what cdn to use. Up to you.

In regards the validation, sorry I haven't test your PR yet, but would that work for ng-required without the CSS hack? If yes, that is AWESOME!

ok didn't know why CDNs were used in the example - but now it makes total sense ;)
I am gonna change that of course!
There is an CSS hack? :D

Yeah, actually it does: https://github.com/leocaseiro/angular-chosen/issues/169#issuecomment-194815291


Adding support for GHAS + Minor improvements

Changes in collaboration with @josunefon

Adding support for GHAS (repos)
Using OIDC Azure authentication in GitHub workflow .github/workflows/gh-contributions.yml with federated credentials
Breaking change: Base name and Storage Account name changed in dashboards/GitHubDashboard-Contributors/infrastructure/deploy.bicep to adapt naming with the new features added related to GHAS

Type of Change
Please delete options that are not relevant.

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

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I've tested the solution locally without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas

Hi @Azure/cco-maintainers, I'm just sending a kindly reminder to review the PR please. Thanks in advance! 😊


Easier access to agenda

When the board has meetings it want to easier be able to show the agenda.

Haven't gotten any demand for this, so closing for now.


Change radius of pointer

Is there any way to change the radius of the pointer ??
As my view is very big, the pointer covers most of my screen. I want to define the radius in case my content is large.
Any way to customize it ?

I got the method to change the radius : setHoleRadius(radius)


Minor AKMicrophoneRecorder change

Forgot to add the option 'DefaultToSpeaker'

THanks, this breaks tvOS, but it shouldn't have microphone recorder anyway, I'll take care of that.

By the way, I just noticed you're making these pull requests from the master branch.  Development should really be done on the "develop" branch so that it can be tested before going into master.  In the future, I will be more careful about this.  Thanks!


[Identity] Add Azure Powershell Authentication to Azure.Identity (using Az.Accounts >= 2.2.0)

See this conversation: https://github.com/Azure/azure-sdk-for-net/pull/17081#discussion_r528187979
For more info see PR https://github.com/Azure/azure-sdk-for-net/pull/17081

@schaabs:  Would you please be so kind as to offer your thoughts?

@schaabs:  Would you please be so kind as to offer your thoughts?

This changes are merged now in PR #17081, we will proceed there

This changes are merged now in PR #17081, we will proceed there


Add MacOS support

The meta file format needs to be adjusted for this to support non-linux systems.

what information do you need in order to add support for freebsd, openbsd and darwin/macOS?

No idea. Since I cannot test it, I would expect someone to provide a PR.


Un-@Ignore BasicAcceptanceTests

@Ignore was added at the class level in a0381182e53e80610e50fab25a1c4fa5dd251ee7 in order to avoid build failures on Travis such as this one: https://travis-ci.org/spring-io/sagan/builds/21877994
:sagan-site:integTest

saganx.BasicAcceptanceTests > classMethod FAILED

java.util.concurrent.TimeoutException at BasicAcceptanceTests.java:50

169 tests completed, 1 failed, 1 skipped

:sagan-site:integTest FAILED

FAILURE: Build failed with an exception.

* What went wrong:

Execution failed for task ':sagan-site:integTest'.

At a glance, it looks like this may be related to the use of FreePortFinder in those tests.

We've reworked the codebase since - I'm closing this issue as outdated.


Using "install_method" under "docker_service" does not install docker.lst repo

Cookbook version
cookbook "docker", "~> 2.15.29"
Chef-client version
12.19.36
Platform Details
Vagrant / Ubuntu 16.04
Scenario:
I'm trying to install a specific version of docker: 17.09-ce
Steps to Reproduce:
Use the following recipe:
docker_service 'default' do
   host    ['tcp://<IP_ADDRESS>:2376', 'unix:///var/run/docker.sock']
   group   node['tb_docker']['docker']['group']
   version node['docker']['version']
   install_method 'package'
   action  [:create, :start]
  end

Expected Result:
Get docker 17.09-ce installed
Actual Result:
I get an error saying "no version was found" because the docker.lst repository is not installed.
Further explanation
If I use the following recipe (I removed version and install_method):
docker_service 'default' do
   host    ['tcp://<IP_ADDRESS>:2376', 'unix:///var/run/docker.sock']
   group   node['tb_docker']['docker']['group']
   action  [:create, :start]
  end

The apt repo docker.lst is properly installed in /etc/apt/sources.list.d/docker.lst and the docker version I get is 17.10-ce because docker.lst is set to edge releases.
Shouldn't install_method: package also imply "add the repo"? I can add the repo manually of course, but this behaviour is a bit tricky.
Am I missing something?

you need use this cookbook to install the apt repo
https://github.com/chef-cookbooks/chef-apt-docker
On Wed, Oct 25, 2017 at 9:30 AM, Panagiotis Atmatzidis <
<EMAIL_ADDRESS>wrote:

Cookbook version
cookbook "docker", "~> 2.15.29"
Chef-client version
12.19.36
Platform Details
Vagrant / Ubuntu 16.04
Scenario:
I'm trying to install a specific version of docker: 17.09-ce
Steps to Reproduce:
Use the following recipe:
docker_service 'default' do
host    ['tcp://<IP_ADDRESS>:2376', 'unix:///var/run/docker.sock']
group   node['tb_docker']['docker']['group']
version node['docker']['version']
install_method 'package'
action  [:create, :start]
end
Expected Result:
Get docker 17.09-ce installed
Actual Result:
I get an error saying "no version was found" because the docker.lst
repository is not installed.
If I use the following recipe (I removed version and install_method):
docker_service 'default' do
host    ['tcp://<IP_ADDRESS>:2376', 'unix:///var/run/docker.sock']
group   node['tb_docker']['docker']['group']
action  [:create, :start]
end
The apt repo docker.lst is properly installed in /etc/apt/sources.list.d/
docker.lst and the docker version I get is 17.10-ce.
Shouldn't install_method: package also imply "add the repo"? I can add
the repo manually of course, but this behaviour is a bit tricky.
Am I missing something?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chef-cookbooks/docker/issues/933, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABKpSECfWaxVRIJzs5lzzCKFAgAUYLxZks5sv2I7gaJpZM4QGS5o
.

Thanks


User Impersonation

I would be a nice feature to utilize user impersonation ( X-Redmine-Switch-User ) for postIssue and updateIssue.

If we add a possible params property like: redmine.request(method, path, {impersonate: 'usename'} )
would this be ok for you.
You will so be able to add an impersonate param to all methods using request under the hood.

added in release 0.2.13 by adding an impersonate value to the params argument:
var params = {impersonate: "userlogin"};

I can't get this to work. For example:  updateIssue returns this.put('issues/' + id, {issue:params}).
So in put and request "params.issue.impersonate" would have to be used to get the usernlogin.  Maybe i am not setting something correct with params.
I also have to set params with retry:1 for every function or i get a TypeError: Cannot read property 'retry' of undefined ... redmine.js:98:20 error since the 0.2.12 release.  I tried calling setMaxTry and setting maxTry in config without luck.
Sorry I don't have more time to fix the code as I am trying to get a release out the door.

Hi,
First thank you for reporting. I have no redmine install available for testing and so i'm blind on developing this lib. I'm looking in rewrite tests with nock when i got a little extra time to give it a try.
Nevertheless, can you please try to use the fix-params branch i just pushed, i think it should fix the two mentioned issues. If so i'll publish a new release right away.
Be aware the signature for post / update methods have changed a little (params is now a separate parameter than the item data sent to the server).
regards,
malko

Finally i reintroduced some unit test using nock.
I spotted some problems arround retry, impersonate and verbose features which all should be gone now.
please update to 0.3.0 version hoping this will be ok for you.
Sorry about the previous buggy version, i'm confident that unit test will avoid such buggy version in the future.
thanks again for reporting!!!

0.3.0 fixed the issues i was having with retry and impersonate.  Thanks for the update.


Sails sockets.(pubClient|subClient) gets corrupted in config

Sails version: 0.12.14
Node version: 8.6.0
NPM version: 5+
DB adapter name: N/A
DB adapter version: N/A
Operating system:

I am trying to provide sails with a socket.io adapter using the sockets config options pubClient and subClient (using an ioredis client which should work with socket.io-redis). However, when I try to use it, I get an error

sub.subscribe is not a function

in other words, socket.io-redis is throwing because the client does not have a subscribe method. However, the object that I pass in DOES have a subscribe method. It is simply the newly constructed ioredis client (which has a subscribe method). The issue is that the subscribe method is on the object's protoype. It seems that when sails ingests the configuration, it destroys prototype information (which I guess makes sense since config should be static info). However, how is one meant to use the pubClient|subClient options while only passing plain javascript objects?

Hi @istrau2,
It sounds like you're possibly having the same issues as this: https://github.com/balderdashy/sails/issues/4207.
This (and another related issue) was patched and released as sails-hook-sockets v0.13.13.  You probably just need to update to that version (you can do npm ls sails-hook-sockets to check your current version).  You can upgrade by completely re-installing Sails, or you can just npm install --save sails-hook-sockets directly into your project.

@sgress454 Ah ok, thanks.


Added Speed Property to mediaelement

Description of Change
Added Speed property for Mediaelement in iOS and Android
Bugs Fixed

Fixes #649

API Changes

PR Checklist

[x] Has a linked Issue, and the Issue has been approved
[ ] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[ ] Changes adhere to coding standard

[ ] Updated documentation

Seems to work great on Android and iOS at least, thanks for this!
Would you be able to add this to the sample app too? Ideally some way to change the new Speed property in the MediaElement page? A slider maybe? Thanks!

@jfversluis I have added the sample in the media element page :)

One last request: could you rebase it please? Or at least update it with the latest changes? I would do it for you, but I don't have the rights to your branch :)

One last request: could you rebase it please? Or at least update it with the latest changes? I would do it for you, but I don't have the rights to your branch :)
Did it :)

OK, so sorry. One last rebase and then it should merge automatically... 😇

OK, so sorry. One last rebase and then it should merge automatically... 😇

OMG... WHY DIDN'T IT DO IT?!

OK, so sorry. One last rebase and then it should merge automatically... 😇

OMG... WHY DIDN'T IT DO IT?!

Is there something that I should do ?

@AswinPG looks like this branch is out-of-date, can you rebase your branch against develop and push?

@AswinPG looks like this branch is out-of-date, can you rebase your branch against develop and push?

It's my first time rebasing. I don't really know if I did it right or wrong...

@AswinPG yeah, something went wrong... We can see based on the number of changed files

I see... Looks like that you made your changes against our main branch instead of the develop branch.

@AswinPG looks like everything is ok now, let's how it goes. I'll let @jfversluis take a final look at this since this rebase dismissed his review.

@AswinPG looks like everything is ok now, let's how it goes. I'll let @jfversluis take a final look at this since this rebase dismissed his review.

Thanks @pictos

@AswinPG looks like everything is ok now, let's how it goes. I'll let @jfversluis take a final look at this since this rebase dismissed his review.

Thanks @pictos

Implemented through #1570 to work around asking for another rebase :)
Thanks so much @AswinPG !

Thanks @jfversluis :)


Fixed KL bug and changed KL calling convention

Fixed issue #33 (missing divide by 2 in one term) and changed calling convention for all KL divergences so that:

The arguments are always a pair of tables containing named parameters (this only affectes dir.kl, which previously took two vectors as arguments)
The first argument is named q and the second is named p, to be consistent with the conventional notation.

Thanks for your vigilance David!

My pleasure. A happy side effect of tracking down all these maddening bugs in my code...


crypto viewer

Same as https://github.com/spignelon/virtual-assistant/pull/17#issuecomment-1277891756
Issue: 1#15


bumped required node engine to v18.18 and required vscode engine to v1.84 and vitest to '~1.0.0'

github workflow is already executed with node v18

Is there anything else (documentation?) to be updated?

Is there anything else (documentation?) to be updated?

See #1277

It's quite a different PR now: Here're the updates as proposed in today's dev meeting.

@sailingKieler sorry I wasn't able to participate in today's meeting. This seems to be a good compromise.

@spoenemann I missed that, but have a solution for this. Either the base package.json or the vite config file must have the ".mts" extension.

@spoenemann I missed that, but have a solution for this. Either the base package.json must be of type module or the vite config file must have the ".mts" extension. We did the later one in sprotty.

done!
@msujew can you have a look again?


add kafka async metrics

Signed-off-by: Vigith Maurice<EMAIL_ADDRESS>

 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.

I can see metrics flowing via /metrics endpoint
# HELP input_kafka_produced_success Number of messages successfully produced to Kafka
# TYPE input_kafka_produced_success counter
input_kafka_produced_success{topic="output-topic",type="async"} 246


TestUtilities: silence warning from updated swift-tools-support-core

The AbsolutePath.appending(_:RelativePath) interface was deprecated in
favour of AbsolutePath(_:relativeTo:) which allows us to anchor the
relative path which is needed to properly form relative paths on
Windows.  This updates the API usage in swift-driver accordingly.

@swift-ci please test


runtime: unrecognized instruction "PCMPESTRI"

What version of Go are you using (go version)?

tip = b859a78e0a71d769274dac8cf0108bdf41ec55a5

What operating system and processor architecture are you using (go env)?

Intel i5-2320
Audrius@Audrius syncthing $ go env
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\GoHome\
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GO15VENDOREXPERIMENT=1
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1

What did you do?

Tried to compile a hello world example

What did you expect to see?

Compiled binary

What did you see instead?

# runtime
C:\Go\src\runtime\asm_amd64.s:1853: unrecognized instruction "PCMPESTRI"
C:\Go\src\runtime\asm_amd64.s:1862: unrecognized instruction "PCMPESTRI"
asm: asm: assembly of C:\Go\src\runtime\asm_amd64.s failed

Seems to have been broken by 6b02a1924725688b4d264065454ac5287fbed535

Could it be that you are still using an old version of Go? What does go version print?
go env from tip doesn't have set GO15VENDOREXPERIMENT=1 in its output.
How did you build go compiler?

Yep, I am a muppet.


Update tool versions

This commit updates the following tools to the specified versions:

cosign updated to v1.8.0
yq updated to v4.25.1
opa updated to v0.40.0

Signed-off-by: Rob Nester<EMAIL_ADDRESS>

Nice.


[e2e] make test bindings friendlier to integration tests

Description
Integration tests may not want to override testTextInput or HttpClient. This lets binding subclasses for tests achieve that.
Related Issues
Part of https://github.com/flutter/flutter/issues/51885
Part of https://github.com/flutter/flutter/issues/57623
Tests
I added the following tests:
Test that the default bindings do override HttpClient and register TestTextInput.
Test that custom bindings using the new properties do not override/register.
Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

Thanks for the change, I was still looking at the PR. LGTM!


RateLimit configuration

Hi there,
I know that dspy uses LiteLLM and that langchain also has a memory rate limiter that you can pass into the chat model.
However I can't figure what should I do for example to retrict to N requests/second or T tokens/seconds etc.
Any example would be great.
Cheers.

Hi @robomotic ,
DSPy doesn't support LiteLLM Router integrations yet, but the mentioned documentation's instructions and an existing PR might be helpful in understanding some examples on how this would work.
Leaving this open!

Hi @robomotic,
You can start Litellm proxy server using this, which will be running on port 4000. Use UI to add models in the proxy server.
Then in your DSPy program you can simply do:
lm = dspy.LM(
    model="openai/LITELLM PROXY PUBLIC MODEL NAME",
    api_base="http://localhost:4000/",
    api_key="sk-YOUR LITELLM MASTER KEY",
    cache=False,
)

That's a good idea thanks!


Refactor moldev_softmax_pro adapter to use plate-reader/REC/2023/09 schema

The Softmax Pro adapter (moldev_softmax_pro) adapter should be refactored to use the ASM plate reader schema included in the 2023/09 release.
https://gitlab.com/allotrope-public/asm/-/blob/main/json-schemas/adm/plate-reader/REC/2023/09/plate-reader.schema.json?ref_type=heads

Addressed in #217


Lorax Hanging in production

System Info
ghcr.io/predibase/lorax:latest
Running within Kubernetes on H100
Information

[X] Docker
[ ] The CLI directly

Tasks

[ ] An officially supported command
[ ] My own modifications

Reproduction
When putting the instance in production, while it receive simultaneous request for different adapters, it will just hang there.
/generate and /health will stop answering
but /info and /docs will continue to be available.
There's no error getting displayed in the logs

Not sure what's the best way to diagnose what the issue could be, but looks to me like it's having some issues fetching multiple adapters in parallel and processing request queued at the same time?
Expected behavior
Should handle live requests for multiple adapters

Hey @karlbernard2, thanks for reporting. It sounds like there's a deadlock that's occurring here that may be triggered under very specific conditions (requests coming it at just the wrong time). Can you share any additional details about your setup (args to lorax-launcher for example) that can help with reproducing the error?
One thing that stands out from the logs you provided is that the adapter NextDayAI/xtraspicy1.0_13b_r32_800 was loaded, successfully processed a request, then offloaded, then loaded back, but never successfully processed any other requests. It's curious that it was offloaded at all, as it looks like only two adapters were loaded, while by default we will allow up to 128 to be loaded before doing any offloading. So whatever is causing the deadlock may be related to that behavior.
I'll try and take a closer look, but if there's anything you can provide to help me repro that would be helpful.

The fact that the /health endpoint is unresponsive but the /info endpoint works would suggest that there's an issue with the Python server, rather than the router. It's possible that the Python server is stuck on some operation.
Something you could try:

Make sure your container is running in privileged mode by adding SYS_PTRACE to the security context of the container as shown here.
SSH into the pod with kubectl exec -it <pod_name> -- /bin/bash
Install py-spy so you can get a backtrace from the Python server: pip install py-spy
Find the Python server process: ps aux | grep python
Run py-spy on the Python server to obtain the backtrace: sudo py-spy dump -p <pid>

If you're able to run that on one of the hung pods, that would be very helpful for debugging the error.

Thanks for the detailed instructions,, I'll try to do that.
Here's how I launched teh container
containers:
- name: lorax-container
image: ghcr.io/predibase/lorax:latest
ports:
- containerPort: 8001
env:
- name: HUGGING_FACE_HUB_TOKEN
value: hf_secret
- name: PORT
value: "8001"
- name: ROPE_SCALING
value: "dynamic"
- name: ROPE_FACTOR
value: "2.0"
args:
- "--max-input-length=7900"
- "--max-total-tokens=8192"
- "--max-batch-prefill-tokens=8192"
- "--model-id=NextDayAI/extraspicy"

@tgaddair My first attempt to replicate didn;t have the same issue (althouh earlier today I got it all the time, so will try more.)
However, since you talked about offloading that shouln't happen, you might find these logs strange:
2023-12-23T03:04:06.827779Z  INFO HTTP request{otel.name=POST /generate http.client_ip= http.flavor=1.1 http.host=inference.spicychat.ai:7001 http.method=POST http.route=/generate http.scheme=HTTP http.target=/generate http.user_agent=axios/1.4.0 otel.kind=server trace_id=1b65c393d478941d6b16797446fc1519}:generate{parameters=GenerateParameters { adapter_id: Some("NextDayAI/xtraspicy1.0_13b_r32_720_adapter"), adapter_source: None, api_token: None, best_of: None, temperature: Some(0.7), repetition_penalty: Some(1.1), top_k: Some(90), top_p: Some(0.7), typical_p: None, do_sample: true, max_new_tokens: 180, return_full_text: None, truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } total_time="3.414291187s" validation_time="3.11ms" queue_time="46.871µs" inference_time="3.41113458s" time_per_token="18.950747ms" seed="Some(18257989878521111275)"}: lorax_router::server: router/src/server.rs:298: Success 2023-12-23T03:04:07.058713Z  INFO lorax_router::loader: router/src/loader.rs:241: adapter __base_model__ offloaded 2023-12-23T03:04:07.058731Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter __base_model__ status to Downloaded 2023-12-23T03:04:07.095727Z  INFO lorax_router::loader: router/src/loader.rs:197: adapter NextDayAI/xtraspicy1.0_13b_r32_760_adapter loaded 2023-12-23T03:04:07.095745Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter NextDayAI/xtraspicy1.0_13b_r32_760_adapter status to Ready 2023-12-23T03:04:07.588268Z  INFO HTTP request{otel.name=POST /generate http.client_ip= http.flavor=1.1 http.host=inference.spicychat.ai:7001 http.method=POST http.route=/generate http.scheme=HTTP http.target=/generate http.user_agent=axios/1.4.0 otel.kind=server trace_id=f94f4f52e417b624f40329c484ee954f}:generate{parameters=GenerateParameters { adapter_id: Some("NextDayAI/xtraspicy1.0_13b_r32_760_adapter"), adapter_source: None, api_token: None, best_of: None, temperature: Some(0.7), repetition_penalty: Some(1.1), top_k: Some(90), top_p: Some(0.7), typical_p: None, do_sample: true, max_new_tokens: 180, return_full_text: None, truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } total_time="541.9253ms" validation_time="2.590788ms" queue_time="64.482779ms" inference_time="474.851989ms" time_per_token="23.742599ms" seed="Some(14941048975292732004)"}: lorax_router::server: router/src/server.rs:298: Success 2023-12-23T03:04:07.625223Z  INFO HTTP request{otel.name=POST /generate http.client_ip= http.flavor=1.1 http.host=inference.spicychat.ai:7001 http.method=POST http.route=/generate http.scheme=HTTP http.target=/generate http.user_agent=axios/1.4.0 otel.kind=server trace_id=676504f7982ff96d058121f59d961dd4}:generate{parameters=GenerateParameters { adapter_id: Some("NextDayAI/xtraspicy1.0_13b_r32_800_adapter"), adapter_source: None, api_token: None, best_of: None, temperature: Some(0.7), repetition_penalty: Some(1.1), top_k: Some(90), top_p: Some(0.7), typical_p: None, do_sample: true, max_new_tokens: 300, return_full_text: None, stop: ["\nEva:", "\nShizuka:", "\n###"], truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } total_time="1.211921872s" validation_time="778.706µs" queue_time="13.944244ms" inference_time="1.197199151s" time_per_token="21.003493ms" seed="Some(5853084379632762489)"}: lorax_router::server: router/src/server.rs:298: Success 2023-12-23T03:04:09.218011Z  INFO lorax_router::loader: router/src/loader.rs:241: adapter NextDayAI/xtraspicy1.0_13b_r32_800_adapter offloaded 2023-12-23T03:04:09.218033Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter NextDayAI/xtraspicy1.0_13b_r32_800_adapter status to Downloaded 2023-12-23T03:04:09.218716Z  INFO lorax_router::loader: router/src/loader.rs:241: adapter NextDayAI/xtraspicy1.0_13b_r32_720_adapter offloaded 2023-12-23T03:04:09.218739Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter NextDayAI/xtraspicy1.0_13b_r32_720_adapter status to Downloaded 2023-12-23T03:04:09.239236Z  INFO lorax_router::loader: router/src/loader.rs:197: adapter NextDayAI/xtraspicy1.0_13b_r32_400_adapter loaded 2023-12-23T03:04:09.239269Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter NextDayAI/xtraspicy1.0_13b_r32_400_adapter status to Ready 2023-12-23T03:04:09.258709Z  INFO lorax_router::loader: router/src/loader.rs:197: adapter NextDayAI/xtraspicy1.0_13b_r32_720_adapter loaded 2023-12-23T03:04:09.258724Z  INFO lorax_router::queue: router/src/queue.rs:135: set adapter NextDayAI/xtraspicy1.0_13b_r32_720_adapter status to Ready 2023-12-23T03:04:09.701608Z  INFO HTTP request{otel.name=POST /generate http.client_ip= http.flavor=1.1 http.host=inference.spicychat.ai:7001 http.method=POST http.route=/generate http.scheme=HTTP http.target=/generate http.user_agent=axios/1.4.0 otel.kind=server trace_id=a9611795aa4b0336970442689421c739}:generate{parameters=GenerateParameters { adapter_id: Some("NextDayAI/xtraspicy1.0_13b_r32_400_adapter"), adapter_source: None, api_token: None, best_of: None, temperature: Some(0.7), repetition_penalty: Some(1.1), top_k: Some(90), top_p: Some(0.7), typical_p: None, do_sample: true, max_new_tokens: 180, return_full_text: None, stop: ["\nYour roommate Amber :", "\nWinston:", "\n###"], truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } total_time="489.291069ms" validation_time="4.884384ms" queue_time="22.199583ms" inference_time="462.207337ms" time_per_token="24.326701ms" seed="Some(16751434335624526013)"}: lorax_router::server: router/src/server.rs:298: Success

Here's the trace:

Thanks for the back trace @karlbernard2, this is very helpful!
Definitely looks like the hanging is occurring the SGMV kernel.
In the short term, you can try disabling SGMV with an environment variable: DISABLE_SGMV=1. That's not a great longterm solution since SGMV is very fast when you have lots of adapters, but it should at least unblock you while I try and repro the issue, and the performance hit shouldn't be very noticeable with fewer than 10 adapters.
I'll see if I can repro this behavior with the adapters you're using here.

Hey @karlbernard2, update on this: I tried running some stress tests today with a variety of request patterns to try and replicate your setup, but was unable to trigger the hanging behavior.
Can you share a few more details about your environment:

What GPU are you running on?
What Nvidia device driver version are you using (from nvidia-smi)?
Is this running on prem or in the cloud? If cloud, which one?

Thanks.

We’re running H100 on NebiusAI Kubernetes. I’ll have to get back to you on Tuesday with info on drivers.

Hey @karlbernard2, I managed to track down the root cause of the deadlock, and has been fixed in #156.


Fix og:image not being downloaded over https

Facebook duh
http://stackoverflow.com/questions/8855361/fb-opengraph-ogimage-not-pulling-images-possibly-https

LGTM, thanks for the fix.
Merging.


Support runners setup via parameters on cirunner.pp

Allow setting up of cirunners not onlly via hiera but also via parameters ( common_config and runners_config)

I just raised a pull request against this functionality my self #147 however my changes are a bit more extensive.  that said as you are using the functionality i would be interested in your comments to my pull

After reviewing #183, I think that functionality supercedes the functionality here... however, after review, I think it makes more sense to extract this functionality into it's own module. The primary scope of this module is to configure and install gitlab-omnibus.


Post sound doesn't complete when it bounces on ground instantly

Mostly happens when you hit the bottom of the crossbar.

Fixed


Add Apcupsd 3.14.13

This relies on a decision being made about https://github.com/caskroom/homebrew-cask/issues/9203 - the "shim script" in this case is an installer_choices.xml that allows for unattended install.
This actually doesn't seem like a very uncommon scenario, so the choice might be to either a) implement this as core functionality (extend cask DSL) or b) allow scripts like this (because of the relative stagnation of the main codebase at the moment)
cc @caskroom/maintainers

@reitermarkus What is the preflight intending to accomplish, exactly?

The preflight unchecks the daemon in the PKG installer, otherwise the installer would fail under El Capitan.

Yeah, this is one of those cases. This is a bad way to fix the issue. The correct way is to contact the developers / fix it upstream. Crippling the cask / building workarounds in homebrew-cask when the issue is an upstream one is never the solution.
Please remove the preflight and instead have a depends_on that’ll prevent it from installing in El Capitan. That is the correct course of action. Unless you know this will never be fixed upstream (perhaps there’s a technical reason that can’t be fixed), in which case it’s not worth adding at all.

Ok, then I better remove it completely and close this because I have added it to https://github.com/Homebrew/homebrew/pull/45899 anyway.

Even better! In the spirit of https://github.com/caskroom/homebrew-cask/issues/14384, not having duplicates between the projects starts to be desirable.


Showcase Page Refinements

Picking up from #501 and #638, here is a list of possible refinements / enhancements for the new Passenger Showcase!
https://thecodingtrain.com/showcase
Individual Passenger Pages
For example: https://thecodingtrain.com/showcase/author/kathy-mcguiness

It would be nice to include additional information as headers (name and link to URL) for each passenger, making it feel a bit more like a "profile page"?
The "preview" image when sharing a page should grab a passenger showcase thumb rather than the default share image.

Showcase Cards

Author links can go to their filtered page (rather than what was entered in the form)?
Handle video titles that are cut-off

new design over 2 lines?
truncate with "..."?
other ideas?

We can use this as a discussion thread and break things out into individual pull requests / issues as needed. Any other thoughts or comments?

RE Name and link to URL. While I like the idea and there might be a one unique URL for an author, that will not always be the case.
Raising a different issue.  There are a couple of authors who have contributed under more than one variation of their name and /or  contributed a showcase jointly with another person.   How do we want to handle this?

These are very good points!

RE Name and link to URL. While I like the idea and there might be a one unique URL for an author, that will not always be the case.

Yes, since the site is a static build and not database driven what we can do here is quite limited. I think we do our best and just pick the URL from the latest submission for any given author? We can see how that goes and if it doesn't work well, we can try something different.

Raising a different issue. There are a couple of authors who have contributed under more than one variation of their name and /or contributed a showcase jointly with another person. How do we want to handle this?

Different author names will unfortunately show up as different filters/pages with the way the site works. Contributors can go back and edit their names in the JSON files if they want to consolidate the pages.  For joint submissions these will end up on their own page since the name is the unique identifier. I'm ok with this for a first try, we can always see if we want to improve the system later!


Called Result::unwrap() on an Err value: VirtualMachine(...

I am getting the following error when I try to deploy latest AssetHub Rococo runtime built from polkadot-sdk master.
panicked at src/task.rs:131:6:
called `Result::unwrap()` on an `Err` value: VirtualMachine(UnresolvedFunctionImport { function: "ext_storage_proof_size_storage_proof_size_version_1", module_name: "env" })

Stack:

Error
    at imports.wbg.__wbg_new_abda76e883ba8a5f (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/executor/dist/esm/chopsticks_executor.js:801:19)
    at wasm://wasm/00b1e282:wasm-function[1108]:0x191195
    at wasm://wasm/00b1e282:wasm-function[1334]:0x1997cf
    at wasm://wasm/00b1e282:wasm-function[1221]:0x1967ac
    at wasm://wasm/00b1e282:wasm-function[115]:0xb9075
    at wasm://wasm/00b1e282:wasm-function[632]:0x16b359
    at wasm://wasm/00b1e282:wasm-function[1476]:0x19aeb1
    at __wbg_adapter_48 (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/executor/dist/esm/chopsticks_executor.js:384:10)
    at real (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/executor/dist/esm/chopsticks_executor.js:366:22)
    at node:internal/process/task_queues:140:7

node:internal/event_target:1096
  process.nextTick(() => { throw err; });
                           ^
Error [RuntimeError]: unreachable
    at wasm://wasm/00b1e282:wasm-function[1108]:0x1912ab
    at wasm://wasm/00b1e282:wasm-function[1334]:0x1997cf
    at wasm://wasm/00b1e282:wasm-function[1221]:0x1967ac
    at wasm://wasm/00b1e282:wasm-function[115]:0xb9075
    at wasm://wasm/00b1e282:wasm-function[632]:0x16b359
    at wasm://wasm/00b1e282:wasm-function[1476]:0x19aeb1
    at __wbg_adapter_48 (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/executor/dist/esm/chopsticks_executor.js:384:10)
    at real (file:///Users/nacho/Desktop/PARITY/Repos/chopsticks/executor/dist/esm/chopsticks_executor.js:366:22)
    at node:internal/process/task_queues:140:7
    at AsyncResource.runInAsyncScope (node:async_hooks:206:9)

Chain-spec:
ah-rococo-new.json

try with allow-unresolved-imports. seems like a new host function is added unrecognised by smoldot

You mean to add
allow-unresolved-imports: true

in the config.yaml, right?
Unfortunately that didn't work

Your runtime includes storage_proof_size::HostFunctions. Is this part of standard build?

@tomaka do we need to implement this on smoldot?

In principle, no, smoldot doesn't really need a change, and allow_unresolved_functions: true should work.
There was a lengthy discussion about this function, and in principle it is used only when parachain blocks are authored or verified by the relay chain.
Maybe it would make sense to implement it in smoldot and return an error if the function is called.

return an error will not help. if allow_unresolved_functions: true didn't help then it's been called

westend-asset-hub is updated and now won't produce blocks because of this

#1742


Potential bug in PPOTrainer code.

Hi, I am comparing your implementation to that of OpenAI's implementation (https://github.com/openai/lm-human-preferences/blob/cbfd210bb8b08f6bc5c26878c10984b90f516c66/lm_human_preferences/train_policy.py#L285)
The difference is that PPO is trained only on the response tokens in OpenAI's implementation vs in your case, it is trained on query+response. Is that a bug or just another variant?
Thanks
Sachin

@younesbelkada As far as logprobs and ref_logprobs are concerned that are used by the PPOTrainer, it only gets it for response and not for query+response.
https://github.com/openai/lm-human-preferences/blob/cbfd210bb8b08f6bc5c26878c10984b90f516c66/lm_human_preferences/policy.py#L114
In this code, ref_logprobs is obtained as below, here you can see they have : logits = result['logits'][:, context_length-1:-1] where context_length is length of query.
def analyze_responses_op(self, queries, responses):
    contexts = self.embed_queries(queries)
    context_length = tf.shape(contexts)[1]
    tokens = tf.concat([contexts, responses], axis=1)
    result = self.step_core(self.model_hparams, tokens)
    logits = result['logits'][:, context_length-1:-1]

    logits /= self.temperature
    return dict(
        logprobs = utils.logprobs_from_logits(logits=logits, labels=responses),
        entropies = utils.entropy_from_logits(logits),
        values = result['values'][:, context_length-1:-1],
    )


Updated README.adoc to fix the sample code given in documentation.

The actors Hashset object created in documentation.
Relationship can directly access INCOMING and OUTGOING constants.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hi @rahulraopune Thank you for catching this oversight.
Would you please sign this https://neo4j.com/developer/cla/ and follow the instructions there and ping me here if you did it? Otherwise I cannot merge this. Thank you.


Logical port is down after creation

I have the following vagrant setup.
1 node cluster with kubernetes master and openvswitch running on the same node. CLUSTER_IP_SUBNET=<IP_ADDRESS>/16, MASTER_SWITCH_SUBNET=<IP_ADDRESS>/24.
Everything have started and interfaces exist:
br-int: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet6 fe80::48d9:3bff:fedc:aa4c  prefixlen 64  scopeid 0x20<link>
        ether 08:00:27:06:b9:de  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 62  bytes 13666 (13.3 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

docker0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::42:b7ff:fe5d:b203  prefixlen 64  scopeid 0x20<link>
        ether 02:42:b7:5d:b2:03  txqueuelen 0  (Ethernet)
        RX packets 3002  bytes 269534 (263.2 KiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 3012  bytes 939730 (917.7 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

eth0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::a00:27ff:fecb:cb16  prefixlen 64  scopeid 0x20<link>
        ether 08:00:27:cb:cb:16  txqueuelen 1000  (Ethernet)
        RX packets 433934  bytes 613898403 (585.4 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 114035  bytes 6555659 (6.2 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

eth1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::a00:27ff:fe06:b9de  prefixlen 64  scopeid 0x20<link>
        ether 08:00:27:06:b9:de  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 26  bytes 3211 (3.1 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

eth2: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::a00:27ff:fe59:e994  prefixlen 64  scopeid 0x20<link>
        ether 08:00:27:59:e9:94  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 27  bytes 3301 (3.2 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

k8s-<IP_ADDRESS>: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        ether c6:c1:8d:41:03:59  txqueuelen 1000  (Ethernet)
        RX packets 2  bytes 112 (112.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 35  bytes 10370 (10.1 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 1  (Local Loopback)
        RX packets 69855  bytes 26598786 (25.3 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 69855  bytes 26598786 (25.3 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

ovs-system: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet6 fe80::8013:f4ff:fe51:3a0f  prefixlen 64  scopeid 0x20<link>
        ether 82:13:f4:51:3a:0f  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 61  bytes 13334 (13.0 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

However, when I create new pod I see that logical port is down:
ovn-nbctl list logical_switch_port default_redis-master-2463511816-vaq6k
_uuid               : e078e54a-0534-4d6c-81da-a993796318a4
addresses           : [dynamic]
dhcpv4_options      : []
dhcpv6_options      : []
dynamic_addresses   : "0a:00:00:00:00:07 <IP_ADDRESS>"
enabled             : []
external_ids        : {}
name                : "default_redis-master-2463511816-vaq6k"
options             : {}
parent_name         : []
port_security       : []
tag                 : []
type                : ""
up                  : false

Moreover, pod is getting network configuration from docker0 bridge.
Have I missed something in documentation?
Thank you in advance for any help you can provide.

@RostyslavFridman
The current documentation and code expects master to not run any containers.  There are multiple reasons for it - one of them being containers mucking around with the north-bound database. So you likely ran just the master-init? That script does not create a CNI plugin and hence the containers started will get IP addresses from docker bridge.
Btwn, thanks for trying it out. We still are a little poor with documentation (and a couple of bugs). I just noticed that there is no mention of how to start kubelet with the OVN CNI plugin.
It should be something like:
./kubelet --api-servers=http://<IP_ADDRESS>:8080 --v=2 --address=<IP_ADDRESS> --enable-server=true --network-plugin=cni

On that note, it should not be a big problem running pods on master as well.
I have indeed a 2 node setup where the kubelet is running also on the master node (mostly to save the hassle of using a 3rd vm to achieve a 2-node testbed).
To do so I think I ran both master-init and minion-init on the master, specifying the same subnet. If I recall correctly minion-init skipped the steps for creating the node ls and connecting it to the cluster lr as they were already performed by master-init, and simply configured the CNI plugin.

It finally works.
Moreover, it is OK to run master-init and minion-init on the same node in the same subnet.
I've struggled mostly with running this solution on CoreOS when everything is containerized.
If anyone would like to repeat this, dockerized kubelet should contain cni binaries, openvswitch binaries and ovn-kubernetes scripts.
Thank you for the help.

@RostyslavFridman
It looks like you are familiar with both k8s and OVN to have got this working with coreos. One challenge of having CNI plugin running in master would be that the core daemons running inside containers will also get OVN private IP addresses?

Which core daemons are you referring to?
I am not using coreos but on ubuntu I have no problem running the kubelet as well on the master node. I agree you probably should not be doing that in production, but in dev envs it gives you an extra node!
From what I recall the kubelet does not invoke network plugins for pods with special security context attributes such as hostnetwork.

@shettyg
Actually running core daemons are not a big issue. They are running with hostNetwork: true, so they are getting host IP address and not OVN private IP addresses.
On the other hand, kube-dns and kube-dashboard are getting OVN private IP addresses.
 Run ovs commands inside container:
docker exec kubelet /usr/share/openvswitch/scripts/ovs-ctl start

...

docker exec kubelet ovn-k8s-watcher --overlay

One caveat is that /etc/cni/net.d should contain some configuration file. It won't be used but it should be present.
Another one is that it is not a CoreOS suggested way of running kubernetes. They suggest to use kubelet-wrapper, but it runs kubelet in rkt and you won't be able to manage OVS in it.


[code]is the refcount's  set value correct?

https://github.com/apache/rocketmq/blob/a62b70bc25423c1d7e18043e32af427d29ef9ac4/store/src/main/java/org/apache/rocketmq/store/ReferenceResource.java#L50-L51
should  this.refCount.set(-1000 - this.getRefCount()); be
 this.refCount.set(-1000 + this.getRefCount());

IMO, the set operation mainly sets the refCount to negative so that cleanupOver can be forced after a timeout. So if you use +, the refCount might still be positive. (The way I understand it, it's probably better to just make refCount negative)

I aggre with you, but the code is really confusing and may be misunderstood as "-1000" has special meaning.

IMO, the set operation mainly sets the refCount to negative so that cleanupOver can be forced after a timeout. So if you use +, the refCount might still be positive. (The way I understand it, it's probably better to just make refCount negative)


Nuget Dependencies

Thanks for this package @nul800sebastiaan, I've replaced my implementation with yours on a new u9 site we are building. My implementation didn't have the swanky BackOffice integration 😄.
Now that 9.0.0 is released, can we updated the dependencies to that? 9.0.0-preview001 is tossing build warnings that it wants me to downgrade to rc001.

Good to hear @cleversolutions! v1.0.0 is out now, depending on v9.0.0 final 👍


Add the ability to create custom vertex formats

This commit makes it possible to create vertex formats
differing from the default position/normal/uv vertex.
It doesn't change much functionality and uses existing
APIs.
NOTE: This might not be the most optimal approach.
It achieves what it achieves by compiling and specializing
pipelines BEFORE the mesh resource provider system.
So please make suggestions how I can make this better.

PS: I did this because I'm working on a voxel renderer for bevy.

Just a heads up there is an existing draft PR open for this as well:
https://github.com/bevyengine/bevy/pull/158

Thanks for the tip! Maybe I can contribute to their solution. I had the same issues as they had, but I managed to get a working solution.

PS: I did this because I'm working on a voxel renderer for bevy.

Hey @walterpie let me know if you want to team up on this. I'm planning to do the same thing. I've made some progress on a voxel renderer, etc for the Amethyst engine, and I want to port it to Bevy: https://github.com/amethyst/voxel-mapper

Hey @bonsairobo, I just added you. Let's discuss the details over on Discord.

Closing in favor of the direction we're taking in #599 (and future follow up prs)


⚠ Contemporanea has degraded performance

In 14eae31, Contemporanea ($SITE_CNTM) experienced degraded performance:

HTTP code: 200
Response time: 13149 ms

Resolved: Contemporanea performance has improved in 7e97af8.