Show created date and duration in expired infraction log

Closes #685

Creation date is displayed in the log.
The duration between the expiration and the creation is displayed as a humanised delta. If the infraction had no expiration, then the duration is relative to the current time rather than an expiration.
The actual actor is displayed rather than always being the bot user.
The member is displayed as a mention rather than an ID.

Before:

After:

Hmm, in addition, should we also show the avatar of use if .get_user() returns a not None? So it will line up with the other embeds


Update README

Coverage remained the same at 88.189% when pulling 6bc0ad0ca238226a7db867e5870ed649427b44bd on dev into 459d5f15f9e2c2eaffd34bcd4cd7794f72dab1a6 on master.


lxd/cluster: Rework heartbeat concurency

With this, only one heartbeat can be running at any one time and
additional requests are ignored. It also replaces the previous logic of
doing a full heartbeat round on dqlite connections to instead doing one
automatically if no heartbeat round succeeded over the past two
intervals.
This should ensure that a newly elected leader will do a hearbeat round
almost immediately and that this heartbeat round will include all data
whereas existing leaders will keep doing normal heartbeats even if
dqlite connections get reset.
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Well, this is deadlocking somehow...

I think the reason of the deadlock is because we lost the ability to force a full heartbeat round when we're already the leader and have just dealt with membership changes. To fix that I've now added logic to reset the lastHeartbeat variable to 0 when a dqlite event happens, this will then force the next heartbeat (whether scheduled or triggered) to do a full round.
Hopefully that was the reason of the issue and this will unblock Jenkins.

And still hitting a deadlock...

And still deadlocking... So I think I'll revert all of this as it's clearly causing issues and just keep the couple of tiny bugfixes in there and add more logging for dqlite connections but otherwise revert to the previous behavior.
This is because some dqlite changes need a cluster reconfiguration and can't wait for the next scheduled heartbeat, so trying to force heartbeats to only be sent on schedule is actually breaking clustering.
With more logging, we'll see just how often dqlite connections are being re-established and if this seems to be happening in some kind of loop for some users, then the source of that behavior should be fixed rather than trying to paper over it here.

Testsuite passed


Supporting conda install in kedro install command

Notice

[x] I acknowledge and agree that, by checking this box and clicking “Submit Pull Request”:

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Motivation and Context
https://github.com/quantumblacklabs/kedro/issues/24
How has this been tested?
What testing strategies have you used? TBC
Checklist

[x] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[x] Updated the documentation to reflect the code changes
[x] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes
[x] Assigned myself to the PR
[x] Added Type label to the PR

@evanmiller29 Are you still interested in completing this PR and eventually making it ready for merge? Is there anything else we can help you with before you finalise it?

@idanov - I've pushed the recommended changes to the CLI and updated the documentation in 03_new_project.md. The circleCI tests seem to be failing so I'll have a look at that

Hey @LorenaBalanQB thank you so much for all your help on this. Yo da real MVP.
I've been building the tests locally using docker and my tests are passing locally on the code build fine, but I seem to be getting a S3 load CSV test error which is really odd because I haven't changed any of that functionality..
Are you familiar with that test suite? I'm a little lost..

I've been building the tests locally using docker and my tests are passing locally on the code build fine, but I seem to be getting a S3 load CSV test error which is really odd because I haven't changed any of that functionality..

Mm I noticed, it happened again after re-build. I can't see it happening on any of our branches. Can you try resolving the conflicts and merging the latest changes from develop?

@LorenaBalanQB - I've fixed the build tests and they seem to be passing. I had a merge issue that created >>>>>> HEAD etc and I stupidly commited that code. How does it look on your end?

@DmitriiDeriabinQB - great. Is there anything else I need to do to get this approved?

@evanmiller29 you just need an approval from @idanov :)

The feature now looks pretty good, thanks for accommodating for our comments! Happy to merge once we see why some tests fail.
Thanks a lot for your contribution @evanmiller29!

Just added in another lint-friendly commit and the builds/test are passing.

Woooo. It looks like all the tests etc are passing. @idanov do I need to do anything more?

@evanmiller29 not really, all is done! I will merge your PR now, thanks a lot for your first contribution! I hope it's a start for many to come.


Build failed - Could not find artifact com.sun:tools:jar

I added mvn plugin and dependencies as documented here.
https://awslabs.github.io/aws-lambda-powertools-java/utilities/sqs_large_message_handling/
I am using  AdoptOpenJDK (build 11.0.8+10)
I am getting this error when I run mvn package
Execution default of goal org.codehaus.mojo:aspectj-maven-plugin:1.11:compile failed: Plugin org.codehaus.mojo:aspectj-maven-plugin:1.11 or one of its dependencies could not be resolved: Could not find artifact com.sun:tools:jar:11.0.8 at specified path
Can you please let me know if there is anything else I need to do?

Thanks @pankajagrawal16!
That issue got resolved now.
When I running my unit test class. I am seeing java.lang.NoClassDefFoundError for SqsLargeMessageAspect.class
Dependencies added in pom.xml-

software.amazon.lambda
powertools-sqs
1.3.0

com.amazonaws
amazon-sqs-java-extended-client-lib
1.1.0

Thanks @pankajagrawal16!
That issue got resolved now.
When I running my unit test class. I am seeing java.lang.NoClassDefFoundError for SqsLargeMessageAspect.class
Dependencies added in pom.xml-
software.amazon.lambda
powertools-sqs
1.3.0
com.amazonaws
amazon-sqs-java-extended-client-lib
1.1.0

Hmm this is Strange, What does you pom.xml looks like? Possible to share here ?

I see this warning during mvn clean package
=========================================
[INFO] Showing AJC message detail for messages of types: [error, warning, fail]
[WARNING] bad version number found in /.m2/repository/org/aspectj/aspectjrt/1.9.6/aspectjrt-1.9.6.jar expected 1.9.2 found 1.9.6
:
[WARNING] advice defined in software.amazon.lambda.powertools.sqs.internal.SqsMessageBatchProcessorAspect has not been applied [Xlint:adviceDidNotMatch]
/.m2/repository/software/amazon/lambda/powertools-sqs/1.3.0/powertools-sqs-1.3.0.jar!software/amazon/lambda/powertools/sqs/internal/SqsMessageBatchProcessorAspect.class:0
pom.xml entries related to SQS Large message processing.

com.nickwongdev
aspectj-maven-plugin
1.12.1

1.8
          1.8
1.8

software.amazon.lambda
powertools-sqs

compile

software.amazon.lambda
powertools-sqs
1.3.0

com.amazonaws
amazon-sqs-java-extended-client-lib
1.1.0

Hi @gyoga99
Did you manage to solve this? You pom entries looks correct. Also the warning that you are seeing is strange, coz this should have worked.
Since this warning seems to be coming from your unit test, can you try adding below test scoped dependency to your pom  as well ?
 <dependency>
            <groupId>org.aspectj</groupId>
            <artifactId>aspectjweaver</artifactId>
            <scope>test</scope>
        </dependency>

Hi @gyoga99
Did you manage to solve this?

Hi @gyoga99,
Wondering if the issue is solved now?

@pankajagrawal16 I have not tried this. Will try and update you.

Resolved this error. The issue was with mocking.
Thanks!


A few update AWS template

My previous pull request #4244 was closed because of lacking CLA. I have signed CLA and this is reopened pull request.

use AWS specific types for particular parameters (key pair, subnet id, vpc id). Also, it will be good to fetch subnet AZ from subnet id, but don't know yet how to do it with user specified subnet id

LGTM


Support for Typescript

Hi,
I am currently working on a JavaScript library using graphlib. I use Typescript to enhance the development.
What about add a typescript support ?
You can support Typescript by defining a "definition file" which expose the external library API. It mainly exists for compile time check.

I would be happy to take a Typescript definition as a contribution. The API is stable at this point as we're into 1.x versions.

Cool ! Working on it then !

Hey! Just coming for news. I finished the job but I want to test it in a real project to be sure that's working.
Come back to you in the next few days.

FYI I've sent out a PR to add TS definitions for graphlib to DefinitelyTyped.
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/10849

... and that PR was merged. This issue can be closed now.
typings install --global dt~graphlib

Thanks @danvk!


Tracking dependencies

It would be nice if we can have a way to tracking inter-components dependencies relationship.
Would be useful in the following cases:

Ensure dependents SLO never exceed the dependencies
I.e. if A depends on (B and C), and B and C each have the SLO of 99%, then A should not have the SLO exceeding 99^2 / 100 = 98.01

Circular dependency between services: Optional opt-out circular dependency detection. 2(or more) Services are circular dependents would have the same SLO which is the min SLO among them.

In principle I think this is a great idea -- e.g. one could add commit/deploy time integration tests for things like #2.
That said, I think in practice for #1 this will be very situational and not sufficiently generalizable. At the very least, there's significant nuance to the interpretation of the nature of dependency and the relative impact of A on B. For example, if A depends on B, but A caches results from B for a specified freshness duration, what is the mathematical nature of the dependence of A on B?

Hi @gplasky,

For example, if A depends on B, but A caches results from B for a specified freshness duration, what is the mathematical nature of the dependence of A on B?

-- begin-thinking-out-loud
I think this is the common circuit breaker pattern yeah? In this example, it would often be wiser to think of A as 2 services: A1 and A2. A1 responsibility is to update the cache(C) from B while A2 serves the data from (C).  The dependencies chain will looks something like:
B -> A1 -> C -> A2

A1 and A2 can be classified with different SLA/SLO and different error budgets.
What happen when B fail?
A1, C, and A2 are all operating under a partial degradation manner: A1 cannot extract real data to populate cache, C holding stale cache which could potentially evict, A2 could be serving stale data. You would not have the ability to scale A up/down, not having the ability to failover A to a new AZ or deploy a new version of A with fresh cache for canary purposes.
I think in term of SLO, this would be impactful, but in term of SLA, end-user might not notice any differences.
-- end-thinking-out-loud

