Fixing projects so that they load in Visual Studio

I've fixed a few of the projects so that they load in Visual Studio as you'd expect without having to run any of the other setup scripts by using repository internal relative paths which I think makes sense.

:+1: this was always one of the things that contributed to this code base being unapproachable.

@citizenmatt do I need to do anything to get this merged?

Nope! I haven't had a chance to have a proper look at this, but on the face of it, it looks great, and I'd like to get it merged pretty quickly. Also, I want to get a little more familiar with the current codebase and tooling before getting stuck in with changes. So, give me a couple of days. And thanks for the PR!


GEODE-8769: help ensure stats are settled before testing

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ X] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ X] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ X] Is your initial contribution a single, squashed commit?

[ X] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>This helps to mitigate the issue but does not completely solve it. The problem happens when multiple threads are opening multiple Redis connections, and the statistics are reset. Depending on the timing of the connection opening and closing, the operations can interleave and result in an inconsistent state for the statistics. The stats will be "eventually correct" unless their values are explicitly reset at an unlucky time.

I agree that this fix is probably not going to address the issue. It's more like the async nature of netty that's causing this. Since we do jedis.close() in the after call and then reset stats in the before the client count decrement 'event' might only happen after the stats are decremented. As it stands I think just awaiting for the client count to reach zero at the start of the before block should fix this issue.
@dschneider-pivotal I'd be hesitant to remove the reset method. Because of the async nature of some of these stats it would still be an issue if you grabbed the 'current' value of a stat at the start of the test. (Perhaps it's only client connections that are affected by this).

@jdeppe-pivotal  for async ops you can always have the test wait to confirm the stat is updated right after close returns. This tells you the async close operation has completed. I don't see how the reset helps (it with async closes is probably the cause of the stat going negative). It would be better to test the product without adding product code if possible. If you do need to keep it then make sure and mark it for testing only. Also check that it actually resets all the stats (I think at least one was missing).

re-submitted under https://github.com/apache/geode/pull/5849

Closing in favor of: https://github.com/apache/geode/pull/5849


Create Suraj.json

Description
For my Portfolio
Link to Website
https://clay990.github.io/Portfolio/

I have read the CLA Document and I hereby sign the CLA


differnt sound

to add different sound for the flute

would want to work on this.. by adding the src for the audio I think the problem could be resolved


Show metric color in value tooltip

When hovering over values in a graph with many metrics, it's sometimes hard to see for which metric the tooltip is shown. While the circle changes its color already, it's sometimes hidden behind the mouse pointer for example.
Suggestion: show the color of the metric again inside of the tooltip. Possibilities: a) use the color as background b) add a small square with the color and put it before/after the value time/name c) something else.

Actually, it would be cool to show the legend for that time series in the tooltip as well. That would make some graphs a lot clearer.

:+1: especially because the legend sometimes only contains information which is not actually from Prometheus labels, but which indiciates which expression a time series is from. If you e.g. add two of the same expressions, but from different Prometheus servers, you might get two identical sets of time series that you can't differentiate via the hover detail currently (but via different legend format strings).

So what about option b) plus a title. Maybe ▢ <server name>: <label string>?


Branching by type in a generic way

In the initial Treecle port, I removed all the functionality that supports type => something object literals and instead it only supports functions for these things. I instead added code to Vastly to transform object literals into functions.
But it just dawned on me: this would be very useful in Treecle too, it's just that we can't depend on node.type returning a useful type. But we could have a config.getType() method, which by default returns the object [[ Class ]], but can be overridden (e.g. Vastly would do config.getType = n => n.type).
Any objections @adamjanicki2 ?

What does [[ Class ]] mean?

Warning: infodumping ahead 😅
When you do Object.prototype.toString.call(obj), you get a string like "[object Foo]". The Foo in there is the object’s internal type, which used to be called [[ Class ]]. You may still see "[[ Class ]] being used since AFAIK there is no better term to describe the concept).
Reading [[ Class ]] is generally more robust than typeof or instanceof, since:

typeof does not give you useful info when you’re dealing with different types of objects (eveyrthing is just "object") and is thrown by primitive wrapper objects, i.e. typeof new String("foo") is "object", not "string"
Because globals are tied to their originating globalThis, the same constructor objects from different contexts will not be equal. E.g. if you grab a reference to an array from an <iframe> and try to do foo instanceof Array in the parent frame, it will be false, because it's an instance of a different Array class. That's why we got Array.isArray() eventually, because simply checking if something is an array was incredibly painful if you want to do it in a robust way (and while you can cut corners on robustness when working on an app you control, since you know what code you're dealing with, you can't when developing a library that needs to work in environments you don't control.

In the past only native objects had the privilege of having a [[ Class ]] that was not "Object", but now authors can use Symbol.toStringTag to override the default.
[[ ]] denotes an internal slot, i.e. a property or method that is implemented by the JS runtime but cannot be accessed or modified directly. See https://medium.com/jspoint/what-are-internal-slots-and-internal-methods-in-javascript-f2f0f6b38de

Following up on this, how do you get the [[ Class ]] for an object?

Like this: https://github.com/LeaVerou/bliss/blob/v2/src/type.js

Like this: https://github.com/mavoweb/treecle/blob/main/src/util.js#L19-L33

Accidentally closed this


Implementing a control to display animated WebP files

I'd like to implement a control, a codec, or anything that will let me render animated WebP files within a C# Windows App SDK application. The solution needs to be performant; on par with the animated GIF rendering natively available via the Image control.
I've been scouring all the resources I can find for some kind of guidance on this. I've come across WIC, Win2D, the Windows.Graphics.Imaging namepsace, several outdated libraries (such as the Lumia Imaging SDK), and this blog post. I don't really have experience with Windows graphics systems, so all of the new knowledge is a little overwhelming, and I'm having a hard time trying to determine which avenue would be the most modern and best starting point. I'm honestly not even sure if this is the right place to ask, but I saw an issue that was closed about adding animated WebP support to the Image control that made me think it might be.
Would someone please help point me in the right direction on this, or offer any advice to help focus my effort? Thanks!

You can use the WebView2 control, which supports animated Webp files

I can give that a shot, but I don't expect that a WebView2 will support virtualization? I'd like to support displaying a large number of files in a staggered layout.

Animated WebP still not supported.

I just tested with WIC (IWICBitmapDecoder) and it works on my Windows 10 OS (22H2)
although WebP is not in Native decoders

I uploaded a test  : WinUI3_WIC_WebP
for a control using a SwapChainPanel (Direct2D) and MMIO functions to read WebP attributes
I tested with WebP files I found on Google or others I created from GIF/MPEG

I have the same issue and it would be great if BitmapImage supports animated webp images, not only gif

I have the same issue and it would be great if BitmapImage supports animated webp images, not only gif

Actually I just found about Magick.NET and decided to convert webp to gif in my application which worked for me.

I have the same issue and it would be great if BitmapImage supports animated webp images, not only gif

Did you test the "WebPControl" control ?

I have the same issue and it would be great if BitmapImage supports animated webp images, not only gif

Did you test the "WebPControl" control ?

I just tried it but the image in WebPControl doesn't seem to enlarge when I increase the height or width of the control. Is it possible to resize the image when the control's size changes?

I just tried it but the image in WebPControl doesn't seem to enlarge when I increase the height or width of the control. Is it possible to resize the image when the control's size changes?

Yes, I have voluntarily put it inside a ScrollViewer to get Scrollbars when the image is too big
You can remove the ScrollViewer and for example use
HorizontalAlignment="Stretch" VerticalAlignment="Stretch"
for the control in XAML
And I have fixed its size and centered it to avoid stretching, at lines :
   //this.Width = m_nCanvasWidth;
   //this.Height = m_nCanvasHeight;

and
 //destRect.left = (nWidth - nFrameWidth) / 2.0f;
 //destRect.top = (nHeight - nFrameHeight) / 2.0f;
 //destRect.right = destRect.left + (float)nFrameWidth;
 //destRect.bottom = destRect.top + (float)nFrameHeight;

then it will be stretched, but it is not great for the resolution :

@castorix I really appreciate your help and your suggested change works without any problem.
However, meanwhile, I also tried using WebView2 as you have mentioned before and I think that control is potentially more suitable for my project.
So I will try using it first and see if it works out.
Thanks for your help.


Flakey e2e: TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled

Saw this test fail on e2e-multiple-runs on #811:
https://github.com/kcp-dev/kcp/runs/5764258751?check_suite_focus=true#step:5:52
fixture.go:116: Starting kcp servers...
    kcp.go:188: running: /home/runner/work/kcp/kcp/bin/kcp start --root-directory /tmp/e2e/TestWorkspaceController/2409225219/data/kcp/shared --secure-port=45015 --embedded-etcd-client-port=36269 --embedded-etcd-peer-port=4[49](https://github.com/kcp-dev/kcp/runs/5764258751?check_suite_focus=true#step:5:49)67 --embedded-etcd-wal-size-bytes=[50](https://github.com/kcp-dev/kcp/runs/5764258751?check_suite_focus=true#step:5:50)00 --kubeconfig-path=admin.kubeconfig --auto-publish-apis --discovery-poll-interval=5s --token-auth-file /tmp/e2e/TestWorkspaceController/2409225219/data/auth-tokens.csv --run-virtual-workspaces=true
    kcp.go:424: success contacting https://<IP_ADDRESS>:45015/livez
    kcp.go:424: success contacting https://<IP_ADDRESS>:45015/readyz
    fixture.go:143: Started kcp servers after 7.563398407s
    --- FAIL: TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled (40.77s)
        util.go:117: Saving test artifacts for test "TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled" under "/tmp/e2e/TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled/508349488".
        fixture.go:116: Starting kcp servers...
        kcp.go:188: running: /home/runner/work/kcp/kcp/bin/kcp start --root-directory /tmp/e2e/TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled/508349488/data/kcp/main --secure-port=38141 --embedded-etcd-client-port=43439 --embedded-etcd-peer-port=32823 --embedded-etcd-wal-size-bytes=5000 --kubeconfig-path=admin.kubeconfig
        kcp.go:424: success contacting https://<IP_ADDRESS>[55](https://github.com/kcp-dev/kcp/runs/5764258751?check_suite_focus=true#step:5:55):38141/readyz
        kcp.go:424: success contacting https://<IP_ADDRESS>:38141/livez
        fixture.go:143: Started kcp servers after 10.338909491s
        controller_test.go:81: Delete all pre-configured shards, we have to control the creation of the workspace shards in this test
        controller_test.go:85: Create a workspace without shards
        controller_test.go:92: Expect workspace to be unschedulable
        controller_test.go:94:
            	Error Trace:	controller_test.go:94
            	            				controller_test.go:218
            	Error:      	Received unexpected error:
            	            	expected state not found: context deadline exceeded, 3 errors encountered while processing 3 events: [expected an unschedulable workspace, got status.conditions: v1alpha1.Conditions(nil), expected an unschedulable workspace, got status.conditions: v1alpha1.Conditions{v1alpha1.Condition{Type:"WorkspaceScheduled", Status:"True", Severity:"", LastTransitionTime:time.Date(2022, time.March, 31, 2, 16, 14, 0, time.Local), Reason:"", Message:""}, v1alpha1.Condition{Type:"WorkspaceShardValid", Status:"True", Severity:"", LastTransitionTime:time.Date(2022, time.March, 31, 2, 16, 14, 0, time.Local), Reason:"", Message:""}}]
            	Test:       	TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled
            	Messages:   	did not see workspace marked unschedulable
        util.go:211: saving artifact to /tmp/e2e/TestWorkspaceController/add_a_shard_after_a_workspace_is_unschedulable,_expect_it_to_be_scheduled/508349488/artifacts/kcp/main/root_e2e-org-292cx/clusterworkspaces.tenancy.kcp.dev_steve.yaml
        kcp.go:181: cleanup: ending kcp server
    kcp.go:181: cleanup: ending kcp server

Didn't realize this existed when we filed #2603. Will keep the latter as it has more details.
/close


fixed Inconsistent Sidebar Styles

Updated

Archives font to be monotype
Archives bullets are unfilled-circle

@Carlynn – please reference issue #.


Tabbar update reference

A user in react realized that the tutorial was not uptodate with the tabbar (we use index now instead of initialIndex).

@philolo1 Thanks! Just one thing, the tutorial is served from gh-pages branch, so if you want to publish stuff immediately you need to merge master into that branch as well.

Thank you @frankdiox , can you merge it into the gh-pages or should i do it?

I'll do it :)

thanks!
2016-08-15 15:04 GMT+09:00 Fran Dios<EMAIL_ADDRESS>
I'll do it :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OnsenUI/tutorial/pull/10#issuecomment-239734893, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF_F5t67y3jT8VLo6O2AjlEEwoZazCy9ks5qgAFjgaJpZM4JkCx_
.


Not working on Brave Browser anymore

Describe the bug
When the project was released, I was using it on Brave and everything was fine. The code right now is very different from before, but is not working anymore.
I know that this is a small project, but I'm just opening this issue because it was working before and now it isn't, maybe it's just a small fix that someone can do it :)
The error message that is showing in the console is:
TypeError: Cannot read property 'className' of undefined

To Reproduce
Open https://liyasthomas.github.io/postwoman/ on a up to date Brave Browser.
Expected behavior
It should work like Chrome, because for me it's still working on Chrome.
Screenshots

Desktop (please complete the following information):

OS: Ubutuntu 18.04 or Window 10
Browser Brave
Version 0.68.131 Chromium: 76.0.3809.100 (Official Build) (64-bit)

Additional context
If you don't want to support Brave you can close this issue, no problem, I can still use it on Chrome.

That problem is not about the brave browser.
It's a common bug right now. It'll be fixed soon.
Thanks for the heads up!
On Wed, Aug 28, 2019, 12:17 AM Igor Guastalla<EMAIL_ADDRESS>wrote:

Describe the bug
When the project was released, I was using it on Brave
https://brave.com/ and everything was fine. The code right now is very
different from before, but is not working anymore.
I know that this is a small project, but I'm just opening this issue
because it was working before and now it isn't, maybe it's just a small fix
that someone can do it :)
The error message that is showing in the console is:
TypeError: Cannot read property 'className' of undefined
To Reproduce
Open https://liyasthomas.github.io/postwoman/ on a up to date Brave
Browser.
Expected behavior
It should work like Chrome, because for me it's still working on Chrome.
Screenshots
[image: image]
https://user-images.githubusercontent.com/22016005/63808543-94795380-c8f6-11e9-83ec-9c9594ff0aec.png
Desktop (please complete the following information):

OS: Ubutuntu 18.04 or Window 10
Browser Brave
Version 0.68.131 Chromium: 76.0.3809.100 (Official Build) (64-bit)

Additional context
If you don't want to support Brave you can close this issue, no problem, I
can still use it on Chrome.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/liyasthomas/postwoman/issues/71?email_source=notifications&email_token=AEEUVLKVWLXG2JOTP6IYPK3QGWKV3A5CNFSM4IQNUO6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HHYLV2Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEEUVLMHWK4E44FYYZUMPLTQGWKV3ANCNFSM4IQNUO6A
.

I'll try to create the required PR tomorrow.

@guastallaigor have you tried to clear the cache perhaps history data too and try again?

I cannot reproduce this issue.

OS: Windows 10 (1709)
Browser: Brave
Version 0.68.131 Chromium: 76.0.3809.100 (Official Build) (64-bit)

Have you installed the PWA or got any cached copies?
Try going to Developer Tools -> (might need to click the double right chevron ») Application tab -> Service Workers -> Unregister and reload the page.


Replace pexpect with paramiko for Unifi direct

paramiko directly implements ssh allowing hass to not depend on an
external ssh binary. Can thus run in a comtainer that has no openssh
installed.
Signed-off-by: Jan Fajerski<EMAIL_ADDRESS>Description:
Logging into my unifi AP failed with pexpect due to an unexpected shell (and failing to set a prompt). Paramiko works with out this kludgy prompt change and does not depend on an external ssh binary.
This probably doesn't yet fulfill all requirements below, will look into tests...advice appreciated.
Checklist:

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

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly (example).
[ ] New dependencies have been added to requirements in the manifest (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

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

Hi @jan--f,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @jan--f,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @jan--f,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @jan--f,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @jan--f,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @jan--f - the asus wrt device tracker uses asyncssh (via its aioasuswrt' library) rather than paramiko`. It doesn't block or use threads like paramiko, so that might be a better direction to head in?

CLA not signed. Open a new PR when ready to sign CLA and address comments.

@balloob well your cla signing page was broken last week....for me at least. Guess I'll maintain my own changes then...


Process 'command 'D:\sdk\android-ndk-r16b/ndk-build.cmd'' finished with non-zero exit value 2

When I modify ndk.dir path has a error message.
ndk version: r16b
Process 'command 'D:\sdk\android-ndk-r16b/ndk-build.cmd'' finished with non-zero exit value 2

add D:\sdk\android-ndk-r16b in path var of system environment, open Android Studio, in Terminal tab,
cd EuhatRtsp\euhatrtsplib\src\main
ndk-build
and let me see what the output says.

witch ndk version shoud to be used?

ndk r12 or above, all source code is available, and the ffmpeg lib can be built from any ndk youself.


ISO8601 format for scrapedAt

I'm looking to build a JSON Schema for the API. This will help others build language wrappers for the returned JSON.
I'm hitting a JSON Schema validation error on scrapedAt because it's not strict ISO8601 format. It appears scrapedAt is way to precise, with too many microseconds. My current workaround to define scrapedAt as a string in the schema, but that defeats the purpose of strict typing and will make language wrapping a little harder.
Any chance of updating the code to adhere to strict ISO8601 for where possible?

This library might be a viable solution: https://github.com/relvacode/iso8601

Forgot to close this before, this was added in this commit: https://github.com/samuel-pratt/bc-ferries-api/commit/990cf650b36398a05865243e961ecae98e551fa1


flixtor.to works?

Hi, i am tried to scrape some stuff from flixtor.to, without success.
example:
import cloudscraper
scraper = cloudscraper.create_scraper()
response = scraper.get('https://flixtor.to/watch/tv/3364154/better-call-saul')
print(response.text)

I have the same response (CF challenge), no matter what i have tried (with/without user agent, with or without Interpreters, etc...).
Is it only for me or?
Thanks!

works fine for me , waht cloudscraper version are you running?
>>> import cloudscraper
>>> cloudscraper.__version__

Hmm, strange.
I have 1.2.32 version, i think is the latest.

User took too long


Translation keys can't be specified for default validation functions

Elixir version
1.17.0
Database and Version
PostgreSQL 16.3
Ecto Versions
3.11.2
Database Adapter and Versions (postgrex, myxql, etc)
postgrex 0.18.0
Current behavior
Specifying a message with translation keys does not work with the built-in validation functions (anything that calls validate_change and accepts a message option). E.g. attempting to use this:
  changeset
  |> put_change(:mime, mime)
  |> validate_inclusion(:mime, ~W(image/jpeg image/png),  message: {"(%{value}) is invalid", value: mime})

raises an error:
** (FunctionClauseError) no function clause matching in anonymous fn/1 in Ecto.Changeset.validate_change/3
    (ecto 3.11.2) lib/ecto/changeset.ex:2430: anonymous fn({:mime, {{"%{value} is not allowed", [value: "application/pdf"]}, [validation: :inclusion, enum: ["image/jpeg", "image/png"]]}}) in Ecto.Changeset.validate_change/3
    (elixir 1.17.0) lib/enum.ex:1703: Enum."-map/2-lists^map/1-1-"/2
    (ecto 3.11.2) lib/ecto/changeset.ex:2430: Ecto.Changeset.validate_change/3

Expected behavior
Built-in validation functions should allow messages to include translation keys, similarly to add_error/4.

Good call. A PR is welcome!

I gave it a go - https://github.com/liamwhite/ecto/commit/f151cd39c7058ed4142ea48b9d4b7eec998689f6
While this is possible without any special considerations for the validate_* family of functions, it does not appear to be possible for constraints without breaking the public API - constraints would need to be changed from String.t() to String.t() | {String.t(), Keyword.t()}, and Repo.Schema would need to be updated to convert the constraint message format list.

Please do send a PR, we can review constraints later!


Add CLI to help select/run codemods

I've added a CLI tool to help move from a version of AVA to another.
It kind of looks like this:

I've created a file codemods.json which should list the versions that contain codemod-able changes. For now, there is only one entry for v0.14, which is the codemods needed to go from <0.14.0 to 0.14.0. Hopefully, we will later only have create the codemods and modify that file.
We could later have a different question for selecting upgrade codemods (0.13 --> 0.14) or transition codemods (mocha --> ava).
Pretty sure this will not when installed globally (it needs to find jscodeshift in the $PATH), would welcome some tips.

Actually, do you guys mind if I create a new repo/package for this and we integrate it? I'm thinking this could be of use for quite a few projects.

@jfmengels Can you fix and merge now and do that later? This is the only thing blocking a new AVA release. Definitely a good idea though.

Sure, will do!

Tests are failing. Maybe also update the docs about running it and what AVA version migrations currently supported.

Should we recommend to install it globally? Makes it easier to run the command.

@jfmengels Yup, use https://github.com/sindresorhus/meow to handle the input and help stuff.

Yes. This should definitely be installed globally.

meow

He's doing everything with prompts now. So no need for meow yet

@jamestalmage I know, but meow handles other things too, like --version, --help: https://github.com/sindresorhus/meow#features We should definitely have --help.

I've updated the PR. Thanks for all the tips @jamestalmage.
Have updated the readme a bit, can be built on later.
@sindresorhus Fair point for --version/--help. I'm adding it now (also fixing a bug now that I've added globby).
The way things work now, all files in the project will be listed in the command when launching jscodeshift. Will that not break a console limit?

Note: https://github.com/jamestalmage/ava-codemods/pull/9#discussion_r58757388

We should warn people the following either in the readme or the CLI. Something like:

Ensure you have a backup of your tests or commit the latest changes before running this.

All comments taken into account.

Nice!

:dancer:

Wondrous‼ ✨


Review Codespaces pricing structure and available quota

Linked to #5 .
See :thread: conversation.
From Lucy
For organisation-owned Codespaces:
See GitHub pricing calculator.
A very rough estimate showed we'd be spending around $8.50 per user per month, assuming 10 hours of use per week, 10GB disk usage and one stored Codespace.
Here's a copy of the billing report from the last three months of usage.
Alternatively, if someone used a personal account, they would get for free:

120 GitHub Codespaces core hours per month per user
15 GB GitHub Codespaces storage per month per user

We know that we can use a basic two-cpu machine for most OpenSAFELY projects. Regarding storage quota, it seems that this is based on the machine rather than the data used by the project. So using a two-cpu machine would use 32GB of storage; however this is billed per month, so deleting the Codespace after half a month would only use 16GB of storage (I think).
Conclusion
Assumptions (thread):

we'll have organisation-owned Codespaces and that it will cost us approximately $10 per user per month (thread).
we'll have around 30 active users per month and around one third of active users will be using Codespaces.
users will use Codespaces more heavily at the start of their project and less heavily near the end. We'll onboard users and projects at a consistent rate, so this usage will average out. Probably over the course of a year.

I think that's likely to be quite a high estimate, because I don't expect usage to be quite that high and it allows each user to store three Codespaces, with an average project size of 10GB, which I think is also quite generous.
Expected monthly cost is going to be around $100 per month for the OpenSAFELY GitHub organisation. So we should keep the monthly limit at $200 or even reduce it slightly, if we want to keep an eye on costs.

From Seb (see thread)

It would not be necessary to track spending to projects, at least initially. It's quite likely to be a requirement at some point in the future.
The $200 is, I believe, an arbitrary limit to force us to have conversations about value.
I suggest as part of this work we:
a) Estimate an upper limit on the costs per project
b) Agree and document that we will pay for codespaces for all users
c) Set a spending limit accordingly
d) Communicate this to ops in Slack
It would be good to have a little think about how/where we record this, so on the inevitable day in > 18 months when someone asks "should we increase this limit? Is this a lot of money" it's easy to recover the original thinking and policy.

