[faucet] new faucet service: replicates existing faucet behavior

New faucet service, put it under client as it is one kind of full node client application.
This diff is focusing on replicating current faucet service feature, improvement will be added later.

Accepts auth key, amount and currency code for creating online testing account with coins transferred from a dd account.
Service start with command line arguments to keep consistent with other service we built. Default values of arguments are set for testnet deployment.
Integrated with libra logger, added simple bootstrap and request log.
Updated libra-swarm to start with new faucet service for manual testing.
Creating account and transfer coins are designed as batch transactions, fire and forget. Client need wait for transaction complete. Resubmit same auth key with new amount / currency code is fine, creating account transaction will fail, but transfer coins will success.
Added a todo for improvement planned.

Docker script is not updated yet, focus on service implementation in this diff.
Test Plan

manual test with libra-swarm
unit test.

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Old faucet was written in python right? Seeing this PR literally made my day. Out of curiosity where does the old faucet code live?

Old faucet was written in python right? Seeing this PR literally made my day. Out of curiosity where does the old faucet code live?

it's docker/mint/server.py

Where does the creation of the DD account happen?

Where does the creation of the DD account happen?

wont be added in this diff, aiming for fix existing problem first before we add new feature to support other use cases.

Where does the creation of the DD account happen?

added into todo list #5943.
We need focus on fixing existing problem first before we add new feature to support other use cases.

Oh I was asking about "Accepts auth key, amount and currency code for creating online testing account with coins transferred from a dd account" <- I  wasn't sure what you meant here. Are you saying faucet is funding accounts through a token DD  account?

Where does the creation of the DD account happen?

added into todo list #5943.
We need focus on fixing existing problem first before we add new feature to support other use cases.

Oh I was asking about "Accepts auth key, amount and currency code for creating online testing account with coins transferred from a dd account" <- I wasn't sure what you meant here. Are you saying faucet is funding accounts through a token DD account?

Yes, didn't realize this is not well known, as we even discussed to have a rich dd account shared for getting coins in Testnet.
So, in current faucet with cli implementation, mint coins does 2 things:

create parent vasp by treasure compliance account
transfer coins from dd account (addres: 0xdd)

Hence you don't see creating dd account. The 0xdd is created by genesis with max coins available for all 3 testnet currencies.

@bors-libra land


how to Close the GoRename floating window

Hello, after the recent update, the location of GoReame has become a floating window. how configure it to close and change it to the previous location?

the nmap for closing floating windows is <ESC>

Actually, I want to disable the floating window, not close it


[python][tuner] Add bindings for MMAIntrinsic

Expose MMA intrinsics to python. This includes both the MMAIntrinsic attribute and the enum values. In addition, expose element/vector types and shapes from the MMA interface (but not the interface itself).
This is so that the tuner can use the compiler as the SoT for all the intrinsics and use them to parse target properties and generate constraints.
I defined the list of intrinsic enum values using x macros -- not very thrilled about this, but this does reduce a lot repetition across C API and pybind code.

I chatted with @makslevental and I want to stop mirroring enums on the CAPI side and instead pass around opaque uint32_t. I'm going to convert this to a draft for the time being.


unit test coverage for actionmenu, basic message protocols; l10n as d…

…ecorator in basicmessage
Signed-off-by: sklump<EMAIL_ADDRESS>

Codecov Report

Merging #284 into master will increase coverage by 0.9%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #284     +/-   ##
=========================================
+ Coverage   80.01%   80.91%   +0.9%
=========================================
  Files         230      230
  Lines       10416    10416
=========================================
+ Hits         8334     8428     +94
+ Misses       2082     1988     -94


Bot not working and confusion

Actually I am first of all confused whether to use a discord bot/bot token or a discord acc/like a real acc token to use the auto catcher.
And I thought of using the discord bot and used it's token to use the autocatcher and filled all the keys properly and the bots starts too but it is here when the problem begin after the bot start.
I used $spam and it didn't work it just goes to the channel and dosent type a single spam.
It shows the bots typing but even a single isn't sent please help me with this I need it so bad.
Please contact me at discord Emerpus#2607
Catch e'm is my bots name I will attach a photo here if I can.

You can give suggestions please.


[bug] TypeError: write() argument must be str, not bytes

Updating source someonewhocares.org from http://someonewhocares.org/hosts/zero/hosts

Updating source hosts-file.net from http://hosts-file.net/download/hosts.txt

Updating source gambling from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/gambling-hosts

Updating source malwaredomainlist.com from http://www.malwaredomainlist.com/hostslist/hosts.txt

Updating source tyzbit from https://raw.githubusercontent.com/tyzbit/hosts/master/data/tyzbit/hosts

Updating source social from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/social-hosts

Updating source StevenBlack from https://raw.githubusercontent.com/StevenBlack/hosts/master/data/StevenBlack/hosts

Updating source porn from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/pornography-hosts
Updating source adaway.org from http://adaway.org/hosts.txt

Updating source mvps.org from http://winhelp2002.mvps.org/hosts.txt

Updating source yoyo.org from https://pgl.yoyo.org/adservers/serverlist.php?hostformat=hosts&mimetype=plaintext&useip=<IP_ADDRESS>

Traceback (most recent call last):
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 504, in <module>
    main()
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 132, in main
    finalFile = removeDupsAndExcl(mergeFile)
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 312, in removeDupsAndExcl
    writeData(finalFile, line)
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 67, in writeData
    f.write(bytes(data, 'UTF-8'))
TypeError: write() argument must be str, not bytes

@StevenBlack

@StevenBlack
Issue happened since https://github.com/StevenBlack/hosts/commit/d67a7bc3be2c31e3b4870b7d236cb868e7939aec
Script from https://github.com/StevenBlack/hosts/commit/ec5178b7a581b40d7c4083b12785b75ecce80a6e working fine.

Thanks Tomasz! @FadeMind

@StevenBlack
Nope. The same issue still:
OK, we'll only exclude domains in the whitelist.
Traceback (most recent call last):
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 504, in <module>
    main()
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 132, in main
    finalFile = removeDupsAndExcl(mergeFile)
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 312, in removeDupsAndExcl
    writeData(finalFile, line)
  File "/home/tomasz/.hup/hosts/updateHostsFile.py", line 67, in writeData
    f.write(bytes(data, 'UTF-8'))
TypeError: write() argument must be str, not bytes

What version of Python, what OS?

archlinux x86_64
python 3.5.1-2
python2 2.7.11-
but python call python 3.5.1

@StevenBlack
calling python2 updateHostsFile.py working fine.
calling python updateHostsFile.py (python 3.5.1) make this error.

[tomasz@arch hosts]$ python2 updateHostsFile.py
Do you want to update all data sources? [Y/n] n
OK, we'll stick with what we've  got locally.
Do you want to exclude any domains?
For example, hulu.com video streaming must be able to access its tracking and ad servers in order to play video. [Y/n] n
OK, we'll only exclude domains in the whitelist.

Success! The hosts file has been saved in folder
/home/tomasz/.hup/hosts/
It contains 370,820 unique entries.
Do you want to replace your existing hosts file with the newly generated file? [Y/n] Moving the file requires administrative privileges. You might need to enter your password.
Flushing the DNS Cache to utilize new hosts file...
[tomasz@arch hosts]$

Will fix for Python3.

Commit c565266416f69f7d1e93ae9e2157b9e1c25b9258  walla!  :-)

:+1:


Two columns display

Hi Jacob,
Is it possible to have a 2 columns text display ? Like in the PHB and other WotC books ?
Thanks !

Ok, thank you for the information.
Too bad we can't see the document as it will be displayed in PDF, but i totally understand.


Testing the pre trained model

Hi,
I am trying to test the pre trained models with my own images using "FT_eval_all_live.py", but I am getting the following error
Traceback (most recent call last):   File "src/eval/FT_eval_all_live.py", line 50, in <module>     x =  im.shape[0] IndexError: tuple index out of range
There is a ".txt" file used for testing, and can you please explain what are those scores appended to each of the test image, so that I can test it on my images ?
Any help would be appreciated !!

The error might because the im is empty,  I guess you have some problems about the input file. You could check if the directory is correct. The scores in LIVE dataset are from 0-100, the smaller score, the better quality.

Hello,
Thank you for the response.
That issue was handled. Can you please guide me how can i test my own
images using the pre-trained model FT_live.caffemodel.
I was able to evaluate the python script src/eval/FT_eval_all_live.py, but
now i am stuck what to do next.
Any leads would be appreciated.
Thanks in advance.
On Mon, Jun 11, 2018 at 5:48 PM, Xialei Liu<EMAIL_ADDRESS>wrote:

The error might because the im is empty, I guess you have some problems
about the input file. You could check if the directory is correct. The
scores in LIVE dataset are from 0-100, the smaller score, the better
quality.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xialeiliu/RankIQA/issues/13#issuecomment-396223758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY2i21E0Kpy2N3-6MD_nX3EBkjeSpDngks5t7mAPgaJpZM4UiF4I
.

To test on your own images, you need first to prepare the same format of .txt file. Then you need to modify the script src/eval/FT_eval_all_live.py to get the network's output in stead of computing LCC and SROCC, which is the final score you need.

You mean I need to create my database in the LIVE2 format database? If yes,
then how can I do it?
Also please help me in modifying src/eval/FT_eval_all_live.py to get the
output.
As I am an amateur in this field and I am trying to learn more, any help
from your side would be appreciated.
Thank you!
On Mon, Jun 11, 2018 at 8:13 PM, Xialei Liu<EMAIL_ADDRESS>wrote:

To test on your own images, you need first to prepare the same format of
.txt file. Then you need to modify the script src/eval/FT_eval_all_live.py
to get the network's output in stead of computing LCC and SROCC, which is
the final score you need.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xialeiliu/RankIQA/issues/13#issuecomment-396268272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY2i285OMtA5I07B7CdJSJg1N-0K4FUTks5t7oH5gaJpZM4UiF4I
.

pre[i] is the average score for a particular image, and med[i] is the median score, because for each test image, 30 patches are sampled from the whole image.

And can you please tell me if my approach is correct or not?
On Tue, Jun 12, 2018 at 4:21 PM, Xialei Liu<EMAIL_ADDRESS>wrote:

pre[i] is the average score for a particular image, and med[i] is the
median score, because for each test image, 30 patches are sampled from the
whole image.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xialeiliu/RankIQA/issues/13#issuecomment-396548393,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY2i2-qYmAwHttATY65Q8CxbXanC0KpFks5t751BgaJpZM4UiF4I
.

Yeah, it's correct.

I am getting the following error for some of the image files:
Traceback (most recent call last):
File "src/eval/FT_eval_all_live.py", line 78, in
x_p = np.random.randint(x-224,size=1)[0]

File "mtrand.pyx", line 993, in mtrand.RandomState.randint
ValueError: low >= high
Can you please explain?
On Wed, Jun 13, 2018 at 4:17 PM, Xialei Liu<EMAIL_ADDRESS>wrote:

Yeah, it's correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xialeiliu/RankIQA/issues/13#issuecomment-396895555,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY2i2w6AV1L5dHwvc75pZbxZLApvqdXeks5t8O3UgaJpZM4UiF4I
.

I guess you have problem with reading images.


Leafmap save image to static image (JPG or PNG) not working.

Environment Information

leafmap version: 0.3.0
Python version: 3.8.7
Operating System: MacOS Big Sur Version 11.2.3, jupyter-notebook : 6.3.0

Description
I'm trying to save the image as JPG or PNG but when I use the method m.to_image() it ends up taking a screenshot of my Desktop Background image.
What I Did
from leafmap import leafmap
m = leafmap.Map()
m  # map got rendered

m.to_image(outfile='./map.png') # run this command on another cell

Yes, it is essentially a screenshot. You need to use the fullscreen mode and then click the toolbar icon to save the image. Update the package using leafmap.update_package() and try it out. You will need two screens if you want to use the to_image() function directly, i.e., fullscreen map on one screen, and run the code on another screen.


From where do you come up with the ratio / weight list ??

Hi ,
I want to know From where do you come up with the ratio / weight list ??
weights = [0.2,0.9,0.7,0.4,0.4,0.3]

Hi ,
I want to know From where do you come up with the ratio / weight list ??
weights = [0.2,0.9,0.7,0.4,0.4,0.3]

Hi,
I have tried different weights of the prediction of the models, and the weight in the code would allow my ensembled models to reach the highest performance in the local dataset.
As a result, I applied this weight to the testing environment.
Anyway, this is a trick.


An error is reported when using the MQTTX client with last will to connect.

https://github.com/mochi-co/mqtt/blob/46babc89c82dd3ae4f45fbffb84ace8b75de1a7c/packets/packets.go#L398
got error: WRN error="read connection: malformed packet: will properties"

This may be related to #206 - try v2.2.8 and let me know if it still occurs.

It's fixed on v2.28，Thx


builder: remove hardcoded bazel version

It seems that older versions of packages are cleaned up after some time
from copr repository. Therefore, it was not possible to build a builder
image as it required a non-existing version of bazel rpm package.

build fails when using newer bazel version


Added bundle build and push on master

What this PR does / why we need it?:
Adds the bundle push command to the make file and has travis build and push the bundle for merges in the master branch

Does this PR introduce a user-facing change?

[ ] User guide
[ ] CHANGELOG.md

Which issue(s) this PR fixes:

Fixes #

@BradleyMayo Looks good. Thanks. Merging.


Make battle music only activate for hostile mobs, and some neutral mobs

Description:
Maybe allow for battle music to only be played for entities that extend EntityMob or similar?  Keep hearing the battle music when my Animania rams butt heads (because they apparently are supposed to become hostile to each other)

I will need to take a look at this.  I would think there would be a way to filter some more.

Alrighty, no need to rush, take your time.
On Thu, Dec 28, 2017 at 10:28 AM, OreCruncher<EMAIL_ADDRESS>wrote:

I will need to take a look at this. I would think there would be a way to
filter some more.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OreCruncher/DynamicSurroundings/issues/190#issuecomment-354305594,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACr5pDCHiSrfkSZzky5phgNfZPZKhiroks5tE7OZgaJpZM4Q98Zk
.

--
Golden rule in Minecraft: Never dig straight down.

Main issue at the moment is not enough coffee. :)

Heh, then more coffee is in order.

Which version of MC are you using?  I want to post a test JAR for you to try out.

1.11.2

Give this a whirl: https://github.com/OreCruncher/DynamicSurroundings/releases/tag/v1.11.2-<IP_ADDRESS>T2

What's supposed to change?
On Thu, Dec 28, 2017 at 1:20 PM, OreCruncher<EMAIL_ADDRESS>wrote:

Give this a whirl: https://github.com/OreCruncher/
DynamicSurroundings/releases/tag/v1.11.2-<IP_ADDRESS>T2
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OreCruncher/DynamicSurroundings/issues/190#issuecomment-354331803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACr5pLOzqZjOsWo5xTIYF4zHjV-K8UIsks5tE9vugaJpZM4Q98Zk
.

--
Golden rule in Minecraft: Never dig straight down.

The rams shouldn't trigger the battle music.  Assuming they aren't IMobs.

Okay, testing with my medieval mod setup, as that has a bunch of rams in close quarters, meaning they fight often.

Well it works flawlessly.

Nice!  The change will be in the next official build.  Thanks for checking.

No problem.  If you manage to compile a test jar of the sound engine fix,
I'd love to test it out.
On Fri, Dec 29, 2017 at 11:46 AM, OreCruncher<EMAIL_ADDRESS>wrote:

Nice! The change will be in the next official build. Thanks for checking.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OreCruncher/DynamicSurroundings/issues/190#issuecomment-354469983,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACr5pKxUsLM8kOphPMuuCLU3pZkkuhXXks5tFRdigaJpZM4Q98Zk
.

--
Golden rule in Minecraft: Never dig straight down.

Actually the test JAR you currently have contains that fix as well.

Oh, wow.  This could be the fix I've waited for.  This'll save a lot of problems.


copilot-debug terminal integration breaks python venv activation

Copilot Chat Extension Version: 0.23.2
VS Code Version: 1.96.2
OS Version: Windows 11
Logs:

Steps to Reproduce:

Start with a clean VSCode profile
Install the python extension
Create a virtual environment and set it as the python interpreter
Open a new terminal and run (get-command python).Source (or equivalent for your shell): it should point to the python interpreter from the virtual environment, because the Python extension activates the environment for you and prepends it to env:PATH
Now, install Copilot Chat and restart VSCode
Open a new terminal and re-run (get-command python).Source: this time, it should point to the global python interpreter.

The issue seems to be that both Python extension and Copilot Chat seem to be trying to modify the PATH at the same time, probably overriding each others changes (even if in my case, Copilot Chat seems to always override Python).
Even worse, Copilot Chat seems to completely ignore the "github.copilot.chat.copilotDebugCommand.enabled": false setting, with the extension still trying to add features to the terminal.

copilot-debug only appends to the path, it doesn't overwrite it. If this is destructive I think there might be something up with how Python does the contribution or how the terminal applies them
https://github.com/microsoft/vscode-copilot/blob/a803c8aa2cdcd8d7a381c663916814b8c9b16479/src/extension/onboardDebug/vscode-node/copilotDebugCommandContribution.ts#L223-L229

copilot-debug only appends to the path, it doesn't overwrite it. If this is destructive I think there might be something up with how Python does the contribution or how the terminal applies them
https://github.com/microsoft/vscode-copilot/blob/a803c8aa2cdcd8d7a381c663916814b8c9b16479/src/extension/onboardDebug/vscode-node/copilotDebugCommandContribution.ts#L223-L229

From what I can see from Show Environment Contribution, both Python and Copilot append/prepend to the PATH by taking the "old path" and prepending/appending to it:
Python:
PATH=c:\Users\myuser\.vscode\extensions\ms-python.python-2024.22.0-win32-x64\python_files\deactivate\powershell;C:\Users\myuser\customer_repos\myfolder\.venv\Scripts;${env:PATH}
Copilot:
PATH=${env:PATH};c:\Users\DanieleAbbatelli\AppData\Roaming\Code\User\globalStorage\github.copilot-chat\debugCommand
My guess is that, for both exensions, the "old path" (${env:PATH}) refers to the PATH before contributions from other extensions.


Decide the technological stack

What kind of technologies will we use.

Scaffolding
We will use create-react-app because it provides all the necessary tools in order to quickly set up a new project.
UI Framework
The are many options here. We have some well-known projects:

Bootstrap it is getting old
Material UI, it is too heavy
Semantic UI, it is too heavy
... and the list never stops

Our requirement here is to use a lightweight library that will boost productivity for the limited deadline. Grommet seems a decent alternative. It uses css-in-js, a modern API and it is used by Netflix and Boeing. css-in-js is the perfect solution for a quick project.
Prettier
For keeping the code consistent
ESLint
For listing our code
jest, react-testing-library
For testing our code
CI/CI
We have a static frontend SPA. So we can use an automated tool that will deploy the static content.
We will try to use github actions in order to setup the CI/CD process and automated testing.


Units and tweaks

This adds units to various numerical properties in the inspector:

(please ignore the "acceleration" property; this comes from a separate PR)

And also resets the player's velocity when they lose a life.
https://phabricator.endlessm.com/T35700

These are good! I'll make a note to myself to remind people to add units with these kinds of exports. They make a big difference :)


Content for Sender Consent to Use Organization Name

We need content for a consent from Senders to allow the CDC to use their Organization's name in presentations and on the website.

I'm reaching out to Rick Hawes to see if we need legal counsel involved in this feature.


Use modern error handling and enforce it via lint

use error un-/wraping with errors.Is and errors.As
we need to make sure this does not introduce regressions

Codecov Report
Base: 44.74% // Head: 44.80% // Increases project coverage by +0.06% :tada:

Coverage data is based on head (974162b) compared to base (f88c70b).
Patch coverage: 28.57% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
+ Coverage   44.74%   44.80%   +0.06%
==========================================
  Files         137      137
  Lines        9868     9864       -4
==========================================
+ Hits         4415     4420       +5
+ Misses       5191     5182       -9
  Partials      262      262

Impacted Files
Coverage Δ

pipeline/frontend/yaml/error.go
0.00% <ø> (ø)

server/model/cron.go
0.00% <0.00%> (ø)

server/remote/gitea/gitea.go
32.59% <0.00%> (ø)

