Updated ollama handler to handle case when multiple tokens are return…

Handling the case when multiple tokens are sent by ollama at the same time
Instead of decoding the entire payload, split it up based on the newline character and parse each one independently

@ronneldavis By multiple tokens do you mean one streaming response from ollama could actually be multiple responses?
Can you let me know the following?

What are the cases where this happens?
How can I replicate it?
What is the current behaviour? Without your changes.

Seems like the ollama folks themselves are handling the streaming response this way.
https://github.com/ollama-ui/ollama-ui/blob/main/api.js#L71-L97

@zya I face this issue cause my output tokens have "\n" in it, its a code generation model.
You can use the following prompt for the wizardcoder moder to test it
Respond in the format <PLAN>...</PLAN>...\nWithin the ... block, write a function called \"main\" that returns a chart.js object to create a graph as a dictionary. \nThe function main should not accept any parameters.\nThe chart options should have responsive set as true and maintainAspectRatio set to false.  \nAdd a relevant title for the chart in options.plugins.title.text of the output object. \nIf the user asks for a metric, set the type of chart in config to me metric and return a single data point in the data.datasets.data array and the datasets.label should be an appropriate unit. \nUse only data from the given files to construct your response, expect the files to be unsorted. \nUse the pandas library load the necessary columns from the CSVs to reduce memory usage. The columns you load SHOULD exist in the dataset. Do not attempt to load columns that don't exist in a file!\nDo not use or import any other libraries other than the built-in python libraries, numpy and pandas.\nBefore performing numerical operations on a column like .sum(), use the convertNum() function to make sure its a numerical value (eg. df['column_name'] = df['column_name'].map(convertNum)). The convertNum has been written already and doesn't need to be created. Do not initialize a function called convertNum!\nWhen correlating different data sources using a common column, make sure to load the columns when importing the dataset. Do not try correlating two files if they both don't have the column you are looking for.\nWithin the <PLAN>...</PLAN> block, mention your plan as well as any assumptions behind the code as an ordered list.\nDo not write code to call or test the function at the end.\nDo not add any text or comments after the function. Do not explain the function after writing it.\n\n### DATASETS: \n  \nTitle: Sample Customers\nDescription: A sample datasource containing the details of customers in an e-commerce setting\nURL: /tmp/0e9180d1-48d9-4150-8388-dd4d67b7e0cd.csv\nColumns:\n[{\"title\":\"Customer ID\",\"description\":\"A unique identifier for the customer across tables\",\"samples\":[\"CG-12520\",\"DV-13045\",\"SO-20335\"]},{\"title\":\"City\",\"description\":\"City of residence\",\"samples\":[\"Henderson\",\"Los Angeles\",\"Fort Lauderdale\"]},{\"title\":\"Country\",\"description\":\"Country of residence\",\"samples\":[\"United States\",\"United States\",\"United States\"]},{\"title\":\"Segment\",\"description\":\"Segment of customer according to purchasing habits\",\"samples\":[\"Consumer\",\"Corporate\",\"Consumer\"]},{\"title\":\"State\",\"description\":\"State of residence\",\"samples\":[\"Kentucky\",\"California\",\"Florida\"]},{\"title\":\"Postal Code\",\"description\":\"Postal Code of customer\",\"samples\":[\"42420\",\"90036\",\"33311\"]},{\"title\":\"Region\",\"description\":\"Region of residence\",\"samples\":[\"South\",\"West\",\"South\"]}],\nTitle: Sample Orders\nDescription: A sample datasource containing the details of orders in an e-commerce setting\nURL: /tmp/f1a04cc3-b2c2-45bf-9eec-7561de369bb7.csv\nColumns:\n[{\"title\":\"Row ID\",\"description\":\"A serial number for every row\",\"samples\":[\"1\",\"2\",\"3\"]},{\"title\":\"Order ID\",\"description\":\"A unique ID for every order\",\"samples\":[\"CA-2017-152156\",\"CA-2017-152156\",\"CA-2017-138688\"]},{\"title\":\"Order Date\",\"description\":\"Order Date in YYYY-MM-DD format\",\"samples\":[\"2017-11-08\",\"2017-11-08\",\"2017-06-12\"]},{\"title\":\"Ship Date\",\"description\":\"Ship Date in YYYY-MM-DD format\",\"samples\":[\"2017-11-11\",\"2017-11-11\",\"2017-06-16\"]},{\"title\":\"Ship Mode\",\"description\":\"The type of shipping for the order\",\"samples\":[\"Second Class\",\"Second Class\",\"Second Class\"]},{\"title\":\"Customer ID\",\"description\":\"Customer ID\",\"samples\":[\"CG-12520\",\"CG-12520\",\"DV-13045\"]},{\"title\":\"Product ID\",\"description\":\"Product ID\",\"samples\":[\"FUR-BO-10001798\",\"FUR-CH-10000454\",\"OFF-LA-10000240\"]},{\"title\":\"Category\",\"description\":\"Category of product\",\"samples\":[\"Furniture\",\"Furniture\",\"Office Supplies\"]},{\"title\":\"Sub-Category\",\"description\":\"Sub-Category of product\",\"samples\":[\"Bookcases\",\"Chairs\",\"Labels\"]},{\"title\":\"Product Name\",\"description\":\"name of the product\",\"samples\":[\"Bush Somerset Collection Bookcase\",\"Hon Deluxe Fabric Upholstered Stacking Chairs, Rounded Back\",\"Self-Adhesive Address Labels for Typewriters by Universal\"]},{\"title\":\"Sales\",\"description\":\"Sales price in USD\",\"samples\":[\"261.96\",\"731.94\",\"14.62\"]},{\"title\":\"Quantity\",\"description\":\"Quantity as a whole number\",\"samples\":[\"2\",\"3\",\"2\"]},{\"title\":\"Discount\",\"description\":\"Discount as a percentage from 0-1\",\"samples\":[\"0\",\"0\",\"0\"]},{\"title\":\"Profit\",\"description\":\"Profit in USD (can be negative)\",\"samples\":[\"41.9136\",\"219.582\",\"6.8714\"]}]\n\n### PROMPT: Create a bar chart showing my orders month over month

You will notice that in a single streaming response, you will see multiple tokens that look like this:
{
    "model": "onvocoder:7b",
    "created_at": "2023-11-06T07:07:15.810698Z",
    "response": "python",
    "done": false
}
{
    "model": "onvocoder:7b",
    "created_at": "2023-11-06T07:07:15.847325Z",
    "response": "\n",
    "done": false
}
{
    "model": "onvocoder:7b",
    "created_at": "2023-11-06T07:07:15.883454Z",
    "response": "import",
    "done": false
}

I think the Ollama folk are using the partialLine implementation to get around it as that's the only meaningful difference I see


QCPR and EPPR78

Addresses #79 and #78

Codecov Report

Merging #81 (e0e99a6) into master (acc2fe5) will increase coverage by 0.11%.
The diff coverage is 71.34%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   89.12%   89.24%   +0.11%
==========================================
  Files         137      142       +5
  Lines        8160     8346     +186
==========================================
+ Hits         7273     7448     +175
- Misses        887      898      +11

Impacted Files
Coverage Δ

src/Clapeyron.jl
100.00% <ø> (ø)

...thods/property_solvers/multicomponent/LLE_point.jl
72.41% <ø> (-25.27%)
:arrow_down:

...dels/LatticeFluid/SanchezLacombe/SanchezLacombe.jl
88.57% <ø> (-0.32%)
:arrow_down:

...dels/LatticeFluid/SanchezLacombe/mixing/SLKrule.jl
100.00% <ø> (+100.00%)
:arrow_up:

src/models/cubic/PR/variants/PR78.jl
100.00% <ø> (ø)

src/models/cubic/PR/variants/UMRPR.jl
100.00% <ø> (ø)

src/models/cubic/PR/variants/VTPR.jl
100.00% <ø> (ø)

src/models/cubic/mixing/mixing.jl
60.00% <ø> (ø)

src/models/cubic/translation/translation.jl
60.00% <ø> (ø)

...rc/models/cubic/translation/ConstantTranslation.jl
33.33% <33.33%> (ø)

... and 9 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 acc2fe5...e0e99a6. Read the comment docs.

There is something weird on MacOS - Julia 1.6

Merge?


Fix UWP padding in first column

UWP was accidently adding a column spacer before the first column in
column sets, resulting in incorrect padding appearing on the left of the
first column, as described in issue #340
Simply added an if check that skips adding the spacer if a column hasn't
been added yet. Tested and verified changes using Visualizer.

@anbare,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Setting a request body doesn't set/update the Content-Length header

When using Request.Builder.post() or any other method with RequestBody parameter the Content-Length header does not get set. This can be troublesome e.g. when changing the request body using interceptors:
OkHttpClient client = new OkHttpClient();
client.networkInterceptors().add(chain -> {
  Request originalRequest = chain.request();
  Request.Builder builder = originalRequest.newBuilder();

  if (originalRequest.method().equals("POST") && originalRequest.body() != null) {
    byte[] newContent = new byte[4];
    builder.post(RequestBody.create(originalRequest.body().contentType(), newContent));
    // The header has to be set manually!
    builder.header("Content-Length", String.valueOf(newContent.length));
  }
  return chain.proceed(builder.build());
});

If the original request already had Content-Length set that is not same as the content length of the new request unexpected things will start happening, such as old payloads being written into the new request buffer or server waiting for more data. This can be especially confusing because RequestBody has a method called contentLength().

If you want to transform the request body like this, you're probably better off using an application interceptor. The content-length header doesn't get locked in until after all application interceptors have run.
https://github.com/square/okhttp/wiki/Interceptors


ReplacementTransform not working properly with add function

Description of bug / unexpected behavior

The first text does not disappear and after the and it suddenly disappears and doubled.
Expected behavior

We expected that the first text disapear and replaced by the second text. When move the square the second text move as well.
How to reproduce the issue

Code for reproducing the problem
        square=Square(side_length=1)

        tex=Tex("$\\sqrt{2}+1$")

        tex2=Tex("$\\frac{x}{\\sqrt{2}}$")

        square.add(tex)

        self.play(Write(square))

        self.play(ReplacementTransform(tex,tex2),run_time=2)

        self.play(ApplyMethod(square.shift,UP*2))

Additional media files

Images/GIFs

Hi, I would like to work on this issue! Thank you!

Hi, I would like to work on this issue! Thank you!

