davinci-resolve: unbreak

Description of changes

Adds python as input to davinci-resolve
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@lourkeur I'm not sure, but I found it since I couldn't run nixpkgs-review on some unrelated changes

I can confirm davinci-resolve breaks my nixpkgs-review run with:
error: undefined variable 'python'

       at /home/runner/.cache/nixpkgs-review/rev-cd8af3ad0ad8cfb2941928b07fc671435288ddc3/nixpkgs/pkgs/applications/video/davinci-resolve/default.nix:157:5:

          156|     xdg-utils # xdg-open needed to open URLs
          157|     python
             |     ^
          158|     # currently they want python 3.6 which is EOL

       … while evaluating the attribute 'passAsFile'

...

Is python an alias of python2 here? :thinking:

I'm pinging @davidak since he merged #152113.

error: Function called without required argument "python" at /home/artturin/.cache/nixpkgs-review/pr-170433/nixpkgs/pkgs/applications/video/davinci-resolve/default.nix:19, did you mean "jython", "python2" or "python3"?

python is an alias to python2 so it can't be used in nixpkgs
https://github.com/NixOS/nixpkgs/pull/170622


Add Keyboard Shortcuts for Common Operations

Create keyboard shortcuts for the following operations:

Create New Inventory Item
Create New Storage Space
Print New Labels

These should be globally accessible, but should not disrupt the user if they are already entering data.

Consider using hooks to accomplish this: https://www.fullstacklabs.co/blog/keyboard-shortcuts-with-react-hooks


Update PossibleIncorrectComparisonWithNull documentation with better example

PR Summary
Fixes comments received in issue #1227 after it was closed
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[ ] NA Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

I applied Rob's suggestion about the whitespace and added Jim's example as well

What seems odd is that swapping the $null in the example, i.e.:
$null -eq @()  # returns $false
$null -ne @()  # returns $true
doesn't make much sense either?
More importantly if a user wanted to check for $null in an array then you have to write this with the $null on the right of the equals test.
I'm not convinced this rule has any value - maybe I'm missing something?


[Feature Request] Frontend for Bedrock Knowledge Base

#428, #139
Frontend Implementation Policy
Implementation Location

Frontend code should be implemented under src/features/bot-kb.

Configuration Options

In the cdk.json settings, the useBedrockKnowledgeBaseForRag: bool option will be used to allow the use of either Knowledge Base (KB) or pgvector, but not both.
By default, useBedrockKnowledgeBaseForRag is set to false.

UI Switching

When useBedrockKnowledgeBaseForRag is true, all knowledge-related UI elements in the bot creation screen will be replaced with those from features/bot-kb.
When useBedrockKnowledgeBaseForRag is false, KB features will not be accessible from the frontend.

Future Direction for v2 Release
Changes in v2

The default value for useBedrockKnowledgeBaseForRag will be changed to true, and the use of pgvector will be deprecated.
After the v2 release, deploying Postgres will be optional, and KB will be the default.
If users wish to maintain Postgres, they will need to explicitly specify the option in cdk.json.

Handling of Existing Bots

After the v2 release, existing bots will be treated as read-only and cannot be updated or edited.
The UI for creating new bots will only utilize KB.

Request example:
{
    "id": "kb-test-bot",
    "title": "kb-test-bot",
    "instruction": "",
    "description": "",
    "embeddingParams": null,  // Ignored on KB
    "generationParams": null,
    "searchParams": null,  // Ignored on KB
    "agent": {
        "tools": [
        ]
    },
    "knowledge": {
                    "sourceUrls": [],  // Ignored on KB
                    "sitemapUrls": [],  // Ignored on KB
                    "filenames": [],
                    "s3Urls": ["s3://example-bucket/1/"] // Newly added. Need to end with slash
                },
    "displayRetrievedChunks": true,  // // Ignored on KB
    "conversationQuickStarters": [],
    "bedrockKnowledgeBase": {  // newly added
        "embeddingsModel": "titan_v1",
        "openSearch": {
            "analyzer": {
                "characterFilters": ["icu_normalizer"],
                "tokenizer": "kuromoji_tokenizer",
                "tokenFilters": [
                    "kuromoji_baseform",
                    "ja_stop"
                ]
            }
        },
        "searchParams": {
            "maxResults": 20,
            "searchType": "hybrid"
        },
        "chunkingStrategy": "fixed_size",
        "maxTokens": 500,
        "overlapPercentage": 10,
    }
}

Is there any way to migrate our data from the postgres to the KB once V2 releases?


In metadata viewer, replace binary chars with (Binary data) instead of "?"

wont fix for now. fine as is


verify client certificate callback with client verify hook

verify_cert plugin is included in the example to show the usage of the client verify hook

Need to fix make check to reflect hook state addition.  Otherwise, looks good.

yah forgot to add the client verify hook in InkAPITest

[approve ci autest]

This might have build issues:
../../../iocore/net/SSLUtils.cc: In function 'SSL_CTX* SSLInitServerContext(const SSLConfigParams*, const ssl_user_config*, std::vector<x509_st*>&)':
../../../iocore/net/SSLUtils.cc:1736:51: error: 'ssl_verify_client_callback' was not declared in this scope
     SSL_CTX_set_verify(ctx, server_verify_client, ssl_verify_client_callback);

https://github.com/apache/trafficserver/pull/3229 -- this might fix the problem


Onboarding skills repo

Description
Onboarding skills repo
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/4378
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We also need to wait for a bit before merging, as skills repo has wrong rootname, wrong m2 folder, and others.
Zan is working to fix those.

Hey @zane-neo can you please take care of this issue https://github.com/opensearch-project/skills/issues/188#issue-2121422972 ?
Thank you


Some fixup and QoL changes (week3.py)

An argument description was wrong (it is used to decide which function to run, not output path)
Make -i and -o required arguments (OpenCV will raise an exception when being fed NoneType as an image path anyway)
Make DELAY a global variable (Since it's very likely to be reused)

Thanks for the comment. I have updated with a new description for -q option (from a copy-paste mistake) . The other comments are up to the student, for example: whether they should handle the wrong arguments or not.


Deflect pip queries when querying .so location

Description
As the title says, instead of querying pip to determine .so locations, this will try first try to look-up to directories from the init.py file and if the .so are found skip querying pip.
The motivation is to avoid querying pip if possible since it's slow (at least on a few machines I've tried).
Fixes # (issue)
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Changes
Please list the changes introduced in this PR:

Change A
Change B

Checklist:

[ ] I have read and followed the contributing guidelines
[ ] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@denera thanks for the review & the comment, I was not aware of PR760, as a result I think we can close this, assuming that 760 will be merged soon.
Before we close this, I have one question, if a user overwrites the used transformer_engine via PYTHONPATH will there be any issue with locating the correct .so? One case I had in mind when making this PR is that I can change the transformer_engine used (the python code via PYTHONPATH) but the pip would still use the .so from the globally installed TE, which would be incorrect. I know it's a corner case and perhaps not the most common case, but want to avoid someone spending time debugging something like this in the future.
Thanks again for the review!

With https://github.com/NVIDIA/TransformerEngine/pull/760, loading the PyTorch C++ extensions (e.g. transformer_engine_extensions.cpython-310-x86_64-linux-gnu.so ) will also load the libtransformer_engine.so it was built with. If you set PYTHONPATH to a local TE build with transformer_engine_extensions, it will also load a local libtransformer_engine.so. If you set PYTHONPATH without building (e.g. if you only want to change the Python code without recompiling C++), it will load the globally-installed transformer_engine_extensions, which will load the globally-installed libtransformer_engine.so.

Thank you both for your responses, I think the issue I'm experiencing will be resolved in #760 and as I result I'm closing this one.


TS Server fatal error:  Debug Failure. False expression.

Issue Type: Bug
TypeScript Version: 4.6.3
Steps to reproduce crash
Unsure, the errors appear randomly (but frequently, ~1-2 time a day).
TS Server Log
tsserver.log
TS Server Error Stack
Server: semantic
Error: Debug Failure. False expression.
    at _loop_2 (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:169026:34)
    at ProjectService.assignOrphanScriptInfoToInferredProject (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:169034:25)
    at ProjectService.assignProjectToOpenedScriptInfo (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:170700:26)
    at /usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:170974:97
    at Object.flatMap (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:508:25)
    at ProjectService.applyChangesInOpenFiles (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:170974:41)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:172133:46)
    at /usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:174069:88
    at IpcIOSession.Session.executeWithRequestId (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:174060:28)
    at IpcIOSession.Session.executeCommand (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:174069:33)
    at IpcIOSession.Session.onMessage (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:174095:35)
    at process.<anonymous> (/usr/share/code/resources/app/extensions/node_modules/typescript/lib/tsserver.js:176730:31)
    at process.emit (node:events:390:28)
    at emit (node:internal/child_process:917:12)
    at processTicksAndRejections (node:internal/process/task_queues:84:21)

VS Code version: Code 1.66.0 (e18005f0f1b33c29e81d732535d8c0e47cafb0b5, 2022-03-30T05:50:41.156Z)
OS version: Linux x64 5.16.18-200.fc35.x86_64
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2900)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
31.20GB (0.87GB free)

Process Argv
--unity-launch --crash-reporter-id 4b8160e1-016a-447a-8c44-7f981c156a47

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome-xorg

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome-xorg

XDG_SESSION_TYPE
x11

Extensions (51)

Extension
Author (truncated)
Version

better-comments
aar
3.0.0

ng-template
Ang
13.3.0

swagger-viewer
Arj
3.1.2

asciidoctor-vscode
asc
2.9.8

markdown-mermaid
bie
1.13.2

mermaid-markdown-syntax-highlighting
bpr
1.2.3

regex
chr
0.4.0

indent4to2
Com
0.1.2

vscode-eslint
dba
2.2.2

githistory
don
0.6.19

gitlens
eam
12.0.6

vscode-html-css
ecm
1.11.0

prettier-vscode
esb
9.5.0

vscode-jest-runner
fir
0.4.48

shell-format
fox
7.2.2

markmap-vscode
ger
0.0.12

vscode-pull-request-github
Git
0.40.0

go
gol
0.32.0

vscode-env
Iro
0.1.0

vscode-peacock
joh
4.0.1

vscode-styled-components
jpo
1.6.6

template-string-converter
meg
0.6.0

git-graph
mhu
1.30.0

vscode-filesize
mkx
3.1.0

