feature: add support to linux arm/v8

Hey team,
Tried installing Surface on a M1 chip and got this error:
github.com/surface-security/surface ‹main› » docker-compose -f dev/docker-compose-in-a-box.yml up
[+] Running 0/11
 ⠼ mysql Pulling                                                                                2.5s
 ⠼ nginx Pulling                                                                                2.5s
   ⠋ 3aa4d0bbde19 Downloading    [======>                                            ]    392kB/2.813MB                                                                                1.0s
   ⠋ 91a0d98ef348 Downloading    [>                                                  ]  73.62kB/7.341MB                                                                                1.0s
   ⠋ acdff1cb5e67 Download complete                                                                                1.0s
   ⠋ dfe2157c7506 Waiting                                                                                1.0s
   ⠋ 0cddbe0663dc Waiting                                                                                1.0s
   ⠋ 2410dd56c3ee Waiting                                                                                1.0s
   ⠋ 9801baac913b Waiting                                                                                1.0s
   ⠋ c8ddb12bc806 Waiting                                                                                1.0s
   ⠋ 025fd5a407e0 Pulling fs layer                                                                                1.0s
no matching manifest for linux/arm64/v8 in the manifest list entries

Both nginx and MySQL images support arm/v8 officially, so I thought we needed to upgrade our ghcr. However, I am not sure if we should drop arm/v7 or if we should maintain support for it. Is there anything using it or has everything been replaced with arm/v8? In fact, neither nginx or MySQL refer to v7, so this change should probably be a drop-in replacement instead.
Cheers,

v7 is no longer maintained in many projects.
Pi 3 (6 years old) is already v8 and I think most cloud providers with arm are already on arm64 as well.
But until it starts being an issue maybe it's not worth removing


Jain does not have an icon

The Jain religion lacks an icon

Here the fix, adding a new icon.
http://www.mediafire.com/file/exg8b4hpzwbn795/jain.rar

This can be closed, the converter gives Jain its icon.


Inverted colour should update on colour change

When claiming/changing colour the selected square's inverted border is not updated to reflect the new inverted colour

Good catch. I have fixed this in commit 818d32394ec5bb77c73c7c17924669cf5bb76583.
The problem was that the jquery sets the border colour once, and does not update when square colour changes. You could make a new jquery call when colour changes, but that seemed overkill. Rather, I eliminated the need for jquery by implementing the functionality in gridItem using styled components.
In general,styled-components are pretty powerful: you can pass props to a component which determine its style. I think eventually we should try eliminate all instances of jquery, and try implement the same functionality in styled-components.
Closing this issue though because the bug is fixed now.


Better defensive array check for tasklist type guessing

Now handles empty arrays. Should fix #272, though I've not tested it fully all the way through myself. @quadrophobiac can you handle that as part of your review?

Can confirm that this fixes the tasklist condition that caused the initial stack trace and that it doesn't impact extant tests


Add CompletionItem with snippet support to the example

Many users are asking about snippet support in Monaco. Enhancing the example on in the playground with a snippet CompletionItem will help users learn about the feature more easily without having to dig through Stack Overflow and GitHub issues.

@alexdima You asked me to make a PR many moons ago. Well, here it is!

Thank you! I really appreciate your help :heart: !


Species Type: Suggestion for Ion Syntax

The species type extension in the section for atoms notes:
"Any extension using this standard can define how to specify the charge state."
Would it be possible/advised to add such a definition?
I recon it is a somewhat common usecase, and would be justified in a standard.
I'd like to suggest the following notation:
Examples: He^2+, H^+, Cl^-, O^2-, #3He^2+
Generalized: atom "^" *DIGIT ( "+" / "-" )
Rationale:

use a delimiter after the atom to specifically denote charge -- I'm not sure that ^ is a good idea, maybe another character is better suited (_, /, |, :?)
allow omission of amount if it is 1
could be used with molecules too (see below)

I'm not aware how existing implementations solve this, maybe copy from them instead?

"Any extension using this standard can define how to specify the charge state."

Yes, this is to avoid confusion and determine which parts have precedence.
For instance, the ED-PIC convention has a clear meaning of charge, we then only add extra meaning with the SpeciesType extension to clarify that something is a Hydrogen or Carbon isotope.

Would it be possible/advised to add such a definition?
I recon it is a somewhat common usecase, and would be justified in a standard.

Unless we have a concrete chemical use case, I would not specify this yet. This is also the reason why the "Molecule" section is intentionally sparse for future use.
The reason behind this is the following: currently, most our use cases are in laser-plasma physics, astrophysics, high-energy physics and some data analysis tasks. In all the listed domain sciences, the charge state changes during the simulation, this we need to encode it per particle outside the SpeciesType extension.
The notions you list are chemical notations and assume the charge state cannot change in a data series.
So yep, good idea to add, but not without a use case (i.e., an actively maintained chemistry code that commits to adopt openPMD). We can keep this open for potential future adopters if you like :)

This touches "What is a species?", and I'd agree that the charge state of individual particles is not part of that -- i.e. it does not make sense to include the charge in a species definition.
Suggestion for standard text:
The base speciesType standard does not support ions.
The charge state ("ionization") is an attribute of an individual particle (or macroparticle), and therefore an implementation SHOULD NOT accept a charge state as part of the speciesType using an implementation-defined syntax.
In the case that the implementation does treat the charge state ("ionization") as a global, unmodifiable property of the speciesType it MAY accept a charge state encoded in an implementation-defined way.

Note: With the Beam Physics extension there is a chargeState array that can be used to specify the charge state of all particles individually.


fix(command/start): can't found cli.js module on daemon mode

daemon mode can't found cli.js because of cli.js has been moved to bin directory

Codecov Report

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

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   86.95%   86.95%
=======================================
  Files          41       41
  Lines        1288     1288
=======================================
  Hits         1120     1120
  Misses        168      168

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 01323b3...6b902f6. Read the comment docs.

@raccoon-lee Thanks, i will release it right now.

ok, Thanks !


Inspect Element causes flickering

To reproduce:

Right-click an image or link.
Click "Inspect Element"
Watch UI flip out.

Noticed it too - this can be amended by disabling the UI refresh within fb.js which updates the UI every 200 milliseconds prior to debugging (line 35).


Transfer all settings from local version to server

Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version
Latest
Expected results
Configure content on a local version then copy/tranfser all settings (tables, dashboards, charts,users) to a server
Actual results
Can manual copy/paste dashboard and chart metadata but this is slow. I don't see a way to copy tables across with the parameters you've set. I think this is me missing some obvious method of doing this but can't see it in the docs. My assumption was that all of the settings are saved in the sql lite database superset makes, and that I'd just need to transfer this over.
Steps to reproduce

This is something I have been wondering about as well. We ended up using Superset (pip install 0.29.0rc7) for a while with the default sqlite superset.db, but now that I am setting up a proper production server for Superset, I'm having trouble importing created roles/users/new permissions of the development server version of our Superset installation. When I look at our sqlite database it only contains the first two admin accouts created with fabmanager, but none of the other users/etc that were created within the superset application show up there.


allow unfinished version when pinning custom toolchain

>rye pin<EMAIL_ADDRESS>Error: unsupported/unknown version for this platform

>rye pin<EMAIL_ADDRESS>pinned<EMAIL_ADDRESS>in .python-version

>rye pin 3.7
pinned 3.7.9 in .python-version

>rye pin<EMAIL_ADDRESS>pinned 3.7.9 in .python-version

>rye pin cpython@3
pinned 3.11.3 in .python-version

>

IMO it could allow even rye pin x86 (without any version) and use its latest version available.

Hidden behind this ticket are really two asks:

.python-version should encode a version request and not a version itself (eg: 3.7 should ask for latest available 3.7)
pin should not try to pin to a specific version

I'm strongly in favor of 1, i'm a bit unsure about how to do 2. I can see two options:
Pin to latest version by default:
rye pin 3.7  # pins 3.7.9
rye pin 3.7 --loose  # pins 3.7

Pin lose by default:
rye pin 3.7  # pins 3.7
rye pin 3.7 --precise # pins 3.7.9

Or a version of that.

There is also a third ask here which is that for manually registered toolchains, a request also picks the latest available which currently is also not yet supported.

Pin to latest version by default:
rye pin 3.7  # pins 3.7.9
rye pin 3.7 --loose  # pins 3.7

