Normalize & standardize tags

There's currently a lot of confusion with tags. As an example we have 4 variants for "lightning talk".

http://pyvideo.org/tag/lightning-talk/
http://pyvideo.org/tag/lightning-talks/
http://pyvideo.org/tag/lightningtalk/
http://pyvideo.org/tag/lightningtalks/

I propose tags be:

space-separated strings
all lowercase, for consistency

I'm also going to go through and clean up the multiple variant situations I find. Anyone else is free to help. :-)

@kojoidrissa I was just about to open an issue about this same thing. I was about to make a YouTube playlist of all PyCon lightning talks and then realized PyVideo might have already solved this problem for me... then I saw the duplicate tag situation for lightning talks.
:+1: for space-separated lowercase strings
I wonder whether things like WebTest or StringIO should be exceptions. Exceptional cases seem like they'd be rare, so maybe we could just deal with stringio and similar looking odd.
Did you clean up the lightning talk ones already?

@treyhunner No, I haven't cleaned them up yet. My fixes there were in a branch that got killed when I was confused about git submodules. I'll get to it in the next day or two.
I considered the 'exceptions' situation, but I felt like it would add to confusion for volunteers wanting to add or update tags. And we've already got MORE than enough confusion as it is. :-)
Perhaps in the future, when things are in more order with the tags, we can go back and consider those types of updates. However, that DOES raise a question of how the search function works on PyVideo: is it case sensitive or not? I don't know.


⚠ Bofrost.be has degraded performance

In f930124, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2340 ms

Resolved: Bofrost.be performance has improved in fa82a35.


Limit view to 100°?

Hey guys. By the way what a great tool! I am using it for my work and i have a very tricky question
for you all.
Everything is working fine but on one special Marzipano i need to limit the view so the user is not able to turn around completely. Why? Because my rendert pano image cant be full 360°. So do you know how i can limit the view for example to just 100°? Would be so nice!
Kindest regards Matthias

You need to write a custom view limiter in Javascript to prevent the non-existent portions of the panorama from becoming visible.
(There is a built-in RectilinearView.limit.yaw, but it probably doesn't do what you want: it limits the range of yaw values you can center the view at, but depending on the fov, the non-existent portions of the pano might still be visible on the sides).

Hi all!
Are there any updates/solutions?

My custom modifications were:
//changed in index.js
    var limiter = Marzipano.RectilinearView.limit.traditional(data.faceSize, 100*Math.PI/180, 120*Math.PI/180);
    var view = new Marzipano.RectilinearView(data.initialViewParameters, limiter);
// to
    var limiter1 = Marzipano.RectilinearView.limit.traditional(data.faceSize, 100*Math.PI/180, 120*Math.PI/180);
    var limiter2 = Marzipano.RectilinearView.limit.yaw(-0.2, 0.2);
    function myLimiter(params){
      var modifiedParams = limiter1(params);
      modifiedParams     = limiter2(params);
      return modifiedParams;
    }
    var view = new Marzipano.RectilinearView(data.initialViewParameters, myLimiter);

And everything's working excellent.

@Boorj - Thank you, In part this is what I was looking for, but i have a little problem because my project has two scenes (scene1 and scene2), how can I do so that scene2 has different limits of scene1?
Thanks in advance.

Here a solution for using different values of yaw and pitch limits in different scenes.
In file data.js
In this example i have two scenes that are referenced in the data.js file as 0-outdoor and 1-indoor
In file index.js
Step one - Find the line of code below and comment it
var limiter = Marzipano.RectilinearView.limit.traditional(data.faceSize, 100Math.PI/180, 120Math.PI/180);
Step two - Replace with the code below
var limiter;
if (data.id == '0-outdoor') {
limiter = Marzipano.util.compose(
Marzipano.RectilinearView.limit.traditional(data.faceSize, 100Math.PI/180, 120Math.PI/180),
Marzipano.RectilinearView.limit.yaw(-0.2, 0.2),
Marzipano.RectilinearView.limit.pitch(-0.1, 0.1),
);
}
else if (data.id == '1-indoor') {
limiter = Marzipano.util.compose(
Marzipano.RectilinearView.limit.traditional(data.faceSize, 100Math.PI/180, 120Math.PI/180),
Marzipano.RectilinearView.limit.yaw(-1.5, 1.5),
Marzipano.RectilinearView.limit.pitch(-0.9, 0.9),
);
}
else {
limiter = Marzipano.RectilinearView.limit.traditional(data.faceSize, 100Math.PI/180, 120Math.PI/180);
}

just some formatting for your solution (haven't check that manually):
var limiter;
var PI180 = Math.PI / 180;

switch (data.id) {
    case '0-outdoor':
        limiter = Marzipano.util.compose(
            Marzipano.RectilinearView.limit.traditional(data.faceSize, 100 * PI180, 120 * PI180),
            Marzipano.RectilinearView.limit.yaw(-0.2, 0.2),
            Marzipano.RectilinearView.limit.pitch(-0.1, 0.1),
        );
        break;

    case '1-indoor':
        limiter = Marzipano.util.compose(
            Marzipano.RectilinearView.limit.traditional(data.faceSize, 100 * PI180, 120 * PI180),
            Marzipano.RectilinearView.limit.yaw(-1.5, 1.5),
            Marzipano.RectilinearView.limit.pitch(-0.9, 0.9),
        );
        break;

    default:
        limiter = Marzipano.RectilinearView.limit.traditional(data.faceSize, 100 * PI180, 120 * PI180);
}


[ADP-2355] split wallet delegation and metadata (database part)

add readDelegation_ field to DBDelegation
removed readWalletDelegation
move the composition of  WalletMeta and WalletDelegation to mkDBLayerFromParts

Comments

Issue Number

ADP-2355

I think the idea of a PR with a bottom inside is not so great ...


How to make the font bold in case i use a font other than specified in PDFBOX

Hi,
I am trying to use a different font which is not part of default pdfbox. so loading the ttf file for that font and adding modifying the contentstream to use that font.
but i am not able to make the font bold where needed in tables. like some columns require bold for the same font and some does not. can you tell me if its possible to bold the font (without using existing fonts like times bold) etc

Have a look at this thread on stackoverflow:

https://stackoverflow.com/questions/21561298/pdfbox-same-stream-with-bold-and-normal-text
this gist illustrates how to use it in easytable: https://gist.github.com/vandeseer/d5c8ac342e52cbfa521ce0a55849b7e4

As said in the answer of the linked stackoverflow post:

The preferable way for creating bold (or otherwise styled) text is to use a font variant which has been created explicitly for that variant.


Form refactor: improve error message when wrong API provided to submit

Minor DX improvement, when providing the wrong API to submit, currently getting:
** (BadMapError) expected a map, got: nil
    (ash_phoenix 0.4.23-rc.1) lib/ash_phoenix/form/form.ex:698: AshPhoenix.Form.submit/3
    (crm 0.1.0) lib/crm_web/live/profile_live.ex:83: CrmWeb.ProfileLive.handle_event/3

Interesting, when I tested it I got this output

If you can recreate w/ a test I will fix :)


ApparmorProfile CRD should be a cluster-wide resource

What would you like to be added:
The  ApparmorProfile CRD should be cluster-wide scope, and not namespaced. I would argue that also SeccompProfile should be the same.
The Profile CRDs  are currently namespaced and there are a number of issues with this:

apparmor profile during the installation, it gets loaded into the Linux kernel of each ndoe, and it is cluster-wide available, any pod can reference it into the security context.
There isn't any mechanism in the security context to allow name-spacing the secruity profiles.
It is possible currently to overwrite a profile from one namesapce with another profile created in a different namesapce if both profiles have the same name. This can be used to preform cross namespaces attacks. For instance a less permissive profile gets overwritten by a profile with more privileges from a different namespace even though the profile creator doesn't have permissions to do so in the original namesspace.

Why is this needed:
This will enhance the security and make it more transparent to RBAC policies that actually this profiles are cluster-wide.

cc @mhils


add tag feature for meter

What about add tag feature in metrics? An example
metrics.meter("requests").tag("userName", "jiming").tag("function", "pay").mark();

Later I can select the data by userName=jiming Or userName=liu ..., to filter the data. Kind of like sql.
Which is going to make metrics super powerful and helpful than ever.

+1. Dimensions would be very a very useful addition especially when using metrics for operational insight.

Already done the v4 branch

hi Ryantenney,
Any plan for v4 release? Or could you please tell me which class is the one you done in master? Eagerly to see the feature!
Thanks a lot and have a nice day!
Jiming

The class is MetricName which has the concept of tags for dimensions, see here - then the MetricRegistry uses MetricNames instead of Strings for the keys, for instance here - you can still pass in just a String based name but the underlying Map is MetricName ---> Metric here

Dear Ryantenney,
Thanks for your quick and detailed reply:)
Are you going to support aggregate the report for same metric name? Such as I would like to query the report for metric "accessApi" with follow condition:

accessApi: get total access count
accessApi(status=success): get total access count which were success
accessApi(status=failed): get total access count which were failed
accessApi(status=failed,source=web): get total access count which were failed and source is web

And one more important question, any plan for release date?
Have a nice day!
Jiming

@ryantenney  Can you please comment commit ID related to this issue/feature? I'm not able to find any commit for this issue.
Thanks,
Piyush

+1, this feature would be very useful for my work! do you have an estimated release date?

Support for tags is planned for the 5.0 version. Please see the discussion in #1175.

Hey Folks,
Whether support for tags in already released ? I am unable to find any documentation on it.
Is it kept on hold for release ?

@shahbaaz31here See https://github.com/dropwizard/metrics/issues/1704.


Zip file problems

The file was unzipped and copied to the Arduino libaries folder. I had to do this because importing through libaries add  azip file- it gave an error as invalid libary, so i copied as stated.
How ever when compiling a program, i get the error "loading library from c:\Users\Alan_\Documents\Arduino\libraries\T-Display-S3-main: invalid library: no header files found"
Also tthe libary does not show up in the files/examples.
Your help would be appreciated.

Just put the T-Display-S3 lib folder in the specified location, and then open any one of T-Display-S3/example to start the next step

My "C:\Users\Alan_\Documents\Arduino\libraries\T-Display-S3-main" folder now only cotains the lib folder.
I selected the  "C:\Users\Alan_\Documents\Arduino\libraries\T-Display-S3-main\lib\TFT_eSPI\examples\160 x 128\Pong_v3" file and tried to upload it, only to get the error as shown in the  error.txt file
errors.docx

