work with output from long-running processes

I am trying to wrap a process that watches for changes on a value, and so intentionally does not terminate; as it stands, it appears that I have no way to use the abstractions provided by turtle to do this.
Here is an example that demonstrates the problem:
*Main Lib Paths_etcdctl> import Turtle
*Main Lib Paths_etcdctl Turtle> import qualified Control.Foldl as Fold
*Main Lib Paths_etcdctl Turtle Fold> :t fold (inproc "yes" [] mempty) Fold.head
fold (inproc "yes" [] mempty) Fold.head
  :: MonadIO io => io (Maybe Line)
*Main Lib Paths_etcdctl Turtle Fold> fold (inproc "yes" [] mempty) Fold.head

which blocks indefinitely. Now I know you are saying in the issue at https://github.com/Gabriel439/Haskell-Foldl-Library/issues/85 that this wont make sense for the foldl library, but I also cannot seem to make it work with the lower-level turtle functions, e.g.
*Main Lib Paths_etcdctl Turtle Control.Foldl Fold> foldShell (inproc "yes" [] mempty) (FoldShell (\_ a-> pure $ Just a) Nothing pure)

I saw this answer on stack overflow and it seemed to perhaps show a way:
https://stackoverflow.com/questions/28421469/how-to-drop-lines-when-streaming-from-a-file-using-haskell-and-the-turtle-librar
But testing it:
*Main Lib Paths_etcdctl Turtle Control.Foldl Fold> view $ limit 10 (inproc "yes" [] mempty)
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"
Line "y"

