[Feature Suggestion] Option to locate missing projects

Is your feature request related to a problem? Please describe.
I have several harddisk and love (and need) to swap them once in a while for various reason. When my drive changed the letter placement however, all of the project will appears as missing.

Describe the solution you'd like
Would be nice if there is a Locate function for missing project, just like in Adobe Premiere Pro if you ever used one. When you right click a missing project and pressed locate, you will be presented by a file choosing window. You can search for a substitute folder. When a substitute is chosen, it will compare the metadata (like is the folder name is the same, the project.godot file, etc.) Once a single file is located, the software will also attempt to recover the path of every single project on that folder's root.
For example, let's say there are 2 missing project, F:\gproject\godot1 and F:\gproject\godot2. Both are now located in G:\newproject*. once you locate the godot1 project (now is located in G:\newproject\godot1, it will attempt to compare the folders inside F:\gproject and G:\newproject, and tries to re-locate all of them.

I have implemented this feature.  I've added a new Icon that will appear in the Action Area, when there are missing projects, that will let you browse to the folder that they are located in, and will update the database, with the ones it finds.  Any Missing Projects it doesn't find, will remain in a Missing Status, allowing you to update multiple projects, in multiple folders.
Screenshots:
Missing Projects, showing the new Yellow Scan Folder Icon

When pressed, a Browse to Folder Dialog popups.

When you located where the projects are now located, you Select the folder.

Godot will Scan through the folder, and find the missing projects, and update your list of Projects with the new information.

Let me know if that looks good, I will be pushing an update for Godot Manager in the next day or so.

Closing this as Fixed, in commits ccb6938..07b8039.


Suggestion: Websockets Improvement

While the Starlette implementation of websockets is well done I think it could use a decent wrapper. Something with some more ... batteries.
My proposal is simple at the moment but I think it would fit very well, Something along the lines of how routing works already. Essentially designed to be used as follows
from starlite import WebSocket, websocket

class Item(BaseModel):
    name: str

@websocket(path="/ws_path/{data}", data=Item)
async def my_websocket_handler(socket: WebSocket) -> None:
    async for message in socket:
        message.data

I figured this as one of the ways of implementation, as using it as a type hint in the function declaration implies it is meant to be used as a variable and that could be counter-intuitive. Although I have thought of several other ways this may be done. Such as:
Parsing whatever is in the string as the type, although would not be very friendly to use

@websocket(path="/ws_path/{data: Item}")
async def my_websocket_handler(socket: WebSocket) -> None:
    ...

Passing the data in as a variable, although a custom class would need to be created to hint
that it will be iterated over. As well as likely another version of starlite.WebSocket that does
not get iterated over or ditch the current version and make the socket var passed to the
function no longer iterated over thus making in these examples data is now what you iterate over
@websocket(path="/ws_path/{data}")
async def my_websocket_handler(socket: WebSocket, data: typing.AsyncIterable[Item]) -> None:
    ...

or
@websocket(path="/ws_path/{data}")
async def my_websocket_handler(socket: WebSocket, data: starlite.CustomIterClass[Item]) -> None:
    ...

This is in no way a final decision, just a proposition. I do plan to supply a PR (after review) because I love this project and just want to have my hand at helping.

Hi @Jabbey92.
First thanks for wanting to contribute.
As to what you are proposing above - I don't quite understand what does data do in path="/ws_path/{data}", is this supposed to be a url parameter?
We already parse and handle parameters (URL, query, header and cookie) for websockets - same as we do for REST http endpoints. See this part of the docs: https://starlite-api.github.io/starlite/usage/3-parameters/
as for using the data keyword in the websocket_handler function, i.e.
from typing import AsyncIterable

async def my_websocket_handler(socket: WebSocket, data: AsyncIterable[Item]) -> None:

Basically now the user needs to call websocket.receive_json() and then either use the dictionary as is, or pass it to a constructor. So this could be a substantial improvement.
You will need to extend the starlite.connection.WebSocket class, and also work on the kwargs injection etc.

My sincerest apologies, like I said in discord I was very distracted while writing this and never even referred to my code I had wrote about this a month ago.
Thank you for being understanding and not brushing me aside with such an off the wall issue. Allow me the chance to re-iterate my intentions, and ask some more appropriate quetions hopefully.
I want to start by re-defining my goal with this issue.
From a strictly objective approach, I think that this is a more intuitive way to use websockets.
from typing import Any, Dict
from starlite import WebSocket, websocket
from pydantic import BaseModel

class Item(BaseModel):
    name: str
    ...

@websocket(path="/ws_path")
async def my_websocket_handler(socket: WebSocket[Item, Dict[str, Any]]) -> None:
    async for item in socket:
        print(item.name)
        ...
        socket.send({"value": 3})

Question 1
Is it safe to assume that in most cases one ws endpoint will imply one data type transfer? In this case it would be receiving Item and sending a dict
Question 2
It doesn't seem necessary to make websocket.acccept() and websocket.close a requirement to explicitely use unless desired.

Either build them into the __aiter__
Or (probably more sound) make the websocket into a context manager, this way the __aiter__ functionality is required per say

Unless I'm not thinking of anything.

+1 from me for the async for pattern.
Let me know if I'm following correctly (BTW, websockets aren't a space that I work in, so if I'm misunderstanding anything, LMK).
The proposal is that WebSocket is generic on send and receive type? We'd validate and parse inbound data inside __aiter__ using the type specified via the generic class parameter so that iterating over the socket yields instances of the parsed type?
What does specifying the send type give us?
I remember going down a rabbit hole to find a reliable way to get access to generic type parameter values at runtime for something but my context has switched so far away from that stuff recently that I can't remember the specifics. This is in my search history: https://stackoverflow.com/questions/48572831/how-to-access-the-type-arguments-of-typing-generic - maybe it's not an issue, but just springs to mind.

@peterschutt
Thank you for the consideration!

What does specifying the send type give us?

I figure, if not for anything else, documentation purposes.
Also something I considered:
Another option for if multiple types are expected for one endpoint is to overload the functions. While I dont think a decorator style like I suggested earlier would work. But there are other ways to do this in pure python.

