Replace go-kit/log with base/log

As part of #120 I'm working on replacing internal/database with base/database package.
base/database depends on base/log while inside Customers project we use go-kit/log.
With this PR I'm replacing go-kit/log with base/log.

Codecov Report

Merging #219 into master will decrease coverage by 0.51%.
The diff coverage is 30.27%.

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   61.13%   60.61%   -0.52%
==========================================
  Files          49       50       +1
  Lines        2722     2745      +23
==========================================
  Hits         1664     1664
- Misses        743      766      +23
  Partials      315      315

Seems fine. Can we flip the order (log before .Problem(...))

This comment is just for keeping the history:
I have few questions/concerns:
First, Customers project uses base/http.Wrap which depends on go-kit/log and in order to replace it with base/log we may:

replace go-kit/log with base/log. This will require us to update other packages like Fed, Wire, Watchman.
extract Wrap method and modify it inside our project without touching any dependencies
create new method with name like ...WrapWithBaseLogger?

Second, replacing this:
logger.Log("accounts", fmt.Sprintf("problem saving account: %v", err), "requestID", moovhttp.GetRequestID(r))

with this:
logger.WithKeyValue("accounts", fmt.Sprintf("problem saving account: %v", err), "requestID", moovhttp.GetRequestID(r))

in many places I see that base/log.Log (which is main method) accepts one argument and it does not match to how we call it in Customers. In Customers we pass pairs of key values. My following questions:

should we change base/log.Log method to accept key/value pairs?
should we change what and how we log data in Customers?

we don't need requestID as it should be logged by default when we use Wrap method
how do we use package name keys like "accounts", "customers", "ofac", later from logs?

Codecov Report

Merging #219 into master will decrease coverage by 0.47%.
The diff coverage is 37.87%.

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   61.13%   60.65%   -0.48%
==========================================
  Files          49       50       +1
  Lines        2722     2758      +36
==========================================
+ Hits         1664     1673       +9
- Misses        743      771      +28
+ Partials      315      314       -1

@vxio I agree with your comment. as we discussed we should think how to move error logging into moovhttp.Problem.


KOGITO-238 - Separate consumer for model and process instance

@ricardozanini we dont need to get rid of if then :)


Fix for marking assert lines as safe bug

Calling _reduce_node_type from GDScriptParser::_parse_block for assert
was using a current class with a scope that didn't include all
functions. Now calling in GDScriptParser::_check_block_types uses the
right class type. We also now check the assert node message. The assert
line was added to the set_errors associated with assert, since before
the error would be reported on the next line
Reported: #38270
This will need to be cherry picked to the 3.2 branch.
Any issues or questions let me know, thanks.

Thanks!

Cherry-picked for 3.2.2.


Bug Report:

hivemind isnt showing up on the menu, nor the files
even after i beat every song in the game it wont show up.

Because that's not how it works


modify editor.fontFamily="新宋体"  Space is 4 ,but now it's 8

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

project-manager
ale
0.20.0

vscode-lua
gcc
0.1.2

Steps to Reproduce:

Reproduces without extensions: Yes/No

@syxcools can you share your settings and provide more details about "Space is 4 ,but now it's 8".


wiki: Crash Course ECS: small additions

I'd use this issue to gather small additions to the wiki.
Single component view´s getter does not require template parameters.
		auto viewPos = registry.view<Position>();
		auto pos& = viewPos.get(entity);

		auto viewVel = registry.view<const Velocity>();
		const auto& vel = viewVel.get(entity);

Yeah. This isn't the first time someone asks for it. It's worth adding a note to the wiki.
I'll do it as soon as I'm back home. Thanks.


[silgen] Add a new convention for allocating inits.

In an allocating init, we forward our inputs to the underlying initializing
init. Since initializing inits take that value at +1, the allocating init should
as well to avoid unnecessary copies. Previously, this was accomplished by having
the allocating init just use the default argument convention (i.e. +1). But in a
world where "normal" arguments are passed at +0, that doesn't work anymore. So
in this commit, I add a new convention just for allocating inits that takes
"normal" arguments at +0 and asserts if asked for self.
rdar://34222540

@swift-ci test and merge


Fixes the parser breaking when parsing beta endpoints

Overview
Fixes the http parser breaking when parsing beta endpoints.
Fixes #159

Thanks for fixing this!!! ❤


Add Repobeats (metrics) to README

Summary
Repobeats is a way to embed contributor analytics into your GitHub project's README.md
Metrics are everything. Data is powerful
Project Examples:

FreeForGeeks
Tabler
Projectdiscovery
ArchiSteamFarm
Growthbook

API Changes
Include a list of all API changes, additions, subtractions as would be required by your proposal. These APIs should be considered placeholders, so the naming is not as important as getting the concepts correct. If possible you should include some "example" code of usage of your new API.
Intended Use Case
Provide a detailed example of where your proposal would be used and for what purpose.

Closing as you have been spamming other repos with the same issue.

@brianlagunas,
I wanted to reach out to apologize for my recent actions regarding the issue I opened about Repobeats in multiple repositories. My intention was never to spam; rather, I was trying to contribute with what I thought was a valuable tool to enhance contributor analytics in open-source projects.
You can check that I closed all the issues. Again, My intention was never to spam.


Remove cdtime as a library dependency

cdtime is only directly imported in analysis_data_preprocess scripts.
There is already a dedicated step for creating an environment specifically for preprocessing:
https://github.com/E3SM-Project/e3sm_diags/blob/e15610c8bc74daa14a37fd9bce07520718a50a7b/analysis_data_preprocess/README#L3
It can be removed as a library dependency easily.
/conda-envs

closed by #627


don't use elevated PowerShell prompt to generate certificates

From a computer running Windows 10 or Windows Server 2016, open a Windows PowerShell console with elevated privileges. These examples do not work in the Azure Cloud Shell "Try It". You must run these examples locally.

If I am using a Powershell as Administrator I will not see the certificates in the system certificate manager or the user certificate manager.
If I am using the normal Powershell I am finding the certificates in the user certificate manager.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 63df9645-af2c-3cfc-8f65-413912d88efe
Version Independent ID: d2edf564-2c83-f1da-dbf0-dbc41c4948bf
Content: Generate and export certificates for Point-to-Site: PowerShell: Azure
Content Source: articles/vpn-gateway/vpn-gateway-certificates-point-to-site.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@wald-tq I was not able to reproduce this on a Windows Server 2016 image. After creating the root cert and the child cert, they both appeared in "Certificates - Current User -> Personal -> Certificates"
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


LibGit2Sharp.NativeBinaries	1.0.185

Type: Missing
Summary:
LibGit2Sharp.NativeBinaries	1.0.185
Details:
License link from Nuget site indicates license is GPL-2.0-with-GCC-exception
Resolution:
Link to copying file in Nuspec file also indicates license is GPL-2.0-with-GCC-exception
Affected definitions:

LibGit2Sharp.NativeBinaries 1.0.185

@mihouc GPL-2.0-with-GCC-exception is a deprecated license identifier and shouldn't be used. See https://spdx.org/licenses/GPL-2.0-with-GCC-exception.html.
This should be curated as GPL-2.0-only AND OTHER since the linking exception text is slightly different than the GCC exception 2.0 text.
LibGit2

In addition to the permissions in the GNU General Public License, the authors give you unlimited permission to link the compiled version of this library

GCC Linking Exception 2.0

In addition to the permissions in the GNU General Public License, the Free Software Foundation gives you unlimited permission to link the compiled version of this file

If you agree, I will update the PR.

