Fix mobile typing bug in messages

The typing area was getting cut off on mobile. This was because we weren't accounting for the mobile browser url bar. This isn't shown in Chrome responsive view, so we missed it.
Closes #5185
TESTING: It's best if you can test this on a real phone, i.e. use the Vercel link on your phone. This bug was only possible to replicate on mobile as the dev tools responsive view wasn't 100% accurate in this case.

Web frontend checklist

[ x ] Formatted my code with yarn format
[ x ] There are no warnings from yarn lint --fix
[ x ] There are no console warnings when running the app
[ x ] All tests pass
[ x ] Clicked around my changes running locally and it works
[ x ] Checked Desktop, Mobile and Tablet screen sizes

Seems to be working now! Though on chrome I am seeing a gap between the text field and the nav bar on my phone:
https://github.com/user-attachments/assets/cc4045be-d1a7-41e5-870c-ef6f69947b14

I'm seeing this now too which is better cause the text I'm composing isn't completely hidden, but partially is. Then I can scroll and there is space below the text field until I scroll back up then I lose my ability to scroll down and see the gap:
https://github.com/user-attachments/assets/52a01a24-7af8-4371-bd48-1cc9509698b9

Here is my latest test, I'd say it's good enough for now:
This is on android/firefox
https://github.com/user-attachments/assets/bd085e80-f375-4203-9a9b-1c9ba71a61e3

I confirmed that on Chrome it is completely fixed, working as expected.

Here is my latest test, I'd say it's good enough for now:
This is on android/firefox

Okay so I just pushed a fix that might fix firefox, so give that a try. BUT if not, since I don't have any device to replicate it, and even the firefox mobile simulator plugin can't replicate it, I'm going to consider it good enough for now! They can still type and it's better than now. I think probably the percentage of Android users using Firefox on the phone is low. Based on the number of Stack Exchange posts in this, it seems to be an Android bug.

@nabramow it's fixed now on firefox too! Thank you!!!

@nabramow it's fixed now on firefox too! Thank you!!!

Oh wow what luck, that was a total stab in the dark! Excellent we are all good then!

@bakeiro if you have a moment this bug fix is ready too!


Microsoft.Cpp.props was not found

[exec] Build FAILED.
[exec]
[exec] "C:\BlrApps\ViewingInfra\Src\Viewing.sln" (Build target) (1) ->
[exec] "C:\BlrApps\ViewingInfra\Src\ViewerSolution\Viewer\Viewer.csproj.metaproj" (default target) (14) ->
[exec] "C:\BlrApps\ViewingInfra\Src\ViewerSolution\DXWrap\DXWrap.vcxproj" (default target) (15) ->
[exec]   C:\BlrApps\ViewingInfra\Src\ViewerSolution\DXWrap\DXWrap.vcxproj(50,3): error MSB4019: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\VC\VCTargets\Microsoft.Cpp.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
[exec]
[exec]
[exec] "C:\BlrApps\ViewingInfra\Src\Viewing.sln" (Build target) (1) ->
[exec] "C:\BlrApps\ViewingInfra\Src\ApplicationUISolution\ApplicationUI\ApplicationUI.csproj.metaproj" (default target) (16) ->
[exec] "C:\BlrApps\ViewingInfra\Src\ImageProcessing3DSolution\ImageProcessing3D\ImageProcessing3D.csproj.metaproj" (default target) (17) ->
[exec] "C:\BlrApps\ViewingInfra\Src\Imaging3DSolution\Imaging3D\Imaging3D.csproj.metaproj" (default target) (18) ->
[exec] "C:\BlrApps\ViewingInfra\Src\Algorithms\Managed\Algorithms.vcxproj.metaproj" (default target) (19) ->
[exec] "C:\BlrApps\ViewingInfra\Src\Algorithms\Managed\Algorithms.vcxproj" (default target) (20) ->
[exec]   C:\BlrApps\ViewingInfra\Src\Algorithms\Managed\Algorithms.vcxproj(52,3): error MSB4019: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\VC\VCTargets**Microsoft.Cpp.props" was not found**. Confirm that the path in the  declaration is correct, and that the file exists on disk.
[exec]
[exec]
[exec] "C:\BlrApps\ViewingInfra\Src\Viewing.sln" (Build target) (1) ->
[exec] "C:\BlrApps\ViewingInfra\Src\DisplayWrapper\DisplayWrapper.vcxproj.metaproj" (default target) (56) ->
[exec] "C:\BlrApps\ViewingInfra\Src\DisplayWrapper\DisplayWrapper.vcxproj" (default target) (57) ->
[exec]   C:\BlrApps\ViewingInfra\Src\DisplayWrapper\DisplayWrapper.vcxproj(55,3): error MSB4019: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\VC\VCTargets\Microsoft.Cpp.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
[exec]
[exec]     0 Warning(s)
[exec]     3 Error(s)

This is entirely about the c++ props/targets it seems. This is not really the right repo for this issue. if this is still happening to you, please file a vs feedback item through visual studio.


Update repositories.txt

Hi @1337encrypted. If you have created a tag in your local clone of the repository, make sure to push it to the remote. You can see here that your GitHub repository does not have any tags, just as reported by the bot:
https://github.com/1337encrypted/BTS7960_Motordriver/tags
Please make sure you can see a tag there. If not, there is no point in triggering the bot repeatedly here because it will always reject your submission with the same error.


Specify output dir

