Move to the subnet delegation model for 1ES hosted pools

Our current deployment model for 1ES hosted pools and 1ES images (Bring your own subscription) is outdated. We should now be onboarding to subnet delegation inside 1ES managed subscriptions.
docs are here: https://eng.ms/docs/cloud-ai-platform/devdiv/one-engineering-system-1es/1es-docs/1es-hosted-azure-devops-pools/subnet-delegation

Release Note Category

[x] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
1ES hosted pools will now create the VMs that run the builds in 1ES hosted subscriptions.

Looks like the process is relatlively straightforward:

Deploy and configure a VNet
Follow the migration guide to connect existing pools to the vnet
Validate that things work as expected after each migration
Change the helix-machines code to deploy new pools pointing to the new vnet.

given our conversation this afternoon, should this issue be assigned to me @riarenas❔

@dougbu Oh, I was thinking that you'd be working on #1145. I was thinking this issue would deal mostly with the conversion of existing resources to the new model once there was a VNet set up for #1145
If you want to tackle them both at the same time, or one after the other I can leave this alone, I don't think I'll get to do more than investigation for this week anyways!

This isn't strictly related to 1es managed images, and it's more of a compliance requirement at this point. Even the pipeline templates task requires this. I'll move this out of the epic and into ops-compliance.

Starting to look at this as a 1ES PT prerequisite

The process here will be:

Stop setting the parameters needed for hosting the machines in our own subscriptions.
Make the code changes to start deploying a VNET with an NSG as part of creating the resource group that will have the hosted pool in it
Correctly set up the networkProfile section of the Pool during deployment, pointing to the new vnet

https://dnceng.visualstudio.com/internal/_git/dotnet-helix-machines/pullrequest/37265

Taking a close look at https://dev.azure.com/dnceng/internal/_build/results?buildId=2381029&view=results to validate in the staging environment.

Womp womp, failed with a nullref. Checking.

This is currently undergoing deployment. As we move to 1ES managed subscriptions, we are finding out that these subs that we get assigned to start out with very low core quota, so we'll need to go through quota increase requests for each pool we deploy.

We are still working with 1ES to get the remaining pools migrated. We have some suggestions from the 1ES team that we will try today.

try again, see if that gets you a different subscription that is allowlisted
try again with a different SKU, to see if that doesn't get denied.

The two pools in this state currently are:

netcore1es-pool-internal-xl
netcorepublic-xl

netcore1espool-internal-xl has been updated after getting our core request approved. Still working on netcorepublic-xl

We were finally granted all the cores we needed for our pools. This is complete

[celebrate]     Mark Wilkie reacted to your message:

From: Ricardo Arenas @.>
Sent: Wednesday, March 20, 2024 8:56:09 PM
To: dotnet/dnceng @.>
Cc: Subscribed @.***>
Subject: Re: [dotnet/dnceng] Move to the subnet delegation model for 1ES hosted pools (Issue #1850)
We were finally granted all the cores we needed for our pools. This is complete
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/dnceng/issues/1850#issuecomment-2010609373, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQKTJNK7IOXZ4XBSMYPRJDYZHZWTAVCNFSM6AAAAABCAVEV6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJQGYYDSMZXGM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Help Desk Template

Hi
I’m positive I have downloaded two different versions of Help Desk Template.
Is this possible? When on the community, people ask which template?

Where in the instructions / video does it instruct me to to change ‘TicketList’ to a new DataSource?... so that it starts saving tickets and we can use the app.
Currently I add tickets, after a bit of time it defaults to the default tickets.

Thank you for reaching out. The template is only available in one version for download.
The steps to remove SharePoint connection from the template and add connection that you create can be found in the Configure the app to use SharePoint list section.
If you run into any issues, please reach out to the support teams with a support ticket for further investigation.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #524 (b88c690) into main (0deaf50) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #524   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


PDF: styling for published documents not according to reference docs

The PDF styling for published documents generally look like:

https://github.com/metanorma/ieee-ref-docs/blob/main/standards/8021X-2020.pdf
https://github.com/metanorma/ieee-ref-docs/blob/main/standards/18002-2020.pdf
https://github.com/metanorma/ieee-stl/blob/main/master-1-standard/master-1-standard.pdf

The current generated PDF uses the styling from the White Paper templates (IEEE-SA-Non_Standard_Projects_Style_Manual.pdf), which is incorrect for standards (2021_IEEE SA_Style Manual.pdf).

I can't repeat, for 'master-1-standard' it renders so:

IEEE xslt updated.


q2-plugin and input conversion to OrdinationResults type (skbio)

Outline of changes:

input type to OrdinationResults (skbio) -- output from deicode & songbird
added setup file with easy install via pip
q2-plugin with visualization output in type qzv
standalone command line out is dir containing assets and index so you can just open from there
example directory with analysis from start to finish with both biplot and rank visual in two commands
updated ReadMe

TODO:

need to update depenencies for CI
most commands still need tests (very low coverage) (prob. some old test failing)
updates to the visual side of the code -- general usability etc...

This was because when you convert from the default deblur type index which is the sequence identifier, to taxa it seems to break. Maybe the best way to solve this is to keep the sequence in the id, that way the seq. identified can be easily copied from a biplot. For example, where the red arrow is in the denominator and the blue in the numerator. This can be copied by just double-clicking on the arrow and pasted into the search box.  Producing the rank-log-ratio plot.


feat: efficient batch range check

see also #388 .
The batch range check component should enable:

[ ] efficient limb batch range checks for emulated arithmetic (Groth16)
[ ] efficient limb batch range checks for emulated arithmetic (PlonK)
[ ] bypass the range check constraints (but still identify the affected variable that need to be range checked) to perform the range checks out-of-circuit.

@ivokub can you update status on this one?

@ivokub can you update status on this one?

All is done. The range check gadget is agnostic to the builder if it either implements Committer or Rangechecker interfaces. If PLONK is to receive Commit method, then range checks start working automatically. Or if we plug in builder (e.g. Wizard) which has native support for range checks, then starts automatically working (i.e. we relay all range checks to the proof system, which handles it on its own).
I'm not sure if should close before PLONK commitment is done? Otherwise done from my side.


🐛 No highlighting in vs code workspaces

Environment information
not needed

What happened?
Open a workspace in vs code (for eg. an application that runs in combination with an other combination)
Highlighting does not work
Open the files without workspaces - highlighting works.
Expected result
That highlighting is working
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

By highlighting, do you mean warnings and errors underlined with squiggly lines ?

By highlighting, do you mean warnings and errors underlined with squiggly lines ?

Yes.

Thanks for clarifying @AdrianSchmidt91.

I am experiencing the same issue however i did a few more tests and it was actually working with workspaces when the folder that contains the biome config is actually the first one in the workspace configuration. So for example this works (if projecta has biome config but projectb and c dont have it:
{
	"folders": [
		{
			"path": "..\\projecta"
		},
		{
			"path": "..\\projectb"
		},
		{
			"path": "..\\projectc"
		}
	],

but this doesnt work:
{
	"folders": [
		{
			"path": "..\\projectb"
		},
		{
			"path": "..\\projectc"
		},
		{
			"path": "..\\projecta"
		}
	],

so it seems to be an issue of the order of the folders in the workspace?!

I am experiencing the same issue however i did a few more tests and it was actually working with workspaces when the folder that contains the biome config is actually the first one in the workspace configuration. So for example this works (if projecta has biome config but projectb and c dont have it:
{
	"folders": [
		{
			"path": "..\\projecta"
		},
		{
			"path": "..\\projectb"
		},
		{
			"path": "..\\projectc"
		}
	],

but this doesnt work:
{
	"folders": [
		{
			"path": "..\\projectb"
		},
		{
			"path": "..\\projectc"
		},
		{
			"path": "..\\projecta"
		}
	],

so it seems to be an issue of the order of the folders in the workspace?!

Also the "Format Document With.." will then not be displayed if the biome-project is not at first place in that folders array.

The same problem occurs when using biome for a single project in a monorepo like pnpm workspaces. Having a biome.jsonin the root works but does not really make sense to have if it is only used in one of the projects.
No syntax highlighting, and Biome won't show up as a formatter in vs code.

The VS Code extension currently uses the workspace.findFiles API to locate a biome.json file at the root of any of the workspaces and it uses the first one that it finds.
https://github.com/biomejs/biome/blob/5179abd24b315194416f357afe6b69e11e6470e7/editors/vscode/src/main.ts#L49-L62

@nhedger what does this mean for us? Sorry I don't know how to interpret your suggestion 😅


What has happened with this repository at 2022.04.13 ?

All git history has disappeared and you have started from scratch, why ? :)

LOL, please don't lose history of commits, history is important for merge with local edits for users like me.
Thanks for the mod!


Automated cherry pick of #11337

Cherry pick of #11337 on release-7.4.
/cc  @hmhealey
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: aa491bd3a6a385ab82ec7bea965a33739193c420


refactor to use custom parser

This is much faster, passes all unit tests, and will be easier to update and maintain without dependencies.

@tunnckoCore want to review this? Let me know if you see any issues?

thanks!


test/e2e/framework/users: fix AdmitWorkspaceAccess

Summary
Related issue(s)
Fixes #

maybe we actually don't need it
/hold

/hold cancel


feat: event notifications as a data source

Description
Avoid errors in DA when using existing resources (#265)
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline
The previous run failed on an unrelated and known error with CBRs.

/run pipeline

/run pipeline

/run pipeline

@shemau I think this may of caused a regression -> https://github.com/terraform-ibm-modules/terraform-ibm-event-notifications/issues/299


Only log iOS signing cert once when pub get in module

Fixes https://github.com/flutter/flutter/issues/92701
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Not sure why this is failing in CI, it passes on my machine.  Need to investigate...

(This is still on my radar, I'll circle back soon)


MacOS builds disabled?

We used to have MacOS builds, I thought, but they seem to be disabled for recent releases.  @oskooi, @ChristopherHogan?

According to @ChristopherHogan, restoring the macOS build will require several days of work unfortunately which we haven't gotten around to yet.

What changed to break this?

I think the issue is related to package conflicts involving the libblas and libcblas dependencies and numpy which are tricky to resolve.

You need to make sure everything is built with the same BLAS — Anaconda numpy uses MKL by default, I think, so Harminv and MPB need to be built against this as well, but this should be straightforward (adding the dependency to the package, and passing the appropriate configure args).

Yes, there seems to be an inconsistency with the BLAS library that is used to compile the Conda package for Harminv  and MPB:
https://github.com/conda-forge/harminv-feedstock/blob/f34d05bbcdd5be76989b880ef7b7d992a16d52c1/recipe/build.sh#L3
https://github.com/conda-forge/mpb-feedstock/blob/c93f4331016aedb411bdd03709f74c3c97a75a9f/recipe/build.sh#L13
Harminv is compiled with OpenBLAS but MPB is not.

Without any changes to the feedstock, the macOS Conda package automatically updated to v1.15.0 yesterday: https://anaconda.org/conda-forge/pymeep. Not sure how/why this happened but it would be good to verify that the macOS Conda package is indeed 1.15 (rather than a mislabeled badge) by inspecting the version number of the module:
import meep as mp
print(mp.__version__)

I checked the tarball and verified that it is really 1.15, so it seems this is fixed.


Introduce BuildFeatures API for Configuration Cache and Isolated Projects

Introduces a new public API that allows introspecting and reacting to build feature status of Configuration Cache and Isolated Projects features.
The API distinguishes between the features being requested and being active, because they serve separate use-cases:

The "requested" information expresses user's intent and useful for gathering feature adoption statistics.
The "active" status shows whether a feature is actually enabled in the build and may have effects on Gradle's behavior.

The latter can be especially useful for incremental adoption of complex features such as Isolated Projects in ecosystem plugins such as Kotlin Gradle Plugin or Android Gradle Plugin.
The PR adds this info about two features:

Configuration Cache
Isolated Projects

Fixes #23840
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gitstream check all

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


Triage finschia-sdk#334

Finschia/finschia-sdk#334
This is our own script for testing. We can apply this if we still need but I recommend to rewrite this script in bash for versatility


Apparent duplicate rule (S307 and PGH001)

Rules suspicious-eval-usage (S307) and eval (PGH001) seem to do the same thing. If they are, in fact, distinct, then the difference between them should probably be explained in the documentation.

I can confirm these look duplicated and have separate implementations. I'm not sure what we should do about it though.

We should remove PGH001, and add a redirect to S307 (IMO).

This got done!


Works on Mac OS X 10.5

FIX: work with gcc > 4.0.0
FIX: compilation error on Mac OS X 10.5
TODO: allow people to report issues / bug report
TODO: VisomicsData address

Dr. Gelas,
Thank you for your interest in Visomics!
We are currently in the process of transitioning Visomics from a Kitware-sponsored phase 1 NIH SBIR to a permissive open-source project, in the fashion of ITK and VTK.
I'll review your branch shortly. The VisomicsData repository will be made available soon, as soon as we can ensure that no confidential or proprietary from our SBIR collaborators is present. I apologize for any build difficulties you may have had due to the absence of VisomicsData.
Brian Helba
Medical Imaging
Kitware, Inc.

Thanks Arnaud.  We are still getting this launched completely, but it is
great that you have already contributed! :-)

Wes

On Thu, Dec 29, 2011 at 6:04 AM, Arnaud Gelas <
<EMAIL_ADDRESS>
wrote:

FIX: work with gcc > 4.0.0
FIX: compilation error on Mac OS X 10.5
TODO: allow people to report issues / bug report
TODO: VisomicsData address
You can merge this Pull Request by running:
git pull https://github.com/arnaudgelas/Visomics master
Or you can view, comment on it, or merge it online at:
https://github.com/brianhelba/Visomics/pull/1
-- Commit Summary --

Add .gitignore
Works with gcc 4.0.0 (Mac 10.5)
Temporarily commented (wrong address)
fix compilation error on Mac 10.5

-- File Changes --
A .gitignore (5)
M Base/Testing/Cpp/voDataObjectTest.cpp (3)
M CMakeLists.txt (2)
M SuperBuild.cmake (2)
-- Patch Links --
https://github.com/brianhelba/Visomics/pull/1.patch
https://github.com/brianhelba/Visomics/pull/1.diff

Reply to this email directly or view it on GitHub:
https://github.com/brianhelba/Visomics/pull/1

--
Wesley D. Turner, Ph.D.
Kitware, Inc.
Technical Leader
28 Corporate Drive
Clifton Park, NY 12065-8662
Phone:<PHONE_NUMBER>

Hi Wes,
I have been working in a systems biology lab, so this software could definitively of interest. I am looking forward to giving it a try!
Best,
Arnaud


CoalescePartitionsExec requires at least one input partition when aggregating empty tables on single-core machines

This only happens on Fly.io (I think I tested this with the same Docker image locally and didn't get the issue):
$ curl -iH "Content-Type: application/json" https://seafowl/q -d '{"query": "CREATE TABLE test2 (key INTEGER, value TEXT)"}'
HTTP/2 200
$ curl -iH "Content-Type: application/json" https://seafowl/q -d '{"query": "SELECT COUNT(*) FROM test2"}'
HTTP/2 400
Internal error: CoalescePartitionsExec requires at least one input partition. This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker

Local Docker (TODO: quadruple-check it's definitely the same Docker image as the one on Fly):
 ~ $ curl -iH "Content-Type: application/json" http://localhost:8080/q -d '{"query": "CREATE TABLE test (key INTEGER, value TEXT)"}'
HTTP/1.1 200 OK
content-type: application/octet-stream
vary: Content-Type, Origin, X-Seafowl-Query
content-length: 0
date: Mon, 31 Oct 2022 16:48:31 GMT

 ~ $ curl -iH "Content-Type: application/json" http://localhost:8080/q -d '{"query": "SELECT COUNT(*) FROM test"}'
HTTP/1.1 200 OK
content-type: application/octet-stream
vary: Content-Type, Origin, X-Seafowl-Query
content-length: 22
date: Mon, 31 Oct 2022 16:48:40 GMT

{"COUNT(UInt8(1))":0}

Doesn't just happen on Fly.io, I had an older debug version in my Docker, could be a regression in a recent DF.

It looks like it's because of the number of cores allocated (becomes num_cpus and then feeds into DF as the target_partition_count setting.
1 core (with ./taskset -c 1 seafowl):

ProjectionExec: expr=[COUNT(UInt8(1))@0 as COUNT(UInt8(1))]      +
   AggregateExec: mode=Final, gby=[], aggr=[COUNT(UInt8(1))]      +
     CoalescePartitionsExec                                       +
       AggregateExec: mode=Partial, gby=[], aggr=[COUNT(UInt8(1))]+
         ParquetExec: limit=None, partitions=[], projection=[key] +

multicore:

ProjectionExec: expr=[COUNT(UInt8(1))@0 as COUNT(UInt8(1))]       +
   AggregateExec: mode=Final, gby=[], aggr=[COUNT(UInt8(1))]       +
     CoalescePartitionsExec                                        +
       AggregateExec: mode=Partial, gby=[], aggr=[COUNT(UInt8(1))] +
         RepartitionExec: partitioning=RoundRobinBatch(4)          +
           ParquetExec: limit=None, partitions=[], projection=[key]+

The physical plan for single-core is missing RepartitionExec.

Seems not to be occurring anymore, even without the fix from #189 (which was removed in #422); closing.


Refresh key without notifying watchers

supports etcd 2.3.0's refresh TTL,

Keys in etcd can be refreshed without notifying watchers,
this can be achieved by setting the refresh to true when updating a TTL
You cannot update the value of a key when refreshing it

doc: https://github.com/coreos/etcd/blob/release-2.3/Documentation/api.md#refreshing-key-ttl

Coverage decreased (-0.2%) to 88.181% when pulling ddc7db0055d27eb5c356b2ca1912de3532868569 on mSOHU:refresh-key into f2e1531ee8aba9002dcbefe39678a2f872e53c62 on jplana:master.


[1.16.5] Error when adding mod to server

Please fill out the template below to describe your issue. Doing so greatly helps us understand what happened!
Description
Whenever I add the mod to my server I get the following message
-- Head --
Thread: main
Stacktrace:
at dev.limonblaze.oriacs.common.item.UmbrellaItem.(UmbrellaItem.java:31) ~[oriacs:0.999] {re:mixin,re:classloading}
-- MOD oriacs --
Details:
Mod File: origins-accessbilities-1.16.5-0.999.jar
Failure message: Origins: Accessibilities (oriacs) has failed to load correctly
java.lang.NoSuchMethodError: net.minecraft.util.math.MathHelper.func_180183_b(FFF)I
Mod Version: 0.999
Mod Issue URL: https://github.com/LimonBlaze/origins-accessibilities/issues
Exception message: java.lang.NoSuchMethodError: net.minecraft.util.math.MathHelper.func_180183_b(FFF)I
Steps to reproduce:
This can be reproduced at any time by adding the jar file to the mods folder on a server.
The log can be found here:
crash-2023-01-10_02.36.57-fml.txt
Mod list:
Mod List:
forge-1.16.5-36.2.35-server.jar                   |Minecraft                     |minecraft                     |1.16.5              |CREATE_REG|Manifest: NOSIGNATURE
origins-accessbilities-1.16.5-0.999.jar           |Origins: Accessibilities      |oriacs                        |0.999               |ERROR     |Manifest: NOSIGNATURE
jei-1.16.5-<IP_ADDRESS>.jar                          |Just Enough Items             |jei                           |<IP_ADDRESS>           |CREATE_REG|Manifest: NOSIGNATURE
caelus-forge-1.16.5-<IP_ADDRESS>.jar                   |Caelus API                    |caelus                        |1.16.5-<IP_ADDRESS>      |CREATE_REG|Manifest: NOSIGNATURE
flywheel-1.16-0.2.5.jar                           |Flywheel                      |flywheel                      |1.16-0.2.5          |CREATE_REG|Manifest: NOSIGNATURE
steampowered-1.16.5-1.2.6.jar                     |Create: Steam Powered         |steampowered                  |1.16.5-1.2.6        |CREATE_REG|Manifest: NOSIGNATURE
Patchouli-1.16.4-53.3.jar                         |Patchouli                     |patchouli                     |1.16.4-53.3         |CREATE_REG|Manifest: NOSIGNATURE
Origins-1.16.5-<IP_ADDRESS>-forge.jar                  |Origins                       |origins                       |<IP_ADDRESS>             |CREATE_REG|Manifest: NOSIGNATURE
create-mc1.16.5_v0.3.2g.jar                       |Create                        |create                        |v0.3.2g             |CREATE_REG|Manifest: NOSIGNATURE
createdeco-1.1.2-1.16.5.jar                       |Create Deco                   |createdeco                    |1.1.2-1.16.5        |CREATE_REG|Manifest: NOSIGNATURE
createautomated-1.16.5-1.4.1a.jar                 |Create Automated              |createautomated               |1.16.5-1.4.1a       |CREATE_REG|Manifest: NOSIGNATURE
Waystones_1.16.5-7.6.4.jar                        |Waystones                     |waystones                     |7.6.4               |CREATE_REG|Manifest: NOSIGNATURE
journeymap-1.16.5-5.8.5p6.jar                     |Journeymap                    |journeymap                    |5.8.5p6             |CREATE_REG|Manifest: NOSIGNATURE
Placebo-1.16.5-4.7.0.jar                          |Placebo                       |placebo                       |4.7.0               |CREATE_REG|Manifest: NOSIGNATURE
createpixelmon-1.16.5-1.0.3.jar                   |Create: Pixelmon              |createpixelmon                |1.16.5-1.0.3        |CREATE_REG|Manifest: NOSIGNATURE
NaturesCompass-1.16.5-1.9.1-forge.jar             |Nature's Compass              |naturescompass                |1.16.5-1.9.1-forge  |CREATE_REG|Manifest: NOSIGNATURE
Bookshelf-Forge-1.16.5-10.4.32.jar                |Bookshelf                     |bookshelf                     |10.4.32             |CREATE_REG|Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
EnchantmentDescriptions-1.16.5-7.1.20.jar         |EnchantmentDescriptions       |enchdesc                      |7.1.20              |CREATE_REG|Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
create-confectionery1.16.5_v1.0.2.jar             |Create Confectionery          |create_confectionery          |1.0.2               |CREATE_REG|Manifest: NOSIGNATURE
fancymenu_forge_2.12.9_MC_1.16.2-1.16.5.jar       |FancyMenu                     |fancymenu                     |2.12.9              |CREATE_REG|Manifest: NOSIGNATURE
Apotheosis-1.16.5-4.8.9A0.jar                     |Apotheosis                    |apotheosis                    |4.8.9A0             |CREATE_REG|Manifest: NOSIGNATURE
architectury-1.32.66.jar                          |Architectury                  |architectury                  |1.32.66             |CREATE_REG|Manifest: NOSIGNATURE
cloth-config-4.16.91-forge.jar                    |Cloth Config v4 API           |cloth-config                  |4.16.91             |CREATE_REG|Manifest: NOSIGNATURE
forge-1.16.5-36.2.35-universal.jar                |Forge                         |forge                         |36.2.35             |CREATE_REG|Manifest: 22:af:21:d8:19:82:7f:93:94:fe:2b:ac:b7:e4:41:57:68:39:87:b1:a7:5c:c6:44:f9:25:74:21:14:f5:0d:90
Disenchanting-forge-1.16.5-1.7.2.jar              |Disenchanting                 |disenchanting                 |1.6.0               |CREATE_REG|Manifest: NOSIGNATURE
valhelsia_core-16.0.15.jar                        |Valhelsia Core                |valhelsia_core                |16.0.15             |CREATE_REG|Manifest: NOSIGNATURE
forbidden_arcanus-16.2.3.jar                      |Forbidden & Arcanus           |forbidden_arcanus             |16.2.3              |CREATE_REG|Manifest: NOSIGNATURE
konkrete_forge_1.5.3-1_MC_1.16.2-1.16.5.jar       |Konkrete                      |konkrete                      |1.5.3               |CREATE_REG|Manifest: NOSIGNATURE
createaddition-1.16.5-20220129a.jar               |Create Crafts & Additions     |createaddition                |1.16.5-20220129a    |CREATE_REG|Manifest: NOSIGNATURE
Pixelmon-1.16.5-9.1.0-server.jar                  |Pixelmon Mod                  |pixelmon                      |9.1.0               |CREATE_REG|Manifest: NOSIGNATURE
ExplorersCompass-1.16.5-1.1.2-forge.jar           |Explorer's Compass            |explorerscompass              |1.16.5-1.1.2-forge  |CREATE_REG|Manifest: NOSIGNATURE
byg-1.3.5.jar                                     |Oh The Biomes You'll Go       |byg                           |1.3.4               |CREATE_REG|Manifest: NOSIGNATURE

I should mention umbrella's didn't appear to work in client side of mod either on a singleplayer world in 1.16.5

Fixed in 0.9991


Fix bug in realtime segment relocator

Fetching helixManager and helixAdmin from pinotHelixResourceManager everytime instead of caching to avoid null initialization

Codecov Report

Merging #2887 into master will increase coverage by 0.03%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master    #2887      +/-   ##
==========================================
+ Coverage   69.15%   69.19%   +0.03%
==========================================
  Files         912      912
  Lines       43623    43621       -2
  Branches     5932     5931       -1
==========================================
+ Hits        30168    30182      +14
+ Misses      11421    11415       -6
+ Partials     2034     2024      -10

Impacted Files
Coverage Δ

...elix/core/relocation/RealtimeSegmentRelocator.java
77.96% <91.66%> (-0.37%)
:arrow_down:

...a/manager/realtime/RealtimeSegmentDataManager.java
50% <0%> (-25%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.22% <0%> (-6.67%)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
68.88% <0%> (-6.67%)
:arrow_down:

.../impl/dictionary/FloatOnHeapMutableDictionary.java
80% <0%> (-6.67%)
:arrow_down:

...n/pinot/core/io/writer/impl/MmapMemoryManager.java
80% <0%> (-6.16%)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
80% <0%> (-5.46%)
:arrow_down:

...e/operator/dociditerators/SortedDocIdIterator.java
59.45% <0%> (-5.41%)
:arrow_down:

...ot/core/query/pruner/ColumnValueSegmentPruner.java
88.13% <0%> (-3.39%)
:arrow_down:

.../pinot/core/common/datatable/DataTableBuilder.java
78.82% <0%> (-1.18%)
:arrow_down:

... and 17 more

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 03852e1...66601a3. Read the comment docs.

The bug was that I was fetching helixAdmin and helixManager from pinotHelixResourceManager in the constructor of RealtimeSegmentRelocator, which happed before pinotHelixResourceManager was initialized in the ControllerStarted.start method


Ending a stream without closing the whole page

Hi,
Since .destroy kill the whole page and .pause and .resume do not allow ffmpeg to catch a stream on the fly, how does one close a stream without closing the whole page (in a way that allow to make a new one later on the same page) ?
I'm confused
Thanks

Fixed by #18


customize resources price for cost estimate

Does infracost support customize price that include enterprise discounts or credits from cloud provider for doing cost breakdown or estimate rather than just using the standard pricing from cloud provider, similar use-case as described here: https://github.com/infracost/infracost/issues/843
How does infracost handle the auto scaling resources cost estimate?

@lindu-dd great questions!

We want to support custom price books as part of our paid product - feel free to email me at<EMAIL_ADDRESS>if you'd like to discuss this more (would be good to know what granuliarty you need for the discounts, e.g. EC2 in regionX, or per instance type per region)
It depends on the resource, e.g. for AWS EKS node group we take desired_size but if the sync-from-cloudwatch option is being used, then this logic is used. In CI/CD, I've seen users use this workaround to get the fetch-from-cloudwatch data without having to create a yaml file in advance: infracost breakdown --sync-usage-file --usage-file /tmp/ignore.yml --format json --out-file infracost.json --path /code
Does that help, or did you have something else in mind? e.g. show the min/max where possible?

@lindu-dd I'm going to close this issue but it would be great if you can post a comment on this issue with which option works for your use-case.


Revert "updated the correct file name and hash"

Reverts berkeley-stat159/project-theta#20

Reverting the accidental merge

Can we revert this revert?


devops(devcontainer): add sqlite3 and duckdb cli

It is useful to have these installed when checking prqlc behavior.

(I originally wanted to install usql, but unfortunately there was no binary release for arm64 Linux)

Does DuckDB take ages to compile & install? Or it copies a binary?
Guessing the latter given your comment re usql. Otherwise we could add to the base image...

Great!


chore: Update issue templates to not include ticket routing in rendered issue

Fixes #N/A
Description
This updates the issue template to use Github alert syntax for surfacing the important routing detail in markdown at the top of issues, but not to include this note in the rendered issue once it's created.
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 82.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4951

Relevant Lines:
5999

💛  - Coveralls


!computers - "Whitty" emoji

There's a new computer location so it needs a new emoji. Also probs best to add a generic emoji for if this happens again

Fixed in 6608db1, closing :)


Headers import Errors

Describe the bug
I would like to test the HttpClient. But both imports will raise Errors, can you explain this import errors? I can´t understand why Headers not init.
I have install lightbug_http via magic at a new project
To Reproduce
Example 1 with import *

from lightbug_http.sys.client import MojoClient
from lightbug_http import *

fn test_request(inout client: MojoClient) raises -> None:
    var uri = URI("http://httpbin.org/status/200")
    var headers = Headers(
        Header("Connection", "keep-alive"),
        Header("Host", "httpbin.org"),
        )
    var request = HTTPRequest(uri,headers)
    var as_str = str(request)
    print(as_str)
    var response = client.do(request^)

    # print status code
    print("Response:", response.status_code)

    # print parsed headers (only some are parsed for now)
    print("Content-Type:", response.headers["Content-Type"])
    print("Content-Length", response.headers["Content-Length"])
    print("Server:", to_string(response.headers["Server"]))

    print(
        "Is connection set to connection-close? ", response.connection_close()
    )

    # print body
    print(to_string(response.body_raw))

fn main() -> None:
    try:
        var client = MojoClient()
        test_request(client)
    except e:
        print(e)

Errors

(arango-mojo) ➜  arango-mojo git:(main) ✗ magic run mojo hello2.mojo
/home/tholo/Scripts/mojo/arango-mojo/hello2.mojo:7:19: error: use of unknown declaration 'Headers'
var headers = Headers(
^~~~~~~
/home/tholo/Scripts/mojo/arango-mojo/.magic/envs/default/bin/mojo: error: failed to parse the provided Mojo source module

Example 2 with import strict imports
from lightbug_http.sys.client import MojoClient
from lightbug_http.http import HTTPRequest
from lightbug_http.uri import URI
from lightbug_http.header import Header, Headers

fn test_request(client: MojoClient) raises:
    var uri = URI("http://httpbin.org/status/200")
    var req = HTTPRequest(
        uri,
        body=String("Hello world!").as_bytes(),
        headers=Headers(
            Header("Connection", "keep-alive"),
            Header("Host", "httpbin.org"))),

    try:
        var response = client.do(req)
        print("Response:", response.status_code)

        # print parsed headers (only some are parsed for now)
        print("Content-Type:", response.headers["Content-Type"])
        print("Content-Length", response.headers["Content-Length"])

        print(
            "Is connection set to connection-close? ", response.connection_close()
        )

        # print body
        #print(to_string(response.body_raw))

    except e:
        print(e)

fn main() -> None:
    try:
        var client = MojoClient()
        test_request(client)
    except e:
        print(e)

Errors

(arango-mojo) ➜  arango-mojo git:(main) ✗ magic run mojo hello.mojo
/home/tholo/Scripts/mojo/arango-mojo/hello.mojo:4:34: error: module 'header' does not contain 'Header'
from lightbug_http.header import Header, Headers
^
/home/tholo/Scripts/mojo/arango-mojo/hello.mojo:4:42: error: module 'header' does not contain 'Headers'
from lightbug_http.header import Header, Headers
^
/home/tholo/Scripts/mojo/arango-mojo/.magic/envs/default/bin/mojo: error: failed to parse the provided Mojo source module

Desktop

OS: Arch Linux
magic --version magic 0.3.0
mojo --version mojo 24.5.0 (e8aacb95)
mojoproject.toml

 [project]
authors = ["dasTholo<EMAIL_ADDRESS>channels = ["conda-forge", "https://conda.modular.com/max", "https://repo.prefix.dev/mojo-community"]
description = "Add a short description here"
name = "arango-mojo"
platforms = ["linux-64"]
version = "0.1.0"

[tasks]

[dependencies]
max = ">=24.5.0,<25"
lightbug_http = ">=0.1.3,<0.2"

@dasTholo thanks for the detailed report! This is probably because main got updated yesterday but new package version wasn't published to the prefix channel. Just updated, you can also trying changing [dependencies] to lightbug_http = ">=0.1.4,<0.2". Worked for me on a test VM, let me know if this doesn't solve the problem. Here's an example that works for me:
from lightbug_http import *
from lightbug_http.sys.client import MojoClient

fn test_request(inout client: MojoClient) raises -> None:
    var uri = URI.parse_raises("http://httpbin.org/status/404")
    var headers = Header("Host", "httpbin.org")

    var request = HTTPRequest(uri, headers)
    var response = client.do(request^)

    # print status code
    print("Response:", response.status_code)

    # print parsed headers (only some are parsed for now)
    print("Content-Type:", response.headers["Content-Type"])
    print("Content-Length", response.headers["Content-Length"])
    print("Server:", to_string(response.headers["Server"]))

    print(
        "Is connection set to connection-close? ", response.connection_close()
    )

    # print body
    print(to_string(response.body_raw))

fn main() -> None:
    try:
        var client = MojoClient()
        test_request(client)
    except e:
        print(e)

Interestingly, with your test code I'm getting URI is nil. Will investigate this and address in a separate issue.

Thank you very much!
Yeah, upgrade will help! But yes "URI is nil" no result or status codes


split put and put_multipart

Instead of doing this, I'm going to try and keep them the same function. The only difference is that PutMode only exists for put and not put_multipart


Doc misleads into supporting ResourceUsageStats for Managed Databases

I think ResourceUsageStats is not supported for Managed Databases. However, this document seems it is supported for instance databases. Thus, I suggest that ResourceUsageStats should not be mentioned here.

https://learn.microsoft.com/en-us/azure/azure-sql/database/metrics-diagnostic-telemetry-logging-streaming-export-configure?view=azuresql&tabs=azure-portal#databases-in-azure-sql-managed-instance

@akkawano11235 -- thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@dimitri-furman -- please look into this issue.

@akkawano11235 thanks for reporting this misleading information. We have updated this documentation article to clarify; the update will be visible in a few hours.

#please-close


cannot import sklearn using annaconda

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
Version and main components

Spyder Version: 3.1.2
Python Version: 3.5.2
Qt Versions: 5.6.0, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.3.0 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
pandas >=0.13.1  :  0.18.1 (OK)
numpy >=1.7      :  1.11.3 (OK)
sphinx >=0.6.6   :  1.4.6 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  1.5.3 (OK)
sympy >=0.7.3    :  1.0 (OK)
pylint >=0.25    :  1.5.4 (OK)

I'm sorry but this is an installation problem not related to Spyder. That's why I'm closing this issue.

sure, you know what, I just created a file named sklearn.py in my directory, that's why I cannot import sklearn from anaconda!!
Sent from my iPad

On 28 Jan 2017, at 4:31 PM, Carlos Cordoba<EMAIL_ADDRESS>wrote:
I'm sorry but this is an installation problem not related to Spyder. That's why I'm closing this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


fix missing dataset download in ensemble notebook

Discovered while implementing #575: demos/ensembles/ensemble-node-classification-example.ipynb is missing a download() of the dataset, so fails to run.

Codecov Report

Merging #816 into develop will decrease coverage by 0.3%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #816      +/-   ##
===========================================
- Coverage    82.53%   82.22%   -0.31%
===========================================
  Files           47       46       -1
  Lines         5295     5169     -126
===========================================
- Hits          4370     4250     -120
+ Misses         925      919       -6

Impacted Files
Coverage Δ

stellargraph/mapper/sampled_link_generators.py
92.3% <0%> (-1.23%)
:arrow_down:

stellargraph/data/explorer.py
97.27% <0%> (-0.15%)
:arrow_down:

stellargraph/random.py

stellargraph/data/unsupervised_sampler.py
92% <0%> (+0.45%)
: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 bda13cc...47f67f5. Read the comment docs.


Feature request: netty InternalLogger implementation

Hi
There is should be io.netty.util.internal.logging.InternalLogger implementation to create tinylog factory for netty logging

Would you be able to submit a pull request? However you can also use Slf4JLoggerFactory, if you use the SLF4J binding for tinylog 2.

i haven't any implementation of this thing, will try to use SL4J binding


Instrumenting servers / paths to add metrics doesn't work when view returns an error

I was trying to add instrumentation around the Atreugo server and/or around Paths.
I created a middleware and used UseBefore and UseAfter() to wire it, and the middle ware registers the processing time, response status, errors, etc.
The problem is, that when a view returns an error, the After middleware doesn't get called, and instead (I think) only the ErrorView is called.
This for me sounds like a really problematic flow as it prevents you from writing such middlewares unless I'm missing something here.
BTW, I saw a few closed tickets here about implementing something similar, but the code snippets in the tickets won't work, as the After middleware won't get called.
Is there a way to fix this?
Am I doing it wrong?

Hi @eranharel,
Sorry for delayed answer.
The workflow is interrupted when you return an error in a middleware (before/after) or view and the ErrorView will be executed, so if you want to continue the chain execution you should save the error (with ctx.SetUserValue("viewErr", err)) an not return it, proccessing that error in your after middleware.
Keep in mind, if you return an error in a view or before middleware, and later return another error after, the execution could be unstable due to don't know when stop the execution of the request, and the atreugo interface would be more complex.
Thanks for use Atreugo 😉


Can't build jaxlib after commit 843e22

Dear jax team,
the jaxlib compilation (with cuda) fails after commit 843e22dd1732d0d537bd6b01656af2978a8e2082 with the message no such attribute 'exec_properties' in 'py_test' rule. If I checkout commit 4c30c0285c509764f3b1b937b8a6bc9f9b8dd061 the compilation works fine.
I'm sure that's an error on my end, but I have not been able to identify the problem. Could you help me with a guess what this commit changed specifically?
I'm on Ubuntu 18.04 and Cuda 10.0. Error log:
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:25:1: @org_tensorflow//tensorflow/compiler/xla
/python:xla_client_test: no such attribute 'exec_properties' in 'py_test' rule
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:104:1: @org_tensorflow//tensorflow/compiler/xl
a/python:semaphore_test: no such attribute 'exec_properties' in 'cc_test' rule
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:131:1: @org_tensorflow//tensorflow/compiler/xl
a/python:shared_device_buffer_test: no such attribute 'exec_properties' in 'cc_test' rule
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:241:1: @org_tensorflow//tensorflow/compiler/xl
a/python:bfloat16_test: no such attribute 'exec_properties' in 'py_test' rule
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:11:1: Target '@org_tensorflow//tensorflow/comp
iler/xla/python:xla_extension' contains an error and its package is in error and referenced by '@org_tensorflow//tensorflow/compiler/xla/python:xla_client'
ERROR: /buffer/schmid/bazel_cache_3/_bazel_ga76yow/64c20291e7b583e08f1dc1b7df40e5f2/external/org_tensorflow/tensorflow/compiler/xla/python/BUILD:11:1: Target '@org_tensorflow//tensorflow/comp
iler/xla/python:xla_client.py' contains an error and its package is in error and referenced by '@org_tensorflow//tensorflow/compiler/xla/python:xla_client'
ERROR: /buffer/schmid/jax/build/BUILD.bazel:23:1: Target '@org_tensorflow//tensorflow/compiler/xla/python:xla_client' contains an error and its package is in error and referenced by '//build:
install_xla_in_source_tree'
ERROR: Analysis of target '//build:install_xla_in_source_tree' failed; build aborted: Analysis failed

Just a wild guess, but maybe this has something to do with the version of bazel that is installed? See https://github.com/google/jax/pull/2137

Thank you for the idea! I used bazel 1.2.1 from the PR and it compiled successfully on the newest jax commit.

PR #2137 is merged; I also increased the minimum version to 1.2.1 so you should also get a better error message. Hope that helps!


Update the wording in Reuse content (How to use) section

Description
We need to replace "Reuse content" title with "How to use" and add a descriptive paragraph to give more context about how to give attribution.
Screenshots
Desktop (image)

Mobile (image)

Texts
The image description text:
Visit the image’s website to download and use it. Make sure to credit the creator by showing the attribution information where you are sharing your work.
The audio description text:
Visit the audio’s website to download and use it. Make sure to credit the creator by showing the attribution information where you are sharing your work.
Mockups
Presentation: page 7
Figma: desktop and mobile
@panchovm: Please note that Grammarly proposed to remove the second to from this sentence:
Visit the image’s website to download and to use it.

A quick question, does this issue contain all of the changes in the copy/text? I think one PR would suffice to include most of the changes (ex. from use this image to Get this image with an icon ) etc.
I'll have a PR ready by today's slack meeting

We try to create small PRs that are easy to test and review, @ramadanomar, whenever possible.
The changes to the 'Get this ' are already done in a separate PR, so this issue is specifically about the "How to use" section.

Grammarly suggestion is correct @obulat, let's go with what you described.


Cannot load from metaphlan

Hello, I am trying to use the function "loadFromMetaphlan" mentioned in the tutorial.
But the function doesn't seem to be implemented.
Can you please fix it?
Otherwise, to use shotgun metagenomic data, I would need to first convert the metaphlan to a phyloseq. Is that correct?
Thank you.

Hi! This function is (or should be) implemented in the latest versions of mia, I just confirmed.
Can you check your version of mia (e.g. R sessionInfo())?
Latest Bioc devel version installs with this (although this functionality is in the current release as well):
BiocManager::install("mia", version="devel")

Hello, I used the Bioconductor, and the version is 1.2.7 (it was installed yesterday).

Hi - the current Bioc release version is 1.6.0 and the Bioc development version is 1.7.0; the very latest bleeding edge development version in Github is 1.7.4.
It seems that you are having an outdated version for some reason.
We can try to troubleshoot that if you provide the output of sessionInfo() in R and the exact installation command.
Or you can try to install the latest version from Github with:
remotes::install_github("microbiome/mia")
However Bioc installation might be more stable with dependencies, it should install with
BiocManager::install("mia", version="devel")
but if you tried that already, then I would next check which R version you are using (R.Version())

Hey, I managed to install the 1.7.4 using the install_github.
Thank you very much. =)

Great,all feedback is very welcome :)


Instrumentation details not printed for persistence model

According to the documentation, adding require 'elasticsearch/rails/instrumentation' to the config/application.rb should add two bits of instrumentation details: A line detailing what query was run, and a bit in the controller response saying how much of the time was doing the elasticsearch query.
Started GET "/traces" for ::1 at 2015-12-08 15:55:21 -0600
Processing by TracesController#index as HTML
  User Load (0.3ms)  SELECT  "users".* FROM "users" WHERE "users"."uid" = $1 LIMIT 1  [["uid"<EMAIL_ADDRESS>  Rendered helpers/_fielded_object.json.jbuilder (0.3ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.0ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_fielded_object.json.jbuilder (0.2ms)
  Rendered helpers/_paginated_results.json.jbuilder (7.2ms)
  Rendered traces/index.json.jbuilder (8.4ms)
Completed 200 OK in 111ms (Views: 9.8ms | ActiveRecord: 0.7ms | Elasticsearch: 0.0ms)

In practice, it looks like the bit that prints the query being run isn't getting output, and the time in the final statement seems to always be 0.0ms.  Not sure if I have something configured wrong or if this is a bug

@Animeye Please have a look at https://github.com/elastic/elasticsearch-rails/issues/305#issuecomment-68552192, you need to add a small wrapper snippet to get the instrumentation to work.

@toao As I understand #305, that is for the repository model. i'm not sure if he's using the AR style or the repository style. Either way, I have the same problem in the AR style I'm trying to troubleshoot.

As I understand it, there is the same underlying issue for both the AR and repository style.
I was able to get some instrumentation hooked up by entering into the gateway, as shown here https://github.com/elastic/elasticsearch-rails/issues/269#issuecomment-69755606, but as noted in that issue, things don't work "out of the box" or as documented in the README

In case it can help anyone, I was able to resolve this in an AR style model (Persistence::Model) as below.
class Article
    include Elasticsearch::Persistence::Model

    gateway do
      def search(query, options = {})
        ActiveSupport::Notifications.instrument "search.elasticsearch", name:
          'Search', klass: host, search: query do
          super
        end
      end
    end

end

It may be worth mentioning that that will only instrument the "search" method, while activerecord users may prefer/expect that other common methods (find, save, destroy) to also contribute to the logged timing.  These methods can get instrumented in a similar fashion (entering into the gateway and overloading each method).

Also worth noting that there doesnt seem to be a consistent way of doing this.  Overloading the methods in gateway works for some methods, but not for others (such as find_in_batches)
Overloading self.find_in_batches at the class level works, but does wind up including time spent in the block as time spent in elasticsearch.  Looking at the source, it looks like a more "correct" solution would require overloading the search and scroll methods of the client object?


identify connections by their local addr when adding to the multiplexer

Fixes #2108.
Fixes the issue described in https://github.com/caddyserver/caddy/pull/2727#issuecomment-526856566.
@mholt Could you run your test with this branch?

Codecov Report

Merging #2111 into master will decrease coverage by 0.23%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2111      +/-   ##
==========================================
- Coverage   85.75%   85.52%   -0.23%
==========================================
  Files         101      101
  Lines        7353     7432      +79
==========================================
+ Hits         6305     6356      +51
- Misses        779      800      +21
- Partials      269      276       +7

Impacted Files
Coverage Δ

multiplexer.go
100% <100%> (ø)
:arrow_up:

http3/server.go
55.97% <0%> (+2.73%)
: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 d1489e5...81be522. Read the comment docs.

Thanks for the quick work!
Unfortunately, the problem still manifests. It seems to just not like being handed a PacketConn that it has seen before. (I'm creating all new http3.Servers, by the way, just giving each one the same underlying PacketConn. I'm not reusing the http3.Server instances.)

Aha, I was finally able to get it to happen with QUIC_GO_LOG_LEVEL=debug enabled on the server! It (the server) reports this error over and over before the client panics:
muxer received a packet with an unexpected connection ID 0xd78dd894048b0c33d84d9c738a31b348

over the course of a few seconds.

Great, that's a different error now!
Maybe #1743 is back? I'll look into it right now.

Ah, yes, now that you mention it, it does seem similar to what we saw in Caddy 1.
Fortunately I can hold out on merging it into Caddy 2 until we get it figured out. :) Let me know how I can help! Thank you for looking into it more! It'll be exciting to get this into people's deployments.

Can you help me reproduce it?
I'm running Caddy with the config file that you gave me in https://github.com/caddyserver/caddy/pull/2727#issuecomment-526856566 with this branch of quic-go, and I've reloaded the config 5 times or so, both with and without debug mode. In both cases the downloads work without any problems, I'm not getting any error messages.

@marten-seemann Sure, I'll see what I can do. I just pushed the latest, sorry, I forgot to do that >_< -- try pulling from the http3 branch of the caddy repo. The latest commit should be this: https://github.com/caddyserver/caddy/commit/dde3cd102080e7d80bdf6d45606dbe67bf77df4b
(it's where I keep track of the listeners myself and close them, since closing the http3 server doesn't close the listener).

@mholt I pulled dde3cd1, but everything still works here. I applied a few new configurations, and all transfers are succeeding.

@marten-seemann
Hmm...
here's my full output (from the client) - in particular, this is the output from the run where I was able to reproduce it with the debug mode enabled at the server:
GET https://example.local:2443
client Starting new connection to example.local ([::]:52577 -> <IP_ADDRESS>:2443), source connection ID (empty), destination connection ID 0xd78dd894048b0c33d84d9c738a31b348, version TLS dev version (WIP)
client Destroying session 0xd78dd894048b0c33d84d9c738a31b348: NO_ERROR: Handshake did not complete in time
client Connection (empty) closed.
panic: Get https://example.local:2443: NO_ERROR: Handshake did not complete in time

goroutine 3 [running]:
main.main.func1(0xc0001162a0, 0x1560a80, 0xc000020c90, 0xc000024641, 0xc000118370, 0x7ffeefbffa13, 0x23)
	/Users/matt/Dev/src/github.com/lucas-clemente/quic-go/example/client/main.go:50 +0x434
created by main.main
	/Users/matt/Dev/src/github.com/lucas-clemente/quic-go/example/client/main.go:47 +0x462
exit status 2

(notice that the connection ID is the same as the one the server reported was unexpected)
Okay, after trying it a few more times, I'm pretty convinced it is easier to reproduce with debug mode OFF at the server. In a few separate but identical trials, I was able to have it succeed after reloads like you are reporting (enabling debug mode helped it seems). But once debug mode was disabled again, the problem reappeared.
What are your commands and their outputs? Mine, for the server (you can see the reload happen in the middle of this output):
$ go run main.go run --config ../../_gitignore/v2/caddy2.json
2019/09/04 00:10:42 Caddy 2 admin endpoint listening on localhost:2019
2019/09/04 00:10:42 [INFO][cache:0xc0000f21e0] Started certificate maintenance routine
2019/09/04 00:10:42 [INFO][example.local] Skipping automatic certificate management because a certificate with that SAN is already loaded
2019/09/04 00:10:42 [INFO] Enabling automatic HTTPS certificates for []
2019/09/04 00:10:42 [INFO] Enabling automatic HTTP->HTTPS redirects for [example.local]
2019/09/04 00:10:42 [DEBUG] Started HTTP/3 listener on :2443
2019/09/04 00:10:42 Caddy 2 serving initial configuration
2019/09/04 00:10:45 [INFO][cache:0xc00016c730] Started certificate maintenance routine
2019/09/04 00:10:45 [INFO][example.local] Skipping automatic certificate management because a certificate with that SAN is already loaded
2019/09/04 00:10:45 [INFO] Enabling automatic HTTPS certificates for []
2019/09/04 00:10:45 [INFO] Enabling automatic HTTP->HTTPS redirects for [example.local]
2019/09/04 00:10:45 [DEBUG] udp/:2443: Usage counter should not go above 2 or maybe 3, is now: 2
2019/09/04 00:10:45 [DEBUG] Started HTTP/3 listener on :2443
2019/09/04 00:10:45 [INFO][cache:0xc0000f21e0] Stopped certificate maintenance routine
2019/09/04 00:10:45 [DEBUG] Fake-closing underlying packet conn

For the client (before config reload):
$ GO111MODULE=on go run main.go -insecure https://example.local:2443
GET https://example.local:2443
client Starting new connection to example.local ([::]:56208 -> <IP_ADDRESS>:2443), source connection ID (empty), destination connection ID 0x86a63fa879c159856af90eb596, version TLS dev version (WIP)
Got response for https://example.local:2443: &http.Response{Status:"200 OK", StatusCode:200, Proto:"HTTP/3", ProtoMajor:3, ProtoMinor:0, Header:http.Header{"Server":[]string{"Caddy"}}, Body:(*http3.body)(0xc0000b01e0), ContentLength:0, TransferEncoding:[]string(nil), Close:false, Uncompressed:false, Trailer:http.Header(nil), Request:(*http.Request)(nil), TLS:(*tls.ConnectionState)(nil)}
Request Body:
Booyah: FOO URI: '/test/abc?foo=bar' PATH: '/test/abc' QUERY: 'foo=bar'
client Closing session.
client Connection (empty) closed.

The config reload:
$ curl -X POST -d @caddy.json -H "Content-Type: application/json" "http://localhost:2019/load"

Then, the client again after the reload:
$ GO111MODULE=on go run main.go -insecure https://example.local:2443
GET https://example.local:2443
client Starting new connection to example.local ([::]:56209 -> <IP_ADDRESS>:2443), source connection ID (empty), destination connection ID 0x0c59e95b17c0be8760ee5b80bbc9cda419, version TLS dev version (WIP)
client Destroying session 0x0c59e95b17c0be8760ee5b80bbc9cda419: NO_ERROR: Handshake did not complete in time
client Connection (empty) closed.
panic: Get https://example.local:2443: NO_ERROR: Handshake did not complete in time

goroutine 34 [running]:
main.main.func1(0xc0000bf3e0, 0x1560a80, 0xc0000beb10, 0xc0000bc599, 0xc0000bc910, 0x7ffeefbffa13, 0x23)
	/Users/matt/Dev/src/github.com/lucas-clemente/quic-go/example/client/main.go:50 +0x434
created by main.main
	/Users/matt/Dev/src/github.com/lucas-clemente/quic-go/example/client/main.go:47 +0x462
exit status 2

I tried it again a couple of times and now the error occurs. Seems to be pretty shaky. I'll keep digging, this definitely should not happen at all.

@mholt I'm really having problems reproducing this. I got it once, and it never occurred on my ~100 attempts afterwards.
The error you're getting
muxer received a packet with an unexpected connection ID 0xd78dd894048b0c33d84d9c738a31b348

should only occur if no server is set: https://github.com/lucas-clemente/quic-go/blob/d1489e5045f4270ebd7be356fee9951a12c0b450/packet_handler_map.go#L214-L217
so I'm wondering if there might be a deadlock in SetServer https://github.com/lucas-clemente/quic-go/blob/d1489e5045f4270ebd7be356fee9951a12c0b450/packet_handler_map.go#L104-L108
I simplified the shutdown logic a bit in #2115. I doubt that it fixes the issue, but could you give it a try and run it with this branch?

I'm glad you were able to reproduce it! ... once, anyway. I feel a little better knowing that. 😅  I checked out that branch and did some trials with it but unfortunately the behavior was the same and the problem still manifests. (With debug mode enabled at the server I was able to see the "unexpected connection ID" error again.)
If it helps any, so you can see what we're doing in Caddy, the only place we interact with the http3 servers is here<EMAIL_ADDRESS>Old servers are garbage-collected on the next config reload because the App goes out of scope.

If this helps:
You can see references to h.server here: https://sourcegraph.com/github.com/lucas-clemente/quic-go/-/blob/packet_handler_map.go#L214:7&tab=references
The only other place that sets server to nil is here, which leads into the function that you pointed out in that linked PR: https://sourcegraph.com/github.com/lucas-clemente/quic-go@d1489e5045f4270ebd7be356fee9951a12c0b450/-/blob/packet_handler_map.go#L112:4
But I'll see if I have a chance to look into potential deadlock there.

It looks like SetServer is in fact called here: https://sourcegraph.com/github.com/lucas-clemente/quic-go@d1489e5045f4270ebd7be356fee9951a12c0b450/-/blob/server.go#L193:17 and the debug output shows that it is listening. I'm a little lost as to where sessionHandler goes (it doesn't look like it is ever used) -- but I see lots of interfaces. Still, is it possible that the sessionHandler which has a server added to it is a different copy from the one that is used to handle requests? (Just throwing ideas out there, feel free to ignore me haha, I don't have much of an idea what is going on this deep.)

Somewhat significant finding, I think: If I comment out these lines in Caddy<EMAIL_ADDRESS>Then the problem appears to go away. In other words: NOT closing the http3.Server and only "closing" the listener (remember, in a reload scenario in Caddy, "Close()" on the PacketConn doesn't actually close the underlying listener) seems to work. In other words, not closing anything seems to work. I also made changes to the config on each reload (changed the response body) and verified that every time, the latest changes were printed by the client.
So, am I leaking resources then? But why is that working?
I hope that helps!

@mholt You're right, it's a bit confusing. The complexity is necessary to be able to handle a server and outgoing connections on the same net.PacketConn...

Then the problem appears to go away. In other words: NOT closing the http3.Server and only "closing" the listener (remember, in a reload scenario in Caddy, "Close()" on the PacketConn doesn't actually close the underlying listener) seems to work

That's an interesting observation. I think we might have had a logical race in the http3.Server, but I'm not entirely sure. I just fixed another issue with the same code (as of #2122, the server can now serve on multiple packet conns at the same time), which might have solved that issue as well. Running Caddy and reloading the config at least didn't cause any further hangs any more.
I'm now waiting for @lucas-clemente to review all these PRs. It's getting confusing now with so many PRs stacked on top of each other.

@mholt Now that all the relevant PRs are merged, can you please try again?
And since this issue here is now closed, let's move the discussion back to https://github.com/caddyserver/caddy/pull/2727.


Figure out what was wrong with the assertion in RaceConfig::ConfigurePlayers

Context for this, an assertion was firing in RaceConfig::ConfigurePLayers during battle mode, however didn't seem to break anything with it removed.
See: https://github.com/stblr/mkw-sp/pull/540


The Big Emigration (ciao sdks)

Description
Removes all of /packages/sdks,  packages/cli, etc  . These will now live at celo-org/developer-tooling see https://github.com/celo-org/developer-tooling/pull/12 for that diff
This is not the end state of the transition, rather it is just the largest chunk. But its better to be swift rather than have big changes lingering for a long time
Follow ups will

upgrade yarn, linting, typescript etc,
remove more packages from root packge.json (once i figure out which are actually needed)
do more clean up for anything left

Other changes
fixes little issue in pre-push script.
fixes little issue in dev-utils package.json
noteworthy

typescript package remains because as soon as we upgrade to eslint-tyescript we can just delete it.
dev-utils remains. I think we might move to a separate repo later.

Tested
existing tests
Related issues

Fixes #10800
part of https://github.com/celo-org/celo-labs/issues/1022

Backwards compatibility
Documentation
n/a

most of the "failing" tests are those that this pr removes. the other is known to fail from time to time

since the packages which run the tests are being remove in the pr we need to turn off the requirement that they pass to merge
@lvpeschke Can you help here?


Command loading

It seems a couple commits got duplicated because of a bad rebase or something?

hmmm odd O_o

As far as I can tell, they're identical. Just with different histories or something, so they have different hashes. Not really sure why.

wait did you merge a branch into itself? or am i reading that merge commit wrong?

Um... yeah, there was a merge conflict between local and remote. Basically, the second to last commit was based on the 2 local commits, which (for some reason) didn't match the remote commits, so there was a conflict in which I basically dropped anything the remote wanted to add.

Do you want to see if there is an easy way to fix the history or shall i just merge it?

You can if you want, but I can't be bothered, tbh.


Fix tests for multiprocessing pickle app and pickle blueprint

The old tests were not quite checking for the right thing.
Fixing the test does not change Sanic code, does not expose any bugs, or fix any bugs.

The original tests were contributed by me here: https://github.com/huge-success/sanic/commit/5cf2144b3ff3438afe0c9f7dbcbcb29cb94c48e0 so its my own bad test I'm fixing.


Command to set FLASK_APP variable in flask/examples/flasker README is not compatible with released version of Flask(0.12.2)

Expected Behavior
In README of the example of flask/examples/flasker has following step to set  FLASK_APP env variable:
export FLASK_APP=flaskr.factory:create_app()
Next step is to initialize the DB:
And it should produce output as follows:
Initialized the database.
Actual Behavior
This method to set FLASK_APP env is not compatible with the current released version of flask.
If set this way following error is observed while running next step "flask initdb":
RuntimeError: Failed to find application in module "flaskr.factory".
The issue is resolved in master branch. Still, this being starting example for many people, should be corrected.
flask initdb
Traceback (most recent call last):
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/bin/flask", line 11, in <module>
    sys.exit(main())
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/flask/cli.py", line 513, in main
    cli.main(args=args, prog_name=name)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/flask/cli.py", line 380, in main
    return AppGroup.main(self, *args, **kwargs)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/click/core.py", line 1061, in invoke
    cmd_name, cmd, args = self.resolve_command(ctx, args)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/click/core.py", line 1100, in resolve_command
    cmd = self.get_command(ctx, cmd_name)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/flask/cli.py", line 351, in get_command
    rv = info.load_app().cli.get_command(ctx, name)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/flask/cli.py", line 237, in load_app
    rv = locate_app(self.app_import_path)
  File "/Users/prasadghagare/Desktop/projects/repo/.my_env/lib/python2.7/site-packages/flask/cli.py", line 109, in locate_app
    % module)
RuntimeError: Failed to find application in module "flaskr.factory"

Environment

Python version: 2.7.13
Flask version: 0.12.2
Werkzeug version: 0.7

I'm currently seeing this issue too.  What is the recommended solution to this problem?

Use the same tag in the repository as the version of the docs you're reading / version you have installed. You're looking at the dev version right now.


Dragging Image Duplicates

Describe the bug
Selecting and dragging an image within the Editor duplicates it.
To Reproduce
Steps to reproduce the behavior:

Go to the demo
Click on the Concerto Model Image or insert a new one
Drag it to a new position
The image is duplicated

Expected behavior
Only the position of the image should be changed.
Screenshots

@irmerk Since the issue still exists so shall I take over this issue.

@irmerk This issue doen't exist any more.


I am doing pub get when i am getting below error.

I am doing pub get when i am getting below error.
Because every version of sqflite from git depends on sqflite_common >=2.2.1+1 <4.0.0 and sqflite_sqlcipher >=1.1.0 <2.0.0-nullsafety.0 depends on sqflite_common ^1.0.1, sqflite from git is incompatible with sqflite_sqlcipher >=1.1.0 <2.0.0-nullsafety.0.
And because every version of floor from path depends on sqflite_sqlcipher ^1.1.1, sqflite from git is incompatible with floor from path.
So, because floor_chiper depends on both floor from path and sqflite from git, version solving failed.
pub get failed (1; So, because floor_chiper depends on both floor from path and sqflite from git, version solving failed.)

@jitendraAn thanks for reporting
I will check that again
Regards


Update installation steps

In order to provide a way to have product also update content and also troubleshoot the frontend dev environment set up for open source folks, this is some initial feedback on @katherinedm-usds's experience getting the environment set up.
Thanks @katherinedm-usds for finding some clear holes in the process!

I’m digging the latest comment on breaking up by role!
Vim Shah (he/him/his)
Software Engineer
United States Digital Service
Work:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
From: Shaun Verch @.>
Reply-To: usds/justice40-tool @.>
Date: Monday, December 6, 2021 at 4:08 PM
To: usds/justice40-tool @.>
Cc: "Shah, Vim K. EOP/OMB" @.>, Author @.***>
Subject: Re: [usds/justice40-tool] Update installation steps (PR #859)
@sverchdotgov commented on this pull request.
From trying to understand this all in #949https://github.com/usds/justice40-tool/issues/949, I believe that the local dev loop involves pointing to the live s3 bucket. Would that be worth documenting here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/usds/justice40-tool/pull/859#pullrequestreview-824480323, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUSCJV33KMRX44NICM7HQWTUPUQ5JANCNFSM5HND35TQ.
Triage notifications on the go with GitHub Mobile for iOS<%3ehttps:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675%3c> or Android<%3ehttps:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub%3c>.


remove case forecasts

see https://github.com/european-modelling-hubs/covid19-forecast-hub-europe/pull/3319

Turns out there still is some case data from some countries so we could revert this ¯_(ツ)_/¯

Happy either way 🤷🏻


Runelite is preventing the ESC key from working

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 11
Bug description
Pressing the ESC Key does not open the inventory nor close out interfaces when on Runelite.
I've tried Safemode and the issue still persisted.
I then opened and verified that the issue does not occur on the the New Client.
Screenshots or videos
No response
RuneLite version
RuneLite version: <IP_ADDRESS>
Launcher version: 2.6.10
Oldschool revision: Rev 218
Logs
client.log
launcher.log

Are you certain the ingame setting to close interfaces when pressing Escape is enabled? If so, does this also happen when creating a new profile and switching to it?

@Nightfirecat

I clicked new profile and switched into it and the ESC key is working again!
What do you suggest at this point?

Install the Debug Helper plugin from the hub and click display keybinds to figure out what plugin Esc is bound to

Oh! @Felanbird
Party Ping hotkey: Escape
Thank you!


Added filter | Issue #255

fix #225
https://github.com/user-attachments/assets/50c69cc6-8fb7-4d08-8494-f2a48f29c2f3

@TejasNasre review this


Problems installing library with setuptools > 58.0.0

I was installing your package recently and it came to my attention that installation fails.
A simple test to replicate:
Adams-MBP:programming claim$ mkdir testing_embedly_installation
Adams-MBP:programming claim$ cd testing_embedly_installation/
Adams-MBP:testing_embedly_installation claim$ python3 -m venv venv
Adams-MBP:testing_embedly_installation claim$ source venv/bin/activate
(venv) Adams-MBP:testing_embedly_installation claim$ pip install -U setuptools
Collecting setuptools
  Using cached https://files.pythonhosted.org/packages/11/b9/adac241e2c4aca7ae4ddd86d3c18227667665b6e7eac550695bfc50c7e3d/setuptools-60.6.0-py3-none-any.whl
Installing collected packages: setuptools
  Found existing installation: setuptools 41.2.0
    Uninstalling setuptools-41.2.0:
      Successfully uninstalled setuptools-41.2.0
Successfully installed setuptools-60.6.0
(venv) Adams-MBP:testing_embedly_installation claim$ python
Python 3.8.3 (v3.8.3:6f8c8320e9, May 13 2020, 16:29:34)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import setuptools
>>> setuptools.__version__
'60.6.0'
>>> ^D
(venv) Adams-MBP:testing_embedly_installation claim$ pip install embedly
Collecting embedly
  Using cached https://files.pythonhosted.org/packages/d7/94/2e387186617fe450fd21da3fd08c4ea1c68914c21622e939a892755620bc/Embedly-0.5.0.tar.gz
    ERROR: Command errored out with exit status 1:
     command: /Users/claim/programming/testing_embedly_installation/venv/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/z9/p68f7pds4nz6lg602cxzhm040000gn/T/pip-install-5c2mwtoc/embedly/setup.py'"'"'; __file__='"'"'/private/var/folders/z9/p68f7pds4nz6lg602cxzhm040000gn/T/pip-install-5c2mwtoc/embedly/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base pip-egg-info
         cwd: /private/var/folders/z9/p68f7pds4nz6lg602cxzhm040000gn/T/pip-install-5c2mwtoc/embedly/
    Complete output (2 lines):
    Warning: 'classifiers' should be a list, got type 'tuple'
    error in Embedly setup command: use_2to3 is invalid.
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

there are 2 issues that need to be addressed here:

classifiers are a tuple (should be a list)
use_2to3 is no longer available in setuptools (since version 58.0.0 - more on that here: https://setuptools.pypa.io/en/latest/history.html#v58-0-0)

For now, I have created a fork that (I think?) addresses the problems as I'm not sure if this lib is still supported (but as my changes remove support for python2 completely, it would be a bit naughty to create a PR to this branch from it :D).

https://github.com/embedly/embedly-python/pull/32


Uncaught Error: ENOENT: no such file or directory, lstat '~/AppData\Local\atom\app-1.48.0\ and JS...

[Enter steps to reproduce:]

...
...

Atom: 1.48.0 x64
Electron: 5.0.13
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat 'C:\Users\vasin\AppData\Local\atom\app-1.48.0\ and JS - Full Reference'
At fs.js:122

Error: ENOENT: no such file or directory, lstat 'C:\Users\vasin\AppData\Local\atom\app-1.48.0\ and JS - Full Reference'
    at Proxy.realpathSync (fs.js:1512:7)
    at Proxy.fs.realpathSync (electron/js2c/asar.js:300:40)
    at TreeView.moveEntry (~/AppData/Local/atom/app-1.48.0/resources/app/static/<embedded>:14:943665)
    at TreeView.onDrop (~/AppData/Local/atom/app-1.48.0/resources/app/static/<embedded>:14:950594)
    at HTMLDivElement.<anonymous> (~/AppData/Local/atom/app-1.48.0/resources/app/static/<embedded>:14:924225)

Commands
     -0:27 tree-view:add-file (div.tool-panel.tree-view)
     -0:25.6.0 core:cancel (input.hidden-input)
     -0:23.3.0 tree-view:add-folder (div.tool-panel.tree-view)
     -0:20.2.0 core:confirm (input.hidden-input)

Non-Core Packages
atom-beautify 0.33.4
atom-ternjs 0.20.0
autoclose-html 0.23.0
csslint 1.2.0
emmet 2.4.3
language-ejs 0.4.0
pigments 0.40.2

Thanks for reporting this!
This has already been fixed in https://github.com/atom/tree-view/pull/1358. It is currently scheduled for release when v1.53.0 reaches Stable.


Add next auth login support for admin

Closes #225
Adds Auth0 login functionality and automatically updates admins schema with user login info

I forgot to ask but have you had the chance to test/try logging-in using a dummy user?
@rithikgavvala will let you take a final pass and approve. We will need some sort of follow-up task for https://github.com/GTBitsOfGood/gen-soln/pull/354#discussion_r516901687

Yeah I logged in as a user through Auth0 and it works fine and updates the database


Lab Refactor: Tier 2: XdrViewer

[ ] Create e2e test for XdrViewer
[ ] Convert XdrViewer page to TS and implement style guide; identify potential future design changes
[ ] Convert XdrViewer subcomponents
[ ] Replace Redux connect with hooks in XdrViewer; use hooks for state
[ ] Convert TransactionSigner subcomponents to TS and implement style guide; identify potential future design changes

Closing in favor of the v2 rebuild


[META] Add BWC tests to bwc_test.py

Plugin BWC tests are introduced for AD: https://github.com/opensearch-project/anomaly-detection/pull/158. Add support to wire these tests in test.py similar to integration tests.
Tasks:
P0

[x] #275
[x] #296
[x] #276
[ ] #572
[ ] #535
[ ] #277
[ ] #278

P1

[ ] Add support to parallelize multiple runs https://github.com/opensearch-project/opensearch-build/issues/243

[Triage] We will revisit the design when someone starts looking into BWC testing.
closing this for now, We will reopen this again later if required.


i18n (check-text) returning false positives

Config:
{
  "rules": {
    "i18n": [
      true,
      "check-id",
      "check-text"
    ]
  }
}

@Component({
  template: `
    <button type="button" mat-raised-button color="primary">
      <mat-icon>add</mat-icon> <!-- unexpected failure reported here -->
    </button>
    <button type="button" mat-raised-button color="primary">
      <mat-icon>
        visibility{{ passwordType === 'text' ? '_off' : '' }} <!-- unexpected failure reported here -->
      </mat-icon>
    </button>
  `
})
class Test {}

perhaps a config option could be added to provide a list of elements to ignore (eg: ["mat-icon"]).

I'm afraid we can't add this option without breaking changes. Thoughts @mgechev?

{
  "rules": {
    "i18n": [
      true,
      "check-id",
      "check-text",
      {
        "blacklist": ["mat-icon"]
      }
    ]
  }
}

Looks fine to me.

Any progress on this? Using @mgechev solution above makes the i18n rule not work at all rather than just ignoring mat-icon elements

Would appreaciate the blacklist rule. As by the moment this produces unnesscassary i18n attributes.

This would be awesome, unfortunately the blacklist property seems to break the template-i18n rule entirely. Does anyone happen to have the blacklist property suggested by @mgechev above working..? I'm wondering if there's something obvious I'm missing. Going through the codelyzer source code I don't think a blacklist property on this rule is possible, but I could be mistaken.
To recap, this works but picks up the mat-icon content in a false positive result:
"template-i18n": [true, "check-id", "check-text"]

This causes the rule itself to be inert (defeating the entire point of having the rule in the first place):
"template-i18n": [
  true,
  "check-id",
  "check-text",
  {
    "blacklist": ["mat-icon"]
  }
]

i can also confirm that blacklisting causes the rule to become inert
angular 8.1.3
tslint 5.18.0
codelyzer 5.1.0

I am @mgechev was just replying to @rafaelss95, suggesting a potential "non breaking change" solution and not pointing to the existing solution for the problem at hand.
I however would love to see this rule being updated with blacklist :+1:

any progress on this? I'm meanwhile thinking about writing a custom linter rule based on that one...

Would be nice to have the configuration option.
I worked around the problem by using string interpolation and a string literal.
So something like this:
- <mat-icon>arrow_back</mat-icon>
+ <mat-icon>{{'arrow_back'}}</mat-icon>

I have to be honest though.
I have no idea if this has any downsides to it.


Archive missing blocks

I had this syncing issue multiple times where the archive misses to index a few blocks. But if we check in the docker logs there are logs saying the block was saved still they are missing in the database thus we start to get a few issues with the processor since there are missing blocks/extrinsics/events.
This also requires to resync everything which is very slow. I would like to suggest to make some kind of "repair" command where it would check for missing blocks.
Not sure this is still relevant for the new archive implementation. Maybe it is not.

Closing as it is not relevant anymore with substrate-ingest


Use-NetMailConfig.ps1 doesn't work with PowerShell Core

PowerShell Core doesn't seem to have a machine.config:
$mcfg = [Configuration.ConfigurationManager]::OpenMachineConfiguration()
$may = if(Test-Path $mcfg.FilePath){'does'}else{'does not'}
"PowerShell v$($PSVersionTable.PSVersion) uses $($mcfg.FilePath), which $may exist"

PowerShell v6.2.0 uses C:\Program Files\PowerShell\6\Config\machine.config, which does not exist
Compare to PowerShell 5.1:
PowerShell v5.1.17134.590 uses C:\Windows\Microsoft.NET\Framework64\v4.0.30319\Config\machine.config, which does exist
On Windows, pwsh.exe could call powershell.exe for native access to the machine.config, but that's a pretty kludgy fix, and doesn't address other platforms.
Use-NetMailConfig.ps1 could fail on PS Core, or it could try finding .NET Framework machine.config value (either through powershell.exe or finding and parsing the config XML manually), or it could use it's own config, or some combination of these strategies.

This should now be fixed.


WindowWrapper.Default() suggestion

Hi, implementing ShareTarget in my app I found that basically WindowWrapper.Default() may not mean the same everyone expects: it returns just the first window. In case when the first operation was ShareTarget (which is a Hosted, not Main window), calling Default() will return those hosted, probably hidden window. And this will result in problems as such Windows do not allow calling nothing from them when hidden, resulting in ComException with "A COM call to an ASTA was blocked because the call chain originated in or passed through another ASTA. This call pattern is deadlock-prone and disallowed by apartment call control.".
I would suggest to retrieve Default windows as something like this:
ActiveWrappers.FirstOrDefault(w=>w.Window.CoreWindow == CoreApplication.MainView?.CoreWindow) ?? ActiveWrappers.FirstOrDefault();
This will at least try to get the MainWindows by default.
Does it make sense?

I will close this so you can "suggest" it with a Pull Request.


[CP Staging] Revert "Add Pay as Business option for invoices sent to an individual who is admin of their primary workspace"

Reverts https://github.com/Expensify/App/pull/43036
This reverts commit 35ae73155548e94dc23932384cc5ed2b5d3d3af4, reversing changes made to 9babbdaf565c374d5a0ca6cdf565a758904c393a.

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/44818
PROPOSAL:
Tests

Precondition:

User A and User B have their own workspace.

[User B] Click pay button > Pay the invoice as individual > Pay elsewhere.
[User A] Send another invoice.
[User A] Note that User A can send another invoice.
[User B] Delete all the workspaces.
[User B] Note that app crashes after deleting the workspace.
[User A] Send another invoice to User B in the same workspace chat.
[User A] Note that user cannot send invoice after User B deletes all the workspaces.
[User B] Go to Account settings > Workspaces.
[User B] Note that the workspace list shows blank workspace under Free plan.

Expected Result:
In Step 8, app will not crash after user B deletes all the workspaces.
In Step 10, User A should be able to send another invoice to User B after user B deletes all the workspaces.
In Step 12, there should not be any workspace in the list. (Or a workspace with an error about annual subscription)

[ ] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same as tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

In test step 4

[User B] Click pay button > Pay the invoice as business > Pay elsewhere.

@mountiny With this revert there would be no Pay the invoice as business, right?

@c3024 correct so lets just do the steps with pay as individual

https://github.com/Expensify/App/assets/36083550/e118ab60-26a2-424d-b8ea-4051bdab51a1

In Step 12, there should not be any workspace in the list.

Here, there will be at least one workspace with the annual subscription error as seen in the video you shared.

Testing Videos

Chrome
https://github.com/Expensify/App/assets/102477862/7b631041-0e9d-44bd-97f4-cbdb24f0d8d3

Desktop
https://github.com/Expensify/App/assets/102477862/dd411190-6621-4970-b6d7-318bf3e5858f

iOS mWeb
https://github.com/Expensify/App/assets/102477862/fbf226ca-b125-4e4b-a88f-a68855bf5d70

iOS

Android mWeb

Android

🚀 Deployed to staging by https://github.com/carlosmiceli in version: 9.0.4-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/carlosmiceli in version: 9.0.5-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 9.0.5-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.6-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