server/remote/gitlab/gitlab.go
21.97% <0.00%> (ø)

server/shared/userSyncer.go
0.00% <0.00%> (ø)

...er/store/datastore/migration/000_legacy_to_xorm.go
63.71% <0.00%> (ø)

server/store/datastore/migration/common.go
38.94% <0.00%> (ø)

server/store/datastore/migration/migration.go
36.00% <0.00%> (ø)

server/model/secret.go
49.01% <66.66%> (ø)

server/cron/cron.go
36.70% <100.00%> (ø)

... and 2 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Dotnet 4468 add back fx sample apps

Add back ASPNetFrameworkApiApplication and SampleAspNetFrameworkApp.
Fix azure-pipelines.yml to build .Net framework apps.
Test build succeeded:

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Create base classes for common settings

At this time AzureDevOpsBuildSettings exists. For calling API functions which doesn't requires the buildid itself, it would be nice to have different settings e.g. AzureDevOpsSettings without the buildid (Collection, Authentication and ProjectGuid/ProjectName).

@pascalberger Is this still required as we introduced the AzureDevOpsBuildSettings class?

@christianbumann #254 introduces three base classes with common settings: One containing credentials, one for collection information and one for project information.


[APR-205] Start to flatten out framing and codecs

The goal here is just to start reducing some complexity so that subsequent changes are easier to make and simpler to review. Where previously we had a Deserializer wrapping a framer wrapping a codec, we now just have a framer and a codec that are used side-by-side. By avoiding the nesting, we're making it easier to introduce a semi-parsed result struct from the codec without needing to thread reference lifetimes through multiple layers of machinery. Actually implementing that change to the codec's parsing function will be the next step, and from there we will be able to rework the rest of the logic (enrichment, differentiating between message types, forwarding, etc) to be a more straightforward function from that semi-parsed representation to a full event, with fewer intermediate steps and less mutation, as well as our ultimate goal, to easily pass in a string/context interner specific to that socket handler.
There's likely to be a few bits that were dropped that need to be added back (as well as some things that ought to be simplified further given the new state), so don't hesitate to point anything out in review 😅

/merge

/merge


problems with display('file.geojson')

Hi.
I'm trying to open my geojson file containing some polygons, applying exactly the code reported but the tool opens the browser without showing my data. If I manually upload my data on geojson.io they are correctly visualized.
Can you help me?
thanks

Hi @Magdalena88,
I noticed the issue title is display('file.geojson'), is that how your code is using geojsonio?
If so, you first have to read the contents, then call display. Would this code from the README work?
with open('file.geojson') as f:
    contents = f.read()
    display(contents)

Does that work?

Thanks for your fast replay :)
Yes, I already tried that code and the issue is related to it. :(

Would you feel comfortable posting the polygon data you're using somewhere along with the code that will reproduce the bug?
My guess is that when the data is too big, we can't use geojson.io this way - the Python library creates an anonymous Github gist then tells geojson.io to download it. When it's too big geojson.io can't download the entire gist. But it works when uploading the file locally. That's just a guess though.

Yes of course. The code I'm using is:
from geojsonio import display
with open('AOI.geojson') as f:
contents = f.read()
display(contents)
f.close()
I have the same issue also when trying to open a single polygon. I'm copying you the polygon data below:
{
"type": "FeatureCollection",
"features": [
{
"geometry": {
"type": "Polygon",
"coordinates": [
[
[
30.84825076017768,
29.84449235157097
],
[
31.59090692089696,
29.83395854639918
],
[
31.58332490917348,
30.21149316913375
],
[
30.89876172494296,
30.24098056978401
],
[
30.84825076017768,
29.84449235157097
]
]
]
},
"type": "Feature",
"properties": {
"Name": "AOI",
"Description": ""
},
"id": 0
}
]
}

@Magdalena88 Hmm...
Here are the contents of my script.py and AOI.geojson:
Running $python script.py opens up the window with a small rectangle in Egypt. I noticed your code has f.close() which isn't needed because of the with but it shouldn't affect the rest of the script. Can you confirm that this doesn't work on your end?
from geojsonio import display

with open('AOI.geojson') as f:
    contents = f.read()
    display(contents)

AOI.geojson:
{
"type": "FeatureCollection",
"features": [
{
"geometry": {
"type": "Polygon",
"coordinates": [
[
[
30.84825076017768,
29.84449235157097
],
[
31.59090692089696,
29.83395854639918
],
[
31.58332490917348,
30.21149316913375
],
[
30.89876172494296,
30.24098056978401
],
[
30.84825076017768,
29.84449235157097
]
]
]
},
"type": "Feature",
"properties": {
"Name": "AOI",
"Description": ""
},
"id": 0
}
]
}

P.S. if it still doesn't work, what OS and browser+version are you using?

my OS is Windoes 7 and I use Chrome 50.0.2661.102 :)
2016-05-18 16:19 GMT+02:00 Jacob Wasserman<EMAIL_ADDRESS>
P.S. if it still doesn't work, what OS and browser+version are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jwass/geojsonio.py/issues/5#issuecomment-220040637


Mac build instructions

It seems the Homebrew libtiff does not work but the conda libtiff/openslide does. @zachmullen could you try this and let me know if it works for you as well?

I followed these instructions, but it's still segfaulting for me when I try to mark a tiff as a large image.

Oh well, since it's not a reproducible fix I suggest we don't merge this in this case. If we want to revisit sometime we can, but docker-compose can remain the environment for Mac.

@zachmullen Did you uninstall libtiff from your initial brew installation before installing the tiff tile source via conda?  My guess is that the brew installation of libtiff is the issue.  Otherwise, it'd be worth capturing machine differences between you and @jeffbaumes to try to understand the issue.

It would not let me uninstall the original brew version, however I did verify that the conda one was the one on the DYLD_LIBRARY_PATH in python. There's an outside chance pylibtiff was somehow finding the old one; I didn't get that deep, but the old one only worked when I explicitly put it on my DYLD_LIBRARY_PATH.

And FWIW this is my minimal segfault repro script:
from libtiff import TIFF

tif = TIFF.open('/Users/zach/Downloads/TCGA-OR-A5J1-01A-01-TS1.CFE08710-54B8-45B0-86AE-500D6E36D8A5.svs', mode='r')
tif.read_image()

