Rank N Polymorphism / unification issue

This is a neat one I think; consider the following module:
module Rec where

type Ext a = { a | x : Int }

data Foo = Bar (Ext { y : Int })
         | Baz (Ext { z : Int })

mapExtFoo : (Ext a -> Ext a) -> Foo -> Foo
mapExtFoo f foo = case foo of
  Bar bar -> Bar <| f bar
  Baz baz -> Baz <| f baz

As both "branches" of the Foo data structure contain things that are Ext, if we have a function that maps Ext to Ext, we should be able to apply it independently to both sides, and thus apply it to Foo as a whole.
I think the issue here is that the f argument wants to unify to two different types, but notice that it can uniquely unify within the scope of each case branch. Compiler output below:
[1 of 1] Compiling Rec                 ( Rec.elm )
Type error on line 12:
Something weird is happening with this value:

        baz

   Expected Type: {a | z : Int}
     Actual Type: {}

Type error on line 12:
Something weird is happening with this value:

        baz

   Expected Type: {}
     Actual Type: {a | y : Int}

Type error on line 12:
Something weird is happening with this value:

        Baz <| (f baz)

   Expected Type: {a | z : Int}
     Actual Type: {}

Type error on line 12:
Something weird is happening with this value:

        Baz <| (f baz)

   Expected Type: {}
     Actual Type: {a | y : Int}

Type error between lines 10 and 12:
Cannot unify rigid type variable 'a'.
It is likely that a type annotation is not general enough.

        case foo of
          Bar bar -> Bar <| (f bar)
          Baz baz -> Baz <| (f baz)

   Expected Type: a
     Actual Type: {y : Int}

Type error between lines 10 and 12:
Cannot unify rigid type variable 'a'.
It is likely that a type annotation is not general enough.

        case foo of
          Bar bar -> Bar <| (f bar)
          Baz baz -> Baz <| (f baz)

   Expected Type: a
     Actual Type: {y : Int}

I'm doing this thing where I make meta issues to track a category of related ideas. Often we have a bunch of different threads open that are mutually exclusive, or depend on each other in tricky ways. This is described more here.
So I am going to close this, but any discussion should continue here!


Add screenshots to README

I can take this.

I have taken these two screenshots.

They're good, but they would be better if you provide a movie with a more humane title, not a code like "IMG _0684". Rest is awesome! Make these changes and commit!

Sure, thing. I don't have any other vids on my machine unfortunately so it will be this, but I can certainly remain. PR incoming 👍

You mean rename, right? Can't wait for the PR!

Ok, how is this @schedutron?

Awesome. For now, make a PR for this and I will accept, but perhaps I would have to modify these in the future once the front-end of the app updates. Thanks!

https://github.com/mohan-mu/home-server/blob/master/README.md

@pjhampton Please make a commit so that I can pull your original contributions.

@schedutron I did - https://github.com/schedutron/home-server/pull/3


Fix scrollbars sizing on MacOS

On MacOS, the source code view always has these scrollbars:

Slack discussion: https://clojurians.slack.com/archives/C03KZ3XT0CF/p1710986919516229?thread_ts=1710963300.401009&cid=C03KZ3XT0CF.

This I would really appreciate a PR, since I can't reproduce it on Linux. I mean, I only see the scrollbars when there is something to scroll because the pane is not big enough to show all the code, but that is expected, nothing to fix there.

Let me slightly update my observations. So when there is nothing to scroll vertically, no scrollbars are shown. But once we have more lines than fit in the screen, both the vertical and the horizontal scrollbars appear. Where horizontal appears to compensate for the space taken by the vertical scrollbar. Ideally, the view should use a narrower wrapping width so that only the vertical scrollbar is necessary. Maybe switch it to be always there?
I can try making a PR too once you confirm that on Linux you have a different experience.

@mrkam2 oh yeah, I see that behavior on Linux too. Would appreciate a PR if you know what is going on already.


The download link for the pre-trained backbone weights is invalid

Hello, the download link for the pre-trained backbone weights seems to be invalid, and some other links seem to be invalid too.

@karfly yo, evacuation from gdrive seems to have impacted the repo

Hi!
I've just updated urls in all READMEs. Now links are valid


fix(Header): correct shorthand props API

Fixes #454
The Header did not conform the latest shorthand props API standards.  Namely, it prevented users from accessing the content div and allowed use of icon/image shorthand props along with children.
This PR adds Header.Content, docs, tests, and refactors the Header to correctly handle the shorthand props API.

Released in<EMAIL_ADDRESS>


Opening RibbonButton drop down using alt key doesn't work

I can't get the drop down list of a RibbonButton to open using alt key. I double checked that all AltKey properties on the form are unique, but nothing happens when is use the keyboard shortcut assigned to the drop down button.

Thanks for reporting this, I did notice this issue some hours ago for myself but was unsure about it.
I'll take a look at it ASAP.

@adriancs2: Could you test this on older Windows versions? On Win10, at least, Keyboard Handling doesn't seem to work at all.
Is this a known issue?

I have checked this in Windows 7, the Alt key for Button-DropDown is not firing too


Add annotations for ClientConfigurationData

Master Issue: #10041
Motivation
In order to automatically generate documents, we need to add annotations to the code first
Modifications
Since FieldContext is in the broker-common module, we can only use ApiModelProperty.

@Anonymitaet Please help check the doc in code

@Anonymitaet PTAL

ClientConfigurationData is not really part of the public API. Instead users will use PulsarClient.builder()

This config is set by the builder, and each field corresponds to it.  Or annotate the Builder?

The existing documentation looks like a description of the config object.
Any suggestions? @merlimat
https://pulsar.apache.org/docs/en/next/reference-configuration/#client


NameError: name 'xrange' is not defined

I got this error while going through the documentation.
/usr/local/lib/python3.4/dist-packages/spectra/core.py in range(self, count)
99         distance = dom[-1] - dom[0]
100         props = [ self(dom[0] + distance * float(x)/(count-1))
--> 101             for x in xrange(count) ]
102         return props
NameError: name 'xrange' is not defined

Thanks for flagging, @cameron-finn! I goofed up on the python3 support. Should be fixed now, in v0.0.4: https://github.com/jsvine/spectra/commit/873f8b763cfa761d1d78886f5e65ecd3caabf5b6. Let me know if you run into any other bugs.


Add cross-owner package copying

This PR adds the ability to copy a package from one owner to another on anaconda.org. I've changed  the unit test for this case appropriately and took a shot at adding an integration test for the new function I added in inspect_binstar.
There one aspect of the changes I made that I don't like, will make a line comment in a moment.

@pelson -- friendly bump...

Thanks for pinging again @mwcraig. I was away, but certainly needed this being pushed up my notifications 👍

@pelson -- think I addressed everything!

Thanks @mwcraig! :+1:


template version with re2c scanner