I already have TFT_eSPI version 2.5.0 by Bodmer installed.
I only received my T-Display S3. When first plug in it did a wifi scan and the tried to connect to  a Chineseweb site and failed. I tried to load the getting started file bi Volos Projects and it would upload. I then downloaded the firmware tools and uploadedthe tft test firmware. This was succesful and I now have the clourful logo and then all the test fonts.
But I can't up load a program. I have no trouble with LilyGO T-DisplayS3 AMOLED, so why am I having trouble with T-Display S3??

If you install TFT through the ArduinoIDE library manager_ ESPI, you need to enable Pinmap for T-Display S3 ,See User_Setup_Select.h, otherwise it will be the default Pinmap.
T-Display S3 AMOLED does not rely on TFT_ ESPI, so no settings are required

This is my User_Setyp_Select.h:
#include <User_Setups/Setup206_LilyGo_T_Display_S3.h>
This is the message i get when i try to upload:
"esptool.py v4.5.1
Serial port COM12
Connecting......................................
A fatal error occurred: Failed to connect to ESP32-S3: Wrong boot mode detected (0x8)! The chip needs to be in download mode.
For troubleshooting steps visit: https://docs.espressif.com/projects/esptool/en/latest/troubleshooting.html
Failed uploading: uploading error: exit status 2"
How do i get into download mode??
For this program (Volos Project)
S3ProgressBar-main.zip

See README FAQ

Where is :
README FAQ
Please??

https://github.com/Xinyuan-LilyGO/T-Display-S3#faq

I put the factory firmware.bin back, followed the FAQ
and tried to upload. Result-
esptool.py v4.5.1
Serial port COM12
Connecting...
A serial exception error occurred: Write timeout
Note: This error originates from pySerial. It is likely not a problem with esptool, but with the hardware connection or drivers.
For troubleshooting steps visit: https://docs.espressif.com/projects/esptool/en/latest/troubleshooting.html
Failed uploading: uploading error: exit status 1

Did you follow the order of the FAQs? I'm referring to putting the ESP into download mode. If all else works, take the shot, including keystrokes, send it to the store where you bought it, and let them handle it.

Yes I did follow the FAQ Hold BOOT,press RST, release RST, release BOOT.
Upload code, and it still fails!
I got the T-Display S3 form LilyGO

You mentioned above that you also have an AMOLED? Is it normal?
T-Display-S3 Please record the operation video and contact the merchant where you purchased it.

The AMOLED display works just fine.
I have started a conversation with LilyGO as to the resetting the upload. Iam waiting to hear back.

From: Lewis He @.>
Sent: 19 August 2023 11:27
To: Xinyuan-LilyGO/T-Display-S3 @.>
Cc: AlanLL @.>; Author @.>
Subject: Re: [Xinyuan-LilyGO/T-Display-S3] Zip file problems (Issue #184)
You mentioned above that you also have an AMOLED? Is it normal?
T-Display-S3 Please record the operation video and contact the merchant where you purchased it.
—
Reply to this email directly, view it on GitHubhttps://github.com/Xinyuan-LilyGO/T-Display-S3/issues/184#issuecomment-1684922797, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIGTR6SDKRU3W5WRPYRZ2ODXWCPIVANCNFSM6AAAAAA3VS72VQ.
You are receiving this because you authored the thread.Message ID: @.***>

Prviously to reset the upload method was not working because I was holding down the BOOT key and pressing I014 button (I thought this was the RST key) Now I Know the RST button is on the side of the BOOT button.
It looks like I've caused my self all of these problems because I did'nt RTFM!


Update cpp-type-system-modern-cpp.md

Add semicolon

Learn Build status updates of commit ae725db:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cpp/cpp-type-system-modern-cpp.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Implement Macro feature

As requested, the macro addon has been implemented and is here in this PR.
Important thing to note is that the allocated flash space has been raised from 8K to 16K.
This addon supports 6 macros with 50 input lines each which also allows for waiting before moving on to the next line. This could be updated in the future, as needed.
Room for improvement is plenty. The logic for macro itself could be significantly optimized, and is frankly dumb. Same can be said for both UI and UX. But right now where it is at seems like a good foundation.
Thanks to @TheTrainGoes and @InfraredAces for testing and feedback.
Extensive testing is still pending, so reviews are appreciated.
Closes #235.
Please review and merge, thanks!

Testing this Action run I ran into issue with setting a macro. I was able to set the Macro Pin, enable Macros, and save. However, when I set a specific macro as "Activated", it would not save and threw an error in the browser console. I've included screenshots below.

Thank you for testing! These should be fixed now.


new(ci): add dependabot for github actions.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
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:

OWNERS [Andreagit97,FedeDP]

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


Update architect-orb to 2.1.0

Update architect-orb to 2.1.0 which will use giantswarm.github.io as catalog base url. The old behaviour was to use giantswarm.github.com which will be deprecated in april by GitHub. Towards giantswarm/giantswarm#15898

ping @yasn77

@ubergesundheit Sorry dude, completely missed the comment from March.
This repo  can be archived


Update budget split and conversions count for Key Strategy 'B'

Fixed daily conversions for Key Strategy B
Fixed budget split for Key Strategy B

added warning for budget split values
updated validation process

@maudnals , applied changes. Can you please approve again?


chore(deps): update actions/checkout action to v4.1.6

This PR contains the following updates:

Package
Type
Update
Change

actions/checkout
action
patch
v4.1.3 -> v4.1.6

Release Notes

actions/checkout (actions/checkout)
v4.1.6
Compare Source

Check platform to set archive extension appropriately by @cory-miller in https://github.com/actions/checkout/pull/1732

v4.1.5
Compare Source

Update NPM dependencies by @cory-miller in https://github.com/actions/checkout/pull/1703
Bump github/codeql-action from 2 to 3 by @dependabot in https://github.com/actions/checkout/pull/1694
Bump actions/setup-node from 1 to 4 by @dependabot in https://github.com/actions/checkout/pull/1696
Bump actions/upload-artifact from 2 to 4 by @dependabot in https://github.com/actions/checkout/pull/1695
README: Suggest user.email to be 41898282+github-actions[bot]@users.noreply.github.com by @cory-miller in https://github.com/actions/checkout/pull/1707

v4.1.4
Compare Source

Disable extensions.worktreeConfig when disabling sparse-checkout by @jww3 in https://github.com/actions/checkout/pull/1692
Add dependabot config by @cory-miller in https://github.com/actions/checkout/pull/1688
Bump the minor-actions-dependencies group with 2 updates by @dependabot in https://github.com/actions/checkout/pull/1693
Bump word-wrap from 1.2.3 to 1.2.5 by @dependabot in https://github.com/actions/checkout/pull/1643

Configuration
📅 Schedule: Branch creation - "every weekday after 4am" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Provide annotation support to  override auto generated operationId from OpenAPI spec

Description:
In relation to this issue https://github.com/ballerina-platform/ballerina-standard-library/issues/5374 , we are suggested to use annotation in case the user needs to override the generated operationId with their value.
Describe your solution(s)
Provide resource level annotation to keep the resource metadata that needs operation need to be included.
import ballerina/openapi;
import ballerina/http;

service /v1 on new http:Listener(9090) {
    @openapi:ResourceInfo {
          operationId: "user provide operationId"
    }
    resource function post pet (@http:Payload json payload) {
    }
}

When providing the operation Id annotation, we need to implement it to return validation error in case two resource functions have the same operationId value user given.
@hevayo , @shafreenAnfar , @chalitha1989 , @NipunaRanasinghe Looking forward to hearing your suggestions on any changes or improvements that can be done 🙂
Related Issues (optional):
https://github.com/ballerina-platform/ballerina-standard-library/issues/5374 , https://github.com/wso2-enterprise/choreo/issues/12436

This improvement provide via this issue https://github.com/ballerina-platform/ballerina-library/issues/6621. Therefore we closed this issue


nodePackages.prettier-plugin-go-template: add package

Description of changes
Added nodePackages.prettier-plugin-go-template. #324008
Things done
I followed the steps here.

Added the package to node-packages.json.

Ran ./pkgs/development/node-packages/generate.sh.

Tested nix-build -A nodePackages.prettier-plugin-go-template.

npm WARN config production Use --omit=dev instead. Not sure if this is relevant.

I also included the full result of ./pkgs/development/node-packages/generate.sh in a second commit.

I'm not sure if prettier should be added as a dependency.

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[ ] Tested basic functionality of all binary files (usually in ./result/bin/)

24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@Luflosi @wegank I probably need to remove the second commit, right?

We're actually migrating packages out of nodePackages (see https://github.com/NixOS/nixpkgs/issues/229475), so adding a package there is not optimal...


backend_oculus: fix controller reader npe

Looks like the NPE caused by the recent controller change is easy enough to fix. @roshanch Please review and verify though.
GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

if ovrControllerReader is instantiated in constructor, it will cause timing issues. I think @NolaDonato or @dnvasilev can comment here. I would say adding null check in viewManager would be proper fix, though i couldn't reproduce the timing issue now.

Let's identify the timing issues. Null check in ViewManager may not be enough since onSurfaceCreated and onResume can happen in different order; so you may never call the reader from onResume.

Now for all backends the reader is instantiated from the ctor; oculus was the exception for unknown reason. Also whatever OvrControllerReader's ctor does is pretty innocent allocation and initialization. I will deal with these timing issues if they are there.

Looked back at the history. Looks like whatever race there is, it is about when a reader is attachReader-ed to the controller and not when the reader is instantiated.


Just doesn't work

Description
installation was successful, but the code is throwing errors.

Checklist

[X] I have read the README before submitting this report.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
Version information
No response
Destination operating system
No response
Xcode version information
No response
Swift Compiler version information
No response

import SwiftUIIntrospect
missing


bug: Failed to run app.py on start

Hi, long time no see. :)
I'm trying to use the jnotebook_reader for internal jupyter notebook users, but I've met an error on starting app.py. I will create a pull request soon to fix this.
% python app.py
Traceback (most recent call last):
  File "app.py", line 16, in <module>
    from flask import Flask
  File "/opt/anaconda3/envs/jnotebook_reader_py38_2/lib/python3.8/site-packages/flask/__init__.py", line 14, in <module>
    from jinja2 import escape
ImportError: cannot import name 'escape' from 'jinja2' (/opt/anaconda3/envs/jnotebook_reader_py38_2/lib/python3.8/site-packages/jinja2/__init__.py)

LGTM


Add always-up-to-date REST API to the docs

Proposal
Depends on #190.