But you are right though. The OpenSLO spec should be flexible enough to enable folks to customize the different weights of the dependency links up to the actual business / organizational requirements. 🤔

Since this is very old, I'm closing it. Feel free to reopen the issue If you want to follow up on this and push it forward, thanks!


Multiple audio players on same page how?

Please use this issue template as it makes replicating and fixing the issue easier for us. If you decide not to use it or you are vague your issue may be close instantly.
Expected behaviour - Multiple instances of PLYR
Actual behaviour - One instance of PLYR and the rest default to standard browser player
Environment

Browser: Chrome
Version: Version 67.0.3396.99 (Official Build) (32-bit)
Operating System: Windows
Version: 10

Steps to reproduce I have duplicated two players on my page, changing only the ID for the second player using the floowing code:
<audio id="player2" controls>                       <source src="/path/to/.mp3" type="audio/mp3">                       <source src="/path/to/audio.ogg" type="audio/ogg">                   </audio>
and i added this at the bottom of my page
<script>const player = new Plyr('#player2');</script>
The second player just defaults to the standard browser player, what am i doing wrong please?

It's worth reading the docs...
https://github.com/sampotts/plyr#setting-up-multiple-players


Copy and pasted text shows some html tag

As I copy and paste from reference to translation panel after comparison or import it turns to show up some HTML tags with style.

The fix in v1.3.3 wasn't complete and needed further work. This has been done now via https://github.com/friendsofagape/autographa-lite/pull/149


Disabled form-floating input uses wrong colour

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
A disabled form-floating input uses the wrong colour.
Reduced test cases
Demo shows:

disabled input in grey - correct
disabled form-floating input in black - incorrect

https://codepen.io/vwhzvdkyrlmuapzyih/pen/MWXgVmm
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Firefox
What version of Bootstrap are you using?
5.2.2

I think we do risk that, but generally that's okay I think as the end result is the same—an input with a label is disabled. I merged your other PR without seeing this—want to cut a new one if you're able?


Ensure espflash is installed on all runners

https://github.com/esp-rs/esp-hal/actions/runs/10036046426/job/27733179517

Closed with https://github.com/esp-rs/esp-hal/pull/1848


Unknown props 'route' for 'Link' component

I have the following code:
file 'context/routes'
const urlPrettifier = new UrlPrettifier(routes)
export const buildUrl = (url, params) => urlPrettifier.linkPage(url, params)

somewhere else
import { buildUrl } from 'context/routes'
...
<Link route={buildUrl('dashboard/retail/orders/show', { id: 123 })}>
  <CreateIcon style={style.icon} />
</Link>
...

It works but at the console, I see the following:
Warning: Failed prop type: Link: unknown props found: route
    in Link (created by PrettyLink)
    in PrettyLink (at TableRow.js:77)
    ....

Any ideas? Or maybe I do something wrong? Thanx

Hi @ShepelievD what version of next-url-prettifier are you using?
This problem should have been fixed in version 1.1.5 (https://github.com/BDav24/next-url-prettifier/issues/5).

@BDav24 use 1.1.4
Thanx for a help! 👍

You're welcome, keep me posted in case version 1.1.5 does not help.


Changed scripts in master and build branches to use /bin/bash

I changed scripts in master and build branches to use /bin/bash instead of /bin/sh for Ubuntu linux and possibly other Linuxes as well.
The default shell in OSx since Panther is bash. It appears that sh under OSx is either a copy of bash or a link to it.
Ubuntu Linux links /bin/sh to /bin/dash.  For some reason dash does not like some of your scripts.
The reality is if you are writing the scripts in bash then it is best to call it. Calling sh which in many cases is a link to another shell that may or may not function will not allow OS portability.
You will not want to merge the links that I changed to allow me to test on the fork of the repo so that it would not go back to the original repo for files.
You will also want to merge the one commit in the build branch in addition to the commits in the master branch.

This was already done in development branch. You can use development branch for now. It's pretty stable

Fixed in v1.0.0-rc1


Remove const-generic feature flag

We don't need the const-generics rust feature flag anymore as beta has support now: https://blog.rust-lang.org/2021/02/26/const-generics-mvp-beta.html

Everything indeed seems to just work.

Stable doesn't, but I'm happy to merge this now and accept a PR to bump the crate version when the new stable comes out.

This just removes the compiler-level flag, when it hits stable you can even remove the crate-level flag.


[Event Request] - OnCheckTotalPaymentPercAfterSetFilters in Table 12170 "Payment Lines"

Hello, please consider adding the following event to Table 12170 "Payment Lines"
    [IntegrationEvent(false, false)]
    local procedure OnCheckTotalPaymentPercAfterSetFilters(var Rec: Record "Payment Lines"; var TempPaymentLines: Record "Payment Lines")
    begin
    end;

and raise it accordingly inside "CheckTotalPaymentsPerc" trigger:
    procedure CheckTotalPaymentsPerc(NewValue: Decimal)
    var
        TempPaymentLines: Record "Payment Lines";
        TotalPaymentPerc: Decimal;
    begin
        TempPaymentLines.CopyFilters(Rec);

        **OnCheckTotalPaymentPercAfterSetFilters(Rec, TempPaymentLines);**

        if TempPaymentLines.CalcSums("Payment %") then
            TotalPaymentPerc := TempPaymentLines."Payment %" + NewValue;

        if TotalPaymentPerc > 100 then
            Error(PaymentPctErr);
    end;

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 49502.


Download .NET API documentation for offline purposes

Is your feature request related to a problem? Please describe.
I want to download the .NET API documentation for offline cases, like other languages such as Java. Because, some people needs this information in a offline context, and can be more productive without need to install Visual Studio entire IDE (plus, this IDE is Windows only, lol)
Describe the solution you'd like
I find the C# offline pdf documentation; I think this can be a solution. Also, the example of Java will be better
Describe alternatives you've considered
For now, we have devdocs.io and Zeal

Edit: devdocs.io don't have .NET API Documentation available. And Zeal .NET API Documentation is broken in various aspects

Additional context
Please, add the "Download PDF" option in .NET API Documentation, or a autogenerated .zip with the docs in .md or in .html format

cc @kexugit

PDF download is currently only enabled for conceptual content on Learn. Support for reference content, including .NET APIs, is currently planned for the current quarter.


Use Hapi's standard codes

Update the source code using the Hapi's guidelines.

Thanku for sharing this useful information.. I would like to share something too...
Thinkwik is an expertise driven web and mobile apps Development Company helping businesses achieve their goal through robust and engaging web and mobile presence. To Build real time application, mobile app and website
Node Js Development Company
Android App Development
iOS App Development


Binary search problem

What is the change?
Added a New Problem under Data Structures  folder which is Finding the Peak_Element_Index_in_Mountain_Array
Explained what is the Problem
Explained the Solution in Java Using Binary Search
closes: #117
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] I have followed the Code of Conduct.
[x] I have  checked there aren't other open Pull Requests for the same update/change.
[x] I have you tested the code before submission.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm  HSOC( Hack_Club_Rait Summer of Code ) Contributor.

Screenshots (if any)

hey @Chaitalishetty  I have modified file and added python code as well
please check and update sooner

LGTM!


Roll engine 7dfdfc6faeb6..d0698f275e5a (1 commits)

git@github.com:flutter/engine.git/compare/7dfdfc6faeb6...d0698f275e5a
git log 7dfdfc6faeb6..d0698f275e5a --no-merges --oneline
2019-08-13<EMAIL_ADDRESS>Roll src/third_party/skia f99631100372..6ef31815a694 (3 commits) (flutter/engine#10976)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (stuartmorgan@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6367979548180480 , https://cirrus-ci.com/task/5242079641337856

Commit queue failed; closing this roll.


Make comms_ctx a required arg

Purpose
We want to make comms_ctx a required argument (not optional) to avoid possible conflicts.
Closes #256
Content

[x] Removes the default value for the comms_ctx argument.
[x] Modifies the hdwrite_regridfile_rll_to_cgll function to take comms_ctx as an argument
[x] Updates docstrings

Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

[x] I have read and checked the items on the review checklist.

bors r+


ES6 import not working

When using js-file-download with an ES6 import it expects the default export to be in .default.
When importing like this:
import fileDownload from "js-file-download";

I get following error:
Uncaught (in promise) TypeError: js_file_download_1.default is not a function

This is probably some incompatibility between CommonJS and ES6 modules. There is a workaround, using the entire module as the function:
import * as fileDownload from "js-file-download";
...
fileDownload(response.data);

just add window.fileDownload = require('js-file-download') in app.js file and use it like fileDownload(fileLocation, filename);

@roti I'm not able to reproduce this in my projects. I use
import fileDownload from 'js-file-download'

without any errors. Are you using Babel or Typescript? Could you please put together a codepen or small repo that reproduces the problem?

Closing due to inactivity. Please feel free to open a new issue if this persists.


user12986714: Watch phukiensamsung(?!.com)

user12986714 requests the watch of the watch_keyword phukiensamsung(?!\.com). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

phukiensamsung(?!\.com) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Why are we watching this? It doesn't match anything. The domain does match, but what indications are that we might see the text without the TLD?


⚠ Bofrost.at has degraded performance

In c08a1c6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1463 ms

Resolved: Bofrost.at performance has improved in 2d828b7.


Docs should have its own analytics

Right now docs.celo.org and celo.org share the same google analytics tracking ID. I think we should create a separate tracking ID for docs as combining them makes the data muddy. for instance / on both is as far as i can tell not differentiated. the home page for both websites are counted together.

Closing, context here


Change german Variables to plural form

In german the plural form of the english word "Variables" is "Variablen". Maybe we can change this ;-)

Have you talked about this with @Snooz82 and @Noordsestern? Also, it would be good to change this at Crowdin as well.

I approve this change. Daniel is right!
I will change it in crowding as well.


Call to undefined function App\Http\Controllers\getDetails()