There is currently nobody else working on this, so go ahead!
Do you have an idea where you have to start looking? This might be caused by something that is buried rather deep; I'd first check whether the mobject duplication happens here (although it should not), and then take a look at this method of Transform which is actually responsible for the replacement; it is called after the last frame of the animation has been rendered.
Actually, the fact that in this second method remove and add are called on the scene instead of the containing mobject is likely (part of) the reason why this does not work correctly.

Hi! After doing some trial and error testing, I realized the problem might be in the ReplacementTransform transforming a submobject. When calling the method of Transform that you mentioned, it looks like the scene doesn't remove the first submobject before the second submobject is added. Have you encountered this problem before?


rav1e_js: New demo website with react

This PR rebuilds the example rav1e_js example website with react.js and makes it a bit more visual.

Coverage remained the same at 81.816% when pulling 7b9f8ef94b89e81c3d9c59e38091ed4e88919d9e on Urhengulas:jsapi-react-exp into 9cb000aba057e746186ab6505b77841320aa4246 on xiph:master.

Might be nicer to format the config and maybe add a flush button. With some css it would be a great demo :)

I actually like plain HTML for the demo purpose, but I will have a look into this :D

The receive packet button is grayed out. It should not.

The receive packet button is grayed out. It should not.

It should only be grayed out when there are no frames yet to be encoded.

it does not work this way right now :)

Squashed all the commits into one :)


Adding "Known Issues" section

There is a known issue, and getting this fixed will have to wait for the Microsoft team.  It's good idea to mention the workaround in the mean time.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit b6c6fb2a701d9429e86c8cc40541f0cb7711b485
https://deploy-preview-8401--docsdocker.netlify.com

LGTM, thanks @ekitagawa


Web UI pushLink functionality

This is an Enhancement
Description
Currently only the Chrome Extension can open links sent via Socket IO. We should re-add support for pushing links directly to the web UI, as was implemented in v1.
If both a web UI socket and Chrome Extension socket is connected for the same user, push the link to the corresponding socket based on the request source (Web or Alexa).

No longer needed.


Setting scale has unusual behaviour

Setting scale (greater than 1) when creating the root Group added to the Display has unusual scaling behaviour.
import displayio
from adafruit_display_shapes.circle import Circle
from adafruit_display_shapes.rect import Rect
from blinka_displayio_pygamedisplay import PyGameDisplay
# import board
import time

INITIAL_SCALE = 2  # Change this

splash = displayio.Group(max_size=20, scale=INITIAL_SCALE)

display = PyGameDisplay(width=800, height=600)
# display = board.DISPLAY
display.show(splash)

# Make a background color fill
color_bitmap = displayio.Bitmap(display.width, display.height, 1)
color_palette = displayio.Palette(1)
color_palette[0] = 0xFFFFFF
bg_sprite = displayio.TileGrid(color_bitmap, x=0, y=0, pixel_shader=color_palette)
splash.append(bg_sprite)

while True:
    for scale in (1,2):
        splash.scale = scale
        print(f"scale={scale}")

        center_x = display.width // 2
        center_y = display.height // 2
        for i in range(5):
            size = 20 * (i + 1)
            size2 = size * 2
            circle = Circle(center_x ,center_y, size, outline=0xFF00FF)
            splash.append(circle)
            rect = Rect(center_x - size, center_y - size, size2, size2, outline=0x00FFFF)
            splash.append(rect)

        time.sleep(2)
        while len(splash)>1:
            splash.pop()

while display.running:
    pass

INITIAL_SCALE = 1

INITIAL_SCALE = 2

This behaviour doesn't happen on PyPortal so it seems to be an issue with the Blinka implementation.
I'm intending to look into this further, time permitting.

I am testing this out with a pi 3 b+ and a ili9341 display. I am seeing the same issue in that context. testing the fix next


Update README-dev.md

I tried to fix the flow docu, is that what we want?
This will always result in merge conflicts.
If we would merge back release , we would not end with merge conflicts I think.
What was the reason again to not merge back?

Codecov Report

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

@@          Coverage Diff          @@
##           master   #583   +/-   ##
=====================================
  Coverage      76%    76%
=====================================
  Files          18     18
  Lines         925    925
=====================================
  Hits          703    703
  Misses        222    222

Flag
Coverage Δ

#bridge_end2end
0% <ø> (ø)
:arrow_up:

#bridge_unit
76% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I guess if we'd merge back to master we'd end up with production release numbers in master. Not sure if that's a big deal, but probably not what we want. If we don't care about this, we could just bump the version once in master and then do the normal ->pre-release->release, no?

I guess if we'd merge back to master we'd end up with production release numbers in master.

I dont know what you mean by that, master is only pointer to a single commit.
If you mean by that that master references to some commit with a release version number, I dont see the issue, because that does not only have the version number, but actually is the release commit.

If we don't care about this, we could just bump the version once in master and then do the normal -> pre-release->release, no?

I also dont understand this, is that pro merging to master and done additionally or just another solution? Bumping from which version to which version?

If you mean by that that master references to some commit with a release version number, I dont see the issue, because that does not only have the version number, but actually is the release commit.

Yes, but (presumably) we would not immediately update this version number in the next commit. So later commits would have still have the same version number, even though there are changes. This is unavoidable, but we could restrict this to dev version numbers. As I said though I'm not sure if this is problematic (after all it's just a number, it's not actually released).

I also dont understand this, is that pro merging to master and done additionally or just another solution? Bumping from which version to which version?

bump in master from the current to the next (release) version
merge ->pre-release->release
As there are no changes in the pre-release and release branches, they point to the same commit as master, so merging back to master is unnecessary.

Yes, but (presumably) we would not immediately update this version number in the next commit.

Thats what I would do, actually before merging to master, so that master always has dev version numbers.

bump in master from the current to the next (release) version ...

Then we dont need a version number if the version number does not represent anything.
The purpose is to know what you are running, so the version number should at least tell you if you run a dev version or a proper version.

I think for further discussion we should have a call.

After the discussion yesterday I've gone ahead and merged back the bridge/release branch to master in the blockchain repository. We need to do this for the projects that use setuptools_scm, so it makes sense to do this for all projects.


Finish Nuxt support

I'm in the process of making Nuxt tests for my plugin, and noticed that the base project does not have full Nuxt support yet.
There seems to be some work on a branch and am curious if that is ready to launch: https://github.com/unjs/unplugin/tree/feat/nuxt
If not it would be best to remove compatibility listings in the project docs.

We kinda leave it on purpose, see #4. The main idea is that Nuxt modules are capable to handle more (for optimizing, or more functionality), it would be better to have the plugin authors write the nuxt module using the unplugin foundation if needed.

If not it would be best to remove compatibility listings in the project docs.

I don't see we mention it anywhere.

In the README you list Nuxt as a viable place to include Unplugins, so my assumption is that something like this:
export default {
  generate: {
    dir: 'dist/nuxt',
  },
  modules: [['unplugin-skypin/nuxt', { /* options */ }]],
}

Is perfectly fine. How would making a Nuxt module using an Unplugin foundation be done?


eval.py error

when run eval.py , it will product error like below :

it is induce by self.inpt that pass from train.py  .   I think it need some modify in source code.

https://github.com/Kyubyong/transformer/blob/master/train.py#L43

https://github.com/Kyubyong/transformer/blob/master/train.py#L43

I met the same question， Thanks @zh215021 ！

same question,soveled,thx.


fix(kicon): remove gear kicon's opacity setting

Summary
the hard-coded fill-opacity on the gear kicon is preventing us from overriding it in other applications
Changes made:
*removed hard coded fill-opacity
PR Checklist

[x] Does not introduce dependencies
[x] Functional: all changes do not break existing APIs and if so, bump major version.
[x] Tests pass: check the output of yarn test packages/
[x] Naming: the files and the method and prop variables use the same naming conventions as other Kongponents
[x] Framework style: abides by the essential rules in Vue's style guide
[x] Cleanliness: does not have formatting issues, unused code (e.g., console.logs), or leftover comments
[x] Docs: includes a technically accurate README, uses JSDOC where appropriate

seems we have inconsistency between kicons on fill-opacity and opacity. Any reason to not just remove fill-opacity from all kicons and have opacity be 0.5 across the board?

I’m thinking all of the icons need some work. We should update to Make then so the same size, same color, same fill, etc. what are your thoughts on waiting to make this change and doing it in an overhaul?

we're just going to use the same icons for now until we get better direction from UX


Caret doesn't render at the beginning of editable text

When there's an editable piece of text with a caret (created from either pyglet.gui.TextEntry and pyglet.gui.Frame, or pyglet.text.layout.IncrementalTextLayout and pyglet.text.caret.Caret) the caret doesn't render at the beginning of the text.
It renders fine if it's not at the beginning:

... But it doesn't render if it's at the beginning (it is at the beginning in this screenshot, but it won't look like it because it doesn't render):

I believe this is because when it renders the text box, anything outside of the text box isn't rendered which probably allows for text scrolling to work properly and some other things, but it also causes the caret to not render when it's at the beginning of the string because it's just outside of that area. Using the default font (Arial for me), the caret renders two pixels to the left of the beginning of the character it's in front of, and there seems to be one pixel of padding from the beginning of the text box culling area to the first character, which means the caret is 1 pixel too far to the left to be rendered when it's at the beginning.
This can be solved by moving the caret one pixel to the right, but that changes visuals a bit so isn't a good solution. Another solution is to extend the culling box thing by one pixel to the left by changing this line:
https://github.com/pyglet/pyglet/blob/b9ed5d3ce61735124983971f91f855c514291a85/pyglet/text/layout.py#L603
to this:
glScissor(self._clip_x-1, self._clip_y - self._clip_height, self._clip_width, self._clip_height)

Which is better, but this still changes visuals slightly by making text scrolled to the right render one pixel more to the left than it otherwise would have.
Normally:

With this change:

(that's a bunch of "m"s making it scroll horizontally and zoomed in using the colour picker tool in PowerToys
A proper solution would be to move all text to the right by one pixel so that the caret has enough space to render even at the beginning. This would minorly affect existing GUIs, but is better than the solution above because it wouldn't be rendering stuff one pixel to the left of the layout's x coordinate which doesn't really make sense.
You could instead maybe somehow make the caret immune to the culling thing, but this might cause issues with scrolling because if you scroll away from the caret and make the text it's on disappear it might still be visible because the culling doesn't get rid of it. You could maybe solve that by instead only adding a special case for when the caret is at the beginning of text, but that would still make it render one pixel to the left of the layout's x coordinate. Alternatively, you could move the caret one pixel to the right only when it's at the beginning of text which would fix this issue, and not move the text in existing GUIs; but it would make the first character have a different distance to the caret than every other character so imo isn't the best solution, though might be depending on how much you care about backwards-comparability.
I'm not sure whether this issue exists in the master (2.0, I think) branch because it seems that the caret doesn't render at all there, whether it's at the beginning or not (using current latest commit da8f0c4d10aeb5aa4de5d5e09b7864b565662b4e), but that seems to be a separate already known issue.
(haven't created an issue/bug report for a decently-sized project before, so I hope I did this right)

Tested with code from https://github.com/pyglet/pyglet/files/7810712/issuePG.txt
It works now on M1 MacOS.

It works for me using v2.0.0 (from PyPI) on Windows 10:

Tested with code from https://github.com/pyglet/pyglet/files/7810712/issuePG.txt

btw, this errored on run for me... It seems like that's a slightly older (though not shown in git history?) version of the text_input.py example, and the latest version of that ran fine for me (thanks to 24108d4).

Layouts now have a Z property, so you will need to change self.layout.position = x, y to self.layout.position = x, y, 0 to get older examples working.

The original issue should be fixed, so I'll close this for now. After accounting for the new Z value, please let us know if you're still having issues.


More love <3

We have one heart on the index.html page can we show love any where else?

I want to take it.

@00aney Yes :)

May i help?

I would love to help :)

@AnitaOlsen hehe pun intended?

Lots of love! ❤ Keep the love going but closing the issue for now :)


KH2 Text Editor Bloats File Size

Unmodified English-patched sys.bar is 96kb. After a few simple edits, 108kb. The only thing changed between these files (attached) is the 183 instances of "Sora" being replaced with "Sanza". I've deduced that the likely problem is new text strings are being declared somehow but with no text appended to them. While the file doesn't crash the game, it causes much of the text to appear blank, likely because of an offset that's happening.
Workflow: Open fontimage and fontinfo.bar in editor -> Open sys.bar -> Export as YAML -> Open in Sublime Text, find all instances of Sora -> Replace with Sanza -> Save, reimport via OpenKH Text Editor -> Save new sys.bar
sys.bar.zip

I did some testing and this behaviour of the inflated sys.bar happens only with the patched one.
The patched version uses a technique to re-use messages that are exactly the same to save space.
The original file behaves in a different way and the Text Editor is built to replicate the original behaviour. But an optional save option to "optimize" the exported text can be built in the editor!

Saving the attached file optimized seemed to have worked (i fought Terra and Zexion, which in the past had the same issue for me).


Implementing player animations

Description
Task: Player animations
Feature: Player sprite update #117
This task consists of using the different sprites of the player to implement animations when the playing is moving right, left etc to improve the design and make the game more interactive.
Dependencies

#154 [Player Animation Component]
[Updated Sprite Generation]

Milestones
List of steps that need to be completed for this task.

[x] Implementing the new sprites/adding them to the game (Sept. 9th)
[x] Making sure that they update when the user presses on the movement keys (Sept. 10th)
[x] Create JUnit tests to check that the player sprite updates correct when pressing corresponding arrow keys (Sept 10th)

Completion Deadline: Sept. 10th
Documentation

Wiki Page For Player Design

Member

@ATOApplett (Alex Ong)
@amacn13 (Alex Macnamara)

finished tests


Memory footprint in Google Colab

Thank you so much for this great model ! Wondeful job ! I have just a little question about the memory required for the separation. The model seem use a lot of memory and require to split the audio of a full song (> 1min / 1min30) in Google Colab (free version because no pro version for european users) and resample the audio to from hires (96000Hz) to lowres (44100Hz).
The current jupiter notebook show only process on very short samples (youtube video), I've slightly modify the code to allow using audio from Google Drive but seem to be limited to low resolution / short duration audio file without using splitting/merging audio subprocess. The same limitation of RAM footprint was resolved with Spleeter (Deezer) by a similar method but with some constraints (zero padding to remove in audio) (issue here : https://github.com/deezer/spleeter/issues/391#issuecomment-652202433).
Is someone already do the job?

Hi MaxC2, thanks for the feedback.
As you mentioned, you have to resample the input file into 44100Hz audio file.
I'll add some code lines for auto resampling later.
Also, you don't have to manually split and merge audio sub-process.
When you call the separate_track function of a pretrained model like
separated = model.separate_track(track.audio, 'vocals')
It automatically split the given track into several sub-audio (each sub-audio has the same number of samples, and  the last sub-audio is zero-padded).

Below is the code for this.
 def separate_track(self, input_signal, target) -> torch.Tensor:

        import numpy as np

        self.eval()
        with torch.no_grad():
                db = SingleTrackSet(input_signal, self.hop_length, self.num_frame)
                assert target in db.source_names
                separated = []

                input_condition = np.array(db.source_names.index(target))
                input_condition = torch.tensor(input_condition, dtype=torch.long, device=self.device).view(1)

                for item in db:
                    separated.append(self.separate(item.unsqueeze(0).to(self.device), input_condition)[0]
                                     [self.trim_length:-self.trim_length].detach().cpu().numpy())

        separated = np.concatenate(separated, axis=0)

        import soundfile
        soundfile.write('temp.wav', separated, 44100)
        return soundfile.read('temp.wav')[0]

The pytorch dataset API SingleTrackSet automatically splits the given track, in an on-the-fly manner.
After iterating every sub-audio file, separate_track merges all outputs by separated = np.concatenate(separated, axis=0)
Thank you.

Yes, it's maybe because i've attempt to use to load 96KHz audio with librosa (sr=96000) before calling separate_track and get a kick out from Google Colab out of RAM. I have retry with a 44.1KHz cutted at ~1min30. So now, I will test will a full song resampled at the right sample rate. Thank you very much for your support, and once again well done for your great model !

OK, I've done some test. The problem come from the use of embedded audio player  display(Audio(audio, rate=rate)) that seem duplicate the audio in some manner and use a lot of RAM. So for a big audio file (ex. more 10 minutes) you're always kicked out from Google Colab out of RAM limits.
To do the trick, the idea is not to use the embbed audio preview and directly call the separate_track process.
In a draft form, using my audio stored in Google Drive, i've write two new cells. The first one is the comman Google Drive mount:
from google.colab import drive
drive.mount('/content/gdrive', force_remount=True)

The second load any audio file, resample (best filter quality) and convert to stereo if needed. Each processed temp.wav are renamed and writed in a destination subfolder in Google Drive (separated for my case) in order to facilitate the download (zip file).
import os
import shutil
import librosa
import resampy

gcolab_root = '/content/Conditioned-Source-Separation-LaSAFT/'
gdrive_root = '/content/gdrive/My Drive/'

destination_folder = 'separated'

default_sample_rate = 44100
sources = ['vocals', 'drums', 'bass', 'other']

def load_audio(audio_path):
  audio, rate = librosa.load(audio_path, sr=None, mono=False)
  if rate != default_sample_rate:
    audio = resampy.resample(audio, rate, default_sample_rate, filter='kaiser_best')
  is_mono = audio.ndim == 1
  if is_mono:
    audio = np.asfortranarray(np.array([audio, audio]))
  return audio, rate, is_mono

def separate_all_sources(audio, gdrive_path):
  for src in sources:
    print("separate '%s'" %src)
    model.separate_track(audio.T, src)
    shutil.copy(os.path.join(gcolab_root, 'temp.wav'),
                os.path.join(gdrive_path, src + '.wav'))

# prepare google drive destination folder
path = os.path.join(gdrive_root, destination_folder)
try:
    os.makedirs(path, exist_ok = True)
except OSError as error:
    print("Directory '%s' can not be created" %path)

print('load audio source')
audio_file = os.path.join(gdrive_root, 'audio/stairway/center.flac')
audio, rate, is_mono = load_audio(audio_file)
separate_all_sources(audio, path)
print('finished')

I need to add a fallback to audio format:

back to mono
back to original sample rate

But for the moment this process work great on my audio files (>10mn, 96kbits 24 bits) without offline preprocessing.
Maybe the idea will be to add a extra method in the python code that do not write the test.wav in the root project folder, but a named .wav (vocals / drums / bass / other) in a project temporary subfolder (separated for example). And do a zip over the folder with a download link in Google Colab after the separation.
That can help some potential users that do not have a Google Drive account.
For my it's OK and fine. Thank you very much.

Thank you for sharing your experience.
I'll update the code to reflect what you've recommended, sooner or later 👍


Exposing all test models

trying to expose all 3 versions of the models.
Note:  should I be creating separate functions for each or be doing something like how I've done it now?

I think it's fine at this stage of this library to change the API and have two functions like you did.
We can remove the constant_no_trend_test functions.
And since the name of test is in the name of the function, in prelude module, we can get ride of the inner modules adf and dickeyfuller and directly 'pub use' the two functions.

What do you think?

sounds good


Db diagram

Hi Code Reviewer👋
In this PR:
-⭐Created DataBase from the diagram:

-⭐Create table for many-to-many relationships.

Hey @1GORDON Thanks a lot for your efforts in reviewing our code

Hey @aamirkhan2478 Thanks a lot 💯 for your efforts in reviewing our code, we really appreciate it 😄

Hey @aamirkhan2478 Thanks a lot 💯 for your efforts in reviewing our code, we really appreciate it 😄

@MostafaShahat2021 my pleasure bro


Slack webhooks should be configurable on the Canary object level

Our organization has a lot of teams so Canary alerts through slack would be hard to find in a single channel for the org. A good long term feature would be to have per-canary Slack channel options so that slack messages can be routed to specific channels for the team that owns that service.

One option would be to use a Slack bot instead of a webhook since a bot can invited to multiple channels. In the canary definition you would add a Slack channel and the Slack token could be a secret mounted as env var in Flagger's pod.


[BUG] missing constexpr causes constants.

missing constexpr for reverse and backward views cause constatns to appear in the binary.
https://github.com/jfalcou/eve/blob/8663aedb52bfdd79d93043c4a81f1d2bca5cc1a9/include/eve/algo/views/reverse.hpp#L90
https://github.com/jfalcou/eve/blob/8663aedb52bfdd79d93043c4a81f1d2bca5cc1a9/include/eve/algo/views/backward.hpp#L97
Godbolt: https://gcc.godbolt.org/z/oebx9vn6W

