Unresponsive in Sketch 3.3.2?

I've installed this plugin as I have other plugins, but when I choose any option (Lorem Pixel, etc.) it does not do anything. This happens if I have a group selected, or nothing at all.
Also, when I tried placekitten, it crashed Sketch.

@finedesign I'm not having any luck reproducing what you described. I'm using Sketch 3.3.2 and Day Player 2.0 https://github.com/tylergaw/day-player/releases/tag/v2.0.0
Are you using the latest version of Day Player?

I can confirm.  Using placekitten crashes Sketch v3.3.3.

@justiny Bummer. Can you give me a few more details about your setup? OS X version...not sure what else, just anything that might help me reproduce. What about other services? Is it only placekitten that causes Sketch to crash?

@justiny You may be running into this issue https://github.com/tylergaw/day-player/issues/11

I'm gonna go ahead and close this out. A lot has changed in the latest 3.0.0 release and I'm not sure this issue is still relevant.


Pass skip_verify: true in opts to disable token signature verification.

This is useful for decoding the contents of a JWT that you don't have the secret key for.

Oops -- I'll get rid of the whitespace changes and update the commit in a bit. Sorry about that!

Updated. (Although, trailing whitespace really should be stripped...)

Just curious when would you want to decode without verifying? What is the use case?
Perhaps better than adding this option it would be to make the get_json function public so you can simply get the payload. Even this way it seems odd to me...

Making the get_json function public would probably be better than using skip_verify. I probably should have put this together, but my use case is one app holds multiple apps, and conditionally decodes and verifies JWTs based on the iss claim. Since there's no way to tell what secret key to use before decoding it, you have to skip the verify until you know which key to use. (Not being able to use a dynamic secret key is another limitation of this library at the moment)

Thanks! I think this should be ok for now. In the update that's coming I think both of these use cases should be covered as well. I think this is the reason behind the none algorithm in the spec.

Yeah, I've been following that issue. Thanks!


Maintain documentation in a separate repository

Moved documentation to https://github.com/morph-kgc/morph-kgc-docs


add support for multi partition kafka topics

Description
This will add support to configure a certain topic class to have a specified number of partitions by including a partitions config in the configuration for that topic class. This currently does not include the consumer metric reporting for delay and lag since it assumes a single partition topic so it uses only partition 0 to emit that metric. But this change will unblock from attempting to use multiple partitions for your kafka topics by not having the producer use specify a key for the producer record such that the kafka messages are always hashed to the same consistent partition.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[X] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[X] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5398 (291b3b2) into master (de3e0a8) will decrease coverage by 72.32%.
The diff coverage is 0.00%.

:exclamation: Current head 291b3b2 differs from pull request most recent head db7e4c8. Consider uploading reports for the commit db7e4c8 to get more accurate results

@@            Coverage Diff             @@
##           master   #5398       +/-   ##
==========================================
- Coverage   76.83%   4.52%   -72.32%
==========================================
  Files         240     240
  Lines       14596   14596
  Branches      672     671        -1
==========================================
- Hits        11215     660    -10555
- Misses       3381   13936    +10555

Impacted Files
Coverage Δ

...whisk/connector/kafka/KafkaConsumerConnector.scala
56.33% <ø> (-25.36%)
:arrow_down:

...whisk/connector/kafka/KafkaMessagingProvider.scala
24.39% <0.00%> (-46.35%)
:arrow_down:

...whisk/connector/kafka/KafkaProducerConnector.scala
0.00% <0.00%> (-87.81%)
:arrow_down:

... and 202 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


"WARNING: ObjectDB instances leaked at exit" on initiating Node2d object with c#

Godot version: 3.2.3 - stable_mono_win64

OS/device including version: Windows 10 Home PC

Issue description:

Tried instantiating a Node2d object from code:

var newNode = new Node2D();
The above command triggers the following error:
WARNING: ObjectDB instances leaked at exit (run with --verbose for details).
at: cleanup (core/object.cpp:2135)
Steps to reproduce:

Create new project

Create main scene

Insert the above command var newNode = new Node2D();  in the _Ready() method

Minimal reproduction project:

TestLeak.zip

I've figured out that if I connect the instance to the SceneTree by
AddChild(newNode); for example, the leak is gone. But I think there should be a garbage collection that picks up unparrented nodes on exit.

This warning makes sense, it's not a bug.
Nodes are not automatically freed on exit, unless they are added to the scene, for example by using add_child(newNode); in your case.
On the other hand, Resources or any class that inherits from Reference use a ref counting system and are freed automatically when not used.
Documentation:
https://docs.godotengine.org/en/stable/getting_started/scripting/gdscript/gdscript_basics.html?#memory-management
Note for contributors:
Maybe the documentation could be clearer on this aspect? It doesn't say anything about Nodes not being managed automatically unless added to the scene.

Documentation:
https://docs.godotengine.org/en/stable/getting_started/scripting/gdscript/gdscript_basics.html?#memory-management

Thanks, good to know. Coming from a .net background I assumed that there is garbage collection.

Indeed that's expected behavior. AddChild(newNode); solves the issue by making the new Node part of the scene tree and thus freed together with the scene tree when you close the game.

Maybe the documentation could be clearer on this aspect? It doesn't say anything about Nodes not being managed automatically unless added to the scene.

I opened an issue for this:
https://github.com/godotengine/godot-docs/issues/4877

This was very helpful. Thanks!


[FLINK-28150][sql-gateway][hive] Introduce the hiveserver2 endpoint and factory

What is the purpose of the change
Introduce the HiveServer2 Endpoint and its Factory. With the HiveServer2 Endpoint, users is avaliable to connect to the SqlGateway with Hive JDBC.
Brief change log

Introduce the endpoint and its factory. With the factory, the SqlGateway is able to load the HiveServer2 Endpoint
Expose the required options to configure HiveServer2 Endpoint. The options are discussed in the FLIP-223 except the option thrift.max.message.size, thrift.login.timeout, thrift.exponential.backoff.slot.length. The newly introduced options are align with the hive, please refer to the HiveConf. The PR also introduce the option module.name that aligns with the option catalog.name as the default hive module name.
Supports OpenSession and CloseSession in HiveServer2Endpoint.

Verifying this change
This change added tests and can be verified as follows:

Added ITCase to openSession and closeSession with HiveServerEndoint2
Added test that the endpoint factory can create the expected endpoint.
Added the client can get expected error message with HiveServer2 endpoint

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure


packtpub site support

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.03.26. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2017.03.26

Before submitting an issue make sure you have:

[x ] At least skimmed through README and most notably FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

can you please add this website to download video courses which they allow to watch online
packtpub.com
course link
https://www.packtpub.com/mapt/video/web-development/9781787122215