@Jabbey92, so the issue with what you propose is that our WebSocket class already accepts generics - WebSocket[User, Auth]- same as ourRequest` class.
I think the simplest way is to use the data keyword for the expected data type, or maybe alternatively add new keywords?

@Goldziher
So you're saying that there would be no way to change this? Why must it accept those generics? Otherwise there would be no real way for function documentation.
Maybe it can be a kwarg in the decorator?
@websocket(path="/ws_path", receives=Item, sends=Dict[str, Any])
async def my_websocket_handler(socket: WebSocket) -> None:
    async for item in socket:
        print(item.name)
        ...

Would this cause breaking changes?
Excuse me for any misunderstanding, I see this in the code, although I don't see it's use in the documentation.
The code get's a little hard to follow, where is it used? I looked briefly through both starlite and starlette code just not fully grasping it's purpose is all.

Just thinking, we could also give Websocket the functionality of being iterable.
It would also have 2 new properties: recieve_type and send_type with default values on initial creation.
Then create a separate class. Say ParsedWebsocket that returns a WebSocket with those new properties filled in. Thus fulfilling the type hinting within the function declaration. Also with the added benefit of backwards compatibility.
Example usage:
@websocket(path="/ws_path")
async def my_websocket_handler(socket: ParsedWebSocket[Item, Dict[str, Any]]) -> None:
    async for item in socket:
        print(item.name)
        ...

where is it used?

Do you mean the User and Auth generics to WebSocket?
The generics define the static type of the return values of the WebSocket.{user,auth} properties:
# connection.py
class WebSocket(StarletteWebSocket, Generic[User, Auth]):
    @property
    def app(self) -> "Starlite":
        return cast("Starlite", self.scope["app"])

    @property
    def user(self) -> User:
        if "user" not in self.scope:
            raise ImproperlyConfiguredException("'user' is not defined in scope, install an AuthMiddleware to set it")
        return cast(User, self.scope["user"])

    @property
    def auth(self) -> Auth:
        if "auth" not in self.scope:
            raise ImproperlyConfiguredException("'auth' is not defined in scope, install an AuthMiddleware to set it")
        return cast(Auth, self.scope["auth"])

    @property
    def query_params(self) -> Dict[str, Any]:  # type: ignore[override]
        return parse_query_params(self)

Here's where I'm at with all this. Firstly, no one is arguing that there aren't improvements to be made with to the websockets interface. However, there's lots of ways to skin the cat, and the devil is always in the detail.
First, I think we should have a minimal example application that demonstrates the current websockets pattern of use. We can do this as another example application inside the starlite-api org here on gh. It should be simple, and complete - i.e., anyone can easily clone the repo and follow some basic instructions in the readme to get the app up and running. Then we can use that app to try out different proposals and see what works and doesn't. Would you be interested in contributing to that @Jabbey92?

See:
https://github.com/starlite-api/starlite-websockets-hello-world
There is nothing websockets about it yet, just a pure clone of the other "hello-world" app in the org.

https://gist.github.com/whitecoop/157702ab76fb8243720fb318abd85341

@peterschutt

Would you be interested in contributing to that @Jabbey92?

Absolutely, I've forked it and working on a PR currently.


feat: Implement "rewrite" using Wasm Plugin

Ⅰ. Describe what this PR did
Implement rewrite strategy using Wasm plugin to gain a better extensibility.
This plugin functions as Rewrite Annotation.
The plugin model is as follows:
type RewriteConfig struct {
	rewriteRules []RewriteRule
}

type RewriteRule struct {
	matchPathType string // prefix | exact | regex
	caseSensitive bool
	matchHosts    []string
	matchPaths    []string
	rewriteHost   string
	rewritePath   string
}

The global configuration enables users to have more flexible rewriting policies (rules).
Ⅱ. Does this pull request fix one issue?

fixes #334
Ⅲ. Why don't you add test cases (unit test/integration test)?
I have used docker compose and WasmPlugin for local functional tests, and the test results are in line with expectations.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #339 (036879c) into main (625c06e) will decrease coverage by 0.07%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
- Coverage   42.01%   41.94%   -0.07%
==========================================
  Files          44       44
  Lines        5850     5850
==========================================
- Hits         2458     2454       -4
- Misses       3217     3221       +4
  Partials      175      175

see 1 file with indirect coverage changes


TFLite GPU failed to delegate with ssd_mobilenet_v2_fpnlite_320 model

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):  2.6.0
Python version: 3.9.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.2
GPU model and memory: GeForce GTX 1050Ti 4G

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I downloaded SSD MobileNet V2 FPNLite 320x320 pretrained model from TF Model Zoo , then used export_tflite_graph_tf2.py script to convert checkpoints to saved model.
python export_tflite_graph_tf2.py \
    --pipeline_config_path ssd_mobilenet_v2_fpnlite_320/pipeline.config \
    --trained_checkpoint_dir ssd_mobilenet_v2_fpnlite_320/checkpoint \
    --output_directory ssd_mobilenet_v2_fpnlite_320 \
    --config_override " \
            model{ \
            ssd{ \
              post_processing { \
                batch_non_max_suppression { \
                        score_threshold: 0.0 \
                        iou_threshold: 0.5 \
                } \
             } \
          } \
       } \
       "

Then convert saved model to tflite
converter = tf.lite.TFLiteConverter.from_saved_model(ssd_mobilenet_v2_fpnlite_320/saved_model)
converter.allow_custom_ops = True
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]
converter.target_spec.supported_types = [tf.float16]
converter.experimental_new_converter = True
tflite_model = converter.convert()
open(output_name, "wb").write(tflite_model)

However, when use NNAPI to delegate this model, it's failed with error :
INFO: Created TensorFlow Lite delegate for GPU.
ERROR: Following operations are not supported by GPU delegate:
CUSTOM TFLite_Detection_PostProcess: Operation is not supported.
DEQUANTIZE:
327 operations will run on the GPU, and the remaining 2 operations will run on the CPU.
ERROR: TfLiteGpuDelegate Init: PACK: Tensor "ssd_mobile_net_v2fpn_keras_feature_extractor/FeatureMaps/top_down/nearest_neighbor_upsampling/nearest_neighbor_upsampling/w_stack" has bad input dims size: 5.
INFO: Created 0 GPU delegate kernels.
ERROR: TfLiteGpuDelegate Prepare: delegate is not initialized
ERROR: Node number 329 (TfLiteGpuDelegateV2) failed to prepare.

Describe the expected behavior
TFLite NNAPI could delegate ssd_mobilenet_v2_fpnlite_320 model.
Contributing

Do you want to contribute a PR? (yes/no):
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

Hey @michaelnguyen11 , can you try this process with the non-FPN SSD version?
Also, you seem to be using the GPU delegate (as shown by your logs), and not the NNAPI delegate - and for float16 models, GPU is indeed the correct delegate - just wanted to confirm.

Hi @srjoglekar246 ,
I'm using the SSD Mobilenet V2 and it's working well (used TF1 to convert to TFLite). And now I want to use FPN SSD versison.
Yes I'm using GPU delegate. Both float16 and float32 have the same issue :
INFO: Created TensorFlow Lite delegate for GPU.
ERROR: Following operations are not supported by GPU delegate:
CUSTOM TFLite_Detection_PostProcess: Operation is not supported.
154 operations will run on the GPU, and the remaining 1 operations will run on the CPU.
ERROR: TfLiteGpuDelegate Init: PACK: Tensor "ssd_mobile_net_v2fpn_keras_feature_extractor/FeatureMaps/top_down/nearest_neighbor_upsampling/nearest_neighbor_upsampling/w_stack" has bad input dims size: 5.
INFO: Created 0 GPU delegate kernels.
ERROR: TfLiteGpuDelegate Prepare: delegate is not initialized
ERROR: Node number 155 (TfLiteGpuDelegateV2) failed to prepare.

ERROR: Restored original execution plan after delegate application failure.

Is CUSTOM TFLite_Detection_PostProcess supported by GPU delegate?

Hi @michaelnguyen11 !
It looks like you are doing the inference in a desktop with Ubuntu OS.  NNAPI and GPU delegates performance meets expectation when inference are done in  Android/IOS  devices.
For Desktop purpose, You can inference with default CPU/XNNPack  delegate.
Please do a benchmark test in android/Desktop machine to see the difference in performance and find the best delegate for your use-case.
Thank you!


Inability to edit map/see markers being currently edited

I have a current map that I have spent a substantial amount of time on as well as using it for my community, for some reason when I use the link that goes with the map it shows the name of the map but none of the markers are their and it takes almost 10 minutes to load I cannot sign in either because it gets stuck on the sign in page. I tried to narrow it down to being just an issue with me but I have had around 12 people check and they are having the same issue.

Hi, server had performance issues. It's resolved


Add the roles package as a dependency

We need some sort of concept of roles to have groups that are allowed to login when an app is in maintenance mode. This should work; https://atmospherejs.com/alanning/roles

#8


Explore: If there is a way to locally browse PVCs in filesystem

Why it needs to get done
We want give the user the ability to browse the files stored in PVCs.
What needs to get done
Research the options for browsing files in PVCs in local file system.
When is the task considered done
Find a tool for browsing files in PVCs.

In our proposed solution we want to mount the same PVC to every notebook server. This means that the user can share the notebooks and data between servers. To edit the contents of the pvc we need a UI which mounts the volume and allows this functionality.
In kubeflow we use https://github.com/canonical/pvcviewer-operator. Using this in DSS would require us to enable istio in the cluster. For now this is not ideal.
For now we decided to go without the separate charm for viewing pvcs. Users can update the content of pvc just by using notebook servers.


QMK Setup

How would you go about using this in a QMK keymap?
Your README seems to suggest that QMK already has support but I can't seem to find any reference to it in QMK itself.

You need to use my plover hid branch of qmk. I haven’t opened a pull
request yet.
Then you need to set PLOVER_HID_ENABLE = yes in your rules.mk and set up
a keymap with PLV_XX keycodes (after including plover_hid.h).
On Fri, 5 Nov 2021 at 17:22, Xervon @.***> wrote:

How would you go about using this in a QMK keymap?
Your README seems to suggest that QMK already has support but I can't seem
to find any reference to it in QMK itself.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dnaq/plover-machine-hid/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ5KBJZ5REW36HP2DYEJODUKQAD7ANCNFSM5HOGXIOA
.

I hadn't seen that branch. Thanks.

Posting the zmk link for posterity: https://github.com/dnaq/zmk/tree/plover-hid-hog-attempt


Error after installing 0.4.4-alpha

Hi, maybe you can take look at it:
2022-09-25 04:56:58.434 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration solarwatt_energymanager which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-09-25 04:57:34.221 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry EnergyManager ERC04-100005983 for solarwatt_energymanager
File "/config/custom_components/solarwatt_energymanager/init.py", line 19, in async_setup_entry

I just tested installing the integration in a fresh HomeAssistant setup, and it is working for me. Can you try again, or maybe provide more info?

Hi, I completely deleted it and reinstalled. Now it´s working fine, again. Thanks a lot.


Natchez log4cats

This is a small integration between Natchez & log4cats to allow us to pull headers from the Natchez kernel into the log4cats MDC so that in Datadog logs can be correlated with metrics
I didn't write any tests since this is so small, please forgive me!

Nice one!

I didn't write any tests since this is so small, please forgive me!

I'm not sure if I'll be able to forgive you that 😉

cc @ovotech/engagement-team


tools/i18n: Unescape HTML and Unicode sequences during sync-translations; sync all latest translations from Transifex

We've previously noticed ^1 some JSON translation files ending up with escaped Unicode sequences on disk, which Transifex indicates is expected behavior ^2, though it is sometimes fixed by manage.py compilemessages ^3. Further, as noted in #23932 [^4], some JSON translation files include HTML-escaped entities like quotation marks.
This script will ingest valid JSON files and output them as proper UTF-8 files with appropriately unescaped (unless otherwise necessary, like double quotes being backslash-escaped) sequences, except when the key itself contains HTML escape sequences (as it's presumed the value of such entries must be pre-escaped before being passed to consumers).
[^4]: Which is not end-to-end fixed yet by this commit: that will require a new release of Zulip Server.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Several JSON files were not unescaped in this run because their contents are unparseable by json.decoder. I've hand-fixed one of these files in the past and assume the issue is likely the same here: in Finnish's case, there were unescaped quotes on the Transifex side which get pulled verbatim; their CLI seemingly (previously?) passed through such invalid JSON. GitHub Gist of this run's log; I can go through and hand-wrangle those tomorrow, whether we think that should go into this PR or a quick-follow (or if we simply have no preference and want to base it on which task gets done first, review of this PR or JSON-wrangling) is somewhat up to Tim/Alex/etc.

Per the stopgap process discussed on CZO I did the following today which in effect allows this PR to supercede #23761 as well:

Synced unbranched translations from Transifex
Ran HTML+Unicode unescape script over them
push-translations
Manually fixed unescapes in Transifex UI for mostly the Romance languages, notably '#&39;' (single quote) needed fixed all over the place
Synced 6.x branch translations from Transifex
Ran HTML+Unicode unescape script over them
Manually audit each locale for missing data

This means the following locales were only updated as far as Step 4, as the 6.x sync resulted in several strings losing translations entirely and warrant further future investigation (with the goal of unblocking 6.1 release):

ar
si
zh_Hans

I believe this is now ready for a final merge review.

A few comments:

This PR is to main, so we should just sync the translations for main to it. We will need to do a separate PR backporting this to 6.x, and for that PR, we'll be running the commands with -b 6.x in order to get translations for that branch.
I am not sure I understand what the plan would be going forward. I think we might want the "Unicode unescaping" part to just be part of sync-translations, in that we want a canonical format there always. But I think the "HTML unescape" part is potentially dangerous when unsupervised, and ideally we want to just be doing on a one-time basis as part of fixing the "Machine translation" mess. Does it make sense to split the tool?

Cool, I can split the 6.x-related commits off to a local "stash branch" we can cherry-pick into a backports PR.
I'll open a discussion in CZO about some thoughts I have on branching in git vs branching in translations-land; those questions and opinions in no way need to block any ongoing work, and are more forward-looking.
Adding a --html flag to the existing CLI tool is completely reasonable to enable unescaping HTML only when called by the "sidebar" script, and I can copy the find incantation (without --html being passed to the new Python script) into sync-translations to get us to that world you just described (which I think is a well-reasoned world).

Merged after the small tweak noted above. @klardotsh one small note -- there's generally no need to use gitlint-ignore; we don't run gitlint in CI at all, and it's a bit noisy (for example, it doesn't allow URLs in commit messages due to line length), so I'd just ignore its alerts that you disagree with. (I wish we could configure it better).
That said, I did shorter the commit message title for the first commit before merging, since it was too long in a way that GitHub and terminals line-wrap in an ugly way.
Thanks so much for debugging this! I'll assume you'll port the small tweaks to the backported copy on the 6.x branch.


updated mongodb (3.2.8-build.1)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as b9c8fbfea85cdfda3dbf163f0895f13994498c8b.


Implemented support for RabbitMQ broker-initiated Basic.Cancel

Implemented support for RabbitMQ broker-initiated Basic.Cancel
Implemented unit and integration tests for RabbitMQ broker-initiated Basic.Cancel.

Hi @awestendorf, I completed my work on the RabbitMQ-specific broker-initiated Basic.Cancel, including unit and integration tests. Please review.
I also refactored scripts/rabbit_extensions into a unittest-based framework and added an integration test for the broker-initiated Basic.Cancel feature: https://github.com/vitaly-krugl/haigha/blob/consumer-cancel-notify/scripts/rabbit_extensions.
Best,
Vitaly

@awestendorf, would you mind taking a look at PR #74 as well? It has several stability fixes as well as test fixes. Thank you.

Hi Aaron, I implemented integration tests for broker-initiated Basic.Return and and Basic.Cancel temporarily under scripts/integration/. Please review. When you create the integration directory under haigha/tests/, I can move these tests there prior to merging of this PR.
To be precise, I refactored scripts/rabbit_extensions into unittest-based test and implemented the broker-initiated Basic.Cancel test along with my original PR. I subsequently made the following changes:

Created scripts/integration/channel_basic_test.py for Channel.basic integration tests and implemented test_unroutable_message_is_returned.
Moved scripts/rabbit_extensions.py to scripts/integration/rabbit_extensions_test.py for RabbitMQ-specific integration tests, removed option parsing logic, and implemented additional test test_unroutable_message_is_returned_with_puback.

@awestendorf: I added rabbitmq service to .travis.yml and moved scripts/integration/ to tests/. Awaiting travis test results now. F.Y.I.

@awestendorf: the integration tests are now getting tripped up by "TypeError: getsockaddrarg() takes exactly 2 arguments (4 given)" when connecting to RabbitMQ broker. This should be addressed by @kevinconway's PR #69.
Aaron, I reviewed #69, and found only a few superficial issues with it. It should work as-is. Once you accept #69, I will rebase my Basic.Cancel PR against master.

@awestendorf: as expected, the combination of this PR's changeset and #69 pass; see TEST PR #86 for test results.

Hi Aaron, everything is in place for this PR, as you requested (integration tests and all). We've been running with the Basic.Cancel change in production problem-free for several months now. This PR now depends on @kevinconway's PR #69 in order to pass the integration tests (due to RabbitMQ having an IPv6 address on Travis). This is demonstrated by my passing test PR #86 that combines this PR with #69.
I think that both PRs (this and #69) are now ready to make the leap into master and release. Is there anything else that needs to be done? Thx.

@vitaly-krugl thank you, sorry for the delay. Between work and summer vacation these threads fell off my radar. I haven't been able to exhaustively test myself, but I reviewed a long time ago and I think everything is in place to ship these features.

Thank you Aaron!


remove marker for missing data at Lineplot Viz

This will address https://github.com/VEuPathDB/web-eda/issues/1411.
After extensive tests and check codes, I found that Bob (I guess) nicely implemented nullZeroHack() function not to connect lines for missing data and the processed data was passed through the function. The request by the ticket is to remove those data points, thus the simplest solution is not to use the function.
Here is the screenshot for the same example in the ticket. Clearly, data points for Year 3 and Year 4 are now gone as compared to the screenshot in the ticket (https://github.com/VEuPathDB/web-eda/issues/1411#issue-1409463546).

Sorry for the belated review. Should there also be a line connecting Year 2 and Year 5? Maybe @danicahelb can weigh in on this?

@dmfalke I think so because there exists another ticket to do that https://github.com/VEuPathDB/web-eda/issues/1519 😄

@dmfalke thank you for your review and approval! 👍


Revert "Disable CI run on Windows"

49ac9ef says:

Disable CI run on Windows
Apparently Microsoft introduced a bug Visual Studio's distribution.
More at:
https://github.com/microsoft/STL/issues/1300
Revert this when things get better.

I am pleased to report that things have gotten better (https://github.com/microsoft/STL/issues/1300#issuecomment-789542493) and excited to provide you with a revert. :)
This change passes CI for me: https://github.com/trevyn/croaring-rs/runs/2139234443
xref: https://github.com/mimblewimble/grin/pull/3596

Great news, thanks!


Reflection.ReflectionError with iOS 12 and the new Xcode beta

Hey when I try to use your library with the new Xcode 10 beta 2 and iOS 12 beta I have following issue:
protocol ITranslation: Value {
    var example: String { get }
}
class TranslationBase: ITranslation {
    var example = "example"
}

let key = "example"
do {
    if let translationString = try translations.get(key: key) as? String {
          print(translationString)
    }

} catch let error {
    print(error) // Reflection.ReflectionError unexpected
}

Do you have an idea how to solve this issue?

https://github.com/Zewo/Reflection/issues/15


License?

I came across this project and found it potentially useful. However, it appears that there is no license information provided. What is the licensing of this project? If it has an appropriate FOSS license, please add the license information so that users can use this project legally. Perhaps releasing this project under Public Domain might be a good idea.

Thank you, @bc-lee, for the heads up. I've added a license file (MIT).


test: introduce go.uber.org/goleak

Just introduce go.uber.org/goleak to prevent an unexpected Goroutine leaks.

We need to drop supports for old Go versions...


[bug] Support passing in api_key from kwargs

Describe the bug
Currently, Guardrails assumes API key is configured from environment variables. Additionally, users should be able to pass in API keys from kwargs. As an example:
guard.parse(
    llm_api=openai.ChatCompletion.create,
    api_key=my_api_key,
    api_base=my_api_base,
    model=my_model,
    temperature=my_temperature,
    ...
)

Addressed in https://github.com/ShreyaR/guardrails/pull/266
See PR description on future improvements.


feat: add ipfs on path option

https://github.com/ipfs-shipyard/ipfs-desktop/pull/906
License: MIT
Signed-off-by: Henrique Dias<EMAIL_ADDRESS>

@olizilla can we get this merged ASAP? 😄 Thanks


Disallow intersection of exact objects

Proposal
Intersections of exact objects does not make sense since they produce impossible type that can't be used. {| a: number|} & {| b: string |}produces an object type that only has property a and nothing else, but also has property b and nothing else. This indicates that intersections can only be used for inexact objects.
In optimal case the error message would be describing and maybe communicate why intersection of exact objects can't be used and suggest to use spreading instead.
This is important especially for newcomers and people who are used to write TS. I find my self very often explaining how and why intersection of exact objects can't be used.
I understand that this might not be big deal for FB, but for community it's important to make experience good for newcomers.

There are corner cases where the intersection type is not empty:
type A = {|
  foo: number | string,
|};

type B = {|
  foo: bool | string,
|};

let x: $ReadOnly<A&B> = { foo: 'a', };

or
type A = {|
  a?: number,
  c: number,
|};

type B = {|
  b?: number,
  c: number,
|};

let x: $ReadOnly<A&B> = { c: 0, };

($ReadOnly prevents adding properties to x that would make the object no longer conform to its intersection type)
But I agree that warning against empty intersections of exact types would be useful in most cases and to a lot of people.

@SillyFreak Interesting, hadn't thought about those cases! Anyhow, do you think there would be any real world use cases with them that couldn't be achieved easily in other ways?

real world use cases I'm not sure, but disallowing intersection types in this specific circumstance, because there are workarounds for when this is actually the desired behavior seems like the wrong approach. Maybe a lint would make sense, but disallowing this seems too harsh.
In principle it should be possible to determine when the resulting type is empty, and disallowing such types (including number & string) would IMO make more sense than the "heuristic" of exact object intersections. But even then, empty types are not entirely useless. Take this function:
function throws(): void { throw new Error(); }

void is not the most specific type that I could use here: void's only value is undefined, but this function never returns undefined. I could specify number & string as well.


Switch to async storage (i.e. indexdb instead of localstorage)

Currently synchronous localstorage is used, which is fine for most things apart from saving history which can grow very large and causes significant slow-down, despite the being debounced to 350ms.
This is most noticeable when editing/creating complex line/polygon features, which result in the history being large, and writes to localstorage taking significant time, locking up the browser every 350ms. iD quickly becomes unusable in this scenario. This can be solved by frequent committing, but it is not an ideal solution.
To avoid a larger re-factor which would necessitate making all storage calls async, perhaps the quickest/easiest solution would be to use async storage for only history.save().
IndexedDB is the obvious choice. Any recommendations for a lightweight wrapper that gets around any IndexedDB bugs or lack of support on browsers currently supported by iD (looking at you, Safari)? We only need a wrapper that implements basic get() and put().

One option: https://github.com/localForage/localForage
Thoughts welcome.

@bhousel where is the list of browsers that iD supports? Couldn't find it but I'm sure I've seen it somewhere. So that we can ensure that any storage option has the same compatibility.

where is the list of browsers that iD supports?

@gmaclennan We should list this.  It's the ones I wrote in #3234
Chrome, Firefox, Safari, IE11, Edge
We also have some legacy code to support Opera pre v15 when they switched to the Blink rendering engine.  Current versions of Opera are essentially Chromium.
These browsers are all auto-updating now except for IE11, so I don't think too much about specific version numbers.
Safari is currently the crashiest because of #2913

@gmaclennan If you're interested in working on this, the work would be similar to what I did recently for #3236.  Steps are:

Add the library to package.json as a dependency
Where you want to use the library, use an es6 import
You'll want to update this block of code in id.js that manages the localStorage.   It probably makes sense to move this code into a module like modules/util/storage.js and add some logic to fallback to localStorage if newer methods return nothing (just so that nobody loses stored history in the switchover).

Are you going to be at SOTM-US this year?   I'm looking for ideas for Monday's hackday so this is something I can add to the list and we can work on it together..

On more than one occasion, I’ve run into Firefox’s LocalStorage size limit, causing me to lose my changes when the browser crashes or iD gets into a weird state. IndexedDB apparently has a much higher size limit, which would be great for large changes.
In the meantime, if anyone runs into the LocalStorage quota in Firefox, these steps worked for me:

Open about:config and increase dom.storage.default_quota.
Run the following commands in the console on the page containing iD:id.history().checkpoint();
id.history().save();

This sets an entry in LocalStorage with a key based on the domain, e.g. iD_https://www.openstreetmap.org_saved_history.
Typically, at this point, you can close the tab and open iD in a new tab, and your changes will be ready to restore. However, if you don’t want to risk losing your only copy of the changes, you can issue one final command in the malfunctioning instance of iD:
id.history().unlock()

This will make it more likely that the new iD instance will offer to restore your changes.

See notes on #4418 for where this ended up.   Switching to async storage has definite benefits, but would introduce a breaking change in how iD starts.. So this will eventually land whenever we do an iD v3.


SPConfig loader instead of global django settings

fix https://github.com/IdentityPython/djangosaml2/issues/280: better using pysaml2 SPConfig loader instead of global django settings!

The change looks good on our side. All our liveserver tests passed without the need of SAML_CONFIG in settings.
Thanks again for your time!

Your time Is more precious than mine, take a look at the code and feel free to pur your claims and ideas, I'll follow you


Fix download links

Also, it seems like it would be nicer to copy the executable in to $HOME/bin instead of /usr/local/bin, and get rid of the need for sudo.


As a developer I want to start my build process without depending on SCM operations from a CI system.

Background
CI executors tend to provide support for SCM operations to check out a repository at the start of a build process.
For example in Jenkins these operations can be configured in the configuration and/or the versioned definition of a pipeline. In github actions, the checkout  action tends to be used.
It would be good if MPyL can do this independently.
Because:

it makes MPyL more self supporting and easier to plug into any CI build executor
we don't require potentially complicated logic to determine the state of the locally checked out repository
MPyL is opinionated about how PR or tag builds should be done. This reduces the amount of checkout scenarios that we need to support. We can keep it simple.

Requirements
We need authentication for the remote repository to be provided in any of the following ways:

via an initialized folder on the local file system
via an ssh key
via username / password combination supplied in env vars

A local workspace directory needs to supplied.
PR builds

We need to know the PR number and corresponding remote ref
All revisions since the base branch need to be present locally
A limited checkout depth to minimize the duration of the checkout phase. Perhaps git clone --shallow-exclude main --single-branch --branch feature/branch https://github.com/Vandebron/mpyl.git could do the job.

Tag / master builds

The relevant merge commit needs to be present. This provides enough enough information to determine the build set.

In github actions (and most other CI runners I presume) there’s no reuse of build executors. So you start with a clean workspace every time.
I think it’s generally a good idea to develop MPyL with that kind of mindset. Nothing is cached implicitly. We can cache images in a docker registry, metadata in a repo, libraries in a GHA cache (https://github.com/actions/cache), etc.


Update baseline profile configuration per build type

This is changed in the following PR https://github.com/android/nowinandroid/pull/1060/files .. maybe just close this one?


Min commission change validation change ignore

Codecov Report

Merging #363 (1e6d012) into main (cfcaab5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #363   +/-   ##
=======================================
  Coverage   18.59%   18.59%
=======================================
  Files         134      134
  Lines       21533    21533
=======================================
  Hits         4003     4003
  Misses      16865    16865
  Partials      665      665

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 cfcaab5...1e6d012. Read the comment docs.

Closed in favor of the must change commission rate approach.
The scripts were super helpful though! (Minor bug, need to change osmosisd tx lockup unlock-by-id to begin-unlock-by-id)


Wrong status for Services

I have this Service:
apiVersion: v1
kind: Service
metadata:
  name: ...
  namespace: ...
spec:
  clusterIP: [HIDDEN]
  externalTrafficPolicy: Local
  healthCheckNodePort: [HIDDEN]
  ports:
  - name: http
    nodePort: [HIDDEN]
    port: [HIDDEN]
    protocol: TCP
    targetPort: http
  - name: https
    nodePort: [HIDDEN]
    port: [HIDDEN]
    protocol: TCP
    targetPort: https
  type: LoadBalancer
status:
  loadBalancer:
    ingress:
    - ip: [HIDDEN]

The type is LoadBalancer, spec.clusterIP is not empty,  status.loadBalancer.ingress is not empty and all objects in status.loadBalancer.ingress has an IP that is not empty.
I expect the status to be ready, but it is in progress.

/remove-lifecycle stale

/remove-lifecycle stale

I don't think the Service object is maintained by the application SIG. @rstefan1 -- maybe move this issue to a different project?

You're right @stribb, but the application controller takes services status into account.
But in some cases the logic is currently wrong, for example a LoadBalancer type service is expected to have an empty spec.clusterIP. So a Service that is ready will be reported as being in progress instead.
I've fixed this case, and other cases of services being falsely reported as ready, here.

/remove-lifecycle stale


[BUG] Plugin does not work

Describe the bug
Plugin simply doesn't work, it doesn't respond to the commands I created.
Logs I receive:
2024-04-21 13:12:37.770 -03:00 [INFO] plugin:CustomCommands CustomCommands loaded!
2024-04-21 13:12:37.792 -03:00 [INFO] plugin:CustomCommands Found default config file.
2024-04-21 13:14:46.573 -03:00 [INFO] plugin:CustomCommands CustomCommands loaded!
2024-04-21 13:14:46.593 -03:00 [INFO] plugin:CustomCommands Found default config file.
2024-04-21 15:40:47.478 -03:00 [INFO] plugin:CustomCommands CustomCommands loaded!
2024-04-21 15:40:47.501 -03:00 [INFO] plugin:CustomCommands Found default config file.

Plugin Version: CustomCommands" (2.0.3) by HerrMagic
CSSharp Version: v213 (1.0.213+dfc9859)
Metamod Version: 2.0.0-dev+1286

Config of the file I use would be this: Commands.json

Hey try the lates release.
https://github.com/HerrMagiic/CSS-CreateCustomCommands/releases/tag/v2.1.0

Same thing, still doesn't work. I use the commands and have no response.

Ok, than I will need to look into that problem in the next days couple of days. But I dont have time right now to fix this.

I will notify you when its fixed

Hey i have tried the version and it works for me. I have no idea why this happens. Maybe try to remove the plugin and add it in again.

Strange, I'm going to do a clean install with the latest versions of CSSharp & Metamod and the plugin. After testing everything, I'll come back here with the results and more logs if necessary.

I tested again, everything is fine now.


New release of 5.0.0?

Would it be possible to get a new release of 5.0.0?  It's been almost six months, and there are some changes since then I'd really like to be able to use without having to build okhttp myself.
Thanks

@scottkennedy How do you build okhttp yourself?

Any updates|plans for this?

Hi Esteemed Developers of okhttp,
Is there any timeline on when v5 of okhttp would be moved out from alpha to stable?

(We’re making a bunch of changes to new APIs, and we don’t have them finalized yet.)

Would it be possible to get a new release of 5.0.0?

seems to be done from looking at https://github.com/square/okhttp/blob/master/CHANGELOG.md ?


Support multi node proxmox cluster

current CAPP supports only single node proxmox cluster.
should support multi node proxmox cluster as well.

Do you have any ideas so far?
My first thoughts would be to use this endpoint https://pve.proxmox.com/pve-docs/api-viewer/index.html#/nodes/{node}/status to get cpu/ram and implement a "scheduler" which decides where to place the node.
Maybe we could introduce a field called nodeName in ProxmoxMachine similar to the default Pod resource. https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/pod-v1/#scheduling
If it`s not set the scheduler decides.
That's just an easy idea i`m aware that it may comes more difficult or require more refactoring.