I agree with the revision, thanks for catching the error
From: Candice Wright<EMAIL_ADDRESS>Sent: Friday, September 11, 2020 10:11 AM
To: clearlydefined/curated-data<EMAIL_ADDRESS>Cc: Michael Houck (Davis Wright Tremaine LLP<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [clearlydefined/curated-data] LibGit2Sharp.NativeBinaries 1.0.185 (#6044)
@mihouchttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmihouc&data=02|01|v-mihouc%40microsoft.com|0fdeb491780749b6612108d85675a850|72f988bf86f141af91ab2d7cd011db47|1|0|637354410615565485&sdata=fF1cavWa4Q%2FbspTCkKA5VKRrgKf29s7KomxKxkw81W8%3D&reserved=0 GPL-2.0-with-GCC-exception is a deprecated license identifier and shouldn't be used. See https://spdx.org/licenses/GPL-2.0-with-GCC-exception.htmlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fspdx.org%2Flicenses%2FGPL-2.0-with-GCC-exception.html&data=02|01|v-mihouc%40microsoft.com|0fdeb491780749b6612108d85675a850|72f988bf86f141af91ab2d7cd011db47|1|0|637354410615565485&sdata=Kt0Y1qUk57uYUBjJiNK7g%2Boj2KdcidIPYEiD4I3Kowo%3D&reserved=0.
This should be curated as GPL-2.0-only AND OTHER since the linking exception text is slightly different than the GCC exception 2.0 texthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fspdx.org%2Flicenses%2FGCC-exception-2.0.html&data=02|01|v-mihouc%40microsoft.com|0fdeb491780749b6612108d85675a850|72f988bf86f141af91ab2d7cd011db47|1|0|637354410615565485&sdata=DSRLv9T7Mhty4WmRFwnS7xKXafPwYuulGJ9VrryyEtw%3D&reserved=0.
LibGit2
In addition to the permissions in the GNU General Public License, the authors give you unlimited permission to link the compiled version of this library
GCC Linking Exception 2.0
In addition to the permissions in the GNU General Public License, the Free Software Foundation gives you unlimited permission to link the compiled version of this file
If you agree, I will update the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclearlydefined%2Fcurated-data%2Fpull%2F6044%23issuecomment-691213011&data=02|01|v-mihouc%40microsoft.com|0fdeb491780749b6612108d85675a850|72f988bf86f141af91ab2d7cd011db47|1|0|637354410615575482&sdata=PLLz1FMleU6rK1sSHTDLuzbTiep4zaHwQHoVcEFh7IA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPO32WUUZHAUSY6U3MK6LJ3SFJK2HANCNFSM4RDK7MGQ&data=02|01|v-mihouc%40microsoft.com|0fdeb491780749b6612108d85675a850|72f988bf86f141af91ab2d7cd011db47|1|0|637354410615575482&sdata=VrGe55muhXY7buh%2BCY0lNWyFoaB42cZBATDRe5lZCQQ%3D&reserved=0.


Create codeql-analysis.yml

@siraben Do you think you could merge this into the existing workflow and have it work as expected with the nix build?

@siraben Do you think you could merge this into the existing workflow and have it work as expected with the nix build?

@siraben poke?


As a user, I can create appointments in my ContaX calendar,

so that I can keep track of my available timings and where I need to be

Closed as duplicate


Tickets: Added remote tickets

Added remote tickets on the website.

Thanks Aris :heart:

The conflict is gone, we can continue with this one

Also, we can remove these lines now :tada:
https://github.com/EuroPython/website/pull/653/files#diff-8eca8cd7b7e03f2f14f1c3955c0921c464f0980727e8e792fd6121b6607c68a0R144-R151

Good, thanks for the comments! 🤗


Expose CORS settings

Hi guys
UAA's CORS settings can be configured in uaa.yml (possible values are visible in spring-servlet.xml).
We're especially interested in overwriting the origins and credentials.
However, the BOSH release does not expose these config values. Can this be added?
We're happy to provide a PR if welcomed.

Hi @MatthiasWinzeler we'd be happy to take a pull request for this. Would be happy to understand as well what use case is driving the need to configure these.
Thanks,
Tian

@tnwang For our web portal, we're trying to submit a POST XHR request to /oauth/token (to log in the user).
Since this transmits credentials, we need UAA to set Access-Control-Allow-Credentials to true - that's why we initially thought we need to set credentials. However, we figured out that UAA sets this when we declare the request as XHR using X-Request-With. This also sets Access-Control-Allow-Origin correctly.
There is one last roadblock: UAA's default XHR CORS policy disallows POST request. We would like to change that.
Even though we only need to set cors.xhr.allowed.methods now, it's probably easiest and most flexible if we expose all config items (as optional) in the BOSH release:
- cors.default.allowed.uris
- cors.default.allowed.origins
- cors.default.allowed.headers
- cors.default.allowed.methods
- cors.default.allowed.credentials
- cors.default.allowed.max_age
- cors.xhr.allowed.uris
- cors.xhr.allowed.origins
- cors.xhr.allowed.headers
- cors.xhr.allowed.methods
- cors.xhr.allowed.credentials
- cors.xhr.allowed.max_age

What do you think about this?

Hi - can we reopen this? You have a PR so it would be great to get that approved and merge.  Thanks!


Wihslist: Drawing paths

Hi @grantmcdermott! I'm in the process of developing a couple of packages centred on animal movements. Lots of the data are paths, and I normally draw them with ggplot's geom_path. But for the package documentation itself, I'm looking for a more lightweight alternative - hence I'm here. 😊
So just to add to the list: It would be really great to have the ability to draw a path (as opposed to line). If it's already possible, then great! (And then you can just change this issue to be one of updating the documentation). 😉

I think what you are looking for is the default behavior in base R and hence also in tinyplot: The plotted values are not sorted by the x-variable automatically. If you want sorting, you need to do that yourself.
Let's consider a simple example data with x and y coordinates sorted by time within group.
d <- data.frame(
  x = c(1, 3, 1, 5, 6, 4, 5, 1),
  y = c(1, 4, 3, 3, 2, 1, 4, 2),
  time = rep(2021:2024, 2),
  group = factor(rep(c("A", "B"), each = 4))
)

Producing a tinyplot of y vs. x by group is essentially what geom_path would give you:
tinyplot(y ~ x | group, data = d, type = "o", col = palette.colors()[2:3], lwd = 2, pch = 19)
ggplot(d, aes(x = x, y = y, group = group, color = group)) + geom_path() + theme_bw()

If you want the result of geom_line() for the same data, you would have to sort d by x first.
Or here we can take the variable time for illustration because it is already sorted with respect to that:
tinyplot(y ~ time | group, data = d, type = "o", col = palette.colors()[2:3], lwd = 2, pch = 19)

That's great to know, thanks a bunch @zeileis! That can be added to the list of ggplot functions that tinyplot covers then! :-)

What @zeileis said ;-)
Another (albeit redundant) example would be:
ggplot(iris, aes(x = Petal.Length, y = Sepal.Length, col = Species)) + geom_line()
ggplot(iris, aes(x = Petal.Length, y = Sepal.Length, col = Species)) + geom_path()
plt(Sepal.Length ~ Petal.Length | Species, iris, type = "l")


[scripts/makehtml] Fix displaying of registers with the same addressOffset and parsing of addressOffset

This pull request fix two bugs in the makehtml.py script:

When parsing the addressOffset value, the value is parsed as base16 even if it's in base10.
When two registers have the same addressOffset, one of the two is suppressed

The first bug is fixed by checking if the addressOffset starts with 0x to parse from base16, or parse from base10 by default (when a register is generated from a patch).
The second bug is fixed by changing the index used for the list of registers from the addressOffset to the register name.

Note about the memory map comparison:
It looks like a lot of registers with splitted fields now have a proper bitOffset, so it explains why there is tons of modifications here


Extend API to get previous run results

Disclaimer: This was not tested!
For instance:
http://check.ocamllabs.io/api/v1/1670612392-3a8f24e33beda3fd110c63f455e18cb232f84b62/package

I'm not sure to see how this API endpoint alone can be useful. Wouldn't you need a new field in api/<version>/latest/packages that says which logdir name to use? (assuming you want this new endpoint to have a persistent link with a known state)

I guess the question is: what is your use-case?

You're right, it may not be the API I need. This was more of an attempt at grasping the code :-)
The one thing I really need is an extra field in the JSON response with the hash of the opam-repo commit upon which the run took place. In the same vein, support for the HEAD request would do the job. I'd like to avoid polling check.ocamllabs.io too much and only download and process fresh runs.


fix: stop checking for clean working dir on revert

:tada: This PR is included in version 2.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix uninitialized variable crash

@beren12 Hi, I came here from the CurseForge page, where people are talking about this error too. Does this pull request look correct to you?

People at https://www.curseforge.com/wow/addons/enhanced-flight-map are also mentioning another Lua error, by the way:
Message: Interface\AddOns\EnhancedFlightMap\nodeinfo.lua:201: attempt to index local 'myNode' (a nil value)

By the way @beren12 I highly recommend getting these tools:

https://www.curseforge.com/wow/addons/bug-grabber/files (download latest retail, then change its .toc file's Interface number to match the TBC client)
https://www.curseforge.com/wow/addons/bugsack (install the TBC version)

Bug catching becomes extremely easy with these tools.

Thanks I'll check this when the servers come back up. Sorry I've been so busy, I thought I fixed all the bugs when you didn't import/cleared the saved setting.

I had done a similar fix it just punts the error down farther. I'll look into it some more.

That’s the old note from way back in original TBC and wrath. I did all the alliance points for classic and gave them to the Dev and redid every point for alliance and hoard for TBC so it’s accurate

It was many hours of flying

@beren12 WoW, thank you, I can imagine like 7 hours of flying. Amazing job!

@Bananaman the fix is too simple and too straight not to be sure about anything :)

@Dramacydal Yeah but I haven't read through the code literally at all, so I don't know that. 😂
@beren12 I've been playing with the proposed fix since June 3rd and everything in Enhanced Flight Map has been working. I don't notice any issues at all. Any chance of investigating this pull request soon?


Really important detailed install instructions

Just added install instructions and Linux limitations

its a documentation enhacement

A link to simplified install instructions that include all of the requirements has been added under the Documentation heading


feat(linter) eslint plugin unicorn: prefer array flat

Current dependencies on/for this PR:

main

PR #1650   👈

This stack of pull requests is managed by Graphite.


Set up the Community Edition

The Community Edition bundles OSS VAST with proprietary plugins. These are developed in a private repository that needs to be cloned over SSH, meaning only Tenzir staff and CI will be able to build vast-ce and vast-ce-static.
Implementation wise we use builtins.fetchGit which acts during evaluation time so Nix can clone the repo wit the current users credentials.
This also adds a withPlugins function to the vast package definition so that we can build VAST with different plugin lists easily.
Usage example:
vast.withPlugins (ps: [ compaction matcher ])

All plugins that are developed in the open source VAST repo are always enabled, there is no way to opt out yet.
This PR also adds version calculation logic to flake.nix, so that pure builds report their correct version:
$ nix run 'git+https://github.com/tenzir/vast?ref=topic/flake-version' -- -q version
{
  "VAST": "v2.4.1-936-gf5b87c2136",
  "Build Configuration": {
    "Type": "Release",
    "Tree Hash": "9e809707b1744cf198c0c1246fb6b2ac",
    "Assertions": false,
    "Address Sanitizer": false,
    "Undefined Behavior Sanitizer": false
  },
  "CAF": "0.18.6",
  "Apache Arrow": "10.0.0",
  "jemalloc": "5.3.0-0-g54eaed1d8b56b1aa528be3bdd1877e59c56fa90c",
  "plugins": {
    "cef": "v0.1.0",
    "parquet": "v1.0.0",
    "pcap": "v1.1.0",
    "sigma": "v1.1.0",
    "web": "v1.0.0"
  }
}

This comes with the caveat that it only works with git+https://github.com/tenzir/vast, but not when the repo is addressed as github:tenzir/vast.
TODO:

[x] Add the repo that holds the proprietary plugins as a regular sub module
[x] Add a CI Job to build and upload the community edition
[x] Onboard the remaining plugins that should be included in the CE
[x] Avoid the need to repeat the listing VAST dependencies in the plugin package
[x] Make it possible to run nix/update.sh without a local Nix installation

Question: Wouldn't it be better to keep this logic outside of Nix such that it works for the regular build and for our Docker builds as well?

Question: Wouldn't it be better to keep this logic outside of Nix such that it works for the regular build and for our Docker builds as well?

That is what the first TODO is about.

That is what the first TODO is about.

Makes sense then. Thanks! So this implies the dependency inversion between our own plugins and VAST?

So this implies the dependency inversion between our own plugins and VAST?

Yes.

resolved tenzir/issues#102

As discussed in Slack, approving modulo the open remaining tasks so you can merge this yourself after carefully verifying that the artifacts are uploaded correctly in CI.

The PR artifacts work now, but we need to wait for the RC to verify the release artifacts.

Big step forward, nice work! 🚀


fix: increase sematic-release version

:tada: This PR is included in version 1.10.2 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Update LICENSE, fix #267

Description

updating year in LICENSE and expanding target versions for black to include py38 are the only material changes. All the rest is black changes from v20.
 One of the reasons I slightly preferred black to yapf what that it let compact dicts lie. Still, I'll wait to see how a grd1, grd0 looks before placing them out-of-bounds of the formatter.


Add icon for file erb

Hello,
Please add icon rails for file extension is erb by

This svg file : https://upload.wikimedia.org/wikipedia/en/e/e9/Ruby_on_Rails.svg

I added the existing ruby icon for the .erb file extension. I think the icon that you posted above, is not really a material icon and so I didn't add it to the theme.


OU-422: Add proxy endpoint to distributed tracing dynamic plugin

Add Golang backend

/api/v1/list-tempostacks to list all tempostack instances on cluster
/proxy/{namespace}/{name} to query a tempostack using its namespace and name

convert usage of yarn to npm
reorganize files to separate frontend into /web and backend into /cmd and /pkg
create /hack file for deploying a test instance of distributed-tracing-console-plugin, tempostack, and opentelemtry collector