Changed: In the generic class basic_json, all JSON value types (array, object, string, bool, integer number, and floating-point) are now templated. That is, you can choose whether you like a std::list for your arrays or an std::unordered_map for your objects. The specialization json sets some reasonable defaults.
Changed: The library now consists of a single header, called json.hpp. Consequently, build systems such as Automake or CMake are not any longer required.
Changed: The deserialization is now supported by a lexer generated with re2c from file src/json.hpp.re2c. As a result, we follow the JSON specification more strictly. Note neither the tool re2c nor its input are required to use the class.
Added: The library now satisfies the ReversibleContainer requirement. It hence provides four different iterators (iterator, const_iterator, reverse_iterator, and const_reverse_iterator), comparison functions, swap(), size(), max_size(), and empty() member functions.
Added: The class uses user-defined allocators which default to std::allocator, but can be templated via parameter Allocator.
Added: To simplify pretty-printing, the std::setw stream manipulator has been overloaded to set the desired indentation. Pretty-printing a JSON object j is as simple as `std::cout << std::setw(4) << j << '\n'.
Changed: The type json::value_t::number is now called json::value_t::number_integer to be more symmetric compared to json::value_t::number_float.
Removed: The key() and value() member functions for object iterators were nonstandard and yielded more problems than benefits. They were removed from the library.

Coverage remained the same at 100.0% when pulling c8c49dae5d4705a7e9ea57c09b38b0eac2c4516c on template into c4d74856bbfc99ce788bebfac981377f5cfc6352 on master.


Question: Using validate_responses=True in connexion 3.*

Description
We are migrating all our services to connexion 3.*. In connexion 2.*, we have used validate_responses=True.
In 2.*, response values specified as type integer but were internally handeled as type Decimal got an implicit cast to integer in the response object and the response validation passed.
In 3.*, the same specification will respond 500 Response body does not conform to specification. 1.0 is not of type 'integer'. The implicit type casting does no longer occur.
I have searched the docs to find how to keep the 2.* behaviour. Do you have any migration guidance?

This is related to register json decoder and the question is invalid


Add signup page.

i had added signup page, also added the facebook, twitter, google sign feature to it.

The styles for the icons and button in the navbar haven't been rendered correctly. Also, the signup card is overlapping with the navbar a bit which you might want to change. The rest looks good!

Ok, I will work on it.


Make the Mullvad info a standalone feature

Feature Description:
Your extension basically duplicates the existing Ubuntu drop down menu and if the mullvad icon is added to the top bar, that becomes clear.  The Mullvad info is yet another click away.  A dedicated drop down showing the Mullvad connection information from the top bar, without all the shutdown/network/etc duplication would be more usable.  Wouldn't that be easier for you to do?  You can do better than I can, but in my mind, one click is better than two.  Otherwise good job, it works.

Could you elaborate what you mean by this? Perhaps draft some screenshots of exactly what you mean.

[Bildschirmfoto von 2021-02-07 09-20-51.png]
With the extension's settings "display indicator icon" set to on, whether you click on the shut down icon or the Mullvad Icon, you get the same result, the same submenu, both of which have another Mullvad icon that needs to be clicked on to see the connection details. So, there is a rather annoying duplication of shutdown menus just to gain access to Mullvad connection details.
[Bildschirmfoto von 2021-02-07 09-23-50.png]
With the extension's settings "display indicator icon" set to OFF, no Mullvad icon appears except a level deeper in the shut down menu. Nothing in the tray. But the duplication of shutdown menus is reduced to one instance. The trouble with this is that there is no indication that there is a Mullvad extension even available unless you know it is there already and click on the only instance of a shut down menu to find it.
My suggestion was to show a single Mullvad icon in the tray, with the the dropdown showing only the connection details that users expected to see when they installed the extension. That is please don't bury the extension in the shutdown menu, then again in a submenu.
Look, you've done a great job. The extension works flawlessly and I can't say the same for many many other extensions I've tried that won't even work let alone not crash the computer.
I use Mullvad and I don't like having the app up on the deskotp. I'd rather have a simple one click tray icon to show me what server I'm connected to. This I'd find very useful. Sorting through the temporary confusion of duplicate shutdown menus to find Mullvad server details buried two levels deeper wasn't a deal breaker, but for many, this would be one click too many.
No criticism intended, it's just how I find it and if my suggestion helps, more users would find it useful. As would I. Thanks.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
Am Samstag, 6. Februar 2021 13:11 schrieb Michael Pobega<EMAIL_ADDRESS>
Could you elaborate what you mean by this? Perhaps draft some screenshots of exactly what you mean.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I think your screenshots didn't attach properly so I'm still a bit confused about the suggestion, sorry.
If I'm understanding correctly what you want though, there is an earlier version that was a tray icon with it's own popout rather than integrated with the Gnome system tray. IIRC you could try that out at commit 7ba751535bfe9520dd8f2c72814cb1688838b71d I believe.
The 'reason' for it being the way it is now is just to integrate smoothly and feel like a Gnome-native application. The primary use case (at least during design) was just to have the icon in the tray to let you know if/when you get disconnected from your VPN.
And don't worry, I'm not offended, criticism is great. I'm just trying to understand exactly what you're asking for.

I believe you got the point already. I'll check out your link. As for the photos I'd attached, they were embedded, but also attached separately. I don't know why you had the difficulty, nevertheless, unembedded, I've attached them again.
Thanks,
P
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
Am Sonntag, 7. Februar 2021 10:17 schrieb Michael Pobega<EMAIL_ADDRESS>
I think your screenshots didn't attach properly so I'm still a bit confused about the suggestion, sorry.
If I'm understanding correctly what you want though, there is an earlier version that was a tray icon with it's own popout rather than integrated with the Gnome system tray. IIRC you could try that out at commit 7ba7515 I believe.
The 'reason' for it being the way it is now is just to integrate smoothly and feel like a Gnome-native application. The primary use case (at least during design) was just to have the icon in the tray to let you know if/when you get disconnected from your VPN.
And don't worry, I'm not offended, criticism is great. I'm just trying to understand exactly what you're asking for.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I believe the screenshots are being lost because Github doesn't seem to properly support email attachments. Please attach them at https://github.com/Pobega/gnome-shell-extension-mullvad-indicator/issues/16 via the web interface

I believe this is fixed in the current release by https://github.com/Pobega/gnome-shell-extension-mullvad-indicator/issues/21


stop polluting EventEmitter

vows pollutes the global EventEmitter when it wraps emit().
this causes issues in tests of code that also wraps emit() (such as EventReactor).
this patch simply replaces use of the global EventEmitter with EventEmitter2 -- problem solved.

Fix all the bugs!


Allow functional tests with content to run from VS

cc @dougbu

@pranavkm isn't this a dupe of #4543? I plan to do that one soon. Thought perhaps that copying an approach you'd taken in another repo (Entropy?) might do the trick.

Didn't realize there was an issue for it. I ran into it when I was trying to debug some functional tests. The directory walk is copied from Entropy.

Ooops, somehow thought this was a new issue. I'll reassign #4543 to you and review this tomorrow morning as the fix for that.

:shipit: w/ one improved name.

@pranavkm can this PR be closed now that 43c75c9 is in?

Done.


异步线程异常，使用的是openai_api_server.py这个文件

System Info / 系統信息
A30 24G CUDA12.4  Docker 容器
Who can help? / 谁可以帮助到您？
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/starlette/_exception_handler.py", line 53, in wrapped_app
glm4_vllm_api_server-glm4_vllm_server-1  |     await app(scope, receive, sender)
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/starlette/routing.py", line 72, in app
glm4_vllm_api_server-glm4_vllm_server-1  |     response = await func(request)
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/fastapi/routing.py", line 278, in app
glm4_vllm_api_server-glm4_vllm_server-1  |     raw_response = await run_endpoint_function(
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/fastapi/routing.py", line 191, in run_endpoint_function
glm4_vllm_api_server-glm4_vllm_server-1  |     return await dependant.call(**values)
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/glm4/basic_demo/openai_api_server.py", line 317, in create_chat_completion
glm4_vllm_api_server-glm4_vllm_server-1  |     output = await anext(predict_stream_generator)
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/glm4/basic_demo/openai_api_server.py", line 458, in predict_stream
glm4_vllm_api_server-glm4_vllm_server-1  |     async  for new_response in generate_stream_glm4(gen_params):
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/glm4/basic_demo/openai_api_server.py", line 199, in generate_stream_glm4
glm4_vllm_api_server-glm4_vllm_server-1  |     async for output in engine.generate(inputs=inputs, sampling_params=sampling_params, request_id="glm-4-9b"):
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/vllm/engine/async_llm_engine.py", line 662, in generate
glm4_vllm_api_server-glm4_vllm_server-1  |     async for output in self._process_request(
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/vllm/engine/async_llm_engine.py", line 756, in _process_request
glm4_vllm_api_server-glm4_vllm_server-1  |     stream = await self.add_request(
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/vllm/engine/async_llm_engine.py", line 561, in add_request
glm4_vllm_api_server-glm4_vllm_server-1  |     self.start_background_loop()
glm4_vllm_api_server-glm4_vllm_server-1  |   File "/usr/local/lib/python3.10/dist-packages/vllm/engine/async_llm_engine.py", line 431, in start_background_loop
glm4_vllm_api_server-glm4_vllm_server-1  |     raise AsyncEngineDeadError(
glm4_vllm_api_server-glm4_vllm_server-1  | vllm.engine.async_llm_engine.AsyncEngineDeadError: Background loop has errored already
Information / 问题信息

[ ] The official example scripts / 官方的示例脚本
[ ] My own modified scripts / 我自己修改的脚本和任务

Reproduction / 复现过程
多点访问量即可复现
Expected behavior / 期待表现
希望能修复这个bug

emm这个是单并发的呀，没有做并发的任务管理的


[AutoPR cognitiveservices/data-plane/FormRecognizer] FRv20-preview- minor updates

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7760

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/8414


Bookmarks:The book name and chapter should not have a colon between them

Issue description : The book name and chapter in the bookmarks should not be separated by a colon
Steps:
From the drawer bar, select Bookmarks
Select  the version and chapter to be bookmarked.
Observation:
In the bookmark heading, the book and chapter are separated by colon , Eg John :16
Expected Result:
The book and chapter are to only have a space and the colon is to be removed. Eg John 16

@neetuy Any update on this issue?

This issue is fixed Fixed

Verified that the colon  between the bookname and chapter is removed.


Change how setting up the bot works

We have discussed how we want to change the bot's setup workflow to use buttons instead of multiple commands as we have it right now. Here is a list of what should change.
Instead of having a config command with multiple subcommands, we should create one command that displays a button per configurable item or a drop down list with option for each. Here is an inspiration of how it could look.
Configuring notifications
Show a channel selection and a disable button or a channel selection option depending on if a channel has already been set up or not.
Configuring a group
Show an integer input option and a disable button or just an integer option depending on if a group has been configured already.
Configuring an rsn
I propose we move setting an rsn to this command as well and make it clear that setting an rsn doesn't mean linking your discord with your in-game account. So we show a string option and a disable button or just a string option depending on if an rsn has already been configured.
Currently the /setrsn command is a little confusing and is leading to people thinking it somehow links their in-game account with their discord which isn't true because multiple people can configure the same rsn. We should make it clear that configuring an rsn only sets a default rsn for the other commands that require an rsn as input.
Other
If a user has admin permissions all options should be available. If the user doesn't have admin permissions only the configuration of an rsn should be available.
When this implemented, we should not immediately remove the currently available commands but rather turn them into a message saying that the configuration of the bot has been moved. The message could has ephemeral set to true so they only show up to the one who sent the command.
What should the new command be? Keep it /config or change it to something else, such as /setup?

I like changing it to /setup , so that it's clear that it would be the 1 entry point for any server admin to setup our bot. I'm not as big of a fan of including /setrsn in that same command, I think it should be a separate command, with very intentional messaging, more on this at the end.

Configuring notifications makes a lot of sense to me, but to expand on that a little (and maybe tweak some of the language), it could be:
1 - run /setup
2 - select "Notifications" (category)
3 - select the notification type from a follow-up dropdown (much like how hom has follow-up hidden messages)
4 - A message would show up, telling you the current configuration for that notification type, and actions below:
4a - if notification type is already selected, two buttons: Change channel and Disable
4b - if notification type isn't selected, one button: Select channel
5 - change/select channel actions would give the user a dropdown of all text channels, and on select it would give them a confirmation message (it would also send an empty msg to that channel to test if it's available, current code already does this)

Configuring group is pretty similar, except i dont think you have have an input and a button side to side, right? so Maybe it would be two buttons Change group and Remove group or just Select group if unconfigured

Note, maybe this input could support ID and wom URL? not sure, or at least just some quick message on where they can find the group ID (group pages now contain this on the header)

I think setrsn (or aliases) needs to be evaluated separately (after config->setup update), I believe a lot of players think this is some kinda of magical linking between their OSRS acc and discord, but it's really supposed to only be used as a default username for player commands. I think it should be a separate command, but still be updated to be a lot more clear on what it actually does. For now it's best to focus on just turning /config* into /setup and we'll worry about setrsn later imo.

I 100% agree that we shouldn't remove the current commands, and just use them as empty "redirect" messages to the new /setuo command


About page revamp

Related Issue

Closes: #33
Description of Changes
Revamped the complete About page, added the contents into a container and made it completely responsive with no margin issues.
Checklist:

[x ] My code adheres to the established style guidelines of this project.
[ x] I have conducted a self-review of my code.
[ ] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[ x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Original

Updated

Please provide any necessary screenshots to illustrate the changes made.
This is my third pr for the same issue, please share your reviews, the last review was terribly vague so added changes according to my discretion.
Thank you

@AKD-01 @srivastavaritik

Whenever ready, please label it as level2
Thank you

Any update?Almost a month

seriously any update?
almost 2 months now, like any update seriously

Literally a timepass project, utter waste of time


chore(DmaBuf): Add a helper to dump contents of DmaBuf

Helper utility for debugging the contents of dma buffer used in io-engine. This dumps the buffer data in traces.

@tiagolobocastro @dsavitskiy I was using this to debug some workflow. If you think this can be helpful in general, we can merge this.

bors try


Automate the process of discovering the apiDocumentation

At present, the agent has been hard coded to fetch the API Doc by appending /vocab to the URL provided. This not aligned with Hydra Spec. The agent should be able to extract the link to API documentation from the homepage of API publisher by looking for Link header in HTTP response with relationship type http://www.w3.org/ns/hydra/core#apiDocumentation

I am really new to hydra ecosystem but I would like to work on this, it might take me some time though (since I am new).

I'd recommend you go through the core vocab and get acquainted with the issue. Here's the link -> https://www.hydra-cg.com/spec/latest/core/](Hydra Core Vocabulary)

Thank you, I will

Welcome @Abhimanyu121, first take some time to get familiar with the concepts at http://www.hydraecosystem.org/ and other doubts you might have. We are available at Gitter.
When you feel a bit more comfortable, I have a tip for the issue. Currently, the Agent simply uses vocab(line 29, graphutils_operations.py):

However, the agent has already the ApiDoc available and you can go to self.api_doc['context']['context']['vocab'] and get the full address for the vocab :)

Thank you @Guttz , I will get familiar with all the concepts and make a pr soon :D

Can I work on this?

You don't need to ask for permission, feel free to work on anything you want.

I'd recommend you go through the core vocab and get acquainted with the issue. Here's the link -> Hydra Core Vocabulary

@vcode11 kindly take a look at this. You'll find it helpful

Can I work on this issue , I am new to hydra . I have set up the env and went through the docs.

No need to ask @kss682.

Is this issue resolved? @xadahiya @Guttz


data_prep.py stuck

I run data_prep.py, but it stuck during parallel downloading.
In my case, it occurs after 1077 (out of 6778, at most in several trials) chair models were downloaded.
How can I fix this?
Are there any workaround?
Thank you.

I don't know why, but it just worked after some more trials.
Maybe it's because of network issues.
Sorry for bothering.
Thank you for your very interesting work.


Please update MCOMEETADV to MCOMEETADD on this page

[Hi guys, I see that this page was recently updated. Since 3rd party ACP providers are no longer supported we need to change the Sku name listed from MCOMEETADV to MCOMEETADD . see also https://dev.azure.com/Supportability/UC/_wiki/wikis/UC.wiki/141864/Automatically-replace-3rd-party-ACP-providers]

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing - Azure AD
Content Source: articles/active-directory/enterprise-users/licensing-service-plan-reference.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @daveba
Microsoft Alias: daveba

@richardojel
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#reassign @Nicholak-MS
Nick - can you also update the author metadata to you so you get these issues directly?

Hi @richardojel,
I need a bit more information here as I am not following why we would change the name of the string for that product. Our catalog still shows the GUID: 0c266dff-15dd-4b49-8397-2bb16070ed52 as Microsoft 365 Audio Conferencing and MCOMEETADV. This is where the information in the Service Plans document is sourced from.  MCOMEETADD is a completely separate product.

@richardojel
Can you please provide the information requested?

Closing. Will reopen upon response.
#please-close


Updating a committee does not send the new title

Updating a committee form /committees/$/edit-committee does not send the title field.
The current payload is:
{
  "action": "committee.update",
  "data": [
    {
      "id": 2,
      "description": "",
      "organization_tag_ids": [],
      "user_ids": [2, 3],
      "forward_to_committee_ids": [],
      "receive_forwardings_from_committee_ids": [1]
    }
  ]
}

Duplicate of #324. -> will be fixed with #330.


Default process ID when PDTV=1

When PDTV is 1 and a transaction is received without a process_id then
then there is no ability to specify a VS-stage (when G-stage is active) or
S-stage (when G-stage is not active) page table to translate the transaction.
Proposed:

Add a configuration bit to DC.tc - DPE - default process_id enabled
Defined process_id value of 0 as the default process_id when DPE is 1
and the transaction does not have a process_id. The default value of 0
is already default in OS like Linux and windows.

Can't we consider that the first stage mode is bare when no process_id is provided with PDTV=1 ?

So I was not clear when I said that there is no ability to specify a VS-stage or S-stage page table. The issue is highlighting that the process will assume Bare when no process_id is provided with PDTV=1.  When the process assumes Bare, it allows a device under such configuration to access all SPA (when G-stage is not active) and all GPA when (G-stage is active) and the hypervisor and guest OS, respectively, lose the ability to control the DMA from devices in such configuration.

Issue approved in IOMMU TG 9/22/22


Installation for systemd

Hallo,
unfortunately I'm very new to linux and didn't get the installation part running.
I found out, that my system is not using init.d, but system.d. So I'm not really sure what to do, to get the installtion part running?
cd /etc/init.d/
#adjust to correct, absolute path below
sudo ln -s ~/odroid-xu4-fan-control /etc/init.d/odroid-fan-controller
sudo update-rc.d odroid-fan-controller defaults

do I have to change the location to /usr/lib/systemd/system/ ?

So, I think linking doesn't work for system.d, or does it?
Do you know how to do?
Thank you.

I'm not running systemd, I will have to research what to do.

Yesterday I was very busy finding a solution for that, but it still doesn't work.
I tried to write a .service file called odroidxu4fancontrol.service for systemd

[Unit]
Description=Enable service provided by daemon.
After=remote-fs.target
[Service]
ExecStart=/home/odroid/odroid-xu4-fan-control/odroid-xu4-fan-control.sh
Type=forking
PIDFile=/var/run/odroidxu4fancontrol.pid
[Install]
WantedBy=multi-user.target

and saved it in /usr/lib/systemd/system and then run:
>$ sudo systemctl restart odroidxu4fancontrol.service
But it was hanging very long and when it responded the answer was:
Job for odroidxu4fancontrol.service failed because a timeout was exceeded. See "systemctl status odroidxu4fancontrol.service" and "journalctl -xe" for details.

So I tried systemctl status odroidxu4fancontrol.service and it said:
* odroidxu4fancontrol.service - Enable service provided by daemon.
Loaded: loaded (/usr/lib/systemd/system/odroidxu4fancontrol.service; enabled; vendor preset: disabled)
Active: failed (Result: timeout) since Sun 2018-06-03 14:02:53 UTC; 3min 8s ago
Process: 4360 ExecStart=/home/odroid/odroid-xu4-fan-control/odroid-xu4-fan-control.sh (code=killed, signal=TERM)
Jun 03 14:01:23 alarm systemd[1]: Starting Enable service provided by daemon....
Jun 03 14:01:23 alarm odroid-xu4-fan-control.sh[4360]: fan control started. Current max temp: 62000
Jun 03 14:01:23 alarm odroid-xu4-fan-control.sh[4360]: For more logs see:
Jun 03 14:01:23 alarm odroid-xu4-fan-control.sh[4360]: sudo tail -f /var/log/syslog
Jun 03 14:02:53 alarm systemd[1]: odroidxu4fancontrol.service: Start operation timed out. Terminating.
Jun 03 14:02:53 alarm systemd[1]: odroidxu4fancontrol.service: Failed with result 'timeout'.
Jun 03 14:02:53 alarm systemd[1]: Failed to start Enable service provided by daemon..

Maybe someone is familiar with systemd and can help me?

Why don't you simplify your service file like this ?
[Unit]
Description=Simple shell script to better control Odroid XU4 fan

[Service]
ExecStart=/usr/sbin/odroid-xu4-fan-control.sh

[Install]
WantedBy=multi-user.target

@QCMR Looks fine to me, does it work?

@QCMR Thank you very much! I think my approach was a bit to complexed although the answer is very simple. I'm very new to linux and systemd.
@f1vefour Yes, it seems to work now.
So, what I did was just to put your odroid-xu4-fan-control.sh file into /usr/sbin/ and then write a .service script in /usr/lib/systemd/system with QCMRs content. After all files are saved I run the command sudo systemctl enable odroidxu4fancontrol.service and sudo systemctl restart odroidxu4fancontrol.service and it works! To change temperature settings, I just edited /usr/sbin/odroid-xu4-fan-control.sh file.
But now I have some other problem. I don't know what caused it, but when my fan is quiet I hear a very noise high pitched beep tone, coming out from my odroid xu4. Do you have any idea what caused it?

What does this say when it's making that noise?

```cat /sys/devices/platform/pwm-fan/hwmon/hwmon0/pwm1