I do not know if its allowed to introduce new fields in a cluster-api-provider. If not i think failureDomain could be used for that not perfect but in fact if you have a proxmox-cluster without HA one node is a kind of a failureDomain. 😅
But personally i would like to use (warning: big thinking) failureDomain as an abstraction for a whole proxmox-cluster so it would be theoretically possible to stretch a kubernetes-cluster over many proxmox-clusters aka. multi-cloud use-cases or just real AZs which are in different physical locations.


Unable to use GENERIC_PACK with url parameters

I am using urls with very long parameters (like this one) to download worlds etc. When trying to download a generic pack, curl fails with curl: (23) Failed writing body (0 != 1377). I think this is because the script tries to download the file to the basename of the url in this line:
https://github.com/itzg/docker-minecraft-server/blob/0f7bd5f4fd58cced5b879a54cc6019aac5557d73/start-finalSetupModpack#L144
Unfortunately baseline doesn't remove url parameters which results in a filename that is too long.
I'm not sure what the best approach for a fix is. Should the parameters be stripped or should the generic pack be downloaded to a fixed filepath like the world file?

Oh yeah, that approach is not robust at all for URLs with any query parameters. I'll explore a little bit because I think there might be a way of addressing that as a curl argument.

The latest should now fix this. Can you pull the latest image and try again with the generic pack URL?