/lgtm

/lgtm
per PeterYurkovich

/lgtm


Expose dockerfile and args arguments under build

Thanks for the pull request!!! Can we just use more precise types? Thanks!

Made the requested changes, @gabrieldemarmiesse can you take another look?

Hey @gabrieldemarmiesse - Following up for your review on this PR

Sure, I'll take a look!

Thank you @gabrieldemarmiesse, can you also cut a new release with this change?

We'll first merge your other PR about pyproject.toml :)


DASH CPIX support for PlayReady

I tried to found if ShakaPlayer support CPIX but I don't find any informacion. Is it supported?
https://dashif.org/docs/CPIX2.1/PDF/Index.pdf

CPIX is not part of DASH clients.  It is for encoders and packagers and DRM license servers to exchange keys at content preparation time.
Section 3 of the CPIX doc states (emphasis added by me):

There is no assumption on the entities exchanging this information but it is not expected that a client device will use this exchange format. The goal is to allow entities involved in the content preparation workflow to get the content protection information so that, for example a DASH MPD can be generated with all content protection information.

Does this answer your question?

Thanks @joeyparrish


which object detector do you use?

which object detector do you use?

We tried pointrcnn and centorpoint.

We tried pointrcnn and centorpoint.

hello，i have trained the modle of centerpoint，then how can i use the modle to this paper，do you have a ros_main.py to run this detector?


fix deepcopy

Coverage decreased (-4.46%) to 95.54% when pulling af0b3c2eed499c5924234780fb7cf57a35ba3103 on zxf:fix-deepcopy into 9bc6ec1e1d2d1d24c08949032749a5019010e786 on mewwts:master.

Great :+1: Can you supply some test cases for this as well?

I have add a test case for deepcopy.

Coverage remained the same at 100.0% when pulling 436a314d1afd6ff9f152b3e041a5a6f2a51e7320 on zxf:fix-deepcopy into 9bc6ec1e1d2d1d24c08949032749a5019010e786 on mewwts:master.

Coverage remained the same at 100.0% when pulling cfb0e941b0b5ccce713e99bc14fc3d05e8a7537f on zxf:fix-deepcopy into 9bc6ec1e1d2d1d24c08949032749a5019010e786 on mewwts:master.

I'll probably merge this in later today, thanks for this. :+1:

Thanks @zxf!


Required packages

Hi @saeed-anwar, can you please provide the list of the packages needed to make the code work? thanks

Hi @saeed-anwar, can you please provide the list of the packages needed to make the code work? thanks

Please tell me If you were able to get it working. Many modules seem to have been removed in updates.

I made it work with these packages and replacing imload and save from scipy.misc to imageio
asn1crypto==0.24.0
cffi==1.14.2
cmake==3.18.0
cryptography==2.1.4
cycler==0.10.0
decorator==4.4.2
future==0.18.2
idna==2.6
imageio==2.9.0
intel-openmp==2020.0.133
keyring==10.6.0
keyrings.alt==3.0
kiwisolver==1.1.0
matplotlib==3.0.3
mkl==2019.0
mkl-include==2019.0
networkx==2.4
numpy==1.18.5
Pillow==7.2.0
pycparser==2.20
pycrypto==2.6.1
pygobject==3.26.1
pyparsing==2.4.7
python-dateutil==2.8.1
PyWavelets==1.1.1
pyxdg==0.25
PyYAML==5.3.1
scikit-image==0.15.0
scipy==1.4.1
SecretStorage==2.3.1
six==1.15.0
torch==1.1.0
torchvision==0.2.1
tqdm==4.19.9
typing==<IP_ADDRESS>


If 567 asset list collapsing resets

Task linked: IF-567 Asset List - subsystems expand automatically (when status reloads) in tree table also after the were collapsed


evi expression returns parsing error

Reminder for myself to check on this
The EVI expression does not work but only if the getInfo() function is used after the EVI is calculated for every image.
EEException: Image.parseExpression: Expression parse error at character 12:
'2.5 * ((nir – red) / (nir + 6 * red – 7.5 * blue + 1))'

This is still not solved. Something to fix in the future.


Cant use other icon images?

Hi,
I tried using
stopIcon = require('../Images/img1.png') startIcon =  require('../Images/img1.png')
but that gives an error. Probably because they are non-SVG images. Any fix for that? To use normal images there?
Also, your component is too exposed. I cant just use the main tags and start using it. I have to import those math functions along. It would be better if you subdivide it in such a way that a user can just use your component tags and give it his wanted values and start using it.
All the angle calculation should be behind the doors as well and the onUpdate function should just be a callback to change something else in the app and not run the basic working of the component itself.
Awesome component btw, really need to use it in my next project. I would be really glad and thankful if you could improve these functionalities.

For stop / start icon you can use any react-native-svg compatible element (https://github.com/bgryszko/react-native-circular-slider/blob/master/src/CircularSlider.js#L227).
onUpdate recieves { startAngle, angleLength } because you can imagine different ways of using those – and Bedtime is just an example (it's easy to convert it to percentage fill for example) – and for that you don't need much of math functions (or I misunderstood what you've meant by "import those math functions along"). In fact, there is only one function exposed – component CircularSlider – nothing else.
This component was built to help you build your custom one on the top of it, and not to cover every case you can imagine.


Allow empty parameters in curried functions

I have a function like so:
var identity = function(val) {
  return val;
}

var doLater = r.curryN(2, identity)

then I want to add the first value to the identity function
onclick(doLater(5))

but when onclick calls doLater it throws an error. This seems like poor behavior. I should be able to decide if I care if something is passed or not, not the curried function.

Perhaps this solution is kind of ugly but when exposing to the real world and need a function that takes no parameters to be execute like so foobar() instead of foobar(undefined), perhaps one could do something like this

R.compose( R.always, R.apply(R.__,[undefined]) )

This would only work if the function can only accept undefined as a parameter.


Also provide build script for linux arm64 like ubuntu.

Also provide build for linux arm64 like ubuntu.
Or
Add docs for building.

Also provide build for linux arm64 like ubuntu. Or Add docs for building.

see https://github.com/nmeum/android-tools


[ BEAM-3788] Updates kafka.py pydocs

Also removes it from in-progress IO connectors list now that it's supported by portable runners as well as Dataflow.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @robertwb
CC: @mxm

Tests pass now. PTAL.

Thanks.


obsolete "in other organism" terms

catabolism by organism of carbohydrate in other organism involved in symbiotic interaction
catabolism by organism of cell wall cellulose in other organism involved in symbiotic interaction
catabolism by organism of cell wall chitin in other organism involved in symbiotic interaction
catabolism by organism of cell wall pectin in other organism involved in symbiotic interaction
catabolism by organism of glucan in other organism involved in symbiotic interaction
catabolism by organism of macromolecule in other organism involved in symbiotic interaction
catabolism by organism of protein in other organism involved in symbiotic interaction
catabolism by organism of xylan in other organism involved in symbiotic interaction
catabolism of substance in other organism involved in symbiotic interaction
disruption by organism of cell wall of other organism involved in symbiotic interaction
disruption by organism of cellular component in other organism involved in symbiotic interaction
Even if these processes exist for the symbiont and the host, does this grouping actually make sense?
The processes aren't  really related  (other than x compound metabolism, which is fine)

modulation of mast cell degranulation in other organism
modulation of molecular function in other organism
modulation of molecular function in other organism involved in symbiotic interaction
modulation of nutrient release from other organism involved in symbiotic interaction
modulation of peptidase activity in other organism involved in symbiotic interaction
modulation of phagocytosis in other organism involved in symbiotic interaction
modulation of platelet aggregation in other organism
modulation of potassium channel activity in other organism
modulation of process of other organism
modulation of process of other organism involved in symbiotic interaction
modulation of programmed cell death in other organism
modulation of programmed cell death in other organism involved in symbiotic interaction
modulation of receptor activity in other organism
modulation of relaxation of muscle in other organism
modulation of relaxation of smooth muscle in other organism
modulation of relaxation of uterine smooth muscle in other organism
modulation of RNA levels in other organism involved in symbiotic interaction
modulation of sensory perception of pain in other organism

I think it's pretty clear whether these are symbiont or host...?????

modulation of transmission of nerve impulse in other organism
modulation of vasoactive intestinal polypeptide receptor activity in other organism
modulation of voltage-gated potassium channel activity in other organism
modulation of voltage-gated sodium channel activity in other organism
movement in environment of other organism involved in symbiotic interaction


FF4207E6 - Title

Validation

[X] I've read the FAQ.
[X] The Xenia build used is from the master branch. (not MLBS/AlexVS/Canary/pull requests, etc)
[X] This issue isn't for help (tech support) with Xenia, or a game.
[X] I've checked if this game doesn't already have a compatibility report.

Xbox 360 Marketplace link
https://marketplace.xbox.com/en-us/Product/Halo-3/66acd000-77fe-1000-9115-d8024d5307e6
Xenia version
https://github.com/xenia-project/xenia/commit/509a1fa3866be4fb3ab9d413f600012688c89a7c/
Issues
.
Log
.
Screenshots
.
Labels
state-intro, apu-silent, gpu-drawing-corrupt, marketplace-delisted

issue-invalid

issue-invalid

issue-invalid

issue-invalid

issue-invalid

marketplace-unlisted

issue-invalid

marketplace-unlisted


[PAXURL-351] Allow to use MAVEN_MIRROR_URL env or mavenMirrorUrl sys …

…property to set global maven mirror

@jbonofre @ffang may I ask for quick review? The goal is to alter pax-url-aether, so it uses Maven mirror URL passed from env/sys variable instead of any mirror available in explicitly (org.ops4j.pax.url.mvn.settings PID property) or implicitly (~/.m2/settings.xml) used settings.


Seeding the database

Need to be able to create seeds for the database so that we can have easily accessible data to test all of our routes and all of the data to ensure all data displays properly and that moving forward we can have full visibility of all issues moving forward

Users: Stores information about users of the marketplace.
user_id (Primary Key)
username
email
password_hash
registration_date
Farmers: Stores information about farmers who sell products on the marketplace.
farmer_id (Primary Key)
user_id (Foreign Key referencing Users table)
farm_name
description
location
Products: Stores information about products available on the marketplace.
product_id (Primary Key)
farmer_id (Foreign Key referencing Farmers table)
product_name
description
price
availability
image_url
Orders: Stores information about orders made by users.
order_id (Primary Key)
user_id (Foreign Key referencing Users table)
order_date
total_price
status (e.g., pending, processing, shipped)
Order Details: Stores information about the products included in each order.
order_detail_id (Primary Key)
order_id (Foreign Key referencing Orders table)
product_id (Foreign Key referencing Products table)
quantity
unit_price


phrase search highlighting does not work for basic pricing tier

Hi, Can we add a note that says phrase search highlighting does not work in basic pricing tier in the "Requirements for hit highlighting" session? We had a customer report having an issue using Phrase search highlighting in basic tier.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e12a8976-0a1d-c1ea-994d-ddc57d6e468b
Version Independent ID: cb0473e9-8f4f-3eab-13fc-dd36f7bf2e20
Content: How to work with search results - Azure Cognitive Search
Content Source: articles/search/search-pagination-page-layout.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

Hello, I am the person that found the mistake. To correct you, it works on the basic tier, the problem is it does not work on the free tier (which is nowhere mentioned). I think a lot of problem will experience this problem as they use the free tier for development.

Hi @WailAbou, I can't repro this behavior.  I see highlight tagging on my phrase search on a free tier service.  I'm using the built-in hotel sample dataset and index, and this query string in Search Explorer:
search="Gacc Capital"&highlight=HotelName&highlightPreTag=&highlightPostTag=$select=HotelName,Description
I get back these results:
{
<EMAIL_ADDRESS>"https://demo-search-svc.search.windows.net/indexes('hotels-sample-index')/$metadata#docs(*)",
"value": [
{
<EMAIL_ADDRESS>7.5213933,
<EMAIL_ADDRESS>{
"HotelName": [
"Gacc Capital"
]
},
"HotelName": "Gacc Capital",
"Description": "Chic hotel near the city.  High-rise hotel in downtown, walking distance to theaters, restaurants and shops, complete with wellness programs."
}
]
}

Hello, could you try highlighting something in the description, because you have chosen the HotelName which only has 1 occurrence of the keywords it is not clear whether it works or not. If you can, try the search term "hotel in" and if it only hits the hotel in part of the phrase it works, but instead it will highlight also the hotel word in the beginning.

@WailAbou, I'm not seeing a difference in tier, but I confirm that hit highlighting appears to exclude stop words (https://github.com/Azure-Samples/azure-search-sample-data/blob/master/STOPWORDS.md).  If my phrase is "in the area", only "area" is highlighted.  If my phrase is "hotel in", only "hotel" is highlighted.
I think you found either a bug or a content gap, but I'm not getting different behaviors per tier.  I see the stopword omission in both my services (free and standard).

Could you send a screenshot of both inputs and outputs, because I tested it and the Microsoft employee also tested it and we got the same unintended behaviour. I think maybe we are understanding each other wrong, per reference here is the intended behaviour at the bottom of the page under the section "Phrase search highlighting": https://docs.microsoft.com/en-us/azure/search/search-pagination-page-layout

Attaching a screenshot, free tier and standard tier side by side.  Same query pasted into the Search Explorer of both services, each having the sample hotels index.  I don't currently have a Basic tier, but in a previous post, you said that it was just Free tier that wasn't working.

.

Free pricing tier

Standard pricing tier (basic pricing tier should also be the same)

As you can see in the screenshots with the exact same dataset and search query the first one highlights the description as if it is searching trough it as keywords while the standard pricing tier has the intended behaviour. Could you also try this search term to double check on your side?

I see your results, but I still can't repro on my side using your query. My results don't change if I switch to the default formatting  either.   I'll delete and recreate my free service to see if I get your results.  If I can't repro after that, all I can do is ping the engine team to ask them how to debug this.

.

My new free tier service is in West US2.  I couldn't repro on that service either.  I'll try Europe.  Thanks for your efforts!!  I really appreciate it.

I just tried in "North Europe" and I still get the expected behavior, where just the phrase is highlighted.
We have a note in the docs about older services producing the experience you're describing:  https://docs.microsoft.com/en-us/azure/search/search-pagination-page-layout#phrase-highlighting-on-older-services
But we both just tried the same queries on new free services created in North Europe, and you get the old behavior, while I get the newer/expected behavior.
I'll ask engineering about this.
@Grace-MacJones-MSFT  Can you remove the "doc enhancement" and relabel this as a product issue??

Hello again, no problem at all for helping. I did some further testing with a new account and created a new Cognitive Search Service on North Europe and I stil get the weird behaviour, here is a screenshot of it.

Also here is the template, I would be willing to further help and solve/identitfy the problem by sharing my resource id's so you or the engineering team can take a look at it.
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "searchServices_testsearchjumsum_name": {
            "defaultValue": "testsearchjumsum",
            "type": "String"
        }
    },
    "variables": {},
    "resources": [
        {
            "type": "Microsoft.Search/searchServices",
            "apiVersion": "2021-04-01-preview",
            "name": "[parameters('searchServices_testsearchjumsum_name')]",
            "location": "North Europe",
            "sku": {
                "name": "free"
            },
            "properties": {
                "replicaCount": 1,
                "partitionCount": 1,
                "hostingMode": "Default",
                "publicNetworkAccess": "Enabled",
                "networkRuleSet": {
                    "ipRules": [],
                    "bypass": "None"
                },
                "encryptionWithCmk": {
                    "enforcement": "Unspecified"
                },
                "disableLocalAuth": false,
                "authOptions": {
                    "apiKeyOnly": {}
                },
                "disabledDataExfiltrationOptions": [],
                "semanticSearch": "disabled"
            }
        }
    ]
}

@WailAbou I haven't gotten a response yet.  Could you file a support ticket for this?  https://azure.microsoft.com/en-us/support/create-ticket/
Or post to Stack Overflow:  https://stackoverflow.com/questions/tagged/azure-cognitive-search?tab=Newest
An engineer will see your issue if you follow either of the above approaches.  Sorry I couldn't help!

No problem, it is not a problem for me anymore as I am using the standard tier anyway, but I imagined it to be nice to have a disclaimer to avoid future confusion for other people. Anyway, thank you and have a nice day :)