is it working with this file ?
i put these two changes in my folder but i am getting this error
PS C:\Users\vicky\downloads> youtube-dl --verbose -o '~/downloads/%(playlist)s/%(playlist_index)s - %(title)s.%(ext)s' h
ttps://www.packtpub.com/mapt/video/web-development/9781787122215 --username<EMAIL_ADDRESS>--password 123
Traceback (most recent call last):
File "c:\users\vicky\appdata\local\programs\python\python35\lib\site-packages\youtube_dl\extractor_init_.py", line
4, in
from .lazy_extractors import *
ImportError: No module named 'youtube_dl.extractor.lazy_extractors'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\vicky\appdata\local\programs\python\python35\lib\runpy.py", line 184, in run_module_as_main
"main", mod_spec)
File "c:\users\vicky\appdata\local\programs\python\python35\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\vicky\AppData\Local\Programs\Python\Python35\Scripts\youtube-dl.exe_main.py", line 5, in
File "c:\users\vicky\appdata\local\programs\python\python35\lib\site-packages\youtube_dl_init.py", line 43, in
from .extractor import gen_extractors, list_extractors
File "c:\users\vicky\appdata\local\programs\python\python35\lib\site-packages\youtube_dl\extractor_init_.py", line
9, in
from .extractors import *
File "c:\users\vicky\appdata\local\programs\python\python35\lib\site-packages\youtube_dl\extractor\extractors.py", lin
e 731
+from .packtpub import (
^
SyntaxError: invalid syntax
PS C:\Users\vicky\downloads>


Add multi-producer and inflight request control for the Kafka benchmark

In this PR, the Kafka client uses multiple producers to dispatch requests. Each producer flushes the queue after dispatching a certain number of requests to mitigate the uncontrolled queueing delay. The number of producers and the outstanding messages per flush can be controlled from kafka.cnf that has five parameters: "total messages", "number of producers", "message size", "messages per flush", "number of partitions".

Add multiple producer support
Add flush per messages support
Remove unnecessary Kafka config changes, only keep the log.segment.delete.delay.ms = 10 setting
Change linger.ms = 0 to avoid client-side buffering

Thanks Xi. Is it expected that -t has no effect on the benchmark? I can see that with and without -t 1, it uses about 800~1200% CPU on Ryzen 9 7950X.

#257 can still be observed using this PR's branch

Thanks Xi. Is it expected that -t has no effect on the benchmark? I can see that with and without -t 1, it uses about 800~1200% CPU on Ryzen 9 7950X.
@caizixian @steveblackburn  We don't control the threading in Kafka server and the number of client producers, so -t won't have an impact on the CPU utilization.

The benchmark uses 8 producers in this PR and each producer creates one background IO thread at the client side, so the clients contribute about half CPU time. If we want to build some connections between the CPU usage and the -t parameter, maybe we could set the number of produces to half of the -t parameter value given that the clients contribute about half total CPU usage?


Error configuring with CMake on Windows

Windows 10, VS 2017, CMake gui 3.13.4, CUDA 10.1.
UPD. I use old CPU which not support AVX, only SSE4.
DLDT was cloned
git clone https://github.com/opencv/dldt.git --recurse-submodules
and after configured via CMake gui.
Error received:
Selecting Windows SDK version 10.0.17763.0 to target Windows 10.0.14393.
 OpenVINO
CMAKE_BINARY_DIR ...................... E:/Lib_prebuild/DLDT/prebuild_x64/inference-engine
OpenVINO_MAIN_SOURCE_DIR .............. E:/Lib_prebuild/DLDT/source
IE_MAIN_SOURCE_DIR .............. E:/Lib_prebuild/DLDT/source/inference-engine
CMAKE_GENERATOR ....................... Visual Studio 15 2017 Win64
CMAKE_C_COMPILER_ID ................... MSVC
CMAKE_BUILD_TYPE ......................  Release will be used
[clDNN] ======================== clDNN Project =======================
[clDNN] Version:          <IP_ADDRESS>
[clDNN]
[clDNN] Build type:       Release (for single-configuration generators)
[clDNN] Av. build types:  Debug;Release;RelWithDebInfo (for multi-configuration generators)
[clDNN]
[clDNN] Output bin directory:
[clDNN]  - "E:/Lib_prebuild/DLDT/source/bin/intel64"
[clDNN] Output lib directory:
[clDNN]  - "E:/Lib_prebuild/DLDT/source/bin/intel64"
[clDNN] Architecture:
[clDNN]  - target: Windows64    (detected: Windows64)
[clDNN]
[clDNN]
[clDNN] Advanced:
[clDNN]  - ICD version used to build:         6.3
[clDNN]
[clDNN]  - Include/Build cldnn core:          ON
[clDNN]  - Include/Build kernel selector:     ON
[clDNN]  - Include/Build tests:               OFF
[clDNN]  - Include/Build core internal tests: OFF
[clDNN]  - Include/Build tutorial:            OFF
[clDNN]
[clDNN]  - Run tests:                     OFF
[clDNN]  - Run core internal tests:       OFF
[clDNN]
[clDNN]  - Use static C++ Runtime:        OFF
[clDNN]  - Allow unsafe size opts:        ON
[clDNN]  - CMake debug trace:             OFF
[clDNN]
[clDNN]
[clDNN] ICD:
[clDNN]  - Root:            E:/Lib_prebuild/DLDT/source/inference-engine/thirdparty/clDNN/common/intel_ocl_icd/6.3
[clDNN]     + Headers:      E:/Lib_prebuild/DLDT/source/inference-engine/thirdparty/clDNN/common/intel_ocl_icd/6.3/windows/include
[clDNN]     + Static libs:  E:/Lib_prebuild/DLDT/source/inference-engine/thirdparty/clDNN/common/intel_ocl_icd/6.3/windows/Release/lib/x64
[clDNN]     + Shared libs:  E:/Lib_prebuild/DLDT/source/inference-engine/thirdparty/clDNN/common/intel_ocl_icd/6.3/windows/Release/bin/x64
[clDNN]     + Libs to link: E:/Lib_prebuild/DLDT/source/inference-engine/thirdparty/clDNN/common/intel_ocl_icd/6.3/windows/Release/lib/x64/OpenCL.lib
[clDNN] =============================================================================
[clDNN] Selected capabilities: public
CMake Error at inference-engine/thirdparty/pugixml/CMakeLists.txt:80 (list):
  list sub-command REMOVE_ITEM requires two or more arguments.

libGNA::API INTERFACE_LIBRARY : E:/Lib_prebuild/DLDT/source/inference-engine/temp/gna_<IP_ADDRESS>1/include
libGNA::KERNEL SHARED_LIBRARY : E:/Lib_prebuild/DLDT/source/inference-engine/temp/gna_<IP_ADDRESS>1/win64/x64/gna.lib
The name ngraph::ngraph is an ALIAS for ngraph. It will be exported to the InferenceEngineDeveloperPackage with the original name.
Configuring incomplete, errors occurred!
See also "E:/Lib_prebuild/DLDT/prebuild_x64/inference-engine/CMakeFiles/CMakeOutput.log".
See also "E:/Lib_prebuild/DLDT/prebuild_x64/inference-engine/CMakeFiles/CMakeError.log".

CMakeFiles.zip

Hi @Bleach665
I did a quick test on my Windows 10 with CMake 3.14.4, Microsoft Visual Studio 2019 and Python 3.7.3.  I did not see the same issue you are seeing, I used the build instructions for Windows.
git clone https://github.com/opencv/dldt
cd dldt
git submodule update --init --recursive
mkdir build
cmake -G "Visual Studio 16 2019" -A x64 -DCMAKE_BUILD_TYPE=Release ..
Which processor are you using?
Regards,
Jesus

Hi @jgespino. Thanks for the test. I used PC with i5 750 CPU.
Now I repeated this test on a clean virtual machine (host x5660 Xeon CPU) and the problem disappeared, so I close this issue.


Regression 4.4.2 - 4.5.0

Summary
Webdrivers cannot find chrome/firefox (old ones), from regular selenium docker image
Debug Info
Use webdrivers 4.4.2 and alpha 6 official docker images
Expected Behavior
Stuff works
Actual Behavior
For chrome:
Failed to find Chrome binary. (Webdrivers::BrowserNotFound)

For firefox:
Expected browser binary location, but unable to find binary in default location, no 'moz:firefoxOptions.binary' capability provided, and no binary flag set on the command line (Selenium::WebDriver::Error::SessionNotCreatedError)

Something has regressed, but it might be a combination of webdrivers/capybara e.t.c. I cannot say for certain. I'll close now and re-create with more info as/when I get time to fully triage.

Something has regressed, but it might be a combination of webdrivers/capybara e.t.c. I cannot say for certain. I'll close now and re-create with more info as/when I get time to fully triage.


Add my school to JetBrains

@hthanggg0221 I am sorry, the domain sis.hust.edu.vn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


PlantUML can't be embeded in mdita like mermaid diagram can

Issue #11 showed that mermaid diagram can be embedded in mdita. But the same couldn't be said for plantUML.
It's reproducible with this md file:
documents/src$ cat src/usecase/guide/testPlantUMLDiagram.md
# Test

Short Description here

First paragraph:

    <foreign outputclass="embed-plant-uml">@startuml
	Alice -> Bob: Authentication Request
	Bob --> Alice: Authentication Response

	Alice -> Bob: Another authentication Request
	Alice <-- Bob: Another authentication Response
    @enduml</foreign>

Run it through dita command (4.1) with -f pdf I got this:
documents/src$ dita -i smartlayer-overview.ditamap -f pdf -o output
[topic-reader] WARN: Unsupported HTML5 element 'foreign'
 [pipeline] WARNING: Supplied DOM declares a default namespace, but is not created as namespace-aware

Above file can be found at my repository's testPlantUMLDiagram.md. There is a branch for testing this.

After some try-and-error, I found a workaround: remove all empty lines within the <foreign> element.
This works
$ cat testPlantUMLDiagram.md
# Test

Short Description here

First paragraph:

    <foreign outputclass="embed-plant-uml">@startuml
	Alice -> Bob: Authentication Request
	Bob --> Alice: Authentication Response
	Alice -> Bob: Another authentication Request
	Alice <-- Bob: Another authentication Response
    @enduml</foreign>

Note that I still hope this issue being kept open until a md test-case is added to the samples directory, for helping newcomers to save the 2 hours I spent on investigating this.

@weiwu-zhang thanks for the feedback, if you want you can add a pull request to this project with extra samples.
In general this project does not control how the Markdown gets converted to DITA XML content.
If when publishing you set the "clean.temp" parameter to "no", after publishing you can look in the temporary files folder and see to what DITA XML topic the Markdown file was dynamically converted while publishing. This might help you debug more easily such problems in the future.

merged.


Fix ShardKeys property by using collection type instead of string.

fix #313

There is a failing test. I will investigate.

Thanks for reviewing @modios , merging now. 👍


Create domain job fails after 300 seconds although domain is created

Running  ./create-domain-job.sh –i create-domain-job-inputs.yaml terminates after 5 minutes with error
=========================================================
The create domain job is not showing status completed after waiting 300 seconds
Check the log output for errors
The domain will be created using the script /u01/weblogic/create-domain-script.sh
Initializing WebLogic Scripting Tool (WLST) ...
Welcome to WebLogic Server Administration Scripting Shell
Type help() for help on available commands
domain_path        : [/shared/domain/my_wls_domain]
domain_name        : [my_wls_domain]
admin_username     : [weblogic]
admin_port         : [7001]
cluster_name       : [cluster-1]
server_port        : [8001]
managed server name is managed-server1
managed server name is managed-server2
Domain Created
[ERROR] Exiting due to failure
=========================================================
No weblogic docker container is running then, but one is in stopped state. It's logfile says:
=========================================================
The domain will be created using the script /u01/weblogic/create-domain-script.sh
Initializing WebLogic Scripting Tool (WLST) ...
Welcome to WebLogic Server Administration Scripting Shell
Type help() for help on available commands
domain_path        : [/shared/domain/my_wls_domain]
domain_name        : [my_wls_domain]
admin_username     : [weblogic]
admin_port         : [7001]
cluster_name       : [cluster-1]
server_port        : [8001]
managed server name is managed-server1
managed server name is managed-server2
Domain Created
Domain Updated
Create boot.properties files for admin and managed servers
Done
Exiting WebLogic Scripting Tool.
Successfully Completed
=========================================================
So - what might be going wrong here?

Just to clarify, I believe you are saying that you've run the domain creation script and it appears to have created the domain but then exited with an error after creating the domain, but before it finished.  Is this correct?  Can you tell us what other resources were created in the target namespace?

Actually the create-domain-job.sh skript exited with error, saying:
"The create domain job is not showing status completed after waiting 300 seconds"
The logfile of the pod says:
"Exiting WebLogic Scripting Tool.
Successfully Completed"
But only AdminServer is completed, no Managed Server.
The resources in the target namespace for the domain (-n wlsdomain1) are:
Pods: domain-mydomain1-job-b4jbr
Jobs: domain-mydomain1-job
Config Maps: domain-mydomain1-scripts
Persistent Volume Claims: mydomain1-pv001-claim
3 Secrets: docker-store-secret, mydomain1-weblogic-credentials, default-token-mwbsk
I do have a docker container on one node, which is in stopped state, with name "k8s_domain-job_domain-mydomain1-job-b4jbr_wlsdomain1_de8e4135-18a7-11e8-9114-020017027e48_0". Its logfile says that the domain, including two managed servers have been created successfully.
Maybe a timing issue?!

I ran it again today. The create-domain-job.sh again fails after 5 minutes with:
"The create domain job is not showing status completed after waiting 300 seconds"
But this time I ran a "docker logs -f " on the container creating the domain. This container continues running even after the create-domain-job.sh failed and ~ 2 minutes later the log output shows:
"Exiting WebLogic Scripting Tool.
Successfully Completed"
After modifying the create script to iterate max=20, it finished successfully after 17 (!) iterations.
So it works now ...

FYI - As part of the pull request #143, I have increased the max iteration of checking the completion of  create domain jobs from 10 to 20.


Name this application!

We need a name for StoryRAMP products that isn't "StoryRAMP", for the following reasons:

It's too close to StoryMap (an ArcGIS product from Esri)
It's not bilingual and wouldn't translate well (Best I can come up with is "PCAR L'Histoire" which rhymes but still doesn't make sense).

Considerations:

It should go into the URL e.g. enviromental-maps.canada.ca/storyramp-histoirepcar
It might need to go on a logo
It will be referenced in countless documents, APM, etc.
We should keep RAMP (PCAR) in the name, if possible.

I forgot about RAMPTales, it actually translates more or less the same, but we'll need to decide if we'd rather be sued by Esri or Disney.

No clever names comes to mind. Some boring suggestions.

RAMP Journeys
RAMP Tour
RAMP Guidebook
RAMP Brochure
RAMP Pamphlet ( RAMPhlet ✔ )
RAMP 'Zine
RAMP Extra! (newspaper term)

Less boring that won't get approved

Scrolly RAMP
RAMP 'n' Scroll
RAMPers Digest
Rampy McRampface (credit to Mr. Wahl)
Tour de RAMP

More boring suggestions:
RAMP folio
RAMP book
folio and tour might be able to stay as folio and tour in translation.. if thats a bonus

Tales from the RAMP
RAMP It Up
RAMP Plot
RAMP Diegesis
RAMP StoryLine
RAMP Stamp

Random ideas:

RAMP Notes
RAMP Slides
RAMP Sheets
RAMP Pages
RAMP Chapters

If we choose RAMP Book I have an icon for us.

Few other ideas that came to mind:

RAMP Quest
RAMP Chronicle
RAMP Album
Art of RAMP
RAMP Journal
Scenes from RAMP

From the existing suggestions, I quite like RAMP Folio, RAMP Book, RAMP Stamp, and Tour de RAMP.

'RAMP Gazetteer' for the throwback.
Shortlist from the suggestions above (thanks everyone!), we will discuss in a future meeting:

RAMP Pamphlet ( RAMPhlet ✔ )
RAMP Pages
RAMPfolio
RAMP Album (or some variation)
RAMP StoryLine
RAMP Journals
Tour de RAMP (I know you were being funny with this but that's actually pretty good)


Feature/1064 refactor max length prop

Summary of the changes
Removed maxLength prop from text field and made character count display when the maxCharacters prop is used instead. Added hideCharCount prop to provide the option to visually hide the character count. Updated tests, including removing maxLength usage / replacing with maxCharacters.
The character count information for screen readers still remains when the visual character count is hidden. I felt that this would be a good idea to make it more obvious to screen reader users that their input is being limited.
I have also updated the character count screen reader information to read out how many characters are remaining as originally the aria-live region was just reading out "10 slash 25", for example; now it would say "15 characters remaining", which I felt was more user-friendly and also matches the GOV.UK behaviour.
Related issue
#1064
Checklist
General

[x] Changes to docs package checked and committed.
[x] All acceptance criteria reviewed and met.

Testing

[x] Relevant unit tests and visual regression tests added.
[x] Visual testing against Figma component specification completed.
[x] Playground stories in React Storybook up to date, with any prop changes and additions addressed.

Accessibility

[x] Accessibility Insights FastPass performed.
[x] A11y unit test added and yields no issues.
[x] A11y plug-in on Storybook yields no issues.
[x] Manual screen reader testing performed using NVDA and VoiceOver.
[x] Manual keyboard testing for keyboard controls and logical focus order.
[x] Correct roles used and ARIA attributes used correctly where required.
[x] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.

Resize/zoom behaviour

[x] Page can be zoomed to 400% with no loss of content.
[x] Screen magnifier used with no issues.
[x] Text resized to 200% with no loss of content.
[x] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.

System modes

[x] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[x] Windows High Contrast mode tested with no loss of content.
[x] System light and dark mode tested with no loss of content.
[x] Browser support tested (Chrome, Safari, Firefox and Edge).

Testing content extremes

[x] Min/max content examples tested with no loss of content or overflow.
[x] All prop combinations work without issue.
[x] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.
[x] Controlled and uncontrolled input components tested.

Should the validation\max length check be triggered on first load. For example if you have a value set that exceeds the maxCharacters?
I have a long value set which exceeds the limit.
Initially this displays as follows:

After pressing a key in the input field, the warning is applied:

The old behaviour appears to be to trigger the error on load:

there no longer seems to be a visual test (with baseline image) where the maxCharacters is exceeded. I think it would be good to have one, just to ensure styling doesn't change

Should the validation\max length check be triggered on first load. For example if you have a value set that exceeds the maxCharacters?
I have a long value set which exceeds the limit.
Initially this displays as follows:
After pressing a key in the input field, the warning is applied:
The old behaviour appears to be to trigger the error on load:

With the changes in this PR, it is not possible to get the "maximum length exceeded" error when typing in characters now because it prevents any more input once the maxCharacters amount has been reached.
I have pushed up some changes now which just cut off the value based on the maxCharacters. So now, if the value prop is set on page load, or if the prop is updated, the input in the text field cannot be over the maxCharacters amount. I felt that this would be the best solution as it essentially matches the behaviour you were to get if you actually typing in the text field (including the fact that if you try to type more characters, the warning will then appear). Hopefully this is ok :)

there no longer seems to be a visual test (with baseline image) where the maxCharacters is exceeded. I think it would be good to have one, just to ensure styling doesn't change

It is no longer possible to go over the maximum allowed number of characters so this error cannot occur any more which is why the image / test has been removed completely

Should the validation\max length check be triggered on first load. For example if you have a value set that exceeds the maxCharacters?
I have a long value set which exceeds the limit.
Initially this displays as follows:
After pressing a key in the input field, the warning is applied:
The old behaviour appears to be to trigger the error on load:

After the changes in this PR, it is not possible to get the "maximum length exceeded" error when typing in characters now because it prevents any more input once the maxCharacters amount has been reached.
I have pushed up some changes now which just cut off the value based on the maxCharacters. So now, even if the value prop is set on page load, or if the prop is updated, the value displayed in the text field cannot be over the maxCharacters amount. I felt that this would be the best solution as it essentially matches the behaviour you were to get if you actually typing in the text field (including the fact that if you try to type more characters after the prop has been set, the warning will then appear). Hopefully this is ok :)

The only slight issue I can see is that, if the initial value gets truncated, it will trigger an icChange event with the new shortened value. Also, if the data was loaded into a form and then gets submitted, the new shorter value would get submitted and, without some validation being done, potentially overwrite existing data on the server side. I think it would be good to just make it clearer (in the maxCharacters prop description?) that the value will get truncated if it exceeds the limit. Just so users are aware and can ensure they validate values

Should the validation\max length check be triggered on first load. For example if you have a value set that exceeds the maxCharacters?
I have a long value set which exceeds the limit.
Initially this displays as follows:
After pressing a key in the input field, the warning is applied:
The old behaviour appears to be to trigger the error on load:

After the changes in this PR, it is not possible to get the "maximum length exceeded" error when typing in characters now because it prevents any more input once the maxCharacters amount has been reached.
I have pushed up some changes now which just cut off the value based on the maxCharacters. So now, even if the value prop is set on page load, or if the prop is updated, the value displayed in the text field cannot be over the maxCharacters amount. I felt that this would be the best solution as it essentially matches the behaviour you were to get if you actually typing in the text field (including the fact that if you try to type more characters after the prop has been set, the warning will then appear). Hopefully this is ok :)