"Call to undefined function App\Http\Controllers\getDetails() " why i am getting this error as I did exactly you did??

@alisalmabadi please consider this issue. Thanks

even though I created this project about 2 years ago, but I tested it and there was not problem for me
did you run composer install?
it seems that helpers.php functions is not loading in your project, so getDetails() method is undefiend for you
I edited helpers.php right now hope be helpful
pull it


add tests for the new protocol parameters added in Alonzo era

tests for:

newly protocol parameters in Alonzo era
update proposals for changing the values of the above parameters

https://github.com/input-output-hk/cardano-node-tests/pull/506


s3: Use connection region to create bucket

Boto3 always tries to create the bucket in US Standard (N. Virginia)
region unless explicitly mentioned. So, give it the region of the
connection so that it does raise an error about IllegalLocation.
Fixes: https://github.com/noirbizarre/flask-fs/issues/31

Coverage remained the same at 94.744% when pulling 7f4ee5b0c7e6a5070a96e1a94ac1382f2223b3f9 on AbdealiJK:ajk-code into 092e9327384b8411c9bb38ca257ecb558584d201 on noirbizarre:master.

@noirbizarre A quick reminder about this

@noirbizarre reminder on this

This package seems dead.


Feature: Eslint

Add eslint and fix warnings / errors

Add eslint
Add lint task to package.json
Fix errors / warnings
replace chalk by gulp-util
replace console.log by gulpUtil.log
Use destructuring instead of importing the while lib

@AngeloGulina Solved and merged!


fix the register page ui

hey @soham0005 the register page ui is not looking appropriate according to the screen size

i will make the register page appropriate for all device widths
please assign me the issue  and mark this issue atleast level2

@AyushSharma72 hello, this issue is already in progress, do consider raising other issue or contributing to other issues.
Happy Coding!


WIP: Convert Modal view to React components

This PR has a staging branch, which means a production build of this branch exists and can be launched. Click here to preview this PR.
This PR converts the Modal view which is currently a Backbone View into React components and integrates it into the reactbridge.
This PR is to be merged after PR #138.

@gabrielcowley can you pull and merge ts_react to clean up the diff here?

@jabooth Will do

All modals have been converted to React. The Intro modal still exists in Backbone form and the old version is still used for when Landmarker is opened. Replacing this with its React version would require a significant overhaul of the startup process. I'm happy for this to be merged as it is.

@gabrielcowley OK, in the interest of unblocking let's get this merged, I'll circle back and tidy up the intro stuff later.


Migrate metastore to mysql

[x] Add Mysql support.
[x] mysql in docker.
[ ] Better support for mysql conf.
[ ] Test and migrate.

Install mysql images
docker pull mysql

Launch a docker container with mysql
docker run -p 3306:3306 --name test-mysql -e MYSQL_ROOT_PASSWORD=root -d mysql:latest

mysql is GPL license, which is incompatible with Apache License. An alternative is https://github.com/datanucleus/datanucleus-rdbms, which is used by hive-metastore

Already Finished.
@manuzhang  GPL license issue will be addressed later.


Maybe implement extension function CountBy

Applies a key-generating function to each element of an IEnumerable and returns am IEnumerable of unique keys and their number of occurrences in the original sequence.

This is easily done in pure linq, these queries return a ValueTuple
customers
    .GroupBy(c => c.Category)
    .Select(g => (g.Key, g.Count()));

or in linq syntax
from c in customers
group c by c.Category into g
select (g.Key, g.Count());

We do not implement this until it is requested with a specific use-case.

This is easily done in pure linq, these queries return a ValueTuple
customers
    .GroupBy(c => c.Category)
    .Select(g => (g.Key, g.Count()));

or in linq syntax
from c in customers
group c by c.Category into g
select (g.Key, g.Count());

We do not implement this until it is requested with a specific use-case.


Fix unknown firstpos: NilClass bug in development.

Description