Thank you! I just noticed that this also happens when using the MODS var to download jars. Error message is Must be HTTP or HTTPS and a JAR file

What is an example of a MODS value that didn't work? That bit of code a little bit different, so it'll need to be addressed differently.

When setting the MODS value to a url with parameters the script fails with [init] ERROR Invalid URL given in MODS and [init]       Must be HTTP or HTTPS and a JAR file.

Latest image should now also fix MODS handling of URLs with query parameters.

Yes this issue seems to be fixed. Thank you very much!


Workaround for #538 (silent failure when buying on Bitstamp)

I ran into issue #538 and after some research I found that:

The bitstamp.js file didn't handle errors coming from the server correctly
and
It failed because it was trying to buy too much. As a workaround I have decreased the amount to buy by 1% but obviously the original calculation goes wrong somewhere.

Could you please PR this against develop? I am not sure if Gekko is compatible with the new version of the bitstamp npm module.

Done, but please consider addressing this in the main branch as well as Gekko is currently unusable on Bitstamp (for me anyway).


Support getting lineage for ArgoCD Applications

This is a wonderful and promising tool.
I have seen a note that currently it does not support ArgoCD Applications. When I tried running on it, I could just see the events generated by these applications instead of the actual child objects. I would like to enhance this tool to add support for ArgoCD applications as well.
argocd app create guestbook --core --repo https://github.com/anandf/argocd-example-apps.git --path kustomize-guestbook --dest-namespace guestbook --dest-server https://kubernetes.default.svc --sync-policy automated --self-heal --sync-option CreateNamespace=true
application 'guestbook' created
argocd app list --core
NAME              CLUSTER                         NAMESPACE  PROJECT  STATUS  HEALTH       SYNCPOLICY  CONDITIONS  REPO                                               PATH                 TARGET
argocd/guestbook  https://kubernetes.default.svc  guestbook  default  Synced  Progressing  Auto        <none>      https://github.com/anandf/argocd-example-apps.git  kustomize-guestbook