yes, it also makes more sense to me too. (I think loose naming not good here.)
and note that it should not limit to minor part. (eg. > rye pin pypy  # pins<EMAIL_ADDRESS>

I think I like neither --loose nore --as-is. Maybe --relaxed?

Or --dynamic (that more familiar with me maybe bcz of project.dynamic in pyproject file)
Or --expandable (that points to it would be able expand to the latest version.)

Most of it is I believe supported in #255 now. Would be nice if you could this this. Note that I did not support rye pin pypy, you need to do rye pin pypy@3 at least.  I don't think there is a lot of value in completely version-less pins.

I don't think there is a lot of value in completely version-less pins.

custom toolchains in most of times associated with single version. so that would be nice prevent always write @....

My biggest worry is that it introduces ambiguous syntax. In theory you can already name your custom tool-chain "1234" too.

it can solve by limit the toolchain naming to match identity stile. (str.isidentifier in py)

it's not that much big problem.


add support for logs

cleaned up code a bit from last pr

@locke105 I'm fine with moving to target specific keys. Won't really affect me as I've been using it in openwhisk where I've needed to copy/paste the code directly into the action file.

This should be merged with commit ed2b78e471ce2c7cc619c919a281e019421f4ec6


WebView stops playing audio when app leaves the foreground

Bug description:
In my WebView I'm playing a video, but when the app goes into the background the audio stops and the video pauses.
I have used expo-av Audio.setAudioModeAsync({ staysActiveInBackground: true }) to ensure that background playback is allowed, and it is working properly with native video playback, just not playback in the WebView.
Is this something that is supported in the WebView? Or will I have to use a native approach instead?
To Reproduce:

Point WebView to a source that plays video in html
Run the app and start playing the video
Press the home button to put the app in the background
Playback stops after a brief delay

Expected behavior:
Video should continue playing and audio should continue while the app is in the background, according to the audio mode set through Expo
Environment:

OS: Android
OS version: 10
react-native version: 0.62
react-native-webview version: 9.4.0

still searching for solutions

still searching for solutions

still searching for solutions

Did anyone find a solution to this?


Add SupStats2 and MedicStats

SupStats2 and MedicStats plugins so we can get more detailed player events.
This will eventually be used for stats and possibly elo type system.

Considering the current issues with stopwatch mode, I am lowering the priority on this issue until those are fixed


Re-export winit monitor types

This simply re-exports a few winit convenience types so Bevy users can now acquire monitor information from the engine itself. Common uses cases would be requesting which video modes the monitor supports or determining the current native resolution.

I think we should create a wrapper around these inside bevy_window instead of directly re-exporting them.

We don't want to have a hard dependency on winit so we provide equivalent types in our own crates for everything
For example, event WindowEvent::CloseRequested for winit gets converted to event WindowCloseRequested from bevy_window https://github.com/bevyengine/bevy/blob/47004dfcb415a049e4c6e68fdf56c26de72f51a1/crates/bevy_winit/src/lib.rs#L290-L295

Agreed. We should create a new bevy-specific api to insulate users from backend-specific details. Feel free to create a new pull request that does this.


Could not find 'php'. The program may not be installed.

"See php installation instructions at https://github.com/Glavin001/atom-beautify#beautifiers or go to http://php.net/manual/en/install.php
You can configure Atom Beautify with the absolute path to 'php' by setting 'Executable - PHP - Path' in the Atom Beautify package settings.
Your program is properly installed if running 'where.exe php' in your CMD prompt returns an absolute path to the executable."
path to 'php'? i upload my script to a FTP server.

You need to have PHP and whatever PHP beautifier you're using installed locally.


When FF bug is fixed, remove ping/pong in peerconnection

When this bug is fixed:
https://bugzilla.mozilla.org/show_bug.cgi?id=929977
We should try removing our ping-pong workaround in peer connection: we can start with simply no sending ping-pong unless peer does. And then once enough people have updated, we can remove the code to do it altogether.

Do we have to remove this? We already removed it last summer only to re-add
it again. Unless this is really hurting us I think we should keep it.
Having a user stuck with a broken proxy is a terrible experience and this
helps prevent it
On Apr 23, 2015 1:33 PM, "iislucas"<EMAIL_ADDRESS>wrote:

When this bug is fixed:
https://bugzilla.mozilla.org/show_bug.cgi?id=929977
We should try removing our ping-pong workaround in peer connection: we can
start with simply no sending ping-pong unless peer does. And then once
enough people have updated, we can remove the code to do it altogether.
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/1389.

We should only remove it when we have good testing infrastructure that will tell us in advance if it breaks.

I agree with @iislucas.

No plans to fix this in the forseeable future.


[WIP] Markdown for challenges description

[x] Install react markdown
[x] Load the markdown dynamically for each challenge (created the simple-nft-example.md)
[ ] Create the remaining MD files from github challanges (remove the telegram / buidlguidl stuff -  take simple-nft-example.md as example)
[x] Style the markdown. Two options

[Chakra default styles
<ReactMarkdown className="markdown"> and create custom styles for markdown > (p, blockquotes, pre, code, etc).

@dgrcode you can take it from here IYW. If not, I'll continue with it later today.

I'm on it. Made some updates related to stylings. I need to fix an issue with inline code and haven't made a commit yet. Will push once I fix it this afternoon.

@carletex check the styles. I've used the ReactMarkdown approach with a utility package to combine it with ChakraUI. I'll add the rest of the MD documents tomorrow morning.

Looks greats! And love the approach. Pushed a commit:

Reduce the container size
Fix a useEffect deps.

For some reason it doesn't pick up the <h1> styles you defined in chakraMarkdownTheme.jsx. It using size 5xl instead of 2xl.

I'll add the rest of the MD documents tomorrow morning.

Check this first #91. It seems the new repo it not updated yet, but I those challenges are the one we are using now.

Yep, there's something weird with the font sizes. I just saw this in the docs:

If you use the size prop, the font size of the heading will automatically decrease in size for smaller screens

So it looks like using size is more flexible, but it doesn't match the fontSize prop 1:1. I think we want to use size, but maybe make them smaller.
I just tested a smaller size and it looks good. Will commit.

Some things that we should probably let Austin know:

Challenges missing: 3, 4, 6, 7
Challenge 5 is named challenge 3 throughout the readme. I didn't update it because it's also in links, so probably someone should look at what needs to be updated.

Also, some nice-to-have that we're missing:

Syntax highlighting. This could be a starting point
Copy button on code blocks. Should be simple.
check lists are shown as [ ] in the resulting web. We can do some fancy transformation on the list items.

I also made a rebase, that's why it's force pushed 😬


add "disableSeekButtons" property for hiding rewind/fast forward

new Prop: disableSeekButtons which hides the rewind + fast forward buttons, without hiding the play/pause button

closes #45

Thank you very much! Really appreciate it.


Upgrade our issue templates to use GitHub issue forms ✍

Introduction
GitHub has recently rolled out a public beta for their issue forms feature. This would allow you to create interactive issue templates and validate them 🤯.
Appwrite currently uses the older issue template format. Your task is to create GitHub issue forms for this repository. Please use Appwrite's issue templates as a reference for this PR.
Tasks summary:

[ ] Fork & clone this repository
[ ] Prepare bug report issue form in .github/ISSUE_TEMPLATE/bug.yaml
[ ] Prepare documentation issue form in .github/ISSUE_TEMPLATE/documentation.yaml
[ ] Prepare feature request issue form in .github/ISSUE_TEMPLATE/feature.yaml
[ ] Push changes to master and test issue forms on your fork
[ ] Submit pull request

If you need any help, reach out to us on our Discord server.
Are you ready to work on this issue? 🤔 Let us know, and we will assign it to you 😊
Happy Appwriting!

Can you assign this to me?

can i work on this issue?

@vendz  All the best :rocket:

@christyjacob4 #19 fixes this issue. please review it!

@vendz Thanks for the PR! 🤯 Please give us some time to review it.


tests: kernel: semaphore: Added tests for semaphore.

The following test cases have been added for semaphore.

Test simple semaphore give/take from an isr.
Test simple semaphore give/take from a task.
Test semaphore take with no wait.
Test semaphore take with no wait and fails.
Test semaphore take with timeout and fails.
Test semaphore take with timeout.
Test semaphore take with timeout as forever
Test semaphore take with timeout from an isr.
Test semaphore take on multiple threads.
Test semaphore give/take from an isr.
Test semaphore multiple threads wait on a semaphore.
Test semaphore measure timeouts.
Test semaphore measure timeout when give is from another thread.
Test semaphore multiple sem take and timeouts.
Test semaphore multiple sem take with diff timeout & semaphore.

Signed-off-by: Adithya Baglody<EMAIL_ADDRESS>

Codecov Report

Merging #6745 into master will increase coverage by 0.24%.
The diff coverage is 98.72%.

@@            Coverage Diff             @@
##           master    #6745      +/-   ##
==========================================
+ Coverage   54.39%   54.63%   +0.24%
==========================================
  Files         444      446       +2
  Lines       42222    42457     +235
  Branches     7996     8012      +16
==========================================
+ Hits        22965    23196     +231
- Misses      16049    16053       +4
  Partials     3208     3208

Impacted Files
Coverage Δ

tests/kernel/semaphore/semaphore/src/main.c
75% <75%> (ø)

tests/kernel/semaphore/semaphore/src/semaphore.c
99.13% <99.13%> (ø)

kernel/posix/pthread.c
69.5% <0%> (-0.5%)
:arrow_down:

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 be52e3c...e971a46. Read the comment docs.

why does main.c and the header file exist?
just put it all in one C file.

makes sense I will combine the header into the c file.
But for main.c file I am bit concerned since some test cases have a dedicated main.c, others don't. So not sure what to follow here.

But for main.c file I am bit concerned since some test cases have a dedicated main.c, others don't. So not sure what to follow here.

use main.c with all test code in the file, if the test does test multiple things in parallel that are not logically related, you can have multiple .c files for each tested component.

delta:

Merged into a single file(i.e main.c)
Handled other review fixes.


I am not able to open external links from whatever in my ubuntu linux.. Apart from that it is great electron app for evernote

Please, check issue #2


parsing of the persian steingass ids and short defs files is coming up short

we're getting only 26938 of 69868 rows of the short defs file and 68014 of 69868 rows of the ids file.  Not sure if it's a parser or data problem yet.

the lexicon client code was fixed by setting the quoteChar on the papaparse to a non-printable unicode character. Remaining issue is one with the data, entered as https://github.com/alpheios-project/stg/issues/1


500 Internal Server Error when using gemini-1.5-pro model

Description of the bug:
I am experiencing intermittent 500 Internal Server Errors when using the gemini-1.5-pro model with the Google generative AI library. Sometimes the model responds correctly, but other times I encounter the error. This issue does not occur when using the gemini-1.5-flash model.
Actual vs expected behavior:
Actual behavior: Intermittent 500 Internal Server Errors when using gemini-1.5-pro model.
Expected behavior: Consistent and correct responses from the gemini-1.5-pro model without errors.
Any other information you'd like to share?

Use the gemini-1.5-pro model
Run the code and provide various inputs.
Example prompt: "hello world"
Answer: Responds normally.
On the second or third question, observe the 500 Internal Server Error:

Traceback (most recent call last):
File "/root/ren/ai.py", line 30, in
response = chat_session.send_message(user_input)
File "/usr/local/lib/python3.10/dist-packages/google/generativeai/generative_models.py", line 578, in send_message
response = self.model.generate_content(
File "/usr/local/lib/python3.10/dist-packages/google/generativeai/generative_models.py", line 331, in generate_content
response = self._client.generate_content(
File "/usr/local/lib/python3.10/dist-packages/google/ai/generativelanguage_v1beta/services/generative_service/client.py", line 827, in generate_content
response = rpc(
File "/usr/local/lib/python3.10/dist-packages/google/api_core/gapic_v1/method.py", line 131, in call
return wrapped_func(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/google/api_core/retry/retry_unary.py", line 293, in retry_wrapped_func
return retry_target(
File "/usr/local/lib/python3.10/dist-packages/google/api_core/retry/retry_unary.py", line 153, in retry_target
_retry_error_helper(
File "/usr/local/lib/python3.10/dist-packages/google/api_core/retry/retry_base.py", line 212, in _retry_error_helper
raise final_exc from source_exc
File "/usr/local/lib/python3.10/dist-packages/google/api_core/retry/retry_unary.py", line 144, in retry_target
result = target()
File "/usr/local/lib/python3.10/dist-packages/google/api_core/timeout.py", line 120, in func_with_timeout
return func(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/google/api_core/grpc_helpers.py", line 78, in error_remapped_callable
raise exceptions.from_grpc_error(exc) from exc
google.api_core.exceptions.InternalServerError: 500 An internal error has occurred. Please retry or report in https://developers.generativeai.google/guide/troubleshooting
This error appears intermittently when using gemini-1.5-pro but does not occur with gemini-1.5-flash.

same issue from yesterday. and still persistent today.

I Got the same issue.

Same issue !

I have had this problem before with only a few requests per day, but the current 500 rate is outrageous for making any production service from Gemini 1.5-pro.

I'm in the free plan, are you in the pay plan?

I have had this problem before with only a few requests per day, but the current 500 rate is outrageous for making any production service from Gemini 1.5-pro.

Hey guys, I've got good news. It seems like I'm rarely getting the 500 Internal Server Error when using the Gemini 1.5 Pro models. How about you guys?

I have had this problem before with only a few requests per day, but the current 500 rate is outrageous for making any production service from Gemini 1.5-pro.

I have tried creating an API key on a new Google account, and the result showed that even with a simple prompt like "hello world," it still resulted in a 500 Internal Server Error at that time.

I have had this problem before with only a few requests per day, but the current 500 rate is outrageous for making any production service from Gemini 1.5-pro.

I have tried creating an API key on a new Google account, and the result showed that even with a simple prompt like "hello world," it still resulted in a 500 Internal Server Error at that time.

yes the same problem

@ferrenza,
Thank you reporting this issue. This looks like an intermittent error and should work now.
This repository is for issues related to Python SDK client bugs or improvements and the client doesn't control the service's responses. For issues related to Gemini API, we would suggest you to use "Send Feedback" option in Gemini docs. Ref: Screenshot below. You can also post this issue on Google AI forum.

Yeah, the service was overloaded. They are working on updating the 500 error to a 503, that "service unavailable" error should trigger automatic retries with backoff.
Also note that in addition to those automatic retries the "request_options" argument lets you pass a retry specification:

https://github.com/google-gemini/generative-ai-python/blob/d0f3359a93a5192238bbf655ed57b6235913aef2/google/generativeai/generative_models.py#L237
https://github.com/google-gemini/generative-ai-python/blob/d0f3359a93a5192238bbf655ed57b6235913aef2/google/generativeai/types/helper_types.py#L36

The same question, is it restricted?

how to handle this error ?

I believe most 500's right now are because the service is overloaded. They are planning to upgrade those to 503 to make that clearer.

I'm getting 500 Internal error encountered when trying to cache ~1.8M tokens using gemini-1.5-pro-001.

Interesting/ This is the first report I've seen of a 500 from caching.
Is it persistent? Only for giant contexts?

I switched from "gemini-1.5-pro-latest" to "gemini-1.5-flash" - there were also some "out of quota" error;

I upgrade from vertexai v1.64.0 to 1.65.0 to fix...

I switched from "gemini-1.5-pro-latest" to "gemini-1.5-flash" and all those errors stopped - there were also some "out of quota" error;

Oh, yeah, that would do it. Quotas are much higher for flash.

@MarkDaoust I spent hours to figure out how to increase my quota and still don't know how ! Isn't all "pay as we go" ? Thank you.

https://ai.google.dev/pricing
No, you start on the free plan  ... and (I think) if you setup billing for you project, then you're on the pay as you go plan.

I upgrade from vertexai v1.64.0 to 1.65.0 to fix...

I wouldn't ecpect changing local software to affect this (also this repo is not connected to vertex).

100% this was the issue. I spent a ridiculous amount of time tying everything. As soon as I upgraded the same code finally worked.


Implement beman.ranges_and_views

This task is to create a new Beman.RangesAndViews library, however, this library should grow incrementally - there's not expectation that it will be implemented all at once :)
The task is to create the project and namespace within Beman, and add to it:

The basic concepts and structs for ranges (range, view sentinal, etc.)
Add utils incrementally, such as:

"P3060: Add std::views::upto(n)" by Weile Wei, Zhihao Yuan
"P3137:	views::to_input" by Tim Song
"P3138: views::cache_last" by Tim Song

(more will be added later, feel free to suggest in comments)

I'm going to take a shot at this one :-)


New label right side

Not sure why the right padding is bigger. Don't want to break Button
cc @cecileboucheron

waiting on https://github.com/keybase/client/pull/16920


Testing Mathlib on top  batteries#793

This PR tests mathlib on top of leanprover-community/Batteries#793

The upstream PR has landed and mathlib has been updated. Can this PR be closed?

Sure. It isn't meant to be merged anyway.


Setuptools Bootstrapper script is deprecated

Hi 👋
Per the header of the Setuptools bootstrapper script (https://bootstrap.pypa.io/ez_setup.py), this installation method is deprecated.
Equally the "curl | python" pattern might not be a great idea - https://jordaneldredge.com/blog/one-way-curl-pipe-sh-install-scripts-can-be-dangerous/
I'll try to look for an alternative when I am I front of a laptop 👍.

Closing this issue as hopefully it has been addressed a while ago, specially since #35 and the introduction of a standard pip install qiskit method (plus the alternate installation methods on the README and documentation). Thanks for reporting, and please don't hesitate on reopening if you feel further discussion or actions are required!


if_lua: stop on errors in vim.command() and vim.call()

This stops execution if vim.command() or vim.call() failed. This fixes
two problems:
The first is the lack of line numbers if something failed:
Error detected while processing command line..function <SNR>48_Cmd:
line   13:
E1174: String required for argument 2
E1174: String required for argument 2
E1174: String required for argument 2
E1174: String required for argument 2
[..repeated many times..]

It's not at all clear what command failed where, and vim.command()
always returns 0. With this, it's:
Error detected while processing command line..function <SNR>49_Cmd:
line   13:
E1174: String required for argument 2
lpeg.vim/lua/lpeg.lua:131: lua: function failed

Which is much more useful.
The second is that if you have some error in a loop you'll be spammed with
errors; this is #8649

The big downside with this patch is that it changes the behaviour; I don't know
if any scripts relied on execution continuing after errors, but they might.
However, Lua itself stops execution out if a (Lua) function failed, vim.eval()
did as well, and it generally just makes sense IMO. It's really confusing and
surprising otherwise
An alternative might be actually returning something from luaV_command();
luaV_call() already does so. You'll have to add "if not vim.command(...) then
error("command failed") end" to everything, which doesn't seem very "Lua-like".
Another point of improvement is that the errors for "vim.fn" are
confusing; for the same as above it prints:
Error detected while processing command line..function <SNR>48_Cmd:
line   13:
E1174: String required for argument 2
[string "vim.fn = setmetatable({}, {..."]:4: lua: call_vim_function failed

Which isn't very good. Need to look in to that.
(I'll update the tests and docs later once the behaviour is decided)

Martin Tournoij wrote:

This stops execution if vim.command() or vim.call() failed. This fixes
two problems:
The first is the lack of line numbers if something failed:
Error detected while processing command line..function 48_Cmd:
line   13:
E1174: String required for argument 2
E1174: String required for argument 2
E1174: String required for argument 2
E1174: String required for argument 2
[..repeated many times..]
It's not at all clear what command failed where, and vim.command()
always returns 0. With this, it's:
Error detected while processing command line..function 49_Cmd:
line   13:
E1174: String required for argument 2
lpeg.vim/lua/lpeg.lua:131: lua: function failed
Which is much more useful.
The second is that if you have some error in a loop you'll be spammed with
errors; this is #8649

The big downside with this patch is that it changes the behaviour; I
don't know if any scripts relied on execution continuing after errors,
but they might.

I do hope script writers avoid any errors.  At least they might upset
users.  Unless it happens very infrequent perhaps.
The argument that it keeps going and spits out tons of errors is a
strong one.  Perhaps that's worth taking a bit of a risk of breaking
something that relies on ignoring errors.

However, Lua itself stops execution out if a (Lua) function failed, vim.eval()
did as well, and it generally just makes sense IMO. It's really confusing and
surprising otherwise
An alternative might be actually returning something from luaV_command();
luaV_call() already does so. You'll have to add "if not vim.command(...) then
error("command failed") end" to everything, which doesn't seem very "Lua-like".
Another point of improvement is that the errors for "vim.fn" are
confusing; for the same as above it prints:
Error detected while processing command line..function 48_Cmd:
line   13:
E1174: String required for argument 2
[string "vim.fn = setmetatable({}, {..."]:4: lua: call_vim_function failed
Which isn't very good. Need to look in to that.
(I'll update the tests and docs later once the behaviour is decided)

When using something like this:
luaL_error(L, "lua: call_vim_command failed");

Does the string get included somehow?  Otherwise it's not very clear.
--
Microsoft's definition of a boolean: TRUE, FALSE, MAYBE
"Embrace and extend"...?
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

When using something like this:
luaL_error(L, "lua: call_vim_command failed");
Does the string get included somehow?  Otherwise it's not very clear.

It relies on the error message from  the error message from do_cmdline_cmd() and call_vim_function(); I tried copying the way Python does it with VimTryStart() and VimTryEnd(), but I couldn't get that to work. It's all a bit confusing as there are quite a few global variables and it's not always clear what gets set when, but msg_list was always NULL.
I can look at it again when we decide which way we want to go with this. I'd hate to break people's scripts, but on the other hand the current behaviour is mostly just annoying during development (in your final version you don't really want any errors), and I don't think all that many people are using the Lua integration, so I think it might be okay, but hard to be sure.
We can also add an option maybe, so you can do e.g. vim.fatalerrors() in your Lua script.

Martin Tournoij wrote:

When using something like this:
luaL_error(L, "lua: call_vim_command failed");
Does the string get included somehow?  Otherwise it's not very clear.

It relies on the error message from  the error message from
do_cmdline_cmd() and call_vim_function(); I tried copying the way
Python does it with VimTryStart() and VimTryEnd(), but I couldn't
get that to work. It's all a bit confusing as there are quite a few
global variables and it's not always clear what gets set when, but
msg_list was always NULL.

What I was thinking about is using a local buffer, using vim_snprintf()
to add the command to the error message.  That only works if the string
is used right away, not when the pointer is used later.

I can look at it again when we decide which way we want to go with
this. I'd hate to break people's scripts, but on the other hand the
current behaviour is mostly just annoying during development (in your
final version you don't really want any errors), and I don't think all
that many people are using the Lua integration, so I think it might
be okay, but hard to be sure.
We can also add an option maybe, so you can do e.g.
vim.fatalerrors() in your Lua script.

--
Linux is just like a wigwam: no Windows, no Gates and an Apache inside.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Oh yeah, adding the actual command is easy: luaL_error() already does sprintf-style formatting. l actually I did that before but removed it because I felt it was a bit noisy for longer commands.

@arp242 is this still useful?

Probably?
I haven't worked with the Lua integration since 2021, so don't really know what the current status of it is.
I don't really have the time to look at this, and don't think I will in the foreseeable future. So I'll just close it.


change: Regenerate Tests

Update change tests to: 1.3.0 258c807
Update generator to use the standard error indicator.
Update solution to pass updated tests.

Hello @iHiD! Thank you for your review. Wanted us to deal with #870 before this one, that's why I took so long to answer.
I understand and agree with your points, but I have a question:
Given how all 3 of these "error cases" are specified on the canonical data as "expected": {"error": ...}, how do you typically differentiate between them in the change_case.rb file? Should I check for the test name using something like this?

I assume you meant @Insti there :)
Yeah you'll need to do something like that. Either check the test name or error text I guess.

Ah! I was confused :)

Oh no! Sorry @iHiD 😅 Wrote that in a hurry and didn't notice it!
@Insti Thanks :+1: I'll work on something soon.

@Insti I just pushed some tentative changes. I didn't follow your suggestions to the letter, but wanted to get your opinion regardless.
Please let me know what you think. I also appreciate suggestions for better error names :+1:

Do you think this is ready to merge now @pgaspar ?

I think so @Insti! Thank you 👍

Thanks for all your work on these @pgaspar :heart:


How can I convert SDK response to json using Jackson?

I want to convert an instance of com.microsoft.azure.management.compute.VirtualMachine to json using Jackson. But some of the getters in the SDK make REST api calls :woman_facepalming: , and JSON serialization gets stuck in a loop of multiple api calls.
Screenshot

Is there a simple way to convert the sdk models to json? something that you guys might be using internally?

If you check azure java sdk examples on this same repository main page, they have written an Utility class to print virtual machine.. Haven't checked if that is in JSON format or not but this information may be useful to you.

I was able convert is using writeAsString(vm.inner()) where vm is an instance of com.microsoft.azure.management.compute.VirtualMachine

(y)

@anuchandy interesting as it seems related to the serialization work you've been investigating

seems like you found an interesting approach to get the serialization side to work. We don't have anything better than that in the SDK.


[Question] Cluster support planned?

Hi there.
First of all, turbo-net & turbo-http are amazing, thank you very much :)
If I´m not mistaken, turbo-http doesn't work in cluster mode yet, as it doesn't implement the same "behind the scenes magick" as the default http module does to "share" an open port.
I´ve just, very naively, tried to map the example from the docs 1:1 - https://nodejs.org/docs/latest/api/cluster.html
So, chances are high that I´m just doing something wrong here, but if not, I´d just simply ask if it is planned to implement cluster compat., or if I should go down the route of forking instances & using a traditional load balancer instead of cluster mode?

If I´m not mistaken, turbo-http doesn't work in cluster mode yet, as it doesn't implement the same "behind the scenes magick"

I think you're right here, just ran into this myself too. So +1

Yes turbo-net does not do any clustering. I'm not sure what's involved to get that working.

@mafintosh Thank you for the answer, it´s not that critical, as I don't want to use it for a production product yet & I´m just toying around with it.
I´ll see if I can find some clues in the node sources, but I doubt that this is a trivial implementation.

haha @asciidisco i've been doing exactly that and... it's been quite the rabbit hole
e.g. https://github.com/nodejs/node/blob/212de3c5ec429a580d2e79ce3c2516b93b52b8f5/lib/_http_common.js#L264 😱

@jacktuck https://github.com/nodejs/node/blob/master/lib/internal/cluster/round_robin_handle.js

The way it is implemented in node is quite understandable. It looks like one could create a separate package that could just wrap turbo-http and make it cluster-able.
If I´m not missing any important point, I´d say that if it can be done as a separate package, it should be done this way, as every implementation in turbo-http would have a performance impact of some sort, even if users wouldn't need the functionality.
Therefore I´m fine with closing this. If >1 instances are needed, people could still use the "classic" load balancer approach.

The way it is implemented in node is quite understandable

Can you give me synopsis of how node's doing it? I think the bit you linked to is related to distributing workers onto 'random' ports
Yeah i do wonder what the performance hit would be

@jacktuck It´s explained in the docs itself (better than I could ever do it): https://nodejs.org/api/cluster.html#cluster_how_it_works

If you need clustering, you can use @rayo/storm which will basically spawn your worker (server) function over all available CPU cores (you can also limit the number).
Something along the lines of:
const rayo = require('rayo');
const turbo = require('turbo-http');
const server = turbo.createServer();

rayo({ port: 5050, server, storm: {} })
  .get('/', (req, res) => res.end('Thunderstruck by rayo, turbo and storm..!'))
  .start((address) => {
    console.log(`Up on port ${address.port}`);
  });

In this case rayo.js will, essentially, just provide the routing capabilities. Also, in this example, you don't need to require @rayo/storm since rayo already comes bundled with it, you just need to turn it on.

I submitted a pull request to turbo-net that enables cluster support.
https://github.com/mafintosh/turbo-net/pull/16

Hi. First of all, great work here.
Are there any updates on this?

Any news?


Implement Logger Notifier

Use Monolog or any other PSR logger as notifier

Thanks for your work on this new feature!


Bug: SetScanPeriod and SetBetweenScanPeriod Methods Missing Return Statements Leading to IllegalStateException

Summary:
In the FlutterBeaconPlugin, the setScanPeriod and setBetweenScanPeriod method handlers within the onMethodCall function do not include return statements after invoking result.success or result.error.
This omission causes the method to continue execution and inadvertently call result.notImplemented(),
resulting in an IllegalStateException due to multiple responses being sent for a single method call.
Steps to Reproduce:

Invoke the setScanPeriod method from the Flutter side with a valid scanPeriod argument.
Observe that after successfully setting the scan period, an IllegalStateException is thrown.
Log: java.lang.IllegalStateException: Reply already submitted

Try the updated version of this library, now ported to Kotlin with support for Flutter 3.x, Gradle, and an improved example:
https://pub.dev/packages/dchs_flutter_beacon


[5.1] optimize: write to temporary file first, then rename it

The cached file is written to as the classes are compiled, which can
result in a partial file being used for serving the application,
resulting in all kinds of possible errors. The client will receive a 500
usually.
By first writing to a temporary file and then moving it over, we
mitigate this factor.

A previous PR was sent and rejected, sorry.

Never try to run composer update on a production app.

Thx; what's the point of it then? Generate and push into git?

Never try to run composer update on a production app.

Not related to composer update. php artisan optimize is executed by default under post-install-cmd.
https://github.com/laravel/laravel/blob/master/composer.json#L33

For reference, the other PR: https://github.com/laravel/framework/pull/10161/files
Should have done my homework.
Still think that, whether devs think this is a good idea or not, that it's simply smart to do it and does not cause side-effects; win-win for everyone.


[Compatible Implementations Announcement] Open Liberty, <IP_ADDRESS>

Organization Name
Name: IBM
Product Version and download URL
URL: Open Liberty <IP_ADDRESS>-beta
Link to CCR (GH issue in the eclipse/microprofile repository)
Link: https://github.com/eclipse/microprofile/issues/380
Media Description
280 chararters includes the organization's name & URL or blog
We are very pleased to announce that @openliberty <IP_ADDRESS>-beta is the compatible implementation used for ratification of the brand new release of MicroProfile 6.1 https://github.com/eclipse/microprofile/releases/tag/6.1. You can find more details on this release https://openliberty.io/blog/2023/09/26/<IP_ADDRESS>-beta.html. Enjoy the new release!
Design Card  (for MicroProfile designer)
Notes
To help with everything

Scheduled


document imported from json and exported as th2 has no header

Original th2 header
encoding  utf-8
##XTHERION## xth_me_area_adjust -2191.0<PHONE_NUMBER>999987 10720.0085744184 15035.2755905497
##XTHERION## xth_me_area_zoom_to 100
##XTHERION## xth_me_image_insert {3967.638503937 1 1.0} {8550.7240157473 4.5} ptopoHawkes/4-Hawkes_p.xvi 0 {}
##XTHERION## xth_me_image_insert {2954.2132988292997 0 1.0} {8013.663358151501 1.6} ptopoHawkes/2-Hawkes_p.xvi 0 {}
##XTHERION## xth_me_image_insert {1823.89361379 1 1.0} {8670.149380629999 1.4} ptopoHawkes/1-Hawkes_p.xvi 0 {}
##XTHERION## xth_me_image_insert {3347.4742519699994 1 1.0} {9198.117086614 3.18} ptopoHawkes/3-Hawkes_p.xvi 0 {}
scrap 3-HawkesPlan-s1 -projection plan ...

Start of exported th2 with no header (in error)
scrap 3-HawkesPlan-s1 ...
This th2 will not open in XTherion

I understand the problem with the image_insert headers if you're working in both wtherion and XTherion, I'll try to create a fix.
Otherwise these headers are completely unnecessary, and a lack of them shouldn't stop XTherion from opening the file.
What error exactly does XTherion give? Or you can just upload the error causing TH2 file.

For the case of th2 imported by WT, and then saved and exported to th2, this produces a valid file that can be opened by XTherion.  Not exactly sure if this is the scenario described in the original post, as in it I mention json.  for now I assume this can be closed.


add i18n support for navigation toggle

Although the toggle only appears when the page is too thin to show both navigation and content, adding i18n support do no harm.

Thanks for the PR.
Honestly, I wasn't aware that the default index page header (which has this text) varys in comparison to the usual site header (which doesn't have it).
The header will change in #151 and we may find a better solution for this. Until #151 is fixed I will keep this PR. Maybe the text still proves to be useful.


Fix spelling error

I assumed the intended word was "lose" over "loosen", please let me know if we should change to that instead.

 Thanks 😄
/lgtm
/approve
/ok-to-test
/override dco

@munnerz whoops didn't see that you already approved - i amended the commit with a sign-off, do you mind approving again?

/lgtm


Reopening #103

It still happens if you have an empty set in any category and bombs with something less explanatory than "auth failed" if you provide invalid credentials unless you go all the way through the traceback.

Reopening #103

If the same issue with #103 still happens in your environment, please reopen #103 itself, and describe the concrete example to reproduce.
Thanks!


npm WARN deprecated<EMAIL_ADDRESS>graceful-fs version 3 and before will fail on newer node releases. Please update to<EMAIL_ADDRESS>as soon as possible.

npm -v
3.3.12

node -v
v5.4.1

exceljs depends on unzip2 that seems unmaintained for a year. unzip2 has currently a dependency on an old version of fstream that depends on graceful-fs 3.
An updated version of fstream exists and depends on graceful-fs 4, however there is no answer from unzip2 author about that issue.
Any chance you can get rid of this outdated unzip2 dependency ?
BTW : exceljs is a nice work and I really appreciate it. The documentation is well written also.

I would like to create a PR for this issue since its failing my CircleCI CI.
Does anyone can advise for an alternative for unzip2?

Yes please provide a fix ; it's causing lots of problems with bundle tools, such as nexe and fuse-box.


[Serialization] Unique SIL generic environments when serializing.

Teach the serialization of SIL generic environments, which used to be
a trailing record following the SIL function definition, to use the
same uniqued "generic environment IDs" that are used for the AST
generic environments. Many of them overlap anyway, and SIL functions
tend to have AST generic environments anyway.
This approach guarantees that the AST + SIL deserialization provide
the same uniqueness of generic environments present prior to
serialization.

@swift-ci please smoke test and merge


Added New Emojicloud (V1)

New Fancy Emojicloud

Better looking Unicode Emojis
Custom Emojis (Animated too!)
Alex's version is still there at /wordcloud
Use /emojicloud to activate this

Lightly Tested. I expect it to be imperfect but will get there in a few iterations and with feedback. Hopefully. 🤞

I am hyped beyond belief.

I am belief beyond hyped.

I am

Resolved Suggestions by Luke


format specifier miss on span details page

I think we need a %v somewhere because the attribute values can be non-string

This issue is fixed on the release https://github.com/ymtdzzz/otel-tui/releases/tag/v0.2.2 :D
Thanks for reporting and please let me know if you find any other issues!


Fixes formatting errors in API docs

Codecov Report

Merging #126 (2392db4) into main (2cef737) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   80.67%   80.63%   -0.05%
==========================================
  Files          65       65
  Lines        2148     2148
==========================================
- Hits         1733     1732       -1
- Misses        415      416       +1

Impacted Files
Coverage Δ

ezflow/functional/data_augmentation/operations.py
85.45% <0.00%> (-1.82%)
:arrow_down:


Frame Application Screen: Facilities Overview

User story

As a facility manager
I want / need a screen to view all the facilities
So that I can have an overlook of all the buildings

Acceptance criteria

[ ] The screen needs to be connected to the correct route in the navigation bar
[ ] Upon visiting the page/screen, the facility manager sees all the facilities in the form of cards
[ ] The facility cards need to be displayed in a grid/flex container
[ ] Each card needs to contain some basic information about the facility, such as name, location, and number of waterpumps
[ ] The screen must use components from Siemens iX

DoD general criteria

[ ] Feature has been fully implemented
[ ] Feature has been merged into the mainline
[ ] All acceptance criteria were met
[ ] Product owner approved features
[ ] All tests are passing
[ ] Developers agreed to release

Dependencies

Item #48
Item #47

I just added myself now because I forgot. Patrick and I wrote most of the code together

small because after implementing the home page this site was pretty much just copy and paste


(Jammy) Update domain-broker deployment

In order to migrate to Jammy stemcells, we want to update our domain-broker deployment to use Jammy
Acceptance Criteria

domain-broker deployment is updated to use Jammy stemcell

Security considerations
None
Implementation sketch

[x] Pause staging and prod deployments: https://ci.fr.cloud.gov/teams/main/pipelines/deploy-domains-broker
[x] Update domain-broker deployment manifest to use Jammy: https://github.com/cloud-gov/cf-domain-broker-alb/blob/6bb65d9da29d120e0a6740ad30de1e9e82fef9c4/bosh/manifest.yml#L8
[x] Update domain-broker pipline to reference Jammy stemcells: https://github.com/cloud-gov/cf-domain-broker-alb/blob/6bb65d9da29d120e0a6740ad30de1e9e82fef9c4/ci/pipeline.yml#L19
[x] Update the domain-broker deployment manifest to point to Jammy
[x] Redeploy domain-broker in development: https://ci.fr.cloud.gov/teams/main/pipelines/deploy-domains-broker/jobs/deploy-development
[x] Confirm whether domain-broker is working by going to the development jumpbox and verifying the health of the VMs. SSH into the VMs and inspect the process health with monit
[x] If everything is healthy, unpause and deploy to staging
[x] Confirm whether domain-broker is working by going to the staging jumpbox and verifying the health of the VMs. SSH into the VMs and inspect the process health with monit
[x] If everything is healthy, unpause and deploy to production

Domain-broker has been updated to use Jammy and deployed without issues.


Assertion failed

Hi everyone,
After updating to the latest commit ( 4405c51 ) I am getting an assertion failed. I am running the beta NVIDIA drivers (422.77) which support Vulkan 1.2.133. Everything was working fine until I updated VMA, is this a bug or am I doing something wrong?
Assertion failed!

Program: Foo.exe
File: vk_mem_alloc.h, Line 15394

Expression: m_VulkanFunctions.vkGetPhysicalDeviceProperties != nullptr

I checked on my NVIDIA card, driver 445.75, Vulkan SDK <IP_ADDRESS> and it works correctly. I also double-checked the code that fetches function pointers and it looks good to me.
Please check any other Vulkan program like command-line "vulkaninfo" if it works for you.

vulkaninfo seems to work (https://hasteb.in/xezepuqi.yaml). I tried 445.75 too and it spits out the same error.

Possibly you didn't set VmaAllocatorCreateInfo::instance member when creating the allocator. Please try setting it and see if it helps. Documentation telling that it's optional is not correct - it's my fault, it's required now.

Ahh yes, that fixed it! Thank you very much.

I clarified the documentation. Thank you for helping me to find this issue.


[Bug Report] Fail to build with gcc-12 and gcc-13

Describe the bug
It fails to build the latest main branch with gcc-12 and even with gcc-13.

gcc-12 cannot build the latest main because it does not support static operator().
gcc-13 supports static opeartor() but it raises an error '-std=c++23' or '-std=gnu++23' [-Werror=c++23-extensions] because it is c++23 feature.
set(CMAKE_CXX_STANDARD 23) should be used in CMakeLists.txt.
But another error occurs like below in decorators.hpp.

Error msg.
FAILED: ttnn/CMakeFiles/ttnn.dir/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp.o
/usr/bin/g++-13 -Dttnn_EXPORTS -I/home/namhyeong/shared/tt-metal/tt_metal/third_party/umd -I/home/namhyeong/shared/tt-metal -I/home/namhyeong/shared/tt-metal/tt_metal -I/home/namhyeong/shared/tt-metal/ttnn -I/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/deprecated -I/home/namhyeong/shared/tt-metal/ttnn/cpp -I/home/namhyeong/shared/tt-metal/tt_metal/third_party/fmt -I/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/experimental/tt_lib -I/home/namhyeong/shared/tt-metal/tt_metal/third_party/pybind11/include -I/usr/include/python3.10 -I/home/namhyeong/shared/tt-metal/tt_metal/hw/inc -I/home/namhyeong/shared/tt-metal/tt_metal/hw/inc/wormhole -I/home/namhyeong/shared/tt-metal/tt_metal/hw/inc/wormhole/wormhole_b0_defines -I/home/namhyeong/shared/tt-metal/tt_metal/third_party/umd/device/wormhole -I/home/namhyeong/shared/tt-metal/tt_metal/third_party/umd/src/firmware/riscv/wormhole -isystem /home/namhyeong/shared/tt-metal/.cpmcache/reflect/e75434c4c5f669e4a74e4d84e0a30d7249c1e66f -O3 -std=c++20 -fPIC -fvisibility=default -MP -Wno-int-to-pointer-cast -fno-var-tracking -mavx2 -fPIC -DFMT_HEADER_ONLY -fvisibility-inlines-hidden -fno-lto -DTT_METAL_VERSIM_DISABLED -Werror -Wdelete-non-virtual-dtor -Wreturn-type -Wswitch -Wuninitialized -Wno-unused-parameter -Wno-deprecated -Wno-attributes -Wno-stringop-overread -Wno-stringop-overflow -Wno-maybe-uninitialized -Wno-missing-requires -Winvalid-pch -include /home/namhyeong/shared/tt-metal/build_Release/ttnn/CMakeFiles/ttnn.dir/cmake_pch.hxx -MD -MT ttnn/CMakeFiles/ttnn.dir/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp.o -MF ttnn/CMakeFiles/ttnn.dir/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp.o.d -o ttnn/CMakeFiles/ttnn.dir/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp.o -c /home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp
In file included from /home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/data_movement/tilize/tilize.hpp:9,
                 from /home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/core/to_layout/to_layout_op.cpp:9:
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:279:17: error: 'static auto ttnn::decorators::detail::lambda_operation_t<lambda_t>::operator()(auto:146&& ...)' may be a static member function only with '-std=c++23' or '-std=gnu++23' [-Werror=c++23-extensions]
  279 |     static auto operator()(auto&&... args) { return lambda_t(std::forward<decltype(args)>(args)...); }
      |                 ^~~~~~~~
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp: In substitution of 'template<auto id, class concrete_operation_t>  requires  static_cast<bool>((SyncOperation<concrete_operation_t>)^ (AsyncOperation<concrete_operation_t>)) struct ttnn::decorators::operation_t [with auto id = 0; concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]':
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:273:83:   required from 'constexpr auto ttnn::decorators::register_operation(const char*) [with concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]'
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/data_movement/tilize/tilize.hpp:45:97:   required from here
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:172:89: error: 'operator()' is not a member of 'ttnn::operations::data_movement::ExecuteTilize'
  172 | concept SyncOperation = requires { [](auto&&... args) { concrete_operation_t::operator()(args...); }; };
      |                                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp: In instantiation of 'constexpr auto ttnn::decorators::register_operation(const char*) [with concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]':
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/data_movement/tilize/tilize.hpp:45:97:   required from here
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:273:83: error: template constraint failure for 'template<auto id, class concrete_operation_t>  requires  static_cast<bool>((SyncOperation<concrete_operation_t>)^ (AsyncOperation<concrete_operation_t>)) struct ttnn::decorators::operation_t'
  273 |     return operation_t<__COUNTER__, concrete_operation_t>{cpp_fully_qualified_name};
      |                                                                                ^
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:273:83: note: constraints not satisfied
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp: In substitution of 'template<auto id, class concrete_operation_t>  requires  static_cast<bool>((SyncOperation<concrete_operation_t>)^ (AsyncOperation<concrete_operation_t>)) struct ttnn::decorators::operation_t [with auto id = 0; concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]':
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:273:83:   required from 'constexpr auto ttnn::decorators::register_operation(const char*) [with concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]'
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/operations/data_movement/tilize/tilize.hpp:45:97:   required from here
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:179:8:   required by the constraints of 'template<auto id, class concrete_operation_t>  requires  static_cast<bool>((SyncOperation<concrete_operation_t>)^ (AsyncOperation<concrete_operation_t>)) struct ttnn::decorators::operation_t'
/home/namhyeong/shared/tt-metal/ttnn/cpp/ttnn/decorators.hpp:178:14: note: the expression 'static_cast<bool>((SyncOperation<concrete_operation_t>)^ (AsyncOperation<concrete_operation_t>)) [with concrete_operation_t = ttnn::operations::data_movement::ExecuteTilize]' evaluated to 'false'
  178 |     requires(static_cast<bool>(SyncOperation<concrete_operation_t> xor AsyncOperation<concrete_operation_t>))
      |             ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

To Reproduce
First, merge https://github.com/tenstorrent/tt-metal/pull/10599 PR to the latest main branch, and then build with gcc-12 and gcc-13.
Expected behavior
No compile error occurs.
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following environment information:

OS: Ubuntu 22.04
Version of software. latest main branch with https://github.com/tenstorrent/tt-metal/pull/10599 applied.

Additional context
Add any other context about the problem here.

cc. @TT-billteng @arakhmati

For the record. #10599 does not solve the issue. It is only  a partial solution.

Seems like the error caused by PR#10601.
Any action for this? @arakhmati @davorchap

Also poking on actions for this. I'm blocked too and don't want my local development get desynced from the latest commit by much.

@namhyeong-kim @razorback3 As a temporary solution. I found that I can build 248bf99c08f6057caa90a8b86b45a972a8264c75 with GCC 13 if I enable C++23 by set(CMAKE_CXX_STANDARD 23). It is still a problem as it forces me to build all downstream projects under C++23.

@namhyeong-kim @razorback3 As a temporary solution. I found that I can build 248bf99c08f6057caa90a8b86b45a972a8264c75 with GCC 13 if I enable C++23 by set(CMAKE_CXX_STANDARD 23). It is still a problem as it forces me to build all downstream projects under C++23.

Thanks for sharing 😀

Close this issue. The latest branch(commit: 66bfeaf) is built by gcc-13 with set(CMAKE_CXX_STANDARD 23).


fix: fix issues with grafana dashboard for version 8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 77.063%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
7761

Relevant Lines:
10071

💛  - Coveralls


Unusual optimizing_photometry.rst file

The optimizing_photometry.rst file ends up being nested under https://jwst-fgs-spelunker.readthedocs.io/en/latest/timeseries_observations.html which feels a very unusual spot for it since it is an API document rather than a tutorial like fgs-spelunker-and-tsos.rst. I would recommend just moving the documentation of optimizing_photometry to the API page.
This issue was made as part of the review for https://github.com/openjournals/joss-reviews/issues/6202

I have fixed this issue by removing optimizing_photometry since the document contents is already available in API.rst.


CADisplayLink code formatting issue

The code block on the CADisplayLink page doesn't have a proper indentation. I can see that it's there in the markdown but for some reason, it doesn't render correctly.
Docs page:

Markdown:

This is a head-scratcher. The XML looks good, the lang attribute has the proper value, the CDATA looks right... I've posted a question to internal support.

Since this issue is about the site experience itself you can file an issue at https://aka.ms/sitefeedback Thank you!


Play All Random - Customizable

After #13 passes and thus a customizable shuffle feature is available, adding more functionality would make sense.
Add following options:

ytpa-random=prefer-newest to randomly play between the latest 20 unwatched videos
ytpa-random=prefer-oldest to randomly play between the last 20 unwatched videos in storage

Possible UI:

Use gear or a dropdown symbol for more options.


How to confirm ROS1 nodes are running when following the "Setup Virtualized Kuberenetes Cluster" example?

Thank you very much for making this repository available to us all!
I followed the instructions to "Install Kubernetes Packages" carefully, except for the Docker and Kubernetes installation, since I already had them installed. Then I pulled the pre-made ROS1 docker image tomoyafujita/ros:noetic  from DockerHub.
Next I followed these instructions here (ROS1 part only): Setup Virtualized Kuberenetes Cluster
So far, so good, everything went well.
After staring the cluster with multiple nodes the tutorial suddenly ends. Therefore my question is "How do I inspect the Kubernetes pods to confirm I have two ROS1 nodes running?"
What I have done so far is to run this command here:
kubectl get pods -A

Which shows all the pods that I have currently running:
NAMESPACE            NAME                                         READY   STATUS    RESTARTS   AGE
kube-system          coredns-565d847f94-2gxsx                     0/1     Pending   0          16m
kube-system          coredns-565d847f94-8k4n5                     0/1     Pending   0          16m
kube-system          etcd-kind-control-plane                      1/1     Running   0          17m
kube-system          kube-apiserver-kind-control-plane            1/1     Running   0          17m
kube-system          kube-controller-manager-kind-control-plane   1/1     Running   0          16m
kube-system          kube-proxy-gcrjt                             1/1     Running   0          16m
kube-system          kube-proxy-stpjf                             1/1     Running   0          16m
kube-system          kube-scheduler-kind-control-plane            1/1     Running   0          16m
local-path-storage   local-path-provisioner-684f458cdd-4w8vt      0/1     Pending   0          16m

I notices that in my output it seems like three nodes are stuck in the Pending state. Regardless, this command provides me the name of the pods. Unfortunately by the name I can't really identify which pod is running a ROS node.
My plan is, once I have the name of the pod where my ROS node is running, to use the kubectl exec command to access the terminal of the pod. For example:
kubectl exec -it <pod-name> -- /bin/bash

If I manage to get inside the terminal of the pod, I would use the rosnode list command to see a list of all active ROS nodes running inside the pod.
My problem is with the command kubectl exec -it <pod-name> -- /bin/bash which doesn't seem to work:
Here are some examples that I have run and the output is right below:
kubectl exec -it etcd-kind-control-plane -- /bin/bash
Error from server (NotFound): pods "etcd-kind-control-plane" not found

kubectl exec -it kube-proxy-gcrjt -- /bin/bash
Error from server (NotFound): pods "kube-proxy-gcrjt" not found

kubectl exec -it kube-proxy-stpjf   -- /bin/bash
Error from server (NotFound): pods "kube-proxy-stpjf" not found

kubectl exec -it coredns-565d847f94-2gxsx  -- /bin/bash
Error from server (NotFound): pods "coredns-565d847f94-2gxsx" not found

kubectl exec -it coredns-565d847f94-8k4n5  -- /bin/bash
Error from server (NotFound): pods "coredns-565d847f94-8k4n5" not found

Is there something I am doing wrong? What am I missing in order to inspect the Kubernetes pods to confirm I have ROS1 nodes running?
Many thanks!

thanks for creating issue, i think you already solved this problem. and which is just a namespace.
kubectl exec -it --tty --stdin -n kube-system kube-proxy-stpjf -- /bin/bash should do the work. be advised that system component pods have kube-system namespace.

Yes the namespace was missing, thank you for your kind advice.
I had also another mistake since I initially skipped the Setup Kuberenetes Cluster document entirely. I skipped it because the initial sentence made me think that the entire document was meant to be the equivalent to the "Setup Virtualized Kuberenetes Cluster document but for physical systems. Therefore, as I already said, I completely skipped the step "Deploy CNI plugin". Later I realized that only section 1 is specific to physical systems and that I have to complete the "Deploy CNI plugin" part also when working with a virtual Kubernetes cluster using single physical host system.
What is still not clear to me is if these sections here:

"Setup Kubernetes API Server"
"Access API-server" and
"Worker Node Join"
are steps specific to physical systems, steps specific to virtual Kubernetes cluster using single physical host system or if they must be complete for both?

I also asume that the Dashboard, described under section "Kubernetes Dashboard" is optional, and the steps work on both cases (physical systems and also virtual Kubernetes cluster in single physical host system).

@rfzeg thanks for the advice, and agree. docs are best effort quality, there are some confusion...


no such host

I can see this message in server log:
lookup localhost<IP_ADDRESS> on xxx: no such host
This code seems to be using the server address thinking it's just a port number.
sever/actions.go
SiteURLPort := *config.ServiceSettings.ListenAddress
URL: fmt.Sprintf("http://localhost%v/plugins/%v/api/complete?token=%s", siteURLPort, manifest.Id, p.configuration.Token

Getting the same error. It's nice that this plugin is promoted by mattermost.com (https://mattermost.com/blog/the-mattermost-teams-10-favorite-productivity-hacks-remote-work/), but it just does not work because of this issue...

@ryota38 @brammeleman it works for us in the community server. what is your config for that settings?

The config.json shows:
{```
"ServiceSettings": {
"SiteURL": "",
"WebsocketURL": "",
"LicenseFileLocation": "",
"ListenAddress": ":8065",
"ConnectionSecurity": "",
"TLSCertFile": "",
"TLSKeyFile": "",
...

Looks like a port is assumed, but the variable name is ListenAddress:
siteURLPort := *config.ServiceSettings.ListenAddress

In combination with:
URL: fmt.Sprintf("http://localhost%v/plugins/%v/api/complete?token=%s", siteURLPort, manifest.Id, p.configuration.Token),

This results in the following error:
"level":"error","ts":1587719469.7949903,"caller":"mlog/log.go:174","msg":"Action integration error","path":"/api/v4/actions/dialogs/submit","request_id":"874cnh1k4bnibykxsg3uo3xrfy","ip_addr":"<IP_ADDRESS>","user_id":"8hfyyi6zi3dwprt4b6jfjesmpw","method":"POST","err_where":"DoActionRequest","http_code":400,"err_details":"err=Post http://localhost<IP_ADDRESS>:8065/plugins/com.cpanato.rent-a-carlos/api/dialog?token=7G6egUxxvWzAPRCukQcSMjGHrMjSci9H: lookup localhost<IP_ADDRESS> on <IP_ADDRESS>:53: no such host"}

So it looks like the value of siteURLPort is set to <IP_ADDRESS>:8065

Thank you for your response.
In my config.json, ListenAddress is "<IP_ADDRESS>:8065".
My mattermost server is setuped with bitnami installer (by other person), I feel maybe some trick in bitnami installer process.
P.S. I'm not familiar with GO, so I just patched hosts file... It's working well.


Set ScratchOrg User Password on Create

Has there been any thought to setting the password for the default scratch org users? Is it possible that OAuth token persistence is required because the default scratch org user has no password, therefore requiring OAuth for even GUI access?
Maybe adding a password parameter to the org:create command?
Maybe some options like this?
sfdx force:org:create --password MyPassword
Some flows might look like this:

POST https://[DevHubOrgDomain].my.salesforce.com/services/data/v51.0/sobjects/ScratchOrgInfo  to create scratch org
GET https://[ScratchOrgDomain].my.salesforce.com/services/oauth2/authorize to get authorization_code
POST https://[ScratchOrgDomain].my.salesforce.com/oauth2/token to get refresh token and initial access token
POST https://[ScratchOrgDomain].my.salesforce.com/services/Soap/s/ to APEX Soap API to set user password using Apex system.setPassword method

All of these steps could be executed in on script without the need to persist tokens.

The password could be set via:

environment variable
script variable

Here are some password options
# create org 1
sfdx force:org:create --password MyPassword123
# create org 2
sfdx force:org:create --password 1234ZuluPassword

Once the password is set its easy to login to the ui and change it if needed. This would level set the security for Production Orgs, Sandboxes, Dev Orgs and Scratch orgs.
The only one that hasn't gotten a password by default are scratch orgs. The null password on scratch org may have been the genesis of persisting access tokens, because frontdoor.jsp and an access token are the only way to access the GUI of a scratch org.


⚠ WSINGRESOS - GENERICO has degraded performance

In d3cb41e, WSINGRESOS - GENERICO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/servicios/wsGenericoPagosWeb.r) experienced degraded performance:

HTTP code: 200
Response time: 7372 ms

Resolved: WSINGRESOS - GENERICO performance has improved in 1392e6c after 7 minutes.


Support override config on call Artifacts.

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
#2549
Description
We compile Artifacts in the development environment. But production we may change the address of our request. For the same Artifacts, our endpoint maybe http://<IP_ADDRESS> or http://dev.service.com. But production it may be a different domain. Or it could be the intranet address of a production cluster.
Fixes #2549
Type of change
Please delete options that are not relevant.

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

Test Environment:

OS: MacOs
@graphql-mesh/...:
NodeJS: 14.17.0

Checklist:

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

I think this would be a great feature, but compared to the {env.VAR} parameter approach it might be a lot easier to "mishandle" the built artifacts, and this becomes only obvious at runtime.
Maybe we could add some more configuration time checks to fail faster?

Closing. And see the discussion in #2549


Picking breaks when moving between screens with different devicePixelRatio

After moving the browser window with a deck.gl visualization between a retina and a non-retina displays the following happens:

picking doesn't work properly
the rendering on retina is very pixelated (non-retina->retina)

Probably this is caused by certain calculations involving devicePixelRatio not being updated. It would be nice if deck.gl could handle this more gracefully. A change in devicePixelRatio could be detected and the necessary calculations could be updated accordingly.

@ilyabo Thanks for the pointer to the matchMedia API. will test and fix.

deck.gl v4.1 should read devicePixelRatio every render, it should be able to adapt without even needing the matchMedia API. @1chandu maybe we can do some tests.

Indeed, this appears to be fixed in v4.1


TitanBrawl doesn't end match right away, and plays dropship sequence

You should probaby move this issue to https://github.com/R2Northstar/NorthstarMods/, since that's the repo with the gameplay scripts

Also 1.1.3 had some evac fixes, you should check if that solved your issue


Autoloading Aliases

Description
This PR adds a new Autoloader property to the config file: $aliases. Aliasing will allow the autoloader to intercept a class request and return an aliased version instead on-the-fly. This will make it easier to transition framework classes with user intervention as well as to replace core classes with user-defined versions.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[X] User guide updated
[X] Conforms to style guide

Okay got the branch issue cleared up, ready for re-review!

I understand why you want to do this, especially with the PSR7 stuff coming up. I guess I'm a little concerned this muddies up when to use this vs using a service. This also wouldn't make services nearly as explicit since you may call one class in the service, but it gets implicitly aliased in a config file. To me this adds a bit of confusion to the framework.
I'm curious to hear other's thoughts.

I would like to hear others' thoughts as well. I wouldn't consider this ideal but it provides two things that we don't have currently:

A way to transition classes that need breaking changes (frequently needed)
A way to replace core classes (frequently requested)

Services can kind of solve the first problem but then you're stuck with the new version in App, which works if you've transitioned to a new namespace but not really if you've tried to do an in-place swap.

Two alternatives that I see:

We control aliases internally and make them conditional on a Config setting. This way it isn't "any class can be any class" but there are a few preset replacements we know are a possibility
We loosen our major version restrictions and start building 5. I know it sucks but the number of BC issues that come up every week could really start piling up to a new version quickly.

My thoughts on this:

Mark this feature as @internal and to be used only for the transitory updating of classes in compliance with PSR7. After the transition is complete, remove this feature with no promise of BC for those who used it.

--or--

Have a set of classes allowed to use this feature so that we control misuse and abuse. Yes, a Config setting would be nice but something that is unreachable from users' tampering.

One more thing to think about: currently there is no way to replace system-defined services. I assume this is a mistake/bug, but it adds another layer of difficulty solved by aliasing.

I second that aliasing would add a layer of confusion. Personally I use Services all over the place and love the simplicity.
Regarding overriding framework classes, in my 4.0.4 project I have "fixed" the Redis TTL bug by creating my own RedisHandler and adding my own class to $validHandlers in app/Config/Cache.php. This allowed me to fix the problem without messing in vendor directory. Happy camper :-)

Thanks all for the feedback. It sounds like we want to stick with our current tools, and avoid adding layers of complexity that something like this aliasing introduces. Closing.


Where can I get the information for each of the filetype?

That depends on the kind of information you are looking for.
Could you specify what you mean by the information?
If you want to know which of the classes are used in which files, then just decompile the entire data_02 archive and use a text search.
If you want to know what the file extensions (FOX2, PARTS etc.) are used for, then you have research that yourself.

I actually meant the latter but those classes are informative too , thanks

I took a short look at the files and came to this result:

Extension
Content

BND
Graph Bounder Data

CLO
Sim Cloth Setting

DES
Destruction

EVF
Event

FOX2
Scene

FSD
Facial Settings Data

LAD
Lip Adjust Data

PARTS
Model Description

PH
Physics Object Description

PHSD
Physics Sound Parameter

SDF
Sound Data File Info

SIM
Simulation Object

TGT
Geometry Target Description

VDP
Vehicle Driving Parameter

VEH
Vehicles

VFXLF
Visual Effects Lense Flare


Fix compilation issues

Description Of Changes
This fixes compilation issues, something about Element not being defined.
Types Of Changes

[x] Bugfix
[ ] New feature
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, appearance (ex) Prettier)
[ ] Build configuration
[ ] Improvement (refactor code)

if you can't apply it, it should be easy enough to fix it in some other way 😄

Hahah, of course we will take it, just making sure how to give you credit

How about Inspired by @aburgel in the commit 😉


media: Fix message controls overlap with message content on mobile.

In mobile devices, message controls were overlapping with the message content
when message is hovered or selected for messages which does not have sender name.
Added top margin to such messages so that there is a clear separation between
controls and content.
Fixes: #11406.
Earlier -

Now -

@overide thanks for working on this!  I'm not sure this is the right solution, though -- it seems like it'll make series of short messages look a lot worse in the mobile web UI.  An alternative proposal would be to mostly hide those message controls, since as an on-hover behavior they're not super usable in any case, and then figure out another way to offer that functionality (possibly just adding "Star message" to the caret menu when in a mobile-side window + having an always-present menu icon).
(As a sidenote, if you can figure out how to fix the clipping of the g in names like Iago, that'd be much appreciated!  Can be a new commit in this PR or another PR)

@timabbott  Thank you for the feedback, I agree with the solution which @thedeveloperr suggested. Other solution would be to move all the options inside the caret menu (I think this is what you are suggesting, right?)

I think doing a (for now, mobile-only, to avoid thinking about the web UI layout changes) restructuring to have the hovered message controls approach might be worth experimenting with (there are a few other ideas worth considering as well).  If we do this right, we can also replace the messy and not super successful attempt at making long-taps work on mobile web (see initialize_long_tap and longtap in that file for this sad code) and just have the behavior be that tapping a message provides this menu that one can interact with to cause a reply or whatnot.
Be sure to read https://github.com/zulip/zulip/issues?q=is%3Aopen+is%3Aissue+label%3A"area%3A+mobile+web" before working on this, so you know what folks are complaining about.
We'd still want the star to appear inline in the message (that's how we show if a message is starred!).

@timabbott so just to be clear our plan should be: We should first make the star control appear persistently on messages (not on-hover). And a long tap for rest of control ?
Did I understand correctly?

That's not what I had in mind -- I was more thinking we eliminate the long-press behavior and make clicking a message trigger the hover behavior in mobile web.
I'm occupied with release management right now; you may want to work on some other things first as I don't have time to think about the details here.  Or do a bit of testing of the mobile web UX for a few other chat tools (Slack, Discord, etc.) and see how they handle interacting with messages and report on it here.

Closing as this was fixed via the recent CSS grid work on the message feed.


Coverage for 'gcloud._apitools.util'

Uses #1189 as a base.

Why remove the # Some systems use a single '*' instead of '*/*'. part?

LMK when you've rebased after the merge

Why remove the # Some systems use a single '' instead of '/*'. part?

Because the calling function already raises an error if the pattern has no slash.

Rebased after merge of #1186.

LGTM
I'll wait for the next rebase ping


Issue with nested shortcodes

I wrote this, so I'm to blame. But my suggested fix is simple.
Example:
{{% highlight go %}}
fmt.Println("foo")
{{< getGoCode >}}
{{% highlight go %}}

As it is now, the result of getCoCode will not get highlighted, as the shortcodes are rendered and processed separately.
I suggest they continue to do so, but make sure that the rendered result of getGoCode is part of the inner argument to highlight.
This means that the result of getGoCode will get a pass through Blackfriday (even if it is marked as "HTML"),  as it is part of hightlight's inner content.
Makes sense, @spf13 ?

This makes sense to me.


[FEATURE] top bits abstraction over movemask on x86

Working item to create a top_bits abstraction.

IS this the companion issue for #386 PR ?

For everything but AVX-512 feature is implemented.
For AVX-512 @jfalcou will fix at some point.

For everything but AVX-512 feature is implemented.
For AVX-512 @jfalcou will fix at some point.

Actually - let's close this one and open a new issue for AVX-512. Why drag issues around.

Actually - let's close this one and open a new issue for AVX-512. Why drag issues around.


Resolve with precondition uses wrong data

Preconditioning currently modifies the QSS class's copy of data. When re-run with preconditioning, that modified data will again be modified, resulting in the wrong objective.

This is no longer an issue now that warm starting has been properly implemented in b2ac15c.


Mpeg2

Hi Friend,
I wanted to ask you: What is the best way to integrate video sources sent via udp protocol and encoded in mpeg2 into ZL?

Now zlmediakit supports RTP streaming of PS, and the encoding format supports H264 / h265 / AAC,the default port is 10000, and the code is in the src / RTP directory.
Do you want to push MPEGTS UDP stream to zlmediakit?

Yes....

??

I'm sorry to tell you that zlmediakit doesn't support MPEGTS streaming now.

If you want to accept the RTP streaming of MPEGTS, you can refer to this code, and then modify the code slightly, and change the PSDecoder object to TSDecoder.

Please refer the code and change ps_demuxer_create to ts_demuxer_create ,ps_demuxer_input to ts_demuxer_input , ps_demuxer_destroy to ts_demuxer_destroy.

If you finally generate H264 / h265 / AAC, you can refer to this code to complete the integration with zlmediakit.

I'm so sorry to reply to you now. I'm still on vacation, there are a lot of things to do....

I made the changes indicated but I receive a dump of errors

Now supported!

Please update the latest code I have implemented mpeg2-ts

Wow... i test it... Thank you very much... you are the best... How are you?


AttachmentData Example

Hi,
This looks great, I've been having a look through the documentation, I've seen the method descriptions.
However, It would be great to see a full example of how to add AttachmentData / files etc.
Thanks,
Jules.

Hi Jules, thank you! This blog post (also on Medium) where I'm going through the implementation of the EmailComposer and then demonstrate how to use it might be helpful for you.

Thanks @gabrieltheodoropoulos
I was really looking for an example project, where I could quickly play around and evaluate.
It seems common to see example code with a package repo.
Specially the file format and mime string etc.

Hi Jules,
To include attachments to the email data, create an instance of the AttachmentData type for every single piece of data or file you want to attach. You must provide it with the following:

The actual data that will be attached to the email. It should be a Data object.
The type of data that you're attaching; that is the mime type and a quick search on the web will give you back lists of mime types. See here for example. It should be  a string value.
A name for the attachment you are adding. That's the file name that the recipient is going to see. It's a plain string value.

Right next you can see an example of making use of the AttachmentData. In this scenario there are two files I want to attach to the email; an image that resides in the Assets catalog, and a PDF file that exists in the application's bundle for convenience.
Once I prepare the image and PDF content as Data objects properly, I initialize two AttachmentData instances where I'm passing that data, the proper mime type as a string, and a file name of my choosing. Then I create the final EmailData instance where I include the attachments, as well as all the other necessary email info.
init() {
    let image = UIImage(named: "avatar")
    let pdfURL = Bundle.main.url(forResource: "colorsFile", withExtension: "pdf")

    guard let imageData = image?.jpegData(compressionQuality: 1),
          let url = pdfURL,
          let pdfData = try? Data(contentsOf: url)
    else { return }

    let imageAttachment = EmailData.AttachmentData(data: imageData,
                                                   mimeType: "image/jpeg",
                                                   fileName: "avatar.jpg")

    let pdfAttachment = EmailData.AttachmentData(data: pdfData,
                                                 mimeType: "application/pdf",
                                                 fileName: "colorsInfo.pdf")

    emailData = EmailData(subject: "Some files for you!",
                          recipients<EMAIL_ADDRESS>                          body: "Hi there, I'm sending you two files to check out.",
                          isBodyHTML: false,
                          attachments: [imageAttachment, pdfAttachment])
}

After that, you're good to go and use the email data in the custom view modifier that sends the email:
Button("Send email") {
    showEmailComposer = true
}
.emailComposer(isPresented: $showEmailComposer,
               emailData: emailData) {
    // Do something when the email composer sheet is dismissed...
}

I hope this helps clear things out.

Awesome, thanks so much.


Add advanced support for splitting strategies for tokens

Function for measuring context windows

A tokenisation algo
number of tokens to split to
Returns a Vec where each string can fit the window.

Hi Will!
Submitted a PR for this, but since it's my first one ever, I don't know how to link it with this issue.


Substitute fails when using named references

A call to
function TRegExpr.Substitute(const ATemplate: RegExprString): RegExprString;
fails with error reeNamedGroupBad when the template contains named references like ${name} defined in the regular expression via (?Pexpression) or (?'name'expression)
Reason:
Substitute calls FindGroupName (within subfunction ParseVarName), and FindGroupName contains this check:
if P >= fRegexEnd then
Error(reeNamedGroupBad);
However, this only works when FindGroupName is called from TRegExpr.ParseAtom, where FindGroupName searches for group names within the regular expression. But when FindGroupName is called from Substitute, it has to search for group names within the ATemplate string, so the above check compares with the wrong end pointer.
Suggested fix:

Change FindGroupName to the following definition:
procedure TRegExpr.FindGroupName(APtr, AEndPtr: PRegExprChar; AEndChar: REChar; var AName: RegExprString);
Change
if P >= fRegexEnd then
to
if P >= AEndPtr then
Change all calls to FindGroupName within ParseAtom to FindGroupName(... , fRegexEnd, ...
Change all calls to FindGroupName within Substitute to FindGroupName(... , TemplateEnd, ...

done as you suggested. thanks for help!
will do the pull-req now. work is in https://github.com/Alexey-T/TRegExpr

Thanks, that's exactly what I meant. Your changes fix the problem.


Convert push and other operations to return Observables

Most Angular 2 developers are writing applications and using libraries that take advantage of the observable pattern more than promises. Because of this, I suggest that the push, remove, and update parameters be changed to return Observables rather than "Thenable" and "firebase.Promise"
As far as converting promises to Observables, it's possible, but with Thenables I don't believe there is any easy support besides hand crafting an Observable yourself.
I am mainly requesting this feature because of the popularity of Observables and to take advantage of additional features from Observables. (If you're using Observables throughout your application, using a promise in one area doesn't make much sense if it can be implemented using Observables)
Version info

Angular: 2.4
Firebase: 3.6.7
**AngularFire:**2.0.0-beta.7
Other (e.g. Ionic/Cordova, Node, browser, operating system): Windows 10

I'm not in favour of this.
Firstly, because it would be a breaking change.
Secondly, it's not as simple as just using the fromPromise operator internally to convert the resultant promise to an observable.
Observables are supposed to be lazy. If a method returns an observable - as with Angular's Http service - the caller could reasonably expect nothing to occur until subscribe is called on the returned observable. If fromPromise were used, that would not be the case with push, et al., as the data would be pushed before - and regardless of - any subsequent subscribe call.
And if a more complicated internal implementation were to be used to make the push call lazy, what would be the meaning of multiple subscribe calls? Multiple pushes? Wouldn't that be confusing?
I think things are clearer and simpler as they are. If you want an observable, just use the fromPromise operator. Also, be aware that promises are understood by RxJS - that is, you can return a promise (instead of an observable) to operators without having to call fromPromse. For example:
http
  .get('some-url')
  .map(response => response.json())
  .mergeMap(data => angularFire.database.list('received-data').push(data))
  .do(() => console.log('Pushed some data successfully.'))
  .catch((error) => console.log(error));

Anyway, that's my two cents.

OK, well the issue with using fromPromise is that push returns a "ThenableReference" instead of a normal promise. And there's no method "Observable.fromThenableReference"

Well, whether or not a Thenable is a promise is going to depend upon what your definition of a promise is.
If your definition is Promises/A+, then I suspect they are promises.
If your definition is a Promise<T> from TypeScript's perspective, then they aren't.
 The following work fine for me with Firebase 3.6.9, AngularFire2 2.0.0-beta.7 and TypeScript 2.1.6:
describe("thenable behaviour", () => {

  it("should be compatible with fromPromise", () => {

    let list = angularFire.database.list("temp/sequence");
    let thenable = list.push({ pushed: true });
    let observable = Observable.fromPromise(thenable);

    return observable
      .toPromise()
      .then((result) => {
        expect(result).to.exist;
        expect(result.key).to.equal(thenable.key);
      });
  });

  it("should be returnable to operators", () => {

    let list = angularFire.database.list("temp/sequence");

    return Observable.of(null)
      .mergeMap(() => list.push({ pushed: true }))
      .toPromise()
      .then((result) => { expect(result).to.exist; });
  });
});

@cartant Thanks for following up. Unfortunately, I'm running the same version of firebase, TypeScript, and rxjs and I get an error in Visual Studio saying
"Argument of type 'ThenableReference' is not assignable to parameter of type 'Promise'.
Property '[Symbol.toStringTag]' is missing in type 'ThenableReference'."

The likely difference is that I am using core-js and @types/core-js and its declaration for Promise<T> is compatible with Firebases's Thenable. I guess you are using something else.
It's a purely TypeScript issue, but I sympathise with your predicament. I see it quite a bit when answering Stack Overflow questions that involve TypeScript.
The good news is that it's really easy to write your own RxJS static fromThenable operator. You could define one like this (in a file named fromThenable.ts):
import { Observable } from "rxjs/Observable";
import { IScheduler } from "rxjs/Scheduler";
import { fromPromise } from "rxjs/observable/fromPromise";
import * as firebase from "firebase";

function staticFromThenable<T>(thenable: firebase.Thenable<T>, scheduler?: IScheduler): Observable<T> {
  const anything: any = thenable;
  return fromPromise(anything as Promise<T>, scheduler);
}

Observable.fromThenable = staticFromThenable;

declare module "rxjs/Observable" {
  namespace Observable {
    export let fromThenable: typeof staticFromThenable;
  }
}

And you'd consume it like this:
import { Observable } from "rxjs/Observable";
import "./fromThenable";

let thenable = angularFire.database.list("some/data").push({ some: "data" });
let observable = Observable.fromThenable(thenable);

nice explanation. Its working from my end.@cartant


test for exclusion mask

This PR resolves #102

So these new "edge cases" are not only edge cases in terms of coordinate order, wild precision, etc. --> The actual index order in the resultant dataframes are different. I have confirmed that the returned geometries are equivalent (within tolerance) across OSs, both visually and programmatically – but in different orders.

macOS (ARM) vs. Ubuntu -- complete

macOS (ARM) vs. Ubuntu -- "bad" subset

Index sort


dotnet run doesn't print out hello world

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/run
Operating System: linux
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you share what dotnet --info returns?
@baronfel any suggestions here on why dotnet run doesn't return anything?