it just says:
80

Raise the 80's in the script to 100

@ubunonymount When the fan doesn't have enough energy (enough PWM value) to make it spin up it makes that squeaking noise. If you would have spun the fan while it makes that noise it would have began spinning.
Nothing to worry about, as I said each XU4 fan is different and requires a different minimum PWM value to start. Never seen one require over 80 but as long as it's working.

Ahh, okay, thank you very much!!! 👍

Sorry, I was off.
Yes, it works and as I'm crazy and I live in a hot country I slighltly increased the number of levels to make the speed very linear.
Note to @ubunonymount, I don't have the squizzing noise because I replaced the original fan by a tiny Zalman + a copper heat sink
new_fan_speed=0
  if (( ${current_max_temp} >= 75000)); then
    new_fan_speed=255
  elif (( ${current_max_temp} >= 73000)); then
    new_fan_speed=234
  elif (( ${current_max_temp} >= 71000)); then
    new_fan_speed=214
  elif (( ${current_max_temp} >= 69000)); then
    new_fan_speed=193
  elif (( ${current_max_temp} >= 67000)); then
    new_fan_speed=173
  elif (( ${current_max_temp} >= 65000)); then
    new_fan_speed=153
  elif (( ${current_max_temp} >= 63000)); then
    new_fan_speed=132
  elif (( ${current_max_temp} >= 61000)); then
    new_fan_speed=112
  elif (( ${current_max_temp} >= 59000)); then
    new_fan_speed=91
  elif (( ${current_max_temp} >= 57000)); then
    new_fan_speed=71
  elif (( ${current_max_temp} >= 55000)); then
    new_fan_speed=51
  elif (( ${current_max_temp} >= 53000)); then
    new_fan_speed=30
  elif (( ${current_max_temp} >= 51000)); then
    new_fan_speed=10
  elif (( ${current_max_temp} >= 49000)); then
    new_fan_speed=0
  else
    new_fan_speed=0
  fi