Fix bug that causes this screen in development when changing Rails code.
Fix taken from: # See: https://stackoverflow.com/questions/44465118/rails-5-1-unknown-firstpos-nilclass-issue-reloading-application
Enabling this fix only in development (since it doesn't affect other environments).
Tests

Verified that bug no longer appears in development.
Verified that code changes are correctly picked up when the page is reloaded once.
Since the bug appears to be related to routing, visited some different pages and verified the routing worked properly.

Not for QA testing, right? Please add QA flags!

OMG! No more red screen? We will need to tell the entire team.


Fixed the fail to build issue on Android.

In runtime/browser/devtools/xwalk_devtools_frontend.h/.cc, the class
xwalk::NativeAppWindowDesktop is needed, but the source files of this
class's implementation at:
runtime/browser/ui/native_app_window_desktop.h/.cc are excluded within
Android build. As XWalkDevToolsFrontend is not needed for Android
platform for now, we'd better exclude its source files for Android
build as well.

@darktears @halton PTAL.

Under what circumstances does the build fail and what does the error message say (these should have been mentioned in the commit/PR message, please remember to do that in the future)? We've been producing Android canaries without any issues.

@rakuco Thanks for your reminder, I think I can open a bug for this, and post the build log onto it.

If the error message is not very big, I tend to put it in the commit message itself so people do not need to go to JIRA to find out what something was fixing in the future. See, for example, b42e0238486f08e12727873a2b2e579f11240048 or d079f0ca280bf8debc0d21c65efe980caa31b639.

Updated per above comments, @rakuco @darktears PTAL again, thanks.

Thanks. I normally keep the commit and PR in sync; the latter is what most people will look at, and it still does not contain all information.
Anyway, I'm still curious about what's different in your setup: I built Android successfully yesterday, all our bots are green and our canaries are being built. Is this a debug build? Are you using different flags?

yes that is a debug build, and I'll try to find out why when get back to office tomorrow, thanks :)

I was able to reproduce it with a Debug build indeed, thanks.
@jondong: personally I'd mention that this fixes the Debug build.
@darktears: PTAL and check if we can't just rename those files in a way that they are not picked up by the Android build.

commit log updated. PTAL again.

LGTM.

@jondong: does this also affect crosswalk-16? If so, can you backport this fix?

@jondong @halton: reping

@rakuco sorry for the late response, yes this patch should also be cherry-picked to crosswalk-16, do I need to submit another PR to do this?

yes, please. also remember to use git cherry-pick -x (the -x is especially important) to cherry-pick your commit.


TFS 3074: Error message "Error setting server name" when adding role

TFS 3074: Error message "Error setting server name" is displayed in server configuration page

The JdkServerPanel.apply method seems to be really complex. The number of threads of executions possible with all the checks seems hard to follow. We should probably break the method up into smaller chunks that are self-descriptive.


refactor(v2): make code blocks scrollable from keyboard

Motivation
Since we are now hiding the focus ring (it was done in #3626) for mouse users, we can make scrollable code blocks again.
This solves the a11y (reported by Rocket Validator) issue and improve experience for keyboard users:

Ensure that scrollable region has keyboard access

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

👍


Asset data for gas infrastructure needs to be retrieved and clearly documented

Both for the load calculation on the gas-network and the business case and other financial calculations, the following properties for all relevant types of pipes for each relevant pressure level need to be documented:

investment cost [EUR]
cost for digging in low building-density area [EUR/meter]
cost for digging in medium building-density area [EUR/meter]
cost for digging in high building-density area [EUR/meter]
lifetime [year]
yearly O&M cost [EUR]
capacity [kW]

For all connecting technologies (GOSs, DRSs, etc.) the following properties need to be documented:

investment cost [EUR]
lifetime [year]
yearly O&M cost [EUR]
capacity (from low to high pressure) [kW]
capacity (from high to low pressure) [kW]
efficiency (from low to high pressure) [%]
efficiency (from high to low pressure) [%]

Perhaps it would be best to use Markdown format such as used for this document. Please add as many (links to) sources as possible.

I'm working on retreiving these numbers in Alliander, but it takes time. My question right now is, in order to prioritize this in relation to retreiving information about heath infrastructure, how necessary is this information to integrate the gas infrastructure in ETMoses? Because when I consider the analogy with electricity these numbers are necessary to create a LES, but the user enters these numbers manually. Is this going to be different for gas?
Another question: the link to the Markdown format does not work, could you provide me a new link?

The efficiency from high to low pressure is 100% (0 kW), because it is a passive process without energy losses.
You request the efficiency in kW, but efficiency is usually a percentage. Do you want a percentage for the efficiency or do you want the losses in kW per compressor/booster?

Do you want a percentage for the efficiency or do you want the losses in kW per compressor/booster?

Percentages please!

Another question: the link to the Markdown format does not work, could you provide me a new link?

Here you go: https://github.com/quintel/etmoses/wiki/gas-infrastructure

My question right now is, in order to prioritize this in relation to retreiving information about heath infrastructure, how necessary is this information to integrate the gas infrastructure in ETMoses?

Right now, the priorities are as follows:

Which types of assets per pressure level need to be included (three types of pipes? Two types of 'connectors'?)
Numbers for the attributes of those types (see above)
Information about the heat network (sprint 3)

@Sanne512 what is the status of this? We are close to the deadline with this.

In the attachments an Excel file with asset data, the corresponding document for the GitHub (the word file is attached here and I'll e-mail the markdown version to @ChaelKruip ) and a picture to explain the structure of the gas infrastructure.
I have a few remaining questions:

In the report from Enexis provided by Netbeheer Nederland you can find the amount of stations and pipe length of the Netherlands and specified by DSO at page 17, table 8. Which ones do you want to use? The numbers they provide for the stations are not the amount of the stations, but the amount of gas streets. A station can have multiple streets, like a medium voltage area can have mutiple transformators.
The connection pipes are neglected, but as shown in the Enexis repport they have a considerable length. What do you want to do with this?

At the moment this is all the information I could gather, that I am allowed to share. As you can see the capacities are missing and the costs cannot be provided in further detail than the total CAPEX and OPEX per type of pipe and station.
Do you have further questions ? Or need other asset data?
Gas infrastructure 20160422 v0.2 word.docx
Kengetallen gasinfra 20160422 v0.2.xlsx

One addition to the information mentioned above: I forgot to inlude that the efficiency of the stations is 100% due to the passive manner of transporting the gas.

the report from Enexis provided by Netbeheer Nederland

Where can I find this report?

The numbers they provide for the stations are not the amount of the stations, but the amount of gas streets. A station can have multiple streets, like a medium voltage area can have mutiple transformators.

Sorry but I don't understand this comparison @Sanne512. Do you mean that a station is the equivalent of  a transformer? Is it possible to calculate back from 'gas streets' to 'stations'? Is a 'gas street' a physical street or something else?

The connection pipes are neglected, but as shown in the Enexis repport they have a considerable length. What do you want to do with this?

Where are they being neglected? In the Excel sheet your provide @Sanne512? Why are they neglected?

One addition to the information mentioned above: I forgot to inlude that the efficiency of the stations is 100% due to the passive manner of transporting the gas.

What about the efficiency for compression of low-pressure gas to higher pressure? Can you provide a number for the losses as a function of pressure jump?

@jeroenvdlogt could you sent the report to chael? And answer any further questions during my holiday?
The gas goes through a straat,  like electricity through a transformer,  and like an electricity station can contain several transformers a gas station can contain several steeds. So a gas street is the actual equipement that transports the gas from a higher to a lower pressure.
It is not possible to calculate back from streets to stations, but we can provide you the ratio of the stations of the alliander area.
The connection pipes are included in the sheet, but not in the different pressure levels, because they connect the buildings to the 100 mbar netwerk.
About the efficiency for transporting gas upstream: the contact for these numbers is on holiday now. I'll contact her again when i'm back.

Thanks for the answers @Sanne512!
The following additional questions have arisen after looking into the Excel sheet provided:

What are the capacities [kW] of the ‘connectors’?
Why does a GOS cost only 100 EUR?
What are the EUR/MW/pressure_jump voor compression?
What are the losses for compression in %/delta_P or %/delta_P/MW?
Is a District-station the same thing as a DRS?

the capacities of the connectors are unknown and i'm not able to retreive these numbers from within alliander
a gos only costs  €100, because alliander only has to pay for the area and structure around the gos, gts pays for the equipement inside It
i don't have the numbers for compression (yet); i'm looking into that after my holiday
a district-station is the same as a drs

Thanks for the spreadsheets and answers, @Sanne512 . I have tried to translate the data provided into the format ETMoses expects it (see Kengetallen.gasinfra.20160422.v0.3.xlsx, Input for YAML sheet.
I have also tried to work out how much gas infrastructure is on average associated with one connection (where I took the national data you provided and 7,195,532 gas connections (based on Energiecijfers.info). This is a straightforward exercise for the connectors, but gets trickier for the pipes.
I have tried to reconstruct the ratios for the different types of pipe within a specific pressure level on the Data Liander sheet from the data on the Data report sheet, but go lost at how the types of pipe on the Data report sheet map to the types on the Data Liander sheet.
What I have done instead is use the ratios @Sanne512 provided on the Data Liander sheet and the total length of pipes within a specific pressure level (where I assumed that all TransportLeiding (TL) are at 8 bar, all DistributieLeiding (DL) are at 4 bar and all AansluitLeiding are at 0.1 bar). This results in some typical number specified on the Input for YAML sheet. Do my reasoning and calculations make sense to you, @Sanne512 ?

where I assumed that all TransportLeiding (TL) are at 8 bar

@Sanne512 please note, that the data-sheet states that these lines are at 1--4 bar right @jorisberkhout?

right @jorisberkhout?

Correct, the referenced datasheet has only one entry for 8 bar pipes.

@Sanne512 can you give an indication when you can validate @jorisberkhout's assumptions and analysis?

I can do it now!
They way they relate to each other is that TL (transport leidingen) takes place at 8 or 4-1 bar, so the high and middle pressure that are specified on the first sheet. (When you compare this with electricity this is the medium voltage.) And the DL (distributieleidingen) correspond with 100 mbar pressure. (When you compare this with electricity this is the low voltage). The AL (aansluitleidingen) correspond with pipes at 30 mbar, which we have not included in the first sheet, because their influence on the capacity can be neglected. This is the final piece of pipe that connects the pipes under the pavement in the streets at 100 mbar to the houses, so they are part of the connection.
I included an Excel sheet that shows how the pipes from the Liander data are related to the open Enexis data. I hope that this clarifies it enough.
Relation pipes Liander - open data Enexis 20160504.xlsx

As for the compressors/boosters to transport gas from low to high pressure: we don't have any asset data about this, because these aren't part of our assets. At the moment transporting gas from low to high pressure only takes place at the Gasunie infrastructure (8 bar+) or at (green gas) producers, where the producer owns the compressor/booster. So for more information about these assets we advice you to contact them.

In case you didn't have the report of Enexis (yet) I uploaded it and I updated the table.
The question about a general rule of thumb for placing new stations has been forwarded to gas experts.
Investeren in de toekomst 20160122.pdf
Relation pipes Liander - report investeren in de toekomst 20160510.xlsx

Thanks for this, @Sanne512 . As discussed, I have combined all data provided by you into one analysis which is attached to this comment.
In this analysis I have determined the number of assets / length of pipes per connection (which is technically part of https://github.com/quintel/etmoses/issues/889). The results are in the table below.

A few remarks with this table:

As discussed, I have combined the Enexis data for 100 and 200 mbar into our 100 mbar category
I have also combined the Enexis data for 1 and 4 bar into our 4 bar category
I found that 650 out of the 1050 GOSses owned by GTS are used by the DSO's (the other 400 are used directly by industry and power plants)

20160510_Kengetallen_gasinfra_v1.0.xlsx

The question about a general rule of thumb for placing new stations has been forwarded to gas experts.

I have a suggestion on how to solve this. Why don't we determine how many GOS, OS, DRS, HAS and AS you need for your LES based on the table above and the number of connections. To We can then round this number to the first higher integer to avoid having fractions of a connector in the LES.

I have a suggestion on how to solve this. Why don't we determine how many GOS, OS, DRS, HAS and AS you need for your LES based on the table above and the number of connections. To We can then round this number to the first higher integer to avoid having fractions of a connector in the LES.<

I really like this idea! I've contacted a few gas experts and they couldn't give a straightforward answer or rule of thumb, because every situation differs a lot. So let's solve it this way.

I have a suggestion on how to solve this. Why don't we determine how many GOS, OS, DRS, HAS and AS you need for your LES based on the table above and the number of connections.

I like this idea!

We can then round this number to the first higher integer to avoid having fractions of a connector in the LES.

I think this is not going to work. If you model a street, it would be weird if your LES contains at least one whole GOS, OS, DRS, HAS etc. although you only use a fraction of that asset for that street in reality.

Can this be closed @Sanne512?


[ncnn] New Port

Describe the pull request

What does your PR fix?
Continuation of #25276 by @holylong

Tested the ncnn:x64-windows usage successfully

Thanks!


feat: add missing type definitions

This PR:

[X] Adds missing type definitions.
[X] Resolves #214.

I'm thinking about adding some type tests to instill some more confidence, let me know if that's something you would be interested in.

That very much would be!
We have an issue in a different eslint-community repo to do exactly that (https://github.com/eslint-community/eslint-plugin-n/issues/212). So I would (possibly) look to 'borrow' the prior art if you do end up adding some tests :)


fix(noUselessFragments): fix result has invalid syntax when under JSX attribute

Summary

closes: #4553
I add judge logic for the fix case like:
<Suspense fallback={<>Loading...</>}>
   {children}
 </Suspense>;

When the fragment child  is a JSXElement like Loading...  here, in the fix process, we will keep the JSXAttributeValue.
So it will fix as:
<Suspense fallback={Loading...}>
   {children}
 </Suspense>;

The attribute fallback's value is a JsxExpressionAttributeValue not a JsxTagExpression.
Test Plan
I add test case.

Thanks for fixing it!


[Refactor] Continuous Metrics

Implementing Brier Score and Token Edit Distance

Absorbed to #925

Was this meant to merge into big-refactor?


update meta field to add sn_available

associate PR: https://github.com/streamnative/sn-docs/pull/560

 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.


Use longer variable names within solver

I really went overboard on the whole short variable names thing within the solver. The algorithm is hard enough to follow, even for me, when variable names aren't descriptive. This issue is more or less a TODO to go through and change var names in a way that will make working in the solving algorithm less daunting.

This issue was moved to golang/dep#412


DirectiveLocation bitwise operators support.

I have restarted the build. Seems that there was just a file lock issue on the build server.

Ok, builds are all green now.


README references a top-level libpldm/ directory

This no-longer exists, and so we should clean it up to reflect the new structure, along with any other affected documentation.
At least the following should be fixed:

https://github.com/openbmc/libpldm/blame/3896ecc787743360b5b1c03e79aa8a2d0fd1c969/README.md#L41
https://github.com/openbmc/libpldm/blame/3896ecc787743360b5b1c03e79aa8a2d0fd1c969/README.md#L61
https://github.com/openbmc/libpldm/blame/3896ecc787743360b5b1c03e79aa8a2d0fd1c969/README.md#L66

hey @amboar , the lines that you mentioned point to openbmc/libpldm
for example this points to this
was that not clear in the documentation ?

hey @amboar , the lines that you mentioned point to openbmc/libpldm
for example this points to this
was that not clear in the documentation ?

Right, but the libpldm/ top-level directory isn't something that exists any more. It did when the code was part of openbmc/pldm.git, but we chopped it out. It's now not self-consistent, so we should fix it. I wasn't sure whether there were other issues along these lines, so opened an issue to try to figure that out. If there's not I can send a patch to fix up the bits I linked to above.


Drop 'start' and 'end' in favor of a single time field.

After extensive discussion about time the conclusion seemed to be that in core we would go to a single time field. Extensions like EO could define additional fields if they wanted to.

[ ] Update the spec fields and narrative in json-spec/
[ ] Update the JSON Schema
[ ] Update the OpenAPI doc and api spec narrative in api-spec/
[ ] Update the samples in the json-spec/
[ ] Update the static STAC narrative and examples in [static-catalog](https://github.com/radiantearth/static-catalog/tree/dev/json-spec

@kasey  - I think  you said you were up to take this on? If you can't let me know and I"ll find someone or take it on myself.

I think the best name for this field would just be "datetime"

I've given it some thought and still agree with there being a single time field. datetime makes sense to me

I agree with a single time field. Why not just call it "time"?

time and datetime both make good sense to me. It does seem like it could be good to have consistency with the 'time' query parameter, but we could name that datetime instead of time.

In programming libraries 'time' is a time field, 'date' in a date field, and a 'datetime' is combined. I'm sure there are some exceptions, but I don't know of any standard datetime libraries where a combined datetime field is just called date, or time.
To a user datetime is more descriptive, if it's just time they will likely look for the date field as well if they don't have an actual data record in front of them. So I think both the field and the query parameter should be 'datetime'

I agree with Matt it should be datetime. A time field here seems to imply a
quantity of time (seconds since 1970 or similar). I assume this would be an
ISO formatted datetime type of format.
On Fri, Mar 16, 2018 at 1:11 PM Matthew Hanson<EMAIL_ADDRESS>wrote:

In programming libraries 'time' is a time field, 'date' in a date field,
and a 'datetime' is combined. I'm sure there are some exceptions, but I
don't know of any standard datetime libraries where a combined datetime
field is just called date, or time.
To a user datetime is more descriptive, if it's just time they will likely
look for the date field as well if they don't have an actual data record in
front of them. So I think both the field and the query parameter should be
'datetime'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/radiantearth/stac-spec/issues/64#issuecomment-373799570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8OzJNTg0NJw5lPK8Xuk1NKK--Jx58Yks5tfABIgaJpZM4SrXNe
.

--
David Hemphill
Independent Contractor,
GravityLab |<PHONE_NUMBER>

I think 'datetime' is perfectly cromulent.  'at' is another classic spelling for this kinda field. FWIW my team considered these options and went with something a little less generic to avoid ambiguity -- is it the time the item was inserted, updated, or originally captured? That's how we wound up with 'observed', with 'at' as a second favorite. New options aside, I'll :+1: 'datetime' if that's where the consensus is headed.

re format, we previously agreed on https://tools.ietf.org/html/rfc3339
Which is a stricter subset of https://www.iso.org/iso-8601-date-and-time-format.html
The argument being that RFC3339 was simpler via strictness and therefore more likely to be handled correctly by implementations.

I think the observed datetime as just datetime is fine and users will know that's what it refers to.
In the first meeting we briefly talked about other datetime fields for metadata updates, but they never made it in core.

updated: datetime when the metadata was updated
created: datetime when the metadata was created

Both could be useful, although they aren't the sorts of fields people would search on. They are really more useful on the backend which is why I think we never included it in core. Implementors can define and use those fields to track metadata updates internally.
On the actual data side you'll often see datetimes for when the data was processed, but I also don't find this useful for users.  They might want to know  a revision or version number of the data, but likely not the datetime.

Looks like consensus is going towards 'datetime'. I do feel it's a slightly a missed opportunity to communicate a bit more information, since it's a 'datetime' of type 'datetime', and we'll have to communicate in the spec that it represents when the asset was acquired / observed. Looking at https://github.com/radiantearth/catalog-implementor-survey it seems like roughly half use some form of 'acquired' and half something that is just 'time' / 'timestamp'. But those are all focused on earth observation type stuff, so acquired may lock us in more than needed, while datetime leaves it wide open for content profiles/extensions to define what it means for them, which seems somewhat desirable.
I'll try to put together a pull request with datetime. Aim to close discussion on this by end of day tuesday, so if anyone has objections to datetime raise them now.
Also note that WFS has moved ahead with some temporal stuff, using RFC3339. See https://github.com/opengeospatial/WFS_FES/pull/89 It looks like it's most all in line with what we're doing, and that this should be compatible. It does use 'time' for the query parameter, and I think that makes sense. For us we'd use datetime for the json field name, and time for the query parameter in STAC API.

Also, I do like updated + created fields, and agree they shouldn't be in core. It could be good for us to make a mini content profile / 'traitset' / 'mixin' that defines updated and created. I think they could be useful at the WFS level (like for features in general, not just for us in STAC land).

PR up - #73
Review is appreciated, thanks all for the good discussion.

I believe this is now updated on dev in all the right places. Would be great if someone could do a second check before release.


Fix issue with clearing screen after part has been drawn

When using the mobile VR interface we are rendering the left eye and right eye on two layers of our render target.
The left eye is then blit to the left side of the screen, the right eye to the right side of the screen.
The problem here is that with OpenGL we were checking if the position of what we're blitting was not Vector2(0,0) and clearing the screen to ensure that the part not written was just left over memory junk.
In this scenario, blitting the left eye didn't trigger the clear, but blitting the right eye did resulting in the left eye data being lost.
This PR changes the logic so clearing can only happen when we draw the first element, and checks the full size of our destination.
I'm not sure of that last bit because this may not account for our title bar.

Thanks!


[Bug Report] Glow Module doesn't work on 1.8.9

Checklist before creating an issue:

[X] I'm using the latest version of Apollo.
[X] I'm using the latest version of Lunar Client. (Relaunched in the last hour)
[X] I'm running Apollo on ONLY the backend server OR the proxy. (YOU CANNOT RUN IT ON BOTH)
[X] I do NOT have both Apollo and the LEGACY API installed on the same server. (YOU CANNOT RUN BOTH AT THE SAME TIME)
[X] I've gathered all screenshots, logs and appropriate information related to creating a bug report.

Issue Description
The Glow Module, as described using the documentation, doesn't apply a glow with most colors, and for other colors it just display a black screen to the player that tries to see the glowing player.
I've also tried compiling the bukkit-example folder and use the /glow override command and it still didn't work properly.
Reproduction steps
1. Compile the bukkit-example plugin
2. Use it on a 1.8.9 paper / spigot server
3. Use the /glow override <player> command
4. The screen gets black whenever the client tries to look at the glowing player

Versions
1.8
Platforms
Spigot / Bukkit
Server Version
1.8
Modules
Optifine
Screenshots
No response
Contact Information
Discord - soladev

Additional Context
No response

Screenshots
No response

I can't believe you didn't include a screenshot of a fully black screen.. 😆
Some additional context after doing further testing, it appears to be the Optifine setting 'Fast Render' being set to 'ON'. It also appears on 1.7 and 1.12 as well as 1.8, as you mentioned. Each version having it's own slightly different side-effect from that setting being enabled. We'll be looking into fixing it, thanks for the report. 👍

I can't believe you didn't include a screenshot of a fully black screen.. 😆

Yeah i thought of adding it, but I guess it wouldn't have been useful to add a 1080x1920 .png of the color #000000 :grin:
Thanks for your answer, at least in the meantime I should be able to try the feature with Fast Render disabled

My screen is also pitch black with the exception of the players outline being stuck frozen on my screen, latest lunar client on 1.7.10.

Hey @soladev & @devfugu,
Thanks for reporting this issue, the most recent Lunar Client (v2.13.9-2406) deploy has resolved this issue.


Virtual pump for open-looping and interfacing with simulator (in silico testing)

Adding a virtual pump mode would give the possibility to use openaps for open-looping with any pump, without the necessity for pump comms.
The virtual pump should be serving the same history data as a standard Paradigm pump, and the treatment decisions should suggested/enacted. A virtual pump would also be ideal for in silico testing of the oref0/oref1 algorithms with a physiological simulator, where the simulator would serve BG and CGM values, and the virtual pump would delivers insulin to the simulated patient.
"If I could do it, I would". But with no or very few coding skills, I hope this idea gathers interest.

I think the best bet is using the simple simulator code for this (https://github.com/openaps/oref0/compare/dev...simple-simulator) within the community for retrospective and individual prospective testing. (Also, I know there's a pending publication reviewing the use of oref(0/1) in silico using the Padova simulator, so this is also being addressed in the research world, too.)  For those who want a real-time virtual pump, I recommend trying AndroidAPS with a virtual pump.


data: omlondhe

Added my profile

Hey, @khushalarora Thank you for replying, changes are done.

By the way, I'm adding a feature to the project. You will receive a pull request in the next 5 minutes, kindly review it.

Hey @khushalarora , done with changes for this page.
The feature that I was talking about is also added in PR, take a look.
Along with that, I have also raised an issue in the production code that I have solved locally, will send you a PR once you merge both of these PRs so the issue will be merged separately.
Thank you!


Add unit test for increase test coverage

Mostly for kube_jwt.go to have 100% test coverage again for https://github.com/istio/istio/pull/15571.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pitlv2109
To complete the pull request process, please assign incfly
You can assign the PR to them by writing /assign @incfly in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: myidpt, pitlv2109
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [myidpt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test integ-pilot-k8s-presubmit-tests-master

istio-codecov-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test istio-codecov-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-codecov-master

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-presubmit-tests-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test integ-pilot-k8s-presubmit-tests-master

istio-codecov-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test istio-codecov-master

integ-mixer-k8s-presubmit-tests-master
47ad161dc48f0779f03bb4062c9ad4c24daf612b
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Not working as of 11.3

on any version the follow issue appears upon opening the program:
java.lang.IndexOutOfBoundsException: Index 1 out of bounds for length 1
at java.base/jdk.internal.util.Preconditions.outOfBounds(Unknown Source)
at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Unknown Source)
at java.base/jdk.internal.util.Preconditions.checkIndex(Unknown Source)
at java.base/java.util.Objects.checkIndex(Unknown Source)
at java.base/java.util.ArrayList.get(Unknown Source)
at sdps.DamageTracker.run(DamageTracker.kt:77)
at sdps.MainKt.main(Main.kt:74)
at sdps.MainKt.main(Main.kt)

Sorry for the late response. Everything is working fine on my end. Is it still happening for you?
If you go into your Documents/My Games/Smite/BattleGame/Logs folder and send me all the .log files in a zip file, I can try to see what happened and possibly release a fix. If you just want it working again, you can move/delete all those .log files and it should be okay again.

I appreciate you getting back to me, I was able to fix it by clicking the
refresh button next to the in game name field. I know my friend had a
similar issue as me but I am unsure how he got it resolved. I will let you
know if I have any more problems.
On Sun, Apr 21, 2024 at 5:05 PM antD @.***> wrote:

Sorry for the late response. Everything is working fine on my end. Is it
still happening for you?
If you go into your Documents/My Games/Smite/BattleGame/Logs folder and
send me all the .log files in a zip file, I can try to see what happened
and possibly release a fix. If you just want it working again, you can
move/delete all those .log files and it should be okay again.
—
Reply to this email directly, view it on GitHub
https://github.com/antD97/SDPS/issues/2#issuecomment-2068198880, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BD7RPZF7ZO2ZCSU2BHB5KT3Y6QSYBAVCNFSM6AAAAABFBQOKS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRYGE4TQOBYGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


[DRAFT] Flash read pipeline

First draft of enhancing flash read pipeline

Allow for separate read and de-scramble stages
Added read buffers
Preparation for bus / flash data width differentiation

Please ignore the ibex commit, that's just a temporary thing to get around #2099.  Once that is merged, this commit will be rebased.
I would also like some comments re. breaking the large fanin for memories. That was causing almost a 3ns delay on flash... I think in the end, we're not going to have single cycle flash anyways, and we need the icache to cover that.
@sriyerg
please see my comment regarding similar behavior in simulation and fpga..I decided not to diverge since this will become a pain to debug.

@eunchan, can you also look at the commit that modifies the assertion inside tlul adapater?

Thanks @tjaychen, I have mainly one question regarding the KNOWN assertion, but otherwise this looks good to me.
Concerning the large fanout question... the behavior of the ASIC implementation will anyway be different in terms of timing, so I think aligning the generic and Xilinx targets as much as possible in terms of functional behavior makes sense to me. Can you nevertheless copy this into a prim_xilinx_flash.sv as well? We will likely have to run FPGA builds using the ImplXilinx define after instantiating the new padctrl due to differences in the prim_pad_wrapper.
We may still want to think about how we want to enable simulation (and lint for that matter) with other implementation targets that require simulation models for library elements / macros... ImplXilinx should be relatively straightforward, since the prims contain only inferrable code right now. But the ASIC case will be a bit more involved...

does this generally make sense to you though? Especially regarding the need for the pipeline, and how I am allocating and replacing buffers?

that's fine with me too.  I could do that.
On Mon, May 4, 2020 at 7:28 PM Michael Schaffner<EMAIL_ADDRESS>wrote:

@msfschaffner commented on this pull request.
In hw/ip/tlul/rtl/tlul_adapter_sram.sv
https://github.com/lowRISC/opentitan/pull/2113#discussion_r419832493:

@@ -280,7 +280,7 @@ module tlul_adapter_sram #(
`ASSERT_INIT(adapterNoReadOrWrite, (ErrOnWrite & ErrOnRead) == 0)

// make sure outputs are defined

`ASSERT_KNOWN(TlOutKnown_A,    tl_o   )

`ASSERT_KNOWN_IF(TlOutKnown_A, tl_o, tl_o.d_valid)

ah right, now I see it.
yeah... the usage of an ASSERT_KNOWN_IF would be ok, but it feels more
consistent if we where to qualify the offending signal similarly as we have
done with rdata...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2113#discussion_r419832493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUIPBFL63KYEFIUDI3RP52VTANCNFSM4MXRWU6Q
.

actually i think i'll gate this change on your other PR, because i think
the mask_fifo rvalid would be a good choice to do the gating on all
non-rdata signals that are returned upstream.
Let me know what you think.
On Mon, May 4, 2020 at 7:31 PM Timothy Chen<EMAIL_ADDRESS>wrote:

that's fine with me too.  I could do that.
On Mon, May 4, 2020 at 7:28 PM Michael Schaffner<EMAIL_ADDRESS>wrote:

@msfschaffner commented on this pull request.
In hw/ip/tlul/rtl/tlul_adapter_sram.sv
https://github.com/lowRISC/opentitan/pull/2113#discussion_r419832493:

@@ -280,7 +280,7 @@ module tlul_adapter_sram #(
`ASSERT_INIT(adapterNoReadOrWrite, (ErrOnWrite & ErrOnRead) == 0)

// make sure outputs are defined

`ASSERT_KNOWN(TlOutKnown_A,    tl_o   )

`ASSERT_KNOWN_IF(TlOutKnown_A, tl_o, tl_o.d_valid)

ah right, now I see it.
yeah... the usage of an ASSERT_KNOWN_IF would be ok, but it feels more
consistent if we where to qualify the offending signal similarly as we have
done with rdata...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2113#discussion_r419832493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUIPBFL63KYEFIUDI3RP52VTANCNFSM4MXRWU6Q
.

Thanks @tjaychen, I have mainly one question regarding the KNOWN assertion, but otherwise this looks good to me.
Concerning the large fanout question... the behavior of the ASIC implementation will anyway be different in terms of timing, so I think aligning the generic and Xilinx targets as much as possible in terms of functional behavior makes sense to me. Can you nevertheless copy this into a prim_xilinx_flash.sv as well? We will likely have to run FPGA builds using the ImplXilinx define after instantiating the new padctrl due to differences in the prim_pad_wrapper.
We may still want to think about how we want to enable simulation (and lint for that matter) with other implementation targets that require simulation models for library elements / macros... ImplXilinx should be relatively straightforward, since the prims contain only inferrable code right now. But the ASIC case will be a bit more involved...

does this generally make sense to you though? Especially regarding the need for the pipeline, and how I am allocating and replacing buffers?

I think so, yes. I mean, if the pipeline helps us to overlap some read cycles with scrambling / ECC calculation, that certainly helps performance.
concerning the buffer allocation strategy... this really depends on what read pattern we want to optimize this for. if it's mostly linear reads, RR as implemented should be fine. and we do have an icache which should already exploit locality on the instruction side. we may want to consider something like LRU / pseudo LRU if we think that this may enhance performance somewhat for data partition accesses. but assuming that we will not read data very often from flash, this may not matter too much...

actually i think i'll gate this change on your other PR, because i think the mask_fifo rvalid would be a good choice to do the gating on all non-rdata signals that are returned upstream. Let me know what you think.
…
On Mon, May 4, 2020 at 7:31 PM Timothy Chen @.> wrote: that's fine with me too. I could do that. On Mon, May 4, 2020 at 7:28 PM Michael Schaffner @.> wrote: > @.**** commented on this pull request. > ------------------------------ > > In hw/ip/tlul/rtl/tlul_adapter_sram.sv > <#2113 (comment)>: > > > @@ -280,7 +280,7 @@ module tlul_adapter_sram #( > ASSERT_INIT(adapterNoReadOrWrite, (ErrOnWrite & ErrOnRead) == 0) > > // make sure outputs are defined > - ASSERT_KNOWN(TlOutKnown_A, tl_o ) > + `ASSERT_KNOWN_IF(TlOutKnown_A, tl_o, tl_o.d_valid) > > ah right, now I see it. > > yeah... the usage of an ASSERT_KNOWN_IF would be ok, but it feels more > consistent if we where to qualify the offending signal similarly as we have > done with rdata... > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#2113 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AAH2RSUIPBFL63KYEFIUDI3RP52VTANCNFSM4MXRWU6Q > . >

Sounds good, let's do that.

i addressed the comments and changed d_error handling to be the same as others.
@msfschaffner , i'll make a copy of xilinx_flash as a separate PR, Is that okay with you?
I actually think if the behavior is the same...the xilinx_flash should really just call the generic one underneath. What do you think? That way if we have bugs we don't need to fix in 2 places.

i addressed the comments and changed d_error handling to be the same as others.
@msfschaffner , i'll make a copy of xilinx_flash as a separate PR, Is that okay with you?
I actually think if the behavior is the same...the xilinx_flash should really just call the generic one underneath. What do you think? That way if we have bugs we don't need to fix in 2 places.

Hey Tim, yes good point. I just realized myself that when a prim is not available for a specific technology, the flow will just include the generic one. I.e., the flow already correctly included the Xilinx prims that where available, and defaulted to generic ones for others like flash. I was under the impression that this has not been done yet and we where still using the generic ones exclusively.
Hence, I think you do not have to do any duplication at all in this case.

i addressed the comments and changed d_error handling to be the same as others.
@msfschaffner , i'll make a copy of xilinx_flash as a separate PR, Is that okay with you?
I actually think if the behavior is the same...the xilinx_flash should really just call the generic one underneath. What do you think? That way if we have bugs we don't need to fix in 2 places.

Hey Tim, yes good point. I just realized myself that when a prim is not available for a specific technology, the flow will just include the generic one. I.e., the flow already correctly included the Xilinx prims that where available, and defaulted to generic ones for others like flash. I was under the impression that this has not been done yet and we where still using the generic ones exclusively.
Hence, I think you do not have to do any duplication at all in this case.

sounds good. thanks for clarifying.


Made the Windows folder have SOMETHING in it

Next up will be to prepare the user files and get the start menu using those :P

:white_check_mark: Build TimeHACK AppVeyor-263 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/be0211b920 by @Alex-TIMEHACK)


Create simplified transaction payload entity

This structure will be able to carry every information related to a single transaction.

List of tasks to be done:

[ ] Create transaction input
[ ] Create transaction output
[ ] Create final transaction payload

[ ] Create bitcoin value converter utilizing satoshi representation.


Client: Allow users to select/view sires

This PR is still in progress and not ready to be reviewed yet.
Closes #43.

@delventhalz @taylorpage We have some rudimentary viewing & selecting of sires happening now! This PR is ready for review.

Viewing a Moji's Details
When viewing a nonsire that belongs to you, you have the option of selecting that moji as a sire (and upon a successful response from the server, will add a 🎩 emoji immediately to the moji's parsed DNA).

Viewing a Collection
When viewing a collection, sires are noted by a different background color and a 🎩 emoji.

Tradeoffs of Current Implementation
Currently, each <MojiItem /> being rendered in a <Collection /> makes its own AJAX call to determine if it's a sire or not. This is so that the <MojiItem /> can be reused in other situations where moji are not strictly rendered in a collection (eg, listing all available sires, listing all moji to be traded in an offer, etc).


Add CI concurrency group

What
Add CI concurrency group limiting to one build per branch, excluding main, canceling any older runs.
Why
Limit CI resource usage. We never need out-of-date commit CI runs to complete on PRs. We do need to make sure that release vs pr builds vs merge group build runs run to completion independently so that is included in the identifier for the concurrency group.

Why isn't this the default?

Why isn't this the default?

Not sure!


Consider supporting Anghami platform

Requested Feature
anghami.com is a music streaming platform that contains the largest music catalog of from the major Arabic/regional labels. I wonder if supporting it would be possible.
Possible implementation
No response

Not planned. spotDL is a tool (quoting our homepage description):

Anghami appears to be a music streaming service, and this project focuses on Spotify.
If you are suggesting it as an audio provider - we would not due to legal ramifications. yt-dlp (youtube-dl fork) handles downloading so we have no obligations for downloading rights.


Modal border radius

Hey!
How can I change the border radius of the Modal? Without using any css important rules.
Can someone post an example?
Thanks

No sure this is the preferred way, but this is how I've been overriding css attributes when using Semantic UI, without using !important: I use the Inspector to check what are the combination of classes being applied to the component, and that the browser is using to compute the relevant attribute.
Using as example your issue with the border radius, if you inspect the modal you will see that its border radiuses are set by these rules: .ui.modal>:first-child:not(.icon) and .ui.modal>:last-child. Let's say I don't want it round at all, here is what I would do:
/* jsx */
<Modal className="square-corners">

/* css */
.square-corners.ui.modal>:first-child:not(.icon) {
  border-top-left-radius: 0;
  border-top-right-radius: 0;
}
.square-corners.ui.modal>:last-child {
  border-bottom-left-radius: 0;
  border-bottom-right-radius: 0;
}

Since this is a more specific rule than what Semantic UI specifies, the browser will use it (for the specified attributes) instead.
For more information: https://developer.mozilla.org/en-US/docs/Learn/CSS/Building_blocks/Cascade_and_inheritance#specificity_2

Thank you very much for your thorough reply and the link provided!
I can now go ahead and start removing !important from my codebase.


Make changes to title bar and bullhorn logo/powered by

Hi,
Im trying to remove the large grey bar with the client name and the powered by text and logo.
Ive tried using the custom CSS within the WP custom text, the style.css within the plugin files and the portal files in the top level of the hosting but it doesn't seem to make a difference.
What am I missing?
Thanks in advance.

Hello
Can you provide an example or link of what you have tried?  Usually CSS Specificity is key here.


Adapt to API version 1.0 of dqlite

Pull Request Test Coverage Report for Build 100

116 of 318 (36.48%)  changed or added relevant lines in 6 files are covered.
120 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-17.9%) to 52.086%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

driver.go
7
8
87.5%

internal/client/client.go
8
17
47.06%

internal/client/request.go
0
16
0.0%

internal/client/message.go
17
45
37.78%

server.go
49
98
50.0%

internal/bindings/server.go
35
134
26.12%

Files with Coverage Reduction
New Missed Lines
%

internal/bindings/server.go
2
36.05%

internal/client/request.go
5
36.14%

internal/client/response.go
8
22.29%

server.go
10
52.08%

driver.go
12
67.05%

internal/bindings/errors.go
16
0.0%

internal/client/connector.go
25
68.63%

internal/client/message.go
42
54.59%

Totals

Change from base Build 73:
-17.9%

Covered Lines:
999

Relevant Lines:
1918

💛  - Coveralls


request for colab inference notebook

@KeisukeToyama great work!
hope you could create a google colab inference notebook, where we could use the colab t4 gpu.
you can give the option for the user to load a .wav or .mp3 piano audio from their computer and then get the midi output using hFT-Transformer.

Hello @GeorvityLabs
Thank you very much for the request.
I wish I could, but I'm afraid it is difficult to prepare a Google colab because I am busy with work.
I appreciate your understanding.

Hello @GeorvityLabs
Thank you very much for the request.
I wish I could, but I'm afraid it is difficult to prepare a Google colab because I am busy with work.
I appreciate your understanding.

If you could give some instructions, I can try to make one

Hello @GeorvityLabs
Thank you very much for the request.
I wish I could, but I'm afraid it is difficult to prepare a Google colab because I am busy with work.
I appreciate your understanding.

If you could give some instructions, I can try to make one

Maybe once model checkpoint is released

I'm wondering if you have read the README.md (https://github.com/sony/hFT-Transformer/blob/master/README.md)?
The evaluation step explains how we can transcribe audio. We've already shared the checkpoint already, so I think you can try it by yourself.

@GeorvityLabs Have you made one sample to load a .wav or .mp3 piano audio to get the midi output? Many thanks!


provided hook

Describe the feature
Please create a new hook, that called when new value provided.
Extend your provide function with this row:
nuxtApp.callHook('provided:' + name, value)
Additional information

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

Final checks

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

Hi :wave: do you have an exemple of how you wish to use this ?
Hookable is extensible meaning you can also implement this on your project

Yes, I know the hooks is extensible, but if I want to extend the another module with my module, I need to know when the original module provided the value. If nuxt will have provided hook we will have an universal way to handle this situations.
For examples:

I extend dayjs-nuxt modul with i18n support via my nuxt-dayjs-i18n modul. I only extend dayjs when the dayjs-nuxt modul prodvided dayjs instance. I resolved it with app:created hook, but provided:dayjs hook would be better.

I would extend vuetify-nuxt-module with vuetify3-dialog plugin via my nuxt-vuetify3-dialog module. I need the vueitfy instance for configuring vuetify3-dialog plugin, but the vuetify instance does not provieded before app:created hook. I made a PR for vuetify-nuxt-module for vuetify:created hook, but to use provided:vuetify hook would be the universal and easier way.


invalid memory address or nil pointer dereference

While running generate.exe I got the following error message:
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x18 pc=0x402383]

goroutine 1 [running]:
main.fillSharedSecret(0xc082391ee8, 0xc082366000, 0x25, 0x2a, 0x0)
        C:/diego-windows-release/greenhouse-install-script-generator/src/generate/generate.go:154 +0x93
main.main()
        C:/diego-windows-release/greenhouse-install-script-generator/src/generate/generate.go:127 +0xc82

goroutine 8 [runnable]:
net/http.(*persistConn).readLoop(0xc08208c0b0)
        c:/go/src/net/http/transport.go:928 +0x9d5
created by net/http.(*Transport).dialConn
        c:/go/src/net/http/transport.go:660 +0xca6

goroutine 9 [select]:
net/http.(*persistConn).writeLoop(0xc08208c0b0)
        c:/go/src/net/http/transport.go:945 +0x424
created by net/http.(*Transport).dialConn
        c:/go/src/net/http/transport.go:661 +0xcc3

MS Windows Server 2012 R2 Standard, running on Openstack. 64Gb RAM

Sorry I had to use Pivotal version, because I'm using PCF

Hi @axelaris , did you get this figured out? It looks like just using the wrong version of generate.exe for your cf version.

Yep, @sbenario, I had to get generate.exe from Pivotal site, not from gitub, because underlying Diego is from PCF deployment, not just CF.


Compiled on arm?

I have been trying to build it on an armv7 device. Latest node js and npm (LTS) installed but every time I run "npm ci" there are errors.
The log file is attached below.
2021-09-02T06_00_13_231Z-debug.log

Is there any documentations or so that I can follow to recreate this project?

Not sure if electron is supported on Lineage OS.
The only documentation there currently is is the code base + the Wiki that is attached to this project. Although most of the documentation is currently incomplete (Due lack of interest so far)
If you need help with the implementation then please reach out via the OpenXbox discord: https://openxbox.org/discord
I am available there with the same username as over here

I am running debian on android terminal. And with few works and tries it managed to run ci command and even executed start but then there was some issue while running, some port failure. I'll attach the log when I open my system.


HellaSwag testing on validation

Hi, I was checking some details in the HellaSwag evaluation and I noticed that the evaluation runs on the validation split of the dataset, even if the test set is available on the HuggingFace dataset. I can imagine that there's a good reason for it, but I don't know what it is. Any suggestions?

Hi! The reason for this is because the test set does not make labels public, so it can't be evaluated automatically by us. You'd have to run, save the predictions on the test set, and upload them to the Hellaswag creators' leaderboard to get test set scores.


tf.raw_ops.MapUnstage: Check failed: 1 == NumElements() (1 vs. 120)Must have a one element tensor Aborted (core dumped)

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.18.0.dev20240717
Custom code
Yes
OS platform and distribution
Ubuntu 20.04.3 LTS (x86_64)
Mobile device
No response
Python version
Python version: 3.10.14
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I encountered a bug in TensorFlow when I used API tf.raw_ops.MapUnstage  with randomly generated tensors. The code is as follows:
import tensorflow as tf

key = tf.random.uniform([15, 8], minval=-10, maxval=10, dtype=tf.int64)
indices = tf.random.uniform([1], minval=-10, maxval=10, dtype=tf.int32)
tf.raw_ops.MapUnstage(capacity=100,memory_limit=100,dtypes=[tf.float64],
                      container="",shared_name="",key=key,indices=indices)

The error message was as follows:
2024-07-22 22:05:52.420257: F tensorflow/core/framework/tensor.cc:852] Check failed: 1 == NumElements() (1 vs. 120)Must have a one element tensor
Aborted (core dumped)

I have confirmed that above code would crash on tf-nightly 2.18.0.dev20240717 (nightly-build). Also, I provided that a colab notebook to reproduce the error.
Standalone code to reproduce the issue
import tensorflow as tf

key = tf.random.uniform([15, 8], minval=-10, maxval=10, dtype=tf.int64)
indices = tf.random.uniform([1], minval=-10, maxval=10, dtype=tf.int32)
tf.raw_ops.MapUnstage(capacity=100,memory_limit=100,dtypes=[tf.float64],
                      container="",shared_name="",key=key,indices=indices)

Relevant log output
2024-07-22 22:05:52.420257: F tensorflow/core/framework/tensor.cc:852] Check failed: 1 == NumElements() (1 vs. 120)Must have a one element tensor
Aborted (core dumped)

I was able to reproduce the issue on tensorflow v2.15, v2.17 and tf-nightly. Kindly find the gist of it here.


discovery: Report PHP DDService name

What does this PR do?
The name provided by the PHP -ddatadog.service= parameter should be reported as the DDService name and not as the generated name. This makes it similar to the implementation for Java.
Motivation
Noticed while working on reporting of source of generated service name.
Describe how to test/QA your changes
Automated tests are included in the PR.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Error when trying to download apps.

Failed when searching source: msstore
An unexpected error occurred while executing the command:
0x8a15005e : The server certificate did not match any of the expected values.

Fix in development #347
will be fixed on next update


Add new ApiVersion 2019-03-01 and add optional flag 'skipShutdown' for existing Poweroff API

Changed readme.md to add new ApiVersion 2019-03-01 tag package.
Cloned a new folder for 2019-03-01 from 2018-10-01 and made version replacements.
Added skipShutdown changes for PowerOff Api in 2019-03-01 compute.json for CRP and VMSS.

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.
[ ] 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.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5713
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5713/commits/7459d4fc2dc1bd3ca41d0fb108055b9e764528a5

Could you please use https://portal.azure-devex-tools.com/app to create the PR. You can choose “Create OpenAPI sepcs”, where you should be able to choose previous api-version as the base for this new api-version. The tool will create a branch in your fork, with necessary initial commits. Then you can add a commit on the top with your change that adds skipShutdown optional flag, this will speed up both ARM and SDK review, since reviewers can go to last commit and just see what exactly changed in the new api-version.

I don't think that is needed as there is  already another pull requests making ApiVersion change which is on the verge of completion and I am piggybacking on this change.
https://github.com/Azure/azure-rest-api-specs/pull/5492

@KrisBash, @ravbhatnagar please review addition of optional query parameter to VM shutdown API.


@click don't work if  is wrapped in  with a fallback

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.7.4
CLI Version:      3.9.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Version
v2.8.1
Reproduction
https://stackblitz.com/edit/nuxt-starter-b5qlnd?file=app.vue
Description

Open the reproduction link and click the button, everything works.
Uncomment the <template #fallback> part which includes another UButton. Reload the page and click the button, it doesn't work.
Add @click="count++ to the fallback button. Reload the page and click the button, it works.
Add @click="count-- to the fallback button. Reload the page and click the button, it decreases instead of increasing.

Additional context
No response
Logs
No response

Hi @MuhammadM1998,
Reproduction link doesnt work anymore. Assuming you were sorted with this. Going to close issue as it has been stale for a while. If you want it re-opened please tag me.


Were the segonly docker images removed from docker hub?

Question/Support Request
Were the segonly docker images removed from docker hub? Was trying to build deepmi/fastsurfer:segonly-gpu-v1.1.1 and noticed that it wasn't found. Looks like only the Complete Fastsurfer images are available
...

Yes, we stopped publishing the seg only and surf only images to reduce complexity. The surf pipeline only increases the size marginally. We have flags to run only surf or seg in the full package. Alternatively users can also specify run_prediction as the entry point directly (in case default run_fastsurfer is not sufficient).

Just do add that tiny bit of documentation @ashmistry36
replace deepmi/fastsurfer:segonly-gpu-v1.1.1 to --entrypoint "python /fastsurfer/FastSurferCNN/run_prediction.py" deepmi/fastsurfer:latest to get the segonly interface in current images.

Unless you want to use the csv-option to process multiple images, you can also just add the --seg-only flag, in wich case there is no need to redefine the entrypoint.


Sorting issue

On several pages we have Md2 Datatable built in which are using "md2SortBy".
We noticed that the buttons in our menu trigger the sort in some way we can't, and is hard to explain.

Steps:
Activate sorting
As seen in the above screenshot, we set the active sorting on the user's id. Everything seems to go fine until here.
The code is set up like this:

		Creation date
		IsNL
		Type
		User ID
		Amount
		Paid out
		In progress
		Processed
		Invoice nr.
		Invoices
		Export

Hover / Mouse leave on buttons.
Notice the buttons in the right top. These are normal buttons with no special events bound to it which in our opinion could trigger the md2SortBy function.
But still, if you look at how the rows are sorted now, there's a slight difference (Notice the dates)
Code of the buttons:
<md-toolbar>

	  	<button md-icon-button click="sidenav.toggle()">
	        <md-icon>menu</md-icon>
	    </button>

	  	<button md-icon-button [md-menu-trigger-for]="friends">
	        <md-icon>group_add</md-icon>
	    </button>
	    <md-menu #friends="mdMenu">
	        <button md-menu-item> 2 new friends </button>
	        <button md-menu-item> 4 new messages </button>
	        <button md-menu-item> 3 friend requests </button>
	    </md-menu>
	    <button md-icon-button>
	        <md-icon>mail</md-icon>
	    </button>
	    <button md-icon-button>
	        <md-icon>keyboard</md-icon>
	    </button>
	    <button md-icon-button>
	        <md-icon>notifications</md-icon>
	    </button>
	    <button md-icon-button (click)="logout()" mdTooltip="Logout">
        	<md-icon>power_settings_new</md-icon>
        </button>

</md-toolbar>

This happens for several tables in our application, while we see no reason why the sorting should change on mouse-enter/mouse-leave of these buttons.
Any help or information would be appreciated!

+1

+1

+1

@ISevenSinsI, we have checked but not able to generate it our end. data-table is working properly with it's all functionalities. could you please provide plunker demo?

Demo: https://github.com/shalugin/md2-datatable-sample
There is no bug without material style (no hover effect).

Step to reproduce on demo http://code.promactinfo.com/md2/#/datatable

Sort by Age asc/desc
Click on "Filter By Name"
Click outside table

@dharmeshpipariya is my pull request incorrect?

@shalugin we need to review it


Allow "page" route parameter to be used in Mvc controllers

Fixes #6660

Do these places need to change as well?
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Razor/RazorViewEngine.cs#L244
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Razor/RazorViewEngine.cs#L321

They don't as yet. If we do fix it in the View Engine, would we do the same thing for the inverse case, i.e. not read controller or action when working with a page?

Maybe....

The layering makes it tricky to determine if you're working with a PageActionDescriptor. I guess we could solve it using a contract IRequiredRouteValueProvider but for the 2.0.1 release, the smallest amount of code we could write would be to test if we're not in a ControllerActionDescriptor

I would look at ActionDescriptor.RouteValues for the presence of page vs controller.

🆙 📅


.sh file for SeqRepo and UTA

would it be practical/worth it to combine the seqrepo and uta acquisitions and put it into a .sh file for ease of use? (that's a real question -- it could be easier for the user experience to see the instructions and perform them themselves, not sure how easy it is to do it all at once)
Originally posted by @jsstevenson in https://github.com/cancervariants/variant-normalization/pull/112#r639136508

wow what a smart idea


Add size to AssetManagerAndQuarantinedFileStorage

To avoid the following exception when calling AttachmentData#file_is_not_empty.
NoMethodError
undefined method `size' for
#<Whitehall::AssetManagerAndQuarantinedFileStorage::File:0x00007f185b9de250>

This should fix the exceptions we've been seeing in production since deploying the changes in
https://github.com/alphagov/whitehall/pull/3734.

Thanks @floehopper. I'll get this merged and deployed.


bananium radiation intensity increases with stack size

About the PR
title
goes from 0.3 up to 3.0 intensity at full size
Why / Balance
being able to stack up 10 bananium and have the same radiation as 1 doesnt make sense
also doesnt make sense that you could have 10 bananium on the same tile and it would heavily irradiate that 1 tile but not spread since their intensity wouldnt combine
makes handling lots of bananium more dangerous as a result
Technical details
added StackRadiationComponent which will update RadiationSourceComponent's Intensity when stack size is changed.
Media
a full stack of bananium with a wide radiation area:

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
:cl:

tweak: Bananium's radiation intensity increases in large stacks now.

if we're merging this the value needs to be way more sane because 3 rads for a stack is literally equivalent to a small singulo

1.0 for a stack?

When shouldn't radiation intensity increase when stacked? Why make it a separate component, instead of checking for a stack in RadiationSystem?

then id have to add baseIntensity to every radiation source (admittedly there arent that many) or have float inaccuracies pile up by restacking and dividing multiplying over and over

then id have to add baseIntensity to every radiation source (admittedly there arent that many) or have float inaccuracies pile up by restacking and dividing multiplying over and over

I don't understand what you think the problem is. Instead of storing the cumulative radiation source, why not just multiply by the number of items in the stack at time of use in UpdateGridcast()?

o thats smart

New patch looks great, thank you very much. I'll give it a quick local test soon.