Relevant bits of the crash report (sadly the system report doesn't contain the names of the python symbols in the call stack)
Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000000004
Exception Codes:       0x0000000000000001, 0x0000000000000004

Termination Reason:    Namespace SIGNAL, Code 11 Segmentation fault: 11
Terminating Process:   exc handler [34310]

VM Region Info: 0x4 is not in any region.  Bytes before following region:<PHONE_NUMBER>19676
      REGION TYPE                    START - END         [ VSIZE] PRT/MAX SHRMOD  REGION DETAIL
      UNUSED SPACE AT START
--->
      MALLOC_NANO (reserved)<PHONE_NUMBER>00-600020000000 [128.0M] rw-/rwx SM=NUL  ...(unallocated)

Application Specific Information:
crashed on child side of fork pre-exec

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   libtiff.6.dylib               	       0x11c573220 _TIFFVGetField + 788
1   libtiff.6.dylib               	       0x11c572f44 _TIFFVGetField + 56
2   libtiff.6.dylib               	       0x11c571674 TIFFGetField + 120
3   libffi.8.dylib                	       0x1023a404c ffi_call_SYSV + 76
4   libffi.8.dylib                	       0x1023a17d4 ffi_call_int + 1336
5   _ctypes.cpython-310-darwin.so 	       0x119580424 _ctypes_callproc + 1332


fix(RecurringRuns): translations moved to common

These are the translations that fixed some issues in the Recurring Runs component.

Test locally.


Fabric 1.20.x version not recognizing REI.

Hi, I love your mod, and I've added it to both of my custom modpacks (one on Forge 1.20.x and one on Fabric 1.20.x), and I noticed that the Forge version seems to work with REI as well as JEI, but the Fabric version doesn't see REI and just tells me "JEI is missing". Please Disregard. I'm assuming it works with REI because the REI team added an REI Plugin Compatibility mod for Forge.

I also didn't realize there was a newer version of JustEnoughBreeding available. Updating to that seems to have fixed my issue.

That is because I am yet to implement official REI support.

Oh. I'm assuming it's planned, though, based on that response.

@Gamemassa Implemented in version 1.1.0


Adding an About Us Page

Fixes Issue🛠
Closes # 488
Description👨💻
While going across the website, I noticed that the About Us page was missing. So
I decided to add an About Us page into the website.
Type of change📄

[] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added demonstration in the form of GIF/video file
[X] I am an Open Source Contributor

Screenshots/GIF📷

@Rakesh9100 Kindly update the status of my PR

@Rakesh9100  kindly review my PR

@Rakesh9100  sir kindly update the status of the PR ,and let me know if it is ok or not . It has already been more than 3 days for the pull request.

Pls work on these changes:

About Page looks so clumsy and congested, pls enhance and design the UI.
Include the navbar and footer sane as homepage.
Don't delete the paragraph of the main page.

@Soumya6Tiwari

@Rakesh9100  ok I shall make the amendments

Status of your changes @Soumya6Tiwari

@Rakesh9100  you can reassign it to someone else


Batoto unable to open chapter

Source information
Bato.to 1.4.34
Source language
All
Steps to reproduce

Open up a random batoto comic
Open a comic
(Sorry for orientation)

https://github.com/keiyoushi/extensions-source/assets/75614340/a936869d-9a7f-4757-818d-04edb792cdd2
Expected behavior
Open a chapter normally
Actual behavior
Page not found on a chapter
Tachiyomi version
Mihon 0.16.3
Android version
MIUI 14 Android 13
Other details
This is due to the batoto website side has updated on their side
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Bump
Same issue on tachiyomiSY 1.10.2

Bump
Same issue on tachiyomiSY 1.10.2

Please use the thumbs-up reaction if you have the same issue. Bumping adds nothing to the discussion.

Can confirm same is happening with TachiyomiJ2K v1.7.4 + batoto v1.4.34

Same issue on tachiyomiSY 1.10.2

Please use the thumbs-up reaction if you have the same issue. Bumping adds nothing to the discussion.

Thanks for advising me on the etiquette!
"Bump" text removed, post liked.
I left my tachi version in the comment still. Lmk if that's useful info, if so I will remove that too


calendar not correct

@ChiperSoft  when i using kalendae the date and day is mismatch.. it shows wrong date.. for example it shows april 4 2017 as sunday in my calander.. but the day of april 4 is tuesday , can u help me to resolve this problem

Are you running your browser at a font size other than 100%? I've seen this issue when people are running at 90%.
Kalendae uses padding as part of the width calculation for individual days, so if you change the font sizes this breaks the calendar.


Wait for file completed in fuse getAttr (#8192)

cherry-pick of #8192
cherry-pick commits are fb36eea63268 9c78eaa12b36 70eb188fa485 c270a101fad6

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4616/
Test FAILed.

jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4617/
Test FAILed.

Failed tests:
  AlluxioFuseFileSystemTest.getattr:153 expected:<0> but was:<-14>

@gpang looks like I need this change first #9499 (though this one might also need this current one?)

alluxio-bot, merge this please


Convert TiLifecycleObserverTest to Kotlin

Converts TiLIfecycleObserverTest to Kotlin and MockK. Only exception is the test remove other observer() test which I have left Mockito based because MockK lacks equivalent of verifyNoMoreInteractions().
Please merge after #174 .

Is there already an open issue at mockk? If not please create one.
Please also refer the issue here please 👍
On Wed, Dec 26, 2018, 10:17 PM lukaszkalnik<EMAIL_ADDRESS>wrote:

Converts TiLIfecycleObserverTest to Kotlin and MockK. Only exception is
the test remove other observer() test which I have left Mockito based
because MockK lacks equivalent of verifyNoMoreInteractions().
Please merge after #174
https://github.com/grandcentrix/ThirtyInch/pull/174 .
You can view, comment on, or merge this pull request online at:
https://github.com/grandcentrix/ThirtyInch/pull/177
Commit Summary

add kotlin support to thirtyinch module tests; convert
TiTestPresenterTest and TestPresenterExampleTest to kotlin
rename test function
use verifyOrder
use testImplementation for kotlin stdlib
update gradle plugin
Revert "update gradle plugin"
convert TiLifecycleObserverTest to kotlin and mockk

File Changes

M build.gradle
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-0 (2)
M thirtyinch/build.gradle
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-1 (3)
D
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/TiLifecycleObserverTest.java
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-2
(339)
A
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/TiLifecycleObserverTest.kt
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-3
(303)
D
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/test/TestPresenterExampleTest.java
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-4
(54)
A
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/test/TestPresenterExampleTest.kt
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-5
(47)
D
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/test/TiTestPresenterTest.java
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-6
(144)
A
thirtyinch/src/test/java/net/grandcentrix/thirtyinch/test/TiTestPresenterTest.kt
https://github.com/grandcentrix/ThirtyInch/pull/177/files#diff-7
(130)

Patch Links:

https://github.com/grandcentrix/ThirtyInch/pull/177.patch
https://github.com/grandcentrix/ThirtyInch/pull/177.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grandcentrix/ThirtyInch/pull/177, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJwYe3nBEyvfoQZL3UVID_IpuWZDiW41ks5u8-d7gaJpZM4ZiGHv
.


[BUG][Native/Android] The icons on the left side menu do not match the icons of the same category on the web page

**Description:**The icons on the left side menu do not match the icons of the same category on the web page
Steps to reproduce:

Launch the app
Login into the account
Press the icon menu on the header
Pay attention to the icons

Expected result:
The icons on the left side menu match the icons of the same category on the web page
Actual result:
The icons on the left side menu do not match the icons of the same category on the web page
The user is new: No
Environment:  App Version 1.13.0-0 (983)
Devices list: Xiaomi Mi8 // Android 8.1.0
Additional info:
The user is registered with self custody method
Backup words: bottom duty enroll crew fabric window warrior cabbage goat potato fine neither
Notes: Also reproduced with the "Copy ID" button on the Profile Privacy screen
Attachment:

@MarianoAmado this looks ok on my dev env.
maybe it is about copying the font assets to the main folder?
if you dowload the app and install  on your device/emulator does it also happen?

The "Copy ID" icon is trimmed on the Profile Privacy screen

Note: The issue with that the icons on the left side menu do not match the icons of the same category on the web page is not reproduced.
Checked on the Android App Version 1.21.0 (1074)
Device: Xiaomi Mi8 // Android 8.1.0

The "Copy ID" icon is trimmed on the Profile Privacy screen

Note: The issue with that the icons on the left side menu do not match the icons of the same category on the web page is not reproduced.
Checked on the Android App Version 1.21.0 (1074)
Device: Xiaomi Mi8 // Android 8.1.0

then please change issue title and description

then please change issue title and description

@sirpy  The issue is not reproduced. The "Copy ID" icon is displayed fully on the Profile Privacy screen
Checked on the Android App 1.22.13-1 (1204)
Device: Xiaomi Mi 8 // Android 8.1.0
Attachment:


Adding bibliography entries (where available) to Google-released datasets.

I've assembled together some of the references describing the datasets previously released by Google and hosted on OpenSLR.
Please see the attached google-openslr-bib.zip. The name of each bibliography file in the directory corresponds to the SLR ID of the dataset.
Would it be possible to modify the index pages to include the corresponding bibentry verbatim?

I'll resolve this for now. May add more in the future.
Thank you guys!


Test the release process and wheels at each commit.

It's difficult to modify the release code because it's not run in the CI at each commit. We also sometimes have bugs that appear only in the wheel. My plan for fast feedback without making the tests too long:
at PR and commits on master:

test for py35 on linux, macos, windows, test py37 for linux only (forward compatibility)
produce the wheels after the tests (the cache is there, so ~10s)
we group the wheels together (upload-wheels)
We add a IF statement at the action pypa/gh-action-pypi-publish@master to disable it.
We download the wheels in a fresh environements on windows/linux/max and install them, along with tensorflow, install them and run a simple import tensorflow_addons check (to check dependencies and python files are all there).
Also run a very short sanity check where a simple tensor is created and passed through a single activation function (test that a single .so is loaded correctly).

---> ~1-2 minutes longer than the current tests
At each release:
We do exactly the same but

We also test for py36, py37 on linux, macos, windows
We also run the action pypa/gh-action-pypi-publish@master.

---> a bit shorter than the current version because we create the wheels after the tests, when the cache is hot.
In practice, it means we have only 3 yaml files in github actions: the backport, the sanity checks and the "tests+releases+nighly" file.
This should remove a lot of code duplication too.

Good to close?

It's missing a small step:

We download the wheels in a fresh environements on windows/linux/max and install them, along with tensorflow, install them and run a simple import tensorflow_addons check (to check dependencies and python files are all there).
Also run a very short sanity check where a simple tensor is created and passed through a single activation function (test that a single .so is loaded correctly).

It's missing a small step:

We download the wheels in a fresh environements on windows/linux/max and install them, along with tensorflow, install them and run a simple import tensorflow_addons check (to check dependencies and python files are all there).
Also run a very short sanity check where a simple tensor is created and passed through a single activation function (test that a single .so is loaded correctly).

:smile:  arguably the most important part


Form element value with type non-string is transformed to string

When some form element value is not a string (number or bool for instance) returns that value as string handleSubmit.
I can reproduce this even in the builder (https://react-hook-form.com/builder). Steps to reproduce it:

Go to https://react-hook-form.com/builder
Add one field as number and generate form.
Fill data in the form (i.e. 7)
Look at Watch (or Submit after submitting ) and see the value is a string ("7" rather than 7)

hey @pabloibanezcom ,
that's actually the standard behavior for all input. if you try this in a chrome dev tool, inspect the value of an input. it will be string.

You're right @bluebill1049 . I close the issue.


Single page app

Current coverage is 85.71%

Merging #31 into master will not affect coverage as of 9b1f81f

@@            master     #31   diff @@
======================================
  Files            1       1
  Stmts           28      28
  Branches         8       8
  Methods          0       0
======================================
  Hit             24      24
  Partial          4       4
  Missed           0       0

Review entire Coverage Diff as of 9b1f81f

Powered by Codecov. Updated on successful CI builds.

#24

Merging to master @yongjun21


Test message not working

Sending mail from my apps/mail server through postal works fine, but the test mail (e.g. postal test-app-smtp<EMAIL_ADDRESS>gives me the following error:
Message was not delivered successfully to SMTP server.
Error: OpenSSL::SSL::SSLError (SSL_connect SYSCALL returned=5 errno=0 state=SSLv3/TLS write client hello)

The smtp_server.log says the following:
ERROR -- : [TOXWNK] An error occurred while processing data from a client.
ERROR -- : [TOXWNK] Errno::EACCES: Permission denied @ rb_sysopen - /etc/letsencrypt/live/example.com/fullchain.pem

My smtp config looks like this:
smtp:
  port: 25
  from_name: postal
  from_address: postal@example

Postal runs as user postal and had Read/write permission on /etc/letsencrypt/live/example.com/fullchain.pem (is also owner of the file)
I run ruby 2.5.5p157
Anyone know how to fix this?

I have come a little further to a solution. When I run postal as root it works, but not if I run it as the postal user.


matrix-synapse: 0.14 -> 0.16

This updates matrix-synapse to the less buggy 0.16 release, and updates and expands configuration support through the NixOS module. The python dependency bleach is also updated to the latest patch revision to satisfy matrix-synapse's requirements.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Why remove bind_port/unsecure_bind_port ? It's still supported by synapse along with listeners. Having both would be really nice, as it'd avoid breaking existing installations of synapse :)
Also, you're mixing up spaces and tabs. It makes reading the nix file in my editor a bit painful.

Updated with precise option types, restored backwards compatibility, and fixed assorted bugs. toJSON was not adopted because the precise option types cause various metadata to clutter up the objects.

Lots of iteration, but at this point I think I'm satisfied that this is ready to merge.

Built locally. It works :)


Accented characters for Croatian language

Only: č, ć, đ, š, ž

I am willing to add this - related proposal on PowerToys

Done: https://github.com/damienleroy/PowerAccent/releases/tag/1.3.4

Thank You!


Add flutter update packages to some tests.

Analyzer_benchmark and flutter_test_performance require to pub get multiple directories. Rather than delegating the recipes to run flutter update-packages we are explicitly running on tests that need it.
Bug: https://github.com/flutter/flutter/issues/75524
Pre-launch Checklist

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

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

@christopherfujino this PR is ready for review.

@godofredoc The build is broken on this PR because of an analyzer error, but I'm not sure if it's related...  Do you have any idea? https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8855549842472159072/+/steps/run_test.dart_for_flutter_plugins_shard_and_subshard_analyze/0/stdout

build is broken on this PR be

The change in this PR was to run flutter update-packages for the analyzer test. The one failing is flutter_plugins, let me retry it to see if it was a flake.

Looks like it's still failing with the same error.

The failure is unrelated to this PR.


fix issue 16948 - broken links in std.stdio due to inccorrect use of WEB macro

Thanks for the fix.

Auto-merge toggled on


Initial Port To Graph 2.2

Initial Port to Graph 2.2
-Raised Version Requirements
-Added Microsoft.Graph.Beta to requirements
-Replaced Applicable Cmdlets with Get-MgBeta* cmdlets
🗣 Description
Initial Port to Graph2.2 - it's not always viable to run 1.x & 2.x of the Graph SDK(module) side by side.
💭 Motivation and context
Scuba didn't work on graph2.0 :(
2.2 is the new version, making it work with the new version
It now requires Microsoft.Graph & Microsoft.Graph.Beta (2.0 or above, although I put it as 2.2 as I'm testing with that)
🧪 Testing
Tested (Non exhaustively!) via running against my own demo tenant, appears to be operating well & not complaining, albeit my experience with scuba is limited. Report appears to be as expected.
✅ Pre-approval checklist
I've read the documentation, as best as I can understand it appears to be very python focused, so my sincere apologies if I misunderstood - I'm new to both PR's and Scuba.
I greatly appreciate any and all mercy sent my way for my first attempt at helping :) .

[Y] This PR has an informative and human-readable title.
[Y] Changes are limited to a single goal - eschew scope creep!
[??] All future TODOs are captured in issues, which are referenced
in code comments.
[Y] All relevant type-of-change labels have been added.
[Y] I have read the CONTRIBUTING document.
[Y] These code changes follow cisagov code standards.
[Y ] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[Y ] Tests have been added and/or modified to cover the changes in this PR.
[ ] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Here's a shot of a successful run, solely on Graph 2.2 -- I plan on hunting the Powershell errors for AAD, but promising results currently...

@buidav - ok thank you for the information and thoughtful explanation - I have some more learning/investigation to do here, will be looking today. :)

@buidav looking significantly happier now, thanks! Still some work to be done, but we're on the right path :)

@schrolla  & @buidav - Hello! Just a quick update, looks like the (obvious) PowerShell issues have been eliminated and we're functional. I'm not knowledgeable (yet) in unit-testing Scuba from end to end, however, I'm eager to learn more. :)

@jpedlow We updated the baseline to use MS Graph 2.x.  It is currenty on our internal development branch, emerald, and will be in our next release.  See #514 for more details.  Thanks for all your help and suggestions.


testing: respect -count when running fuzz corpus tests

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

$ go version
go version go1.18 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/e/.cache/go-build"
GOENV="/home/e/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/e/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/e/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/nix/store/h2w6bc9dpcfvr76cfz8ymv7l5llfjjsj-go-1.18/share/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/nix/store/h2w6bc9dpcfvr76cfz8ymv7l5llfjjsj-go-1.18/share/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/e/Downloads/fuzzcount/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1871101059=/tmp/go-build -gno-record-gcc-switches"

What did you do?
$ cat fuzzcount_test.go
package fuzzcount

import "testing"

func TestA(t *testing.T) {
	t.Logf("TestA: ran")
}

func FuzzA(f *testing.F) {
	f.Add([]byte("test"))
	f.Fuzz(func(t *testing.T, _ []byte) {
		t.Logf("FuzzA: ran")
	})
}
$ go mod init fuzzcount
$ go test -count 5 -v .
=== RUN   TestA
    fuzzcount_test.go:6: TestA: ran
--- PASS: TestA (0.00s)
=== RUN   TestA
    fuzzcount_test.go:6: TestA: ran
--- PASS: TestA (0.00s)
=== RUN   TestA
    fuzzcount_test.go:6: TestA: ran
--- PASS: TestA (0.00s)
=== RUN   TestA
    fuzzcount_test.go:6: TestA: ran
--- PASS: TestA (0.00s)
=== RUN   TestA
    fuzzcount_test.go:6: TestA: ran
--- PASS: TestA (0.00s)
=== RUN   FuzzA
=== RUN   FuzzA/seed#0
    fuzzcount_test.go:12: FuzzA: ran
--- PASS: FuzzA (0.00s)
    --- PASS: FuzzA/seed#0 (0.00s)
PASS
ok  	fuzzcount	0.001s

What did you expect to see?
5 instances of FuzzA: ran.
What did you see instead?
Only one instance of FuzzA: ran.
Specifying -count XX to go test is for repeating test runs XX times; I'd expect the same behaviour for tests implied by a fuzz target corpus.

(attn @golang/fuzzing)


Replace property with flag in premake5.lua

Fixes this error on my machine:
Error: premake5.lua:23: attempt to call a nil value (global 'staticruntime')

I just saw the https://github.com/gmodstore/gluac/commit/6643b9a4ab0e7e1654a1d54b8d7bf9312c961c19 commit, but it just causes the error above.

Just compile premake5 :)
On Sat, 14 Apr 2018, 21:52 William Venner<EMAIL_ADDRESS>wrote:

I just saw the 6643b9a
https://github.com/gmodstore/gluac/commit/6643b9a4ab0e7e1654a1d54b8d7bf9312c961c19
commit, but it just causes the error above.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gmodstore/gluac/pull/1#issuecomment-381359789, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABUu6-RtlTl2wT_VgMr7E8BzzWpIgshoks5tomGWgaJpZM4TVOId
.


classify related-files into files-importing and files-imported

files-imported: list of files imported in this file
files-importing: list of files importing this file

these lists may have overlaps and may not be complete because they use just scope-graphs and not search-based navigation.

It is working fine, although a bit slow and sometimes times out


Can I translate your project in Korean?

I translated your project to Korean version.
https://github.com/devjiro76/emoji-cz-kr
Sorry about I didn't ask you before fork.
Question is late but can I translate your project in Korean?
If you mind I'll delete this repo.

That's totally OK, the package is licensed under MIT, you can read the full license here. Thanks for the appreciation.


Automate does not allow addition of organization of the same name in different Chef Servers.

Describe the bug
Automate does not allow addition of organization of the same name in different Chef Servers.
Automate UI produces error.
To Reproduce
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

OS: [e.g. rhel6]
Browser [e.g. chrome, safari]
Automate Build Number [e.g.<PHONE_NUMBER>2009]

Additional context
Add any other context about the problem here.

@kalroy
I have some scenerio like
user 1 - logged in
user 1 have 2 server's
server1 -
server have 1 org
org - 1
server2 -
if user going to create org with name org - 1 we need to restrict with same name
user 2 - logged in
user2 have 1 server
server 1 -
user have 1 org, org 21
user going to create 2nd org with with name org -1 in this case what we need to do?


Bump up dependency versions

I think I should remove unused crate features.

https://github.com/launchbadge/sqlx/issues/615


Analyze aibolit report

Let's analyze report (check this action) generated by aibolit and try to follow its recommendations to make our code better.
I'm proposing the following process:

choose class from the report to work on,
fix relevant problems and submit PR,
report bugs (if any) to Aibolit,
as for now there is no possibility to suppress bugs found by Aibolit, add comment here with small summary about the class you worked on: class name, problems left and why

Feel free to report any suggestion to Aibolit team and discuss any code related problems with me.
Do not close this issue when first PR is closed, I will close it when whole report by Aibolit is analyzed, or we will not need it anymore.

@acheshkov hi, thanks for integrating Aibolit into rpm-adapter! I'd like to start analyzing Aibolit report for rpm-adapter and reporting found problems (if any) to Aibolit. Don't you mind?
@g4s8 what do you think of proposed prosecc?

@olenagerasimova yes, I think you can report all problems here: https://github.com/yegor256/aibolit/


'jsdocCommentFound' event in plugins doesn't actually pay attention to comment changes.

The documentation for jsdocCommentFound in the readme suggests that you can use this event to modify
the contents of a comment before it is parsed.
Based on this, I made a plugin to add '@const' to every comment found (just a toy example, don't worry, wouldn't actually do it):
exports.handlers = {
    jsdocCommentFound: function (e) {
            // e.filename, e.comment, e.lineno
            // replace the last `*/` with `@const */`
            e.comment = e.comment.replace(/\s*\*\/\s*$/, '\n@const\n*/');
            console.log(e.comment);
    }
}

On a test file:
/** constant asdf */
const asdf = 'asdf';
/** constant fdsa */
const fdsa = 'fdsa';

This outputs the (modified) comments:
/** constant asdf
@const
*/
/** constant fdsa
@const
*/

so I know that the comment is being modified correctly.
Expected output
the doclets have kind set to constant and are documented as so (default template), as the modified comments are then fed in to the parser.
Actual output
the doclets have kind set to member and as such are not documented as constants.
Fix?
Either:

jsdocCommentFound event does not let you modify e.comment to modify a comment before it is parsed; then remove the documentation (from plugins/README.md) suggesting this can be done;
or, the (potentially modified) object e should be used when the event is fired to update the comment.

Or - how could I modify a comment's text before it gets parsed into a doclet? the fileBegin method is a bit too broad for me - I really need a per-doclet event.

Hiya @hegemonic & friends, so sorry to resurrect such an ancient issue, but I do believe there's been a regression here. Using<EMAIL_ADDRESS>I can recreate the issue using the original test case.
Incidentally, I originally ran into this problem by following the documentation for plugins. The text is clearly being modified during parsing, as the console logging shows. The output file, however, does not reflect those changes.
I'll never get those four hours back, but hopefully my loss is someone else's gain and this regression can be fixed up!
(ps using identical functions for both the jsdocCommentFound event and the symbolFound event is no longer an effective workaround)


Implement function types and closures

After #77

Closed. Needs https://github.com/austral/austral.github.io/issues/22 first.


Fix IE issues for mobile platforms

This PR:

Establishes Quality Settings explicitly for Desktop and Mobile targets, and sets the desktop/android defaults appropriately
Adds late-latching and removes device overrides from all IE example scenes, enforcing common rig settings across the board
Fixes a garbage-per-frame bug in the IE that snuck in with the Interaction Controller feature
Shifts max workstation distance (Example 6) to be 4 cm closer

To test:

[x] Ensure example scenes behave properly and hands don't judder
[ ] Confirm the Rig (service provider, timewarping) settings are the same across all example scenes
[x] Make sure example scenes function correctly on Android

Got a conflict now D:  Also you talk about quality and have modified the quality settings.  Are these settings very important?  Since like the other settings they will not be packaged up with the package when we release.

Good point, this PR shouldn't be merged until we've written out these
quality settings to the Wiki as one of the early points. I'll do that and
rewolve the conflict tomorrow morning.
On Tue, Jun 20, 2017, 6:24 PM Alex Marcolina<EMAIL_ADDRESS>wrote:

Got a conflict now D: Also you talk about quality and have modified the
quality settings. Are these settings very important? Since like the other
settings they will not be packaged up with the package when we release.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/leapmotion/UnityModules/pull/750#issuecomment-309936064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AERxMhKzN9gyqZ9db0yb54bkvAGDsZpFks5sGHDfgaJpZM4N_44Q
.

--
NOTE: The information in this message and any attachments is confidential
information intended only for the individual to whom it is addressed.  If
you are not the intended recipient, any distribution of this message or its
attachments is prohibited.  If you have received this email in error,
please immediately notify us and delete the message.  Thank you.

OK, this should be ready to merge if this page suffices for some project settings guidelines on mobile:
https://github.com/leapmotion/UnityModules/wiki/Recommended-Mobile-Project-Settings
(Desktop settings don't matter as much)

thanks @nickjbenson very helpful. this will be unlisted for now, right?

Yep! It's unlisted.


TaskExtensions: Rename AndForget() to CatchAndLog()

Description
The name Task.AndForget() implies discarding the task. However, AndForget() does something very important, it observes the task in the background and catches any exceptions which are then forwarded to the global unhandled exception handler in MudGlobal. If we can not await a task we should never simply discard it like this: _ = SomeMethodAsync();. Instead we should do SomeMethodAsync().AndForget();. To make the significance of not discarding more evident I renamed the method AndForget to CatchAndLog ...
SomeMethodAsync().CatchAndLog();  ... discards the task in a safe manner.
Type of Changes
Refactor

Name is not obvious about its intent to me at least. Something like FireAndForwardExceptions is longer but more self-explanatory and this method should be used sparsely anyway

Catch is pretty self explanatory, no need to explain it. Log as well, it expresses that exceptions are not discarded (as oposed to AndForget which hints at discarding) and I think Log it is better or at least as good as Forward. If we want maximum self-explanation we'd have to do DontAwaitButExecutInBackgroundAndCatchExceptionsAndForwardToGlobalExceptionHandler but I rather prefer a short and sexy name such as CatchAndLog which includes the most important details. It is self-evident from the code that we don't await, no need to express this in the name (that's also why I don't like AndForget which hints at the phrase fire-and-forget).

ObserveExceptions() would also be a good one

Or DiscardAndObserve()


Api version key

Add api version key as part of the required yaml configuration
See #1

Closes #1


Support for OpenID authorization

Is there a way to setup login access to the SC workbench with an OpenID (OIDC) provider?  If so are there instructions for setting that up?

Hi, at this moment only the SAML protocol is supported. OIDC is not supported right now.
Some IDPs support multiple protocols, so maybe you find the SAML docs useful. The instructions and an example oh how to set up SAML for Auth0 can be found here: https://github.com/awslabs/service-workbench-on-aws/blob/master/docs/docs/deployment/configuration/auth/configuring_auth0.md
Please follow the instructions on https://github.com/awslabs/service-workbench-on-aws/tree/master/docs to render the docs.


[stable/traefik] Fix traefik.toml parsing error

Signed-off-by: Le Thanh Tung<EMAIL_ADDRESS>
Is this a new chart
No

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This will fix the issue cause 'traefik.toml' parsed error after this commit: https://github.com/helm/charts/commit/afe9e3c99ad8782e930b77cf7d684d05872fc482
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #19267

Special notes for your reviewer:
N/A
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test


problems with the hxdiscord_rpc library

Describe your bug here.
Mind you, if you are modding, please try testing your bug in a clean version of the engine instead, this ensures the bug is actually caused by the engine itself.
so im trying to compile the engine and the only error im getting is "Error: Could not find haxelib "hxdiscord_rpc", does it need to be installed?" i keep installing it over and over and it keeps giving me the same error. why is this happening?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
Which branch of the engine are you using?
Latest (main)
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
i havent edited anything yet besides project.xml, in which i only edited the line with the window title.

Duplicate of https://github.com/ShadowMario/FNF-PsychEngine/issues/13771


Update module github.com/getsops/sops/v3 to v3.9.0

This PR contains the following updates:

Package
Type
Update
Change

github.com/getsops/sops/v3
require
minor
v3.8.1 -> v3.9.0

Release Notes

getsops/sops (github.com/getsops/sops/v3)
v3.9.0
Compare Source
Installation
To install sops, download one of the pre-built binaries provided for your platform from the artifacts attached to this release.
For instance, if you are using Linux on an AMD64 architecture:

### Download the binary
curl -LO https://github.com/getsops/sops/releases/download/v3.9.0/sops-v3.9.0.linux.amd64

### Move the binary in to your PATH
mv sops-v3.9.0.linux.amd64 /usr/local/bin/sops

### Make the binary executable
chmod +x /usr/local/bin/sops

Verify checksums file signature
The checksums file provided within the artifacts attached to this release is signed using Cosign with GitHub OIDC. To validate the signature of this file, run the following commands:

### Download the checksums file, certificate and signature
curl -LO https://github.com/getsops/sops/releases/download/v3.9.0/sops-v3.9.0.checksums.txt
curl -LO https://github.com/getsops/sops/releases/download/v3.9.0/sops-v3.9.0.checksums.pem
curl -LO https://github.com/getsops/sops/releases/download/v3.9.0/sops-v3.9.0.checksums.sig

### Verify the checksums file
cosign verify-blob sops-v3.9.0.checksums.txt \
  --certificate sops-v3.9.0.checksums.pem \
  --signature sops-v3.9.0.checksums.sig \
  --certificate-identity-regexp=https://github.com/getsops \
  --certificate-oidc-issuer=https://token.actions.githubusercontent.com

Verify binary integrity
To verify the integrity of the downloaded binary, you can utilize the checksums file after having validated its signature:

### Verify the binary using the checksums file
sha256sum -c sops-v3.9.0.checksums.txt --ignore-missing

Verify artifact provenance
The SLSA provenance of the binaries, packages, and SBOMs can be found within the artifacts associated with this release. It is presented through an in-toto link metadata file named sops-v3.9.0.intoto.jsonl. To verify the provenance of an artifact, you can utilize the slsa-verifier tool:

### Download the metadata file
curl -LO  https://github.com/getsops/sops/releases/download/v3.9.0/sops-v3.9.0.intoto.jsonl

### Verify the provenance of the artifact
slsa-verifier verify-artifact <artifact> \
  --provenance-path sops-v3.9.0.intoto.jsonl \
  --source-uri github.com/getsops/sops \
  --source-tag v3.9.0

Container Images
The sops binaries are also available as container images, based on Debian (slim) and Alpine Linux. The Debian-based container images include any dependencies which may be required to make use of certain key services, such as GnuPG, AWS KMS, Azure Key Vault, and Google Cloud KMS. The Alpine-based container images are smaller in size, but do not include these dependencies.
These container images are available for the following architectures: linux/amd64 and linux/arm64.
GitHub Container Registry

ghcr.io/getsops/sops:v3.9.0
ghcr.io/getsops/sops:v3.9.0-alpine

Quay.io

quay.io/getsops/sops:v3.9.0
quay.io/getsops/sops:v3.9.0-alpine

Verify container image signature
The container images are signed using Cosign with GitHub OIDC. To validate the signature of an image, run the following command:
cosign verify ghcr.io/getsops/sops:v3.9.0 \
  --certificate-identity-regexp=https://github.com/getsops \
  --certificate-oidc-issuer=https://token.actions.githubusercontent.com \
  -o text

Verify container image provenance
The container images include SLSA provenance attestations. For more information around the verification of this, please refer to the slsa-verifier documentation.
Software Bill of Materials
The Software Bill of Materials (SBOM) for each binary is accessible within the artifacts enclosed with this release. It is presented as an SPDX JSON file, formatted as <binary>.spdx.sbom.json.
What's Changed

Respect aws_profile from Keygroup Config by @Kouzukii in https://github.com/getsops/sops/pull/1049
build(deps): Bump the go group with 6 updates by @dependabot in https://github.com/getsops/sops/pull/1327
build(deps): Bump the go group with 3 updates by @dependabot in https://github.com/getsops/sops/pull/1328
build(deps): Bump the go group with 1 update by @dependabot in https://github.com/getsops/sops/pull/1330
build(deps): Bump github.com/docker/docker from 20.10.24+incompatible to 24.0.7+incompatible by @dependabot in https://github.com/getsops/sops/pull/1336
build(deps): Bump the go group with 3 updates by @dependabot in https://github.com/getsops/sops/pull/1334
Improve README.rst by @Ph0tonic in https://github.com/getsops/sops/pull/1339
Fix typos by @felixfontein in https://github.com/getsops/sops/pull/1337
build(deps): Bump the go group with 7 updates by @dependabot in https://github.com/getsops/sops/pull/1344
Add --mac-only-encrypted to compute MAC only over values which end up encrypted by @mitar in https://github.com/getsops/sops/pull/973
build(deps): Bump the ci group with 1 update by @dependabot in https://github.com/getsops/sops/pull/1347
build(deps): Bump the go group with 7 updates by @dependabot in https://github.com/getsops/sops/pull/1348
README: Correctly tag code as yaml-stream by @felixfontein in https://github.com/getsops/sops/pull/1350
build(deps): Bump the go group with 5 updates by @dependabot in https://github.com/getsops/sops/pull/1354
use getUserConfigDir also in tests by @bkreitch in https://github.com/getsops/sops/pull/1346
Overwrite home dir in vault test by @bkreitch in https://github.com/getsops/sops/pull/1349
Allow configuration of indentation for YAML and JSON stores by @Ph0tonic in https://github.com/getsops/sops/pull/1273
build(deps): Bump github.com/go-jose/go-jose/v3 from 3.0.0 to 3.0.1 by @dependabot in https://github.com/getsops/sops/pull/1357
build(deps): Bump the go group with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1360
build(deps): Bump the ci group with 1 update by @dependabot in https://github.com/getsops/sops/pull/1359
Fix TestGnuPGHome_Validate by @stormentt in https://github.com/getsops/sops/pull/1370
build(deps): Bump the go group with 6 updates by @dependabot in https://github.com/getsops/sops/pull/1373
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1376
Docs: indent is used both for encryption and decryption by @felixfontein in https://github.com/getsops/sops/pull/1372
fix a bug where not having a config results in a panic by @stormentt in https://github.com/getsops/sops/pull/1371
Polish the 'sops help' output a bit by @felixfontein in https://github.com/getsops/sops/pull/1341
introduce a --pristine flag to sops exec-env by @e-nikolov in https://github.com/getsops/sops/pull/912
Allow to pass multiple paths to 'updatekeys' by @sj14 in https://github.com/getsops/sops/pull/1274
Warn/fail if the wrong number of arguments is provided by @felixfontein in https://github.com/getsops/sops/pull/1342
Allow to override fileName with different value by @felixfontein in https://github.com/getsops/sops/pull/1332
Sort masterkeys according to decryption-order by @bkreitch in https://github.com/getsops/sops/pull/1345
build(deps): Bump the ci group with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1382
build(deps): Bump the go group with 5 updates by @dependabot in https://github.com/getsops/sops/pull/1381
Deprecate the --background option to exec-env and exec-file by @felixfontein in https://github.com/getsops/sops/pull/1379
build(deps): Bump golang.org/x/crypto from 0.16.0 to 0.17.0 by @dependabot in https://github.com/getsops/sops/pull/1383
build(deps): Bump the ci group with 1 update by @dependabot in https://github.com/getsops/sops/pull/1386
build(deps): Bump the go group with 5 updates by @dependabot in https://github.com/getsops/sops/pull/1385
Consolidate Flatten/Unflatten pre/post processing by @lancerushing in https://github.com/getsops/sops/pull/1356
INI, DotEnv stores: shamir_threshold is an integer by @felixfontein in https://github.com/getsops/sops/pull/1394
Improve integration tests: reformat test code, make sure exit codes are always checked by @felixfontein in https://github.com/getsops/sops/pull/1390
Make check whether file contains invalid keys for encryption dependent on output store by @felixfontein in https://github.com/getsops/sops/pull/1393
Create a constant for the 'sops' metadata key by @felixfontein in https://github.com/getsops/sops/pull/1398
build(deps): Bump github.com/cloudflare/circl from 1.3.3 to 1.3.7 by @dependabot in https://github.com/getsops/sops/pull/1408
build(deps): Bump the ci group with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1425
build(deps): Bump github.com/opencontainers/runc from 1.1.5 to 1.1.12 by @dependabot in https://github.com/getsops/sops/pull/1428
build(deps): Bump the go group with 10 updates by @dependabot in https://github.com/getsops/sops/pull/1429
build(deps): Bump the ci group with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1432
Warn if more than one command is used by @felixfontein in https://github.com/getsops/sops/pull/1388
Move extraction of encryption and rotation options to separate functions by @felixfontein in https://github.com/getsops/sops/pull/1389
Add separate subcommands for encryption, decryption, rotating, editing, and setting values by @felixfontein in https://github.com/getsops/sops/pull/1391
Add basic tests for exec-env and exec-file by @felixfontein in https://github.com/getsops/sops/pull/1396
build with go 1.21 and update dependencies by @b-dean in https://github.com/getsops/sops/pull/1427
build(deps): Bump the go group with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1439
sops: fix typo in interface name by @duthils in https://github.com/getsops/sops/pull/1477
Fix function name in comment by @pullmerge in https://github.com/getsops/sops/pull/1484
build(deps): Bump the ci group across 1 directory with 9 updates by @dependabot in https://github.com/getsops/sops/pull/1498
build(deps): Bump github.com/go-jose/go-jose/v3 from 3.0.1 to 3.0.3 by @dependabot in https://github.com/getsops/sops/pull/1454
build(deps): Bump google.golang.org/protobuf from 1.32.0 to 1.33.0 by @dependabot in https://github.com/getsops/sops/pull/1460
build(deps): Bump github.com/docker/docker from 24.0.7+incompatible to 24.0.9+incompatible by @dependabot in https://github.com/getsops/sops/pull/1466
build(deps): Bump golang.org/x/net from 0.21.0 to 0.23.0 by @dependabot in https://github.com/getsops/sops/pull/1489
build(deps): Bump actions/setup-go from 5.0.0 to 5.0.1 in the ci group by @dependabot in https://github.com/getsops/sops/pull/1503
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1508
LoadCreationRuleForFile: correctly handle nil without error in callers by @felixfontein in https://github.com/getsops/sops/pull/1506
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1510
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1516
Fix CVE-2024-2660 by @TheoBrigitte in https://github.com/getsops/sops/pull/1519
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1521
fix(Makefile): set locale to make tests pass by @jonasbadstuebner in https://github.com/getsops/sops/pull/1492
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1524
Adjust Makefile to new goreleaser 6.0.0 release by @felixfontein in https://github.com/getsops/sops/pull/1526
build(deps): Bump the go group across 1 directory with 23 updates by @dependabot in https://github.com/getsops/sops/pull/1525
update documentation for age by @tomaszduda23 in https://github.com/getsops/sops/pull/1399
exec-file: if --filename is used, use the provided filename without random suffix by @felixfontein in https://github.com/getsops/sops/pull/1474
build(deps): Bump github.com/Azure/azure-sdk-for-go/sdk/azidentity from 1.5.2 to 1.6.0 by @dependabot in https://github.com/getsops/sops/pull/1528
Do not use DotEnv store for exec-env, but specialized environment serializing code by @felixfontein in https://github.com/getsops/sops/pull/1436
build(deps): Bump the ci group with 2 updates by @dependabot in https://github.com/getsops/sops/pull/1534
Decryption: do not fail if no matching creation_rule is present in config file by @felixfontein in https://github.com/getsops/sops/pull/1434
add filestatus command by @endorama in https://github.com/getsops/sops/pull/545
build(deps): Bump the go group across 1 directory with 16 updates by @dependabot in https://github.com/getsops/sops/pull/1540
build(deps): Bump the go group across 1 directory with 4 updates by @dependabot in https://github.com/getsops/sops/pull/1543
add command unset by @duthils in https://github.com/getsops/sops/pull/1475
Update sops.go comment by @rjeczkow in https://github.com/getsops/sops/pull/1544
Support using comments to select parts to encrypt by @felixfontein in https://github.com/getsops/sops/pull/1392
Feature: merge key for key groups and make keys unique by @jonasbadstuebner in https://github.com/getsops/sops/pull/1493
Update all go deps by @felixfontein in https://github.com/getsops/sops/pull/1545
Release 3.9.0 by @felixfontein in https://github.com/getsops/sops/pull/1546

New Contributors

@Kouzukii made their first contribution in https://github.com/getsops/sops/pull/1049
@Ph0tonic made their first contribution in https://github.com/getsops/sops/pull/1339
@bkreitch made their first contribution in https://github.com/getsops/sops/pull/1346
@stormentt made their first contribution in https://github.com/getsops/sops/pull/1370
@e-nikolov made their first contribution in https://github.com/getsops/sops/pull/912
@sj14 made their first contribution in https://github.com/getsops/sops/pull/1274
@lancerushing made their first contribution in https://github.com/getsops/sops/pull/1356
@b-dean made their first contribution in https://github.com/getsops/sops/pull/1427
@duthils made their first contribution in https://github.com/getsops/sops/pull/1477
@pullmerge made their first contribution in https://github.com/getsops/sops/pull/1484
@TheoBrigitte made their first contribution in https://github.com/getsops/sops/pull/1519
@jonasbadstuebner made their first contribution in https://github.com/getsops/sops/pull/1492
@tomaszduda23 made their first contribution in https://github.com/getsops/sops/pull/1399
@endorama made their first contribution in https://github.com/getsops/sops/pull/545
@rjeczkow made their first contribution in https://github.com/getsops/sops/pull/1544

Full Changelog: https://github.com/getsops/sops/compare/v3.8.1...v3.9.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

39 additional dependencies were updated

Details:

Package
Change

cloud.google.com/go/iam
v1.1.2 -> v1.1.8

cloud.google.com/go/kms
v1.15.2 -> v1.18.0

filippo.io/age
v1.1.1 -> v1.2.0

github.com/Azure/azure-sdk-for-go/sdk/azcore
v1.8.0 -> v1.12.0

github.com/Azure/azure-sdk-for-go/sdk/azidentity
v1.4.0 -> v1.7.0

github.com/Azure/azure-sdk-for-go/sdk/internal
v1.3.0 -> v1.9.0

github.com/Azure/azure-sdk-for-go/sdk/security/keyvault/azkeys
v1.0.1 -> v1.1.0

github.com/ProtonMail/go-crypto
v0.0.0-20230923063757-afb1ddc0824c -> v1.1.0-alpha.3-proton

github.com/aws/aws-sdk-go-v2
v1.21.1 -> v1.30.0

github.com/aws/aws-sdk-go-v2/config
v1.18.44 -> v1.27.21

github.com/aws/aws-sdk-go-v2/credentials
v1.13.42 -> v1.17.21

github.com/aws/aws-sdk-go-v2/feature/ec2/imds
v1.13.12 -> v1.16.8

github.com/aws/aws-sdk-go-v2/internal/configsources
v1.1.42 -> v1.3.12

github.com/aws/aws-sdk-go-v2/internal/endpoints/v2
v2.4.36 -> v2.6.12

github.com/aws/aws-sdk-go-v2/internal/ini
v1.3.44 -> v1.8.0

github.com/aws/aws-sdk-go-v2/service/internal/presigned-url
v1.9.36 -> v1.11.14

github.com/cloudflare/circl
v1.3.3 -> v1.3.9

github.com/fatih/color
v1.15.0 -> v1.17.0

github.com/getsops/gopgagent
v0.0.0-20170926210634-4d7ea76ff71a -> v0.0.0-20240527072608-0c14999532fe

github.com/golang/protobuf
v1.5.3 -> v1.5.4

github.com/google/uuid
v1.3.1 -> v1.6.0

github.com/googleapis/enterprise-certificate-proxy
v0.3.1 -> v0.3.2

github.com/googleapis/gax-go/v2
v2.12.0 -> v2.12.5

github.com/hashicorp/go-retryablehttp
v0.7.1 -> v0.7.7

github.com/hashicorp/go-secure-stdlib/parseutil
v0.1.6 -> v0.1.8

github.com/hashicorp/go-sockaddr
v1.0.2 -> v1.0.6

github.com/hashicorp/vault/api
v1.10.0 -> v1.14.0

github.com/kr/pretty
v0.2.0 -> v0.3.1

github.com/mattn/go-isatty
v0.0.17 -> v0.0.20

github.com/pkg/browser
v0.0.0-20210911075715-681adbf594b8 -> v0.0.0-20240102092130-5ac0b6a4141c

github.com/urfave/cli
v1.22.14 -> v1.22.15

golang.org/x/crypto
v0.21.0 -> v0.24.0

golang.org/x/net
v0.22.0 -> v0.26.0

golang.org/x/sync
v0.3.0 -> v0.7.0

golang.org/x/sys
v0.18.0 -> v0.21.0

golang.org/x/term
v0.18.0 -> v0.21.0

golang.org/x/text
v0.14.0 -> v0.16.0

golang.org/x/time
v0.0.0-20220609170525-579cf78fd858 -> v0.5.0

google.golang.org/api
v0.146.0 -> v0.186.0


Command not valid for teams

This command does not work for teams; you have to use the undocumented command New-TeamsRetentionComplianceRule

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

ID: 0e8e582d-6b99-0a36-59e4-954e622154ce
Version Independent ID: bb512026-47d0-a419-d56a-1aa3ca57b255
Content: New-RetentionComplianceRule (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/New-RetentionComplianceRule.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@stephenc01, I'm told that the *TeamsChatLocation* and *TeamsChannelLocation* parameters in the *-RententionCompliancePolicy cmdlets are the backend components of the current UI M365 Compliance center and/or Security & Compliance Center.
The *-TeamsRetentionCompliancePolicy/Rule cmdlets were deprecated some time ago, but appear to still linger.
So are you saying that New/Set-RetentionCompliancePolicy allow you to select Teams locations, but you need to use New/Set-TeamsRetentionComplianceRule to get the retention rules to work properly?

@stephenc01, I'm told that the *TeamsChatLocation* and *TeamsChannelLocation* parameters in the *-RententionCompliancePolicy cmdlets are the backend components of the current UI M365 Compliance center and/or Security & Compliance Center.
The *-TeamsRetentionCompliancePolicy/Rule cmdlets were deprecated some time ago, but appear to still linger.
So are you saying that New/Set-RetentionCompliancePolicy allow you to select Teams locations, but you need to use New/Set-TeamsRetentionComplianceRule to get the retention rules to work properly?

@stephenc01, I'm told that the TeamsChatLocation and TeamsChannelLocation parameters in the *-RententionCompliancePolicy cmdlets are the backend components of the current UI M365 Compliance center and/or Security & Compliance Center.
The *-TeamsRetentionCompliancePolicy/Rule cmdlets were deprecated some time ago, but appear to still linger.
So are you saying that New/Set-RetentionCompliancePolicy allow you to select Teams locations, but you need to use New/Set-TeamsRetentionComplianceRule to get the retention rules to work properly?

Hey, @chrisda. This was part of MSCase 23287620. *RetentionCompliancePolicy does not work for teams location.
Description: This code produces the error below. $name = "Default XX Year Retention Teams [XX2]" $pol = New-RetentionCompliancePolicy -Name $name -TeamsChatLocation "XXXXXX" New-RetentionComplianceRule -Name $name -RetentionDuration 1825 -Policy $pol.Guid.Guid -RetentionDurationDisplayHint "Years" -RetentionComplianceAction "KeepAndDelete" -ExpirationDateOption "CreationAgeInDays" The parent policy 'FFO.extest.microsoft.com/Microsoft Exchange Hosted Organizations/XXXXXX.onmicrosoft.com/Configuration/Default XX Year Retention Teams [XX2]' is a Teams policy. Configuring non-Teams supported parameters of this rule are not allowed. + CategoryInfo : NotSpecified: (:) [New-RetentionComplianceRule], ErrorModifyingT...ameterException + FullyQualifiedErrorId : [Server=BN3NAM04WS024,RequestId=7f9c346e-987e-43b1-927e-bc5500874f38,TimeStamp=12/12/2020 5:29:03 PM] [FailureCategory=Cmdlet-ErrorModifyingT eamsRuleWithNonTeamsAllowedParameterException] E08251C8,Microsoft.Office.CompliancePolicy.Tasks.NewRetentionComplianceRule + PSComputerName : nam04b.ps.compliance.protection.outlook.com

@stephenc01, I'm told that the TeamsChatLocation and TeamsChannelLocation parameters in the *-RententionCompliancePolicy cmdlets are the backend components of the current UI M365 Compliance center and/or Security & Compliance Center.
The *-TeamsRetentionCompliancePolicy/Rule cmdlets were deprecated some time ago, but appear to still linger.
So are you saying that New/Set-RetentionCompliancePolicy allow you to select Teams locations, but you need to use New/Set-TeamsRetentionComplianceRule to get the retention rules to work properly?

Hey, @chrisda. This was part of MSCase 23287620. *RetentionCompliancePolicy does not work for teams location.
Description: This code produces the error below. $name = "Default XX Year Retention Teams [XX2]" $pol = New-RetentionCompliancePolicy -Name $name -TeamsChatLocation "XXXXXX" New-RetentionComplianceRule -Name $name -RetentionDuration 1825 -Policy $pol.Guid.Guid -RetentionDurationDisplayHint "Years" -RetentionComplianceAction "KeepAndDelete" -ExpirationDateOption "CreationAgeInDays" The parent policy 'FFO.extest.microsoft.com/Microsoft Exchange Hosted Organizations/XXXXXX.onmicrosoft.com/Configuration/Default XX Year Retention Teams [XX2]' is a Teams policy. Configuring non-Teams supported parameters of this rule are not allowed. + CategoryInfo : NotSpecified: (:) [New-RetentionComplianceRule], ErrorModifyingT...ameterException + FullyQualifiedErrorId : [Server=BN3NAM04WS024,RequestId=7f9c346e-987e-43b1-927e-bc5500874f38,TimeStamp=12/12/2020 5:29:03 PM] [FailureCategory=Cmdlet-ErrorModifyingT eamsRuleWithNonTeamsAllowedParameterException] E08251C8,Microsoft.Office.CompliancePolicy.Tasks.NewRetentionComplianceRule + PSComputerName : nam04b.ps.compliance.protection.outlook.com

@stephenc01, could the cause of this error be the RetentionDurationDisplayHint and ExpirationDateOption parameters don't apply to Teams retention? That's the vibe I'm getting...

@stephenc01, could the cause of this error be the RetentionDurationDisplayHint and ExpirationDateOption parameters don't apply to Teams retention? That's the vibe I'm getting...

@stephenc01 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@stephenc01 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


False Code Analysis Errors (pyflakes e) when using lambda

Problem Description
IDE's code analysis suggests an "undefined name/variable" when said name is used within a lambda function.
This is not a runtime error that presents the code from executing. In fact, the code works perfectly. This is a simple error flag from the IDE's code analysis feature.
What steps reproduce the problem?

Use the following python code in the IDE.
Look along the right-side of the code for a little red dot indicating a code analysis error (i.e. right next to the lambda function).
Code analysis error should state that a previously stated variable name is undefined (i.e. ffill_cols)

`
ffill_cols = [
'CSHFDQ',
'CSHOQ',
'DLTTQ',
'DPQ',
'PPENTQ',
'PSTKQ',
'MIBTQ',
]
interpolated_values = data.groupby('GVKEY', group_keys=False).apply(lambda group: group[ffill_cols].interpolate(method='linear')).copy()
`
What is the expected output? What do you see instead?
No code analysis error flag.
Paste Traceback/Error Below (if applicable)
N/a
Versions

Spyder version: 5.4.1
Python version: 3.8.16
Qt version: n/a
PyQt version:  n/a
Operating System name/version: Windows 11

Hey @loewenm, thanks for reporting. I can't reproduce this error on my side. I don't get any error related to the ffill_cols variable in the editor.
Could you update to our latest version (5.4.3) and try again? Thanks!

Upgraded to v5.4.3 and checked, unfortunately it's still there.

I have that line of code inside a very basic function.
This is strange...

This is strange...

Agreed. Could you post the entire code of your clean_compustat_data (as plain text) function here? That way I could check it on my side.

My apologies ccordoba12 - I was away after opening this issue.
I've included a cleaned-up basic version of this file to this message that includes the error pyflakes code analysis error.
--- Code ---
def clean_compustat_data(data):

    # Interpolate fields where appropriate
    ffill_cols = [
        'col1',
        'col2',
        ]
    interpolated_values = data.groupby('GVKEY', group_keys=False).apply(lambda group: group[ffill_cols].interpolate(method='linear')).copy()

    del ffill_cols, interpolated_values

    return interpolated_values

#%% Main
if __name__ == '__main__':
    print('Hello world!')

--- End Code ---
Please let me know if you see this error when copy/pasting this into the Spyder IDE. Thank you.

@loewenm, thanks for your answer. I was able to reproduce your problem:

but the problem is not the lambda, is the del statement on line 13. If I comment it, then the errors disappear:

If really need that statement, then you can add a # noqa comment next to the lines with errors so that Spyder doesn't display them:

Let us know if we can help you with something else.

Closing due to lack of response.


Remove impl_Identifiable macro

Please hold off on opening more of these until we've dealt with the others.

https://github.com/diesel-rs/diesel/pull/692#issuecomment-298238188