On running lineage command, I get this output
 k lineage application guestbook
NAME                                   READY   STATUS                                                                                AGE
Application/guestbook                  -                                                                                26s
├── Event/guestbook.17f5a5a5b1bdf768   -       ResourceCreated: Unknown user created application                                          26s
├── Event/guestbook.17f5a5a5fa724984   -       OperationStarted: Initiated automated sync to 'd7927a27b4533926b7d86b5f249cd9ebe7625e90'   25s
├── Event/guestbook.17f5a5a5fa90d1f4   -       ResourceUpdated: Updated sync status:  -> OutOfSync                                        25s
├── Event/guestbook.17f5a5a5fabf4424   -       ResourceUpdated: Updated health status:  -> Missing                                        25s
├── Event/guestbook.17f5a5a67f31db2d   -       ResourceUpdated: Updated sync status: OutOfSync -> Synced                                  23s
├── Event/guestbook.17f5a5a67f483a16   -       OperationCompleted: Sync operation to d7927a27b4533926b7d86b5f249cd9ebe7625e90 succeeded   23s
└── Event/guestbook.17f5a5a67f551fb2   -       ResourceUpdated: Updated health status: Missing -> Progressing                             23s

Expected Output
|- Namespace/guestbook   -                8m45s
|- Deployment/kustomize-guestbook-ui                                           0/1                                                                                5m44s
 └── ReplicaSet/kustomize-guestbook-ui-9b9c89b69                             0/1                                                                                5m44s
    └── Pod/kustomize-guestbook-ui-9b9c89b69-qnr66                          0/1     ContainerCreating                                                                                5m44s                                                                                5m44s
|- Service/kustomize-guestbook-ui                   -                7m36s
└── EndpointSlice/kustomize-guestbook-ui-b5xbb   -                7m36s

@anandf this is a great idea 😊
I suppose it's just matching Argo CD Application/<APP_NAME> custom resource with other k8s resources containing the argocd.argoproj.io/instance=<APP_NAME> label?

I haven't had much time these while to do maintenance on this tool, but do feel free to work on this & I'll do my best to review it & publish a version 🙏

ArgoCD uses either a label app.kubernetes.io/instance: some-application or an annotation argocd.argoproj.io/tracking-id or both to track application. There is also an option to specify a custom label. Once the direct resources are identified, we need to track its individual children using existing logic. Custom label selector can be supported by having an additional optional argument for the lineage command

I see, thanks for sharing this!
Would greatly appreciate if you can also update over here when you've started working on this 🙏


Kindle Gen not getting recognised

When I try to open the amazon kindlegen, it says it isnt there so i checked a random website that had it, so i downlaoded and moved it to the folder were kindle comic is and i still get it saying it still cant find the file, would be grateful if you could help me.

It appears that your Kindle Comic Converter application is just a shortcut. Where is the actual ".exe" file located?


Voice Conversion - Not converging after 50 epochs

How many epochs are required to get a decent voice conversion?
I am using the default dataset of CMU Arctic and folllowed all the steps but the converted voice are not all audible.

Hi, which recipe are you using (s1 or s2)?

Didn't you try with s2?

Is s2 recipe works with python 2? Since I am facing a lot of error beacuse of python 3

Yes, I think so, Python 2.7

After creating new environment with python 2.7 and running s2 script, I got stuck in the second step
Here is the log
Prepare acoustic features using MAGPHASE vocoder...

Analysing file: arctic_a0020.wav................................
Analysing file: arctic_a0001.wav................................

Analysing file: arctic_a0039.wav................................

Analysing file: arctic_a0058.wav................................
Extracting epochs with REAPER...
Extracting epochs with REAPER...
Extracting epochs with REAPER...
Extracting epochs with REAPER...
Residual symmetry: P:1057.966064  N:2080.696289  MEAN:-1.834260
Residual symmetry: P:779.040894  N:1617.814087  MEAN:-0.248217
Residual symmetry: P:565.581360  N:1208.755005  MEAN:0.239541
Residual symmetry: P:593.399414  N:1381.622070  MEAN:0.091667
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
Traceback (most recent call last):
  File "/home/ubuntu/merlin/misc/scripts/vocoder/magphase/extract_features_for_merlin.py", line 82, in <module>
    lu.run_multithreaded(feat_extraction, l_wavfiles, out_dir)
  File "/home/ubuntu/merlin/tools/magphase/src/libutils.py", line 62, in run_multithreaded
    results = pool.map(func_wrapper, l_iterable_args)
  File "/home/ubuntu/anaconda2/lib/python2.7/multiprocessing/pool.py", line 253, in map
    return self.map_async(func, iterable, chunksize).get()
  File "/home/ubuntu/anaconda2/lib/python2.7/multiprocessing/pool.py", line 572, in get
    raise self._value
ValueError: operands could not be broadcast together with shapes (0,10) (396,1)
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found
/bin/sh: 1: /home/ubuntu/merlin/tools/magphase/tools/bin/mcep: not found

I checked my folder /merlin/tools/bin/SPTK-3.9, it is empty. Do you think I missed some step of installation?

Finally the scripts after installing csh which was the cause of my SPTK installation fail and hence other issues arose.
I was able to use s2 script for voice conversion. Thanks @felipeespic

You are welcome

The demo recipe did work but it doesn't not work with other data I am using. You can find the wav files here

@felipeespic  Also if I can find out some documentation or tutorials regarding this voice conversion method provided here, it would be nice to understand the overall process.

I realised that actually it was just a warning, so it shouldn't affect the performance. Anyway, I just got rid of that warning and I pushed it to the master, so you can download the new version of MagPhase if wanted.
For the documentation, unfortunately I don't have any by now, but if I find something I will post it.
Thanks

@felipeespic  Do you think I can use this as recipe for emotonal voice conversion, i.e neutral to emotional speech?
If yes, can you give some hints. I am reading this paper about https://www.researchgate.net/publication/306928922_Emotional_voice_conversion_using_deep_neural_networks_with_MCC_and_F0_features and was thinking if could use a similar approach with this tool.

@felipeespic  Thank you for the recipe and supporting regularly with the issues.
Is it possible talk to you on Gitter for some time, so that I can clarify few of the things. It will be not long, I will just need to ask few things.
Since that will help me a bit to get a clear picture of the recipe and also the possibilities.

Yes, I am connected now actually.


Add shortstr limit to domain types

The limit imposed (especially on exchange and queue names) is quite important to know. From the type name alone one can imagine that it is short, but not the exact limit.

@andreas-schilling Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@andreas-schilling Thank you for signing the Contributor License Agreement!

I will cherry-pick to live.


Prevent duplicate workflow prompts from sending

Decided on doing the filtering on the back-end -- just so we send over less data, that's it

Codecov Report

Merging #352 (a1cb8e1) into 2021.9.x (c75e448) will increase coverage by 0.79%.
The diff coverage is 54.98%.

@@             Coverage Diff              @@
##           2021.9.x     #352      +/-   ##
============================================
+ Coverage     42.35%   43.14%   +0.79%
============================================
  Files           781      782       +1
  Lines         73463    73965     +502
============================================
+ Hits          31115    31912     +797
+ Misses        37946    37533     -413
- Partials       4402     4520     +118

Impacted Files
Coverage Δ

automation/rest/request/session.go
4.57% <0.00%> (ø)

automation/service/session.go
50.44% <0.00%> (-1.39%)
:arrow_down:

automation/service/trigger.go
55.65% <ø> (+1.38%)
:arrow_up:

automation/types/function.go
70.14% <ø> (ø)

compose/rest/automation.go
4.76% <0.00%> (ø)

compose/rest/module.go
74.71% <0.00%> (ø)

compose/rest/page.go
71.68% <0.00%> (+9.73%)
:arrow_up:

compose/rest/request/automation.go
0.00% <0.00%> (ø)

compose/rest/request/notification.go
0.00% <0.00%> (ø)

compose/service/locale.go
0.00% <0.00%> (ø)

... and 136 more

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 2653c38...a1cb8e1. Read the comment docs.


How to make it work?