Continue the work of @tony to add support for specifying the output directory (resolve #452).
I also added an according cli option:
$ cookiecutter -o tmp/foobar gh:hackebrot/cookiedozer

or
$ cookiecutter --output-dir tmp/foobar gh:hackebrot/cookiedozer

You can also use the Python API via:
from cookiecutter import main
main('gh:hackebrot/cookiedozer', output_dir='tmp/foobar')

I also think it would be great to include this in the upcoming Cookiecutter 1.1.0 release (#530).
Please note that I did not include the changes to existing tests of #452.
cc @tony, @cabello, @jefftriplett and @davedash

Apparently the template url is not identified as a local folder on Windows (related to #490).
Also travis is currently struggling with build delays :cry:

Please review #532 first. Thanks :bow:

Current coverage is 99.16%

Merging #531 into master will increase coverage by +0.01% as of bac61e4

@@            master    #531   diff @@
======================================
  Files           13      13
  Stmts          476     477     +1
  Branches         0       0
  Methods          0       0
======================================
+ Hit            472     473     +1
  Partial          0       0
  Missed           4       4

Review entire Coverage Diff as of bac61e4

Uncovered Suggestions

+0.42% via ...iecutter/init.py#15...16
+0.42% via ...ookiecutter/utils.py#27...28

Powered by Codecov. Updated on successful CI builds.


adds a simple PR template

This template will be applied when any new PRs are created against Traffic Control. The idea is that it will help capture more information for potential reviewers/mergers and lead to better PRs and faster merges.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2251/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2252/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2253/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2254/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2255/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2256/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2258/
Test PASSed.


Correct supported Meilisearch version

Codecov Report

Merging #230 (81b0408) into main (7ba1fbe) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #230   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          402       402
=========================================
  Hits           402       402

Continue to review full report at Codecov.

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


iirdes: fix number of zeroes in the docs

The number of analog zeros that this function outputs is 2*L; fix the documentation to match that.

Hmm the coverage check appears to be flaky, showing a change in a line of code that wasn't actually impacted in any way (indeed this commit here is a one line comment-only change).

I think what happened was that I had made a commit between when you forked/committed and this merge that dropped the coverage down by a couple of lines. And that made the merge itself drop some lines. But it's really not a big deal. I'm just trying to avoid merging in massive changes that aren't properly checked. It's a bit annoying that codecov throws such a fuss over minor changes like this. I'm sure there's a setting somewhere to change this, but I can't be bothered to look it up 🤣


Array of struct type to kernel giving CUDA build failed error

I am trying to create a struct with one of its variable being an array of datatype of another struct. I am able to assign the struct variables without any error. But when I send it to the kernel I receive CUDA build failed error.
Error:
Warp NVRTC compilation error 6: NVRTC_ERROR_COMPILATION (/buildAgent/work/3db450722a274445/warp/native/warp.cu:1107)
default_program(63): error: identifier "MyStruct" is undefined
default_program(67): error: identifier "MyStruct" is undefined
default_program(79): error: identifier "MyStruct" is undefined
default_program(81): error: identifier "MyStruct" is undefined
default_program(214): error: identifier "MyStruct" is undefined
The code that I am trying to run is :
@wp.struct
class MyStruct:
param1: int
param2: float
param3 : wp.array(dtype=int)

@wp.struct
class struct2:
index : wp.array(dtype=MyStruct)
ptr : int

var = MyStruct()
var.param1=1
var.param2 = 2.0
var.param3 = wp.array([1,2,7], dtype=int)
var1 = MyStruct()
var1.param1=1
var1.param2 = 3.0
var1.param3 = wp.array([1,2,8,9], dtype=int)
teststruct = struct2()
teststruct.index = wp.array([var, var1], dtype=MyStruct)
teststruct.ptr=1
@wp.kernel
def struct2_reader(test : struct2):
k = wp.tid()
wp.printf('\n%d',test.index[k].param3[2])

wp.launch(struct2_reader, dim=2, inputs=[teststruct])
Please tell me if I am making some mistake or such operations arent supported yet in Warp. Any help is appreciated.
Thanks in advance.

Hi @nithinsomu,
Thanks a lot for the report - I could repro this bug. locally, the nested array struct was not being picked up for code-gen. I have a simple fix here:
https://github.com/NVIDIA/warp/commit/0d2035d1cf61dd0ff6b48eab7ff91803ec912eb0
This will make it into the next release.
Cheers,
Miles


[BUG] Make id of Hit Nullable

What is the bug?
Executing a search from Java Opensearch client by disabling stored fields ("stored_fields: none")  fails with an MissingRequiredPropertyException. Even though, the query is executed by the Opensearch server (confirmed through Opensearch console), the Java client cannot process a response with null _id field.
How can one reproduce the bug?
Create a new query with stored_fields as none and opensearch client will throw an error.
SearchRequest.Builder searchRequest = new SearchRequest.Builder();   searchRequest.storedFields("_none_");   return searchRequest.build();
Execute the above search request for any filter/query and you can notice that the Java client throws an MissingRequiredPropertyException: Missing Required Property 'Hit.id'
What is the expected behavior?
Since the query with "none" stored fields is executed by opensearch server, the Java client should be able to process it and return the response without throwing an exception.
What is your host/environment?
Opensearch Java - 2.7.0
Do you have any screenshots?

Do you have any additional context?
Searching & Retrieving documents with id field is  very slow potentially due to reading the id from stored fields. Executing the search by disabling stored fields (by using "stored_fields: none") makes the search faster.  However, the Java client cannot process responses with null Id from Opensearch server.

Looks like a legit problem. Want to try and write a (failing) unit test for this @hubertazg?


Divergence from whisper.transcribe.transcribe

Looks like whisper changed the parameter name from no_caption_threshold to no_speech_threshold in this commit.
Great project!

updated


60fpsBlackscreenFix and CutsceneFix mod has released

https://www.reddit.com/r/NewYuzuPiracy/comments/13hq70a/60_fps_mod_black_screen_fix_not_thoroughly_tested/?utm_source=share&utm_medium=web2x&context=3

60fps CutsceneFix mod has released in LOZ: Tears of the Kingdom 20fps, 30fps & 60fps Patch <IP_ADDRESS>
https://gbatemp.net/download/loz-tears-of-the-kingdom-20fps-30fps-60fps-patch.37996/

Hello, we added the cinematic fix from theboy181 but not released yet, about the blackfix fix i can't do anything i don't get over 35fps and my blackscreen issues are too rare to test the fixes, i ask to someone else from the repo to add the better blackscreen fix.


feat: allow to set an alias and view them

Status - just waiting to verify some nginx changes before proceeding with this PR


Add E2E test scenarios for JS source map upload

Goal
Adds mazerunner scenarios that verify the BAGP can upload JS source maps in a react native app.
Changeset
Note: bugsnag-source-maps has not been fully implemented yet so the gradle plugin does not currently upload source maps. The source map scenarios will therefore fail - but should pass as soon as this functionality is added.

Made it possible for each RN fixture to enable productFlavors, to enable Hermes, and to disable bugsnag's uploadReactNativeMappings flag

Added scenarios for:

Assembling an app
Bundling an app
Assembling an app that uses product flavors
Manually invoking the source map upload task
Disabling sourcemap upload
Uploading a Hermes sourcemap

Steps were added to mazerunner to help verify the source map request contained the expected fields.

Addressed review feedback


zip, tar.gz, and tar.bz2 links do not work in the latest version of the documentation

I tried downloading the standalone liquigraph component from the help page, and it can't be found in the maven repository.
I stepped back one less version in the documentation, and I noticed that those links do work.
It looks to just be a link reference error.

Hi,
Thanks a lot for the report! I'll fix the versions in the links, unless you want to submit a pull-request on the documentation? :)

It should be fixed now, thanks for reporting!


[test] Remove no-op artifacts builds

See discussion in cl/533274447.
I think once we remove the grpc_build_artifact configs, we should remove the grpc_build_packages.cfg and grpc_distribtests.cfg jobs as well?


Fixes Issue: #365 Dockerizing the project

What is the change?
Dockerzed the project
Related issue?
close: #365
How was it tested?
Tested Locally
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Did you lint your code before making the Pull Request?
[x] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

losing for now, reopen if required


Adding a custom preloader ⌛

Hey! 👋😃 I am a very beginner level front end dev, I love to learn new things. I am designing a small personal blog for me and I am using your theme it is very awesome theme 😍. I was thinking about if there is any way that we can enable or add a custom "preloader" (sample 1, sample 2) for the website so it will show up on screen instead of blank screen until everything loaded because I saw this blank screen showed up until everything loaded. Sorry if I sounds like noob because I am trying to learn new things. I hope you can navigate me.
Thank you! ❤

Hey 👋🏼
Adding a preloaded is not an easy thing to do since it requires quite work in order to know exactly when all the data and all the resources are downloaded and ready to use, it's not just about displaying a fancy css animation.
Also I believe that a preloader is bad for UX in this case and you have to ask yourself if it's something that you really need, I would say no. I say it's bad for UX because the user can't use the website until you decide to hide the preloader, maybe for complex apps is useful but not for a blog where I think is more important to let users to read the content as soon as possible even if some resources are not ready yet.
I guess you can implement it as a fake preloader. For example, to implement a fake loader that looks like the first example, you can do the following:

Copy the HTML from that example and paste it after the <body> tag in the file default.hbs

Then you can use the style from that example and inject it into the Header section of the Code injection page in your admin panel. You can't use SCSS so you have to convert it to CSS first.

Finally paste the following code into the Footer section of the Code injection page in your admin panel:

<script>
window.addEventListener('load', function() {
  const preloader = document.querySelector('.loader');
  preloader.remove();
});
</script>

Thanks a lot ❤  it worked 💯 👍


restore test_apply_paddings_check runtime_checks test

The main idea is that we need to call jax.effects_barrier(), because the error may be raised in an XLA computation that is asynchronous with the main Python thread and therefore we need to block. (There may have been a recent change in behavior, where JAX runs more computations asynchronously on the CPU backend.) We could put that call to jax.effects_barrier() in the test code (and corresponding user code), or we could bulid it into the runtime_checks context manager.  Currently this commit does the latter.
I also tweaked the runtime_checks logic to use a try/finally pattern to restore the state when the context is exited, even when it's exited via exception. We may want to do the same to context managers like numeric_checks.
While the test now passes, there is a gross warning printed about "Exception ignored in atexit callback". That may be a JAX internal bug, or it may be some quirk of CPython 3.10; I haven't investigated further. Let me know if that seems like a problem.
What do you think?

My intrepid teammates @yashk2810 and @hawkinsp noticed that in the most recent release of JAX we no longer raise jaxlib.xla_extension.XlaRuntimeError but rather jax.errors.JaxRuntimeError. I'll try to update that in this PR (under a version switch), or send a follow-up PR if this PR gets merged before I make the fix.

Thank you!


feat: support filter api

Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:
CI Switch
ci-runs-only: [Chaos CI, Cargo Clippy, Coverage Test, E2E Tests, Code Format, Unit Tests]

Codecov Report

Merging #202 (c917cb6) into main (af936e0) will decrease coverage by 0.96%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #202      +/-   ##
==========================================
- Coverage   42.58%   41.62%   -0.97%
==========================================
  Files          87       88       +1
  Lines        5971     6114     +143
==========================================
+ Hits         2543     2545       +2
- Misses       3428     3569     +141

Impacted Files
Coverage Δ

core/api/src/jsonrpc/filter.rs
0.00% <0.00%> (ø)

core/api/src/jsonrpc/mod.rs
0.00% <0.00%> (ø)

core/api/src/jsonrpc/web3_types.rs
0.00% <0.00%> (ø)

core/api/src/jsonrpc/ws_subscription.rs
0.00% <ø> (ø)

protocol/src/codec/mod.rs
60.00% <0.00%> (+3.47%)
:arrow_up:


Drop Core framwork for OS X

At the moment PlatformFolders uses CoreServices to find the relevant folders on OS X. However that may be overkill.
Unlike other operating system (like Linux and Windows) OS X does not allow the common directories to be moved or renamed. Moving a folder to another drive involves symlinking it and translations are done fully in the UI.
The "Public" folder is already hardcoded as it is not supported by FSFindFolder. I wonder if I should do the same with the rest. Can anyone see anything bad in this approach?
Reference:
https://apple.stackexchange.com/questions/145476/move-documents-folder-to-different-drive
https://developer.apple.com/library/archive/documentation/General/Conceptual/MOSXAppProgrammingGuide/AppRuntime/AppRuntime.html
https://apple.stackexchange.com/questions/24159/how-do-i-rename-the-downloads-folder-in-lion/24258#24258

I don't know much about macOS, but if they made the folders unmoveable then I guess hardcoding paths isn't terrible. Especially since those FSXXX calls are deprecated.

Looking a bit further. There might be a way to call the new API. While all Apple documentation refers to the Objective-C way, the underlying C interface is apparently exposed: http://opensource.apple.com//source/Libc/Libc-825.40.1/include/NSSystemDirectories.h
This seems to be the right way.

But is that header actually part of any public API, or is it subject to be changed on a whim by Apple?

Reading the source code http://opensource.apple.com//source/Libc/Libc-825.40.1/gen/NSSystemDirectories.c
it turned out that all values are hard coded. The function also contains "~" in the result and expanding is left as an exercise to the reader. So it makes sense to just do like "Public" for all Mac folders.

All paths for Mac are now hard coded.


Overview

Pulls info dynamically by API. Price, thumbs, and style selector complete. Header art added. Style id moved to App. Git auth example file created.

WOW


Feature: [Dev Experience] Add dev container configuration for GitHub Codespaces

Description
Fixes #4163
Motivation
I would like to propose adding a dev container configuration for GitHub Codespaces. This would make it easier for developers to set up their environment and work on the project without much hassle and effort. Setting up the project for the first time requires CMake, Docker, node, yarn, awslocal protoc and other tools. However, installing and configuring these tools manually can be time-consuming and error-prone, especially for new contributors or low-end machines.
Solution
It would be nice to have a dev container configuration for GitHub Codespaces that would automate the setup of the development environment and provide a consistent and reproducible experience. This would make contributing to the project easier as it would:

Lower the barrier to entry for new contributors due to the ability to quickly spin up a fully set up environment and focus on making a contribution instead of spending hours trying to get the right version of some tool installed.

Tasks to make this happen

[x] Add dev Container configuration
[ ] Update README.md and CONTRIBUTING.md to include information on how to get started with a codespace

How was this PR tested?
This Pr was testing by:

Creating a GitHub codespace using the generated devcontainer configuration and checking that all the required tools were set up successfully.
Running the common commands that a contributor will be expected to run and confirming that they worked.

@fulmicoton Please take a look

thanks a lot @unpervertedkid
I tested it and it worked well, the dev UX is really really nice. I saw 3 issues:

the disk space is small, I did a cargo build and make test-all and run out of disk space
I got errors in some tests, I'm unsure if it was related to the disk space or not, I will rerun the test suite on a new codespace
the price but it's not related to your code.

Overall, I'm good with your PR. @guilload do you want to check the content?

@fmassot The disk space for codespaces is limited for freee codespaces but if one has access to paid codespace they can then configure a codespace with more disk but for the mean time I set the minimum requirements to be the maximum disk and compute space for the free codespace configuration.
Let me try to recreate the errors you ran into and try to figure out what the issue could be. I will also try and look into caching but I would want to get this PR in and then I can open another PR to work on that.

Looks good to me!

The price is not related to your code: quickwit takes time to compile so we need a lot of compute, it would be nice to be able to do so some caching there.

Hi @fmassot, I was thinking about your suggestion to use caching to speed up the compilation. I think it’s a great idea and I would love to work on it. My first thought was to use sccache, a distributed compiler cache for Rust. Do you think that would be a good solution? If so, could you please create an issue with a clear guide on what we should cover, and I will pick it up then. Thank you!


update PyO3

Would it be possible to update PyO3? The conda-forge recipe fails to support PYPY interpreter, which would require v0.15.2 upward to circumvent the UTF8 undefined attribute error.
Updating to 0.15.2 would require to update:
return Err(exceptions::ValueError::py_err("Invalid language_code"))

Updated to lastest PyO3 (0.2.3) in https://github.com/polkascan/py-bip39-bindings/releases/tag/v0.2.0

Thanks. Note that conda-forge dropped support for PyPy


fix(#199): Align dosage of MedicationLine profile with the future one from MedicationRequest.

I'm going to convert this to draft until this is cleared up. See #206

WG meeting 02-07: we decide not to proceed with this change - the Medication Line dosage remains 0..*


[r/python] Ensure that all public APIs have docstrings and lifecycle-maturity tags in their docstrings

Not yet to worry about formatting for readthedocs or other tools -- rather, here just to focus on whether all APIs have available information accessible from on-line help.

Appears to be a duplicate of #602


New paragraph at the beginning and new setting to hide money text after username

[x] Remove extra space
[x] Paragraph at the beginning with additional information
[x] Added setting to hide money text after username

Finally I have thought that it is not such a good idea to interpret HTML code for security, for that reason I am going to leave it like this and regarding the slug, I will leave it for a future since it is not so important.

Finally I have thought that it is not such a good idea to interpret HTML code for security, for that reason I am going to leave it like this and regarding the slug, I will leave it for a future since it is not so important.


Tests regarding frontend-caching are failing

Describe the bug
Since KC 24.0 has changed the behavior for the welcome page/content, there seems no more available static resource, which we can request to check the caching header. Thus, the tests are failing. For a quick "fix", the tests have been disabled, but should be enabled as soon as possible again.
This feature was introduced with #86
Version
3.3.0
Expected behavior
No response
Actual behavior
No response
How to Reproduce?
No response
Relevant log output
No response
Anything else?
No response

fixed with 02e2d0bdeef41e66db4555dd1c88a942a278a417


"aria-expanded" attribute on the "button" element, but not on the "span" element

What for do we need set a new aria-expanded attribute on the button element? Because in index.js we get exactly the button, not the inner span.
https://github.com/kevin-powell/space-tourism/blob/558f56b84625694d126b73de569766ee0af4b279/lesson-files/36/02-end/index.js#L2-L9

Before click on the button:

After click on the button:

We don't need the aria-expanded on the span itself. The aria-expanded is referring to what the button controls, as defined by the aria-controls. The span is just the text that's inside the button.

That's why I made the issue, because in the course (Scrimba) I noticed that we set the aria-expanded attribute on the span.
https://github.com/kevin-powell/space-tourism/blob/558f56b84625694d126b73de569766ee0af4b279/lesson-files/30/02-end/index.html#L24
Thanks for the quick response 👍

Oh no! I misunderstood, sorry about that! I'll dive in and take a look :)
On Wed, 2 Feb 2022 at 04:04, Rinat Valiullov @.***>
wrote:

That's why I made the issue, because in the course (Scrimba) I noticed
that we set the aria-expanded attribute on the span.
https://github.com/kevin-powell/space-tourism/blob/558f56b84625694d126b73de569766ee0af4b279/lesson-files/30/02-end/index.html#L24
Thanks for the quick response 👍
—
Reply to this email directly, view it on GitHub
https://github.com/kevin-powell/space-tourism/issues/4#issuecomment-1027721581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGEOPH46SF55LFG42PBAVZTUZDXS5ANCNFSM5M6L6IBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Attestation token is missing

2024/07/01 13:43:43 No existing device registration found, will perform registration now

What is your phone number's country code? (digits only): 1

What is your phone number? (digits only): <redacted>
[AUTHY_DEBUG] Sent request to: https://api.authy.com/json/users/1-<redacted>/status, got response: {"message":"Attestation token is missing","success":false,"errors":{},"error_code":"60000"}
2024/07/01 13:43:49 Device registration failed: There doesn't seem to be an Authy account attached to that phone number

https://www.bleepingcomputer.com/news/security/hackers-abused-api-to-verify-millions-of-authy-mfa-phone-numbers/
Most likely related to this

I'm getting the same message, will this require api changes before this tool will work again?


Internal improvements in XML document schema implementation

Codecov Report

Merging #1072 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1072   +/-   ##
========================================
  Coverage    63.58%   63.58%
========================================
  Files          335      335
  Lines        12160    12160
========================================
  Hits          7732     7732
  Misses        4428     4428

Continue to review full report at Codecov.

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


Does anyone here has this integration running without composer?

Does anyone has this integration running without composer right now?
Can you please tell us where you have downloaded the lib and what is your 'require' line please?!

Duplicate issue - redirecting people here https://github.com/sendgrid/sendgrid-php/issues/341#issuecomment-270430098


QIcon cannot read .png using relative path after compiling

nuitka version
1.5.7
Commercial: None
Python: 3.10.11 | packaged by Anaconda, Inc. | (main, Apr 20 2023, 18:56:50) [MSC v.1916 64 bit (AMD64)]
Flavor: Anaconda Python
Executable: D:\miniconda3\envs\default\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10

command I use
python -m nuitka --output-dir="D:\Users\abcdesteve\slsqq/output" --onefile --windows-disable-console --standalone  --enable-plugin=pyside6 --include-data-files="D:/Users/abcdesteve/Desktop/神龙/神龙OSD/神龙OSDv1.1/icon.png"="icon.png"   --product-name="OSD" --product-version="1.1" --company-name="Avoconal" --copyright="abcdesteve"   D:/Users/abcdesteve/Desktop/神龙/神龙OSD/神龙OSDv1.1/main.py

pyside6 version
6.5.0

behavior

no waring
no icon shown
do show icon when place the icon in the same dir(I have contain the icon into the exe,see # command I use)

You really need to read the section about finding files from the user manual. You are probably using "current directory" relative paths that are not going to work and need to change it accordingly.


[goldfinch-membership] goldfinch-membership strategy

New strategies for goldfinch protocol
See also https://github.com/snapshot-labs/snapshot-strategies/pull/497
Changes proposed in this pull request:

A goldfinch membership strategy that checks for eligibility via a UID or the goldfinch golist

@ChaituVR thanks for the feedback. I moved voting-power to https://github.com/snapshot-labs/snapshot-strategies/pull/497

Thanks, updated both PRs

Thanks @sanjayprabhu Looks good


Binary artifacts in Druid source code

There was a potential issue raised in the Dev sync today regarding binary artifacts. It was mentioned that in order to comply with the ASF release standards, the release package should not contain binary artifacts.
I did a quick scan of the master branch and found the following files:
$ find . -type f ! -name "*.jpg" ! -name "*.png" ! -name "*.pdf" | perl -lne 'print if -B' | grep -v "./.git/"
./core/src/test/resources/keystore.jks
./processing/src/test/resources/v8SegmentPersistDir/00000.smoosh
./processing/src/test/resources/v8SegmentPersistDir/version.bin
./publications/whitepaper/modii658-yang.zip
./server/src/main/resources/static/old-console/images/favicon.ico
./docs/_graphics/druid.graffle
./examples/quickstart/tutorial/wikiticker-2015-09-12-sampled.json.gz
./indexing-hadoop/src/test/resources/test-segment/index.zip
./extensions-contrib/parquet-extensions/example/wikipedia_list.parquet
./extensions-contrib/parquet-extensions/example/284a0e001476716b-56d5676f53bd6e85_115466471_data.0.parq
./extensions-contrib/parquet-extensions/example/test_date_data.snappy.parquet
./extensions-contrib/time-min-max/src/test/resources/druid.sample.tsv.zip

However, I'm not sure how much of an issue this is:

I also scanned Apache's GitHub organization for such file extensions and found many existing examples (i.e. .jks):

https://github.com/search?utf8=%E2%9C%93&q=org%3Aapache+extension%3Ajks&type=Code&ref=advsearch&l=&l=

I couldn't find any written ASF guidelines on the matter which makes sense. The closest thing I found is RELEASE CREATION PROCESS: WHAT IS A VALID RELEASE PACKAGE?.

Once we figure out which the audit rules we can add this into the CheckStyle or to the integration tests.

@yurmix thanks for looking into this!

Once we figure out which the audit rules we can add this into the CheckStyle or to the integration tests.

Hm, deciding whether a non-code binary artifact should be allowed seems pretty subjective and context dependent to me, seems like a Checkstyle rule would be too blunt (My impression is that this really requires a manual case-by-case review).
e.g., would we forbid "zip" files? I think it's reasonable to include sample data for tests in a zip file.

I believe all of these files fall into the following categories:

Test data (either binary or packaged textual).
Binary content (Image files, PDFs).

I couldn't find any document which states that these types of files are not allowed in a source release (and I found multiple cases where such files exists in the GitHub Apache organization).
Therefore I believe we can close this issue.
Please review the list and see if you reach the same conclusion:

File name
Category

./core/src/test/resources/keystore.jks
Test data

./processing/src/test/resources/v8SegmentPersistDir/00000.smoosh
Test data

./processing/src/test/resources/v8SegmentPersistDir/version.bin
Test data

./publications/whitepaper/modii658-yang.zip
Binary content

./server/src/main/resources/static/old-console/images/favicon.ico
Binary content

./docs/_graphics/druid.graffle
Binary content

./examples/quickstart/tutorial/wikiticker-2015-09-12-sampled.json.gz
Test data

./indexing-hadoop/src/test/resources/test-segment/index.zip
Test data

./extensions-contrib/parquet-extensions/example/wikipedia_list.parquet
Test data

./extensions-contrib/parquet-extensions/example/284a0e001476716b-56d5676f53bd6e85_115466471_data.0.parq
Test data

./extensions-contrib/parquet-extensions/example/test_date_data.snappy.parquet
Test data

./extensions-contrib/time-min-max/src/test/resources/druid.sample.tsv.zip
Test data

Thank you @yurmix for looking into this. I agree with your conclusion, and think that it makes sense to go ahead with the RC including these files. If our mentors or the incubator PMC expresses any concerns, we can address them at that time. Thanks again!

Resolved (As per the feedback in comments).


Safari extension

Not really an issue, more of a request - are there any plans to offer a Safari version? That would help for Apple devices (computers and mobile devices).

Thanks, that's interessting, but that's not my question. The question was, is this project planning to offer a Safari extension (via the App Store)? Since Safari is the #2 browser in use (behind Chrome, far ahead of Firefox, Edge and Opera), it would be a shame to limit coverage to only a subset of consumer devices.

We tried this, but the problem is that Safari does not allow any of 2 way how we can collect cookie changes.

cookies.onChanged
change event of cookie store API

This is needed here.  Without that, CookieBlock in Safari would be just a bit smarter cookie remover when the user triggers the action. But even our features extraction is harmed, so with lower accuracy than other browsers.
If you know some other way to solve this, let us please know!


Can't archive with Xcode 8.2.1 and PromiseKit 3.5.3

PromiseKit version: 3.5.3
Xcode version: 8.2.1
CocoaPods version: 1.2.1

I created a fresh new Single Page Application and just added PromiseKit pod but I can't archive.

Dupe

I closed #669 in favor of this one.

I would have fixed this by now if I was provided with a test project. As it stands fixing it is otherwise a lot of work. And I already have a lot of work.

Thanks a lot for your attention. I updated my project to Swift 3 and it worked perfectly.

At this point I think this is too old to be worth fixing.


UTJS-2 Introduce InjectionContext

allow InjectionProvider to register a callback invoked when
the handler requesting the injection is removed from service
also fix CDIInjectionProvider (right now, CDIInjectionProviderTestCase does not pass in the master)

This looks a bit broken for two reasons:

handlerDiscardCallbacks  is not thread safe
injections should be scoped to the invocation not the life time of the handler. It looks like even if the list was thread safe you would be adding a new object to it for every request?

Hm, I thought the injection is only performed once (for the handler). But I was obviously wrong ;-)
Will try to rework this pull request then.


Added autumn pyladies course 2024 for Olomouc

@Kobzol @encukou  @frenzymadness
WDYT?

Thanks for preparing it. As always, we will probably need to make changes during the course, but that is fine. I have merge rights here, btw, so we can be "self-contained" in terms of modying our Olomouc materials :)


Add some basic testground tests

Here are the tasks breakdown:

[x]  Have a bunch of nodes each creating tons of records in their own threads, then make sure all nodes have the same view eventually. 1d
[x]  Same as the above, but have the records referencing each other nodes' records in different threads. 1d
[x]  Given an already fully synced network, make sure a newly joined node can be fully synced with the rest. 0.5d
[x]  Record the time metrics of each test and store them as the baseline for further improvements. 1d
[x]  Add variations of the plans to introduce slow / disconnected / failing intermittently nodes. 0.5d

👀

I re-ran the tests. The failing test from previous run looks like an intermittent thing that I thought was resolved in the past:
--- FAIL: TestModifiedSince (0.00s)
    --- FAIL: TestModifiedSince/WithCreateSaveAndDelete (1.19s)
Error:         collection_test.go:956: should have modfied id 01f65p2j6x7d6phffkzdbb2asd

cc @carsonfarmer if that rings any bells.

Just realized this was not merged. The failed test is not new, but will be taken care of as part of this sprint.

Oh cool. I just created an issue for that failure. https://github.com/textileio/go-threads/issues/520

I think I have a really simple fix for this. @merlinran do you want me to PR the fix here? Alternatively, I can just past the fix below and you can sneak it in yourself?

Either way works @carsonfarmer . If you paste here, I'll add it to my ongoing PR to improve testground tests reliability.

Cool, line 955 in db/collection_test.go (TestModifiedSince):
		if mods[0] != res[0] && mods[1] != res[0] {
			t.Fatalf("should have modified id %s", res[0])
		}

The ordering is arbitrary. The test was overly naive, that should catch either one being the correct value,

Done https://github.com/textileio/go-threads/commit/17873f261d501a135b33b898e0c9d01a1591f104


Unused API warning

I'm using the package up to date but I'm getting this warning, there's no problem with my app working but I just wanted to let you know.

Note: .......\Pub\Cache\hosted\pub.dev\google_mlkit_commons-0.5.0\android\src\main\java\com\google_mlkit_commons\InputImageConverter.java
uses unchecked or unsafe operations.

Note: ......Pub\Cache\hosted\pub.dev\google_mlkit_text_recognition-0.10.0\android\src\main\java\com\google_mlkit_text_recognition\TextRecognizer.java
 uses unchecked or unsafe operations.

bump, meet the same issue.

same here with google_mlkit_text_recognition package


Error in downloading from youtube - RTMPe

Hi,
I am not able to download https://www.youtube.com/movie?v=IYZEu01yy-0
Find attached screenshot of the details. I am using WinXP 32bit, SP3

Duplicate of #1797.


feat: Added redeploy confirmation if different app was already deployed using same api key

In this scenario, wouldn't it be simpler to use the absolute path of the folder that contains the application as a reference?
The only benefit of using app id is to support directory rename. If you go further in this direction, I would recommand to add it on the app provisionning or in the writer framework code, not in the deployment code.
alternate propsition that does not change the ui.json :

have a file deployments.json

{
  "<token>": "<absolute path of project>",
  "<token2>": "<absolute path of project 2>"
}

raise a warning when a token is already used for a path that is different from the previous one

Good to close this?


update dependencies

Coverage remained the same at 90.268% when pulling 74e4c448f9a42b027466986dad75f0928d4aff26 on feature/eslint-update into eb7cd966b16a1e3c92342e5007bcbbb5dbe19ac0 on master.


Uncaught InvalidCharacterError: Failed to execute 'add' on 'DOMTokenList': The token provided ('highlight-selected background') contains HTML space characters, which are not valid in tokens.

[Enter steps to reproduce below:]

...
...

Atom Version: 0.183.0
System: Mac OS X 10.10.2
Thrown From: Atom Core
Stack Trace
Uncaught InvalidCharacterError: Failed to execute 'add' on 'DOMTokenList': The token provided ('highlight-selected background') contains HTML space characters, which are not valid in tokens.
At /Applications/Atom.app/Contents/Resources/app/src/highlights-component.js:63

Error: Failed to execute 'add' on 'DOMTokenList': The token provided ('highlight-selected background') contains HTML space characters, which are not valid in tokens.
  at Error (native)
  at HighlightsComponent.module.exports.HighlightsComponent.updateHighlightNode (/Applications/Atom.app/Contents/Resources/app/src/highlights-component.js:63:33)
  at HighlightsComponent.module.exports.HighlightsComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/src/highlights-component.js:47:28)
  at LinesComponent.module.exports.LinesComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/src/lines-component.js:118:32)
  at TextEditorComponent.module.exports.TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:213:27)
  at /Applications/Atom.app/Contents/Resources/app/src/text-editor-component.js:283:28
  at ViewRegistry.module.exports.ViewRegistry.performDocumentUpdate (/Applications/Atom.app/Contents/Resources/app/src/view-registry.js:159:9)
  at /Applications/Atom.app/Contents/Resources/app/src/view-registry.js:3:61

