Faq section aligned and styled in mobile view

Description
Faq section aligned and styled in mobile view
Please include a summary of the change and which issue is fixed.
Fixes #390
List any dependencies that are required for this change
Na
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
On my local host
UI /UX changes
Attach gif or screenshot for changes.

Before :

- After :

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

REMOVE MERGE CONFLICTS @harshb02

Removed the conflicts @ravikr126

Aren't these changes same as #429

No in #429 I have aligned the Sharpen your Skills sectionin mobile view and in this PR I have aligned the FAQ section in mobile view.

Aren't these changes same as #429

No in #429 I have aligned the Sharpen your Skills sectionin mobile view and in this PR I have aligned the FAQ section in mobile view.

Okay both the changes are present here so I thought they were the same changes anyways do the requested changes we can merge it.

I have removed the div because it was for not displaying the social sidebar in the mobile view but eventually it was done by someone else so I have removed the div and have done the changes @harikesh409

I have removed the div because it was for not displaying the social sidebar in the mobile view but eventually it was done by someone else so I have removed the div and have done the changes @harikesh409

okay resolve the conflicts so that we can merge.

resolve the conflict @harshb02

Removed the conflicts @ravikr126  @harikesh409


feat: Add Typescript Type Declarations

Added typescript declaration file as described by amokrunnr in #7

@rang-ali any interest in adding this ts support?


Webhook not working when using self signed certificates in chain.

Describe the bug
When using self-signed certificates in the chain the webhook is not working.
I already tried to add my CA cert to /usr/local/share/ca-certificates directory and using update-ca-certificates to update it, and tried to add it manually to /etc/ssl/certs/ca-certificates.crt but that didn't help.
To Reproduce
Setup a webserver or api endpoint using SSL with a self-signed certificate, activate the webhook in camera.ui.
Expected behavior
Working Webhook.
Logs
[2/3/2022, 2:00:11 PM] [CameraUI] VIDEOANALYSIS: New message: Data: [{"zone":"region0","percent":83.41078186035156,"sensitivity":18,"dwell":60,"forceClose":3}] - Motion: detected - Camera: Hallway
[2/3/2022, 2:00:11 PM] [CameraUI] Hallway: Motion ON
[2/3/2022, 2:00:11 PM] [CameraUI] Hallway: An error occured during sending webhook notification
[2/3/2022, 2:00:11 PM] [CameraUI] RequestError: unable to verify the first certificate
[2/3/2022, 2:00:12 PM] [CameraUI] Hallway: Motion OFF - Motion handler timeout.
Screenshots
Environment
Node.js Version: v16.13.2
Homebridge Version: v1.4.0 (HAP v0.10.0)
Camera UI Version: v5.0.16-latest
Operating System: Debian 10 with Docker Alpine Linux (3.12.9)
Browser Version: Edge 97.0.1072.76 (Official build) (64-bit) on Windows 11

should be fixed in v5.0.17-beta.4


Porcupine Issue: npm install failing for web demo on Windows

I am trying to run the web demo of Porcupine on Windows.
Expected behaviour
After cloning into the repository and running:
cd porcupine/demo/web
npm install

packages should install without errors.
Actual behaviour
After running npm install, I get
><EMAIL_ADDRESS>postinstall
> npx pvbase64 -i ../../lib/common/porcupine_params.pv -o ./porcupine_params.js && cp ../../resources/keyword_files/wasm/porcupine_wasm.ppn ./

Encoding '../../lib/common/porcupine_params.pv'
Done! Saved file to './porcupine_params.js'.
'cp' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code 1
npm ERR! path C:\Users\jblak\Documents\porcupine\demo\web
npm ERR! command failed
npm ERR! command C:\Windows\system32\cmd.exe /d /s /c npx pvbase64 -i ../../lib/common/porcupine_params.pv -o ./porcupine_params.js && cp ../../resources/keyword_files/wasm/porcupine_wasm.ppn ./

npm ERR!      C:\Users\jblak\AppData\Local\npm-cache\_logs\2023-01-13T07_44_58_102Z-debug-0.log

Steps to reproduce the behaviour
On Windows, simply clone the Porcupine repo git clone --recurse-submodules https://github.com/Picovoice/porcupine.git
and run
cd porcupine/demo/web
npm install

Thank you for reporting; the issue is fixed now and the web demo should run flawlessly on Windows.


Please add support for Molmo

This is SOTA open source models based on qwen2 vl - With unprecedented accuracy it can mark the coordinates of objects on screenshots, which distinguishes it from other VLMs. You can make amazing projects of autonomous computer control, but so far there is access only to FP-32 scales that simply do not fit into the RAM of even a 36GB Mac

Awesome!
Thanks for the suggestion, I will work on it after v0.1.0 that's coming out soon.

Any update on this?

Yes.
Check #112

I'm about to complete the language model and then will move to the vision model

Awesome, thank you so much!!!

My pleasure!


Make textbox focus clearer

To me difference between focused and unfocused text box is too subtle, and I often find myself accidentally activating browser shortcuts when I've meant to type code.
Fortunately playpen remembers text when I exit the page via accidental backspace (thanks for that), but I'd rather not make that mistake in the first place.
I think the most deceptive thing for me is that current line is highlighted in yellow even if the box is not in focus.
When text entry doesn't have focus I suggest removing current-line highlight or changing it to light gray. Darker/lighter border of the text box would be useful as well.

https://github.com/ajaxorg/ace/pull/2397 changes line highlight to be gray in blurred state


Functionality for more easily indexing data by non-coordinate variable

We want to have TIME rather than STATION or CAST coordinates.
Still, it it often more convenient to access e.g. a CTD profile based on STATION or CAST than a time stamp.
This is a little clunky in xarray - want to write wrappers in xr_funcs.py for doing this more smoothly.
What we want may effectively be wrappers for sel where we can use non-coordinate variables.
(Maybe call it pick or something?)

Fixed (file.xr_funcs.pick())


[WORKING DRAFT] Add support for Mixtral 8x7b

Adds mixtral 8x7b using openrouter.ai
Fixes couple supposed isssues around
client.models.list().data in openrouter.py did not return dictionary
also I've added cached_model_details = None in line 20 because otherwise it seems not to upload models from openrouter.py but uses openAI instead
Code works as is but Mixtral requires some tuning on prompts side, so this code is not useful solution yet

Consider an option to allow it to used a locally hosted version of Mixtral 8x7b instead of openrouter.ai

Consider an option to allow it to used a locally hosted version of Mixtral 8x7b instead of openrouter.ai

I do not have hardware to run it locally


small font size not working in 4:3

I just updated the theme files with the latest versions and the Small font is displaying as the same size as Medium when in 4:3. I switched the aspect ratio to 3:2 and Small is the correct size, but 4:3 is the only one where all the UI elements line up correctly on my screen (Galaxy Fold 4).
I replaced aspect-ratio-4-3.xml with the version from July 22nd and the font is back to the expected size. I am more than happy to keep using the old file for now, but I would like to keep everything up to date if possible.
Thanks!

Hey, thanks for catching this. I'd been optimizing the font sizes to work better with the new smaller 4:3 screens coming out. I think the Small font got bumped up a bit too far. I've scaled it down slightly so it's noticeably different from Medium. Let me know if that works. Thanks!

You are awesome! It looks great now. I think it's perfect for those newer 4:3 devices like that SP one Anbernic put out recently.  And the Fold is such a weird screen size/ratio to try and target. I will never begrudge any devs not going out of their way to optimize things for it.
Thank you! Your themes are the best. It took me a long time to decide between Canvas and Iconic. I still go back and forth sometimes. I appreciate the work you've done on these so much.


Display Excel-Data