blt_cuda_smoke compiled with default arch

I noticed that blt_cuda_smoke is built with default arch, while most of the code I want to test is built with extra arch flags.
That can lead to a blt_cuda_smoke that works when other exes do not.
This is a BLT usage thing we should make folks aware of.

This is somewhat related, but with CMake 3.21.1, I get a warning regarding the cuda smoke test:
CMake Warning (dev) in cmake/blt/tests/smoke/CMakeLists.txt:
Policy CMP0104 is not set: CMAKE_CUDA_ARCHITECTURES now detected for NVCC,
empty CUDA_ARCHITECTURES not allowed.  Run "cmake --help-policy CMP0104"
for policy details.  Use the cmake_policy command to set the policy and
suppress this warning.
CUDA_ARCHITECTURES is empty for target "blt_cuda_openmp_smoke".
This warning is for project developers.  Use -Wno-dev to suppress it.

This is something we should not be setting directly in BLT because it is platform specific.  Most projects have this in their generated host-configs:
set(CMAKE_CUDA_ARCHITECTURES "70" CACHE STRING "")

This CMake property was added in CMake 3.18 https://cmake.org/cmake/help/git-stage/variable/CMAKE_CUDA_ARCHITECTURES.html


doc: Release notes initial (empty) draft

Get things started for the 1.4 release notes.
Individual teams will update their appropriate sections.
Signed-off-by: David B. Kinder<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #14445   +/-   ##
=======================================
  Coverage   51.98%   51.98%
=======================================
  Files         308      308
  Lines       45505    45505
  Branches    10543    10543
=======================================
  Hits        23655    23655
  Misses      17050    17050
  Partials     4800     4800

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 f992e97...1fa9d5f. Read the comment docs.


Bluetooth: BAP: Support setting different values per dir in CIG

The interval and latency for a CIG are set for each direction now, allowing applications to use e.g. 10ms for sink ASEs and 7.5ms for source ASEs.
fixes https://github.com/zephyrproject-rtos/zephyr/issues/64186

CI will fail until https://github.com/zephyrproject-rtos/zephyr/issues/74544 is fixed or the test is disabled.

@carlescufi Why was this closed?
This PR is implementing a feature in the host and https://github.com/zephyrproject-rtos/zephyr/pull/76453 is fixing something in the controller.

@carlescufi Why was this closed?
This PR is implementing a feature in the host and #76453 is fixing something in the controller.

carlescufi closed this in https://github.com/zephyrproject-rtos/zephyr/pull/76453

Probably because Github is not that smart :), and writing "fixes #*" causes that something to be closed even if it is another PR


SQL format changes and enhancements

Description of changes:
Changes to SQL formatting;annotation changes in .sql files
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Tested site functionality across all pages
Adjusted files fall within single lab and Cost Pillar


compile error on msgraph_onedrive.pks