The only slight issue I can see is that, if the initial value gets truncated, it will trigger an icChange event with the new shortened value. Also, if the data was loaded into a form and then gets submitted, the new shorter value would get submitted and, without some validation being done, potentially overwrite existing data on the server side. I think it would be good to just make it clearer (in the maxCharacters prop description?) that the value will get truncated if it exceeds the limit. Just so users are aware and can ensure they validate values

Added a note to the prop description 👍


Incorrect content displayed

Content issues have been reported when displaying images.
Ex. when face cards first show for a player they flash incorrect cards then change to the correct cards.
Investigation underway to find the cause.

No reports of this since d97a73af8f5acc0953639447646768f3ce3f1859, Holdero repo would be more appropriate.
Closing here.


Loki does not properly use ingress

Bug Description
See, for instance, https://github.com/canonical/loki-k8s-operator/blob/adf626b53638edd831dce03fc78aa38ad8d9a236/lib/charms/loki_k8s/v0/loki_push_api.py#L1215
To Reproduce

Environment

Relevant log output
-

Additional context
No response

A side note here @simskij :
In Prometheus, Loki and Grafana we are using traefik-k8s lib v0.
The last (and only) traefik lib version published is v1
Should we update this lib in our charms prior to reach RC?

A side note here @simskij :
In Prometheus, Loki and Grafana we are using traefik-k8s lib v0.
The last (and only) traefik lib version published is v1
Should we update this lib in our charms prior to reach RC?