Hey guys!
I'm on OS X, trying to make PS4 Controller work with steam, tried putting your file into game folders of several games, modified steam config file (although it gets changed back when a game is launched) and it does not work. Any suggestions?

Hi, this database is not for steam itself. It is for game developers mostly. Many games aren't made with SDL, or don't use the file either.
You are better off contacting support for either steam or the games you play.


Add p-value indicator

In this PR:

Add random cohort generation to make it easy to test out plots without using private data.
Add a significance indicator that shows * if <= 0.05, otherwise "ns", for both Fisher's and MW.

@arahuja

Nice, like the random cohort. Only addition might making the significance level configurable, but not a must have.

Coverage decreased (-2.6%) to 59.96% when pulling 952967eef8699cd7e518c1803824134ec23ec09c on p_star into bbaf7aff45f93c907e0bfddafd46fb01cb3aed2e on master.

Coverage decreased (-2.6%) to 59.96% when pulling f3c2c3f0a23b35178acc1ca8e232a046f9467e7f on p_star into bbaf7aff45f93c907e0bfddafd46fb01cb3aed2e on master.

@arahuja I'm now using set_yticks and get_yticks to filter negative ticks, and the same strategy for non-0 to 1 ticks in the case of Fisher's exact. I imagine we can generalize the latter in a later PR, as well as let 0.05 be customizable.

Coverage decreased (-3.0%) to 59.104% when pulling 8dd7411857b9a57605ac9aeed5a5fad730ccb6e3 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

@arahuja added a fix for the Fisher's exact ordering as well as custom styling in styling.py.

Coverage decreased (-2.3%) to 59.77% when pulling 2e90548ecca55bf449593154d0e543a6b4ea7747 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

Coverage decreased (-2.3%) to 59.77% when pulling 2e90548ecca55bf449593154d0e543a6b4ea7747 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

Also just simplified __str__s for MannWhitneyResults and Fisher's to not pollute notebooks.

Coverage decreased (-2.4%) to 59.696% when pulling e22c65e8bc347cd49fc7ad869c728db1a9041cc5 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

@arahuja sorry to keep adding to this PR, but also just made it easier (https://github.com/hammerlab/cohorts/pull/112/commits/0d70a60c71d25d912ed6fd2dacc948f92787c344) to deal with situations where I want to split on something other than benefit that isn't presently in the DataFrame: I can include it as another function in on, and then specify which function is the plot_col vs. the boolean_col:
Example:
cohort.plot_boolean(on={"Liver Met": lambda row: row["Liver Mets"] == 1, "Split": split_patients},
                    plot_col="Liver Met",
                    boolean_col="Split")

Coverage decreased (-2.8%) to 59.301% when pulling 0d70a60c71d25d912ed6fd2dacc948f92787c344 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

Cool, the latest addition was something I would've found useful recently and may let me re-work how I did #111 to allow more customization. Will leave that off for now.
Left some general comments but makes sense if we want to get this in and come back to clean up some minor aspects.

Ah another thing I just realized, I use stripboxplot on it's own frequently, with no stat. test, might be useful (in the future) to not plot the line, or None as an option for significance

I think all comments are addressed and will plan to merge on 🍏  unless I hear anything!

Coverage decreased (-2.9%) to 59.189% when pulling bb09875f8e021639a2e86121f032bc2c311ec4e9 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.

Coverage decreased (-2.9%) to 59.189% when pulling 0bfdc821cb902dca338ec72680383c5e52f32803 on p_star into 4a45617382b8c03a3c13eec2911ae90f8a54729b on master.


Examples in lookahead are unclear.

The explanation and examples are not clear.  Better would be:
Explanation, from https://www.stefanjudis.com/today-i-learned/the-complicated-syntax-of-lookaheads-in-javascript-regular-expressions/:
Positive and negative lookaheads:
x(?=y) – positive lookahead (matches 'x' when it's followed by 'y')
x(?!y) – negative lookahead (matches 'x' when it's not followed by 'y')
Updated example to show difference between two for each string:
let quit = "qu";
let noquit = "qt";
let quRegex= /q(?=u)/; // Will generate a result q only if q is followed by u, not otherwsie
let qRegex = /q(?!u)/; // Will generate a result q only if q is NOT followed by u
quit.match(quRegex); // Returns ["q"] because q is followed by u
quit.match(qRegex); // Returns ["null"] because q is followed by u
noquit.match(quRegex); // Returns ["null"] because q is not followed by u
noquit.match(qRegex); // Returns ["q"] because q is not followed by u
Source of page:
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/regular-expressions/positive-and-negative-lookahead;
https://github.com/freeCodeCamp/freeCodeCamp/blob/master/curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/positive-and-negative-lookahead.english.md

Duplicate of #37481 ? Re-open if unrelated.


Question about CosyVoice

Hi there! 👋 I found your python project CosyVoice very impressive. Great work! 🚀 Could you send me more details on Telegram? Also, please review my work and follow me on GitHub @nectariferous. Thanks!

our work is all opensourced on github and github.io


chore: bump emotion to help with cache clobbering

SUMMARY
we were seeing some instances where emotion styling wasn't showing up on sql lab. I think it may be related to this issue: https://github.com/emotion-js/emotion/issues/2209 so I bumped emotion to bring in the patch fix. So far testing looks positive that it fixed the issue.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
Reload sql lab often- each load should have all the styles.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

You appear to be on an old version of npm (lock file version is downgraded from 2 to 1 in this PR). Can you make sure you're running npm i on a npm version 7?

Thanks for catching that @villebro!


Test: Remove tests packages from NugetPackageVerifier exclusion list

NugetPackageVerifier is an fxcop-like tool for required NuGet package and assembly metadata and for proper code signing that now runs automatically in our build. When it was enabled recently in our branch tests packages we produce were excluded.
However per #3016 these packages will soon be published to NuGet, hence the exclusion will no longer apply.

We should try removing the packages from the exclusion list and fix whatever we need to fix. No need to break the build :smile:
cc @Eilon @ajaybhargavb.

Updated the PR with this change https://github.com/aspnet/EntityFramework/pull/3157/

b4edf2eabd05752420f434b490c317e8f27488de


About Facebook::getTokenFromRedirect()

i read about this instruction

there is a wrapper for getTokenFromRedirect() that defaults the callback URL to whatever is set in the config.

But when i set redirect_uri in config, the function Facebook::getLoginUrl() run normally with redirect_uri from config file
And when i using getTokenFromRedirect() without parameter, i getting error

Unable to obtain access token from redirect facebook

But if i set the paramter like this Facebook::getTokenFromRedirect(url('/whatever/link/'));
it's return token with experies date.

@samm

Glad you figured it out @robertlagrant!


Prepare azidentity v1.3.0-beta.2 for release

hi @chlowell when is azidentity 1.3.0 planned to be released?
we need this for using workload identities via env (https://github.com/Azure/azure-sdk-for-go/pull/19503)

We plan to release a stable 1.3.0 in early April. In the meantime, please try the beta and open an issue if you find any problems.

hi @chlowell , any update on the release schedule for azidentity 1.3.0?

No, still planning to publish tomorrow.


Unable to merge mp4 files

I am trying to merge these 2 mp4 files attached.
Only first video ends up in final video with size of that equalling addition of both files.
This is the code which I am using to merge
Any help is highly highly appreciated.
`fun combine(){
        val inMovies = arrayOf<Movie>(MovieCreator.build(getVideoFile(1))
                 , MovieCreator.build(getVideoFile(2))/*
                , MovieCreator.build(getVideoFile(3))
                , MovieCreator.build(getVideoFile(4))
                , MovieCreator.build(getVideoFile(5))
                , MovieCreator.build(getVideoFile(6))*/)

        val videoTracks = LinkedList<Track>()
        val audioTracks = LinkedList<Track>()
        for (m in inMovies) {
            for (t in m.tracks) {
                if (t.handler == "soun") {
                    audioTracks.add(t)
                }
                if (t.handler == "vide") {
                    videoTracks.add(t)
                }
            }
        }
        val result = Movie()
        audioTracks.forEach {
            result.addTrack(it)
        }

        videoTracks.forEach {
            result.addTrack(it)
        }

        val out = DefaultMp4Builder().build(result) as BasicContainer
        val fc = RandomAccessFile(
                String.format(getOutputPath()), "rw").channel
        out.writeContainer(fc)
        fc.close()
    }`

video.zip

@sannies


Available GDR for SQL 2016SP3 is missing from the Latest updates table

In the "Latest updates" table, the row for SQL Server 2016 SP3 should indicate that a GDR is applicable. The GDR which is valid is KB5014355 and was derived from https://support.microsoft.com/en-us/topic/kb5016394-cumulative-update-17-for-sql-server-2019-3033f654-b09d-41aa-8e49-e9d0c353c5f7.

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

ID: d1353d4d-3899-d6cf-e44f-8536c7828335
Version Independent ID: de0210cf-386a-f8c8-1157-40cea207ed4b
Content: Latest updates for SQL Server - SQL Server
Content Source: docs/database-engine/install-windows/latest-updates-for-microsoft-sql-server.md
Product: sql
Technology: install
GitHub Login: @SrinivasSQLMS
Microsoft Alias: skandi

@Kentix -- thank you for bringing this to our attention.
@SrinivasSQLMS -- please look into this issue.

@kentix,
when I look at https://docs.microsoft.com/sql/database-engine/install-windows/latest-updates-for-microsoft-sql-server I see that SQL Server 2016 SP3 includes the GDR.
CC: @harveymorams  - is there an updated needed here?

@@.***
On second look, I think it’s fine. I think I was looking for “GDR” within the table and fixated on its existence in the CU column as with some of the other line items. Somehow I missed the the entire GDR column. This issue can be closed out, apologies for the noise!
On Sep 12, 2022, at 2:04 PM, Mike Ray @.***> wrote:

@Kentixhttps://github.com/Kentix,
when I look at https://docs.microsoft.com/sql/database-engine/install-windows/latest-updates-for-microsoft-sql-server I see that SQL Server 2016 SP3 includes the GDR.
CC: @harveymoramshttps://github.com/harveymorams - is there an updated needed here?
[image]https://user-images.githubusercontent.com/15928587/189724729-64500389-18d6-4142-9200-1075de9219bd.png
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/sql-docs/issues/8055#issuecomment-1244107650, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHP5IOEWTXXPL3A3RRJ7L3DV55WD7ANCNFSM6AAAAAAQICQHE4.
You are receiving this because you were mentioned.Message ID: @.***>

#please-close


Does not format space before "else" in else if statement

Is there a way to configure esformatter to format this:
if (a == 1) {
  b;
}else if (a == 2) { // notice the space before else
  c;
} else {
  d;
}

into
if (a == 1) {
  b;
} else if (a == 2) { // notice the space before else
  c;
} else {
  d;
}

I read your code and it does not seem to handle that specific space.

the code is always formatted the way you expect with the default settings.
the space between } and else is handled by whiteSpace.after.IfStatementClosingBrace, try esformatter --whiteSpace.after.IfStatementClosingBrace=1 and esformatter --whiteSpace.after.IfStatementClosingBrace=0 to see the difference in the behavior.


test: use t.TempDir to create temporary test directory

What type of PR is this:

/kind tests
/kind code-refactoring
What does this PR do / why we need it:
This pull request replaces ioutil.TempDir with t.TempDir. We can use the T.TempDir function from the testing package to create temporary directory. The directory created by T.TempDir is automatically removed when the test and all its subtests complete.
This saves us at least 2 lines (error check, and cleanup) on every instance.
Reference: https://pkg.go.dev/testing#T.TempDir
func TestFoo(t *testing.T) {
	// before
	tmpDir, err := ioutil.TempDir("", "")
	if e != nil {
		t.Fatal(e)
	}
	defer os.RemoveAll(tmpDir)

	// now
	tmpDir := t.TempDir()
}

PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/ok-to-test

/override windows-integration-test/Windows-test
Not related


Increase data reading frequency

Hello !!!
I have implemented the fanuc-driver to read data from Fanuc and read it through an MQTT Broker. I would like to know how I can increase the frequency of data reading. I have tried lowering the sweep_ms to low values (5 ms) and keeping stay_connected=TRUE, however I am getting a read rate of 2/3 seconds in MQTT Explorer.
Disabling collectors lowers the reading frequency but I don't think it is the best solution since I need that data. Is there any other way to improve reading frequency?

Another way is to use the non-embedd port.  Embed ethernet is shared with other compute.
Alternatively, custom collectors could be developed to get exactly the data you need.
What frequency do you need and what data are you after?


Custom Converter Setup

I'm trying to setup a Customer Converter - following the information setup on the readme documentation. My configuration setup is setup in the appsettings.json file and below is what I would expect to work and what is required to work.
Please note the following two lines:
This is the one I would like to get working.
 .WriteTo.ApplicationInsights(new CustomConverter(), LogEventLevel.Verbose)

This is the one (passing in a "telemetryConfiguration") I would like to avoid - if possible.
 .WriteTo.ApplicationInsights(telemetryConfiguration, new CustomConverter(), LogEventLevel.Verbose)

Is it possible that the issue is I don't have my "CustomConverter" defined in the `appsettings.json" file in this section:
      {
        "Name": "ApplicationInsights",
        "Args": {
          "restrictedToMinimumLevel": "Verbose",
          "telemetryConverter": "[THE FULL PATH TO THE FILE], Serilog.Sinks.ApplicationInsights.Sinks.ApplicationInsights.TelemetryConverters.TraceTelemetryConverter, Serilog.Sinks.ApplicationInsights"
        }
      }

Expected
What I would Expected to make it work:
            Log.Logger = new LoggerConfiguration()
                .ReadFrom.Configuration(_configuration)
                .WriteTo.ApplicationInsights(new CustomConverter(), LogEventLevel.Verbose)
                .CreateLogger();

Actual
What is Required to make it work:

            var telemetryConfiguration = new TelemetryConfiguration {
                InstrumentationKey = "SOME KEY"
            };

            Log.Logger = new LoggerConfiguration()
                .ReadFrom.Configuration(_configuration)
                .WriteTo.ApplicationInsights(telemetryConfiguration, new CustomConverter(), LogEventLevel.Verbose)
                .CreateLogger();

Appsettings.Json**
Here is my setup. for appsettings
{
  "Serilog": {
    "Using": [ "Serilog.Sinks.Console", "Serilog.Sinks.File", "Serilog.Sinks.ApplicationInsights" ],
    "MinimumLevel": {
      "Default": "Verbose"
    },
    "WriteTo": [
      {
        "Name": "Console",
        "Args": { "outputTemplate": "{Timestamp} [{Level:u3}] {Message}{NewLine}{Exception}" }
      },
      {
        "Name": "File",
        "Args": {
          "path": ".\\ConsoleApp.log.txt",
          "fileSizeLimitBytes": 20971520,
          "rollOnFileSizeLimit": true
        }
      },
      {
        "Name": "ApplicationInsights",
        "Args": {
          "restrictedToMinimumLevel": "Verbose",
          "telemetryConverter": "Serilog.Sinks.ApplicationInsights.Sinks.ApplicationInsights.TelemetryConverters.TraceTelemetryConverter, Serilog.Sinks.ApplicationInsights"
        }
      }
    ],
    "Enrich": [ "FromLogContext", "WithMachineName", "WithThreadId"],
    "Destructure": [],
    "Properties": {
      "Application": "ConsoleApp"
    }
  }
}

We were able to get this working by using the following configuration:
      {
        "Name": "ApplicationInsights",
        "Args": {
          "restrictedToMinimumLevel": "Verbose",
          "telemetryConverter":
            "MyApp.ConsoleApp.Infrastructure.Logging.CustomConverter, MyApp.ConsoleApp"
        }
      }

My Setup is now:
            Log.Logger = new LoggerConfiguration()
                .ReadFrom.Configuration(_configuration)
                .Enrich.WithProperty("SessionId", SessionId)
                .CreateLogger();


Update to latest videojs-contrib-ads

As of this filing we're on 3.1.2 and they've released 4.2.8.
This should also lead to a resolution of #167

It will also be wonderful if video.js dependency is updated or at least changed from ~5.3 to ^5.3 and let us decide wich version of video.js to use.
For now I´m able to use videojs-ima 0.6.0 with video.js 5.20.1 without issues (tested in Chrome, Firefox, Edge, iOS, Android) but I had a hard workaround using Yarn + Webpack because videojs-ima was always being init in a video.js 5.3instance (because dependency range says it can´t work with >5.3.X)

Our plugin is pretty complex, so we like to test it with every new 5.X version that comes out before we say we can support it. For this reason we'll leave the config as ~5.X, but feel free to try out newer versions in your implementation to see if they work (though we can't guarantee that they should until we update the package.json file).

hello @shawnbuso I suggested a PR to update to videojs-contrib-ads 5.0.3: https://github.com/googleads/videojs-ima/pull/405
Based on review of their changelog, migration guides for 4.X and 5.X and manual testing, it seems we just need to add a ads-ad-started event.
Can you think of something else that needs to be checked or should be changed?

I think that's about it - I took a look at the PR and it looks like you just need to sign our CLA and we can merge the change. Thanks!


feat(cli/datacontract): Add data quality assertion support

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@asikowitz btw we should probably write some tests for the contracts cli

What exactly is this contracts CLI going to be used for?
Can we provide some examples?
Also, we are moving quickly on new assertion types, how should we think about adding new types of assertions into contracts?

https://linear.app/acryl-data/issue/ING-391/add-tests-for-data-contract-cli-support-more-assertions created


Missing references

Hello :)
After i cloned your project and opened it in Unity 2018.2.0b many error like "namespace name 'GLTF' could not be found". Any idea?
Thanks :)