#please-close

Thanks for all your help @HeidiSteen !
As Heidi stated we would need to connect you with a support engineer to investigate further. @WailAbou, if you would like for me to open a  free one time support ticket I can help you with that. please send me an email at<EMAIL_ADDRESS>with the subject ATTN: Grace and in the body of the email include your Azure Subscription ID.


Actualize chain_get_block RPC rc2

Resolves: #

Summary
TODO

[ ] ...

Checklist

[ ] Code is properly formatted
[ ] All commits are signed
[ ] Tests included/updated or not needed
[ ] Documentation (manuals or wiki) has been updated or is not required

Is it necessary to have BlocV2 when Hash, Header and Body are also there?


Vanity insanity x3

About the PR

Why / Balance

Technical details

Media

Requirements

[ ] I have read and am following the Pull Request and Changelog Guidelines.
[ ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

may need to revert this later, note to self.

please dont look horrid


Disable Add button after clicking submitting changes [ENG-130]

Purpose
We've been seeing DB deadlocks happening due to the v1 contributor endpoint.  While we're not sure of the root of issue, we can probably prevent mitigate it by preventing clickmashing on the add contrib modal.
Changes

Disable the "Add" button after adding contributors in the add contributor modal
Use the canSubmit KO variable

QA Notes

Start on the "Contributors" page of a node
Click the "+Add" button
Search for some new contributors
Choose a contributor to add
Click the "Add" button

Documentation
Frontend, none needed

Side Effects
None anticipated

Ticket
https://openscience.atlassian.net/browse/ENG-130

this was all Erin, just making a small addition. LGTM.


equation used in LevelMapper

❓ Questions and Help
The code leverages the equation: target_lvls = torch.floor(self.lvl0 + torch.log2(s / self.s0 + self.eps)) to find the corresponding feature level for each RoI. Here self.s0 by default is 224. While in original RPN paper, the authors said that they use 224 because 224 is the canonical ImageNet pre-training size. However, if we use a different dataset with different image size (e.g., COCO), then do I need to change 224 to another value?
Thanks!

hi,@shanggdlk  ,have you found the answer?


Quick Create doesn't use Hyper-V Settings for VM Image Download.

** Description**
Trying to use Hyper-V quick create to install a Windows 11 VM on my workstation. My OS is installed to C: on a 256GB SSD, and there is 4 * 1TB SSD installed as well.
The default location in Hyper-V for new VM and disks is D: (1TB drive, mostly empty).
However, Quick Create doesn't use those settings for downloading the image and fails to setup the VM due to the C: not having enough free space (there is about 15GB free and the image needs 19GB for downloading).
Quick Create should either use the default Hyper-V location settings, or allow the user to select where the VM is created.
Link to doc:

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


remove schema1 manifest

follow upstream
https://github.com/distribution/distribution/pull/4023
https://github.com/distribution/distribution/pull/4007
https://github.com/distribution/distribution/pull/4002
https://github.com/distribution/distribution/pull/3987

close via https://github.com/ttys3/reg/commit/28160133989584ef12a07e7af0bb6d2207399143


Amazon role creation wizard change

File: docker-cloud/cloud-swarm/link-aws-swarm.md, CC @londoncalling
According to docs the first step when creating a role is to give it a name. However that has been moved to the last (4th) step in the wizard which would correspond to around bullet point 8 in your documentation.

@dbackeus thanks! please review the PR linked above ^^


Handle internet connection loss

Current behavior when internet is turned off during an upload : app crashes.

It now just displays a "Failed upload" error message.


Admin requests page crashes due to an uncaught exp

When loading the requests page on my server, It crashes to a blank page everytime. Debugging shows an error
TypeError: Cannot read property 'seasonNumber' of undefined
Not sure how to replicate. The show it is crashing on seems on par with the TVdb standard
Screenshot

Please follow the bug report format when reporting issues. I will close this, and let you re-make it with the correct format.


Updated Dockerfile for devcontainer usage (IDFGH-12213)

Info
Added pip and get_idf alias to .bashrc to use the docker container in a development container (e.g. github devcontainer).
Without pip, the docker image is not easily usable inside a vscode environment.
Without the alias, it is harder to get idf.py working (e.g. manually call export.sh) if not using entrypoint.sh or spawning additional terminals in a dev container environment.

Dockerfile modification LGTM, good idea!
@timoxd7 Can you please update your commit messages as suggested by Danger bot?

sha=626aa9f4ca752e68932187c6ef7cac49fcc37ce7

Thank you for the contribution. This will probably merged soon.


Move default value for tag outside of SidecarProperties.java

Would be better for release process not to have to change Java code

good point - we can add application.properties and make it a required property from the code perspective


String Rotation

I would like to write a code for testing whether a string is rotation of another or not.

/assign

Always mention the language @divyaachoudharyy .

Yes, I'll keep in mind next time.


Implementing RFC 31/WAKU2-ENR (Part 1)

Problem
As per the spec, adding the capabilities (E.g. relay, store, filter, lightpush) of a node to the DNS discovery mechanism.
Solution
Added the ability to get/set waku2 field on the enr.
The waku2 field needs to be encoded/decoded (Waku2 object <-> Uint8Array) so added a codec to do so.
Notes

Related to #405

@jemboh happy to merge? if so write bors merge :)

bors merge