Describe the bug
error when compiling, msgraph_onedrive.pks, saying that in line 178 folder_child_count must be defined.
I think maybe you can fix by replacing v_value in these that line and the following line with v_items(v_items.count) like so:
IF p_recursive = 'Y' AND v_items(v_items.count).folder_child_count > 0 THEN
        v_child_items := list_folder_children ( p_drive_id => p_drive_id, p_parent_item_id => v_items(v_items.count).id, p_include_parent => 'N', p_recursive => 'Y' );

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Wow, that was fast!  Thanks for putting this together, by the way.  I’ve been doing cobbled together pieces of this myself, it’s so great that you built something like this!
Warren Caples | HX5 | Director, Management Information Systems
O:<PHONE_NUMBER> | M:<PHONE_NUMBER> |<EMAIL_ADDRESS>212 Eglin Parkway SE | Fort Walton Beach, FL 32548
HXFIVE.COM
NOTICE - This communication may contain confidential and privileged information that is for the sole use of the intended recipient. Any viewing, copying or distribution of, or reliance on this message by unintended recipients is strictly prohibited. If you have received this message in error, please notify us immediately by replying to the message and deleting it from your computer.
From: Paul Jereb @.>
Sent: Thursday, November 16, 2023 11:49 AM
To: cdngmnks/msgraph-sdk-plsql @.>
Cc: Warren Caples @.>; Author @.>
Subject: Re: [cdngmnks/msgraph-sdk-plsql] compile error on msgraph_onedrive.pks (Issue #25)
Closed #25https://github.com/cdngmnks/msgraph-sdk-plsql/issues/25 as completed via #26https://github.com/cdngmnks/msgraph-sdk-plsql/pull/26.
—
Reply to this email directly, view it on GitHubhttps://github.com/cdngmnks/msgraph-sdk-plsql/issues/25#event-10984107736, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7EPTPSDCEJPHRPAD5P5MODYEZGZNAVCNFSM6AAAAAA7OPUSAGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHE4DIMJQG43TGNQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Glad if it helps, thanks for pointing out the bug!
BR Paul

From: warrencaples @.>
Sent: Thursday, November 16, 2023 5:55:22 PM
To: cdngmnks/msgraph-sdk-plsql @.>
Cc: Paul Jereb @.>; Assign @.>
Subject: Re: [cdngmnks/msgraph-sdk-plsql] compile error on msgraph_onedrive.pks (Issue #25)
Wow, that was fast! Thanks for putting this together, by the way. I’ve been doing cobbled together pieces of this myself, it’s so great that you built something like this!
Warren Caples | HX5 | Director, Management Information Systems
O:<PHONE_NUMBER> | M:<PHONE_NUMBER> |<EMAIL_ADDRESS>212 Eglin Parkway SE | Fort Walton Beach, FL 32548
HXFIVE.COM
NOTICE - This communication may contain confidential and privileged information that is for the sole use of the intended recipient. Any viewing, copying or distribution of, or reliance on this message by unintended recipients is strictly prohibited. If you have received this message in error, please notify us immediately by replying to the message and deleting it from your computer.
From: Paul Jereb @.>
Sent: Thursday, November 16, 2023 11:49 AM
To: cdngmnks/msgraph-sdk-plsql @.>
Cc: Warren Caples @.>; Author @.>
Subject: Re: [cdngmnks/msgraph-sdk-plsql] compile error on msgraph_onedrive.pks (Issue #25)
Closed #25https://github.com/cdngmnks/msgraph-sdk-plsql/issues/25 as completed via #26https://github.com/cdngmnks/msgraph-sdk-plsql/pull/26.
—
Reply to this email directly, view it on GitHubhttps://github.com/cdngmnks/msgraph-sdk-plsql/issues/25#event-10984107736, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7EPTPSDCEJPHRPAD5P5MODYEZGZNAVCNFSM6AAAAAA7OPUSAGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHE4DIMJQG43TGNQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>—
Reply to this email directly, view it on GitHubhttps://github.com/cdngmnks/msgraph-sdk-plsql/issues/25#issuecomment-1814961476, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBHMWV6NFPJK3PTQVBLMALYEZHQVAVCNFSM6AAAAAA7OPUSAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUHE3DCNBXGY.
You are receiving this because you were assigned.Message ID: @.***>


GulpUglifyError: unable to minify JavaScript

Here's the error:
yarn run v1.22.19
warning<EMAIL_ADDRESS>The engine "ghost" appears to be invalid.
$ gulp zip
[06:30:55] Using gulpfile [...]/Casper/gulpfile.js
[06:30:55] Starting 'zip'...
[06:30:55] Starting 'css'...
[06:30:56] Finished 'css' after 728 ms
[06:30:56] Starting 'js'...
[06:30:56] 'js' errored after 140 ms
[06:30:56] GulpUglifyError: unable to minify JavaScript
Caused by: SyntaxError: Unexpected token: name «s», expected: punc «;»
File: [...]/Casper/assets/js/lib/casper.js
Line: 208
Col: 1113
[06:30:56] 'zip' errored after 870 ms
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

May I consider switching to terser?

Hey @gllmhyt, the error is in /assets/js/lib/casper.js which doesn't exist. Is this a custom JS file?

@minimaluminium That's really weird, I have no such file...

I couldn't reproduce the issue on my end. Could you try pulling the latest changes, delete /node_modules and run yarn and yarn dev again? I assume you got this error while trying to run yarn dev?

Indeed when I clone the upstream branch I don't have that problem, I will try to rebase my branch against it and see what's causing the problem

Sounds great! We're using gulp-uglify for all our themes, and if we need to replace this, we'd also need to replace it everywhere. We're not planning to change this at the moment. We really appreciate your idea and contribution 🙂


Support Unicode 15.1.0

The grapheme break specs have roughly doubled in size and are currently failing due to the addition of rule GB9c to the list of grapheme cluster boundary rules in UAX29 Revision 43.
cc @straight-shoota @naqvis

Implementing GB9c seems non-trivial. It requires information which is currently unavailable in the break detection algorithm. I'm not sure how it could be integrated into the current implementation.
I don't think this should hold up adoption of Unixode 15.1.0. Let's just mark the specs as pending.


Runscope plugin

Very nice! I left a few comments about some details that need to be fixed.
I have two questions though:

Wouldn't it be best to just name it runscope? The mashape-analytics plugin will be named galileo, and the Loggly plugin is named loggly, same for syslog. Granted they have "log" in their name but the consistency here is that all plugins are simply named after the plugin they are integrating Kong with.
I remember you wanted to implement a queue mechanism and I suggested lua-resty-logger-socket. What decided you not to implement this mechanism? (Time consumption or is there any other reason that has to do with Runscope itself, would be a way to reformulate my question).

@thibaultCha Re: queuing mechanism -- pushing that out. Needed more time to grok that library, plus wasn't sure how bleeding edge that was (the README wasn't very assuring).

@thibaultCha Can you look things over? I believe all optimizations you've recommended have been implemented.

@thibaultCha Made the update -- is this correct? https://github.com/Mashape/kong/commit/bcf43b064347967846bef4ff4f57ff15559f983b

Okay, here if help needed

Reapplied Runscope plugin code to 'next' branch. Closing out this PR. https://github.com/Mashape/kong/pull/863


Comments now must start with a single # otherwise it will error

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 558d1be7e3c9242b39fe78efe74ada1298112892 dirty (hyprpm: Improve Hyprpm Update Performance (5530)).
Date: Wed Apr 10 19:33:50 2024
Tag: v0.38.0-88-g558d1be7, commits: 4421

flags: (if any)

System Information:
System name: Linux
Node name: ayham
Release: 6.1.84-1-MANJARO
Version: #1 SMP PREEMPT_DYNAMIC Thu Apr  4 04:30:51 UTC 2024

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation TigerLake-H GT1 [UHD Graphics] [8086:9a60] (rev 01) (prog-if 00 [VGA controller])
0000:01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA107M [GeForce RTX 3050 Ti Mobile] [10de:25a0] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Manjaro Linux"
PRETTY_NAME="Manjaro Linux"
ID=manjaro
ID_LIKE=arch
BUILD_ID=rolling
ANSI_COLOR="32;1;24;144;200"
HOME_URL="https://manjaro.org/"
DOCUMENTATION_URL="https://wiki.manjaro.org/"
SUPPORT_URL="https://forum.manjaro.org/"
BUG_REPORT_URL="https://docs.manjaro.org/reporting-bugs/"
PRIVACY_POLICY_URL="https://manjaro.org/privacy-policy/"
LOGO=manjarolinux

Bug or Regression?
Bug
Description
I updated my system and I am using hyprland-git and suddenly all my default configurations started giving me errors, it turned out that any comment line like ###################################### Plugins ######################################### printed an error, I suspected it was something with comment parsing so I tried making it # ##################################### Plugins ######################################### and it worked!
How to reproduce
Start a line in any config with 2+ hashtags # like ### test ###
Crash reports, logs, images, videos

https://github.com/hyprwm/Hyprland/issues/4754


Error while starting Sync: Connection refused for the request

VSCode Version: 1..48.0
OS Version: Windows 7 32 Bit

Steps to Reproduce:

Open VSCode
Turn On Sync

Does this issue occur when all extensions are disabled?: Yes
LOGS:
[2020-08-16 12:57:57.192] [userDataSync] [info] Using settings sync service https://vscode-sync.trafficmanager.net/
[2020-08-16 12:57:57.192] [userDataSync] [info] Auto Sync is disabled.
[2020-08-16 12:58:07.463] [userDataSync] [info] Request failed https://vscode-sync.trafficmanager.net/v1/manifest
[2020-08-16 12:58:35.532] [userDataSync] [info] Request failed https://vscode-sync.trafficmanager.net/v1/manifest
[2020-08-16 13:04:05.442] [userDataSync] [info] Request failed https://vscode-sync.trafficmanager.net/v1/manifest
[2020-08-16 13:04:38.551] [userDataSync] [info] Request failed https://vscode-sync.trafficmanager.net/v1/manifest

I have the same problem

Opening the link in the browser gives privacy error. Saving the rejected certificate and installing it via Internet Options as trusted certificate fixed the issue.

solved.

Opening the link in the browser gives privacy error. Saving the rejected certificate and installing it via Internet Options as trusted certificate fixed the issue
i did not get what the solution is


Upload errors

Upload errors have been fairly common-ish and seem to come in waves. There is no way to re-attempt failed uploads.
Would it be possible to at least retry failed uploads (and maybe a notification that one failed if not auto-retry)? As of now, I have been having to use an alternative uploader to at least get them to HotS Logs.
You can see below I had 3 games fail to upload. They were Unranked Draft games and I did not DC or anything.

App automatically retries any failed uploads after a restart. Uploads will be automatically retried after I implement #8
During active development server issues might be fairly common, later on it will be more stable. It's a tradeoff between development speed and server stability.

I think a retry button should still be included for errors unrelated to server maintainance, like network issues. Restarting the app is an annoyance


AWS Lambda Sink flaky tests & logging

Description
This fixes a flaky test.
LambdaSinkServiceTest > lambda_sink_test_batch_enabled() FAILED
    org.mockito.exceptions.verification.WantedButNotInvoked at LambdaSinkServiceTest.java:316

https://github.com/opensearch-project/data-prepper/issues/3481#issuecomment-2221378060
Two changes:

Use a max event count of 1 to verify that we call the Lambda API.
Adds SLF4J logging to this project. That is partially how I was able to find the cause.

Issues Resolved
N/A
Check List

[ ] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Locally, I changed this test to be a @RepeatedTest(20). When running, I found that it typically failed 18 out of the 20 times! With the change introduced here, it passed in all 20 runs.


GH-69: add capslock icons to the virtual keyboard

Virtual keyboards reacts to shift or capslock buttons and updates layout
Closes #69

caps lock doesn't work.
STR:

Open an app in the browser
Click on 'Play' icon
Press on 'Caps lock' key on the real or virtual keyboard

Actual result: caps lock mode hasn't been saved
Expected result:

it should be possible to enable caps lock mode
the caps lock button should be highlighted if the user presses the key on real / virtual keyboard

https://user-images.githubusercontent.com/8956849/215719924-7fcf4208-8c22-4593-8231-1ab2a27ef6bd.mov

@alex-waverley @elfro I guess the reason is in the nature of a capslock and onRelease implementation. We shouldn't change to default if it was a capslock released.

@alex-waverley below you can find TC:

Physical keyboard is a prior one.
State of Shift key should be checked by the released event

Key
Initial state
Steps
Expected condition

Caps Lock
Inactive
when a user presses the caps lock on the physical kbd and then clicks on it on the virtual kbd
1. the virtual click should be ignored 2. the uppercase mode is still active 3. Caps lock key is highlighted, 4. and uppercase icons are shown.

Caps Lock
Inactive
when a user clicks on the the caps lock on the virtual kbd first and then presses the caps lock key on the physical kbd
1. the physical pressed should be ignored. 2. the uppercase mode is still active. 3. Caps lock key is highlighted. 4. uppercase icons are shown.

Caps Lock
Active
when a user presses the caps lock on the physical kbd and then clicks on it on the virtual kbd
1. virtual click should be ignored. 2. the uppercase mode is still inactive 3. lowercase icons are displayed

Caps Lock
Active
when a user clicks on the the caps lock on the virtual kbd first and then presses the caps lock key on the physical kbd
1. the physical pressed should be ignored. 2. the uppercase mode is still inactive 3. lowercase icons are displayed.

 2. the button is highlighted on the UI 3. the uppercase icons are displayed.

 2. the button isn’t highlighted on UI 3. lowercase icons are displayed.

Shift
Inactive
when a use clicks shift on the virtual kbd and then presses any other key on the physical kbd
the associated movement is translated to the robot and the shift key is released and becomes unhighlighted on UI applying lowercase icons to the virtual kbd.

Shift
Inactive
when a use presses and holds shift on the physical kbd and then clicks on any other key on the virtual kbd
the associated movement is translated to the robot and the shift key is released and becomes unhighlighted on UI applying lowercase icons to the virtual kbd.

[ ] Caps lock isn't highlighted when the uppercase mode is enabled with physical kbd.
STR:

Open an app in the browser
Press on the Caps lock key on the physical kbd.
Actual result: icons layout is changed, but Caps lock key color is still grey
Expected result: button should be highlighted by blue color.

[ ]  The movements, that mapped with the following keys, don't work in uppercase layout:
caps lock + A / S / D / F / G / H / 1 / 2 / 3 / 4 / 5 / 6.
shift + A / S / D / F / G / H.

Unknown issues:

when a user enables caps lock on the virtual kbd and then presses it on the physical kbd, it's impossible to ignore physical press on the same button. As a result, the states will be out of sync.

Caps lock highlighting works fine

@alex-waverley, please resolve conflicts.


How to prevent draggable from returning to its position when overlay leaves sortable items area.

If you take a look on the example : https://master--5fc05e08a4a65d0021ae0bf2.chromatic.com/?path=/story/presets-sortable-multiple-containers--basic-setup
And try to drag item out of the box, outside of sortable context, you will see that dragged item retains its position.
Here is the video:
https://user-images.githubusercontent.com/14318395/178346244-3ccbd3a4-0e2f-4266-8e8d-8b61e92a3a57.gif
But when I do the same thing with my simple implementation, I see that draggable returns to its initial position as soon as overlay leaves sortables. Here is the example:
https://user-images.githubusercontent.com/14318395/178346987-50a1c05d-3acb-4679-b089-9fb5059e88a6.gif
Column highlighted when overlay dragged over a component that contains sortable context.
So overlay left draggables, and moved over a parent component.
So my question is following: how to achieve the same behaviour as in the example?
P.S. Sorry for links instead of preview, didn't find a way how to resize gifs here.

And may  be you could suggest how to understand is overlay dragger at the top or bottom of droppable container?

Same issue here (and other test cases found online). Seems to that when the dnd calculates that it's droppable zone is now the container the draggable already belongs to, the sort reverts itself back to the first state it had when the draggable started dragging or entered the container?
Reproduced in a sandbox: https://codesandbox.io/s/dnd-kit-multi-containers-swap-issue-fn9092


If a site is blacklisted and the extension icon clicked, the switch animation runs.

Basically, the switch animates from off to on, although it should just be on right away without the animation happening every time.

Though it's an expected behavior and is not a bug, it's not an ideal solution to the problem. I have to read more about why the script doesn't detect the checkbox element without a delay.
I think the best solution to this "problem" is to append the html elements, first of which is the blacklisting checkbox atm, programmatically in the same popup.js script, so that for sure we can know that every single time the rest of the code will findElementById because that element will have been appended earlier in the script.

I'm not too familiar with the code here, but perhaps something you could look into is having a loading spinner of some sort while data is being fetched? I know that Bitwarden extension has that so it definitely should be doable. The general idea here is to by default start with the spinner and then disable it in the callback of the function that fetches the blacklist.

Actually, I never noticed the other extensions also having such visible hydration and toggle button animation. Perhaps the fact that pop-boy has this one big toggle that draws my attention to it so much :D I think you can close the issue, as it is a nonissue really. Regarding dark reader, I think they just have a well-distributed backend so the API requests are resolved super quick.


API mocking with MSW fails after upgrade to miniflare 2.2.0

Hey @mrbbot, thanks for all the work on miniflare, it's been a game-changer!
With miniflare 1, I've been using mock service worker to mock API responses. When attempting to upgrade to miniflare 2.2.0, my tests failed. I've no idea where to start looking for a solution, so I've put up a fairly minimal demo, hoping that'll make it easier to pinpoint the cause. HEAD is using miniflare 2.2.0, while the miniflare-1.4.1 branch shows what it looks like when it's working. Running npm test should show failing tests for HEAD and passing for the latter.

Thanks! That'll get me back on track, probably using the built-in mocking feature of undici for now.

Hey @fbarthez,
Were you ever able to get this to work?

Hey @SupremeTechnopriest, no it's still on my todo list and will probably stay there for at least another week. ;/

I kind of need to be able to mock the endpoints.  I also really like the MSW API.  I might bite the bullet and write the interceptor, but it might also be nice for jest-environment-miniflare to expose its own mocking framework.  Maybe a wrapper around the undici ones.  @mrbbot what do you think about starting a discussion around that?

Seems like there are multiple modules that have undici as a dependency... miniflare, @minflare/core, @miniflare/http-server etc... Not sure which one is the one actually performing the request.  I'm pretty sure you need to use the exported mocking functions from the instance of undici doing the request.  The below test fails no matter where I import undici from.
import { MockAgent, setGlobalDispatcher } from '../../node_modules/@miniflare/core/node_modules/undici'

const agent = new MockAgent()
agent.disableNetConnect()

agent.get('http://foo.com')
  .intercept({
    path: '/',
    method: 'GET'
  })
  .reply(200, 'bar')

setGlobalDispatcher(agent)

describe('undiciMock', () => {
  test('Mocks response', async () => {
    const request = new Request('http://foo.com/')
    const res = await fetch(request)
    const text = await res?.text()
    expect(text).toBe('bar')
  })
})

If you have any insight as to what instance of undici we should be mocking that would be very helpful @mrbbot.  Still think we should expose the mocking features of undici more cleanly.
Using jest-environment-miniflare with the default config.

@mrbbot,
That's exactly what I was thinking! Miniflare has made my life soooooooo much better and this would be the icing on the cake.  Let me know how I can help!

Your fetch spy works by the way @mrbbot.

Wrote a little warpper while we sort out the mock global.
export function MockFetch (base: string) {
  const mocks: IFetchMock[] = []
  const fetchSpy = jest.spyOn(globalThis, 'fetch')
  fetchSpy.mockImplementation(async (input: string | Request, init?: Request | RequestInit | undefined): Promise<Response> => {
    const request = new Request(input, init)

    for (const mock of mocks) {
      if (`${base}${mock.path}` === request.url && request.method === mock.method) {
        const resInit = {
          status: mock.status,
          statusText: mock.statusText,
          headers: {}
        }
        if (mock.headers) resInit.headers = mock.headers
        return new Response(mock.body, resInit)
      }
    }
    return new Response(null, { status: 404, statusText: 'Not Found' })
  })

  const intercept = (mock: IFetchMock) => {
    mocks.push(mock)
  }

  const restore = () => {
    mocks.length = 0
    jest.restoreAllMocks()
  }

  return { intercept, restore }
}

const { intercept, restore } = MockFetch('http://foo.com')

intercept({
  path: '/',
  method: 'GET',
  status: 200,
  statusText: 'OK',
  body: 'bar',
  headers: {
    'X-Foo': 'bar'
  }
})

afterAll(restore)

test('Mock', async () => {
  const request = new Request('http://foo.com/')
  const res = await fetch(request)
  const text = await res?.text()
  expect(text).toBe('bar')
  expect(res.headers.get('x-foo')).toBe('bar')
})

This was super helpful to me, as jest-fetch-mock doesn't return actual responses!
Here's the wrapper I'm using now:
// @test/helpers/MockAgent.ts

type Route = {
  matcher: (input: Request | string, init?: RequestInit | Request) => boolean
  response: Response
}

/**
 * Returns a handler function composed of passed Handlers,
 * falling back to a 404 Response on no match.
 *
 * @see https://github.com/cloudflare/miniflare/issues/162#issuecomment-1059549646
 */
export function createRequestHandlers(routes: Route[] = []) {
  return async function handler(
    input: Request | string,
    init?: RequestInit | Request,
  ) {
    for (const { matcher, response } of routes) {
      if (matcher(input, init)) return response
    }

    return new Response('Not Found', { status: 404 })
  }
}

/**
 * Convenience function returns true if the GET request matches passed pathname
 */
export function createGetMatcher(pathname: string) {
  return function (input: Request | string, init?: RequestInit | Request) {
    const req = new Request(input, init)
    const url = new URL(req.url)
    return req.method === 'GET' && pathname === url.pathname
  }
}

import { jest } from '@jest/globals'
import {
  basicRequestHandler,
  createRequestHandlers,
  createGetMatcher as get,
} from '@test/helpers/MockAgent'

afterEach(() => {
  jest.restoreAllMocks()
})

test('should mock different routes', async () => {
  let mockBarOnce = true

  const requestHandlers = createRequestHandlers([
    {
      matcher: get('/foo'),
      response: new Response('foo'),
    },
    {
      matcher: (input, init) => {
        const req = new Request(input, init)
        const url = new URL(req.url)
        const matches = req.method === 'GET' && url.pathname === '/bar'
        if (matches && mockBarOnce) {
          mockBarOnce = false
          return true
        }
        return false
      },
      response: new Response('bar'),
    },
  ])

  const fetchSpy = jest.spyOn(globalThis, 'fetch')
  fetchSpy.mockImplementation(requestHandlers)

  let req = new Request('http://localhost/')
  let res = await fetch(req)
  expect(res.status).toBe(404)

  req = new Request('http://localhost/foo')
  res = await fetch(req)
  expect(res.status).toBe(200)
  expect(await res.text()).toBe('foo')

  // mock GET /bar only once
  req = new Request('http://localhost/bar')
  res = await fetch(req)
  expect(res.status).toBe(200)
  expect(await res.text()).toBe('bar')

  // second call to GET /bar passes through
  req = new Request('http://localhost/bar')
  res = await fetch(req)
  expect(res.status).toBe(404)

  expect(fetchSpy.mock.calls).toHaveLength(4)
})

This looks nice! Good stop gap until the true undici mocks are exposed.

I was hoping that perhaps this issue had been resolved with the release of 2.4.0 yesterday, but it has been postponed to 2.5.0.
I am very much looking forward to the next release 👀

@ptim and @SupremeTechnopriest if I'm not mistaken your mock fetch is only being used in the jest test code itself because you're calling fetch directly in the test. So any invokation to miniflare via something like miniflare.dispatchFetch will not use your mock. Is that right or am I understanding it correctly?
I'd love to do a simple spyOn too but miniflare still uses the undici fetch since jest and miniflare don't share a module cache as @mrbbot pointed out.

It should mock the fetch call for your module code as well.  Are you seeing something different?

@SupremeTechnopriest yeah, if I add a mock implementation that simply throws an exception and then call my worker via miniflare.dispatchFetch the request goes through and calls the endpoint. Example that should just fail but works fine:
const fetchSpy = jest.spyOn(globalThis, 'fetch')
fetchSpy.mockImplementation(async (input: string | Request, init?: Request | RequestInit | undefined): Promise<Response> => {
    throw "No fetch for you"
})

it('shouldn throw', async () => {
     const miniflare = new Miniflare({
       // ... setup miniflare vars
     });
    miniflare.dispatchFetch("https://google.com");
});

Maybe it's because I'm not using the new Modules syntax for CF Workers?

@SupremeTechnopriest and @LukePammant, I can confirm that mocking the globalThis.fetch doesn't work with miniflare.dispatchFetch. We have some fetch() calls to some external APIs inside our worker, we wanted to test those calls, but spying fetch was not possible. We used this library to workaround. We start a local HTTP server and use this server instead of external APIs.

Works for me if I call the handler directly, but not with mf.dispatchFetch:
Works:
// src/index.ts
export async function handleRequest(request: Request, env: Bindings) {
  const res = await fetch("http://not-a-real-domain.foo");
  return res;
}

import { handleRequest } from "@/index";
// ...

test("should provide mockFetch", async () => {
  const _fetchSpy = mockFetch(
    "http://not-a-real-domain.foo",
    new Response("mocked")
  );

  const env = getMiniflareBindings();
  const req = new Request("http://localhost/");
  const res = await handleRequest(req, env);

  expect(_fetchSpy).toHaveBeenCalledTimes(1);
  expect(res.status).toBe(200);
  expect(await res.text()).toBe("mocked");
});


Put my website link in contributors

Is your feature request related to a problem? Please describe.
The link of my profile pic in the Contributors page points to my Github profile.
Describe the solution you'd like
I would like it to link to my website, https://locness3.github.io.

Fixed c6d76ea

@Andreto  You could the inverse (putting Fixes #58 in commit message)


Error: 'bbb_docker_user' is undefined

Hi, I'm using Ansible 2.10.2 on Ubuntu20 LTS and run into the following error:
TASK [bbb23-node : Log into DockerHub (avoid rate limits)] *************************************************************
fatal: [bbb-23n1.rz.tu-clausthal.de]: FAILED! => {}

MSG:

The conditional check 'bbb_docker_user and bbb_docker_passwd' failed. The error was: error while evaluating conditional (bbb_docker_user and bbb_docker_passwd): 'bbb_docker_user' is undefined

The error appears to be in 'ansible-role-bigbluebutton/tasks/docker.yml': line 17, column 3, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:
- name: Log into DockerHub (avoid rate limits)
  ^ here

It seems the line should be bbb_docker_user is defined and bbb_docker_password is defined...

Thanks @anadahz - didn't check the pull requests 😬 Will close this issue now.


AKPlayer duration completely wrong

Version of AudioKit : Latest
Version of Xcode : Latest available ( not beta )
When you try to open file with .load(url: file) and this file it was made by concatenating two file with AVAssetExportPresetPassthrough will show completely wrong duration. Will show only the duration of first content and not the total. Infact if you try to use .setPosition where the position is in the middle of second part of file will play from 0.0 and not from this point. If you try with the same file something like AVURLAsset(url: voice).duration will show the correct duration.
You can find file example attach to this message. Try to use .setPosition(5.61) and you will see the problem.
Traccia_7_Voce.mp4.zip

Hey @Ermahiho sorry to bother, I'm contacting you because you mentioned that you're using the latest version's of AudioKit and XCode; I tried to create a hello world without success ( https://github.com/AudioKit/AudioKit/issues/2034 ). Would you mind sharing a hello world with latest swift 5.2 and Xcode 1.4, as it seems you managed it? Thanks!

actually your file format isn't something I support right now, sorry about that. If you drag into logic you see that you have 2 tracks inside your mp4 container.

Twisted Wave doesn't open your file at all.
I'm actually surprised that AKPlayer agrees to play it at all. The duration is taken from AVAudioFile, which appears to just be looking at track 1 of your file. If you want to use AKPlayer with this file, you should export to a M4A first. you can do that with AKConverter. File attached as an example.
Traccia_7_Voce.m4a.zip
`
/// The internal audio file
@objc public private(set) var audioFile: AVAudioFile?
/// The duration of the loaded audio file
@objc public override var duration: Double {
    guard let audioFile = audioFile else { return 0 }
    return Double(audioFile.length) / audioFile.fileFormat.sampleRate
}

`


ContextualVersionConflict: (cryptography 1.3.1 (/opt/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages), Requirement.parse('cryptography>=1.3.4; extra == "security"'), set(['requests']))

Traceback (most recent call last):
File "/opt/psa/var/modules/letsencrypt/venv/bin/certbot", line 11, in
sys.exit(main())
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/certbot/main.py", line 825, in main
plugins = plugins_disco.PluginsRegistry.find_all()
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/certbot/plugins/disco.py", line 183, in find_all
plugin_ep = PluginEntryPoint(entry_point)
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/certbot/plugins/disco.py", line 34, in init
self.plugin_cls = entry_point.load()
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/pkg_resources/init.py", line 2290, in load
self.require(*args, **kwargs)
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/pkg_resources/init.py", line 2307, in require
items = working_set.resolve(reqs, env, installer)
File "/usr/local/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages/pkg_resources/init.py", line 854, in resolve
raise VersionConflict(dist, req).with_context(dependent_req)
ContextualVersionConflict: (cryptography 1.3.1 (/opt/psa/var/modules/letsencrypt/venv.RFhL6/lib/python2.7/site-packages), Requirement.parse('cryptography>=1.3.4; extra == "security"'), set(['requests']))

Thank you for reporting.
We will fix it ASAP.

I've tried to update cryptography package with pip without success

@alessiodionisi the dependency has been fixed - try to reinstall the extension.

It works perfectly 😄
Thank you


Source profile upon creation

After executing sudo ./update-golang.sh (when performing a clean install), I expect to immediately be able to run go commands from anywhere. However, since the profile modifications aren't sourced after being created or updated, the user will have to either do it themselves manually or re-login to their session to access it globally.
Suggested changes:
Add source "$abs_profiled" after line #320

Problem is, the script usually is run as root sudo ./update-golang.sh. Sourcing the profile in the sudo context will not affect the user shell when sudo exits.
Maybe, another approach is to leave a message to user that one might need to source the profile in order to get immediate effect?

Very good point! I do think your suggestion of outputting a message to the user to source it upon completion. You might want to consider putting it at the end of the script output, separating the message with a newline at the start, otherwise users may not see it as easily.

I have added this https://github.com/udhos/update-golang/commit/20a10826721e76d6db24d05282b65fac2e94661f:
update-golang.sh:
update-golang.sh: HINT: If this is the first time you run this script, the env vars
update-golang.sh:       updated in the profile will only take effect for new shells.
update-golang.sh:       If you want them to affect the current shell, use this:
update-golang.sh:
update-golang.sh:       source /etc/profile.d/golang_path.sh
update-golang.sh:

Does it help?

Yep, looks good!


make storage batch request honor options.getHost()

To fix #3254

@andreamlin By default yes, but users can call ServiceOptions.Builder.setHost() to change the default setting and storage wouldn't have been able to honor that :)

LGTM


s3ninja appears to require signed chunks for PUT

Hi. Thanks for s3ninja, it's been really helpful for us!
We're encountering a problem with pre-signed S3 URLs. We're attempting to do a plain old HTTP PUT on a pre-signed URL (not using the AWS SDK) with the body data foo. When we do this, we see an error in the s3ninja log and the PUT fails:
Caused by: java.lang.NumberFormatException: For input string: "foo"
	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
	at java.lang.Integer.parseInt(Integer.java:580)
	at ninja.SignedChunkHandler.handle(SignedChunkHandler.java:55)
	at sirius.web.http.WebServerHandler.channelReadHttpContent(WebServerHandler.java:310)
	at sirius.web.http.WebServerHandler.channelReadLastHttpContent(WebServerHandler.java:320)
	at sirius.web.http.WebServerHandler.channelRead(WebServerHandler.java:275)

As far as I can tell from the code, it looks like s3ninja assumes that the request will always be chunked and that the chunks will always be signed by the client, however in this case the client is doing a plain, non-chunked PUT request to a pre-signed URL.
This used to work when creating the pre-signed URL with AWS SDK 1.10.x (so AWS v2 signing) and s3ninja 2.7. Now we need to upgrade our system to use AWS SDK 1.11.x for other reasons, so our generated pre-signed URLs are now using AWS v4 signing, and so we need to upgrade s3ninja (because 2.7 will not support v4 signing). Unfortunately s3ninja versions after 2.7 no longer support a plain PUT (chunks without signatures) to a pre-signed URL.
I've tried every version 4, 5, 5.1, 5.2, 5.2.1, 5.2.2, 6 and the current HEAD (bd64601de4032b770e1f94e2dd38f064e4b46926).

Thank you, @joelittlejohn 👍


Move scenes to its own repository: mcthings_scenes

In order to maintain some compatibility with videos (hard to update) and doc, let's conserve inside mcthings the current scenes folder.

Move all the scenes to the new repo and leave in the current repo the scenes in their current status with a clear messages to redirect to the new ones.

All done! Much better organized now. Probably more work two coordinate two repositories, but at the end, the right decision.
https://github.com/juntosdesdecasa/mcthings_scenes/


Environment variables read by build logic depends on JDK used and daemon setting

When using JDK9, System.getenv does not reflect the environment the build was started from unless the daemon is disabled and a new JVM is used on each build.  On JDK7&8, we copy the environment variables from the client JVM to the daemon process.
We stopped doing that for JDK9:
https://github.com/gradle/gradle/blob/master/subprojects/native/src/main/java/org/gradle/internal/nativeintegration/processenvironment/AbstractProcessEnvironment.java#L81-L87
Expected Behavior
From build logic, I have a way to get values from the environment Gradle was run from.
Current Behavior
System.getenv can be used for JDK7 and 8.  For JDK9, I have to disable the daemon completely or avoid the use of environment variables.
Context
This is impacting gradle-native: https://github.com/gradle/gradle-native/issues/280
This will probably impact more things going forward.
Steps to Reproduce (for bugs)
With JDK9, given build.gradle:
task printVar() {
  doLast {
    println System.env.VAR
  }
}

export VAR=foo
gradle printVar // prints foo
export VAR=bar
gradle printVar // prints foo

Your Environment
Any OS, JDK9+, any Gradle that supports JDK9.

Java 9 does not allow us to overwrite the environment anymore.
We could ship them over via a dedicated channel and offer something like StartParameter.getEnvironment. Or we could expand our new build option infrastructure to be extensible and also be configurable from the environment. In both cases we need to ship the environment to the daemon in some form. The difference is just how we surface that in the API.

Yep, I tried to avoid prescribing a solution (e.g., System.getenv should just work).

Hi,
@oehme is it still planned to add a StartParameter.getEnvironment?
Because I've the problem that my CI server uses some environment variables to hand over some information of the build itself. Those variable are read via System.env, but as I use the Gradle daemon the second and subsequent build are using the old and wrong variables.
Of course I could add all of those variables to the command line but that would make the command awefully long.
So I would really appreciate if you could add such a new method. What do you think @oehme ?
Thanks in advance.

Fixed by #6170


remove yarn version restriction from package json + prettier endOfLine

React Boilerplate CRA Template
⚠ Clear this template before you submit (after you read the things below)
Thank you for contributing! Please take a moment to review our contributing guidelines
to make the process easy and effective for everyone involved.
Please open an issue before embarking on any significant pull request, especially those that
add a new library or change existing tests, otherwise you risk spending a lot of time working
on something that might not end up being merged into the project.
Before opening a pull request, please ensure:

[ ] You have followed our contributing guidelines
[ ] Double-check your branch is based on dev and targets dev
[ ] Pull request has tests (we are going for 100% coverage!)
[ ] Code is well-commented, linted and follows project conventions
[ ] Documentation is updated (if necessary)
[ ] Internal code generators and templates are updated (if necessary)
[ ] Description explains the issue/use-case resolved and auto-closes related issues

Be kind to code reviewers, please try to keep pull requests as small and focused as possible :)
IMPORTANT: By submitting a patch, you agree to allow the project
owners to license your work under the terms of the MIT License.

why is 'endOfLine' needed in prettier?

Closing: No response and implemented partially in the new release


Fix manifest excludes not being updated for the src layout relocation

Removed .pyd and .so since we do distribute these files

Sorry about that, I processed that wrong. Fixed in #36


write unit tests

I have an idea for unit tests. We can write a program to check wether the connection of the hardware is OK.
for example, a dummy program to check the Ultrasonic Sensor, RFID, and buzzer is working properly and return (OK).


upgrade-buildpacks pipeline does not show filename/version on running cf buildpacks after upgrading

Issue
after running upgrade-buildpacks pipeline.
and doing a cf buildpacks
the column under 'filename' is empty, so it is hard to determine the current version of upgraded buildpacks.
Expected result
seeing the filename of the upgraded buildpack with its name as if I would have downloaded it from pivnet and ran a cf update-buildpack manually
Current result
'filename' column of upgraded buildpacks is empty on a cf buildpacks

might be that the missing filenames only occur if there is an error while 'uploading' the buildpacks ... still investigating this

Thanks @dhoffi, let us know if you find this is still an issue.

Closing this issue. If it is indeed still a problem, please reopen.


#35 Added async variants for table members

Fixes #35
I think I have now covered all the table members with async variants.
As always, just let me know if there is something I ought to amend or rethink.
Stewart

Thanks again :-)

My pleasure. (literally! ;-))