Once we have an auto-generated OpenAPI spec (#190), we can insert it into our documentation site.
Our docsite can be found in /docs.
We use Mintlify. Mintlify has a feature for this. Read more about it here.

Use-Case
No response
Is this a feature you are interested in implementing yourself?
No

I'd like to take this.

@arielweinberger
I'm trying to set this up. Unable to run the prisma migrations. Says that the authentication credentials are incorrect for Postgres.

Hi @SniperBuddy101 it's possible that you haven't started Postgres using Docker Compose. Please check the READM.md and if you have any issues please share logs.
This task is blocked by an existing PR. I plan to review and merge it today/tomorrow and then this will be available.
Please do keep me posted about whether you managed to run Pezzo locally or not. If you did, happy to let you take it!

@arielweinberger
Understood. There was a process running on port 5433.
Was able to get pezzo working locally. Will work on this issue once the dependent PR is merged.

@SniperBuddy101 Feel free to give it a shot. You can use the following endpoint which contains the API spec for now: https://api.staging.pezzo.ai/api/spec
And here are the docs on how to do it:
https://mintlify.com/docs/api-playground/configuration
No need for authentication/playground. Please try to finish this in the next 1-2 days, and keep me posted if you need any help/any delays. Good luck!


rpc: Update getHealth documentation

Closes https://github.com/stellar/stellar-docs/issues/515

Preview is available here:http://soroban-docs-pr782.previews.kube001.services.stellar-ops.com/

Overriden by https://github.com/stellar/stellar-docs/pull/549


Fixed missing word of README.md

Fixed missing word.

https://twitter.com/sayabeans_0011/status/1626106032383750144?s=46&t=h1XEQmhEX46VuztnglQAmA


Update image classification accuracy to 5 significant digits

...according to https://github.com/mlperf/inference_policies/issues/38.

I don't have any data for the ONNX/PyTorch model as I cannot run it.

But I have updated CK instructions on how to preprocess exactly in the same way as the official code does with 58GB of disk space required for both ResNet and MobileNet, and how to preprocess with a little drop in accuracy but with only 7GB of disk space required.


[7.2] File picker gets stuck at loading, but 7.0 get through

The setup for OD file picker is

If I use the 7.2 version, it gets stuck at https://<IP_ADDRESS>:8181/abc/xyz/xyz0102e.jsf?oauth={"clientId"%3A"....
The version 7.0 works fine, but since I am going to upgrade to 7.2 for being able to pick file from sharepoint, I would like to ask for help.
Thanks

@tvanh I would advocate you upgrade to v7.2 if not already as it includes a number of bug fixes. We do not plan to retroactively patch v7.0.
Please let us know if you still experience this problem in v7.2. Thank you.

I would like to re-open this.
In 7.2, I got this error:
Uncaught TypeError: (intermediate value)(intermediate value).scopes.map is not a function
at OneDrive.js:2
at T (OneDrive.js:2)
at HTMLDocument. (OneDrive.js:2)
Any idea about this?

I would like to re-open this.
In 7.2, I got this error:
Uncaught TypeError: (intermediate value)(intermediate value).scopes.map is not a function
at OneDrive.js:2
at T (OneDrive.js:2)
at HTMLDocument. (OneDrive.js:2)
Any idea about this?

@tvanh can you use the unminified build at http://js.live.net/v7.2/OneDrive.debug.js and paste the stack-trace?
Also, what browser(s) does this occur in?

Here is the stack trace
OneDrive.debug.js:764 Uncaught TypeError: config.scopes.map is not a function
at buildAADOauthV2Url (OneDrive.debug.js:764)
at redirectToOauthPage (OneDrive.debug.js:737)
at handleOauth (OneDrive.debug.js:690)
at HTMLDocument. (OneDrive.debug.js:682)

oauthConfig
clientId: "XXXX-XXXX-XXXX-XXXX"
endpoint: 1
origin: "http://localhost:8886"
redirectUri: "http://localhost:8886/XXX/XXXX/XXX0102e.jsf"
scopes: "["Files.ReadWrite.All"]"
state: "http://localhost:8886_m1dII"

oauthConfig
clientId: "XXXX-XXXX-XXXX-XXXX"
endpoint: 1
origin: "http://localhost:8886"
redirectUri: "http://localhost:8886/XXX/XXXX/XXX0102e.jsf"
scopes: "["Files.ReadWrite.All"]"
state: "http://localhost:8886_m1dII"

@tvanh, can you change scopes from a string to an Array<string> in your configuration?
...
scopes: ["Files.ReadWrite.All"]
...

If this is the problem, TypeScript and *.d.ts files for our public interfaces will help avoid this going forward cc: @aditima.

I think the config is the result from parse
line690: redirectToOauthPage(JSON.parse(redirResults[PARAM_OAUTH_CONFIG]));
So I can't change the config file. Going to change the method in ur public interfaces will help imo.
Bests

line 764, by changing
config.scopes.map(function (s)
to
JSON.parse(config.scopes).map(function (s)
the bug gone

The site selection on the left is missing.
I got these following errors
odbonedrive-mini-cfc25a4d.js:1 Refused to get unsafe header "SPClientServiceRequestDuration"
h._onReadyStateChange @ odbonedrive-mini-cfc25a4d.js:1
(anonymous) @ odbonedrive-mini-cfc25a4d.js:1
d @ odbonedriveapp-mini-5f4648eb.js:1
13:46:47.522 odbonedrive-mini-cfc25a4d.js:1 Refused to get unsafe header "SPRequestGuid"
h._onReadyStateChange @ odbonedrive-mini-cfc25a4d.js:1
(anonymous) @ odbonedrive-mini-cfc25a4d.js:1
d @ odbonedriveapp-mini-5f4648eb.js:1
13:46:47.522 odbonedrive-mini-cfc25a4d.js:1 Refused to get unsafe header "SPRequestGuid"
r.getCorrelationId @ odbonedrive-mini-cfc25a4d.js:1
e @ odbonedrive-mini-cfc25a4d.js:1
h._onReadyStateChange @ odbonedrive-mini-cfc25a4d.js:1
(anonymous) @ odbonedrive-mini-cfc25a4d.js:1
d @ odbonedriveapp-mini-5f4648eb.js:1
May I get help?

@tvanh let's isolate this specific issue to the JSON.parse bug.
Please open a new separate issue referencing the absence of site selection (left nav). As part of that issue, please provide your File Picker configuration with clientId and tenantId omitted so that we can debug.

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


⚠ Jenkins Plugins Artifactory has degraded performance

In 0a3161a, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1233 ms

Resolved: Jenkins Plugins Artifactory performance has improved in af64549 after 47 minutes.


Add an ilog2 function.

Closes #167. An ilog2 function has been added and all the calls to (1- (integer-length x) has been replace.

Hi @notmgsk @stylewarning
Thanks for reviewing the code! I made the changes as suggested. Please let me know if you think more changes are necessary. Thanks!

Thanks!

Keep em coming. :)


Checkpoint syncing providers system

#1185  👈
feature/proof-of-stake

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

Join @lazynina and the rest of your teammates on  Graphite


The NETCore website failed to start process with commandline '"xxx.exe" ', ErrorCode = '0x800700d8'.

The NETCore website failed to start process with commandline '"xxx.exe" ', ErrorCode = '0x800700d8'.
stdout log is empty.
OS: Windows 7 enterprise, 32 bit
.NET Core 1.0.0 runtime
The website works fine in windows server 2008R2.
Already checked the known issue in https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/iis/troubleshoot?view=aspnetcore-2.1
but no lucky. BTW, the windows update was done by company LAN tool. Any suggestion/idea would be appreciated.
See below for web config:

Hello @sdluhaibing ...
That error code might be: ERROR_EXE_MACHINE_TYPE_MISMATCH. It's not a common error code and doesn't appear in the IIS Integration repo issues. It might be a platform architecture error [x64 (64-bit) vs. x86 (32-bit)]. If this is a self-contained app, try specifying a 32-bit runtime (for example, win7-x86) and see if that will run on your Win7 machine.
If that doesn't work out (or it's a framework-dependent app), I recommend you open this over on the aspnet/Home issues for the engineers to see:
https://github.com/aspnet/Home/issues
Give them as much info about your setup as you can. If you put up a repro project on GitHub for them to see the app (or a small test app that breaks the same way), that can be helpful for them.
Either ping me on the issue "cc @guardrex" ... or please come back and let me know what the cause of the problem was. That's not a typical error code for an ASP.NET Core app, and we might need to add an entry to the Common errors topic to cover it.

hello, already tried specify runtime win7-x86 for my app. but found that I can build with any runtime name like "testruntime" and also can publish with this runtime. Seems like dotnet did not verify the runtime when building. I am not sure if it is due to this.

This is a self-contained app? How are you publishing ... are you specifying the RID in the project file and in the publish command?
<PropertyGroup>
  <TargetFramework>netcoreapp2.1</TargetFramework>
  <RuntimeIdentifiers>win7-x86;win7-x64</RuntimeIdentifiers>
</PropertyGroup>

For your Win7 machine (x86) ...
dotnet publish --configuration Release --runtime win7-x86

Published output at: \bin\Release\netcoreapp2.1\win7-x86\.
For your WS2008R2 (if x64) ...
dotnet publish --configuration Release --runtime win7-x64

Published output at: \bin\Release\netcoreapp2.1\win7-x64\.

hello @guardrex,
No, because we need to install DotNetCore on server side. We build with command "dotnet build --configuration Release -r win7-x86" and then publish with "win7-x86" to local file folder in visual studio. At last copy to the server and finish the deployment.
See the below for the project.json:
`  "dependencies": {
"Domain.ViewModel": "1.0.0-*",
"Microsoft.AspNetCore.Diagnostics": "1.0.0",
"Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore": "1.0.0",
"Microsoft.AspNetCore.Mvc": "1.0.0",
"Microsoft.AspNetCore.Mvc.WebApiCompatShim": "1.0.0",
"Microsoft.AspNetCore.Server.IISIntegration": "1.0.0",
"Microsoft.AspNetCore.Server.Kestrel": "1.0.0",
"Microsoft.AspNetCore.StaticFiles": "1.0.0",
"Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0",
"Microsoft.Extensions.Configuration.FileExtensions": "1.0.0",
"Microsoft.Extensions.Configuration.Json": "1.0.0",
"Microsoft.Extensions.Configuration.UserSecrets": "1.0.0",
"Microsoft.Extensions.Logging": "1.0.0",
"Microsoft.Extensions.Logging.Console": "1.0.0",
"Microsoft.Extensions.Logging.Debug": "1.0.0",
"Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0"
},
"frameworks": {
"net451": {
"dependencies": {
"Domain.JIT": {
"target": "project"
}
},
"buildOptions": {
"emitEntryPoint": true,
"preserveCompilationContext": true,
"copyToOutput": "appsettings.json"
},
"runtimeOptions": {
"configProperties": {
"System.GC.Server": true
}
},
"runtimes": {
"win7-x64": {},
"win7-x86": {}
}`

Oh ... I see.
project.json/xproj-based .NET Core is no longer supported. You'll need to upgrade the project and use the latest SDK. See the migration topics over on the .NET Core repo: https://docs.microsoft.com/dotnet/core/migration/

hello @guardrex,
Thanks so much for your information. Checked the latest SDK and link you mentioned. Latest NetCore  2.1 can only work with visual studio 2017. Currently, we are still on visual studio 2015 and can not free to update to 2017 due to the license.  Any problem about the project opened in visual studio 2015 if we only upgrate NetCore to 2.1 and migrate project.json to .csproj?

The minimum VS version supported for the migration to MSBuild/csproj is 2017.
You might be able to get away with using VS2015 purely as an editor. Use the command-line tools to restore/build/publish/etc. However, I'm fairly certain that Intellisense won't work. It may also throw a bunch of warnings and errors. It really isn't supported, and I don't know how 2015 will behave. Even if can work purely as a file editor, it might be painful to use.
Under the circumstances if you can't use 2017 Community Edition, then I recommend Visual Studio Code.
https://code.visualstudio.com/


Refactor alerts and alert redux state

@vgrichina @kcole16
This is a completely refactored alerts, it will be much easier to use and will be handier to debug.
Main features:

Whole alert stuff and formLoader (now mainLoader) are now in the separate reducer. This is a part of refactoring Redux state.
Instead of requestStatus state that was stored action status for the last action, now there is actionStatus that stores status for all actions that ware dispatched. It has a form of the object instead of an object array, so it will be easier to handle. It also stores all error info, so we can easily clear globalAlerts without worrying about clearing error info.
GlobalAlert now handles multiple alerts, so if there is a need we can display several alerts.
I simplified default translations for alerts. Now it will be based on action type, so we don't have to add (but it's available as an option) alert translation ID in meta. Also, all alerts translations will be in one place.
There is a mainLoader that checks if any action is pending.
There is also a separate pending state for every action in actionStatus state.
requestStatus state is very similar, but now it's called localAlert.
Now we can decide to show globalAlert in three ways:

showAlert added to action meta
walletError
dispatchWithAlert - for a situation where we want to show an alert only for specific dispatch.

Now globalAlert handles a few additional options like:

onError and onSuccess, to show an alert for error or success only
console to show or hide the console error message
messageCodeHeader for custom alert header

There is also actionsPending function, that will check if action or actions are in pending state - so it will be easier to use than previously actionsPending state.

@vgrichina For now, new alerts are only implemented for a few actions. Please let me know what do you think - if you would like to add any additional features. I implement new alerts for the whole project then.

@vgrichina @ihaid
It's ready.
Big request, if we could handle it quickly, and don't merge any other changes before it (except for urgent stuff) - the problem is that it's touching many files and functionalities, and on every rebase I need to check again many things and it takes a lot of time.

Found only one issue that alert shows on background

Thanks @ihaid fixed it.

@marcinbodnar looks good overall and should be ok to merge into current release after resolving conflicts. See questions on alerts structure however.

@marcinbodnar looks good overall and should be ok to merge into current release after resolving conflicts. See questions on alerts structure however.


Ghostwriter doesn't run and returns SIGSEGV

Describe the bug
kdePackages.ghostwriter package is not working, because of missing dependencies, returns SIGSEGV when trying to run it.
Steps To Reproduce
Steps to reproduce the behavior:

Install kdePackages.ghostwriter
Run ghostwriter

Expected behavior
Ghostwriter application should run
Screenshots

Additional context
Add any other context about the problem here.
Notify maintainers

@NixOS/qt-kde
@K900
@NickCao
@SuperSandro2000
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.10.6, NixOS, 24.11 (Vicuna), 24.11.20240821.c374d94`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/j8pbrsb3nybdap3hhg9kw0ffqd4rlbx6-source`

Add a :+1: reaction to issues you find important.

Looks like it runs fine from master branch

But dependencies still missing


Added testcase for  with inner

From https://github.com/NLog/NLog/issues/1577

Codecov Report

Merging #3471 into dev will increase coverage by 1%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             dev   #3471      +/-   ##
========================================
+ Coverage     81%     82%      +1%
========================================
  Files        343     485     +142
  Lines      27834   38459   +10625
  Branches    3766    5598    +1832
========================================
+ Hits       22447   31560    +9113
- Misses      4295    5541    +1246
- Partials    1092    1358     +266


Setting openshift_logging_install_eventrouter=True in 3.9 and above.

/lgtm

$ sjb/push-update-automatic.sh
[INFO] Checking for existence of test_branch_origin_aggregated_logging_json_file...
[INFO] Checking for existence of test_branch_openshift_ansible_logging_json_file_310...
[INFO] Checking for existence of test_pull_request_openshift_ansible_logging...
[INFO] Checking for existence of test_branch_openshift_ansible_logging_json_file...
[INFO] Checking for existence of test_branch_openshift_ansible_logging...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_journald...
[INFO] Checking for existence of test_pull_request_openshift_ansible_logging_310...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_journald_39...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_json_file_39...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_json_file_310...
[INFO] Checking for existence of test_branch_origin_aggregated_logging_json_file_310...
[INFO] Checking for existence of test_branch_openshift_ansible_logging_310...
[INFO] Checking for existence of test_pull_request_openshift_ansible_logging_39...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_json_file...
[INFO] Checking for existence of test_pull_request_origin_aggregated_logging_journald_310...
[INFO] Updating test_pull_request_origin_aggregated_logging_journald_310...
[INFO] Updating test_pull_request_origin_aggregated_logging_journald_39...
[INFO] Updating test_branch_origin_aggregated_logging_json_file...
[INFO] Updating test_pull_request_origin_aggregated_logging_journald...
[INFO] Updating test_branch_openshift_ansible_logging_json_file...
[INFO] Updating test_pull_request_openshift_ansible_logging...
[INFO] Updating test_branch_openshift_ansible_logging...
[INFO] Updating test_branch_openshift_ansible_logging_310...
[INFO] Updating test_pull_request_origin_aggregated_logging_json_file_310...
[INFO] Updating test_branch_openshift_ansible_logging_json_file_310...
[INFO] Updating test_branch_origin_aggregated_logging_json_file_310...
[INFO] Updating test_pull_request_openshift_ansible_logging_39...
[INFO] Updating test_pull_request_openshift_ansible_logging_310...
[INFO] Updating test_pull_request_origin_aggregated_logging_json_file...
[INFO] Updating test_pull_request_origin_aggregated_logging_json_file_39...


Orientation changes not displayed in Change preview

Tested feature: Batch resize - Change preview

Tested on: Android 12, Snapdragon 665

Use case:
Load several images with different orientation for batch processing.
Change orientation of one or more images

Observed result:
The Changes made are not displayed in Change preview.
Clicking an already changed image will be showen in the original/unchanged orientaion.

Expected result:
In Change preview the changes made should be visible, e.g. show images with chnages or remark the images accordingly.

That's expected behavior, but i can add feature to not reset settled params when changing image preview

Thank You, sound good.
I think from an user perspective it sholud be be visible what changes were made before processing the batch.

Done :)

Thank's a lot!


build error "deriveFromJson"

I get this error when I try to build :
make
idris2 --build lsp.ipkg
1/67: Building Language.JSON.Interfaces (src/Language/JSON/Interfaces.idr)
2/67: Building Language.LSP.Message.Derive (src/Language/LSP/Message/Derive.idr)
Error: While processing right hand side of deriveToJSON. When unifying FC and Name.
Mismatch between: FC and Name.

src/Language/LSP/Message/Derive.idr:104:68--104:75
     |
 104 |     let funclaim = IClaim EmptyFC MW Export [Inline] (MkTy EmptyFC EmptyFC funName `(~(var name) -> JSON))
     |                                                                    ^^^^^^^

Error: While processing right hand side of deriveFromJSON. When unifying FC and Name.
Mismatch between: FC and Name.

src/Language/LSP/Message/Derive.idr:174:68--174:75
     |
 174 |     let funClaim = IClaim EmptyFC MW Export [Inline] (MkTy EmptyFC EmptyFC funName `(JSON -> Maybe ~(var name)))
     |                                                                    ^^^^^^^

make: *** [Makefile:3: build] Error 1

Here is my version of idris2:
Idris 2, version 0.3.0

If you can't build this with the Idris2 commit specified for the submodule, reopen the issue.


fix return type in mpl

Why the change of alias from mpl -> mp? Would revert to mpl since everyone uses it.

I/We normally abbreviate moscot.plotting with mpl. This causes issues when import both, e.g. in the tests. Do you prefer to change the abbr. of moscot.plotting instead?

I/We normally abbreviate moscot.plotting with mpl. This causes issues when import both, e.g. in the tests. Do you prefer to change the abbr. of moscot.plotting instead?

Would suggest using import moscot as msc; msc.pl, but am curious to hear from others (@giovp @zoepiran).

I/We normally abbreviate moscot.plotting with mpl. This causes issues when import both, e.g. in the tests. Do you prefer to change the abbr. of moscot.plotting instead?

Would suggest using import moscot as msc; msc.pl, but am curious to hear from others (@giovp @zoepiran).

I like import moscot as msc; msc.pl ! :)


Recording article page numbers

I'm a little confused by the meaning of the references.pages field in CFF. The spec defines this as 'The number of pages of the work' and requires an Integer. For articles, it is important to specify page numbers for the article. This requires ranges (e.g. 10--20 in bibtex) and need not be numeric (e.g. one of my papers is e1006791). How should these be specified in CFF? Should the section: key be used instead of pages:?

Looking at the documentation, https://zenodo.org/record/1405679#.XV0DgfxS9tc, I think this key was meant to record the total number of pages of e.g. a book. I believe sometimes you see that denoted in references as pp.
I'd say you are looking for the start and end keys. See table 3 on page 7 of the url above.

I'd say you are looking for the start and end keys. See table 3 on page 7 of the url above.

Correct.

Thanks for clarifying!
Maybe it would be worth including this in the 'article' reference example. The current example is from peerJ, which unusually doesn't use page numbers, so it was hard to find the start and end keys.


Can't assign enum properties from ActionController::Parameters in Rails 5

Can't assign enum properties from ActionController::Parameters in Rails 5:
irb(main):005:0> Desk.new(ActionController::Parameters.new({title: "some", type: 'timeline'}))
=> #<Desk uuid: nil, created_at: nil, title: "some", type: :standard, updated_at: nil>

(type is ignored, default value is used instead)
The issue seems to be related to rails 5:
In fact ActionController::Parameters is no longer a child of Hash.
  The index / constraint functionality has been removed and replaced by warnings / errors, and now I need to get all of the tests working that expect the indexes / constraints to be created automatically

(all that to say I can look at this afterward if you need help ;)


GPU initialization action does not create GPU metrics