Fixed by https://github.com/jfalcou/eve/pull/1355


Cache not updated

So as mentioned on twitter the cache doesnt update even it go triggered as you can see in the log file.
2015-07-13 16:10:20,261 [6] INFO  Umbraco.Core.Publishing.PublishingStrategy - [Thread 28] Content 'Satelec Scaling-Spitze 10X' with Id '1912' has been published.
2015-07-13 16:10:26,185 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 28] Issueing Full Reset Cache after 'reload entity with id'
2015-07-13 16:10:26,185 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 28] Request to reset cache, (re)setting timer
2015-07-13 16:10:36,192 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 36] Request to reset cache, timer triggered
2015-07-13 16:10:36,196 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 48] Rebuilding new cache
2015-07-13 16:10:37,105 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 48] Switching to new cache
2015-07-13 16:10:37,105 [6] INFO  uWebshop.Umbraco6.UmbracoLoggingService - [Thread 48] Switched to new cache

is there a quick fix to refresh the cache?


Header "component"

not really web component, just template

As discussed with Jake this is going to be njx include - no markdown to edit the text

This landed with #115


Collapsible not working.

It appears that the Collapsible View is not working in ios. I've tried updating to the latest version ("react-native-collapsible": "^0.8.0") and I am still having issues.
Seems this remains hidden regardless.
<Collapsible collapsed={false}>
      <Text> Yo </Text>
</Collapsible>

Are you on React Native 0.41?

No 0.40.0.

Works!!!! :) Thanks


could anybody tell me how to install codemirror2 on react18.2.0

Having same issue

O man, I'm also having the same issue


Cannot add module @nuxtjs/axios to project

After install and add module
modules: [
    '@nuxtjs/axios'
]

npm nuxt crashed with message
[nuxt] Unable to resolve module @nuxtjs/axios
[SyntaxError: Unexpected token {]
when I remove this line '@nuxtjs/axios', the project starts correctly. Where is mistake?

Would you please provide full version of your nuxt.config.js ?

it crashed event at not filled config file,  example
module.exports = {
  modules: [
    '@nuxtjs/axios'
  ]
}

I have tried to remove line by line in file nuxt.config.js

Are you sure also using latest nuxt version 1.0.0-alpha.4 ? That's strange problem and i can reproduce it .Please double check installed version and cleaning up .nuxt  & node_modules before run.
https://mica-bulb.glitch.me/
https://glitch.com/edit/#!/mica-bulb?path=nuxt.config.js:3:18

Yes, nuxt version is latest: 1.0.0-alpha.4.
I've copied you package.json to new folder, runned npm install and copied nuxt.config.js
{
  "name": "nuxt-hello",
  "scripts": {
    "start": "nuxt"
  },
  "dependencies": {
    "nuxt": "^1.0.0-alpha.4",
    "@nuxtjs/axios": "latest"
  }
}

module.exports = {
  modules: [
    '@nuxtjs/axios'
  ]
}

And result was same:
S:\web\test\nuxt-test>npm run start
> nuxt-hello@ start S:\web\test\nuxt-test
> nuxt

[nuxt] Unable to resolve module @nuxtjs/axios
[SyntaxError: Unexpected token {]

Perhaps this depends of operation system> My os is Win7

Success! I was because of old node vesrion. I've updated it to latest version 6.11.0 and all is done! THank you very match and my apologises!

I am using 16 and it happens to me. I don't think it is due to node version (for me).


JACK CV in/out?

Hi. JACK sound system (and the LV2 plugin format) support audio-rate control signal to be patched between apps (and plugins). It would be neat if torchsynth could interface with that some day.

Thanks for the suggestion @mxmilkiib -- super cool idea. If I understand correctly this would be for CV signals to be transferred to and from apps and plugins in realtime? Is there python support for this?
If it is more for realtime applications I'm not sure if it is totally inline with the current torchsynth paradigm which is focused on fast offline parallel rendering of patches as opposed to realtime synthesis. Very cool idea but not sure how it would integrate. Open to suggestions though and thanks again for the input!


QuantizedReshape Op support

Current quantization process will produce QuantizedReshape op which uTensor do not support yet
Reference:
https://github.com/tensorflow/tensorflow/blob/f7ec99516ce0e0937e0b865e90aa02c748cd36c6/tensorflow/core/kernels/quantized_reshape_op.cc

Yes.
However we need it if we want to make code generator support normal reshape operations.
Some reshape op in Tensorflow will be transformed into quantized reshape op depending on the architect of the input graph.
As far as what I know, there is no workaround on this instead of implementing the quantized reshape op.

Hello,
I tried uTensor with the deep_mlp example and I encountered the following error

Any suggestions and help would be really helpful to proceed further.

I would like to use uTensor for my neural network model to be implemented on the controller. I am facing the same problem even if I provide my file as input.

@akshu2025  Thanks for bringing this to our attention. At the moment, this look like a quantization problem rather than missing operators. @dboyliao Might be a better person to respond to this. In the meantime, could you please try feeding the utensor-cli with a none-quantized pb file?

Hello,
I think you are right. Its working without quantistion:) The cpp and hpp files are generated.
I have a query, It would be really helpful if you can help in solving it.
I have trained a LSTM model in Keras and used keras_to_tensorflow.py from github to convert it to a tensorflow freez graph. My aim is to implement it on controller and hence i am using uTensor.
I have not quantised my pb file, but I am facing the below error. Any idea regarding this.

@akshu2025 Honestly not too sure about that error, might be related to the following
https://github.com/tensorflow/models/issues/1879
https://github.com/uTensor/utensor_cgen/issues/27
My guess is there is an issue with handling the recurrence from the LSTM.
Is this the script you are using to do the transform? https://github.com/amir-abdi/keras_to_tensorflow/blob/master/keras_to_tensorflow.py
Finally, can you try removing the "quantize_nodes" flag on this line?
https://github.com/uTensor/utensor_cgen/blob/develop/utensor_cgen/transformer/quantize.py#L15

Hi, @akshu2025.
Thanks for trying the code generator on a RNN.
This is sth I didn't have time to test it through.
As for the problem you encounter, I can better figure out what the bug is if you can provide me the pb file you use.
Is that pb file confidential?

Hello @dboyliao and @mbartling  , Thank you for the reply:)
Yes, I am using the script keras_to_tensoflow as you stated in the link @mbartling
I think I am using a different branch of utensor which do not contain the file quantise.py (i did pip install utensor_cgen)
But I have disabled quantize field from keras_to_Tensorflow.py file, but no luck, seems to give same error.
Neverthless, I will try with develop-branch of the uTensor which you mentioned above and update the status.

The error seems to be the same with latest uTensor0.1.11.
Also, uTensor0.1.11 does not support Identity op? (found this when I tried with another example)

It's probably because of training ops.
You can take a look at this demo example for how to serve a model to uTensor:
https://github.com/uTensor/utensor-mnist-demo/blob/master/tensorflow-models/deep_mlp.ipynb
I should make it clean or check for you in the code generator.
Nice user feedback!

@dboyliao : I cannot share my .pb file as it is confidential, but it regenerated the error using the LSTM network similar to mine and I am sharing that .pb file
LSTM_stateful.h5.pb.zip
The error is

@dboyliao @akshu2025 @mbartling @Knight-X
Although the Context class has features to support RNN (via static_add), I don't believe we have started the work on the code-gen side. It's in the plan though but still probably months away.
My recommendation is to convert it to a MLP (or, CNN if you can wait for a couple weeks), and run this.
I should also mention that we are working on a contributor guide, if that's of interest : )

I agree.
As for me, it's always good to look into what's going wrong for the code-gen.
At least it can tell me what's the limitation of code-gen or what should we do next for resolving the issue (maybe a better IR, which is crucial to the code-gen).
I may not do it right away, but it's still good to have a first grasp of the idea.

@akshu2025 Sorry to say that we should have made it more clear what's supported at the current time. For now, can you please try prototyping with FC or CNN?

@neil , I am using a time series data. So I am not sure if I can really
achieve the required output with CNN or FC.
Nevertheless I will give you my feedback on Monday after consulting my
supervisor.
I would really appreciate if you have any suggestions regarding
implementation of neural network model on to microcontroller.
On 15-Jun-2018 7:10 PM, "Neil Tan"<EMAIL_ADDRESS>wrote:

@akshu2025 https://github.com/akshu2025 Sorry to say that we should
have made it more clear what's supported at the current time. For now, can
you please try prototyping with FC or CNN?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uTensor/uTensor/issues/97#issuecomment-397685383, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbaDbyGaryg-0s8J1Ex8twxfRklb8y6Aks5t8-pogaJpZM4TeXiM
.

@akshu2025
Understood
Checkout https://github.com/uTensor/ADL_demo
A far cry from RNN, but that’s how I deal with it now.

@neil
Ya sure, I will check with this version and update you.
On 15-Jun-2018 8:49 PM, "Neil Tan"<EMAIL_ADDRESS>wrote:

@akshu2025 https://github.com/akshu2025
Understood
Checkout https://github.com/uTensor/ADL_demo
A far cry from RNN, but that’s how I deal with it now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uTensor/uTensor/issues/97#issuecomment-397711094, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbaDb83pN5qRUt7olvF8Z4-beL2MEAAmks5t9AHBgaJpZM4TeXiM
.

@akshu2025
I inspect the pb file you gave me, I summarize a few things here:

Identity op is not available in uTensor (should be a quick fix). You can use --output-nodes option to specify the output nodes rather than the Identity op (in your pb file, it would be dense_4/Softmax)
Tensorflow api fails on converting tf.resource to a numpy data type which is used in utensor_cgen. Maybe it's our implementation fighting with Tensorflow.

BTW, there is no documentation on tf.resource (if you consider one line of sentence as documentation, well, there is one) and the tf team seems not care about any user's questions. Every stackoverflow threads about tf.resource is not properly answered. So the best I can do is to read the source code which takes time to figure out how to fix it in the code generator. So even you fix the first issue with --output-nodes, I can guarantee you this will not work.

Sorry for the bad news, I've tried my best. Maybe using numpy dtype as a mapping for tf dtypes in code generator is a bad idea. But what can I say? There is a public api in Tensorflow which convert a tf.Dtype into numpy dtype so I though it would be great to use numpy dtype as a generic representation for tf.DType in code generator.
As what I've found now, maybe the answer is a big NO! FML.