The project GLTFSerialization/GLTFSerialization.sln needs built to generate the dll's. Please refer to the instructions in the Readme.md.
Let me know if that fixes the issue

I never before built a file in vs :/
When i open the sln file in vs2015 then it says: The project requires a platform SDK (UAP, Version=10.0.14393.0)
After installing the windows sdk this error does not disappear.

The sln does need to be built using VS 2017 I believe. We do have a task out to make the dll's build via our build server.

now it works :) thanks

@Jennilein1991 Hi! can you please help me out here, how you resolved the issues?


Can't use map instance ?

EDIT: Solved, it was pretty stupid, I needed to use map.instance.setCenter, however I'd like a confirmation that this is the "way to go" before closing if possible :) Thanks a lot and have great day everyone !
Hi,
I've been encountering an issue with the "map" in the callback function on .ready().
After implementing a reactive map as seen here, I can't use the "map" argument at all..
This code doesn't work:
GoogleMaps.ready('exampleMap', function(map) {
        // Add a marker to the map once it's ready
        Tracker.autorun(function() {
          if(Session.get('lat') && Session.get('lng')) {
            map.setCenter({ lat: Session.get('lat'), lng: Session.get('lng') });
          }
        });
});

It tells me that

TypeError: map.setCenter is not a function

So I have to use that code instead:
GoogleMaps.ready('exampleMap', function(map) {
        // Add a marker to the map once it's ready
        Tracker.autorun(function() {
          if(Session.get('lat') && Session.get('lng')) {
            GoogleMaps.maps.exampleMap.instance.setCenter({ lat: Session.get('lat'), lng: Session.get('lng') });
          }
        });
});

How come? I found it to cause issue everywhere, let's say in my geocoder function, where I add a market, I get

InvalidValueError: setMap: not an instance of Map; and not an instance of StreetViewPanorama

if I use map instead of the whole map instance call in the following code:
function geocodeAddress(gc, map) {
  var address = Product.findOne({permalink: FlowRouter.getParam('product')}).realAddress;
  gc.geocode({'address': address}, function(results, status) {
    if (status === google.maps.GeocoderStatus.OK) {
      Session.set({
        "lat": results[0].geometry.location.lat(),
        "lng": results[0].geometry.location.lng()
      });
      var marker = new google.maps.Marker({
        map: GoogleMaps.maps.exampleMap.instance,
        position: results[0].geometry.location
      });
    } else {
      alert('Geocode was not successful for the following reason: ' + status);
    }
  });
}