This PR was included in a batch that successfully built, but then failed to merge into master. It will not be retried.
Additional information:
{"message":"Commits must have valid signatures.","documentation_url":"https://docs.github.com/articles/about-protected-branches"}

@jemboh https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits


Add [Mobile] 💠 [Popover]

Duplicates ❌

[X] I have searched the existing issues

Libraries

[ ] Vitamin Web UI Kit
[X] Vitamin Android UI Kit
[X] Vitamin iOS UI Kit
[ ] Vitamin Icons
[ ] Vitamin Assets
[ ] Vitamin Core theme (Light mode)
[ ] Vitamin Core theme (Dark mode)
[ ] Vitamin Base colors palette
[ ] Vitamin Extras kit
[ ] Not related to one Figma library

Summary 💡
Add popover to overlay component for iOS and android.
Dependencies 📦
CSP and Sport experience team (Domyos E Connected, Decathlon Connect, etc.) needs it to implement a tutorial flow that features popovers -> https://decathlon.atlassian.net/browse/CNT-1015
Examples 🌈
Tutorial flow that use popover

Motivation 🔦
We needed a component to help the user focus on a specific element on the screen that could send him to a full page experience : a tutorial, dedicated to this specific element.
New 🆕

[x] Validated in a grooming session

Backlog 📋

[x] Has a good summary
[x] Has example link(s)

Design in progress 🏗

[x] Analysis (functional & technical)
[ ] Design refinement
[ ] Dev qualification (properties, breakpoints, tokens etc.)
[ ] Documentation refinement (inside the Figma branch)

Design review 👀

[ ] Figma branch merged
[ ] Documentation review _(and move the documentation into Vitamin Documentation Figma file)

Discuss With Core team, Won't be done


compiled standalone with importlib and psutil

>python --version
Python 3.10.9

>python -m nuitka --version
1.4.8
Commercial: None
Python: 3.10.9 (tags/v3.10.9:1dd9be6, Dec  6 2022, 20:01:21) [MSC v.1934 64 bit (AMD64)]
Flavor: CPython Official
Executable: d:\MyProjects\py\test\.venv\Scripts\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: C:\devFiles\vsBuildTools\VC\Tools\MSVC\14.34.31933\bin\HostX64\x64\cl.exe (cl 14.3).

>python -m pip freeze
black==23.1.0
click==8.1.3
colorama==0.4.6
mypy-extensions==1.0.0
Nuitka==1.4.8
ordered-set==4.1.0
packaging==23.0
pathspec==0.11.0
platformdirs==3.0.0
psutil==5.9.4
tomli==2.0.1
zstandard==0.20.0

I found a weird thing about compiled standalone exe with importlib and psutil.
I wrote a short code to reproduce the thing.
from importlib import resources
import psutil

def getPath(filename: str, package: str = "resource") -> str:
    source = resources.files(package).joinpath(filename)
    print("source = ", source)
    print("type(source) = ", type(source))
    with resources.as_file(source) as f:
        return str(f.absolute())

if __name__ == "__main__":
    print(getPath("test.txt"))

and the follwing is the file tree:
workspaceFolder
├ resource
│  └ test.txt
└ test.py

and below is options to compile script:
nuitka ^
--standalone ^
--include-data-dir=resource=resource ^
--remove-output ^
--output-dir=bin ^
--output-filename=test.exe ^
test.py

and then the output is
>bin\test.dist\test.exe
source =  D:\MyProjects\py\test\bin\test.dist\resource\\test.txt
type(source) =  <class 'nuitka_resource_reader_files'>
Traceback (most recent call last):
  File "D:\MyProjects\py\test\bin\test.dist\test.py", line 14, in <module>
  File "D:\MyProjects\py\test\bin\test.dist\test.py", line 10, in getPath
AttributeError: 'nuitka_resource_reader_files' object has no attribute 'absolute'

psutil did nothing just imported but it cause an error.
If I remove import psutil, then it works well without any problem.
I don't know why it happens..
Output without import psutil is
>bin\test.dist\test.exe
source =  D:\MyProjects\py\test\bin\test.dist\resource\test.txt
type(source) =  <class 'pathlib.WindowsPath'>
D:\MyProjects\py\test\bin\test.dist\resource\test.txt

And, in the first output, print("source = ", source) said source =  D:\MyProjects\py\test\bin\test.dist\resource\\test.txt.
It has a wrong path with doubled back-slash in the last joinning.
How can I use importlib and psutil properly..?

It seems we need to provide a method absolute for the our loader files object too. Should be easy enough. Should be in a hotfix 1.4.9 if there is any, but I have to yet create the code of course.

The object returned by resources.files has a path with trailling backslash when compiled. Is it intended..? It causes double backslash when using joinpath.

@DukSon1224 of course it is, and always make sure to mention me, no matter how urgent or important things are, that way I have less time to waste on other issues

I was just checking, and absolute is really just one of way too many functions that the Python object exposes. There is basically the whole os module exposed in another form.  Thanks for the reproducer, that is nice to work with.

@kayhayen I'm sorry for taking your time. I was hasty in creating the issue without looking into it in detail. As you said, the implementation of absolute was not required. Thank you for your kind comment.
I fixed my code just using os.path.abspath(str(f)).

I never criticized your report, I think I said "hanks for the reproducer, that is nice to work with", However, it seems you missed my point, which is no surprise when using irony, successful communication becomes unlikely, but sometimes I cannot help myself.

This is part of the 1.5 release, there was no hotfix to put it into anymore.


Techmap pass makes attributes disappear

Steps to reproduce the issue
techmap_attr_failure.tar.gz