Commands
     -3:03.6 pane:show-previous-item (atom-text-editor.editor.is-focused)
     -3:03.0 pane:show-next-item (atom-text-editor.editor)
     -3:01.9 pane:show-previous-item (atom-text-editor.editor.is-focused)
  3x -3:01.3 pane:show-next-item (atom-text-editor.editor)
     -2:59.1 pane:show-previous-item (atom-text-editor.editor)

Config
{
  "core": {
    "ignoredNames": [
      "tags"
    ],
    "disabledPackages": [
      "rails-transporter",
      "term2",
      "term2",
      "term2",
      "vim-mode",
      "atom-ctags"
    ],
    "themes": [
      "atom-dark-ui",
      "monokai"
    ]
  },
  "editor": {
    "fontSize": 13,
    "showIndentGuide": true,
    "autoIndent": false,
    "preferredLineLength": 120,
    "invisibles": {},
    "softWrap": true
  }
}

Installed Packages
# User
atom-color-highlight, v3.0.8
autocomplete-paths, v1.0.2
autocomplete-plus, v2.2.0
color-picker, v1.4.4
command-logger, v0.22.0
file-icons, v1.4.11
git-tab-status, v1.5.3
highlight-selected, v0.9.0
language-emblem, v0.1.0
language-haml, v0.15.0
linter, v0.11.1
linter-clang, v2.25.1
linter-coffeelint, v0.1.7
linter-csslint, v0.0.11
linter-erb, v0.1.0
linter-javac, v0.1.3
linter-jshint, v0.1.0
linter-ruby, v0.1.4
linter-scss-lint, v0.0.14
monokai, v0.12.0