While testing I found that the GPU initialization action does not actually create the stackdriver monitoring metrics.
I had to create the metrics externally by running the following code in cloud shell taken from here
git clone https://github.com/GoogleCloudPlatform/ml-on-gcp.git
cd ml-on-gcp/dlvm/gcp-gpu-utilization-metrics
pip install -r ./requirements.txt
python create_gpu_metrics.py

it's not clear if the user should be creating the metrics or this should be done by the init action?

To create GPU metrics in init action you need to set install-gpu-agent=true metadata value when creating a cluster: https://github.com/GoogleCloudDataproc/initialization-actions/tree/master/gpu#supported-metadata-parameters


Textareas have a blue background when focused

Here's what it looks like to me when I create a new issue or write a comment:

This started happening a few days ago. Previously the background was dark gray, which is much preferable.
Edit: I have the Refined Github extension enabled but I have verified that the problem still occurs with the extension disabled. I'm in Chrome 62.0.3202.62

Hi @IceCreamYou!
Yep, sorry, I pushed that commit and fixed in it within 30 minutes. Use the force update to grab the lastest style.

Oh, I had tried to update the userscript unsuccessfully, but I just read more documentation and found the "force update styles" in the settings and that worked. Thanks!


MINIFICPP-700: Add MSI Support via CPACK

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

reviewing

hey @phrocker, just wanted to check on the status of this.  are there remaining changes you wanted to do or is it good for another verification?

@apiri Should be good. I can't get cpack with install to work on appveyor but I'm not the only one. Will continue to take a look for a subsequent PR.

@phrocker Sounds good.  Sorry to miss this before the big one got merged.  Would you mind rebasing this and then I can take care of getting the merge on.

@apiri the test failure is due to https://issues.apache.org/jira/browse/MINIFICPP-743 and something I'm looking to solve separately.


Filled area support

Overview
This PR add a new Canvas.area function to support the creation of filled area plots with Datashader.
Examples
Imports and dataset
import pandas as pd  # doctest: +SKIP
import numpy as np
import datashader as ds
from datashader import Canvas
import datashader.transfer_functions as tf
cvs = Canvas()
df = pd.DataFrame({
   'A1': [1, 1.5, 2, 2.5, 3, 4],
   'B1': [10, 12, 11, 14, 13, 15],
   'B2': [2, 9, 10, 7, 8, 12]}, dtype='float64')

Plot 'A1' vs 'B1' and 'B2' as lines
agg = cvs.line(df, x='A1', y=['B1', 'B2'], axis=0)
tf.spread(tf.shade(agg))

Plot filled area from 'B1' to y=0.
agg = cvs.area(df, x='A1', y='B1')
tf.shade(agg)

Plot filled area from 'B1' to 'B2' by specifying a two-element list as the y argument to Canvas.area.
agg = cvs.area(df, x='A1', y=['B1', 'B2'])
tf.shade(agg)

Documentation
I added a new area plot section to the Timeseries user guide. See the user guide for full code:
cvs = ds.Canvas(plot_height=300, plot_width=900)
agg = cvs.area(df, 'Time', 'a')
img = tf.shade(agg)
img

Performance
I haven't done extensive benchmarking, but in all of the cases I've tried the render time has been within a factor of two of the time to render the line by itself.

Seems some tornado version incompatibility may have cropped up in the tests.
looks like nbsmoke/nbconvert is raising an error:
AttributeError: module 'tornado.web' has no attribute 'asynchronous'

@jonmmease I restarted the failing build and it seems to be passing now. Maybe some other dependency ended up pinning tornado for us? At any rate, I'll now have a look at this PR...

Just a quick note to say I've tried the PR and it seems to be working as expected! I've tested whether this could be used to draw axis-aligned filled boxes for a project I am working on and it behaves as I would expect.

One issue I have spotted - replacing .line( with .area( everywhere in '3_Timeseries.ipynb' of the user guide, you get this error at the end:

I suppose the axis argument isn't yet supported by area?

@jonmmease , it would be great to have the axis support as well, to make it simple to switch from line to area and back as needed.
One possible feature request: If we're showing the area between two curves a and b, would it be possible to somehow indicate the regions where a was higher than b differently from the regions where b is higher than a?  E.g. the first case could aggregate positively, and the second negatively?  I'm trying to think about the cases where people use area shading between curves in general, where usually it's to highlight differences between the curves, and it seems like the sign of that difference is crucially important in that case.

Just so we can have a pretty plot demonstrating areas :-)

Thanks for the feedback and trying this out @jlstevens
Yes, that's right that the axis argument isn't implemented.  Other that time, the main reason I didn't pursue it was that I wasn't sure what it should mean.  For the line case with axis=1, each row of the dataframe represents a single distinct line.  For area there is the single line case (fill to zero) and the two line case (fill between lines), but I wasn't sure how to think about the case with a separate line per row in a dataframe.  They could be treated as a bunch of instances of the single line case, or there could be some kind of option to toggle two line behavior where every other row would be filled between.
Happy to talk more about it if someone has ideas for how the axis=1 API should work.  My initial impression is that it could get messy and I didn't know what use cases there might be for it.
Regarding aggregation, perhaps we could define a new comparison_count aggregator that would accept two field names. The aggregator would return +1 if col1 > col2 and 0 of col1 == col2 and -1 if col1 < col2. This aggregator could be used to display a change in filled area plot when the lines cross, like @jbednar suggested, but it could also be used in other glyphs as well.  Would that make sense?

I would expect the axis argument support to work just as for cvs.line, i.e. only really being relevant for the case of filling from the x axis to the line.  I don't think there's a meaningful extension of the between-the-lines filling for that case, and doing every other line sounds quite arbitrary, so I'd be very happy if it simply did all that cvs.line does, plus separate support for filling between two lines (the only bit of the API or behavior that wouldn't match cvs.line, and strictly being an extension/superset rather than a difference.  The goal is that any user of cvs.line should be able to switch to cvs.area to try it out without encountering any issues, and I think that's achievable here.
The proposed aggregator sounds great!

The goal is that any user of cvs.line should be able to switch to cvs.area to try it out without encountering any issues, and I think that's achievable here.

Sounds good!  I'll see what makes sense in terms of the fill-between-lines API as a get a bit further in refactoring this.

Looks promising; tag me and @jlstevens when it's ready for re-review!

Looks promising; tag me and @jlstevens when it's ready for re-review!

Will do!  Should be soon now 🙂

@jbednar @jlstevens Alright, this is ready for another look!

Looks great!  I really appreciate your careful docstrings, comments, and test cases, and I was able to use it anywhere I expected cvs.line to work, without any problems.  The underlying numerical code makes my brain go numb, so I can't comment on that, but it certainly seems to behave properly.
Does the y_stack argument work for only a single subtracting line, or can it also be used in the multiple-column or multiple-row cases to provide a baseline for each such line?  Either is fine, I just couldn't tell from the 3_Timeseries.ipynb notebook.
Also, were you able to look into an implementation of comparison_count?  That shouldn't hold up this PR being merged; just wondering.
Unless @jlstevens can spot any problems, I'm happy to merge this!

Thanks for taking a look @jbednar,

Does the y_stack argument work for only a single subtracting line, or can it also be used in the multiple-column or multiple-row cases to provide a baseline for each such line?

Yes, y_stack can be used in every case (single, multi, ragged, axis=0, axis=1, etc.).  It just needs to have the same form as y (i.e. both strings, both lists of strings with the same length, both ragged, both numpy arrays with the same length, etc.).  I didn't make examples of all of them because that's another factor of 2 and I'm not sure which will actually be useful in practice 🙂

Also, were you able to look into an implementation of comparison_count? That shouldn't hold up this PR being merged; just wondering.

No, not yet. But I take a look soon.

Sounds great!

Looks good to me! My only observation here is that glyphs.pyis getting rather long and at some point it might be good to split up...


Unhandled zlib error event when downloading 6.0.0

Here's the error:
~/Work $ esvm
Nov 20, 2017 12:54 PM INFO cluster Downloading & Installing 6.0.0
events.js:182
      throw er; // Unhandled 'error' event
      ^

Error: incorrect header check
    at Gunzip.zlibOnError (zlib.js:152:15)

Does this happen regularly? Maybe there was corruption over the wire. We aren't currently doing any sort of checksum verification before un-archiving so this could have been a transient failure.

This is the first time I've hit this error in the 6 or so months that I've been using the library. To note the directory was created in my esvm folder which I had to delete in order to try again to reproduce. When I mindlessly run the command again tomorrow I'll report back to let you know if it's still happening.

Hey, so I was able to repro on my Mac at work and confirmed that it seems to be only affecting 6.0.0. All other versions seem to install fine.
To reproduce you should be able to remove ~/.esvm/6.0.0 and rerun the esvm command to repro.
Unless this software just hates me specifically, which very well could be the case :)

Okay, I should have just tried it, I'm seeing the same thing. I also double checked and the 6.0 snapshot downloads and extracts fine when accessed directly... Further investigation is underway...

Alright, so the issue was with esvm-props.kibana.rocks, it was pointing to the wrong 6.0 release url.
I fixed the issue there so esvm 6.0 should work now.
You can also include esvm 6.0 --fresh and it will disregard the already downloaded (and invalid) install and force a re-download. You might have a bad version of the builds manifest locally, and once elastic/libesvm#50 is merged esvm 6.0 --fresh will also clear that cache and prevent stale caches from being an issue (in the future).

Thanks man. That kibana server strikes again, eh? It's making me question their choice in TLD! Haha!


Add Phi (AKA φ)

Adding a language

[x] The code displays "Hello World"
[x] I have updated the readme to include the new language
[x] I have incremented the language count in the readme
[x] I have no association with the language

Link to programming language: https://assetwolf.com/learn/phi

Only Phi should be used.


Irrelevant Welcome message for mobile apps

@zulipbot claim

@pulkonet If this is a mobile app issue, I think it's a better idea to open this in the zulip/zulip-mobile repository?

I guess you understood that it is not mobile app issue. Shall I start working on this? @synicalsyntax

No, I accidentally pressed the wrong button (Close issue instead of Comment). Your issue title makes it seem that this bug is limited to mobile apps only; if so, I'd repost this issue in zulip/zulip-mobile and close this one instead.
Do you have any ways to reproduce this issue on web or mobile?

Yes, you can actually create a new account to see the welcome message by
welcome bot. But without that I don't see any way to reproduce this.
On Dec 31, 2017 04:31, "Cynthia Lin"<EMAIL_ADDRESS>wrote:

No, I accidentally pressed the wrong button (Close issue instead of
Comment). Your issue title makes it seem that this bug is limited to mobile
apps only; if so, I'd repost this issue in zulip/zulip-mobile and close
this one instead.
Do you have any ways to reproduce this issue on web or mobile?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/7955#issuecomment-354573449, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AX05llRskNzhptOTC94CNkLLuX0xQQkFks5tFsDRgaJpZM4RPsAu
.

All new users are supposed to receive a new message from the Welcome Bot. Were you a returning user?

The issue is not with the message displaying by the Welcome Bot. Issue is with the content it is showing as the content seems to be for web not for app (This was viewed on Android app still it was prompting to download the app).
Also, pressing '?' or 'r' on mobile devices doesn't make sense.

Okay, so the issue isn't an welcome message inappropriately appearing in mobile apps, but the welcome message contains instructions that aren't applicable to mobile apps. Can you change your issue title and description to match that?
Pinging @rishig to review this also.

@rishig I don't think this is something that makes sense to change, so I'm closing this.  Fundamentally, a message is fixed content, and the recommendation is still sorta useful regardless of which app you start with (since it suggests desktop and mobile).

The plan is to move most of the information from the PM into hotspots and other places anyway; it's currently too much text for people to read. The current PM is kind of a placeholder for that project.
Thanks @pulkonet for reporting this though! It's a good point, and we should be cognizant of people being onboarded via mobile when thinking about onboarding.

I think each organization should have their own welcome message. The settings could have a field for the welcome message (regardless of media) and each organization would give the first instructions there.
We don't want people to practice, for instance. It is easy to reply, etc.
On the other hand, we need to give instructions on what to put on the topic. Of course, we just need to create a second welcome bot for the specific uses.


Add heatmap "averaging"

The PINDA has some variance reading to reading. Would be nice to add a function to do the G80 multiple times and average the results. This will allow us to perform even more fine tuned leveling of the bed using wave springs.

With Arksine's findings regarding PINDA accuracy, I don't believe this is needed any longer.


Notero not working when window closed on MacOS

Notero not working when I close and restart the Zotero window
I'm using Zotero on m1 MacBook Air, and notero seems to stop working every time I close the Zotero window(i.e. pressing Cmd+W and reopen the Zotero window).
I can restart notero by completely closing Zotero(i.e. Cmd+Q), and restart. So I'm wondering whether other people also have this issue, is there a way to fix it?
btw, Thanks for this amazing plugin, it helped me a lot with my paper reading!

Hi @liu-jinyu, thanks for reporting this!
I've run into this same issue, and I haven't dug into it enough to figure out what a solution might be. I believe it's related to how Zotero loads plugins. I think the Zotero team is changing the process for loading plugins, and that may require me to rework how Notero loads anyways. Hopefully that will solve this issue.

I'm having the same issue on an M1 MacBook.
Also massive thank you from me for creating this plug-in. It's so insanely useful!

same on intel MacBook pro 2019. issue fixed by restarting the Zotero

same issue. my link/connection stops working after a while. I tried to restarting zotero but it did not work for me. I am running notero on a m2 macbook pro. I am wondering if anyone has the same issue or has a solution. Great plug-in!

same issue. my link/connection stops working after a while. I tried to restarting zotero but it did not work for me. I am running notero on a m2 macbook pro. I am wondering if anyone has the same issue or has a solution. Great plug-in!

I have the same issue! I essentially have to disable Notero, restart Zotero, enable Notero, restart Zotero and change a tag every time. Thanks for the great plugin and I would love a fix for this issue, if someone has one.

Looks like we'll need to handle re-initialization. See discussion with Dan Stillman:

Is there a recommended approach for re-initializing things when the main Zotero window is closed and re-opened (on macOS)?
For example, when my plugin starts up with Zotero, the startup function is called, and I add a menu item to the tools menu. If the Zotero window is closed and re-opened, the startup function is not called again, and thus my menu item is no longer in the tools menu.
Do I need to use Services.wm.addListener from my startup function and listen for whenever the main Zotero window is opened again?