Build and install yosys (https://github.com/YosysHQ/yosys/commit/4160acc0b19532d50435abbc66bd2f6aab9473b6)
Unpack the tar
cd techmap_attr && make

Expected behavior
The Makefile checks whether the LOC constraint on the primitive cell is present both in the pre- and post- mapped verilog output. The LOC attribute (or any custom attribute) should not get lost.
Actual behavior
The LOC attribute gets lost in the post-mapped output.
Insights
I have run a git bisect and found the first "bad commit" being 76c4ee4e

Reproduced and confirmed the causative commit. Thanks for the great test case and bisection!

Fixed in https://github.com/YosysHQ/yosys/pull/2205

@whitequark Great, thanks for the quick fix! I can confirm it works fine with my original test from which I extracted the reduced one.


[question] How do I support import and export with buble

Only use buble to transform import and export into require and exports.
var buble = require( 'buble' );
var result = buble.transform( source ); // { code: ..., map: ... }

hi @fengxinming , import/export and require/exports are module standards which first one belongs to ESModule and second one belongs to CommonJS. I believe buble is more focusing on compiling syntaxes to ES2015, without transpiling your ESModule to CJS modules. check https://buble.surge.sh/guide/#using-es-modules
similar issue here https://github.com/bublejs/buble/issues/143

Yep, out of scope for this library.


ESP32 support

Hello
I am curious will it work on ESP32 - C3?
Thank you

It works, but you need to upgrade to embedded_hal 1.0, and I've already read it on my esp32c3.


coverZeros Bug

CoverZero() in correctly covers zeros in the following 6x6 example
Task 1,Task 2,Task 3,Task 4,Task 5,Task 6,Task 7
Person 1,6,2,1,3,1,2,2
Person 2,1,4,5,5,3,6,6
Person 3,7,6,3,2,4,7,7
Person 4,2,3,4,6,5,5,5
Person 5,3,7,7,4,6,4,4
Person 6,5,5,2,1,2,1,1
Person 7,4,1,6,7,7,3,3
I've not been able to find a way to modify it yet so that this is no longer an issue.
The first pass Covers correctly
5,1,0,2,0,1,1           x
0,3,4,4,2,5,5           0
5,4,1,0,2,5,5           0
0,1,2,4,3,3,3           0
0,4,4,1,3,1,1           0
4,4,1,0,1,0,0           x
3,0,5,6,6,2,2           x
x,0,0,x,0,0,0
But the second pass covers incorrectly
6,1,0,3,0,1,1           x
0,2,3,4,1,4,4           0
5,3,0,0,1,4,4           x
0,0,1,4,2,2,2           x   <
0,3,3,1,2,0,0           x
5,4,1,1,1,0,0           x
4,0,5,7,6,2,2           0
x,x,0,0,0,0,0
^
The indicated Row/Column is covered on the Row instead of column which would cover all zeros with a lower cost

@hamilTek were you able to fix this issue?


Alternative solution for issue #4787

@phihag
Currently the *#*1 https://github.com/rg3/youtube-dl/commit/3a0d2f520a0f95c2f87b1c95049135b10206f97f solution provided by @dstftw is not the best in my opinion because it does not completly solves the encoding problems.
On Windows you lose part of the filename which for most users is not ok.
See issues (#5045, #5182, etc..) and on my repository https://github.com/MrS0m30n3/youtube-dl-gui/issues/39
Can we implement the *#*2 solution suggested from @dstftw? (#4787)

Perform all the subprocessing on temporary file with plain ASCII name and rollback to original name right after it's done.

This way we can completly fix those issues.
You can assign me in this issue if you want.

I have been using [this python script] (https://github.com/maleficarium/Scripts/blob/master/merge.py) that I wrote for merging the files that youtube-dl failed to.
I'm currently looking into how to integrate it into youtube-dl but since I am not familiar with the code yet it will probably take me some time.

Hi, @maleficarium
You also need to consider the path name.
For example if the filename that we pass to the post_process method (https://github.com/rg3/youtube-dl/blob/master/youtube_dl/YoutubeDL.py#L1502) is something like
/home/user/Downloads/λήψεις/test.flv then you can't pass that to the subprocess Popen.
Thus i'm starting to see some drawbacks in the *#*2 solution. You can't really rename the whole path,
maybe you could change the working directory and then pass only the filenames into the subprocess Popen or maybe we could use a temporary path to download the files when we detect that the post_process won't be able to handle those.
Also, we should consider the *#*3 solution suggested by dstftw.
It's not so hard to write a minimalistic wrapper in C for the CreateProcess to support unicode.

@MrS0m30n3 I finaly got some time to work on this.
Thankfully youtube-dl is passing only the filenames to ffmpeg so encoding the path is not necessary. Simply hashing the filenames before processing them and renaming them back afterwards solves the problem.
Even using youtube-dl.exe -f bestvideo+bestaudio https://www.youtube.com/watch?v=XXXXX -o "C:/週連続/%(title)s-%(id)s.%(ext)s" will process the files properly and store them in whatever your system interprets C:/週連続/ as (C:/###/ in my case). This is an issue with unicode support in command line and it's the users responsibility to use valid paths when using -o.
I will send a pull request for this fix: https://github.com/maleficarium/youtube-dl/commit/21aa58a2f178a80c751f487c1906e874ef69e90c.

I will be happy if someone could fix this.

Why just not to add transliteration option? For instance, Чернобыль will be transliterated to Chernobyl, and almost everyone understands transliteration.

@tushevorg how will you transliterate non-cyrillic Unicode?

@dstftw Japanese language has romaji, Greek language also has its own system.
I'm not a python developer, but afaik there are modules for this already, like https://pypi.python.org/pypi/transliterate/1.7.6
All we need is to use them (or extend 'em with romaji/chinese/etc )

Unicode is not bound to languages' characters only.

@dstftw then nothing changes - it simply drops the characters as it does now


feat: add github actions support

Should make it easy to enforce dart style and analysis cleanliness, etc.

New version (3.0.1) should be pana-approved btw


All the things, modern workflow necessities

Sam and I were chatting and I made him a list of all the modern workflow necessities I would look for when choosing roots.  He mentioned roots already does many of these items but may be lacking others.
We're a SPA shop, and we have a massive amount of JS files and assets.  We're looking to dump Rails in favor of a purely front-end focused workflow that can be deployed statically.  But I don't want to give up any of the control I have with Rails.
Here's what I'm looking for:
CSS/Images

[ ] Sprite Generation (Compass)
[x] SASS Support
[x] Fine grained control over CSS manifests (being able to concat multiple .scss files into separate manifests at will)
[x] Control different manifest strategies for development vs production
[x] CSS Source Maps
[x] Minification

Templating

[x] Automatically precompile ECO templates when they change
[x] Set their file path as the key in the global JST object for easy reference

Environments

[x] Automatically control and set environment variables for test / development / production

Javascript

[x] Fine grain control over manifests (ability to control which js files get concat'd into which manifests)
[x] Control different manifest strategies for development vs production
[x] Source Maps
[x] Minification
[x] Obfuscation

Development

[x] Watches for any file changes and regenerates the changed assets
[x] LiveReload

Testing

[ ] Easy way to serve and organize static json fixture files used in tests without having to use sinon.js to manually fake XHR requests
[ ] Testem / Karma Runner support
[ ] Mocha / Chai

Deployment

[x] Hash Digest all assets (JS manifests, CSS manifests, images)
[ ] Generate a manifest.json file which is a key/val of all assets and their digest version
[x] Deployment to s3
[ ] GZipping all assets
[ ] Zipping up the deployed /dist source, timestamping it (allowing for easy rollbacks)
[ ] Execute arbitrary commands after deployment (like sending Pusher a message)

Ok so sourcemaps have been implemented and are fully functional in the sourcemaps branch. This will be merged and shipped out tomorrow. Pending a bit more polish on the docs site and some small adjustments to the deployment script, we will be merging v3 and going public with it. All that being said, I'm going to close out this issue for now. Thanks so much for bringing all these things up!
If there are any other issues here you'd like to see that I missed for some reason, we should open a single issue to discuss that feature :grinning:


Fetch mission data

Changes made On this pull request are as follows
This PL covers ticket No.: #24 #20 #18 #15 #10 #9 #6 #2
Kanban Board: https://github.com/users/RiyaBulia12/projects/6

Created a route and a view component. Using

Created a route and a mission view component.

Create a directory for all Redux state slice files.

Fetch data from the Missions endpoint (https://api.spacexdata.com/v3/missions) when a user navigates to the Missions section.

Once the data are fetched, dispatch an action to store the selected data in the Redux store:
- mission_id
- mission_name
- description

Used useSelector(), Redux Hook, to select the state slices and render lists of missions in corresponding routes

When a user clicks the "Join Mission" button, an action will dispatch to update the store. it will get the ID of the selected mission and update the state.

Follow the same logic as with the "Join mission" - but you need to set the reserved key to false.

Dispatched these actions upon clicking on the corresponding buttons.

Missions that the user has joined already will show a badge "Active Member" instead of the default "NOT A MEMBER" and a button "Leave Mission" instead of the "Join Mission" button (as per design).

Rendered a list of all joined missions (using filter()) on the "My profile" page.

Features remaining -
for the page navigation links and style active class to indicate which section/page the user is currently on (underline active navigation link).

Hay @RiyaBulia12 thank you for the feedback, I have fixed the issues you commented on and pushed the branch.
Please have a look at it and let me know.
Regards

Hay @RiyaBulia12 thank you for the feedback, I have fixed the issues you commented on and pushed the branch. Please have a look at it and let me know. Regards

Everything looks good. You can merge your PR


Error with the reactHotLoader flag on

It seems the project is setup for HMR regardless of this flag in package.json.
It is always on, which is great.
But when I switch it true and start the dev mode yarn watch, it errors out.
I check with the latest code.
Also I found that HMR 3 stable is released at v1.3.1 (finally!!)
https://github.com/gaearon/react-hot-loader/releases
I can dig into this problem and make a PR but it will take some time.
Thank you guys for the awesome project!
ReferenceError: vendor is not defined
    at Object.49 (/Projects/boilerplates/apollo-fullstack-starter-kit/external "vendor":45:1)
    at __webpack_require__ (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:620:1)
    at fn (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:44:1)
    at Object.0 (/Projects/boilerplates/apollo-fullstack-starter-kit/delegated ./node_modules/webpack/buildin/harmony-module.js from dll-reference vendor:1:1)
    at __webpack_require__ (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:620:1)
    at fn (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:44:1)
    at Object../src/server/index.js (/Projects/boilerplates/apollo-fullstack-starter-kit/build/server/index.js:2172:56)
    at __webpack_require__ (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:620:1)
    at fn (/Projects/boilerplates/apollo-fullstack-starter-kit/webpack/bootstrap 20283c0549df03e733c5:44:1)
    at Object.50 (/Projects/boilerplates/apollo-fullstack-starter-kit/build/server/index.js:5267:1)

@zirho Thanks for this report, I have reproduced this problem and looking at it now.

That's awesome!
Thank you!


Bug: Use existing S3 bucket if stack is redeployed in prod

Geostore provisions several s3 buckets (prod and staging) within its stack. These buckets are town down in non-prod environments when the cdk destroy command is issued, but they are set to retain in production. This is by design and also an expected behaviour.
However, should we need to destroy the stack in production one day and redeploy it, we would run into problems during (re)deployment since the bucket is already in place. We would likely get a resource already exist error when we try to (re)deploy the stack in production.
We should investigate the canonical way of dealing with such situation and update our stack accordingly.
Things to keep in mind:

Bucket permission (i.e. how it could be made available to the new stack)

Would this also apply to resources like dynamodb?

Is it a good idea to do a periodic tear down the production stack and redeploy (perhaps in a different aws account) to capture potential problems?


Improve Spanner test coverage

This issue is from @danoscarmike:

There are a number of files in google-cloud-ruby/google-cloud-spanner that have <=90% code coverage.

As we push past Alpha and head for Beta can you please review and get relevant files up to an 80% minimum threshold?

Screenshot of coveralls report attached.  Search term: "spanner"

The lowest 3 in this report are GAPIC code that should be excluded:
/lib/google/cloud/spanner/v1/spanner_client.rb
...spanner/admin/database/v1/database_admin_client.rb
...spanner/admin/instance/v1/instance_admin_client.rb

@lukesneeringer @geigerj can you look at updating CI to exclude the 3 x GAPIC files identified by @quartzmo above?  TY!


    @ Base ./operators.jl:544
 [26] _foldl_impl
    @ ./reduce.jl:68 [inlined]
 [27] foldl_impl
    @ ./reduce.jl:48 [inlined]
 [28] mapfoldl_impl
    @ ./reduce.jl:44 [inlined]
 [29] mapfoldl
    @ ./reduce.jl:175 [inlined]
 [30] mapreduce
    @ ./reduce.jl:307 [inlined]
 [31] render!
    @ ~/.julia/dev/Krang/src/schemes/schemes.jl:22 [inlined]
 [32] gputest1()
    @ Main ~/src/krang1.jl:50

Frustratingly, I cannot find the dynamic invocation as when I try to inspect it with @code_warntype and Cthulhu.jl it looks like it should all be static (this is not the first time I've had this problem with GPU).
I have a couple of small fixes incoming that are needed for the camera.
Are you able to test this with CUDA on your end?

I don't have CUDA. I've been running everything with Metal. I haven't seen this error with Metal before.

Maybe your issue is that you mixed types? Should this be
CUDA.fill(0f0, sze, sze)

I don't think so, CUDA mostly automatically converts to Float32.  I will look into this more thoroughly later and see if I can fix it.

Ok, I've figured out what's going on here.  The issue is that I was mistaken by the various different return types of the materials.
First, let me show a working example which is close to my example above but makes no physical sense:
function makescene(θs, rmin, rmax)
    #mat = Krang.ElectronSynchrotronPowerLawIntensity()
    mat = Krang.CoordinateRadius()
    meshes = ntuple(Val(3)) do j
        Krang.Mesh(Krang.ConeGeometry(θs, (j, rmin, rmax)), mat)
    end
    Krang.Scene(meshes)
end

function gputest1()
    metric = Krang.Kerr(0.99f0)
    θo = Float32(45 * π / 180)
    sze = 400
    rmin = Krang.horizon(metric)
    rmax = 10.0f0
    ρmax = 15.0f0

    θs = Float32(π/12)
    scene = makescene(θs, rmin, rmax)

    camera = Krang.SlowLightIntensityCamera(metric, θo, -ρmax, ρmax, -ρmax, ρmax, sze, A=CuMatrix)

    store = CUDA.fill(0.0f0, sze, sze)

    # this *would* work for other material types but only 4 one mesh because of `+`
    #store = cu(Matrix{NTuple{4,Float32}}(undef, sze, sze))

    Krang.render!(store, camera, scene)
end

The first problem was that I was confused by the various return types of the materials.  The second problem is that the existing method for render! only really works for materials which return scalars, or more specifically for objects which have + methods.
The good news is that indeed it's looking like the real math in this package runs just fine on GPU (I haven't done any thorough benchmarking but it also seems like an appreciable speedup over CPU threads).  The bad news is that some of the interface here has issues.
I'm not sure what would be the best way to resolve this.  One possibility is to dispatch the kernels on the material type, so that e.g. for Intensity it knows to take the intensity component.  Another, more complicated solution would be to write methods that prepare the output container for render! and pick appropriate methods for combining the results.  It seems to me that this package doesn't really handle any kind of occlusion, so maybe another option would be to restrict to one mesh?
Anyway, would be glad to help test whatever you decide on CUDA.

Wow great! Glad it works. Yeah, the package doesn't currently handle occlusion. I wasn't going to put it in until I needed to/someone asked me to (I likely will need to in a few weeks/months for my own projects).
I intended for the default behavior of multiple mesh contributions to be additive blending, but it doesn't make sense to do additive blending on the weird CoordinatePoint and CoordinateRadius materials I defined. I think I might scrap those coordinate related materials and put their functionality somewhere else since it seems to clash strongly with the philosophy of the code.

I've removed the CoordinatePoint and CoordinateRadius materials, and updated the docs accordingly in 1cc23dc9e8824a9100448761df23bd6b7cf6009b


Set Host header for readiness probe

If Allowed Hosts is not '*', then the readiness probe always fails with 500 Server Error (and emails the admin). This can be avoided by setting the Host header on the readiness probes, such as:
     readinessProbe:
            httpGet:
              path: /
              port: http
              httpHeaders:
                - name: Host
                  value: hostname.example.com

The value for the header could be the first value in the allowedHosts list.

This is fixed in chart version 2.1.0, which includes NetBox 2.8.1 (but can support NetBox 2.7.11 or above).

This needs to be applied to the nginx container readinessProbe as well, as the request is forwarded onto the app.


Don't double-spend unspendable inputs

When filtering inputs that can be used for swaps, we filter out all the inputs that are already used in pending funding transactions. However, we ignored previously confirmed funding transactions, assuming that our electrum server would have already filtered them from our wallet state.
Unfortunately, we cannot assume that our electrum server is up-to-date (especially if we connect to a different one after a restart) and behaving correctly, so we must also exclude inputs use in the latest confirmed funding transaction.

Because this isn't using the same type anymore, we cannot factor ConfirmedFundingTx which only has a Transaction, not a SharedTransaction.

You could stil factor on Transaction, but I suppose you didn't want to call sharedTx.tx.buildUnsignedTx()?

Exactly, we'd be losing information if we did that, and would be including remote inputs as well.
When we only have a Transaction we can't avoid it, but when we can, I think we should, it's more correct that way.


API get null response

I always got null response after I trained nlu model
"response_selector":{"default":{"response":{"name":null,"confidence":0.0},"ranking":[],"full_retrieval_intent":null}
intent prediction is correct, but no response actions.
Appreciate if someone help me with this!!

FYI, I use luis migrated model

Thanks for the question, our Github issue board is for bugs and feature requests only, though 😕
Please ask this question in the forum✨
If you were describing a bug, please create a report and fill out the template.

Thanks for raising this issue, @closed will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


Documentation📄: Open Horizon supports Intel-only Macs, not arm-based Macs

Clarify that Open Horizon supports Intel-only Macs, not arm-based Macs.
Several pages might need to be updated to clarify the differences between Intel-only Macs, not ARM64 Macs.
including docs/installing/adding_devices.md

I reviewed the resulting pages:  (I double bounce on Intel only - will fix in the next update to adding_devices.md )

https://open-horizon.github.io/docs/installing/adding_devices.html

Closed #262 via #285


Goal indicators fix

Fix to proper vertically align goal indicators when the budget row
height feature is active.

Thank you!


Added more support for bundle API

Added support for PUT, POST, DELETE endpoints for bundle API
added support for loading credentials from file (updated README)

implements https://github.com/IBM-Bluemix/gp-python-client/issues/10


[MP Twitter Social Media Call to Action in January 2021] to push in 2021 schedule

Let's refresh and review the messges and designs.  Required: the  creation of new git tickets for each of the items:

Speaker session submission Let's highlight the Branding page and drop all the stuff in that page instead of the wiki.

Starter

Youtube

SharingIsCaring- Content

Community Call

White Paper
It needs to be updated.

Onboarding Interview
It seems that we can add a call to action in the Contributors tab...

ebooks: currently 2

New call to actions media pushes:

Steering Committee calls to access and reminders (4 times per year) already in the calendar

MPWG page push

Prospects quote highlight: each ORG gets its own branding card highlighted in the card (say 1 per month- the same week, all quotes are pushed). The template needs to be created for Quotes and to be used by all quotes, ORGs, and individuals.

Contributors page push

Projects page push (when fixed)

Wondering

Inventory document

the work continues with inventory listed in the MP Drive + individual tickets per item.  https://drive.google.com/drive/u/0/folders/11LJ6nRwDIiID0oMeR_TCGZZoMb-ya7Ky


Peculiar behavior with tf.self_adjoint_eig

Hi,
I've come across a really peculiar bug that I think is actually a sign of a much larger problem, but what that is, I have no idea.  The following code:
x = np.random.randn(10, 10)

x = (x+x.T)/2

X = tf.placeholder(dtype=tf.float32, shape=[10, 10])

E, V = tf.self_adjoint_eig(X)

with tf.Session() as sess:
	e, v = sess.run([E, V], feed_dict={X: x})

	print(e.shape)

throws the error:
raise TypeError("'Tensor' object is not iterable.")

but curiously:
x = np.random.randn(10, 10)

x = (x+x.T)/2

X = tf.placeholder(dtype=tf.float32, shape=[10, 10])

E = tf.self_adjoint_eig(X)

with tf.Session() as sess:
	e = sess.run(E, feed_dict={X: x})

	print(e.shape)

returns (11, 10), and I have no idea how that's even possible, since the docs say that tf.self_adjoint_eig returns two tensors.  While investigating this, I discovered that a lot of my install (from pip) differs substantially from what's documented here even though I have the latest version (tensorflow==0.10.0rc0 from pip freeze).  In particular, tf.self_adjoint_eig is not defined in linalg_ops.py, and I can't find where it's being called from.  Any ideas?
Thanks,
Shawn
Update:
I just found it where it's called from.  According to the comment in gen_linalg_ops.py:
  The result is a M+1 x M matrix whose first row is the eigenvalues, and
  subsequent rows are eigenvectors.

This is different than the API docs.  One or the other (from a usefulness standpoint, probably the code) should be changed.
Further Update:
This actually might be an install problem as mentioned above.  The code here calls self_adjoint_eig_v2 which doesn't exist in the latest install.  Why is that?

Hi @shawnjhenry . 0.10.0rc0 has the older implementation where self_adjoint_eig returns a single value. The documentation you are reading online is the live version with this particular change:
============================================================
Changes {batch_}self_adjoint_eig ops to return eigenvalues and eigenv…
…ectors as separate outputs. This is implemented as a new set of kernels SelfAdjointEigV2 and BatchSelfAdjointEigV2. {batch_}self_adjoint_eig become Python wrappers that call the new kernels.
Adds new ops self_adjoint_eigvals and batch_self_adjoint_eigvals that compute the eigenvalues but not the eigenvectors, which is faster and uses less memory than computing both.
NOTICE: This changes the public API and TensorFlow models calling tf.self_adjoint_eig or tf.batch_self_adjoint_eig from Python will most likely break. Models saved as serialized graphs should be unaffected.
This CL also cleans up the python shape inference functions in linalg_ops.py a bit.
Change: 129390113
So you code will work as you expect with the next release.

Adding @rmlarsen for further comments.

Thanks!  Any idea when the next release will be?

BTW, it may be available under nightlies --
https://github.com/tensorflow/tensorflow#installation
On Tue, Aug 16, 2016 at 1:06 PM, Shawn Henry<EMAIL_ADDRESS>wrote:

Thanks! Any idea when the next release will be?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3854#issuecomment-240221340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABaHBcyouv4IT-Hk0pylUo2QbbSVbj8ks5qghgogaJpZM4Jlwwv
.

It is.  Thanks for the suggestion!

Sounds like it's resolved? I will close the issue then. Thanks!


Which IP do I point my domain at?

In the screencast example, it shows two ingress resources: echoserver and kube-lego. Each one has 2 external IP addresses. Which one do you use for the demo.kube-lego.jetstack.net A record?
It'd be helpful to include this in the docs.

The External IP listed under the echoserver kubectl service is what gets used for the A record.

I think this depends on the environment you are running your cluster in.
In general you want to expose only the ports 80/443 of the nginx pods. This is done by the nginx service with the type LoadBalancer. The ingress information is misleading as it contains only the public addresses of the nodes, that run the nginx pods.
You can get the right IPs/hostnames with kubectl describe svc nginx.
For AWS the service creates an ELB and you should point the domain to the Hostname via CNAME.
The hostname should look like that: <name>.eu-west-1.elb.amazonaws.com
For AWS the service creates an Forwarding Rule and you can create an a record to the LoadBalancer Ingress in the describe output.
Gonna add this to the example docs later

Got it, thanks. Is it ok to increase the nginx deployment replica count or would they conflict? Since nginx is proxying all traffic I'd like to have more than one running in case of failure.

Yeah sure for a production-like use case you want to have more replicas. There is not really a conflict.
The only problem is around TLS Session caching and ticketing. As the nginx instances don't share the cache nor the secrets for ticketing, session resumption is is not working and the TLS has to re-initialised every time it hits another nginx instance.

Would it make sense to write cache to a dir and read it on startup? That way we could use persistent volumes to share cache among replicas.

One way could be to generate a key for tickets and disable session cache: http://nginx.org/en/docs/http/ngx_http_ssl_module.html#ssl_session_ticket_key
(This key has to be rotated quite regularly.)
To share the session cache you have to modify nginx. (ssl_session_cache has to support something like redis/memcache) (I think there is something like ssl_session_store_by_lua on openresty's road map)

I am now at least mentioning how to find out about the IP address/hostname
https://github.com/jetstack/kube-lego/commit/422b42bfbc17ea43e1debfd80baf256bc2247831


Missing reaction names in Human-GEM

Description of the issue:
Only 4724 reaction out of 13069 has reaction names given in Human-GEM model. Although I can see the reaction codes from rxns list under the model, rxnNames list does not consists the majority of their names.
Expected feature/value/output:
All reactions to have their reaction names stated in rxnNames list.
Note: I tried to download the latest version manually as well for the model, but the reaction names are still missing.
I hereby confirm that I have:

[x] Tested my code on my own computer for running the model
[x] Done this analysis in the main branch of the repository
[x] Checked that a similar issue does not exist already

Thank you for the question @Selimvrl.
There have been long-standing issues regarding the reaction names, and only incremental have been made so far in https://github.com/SysBioChalmers/Human-GEM/issues/333 and https://github.com/SysBioChalmers/Human-GEM/issues/181.
If your question is open-ended, then we would love your input and suggestions in #181, in which case this issue could be closed. If you question, however, is more related to the reaction annotation, then it would be great if you could share more about your use-case.

duplicate to #181


feat(artifact): rename secret to token

I'd rename this also in auth
https://github.com/i-am-bee/bee-api/blob/6c203cc28a8423151e2fcb398c4c8da9bfcccf39/src/ui/ui.module.ts#L57


Fix deleting all Pods in namespace via details view

Description
Changes proposed in this pull request:

Change the way we filter pods from an URL to a filter function

Related issue(s)

/retest


SVG not visible in courses.html

In course page the .svg image is not working  beside achievements

File Courses.html

Have a look at your about.html page. I think that's where you want to place the .svg image next to achievements (but maybe not).
https://moonlight-7k.github.io/suhelkhan.github.io/about.html

It was issue of the scr link at line 63.


Alt and friends

See https://github.com/fantasyland/fantasy-land/pull/197
@safareli could you take a look?

Will do tomorrow 🍭

Looks good

Great, thank you for taking a look!

Welcome!


Feature/adjust api

Work in progress

Closing this in favor of the second pull request for the same features.


request: Include zls in devShell

Thanks for making this flake!
I was thinking that it'd be nice if zls was included in the devShell, or at least optionally. Especially since at the moment in nixpkgs zig is 0.13 but zls is stuck at 0.12.

I was able to add this to my own flake, so not having it isn't a blocker to me or anything. So I'm okay if you decide not to add zls to this project.
If anyone is searching and comes to this issue here is how to add zls to your flake:
diff --git a/flake.nix b/flake.nix
index fa18218..43512f9 100644
--- a/flake.nix
+++ b/flake.nix
@@ -2,12 +2,14 @@
   description = "Zig project flake";

   inputs = {
+    zls.url = "github:zigtools/zls?ref=0.13.0";
     zig2nix.url = "github:Cloudef/zig2nix";
   };

-  outputs = { zig2nix, ... }: let
+  outputs = { zig2nix, zls, ... }: let
     flake-utils = zig2nix.inputs.flake-utils;
   in (flake-utils.lib.eachDefaultSystem (system: let
+      zlsPkg = zls.packages.${system}.default;
       # Zig flake helper
       # Check the flake.nix in zig2nix project for more options:
       # <https://github.com/Cloudef/zig2nix/blob/master/flake.nix>
@@ -79,6 +81,8 @@
       apps.zon2nix = env.app [env.zon2nix] "zon2nix \"$@\"";

       # nix develop
-      devShells.default = env.mkShell {};
+      devShells.default = env.mkShell {
+        nativeBuildInputs = [zlsPkg];
+      };
     }));
 }

I was thinking about zls integration, but i think having it in the project flake like you have done is the best way to go.

Ok thanks!


Disfavor shared optional dynamic member lookup

This makes the default work as expected, avoiding the issue brought up in #3169.
Technically this breaks the following invocation:
if let wrapped = $shared.optional { /* ... */ }

And limits the warning to:
func share<T>(_: Shared<T>?) {}
share($shared.optional)

We consider this lookup to be a bug, though, which is already deprecated in 1.11.0.
Since we encourage folks to migrate by one minor version at a time, we should only merge this when we plan on releasing 1.12.0.

What would be the proper way to do something like this in newer versions of TCA? I'm seeing the subscript(dynamicMember:)' is deprecated warning:
 guard let row = state.$presentedData[id: itemId] else { return .none }

@acosmicflamingo The warning should give a hint. Is it not helpful?
You can use Shared.init to explicitly unwrap an optional shared, just like Binding.init.


Not meant to call addPart on MultiPartBlockStateBuilder

In: https://docs.neoforged.net/docs/resources/client/models/datagen/#configuredmodelbuilder
Code provided by the docs:
MultiPartBlockStateBuilder multipartBuilder = getMultipartBuilder(MyBlocksClass.EXAMPLE_BLOCK.get());
// Add a new part. Starts with .part() and ends with .end().
multipartBuilder.addPart(multipartBuilder.part()
    // Step one: Build the model. multipartBuilder.part() returns a ConfiguredModel.Builder,
    // meaning that all methods seen above can be used here as well.
    .modelFile("minecraft:block/cobblestone")
    // Call .addModel(). Now that the model is built, we can proceed to step two: add the part data.
    .addModel()
    // Add a condition for the part. Requires a property
    // and at least one property value; property values are a vararg.
    .condition(BlockStateProperties.FACING, Direction.NORTH, Direction.SOUTH)
    // Set the multipart conditions to be ORed instead of the default ANDing.
    .useOr()
    // Creates a nested condition group.
    .nestedGroup()
    // Adds a condition to the nested group.
    .condition(BlockStateProperties.FACING, Direction.NORTH)
    // Sets only this condition group to be ORed instead of ANDed.
    .useOr()
    // Creates yet another nested condition group. There is no limit on how many groups can be nested.
    .nestedGroup()
    // Ends the nested condition group, returning to the owning part builder or condition group level.
    // Called twice here since we currently have two nested groups.
    .endNestedGroup()
    .endNestedGroup()
    // End the part builder and add the resulting part to the multipart builder.
    .end()
);

You are not meant to call addPart directly
"you're meant to use the fluent interface of part() (which returns a builder for a model)"
"and then call addModel() on that builder afterwards" - sciwhiz12
Link to chat in discord: https://discord.com/channels/313125603924639766/1249305774987939900/1301716230461456405

Also i want to point out that in the docs when they use withExistingParent() EVERYTIME they just put 1 string, when in code it needs 2 strings or 1 string and one resource location

Also want to add in here: https://docs.neoforged.net/docs/resources/client/models/datagen/#item-model-datagen
It uses MyItemsClass.EXAMPLE_BLOCK_ITEM.get() but it actually expects, again, two strings or one string and one resourcelocation


haveImageID: -f -> --format

haveImageID: `-f` -> `--format`

Ethier this has never worked or the Docker CLI's API has changed. The
current meaning of the `-f` flag is filter, not format.

For me this restores thumbnails in the web interface for HEIC images, as I don't have a recent version of imagemagick on my machine.

@bradfitz review plz!

@bradfitz review plz!

Turns out this is not true, something else fixed it.

Turns out this is not true, something else fixed it.


Non-interactive list

I love the fzf integration, but sometimes I just want to see a list of contexts or namespaces.
This adds a -l/--list option to execute non-interactive ListOp.

Sorry but this is a huge behavioral change, and it is by design.
Instead of doing a -l/--list you can do kubectx | cat or disable fzf (either use the env override to do so, or uninstall).
Consider opening a proposal first for notable user-visible changes, as the written code might get rejected.

No problem. I implemented it for myself, and thought it might be useful to others.


Fix get bulk secret for hashicorp vault.

Description
Fix get bulk secret for hashicorp vault.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: None
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

I was surprised when I saw that:

The list call to KV was not even working.
It was not including the secret's key in the dictionary and building it in a funny way.

Let's all take a moment to thank Java SDK's ITs for saving our a** one more time.

I was surprised when I saw that:

The list call to KV was not even working.
It was not including the secret's key in the dictionary and building it in a funny way.

Let's all take a moment to thank Java SDK's ITs for saving our a** one more time.


light.piglow update

Description:
Update brightness logic and add assumed state functionality.
Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

🎉 🐬


A question about Experimental results

First, thank you for release your code.
I'm wondering about your experimental results.
First, reported distillation models' classification accuracy is the final checkpoint's accuracy or best model's accuracy?
Second, how similar the results are when running the experiment multiple times?

Hi
Here is details about reported performance.

For CIFAR-100, we conducted 5-times training. And median accuracy over 5 final models was reported.
For ImageNet and detection, performance of best model was reported.
For segmentation, performance of final model was reported.

Because I don't have results over multiple time, I cannot tell the exact numbers.
I'm sorry for that.
In my experience, the performance variance of distillation is similar to that of training a network with a baseline.


QuotaExceededError: The quota has been exceeded.

What I was doing : create a new web storie (the first one), add two pictures, select one of them, then click on "layer".
QuotaExceededError: The quota has been exceeded.
in Panel
in SimplePanel
in LayerStylePanel
in Unknown
in form
in designPanel__Form
in DesignPanel
in DesignPanels
in div
in designInspector__TopPanels
in div
in designInspector__Wrapper
in DesignInspector
in div
in inspectorContent__InspectorPane
in section
in inspectorContent__InspectorPanes
in Inspector
in div
in inspectorLayout__InspectorContainer
in section
in inspectorLayout__Layout
in InspectorLayout
in div
in dropTarget__Content
in Unknown
in div
in dropTarget__DropTargetComponent
in UploadDropTarget
in Unknown
in InspectorProvider
in Yh
in q
in div
in layout__InspectorArea
in Unknown
in RichTextProvider
in Workspace
in Unknown
in UnitsProvider
in Unknown
in CanvasProvider
in section
in layout__Editor
in Unknown
in ForwardRef(WithOverlay)
in Unknown
in LayoutProvider
in rm
in PrepublishChecklistProvider
in Unknown
in DropTargetsProvider
in Unknown
in TransformProvider
in Unknown
in MediaProvider
in Unknown
in FontProvider
in Unknown
in StoryProvider
in Unknown
in SnackbarProvider
in Unknown
in HistoryProvider
in Unknown
in Media3pApiProvider
in Unknown
in FileProvider
in Unknown
in APIProvider
in Unknown
in ConfigProvider
in q
in Ge
in se
in App
in StrictMode
in FlagsProvider

Thanks for your report @GDMarket! Which browser are you using, and were you perhaps using it in private/incognito mode?

I'm using Firefox 84.0.2 64bits, and I was in non-private mode. In fact, when I click on any menu item on the right, it "crashes".

Thanks for your report @GDMarket! Which browser are you using, and were you perhaps using it in private/incognito mode?

I'm using Firefox 84.0.2 64bits, and I was in non-private mode. In fact, when I click on any menu item on the right, it "crashes".

@GDMarket Very helpful, thanks!

@GDMarket Very helpful, thanks!

@GDMarket Very helpful, thanks!

You're welcome.
It says: "5200000 characters were stored successfully, but 5300000 weren't."

@GDMarket Very helpful, thanks!

You're welcome.
It says: "5200000 characters were stored successfully, but 5300000 weren't."

@carlos-kelly Another Pea pod ticket I believe. There should be  try { } catch blocks everywhere when working with localStorage.setItem


Adding missing import and arguments

Was missing import and arguments in tutorial-service.md.

Looks like we ended up fixing this in another PR, so closing this one out.  Thanks @brunocriado!


Replace boilerplate code with FOCA archetype

A lot of the code shared between this service and others in the organizationhas been moved to the FOCA archetype.
Duplicate code should be removed from this service and, where necessary, the remaining code should be refactored to make use of FOCA.

@sarthak-sopho: for your interest

Sure Alex :)

Great - note that if you do that, you can copy this meta issue to proTES, proWES etc as well (see issue description for full list). It's probably better to change one module at a time in all services that use that module rather than changing all modules in one service in one go.

Yeah, I'll change one module at a time.

Guess at this point it makes sense to replace everythin at once :)

Picking this up on priority as this is posing as a blocker.


Switch to using singleton function in service provider

Laravel 5.4 depreciated the container share function

Thanks !


Node 16 upgrade

We are receiving warnings from Github actions about node 12 would be deprecated and that this action is still using it.
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>I noticed there is an open PR https://github.com/abhilash1in/aws-secrets-manager-action/pull/45 with the upgrade, any ETA on when this will be merged?

@ejgutierrezvega I ran into the same issue, and switched over to using aws-actions/aws-secretsmanager-get-secrets@v1 which is very similar to this one.