# Dev
No dev packages

@nathansobo Is this a problem in the highlight-selected package? Seeing it for the first time in 0.183, so just want to double-check.

Yes. They are probably passing in 'highlight-selected background' as a className for some decoration, which is not allowed by element.classList.add. We could check for this, though, and throw a warning?

We should probably just support it by breaking the class on spaces.


Got error when work with Acumatica 2020 R2 Build 20.201.0051

Using the latest smart sheet code against  Acumatica 2020 R2 Build 20.201.0051， and after clicking get token and click allow in the pop up window, it gives error as below

Is current smartsheet integration works with this version acumatica?

Similar issue after publishing customization when user attempts to login. Acumatica 2020 R1 Build 20.109.0032

@kma77 I am unable to reproduce the error in a QA environment.
Would you have time to meet via Zoom some time this week?

Yes I’m available tomorrow, anytime. Just let me know what works for you.
Patrick Wills
Senior Consultant | ACC Software Solutions
535 Route 38, Suite 320, Cherry Hill, NJ 08002
856.335.1010 x103
https://www.linkedin.com/company-beta/1102619 https://twitter.com/ACC_Software https://www.facebook.com/ACCsoftwaresolutions/ https://www.instagram.com/accsoftware/ http://www.youtube.com/c/4accAccountingSolutions
<EMAIL_ADDRESS><EMAIL_ADDRESS>From: famadoz<EMAIL_ADDRESS>Sent: Monday, November 23, 2020 12:26 PM
To: Acumatica/Acumatica-Smartsheet<EMAIL_ADDRESS>Cc: pwills0926<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Acumatica/Acumatica-Smartsheet] Got error when work with Acumatica 2020 R2 Build 20.201.0051 (#5)
@kma77 https://github.com/kma77  I am unable to reproduce the error in a QA environment.
Would you have time to meet via Zoom some time this week?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/Acumatica/Acumatica-Smartsheet/issues/5#issuecomment-732307889 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AR2Q4V4UPRGTVXYKQS46ZADSRKLLNANCNFSM4TZLLKLA .  https://github.com/notifications/beacon/AR2Q4V7OHGUXGT4TL65YPGTSRKLLNA5CNFSM4TZLLKLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFOTCDMI.gif

@pwills0926 I will contact you via email

@famadoz  Sure please let me know a time, i should be available most time this week, please send me a meeting invite a day ahead, thanks. By the way i am using MySql database, do you think that matters.

@famadoz Can you please let me know what is a good time for a screen share session, thanks

@kma77 could you please share your email address to contact you over there? Thanks.

@kma77 could you please share your email address to contact you over there? Thanks.

@famadoz<EMAIL_ADDRESS>

Updated package that addresses this error is available in GitHub


[SR-839][Sema] Better fixits for optional expressions

In member ref expressions, if the base is optional, and the expected expression result is either optional or unknown, suggest a fixit that makes it into an optional chain expr rather than force unwrapping.
Since in many cases the actual fixit is emitted during diagnosis, and thus, while type checking sub exprs with no contextual type specified (so nothing to check for preferring optionality), we also need an additional flag to pass down from FailureDiagnosis for whether we prefer to fix as force unwrapping or optional chaining.
I attempted to do this same job via providing a convert type but setting the ConvertTypeIsOnlyAHint flag on the type checker, but unfortunately there are a lot of other moving parts that look at that type, even if it is only supposed to be a hint, so an additional flag to the CS ended up being cleaner.

@swift-ci Please test


Wilmawinterbottom patch 2

Website (if applicable): https://jackiechalarca.org <-- TODO: CHANGE ME
This PR is:

[x] Adding a new domain
[ ] Updating existing domain
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (darktheme.club site)
[ ] Other not listed

[x] The domain is in the correct alphabetical order
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ page, particularly the red section about inappropriate content, and I attest that this site is appropriate based on those criteria.

[x] Check to confirm

- domain: jackiechalarca.org
  url: https://jackiechalarca.org/
  method: mediaquery
  last_checked: 2023-09-28

Thanks!


[Feature Request] Custom placement for Select's dropdown

Existing Component
Yes
Component Name
select
Description
Custom placement for Select's dropdown

Duplicate of #13881


Support for Servant 0.17 and up

Servant 0.17 has changed the way NoContent works, which will probably require some changes on our end.

Sounds like it's not, cheers!

Would you be willing to raise/remove the servant upper bound and do a new release?


React-Leaflet v4 Support?

Hey there, does this plugin work with React-leaflet (specifically the latest version)?
I've been spinning my wheels trying to figure out how to get this to work.
Any ideas?
Thanks for your time!

@atipezda I noticed in the other issue you were using this plugin with React-Leaflet. Are you using React-Leaflet v4 for this? If so, any examples I could use as a base?

Hi @masonjwhite - did you manage to make it work? If so I'm quite interested.

No dice unfortunately. Ended up going in a different direction. I’ll keep you posted if I pick it back up though.
You mind doing the same?

@masonjwhite Trying to. Got something "encouraging" (I wanted to say "working" but that's a big word). Will keep you posted if I manage to make it really work. I need to better understand how the coordinate system works in that case.

Hi guys, i finally changed to Cesium library

I too have been trying to get custom non-geographic tiled maps to work using React-Leaflet and ain't gotten it to work yet (it keeps either requesting tiles that don't exist or just not working at all)


top level await

Hello,
The browser still does not like the top level await here:
https://github.com/bytecodealliance/jco/blob/785cf662c0d1503d37329d9f090532a2d7775d53/src/browser.js#L1-L3
What do you  think about something like this:
import { $init, generate, generateTypes } from '../obj/js-component-bindgen-component.js';

let initialized = new Promise((resolve, reject) => {
  $init.then(() => {
    resolve(true);
  });
});

export const transpile = function(component, options) {
  // if $init has not yet resolved, wait for it to resolve
  // before calling generate
  // if it has already resolved, pass args directly to generate and retun the result
  return initialized.then(() => {
    return generate(component, options);
  });
}

export { transpile, generateTypes }

The code above would move the await out of the top level, and push it to the user when they call transpile
let { files, imports, exports } = await transpile(wasmBytes, opts);

I've tried it out in my local repo and it seems to work better than the await $init (which I can't get to work at all)
Open to other suggestions though