For now, yes, though we could consider adding an easier hook for this in Zotero 7. (Make It Red doesn't deal with this either currently and has the same problem.)

The Zotero team has added better support for this in Zotero 7. We'll need to update Notero to take advantage of this.

Hi all! About a week ago my my collections stopped syncing to Notion - I just noticed and tried a number of the fixes mentioned in issues here (disabling and re-enabling, totally removing and re-adding the add-on, turning off other add-ons like the LibreOffice integration) and nothing is bringing back "Sync to Notion" in the context menus! However, adding a "notion" tag seems to trigger a sync. Looking forward to the auto-sync fix!

Hi everyone, I just released a fix for this issue in v0.4.13!
Please give it a try and let me know if you're still having issues with Notero not working when you close and reopen the Zotero window.

Fixed! Thank you!!
Notero is an absolute lifesaver - thank you so much for all you do.


[BUG] Virus in Windows Installer .exe

Describe the bug
Windows Defender detects a virus when downloading the Installer .exe (v1.1.10). Trojan:Win32/Wacatac.B!ml
To Reproduce
Download WirepodInstaller-v1.1.10.exe (better not...) on a machine with up-to-date Windows 10 or 11
result:
File removed by Windows Defender
Expected behavior
no virus in installer...
Device you are running wire-pod on (please complete the following information):
Windows 10 or 11
Additional context
could maybe come from the machine the .exe was created on

It's a false positive.


added union testnet

union uses, tendermint/PrivKeyBn254 for its key type.

union uses, tendermint/PrivKeyBn254 for its key type.

Do we need to add bn254 to the chain schema to the key_algos enum, then?


Vote Links GUI is unstable

Reported by piedrops, UForced#9799
The vote link GUI should be recreated in a more stable way.
The GUI is also found complicated.

I came up with a clean solution to tackle this problem. Building is nearly finished!

10 hours of building, but I have a solution ready

+1 from Piedrops for the solution


Add TypeScript type definitions

This depends on micromark/micromark-extension-gfm-strikethrough#2

@ChristianMurphy I think that’s the “This depends on micromark/micromark-extension-gfm-strikethrough#2” part

I'm a bit ahead of time with this one 😅
I just really need to use this package in a TypeScript project.


Implemented Converter

Converter implemented via regex and matching.  Created regex to identify twitter link.  Regex will accept either links containing twitter.com or x.com depending on whether you are copying from desktop or mobile.
ran python fmt.py already.  Changes to YAML include adding:
# Converter Settings
ENABLE_X_CONVERTER: true

lgtm


some proposal about  tf.metrics

Describe the problem
I want to know true-positive,ture-nagetive,false-positive,false-nagetive,recall,precision for each class.
Although I can implement it at this version(1.40), it need lots of code. I mean the functions don't
support the multi-class well, it just for the 2 class condition. I think if I can implement them with few
code, it looks more elegant.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

sorry, I think the information is for BUG. So I didn't write them down.  And I have updated them.

Yup, the robot sometime is a little annoying. Sorry for that.

Closing this issue, as TruePositives, TrueNegatives, FalsePositives, and FalseNegatives have been implemented for tf.metrics. Thanks! 😊


Regular Expression Matching (Hard)

Want to add solution of this leetcode question
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

@Ayush7614 please assign this to me under LGM SOC.


LocalMaster WaitForNotification cause master to hang

Could not understand why here we need to WaitForNotification after timeout?
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/distributed_runtime/local_master.cc#L39
This wait for notification could cause master or worker could not exit normally, the later exit one will hang at this step, because nobody will give them a message.
The comment says that the call has borrowed pointers to request and response messages, I can not understand the call means what? which object and which class? and where it borrows its pointers?

@mrry

I am trying to remove the line, but the program cored. I set operation_timeout_in_ms in tf.ConfigProto to 20*1000.

@chengmengli06 I apologize, but I am having a hard time understanding what the problem is, where the problem is, and what version it affects. Can you please provide a small repro describing your issue?
Also provide following information. Thanks!
System information

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

Why do not mark this as cancelled, and remove the registered callback function? @mrry

That's not how the gRPC-style methods work: the callback std::function is no longer accessible to the caller, so the caller must wait for it to be called (and can use call_options->StartCancel() to make that happen sooner).


Accessibility - tabbing through buttons and panes

It should be possible to tab through all the IDE buttons, and any other things that can take user input - currently the Code pane, and the Console.  On the second one, see alsoL: #834

TODO

Remove recognition of Escape from field. Stef to then make it work at global level meaning: 'Escape from the Code pane and focus on the first Button in the IDE'. When implemented, check that if you tab back into the Code pane, you go back to the Frame or Field that is selected.

Update CSS so that buttons indicate when they have focus. Ditto the Console

Make the input cursor more visible.

Documentation of Esc

I need to tweak the styling for buttons.
To indicate focus, instead of changing the background, change the colour of the text in the button to blue


Chalice integration does not report exception in authorizer

How do you use Sentry?
Sentry Saas (sentry.io)
Version
1.5.6
Steps to Reproduce
Chalice integration does not report exceptions raised during an authorizer. I digged (quite a bit I must admit) but Chalice lifecycle around authorizers are pretty obscure.
import sentry_sdk

from boto3.dynamodb.conditions import Key
from chalice import Chalice
from sentry_sdk.integrations.chalice import ChaliceIntegration

sentry_sdk.init(
    dsn="A_PROPER_DSN",
    integrations=[ChaliceIntegration()],
    debug=True,
)

app = Chalice(app_name="the_app")

@app.authorizer()
def custom_authorizer(auth_request):
    raise Exception("Authorizer exception")

@app.route("/boom", authorizer=custom_authorizer)
def boom():
    return Response(body="")

Expected Result
I'd expect such an error to be tracked by Sentry.
Actual Result
No issue is reported

Hey @b4stien !
Thanks for writting in. I am currently working on something else that uses a Chalice project, so maybe I can try to reproduce this today!

Let me know if I can help somehow. My investigations stopped around https://github.com/aws/chalice/blob/master/chalice/app.py#L1253-L1298 which is not instrumented by the chalice integration afaik.

Could you please test this:
If your authorizer does not raise an Exception but your boom() method does raise an Exception. Is this exception ending up in Sentry?
This is just to see if the authorizer decorator somehow swallows all exceptions.

If your authorizer does not raise an Exception but your boom() method does raise an Exception. Is this exception ending up in Sentry?

From memory it does end up in Sentry, I'll re-test asap.
The "Authorizer" part in Chalice (and AWS Lambda subsequently) is very specific, it's a different AWS Lambda called internally by AWS API Gateway. As far as I understood the code, Sentry's Chalice Integration does not instrument this "other AWS Lambda", hence the no report. But as soon as we go back to the "main" AWS Lambda (in the endpoint/route itself) the integration works.

Ok, thanks for the info. So it is not as trivial as I thought. I am putting this in our low prio backlog and we will see.

Hey this is happening for me too.

For the record we ended up phasing out Chalice completely and wrote a pure WSGI app (Flask), wrapped with apig-wsgi and deployed with pulumi.
Imo Chalice should stop being recommanded, it's riddled with bugs, half documented at best and not maintened anymore.


fixes #15342 - add util/types to node target

fixes #15342
What kind of change does this PR introduce?
Adding util/types to nnode target - bug fix

Thanks


⚠ Bofrost.de has degraded performance

In 908d8fd, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1659 ms

Resolved: Bofrost.de performance has improved in 1107e7d after 29 minutes.


USDecouple from UniversalStorage2

I don't get it, the patching is properly, it's even inside the ModuleManager.ConfigCache, but ingame in VAB it's gone.
Could it be that codewise any part that does not contain ModuleDecouple is ignored and the ModuleDecouplerShroud is dynymically removed?
excerpt from ModuleManager.ConfigCache:
UrlConfig
{
	parentUrl = UniversalStorage2/Parts/Control/ACDLarge.cfg
	PART
	{
		module = Part
		author = Daishi, DMagic, and Paul Kingtiger
		manufacturer = New Horizons & Nox Industrial
		subcategory = 0
		title = Universal Storage: Wheel-O-Decoutron ACD (2.5m)
		category = Coupling
		tags = cck-universal-storage Universal Storage Decoupler Stack Separator Reaction Wheel
		description = The Attitude Control Decoupler combines a stack separator and in-line stabilizer to provide staging and reaction wheel systems in a single, low weight package with an access void for crew transfer.  Available in Apollo Silver, Gemini Black, Restock and Tantares color schemes.
		cost = 300
		mass = 0.16
		fuelCrossFeed = True
		maxTemp = 2000
		bulkheadProfiles = size2
		crashTolerance = 6
		TechRequired = generalConstruction
		entryCost = 2200
		stagingIcon = DECOUPLER_VERT
		stageOffset = 1
		childStageOffset = 1
		sound_decoupler_fire = decouple
		name = USACDLarge
		rescaleFactor = 1
		attachRules = 1,0,1,0,1
		angularDrag = 1
		dragModelType = default
		maximum_drag = 0.2
		minimum_drag = 0.2
		MODULE
		{
			name = ModuleConnectedLivingSpace
			passable = true
		}
		MODULE
		{
			name = USDecouple
			ejectionForce = 150
			explosiveNodeID = InnerNode
			menuName = Decouple
			DecoupleAnimationName = Take 001
			DecoupleTime = 0.08
			DecoupleEVA = true
			AnimationSpeed = 1
		}
		MODULE
		{
			name = ModuleToggleCrossfeed
			crossfeedStatus = false
			toggleEditor = true
			toggleFlight = true
		}
		MODULE
		{
			name = ModuleReactionWheel
			PitchTorque = 1.5
			YawTorque = 1.5
			RollTorque = 1.5
			RESOURCE
			{
				name = ElectricCharge
				rate = 0.3
			}
		}
		MODULE
		{
			name = ModuleTestSubject
			useStaging = True
			useEvent = False
			situationMask = 127
			CONSTRAINT
			{
				type = REPEATABILITY
				value = ALWAYS
				prestige = Trivial
			}
			CONSTRAINT
			{
				type = REPEATABILITY
				value = BODYANDSITUATION
				prestige = Significant
			}
			CONSTRAINT
			{
				type = REPEATABILITY
				value = ONCEPERPART
				prestige = Exceptional
			}
			CONSTRAINT
			{
				type = ALTITUDE
				test = GT
				value = 0
				situationMask = 8
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = GT
				value = 4000
				prestige = Trivial
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = LT
				value = 8000
				prestige = Trivial
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = GT
				value = 2000
				prestige = Significant
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = LT
				value = 4000
				prestige = Significant
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = GT
				value = 1000
				prestige = Exceptional
			}
			CONSTRAINT
			{
				type = ALTITUDEENV
				test = LT
				value = 2000
				prestige = Exceptional
			}
			CONSTRAINT
			{
				type = SPEED
				test = LT
				value = 2000
				situationMask = 8
			}
			CONSTRAINT
			{
				type = ALTITUDE
				test = LT
				value = 300000
				situationMask = 16
				body = _NotSun
			}
			CONSTRAINT
			{
				type = ALTITUDE
				test = LT
				value = 600000
				situationMask = 32
				body = _NotSun
			}
		}
		DRAG_CUBE
		{
		}
		MODEL
		{
			model = UniversalStorage2/Parts/Control/ACDLarge
			scale = 1, 1, 1
		}
		MODULE
		{
			name = USSwitchControl
			SwitchID = 0
			ButtonName = Appearance
			ModuleDisplayName = Appearance
			CurrentVariantTitle = Appearance
			ObjectNames = Apollo Silver;Gemini Black;Restock White;Tantares Olive;Tantares White;Tantares Grey;Tantares Black
			ShowPreviousButton = False
			VariantColors = #565656,#565656;#1f1d1b,#1f1d1b;#eeeeee,#565656;#eeeeee,#565656;#eeeeee,#565656;#eeeeee,#565656;#eeeeee,#565656
		}
		MODULE
		{
			name = USMeshSwitch
			SwitchID = 0
			MeshTransforms = ACDLargeApollo;ACDLargeGemini;ACDLargePorkjet;ACDLargeOlive;ACDLargeWhite;ACDLargeGrey;ACDLargeBlack
			AffectColliders = False
		}
		NODE
		{
			name = OuterNode
			transform = OuterNode
			size = 1
			method = FIXED_JOINT
		}
		NODE
		{
			name = InnerNode
			transform = InnerNode
			size = 1
			method = FIXED_JOINT
		}
		MODULE
		{
			name = ModuleDecouplerShroud
			shroudEnabled = false
			autoDetectSize = true
			textureName = Default
			radialSnapMargin = .025
			antiZFightSizeIncrease = 0.01
			outerEdgeLoops = 20
			topEdgeLoops = 8
			editorMinAlpha = .15
			jettisonVelocity = 2
			collisionEnabled = false
			defaultBotWidth = 2.5
			defaultVertOffset = 0
			nSides = 24
			defaultThickness = 0.1
			bottomEdgeSize = .05
			topBevelSize = .04
		}
		MODULE
		{
			name = tjs_DecouplerLight
			lightMultiplier = 1.0
			lightRange = 15.0
			lightRed = 1
			lightGreen = 0.88
			lightBlue = 0.68
			lightDuration = 0.1
		}
		MODULE
		{
			name = Reliability
			type = ModuleReactionWheel
			title = Reaction Wheel
			redundancy = Attitude Control
			repair = true
			mtbf = 36288000
			extra_cost = 2.5
			extra_mass = 1.0
			rated_radiation = 0.15
			radiation_decay_rate = 2
		}
		MODULE
		{
			name = ShipEffectsCollisions
			num1 = 5.4
			num2 = 2.7
			num3 = 1.35
			num4 = 0.135
			CollisionEnter
			{
				SOUNDLAYER
				{
					name = Hull_Impact-Low
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-Low-1
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-Low-2
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-Low-3
					loop = false
					spread = 0.1
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 0.135 0.0
					volume = 1.35 1.0
					volume = 2.7 0.0
					pitch = 0.0 0.8
					pitch = 0.135 0.95
					pitch = 2.7 1.05
					massToVolume = 0.0 0.5
					massToVolume = 15.0 0.8
					massToVolume = 30.0 1.0
					massToPitch = 0.0 1.2
					massToPitch = 15.0 1.0
					massToPitch = 30.0 0.8
				}
				SOUNDLAYER
				{
					name = Hull_Impact-High
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-High-1
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-High-2
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Hull_Impact-High-3
					loop = false
					spread = 0.1
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 2.7 0.0
					volume = 5.4 1.0
					pitch = 0.0 0.8
					pitch = 2.7 0.95
					pitch = 5.4 1.05
					massToVolume = 0.0 0.5
					massToVolume = 15.0 0.8
					massToVolume = 30.0 1.0
					massToPitch = 0.0 1.2
					massToPitch = 15.0 1.0
					massToPitch = 30.0 0.8
				}
				SOUNDLAYER
				{
					name = Ground_Impact-Low
					data = concrete-dirt
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-Low-1
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-Low-2
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-Low-3
					loop = false
					spread = 0.25
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 0.1 0.0
					volume = 15 1.0
					volume = 30 0.0
					pitch = 0.0 0.8
					pitch = 0.1 0.8
					pitch = 15 1.2
				}
				SOUNDLAYER
				{
					name = Ground_Impact-High
					data = concrete-dirt
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-High-1
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-High-2
					audioClip = RocketSoundEnhancement/Sounds/Physics/Impacts/Ground_Impact-High-3
					loop = false
					spread = 0.25
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 15 0.0
					volume = 30 1.0
					pitch = 0.0 0.8
					pitch = 15 0.8
					pitch = 30 1.2
				}
			}
			CollisionStay
			{
				SOUNDLAYER
				{
					name = Hull_CollisionScrape-concrete
					audioClip = RocketSoundEnhancement/Sounds/Physics/Hull_Scrape-1
					data = concrete
					loop = true
					loopAtRandom = true
					spread = 0.12
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 1.0 0.0
					volume = 15.0 0.8
					volume = 30.0 1.0
					massToVolume = 0.0 0.5
					massToVolume = 25 0.8
					massToVolume = 50 1.0
					massToPitch = 0.0 1.2
					massToPitch = 25 1.0
					massToPitch = 50 0.8
				}
				SOUNDLAYER
				{
					name = Hull_CollisionScrape-dirt
					audioClip = RocketSoundEnhancement/Sounds/Physics/Hull_Scrape-2
					data = dirt
					loop = true
					loopAtRandom = true
					spread = 0.12
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 1.0 0.0
					volume = 15.0 0.8
					volume = 30.0 1.0
					massToVolume = 0.0 0.5
					massToVolume = 25 0.8
					massToVolume = 50 1.0
					massToPitch = 0.0 1.2
					massToPitch = 25 1.0
					massToPitch = 50 0.8
				}
				SOUNDLAYER
				{
					name = Hull_CollisionScrape-Metal
					audioClip = RocketSoundEnhancement/Sounds/Physics/Hull_Scrape-3
					data = concrete-vessel
					loop = true
					loopAtRandom = true
					spread = 0.12
					channel = ShipBoth
					volume = 0.0 0.0
					volume = 1.0 0.0
					volume = 30.0 1.0
					massToVolume = 0.0 0.5
					massToVolume = 25 0.8
					massToVolume = 50 1.0
					massToPitch = 0.0 1.1
					massToPitch = 25 1.0
					massToPitch = 50 0.9
				}
			}
		}
		MODULE
		{
			name = ModuleCargoPart
			packedVolume = 2083
			KSP_PartVolume = true
		}
		MODULE
		{
			name = ModulePartInfo
			originalPartName = USACDLarge
		}
		MODULE
		{
			name = ChillingOut
			active = True
		}
	}
}

This is because the mod looks specifically for ModuleDecouple.
I'll probably adopt this mod and fix this


Mean IoU is 0.643 after 26400 iterations and initialized from provided model.

root@c8393a415c4e:/opt/data/penggao/deep_segmentation/tensorflow-deeplab-resnet# CUDA_VISIBLE_DEVICES=6 python evaluate.py --restore_from ./snapshots_no_index/model.ckpt-26400
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so locally
WARNING:tensorflow:From evaluate.py:84 in main.: all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Please use tf.global_variables instead.
I tensorflow/core/common_runtime/gpu/gpu_device.cc:885] Found device 0 with properties:
name: Tesla P100-SXM2-16GB
major: 6 minor: 0 memoryClockRate (GHz) 1.4805
pciBusID 0000:89:00.0
Total memory: 15.90GiB
Free memory: 13.57GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:906] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:916] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Tesla P100-SXM2-16GB, pci bus id: 0000:89:00.0)
WARNING:tensorflow:From evaluate.py:99 in main.: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use tf.global_variables_initializer instead.
WARNING:tensorflow:From evaluate.py:102 in main.: initialize_local_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use tf.local_variables_initializer instead.
 There is a big gap between the paper.