See update https://github.com/opensafely-core/codespaces-initiative/issues/5#issuecomment-2037714770

I've documented this change in the team manual and I've communicated it to ops, so I believe this is now done.


Touchscreen, battery and so on..

Apologize for my disturbance. I just got my HP Prime and ready to coding (Hope to have one since high scrool and I'm in college now)! After surfing lots of web museums I'm so excited that you have made this repo I can learn from. However, after reading your code, there is no interface abou touchscreen and battery and so on. How can I get resources about these things? Thanks.
ps: s3c2416 related resources seems not enough.
Sent from PPHub

There is a fork of QEMU out there that supports the HP Prime G1. The most advanced version I know of is this fork, which supports the touch screen: https://github.com/opcod3/qemuPrime. From what I recall, that person reverse-engineered the touch screen's Linux driver source code to emulate the device inside QEMU because we could not find a datasheet for this chip.
As for the battery level I do not know. It's probably either measured with an ADC over a high impedance voltage divider (like the NumWorks calculator: https://www.numworks.com/resources/engineering/hardware/electrical/schematics/) or there's a battery management chip that the calculator communicates with. One of the original developers confirmed it's a voltage divider for the G2 model, but he did not remember how it was done for the G1 model: https://hpmuseum.org/forum/thread-14125-post-124857.html#pid124857
Note that nearly all my work on the HP Prime was done on the G1 model, which was discontinued a couple of years ago. Somebody else played around with the current G2 model, if that's what you have: https://www.zephray.me/post/hp_prime_g2_linux/
I haven't played around with HP Prime calculators in a long time, but feel free to ask questions. There aren't a lot of resources on these out there and there's always the odd chance I remember something useful if I'm asked something specific.

There is a fork of QEMU out there that supports the HP Prime G1. The most advanced version I know of is this fork, which supports the touch screen: https://github.com/opcod3/qemuPrime. From what I recall, that person reverse-engineered the touch screen's Linux driver source code to emulate the device inside QEMU because we could not find a datasheet for this chip.
As for the battery level I do not know. It's probably either measured with an ADC over a high impedance voltage divider (like the NumWorks calculator: https://www.numworks.com/resources/engineering/hardware/electrical/schematics/) or there's a battery management chip that the calculator communicates with. One of the original developers confirmed it's a voltage divider for the G2 model, but he did not remember how it was done for the G1 model: https://hpmuseum.org/forum/thread-14125-post-124857.html#pid124857
Note that nearly all my work on the HP Prime was done on the G1 model, which was discontinued a couple of years ago. Somebody else played around with the current G2 model, if that's what you have: https://www.zephray.me/post/hp_prime_g2_linux/
I haven't played around with HP Prime calculators in a long time, but feel free to ask questions. There aren't a lot of resources on these out there and there's always the odd chance I remember something useful if I'm asked something specific.

Thank you so much for replying to me in such detail. I'm using G1 (v2) now and planing to run FreeRTOS on it. Before that I've tried your Epsilon repo but it cannot run properly (screen blinks and reboot loop), which I'm trying to fix. Because I'm a CSharper, so I'm also planning to run my c# code on hp prime.
Anyway, your reply helps a lot! Thank you.


Appealing labels removes self-labels

Describe the bug
When a user self-labels a post, it is then moderated by the official service. If the official service and the user use the same label shortname, and the user appeals it, it effectively removes both labels - completely unlabelling the post.
This is a significant issue as the two labels appear to be different in the UI (see #3870) - and operate differently in terms of user expectation.
To Reproduce
Steps to reproduce the behavior:

Make a self-labelled post
Wait for the AI moderation to label it
Appeal the label successfully
The post is no longer labelled at all

Expected behavior
Appealing a label from the official service should not negate the label already on the content - it should have the effect of reverting to the self-label.

Seems like this part of the problem isn't actually caused by the label being removed, but by the fact that you can't actually see a self label.
In this case the situation looks like this:

You post an image and add the self-label nudity
The post gets upgraded to the porn label immediately. To the user, this Bluesky upgraded label appears to be their self label
If the user realizes that the label was upgraded, they appeal the upgrade
Bluesky moderation accepts the appeal, and downgrades it back to nudity
To the poster, the UI makes it look like all labels were removed from their post, even though they were not.

This should adjust that behavior so more tweaks can be made. https://github.com/bluesky-social/atproto/pull/2467

Ooooh - I misunderstood the pattern, yes that looks like it solves.
Would that then re-use the 'X labels has been applied...' pattern?

Needs a separate fix, but yea that's ideal. Even with this fix you still wouldn't see the label if you had the setting to "Show" so need to add the little indicator.

I linked the PR so it will close when it gets merged.

Part two is here: https://github.com/bluesky-social/social-app/pull/3874


Profile 2+ Pin Mapping is Wiped on Reboot or Does not Save

Please read these quoted sections before submitting, and remove them (including this one) when you have provided what
is asked, in order to make addressing the issue easier for everyone. Thank you!
Be sure to:

Check the documentation HERE to answer many questions on firmware capabilities and usage.
Make sure that your issue has not already been filed HERE.
Confirm your issue exists on the latest version of the firmware, found
HERE.

Expected Behavior
Create profile 2 and retaining all pin mappings upon reboot.
Current Behavior
After creating a profile 2 and making changes or even copying base profile, hitting save and rebooting into controller. ALL pin mappings on Profile 2 are unset/wiped. If you ever switch to Profile 2 in controller mode, none of the buttons/pins are mapped and you're forced to manually go into BOOTSEL and use the force webconfig flash file to get back into web config. Same behavior on additional profiles (profile 3, etc.)
Context
Firmware Version: v0.7.10-RC1
Board config name: GP2040-CE_0.7.10_Haute42COSMOX.uf2
Name of device: Haute T16
Steps to Reproduce

Navigate to pin mapping
Add Profile (2)
Copy Base Profile and/or Map Any pins manually
Hit Save
Reboot into Web Config or controller then boot up into web config
(Tested with GP2040 and PS4 input display in web config)

Screenshots & Files

Please include any relevant screenshots, photos/recordings, or files here.

Also to note, there's doesn't seem to be a way to delete profiles. Only Disable them.

This should be #1154.

This will be fixed in v0.7.10-RC2.


Merge -lib into -core

comes with https://github.com/CounterpartyXCP/Documentation/pull/190

Nice! 🎉 Let's also update the release notes + docs, with instructions for upgrading

done!


gmtp: fix the Using the 'memory' GSettings backend issue.

Motivation for this change
gmtp unable to save its settings.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Please use wrapGAppsHook instead - it automatically adds dconf, too (you don't need to specify it explicitly).

I was going to sleep and didn't notice that it already uses wrapGAppsHook, sorry. We have recently made it so wrapGAppsHook automatically adds dconf as necessary; have you seen if the latest channel works for you?

When I read the code of wrap-gapps-hook.sh it still requires dconf as a dependency in this case.

See how wrapGAppsHook is defined (I think in all-packages.nix), it should have this package propagated so it finds it later. I would have given a link but I'm from the phone.
Nikolay.

This is totally my bad, I just tested again against latest unstable channel and indeed everything works when using wrapGAppsHook so no changes are needed. Thank you for catching that and sorry for the noise.


Making input types for beta diversity more flexible

It would be nice if we could make the beta diversity input types more flexible.
For instance, some distance metrics that operate directly on proportions can perform better when working with the raw counts.  In addition, jackknifing / bootstrapping is best applied to the raw counts.

Can you describe what types of input you'd like qiime diversity beta and/or qiime diversity beta-phylogenetic to support? Right now they support an input of FeatureTable[Frequency], which is a table of raw counts (both examples you listed seem to be satisfied with the currently supported input type).
Is it the Properties(['uniform-sampling']) requirement that you'd like to see relaxed? Maybe I'm misunderstanding what you mean by "raw counts".

@jairideout you nailed it.  It would be nice if Properties(['uniforming-sampling']) could be relaxed.  Right now, if I wanted to run beta diversity metrics on a FeatureTable[Frequency] that isn't normalized, I'd need to force the type into FeatureTable[Frequency] % Properties(['uniform-sampling'])

@mortonjt, wouldn't a jacknifing/bootstrapping situation require a new method anyhow? It would be free to accept any input it wanted. As for proportional metrics, maybe there needs to be more granular methods with different specific requirements on input instead of a catch-all that passes through to skbio/scipy.
Another thought:
Would being able to override the type (flagging a warning in provenance) suite your purpose? In other-words QIIME 2 would just match on the view-type and as long as a transformer existed would permit the operation while complaining about how much it doesn't like what is happening.
This is an escape-hatch that has been on the backburner for a long time, but I think we kind of anticipated that some people may need to escape the safety of the semantic types in certain situations.

Just discussed this with @antgonza and @rob-knight. I'll work on this for this release.


Update scalafmt-core to 3.0.5

Updates org.scalameta:scalafmt-core from 3.0.0-RC3 to 3.0.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-pre-release

Superseded by #331.


SvelthreeGLTF.ts + x : dynamically generated components are not being destroyed (at all? / properly?), reactive statements bound to $svelthreeStores[sti] cause errors

e.g. in the Mesh component:
https://github.com/vatro/svelthree/blob/d98605278ec144997a1632ccc61b5c3bac8fd3f6/src/components/Mesh.svelte#L785
I've stumbled upon this when changing test-scenes (pages) in the SvelteKit test-app. The error was thrown only when switching from the gltf-to-comp-1 test-scene to any other. gltf-to-comp-1 test-scene generates svelthree-components from a loaded GLTF file.
If we e.g. change the line above to
$: currentSceneActive = $svelthreeStores[sti]?.scenes[scene.userData.index_in_scenes]?.isActive
the error is not being thrown.
Why?
It seems like that somewhere during the destruction process, the svelthreeStores store is gone / nullified?, but generated components are still alive and react to that change by triggering corresponding reactive statements. Adding ? to all $svelthreeStores[sti] related statements doesn't feel like a real fix.
TODO:

[ ] examine destruction of dynamically generated components (SvelthreeGLTF.ts + X)

I just stumbled upon this issue: https://github.com/sveltejs/svelte/issues/7488 which should be fixed in Svelte 3.49.0 and which may also solve the problem with dynamically generated components described above, see: https://github.com/sveltejs/svelte/blob/master/CHANGELOG.md#3490

Fix DOM-less components not being properly destroyed (https://github.com/sveltejs/svelte/issues/7488)

TODO: check + eventually close ...

see https://stackoverflow.com/questions/68335375/ondestroy-not-working-when-component-target-is-removed?rq=1

If you create the component by hand, you also need to destroy it by hand. You need to call $destroy on its instance before removing the Dom element


error in Annotated Ruleset docs

Describe the bug
The docs page https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/Annotated-Ruleset says:

This XML file can be named anything you like, as long as it has an xml extension and complies to the ruleset.xml format. The file can be stored anywhere, making it perfect for placing under version control with a project's source code and unit tests.

I'm not sure this is true.
When I register a path to a standard with the 'installed_paths' config, I find the following:

the ruleset XML file must be called ruleset.xml
the ruleset.xml must be inside a folder that has the name of the ruleset, and is a child folder of the path declared to installed_paths.

In other words, this appears to be the required structure:

path/declared/to/installed_paths
-- MyRuleset
--- ruleset.xml

Versions (please complete the following information)

Operating System
MacOS 13.6.8]

PHP version
8.3

PHP_CodeSniffer version
3.10.2]

Standard
custom

Install type
Composer local

Additional context
Add any other context about the problem here.
Please confirm

[x] I have searched the issue list and am not opening a duplicate issue.
[x] I have read the Contribution Guidelines and this is not a support question.
[x] I confirm that this bug is a bug in PHP_CodeSniffer and not in one of the external standards.
[x] I have verified the issue still exists in the master branch of PHP_CodeSniffer.

@joachim-n Thanks for opening this issue.
The docs are actually correct, but I do agree this can be clarified some more.
Basically it comes down to this:

For a standard (which can have its own sniffs), there are strict naming conventions, both for the ruleset as well as the sniffs and the directory layout.
Those naming conventions are like you mentioned above and those are outlined in the Coding Standards Tutorial.
For a project ruleset, there are no naming conventions other than that the file is an XML file.
The recommended way is to save the file to the project root and name it [.]phpcs.xml.dist, which will allow for it to be automatically picked up by PHPCS, but as stated in the paragraph you quoted above, the file can be called anything and placed anywhere.

Both of these are rulesets though and everything that is outlined on the Annotated Ruleset wiki page can be used in either of them.
There are, of course, best practices about what directives to add in a standard and what to add in a _project ruleset, but all directives should work just fine in either type of ruleset.
Does that clarify things ?

Thanks for the quick response!
I hadn't realised that a Standard and a ruleset are two different things.
In the docs TOC on the RHS, there's a subheading 'For coding standard creators' and the first page under that is 'Annotated Ruleset'.
I've now found the docs that explain the structure for standards in 'Coding Standard Tutorial', but that's under 'For sniff developers' heading, so I didn't look at that originally!

I hadn't realised that a Standard and a ruleset are two different things.

Well, they are and they aren't 😅. That's what makes it complicated.
For the ruleset file, there is not that much difference, other than in best practices.
The biggest difference, from a technical point of view, is really that a Standard can contain sniffs and can be "installed" and once installed, it can be referenced by name in (other/project) rulesets, including referencing individual sniffs/error codes by name (and setting properties on those etc).
What makes it complicated (terminology-wise) is that a "standard" from a functional point of view can be multiple things.

A standard can be a descriptive document, outlining what to do and what not to do, like PSR12.
This descriptive standard can then be codified into a PHPCS ruleset using only existing sniffs and published as a project/package ruleset (typically using a [.]phpcs.xml.dist file).
If a project/package standard, however, is to be shared by group of projects/packages, it can be beneficial for it to be published as an (external) standard for PHPCS which can be installed, like, for example, the Doctrine Standard. This is also how the PSR-PER standard(s) are intended to work in the future (see #29).
In that case, the standard has to have a name usable on the command-line and a ruleset.xml file.
Depending on the need for this, this external standard for PHPCS (ruleset.xml) can then be enhanced with custom sniffs, either by contributing those to existing sniff libraries, which are already in use by the (external) standard, or by adding them to the standards package itself. An example of this is the PSR12 standard which is included in PHPCS itself, or for example, the WordPress standard.

In the docs TOC on the RHS, there's a subheading 'For coding standard creators' and the first page under that is 'Annotated Ruleset'.
I've now found the docs that explain the structure for standards in 'Coding Standard Tutorial', but that's under 'For sniff developers' heading, so I didn't look at that originally!

For the "Annotated Ruleset" page, typically, the audience would be the people working on standards as described in the second and third bullet above. ("coding standard creators").
For the "Coding Standard Tutorial" page, typically, the audience would be the people working on a standard as described in the fourth bullet ("sniff developers"), though I agree that there is information on that page which also needs to be read by people working on a standard as described in the third bullet.
Does that help explain why things are under certain headers a little better ?
I'll have a think about how to make all of this clearer, but this will probably have to wait until there is a proper website.
I do find these type of questions useful though as input for that. Happy to have your help in writing things up for a website/reviewing content and structure (by the time I get to it).


init vs create

Do we "create" a new project (directory) or "init" inside a directory?

we don have "conda project install [package_spec ...]"

Create would provide an optional directory name, default to .. This would create an empty environment.yaml and tiny conda-project.yml.
craete would accept

--conda-configs
--platforms
--channels (maybe??)

Skip the lock until package are added (or the install command is run on the empty environment).
This is similar to npm and terraform.
maybe -- separator for package_spec?


Uninitialized public fields don't conform to tc39 spec

TypeScript Version:  3.5.1

Search Terms: public field uninitialized initialized tc39
Code
class X {
    foo: number;
}

outputs
"use strict";
var X = /** @class */ (function () {
    function X() {
    }
    return X;
}());

(Notice how the field foo doesn't exist`)
Expected behavior:
Output JS should match the tc39 spec (initialize the field to undefined) as specified here. See babel playground for implementation that matches tc39 spec
Actual behavior:
Output JS simply ignores the field
Playground Link: https://www.typescriptlang.org/play/?target=1#code/MYGwhgzhAEAa0G8BQ1XQGYHtMC5oDsBXAWwCMBTAJwG4kBfIA
Related Issues: none

Duplicate of #27644?

@IllusionMH, very similar, I think #27644 addresses the broader spec, whereas this issue captures only the uninitialized fields.
Feel free to dupe (this issue can serve as improved search-ability =])


Update predict + detection TL notebooks and update makefile/readme

detection_transfer_learning.ipynb

CHECKPOINT_DIR = '/home/notebook_ckpts/' is absolute path. I suggest to change it to use use relative paths to not assume any specific folder structure  and OS (/home would fail on windows)

Notebook emits warning  Deprecation: original_target_format should be of type ConcatenatedTensorFormat instead of DetectionTargetsFormat.Support for DetectionTargetsFormat will be removed in 3.1

notebooks/how_to_run_model_predict.ipynb

Do we want to add pose estimation predict example?

detection_transfer_learning.ipynb

CHECKPOINT_DIR = '/home/notebook_ckpts/' is absolute path. I suggest to change it to use use relative paths to not assume any specific folder structure  and OS (/home would fail on windows)

Notebook emits warning  Deprecation: original_target_format should be of type ConcatenatedTensorFormat instead of DetectionTargetsFormat.Support for DetectionTargetsFormat will be removed in 3.1

Done

notebooks/how_to_run_model_predict.ipynb

Do we want to add pose estimation predict example?

Done


Clarify behavior of ResponseBody.string() in its javadoc

See https://github.com/square/okhttp/issues/2461#issuecomment-205260542 and bunch of other issues.

Changed {@linkplain} to {@link}

Merged manually!


Doesnt work with AWS load balancer

Its been mentioned on closed tread but that solution was for MS server, I have same issue of 502 when trying to use on AWS server as the ajax request url to update the table is 3000 chars and I dont think it will accept.
Has anybody found a solution to this?

Ok, so seems from another closed issue from 5 years ago, it seems the problem can be solved using POST request rather than GET.
Is there a built in method for this switch?


Slow scrolling on some platforms

Wheeler scroll is frustratingly slow on some platforms.

This was solved in 2.0.0-alpha.1 by reading the step size from the wheel event.


french frotnend

there is inconsistency with how word audio is fetched. ennui, a french word on page 88, is fetched using the same word audio setup as all of the other words, however, the acceptance criteria specifies that a different audio url is used.
also, it would be preferred to use the same audio scheme for all words if that is possible

actually, the audio links within the issue are unclear. do they specify the definition or the word audio?

check page 32 for bon garcon

unsure of how it works with phrase audio. this is the url slug it is currently getting from a more complicated phrase:
https://brainy-literacy-assets.s3.amazonaws.com/audio/french/allez_vous-en_allez_vous-en_sapristi.mp3

unsure of how it works with phrase audio. this is the url slug it is currently getting from a more complicated phrase:
https://brainy-literacy-assets.s3.amazonaws.com/audio/french/allez_vous-en_allez_vous-en_sapristi.mp3

It appears that not all of the audio is on AWS yet. I'll update you tomorrow on this.

@Fattimo @joe-lazzarini So the audio files aren't exactly consistent with the list of phrases. For the example of 'allez_vous-en_allez_vous-en_sapristi' it should be split up into 'allez_vous-en' and 'sapristi' rather than keeping it as one phrase. Other phrases need to be shortened/altered a little bit to access the audio files. If it would be helpful I can make a list of the phrases and their corresponding urls and hand that off to @joe-lazzarini

Assuming @fattimo is dealing with removing punctuation, if you send me which a list of urls or phrases that match to audio I can deal with this @cmurray191

@cmurray191 Do these urls include all of the french words, or phrases only?

@cmurray191 Do these urls include all of the french words, or phrases only?

Should be everything. At least it's all of the french translation audio we have, idk if there are any french words that they just didn't record audio for.

The only phrase affected by the issue @cmurray191 mentioned is allez-vous. However, the bigger problem is that the urls are not consistent - some do not include the entire phrase, and most/all remove special characters and change french to english ones
(é -> e)

I'm thinking a solution is that I parse the urls along with the words and add a field in each document for french words along the lines of
'audio': 'https://brainy-literacy-assets.s3.amazonaws.com/audio/french/allez_vous-en_allez_vous-en_sapristi.mp3'
I'm not sure how slow this might be, but I don't know how else we would get these urls.
@Fattimo Let me know if this sounds good or if there is another way you want to do this

Based on what Chris said, then there are also a lot of words/phrases that have no audio

The only phrase affected by the issue @cmurray191 mentioned is allez-vous. However, the bigger problem is that the urls are not consistent - some do not include the entire phrase, and most/all remove special characters and change french to english ones (é -> e)
I'm thinking a solution is that I parse the urls along with the words and add a field in each document for french words along the lines of 'audio': 'https://brainy-literacy-assets.s3.amazonaws.com/audio/french/allez_vous-en_allez_vous-en_sapristi.mp3'  I'm not sure how slow this might be, but I don't know how else we would get these urls. @Fattimo Let me know if this sounds good or if there is another way you want to do this

i think including a field with the audio link is a good solution
for special characters, I created a function that latinizes all of the special characters to their english equivalent, so as long as that is logical and works out that will solve that problem

see https://github.com/Fattimo/hear-a-tale-awakening_JIB1339/blob/235bd679d20a5ab368b378dfa8a0422c03136a6b/utils/util.js @joe-lazzarini

Pr for these changes is up #187. It seems almost all french words supposedly don't have audio, so here is what I would recommend. If there is a french and an audio field for a word/phrase, use the audio field. For french words without an audio field, do whatever you are doing now which may or may not work. I am skeptical there is no audio for any of these.

Pr for these changes is up #187. It seems almost all french words supposedly don't have audio, so here is what I would recommend. If there is a french and an audio field for a word/phrase, use the audio field. For french words without an audio field, do whatever you are doing now which may or may not work. I am skeptical there is no audio for any of these. @Fattimo

sounds good, currently, if a word cue fails, it falls back on a french url slug attempt, and i can make it fall back a third time onto a database query attempt. the only concern with this is increasing database load.
for these special cases, is it ok if I check if there is a ' ' space character within the word before querying the database? i would assume that single word french characters are handled as expected with no problems which is why i am suggesting this

I'll leave that up to you. I think anything that doesn't work with these three attempts is probably very uncommon if it exists at all.

page 141 on a si tu demonstrates new functionality as detailed above @cmurray191

@joe-lazzarini either remove all periods or no periods, for page 18 mais ce phrase removed the period in the db which caused some issues
very edge case but for now don't remove any periods, my regex doesn't search for them anymore

@joe-lazzarini either remove all periods or no periods, for page 18 mais ce phrase removed the period in the db which caused some issues
very edge case but for now don't remove any periods, my regex doesn't search for them anymore

I manually added it back in

also, half of these french words are in the part of the book that dr evans had us cut out because its the short stories after @cmurray191

This period thing is probably because some of the phrases we were given have periods and most don't. It would be trivial to remove them all. Let me know if you want this.

This period thing is probably because some of the phrases we were given have periods and most don't. It would be trivial to remove them all. Let me know if you want this.

oh i see the issue then, the given phrase didn't have a period but the reader logic attaches the period to the end of the sentence because the regex no longer parses for periods. no need to do anything, but weird edge cases like this might be good to note down in hand off documentation


TAJO-1434: Fix supporting version of Hadoop

Rebased.

According to the discussion in https://issues.apache.org/jira/browse/TAJO-1434, pom.xml is updated also.

+1 LGTM
I'll commit it soon

@jinossy Thank you very much!


Travis fails because it expects environment variables?

I did a PR but the build failed on this:
Warning: SAUCE_USER_NAME and SAUCE_API_KEY must be set in the environment Use --force to continue.

Is Travis supposed to be configured with these env vars somehow?

This is because we are using a Sauce Labs account to test the polyfills that's owned by the FT.  To run the CI tests, I have to push your PR to a branch of the main repo.

OK. Is this resolveable or shall I close this issue?

There is a way to add encrypted env vars to the travis.yml file:
http://docs.travis-ci.com/user/encryption-keys/
We could do that, I suppose.  But then anyone would be able to run tests on our Sauce account, which just doesn't strike me as a good idea.

Actually Travis doesn't allow PR tests to use encrypted environmental variables.
http://docs.travis-ci.com/user/pull-requests/#Security-Restrictions-when-testing-Pull-Requests

Ah, so not resolvable then.  Thanks for pointing that out.

Well you could use something like https://github.com/twbs/savage

However it would be nice if Travis didn't always fail, so just skip Sauce Labs per the example in the documentation above for PR:s.


Revise services.ts for increased automation

Bring Pretty Diff's automation into this project.  Goals:

A revised build.

build a js/sparser_browser.js file to replace runtimes/browsertest.js
allow output options: arrays, markdown, minimal, objects, table
merge lexers into parse.js file and eliminate the lexers/all.ts file
add simulation tests to validate all features

Automatic options documentation from a centralized data object.

Generate an options.md file in the docs directory.
Add lexer options to the appropriate lexer markdown document at the bottom
Apply options from file naming convention of test units
Apply options from query string in demo tool
Apply options from file names when using testprep command

Demo tool options

Build options into demo tool HTML
Populate options from form controls
Store state in localStorage
Restore state from localStorage
Query string takes precedence over stored state, but not modified state

Add a parse command to services.ts

must be able to read from stdin
must be able to read from a file path

merge the nodetest.ts file into services.ts
Revise validation and unit tests

review diff
review test sample preparation

Review all documentation and the readme.md file
Output text to the demo tool if the option 'format` is used.

Now it appears all the functional work is complete.  I just need to add simulation tests, review documentation, and then fix/clean anything that comes up.
Once these are complete.  I will change the package name on Github and publish to the new NPM name.

Running into a problem with language auto detection when parsing multiple files async.  I will track this down tomorrow.

Just need to review documentation now.


DRAFT: notional-finance subgraph (round 1)

@this-username-is-taken @SerenityFund The main purpose of this DRAFT PR is to highlight some of the methodology and implementation challenges. Would love for you to review this section and share your thoughts with me on how to approach some of them.
Regarding QA, I'm going to do brief QA later this week and I will reach out for a lightweight QA from the messari team. The purpose of lightweight QA is to catch errors sooner and fix them before the final review. I expect certain errors to pop-up given the protocol/contract implementation style.

Thanks @jaimehgb. Will get back to Notional this week and dive into your comments.


Add MAINTAINERS and lgtm.co config

This sets up LGTM, which Sam already enabled for this repo, so we need approval to merge this branch now 😄   It make master a protected branch requiring a "LGTM", +1, 👍 , or :shipit: on PRs to merge.
Please let me know if you would like anything here changed.
I don't know what's up with GitHub rendering in the diff but it's mangling "Gastón" in my browser even though the file is properly encoded and looks correct on my machine. I'm assuming it will work, but will fix if not…
@jvican @ChrisCoffey @ktonga

It's working perfect on my phone, so seems to be a desktop-only problem. Maybe we can send them a PR, oh wait! @github is not on GitHub.
Unicode is a new concept and that "diff" feature isn't used that much anyway.

At the risk of sounding snooty... it LGTM

Hey @ches MAINTAINERS is always pulled from master so we are having a chicken and egg situation in this PR.

I wondered if that was going to happen… seems like an obvious gotcha situation that lgtm.co needs to solve. But we might just need to have @fommil toggle the integration off until we merge this then.


Use controller-runtime informers for refactored controllers (1)

How to categorize this PR?

/area scalability networking
/kind enhancement
What this PR does / why we need it:
Follow-Up of #3658. After this PR, multiple watches are created via controller-runtime and via the "native" Kubernetes informers. This not only leads to increased network I/O and memory usage but also to race conditions as the informer backing the event handler is different from the informer backing the (cached) client used to retrieve an object after an event. Hence, if not both caches are synced fast enough the reconciliation cannot be performed.
Which issue(s) this PR fixes:
Part of #2822
Special notes for your reviewer:
/assign @timebertt
/assign @timebertt
After this PR is merged I will open a similar PR for the refactored controllers in #3681.
Release note:

NONE

You didn't squash-merge... :)

Yeah, wasn't sure what you preferred, so I acted like a robot and checked whether the label was present and used the default merge mechanism accordingly 🤖 😉

Touché :)


fix: nest deps chunk cannot be loaded when optimizing deps in build mode

Description
fixes #13933
Align the load logic of optimizedDepsBuildPlugin with optimizedDepsPlugin. After loading the file from the cache, throw an exception if an error is encountered.
Additional context
As mentioned in the last context I provided in the issue. I noticed that the file field for the chunks in the .vite/deps/_metadata.json is processed with flattenId. But the chunk files generated by esbuild on the disk do not undergo the flattenId process. Is this behavior expected? If it's necessary to flatten the chunk filenames, would it be better to flatten the filenames when generating the chunk files with esbuild? However, at the moment, I'm not sure how to flatten the generated chunks.

What is the purpose of this pull request?

[X] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the PR Title Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[X] Ideally, include relevant tests that fail without this PR but pass with it.

@bluwy Sorry to bother you, let me know if you have any feedback on this bug fix.

Closing as the build-time dep-optimization is now removed (https://github.com/vitejs/vite/pull/15184).


Slots in distinct goroutine

Hi!
Just a note...
You said

Once a signal gets emitted, all the slots connected are being executed in the distinct goroutines.

And this is the Emit function
func (sig *Signal) Emit(data interface{}) {
	for i := 0; i < len(sig.cons); i++ {
		sig.cons[i].slot(&Call{Data: data})
	}
}

But I think that the function could be rewritten in this way
func (sig *Signal) Emit(data interface{}) {
	for i := 0; i < len(sig.cons); i++ {
        go func(c conn) {
		    c.slot(&Call{Data: data})
        }(sig.cons[i])
	}
}

Nice catch! I somehow managed to lose the go call in between commit.

I'm here again :smile: !
Now that there is go function maybe it would be useful to give some "wait handler", otherwise your example doesn't print anything.
I  suggest to modify the Emit function in this way:
// Emit executes all the connected slots with data given.
func (sig *Signal) Emit(data interface{}) *sync.WaitGroup {
	wg := sync.WaitGroup{}
	wg.Add(len(sig.cons))

	for i := 0; i < len(sig.cons); i++ {
		go func(i int) {
			sig.cons[i].slot(&Call{Data: data})
			wg.Done()
		}(i)
	}

	return &wg
}

So in the main program you could wait (if you want/need) the end of signal propagation
wait := foo.Done.Emit(42)

// more lines of code...

wait.Wait()

If this goes beyond your example, just put a simple time.Sleep(1 * time.Second) to wait routines

I'd do what I did in tests: emit signals from a goroutine:
go func() {
    <-time.After(10 * time.Millisecond)
    foo.Done.Emit()
}

It might be enough.

Well, I think that  <-time.After(10 * time.Millisecond) in the test doesn't anything useful, because program stops here x := <-pipe waiting for some data, so I don't understand where do you want to use.

Oh, I just realized what you were talking about. Gimme a sec, looks like I'd need to redesign the whole thing.


Improve the error message when validating the audience

This change provides additional context when validating the audiences fails

:+1:


Quickstart: Create a Game Server

Hey!
I'm following the Quickstart guide, and I ran into an issue with the "Create a GameServer" step. Here's the suggested command:
kubectl create -f https://raw.githubusercontent.com/GoogleCloudPlatform/agones/release-0.9.0/examples/simple-udp/gameserver.yaml

Then here's the error message I saw:
The GameServer "simple-udp-w9tjp" is invalid: []: Invalid value: map[string]interface {}{"kind":"GameServer", "metadata":map[string]interface {}{"generateName":"simple-udp-", "namespace":"default", "generation":1, "uid":"e9f00fd8-66f0-11e9-9a00-42010a80017d", "selfLink":"", "name":"simple-udp-w9tjp", "creationTimestamp":"2019-04-25T00:27:31Z", "finalizers":[]interface {}{"stable.agones.dev"}}, "spec":map[string]interface {}{"template":map[string]interface {}{"metadata":map[string]interface {}{"creationTimestamp":interface {}(nil)}, "spec":map[string]interface {}{"containers":[]interface {}{map[string]interface {}{"image":"gcr.io/agones-images/udp-server:0.7", "name":"simple-udp", "resources":map[string]interface {}{"limits":map[string]interface {}{"cpu":"20m", "memory":"32Mi"}, "requests":map[string]interface {}{"cpu":"20m", "memory":"32Mi"}}}}}}, "container":"simple-udp", "health":map[string]interface {}{"failureThreshold":3, "initialDelaySeconds":5, "periodSeconds":5}, "ports":[]interface {}{map[string]interface {}{"protocol":"UDP", "containerPort":7654, "name":"default", "portPolicy":"Dynamic"}}, "scheduling":"Packed"}, "status":map[string]interface {}{"address":"", "nodeName":"", "ports":interface {}(nil), "state":"Creating"}, "apiVersion":"stable.agones.dev/v1alpha1"}: validation failure list:
spec.ports.portPolicy in body should be one of [dynamic static]

Such version of Agones do you have installed?

Looks like 0.8.1 - I just installed via Google Click to Deploy an hour ago

Use the top right dropdown to switch to the 0.8.x documentation 👍

Will do, thanks!


Hit reset then switching mapping type re-renders tree

When you submit a tree with a mapping file, then hit reset, then change the mapping type, it tries to re-render the tree even though no files are there.

Fixed (79635f1b4f50cf7ac656d2f3935e087354cbeec1).


Change calculation to d3

The goal is transition the calculation of charts to d3.
This makes further development easier and also provides more stability.

:tada: This issue has been resolved in version 1.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Tinyint wrong conversion to boolean

The tinyint field is being converted to boolean when it should be tinyInteger.
Excellent tool! thank you!

Are you using mysql? In mysql if you define a column as tinyint(1) this is how it handles boolean and therefore it will be converted to boolean on the migration file.

@cacpmw thank you!
Yes. I use tinyint not to save boolean data, but numbers from 1 to 10. And tinyInteger is available in Laravel schemas.

@JoaoPortella Then if you are using tinyint(1) shouldn't you only be able to do 0-9?...

@theWebKeyGuy I did not try to use as boolean, just changed to tinyint

Joao is correct - this is a nasty bug.
Just because Laravel maps boolean to tinyint in MySQL, one should not assume that ALL tinyint fields in MySql are meant to be booleans.  It is common practice to use one as a signed status field where the range of values are below 100.

The tinyint field is being converted to boolean when it should be tinyInteger.
Excellent tool! thank you!

I'm also experiencing the same problem...
But anyway, this package is helping me a lot!
Congratulations to those involved.


Package request: remotedesktopmanager, rdm, remote desktop manager

Project description
Remote Desktop Manager (RDM) centralizes all remote connections on a single platform that is securely shared between users and across the entire team. With support for hundreds of integrated technologies — including multiple protocols and VPNs — along with built-in enterprise-grade password management tools, global and granular-level access controls, and robust mobile apps to complement desktop clients for Windows and Mac, RDM is a IT toolbox for remote access. RDM empowers IT departments to drive security, speed and productivity throughout the organization, while reducing inefficiency, cost and risk.
Metadata

homepage URL:  https://devolutions.net/remote-desktop-manager/home/downloadfree/
source URL:  deb and rpm package
license: Free for Personal & Commercial Use
platforms: linux

Add a :+1: reaction to issues you find important.

The application is written in dotnet and currently only has deb and rpm packages available. Using the deb package, I wrote the following code. When I build and run it, it throws an error and crashes. Upon examining the logs, I get the following error. When I visit the URL mentioned in the error, it suggests setting the DOTNET_SYSTEM_GLOBALIZATION_INVARIANT="1" environment variable. As you can see in the code, I have already set the environment variable, but I am still getting the same error.
"ERROR System.Globalization.CultureNotFoundException: Only the invariant culture is supported in globalization-invariant mode. See https://aka.ms/GlobalizationInvariantMode for more information. (Parameter 'name')
en-US is an invalid culture identifier."
{ lib,
  stdenv,
  fetchurl,
  dpkg,
  webkitgtk,
  cacert,
  libsecret,
  vte,
  libxcrypt,
  libsoup_3,
  webkitgtk_4_1,
  lttng-ust_2_12,
  libxcrypt-legacy,
  icu,
  gtk3,
  dotnet-sdk_8,

  makeWrapper,
  autoPatchelfHook

  }:

stdenv.mkDerivation rec {
  pname = "rdm";
  version = "2<IP_ADDRESS>";

  src = fetchurl {
    url = "https://cdn.devolutions.net/download/Linux/RDM/${version}/RemoteDesktopManager_${version}_amd64.deb";
    sha256 = "C70B33945618D000825994A9261C1473DA05525717CB06AACE3220F17197C780";
  };

  dontConfigure = true;
  dontBuild = true;

  nativeBuildInputs = [ dpkg makeWrapper autoPatchelfHook];

  buildInputs = [ dpkg
                  cacert
                  libsecret
                  vte
                  libxcrypt
                  libsoup_3
                  webkitgtk
                  webkitgtk_4_1
                  lttng-ust_2_12
                  libxcrypt-legacy
                  icu.dev
                  icu
                  gtk3
                  gtk3.out
                  dotnet-sdk_8

                  ];

  unpackPhase = ''
    dpkg-deb -x $src $out
  '';

  installPhase = ''
    runHook preInstall

    export DOTNET_SYSTEM_GLOBALIZATION_INVARIANT="1"

    mkdir -p $out/usr/bin

    echo "export DOTNET_SYSTEM_GLOBALIZATION_INVARIANT=1" > $out/bin/remotedesktopmanager
    echo "exec $out/usr/lib/devolutions/RemoteDesktopManager/RemoteDesktopManager \$@" >> $out/bin/remotedesktopmanager

    ln -s $out/bin/remotedesktopmanager $out/bin/rdm
    ln -s $out/bin/remotedesktopmanager $out/usr/bin/remotedesktopmanager

    wrapProgram $out/bin/remotedesktopmanager --prefix LD_LIBRARY_PATH : ${gtk3}/lib:${gtk3.out}/lib

    runHook postInstall
  '';

  shellHook = ''
    export DOTNET_SYSTEM_GLOBALIZATION_INVARIANT="1"
  '';

  meta = {
    description = "Devolutions Remote Desktop Manager";
    homepage = "https://devolutions.net/remote-desktop-manager/";

  };
}

I changed the code. It worked after change the config file directly. I’m stuck on the error below.
Error
ERROR SILENT GLib.GException: Unrecognized image file format
  at Gdk.PixbufLoader.Close()
  at Gdk.PixbufLoader.InitFromBuffer(Byte[] buffer)
  at Devolutions.RemoteDesktopManager.ImageHelper.CreateImageFromSVGContent(String content, Int32 size)
  at Devolutions.RemoteDesktopManager.Managers.ImageManager.SvgConverterService.Convert(String svgText, String name, Int32 size)
  at Devolutions.RemoteDesktopManager.Managers.ImageResources.GetSvgImage(String svgName, ImagePaletteType palette, Int32 originalSize, ImageEffect effect, Nullable`1 deviceDPI)
  at Client.Helpers.BuilderHelper.GetSvg(String name, String size, String color)
  at Client.Helpers.BuilderHelper.TranslateImage(Image image)

Nix Code
{ lib,
 stdenv,
 fetchurl,
 dpkg,
 webkitgtk,
 cacert,
 libsecret,
 vte,
 libxcrypt,
 libsoup_3,
 webkitgtk_4_1,
 lttng-ust_2_12,
 libxcrypt-legacy,
 icu,
 gtk3,
 gdk-pixbuf,
 librsvg,
 shared-mime-info,
 dotnet-sdk_8,

 makeWrapper,
 autoPatchelfHook

 }:

stdenv.mkDerivation rec {
 pname = "rdm";
 version = "2<IP_ADDRESS>";

 src = fetchurl {
   url = "https://cdn.devolutions.net/download/Linux/RDM/${version}/RemoteDesktopManager_${version}_amd64.deb";
   sha256 = "C70B33945618D000825994A9261C1473DA05525717CB06AACE3220F17197C780";
 };

 dontConfigure = true;
 dontBuild = true;

 nativeBuildInputs = [ dpkg makeWrapper autoPatchelfHook];

 buildInputs = [ dpkg
                 cacert
                 libsecret
                 vte
                 libxcrypt
                 libsoup_3
                 webkitgtk
                 webkitgtk_4_1
                 lttng-ust_2_12
                 libxcrypt-legacy
                 icu.dev
                 icu
                 gtk3
                 gtk3.out
                 gdk-pixbuf
                 librsvg
                 shared-mime-info
                 dotnet-sdk_8

                 ];

 unpackPhase = ''
   dpkg-deb -x $src $out
 '';

 installPhase = ''
   runHook preInstall
   mkdir -p $out/usr/bin

   ln -s $out/bin/remotedesktopmanager $out/bin/rdm
   ln -s $out/bin/remotedesktopmanager $out/usr/bin/remotedesktopmanager

   runHook postInstall
 '';

preFixup = ''
gappsWrapperArgs+=(
   # Thumbnailers
   --prefix XDG_DATA_DIRS : "${gdk-pixbuf}/share"
   --prefix XDG_DATA_DIRS : "${librsvg}/share"
   --prefix XDG_DATA_DIRS : "${shared-mime-info}/share"
 )
 '';

preInstall = ''
 echo "exec $out/usr/lib/devolutions/RemoteDesktopManager/RemoteDesktopManager \$@" > $out/bin/remotedesktopmanager

 wrapProgram $out/bin/remotedesktopmanager --prefix LD_LIBRARY_PATH : ${gtk3}/lib:${gtk3.out}/lib
 wrapProgram $out/bin/remotedesktopmanager --prefix LD_LIBRARY_PATH : ${icu}/lib:${icu.dev}/lib

 tee  $out/usr/lib/devolutions/RemoteDesktopManager/RemoteDesktopManager.runtimeconfig.json << END
   {
       "runtimeOptions": {
       "tfm": "net8.0",
       "includedFrameworks": [
         {
           "name": "Microsoft.NETCore.App",
           "version": "8.0.4"
         }
       ],
         "configProperties": {
             "System.Reflection.Metadata.MetadataUpdater.IsSupported": false,
             "System.Runtime.Serialization.EnableUnsafeBinaryFormatterSerialization": false,
             "System.Globalization.Invariant": false
       }
     }
   }

'';

postInstall = ''

'';

 meta = {
   description = "Devolutions Remote Desktop Manager";
   homepage = "https://devolutions.net/remote-desktop-manager/";

 };
}


Allow setting custom images for different modules

As of today, operator will use hard references to container images to use. eg. apicurio/apicurio-studio-ws:latest for the WS module.
We should provide a way to override this and setup custom images for the different modules.
Proposed CR syntax:
apiVersion: studio.apicur.io/v1alpha1
kind: ApicurioStudio
metadata:
  name: apicurio-sample
spec:
  name: apicurio-sample
  wsModule:
    image: quay.io/lbroudoux/apicurio-studio-ws:latest

Done


Gracefully end queue threads

Gracefully end queue threads.

Closed by https://github.com/project-codeflare/rayvens/pull/53


Update Migration Guide

This first commit is just for the Overview page.
I have several other pages I will add or edit.
All the content that was in the overview page will exist in a different location.

ping @adegeo
Can you look at this stale PR?

I think @StephenBonikowsky moved on from working in this area. As this PR indicates that there were more articles to be added from the information deleted out of the overview, I think we should close this as incomplete.

Yes that is fine, priorities changed.


added sendChatAction typing for telegram bot

fix #199

Thanks! I just tried it out and it seems to work well.


Updated amazon adapter to use v3 credentials and settings

Amazon uses a new sdk/api version. This update works with it.

Looks good to me. Not sure about the change of use from constants to just strings, but not unsure enough to really make any issue on it :)

Agreed on the strings instead of constants, but i could not find the constants anywhere in the amazon sdk classes. So had to resort to string.

That's fair - SQS seems to be the primary use case for this module from everything i've seen - so i'm happy to include this update.
(i find this interesting because i never did full on production testing of SQS due to primarily working on non-AWS infrastructure in my magento career :) )

Full production test of this will happen pretty shortly for this project im using it on. If it gives errors, i'll let you know what it is.


Not able to set the event subset.

Hey,
I am adding a new subset with the prefix 'e'. I want the data to be published whenever the value is updated. However, I am continuously receiving published data but the value has not changed.
Thanks in Advance,

Closing this in favor of https://github.com/ThingSet/thingset-zephyr-sdk/issues/2.


Support for xiaomi.derh.lite (Xiaomi Smart Dehumidifier Lite)

Hello!
it's possible to give support to this device:
Xiaomi Smart Dehumidifier Lite (xiaomi.derh.lite)
https://home.miot-spec.com/s/xiaomi.derh.lite
Thanks a lot!

It's seems that is the international version of the xiaomi.derh.13l device

Does  it  not  work  now?  Are  there  no  entities  or  logs?

Thanks for your reply
I was thinking about buying the device, but first I looked for support at https://home.miot-spec.com/s/xiaomi.derh
I saw that it did not have the "miot-local" badge and that it was not on the list of devices with local support either: https://github.com/al-one/hass-xiaomi-miot/blob/master/custom_components/xiaomi_miot/core/miot_local_devices.py

It  is  most  likely  to  be  supported.  If  you  purchase  it  and  integrate  it  successfully  using  ip/token,  I  will  add  it  to  the  local  device  list.

I have bought one and provide the required information if you walk me through it.

It  is  most  likely  to  be  supported.  If  you  purchase  it  and  integrate  it  successfully  using  ip/token,  I  will  add  it  to  the  local  device  list.

It's possible to add this dehumidifier to the integration but all the sensors and settings are unavailable.
The only setting is available is "set humidity level" which is anyway reporting 0

I solved the issue by adding custom option chunk_properties: 1 to the device.
See https://github.com/al-one/hass-xiaomi-miot/issues/1055#issuecomment-1482504030


How to implement router?

I need to define different services to handle different tasks, such as:
#[tarpc::service]
pub trait UserService {
    async fn login(name: String);
}

#[tarpc::service]
pub trait OrderService {
    async fn query(name: String);
}

The current problem is how to register these two services into the framework, and then create clients using 'UserServiceClient::new' and 'OrderServiceClient::new' and call them?

Thanks for the question! Tarpc doesn't natively support registering multiple services on a single server. I can suggest a few options:

Use a separate transport for each service
Use a multiplexing transport that can forward to different services. See the example at https://github.com/google/tarpc/issues/300#issuecomment-1048229777.

The "multiplexing transport" solution is good, but it requires writing a lot of code. Can't the tarpc framework handle these things itself?
The microservice framework should be designed to be modular so that we can organize the code more conveniently.

The main reason tarpc can't easily support native multiplexing is because the transport is typically generic over the request/response type.
Could the multiplexing code be written generically as a library? Then it wouldn't have to be duplicated by each user. If it were proven popular enough, perhaps it could be merged into tarpc.

Currently, I'm using the quic-rpc library which can meet my need for modularizing the code, but writing code is not as concise as using the tarpc framework.


CodePen Support

Any idea what's not right about this CodePen?

Not sure what you mean? It seems to working for me.

There's no syntax error, but it still doesn't work. The handleResize method that should get called on window resize is never called and the render method should display the component's dimensions:
<MeasureComponent bounds onResize={this.handleResize}>
  {({ measureRef }) => (
    <div ref={measureRef} className='container'>
      {JSON.stringify(dimensions)}
    </div>
  )}
</MeasureComponent>

Ah I think it's because I have the Chrome flag turned on so it might be using the native one 😞 sorry about that. I'll work on fixing the build as soon as I can.

Any news on this?  I'd like to demo my component using react-measure but when I wrap it in  nothing shows up.


New Complex base class breaks Fluid example

Reported by dietmarw on 17 Feb 2010 12:54 UTC
During clean up of another issue (see #292) I discovered that the model Modelica.Fluid.Examples.AST_BatchPlant.BaseClasses.TankWithTopPort
fails upon check. Using a bisecting techniqe I could determine that the reason is most likely to find in the addition of the Complex base class.
Here the output from git bisect (translated into svn revisions)
2a2fde020554a1f0451ff5bc6ef4f0523fe143ef is the first bad commit
commit r3321
Author: otter <otter@7ce873d0-865f-4ce7-a662-4bb36ea78beb>
Date:   Tue Jan 26 22:36:25 2010 +0000

    Added Complex number definition in global scope

A	Complex.mo

Migrated-From: https://trac.modelica.org/Modelica/ticket/293

Comment by AHaumer on 17 Feb 2010 18:59 UTC
We had problems with Machines exmaples as soon as Complex was loaded, too.
It was related to operator overloading in Dymola 7.3 and should be fixed in the upcoming Version 7.4

Comment by dietmarw on 18 Feb 2010 06:43 UTC
Silly typo in title corrected.

Comment by dietmarw on 18 Feb 2010 07:18 UTC
I'm afraid it's not that simple. The final version of Dymola 7.4 shows exactly the same error. The thing is that the model in question does not even use any Complex stuff (AFAIKT).

Comment by dietmarw on 9 Mar 2010 12:47 UTC
We should not have a MSL with broken examples...

Comment by msielemann on 12 Mar 2010 15:37 UTC
At least in Dymola 7.3 the error seems to be due to the following constructs:
```text/x-modelica

for i in 1:nTopPorts loop
   topPorts[i].Xi_outflow = X_start[1:Medium.nXi];
end for;

for i in 1:nPorts loop
    ports[i].Xi_outflow = medium.Xi;
end for;
```

I am trying to fix this now.

Comment by msielemann on 12 Mar 2010 16:13 UTC
This problem was fixed in this and a few additional components in 2494d191d840a22729953646fbd3bdbebb997ab1. I checked the Fluid package after the modifications and now it succeeds (no check w/ simulation because of lack of time, sorry).
This seems to be a bug in the Modelica tool, which has to be fixed independently (perhaps you should have done the bisecting on another repository... :)

Comment by dietmarw on 13 Mar 2010 17:46 UTC
Unfortunately the situation is now even worse:
Finished

Error: Simulations failed for:

Modelica.Fluid.Examples.PumpingSystem

Modelica.Fluid.Examples.Tanks.ThreeTanks

Modelica.Fluid.Examples.Tanks.EmptyTanks

Modelica.Fluid.Examples.ControlledTankSystem.ControlledTanks

Modelica.Fluid.Examples.AST_BatchPlant.Test.OneTank

Modelica.Fluid.Examples.AST_BatchPlant.Test.TwoTanks

Modelica.Fluid.Examples.AST_BatchPlant.Test.TankWithEmptyingPipe1

Modelica.Fluid.Examples.AST_BatchPlant.Test.TankWithEmptyingPipe2

Modelica.Fluid.Examples.AST_BatchPlant.Test.TanksWithEmptyingPipe1

Modelica.Fluid.Examples.AST_BatchPlant.Test.TanksWithEmptyingPipe2

Modelica.Fluid.Examples.IncompressibleFluidNetwork

Modelica.Fluid.Examples.HeatExchanger.HeatExchangerSimulation

Modelica.Media.Examples.SimpleLiquidWater

This was done with Dymola 7.4 "check with simulation". This is really quite critical as I don't know how we can have a beta without even the simple test succeeding.

Comment by otter on 13 Mar 2010 18:32 UTC
There is a bug in the code generation of Dymola 7.4 related to the conversion from degC to Kelvin (so the failure of the simulation seems to be not due to the Complex record). Depending on factors that I do not understand, the generated code for this conversion is sometimes correct and sometimes not, and the fix from Michael gave a situation where more often the wrong conversion code is generated.
I detected this a few days ago, since some FluidDissipation examples failed and reported this to Dymola support. In the meantime this is fixed, but requires a new Dymola executable. The issue is that the conversion function results in the following wrong C-code:
double   Kelvin0_0;
      Kelvin0_0 = 288.15;
      /* Start of real code */
        Kelvin0_0 = Celsius0_0--273.15;

In order that people can still work with MSL3.2 with standard Dymola 7.4, I have changed the code of SIunits.Conversions.from_degC by using a literal value for T_zero and not a reference to the constant. So this issues is fixed with 918abf00431faa6645edeba364f2d71c61b006bb.

Comment by dietmarw on 20 Dec 2016 14:01 UTC
Unhide for GitHub transition.


Fix integration CI

This should fix the integration CI.
It looks like the sg-core's plugins aren't inside the plugin folder. I see the following errors in the CI output when it's trying to build the plugins:
error obtaining VCS status: exit status 128
Use -buildvcs=false to disable VCS stamping.

From what I found, go >= 1.18 is adding some version control information to the built binaries and this seems to be failing right now. This PR disables that. I don't see the feature being much useful. If we decide it is useful, we can find another way.

Lets see if this works

The fix worked, but one of the 4 actions failed because of a timeout on running rsyslog. The timeout was 180s and on the successful run it took a little 150s for rsyslog to start. I increased the timeout to 240s hoping, that it'll improve the CI stability.

I have a hunch this VCS info could be required for downstream builds (which depend on this build.sh), but  I haven't been able to prove it, yet.

This LGTM but it's worth noting that we build with golang 1.18.10 downstream and have not needed this flag. All else being equal, I'd rather the vcs info is in the binary if possible. So my vote is to disable the feature only where it's failing in CI.
CC: @leifmadsen @compi-migui Heads up on this in case we notice vcs related problems downstream (either because it's still on and it's broken, or we turned it off and it's required)


app.teardown_request and app.errorhandler investigation

Hello party people,
My understanding was that @app.teardown_request function would be called after a request whenever an error occurred or not.
However, it seems that when I set an @app.errorhandler(Exception), the error handler is called but then the @app.teardown_request function is not.
Is this the expected behaviour?
Best regards,
Rémy
@app.teardown_request
def commit_or_rollback_db_session(ctx):
    try:
        if g.get("rollback_db_session"):
            g.db_session.rollback()
        else:
            g.db_session.commit()
    except Exception:
        g.db_session.rollback()
        raise

@app.errorhandler(Exception)
@app.errorhandler(500)
def handle_internal_server_error(e):
    g.rollback_db_session = True
    return jsonify(error=True, code=500)

I can't reproduce this issue. In the following example, "handle_500 bad" and "teardown None" are printed, and "Handled 500" is returned. e is None in the teardown because it was already handled in the error handler.
from flask import Flask

app = Flask(__name__)

@app.route("/")
def index():
    raise Exception("bad")

@app.teardown_request
def teardown(e):
    print("teardown", e)

@app.errorhandler(Exception)
@app.errorhandler(500)
def handle_500(e):
    print("handle_500", e)
    return "Handled 500"

I guess the difference between your code and my code is the raise inside the teardown_request


optimize: keep the same logic of get table meta

Ⅰ. Describe what this PR did
Keep the same logic of get table meta. And make it more readable.
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2561 into develop will decrease coverage by 0.15%.
The diff coverage is 37.50%.

@@              Coverage Diff              @@
##             develop    #2561      +/-   ##
=============================================
- Coverage      51.42%   51.27%   -0.16%
+ Complexity      2807     2805       -2
=============================================
  Files            553      553
  Lines          17691    17668      -23
  Branches        2091     2050      -41
=============================================
- Hits            9098     9059      -39
+ Misses          7738     7731       -7
- Partials         855      878      +23

Impacted Files
Coverage Δ
Complexity Δ

...asource/sql/struct/cache/OracleTableMetaCache.java
75.90% <0.00%> (ø)
9.00 <0.00> (ø)

...rce/sql/struct/cache/PostgresqlTableMetaCache.java
76.40% <33.33%> (+0.31%)
10.00 <1.00> (ø)

...tasource/sql/struct/cache/MysqlTableMetaCache.java
82.55% <66.66%> (ø)
10.00 <0.00> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

... and 24 more


Fix legacy Surface.BlitChannel source rect

When you see it... :)
(I don't see any equivalent on V2, but if there was any copy&paste maybe we should double check that too).

Thanks :)


Update cosmos-readme.md

Docs Build status updates of commit 3e6b457:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/cosmos-readme.md
:warning:Warning
View (azure-python)
Details

docs-ref-services/latest/cosmos-readme.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning: ms-prod-technology-invalid - See documentation] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'cosmos'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 26, Column 114: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/' will be broken in isolated environments. Replace with a relative link.
Line 28, Column 28: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/sql-query-getting-started' will be broken in isolated environments. Replace with a relative link.
Line 28, Column 146: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/introduction' will be broken in isolated environments. Replace with a relative link.
Line 35, Column 9: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/account-overview' will be broken in isolated environments. Replace with a relative link.
Line 38, Column 71: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/cli/azure' will be broken in isolated environments. Replace with a relative link.
Line 96, Column 49: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/databases-containers-items' will be broken in isolated environments. Replace with a relative link.
Line 159, Column 39: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/analytical-store-introduction' will be broken in isolated environments. Replace with a relative link.
Line 159, Column 192: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/synapse-link' will be broken in isolated environments. Replace with a relative link.
Line 292, Column 77: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/how-to-sql-query' will be broken in isolated environments. Replace with a relative link.
Line 339, Column 34: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/time-to-live' will be broken in isolated environments. Replace with a relative link.
Line 348, Column 1: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/rest/api/cosmos-db/http-status-codes-for-cosmosdb' will be broken in isolated environments. Replace with a relative link.
Line 395, Column 68: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off


Feat: Download all videos together

Added the option to download all videos at once on the video tab.
https://github.com/openedx/openedx-app-android/assets/5153976/55401f90-ba50-4de8-b327-2356c5668c17

Not sure about moving ProfileNotifier to the core module.
Could you please explain the purpose of doing this? Some event needs to be consumed outside of the profile?