If you to poke around, I made a branch with these changes:
https://github.com/DougAnderson444/jco/tree/no-top-level-await
Happy to consider feedback or make a pull request


Update header and footer links to reflect changes on Hugo site

[description]

Update header and footer links to reflect changes made in: https://github.com/cockroachlabs/hugo-website/pull/327

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/25bb5e1dffb6d666f75c5bda5e7d829f1861dae8/


Is there a way to convert to code generate by protobuf oneof

Have you read the project readme?

[x] Yes, but it does not include related information regarding my question.
[ ] Yes, but the steps described do not work.
[ ] Yes, but I am having difficulty understanding it and want clarification.

Describe your question
I'm trying to convert a pure Go struct to a struct generated using protobuf that include oneof. The generated code for oneof create an un-exported interface ref with multiple struct that implement the interface. I didn't find a way to use goverter to convert such a field. I tried using a custom Mapping Method but since the type is not exported by protobuf, it's not possible to define such method. One workaround to work with oneof is to create a type alias in the same module as the generated code. However, goverter match type based on there name ref so an alias wouldn't work. Is there a way that I missed to convert a struct that contains such a oneof?

Could you provide a minimal example which goverter cannot convert (input, output struct & goverter converter interface)?


Spotify Songs Recommendation System

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : Spotify Songs Recommendation System
:red_circle: Aim : Create a ML model which will recommend the songs as per the taste of the users.
:red_circle: Dataset : https://www.kaggle.com/datasets/corneliusjustin/spotify-songs-recommendationbinary-classification
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'SWOC' and 'JWOC' contributors of 'ML-Crate' project.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID (If not, then put NA) :
Approach for this Project :
What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Full name : Aryan Agrawal
GitHub Profile Link: https://github.com/aryanag7
Participant ID : NA
Approach for this Project : I would be using contest based filtering that determines the features which are most important for suggesting the songs.

Are you contributing here as a contributor or, as a part of any open source program?
@aryanag7

Hey, I'm contributing as a contributor .

That's great. Your approach looks pretty impressive. Issue assigned to you @aryanag7.
Best wishes!

Hey @aryanag7 updates regarding this issue.

Hey @abhisheks008 , I'm working on the last part of the implementation and will create  a PR within a week.


MoonZoon version + bug

Hi,

I've created a MoonZoon version: github.com/MoonZoon/vigenere-cipher
I've probably found a bug in the demo: try to write Martin K into it.

@rsdlt I've tried to contact you directly but mails fail with Address not found and Twitter messages are probably disabled.

Hi @MartinKavik thanks for bringing the bug to my attention, will review. Odd that my email failed, did you try<EMAIL_ADDRESS>Thank you.

Hi @MartinKavik, thanks again for raising the issue. It's now fixed in release v.0.1.2 Cheers!

Hi @MartinKavik thanks for bringing the bug to my attention, will review. Odd that my email failed, did you try<EMAIL_ADDRESS>Thank you.