Thanks for your help, I might have totally screw up something as it is my first time dealing with it so feel free to point out how I can make it better or if you need more informations :).

Hey @Yonben yep that's it, map.instance see here: https://github.com/dburles/meteor-google-maps#ready


Using MongoDB instead of local json files for data storage

Fixes #19
Fixes #17

Rest, the changes seem fine to me. @vaibhavk What do you think ?


Agent failed to run: server returned non-202 status: 200

No Connection between Agent und Server. Server is listening an Port 80

Agent initialized: {WIN11****C\user c38582bafcf67921897fc89d114d6c20}
Agent failed to run: server returned non-202 status: 200

Are you sure the webhook URL is correct?

My false. Now I got it working.


Introduce PinotInputFormat and PinotOutputFormat

to allow easy integration between different systems

I looked at poll.ly and found this.  Whats this?  Like integration with Spark, Flink, etc?
https://poll.ly/#/2YeaKQ5o

Yes, Presto, spark, hive, Flink etc
Related issues

https://github.com/apache/incubator-pinot/issues/4809
https://github.com/mangrrua/spark-pinot-connector


Audio stops when locking the device

Hi, I have implemented your "hack" that allows background mode, it works when I press the home button, but when I lock the screen, it just update my playback status to paused. Is there something I am missing?
Thanks for your help

I've had the same problem. Using the home button and then pressing the power button to lock the device will work. However if I am in the app and then press the lock button, the audio will stop.
I'm guessing the problem is caused because UIApplicationWillResignActiveNotification is never called while the app is still open and you press the power button to lock the screen. I've looked at two methods to try and fix it. The first being was adding the following line to the load method in MPMoviePlayerController+BackgroundPlayback.
[defaultCenter addObserver:self selector:@selector(backgroundPlayback_applicationWillResignActive:) name:UIApplicationDidEnterBackgroundNotification object:nil];
This didn't work and I'm assuming this is because it's too late to run the code which strips the video layer out of AVPlayer. (if only there was a UIApplicationWillEnterBackgroundNotification notification).
The second attempt I took was following Playing media while in the background using AV Foundation on iOS. However all this achieved was my own version of MPMoviePlayerController+BackgroundPlayback which really didn't achieve much.
I'm not really too sure what else can be done. Does anyone else have any thoughts?

Had same problem

Has anyone solved this, as I am also having the same issue.

I don't use MPPlayer or the XCD player (XCDYouTubeClient for URL + AVPlayer), but I did get pauses a while back, so it might be helpful. In my case it had to do with AVPlayerLayer getting set/unset. (see WWDC2016 Session 503)
I fixed my issue by monitoring UIApplicationDidEnterBackgroundNotification and UIApplicationDidBecomeActiveNotification and calling play if needed. Pre-iOS 10 this did cause a 0.5~1s pause, but as of iOS10, there is no interruption.
These two notifications always get called when pressing Home or when locking the phone.

How exactly have you implemented the hack that allows background mode? I just tested with the demo app on an iPhone 6 running iOS 10.0.2 and background playback is working fine, both when pressing the home button and locking the iPhone.

No reply from original poster, so I’m closing the issue.


Picnob Bridge and Picuki Bridge for Instagram access works but does not show the image

Describe the bug
I tried both these bridges, and they both cannot show an image in html view or in InoReader. If I click the attachement it take me to the image in pixwox.com. If I click broken image icon it takes me to the post in pixwox.com
To Reproduce
Create a Picnob bridge
View it as HTML
Expected behavior
Text and photo should be shown for each post
Screenshots

Desktop:

OS: Windows 10
Browser: Chrome

Additional context
Private hosting Windows 10 Docker

It is the same if I use the default value aesoprockwins
<?xml version="1.0" encoding="UTF-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" xmlns:media="http://search.yahoo.com/mrss/">
  <title type="text">Username aesoprockwins - Picnob Bridge</title>
  <link rel="alternate" type="text/html" href="https://www.picnob.com/profile/aesoprockwins/"/>
  <link rel="self" type="application/atom+xml" href="http://<IP_ADDRESS>:31234/?action=display&bridge=PicnobBridge&context=Username&u=aesoprockwins&format=Atom"/>
  <icon>https://www.picnob.com/favicon.ico</icon>
  <logo>https://www.picnob.com/favicon.ico</logo>
  <id>http://<IP_ADDRESS>:31234/?action=display&bridge=PicnobBridge&context=Username&u=aesoprockwins&format=Atom</id>
  <updated>2024-03-24T11:16:18+00:00</updated>
  <author>
    <name>RSS-Bridge</name>
  </author>
  <entry>
    <title type="html">Daylight shirts &amp; hoodies loading… 🎥/👕 - <a href="htt...</title>
    <published>2024-03-13T11:16:18+00:00</published>
    <updated>2024-03-13T11:16:18+00:00</updated>
    <id>urn:sha1:3c5b32747cfeb65045f7b1e1d6d4e651122eabc8</id>
    <link rel="alternate" type="text/html" href="https://www.picnob.com/post/6823283666526448401245/"/>
    <content type="html">

(video)

Daylight shirts &amp; hoodies loading… 🎥/👕 - @twincitytees
</content>
    <link rel="enclosure" type="image/jpeg" href="https://sp1.pixwox.com/p/pt_6823283666526448401245_0_146175156a6fc86221b7334804071baa.jpg?v/t51.29350-15/432397363_1110306910172374_7221255011057256122_n.jpg?stp=c0.280.720.720a_dst-jpg_e15_s640x640&_nc_ht=scontent-nrt1-2.cdninstagram.com&_nc_cat=110&_nc_ohc=44DVblkB3ecAX-GaR9m&edm=ABmJApABAAAA&ccb=7-5&oh=00_AfBzMU_WADbYqOQlnHKVQsGFMeoIaYz3Ci5lYJqc6NP84A&oe=6601AB18&_nc_sid=b41fef"/>
  </entry>
  <entry>
    <title type="html">https://www.picnob.com/</title>
    <published>2024-03-10T11:16:18+00:00</published>
    <updated>2024-03-10T11:16:18+00:00</updated>
    <id>urn:sha1:6a4229ec64e04e9dc77149bf360a02a1ab0131bb</id>
    <link rel="alternate" type="text/html" href="https://www.picnob.com/post/6823661966727210492949/"/>
    <content type="html">

Hello !
Sorry, but I won't be able to fix those bridges : my dev environment is blocked by Cloudflare, and I won't be able to test if a fix works.
If anyone want to fix those bridges, feel free to fix them. If no one has fixed them, I will delete them.

@sysadminstory Please do not delete those bridges. Even if the image does not appear, the bridges still serve an important purpose because their RSS feeds still update properly. Some users (like me) use the feeds so that we are alerted whenever there is a new post. It takes only one extra click to go see the images directly on the sites. It would be a shame for those bridges to be completely deleted because of the need to take that extra step.

I tried to find out the issue using a RSS Bridge Public instance : the URL of the image seems to be OK, but the website uses Cross Origin Resource Sharing header that prevents the use of the image on any other website than https://www.pixwox.com/ :
cross-origin-resource-policy: same-site
If your RSS reader is not web based, then it should work, but if your RSS reader is a web page, then the CORS header will prevent you to display the image.
We could imagine to use some external CORS Proxy or unblocker in this case. Doing this inside RSS Bridge would cause too much load, especially for any public instance.

@stavnskov @pin-grid-array
Did #4068 fix the Picuki issue ?

Can you close the issue ?


⚠ Bofrost.lu has degraded performance

In 850ce08, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1595 ms

Resolved: Bofrost.lu performance has improved in 8449f7b after 6 minutes.


[WiP] Initialises readme and adds example

not ready for review
@finnhodgkin and I have added an intro to webrtc and an example of how to create a text with video chat using it.
We still need to add styling as it is currently completely unstyled, and add tests (which may be difficult due to the interaction with SimpleWebRTC).
#3

@ZooeyMiller this is looking really good so far! 🎉


Improve documentation

Improve documentation

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   90.62%   90.62%
=======================================
  Files           2        2
  Lines          64       64
  Branches       22       22
=======================================
  Hits           58       58
  Misses          6        6

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 f3f1560...79c604c. Read the comment docs.


Missing OAuth Connectivity

Team, the Data Factory v2 OData connector is missing OAuth authentication.

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

ID: 8193f2a3-80f3-14e9-7022-27ea5fa02001
Version Independent ID: 9eff4721-c265-74a7-11fd-c6920f309982
Content: Copy data from OData sources using Azure Data Factory
Content Source: articles/data-factory/connector-odata.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@simonnuss Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@simonnuss thanks for the feedback. You are right that currently OData connector doesn't support OAuth. To help us track the ask, could you let know which data store you want to connect via OData + OAuth?

Sure, it’s Microsoft Project Online.
Thanks,
Simon
On Tue, 10 Jul 2018 at 3:20 am, Linda Wang<EMAIL_ADDRESS>wrote:

@simonnuss https://github.com/simonnuss thanks for the feedback. You
are right that currently OData connector doesn't support OAuth. To help us
track the ask, could you let know which data store you want to connect via
OData + OAuth?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/11417#issuecomment-403725912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXY-dI0JRH2p6v2nq6ishEwFk8ww7gVFks5uFFXJgaJpZM4VHsPL
.

@simonnuss thanks for sharing. I've tracked the ask. For now we don't have concrete timeline for OAuth support yet.
#please-close

@simonnuss We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

Hello, any news on "Data connector doesn't support OAuth." ?

@klabir OData connector added supports for Service Principal and Managed Identity auth. Let know what additional OAuth you need and the corresponding data store you need to connect using such auth.
(please open a new issue if you have follow up questions, as comments on closed issues are easily got buried)


combined HTML to one page

You may want to delete the second page of HTML from the main before you merge this, just to make life a little easier for yourself.