WIP: Attempt to fix the avx512 header issue

Codecov Report

Merging #937 (d7e7c35) into develop (f228cf8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #937   +/-   ##
========================================
  Coverage    90.46%   90.46%
========================================
  Files          129      129
  Lines         9428     9428
========================================
  Hits          8529     8529
  Misses         899      899

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This does seem to fix the issue for me.

For future reference, the underlying cause of the issue appears to be a conflict between some of the cuda system headers and the avx intrinsics headers that were being included by axom. To summarize, cuda implements _Float16 by defining a new type __half and using a macro to forward _Float16 -> __half:
sam@provolone:/usr/local/cuda$ grep -r "_Float16" .
./include/cuda/std/detail/__config: #define _Float16 __half

but the avx intrinsic headers also use _Float16

oops, I sent my previous message before finishing it! I'll edit it to complete the thought.

Closed in favor of an axom update.


installation.md: Remove manual gsplat installation for Windows

I'm reluctant to make changes to the extremely fragile Windows installation instructions as I don't use those instructions myself, but there's been several reports on Discord that this manual installation of gsplat breaks the installation process now that gsplat 1.1.1 is out and not yet compatible with Nerfstudio.
I don't know why gsplat was being installed manually on Windows in the first place since Nerfstudio pulls the right version in itself perfectly fine, including on Windows.
As a sidenote, I have found WSL2 to be much more reliable and consistent for Windows installations (since, once installed, it's just Linux and the Linux instructions apply) and wrote a guide for it a while ago.
Would it be worthwhile including WSL2 instructions as an alternative install method in the official docs?
It does mean a whole different process to setting it up though, so it would probably be better off split off to a separate file if so.

A problem is that the pip installed version(nerfstudio pulled one) will use JIT compile when training for the fisrt time. However, for windows system you have to set up the VC env beforehand or the compile will 100% fail. So gsplat need to be installed from source with the right VC env which makes things easier.
I think a better fix might be fix gsplat to an older version (? and I agree the Windows installation instructions is really fragile.
I use nerfstudio on both wsl and windows and sometimes bug occurs when using vscode+wsl2+Viser (also some performace loss). So I would prefer using them on Windows in my case(

After some discussion on Discord (included below), I've modified the changes to hopefully be more failsafe.
I took some liberties with the format though, and have not been able to test if it works in the generated docs, so if any changes are desired please let me know.
I do intend to write a proper upstream-includeable WSL2 guide that can be included later, but as of ~yesterday Pixi suddenly broke on WSL2, I still have to investigate that, so for now I linked my existing gist guide with a warning.
KevinXu — Today at 10:59 AM
A problem is that the pip installed version(nerfstudio pulled one) will use JIT compile when training for the fisrt time. However, for windows system you have to set up the VC env beforehand or the compile will 100% fail. So gsplat need to be installed from source with the right VC env which makes things easier.
Indeed setting up nerfstudio is really hard on Windows systems and the version reliance on gsplat would break things.🥲

[...]

SharkWipf — Today at 1:13 PM
Hmm, right. But don't you just need to run the vcvarsall step first before running splatfacto the first time? Even with automatically pulled gsplat, does that not fix it?

KevinXu — Today at 1:49 PM
Maybe yes?(I haven't tested it yet. For me it's a little bit awkward if I need to find useable ns-data and run train on it when setting up the ns env for the first time (which is likely to fail several times in the middle)
Anyway, I think a warning for windows installation is necessary

SharkWipf — Today at 1:55 PM
I feel like directly specifying a gsplat version in the docs should be avoided if possible, for 2 reasons:
The user may want a specific nerfstudio version (the latest doesn't support 10-series GPUs and earlier anymore)
The docs won't be kept in sync until people start complaining

So if the vcvarsall thing works, rewording the docs to make that more clear should be preferred I think

KevinXu — Today at 1:57 PM
Make sence. I think we should fix that this way.

Thanks a lot for the doc update! Unfortunately I don't have access to a windows machine so I can't test it but if you and @KevinXu02 are happy with the changes now I can merge this. To test the generated docs you can use this, then VSCode has a Live Server extension that you can open the index.html file in. If you can't test it I can but it may take me some time to get around to it.

Noted, thanks. I'll see if I can get that working later this week.
I should note I haven't been able to directly tested this myself yet either, but I am pretty sure that this is how it works, it seems to have worked for others, and if nothing else it's not worse than the current version, which is just broken atm.

Took a little longer than anticipated due to me getting sick, but the docs seem fine when built, no issues there.
I made a minor change in wording in the troubleshooting section, everything looks good (enough) to me now, at least until a proper full rewrite/cleanup gets done.


bump to sdk v0.46.4

Summary
The change noted here means we should remove the event emission we do after context caching. Should be done in combination with the SDK bump

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Please note event emission should be left for an situation in which we are using the cacheContext() for msg routing as the msg router will create a new event manager. We should still emit events from the msg result


Documentation Backlog

High Priority
Kind

[ ] Apply
[x] Unapply
[x] Compose
[x] Pipe
[ ] Reify

Type

[ ] Cast
[ ] Infer
[ ] Display
[ ] ValueOf

Function

[ ] Identity
[ ] Constant

Conditional

[x] IsSupertypeOf

List

[x] Reduce
[x] Filter
[x] Map
[ ] Every
[ ] Some
[ ] Accumulate
[ ] Iterate
[x] Zip
[ ] Range

String
Object
Lower Priority
List

[ ] Pair
[ ] Times
[ ] Pop
[ ] PopN
[ ] Push
[ ] Shift
[ ] ShiftN
[ ] Unshift
[ ] Find
[ ] Includes

Number
Union

Superseded by #52


Incomplete typing for AsyncConnection.execute's parameters

Describe the bug
AsyncConnection.execute's parameters typing info is missing Sequence[Mapping[Any, Any]]
Expected behavior
The following example to not have type error.
To Reproduce
import asyncio
from sqlalchemy import Table, MetaData
from sqlalchemy.ext.asyncio import create_async_engine

meta = MetaData()
t1 = Table("t1", meta)

async def async_main() -> None:
    engine = create_async_engine(
        "postgresql+asyncpg://scott:tiger@localhost/test",
        echo=True,
    )

    async with engine.begin() as conn:
        await conn.run_sync(meta.drop_all)
        await conn.run_sync(meta.create_all)

        await conn.execute(
            t1.insert(), [{"name": "some name 1"}, {"name": "some name 2"}]
        )
    await engine.dispose()

asyncio.run(async_main())

Error
> mypy --strict .\test.py
test.py:20: error: Argument 2 to "execute" of "AsyncConnection" has incompatible type "List[Dict[str, str]]"; expected "Optional[Mapping[Any, Any]]"
Found 1 error in 1 file (checked 1 source file)

Versions.

OS: Windows 10
Python: 3.9.6
SQLAlchemy: 1.4.23
mypy: 0.910
SQLAlchemy2-stubs: 0.0.2a13

Have a nice day!

Thanks for reporting

Released as v0.0.2a15


Python Skill Code is missing an argument.

In the documentation the the python script simple-skill.py is missing something in line 33
  File "simple-skill.py", line 33
    client.publish("hermes/tts/say", json.dumps({"text": , "siteId": site_id}))
                                                         ^
SyntaxError: invalid syntax

What is needed?

Thanks for reporting. This should be
client.publish("hermes/tts/say", json.dumps({"text": sentence, "siteId": site_id}))

I updated the documentation accordingly. @synesthesiam are the docs automatically rebuilding or do you have to start a rebuild manually?

They used to re-build automatically, but now ReadTheDocs seems to want me to click a button. Rebuilding now...

It's in the docs now


Use $transitions and AuthService

Ui-Router 1.* deprecated the $stateChange* events, but provides
a shim for migrating with minimal effort. In the long-run
we should use ui-router's own, new access-control-system
and the $transition-API.
An example for a migration path can be found here.

@m0ru still an issue?

yep it is. then again angular 1.x is deprecated in general. i think we can close this issue as we'll need to change the entire thing anyway (or defer the decision until we get around to switching to ts+react or elm)

closed as it is just one symptom of our need to update angular (or get rid of it)


Clear Indication of copying files in background

What's the Problem?
It is not very clear that files are being copied. Especially if one closes the window after having started the process. The files are still being copied which is unclear for the user
Solution/Idea
If that happens, a copy dialog box like the standard windows explorer would be nice.
Alternatives
an option to toggle such a dialog that would either inform the user that files will still be copied in the background or if the files explorer is closed, that a copying pop up dialog appears until the files have finished being copied.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 Insider Preview 22533.1001
Comments
No response

Merging with #8207


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-30-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4706

Questions ?
Please refer to the Backport tool documentation


Grant Azure Stream Analytics access to Azure sql db without public network access

[Enter feedback here]
Is it possible to have input/output sql database in ASA without setting public network access in sql db side ? in this document it seems enable public network access is a requirements and not sure how it will work for the organization with higher security measures in place ( deny public network access)
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: daca22cc-4265-c99d-f0df-00c7dda3aefd
Version Independent ID: f83514e0-bbaa-239a-0077-98939721c220
Content: Use managed identities to access Azure SQL Database or Azure Synapse Analytics - Azure Stream Analytics
Content Source: articles/stream-analytics/sql-database-output-managed-identity.md
Service: stream-analytics
GitHub Login: @an-emma
Microsoft Alias: raan

@Hamid-R
Thanks for your feedback! We will investigate and update as appropriate.

@Hamid-R
Thank you for your question. In the document you referenced, it is mentioned that enabling public network access for the SQL Database is a requirement for using SQL Database as an output for Azure Stream Analytics. However, it is possible to use Private Endpoints to connect to Azure SQL Database without allowing public network access.
Private Endpoints allow you to securely access Azure SQL Database over a private endpoint within your virtual network. This means that you can access the SQL Database without allowing public network access. To use Private Endpoints with Azure Stream Analytics, you will need to create a Private Endpoint for the SQL Database and then configure the ASA job to use the Private Endpoint.
For more information on using Private Endpoints with Azure SQL Database, please see the following documentation:
Azure SQL Database Private Endpoint
Connect to Azure SQL Database over Private Endpoint
I hope this helps! Let me know if you have any further questions.

The issue is solved by Microsoft suggestion after we submitted a ticket. We can not use PE or VNet in Straem Analytics job and we need to create Stream Analytics Cluster first and then use it to run our Stream Analytics job. ( https://learn.microsoft.com/en-us/azure/stream-analytics/cluster-overview#how-do-i-choose-between-a-stream-analytics-cluster-and-a-stream-analytics-job).


Synchronize scale with NetworkTransformBase

Closes #999
Previously only position and rotation were synchronizedin NetworkTransformBase, with this commit scale is synchronized similarly to position.
I played with it a little and it seems to work as expected. (Created an empty project and changed the scale on a NetworkTransform-ed object manually and through a script as well and it synced correctly)
Please let me know if I should do something differently, or write some tests, or whatever else.

Looks like a clean modification to me. This will help various usage cases. I approve, let's see what the others have to say.

This would increase the bandwidth usage a lot... I'd sync it separately only when it changes.

@MichalPetryka  in most games scale won't change,  so this won't increase the amount of messages sent back and forth.  It will increase the size of each message by 12 bytes. Not the end of the world,  and fairly clean in my opinion.
What we could do,  is have a 1 byte bitmap at the beginning indicating which one of the 3 fields is changing,  that way we only send the data that changed

Another option is to have 3 booleans in the component to indicate which fields are being synchronized.
public bool SyncPosition = true;
public bool SyncRotation = true;
public bool SyncScale = true;

public void Serialize() {
     if (SyncPosition) {
           //  serialize the position
    }
    if (SyncRotation) {
        // serialize the rotation
   }
   if (SyncScale) {
      //  serialize the rotation
   }
}

Agree with Petris and Paul.  Scale isn't a common ask, and I'd think rarely changes relative to the frequency of change of position & rotation, so carrying that extra 12 bytes in every message to the server and out to all clients seems like unnecessary bloat, especially when there's a few hundred clients sending and receiving these messages @ 10Hz.  A Flags Enum makes sense to me to control what gets synced, and/or a control bit to indicate what's being changed in a given msg.

@paulpach With a bitmap we could also separately diff x, y and z of location and rotation to save even more, ie 3 bits for location, 3 bits for rotation and 1 bit for scale

I wouldn't worry about bitmaps yet. let's add scale first, see if it works well, then we can still optimize :)

nice PR @Zajozor . are you in our discord?

@vis2k Thanks, I am, same nick.
I could also look into selective syncing based on flags enum, if you find it useful.
As far as I understand it shouldn't be lot of a work and involves only changes in the NetworkTransformBase, doesn't it?

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


LinearGradient in ART for Android cant work, display black

LinearGradient in ART for Android cant work, display black

This features are not implemented on Android yet)

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/lineargradient-in-art-for-android-cant-work-display-black
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain thumbmunkeys.com has exceeded the max emails per hour (62/60 (103%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from filter01.reliabledns.org ([<IP_ADDRESS>]:56043)
by en02-ams.stablehost.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1bzsux-0003w6-7H
for<EMAIL_ADDRESS>Thu, 27 Oct 2016 22:13:00 +0000
Received: from github-smtp2-ext1.iad.github.net ([<IP_ADDRESS>] helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by filter01.reliabledns.org with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.86)
(envelope-from<EMAIL_ADDRESS>id 1bzsuv-0000MT-QN
for<EMAIL_ADDRESS>Thu, 27 Oct 2016 15:12:58 -0700
Date: Thu, 27 Oct 2016 15:12:56 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1477606376;
bh=oTzDSk0DJ/cDcDaw1bekQsRQtCtSB914EZiuYu6+WO8=;
h=From:Reply-To:To:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=jVXclKB90A6/+jqmehRUQdDnocobmcqFQSfEwgDXrxAzPEl1D5ABEwq04a9waofj2
7fk5Ip1LPhf9duLUdpZO9qMoWcRCSj5lckctaiP0hz+RO737Ad+eeh0Vy7MKjOFMNW
Wk0RbUcqlzc6kTvs0CwSRoI+t5FJSZGFBkv8XloY=
From: Martin Konicek<EMAIL_ADDRESS>Reply-To: facebook/react-native<EMAIL_ADDRESS>To: facebook/react-native<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [facebook/react-native] LinearGradient in ART for Android cant
work, display black (#8506)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_58127be8d4572_12f83fb4d97e92bc638068";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: mkonicek
X-GitHub-Recipient: bunnyboy1
List-ID: facebook/react-native <react-native.facebook.github.com>
List-Archive: https://github.com/facebook/react-native
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ARoNg-gtjdX407Bab1MLaTBkWrYL9Q1Dks5q4SHogaJpZM4JB_Pk
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>Received-SPF: pass (filter01.reliabledns.org: domain of github.com designates <IP_ADDRESS> as permitted sender) client-ip=<IP_ADDRESS><EMAIL_ADDRESS>helo=github-smtp2a-ext-cp1-prd.iad.github.net;
X-SPF-Result: filter01.reliabledns.org: domain of github.com designates <IP_ADDRESS> as permitted sender
Authentication-Results: filter01.reliabledns.org; dmarc=pass header.from=github.com
X-Filter-ID: s0sct1PQhAABKnZB5plbIamwIYzgudGL4tMY9zS3pea7KY6rzXjgC+rgKYUjsj8D0JuF70A4EJMc
Wn6O/bsgoWSYW6KbZfj8apx6ohl75uX35Caz8bqM3i8tzwcg+lnkZzoX7PhhDs80mBQKIGWwu1FO
8N/DTQ2/uVBeekp7txxwJVSL6KLw+g28lkb31HyS94pE1Cssu3RLQ0/o4r6NbA8zp1yYIFRkRN8y
Bn+CxV2xPNeVsvIjmFzFQveyO9Rcuz9ZF/JHvUjHA5dml8mY85z3A1ed2ITkOuMtrwds5D6JpTbQ
ANmQ7L2TfsK5NgTRJq/4FZbX1lzD9O+fyiIjB8wTNR/tjjZvmBdM/4lPZO2sASVdHfbtk3LlI8pB
wr4jGeKz8vlEl/GSEiCnxc6Ard0BaVp3jhK5GOBJIv+LCtD+V9//iYPXcUPDr8DL9oJhN0JkcMmQ
sV7j9toT40ga/GYzx2Zgyhf3OXiddZgacWc1/+H6OHgtV1bmxcq0CB3pmrFpDzdgJQc/rZDiOC7K
RpZxGk3X7U/CrpemaTYEqjR6fyHrrsngmi97syQy+4L3p7+eg7tptOUU98CNo4IBbK0jiD6XqsJZ
tjQxlyCdsezJow6Hi0pxrFS5O4yOAY8o6W0BvTIC60F9LNR/0kDSJQ==
X-Report-Abuse-To<EMAIL_ADDRESS>Authentication-Results: reliabledns.org; spf=pass<EMAIL_ADDRESS>Authentication-Results: reliabledns.org; dkim=pass header.i=github.com
X-SpamExperts-Class: ham
X-SpamExperts-Evidence: dnswl/se-reputation (trusted ham)
X-Recommended-Action: accept
X-Spam-Status: No, score=1.1
X-Spam-Score: 11
X-Spam-Bar: +
X-Ham-Report: Spam detection software, running on the system "en02-ams.stablehost.com",
has NOT identified this incoming email as spam.  The original
message has been attached to this so you can view it or label
similar future email.  If you have any questions, see
root@localhost for details.
Content preview:  Closed #8506. -- You are receiving this because you are subscribed
to this thread. Reply to this email directly or view it on GitHub: https://github.com/facebook/react-native/issues/8506#event-839325086
[...]
Content analysis details:   (1.1 points, 5.0 required)
pts rule name              description

-0.0 RCVD_IN_DNSWL_NONE     RBL: Sender listed at http://www.dnswl.org/, no
trust
[<IP_ADDRESS> listed in list.dnswl.org]
1.5 SPF_SOFTFAIL           SPF: sender does not match SPF record (softfail)
1.6 HTML_IMAGE_ONLY_24     BODY: HTML: images with 2000-2400 bytes of words
-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%
[score: 0.0001]
0.0 HTML_MESSAGE           BODY: HTML included in message
-0.1 DKIM_VALID             Message has at least one valid DKIM or DK signature
-0.1 DKIM_VALID_AU          Message has a valid DKIM or DK signature from author's
domain
0.1 DKIM_SIGNED            Message has a DKIM or DK signature, not necessarily valid
X-Spam-Flag: NO
----==_mimepart_58127be8d4572_12f83fb4d97e92bc638068
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #8506.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/facebook/react-native/issues/8506#event-839325086
----==_mimepart_58127be8d4572_12f83fb4d97e92bc638068
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #8506.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_58127be8d4572_12f83fb4d97e92bc638068--


Allow Object Hydration

As PDO allows results to be mapped to a specific object, Pixie has the means to use custom objects as the return object
This will need to be built custom as WPDB uses its own return types.

We have access to the container,so this could be done. Will have to use the WPDB return types, to determine how we populate these (array, object and using keys)


Use stack's ghc-pkg

If I use ihaskell install --stack to install kernel, should it uses stack's ghc-pkg instead of tries to find in $PATH?

How do you propose using Stack's ghc-pkg?

stack exec ghc-pkg or set the search path for all tools to the output of stack path --bin-path?

My understanding is that we are doing essentially this here: https://github.com/IHaskell/IHaskell/blob/465fded2f705cedbc791d210508ca9febc04a208/main/Main.hs#L143-L144
If you have ideas for making this more robust, let me know.

Strange... According to the code, kernel will try to run stack, and check whether The Haskell Tool Stack is printed. If so, it will run stack exec env. I checked that running stack does print the string, and running stack exec env will be very slow because Stack will download and install GHC (it hasn't been installed yet). However, when I create a notebook with IHaskell kernel, log immediately shows ghc-pkg cannot be found. Apparently stack exec env is not invoked.
Is there anything I can provide to help?

Why does running stack exec env download and install GHC? That indicates that there is a version mismatch between the LTS in the ~/.stack/global-project.yaml file and the LTS used to install IHaskell.

Sorry, my mistake.
After edited ~/.stack/global-project/stack.yaml to match the LTS version (18.28), I re-tested these:

stack exec env, find the PATH=... line.
export PATH=...
Confirm that ghc-pkg is now in PATH and is functional.
Open a new shell where PATH has not been modified.
jupyter-notebook, connect, create an IHaskell notebook. It fails with:

[I 12:31:22.151 NotebookApp] Kernel started: 624c4434-4a6c-4ca0-bdde-0a387710d176, name: haskell
ihaskell:
Ran commands:
ghc-pkg check
which ghc-pkg

Exception: user error (shelly did not find ghc-pkg in the PATH: /home/app/.local/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games)

Seems IHaskell is not using the PATH from stack exec env?

Did you install IHaskell with ihaskell install --stack?

Yes. This is the content of ~/.local/share/jupyter/kernels/haskell/kernel.json:
{"display_name":"Haskell","language":"haskell","argv":["/home/app/.local/bin/ihaskell","kernel","{connection_file}","--ghclib","/home/app/.stack/programs/x86_64-linux/ghc-tinfo6-8.10.7/lib/ghc-8.10.7","+RTS","-M3g","-N2","-RTS","--stack"]}

https://github.com/commercialhaskell/stack/commit/49bf07ce76e6c8e9e63a0a582e8a2ae0078e93b3
stack changed behavior in this commit.
Before: Print on stdout and exit with 0.
After: Print on stderr and exit with 1.


Contract supporting migration staging path

Closes: #13

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (dd81300) 74.30% compared to head (12c2eb9) 75.32%.

Files
Patch %
Lines

contracts/StagedContractUpdates.cdc
94.44%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   74.30%   75.32%   +1.01%
==========================================
  Files           1        1
  Lines         144      154      +10
==========================================
+ Hits          107      116       +9
- Misses         37       38       +1

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


Add filtering option on warnings and errors in Problems Output tab

Similar to Unity engine console logger..
While we may be able to filter by problems we are not able to filter by errors or warnings instead I am presented with huge list and have to scroll through it! I would like to have option to quickly hide and unhide warnings instead seeing them all in list.. As well as having option to hide and unhide specific errors... Thank you!

Currently, it's possible to filter by "info", "warn", "error" strings - it works fine.
The problem is, it's not quite intuitive. I discovered it just now from the issue #8305
Providing input suggestions to input string would improve the discoverability of the feature.
VisualStudio-style toggles by type is what people most likely expect to see in Problems window.

There should be an additional dropdown for filter based on type, next to the filter textbox. It would be more convenient.

Toggle buttons for every level (VisualStudio-style) seems more convenient to me, since it allows for "show everything except info" behaviour.
The only thing I'm not sure about is the possible UI clutter with more controls. But additional dropdown will take as much space.
One possible approach to avoid cluttering UI with controls and provide more flexibility at the same time:
Everything still has to be done in filter box, but dropdown with some helpers has to be added to it, that will help to construct queries - include and exclude stuff. Something that will allow to select levels, sources (plugins), strings (may be prefilled with frequent words from existing problems) and has two buttons - "include" and "exclude".
List of frequent queries would also be helpful.

also, typing error into the box to filter for errors doesn't work as expected - problems of type error remain, but problems of type info (the ones with the blue i) also remain, if their text contains the word 'error'.

Duplicate of #8305


sometimes below_the_fold lacks propertyHistoryInfo

Sometimes, the below_the_fold API call doesn't return a propertyHistoryInfo key. I speculate it's because this house is off-market.
response = Redfin().search("3947 Kings Row Reno Nevada")
url = response['payload']['exactMatch']['url']
initial_info = Redfin().initial_info(url)
property_id = initial_info['payload']['propertyId']
btf_data = Redfin().below_the_fold(property_id)
print(btf_data["payload"]["propertyHistoryInfo"])
Traceback (most recent call last):
  File "<console>", line 1, in <module>
KeyError: 'propertyHistoryInfo'

When I view the page on Redfin, the data is there.
I also tried to find this data in other API calls but couldn't find it.
Any help is appreciated!

I found you can get this data by setting accessLevel
mls_data = Redfin().meta_property('belowTheFold', {'propertyId': 68008829, "accessLevel":1}, page=True)

I found setting listingId worked to get propertyHistoryInfo always:
mls_data = Redfin().meta_property('belowTheFold', {'propertyId': property_id, "listingId":listing_id}, page=True)

Haha, related to #6 -- I guess it's an optional parameter.

Noted by newest commit (1696beeb1d56118d75d15a2577a8472ea268df4e).

My issue was the avm_details call failing if listing_id was missing.
FWIW, I found passing a dummy listing_id value did the trick...
avm_details = client.avm_details( property_id , 1 ) ;


Switch but externally controlled check prop

🚀 Feature request
Recently @zag-js/switch has been released. I'm looking for an option to externally (via props) control the checked state
🧱 Problem Statement / Justification
While it's possible to define defaultChecked which sets the initial state of the machine, it's not possible to update the switch's state from the outside (via props)
✅ Proposed solution or API
I tried to use useEffect together with api.setChecked but the state won't change:
import type { FC } from 'react';
import * as zagSwitch from '@zag-js/switch';
import { useMachine, normalizeProps } from '@zag-js/react';

interface SwitchProps {
  checked: boolean;
  disabled?: boolean;
}

export const SwitchTmp: FC<SwitchProps> = ({ checked, disabled }) => {
  const [state, send] = useMachine(
    zagSwitch.machine({
      disabled,
      id: generateId(),
    }),
  );
  const api = zagSwitch.connect(state, send, normalizeProps);

  useEffect(() => {
    api.setChecked(defaultChecked);
  }, [checked]);

  return <>...</>;
};

@nephix Please attach a codesandbox reproduction of what you're trying to do.

@anubra266 I got it to work now, was a mistake on my end 😅


Modify the function of the same name

Blynk has the same function _send as its parent class BlynkProtocol. When BlynkProtocol is ready to send data, self.conn.send(data) in the Blynk class cannot be called because of the same name function _send in the BlynkProtocol class. Therefore, the socket does not send any data, which will raise connection failure.
I noticed that in the recent commits, you modified the function name, causing the function to have the same name issue.

Suggest a better name, instead of just removing the underscore.
Maybe something like "msgSend"

Sorry, this was fixed already


Report partially covered branches as not covered

Since coverage.io does not currently support branch coverage separately, I would like to mark those lines with only partial branch coverage as not covered.
If branch coverage is collected, this marks partially covered lines as not covered. If this implicit approach is not preferred, I could add an option for enabling this.

This change is

The failures seem to be from dependencies, not my code, right?

@jvarho hey, thank you for contribution. Unfortunately I'm not planning to maintain this project, your PR urged me to to state it explicitly #122

Too bad, hopefully you find someone to take over. If not, I may fork this with a subset of the functionality.

Hey there! This repo just underwent a change in maintainers -- myself and @goanpeca will be taking over this project. Give us some time to get up-to-speed and we'll make sure to take a look at this PR soon.

It looks good @TheKevJames, but no time to test it right now


Remove tests on remote meilisearch server

fixes: #642
Since tests were only done on one environment any way during CI but also local testing I removed the testing all together on playground using the remote meilisearch instance

bors merge

bors merge


Re-add usage bars to org and space cards

Fixes #3225

Discussed this with Nathan. We're putting this on hold until both the org/space status indicator & app scaling in cf section has been integrated

Need to see how this looks with more colour on screen
In some cases we won't be able to show the quota bar (we can't determine the stats due to scaling issues). Need to see how this looks
Given the above, need to consider removing the bars and instead, to indicate quota near/reached, colour the consumed value. This would match the pattern that we use in the smaller cards in the org/space summary pages


cannot install cirq

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
The cirq package cannot be installed in a plain test environment.
$ conda create -n test python=3.9
$ conda activate test
$ conda install cirq
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: -
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions
Note that strict channel priority may have removed packages required for satisfiability.

My condarc:
$ cat ~/.condarc
channels:
  - conda-forge
channel_priority: strict
changeps1: false
auto_activate_base: false
default_threads: 3
# extra_safety_checks: True
create_default_packages:
  - coreutils

Installed packages
# Name                    Version                   Build  Channel
bzip2                     1.0.8                h0d85af4_4    conda-forge
ca-certificates           2022.6.15            h033912b_0    conda-forge
coreutils                 9.1                  h5eb16cf_0    conda-forge
libffi                    3.4.2                h0d85af4_5    conda-forge
libsqlite                 3.39.2               h5a3d3bf_1    conda-forge
libzlib                   1.2.12               hfe4f2af_2    conda-forge
ncurses                   6.3                  h96cf925_1    conda-forge
openssl                   3.0.5                hb81d4ab_1    conda-forge
pip                       22.2.2             pyhd8ed1ab_0    conda-forge
python                    3.9.13          hf8d34f4_0_cpython    conda-forge
readline                  8.1.2                h3899abd_0    conda-forge
setuptools                65.3.0             pyhd8ed1ab_1    conda-forge
sqlite                    3.39.2               hd9f0692_1    conda-forge
tk                        8.6.12               h5dbffcc_0    conda-forge
tzdata                    2022c                h191b570_0    conda-forge
wheel                     0.37.1             pyhd8ed1ab_0    conda-forge
xz                        5.2.6                h775f41a_0    conda-forge

Environment info
active environment : test
    active env location : /Users/tpdownes/local/miniconda3/envs/test
            shell level : 2
       user config file : /Users/tpdownes/.condarc
 populated config files : /Users/tpdownes/local/miniconda3/.condarc
                          /Users/tpdownes/.condarc
          conda version : 4.14.0
    conda-build version : not installed
         python version : 3.10.6.final.0
       virtual packages : __osx=12.5.1=0
                          __unix=0=0
                          __archspec=1=x86_64
       base environment : /Users/tpdownes/local/miniconda3  (writable)
      conda av data dir : /Users/tpdownes/local/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/osx-64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /Users/tpdownes/local/miniconda3/pkgs
                          /Users/tpdownes/.conda/pkgs
       envs directories : /Users/tpdownes/local/miniconda3/envs
                          /Users/tpdownes/.conda/envs
               platform : osx-64
             user-agent : conda/4.14.0 requests/2.28.1 CPython/3.10.6 Darwin/21.6.0 OSX/12.5.1
                UID:GID : 828971:89939
             netrc file : None
           offline mode : False

I have tried adding the defaults channel but it has no impact.

Can you please post the full output of this:
conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions
Note that strict channel priority may have removed packages required for satisfiability.

I need to know which packages are incompatible.

That is the full output. conda is notoriously unhelpful about package conflicts. You might look at mamba as a tool which may be faster/better/more useful.

Hmm weird. Can you please run it again with mamba then? Its output should be more helpful.

Have you tried replicating my report?

Yes not getting the error.

Ok was able to verify this in a virtual box. The error produced by mamba is nothing provides h2 3.* needed by httpcore-0.11.1-py_0.
Please file an issue in either of these feedstocks as this is not related to the cirq distribution.

Well, it's a warning sign that this wasn't detected in the automated builds of cirq.
I think the issue is that the httpcore conda package inaccurately claims that h2==3.* is required (vs "extra").

https://github.com/conda-forge/httpcore-feedstock/blob/46295b93e1f9d5aee5e44a88492d4417aab46ace/recipe/meta.yaml
https://github.com/encode/httpcore/blob/0.11.1/setup.py

On top of that:

h2==3.* is sufficiently old that there is no Python 3.9 build
I'm dubious that whatever is depending on httpcore==0.11.* (late 2020) couldn't make use of a more recent copy of httpcore

Well, it's a warning sign that this wasn't detected in the automated builds of cirq.

The CI just tests that the package can be installed under our newest supported version. As I said, this is not a bug but just a missing combination of OS, Python version and cirq version.

I think the issue is that the httpcore conda package inaccurately claims that h2==3.* is required (vs "extra"). pip install cirq does not install h2.

That could be. In that case, please file an issue in their feedstocks, as this one manages only the cirq recipe.

Could you show me what package is actually depending upon httpcore?

Cirq-rigetti 0.15. Your solver cannot solve your installation of v1 as that fails with nothing provides msgpack-python >=0.6,<1.0 needed by rpcq-3.10.0-pyhd8ed1ab_0, downgrades to a lower version and fails there as well.
Ref: https://github.com/quantumlib/Cirq/blob/v0.15.0/cirq-rigetti/requirements.txt

I appreciate that there are problems downstream of this repo. At the same time, there are presently 4 supported releases of Python on conda-forge: 3.7-3.10. Very soon that will be 3.8-3.11. The evidence above and below suggests to me that this package only works on the soon-to-be oldest release of Python on conda-forge.
$ conda create -n test310 python=3.10
$ conda activate test310
$ conda install cirq
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: \
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions
Note that strict channel priority may have removed packages required for satisfiability.

$ conda create -n test38 python=3.8
$ conda activate test38
$ conda install cirq
# goes on to succeed because h2==3.2.0 is built for 3.8

I am able to understand the problem downstream, but there is a problem with how conda-forge is testing this package if the automated checks are not actually catching whether it can be installed on the 2 most recent (and supported) releases of Python.

Yes and no. These supported python ranges are only for compilation and indicate where it can be compiled but not necessarily where it should be.  For noarch packages, this is different.
Also, supported does not mean that it will actually work on these versions if there are version constraints, something is not available for a specific OS, etc. If you run the commands on Linux it should give you a different output.
As we are entirely volunteer-run, conda-forge strives to include most possibilities but if something is missing, the person needing it should add it ideally themselves.

I am able to understand the problem downstream, but there is a problem with how conda-forge is testing this package if the automated checks are not actually catching whether it can be installed on the 2 most recent (and supported) releases of Python.

No, because of the mentioned above. We do not test all supported versions for packages that are noarch, only for compiled ones as the constraint is that something is not available for a platform, not that the package is broken there.


Add codemirror to existing text areas

@pjhampton @benoitc Thoughts ?

Wow! Great job @sivsushruth
This looks awesome :+1:

Just built from PR, works like a charm )) Great! 👏
https://gist.github.com/ermouth/953691b6b62c9cab4ff6
Here is an old gist showing how to use Uglify to both validate JS and figure out global vars used. To make it work fine, anonymous function‘s source must be wrapped with ().

@pjhampton  @ermouth Thanks for the quick turnaround.
Will work on this further.

I intend to wrap up the editor functionality a bit more along with the validation @ermouth suggested.
Although, do you guys think, the PR needs more work done before merging with master ?
If this is merged, can take this up for a new branch for 'futon-upgrade' and work on it further.
Let me know

i will test this morning :) imo if it works let's merge it and do the rest
in a separate patch :)
On Sun, 3 Apr 2016 at 05:46, Sushruth Sivaramakrishnan <
<EMAIL_ADDRESS>wrote:

I intend to wrap up the editor functionality a bit more along with the
validation @ermouth https://github.com/ermouth suggested.
Although, do you guys think, the PR needs more work done before merging
with master ?
If this is merged, can take this up for a new branch for 'futon-upgrade'
and work on it further.
Let me know
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/barrel-db/barrel-platform/pull/36#issuecomment-204861571

merged. Thanks!


specific example on how to call native Android method to save photo?

Can you add a specific example on how to call native Android method to save photo?
Thanks

1


Visual Studio Debug Problem

I try to use Visual Studio Code to debug
I follow this instruction
In vscode launch config:
{
    "name": "Launch via NPM",
    "type": "node",
    "request": "launch",
    "cwd": "${workspaceRoot}",
    "runtimeExecutable": "npm",
    "runtimeArgs": [
        "run-script", "debug"
    ],
    "port": 5858
}

In packages.json, I add this in scripts::
"debug": "webpack-dev-server --nolazy --debug-brk=5858 --open --hot --inline --config webpack.dev.js",

However, debug doesn't work. How to fix?
If not, any other IDE with debug you suggest?
Thank you

Hello
You can debug directly in the browser, its the best way to do it, the debug toolset in chrome or FF is quite good.
Thanks a lot!


[Feature] Task timeout and SLA support

Description
In Airflow, both DAGs and tasks can have timeout and SLA times specified. Since dbt models likely have varying expected execution times for each layer, there could be cases where users want to apply SLAs individually to each node.
How about having Cosmos retrieve timeout and SLA times from the node metadata and apply them individually when rendering nodes?
Use case/motivation
No response
Related issues
No response
Are you willing to submit a PR?

[x] Yes, I am willing to submit a PR!

hi @t0momi219 . Sounds like a great idea. I am definitely up for the idea of timeouts. When it comes to SLA, I'm hearing that they might take away the concept of SLAs in Airflow 3 and come up with something better in later releases of AF 3. So, if you're willing, definitely would encourage the contribution for supporting timeout from node metadata and applying it to AF tasks.


Sound name is not valid

Hello, when I try to type someone with a snowball, in the server logs it is noted: Sound Name: SHOOT_ARROW is not valid. Can you fix this problem please?
(I tried to put another name of his but it doesn't work either)

What Minecraft version are you using?


Staking feasibility check should ensure the results are de-deplicated

Should add tests to ensure exactly that the submitted winners do not contain duplicates. We totally missed this in the current round and it seems to be possible to submit duplicates.

This is already ensured:
As of now, we do:
https://github.com/paritytech/substrate/blob/38d5bb32c3064113f897cb8ec33eea0f8570981b/frame/staking/src/lib.rs#L2467-L2475
which ensures that the number of unique targets in compact is same as winner.len(). Moreover, in build_support_map() we ensure that all of these targets map to one of the w in winners. Thus, duplicates are detected.
For example, assume there are 4 winners but 3 of them are unique. Then, for the above check to pass the compact MUST present 4 unique targets. Once we reach build_support_map, at least one of these will not be present in the winners list and the call will fails.


feat(checkbox): checkbox-group support checkbox slot

Types of changes

[x] New feature
[ ] Bug fix
[ ] Enhancement
[ ] Component style change
[ ] Typescript definition change
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Test cases
[ ] Continuous integration
[ ] Breaking change
[ ] Others

Background and context

Solution

How is the change tested?

Changelog

Component
Changelog(CN)
Changelog(EN)
Related issues

checkbox
checkbox-group 增加 checkbox 插槽
checkbox-group adds checkbox slot

Checklist:

[ ] Test suite passes (npm run test)
[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others
should be submitted to main branch)

Other information

看到有文档 slots 没更新，应该要跑下 docgen


Types: Export UpdateModeEnum as const

Allows a typescript project to consume UpdateModeEnum.
It is basically the same fix from https://github.com/chartjs/Chart.js/pull/9046 applied to a different enum.
I'd consider this as an extended fix for https://github.com/chartjs/Chart.js/issues/8993

@kurkle I think we need to revert of fix this. It seems to be causing errors in #10728

How could it be fixed? Reverting is ok by me.

Got it fixed in #10729. It needs to be export declare const


How do I show hidden files by default?

Is your feature request related to a problem? Please describe.
I have the following in my rc.conf
set show_hidden true

and in vanilla ranger, I can see hidden files by default. However, in this plugin, I can't see hidden files by default. I don't necessarily want to enable "vanilla mode" in this plugin, but I can't find any other way based on the documentation to show hidden files by default.
Describe the solution you'd like
I would like to have an option that I can set in order to show hidden files by default
Describe alternatives you've considered
I considered creating a separate configuration using g:rnvimr_urc_path but I'm not sure if this is a future proof idea. It means that if the default configurations in this plugin change with an update, I won't receive that update when I update the plugin, no? Also, I need to copy the current default configuration into this new config file, no?
Another alternative I can imagine is to provide the option to somehow merge 2 config files. So similar to g:rnvimr_vanilla but it should be an option that merges the default settings defined by this plugin with the user settings defined in a file or the global ranger config.

can't reproduce, try minimal config.

I can't reproduce it anymore either


Allow user input of melodies

Predecessors:

[ ] #10
[ ] #11

Tasks:

[ ] Combine fields of #10 & #11
[ ] Evaluate input
[ ] Pass input to search controller

input added


python/tests: fix some pylint errors

The lint errors can be found here https://github.com/intelzhongjie/cc-trusted-vmsdk/actions/runs/8153232608/job/22284197488


Add a note about ip address in getting started page

Add a note about ip address in getting started page

/lgtm but it's not a good practice to open too many PRs for a tiny change....
Webpage edit is only for freshman -_!


Introduce RowTable and ColTable transforms

For row table inputs, will the output of a transform necessarily have rowaccess?  Is there a general principle guiding the form of the output in general? Can I arrange that the ouput is Tables.materialized?
In MLJ  we generally materialize based on input type, but is has been argued that an option to specify the output type would be useful. I'm wondering what the authors of  TableTransforms.jl think about such questions.

This is a good question. We currently materialize the same type of the
input assuming that users chosen that type for a reason. We could even
introduce transforms that just change the representation to row or column.
I think it addresses two cases.
Em qui., 28 de abr. de 2022 19:28, Anthony Blaom, PhD <
@.***> escreveu:

For row table inputs, will the output of a transform necessarily have
rowaccess? Is there a general principle guiding the form of the output in
general? Can I arrange that the ouput is Tables.materialized?
In MLJ we generally materialize based on input type, but is has been
argued that an option to specify the output type would be useful. I'm
wondering what the authors of TableTransforms.jl think about such questions.
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaML/TableTransforms.jl/issues/62, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3ISWMQ2UDNC3QNMF5TVHMGJFANCNFSM5UT6D54Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@ceferisbarov would you like to git this issue a try? Basically, we need two structs RowTable and ColTable that apply the function Tables.rowtable and Tables.columntable lazily in pipelines. For example, something like:
struct RowTable end

apply(::RowTable, table) = Tables.rowtable(table), table
revert(::RowTable, newtable, cache) = cache

Let me know if you are interested, it should be a good exercise to keep practicing the GitHub PR model.

@juliohm Yes, absolutely. Thanks for letting me know.


Rename evalaution.rs to accumulaion.rs

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.68%. Comparing base (f99ca0d) to head (3ccd166).

Additional details and impacted files
@@                       Coverage Diff                        @@
##           spapini/03-15-Move_SecureColumn     #479   +/-   ##
================================================================
  Coverage                            95.68%   95.68%
================================================================
  Files                                   59       59
  Lines                                 8835     8835
  Branches                              8835     8835
================================================================
  Hits                                  8454     8454
  Misses                                 330      330
  Partials                                51       51

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

Merge activity

Mar 21, 7:23 AM EDT: @spapinistarkware started a stack merge that includes this pull request via Graphite.


Test Analysis: Searching for Faults in Tests (N)

http://ieeexplore.ieee.org/xpl/articleDetails.jsp?arnumber=7372004&punumber%3D7371449%26filter%3DAND(p_IS_Number%3A7371976)%26pageNumber%3D2

Need to mail authors, link isn't working.
www.cse.unl.edu/∼waterloo/ase2015

Got the dataset after mailing the authors


User Namespace in RegisterController is still use App\User instead of use App\Models\User

Laravel Version: 7
PHP Version: 7.2.5
Database Driver & Version:

Description:
RegisterController User Namespace is still use App\User instead of use App\Models\User
Steps To Reproduce:
create new laravel app then run --auth

Why would you expect it to use the App\Models feature? That's a Laravel 8 thing.


For #613 - Updating NAIC components dependency to support responsive …

…table font sizes and styles. Adjusting deadweight table cmd column size to fix horizontal scroll issue in desktop screens.

I guess I'd like to see a demo before approving, though I'm sure the code is fine as such.

(Or at least screenshots.)


chore: switch from xz2 to liblzma

Closes #132

@ctron Thanks for the PR an the explanation in #132


Support for the css prop

It seems the css prop is not supported.
I am using bare bones setup found here:
https://styled-components.com/docs/tooling#for-stylelint-v15-recommended
I can verify that the css prop is not being supported here. It does work if you use the css function with the css prop but that is just redundant.
It would be quite nice to have this feature—is it possible to support this?

I think technically it is possible to add support, but I'm not sure if it's a good idea. css prop in this form seems to be used only in styled-components library. And I have a feeling css props is not used that much. At least from my experience.

I think technically it is possible to add support, but I'm not sure if it's a good idea. css prop in this form seems to be used only in styled-components library. And I have a feeling css props is not used that much. At least from my experience.

Fair enough. Thanks for the reply.


[Bug]: Modern build failed when the buildPreset is npm component

版本信息
system： windows11
version: 2.34.0
pnpm version: 8.9.2
node version: 16.16.0

问题详情

复现链接
https://github.com/web-infra-dev/modern-js-examples
复现步骤
1.下载modern-js-examples
2.到basic-module下，将buildPreset改成npm-component
3.pnpm build 之后报错

Hi, can you try  these commands in a new project?  https://ast-grep.github.io/guide/api-usage.html
pnpm add @ast-grep/napi

import { js } from '@ast-grep/napi';

const source = `console.log("hello world")`
const ast = js.parse(source)
const node = ast.root().find('console.log')

If the code example works, I would speculate the problem is pnpm.

Can you list the content inside node_modules of https://github.com/ast-grep. Probably we need all directories inside there.
Example screenshot

This example also throws the same error.

Can you try listing the files under node_modules? Especially @ast-grep.

I noticed you are using MinGW, can you try it without using MinGW?

If there are no other issues, this will be closed. @fengjiling123


How to compile this project on Windows?

How to compile this project on Windows?
If possible, put a guide or trick to compile the project on Windows? Without using Docker.

I didn't experiment with compiling it natively on Windows. You can, however, use WSL (Windows Subsystem for Linux) with Ubuntu:
Compile curl-impersonate using the Dockerfile
docker build -t curl-impersonate-chrome chrome/

then copy /build/out/curl-impersonate and /build/out/curl_chrome99 from the docker image (i.e. by creating a temporary container and using docker cp).
The binary works well on WSL (tested with Windows 10 and the Ubuntu app)

I've opened this https://github.com/lwthiker/curl-impersonate/issues/37 to note the requirement of a native Windows build. So I'm closing this issue for now.


ACM-2286 Key ansible templates on index only (same template repeated allowed)

Signed-off-by: Kevin Cormier<EMAIL_ADDRESS>

/cherry-pick release-2.6

@jeswanke @RobDolares I propose that we override the test coverage requirement for this PR. Many of the uncovered lines are in code that is anticipating future support of scale and destroy operations for the ClusterCurator. That is not currently supported and is disabled.

/cc @jeswanke

/cc @RobDolares


Feature/43 angular velocities bias

This is a draft. The below description will be written properly later.
Main Changes
Associated Issues

Closes
Addresses #43

@gunturiCM Should we close this?

@gunturiCM Should we close this?

Yes, professor.


Create Bipartite graph.cpp

Description of Change

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

This seems to be a duplicate of graph/is_graph_bipartite.cpp.


antivirus alert

Issue Type: Feature Request
when i run my c++ code in terminal then the code run smothly but affter a few seconds my antivirus alert me with a warning of virus in the file where my code is . please check it
VS Code version: Code 1.55.0 (c185983a683d14c396952dd432459097bc7f757f, 2021-03-30T16:01:55.261Z)
OS version: Windows_NT x64 10.0.19041

Sounds like this is a problem with your antivirus software, not VS code


Update submodule ocr-fileformat

Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

This update is missing in the latest release of ocrd_all.


Update LNDmanage

There is a new version available - see https://github.com/rootzoll/raspiblitz/issues/2031#issuecomment-840413361 .. it makes sense to update the version in the script for v1.7.1 and then if all is OK and there is further user request re-add it to the SSH menus.

Did update to new verison, install itself worked - but throwing errors when trying to start. WIll keep branch 2314-lndmanage open but to recheck on later release if nobody fixing this.
(venv) admin@<IP_ADDRESS>:~/lndmanage ₿ lndmanage
Traceback (most recent call last):
  File "/home/admin/lndmanage/venv/bin/lndmanage", line 5, in <module>
    from lndmanage.lndmanage import main
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/lndmanage/lndmanage.py", line 18, in <module>
    from lndmanage.lib.fee_setting import FeeSetter, optimization_parameters
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/lndmanage/lib/fee_setting.py", line 12, in <module>
    from lndmanage.lib.forwardings import ForwardingAnalyzer
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/lndmanage/lib/forwardings.py", line 9, in <module>
    from lndmanage.lib.node import LndNode
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/lndmanage/lib/node.py", line 15, in <module>
    import lndmanage.grpc_compiled.lightning_pb2 as lnd
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/lndmanage/grpc_compiled/lightning_pb2.py", line 33, in <module>
    _descriptor.EnumValueDescriptor(
  File "/home/admin/lndmanage/venv/lib/python3.9/site-packages/google/protobuf/descriptor.py", line 755, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

TODO: merge #3575 & test for v1.9.0rc2

Updated lndmanage  to v0.15.0 - installs OK but I run on my test machine into timeouts.
This should be tested by community when 1.9.0RC5 or even after final 0.19.0 release.

Check if new release works:
https://github.com/bitromortac/lndmanage/releases/tag/v0.16.0

lndmanage 0.16.0 looks good - open PR #4612 - recommend for final testing


Fix incremental rollouts backwards compat upon app updates

Task/Issue URL: https://app.asana.com/0/488551667048375/1205213514616979/f
Description
Incremental rollouts did not maintain backwards compatibility upon app update
Bug repro steps:

[ ] build and install play debug build from 4e1221a99 (before incremental rollouts)
[ ] verify "Autofill logins" appears under app settings
[ ] build and update play build from 11b2c9abc (incremental rollouts)
Expected:

[ ] "Autofill logins" are present under app settings

Actual

[ ] "Autofill logins" are NOT present under app settings

Steps to test this PR
Bug repro steps:

[x] build and install play debug build from 4e1221a99 (before incremental rollouts)
[x] verify "Autofill logins" appears under app settings
[x] build and update play build from 11b2c9abc (incremental rollouts)
[x] verify "Autofill logins" are no longer present under app settings (BUG)
[ ] build and update play build this branch
[ ] Verify "Autofill logins" are present under app settings

Current dependencies on/for this PR:

develop

PR #3419   👈

This comment was auto-generated by Graphite.

This does not work for a fresh install. A fresh install on this branch results in autofill disappearing again.
(upgrade path works ok as you've described in the testing steps).

Addressed the comments and fixed (and added specific test) for the fresh install and subsequent update


Ability to customize theme/design

Describe the feature request
It would be great to have the ability to customize the theme/design via custom CSS
Why do you personally want this feature to be implemented?
This can be a simple options in yaml that let users add some custom CSS then this will be injected to the footer of the HTML output.
How long have you been using this project?
1 mo
Additional information
N/A

+1, just wanted to say realy like this project so far 🚀, just the css mould need some light tweaking and it would be perfect

@sparanoid ohh nice, dont know any next/react tho lul, my gota do somting like that, but would still be cool to have a way to add some minimal css to the standart site

FYI custom CSS is now on my radar. I'm not sure how I'm going to implement it yet, but a rough idea of what I have in mind right now is that I want to make it so people can choose from multiple "themes", all of which is essentially just pre-written custom CSS under the hood, and alternatively, the ability for people to just provide their own CSS to design it as they see fit.
I've been wanting to redo Gatus' entire UI for a while now, but I just haven't had the time.
Also @sparanoid that looks amazing!

Implementation will be far simpler than what I initially envisioned. For now, at least.
Here's what the configuration will look like:
ui:
  custom-css: |
    .endpoint-group {
      background-color: blue !important;
    }
    .endpoint-group-content {
      background-color: red !important;
    }

Using the example above, here's what the dashboard would look like:


Add publishing of a docker image

Solves #125

closes #119

@primetheus auto complete got me there haha
What's with the linting? Doesn't it like changes in the GitHub actions files only?

@Robbilie yea, I'm probably going to pull out the linter and just use Black instead... the Super Linter is a bit overkill for this project


Add disable_http_keep_alive option

Added disable_http_keep_alive option
The default value for disable_http_keep_alive is false
When disable_http_keep_alive is true, mkr requests mackerel without KeepAlive

Could you marge master branch onto your appare45:feat-disable-keep-alive? CI failure may be fixed.

Could you marge (or rebase) master branch onto your appare45:feat-disable-keep-alive? CI failure may be fixed.

@rmatsuoka Thank you, I've rebased it and passed the CI

Have you tested the modified mackerel-agent and confirmed whether it sends metrics to Mackerel when disableKeepAlive is set to both true and false? If not, please test it.

Have you run the modified mackerel-agent and confirmed whether it sends metrics to Mackerel when disableKeepAlive is set to both true and false? If not, please test it.

I've captured the packets between mackerel api and mackerel agent with mitmproxy.
Without disable_http_keep_alive (Default behaviour)
Since there is no connection header, HTTP/1.1 (mackerel api) reuses the connections ^1.

With disable_http_keep_alive
Since Connection header is set to close, the connection won't be reused.


Add QuickLook integration

Is your feature request related to a problem? Please describe.
It's not a problem, just an improvement. There's a popular app called QuickLook (https://github.com/QL-Win/QuickLook) that mimics the Quick Look functionality from MacOs and it'd be awesome if you could integrate pressing the space-bar on a file to trigger QL.
Describe the solution you'd like
Pressing space-bar with a file selected, triggers QuickLook to show a preview of said file.
Describe alternatives you've considered
Nothing other than using the normal WIndows File Explorer.
Additional context
It'd look something like this:

Files UWP should already be integrated with QuickLook, can you check if you see a QuickLook item when right clicking on a file or folder?

Oh wow I had no idea! But no, there's no "Quicklook" item and pressing space-bar does nothing. I'm using the Windows Store version of Quicklook, could that be the problem?

I have the store version (<IP_ADDRESS>) and QuickLook is still working for me.

For me QuickLook does not work as well.

Does this feature has to be enabled somehow?

Hope this will work soon 😐

@lampenlampen I think this issue is a bug introduced by the changes to the full-trust process to add support to the recycle bin. The StartupTasks is called too early while the full-trust process is not yet running. I'll see what to do.

This issue has been fixed with #1061 and will roll out to users when v0.11 is released.


Not loading dungeon interiors for any modded maps

Currently, when I try to navigate to a modded moon the logger prints an error "Did not find local config for ${level.name}({level.PlanetName}). Did the configuration files generate properly?" and the interior that I have set for the map doesn't load.
Inside of LethalQuantities/Global/Configuration.cfg I have DungeonGenerationEnabled set to true for each map individually. Everything else in the .cfg is false. I have weights set to each planet's DungeonGeneration.cfg and everything else is default.
I deleted the mod, cleared the configuration folders, and reinstalled to remake the config files from scratch and still got the error on every modded map.
This is the first issue I have posted before so lmk if you need any more info. It works as intended on vanilla maps.

Thanks, this is an issue with LethalLevelLoader changing the object names of custom moons which made it not possible for LethalQuantities to identify the moons based on name. This will be fixed next update.


Minor: Improve documentation about OnceAsync

Which issue does this PR close?

Closes #.
Rationale for this change
@comphead  said this would be amazing :) : https://github.com/apache/datafusion/pull/13203#discussion_r1825932243

Would be amazing to clarify what is OnceAsync and why clone is not fit for it?

So I tried to improve the documentation
What changes are included in this PR?
Improve documentation
Are these changes tested?
N/A

Are there any user-facing changes?
No, just comments

Thanks @alamb that was pretty interesting to read. So this is sort of memory barrier where multiple consumers can hold the resource but its released whenever first consumer send OnceAsync?

I think it is more like the shared rendezvous point for all the output streams -- aka to start producing data out of each partition, the left side is read in entirely. The way the output streams wait for the left side to be read is by waiting on the OnceAsync -- and once it resolves all the output streams get a (read only) copy of the data to work on

Perhaps @korowa  has some additional suggestions on how to improve the wording or something


Forgot a dangling experiment group in groups.ts

🤦
For #15530

Codecov Report

Merging #15651 (658ea14) into main (a1911a5) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #15651   +/-   ##
======================================
- Coverage     64%      64%   -1%
======================================
  Files        557      557
  Lines      26813    26810    -3
  Branches    3901     3900    -1
======================================
- Hits       17369    17366    -3
  Misses      8710     8710
  Partials     734      734

Impacted Files
Coverage Δ

src/client/common/experiments/groups.ts
100% <ø> (ø)


Moon phases backwards

I just started scanning the lunar phases a few nights ago, starting from a new moon, looking for Waxing Gibbous notes.  After a few nights, I finally noticed that the phases that should be labeled "waxing" were labeled "waning".  As far as I can tell, this is just a naming issue, as the notes match the moon's appearance when it rises.

Looking at a quick google search on waxing vs waning, I think that it'd be fair to say that this is partly Minecraft's fault, for having the moon's "spin" perpendicular to its "orbit".

duplicate #759


Django 2.0 error

After upgrading django to 2.0, I get this error:
File "/home/x/y/.venvs/z/lib/python3.6/site-packages/vote/models.py", line 31, in Vote
    content_type = models.ForeignKey(ContentType)
TypeError: __init__() missing 1 required positional argument: 'on_delete'

https://docs.djangoproject.com/en/2.0/ref/models/fields/#django.db.models.ForeignKey

class ForeignKey(to, on_delete, **options)[source]¶
A many-to-one relationship. Requires two positional arguments: the class to which the model is related and the on_delete option.

Any chance of a new release on pypi, then we can close this? Thanks!

Done django-vote==2.1.6


Fix for issue #1722. Fix for time picker display covering the entire display and not showing the clock.

Time Picker issue screenshot:

Date picker issue screenshot:

Please go to PR #1753


Is there a way to make the reload faster?

I turned off source maps but then the time saved goes into debugging.

In the last commit I've splited the app code and now reload more faster. Try it


Enable OpenAI Realtime API

Describe the feature or improvement you're requesting
Would love to have openAIs realtime API which uses websockets added to the elixir library.
API description: https://platform.openai.com/docs/guides/realtime
You can see an example client in JS here: https://github.com/openai/openai-realtime-console?tab=readme-ov-file
Additional context
No response

@grahac I took a look at the spec, and I'm wondering about your use case. If it's a website where the user is on a browser anyway, does the elixir app need to do anything beyond keeping the API key secret and forwarding websocket traffic both ways (assuming this is architecturally feasible)?

Yeah my use case is likely going to hook it up to a phone line or web browser by linking the websockets.So maybe the answer is just to use a relay server and go direct HTTP calls?

@grahac you could do relay server with Elixir, I tried it yesterday, worked fine.
Or
You can call the OpenAI Realtime API directly in your Elixir app using something like Websockex.
You can see an example of connecting to the OpenAI Realtime API using Websockets here: https://github.com/francoabaroa/openai_realtime_ex/blob/main/README.md.
It's still a WIP, but for now it successfully connects and establishes the connection with OpenAI.
Just need to add support for capturing voice and sending it to OpenAI, getting the response and playing it back. To be updated in the coming days.


When do change unstable feature to stable?

Now, async-std have many unstable function and module.
When do change unstable feature to stable?

@k-nasa that's an excellent question! -- There's no concrete plan in place, but I think we can review it on a case by case basis. In general I'd think we could do the following:

Stream combinators: we can probably stabilize most, if not all of these

We can probably stabilize more of streams as well. Probably the only things we'd need @stjepang to take a look at are FromStream / IntoStream since those have some rough edges.

Print macros: we should probably remove all of these. They're not safe, and not very useful either.
Pin submodule: we should remove this.
Process submodule: the new runtime would resolve this. Probably keep this as unstable? We can also remove, and later decide to add back.

How does this sound? If you'd like to make PRs for this, I'd be happy to review. It'd be great to put out a new async-std release this month, and stabilizing things sounds like a great idea!

@yoshuawuyts

I agree with the stream combinator. Almost stable.
I think too print macro delete
I think unnecessary re-export std module. So I think it should delete.

I'll submit remove macro and re-export std module. Please review!

@k-nasa sounds great; will do!

@yoshuawuyts @k-nasa:
I have some concerns about traits stabilized in #719:

Known issue of FromStream (#639): Fixing this may require breaking changes.

Other traits that use dyn trait need to consider similar issues to FromStream. (Extend, Sum, Product)

(AFAIK) async-std's FusedStream does not work without specialization, unlike futures's FusedStream. And, currently, async-std doesn't provide support for specialization. So, I don't think this needs to be stabilized immediately.

DoubleEndedStream does not use extension trait pattern, unlike existing traits. This is a small trait, but note that this may prevent providing more granular options in the future. (e.g., provide only traits without providing utilities. This is what futures and tokio currently do.)

These seem to be stabilized with little discussion in #719 and this issue. So, I suggest that we revert the stabilization of these traits (or all traits stabilized in #719. there may be more issues, given that I haven't spent much time looking for this...).

@taiki-e thanks! I think these concerns should block.


chaos(engine): pin version of busybox to avoid dns lookup failure

What problem does this PR solve?

Issue Number: close #7845
What is changed and how it works?
Pin docker image version of busybox in server-master/executor to fix nslookup failure
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

chaos test passed: https://github.com/pingcap/tiflow/actions/runs/3644959064

/merge

/run-verify

/run-verify
/run-integration-test


How to fix compilation errors when not building for web?

I have a flutter app that should support mobile and web, but compilation fails when building for mobile with errors such as "Dart library 'dart:js_interop' is not available on this platform.".
In my code I import dart:io and web/web and use kIsWeb to call IO methods or web methods.
When compiling for web, importing dart:io does not cause any problem.
I've read about creating a stub file and importing io or web implementations, but my code is sprinkled with kIsWeb checks with tiny bits of code.
Is it really the only way to make this work? How come importing dart:io is fine but web/web is not?
Could this package provide a stub so we can write something like this:
import 'package:web/web_stub.dart' as web
  if (dart.library.js_interop) 'package:web/web.dart';

kIsWeb separates code logic so that you only call web APIs when compiling to web and vice versa. In this case, a stub could work, but checking for kIsWeb everywhere is quite clunky. Instead, I'd stub out the platform-specific logic and then call those stubs.

When compiling for web, importing dart:io does not cause any problem.

Can you elaborate? Are you using a stub there to avoid the unsupported error?

kIsWeb is convenient when only a tiny portion of your code is dependent of the platform. Without this I have to create 3 files (stub, io, web) and create a public function for just 3 lines of code in each platform, instead of writing if(kIsWeb) doWebStuff else doIOStuff. It's cumbersome to implement and maintain. So I wanted to know if there was another way.
If I import dart:io, to use Platform.isAndroid for instance, this compiles fine for web and I can call the io code conditionally at runtime. If I import web/web is does not compile for io, which prevents to use kIsWeb to branch platform-specific code as described above. Actually is seems to be more dart:js_interop that complains, but since web/web imports it, it is basically the same.
So if I ever need to import web/web, I have to go the stub way, which is overkill in my opinion in several instances in my code.
I'm not very familiar with all this: I've just stumbled upon this kind of problem recently. So if you tell me there's no way to import web/web without going the stub way, that's fine.

@srujzs I am having the exact same issue.
Even though my code logic is wrapped around if (kIsWeb), just by having the import
import 'package:web/web.dart';
it causes my project to stop compiling to native platform targets.
Just like @gdurandrexel suggested, the solution in this case was to create a stub file and add a conditional import.
import 'package:web_stub.dart'
  if (dart.library.js_interop) 'package:web/web.dart';

I would be really nice to have this stub file created inside the package so it doesn't affect native platform targets or maybe just add this info to the package's home page, just in case someone else has the same problem.

Without this I have to create 3 files (stub, io, web) and create a public function for just 3 lines of code in each platform, instead of writing if(kIsWeb) doWebStuff else doIOStuff.

I may be wrong, but I think you only need two stubs - one for the VM and one for web and then just do a conditional like you were thinking of for package:web:
import 'vm_impl.dart' as impl
  if (dart.library.js_interop) 'web_impl.dart';

So if you tell me there's no way to import web/web without going the stub way, that's fine.

Ultimately if you want to avoid creating your own more general stubs and want to still use kIsWeb, a stub of all of web is the only remaining solution. The VM will indeed complain you're importing a platform-specific library, because that's what package:web is.

I would be really nice to have this stub file created inside the package so it doesn't affect native platform targets or maybe just add this info to the package's home page, just in case someone else has the same problem.

It's a good idea to put this in the README or an FAQ. I'm not completely opposed to a stub - it just feels like the wrong solution when you're comparing with much of the package ecosystem.

Yes, you're right, I need only 2 files as you explained. I had come across several examples with another stub file and at that time I was still in my copy/paste phase regarding this new kind of problem.
Yes, a full stub of web/web may work, and I proposed this solution because the code is generated, so it might not be too much work or need too much maintenance (though it may, I have no idea). There would still be the problem of dart:js_interop that surfaces in the API. If we cannot apply the same technique for this package also, then there is no point in creating a stub for web/web I suppose.

If we cannot apply the same technique for this package also, then there is no point in creating a stub for web/web I suppose.

We may be able to get away with typedefs, but I'm not sure. We definitely do not want to either do a huge breaking change to web or stub dart:js_interop on the VM.

Is there no way to override dart:* package paths, similar to passing a path to an overridden package_config.json?

Is there a way to override dart:* package paths? Similar to passing a path to an overridden package_config.json.

Nope. dart: libraries are hard-wired. What do you want to override? (Also, this is a bit off-topic)

@kevmoo I mean that we could use it to provide paths to the patched web and js_interop packages, if that’s possible.

you can override web, but js_interop is baked in.


Validate that input adheres to openapi spec

It seems that the post data is not validated against the openapi specification.
Is there a possibility to enable data validation against the specification?
A 400 BadRequest should be returned in case the data does not adhere to the specification.

Okay just saw that request validation is currently not supported.
Is this something that is planned?
Which openapi generator is used here?
The ones I know are usually generation input validation for server stubs.

It seems a simple action to do by modifying the code at openapi validator
https://github.com/muonsoft/openapi-mock/blob/master/internal/openapi/handler/ResponseGeneratorHandler.go#L54
It'd be nice to give user an option to choose being strict or not. If yes, then validate request body, if not, ignore request body


Delete TODO, move to wiki

Closes #1676
https://github.com/vhf/free-programming-books/wiki/TODO-list

Thank you!


Update readme to show new development repo

read the readme, but you never really voiced your opinon on it @nokonoko so you can decide here ;p

https://github.com/pomf/pomf should be a link.

whoops forgot to do a commit message, but fixed now

I'll accept it in a while

21 juni 2016 kl. 22:41 skrev Austin<EMAIL_ADDRESS>read the readme, but you never really voiced your opinon on it @nokonoko so you can decide here ;p
You can view, comment on, or merge this pull request online at:
https://github.com/nokonoko/Pomf/pull/49
Commit Summary
Update readme to show new development repo
File Changes
M README.md (50)
Patch Links:
https://github.com/nokonoko/Pomf/pull/49.patch
https://github.com/nokonoko/Pomf/pull/49.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Consider using fat-rs

We should consider switching to FAT-rs (https://gitlab.com/susurrus/fat-rs/tree/master). No sense in duplicating work, if it:

Works on #[no_std]
Builds on stable
Doesn't use alloc

(https://github.com/rafalh/rust-fatfs fails this test because it uses core_io, which needs nightly, and uses alloc)

From fat-rs's README:

Eventually this will be migrated to use core_ios traits in order to be compatible with other libraries in the nascent no_std Rust ecosystem.

Looks like they might be moving towards core_io, too.  Also, perhaps this is an ignorant question, but is it odd that these crates are moving towards core_io which requires alloc?  Isn't the idea with embedded to avoid allocation whenever possible, meaning the utility of these crates becomes more limited by using implementations that require allocation?
Finally, I haven't been able to find a published crate on crates.io for fat-rs - perhaps I'm just missing it? 🙂

Please note https://github.com/rafalh/rust-fatfs requires alloc only if you need LFN support.

We have write support now. But we should keep an eye on what other crates are doing.


Fix broken link for "Manage data in containers"

File: engine/tutorials/index.md, CC @mistyhacks
The link pointed to by the text "Manage data in containers" is incorrect.
It currently points to "https://docs.docker.com/engine/tutorials/dockervolumes/" which does not exist.  I suspect it should be pointing to "https://docs.docker.com/engine/admin/volumes/volumes/" instead.

The first link actually redirects to the second, so it is working as intended. Be sure Javascript is enabled and clear your cache, and re-open this issue if the link still does not work for you.


Update PipelineRun details test

Closes: RHOAIENG-4729
Description
This PR validates that pipelineRun details right drawer (Input/output, details) have tabs with data, and that the bottom drawer (run output) has tabs with data. Action drop down tests covered in separate PR 2746
How Has This Been Tested?
npm run test
Test Impact

Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/lgtm


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_882e1e46-23af-4a8c-8cec-8f94f4b7e765/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_882e1e46-23af-4a8c-8cec-8f94f4b7e765.git

Docs Build status updates of commit 9e91a39:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-08-18-06-15-12 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-08-18-06-15-12

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


Implemented pybricks protocol v1.4.0

Adding pybricks protocol v1.4.0 handling as a first step on to merge hubcenter feature in #2315.

Hi! I wanted to check in and see how the review is going on this PR. If there’s anything specific I can assist with or if you have any feedback, just let me know.

There was still quite a bit to do and I had some time to work on it, so I went ahead and fixed it up.

Thanks both of you!


Update pt-colorpicker-editor.gif

Updated the colorpicker gif to reflect recent UX changes
New gif:

cc @dedavis6797

@niels9001 is this a 37.2 adjustment or .39+ adjustment?

Still to come I believe.. this was the PR updating the UI: https://github.com/microsoft/PowerToys/pull/11384

Merging as this update was reflected in the v0.41 update. Thanks @niels9001!


JSON linter not commenting on the correct line

$ cat test.json
{
  "one": "value",
  "two": 2,
  "three": "three",
}
$ jsonlint test.json
test.json:5:0: Warning: Strict JSON does not allow a final comma in an object (dictionary) literal
   |  At line 5, column 0, offset 52
   |  Object started at line 1, column 0, offset 0 (AT-START)
test.json: ok, with warnings

If "three": "three" was added in the PR, lintreview wouldn't comment on this error, as it reports the error 1 line down.
I'm not sure what the solution so this problem is.

I guess this is a JSON issue, and not a jsonlint issue, but maybe we should switch to the npm-installable one:
± cat test.json
{
  "one": "value",
  "two": 2,
  "three": "three",
}
± jsonlint -c test.json
test.json: line 4, col 19, found: '}' - expected: 'STRING'.

I can't see public access to the source for demjson or a public issue tracker either.

Seems legit:
$ jsonlint -c has_errors.json
has_errors.json: line 1, col 0, found: 'INVALID' - expected: 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '['.
$ jsonlint -c has_warnings.json
has_warnings.json: line 2, col 8, found: 'INVALID' - expected: 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '['.
$ jsonlint -c no_errors.json
{
  "one": "value",
  "two": 2
}```

I take that back.
The errors are definitely not as pretty, and it doesn't have have warnings. The npm one hasn't been updated in two years.
± jsonlint -c -s tests/fixtures/jsonlint/has_warnings.json
tests/fixtures/jsonlint/has_warnings.json: line 2, col 8, found: 'INVALID' - expected: 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '['.

± virtualenv/bin/jsonlint tests/fixtures/jsonlint/has_warnings.json
tests/fixtures/jsonlint/has_warnings.json:2:9: Warning: String literals must use double quotation marks in strict JSON
   |  At line 2, column 9, offset 11
tests/fixtures/jsonlint/has_warnings.json:3:12: Warning: JSON does not allow identifiers to be used as strings: u'three'
   |  At line 3, column 12, offset 32
tests/fixtures/jsonlint/has_warnings.json:3:28: Warning: Strict JSON does not allow a final comma in an object (dictionary) literal
   |  At line 3, column 28, offset 48
   |  Object started at line 1, column 0, offset 0 (AT-START)
tests/fixtures/jsonlint/has_warnings.json: ok, with warnings

Looking at how Yelp's pre-commit does checking if JSON is correct and compares it to pretty formatted JSON, maybe we should do this ourselves?
Though I guess as a standalone library?

@zoidbergwill If you build a tool that can be integrated with lintreview, we could switch tools :smile:


sql error

$builder = $db->table('mytable');
$builder->set('field', 'field+1', FALSE);
$builder->where('id', 2);
$builder->update(); // gives UPDATE mytable SET field = field+1 WHERE id = 2
($builder->set('field', 'field+1', FALSE);)  this is bug !!!!
error:  UPDATE mytable SET field = 'field+1' WHERE id = 2

Yeah, those situations have always been painful for CI's Query Builder IIRC. I got to where I was writing those queries manually.
However, there's new increment() and decrement() methods in Query Builder you can use instead:
$db->table('mytable')
    ->where('id', 2)
    ->increment('field', 1);

Note that these do run the query right then, though...


PYIC-1867 Update Welsh and English content after content audit

Proposed changes
Content in the English and Welsh yaml files are updated to improve the user experience
What changed

Welsh content is corrected and typographically correct apostrophes are added where necessary to English and Welsh content.
Why did it change

Updates were prompted by the review of the Welsh content by the DWP Welsh content team.
Issue tracking

PYIC-1867

Checklists
Other considerations

[ ] These updates do not change currently visible content except for the typographically correct apostrophes in pages.yaml.

I have tested this locally and looks good


Fix fuzzydate string matching logic

I'm seeing a bug in the fuzzydate function when uploading a CSV which includes a cell with the value THIS IS A BUG 90 - it gets parsed to 1 January 1990. It looks to me like the update in commit 5b67ac0 to fix #772 altered the logic as to what constituted a date, but should also have swapped the returned values to reflect that updated logic.

Good catch!


Deprecate this package

As I understand this package is not maintained anymore (and the author didn't care to find a new maintainer). Still the number of stars and npm downloads can trick a user to consider this to be a reliable library.
Its a good idea for the author to mark the package as 'deprecated' and recommend another (maintained) package instead.

@yentsun would you have any suggestions of such an alternative package? I'm quite interested primarily because I want something that is treeshakeable, as it stands importing country-data in a project creates a pretty hefty bundle (116 kb when a treeshakeable bundle could result in only getting <= 10 kb of country data).

@Favna This https://www.npmjs.com/package/i18n-iso-countries. Probably an overkill as it provides country info in several languages but I use it on backend (no bundle) and its maintained well.
And you're right - a project like this could be done in a weekend if nothing else is suitable.


Create a Dashboard to monitor outstanding PRs

We can use the Metrics code to fetch data on outstanding PRs for the opensafely repo. This could be used to monitor whether the PRs we're raising for devcontainer configuration changes are being merged or not.
We should create a dashboard, similar to the existing PRs dashboard.
Requires #47 to be completed first

We aren't planning to automatically create PRs any more, so I'm closing this ticket.


Support Closure Compiler generic types

Closure Compiler's docs now include a section on generic types, which are defined using the @template tag.
We should implement this tag, using the same semantics as Closure Compiler.

Fixed on master.


Test is QT wallet show correctly message "You get coins"

Please test with all transports that qt wallet correctly shows the message that some coins was received.
Any OS is fine

Message shows: Info: Congratulations! you received xxx MWC.

Hence closing this ticket.


rmdirSyncRecursive silent fail

Hi there. I have a case where I'm using wrench.rmdirSyncRecursive('my/folder'); as part of a build phase for deleting a folder that may or may not exist. If the folder doesn't exist, wrench blows up with a no such file or directory error. I can easily surround the use of wrench with a try/catch but even slicker would be a 'failsilent' option or something. wrench.rmdirSyncRecursive('my/folder', true); Thoughts?

I'm totally down; if you wanna patch/pull request, I'll gladly merge it in. ;)


[Feature]: Trace viewer log type actions

🚀 Feature Request
It can be great to have the possibility to add "log" action to the list of actions on the trace viewer to better following the tests execution and so easy readiness. It can also facilitate debugging.
Example
For example, if we are writing a Junit integration test like that.
myService.createOrder(new order)
myService.addOrderLine(myOrder, new line1)
myService.addOrderLine(myOrder,new line 2)
myApp.jmsPublishOrder(myOrder)
page.navigate(toPublishedOrder)
assertThat(page.locator(myOrderId).isVisible()
myApp.jmsValidateOrder(myOrder)
page.navigate(toValidatedOrder)
assertThat(page.locator(myOrderId).isVisible()
myApp.doSomeMoreTimeConsimingActions()
page.navigate(toOrderFinised())
===========>
It can be good to see:
Log Preparing new order 1
Log Send order
Page.navidate toPublishedOrder
Locator.expect myOrderId
Log Validate order 1
Page.navidate toValidatedOrder
Locator.expect myOrderId
Log do some more time consuming actions
page.navigate(toOrderFinised())
Motivation
We are using Playwright for simple gui tests and complexe automatique  integration tests. Our complexe tests can have a lot of steps and are used on a system that interact with:

Real hardware's (robots, sensor etc...),
Physicals user actions
External systems that send actions to do to the system

The web application uses a lot of Server Sent Event that update the display of the GUI.
The Playwright Traces are saved as proof of tests execution success and so it can be wery useful to be able to better follow the test and to understand which external action influence the gui display and so the Playwright navigations, actions and checks we are doing.
Workaround
Currently we are doing as workaround Page.evaluate("console.log('my log')"). It allows to understand test execution but in the list of actions we have juste Page.evaluate. The detail of the log can be seen in the detail of the evaluate or in the console.
As we have a GUI snapshot for Page.evalute, this solution as also the disadvantage of increasing the size of the Trace zip

Related request in python https://github.com/microsoft/playwright-python/issues/1949

idea:  we could add a TestStep fixture that allows this?
public void fooTest(TestStep step, Page page) {
  step.describe("Prepare new order", () -> {
    // some playwright actions
  }
}

Although this won't solve any logging any non-playwright actions but I don't think playwright currently allows that anyways.  There are other issues similar to this most recently https://github.com/microsoft/playwright/issues/30610.

Step is a good idea to add more context. For my use case, as I'm doing complexe integration tests with multiple "actors", it is important to have the possibility to add "non playwrigth" actions. I'm doing external actions that sents update GUI using SSE.
For example, i can do :

check Motor power OFF on GUI
External action, power ON Motor => new SSE with power ON
check Motor power ON on GUI

if i don't have the information of the external action, it is difficult to understand why I'm doing a check on OFF and just after ON without any apparent change:

check Motor power OFF on GUI
check Motor power ON on GUI


Compatibility with ESP8266 Node MCU

I tried to use the library but it does not compile with ESP8266 (in Arduino IDE). I would like this feature, or at least an indication of what I need to modify in order to use it with ESP8266 Node MCU. Thanks!

You can use https://github.com/adafruit/Adafruit_SSD1306 with ESP8266, have not tested this library with anything else apart from ATTiny85. If you have some specific use case, do let me know, will try to help


Naming of lists component does not match naming of onsList macro

The component and macro should take one of the following forms:
Component Name: list Macro Name: onsList
Component Name: lists Macro Name: onsLists (doesn't really make sense)

Similar to #2071

Maybe a breaking change