@stewart-r just a small question regarding the above - I'm thinking of rewriting this bit recursively / with a fold but I'm unclear of the behaviour of the continuation token. Does it mutate the exist token (in which case does it really set it to null?) or should there be a "new" token passed out?

Oh no - that's a mistake.
getBatch should return batch.ContinuationToken - not the original contTkn:
let getBatch contTkn = async {
        let! batch = (getTable tableName connection).ExecuteQuerySegmentedAsync(query,contTkn) |> Async.AwaitTask
        batch
        |> Seq.map(mapToReturnEntity)
        |> resSet.AddRange
        return batch.ContinuationToken
    }
    let! firstContTkn = getBatch null
    let mutable contTkn = firstContTkn
    while (contTkn <> null) do
        let! nextTkn = getBatch contTkn
        contTkn <- nextTkn

Sincere apologies, Isaac. I should have spotted this but, more importantly, I should have had this covered by a test. It never occurred to me that I didn't have a multi-batch test.

No worries :-) I'll push a fix out - code hasn't been released yet anyway.

Would you have time to write a multi-batch test though? :-)

Certainly - its the least I can do. :-)
I'm just starting work right now but will either do it at lunchtime or this evening

That's great - I've pushed the fix, so no rush on the test case. Thanks for your help here :-)