I'd like @PietroPasotti to weigh in on this, but yes, that seems like it would make sense.

nearly done on
https://github.com/canonical/prometheus-k8s-operator/pull/330
and the same work will likely apply to loki (should be 10m of effort)
that pr is stuck until I get unstuck getting the CI in https://github.com/canonical/traefik-k8s-operator/pull/71#pullrequestreview-1045029605 to work, I am having some races with deploy + add relation but haven't had the time to fix them yet this week

A side note here @simskij :
In Prometheus, Loki and Grafana we are using traefik-k8s lib v0.
The last (and only) traefik lib version published is v1
Should we update this lib in our charms prior to reach RC?

I'd like @PietroPasotti to weigh in on this, but yes, that seems like it would make sense.

V0 is technically published, I removed it from the master branch but it still lives on in some separate maintenance branch (or it should)


fix(zeebe-plugin): correct misleading contact point hint

Ensures we show a placeholder mirroring the expected format.

(Previously we showed <IP_ADDRESS>:26500 which is completely missleading.

develop is fine.


refactor: change json file name

change json file emoji-list.json -> emoji.json

This branch has conflicts that must be resolved
I couldn't merge due to a problem, and I copied the file and updated it.
Thank you.


(Puppetfile) bump lsst/* mods to allow puppet/systemd 6.x

This should be ready to go as soon as lsst/ccs_software 2.2.0 is showing up in the forge API.


Make files conform to a package structure

This is important, I'm told.

More-or-less fixed in 728597d456802d7a42608d839ea8f589fa7ba99c


Fix chcp parsing for the current codepage

Fix #874
Fix #884
Fix #885
Related #886
cc @gh4w
Patch is adapted from stackoverflow answer, https://stackoverflow.com/a/10535753

Thanks. Maybe I should port this to fzf?

Yes.

The parsing doesn't take into account codepage values mixed with non-numeric characters. Need sed or some cmd.exe alternative to strip out non-numeric characters after the for-loops.


Allow the utility_meter to net meter rather than only allow increases.

Description:
This PR allows utility_meter components to be net metered.  My personal use-case is monitoring net power consumption because of solar feeding back to the grid.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8631
Example entry for configuration.yaml (if applicable):
utility_meter:
  daily_energy:
    source: sensor.solar_raw_lifetime_net_energy
    cycle: daily
    rollover: False
  monthly_energy:
    source: sensor.solar_raw_lifetime_net_energy
    cycle: monthly
    rollover: False

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

[ ] Tests have been added to verify that the new code works.

I've been thinking maybe the name should be more obvious to users:
"net_comsumption" ?

I agree that net_consumption would be a better name.  I'll change that along with the other comments.

tks! 🥇


If my data is small,such as 6000 images,is it difficult to train the model?

I got my loss at:
2018-12-05 09:12:04,200 Epoch[9] Train-RPNAcc=0.929830
2018-12-05 09:12:04,200 Epoch[9] Train-RPNLogLoss=0.219037
2018-12-05 09:12:04,200 Epoch[9] Train-RPNL1Loss=0.138239
2018-12-05 09:12:04,200 Epoch[9] Train-FCISAcc=0.954637
2018-12-05 09:12:04,200 Epoch[9] Train-FCISAccFG=0.215569
2018-12-05 09:12:04,200 Epoch[9] Train-FCISLogLoss=0.174942
2018-12-05 09:12:04,200 Epoch[9] Train-FCISL1Loss=0.012621
2018-12-05 09:12:04,201 Epoch[9] Train-FCISMaskLoss=0.374442
2018-12-05 09:12:04,201 Epoch[9] Time cost=4398.860
And its mAP is all 0.How it happened? Anyone meet this problem?

If you use your own data,please be sure your code to get the data is right.

I got my loss at:
2018-12-05 09:12:04,200 Epoch[9] Train-RPNAcc=0.929830
2018-12-05 09:12:04,200 Epoch[9] Train-RPNLogLoss=0.219037
2018-12-05 09:12:04,200 Epoch[9] Train-RPNL1Loss=0.138239
2018-12-05 09:12:04,200 Epoch[9] Train-FCISAcc=0.954637
2018-12-05 09:12:04,200 Epoch[9] Train-FCISAccFG=0.215569
2018-12-05 09:12:04,200 Epoch[9] Train-FCISLogLoss=0.174942
2018-12-05 09:12:04,200 Epoch[9] Train-FCISL1Loss=0.012621
2018-12-05 09:12:04,201 Epoch[9] Train-FCISMaskLoss=0.374442
2018-12-05 09:12:04,201 Epoch[9] Time cost=4398.860
And its mAP is all 0.How it happened? Anyone meet this problem?

Hello, I also trained my own data set, but map=0, how do you solve it


TYPO?: COLUMN_TYPE in MysqlDriver

I was trying to generate Models for my Mysql Database when i encountered an error:
UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'substring' of undefined
    at XXX\node_modules\typeorm-model-generator\dist\src\drivers\MysqlDriver.js:210:68

Then i found out that everywhere is used the uppercase ATTRIBUTE (eg: CHARACTER_MAXIMUM_LENGTH), except for column_type:
dist/src/drivers/MysqlDriver.js:206-211
case "enum":
         colInfo.ts_type = "string";
         colInfo.sql_type = "enum";
         console.log(resp);
         colInfo.enumOptions = resp.column_type.substring(5, resp.column_type.length - 1).replace(/\'/gi, '"');
         break;

Changing the attribute column_type to uppercase resolves the issue (for me).

Please try updating typeorm-model-generator. I don't see any lowercase column_type in the current code. Line number from the error doesn't match either.

Ok strange, appearently i had an old file in node_modules/node/bin called typeorm-model-generator(.cmd|.exe) that somehow blocked the updating of the package.
Solution
Deleting all the files that are named in the above scheme. (I also used --force when installing)
Issue resolved, thanks.


TabContent / TabContext / TabPane - shared state for all instances

I'm just learning Svelte and Sveltestrap -- thanks for a great lib on a great tool, has made me productive and eased the migration to adoption w bootstrap (also learning..).
In my UI naivety (I'm no UX designer), I wanted to put multiple tab components (whether tabs or pills, horz or vert) on the same page. However, the tab state for activeTabId is held in a shared writable store. Is it possible to adopt similar?
I've built an alternative in the meantime (still using Nav, NavItem, NavLink) - using a container element so that getContext/setContext can be used. Happy to donate is useful (I'm probably losing a bunch of flexibility, since I'm fixing the nav UI rather that two named slots, say). It has a reduced footprint as the nav items are passed as lightweight array.

Hi @kenhorn , ah you are correct, that is wrong behavior in sveltestrap.  We should be able to have multiple tab panes on same page.  We'll look at this to correct soon

Hi @kenhorn , this should be corrected by #213, and was released in<EMAIL_ADDRESS>Please confirm and can close the issue.

That fixes it -- though I'm not clear on the correct tab example -- I've
put my test below.   Also, have added my own tab impl, which is similar but
compact (but not fully bootstrap compliant etc).
Thanks for the fast fix and release!

activeTabId1 = "x"}>X1

activeTabId1 = "y"}>Y1

x1x1

y1y1

activeTabId2 = "x"}>X2

activeTabId2 = "y"}>Y2

x2x2

y2y2

Example

AA

BB

TabNav

{#each items as item}

activeTabId = (item.id || item.label)}>{item.label}

{/each}

TabPanel

On Tue, Dec 22, 2020 at 4:13 PM bestguy<EMAIL_ADDRESS>wrote:

Hi @kenhorn https://github.com/kenhorn , this should be corrected by
#213 https://github.com/bestguy/sveltestrap/pull/213, and was released
in<EMAIL_ADDRESS>Please confirm and can close the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bestguy/sveltestrap/issues/212#issuecomment-749625999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOPSOXX7JWKSECMCCHLQMDSWDAS3ANCNFSM4VBCJPWQ
.

Thanks, yeah that's a good idea, I think these components should be simplified for this usage.  We are prepping v5 and will look at adding something like this.

Hi @kenhorn ,
Just FYI that TabContent and TabPane were reworked in Sveltestrap v5 to make the usage a lot easier.
Would be helpful if you'd like to take a look and comment: https://sveltestrap.js.org/?path=/story/components--tabs


Macro to support Actor<Box<dyn Trait>> based actor creation

See issue #1

Implemented in version 0.1.2


ci: run github action on push

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@cfe55ec). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main        #3   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         2
  Lines           ?        62
  Branches        ?         0
========================================
  Hits            ?        62
  Misses          ?         0
  Partials        ?         0

Continue to review full report at Codecov.

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


Controls for ailerons and elevator not working in "Landing Practice" mode

Platform/OS: Linux (Debian 11.x)
YSFlight version: 20181124 - AFFECTED
YSCE version: master - AFFECTED

Expected behavior
Control for ailerons and elevator should work as expected in "Landing Practice" mode.
Actual behavior
In "Landing Practice" mode controls for ailerons and elevator are NOT working (stuck?) — when press assigned shortcuts there are NO any effect.
Other shortcuts (including rudder) seems like work as expected.
This bug detected since YSFlight 2017's test versions under Linux (has not tested Windows & Mac).
Screenshots

I don't believe we have the resources to fix this just yet, but we'll take a look at it and keep it in mind to patch when we can.


Integrate with Rivertuner SS

Have an option in the main menu for launching Rivertuner Statistics Server, handy for those who have frame limiters on RSS.

Add a more general option. A setting for a custom Main Menu item which the user can set to any application.


fix(argo): Fix latest k8s provider

Description
Fixes problems with k8s provider.

Type of change

[x] A bug fix (PR prefix fix)
[ ] A new feature (PR prefix feat)
[ ] A code change that neither fixes a bug nor adds a feature (PR prefix refactor)
[ ] Adding missing tests or correcting existing tests (PR prefix test)
[ ] Changes that do not affect the meaning of the code like white-spaces, formatting, missing semi-colons, etc. (PR prefix style)
[ ] Changes to our CI configuration files and scripts (PR prefix ci)
[ ] Documentation only changes (PR prefix docs)

How Has This Been Tested?

Closing in favor of https://github.com/lablabs/terraform-aws-eks-grafana/pull/5


Showcase Submission for noc/3-angles/4-polar-coordinates

Thank you panna for your contribution! A member of the Coding Train team will review it shortly.

Birthday Cake
panna

so cute!


mobile view nav bar bug

Is this behavior expected ? @chirag828049  @rajivharlalka

No this is a bug.
For mobile view it should look like : https://rajivharlalka.github.io/
will you take up this issue @sidkhuntia ?

I can try @chirag828049

Assigned you.

#35


Support for addresses with staking keys and wallet lookups.

This is the current "Staking keys support" pr for plutip. related Issues: #103 #104

Add stake keys to BpiWallet.
Generate stake keys in addWallets
Set pcOwnStakePubKeyHash for BPI.
Pass WalletLookups to contracts instead of list of PubKeyHash's.
wallets initialized with WalletTag which specifies a) how wallet is generated b) how wallet is accessed in test case

Contains changes introducing staking keys support, but together with changes to the wallet initialization and to the interface of providing user with keys in the contract.
It aims to provide lookups like so:
assertExecution
  "Values asserted in correct order with withContract"
  (initAda (EnterpriseTag "w0") [100]
    <> initAda (BaseTag "w1") [200]
  )
  (withContract $ \wl -> do
    EnterpriseInfo w1pkh <- lookupWallet wl (EnterpriseTag "w0")
    BaseInfo w2pkh <- lookupWallet wl (BaseTag "w1")
    _ <- payTo w1pkh (toInteger payTo1Amt)
    _ <- waitNSlots 2
    payTo w2pkh (toInteger payTo2Amt))
  [shouldSucceed]

EDIT: nevermind

I removed substituteTags from withConfiguredCluster. Before it basically acted as identity, only adding the type info and making things compile, now can get rid of.

will note here, as probably won't be able to work on this, that this change enables #121 with no more api changes. Simmilarly extra-signing-keys could be initialized in TestWallets (tho no need to remove them from config as second option).


Using SVProgressHUD in both extension and app

Guys,
Sorry just need some clarification, after adding SV_APP_EXTENSIONS=1 to my pod's SVProgressHUD, does this make my app not being able to use SVProgressHUD?
Cause for some reason .show() doesn't do anything now.

TestSV.zip

TestSV.zip

Same behavior here. Setting the extensionView does not seam to do anything either.

This is old. Does anyone know if this is an intended behaviour or? I reckon
it's not a rare use case that I want to use SV for both my app and extension
On Wed, 25 Jan 2017 at 6:18 AM, Dan Niemeyer<EMAIL_ADDRESS>wrote:

Same behavior here. Setting the extensionView does not seam to do anything
either.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SVProgressHUD/SVProgressHUD/issues/643#issuecomment-274957436,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMja3ez_YhtagQcmtkNuq-ay23QY1N7ks5rVniqgaJpZM4JIANK
.

I just ran into this issues myself. Unfortunately, it is a serious limitation of CocoaPods, rather than an SVProgressHUD bug. Ideally, you should be able to bundle 2 versions of the framework - one with SV_APP_EXTENSIONS=1 to use in the extension, and one without. CocoaPods doesn't currently allow that :(
Another solution to this would be figuring out how to make SVProgressHUD always work without depending on API's not available in the extensions. I just had a brief look at the source and, while not impossible, it will require significant changes.


Better URL Checking

At present, the URLs submitted in the form are checked to make sure they have at least a http:// or https:// at the start of the URL.
I'd also like to make sure it ends with a .com (or similar prefix)
I'd also like to make sure that the emails are compliant (have a start, an @ and an end)
Not sure how to tackle these ones yet, particularly the email situation.

Actually it seems I don't need email checking.
The HTML5 input type of "Email" seems to do the checking for me, complete with popup messages asking the users to write in a properly formed email.

Are you looking to validate any possible tld?   I mean there must surely be a nice way of doing as sign up
forms seem to check you've entered something valid all the time.
On 13 Jan 2015 22:07, "drfu"<EMAIL_ADDRESS>wrote:

Are you looking to validate any possible tld? There are a large number
besides the standard com, org, net, mil, edu, gov set:
http://www.iana.org/domains/root/db
—
Reply to this email directly or view it on GitHub
https://github.com/Charlotteis/guestbook/issues/13#issuecomment-69830075
.

The iana page is the authoritative list, but they also have a text version: http://data.iana.org/TLD/tlds-alpha-by-domain.txt.  As a bonus it has the punycode versions of i18n domains as well.  Very simple to retrieve and parse, you can probably cache a copy nightly or monthly and know that you can cover all tlds.


gh-pages branch?

Could you make a GH Pages branch? This would allow the CSS files to be added using GitHub as a "CDN" for quickly wireframing sites, eg:
http://skeleton.github.com/

Great suggestions, but in an effort to keep Skeleton simple I'm just going to keep with the remote docs on www.getskeleton.com and the source here on Github.


With NetCore workload, AspNetCoreWebApp_NoAuth_1 : open with VS, prompts to install Web workload

Steps to reproduce
Using VS 2017 RC.3 (Build d15rel 26109.02)
Install .NET Core workload
git clone https://github.com/balaTest/ProjectKTestProjects
Open solution under Dev14_K101/Tooling202/AspNetCoreWebApp_NoAuth_1
Actual behavior
Prompts to install web workload

Expected behavior
Project should open and migrate without needing the web workload

@basoundr ported to 367212. Please review and fill out the "Ask Mode" tab and send to Matt to approve

Fixed in the VS Side


Implement Voronoi Fracture?

It would be nice for godot to support Voronoi fracture.
I know bullet physics does.

CC @AndreaCatania @reduz

This is one of the thing that I've to implement, but right now I'm not able to tell you when I can start to work on it. I let you know :grinning:

I wrote a Godot plugin using the C++ interface for Qhull (http://www.qhull.org/). It works for 2D rectangles and 3D boxes returning a number of defined fragments.
Left corner: Voronoi cells on with random colors and alpha overlaid on sprite.
Main scene: A body which is replaced with Voronoi fragments upon impact.

What would be needed to integrate it into the Godot engine?
Usage
2D
voro = Voronoi.new()
voro.voronoi2d(100)
Calculate 100 2d Voronoi points inside a square
voro.get_vertexes()
Returns a dictionary where the vertex index is the key and the value is a Vector2
voro.get_faces()
Returns an array of arrays where the inner array contains all vertex indexes for a face
3D
voro.voronoi3d(25)
Calculate 25 2D Voronoi points inside a cube
voro.get_faces()
Returns an array of arrays which contain the triangles to generate the mesh for the Voronoi fragment.
Performance
2D
i5-2500, Windows 10, compiled with standard settings

points
msec

10
6

25
11

50
22

250
104

1000
419

3D

points
msec

10
24

25
68

50
147

250
838

1000
3732

Awesome Job.
Would be useful try it in order to understand it and then discuss how to integrate it inside the editor.
Do you think is it possible test it?

@AndreaCatania an example project is attached
voro_test.zip
If you check out the voronoi branch from my Godot fork (https://github.com/Ashafix/godot/tree/voronoi), the attached project should work (tested on Windows only).
The teleport effect is due to the fact that the edges of the box are of length 2 while the Voronoi box has a length of 1. It obviously needs some more work.

The teleport effect is due to the fact that the edges of the box are of length 2 while the Voronoi box has a length of 1. It obviously needs some more work.

To be merged we require that you implement it properly. Let us know when it's done

Hello there is any chance this will be implemented in Godot 3.2? Also with Lloyd’s Relaxation? It is really useful for procedural generation. Now we have OpenSimplexNoise in Godot 3.1 but alone is not enough.

I would love for Voronoi and Lloyd's Relaxation to be native to the engine! They're crucial to an upcoming project of mine and implementing them seems like a huge pain. If I get a good grasp of the algorithms I'll start working on adding them to the engine! I only personally care about the 2d implementation, but if it seems easy to extend to higher dimensions I can give that a go as well

@CyanBlob I implemented qHull in Godot but didn't find the time to really finish it. Feel free to check my Godot and send me an email if you want to collaborate on it.

@Ashafix how is the feature going ? I would love to use it in my VR game :)

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Add unit test for pkg/fuse/

This brings the coverage of pkg/fuse to 80%

Codecov Report

Merging #1123 (d19b1b5) into main (94ab4c8) will increase coverage by 0.83%.
The diff coverage is 12.50%.

@@            Coverage Diff             @@
##             main    #1123      +/-   ##
==========================================
+ Coverage   56.43%   57.26%   +0.83%
==========================================
  Files         124      124
  Lines       20503    20503
==========================================
+ Hits        11571    11742     +171
+ Misses       7533     7360     -173
- Partials     1399     1401       +2

Impacted Files
Coverage Δ

pkg/fuse/fuse.go
76.14% <ø> (+45.91%)
:arrow_up:

pkg/meta/base.go
75.65% <0.00%> (ø)

pkg/vfs/vfs.go
77.58% <33.33%> (-0.25%)
:arrow_down:

pkg/meta/redis.go
71.16% <0.00%> (+0.01%)
:arrow_up:

pkg/meta/tkv.go
76.14% <0.00%> (+0.71%)
:arrow_up:

pkg/meta/sql.go
68.34% <0.00%> (+0.73%)
:arrow_up:

pkg/vfs/handle.go
86.55% <0.00%> (+0.84%)
:arrow_up:

pkg/fuse/context.go
56.75% <0.00%> (+5.40%)
:arrow_up:

Continue to review full report at Codecov.

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


Filter/Hide icon hidden behind scrollbar

When repository contains a lot of tags and Show Tags as Tree is enabled, then it's not possible to click Filter/Hide icon as it's hidden behind the scroll bar:

With Show Tags as Tree disabled icon is moved slightly to the left and can be clicked:

I would suggest to move it in both scenarios, so it doesn't overlap with the scroll bar.

Fixed. You can download the latest CI build from Github Action.

Thanks!


[⚙] Copy challenges.yml over to www-project-juice-shop after a release

:⚙: Automation request
Description
The project website now uses a (manually committed) copy of the challenges.yml to render a table with all challenges grouped by their category:

(Source: https://owasp.org/www-project-juice-shop/#div-challenges)
It would be great to automate this copy+paste process whenever a new version is released.
Solution ideas
Implement as an after_deploy step for tag (or master branch?) builds in the travis.yml.
Possible alternatives
Implement as a GitHub action triggered by changes of challenges.yml on master branch or upon tag push.

This table is now also generated from the challenges.yml instead of a separate (and somewhat redundant) data file listing the tutorialized challenges:

Would it perhaps be better if instead of opening a pull request on when a commit is pushed with the release tag, we get the metrics endpoint to also expose the list of all challenges and then the website could fetch it from one of the permanent instances like https://demo.owasp-juice.shop/#/ or https://juice-shop.herokuapp.com/#/ ?

I really don't want to dynamically load any data from the inherently insecure Juice Shop into the official website... 😅
But opening a PR instead of just pushing a commit would be an option, but it's not really needed as a manual intervention step. The Juicy Bot user (which is also doing the linting) should probably do just a commit.

If there's a way to load the challenges.yml from the Juice Shop repo as a data source in the website repo, then no commits etc. would be needed. But that's still kind of a "remote file inclusion" and afaik it is not supported by Jekyll.

oh ok, I'll go ahead with the workflow then


Allow dependency checker to run on more subprojects

Reviewing cheatsheet
Before merging the PR, comments starting with

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

@bot-gradle test this please.

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

PullRequestFeedback build


Error when searching for an address (reading 'sent' of undefined)

open https://wallet.stg.oasisprotocol.org/account/oasis1qqekv2ymgzmd8j2s2u7g0hhc7e77e654kvwqtjwm/stake
switch to Testnet
search for oasis1qz2tg4hsatlxfaf8yut9gxgv8990ujaz4sldgmzx
error Cannot read properties of undefined (reading 'sent')

Here's the stacktrace from the console log, when run from chrome in incognito mode (no extensions loaded)
2.68ffbd83.chunk.js:2 TypeError: Cannot read properties of undefined (reading 'sent')
at nc (main.17701355.chunk.js:1)
at oi (2.68ffbd83.chunk.js:2)
at Gu (2.68ffbd83.chunk.js:2)
at Pc (2.68ffbd83.chunk.js:2)
at Ec (2.68ffbd83.chunk.js:2)
at kc (2.68ffbd83.chunk.js:2)
at yc (2.68ffbd83.chunk.js:2)
at 2.68ffbd83.chunk.js:2
at t.unstable_runWithPriority (2.68ffbd83.chunk.js:2)
at Fo (2.68ffbd83.chunk.js:2)
uu @ 2.68ffbd83.chunk.js:2

Was looking at it and this was resolved by handling unrecognized transaction type  #412

Oh yeah, this is now rendering successfully

Let's close this out. We can open another issue if there's desire to parse these voting transactions for real.


[FLINK-13057][state] Correct comments in ListState class

What is the purpose of the change
This pull request fixes the comments in ListState.
ListState can be a keyed state or an operator state, but the comment in ListState said it can only be a keyed state
Brief change log

Correct comments in ListState class, i.e. ListState can be a keyed state or an operator state.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

CI report for commit a0cd89a69dbce15adc26d5dc28ef49640ab8c8c8: FAILURE Travis

CI report for commit a0cd89a: FAILURE Travis

@zentol Thanks a lot for the review.
As for the FAILURE, it seems some dependencies could not be resolved. Maybe it is not related to this PR? or something I have missed?
I have updated my PR and rebased to the latest master. I will check travis and see whether the problem still exists. Thanks a lot.
Best,
Hequn

Good fix, thank you!
Merging this...

@StephanEwen Thank you very much. :-)


How to generate UDP  models from a video?

How to generate UDP models from a video?

First of all, UDP is not a model. We introduced UDP in our paper, and the definition is given in 3.3.
Regarding the generation of UDP, we recommend two methods:

Directly using 3D software. Since this requires 3D knowledge, we provide two samples, one with short hair and one with a double ponytail.
As for the details: Specifically, we use MikuMikuDance to generate UDP, a software for 3D animation which needs a 3D model and an action sequence as input. Although we haven't provided an official method for this translation, there is a version written by a community author @KurisuMakise004. We have not tried it yet, but provide it for your reference: https://github.com/KurisuMakise004/MMD2UDP
Use the UDP detector. Although it is not mentioned in detail in the paper, we provide the code and weights. It can accept anime character images in png format and generate UDP from the images. For details, please refer to the parser test_pose_use_parser_udp in train.py.

Coming back to your question, if you need to generate UDP from video (although we deprecate that way), you can use the second method above. First split the video into pngs, then separate the foreground and background for each frame, take the foreground characters, and put these pngs into the same folder.
We recommend that you use this parameter when running train.py:
python -m torch.distributed.launch \
--nproc_per_node=1 train.py --mode=test \
--world_size=1 --dataloaders=2 \
--test_input_poses_images={dir_to_poses} \ # input the directory where you put the foreground pngs
--test_input_person_images={dir_to_character_sheet} \
--test_output_dir={dir_to_save_result} \
--test_checkpoint_dir={dir_to_weights} \
--test_pose_use_parser_udp=True

Hope this reply can help you and thank you for this issue!

Thank you very much, and I will try the second method!  ♪(･ω･)ﾉ

First of all, UDP is not a model. We introduced UDP in our paper, and the definition is given in 3.3. Regarding the generation of UDP, we recommend two methods:

Directly using 3D software. Since this requires 3D knowledge, we provide two samples, one with short hair and one with a double ponytail.
As for the details: Specifically, we use MikuMikuDance to generate UDP, a software for 3D animation which needs a 3D model and an action sequence as input. Although we haven't provided an official method for this translation, there is a version written by a community author @KurisuMakise004. We have not tried it yet, but provide it for your reference: https://github.com/KurisuMakise004/MMD2UDP
Use the UDP detector. Although it is not detailed in the paper, we provide the code and weights. It can accept anime character images in png format and generate UDP from the images. For details, please refer to the parser test_pose_use_parser_udp in train.py.

Coming back to your question, if you need to generate UDP from video (although we deprecate that way), you can use the second method above. First, split the video into pngs, then separate the foreground and background for each frame, take the foreground characters, and put these pngs into the same folder. We recommend that you use these options when running train.py:
python -m torch.distributed.launch \
--nproc_per_node=1 train.py --mode=test \
--world_size=1 --dataloaders=2 \
--test_input_poses_images={dir_to_poses} \ # input the directory where you put the foreground pngs
--test_input_person_images={dir_to_character_sheet} \
--test_output_dir={dir_to_save_result} \
--test_checkpoint_dir={dir_to_weights} \
--test_pose_use_parser_udp=True

I hope this reply can help you, and thank you for this issue!

Hi~When I run this script:
python3 -m torch.distributed.launch \
--nproc_per_node=1 train.py --mode=test \
--world_size=1 --dataloaders=2 \
--test_input_poses_images=./character_sheet/ \
--test_output_udp=True \
--test_checkpoint_dir=./weights/ \
--test_output_dir=./results_udp/ \
--test_pose_use_parser_udp=True

Another issue occurs:
batch_size: 4                                                                                [32/1197]
[['./character_sheet/0.png', './character_sheet/0.png', './character_sheet/1.png', './character_sheet/2.png', './character_sheet/3.png'], ['./character_sheet/1.png', './character_sheet/0.png', './character_sheet/1.pn
g', './character_sheet/2.png', './character_sheet/3.png'], ['./character_sheet/2.png', './character_sheet/0.png', './character_sheet/1.png', './character_sheet/2.png', './character_sheet/3.png'], ['./character_sheet/
3.png', './character_sheet/0.png', './character_sheet/1.png', './character_sheet/2.png', './character_sheet/3.png']]
---building models
---test images:  4
  0%|                                                                         | 0/4 [00:00<?, ?it/s]Traceback (most recent call last):
  File "train.py", line 203, in <module>
    test()
  File "train.py", line 72, in test
    infer(args, conrmodel, image_names_list)
  File "train.py", line 109, in infer
    pred = humanflowmodel.model_step(data, training=False)
  File "/data/pengjun/motion/conr.py", line 167, in model_step
    pred = self.shader_pose_encoder_forward(data, pred)
  File "/data/pengjun/motion/conr.py", line 288, in shader_pose_encoder_forward
    _, features = self.target_pose_encoder(
  File "/data/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/pengjun/motion/model/backbone.py", line 241, in forward
    x, features = self.forward_backbone(*input)
  File "/data/pengjun/motion/model/backbone.py", line 262, in forward_backbone
    x = child(x)
  File "/data/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/anaconda3/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 447, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/data/anaconda3/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 443, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Expected 3D (unbatched) or 4D (batched) input to conv2d, but got input of size: [1, 0, 4, 256, 256]
  0%|                                                                         | 0/4 [00:09<?, ?it/s]

It seems that @P2Oileen took the wrong version when releasing the code .
Try this one instead https://github.com/transpchan/CoNR

Hello, misska!
Sorry for this problem; since the UDP detector is an add-on to our work, it has not been tested rigorously for the time being. I tried to modify a part of the code, and the options you use work fine for me.
I used 3 sample character sheets as ACS input and 1 sample character sheet as pose input, and then I got these:

Can you please pull it and see if this problem is solved?
Please feel free to update any issues you have. This is very helpful for us💕

Hello, misska! Sorry for this problem; since the UDP detector is an add-on to our work, it has not been tested rigorously for the time being. I tried to modify a part of the code, and the options you use work fine for me. I used 3 sample character sheets as ACS input and 1 sample character sheet as pose input, and then I got these:
Can you please pull it and see if this problem is solved? Please feel free to update any issues you have. This is very helpful for us💕

Hi~  I successfully completedthe conversion script!
But I have a few questions:

How to generate high-resolution  images ? By reducing the size of the color block?
Character sheets need  pictures  without background , but videos  usually have backgrounds.Could you offer some methods to  remove the backgrounds automatically?

@misska1

High-resolution CoNR model is not yet trained due to limited GPU memory. Try giving me some free GPUs :)
If I understand the question correctly, there are many existing tools for background removal. For example, Adobe Photoshop, remove.bg, etc.  You may also try this if you want one that is trained on anime images.

Hi, misska;

CoNR currently only has 256 resolution weights, we don't have weights for higher resolutions now. Some adjustments to the network may be able to adapt to higher resolutions, but we have not tried it with limited energy. Sorry for that😢
There are a lot of algorithms for image cut-out at present, and we also have a UDP detector version with cut-out. Still, since the UDP detector is only an add-on content of CoNR, we did not do more enhancement in it, resulting in a limited effect. To make the user experience better, only the current version without cut-out has been released.
Multi-frame matting on video is a very popular topic, and you can find a lot of work on paperswithcode-video matting; As for anime images, we recommend waifucutout, although it can be difficult for cutting out large amounts of images.
Thank you for your question! For some urgent problems, we will record them and try to modify them in future versions iteratively.

Thank you very much! I found a new paper in Image Segment Area and the demo works pretty well ~
ECCV2022/dis-background-removal

How many A100 gpus did you use to train the model（256 resolution weights）, and what about  to train a bigger one?
Do you have plans to release a commercial version?

@misska1

We are too poor to use A100.
Live3D starts as a personal research project. Although it's now supported by Megvii, there is no plan to convert it to a commercial product.

That's nice! We will try it later💕

As shown in 4.1 in the paper, for 128 resolution, we used 4 Nvidia 2080Ti GPUs to train for a week. We have not tried A100 yet.
As for the commercial version: Most of the authors of CoNR are researchers, so part of our purpose of open source is that developers will be free to develop any software based on it. We are also more familiar with this type of collaboration; For example, our author, Zhewei Huang's RIFE has several commercial software versions. We hope that CoNR can have similar opportunities, and look forward to having community authors join the development of CoNR software ☺

For the training strategy of the 256 resolution version, we will write it in the new version of the paper after confirmation.

The same with the paper, apart form that it takes three weeks without random backgrounds.  @misska1 And that's why you can't use images with random background as inputs now.


filters: update iOS type for better conformance

Description: Updates filter status types to be in line with the types in Envoy and elsewhere in the library.
Risk Level: Low
Testing: Local
Signed-off-by: Mike Schore<EMAIL_ADDRESS>

Updated already on Android branch.

@rebello95 sure :)


Security: Voyager compass commands requires an unsafe Apache installation

Version information

Laravel: v5.6.28
Voyager: v1.1.3 (Installed 2018-07-31)
PHP: 7.2.7
Database:  MySQL 5.7.22-0ubuntu18.04.1

Description
Under /admin/compass > commands.
Some of the "php artisan ..." commands require write permissions on the filesystem, for example, all of the "php artisan make:..." commands create new PHP files.
But this is unsafe, against best practices, and presents an attack vector: the web server User ID (like "www-data" on Linux Apache) should not have write permission to any directory under DocumentRoot, aside from the ones listed in the Laravel docs: ./bootstrap/cache and ./storage. (Those two special directories are managed by the PHP code and have permission logic.) In contrast, if the web server can write executable PHP files, the webserver can (maybe) be tricked into writing a PHP file that will cat /etc/passwd (or whatever other attack).
Expected behavior
Laravel should not offer a GUI feature that requires an unsafe installation by default. If it is necessary to offer "php artisan ..." commands that write PHP files, then those php artisan commands should be restricted in an /etc/sudoers file to run as the user who owns all of Laravel's PHP files (such as user 'laravel' or maybe user 'root' if using a V.M. or appliance O.S.). The Voyager documentation should explain the risks and give clear setup instructions.
However, since creating new PHP files is only something that is done during development, and should never be done on a production Laravel installation, perhaps this feature should only be present when APP_DEBUG=true. That way developers can use these handy commands on their unsafe dev environments (like their VMs), but the feature would disappear once it is on a production server. That's probably better than setting up a hand-crafted sudo environment.
I would either (a) remove the feature or (b) make it only run with APP_DEBUG=true. But if you want to keep it functional as currently designed, please create docs on how to use sudo appropriately and the risk associated with a web server that can write its own PHP files...
Additional context
Bug #3267

This package has no control over how you manage your system.  If you want to add those commands to a sudoers file to make it more secure, by all means (Voyager has no possible way of automating this for you).  If you want to restrict write access to the www-data user as a security practice, then go ahead, and those commands will simply fail.
Just because we provide a feature, doesn't mean it's 100% secure without manual intervention.

Just because we provide a feature, doesn't mean it's 100% secure without manual intervention.

With respect, I am stunned by this attitude.  This is the opposite of "Secury By Default", and is not what businesses want to hear.
https://en.wikipedia.org/wiki/Secure_by_default
A PHP-based administrator panel, of all things, should put security first.

I agree that security should be (and is) a concern for this package, however you've essentially suggested that Voyager should manage/configure your server at the OS level by editing /etc/sudoers, which is equally ridiculous.  It's not our job to tell you how to harden your system security.  We as a package provide functionality.  If you wish to use that functionality in a production environment, you will need to ensure it's working in a secure manner, i.e. by restricting write access or editing the sudoers file.
I'm not dismissing your concerns.  I'm dismissing your suggested solution.

Thanks for the clarification. I'm loving Voyager and I appreciate the time and effort you put into it.
Some quick notes:

[It is ridiculous that] Voyager should manage/configure your server at the OS level by editing /etc/sudoers

Agreed, Voyager should not edit /etc/sudoers. I meant only that the administrator would use sudo to allow the webserver to run a limited and pre-approved set of commands, as you described. Other PHP apps do this too, to e.g. schedule cron jobs, manage containers, or edit .sieve or .procmailrc files.

It's not our job to tell you how to harden your system security. We as a package provide functionality.

It's an admin panel. The Voyager documentation should explain the risks and give clear setup instructions for a secure deployment. It is not reasonable to expect users to read source code to understand what vulnerabilities are introduced by simply installing the package.
This exact example (webserver admin permissions) is even used in the intro to
https://en.wikipedia.org/wiki/Secure_by_design


Reprise License Manager (rlm) client licensing library does not work with floating license server

Your Windows build number:  10.0.15063 (15063.332) and Insider build 16251.0

What you're doing and what's happening:

Reprise License Manager (RLM) licensing tool does not work for accessing a network license server.  As such, products that are developed (or being developed) against the RLM library cannot run with a floating license configuration.  Some products are only available in a floating capacity and therefore cannot be used due to this issue.

Downloaded RLM toolkit from http://www.reprisesoftware.com.
Installed organization's RLM license keys.
Compiled source code.
Set RLM_LICENSE variable to point to a running RLM server.
Run ./rlmclient
See the following error message:

Error checking out rlmclient license
Communications error with license server (-17)
Read error from network (-105)

What's wrong / what should be happening instead:

Error checking out rlmclient license
License server does not support this product (-18)

(This is expected since the server does not have a license configured for the rlmclient).

This problem was shared with the Reprise Software team.  The ticket has since been closed.  Please contact me for more info or for me to run any other diagnostic tools.

A full strace of the rlm test client "rlmclient" is attached.  Please let us know if you need further information to troubleshoot.
rlmclient.strace.log.zip

I am submitting a fix for this. It should soon hit insider builds.

No indication here what was broken or what was fixed, but whatever it was, it is fixedinsiders now since 14 Sep 2017 was after Fall Creators. Possibly something to do with select() on ipv6, shrug.
select(16384, [3], NULL, NULL, {5, 0})  = -1 EINVAL (Invalid argument)


TypeError raised from urllib in ListingItems.get_listing_item() in latest release

Describe the bug
When I call get_listing_item with a valid SKU it fails on the first and any call with this ValueError. Switching between the old version and new version solves and reproduces the issue.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I expect reading from this endpoint to work or not raise a ValueError from inside urllib
Desktop (please complete the following information):

OS: [e.g. Linux/Mac/Windows]

Linux
Additional context
I updated to the latest 0.18.8 and get this exception when fetching information about SKUs. Reverting back to 0.17.5 fixes this problem. I also updated ONLY this library in a working python environment and was able to reproduce so I don't think it is a dependency issue.
File /mnt/data/robert/sjerp/markets/spapi/listing.py:40, in spapi_get_sku_asins(sku)
     36 listingsitem_rate_limit()
     38 logger.info(f"fetching sku '{sku}'")
---> 40 out = ListingsItems(credentials=credentials).get_listings_item(
     41     sellerId=SJSELLER, sku=sku, includedData="summaries"
     42 )
     43 logger.debug(f"listingitems rate limit: {out.rate_limit}")
     45 ret = {}

File /mnt/data/robert/sjerp/.venv-20230627/lib64/python3.9/site-packages/sp_api/base/helpers.py:21, in sp_endpoint.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
     15 @functools.wraps(function)
     16 def wrapper(*args, **kwargs):
     17     kwargs.update({
     18         'path': path,
     19         'method': method
     20     })
---> 21     return function(*args, **kwargs)

File /mnt/data/robert/sjerp/.venv-20230627/lib64/python3.9/site-packages/sp_api/api/listings_items/listings_items.py:61, in ListingsItems.get_listings_item(self, sellerId, sku, **kwargs)
     39 @sp_endpoint('/listings/2021-08-01/items/{}/{}', method='GET')
     40 def get_listings_item(self, sellerId, sku, **kwargs) -> ApiResponse:
     41     """
     42     get_listings_item(self, sellerId, **kwargs) -> ApiResponse
     43     Returns details about a listings item for a selling partner.
   (...)
     58         ApiResponse:
     59     """
---> 61     return self._request(fill_query_params(kwargs.pop('path'), sellerId, sku), params=kwargs)

File /mnt/data/robert/sjerp/.venv-20230627/lib64/python3.9/site-packages/sp_api/base/helpers.py:10, in fill_query_params(query, *args)
      9 def fill_query_params(query, *args):
---> 10     return query.format(*[parse.quote(arg, safe='') for arg in args])

File /mnt/data/robert/sjerp/.venv-20230627/lib64/python3.9/site-packages/sp_api/base/helpers.py:10, in <listcomp>(.0)
      9 def fill_query_params(query, *args):
---> 10     return query.format(*[parse.quote(arg, safe='') for arg in args])

File /usr/lib64/python3.9/urllib/parse.py:871, in quote(string, safe, encoding, errors)
    869     if errors is not None:
    870         raise TypeError("quote() doesn't support 'errors' for bytes")
--> 871 return quote_from_bytes(string, safe)

File /usr/lib64/python3.9/urllib/parse.py:896, in quote_from_bytes(bs, safe)
    891 """Like quote(), but accepts a bytes object rather than a str, and does
    892 not perform string-to-bytes encoding.  It always returns an ASCII string.
    893 quote_from_bytes(b'abc def\x3f') -> 'abc%20def%3f'
    894 """
    895 if not isinstance(bs, (bytes, bytearray)):
--> 896     raise TypeError("quote_from_bytes() expected bytes")
    897 if not bs:
    898     return ''

EDIT: I checked and 0.18.0 is the latest release that does not have this issue for me. I am not doing aything fancy just calling the endpoint with  includedData="summaries"

Thanks for reporting, I'll check. What's the sku? Any non alphanumerics?

I'm not sure exactly which SKU was queued first but I did verify it was passed a real SKU and not some bug on our side. It was in the auto generated pattern like 0B-X8EL-V8ZY


SequelizeConnectionRefusedError: connect ECONNREFUSED <IP_ADDRESS>:3306 inside DevContainer

Extension Version
v1.0.12
OS and Version
Windows 11 / WSL (Ubuntu 22.04 + Docker)
VS Code Version
Version: 1.85.2 (user setup) Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681 Datum: 2024-01-18T06:40:10.514Z Electron: 25.9.7 ElectronBuildId: 26354273 Chromium: 114.0.5735.289 Node.js: 18.15.0 V8: <IP_ADDRESS>-electron.0 Betriebssystem: Windows_NT x64 10.0.22621
Database name and Version
mysql/mysql-server:latest
Steps to reproduce

Create laravel application with sail and --devcontainer option to get started with vscode devcontainer and laravel.
Open project within the devcontainer.
Try to open view DevDB view.
The Error appears and there is only the option to open a SQLite Database, but there is a mysql connection given in the .env file.

Expected behavior
Give me multiple options for databases to view.
Screenshots
Error:

.env:

docker-compose.yml:

Additional context
DevDB is not showing from where the requested ip <IP_ADDRESS> (default loopback ip) is coming from. It seem like it is not using the .env or detecting the laravel instance in the devcontainer.

Hello @Nowocyn , sorry for bot-closing this. It was due to sloppy regex. Now fixed.
Per your issue, I think it is related to DevDb not currently having support for mariadb and/or some Windows WSL issue. I currently use Linux and hence I do not have a windows system to repro this at the moment. So, I will leave this closed for now.
I am however open to reviewing a PR that addresses this.
Thank you.

I am also coming across this same issue. I'm attaching my VS Code workspace to a running docker instance and it seems to only be trying to connect to the loop back address rather than using the settings in my .env. I'm running Windows 11 with WSL2 backed Docker Engine and using Laravel 11 with sail.

@RecklessBoon Correct me if I am wrong but either sail/devcontainer/wsl or whatever you are using in your local development is still localhost. The only difference is that the port should be different, right?
Or perhaps the fix here is to use localhost instead of the currently used loopback address?

Actually, it's the opposite with a different host name but same port. The db is running in a different container, and the hostname is the container name. The default for sail appears to be "mysql" as the hostname.

Really, the connection info should match what is in .env which has the correct hostname and port.

Seems you don't get my point. I'll give more details:

Configuration specified in .env in a Sail application is essential for docker-compose.yml. It is for comunication across containers.
DevDb does not run in container, so it needs to access the database from outside the container
Hence, why DevDb actually really needs the port the database is running in the host (which is most likely always localhost)

Makes sense?

If I was running my VS Code from my desktop, sure that'd make sense. However, I am running VS Code attached to the app container (within the container). Since it is in the container, Vs Code runs in the containers network, not the host machine's. I do this so I don't have to type "sail" before every command and my extensions can utilize the build tools installed in the container. I have a .devdbrc with the same connection name and it works with "mysql". If this scenario isn't going to be supported, that's fine, but it should probably be noted in the "no-config" section of the readme.

Apologies, I wasn't clear on my setup!

Oh that kinds makes sense now. Yeah so your db server is definitely not on localhost that way.
I'm not sure how extensions work in "VS Code attached to the app container", so not sure if I will be supporting this feature yet. Good thing devdbrc covers this already.

@damms005 thanks for your answers.
Would the following scenario be possible without many changes?
Use the env variable DEVDB_HOST_FROM_ENV=true and read the given host from DB_HOST instead of using the default one and the default value of this configuration will be false so that only the user of a dev container will set it and also will set it to true.
Would be the perfect situation for these lines without changing a lot:
https://github.com/damms005/devdb-vscode/blob/86f419fbf94fa1724e0049cb79f53c6e5da85a13/src/services/laravel/env-file-parser.ts#L42
Your answers were very helpful thanks a lot.
Or should I create a fork with a PR with the wanted changes?

Yeah your suggestion is awesome and I love it! Thanks, @Nowocyn
I'll appreciate if you can open a PR that addresses this and I'll be glad to review and merge.
PS: I'd suggest you consider a DEVDB_HOSTNAME=<hostname.to.connect> variable in env file instead of DEVDB_HOST_FROM_ENV=true. Then if this value exists and is filled DevDb uses that. It will solve this issue and other cases that the host differes for whatever reason or just want to debug other databases. I think this approach will just open up some other use-cases.
Thoughts?

Just documenting my thoughts here...
https://masteringlaravel.io/daily/2024-06-11-what-is-the-difference-between-localhost-and-127001


Add ARM build

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

/Users/runner/miniforge3/conda-bld/lua-luaposix_1720344522566/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/luarocks: line 3: /Users/runner/miniforge3/conda-bld/lua-luaposix_1720344522566/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/lua: Bad CPU type in executable
/Users/runner/miniforge3/conda-bld/lua-luaposix_1720344522566/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/luarocks: line 3: /Users/runner/miniforge3/conda-bld/lua-luaposix_1720344522566/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/bin/lua: Undefined error:

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@izahn and @jerowe Can you take a look at this pull request?

@xhochy If you have a moment, can you also take a look at this pull request? The recipe was not update in the last three years, so I assume the maintainers are no longer active.


OMAP Overview table for v1.3 HRA release

Updating https://hubmapconsortium.github.io/ccf/pages/omap.html table with new numbers for OMAP table AS, CT, BP. Please see .csv uploaded here.
I've put both the OLD numbers from v1.2 release and new numbers for v1.3 release.
OMAP3rd-4threleaseNumbers.csv

@bhushankhope these are the new numbers for the HRA Portal's OMAP page. Please implement and let us know when it's merged.

Implemented at https://humanatlas.io/omap


MSID: 94064 Version: 1 DOI: 10.1101/2<IP_ADDRESS>98570

MSID: 94064
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>98570
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94064
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>98570

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94064v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>98570
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94064/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@fred-atherden Zhanxian Wang's email address is incorrect

Well spotted @naushinthomson. Corrected package uploaded. Should be resolved in the next couple of hours.


WebGL

Just want to let you know that you can use WebGL for draw graph like plotly https://plotly.com/python/webgl-vs-svg/

Cytoscape.js does not support using a GL context as far as I can see. But it's using the 2D context of a Web Canvas. Which is already relatively fast. You can also go up to a million points, not as smooth as WebGL but still way faster than SVG.

Thanks for the feedback @martinRenou!
@gqpj if you're interested in talking or getting involved somehow QuantStack has an open channel on gitter that you can join: https://gitter.im/QuantStack and https://gitter.im/QuantStack/core_development, thanks for the issue =)

Thank you for explanation! I'm going to close this.


Fixed IE issue

Fixes an IE 'Invalid Argument' exception with textareas:
Error happens, because (for whatever reason) IE temporarily converts  to foo. This triggers a diff patch on the 'foo' innerHTML as a VTEXT node, but by the time vdom tries to set the VTEXT's value, it has been orphaned by IE.
IE throws an error whenever you try to set the nodeValue of a parentless node which exposes itself as an 'Invalid Argument' exception.

Coverage remained the same at 97.531% when pulling 8aadfcb6d35cbe50afbe3759cce415f08fde8273 on mikeangstadt:master into 947ecf92b67d25bb693a0f625fa8e90c099887d5 on Matt-Esch:master.

Try always putting a value in the 3rd arg.  Ex:
const attrs = {
  "placeholder": 'dogs'
};

h('textarea', attrs, '');