After 29600 iterations, mIoU is 0.653

Is it after running 'fine_tune'?

0.653 is from deeplab_init_model (30000)
while 0.789 is from deeplab_pretrained_model (4000)

I haven't really followed the optimisation methods outlined in the paper (e.g., as I remember they use SGD with a particular schedule for the learning rate vs. Adam in this repository), so first you can try to consult the paper on the subject of the training hyperparameters. I think this should give a performance boost.

@gaopeng-eugene did you train your model from the initialization weights coming from (deeplab_resnet_init.ckpt, given in the instructions)? I tried to use that and also converting the init.caffemodel myself as an initialization model, but in both cases my mIoU is around 53% using the default hyperparameters in this repo. Even after 50k iterations the mIoU is not improving.
Did you have to make any changes in the training strategy to get to 65.3%? Also, have you tried tweaking the hyperparameters (learning rate in particular) as per paper description (poly strategy), and did it result in any improvements? I am just wondering, given that adam is already adapting the learning rate, using SGD with poly learning rate will not make much of a difference. Anyhow, I'll be pleased to know what you discovered?


Fixed showing constraints gui

Codecov Report

Merging #89 into master will decrease coverage by 0.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   56.49%   55.52%   -0.97%
==========================================
  Files          32       32
  Lines        2696     2741      +45
  Branches      436      445       +9
==========================================
- Hits         1523     1522       -1
- Misses       1037     1080      +43
- Partials      136      139       +3

Impacted Files
Coverage Δ

...hudson/plugins/performance/TaurusStatusReport.java
6.45% <0%> (-93.55%)
:arrow_down:

.../hudson/plugins/performance/PerformanceReport.java
62.42% <0%> (-15.03%)
:arrow_down:

...ain/java/hudson/plugins/performance/UriReport.java
60.73% <0%> (-7.98%)
:arrow_down:

...on/plugins/performance/PerformanceBuildAction.java
78.12% <0%> (-3.13%)
:arrow_down:

.../java/hudson/plugins/performance/TaurusParser.java
100% <0%> (+2.22%)
:arrow_up:

...ava/hudson/plugins/performance/AbstractParser.java
68.33% <0%> (+6.66%)
:arrow_up:

.../java/hudson/plugins/performance/JMeterParser.java
71.21% <0%> (+12.12%)
:arrow_up:

...va/hudson/plugins/performance/JMeterCsvParser.java
81.3% <0%> (+75.58%)
:arrow_up:

Continue to review full report at Codecov.

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


Update README

While reading the documentation for Passport I noticed the word 'flexibility' was spelled incorrectly on line 15. This pull request fixes the spelling error.

Coverage remained the same at 100.0% when pulling eb0b4356eae5bbddd8e9f3a2e35a759e903522b3 on amZotti:patch-1 into d0f6b6c5ab4d292efdd83b0eeeb7b0e5dcfd823a on jaredhanson:master.

+1


[CI] Update Xcode to 13.2.1

Release notes: https://developer.apple.com/documentation/xcode-release-notes/xcode-13_2_1-release-notes
Description: Update the Xcode version used in build configuration files to use the latest stable Xcode release: Xcode 13.2.1.
Risk Level: Medium, this updates the versions of llvm, clang, swift as well as the Apple frameworks we link against.
Testing: Existing tests
Docs Changes: Updated.
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

@lfpino is there a documented process for updating EngFlow to install new Xcode versions on its remote execution workers?

Hi JP! We don't have an automatic way to do it, but we can try to generate new MacOS images with the new version. Let me check with our team and get back to you.
When would you like for this to happen?
CC @ulfjack @Yannic

Thanks for your quick reply, Luis!

When would you like for this to happen?

It's not urgent, but I was expecting to be able to handle the changes myself fairly quickly, so it came as a surprise that there's infrastructure that I can't configure that's involved here.
In fact, by temporarily disabling EngFlow in the GitHub Actions scripts, everything works with Xcode 13.2.1 by executing jobs directly on the GitHub Actions agents. Of course, this takes much longer to compile so I'd like to wait for Xcode 13.2.1 support in EngFlow before merging this.
I think whatever comes of this I'll want to document the process for updating Xcode versions so that moving forward we know what to expect, and can give you a proper heads up, etc.

Yeah, that makes a lot of sense :-)
I can file an internal feature request for us to track this. IIRC we just haven't had this use case before.
And 100% agree that having good documentation on how to handle this is critical, so maybe we can add some instructions in the Envoy as part of the upgrade.
WDYT?

And 100% agree that having good documentation on how to handle this is critical, so maybe we can add some instructions in the Envoy as part of the upgrade.
WDYT?

Yes, that'd be great. Ideally we'd have better documentation about how CI works overall, for both Envoy and Envoy Mobile. Maybe we can collaborate to write that up?
Based on my understanding, which likely is incorrect in some ways:

Envoy primarily uses Azure Pipelines for its CI, with some remote execution happening via remotebuildexecution.googleapis.com
Envoy Mobile primarily uses GitHub Actions for its CI, with remote execution happening via EngFlow
For both projects, environment dependencies need to be mirrored with matching versions in both the CI host (Azure Pipelines & GH Actions) as well as its remote execution service (Google RBE & EngFlow)

Am I getting that right?

I can file an internal feature request for us to track this. IIRC we just haven't had this use case before.

I don't mean to rush you with this question, I'm just trying to understand: how much lead time should we expect to request a new Xcode version on EngFlow executors? Are we talking on the order of days, weeks or months?

I'm super familiar on how Envoy mobile CI works but not so much about Envoy. I can definitely help writing something :-)
I believe the three points you mention are mostly correct, except that for Envoy Mobile we use Engflow and the GH runner doesn't necessarily need to match the Engflow worker configuration but we do use the same docker image for both (i.e. running the CI job and running the actual Bazel actions).
W.r.t timing, once Apple has made XCode available it's usually a matter of days (even hours) to upgrade, so I'd say that asking us  ~1-2 weeks in advance is enough. We also have an oncall that can take care of this quicker for urgent situations. So it's just a matter of asking and it may not even be worth automating this fully since there's only a few versions per year, but we can always discuss it.

Ok thanks for sharing. We’d like to request support for Xcode 13.2.1 in EngFlow whenever possible, and we’ll merge this as soon as that’s available. There’s no big rush but we’re ready when you are.

It's unfortunately not super straight forward to upgrade Xcode on our side. Bazel only sends the Xcode version to RE, so we need to have Xcode installed on the executors (because Apple's license says so). The way we do that today is by installing Xcode when building our release images, which means that we need to cut a release to update Xcode. Additionally, Xcode needs a lot of space on the images, so we can only fit a certain number of versions into the image. Plus, we need to coordinate changes there between everyone using the image which takes time.
I have something on my todo list to improve the situation and at least allow us to install more than 2-3 Xcode versions at a time in an image, but there's a lot of other things on my todo list as well.
We'll start the process of adding Xcode 13.2.1 to our images now, but it'll probably be next week until it's rolled out TBH.

If I'm understanding you correctly, you can only have a single macOS image rolled out at any given time across all your customers?
Thanks for kicking off the process. Please keep us posted of your progress.

Hi @Yannic & @lfpino, how's the process going to add this version of Xcode to the EngFlow macOS image going?

Hey @jpsim, Yannic has been making some changes to our images to add the new XCode, should be ready in the next few days :)

Hi @Yannic & @lfpino, how this going? We're keeping this older version of Xcode provisioned in our CI fleet just for this project now, so it'd be good to move to Xcode 13.2.1 soon. Thanks!

Hey @jpsim, we're very close, @Yannic is sending some internal changes to make this process simpler, should be done soon :)

ok thanks for the update!

You should be able to try now, we rolled out new images that contain the new XCode. Let us know if you see any problems.

Thanks @Yannic & @lfpino, CI passed with remote exec. This is ready for review now.


CardMedia className prop not applied to any resulting DOM

Problem description
CardMedia doesn't do anything with passed-in className. Card does apply its className props, and per the docs, every component should, so this seems like a bug.
I'm happy to PR if someone can point me to an example. It wasn't obvious to me how the className prop was being applied in the Card.js source.
Example code
<Card className='my-card-class'>
  <CardMedia className='my-cardmedia-class'>
    <img src='foo' />
  </CardMedia>
</Card>

my-card-class is applied, but my-cardmedia-class is not.
Versions

Material-UI: 0.17.0
React: 15.4.2
Browser: Chrome 56.0.2924.87

@chellberg Thanks for providing a clear problem description with example! :+1:
I've just had a quick look at the CardMedia source, and ...other props are being spread to the root  div element [1], so no reason this shouldn't work. Are you sure it isn't showing up when you inspect the DOM? Are you trying to restyle any built-in inline styles, and if so did you remember !important?
 I wonder why the <div {...other}> isn't passing it through. Looks like the component hasn't changed since October 2016, so shouldn't be an issue with my version being behind master.

We have been porting the component on the next branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.

React: 15.5.4
Material-UI: 0.18.3
I can confirm this is also an issue with TextField, the className is not applied.
Slowing dev down very much because of this.

I can confirm this issue is also evident in the TexField component. Setting className has no effect, the DOM tree shows no resulting class.
React:15.5.4,