Hello,
In reference to your "todo-comment":
https://github.com/AdyStudios/LogiReader/blob/0b06c09c8a577d8cc9bf63d4bd5f91efad7e3308/src/App.jsx#L40
Do you have sample datasets that can be used here? Unfortunately, I do not have access to LogiScool.

Yes sorry, I'll provide it here:
SurveyAnswers-20230811185844739.xlsx

Thanks!


Add add_<shape_type> method for each shape type

Description
As proposed in #2956 this PR adds individual convenience methods for rectangles, ellipses, polygons, lines and paths.
The methods are all calling the existing add() method after validating the number of vertices for each shape in the given data. This validation seems to be sufficient even for nD shapes - although this is only because at the moment they are defined within planes. Once we move to cuboids and spheroids we would have to do a bit of arithmetic to compute valid number of vertices using the dimensions of the coordinates. This can be done per shape though, so I think the current validation method would be reusable in that case.
Valid vertices for each shape as currently defined:

Rectangles: 2 or 4
Ellipses: 2 or 4
Polygons: 3+ (looking at the code this was 2, but I don't think a valid polygon has 2 vertices...)
Lines: 2
Paths: 2+

I've also added tests for each of the methods, including for nD where applicable.
NB: I'm not currently doing any validation for the properties, as that is already done downstream. However, maybe there's more validation that could be done per shape?
Type of change

[x] New feature (non-breaking change which adds functionality)

References
Closes #2956
How has this been tested?
I've added tests for each method for both 2D and 3D cases, and have parametrized each test with the same input we give to the existing tests.
Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@jni after updating the docstrings I checked to make sure that adding a single shape did work, and the validation was failing. I've made changes to validate_num_vertices to fix it but it feels a bit awkward?

I've made changes to validate_num_vertices to fix it but it feels a bit awkward?

Nah, it's par for the course for this type of magic. 😊


Add isBounded property so items could be constrained within grid

This PR fixes #546

@STRML, what do you think about that PR? To clarify things a little bit more, these changes add a property which controls wether placeholders should be bounded within grid while dragging items around

@artembykov, there are few file conflicts.
@STRML please, pay attention to this PR and merge as a fix is really important.

Any ETA on getting this merged ?

ping @STRML
I've rebased on master today with one small perf improvement.

https://github.com/STRML/react-grid-layout/pull/549

@artembykov there are file conflicts again.

@artembykov there are file conflicts again.

Resolved!

@STRML would be great if you could merge this !

+1 for this

+1 for this

+1 for this

Let's get this in please! This is an important feature

This is still needed! #nostale

+1 still needed! Any chance this can be included in a future milestone?

Continued in #1248.

Published in 1.0.0!

Thank you thank you thank you!!


DNS leak on e2e tests

@sridhargaddam @mangelajo, could be related to #567
External DNS bombarded with PTRs to internal addresses.
Example:

Produced with sudo tcpdump -i any udp port 53 on host
Could be related to #567 if only after NXDOMAIN there is a fallback to actual destination (just guessing)

Try using -n on tcpdump, could be tcpdump trying to resolve the inverse of each ip it sees

no tcpdump running on the kind nodes.

@mangelajo I ran another test to verify -- did not run iptables in kind. Used -n flag. No WG.
Still many PTR for internal addresses.
Plus, lots of DNS traffic -- mostly to docker.io


Agents and tools that work with document chunks and not with string / chat inputs?

Most of the agents and tools I see in Langchain doc work with simple string or chat inputs.
However, what if I want to make an agent or a tool that would take a set of documents as input, process it, and only then send it to an LLM model?
For instance, I want to propose another method of reducing dimensionality of a text document before it gets sent to the summarization function. And I don't want to use map_reduce or refine, but, rather, an external service that would extract the most relevant parts of this text for me and then feed them to the model.
Are there any agents and tools that work like that? What would be the best way to integrate it into Langchain?
So far, I only see Summarization and AnalyzeDocument chains, but they seem to be fully integrated into core modules and not offered as external tools / agents that would need an API key to function.

Interesting - yeah I think creating your own subclass of AnalyzeDocument chain would be the closest way to do it at the moment.
I could also see a case for expanding that chain to accept an array of document transformers that operate on the input documents? And then the "external service" would just be an instance of a document transformer.

You can always make a new DynamicTool (or a structured variant of this if you want the LMM to be able to give it other kinds of input than just a string) for your agent to use, with a function that fetches the documents (based on the inputs), processes them (in your case with an external tool or API or whatever it is), and gives them to the LMM back in a string format? I'm just spitballing here, and am not sure if this answers your question that well.


fix inconsistent error message

Codecov Report

Merging #257 (5e2f77c) into master (b3c9cfc) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #257   +/-   ##
=========================================
  Coverage     47.15%   47.15%
  Complexity      481      481
=========================================
  Files           116      116
  Lines          2585     2585
  Branches        181      181
=========================================
  Hits           1219     1219
  Misses         1290     1290
  Partials         76       76

Impacted Files
Coverage Δ
Complexity Δ

...java/seedu/address/logic/commands/ViewCommand.java
50.00% <100.00%> (ø)
7.00 <0.00> (ø)

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 b3c9cfc...5e2f77c. Read the comment docs.

👍👍


Backport 2.16: Change the use of pylint to optionally use pylint3

Description
This is a backport of PR #3100 to mbedtls-2.16.
Pylint when installed as a distro package can be installed as pylint3, whilst as a PEP egg or directly from the pylint website, it can be installed as pylint.
This pull request changes the scripts to first use pylint if installed, and optionally look for pylint3 if not installed. This is to allow a preference for the PEP version over the distro version, assuming the PEP one is more likely to be the correct one.
I'll do the back ports once this PR has been reviewed and approved, and thereby approving the approach.
Status
READY
Migrations
NO
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog updated
[ ] Backported

Passing CI, reviewed and approved so promoting to 'ready for merge'.


[CDS-84506] CDS 815xx relnotes

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore
[x] Release notes

If tied to an Issue, list the Issue(s) here:

CDS-84506: CDS 815xx release notes

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


fix(serial): handle close event while opening serial port

It was reported that the serialport open call sometimes blocks. This is an attempt at fixing it.

@zwave-js-bot pack this


Automator: update istio/api@master dependency in istio/istio@master

Generated by Automator - 2022-07-02T05:29:13+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
b3988e54d1109cfcfe1166bd5b4a764de26d2d49
link
true
/test integ-telemetry-mc_istio

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

/test integ-telemetry-mc_istio

Merged as part of https://github.com/istio/istio/commit/2a966a23799e42372c0a01cb347cfffa08fb4afd


Forward output from the githooks to stdout

When you push to a remote repository like Heroku, Dokku etc. would be cool to see the output from the remote hooks of these services.

Hi @bkzl, thanks for the suggestion. I think it will be easy to redirect output from the sub processes to the main process's stdout/stderr

Published in version 0.4.0.  Include the --verbose flag to get all the stdout/stderr from subcommands.  Thanks for the suggestion @bkzl!

✨Super cool! Thank you! ✨


mcuboot fails to compile due to missing intelhex python module

When compiling mcuboot for Zephyr following the instructions at Building and using MCUboot with Zephyr
I received the following error message
[13/194] Generating zephyr/autogen-pubkey.c
FAILED: zephyr/autogen-pubkey.c
cd /home/piotr_m/work/mcuboot/boot/zephyr/build && /usr/bin/python3 /home/piotr_m/work/mcuboot/scripts/imgtool.py getpub -k /home/piotr_m/work/mcuboot/root-rsa-2048.pem > /home/piotr_m/work/mcuboot/boot/zephyr/build/zephyr/autogen-pubkey.c
Traceback (most recent call last):
  File "/home/piotr_m/work/mcuboot/scripts/imgtool.py", line 20, in <module>
    from imgtool import image
  File "/home/piotr_m/work/mcuboot/scripts/imgtool/image.py", line 21, in <module>
    from intelhex import IntelHex
ModuleNotFoundError: No module named 'intelhex'

intelhex python module is not included as the Zephyr build dependency in scripts/requirements.txt.
The mcuboot documentation should probably mentioned that this module needs to be installed by hand or it should be added to Zephyr dependencies in scripts/requirements.txt.

It is included: https://github.com/runtimeco/mcuboot/blob/master/scripts/requirements.txt#L2
Are you running master?

Yes, master. But I didn't know I'm supposed to run
cd ~/mcuboot  # your directory where mcuboot is cloned
pip3 install --user -r scripts/requirements.txt

from mcuboot. Documentation mentions only that mcuboot should be build as a standard Zephyr app, there is no mention of extra dependencies. These instructions should probably be added to the above mentioned "Building and using MCUboot with Zephyr" page?

Ah, I see. It is documented on the imgtool doc but maybe adding a pointer from readme-zephyr.md could be a good idea. We are open to PRs! ;-)