BTW, you can find the tf.resource documentation here:
https://www.tensorflow.org/api_docs/python/tf/DType
And if you click on tf.resource, it give you a 404.

I've read the source code of RNNCell or LSTMCell in Tensorflow.
I think it's possible to remove all the nodes we don't want but still implement a RNN as desired in Tensorflow.
@neil-tan @mbartling I can do this maybe in the next release. I think it's better to say that we don't support RNN in uTensor for Tensorflow.

Thank you so much for the reply @dboyliao @neil-tan @mbartling
I think FC or MLP does not serve my purpose.
Nevertheless, If I am using this tool, I will surely update it here.

@akshu2025 Understood. Check back here from time to time. We are releasing the Test-Release in  next couple days. Mostly, updated readme. But, you may find the future roadmap useful.


Fixed CSS of navbar

Related Issue
#799
Description
Fixed CSS of Navbar
Type of PR

[X] Bug fix
[] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[X] I have performed a self-review of my code
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
please merge the issue and add gssoc badge @Suchitra-Sahoo

Please check the PR I have fixed the issue and merge it asap with gssoc label
Thanku @Suchitra-Sahoo

Please merge this PR @Suchitra-Sahoo with gssoc label


v8 Ring no longer supports different passwords per shard

v8 Ring no longer supports different passwords per shard. In v7 there is a Passwords field that allows user to set password per shard. In v8 this is no longer available somehow.

See the last example in https://redis.uptrace.dev/guide/ring.html#quickstart


Fully Structured Repo

In this PR, I have edited the repo as per @mariobehling  guidelines of making three different folders of images, schematics and experiments and merging Experiments.md with README.md
Solves issue : #3

@mariobehling   @CloudyPadmal   Please review this PR and suggest changes if needed.

Thanks. Let's also add links on the reader.


Add 2D trajectory builder option to accept initial pose estimation

This PR add an option to mapping_2d::LocalTrajectoryBuilder to accept initial pose estimation.
User can provide initial pose estimation when they start a new trajectory.

Can you elaborate a bit on the aim of this PR?
As it is right now, it seems there is no need to add this to Cartographer. In the single trajectory case, you could just transform the poses outside of Cartographer itself. If there are multiple trajectories, you'd check the connectivity to determine whether you'd want to transform them.
Most importantly, the proposed change does not properly influence results, e.g. like one would want if there is already knowledge about where to start related to an already existing trajectory.

This PR aims multi trajectory cases.
The motivation of this PR is:

I'd like to use a map before it is well optimized to navigate robots if I can provide a appropriate initial estimation.
Sometimes it's useful to adjust initial coordinate of a entire map by giving initial estimation to the first trajectory. (e.g. To build a map with the coordinate same as former maps but need to change start point)

Most importantly, the proposed change does not properly influence results, e.g. like one would want if there is already knowledge about where to start related to an already existing trajectory.

I expected this scenario and tested in such situation. It seems to work well in that case.
Can you tell me a little more detail why do you think this doesn't properly work?

Can you elaborate more about your use cases.

If you are only interested in changing the coordinate system, this could obviously be done on the outside, i.e. why is just transforming the output not sufficient for your use case?

This change does not influence loop closures because it does not change trajectory_connectivity_ in SparsePoseGraph. See also #329.

It seems to me the feature you want should be added on the pose graph level, not the local trajectory builder. The reason for that is that conceptually the poses in the local trajectory builder track the relative changes without loop closure. The optimized poses in the pose graph can differ a lot. Your change sets the local pose to an initial estimate in the map frame of the pose graph. Since the pose graph does not know about this it will still treat the initial pose as arbitrary and do global loop closure matches against all other submaps.

@aochiai What is the status of this PR?

Closing this for now, please re-open if you disagree.

Thank you for reviewing and comments.
I will revise my PR.


Fix auth in GUI not working.

Updating the top flow would need to be done inside the event emit as there are 2 other places where this would be needed.
I don't want Roles or Gui to depend on each other, so although this would be very small having its own file would be preferable.
Best solution might be to use _C.opt_require inside gui/_require.lua to check if the roles module exists then connect to it.

I believe that a494391  should fix that

I will be treating this like a patch, so I am changing the base branch to merge into a patch.


add Semiring Disj instance

PR incoming

I'm actually not sure that this is a great idea, since Disj is meant to focus on one aspect of a potential semiring. I may implement what I need in another way...

If you do do it we have a purescript-semirings too, so it may be more appropriate there - I think it depends on monoids - https://github.com/purescript/purescript-semirings.

Oh, nevermind, that would be an orphan then, of course.


Bug: Invalid handle of PUT data

PHP Version
8.0
CodeIgniter4 Version
4.1.5
CodeIgniter4 Installation Method
Composer (as dependency to an existing project)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
No response
What happened?
I get my HTML forms and send it with proper HTTP Method by JavaScript code, that looks something like this:
formData = new FormData(form);
let xmlHttpRequest = new XMLHttpRequest();
xmlHttpRequest.open(form.dataset.method, url);
xmlHttpRequest.setRequestHeader('X-Requested-With', 'xmlhttprequest');
xmlHttpRequest.responseType = 'json';
xmlHttpRequest.send(formData);

My example PUT request look something like that:
Header (that count here):
Content-Type: multipart/form-data; boundary=---------------------------293903230717728861911868140501
Data (in this example email is empty):
-----------------------------293903230717728861911868140501
Content-Disposition: form-data; name="name"

asdf
-----------------------------293903230717728861911868140501
Content-Disposition: form-data; name="email"

-----------------------------293903230717728861911868140501
Content-Disposition: form-data; name="password"

asdf
-----------------------------293903230717728861911868140501
Content-Disposition: form-data; name="active"

1
-----------------------------293903230717728861911868140501--