I've just copied the only address with a typo... and it's a bit hard to notice it :)
(Feel free to remove this comment because it's a bit off-topic)


added more rules to gitignore (#43) and fixed typo in readme

Please review and merge.

closing - if still needed please submit appropriate subset.


Improve success rate of messages by sending them according to the spec

See links within doc blocks for details. But to summarize:

require userId OR email instead of a strict requirement on userId
key the lock using email or userId
format event metadata according to the docs

Probably needs some more testing.
@sperand-io @hankim813

wooo! looks awesome


docs: remove duplicate in README.md

the the End to End test allows to check from a browser (for example for web applications)part of the text was repeated twice

@Guillaume-N Hi ! Sorry for the delay. Thank you for your time and the correction ;-)


don't show color picker if colorful opts are disabled

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #android-bugs.

This occurs in a chat, and in the chat details

Fixed. My beta branch


Add (optional) VISSv2 interface

Add VISSv2 interface to databroker.
Gated by feature flag viss (not built by default).
Supports:

get, set, subscribe and unsubscribe
Authorization (using same jwt scope format as the other interfaces)

Not implemented:

Filters
TLS

Isn't it already implemented through https://github.com/eclipse-kuksa/kuksa-viss ?

Isn't it already implemented through https://github.com/eclipse-kuksa/kuksa-viss ?

Sure. But that requires running a separate service (written in python). Useful for demonstration purposes, but not very resource efficient or easy to make "production ready".
With that said, I haven't really evaluated the need for a VISSv2 service, just wanted to check the feasibility of doing it (and took it up as a fun weekend project :) ).
Conclusion is that it seems very feasible. And apart from the lower footprint, it would IMHO also be a lot easier to make production ready thanks to the type safety / thread safety provided by Rust.

Should we not add a short documentation somewhere. Like how to build it, how to test it. What limitations are there. Shall it for now be considered as "alpha" (see https://github.com/eclipse/kuksa.val/wiki/KUKSA.val-Component-Maturity), i.e. not suitable for production use? I.e. for now we will not test the interface as part of release testing?
How much have been tested? Have we for instance tested with kuksa-client (python) using websocket towards the Databroker VISS interface?

How much have been tested? Have we for instance tested with kuksa-client (python) using websocket towards the Databroker VISS interface?

It has been tested with kuksa-client. Feel free to give it a go.
Since kuksa-client isn't fully VISSv2 compliant (or compliant with any VISS version really?), a small change is needed in kuksa-client to support VISSv2 authorization (I have a local branch with that).

Should we not add a short documentation somewhere. Like how to build it, how to test it. What limitations are there.

If anyone is up for that, sure. This was just a "weekend project" which I think would be a useful addition to the project. But it is not a priority at the moment.
With that said, my impression is that it would - even in it's current state - be a better option for "supporting VISS" than the available alternatives.

It implements standards compliant VISSv2 (which I don't think is the case with kuksa-val-server)
It seems to be "more" standards compliant and feature complete than kuksa-viss referenced above.
It would be a more realistic thing to make production ready at some point.

And as the whole thing is gated behind a feature flag, one option would be to have it "hidden" for now, but easily enabled by cargo build --features viss for anyone who wants to. If we decide it would be a useful thing to have going forward, documentation etc can be added.
I do think it would be useful to merge though, as that would prevent the code from rotting.

I support merging. One of the problem with VISSv2 is the lack of (commonly used) reference implementations. One which possibly could be considered as the "official one" exists at https://github.com/w3c/automotive-viss2, and if VISSv2 gets higher priority it could be an idea to verify that "their solution" and "our solution" interprets the VISSv2 standard in the same way.

I am still not sure, what the "correct" way is to move forward here (in terms of maintainability/usage), but letting this rot would also be sad. Moving this behind a feature flag for now is a good first step. I think to merge this, there are few other things we need to do and should not do

Even when compiled (with feature flag), it should need to be enabled explicitly by command line argument and when that is enabled should print a warning along the lines of "You enabled VISS; this is an experimental feature"

The rationale here is, if we would - at a later point -  decide to include this in our default/released builds and something "is wrong", i.e. not secure, or can crash/confuse db, it would not be running by default. And it is a good idea anyway not unnecessarily exposing additional network ports

Write a small doc how to enable it and what VISS is supported: Not a "book", but I would expect something like "VISS get call, no history, no filters, no wildcards, and some example message (like in the VISS transport spec does)
Haven't checked here, but in general: Do NOT extend VISS. Have a ssubset of VISS, not something that somehow "overlaps", i.e. we do not need to be able to do providers (i.e. set sensors) in VISS. Let's keep this an "application" protocol as god W3C Ulf intended it to be

Even when compiled (with feature flag), it should need to be enabled explicitly by command line argument and when that is enabled should print a warning along the lines of "You enabled VISS; this is an experimental feature"

That's how it's implemented now (except for the warning). Even if the viss feature is enabled, no "VISSv2" code will run unless --enable-viss is also supplied as an argument at startup.

Haven't checked here, but in general: Do NOT extend VISS.

The code is strictly following the standard (barring bugs). And I agree that it should continue to do so.

Write a small doc how to enable it and what VISS is supported

That is missing, but I can add it.

Rebased.
Added documentation.
Included VISSv2 support for kuksa-client (which lived in a separate branch previously).
VISSv2 in kuksa-client is only enabled if the negotiated subprotocol is VISSv2. If no subprotocol is negotiated, it reverts to it's previous behaviour in order to remain backwards compatible with kuksa-val-server which isn't compatible with VISSv2 (and doesn't negotiate a subprotocol).

@erikbosch  Can you check this (again). Also I assume we want to merge it fast, but after "release"/tag?

Also I assume we want to merge it fast, but after "release"/tag?

Before or after release tag shouldn't matter as it's not included in the release artifacts anyway.


Babelyoda chokes on CFBundleDisplayName in InfoPlist.strings

It is completely eligible to add following lines to your InfoPlist.strings file:
/* Localized app name for iOS home screen */
CFBundleDisplayName = "My l10n app";

Both iOS and OS X recognizes such a text and handle them correctly. However Babelyoda fails with pretty obscure error message while trying to process this file.
I think Babelyoda should support this type of text and handle it correctly.

Closing as the project is being archived.


Disruptor batching v2

I wanted to come up with an alternative to #694 that does batching at the queue level instead of at that spout level.  This is the result of that work, although it still needs some testing/cleanup before I consider it fully ready. This is based off of #750 using the latest version of disruptor, and has a few bug fixes for automatic back-pressure that I need to split out and include as a separate pull request.
The work here is driven by results I got from some micro benchmarks I wrote looking at what the bottlenecks are in storm and what the theoretical limit is for storm.  In this case doing a simple word count topology as represented by Q_SWC_X_04.  With the same batching here the throughput when from 300,000 sentences/second processed to around 1.5 million, as run on my Mac Book Pro Laptop.
https://github.com/revans2/storm-micro-perf
The reason for queue level batching is that it gives a finer grained control over latency, which I would like to use for automatic tuning in the future.  It also is a smaller code change.  If others disagree I am okay with #694. But I do want to do a head to head comparison between the two approaches.

@revans2 this looks great . I am in favor of batching at disruptor. So this means we don't have any issues with acking?

I am just rebasing the code now so you can test that out yourself.  This code has no issues with acking.  But there are a few real issues.
First the latency on low throughput queues is much higher.  This is because it has to wait for the batch to time out.  That timeout is set to 1 ms by default, so it is not that bad, but we should be able to do some on the fly adjustments in a follow on JIRA to dynamically adjust the batch size for each queue to compensate.
Second the number of threads used is a lot more.  1 more per disruptor queue.  I expect to reduce the total number of disruptor queues once I have optimized other parts of the code.  As it stands right now I don't want to do that, because the two queues per bolt/spout design still improves performance in many cases.
Third in the worst case situation it is possible to allocate many more objects than previously.  It is not actually that many more, we already allocate a lot of objects, which needs to be looked at on a separate JIRA at some point.
Also I don't want to shove this code in without doing a real comparison between the two approaches and the code.  This is one way of doing batching, but there are others that may have advantages over this, or may compliment this approach as well.  I just want storm to eventually be a lot closer to the 1 million sentences/second mark than it is now.

Curious to see first performance results :)

@mjsax The difference is not that huge yet.  It is 2x in a number of use cases.  But we have a lot of slowness in the metrics processing code, which, at least in the case of word count, is dominating the performance, which is probably why you were not seeing a 6x performance improvement in your pull like you were seeing before.  I am going to be doing some follow on optimization work on that.  The big thing right now is that the System CPU utilization went from 15-30% depending on the OS, etc. to 2-6%.  This frees up a huge amount of CPU for other processing, so the overall throughput capability of a node increased significantly.

Really exiting work! It is very interesting that we move the overflow buffer inside the disruptor queue and use it for batch publishing purpose. I would also look forward to seeing how tuning the disruptor queue size, batch interval will impact the throughput of topologies with different tuple-sending/processing speeds.

Ran some sanity tests. LGTM. +1

I have some new test results.  I did a comparison of several different branches.  I looked at this branch, the upgraded-disruptor branch #750, STORM-855 #694, and apache-master 0.11.0-SNAPSHOT (04cf3f6162ce6fdd1ec13b758222d889dafd5749).  I had to make a few modifications to get my test to work.  I applied the following patch https://gist.github.com/revans2/84301ef0fde0dc4fbe44 to each of the branches.  For STORM-855 I had to modify the test a bit so it would optionally do batching.  In that case batching was enabled on all streams and all spouts and bolts.
I then ran the test at various throughputs 100, 200, 400, 800, 1600, 3200, 6400, 10000, 12800, 25600. and possibly a few others when looking for it to hit the maximum throughput, and different batch sizes.
Each test ran for 5 mins.    I don't really like that test, even though I wrote it, because the throughput is limited only by storm, so with acking disabled it is measuring what the latency is when we hit the wall, and cannot provide any more throughput.  No one should run in production that way. When acking is enabled and we are using max-spout pending for flow control the throughput is directly related to the end to end latency.  This too shouldn't be the common case in production because it means we cannot keep up with the incoming rate and are falling behind.
This seems to indicate that the only time STORM-855 makes since is when looking at the 99%-ile latency at a very low throughput, and then it only seems to save 1/20th of a ms advantage over the others.  In other cases it looks like the throughput per host it can support is about 1/2 of that without the change.  This branch however has a weakness on the low end when batching is enabled it is about 12 ms slower, but on the high end it can handle more then 2x the throughput with little change to the latency.  If that 12 ms is important I think we can mitigate it by allowing the batch size to self-adjust on a per queue bases.
I really would like others to look at my numbers and my test to see if there are issues with it that I am missing, because like I said it seems to contradict the numbers from STORM-855.  The only thing I can think of is that the messaging layer is the bottleneck in the speed of light test, which is what it was intended to stress test, and STORM-855 is giving a significant batching advantage there.  If that is the case then we should look at what STORM-855 is doing around that to try and combine it with the batching we are doing here.
@ptgoetz @d2r @rfarivar @mjsax @kishorvpatil @knusbaum please let me know what you think.

Sorry I forgot to mention that for one test STORM-855-100 was able to run at a throughput of 6000 sentences/second successfully, but in other runs it failed, so I set the number to 5500, even though a 6000 is shown in the results.

I just rebased on the latest fixed disruptor upgrade code, now in pull request #797.
The critical path did not change at all, so the numbers should be similar.

I had a look at the numbers. It's a lot of stuff and hard to parse for me... I am not sure what you mean by "contradict the numbers"? Can you explain in more detail?
However, I agree with your thought about STORM-855 #694 It basically reduces the contention on the output queue, because less calls are made here. I profiled Storm once and observed that on high-end data-rates (when we hit the wall) the contention on the output-queue is the bottleneck (the writing and reading thread have to lock the queue and a lot of waiting for the lock consumes a fair share of the consumed time).

@mjsax
What I saw when testing STORM-855 was that the maximum throughput was cut almost in half from 10,000 sentences per second to 5,500.  But your numbers showed maximum throughput more than doubling from around 7,960,300 tuples sent in 30 seconds to 16,347,100 in the same time period (no-acking).  And 1,832,160 in 30 seconds to 2,323,580 an increase of 25% with acking.
To me this feels like a contradiction. The only thing I can think of is that the messaging layer is so scary slow that cutting the maximum throughput of a worker by half has no impact on the overall performance if it can double the throughput of the messaging layer, by doing more batching.
This is likely the case, as on the high end 16,347,100 / 30 seconds / 24 workers is about 22,000 tuples per second per worker, where as 5,500 sentences per second results in about 181,500 total tuples per second/worker being processed.
I'm just looking for feedback from others on this, but it looks like I need to do a distributed apples to apples comparison as well to see the impact the messaging layer has.

Thanks for clarification. :)

So I have been running a number of tests trying to come to a conclusive decision on how storm should handle batching, and trying to understand the difference between my test results and the test results from #694. I ran the word count test I wrote as a part of #805 on a 35 node storm cluster.  This was done against several different storm versions, the baseline in the #805 pull request; this patch + #805 (batch-v2); and #694 + #805 + modifications to use the hybrid approach to enable acking and batch to work in a multi-process topology (STORM-855).  To avoid having all of the numbers be hard to parse I am just going to include some charts, but if anyone wants to see the raw numbers or reproduce it themselves I am happy to provide data and/or branches.
The numbers below were collected after the topology had been running for at least 200 seconds.  This is to avoid startup issues like JIT etc.  I filtered out any 30 second interval where the measured throughput was not +/- 10% of the target throughput on the assumption that if the topology cannot keep up with the desired throughput or it was trying to catch up from previous slowness it would not be within that range.  I did not filter based off of the number of failures that happened, simply because that would have resulted in removing all of the STORM-855 with batching enabled results.  None of the other test configurations saw any failures at all during testing.

This shows the 99%-ile latency vs measured throughput.  It is not too interesting except to note that batching in STORM-855 at low throughput resulted in nothing being fully processed.  All of the tuples timed out before they could finish.  Only at a medium throughput above 16,000 sentences/second were we able to maintain enough tuples to complete batches regularly, but even then many tuples would still time out. This should be able to be fixed with a batch timeout, but that is not implemented yet.
To get a better view I adjusted the latency to be a log scale.

From this we can see that on the very low end batching-v2 is increasing the 99%-ile latency from 5-10 ms to 19-21 ms.  Most of that you can get back by configuring the batch size to 1, instead of the default 100 tuples.  However, once the baseline stops functioning at around 7000 sentences/sec the batching code is able to continue working, with either a batch size of 1 or 100.  I believe that this has to do with the automatic backpressure.  In the baseline code backpressure does not take into account the overflow buffer, but in the batching code it does.  I think this gives the topology more stability in maintaining a throughput, but I don't have any solid evidence for that.
I then zoomed in on the graphs to show what a 2 second SLA would look like

and a 100 ms SLA.

In both cases the batching v2 with a batch size of 100 was able to handle the highest throughput for that given latency.
Then I wanted to look at memory and CPU Utilization.

Memory does not show much, the amount of memory used varies a bit from one to the other, but if you realize this is for 35 worker processes it is varying from 70 MB/worker to about 200 MB/worker.  The numbers simply show that as the throughput increases the memory utilizations does too, and it does not vary too much from one implementation to another.

CPU however shows that on the low end we are going from 7 or 8 cores worth of CPU time to about 35 cores worth for the batching code.  This seems to be the result of the batch flushing threads waking up periodically.  We should be able to mitigate this by adjusting that interval to be larger, but that would in turn impact the latency.
I believe that with further work we should be able to reduce that CPU utilization and the latency on the low end by dynamically adjusting the batch size and timeout based off of a specified SLA.  At this point I feel this branch is ready for a formal review and inclusion into storm, the drawbacks to this patch do not seem to out weigh the clear advantages to it.  Additionally with the stability problems associated with #694 I cannot feel good in recommending it at this time.   It is clear that some of what it is doing is worthwhile, and I think we should explore the alternative batch serialization between worker processes as a potential stand alone piece.
@d2r @knusbaum @kishorvpatil @harshach @HeartSaVioR @ptgoetz
If you could please review this remembering that it is based off of #797, I would like to try and get this in soon.

Great analysis!
I would like to run this tests, too. Especially to work on the flushing of batches.  To get a fair comparison, I would like to fix this, and redo the test.
Btw: I am still wondering, you #694 behaves so badly with low throughput. From a conceptual point of view there is no reason for this. Maybe my code is not good enough (as I am a Clojure newbe). Hope I can reproduce your result and get better insight into the problem.

@mjsax If you want to take some time to debug things for a better comparison I am happy to wait. It is entirely possible that I somehow messed things up when I was enabling hybrid-mode so getting and apples to apples comparison is important.  I want the best final solution no matter what it is, but I also don't want to wait forever.  For efficiency and cluster utilization purposes I really would like to pull in batching into the clusters that I run, but because the configuration/interfaces are incompatible with one another between the two approaches I don't want to do it until open source has decided on a direction they want to go in.
In the mean time I am going to be working on improving the metrics gathering.  You said previously that with your first batching attempt you saw up to 6x the throughput.  I suspect a lot of that is because the metrics are so very very slow.

I am not sure how the metric relate to it... Furthermore, I also used Aeolus on the current master and get 6x improvement (with acking disabled -- acking is currently not supported by Aeolus).

@mjsax At least in my profiling the next largest consumer of CPU in word count, without acking enabled, after adding in batching is the metrics.  They are horribly slow.  I assumed that because Aeolus was only counting batches, not individual tuples that it mitigated a lot of this slowness.

Make sense. It might be worth explore counting in batches... (ie, increase a counter by "batch-size" for each batch). So we still get the correct value. Of course the metric will not increase slowly over time but "jump" in larger intervals. Not sure if this i acceptable behavior and/or what other metrics are there that cannot be updated in a "batch" fashion... count might be a too simple example.

Just rebased.

@d2r @knusbaum @mjsax  @harshach @HeartSaVioR @ptgoetz
Now that #797 is in. We should consider pulling in batching solution. Can you please review  this PR and pass on your opinion?
I am still +1 on this patch.

I did not find time to rerun the tests and fix #694. If you want to move on with this PR, just go ahead... I will still try to get #694 in better shape --- still hope I can get it accepted. (If you are still interested.)

@mjsax I am still very interested in #694 if it makes thing better.  I am skeptical that it will add much over this does because they both cover similar areas, but if it does then we should try to use it.

ran few topologies with acking and without. I am +1 on merging this in.


Fix coping of font directory

Fonts aren't getting copied into dist.

Fixed: https://github.com/bmarshall511/frontend/commit/f7a3dba5a05e451f4af845678dda70609f464779


style adjustment for message type

currently this method only provide alert box with class="alert alert-success alert-dismissable" even the $sessionName is 'info','danger' or 'warning' (base on 4 alert box type on Bootstrap). With little change than this method can show alert box with class="alert alert-info alert-dismissable" , "alert alert-danger alert-dismissable" or "alert alert-warning alert-dismissable" depend on giving $sessionName parameter

@cahyadsn Maybe you should also check for the type of alert, if it's set or not. And also add the
role="alert"

to the end of the alert div, just as it should be according the docs of bootstrap.

the problem with this approach you loose the ability to have a custom message in the alert div. The $sessionName needs to be one of the 4 types to apply the color but then it's also used to show inside of the div results in cases like this:
$msg = Session::pull($sessionName');
if (!empty($msg)) {
    return "

            <button type='button' class='close' data-dismiss='alert' aria-hidden='true'>×</button>

 success

";
}

will output:

    <button type='button' class='close' data-dismiss='alert' aria-hidden='true'>×</button>

 success

actually realised I was wrong!


Add whitespace trimming for argument parse workaround

Closes #2147.

Closing in favor of the more comprehensive fix in #2151.


Fix: Accessibility div not positioned correctly

Problem
Accessibility div does not position correctly on a webpage with other positioned elements than just Pixijs canvas.
Proposed Solution

As getBoundingClientRect() returns the coordinates relative to the viewport, not the closest positioned element. Accessibility div position should be fixed since we are using getBoundingClientRect() to position it relative to the canvas.

@bigtimebuddy need any help on this?

@GoodBoyDigital is the original author, and a user of this feature. I'd like to get his review before we merge.
That being said, it makes sense to me!

Closing this because of no reviews


Unable to confirm slots due to rush

Hi ,
Can we modify this code to make it automatically book the slots , any location , any time , any date?
Regards,
Prasanth

It is possible but that would have far reaching consequences

Thank you for responding , If you are talking about only distance, i am ok , since i am already selecting district as BBMP, i dont mind travelling within the BBMP limits, also the slots are not opening up everywhere, only in few places for 18- 44 at a time.

I understand that. There's also the problem of how do I test and make sure this works - this is all live. I can try to provide a "beta" version in a separate branch.

Agree to your point, separate branch make sense

This branch has the feature: https://github.com/pallupz/covid-vaccine-booking/tree/feature/auto-book
I tested it and it does work - books first available center, date, and slot. Please do let me know if you run into any issues.


iotedge stored system

Hi everyone
I want answer to some questions.

How do store data in iotedge?
queue? blob? Variables inside the code? log file?

Where is the view my stored data

Thank you for reading.

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

ID: 6f8e1616-f3c9-ba1c-2ebd-9d7f74d79005
Version Independent ID: 3158bd35-4d7d-63ec-1df7-520f00c053c0
Content: Operate devices offline - Azure IoT Edge
Content Source: articles/iot-edge/offline-capabilities.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@LeeHyeongWon Thank you for your feedback. Could you link the URL of the documentation you were following? That way, we can pass your feedback to the appropriate content author. Also you might want to read this doc Azure IoT Reference Architecture 2.1

@Alberto-Vega-MSFT
https://docs.microsoft.com/en-us/azure/iot-edge/offline-capabilities#optional-offline-settings
what is stored "/iotedge/storage/"?
iotedge logs edgeHub < command in linux
2019-04-19 05:33:40.865 +00:00 [INF] - Updated the edge hub store and forward configuration
↑ what does store in this log?
How to store iot edge messages? (ex. queue, blob, log file, etc...)

@LeeHyeongWon Thanks for sharing the link to the corresponding document. Now to answer your question:
what is stored "/iotedge/storage/"?
You can store the messages your devices generate when they are offline.
Please read the Azure IoT Reference Architecture  document. You can download it here http://aka.ms/iotrefarchitecture if should give you a good idea on how to architect your solution. With that said the documentation issues are meant for documentation feedback like errors and improvements. These questions you posted are better suited for the developer forums like MSDN  and StackOverflow (where there is a community that can help answer).
Please open a question on the developer forums. Just make sure to add more details, having more details on what you are trying to build will help the community to help you.

@LeeHyeongWon Was the information useful?


fix(scripts): fix graphdef formater

What type of PR is this?
/area model
/kind bug

What this PR does / why we need it:
fix graphdef formater
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @FogDong @judgeeeeee

Does this PR introduce a user-facing change?:

fix graphdef formater

ping @judgeeeeee

ping @FogDong

/lgtm


Flag for outputting IRs

Following the recently merged (#294) flags --build and --compile, I would really appreciate having a compiler flag for outputting the generated intermediate representations (as is done here on the website).

What do you typically use this for and why does the vscode plugin / lsp not suffice?

I personally typically use this for examination of the evidence inferred by the compiler (e.g. to see whether things are implemented correctly or whether there is potential for optimization), but I think it is generally interesting to be able to inspect IRs for any compiler, i.p. for people developing compilers themselves to see how other compilers work internally.
I do not use vscode and I generally dislike the idea of having a feature like that tied to a specific editor, because I deem this a feature of the compiler itself. Language server support might be an option, but I do not really know how I can implement this e.g. for vim (what kind of request do I have to make?), so that still hinges on having language-server support for Effekt with this feature implemented in the editor used.
On the other hand, if it is a lot of work, it might not be worth it, as the audience appreciating this might be too small.

It shouldn't be too hard since the functionality is already there for VSCode...
https://github.com/effekt-lang/effekt/blob/cf3b7d1fa7eda3a92f74351d99fcd3de159c900f/effekt/jvm/src/main/scala/effekt/Server.scala#L41-L84

Maybe that could be a good starter issue for @marvinborner ? What do you think?

Yes, sounds like a good issue to get to know the structure of the compiler.
Concerning the name of the flag, I like showIR as in the vscode plugin. Maybe the compiler could then be called with something like effekt --showIR=core foo.effekt for thecore IR and similarly for lifted and potential others.

Should it be shown it written to a file?

I think writing to stdout is fine as long as no other things, e.g. warnings (which I think should rather be written to stderr btw.), are written there. Then one can pipe the stdout to a file if desired.

IMHO outputting to a file might be better. 🤔 Then, we could also support outputting multiple IRs in a more useful way (which can be helpful since, e.g., IIRC some symbol names change between compilation runs).
However, this can also be extended/changed in a later issue.

You mean something like outputting different IRs to different files along with some additional information? That indeed sounds a bit more complicated and might be worth its own issue.

I think it is not very difficult (I would guess 20-30 lines of code for everything). For the VSCode extension, I already generate separate files with file pseudo file extensions like

The VSCode extension even syntax highlights them :)

But I guess there is no additional information (like how symbol names have changed, or something), or is there?
So, there could either be a switch --showIRs outputting the different IRs into different files or a switch --showIR= which outputs a specified IR either into a file or to stdout (writing to stdout is probably a bit more unix-like). An advantage of the former option is that the user does not have to know the names of the IRs in advance. But of course that could be easily made available in --help, so I think either option (or even having both) is fine.


QT API timeout/disconnect issue

Hello team,
I have been running the API for a couple weeks now and it works fine; however, it seems to time out after a couple days or only after 1 or 2 days.
There is no exact interval of disconnect/timeout period. Please see the error below. I have to go into APP HUB and refresh the token and input into code then it starts working again. Is there a way to avoid this timeout issue?
Thank you.
Traceback (most recent call last):
File "/home/compadmin/Documents/PYTHON/QT_APP_01/API.py", line 6, in
print(q.time)
File "/home/compadmin/.local/lib/python3.10/site-packages/questrade_api/questrade.py", line 81, in time
return self.__get(self.config['API']['time'])
File "/home/compadmin/.local/lib/python3.10/site-packages/questrade_api/questrade.py", line 49, in __get
r = urllib.request.urlopen(req)
File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.10/urllib/request.py", line 519, in open
response = self._open(req, data)
File "/usr/lib/python3.10/urllib/request.py", line 536, in _open
result = self._call_chain(self.handle_open, protocol, protocol +
File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
result = func(*args)
File "/usr/lib/python3.10/urllib/request.py", line 1391, in https_open
return self.do_open(http.client.HTTPSConnection, req,
File "/usr/lib/python3.10/urllib/request.py", line 1352, in do_open
r = h.getresponse()
File "/usr/lib/python3.10/http/client.py", line 1375, in getresponse
response.begin()
File "/usr/lib/python3.10/http/client.py", line 318, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.10/http/client.py", line 287, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response
compadmin@VM01:~/Documents/PYTHON/QT_APP_01$ /bin/python3 /home/compadmin/Documents/PYTHON/QT_APP_01/API.py
Traceback (most recent call last):
File "/home/compadmin/Documents/PYTHON/QT_APP_01/API.py", line 6, in
print(q.time)
File "/home/compadmin/.local/lib/python3.10/site-packages/questrade_api/questrade.py", line 81, in time
return self.__get(self.config['API']['time'])
File "/home/compadmin/.local/lib/python3.10/site-packages/questrade_api/questrade.py", line 49, in __get
r = urllib.request.urlopen(req)
File "/usr/lib/python3.10/urllib/request.py", line 216, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.10/urllib/request.py", line 519, in open
response = self._open(req, data)
File "/usr/lib/python3.10/urllib/request.py", line 536, in _open
result = self._call_chain(self.handle_open, protocol, protocol +
File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
result = func(*args)
File "/usr/lib/python3.10/urllib/request.py", line 1391, in https_open
return self.do_open(http.client.HTTPSConnection, req,
File "/usr/lib/python3.10/urllib/request.py", line 1352, in do_open
r = h.getresponse()
File "/usr/lib/python3.10/http/client.py", line 1375, in getresponse
response.begin()
File "/usr/lib/python3.10/http/client.py", line 318, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.10/http/client.py", line 287, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

This is likely caused by Questrade's REST API being unavailable for a short time.


!! Emergency !! Smapp versions newer than 1.2.7 are unstable on Ubuntu

Environment

Smapp version: 1.2.14 , 1.2.13, 1.2.12, 1.2.11, 1.2.10, 1.2.9
OS: Ubuntu 20.04

Describe the bug
Newer than 1.2.7 are unstable on Ubuntu.
OS hang-up = 1.2.14
SQLite error = 1.2.12(This is occurred Windows 10 too)
Steps to reproduce
Try to narrow down your scenario to a minimal working/failing example. That is, if you have a big program causing a problem, start with deleting parts not relevant to the issue and observing the result: is the problem still there? Repeat until you get the most straightforward sequence of steps to reproduce the problem without any noise surrounding it.

Install 1.2.14 , 1.2.12, 1.2.11, 1.2.9 Smapp.
Wait to green lamp.
When yellow lamp to green lamp about "SMESHING", this is happen.

Expected behavior
Just work fine.
Actual behavior
Newer than 1.2.7 are unstable on Ubuntu.
OS hang-up = 1.2.14
SQLite error = 1.2.12(This is iccurred Windows 10 too)
Logs
This is 1.2.14 case.
[Uploading spacemesh-log-7c8cef2b.txt.
syslog.zip
spacemesh-log-7c8cef2b 2.txt.zip
zip…]()
How to fix
If I downgrade to 1.2.7, this is work fine.
So all newer than 1.2.7 version has problem for Ubuntu Smapp.
I strongly wish to fix this until next round because of this announcement https://discord.com/channels/623195163510046732/691258865861394432/1187731532706754645

It's not valid anymore.


Automatic English locale generation

Closes #6
Features
Adds a script which can be run with npm run build:locales. This scans all mcdoc files and writes to files: /locales/keys/en.json and /locales/docs/en.json.
Tasks

[x] Use Spyglass to parse all .mcdoc files
[x] Process the ASTs to list all keys and all doc comments
[x] Transform keys to human readable strings
[x] Create GitHub workflow to run the script and commit on push to main
[x] Tested the workflow in private repo

Future work

Allow overwriting keys English locale
Detect field renames by comparing the previous locale

Superseded by mcdoc-cli


[🐞] Importing image as a component and passing it as a prop causes error in certain cases. "Error: Code(3): Only primitive and object literals can be serialized"

Which component is affected?
Qwik Runtime
Describe the bug
Doing:
The real use case is the same as this issue I opened, but this is a different bug so I opened a new one.
In the real use case, I have a list of cards with images on them and I would like to utilize Qwik image optimization so I need to import images as components and pass them in a card component. Preview use case here: https://linkfang-portfolio.vercel.app/web-apps/
Repro is simplified to just an element with a button.
Expect:
I expect the web app will run properly without crash.
Happening:
The web app will crash when I:

Import the image as a component
Pass it as a prop into another component (card component)
In card component, including an event handler that will change a boolean signal
In card component,  put the image prop into a ternary

With all 4 conditions above, the app will crash.
In the repro,

Passing imported image component as prop won't cause an issue;
Using ternary with a non-imported image component won't cause the issue.
Using imported image component inside the desired component (not passing as a prop) in a ternary won't cause the issue.
Using imported image component and pass it as a prop and use it with ternary BUT without any event handler that will change the signal won't cause the issue.

Reproduction
https://stackblitz.com/edit/qwik-starter-vif4tk?file=src%2Froutes%2Findex.tsx
Steps to reproduce
Open the repro link should automatically run npm i & npm run dev, when everything is up and running,

Click on the 2 buttons on the page to see everything is working fine
Uncomment the code in line 70, then the app will crash with error:

QWIK ERROR Error: Code(3): Only primitive and object literals can be serialized

System Info
System:
    OS: Windows 11 10.0.22621
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 6.10 GB / 15.60 GB
  Binaries:
    Node: 20.9.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.1.0 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.12.1 - ~\AppData\Local\pnpm\pnpm.EXE
  Browsers:
    Edge: Chromium (120.0.2210.91)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @builder.io/qwik: ^1.3.1 => 1.3.1
    @builder.io/qwik-city: ^1.3.1 => 1.3.1
    @builder.io/vite-plugin-macro: ~0.0.7 => 0.0.7
    undici: ^5.26.0 => 5.28.2
    vite: ^5.0.6 => 5.0.10

Additional Information
No response

Fixed with Qwik v2 🚀


Simplify Project.

I "simplify" the class out of existence, preparatory to reading it all from the dotfiles.

(And this reason for this pull request seems a little obscure - I pulled it from a longer chain to make reviewing easier.  The exact point will be clearer, later...)


Add v0.6.0 to CHANGELOG

This adds v0.6.0 to CHANGELOG.md and fixes "Unreleased" to point to unreleased changes again.

Need to find a way to update this CHANGELOG automagically


sbt 1.0 support

Could you guys publish plugin for sbt 1.0 and 1.0.3 ? I tried this
[error] (*:update) sbt.librarymanagement.ResolveException: unresolved dependency: org.foundweekends#bintray-sbt;0.5.1: not found

The name was changed. It's not bintray-sbt anymore, it's sbt-bintray.


images readed

read the images

hi @ashwanisng its better if you use matplotlib for showing input  image.

images read and used matplotlib to show input images