mongodb-vscode
mon
0.9.2

vscode-docker
ms-
1.21.0

remote-containers
ms-
0.231.5

remote-ssh
ms-
0.78.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.0

vscode-remote-extensionpack
ms-
0.21.0

vscode-typescript-tslint-plugin
ms-
1.3.4

vsliveshare
ms-
1.0.5449

angular-console
nrw
17.15.0

tsimporter
pmn
2.0.1

vscode-commons
red
0.0.6

vscode-xml
red
0.20.0

vscode-yaml
red
1.6.0

LiveServer
rit
5.7.5

vscode-coverage-gutters
rya
2.9.1

vscode-scss-formatter
sib
2.4.2

rewrap
stk
1.16.3

code-spell-checker
str
2.1.11

code-spell-checker-german
str
2.0.3

open-in-browser
tec
2.0.0

vscodeintellicode
Vis
1.2.18

vscode-nginx
wil
0.7.2

markdown-pdf
yza
1.4.4

markdown-all-in-one
yzh
3.4.0

html-css-class-completion
Zig
1.20.0

es6-string-javascript
zjc
1.0.1

I don't see anything unusual in the log that would be enough to have us figure out what went wrong here, unfortunately. Are you able to share a repo + steps that can reproduce the problem?

Sadly no to both. I'm working on a proprietary project for an external customer.
Our setup is a rather large Nx workspace though - which means a lot of different tsconfigs and paths-aliases. The error occurs seemingly randomly, but most prominently when I'm switching branches and/or having merge conflicts.
I also perceive my development experience to be "rather slow" (IDE/VS Code sometimes takes maybe a second to show hints/autocompletions, compile errors stay even though the code has changed, etc.). Maybe that's related too?
Regardless, I'm sorry that I can't help much. If there is anything else I do please let me know. However, I understand if you can't do much but close the issue.


Put http://mochajs.org/ behind cloudflare

Should improve caching, adds TLS and should improve search engine ranking (slightly).
What do you guys think?

Current Nameservers
Change Nameservers to:

ns1.dreamhost.com
chad.ns.cloudflare.com

ns2.dreamhost.com
lorna.ns.cloudflare.com

ns3.dreamhost.com
Remove this nameserver

@jbnicolai sounds fine to me.  how do we do that?

@boneskull I've already created a cloudflare account for mochajs.org. All that would need to happen is for the DNS nameservers to be updated to the ones mentioned above, and I'll need to figure out if I can somehow invite you and @travisjeffery to that account so we share ownership.
Of course, if you ever want to revert this you simply point back to the original nameservers and cloudflare is out of the loop.

@jbnicolai okay, see about those accounts.  I'll update the DNS records

@jbnicolai records updated.  let me know if it doesn't propagate properly

@boneskull seems like I should've done my homework before urging you to do this. Shared admin access to a website on cloudflare is a paid-only feature.
A way around this is to simply have a dedicated account for mochajs.org, the credentials for which we share. However, this would involve transfering the account, see https://support.cloudflare.com/hc/en-us/articles/200167856-How-do-I-transfer-or-move-a-domain-to-another-account-
tr;dr: if you'd revert the nameserver changes, I'll delete this cloudflare account and set up one separate from my personal account for mochajs.

No worries btw, nothing will break ;)

@jbnicolai reverted for now

Ok, transfered.
The two new nameservers are adi.ns.cloudflare.com and west.ns.cloudflare.com and you should've received the credentials via PM.

done


How to stop creating Thumbnails

Hi, I don't want to create thumbnail to my images, how to stop that?

In this section of config/lfm.php, set should_create_thumbnails  to false.


[CLOUD-2367] Rename existing templates using passthrough TLS termination mode

Renamed:    sso72-mysql-persistent.json -> sso72-passthrough-mysql-persistent.json
Renamed:    sso72-mysql.json -> sso72-passthrough-mysql.json
Renamed:    sso72-postgresql-persistent.json -> sso72-passthrough-postgresql-persistent.json
Renamed:    sso72-postgresql.json -> sso72-passthrough-postgresql.json
Renamed:    sso72-https.json -> sso72-passthrough.json

Also:

Replace 'SSO' with 'RH-SSO' where appropriate,
Add 'displayName' field to MEMORY_LIMIT variable

Signed-off-by: Jan Lieskovsky<EMAIL_ADDRESS>Note: This is a replacement for original CLOUD-2367 one. Instead of removing the HTTP_HOSTNAME / HTTPS_HOSTNAME variables, the agreement was to:

Keep the HTTP_HOSTNAME / HTTPS_HOSTNAME` variables,
Rename the templates using passthrough TLS termination mode for routes

instead of the original approach.

PR updated. @rcernich @maschmid PTAL (once got a chance).


Velero external endpoint not considered

When enabling the DR module with an external endpoint configured, furyctl also installs a MinIO instance inside the kube-system namespace and still uses the default endpoints and credentials.
Snippet
furyctl.yaml:
...
      dr:
        type: on-premises
        velero:
          externalEndpoint:
            accessKeyId: REDACTED
            bucketName: velero
            endpoint: other.minio.instance:9000
            insecure: true
            secretAccessKey: REDACTED
...

Which version are you using?

furyctl v0.28.0 and distribution v1.28.0

you should set .velero.backend = "externalEndpoint" because the default value is "minio" if it's not set:
      dr:
        type: on-premises
        velero:
          backend: externalEndpoint
          externalEndpoint:
            accessKeyId: REDACTED
            bucketName: velero
            endpoint: other.minio.instance:9000
            insecure: true
            secretAccessKey: REDACTED

to improve the UX maybe we should add a validation to that field

Thank you @Al-Pragliola, I missed that field.
What to do in case of installing velero in a EKS cluster? Should we set type: on-premises if we only want to use the external endpoint?

The schema for an EKSCluster is different, here's an example:
      dr:
        type: eks
        velero:
          eks:
            bucketName: example-velero
            region: eu-west-1

On EKS, velero do it's backup to an s3 bucket that furyctl manages, there is no externalEndpoint feature on that provider


Req: More detail or example for usage

Hi OP,
This would be great if you put an example on how to use or more detail about usage.
Thank you from a guy new to L5.

Yes, I haven't updated this wiki. Maybe I would create some video screencast or just update this wiki.

It is a good package, just read the code you will know how to use it.
maybe the doc needs more infomation.

I am working on the documentation :)

need some help?

If you want to help me, you can try creating or updating the wiki of this
repo. help is welcome and greatly appreciated.
On Sel, 9 Jun 2015 at 12.59 adangbryen<EMAIL_ADDRESS>wrote:

need some help?
—
Reply to this email directly or view it on GitHub
https://github.com/pingpong-labs/admin/issues/30#issuecomment-110234217.

I have create a page for creating admin Role and User here:
Admin Role and User

@syahzul Thank you :)


How to serialize dns solver requests

I am using lego through traefik. The provider I am using is DuckDns.
There is a imitation with duckdns that it only has one txt record per account, that txt record will be presented for any lookup request on any subdomain.
So test1.test.duckdns.org and test2.test.duckdns.org do not have different txt records. Currently I want to be able to request in traefik
 [[acme.domains]]
   main = "test1.test.duckdns.org"
 [[acme.domains]]
   main  "test2.test.duckdns.org"

I was wondering how I could serialize the requests such that the two requests do not overlap and that test1 is validated and then test2 is validated (currently test2 will succeed and then eventually test1 will get retried). I am okay with all requests going through DuckDNS are serialized, but if possible it would be nice to limit it only to one per account. I am curious to this interaction across the operation that spans both Present and CleanUp.
@ldez - due to already paging in context on https://github.com/xenolf/lego/pull/676
Thanks!

I saw the problem with DuckDNS tomorrow, and I may have a solution.
For the moment, I have to think about it.

Closed by #700


Trigger Golang 1.18 Prod Release

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Add additional tests and methods in ResidentDescriptorBuilder class

Additional tests are used to validate the functionalities of fields in the filter feature that were not isolated and tested.
The methods added are used to overload current methods in ResidentDescriptorBuilder to make the code cleaner and easier for future use of the class.

Codecov Report

Merging #90 (453bdf6) into master (c338e90) will increase coverage by 0.19%.
The diff coverage is n/a.

:exclamation: Current head 453bdf6 differs from pull request most recent head ecc4e10. Consider uploading reports for the commit ecc4e10 to get more accurate results

@@             Coverage Diff              @@
##             master      #90      +/-   ##
============================================
+ Coverage     72.68%   72.88%   +0.19%
  Complexity      522      522
============================================
  Files            82       82
  Lines          1501     1501
  Branches        159      159
============================================
+ Hits           1091     1094       +3
  Misses          361      361
+ Partials         49       46       -3

Impacted Files
Coverage Δ

...t/predicates/AttributesMatchKeywordsPredicate.java
76.92% <0.00%> (+23.07%)
:arrow_up:

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


Botany.

WIP issue - This accidentaly posted because I hit control enter, and it'll stick around even if I close it, so just treat this as a WIP issue.
This is the first department-wide issue open on this discord. Because we don't have that much solid thought put into the concept of departments, role, access, fun, player-player interaction, etc yet, we might go through some of these as we develop botany and split them off into the proper issue.
Themes.
I think the most important facet of botany to start with is the themes. SS14 strives to be a game that creates a powerful interactive narrative, and that comes easiest when people can get truly immersed into their character and surroundings. We should strive to allow for a strong choice of themes to survive in the final iteration of botany, only sacrificing the least evocative if need be for fun and balancing.
Rustic agriculture.
Waking up in an old wood & straw house as weak, early sunlight pours in through the window. Straw hats, loose shirts and pants. A scythe mowing down breeze-tickled fields of grasses and grains for the harvest. Black soil, dirt, and disarray. Weeds, pests, and vines overpowering everything with time, the farmer clearing them out of only the most important areas. Labouring into the sunset.
Modern industrial agriculture.
Industrial warehouses, diesel generators, acerbic lighting. Teams of polo-shirted and capped workers. Barrels of oil, vats of pesticide sprays, bales of hay, conveyor belts and tanks of the harvest. Tractors, combine harvesters, machines seemingly designed to catch you and grind you into meat. Water lines automatically spray fine mist into orderly lines of monoculture at dawn and dusk.
Potted gardens.
Watering the plants week by week. Moving the small ones into bigger pots as they grow. Getting soil and seed packets from the store.
Hydroponics.
A musty basement. Strong lights from above. Plastic wrap and several rows of connected thin, flimsy trays, each with a dozen plants suspended in it. Pouring a little bit out of the nutrient bottle into the water reservoirs. A specialized computer tracks the time and water level, adjusting the latter as need be.
Genetic engineering and Lysenkoism.
In the west, sterile laboratories. Precise instruments, humming machines, computer displays. A slow, steady, darwinian and epigenetic march towards improvements in yields, resistances, and special features.
In the east, Lysenko continues in Lamarck's footsteps: heritable acquired characteristics in plants. "Nonsense" that actually seems to kind of work: planting into patterns and clusters, treating seeds to strange conditions, experimental fertilizers. Rewards seen at harvest.
And so on.
I could continue with more types and styles of agriculture, but in short, we can see that SS13 has pretty much only gone for the "hydroponics" theme, and barely even at that. Is it a surprise that most botanist RP ends up being dude drugs lmao when it doesn't actually feel very much like farming?
Hence, it's a good idea to allow Botany to be somewhat reminiscent of real farming techniques and experiences, even when reality does somewhat dictate that space agriculture would likely be hydroponics. So what should we try to put into SS14's botany?
Theme takeaways.

Large-scale agriculture. SS13 currently scales everything around being responsive and appropriately sized for supplying the chef: this comes at the expense of botanists not feeling very much like an agricultural operation. The scale of botany should increase compared to SS13: more space, more rooms, larger growing operations, more time until fruition. This will, however, mean that botany needs a new primary consumer of varied growns, with the chef being secondary; the only candidate I can think of is to export to NanoTrasen/other stations/whatever through cargo.
Seasonal agriculture. There are effectively no seasons in SS13. To be fair, it's hard to shoehorn seasons into a space station's agriculture department, especially when IRL hydroponics tends to bypass them. Seasons primarily consist of temperature, lighting, rain, and mood; we should look for a way to at least implement some of these into botany.
Industrial machinery. Everything is one tile big in SS13's botany. We can do better; larger vats and larger machines. Inputs to the current botany system are cyclical; everything required to grow plants, other than water, can be biogenerated out of those plants themselves, which isn't very thematic. The tools themselves are miniature buckets and spray bottles. To upscale and solve both problems at once, nutrients, sprays, and other inputs can be upscaled into nutrient slurry tanks and pipes, large sprayers, potentially more autonomous machines like farmbots.
TBD

Constraints and minor features.

Botany, thematically, could be composed of anything between individual plant organisms like trees to hundreds-of-organisms-in-the-same-place like grasses, to very-basic-cellular-structures like molds and algaes.
Botany, thematically, is going to contain anything from very small plants like mushrooms, to very tall plants like full trees.
Botany, thematically, can contain more than plants: livestock and bees are common in ss13, but we could also add maggot and pest cultivation, fish breeding, etc.
Botany, thematically, can also not take place in botany: ghetto botany, random habitable spots on the station, other stations, and planets could all contain plants, so it's important that whatever gameplay is created around botany doesn't essentially limit plants to being grown only in specific places and under specific conditions.

Closing to merge into hackmd docs


matplotlib markerstyle is incomplete

Using something like ax.scatter(s['date'], s['values'], marker = 'o', color = 'red') gets a complaint
Argument of type "Literal['o']" cannot be assigned to parameter "marker" of type "MarkerStyle | None" in function "scatter"
  Type "Literal['o']" cannot be assigned to type "MarkerStyle | None"
    "Literal['o']" is incompatible with "MarkerStyle"
    Type cannot be assigned to type "None"

It looks like some work should be done to update MarkerStyle to allow the various literal values documented by matplotlib:
https://matplotlib.org/stable/api/markers_api.html#module-matplotlib.markers
(Note - unlike the work I did with pandas, I won't be contributing to matplotlib type fixes as I rarely use it!)

Closing as we pretty much redid the matplotlib stubs from scratch in August.


Profile Picture Not Showing

Hello,
First of all, thank you so much for this code.
Im facing a little problem here. Profile picture not showing after i save and refresh the page. It's not showing either on the profile section or on my page section. Whe i refresh the page after saving my profile picture, it just showing my username. I have asked my friend to install this code on their computer and we all facing the same problem. This problem happens on localhost and on my website. Maybe if you want to check it directly you can go to my web sudutbkt.com, but i also attached some screenshot.

Thank you so much, hope you can help me.

You need to link your storage to public folder.
php artisan storage:link


Update Makefile.PL

Hi Daisuke Maki,
i'm new to Pull request (parecipating to PullRequest Challenge).
Trying to install the module on strawberry for Windows, there was error on compilation:
C:\STRAWB~1\c\lib\libmpfr.a(clear.o):(.text+0x1e): undefined reference to __gmp_get_memory_functions' C:\STRAWB~1\c\lib\libmpfr.a(mul.o):(.text+0x4e4): undefined reference to __gmpn_mul'
C:\STRAWB~1\c\lib\libmpfr.a(mul.o):(.text+0x517): undefined reference to __gmpn_lshift' C:\STRAWB~1\c\lib\libmpfr.a(mul.o):(.text+0x7e7): undefined reference to __gmpn_lshift'
Looking at mpfr.org i've found this faq:
[from http://www.mpfr.org/faq.html#undef_ref1]
5. When I link my program with MPFR, I get undefined reference to __gmpXXXX.
Link your program with GMP. Assuming that your program is foo.c, you should link it using:
	cc link.c -lmpfr -lgmp
MPFR library reference (-lmpfr) should be before GMP's one (-lgmp). Another solution is, with GNU ld, to give all the libraries inside a group:
	gcc link.c -Wl,--start-group libgmp.a libmpfr.a -Wl,--end-group
See INSTALL file and ld manual for more details.

I've touched Makefile.pl and added gmp library.
Seems that the Makefile.pl is different in gitHub and in Cpan.
Have a nice day and CIAO
Ulderico
PS: excuse for my english

@nric62 Hi. Tests on Travis CI failed so I looked into it, and it turns out I must've stopped short while trying to set things up a long time ago. I reinstated the tests, and I probably added a fix to your root cause.
Can you please check out current master branch and see if it works for you?

Hi lestrrat,
I've tried to install your new distribution, but i've found some matters:

on windows distribution tar is not present
you the lib mpfr ang gmp are present on strawberry distribution (so
is not necessary to download)
after some errors, i've done:

perl makefile.pl
dmake
dmake test

the test signal me the following errors:#   Failed test 'Got
1903-06-22T23:57:14, expected 1903-06-23'

at xt/101_solar_term.t line 23.
got: '1903-06-22'
expected: '1903-06-23'
xt/101_solar_term.t ........... 367/?
Failed test 'Got 1915-05-06T23:55:41, expected 1915-05-07'
at xt/101_solar_term.t line 27.
got: '1915-05-06'
expected: '1915-05-07'
xt/101_solar_term.t ........... 421/?
Failed test 'Got 1917-09-23T23:52:53, expected 1917-09-24'
at xt/101_solar_term.t line 23.
got: '1917-09-23'
expected: '1917-09-24'
xt/101_solar_term.t ........... 448/?
Failed test 'Got 1919-03-06T23:58:26, expected 1919-03-07'
at xt/101_solar_term.t line 27.
got: '1919-03-06'
expected: '1919-03-07'
xt/101_solar_term.t ........... 776/?
Failed test 'Got 1932-10-23T23:56:42, expected 1932-10-24'
at xt/101_solar_term.t line 23.
got: '1932-10-23'
expected: '1932-10-24'
xt/101_solar_term.t ........... 1159/?
Failed test 'Got 1948-05-06T00:45:05, expected 1948-05-05'
at xt/101_solar_term.t line 27.
got: '1948-05-06'
expected: '1948-05-05'
xt/101_solar_term.t ........... 1186/?
Failed test 'Got 1950-01-20T23:52:57, expected 1950-01-21'
at xt/101_solar_term.t line 23.
got: '1950-01-20'
expected: '1950-01-21'
xt/101_solar_term.t ........... 1267/?
Failed test 'Got 1953-01-05T23:55:14, expected 1953-01-06'
at xt/101_solar_term.t line 27.
got: '1953-01-05'
expected: '1953-01-06'
xt/101_solar_term.t ........... 1539/?
Failed test 'Got 1964-09-07T23:52:46, expected 1964-09-08'
at xt/101_solar_term.t line 27.
got: '1964-09-07'
expected: '1964-09-08'
xt/101_solar_term.t ........... 1566/?
Failed test 'Got 1965-11-07T23:59:25, expected 1965-11-08'
at xt/101_solar_term.t line 27.
got: '1965-11-07'
expected: '1965-11-08'
xt/101_solar_term.t ........... 1975/?
Failed test 'Got 1982-10-08T23:55:10, expected 1982-10-09'
at xt/101_solar_term.t line 27.
got: '1982-10-08'
expected: '1982-10-09'
Failed test 'Got 1983-05-21T23:59:34, expected 1983-05-22'
at xt/101_solar_term.t line 23.
got: '1983-05-21'
expected: '1983-05-22'
xt/101_solar_term.t ........... 2699/?
Failed test 'Got 2013-04-04T23:54:37, expected 2013-04-05'
at xt/101_solar_term.t line 27.
got: '2013-04-04'
expected: '2013-04-05'
xt/101_solar_term.t ........... 2891/?
Failed test 'Got 2021-02-03T23:50:29, expected 2021-02-04'
at xt/101_solar_term.t line 27.
got: '2021-02-03'
expected: '2021-02-04'
xt/101_solar_term.t ........... 2946/?
Failed test 'Got 2023-01-05T23:56:11, expected 2023-01-06'
at xt/101_solar_term.t line 27.
got: '2023-01-05'
expected: '2023-01-06'
xt/101_solar_term.t ........... 3000/?
Failed test 'Got 2025-12-21T23:54:31, expected 2025-12-22'
at xt/101_solar_term.t line 23.
got: '2025-12-21'
expected: '2025-12-22'
xt/101_solar_term.t ........... 3111/?
Failed test 'Got 2030-02-18T23:51:18, expected 2030-02-19'
at xt/101_solar_term.t line 23.
got: '2030-02-18'
expected: '2030-02-19'
xt/101_solar_term.t ........... 3138/? # Looks like you failed 17 tests
of 3144.
xt/101_solar_term.t ........... Dubious, test returned 17 (wstat 4352,
0x1100)
Failed 17/3144 subtests
xt/999_leak.t ................. skipped: Test requires module
'Test::Valgrind' b
ut it's not found
xt/999_pod-coverage.t ......... skipped: Test requires module
'Test::Pod::Covera
ge' but it's not found
xt/999_pod.t .................. skipped: Test requires module
'Test::Pod' but it
's not found
I hope my test are interesting for you ,  i'm using
Summary of my perl5 (revision 5 version 20 subversion 1) configuration:
osname=MSWin32, osvers=6.3, archname=MSWin32-x64-multi-thread
uname='Win32 strawberry-perl <IP_ADDRESS> #1 Mon Sep 15 13:26:45 2014 x64'
Have a nice day
Ulderico
2015-03-16 1:42 GMT+01:00 lestrrat<EMAIL_ADDRESS>
@nric62 https://github.com/nric62 Hi. Tests on Travis CI failed so I
looked into it, and it turns out I must've stopped short while trying to
set things up a long time ago. I reinstated the tests, and I probably added
a fix to your root cause.
Can you please check out current master branch and see if it works for you?
—
Reply to this email directly or view it on GitHub
https://github.com/lestrrat/DateTime-Astro/pull/1#issuecomment-81313640.

@nric62
What... tars???? I have no idea what you are talking about.
My question is "does the ORIGINAL problem still exist?". Remember, this is a Pull Request, and therefore we talk about a change that you would like to propose

I'm assuming this is okay. Closing.


Config system proposal

I wrote a config files system for Unity which I actively update. We use it with every our project for debugging and testing along with other systems, such as a console, and it is really useful.
General use case is the following: developer puts as much variables as possible into the config file - such as a speed of something, a chance of generating something, sizes, colors, quantities, start values like money or health, graphic tweaks and so on. Then others can test the build changing these values. One base config file is packed into game resources, and others are exposed into the external folder (near the executable). One can create as many configs as needed (with different sets of settings) and engine finds and imports them all, rewriting base values. Then you can get these properties as simple as float h = CFG.F("Health_start");.
I have already rewritten this for Godot mono, and can also adapt for gdscript. Also I've made a VSCode extension to use with this system.
When I discovered Godot, I found that many of my other plugins, functions of which are not present or are not very good natively in Unity (for example, Languages, Controls, and of course Tween are already inside Godot and perfectly executed! But, I haven't seen a config system, which I find really useful.
Maybe, it is not a feature proposal, and I just have to publish something like a plugin into asset database. If so, tell me please what to do, I'm new to this coimmunity. Also, tell me please what do you think of console system and about the fact that my languages plugin can automatically download translations from Google Sheets. Should I contribute it to the Godot community or it is boring and unneeded?
Thank you for the reading.

Isn't it what ProjectSettings is, along with project.godot? In addition to the existing variables in it, you can add your own and get it anytime with ProjectSettings.get_setting(key).
Otherwise, I don't see why this would have to be also in the core, sounds like a system that would be perfectly fine as a plugin.

Isn't it what ProjectSettings is, along with project.godot? In addition to the existing variables in it, you can add your own and get it anytime with ProjectSettings.get_setting(key).

Not quite, as I can create as many files as I want, to test different sets of configs - one is packed liek project settings, other is just for testing of gameplay, one will be exposed for players like a regular *.ini file with settings that they can change if they want... Also, there is a various functionality inside, like automatic parsing of enums.

Otherwise, I don't see why this would have to be also in the core, sounds like a system that would be perfectly fine as a plugin.

I don't say it should be in a core, please point me in a right direction. It should be some editor plugin or what? In unity it is "asset" which I can import and place a manager object on scene.

You can surely script this, and create a singleton, for example: https://docs.godotengine.org/en/3.0/getting_started/step_by_step/singletons_autoload.html
If editor integration is needed (i.e need special editor GUI, custom importer etc), you can create an EditorPlugin: https://docs.godotengine.org/en/3.0/tutorials/plugins/editor/index.html
Have a look here for plugins, libraries and assets other people have made: https://godotengine.org/asset-library/asset

The ConfigFile class can likely be used to achieve what you want. It's also possible to centralize default values using a singleton or project settings, see godot-settings-manager for an example.


Integrate Spotless in the pom.xml

Spotless is a popular tool to check that the code is correctly formatted and, if needed, can automatically format code according to a standard.
It has also a Maven plugin: https://github.com/diffplug/spotless/tree/main/plugin-maven
The goal of this issue is simply to integrate it into the pom.xml
Note: This will make Spotless only available in the project. I recommend that the maintainers of Trevas-Jupyter run the automated formatting and then commit all changed files (as probably many files will change the first time the tool is run):
./mvnw spotless:apply

Then, I leave it also to the maintainers to integrate it in Github actions. Eg. when using this command Github actions will fail if a developer committed files that do not correspond to the standard format:
./mvnw spotless:check

One could also add some documentation on how to add it as a Git pre-commit hook: https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks, to ensure that wrongly formated code can be committed in the first place.
Unfortunately, this has to be done by the developer manually.

Fix 85eccea13b3bce04ab13bd339281ca4859a8af4a


Seeing "Unhealthy registry pod" critical condition on a non-registry pod

Describe the bug
This critical condition is getting raised on a MigMigration.
Migration Registry Pod nginx-pv-30/stage-nginx-deployment-557dd97bf8-w69hw-s6vx6 is in unhealthy state on cluster host

The strange thing about this is... The above is not a registry Pod.
To Reproduce
Steps to reproduce the behavior:

Run a stage migration on nginx-pv with mig-controller latest

Expected behavior
Migration runs to completion without any pauses
Additional context
Expect this is a bug in recent registry health check work done in #681 and #717

Err is coming from here https://github.com/konveyor/mig-controller/blob/0dcd0b4faa24762f3219170c8cf31332bf77eba8/pkg/controller/migmigration/validation.go#L293-L296

The problem is that getRegistryPods doesn't filter pods correctly. It just selects every pod in any namespace with a migplan label.
https://github.com/konveyor/mig-controller/blob/0dcd0b4faa24762f3219170c8cf31332bf77eba8/pkg/controller/migmigration/validation.go#L312-L326


Make $TMPDIR; fix quotes on command

Dropping since the quote fix was not accepted. Will resubmit with the $TMPDIR fix (still an issue).


Drain support

Hi,
While draining a node, some pods will never stop (example kube-proxy, calico etc...). The problem is when you reboot a node when he has been drained but has kept some pods, I receive a lot of alerts (1 per pod saying not ready). It would be great to filter only on non drained nodes to avoid a such behavhior.
What do you think ?
Thanks

That would be a great enhancements.
Sorry for the slow reply, I've been wishing I have time to implement this but that doesn't seem to be possible. Feel free to submit patch in the mean time.


Pack creep sprite sheets tighter

To reduce size on disk and VRAM usage.
Current packing method makes a grid of sprites with the size of the biggest sprite. I originally made it this way because I assumed that sprite centers had to be the same for sprites in the same animation.
It should be okay to pack tighter if we save the offset between original sprite center and packed sprite center, for each individual sprite. Then load those offsets during runtime and apply offsets in a function connected to AnimatedSprite2D's frame_changed() signal.
Since tigther packing will not be grid-based, scripts will also need to generate/consume information about where each spritesheet is located in the spritesheet.
Scripts/PackSpriteSheet.gd generates packed spritesheets and metadata.
Scenes/CreepSprite.gd consumes packed spritesheets and metadata.

Moved to Backlog https://github.com/Praytic/youtd2/discussions/407


CLOUDSTACK-9550: Use context to filter items in a metrics view

Use available context to filter a metrics view based on zone, cluster, host
in the context object. This fixes metrics view filtering when metrics view is
viewed via Zone->Compute and Storage-> for a resource.
/cc @jburwell @karuturi -- this is a pure UI fix, one manual test LGTM would be required.

@borisstoyanov can you help with manual testing of the fix with some screenshots, thanks.

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-64

I've navigated through Zone -> Compute and Storage -> to clusters, hosts and pr storages -> Metrics View. All the resources are there and they seem to be showing the same as from infrastructure. LGTM 👍

This should be good to merge, purely UI change? /cc @murali-reddy @abhinandanprateek @karuturi

@rhtyd LGTM on code review

Merging this since it's a pure UI change, validated with manual test LGTMs and code review LGTMs.


Associate DWN endpoints with newly created ION DIDs

ProfileManager is currently responsible for creating new profiles. When a new profile is created with the ion did method, associate DWN endpoints with it.
We can take the same logic that web5-js is doing within the connect function here: https://github.com/TBD54566975/web5-js/blob/main/packages/web5/src/web5.ts#L120-L122

DWN endpoints are currently behind a tech preview, which isn't exported by the @web5/api package. Opened a PR to export that here:
https://github.com/TBD54566975/web5-js/pull/227
Once that's available, we can use those endpoints in IdentityAgentManager when we create ION DIDs.


Menu is buggy when scrolling on mobile devices

Describe the bug
When on a mobile device the menu is opened and then the user scrolls down the page, the menu box sometimes scrolls along, then jumps back up and is generally very buggy.
To Reproduce
Steps to reproduce the behavior:

Go to home or result page on a mobile device. (This bug is non-reproducible on an emulated mobile device.)
Click on the hamburger button in the top right corner.
Scroll down the page.
See error

Expected behavior
When the menu is open, if the user taps anywhere outside of the menu box, the menu should collapse automatically.
Smartphone (please complete the following information):

Device: iPhone XR
OS: iOS 14.4.2
Browser: Chrome

@KaiDrieHda please checkout if the bug is still exists in #376

I can't reproduce the bug itself but as described in "Expected behavior" the menu should be closed automatically as soon as the user touchs at something outside of the area. It works if the user "clicks" on something but it doese not work if he only wants to scroll. Close the menu when the user starts scrolling.


Mobile navigation needs trapping

When it comes to keyboard access, the mobile menu of the main header should probably include trapping (like the Dialog component). Meaning, when it's toggled open, tabbing through will recycle back to the MENU button instead of tabbing to the main content which lays underneath, and no longer accessible.
https://codyhouse.co/ds/components/app/main-header

Hi there! I'm not 100% sure trapping would be the best approach in this case. You can go back to the navigation by clicking shift+tab (or swift left on a phone). However, it is true blind users don't know they're "leaving" the navigation. Plus the next focusable item could be under the navigation popup, causing issues to users navigating the content with their keyboard. We'll collect more data and get back to this. Thanks for the feedback!
I'm closing this issue because it's not strictly related to the framework, but to a component.


[Grid][RSC] Nested Grid doesn't work when the parent component is a server component

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://codesandbox.io/p/sandbox/https-github-com-mui-material-ui-issues-37932-8gdqjz?file=%2Fapp%2FGrid.tsx%3A1%2C1
Steps:

Create a Next.js app router app
In page.tsx, replace the content with nest grid using Grid from @mui/joy or Unstable_Grid2 from @mui/material.
Toggle the "use client" declarative in page.tsx

Current behavior 😯
Nested grid is not functional at all when the parent component is a server component.
Expected behavior 🤔
Nested grid should be functional no matter whether the parent component is a server or client component.
Context 🔦
Observation
The nested grid behavior seems to rely on the muiName of the Grid component to identify whether the children are Grid and apply margins accordingly. Yet, muiName seems to be lost when the parent component is a server component.
Perhaps, it can be changed for identifying the class name? e.g. gridClasses.container?
Your environment 🌎

npx @mui/envinfo
  System:
    OS: Windows 10 10.0.22621
  Binaries:
    Node: 18.16.1 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 9.5.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.22621.1928.0), Chromium (114.0.1823.58)
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.7
    @mui/core-downloads-tracker:  5.14.0
    @mui/joy: ^5.0.0-alpha.87 => 5.0.0-alpha.87
    @mui/private-theming:  5.13.7
    @mui/styled-engine:  5.13.2
    @mui/system:  5.14.0
    @mui/types:  7.2.4
    @mui/utils:  5.13.7
    @types/react: 18.2.14 => 18.2.14
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    typescript: 5.1.6 => 5.1.6

It reproduces by copying and pasting this demo:
Without "use client" on the page:

The nested grid behavior seems to rely on the muiName of the Grid component to identify whether the children are Grid and apply margins accordingly. Yet, muiName seems to be lost when the parent component is a server component.

@mwskwong I think you're on the right track, are you interested in investigating further and contributing a PR ? CC @siriwatknp

The nested grid behavior seems to rely on the muiName of the Grid component to identify whether the children are Grid and apply margins accordingly. Yet, muiName seems to be lost when the parent component is a server component.

@mwskwong I think you're on the right track, are you interested in investigating further and contributing a PR ? CC @siriwatknp

I noticed @sai6855 has already taken it so I will let him be. However, I could provide a bit more info.
The muiName is lost because Grid is lazy loaded when the parent component is an RSC. That means strictly speaking children no longer be an array of Grid.
Workaround
Manually set the unstable_level prop. i.e.
<Grid container spacing={2} columns={24}>
  <Grid xs={8}>
    <Item>xs=8/24</Item>
  </Grid>
  <Grid container unstable_level={1} xs={16}>
    <Grid xs={12}>
      <Item>nested xs=12/24</Item>
    </Grid>
    <Grid xs={12}>
      <Item>nested xs=12/24</Item>
    </Grid>
  </Grid>
  <Grid xs={8}>
    <Item>xs=8/24</Item>
  </Grid>
  <Grid container unstable_level={1} xs={16} columns={12}>
    <Grid xs={6}>
      <Item>nested xs=6/12</Item>
    </Grid>
    <Grid xs={6}>
      <Item>nested xs=6/12</Item>
    </Grid>
  </Grid>
</Grid>

@mwskwong please feel free to open a PR if you wish, i'm not entirely confident of fixing the issue

opened PR https://github.com/mui/material-ui/pull/38129


Corrected broken URL for ggplot

Getting a HTTP 404 for the URL http://yhat.github.io/ggplot/
Looks like the project has moved to a new URL at http://yhat.github.io/ggpy/

cc: @glamp from the ggplot team for visibility.

Thanks – yes, they changed the name of the project since I published this.

The name of the project changed as well, so I'll fix in another commit. Thanks!


Fix exception caused by missing Mailchimp key

I think a better fix would be to just return after complaining.

Ok, will do


deploy.cloud.run intermittently works

When using deploy.cloud.run sometimes Cloud Shell doesn't launch the right image.
I think this is because it redirects to like:
https://ssh.cloud.google.com/cloudshell/editor?cloudshell_git_repo=https:%2F%2Fgithub.com%2Fjamesward%2Fhello-netcat.git&cloudshell_image=gcr.io%2Fcloudrun%2Fbutton

Where the other one redirects to like:
https://console.cloud.google.com/cloudshell/editor?shellonly=true&cloudshell_image=gcr.io/cloudrun/button&cloudshell_git_repo=https://github.com/jamesward/hello-netcat.git

Note the domain difference & shellonly=true - which may be exposing some underlying Cloud Shell bug.

Our domain in the code is actually console.cloud.google.com but it looks like I got redirected to ssh.cloud.google.com so the domain doesn't seem like it is the problem.  So probably just the shellonly=true param.

Figured out what was happening...  There wasn't intermittent wrong container usage like I thought.  It was actually that internally there is just a bug in Cloud Shell.  The shellonly param doesn't have anything to do with that.  But adding it is good so that the code editor doesn't show up.  So unrelated to what we were trying to fix.  But good anyhow.


#239 - Safely merge HTTP QB params and server-side QB params avoiding…

… group collisions
This should be a fix to the issue with QB Group collisions (and mergers)... there are some nuances with how QB tracks Group Ids in in the PredicateGroup objects.... which is a mix of specified group Ids, and the the index in an internal predicates array (and they might not always match!)
This issue can be worked around, by preferring the Predicate/Group Java APIs, removing any previously computed Predicate Names (ie. x_group) and then merging those nameless Predicates into an existing Predicate group, at which point, the nameless predicates will be given a non-conflicting name (ie. group). The exception to this, is the special "p" group, which we need merge together (which should be fine).
In this impl, server-side params are merged into the http Params, in an attempt to preserve the HTTP params' group Ids (mostly for debugging)...  when combining the query params (http and server) the server will likely get assigned a group that maps to an unused widget (ie if 3_group is the last http group, then the first server side group will be auto-computed to 4_group).

Ill try to remember tomorrow else I’ll be in Iceland till sept - also we should figure out why the changelog reminder @justinedelson setup on the repo stopped working :(

no problem @davidjgonzalez, I can add it in post-merge. Enjoy the travels!


New Feature: Jump for FTP

Hi everybody,
it would be great if you could also add jump for FTP so we can access to remote FTP server over SSH-Tunnel (Jump Server).
Best regards
Floh

Thank you for the suggestion. This is indeed possible to implement.


feat: configure nodes to use resource name in dns

Fixes #6097
Description
The default "IP name" hostname type is called out in the documentation as the legacy type, and the docs recommend using the resource name hostname type. This also makes operations easier because you can more easily determine the instance ID from the nodes list.
This patch sets the hostname type to HostnameTypeResourceName during launch template creation. This patch does not introduce configuration options for this change, which keeps the surface area simple and matches my interpretation of the launch template design document.
How was this change tested?
Not tested.
Does this change impact docs?

[x] No

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

AFAIK, we have to be really careful with this change because I believe it affects interactions with CCM. I recall us hitting an issue with something like this when instance-id based naming was first introduced.

I'm unfortunately not going to be a great collaborator on this. I don't have much knowledge of k8s internals (no idea what CCM is), but this change would make my job a lot easier when trying to build tooling around k8s security detections and alerts, and include extra context for those alerts based on information available in AWS.


SLOWNESS with vendor.js

I have noticed a major performance issue with my lower power workstations and mobile devices.
Tested with Firefox and Chrome on Xubuntu 16.04 LTS, both had major preformance issues with CNC webpage.
Tested: http://threejs.org/examples/#webgl_multiple_renderers

Firstfox showed some lag with webgl_multiple_renderers
Chrome showed NO lag with webgl_multiple_renderers

Firefox Developer Tools Console:
mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create vendor.js:41:126
THREE.WebGLRenderer 78 vendor.js:26:25127

preformace-profile.json.txt
Possible Solution Explanations / Related Issues:

http://stackoverflow.com/questions/23807805/why-is-mutating-the-prototype-of-an-object-bad-for-performance
https://github.com/mrdoob/three.js/issues/9364

I think most of the performance issue is related to the preview rendering.
Anyway the preview widget could have the option to be disabled?
I would like to have the ability on tablets to disable the proview for performance reasons.

Performance seems better as of v1.6.5
However, multiple devices do not sync current view.
Last device to connect get updates as CNC runs though gcode.

Related Issue: https://github.com/cheton/cnc/issues/33

This issue can be closed now since WebGL is no longer required by disabling 3D view in version 1.7.0.
Related issues: #33, #37, #86


Is cuda/any gpu needed

Do i need to have cuda or any other gpu to run this or can i do it only with a cpu?

Yes, the solver implementation here is set up to require CUDA, as the simulation would probably not be real-time on a CPU. That  said, the code uses Thrust to launch the GPU kernels, and could probably be modified to run on CPU.


AbortSignal is ignored when using response patching

When aborting a request with an AbortController, fetch rejects with an AbortError.
This does not happen when we are using response patching at the moment.
import { FetchInterceptor } from '@mswjs/interceptors/fetch'

const interceptor = new FetchInterceptor();
interceptor.apply()

interceptor.on('request', async (request) => {
  if (request.url.endsWith("/blocked")) {
    request.respondWith(new Response("Blocked", { status: 403 }))
  }
})

// Rejects with an AbortError as expected
const controller1 = new AbortController();
fetch("/trusted", {signal: controller1.signal })
  .then(console.log, console.error);
controller1.abort();

// Fulfills with the patched response
const controller2 = new AbortController();
fetch("/blocked", { signal: controller2.signal })
  .then(console.log, console.error);
controller2.abort();

To keep things consistent with the fetch API, I would suggest to throw the same error when we are patching the response.

Hi, @hpohlmeyer. That's a great observation. We agree, we should not alter the original fetch in any way, which includes forwarding of whichever properties it has upon declaration (like AbortController). Thanks for opening a pull request with the fix so quickly! I will take a look when I have a minute.

Released: v0.22.11 🎉
This has been released in v0.22.11!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


'react.js' does not provide an export named 'useMemo'

Describe the bug
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'useMemo'

Reproduction
https://github.com/down-the-whole/intro
Branch: react
System Info

required vite version: ^1.0.0-rc.3
required Operating System: macOS
required Node version: v12.18.3

Logs (Optional if provided reproduction)
Uncaught SyntaxError: The requested module<EMAIL_ADDRESS>does not provide an export named 'useMemo'

Use the template from https://github.com/vitejs/create-vite-app to use React with Vite. React itself is distributed in a non-ESM-compatible format.


add a log if proxy can not resolve StatsdUdpAddress

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f3381c7f42e83acc60dd67186079a825c5f127bd
link
/test istio-unit-tests

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

/retest

kindly ping @rshriram

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f4fff363410a391681d34228946ac587d17f4770
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
f4fff363410a391681d34228946ac587d17f4770
link
/test istio-pilot-multicluster-e2e

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


net: lwm2m: DTLS session cache enable and queue mode update

Enabled DTLS session cache for support session resume.
Fixed LwM2M queue mode for close connection and reconnect automatically.
Re-connect will do Registration update before it send queued data.
Coap Confirmation pending class allocation fix.
Registration update failure trig full Registration. This will handle session resume failure.

@rlubos @SeppoTakalo @jarlamsa Please review.  I have tested this with Queue mode which was not working properly. But with this It keep registration valid at Coiote server and DTLS session resume is working also.

static void socket_loop(void)
{
int i, rc;
int64_t timestamp;
int32_t timeout, next_retransmit;
while (1) {
	timestamp = k_uptime_get();
	timeout = lwm2m_engine_service(timestamp);

	/* wait for sockets */
	if (sock_nfds < 1) {
		k_msleep(timeout);
		continue;
	}

	That cause problems still. If I observe something We never reach notifcation if socket is closed :(

@jheiskan81 and @rlubos I fix observation that I close socket and re open but not connected.  I added to connection re-open to socket_loop() when socket is not accept to send and it have pending tx. That solve problem and now observation re-connect client to server properly.

@rlubos Hi. I update PR. Unfortunately I was not enable find solution for keep socket closed until we need re-connect. Problem is that resend and observation updates at socket_loop is not handled.
I change place which do reconnect. Now Re-register and lwm2m_message_asynch_send() trig that. I update functionality that UDP socket is not close socket and re-open like you wish.

@rlubos @VeijoPesonen @SeppoTakalo @jarlamsa Please review and comment's. AVSystem server not accept our Registration update if we build Queue client with cache. With Cache Client works very smoothly.
Changes from last are:

Wake-up do re-connect and Registration update to server
Client data is buffered and flushed to engine when Registration is ready

@rlubos @SeppoTakalo @jarlamsa @VeijoPesonen I simplify that Socket connect is handled only 1 place:
Re-registartion state. Message send trigger that state from RF_OFF state.

@rlubos @SeppoTakalo I have updated PR:
It include Coap module fix for allocate Coap pending structure after init.
Now Registration update failure not re-open DTLS it just do full registration.

I would like step to make PR to zephyr Upstream soon.

@VeijoPesonen I fix your comment's.

LGTM, just some small nits about the documentation in header. Btw, you seem to be missing the sauce tags for the commits. The CI is nagging because of that.

Ah I will add those.

@jarlamsa I fix all expected that mbed tls mention.

@SeppoTakalo and @rlubos Do you have any change request or Can I merge this to WIP branch and start doing Upstream PR?

@SeppoTakalo Thanks for comment. I have fix those now.


Replace custom expect library with Jest

I hadn't seen that Jest exports their own expect library that works
inside the dom before completing the integration tests. Replace
ours with theirs in the assertion stories because their error
messages are better and library is more complete.
Test plan: no screenshot changes

Sweet 😄


Suggestion: fail without throwing an error

Currently when IntersectionObservers is not available, sal disables the animations gracefully but throws an Error afterwards.
This causes it to show up in Sentry, but more importantly, when it is bundled with other scripts (in my case by WP Rocket) all other scripts in the same file are not executed anymore.
Would you consider adding an option to disable the error and just let sal.js exit silently?

I'd like to not add such option, because throwing such error tells developer that Intersection Observer is not loaded. If you want to silence this error, you can catch it with try/catch, e.g.:
try {
  sal();
} catch(error) {
  // do something with you error, i.e. console.error(error)
}

Using try/catch Sentry won't get this error and it will not stop the execution of other scripts.
Let me know if it helps you.

Thanks for your response, using the try/catch indeed does the trick.


Update getting-started.rst

When dealing with a multi currency site, FE, it can drive you nuts to understand Money initialization if you don't realice you've to create the object represented in the smallest unit. The examples in the documentation use EUR and USD as example currencies, and both hace 2 minor units. I think it'd be very beneficial to state that other currencies may have other minor units, or none!
It took me 5 hours to notice it :)

Sorry, I dont see this as a beneficial. I simply don't get what has been added to docs.


Create a simple example *.proto && Makefile, to quickly test gorpc.

Is your feature request related to a problem? Please describe.
Create a simple example *.proto && Makefile, to quickly test gorpc.

done


RuntimeError: cuda runtime error (38) : no CUDA-capable device is detected at /pytorch/torch/lib/THC/THCGeneral.c:70

help me
return model.cuda( )

Traceback (most recent call last):
File "/home/hubin/super-events-cvpr18/train_model.py", line 249, in
rgb_results = eval_model(rgb_model, dataloaders['val'], baseline=True)
File "/home/hubin/super-events-cvpr18/train_model.py", line 121, in eval_model
for data in dataloader:
File "/home/hubin/anaconda2/lib/python2.7/site-packages/torch/utils/data/dataloader.py", line 417, in iter
return DataLoaderIter(self)
File "/home/hubin/anaconda2/lib/python2.7/site-packages/torch/utils/data/dataloader.py", line 242, in init
self._put_indices()
File "/home/hubin/anaconda2/lib/python2.7/site-packages/torch/utils/data/dataloader.py", line 290, in _put_indices
indices = next(self.sample_iter, None)
File "/home/hubin/anaconda2/lib/python2.7/site-packages/torch/utils/data/sampler.py", line 119, in iter
for idx in self.sampler:
File "/home/hubin/anaconda2/lib/python2.7/site-packages/torch/utils/data/sampler.py", line 50, in iter
return iter(torch.randperm(len(self.data_source)).long())
RuntimeError: invalid argument 1: must be strictly positive at /pytorch/torch/lib/TH/generic/THTensorMath.c:2247

Please install Pytorch using right command by selecting versions of python and OS @ "https://pytorch.org/". This solve above error for me...

If the error is a cuda runtime error(38): no CUDA-capable device you might want to change the default arg passed as the gpu parameter from '1' to '0' as by default most single GPU systems have their device index as 0 not 1. This fixed the problem for me.

RuntimeError: cuda runtime error (38) : no CUDA-capable device is detected at /tmp/pip-req-build-4baxydiv/aten/src/THC/THCGeneral.cpp:50  I can't find the location where i can modify the gpu from '1' to '0'


[BUG-Web] Showing dialog above html editor uses editors focus events

If a dialog is shown above an html editors edit space, you could not use the dialogs buttons or other pointer events.
Steps to reproduce the behavior:

Create view with html editor, expand it over the whole viewport
Open dialog with buttons
Try to click button
See error

Please see https://github.com/tneotia/html-editor-enhanced/issues/30 for the fix. This is an issue with Flutter :)

Must have missed this entry, but thanks a lot, worked right away :)


Fix broken link in GET_A_BLOT.md

Not a Blot piece, just fixed a broken link :)

thanks for fixing - typos suck


fix(guide): Guide article moved to its right location(closes #23349)

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #23349

@RandellDawson PR #20125 has fixed the issue, so I am closing this PR.


Use Python 3.7 for JupyterLab

JupyterLab dropped its support for Python 3.6
(Jupyter notebook still works on Python 3.6).

https://github.com/jupyterlab/jupyterlab/pull/11740
https://github.com/jupyter/notebook/blob/master/notebook/traittypes.py#L17
https://github.com/jupyter-server/jupyter_server/blob/main/jupyter_server/traittypes.py#L7

Update document to use Python 3.7 for JupyterLab.
Fixes #240
Signed-off-by: Gigon Bae<EMAIL_ADDRESS>

Was wondering if we just require Python >= 3.7 for the whole project, but MONAI core is still setting >=3.6.
@MMelQin

Right, it could be better if we could require Python >= 3.7 as Python 3.6 is the end of life now.
Maybe we need to request MONAI core team to drop Python 3.6 support.


how to change  height of hud view

I want to change the height of view.
thank you .

The height of the view is determined by the contents and cannot be changed. You can however either set the square property to true, which which always keep the height and width equal.


Wheelie should convert a "--production" argument into a "production" option

With the following cli invocation:
$ gulp build --production

Wheelie should provide a global options.production flag used by all tasks to control their actions. A common example may be the uglify process that should prevent any minification if the build process is executed without the --production flag

yargs is used to get the production status and the globals object provided to each task now contains:
{
// ...
production: true | false
dest: <the_destination_folder_according_to_production>
// ...
}


Azure documentation is very messed up.

Azure documentation is very messed up, this is not what end years expect. They have so much difficulty following or understanding the procedures. AWS has better documentation.

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

ID: 0dbd38fa-89f0-320a-b517-b440be6927f9
Version Independent ID: 8b2eeee3-fb49-37e5-917d-f36bcd60ad29
Content: Azure Disk Encryption Prerequisites
Content Source: articles/security/azure-security-disk-encryption-prerequisites.md
Service: security
GitHub Login: @mestew
Microsoft Alias: mstewart

@sam90909 I'm sorry to hear that! Is there anything specific you would like to improve? Please provide your feedback in the comments.

Thank you for responding. If there is an option to choose only the commands for CLI and Powershell separately it would be good. I don't think anyone would want to use the PowerShell and CLI at a time. It would be easy for the reader if it is little organized.  If there is an explanation for why the AD app is required or connected or an explanation for why the AD app is required would make it easy for the user to understand to implement disk encryption. I was stuck there thinking why we need that. There is all the Azure information on the internet but it is not well organized which is making it difficult for end users to understand and complex. This is not just my struggle for many months.


.as fails if the option is given but empty

Hi!
Maybe I am missing something here, but I've looked through the readme, example, wiki, and the issues, and couldn't find anything.
If I do this:
std::string input_file;
options.add_options().("i,input", "Input", cxxopts::value(input_file));

and then I call the program with -i (with the option, but without giving a string), then I'm getting a crash (run-time error).
Likewise, if I do:
options.add_options().("i,input", "Input", cxxopts::value<string>());
const auto result = options.parse(argc, argv);
std::string input_file;
if (result.count("input"))
    {
        input_file = result["input"].as<string>();
    }

it crashes as well - obviously, because it tries to access a string that is not there.
This kind of seems like a design flaw to me, but I am pretty sure that I just haven't figured out yet how to do this properly. I am surprised though to not find the solution in the readme or an example, or people already asking this in another issue.
Am I just blind or am I not seeing the obvious?
Thanks! :-)

What sort of a crash is it? If it's throwing an exception then that's fine. It's generally expected that you wrap your argument processing code in a try catch. That is possibly not so well documented (if at all), which I can definitely fix.
If it is giving you a segmentation fault then that is a bug.

Ah, of course. That's it. That was the bit that was missing..... :-)
I think a sentence about that in the readme or wiki would definitely help. But maybe that's just me, since nobody seemed to have run into that before!
Thank you very much for the help and quick reply!


syntax error with filepath and outpath change

Hi,
I am trying to use the ZeekToCSV.sh on windows and you mention changing FILEPATH and OUTPATH but I don't know if I am approaching properly b/c of the error mentions /r command not found resulting
Is there a way to give an example of what changes are needed.  For instance are all mentions of FILEPATH needing to be replaced or just between the $ FILEPATH$ or $OUTPATH$ to the actual log file name?

This is the code that is giving me problems if you see any issues with it:
#!/bin/bash
FILEPATH=/"path_to_the_input_logs_directory"/ #path to input logs directory
OUTPATH=/"path_to_the_output_logs_directory"/ #path to output logs directory
echo "Transforming to csv ..."
for log in $(ls $FILEPATH); do
FILEIN=$FILEPATH$log
FILEOUT=$OUTPATH$log

# -d option in zeek-cut convert time values into
# human-readable format. %Y-%m-%dTH%:%M:%S
cat $FILEIN | zeek-cut -c -d > $FILEOUT

# If you don´t want to use zeek-cut, comment the line above
# and uncomment the line below
#sed -i '$d' $FILEOUT

# Remove line 8
sed -i '8d' $FILEOUT

# Remove lines 1 to 6
sed -i '1,6d' $FILEOUT

# Delete #fields
sed -i 's/#fields\t//' $FILEOUT

# Replace '\t' with ','
sed -i 's/\t/,/g' $FILEOUT

done
echo "Done"

This is the code that is giving me problems if you see anything wrong with my input to your FILEPATH and OUTPATH mentions.  thank you

#!/bin/bash
bro=/"path_to_the_input_logs_directory"/ #path to input logs directory
bro_csv=/"path_to_the_output_logs_directory"/ #path to output logs directory
echo "Transforming to csv ..."
for log in $(ls $bro); do
FILEIN=$bro$log
FILEOUT=$bro_csv$log

# -d option in zeek-cut convert time values into
# human-readable format. %Y-%m-%dTH%:%M:%S
cat $FILEIN | zeek-cut -c -d > $FILEOUT

# If you don´t want to use zeek-cut, comment the line above
# and uncomment the line below
#sed -i '$d' $FILEOUT

# Remove line 8
sed -i '8d' $FILEOUT

# Remove lines 1 to 6
sed -i '1,6d' $FILEOUT

# Delete #fields
sed -i 's/#fields\t//' $FILEOUT

# Replace '\t' with ','
sed -i 's/\t/,/g' $FILEOUT

done
echo "Done"


integrate with database like MySQL's query result

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Right now, the project only support the uploaded datasource.
Is there possible to translate user's language to SQL by codex model, then query the result as datasource to do data analysis and visualization?
✔ Solution
Because the data source just supports the uploaded files.
And I want to use different datasources like database as data sources.
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

You should be able to ask it to connect to a database. You will need to provide a database connection string. Dedicated support for something like this I consider to be outside of the scope of the project, I want to keep it simple.


Pad shape offset

Great tool!, I find a small bug on pad visualisation:
Apparently the viewer don't take into account the pads shape offset.
Pads on the viewer

Pad on Kicad

Pad Parameter

Thanks for the report.


Refactor and fix some bugs

Fix a few bugs
Refactor the files: sources are in tools/operf-micro, while benchmarks are in share/operf-micro

You could have made the CI happy...


Performance impact to VMs

It would be great to post a guidance on the type of performance impact that can be expected of running the scan.  For instance, the scans will increase CPU by X% and memory by Y%.  Would this be possible?

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

ID: a9303652-2bf3-e20a-4213-628466cc209d
Version Independent ID: b73187c6-e566-49af-ef57-d517557074a7
Content: Advanced Data Security for IaaS in Azure Security Center
Content Source: articles/security-center/security-center-iaas-advanced-data.md
Service: security-center
GitHub Login: @monhaber
Microsoft Alias: v-mohabe

@SenthuranSivananthan Thanks for your feedback! We will investigate and update as appropriate.

Looking into this - will can back to you within a few days.

#reassign:memildin

There are two relevant answers to this:
For threat protection, the ATP workload is carried outside of the SQL Server Engine, and therefore has a negligible performance impact on the SQL Server Process.
For vulnerability assessment, the scan runs a series of queries (under 100) on the database and will, therefore, consume some server resources during the scan. Scans are executed once a week and usually end within a few seconds. The impact of the scan on the server depends on the size of the server and the load at the time of the scan.
#please-close

@memildin thanks for the background on the impact.  Is there a doc update that can be made so this information is not lost?


io.BytesIO.seek overload issues

The following should work:
import io
io.BytesIO().seek(0L, 0)

but results in

TypeError: Multiple targets could match: seek(long, object), seek(float, int)

Also, the following should raise a TypeError but does not:
import io
io.BytesIO().seek(0, 1.0)

Simple workaround is to make the second argument a long (e.g. io.BytesIO().seek(0L, 0L)).


rewrite-test kotlin tests are no longer able to be compiled on Windows

When using the backtick method names, these are transformed into subclass definitions. As a result, great care has to be taken to make sure that the method names remain OS-agnositic. At present, there are some in the org.openrewrite.java.cleanup.RemoveMethodCallVisitorTest that will need to be resolved to re-allow the project to be able to be compiled on Windows.

Fixed by #2068

Sorry about that. That's my mistake


Use FRU for with_alphabet, with_check

Item 1 from issue #43

@Nemo157 any notes about this change?

Looks good, thanks
bors r+


error: Unable to find vcvarsall.bat

when trying to run the PyZMQ Test Suit the error above occurs. Using Windows10 machine.

What version(s) of Visual Studio and/or Windows SDK are you using?

Try installing Microsoft Visual C++ Compiler for Python if you haven't already.


Added Reverse Link List in C++

Have you read the Contributing Guidelines on Pull Requests?
There is no Guidelines present
Description
Reverse Link List in C++ in both Recursive and Iterative approach.
Clone of head pointer so that it can easily take the inputs.
Resolve errors and modify the code.
Checklist

[x]  I've read the contribution guidelines.
[x]  I've checked the issue list before deciding what to submit.
[x]  I've edited the README.md and link to my code.

Related Issues or Pull Requests
Output of above code.
(Write your answer here.)

Output of above code.

@Kajol-Kumari @SKAUL05 @Abhijit2505 @plazzy99 @siddharth25pandey
please review my PR .

Output of above code.

Output Of the above Interpolation Search code.


Refactor Printing and Formatting of Messages in CLI

CLI code is hard to read as the externalized string package being used requires a map of keys and values. This process should be abstracted to make the code more readable.

@csantanapr, @mdeuser, this is technical debt, and should be fixed ASAP.

Looking forward to a PR :)

As part of doing this, consider: adding quiet mode which omits "ok:" and "error:" lines so that raw Json responses are more amenable to post processing.

Alternatively: --json or --raw to explicitly make the output machine-readable?

@markusthoemmes, a flag like --raw would be accomplished through this issue: https://github.com/openwhisk/openwhisk/issues/2064.

A quiet mode would be handy for property get too. That way I wouldn't need to do this:
wsk property get --cliversion | rev | cut -f1 | rev

Issue moved to apache/incubator-openwhisk-cli #142 via ZenHub


Add images to API

Hope to add images to the API very soon. The entry endpoint will have an additional key, image, that has a URL of the image (hosted on the same heroku server). That URL will be /entry/<entry>/image, which will serve the image in a png of jpeg format.

All images are to be in PNG format and 280x280 pixels.

Finished in https://github.com/gadhagod/Hyrule-Compendium-API/commit/af471bc54213036f81680ce85c2225dc320a3826.


Trial

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hello @Rahul72383! I see this is your first ever GitHub issue. Can you explain the purpose of this ticket in more detail (e.g. feature request, bug report, support question, etc.)?
I'll re-open when there's something actionable.


[MRG, MAINT] Fetching Datasets Public API "fetch_dataset" used by all internal MNE datasets now

Reference issue
Closes #9736
Closes #9774
What does this implement/fix?
This adds the public function mne.datasets.fetch_dataset and refactors the existing MNE datasets to use that function.
This also refactors _data_path function to be more generic.

option 1 seems better; I like it being in the same place as the hash.  With option 2 it's not clear how users would handle private datasets that also needed unzipping/untarring.  However you'll need to figure out how to still make pooch an optional dependency since with option 1 it's used outside of a function.

option 1 seems better; I like it being in the same place as the hash. With option 2 it's not clear how users would handle private datasets that also needed unzipping/untarring. However you'll need to figure out how to still make pooch an optional dependency since with option 1 it's used outside of a function.

I played around with it, and there is no easy way to make it all option 1 because some of the datasets require some customized pathing anyways :/, so I went w/ option 2.
e.g.
    # instantiate processor that unzips file
    processor = pooch.Untar(
        extract_dir=path, members=[f'hf_sef/{subdir}' for subdir in
                                   ('MEG', 'SSS', 'subjects')]),

We still need this code inside data_path(), rather than config as a result. It seems actually less error-prone to store the extraction logic along w/ the type or processor in the same place. I don't see the zip/tar/nested-tar changing frequently where this would be an issue. The pattern is also pretty consistent for adding a new dataset.
Lmk wyt. I can also make a quick example to demonstrate mne.datasets.fetch_dataset if you think it's worthwhile, else I can just add more information to the doc string.

I guess I can actually just get rid of _data_path

Pair PR in: #9764

CI Failure seems like its the recurring test in whitener issue.
https://34581-1301584-gh.circle-artifacts.com/0/dev/generated/mne.datasets.fetch.fetch_dataset.html#mne.datasets.fetch.fetch_dataset

if we already have a clear usecase for fetch_dataset function then let's do it.
2 things:

I would expose it as mne.datasets.fetch_dataset
I would remove the _data_path that now seems superfluous.

thanks for the clarifications @adam2392

No strong opinion about that, so leave config.py as-is if you think that's cleaner.
Sent from ProtonMail mobile
-------- Original Message --------
On Sep 22, 2021, 14:42, Adam Li wrote:

@adam2392 commented on this pull request.

In mne/datasets/fetch.py:

dataset_params : dict of dict

   The dataset name and corresponding parameters to download the dataset.

   The dataset parameters that contains the following keys:

   ``archive_name``, ``url``, ``folder_name``, ``hash``,

   ``config_key`` (optional). See Notes.

Makes sense to me.
Should I change the format in config.py, or handle internally the MNE datasets to go from dict of dict -> list of dict if needed when calling fetch_dataset?
I think the format in config.py is nice cuz it consolidates LOC per dataset.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hmmm some Sphinx error left over.
Not sure why fetch_dataset isn't getting picked up.
WARNING: [autosummary] failed to import 'mne.datasets.fetch_dataset': no module named mne.datasets.fetch_dataset
[autosummary] generating autosummary for: D:\a\1\s\doc\generated\mne.AcqParserFIF.rst, D:\a\1\s\doc\generated\mne.Annotations.rst, D:\a\1\s\doc\generated\mne.BaseEpochs.rst, D:\a\1\s\doc\generated\mne.BiHemiLabel.rst, D:\a\1\s\doc\generated\mne.Covariance.rst, D:\a\1\s\doc\generated\mne.Dipole.rst, D:\a\1\s\doc\generated\mne.DipoleFixed.rst, D:\a\1\s\doc\generated\mne.Epochs.rst, D:\a\1\s\doc\generated\mne.EpochsArray.rst, D:\a\1\s\doc\generated\mne.Evoked.rst, ..., D:\a\1\s\doc\generated\mne.write_events.rst, D:\a\1\s\doc\generated\mne.write_evokeds.rst, D:\a\1\s\doc\generated\mne.write_forward_solution.rst, D:\a\1\s\doc\generated\mne.write_head_bem.rst, D:\a\1\s\doc\generated\mne.write_label.rst, D:\a\1\s\doc\generated\mne.write_labels_to_annot.rst, D:\a\1\s\doc\generated\mne.write_proj.rst, D:\a\1\s\doc\generated\mne.write_source_spaces.rst, D:\a\1\s\doc\generated\mne.write_surface.rst, D:\a\1\s\doc\generated\mne.write_trans.rst

I still don't understand the SphinxWindows failure, but this one on CircleCI seems legit (I can replicate locally):
$ python -c "import mne; mne.datasets.fieldtrip_cmc.data_path(verbose=True)"
Using default location ~/mne_data for fieldtrip_cmc...
Downloading file 'SubjectCMC.zip' from 'https://osf.io/j9b6s/download?version=1' to '/Users/larsoner/mne_data'.
100%|███████████████████████████████████████| 329M/329M [00:00<00:00, 1.76TB/s]
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<decorator-gen-500>", line 22, in data_path
  File "/Users/larsoner/python/mne-python/mne/datasets/fieldtrip_cmc/fieldtrip_cmc.py", line 20, in data_path
    return _download_mne_dataset(
  File "/Users/larsoner/python/mne-python/mne/datasets/utils.py", line 175, in _download_mne_dataset
    return fetch_dataset(dataset_params=dataset_params, processor=processor_,
  File "/Users/larsoner/python/mne-python/mne/datasets/_fetch.py", line 260, in fetch_dataset
    fetcher.fetch(
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/core.py", line 549, in fetch
    return processor(str(full_path), action, self)
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/processors.py", line 86, in __call__
    self._extract_file(fname, self.extract_dir)
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/processors.py", line 196, in _extract_file
    with TarFile.open(fname, "r") as tar_file:
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/tarfile.py", line 1604, in open
    raise ReadError("file could not be opened successfully")
tarfile.ReadError: file could not be opened successfully

+1 for MRG if Ci failure is unrelated

I still don't understand the SphinxWindows failure, but this one on CircleCI seems legit (I can replicate locally):
$ python -c "import mne; mne.datasets.fieldtrip_cmc.data_path(verbose=True)"
Using default location ~/mne_data for fieldtrip_cmc...
Downloading file 'SubjectCMC.zip' from 'https://osf.io/j9b6s/download?version=1' to '/Users/larsoner/mne_data'.
100%|███████████████████████████████████████| 329M/329M [00:00<00:00, 1.76TB/s]
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<decorator-gen-500>", line 22, in data_path
  File "/Users/larsoner/python/mne-python/mne/datasets/fieldtrip_cmc/fieldtrip_cmc.py", line 20, in data_path
    return _download_mne_dataset(
  File "/Users/larsoner/python/mne-python/mne/datasets/utils.py", line 175, in _download_mne_dataset
    return fetch_dataset(dataset_params=dataset_params, processor=processor_,
  File "/Users/larsoner/python/mne-python/mne/datasets/_fetch.py", line 260, in fetch_dataset
    fetcher.fetch(
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/core.py", line 549, in fetch
    return processor(str(full_path), action, self)
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/processors.py", line 86, in __call__
    self._extract_file(fname, self.extract_dir)
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/site-packages/pooch/processors.py", line 196, in _extract_file
    with TarFile.open(fname, "r") as tar_file:
  File "/Users/larsoner/opt/miniconda3/lib/python3.8/tarfile.py", line 1604, in open
    raise ReadError("file could not be opened successfully")
tarfile.ReadError: file could not be opened successfully

Ah yeah the file is a zip file, so we just needed to make it nested_unzip instead of nested_untar.
Original code I copied over was accidentally nested_untar (Ref: https://github.com/mne-tools/mne-python/pull/9742/files)

With the exception of the 3 windows Azure CI failures that I have no idea on, this is good to go by me.
I also tested it out w/ my downstream repo and the fetch_dataset works nicely for a private GITHUB repo.
lmk if you find other things I need to fix.

I don't understand the Windows Sphinx error -- I'll probably merge ignoring that assuming the empty [circle full] commit I just pushed succeeds. I can look into the Windows Sphinx error on Monday if it persists.

Next:
https://circleci.com/gh/mne-tools/mne-python/34733?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Downloading file 'sample_reference_MEG_noise-raw.zip' from 'https://osf.io/drt6v/download?version=1' to '/home/circleci/mne_data'.
100%|██████████████████████████████████████| 91.0M/91.0M [00:00<00:00, 102GB/s]
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<decorator-gen-497>", line 22, in _download_all_example_data
  File "/home/circleci/project/mne/datasets/utils.py", line 264, in _download_all_example_data
    refmeg_noise.data_path()
  File "<decorator-gen-527>", line 24, in data_path
  File "/home/circleci/project/mne/datasets/refmeg_noise/refmeg_noise.py", line 17, in data_path
    return _download_mne_dataset(
  File "/home/circleci/project/mne/datasets/utils.py", line 177, in _download_mne_dataset
    return fetch_dataset(dataset_params=dataset_params, processor=processor_,
  File "/home/circleci/project/mne/datasets/_fetch.py", line 260, in fetch_dataset
    fetcher.fetch(
  File "/home/circleci/.local/lib/python3.8/site-packages/pooch/core.py", line 549, in fetch
    return processor(str(full_path), action, self)
  File "/home/circleci/.local/lib/python3.8/site-packages/pooch/processors.py", line 86, in __call__
    self._extract_file(fname, self.extract_dir)
  File "/home/circleci/.local/lib/python3.8/site-packages/pooch/processors.py", line 196, in _extract_file
    with TarFile.open(fname, "r") as tar_file:
  File "/usr/local/lib/python3.8/tarfile.py", line 1606, in open
    raise ReadError("file could not be opened successfully")
tarfile.ReadError: file could not be opened successfully

@adam2392 can you try to fix, and when you make your commit and push have [circle full] in the message?

So I think all the downloads work now, but this is a weird error:
 File "/home/circleci/project/tutorials/clinical/20_seeg.py", line 72, in <module>
    head_mri_t = mne.coreg.estimate_head_mri_t('sample_seeg', subjects_dir)
  File "<decorator-gen-74>", line 24, in estimate_head_mri_t
  File "/home/circleci/project/mne/_freesurfer.py", line 445, in estimate_head_mri_t
    lpa, nasion, rpa = get_mni_fiducials(subject, subjects_dir)
  File "<decorator-gen-73>", line 24, in get_mni_fiducials
  File "/home/circleci/project/mne/_freesurfer.py", line 419, in get_mni_fiducials
    mni_mri_t = invert_transform(read_talxfm(subject, subjects_dir))
  File "<decorator-gen-75>", line 24, in read_talxfm
  File "/home/circleci/project/mne/_freesurfer.py", line 485, in read_talxfm
    ras_mni_t = read_ras_mni_t(subject, subjects_dir)
  File "/home/circleci/project/mne/transforms.py", line 1466, in read_ras_mni_t
    fname = _check_fname(
  File "/home/circleci/project/mne/utils/check.py", line 180, in _check_fname
    raise FileNotFoundError(f'{name} does not exist: {fname}')
FileNotFoundError: FreeSurfer Talairach transformation file does not exist: /home/circleci/mne_data/MNE-sample-data/subjects/sample_seeg/mri/transforms/talairach.xfm

I see sample_seeg/ folder  in MNE-misc-data/seeg/ though not in the sample-data. Unless the version is wrong??

So in summary the circle CI fixes are:

there were some datasets that were unzip instead of untar and vice versa that was never caught in the original pooch PR
some other PRs never caught the change in misc dataset, which renamed some data files, and thus broke a few tests and examples

This PR now addresses those and updated misc dataset to v0.22. Fingers crossed for this [circle full] commit.

Green, in it goes!

Great stuff @adam2392 🎉 Im excited to add a few datasets to MNE-NIRS using this, thanks.