... which still blocks indefinitely, but no longer provides any output.
I have seen a few issues that seem like they may perhaps be related (especially #22), but I believe this is separate.
I have had to drop down to the process library in the past to do some things, but this just really surprised me, since turtle is built with streaming in mind.

@joelmccracken: It is not possible to write a lazy version of limit.  The Shell type is implemented in such a way that this cannot be done.  The main reason why is due to the need to support safe exception handling, which doesn't play nice with terminating streams early.

@joelmccracken: It is not possible to write a lazy version of limit.  The Shell type is implemented in such a way that this cannot be done.  The main reason why is due to the need to support safe exception handling, which doesn't play nice with terminating streams early.

I see. I found this quite surprising given that Turtle attempts to support streaming in general with Shell; Can I add this to documentation somewhere? I can totally understand that there are things that Turtle doesn't attempt to handle, I just found this to be very surprising and it took me a while to track down what was going on.

I see. I found this quite surprising given that Turtle attempts to support streaming in general with Shell; Can I add this to documentation somewhere? I can totally understand that there are things that Turtle doesn't attempt to handle, I just found this to be very surprising and it took me a while to track down what was going on.


⚠ 한경주요지수-코스피200 has degraded performance

In 08d6d45, 한경주요지수-코스피200 (https://markets.hankyung.com/indices/kospi200) experienced degraded performance:

HTTP code: 200
Response time: 6259 ms

Resolved: 신규상장 performance has improved in ea980a4.


Add table or list of available geographical locations to use for timezone.location()

New user feedback!
Page: https://docs.influxdata.com/flux/v0.x/stdlib/timezone/location/
Feedback: Where can I find a list of locations that are used? Are they the standard ones or are there specific ones?

timezone.location() can take in a geographic location as input. but the docs doesnt show which geographic locations are valid inputs. Create a list or table showing the list of valid inputs.
timezone.location(name: "America/Los_Angeles") is available but what are the other possible options?
Relevant URLs
https://docs.influxdata.com/flux/v0.x/stdlib/timezone/location/

This wholly depends on the TZ database provided by the host OS. This is something that should and will be outlined in the doc about time mechanics in Flux. Related to https://github.com/influxdata/docs-v2/issues/3394.

Ill close this ticket out as duplicate.


Associate O365 group to new work space

Hi @magiesMSFT, Is there any document on how to associate an O365 group or Sharepoint site to the new workspace? Also, when it says No One Drive button, is it the button which says "One Drive - Business"  and we should be using Share point - Team sites to connect files? I am looking for a way like in old workspaces where the files uploaded in One Drive are already connected and only accessible from a related workspace.

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

ID: 11357d2a-97f6-c5d6-2a62-0d5cae1a1c3d
Version Independent ID: 51900b79-f5ed-25b8-ee89-9078129bdb15
Content: Create the new workspaces (preview) - Power BI - Power BI
Content Source: powerbi-docs/service-create-the-new-workspaces.md
Service: powerbi
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @deepu299 -- just so you know, I'm checking on your questions. Haven't got an answer yet but I'm checking.
Re this part: "when it says No One Drive button, is it the button which says "One Drive - Business" and we should be using Share point - Team sites to connect files?"
Yes, it's that button, and yes, at least for the preview you can use SharePoint - Team Sites to connect to files.

I can't figure out how to associate an O365 group either to a workspace, only users

Hi, @deepu299 and @JustinGrote -- good news! Connecting a OneDrive or SharePoint is now on the Create a Workspace pane, when you first create it, or when you go back and edit it. If you look in the article now, under step 4 you'll see:
"Specify a Workspace OneDrive by typing just the name of the OneDrive location, not the URL. In this step, you're not creating a OneDrive location. You're specifying an existing location."
It can also be a SharePoint site.
Hope that helps and sorry for keeping you waiting!

@maggiesMSFT Thanks for the update Maggie, I am able to see the option now. Appreciate for the responses and the work done.


Feature/make new

Update default branch to include new CLI features.

Codecov Report

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

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   93.65%
=======================================
  Files           ?        8
  Lines           ?      189
  Branches        ?        0
=======================================
  Hits            ?      177
  Misses          ?       12
  Partials        ?        0

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 be641e3...6cffef3. Read the comment docs.


Sunnyupdate/docu 2787

Description
What did you change and why?
Update the title from portal auth guide to portal rbac
Testing instructions
Netlify link:
Checklist

[ ] Review label added
[ ] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

Closing as a duplicate to https://github.com/Kong/docs.konghq.com/pull/4897


Changes to support IP field in star tree indexing

Description
The changes include support for IP field as part of star tree index as part of dimensions specifically.
Related Issues
Resolves #16642
Check List

[x] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

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.

Tests are added as part of integ tests as well. Codecov is failing as it does not include coverage from integ tests.


Not receiving multiple messages in group chat

Hi,
I'm having an issue where a Bot inside a group chat (with privacy mode off) will trigger the event (such as bot.onText) only the first time but not on subsequent messages.
How to reproduce:

Create / load a bot which has privacy mode turned off and invite it into a group chat (Use something similar to the example code and long-polling).
Trigger the onText event by sending a message to the group-chat room manually.
Wait about a minute then try to trigger the bot event again (doesn't have to be the same text). This time the bot's event will fail to fire.
Trigger the event by sending a message to the bot directly (This time it will work).
Do step 4 again, there is no problem.

Is anyone else able to reproduce this?

+1

Issues possibly addressing your bug:

#174
#190

If information from any of the above issues fixes it for you, please mention it here and close the issue! Otherwise, am marking this issue stale to be closed in the future, should no activity be seen on this thread!

/cc @Panthro (due to your issue cross-referencing!)

Issue went too stale...


Issue observed while running tests on a Mac machine

I had written some botium-watson tests on the windows machine and was trying to run them on a Mac for a different user. Observation is, lets say i changed the JSON assert value such that the tests will fail, but on the Mac the test still passes. Not really sure whats going on here.
Below is the image of the same test run on windows and Mac [macOS Catalina], on windows it fails as expected but on the Mac it Passes
On windows

On Mac

Honestly, no idea. can you bring this down to the most easy case to reproduce ?


getBodyFatPercentageSamples coming back as undefined

Describe the bug
Calling the getBodyFatPercentageSamples() throws errors.
Possible Unhandled Promise Rejection (id: 0):
TypeError: undefined is not a function (near '..._reactNativeHealth.default.getBodyFatPercentageSamples...')

Hey @hammad0316!
Thanks for the submission. Realized that the existing implementation for  getBodyFatPercentageSamples() is not being exported for the React Native interface
Added a fix for that in #37

Hey @hammad0316!
Thanks for the submission. Realized that the existing implementation for  getBodyFatPercentageSamples() is not being exported for the React Native interface
Added a fix for that in #37


Flush cache when custom pricing changes

This includes turning custom pricing on/off and changing any of the following settings:

CPU
SpotCPU
RAM
SpotRAM
GPU
Storage

Testing for this is here: https://github.com/AjayTripathy/cost-analyzer-frontend/pull/6


SOMYA RANJAN DASH Worked with Satyajeet Das

https://github.com/unlim-int-soumya/lab2b_esp

I Unknowingly submitted gain.


Feature - last highlighted slice on mouse out transition

Description

Added some additional logic for the new property hasLastHoverSliceHighlighted, which highlights the slice after mouseout event is called. Note: if donut char has both hasFixedHighlightedSlice and hasLastHoverSliceHighlighted as true, then hasLastHoverSliceHighlighted becomes the dominant factor. Otherwise, what's the point of setting it true (right?).
Motivation and Context

https://github.com/eventbrite/britecharts/issues/692
How Has This Been Tested?

Manual tests:

Screenshots (if appropriate):
Types of changes

[ ] Refactor (changes the way we code something without changing its functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style guide of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

No problem! I just added setter/getter tests, but the DOM tests with Karma/Jasmine do not work if I try to check for the difference between d attributes before and after mouseover event, something like:
            describe('when hasLastHoverSliceHighlighted is true', () => {

                it('hovered dounut chart slice should remain highlighted on mouseout', () => {
                    donutChart.hasLastHoverSliceHighlighted(true);
                    containerFixture.datum(dataset).call(donutChart);

                    let firstSlice = containerFixture.select('.chart-group .arc path');
                    let firstSliceNode = firstSlice.node();
                    let beforeAttr = firstSliceNode.getAttribute('d');
                    let afterAttr = null;

                    donutChart.on('customMouseOver', () => {
                        afterAttr = firstSliceNode.getAttribute('d');
                        // fails because the d value does not change
                        expect(beforeAttr).not.toBe(afterAttr);
                    });
                    firstSlice.dispatch('mouseover');
                });
            });

There is no example of a test that inspects slices' states before and after mouseover anywhere though.


map to nnoremap

What is the advantage of nnoremap here? As far as I can tell, there is no case in which recursive mapping would happen anyway?

ok, is only a personal preference, always nnoremap over nmap


Start code generator

Take inspiration from nanopb's python generator
Aim for varint and non-plugin way first.

closed and even bootstrapped.


New field added client-secret for Azure client secret ID.

Also updated previous UT for better accuracy.

NOTE: There is already one key named azure-client-secret with different regex. Need to confirm whether we need to add new entity or update older one.
@srics @gr8nishan


[release 1.4] Add annotations for setting cpu/memory limits on sidecar

When a limitrange is active, not setting the limits will result
in an error. This patch will allow setting limits for the sidecar.
Fixes #16126

Hi @rolandkool. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

Hey guys, we're hoping to use this one when it goes in. Any idea when that might happen? Thanks!


3.1.0 vs 4.0.0

It looks like this repo shows package.json as 3.1.0, but the most recent version on NPM is 4.0.0. Is there a reason for this, or was pushing v4 accidental? I'd like to stay up-to-date with the most recent version on NPM, but I'm confused at the version mismatch. Thanks for your work maintaining this product!

Hi @christianbundy,
sorry for that. Just forgot to commit package.json as I updated last time.
Cheers, Dustin

Thanks Dustin, really appreciate it!


Improved regex so that virtual functions don't fail in certain scenarios

Fixes #105

Realised there is an issue with this PR. It may offset the span which would break accurate error reporting


Dev lux submodule / Reset blockchain

Using neo-lux submodule instead of NuGet dependency.
*We had a problem with git, so there are several commits added, but you can see from the changed files that:

neo lux added as a submodule
changed solution/project files to use the submodule
fixed a window issue
resetting blockchain

Award lock9 200 points

https://neoscan.io/transaction/630c74684cc6dad330397c63e8c7aa6403c043cc1f241638cff87e3b97a40716


Created Frederic (code) for Frederic Lestage (new administration).

Authored by<EMAIL_ADDRESS>
Frederic: Best developper
New municipal administration Frederic Lestage
--
Automated submit by PRB0t

Test


Revert the use of line breaks when tostringing Union types

A while ago, I submitted https://github.com/Roblox/luau/pull/487 which would stringify union / interesection types onto multiple lines. The motivating example was a long list of function overload intersections, where this type of formatting would be useful.
In practice though, the union types aren't typically too long, and this formatting "looks off" in many cases.

Some examples:

I'm thinking that it may be best to revert this change. Either

Do not stringify union types on multiple lines at all
Only stringify on multiple lines if there are more than X types in the union (e.g. X = 5?)

Maybe this should be extended to intersections as well? Only format intersections if it contains function types, i.e. the original motivating example?
Of course, this opinion is subjective, so I opened this issue to gather thoughts

Yeah it feels like we should just have some cutoff. Two-element unions probably should not be broken up. Alternatively we can restrict this to cases where all union elements are function types.

A good heuristic might be to break lines if any of the union options are tables or functions.


Add packages dependencies facility.

In my case we use exim instead of postfix, and I need to avoid the Ansible role to replace the mail sender.
So I suggest to add the packages dependencies as a default variable to be able to tweak it.

This adds a conflict since the structure has changed; but I've added a new gitlab_dependencies var that can be overridden, like this PR does. Thanks!


how to begin troubleshooting if plugin does not seem to have mapped keys?

I want to try this plugin, but, after installing, it still does not work.  :checkhealth shows no mapping conflicts.  I tried moving it to the top and the bottom of my Plug list.  I tried adding the config to my .vimrc inside of lua << EOF ... EOF tags, but it threw errors, and I don't think I have time to learn how to use nvim with Lua right now.

Do a sanity check with :nmap s, :nmap S, etc., that will show you if those keys are already remapped to something (in that case, Lightspeed will not overwrite those)

That helped, thanks!


Update wasmtime to 0.29.0

This updates wasmtime to 0.29.0 and its new api.

Removes the state-holder and makes use of the data that is being attached to the state
Make WasmRuntime trait take mut self

Closes: https://github.com/paritytech/substrate/issues/9068
polkadot companion: https://github.com/paritytech/polkadot/pull/3834

It's a shame the reads require &mut now, but I guess the context does get changed. I almost wonder if we could hide that away behind some internal mutability...

The changes look good. Some nits left above.

Forgot to push my changes :D

bot merge

bot merge

bot merge


Pagination only works in ASC mode for File Transport

Because winston reads the file from the first line to the last. if you offset with the start property what will happen is it will read the first few lines until it reaches the start point and then it will insert records into the array.
but since the array only cares about the last ones it will always end up with the same result. if the option.start is close to the log length then you will only end up with a subset of the first result. and therefore cannot paginate
Here are my queries
   rows: 100,
   start: 100,
   from: moment("1918-03-17T11:31:08.505"),
   until: moment("2018-03-18T11:31:08.505"),
 }

{
  rows: 100,
  start: 0,
  from: moment("1918-03-17T11:31:08.505"),
  until: moment("2018-03-18T11:31:08.505")
}

i mean to use it for a winston gui
https://github.com/spearmootz/winston-dashboard

@spearmootz If you still interested, maybe can give a test on my PR ? Cheers!


Characters should be infinite in number (bug with only 1 character type per account logged in)[3 characters max per server]

There can only be 3 characters logged in at one time (one Warrior, one Rogue, one Wizard) and this should be where it is ~infinite amounts of Warriors, Rogues and Wizards with ~infinite amount of accounts. So a server can have more than 3 player max per time oops lol but its really good so far.
It might be tied to the cool guest account where it has placeholder login and password and the server might be like seeing the same login / password / account name and trying its best to not have 2 characters on screen with the same account info to run properly. I could be wrong but its worth a little peak under the hood for.
This is super cool and thanks for making the login screen its really cool! I'm really excited about login features and account creation and account data storage like it is so close its amazing! Really good job too super cool!
😃 👍 ✨ 🗺 🏰 💯

The gateway system is still in experimental state as mentioned in the last commit message, I'll work on making it more stable but it looks like your error likely come from the expiration timer, it's supposed to happen when you stay too long on the gateway (15 minutes). Anyway I tried to replicate the issue on my side and as you see below it's working well.

0d15dcf66603519d98f777e3377086c43bbb8612 should solve the timer issue.

Thank you very much and it's looking really good, great job! 👍 😃 ✨

It works great! Keep up the good work, its really awesome. Thank you for the working login and account system its really cool to see it in action. It's a really big success! 😃 👍 💯 ✨ 🥇 ⛰ 🏰 ⚔ 🗺


Update resolution template to include new Steering Committee members.

The default resolution template contains a list of Steering Committee members that needs to be updated whenever there is a SC membership change. We need to update the current one, but should make sure we have a process in place to update this template whenever there is a SC membership change, or remove the hard-coded list of SC members.

This probably should be part of a general "things to do when there's an SC membership change" playbook, which doesn't exist but probably ought to. I doubt that template is the only thing that'll need updating.


Fix/story rendering

This PR comes to fix following Storybook issue: https://github.com/okp4/ui/issues/201

Codecov Report

Merging #202 (1047e4b) into main (0b2ff91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #202   +/-   ##
=======================================
  Coverage   96.10%   96.10%
=======================================
  Files          38       38
  Lines         359      359
  Branches       44       44
=======================================
  Hits          345      345
  Misses         12       12
  Partials        2        2

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ci(test): cla-assistant

refs: https://github.com/SpringQL/SpringQL/issues/74

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Great! We confirmed SpringQL-CLA has signed to CLA in cla-assistant web console.


~/.config/fish/conf.d/insulter.fish (line 80): $(...) is not supported. In fish, please use '(random)'.

I am getting this error after installing
fisher install version 4.4.4
Fetching https://api.github.com/repos/alaz-oz/fish-insulter/tarball/HEAD
Installing alaz-oz/fish-insulter
           /home/mj/.config/fish/conf.d/insulter.fish
~/.config/fish/conf.d/insulter.fish (line 80): $(...) is not supported. In fish, please use '(random)'.
	set RANDOM $(random)
	           ^
from sourcing file ~/.config/fish/conf.d/insulter.fish
	called on line 189 of file ~/.config/fish/functions/fisher.fish
in function 'fisher' with arguments 'install Alaz-Oz/fish-insulter'
source: Error while reading file “/home/path/.config/fish/conf.d/insulter.fish”
Installed 1 plugin/s

when i run new terminal i am getting this message
~/.config/fish/conf.d/insulter.fish (line 80): $(...) is not supported. In fish, please use '(random)'.
	set RANDOM $(random)
	           ^
from sourcing file ~/.config/fish/conf.d/insulter.fish
	called on line 253 of file /usr/share/fish/config.fish
from sourcing file /usr/share/fish/config.fish
	called during startup
source: Error while reading file “/home/path/.config/fish/conf.d/insulter.fish”
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish

Feel free to reopen this issue if the problem persist.

Could you give more details,
* OS

* fish version

* fish config (~/.config/fish/config.fish)

Are you running fish shell in some kind of debug mode? As I am not getting these warnings on my computer.

lsb_release -a                                             (base) 0 (0.003s)
No LSB modules are available.
Distributor ID:	Pop
Description:	Pop!_OS 22.04 LTS
Release:	22.04
Codename:	jammy
fish -v                                                    (base) 0 (0.078s)
fish, version 3.3.1
i am not running fish in any debug mode.

@mydoom2010 Is it working fine? Or you are still getting errors?


fix(emulator): polyfill setAppBadge/clearAppBadge

Calls to clearAppBadge and setAppBadge were crashing Puppet/Chrome. It's a known issue in headless, that appears to be resolved in chrome 89 (https://chromium.googlesource.com/chromium/src.git/+/c8a9fc64753d988e7ee59045a52aea14773fbe0d)
As part of this, I'm proposing we log Chrome stdout and stderr to the database by default, not with additional logging. It was hard to find our crash issue without it.

Let's discus this PR...
The changes in emulate-browsers/base/injected-scripts/navigator.ts, which are only needed in Chrome 88 and below, is a good reason why I believe using @secret-agent/emulate-browsers-base as a common dependency for all emulators is an icky approach.

Sure, we can discuss. I don't think this changes my previous position though. This is mixed into another file because I was trying not to conflict with your ongoing (29 days old!!) work on getting the rest of the emulators generated. I figured we could come back and put app badge into it's own file, and only reference this one up to chrome 89. That said, it doesn't have any negative effect (beyond extra work) to put it into chrome 89.
Given I don't have access to a "slab" project where I can change Chrome 89 to stop adding a particular script, I went in the direction of just adding it into the navigator script.
FWIW, to summarize my memory of our previous discussions about this:

You've proposed we move the maintenance of these scripts into Slab and auto-copy them into each monorepo project
Your thought was this will reduce conflicting versions, where updating a polyfill/script would impact plugins you don't want it to

List time we talked this through, we came to the conclusion that point 2 is not actually a problem, because you a plugin refers to a specific emulator-base version, so the scripts will not change out underneath it.

I'm concerned with the maintenance of copying all these files into tons of emulators, and/or checking in tons of auto-generated code.
I also dislike the thought of managing these in a different repo. Maybe I can get used to it, or we can have one folder for "active" scripts that's different than the ones copied into each emulator
I think we can work around these issues by simply creating different "versions" of the scripts, and then importing the appropriate ones when we generate the slab plugins. I find this less confusing to think about than the idea that different emulators actually reference different "versions" of the git history of each script in slab.

Ha. Ok. End of discussion.

On Mar 20, 2021, at 10:23 AM, Blake Byrnes @.***> wrote:

Sure, we can discuss. I don't think this changes my previous position though. This is mixed into another file because I was trying not to conflict with your ongoing (29 days old!!) work on getting the rest of the emulators generated. I figured we could come back and put app badge into it's own file, and only reference this one up to chrome 89. That said, it doesn't have any negative effect (beyond extra work) to put it into chrome 89.
Given I don't have access to a "slab" project where I can change Chrome 89 to stop adding a particular script, I went in the direction of just adding it into the navigator script.
FWIW, to summarize my memory of our previous discussions about this:
You've proposed we move the maintenance of these scripts into Slab and auto-copy them into each monorepo project
Your thought was this will reduce conflicting versions, where updating a polyfill/script would impact plugins you don't want it to
List time we talked this through, we came to the conclusion that point 2 is not actually a problem, because you a plugin refers to a specific emulator-base version, so the scripts will not change out underneath it.
I'm concerned with the maintenance of copying all these files into tons of emulators, and/or checking in tons of auto-generated code.
I also dislike the thought of managing these in a different repo. Maybe I can get used to it, or we can have one folder for "active" scripts that's different than the ones copied into each emulator
I think we can work around these issues by simply creating different "versions" of the scripts, and then importing the appropriate ones when we generate the slab plugins. I find this less confusing to think about than the idea that different emulators actually reference different "versions" of the git history of each script in slab.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Docker support for tests execution

Related to #113
Added script for running tests in docker container with appropriate python version. Modified README to reflect changes.

I only looked at the code for ~5 minutes, but any reason not to use --rm with docker run --rm <aoeu> instead of checking for and removing the container on new tests?

Sticking with Vagrant for now (see discussion in #113).


How many runs are required for the new benchmarks, e.g., BERT, DLRM?

Closed as duplicate: https://github.com/mlperf/training_policies/issues/297


how to configure istio-core manifests with a namespace defined at runtime

To enabling deploying istio to the kubernetes namespace that corresponds to its Juju model, we need to render manifests at runtime that refer to the desired namespace.  The manifests for istio's control plane reference the control plane namespace about 45 times (example manifest - search istio-systemmm:

~3/4 are the namespace field in other objects (eg: the namespace a ServiceAccount or Deployment is put into)
~6 are the roleRef and subjects of 3 ClusterRoleBindings
~3 are settings in Validating and Mutating webhook configurations
~5 spread inside the data of a few ConfigMaps
2 are in a Deployments configuration

Our plan is to mostly render these manifests at compile time, then modify those during runtime.  How should we tackle this modification part of the work?  I don't think we want to do this by hand for every istio version, so maybe we need some scripting to search for these?
One spot that worries me is if a new istio release adds a new setting that uses namespace - how do we confidently find this new setting?  With ~45 usages of namespace in the manifests, I don't think we can rely on a human noticing when they change.  Sounds like we'd need something robust and automated there as well

Another fun complication I've hit today - some of the configmaps istio creates include jinja (or very jinja-like) templates.  I haven't tried much debugging, but on the first pass if I tried to render the manifests istioctl generated through jinja, jinja raised errors.  Might be a way I can suppress these though

Closed by #4 because we used istioctl to do this dynamically


Add Validate RPC to Config service SDK.

Closes #53

@azdagron Sorry, I resolved your "notes" request, because I agreed with the change and pushed it.  I'll keep it open next time so you can track it yourself, if you so desire.


The Logo Ballot! DEADLINE: Wednesday 26th May @ 23:59:59

The issue where logos can be placed to be eligible for voting.
For an overview of the rules/system, see: https://github.com/kmonad/kmonad/issues/239#issuecomment-846367651

Gradient K:

Haskell Double-K

Source SVG

Haskell meets QMK

(Created by @thoelze1, please decide about a license.)

Poll closed.
Ballot box open: #269

By the grace of @slotThe:
Haskell meets QMK: light-mode


Error on using :Files command

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[x] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

I'm currently using GVim with the fzf pluggin and after using the command :Files for the first time sometimes their seems to be a bug where using :Files return the following error
Error running xterm -T "[fzf]" -bg "#282c34" -fg "#abb2bf" -geometry 230x31+0+53 -e bash -ic ''"'"'/home/spark/.fzf/bin/fzf'"'"'  '"'"'-m'"'"' '"'"'--prompt'"'"' '"'"'/home/spark/.vim/rc.d/'"'"' --expect=ctrl-v,ctrl-x,ctrl-t > /
tmp/vVfUCiX/18'
Do anyone else has this bug or is there a way to fix this ?

Solved in https://github.com/junegunn/fzf/pull/1019#issuecomment-327286226


Update quickstart-create-and-configure-projects.md

Owner is not required based on testing and discussions. Switching to Contributor and User Access Administrator as recommending Owner role is not generally not an advised practice.

@RoseHJM
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


ReferenceError: Status is not defined when using Hocuspocus NPM package

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
When trying to use the Hocuspocus package with the Supabase edge runtime, the following error is thrown:
ReferenceError: Status is not defined
    at Server.<anonymous> (file:///home/deno/main/index.ts:164:13)
    at eventLoopTick (ext:core/01_core.js:64:7)
    at async #respond (https://deno.land/std@0.182.0/http/server.ts:220:18)
serving the request with /home/deno/functions/document-sync
runtime has escaped from the event loop unexpectedly: event loop error: TypeError: Deno.addSignalListener is not a function
    at process.on (node:process:345:14)
    at Hocuspocus.listen (file:///tmp/sb-compile-edge-runtime/node_modules/localhost/@hocuspocus/server/2.11.3/dist/hocuspocus-server.esm.js:2509:21)
    at file:///home/deno/functions/document-sync/index.ts:5:8
failed to send request to user worker: connection error: Connection reset by peer (os error 104)
user worker failed to respond: connection error: Connection reset by peer (os error 104)
InvalidWorkerResponse: user worker failed to respond
    at async Promise.allSettled (index 1)
    at async UserWorker.fetch (ext:sb_user_workers/user_workers.js:70:21)
    at async Server.<anonymous> (file:///home/deno/main/index.ts:146:12)
    at async #respond (https://deno.land/std@0.182.0/http/server.ts:220:18) {
  name: "InvalidWorkerResponse"
}

To Reproduce
The function file:
import { Server } from 'npm:@hocuspocus/server@latest'

const server = Server.configure({
  port: 8000,
})

server.listen()

Expected behavior
The function is working when using the deno run command of the Deno CLI, but fails on the Supabase edge runtime, both in the hosted and in the local environment.

Note: https://github.com/ueberdosis/hocuspocus/blob/930a61c81e111cda2b692bf1d89b38bfce5d721d/packages/server/src/Hocuspocus.ts#L180-L182

event loop error: TypeError: Deno.addSignalListener is not a function

We probably shouldn't allow receiving Unix signals for user workers. But I think we could replace this with a mock.

ReferenceError: Status is not defined
at Server. (file:///home/deno/main/index.ts:164:13)
at eventLoopTick (ext:core/01_core.js:64:7)
at async #respond (https://deno.land/std@0.182.0/http/server.ts:220:18)

Note: https://github.com/supabase/cli/blob/develop/internal/functions/serve/templates/main.ts#L174-L179
cc @laktek

// ./examples/hello-world/index.ts

import { Server } from 'npm:@hocuspocus/server@latest'

const server = Server.configure({
	port: 8000,
})

server.listen()

GET /hello-world HTTP/1.1
Transfer-Encoding: chunked
User-Agent: PostmanRuntime/7.37.0
Accept: */*
Cache-Control: no-cache
Postman-Token: 4eb4fe07-d4fc-43d7-adda-ee4d6765a5d7
Host: <IP_ADDRESS>:9998
Connection: keep-alive

HTTP/1.1 200 OK
content-type: text/plain
vary: Accept-Encoding
content-length: 2
date: Mon, 25 Mar 2024 04:21:30 GMT

OK

Hey @colin-chadwick 😁
I've created a patch and tested it, that package seems to simply output OK.
Is this the expected output?

Hi @nyannyacha 👋
Yes, that's the expected output. Nice job! I'll test if the websocket connection works as well, but I think it should.

Hello @colin-chadwick 😁
I'm glad the output I showed you is what you expected.
However, if you are using the Supabase cloud (not self-hosted), my understanding is that websocket connections are not possible. (Note that I am not a member of the Superbase team)
If you are planning to do that, you will need to reach out to the team.
Have a good day!

Thanks for the follow up. I'm self hosting Supabase, so that. shouldn't be a problem.
Best,
Colin

Then you should be fine 😎
If you find any other issues, please don't hesitate to report us!


Allow setting configuration in Mix.Config

You can define from mix.config SMTP config, local SMTP or test config based on value in config.exs:
config :mailman,
  relay: "localhost",
  port: 1025,
  auth: :never

Thank you @mremond ! This is fantastic


Core Bio Labs - CSV Sender Onboarding

Sender Info:

Sender/Company Name: Core Bio Labs
Type of Sender: Commercial Laboratory
Location: OH
RS sender_id: corebiolabs-oh

Initial Interest & Follow Up

[x] Document the date & time the Sender expressed interest -- 2/2/2022

[x] Send Std Kickoff email.  This includes:
   - Links to TOS & TOS Registration
           - https://reportstream.cdc.gov/terms-of-service
           - https://reportstream.cdc.gov/sign-tos
   - Links to the CSV upload guide and CSV schema documentation
           - https://reportstream.cdc.gov/getting-started/testing-facilities/overview
   - Provide an option to have a Kickoff call to review these documents and answer any questions.

[x] Schedule Kickoff / Q&A meetings at the Senders request.

[x] Received TOS.

Sender Setup - Create OKTA group & OKTA account

[x] Define the RS sender_id  --  corebiolabs-oh
[x] Create an OKTA group for the sender_id
[x] Create an OKTA account using the RS Registration information.
[x] Verify OKTA account registration by the Sender.

Sender Setup - Create Sender Settings for Staging & PROD

[x] Locally, create and test Settings in organizations.yaml
[x] Create the Staging & Production YAML settings files

Testing - Prod/Test Internal

[x]  Move all RS development to Staging & PROD.
[x] Make settings change to block results to the STLT, or send all results to the STLT's Test Feed.
[x] Validate Production data and make all necessary RS changes.  Iterate through testing until validation is complete.

Testing - Prod/Test External (if required by the STLT

[x] Contact the STLT to setup testing with Production data in their Production environment.
[x] Verify which Feed the STLT wants to receive the results.  Make settings changes as-needed and deploy to PROD.
[x] Work directly with the STLT to complete Integrated Testing and gain approval to move into Production.
- Prod/Test External (if required by the STLT)

Implementation - Plan & Schedule GO LIVE

[x] Change setting "customerStatus:" to active from testing.
[x] Contact RS Pipeline Team to turn on HHSProtect reporting.
[x] Determine if there is a backlog of results.  If so, contact the STLT to determine what results are needed.  Example, in at one STLT, if a Sender has reported positives only to the STLT, then the STLT wants all COVID (positive & negative) since the Sender started testing.
[x] Coordinate and schedule a start date for daily results and backlog results between the STLT and the Sender.

Implementation - GO LIVE and Post LIVE Review

[ ]  Monitor daily uploads for errors and contact Sender if results need to be corrected and resent.  Monitoring will typically be 1 week in duration unless issue are found.

Email'd CoreBio to give them a heads on about receiving an activation email.  Turns out they have an OKTA account, so I created an OKTA group and added their account.  I also created and deployed their YML Staging & PROD settings today, tomorrow at the latest.

Updated Staging & PROD and created PR #: 4168.  Email'd Core Bio Labs that they are ready to test.

They are configured on our side.  We have a kickoff call on 2/4/22 at 9:30a CT

Adrian is contacting Core Bio because they didn't join today's call.

2/7/22 Below are notes from Victoria.  Summary, kickoff completed, CSV development will be started by Core Bio.  No estimate on starting testing.
Some insights from follow up kick-off call with CoreBioLabs for CSV upload pilot. cc: @Rick Hood @Adrian Brewster @sarah add anything I missed.
@Rick Hood went over the unique schema requests for our standard schema (LOINC and equipment model name)(Berni and Drew maybe we call this out in the schema documentation); With them they will likely have different facility names since they work with a lot of schools.
Value added for them is that data they send to CA PHD will be sent to Los Angeles County.
They would like to discuss long-term moving to an HL7 option (@Adrian Brewster)
They asked in CSV was case sensitive (@Rick Hood has a ticket out there for this)
They asked about estimated time to be up and running noting this is a common question potential sites are asking.
Research will f/u with them once in production to debrief and run query on Metabase and get post - to evaluate the pilot.
@Dawn L. Low (IL, she/her) @drew.gardner they are open to do a call to on submission history or other designs or feedback on error messaging. They are open to it and I affirmed this was part of pilot.

cc https://github.com/rick/you-rang/issues/1

Moved Core Bio Labs to Production.  We'll watch for results.

2/11/22 Three results sent.  Waiting on CA-DPH validation.
2/12/22 Seven results sent.  Waiting on CA-DPH validation.

'2/12/22 thru 2/14/22 Only 10 results.  CA-DPH has accepted all, but with a request to receive RACE codes.  Currently receiving OTHER.

'2/16/22 Create an addition OKTA account
2/16/22 CA-DPH is asking Core to collect the Race & Ethnicity consistently.
2/18/22  Moving Core Bio to Done/Completed.


Change text on primary button on landing

The current text is "Get Started".
This sounds like something I'd click after I'd read all about the framework, was sold on the ideas, and was ready to, you know, start.
Given that the button sends folks to "What is Tailwind?", I think something like "Learn More" is more congruent.
New text in place:

My only gripe is that I think people are generally going to be more interested in trying it than learning about it

I'd dispute that.
After landing on that page, I only know this: Tailwind is a utility-first CSS framework for rapid UI development.
Putting aside whether most people who land on the page know what "utility-first" means, I feel like asking folks to "Get Started" after knowing so little is a bit like having a "Buy Now" button right below your headline.
The headline draws them in, but is it really sufficiently compelling to make them install it right away?
And if they did install it right away, wouldn't they still need to "Learn More" to start using it?

Slept on it and what the hell ¯_(ツ)_/¯ I dig it.


Wrong default localhost

The default port used on localhost is now 7236 instead of 5001.
https://localhost:7236/
Please update the tutorial.

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

ID: ca6139d4-022c-395a-0abf-539660550df1
Version Independent ID: 3dd23f8a-300e-3f78-1dc5-8b759aabdccc
Content: Get started with ASP.NET Core
Content Source: aspnetcore/getting-started/index.md
Product: aspnet-core
Technology: aspnetcore-getstarted
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

There is no default port now.


si.diskLayout and si.memLayout

Issue Type

[x] Bug
[ ] Feature Request
[ ] Question

Expected behavior
Run in electron app and show information about disk and ram
Actual behavior
in case of disk
in browser console output error:
/home/ubikone/Projekty/kontrolki/node_modules/systeminformation/lib/filesystem.js:587 Uncaught TypeError: Cannot read property '1' of null
at /home/ubikone/Projekty/kontrolki/node_modules/systeminformation/lib/filesystem.js:587:64
at Array.forEach (native)
at /home/ubikone/Projekty/kontrolki/node_modules/systeminformation/lib/filesystem.js:577:21
at ChildProcess.exithandler (child_process.js:209:7)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:194:7)
at maybeClose (internal/child_process.js:899:16)
at Socket. (internal/child_process.js:342:11)
at emitOne (events.js:96:13)
at Socket.emit (events.js:191:7)
in terminal output:
cat: /sys/block/cdrom/queue/rotational: Nie ma takiego pliku ani katalogu
incase of ram:
there ids no output and error
Steps to reproduce the problem
full code at https://gitlab.com/ubikOne/kontrolki/tree/master/js
problem is with funtcion at line 133 driveShow and line 105 ramShow in sysinfo.js
Specifications

package version: 3.33.6
operating system: Ubuntu 16.04 4.14.0-041400-generic
hardware: A8-4500M APU with Radeon(tm) HD Graphics, 8 GB RAM, SSD

Problem and possible solution
about disk i can have problem with: const sizeValue = sizeString.match(/(([^)]+))/)[1];
line 587 in lib/filesystem.js
about ram i have no idea what is going on.
on windows 10 it's working ok

Hi, I published a fix for the diskLayout() yesterday (v3.33.7). Can you recheck, if this is working on your side? For the memLayout() I will try to find a solution asap ...

I'll check diskLayout in evening, and replay with information.

there is a little progress... no errors on browser console, but still display anything, and system terminal returns the same problem.

Hi again,
can you post (for disk layout) the output of
export LC_ALL=C; lshw -class disk; unset LC_ALL
and (for mem layout) of
dmidecode -t memory

in a bash console?
Thank you in advance

dmidecode 3.0
Getting SMBIOS data from sysfs.
SMBIOS 2.7 present.
Handle 0x0005, DMI type 5, 20 bytes
Memory Controller Information
Error Detecting Method: None
Error Correcting Capabilities:
None
Supported Interleave: One-way Interleave
Current Interleave: One-way Interleave
Maximum Memory Module Size: 8192 MB
Maximum Total Memory Size: 16384 MB
Supported Speeds:
Other
Unknown
70 ns
60 ns
50 ns
Supported Memory Types:
Other
Unknown
Standard
FPM
SIMM
Burst EDO
Memory Module Voltage: 3.3 V
Associated Memory Slots: 2
0x0006
0x0007
Enabled Error Correcting Capabilities:
None
Single-bit Error Correcting
Double-bit Error Correcting
Error Scrubbing
Handle 0x0006, DMI type 6, 12 bytes
Memory Module Information
Socket Designation: DIMM 0
Bank Connections: None
Current Speed: 1 ns
Type: DIMM
Installed Size: 4096 MB (Single-bank Connection)
Enabled Size: 4096 MB (Single-bank Connection)
Error Status: OK
Handle 0x0007, DMI type 6, 12 bytes
Memory Module Information
Socket Designation: DIMM 0
Bank Connections: None
Current Speed: 1 ns
Type: DIMM
Installed Size: 4096 MB (Single-bank Connection)
Enabled Size: 4096 MB (Single-bank Connection)
Error Status: OK
Handle 0x0022, DMI type 16, 23 bytes
Physical Memory Array
Location: System Board Or Motherboard
Use: System Memory
Error Correction Type: None
Maximum Capacity: 8 GB
Error Information Handle: 0x0027
Number Of Devices: 2
Handle 0x0023, DMI type 17, 34 bytes
Memory Device
Array Handle: 0x0022
Error Information Handle: 0x0025
Total Width: 64 bits
Data Width: 64 bits
Size: 4096 MB
Form Factor: SODIMM
Set: None
Locator: DIMM 0
Bank Locator: CHANNEL A
Type: DDR3
Type Detail: Synchronous Unbuffered (Unregistered)
Speed: 667 MHz
Manufacturer: Not Specified
Serial Number: 00000000
Asset Tag: Asset Tag:
Part Number: GR1333S364L9S/4G
Rank: 1
Configured Clock Speed: 667 MHz
Handle 0x0024, DMI type 17, 34 bytes
Memory Device
Array Handle: 0x0022
Error Information Handle: 0x0026
Total Width: 64 bits
Data Width: 64 bits
Size: 4096 MB
Form Factor: SODIMM
Set: None
Locator: DIMM 0
Bank Locator: CHANNEL B
Type: DDR3
Type Detail: Synchronous Unbuffered (Unregistered)
Speed: 667 MHz
Manufacturer: Not Specified
Serial Number: 00000000
Asset Tag: Asset Tag:
Part Number: GR1333S364L9S/4G
Rank: 1
Configured Clock Speed: 667 MHz

And if you call diskLayout() and memLayout() directly from node ... what is the exact output?
Like so:
const si = require('systeminformation');

si.diskLayout()
	.then(data => console.log(data))
	.catch(error => console.error(error));

Thank you ... and for memLayout()?

... and one more question ... did you try to run the above node.js code as root? Like you did with the underlying commands (previous email)

diskLaout() with sudo works

memLaout() without sudo

with sudo

So I need to check, if I find a solution non non-sudos ... Will provide an answer this weekend, OK?

Fine for me

bois() the same story as memLayuot() and diskLayout(), no visible without sudo

I am closing this for now as I do not see this really as a bug. There are several functions that - depending on which OS - might need root or admin privileges. I try to keep this as low as possible but where I didn't (yet) found a solution without admin privileges I at least implemented a version the can get correct values with admin privileges. Anyway I will investigate, if I can get values (at least some of them) without being admin.


Volume resizing at creation time

Add support for resizing volumes via the Open Stack API (ciao-cli volume add) and via workload definitions when the source for the volume is either another volume or an image.

Coverage decreased (-0.04%) to 65.905% when pulling 7865e0bcd98c9f4a19db22c884a743e0666eea63 on rbradford:volume-resize into a16660bbedd1aa3479b38a7b05b5812f7ece8114 on 01org:master.

I'm going to merge this one as I need it for kubicle.


Pass mutable scene builder reference to paint

Instead of the paint method returning a Rendered which was just a newtype around SceneFragment, pass in &mut SceneBuilder. Also store a scene fragment in each Pod, which is the storage for these builders.
This change has a number of consequences. Less boilerplate in the paint method, and also less allocation, as the storage can get reused. More importantly, though we don't have the logic for this yet, it sets the stage for retaining scene fragments when widgets don't change.

Could we make the scene_builder argument a member of PaintCx like it was in druid?
And i am trying to understand how widgets render their children. The paint method of Pod has no scene_builder parameter and there is no method to retrieve the SceneFragment of the Pod. I would be in favor of adding the scene_builder parameter and let the Pod apply transform and write its SceneFragment into that of its parent.

Is this not addressed in #10?
I haven't implemented containers yet (soon), so it's not clear from the code. Pod will have a method to return a scene fragment reference, and the parent will do append() with that fragment (also apply a transform based on layout / scrolling / etc). The motivation for this is retaining the rendered scene fragments, so you only call into paint when they're invalidated. The idea is that append is cheap - currently it is a handful of memcpy operations, but shouldn't allocate in the steady case, and if the memcpy operations do turn out to be expensive, we could go to non-contiguous storage.

Hopefully that commit makes what I'm trying to do a little clearer.

Yes, thanks :)
From what i understand painting a child widget will look like this most of the time:
child.paint(ctx);
builder.push_layer(..., Affine::translate(child.origin().to_vec2()), ...);
builder.append(child.fragment());
builder.pop_layer();

What i wanted to say is that is could be done by the Pod of child if its paint method receives the parent SceneBuilder. This was also my rationale for proposing to include the scene_builder argument into the PaintCx

Ah, now I understand. I was confusing the paint method of Pod with the paint method on the Widget trait.
In the general case we will not need push_layer(), only when doing clips and blends. Also note that the affine transform has changed from stateful (PostScript/canvas-like) to stateless, so it needs to be given to the append method. Thus, though we could certainly add a convenience method to Pod to append its fragment, I'm not sure it helps that much.

Ah! I over read the transform parameter in append. Now it makes sense. Thanks for the explanation :)


Document the package name in the User Guide and perhaps landing page

...perhaps alongside the Maven coordinates.

What do you mean by "landing page"? https://jspecify.dev/ or start-here?

Hmm, good question. Probably start-here?

I don't think the User Guide changes covered this one, right?

I had a note in the commit that it might have closed this, and GitHub leapt to that conclusion.

Oops, now I'm reading the PR comment :)

Now that the annotations are all linked to Javadoc, this might be sufficient to close #524 as well, at least with respect to the User Guide.

I'd still be inclined to keep this open as a reminder for me to cover it when I take care of the build-system issues, but if I fail, I don't think we need to block 1.0 over it.


Add 299 to Microsoft.AspNetCore.Http.StatusCodes

Would anyone consider accepting a new status code 299 Disappointed? ;-)
For Reference: @acathrow's tweet

Haha, kinda expected that exact response ;-) This PR is the kind of thing that happens during coffee break...


micro-Ros issue installing for Arduino Portenta H7

- Hardware description: I'm using and Arduino Portenta H7
- Software: I'm running it on Arduino IDE 1.8.19, with the micro-ros  Humble library (I've tried both 2.0.5 and 2.0.7)
Steps to reproduce the issue
I'm literally trying to run any of the ROS examples.  When I go through my files I can find most of them. I suspect it might be a compatibility issue but I have no idea how to resolve it.
Here are the error messages:
_Build options changed, rebuilding all
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/native_ethernet_transport.cpp: In function 'size_t arduino_native_ethernet_udp_transport_read(uxrCustomTransport*, uint8_t*, size_t, int, uint8_t*)':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/native_ethernet_transport.cpp:74:21: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
   return (available < 0) ? 0 : available;
           ~~~~~~~~~~^~~
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/wifi_transport.cpp: In function 'size_t arduino_wifi_transport_read(uxrCustomTransport*, uint8_t*, size_t, int, uint8_t*)':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/wifi_transport.cpp:53:33: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     while(millis() - start_time < timeout && udp_client.parsePacket() == 0){
           ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/wifi_transport.cpp:59:20: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
     return (readed < 0) ? 0 : readed;
             ~~~~~~~^~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/Ethernet/src/Ethernet.cpp: In member function 'int arduino::EthernetClass::begin(uint8_t*, long unsigned int, long unsigned int)':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/Ethernet/src/Ethernet.cpp:13:1: warning: control reaches end of non-void function [-Wreturn-type]
 }
 ^
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedClient.cpp: In member function 'size_t arduino::MbedClient::write(const uint8_t*, size_t)':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedClient.cpp:219:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   } while ((ret != size && ret == NSAPI_ERROR_WOULD_BLOCK) && connected());
             ~~~~^~~~~~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedUdp.cpp: In member function 'virtual int arduino::MbedUDP::endPacket()':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedUdp.cpp:76:22: warning: variable length array 'buffer' is used [-Wvla]
   uint8_t buffer[size];
                      ^
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedUdp.cpp:77:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for (int i = 0; i < size; i++) {
                   ~~^~~~~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedUdp.cpp: In member function 'virtual size_t arduino::MbedUDP::write(const uint8_t*, size_t)':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/MbedUdp.cpp:97:20: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for (int i = 0; i<size; i++) {
                   ~^~~~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/SocketHelpers.cpp: In member function 'int arduino::MbedSocketClass::download(const char*, bool, mbed::Callback<void(const char*, long unsigned int)>)':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/SocketHelpers.cpp:187:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for(int i=0; i<header_fields.size(); i++) {
                ~^~~~~~~~~~~~~~~~~~~~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c: In function 'http_parser_execute':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c:1817:31: warning: this statement may fall through [-Wimplicit-fallthrough=]
               parser->upgrade = 1;
               ~~~~~~~~~~~~~~~~^~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c:1819:13: note: here
             case 1:
             ^~~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c: In function 'http_parser_parse_url':
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c:2376:18: warning: this statement may fall through [-Wimplicit-fallthrough=]
         found_at = 1;
         ~~~~~~~~~^~~
/home/robot2/.arduino15/packages/arduino/hardware/mbed_portenta/4.1.1/libraries/SocketWrapper/src/utility/http_parser/http_parser.c:2379:7: note: here
       case s_req_server:
       ^~~~
sketch/micro-ros_publisher.ino.cpp.o: In function `timer_callback(rcl_timer_s*, long long)':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:36: undefined reference to `rcl_publish'
sketch/micro-ros_publisher.ino.cpp.o: In function `set_microros_transports':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/src/micro_ros_arduino.h:33: undefined reference to `rmw_uros_set_custom_transport'
sketch/micro-ros_publisher.ino.cpp.o: In function `setup':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:49: undefined reference to `rcutils_get_default_allocator'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:52: undefined reference to `rclc_support_init'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:55: undefined reference to `rclc_node_init_default'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:58: undefined reference to `rosidl_typesupport_c__get_message_type_support_handle__std_msgs__msg__Int32'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:58: undefined reference to `rclc_publisher_init_default'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:66: undefined reference to `rclc_timer_init_default'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:73: undefined reference to `rclc_executor_init'
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:74: undefined reference to `rclc_executor_add_timer'
sketch/micro-ros_publisher.ino.cpp.o: In function `loop':
/home/robot2/Arduino/libraries/micro_ros_arduino-2.0.5-humble/examples/micro-ros_publisher/micro-ros_publisher.ino:81: undefined reference to `rclc_executor_spin_some'
collect2: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino Portenta H7._

Additional information
If anyone could also suggest a way I can use micro-ros without using the arduino library, i dont mind switching hardware. I'll also take it.
Thanks :)

I do also have the same error prompts in the bottom half. I've tried tons of fix based on other similar issues including patching and building meta file.


Update doc references to spring boot [skip ci]

Signed-off-by: John Collier<EMAIL_ADDRESS>What type of PR is this?
/kind documentation
[skip ci]
What does does this PR do / why we need it:
A while back, the default devfile stacks were updated to follow a consistent naming scheme (e.g. java-spring-boot -> java-springboot), but it appears some of the documentation that references the spring boot stack was not updated accordingly.
Which issue(s) this PR fixes:
Fixes https://github.com/openshift/odo/issues/3863
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[x] Documentation

[x] I have read the test guidelines

Is [skip ci] still not working? 🤔

/lgtm

/approve
Thank you!


ModNewsAPI

In translation files, sometime there's a comma at end of line, and sometimes not. Maybe both are OK I'm not sure, but I think it would be best to use either comma or not, but not having both mixed up. Your thoughts?

In translation files, sometime there's a comma at end of line, and sometimes not. Maybe both are OK I'm not sure, but I think it would be best to use either comma or not, but not having both mixed up. Your thoughts?

Can you specify a line of code that this apply to please?
Not sure to understand what you mean here.

https://github.com/Konijima/PZ-Community-API/pull/54/files#diff-3207e9660ef72ab967c8df840fbe14e6cd14ea9f0d8c3a4194d016bbfd2a1b51R8

Best would be to remove the commas IMHO, because they're not needed

And ModNewsButton.lua is missing a newline character at end of file.

Finally, and then I'll stop being a pain (feel free to tell me if its bothering you), I would rebase the branch to squash commits that we don't need to see in changelog, like no need to have multiple commits to edit the README.md for instance, etc. Also no need to reference to a fix for a bug that was introduced in the same pull request etc.

Sorry one more, I can spot a few problems with indentation, for example these lines:
https://github.com/Konijima/PZ-Community-API/pull/54/files#diff-6ad848317ac690bd43febe87ea2fca02c9245baebcdf4dcca88d0705348c93a0R11
https://github.com/Konijima/PZ-Community-API/pull/54/files#diff-6ad848317ac690bd43febe87ea2fca02c9245baebcdf4dcca88d0705348c93a0R21
There's probably others.


position property fix

Hi @gxrsti, This merge fixes #4.

Hey @Hackerbone,
I like this solution but top-center or bottom-center would not work with this. Maybe you could find a way to also support these cases.

Hi @gxrsti,
I tested it and found out that in those cases only the top or bottom property will have "0" value (i.e. left or right will not be set) which will center it automatically. Thus, no changes required.


Fix crash with Angelica when looking at hydrodam pressurized water

This is a workaround for https://github.com/GTNewHorizons/Angelica/issues/195, where the client crashes when looking at Hydrodam water. It makes the conditions of this statement false, causing it to use the seemingly Angelica compatible alternative path instead.
Note that I couldn't get the build system to replace the GRADLETOKEN_GROUPNAME token, so I replaced it manually for now. Let me know if there's a better way.

Also fixed the crash when returning to main menu after entering a world with a hydrodam.

Should probably properly fix it rather than relying on limited rendering

Idk, I think it's significantly more important that crashes don't happen in the first place

Should probably properly fix it rather than relying on limited rendering

I don't disagree, but I'm not sure what even makes it "limited" in this case, and it's not a hugely popular mod/mechanic, so it doesn't strike me as the highest priority thing to spend time on.

Should probably properly fix it rather than relying on limited rendering

I don't disagree, but I'm not sure what even makes it "limited" in this case, and it's not a hugely popular mod/mechanic, so it doesn't strike me as the highest priority thing to spend time on.

Yeah SirThor left a long time ago


Files in two source directories don't see each other

If module source files are spread in two source directories (eg 'source' and 'test-source'), 'test-source' code don't see 'source' stuff, when you compile them in shell.
That's a problem for whitebox testing, where test and app code must be in the same module.
Note that it's OK when compiling from eclipse, or for java.
Example:
//source/app/run.ceylon
shared void run() {}

// test-source/app/tests.ceylon
shared void runTests() {
    run();
}

$ceylon compile-js
Error: Errors found. Compilation stopped.
Note: Created module default
Note: Created module app/1.0.0
error encountered [function or value does not exist: 'run'] at 3:4-3:6 of tests.ceylon
ceylon compile-js: 1 errors.

you must specify the test-source in the command, since the default is just source

Problem persists with:
ceylon compile-js --src=source:test-source

BTW I think I found another issue: if the second path in '--src' argument doesn't exists (--src=source:dummy), I get a
java.lang.StringIndexOutOfBoundsException: String index out of range: -1

I tried with test-source and am not getting this error; files are compiled just fine. But I do get an error if I specify an invalid path in --src.


"Preview Pre..."

That button name might confuse some people.
(Preview Premix)

missing a single letter to Passwor...

Is this all occurring on the M1 system? If so will likely need to wait for a native release on the platform to fix.

Yes, M1 but I suspect is could be OSX resolution / scaling specific.

Please retest with the M1 native release: https://github.com/sparrowwallet/sparrow/releases/tag/1.6.4-beta1


Support min_tokens for api_server

The weird thing is, min_new_tokens did not work. @irexyc

from lmdeploy import pipeline
pipe = pipeline('/nvme/shared/vicuna-7b-v1.5/')
pipe('hello', gen_config=GenerationConfig())
# Response(text='ಠ\\_ಠ', generate_token_len=7, input_token_len=41, session_id=8, finish_reason='stop', token_ids=[227, 181, 163, 20122, 227, 181, 163], logprobs=None, index=0)

pipe('hello', gen_config=GenerationConfig(min_new_tokens=8))
# Response(text='ಠ\\_ಠ How can I assist you today?', generate_token_len=14, input_token_len=41, session_id=7, finish_reason='stop', token_ids=[227, 181, 163, 20122, 227, 181, 163, 1128, 508, 306, 6985, 366, 9826, 29973], logprobs=None, index=0)

But if I set min_new_tokens=100 for internlm2-chat-1_8b, the result is less than 100 tokens.

It only ignore eos token when length of the generated token is less than min_new_tokens. Its behavior should be consistent with transformers, except that transformers support multiple eos tokens
But If encountered stop words token, it will still stop.

Should we update completions_v1 as well?

It is a lagecy api from OpenAI. I tend to not update it.


Fix order of waypoints in DriveDetailPage

Before, it could happen that the end of the whole drive was placed before another Waypoint (of course only if the time is the same, which should probably not happen in reality - but it happened all the time with our fake data).

I also added a test case that was failing with the previous code. Now the test actually tests which WaypointCards are built and in which order.


feat(client): re-implement following the new protocol specification

This branch is based on the server branch and implements the protocol described in the protocol branch.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MathJax in doxter fields (already working)

I just want to let you know that MathJax-formulas can be written inside doxter fields which is a very nice feature in the engineering / science world. Maybe you can advertise this as a doxter feauture!
Here is an example template:
{% extends "_layout" %}
{% includeJsFile "http://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-MML-AM_HTMLorMML" %}
{% set MathJaxConfig %}
MathJax.Hub.Config({tex2jax: {inlineMath: [['$','$'], ['\\(','\\)']]}});
{% endset %}
{% includeJs MathJaxConfig %}

{% set markdown %}
# Doxter
**Doxter** is a markdown plugin designed to improve the way your write documentation including
[MathJax](http://www.mathjax.org/)-formula.

## Example

When $a \ne 0$, there are two solutions to \\(ax^2 + bx + c = 0\\) and they are
$$x = {-b \pm \sqrt{b^2-4ac} \over 2a}.$$
{% endset %}

{% block content %}

  {{ markdown | doxter }}

Reference output without doxter

  When $a \ne 0$, there are two solutions to \(ax^2 + bx + c = 0\) and they are
  $$x = {-b \pm \sqrt{b^2-4ac} \over 2a}.$$
{% endblock %}

That is cool, I wasn't even aware of that.
Thanks for letting me know, I'll update the docs for the next release.


Update package.json v1.1.3

What's Changed

Bump ws from 8.16.0 to 8.17.1 by @dependabot in https://github.com/paritytech/get-fellows-action/pull/25
Bump braces from 3.0.2 to 3.0.3 by @dependabot in https://github.com/paritytech/get-fellows-action/pull/26

Full Changelog: https://github.com/paritytech/get-fellows-action/compare/v1.1.2...v1.1.3

You need to also change the version in https://github.com/paritytech/get-fellows-action/blob/main/action.yml#L15 to match this new version

thanks, updated


First player doesn't work on page with multiple players (HLS/MP3) in Chrome for macOS

When I embed multiple <audio/> tags, the second and subsequent players play just fine, but the first track just shows an indeterminate progress bar and spins indefinitely:

There's nothing in the JavaScript console and no apparent network activity.
It doesn't appear to be a problem with the streams themselves — I tried reversing the order and whichever one's first is the one that exhibits the problem.

example (two players)

With multiple players, the first player doesn't work but all subsequent players seem to.

example (multiple players)

I'm seeing this on Chrome for macOS Version 86.0.4240.111. Safari doesn't exhibit the same problem.

Update: If I set preload="none" on the first player instead of preload="auto", it works. But it's not just that preload="auto" causes problems — if I set preload="none" on the first player and preload="auto" on the second, the second still works.
Note: I'm probably going to push out a change to our site that sets everything to preload=none. I've saved the current state of the above pages in a gist and two JSFiddles:

two players
many players

Possibly related: if I set the first <audio/> tag to preload="metadata", it works, but the metadata doesn't load (it behaves the same as none). If I set the first <audio/> to preload="none" and the second to preload="metadata", they both behave as if I'd set preload="auto".
What's odd about this is that it looks from the HLS renderer code as if there shouldn't be any difference between metadata and auto at all, but maybe Chrome is interpreting the attribute and doing something with it.


Add :enabled option to config

Hi. Thank you for working on this!
I have a module like this in my code:
defmodule TextMessage do
  def send(phone_number, body) do
    case Application.get_env(:our_app, :env) do
      :test ->
        nil

      _ ->
        ExTwilio.Message.create(%{
          body: body,
          to: phone_number,
          from: our_phone_number
        })
    end
  end
end

Many libraries have :enabled option, so that I could set it to false in test environment, and just use the code without this additional setup. Would it be possible to add here as well?

I think this would be an excellent place to make use of mocking in your code. Check out this library from the Elixir creators: https://github.com/dashbitco/mox


Updating Toolbox-ESD manual mode

I have one of the not supported IOCs so I installed the Toolbox the manual way.
What is the right way to update if you (jilleb) now publish a new toolboxversion where your main ESD-file ist modified.
In my understanding the "New scripts"-button can not work because it won't update the main .esd-file it only gets new files.
Do I have to perform just the same steps like I've done for the manual installation (copy the mqb....esd to the right location) or is there anything more to do?

Good question!
The method I'm using myself most of the time is installing through the feature that gets a new green menu file onto the unit. If you put a new version of mqbcoding.esd into the folder where the new green menu files need to be on your SD, then it can import.
Scripts, leave them where they are, and hit the "import new scripts" button.

okay that sounds much easier than I thought.
So just to clarify: The script overwrites files if they are named the same and so you can update through the "import new scripts" function... except the function itself I suggest.. because it's running in the same moment :D

exactly that :-)

This "import new scripts" is a left-over from developers... it shouldn't be there, because it's a backdoor to import scripts and CHMOD777 them. It doesn't always seem to work right, I'm having some issues lately, as it doesn't seem to import my scripts most of the time. Maybe it's because of a shitty SD-card.

Okay I´ve update and it worked. Only got a script error but I think I can ignore that.

The "Import new Scripts" always gives me a "Done" instantly... i don´t really know if it´s actually working :D
I needed to update this way because I previously installed the latest release but many files changed since then here. So maybe publish a new release @jilleb ;-)
And maybe put a "How to update" in the FAQ or readme.
For new users a step by step for the update:

Clone the repo and put it onto a SD-Card
Go into folder \PersonalPOI\PayLoad\0\default on the SD-Card
Copy the "payload.sh" file to the folder \GreenMenu
Rename the file to "mqbcoding.esd"
Put the SD-Card in your unit
Open Greenmenu
Open mqbcoding and navigate to "customization" -> "greenmenu"
Import new Greenmenu
Restart Greenmenu
Done

haha yes, the "import new scripts" is a leftover from the original system Developers. It doesn't give much output, which frustrates me, because sometime it just doesn't work :-P
Anyways I'll make a script+ menu function for just the updating, that will be the most user-friendly I guess. Then it would be enough to copy the new release to SD-card, and the script will just copy from the payload.sh

The toolbox now has an update function :-)
All you have to do is run the script on the main page, while having the latest commit on an SD-card and it will import all the latest scripts and mqbcoding.esd to the unit.


feat: update_table returns stats

update_table returns

num of rows updated for billing - writes

Codecov Report
Attention: Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.10%. Comparing base (c0e1f15) to head (5b72393).

Files with missing lines
Patch %
Lines

rust/lance/src/dataset/write/update.rs
88.88%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2867      +/-   ##
==========================================
- Coverage   78.12%   78.10%   -0.03%
==========================================
  Files         229      229
  Lines       70492    70499       +7
  Branches    70492    70499       +7
==========================================
- Hits        55073    55064       -9
- Misses      12305    12315      +10
- Partials     3114     3120       +6

Flag
Coverage Δ

unittests
78.10% <90.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[WIP] Refactor struct

Refactor chat struct
Implemented missing sharp resource and refactor code

Merged with #25


Could not find VRouterScope above this Builder Widget. when i use vRouter with Getx

final vRouterDelegate = VRouterDelegate(
routes: GetRouter.getRoutes(),
);
return GetMaterialApp.router(
routeInformationParser: VRouteInformationParser(),
routerDelegate: vRouterDelegate,
// Here is the equivalent to VRouter.routes
backButtonDispatcher: VBackButtonDispatcher(),
);

You must put VRouterScope at the top of your widget tree, as mentioned in the error message :wink:

But before today, no such error was reported.

When I use GetX with VRouter, the root widget is GetMaterialApp

But before today, no such error was reported.

You must have updated VRouter

When I use GetX with VRouter, the root widget is GetMaterialApp

Wrap GetMaterialApp in VRouterScope like so:
VRouterScope(
  child: GetMaterialApp(...)
)

Thank you for your answer. Problem solved.


[harfbuzz] build failure

install-x86-windows-dbg-out.log
Host Environment

OS: Windows 7
Compiler: 19.00.24215.1 for x86

To Reproduce
Steps to reproduce the behavior:
vcpkg.exe install harfbuzz[icu] --triplet x86-windows --recurse
Failure logs
Computing installation plan...
The following packages will be built and installed:
    harfbuzz[core,icu,ucdn]:x86-windows
Starting package 1/1: harfbuzz:x86-windows
Building package harfbuzz[core,icu,ucdn]:x86-windows...
Warning: abi keys are missing values:
    freetype
    ragel

-- Downloading https://github.com/harfbuzz/harfbuzz/archive/2.5.3.tar.gz...
-- Extracting source C:/work/vcpkg/downloads/harfbuzz-harfbuzz-2.5.3.tar.gz
-- Applying patch 0001-fix-cmake-export.patch
-- Applying patch 0002-fix-uwp-build.patch
-- Applying patch 0003-remove-broken-test.patch
-- Applying patch find-package-freetype-2.patch
-- Applying patch glib-cmake.patch
-- Using source at C:/work/vcpkg/buildtrees/harfbuzz/src/2.5.3-984271955b
-- Configuring x86-windows
-- Building x86-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:136 (message):
    Command failed: C:/work/vcpkg/downloads/tools/cmake-3.14.0-windows/cmake-3.14.0-win32-x86/bin/cmake.exe --build . --config Debug --target install -- -v
    Working Directory: C:/work/vcpkg/buildtrees/harfbuzz/x86-windows-dbg
    See logs for more information:
      C:\libgdiplus\libgdiplus_cario\vcpkg\buildtrees\harfbuzz\install-x86-windows-dbg-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_cmake.cmake:91 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
  ports/harfbuzz/portfile.cmake:72 (vcpkg_install_cmake)
  scripts/ports.cmake:90 (include)

config-x86-windows-out.log
install-x86-windows-dbg-out.log

See https://github.com/harfbuzz/harfbuzz/issues/1990 and https://github.com/harfbuzz/harfbuzz/issues/1993
It is an issue with a missing semi-colon on line 53 of hb-icu.cc

#define HB_ICU_STMT(S) do { S } while (0)
should read
#define HB_ICU_STMT(S) do { S; } while (0)

Later versions of harfbuzz have this issue resolved as vcpkg is using v2.5.3 and the github/harfbuzz latest is v.2.6.4

@timautry, thanks for the helpful information! I will have a look at this issue.


Manager UI flickers and Table do not expand when clicking on the expand button on the Table

Describe the bug
Table in Manager do not expand when clicking on the expand button on the Table. it only flickers , I don't see any console error.
Test app: https://sg-em-devext.mapsdevext.arcgis.com/apps/instant/manager/index.html?appid=580af16b1142418bb69c2991008039a7

Expected Behavior
Table should expand when expand button is clicked.
Reproduction Steps & Sample
Test app: https://sg-em-devext.mapsdevext.arcgis.com/apps/instant/manager/index.html?
Other Relevant Info
No response

Can't repro this anymore in devext


overlapping functionality in musicanalysis.performance_codec and utils.music

The functions get_time_maps_from_alignment and get_matched_notes of utils.music do very similar things as to_matched_score and other functions in musicanalysis.performance_codec. musicanalysis.performance_codec.monotonize_times seems to be non-functional and/or wrongly documented.
A bit of refactoring and cleanup in the areas of alignment and performance_codec would be great!

Maybe break the file in music into more files for organizational purposes. Have to carefully verify for circular imports.

more comments: note_features and performance_codec seem sowhat misplaced in music_analysis, maybe we should have them in utils as they are basically feature extractors.


Make Profile/Profile Views retain the same order of columns as original DataFrame

When logging a dataframe, order of the columns was not retained in profile/profile_view.
The problem is that frozenset doesn't retain order.

Changed schema's get_col_names to return a tuple instead of frozenset (to ensure order and immutability)
Replaced difference() (that only works with sets) with logic that works with tuples
Added test to assert that order is retained

Description
This repository needs
The commits in this pull request will
Changes

Imperative commit or change title (commit ID)

Add some bullet points to explain the change

Next commit or change (commit ID)

Related
Relates to organization/repo#number

Closes organization/repo#number

[ ] I have reviewed the Guidelines for Contributing and the Code of Conduct.

One thing I'd recommend is to go with an OrderedDict from collections (which is almost the same as the default dict 3.7+ but good to use from collections for capability). This will maintain the O(1) look up we had with the set which will make your list comprehension more efficient while maintaining the order.
Now that does remove the immutability which I'm glad you pointed out. In this case since we are getting the names of something are we super worried about getting the immutability? If so then Tuples would be best way to go.
Personally, it's a pretty common practice to return a list of keys which is mutable. I personally think the api would be in line with others. What do you think?


Added error msg in form fields in donate.html issue #1537

Related Issue
Closes: #1537

Description
I have made changes in form input field in donote.html. In which user can notified by each step about the validation error and correct them.

Screenshots

https://github.com/akshitagupta15june/PetMe/assets/113226919/a2388509-2227-4a67-bcf2-472bd1b092c8
Checklist

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

Could you add the label. @0rigin-c0de @akshitagupta15june

can u fetch and merge with current codebase, so as to resolve conflicts

I have resolved it. Kindly merge it. @akshitagupta15june

could you check my code . where I have implemented validation error . I think it is level2
@akshitagupta15june


feat: Loki Log Streaming

Rebased and resolved package conflicts

@amanji there's a pull request here that refactors some startup to a manage script. Any thoughts on if we merge that if that would cause any issues (or should have change to include) the docker compose startup instructions in this PR?
https://github.com/bcgov/traction/pull/1388/files

@loneil we'd need to add this to the manage script:
https://github.com/bcgov/traction/blob/86fb4304e9c14f893551f8afb8e3abad7308e005/scripts/README.md?plain=1#L132

FYI trying on the deployed PR and hitting start on the logs doesn't seem to show output or an error
See an aggregateError in the tenant UI backend logs:

FYI trying on the deployed PR and hitting start on the logs doesn't seem to show output or an error
See an aggregateError in the tenant UI backend logs:

We need to build in three more services for the logging. I'm assuming the deployed services dont account for those yet

FYI trying on the deployed PR and hitting start on the logs doesn't seem to show output or an error
See an aggregateError in the tenant UI backend logs:

We need to build in three more services for the logging. I'm assuming the deployed services dont account for those yet

@i5okie do you have any suggestiono n how to set things up for the supporting services? i.e.: we list them as pre-requisites or do we include (optional) depenedncies in the Helm chart?

FYI trying on the deployed PR and hitting start on the logs doesn't seem to show output or an error
See an aggregateError in the tenant UI backend logs:

We need to build in three more services for the logging. I'm assuming the deployed services dont account for those yet

@i5okie do you have any suggestiono n how to set things up for the supporting services? i.e.: we list them as pre-requisites or do we include (optional) depenedncies in the Helm chart?

In my opinion, I think Traction should work without throwing exceptions or failing when it runs without Loki backend being configured. As such, I think Loki should be an optional component setup separately by the user. I'd imagine, if loki endpoint is not provided/log-streaming is disabled, the feature should be turned off in someway. Or at least say logs unavailable in a friendly (not app crashing-way)  manner on the page.
As for the promtail, it should be included as an optional sidecar.

FYI trying on the deployed PR and hitting start on the logs doesn't seem to show output or an error
See an aggregateError in the tenant UI backend logs:

We need to build in three more services for the logging. I'm assuming the deployed services dont account for those yet

@i5okie do you have any suggestiono n how to set things up for the supporting services? i.e.: we list them as pre-requisites or do we include (optional) depenedncies in the Helm chart?

In my opinion, I think Traction should work without throwing exceptions or failing when it runs without Loki backend being configured. As such, I think Loki should be an optional component setup separately by the user. I'd imagine, if loki endpoint is not provided/log-streaming is disabled, the feature should be turned off in someway. Or at least say logs unavailable in a friendly (not app crashing) manner on the page.
As for the promtail, it should be included as an optional sidecar. We could put together documentation, with example configs on how to set Traction up with Loki. I can help with the latter two.

I'll update the PR to hide the log feature when the FRONTEND_LOG_STREAM_URL is not set.

I'll update the PR to hide the log feature when the FRONTEND_LOG_STREAM_URL is not set.

@amanji it does not look like changes have been pushed since this - can you confirm? Would you be able to push the above tweak so we can move forward with reviewing/merging the pr?


Cabal version not matching the version on Hackage

Hackage has version <IP_ADDRESS>, while the version in the Cabal file is <IP_ADDRESS>.

Has been fixed


Upload PPT to Records Log

Title of ticket:
Description
Summarize issue
PPTs are sometimes part of the records package. Need to support upload and marking of the PPT as an incompatible file type.
Dependencies
Are there any dependencies?
DOD

[ ] PPT can be uploaded to Records log
[ ] Mark PPT as incompatible file type
[ ]
[ ]
[ ]

Confirmed working, and works with other records gathering functionality (e.g., harms package download, deduplicate, etc.)


NotImplementedError Error thrown during migration

Hello,
Thank you for this amazing mocking library, thanks to it I get rid of ksp and decreased my build times, also finally I was able to move my project to Kotlin 2.0.
During the migration from Mockative to Mokkery I had various issues and couldn't fix 3 of them. This issue is targeting to one of it.
If you check the linked PR, I have added 3 test classes back(These test classes were working fine with Mockative), I had to remove these classes during Mokkery migration since it was giving the below error:
e: Compilation failed: Generation of stubs for class org.jetbrains.kotlin.ir.symbols.impl.IrTypeParameterPublicSymbolImpl is not supported yet

 * Source files:
 * Compiler version: 2.0.0
 * Output kind: PROGRAM

e: kotlin.NotImplementedError: Generation of stubs for class org.jetbrains.kotlin.ir.symbols.impl.IrTypeParameterPublicSymbolImpl is not supported yet
	at org.jetbrains.kotlin.backend.common.linkage.partial.MissingDeclarationStubGenerator.getDeclaration(MissingDeclarationStubGenerator.kt:62)
	at org.jetbrains.kotlin.backend.common.linkage.partial.ClassifierExplorer.exploreSymbol(ClassifierExplorer.kt:96)
	at org.jetbrains.kotlin.backend.common.linkage.partial.ClassifierExplorer.exploreType(ClassifierExplorer.kt:75)
	at org.jetbrains.kotlin.backend.common.linkage.partial.ClassifierExplorer.exploreType(ClassifierExplorer.kt:65)
	at org.jetbrains.kotlin.backend.common.linkage.partial.PartiallyLinkedIrTreePatcher.explore(PartiallyLinkedIrTreePatcher.kt:946)
	at org.jetbrains.kotlin.backend.common.linkage.partial.PartiallyLinkedIrTreePatcher.toPartiallyLinkedMarkerTypeOrNull(PartiallyLinkedIrTreePatcher.kt:949)
	at org.jetbrains.kotlin.backend.common.linkage.partial.PartiallyLinkedIrTreePatcher.access$toPartiallyLinkedMarkerTypeOrNull(PartiallyLinkedIrTreePatcher.kt:43)
	at org.jetbrains.kotlin.backend.common.linkage.partial.PartiallyLinkedIrTreePatcher$DeclarationTransformer.rewriteTypesInFunction(PartiallyLinkedIrTreePatcher.kt:387)
	at org.jetbrains.kotlin.backend.common.linkage.partial.PartiallyLinkedIrTreePatcher$DeclarationTransformer.visitSimpleFunction(PartiallyLinkedIrTreePatcher.kt:265)
	at org.jetbrains.kotlin.ir.visitors.IrElementTransformerVoid.visitSimpleFunction(IrElementTransformerVoid.kt:131)
	at org.jetbrains.kotlin.ir.visitors.IrElementTransformerVoid.visitSimpleFunction(IrElementTransformerVoid.kt:19)

You can see the full log in the link I shared below.
Let me know if anything is needed from my end!
Log: https://github.com/Oztechan/CCC/actions/runs/9615001222/job/26521152675?pr=3583
PR: https://github.com/Oztechan/CCC/pull/3583

Hi! This error is probably associated with mock class generation. Could you check which type is causing the error?

Ok. I was able to reproduce the error on my own. I can confirm this is a bug and I'm working on a fix. Thank you for reporting the issue!

The issue is fixed! I should be able to release it in a few days.

Thank you @lupuuss I confirm that the newly released version fixed the issue.


Detection of dangling documentation

Tasks
This PR resolves #114
Changes

[x] Support of detecting dangling documentation (at the end of Block, UncheckedBlock, ContractDefinition).
[x] Fixes to statement nodes to properly reflect StructuredDocumentation as part of their children.
[x] Related tweaks to documentation reconstruction logic for ASTReader postprocessor.
[x] Related tweaks to ast-to-source writing components.
[x] Related tweaks to affected test snapshots.

Regards.

Codecov Report

Merging #115 (eca25d0) into master (8f22800) will decrease coverage by 0.09%.
The diff coverage is 90.66%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   91.93%   91.83%   -0.10%
==========================================
  Files         253      254       +1
  Lines        5271     5354      +83
  Branches      839      859      +20
==========================================
+ Hits         4846     4917      +71
- Misses        262      269       +7
- Partials      163      168       +5

Impacted Files
Coverage Δ

src/ast/implementation/statement/for_statement.ts
100.00% <ø> (ø)

src/compile/version.ts
100.00% <ø> (ø)

...rc/ast/implementation/statement/unchecked_block.ts
85.71% <66.66%> (-14.29%)
:arrow_down:

src/ast/documentation.ts
76.59% <76.59%> (ø)

...cessing/structured_documentation_reconstruction.ts
98.03% <95.12%> (-1.97%)
:arrow_down:

.../implementation/declaration/contract_definition.ts
100.00% <100.00%> (+2.38%)
:arrow_up:

src/ast/implementation/statement/block.ts
100.00% <100.00%> (ø)

...ast/implementation/statement/do_while_statement.ts
100.00% <100.00%> (ø)

src/ast/implementation/statement/emit_statement.ts
100.00% <100.00%> (ø)

...t/implementation/statement/expression_statement.ts
100.00% <100.00%> (ø)

... and 11 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add author meta tags (profile, links)

Include optional author metadata:

profile - Confirm with authors that they want their photo used as an icon on their page / authors page
github
stackoverflow
twitter

Down the line, extend options for Project and Department Icons
Most of our current iconography exists via Expression Design, but should be converted into SVG.  Partial attempt at conversion exists at Xaml to SVG repo.
It would be good to get those icons out of Expression anyway


Implements test execution Depends On #PR41

Overview
Implements test execution (draft)
Testing/Steps taken to ensure quality
Notes
Needs
PR41 to be merged first
Testing Instructions
How to test this PR Start after checking out this branch (bulleted)

Include test case, and expected output

test


[9.0.3] Stringification does not pass error to componentDidCatch

{h} = require 'dio.js'

class Root
  componentDidCatch: ->
    console.log 'should be called'
  render: ->
    throw new Error 'test'

h(Root) + ''

should be called is not logged (works in 8.2.4)

Makes sense, though then it seems like it should throw
{h} = require 'dio.js'
class Root
  render: ->
    throw new Error 'test'

try
  h(Root) + ''
catch err
  err.message is 'test'

Perhaps that is incorrect. You may still end up with a valid tree (with the failed component removed), which you need access to. Put it on the error object?

I don't understand.


Expose CustomMessageHandler feature

Problem
On the native platforms, it is possible to use two-way communication between the integration and custom-built Player Web UI. This is not possible when using this React Native library.
Changes

Introduced CustomMessageHandler support on PlayerView to enable two-way communication between the React Native integration and Player Web UI when using custom-built Player Web UI.
Updated CustomHtmlUi sample screen to showcase this feature.

Closes #129 and #107