While getting data via $request->getRawInput() (IncomingRequest Class) I get:
array(2) {
  ["-----------------------------293903230717728861911868140501
Content-Disposition:_form-data"]=>
  string(0) ""
  ["name"]=>
  string(78) ""active"

1
-----------------------------293903230717728861911868140501--
"
}

which is definitely not correct.
Adding <input type="hidden" name="_method" value="PUT" /> do not change anything.
When subbmiting same form, same code via JavaScript but by POST method, POST data are correct:
array(4) {
  ["name"]=>
  string(4) "asdf"
  ["email"]=>
  string(0) ""
  ["password"]=>
  string(4) "asdf"
  ["active"]=>
  string(1) "1"
}

If I send via POST but add in JavaScript: formData.append('_method', 'PUT'); before send I got proper result (with _method field) from getPost() method and empty array from getRawInput() method.
Steps to Reproduce
Send PUT request base on HTML Form data by ajax like:
formData = new FormData(form);
let xmlHttpRequest = new XMLHttpRequest();
xmlHttpRequest.open('PUT', url);
xmlHttpRequest.setRequestHeader('X-Requested-With', 'xmlhttprequest');
xmlHttpRequest.responseType = 'json';
xmlHttpRequest.send(formData);

Try to read requested data like that in controller:
$data = $this->request->getPost();
$data2 = $this->request->getRawInput();
var_dump($data, $data2);

Expected Output
Array with proper data from getRawInput() or just untouched raw input.
Anything else?
Controller method that handle that request is declared in Config Routes by add method.

Can't reproduce on develop.
<?php

namespace App\Controllers;

class Home extends BaseController
{
    public function index()
    {
        $data2 = $this->request->getRawInput();
        var_dump($data2);
    }
}

I try to reproduce it on local server also and don't succeed. Then I upload test script to server and then I've got error.
On fresh install (I download CI4 from page).
My routes config:
$routes->get('/', 'Home::index');
$routes->add('/test', 'Home::test');

My Controller (replace example.com with Yours domain):
<?php

namespace App\Controllers;

class Home extends BaseController
{
    public function index()
    {
        $body = <<<EXAMPLE

<form><input type="text" name="name"><input type="email" name="email"><input type="password" name="password"><input name="active" value="1" type="checkbox"><button type="submit">Save</button></form>

EXAMPLE;
        $this->response->setBody($body);
        $this->response->send();
    }

    public function test()
    {
        $data = $this->request->getRawInput();
        var_dump($data);
    }
}

My request data:
-----------------------------140165576325122211682143310948
Content-Disposition: form-data; name="name"

Test
-----------------------------140165576325122211682143310948
Content-Disposition: form-data; name="email"

<EMAIL_ADDRESS>-----------------------------140165576325122211682143310948
Content-Disposition: form-data; name="password"

123
-----------------------------140165576325122211682143310948
Content-Disposition: form-data; name="active"

1
-----------------------------140165576325122211682143310948--

And response:
array(2) {
  ["-----------------------------140165576325122211682143310948
Content-Disposition:_form-data"]=>
  string(0) ""
  ["name"]=>
  string(78) ""active"

1
-----------------------------140165576325122211682143310948--
"
}

My PHP version is 8.0.10.

@kenjis Any idea what's happened here?

@pawelkg
I can't reproduce the error.

I tried with Nginx server. But can't reproduce it.

@pawelkg
Can't reproduce the error, so it is not a bug of the framework.
It seems depending on your environment.
Please ask your support questions in the forums. Thanks!
https://forum.codeigniter.com/forum-30.html


Fix lakefile.lean capitalisation of @[defaultTarget]

When I install a fresh nightly toolchain I get errors while trying to build mathlib.
I hope this change will fix that.

Nevermind, it seems I'm just confused as to which toolchains I can install crash-free and which can actually build mathlib. It appears that default_target is the new casing.

mathlib4 only promises to work on the lean nightly version specified in the lean-toolchain file. You should use the same version as mentioned there in your own projects depending on mathlib.


Linux development environment setup

Add Linux instructions

@ines-cruz ping here when this is ready to be reviewed again!


fix: white space margin

Closes #937

Removed scroll jumping
All white margins at all breakpoints should be gone - should now look like this:

Medium breakpoint

TESTING:

[ ] http://localhost:3000/

@francinelucca not seeing it 👀 let me know if you see it again?

I'm not seeing the white to the right, unless my eyes are playing tricks on me. Can we add some comments to the css on why we have the -1px/1px margins, this feels weird to me to have to add and would be good to explain why its there.


[Enhancement] Navigation extension method to DIsmiss popup

Summary
Provide some way to Dismiss from within ViewModel (for example another Navigation extension). Right now I need to pass the whole page into view model to be able to dismiss it
API Changes
Dismiss(this Navigation)
Intended Use Case
To be able to easier use Popup in MVVM
Who Will Do The Work?

[ ] I am willing to take this on myself
[x] Just putting this out there for someone to pick up

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


The Discord join link is out of date

The "Find us at" section in the footer points to an out-of-date Discord link https://discord.gg/YWSDRqdW5n
I don't have enough experience with Discord servers to know who's generating these links.

I'll work on getting it fixed...for now: https://discord.gg/gbNYE8H8... @taipeicityeth?


Add support to pick up schemas again

With the Azure team's breaking change for resources to be objects instead of arrays

Estimate of ~3 days work


Suggestion for improvement

When the wizard is complete, click Exit on the Configuration complete page. Group Writeback will be automatically configured.

After enabling Group writeback, it is not clear that you still need to enable the V2 feature using Powershell.
Step 8 implies that Group Writeback will be configured, where the admin needs to do an extra mandatory step.

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

ID: 0ef5e848-ca7a-8924-8da2-73163a489e5b
Version Independent ID: ad118082-f4bb-624c-b541-942ba235f370
Content: Azure AD Connect: Group Writeback - Microsoft Entra
Content Source: articles/active-directory/hybrid/how-to-connect-group-writeback-v2.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

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

@BakkerJan
Thank you for pointing this out. Let me investigate and revert back to you

@BakkerJan Thank you for pointing this out. Let me investigate and revert back to you

Thanks. I just enabled group writeback from the ADConnect server (latest version) and it seemed to enable V1. My guess was that V2 would be enabled from the wizard directly, if group write back was not enabled before.
Also, it is not very clear to me, how to migrate from V1 to V2. Whats happens when you enable V2 if you had writeback enabled already?
Thanks again!

@BakkerJan Google-fu led me here and you've solved my problem. Was only getting M365 groups despite enabling everything for security groups. Thanks!

@BakkerJan
Yes, to enable this feature both steps have to be peformed in the same order mentioned in the article. I have also informed my internal team to mention as steps 1 and step 2 in the article to avoid the confusion.


[W5.10c][W14-A2]Sim Kwan Tiong Damien

@Damienskt
Some comments added. Please ack & close the PR after reading comments.


Allow excluding some constants from Style/Documentation

For example, a very common idiom is to define a ClassMethods module that
contains the class methods created by the parent module. In such cases,
it might be desired to document the parent module, but not the
ClassMethods module.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks, updated according to comments

The CI build is currently failing.

The CI build is currently failing.

Indeed. It appears this is because I modified the default.yml and thus many tests fail because the generated config is now larger. What is the expected change here? Should I change the auto gen config test? Or should I remove the change from the config? Both changes seem to make things work.
BTW, should I add a VersionChanged: <<next>> tag?

Should I change the auto gen config test?

That'd be fine. Sorry about the radio silence. I somehow forgot I had to respond here.

Thanks, I updated the PR. I had to extract a nodoc_self_or_outer_module? method, otherwise check would fail the CyclomaticComplexity cop. I couldn't find a better name :)

The CI is failing and I see there's a conflict in one of the specs.

Thanks!


✨ Custom service readiness health check

What are you trying to do?
Some software may need more complex readiness checks in addition to checking port availability (eg. to deal with eventual consistency).
Most common example is calling an HTTP endpoint and checking it's result.
Why is this important to you?
Both Docker Compose and Kubernetes have these kinds of checks. When moving from any of those to a Dagger based developer environment, this may be an important bit when checking feature parity.
How are you currently working around this?

Call Start on the service
Run a health check in a Container and wait for it
Continue with the test/build/whatever I'm doing in Dagger

The resulting code is not very nice though and requires several synchronization steps.

We've talked about customizing healthchecks before, e.g.:

https://github.com/dagger/dagger/pull/5158#issuecomment-1576536740

May be time 🙂

Additionally, it would be nice to support [service_completed_successfully](https://docs.docker.com/compose/compose-file/05-services/#depends_on:~:text=service_completed_successfully) since it would ensure services/commands are executed sequentially. Related discussion https://discord.com/channels/707636530424053791/1229517338991132672/1229517338991132672


Custom masking function

I have cases with different types of atoms which don't interact. I would like to minimise the size of the pairlist buffer at setup so have implemented an optional function.

I have now signed CLA, please rescan

Having used it I now seen the sparse neighbor list case too.
Index of particle 1 is given by the index of the array in the first dimension.
Index of particle 2 is given by the value in the array.
I will update the documentation slightly as I had said ‘square’ array

Thanks for the contribution, I think a custom mask function could make a lot of sense.
A few quick comments:

Neighbor lists are constructed using a few phases. First, a "broad phase" in which candidate neighbors are identified (possibly using cell-lists) and then a pruning step in which only candidates that are within the specified cutoff are included. Currently, the mask is applied in between the broad phase and the pruning phase. However, the pruning phase is also the step that converts the neighbor list into the appropriate format. Therefore, as written the mask is always applied to neighbor lists whose format resembles the dense case. (It is useful to apply the mask before pruning because then the capacity is properly estimated). Therefore, I actually think that your original documentation was correct!

Can you include a new test following the example either in energy_test.py or partition_test.py that exercises the custom mask functionality?

Once these changes are addressed I'm more than happy to merge. Thanks again!

Thank you for the speedy reply.
Replying to your comments:

When cell list is enabled and a sparse representation is used, idx is not square.
When you have cell list disabled, it's square.
In the image below, the top two are from cell list on, bottom one is cell list disabled.

Will add tests
This is from the notebook in #168 and may link to it.

Thanks! Just to clarify, whether or not idx is square has to do with the use of cell-lists rather than the format of the neighbor list. When cell lists aren't used idx is an (N,N) matrix containing every pair of ids. This list is then pruned, but it costs O(N^2) time. When using cell-lists, the initial idx list is (N, max_neighbors_per_cell) before pruning. In this case, the pruning function is the same but it costs O(N * max_neighbors_per_cell) time which can be much faster for large systems. Typically the max neighbors per cell is a constant independent of N.

Yes. All makes sense. (Though max_neighbors_per_cell being ~24x the number of particles in the system seems a bit excessive.)
Tests added, you'll see a simple base wrapping of the function acceptable_id_pair to work with the neighbor lists.
Base function:
def acceptable_id_pair(id1, id2):
  # Don't allow particles to have an interaction when their id's are closer than 3 (eg disabling 1-2 and 1-3 interactions)
  return jnp.abs(id1-id2)>3

Wrapper:
def mask_id_based(idx: Array, ids: Array, mask_val: int, _acceptable_id_pair: Callable) -> Array:
  '''
  _acceptable_id_pair mapped to act upon the neighbor list where:
      - index of particle 1 is in index in the first dimension of the array
      - index of particle 2 is given by the value in the array
  '''
  @partial(vmap, in_axes=(0,0,None))
  def acceptable_id_pair(idx, id1, ids):
    id2 = ids.at[idx].get()
    return vmap(_acceptable_id_pair, in_axes=(None,0))(id1,id2)
  mask = acceptable_id_pair(idx, ids, ids) # Note the id's twice, one iterated, one is passed fully in (to then be coverted to the second, value-based, index)
  return jnp.where(
    mask,
    idx,
    mask_val
  )

ids = jnp.arange(n_particles) # id is just particle index here.
mask_val = n_particles
custom_mask_function = partial(mask_id_based, ids=ids, mask_val=mask_val, _acceptable_id_pair=acceptable_id_pair)

So @sschoenholz is this now ready for merging?

Okay, now it should be in a shape to merge.


Add Trend Quality Indicator

Trend Quality was created by David Sepiashvili and was unveiled in an issue of Stocks and Commodities magazine in 2004.
The basic idea behind it is that it is not enough to rate a market as "trending" or "not trending".  We must elaborate more on the "strength" or "quality" of a trend so that we can make better trading decisions.  In the original issue, the author created 2 indicators called the trend quality "q" and "b" indicator.  I won't discuss the "b" indicator here because I think it is less useful and adopted less by the trading community.
The "q" indicator is quite useful because it returns values based on the quality of the trend. From the author:
Readings in the range from +2 to +5, or from -2 to -5, can indicate moderate trending, and readings above Q=+5 or below Q=-5 indicate strong trending. Strong upward trending often leads to the security’s overvaluing, and strong downward trending often results in the security’s undervaluing.
I myself have noticed tq values go as high as 20 since in FX markets can seemingly trend forever.
Since it's unveiling tq has been rewritten for a few trading platforms, most notably mt4/5, but you can also find versions for it in ThinkOrSwim and TradingView.  The version you see in most trading platforms is the "q" indicator and it is directional in nature.
The goat of MT4/5 indicator development mladen did a megathread on trend quality which you can find here: https://www.mql5.com/en/forum/181010.
I tried to port trend quality to python but I haven't produced anything usable yet.  Here's what I've done so far: https://github.com/kbs-code/trend_quality_python/tree/master
In that repo, the file https://github.com/kbs-code/trend_quality_python/blob/master/reference/trend_quality_q1_directional_indicator.mq4 is a good reference for how the indicator should behave in mt4.
In the mql4 code, the indicator works on a for loop as it iterates over close prices.  I tried to reverse engineer this behavior and apply the calculation in a vectorized manner using pandas data frames but failed.  I've considered using lambdas or itertuples and have avoided trying iterrows because it's frowned upon.  I've tried masks as well but I'm afraid that might introduce look-ahead bias.
Sources:
https://www.prorealcode.com/wp-content/uploads/2019/03/24-Q-Indicator.pdf
https://github.com/kbs-code/trend_quality_python/tree/master
https://www.mql5.com/en/forum/181010
https://usethinkscript.com/threads/trend-quality-indicator-with-alert-for-thinkorswim.9/
https://www.tradingview.com/script/k3mnaGcQ-Trend-Quality-Indicator/

added "pandas_ta - like" boilerplate code for tq.  Still stuck on calculating tq values.  Please see https://github.com/kbs-code/trend_quality_python/blob/master/research.ipynb

added cells demonstrating chunking by macd sign
https://github.com/kbs-code/trend_quality_python/blob/master/research.ipynb

Job post for this task added on Upwork: https://www.upwork.com/jobs/~012fd863b8c28f53ab

@kbs-code
So you want to pay someone other than the maintainer? 🤷🏼♂ 😕

When I posted that, I thought that you could take up the task as well.  It was open to anyone.

Hi @kbs-code, I came up with the following code. However, I am not sure if my chunking works as intended since I used a different approach that reduces the number of loops to the number of chunks instead of iterating over each row. But feel free to take a look and try it out. Hope it helps!
# Trend Quality Indicator
def trend_quality(series: pd.Series, fast_period: int = 10, slow_period: int = 40,
trend_period: int = 10, noise_period: int = 10, c: float = 2.0, noise_type: str = "squared") -> pd.DataFrame:

    # calculate moving averages
    ma_fast = series.ewm(span=fast_period).mean()
    ma_slow = series.ewm(span=slow_period).mean()

    # calculate crossovers
    fast_shift = ma_fast.shift(1)
    slow_shift = ma_slow.shift(1)
    up_cross = (ma_fast > ma_slow) & (fast_shift < slow_shift)
    down_cross = (ma_fast < ma_slow) & (fast_shift > slow_shift)
    crosses = series[up_cross | down_cross].index.to_list()
    crosses.append(None)  # add None to include last cycle

    # calculate price change
    pc = series.diff().fillna(0)

    # calculate cumulative price change
    cpc = pd.Series(0., index=series.index)
    last_idx = None
    for idx in crosses:
        cpc.loc[last_idx:idx] = pc.loc[last_idx:idx].cumsum()
        last_idx = idx

    # calculate trend
    trend = cpc.ewm(span=trend_period).mean()

    # calculate noise
    if noise_type == "linear":
        abs_diff = (cpc - trend).abs()
        noise = abs_diff.ewm(span=noise_period).mean()
    elif noise_type == "squared":
        square_diff = (cpc - trend) ** 2
        noise = np.sqrt(square_diff.ewm(span=noise_period).mean())
    else:
        raise ValueError("Noise type invalid.")

    # calculate q indicator
    q_indicator = trend / noise * c

    # calculate b indicator
    b_indicator = trend.abs() / (trend.abs() + noise) * 100

    # return indicators
    return pd.DataFrame({
        "Price Change": pc,
        "Cumulative Price Change": cpc,
        "Trend": trend,
        "Noise": noise,
        "Q_Indicator": q_indicator,
        "B_Indicator": b_indicator
    }, index=series.index)

Thank you very much @chris-official for sharing this.  I did an initial test of your code which you can find at https://github.com/kbs-code/trend_quality_python/blob/master/user_submission.ipynb.
I only spent a few hours putting that together but so far, the results don't seem production-ready.  I'll take a deeper look into your code when I have time as I only changed your parameters to match mladen's default ones.

Good news @kbs-code, I again looked into my code as well as the test notebook you provided. While my first implementation is based on the original paper, I implemented the indicator a second time but this time focused on recreating the MetaTrader code of your file "trend_quality_q1_directional_indicator.mq4". Now, the results look much more similar to what is shown in your reference image.
Here are some of the most noticeable differences of the new implementation:

Now, the "absolute price change" is used instead of the "price change". This is unique to mladen's implementation and is not mentioned in the original paper by Sepiashvili.
The Trend is now also reset to zero at trend changes and not just an EMA of the CPC. This behavior is also not mentioned in the original paper.
The calculation for trend changes and CPC are different now.
Now, a SMA is used for smoothing of the noise instead of an EMA. Paper does not define a specific MA here.

Some important considerations:

When testing the new indicator the same way you did with my first one you may still notice a deviation from the original values. This is due to your test dataset only having 92 values while the noise period is 250 and therefore still in it's warm-up period. You need at least 250 previous values to accurately compare the magnitude of the Q Indicator. However, the shape should already be pretty similar.
The new implementation has the advantage of being more readable since it produces clear, sharp drop-offs at the trend changes.
However, note that my first implementation produces much less lag due to the following two factors:

Using the EMA for noise smoothing instead of the SMA.
Using the absolute price change instead of the price change. This results in the Q Indicator still rising until the trend sign flips even if the trend already reversed earlier. My first implementation will already move downwards as soon as the price does and therefore the peaks of my first Q Indicator usually produce better signals at an earlier point but are not always characterized by a sharp drop-off and thus harder to spot.

Feel free to try out the new code and make changes as you like. Maybe you can come up with your own improved indicator version! I hope this helps!
def trend_quality(price: pd.Series, fast_period: int = 7, slow_period: int = 15,
                  trend_period: int = 4, noise_period: int = 250, c: float = 2.0, noise_type: str = "squared") -> pd.DataFrame:

    # calculate moving averages
    emaf = price.ewm(span=fast_period).mean()
    emas = price.ewm(span=slow_period).mean()

    # calcualte trend sign
    macd = emaf - emas
    sign = pd.Series(np.nan, index=price.index)
    sign[macd > 0] = 1
    sign[macd < 0] = -1

    # calculate price change
    change = price.diff().abs()

    # calculate cumulative price change and trend
    cpc = pd.Series(np.nan, index=price.index)
    trend = pd.Series(np.nan, index=price.index)
    for i in range(1, len(sign)):
        if sign[i] != sign[i-1]:
            cpc[i] = 0
            trend[i] = 0
        else:
            cpc[i] = sign[i] * change[i] + cpc[i-1]
            trend[i] = cpc[i] * (1 / trend_period) + trend[i-1] * (1 - (1 / trend_period))

    # calculate noise
    if noise_type == "linear":
        dt = (cpc - trend).abs()
        avgDt = dt.rolling(noise_period, min_periods=1).mean()
        noise = c * avgDt
    elif noise_type == "squared":
        dt = (cpc - trend) ** 2
        avgDt = dt.rolling(noise_period, min_periods=1).mean()
        noise = c * np.sqrt(avgDt)
    else:
        raise ValueError("Noise type invalid.")

    # calculate q indicator
    trendQ = (trend / noise)

    # return indicators
    return pd.DataFrame({
        "Price Change": change,
        "Cumulative Price Change": cpc,
        "Trend": trend,
        "Noise": noise,
        "Q_Indicator": trendQ,
    }, index=price.index).fillna(0)

As you can see the peaks of the new indicator are much more pronounced but several bars later than the old indicator.

@chris-official thanks a lot for this.  I wish there were some things explained more clearly in the original paper.  In the paper there is a diagram which shows the Q indicator resetting abruptly if the trend reverses, and I think the macd changing signs triggers this.  Although macd is not explicitly mentioned, from what I understand the calculation is still the same.
I think a long or short Q indicator is useful and illustrated in the original paper.  The only thing he says about it is "Here, the reversal indicator is built in".  I assume he coded his own directional indicator but didn't reveal all the code / math.  And because he didn't reveal everything, mladen sought to add more code to match the illustration.  I could be wrong.
I also don't know why I see a 250 noise period as default but for MetaTrader the default value is often 250.  I'm not sure if mladen started that or someone else.
I think the question now is which version of the TQ should make it into pandas.  Other traders who use MetaTrader will be more familiar with mladens versions, but some traders may prefer the original authors indicator.  Some versions reset slowly and others quickly.  Some use different moving averages for smoothing.
Thanks for all the pointers and writing the code.

@twopirllc If I wanted to contribute the tq indicator to pandas_ta do I have to follow any guidelines?  I have been using your macd as a template of sorts.  Can I leave out talib mode and using pandas_ta macd instead of calculating it from scratch in the tq function?

@kbs-code

do I have to follow any guidelines?

You must checkout the development branch and style your code similarly to other indicators (or as close as you can as possible).

Can I leave out talib mode?

I will most likely put it anyhow if you leave it out and on by default. It's just an option so not a big deal.

some traders may prefer the original author's indicator.

Based on xp. This is also true.
KJ


Support documentation and source viewing for arbitrary URLs

<EMAIL_ADDRESS>This should simply reuse src/Docs.tsx

We might want to proxy these URLs through the worker so we don't have CORS issues.

Can be closed because doc.deno.land


Implement the dict-based optional parameters and the other first wave ops

Fix #24 , #13, #12, #10, #9, #8, #14, #16 and #17.
Preview:

test
docs

The changes include:

Implemented the dict-based optional parameters introduced by https://github.com/webmachinelearning/webnn/pull/112
Implemented new 18 first wave ops including gru, gruCell, gemm, sigmoid, tanh, slice, sub, concat, squeeze, div, min, max, exp, sqrt, batchNormalization, clamp, split and leakyRelu.
Implemented the test cases for these new ops.
Fixed the issues of multiple outputs support and memory leaks of duplicated constants.

@wchao1115, also please take a look. Thanks!

Thanks for review and approval!


[CP] Avoid crash when backdrop filter is null for PlatformViews

Issue Link
https://github.com/flutter/flutter/issues/127095
Commit Hash
01bb6af
Target
stable
PR Link
https://github.com/flutter/engine/pull/43982
Changelog Description
Avoid crash when a null backdrop filter is applied to PlatformView
Impacted Users
Flutter devs, end-customers
Impact Description
App crash
Workaround
No
Risk
low
Test Coverage
yes
Validation Steps
Use the sample code in https://github.com/flutter/flutter/issues/127095#issuecomment-1592872318 to reproduce the crash and validate the fix.

@cyanglaz

Marking "cp: approved" on behalf of @cyanglaz as the PR was approved.

Any updates here?

@CaseyHillers Can this be closed? It looks like it landed on the branch.

3.10.7 doesn't exist on stable channel. Is this fix included in 3.13 ?

Shipped in 3.10.7. To get it, you'll need to run git fetch --tags && checkout 3.10.7 in your current flutter checkout.

3.10.7 doesn't exist on stable channel. Is this fix included in 3.13 ?

@CaseyHillers we have the same question, we really need this fix. Thanks in advance!

@CaseyHillers we have the same question, we really need this fix. Thanks in advance!

Yes, the fix is also on 3.13.

I still have the same crash after upgrading to 3.13 😢
In my case, when I try to open a video dialog (openDialog with Chewie), then the crash happens...
    thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)

 frame #0: 0x0000000105bb932c Flutter`flutter::IOSExternalViewEmbedder::CompositeEmbeddedView(long long) + 1828

Flutter`flutter::IOSExternalViewEmbedder::CompositeEmbeddedView:

-> 0x105bb932c <+1828>: ldr x8, [x0]

 0x105bb9330 <+1832>: ldr x8, [x8, #0x30]

 0x105bb9334 <+1836>: blr x8

 0x105bb9338 <+1840>: cbz x0, 0x105bb951c ; <+2324>

Target 0: (Runner) stopped.

@sang-makebetter This is likely a different issue. Could you open a new issue tickets with details? Thank you!


[ENH] moving towards multi-platform testing

When complete this PR will address:

#407
#417
(possibly) #322

Pull Request Test Coverage Report for Build 2258

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.277%

Totals

Change from base Build 2236:
0.0%

Covered Lines:
1747

Relevant Lines:
1761

💛  - Coveralls

pysal/libpysal#247
pysal/libpysal#248
pysal/libpysal#250
pysal/libpysal#251

Codecov Report

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

@@            Coverage Diff            @@
##             master     #418   +/-   ##
=========================================
  Coverage          ?   98.08%
=========================================
  Files             ?        6
  Lines             ?     1771
  Branches          ?      220
=========================================
  Hits              ?     1737
  Misses            ?       14
  Partials          ?       20

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 9a3cb04...a262eef. Read the comment docs.

@sjsrey All builds are passing with the new pytest/codecov scheme and environment-style testing for TravisCI (based on those in libpysal), which appveyor uses to piggyback tests. However, I think appveyor isn't being triggered because it isn't in master yet. If you could give this PR a once over, I will merge it in to see if appveyor kicks in and debug from there (or revert).
Note sure if this is related, but when I try to authorize appveyor for spaghetti (first image), I am getting a message that the action can only be an organization owner (second image).

I authorized appveyor on pysal/spaghetti so this should trigger a build.

The example dataset is not installed.

The example dataset is not installed.

Thanks! This is being addressed in #419
Also, I am trying to get the appveyor badge for spaghetti, but it seems like I don't have the permissions to even view the pysal/spaghetti stuff for appveyor, only the jGaboardi/spaghetti stuff.

Both that badge (and now I see the one for libpysal is too) are for your repos:
--> https://ci.appveyor.com/project/SergeRey/spaghetti
--> https://ci.appveyor.com/project/SergeRey/libpysal/branch/master
We should be able to get actual project repo badges, like scipy has:
--> https://ci.appveyor.com/project/scipy/scipy

the pysal org account is under my SergeRey appveyor account.
my forks are under my sjsrey appveyor account.

WIll have to investigate migration: https://www.appveyor.com/docs/team-setup/#setting-up-appveyor-account-for-github-organization

+1
Sent with GitHawk

For the failing import, maybe move the load_example inside the setup for the test:
from libpysal import examples
examples.load_example("newHaven")
pth = examples.get_path("whateverfileyouwant")

For the failing import, maybe move the load_example inside the setup for the test:

Excellet idea! And awesome on the accounts/badge.

fix for #419

For the failing import, maybe move the load_example inside the setup for the test:
from libpysal import examples
examples.load_example("newHaven")
pth = examples.get_path("whateverfileyouwant")


Use NVML to query device status

Hello,
filter_free currently looks at nvidia-smi output to determine busy devices. For driver version 352.39 (and probably some other recent releases), nvidia-smi no longer refers to running processes as "Compute processes" but just "Processes". As a result, the current version of filter_free cannot find any matches in the output.
I modified the function to use the more robust NVIDIA Management Library. In versions 8 and later of CUDA Toolkit, the library is already included. For those who do not have the library and prefer not to install it, I kept the nvidia-smi solution and fixed it to work with newer drivers without breaking backward compatibility.

Thanks, @yolcu. Sorry I wasn't paying attention for a good while, this looks really awesome.

Would be great to have this in the next release. @yolcu can you update this and squash it? Otherwise I will do so after #65 lands.

This will need to be rebased onto master to get CI to pass :)

Updated and squashed. It should be good now.

Perfect. Thank you.


Enforce RFC 7540 rules about stream weights

Fixes #50.

Test updated; changeling entry added.

Changeling entry? How very Strossian of you.

Cool, this looks good. We'll see if the tests pass, of course. ;)


Refresh of writing rust policies.

In the third, 03-define-policy-settings, I am having trouble with the worked Rust examples. I need a little help. Thanks.
When complete and merged will close #294.

Hi all, it's still draft, I want another read through, but I've marked it as ready for review, to get some eyes on it. Thanks. Especially @jvanz for patiently guiding me through the rusty bits.


Get sequences 8b5caf3398 and b20a261fdf

How did you obtain the two data sequences 8b5caf3398 and b20a261fdf? Currently, in ScanNet v2, I only see names similar to scene0000_00 and haven't found sequences in the format of 8b5caf3398 or b20a261fdf. Could you please tell me where I might have overlooked or made a mistake? Thankes!

I accidentally confused ScanNet-v2 with ScanNet++, embarrassing.


Update the virtual kubelet documentation

Description
This PR updates the documentation concerning the operations performed by the virtual kubelet, including the node lifecycle management, the pod offloading process and the resource reflection.

Closing, as superseded by the new documentation.


Integrate documentation site with Octopus Deploy

Background
We need to dogfood our work. Adding our own site to Octopus is the best way to do this.
Action requested
Make sure our site is deployable to Octopus via integration with GH actions and a helm chart.
Acceptance Criteria

create a helm chart for the doc site
add a deployment flow for the doc site in GH actions
Make sure doc site is deployable via Octopus, as well, and all proper observability is in place.

Additional context
Add any other context or screenshots about the feature request here.

Architecturally obsolete. Closing as "Won't Do".


fix: flaky integration tests

wrong index: https://github.com/WalletConnect/notify-server/actions/runs/5893092736/job/15984691236
wrong index: https://github.com/WalletConnect/notify-server/actions/runs/5893092736/job/15984183285
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/5912008613/job/16035023915
wrong index: https://github.com/WalletConnect/notify-server/actions/runs/5912008617/job/16035091718
wrong index: https://github.com/WalletConnect/notify-server/actions/runs/5918227269/job/16047374685#step:3:1
wrong index: https://github.com/WalletConnect/notify-server/actions/runs/5932292040/job/16085931625
wrong tag: https://github.com/WalletConnect/notify-server/actions/runs/5940223449/job/16108862783
wrong tag: https://github.com/WalletConnect/notify-server/actions/runs/5941506682/job/16112951153
wrong tag: https://github.com/WalletConnect/notify-server/actions/runs/5941506682/job/16113352647
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/6000432007/job/16272879288
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/6000789840/job/16274131538
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/6007287981/job/16293347795
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/6007287981/job/16293600378
ws disconnected: https://github.com/WalletConnect/notify-server/actions/runs/6013679125/job/16312090218
wrong tag: https://github.com/WalletConnect/notify-server/actions/runs/6016685858/job/16321356540
wrong tag: https://github.com/WalletConnect/notify-server/actions/runs/6096071122/job/16541222041
4011 not 4001: https://github.com/WalletConnect/notify-server/actions/runs/6198593543/job/16829667602
4011 not 4001: https://github.com/WalletConnect/notify-server/actions/runs/6200036241/job/16834560749

I think these are resolved now somehow


Deployments can get stuck if cron job is running

See Preventing stuck deployments caused by long-running cron jobs on community.platform.sh for background.
 While the script I wrote solves my problem and presumably solves it for others, it's not very elegant, makes for a lot of duplicated effort and is not easy to discover for other users.
A better solution could be to build this into the Platform.sh CLI, for example in the form of an environment:deploy command.
An even better solution could be to entirely reject deployments whenever a cron job is running.

For an example support ticket, see Stuck deployment.

Hi, we will be soon releasing the support for Crons Activities, as well as Cancellable Crons.
As a user, you will have way more visibility on running crons and their status (currently, only in the log file), and you will be able to cancel a running cron. Stay tuned.

#910


database.rdsinstance: Add DeleteAutomatedBackups

Description of your changes
Add flag DeleteAutomatedBackups which is an option for DeleteDBInstance. This might be required for users of AWS Backup.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review. (I have an issue with goanalysis_metalinter but CI will run that)

How has this code been tested

Added unit test
Verified deletion of instance with AWS Backup, it not works when setting deletedAutomatedBackups: false (thanks @niralmehtasb1)

@petteja discovered an issue and will follow up with a fix


Add .reload() function to GetStorage

Hi! I'm trying to use GetStorage with AndroidAlarmManager. As you say, I can use GetStorage on any situation you currently use sharedPreferences. However I don't know how to:

Import it on Application.java, with SharedPreferences, I can use import io.flutter.plugins.sharedpreferences.SharedPreferencesPlugin; but I don't know how to do that with GetStorage.
When AndroidAlarmManager calls callback, GetStorage.read() returns non updated values. SharedPreferences introduces ".reload()", but how can I reload GetStorage?

Thanks U!

any update?


bind to correct port when doing broadcast/multicast

to allow better re-use of ports. Currently in order to broadcast/multicast you have to bind to a port… - if not specified we were binding the local port to the remote port number. We should be binding locally to a random port if not otherwise specified.

Coverage remained the same at 87.755% when pulling a628e55c41fa122d1b6f1c5d51d0608e44c9732a on UDP-better-port-reuse into 0d08dc410e38922c6fe110ace2f7ded5f4cb4dff on master.


Handler gets lost

I'm using the latest MediatR with Autofac and noticed some strange behaviours,
after a couple of hours some of my Handlers that worked before can not be found anymore.
Until i restart the IIS Process i'm getting the Invalid Operation Exception from the RequestHandler Class:
InvalidOperationException("Handler was not found for request of type " + message.GetType() + ".\r\nContainer or service locator not configured properly or handlers not registered with your container.", innerException);
I'm using the Autofac Wireup from the Examples
            //builder.RegisterSource( new ContravariantRegistrationSource() );  // Does not work for me
            builder.RegisterType<MediatrR>().As<IMediatR>().InstancePerLifetimeScope();

            builder.Register<SingleInstanceFactory>(ctx =>
            {
                var c = ctx.Resolve<IComponentContext>();

                return t =>
                {
                    object o;
                    return c.TryResolve(t, out o) ? o : null;
                };

            }).InstancePerLifetimeScope();

            builder.Register<MultiInstanceFactory>(ctx =>
            {
                var c = ctx.Resolve<IComponentContext>();
                return t => (IEnumerable<object>) c.Resolve(typeof(IEnumerable<>).MakeGenericType(t));

            }).InstancePerLifetimeScope();

I'm not sure if this belongs to the fixed Handler Caching and the usage of:
LazyInitializer.EnsureInitialized(..)
But there is no other error recorded in my logs. Any Ideas?

Ok, this seems to be an other problem.

ha ok WHEW