Done: #325


[Blazor] DynamicComponents - fixes samples vulnerability + related adjustments

@guardrex, this draws the new samples from https://github.com/dotnet/blazor-samples/pull/256 and includes related adjustments

Internal previews

📄 File
🔗 Preview link

aspnetcore/blazor/components/dynamiccomponent.md
Dynamically-rendered ASP.NET Core Razor components

BTW @hakenr ... We're using a tool that Kirk Larkin (@serpent5) made to efficiently get XREF links. For API like Dictionary<TKey, TValue>, where there's some encoding and numbering, it's very helpful. It taps the same service the the API browser uses ...
https://rs-xrefs.azurewebsites.net/

BTW @hakenr ... We're using a tool that Kirk Larkin (@serpent5) made to efficiently get XREF links. For API like Dictionary<TKey, TValue>, where there's some encoding, special characters, and numbering, it's very helpful. It taps the same service that the API browser uses ...
https://rs-xrefs.azurewebsites.net/

BTW, my first (local) version was using the XREF to Dictionary<TKey, TValue>, but in the end I decided to use the Dictionary<string, Type> version in the text and thus dropped the XREF (is there a way to combine the two?).
But don't worry, the generic Dictionary<TKey, TValue> works here, no need to use type-specific version in the text.
For me, another concern with using the XREF is deciding whether to sacrifice the code style in favor of a link. (In our component library, we combine these into CodeWithLink, but it is obviously not used in MS docs.

Indeed, I've had my own tiny pet peeve about the styling for many years.
Anyway ... yes ... combine the XREF with code-fenced link text ...

[`Dictionary<string, Type>`](xref:System.Collections.Generic.Dictionary%602)

That format would be a bit atypical from a C# Guide perspective. I don't think they ever do that, but we sometimes do :smile:. I'm sure if Bill Wagner saw some of the wild stuff over here, he'd roll his eyes and curse our bones! :laughing: I've used atypical link text in some cases. I doubt anyone is very confused or upset by it. I think the API browser cross-links are valuable in that case, and it did occur to me when I floated the suggestion to do that. However, I think just cross-linking the dictionary is best. Readers should probably get used to seeing TKey/TValue things, as it's everywhere in the API browser and C# Guide.
I would like to get more of the types without cross-links addressed. Sometimes, I'm just rushing and don't place them. Sometimes, new API can't be cross-linked because we're in a preview. Sometimes, the PU ... ours and other PUs ... fail to expose them in the public API. We must put in a request to have them surfaced, and then I hopefully have a tracking item to circle around and fix up the text later.

👍


[Feature]

It would be cool to add the ability for a message to be displayed above the player’s inventory (so as not to clog up the chat).  How can I implement this?  I opened every file in the plugin, but still couldn’t find the lines responsible for this...

Hi there!
I'm not sure what you mean by displaying the message above the player's inventory. Perhaps, you meant above the hotbar where the item names are displayed( I forgot it's name ).
This event is responsible for handling the player death events.
This function sets the death message (which is seen in chat) and the kill streak broadcasts can be seen here but can be disabled using the config.
You can create another plugin that sets the death message to null but sends the player the text using the way you would like.
You'll have to set the priority to high on that event as AdvanceDeaths sets it to low to be called the first.
You may also add this feature to AdvanceDeaths and make it configurable.
Users will be able to change back and forth using a key/value as such:
announceDeath: false
showasblahblah: true

I understood the issue a little.  So, there is a sendMessage event, and there is a sendPopup.  So I propose in one of the updates to add the ability to display death messages not in chat, but as a Popup

You can do that
Feel free to make a PR or a plugin
Best of luck!


Fix Zip(IEnumerable) NullReferenceException if a source completes immediately

If a source to the Zip operator completes immediately before the SubscribeSafe returned, the associated entry in _subscriptions would be still null, thus the Dispose() call fails with a NullReferenceException.
The fix checks if the _subscriptions is still valid and uses the deferred dispose approach to clean up the indexed entry.

Early termination aside, the fact that we wouldn't handle the disposal with TryDispose at that specific location was a bug in its own right, wasn't it?

Yes, a concurrent completion would have potentially experience that null slot too.


Break validate-against-schema to multiple rules?

I’m thinking about breaking the validate-against-schema rule to multiple rules. At the moment it runs just validate of graphql-js with all default rules.
I see more and more people needs to customize it, enabling/disabling rules, and I don’t like the fact that we run multiple checks at the same rule.

Done in https://github.com/dotansimha/graphql-eslint/commit/86c4fbca10d6a033c734b2b97339be878e67d831


Added list_namespace to client and fixed gRPC request

list_namespace was broken because of this PR merged into the Protos repo. This fixes this and also exposes list_namespaces as a function on the client (we're wanting to use this functionality at Stripe)

Hey @antstorm! 👋

Can you please add a unit test (in addition to the integration test)?

Added :)

I wonder if it makes sense to allow the caller to specify a page size without the offset? Otherwise there's no way to get to results beyond your page size without increasing it

Woops overlooked this... you're right! Added next_page_token to the function params

I wonder if we should start wrapping responses from gRPC in order to prevent proto exposure to the users of the SDK. I know that other methods might have the same issue, so it's optional as far as this PR is concerned

Interesting point, not sure how you track work like this that needs to be done but feel free to add it there!


[SUGGESTION ] - Replace VideoSlimer as video compressor on Android

https://github.com/zolad/VideoSlimmer seems like not work well with video with modern aspect ratios.
If you do various tests on various video aspect ratios, the module works well with traditional ratios like 4:3 or 16:9.
While rotating video on a modern ratio like 18:9 on 1920x1080. This also happens in the test video by the module author.
Author compress video with 1920x1080 and output becomes 200x360, The video actually reversed.

@emmbyiringiro i already replace you can check in build.gradle.


Why is "editor.cursorBlinking":"hidden" a thing?

An editor with an invisible cursor doesn't seem particularly intuitive.

The "hidden" option got removed in 9bcb8b1838a6cba8e4a95b739a88a8dc5db99f1e


Inquiry on the Determination of Representative Structural Variants in Merged VCF Sets

Hi ,
Thanks for your tool, and I try to test in my research.
I am attempting to merge multiple VCF sets from a sample, including sets from SVIM, cuteSV, SVision, Debreak, and Sniffles. However, I am unclear about how the representative structural variants (SVs) are determined in the output VCF set. I have observed that Sniffles2 is frequently chosen as the representative in many cases. Could you please explain if there is a specific reason for this preference?
Best wishes

The representative variant is determined by the --choose parameter
https://github.com/ACEnglish/truvari/wiki/collapse#--choose-behavior

I apologize for my oversight.
Best wishes


Device Group Messaging for iOS doesn't work when app is in background

iOS devices don't receive messages sent through device group messaging when app is in background. It works when app is in foreground.

@baranbaygan We know about this, should be fixed soon.


Bringing UP the INT_IF NIC

After destroying and re-creating the libvirt network
need to bring UP the NIC that's attached to the bridge.
Signed-off-by: Alexander Chuzhoy<EMAIL_ADDRESS>

Build FAILURE, see build http://<IP_ADDRESS>:8080/job/dev-tools/470/

recheck

lgtm


API endpoint manifest_diff is incorrectly marking processes as being removed and re-added rather than indicating process-level changes

Context
Currently seeing remove and add operations rather than a replace operation when enough changes have been made to a process. This seems to be a result of the diffing library that is being used.
Current Behavior
{
  "diff": [
    {
      "op": "remove",
      "path": "/applications/0/processes/0/disk_quota",
      "was": "1024M"
    },
    {
      "op": "remove",
      "path": "/applications/0/processes/1",
      "was": {
        "disk_quota": "1024M",
        "health-check-type": "process",
        "instances": 1,
        "memory": "1024M",
        "type": "worker"
      }
    },
    {
      "op": "add",
      "path": "/applications/0/processes/1",
      "value": {
        "instances": 1,
        "type": "worker"
      }
    }
  ]
}

Acceptance Criteria
Process changes should be shown via a replace operation
Given an app named my-app with one or more processes
When I make changes to one or more processes
And I use the manifest_diff endpoint
Then I should see in the diff a replace operation for the altered process(es)
And I should not see remove and add operation(s) for the altered process(es)
Notes
Consider whether sidecars are being implemented in the same way, may need to add a sidecar key.

The json-diff gem uses a similarity function to determine if a given array element is present on both sides of a diff. If the calculated similarity is below a hard-coded threshold, the gem will treat the old element as removed and the new element as added (instead of a replace). For the processes array (or any array of JSON objects), removing enough key-value pairs from the object will push the similarity between the before and after versions of the object below the threshold, causing the behavior described in this issue.
The gem allows the caller to provide a custom similarity function. We plan to pass a custom similarity function that will compare the processes' type field. Since the type is unique per-process, comparing it should allow us to handle this case.
We may need to make a similar change for other arrays of objects in the manifest, like sidecars and services.


infery error,  NotImplementedError: A UTF-8 locale is required. Got ANSI_X3.4-1968

💡 Your Question

Versions
No response

This solves the error
import locale
locale.getpreferredencoding = lambda: "UTF-8"
but raises:
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
ERROR: Could not find a version that satisfies the requirement infery (from versions: none)
ERROR: No matching distribution found for infery

We have updated our notebooks to not be using infery any more. Closing this one for now.


Fix Engine.UpdateVariable for Int and add more overloads

Consistent with how we pull data in from connectors, UpdateVariable with an int should return a Decimal instead of a Number.  In addition, this PR adds additional overloads for common data types that a C# host might want to use.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


Apply new gas mechanism

Subtask of https://github.com/klaytn/rosetta-klaytn/issues/33

[ ] Apply gas burning policy.
[ ] Use effectiveGasPrice to calculate gas price used when process a tx (dynamic gas fee).
[ ] Use baseFee field in Header or Block type.

Need to check TestBlock_13998626 test to make sure rosetta operates correctly.

refer to https://github.com/klaytn/klaytn/pull/1409

This implementation is merged, and this will be released as version v1.0.1


Pipe file to express response

I want to download a remote file, for this I'm using the request module to pipe the file to the user.
this is the code that I'm using
var options = {
method: 'GET',
url: 'https://www.onsemi.com/pub/Collateral/P2N2222A-D.PDF'
};
__request(options)
.on('end',function(){
console.log('end');
})
.on("error",function(err){
console.log('error')
})
.pipe(__fs.createWriteStream('test.pdf'))
//.pipe(res);
when doing  .pipe(__fs.createWriteStream('test.pdf')) I'm getting the file on disk without any issue
but when doing .pipe(res); I'm receiving the below response header which seems correct but I'm not receiving the file on the client side, what I'm doing wrong ?
ResponseHeader:
HTTP/1.1 200 OK
X-Powered-By: Express
server: Apache/2.2.15 (Oracle)
content-disposition: inline; filename="P2N2222A-D.PDF"
content-type: application/pdf
content-length: 122542
cache-control: max-age=0
expires: Mon, 17 Jun 2019 15:10:57 GMT
date: Mon, 17 Jun 2019 15:10:57 GMT
connection: close
access-control-allow-headers: X-Requested-With
access-control-allow-origin: *

Might I ask why you are doing this manually instead of using send?  With that you could just do:
send(req, 'test.pdf').pipe(res)

https://www.npmjs.com/package/send

Send is a library for streaming files from the file system and my files are not on the file system, there are hosted in another app.
and I don't want to download them completely on the server then send them back to the client because the app will have a lot of traffic and the files and not small this will take space and time so its not an efficient solution for me, the best is to pipe the stream at real time.

Oh your code was all mangled so I didn't really read it closely.  Anyway, piping should work, It has always worked for me before.  Here is an example straight from the request docs:
request.get('http://mysite.com/doodle.png').pipe(resp)

If you can pin point this as an express issue feel free to reopen, but this is not really a support channel.  Try posting on Stack Overflow or in the Express Gitter


Set of NPCs

Hello,
Would it be possible to create a set of NPCs that are linked to UUID which isn't player's?
I'm trying to assign NPCs to a bentobox island's UUID with a Set, but I'm confused how it'd look since I'd need to rewrite code from NPCManager and NPCBase.
I want the Set to be retrieved when player logs in and has .getIsland().getUniqueId() that is in the set so it can retrieve all the NPCs added to it and show it to him.

Hi, so I've created
    private final Map<String, Set<NPC>> npcs = new HashMap<>();

    public void NPClist(String uuide) {

        npcs.put(uuide, addNPCs());
    }

    public Set<NPC> getNPClist() {

        return npcs.get(uuid);
    }

    public void showNPClist() {

        for (NPC npc : getNPClist()) {
            npc.show(p);
        }
    }

    public Set<NPC> addNPCs() {

        Set<NPC> list = new HashSet<>();
        list.add(Chancellor());

        return list;
    }

    public NPC Chancellor() {

        NPC Chancellor = npcLib.createNPC()

        Chancellor.create();

        return Chancellor;
    }

It's initialized, it saves to the map, but it won't show to the player.

@Mooselk any ideas?


Fastfile before_all block runs before each lane.

before_all block should run before all lanes, not before each lane.

Are you referring to https://github.com/KrauseFx/fastlane/blob/master/Advanced.md#run-multiple-lanes

Keep in mind the before_all and after_all block will be executed for each of the lanes.

You think this should be changed? Why?
I think it makes sense, just imaging having 2 different beta testing services. You will probably want to increase the build number for both, which might be done in the before_all block.

I think this could be confusing if you are used to using most testing frameworks (i.e. rspec or Kiwi) where before :all is run once, and before :each is run before every step/test.
I would recommend changing it to before_each and maybe adding a before_all block that executes only once. I would also considering doing the same with after_all for consistency/symmetry.
As an example, I would want use before_all to install cocoapods or select an xcode version to use for lanes.

I agree with the before :all and before :each :+1:

How about switching to also offering before :all and before :each and still supporting the old before_all (with the same behaviour as it's now)

And update the documentation to only make use of the new syntax.

:+1:

Any update on this issue? I'm looking to do a build of my app in both our staging and production configurations in a single command and more importantly in a single commit. I have daily builds of our staging app being run, but I'd like to be able to use that same lane but also include a production lane at the same time. Is there still even support for running multiple lanes in one action?
To sum up what I'm trying to do:
When I run
fastlane ios staging

Increments the build number.
Builds the staging configuration of the app
Delivers to HockeyApp and TestFlight.
Commits, tags, and pushes updates to the remote repo.

When I run
fastlane ios production

Increments the build number.
Builds both the staging AND production configurations.
Delivers both apps to HockeyApp and TestFlight (different app identifiers / configurations in each service for both apps).
Commits, tags, and pushes updates to remote repo.

Is there an easy way to do this without copy / pasting the staging lane details or creating a bunch of different lanes that just do parts of this? I feel like I might have missed something here...

@Spencevail There are multiple ways to solve this.

You can add methods to your Fastfile as seen in this example.
You can call fastlane multiple times like this: fastlane ios && fastlane production

Does that answer your question?

Going to close this issue since it solves the original problem and has been open a while without comment.

Hey guys can we get an update on the before :all implementation? Not a big fan of calling the same lane multiple times before running a build lane.

@arunks we already have that, just use
before_all do
  gym
end

@KrauseFx  before_all runs before each lane. This becomes an issue if I am trying to call multiple lanes in my build job.

@arunks Thanks for following up. This is an interesting idea. We'll look into it :+1:

If anyone is looking for before_each or after_each blocks - it was just merged here: https://github.com/fastlane/fastlane/pull/4264


unbound syms (weird behaviour)

Seems I recently messed something up:
@9: (use std/bench)
ERR: unbound sym: start-time
BACKTRACE:
(_load 'std/bench)
  ;; hidden tail calls: 1
(_with-var '_*incomplete-mods* '(std/bench) #sub(id=0x7f7c158205c5 name=@_use-mod argc=0 take-rest?=#f))
(_use-mod 'std/bench)
(each '(std/bench) #sub(id=0x7f7c162dcf4d name=_use-mod argc=1 take-rest?=#f))
  ;; hidden tail calls: 1

@10: (use std/bench)
(std/bench std/math)
@11: (say-time (times 50000 |(in-reg (iota 100 1 1) ())))
ERR: unbound sym: say-timex	|
BACKTRACE:

@12: (say-time (times 50000 |(in-reg (iota 100 1 1) ())))
ERR: unbound sym: say-time
BACKTRACE:

This is the kind of bug that will probably take me a while to find.

I had time to look into this now and fixed it in e441e9587261a4bddbeaae171f074cc744766e57


feat: biomass pseudo-reaction requirements

In the current yeastGEM, it lacks some specific cofactors and metal irons, which need to be added with the improvement of the model coverage when hundreds of new reactions will be added to the yeastGEM.

@hongzhonglu sounds great! could you list those new requirements here? maybe we can get some help to check if they make sense or not from @SysBioChalmers/yeast

update: this issue has been addressed in #174 and will be closed on the next release


Remove README link

My article doesn't recommend dropping the numerical prefixes 👀

Ugh, apologies.

No worries 😁


Rules cannot iterate through facts?

I understand one can apply multiple rules to any value on the facts map.
But what if I have a List of objects that I want to have evaluated by the same rule pipeline?
Imagine I have something like:
List<Person> people = new ArrayList<Person>();
people.add(new Person("Steve", 30));
people.add(new Person("Eddie", 42));
people.add(new Person("Mary", 32));

RuleBook ruleBook = ruleBookBuilder.addRule(RuleBuilder.create().withFactType(JsonObject.class)
                .when(person -> person.age < 40)
                .then(person -> person.getName())
                .build()).build();

        ruleBook.run(people);

-------OUTPUT------

Steve
Mary

It seems that if I want to put a List/Set of objects through a rule pipeline at the moment I always need to recreate a new Rulebook, is this right?

You do not need to create a new RuleBook each time you run it. RuleBooks can, however, change facts. So, if you rerun the same facts in a RuleBook multiple times, then you could certainly get a different state after subsequent runs.
Taking a modified version of your example:
   NameValueReferableMap<Integer> facts = new FactMap<>();

    facts.setValue("Steve", 30);
    facts.setValue("Eddie", 42);
    facts.setValue("Mary", 32);

    RuleBook ruleBook = RuleBookBuilder.create().addRule(rule -> rule.withFactType(Integer.class)
        .then(people -> people.entrySet().stream()
            .map(entry -> entry.getValue()) //map entries to people Facts
            .filter(person -> person.getValue() < 40) //filter out people Facts for those who are under 40
            .map(person -> person.getName()) //map people to their names
            .forEach(System.out::println))) //print the names of the people
        .build();

    ruleBook.run(facts);
    ruleBook.run(facts);

RuleBooks accept data as Facts, which can also be created or manipulated within a Rule. A Rule is provided all of the matching facts as an input. So, whereas the above example that I provided from your example does work, it's probably not the best use case for RuleBook. Something more akin to the MegaBank example, the POJO MegaBank Example or another scenario where there are different facts that determine whether or not an action should be taken is probably a better scenario.
Anyway, to your question about having to create a new RuleBook for each run. In the example I used above, the RuleBook I created is run twice from the same instance. In both cases the results are:
Steve
Mary
Thanks,
Clayton

Thanks will give this a try.


Concerns with sort behavior in the UI

Let's say I hit 'find traces' for a particular service name, with the other thing set to 'all'. Ok. The top 10 results come back... let's say I get some spans ranging from 4.5s to 1.0s. Cool.
A few seconds later, I go back to the main page, and I hit 'find trace' for the same service name. This time the top result is 40ms, not 4.5s. This seems fishy, and very unexpected.
I played around with it a bit more, and if I increase the limit to something large, then I can see the original spans that came back in my original query.
What gives? I had assumed that when I have sort selecting 'longest first', that it would return the longest spans in the given time range. However, it seems to be just picking LIMIT entries at random, then sorting those. That seems less than ideal -- particularly because duration queries never work on my single-node cluster (always times out).
Zipkin 1.6, cassandra backend.

Answering one part:
The UI element is only affecting the order of results client-side. It isnt
a server side filter.

trying to parse through your frustration.. sorry you are frustrated.
zipkin's api is a timeline one. results are always closest to the end time,
filtered by the criteria mentioned. Think of it like twitter itself. I
can't search twitter in reverse order and then by longest tweet.
Bottom-line, this isn't SQL: you can't do server-side sorts in arbitrary
order in all backends.
Anyway you ask for traces without putting a constraint on duration, there
is no constraint. They are ranked by timestamp descending on the server.
You can put a duration constraint, but in the case of cassandra, the
performance of such a query is not yet sorted
https://github.com/openzipkin/zipkin/pull/1204.
I think the biggest problem is the UI element is confusing things.. it
hints at a feature that doesn't exist.  It should be more clearly moved or
worded to indicate that it is for resorting results, not a server-side
ordering task. I may be wrong in this thinking.
ps. While the api is like twitter, unlike twitter, we don't have advanced
timeline controls https://dev.twitter.com/rest/public/timelines, nor a
thousand engineers. We are primarily limited by able hands and secondarily
by what's supported on mysql vs elasticsearch vs cassandra. IOTW help
wanted!

The frustration is mostly from thinking that it worked one way, and accidentally discovering that it worked another. :)
If the UI display is clearer about what one was actually doing, that would be a good start.

so i have to modify zipkin source codes  to add back end sorting and remove front sorting

so i have to modify zipkin source codes to add back end sorting and remove
front sorting

if you are able to do that across all backends, please do report back.

I'm not the only one who was confused.  Everyone where I work is confused when they learn that the sorting is on the front end.  It's unfortunate that the QueryRequest object doesn't even support the option of sorting on the back end.

the api doesn't support sorting. would it be less confusing if we removed
the client side sort option? we could make a config parameter to disable it.

we are switching the order to newest first which matches the server-side ordering #1926
however, we'll still have people surprised that we can't arbitrarily sort in all backends by fields such as duration. That's not something we have any good alternatives to, so we will try to address this in docs in #2488


Document invariant testing + patterns

https://github.com/foundry-rs/foundry/pull/1572
https://github.com/maple-labs/revenue-distribution-token/tree/e0eca03c2ff05c36000a097de678543d7234f7cc/contracts/test
document simple invariant breaking, the actor model from Maple's RDT, usage of targetContract(s) etc

Getting this done.

Thoughts on patterns:
Open Invariant Testing
This testing pattern deploys all relevant contracts in the setUp function and exposes all the functions in these contracts to the fuzzer. This pattern must use revert_on_failure = false in the foundry.toml, since this pattern will make calls with unbounded values.

Pros
Cons

- Checks all functions- Unbounded params
- Single actor- Reversions

Bounded Actor-Based Invariant Testing
This testing pattern deploys all relevant contracts in the setUp function and deploys wrapper contracts to call the functions on the contract. These contracts act as actors in a more realistic simulation of interactions against the desired contracts. These contracts can have relevant permissions and will perform calls with parameters bounded to reasonable values. This pattern must use revert_on_failure = true in the foundry.toml, since this pattern should expose any unexpected reversions over the course of the fuzzing campaign.
User contract reference: https://github.com/maple-labs/revenue-distribution-token/blob/be9592fd72bfa7142a217507f2d5500a7856329e/contracts/test/accounts/Staker.sol#L46
To take it a step further, a contract can be used to dynamically deploy new actors during the fuzzing campaign and perform actions from these actors: https://github.com/maple-labs/revenue-distribution-token/blob/be9592fd72bfa7142a217507f2d5500a7856329e/contracts/test/accounts/Staker.sol#L97
Another use case for this pattern is to create a contract that will call the vm.warp cheatcode at random intervals to simulate the passage of time and ensure that invariants are still held.
This pattern must add all non-wrapper contracts deployed in the setUp function into the excludeContracts array as to ensure that these contracts are only called through the wrapper contracts.

Pros
Cons

- Many actors- Can simulate passage of time- Can expose unexpected reversions
- Potential human error in incorrectly implementations of wrapper functions

Unbounded Actor-Based Invariant Testing
Similar pattern to above, except with unbounded parameters. This allows for the contract to be called without any artificially bounded parameters, potentially exposing previously unconceived vectors to break invariants.

Pros
Cons

- Many actors- Can simulate passage of time- Unbounded parameters
- Reversions

Hey @lucas-manuel can we get examples of the first and last example too please, thanks

Currently all the invariant examples we have are private, but here is an open source implementation of bounded actor-based invariant testing: https://github.com/maple-labs/revenue-distribution-token/blob/add-forge-invariants/contracts/test/Invariants.t.sol

When are Invariant tests expected to be released to the public? Looking forward to it. Thanks!

They are already released!

They are already released!

What!
According to the foundry book they are not yet supported. https://book.getfoundry.sh/forge/advanced-testing
Am I looking at the wrong version of the foundry book? Or, can I have a link to the documentation of invariant tests? Damn I'm excited!

https://github.com/foundry-rs/foundry/tree/master/testdata%2Ffuzz%2Finvariant

https://github.com/foundry-rs/foundry/tree/master/testdata%2Ffuzz%2Finvariant

And an example project where it is used?
Thank you very much!

Scroll up

There is no official docs for this still?

Soon :) For now take a look at https://github.com/lucas-manuel/invariant-example

take a look at https://github.com/lucas-manuel/invariant-example and https://github.com/foundry-rs/book/pull/760

Also maple-core-v2.

what is the best way of managing blockchain time in invariant tests?
So far I did a work around using a storage variable of the handler contract and a modifier that would warp time before each of the target functions, but I was wondering if there was a better way. Thanks

What does dictionary_weight mean and how does it affect the fuzzer performance? I can't find any docs related to it.

Could you guys please open new issues about these questions, for us to document them in the book?


feat: implementation of docker-compose file

[x] Create docker-compose file to run all services with docker build

Implementation done


path/filepath:  WalkDir func lexical ordering makes unexpected behavior with SkipDir

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

$ go version
go version go1.18.6 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/sudosapan/.cache/go-build"
GOENV="/home/sudosapan/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/sudosapan/gowork/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/sudosapan/gowork/"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/usr/lib/golang"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/golang/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18.6"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/sudosapan/work/code/ttt/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1453100211=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Searching for a file named z in root directory using filepath.WalkDir function. If the file is found in directory then skipping subdirectories with retrurning filepath.SkipDir
example directory, z is the file.
# tree a
a
└── b
    └── c
        ├── d
        │   └── z
        └── z

Code
filepath.WalkDir("./a", func(path string, d fs.DirEntry, err error) error {
  if !d.IsDir() && d.Name() == "z" {
    fmt.Printf("'z' file found %s skipping subdirectories\n", path)
	return filepath.SkipDir
    }
 return nil
})

What did you expect to see?
I expect program to find c/z and skip subdirectories of c such that d/z is not shown in results.
$ go run main.go
'z' file found a/b/c/z skipping subdirectories

What did you see instead?
Walk visits a/b/c/d/z before a/b/c/z resulting in output as below.
$ go run main.go
'z' file found a/b/c/d/z skipping subdirectories
'z' file found a/b/c/z skipping subdirectories

I understand this is due to filepath.WalkDir reading directory in lexical order and a/b/c/d/z occurs before a/b/c/z in lexical ordering.
My concern is that in such scenarios purpose of filepath.SkipDir is lost. One would use filepath.SkipDir to skip subdirectories based on some condition met in the parent directory, but if subdirectories are visited and presented to WalkDirFunc before parent directory, then the skip functionality can not be achieved.

SkipDir is for skipping the current directory,

I assume you mean skip current directory and subdirectories and contained files, and not just the current directory, because skipping current directory does not make sense after WalkDirFunc has been called.
Please see behaviour of same code with slightly modified directory and printing each path visited, file being a1 in this case.
tree a
a
├── a1
└── b
    └── c
        ├── a1
        └── d
            └── a1

filepath.WalkDir("./a", func(path string, d fs.DirEntry, err error) error {
    fmt.Println("Visiting %s\n", path)
	if !d.IsDir() && d.Name() == "a1" {
	    fmt.Printf("'a1' file found %s skipping subdirectories\n", path)
     	    return filepath.SkipDir
	}
    return nil
})

Output:
Visiting ./a
Visiting a/a1
'a1' file found a/a1 skipping subdirectories

As is visible that upon returning filepath.SkipDir, entire a directory is skipped including

a/b
a/b/c
a/b/c/a1
a/b/c/d
a/b/c/d/a1

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions

Am aware, posted it here as it looks like a gap behaviour.


Wrong font family

Hello,
my Toast is using the default browser font and not the roboto font. I am using this plugin in combination with vuetify. Did anyone have the same problem?

My fault. I did not see that vuetify is the problem. It sets the font-family only to the application class not the entire body by default.

My fault. I did not see that vuetify is the problem. It sets the font-family only to the application class not the entire body by default.

Can you please share how you did this? im experiencing the same thing.

How did you fix it?

I have simply added the necessary css inside the head in the index.html.
<style>
  .toasted {
    font-family: 'Roboto', sans-serif;
    font-weight: 300;
  }
</style>


Audit compile time

ArithmeticTools takes a considerable amount longer to compile than any other framework.

Looks like:
/// - returns: A `Set` of 0, 1, or 2 x-intercepts for the given coefficients.
public func quadratic (_ a: Double, _ b: Double, _ c: Double) -> Set<Double> {

    let discriminant = pow(b,2) - 4 * a * c

    guard discriminant > 0 else {
        return Set()
    }

    return Set(
        [1,-1].map { sign in (-b + sign * sqrt(discriminant)) / (2 * a) }
    )
}

adds a ton of time.
At ArithmeticTools:Quadratic:25:0.


random(range) should return integer instead of number when appropriate

random(1..10) produces a number value, not an integer as you might expect.

Hey @christianp where I find the files related to this issue.

Hi @Shiv1202, thanks for offering to help. This issue is actually fixed now, I just forgot to close it. Have a look at the issues labelled "good first issue" for something to get started on.


Update Node.js versions available on Azure Web Apps.

Hi @DavidRouyer, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Hi @DavidRouyer, looks like you need to do another sync with master before we can accept this PR.
Thanks,
Dave


Functionality

Is it possible to make separate volume buttons near power button and some kind of joystick buttons?
Now if you like to make volume up it like impossible with selecting that button from list with apps

No, it's not possible due to iOS and HomeKit limitations, but I'm waiting for iOS 16: I'll see what will be possible in this new context

Did not want to create another issue but my question is somewhat related. Is it possible to increase or decrease volume more than once? Right now when i click on VolumeDown it decreases by 1 and do not seem to have a way to reduce the volume further.


Map options camera target is not accepting LatLngBounds input

I'm submitting a ...

[ ] question
[x] any problem or bug report

OS:

[x] Android
[ ] iOS

Current behavior:
Snippet 1 is working :
const latLngBounds = new GoogleMaps.LatLngBounds([
	new GoogleMaps.LatLng(51.56, 10.38),
	new GoogleMaps.LatLng(41.15, -9.86)
]);

let mapOptions = {
	backgroundColor: "white",
	controls: {
		myLocation: false,
		myLocationButton: false,
		zoom: false,
		mapToolbar: false
	}
};
map = plugin.google.maps.Map.getMap(mapDiv, mapOptions);
map.on(plugin.google.maps.event.MAP_READY, () => map.moveCamera({target: latLngBounds}));

Snippet 2 is not working
const latLngBounds = new GoogleMaps.LatLngBounds([
	new GoogleMaps.LatLng(51.56, 10.38),
	new GoogleMaps.LatLng(41.15, -9.86)
]);

let mapOptions = {
	backgroundColor: "white",
	controls: {
		myLocation: false,
		myLocationButton: false,
		zoom: false,
		mapToolbar: false
	},
       camera: {
		target: latLngBounds
	}
};

map = plugin.google.maps.Map.getMap(mapDiv, mapOptions);

Expected behavior:
Both snippets are working.

let mapOptions = {
	backgroundColor: "white",
	controls: {
		myLocation: false,
		myLocationButton: false,
		zoom: false,
		mapToolbar: false
	},
       camera: {
		target:[
                   {lat: 51.56, lng: 10.38},
                   {lat: 41.15, lng: -9.86}
                ]
	}
};

Cheers for the tip, it's a way to initialize the camera as I need and I confirm it's working as expected.
Please note that snippet is working too :
let mapOptions = {
	backgroundColor: "white",
	controls: {
		myLocation: false,
		myLocationButton: false,
		zoom: false,
		mapToolbar: false
	},
       camera: {
		target:[
                        new GoogleMaps.LatLng(51.56, 10.38),
			new GoogleMaps.LatLng(41.15, -9.86)
                ]
	}
};

However, I'm not sure what I need to know with that issue: IMO, the snippet above show a defect so the issue should stay open while someone fix it (you or a contributor). What are your thought about that?

Is there any description the mapOptions.camera.target accepts LatLngBounds instance?

Not in the official doc, I agree, but is there any reason to have camera.target property working differently in a mapOption object than in a CameraPosition object ?
Moreover, in ionic-native, mapOptions.camera is a CameraPosition object :  https://github.com/ionic-team/ionic-native-google-maps/blob/master/documents/googlemapoptions/README.md
And a CameraPosition object accept LatLngBounds instance https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/v2.3.0/class/CameraPosition/README.md

Ah, the page is old.

Here is a tip.
Creating new LatLngBounds() for just camera is bad performance.
Because the maps plugin only copies lat, lng properties.
Creating new LatLng() as well.
If you want to fit the camera for one region but you don't know the sw/ne latlng pairs,
you just pass all {lat: , lng: } literals as array.
var bounds = [];
positionList.forEach(function(position) {
  bounds.push({
    lat: position.location.latitude,   // make sure value must be number
    lng: position.location.longitude
  });
});

map.moveCamera({
  target: bounds
});


feat: add tiflow code cache

replay test

https://ci2.pingcap.net/blue/organizations/jenkins/cdc_ghpr_kafka_integration_test/detail/cdc_ghpr_kafka_integration_test/14097/pipeline
https://ci2.pingcap.net/blue/organizations/jenkins/cdc_ghpr_kafka_integration_test/detail/cdc_ghpr_kafka_integration_test/14097/pipeline
https://ci2.pingcap.net/blue/organizations/jenkins/dm_ghpr_integration_test/detail/dm_ghpr_integration_test/8965/pipeline/37/


"Ugly hack" for BlueCoat proxy makes pasv connection fail on legitimate use in private network address ranges

FTP Server OS: Unix / Windows / Embedded
Linux but true for any
FTP Server Type: Vsftpd / glFTPd / BFTPd / ProFTPD / Pure-FTPd / IBM CS FTP (z/OS, OS/400) / Windows CE / Windows Server IIS / Vax / VMS / OpenVMS / Tandem / HP NonStop
Any
Client Computer OS: Windows / Ubuntu / Debian / Linux Mint / Arch Linux
Ubuntu but true for any
FluentFTP Version: ?
Framework: .NET 2 / 3.5 / 4 / 5 / 6 / 7 / 8 / UWP / Xamarin / Mono
6
<write details about your bug report / feature request here>
In GetPassivePort.cs line 75
// Fix #409 for BlueCoat proxy connections. This code replaces the name of the proxy with the name of the FTP server and then nothing works.
if (!IsProxy()) {
//use host ip if server advertises a non-routable IP
m = Regex.Match(host, @"(^10.)|(^172.1[6-9].)|(^172.2[0-9].)|(^172.3[0-1].)|(^192.168.)|(^<IP_ADDRESS>)|(^<IP_ADDRESS>)");
			if (m.Success) {
				host = m_host;
			}

This broke the connection of PASV as the host is not the one returned by the server !
At least enable this hack with a non-defaulted option in the config

Thanks for the friendly report.
I would greatly appreciate that you provide a nicely documented example of why/how

This broke the connection of PASV as the host is not the one returned by the server !

so that we can understand the reasoning behind your polite request:

At least enable this hack with a non-defaulted option in the config

Maybe in between you can also provide feedback on the we did for you in GnuTLS in your other issue, never heard from you again.

Allow me to edit the title of the issue. It is an "ugly" hack.

if (!IsProxy()) {
//use host ip if server advertises a non-routable IP
m = Regex.Match(host, @"(^10.)|(^172.1[6-9].)|(^172.2[0-9].)|(^172.3[0-1].)|(^192.168.)|(^<IP_ADDRESS>)|(^<IP_ADDRESS>)");

			if (m.Success) {
				host = m_host;
			}

This entire sequence is "ugly". it tries to detect the "server behind NAT" situation and switches to the originally used dnsname-or-ipad. I.e. "if the server responds with a local ipad", we should come in on the original  "outside" dnsname-or-ipad.
This already caused the #409 issue, so the code then was modified to only do this on a non-proxy situation. Which of course didn't change the ugliness of this.
There is a better way, possibly, which is used throughout. Comparing the original ipads type and comparing to the advertised one, slightly more complex but better.
Let me think about it, it would be better than a config.

Sorry English is not my native language.
De : FanDjango @.>
Envoyé : mercredi 10 mai 2023 18:09
À : robinrodricks/FluentFTP @.>
Cc : acorchia @.>; Author @.>
Objet : Re: [robinrodricks/FluentFTP] Hugly hack for BlueCoat proxy fail pasv connection (Issue #1258)
So a legitimate (working correctly) proxy inside a (big) internal network.
Allow me to edit the title of the issue. It is an "ugly" hack.
—
Reply to this email directly, view it on GitHub https://github.com/robinrodricks/FluentFTP/issues/1258#issuecomment-1542470698 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6RTYDWLREDMWIB6QCZ7NTXFO4RDANCNFSM6AAAAAAX42ZOUQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AC6RTYAOZGRFGNISRREPI3TXFO4RDA5CNFSM6AAAAAAX42ZOUSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS36A6CU.gif Message ID: @.*** @.***> >

Whatever the solution is, I really believe that such action should generate a log.
It’s very very complicate to debug when the ftp log say “Connecting to IP #1= ***:30671”
The *** does not let you understand the IP rewriting.
Actually Why do you hide that IP ?
Regards
Alain Corchia
De : FanDjango @.>
Envoyé : mercredi 10 mai 2023 18:34
À : robinrodricks/FluentFTP @.>
Cc : acorchia @.>; Author @.>
Objet : Re: [robinrodricks/FluentFTP] "Ugly hack" for BlueCoat proxy makes pasv connection fail on legitimate use in private network address ranges (Issue #1258)
if (!IsProxy()) {
//use host ip if server advertises a non-routable IP
m = Regex.Match(host, @"(^10.)|(^172.1[6-9].)|(^172.2[0-9].)|(^172.3[0-1].)|(^192.168.)|(^<IP_ADDRESS>)|(^<IP_ADDRESS>)");
                   if (m.Success) {
                           host = m_host;
                   }

This entire sequence is "ugly". it tries to detect the "server behind NAT" situation and switches to the originally used dnsname-or-ipad. I.e. "if the server responds with a local ipad", we should come in on the original "outside" dnsname-or-ipad.
This already caused the  https://github.com/robinrodricks/FluentFTP/issues/409 #409 issue, so the code then was modified to only do this on a non-proxy situation. Which of course didn't change the ugliness of this.
There is a better way, possibly, which is used throughout. Comparing the original ipads type and comparing to the advertised one, slightly more complex but better.
Let me think about it, it would be better than a config.
—
Reply to this email directly,  https://github.com/robinrodricks/FluentFTP/issues/1258#issuecomment-1542501428 view it on GitHub, or  https://github.com/notifications/unsubscribe-auth/AC6RTYAE7UX4XJQMPA55IZ3XFO7N7ANCNFSM6AAAAAAX42ZOUQ unsubscribe.
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AC6RTYERDSAIFCGWRB6HZ4DXFO7N7A5CNFSM6AAAAAAX42ZOUSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS36C2DI.gif Message ID: < @.> @.>

Whatever the solution is, I really believe that such action should generate a log.

I agree with that. It will be done.

It’s very very complicate to debug when the ftp log say “Connecting to IP #1= ***:30671”
The *** does not let you understand the IP rewriting.
Actually Why do you hide that IP ?

There a config option for logging, to hide user name, to hide password, to hide ipad, in case the user wants that, making the log information more confidential. You can turn clear text on in any combination, if you wish.
I am considering possible solutions, so please be patient about the PR you submitted.

Actually this is not really a proxy.
The customer has a load balancer with a front-end at .20 and 2 actual server at .19 and .18
This is more a question of redondances
Alain Corchia

From: FanDjango @.>
Sent: Wednesday, May 10, 2023 6:08:49 PM
To: robinrodricks/FluentFTP @.>
Cc: acorchia @.>; Author @.>
Subject: Re: [robinrodricks/FluentFTP] Hugly hack for BlueCoat proxy fail pasv connection (Issue #1258)
So a legitimate (working correctly) proxy inside a (big) internal network.
Allow me to edit the title of the issue. It is an "ugly" hack.
—
Reply to this email directly, view it on GitHubhttps://github.com/robinrodricks/FluentFTP/issues/1258#issuecomment-1542470698, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC6RTYDWLREDMWIB6QCZ7NTXFO4RDANCNFSM6AAAAAAX42ZOUQ.
You are receiving this because you authored the thread.Message ID: @.***>

Actually this is not really a proxy.

Yes, I corrected my text up above already. It is general problem for FTP PASV which is not solved well in the code.
Current solution makes the wrong assumption that client is coming in from outside a NAT, for example. This is true in many cases, but as you can see, not in all cases.

Using a config option is one popular solution, WinSCP does this for example. Give some more time to investigate other solutions to this. But the log message is of course independant, it is needed.

Command:  PASV
Status:   Waiting for response to: PASV
Response: 227 Entering Passive Mode (192,168,1,109,176,73). [12ms]
Status:   PASV advertised non-routable IPAD will be force-used (PASVUSE)
Status:   Connecting to IP #1= ***:45129

So thanks for reporting this. I have submitted a suitable PR to improve the PASV behaviour.
I'd like to make sure that this is seen by @robinrodricks - he might be able to suggest a better acronym than "PASVUSE", which is the first one that came to my mind, before merging the PR.


access denied on mysql server

here is my config file
##--------------------------------------------------------------------
## MySQL Auth/ACL Plugin
##--------------------------------------------------------------------

## Mysql Server 3306, <IP_ADDRESS>:3306, localhost:3306
auth.mysql.server = <IP_ADDRESS>:3306

## Mysql Pool Size
auth.mysql.pool = 8

## Mysql Username
auth.mysql.username = emqtt

## Mysql Password
auth.mysql.password = REDACTED

## Mysql Database
auth.mysql.database = mqtt

## Variables: %u = username, %c = clientid

## Authentication Query: select password or password,salt
auth.mysql.auth_query = select password from mqtt_user where username = '%u' limit 1

## Password hash: plain, md5, sha, sha256 bcrypt
auth.mysql.password_hash = sha256

## sha256 with salt prefix
## auth.mysql.password_hash = salt sha256

## bcrypt with salt only prefix
## auth.mysql.password_hash = salt bcrypt

## sha256 with salt suffix
## auth.mysql.password_hash = sha256 salt

## pbkdf2 with macfun iterations dklen
## macfun: md4, md5, ripemd160, sha, sha224, sha256, sha384, sha512
## auth.mysql.password_hash = pbkdf2 sha256 1000 20

## %% Superuser Query
auth.mysql.super_query = select is_superuser from mqtt_user where username = '%u' limit 1

## ACL Query Command
auth.mysql.acl_query = select allow, ipaddr, username, clientid, access, topic from mqtt_acl where ipaddr = '%a' or username  = '%u' or username = '$all' or clientid = '%c'

As you can see it has a user and password in the file and it is uncommented but for some reason it doesn't try to log into the sql database with them.  Below is what is in the log file.
2017-05-22 09:35:26.394 [error] <0.1289.0> CRASH REPORT Process <0.1289.0> with 0 neighbours exited with reason: no match of right hand value {error,{shutdown,{failed_to_start_child,emq_auth_mysql,{shutdown,{failed_to_start_child,worker_sup,{shutdown,{failed_to_start_child,{worker,1},{1045,<<"28000">>,<<"Access denied for user ''@'localhost' (using password: NO)">>}}}}}}}} in emq_auth_mysql_app:start/2 line 33 in application_master:init/4 line 134

What is invalid about it?