[common]: Preserve Previously Released Assets in GitHub Release Section

Describe the problem
After each successful kernel compilation, the previous release asset is deleted before the new one is uploaded to the GitHub release section. This causes a loss of historical release assets. Is there a way to modify this behaviour to keep all previously released assets available in the release section?

try 7255d98


Added a few of the missing status codes to improve error reporting

Codecov Report

Merging #228 into master will decrease coverage by 0.01%.
The diff coverage is 3.12%.

@@            Coverage Diff             @@
##           master     #228      +/-   ##
==========================================
- Coverage   54.15%   54.13%   -0.02%
==========================================
  Files         332      332
  Lines       56393    56424      +31
==========================================
+ Hits        30537    30544       +7
- Misses      25856    25880      +24

Impacted Files
Coverage Δ

.../lib/profiles/data-management/Current/MessageDef.h
0% <ø> (ø)
:arrow_up:

src/lib/support/StatusReportStr.cpp
57.92% <3.12%> (-13.51%)
:arrow_down:

src/lwip/standalone/sys_arch.c
69.48% <0%> (+0.46%)
:arrow_up:

third_party/lwip/repo/lwip/src/core/pbuf.c
36.79% <0%> (+1.59%)
:arrow_up:

Continue to review full report at Codecov.

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


Please update new version weapon paints

Please update new version weapon paints

++
I'm waiting for zeus and pins to be added

🧐

any update about it? or is this site discontinued?

any update about it? or is this site discontinued?

Currently have no time to work on it, when i will have time i will continue this project.
have some planning for the future, soon.


[MacOS] Trailing whitespace in multiline TextField overflows and doesn't go to newline.

Steps to Reproduce

Use a multiline TextField.
Input a lot of whitespace in the content tail.
Whitespace is overflow and not switch to a newline (Tap Ctrl + A to select all).

Expected results: Auto switch to a new line.
Actual results: Whitespace is overflow.

Code sample
const SizedBox(
    width: 400,
    height: 200,
    child: TextField(
      maxLines: null,
      keyboardType: TextInputType.multiline,
      decoration: InputDecoration(
        border: InputBorder.none,
        filled: true,
        fillColor: Colors.pink
      ),
      expands: true,
    ),
  )

Logs

[✓] Flutter (Channel stable, 2.8.1, on macOS 12.0.1 21A559 darwin-arm, locale
    zh-Hans-CN)
    • Flutter version 2.8.1 at /Volumes/AOSP/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (2 months ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/jinguoyang/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.64.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • macOS (desktop) • macos    • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)    • chrome   • web-javascript • Google Chrome 98.0.4758.109

• No issues found!

https://user-images.githubusercontent.com/15059084/155669905-5de481ac-716f-421e-987b-f1c705a036bc.mov

@MegatronKing
I verified this behavior in a native macOS app (Notes) and observed that it behaves the same as you reported, ie, entering whitespaces doesn't move to new line which indicates this is an intended behavior since the same occurs on macOS native app. You can check the same for your reference.
I'll close this as WAI. If you disagree, write in comments and I'll reopen it.

@darshankawar Why don't you try XCode or VSCode Editor? I don't think this behavior of Notes is good enough. I face this issue when I develop a simple code editor with TextField. A workaround is helpful rather than closed this issue.

@MegatronKing
Thanks for the feedback. Below are the things I observed after revisiting this use case:

on iOS native app (Notes): pressing space bar till the end of the line doesn't directly put the cursor to new line but if we try to type in anything, it automatically starts at new line. Same behavior on Flutter mobile app (running on iOS simulator for example).
on native macOS (desktop, sublime text), pressing spacebar till the end of the line takes the cursor to newline automatically.  On Flutter macOS (desktop) app, end of the line doesn't take the cursor to newline, but if you try to type in, it directly starts from the new line, which seems to be the case you are reporting.
Same as above on Flutter web.

Are you looking to have the native macOS desktop feature / behavior to be same on flutter macOS app ?  (point # 2 above) ?

@darshankawar  Correctly!

Thanks for confirmation. Based on my observation and verification between native macOS feature and Flutter, below are my findings:

Flutter mobile, desktop and web behavior using trailing whitespace using multline textfield:

Pressing space bar till the end of the line doesn't directly put the cursor to new line but if we try to type in anything, it automatically starts at new line.
https://user-images.githubusercontent.com/67046386/156302710-47a77384-6928-47a0-9760-2919781278a7.mov

Native macOS app (ex: Sublime Text) behavior using trailing whitepace:

Pressing spacebar till the end of the line takes the cursor to newline automatically.
https://user-images.githubusercontent.com/67046386/156302820-fef46622-586a-4a20-8fbd-3ef680639d8b.mov

 stable, master flutter doctor -v

[✓] Flutter (Channel stable, 2.10.2, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.10.2 at /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (2 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (4 available)
    • iPhone 12 Pro Max (mobile)                     •
      A5473606-0213-4FD8-BA16-553433949729 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • iPad Pro (12.9-inch) (4th generation) (mobile) •
      AD9F1B85-B622-485E-97E8-E4AC866D5770 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)                                • macos
      • darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)                                   • chrome
      • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.11.0-0.0.pre.731, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 2.11.0-0.0.pre.731 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 8f360c977a (5 hours ago), 2022-03-01 16:26:11 -0800
    • Engine revision a5718ab3c3
    • Dart version 2.17.0 (build 2.17.0-156.0.dev)
    • DevTools version 2.11.1

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (4 available)
    • iPhone 12 Pro Max (mobile)                     •
      A5473606-0213-4FD8-BA16-553433949729 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • iPad Pro (12.9-inch) (4th generation) (mobile) •
      AD9F1B85-B622-485E-97E8-E4AC866D5770 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)                                • macos
      • darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)                                   • chrome
      • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Talking with @LongCatIsLooong and @Renzo-Olivares in triage, we think this behavior (not wrapping whitespaces) is at the Skia level. I definitely see why you would want it to wrap in the context of a code editor, though.
Also, the selection extending horizontally outside of the field definitely seems like a bug we should fix either way.


return selected nodes via commands

Typical of other select commands, it's a nice feature to return the list of selected nodes.
To see an example:

Open galFiltered
Select one node
Run network select nodeList="selected" firstNeighbors="any"

You'll see the return includes a list of nodes in JSON format. This is very useful for automation use cases.
RCy3 and py4cy can parse this return to provide a nice list for scripters.

Fixed.


notepad++ plugin - loading schematron files with the FHIR plugin loaded crashes notepad++

If I attempt to edit, say, valueset.sch (from the downloads page), notepad++ crashes with an access violation in fhirnpp.dll

Fixed as part of #33


Error parsing adding null terminated values?

I don't think this is a bug I am just needing a nudge in the right direction. I am currently working on adding a better compiler for Elixir error messages to put better entries into the quickfix list. As of right now I am able to get entries looking perfect in the quickfix list but the virtual text and symbols are not showing up in the buffer of the test. Here is the log output when running

Only thing I can see is that when building the error message the null terminated ^@ is being added to before each line of parsing my error format (this can be seen in the longest line to the right of the image)
Any advice for how to remove those or focus my efforts on the real issue would be very appreciated!

Just adding more information. Here is a screenshot of quickfix list with entry in it and the buffer above. I should have stated that I am an avid user of neomake and I can with my classic setup get the virtual text and symbols to pop up normally no problem


Feature request: flow type support

It would be great to add a flow type support for this library, either here or in flow-typed.

Hopefully coming soon. See #151.